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

Implement integration test as example #6

Closed
prosku opened this issue Jun 9, 2021 · 1 comment · Fixed by #11
Closed

Implement integration test as example #6

prosku opened this issue Jun 9, 2021 · 1 comment · Fixed by #11
Assignees
@prosku

Comments

@prosku
Copy link
Collaborator

prosku commented Jun 9, 2021

Design and execute an integration test with thorough documentation that can serve as an application example.
@prosku prosku self-assigned this Jun 9, 2021
prosku added a commit that referenced this issue Jun 11, 2021
@prosku
Start work on #6
9bf3d09 
Add new requirements.
Add file for integration test starting from an old implementation
script.
@prosku
Copy link
Collaborator Author

prosku commented Jun 14, 2021

  • Clean up history before merging this issue

prosku added a commit that referenced this issue Jun 14, 2021
@prosku
Start work on #6
6359290 
Add new requirements.
Add file for integration test starting from an old implementation
script.
prosku added a commit that referenced this issue Jun 14, 2021
@prosku
Decide to treat this as illustrative example, not test, ref #6
1f8d3b9
@prosku prosku mentioned this issue Jun 14, 2021
Merged
@prosku prosku mentioned this issue Aug 20, 2021
Closed
5 tasks
prosku added a commit that referenced this issue Aug 20, 2021
@prosku
Prepare integration_example to almost pass pylint, ref. #6
27ac35b 
pylint still complains about a range(len()) instead of enumerate in a
test, and also about the Dataset from netCDF4, but this seems to be a
broad problem that I don't know how to handle
(pylint-dev/pylint#2932,
pylint-dev/pylint#1524). Maybe you can help with
this, @marstaa?)

The previous commits also refer to #6.
marstaa pushed a commit that referenced this issue Sep 18, 2021
@prosku
Start work on #6
ace574d 
Add new requirements.
Add file for integration test starting from an old implementation
script.
marstaa pushed a commit that referenced this issue Sep 18, 2021
@prosku
Decide to treat this as illustrative example, not test, ref #6
e5d17c5
marstaa pushed a commit that referenced this issue Sep 18, 2021
@prosku
Prepare integration_example to almost pass pylint, ref. #6
5d89654 
pylint still complains about a range(len()) instead of enumerate in a
test, and also about the Dataset from netCDF4, but this seems to be a
broad problem that I don't know how to handle
(pylint-dev/pylint#2932,
pylint-dev/pylint#1524). Maybe you can help with
this, @marstaa?)

The previous commits also refer to #6.
prosku added a commit that referenced this issue Sep 21, 2021
@prosku
Implement marstaa's feedback regarding Issue #6 & pull request #11
2ab7d9a 
Add author name
Remove unit.24
Fix DeprecationWarning
Make lons and lats arguments of the function plot_sph_harm
marstaa pushed a commit that referenced this issue Sep 21, 2021
@prosku
Implement marstaa's feedback regarding Issue #6 & pull request #11
3dcdfcb 
Add author name
Remove unit.24
Fix DeprecationWarning
Make lons and lats arguments of the function plot_sph_harm
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@prosku prosku

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Implement integrationexample marstaa/PySphereX
1 participant
@prosku