Skip to content

Replace Jupyter NB with PY script to generate replicates and update README#7

Merged
shachafl merged 2 commits intomainfrom
dev
Apr 12, 2025
Merged

Replace Jupyter NB with PY script to generate replicates and update README#7
shachafl merged 2 commits intomainfrom
dev

Conversation

@shachafl
Copy link
Copy Markdown
Collaborator

No description provided.

@shachafl shachafl requested a review from Copilot April 12, 2025 02:20
@github-actions
Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

README.md:36

  • The description still refers to a 'Jupyter notebook' despite linking to a Python script; update the wording to accurately reflect that the tool is now a Python script.
We used our Jupyter notebook [Generate_replicates_for_network_inference.py](/CODE/Generate_replicates_for_network_inference.py) to call GeneNetWeaver to generate multiple replicates of steady-state and time-series gene expression datasets for realistic in silico networks of sizes of 50, and 100 genes containing various experimental conditions (knockouts, knockdowns, multifactorial perturbation, etc.).<br>

CODE/Generate_replicates_for_network_inference.py:60

  • Using os.getcwd() here can be fragile if the working directory is not the expected simulation output folder; consider explicitly storing and referencing the simulation output directory.
for generated_file in os.listdir(os.getcwd()):

@shachafl shachafl merged commit 9ed4127 into main Apr 12, 2025
4 checks passed
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.

2 participants