wikipathways / wikipathways-help

1 stars 1 forks source link

I don't find my pathways on the new site #15

Closed docmattc closed 1 year ago

docmattc commented 1 year ago

Pathways I've created (eg WP5277, WP5276, WP5279, WP5280, WP5324) don't seem to be on the new site

egonw commented 1 year ago

I was about to write: "@docmattc, that is likely because they have not been approved."

But https://classic.wikipathways.org/instance/WP5277 has been approved a long time ago... this URL can be used as workaround.

danidi commented 1 year ago

WP5779 doesn't exist on the old page either, is there a typo with the number? WP5324 is still under construction, so here it is not showing because it has not been approved. For the rest, could the order of the approved tag matter? They all have the lipid maps tag before the approved tag.

docmattc commented 1 year ago

WP5779 -> WP5279

egonw commented 1 year ago

Checking the website database:

cc @AlexanderPico @mkutmon

danidi commented 1 year ago

Ok, WP5279 has the approved tag first, so that can't be the issue.

fehrhart commented 1 year ago

Same here, I am missing about 40 pathways from the rare disease community. The new page has 83, the old rare disease community tag was for about 120 pathway. Example of missing pathways: https://classic.wikipathways.org/index.php/Pathway:WP5223 https://classic.wikipathways.org/index.php/Pathway:WP5224 https://classic.wikipathways.org/index.php/Pathway:WP5221 https://classic.wikipathways.org/index.php/Pathway:WP5222

All approved :)

larsgw commented 1 year ago

The daily GPML sync only seems to act on pathways that have been changed in the last 24 hours:

WSPathwayInfo [] changedPwy = client.getRecentChanges(new SimpleDateFormat("yyyyMMddhhmmss").parse(args[0]));

https://github.com/wikipathways/gpml-sync-action/blob/b514398629f9a429762a2038b84d3004de3f8b6b/src/org/wikipathways/SyncAction.java#L30

If these pathways were approved after the last manual sync (I'm having trouble finding when that was) they would not be picked up by the automatic sync.

Example of missing pathways: https://classic.wikipathways.org/index.php/Pathway:WP5223 https://classic.wikipathways.org/index.php/Pathway:WP5224 https://classic.wikipathways.org/index.php/Pathway:WP5221 https://classic.wikipathways.org/index.php/Pathway:WP5222

These do seem to be available now: https://www.wikipathways.org/pathways/WP5221.html, https://www.wikipathways.org/pathways/WP5222.html, etc.

danidi commented 1 year ago

I think there are two different issues here: the rare disease pathways are missing from the list here: https://www.wikipathways.org/communities/rarediseases.html. While https://classic.wikipathways.org/index.php/Pathway:WP5224 has the rare disease tag, the corresponding pathway from the new site https://www.wikipathways.org/pathways/WP5224.html does not have any communities connected. Some of the lipid maps pathways are however completely missing.

egonw commented 1 year ago

the rare disease pathways are missing from the list here

Pathways have to be added by hand to communities. I don't think we automated that yet. @danidi, that's a separate issue. This can be done as explained on this page: https://www.wikipathways.org/help_communities.html

But tl;dr, edits the txt files in this folder: https://github.com/wikipathways/wikipathways-database/tree/main/communities

egonw commented 1 year ago

The daily GPML sync only seems to act on pathways that have been changed in the last 24 hours

Okay, then we just need to add them by hand.

AlexanderPico commented 1 year ago

That's correct: the sync action only checks for recent changes or new pathways in past 24 hours.

@fehrhart If you have a long list of WPIDs that should be added, I can run the local script easily and do them as a batch. Just provide me with the list of WPIDs.

AlexanderPico commented 1 year ago

Ok. I updated the SyncAction.jar to also sync any approved pathway not previously processed, regardless of it having been recently changed. So, this will now catch case B above.

I just ran it now and it added:

@docmattc That added 5 more new pathway to your list :)

If anyone can find additional approved WPIDs that are not on the new site, please reopen and list examples.

Thanks for catching this!