Tom Vo

Results 265 comments of Tom Vo

Hey @xCDAT/core-developers, here is my initial PR draft to simplify the contributing guide. It is based on Xarray's contributing guide, but geared towards xCDAT and made even simpler. If any...

Hi @chengzhuzhang, just tagging you for review on this PR. It's not urgent, so whenever you have time for it that would be great.

> @tomvothecoder – I did [make some changes](https://github.com/pochedls/xcdat/commit/564aa68040a637913474f4a5ae05a036e8875788) locally (from my forked repo). I basically just deleted mamba stuff ([conda now uses the mamba solver by default](https://docs.conda.io/projects/conda/en/latest/release-notes.html#id23)). @pochedls Thanks for...

@pochedls Just bugging you again for your approval

Additional item from today's meeting: - [x] fix tagging of GH issues for beginners to work on -- I tagged all of the "docs" issues as [good-first-issue](https://github.com/xCDAT/xcdat/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc+label%3Agood-first-issue). I'll tag a...

> - Doesn't the first code block do (1) and (2)? Should the following be outside the code block?: (1) does not do (2). (1) Creates the dev environment with...

> Doesn't `make install # or python -m pip install .` in the code block under _1. Install the build dependencies_ actually build/install xcdat? > > It seems like the...

I addressed the suggestions and will merge this PR now.

FYI @ezekiel-lemur. I will close this issue now. > Maybe the masked array can automatically be converted to a numpy array with missing values represented by `np.nan` (using this [code](https://github.com/Unidata/cftime/issues/328#issuecomment-2089151810)),...

This is now complete with all of the merged PRs.