Nicholas Geneva comments

Results 40 comments of


                                            Nicholas Geneva

Update dali dependency to a cuda 12 compatible version

/blossom-ci

Fixed indexing error for modulus loss function

/blossom-ci

Fixed indexing error for modulus loss function

Instead cropping we should consider padding @ktangsali We should pad the repo with a reflection boundary so we can compute edge derivatives. Makes the domain periodic iirc so the calculation...

Fixed indexing error for modulus loss function

Merged here: https://github.com/NVIDIA/modulus/pull/545 Thanks for the patch.

config updates for clearity and better defaults

/blossom-ci

Error cell [14] issubclass() arg 1 must be a class

Hi @DanD364 This looks like a problem with the HfArgParser and some Python magic it uses. For Python 3.9 I have to [hack around a solution](https://github.com/zabaras/transformer-physx/blob/main/trphysx/config/arg_parser.py#L86) to get it to...

:bug: [BUG]: Incorrect get_num_losses return value when using custom loss function

Thanks for the report, I agree this is a bug. The `get_num_losses` in [domain](https://github.com/NVIDIA/modulus-sym/blob/main/modulus/sym/domain/domain.py#L128) uses the output names of each constraint to get a total number of losses. The custom...

🚀[FEA]: Package system doesn't natively work with Zarr Stores

Updating to feature request, doing this on NGC will potentially be quite challenging due to how file access works... specifically for authenticated users.

🐛[BUG]: ablation_sampler removal from modulus

Fixed in RC Branch: https://github.com/NVIDIA/earth2studio/pull/136

🚀[FEA]: Will Earth2Studio support training of AI weather models？

Hi @LevineHuang At the moment Earth2Studio is a inference focused package. Training of weather AI models is something we leave to other repos, whether modulus, makani or the research repo's...