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

get_reactions_kcat_mapping error with Salmonella species #23

Open
FraserAndrews7 opened this issue Nov 12, 2022 · 1 comment
Open

get_reactions_kcat_mapping error with Salmonella species #23

FraserAndrews7 opened this issue Nov 12, 2022 · 1 comment

Comments

@FraserAndrews7
Copy link

Hi @Paulocracy,
I would like to say congratulations on this an absolutely fantastic piece of code/paper you published :D

I have however ran into a problem with the kcat_database_combined.json file that is generated from the create_combined_kcat_database.py function. It creates kcat values for the species Salmonella enterica subsp. enterica serovar Typhimurium A0A0F6B484. This entry is not in the cache/ncbi_taxonomy folder. This causes an error later on when I use the function...

get_reactions_kcat_mapping(sbml_path, project_folder, project_name, organism, kcat_database_path, protein_kcat_database_path)

It runs halfway before raising the error 'OSError: [Errno 22] Invalid argument: C:/file_path/cache/ncbi_taxonomy/Salmonella enterica subsp. enterica serovar Typhimurium A0A0F6B484'.

It is possible to solve and allow complete model construction by manually deleting the 'Salmonella enterica subsp. enterica serovar Typhimurium A0A0F6B484' entries in the kcat_database_combined.json file.

Whilst it is possible to work around this, I thought it would be worth raising this for anyone else whom may have this problem.

Keep up the good work :D

@Paulocracy
Copy link
Member

Thank you (very late :3) for mentioning this issue and your positive assessment of AutoPACMEN! I will keep this issue open for anyone with a similar problem. Since a solution would need major changes in AutoPACMEN for which I didn't have time yet, your workaround is the best solution for now :-)

@Paulocracy Paulocracy pinned this issue Nov 20, 2024
@Paulocracy Paulocracy changed the title get_reactions_kcat_mapping get_reactions_kcat_mapping error with Salmonella species Nov 20, 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

2 participants