[bug] series of repeated key presses crashes and unlocks

After monitors power off, if I press a key repeatedly as the monitors are powering up (say shift 10 times) screen turns red and then unlocks. I'm hoping that this is easily reproducible.
I'm using hyprlock 0.3.0, but the bug was present in the last release as well.
hypridle.conf:
general {
    lock_cmd = pidof hyprlock || hyprlock
    before_sleep_cmd = loginctl lock-session
    after_sleep_cmd = hyprctl dispatch dpms on
    ignore_dbus_inhibit = false
}

listener {
   timeout = 900
   on-timeout = loginctl lock-session
}

listener {
    timeout = 1200
    on-timeout = hyprctl dispatch dpms off
    on-resume = hyprctl dispatch dpms on
}

Not sure how helpful this stack trace is:
             Stack trace of thread 135738:
                #0  0x00007f3ba6eae834 __pthread_kill_implementation (libc.so.6 + 0x90834)
                #1  0x00007f3ba6e5c8ee raise (libc.so.6 + 0x3e8ee)
                #2  0x00007f3ba6e4498f abort (libc.so.6 + 0x2698f)
                #3  0x000055e2b3ec2505 _ZL20handleCriticalSignali (hyprlock + 0x29505)
                #4  0x00007f3ba6e5c9a0 __restore_rt (libc.so.6 + 0x3e9a0)
                #5  0x000055e2b3ec6ee5 _ZN19CSessionLockSurface6renderEv (hyprlock + 0x2dee5)
                #6  0x000055e2b3ec7c43 _ZN9CHyprlock5onKeyEjb (hyprlock + 0x2ec43)
                #7  0x00007f3ba6a14056 ffi_call_unix64 (libffi.so.8 + 0x9056)
                #8  0x00007f3ba6a105ef ffi_call_int.lto_priv.0 (libffi.so.8 + 0x55ef)
                #9  0x00007f3ba6a133fe ffi_call (libffi.so.8 + 0x83fe)
                #10 0x00007f3ba7422f0e wl_closure_invoke.constprop.0 (libwayland-client.so.0 + 0x7f0e)
                #11 0x00007f3ba7423793 dispatch_event.isra.0 (libwayland-client.so.0 + 0x8793)
                #12 0x00007f3ba7423a3c wl_display_dispatch_queue_pending (libwayland-client.so.0 + 0x8a3c)
                #13 0x000055e2b3ecb3da _ZN9CHyprlock3runEv (hyprlock + 0x323da)
                #14 0x000055e2b3eb0073 main (hyprlock + 0x17073)
                #15 0x00007f3ba6e4614a __libc_start_call_main (libc.so.6 + 0x2814a)
                #16 0x00007f3ba6e4620b __libc_start_main@@GLIBC_2.34 (libc.so.6 + 0x2820b)
                #17 0x000055e2b3eb1985 _start (hyprlock + 0x18985)

I think it probably unlocks because of the grace period. Do you have grace set?
Because when hyprlock crashes it will try to recover by restaring hyprlock. Otherwise you would be stuck with the red screen and until now i think we missed that grace is a problem when doing that.

But sadly I am not able to reproduce the crash when monitors are turning back on.

Could it be related to incorrect memory address for frameCallback?
https://github.com/hyprwm/hyprlock/blob/c87af3aa1f6e6bd06cffaabcc400bd45e26d565a/src/core/LockSurface.cpp#L129-L142
Valgrind output:
==23669== 1 errors in context 1 of 1:
==23669== Invalid read of size 8
==23669==    at 0x4318C2: CSessionLockSurface::render() (LockSurface.cpp:132)
==23669==    by 0x43C1C0: CHyprlock::onKey(unsigned int, bool) (hyprlock.cpp:867)
==23669==    by 0x43B1F1: handleKeyboardKey(void*, wl_keyboard*, unsigned int, unsigned int, unsigned int, unsigned int) (hyprlock.cpp:674)
==23669==    by 0x5845055: ffi_call_unix64 (unix64.S:104)
==23669==    by 0x584169F: ffi_call_int.lto_priv.0 (ffi64.c:673)
==23669==    by 0x58444ED: ffi_call (ffi64.c:710)
==23669==    by 0x4F09F2D: wl_closure_invoke.constprop.0 (connection.c:1025)
==23669==    by 0x4F0A7A2: dispatch_event.isra.0 (wayland-client.c:1631)
==23669==    by 0x4F0AA4B: UnknownInlinedFun (wayland-client.c:1777)
==23669==    by 0x4F0AA4B: wl_display_dispatch_queue_pending (wayland-client.c:2019)
==23669==    by 0x43A173: CHyprlock::run() (hyprlock.cpp:482)
==23669==    by 0x44C8DF: main (main.cpp:55)
==23669==  Address 0x70 is not stack'd, malloc'd or (recently) free'd

valgrind-out-native2.txt


fix(TDI-43085): Fix failing test for snowflake

What is the current behavior? (You can also link to an open issue here)
Test is failing because of extra reject message appeared
What is the new behavior?
Test fixed
Please check if the PR fulfills these requirements

[ ] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: success :star:
Sonar results


StrKeyTranslator excludes Z

We have a loop in test_utils that reads a little strange
        for (i, c) in (b'A'..b'Z').enumerate() {
            let key = String::from_utf8(vec![c]).unwrap();

Why is Z excluded?

Lol, I'm sure this was a mistake, though probably not a very important one.


Bump gevent version to avoid Cython cpdef issue

What version of gRPC and what language are you using?
gRPC v1.50.1 and I'm using C++ but this issue is with a Python dependency
What operating system (Linux, Windows,...) and version?
Ubuntu 22.04.01 LTS in WSL
What runtime / compiler are you using (e.g. python version or version of gcc)
$ gcc --version
gcc (Ubuntu 11.3.0-1ubuntu1~22.04) 11.3.0
Copyright (C) 2021 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
$ python -V
Python 3.10.6
What did you do?
$ bazel query //...
What did you expect to see?
The list of available labels.
What did you see instead?
I get an error when running query.  This seems to be due to a dependency on gevent which is too old.
ERROR: Could not find a version that satisfies the requirement gevent==21.1.2 (from versions: 0.9.2, 0.9.3, 0.10.0, 0.11.0, 0.11.1, 0.11.2, 0.12.0, 0.12.1, 0.12.2, 0.13.0, 0.13.1, 0.13.2, 0.13.3, 0.13.4, 0.13.5, 0.13.6, 0.13.7, 0.13.8, 1.0, 1.0.1, 1.0.2, 1.1a1, 1.1a2, 1.1b1, 1.1b2, 1.1b3, 1.1b4, 1.1b5, 1.1b6, 1.1rc1, 1.1rc2, 1.1rc3, 1.1rc4, 1.1rc5, 1.1.0, 1.1.1, 1.1.2, 1.2a1, 1.2a2, 1.2.0, 1.2.1, 1.2.2, 1.3a1, 1.3a2, 1.3b1, 1.3b2, 1.3.0, 1.3.1, 1.3.2, 1.3.2.post0, 1.3.3, 1.3.4, 1.3.5, 1.3.6, 1.3.7, 1.4.0, 1.5a1, 1.5a2, 1.5a3, 1.5a4, 1.5.0, 20.4.0, 20.5.0, 20.5.1, 20.5.2, 20.6.0, 20.6.1, 20.6.2, 20.9.0, 20.12.0, 20.12.1, 21.1.0, 21.1.1, 21.1.2, 21.8.0, 21.12.0, 22.8.0, 22.10.1, 22.10.2)
ERROR: No matching distribution found for gevent==21.1.2
Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/home/shareefj/.cache/bazel/_bazel_shareefj/ebd63933a37b436858f7ab2b2d82e5ea/external/io_bazel_rules_python/python/pip_install/extract_wheels/__main__.py", line 5, in <module>
    main()
  File "/home/shareefj/.cache/bazel/_bazel_shareefj/ebd63933a37b436858f7ab2b2d82e5ea/external/io_bazel_rules_python/python/pip_install/extract_wheels/__init__.py", line 76, in main
    subprocess.run(pip_args, check=True, env=env)
  File "/usr/lib/python3.10/subprocess.py", line 524, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['/home/shareefj/git/grpc/.venv/bin/python3', '-m', 'pip', '--isolated', 'wheel', '-r', '/home/shareefj/.cache/bazel/_bazel_shareefj/ebd63933a37b436858f7ab2b2d82e5ea/external/com_github_grpc_grpc/requirements.bazel.txt']' returned non-zero exit status 1.

An associated ticket on gevent:  https://github.com/gevent/gevent/issues/1801
If I manually try to install gevent as you have it in requirements.bazel.txt, gevent==21.1.2, I get the same error.

So the safe tags to use are:
$ git tag --contains 1aac7caba91
v1.51.0
v1.51.0-pre1
v1.51.1


Very long build times

After switching on this module, we've experienced that the build time increased from less than two seconds to around 15. The consequence is that we can't really use this module.
Are we doing something wrong or is it because the module doesn't handle large projects too well?

It does currently every single directory of every single file you're building, including duplicates. If that is a large number I imagine it'll probably become slow.
I might be able to do two things to speed it up:

Cache directories it has already checked and knows the case of, this should reduce the number of file system operations drastically since currently it checks all the way back to root. Eliminating the need to check the entire directory tree up to your project directory on every file should speed things up.
It currently checks directories synchronously. Splitting up the path and checking each one async will probably save some time as well.

I don't have a timeframe on those, unfortunately - under my own deadline at work, but maybe this weekend.

I was able to add a simple caching system to the plugin which dramatically reduces the number of filesystem operations required. Our project required 25,628 path.fsreaddirSync calls from this plugin. With the changes, that's down to 278 - nearly a 100x reduction.
I've published v1.1.0 to NPM. Please let me know if it helps.

Great! I'll take a look.

I've pushed a patch (1.1.1) which resolves an issue related to the cache becoming stale when used in a hot-reloader.
Any luck with build time reductions?

It's still very very slow, it even took more time than UglifyJsPlugin.

I'm sorry for not getting back to you @Urthen - I am on Linux and the coworkers on my team with Mac lost interest and removed the module :-/
Anyway - I am on a new team now and one of the new team members is complaining about case sensitive file names on MacOS, so obviously I've asked her to try this module.


Install missing unzip dependency

Required to unzip terraform archive.

lgtm, I thought this got fixed in code review of #135 but sounds like it got lost

Build FAILURE, see build http://<IP_ADDRESS>:8080/job/dev-tools/235/


rexray: read proxy.env

High Level Description
REX-Ray does not currently source the proxy configuration in /opt/mesosphere/etc/proxy.conf. This means AWS agents behind a proxy can't contact the EBS API. This PR adds the proxy configuration to the dcos-rexray.service file to enable this behaviour.
Related Issues

DCOS_OSS-1283 rexray not honouring proxy config

Checklist for all PR's

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here: Needs a proxy test environment configured. Tested manually.
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label Ready For Review

@mesosphere-mergebot label ship-it


Add helper to extract OrgID from the context

You're right. I thought it would be easier to access, but haven't realy given it too much thought.
I agree we should be consistent here, lets not add it :+1:


Cannot add custom policy definitions using \lib

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Versions

terraform: 1.0.4
azure provider: 2.71
module: 0.4.0
Description
I have added the lib folder for custom archetypes. I also have some custom policy assignments there. I ran into a situation where I wanted to create custom policy definition. I have named it "policy_definition_my_custom_policy.json". To my surprise, terraform plan does not do anything or "recognize" the policy definition. Is this by design?  And if I wanted to add one or two custom policies, how to do it?

Steps to Reproduce

create lib folder in the repo
add "policy_definition_custom_policy.json"
run terraform plan

Expected behaviour
I'd expect the module to grab the policy definition the same way as it does with custom policy assignments.

Screenshot

Hey @tohov,
Have you added the policy key name to an archetype definition in the policy definitions list that is used for one of your management groups?
Example here: https://github.com/Azure/terraform-azurerm-caf-enterprise-scale/blob/edd4ec093c5d3aa2468cf58dc867ae20e0698254/modules/archetypes/lib/archetype_definitions/archetype_definition_es_root.tmpl.json#L13
Let us know 👍

Nice. No I haven't. 😄 My bad. Thanks for the help @jtracey93! It works like a charm now. No bugs.

Superb! Not a problem at all and glad to help 👍
Will close this issue off now 👍

I would also recommend that you check out our feature which enables you to Expand built in archetype definitions.
This should significantly reduce any maintenance overhead on managing custom archetype definitions.

Note that you can also exclude specific items from the built-in archetype definitions too (guidance in the same Wiki page).


How to support json parameter and response?

Seems current swagger annotations and models target for java bean model.
So I have a question:
If the parameter and response payload are jsons expressed with a framework like form-binder-java, how can I transfer the structure and constraint info to swagger models?

p.s. the mapping definition can be put outside a method.

Close it, since I have found the way and implemented it in binder-swagger-java.
Welcome to use it. ^^


[hotfix][docs] Fix some formatting errors in Chinese documents.

What is the purpose of the change

In english document:

In chinese document:

Brief change log

Fix some formatting errors in Chinese documents.

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure

@MartijnVisser , Sorry to bother you, can you help me review this pull request? grateful.

@liuzhuang2017 Can you please fetch all upstream commits, rebase and push again? Right now all tests fail which make it not possible to merge.
Something like:

git fetch upstream (depending on how you've named the remote that's pointing to github.com/apache/flink)
git rebase upstream/master to rebase on the master branch from apache/flink

@MartijnVisser , Sorry to bother you again, can you help me review this pull request? grateful.


Fix ThemeToggle component

What kind of changes does this PR include?

Minor fixes (broken links, typos, css, etc.)

Description

Fix disappearing theme button when transitioning to different page. (In demo website, try to navigate Home <-> About page)
Improve dark-light icons to load better by using dark modifier instead of aria-pressed attribute.

Hi @atriple thank you for taking the time to do this and bringing it to my attention, I hadn't noticed this was happening.
The issue with the icon disappearing is an Astro bug involving persistence, and only after adding named transitions does the issue appear.
For some reason the two unrelated elements break persistence set in the Header, hence why your fix works by having to continually set  const button = ... with every function invocation.
Until it's fixed, I've reverted the theme to not using View Transitions as this will trip people up in various instances, and left a message in the README to your request as a current solution.
Thanks again

Hi @atriple thank you for taking the time to do this and bringing it to my attention, I hadn't noticed this was happening.
The issue with the icon disappearing is an Astro bug involving persistence, and only after adding named transitions does the issue appear.
For some reason the two unrelated elements break persistence set in the Header, hence why your fix works by having to continually set const button = ... with every function invocation.
Until it's fixed, I've reverted the theme to not using View Transitions as this will trip people up in various instances, and left a message in the README to your request as a current solution.
Thanks again

We might use transition:persist directive on the Header component instead, for fixing the issue.
src/components/layout/Header.astro:
<header
	id="main-header"
	class="group relative mb-28 flex items-center sm:ps-[4.5rem]"
	transition:persist
>
</header>

Thanks, and that's what I thought but unfortunately when this issue was created, persist was included. It worked until you then added a named transition on an unrelated element.
I've since changed the Toggle to a web component and removed the inline code, but it was those inconsistencies which meant that I just removed view-transitions as a default.

Thanks, and that's what I thought but unfortunately when this issue was created, persist was included. It worked until you then added a named transition on an unrelated element.
I've since changed the Toggle to a web component and removed the inline code, but it was those inconsistencies which meant that I just removed view-transitions as a default.

In that case, the latter approach works just fine. Thanks for clearing that up!


RAPL code refactoring

Due to https://github.com/sustainable-computing-io/kepler/issues/591, made below changes:

Correct some typo on code.
Simplify code logic and improve code efficiency in RAPL Sysfs.
Correct code logic error in RAPL MSR.
Align with the latest Intel SDM.

Signed-off-by: Jie Ren<EMAIL_ADDRESS>

Btw, thanks @jiere for contributing to improve the Kepler code!

do we kepler need to take care both client-platform CPUs and server-platform CPUs? I once have an assumption that it is meaningful to deal with Data Center power efficiency and carbon awareness.

when you say client, is it the kepler running on platform? I think the focus is the measurement target, so mostly server side you mention?

Indeed, we are aware that some platforms may not have uncore (PP1) or psys.
However, this is not an issue if a power source does not exist. They will just have 0 values.
Note that differentiating between client and server platforms is not necessary. What matters is that in the MSR logic, we need to take into account that "The DRAM units is different for different CPU models," as can be seen in the code that I shared.
Thank you for catching the issue with the sysfs logic!
It should be a simple fix by checking if the string package-0 contains the substring package.

What matters is that in the MSR logic, we need to take into account that "The DRAM units is different for different CPU models," as can be seen in the code that I shared.

Double checked on ark.intel.com, all those models(Haswell EP/Broadwell EP/Skylake X/Xeon Phi) are in "Discontinued" marketing status, which means EOL. So we are safe to simply regard that the DRAM energy unit always equals to CPU energy unit.
Thank you for your careful and nice comments, let me kick off a new diff later.

@jiere please check my comments in the PR ##601

The reason for power pkg to maintain a "pkg-to-corelist" map is that current powerInterface methods are package level organized, and each cpu core has its "/dev/cpu/*/msr".
Ideally when we need CPU package level power consumption metrics, we can loop the package's each core's MSR to get specific data.
Up till now, Intel could only support package level power meter, so all cpu cores' MSR power data are same if they belong to the same package.
But I did heard that AMD has a patch on core level power meter, which is perf event based.

Right, I just want to ensure that we avoid redundant works and conduct topology discovery only once and through a single method to have consistency in the code.

I cannot agree more :-)

@marceloamaral and all other reviewers, I have rebased my diff and kicked off the update, please continue reviewing, thanks.
For the "bash -c" code style issue, I have found a solution, referenced code here.
I do agree with @marceloamaral 's point that we need to maintain as little as possible redundant code in a project, currently we have so many utils.go among various of pkgs, maybe someday we can consolidate them to a common place or project scope pkg to maintain them as one piece. But currently I choose not, the config pkg may not be very proper to act as the whole project's common API/lib repository.

@jiere did you run make tidy-vendor before pushing the PR?

@jiere did you run make tidy-vendor before pushing the PR?

Yes, I did. And below are the current status of my local workspace:
~/dev/kepler$ git status
On branch rapl-enhance
Untracked files:
(use "git add ..." to include in what will be committed)
vendor/github.com/StackExchange/
vendor/github.com/ghodss/
vendor/github.com/go-ole/
vendor/github.com/jaypipes/
vendor/github.com/mitchellh/
vendor/howett.net/
nothing added to commit but untracked files present (use "git add" to track)
Shall I include these vendor/xxx untracked files also in current PR?


Update example vcpkg-configuration.json in test-registry-ports-gha.md

As I was following along with this documentation, vcpkg gave an error:
vcpkg-configuration.json: error: $.registries[0].packages: mismatched type: expected a package pattern array
It appears packages value must be a JSON array.

If you prefer to leave it as is, that's fine. I just noticed the bug and thought I'd try to fix it. It's a small thing either way. Thanks for looking at it though.


Remove remaining feature flags

This commit removes feature flags which are now enabled on all environments, which are:

USER_DASHBOARD
SUPPLIER_DASHBOARD
EDIT_SUPPLIER_PAGE

None of the other apps have feature flags which can be removed, therefore this completes story 102080450.

:+1:


inspect role for nonfungibles_v2

Add an interface to read-only the role of pallet-nfts
Since using pallet-nfts, we need a way to read the NFT's role to implement other functions.

bot fmt

Code-wise, everything looks good! Pls update the description and change the doc comments as per the suggestions

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3037672

Pls try to merge the master branch to make the CI happy. But I would guess that we need to get that PR: https://github.com/paritytech/substrate/pull/14424 merged into master first.

bot merge


Investigate issue with TypeScript update

Changes in this PR
We update our integration test tsconfig file to bring it more line line with CAS1's equivalent file, and our own main tsconfig file. This resolves an issue where on upgrading to TypeScript 5.1.3, our typecheck command appears to run out of memory checking integration tests
See https://github.com/ministryofjustice/hmpps-temporary-accommodation-ui/actions/runs/5371348478/jobs/9744052370?pr=417 for an example
Release checklist
Release process documentation.
Pre-merge checklist

[ ] Are any changes required to dependent services for this change to work?
(eg. CAS API)
[ ] Have they been released to production already?

Post-merge checklist

[ ] Manually approve
release to test
[ ] Manually approve
release to preprod
[ ] Manually approve
release to prod

 Investigate issue with TypeScript update


Fix a race condition where ESR sometimes failed to start ETR

Applicable Issues

N/A
Description of the Change

Move check for activity finished before getting environments so that we always check for environments even if the activity is is finished. This fixes a race condition where the activity would finish before we've requested all environments from the event repository.
Alternate Designs

We should add more logic, checking that the number of suites started corresponds to what the environment provider reports. But that would be a much larger change that would take a longer time to fix.
Benefits

No race conditions (in this part of the code at least)
Possible Drawbacks

None
Sign-off

Developer's Certificate of Origin 1.1
By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.
Signed-off-by: Tobias Persson<EMAIL_ADDRESS>

Since @andjoe-axis is on vacation I will approve this in his stead.


BoringSSL build fails during packaging object files

$ git clone https://boringssl.googlesource.com/boringssl
$ docker run -ti --rm -v `pwd`/boringssl:/tmp/boringssl -w /tmp/boringssl/ golang bash
>$ apt-get update && apt-get install -y ninja-build cmake
>$ mkdir build && cd build
>$ cmake .. -GNinja
>$ ninja
/usr/bin/ar: crypto/x509/CMakeFiles/x509.dir/x_pkey.c.o: Operation not permitted

Full container's logs on Pastebin
I don't think the problem is in any way specific to BoringSSL, it's just the first thing that fails in my own project's build. :)
This looks to be a problem on the filesystem's (9p) end, and unfortunately blocks me from migrating from docker-machine to dlite.

yeah, 9p was the wrong path to go down. you might try dlite 1.1.4 instead as it uses NFS. i'm hoping to have a new release out soon with an entirely different filesystem sharing layer

went back to nfs in master (not yet released) so closing


Course Homepage: Episode list main title has random borders

Episode list in a course homepage appears to have borders that shouldn't be there
Steps to reproduce
Navigate to a course homepage
In this example I'm on 'Introduction to Google's Material Design'
Expected behavior
Episode list only has borders between episodes
Actual behavior
Episode list has border above and below main title 'Course Episodes'

Environment

iOS Version: 14
raywenderlich.com App Version: 1.0.5
Device: iPhone 11

This issue should be closed the issue, Design has been corrected.

@luke-freeman is this now fixed?


Ruler Concurrency: Make rules sortable

We want to expand concurrency of rules in Mimir but there's no easy way to hook in a new feature to sort rules before running them:

GroupLoader runs before dependencies are computed, which would lead to duplicated work
RuleConcurrencyController works on individual rules after the order is baked in. It's too late at that point
EvalIterationFunc calls .Eval() on a group but the order is also baked in that function

This PR adds a RuleSortingController interface to the the concurrency controller so that we can optionally sort rules before running them. I think it makes sense that it's bundled with the concurrency controller because there's not much possible benefit to sorting rules if we're running them sequentially

Discussed offline, I think we're moving in a different direction so we won't be proceeding with this one.


more broken test cases

@fearphage, I am going to revert your changes. Unless you have any other ideas

I can tell you that calling 0 functions will generally be faster than calling N functions (where n is a positive integer :stuck_out_tongue_winking_eye:). However, please feel free to revert as you see fit. It is your project. My PR was merely a suggestion.
Fix?
return x * 1 === x && x % 1 === 0 && x > 0

Explanation
// Is it a number?
return x * 1 === x
  // Is it an integer?
  && x % 1 === 0
  // Is it greater than 0?
  && x > 0
;

fixes #9

@fearphage Thank you!


When generating server_index.cshtml, replace default  tag

When generating server_index.cshtml, replace default  tag.
As we inherit from the Blazor WASM version, there is a missing bit that can do some incorrect behavior if mapping a server-side version somewhere other than a root path.
Actually the generated file do a:
<base href="/" />

it must be instead:
<base href="~/" />

on the server-side version.

Done


chore: Remove pylint test

pylint has not been catching anything that flake8 wouldn't also catch, and flake8 continues to improve, yielding pylint somewhat obsolete.

LGTM!


fix shebang

See https://en.wikipedia.org/wiki/Shebang_(Unix)#Portability

Yes.

Done. Also fixed the shebang for the macos patch.

Nice work. Thanks for your help!


train_test_split, sample option random

Hello,
We are a problem with train_test_split function and sample function with parameter method = random.
When we use 2 times these functions, we doesn't have the same results. And when we want 30/70 partition we can have 25/75 or 35/65 (or worse).
I saw the source code and I saw you create a new column random, and take lines where random column < 0.3. But random is random.
I think you should use quantile function to select your lines like this : random column < quantile(0.3).
with this code for sample function, I obtain 0.33 of my dataset exactly:
x=0.33
print("true value : ",len(titanic)*x)
titanic = load_titanic(schema = schema,cursor = cur, name='titanic')
titanic.eval("random", "random()")
drop(schema + ".titanic2" ,cur,method = "table")
titanic.to_db(schema + ".titanic2" , relation_type = "table")
titanic2 = vDataFrame(input_relation = schema + ".titanic2", cursor = cur)
len(titanic2.search("random < " + str(titanic2["random"].quantile(x))))

the result is :

Can you change this in sample and train_test_split functions?
Regards,
Alexandre

Hi Alexandre,
You can use the parameter "random_state" to seed the randomness.
Badr

Hello,
Even if we use the parameter "random_state", if we want test_size = 0.3, the test size varies between 0.25 and 0.35.
If you use quantile(test_size) instead of test_size, we can have exactly 0.3 for test size.
Alexandre

That's a good improvement, we can seed the randomness and use a quantile to filter. Thanks a lot for your help.


Typo

In the "About" description for the project, "OpenBSD tool to signs and verify signatures on files. Portable version."
"signs" should be "sign"

Fixed, thanks for letting me know @za3k!


Add .pyi stub generation

Description
Closes #1252. IDEs can use .pyi stub files located in the package to greatly enhance code-completion and documentation for Python C extension modules.
Add as part of setup.py installs, as well as CMake via new OPENASSETIO_ENABLE_PYTHON_STUBGEN variable (ON by default).

[x] I have updated the release notes.
[ ] I have updated all relevant user documentation.

Test Instructions
Worth checking alternative IDEs.

I installed Visual Studio Code fresh (didn't have it installed before now) in order to test this works with IDEs other than PyCharm.

pip install . works
pip install ./openassetio-1.0.0b2.post1-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (i.e. pre-built wheel) works. So presumably the PyPI package will work too.
pip install --editable doesn't work - Visual Studio Code's default Python plugin doesn't seem to be able to detect the editable install at all, regardless of .pyi stubs. So there's a deeper issue there to do with either my set-up or a limitation of VSCode.
cmake --install, plus .pth file in the venv pointing into the install directory (closest I could come up with to PyCharm's "Mark Directory As"), doesn't work - similar problem to --editable - no trace of openassetio.


Security of offline downloads

Shaka support offline download and persistent licenses for widevine.
How secure is offline download and persistent licenses?
Are the widevine licenses are node locked? if not node locked then the user can download the content offline and use the persistent token to play it in other nodes and distribute the content? Because it is easy to copy the token from the chrome networking trace or by putting breakpoints in js code.

This is a question best for the Widevine team: http://www.widevine.com/contact.html
The handling of licenses and persistent files is handled by the browser, so we don't do anything for it.  All we do is download the segments, save them, and fetch the license (which we just give it to the browser).
Widevine is highly secure; from my understanding: the license is encrypted over the wire, so you can't read the keys from the Network tab.  The response is timestamped and has nonces, so you can't use the response in another computer, save it to use later, or give it to Chrome twice.  Also you can't get the keys from the license, so there is no way to distribute the decrypted content, only the encrypted files.  Only Chrome can use the keys and it will only allow playing the content, not just decrypting it.

I will add that licenses, as they appear over the wire, are only readable by the CDM (a content security component inside the browser) that requested the license.  Persistent licenses are stored securely by the CDM itself, and cannot be tampered with or transferred to other machines.
Does this help?

ok...thanks for the detailed explain


test: add showcase test for api-version

This pr updates gapic-showcase version to 0.35.0 and adds showcase tests to verify behavior of api version headers being emitted (changes in #2630 and #2671).

LGTM. Just a few nits/ questions, but I'll approve once those are addressed.


feat(pie-docs): DSW-1884 remove /overview in foundation page urls

Describe your changes (can list changeset entries if preferable)

"pie-docs": minor
[Changed] - Removed /overview urls from foundations pages and use overview content as the index.html of each directory. I.e. /foundations/colour for the overview page instead of /foundations/overview/colour. Also now uses the 404 rather than coming soon partial for empty pages as the coming soon was misleading and often not true.
Author Checklist (complete before requesting a review)

[x] I have performed a self-review of my code
[x] I have added thorough tests where applicable (unit / component / visual)
[x] I have reviewed the PIE Storybook/PIE Docs PR preview
[x] I have reviewed visual test updates properly before approving

Reviewer checklists (complete before approving)
Reviewer 1 - @leksaBoiko

[x] I have reviewed the PIE Storybook/PIE Docs PR preview
[x] If there are visual test updates, I have reviewed them

Reviewer 2

[ ] I have reviewed the PIE Storybook/PIE Docs PR preview
[ ] If there are visual test updates, I have reviewed them

Looking good! Do we have any hard-coded links in the docs site that point to /overview that need updating?

fantastic spot/point! I have updated all links I could find 👍


Hardcoded value in example

When going through the tutorial, I noticed there's a hardcoded value in "Create a local network gateway". Please fix the Location-param.
New-AzLocalNetworkGateway -Name $LNG1 -ResourceGroupName $RG1 `
-Location 'East US' -GatewayIpAddress $LNGIP1 -AddressPrefix $LNGprefix1,$LNGprefix2
Should be changed to
New-AzLocalNetworkGateway -Name $LNG1 -ResourceGroupName $RG1 `
-Location $Location1 -GatewayIpAddress $LNGIP1 -AddressPrefix $LNGprefix1,$LNGprefix2
Create a local network gateway
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a6e337ee-ab03-ec53-e606-b377923013fd
Version Independent ID: afc1dcb7-7c2b-466c-611c-f13c93ea53ac
Content: Azure VPN Gateway: Create and manage S2S VPN connections: Tutorial
Content Source: articles/vpn-gateway/vpn-gateway-tutorial-vpnconnection-powershell.md
Service: vpn-gateway
GitHub Login: @yushwang
Microsoft Alias: yushwang

@tennijo , Thank you for your feedback. We will investigate and get back to you shortly.

PR: https://github.com/MicrosoftDocs/azure-docs-pr/pull/110050


Dataset size and CT scans

Hello, is your dataset composed of just those images in data folder?
If yes I think that is a dataset too little to train a reliable generalized model, isn't it?
Apart from this I would like to point you to my project, which shares the objectives with yours and has a larger dataset of both RX and CT scans.
You can find it here: https://github.com/AleGiovanardi/covidhelper
I also would like to suggest you to try out the analysis of CT scans instead of RX for three main reasons:

CT are widely more used for diagnostic purposes.
Pathological regions are more clear to see and shows clear signs of identification of the disease like you can see here: https://www.auntminnie.com/index.aspx?sec=sup&sub=cto&pag=dis&ItemID=128300
In CT scans there is less noise compared to RX scans

I want also to congratulate with you for your job, it is an amazing work imho.
Keep it up.
Kind regards,
Ale.

Hello, is your dataset composed of just those images in data folder?
If yes I think that is a dataset too little to train a reliable generalized model, isn't it?
Apart from this I would like to point you to my project, which shares the objectives with yours and has a larger dataset of both RX and CT scans.
You can find it here: https://github.com/AleGiovanardi/covidhelper
I also would like to suggest you to try out the analysis of CT scans instead of RX for three main reasons:

CT are widely more used for diagnostic purposes.
Pathological regions are more clear to see and shows clear signs of identification of the disease like you can see here: https://www.auntminnie.com/index.aspx?sec=sup&sub=cto&pag=dis&ItemID=128300
In CT scans there is less noise compared to RX scans

I want also to congratulate with you for your job, it is an amazing work imho.
Keep it up.
Kind regards,
Ale.

Good day, Alle.
I think the Covid19 dataset I use is too large to be uploaded to github.

I had mentioned in the repository in the code setup section, that the dataset (containing 188 images) was available on a google drive link of mine.

The data was taken from some Covid19 papers, as well as from a repository by Dr Jospeh Cohen, which contains 200+ images.

The goal is to use any type of image data available, in a consistent way. So far, I've been using coronal-view (frontal) based images.

What types of views have you been using?
What is your Sensitivity/Accuracy/Specificity so far?

Hello, is your dataset composed of just those images in data folder?
If yes I think that is a dataset too little to train a reliable generalized model, isn't it?
Apart from this I would like to point you to my project, which shares the objectives with yours and has a larger dataset of both RX and CT scans.
You can find it here: https://github.com/AleGiovanardi/covidhelper
I also would like to suggest you to try out the analysis of CT scans instead of RX for three main reasons:

CT are widely more used for diagnostic purposes.
Pathological regions are more clear to see and shows clear signs of identification of the disease like you can see here: https://www.auntminnie.com/index.aspx?sec=sup&sub=cto&pag=dis&ItemID=128300
In CT scans there is less noise compared to RX scans

I want also to congratulate with you for your job, it is an amazing work imho.
Keep it up.
Kind regards,
Ale.

Good day, Ale. You'll find that lack of data is a general issue. In my repository, I mentioned this.
Also, about 6 weeks before your inquiry/comment, a few CT scan images were used initially, and this is why the repo had CT scan in the name since Feb 9, 2020, although CT scan image data is scarce.

I made a request for more data via Chinese researchers, though no response yet.
I made a call to my Government on February 17, to try to use their administrative powers to liaise with countries with large amounts of Covid lung scan data, to try to acquire said data.

I think the Covid19 dataset I use is too large to be uploaded to github.

I had mentioned in the repository in the code setup section, that the dataset (containing 188 images) was available on a google drive link of mine.

The data was taken from some Covid19 papers, as well as from a repository by Dr Jospeh Cohen, which contains 200+ images.

The goal is to use any type of image data available, in a consistent way. So far, I've been using coronal-view (frontal) based images.

What types of views have you been using?
What is your Sensitivity/Accuracy/Specificity so far?


Nodes can be dragged/created outside the viewport

Describe the bug
The nodes in a graph can be dragged outside the viewport. On dragging a node outside the viewport, it becomes invisible (svg overflow is not set). Also, on dynamically adding new nodes (like in the sandbox examples), it is possible that the newly added node renders outside the viewport.
To Reproduce
Steps to reproduce the behavior:
Open any sandbox dataset demo and drag a node outside the viewport.
Expected behavior
Have the graph bound to the size of the container
Screenshots

Environment:

OS: [e.g. iOS]: Mac
Browser: Chrome
Version: 78.0.3904.97
react-d3-graph version: 2.3.0

Additional context
If this is expected behavior, is there a way to make the the graph bounded like here ?

Thanks for the very specific detail @pushpinder107, this is currently not possible with react-d3-graph. I will tag this as a feature request, let's see if someone wants to pick this up, as I'm currently with no time to look into it. PRs are always welcome :slightly_smiling_face:


#2350 Source lines

#2350 proof-of-concept

I think you should update the files with the current copyright notes.

It's hard to comment the overall functionality when it's not possible to really test it. As far as I can see, this PR just has code to somehow get the line numbers but doesn't actually add them into the HTML output.
The bigger problem here is that, as I commented in #2350, I fail to see use cases to show line numbers in Libdoc's output. You said request come from testers who already know Python, but I would then consider just using normal Python documentation generators (e.g. Sphinx) in the first place rather than Libdoc.
Another problem is that the code uses lot of Robot's internal APIs related to test libraries to get information it needs. These APIs are subject to change in the future, in fact the whole library import/usage logic is likely to be rewritten at some point. That would mean breaking all this code, and we maintainers then would need to fix it. That's not a chore I want to accept unless benefits are very big.
Then there's also the problem that, as I commented in #2350, this functionality could only be supported with a subset of library/resource types Libdoc in general supports.
Instead of trying to add this functionality into Libdoc, I would recommend you to consider these alternatives:

Implement a custom tool that extends Libdoc. It wouldn't need to care about library/resource types you don't need.

Switch to other documentation generators. If we enhance Libdoc's reStructuredText support as planned (see #2507), you could actually generate docs from the same source both using Libdoc and some more programmer oriented tool.

It's hard to comment the overall functionality when it's not possible to really test it. As far as I can see, this PR just has code to somehow get the line numbers but doesn't actually add them into the HTML output.
The bigger problem here is that, as I commented in #2350, I fail to see use cases to show line numbers in Libdoc's output. You said request come from testers who already know Python, but I would then consider just using normal Python documentation generators (e.g. Sphinx) in the first place rather than Libdoc.
Another problem is that the code uses lot of Robot's internal APIs related to test libraries to get information it needs. These APIs are subject to change in the future, in fact the whole library import/usage logic is likely to be rewritten at some point. That would mean breaking all this code, and we maintainers then would need to fix it. That's not a chore I want to accept unless benefits are very big.
Then there's also the problem that, as I commented in #2350, this functionality could only be supported with a subset of library/resource types Libdoc in general supports.
Instead of trying to add this functionality into Libdoc, I would recommend you to consider these alternatives:

Implement a custom tool that extends Libdoc. It wouldn't need to care about library/resource types you don't need.

Switch to other documentation generators. If we enhance Libdoc's reStructuredText support as planned (see #2507), you could actually generate docs from the same source both using Libdoc and some more programmer oriented tool.

Hi Pekka,
the lines can be see in Console View and XML output if You add switch -S. I do not even consider HTML since normal lib specs imported by RIDE are generated as XML and its enough.
If You need to check it just create simple 'debugBase.py` file with content like below:
class debugBase:
    def done1(self, name):
        return

And execute command: python -m robot.libdoc -S debugBase.py show and You will see in output additional line: Source D:\wyp dump\untitled1\untitled1\debugBase.py:done1:2.
About RF and Python documentation are big difference since if You will take look regarding decorators (embedded keywords possibility), dynamic keywords API and function names You will see that is not so tricky to say what is really keyword name. About real requests just see: https://github.com/nokia/RED/issues/54 . I think Sphinx will do not know where Java keywords exists or dynamic keywords and the source code upload by me deals with Java and Bytecode files, so is not economical to made in each editor own logic to get an information where RF keywords exists. If You check the logic is very simple get method line.

A decision was made not to add this functionality to Libdoc. Seeing source lines is not a feature normal Libdoc users would need, implementation is complex, and it isn't possible to implement this in generic manner with all supported library types. It is better to implement a custom tool that uses Libdoc, inspect and possibly Javadoc internally. See discussion at #2350 for more details.


Angular JS 2 Stable support?

Hey , i want to make it work on ionic 2 - You think its possible to do it? They code will work on magento 2.1.4?

@refaelgold Not familiar with Ionic 2, but I think it's definitely possible. I haven't got time to support or test it against Magento 2.1.4 and the official Angular 2.x versions.

have you a demo from it?

i want to take it over and do pulls for this projects. i will try to make more stable version for angular js 2.4


Surjection-proof combine API

Closing as it is badly out of date. Will revisit if we decide to work on combining surjection proofs again.


Create an application that can add tasks and integrate google maps in the application

The list should be local and synced to drive. No other 3rd parties.

Can integrate with google forms


Improve performance of set_pm_conversation_count in static/js/pm_list.js

I was doing some frontend profiling earlier today on chat.zulip.org, and ran into a case where update_dom_with_unread_counts() took 163ms, which is definitely high for that sort of update.  100ms of that was 20 calls to set_pm_conversation_count in static/js/pm_list.js, each of which was consuming 5ms.  This is a bit odd because I definitely didn't have 20 conversations with unread messages in the PM list.
I suspect this is partially some sort of UI scalability issue, where this only happens because there's a large number of users in the organization, but also in part a bug that exports.get_li_for_user_ids_string is such an expensive operation and we're calling it not just for the PM list entries whose unread counts might have changed.
Tagging as difficult because I think this sort of profiling issue can be difficult to track down and optimize.

@andersk was this potentially fixed by your rerendering changes in https://github.com/zulip/zulip/pull/11792?

@timabbott Unlikely, unless by accident (which would be the worst kind of fix). It would be good to have a reproducible test case.


Apply the 'no-unused-variable' tslint rule to our codebase

There is also a check-parameters option, but that would've been a much bigger, less reviewable change.

That is lots of unused stuffs

:+1:  just small comments

:+1:


New version: ExactFieldSolutions v0.1.5

Registering package: ExactFieldSolutions
Repository: https://github.com/tduretz/ExactFieldSolutions.jl
Created by: @tduretz
Version: v0.1.5
Commit: 745b69c9c853461c0548b7b61cb1d862a88f9426
Reviewed by: @tduretz
Reference: https://github.com/tduretz/ExactFieldSolutions.jl/commit/745b69c9c853461c0548b7b61cb1d862a88f9426#commitcomment-147870507
Description: Full field solutions of Poisson and Stokes problems in 1D, 2D and 3D. Full field solutions to other problems are welcome!
Release notes:

## Breaking changes

- add exact solution for Stefan problem

[merge approved]


rm: security: improve boot software updates article for zynqmp

Improve details about usage of multi_boot U-Boot cmd.
Signed-off-by: Igor Opaniuk<EMAIL_ADDRESS>Readiness

[ ] Merge (pending reviews)
[ ] Merge after date or event
[ ] Draft

Overview
Why merge this PR? What does it solve?
Checklist
Optional. Add a 'x' to steps taken.
You can fill this out after opening the PR. "Did I..."

[ ] Run spelling and grammar check, preferably with linter.
[ ] Avoid changing any header associated with a link/reference.
[ ] Step through instructions (or ask someone to do so).
[ ] Review for wordiness
[ ] Match tone and style of page/section.
[ ] Run make linkcheck.
[ ] View HTML in a browser to check rendering.
[ ] Use semantic newlines.
[ ] follow best practices for commits.

[ ] Descriptive title written in the imperative.
[ ] Include brief overview of QA steps taken.
[ ] Mention any related issues numbers.
[ ] End message with sign off/DCO line (-s, --signoff).
[ ] Sign commit with your gpg key (-S, --gpg-sign).
[ ] Squash commits if needed.

[ ] Request PR review by a technical writer and at least one peer.

Comments
Any thing else that a maintainer/reviewer should know.
This could include potential issues, rational for approach, etc.

Docs for 4fc19c1728c1c8f5766063b78b1445a67d1ce95b are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1655/docs/artifacts/html/index.html

Docs for e13d1a1501939bbea9123b425ae5549004abdf07 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1656/docs/artifacts/html/index.html

Docs for d8d55ecdb2be0406be2ce546b392b6c323508951 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1660/docs/artifacts/html/index.html

Docs for dc8dad84ddfd58320af7a17de71c6541ccd52cff are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1662/docs/artifacts/html/index.html

Thanks @kprosise for suggestions. Addressed your comments, and PR is ready for merge


User Help&How-tos: Configurable screens - Sale & Project

@kirstiaa
In Edit layout for archives/Section tabs: You can define advance options for single/double click to change default behaviour.

Do you have any examples / use cases for these options? What would be a good example to illustrate the usefulness of changing these options?
Do you have a link to additional resources for setting up url and soprotocol?
Torgeir


Create BigOffer component

Create a BigOffer component in the organisms folder
Use Swiper.js
It should match our design

Kindly assign this issue to me, I believe I would be able to contribute positively

Kindly assign this issue to me, I believe I would be able to contribute positively

I have assigned it. Good luck


Now that Tailwind 4 is coming?

Description
Can Nuxt UI easily can adapt to the upcoming tailwind 4?

Not easily, this will be done in the next major: https://github.com/nuxt/ui/issues/1289


Keep date formatting behavior aligned with ES API for spark serialization

I've noticed that when streaming documents into spark with fields that have type date, the format that is passed to the spark job itself is that of the java.util.Date default. This occurs even if the underlying document has UTC millis as its format. The problem we encountered is when we reindex our data, we stream in one format, and write out a different one.
I tracked this behavior down to this serializer: https://github.com/elastic/elasticsearch-hadoop/blob/master/spark/src/main/scala/org/elasticsearch/spark/serialization/ScalaValueReader.scala#L100-L101
I bring this up because I think this is more than a implementation preference. The ES API itself returns the format in which it was indexed with so I think that's justification for making sure the serializer behaves the same way.
The behavior in the spark connector also differs from that of the hadoop implementation, which is aligned with how the REST API behaves.

So if I understand correctly, the issue in your case is the fact that you would like the date (which can be only represented as either a long or a String) in a primitive form instead of a Date, correct?

I tracked this behavior down to this serializer

That's actually the deserializer as it reads the data from the wire.

The behavior in the spark connector also differs from that of the hadoop implementation, which is aligned with how the REST API behaves.

There is no Date representation in Hadoop and thus the data is represented as a long. Where it does exist (Pig, Hive, Spark) the rich types are used instead. I can see though that more control over how a Date is represented would be convenient and having an option to enable/disable the rich object mapping would be helpful.

Raised #422

@costin Correct, I'd like to read back the same format in which it was written. From my observations, this seems to be the behavior of the REST API.
My point w/r/t the Hadoop implementation is that when considering the ES REST API, Hadoop, and Spark implementations, they all produce different views of that data type by default. Consistency would certainly be a desirable milestone here, with the preference being to copy the behavior from the API.

Closing the issue; if the problem persists please create a new one.


For sentences with anchor tags

I been trying to use your library so I can mark sentences on a the body of the page using a chrome extension but I run into a few problems with this special characters. I wish to know with the is a better way to approach this problem.
var markInstance = new Mark(document.body);
	var options = {
		element: `igreeai-mark`,
		className: `igree-highlight-${position}`,
		exclude: ['.ignore-iagree'],
		separateWordSearch: false,
		debug: true,
	};
	markInstance.mark(message, options);

some sentences with hyperlinks inside gave problems to mark

Hi,
Thanks for submitting.
Please read the contribution guidelines and provide the required information named there.


Access violation crash during gameplay

Happened during gameplay on a gun game server that was running Colony. Can't seem to replicate the issue reliably but it's happened twice. Am running latest release 1.3.0.
nslog04-01-2022 08-03-10.txt
Dump file here

same problem, was on forward base kodai when it happened,
nslog2022-01-21 14-19-01.txt


Enable bindings in macos and bump build number to 3

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I Failed to even lint the recipe, probably because of a conda-smithy
bug :cry:. This likely indicates a problem in your meta.yaml, though. To get a traceback to help figure out what's going on, install conda-smithy and run conda smithy recipe-lint --conda-forge . from the recipe directory.
This message was generated by GitHub actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/11157437869.

macos bindings still segfault:
export PREFIX=/Users/runner/miniforge3/conda-bld/icub-main-split_1727939211813/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehol
export SRC_DIR=/Users/runner/miniforge3/conda-bld/icub-main-split_1727939211813/test_tmp
import: 'icub'
/Users/runner/miniforge3/conda-bld/icub-main-split_1727939211813/test_tmp/conda_test_runner.sh: line 3: 21730 Segmentation fault: 11  "/Users/runner/miniforge3/conda-bld/icub-main-split_1727939211813/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehol/bin/python" -s "/Users/runner/miniforge3/conda-bld/icub-main-split_1727939211813/test_tmp/run_test.py"
WARNING: Tests failed for icub-main-python-2.6.3-py312ha5311f2_3.conda - moving package to /Users/runner/miniforge3/conda-bld/broken
Traceback (most recent call last):
  File "/Users/runner/miniforge3/lib/python3.12/site-packages/conda_build/build.py", line 3483, in test
    utils.check_call_env(
  File "/Users/runner/miniforge3/lib/python3.12/site-packages/conda_build/utils.py", line 404, in check_call_env
    return _func_defaulting_env_to_os_environ("call", *popenargs, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/runner/miniforge3/lib/python3.12/site-packages/conda_build/utils.py", line 380, in _func_defaulting_env_to_os_environ
    raise subprocess.CalledProcessError(proc.returncode, _args)
subprocess.CalledProcessError: Command '['/bin/bash', '-o', 'errexit', '/Users/runner/miniforge3/conda-bld/icub-main-split_1727939211813/test_tmp/conda_test_runner.sh']' returned non-zero exit status 139.

During handling of the above exception, another exception occurred:

bt of local fail:
˜˜˜

thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x10)

frame #0: 0x00000001064005fc libpython3.12.dylib_PyObject_Malloc + 36 frame #1: 0x00000001064359a0 libpython3.12.dylibPyUnicode_New + 208
frame #2: 0x0000000106433368 libpython3.12.dylibunicode_decode_utf8 + 228 frame #3: 0x0000000105b798d0 _icub_python.soPyInit__icub_python + 88
frame #4: 0x0000000104ee3730 python3.12_PyImport_LoadDynamicModuleWithSpec + 728 frame #5: 0x0000000104ee2b18 python3.12_imp_create_dynamic + 488
frame #6: 0x0000000104de3344 python3.12cfunction_vectorcall_FASTCALL + 248 frame #7: 0x0000000104ea34a4 python3.12_PyEval_EvalFrameDefault + 62644
frame #8: 0x0000000104d88f6c python3.12object_vacall + 384 frame #9: 0x0000000104d88d18 python3.12PyObject_CallMethodObjArgs + 124
frame #10: 0x0000000104edf664 python3.12PyImport_ImportModuleLevelObject + 1744 frame #11: 0x0000000104e9eb20 python3.12_PyEval_EvalFrameDefault + 43824
frame #12: 0x0000000104e930c0 python3.12PyEval_EvalCode + 300 frame #13: 0x0000000104e8fcc8 python3.12builtin_exec + 1412
frame #14: 0x0000000104de31e8 python3.12cfunction_vectorcall_FASTCALL_KEYWORDS + 144 frame #15: 0x0000000104ea34a4 python3.12_PyEval_EvalFrameDefault + 62644
frame #16: 0x0000000104d88f6c python3.12object_vacall + 384 frame #17: 0x0000000104d88d18 python3.12PyObject_CallMethodObjArgs + 124
frame #18: 0x0000000104edf664 python3.12PyImport_ImportModuleLevelObject + 1744 frame #19: 0x0000000104e9eb20 python3.12_PyEval_EvalFrameDefault + 43824
frame #20: 0x0000000104e930c0 python3.12PyEval_EvalCode + 300 frame #21: 0x0000000104f0bdf0 python3.12run_mod + 228
frame #22: 0x0000000104f0c308 python3.12PyRun_InteractiveOneObjectEx + 732 frame #23: 0x0000000104f0b178 python3.12_PyRun_InteractiveLoopObject + 176
frame #24: 0x0000000104f0aabc python3.12_PyRun_AnyFileObject + 112 frame #25: 0x0000000104f0f460 python3.12PyRun_AnyFileExFlags + 208
frame #26: 0x0000000104f37ac8 python3.12Py_RunMain + 3552 frame #27: 0x0000000104f3859c python3.12pymain_main + 628
frame #28: 0x0000000104d1f9b4 python3.12main + 56 frame #29: 0x000000018ebbb154 dyldstart + 2476
˜˜˜

I am afraid the problem is the link to the python library (I remember some similar error in robostack):
(icubmaindev) icub@iCubs-Mac-mini site-packages % otool -L _icub_python.so
_icub_python.so:
	@rpath/libpython3.12.dylib (compatibility version 3.12.0, current version 3.12.0)
	@rpath/libiKin.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/libskinDynLib.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/liboptimization.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/libctrlLib.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/libYARP_math.3.dylib (compatibility version 3.0.0, current version 3.9.0)
	@rpath/libYARP_sig.3.dylib (compatibility version 3.0.0, current version 3.9.0)
	@rpath/libYARP_os.3.dylib (compatibility version 3.0.0, current version 3.9.0)
	@rpath/libc++.1.dylib (compatibility version 1.0.0, current version 1.0.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1345.100.2)
(icubmaindev) icub@iCubs-Mac-mini site-packages % otool -L _yarp.so
_yarp.so:
	@rpath/libYARP_dev.3.dylib (compatibility version 3.0.0, current version 3.9.0)
	@rpath/libYARP_init.3.dylib (compatibility version 3.0.0, current version 3.9.0)
	@rpath/libYARP_sig.3.dylib (compatibility version 3.0.0, current version 3.9.0)
	@rpath/libYARP_os.3.dylib (compatibility version 3.0.0, current version 3.9.0)
	@rpath/libc++.1.dylib (compatibility version 1.0.0, current version 1.0.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1292.0.0)

@conda-forge-admin, please rerender

I am afraid the problem is the link to the python library (I remember some similar error in robostack):
(icubmaindev) icub@iCubs-Mac-mini site-packages % otool -L _icub_python.so
_icub_python.so:
	@rpath/libpython3.12.dylib (compatibility version 3.12.0, current version 3.12.0)
	@rpath/libiKin.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/libskinDynLib.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/liboptimization.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/libctrlLib.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/libYARP_math.3.dylib (compatibility version 3.0.0, current version 3.9.0)
	@rpath/libYARP_sig.3.dylib (compatibility version 3.0.0, current version 3.9.0)
	@rpath/libYARP_os.3.dylib (compatibility version 3.0.0, current version 3.9.0)
	@rpath/libc++.1.dylib (compatibility version 1.0.0, current version 1.0.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1345.100.2)
(icubmaindev) icub@iCubs-Mac-mini site-packages % otool -L _yarp.so
_yarp.so:
	@rpath/libYARP_dev.3.dylib (compatibility version 3.0.0, current version 3.9.0)
	@rpath/libYARP_init.3.dylib (compatibility version 3.0.0, current version 3.9.0)
	@rpath/libYARP_sig.3.dylib (compatibility version 3.0.0, current version 3.9.0)
	@rpath/libYARP_os.3.dylib (compatibility version 3.0.0, current version 3.9.0)
	@rpath/libc++.1.dylib (compatibility version 1.0.0, current version 1.0.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1292.0.0)

Solution provided in https://github.com/robotology/icub-main/pull/987 .

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to rerender for you, but it looks like there was nothing to do.
This message was generated by GitHub actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/11160853995.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.


Supporting Multiple Push SDKs?

Is there support for several push notification SDKs?
(In my scenario I need to use it with Firebase)

https://docs.oracle.com/en/cloud/saas/marketing/responsys-develop-mobile/react/react-multiple-sdk.htm


How to launch the training at the first time to generate the train_temporal_windows_{T}_{gap}.json

How to launch the training at the first time to generate the train_temporal_windows_{T}_{gap}.json?
I try to run "python -m torch.distributed.launch --nproc_per_node 4 ddp_train.py --data_root=/data/xuanda/data/HO3D_v2/ --model "deformer" --train_batch=1 --test_batch=3 --workers 12 --T 7 --gap 10 --transformer_layer 3 --temporal_transformer_layer 3 --lr 1e-5 --motion_dis_lr 1e-3 --motion_discrimination --loss_base "maxmse" --snapshot 5 --host_folder=experiments/ho3d_train --run_name ho3d_train".
Then I still get "AssertionError: File does not exists: ho3d-process/train_temporal_windows_7_10.json".

Good catch, thanks for spotting the issue. We fixed it by adding the preprocess script, could you please try the latest commit (62d8b21)?


remove cdn from explanation dashboard

removes the use_cdn flag and related code from explanation dashboard in order to simplify maintenance.  Users who continue to pass use_cdn flag will see a warning that it has been deprecated.

#up


Fix katalog tiny height

A picture tells a thousand words
Before this PR
Small katalog
After this PR
Big katalog?

Coverage remained the same at 42.784% when pulling ce8da8ec63dc64ad904306981a5e4b7c27d12c9b on fixKatalogTinyHeight into a64542464d68e5fcbf94c5d307e484f3526137c3 on master.


Add relationship information to contacts

Feature Description:
Currently, we are able to store contacts, but we want to be able to distinguish between clients/caseworkers/coworkers/ etc.
Proposed Solution:
Add a couple more fields to each contacts entry that contains information on the relationship.
{
  person: <webId>, //required: the webId of the connected person
  pod: <podUrl>, // optional: the person's preferred pod url. If not present, we fetch the first pod url on the profile document. If that's not present, we derive it from the webid
  relationship: <client / case manager / associated organization>, // optional: an enum value describing how the user is related to this person
  relationship_status: <active / archived / etc>, // optional: an enum describing if the user is actively working with the person, or if this is a dormant or past relationship
}

I'd be interested in this


Update MSMPI to version 8.1

Additionally I've improved the error message in the case where the installed redistributable package version does not match the version in vcpkg. This should resolve #1375

Nice! Thanks!


Can not sign with package qdigidoc and eID card

Hello,
please find below a bug report about the Nix package qdidgidoc:
Description
I can not sign any more since the  24.5 update.
Error message
Failed to sign container. Please check the access to signing services
and network settings.
ASiC_E.cpp:371 Failed to sign container.
Connect.cpp:95 Failed to connect to host: 'dd-at.ria.ee:80'
DECODER routines:0 error:1E08010C:DECODER routines::unsupported
BIO routines:0 error:10080002:BIO routines::system lib
Additional tests
I also checked if I am able to authenticate with https://web-eid.eu/ .
Both, authentication and singing works.
Diagnostic output
For completeness, here my diagnostics output:
Locale: English / en_US.UTF-8
Application version: <IP_ADDRESS> (64 bit)
OS: NixOS 24.05 (Uakari) (x86_64/x86_64)
CPU: AMD Ryzen 5 PRO 4650U with Radeon Graphics
Kernel: Linux 6.6.44 #1-NixOS SMP PREEMPT_DYNAMIC Sat Aug 3 06:54:42
UTC 2024 x86_64
Libraries:
QT (5.15.14)
OpenSSL build (OpenSSL 3.0.14 4 Jun 2024)
OpenSSL current (OpenSSL 3.0.14 4 Jun 2024)
Arguments: qdigidoc4
Library paths: /run/wrappers/lib/qt-
5.15.14/plugins;/home/getreu/.local/share/flatpak/exports/lib/qt-
5.15.14/plugins;/var/lib/flatpak/exports/lib/qt-
5.15.14/plugins;/home/getreu/.nix-profile/lib/qt-
5.15.14/plugins;/nix/profile/lib/qt-
5.15.14/plugins;/home/getreu/.local/state/nix/profile/lib/qt-
5.15.14/plugins;/etc/profiles/per-user/getreu/lib/qt-
5.15.14/plugins;/nix/var/nix/profiles/default/lib/qt-
5.15.14/plugins;/run/current-system/sw/lib/qt-
5.15.14/plugins;/nix/store/h8m7pcd6d20jsx7hn81rsgkq0h884893-qtwayland-
5.15.14-bin/lib/qt-
5.15.14/plugins;/nix/store/vc9wklk5w3cskkymp52g7js83my4d9ch-qttools-
5.15.14-bin/lib/qt-
5.15.14/plugins;/nix/store/plix80gx31963j2qpsws7rid1gfr6cyn-
qtdeclarative-5.15.14-bin/lib/qt-
5.15.14/plugins;/nix/store/h32d5q8ngcy68nddbw5vja3r316s20p3-qtsvg-
5.15.14-bin/lib/qt-
5.15.14/plugins;/nix/store/fbddznz5ln765jbl2i2mfnf8h7rkkmd9-qtbase-
5.15.14-bin/lib/qt-
5.15.14/plugins;/nix/store/h4li58h9pni11wigz8xldad9kgzc0q3c-qdigidoc-
4.5.1/bin
URLs:
CONFIG_URL: https://id.eesti.ee/config.json
SID-PROXY-URL: https://dd-sid.ria.ee/v1
SIDV2-PROXY-URL: https://dd-sid.ria.ee/v1
SID-SK-URL: https://dd-sid.ria.ee/v1
SIDV2-SK-URL: https://dd-sid.ria.ee/v1
MID-PROXY-URL: https://dd-mid.ria.ee/mid-api
MID-SK-URL: https://dd-mid.ria.ee/mid-api
RPUUID: is set by default
TSL_URL: https://ec.europa.eu/tools/lotl/eu-lotl.xml
TSA_URL: http://dd-at.ria.ee/tsa
SIVA_URL: https://siva.eesti.ee/V3/validate
CDOC2:
CDOC2-DEFAULT: false
CDOC2-USE-KEYSERVER: true
CDOC2-DEFAULT-KEYSERVER: ria-test
TSL signing certs:
Patrick Kremer (Signature)
European Commission
European Commission
JEROEN ARNOLD L RATHE
APOSTOLOS APLADAS
CONSTANTIN-ADRIAN CROITORU
TSL cache:
EE.xml (65)
eu-lotl-pivot-335.xml (335)
eu-lotl-pivot-341.xml (341)
eu-lotl.xml (343)
g0xdp6w34ric1mdh8g7r0v8h85idkcg1-eu-lotl-pivot-300.xml (300)
Central Configuration:
Smart Card service status: Running
Smart Card readers:
Alcor Micro AU9540 00 00 max APDU size 65536
Reader state: PRESENT, INUSE
ATR cold - 3BD...
ATR warm - 3BD...
AID35: 6a86
UPDATER_AID: 6a86
AID_IDEMIA: 9000 (OK)
ID - 3...
USB info:
Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub
Bus 002 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub
Bus 002 Device 002: ID 5986:2130 Bison Electronics Inc. Integrated
Camera
Bus 003 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub
Bus 004 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub
Bus 004 Device 002: ID 058f:9540 Alcor Micro Corp. AU9540 Smartcard
Reader
Bus 005 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub
Bus 006 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub
Bus 006 Device 002: ID 0489:e0cd Foxconn / Hon Hai MediaTek Bluetooth
Adapter
Bus 007 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub

Add a :+1: reaction to issues you find important.

I can reproduce. The "Signing" animation is shown for a while, and then an error too:
The error message mentions dd-at-ria.ee not being reachable:

This hostname does not even have a DNS record.
I spotted https://github.com/open-eid/libdigidocpp/commit/2b5db855ba3ceb9bae1f11589ea1aea22bb7595a#diff-1e7de1ae2d059d21e1dd75d5812d5a34b0222cef273b7c3a2af62eb747f9d20aR25 changing the "Default TSA URL".
Maybe they updated the URL in some places, but forgot updating elsewhere?
I found this URL in my qdigidoc settings too:

Changing it to the new URL unfortunately also didn't help, as it doesn't like the Digicert certificate.
Maybe unrelated, maybe not - I also see a 2024-08-13T23:10:20Z E [TSL.cpp:302] - TSL eu-lotl.xml signature is invalid in my logs.
This looks like an issue in the application itself / some config shipped somewhere, but nothing NixOS-specific. I propose opening an issue upstream in qdigidoc, maybe linking to here for context.

I tried manually bumping libdigidocpp, and I tried using digidoc-tool from libdigidocpp. It mentions a signature to be invalid:
❯ /nix/store/802v0dcr9b7vsb1l1vi18fc98l2b7bqj-libdigidocpp-unstable-2024-07-17-bin/bin/digidoc-tool create --file=CONTRIBUTING.md contributing.asice
Version
  digidoc-tool version: <IP_ADDRESS>
  libdigidocpp version: <IP_ADDRESS>
2024-08-14T07:39:11Z E [TSL.cpp:311] - TSL eu-lotl.xml signature is invalid
2024-08-14T07:39:12Z I [X509CertStore.cpp:63] - Loaded 0 certificates into TSL certificate store.
Available certificates:
  label: XXXX
Selected:
  label: XXXX
Please enter PIN for token 'XXXX' or <enter> to cancel:
2024-08-14T07:39:16Z W [TSL.cpp:126] - Failed to parse TSL  /home/flokli/.digidocpp/tsl/EE.xml: /home/flokli/.digidocpp/tsl/EE.xml:1:2 error: invalid document structure
2024-08-14T07:39:16Z E [TSL.cpp:311] - TSL EE.xml signature is invalid
2024-08-14T07:39:16Z I [X509CertStore.cpp:63] - Loaded 58 certificates into TSL certificate store.
    Validation: OK

Opened an issue upstream: https://github.com/open-eid/DigiDoc4-Client/issues/1281

@getreu I got a link to https://github.com/open-eid/DigiDoc4-Client/issues/1276, mentioning a workaround.
However it looks like the libdigidocpp changes linked there are not a sufficient fix, so I cannot patch this in nixpkgs, and it for now requires each user to manually set that address and import certificates.

As I understand Update OpenSSL 3.0.14 by metsma · Pull Request #597 · open-eid/libdigidocpp now fixes this issue with the related commit: Uppdate OpenSSL 3.0.14 (#597) · open-eid/libdigidocpp@2b5db85.
Do you consider a patch release?

Unless I'm missing something, https://github.com/open-eid/libdigidocpp/commit/2b5db855ba3ceb9bae1f11589ea1aea22bb7595a#diff-1e7de1ae2d059d21e1dd75d5812d5a34b0222cef273b7c3a2af62eb747f9d20aR25 is the only relevant change for us from this commit (which sets the TSA_URL flag).
The patch doesn't apply, but IIRC I tried setting this manually via cmakeFlags and it didn't fix things for me back then.
I can try again tomorrow or so, re-rolling that patch and trying to sign again with a fresh profile.

Do you consider a patch release?

Yes, absolutely, if there's a patch we can apply to fix this for nixpkgs users, at least when starting with empty state I'd like to do it.

I did apply the TSA_URL change in libdigidocpp, and it seems this was enough to get signing to work.
PR is up at https://github.com/NixOS/nixpkgs/pull/345532, please test and report back there.

I installed the patched version and still get an error when signing:
Failed to sign container. Please check the access to signing services and network settings.
Details:
ASiC_E.cpp:371 Failed to sign container.
Connect.cpp:152 Failed to create ssl connection with host: 'eid-dd.ria.ee:443'
SSL routines:0 error:0A000086:SSL routines::certificate verify failed

Did you reset your settings to the defaults / wipe application state?

I deleted the ~/.digidocpp directory and I reset the settings. Both did not help.

And this is a $(nix-build -A qdigidoc)/bin/qdigidoc from a current nixpkgs checkout? What's the exact store path and nixpkgs commit, so I can try to repro?

You're using /nix/store/h3dn7hm1418sn02jbfnrbzm0dz673wad-qdigidoc-4.5.1, as can be seen all the way in the right of the library path. I don't know which nixpkgs checkout this is coming from.
I confirmed I also cannot sign with this, but I am able to by using one built straight from nixpkgs:
git clone https://github.com/NixOS/nixpkgs/
cd nixpkgs
$(nix-build -A qdigidoc)/bin/qdigidoc

That's /nix/store/nywycvjm6rrw3ghpm5gmvr5rrlfyr11k-qdigidoc-4.5.1 for me (you can also nix-store -r it). Please try executing qdigidoc from there (wiping ~/.digidocpp before, and making sure no other qdigidoc is running).

Do you have some funny MITM proxy between your connection with eid-dd.ria.ee:443?
Even though I cannot reproduce your store path (and still not know which nixpkgs revision you're on) it suggests https://github.com/presto8/nixpkgs/commit/6fe31b59dda4539905e16b4f287f32f2c0f41016 is applied. At least if the config is indeed cleared / reset to defaults.

I compiled qdigidoc following your instructions after cloning nixpkgs. The artifacts are in ./nixpkgs/result/bin/qdigidoc and ./nixpkgs/result/share/... and therefore have no hash in their storage path.

No proxy. In Firefox I can reach https://eid-dd.ria.ee/. It returns an empty white page.

Check the location of the result symlink, it points back into the Nix store. Also, the rev in git show, so I know which nixpkgs commit you're on.

The symlink points to /nix/store/nywycvjm6rrw3ghpm5gmvr5rrlfyr11k-qdigidoc-4.5.1
git show
commit 146e83d76bd8378bf56c0db8f9a19573e6e74c5d (HEAD -> master, origin/master, origin/HEAD)
Merge: 8af94f7ca2b8 611b1d53b74a
Author: Florian Klink<EMAIL_ADDRESS>Date:   Tue Oct 8 13:09:11 2024 +0300

    dhcpcd: enable sandboxing options (#208780)

Can you check out this PR? https://github.com/NixOS/nixpkgs/pull/350685
It bumps libdigidoc and qdigidoc to their latest versions.

Deleted my config in ~/.digidocpp then I compiled your patch, it did not help:

Please find attached all versions and the hashes of the dependencies in the diagnostics log:
qdigidoc4_<IP_ADDRESS>_diagnostics.txt

I really don't understand what's going on here, but it looks like it's either something with your network connection or configuration of which certificate roots you trust. Or maybe an incompatible combination of library versions?
We could at least rule out the library combination bits, can you $(nix-store -r /nix/store/2kln9y140vya6iqd9r2akjk0frm183av-qdigidoc-4.6.0)/bin/qdigidoc4 (that's qdigidoc from current nixpkgs master) and check if that allows you to sign?

The path is right. It opens Digidoc 4. Still the same error: "Failed to create SSL connection..." See screenshot above.
This is dumped to the console:
$(nix-store -r /nix/store/2kln9y140vya6iqd9r2akjk0frm183av-qdigidoc-4.6.0)/bin/qdigidoc4
warning: you did not specify '--add-root'; the result might be removed by the garbage collector
Failed to parse public key
QObject: Cannot create children for a parent that is in a different thread.
(Parent is QSigner(0x2d0d5160), parent's thread is QThread(0x2cf7c920), current thread is QSigner(0x2d0d5160)
Loading: "opensc-pkcs11.so"
2024-10-29T20:12:12Z I [Container.cpp:122] - Libxml2 version: 2.13.4
2024-10-29T20:12:12Z I [Container.cpp:123] - Xmlsec1 version: 1.3.5
2024-10-29T20:12:12Z I [Container.cpp:124] - digidocpp version: <IP_ADDRESS>
"OpenSC Project                   (2.20)"
 "OpenSC smartcard framework       (0.25)"
 Flags: 0
2024-10-29T20:12:13Z E [TSL.cpp:275] - TSL eu-lotl.xml signature is invalid
2024-10-29T20:12:13Z I [X509CertStore.cpp:61] - Loaded 0 certificates into TSL certificate store.
TSL loading finished

Note: "TSL eu-lotl.xml signature is invalid" and "Loaded 0 certificates into TSL certificate store."
BTW: I am able to connect:

Fwiw, I can sign with ID card now: see https://github.com/open-eid/DigiDoc4-Client/issues/1281#issuecomment-2453452094
But I still can't sign with smart ID.

SmartID is tracked in https://github.com/NixOS/nixpkgs/issues/307927, let's keep this issue on topic.

Fwiw, the certificate got updated recently, again.
I ran this command to update:
$ openssl s_client -showcerts -connect eid-dd.ria.ee:443 </dev/null 2>/dev/null|openssl x509 -outform PEM > eid-ts.pem

@hellwolf Could you elaborate a bit?
How can I update the certificate to solve my signing problem?
Concering NixOS: what package needs to be updated?

@hellwolf Could you elaborate a bit? How can I update the certificate to solve my signing problem? Concering NixOS: what package needs to be updated?

$ openssl s_client -showcerts -connect eid-dd.ria.ee:443 </dev/null 2>/dev/null|openssl x509 -outform PEM > eid-ts.pem

go to the settings of qdigidoc4 -> signing services ->  use manually configured access -> select the certificate you obtained via step (1).

settings ->

@hellwolf : It works! Thank you a lot!
Your solution with screenshot:

Download certificate:
 $ openssl s_client -showcerts -connect eid-dd.ria.ee:443 </dev/null 2>/dev/null|openssl x509 -outform PEM > eid-ts.pem

Goto:
 Settings of qdigidoc4 ->
 signing services ->
 (Access to Time-Stamping service)
 use manually configured access ->
 [ADD CERTIFICATE] ->
 select the certificate you obtained via step (1).

I don't know if/where qdigidoc does pin this certificate, and why this keeps failing. Users on other distros clearly don't need to manually do this config change every time, so it'd be good to understand what goes wrong in our case and how to get it to work without manual user intervention.

https://src.fedoraproject.org/rpms/qdigidoc/blob/rawhide/f/sandbox.patch
I found this patch used by the rpm packaged by fedora.

Yeah, they also just vendor in the XML file essentially. Maybe we can get a stable permalink from there, so we can avoid vendoring it in nixpkgs, but I'd also assume it'd compress sufficiently and changes rarely. Let's see if it fixes the issues.

I'd also assume it'd compress sufficiently and changes rarely

Yes, I think so also. Besides, shipping Digidoc with default well known certificates, as e.g. Fedora does (see @hellwolf), is far more secure than letting the Digidoc to download certificates them at runtime.

Fedora had to embed it because the build system doesn't check hash of the cert downloaded (during build).
The way we package in nixpkgs doesn't require that patch, since there is hash checks.

I also think the certificate rolling has one month period; so I guess it makes packing this package a bit more timely than otherwise during Oct/Nov.

I checked for the XML file on archive.org, It does regularly get updated, so we'd need to periodically keep these files refreshed, as does fedora.
I tried updating our packaging to do the same thing as fedora, and after a lot of back and forth ended up going with the same patch as them. PR at #357428, PTAL.


fix: potential buffer overrun

clang-cl compiler on Windows reports this error:
D:\Temp\lean_windows\util\shell.cpp(243,6): error C2440: 'initializing': cannot convert from 'const char [13]' to 'const char'
because this command line option is the full 12 chars reserved in struct option which leaves no room for the null terminator.
    {"print-prefix", no_argument,       &print_prefix, 1},

Could you please squash? For some reason, the GitHub "Squash and merge" is not available for this PR.

Does this help?
https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/configuring-pull-request-merges/configuring-commit-squashing-for-pull-requests

Still doesn't work. Could you please reduce the PR to a single commit without merge commits?

ok, I'm creating a new fork and pull request, see https://github.com/leanprover/lean4/pull/839


fix broken example code

Example code doesn't work this fixes
https://github.com/aurelia/documentation/issues/279

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Thanks for helping!


Unable to run the application Execution failed for task ':flutter_osm_plugin:kaptGenerateStubsDebugKotlin'.

Hi,
I am unable to run the application after adding flutter_osm_plugin: ^0.7.5+2 in to pubspec.yaml file. I also tried with fresh application. I am trying to run it on mobile
TO REPRODUCE

Create new project and add flutter_osm_plugin: ^0.7.5+2 in pubspec.yaml file and run the application.

Launching lib\main.dart on Mi 9T in debug mode...
Parameter format not correct -
Note: D:\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\location-4.1.1\android\src\main\java\com\lyokone\location\FlutterLocation.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
e: D:\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_osm_plugin-0.7.5+2\android\src\main\kotlin\hamza\dali\flutter_osm_plugin\FlutterInfoWindow.kt: (42, 56): Expecting an argument
e: D:\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_osm_plugin-0.7.5+2\android\src\main\kotlin\hamza\dali\flutter_osm_plugin\FlutterOsmPlugin.kt: (60, 53): Expecting an argument
e: D:\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_osm_plugin-0.7.5+2\android\src\main\kotlin\hamza\dali\flutter_osm_plugin\FlutterOsmPlugin.kt: (61, 23): Expecting an argument
e: D:\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_osm_plugin-0.7.5+2\android\src\main\kotlin\hamza\dali\flutter_osm_plugin\FlutterOsmPlugin.kt: (91, 34): Expecting an argument
e: D:\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_osm_plugin-0.7.5+2\android\src\main\kotlin\hamza\dali\flutter_osm_plugin\FlutterOsmPlugin.kt: (92, 19): Expecting an argument
e: D:\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_osm_plugin-0.7.5+2\android\src\main\kotlin\hamza\dali\flutter_osm_plugin\FlutterOsmView.kt: (95, 31): Expecting a parameter declaration
e: D:\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_osm_plugin-0.7.5+2\android\src\main\kotlin\hamza\dali\flutter_osm_plugin\FlutterOsmView.kt: (926, 52): Expecting an argument
e: D:\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_osm_plugin-0.7.5+2\android\src\main\kotlin\hamza\dali\flutter_osm_plugin\FlutterRoad.kt: (12, 38): Expecting a parameter declaration
e: D:\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_osm_plugin-0.7.5+2\android\src\main\kotlin\hamza\dali\flutter_osm_plugin\OsmFactory.kt: (18, 41): Expecting a parameter declaration
e: D:\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_osm_plugin-0.7.5+2\android\src\main\kotlin\hamza\dali\flutter_osm_plugin\OsmFactory.kt: (23, 24): Expecting a parameter declaration
e: D:\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_osm_plugin-0.7.5+2\android\src\main\kotlin\hamza\dali\flutter_osm_plugin\OsmFactory.kt: (31, 27): Expecting an argument
e: D:\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_osm_plugin-0.7.5+2\android\src\main\kotlin\hamza\dali\flutter_osm_plugin\network\ApiClient.kt: (17, 39): Expecting a parameter declaration
e: D:\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_osm_plugin-0.7.5+2\android\src\main\kotlin\hamza\dali\flutter_osm_plugin\utilities\FlutterPickerViewOverlay.kt: (10, 34): Expecting a parameter declaration

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':flutter_osm_plugin:kaptGenerateStubsDebugKotlin'.
> Compilation error. See log for more details

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 6m 0s
Exception: Gradle task assembleDebug failed with exit code 1
Exited (sigterm)

flutter doctor -v
Microsoft Windows [Version 10.0.19042.985]
(c) Microsoft Corporation. All rights reserved.

C:\Users\Usama Karim>flutter doctor -v
[√] Flutter (Channel stable, 2.0.6, on Microsoft Windows [Version 10.0.19042.985], locale en-PK)
    • Flutter version 2.0.6 at D:\Flutter\flutter
    • Framework revision 1d9032c7e1 (2 weeks ago), 2021-04-29 17:37:58 -0700
    • Engine revision 05e680e202
    • Dart version 2.12.3

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at D:\Flutter\Android
    • Platform android-30, build-tools 30.0.3
    • ANDROID_SDK_ROOT = D:\Flutter\Android
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 4.1.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] VS Code (version 1.56.2)
    • VS Code at C:\Users\Usama Karim\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.22.0

[√] Connected device (3 available)
    • Mi 9T (mobile) • ca0d513e • android-arm64  • Android 10 (API 29)
    • Chrome (web)   • chrome   • web-javascript • Google Chrome 90.0.4430.212
    • Edge (web)     • edge     • web-javascript • Microsoft Edge 89.0.774.75

• No issues found!

Hi,
follow this step

remove flutter_osm_plugin from pubspec, after that pub get
open android module in android studio ( right click in name of project -> flutter-> open android module in android studio)
update gradle version to 4.1.1 ( IDE will show popup to make update)
update kotlin version to 1.4.21 & re-build the project
re-add flutter_osm_plugin in pubspec , pub get ( or flutter clean & pub get )

and you need also to supoort multidex in your project
see this link   https://github.com/liodali/osm_flutter/issues/70#issuecomment-816638884

and you need also to supoort multidex in your project
see this link liodali/osm_flutter/issues/70#issuecomment-816638884

This one worked. Thanks for your support <3


Easier syntax for methods that just return a capture

A very common pattern in qdbp is:
value := expr
{
  Method[value]
}

i.e. evaluate an expression, save it into a variable, and then capture that variable and return it. It would be nice if we had some syntactic sugar for the pattern. Potentially something like
{
  Method := expr
}

Since . is used do indicate "evaluate this method right now," maybe we could have
object := {
  Method[body]
  Field[value].
}

Note the period after Field, indicating "we want to evaluate Field right now.


Add private constructor to utility classes to hide the implicit public one

See the issues flagged by sonar cloud
https://sonarcloud.io/project/issues?resolved=false&rules=java%3AS1118&severities=MAJOR&id=open-metadata_OpenMetadata

I can work on this issue.

Thanks @ozanuslan assigning it to you


Update existing charms to 1.4, including multi-user features

This PR implements multiuser support for kubeflow pipelines.  Included are:

setting kfp-api to multiuser and updating kf release 1.4
setting kfp-ui to multiuser and updating to kf release 1.4
adding charm for the kubeflow-pipelines-profile-controller

Will replace with another PR


Docker installation note

Added note for add user to docker group

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Compatibility with Analytics v4

As you may have seen, Analytics has a new version and the ID given doesn't look like the UA-000000000-1.
When I create a new website on Analytics, I have an ID like G-XXXXXXXXXX and a "Flux ID" that looks like 000000000.
Could you please confirm your package is / will be compatible with these identifiers ?
And which one of these two ID should I use ?
I guess your README should also be updated for that.
Thanks !

This seems to be a greater change. We are not able to support such an intense of work right now. Maybe you can do it yourself and make a PR - or you should look for another package.
Hope that answer helps.


Does basscss-forms assume .border-box?

The styles in basscss-forms seem to only play nice with margin and padding rules of the parent if each input is also given a .border-box class. Without it all .input elements are too wider for their container. .textarea and .select are also affected.
Am I missing something or should the forms module be setting border-box for us?

Yes, I should probably either mention that in the docs or just add box-sizing to the styles that depend on that.

@jxnblk I'll send a PR that adds box-sizing to them.


Unauthorize live subscriptions which are no longer authorized.

Currently RecordsSubscribe does not check if a subscription is valid beyond when it is processed. This means that while the subscription is live, if something changes that would make the subscription invalid (ex. delete of protocol role, or revocation of a grant) the subscriber will still receive updates until they close the connection.
We would like to proactively evict these subscriptions where possible, or potentially re-authorize on a TTL if evicting is too cumbersome.

Can you just drop any connections associated with a revoked subscription?

Can you just drop any connections associated with a revoked subscription?

Well the subscription itself is not revoked, but rather a user's protocol role may be deleted, or a grant which invalidates the subscription authorization.
But yeah that's the idea, is to stop emitting messages to the handler provided.
The dwn-sdk-js package itself keeps this transport agnostic, so the dwn-server itself will deal with dropping web socket connections if necessary.
Nothing too complex, just breaking it out into separate issues so that we can prioritize and chunk out subscriptions into smaller PRs.


Seach in the current line

General information

Terminal program: QTerminal 0.16.1
Operating system: Debian Bullseye
ZSH framework: none
ZSH version: 5.8
ZVM version: ?

Basic examination

[x] I have read through the README page
[x] I have the latest version of zsh-vi-mode
[x] I have tested with another terminal program

Problem description
I'm curious if a feature allowing to search in the current line using / or ? is feasible. Currently those key bind to command history search. I'm aware that it would be a big change, but I'm sure there's a way to combine the two (using :/; to search through history and //? to search in the current line).
There's a plugin that tries to solve the very problem: soheilpro/zsh-vi-search.

Hi @adrp
Thanks for the delay, I'm happy that you commit this issue. Currently, I'm busy, but I will work on this feature in the future. You can star this project for further update in the future. Hope you have a good day! : )
Thanks and Regards

it's 11/26/2023 now and this feature is still missing
Hum, when we say vim mode, isn't it natural to use '/' to start searching in current input? why bind it to history search?

I also except this feature.

I would also like to see this if possible searching in the current line.  I don't need this plugin to handle history search since I already can do that with fzf.


Improve Bvh abstraction

The Bvh is currently defined as
struct Bvh
{
    std::vector<BvhNode>   nodes;
    std::vector<Positions> positions;
    std::vector<std::size_t> positionIndices;
};

Is the positions attribute array strictly speaking required to be created as a part of the bvh construction? Would it be more natural for positions attribute array to be just like any other attribute array?

Closed in 39bf21003cfc6f9edf8a6e434a626960fee44901.


Bump version to 2.0.0

Signed-off-by: Joshua Li<EMAIL_ADDRESS>Description
bump plugin to 2.0
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #56 (decbdec) into main (daac784) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #56   +/-   ##
=======================================
  Coverage   95.30%   95.30%
=======================================
  Files           5        5
  Lines         149      149
  Branches       45       45
=======================================
  Hits          142      142
  Misses          7        7

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 daac784...decbdec. Read the comment docs.


fix(retry): Storing _eventsToRetry in OfflineRetryHandler

Summary
If my understanding of the idea behind OfflineRetryHandler is right then this PR fixes a bug regarding storing _eventsToRetry
Details
Events passed to _onSendError where only events from the current batch (flush). But OfflineRetryHandler tries to send events from the current batch concatenated with _eventsToRetry. As a result:

only events from the last batch had a chance to be stored in _eventsToRetry and after one attempt to send them again they were gone, no matter what was the result of the request.
I assume the line newEventsToRetry = events.filter((_, index) => !invalidEventIndices.has(index)); wasn't working properly as invalidEventIndices were computed based on events that were send (_eventsToRetry.concat(events)) and not events. Edit that's my assumption about how API works

Checklist

[x] Does your PR title have the correct title format?
Does your PR have a breaking change?:  no

We're actually using your NodeJS sdk in an Electron app. Interesting things pop out during the process...

Makes sense to me. I verified out that the existing offline handler only retries the most recently failed batch of events, and drops all previous ones. And this PR fixes that behavior. @jooohhn @kelvin-lu any thoughts?

published this change in v1.8.3