Skip to content
This repository was archived by the owner on Aug 25, 2024. It is now read-only.

df: base: create Definition for spec#627

Merged
johnandersen777 merged 15 commits intointel:masterfrom
sk-ip:df_base_spec
May 16, 2020
Merged

df: base: create Definition for spec#627
johnandersen777 merged 15 commits intointel:masterfrom
sk-ip:df_base_spec

Conversation

@sk-ip
Copy link
Copy Markdown
Contributor

@sk-ip sk-ip commented May 11, 2020

the Definition for spec (NamedTuple) types will be created.

fixes #601

Signed-off-by: Sudhanshu Sudhanshukumar5459@gmail.com

Copy link
Copy Markdown

@johnandersen777 johnandersen777 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On the right track! Minor change though. You're current setup will result in everything being treated as a map. We'll still need that if statement for non-NamedTuple, non-DataClass cases

@sk-ip sk-ip changed the title df: base: create Definition for spec [WIP] df: base: create Definition for spec May 12, 2020
@sk-ip
Copy link
Copy Markdown
Contributor Author

sk-ip commented May 13, 2020

Adding a checklist for myself

  • Complete the subspec part
  • Write tests for spec / subspec
  • Add tests in the examples of the docstrings in @op
  • Add extra validation to the Input.__init__ side

Copy link
Copy Markdown

@johnandersen777 johnandersen777 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just tests left. This will be very handy! Thanks!

@sk-ip sk-ip changed the title [WIP] df: base: create Definition for spec df: base: create Definition for spec May 16, 2020
@sk-ip sk-ip requested a review from johnandersen777 May 16, 2020 15:00
@johnandersen777 johnandersen777 merged commit 9d622eb into intel:master May 16, 2020
@sk-ip sk-ip deleted the df_base_spec branch May 16, 2020 18:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

df: base: Auto create definitions when they might have a spec

3 participants