https://dagster.io/ logo
Join Slack
Channels
announcements
dagster-airbyte
dagster-airflow
dagster-bigquery
dagster-cloud
dagster-cube
dagster-dask
dagster-dbt
dagster-de
dagster-ecs
dagster-feedback
dagster-kubernetes
dagster-noteable
dagster-releases
dagster-serverless
dagster-showcase
dagster-snowflake
dagster-support
dagster-wandb
dagstereo
data-platform-design
events
faq-read-me-before-posting
gigs-freelance
github-discussions
introductions
jobs
random
tools
豆瓣酱帮
Powered by Linen
announcements
  • s

    schrockn

    11/14/2018, 12:10 PM
    and then consider other alternatives
  • s

    schrockn

    11/14/2018, 12:10 PM
    this is a decision that will have a lot of ramifications
  • s

    schrockn

    11/14/2018, 12:11 PM
    but i’m happy you flagged it. it’s in a non-ideal state, both from implementation and usability perspective
  • s

    schrockn

    11/14/2018, 12:12 PM
    can you quickly make the config editing pane either resizable or full screen
  • f

    freiksenet

    11/14/2018, 12:13 PM
    ok
  • f

    freiksenet

    11/14/2018, 12:13 PM
    it's possible to special case it
  • f

    freiksenet

    11/14/2018, 12:13 PM
    but I feel a name special case would be better
  • f

    freiksenet

    11/14/2018, 12:13 PM
    for editing experience
  • s

    schrockn

    11/14/2018, 12:13 PM
    special case the context to support the tagged union thing
  • f

    freiksenet

    11/14/2018, 12:14 PM
    yeah
  • s

    schrockn

    11/14/2018, 12:14 PM
    hmmmm
  • f

    freiksenet

    11/14/2018, 12:14 PM
    exactly
  • s

    schrockn

    11/14/2018, 12:14 PM
    what do you envision the special case would look like
  • f

    freiksenet

    11/14/2018, 12:15 PM
    each context is a type, type of "context" parameter initaliazed based on an enum value
  • s

    schrockn

    11/14/2018, 12:16 PM
    mmkay. but dig deeper.
  • s

    schrockn

    11/14/2018, 12:16 PM
    where do you encode the config type of each context type
  • s

    schrockn

    11/14/2018, 12:16 PM
    and where does that get rendered in the dagit type browser
  • f

    freiksenet

    11/14/2018, 12:17 PM
    yeah, it needs a lot of special casing
  • s

    schrockn

    11/14/2018, 12:18 PM
    yeah that is my concern
  • s

    schrockn

    11/14/2018, 12:18 PM
    the approach there now felt like the least special casing
  • s

    schrockn

    11/14/2018, 12:18 PM
    e.g. a struct where only one of the keys can be set
  • s

    schrockn

    11/14/2018, 12:19 PM
    s/keys/fields/
  • f

    freiksenet

    11/14/2018, 12:19 PM
    hmhm
  • f

    freiksenet

    11/14/2018, 12:19 PM
    maybe it's okay
  • f

    freiksenet

    11/14/2018, 12:19 PM
    maybe we should express it in type system somehow tho
  • f

    freiksenet

    11/14/2018, 12:19 PM
    so I don't need to special case it too much
  • s

    schrockn

    11/14/2018, 12:20 PM
    i wonder if we are going to have other instances of this
  • s

    schrockn

    11/14/2018, 12:20 PM
    this is basically input unions
  • s

    schrockn

    11/14/2018, 12:20 PM
    pretty funny
  • f

    freiksenet

    11/14/2018, 12:20 PM
    is there any way to tell from a type how it serializes to json?
Title
f

freiksenet

11/14/2018, 12:20 PM
is there any way to tell from a type how it serializes to json?
View count: 1