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

Code review of lib_metadata_db codebase #50

Open
kallewesterling opened this issue Sep 21, 2022 · 5 comments
Open

Code review of lib_metadata_db codebase #50

kallewesterling opened this issue Sep 21, 2022 · 5 comments
Assignees

Comments

@kallewesterling
Copy link
Collaborator

kallewesterling commented Sep 21, 2022

(This issue was originally intended as a submission to the DH Code process for code review)

@kallewesterling kallewesterling self-assigned this Sep 21, 2022
@griff-rees
Copy link
Collaborator

griff-rees commented Dec 6, 2022

A starting point (of potential tickets...).

  • Not sure how the django_initialiser module is used. I can see the value of its __init__.py in some contexts but wondering if those are still applicable.
  • Potential refactoring folder structure
  • GeoDjango should be available since 699a063. Does it make sense to add a PointField to Place in Gazetteer and a MultiPolygonField to Census Record

@griff-rees
Copy link
Collaborator

Might be worth having a peak at this @kallewesterling: maybe helpful agenda for a meeting. A significant refactor is done.

@kallewesterling
Copy link
Collaborator Author

Sorry @griff-rees - I had missed your comment here.

Re: this

Not sure how the django_initialiser module is used. I can see the value of its init.py in some contexts but wondering if those are still applicable.

It is needed to load Django inside of a Jupyter notebook. We used it this summer to make sure that django is correctly set up, etc. in the top line of the notebook:

import django_initialiser

Ideally, we'd have instructions for how to do this more explicitly for folks who want to work with Jupyter Notebooks/Lab + the metadata db eventually... It was just a shortcut so I wouldn't have to spend time on it this past summer.

@griff-rees
Copy link
Collaborator

Thanks @kallewesterling and sorry for not replying sooner. I think this form is what's necessary for this: https://docs.google.com/forms/d/e/1FAIpQLSeFLFSS2NEttNimGo-lZmDiSjWzcoefV8GojEZGHPd-YVtBkQ

@griff-rees
Copy link
Collaborator

I've created ticket #171 to address the first element on my list in the comment above

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