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

Add automatic FCS file reading from FlowJo WSP files #168

Merged
merged 6 commits into from
Sep 12, 2023
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Handle failure to find files by URI
Signed-off-by: Hersh Bhargava <hershkbhargava@gmail.com>
  • Loading branch information
hbhargava7 committed Sep 4, 2023
commit 3d00e290129b05695accc978f29db3cee8345564
12 changes: 8 additions & 4 deletions flowkit/_models/workspace.py
Original file line number Diff line number Diff line change
Expand Up @@ -100,12 +100,16 @@ def uri_to_path(uri):
path = uri_to_path(sample_uri)

# Read in the sample files
sample_filedata = sample_utils.load_samples(path)[0]
try:
sample_filedata = sample_utils.load_samples(path)[0]

# Update the ID of the loaded data (otherwise analysis breaks)
sample_filedata.id = sample_name
# Update the ID of the loaded data (otherwise analysis breaks)
sample_filedata.id = sample_name

tmp_sample_lut[sample_name] = sample_filedata
tmp_sample_lut[sample_name] = sample_filedata

except FileNotFoundError:
warnings.warn("Sample file not found at path: {}".format(path))

# save group sample membership, we'll filter by loaded samples next
group_lut = wsp_data['groups']
Expand Down