Skip to content

Revert "fix: parameter rule" #8070

New issue

Have a question about this project? No Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “No 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? No Sign in to your account

Conversation

AAEE86
Copy link
Contributor

@AAEE86 AAEE86 commented Sep 6, 2024

Reverts #8064
The official has not modified the range of top_k.
official documentation
The bug I reported is that the step setting in the front end is incorrect. It should be set to step="1".
<input class="shrink-0 block ml-4 pl-3 w-16 h-8 appearance-none outline-none rounded-lg bg-gray-100 text-[13px] text-gra-900" max="6" min="1" step="0.1" type="number" data-tabindex="" tabindex="-1">

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Sep 6, 2024
@crazywoola crazywoola merged commit 0cef25e into langgenius:main Sep 7, 2024
6 checks passed
@AAEE86 AAEE86 deleted the revert-8064-fix/8057-bug-spark-series-models-parameter-top_k-should-be-int branch September 9, 2024 08:02
@AAEE86 AAEE86 restored the revert-8064-fix/8057-bug-spark-series-models-parameter-top_k-should-be-int branch September 9, 2024 08:03
@AAEE86 AAEE86 deleted the revert-8064-fix/8057-bug-spark-series-models-parameter-top_k-should-be-int branch September 9, 2024 08:03
cuiks pushed a commit to cuiks/dify that referenced this pull request Sep 26, 2024
lau-td pushed a commit to heydevs-io/dify that referenced this pull request Oct 23, 2024
idonotknow pushed a commit to AceDataCloud/Dify that referenced this pull request Nov 16, 2024
No Sign up for free to join this conversation on GitHub. Already have an account? No Sign in to comment
Labels
size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants