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

Technical review A Digital Toolbox for Studying the LGBTQ World, 1965-1990 #105

Closed
eliselavy opened this issue Mar 22, 2023 · 9 comments
Closed
Assignees

Comments

@eliselavy
Copy link
Collaborator

eliselavy commented Mar 22, 2023

Technical review A Digital Toolbox for Studying the LGBTQ World, 1965-1990
https://github.com/jdh-observer/X3MGSKqAycaT

see email for question

eliselavy added a commit to jdh-observer/X3MGSKqAycaT that referenced this issue Mar 22, 2023
@eliselavy
Copy link
Collaborator Author

Adaptation needs to be made on the frontend
C2DH/journal-of-digital-history#533

@eliselavy eliselavy self-assigned this Mar 22, 2023
eliselavy added a commit to jdh-observer/X3MGSKqAycaT that referenced this issue Mar 22, 2023
@eliselavy
Copy link
Collaborator Author

eliselavy commented Mar 22, 2023

  • Anonymisation
  • Need to tag the figure / table

=> you can resynchronise from here: https://github.com/jdh-observer/X3MGSKqAycaT/blob/main/article.ipynb

install.packages('IRkernel')  # Don’t forget step 2/3!
IRkernel::installspec()

If you use Docker and for mybinder FROM rocker/binder:4.2.0 needs to be use

@regan008
Copy link

Hi @eliselavy! When you say "=> you can resynchronise from here: https://github.com/jdh-observer/X3MGSKqAycaT/blob/main/article.ipynb" do you mean I need to bring my version of the notebook up to date with the version linked here? I'm a bit unclear on what exactly I need to do.

I'm also a bit unclear on what you mean by "If you use Docker and for mybinder FROM rocker/binder:4.2.0 needs to be use" -- I do have docker installed. Where would I find rocker/binder:4.2.0 and how would I install that? I may be misunderstanding the direction here.

@regan008
Copy link

And for the leaflet map - I can't seem to get the map to display like you have it above. I've installed those R packages you mention but the code in the screenshot doesn't work for me. I get no output when I run that cell. Is the adaption that needs to happen on my end or yours? I'm a bit unsure about next steps here.

@eliselavy
Copy link
Collaborator Author

eliselavy commented Mar 23, 2023

Hi @eliselavy! When you say "=> you can resynchronise from here: https://github.com/jdh-observer/X3MGSKqAycaT/blob/main/article.ipynb" do you mean I need to bring my version of the notebook up to date with the version linked here? I'm a bit unclear on what exactly I need to do.

I'm also a bit unclear on what you mean by "If you use Docker and for mybinder FROM rocker/binder:4.2.0 needs to be use" -- I do have docker installed. Where would I find rocker/binder:4.2.0 and how would I install that? I may be misunderstanding the direction here.

@regan008 In fact only tag for figures have been added and the output genererated, you can add the tag by yourself with the name you want as it specified in the documentation.

For mybinder i will do the technical check if you want - i see you specify a version to use for R that is not compatible with the leaflet library

@eliselavy
Copy link
Collaborator Author

And for the leaflet map - I can't seem to get the map to display like you have it above. I've installed those R packages you mention but the code in the screenshot doesn't work for me. I get no output when I run that cell. Is the adaption that needs to happen on my end or yours? I'm a bit unsure about next steps here.

@regan008
Our frontend (it means the JDH interface) needs to be adapted as i wrote you but at least the map needs to be generated in your notebook.
The map is not generated in your notebook, isn'it? Do you have an error? Check the version you use by executing the following code in a cell code: R.version and share it

@regan008
Copy link

@eliselavy Yeah for whatever reason, I'm having trouble getting that map to show up in my notebook. Most of the time I just get nothing in return. I'm running R version 4.0.4 (2021-02-15) in the docker setup recommended by the journal. Any ideas?

@eliselavy
Copy link
Collaborator Author

eliselavy commented Mar 28, 2023

@regan008

You should at least have an error message:

If the package is not installed:

Error in library(leaflet): there is no package called ‘leaflet’
Traceback:
1. library(leaflet)

Check where the package is installed, run the line below in a notebook code cell
system.file(package='leaflet')

When I run via Docker:
Screenshot 2023-03-28 at 17 14 27

When I run locally:
Screenshot 2023-03-28 at 17 19 59

You mentioned version R version 4.0.4 (2021-02-15) in the docker setup but the version in the file is 4.2.0, see documentation here: https://github.com/C2DH/journal-of-digital-history-jupyter-stack#for-rs-user
Maybe we can a quick talk via webex ?, please contact me by email, free tomorrow except 2pm-3pm Luxembourg time

@eliselavy
Copy link
Collaborator Author

appointment fixed for today 3pm

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

No branches or pull requests

2 participants