neuraloperator icon indicating copy to clipboard operation
neuraloperator copied to clipboard

Imports in the examples broken

Open matteoguarrera opened this issue 1 year ago • 4 comments

from neuralop.data.datasets import load_darcy_flow_small this should be from neuralop.datasets import load_darcy_flow_small all the examples have this issue :cry:

matteoguarrera avatar Jun 20 '24 00:06 matteoguarrera

Hi @matteoguarrera, could you describe this issue in more detail? #332 refactored the datasets module to data.datasets and data.transforms. load_darcy_flow_small should now be located in neuralop.data.datasets.darcy. Is the code throwing an error?

dhpitt avatar Jun 20 '24 01:06 dhpitt

You are right, I guess the error is in the release. The repo is ok, but if you pip install the package it throws the error

matteoguarrera avatar Jun 20 '24 01:06 matteoguarrera

Right, we’re planning to do a bigger release soon. Thanks for raising this issue.

On Wed, Jun 19, 2024 at 6:55 PM Matteo Guarrera @.***> wrote:

You are right, I guess the error is in the release. The repo is ok, but if you pip install the package it throws the error

— Reply to this email directly, view it on GitHub https://github.com/neuraloperator/neuraloperator/issues/341#issuecomment-2179675718, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJWQJOHPWA2XSYRLQAL5PHTZIIZCRAVCNFSM6AAAAABJS6CCR2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZZGY3TKNZRHA . You are receiving this because you commented.Message ID: @.***>

dhpitt avatar Jun 20 '24 02:06 dhpitt

:)

arcturusannamalai avatar Sep 06 '24 15:09 arcturusannamalai

This is stale post 1.0.2 release

dhpitt avatar Mar 25 '25 17:03 dhpitt