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 Economic union in the debates on the creation of the euro: new evidence from the tapes of the Delors Committee meetings #112

Closed
eliselavy opened this issue Apr 6, 2023 · 7 comments
Assignees

Comments

@eliselavy
Copy link
Collaborator

eliselavy commented Apr 6, 2023

https://github.com/jdh-observer/4ZDXFXQwhoA8

@eliselavy eliselavy self-assigned this Apr 6, 2023
@eliselavy
Copy link
Collaborator Author

eliselavy commented Apr 6, 2023

  • no mention of libraries's version requirements.txt , need to be generated
  • no test made for mybinder
  • no change made in the template for the cover image, still placeholder
  • NO LABEL defined for the table
  • Design problem: url mentioned in the table of the figure
  • Wrong tagging for the figures (from figure 9 to figure 19) and no figure-1
  • Wrong tagging image

Screenshot 2023-04-06 at 14 35 46

Screenshot 2023-04-06 at 14 37 06

  • Many ressources mentioned but not called (see output files ) but code for performing OCR on ECB textual documents NOT TESTED in the review
  • Error running nbconvert due to missing NLP model , addition of the GitHub URL of the model to the requirements.txt
  • Error running nbconvert problem version spacy with python version
nbclient.exceptions.CellExecutionError: An error occurred while executing the following cell:
    200     # language subclass) while keeping top-level language identifier "lang"
    201     lang = meta.get("lang_factory", meta["lang"])
--> 202     cls = get_lang_class(lang)
    203     nlp = cls(meta=meta, **overrides)
    204     pipeline = meta.get("pipeline", [])

     73     # Check if language is registered / entry point is available
---> 74     if lang in registry.languages:
     75         return registry.languages.get(lang)

@eliselavy
Copy link
Collaborator Author

Solve by adding : catalogue==2.0.8
catalogue package is a dependency of spacy, and is responsible for managing the available language models and pipeline components.
this dependancy was not added by author and not found by using https://pypi.org/project/pipreqs/

@eliselavy
Copy link
Collaborator Author

Need to check error after nbconvert
Screenshot 2023-04-06 at 14 39 55

eliselavy added a commit to jdh-observer/4ZDXFXQwhoA8 that referenced this issue Apr 7, 2023
eliselavy added a commit to jdh-observer/4ZDXFXQwhoA8 that referenced this issue Apr 7, 2023
eliselavy added a commit to jdh-observer/4ZDXFXQwhoA8 that referenced this issue Apr 7, 2023
@eliselavy
Copy link
Collaborator Author

eliselavy commented Apr 7, 2023

Mybinder
Screenshot 2023-04-07 at 09 29 48
Need to do a docker file: https://hub.docker.com/r/ibbd/spacy/dockerfile

@eliselavy
Copy link
Collaborator Author

Docker file didn't solve the issue:
2023-04-07T13:42:41Z [Normal] Created container notebook
2023-04-07T13:42:41Z [Normal] Started container notebook
Spawn failed: Timeout
Launch attempt 3 failed, retrying...
Server requested
2023-04-07T13:47:53.168437Z [Normal] Successfully assigned prod/jupyter-jdh-2dobserver-2d4zdxfxqwhoa8-2d7fv5frr2 to gke-prod-user-202201-0769dbf8-fd9h
2023-04-07T13:47:54Z [Normal] Container image "jupyterhub/mybinder.org-tc-init:2020.12.4-0.dev.git.4289.h140cef52" already present on machine
2023-04-07T13:47:54Z [Normal] Created container tc-init
2023-04-07T13:47:54Z [Normal] Started container tc-init
2023-04-07T13:47:55Z [Normal] Pulling image "gcr.io/binderhub-288415/r2d-staging-g5b5b759-jdh-2dobserver-2d4zdxfxqwhoa8-4eb3aa:8eab35776044dfc2728a40e1aed4e452c4446f7b"
2023-04-07T13:48:49Z [Normal] Successfully pulled image "gcr.io/binderhub-288415/r2d-staging-g5b5b759-jdh-2dobserver-2d4zdxfxqwhoa8-4eb3aa:8eab35776044dfc2728a40e1aed4e452c4446f7b" in 54.794581341s
2023-04-07T13:48:50Z [Normal] Created container notebook
2023-04-07T13:48:50Z [Normal] Started container notebook
Spawn failed: Server at http://10.0.20.141:8888/user/jdh-observer-4zdxfxqwhoa8-7fv5frr2/ didn't respond in 30 seconds
Spawn failed: Server at http://10.0.20.141:8888/user/jdh-observer-4zdxfxqwhoa8-7fv5frr2/ didn't respond in 30 seconds

eliselavy added a commit to jdh-observer/4ZDXFXQwhoA8 that referenced this issue May 15, 2023
eliselavy added a commit to jdh-observer/4ZDXFXQwhoA8 that referenced this issue May 15, 2023
eliselavy added a commit to jdh-observer/4ZDXFXQwhoA8 that referenced this issue May 15, 2023
@eliselavy
Copy link
Collaborator Author

Related to #117

@eliselavy
Copy link
Collaborator Author

Send to review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant