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

Feature: use fanc meta from neck connective repo #22

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

jefferis
Copy link
Contributor

specifically from https://github.com/flyconnectome/2023neckconnective/blob/dev/data/fanc-neckconnective-anns.tsv

This should get us going with fanc-manc matching. But it is also possible to use another local source by setting options(coconatfly.fanc_meta). This can be specified either as a function or a path to a local file on disk.

options(coconatfly.fanc_meta=function() {
      fafbseg::flywire_sirepo_file_memo('https://github.com/flyconnectome/2023neckconnective/blob/dev/data/fanc-neckconnective-anns.tsv', read=TRUE)
    })

note that the use of an anonymous function will delay evaluation of what is in inside the function until the option is queried. In this case flywire_sirepo_file_memo will update a local copy of the file from github if required.

options(coconatfly.fanc_meta='~/flyconnectome/data/fanc-neckconnective-anns.tsv')

* this is a hack of course
* if we keep it then we should try to make it more generic
  and allow other datasets to do something similar
* necessary if we want to do something such as trigger/update a remote download
* sourced from neck connective repo
* needed for reading tsvs
@jefferis jefferis force-pushed the feature/more-fanc-meta branch from d6d0da1 to 895549d Compare January 2, 2025 13:27
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.

1 participant