C2DH / jdh-notebook

A collection of Jupyter notebooks for the Journal of Digital History
https://journalofdigitalhistory.org
GNU Affero General Public License v3.0
4 stars 1 forks source link

Remerge last revision for A Digital Toolbox for Studying the LGBTQ World, 1965-1990 #109

Closed eliselavy closed 1 year ago

eliselavy commented 1 year ago

https://github.com/jdh-observer/X3MGSKqAycaT

eliselavy commented 1 year ago
eliselavy commented 1 year ago

But problem dataframe empty in the last version provided see: https://app.reviewnb.com/jdh-observer/X3MGSKqAycaT/pull/1/

Screenshot 2023-04-07 at 18 09 39

@regan008 is the code should be commented as before?

regan008 commented 1 year ago

@eliselavy Ah, there was an issue with the RDS dataset. I've fixed it and pushed the revisions. Now the code should run as expected.

eliselavy commented 1 year ago

@regan008 indeed the dataframe is no more empty but another problem:

regan008 commented 1 year ago

@eliselavy I think I fixed the data frame and pushed a revised version. Can you take another look?

eliselavy commented 1 year ago

@regan008 ok perfect it works locally / mybinder ok Your article can be send to the peer review. Don't forget to resynchronise with us, as some changes in metadata and tagging have been made. (cf label updated - metadata updated - tag the map, undefined reference) This issue goes to be implemented on our side https://github.com/C2DH/journal-of-digital-history/issues/533