Skip to content

Update crosswalks image#119

Open
meldra wants to merge 1 commit intocodemeta:masterfrom
meldra:diagramsvg
Open

Update crosswalks image#119
meldra wants to merge 1 commit intocodemeta:masterfrom
meldra:diagramsvg

Conversation

@meldra
Copy link
Copy Markdown
Contributor

@meldra meldra commented Mar 25, 2026

This is the .svg adapted from @lindangulopez codepen prototype adjusted according to discussion with @progval

Replaces pr #118

Co-authored-by: Linda Angulo Lopez <lindangulopez@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@lindangulopez lindangulopez left a comment

Choose a reason for hiding this comment

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

@meldra can you keep PR: #118 - the CodeMeta Team agreed on this version.

Copy link
Copy Markdown
Contributor

@lindangulopez lindangulopez left a comment

Choose a reason for hiding this comment

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

[https://github.com//pull/118]

Image please keep this .svg

Copy link
Copy Markdown
Contributor

@lindangulopez lindangulopez left a comment

Choose a reason for hiding this comment

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

@progval @meldra can you plese revert to PR#118 : this was the agreeded diagram - it is in svg.
[[https://github.com//pull/118](https://github.com//pull/118%5C)]

Image

@progval
Copy link
Copy Markdown
Member

progval commented Mar 27, 2026

@progval
Copy link
Copy Markdown
Member

progval commented Mar 27, 2026

@lindangulopez I find the one in this PR to be more readable, because it shows links between terms instead of only relying on color. It's also more correct to have "(exact_match)" be a property of the relation than have it par of the node label, because it's not a property of the node.

@moranegg
Copy link
Copy Markdown
Member

@progval The other image was validated in a CodeMeta meeting and is a community-agreed version.

There are two options:

  1. to take the time to re-discuss the diagram in a community meeting
  2. push into the user documentation the validated crosswalk image

I'm inclined to stay as is and not put more time into it.

@douardda
Copy link
Copy Markdown

I agree the proposal from this PR is far more readable and easier to understand than the alternatives.

Also, I don't get why this needs to be discussed in a community meeting. This is not a change of the spec, nor any technical modification of any crosswalk or so. A simple validation from MR's review process seems adequate here.

(but that's just my opinion)

@moranegg
Copy link
Copy Markdown
Member

CodeMeta is a community-led standard with regular calls to discuss how to improve the metadata vocabulary and usability.

This diagram was discussed in a call.
This is why I'm proposing option 1 to discuss the new option before merging anything here.

The effort put into keeping this community driven is important.

I hope this clarifies my comment.

@meldra
Copy link
Copy Markdown
Contributor Author

meldra commented Mar 27, 2026

Can you clarify the reasons the team chose to label the relations this way, for those of us who were not on the community calls?

@lindangulopez
Copy link
Copy Markdown
Contributor

Ah, I will redo a push from the master branch then @progval

@moranegg
Copy link
Copy Markdown
Member

moranegg commented Mar 27, 2026

@meldra They didn't choose; they only saw the previous version.
This is why option 1 states, "to take the time to re-discuss the diagram in a community meeting."

It can also happen in this PR, but for now, only SWH team members are involved in this discussion here.

Both diagrams are an improvement over no diagram.

@meldra
Copy link
Copy Markdown
Contributor Author

meldra commented Mar 27, 2026

These PRs do replace an existing diagram though. This goes on the crosswalks page replacing a diagram you, Linda, and I had already discussed on a call last year and was merged.

My understanding was that this new image process happened due to pr #99. In that PR we rejected and I retracted a confused attempt at a diagram for the user-guide page in favor of Linda crowdsourcing a new one. Is that one also in the works?

@lindangulopez
Copy link
Copy Markdown
Contributor

Hi @progval, @meldra @moranegg & @douardda I will discuss this in our next meeting, on Tuesday. and let you know which image we should keep. Kind regards, Linda.

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.

5 participants