Open hjabbot opened 6 months ago
Merging meshes
Antimeridian
Vector splitting
Dataloaders
cellbox.py:
aggregated_cellbox.py: (All fixed in PR #70)
neighbour_graph.py:
remove_node_from_neighbours()
docstring says int values passed to method, but method fails when cellbox_indx is an int. Unsure how this method is actually meant to work, I think there's an error with the casting between int and str somewhereupdate_neighbour
, update_corner_neighbours
, update_neighbours
all exist and do different things. update_corner_neighbours appears most similar to update_neighbour, and not update_neighboursget_neighbour_case()
, get_neighbour_case_bounds()
, get_global_mesh_neighbour_case()
, initialise_map()
all take self as argument but have no call to self, should be staticmethods insteadget_global_mesh_neighbour_case()
doesn't return 0 when cellboxes aren't connected. This hasn't cropped up before because the method is only called within get_neighbour_case()
after doing a check to see if they're connectedupdate_neighbours()
docstring says new_neighbour_indx is an int, but references it in code as if it is a list of at least length 2. Unclear what intended input is.cli.py
The existing tests are old an need updating. This issue is meant to be a place to host a list of tests we should implement. We should also prune through the existing tests to see what's necessary or redundant.
We can also create a bash script to automatically figure out what tests to run, and then run them, without us having to figure it out manually