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.

Sign up for GitHub

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

Jump to bottom

JSON-LD Bulk importer raises TypeError when reading a file #11470

Closed
jacobtylerwalls opened this issue Sep 19, 2024 · 0 comments · Fixed by #11471
Closed

JSON-LD Bulk importer raises TypeError when reading a file #11470

jacobtylerwalls opened this issue Sep 19, 2024 · 0 comments · Fixed by #11471
Assignees
@jacobtylerwalls
Labels
Subject: ETL Modules Type: Bug Something isn't working

Comments

@jacobtylerwalls
Copy link
Member

jacobtylerwalls commented Sep 19, 2024 •
edited
Loading

JSON-LD importer no longer can read files.

Regression in 8532cc6

  File "/Users/jwalls/prj/arches/arches/app/etl_modules/jsonld_importer.py", line 81, in read
    self.prepare_temp_dir(request)
  File "/Users/jwalls/prj/arches/arches/app/etl_modules/base_import_module.py", line 237, in prepare_temp_dir
    self.temp_dir = os.path.join(settings.UPLOADED_FILES_DIR, "tmp", self.loadid)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen posixpath>", line 90, in join
  File "<frozen genericpath>", line 164, in _check_arg_types
TypeError: join() argument must be str, bytes, or os.PathLike object, not 'NoneType'
@jacobtylerwalls jacobtylerwalls self-assigned this Sep 19, 2024
@chiatt chiatt added this to pipeline Sep 19, 2024
jacobtylerwalls added a commit that referenced this issue Sep 19, 2024
@jacobtylerwalls
Fix failure to launch JSON-LD bulk importer #11470
526c72b
@jacobtylerwalls jacobtylerwalls mentioned this issue Sep 19, 2024
Merged
10 tasks
@jacobtylerwalls jacobtylerwalls moved this to 👀 In Review in pipeline Sep 19, 2024
@jacobtylerwalls jacobtylerwalls linked a pull request Sep 19, 2024 that will close this issue
Fix failure to launch JSON-LD bulk importer #11470 #11471
Merged
10 tasks
njkim added a commit that referenced this issue Sep 20, 2024
@njkim
Merge pull request #11471 from archesproject/jtw/fix-json-ld-launch-f…
d67c669 
…ailure

Fix failure to launch JSON-LD bulk importer #11470
@github-project-automation github-project-automation bot moved this from 👀 In Review to ✅ Done in pipeline Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jacobtylerwalls jacobtylerwalls

Labels
Subject: ETL Modules Type: Bug Something isn't working
Projects
pipeline
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix failure to launch JSON-LD bulk importer #11470 archesproject/arches
1 participant
@jacobtylerwalls