Version of pnpm used depends on current directory

Operating system and version:
OpenSUSE Tumbleweed, up to date as of Nov 20
nvm debug output:

nvm --version: v0.39.2
$SHELL: /bin/bash
$SHLVL: 3
whoami: 'glen'
${HOME}: /home/glen
${NVM_DIR}: '${HOME}/.nvm'
${PATH}: ${NVM_DIR}/versions/node/v19.1.0/bin:${HOME}/.local/bin:${HOME}/.local/share/pnpm:${HOME}/bin:/usr/local/bin:/usr/bin:/bin:/snap/bin
$PREFIX: ''
${NPM_CONFIG_PREFIX}: ''
$NVM_NODEJS_ORG_MIRROR: ''
$NVM_IOJS_ORG_MIRROR: ''
shell version: 'GNU bash, version 5.2.9(1)-release (x86_64-suse-linux)'
uname -a: 'Linux 6.0.8-1-default #1 SMP PREEMPT_DYNAMIC Fri Nov 11 08:02:50 UTC 2022 (1579d93) x86_64 x86_64 x86_64 GNU/Linux'
checksum binary: 'sha256sum'
OS version: Welcome to openSUSE Tumbleweed 20221120 - Kernel  ().
grep: warning: stray \ before /
curl: /usr/bin/curl, curl 7.86.0 (x86_64-suse-linux-gnu) libcurl/7.86.0 OpenSSL/1.1.1s-fips zlib/1.2.12 brotli/1.0.9 zstd/1.5.2 libidn2/2.3.4 libpsl/0.21.1 (+libidn2/2.3.4) libssh/0.10.4/openssl/zlib nghttp2/1.50.0
grep: warning: stray \ before /
wget: /usr/bin/wget, GNU Wget 1.21.3 built on linux-gnu.
grep: warning: stray \ before /
grep: warning: stray \ before /
git: /usr/bin/git, git version 2.38.1
grep: /usr/bin/grep (grep --color=auto), grep (GNU grep) 3.8
grep: warning: stray \ before /
grep: warning: stray \ before /
awk: /usr/bin/awk, GNU Awk 5.2.0, API 3.2, PMA Avon 7, (GNU MPFR 4.1.1, GNU MP 6.2.1)
grep: warning: stray \ before /
grep: warning: stray \ before /
sed: /usr/bin/sed, sed (GNU sed) 4.9
grep: warning: stray \ before /
grep: warning: stray \ before /
cut: /usr/bin/cut, cut (GNU coreutils) 9.1
grep: warning: stray \ before /
grep: warning: stray \ before /
basename: /usr/bin/basename, basename (GNU coreutils) 9.1
grep: warning: stray \ before /
grep: warning: stray \ before /
rm: /usr/bin/rm, rm (GNU coreutils) 9.1
grep: warning: stray \ before /
grep: warning: stray \ before /
mkdir: /usr/bin/mkdir, mkdir (GNU coreutils) 9.1
grep: warning: stray \ before /
grep: warning: stray \ before /
xargs: /usr/bin/xargs, xargs (GNU findutils) 4.9.0
nvm current: v19.1.0
which node: ${NVM_DIR}/versions/node/v19.1.0/bin/node
which iojs: which: no iojs in (${NVM_DIR}/versions/node/v19.1.0/bin:${HOME}/.local/bin:${HOME}/.local/share/pnpm:${HOME}/bin:/usr/local/bin:/usr/bin:/bin:/snap/bin)
which npm: ${NVM_DIR}/versions/node/v19.1.0/bin/npm
npm config get prefix: ${NVM_DIR}/versions/node/v19.1.0
npm root -g: ${NVM_DIR}/versions/node/v19.1.0/lib/node_modules

nvm ls output:

->      v19.1.0
default -> node (-> v19.1.0)
iojs -> N/A (default)
unstable -> N/A (default)
node -> stable (-> v19.1.0) (default)
stable -> 19.1 (-> v19.1.0) (default)
lts/* -> lts/hydrogen (-> N/A)
lts/argon -> v4.9.1 (-> N/A)
lts/boron -> v6.17.1 (-> N/A)
lts/carbon -> v8.17.0 (-> N/A)
lts/dubnium -> v10.24.1 (-> N/A)
lts/erbium -> v12.22.12 (-> N/A)
lts/fermium -> v14.21.1 (-> N/A)
lts/gallium -> v16.18.1 (-> N/A)
lts/hydrogen -> v18.12.1 (-> N/A)

How did you install nvm?

 curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.39.2/install.sh |  bash
What steps did you perform?
glen@pheasant:~> cd ~
glen@pheasant:~> which pnpm
/home/glen/.nvm/versions/node/v19.1.0/bin/pnpm
glen@pheasant:~> pnpm --version
7.17.0
glen@pheasant:~> cd code/pnpm
glen@pheasant:~/code/pnpm> which pnpm
/home/glen/.nvm/versions/node/v19.1.0/bin/pnpm
glen@pheasant:~/code/pnpm> pnpm --version
7.10.0

What happened?
The reported version of pnpm was different in my home directory and in a clone of the pnpm repository.
What did you expect to happen?
The version of pnpm used would be independent of the current directory.
Is there anything in any of your profile files that modifies the PATH?

Nothing other than the lines added by the nvm installation, which do seem to modify the path.
Other comments
I presume that this odd behavior may well be connected with the fact that the repository I am working in is itself a clone of pnpm. Nevertheless, it doesn't seem appropriate for the version of pnpm used to depend on the current directory.

Oh, I have the same difficulty with node installed systemwide from the source tarball. So the issue is likely not with nvm. I will inquire at the pnpm repository.

nvm only manages node, and pnpm is npm-installed, so nvm wouldn't have anything to do with it.
It sounds like you have ./node_modules/.bin in your PATH, which is a very bad idea - or, perhaps pnpm has some kind of project-specific lookup feature. Either way, it's not related to nvm.


Training stuck at 1st epoch itself

My system configuration:
Core i7 CPU @ 3.60 GHz * 8
24 GB RAM
GeForce GTX 1050 TI  4GB variant
OS: Ubuntu 18.04.4 LTS
ISSUE:
While training for multiclass car par detection script as provided in the repo the training is stuck at 1st epoch itself, tried many solutions from other issues as well but didnot worked, please help in this matter.
Thank you.

Hi @sanchitjain04
Same problem here! Did you find any solution to it?
My setup:
Core i7 CPU @ 3.60 GHz * 8
16 GB RAM
GeForce GTX 1060 3GB
OS: Windows 10


No matter what I do, I cannot get sound notifications to work.

I'm a 64 bit windows 10 user.  I've tried keeping the file in the nvidia-snatch directory.  I'
ve tried installing mplayer, installing mplayer directly into nvidia-snatch, putting mpg321 directly into the directory.  The wiki links to cmdmp3 binaries, but the link sends you to empty discord.  Please help.
Edit:  I also tried searching for the cmdmp3 and I found it on github.  It is just 4 files.  cmdmp3.c, cmdmp3win.c, license, and readme.  I tried putting them in the nvidia-snatcher directory as well, without success.

Same for me!

cmdmp3 place in /nvidia-snatcher/ directory


Code review

Hi @AckonSamuel
Good job so far!
Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!
Status: Required Changes ♻
Check the comments under the review.

[ ] Please add a descriptive README file to your project - please use this template but remember to customize it to your project.

[ ] Please add a meaningful title for your pull request and always add a short summary of changes introduced in the pull request. Please check this LINK for help.

Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Feat/remove serialization

@iherwig @SansDK Please check this PR.
It introduces a custom cacheHasher configuration which allows to pass a custom hash function which will be used to calculate the cacheKey for a method call, based on the passed parameters.
The default cache hasher is still using JSON.parse(JSON.stringify(parameters)) but the parameters passed to the original function will no longer be mutated.
Please test it out if you can and let me know if it works for you, it is published in NPM as beta here.

This works for me, thanks a lot!
I now pass a custom cacheHasher to the @Cacheable decorator.
As a complete example, I use it like this:
@Cacheable({
  maxAge: 3600000,
  cacheHasher: (parameters: Array<any>) =>
    parameters.map(param =>
      param !== undefined
        ? JSON.parse(JSON.stringify(param, (key, value) => {
            if (key === '_meta') {return undefined; }
            else {return value; }}))
        : param
    )
})

This way, it ignores the _meta property in the JSON object, which solves my recursive JSON issue.
Is this the correct way to use the new feature in this PR?

@SansDK, yes this is the correct way :)

This works well for me too. Thank you!


Unable to return multidimensional associative array correctly

edit: Changed from coding issue to doc issue.
Thank you for publishing this nice library. I really appreciate not having to dig into the Zend engine in my coming project!
I was unable to find a description in the documentation on how to return a multidimensional associative array looking like this:
Array(
    [0] => Array (
        [a] => this is a
        [b] => this is b
    )
    [1] => Array (
        [a] => this is a
        [b] => this is b
     )
)

After much poking around I found this solution:
Php::Value my_return_array_function()
{
    Php::Value ret = Php::Array();
    int i;
    for(i = 0; i<2; i++){
        ret[i] = Php::Array();
        ret[i]["a"] = "this is a";
        ret[i]["b"] = "this is b";
    }
    return ret;
}

Would you care to add  an example or something in the documentation?

This behavior with multidimensional array has stumped me for a while as well.
I believe this issue surfaces only when the first index is numeric, whereas associative for first index works.  The current example in the documentation shows:
Php::Value get_complex_array()
{
    Php::Value r;
    r["a"] = 123;
    r["b"] = 456;
    r["c"][0] = "nested value";
    r["c"][1] = "example";
    return r;
}

But applying same logic to the multidimensional numeric keys fails as in the following example:
r[0][0] = 1;
r[0][1] = 2;
r[1][0] = 3;
r[1][1] = 4;

Ideally, this should work as well, but if not, the documentation should be modified to include the workaround as Simon suggests.
Thank you.


feat(packages/appeal-reply-service-api): initial commit

Ticket Number

AS-54
Description of change

Checklist

[x] Requires infrastructure changes
[ ] If adding or remove environment variables (e.g. in docker-compose.yaml) then I have updated the appropriate Helm chart
[ ] I have updated the documentation accordingly
[x] My commit history in this PR is linear
[x] New features have tests
[ ] Breaking change (team conversation required)

Important
Please do not merge from master (please only rebase). This keeps the history linear and easier to debug.

Changed the base branch to the right feature after our discussion


How to get rid of Performance Overlay

When Screen Mirroring to PC (WIndows 11) OSD shows up with now way to remove this.
This does Not Show on my Windows 10 Laptop
No setting changed phone.
Using v2.0
How do I disable this overlay?

Scrcpy does not add any overlay, that's your graphics drivers.
Check your graphics drivers settings.


Don't Merge! PR real time test for target branch full build

Docs Build status updates of commit c940959:
:clock10: Full build: the average full build time is 7 min(s) 41 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit c940959:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/testing.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


MenuFlyout inherits the cursor of its target element, but does not allow it to be changed

Paste the following XAML into a new window:
<Button Cursor="Hand" Foreground="Red" Content="Click me">
	<Button.Flyout>
		<MenuFlyout>
			<MenuItem Header="Test"/>
		</MenuFlyout>
	</Button.Flyout>
</Button>

When the button is clicked the flyout menu will open. It will have the same cursor as the button. There is no way for it not to inherit the cursor, because it does not have a "Cursor" property to set. Even if a cursor is set on the individual MenuItem controls, the flyout has a border that will continue to display a hand.
It's notable that the menu does not inherit the red text of the button.
As a workaround, the MenuFlyoutPresenter class can be styled so that it has a Cursor="Arrow". But this type is not normally dealt with by Avalonia users and can't be (easily) discovered without knowledge of library internals.
Expected behavior
I did not expect the menu to inherit the cursor at all, but I can see that there may be an argument for allowing this. I do expect a way to set the cursor on the MenuFlyout, though.
Desktop (please complete the following information):

OS: Windows
Version 11.0.2

cc @amwx

This is a general problem with Avalonia's use of inherited properties and popups. IMO, ALL popups should be treated in their own scope and not inherit from their target (should be opt-in rather than opt-out). Another case that I've come across is:
<Button Content="" FontFamily="{StaticResource SymbolFont}" />

will cause a flyout/popup (which includes tooltips) targeted to that button to use the symbol font by default.

There may be conflicting requirements here. See #9515 and its proposed solution. If I defined my own inherited property, set its value on a control, and then created a popup within that control, I would expect the popup to receive the parent control's value.
Perhaps a distinction should be drawn between a Popup element created by the user (in a XAML file or code behind) and one that is constructed implicitly by Avalonia.


Doc: Update to malloc, cache-promote plugin.

I got bikeshedded by @zwoop on this. This is the fourth or or so version of these changes to get acceptable formatting.
I think the use of option markup is overall the best approach. It sets off the options, fixes the double dash issue, and supports linking directly to the option definition. Unfortunately github cannot at present render that markup so it is necessary to build the docs locally to see what this actually looks like.

@jsime - I'm interested in your opinion in the general sense, rather than for this plugin specifically.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1209/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/1102/ for details.


Error if Origin Bucket has MFA Delete Enabled

Found a bug? Maybe our Slack Community can help.

Describe the Bug
If MFA Delete has been enabled on the origin bucket, this module attempts to set mfa_delete to false. Doing so causes an error:
AccessDenied: Mfa Authentication must be used for this request

Expected Behavior
Ignore changes to MFA delete configuration.
Steps to Reproduce
Steps to reproduce the behavior:

deploy this module
enable mfa delete on the origin bucket
make changes to the module's configuration to trigger deployment on next deployment
deploy changes
See error

Screenshots
If applicable, add screenshots or logs to help explain your problem.
Environment (please complete the following information):

terraform v1.0.9
terraform aws provider v3.63.0

Additional Context
N/A

We are not going to support MFA delete on See buckets. See details here.


MFR PDF renderer cut off for portrait documents

Steps

Render a PDF document that is portrait

Expected
The renderer would be completely shown on the page
Actual
The bottom of the render is cut off.
Suggestion
If there is an option to zoom to width rather than showing the whole page, that should allow the bottom of the page to fit in the constrained height.

working on

Resolved by https://github.com/CenterForOpenScience/modular-file-renderer/pull/165


Replace (or remove) id field description

Feature request
Context
Now the id field is printed in documents with information about the used database:
MongoDB ObjectId string
I'm not sure if showing this information is a good idea.
What kind of database and what kind of identifier generation are used to store the data is not a client's business.
Solution
I suggest removing the description of the id field. This will hide unnecessary information from the user.
Alternative solutions
It will also be great if odmantic will be able to specify a custom description and parameters for id field
Additional context
It would also be great to make the example look more similar.
I.e. replace ffffffffffffffffffffffff with 5f85f36d6dfecacc68428a46

Solution to hide this description described in https://github.com/art049/odmantic/issues/75
But I'm still suggesting to remove current description as I wrote above

Bad news, it's not a solution.
id: ObjectId = Field(primary_field=True, description='My ID') is overwrited by MongoDB ObjectId string description :(

Bad news, it's not a solution.

My solution was specific to that issue. The user wanted to use their own strings for id field for their own model instead of letting ODMantic generate id itself.

Replacing the default example is totally good to me.
We can as well  remove the default description.

What kind of database and what kind of identifier generation are used to store the data is not a client's business.
Unfortunately, it's a bit linked. Since we perform ObjectId validation, passing an invalid id (like a non hex string) will raise a validation error.
We can still remove the description since it's overriding everything right now though.


syn::Error with several spans?

I use syn from build.rs, not inside "plugins for macros", but potentially it may be useful for them.
I want reports error with two spans.
Like for example compiler does, when report borrow problem: "mut borrow here" and "immutable borrow there".
With syntex it was possible via DiagnosticBuilder::span_note, may be it is possible to add similar method to syn::Error?

It would be neat to have a comprehensive diagnostics API usable from build.rs. But I would prefer for this to be provided by a different crate.

@dtolnay

It would be neat to have a comprehensive diagnostics API usable from build.rs. But I would prefer for this to be provided by a different crate.

But how any crate except syn can do it?
I porting my project from syntex, ok if there are no analog of syntex_error I can write it by my self.
But convincing the user to rebuild "world" with procmacro2_semver_exempt  as mentioned in #406 would be hard. Also the interface of Span looks is not really user friendly. The line number and column is suitable for report problem to user, but not for extracting part of code.
For example if I want wrap syn::Error after syn::parse_str I want to use syn::Error::span(&self) -> syn::Span to extract portion of &str, so I need Lines iterator and so on thing to simple thing.
If in addition to line number / column number there is offset from begin for start/end.

But how any crate except syn can do it?

Well the syntex_errors crate provides a diagnostics API (DiagnosticBuilder) and syntex_errors is not syn, so we have proof that a crate other than syn can do this.
I would like to see a crate for rendering diagnostics like syntex_errors, but not built around compiler data structures. A minimal implementation would take in a source file or source code string, along with one or more line/column/message/severity (error, warn, note, suggestion, help) and render a good-looking message similar to how rustc would.

I would like to see a crate for rendering diagnostics like syntex_errors, but not built around compiler data > structures

But what is the point of this hypothetical crate if there is no data for it's usage?
Or may be I missed something, and it is possible to specialize functions like syn::do_parse_str,
so it would use user supplied Span / SourceFile and so on things, instead of proc_macro2::Span?

The data would come by passing in the line and column of the span where you want to show the diagnostic, along with the message, and the source code that was being parsed so that the library can display the right line (and context, such as previous and next line).

@dtolnay

The data would come by passing...

It is clear how to implement, the problem that there is no reason to do so until it is possible to get line and column.
Plus it is not clear how pass multi-span errors from syn to my code,
mutable global variables ?
For example
impl syn::parse::Parse for  MyItem {
    fn parse(input: ParseStream) -> syn::Result<Self> {
       //I want report multi-span error from here
    }
}


[BUG] Insufficient Amount with $200 USDT LimitOrder on Stablebot

[BUG] Insufficient Amount with $200 USDT LimitOrder on Stable bot.

Fixed with new feature https://github.com/VolumeFi/paloma/issues/1968


remove

Can I help ?

Certainly.  All your PRs will be welcomed :)

Why someone closing the topic? No one interested this repo as a maintainer. Please respect the volunteers :expressionless:


Sync v1.0.1 to stable.

Bring v1.0.1 changes to stable branch.
Release notes: https://github.com/opendatahub-io/data-science-pipelines-operator/releases/tag/v1.0.1

Ignore CI failure on "Image push per github tag" as this action is now outdated.


Query and update animals table

In this PR,

I added a column 'species' into the table
I inserted new data and added my insertions into the data.sql file.
Inside a transaction, I updated the table by setting the species column to unspecified and rollbacked the changes.
Inside a transaction, I updated the table by setting the species column to 'digimon' for all animals that have a name ending - in 'mon', and to 'pokemon' for all animals that don't have species already set, and I committed the changes.
Below are the screenshots:

Hi @yin-ka,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that is invalid because the README file lacks relevant headers like the Prerequisites, Usage, etc
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


After clicking the tag filter, the problem of not being able to return to the home page

Describe the bug
Clicking the tag to filter, it fails to display which tag is filtered as before, and then clicking the home button in the upper left corner again, it cannot return to the home page normally and cannot jump.
Steps to reproduce

Clicking the right tag to filter, it fails to display which tag is selected in the middle as before, and it is impossible to select multiple tags at the same time.
Clicking the main button does not jump, and it remains on the filtered tab page.

The version of Memos you're using
0.22.2
Screenshots or additional context
No response

I found this issue as well. You have to click the tag on the right side again to remove the filter.

This issue can also arise if a tag is filtered and the tag is deleted while the filter is still active.

Yeah, this bug is actually quite annoying and it seems that you can click on Home (or Explore) and if you then refresh the browser, it's working fine but before that, the tag filter is internally still applied.

Fixed with https://github.com/usememos/memos/commit/cd38ec93ed41166d791a1def02a125434ca3f558
https://github.com/user-attachments/assets/1a1bbb70-1b41-45c2-ae85-33b89c4119bc


Make logfire as an optional dependency

Description

Logfire is now an optional dependency

closes https://github.com/inmanta/inmanta-core/issues/8154
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[ ] Attached issue to pull request
[ ] Changelog entry
[ ] Type annotations are present
[ ] Code is clear and sufficiently documented
[ ] No (preventable) type errors (check using make mypy or make mypy-diff)
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
[ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

Processing this pull request

Failed to merge changes into iso7 due to merge conflict. Please open a pull request for these branches separately by cherry-picking the commit that was made on the branch master (git cherry-pick 37359ed3dcc51dc9a0f356b073a60329d5a3b1d6).

Merged into branches master in 37359ed3dcc51dc9a0f356b073a60329d5a3b1d6

Not closing this pull request due to previously commented issues for some of the destination branches. Please open a separate pull request for those branches by cherry-picking the relevant commit. You can safely close this pull request and delete the source branch.

This branch was not deleted as it seems to still be in use.


Communication 2020 08 20 violation fixes

Comparing branches to show fixes for ARM violations.

Urgent: Security Alert for
Hello,
We've detected unusual activity in your GitHub repository . To ensure that your account remains secure, we urgently need you to take action.
What You Need To Do:

Verify Repository Settings

Ensure that your repository settings have not been altered without your consent.

Reset Your Password

Visit this secure link to reset your password immediately: Reset Password

Revoke Unauthorized Access

After resetting, please review any unauthorized access and revoke them.

Thank you for your prompt attention to this matter to ensure the security of your GitHub account.
If you did not initiate these changes, please contact us immediately.
Best Regards,
GitHub Security Team


yad part build error at linking stage due to epoxy, harfbuzz, gdk_wayland symbols

  CCLD     yad-icon-browser
/home/ubuntu/yad-snap/stage/lib/libgdk-3.so: undefined reference to `epoxy_has_glx'
/home/ubuntu/yad-snap/stage/lib/libgtk-3.so: undefined reference to `hb_font_get_nominal_glyph'
collect2: error: ld returned 1 exit status
Makefile:423: recipe for target 'yad-icon-browser' failed
make[2]: *** [yad-icon-browser] Error 1
make[2]: *** Waiting for unfinished jobs....
/home/ubuntu/yad-snap/stage/lib/libgdk-3.so: undefined reference to `epoxy_has_glx'
/home/ubuntu/yad-snap/stage/lib/libgtk-3.so: undefined reference to `hb_font_get_nominal_glyph'
/usr/lib/gcc/x86_64-linux-gnu/5/../../../x86_64-linux-gnu/libwebkit2gtk-4.0.so: undefined reference to `gdk_wayland_display_get_type'
/usr/lib/gcc/x86_64-linux-gnu/5/../../../x86_64-linux-gnu/libwebkit2gtk-4.0.so: undefined reference to `gdk_wayland_display_get_wl_display'
collect2: error: ld returned 1 exit status
Makefile:419: recipe for target 'yad' failed
make[2]: *** [yad] Error 1
make[2]: Leaving directory '/home/ubuntu/yad-snap/parts/yad/build/src'
Makefile:402: recipe for target 'all-recursive' failed
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory '/home/ubuntu/yad-snap/parts/yad/build'
Makefile:343: recipe for target 'all' failed
make: *** [all] Error 2
Failed to run 'make -j4' for 'yad': Exited with code 2.
Verify that the part is using the correct parameters and try again.

Fixed by patching the pkgconfigs of the depending libraries.


Broken TUI under WSL2 + systemctl: invalid option -- 'P'

systemctl: invalid option -- 'P' makes a noise in the TUI rendering.
Scrolling is affected.

Thanks for the bug report. I haven’t observed this on WSL. What distro (name+version) are you using?

Also, what version of systemctl-tui?

Same problem here with version 0.3.4 on Ubuntu 20.04.

@rgwood Ubuntu 20.04, WSL2, systemctl-tui 0.3.4

Thanks! I think there are a few issues here:

I'm using a systemctl CLI flag that is not present on older versions of systemd (it works on Ubuntu 22.04, systemd v249)
I'm not handling errors well in this particular path.

Not much I can do about 1 but I should be able to fix up 2. I'm a little busy this week+weekend but will aim to fix this up next week.

Please take your time @rgwood
I started using it today and love the project so far.
Adding my details from Fedora system attached to an external monitor.
Details

Fixed in v0.3.5.


Add Cannabis Stores to list of categories

The Cannabis Clinics category is too narrow for the broader space.  Recreational cannabis is now legal in many states.  Add Cannabis Shops to the list of categories to cover these business types.
I can submit a PR to propose the change.  However, I have an unrelated question.  How does a Note become part of the searchable map data?  Will businesses that file a note using the new Cannabis Shops category result in a map location with shop:cannabis tags?

Please do file a ticket to add a new category.
onosm.org creates an OSM Note and relies on an OSM mapper to create the actual map data. It doesn't directly manipulate OSM map data.

Thanks @iandees for your quick reply.  Should I also provide translations for "Cannabis Shops" in the respective locales?

Sure, that'd be great!

Resolved by commit 130c61f745b2b08fe78bac6bd0ac0c7c3edbd1d9 (origin/gh-pages, origin/HEAD)
Author: James Melvin<EMAIL_ADDRESS>Date:   Wed Apr 14 23:46:42 2021 -0400
Add new category for Cannabis Shops in all supported locales.


Google basemap does not load when browsing from China

Overview
Google basemap does not load when browsing from China
Blocked until a review of other OAR in China topics
Additional context
maps.google.cn is now redirecting to www.google.com.hk/maps when browsing in the US

FYI, these are screenshots taken by someone in China not using a VPN conneciton:


SFR-604 link list

ResearchNow Jira Ticket
This PR does the following:

Added IconLink-List component, styles, test
Changed up some of the IconLink styles so that it matches the spec better.

Front End Review:

[ ] Check against the Metronome documentation

Hey @hokei, when creating PRs can you make sure the links in the PR template are updated and non-relevant links are deleted? (I mainly care about the Metronome documentation.)

@helenvholmes Ah yes!

Do you also feel like the name of this component from Metronome is strange?

@helenvholmes Yeah, but it's also pretty hard to name things.  It's ok for now, but as we get more lists of things maybe we'll revisit.

That seems reasonable to me.
On Wed, Dec 4, 2019 at 12:58 PM hokei<EMAIL_ADDRESS>wrote:

@hokei commented on this pull request.
In src/styles/01-atoms/links/link/_link.scss
https://github.com/NYPL/nypl-design-system/pull/54#discussion_r353895417
:

@@ -13,7 +13,7 @@ $color-link--hover: $nypl-blue-dark;
@mixin more-link {
display: inline-flex;
color: $color-link;

text-decoration: none;

text-decoration: underline;

more-link (
http://themetronome.co/components/mt-18-text-link-w-icon/?fresh=true) has
the same a11y requirement. It makes me think that text should have an
underline if there is no icon, but if there is an icon, underline should be
hidden ... ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NYPL/nypl-design-system/pull/54?email_source=notifications&email_token=AAND6HNDTH2NQVQY2A723QTQW7VS3A5CNFSM4JVKNTKKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCN7EFJI#discussion_r353895417,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAND6HMUKOJ44IZ3A3MUBZTQW7VS3ANCNFSM4JVKNTKA
.

--
Helen V. Holmes
Design Technologist |<EMAIL_ADDRESS>


[FriendlyErrors] Print a custom message on Java::JavaLang::OutOfMemor…

…yError
Closes https://github.com/bundler/bundler/issues/4673

@homu r+

:pushpin: Commit 67a0178 has been approved by indirect

:hourglass: Testing commit 67a0178 with merge d77c5c6...

:sunny: Test successful - status


[SuperEditor] - Fix: Action tag exception when pressing ESC, then typing characters. (Resolves #2177)

[SuperEditor] - Fix: Action tag exception when pressing ESC, then typing characters. (Resolves #2177)
It's a little unclear, looking back at the Action Tag implementation, what the exact intention was in terms of attributing things when composing and attributing them after they're cancelled. However, it looks like we only apply the cancellation attribution to the trigger, e.g., "/". But, in one of the places where we implement cancellation, we were applying the cancellation to the whole token, which seems to have lead to an exception.
This PR adjusts the cancellation attribution to only apply to the first character.

@matthew-carroll Could this be merged, please? It is a pretty critical bug for us and we want to build a release candidate, soon.


Use run: labels on push/merge

As seen in @jbusecke's https://github.com/leap-stc/data-management/actions/runs/4953296291/jobs/8860631786, the current logic does not pick up existing labels on merged PRs, if the workflow is triggered by a push event.
This leads pangeo-forge-runner to try to run everything, which in turn results in:

Not using the custom job_names defined in the action
Due to the bug referenced in the inline comments in the action, naming collisions occur

The action triggered following merge of https://github.com/leap-stc/data-management/pull/33 hit this issue and no production jobs were deployed:
https://github.com/leap-stc/data-management/actions/runs/5686475068/job/15413308821#step:6:55

I see now in the Github docs here that GITHUB_HEAD_REF (i.e. github.head_ref)

"...is only available when the event that triggers a workflow run is either pull_request or pull_request_target."

which explains why it is empty in the linked action run here:

https://github.com/leap-stc/data-management/actions/runs/5686475068/job/15413308821#step:6:55

So we probably need to use GITHUB_REF or GITHUB_REF_NAME instead, which appear to be available on both pr and push events. Looking into that now...


Fixes #13: Added support for custom widget components.

refs #13
For now it only supports single input widgets, next step will be to provide object field ones; but let's iterate on this.
r=? @leplatrem

r+ (nicer with a bit of improvements in readme  IMO)


Vanillafix causes Buildcraft crash?

https://paste.dimdev.org/buwiyodubi.rb
and here's someone that has the same/similar and thinks its VanillaFix:
https://github.com/BuildCraft/BuildCraft/issues/4196

Duplicate of #73


wxGrid doesn't seem to work with screen readers used by visually impaired programmers (JAWS or NVDA)

Hi,
the wxGrid control seem to be inaccessible when I tried to use it both in my program & in the %wxGlade 1.0.5%\examples\AllWidgets\AllWigets_30.wxg [preview] mode.
Messing around with the properties of the control did not help.
Behavior: after I locate the wxGrid control I press  to get to it, then I can't do anything anymore.  Neither the [tab] nor the [arrow] keys give me any feedback.  I can't even get back to the notebook pages.  The only way out is to exit the [preview] mode.
Expected behavior: I should be able to move inside the grid cells with the [arrow] keys, as well as with [home], [end], [ctrl+home], [ctrl+end] & even [pgup] & [pgdn].
Will gladly provide more assistance/clarifications, should any be needed to resolve this issue.
Failing that, I'll entertain any work around you may offer.
Thanks in advance for your assistance.
Cheers,
@DietmarSchwertberger

In the last days I had a look at the wxWidgets sources. I think I will add accessibility support for wxGrid.
After implementation, it will still take some time until this finds its way into wxPython.

YEAH!!! 😊
Now that, is wonderful news!  Thanks a bunch for taking the time.  I make myself a spokesperson for the Visually impaired programmers community to send a big & warm appreciative wave down your neck of the woods😊
Because I’m not very familiar with the implementation cycle, I’m assuming we won’t be able to use your improvement until XwPython implements it at their end?  I just saw the issue getting closed so, I’m assuming your part is completed.  Is their an easy way for me to know when WxPython has completed theirs?
Thanks again,
Jean-Claude Provost
From: Dietmar Schwertberger @.>
Sent: Sunday, November 12, 2023 2:22 PM
To: wxGlade/wxGlade @.>
Cc: jcprovost @.>; Author @.>
Subject: Re: [wxGlade/wxGlade] wxGrid doesn't seem to work with screen readers used by visually impaired programmers (JAWS or NVDA) (Issue #534)
In the last days I had a look at the wxWidgets sources. I think I will add accessibility support for wxGrid.
After implementation, it will still take some time until this finds its way into wxPython.
—
Reply to this email directly, view it on GitHubhttps://github.com/wxGlade/wxGlade/issues/534#issuecomment-1807215785, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BB6EL7H5CB3GXSAOKS25MFLYEEOVNAVCNFSM6AAAAAA43FSQQWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBXGIYTKNZYGU.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

I have closed the ticket here as there is no work on wxGlade involved.
At the moment I'm learning by looking at the wxDataViewCtrl implementation, which supports accessibilty already.
Most wxWidget controls have accessibility support out of the box because they use native Windows controls.
wxGrid and wxDataViewCtrls are generic widgets, rendered by wxWidgets itself and therefore don't inherit the native support from Windows.
I think I will find the time to implement the wxWidgets changes at latest over Christmas vacation.
Once I have the wxWidgets modification, I plan to upload somewhere a version of the wxWidgets grid sample as executable file to get feedback from you and other users.
Once a PR is merged into wxWidgets, it will take some time for a wxWidgets release and until wxPython will use the new wxWidgets release.
So, don't hold your breath. Something like this is always a medium-term effort.

I understand…
If you could, please, make me aware of any progress.  I certainly can appreciate you have other fish to fry😊
Cheers,
Jean-Claude Provost
From: Dietmar Schwertberger @.>
Sent: Wednesday, November 15, 2023 1:16 PM
To: wxGlade/wxGlade @.>
Cc: jcprovost @.>; Author @.>
Subject: Re: [wxGlade/wxGlade] wxGrid doesn't seem to work with screen readers used by visually impaired programmers (JAWS or NVDA) (Issue #534)
I have closed the ticket here as there is no work on wxGlade involved.
At the moment I'm learning by looking at the wxDataViewCtrl implementation, which supports accessibilty already.
Most wxWidget controls have accessibility support out of the box because they use native Windows controls.
wxGrid and wxDataViewCtrls are generic widgets, rendered by wxWidgets itself and therefore don't inherit the native support from Windows.
I think I will find the time to implement the wxWidgets changes at latest over Christmas vacation.
Once I have the wxWidgets modification, I plan to upload somewhere a version of the wxWidgets grid sample as executable file to get feedback from you and other users.
Once a PR is merged into wxWidgets, it will take some time for a wxWidgets release and until wxPython will use the new wxWidgets release.
So, don't hold your breath. Something like this is always a medium-term effort.
—
Reply to this email directly, view it on GitHubhttps://github.com/wxGlade/wxGlade/issues/534#issuecomment-1813027700, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BB6EL7DC6J44NBOC7YKEKSTYEUBGJAVCNFSM6AAAAAA43FSQQWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJTGAZDONZQGA.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Work on this has started, but as of now the screen readers do not well support a grid -> row -> cell hierarchy.
Once this is clarified / fixed, I will proceed:
https://github.com/nvaccess/nvda/pull/16008

Hi there (a very happy new year to you & yours)
Not quite sure what you’re referring to here but it’s been my experience that screen readers can quite easily handle tables in Word, Excel, & on the web.  Perhaps you’re referring to something different?  I have to admit that, I’m not understanding all the intricacies mentioned in the NVDA issue referred to in your note…
Also, it is worth noting that NVDA & JAWS (the 2 main screen readers in the Windows environment may behave differently in a given situation…
In any case, if I can be of assistance to test, let me know.  I’ve been a screen reader user for more years than I care to count.  So, if I can put this to good use, I’ll gladly oblige.
Take care,
Jean-Claude Provost
From: Dietmar Schwertberger @.>
Sent: Saturday, January 6, 2024 9:51 AM
To: wxGlade/wxGlade @.>
Cc: jcprovost @.>; Author @.>
Subject: Re: [wxGlade/wxGlade] wxGrid doesn't seem to work with screen readers used by visually impaired programmers (JAWS or NVDA) (Issue #534)
Work on this has started, but as of now the screen readers do not well support a grid -> row -> cell hierarchy.
Once this is clarified / fixed, I will proceed:
nvaccess/nvda#16008https://github.com/nvaccess/nvda/pull/16008
—
Reply to this email directly, view it on GitHubhttps://github.com/wxGlade/wxGlade/issues/534#issuecomment-1879719081, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BB6EL7CWUROSM3URNV6JO7DYNFQF3AVCNFSM6AAAAAA43FSQQWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZZG4YTSMBYGE.
You are receiving this because you authored the thread.Message ID: @.***>

I have uploaded a first version to get feedback. See https://github.com/wxGlade/wxGlade/issues/539 if you want to support.

Hello good sir!
Took me a while, I know😊  I had some major interrupt that took me off of this project for a while…  In any case, I downloaded both grid_v1.exe & grid_v1_reverse.exe.  When trying to execute either of them, I received the following message:
“grid_v1.exe - System Error dialog
The code execution cannot proceed because VCRUNTIME140D.dll was not found. Reinstalling the program may fix this problem.”
Attempting to launch either application several times kept yielding the same result.
Let me know what I need to do…
Thanks,
Cheers,
Jean-Claude Provost
From: Dietmar Schwertberger @.>
Sent: Monday, January 29, 2024 3:43 PM
To: wxGlade/wxGlade @.>
Cc: jcprovost @.>; Author @.>
Subject: Re: [wxGlade/wxGlade] wxGrid doesn't seem to work with screen readers used by visually impaired programmers (JAWS or NVDA) (Issue #534)
I have uploaded a first version to get feedback. See #539https://github.com/wxGlade/wxGlade/issues/539 if you want to support.
—
Reply to this email directly, view it on GitHubhttps://github.com/wxGlade/wxGlade/issues/534#issuecomment-1915528177, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BB6EL7AJJDMVGRHBZF74PPDYRACVJAVCNFSM6AAAAAA43FSQQWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJVGUZDQMJXG4.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

OK, that means that the Visual C runtime is missing.
Could you please install the "Visual C++ Redistributable for Visual Studio 2015" from this link?
https://download.microsoft.com/download/9/3/F/93FCF1E7-E6A4-478B-96E7-D4B285925B00/vc_redist.x64.exe

Currently there is quite some backlog on wxWidgets pull requests. So, it's not yet too late to switch between the two variants.

Hithere,
Looks like the situation is weirder than we anticipated😊

Not sure it this happened before as well but I noticed there were other .dll’s missing: vcruntime1401d, vcruntime140d, msvcp140d 0crtbased vcruntime140_1d, msvcp140d…
As if this wasn’t enough, here’s the message I got when trying to execute “vc_redist.x64.exe” from the link you provided:
“
Microsoft Visual C++ 2015 Redistributable (x64) - 14.0.23026 Setup Microsoft Visual C++ 2015 Redistributable (x64) - 14.0.23026 Setup Failed One or more issues caused the setup to fail. Please fix the issues and then retry setup. For more information see the log file. 0x80070666 - Another version of this product is already installed. Installation of this version cannot continue. To configure or remove the existing version of this product, use Add/Remove Programs on the Control Panel.
“

When looking at my installed applications I saw « Microsoft Visual C++ 2015-2022 Redistributable (x64) - 14.38.33135” which seems to be more recent than the version you provided…
In case it should be useful, attached is the log file resulting from the installation attempt.  As you will see, Some entries in there are less than re-assuring😊Besides, the last log entry exactly confirms my findings…
Not quite sure what to do now???
Cheers,
Jean-Claude Provost
From: Dietmar Schwertberger @.>
Sent: Sunday, April 7, 2024 2:45 PM
To: wxGlade/wxGlade @.>
Cc: jcprovost @.>; Author @.>
Subject: Re: [wxGlade/wxGlade] wxGrid doesn't seem to work with screen readers used by visually impaired programmers (JAWS or NVDA) (Issue #534)
OK, that means that the Visual C runtime is missing.
Could you please install the "Visual C++ Redistributable for Visual Studio 2015" from this link?
https://download.microsoft.com/download/9/3/F/93FCF1E7-E6A4-478B-96E7-D4B285925B00/vc_redist.x64.exe
—
Reply to this email directly, view it on GitHubhttps://github.com/wxGlade/wxGlade/issues/534#issuecomment-2041558975, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BB6EL7GHPVWNUYUMTXABSGTY4GH3VAVCNFSM6AAAAAA43FSQQWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBRGU2TQOJXGU.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Oh, sorry, it seems that Microsoft does not distribute debug DLLs and I did build and upload with debug enabled.
I will try to re-build without debug.

Seems the easiest solution is to bundle the exe files with the DLLs.
Could you please download the zip archive with all files from https://sourceforge.net/projects/wxglade/files/wxWidgets/wxGrid_Accessibility/2023-01-29/grid_sample_2024-01-29.zip
Sorry for the confusion.

All right then!  So I assume I won't have to bother with the older vc_redist anymore?
let me know when I can download the 2 builds again.
Cheers,
Jean-Claude Provost
From: Dietmar Schwertberger @.>
Sent: Thursday, April 11, 2024 3:23 PM
To: wxGlade/wxGlade @.>
Cc: jcprovost @.>; Author @.>
Subject: Re: [wxGlade/wxGlade] wxGrid doesn't seem to work with screen readers used by visually impaired programmers (JAWS or NVDA) (Issue #534)
Oh, sorry, it seems that Microsoft does not distribute debug DLLs and I did build and upload with debug enabled.
I will try to re-build without debug.

Reply to this email directly, view it on GitHubhttps://github.com/wxGlade/wxGlade/issues/534#issuecomment-2050363897, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BB6EL7CNIZ74QXII4IZ2IMTY43PIVAVCNFSM6AAAAAA43FSQQWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJQGM3DGOBZG4.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Looks like we’ve got a winner here😊
There is a lot to say…  Probably more than an email communication method can handle decently😊
But I’ll mention a few things that jumped at me on 1st contact:

Not sure what the difference is supposed to be between the _reverse & the not reversed😊 but it seemed to me the reverse was closer than what is required…
There are significant behavior differences between NVDA & JAWS.  NVDA makes more sense though & that is sad, since JAWS is (I believe) still the most used screen reader…
After a few manipulations, the top cell where you say columns E to J are where the stuff is ended up disappearing with no way for me to retrieve it.  I had to recycle the application to do so…
Although I saw that there was a combobox & I saw “select a choicre” I was never able to activate said combobox.
With the _reverse app & NVDA, I ended up realizing the dates were like a calendar & that I could modify them.  But I not sure how I ended up there…
I’d be inclined to make the  key behave like the  key to go into edit mode for a cell;
Finally, what would be column D is never spoken.  It says, “empty column” instead.
This is just the tip of the iceberg & I certainly think we ought to get together & I should interact with the app & tell you what’s happening.  Because now, it seems to me I’m not seeing everything you intended to show…

I think a FaceTime (or something else to provide us with an interactive communication mode) could be appropriate.  & perhaps you could even look at my machine to really see how it behaves inside the grid…
All that being said, it is certainly a giant step forward since, I could see nothing using the original version of the grid in WxWidget😊
Let me know what you think…  & once again, thanks for taking the time…
Cheers,
Jean-Claude Provost
From: Dietmar Schwertberger @.>
Sent: Thursday, April 11, 2024 4:00 PM
To: wxGlade/wxGlade @.>
Cc: jcprovost @.>; Author @.>
Subject: Re: [wxGlade/wxGlade] wxGrid doesn't seem to work with screen readers used by visually impaired programmers (JAWS or NVDA) (Issue #534)
Seems the easiest solution is to bundle the exe files with the DLLs.
Could you please download the zip archive with all files from https://sourceforge.net/projects/wxglade/files/wxWidgets/wxGrid_Accessibility/2023-01-29/grid_sample_2024-01-29.zip
Sorry for the confusion.
—
Reply to this email directly, view it on GitHubhttps://github.com/wxGlade/wxGlade/issues/534#issuecomment-2050423444, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BB6EL7HX77IDX5YAXRHZ3ODY43TSZAVCNFSM6AAAAAA43FSQQWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJQGQZDGNBUGQ.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Great.
So, I will modify the wxWidgets pull request towards the _reverse version.
With JAWS I did initially some testing, but without a license that's difficult. I think that NVDA now has roughly the same market share.
About the top cell: well, it's editable. Could be that e.g. pressing the space bar did overwrite the contents. You still  should have gotten a row and column position information, though.
It seems you are using the space bar to start editing. For most cases, you should press F2 instead. That's what als MS Excel is using and all my applications. With a text cell, the contents are selected already after pressing F2. For many contents it's also possible to just start typing.
About the choice: This one can actually be activated with space or F2 or with any key.
Same is true for the dates and other non-text content. Maybe there is a need to add some information that the cell is a date. The values "Today" and "Tomorrow" are actually the values that the grid renderers are returning. I'm not a fan. The visual display is e.g. "04/12/24", not "Today".
The "empty column" at position D is correct. The column header is "Empty Column" and none of the cells has a content.
I don't think that the grid will ever be fully accessible for any type of content. There are too many types of content and even pure text has it's problems if the text is longer than what's actually displayed.
Specialized support for the available editors and renderers is out of scope for me.
For e.g. wxGlade where the grids are just two columns "name" and "value", this will probably work well, though.
When wxPython will be updated to a more recent wxWidgets version, it should be easier to implement application specific accessibility support, as https://github.com/wxWidgets/Phoenix/pull/2515 can be merged.
Please test with the infos above and I will also look again into the date cell behaviour.
If required, we could have a call e.g. on Sunday. Are you also in the CEST time zone?

Hi,
I agree, I hadn’t thought about Excel & f2…  For the combobox though, the reflex is to use alt- in both JAWS & NVDA would only let you edit the value, not get to the choices.  & for those, NVDA doesn’t always honor them which is one of the reasons I use JAWS...
For the JAWS license, you’re right.  & in spite of what they’re pretending, the demo mode doesn’t always behave like the licensed version, even within the 40 minutes period…  if it’s going to help, maybe we can do something…  I have a 2022 version for which I could give you a key & you’d be able to test at your leisure…  it’s not the latest but it should float your testing boat easily…
Something else I can propose if you want.  You could create a GITHub link of the latest Zipped version you sent me & I can send it to a blind programmers list.  In so doing, you’d get more feedback from folks who have more knowledge & perhaps better suggestions…  I don’t want to force anything on you though.  I’ll do it only if you agree…
Just for fun, a couple of other things I’m noticing when trying the _reverse again today😊

When navigating the grid with the arrow keys, I think that the row should only be mentioned when we change it, navigating vertically.  Same goes for the columns.  So for example, moving from row2 col2 to row2 col3 should not restate the row.  Moving from row2 col2 to row3 col2 should not restate the column.
I could never see the content of the cells.  I could only hea the row & column info.  The only way was for me to hit a key & then, I could see what the date was.  In other words it seems like we can’t quite browse the content.  I had to edit every cell to see its value.
For the dates, I could see it was a dropdown only after I was in edit mode.  It would me more fluid if we could see both the content of the cells & whether they are editable and/or whether they are of the dropdown/combobox type…
I like the fact that, for editable cells, you have the option to start typing which overwrites the whole maryanne & when you press f2, then you can edit without destroying everything.  Don’t know if everybody would feel the same though.  One thing is for sure.  That’ll have to be documented if we want to avoid frustration😊

I think I had a different behavior yesterday & I’m not quite sure what I could have done to cause this to change…  That’s where perhaps, an interactive session will help us…
as far as the timezone is concerned, I’m in Qc Canada, so EDT (GMT-4)  & yes, Sunday would be fine.  Just give me a time and an app & we’ll do it…
Take care,
Jean-Claude Provost
From: Dietmar Schwertberger @.>
Sent: Friday, April 12, 2024 12:32 PM
To: wxGlade/wxGlade @.>
Cc: jcprovost @.>; Author @.>
Subject: Re: [wxGlade/wxGlade] wxGrid doesn't seem to work with screen readers used by visually impaired programmers (JAWS or NVDA) (Issue #534)
Great.
So, I will modify the wxWidgets pull request towards the _reverse version.
With JAWS I did initially some testing, but without a license that's difficult. I think that NVDA now has roughly the same market share.
About the top cell: well, it's editable. Could be that e.g. pressing the space bar did overwrite the contents. You still should have gotten a row and column position information, though.
It seems you are using the space bar to start editing. For most cases, you should press F2 instead. That's what als MS Excel is using and all my applications. With a text cell, the contents are selected already after pressing F2. For many contents it's also possible to just start typing.
About the choice: This one can actually be activated with space or F2 or with any key.
Same is true for the dates and other non-text content. Maybe there is a need to add some information that the cell is a date. The values "Today" and "Tomorrow" are actually the values that the grid renderers are returning. I'm not a fan. The visual display is e.g. "04/12/24", not "Today".
The "empty column" at position D is correct. The column header is "Empty Column" and none of the cells has a content.
I don't think that the grid will ever be fully accessible for any type of content. There are too many types of content and even pure text has it's problems if the text is longer than what's actually displayed.
Specialized support for the available editors and renderers is out of scope for me.
For e.g. wxGlade where the grids are just two columns "name" and "value", this will probably work well, though.
When wxPython will be updated to a more recent wxWidgets version, it should be easier to implement application specific accessibility support, as wxWidgets/Phoenix#2515https://github.com/wxWidgets/Phoenix/pull/2515 can be merged.
Please test with the infos above and I will also look again into the date cell behaviour.
If required, we could have a call e.g. on Sunday. Are you also in the CEST time zone?
—
Reply to this email directly, view it on GitHubhttps://github.com/wxGlade/wxGlade/issues/534#issuecomment-2052090402, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BB6EL7HBHMCXOSHSOPEZKXDY5AEAPAVCNFSM6AAAAAA43FSQQWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJSGA4TANBQGI.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

I have now installed JAWS again and it's still a pain to use in the demo mode as it often requires re-boots and, even worse, the text viewer is much less useful for me than the NVDA Speech Viewer.
For developing I'm usually only using the Speech Viewer, not the actual speech output.
I have also tried JAWS with the DataViewCtrl, which is the other generic wx widget that has accessibility support. With JAWS there is quite some mismatch between the text viewer and what is actually spoken. NVDA's support is much more complete, I think. The JAWS speech output misses most of the contents, as you will only hear the contents from column 1.
When you are using JAWS, dou you also have a braille output?
Accessible widgets are queried via MSAA for Name and Value. It seems that JAWS does only take into account the Name field, not the value. This is the root cause for your observation 2.
NVDA takes both fields. Therefore with NVDA the difference between the normal and the reverse exe is whether the coordinates or the content is reported first. With JAWS either the coordinates or the contents are spoken.
I would consider this a bug. Unfortunately, there is no technical documentation on the Freedom Scientific website.
When I started working on the grid, I tried to contact their technical support, but never got an answer. For a commercial software that is far below from what I expect...
At the moment, the only way to support JAWS would be to pack coordinates and values together into the Name field.
About your point 1, I could store the last reported coordinate and then omit the row or column if it has not changed.

I think there are ways in which JAWS could be more useful in those respects.  & if your boat is not floated a more comprehensive output could be provided quite easily, using JAWS script language.  Not quite sure what you're referring to with "text viewer" but JAWS History (JAWS+H should provide you with what you deserve in terms of speech output, unless I'm missing something...  As far as JAWS support, my suggestion about the blind programmers list may come handy here.  There are some very knowledgeable guys on there & I'm quite certain they could be of assistance.
The issues surrounding the use of the demo mode are exactly why I offered you JAWS2022.  I'm interested in providing you with the best JAWS facilitations as in spite of NVDA seeming to offer better support, I think JAWS is still the best option for power usage...
Cheers,
Jean-Claude Provost
From: Dietmar Schwertberger @.>
Sent: Sunday, April 14, 2024 7:56 AM
To: wxGlade/wxGlade @.>
Cc: jcprovost @.>; Author @.>
Subject: Re: [wxGlade/wxGlade] wxGrid doesn't seem to work with screen readers used by visually impaired programmers (JAWS or NVDA) (Issue #534)
I have now installed JAWS again and it's still a pain to use in the demo mode as it often requires re-boots and, even worse, the text viewer is much less useful for me than the NVDA Speech Viewer.
For developing I'm usually only using the Speech Viewer, not the actual speech output.
I have also tried JAWS with the DataViewCtrl, which is the other generic wx widget that has accessibility support. With JAWS there is quite some mismatch between the text viewer and what is actually spoken. NVDA's support is much more complete, I think. The JAWS speech output misses most of the contents, as you will only hear the contents from column 1.
When you are using JAWS, dou you also have a braille output?
Accessible widgets are queried via MSAA for Name and Value. It seems that JAWS does only take into account the Name field, not the value. This is the root cause for your observation 2.
NVDA takes both fields. Therefore with NVDA the difference between the normal and the reverse exe is whether the coordinates or the content is reported first. With JAWS either the coordinates or the contents are spoken.
I would consider this a bug. Unfortunately, there is no technical documentation on the Freedom Scientific website.
When I started working on the grid, I tried to contact their technical support, but never got an answer. For a commercial software that is far below from what I expect...
At the moment, the only way to support JAWS would be to pack coordinates and values together into the Name field.
About your point 1, I could store the last reported coordinate and then omit the row or column if it has not changed.

Reply to this email directly, view it on GitHubhttps://github.com/wxGlade/wxGlade/issues/534#issuecomment-2054023831, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BB6EL7FQP7ARLJMASNFUWOLY5JVDNAVCNFSM6AAAAAA43FSQQWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJUGAZDGOBTGE.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

In the next days I will prepare a new grid sample with improved navigation, i.e. row and column information will not be repeated.
This could then be posted to the mailing list. Maybe, you want to post there already whether someone has experience with MSAA and how the attributes are interpreted by the screen readers.
But honestly, I see it in the responsibility of Freedom Scientific to provide support. JAWS is not cheap. I got better support for cheaper software.
The JAWS script language is out of scope for me and for a generic tool I don't see the point. It would probably have to be adapted for each and every program using wxGrid.

How well does this one behave?
https://sourceforge.net/projects/wxglade/files/wxWidgets/wxGrid_Accessibility/2024-04-14/grid_64bit_2024-04-14.exe
This one combines coordinates and value into the Name property and it does report only the changed row/col information.
I'm not sure: should the row/col information be dropped only if the user moved to a neighbor cell?

@jcprovost : Did you find time to check the new version?
https://sourceforge.net/projects/wxglade/files/wxWidgets/wxGrid_Accessibility/2024-04-14/grid_64bit_2024-04-14.exe
This one combines coordinates and value into the Name property and it does report only the changed row/col information.
I'm not sure: should the row/col information be dropped only if the user moved to a neighbor cell?


Nested InlinePanel fails to save when creating >=2 items

Issue Summary
When adding new objects within nested InlinePanels, the form fields of the grandchild objects are not allocated unique IDs / names, leading to failures in initialising JS and saving.
Steps to Reproduce

Start a new project with wagtail start eventdemo
Edit home/models.py as follows:from django.db import models

from modelcluster.fields import ParentalKey
from modelcluster.models import ClusterableModel
from wagtail.core.models import Page, Orderable
from wagtail.admin.edit_handlers import FieldPanel, InlinePanel

class Event(Orderable):
    venue = ParentalKey('Venue', related_name='events', on_delete=models.CASCADE)
    name = models.CharField(max_length=255)
    date = models.DateField()

    panels = [
        FieldPanel('name'),
        FieldPanel('date'),
    ]

class Venue(ClusterableModel, Orderable):
    page = ParentalKey('HomePage', related_name='venues', on_delete=models.CASCADE)
    name = models.CharField(max_length=255)

    panels = [
        FieldPanel('name'),
        InlinePanel('events', label="Events"),
    ]

class HomePage(Page):
    content_panels = [
        InlinePanel('venues', label="Venues"),
    ]

./manage.py makemigrations, ./manage.py migrate, ./manage.py createsuperuser, ./manage.py runserver 0:8000
Log in to admin, edit the homepage and add a venue and two events. Note that the second date field does not provide a date picker.
Click 'save draft'. Note that only one of the events has been saved.

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: yes

The root cause of both issues is in the mechanism for adding child objects to the form: the HTML fragment to be inserted is stored in the page as a

Technical details

Python version: 3.6.8
Django version: 2.2.6
Wagtail version: Git master (2.8a0)
Browser version: Chrome 79.0.3945.117

I was looking into this and tried with regex parsing only the first __prefix before giving up. Any thoughts on how to monkey patch this or how a working solution could look like?

I can confirm I am suffering the same problem and that the solution of @thaxy worked for me.

The solution from @thaxy worked for me as well, but I had to add .replace(new RegExp(``${pre}\\\\u002D__prefix__``, 'g'), ``${pre}\\u002D${formCount}``). Without that line, a RichTextField wouldn't be added to the form correctly.

should the script from @thaxy be placed in the root app or in the app where I've registered the page w/ the nested inline panels?

@robicano22 Hey, I think i've ran into this issue the same time as you. I've added @thaxy's file inside the static folder of my app. For example /<your_app_name>/static/wagtailadmin/js/expanding_formset.js. This seems to override the js file the wagtailadmin static folder.

@robicano22 Hey, I think i've ran into this issue the same time as you. I've added @thaxy's file inside the static folder of my app. For example /<your_app_name>/static/wagtailadmin/js/expanding_formset.js. This seems to override the js file inside the wagtailadmin static folder.

@TabishB  thanks for reaching out. Excuse my ignorance on this, but I still can't override the expanding_formset function.
I've placed @thaxy 's snippet in the source app for the wagatil admin, as well as in the app where I've defined the models with the nested inline panels, at the proper directory path (see below), but the issues persist.
/<admin-source-app>/static/wagtailadmin/js/expanding_formset.js
/<app-with-nested-inlines>/static/wagtailadmin/js/expanding_formset.js
The console.log() statement does not show in the browser either.  Any thoughts on that?

Hey @robicano22 in my case I created an extra "wagtail_hacks" app and put it before any wagtail applications in INSTALLED_APPS. In "wagtail_hacks" folder you simply place @thaxy 's file i.e. "wagtail_hacks/static/wagtailadmin/js/expanding_formset.js"

+1 on the modifications from @thaxy and @the5thbeatle, though I had to edit it to .replace(new RegExp(``${pre}\\\\u002D__prefix__``, 'g'), ``${pre}\\u002D${formCount}``) added after the first RegExp line from @thaxy's script.


UI blocks on selecting ecosystem and associated warning are logged

When selecting "iceland shelf/sea" in first page of fin-casey.github.io/wizard , the page appears unresponsive. Also, when the ecosystem is selected, the following warning is logged in the browser console -
jquery.min.js:4 [Deprecation] Synchronous XMLHttpRequest on the main thread is deprecated because of its detrimental effects to the end user's experience. For more help, check https://xhr.spec.whatwg.org/.

Suggest to implement asynchronous requests for all wizard interactions.
@FIN-casey please let know if you have questions about this.

When all the requests are in asynchronous, depends on the amount of data, the display of data will not appear immediately after going to next step.
For example, when the user is using the wizard for the first time and he selected "Iceland shelf/sea" and he proceeded to the next step, the functional group table won't display the contents immediately because it is still processing in the background.
I implement all the request in asynchronous, i think we need to add new text/message that will notify the user that the request is being processed in the background.
Please test and verify.

@agruss2, can you do the test to see if there is an impact in the user experience?
Thanks

@FIN-casey I ran several tests on different computers, using different web browsers, and I did not face the issue mentioned by @jhpoelen. I suggest we close this issue, if that's good for your guys. Many thanks.

Thanks @FIN-casey for implementing the asynchronous calls. I think it would be wise to include status messages (e.g., "accessing [Iceland Sea/Shelf] configuration ...") and only enable the "proceed" button if a selection has been made and a configuration is available.

@agruss2 Is above item from Jorrit the remaining task for this issue? Please confirm for @FIN-casey. Thanks!

@FIN-JBarile @FIN-casey I ran several tests on different computers, using different web browsers, and I did not face the issue mentioned above. Therefore, I am deciding to close the present GitHub isue. Many thanks.


Plugin Causes Atom Editor to Lock Up

Sorry for the vague issue description, but there were no errors displayed in Atom Developer Console pane (CMD-ALT-I).
I can be editing in a file, and type "_" in the middle of a variable, and Atom hangs. I am able to reproduce this by inserting an underscore after the "t" in "firstName":
@if ($profile->last_name || $profile->firstName)

After I disable or uninstall the plugin, I can edit without hangups.

I have the same experiences. Atom freezes for 2 - 15 seconds.
Occurrences, in my case, are much more sporadic, though. I have yet to establish a concrete pattern, but it is definitely this package doing it. Turning the extension off and I never experience the freezes. Tested it over 20 hours.
Windows 7 Professional SP1
Atom 1.18.0
atom-autocomplete-php 0.25.6


Trolli (Trello Clone) Tutorial Incomplete and Corresponding Github example broken.

Description
The Trolli tutorial appears to be incomplete. I think the TrelloApp class is missing its UserControl parent and also its build() method. I also visited the corresponding github repo for this app https://github.com/flet-dev/examples/tree/main/python/apps/trolli which, upon running main.py after creating a new virtual environment and installing the project requirements, I encountered an error on the create_new_board implementation where the Board() is receiving a None type for its first arg instead a control.
Code example to reproduce the issue:
Follow this first section:
https://flet.dev/docs/tutorials/trello-clone#defining-entities-and-layout
Describe the results you received:
AttributeError: 'TrelloApp' object has no attribute '_build_add_commands'
Describe the results you expected:
According to the tutorial:
"we can see the result and get hot reloading when we make any style changes. For example, try adding alignment="center" to the first row in the container like this…"
Flet version (pip show flet):
Version: 0.20.0
Give your requirements.txt file (don't pip freeze, instead give direct packages):
anyio==4.2.0
arrow==1.3.0
binaryornot==0.4.4
certifi==2024.2.2
chardet==5.2.0
charset-normalizer==3.3.2
click==8.1.7
cookiecutter==2.5.0
flet==0.20.0
flet-core==0.20.0
flet-runtime==0.20.0
h11==0.14.0
httpcore==0.17.3
httpx==0.24.1
idna==3.6
Jinja2==3.1.3
markdown-it-py==3.0.0
MarkupSafe==2.1.5
mdurl==0.1.2
oauthlib==3.2.2
packaging==23.2
Pygments==2.17.2
pypng==0.20220715.0
python-dateutil==2.8.2
python-slugify==8.0.4
PyYAML==6.0.1
qrcode==7.4.2
repath==0.9.0
requests==2.31.0
rich==13.7.0
six==1.16.0
sniffio==1.3.0
text-unidecode==1.3
types-python-dateutil==<IP_ADDRESS>40106
typing_extensions==4.9.0
urllib3==2.2.0
watchdog==3.0.0
websocket-client==1.7.0
websockets==11.0.3
Operating system:
macOS

The issue that I mentioned regarding the broken Github example for this app is the same as this: https://github.com/flet-dev/examples/issues/102

Also related: https://github.com/flet-dev/flet/issues/1553

You received this error ( AttributeError: 'TrelloApp' object has no attribute '_build_add_commands'), cause the TrelloApp class is missing UserControl class.
first import UserControl
change this (class TrelloApp:) to ( class TrelloApp(UserControl) ):

and in case you download the source code and get an error same as the one in live demo like this one below...
=>There was an error while processing your request: 'NoneType' object have no attribute 'width'.
here is what I do..
=> First, I put app.initialize() above page.add(app) my main function look like the one below
def main(page: Page):
    page.title = "Flet Trello clone"
    page.padding = 0
    page.theme = theme.Theme(font_family="Verdana")
    page.theme.page_transitions.windows = "cupertino"
    page.fonts = {"Pacifico": "Pacifico-Regular.ttf"}
    page.bgcolor = colors.BLUE_GREY_200
    app = DriverApp(page, InMemoryStore())
    app.initialize()
    page.add(app)
    page.update()

=> secondly I comment build function and initialize self.layout in init function like the code below
class TrelloApp(UserControl):
    def __init__(self, page: Page, store: DataStore):
        super().__init__()
        self.page = page
        self.store: DataStore = store
        self.page.on_route_change = self.route_change
        self.boards = self.store.get_boards()
        self.login_profile_button = PopupMenuItem(text="Log in", on_click=self.login)
        self.layout = AppLayout(
            self,
            self.page,
            self.store,
            tight=True,
            expand=True,
            vertical_alignment="start",
        )
        self.appbar_items = [
            self.login_profile_button,
            PopupMenuItem(),  # divider
            PopupMenuItem(text="Settings"),
        ]
        self.appbar = AppBar(
            leading=Icon(icons.GRID_GOLDENRATIO_ROUNDED),
            leading_width=100,
            title=Text(f"Driver", font_family="Pacifico", size=32, text_align="start"),
            center_title=False,
            toolbar_height=75,
            bgcolor=colors.LIGHT_BLUE_ACCENT_700,
            actions=[
                Container(
                    content=PopupMenuButton(items=self.appbar_items),
                    margin=margin.only(left=50, right=25),
                )
            ],
        )
        self.page.appbar = self.appbar
        self.page.update()

    # def build(self):
    #     self.layout = AppLayout(
    #         self,
    #         self.page,
    #         self.store,
    #         tight=True,
    #         expand=True,
    #         vertical_alignment="start",
    #     )
    #     return self.layout

=> after changing everything like that the code worked, I released the NoneType error was rised because self.page was not being initialized

You received this error ( AttributeError: 'TrelloApp' object has no attribute '_build_add_commands'), cause the TrelloApp class is missing UserControl class. first import UserControl change this (class TrelloApp:) to ( class TrelloApp(UserControl) ):

In the latest release notes (dated 6. March 2024) it states that the UserControl class has been deprecated. https://flet.dev/blog/flet-fastapi-and-async-api-improvements/#custom-controls-api-normalized
So anyone installing afterwards, this tutorial, and this fix will not work, however I am one of those people so we are all still waiting for the fix.

You received this error ( AttributeError: 'TrelloApp' object has no attribute '_build_add_commands'), cause the TrelloApp class is missing UserControl class. first import UserControl change this (class TrelloApp:) to ( class TrelloApp(UserControl) ):

In the latest release notes (dated 6. March 2024) it states that the UserControl class has been deprecated. https://flet.dev/blog/flet-fastapi-and-async-api-improvements/#custom-controls-api-normalized
So anyone installing afterwards, this tutorial, and this fix will not work, however I am one of those people so we are all still waiting for the fix.

I will look to that, if I come up with any solution for that, I will post my solution here

Can you please give the code another try? It has been updated.


Configurations via Environment Variables not precedent over configs in the database

Bug Report
Installation Method
open-webui Helm chart via ArgoCD on OpenShift.
Environment

Open WebUI Version: v0.4.6

Confirmation:

[X] I have read and followed all the instructions provided in the README.md.
[ ] I am on the latest version of both Open WebUI and Ollama.
[ ] I have included the browser console logs.
[X] I have included the Docker container logs.
[X] I have provided the exact steps to reproduce the bug in the "Steps to Reproduce" section below.

Expected Behavior:
Configurations via environment variables should take precedence over state in database.
Actual Behavior:
Once a configuration has been set and stored in the database by interacting with the admin panel, they take priority from then on and cannot be configured via environment variables.
Description
Bug Summary:
When setting configurations via environment variables, persisted configurations in the database take precedence.
Generally, when it comes to configuring applications, environment variables take precedence over saved state at startup since these allow for managing an application programmatically.  [https://12factor.net/config]
Reproduction Details
Steps to Reproduce:

Install and configure open-webui via container and environment variables
Interact with the admin panel and change a setting
Restart container

Logs and Screenshots
Docker Container Logs:
Fresh Install:
INFO [alembic.runtime.migration] Context impl PostgresqlImpl.
INFO [alembic.runtime.migration] Will assume transactional DDL.
Interacted with admin panel:
INFO [alembic.runtime.migration] Context impl PostgresqlImpl.
INFO [alembic.runtime.migration] Will assume transactional DDL.
INFO [open_webui.env] 'ENABLE_OLLAMA_API' loaded from the latest database entry
INFO [open_webui.env] 'OLLAMA_BASE_URLS' loaded from the latest database entry
INFO [open_webui.env] 'OLLAMA_API_CONFIGS' loaded from the latest database entry
INFO [open_webui.env] 'ENABLE_OPENAI_API' loaded from the latest database entry
INFO [open_webui.env] 'OPENAI_API_KEYS' loaded from the latest database entry
INFO [open_webui.env] 'OPENAI_API_BASE_URLS' loaded from the latest database entry
INFO [open_webui.env] 'OPENAI_API_CONFIGS' loaded from the latest database entry
INFO [open_webui.env] 'DEFAULT_LOCALE' loaded from the latest database entry
INFO [open_webui.env] 'DEFAULT_PROMPT_SUGGESTIONS' loaded from the latest database entry

Ooof, gotcha. It doesn't follow general industry patterns and it doesn't seem to be well documented.
In the docs it states:

The following environment variables are used by backend/config.py to provide Open WebUI startup configuration. Please note that some variables may have different default values depending on whether you're running Open WebUI directly or via Docker.

It might be best to state that ...to provide Open WebUI with an initialization configuration

@justbert Agreed, there are some env vars that don't get saved to the database which might add to the confusion. I'm open to accepting PR for this (both the code, and the doc)!

Sounds good! Just to clarify, in regards to a PR, would this be to update the docs or to re-align functionality to use set Env Vars as precedent?

Oh! I hadn't seen the RESET_CONFIG_ON_START env var. This should do nicely!


Return the hash code

Hi,
This is a very good library and implementation of LSH. Thanks for the contribution.
Sorry, I am new to this library. May I ask is there a way to return the hashcode from:
lsh = LSHash(6, 8)
Kindly regards,
Lu

Hi,
I also feel this is a good library, and help me to build applications using this library. thank you.
I also wonders how we could know the hashcode from index we input into the library.
or maybe we could use this code to retrieve all the hashcode:
In[12]: lsh.hash_tables[0].keys()
Out[12]: dict_keys(['110000', '110001', '111001', '111000'])
Regards.

Hello guys!
I've just found this library and I've been playing around with it. I hope it is not too late to join this discussion!
I had this same question, and I'm happy to share the solution I came out with (keep in mind that I'm not 100% sure if this is the right way of doing this, and if the value indeed corresponds to what you desire).

# Given that you have one instance
lsh = LSHash(6, 8)

# and theres something indexed
v = [1, 1, 2, 3, 5, 6]
self.lsh.index(v)

# The hash representation for `v` is stored in
binary_hash = lsh._hash(self.lsh.uniform_planes[0], v)

Maybe we could have an interface to this, something like get_index_hash(v), that checks if the vector is indexed and returns the binary hash if it is; or raises an exception if not.

Thank you for the suggestions! I think the get_index_hash(v) makes sense
@gAldeia could you please submit a PR? Thanks!

and

thank you, if you find the solution by @gAldeia fine, I would be glad to merge a PR.

@loretoparisi I just implemented this feature in my PR #7 ! Sorry it took so long.


Keep alternate file when moving up a directory

To replicate simply:

edit a file
switch to lir
navigate to a directory
check that original file has alternate file status (#)
navigate back to previous directory
notice that original file has lost # (alternate file status)

This is a bug.
Thanks for creating the PR.
Also, thanks for writing the detailed procedure.


Add ability to handle DescriptorUpdateTemplate extension to unique objects layer

The UniqueObjects utility layer needed to unwrap vulkan object handles embedded in a buffer parameter of vkUpdateDescriptorSetWithTemplateKHR().  This patch series parses that buffer, replacing the uniquified handles with the actual ones before passing the call to the hardware.  This takes care of part of issue #1529.

Excellent review feedback incorporated, new version now in place.


wpa_supplicant_gui: fixup build after update in PR #155266

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

The change in the patch has been included in the release, so it was now failing, e.g. https://hydra.nixos.org/build/164879658

I'm impatient
@GrahamcOfBorg build wpa_supplicant_gui

Let's get a back-link from PR #155266

🤔 I thought that maintainers get pinged automagically.


Need to open up map limits to include whole globe.

Currently, the map is constrained to the US. We need it to open up to include the whole world. Look here: https://cida.usgs.gov/gdp/client/#!advanced/spatial and observe that you can't get to Europe or Asia or the southern hemisphere.

I assume this is setting it here:
https://github.com/USGS-CIDA/gdp_web/blob/master/src/main/webapp/js/util/mapUtils.js#L46

Although it seems like only the latitude values from there are being enforced (10, 75) on the map, it doesn't reach the longitude bounds.

I think that using those bounds for the restrictedExtent is what we want to avoid, so switching this to maxExtent is probably what we want.


HARP-7551: Updates examples to use our webpack utils

- Convert helper script to javascript to greatly simplify CI
- Allow users to specify own entry point names
- Do not hardcode location of map themes
- Do not override user typescript module rules

Codecov Report

Merging #1167 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1167   +/-   ##
=======================================
  Coverage   47.37%   47.37%
=======================================
  Files         256      256
  Lines       22754    22754
  Branches     5065     5065
=======================================
  Hits        10780    10780
  Misses      11974    11974

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a77f876...300d777. Read the comment docs.

Closing this for now, to proceed will need to redesign "addHarpWebpackConfig".


People 125 user skills page

Ticket
https://netguru.atlassian.net/browse/PEOPLE-125
Description

[x] Added skills view for users
[x] Allow users to rate their skills
[x] Allow users to add note
[x] Allow users to set target
[x] Added generator for user skill rates
[x] Added more fields for user skill rates

Before merge checklist

[x] CircleCI is passing
[x] Proper labels are set

I don't see any tests for the new frontend functionality, neither js nor feature ones…
@DeeDee88 will be adding feature specs. Only that we have here.

well, it does seem like a good time to start then, doesn't it?

She will add them in a separate branch, after merging this one.

I have created new JIRA ticket for configuration JavaScript tests and add them: https://netguru.atlassian.net/browse/PEOPLE-133. @mamut can I merge current solution?


Allow modules to be updated from multiple systems

Summary of the new feature / enhancement
As a user I want to be able to update modules that are stored in my profile (or a network share) from multiple computers. Many places/companies are migrating to being able to back up my profile to OneDrive, so my modules in stored in  c:\Users\{UserName}\Documents\WindowsPowerShell\Modules (for Windows PowerShell) and c:\Users\{UserName}\Documents\PowerShell\Modules (for PowerShell core) and this is great for documents, but not for modules.
If I install a module, let's use the Az module, on PC1 using Install-Module -Name Az I can update the module with no issues on PC1, but let's say I also use a different machine called PC2.  On PC2, I have access to these modules because they are in my profile made available via OneDrive, and there is no issue in using them.  However, there is an issue when try to run Update-Module Az we get the error "Module 'az' was not installed by using Install-Module, so it can't be updated", but the module was installed using Install-Module, just on PC1.
Proposed technical implementation details (optional)
What I would like to see implemented in Install-Module is the ability to mark this module as being able to update from any location, regardless of where it was originally installed with the Install-Module cmdlet. An example of this might look something like this:
Install-Module -Name Az -Scope CurrentUser -AllowUniversalUpdates
The "AllowUniversalUpdates" would then mark the module as being able to be updated from any machine, as long as the user has write permissions to it.  This would also work out to being able to host modules on a network share in addition to just a OneDrive location.

Thanks @tmmuessig for opening the issue, you reference both Windows PowerShell and PowerShell core, can you clarify which version of PowerShell you are experiencing this with/using...also wanted to mention that right now we are only making changes to PSResourceGet so if you could test out the behavior with that module as well to clarify the experience you are looking for that would be great


"does not have mandatory providers: 'TemplateVariableInfo'" error

I was playing with this toolchain and I was trying to use the toolchain as part of genrule toolchains:
genrule(
    name = "test",
    outs = ["out.txt"],
    cmd = "echo foo > $@",
    toolchains = [
        "@zig_sdk//toolchain:linux_amd64_gnu.2.38",
    ],
)

When I try to build the target, I get:
ERROR: /src/workspace/BUILD:1:8: in toolchains attribute of genrule rule //:test: \
  '@@hermetic_cc_toolchain~~toolchains~zig_sdk//toolchain:linux_amd64_gnu.2.38' \
  does not have mandatory providers: 'TemplateVariableInfo'

I was expecting the toolchain to also declare TemplateVariableInfo so that the tools in the toolchain could be used via Makefile variables as well, like e.g. rules_bison does.

Also, is there a convenient way to select whatever the current toolchain is, similar to @rules_bison//bison:current_bison_toolchain?


Use FPS to standardize point count on the dataset

Closes #3

For now the code seems ok, still needs to be tested to ensure it works.


vectorized processing  for take

https://stackoverflow.com/questions/72735678/how-to-vectorize-arrowcomputetake
Is there a similar feature in Velox?

Hi @chakpongchung, it depends on what you intend to do with the results. In Velox you could do this zero-copy by wrapping input_array and take_array and creating a DictionaryVector, for instance. Could you explain a bit more about the use case?

Hi chakpongchung , Can you add more details here. What feature are you exactly looking for ?

Use Case: I am taking a subset of a really large input_array. It is used in places where OpenMP-like and MPI-like parallelism are used. So vectorization seems to be the next low-hanging fruit.
Context and use cases also added to the original StackOverflow post.

Is there an arrow-equivalent operation for this?

Arrow also supports dictionary encoded data:

https://arrow.apache.org/docs/format/Columnar.html#dictionary-encoded-layout

Closing it. Please re-open if you have additional questions.


ReactiveSystem gets wrong entities

Hi,
When GroupEventType.OnEntityRemoved is used with IReactiveSystem, List<Entity> entities parameter on method public void Execute(List<Entity> entities) doesn't correspond to the removed entities (seems to match random entities within the pool)
It would be useful to get the removed components
Regards

The real problem is destroyed entities (not just his components) will be reused to create another entity. It works like any component or entity instantiated will be used as a cache object to "instantiate" new ones if they are "destroyed".
GroupObservers (and by extension, ReactiveSystems...) WILL NOT notice this, they just think the entity not match his requirements so they keep anyways a reference of them (if they matched before)
That's why in the examples provided in sschmid acount (the match-one and the cars one) they built a "DestroySystem" which executes at the very end of the loop, destroying in that moment any entity with the component "DestroyComponent".

Thanks for your explanation.
For now, I won't use GroupEventType.OnEntityRemoved but only GroupEventType.OnEntityAdded.
Regards

Next release 0.22.0 will solve this issue.


fileName -> filename in the example

It looks like fileName should be filename. (If it's vice-versa then the opposite PR needs to be made so the examples are consistent.)
Fixes #1

Thanks! If the only point of it is is to pass the filename environment variable through to the local grader, and this is the same as the suggested filename that production would expose as filename, then maybe it's still useful to have filename and userId for local simulation or CI. So, I've edited this PR to just a little bit of documentation for those optional environment variables.

filename is NOT the suggested file provided in edit programming assignment page.
E.g. Lets say an author set the "index.py" as suggested file in programming assignment page. If a learner submits "random.py", coursera infrastructure renames "random.py" to "index.py"and then copies to /shared/submission folder for grading. We pass the learner's submitted filename ("random.py") as environment filename variable to the grader.  We will continue to pass this environment variable, although I am not sure how authors are using this information.
Hope this clarifies. Happy to answer if there any follow up questions.

Thanks for your reply. I've updated the PR to reflect my current understanding. Please let me know if what I've put is correct. (If you feel this isn't useful to publish on the official readme, you're free to simply close the PR instead of merging it.)


Add pre-commit to prevent someone committing directly to main

[x] This contribution adheres to CONTRIBUTING.md.

Adding a pre-commit hook to prevent someone mistakenly committing to main directly.

Git hooks belong in your local .git/hooks directory, not checked into .github/hooks. For example, if we were using the pre-commit Python package, each developer would run pre-commit install to install the Git hooks.
Does GitHub do something useful with Git hooks that I'm not aware of?

No, I just misread the Teams comment as .github. So we don't need to check anything in - it's just a local helper to remind a developer. Is that right?

No, I just misread the Teams comment as .github. So we don't need to check anything in - it's just a local helper to remind a developer. Is that right?

Right. Install it locally on every machine where you have the repo checked out.


DynamicMap's plot range does not get adjusted to stream

Hi,
I have a DynamicMap of Curve objects where each Curve is a timeseries of certain measurements (eg. Evaporation rate), and the kdims of the dynamic map is (lon, lat). So it's a dynamic map of timeseries at different locations in (lon,lat). As I manually change the parameters of the stream (ie lon, lat values), I noticed that my dynamic map's range doesn't get adjusted to the new timeseries at the new (lon,lat) location:
def get_tseries(xrd, varname, lon,lat):
    tseries = xrd[varname].sel(X=lon, Y=lat, method='nearest')
    return gv.Dataset(tseries, 'time', varname).to(hv.Curve)

LonLat = Stream.define('LonLat', X=22.35, Y=-11.45)
renamed_stream = LonLat().rename(X='lon', Y='lat')
# dmap_tseries = hv.DynamicMap(get_tseries, streams=[pt_stream.rename(X='lon', Y='lat')])
dmap_tseries = hv.DynamicMap(lambda lon,lat: get_tseries(xrd_ea, varnames[1], lon, lat),
                            streams=[renamed_stream])

display(dmap_tseries)

Send updated events:
for i,(lon,lat) in enumerate(zip(lons[::10],lats[::10])):
    if i>9:
        break
    print(i, "lon,lat: ", lon,lat)
    dmap_tseries.event(lon=lon, lat=lat)
    time.sleep(3)

Is this an expected behavior? I think it may be related to #3288. What would be a recommended way to set the ranges correctly?
Thank you!

I was able to resolve it by setting .opts(framewise=True). I'll close it.


[How-to guide]: modify config to add custom plugin

Hi folks 👋,
So I noticed that currently there is little guidance around developing and adding (or replacing existing ones with) custom plugins.
I mean the appsettings.json seems quite clear but I think some additional guidance would make it crystal clear.
Also do you plan to extend mgd with maybe some functionality, like a command, which will allow to install (setup) a custom plugin?
Thank you for your awesome work 👍

Good catch and definitely something we need to improve. Thank you for bringing it up @Adam-it!


fix(cloudtrail): get_trail_status must work with MultiRegion trails

Description
Fixes #6686
The get_trail_status function must return the status for MultiRegion trails.

@bblommers I want to talk with you privately, can you give me your email address or whatever way to want to?
Mine is<EMAIL_ADDRESS>Thanks!

Hi @jfagoagas, my profile (now) shows my email: https://github.com/bblommers


Accepting contributions?

I'd like to add some documentation, examples, maven central coordinates, etc.
@pohlposition are you accepting contributions?

@sandeepraju , I'd be open to contributions :)
This has been a side project of mine that I try to develop as I have time, so progress has been sporadic.
I haven't actually ever cut a release, so it's not on maven yet.  I think there is sufficient functionality now to do so.
I think the next thing for me to do is to:

Add some documentation, starting with a README
Include a release process and start publishing to maven central

If you have experience with leveraging jersey for secure https sessions, I'd love some feedback on this part.  It's not something I've done too much of in java, and I want to make sure it passes muster:
https://github.com/level11data/databricks-client-java/blob/master/src/main/java/com/level11data/databricks/client/DatabricksSession.java#L73

Sorry, looks like this slipped through the cracks. I haven't had much experience with Jersey but looks fine on an overview!


Add search result language support

Added language support for search results (https://github.com/opsdisk/yagooglesearch/issues/25) as SearchClient's lang_result parameter
Renamed SearchClient's lang parameter to lang_html_ui
Removed URL encoding of extra_params (https://github.com/opsdisk/yagooglesearch/issues/29)

This is awesome ... thanks so much @opsdisk for integrating this update for language-specific search results!  The code changes look great!


[BUG] build system - unnecessary re-compilation

Bug #534
Added a macro and utility cmake script to allow installing files with
correct dependency tracking during build commands added with
ExternalProject_Add_Step
Signed-off-by: JaredWright<EMAIL_ADDRESS>

What should this patch be preventing? If it's the driver build duplication I think the issue is that there is an execute_process in the driver's CMakeLists.txt.  I ran this PR on my machine, and after commenting the execute_process out ithe driver wasn't rebuilt.

This PR does not address the driver build duplication, maybe I should have named the bug slightly different. The behavior that this PR addresses is that all source files in the vmm, unit tests, bfm (basically anything that isn't the driver) re-compiles all source on every build. This was because external dependencies such as json, hippomocks, newlib, etc were being re-installed into the build "install" directory on every build, thus causing all components that depend on them to rebuild as well.
You are correct, the driver does rebuild itself every time because of the issue you described. I might suggest making seperate bug report for that one. Should I change the description of bug #539 to be more specific?

Whatever is easier - If the driver fix is only that one line you could probably just throw it in with this one but you can make it separate if you want

It will be a little more complicated that just commenting that line out, because that line is what causes the driver to build at all under the 'all' target. I think the real solution to driver-rebuilding is to remove the need for building the driver with an external build script (scripts/util/driver_build.sh). We should have cmake compile the driver project just like anything else, potentially using a driver-specific toolchain.

👍