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.

Sign up for GitHub

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

Add automatic FCS file reading from FlowJo WSP files #168

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
ee631fc
Update `Workspace.__init__` and `_parse_wsp_samples` to find FCS file…
hbhargava7 Aug 31, 2023
194f713
Update the sample filedata id to match the FlowJo name
hbhargava7 Sep 1, 2023
964f59d
Update `find_fcs_files_from_wsp` to handle relative paths
hbhargava7 Sep 4, 2023
3d00e29
Handle failure to find files by URI
hbhargava7 Sep 4, 2023
c9be7fb
Bug fix in error handling
hbhargava7 Sep 4, 2023
b314cd6
Bug fix in relative path handling for fcs file autofinding
hbhargava7 Sep 4, 2023
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
Bug fix in error handling 
Signed-off-by: Hersh Bhargava <hershkbhargava@gmail.com>
  • Loading branch information
@hbhargava7
hbhargava7 committed Sep 4, 2023
commit c9be7fbba7a2fd83d637cdf9df8cc7f4028340f5
2 changes: 1 addition & 1 deletion flowkit/_models/workspace.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ def uri_to_path(uri):

tmp_sample_lut[sample_name] = sample_filedata

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

# save group sample membership, we'll filter by loaded samples next
Expand Down