-
Notifications
You must be signed in to change notification settings - Fork 15
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
Eliminate Need for User-Facing Schema
#94
Comments
I agree we don't need to introduce Schema in demos, the current new syntax doesn't need them. Question: if Schema is not user-facing anymore, what kind of demo you're expecting? Further Improvement OptionsInstead of Schema, why not use protobuf https://protobuf.dev/getting-started/pythontutorial/ which is a common way to serve for data transformation?
Why do I propose Protobuf?
|
I refactored demos as the followup for updating syntax for #84, this issue will need more work. I'm unassigning myself for now. |
Hi @chjuncn, thanks for taking the lead on this issue and refactoring the demos to not rely on the I think the answer to the two points/questions you raised above boil down to one idea: we want to make the experience for new users as simple as possible. Given this goal, I hope my answers below will make sense.
Even if we don't have
Similar to point 2. above, for new users I think we primarily want to showcase the ability to use simple Python lists & dictionaries to specify |
Make sense! I totally agree. To summarize:
|
While some users may want to use
Schema
s as a nice way to organize fields, others simply want to specify field names, types, and descriptions as part of their PZ program. (See the new.add_column(...)
syntax).I think we should continue to support
Schema
s, especially for under-the-hood operations.Rather than eliminate the
Schema
class, this issue simply proposes that we rewrite our demos / quickstart(s) to use non-Schema
syntax. This issue would also add a page in our documentation which shows users how to use theSchema
class should they choose to do so.To summarize, the acceptance criteria for this issue are:
Schema
syntax.Schema
syntax as a convenience for power users (e.g.demos/schema-demo.py
.Schema
s.The text was updated successfully, but these errors were encountered: