https://flyte.org logo
Join the conversationJoin Slack
Channels
announcements
ask-the-community
auth
conference-talks
contribute
databricks-integration
datahub-flyte
deployment
ecosystem-unionml
engineeringlabs
events
feature-discussions
flyte-bazel
flyte-build
flyte-console
flyte-deployment
flyte-documentation
flyte-github
flyte-ui-ux
flytekit
flytekit-java
flytelab
great-content
hacktoberfest-2022
helsing-flyte
in-flyte-conversations
introductions
jobs
konan-integration
linkedin-flyte
random
ray-integration
ray-on-flyte
release
scipy-2022-sprint
sig-large-models
workflow-building-ui-proj
writing-w-sfloris
Powered by Linen
hacktoberfest-2022
  • s

    Sanjiv Anand

    10/03/2022, 4:55 AM
    Hey Can someone from the team heck the article above
    s
    • 2
    • 2
  • c

    cryptic

    10/05/2022, 1:17 PM
    Hello guys, I was going through the related PR that was attached with the TypeTransformer plugin for
    tf.tensor
    issue. I'd a doubt, what does it mean when it was said that "The plugin could specify an unpinned
    torch
    as a dependency".
    I have the same doubt that was raised by @Samhita Alla. How do we prevent re-installing
    torch
    ? Can someone explain me?
    s
    • 2
    • 5
  • s

    Samhita Alla

    10/06/2022, 7:41 AM
    @Prafulla Mahindrakar @Yuvraj, could you please review https://github.com/flyteorg/flytectl/pull/359 PR?
  • s

    Samhita Alla

    10/07/2022, 12:28 PM
    @Kevin Su / @Eduardo Apolinario (eapolinario), https://github.com/flyteorg/flytekit/pull/1229: need a +1.
  • r

    Ryan Nazareth

    10/10/2022, 9:39 AM
    I’m trying to add a tutorial https://github.com/flyteorg/flyte/issues/2676 to the Flyte docs but where does this need to go ? I don’t see any tutorials rst files here https://github.com/flyteorg/flyte/tree/master/rsts
    s
    • 2
    • 1
  • s

    Samhita Alla

    10/10/2022, 2:01 PM
    @Kevin Su @Eduardo Apolinario (eapolinario), can someone review https://github.com/flyteorg/flytekit/pull/1219?
  • s

    Samhita Alla

    10/12/2022, 6:45 AM
    https://github.com/flyteorg/flytetools/pull/66/files: needs to be reviewed.
    • 1
    • 1
  • s

    Samhita Alla

    10/12/2022, 6:51 AM
    @Yee, can you review https://github.com/flyteorg/flytepropeller/pull/491 PR?
  • k

    Krishna Agarwal

    10/14/2022, 11:06 AM
    I've filled the swag form of UnionML of Hacktoberfest, will I receive any confirmation mail?
    s
    • 2
    • 5
  • t

    Thusal Ranawaka

    10/15/2022, 10:54 AM
    Hi, I am a newbie to this Flyte Project, I can contribute for Flyte in documentation.
    s
    • 2
    • 1
  • r

    Ryan Nazareth

    10/18/2022, 1:03 PM
    Hello Flyte team, Ive updated and added a few PRs https://github.com/flyteorg/flytekit/pull/1240 https://github.com/flyteorg/flytekit/pull/1230 and had an initial go at adding an NLP tutorial but not sure why some of the tests are failing https://github.com/flyteorg/flytesnacks/pull/911/files
    s
    d
    • 3
    • 4
  • s

    Samhita Alla

    10/19/2022, 3:59 PM
    @Kevin Su, could you please review https://github.com/flyteorg/flytekit/pull/1230 PR?
  • s

    Samhita Alla

    10/20/2022, 6:13 AM
    @Eduardo Apolinario (eapolinario) @Kevin Su, please review https://github.com/flyteorg/flytekit/pull/1236 PR.
  • a

    Aditya Bisht

    10/24/2022, 5:49 PM
    Wanted to know that this issue https://github.com/flyteorg/flyte/issues/2908 is for this repo https://github.com/flyteorg/flyteconsole right?
    k
    • 2
    • 1
  • s

    Samhita Alla

    10/25/2022, 12:30 PM
    Hey @Niels Bantilan! Regarding https://github.com/flyteorg/flyte/issues/2759,
    tf.Module
    is the parent class for
    tf.keras.layers.Layer
    and
    tf.keras.Model
    (see the note on https://www.tensorflow.org/guide/intro_to_modules page). So we need a type transformer for
    tf.Module
    . But we have two PRs — https://github.com/flyteorg/flytekit/pull/1242 to fix https://github.com/flyteorg/flyte/issues/2759 and https://github.com/flyteorg/flytekit/pull/1241 to fix https://github.com/flyteorg/flyte/issues/2570 that have the same functionality — a type transformer subclassing
    tf.keras.Model
    . We’ll need to tell one of them to modify the code. So which PR shall we retain?
  • s

    Samhita Alla

    10/25/2022, 1:07 PM
    Can someone review https://github.com/flyteorg/flytetools/pull/66 PR, please?
  • s

    Samhita Alla

    10/25/2022, 1:45 PM
    Also @Niels Bantilan, regarding https://github.com/flyteorg/flytekit/pull/1240, are there any use cases you could think of?
    n
    r
    d
    • 4
    • 10
  • r

    Ryan Nazareth

    10/26/2022, 3:39 AM
    I am trying to run the script in my flytesnacks PR https://github.com/flyteorg/flytesnacks/pull/911 on a flyte demo cluster to check out the flytedeck plot but i am getting an error - seems like the kubernetes pods are not responding ? (see screenshot below) ive created the demo cluster as per the instructions in the docs https://docs.flyte.org/en/latest/getting_started/index.html#creating-a-demo-flyte-cluster and exported
    KUBECONFIG
    and
    FLYTECTL_CONFIG
    environment variable values which were printed out after the cluster was setup. Do i need to login to the kubernetes ui and/or configure anything else ?
    k
    s
    • 3
    • 5
  • s

    Sriniketh J

    10/26/2022, 5:54 AM
    Hey team!! I wanted my blog to be reviewed. Here is the link: https://dev.to/srini047/what-is-data-orchestration-4kfb-temp-slug-7953970?preview=44f2a4a274977f9272ea655b1c6839766e9d423bd8ae68c54d7223fba32443a0094b8e7455b66550cb1c5ad94e5d3eab3465b770d3b6de80b2574723
    s
    • 2
    • 4
  • s

    Samhita Alla

    10/26/2022, 7:29 AM
    @Eduardo Apolinario (eapolinario) @Kevin Su, can https://github.com/flyteorg/flytekit/pull/1236 be merged? Also Eduardo, do you think
    typing.TypeVar
    needs to be replaced with
    typing.Annotated
    in FlyteFile even if it isn’t throwing any mypy errors?
    e
    • 2
    • 4
  • c

    cryptic

    10/27/2022, 5:30 AM
    One of the tests on GA is failing due to pytest not being recognized as a command on Windows build (Python 3.7 -spark2) (https://github.com/flyteorg/flytekit/actions/runs/3326738618/jobs/5511139310#step:6:12)
    e
    • 2
    • 38
  • s

    Samhita Alla

    10/27/2022, 7:05 AM
    https://github.com/flyteorg/flytekit/pull/1258#issuecomment-1293083159; @Niels Bantilan @Kevin Su @Eduardo Apolinario (eapolinario), would like to know your thoughts.
  • s

    Samhita Alla

    10/27/2022, 8:54 AM
    @Ryan Nazareth @Kevin Su, left two comments on https://github.com/flyteorg/flytekit/pull/1230 PR. Let me know what you think.
  • s

    Samhita Alla

    10/27/2022, 9:57 AM
    @Kevin Su, apologies for the constant pings 😅. Can you take a look at https://github.com/flyteorg/flytekit/pull/1241#issuecomment-1293274305 comment when you find the time?
  • r

    Ryan Nazareth

    10/27/2022, 10:50 PM
    @FlyteCoreTeam Having another issue when running one of the tasks for the nlp PR. https://github.com/flyteorg/flytesnacks/pull/911 on demo cluster with custom built image I am getting this error for some of the downstream tasks which take the serialised flytefile (
    .model
    ) as input:
    Pod failed. No message received from kubernetes.
    [fb8f63b27ec284bbfbc8-n3-0] terminated with exit code (137). Reason [Error]. Message:
    `tar: Removing leading
    /' from member names
    See screenshot below. The error seems like its trying to extract something from tar archive but Im not doing this anywhere in my task logic ? Also, think someone else has reported this error as well https://flyte-org.slack.com/archives/CP2HDHKE1/p1664441568396559?thread_ts=1664433734.963769&cid=CP2HDHKE1
    e
    n
    • 3
    • 3
  • s

    Samhita Alla

    10/28/2022, 4:40 AM
    @Kevin Su / @Eduardo Apolinario (eapolinario), need a +1: https://github.com/flyteorg/flytekit/pull/1230
  • s

    Samhita Alla

    10/28/2022, 6:00 AM
    @Prafulla Mahindrakar, can I get a +1 for https://github.com/flyteorg/flyte/pull/3033 PR? It fixes the issue you filed.
  • c

    cryptic

    10/28/2022, 6:40 AM
    @Eduardo Apolinario (eapolinario) Would it be a proper way to serialize tensors using pickle?
    s
    • 2
    • 14
  • c

    cryptic

    10/28/2022, 10:25 AM
    @Samhita Alla I have one last question, how do I fix lint errors on
    make lint
    ? I'm sorry but I'm not quite sure about it (https://github.com/flyteorg/flytekit/actions/runs/3326738618/jobs/5511140308#step:6:360)
    s
    • 2
    • 14
  • c

    cryptic

    10/28/2022, 5:34 PM
    @Samhita Alla I'm done pushing the mentioned changes except for linting issues. I was unable to sign off properly because I messed up my pre-commit hooks, will GA run?
    k
    e
    s
    • 4
    • 9
Powered by Linen
Title
c

cryptic

10/28/2022, 5:34 PM
@Samhita Alla I'm done pushing the mentioned changes except for linting issues. I was unable to sign off properly because I messed up my pre-commit hooks, will GA run?
@Eduardo Apolinario (eapolinario) Can my GA builds be run if the DCO check failed? I've to fix my git before rebasing my commits
k

Kevin Su

10/28/2022, 10:10 PM
yes, it can. could you share the link of the pr?
e

Eduardo Apolinario (eapolinario)

10/28/2022, 10:36 PM
@Kevin Su, https://github.com/flyteorg/flytekit/pull/1243 is the PR. I approved the tests.
c

cryptic

10/29/2022, 9:18 AM
@Kevin Su I've given a go at fixing my linting errors, can the GA builds be re-run?
@Eduardo Apolinario (eapolinario) Please approve my tests, i've fixed linting errors
@Ketan (kumare3) Can you please approve my tests?
I've messed up my previous PR, while trying to fix an unexpected dep issue with flyteidl, i've opened a fresh one #1269 (https://github.com/flyteorg/flytekit/pull/1269/) Please allow tests to run @Eduardo Apolinario (eapolinario) (Sorry for the multiple pings, i'm trying to fix issues ASAP)
@Kevin Su Sorry for bothering again, can my tests be approved? Yesterday i'd to face an unexpected dependency issue with flyteidl versioning (#2522), i'd messed up my previous PR so i've opened a new PR here with all the changes (https://github.com/flyteorg/flytekit/pull/1269/)
s

Samhita Alla

10/31/2022, 4:31 AM
@cryptic, I’ll review your changes. Give me some time.
View count: 1