-
Notifications
You must be signed in to change notification settings - Fork 173
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
Added information on fields for storing DNA methylation information #161
Conversation
VCFv4.3.tex
Outdated
store information on DNA methylation from bisulfite sequencing | ||
experiments and other sources alongside information about genome | ||
sequence variation. Zero length fields are not allowed, a dot (".") | ||
should be used instead. In order to ensure interoperability across |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should? MUST!
No problem changing ‘should’ to ‘must’. Do I have to submit a new pull request to do this?
Simon
… On 3 Feb 2017, at 16:24, Yossi Farjoun ***@***.***> wrote:
@yfarjoun commented on this pull request.
In VCFv4.3.tex <#161 (review)>:
> It contains meta-information lines (prefixed with "\#\#"), a header
-line (prefixed with "\#"), and data lines
-each containing information about a position in the genome and genotype
-information on samples for each position
-(text fields separated by tabs). Zero length fields are not allowed, a dot (".") should
-be used instead.
-In order to ensure interoperability across platforms, VCF compliant implementations must support
-both LF (\texttt{\textbackslash n}) and CR+LF (\texttt{\textbackslash r\textbackslash n}) newline conventions.
+line (prefixed with "\#"), and data lines each containing information
+about a position in the genome and genotype information on samples for
+each position (text fields separated by tabs). The VCF format can also
+store information on DNA methylation from bisulfite sequencing
+experiments and other sources alongside information about genome
+sequence variation. Zero length fields are not allowed, a dot (".")
+should be used instead. In order to ensure interoperability across
should? MUST!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#161 (review)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ADHPd0eD0N93t1Ps7Utg1B038ml5Dmrzks5rY0aigaJpZM4JZnC7>.
|
It is not necessary to create a new pull request, but please rebase your branch against master to ensure that conflicts are resolved. Then you can safely replace "should" with "must" where still necessary. |
@heathsc It also helps if when you create future pull requests you create it on a branch and then tick the box on the pull request allowing allowing maintainers with write access to commit to your branch. |
@MarcosFernandez, could you please rebase these changes against the master branch? As you can see on this page, there are several files with conflicts, even though they are not related with the VCF specification. |
Conflicts are already solved. Is it able to merge?? |
this is a bit of a stale PR. if there's still interest in it, please rebase on master and re-open. |
There is certainly interest, however we never got any feedback after the previous attempt. The approval process does appear to be very opaque! For IHEC it is important that the methylation specific tags are documented officially as part of the VCF spec - so how should we proceed to make this happen?
Simon
… On 11 Apr 2019, at 17:51, Yossi Farjoun ***@***.***> wrote:
this is a bit of a stal PR. if there's still interest in it, please rebase on master and re-open.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#161 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ADHPdyw93kK4HoBtRDD76Hms-NwAIiVXks5vf1nogaJpZM4JZnC7>.
|
you have comments given to you on this PR. please respond to them. you do not need to open a new PR, rebase this one on top of the master branch: $ git fetch
$ git rebase origin/master since there are conflicts you'll probably have to tidy them up (https://help.github.com/en/articles/resolving-merge-conflicts-after-a-git-rebase) then you'll need to force push
In addition to rebasing there were particular requests, so please respond to them, commit and re-push your changes and they will be reflected in this PR. Sorry for the delay in response. |
Base modification support has been added to VCFv4.5. There is a public consultation period before the specifications are finalised so please review the proposed design and highlight any issues/limitations in the proposed model as the encoding is quite different to the one proposed in this PR. |
Added a description of new fields that allow the storing of DNA methylation information to be stored with DNA sequencing variation in a VCF file. This is important to allow the results of bisulfite sequencing experiments which generate both sequence variation and DNA methylation information to be stored in a standard format.
These proposed changes were originally submitted as a new format, and are now being resubmitted as a modification of the VCFv4.3 standard following a suggestion from the original pull request.