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

Remove direct dependencies on numpy/scipy in edx-platform #35962

Open
ormsbee opened this issue Dec 4, 2024 · 0 comments
Open

Remove direct dependencies on numpy/scipy in edx-platform #35962

ormsbee opened this issue Dec 4, 2024 · 0 comments
Assignees

Comments

@ormsbee
Copy link
Contributor

ormsbee commented Dec 4, 2024

We still use them in openedx-calc (for now) and openedx-chem (for now), but there are places in edx-platform where we can substitute our current usage with Python stdlib calls, such as:

return dict(
min=min(data),
max=max(data),
mean=np.around(np.mean(data)),
median=np.around(np.median(data)),
mode=stats.mode(data, axis=None)[0],
)

The long term goal would be to isolate numpy/scipy to just the parts of the XBlocks that need them (and possibly isolate them to just the codejailed execution environment). This would reduce memory usage and make upgrades less painful.

We have to be really careful if we're touching anything related to grading.

@ormsbee ormsbee self-assigned this Dec 4, 2024
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

No branches or pull requests

1 participant