Resolve issue #22, #26 and update environment.yml

calum-chamberlain / ESCI451-Python

Introduction to Python for VUW ESCI 451 course.

GNU General Public License v3.0

11 stars 5 forks source link

Closed FinniganIK closed 1 year ago

FinniganIK commented 1 year ago

Use latest version of pygmt and resolve issue #22

review-notebook-app[bot] commented 1 year ago

Check out this pull request on

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

calum-chamberlain commented 1 year ago

It looks like the test is flagging that FirstIndex isn't defined. Might be worth checking that variable name.

calum-chamberlain commented 1 year ago

I mucked up and forgot to merge this before making lots of other changes. I have cherry picked these changes and applied them to master.