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.

Sign up for GitHub

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

Jump to bottom

Add param & dim math config #712

Draft
wants to merge 25 commits into
base: main
Choose a base branch
from
Draft

Add param & dim math config #712

wants to merge 25 commits into from

Conversation

brynpickering
Copy link
Member

Fixes #642

Very much WIP - enjoy @irm-codebase !

Reviewer checklist

  • Test(s) added to cover contribution
  • Documentation updated
  • Changelog updated
  • Coverage maintained or improved

irm-codebase and others added 25 commits July 15, 2024 19:28
@irm-codebase
isolate _def_path, scenario overrides, and math object (not yet integ…
a47e1df 
…rated)
@irm-codebase
Extract math documentation from model file
d2d70ce
@irm-codebase
add model math tests
3f44bbf
@irm-codebase
Improve model math tests, remove duplicates from test_core_model
28f8ebb
@irm-codebase
extended validation function, added logging tests
2872060
@irm-codebase
Add dict method, remove underscores in attributes
31de31b
@irm-codebase
code now uses new math object (tests exected to fail)
a6e8c7c
@irm-codebase
all tests passing
fda1ea0
@irm-codebase
fix docs creation, add changelog
480bcc1
@irm-codebase
removed _model_def_dict
6e3e334
@brynpickering
Trigger CI (and minor logging string fix)
8da8dac
@irm-codebase
PR: now CalliopeMath, better backend init, small fixes
6730409
@irm-codebase
PR: comment improvements
09b09f7
@irm-codebase
update changelog
1530b8f
@irm-codebase
Merge branch 'rework-def-dict' into rework-math
41b7fe5
@brynpickering
Merge branch 'main' into rework-math
434236d
@brynpickering
Post-merge fixes
06efe33
@brynpickering
Move math to build step; fix clustering issues
a2ee8f0 
- clustering had representative days that didn't represent themselves
@irm-codebase
PR improvements: math components in CalliopeMath, small fixes (#665)
3284a55
@brynpickering
Merge branch 'main' into rework-math
a615252
@brynpickering
Initial work to add param & dim defs to math YAML
552ad83
@brynpickering
Move math to main model def
01fd4d6
@brynpickering
Move model_math to preprocessing
536d2e0
@brynpickering
Merge branch 'main' into add-param-dim-math-config
38c70ff
@brynpickering
Add lookups; dimensions -> dims; model_data -> inputs & results
a8836f0
@brynpickering brynpickering mentioned this pull request Nov 21, 2024
Draft
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@irm-codebase irm-codebase Awaiting requested review from irm-codebase

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Letting users define parameters and additional dimensions in YAML files
2 participants
@brynpickering @irm-codebase