Damien Irving

Results 48 comments of Damien Irving

No problem, @TomNicholas. I totally agree that a combination of upstream work in xarray, pint-xarray, and cf-xarray should eventually be able to replace much of xclim's functionality in a more...

With respect to different lengths of control runs, in my experience most people choose to be more accurate (i.e. use all the available control data) than "fair" (by using the...

This is now ready for review. I've trimmed back the assertion content and added new content on exception handling (as was the brief for this PR). This means I've left...

Great. Thanks, everyone. I've started a draft pull request at #981 and I'll let you know when it's ready for review (hopefully in the next week or two).

#981 is now ready for review if people want to take a look

@maxim-belkin, @tobyhodges Is there anything I can do to help progress this? The PR is ready to go #981

@wking Totally agree that most people (including myself) teach python before git, so I definitely don't think we should go down the route of having a version controlled copy of...

@abostroem I think this conversation is getting a little confused about exactly what I was suggesting. All I want to do is remove the `readings-01.py`, `readings-02.py`, etc files from the...

@abostroem Exactly (and yes to indicating in the instructions). I have no intention of mentioning version control in this lesson - I just want to avoid exposure to the poor...

A nice little addition would be to highlight the changes from one in place edit to the next (e.g. new text in bold), however I have no idea how to...