Skip to content

[WIP] Magnifier python sample#17416

Closed
seanbudd wants to merge 9 commits into
masterfrom
magnifierSample
Closed

[WIP] Magnifier python sample#17416
seanbudd wants to merge 9 commits into
masterfrom
magnifierSample

Conversation

@seanbudd

Copy link
Copy Markdown
Member

A magnifier sample based on the Microsoft Example

This is not intended for future merge.

@seanbudd seanbudd changed the title [WIP] aMagnifier sample [WIP] Magnifier python sample Nov 19, 2024
@AppVeyorBot

Copy link
Copy Markdown
  • PASS: Translation comments check.
  • PASS: License check.
  • FAIL: Unit tests. See test results for more information.
  • PASS: Lint check.
  • PASS: System tests (tags: installer NVDA).
  • Build (for testing PR): https://ci.appveyor.com/api/buildjobs/ddmehdfi1d214kcv/artifacts/output/l10nUtil.exe nvda_snapshot_pr17416-34602,1ba71923.exe
  • CI timing (mins):
    INIT 0.0,
    INSTALL_START 1.4,
    INSTALL_END 0.9,
    BUILD_START 0.0,
    BUILD_END 26.0,
    TESTSETUP_START 0.0,
    TESTSETUP_END 0.3,
    TEST_START 0.0,
    TEST_END 18.7,
    FINISH_END 0.1

See test results for failed build of commit 1ba71923b6

@AppVeyorBot

Copy link
Copy Markdown
  • PASS: Translation comments check.
  • PASS: License check.
  • FAIL: Unit tests. See test results for more information.
  • PASS: Lint check.
  • PASS: System tests (tags: installer NVDA).
  • Build (for testing PR): https://ci.appveyor.com/api/buildjobs/bcw5vnngxtoh72kf/artifacts/output/l10nUtil.exe nvda_snapshot_pr17416-34640,7db453d7.exe
  • CI timing (mins):
    INIT 0.0,
    INSTALL_START 1.5,
    INSTALL_END 1.0,
    BUILD_START 0.0,
    BUILD_END 29.3,
    TESTSETUP_START 0.0,
    TESTSETUP_END 0.4,
    TEST_START 0.0,
    TEST_END 19.0,
    FINISH_END 0.2

See test results for failed build of commit 7db453d71b

@seanbudd seanbudd closed this Jan 13, 2025
@CyrilleB79

Copy link
Copy Markdown
Contributor

@seanbudd what are the conclusion of this experiment? Is it usable for a future NVDA magnifier? Or will you look for other alternatives?

I have seen that Magnifier is in NVDA's roadmap, but in the list of "ambitious goals".
Is NV Access working on this in background? Or is this PR a piece of material usable by any contributor wanting to implement Magnifier feature?

I am interested in the topic, but not very sur to have so much time to dedicate to such an ambitious goal...

@CyrilleB79 CyrilleB79 mentioned this pull request Jan 21, 2025
5 tasks
@seanbudd

Copy link
Copy Markdown
Member Author

@CyrilleB79 - we discovered (a bit too late) that the full Magnification API is unsupported on WOW64

"The Magnification API is not supported under WOW64; that is, a 32-bit magnifier application will not run correctly on 64-bit Windows." Reference

we'll be putting our energy into converting NVDA to 64bit first, while developing this on the backburner

@CyrilleB79

Copy link
Copy Markdown
Contributor

OK thanks for this valuable clarification.

@CyrilleB79

Copy link
Copy Markdown
Contributor

@seanbudd, now that NVDA has migrated to 64-bit, do you (NV Access) intend to resurrect this work?

@seanbudd

seanbudd commented Nov 4, 2025

Copy link
Copy Markdown
Member Author

@CyrilleB79 - a couple of community members we've been in contact with have been working on it. Visible work should be returning soon

@CyrilleB79 CyrilleB79 mentioned this pull request Nov 6, 2025
8 tasks
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