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

Update main.py to load the dataset from separate file. #2

Closed
wants to merge 4 commits into from

Conversation

aelnosu
Copy link

@aelnosu aelnosu commented Aug 6, 2024

This MR/PR update main.py to load datasets from separate file, also marks the datasets as binaries so git diff and merge does ignores those files.

aelnosu added 3 commits August 5, 2024 22:00
This commit extracts datasets out of main.py into separate files to inprove programing experiences.
This also mark the datasets as bins so git does not try to diff/merge them.
Also modify the datasets in main.py to be empty types for intellisense.
@aelnosu
Copy link
Author

aelnosu commented Aug 6, 2024

PS. It would be great if this project is licensed under AGPL v3 or other licenses.

@ZribeDev
Copy link
Contributor

ZribeDev commented Aug 6, 2024

Hi, please rename the datasets into .py files so u can just directly import them using the python import statement

@Ubuntufanboy
Copy link

You could also do

with open("dataset.txt") as f:
    dataset_as_string = f.read()

@aelnosu
Copy link
Author

aelnosu commented Aug 7, 2024

@Ubuntufanboy Just by your name, I would assume that you are pretty familiar with GNU/Linux. I have spent the past few hours trying (and failing) to dockerize the project. The main challenge is trying to pass /dev/snd to the Docker container and make pygame work. I hope you can help.

Eason

@aelnosu
Copy link
Author

aelnosu commented Aug 7, 2024

I have tried to use ALSA but without the --privileged flag it does not seem to work.

@ZribeDev
Copy link
Contributor

ZribeDev commented Aug 7, 2024

You could also do

with open("dataset.txt") as f:
    dataset_as_string = f.read()

that is not good. instead of reading a file just use python itself

@aelnosu aelnosu closed this Jan 22, 2025
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

Successfully merging this pull request may close these issues.

3 participants