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

Halo75 - remap is written but not applied nor saved #19

Closed
pngo opened this issue Jan 2, 2023 · 13 comments · Fixed by #20
Closed

Halo75 - remap is written but not applied nor saved #19

pngo opened this issue Jan 2, 2023 · 13 comments · Fixed by #20
Labels
bug Something isn't working

Comments

@pngo
Copy link

pngo commented Jan 2, 2023

Remaps are said to be written
image

Typing in Notepad, the mapping is not applied.

Closing nudelta, opening nudelta again, and reloading keyboard, the default keymap is shown. (proving the keymap has not been written)

Nudelta version : 0.6.3
OS: Win 10
Keyboard: Halo75
Firmware: 0102

@donn
Copy link
Owner

donn commented Jan 2, 2023

Nudelta does not read the incoming keymap from your keyboard, so reloading it will always show the default keymap.

Out of curiosity- can you try switching your keyboard to Mac mode and seeing if it remapped that one?

@donn donn added the bug Something isn't working label Jan 2, 2023
@pngo
Copy link
Author

pngo commented Jan 2, 2023

Switched to Mac mode, and the keymap works. So yes it was writing the keymap to the Mac Mode, even though it detected Windows correctly at start up of nudelta (refer back to screenshot in first post)

Checked the other way around, Nudelta Mac Mode, and write the changes. The keymap is applied to Windows Mode on Keyboard.

Conclusion: the mode selection vs writing is flipped.
Nudelta mode MAC writes to WIN keymap
Nudelta mode WIN writes to MAC keymap

@nocturne1
Copy link

Can confirm this behavior. We're almost there!

@donn
Copy link
Owner

donn commented Jan 3, 2023

…I'm confused now. I flipped it in the first place because @nocturne1 reported this same behavior on 0.6.2.

Can you both try 0.6.2? Literally the only change from 0.6.3 is that the Mac and Windows mode are swapped around. That's it: https://github.com/donn/nudelta/releases/tag/0.6.2

@pngo
Copy link
Author

pngo commented Jan 3, 2023

With 0.6.2, I have the following

  • mapping Nudelta in MAC mode=> applies keymap to MAC mode, but also the Windows keys (Ctrl, Win, Alt, Space)
  • mapping Nudelta in Windows mode => applies keymap to Windows mode, but also the MAC keys (Control, Option, Command, Space)

Are people getting confused with the M and W switch, depending on how they look on their keyboard, and thus expecting wrong OS keys?
I check the orientation of the other icons, from left to right it would be :
W M ------ OFF, Wired, Wireless ----- USB plug

@nocturne1
Copy link

nocturne1 commented Jan 3, 2023

No confusion here with the physical switch.

  • 0.6.2 physical switch in M mode, software in Mac mode. Would program right command key to right-control, and then left command key would change to windows mode (forgot what key it went to). Could only get command key back by moving switch to W.

  • 0.6.3 physical switch in M mode, software in Mac mode. Would program right command key to right-control, and then nothing would change. Moved physical switch to W mode, then I saw that the right command key would be right-control, and other keys were still windows.

0.6.2 messed up my mapping so much, I had to use Windows boot camp to re-flash the firmware (or I guess I could have used Windows mode, but I did't want to do that).

@pngo
Copy link
Author

pngo commented Jan 3, 2023

Difference in Windows / Mac builds then ?

@donn
Copy link
Owner

donn commented Jan 3, 2023

The builds are identical.

From what I'm seeing here- it appears that reverting the codebase to the state of 0.6.2 but also switching the YAML files around is probably the right approach. I'll try that when I have some time.

@Weathercold
Copy link

So the problem was that the yaml files are reversed? I tried to inform you about this a week ago but I think you didn't see it 😅 99c3872#r94643080

@donn
Copy link
Owner

donn commented Jan 6, 2023

@Weathercold You're right!! I didn't! I scrolled all the way down, didn't see anything and I was like "weird, is it a glitched notification?"

Thanks for the alert, this is simpler than I expected lol

@donn
Copy link
Owner

donn commented Jan 6, 2023

Try 0.6.4. Sorry this is taking so long…

@pngo
Copy link
Author

pngo commented Jan 6, 2023

Can confirm it works now as expected! Thank you and ill be happy to test in future as well.

@nocturne1
Copy link

Can also confirm this fixed things. Thanks @donn !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants