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

Supporting the Max and Min value for FLOAT parameters. Supporting the Min and Max value from register and not only from formula #110

Merged
merged 6 commits into from
Jul 21, 2024

Conversation

CosbyBs86
Copy link
Contributor

Even for generic and float parameter the system should support the Max and Min values of the parameter. The PMin and PMax should also be able to support registers and not only formulas

@CosbyBs86 CosbyBs86 changed the title Supporting min and max parameter values accessing register value and not only the formula Supporting the Max and Min value for FLOAT parameters. Supporting the Min and Max value from register and not only from formula Jul 18, 2024
@CosbyBs86
Copy link
Contributor Author

Hello @Touseefelahi can you have a look on this PR? many thanks
Please let me know if I can help you further. I still have a couple of changes to submit but I would like to go step by step avoiding to waste your time.

@Touseefelahi Touseefelahi merged commit f590c64 into Touseefelahi:master Jul 21, 2024
1 check passed
@Touseefelahi
Copy link
Owner

Hello @CosbyBs86, Thank you for the PR, I really appreciate the effort.

@CosbyBs86
Copy link
Contributor Author

CosbyBs86 commented Jul 21, 2024

Thank you @Touseefelahi for approving the PR. It is a pleasure to contribute in such library. I have two more PR and then I finished for the moment. I swear :)
I’ll submit the PR tomorrow.
Have a good Sunday and let me know if I can help you further

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