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

Allow negative values in config frame #916

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

rcn8263
Copy link

@rcn8263 rcn8263 commented Mar 7, 2025

Added ability to input negative config values in gui.

This is needed specifically for some emane models which have config parameters that may expect a negative value (fixedantennagain, txpower, etc.)

@siliconja siliconja requested review from siliconja and bharnden March 7, 2025 18:19
Copy link
Member

@siliconja siliconja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@bharnden
Copy link
Contributor

bharnden commented Mar 7, 2025

The main negative is potentially allowing it in cases where it is not allowed as a blanketed way, but it might be easier to do that, than deal with the nuance and let users just deal with knowing better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants