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

Arbres reels #50

Merged
merged 8 commits into from
Jun 11, 2021
Merged

Arbres reels #50

merged 8 commits into from
Jun 11, 2021

Conversation

Jean-Baptiste-Camps
Copy link
Member

4 «arbres réels» of printed editions, extracted from the paper by Trovato and Guidi 2004.

@Jean-Baptiste-Camps
Copy link
Member Author

@GusRiva and @gabays , I'll be glad to have you validation on these ones.

@GusRiva GusRiva self-assigned this Feb 10, 2021
Copy link
Contributor

@GusRiva GusRiva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added a node that was missing in Arcadia.

To23->To30;
To16->Vr17;
Si17->Fi14[style="dashed"];
Pa23->To23[style="dashed"]; //small doubt as to the exact target of this one.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's hard to tell, but probably it means that the contamination is to some (hypothetical?) ancestor of To23? It's strange in the case of these trees. Maybe it will be neccesary to take a look at the article or the edition and see if that is mentioned.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, we here too have arrows going back in time. What is this ???

data/ita/Trovato_2004_Sannazaro-Arcadia/stemma.gv Outdated Show resolved Hide resolved
@Jean-Baptiste-Camps
Copy link
Member Author

@GusRiva : I have opened an issue on this particular problem in #65 . If that's fine with you, I suggest merging this and correcting the issue later on !

Copy link
Contributor

@GusRiva GusRiva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As we discuss, we merge and deal with any open issues later. Who knows, we might discover the secrets of time travel...

@GusRiva GusRiva merged commit 6ef3625 into main Jun 11, 2021
@Jean-Baptiste-Camps Jean-Baptiste-Camps deleted the arbresReels branch June 11, 2021 21:44
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.

2 participants