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

v2.0.0 - ampycloud article release #133

Merged
merged 62 commits into from
Jul 12, 2024
Merged

v2.0.0 - ampycloud article release #133

merged 62 commits into from
Jul 12, 2024

Conversation

fpavogt
Copy link
Member

@fpavogt fpavogt commented Jul 10, 2024

Description:

This PR contains all the modifications added to ampycloud as part of the review process of the ampycloud scientific article.

⚠️ It includes non-backward compatible changes in the user interface.

In addition to a few bugfixes, the main changes include:

  • consistent use of the term "height" throughtout the code and UI
  • support for numpy >2.0
  • handling of NSC code

Error(s) fixed:

Checklists:

  • New code includes dedicated tests.
  • New code has been linted.
  • New code follows the project's style.
  • New code is compatible with the 3-Clause BSD license.
  • CHANGELOG has been updated.
  • AUTHORS has been updated.
  • Copyright years in module docstrings have been updated.

regDaniel and others added 30 commits April 8, 2024 18:20
@regDaniel
AMAROC-768 add heigh cloud flag and allow NSC
a155343
@regDaniel
AMAROC-768 formatting (easy fixes)
2e2e2b0
@regDaniel
AMAROC-768 update copyright years
4ffcc62
@regDaniel
AMAROC-768 update changelog
cdbb945
@regDaniel
AMAROC-768 fix f-string
8b18066
@regDaniel
AMAROC-768 NSC if sligrolay in buffer
fa4db2a
@regDaniel
AMAROC-768 Restrict python versions for pylinter
798f5b7
@regDaniel
AMAROC-768 install setuptools for pylinter
745a50a
@regDaniel
AMAROC-768 add comments
c22cd99
AMAROC-765 Add mypy to the setup
17930a5
@regDaniel
AMAROC-768 drop attrs, add CeiloChunk attribute
ba885da
@regDaniel
AMAROC-768 remove unused import
5aa7fae
@regDaniel
AMAROC-768 add NSC warning to scope.rst
456dbf5
@regDaniel
AMAROC-768 improve message in CHANGELOG
d09d1f5
@regDaniel
Merge pull request #114 from MeteoSwiss/feature/AMAROC-768-nsc-flag
c2e5d23 
AMAROC-768 add heigh cloud flag and allow NSC
@fpavogt @regDaniel
Set mock exemple to be 900s long
804015d
@fpavogt @regDaniel
Use 'height' in 'ft aal' throughout
281be4e
@fpavogt @regDaniel
As of Python 3.12, setuptools needs to be explicitely requested as an…
6108174 
… 'extras' in setup.py for dev work
@fpavogt @regDaniel
Bump version to reflect the breaking API changes
c4d9847
@regDaniel
AMAROC-768 remove explicit build of setuptools
c732f2b
@regDaniel
Merge pull request #113 from MeteoSwiss/develop_vof
f73ce81 
Terminology correction: alt -> height
AMAROC-765 Remove not necessary converting data
cc35cfd
AMAROC-765 Correct easy type issues
39908d0
AMAROC-765 solve issue in texify and add test
1e48c69
AMAROC-765 fix arraylike type issue
cf87c4b
AMAROC-765 fix mixing type issues
c2d65fe
Merge branch 'develop' into bugfix/AMAROC-765-fixMypyIssues
2b780ca 
# Conflicts:
#	setup.py
#	src/ampycloud/data.py
#	src/ampycloud/plots/diagnostics.py
#	src/ampycloud/utils/mocker.py
#	src/ampycloud/utils/utils.py
AMAROC-765 fix leftover issues after merge
19d3f25
AMAROC-765 fix conf.py
9b28aec
AMAROC-765 fix np.array in mocker.py
65e9ce0
fpavogt and others added 24 commits April 23, 2024 16:56
@fpavogt
Add additional test case following review
7bd4b28
@fpavogt
Fix #119
c4d7240
@fpavogt
Minor code cleanup
4d05ff4
@fpavogt
Merge pull request #118 from MeteoSwiss/develop_vof
ab0fbca 
Fixes #116 and #119.
@srethore
Merge branch 'develop' into bugfix/AMAROC-765-fixMypyIssues
7bbd67d
@srethore
Merge pull request #121 from MeteoSwiss/bugfix/AMAROC-765-fixMypyIssues
68ba86c 
Bugfix/amaroc 765 fix mypy issues
@fpavogt
Fix #123
79413f1
@fpavogt
Merge pull request #125 from MeteoSwiss/fix_123
14025a4 
Fix #123
@fpavogt
Add one rogue point to the mock example
5f433e3
@fpavogt
Correct col type
991c626
@fpavogt
Swap instead of add rogue hit
9aafcb8
@fpavogt
Merge pull request #126 from MeteoSwiss/develop_vof
1ab73ce 
Add rogue point to mock dataset
@fpavogt
Add support for numpy >2.0
0674d3e
@fpavogt
Update docs
8a2b05b
@fpavogt
Update docs
8c3c3a2
@fpavogt
Merge pull request #129 from MeteoSwiss/develop_vof
00bfab6 
AMAROC-819 Add support for numpy >2.0
@fpavogt
Allow to trigger the weekly tests manually
3bf417c
@fpavogt
Update CHANGELOG
9e52f4d
@fpavogt
Merge pull request #130 from MeteoSwiss/develop_vof
880a9f2 
Allow to trigger the weekly tests manually
@fpavogt
Fix manual trigger of automated tests (#131)
5843bb7 
* Fix manual trigger
@fpavogt
Update CONTRIBUTING.md
4d9ca27 
Minor clean-up of the Contributing guidelines.
@fpavogt
Merge pull request #132 from MeteoSwiss/fpavogt-patch-1
0c8e618 
Update CONTRIBUTING.md
@fpavogt
Update CONTRIBUTING.md
cb70e14
@fpavogt
Update CONTRIBUTING.md
6818b2f
@fpavogt fpavogt added this to the v2.0.0 milestone Jul 10, 2024
@fpavogt fpavogt requested a review from regDaniel July 10, 2024 06:57
@fpavogt fpavogt self-assigned this Jul 10, 2024
@fpavogt
Copy link
Member Author

fpavogt commented Jul 10, 2024

@regDaniel: if ok for you, I'll merge this, so that we can formally release v2.0.0 ?

regDaniel
regDaniel approved these changes Jul 11, 2024
View reviewed changes
Copy link
Collaborator

@regDaniel regDaniel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @fpavogt! Since I assume there's nothing new here, feel free to merge.

@fpavogt fpavogt merged commit de86aef into master Jul 12, 2024
15 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@regDaniel regDaniel regDaniel approved these changes

Assignees

@fpavogt fpavogt

Labels
None yet
Projects
None yet
Milestone
v2.0.0
Development

Successfully merging this pull request may close these issues.
3 participants
@fpavogt @regDaniel @srethore