-
-
Notifications
You must be signed in to change notification settings - Fork 42
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
Use same data object for teal_data and in modules #937
Labels
Comments
21 tasks
Fair enough, so maybe CodeClass could be built on top of qenv rather than being completely independent etc. |
Closed
3 tasks
This was referenced Aug 11, 2023
Closed
This was referenced Oct 4, 2023
This was referenced Oct 18, 2023
This comment was marked as outdated.
This comment was marked as outdated.
This was referenced Nov 1, 2023
Merged
This was referenced Nov 22, 2023
Merged
vedhav
added a commit
that referenced
this issue
Dec 8, 2023
part of #937 --------- Signed-off-by: Vedha Viyash <49812166+vedhav@users.noreply.github.com> Signed-off-by: Aleksander Chlebowski <114988527+chlebowa@users.noreply.github.com> Signed-off-by: kartikeya kirar <kirar.kartikeya1@gmail.com> Signed-off-by: André Veríssimo <211358+averissimo@users.noreply.github.com> Co-authored-by: Aleksander Chlebowski <114988527+chlebowa@users.noreply.github.com> Co-authored-by: Dony Unardi <donyunardi@gmail.com> Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: André Veríssimo <211358+averissimo@users.noreply.github.com> Co-authored-by: Aleksander Chlebowski <aleksander.chlebowski@contractors.roche.com> Co-authored-by: kartikeya kirar <kirar.kartikeya1@gmail.com> Co-authored-by: Vedha Viyash <49812166+vedhav@users.noreply.github.com> Co-authored-by: 27856297+dependabot-preview[bot]@users.noreply.github.com <27856297+dependabot-preview[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
todo: Send
teal_data
to the modules instead oftdata
.This will break all the modules, so it needs to be done together with:
datasets
arg indata_extract_srv
,merge_expression_srv
to work withteal_data
teal.transform#159teal_data
instead oftdata
teal.modules.general#588teal_data
instead oftdata
teal.modules.clinical#838teal_data
instead oftdata
teal.modules.hermes#338teal_data
instead oftdata
teal.osprey#230teal_data
instead oftdata
teal.goshawk#242The text was updated successfully, but these errors were encountered: