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

Python modeling component #1045

Closed
wants to merge 53 commits into from
Closed

Python modeling component #1045

wants to merge 53 commits into from

Conversation

alxlyj
Copy link
Contributor

@alxlyj alxlyj commented Sep 5, 2024

Project Robyn

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.

Fixes # (issue)

Type of change

  • fix: Bug fix (non-breaking change which fixes an issue)
  • feat: New feature (non-breaking change which adds functionality)
  • docs: API change with a documentation update
  • test: Additional test coverage
  • reformat: Code cleanup, recoding or reformating code

How Has This Been Tested?

  • Fork the repo and create your branch from master (latest dev version).
  • If you've changed docuemntation, run devtools::document(). Should update .Rd files.
  • Ensure all tests pass, run devtools::check(). Should have no notes, no warning, no errors.
  • Make sure your code lints.

sumane81 and others added 30 commits August 4, 2024 13:16
This is initial commit for modular design. Most of it is AI generated code from reverse engineering of old code. It need multiple iterations to clean up and finalize the interface design with clear mutable and immutable data class designs.
Goal is to finalize interface design before implementation.
* modified check_datevar

* added more interface methods to mmmdata_validation

* Added more check.py methods

* addressed comments

* added calibration_input_validation

* added check_mmmdata_collect

* added more methods

---------

Co-authored-by: Dhaval Patel <[email protected]>
…_interface

Revert "Added method signatures from check.py"
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 5, 2024
@alxlyj alxlyj requested a review from sumane81 September 5, 2024 07:36
@alxlyj alxlyj closed this Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants