-
Notifications
You must be signed in to change notification settings - Fork 7
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
chore: update dependencies #188
Closed
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Not using any cache anymore Always get the data on the fly
Renamed the logger to `copernicusmarine` to fit with the convention that the logger is usually with the same name as the library
If file modification date cannot be changed, save the file anyway but with the downloaded date instead
Not using any cache anymore Always get the data on the fly
Reordering the attributes of the dimensions, we considered that it was best to keep the attribute `positive`, to have it clear where the dimension depth points (down for depth, up for elevation).
Deleting the valid_min and valid_max attributes from the dimensions, as requested to comply with CF conventions.
We detected that there was no warning when the depth dimension was being set out of bounds... we corrected the code, so that it raises a warning when the requested interval is out. Careful, this is a breaking change as it can change the output. It can also change the behaviour if --subset-method is set to strict.
take into account the timezone when parsing datetime and getting the time from the computer
Fix bug for semver check with the poetry syntax of pre release versions and test the pre-releases/ branches
first commit with some html documentation of the toolbox
Support more date format: support timezones and uses pendulum
Co-authored-by: Renaud <[email protected]> Implementation of a new method to choose how the requested interval affects the retrieved data: the bounding box method.
We now display an approximation of the amount of data that will be downloaded next to the expected size of the final file.
have a defined return object with more information and a dry run to access this object without downloading anything
add favicon and source code from the repository and change to the blue of copernicusmarine Also add a copy button for the documentation
Making sure the formats of the variables have the same accuracy (if float32, then float32, not float in general)
Fix a bug where we would have some coordinates for variables that don't have it in the describe
Not using aiohttp anymore nor nest-asyncio Also changed from multiprocessing to multi threading for get command
Readthedocs conf to be able to build and deploy there
fix of a bug introduced by v2 features
Co-authored-by: Renaud <[email protected]> Add a compliance checker to make sure that the netcdf output files are as expected.
Fix a bug where the decimals for the longitude wouldn't be considered properly for the check of bounds
Handle values for retention dates as some datasets don't have min and max but only a list of values that should be updated
fix of bug introduced by another commit where we handle retention period
Making it easier for users to understand the login functionality.
change the name of vertical-dimension-as-originally-producted to vertical-dimension-output
#145) pour the readme in the documentation
Co-authored-by: Renaud <[email protected]> Co-authored-by: renaudjester <[email protected]>
Correcting the documentation so that it is in the correct place!
upgrade the GitHub Action workflow to use mamba shells to keep the code more robust.
Use the s3 product dataset mapping and add errors in the doc and in the copernicus package
Delete some deprecated functions
Change the depth default values for the option `--create-template` to be consistent and not pose any problems.
updated the output when listing the files on the remote server for users to know why it is taking (maybe) too long.
Get rid of the `--service` option for the get command since only one asset is available
correct a lot of typos and improve the doc
Add a functionality to check if the user's credentials are valid. Deletes the `--skip-if-logged-in` functionality
No need to specify `--netcdf-compression-enabled` to be able to compress. Works also as a flag to compress to level 1.
correct a bug that was breaking the describe
Example in the helpers are now unwrapped without modifying the documentation
Co-authored-by: Renaud <[email protected]> Add the link to directly download the binaries in the documentation.
Co-authored-by: Renaud <[email protected]> Refactoring to try to centralise as much as possible the documentation. Rewording some definitions and adding some documentation were necessary.
Co-authored-by: Renaud <[email protected]> Come small changes to finish up the review of the documentation.
Co-authored-by: renaudjester <[email protected]> Adding an overview notebook for users to have something to start on to code with the toolbox.
simplifying the wording of the documentation
Try to connect to the new authentication system first and then falls back to the old one
renaudjester
force-pushed
the
copernicusmarine-toolbox-v2
branch
from
October 28, 2024 14:24
18b4f51
to
615b421
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Let's try to update to numpy v2 since there is a new release of xarray that should solve the latency problem (fix #159 )
Let's see if we can use
h5netcdf
instead ofnetcdf4
to be able to update to python 3.13