Fix ArgoFloat configuration parameter availability checking inconsistency - #644
Merged
gmaze merged 39 commits intoJul 15, 2026
Conversation
- deprec lsprofiles - update describe_profiles - new ls_profiles_for - new ls_profiles - new open_profile
- fix bug whereby the default ascending direction was not set
- refactor toward: open_profile and open_profiles methods - more private place holder for perf. - better docstrings
update facade docstring
- improve typing - new lsp() as a profiles counter part to ls() - change column names in describe_profiles
- fix bug whereby argosplitpath could not handle path from the auxiliary folder
- New CYCLE_NUMBERS attribute - fix bug whereby ls_profiles() would not work for floats with cycle numbers not identical from the euroargofleet API and the netcdf files, this would lead to a wrong N_CYCLES attributes - N_CYCLES now depends on CYCLE_NUMBERS - CYCLE_NUMBERS is determined on the fly by looking at the GDAC folder 'profiles' content, not from API or netcdf content
- clean up 'open_profiles'
- this is better english - this is more coherent with ls_profiles
- now include coverage for new profiles-related features
long due minimal doc for config and launchconfig
fix docstrnigs
refactoring of internal place holders (more explicit names)
I'm trying to make it as simple as simple for users, by limiting the list of methods available on the facade. - af.ls > af._ls - af.lsp > af._lsp - af._ls > af.__ls - af. ls_profiles_for > af. _ls_profiles_for - af. describe_profiles > af. profiles_to_dataframe Signed-off-by: Guillaume Maze <gmaze@ifremer.fr>
Signed-off-by: Guillaume Maze <gmaze@ifremer.fr>
Signed-off-by: Guillaume Maze <gmaze@ifremer.fr>
Signed-off-by: Guillaume Maze <gmaze@ifremer.fr>
gmaze
added
invalid
Signed-off-by: Guillaume Maze <gmaze@ifremer.fr>
…lability-check-is-not-consistent Signed-off-by: Guillaume Maze <gmaze@ifremer.fr>
…lability-check-is-not-consistent
github-project-automation
Bot
moved this from In Progress
to Done
in Argopy Management Dashboard
gmaze
deleted the
643-argofloat-configuration-parameter-availability-check-is-not-consistent
branch
July 15, 2026 15:03
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Closes #643