Skip to content

make all/test both ok #123#125

Closed
turbomam wants to merge 4 commits into
mainfrom
issue-123-smaller-Pipfile-dev
Closed

make all/test both ok #123#125
turbomam wants to merge 4 commits into
mainfrom
issue-123-smaller-Pipfile-dev

Conversation

@turbomam

@turbomam turbomam commented Oct 4, 2021

Copy link
Copy Markdown
Member

after moving mike* out of Pipfile's dev dependencies

I didn't consider any other criteria other than getting those two Makefile steps to work

after moving mike out of Pipfile's dev dependencies
@turbomam turbomam requested a review from gaurav October 4, 2021 16:09

@gaurav gaurav left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this, Mark! I don't think it's a good idea to remove the prereqs for sheet2linkml until we've rewritten the Makefile so that it no longer runs sheet2linkml from within this repo (i.e. I think those changes should happen at the same time that we remove sheet2linkml from this repo, add sheet2linkml as a prereq, and then modify the Makefile so that running make generate-model runs sheet2linkml as installed from PyPI). Could you try to add that to this PR? I could take a stab at doing that if you prefer.

@turbomam

turbomam commented Oct 14, 2021

Copy link
Copy Markdown
Member Author

Working on this now.

ccdhmodel$ make install

Could not find a version that satisfies the requirement sheet2linkml==1.0.1

pipenv install
Creating a virtualenv for this project…
Using /usr/bin/python3 (3.8.10) to create virtualenv…
⠋created virtual environment CPython3.8.10.final.0-64 in 125ms
creator CPython3Posix(dest=/home/mark/.local/share/virtualenvs/ccdhmodel-xtc7c06K, clear=False, global=False)
seeder FromAppData(download=False, pip=latest, setuptools=latest, wheel=latest, pkg_resources=latest, via=copy, app_data_dir=/home/mark/.local/share/virtualenv/seed-app-data/v1.0.1.debian.1)
activators BashActivator,CShellActivator,FishActivator,PowerShellActivator,PythonActivator,XonshActivator

Virtualenv location: /home/mark/.local/share/virtualenvs/ccdhmodel-xtc7c06K
Pipfile.lock not found, creating…
Locking [dev-packages] dependencies…
Locking [packages] dependencies…
Updated Pipfile.lock (d72d1b)!
Installing dependencies from Pipfile.lock (d72d1b)…
An error occurred while installing sheet2linkml==1.0.1! Will try again.
🐍 ▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉ 109/109 — 00:00:39
Installing initially–failed dependencies…
☤ ▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉ 0/1 — 00:00:00
ERROR: Could not find a version that satisfies the requirement sheet2linkml==1.0.1 (from -r /tmp/pipenv-kqq3qulp-requirements/pipenv-wg6rpzb0-requirement.txt (line 1)) (from versions: 1.0.0)
ERROR: No matching distribution found for sheet2linkml==1.0.1 (from -r /tmp/pipenv-kqq3qulp-requirements/pipenv-wg6rpzb0-requirement.txt (line 1))

☤ ▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉ 0/1 — 00:00:01
make: *** [Makefile:59: install] Error 1

@turbomam

Copy link
Copy Markdown
Member Author
$ pipenv shell
(ccdhmodel-xtc7c06K) ccdhmodel$ pip install sheet2linkml

...
Installing collected packages: setuptools, sheet2linkml
Attempting uninstall: setuptools
Found existing installation: setuptools 44.0.0
Uninstalling setuptools-44.0.0:
Successfully uninstalled setuptools-44.0.0
Successfully installed setuptools-52.0.0 sheet2linkml-1.0.0

but having trouble installing sheet2linkml from Pipenv
@turbomam

Copy link
Copy Markdown
Member Author

My latest push is working off of the PyPI sheet2linkml on my LBL MacBook now.

I created a new makefile rule: model/schema/crdch_model.yaml

I'm having trouble with the Linux workstation I've been using for a few weeks now, but that could be due to the slightly different dev environment, or maybe I didn't move over some of the config files. I couldn't get sheet2linkml 1.0.1 to install on my Linux workstation, but 1.0.0 installed.

@turbomam

Copy link
Copy Markdown
Member Author

works in Ubuntu 20 now with 3.9 requirement in Pipfile

@gaurav

gaurav commented Oct 14, 2021

Copy link
Copy Markdown
Collaborator

Might overlap with #128

@turbomam

Copy link
Copy Markdown
Member Author

I believe the functionality I was trying to add has already been added in one of @sujaypatil96's PRs.

@turbomam turbomam closed this Nov 10, 2021
@turbomam turbomam deleted the issue-123-smaller-Pipfile-dev branch November 10, 2021 17:36
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.

2 participants