Skip to content

Optimize#28

Open
danlipsa wants to merge 5 commits intoKitware:masterfrom
danlipsa:optimize
Open

Optimize#28
danlipsa wants to merge 5 commits intoKitware:masterfrom
danlipsa:optimize

Conversation

@danlipsa
Copy link
Collaborator

No description provided.

@danlipsa
Copy link
Collaborator Author

@jourdain Please review

Points are created from variables in the connectivity file:

double grid_corner_lat(grid_size, grid_corners) ;
double grid_corner_lon(grid_size, grid_corners) ;
@jourdain
Copy link
Collaborator

Can you follow https://www.conventionalcommits.org/en/v1.0.0/ for your commit message. That way, the changelog and updated version on PyPI/conda will reflect your work.

If you don't mind not keeping your separate commits, I can squash them using a correct syntax from the web UI. Just let me know.

@jourdain
Copy link
Collaborator

Otherwise your changes seems reasonable.

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.

2 participants