Skip to content

Conversation

JoanFM
Copy link
Contributor

@JoanFM JoanFM commented Jun 28, 2023

Pull Request check-list

Please make sure to review and check all of these items:

  • Does $ tox pass with this change (including linting)?
  • Do the CI tests pass with this change (enable it first in your forked repo and wait for the github action build to finish)?
  • Is the new or changed code fully tested?
  • Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?
  • Is there an example added to the examples folder (if applicable)?
  • Was the change added to CHANGES file?

NOTE: these things are not required to open a PR and can be done
afterwards / while the PR is open.

Description of change

According to this https://redis-py.readthedocs.io/en/stable/examples/search_vector_similarity_examples.html#KNN-Queries, there is a missmatch between the type hints of the method and the actual expected usage. This makes mypy complain when being used

@dvora-h dvora-h added maintenance Maintenance (CI, Releases, etc) docs-review labels Jul 16, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jul 16, 2023

Codecov Report

Patch coverage has no change and project coverage change: -1.71% ⚠️

Comparison is base (9f50357) 92.33% compared to head (b24b952) 90.63%.
Report is 9 commits behind head on master.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2817      +/-   ##
==========================================
- Coverage   92.33%   90.63%   -1.71%     
==========================================
  Files         119      126       +7     
  Lines       30522    32229    +1707     
==========================================
+ Hits        28184    29210    +1026     
- Misses       2338     3019     +681     
Files Changed Coverage Δ
redis/commands/search/commands.py 85.26% <0.00%> (+0.65%) ⬆️

... and 85 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@uglide uglide changed the title fix type hints Fix type hints in SearchCommands Aug 2, 2023
@uglide
Copy link
Contributor

uglide commented Aug 2, 2023

LGTM

@dvora-h
Copy link
Collaborator

dvora-h commented Aug 3, 2023

@JoanFM Can you please fix the linters failure?

@JoanFM
Copy link
Contributor Author

JoanFM commented Aug 6, 2023

@JoanFM Can you please fix the linters failure?

Hey @dvora-h,

It would be hard to do this for a couple of weeks that I will not have my laptop with me.

Also, I do not see how to fix this, as the lines related are not even touched by me.

Would it be possible for you or someone else to maybe fix it on a branch and do a PR to my fork perhaps?

@dvora-h
Copy link
Collaborator

dvora-h commented Aug 6, 2023

@JoanFM Thanks for your response, no problem, I will push a fix to your branch.

@dvora-h dvora-h merged commit 3e50d28 into redis:master Aug 6, 2023
chayim added a commit that referenced this pull request Aug 31, 2023
Co-authored-by: Viktor Ivanov <[email protected]>
Co-authored-by: Sergey Prokazov <[email protected]>
Co-authored-by: Anuragkillswitch <[email protected]>
Co-authored-by: dvora-h <[email protected]>
Co-authored-by: Alex Schmitz <[email protected]>
Co-authored-by: Alex Schmitz <[email protected]>
Co-authored-by: Chayim <[email protected]>
Co-authored-by: Bar Shaul <[email protected]>
Co-authored-by: CrimsonGlory <[email protected]>
Co-authored-by: Raymond Yin <[email protected]>
Co-authored-by: zach.lee <[email protected]>
Co-authored-by: James R T <[email protected]>
Co-authored-by: dvora-h <[email protected]>
Co-authored-by: Marc Schöchlin <[email protected]>
Co-authored-by: Nick Gerow <[email protected]>
Co-authored-by: Igor Malinovskiy <[email protected]>
Co-authored-by: Chayim I. Kirshen <[email protected]>
Co-authored-by: Leibale Eidelman <[email protected]>
Co-authored-by: Thiago Bellini Ribeiro <[email protected]>
Co-authored-by: woutdenolf <[email protected]>
Co-authored-by: shacharPash <[email protected]>
Co-authored-by: Mirek Długosz <[email protected]>
Co-authored-by: Oran Avraham <[email protected]>
Co-authored-by: mzdehbashi-github <[email protected]>
Co-authored-by: Tyler Hutcherson <[email protected]>
Co-authored-by: Felipe Machado <[email protected]>
Co-authored-by: AYMEN Mohammed <[email protected]>
Co-authored-by: Marc Schöchlin <[email protected]>
Co-authored-by: Avasam <[email protected]>
Co-authored-by: Markus Gerstel <[email protected]>
Co-authored-by: Kristján Valur Jónsson <[email protected]>
Co-authored-by: Nick Gerow <[email protected]>
Co-authored-by: Cristian Matache <[email protected]>
Co-authored-by: Anurag Bandyopadhyay <[email protected]>
Co-authored-by: Seongchuel Ahn <[email protected]>
Co-authored-by: Alibi <[email protected]>
Co-authored-by: Smit Parmar <[email protected]>
Co-authored-by: Brad MacPhee <[email protected]>
Co-authored-by: Shahar Lev <[email protected]>
Co-authored-by: Vladimir Mihailenco <[email protected]>
Co-authored-by: Kevin James <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: David Pacsuta <[email protected]>
Co-authored-by: Rich Bowen <[email protected]>
Co-authored-by: gmbnomis <[email protected]>
Co-authored-by: Vivanov98 <[email protected]>
Co-authored-by: Kosuke <[email protected]>
Co-authored-by: Sergey Prokazov <[email protected]>
Co-authored-by: jmcbailey <[email protected]>
Co-authored-by: Galtozzy <[email protected]>
Co-authored-by: Abhishek Kumar Sinha <[email protected]>
Co-authored-by: Eom Taegyung "Iggy <[email protected]>
Co-authored-by: Mehdi ABAAKOUK <[email protected]>
Co-authored-by: Dongkeun Lee <[email protected]>
Co-authored-by: woutdenolf <[email protected]>
Co-authored-by: Kurt McKee <[email protected]>
Co-authored-by: Juraj Páll <[email protected]>
Co-authored-by: Joan Fontanals <[email protected]>
Co-authored-by: Stanislav Zmiev <[email protected]>
fix (#2566)
Fix unlink in cluster pipeline (#2562)
Fix issue 2540: Synchronise concurrent command calls to single-client mode. (#2568)
Fix: tuple function cannot be passed more than one argument (#2573)
Fix issue 2567: NoneType check before raising exception (#2569)
Fix issue 2349: Let async HiredisParser finish parsing after a Connection.disconnect() (#2557)
Fix issue with `pack_commands` returning an empty byte sequence (#2416)
Fix #2581 UnixDomainSocketConnection' object has no attribute '_command_packer' (#2583)
Fix #2581 UnixDomainSocketConnection' object has no attribute '_command_packer' .
Fix for `lpop` and `rpop` return typing (#2590)
Fixed CredentialsProvider examples (#2587)
Fixed issue #2598 - make Document class subscriptable
fix: replace async_timeout by asyncio.timeout (#2602)
Fix behaviour of async PythonParser to match RedisParser as for issue #2349 (#2582)
Fix (#2641)
fix: do not use asyncio's timeout lib before 3.11.2 (#2659)
Fix issue 2660: PytestUnraisableExceptionWarning from asycio client (#2669)
Fixing cancelled async futures (#2666)
Fix async (#2673)
Fix memory leak caused by hiredis (#2693) (#2694)
Fix incorrect usage of once flag in async Sentinel (#2718)
Fix topk list example. (#2724)
Fix `ClusterCommandProtocol` not itself being marked as a protocol (#2729)
Fix potential race condition during disconnection (#2719)
fix CI (#2748)
fix parse_slowlog_get (#2732)
fixes for issue #1128
fix create single_connection_client from url (#2752)
Fix `xadd` allow non negative maxlen (#2739)
Fix JSON.MERGE Summary (#2786)
Fixed key error in parse_xinfo_stream (#2788)
Fix dead weakref in sentinel connection causing ReferenceError (#2767) (#2771)
Fix dead weakref in sentinel conn (#2767)
fix redirects and some small cleanups (#2801)
Fix type hint for retry_on_error in async cluster (#2804)
Fix CI (#2809)
Fix async client with resp3 (#2657)
Fix `COMMAND` response in resp3 (redis 7+) (#2740)
Fix protocol version checking (#2737)
Fix parse resp3 dict response: don't use dict comprehension (#2757)
Fixing asyncio import (#2759)
fix (#2799)
fix async tests (#2806)
Fix socket garbage collection (#2859)
Fixing doc builds (#2869)
Fix a duplicate word in `CONTRIBUTING.md` (#2848)
Fix timeout retrying on Redis pipeline execution (#2812)
Fix type hints in SearchCommands (#2817)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Maintenance (CI, Releases, etc) ready-for-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants