Skip to content

Conversation

@sijunhe
Copy link
Contributor

@sijunhe sijunhe commented Nov 30, 2022

PR types

Function optimization

PR changes

Others

Description

  • support multi-label setup in Text Classification Taskflow

wj-Mcat and others added 6 commits November 30, 2022 19:47
* add basic ci & pypi workflow

* add Makefile file to enable more custom scripts

* update makefile & workflow

* add manifest.in file

* save requirements

* improve makefile

* add test running command

* complete simple CI test script

* complete simple CI test script

* complete first version of workflow

* failure at first

* remove CI workflow

* update workflows

* add dev-dependency

* complete format & lint

* add paddle dependency

* complete format & lint & test command

* upgrae workflow files

* use pytest to do test

* update need keywords

* change the description

* update lint script

* upgrade workflow

* update parameterized

* update makefile

* fix lint

* update test command in workflow

* fix __init__ lint

* ignore all __init__.py file

* fix script workflow

* udpate flake8 config

* update flake config

* update flake8 config

* remove redefinition of unused

Co-authored-by: Sijun He <[email protected]>
* Add multi recall of semantic search for pipelines

* Update multi recall semantic search README.md

* remove unused imports

* remove unused imports

* Update __init__.py

* remove unused imports

* restore __init__.py

* skip retriever __init__.py
@sijunhe sijunhe requested review from wawltor and wj-Mcat November 30, 2022 12:22
@paddle-bot
Copy link

paddle-bot bot commented Nov 30, 2022

Thanks for your contribution!

@sijunhe sijunhe self-assigned this Dec 1, 2022
@sijunhe sijunhe added the taskflow Taskflow label Dec 2, 2022
postprocessed_output = {}
postprocessed_output["predictions"] = [{"label": self.id2label[label], "score": score[label]}]
postprocessed_outputs.append(postprocessed_output)
else: # multi_label
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里是不是elif model == "multi_label" 更好一点? 后面需要考虑一下层次分类接入

Copy link
Contributor

@wawltor wawltor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sijunhe sijunhe merged commit f2a583c into PaddlePaddle:develop Dec 14, 2022
@sijunhe sijunhe deleted the multilabel_taskflow branch December 14, 2022 03:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

taskflow Taskflow

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants