Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add NanoBoyAdvance v35 #5482

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

liberodark
Copy link

Build OK

@nadenislamarre
Copy link
Collaborator

i mark it for v34, ok ?

@liberodark
Copy link
Author

liberodark commented Jan 19, 2022

Yes is for v34
thank for the tag

@zognic
Copy link

zognic commented Jan 19, 2022

great sound improvement, very good alternative for GBA Emulation

@nadenislamarre
Copy link
Collaborator

conflicting ;-(

@liberodark
Copy link
Author

Fixed @nadenislamarre

@nadenislamarre
Copy link
Collaborator

needs to be patched to avoid hacks like
copy_tree(nba_Config, nba_Homedir)

features: [padtokeyboard, decoration]
shared: [hud]
is no more valid.

features decoration must not be used anymore. (due to very recent changes)

@nadenislamarre
Copy link
Collaborator

@liberodark why to you need to copy /usr/nba/ ?

@liberodark
Copy link
Author

liberodark commented Mar 20, 2022

@nadenislamarre It's like redream it needs to be present with its files otherwise it won't work.
Also have remove decoration and update this part.

@nadenislamarre
Copy link
Collaborator

but contrary to redream, it can be fixed while sources are here.

@liberodark liberodark changed the title Add NanoBoyAdvance v34 Add NanoBoyAdvance v35 Apr 30, 2022
@nadenislamarre nadenislamarre added 35 and removed 34 labels Apr 30, 2022
@Hew-ux
Copy link

Hew-ux commented Jun 23, 2022

Tried building this branch. Pulled it in, rebased from master, resolved the conflicts and added it to GeneratorImporter.py. Crashes when attempting to launch, here is stderr:

2022-06-23 22:21:15,006 ERROR (emulatorlauncher:564):<module> configgen exception:
Traceback (most recent call last):
  File "/usr/bin/emulatorlauncher", line 562, in <module>
    exitcode = main(args, maxnbplayers)
  File "/usr/bin/emulatorlauncher", line 97, in main
    return start_rom(args, maxnbplayers, args.rom, args.rom)
  File "/usr/bin/emulatorlauncher", line 155, in start_rom
    wantedGameMode = generator.getResolutionMode(system.config)
AttributeError: 'tuple' object has no attribute 'getResolutionMode'

Notably this seems unrelated to the NbaGenerator itself, as NbaGenerator.py correctly calls for 'getResolution' and not 'getResolutionMode'.

@Hew-ux
Copy link

Hew-ux commented Jun 26, 2022

Changes needed for the updates to configgen in v35: master...Hew-ux:nanoboy-rebased

@lbrpdx
Copy link
Contributor

lbrpdx commented Jul 4, 2022

This is conflicting for v35. What is the interest of adding NanoBoy Advance compared to mgba or vba-m? Again, we're nearing the limit of what the system can support before having everyone to re-partition again,

@Hew-ux
Copy link

Hew-ux commented Jul 4, 2022

@lbrpdx the interest I believe is more accurate emulation and less audio clipping.

@nadenislamarre
Copy link
Collaborator

copy_tree must be avoided.

@liberodark liberodark marked this pull request as draft July 16, 2022 09:17
@n2qz n2qz added the conflicts label Dec 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants