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

Fix adjustments #16

Merged
merged 7 commits into from
Feb 28, 2022
Merged

Fix adjustments #16

merged 7 commits into from
Feb 28, 2022

Conversation

tomgp
Copy link
Contributor

@tomgp tomgp commented Feb 28, 2022

The adjustValue function wasn't working.
This PR...
Fixes the adjustValue function, providing the required list of indicators to consider in the calculation.
Adds the ability to reset indicator values either individually or en masse.
Adds tests so it doesn't break again.
Bumps the package version and updates the docs.

@tomgp tomgp requested a review from celltowertrees February 28, 2022 16:32
indexEntity already clones the entity
@tomgp tomgp merged commit bf285d0 into main Feb 28, 2022
@tomgp tomgp deleted the fix-adjustments branch February 28, 2022 16:59
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.

2 participants