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 return type of validation after data refactoring #429

Merged

Conversation

danielhuppmann
Copy link
Member

@danielhuppmann danielhuppmann commented Sep 28, 2020

Please confirm that this PR has done the following:

  • Tests Added
  • Documentation Added
  • Description in RELEASE_NOTES.md Added

Description of PR

This PR reverts the behavior of the validate() function to the status before the refactoring of the data-backend.

  • changes the return type back to a pandas.DataFrame
  • moves all validation tests to an own file (ignore the first commit to see only the relevant functional changes)
  • refactors all related tests to explicitly test the return type
  • adds a Returns section to the docstring

closes #428

tests/test_feature_validation.py Outdated Show resolved Hide resolved
tests/test_feature_validation.py Outdated Show resolved Hide resolved
@danielhuppmann danielhuppmann marked this pull request as ready for review September 28, 2020 16:27
@danielhuppmann danielhuppmann self-assigned this Sep 28, 2020
@danielhuppmann danielhuppmann added the data back-end Anything related to the (timeseries) data back end implementation label Sep 28, 2020
@byersiiasa
Copy link
Collaborator

will likely test this on Monday/Tuesday as will run soem vetting scripts then

@danielhuppmann
Copy link
Member Author

merging to collect clean-up PRs before the next release, please start a new issue referencing this and #428 if problems are not solved

@danielhuppmann danielhuppmann merged commit 039b5a8 into IAMconsortium:master Oct 12, 2020
@danielhuppmann danielhuppmann deleted the fix/validate-return branch October 12, 2020 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data back-end Anything related to the (timeseries) data back end implementation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Check output consistency of .validate and similar
3 participants