Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Feature/sbachmei/mic 5127 update standard observers #441

Conversation

stevebachmeier
Copy link
Contributor

Minor changes required when updating mnch maternal observers

Description

Changes and notes

  • reset the index in the default results_formatter to ensure that
    the stratifications make it out to the written data
  • specify .astype(float) to prevent warnings
  • use instance of CategoricalDType to prevent warnings
  • implement a get_configuration_name method for default_configurations

Testing

tests pass

@stevebachmeier stevebachmeier merged commit 5272a09 into epic/MIC-4968-results-processing Jun 27, 2024
6 checks passed
@stevebachmeier stevebachmeier deleted the feature/sbachmei/MIC-5127-update-standard-observers branch June 27, 2024 15:56
stevebachmeier added a commit that referenced this pull request Jul 8, 2024
* reset index in default results_formatter

* handle some .astype() warnings

* more observers configuration_defaults stratification name into getter method
stevebachmeier added a commit that referenced this pull request Jul 12, 2024
* reset index in default results_formatter

* handle some .astype() warnings

* more observers configuration_defaults stratification name into getter method
stevebachmeier added a commit that referenced this pull request Aug 13, 2024
* reset index in default results_formatter

* handle some .astype() warnings

* more observers configuration_defaults stratification name into getter method
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants