profile
viewpoint
Régis Behmo regisb France https://minutebutterfly.com Freelance teacher, fullstack dev, data scientist, data architect. Expert in machine learning, computer vision, Python. I work remotely EXCLUSIVELY.

overhangio/tutor 573

The docker-based Open edX distribution designed for peace of mind

overhangio/tutor-discovery 10

Course Discovery plugin for Tutor

overhangio/tutor-ecommerce 10

Ecommerce plugin for Tutor

overhangio/tutor-figures 10

Figures plugin for Tutor, for analytics and reporting

openedx/build-test-release-wg 9

Open edX Build / Test / Release Working Group

overhangio/tutor-minio 6

MinIO plugin for Tutor

overhangio/openedx-e2e 5

End-to-end testing for any live-running Open edX platform [BETA]

overhangio/tutor-xqueue 5

Xqueue plugin for Tutor

overhangio/tutor-notes 4

Student notes plugin for Tutor

issue commentopenedx/build-test-release-wg

There is no way to disable identity verification (IDV) in the LMS

Thanks for investigating this issue @ashultz0. How did you run ecommerce? In the devstack, the native installation or with Tutor? How did you manage to configure the course SKU? IDV messages should be considered bugs on platforms that do not make use of the feature, so could you please post screenshots of the places where the messages appeared?

regisb

comment created time in 38 minutes

PullRequestReviewEvent

issue commentoverhangio/tutor

Support for ARM64

That's great, thanks a lot.

Would you prefer I write it as a forum post or open a PR against the documentation?

Whichever you are most comfortable with. You have a good understanding of how Tutor works, so I guess you don't necessarily have to go through the forum for validation first. Feel free to add a commit on top of your PR, or open a different one.

throwaway-a

comment created time in 7 hours

pull request commentedx/edx-platform

feat: theme-agnostic view to fetch theme assets

Where/how can we document this?

I have no idea. It is not an API endpoint, setting or a feature flag. Maybe we should be adding some words here? https://edx.readthedocs.io/projects/edx-installing-configuring-and-running/en/latest/configuration/changing_appearance/theming/enable_themes.html

regisb

comment created time in 9 hours

pull request commentedx/edx-platform

feat: theme-agnostic view to fetch theme assets

I believe that you should define the LOGO_TRADEMARK_URL environment setting in your MFE to point to the platform logo.

regisb

comment created time in 11 hours

push eventoverhangio/tutor

Régis Behmo

commit sha c3e6f57b08a90854caa83ade104c3c8aba89bbb4

feat: fail `images ... [image]` commands on incorrect image arg Previously, `tutor images build doesnotexist` was a no-op. This was counter-intuitive -- in particular, when users have a typo in their command. Now, the `images build/pull/push/printtag` commands fail if the corresponding image was not found. See discussion: https://discuss.overhang.io/t/return-something-when-tutor-images-build-contains-a-typo/2174

view details

Régis Behmo

commit sha c1525787835da8badfed820c1c968bf001e3bd56

Merge branch 'master' into nightly

view details

push time in 13 hours

delete branch overhangio/tutor

delete branch : regisb/check-image-names

delete time in 14 hours

push eventoverhangio/tutor

Régis Behmo

commit sha c3e6f57b08a90854caa83ade104c3c8aba89bbb4

feat: fail `images ... [image]` commands on incorrect image arg Previously, `tutor images build doesnotexist` was a no-op. This was counter-intuitive -- in particular, when users have a typo in their command. Now, the `images build/pull/push/printtag` commands fail if the corresponding image was not found. See discussion: https://discuss.overhang.io/t/return-something-when-tutor-images-build-contains-a-typo/2174

view details

push time in 14 hours

PR merged overhangio/tutor

feat: fail `images build/pull/push` commands on incorrect image arg

Previously, tutor images build doesnotexist was a no-op. This was counter-intuitive -- in particular, when users have a typo in their command.

Now, the command fails if the corresponding image was not found.

See discussion: https://discuss.overhang.io/t/return-something-when-tutor-images-build-contains-a-typo/2174

+46 -8

4 comments

2 changed files

regisb

pr closed time in 14 hours

issue commentoverhangio/tutor

Support for ARM64

Hey @bradenmacdonald thanks for the comprehensive explanations! Considering the results of the benchmarks you made, I agree with you that the best way to support the arm64 platform is to advise users to switch to MariaDB. They would also have to build the tutor-specific Docker images themselves, thanks to that #543 PR you opened. Would you have time to write a tutorial on how to run Tutor on arm64/Apple M1? If not I can do it myself, based on your testing and reports.

throwaway-a

comment created time in 15 hours

Pull request review commentoverhangio/tutor

Unit tests Improvement

 def convert_json2yml(root: str) -> None:         return     if os.path.exists(config_path(root)):         raise exceptions.TutorError(-            "Both config.json and config.yml exist in {}: only one of these files must exist to continue".format(-                root-            )+            f"Both config.json and {_CONFIG_FILE} exist in {root}: only one of these files must exist to continue"

Well, yes because we only dropped the python 3.5 compatibility in the nightly branch, while your PR targets the master branch.

takato1314

comment created time in 2 days

PullRequestReviewEvent

PR opened overhangio/tutor-ecommerce

feat: WIP upgrade to Maple

This is currently non-functional. At the moment, this branch exists to allow other people to troubleshoot the issues listed here: https://github.com/openedx/build-test-release-wg/projects/1?card_filter_query=label%3A%22help+wanted%22+label%3Aaffects%3Amaple

+46 -51

0 comment

11 changed files

pr created time in 2 days

create barnchoverhangio/tutor-ecommerce

branch : maple

created branch time in 2 days

issue commentopenedx/platform-roadmap

Working Ecommerce in Maple

So far I labelled these issues as "help wanted" and "affects:maple" in the BTR working group project board: https://github.com/openedx/build-test-release-wg/projects/1?card_filter_query=label%3A%22help+wanted%22+label%3Aaffects%3Amaple

e0d

comment created time in 2 days

issue openedopenedx/build-test-release-wg

In the payment MFE, the Cybersource payment processor does not work

The payment fields in the payment MFE are greyed out when attempting to make payments with the Cybersource payment processor. Although I did not manage to get to a point where I could reproduce the issue, there are multiple credible reports from actual users:

https://discuss.overhang.io/t/have-you-configured-cybersource-in-liliac/2170/1 https://discuss.openedx.org/t/payment-mfe-configuration/5375/3 https://discuss.overhang.io/t/payment-mfe-and-cybersource/1864

@sambapete has reported that the Cybersource payment processor works in the devstack, so it might only be a matter of backend configuration.

created time in 2 days

delete branch overhangio/tutor

delete branch : regisb/drop-python-3.5

delete time in 2 days

push eventoverhangio/tutor

Régis Behmo

commit sha 1f8555b80e35c16569127bedcc358120ab93b184

chore: drop python 3.5 compatibility Python 3.5 has reached end of life in September 3.5. Anyway, Tutor was not compatible because some dev dependencies, such as astroid 2.8.3, are no longer available in 3.5. This means that we can now start using many python 3.6 niceties, such as f-strings \o/

view details

push time in 2 days

PR merged overhangio/tutor

chore: drop python 3.5 compatibility

Python 3.5 has reached end of life in September 3.5. Anyway, Tutor was not compatible because some dev dependencies, such as astroid 2.8.3, are no longer available in 3.5.

This means that we can now start using many python 3.6 niceties, such as f-strings \o/

+2 -2

0 comment

2 changed files

regisb

pr closed time in 2 days

push eventoverhangio/tutor

Maxim Beder

commit sha 19157d94bca8a4998271082045448a919bcd4761

feat: allow to specify extra pip packages in config Added OPENEDX_EXTRA_PIP_REQUIREMENTS setting, which allows to specify extra pip packages that should be installed. Moved "openedx-scorm-xblock" package from Dockerfile to the new setting in the config.yml.

view details

Régis Behmo

commit sha e9e1df9e701c00083d2311eb5b1e6721f98b21f8

Merge branch 'master' into nightly

view details

push time in 2 days

PR opened overhangio/tutor-notes

feat: upgrade to Maple
+43 -42

0 comment

14 changed files

pr created time in 2 days

create barnchoverhangio/tutor-notes

branch : maple

created branch time in 2 days

pull request commentedx/edx-platform

feat: theme-agnostic view to fetch theme assets

This is ready for review.

regisb

comment created time in 3 days

push eventoverhangio/tutor

Maxim Beder

commit sha 19157d94bca8a4998271082045448a919bcd4761

feat: allow to specify extra pip packages in config Added OPENEDX_EXTRA_PIP_REQUIREMENTS setting, which allows to specify extra pip packages that should be installed. Moved "openedx-scorm-xblock" package from Dockerfile to the new setting in the config.yml.

view details

push time in 3 days

PR merged overhangio/tutor

feat: allow to specify extra pip packages in config

Description

Allow specifying extra pip packages through OPENEDX_EXTRA_PIP_REQUIREMENTS in config.yml, akin to how it's done in tutor-ecommerce.

Set the default value for OPENEDX_EXTRA_PIP_REQUIREMENTS setting to "openedx-scorm-xblock<13.0.0,>=12.0.0" and removed the corresponding line from Dockerfile.

Useful links

+31 -4

11 comments

4 changed files

Cup0fCoffee

pr closed time in 3 days

pull request commentoverhangio/tutor

feat: allow to specify extra pip packages in config

Thanks for your PR @Cup0fCoffee!

Cup0fCoffee

comment created time in 3 days

PullRequestReviewEvent

pull request commentdr-leo/PyHyphen

chore: upgrade python requirements

I managed to install pyhyphen==4.0.2 with Pyuthon 3.10.0 :+1:

$ pyenv install 3.10.0
Downloading Python-3.10.0.tar.xz...
-> https://www.python.org/ftp/python/3.10.0/Python-3.10.0.tar.xz
Installing Python-3.10.0...
patching file aclocal.m4
patching file configure
Hunk #5 succeeded at 10537 (offset -15 lines).
Installed Python-3.10.0 to /home/regis/.pyenv/versions/3.10.0
$ ~/.pyenv/versions/3.10.0/bin/python -m venv venv3.10
$ source venv3.10/bin/activate
$ pip install pyhyphen
Collecting pyhyphen
  Downloading PyHyphen-4.0.2.tar.gz (39 kB)
Collecting wheel>=0.36.0
  Using cached wheel-0.37.0-py2.py3-none-any.whl (35 kB)
Requirement already satisfied: setuptools>=52.0 in ./venv3.10/lib/python3.10/site-packages (from pyhyphen) (57.4.0)
Collecting appdirs>=1.4.0
  Using cached appdirs-1.4.4-py2.py3-none-any.whl (9.6 kB)
Collecting requests>=2.25
  Using cached requests-2.26.0-py2.py3-none-any.whl (62 kB)
Collecting charset-normalizer~=2.0.0
  Downloading charset_normalizer-2.0.8-py3-none-any.whl (39 kB)
Collecting urllib3<1.27,>=1.21.1
  Using cached urllib3-1.26.7-py2.py3-none-any.whl (138 kB)
Collecting certifi>=2017.4.17
  Using cached certifi-2021.10.8-py2.py3-none-any.whl (149 kB)
Collecting idna<4,>=2.5
  Using cached idna-3.3-py3-none-any.whl (61 kB)
Using legacy 'setup.py install' for pyhyphen, since package 'wheel' is not installed.
Installing collected packages: urllib3, idna, charset-normalizer, certifi, wheel, requests, appdirs, pyhyphen
    Running setup.py install for pyhyphen ... done
Successfully installed appdirs-1.4.4 certifi-2021.10.8 charset-normalizer-2.0.8 idna-3.3 pyhyphen-4.0.2 requests-2.26.0 urllib3-1.26.7 wheel-0.37.0
WARNING: You are using pip version 21.2.3; however, version 21.3.1 is available.
You should consider upgrading via the '/home/regis/tmp/venv3.10/bin/python -m pip install --upgrade pip' command.
(venv3.10) $ python
Python 3.10.0 (default, Nov 30 2021, 11:23:31) [GCC 9.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from hyphen import Hyphenator
>>> h = Hyphenator('de_DE')
>>> s = 'Politikverdrossenheit'
>>> h.pairs(s)
[['Po', 'litikverdrossenheit'], ['Poli', 'tikverdrossenheit'], ['Politik', 'verdrossenheit'], ['Politikver', 'drossenheit'], ['Politikverdros', 'senheit'], ['Politikverdrossen', 'heit']]

Success!

regisb

comment created time in 3 days

PR opened edx/frontend-app-gradebook

feat: add compatibility with transifex

Now that the frontend-app-gradebook resource was added to Transifex, we can start pulling strings fromt there. For now, the project contains very few translated string.

Strings were pulled by running: make pull_translations

This is for https://github.com/openedx/build-test-release-wg/issues/107

Cherry-pick of https://github.com/edx/frontend-app-gradebook/pull/217

FYI: @edx/masters-devs-gta

+362 -10

0 comment

7 changed files

pr created time in 3 days

more