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-on-gcp
flyte-ui-ux
flytekit
flytekit-java
flytelab
great-content
hacktoberfest-2022
helsing-flyte
in-flyte-conversations
integrations
introductions
jobs
konan-integration
linkedin-flyte
random
ray-integration
ray-on-flyte
release
scipy-2022-sprint
show-and-tell
sig-large-models
torch-elastic
workflow-building-ui-proj
writing-w-sfloris
Title
r
Ryan Nazareth
10/18/2022, 1:03 PMHello 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
also silly question - are the requirements.txt file in plugins auto generated or do you manually adapt the libraries required for the plugin based off another plugin template ?
s
Samhita Alla
10/18/2022, 1:38 PMThank you, @Ryan Nazareth! Will review the PRs shortly.
You need to manually update the requirements the first time. We do have a bot that creates PRs to update the requirements.txt file in case a new library version is released.
d
Dennis O'Brien
10/20/2022, 6:56 AMIt would be great to get 1240 expedited as there are a number of other Tensorflow and Keras related issues for Hacktoberfest and this PR will make some of the others a bit easier by providing a working example.
r
Ryan Nazareth
10/22/2022, 7:14 PMah, Ive just noticed a number of other tensorflow active PRs - ive tried to mimic the changes to the requirements to avoid conflicts with those PRs - also there is this thread regarding some issues re:
profobuf<http://dev-requirements.in|dev-requirements.in>