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

Merge repos #6

Merged
merged 23 commits into from
Feb 4, 2021
Merged

Merge repos #6

merged 23 commits into from
Feb 4, 2021

Conversation

tacazares
Copy link
Member

This is the code used for the Tn5 window analysis and is a copy of the code Balaji and Faiz are using with additional ROI, prediction, and benchmarking code.

Cazares, Tareian added 23 commits January 17, 2021 23:55
There was an issue in the data generator where the for loop terminated earlier than it was supposed to because it was improperly indented.
This code was hanging on OSC with the use of keras.fit() instead of fit_generator(). I switched back to fit_generator(). I was able to run on my laptop, but will test OSC.
I tried using the fit() method instead of the fit_generator method with a class object I made, but it would prematuraely exit. I will test this version on OSC.
I did a rough merge of the maxATAC repo for the Miraldi Lab. This repo mostly had average, normalization, and docs to contribute. I merge it with the branch balaji has been working on : maxatac_v2 branch pcpc-rr-MM-Q-hg38-mCL
There was an error with embedding an image into a pdf so I changed the extensions to png.
The headers that the ROI generator was producing was different than the ones that the training script was referencing.
@tacazares tacazares merged commit 42e9df0 into main Feb 4, 2021
@tacazares tacazares deleted the merge_repos branch February 4, 2021 02:44
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.

None yet

1 participant