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

metadata &textfile generator script #777

Open
wants to merge 6 commits into
base: testing
Choose a base branch
from

Conversation

Alexanderlacuna
Copy link
Contributor

Description

How should this be tested?

Any background context you want to provide?

What are the relevant pivotal tracker stories?

Screenshots (if appropriate)

Questions

@Alexanderlacuna
Copy link
Contributor Author

@BonfaceKilz @zsloan could you help in reviewing this

@Alexanderlacuna
Copy link
Contributor Author

you can use this branch for testing:#780

@zsloan
Copy link
Contributor

zsloan commented May 15, 2023

I'm testing this now (btw sorry about the delay on that - in the future feel free to just bug me over e-mail about stuff like this), and I think it's working correctly because it's been running a long time without errors.

How do I test that it's run correctly? It's still running, but the directory I think it's writing to (in this case /tmp/Probesets) hasn't been updated since soon after I ran the script (which might be normal, but not sure).

@zsloan
Copy link
Contributor

zsloan commented May 15, 2023

After running for a while, I got the following error (which seems to be due to hitting some size limit and is likely due to the LMDB installation on Tux01 or something rather than anything in the script):
lmdb.MapFullError: mdb_put: MDB_MAP_FULL: Environment mapsize limit reached

@Alexanderlacuna
Copy link
Contributor Author

After running for a while, I got the following error (which seems to be due to hitting some size limit and is likely due to the LMDB installation on Tux01 or something rather than anything in the script): lmdb.MapFullError: mdb_put: MDB_MAP_FULL: Environment mapsize limit reached

it has to do with allocating size of lmdb I will fix this ....Hope this is not occurring for --metadata they are quite small.

@zsloan
Copy link
Contributor

zsloan commented May 16, 2023

Ah thanks, I'll try it again with that latest commit.

@fredmanglis fredmanglis force-pushed the testing branch 2 times, most recently from 04f10df to 5474e66 Compare April 24, 2024 04:54
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.

None yet

2 participants