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

feat: updated to python 3.12 #118

Merged
merged 6 commits into from
Nov 15, 2024
Merged
Show file tree
Hide file tree
Changes from 4 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ jobs:
strategy:
matrix:
os: [ubuntu-20.04]
python-version: ['3.11']
python-version: ['3.12']
toxenv: [quality, pii_check, django42]

steps:
Expand All @@ -37,7 +37,7 @@ jobs:
run: tox

- name: Run coverage
if: matrix.python-version == '3.11' && matrix.toxenv == 'django42'
if: matrix.python-version == '3.12' && matrix.toxenv == 'django42'
uses: py-cov-action/python-coverage-comment-action@v3
with:
GITHUB_TOKEN: ${{ github.token }}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/pypi-publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ jobs:
- name: setup python
uses: actions/setup-python@v2
with:
python-version: 3.8
python-version: 3.12

- name: Install pip
run: pip install -r requirements/pip.txt
Expand Down
4 changes: 4 additions & 0 deletions CHANGELOG.rst
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,10 @@ Unreleased
**********
* Add LearningAssistantMessage model

4.3.3 - 2024-10-15
Copy link
Member

Choose a reason for hiding this comment

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

Need to update version number and date.

******************
* Upgraded to use ``Python 3.12``

4.3.3 - 2024-10-15
******************
* Use `LEARNING_ASSISTANT_PROMPT_TEMPLATE` for prompt
Expand Down
1 change: 1 addition & 0 deletions learning_assistant/api.py
Original file line number Diff line number Diff line change
Expand Up @@ -102,6 +102,7 @@ def get_block_content(request, user_id, course_id, unit_usage_key):
return cache_data['content_length'], cache_data['content_items']


# pylint: disable=too-many-positional-arguments)
UsamaSadiq marked this conversation as resolved.
Show resolved Hide resolved
def render_prompt_template(request, user_id, course_run_id, unit_usage_key, course_id, template_string):
"""
Return a rendered prompt template.
Expand Down
2 changes: 1 addition & 1 deletion learning_assistant/text_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ def cleanup_text(text):
return stripped


class _HTMLToTextHelper(HTMLParser): # lint-amnesty, pylint: disable=abstract-method
class _HTMLToTextHelper(HTMLParser):
"""
Helper function for html_to_text below.
"""
Expand Down
33 changes: 15 additions & 18 deletions requirements/base.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
#
# This file is autogenerated by pip-compile with Python 3.11
# This file is autogenerated by pip-compile with Python 3.12
# by the following command:
#
# make upgrade
Expand All @@ -10,15 +10,15 @@ attrs==24.2.0
# via -r requirements/base.in
certifi==2024.8.30
# via requests
cffi==1.17.0
cffi==1.17.1
# via
# cryptography
# pynacl
charset-normalizer==3.3.2
charset-normalizer==3.4.0
# via requests
click==8.1.7
# via edx-django-utils
cryptography==43.0.0
cryptography==43.0.3
# via pyjwt
django==4.2.16
# via
Expand All @@ -33,7 +33,7 @@ django==4.2.16
# edx-drf-extensions
django-crum==0.7.9
# via edx-django-utils
django-model-utils==4.5.1
django-model-utils==5.0.0
# via -r requirements/base.in
django-waffle==4.1.0
# via
Expand All @@ -44,33 +44,33 @@ djangorestframework==3.15.2
# -r requirements/base.in
# drf-jwt
# edx-drf-extensions
dnspython==2.6.1
dnspython==2.7.0
# via pymongo
drf-jwt==1.19.2
# via edx-drf-extensions
edx-django-utils==5.15.0
edx-django-utils==7.0.0
# via
# edx-drf-extensions
# edx-rest-api-client
edx-drf-extensions==10.4.0
# via -r requirements/base.in
edx-opaque-keys==2.10.0
edx-opaque-keys==2.11.0
# via
# -r requirements/base.in
# edx-drf-extensions
edx-rest-api-client==5.7.1
edx-rest-api-client==6.0.0
# via -r requirements/base.in
idna==3.8
idna==3.10
# via requests
jinja2==3.1.4
# via -r requirements/base.in
markupsafe==2.1.5
markupsafe==3.0.2
# via jinja2
newrelic==9.13.0
newrelic==10.2.0
# via edx-django-utils
pbr==6.1.0
# via stevedore
psutil==6.0.0
psutil==6.1.0
# via edx-django-utils
pycparser==2.22
# via cffi
Expand All @@ -79,19 +79,16 @@ pyjwt[crypto]==2.9.0
# drf-jwt
# edx-drf-extensions
# edx-rest-api-client
pymongo==4.8.0
pymongo==4.10.1
# via edx-opaque-keys
pynacl==1.5.0
# via edx-django-utils
requests==2.32.3
# via
# edx-drf-extensions
# edx-rest-api-client
# slumber
semantic-version==2.10.0
# via edx-drf-extensions
slumber==0.7.1
# via edx-rest-api-client
sqlparse==0.5.1
# via django
stevedore==5.3.0
Expand All @@ -100,5 +97,5 @@ stevedore==5.3.0
# edx-opaque-keys
typing-extensions==4.12.2
# via edx-opaque-keys
urllib3==2.2.2
urllib3==2.2.3
# via requests
14 changes: 7 additions & 7 deletions requirements/ci.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
#
# This file is autogenerated by pip-compile with Python 3.11
# This file is autogenerated by pip-compile with Python 3.12
# by the following command:
#
# make upgrade
Expand All @@ -10,25 +10,25 @@ chardet==5.2.0
# via tox
colorama==0.4.6
# via tox
distlib==0.3.8
distlib==0.3.9
# via virtualenv
filelock==3.15.4
filelock==3.16.1
# via
# tox
# virtualenv
packaging==24.1
# via
# pyproject-api
# tox
platformdirs==4.2.2
platformdirs==4.3.6
# via
# tox
# virtualenv
pluggy==1.5.0
# via tox
pyproject-api==1.7.1
pyproject-api==1.8.0
# via tox
tox==4.18.0
tox==4.23.2
# via -r requirements/ci.in
virtualenv==20.26.3
virtualenv==20.27.0
# via tox
Loading
Loading