Skip to content
This repository has been archived by the owner on Dec 16, 2022. It is now read-only.

Reduce the number of imports needed for a basic program #1797

Closed
schmmd opened this issue Sep 20, 2018 · 2 comments
Closed

Reduce the number of imports needed for a basic program #1797

schmmd opened this issue Sep 20, 2018 · 2 comments

Comments

@schmmd
Copy link
Member

schmmd commented Sep 20, 2018

Here's some user feedback about the tutorial that applies to AllenNLP in general:

  • the huge imports section where many components are important from many different packages can be a big "turn off" for adoption, imo, as it requires remembering many different things (which modules to import and where to import them from) just to get a very basic program running. I think adding a meta-package (i.e. allennlp.all) which you can import everything from will make things much more user friendly also for quick-and-dirty stuff.
@DeNeutoy
Copy link
Contributor

We can simplify this considerably by doing from allennlp import data and then using data.Field in the tutorial. This should reduce the initial cognitive load.

@schmmd
Copy link
Member Author

schmmd commented Sep 28, 2018

We're going to wait and see if we get additional similar feedback before we take action here

@schmmd schmmd closed this as completed Sep 28, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants