Skip to content
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

Access secrets from Forks #229

Closed
wants to merge 256 commits into from
Closed

Conversation

sainivedh
Copy link
Contributor

Why

  • Allows repo secrets to be accessed from external PRs

zeiler and others added 30 commits December 2, 2021 13:50
Added initial listing of resources and auth helper
* Added steps to install package from source

* Added instruction
* use input_success_status

* Address comments re:docstring and spacing

* variable name change
* added support for the base arg

* updated comment

* fixed up stub

* bump version
* making a branch

* fixed

* added pre-commit hook

* added test and workflow

* renmae file

* remove one criteria

* older pytest

* older pytest
sainivedh and others added 27 commits October 16, 2023 22:40
* Version 9.9.3

* downgrade clarifai grpc to 9.8.1
…202)

* changed base_workflow to Empty in doc strings

* Changed base_workflow to Empty
…er model (#201)

* init

* fix + update doc

* addressed comments

* fix cast to number

* minor fix

* address comment

* address comments

---------

Co-authored-by: Arman C. Kizilkale <[email protected]>
* added table of contents and other small chnages to Readme.md

* Made Create PAT and Get UserID as bullet points

* added new line inbetween ToC sections 1 to 4

* added new line inbetween ToC sections 1 to 4

* removed newline between bullet points

* fixed spellings and changed numbers to bullet points

* lint fixes

* added few more descriptions

* removed spacing

* made changes on  sizing and descriptions

* lint fixes
* add inference params and remove input_type in predict

* add Inputs obj to Workflow

* revert removing input_type param

* fix model deploying wait error

* revert base.py

* fix workflow create tests flakiness

* fix app tests flakiness

* update README.md

* add test for model load info
* added table of contents and other small chnages to Readme.md

* Made Create PAT and Get UserID as bullet points

* added new line inbetween ToC sections 1 to 4

* added new line inbetween ToC sections 1 to 4

* removed newline between bullet points

* fixed spellings and changed numbers to bullet points

* lint fixes

* added few more descriptions

* removed spacing

* made changes on  sizing and descriptions

* lint fixes

* fixed table of contents
* Update README.md

* Update README.md
* wip

* change input_types to list

* add input search tests

* add query example in docstring

* fix data upload test

---------

Co-authored-by: Isaac Chung <[email protected]>
Support for model inference params
PostInputsSearch Support
Bump clarifai_grpc==9.10.0
* change pycocotools to extars require

* add pillow to req

* change data to all
* fix topk and input_dataset_ids bugs

* fix search test

* remove max_top_k limit and add resp msg
* Refactor dataset upload process

* add dataset upload type constants

* rename url_init arg

* delete clarifai.auth files

* remove dataset upload examples, loaders

* Refactor change to DATASET_UPLOAD_TASKS

* remove symbolic link clarifai_utils

* remove sphinx docs rst

* remove readthedocs yaml

* remove module_dir in upload_dataset

* fix circular import error

* add loaders back

* change datagen_obj to data_generator

* change typing

* change datagen_item to data_item

* use DATASET_UPLOAD_TASKS in NotImplementedError

* change url_init to just url

* add base_url support to Search

* fix dataloader upload test

* Review Comments
* add dataset upload status feature

* Improve warning logging

* fix typing

* Review Comments

* fix text cls upload

* add timeout as arg
* support pat as arg

* use staticmethods in inputs and necessary changes

* change defaults to None
@sainivedh sainivedh changed the base branch from master to legacy-master November 28, 2023 12:09
@sainivedh sainivedh closed this Nov 28, 2023
@sainivedh sainivedh deleted the allow-secrets-from-forks branch November 28, 2023 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.