Difference between revisions of "User talk:GaryGriffin"

From Gramps
Jump to: navigation, search
(New DNA gramplet feedback)
(Refresh documentation on DNAgramplet)
(Tag: Replaced)
Line 1: Line 1:
'''Welcome to ''Gramps''!'''
We hope you will contribute much and well.
You will probably want to read the [https://www.mediawiki.org/wiki/Special:MyLanguage/Help:Contents help pages].
Again, welcome and have fun! [[User:Nickw|Nickw]] ([[User talk:Nickw|talk]]) 18:24, 28 November 2016 (MST)
=DNA Gramplet=
=DNA Gramplet=
I apologize for jumping the gun on moving the DNA Gramplet documentation I did not [https://github.com/gramps-project/addons-source/pull/408#issuecomment-682188789 realize] the documentation was referring to an unreleased addon not available to anyone but yourself, which explains my confusion when I was attempting to update based on the existing version and seeing different results than your screenshots show :( Will you be sharing it for testing myself & possibly [[User:Matt]] are interested? --[[User:Gioto|Gioto]] ([[User talk:Gioto|talk]]) 22:21, 20 September 2020 (UTC)
:[[User:Matt|Matt]] [[User:Gioto|Gioto]]
:Added access to test version at [https://github.com/GaryGriffin/addons-source/tree/DNA-gramplet-overhaul DNA]. I will be pushing this once another PR has been accepted ([https://gramps-project.org/wiki/index.php/Addon:SyncAssociation Addon:SyncAssociation]is loosely connected to the DNA gramplet) [[User:GaryGriffin|GaryGriffin]] ([[User talk:GaryGriffin|talk]]) 17:25, 20 Sep 2020 (PST)
::Thanks, I'll try to check-it out in a few days. --[[User:Gioto|Gioto]] ([[User talk:Gioto|talk]]) 00:03, 26 September 2020 (UTC)
==dna feedback from matt==
[[User:GaryGriffin|GaryGriffin]]: Thank you for sharing. Testing seems to work fine. Some notes and ideas:
# The loading speed might become a problem. For 8 associations it already takes ~3 seconds to load.
# Overlapping segments are hidden below others and currently unaccessible. A way to draw overlapping segments in seperate rows might be an interesting feature in future.
[[User:Matt|Matt]] ([[User talk:Matt|talk]]) 10:47, 25 September 2020 (UTC)
:[[User:Matt|Matt]]: Hmmm, I have 18 Associations for one person and it is less than a second to draw for me. I am on a 2013 Mac, so not the fastest system. This may be more of a function of PyCairo implementation on the platform. Can you provide more info on platform and range of overlaps.
:# Are most of your segments tied to a maternal/paternal side or unknown (which can definitely get busy)? Almost all of mine are M/P known. The drawing format is the standard one (for each segment an M and a P section). I fear I would add more confusion if I diverted from that. See [https://DNApainter.com DNApainter] as an example.
:# I draw in order of Association. I cannot think of how I would draw in an order that guaranteed the largest was always behind all smaller overlaps. I have only 1 totally obscured overlap, so hadn't thought about this problem. I only realized I had this when I made the segments more transparent (see next item).
:# Maybe if all were less opaque, it would be an easier view? Change line 330 of dna.py to the following. This will change the opacity to 0.5 for M/P and 0.33 for U. The one I posted was 1 for M/P and 0.5 for U. Would appreciate your feedback on this change.
  cr.set_source_rgba(rgb_color[0], rgb_color[1], rgb_color[2], 1/(chr_mult+1))
:[[User talk:GaryGriffin|talk]]) 17:40, 25 Sep 2020 (PST)
::[[User:GaryGriffin|GaryGriffin]]: I'm on Windows 10 using GrampsAIO64-5.1.3-1 and Pycairo 1.16.1. All segments are tied to a maternal/paternal side since I only added the known ones. I do have some segemnts overlapping since many of them are from the same common ancestor couple. Yes drawing one maternal and paternal row for each chromosome is standard, but the DNA Painter also has an option to expand and collapse these rows, so you can see all overlapping segments. (see: https://dnapainter.com/example, click on the chromosome number or the plus/minus icon)
::[[User:Matt|Matt]] ([[User talk:Matt|talk]]) 08:09, 26 September 2020 (UTC)
<gallery mode="packed-overlay" widths=300px heights=300px>
File:DNA-gramplet.PNG|DNA Gramplet
File:DNA-gramplet2.PNG|DNA Gramplet transparent
File:DNA Painter example.png|DNA Painter with expanded Chromosome 1
:[[User:Matt|Matt]]:  So you don't see value in changing the transparency, right? Then I will leave as is and make a note in the docs. I was not aware of the expansion in DNApainter. Thanks for pointing that out. Let me ponder implementing that. I am concerned that it will become very busy. Especially if I do all open or all small (using an overall PLUS rather than a per chromosome PLUS). What are your thoughts on individual plus vs just an overall plus?
:[[User talk:GaryGriffin|talk]]) 17:11, 29 Sep 2020 (PST)
::[[User:GaryGriffin|GaryGriffin]]: No, I don't think that transparency is a solution. Unfortunately I don't know what would be the best way to implement it. I'm afraid you can only try it that way and then try other solutions if it isn't working well. Maybe it would be more efficent to try to create it as a view or report (which calulates it once for <s>two</s> the selected persons) instead of a gramplet (which calculates it every time you change a person or switch to another active person)?
::[[User:Matt|Matt]] ([[User talk:Matt|talk]]) 14:47, 2 October 2020 (UTC)
:::[[User:Matt|Matt]]: If you wanted just the view of one other person, you can run SyncAssociation to copy the Note from the first person to the associated person (if it is not already there) and then switch to the associated person and run the DNA gramplet. That would (probably) only have one pair of people in the view. I assumed that the value of this gramplet was when there were many associated people to look at the big picture. I thought that there would be only a small number (often just one) of people with very long Associated lists. Looking at the reciprocal view would be a way to reveal overlaps. But I didnt think that would actually be interesting normally.
:::[[User talk:GaryGriffin|talk]]) 09:42, 04 Oct 2020 (PST)
::::[[User:GaryGriffin|GaryGriffin]]: Sorry that was a typo, I meant "which calulates it once for <s>two</s> the selected persons", so still all matches drawn for users to look at the big picture. :)
:::: [[User:Matt|Matt]] ([[User talk:Matt|talk]]) 17:18, 4 October 2020 (UTC)
=New DNA gramplet feedback =
Hi Garry, Thanks for the updated version = '1.0.4', gramplet, some feedback:
* You mention that in the Chromosome segments the top is P and bottom is M and U is unknown; would it be possible to add this information to the tooltip popup along with the persons Gramps ID the screenshot below would then show {{man label|Garner, Maude [I0651] 11.2cMs [Maternal side]}}
[[File:Maternal dna segment tootltip .png]]
* Even better if a possible guess at the relationship range the cMs represent was also added eg: {{man label|Garner, Maude [I0651] 11.2cMs [Maternal side: 5th to 8th Cousin]}} or if the relationship already exist display that instead?
* One more if you could click on the Gramps ID eg:[I0651] to take you that person it would be great.
--[[User:Gioto|Gioto]] ([[User talk:Gioto|talk]]) 04:45, 15 October 2020 (UTC)
:[[User:Gioto|Gioto]] Some of the feedback includes adding the Segment Length in the tooltip as well as your suggestion to add the ID and relationship. And potentially dropping the M/F/U flag in the input. I do not want to add a guess at the relationship as that is way too much info (it is actually a distribution of possible relationships if you use the Shared cM Project approach). Do you see any issues with me breaking compatibility from the experimental gramplet input format (removing M/F flag and adding segment length)?
--[[User:GaryGriffin|GaryGriffin]] ([[User talk:GaryGriffin|talk]]) 12;15, 15 Oct 2020 (PST)
I will be updating the PR with a new version in a few days. I am updating the documentation now so that you can see the changes per your good suggestions. If you see issues based on the doc, I will try to address before I submit the new PR.
::No one is really using the Gramplet so now is the time to change it how you see fit, thank you for improving it. Yes remove the  M/F flag and add the more useful segment length. --[[User:Gioto|Gioto]] ([[User talk:Gioto|talk]]) 22:52, 16 October 2020 (UTC)
[[User:GaryGriffin|GaryGriffin]] ([[User talk:GaryGriffin|talk]]) 16:12, 17 Oct 2020 (PST)

Revision as of 23:50, 17 October 2020

DNA Gramplet

Matt Gioto

I will be updating the PR with a new version in a few days. I am updating the documentation now so that you can see the changes per your good suggestions. If you see issues based on the doc, I will try to address before I submit the new PR.

GaryGriffin (talk) 16:12, 17 Oct 2020 (PST)