-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Install steps - Add numpy requirement in manual installation steps #34
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
When `pip install -r requirements/edx/pre.txt` is omitted before running `pip install -r requirements/edx/base.txt`, it would cause API versions mistmatch when `make cms` is run. `make cms` would automatically run pip with pre.txt and install a different version of numpy, *after* base.txt and post.txt have been run through pip.
Makes total sense, @jtauber could you take a peek and just make sure things are good to merge? |
jtauber
added a commit
that referenced
this pull request
Jun 2, 2013
Install steps - Add numpy requirement in manual installation steps
chrisrossi
referenced
this pull request
in jazkarta/edx-platform
Dec 18, 2014
chrisrossi
referenced
this pull request
in jazkarta/edx-platform
Jan 16, 2015
chrisrossi
referenced
this pull request
in jazkarta/edx-platform
Jan 16, 2015
bdero
referenced
this pull request
in mitocw/edx-platform
Feb 13, 2015
Fix reference typo in schedule template JS
sarina
pushed a commit
that referenced
this pull request
Apr 2, 2015
Story #35 As a coach I can see and edit the json for the grading policy. Story #34 Recalculate grading policy Repair the broken test for grading by providing an explicit POC context for the request to run in I am not certain this is the right way to do this, might be better to test the session facility by setting the session for self.client, but this will do for now.
nedbat
pushed a commit
that referenced
this pull request
Apr 7, 2015
Story #35 As a coach I can see and edit the json for the grading policy. Story #34 Recalculate grading policy Repair the broken test for grading by providing an explicit POC context for the request to run in I am not certain this is the right way to do this, might be better to test the session facility by setting the session for self.client, but this will do for now.
singhularity
pushed a commit
to singhularity/edx-platform
that referenced
this pull request
Sep 8, 2015
…to amplify-master * commit '4befb264aed102a5bc2d147f24a414ca4ba84cee': add login_type for amplify SSO'
4 tasks
ooduye
pushed a commit
to ooduye/edx-platform
that referenced
this pull request
May 25, 2016
…ount-page [#119281085] add 'Account' to title, change button text
naeem91
pushed a commit
to naeem91/edx-platform
that referenced
this pull request
Feb 9, 2017
…ail-update updated account activation email's body text
prabhanshu
pushed a commit
to prabhanshu/edx-platform
that referenced
this pull request
Oct 13, 2018
Automatic sign-out should redirect to the sign-in page
mumarkhan999
pushed a commit
to mumarkhan999/edx-platform
that referenced
this pull request
Mar 4, 2019
FIX: remove unnecessary files
yoann-mroz
referenced
this pull request
in weuplearning/edx-platform
Nov 30, 2020
Sujeet1379
pushed a commit
to chandrudev/edx-platform
that referenced
this pull request
Nov 17, 2022
…x#34) This requires some defensive programming here and there to let things load in a minimal state.
timmc-edx
added a commit
that referenced
this pull request
Apr 29, 2024
…ary (#34…" (#34661) This reverts commit 52adce4 because we were getting this in the build pipeline: ``` WARNING: lxml 5.2.1 does not provide the extra 'html-clean' ERROR: Exception: Traceback (most recent call last): File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_internal/cli/base_command.py", line 173, in _main status = self.run(options, args) File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_internal/cli/req_command.py", line 203, in wrapper return func(self, options, args) File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_internal/commands/install.py", line 315, in run requirement_set = resolver.resolve( File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_internal/resolution/resolvelib/resolver.py", line 94, in resolve result = self._result = resolver.resolve( File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_vendor/resolvelib/resolvers.py", line 472, in resolve state = resolution.resolve(requirements, max_rounds=max_rounds) File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_vendor/resolvelib/resolvers.py", line 366, in resolve failure_causes = self._attempt_to_pin_criterion(name) File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_vendor/resolvelib/resolvers.py", line 221, in _attempt_to_pin_criterion satisfied = all( File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_vendor/resolvelib/resolvers.py", line 222, in <genexpr> self._p.is_satisfied_by(requirement=r, candidate=candidate) File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_internal/resolution/resolvelib/provider.py", line 178, in is_satisfied_by return requirement.is_satisfied_by(candidate) File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_internal/resolution/resolvelib/requirements.py", line 84, in is_satisfied_by assert candidate.name == self.name, ( AssertionError: Internal issue: Candidate is not for this requirement lxml[html-clean,html-clean] vs lxml[html-clean] WARNING: You are using pip version 21.2.1; however, version 24.0 is available. You should consider upgrading via the '/edx/app/edxapp/venvs/edxapp/bin/python -m pip install --upgrade pip' command. ```
awais786
added a commit
to awais786/edx-platform
that referenced
this pull request
Apr 30, 2024
feat!: upgrading pymongo to 4.0 fix: Update Makefile feat!: Remove md5 checksums from gridfs and remove disable. feat!: Remove md5 checksums from gridfs and remove disable. feat!: Remove md5 checksums from gridfs and remove disable. feat!: Remove md5 checksums from gridfs and remove disable. feat!: Remove md5 checksums from gridfs and remove disable. feat!: Remove md5 checksums from gridfs and remove disable. feat: Upgrade Python dependency enmerkar-underscore Update to a Python 3.11 compatible version Commit generated by workflow `openedx/edx-platform/.github/workflows/upgrade-one-python-dependency.yml@refs/heads/master` feat: Upgrade Python dependency edx-codejail Update to a Python 3.11 compatible version Commit generated by workflow `openedx/edx-platform/.github/workflows/upgrade-one-python-dependency.yml@refs/heads/master` Revert "build: Python requirements upgrade and lxml build without binary (openedx#34…" (openedx#34661) This reverts commit 52adce4 because we were getting this in the build pipeline: ``` WARNING: lxml 5.2.1 does not provide the extra 'html-clean' ERROR: Exception: Traceback (most recent call last): File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_internal/cli/base_command.py", line 173, in _main status = self.run(options, args) File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_internal/cli/req_command.py", line 203, in wrapper return func(self, options, args) File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_internal/commands/install.py", line 315, in run requirement_set = resolver.resolve( File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_internal/resolution/resolvelib/resolver.py", line 94, in resolve result = self._result = resolver.resolve( File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_vendor/resolvelib/resolvers.py", line 472, in resolve state = resolution.resolve(requirements, max_rounds=max_rounds) File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_vendor/resolvelib/resolvers.py", line 366, in resolve failure_causes = self._attempt_to_pin_criterion(name) File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_vendor/resolvelib/resolvers.py", line 221, in _attempt_to_pin_criterion satisfied = all( File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_vendor/resolvelib/resolvers.py", line 222, in <genexpr> self._p.is_satisfied_by(requirement=r, candidate=candidate) File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_internal/resolution/resolvelib/provider.py", line 178, in is_satisfied_by return requirement.is_satisfied_by(candidate) File "/edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/pip/_internal/resolution/resolvelib/requirements.py", line 84, in is_satisfied_by assert candidate.name == self.name, ( AssertionError: Internal issue: Candidate is not for this requirement lxml[html-clean,html-clean] vs lxml[html-clean] WARNING: You are using pip version 21.2.1; however, version 24.0 is available. You should consider upgrading via the '/edx/app/edxapp/venvs/edxapp/bin/python -m pip install --upgrade pip' command. ```
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Manual installation steps were missing the
requirements/edx/pre.txt
file. So when I followed instructions from the README, without usingscripts/create-dev-env.sh
, I ranpip install -r requirements/edx/base.txt
directly, which used my system-wide numpy. It had a different version, which caused API versions mistmatch later on when I ranmake cms
, asmake cms
would automatically run pip with pre.txt and install a different version of numpy, after base.txt and post.txt have been run through pip.Hope this make sense : )