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

drop nsls2forge #765

Merged
merged 11 commits into from
Dec 13, 2022
Merged

drop nsls2forge #765

merged 11 commits into from
Dec 13, 2022

Conversation

prjemian
Copy link
Contributor

@prjemian prjemian commented Dec 9, 2022

@prjemian prjemian added the task Something to be done. label Dec 9, 2022
@prjemian prjemian added this to the 1.6.10 milestone Dec 9, 2022
@prjemian prjemian self-assigned this Dec 9, 2022
@prjemian
Copy link
Contributor Author

The problem is that this package cannot be installed locally by pip due to installation of additional requirements. The directory for those requirements is not wrtieable:

Run pip install --no-deps -e .
  pip install --no-deps -e .
  shell: /usr/bin/bash -l {0}
Defaulting to user installation because normal site-packages is not writeable
Obtaining file:///home/runner/work/apstools/apstools
  Installing build dependencies: started
  Installing build dependencies: finished with status 'done'
  Checking if build backend supports build_editable: started
  Checking if build backend supports build_editable: finished with status 'done'
  Getting requirements to build wheel: started
  Getting requirements to build wheel: finished with status 'done'
  Installing backend dependencies: started
  Installing backend dependencies: finished with status 'done'
  Preparing metadata (pyproject.toml): started
  Preparing metadata (pyproject.toml): finished with status 'done'
Installing collected packages: apstools
  Running setup.py develop for apstools
    error: subprocess-exited-with-error
    
    × python setup.py develop did not run successfully.
    │ exit code: 1
    ╰─> [3[2](https://github.com/BCDA-APS/apstools/actions/runs/3660647436/jobs/6188017820#step:6:2) lines of output]
        running develop
        /usr/lib/python[3](https://github.com/BCDA-APS/apstools/actions/runs/3660647436/jobs/6188017820#step:6:3)/dist-packages/setuptools/command/easy_install.py:158: EasyInstallDeprecationWarning: easy_install command is deprecated. Use build and pip and other standards-based tools.
          warnings.warn(
        WARNING: The user site-packages directory is disabled.
        /usr/lib/python3/dist-packages/setuptools/command/install.py:3[4](https://github.com/BCDA-APS/apstools/actions/runs/3660647436/jobs/6188017820#step:6:5): SetuptoolsDeprecationWarning: setup.py install is deprecated. Use build and pip and other standards-based tools.
          warnings.warn(
        error: can't create or remove files in install directory

@prjemian
Copy link
Contributor Author

Rebuild workflow and test locally with act starting from simple example.

@prjemian
Copy link
Contributor Author

Or a conda environment?

@prjemian
Copy link
Contributor Author

Actually, action sphinx-notes/pages does it own pip install which is where the failure occurs.

Could try an alternative to this action, build documentation with sphinx using existing setup in the workflow, then use action peaceiris/actions-gh-pages@v3 to copy to gh-pages branch..

@prjemian
Copy link
Contributor Author

See also: spatialaudio/nbsphinx#24

@prjemian
Copy link
Contributor Author

Don't try to document the specnexus package.

2022-12-12T22:35:57.2403263Z   /home/runner/micromamba-root/envs/anaconda-test-env-py-3.10/lib/python3.10/site-packages/spec2nexus-2021.0.1-py3.7.egg/spec2nexus/spec.py:15: DeprecationWarning: invalid escape sequence '\s'
2022-12-12T22:35:57.2403625Z 
2022-12-12T22:35:57.2404433Z apstools/callbacks/tests/test_filewriter.py::test_SpecWriterCallback_writer_default_name
2022-12-12T22:35:57.2404884Z apstools/callbacks/tests/test_filewriter.py::test_SpecWriterCallback_writer_default_name
2022-12-12T22:35:57.2405648Z   /home/runner/micromamba-root/envs/anaconda-test-env-py-3.10/lib/python3.10/site-packages/spec2nexus-2021.0.1-py3.7.egg/spec2nexus/plugins/spec_common.py:267: DeprecationWarning: invalid escape sequence '\d'

@prjemian
Copy link
Contributor Author

unit tests for some spec2nexus features were failing (different test results on local vs. GHA): removed the extra tests

@prjemian prjemian merged commit 4b68ba9 into main Dec 13, 2022
@prjemian prjemian deleted the 764-drop-nsls2forge branch December 13, 2022 01:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
task Something to be done.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI failed to build documentation when creating pip installation drop nsls2forge
1 participant