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

Transition away from Conda #5

Open
mrl280 opened this issue Jun 26, 2024 · 0 comments · May be fixed by #7
Open

Transition away from Conda #5

mrl280 opened this issue Jun 26, 2024 · 0 comments · May be fixed by #7
Labels
dependency 📦 This issue pertains to dependency changes or deprecation warnings enhancement ✨ An improvement or new feature priority | medium 🌟 This issue is important and will require attention soon, but does not pose any immediate risks

Comments

@mrl280
Copy link
Collaborator

mrl280 commented Jun 26, 2024

Description:
Currently, we use Conda for this example because it is the easiest way to install GDAL, which is required to read the spectrogram masks from the Spectrogram Sensing dataset.

However, Google Colab doesn't provide native Conda support. Instead, we'd have to use condacolab, which at the time of writing only supports Python 3.10 and older versions.

Consequently, we'd like to transition this example away from Conda, which requires converting the Spectrogram Sensing dataset so we can remove GDAL as a project dependency. Luckily, we are converting the MathWork's Spectorgram Sensing dataset to an HDF5 RIA dataset anyway, as part of this issue here: Convert MathWorks' Spectrogram Sensing dataset to HDF5.

Once this transition is complete, this project needs to be updated to download and use the new HDF5 file, instead of the individual .PNG and .HDF frames.

Project Area:

  • Dataset
  • Virtual environment

Additional context:
As part of these changes, we should consider downgrading to Python 3.10.

We do not want to use the RIA radio dataset or its dataset builder; only the source file. The rationale is that in the future, we plan to create another example that leverages RIA tooling to simplify the end-to-end model training and validation process. Therefore, we aim to limit this example as much as possible to what's achievable with the current tooling.

We will need to replace the environment.yaml file with a requirements.txt file containing the new environment definition.

@mrl280 mrl280 added enhancement ✨ An improvement or new feature dependency 📦 This issue pertains to dependency changes or deprecation warnings labels Jun 26, 2024
@mrl280 mrl280 added the priority | medium 🌟 This issue is important and will require attention soon, but does not pose any immediate risks label Jun 28, 2024
@mrl280 mrl280 linked a pull request Jul 19, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependency 📦 This issue pertains to dependency changes or deprecation warnings enhancement ✨ An improvement or new feature priority | medium 🌟 This issue is important and will require attention soon, but does not pose any immediate risks
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant