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

Make repo testable on Binder #13

Merged
merged 12 commits into from
Mar 2, 2022
Merged

Make repo testable on Binder #13

merged 12 commits into from
Mar 2, 2022

Conversation

gmaze
Copy link
Member

@gmaze gmaze commented Feb 25, 2022

No description provided.

@gmaze gmaze marked this pull request as draft February 28, 2022 15:50
@gmaze gmaze marked this pull request as ready for review March 1, 2022 12:33
@gmaze
Copy link
Member Author

gmaze commented Mar 1, 2022

I think this is ready:

Binder

I decoupled Binder set-up from this repo content using the new binder-sandbox repo !

@gmaze gmaze requested review from EPauthenet and quai20 March 1, 2022 12:35
Copy link
Collaborator

@EPauthenet EPauthenet left a comment

Choose a reason for hiding this comment

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

Le chargement d'une trajectoire Argo ne fonctionne pas et donne cette erreur :
FileNotFoundError: ['https://www.ifremer.fr/erddap/tabledap/ArgoFloats.nc?data_mode,latitude,longitude,position_qc,time,time_qc,direction,platform_number,cycle_number,config_mission_number,vertical_sampling_scheme,pres,temp,psal,pres_qc,temp_qc,psal_qc,pres_adjusted,temp_adjusted,psal_adjusted,pres_adjusted_qc,temp_adjusted_qc,psal_adjusted_qc,pres_adjusted_error,temp_adjusted_error,psal_adjusted_error&platform_number=~"3901602"&distinct()&orderBy("time,pres")']

  • Sinon tout fonctionne mais avec des warnings de Tensorflow et Panda qui préviennent que certaines fonctions seront plus à jour.

Copy link
Member

@quai20 quai20 left a comment

Choose a reason for hiding this comment

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

ça tourne parfaitement de mon côté. Juste un warning de deprec xarray un peu pénible sur
float ou int .get_loc

@gmaze
Copy link
Member Author

gmaze commented Mar 2, 2022

Le chargement d'une trajectoire Argo ne fonctionne pas et donne cette erreur :

ca devait etre momentanee et une erreur du serveur erddap Argo, on n'y peut rien, faut juste recommencer plus tard

@gmaze
Copy link
Member Author

gmaze commented Mar 2, 2022

Oui tous les warnings c'est penible

C'est a cause de la derniere version de pandas (1.4.0) pour laquelle xarray doit faire une maj, cf pydata/xarray#6195

la aussi, on n'y peut rien et ca devrait disparaitre assez vite avec les nouvelles releases

@gmaze gmaze requested a review from EPauthenet March 2, 2022 09:37
@gmaze gmaze merged commit a40ad72 into main Mar 2, 2022
@gmaze gmaze deleted the binder branch March 7, 2022 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants