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 function to access popParameter String values by key in Parameters class #113

Open
Jannetty opened this issue Nov 20, 2024 · 1 comment · May be fixed by #115
Open

Add function to access popParameter String values by key in Parameters class #113

Jannetty opened this issue Nov 20, 2024 · 1 comment · May be fixed by #115
Assignees
Labels
category: util Related to the util subpackages package: core Related to the core package priority: medium Non-urgent but important task type: feature Improvement or additions to the code base

Comments

@Jannetty
Copy link
Member

Right now filter is the only function you can use to access String values in the popParameters MiniBox in the Parameters class. Because filter is used to select all parameters with a certain prefix, it trims the term it searches from the key and returns a minibox. This makes it clunky for use when trying to access a single parameter value that only has one name; you are returned a minibox with one entry that has an empty string as the key and your desired value as the value.

I propose adding a function mimicking getInt and getDouble for strings. I will use this function to access the "class" population parameter from my cell constructors.

@Jannetty Jannetty added category: util Related to the util subpackages package: core Related to the core package priority: medium Non-urgent but important task type: feature Improvement or additions to the code base labels Nov 20, 2024
@Jannetty Jannetty self-assigned this Nov 20, 2024
@Jannetty
Copy link
Member Author

Update: with the changes I propose in #112, the filter function will no longer be able to return parameter values that have keys without tag separators. #115 addresses this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: util Related to the util subpackages package: core Related to the core package priority: medium Non-urgent but important task type: feature Improvement or additions to the code base
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant