Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • 19:45

    DailyDreaming on 2006-fix-aws-docs-ordering

    (compare)

  • 19:45

    DailyDreaming on master

    Fix aws docs numbering broken b… (compare)

  • 19:45
    DailyDreaming closed #3596
  • 18:53
    w-gao opened #3596
  • 18:49

    w-gao on 2006-fix-aws-docs-ordering

    Fix aws docs numbering broken b… (compare)

  • 16:44
    DailyDreaming synchronize #3572
  • 16:44

    DailyDreaming on 3537-fix-cwl-test-20

    Fix type hints (#3590) Add type hints to misc.py (#359… Update cactus test to fix broke… and 1 more (compare)

  • 16:44
    DailyDreaming synchronize #3595
  • 16:44

    DailyDreaming on 3461-stream-from-cache

    Update cactus test to fix broke… Merge branch 'master' into issu… (compare)

  • 01:18

    DailyDreaming on 3593-update-cactus

    (compare)

  • 01:18

    DailyDreaming on master

    Update cactus test to fix broke… (compare)

  • 01:18
    DailyDreaming closed #3594
  • May 06 18:53

    DailyDreaming on 3593-update-cactus

    Cruft. (compare)

  • May 06 18:53
    DailyDreaming synchronize #3594
  • May 06 17:16
    DailyDreaming synchronize #3594
  • May 06 17:16

    DailyDreaming on 3593-update-cactus

    Add missing req. (compare)

  • May 06 04:57
    DailyDreaming closed #3519
  • May 05 21:02
    adamnovak review_requested #3595
  • May 05 21:02
    adamnovak opened #3595
  • May 05 20:59

    adamnovak on 3461-stream-from-cache

    Handle streaming reads from the… (compare)

Lon Blauvelt
@DailyDreaming
@karma29:matrix.org What's the issue? @adamnovak D:
Lon Blauvelt
@DailyDreaming
[Adam Novak, UCSC GI] Oh, looks like it works.
karma29
@karma29:matrix.org
[m]

hello! just curious about the code here : https://github.com/DataBiosphere/toil/blob/master/src/toil/cwl/cwltoil.py#L736

mutable is by default set to false, which means that there is no downloaded copy of the file (and a link to it is created instead). in the function call examples, there wasn't any explicit declaration of mutable to true, so do they exhibit "streamable" properties? what more changes should we make here?

karma29
@karma29:matrix.org
[m]
actually i'm a gsoc'21 applicant so i wasn't sure if it would be more suitable to reply to the comment threads on github or join the irc. what's recommended though? i'm fine with either! 😃
Lon Blauvelt
@DailyDreaming
[Adam Novak, UCSC GI] We watch Github issues, and the chat here, but not really Github code comments. Either there or here is fine; here is maybe better fro questions that are not themselves bugs.
[Adam Novak, UCSC GI] I think the idea behind CWL streamable is that you will get a pipe (a FIFO) presented to the tool instead of a normal file? I'm not really sure.
Peter Amstutz
@tetron
yes
Lon Blauvelt
@DailyDreaming
[Adam Novak, UCSC GI] Anyway, readGlobalFile produces an ordinary file, with a filename and all the data on disk. It might be given via a symlink, and the lack of mutable means the user code isn't allowed to modify it, as other jobs may be using the same copy.
Peter Amstutz
@tetron
and if the data is coming from a remote location, it can be streamed incrementally instead of waiting for a full download
Lon Blauvelt
@DailyDreaming
[Adam Novak, UCSC GI] Toil already has a file_store.readGlobalFileStream function, but that returns a Python file object, and doesn't currently produce a FIFO on disk. So I think getting CWL streamable support would involve handling streamable requests from CWL by using file_store.readGlobalFileStream and producing a FIFO and a thread to fill it with data.
Michael R. Crusoe
@mr-c
Or by using a third party utility to achieve the same thing, given the s3 url
karma29
@karma29:matrix.org
[m]

okay, i see! thank you 👍️ so we need to implement a FIFO process for

  1. a sequence of files (irrespective of whether they're downloaded or not, because i think right now it waits for all the input files to be processed to release the outputs?)
  2. a sequence of data within the file, in case the file comes from a remote location or s3 url (so we don't need to download it)
  3. combination of the above two, if there are more files from a remote location or s3 url in the pipe

is that correct?

Lon Blauvelt
@DailyDreaming
@karma29:matrix.org I'm sorry, it seems the matrix.org connection doesn't sync to our slack channel, so your message was missed. In general, this will need to be one FIFO per file. I would focus on the AWS s3 functionality alone first.
I'll try to check the gitter manually more often.
Michael R. Crusoe
@mr-c
@DailyDreaming does your slack connection sync via gitter? Maybe switch to directly syncing your slack channel with the matrix version?
Lon Blauvelt
@DailyDreaming
I'll try that.
Thanks!
karma29
@karma29:matrix.org
[m]
Okay! Is there a link to the slack group i can join?
1 reply
I'd be grateful if you could give feedback on it! I'm a bit confused regarding how we would actually implement streamable properties, except for the part where we change file_store.readGlobalFileStream so any inputs regarding that would be much appreciated
karma29
@karma29:matrix.org
[m]
Also is a "job class" a Python class? I have heard of text files getting pickled. So does a job class getting pickled mean that the job class is being implemented as an object and then their contents being pickled to a file?
Lon Blauvelt
@DailyDreaming
[Adam Novak, UCSC GI] The jobs in Toil are instances of various "job classes", like CWLJob and JobFunctionWrappingJob. We do indeed instantiate these classes and then pickle the resulting objects into files.
karma29
@karma29:matrix.org
[m]

Okay, thank you for the feedback!

I've made some changes - if it looks good, may I go ahead and submit the proposal?

karma29
@karma29:matrix.org
[m]

Hello!

Just a follow-up to my previous message, since there's ~ 2 hours for the submission to end 😃 crusoe @DailyDreaming

1 reply
karma29
@karma29:matrix.org
[m]
Thank you, just updated with another query. Can you please check the doc? Thank you crusoe @DailyDreaming
Dennis R Kennetz
@drkennetz
Is it appropriate to post job openings in here? We have a more senior position open that is CWL/toil related for bioinformatics.
Michael Milton
@TMiguelT
Is there any interest in my suggestion here: https://github.com/DataBiosphere/toil/issues/1768#issuecomment-818475173 ? I'd be happy to give it a go if it's okayed by a maintainer
Lon Blauvelt
@DailyDreaming
[Adam Novak, UCSC GI] It sounds like a good idea to me!
Lon Blauvelt
@DailyDreaming
@drkennetz That's fine by me.