Chansung Park

Results 152 comments of Chansung Park
trafficstars

#174 for the first component

@rcrowe-google actually, the project proposal has been merged a while ago. the thing is I have suggested two separate custom components, but we decided to merge two into one. https://github.com/tensorflow/tfx-addons/blob/main/proposals/20220823-huggingface_model_pusher.md

this sounds very reasonable. what is the current status of this idea?

and is there a way to terminate pipeline from a component? maybe just raise an error?

sounds good and sounds easy to implement I guess. What is your plan on this?

@rcrowe-google so this is equivalent to `FirebasePublisher` mentioned in https://github.com/tensorflow/tfx/issues/4125

@rcrowe-google yes! I have joined the group already :) What are the next steps to proceed?

since their purposes are different, I want TFX users to choose both or just `HFSpacePusher` in their pipeline.

@sayakpaul This component is designed to work without HFModelPusher. Optionally, it can work with Trainer only. Since HFModelPusher can potentially be complex if the model card feature is added later,...

@sayakpaul I thought the former is ideal since this component pushes things to the Space. WDYT? maybe @gante could join this discussion