Skip to content

Conversation

AlfredChester
Copy link
Contributor

如题,增加了对均匀生成高维区间查询的支持,已经经过测试

@Mr-Python-in-China
Copy link
Collaborator

@AlfredChester
Copy link
Contributor Author

AlfredChester commented Dec 19, 2024

修了,哥们大概 %1 的概率被你碰上了 @Mr-Python-in-China

@Mr-Python-in-China
Copy link
Collaborator

快忘了这还有个issue了

Copy link
Collaborator

@Mr-Python-in-China Mr-Python-in-China left a comment

Choose a reason for hiding this comment

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

还有一些地方没有通过类型检查 留注释了 你再看看

@AlfredChester
Copy link
Contributor Author

@Mr-Python-in-China 咋 FST 了,好像不是我的锅

@AlfredChester
Copy link
Contributor Author

好像是上游工具锅了,我先在我的仓库里面降级看看能不能过 test

@Mr-Python-in-China
Copy link
Collaborator

你怎么把test workflow改了

@AlfredChester
Copy link
Contributor Author

你怎么把test workflow改了

因为不改不知道为什么 tox 没办法找到其他版本的解释器(在 GitHub 上)了,看我第一次 FST 的那个 commit

@Mr-Python-in-China
Copy link
Collaborator

Mr-Python-in-China commented Jan 26, 2025

你怎么把test workflow改了

因为不改不知道为什么 tox 没办法找到其他版本的解释器(在 GitHub 上)了,看我第一次 FST 的那个 commit

不用测这么全啊 3.6没问题3.8没问题3.7也很难有问题

稍微节约点额度 测一次的时间还不少

@AlfredChester
Copy link
Contributor Author

哦好的,那我改成之前的

@Mr-Python-in-China
Copy link
Collaborator

考不考虑什么更多的功能啊

比如说随机更大/更小的区间来满足hack之类的要求

@AlfredChester
Copy link
Contributor Author

考不考虑什么更多的功能啊

比如说随机更大/更小的区间来满足hack之类的要求

考虑的,等省选之后我抽空会实现

@Mr-Python-in-China Mr-Python-in-China force-pushed the master branch 2 times, most recently from 2f520f5 to 7056a5a Compare June 30, 2025 15:49
@AlfredChester
Copy link
Contributor Author

AlfredChester commented Aug 7, 2025

@Mr-Python-in-China 希望早日 merge 吧,感觉再写会写的很臭啊,而且实在想不到啥 feature 了

Copilot

This comment was marked as outdated.

@Mr-Python-in-China Mr-Python-in-China self-assigned this Aug 26, 2025
@Mr-Python-in-China
Copy link
Collaborator

?为啥我没权限推代码

@AlfredChester
Copy link
Contributor Author

?为啥我没权限推代码

这么深刻,我看看

@AlfredChester
Copy link
Contributor Author

我啥也没干啊,你之前不是,push 过吗?

@Mr-Python-in-China
Copy link
Collaborator

image

问号

@AlfredChester
Copy link
Contributor Author

等下,我自己怎么都没办法访问 settings 页了?????

@AlfredChester
Copy link
Contributor Author

你看看行了吗

@langningchen

This comment was marked as outdated.

@Mr-Python-in-China
Copy link
Collaborator

还是不行啊

@langningchen

This comment was marked as outdated.

@langningchen

This comment was marked as outdated.

@Mr-Python-in-China
Copy link
Collaborator

好了

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for generating high-dimensional range queries with uniform distribution. The implementation includes a new RangeQuery class with comprehensive test coverage and documentation.

  • Added a new RangeQuery class in cyaron/query.py with support for multi-dimensional range query generation
  • Implemented comprehensive test suite covering various query generation modes and edge cases
  • Updated project dependencies and CI configuration to support the new functionality

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
cyaron/query.py Core implementation of RangeQuery class with random query generation functionality
cyaron/tests/range_query_test.py Comprehensive test suite for range query functionality
cyaron/__init__.py Export RangeQuery class in module's public API
cyaron/tests/__init__.py Include range query tests in test suite
pyproject.toml Pin poetry-core version for build stability
cyaron/graders/noipstyle.py Minor code formatting improvements
.github/workflows/test.yml Update CI platform and conditional Python 3.6 setup

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <[email protected]>
@Mr-Python-in-China
Copy link
Collaborator

你再看一眼,我准备 merge 了

@AlfredChester
Copy link
Contributor Author

好的,应该没啥了

@Mr-Python-in-China Mr-Python-in-China merged commit e6ace5c into luogu-dev:master Aug 26, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants