Skip to content

Add simplify commands - #1

Merged
j6k4m8 merged 4 commits into
mainfrom
add-simplify-commands
May 6, 2025
Merged

Add simplify commands#1
j6k4m8 merged 4 commits into
mainfrom
add-simplify-commands

Conversation

@j6k4m8

@j6k4m8 j6k4m8 commented May 5, 2025

Copy link
Copy Markdown
Member
  • Updated the simplify_synapse_data function to accept a file-like object instead of a string path for the raw file, improving flexibility.
  • Removed the file handling for the raw file within simplify_synapse_data, allowing it to directly read from the provided stream.
  • Adjusted the main function to reflect the new parameter structure for simplify_synapse_data.
  • Deleted the process_contactome.sh script as its functionality is now integrated into the Python codebase, streamlining the processing workflow.
uv run manage.py synapses simplify --raw-file dynamo-out-40k.csv --output-file ./graph.elist --simple

j6k4m8 added 3 commits May 5, 2025 11:25
and rename export function to reflect its scope better
- Updated the `simplify_synapse_data` function to accept a file-like object instead of a string path for the raw file, improving flexibility.
- Removed the file handling for the raw file within `simplify_synapse_data`, allowing it to directly read from the provided stream.
- Adjusted the `main` function to reflect the new parameter structure for `simplify_synapse_data`.
- Deleted the `process_contactome.sh` script as its functionality is now integrated into the Python codebase, streamlining the processing workflow.

@hannah-martinez hannah-martinez left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Changes look great to me, can you add examples of the new commands in the README before merging

@j6k4m8

j6k4m8 commented May 5, 2025

Copy link
Copy Markdown
Member Author

ooh good idea @hannah-martinez :)

@j6k4m8
j6k4m8 requested a review from hannah-martinez May 6, 2025 12:11
@j6k4m8
j6k4m8 merged commit f0d5785 into main May 6, 2025
@j6k4m8
j6k4m8 deleted the add-simplify-commands branch May 6, 2025 15:22
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