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

Host documentation on readthedocs #32

Closed
2 of 3 tasks
lukehsiao opened this issue Mar 29, 2018 · 1 comment
Closed
2 of 3 tasks

Host documentation on readthedocs #32

lukehsiao opened this issue Mar 29, 2018 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@lukehsiao
Copy link
Contributor

lukehsiao commented Mar 29, 2018

There's a few parts to this task.

  • Setup readthedocs so that documentation is auto built
  • Fix the import errors with readthedocs
  • Go through the code and improve docstrings throughout (this will be ongoing...)

First part is easy, second part will take time.

@lukehsiao lukehsiao added the enhancement New feature or request label Mar 29, 2018
@lukehsiao lukehsiao self-assigned this Mar 29, 2018
@lukehsiao lukehsiao added this to the v0.1.5 milestone Mar 29, 2018
@lukehsiao lukehsiao modified the milestones: v0.1.5, v0.2.0 Mar 30, 2018
@lukehsiao
Copy link
Contributor Author

After mocking the relevant imports, we have:

reading sources... [100%] user/lf_helpers                                                            
WARNING: autodoc: failed to import module 'fonduer.lf_helpers'; the following exception was raised:
Traceback (most recent call last):
  File "/home/lwhsiao/repos/fonduer/.venv/lib/python3.5/site-packages/sphinx/ext/autodoc/importer.py", line 140, in import_module
    __import__(modname)
  File "/home/lwhsiao/repos/fonduer/fonduer/__init__.py", line 10, in <module>
    from fonduer.async_annotations import BatchFeatureAnnotator
  File "/home/lwhsiao/repos/fonduer/fonduer/async_annotations.py", line 15, in <module>
    from fonduer.features.features import get_all_feats
  File "/home/lwhsiao/repos/fonduer/fonduer/features/features.py", line 1, in <module>
    from fonduer.features.content_features import *
  File "/home/lwhsiao/repos/fonduer/fonduer/features/content_features.py", line 7, in <module>
    from fonduer.lf_helpers import get_left_ngrams, get_right_ngrams, tokens_to_ngrams
  File "/home/lwhsiao/repos/fonduer/fonduer/lf_helpers.py", line 12, in <module>
    from fonduer.models import Phrase
  File "/home/lwhsiao/repos/fonduer/fonduer/models/__init__.py", line 1, in <module>
    from fonduer.snorkel.models.candidate import candidate_subclass
  File "/home/lwhsiao/repos/fonduer/fonduer/snorkel/__init__.py", line 3, in <module>
    from .models import Meta
  File "/home/lwhsiao/repos/fonduer/fonduer/snorkel/models/__init__.py", line 32, in <module>
    from .context import Context, Document, Sentence, TemporarySpan, Span
  File "/home/lwhsiao/repos/fonduer/fonduer/snorkel/models/context.py", line 304, in <module>
    class Span(Context, TemporarySpan):
TypeError: metaclass conflict: the metaclass of a derived class must be a (non-strict) subclass of the metaclasses of all its bases

@lukehsiao lukehsiao removed this from the v0.2.0 milestone Mar 31, 2018
@senwu senwu closed this as completed in #42 Apr 3, 2018
stackoverflowed pushed a commit to stackoverflowed/multimodal that referenced this issue Dec 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant