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

Page4 optim #123

Merged
merged 3 commits into from
May 18, 2022
Merged

Page4 optim #123

merged 3 commits into from
May 18, 2022

Conversation

statnmap
Copy link
Collaborator

@statnmap statnmap commented May 18, 2022

  • Est-ce que mon code respecte les standards de qualité de mise en production de packages ?
  • Est-ce que la personne qui révise à toutes les informations pour valider les fonctionnalités / résolutions de problèmes sans trop de recherches ?
  • Est-ce que le client qui validera les tickets associés à les informations pour le faire sans perte de temps ?

Issues à faire valider pour fermer :

Issues traitées à garder ouvertes ou en cours :

  • issue #

Checklist:

  • Est-ce que le check du package passe en local ?
  • Est-ce que le CI passe ?
  • Est-ce que les fonctionnalités ajoutées / corrigées, sont documentées, testées ?
  • Est-ce que les fonctionnalités ajoutées / problèmes résolus sont brièvement présentées dans le message de la MR ?
  • Est-ce que les modifications sont liées à des tickets / issues que j'ai listés dans les commits et dans la MR elle-même ?
  • Est-ce que les tickets sont en mode "Révision" dans le Board de suivi du projet ?
  • Est-ce que chaque ticket, s'il doit être fermé après acceptation de la MR contient un commentaire qui dit comment le valider ?

Why?

Separating functions an using more database may generate output modifications

What?

- Create unit test to verify expected outputs
- Use SQL as mush as possible

issue #19
Why?

Be sure reformatting gives correct results

What?

Test intermediate outputs

issue #120
Why?

Next database version allows multiple climatic scenarios

What?

Update `compute_nmax_eh1()` and upper functions to allow this selection

issue #121
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.

1 participant