Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Statistics on ParquetFile subset. #940

Merged
merged 3 commits into from
Oct 29, 2024

Conversation

yohplala
Copy link

@yohplala yohplala commented Oct 29, 2024

Fixes #938

@yohplala
Copy link
Author

Hi @martindurant
I checked all failing environments, it is systematically related to a trouble with the setting of the conda env. and tests are actually not run.

C:\Users\runneradmin\micromamba-bin\micromamba.exe shell deinit -s bash -p C:\Users\runneradmin\micromamba-root -y --log-level warning
  The following argument was not expected: -p
  Run with --help for more information.
  D:\a\_actions\mamba-org\provision-with-micromamba\main\dist\post\index.js:62[5](https://github.com/dask/fastparquet/actions/runs/11574863592/job/32220088198?pr=940#step:9:5)85
      throw Error(`Failed to execute ${JSON.stringify(args)}: ${error}`)
            ^
  
  Error: Failed to execute ["C:\\Users\\runneradmin\\micromamba-bin\\micromamba",["shell","deinit","-s","bash","-p","C:\\Users\\runneradmin\\micromamba-root","-y","--log-level","warning"]]: Error: The process 'C:\Users\runneradmin\micromamba-bin\micromamba.exe' failed with exit code 10[9](https://github.com/dask/fastparquet/actions/runs/11574863592/job/32220088198?pr=940#step:9:9)

@yohplala
Copy link
Author

@martindurant , I am not super expert with the definition of cached properties. Don't hesitate to ask for an alternative fix if this one is not appropriate (I would do with some directions)

@martindurant martindurant merged commit 25f3370 into dask:main Oct 29, 2024
19 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

statistics does not work on a ParquetFile subset?
2 participants