Question: How to query contract event messages?

I need to monitor the transfer events of our NFTs, and I've attempted several methods to retrieve this information:

explorerclient.GetAccountTxs: This method only returns the user's transactions and does not include contract transactions.

txsResp, err := explorerClient.GetAccountTxs(ctx, explorerPB.GetAccountTxsRequest{
	Address:    "inj1n96zr06eqttpef7prspntlc7uqdumyq4ugq23w",
	After:  19750950, // [after, before],
	Before: 19750951,
})
if err != nil {
	panic(err)
}
str, _ := json.MarshalIndent(txsResp, "", " ")
fmt.Println(string(str))

// output: {"paging": {}}

explorerPBClient.GetContractTxs: It only provides the total transaction count without detailed information.

client := explorerPB.NewInjectiveExplorerRPCClient(explorerClient.QueryClient())
txsResp, err := client.GetContractTxs(ctx, &explorerPB.GetContractTxsRequest{
	Address: "inj1n96zr06eqttpef7prspntlc7uqdumyq4ugq23w",
	FromNumber:   19750950, // [after, before],
	ToNumber:  19750951,
})

// output: {"paging": { "total": 3 }}

explorerclient.GetTxs: While this method returns all transactions, it requires manual parsing of tx.messages to filter out contract transactions, which is not an efficient solution for my needs.

txsResp, err := explorerClient.GetTxs(ctx, explorerPB.GetTxsRequest{
	After:  19750950, // [after, before],
	Before: 19750951,
})

// output: {"paging": { ... }, "data": { ... }}

Is there an easier way to fetch contract messages?

This is a question and not an issue. Questions should be posted in the support channel in Discord.


add nomadic indexer, following the model of tzkt

hardcode to latest version for now

Documentation should mention nomadic as another indexer option


Build & Publish matchbox_server on tagged release

Add a CI job to build a docker container containing matchbox_server and publish it to Github Container Registry (ghcr.io) on tagged releases

No worries, cheers for the merge. I've got the fun of doing a lot of the CI work in my day job so If there's anything else you'd like to do CI wise feel free to give me a message

Thanks, actually I had a little issue just now: it skipped all the checks on my other PR here: https://github.com/johanhelsing/matchbox/pull/74

Thanks, actually I had a little issue just now: it skipped all the checks on my other PR here: #74

Hmm, that's very odd, the workflow file used for this job seems to be the version prior to my change, but the skipped pull_request jobs imply the skips I implemented were in place. Is there any chance you could re-run the CI?

Tried re-running, but same thing. I also tried rebasing, but same result. Maybe the event name is pull_request?

The following section of the workflow definition should mean that whenever pull request is created the CI will be run against both the pushed commit and the pull_request (a somewhat unfortunate default behaviour)
on:
  push:
  pull_request:

The check below skips the checks due to pull_requests - allowing those where the head is a fork to run as the push event won't be shown
if: github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name != github.repository

You can see this in effect on my colleagues pull requests earlier today

Actually, I was a bit quick (about the run after rebasing). It actually ran the workflow twice, once for the push and once for the pull request.
So I think it worked, it only skipped jobs for the pull_request, and not for the push... I was simply jumped to a conclusion because I jobs starting to get marked as skipped.

Ah good! I was worried I was going to have a lot of repos to fix tomorrow if the behaviour had changed for some reason :sweat_smile:

I think perhaps it may be better to test on the pull_request event... as far as I understand the docs that actually does the merge before running the test, so it's a better indication as to whether the PR will break anything if merged.
The push event OTOH checks out the head of the branch.

That's a really good observation, I had never considered it as we always enforce linear histories. I wouldn't recommend disabling the push CI checks as they're a good indicator whilst working on a branch. Perhaps the best option is just to run both since public repo minutes are free?


Allow PrettyVersion 2.0

This simple bump allows for the 2.0 version of this lib, which leverages Composer 2 APIs and drops any sub dependency.
There are basically no BC; for more details, see the the changelog.

Thanks @Jean85!


[Plasma] Allow Plasma to send batched notifications.

This PR allows Plasma store to send batched notifications.

Codecov Report

Merging #5626 into master will increase coverage by 0.55%.
The diff coverage is 97.69%.

@@            Coverage Diff             @@
##           master    #5626      +/-   ##
==========================================
+ Coverage   88.92%   89.47%   +0.55%
==========================================
  Files         988      796     -192
  Lines      134220   118814   -15406
  Branches     1501        0    -1501
==========================================
- Hits       119353   106308   -13045
+ Misses      14502    12506    -1996
+ Partials      365        0     -365

Impacted Files
Coverage Δ

cpp/src/plasma/store.h
100% <ø> (ø)
:arrow_up:

cpp/src/plasma/client.h
100% <ø> (ø)
:arrow_up:

cpp/src/plasma/protocol.h
100% <ø> (ø)
:arrow_up:

cpp/src/plasma/plasma.h
100% <ø> (ø)
:arrow_up:

cpp/src/plasma/protocol.cc
94.38% <0%> (-0.43%)
:arrow_down:

cpp/src/plasma/store.cc
91.64% <100%> (+0.41%)
:arrow_up:

python/pyarrow/tests/test_plasma.py
96.23% <100%> (ø)
:arrow_up:

cpp/src/plasma/plasma.cc
57.89% <100%> (-11.34%)
:arrow_down:

cpp/src/plasma/test/client_tests.cc
99.17% <100%> (+0.04%)
:arrow_up:

python/pyarrow/_plasma.pyx
91.23% <100%> (+0.13%)
:arrow_up:

... and 199 more

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 929c9f6...92a2ab6. Read the comment docs.

@danyangz can you please open up a JIRA for this?  Will you be updating the java client to support this as well?


Add support for TLS

@FokkeZB any solution?

Hi @JosefJezek - Sorry, I don't work for TTN anymore.
/cc @johanstokking


ENH: Return dictionary of contrast images with "all" argument

This PR adds an output_type='all' option to FirstLevelModel.compute_contrasts, which causes the method to return a dictionary with each image, keyed by output type.
Closes #180.

Hey @effigies Thanks for this and apologies for leaving this in limbo for so long.
Would you do a pull from the master to update the TravisCI configs?

Thanks!

We should add this in the What's New.rst and then I will merge it.

@bthirion Review & Finalize but don't do any merges yet. I am exploring some merges on Nistats and if we should revert them and I don't want to complicate it if it comes to that.

This is good to merge and I will do so shortly.


Can not Create Tuna Record. Tuna record creation time is forever.

Tuna record creation time is forever.
Tried with both
with sawtooth-default.yaml from master
and with 0.8 images from this PR https://github.com/hyperledger/education/pull/12
During the creation of the "Tuna Record" call to
http://localhost:8080/batch_status?id=3b7062802b579ace1e1e89bb1af6b37eaee521026f46a3ed1a17db70aec38307539e7bba5d525c1ef8bfd96dafef396eae4499a0d5d30b2956db9293b807b3e3&wait"
returns:
data: [
{
id: "3b7062802b579ace1e1e89bb1af6b37eaee521026f46a3ed1a17db70aec38307539e7bba5d525c1ef8bfd96dafef396eae4499a0d5d30b2956db9293b807b3e3",
invalid_transactions: [ ],
status: "PENDING"
}
],
link: "http://localhost:8080/batch_status?id=3b7062802b579ace1e1e89bb1af6b37eaee521026f46a3ed1a17db70aec38307539e7bba5d525c1ef8bfd96dafef396eae4499a0d5d30b2956db9293b807b3e3&wait""
}

part of the log from dockers:

sawtooth-validator-default | [2017-12-30 22:17:47.925 DEBUG    selector_events] Using selector: ZMQSelector
sawtooth-validator-default | [2017-12-30 22:17:47.925 INFO     interconnect] Listening on tcp://eth0:8800
sawtooth-validator-default | [2017-12-30 22:17:47.925 DEBUG    dispatch] Added send_message function for connection ServerThread
sawtooth-validator-default | [2017-12-30 22:17:47.926 DEBUG    dispatch] Added send_last_message function for connection ServerThread
sawtooth-validator-default | [2017-12-30 22:17:47.927 INFO     chain] Chain controller initialized with chain head: 42cf152c(0, S:24d4c66e, P:00000000)
sawtooth-validator-default | [2017-12-30 22:17:47.927 INFO     publisher] Now building on top of block: 42cf152c(0, S:24d4c66e, P:00000000)
sawtooth-validator-default | [2017-12-30 22:17:47.929 DEBUG    publisher] Loaded batch injectors: []
sawtooth-client-default | creating key directory: /root/.sawtooth/keys
sawtooth-client-default | writing file: /root/.sawtooth/keys/root.priv
sawtooth-client-default | writing file: /root/.sawtooth/keys/root.pub
sawtooth-validator-default | [2017-12-30 22:17:54.502 DEBUG    interconnect] ServerThread receiving CLIENT_STATE_LIST_REQUEST message: 81 bytes
sawtooth-validator-default | [2017-12-30 22:17:54.502 DEBUG    interconnect] ServerThread sending CLIENT_STATE_LIST_RESPONSE to b'bffc3624c2ef4caf'
sawtooth-validator-default | [2017-12-30 22:18:02.420 DEBUG    interconnect] ServerThread receiving CLIENT_BATCH_SUBMIT_REQUEST message: 924 bytes
sawtooth-validator-default | [2017-12-30 22:18:02.422 DEBUG    executor] no transaction processors registered for processor type transfer-chain: 0.0: application/json
sawtooth-validator-default | [2017-12-30 22:18:02.423 INFO     executor] Waiting for transaction processor (transfer-chain, 0.0, application/json)

I can't get Create Tuna Record to work. I started the processor with the following text as the last message in the terminal: "Initializing JSON handler for Sawtooth Tuna Chain." Does that mean the processor is running? Any ideas what I can do to make this work?


when run  react-native run-ios,please tell me what should i do?

** BUILD FAILED **
The following commands produced analyzer issues:
Analyze /Users/vizard/WebstormProjects/AwesomeProject/node_modules/react-native/ReactCommon/yoga/yoga/YGNodeList.c
Analyze /Users/vizard/WebstormProjects/AwesomeProject/node_modules/react-native/ReactCommon/yoga/yoga/Yoga.c

(2 commands with analyzer issues)
The following build commands failed:
PhaseScriptExecution Install\ Third\ Party /Users/vizard/WebstormProjects/AwesomeProject/ios/Build/Intermediates/React.build/Debug-iphonesimulator/double-conversion.build/Script-190EE32F1E6A43DE00A8543A.sh
(1 failure)
Installing build/Build/Products/Debug-iphonesimulator/AwesomeProject.app
An error was encountered processing the command (domain=NSPOSIXErrorDomain, code=2):
Failed to install the requested application
An application bundle was not found at the provided path.
Provide a valid path to the desired application bundle.
Print: Entry, ":CFBundleIdentifier", Does Not Exist
Command failed: /usr/libexec/PlistBuddy -c Print:CFBundleIdentifier build/Build/Products/Debug-iphonesimulator/AwesomeProject.app/Info.plist
Print: Entry, ":CFBundleIdentifier", Does Not Exist

How did you solve this? I'm running into the same problem.

not yet

Same problem here

Facing same issue.
The following build commands failed:
PhaseScriptExecution Install\ Third\ Party ... /ios/build/Build/Intermediates/React.build/Debug-iphonesimulator/double-conversion.build/Script-190EE32F1E6A43DE00A8543A.sh
(1 failure)

Me too.
Same script as above: double-conversion.build/Script-190EE32F1E6A43DE00A8543A.sh

Fixed using earlier versions. Try this
react-native init --version="0.42.0" AwesomeProject

The following has fixed the double-conversion errors for me by removing ~/.rncache, cleaning the ios project(Product->Clean), cleaning the build folder(option + Product -> Clean Build Folder...) and kicking off the packager with npm start -- --reset-cache. I believe the problem is in cached files in ~/.rncache.

Same issue.
react-native-cli: 2.0.1
react-native: 0.46.4
Cannot use version lower than 0.45 as getting issue
Update RN version to 0.45
Tried clearing NPM cache, iOS project clean, but nothing works.

you should download boost_1_63_0 manually from web and put in react-native/third-party

Duplicate of #14423


engine: fix backward compatibility issue of raft-engine.recovery-mode

What problem does this PR solve?
Issue Number: close #11489
What is changed and how it works?
What's Changed:
Update raft engine to include https://github.com/tikv/raft-engine/pull/159, also add a test case.
Related changes
Check List
Tests

Unit test

Release note
None

I'm not sure about downloading file in a unit test, maybe there's a better way or I can just remove it.

/merge

/run-test


Marklogic, email and SSL oh my

Looks good. How was the mail part tested? With a local postfix?

Yes, local delivery only for testing so far.  That's all we need for FRDR.  I put the other code in there for you, in case you were testing on a machine that didn't have a local MTA.  Remote mail server with login and ssl wasn't tested yet.

It's throwing a lot of \python3\tools\lib\site-packages\requests\packages\urllib3\connectionpool.py:838: InsecureRequestWarning: Unverified HTTPS requestis being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/security.html for me...

Those have always been happening.... I thought the one commit was going to get rid of most or all of them, guess not

It could be in the CKAN or Sickle packages making secure requests.  What repo?

Currently testing with CORA + BC Data Catalogue + SFU Radar, and I'm seeing it more often than before if anything :) Not a big deal though

Yeah I'm switching all the repo URLs over to https where possible, in anticipation of some of them turning off their insecure requests.  So the volume will go up with that.

OK, testing is going well otherwise, merging!


Opcache should be disabled by default

Usually ends up being a pain for local development

So that explains why file changes often don't take effect immediately, +1 for this!

For my own curiosity, before we just blanket remove OPCache, can we tweak a setting which would tell OPCache to use the files timestamp for validation and see if we are still seeing the need for a reload?
@bencroker can you ssh into nitro (nitro ssh) and edit the file:
sudo vi /etc/php/7.4/fpm/conf.d/10-opcache.ini

Add the the following into the file:
opcache.validate_timestamps=1

Finally restart php-fpm using:
sudo service php7.4-fpm restart

If you are logged into your control panel, you should see
opcache.validate_timestamps      On

@jasonmccallister Without doing this, I see that opcache.validate_timestamps is already set to On.

OPCache does appear to be validating timestamps, as the changes are detected, however the changes are sometimes detected immediately and sometimes takes up to a few seconds to take effect, which is rather frustrating during development.

@bencroker you are right, can you instead add this setting:
opcache.revalidate_freq=0

and then restart php-fpm?

opcache.revalidate_freq changed from 2 to 0. This doesn't appear to have helped though. Could it be that when I edit a file, it takes some time for Nitro to detect and mount the modified file into Multipass?

Tested with a PHP file and changes take effect immediately.
So, this appears to have to do with compiled twig templates being cached. I've submitted a PR to Craft core which should help resolve this.
https://github.com/craftcms/cms/pull/6046
As per my tests, with the PR above, OPCache can remain enabled as it is.

What would the benefit be of having opcache available in a local dev environment?

It is faster and when @bencroker and I were troubleshooting, there was still a delay in the twig template rendering. His PR will resolve that when using devMode.

Right, I know opcache is faster but for anything caching, I'd want to opt-in for local development, I'd think?
Re: the PR @bencroker my question would be... why does this only show up when doing local dev with Nitro?

@khalwat Possibly because of the way Nitro mounts the directory, but that's more of a question for @jasonmccallister.

@khalwat I think it surfaced in Nitro but I have seem the exact same behavior when running Craft in Docker.

https://github.com/craftcms/cms/pull/6046 was rejected so this issue remains unresolved.

Twig determines whether a template needs to be recompiled based on its modify time, which is cached by OPcache. When you disable Twig “caching”, you’re disabling its ability to save the compiled templates as PHP files for future requests.
If we have to choose one, OPcache is the one that should be disabled, not Twig caching.

Hmm, would be interesting to add a custom loader that does not use a filesystem: https://twig.symfony.com/doc/3.x/api.html#create-your-own-loader

Either way, will make those changes for the next beta release.

We do use our own loader, but that wouldn’t solve this problem. That’s just for loading the actual template files, which are always going to be file-based, so will suffer from this problem as long as OPcache is enabled.

This will be released with beta 3.

@jasonmccallister Was this released with beta 3?

Hi @jskrivanek this was released with a beta of Nitro v1. We are actively working on Nitro V2 that is based on Docker instead of Multipass. If you are referring to v2 can you please create a new issue and describe any issues you are having?


Investigate Eurofins messages from Cue

Problem statement
Peter Boersma (CDC) asked us to investigate some messages from Cue:

I have two follow-up questions that I would like your help with.
Eurofins sent over 33 tests conducted as of March 3rd. I could identify 30 of the 33 in the ReportStream dataste. The below three testIds were not found within Protect. Could you possibly check and see if you see them anywhere in the pipeline?
3A9E3448-2DD7-4AA7-8A37-7A77F6384047
94A2CA65-4658-43DD-BF29-C4B351647F0C

The tests below have a report date of 3/3 (two days day after the fix), but are lacking the ‘OpET – CEI’ tag in the performingFacility field. Could you possibly check to see if there is any reason the tag is not coming through?
3D67CE8D-A5F1-4087-A529-1604AFFBF66E
A4A532AE-5377-4FFF-9BF8-9F0B1F20BE46
F8C16D96-FDDB-4097-A166-628037BA887E
0BF69C27-236F-45E7-9869-00D76374C403

Many thanks for all the help,
Peter

Email to Peter:
I checked for the 3 testIds and did not find them in ReportStream. This usually means they did not get to us. But if they did, and you have a message reply we sent back for any of them, that could help to further trace them through.
The 5 tests... these were interesting.  The code we have is looking for (and outputs) the exact string you have below, OpET – CEI. The message examples contain OpET - CEI. The difference being that the former uses an en-dash character (unicode 8211) and the latter uses a regular old dash (ASCII 45). Great Microsoft autoreplacement for pretty documents, not so great for consistency. Worse yet, in text editors, these characters look exactly the same.
So, solution time. I'm going to guess that some messages are getting to HHS and have the tag in performingFacility. The obvious fix is to look for either string and output just one of the two versions. Do you have a preference which version we sent to HHS going forward (en-dash or dash)?


How to get current VPN state after connected and the app is killed or put in background

Thanks for creating such amazing project! With little effort I can connect to my VPN server.
After connected, then I killed my App and relaunched my App, it seems getCurrentState() always return 0 even it was connected. When my App is in the foreground, connect/disconnect/getCurrentState everything work as expected. Not sure this is expected result or maybe the way I used it was wrong. If you could shed some light on how to get the current VPN state in this case would be appreciated.
Thanks again!

With some help of googling, I think I found a solution. I will create a pull request.
Thanks!

Thank you very much I made other vpns as well unfortunately I cannot put
them as public but I can update this one I will update many fix and many
more features to this library.
Also any PR will be welcome
On Thu, Mar 11, 2021, 4:12 AM Jacky Wu @.***> wrote:

With some help of googling, I think I found a solution. I will create a
pull request.
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sijav/react-native-ip-sec-vpn/issues/9#issuecomment-796328433,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAUOCSD3D3IHKU73MEIYUZ3TC77YDANCNFSM4Y7DSUEA
.


Fix missing jsx-runtime in npm package

Initial checklist

[x] I read the support docs
[x] I read the contributing guide
[x] I agree to follow the code of conduct
[x] I searched issues and couldn’t find anything (or linked relevant results below)
[x] If applicable, I’ve added docs and tests

Description of changes
jsx-runtime.js is part of the exports, but it’s not included in the files to publish.

Released in 3.0.1!


Previews are not properly added on initial submit

I just validated this edit which creates a new asset: https://godotengine.org/asset-library/asset/edit/40
The resulting asset had (at the time of this writing) no previews: https://godotengine.org/asset-library/asset/9

Lemme guess - I'm missing yet another asset_id != -1 check :laughing:


Cant Change Creation date:

Describe the bug
Attempting to change the date results in "Invalid Date" appearing where the date is and saving new date results in "Transport error: Not an integer" appearing in a notification pop-up. No choice but to cancel with no date change.
Steps to reproduce
1: make a note
2: save it
3: go to edit the note
4: click on the date
5: select a different date -> Results in the words "Invalid Date" appearing where the date was and the calendar does not close. Even if you select the same date, this does not change.
6: Click Save -> Results in a notification box that says "Transport error: Not an integer"
7: no choice but to select cancel to get out of edit. -> Reults in the date remaining unchanged.
The version of Memos you're using.
Was true in 22.3, upgraded to 22.5 and it still happens (note: In settings it says 23.0)
Screenshots or additional context

Duplicate of https://github.com/usememos/memos/issues/3812


Refactor strings for HTML to escape the new line instead of appending text.

Change
'<div>'+
   ' Stuff'+
'</div>'+

To:
'<div> \
  Stuff \
</div>'

On second thought it seems to be easier to read as concatenation. It also enables JS comments when need be


Permanent thunderstorms when enabled

After having a permanent thunderstorm with purple lightning on my server, I pinpointed this mod as the issue. Could be an issue with this mod standalone, but it could also be a conflict issue. Here is the r2modman code for the modprofile that causes this issue: 0184d0dc-6abf-1f60-6813-94c7f30f6724

probably fixed with mistlands update


Support for multiple inductions in loop unroller

Adds support to unrolls loops with multiple induction variables. The condition remains that the each loop must only be bound by one induction variable.
for(int i = 0, j =2, k=0; i < 10; ++i, ++j, ++k)
Would be unrolled, however:
for(int i = 0, j =2, k=0; i < 10 && k < 10; ++i, ++j, ++k)
Will not be unrolled.

This making progress.  It can unroll one of the larger cases we want to get, but it then gives a validation error:

error: 5193: Loop header 932 is targeted by 0 back-edge blocks but the standard requires exactly one

I'll try to reduce it to a smaller test case that I can share.  It looks like a problem handling structured control-flow.

It looks like the OpLoopMerge for the loops are not being removed even though the loop are fully unrolled.

That sounds unusual, there are a few tests which check that the OpLoopMerge is removed. If you can send us a sample I'll take a look at it @s-perron

               OpCapability Shader
               OpMemoryModel Logical GLSL450
               OpEntryPoint Fragment %main "main" %o
               OpExecutionMode %main OriginUpperLeft
               OpSource GLSL 430
               OpDecorate %o Location 0
       %void = OpTypeVoid
          %6 = OpTypeFunction %void
        %int = OpTypeInt 32 1
     %int_n2 = OpConstant %int -2
      %int_2 = OpConstant %int 2
       %bool = OpTypeBool
      %float = OpTypeFloat 32
%_ptr_Output_float = OpTypePointer Output %float
          %o = OpVariable %_ptr_Output_float Output
    %float_1 = OpConstant %float 1
       %main = OpFunction %void None %6
         %15 = OpLabel
               OpBranch %16
         %16 = OpLabel
         %27 = OpPhi %int %int_n2 %15 %26 %18
         %21 = OpSLessThanEqual %bool %27 %int_2
               OpLoopMerge %17 %18 Unroll
               OpBranchConditional %21 %22 %17
         %22 = OpLabel
         %23 = OpLoad %float %o
         %24 = OpFAdd %float %23 %float_1
               OpStore %o %24
               OpBranch %18
         %18 = OpLabel
         %26 = OpIAdd %int %27 %int_2
               OpBranch %16
         %17 = OpLabel
               OpReturn
               OpFunctionEnd

Here is an example,  I don't think it has to do with this change.  The problem seem to be when the loop exit condition is in the same block as the OpLoopMerge.

@s-perron Ok, pushed a fix. Seems issue is that there are temporarily two branch instruction in the condition block, and since the merge instruction is found using tail()-- it just finds the other branch. Because we needed to keep some instructions around in the pass we have a vector of instructions marked for future deletion and delete them at the end. This isn't necessary for the conditional blocks branch however so I just changed it to delete that on the spot.
I'll integrate that sample into the regression tests and will add some missing asserts too.

Not that it has the order correct, I cannot find anything else I want changed.

Merging now.

@davors72 the branch is not merging cleanly with master, could you rebase and squash against master? I'll push the result.

Done @dnovillo

Thanks!  Running final tests and merging.

Re-tested and pushed as e354984b09624c7fa2200e40555c98ca1a8fa5c6.


Use flag WITHDIST

Add a way to use the flag "WITHDIST", i mean return a distance of the returned items from the specified center.
GEORADIUS

This sounds like an issue for Node Redis.

Or were you thinking it would be nice to return the Distance when you do a GEO search with Redis OM? If that's the case, I'll mark this as an enhancement request.


Add CI Action with Spell Check

merge #63 and #67 first

Nice!

@hf-kklein, do you think we should enforce codespell with a dedicated gh-action?

=> #73


Ascend/qgc edits

Please use PX4 Discuss or Slack to align on pull requests if necessary. You can then open draft pull requests to get early feedback.
Describe problem solved by this pull request
A clear and concise description of the problem this proposed change will solve.
E.g. For this use case I ran into...
Describe your solution
A clear and concise description of what you have implemented.
Describe possible alternatives
A clear and concise description of alternative solutions or features you've considered.
Test data / coverage
How was it tested? What cases were covered? Logs uploaded to https://review.px4.io/ and screenshots of the important plot parts.
Additional context
Add any other related context or media.

Codecov Report

:exclamation: No coverage uploaded for pull request base (voxl-dev@17596b0). Click here to learn what that means.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             voxl-dev       #9   +/-   ##
===========================================
  Coverage            ?   50.23%
===========================================
  Files               ?      722
  Lines               ?    56804
  Branches            ?        0
===========================================
  Hits                ?    28536
  Misses              ?    28268
  Partials            ?        0

Flag
Coverage Δ

mavsdk
34.33% <0.00%> (?)

unittests
35.07% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 17596b0...072f81b. Read the comment docs.


NM: Refresh NM client before querying devices

This create a segmentation fault locally but might be a good thing to do to fix problems with the vlan test, since the vlan device seems to be cached by the client even when it is already gone and cannot be retrieved in a different nmclient instance. I guess ideally the mainloop would run for longer to take care of this.
Signed-off-by: Till Maas<EMAIL_ADDRESS>

I just realized this myself with @thaller help this morning/noon.
But the refresh should not be done here... as this is in many times in the mainloop (we may call it from apply).
See #170 .

@thom311 this test get's libnm(?) to crash. Not sure if you like to debug it.

@tyll Yes, interested :) Can you help getting more information? Backtrace would be best. Or maybe run it under valgrind?
Als G_DEBUG=fatal-warnings is helpful. Actually, I recommend that you set that environment variable in all your tests, so that libnm/glib dumps core on all warnings. These kinds of warnings always indicate a bug somewhere.

Sent logs and core created with G_DEBUG=fatal-warnings,gc-friendly G_SLICE=always-malloc valgrind --num-callers=100 --log-file=/tmp/log pytest -vkk vlan to @thom311
OPnened a new PR to set G_DEBUG=fatal-warnings in integration tests (#181).


Error getting access token for service account: 401 Unauthorized while calling refresh on Google Credentials

I am using a service account to access the google cloud features like Drive and Contact to save data through my app. I have suddenly started getting this for all users of some clients. Rest of them seem to be working fine.
"exception_class": "com.google.auth.oauth2.GoogleAuthException",
"exception_message": "Error getting access token for service account: 401 Unauthorized
POST https://accounts.google.com/o/oauth2/token, iss: XXXX.iam.gserviceaccount.com
...
Caused by: com.google.api.client.http.HttpResponseException: 401 Unauthorized
POST https://accounts.google.com/o/oauth2/token
    ... suppressed 3 lines
    ... 61 more

Was and still working fine for certain clients, failing for all users of the other client. What setting should I verify? Domain wide delegation is correct. Clients say they haven't changed any setting on their end.

Hi @nusbahcodes, thanks for reporting this issue. Could we get some code snippets and dependency versions to help us debug this issue? Specifically the versions of the auth library and client libraries (drive + contact).
At first glance, the token endpoint seems odd. I was expecting https://oauth2.googleapis.com/token, but it may by due to an older version of the auth library.

Lib versions:
google-auth-library-oauth2-http-1.23.0.jar
google-api-services-drive-v3-rev20221023-2.0.0.jar

Thanks, but I would need some code snippets to figure out how you're using the library. Would you be able to share that to help with debugging?

We have raised a support case for the same.


Expose Sampling llama.cpp implementation

Expose Sampling llama.cpp implementation
:recycle: Current situation & Problem
As of now, functions from the sampling.h header couldn't be called from outside entities as only the header file was included and exposed, not the underlying C++ implementation file.
:gear: Release Notes

Expose sampling functionality of llama.cpp to enable convenient sampling

:books: Documentation
--
:white_check_mark: Testing
Locally tested
Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

[x] I agree to follow the Code of Conduct and Contributing Guidelines.

Thank you @philippzagar & @Supereg 🚀


myworkplan when non-staff logged in

#1787

Show nothing in myworkplan when non-staff member logged in
Fixed filter select returning <SELECT_ALL> as selected value

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (develop@5911800). Click here to learn what that means.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #1788   +/-   ##
==========================================
  Coverage           ?   64.80%
==========================================
  Files              ?      251
  Lines              ?     8547
  Branches           ?        0
==========================================
  Hits               ?     5539
  Misses             ?     3008
  Partials           ?        0

Flag
Coverage Δ

epictrack-api
64.80% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


"limit" command blocks instead of limiting (Ethernet)

Greetings,
I'm fully aware that this question was asked before; What's special in mine is that the setup is different and it (the question) might turn into a feature request.
Great tool, by the way. It works perfectly for devices connecting via Wifi, the problem is when I try to limit the bandwidth of a device connected via Ethernet, it turns out to be blocking it, is this the behavior of the current version of the software? i.e. does it blocks devices connecting via ways other than 802.11 protocols?
If so, can we add support for devices connected via an RJ45 cable? 😊
Thanks in advance.

I solved it by disabling my firewall


Load in configuration when doing docker run

I typically use vmx to test some kind of automation. To do this I typically need to set certain interface configuration, add a user with ssh key, set some protocol options etc.
Being able to load this in when starting the container would be nice.

Do you typically have a configuration file that you want to merge with the running vrnetlab standard config? If that's the case then a possible solution that isn't model specific is to add functionality to config-engine-lite to specify --router and --config and the contents of --config would be merged with the running config on the --router. How about that?

The configuration applied by vrnetlab today is a very small configuration and essentially consists of:

IP connectivity
SSH and NETCONF
credentials for SSH and NETCONF (which you can change if you want)

We write this config over the serial console and to safely do this (we all know how serial console can be) it is an explicit goal to make that initial configuration small. We could potentially use CVAC (or the equivalent) to load the config from some file instead but for example, on XR it's not possible to generate crypto keys from a configuration - you have to login to the device, which is what we do: https://github.com/plajjan/vrnetlab/blob/master/xrv/docker/launch.py#L120. These kinds of things make it difficult to let vrnetlab support loading a generic config. I'm afraid that we will turn into a full-fledged configuration management system ourselves if we attempt it.
It is convenient to be able to specify a config file when starting the router but would @emjemj approach be close enough?
docker run --name my-router ......
config-engine-lite --router my-router --config my-config-file --wait-for-boot

The --wait-for-boot arg would make config-engine-lite block until it can reach the router and configure it (chances are normal timeouts would hit you way before otherwise).

I've open a pull request #114 that implements this single router configuration mode and the --wait-for-boot blocking. Will this solve your issue, @htj ?


Application Profile data when to show what

When a application is in draft what do we show to the application profile, inprogress, completed etc

[ ] When application is in any state, always show most up to date profile information
[ ] Schedulers will use most up to date data (ie profile). (with possible exceptions)

verified this is current behavior


Remove unused dispatchers

It seems that only server/opentsdb dispatchers are used so I thought I'd clean up this file.

Current coverage is 82.69%

Merging #269 into master will increase coverage by +0.10% as of 47124b0

@@            master    #269   diff @@
======================================
  Files           54      54
  Stmts         2971    2971
  Branches         0       0
  Methods
======================================
+ Hit           2454    2457     +3
  Partial          0       0
+ Missed         517     514     -3

Review entire Coverage Diff as of 47124b0

Powered by Codecov. Updated on successful CI builds.

Nice catch, some of those dispatchers are definitely a relic from a very old version. :+1:


Fix pokedex dump -l argument error (#295)

pokedex/main.py -
create_parser() -

Change the help message for the langs argument in the dump subparser
to show the actual default and state that the 'all' and 'none' codes
cannot be used alongside other codes.

command_dump() -

Check if 'all' or 'none' codes are passed alongside other codes. If
they are, error message is printed and program ends.

pokedex/db/load.py -
dump() -

Add check if langs code is 'all' or 'none'.
If langs wasn't passed to the parser or 'all' was passed (they are
the same since the default is 'all'), then everything will get
dumped to the csv files.
If 'none' was passed to the parser, then nothing new should be
dumped to the csv files.

pokexed/.travis.yml -

Re-added 'pokedex dump -l all' that was previously remove on
77e3d9df163ac602d5c939cee75750f8be0dc725

Resolves: #295

@magical Any update on this PR? I know it is failing the Travis CI build but that's only because the original dump command was giving the wrong result (which is what this PR fixes) so the git --no-pager diff --exit-code pokedex/data/csv/ script is what is failing.

@rluzuriaga Sorry. Thanks for the reminder. I'll take a look.
It seems to be failing because it is dumping the Czech translations where it wasn't before. Hmm. I do still want to test dump -l all but maybe we should follow that with a normal dump so that the diff test works.


iptables -P INPUT DROP kills the VPN, even if the listen port is allowed

Adding these rules to iptables:
-A INPUT -p tcp -m tcp --dport <port> -j ACCEPT
-P INPUT DROP

cause the VPN to stop working.
I'm not sure why. Setting -P INPUT ACCEPT makes it work again.
I'm a total iptables noob so what is going on here?

oops, looks like WireGuard uses UDP, not TCP!
-A INPUT -p udp -m udp --dport <port> -j ACCEPT

found by using -A INPUT -j LOG and looking in /var/log/kern.log for traffic on the WireGuard port


Change ticketsExist to boolean. Cast string to boolean

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


update example configuration of kafka sink in user guide

line 2833 change ki to k1

Hi @Robin021 ,
Thank you for your patch.
The fist commit looks good to me as well, but it seems that you accidentally pushed a new change to this branch/pull request.
If you are fine with it we can push the first commit to the main branch and close this PR.

Yes, please commit the first commit.
other changes are only customer use of my testing program , will update later.
thanks!


Can I create a middleware in Blazor Server App in order to have access to the users claims ?

I'm asking this question since I've read that the HttpContext is not available in server-side applications.
Thanks...

You can access the claims by injecting AuthenticationStateProvider ... for example
The hard part is getting OIDC auth working (which isn't impossible, fortunately, although it requires a lot of DIY).

Thanks for contacting us, @enetstudio.
@MV10 's suggestion is one way to do this. Hope that helps.
In general, the statement you're referring to is correct, you shouldn't depend on HttpContext in Blazor Server apps.

@mkArtakMSFT,  I know perfectly well how to handle and manipulate AuthenticationStateProvider and all the related components. I also believe I know that I can access the HttpContext  in middleware and access the user's claims, but only once when the SignalR connection starts, and then the HttpConext is no longer available... I need the assurance that I'm right... that it is doable and proper to access the HttpContext that way. Thanks...


Add button toggle example

Updated button class list according to a toggle props. Please let me know if it a right behaviour. As an alternative we can pass a value to the Button component using v-model and set button active state base on value.

Fixes #348


[rke2] Windows unable to pull registration command script in a proxied environment

Rancher Server Setup

Rancher version: v2.6.4-rc3
Installation option (Docker install/Helm Chart): helm

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): rke1

Proxy/Cert Details: self signed

Information about the Cluster

Kubernetes version: 1.21.9/1.22.6 (rke2)
Cluster Type (Local/Downstream): downstream

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider): custom + windows

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom): any

Describe the bug

when registering a windows node in a proxied setup (proxied rancher, proxied windows node) windows is unable to register. The script does not start to pull from rancher.
To Reproduce

deploy rancher in a proxied environment
deploy a proxied downstream custom cluster

register windows worker node in the cluster

Result

windows node is able to curl the rancher URL/other public URLs, however can't pull the script to register with the cluster.

Expected Result

windows should be able to pull the script in a proxied environment
Additional context

followed these instructions on windows node to get proxy setup
not sure if this will root cause as a rancher or windows issue.
setup available for debugging upon request

After bringing up another environment, windows team was able to find and debug the issue. @rosskirkpat will file the underlying bug(s) shortly.

@slickwarren Can you provide the Jenkins job(s) that you used to setup the environment?

This issue is very likely a duplicate and has already been QA'd.

tested using v2.6.5-rc1:
the reg. command is able to finish pulling and runs the script. However, it hangs just before running rke2. From the rancher-wins logs:
Message : Pulling image index.docker.io/rancher/system-agent-installer-rke2:v1.22.8-rke2r1

Message : Using private registry config file at /etc/rancher/agent/registries.yaml

Message : [Applyinator] Extracting image rancher/system-agent-installer-rke2:v1.22.8-rke2r1 to directory
          C:\var\lib\rancher\agent\work\20220418-221552\0ac288cdd81de8b67006f5dd3d6e0f79beba9fc0d24e9e3d71d590f5ea3905f0_0

Message : [Applyinator] Applying one-time instructions for plan with checksum 0ac288cdd81de8b67006f5dd3d6e0f79beba9fc0d24e9e3d71d590f5ea3905f0

Message : Detected first start, force-applying one-time instruction set

Message : Starting /v1, Kind=Secret controller

Message : Listening on \\.\pipe\rancher_wins

Message : Starting remote watch of plans

Message : Setting C:/var/lib/rancher/agent/work as the working directory

Message : Listening on \\.\pipe\rancher_wins_proxy

Message : Rancher System Agent version dev (HEAD) is starting

Message : Stackdump - waiting signal at Global\stackdump-3468

reopening for now. Please ping me as I still have a setup around for testing.

same issue as https://github.com/rancher/rancher/issues/36552#issuecomment-1110315090, blocking on this ticket

Once https://github.com/rancher/rancher/pull/37513 is merged and present in an RC, this is ready to be tested again.

@luthermonson is fixing a bug in the wins config file for whitelisting, which is requiring a v0.2.5 release. He will be coordinating a new PR and attach it to this issue.

https://github.com/rancher/rancher/pull/37527 is merged. Moving this to waiting on Rancher RC

tested on v2.6.5-rc6 - when registering the windows rke2 node (1.22.9+rke2r1) the cluster gives the following error: error applying plan -- check rancher-system-agent.service logs on node for more information, waiting for agent to check in and apply initial plan
Ross is currently investigating the issue.

Will be fixed by PR https://github.com/rancher/rancher/pull/37586

using rancher v2.6.5-rc8 there is an issue when trying to connect the windows node to rancher
Start-Service : Service 'Rancher Wins (rancher-wins)' cannot be started due to the following error: Cannot start service rancher-wins on computer '.'.

reopening for this issue (Ross is aware of the issue and is working on a fix currently)

This issue is ready to be tested again. There were two fixes that went in on Friday, one fixed a nil pointer in system-agent specific to windows and the other was improving the logic in wins for csi-proxy and tlsConfig

there was an issue found on v2.6.5-rc10 however the full fix won't be able to make it in 2.6.5. We have the following PRs:
https://github.com/rancher/rancher/issues/37688
https://github.com/rancher/rancher/pull/37684
that have been merged and will be part of 2.6.5 so that the OOB fix will work.

With the latest releases, this should go back to test.

tested on 2.6-head (c8d01c6) using k8s 1.23.6+rke2r2 and just realized that KDM has not yet been released to test this yet. Wednesday is the release for kdm.


udev : ID_INPUT_JOYSTICK property handling ?

SDL 2 uses the udev ID_INPUT_JOYSTICK property (https://hg.libsdl.org/SDL/file/704a0bfecf75/README-linux.txt) to find controller in event devices, and this is not set for the hid-wiimote controllers.
The rule
SUBSYSTEM=="input", DRIVERS=="wiimote", MODE="0666", ENV{ID_INPUT_JOYSTICK}="1"

is generic but causes problem because all devices created by hid-wiimote are recognized as hoystick, even the accelerometer of the IR (can break detection of the active controller).
I use
SUBSYSTEM=="input", ATTRS{name}=="Nintendo Wii Remote Classic Controller", MODE="0666", ENV{ID_INPUT_JOYSTICK}="1"

but it concern only one controller.
Maybe
SUBSYSTEM=="input", ATTR{name}=="Nintendo Wii Remote Classic Controller", MODE="0666", ENV{ID_INPUT_JOYSTICK}="1"

could be needed too for the input devices (I don't know where this flag should be set outside for SDL2).
Can/should hid-wiimote handle that in the driver ? Or maybe pushing a udev rule in "linux" (udev repo I guess ?) ?

Perhaps you could reconsider support for including the ID_INPUT_JOYSTICK property by default.
I think it's important to separate capability from use-case.  Personally, I feel that at the driver level it is important to present to the system the capabilities of the device, and not be concerned with the ultimate use-case that the device will be applied to.  Providing joystick input is a capability of the Classic Controller, being a gamepad is only a specific use-case.    If I hook it up to a midi control board it becomes a musical instrument.  Conversely, the SNES controller may be used as a gamepad, but it is definitely not capable of providing joystick input.
While a udev rule can be used to apply the ID_INPUT_JOYSTICK, I don't think it is a good idea to not include it be default.  When programming, if I query the input devices, looking for the specific type of input that a joystick provides*, the current driver does not provide any indication to me that the Classic Controller (Pro) is capable of providing that input, when in fact the physical device is entirely capable.  I feel the driver should try to best reflect the capabilities of the physical device, and ID_INPUT_JOYSTICK is a reasonable property to use to address this.
Finally, defining exactly what an input device "is" (joystick, touchpad, gamepad) is a bit of a hopeless task (again capability vs use-case) an argument could be made that the Classic Controller (Pro) "is" a joystick, in fact it could be argued that it actually "is" two joysticks.  Equally the Nunchuk attachment, it could be argued, "is" a joystick.  I personally am more concerned with the capability of
the device then the definition of it, but I wanted to point out that this is a reasonable argument to make.
*Note: For the sake of completeness my definition of joystick input is: Two absolute axis, offset 90° from each other and intersecting at a central point along their range, that allow for combined and simultaneous manipulation throughout a full 360° range of motion.
Also, thanks for providing this excellent driver, and extra thanks for not adding the ID_INPUT_JOYSTICK to the Wiimote itself.

Okay, so after having taken a deeper look into this issue, I'm beginning to see the complexities involved in the decision, and perhaps why things have turned out the way they have.  I still believe that, at current, triggering the ID_INPUT_JOYSTICK property would be a proper fit for the Classic Controller (Pro) and Nunchuk, but given the state of input device handling in Linux, I can understand the desire to leave things as they are.  Thankfully this library provides a solid foundation for building a custom uinput solution.
Thanks again.

There's an X11/Wayland developer conference next week, I can try to bring this up again. But I don't think anyone cares enough for legacy input handling to be fixed.
A lot of work has been done to overhaul input-handling for Wayland compositors. We now have libevdev for accessing the raw kernel APIs, we have libinput to interpret that data, and we have wayland APIs to transmit it to clients. The basic stuff has been implemented already (keyboard, mice, touchpad, drawpads), but other more special devices are still missing. Introducing gamepad support should be fairly straightforward, though.
However, this will not magically fix legacy applications like SDL. As long as they don't jump on the bandwagon and catch up with Wayland compositors, I doubt anyone will spend much time fixing their input handling. It's sad, but it's how it is.
Anyway, regarding xwiimote: I stopped working on it (and I should probably abandon this repository). I still maintain the kernel driver (hid-wiimote, which does all the heavy lifting), but the user-space xwiimote part is just an example how to deal with Wiimotes. And I think xwiimote should really be treated as just an example. That's also why I don't think shipping udev rules will change anything. If SDL cares, they can easily ship their own udev rules.

I subscribed this issue long ago because I encountered a similar issue with a non-wiimote device. I was using an 8bit FC30 gamepad in wired mode. ppsspp built with sdl2 won't detect key presses, while sdl1 does not use udev and works as expected.
I ended up with a custom udev rule similar with Alex's:
SUBSYSTEM=="input", ATTRS{name}=="8Bitdo FC30 GamePad", KERNEL=="event*", ENV{ID_INPUT_JOYSTICK}="1"

The thing is, I am not sure whether it is a hardware or driver issue, or a udev issue, or a sdl issue. And I became even more confusing after reading some comments. It looks like the issue is not related to Wayland/X11 to me. If we are not going to add ID_INPUT_JOYSTICK property, it seems sdl is probably the right place to fix this but I am not sure.


Fix the UI of the table on project resource page

Describe the task
Whenever, there is no such project found while searching a project the UI does not look good at the last column of the table. Hence, it needs to be fixed.
Attached a screenshot for the reference:

Expected behavior
Attached a screenshot for the reference:

@rahulpanwar16 Could you please check if this issue is reproducible on your local? It seems that there is no UI case for the project resource page in the project.

@rahulpanwar16 Could you please check if this issue is reproducible on your local? It seems that there is no UI case for the project resource page in the project. If this issue is already done by someone please close it.

Yes, I have checked the UI of page is fine and I am closing this issue.


Remove use of underscore-prefixed methods in generators

@robwise still an issue? If so, maybe @aaronvb can help you?

@robwise @justin808 Sure let me know if you need help with this

@aaronvb Awesome. Thanks!

@robwise Can you give me a little more information on this issue?

@aaronvb Oh, well we originally were following a style convention to name all of the methods that we didn't intend to be used outside of the class with and underscore (_) character. We've since changed our styles and we no longer do this. Therefore, we need to delete the underscores from all of the methods that have them in the generated javascript.
Basically, we have the opposite of this rule:
https://github.com/airbnb/javascript#22.4

@robwise I went through all the JS files in the generator template files and did not find any functions starting with an underscore. Just to be safe I ran a regular expression search on that directory and came up with nothing.

@aaronvb Roger, thank you for checking that. Someone must have hit this at some point so I'm closing.


Better tsconfig checker

[x] examine pkg jsons against tsconfigs
[x] flag any bad paths for references in tsconfig
[x] flag any missing refs
[x] flag any extra refs
[x] add npm script

wtf >300 file I only added 6 :roll_eyes:


d3js v4

retest this please

retest this please

retest this please

:shipit:

what wrong?

We use d3 for load content in attachments, look at face/src/components/attachment/AttachmentView.js. There have been any test in demo for check it?

summon @just-boris

LGTM from me

@sseliverstov thanks!


Give option to hide path and just show only file in debug preview in code

Type: Feature Request
Right now inside code is often long path depend project, but it's not required due project.. and just files are important.
Long path doing that user must watching end line where is file and line number maybe.
Give option to hide path and just show only file in debug preview in code, by this in preference is need option.
I'm still forcing needs to change view of this in code document field where is bug presented.
https://github.com/microsoft/vscode/issues/101532
Eg. if long path and small screen and editing line then icon as yellow around doing some quirky behavior where is hard to find space between point. Now we have 2 yellow icons as arrows (line number and next to word) where second is not needed maybe due col is indicated by red arrow to up. Soo next to line numbers should be icon to hide exception and then close exception (not at the end of name of exception eq. "Exception has occurred: TypeError - X" this must be X - Exception has occurred: TypeError
, then before / X Exception has occurred: TypeError
Also many reason to hide path.. eg. ctr+c/v on stock, GPT maybe and in case to paste here image of this case as bug/problem.. most of user must painting this.. but why..
For long path can be a tooltip also when file is focused.
VS Code version: Code 1.89.1 (dc96b837cf6bb4af9cd736aa3af08cf8279f7685, 2024-05-07T05:13:33.891Z)
OS version: Windows_NT x64 10.0.22631
Modes:

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.91.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Not clear what you're asking for. A screenshot of what you're seeing might help

it's clear.. eg. watch here.. to hide private data I must hide this:
https://github.com/microsoft/vscode/issues/101532#issue-649196278
https://user-images.githubusercontent.com/25403861/86278911-6ed56e80-bbd9-11ea-8339-a8537f364456.png

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


ClientModel: Non-freezable Options

We began looking at what it would look like to freeze and cache a pipeline instance and options in https://github.com/Azure/azure-sdk-for-net/pull/39740.  This would give us a large perf win by not needing to create a new pipeline instance in pipeline.Send when RequestOptions adds new policies into the pipeline.
Given the experimental nature of the cached-pipeline work, we'd like to know we have a "min-viable" solution for our Options types in case that approach doesn't work out.  This PR implements Options types following the same pattern as other ClientModel types with regard to Azure.Core 2.0 -- i.e. it moves complex functionality, of any, into base types in ClientModel, reuses them in Azure.Core types, and keeps APIs as similar as possible across the two libraries.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Core


Add additional tracing fields specific to Scylla

Introduction of some new struct fields that express the additional fields that are present in the
Scylla schemas for system_traces.sessions and system_traces.events.
Since I was unable to write a FromRow implementation that I was happy with in any form, the
provided implementation will perform conversion for only the fields present in both databases.
This therefore also stands for Session::get_tracing_info, where the documentation has been
updated.
Pre-review checklist

[x] I have split my patch into logically separate commits.
[x] All commit messages clearly explain what they change and why.
[ ] I added relevant tests for new features and bug fixes.
If there's any feedback about what kind of tests should be added for this type of change let me know and I'll do them.
[x] All commits compile, pass static checks and pass test.
[x] PR description sums up the changes and reasons why they should be introduced.
[x] I have provided docstrings for the public items that I want to introduce.
[x] I have adjusted the documentation in ./docs/source/.
[x] I added appropriate Fixes: annotations to PR description.

Hi @karlpvoss, could you tell me about the use case for this PR? As you wrote, this PR adds fields to the tracing-related structs, but never initializes them. Was that the intention, or do you need tips on how to fill these structs in case when connected to Scylla? If it's the former, then I'd rather not add fields that are unused - it's slightly confusing as users who won't carefully read the docstring for get_tracing_info or the docs.
Implementing it right now might be a bit convoluted, but doable, as you would need to first detect whether you are connected to Scylla or Cassandra and then use a query that fetches the columns supported by either Scylla or Cassandra. With better deserialization traits (https://github.com/scylladb/scylla-rust-driver/issues/462), you could SELECT * from the tables and then fill in the struct in the deserialization trait as you would get information about the names of the columns, though it's difficult for me to tell when the new deserialization traits will be implemented and merged.

@karlpvoss what's the status of this?


🚨 mull-cxx deprecation and eventual removal

Just a heads up: mull-cxx as a tool is deprecated and will be removed in a future release.
It is replaced by the IR Frontend.

It's gone https://github.com/mull-project/mull/pull/984

mull-cxx is removed as of 0.17.0 https://github.com/mull-project/mull/pull/988


React ref on header links

Theres a ref attribute on links in header, which seems unused.
Does this serve any purpose or was it forgotten?
webpackStats.css.map((css, i) => <link href={css} ref={i} media="screen, projection" rel="stylesheet" type="text/css"/>)

I think it's supposed to be key, as that's outputting an array of components. Although I think there is usually only one so react doesn't complain.

Thanks, @gonsfx.


Styling components with the style prp doesn't work in older browsers

Describe the bug
Because we rely on nested styles for this feature, the following browsers won't render styles correctly:

(Source: https://caniuse.com/css-nesting)
We need to instead flatten the CSS so that it will work everywhere.
Reproduction
Not relevant
Logs
No response
System Info
Not relevant

Severity
blocking all usage of opus ui

Fixed in version 1.1.7


set up getEventById endpoint

set up event/:id/get endpoint to retrieve a certain event's data

/cib Get_Event_By_Id_Endpoint

Branch 12-get_event_by_id_endpoint created!


[CT-653] [Feature] Allow running unit/integration tests locally with any supported Python version

Is this your first time opening an issue?

[X] I have read the expectations for open source contributors

Describe the Feature
This is more of a rough edge for a non-professional dbt-core developer: the Makefile hard-codes py38 as the preferred python version in tox for running unit tests and integration tests; we use Python 3.9 at my day job, so when I need to do work on dbt-core I need to remember to swap out the version in the Makefile to py39 before I run dbt tests and then switch it back before I send a PR upstream.
Describe alternatives you've considered
I would love it if the Makefile was either a) configured to work with the Python version that I had installed locally as long as it was supported (i.e., if the default in the Makefile was py and py-integration instead of py38 and py38-integration), or, if that presents a problem for e.g. CI or some such thing, if it was easy (like via an environment variable that was documented in the CONTRIBUTING.md guide) for me to override the setting in the Makefile on my local machine.
Who will this benefit?
Developers who want to create PRs for dbt-core who do not work at dbt Labs.
Are you interested in contributing this feature?
Happy to!
Anything else?
Issue originally discussed in a Slack thread in dbt-core-development: https://getdbt.slack.com/archives/C50NEBJGG/p1652466095907019

Using py and py-integration makes tons of sense—it will just uses your own local PATH version of Python, right?
We don't use the Makefile in CI (we hit tox directly), so I don't think there should be any implications there. Let's do a nice thing for non-professional dbt-core developers :)
The only edge I could see is someone very eagerly upgrading to a new minor version of Python, before dbt-core officially supports it, e.g. Python 3.10 prior to the v1.1 release. But I'd like to see us supporting new Python minor versions more proactively going forward, especially when they promise to have some good stuff inside...
@jwills proceed at your leisure!


Specifying batch_size on bulk_update would only process the first batch

The remaining data is not processed when specifying batch_size when using bulk_update.helper.bulk_update function. This is because lenpks is returned after processing the first batch. Ideally, lenpks should be returned after all data are processed right?
connection.cursor().execute(sql, parameters)
return lenpks

Yes, you are right. It is fixed with v1.1.10, please upgrade your package.
Thanks.


useOverlayPosition() is buggy in a few cases when placement=top

🐛 Bug Report
useOverlayPosition() sets the bottom css prop when placement=top.  This currently is buggy in two cases I know of:

If the container is not body; computePosition() is only computed using boundaryDimensions, when it should be using containerOffsetWithBoundary instead
When document.body as the default container has position: relative; as a positioned offset parent, the bottom needs to be computed relative to the container's actual height, instead of the "visible height" computed by getContainerDimensions

🤔 Expected Behavior
placement=top would align overlay above trigger element correctly.
😯 Current Behavior
placement=top only works properly right now if the overlay container is document.body, and it has position: static
💁 Possible Solution
PR incoming :smile:
🔦 Context
Noticed this when using useSelect when shouldFlip kicks in
🌍 Your Environment

Software
Version(s)

react-spectrum
3.2.0

Browser
Chrome 84

Operating System
Ubuntu 20.04

🧢 Your Company/Team
Plasmic

Thanks for finding this and creating a PR to fix it!


As a git-savvy and cli-savvy compliance professional, I'd like to have a demo or tutorial

Describe the bug
I came to the OSCAL workshop hoping for a Trestle demo so I can understand
how it works.  The current docs have some CLI documentation which is great but newcomers need more (or at least I do).  I like what I've seen of your workflow and model, and I think it's really promising. I'd love to understand it better.
Can you please shoot and publish a screencast to demo some of trestle's functionality? Or provide a step-by-step walk-through to end up with, say, a sample SSP or a component model?
Thanks!

Great suggestion. - Will put it into the backlog.

Actions to be taken based on internal team review:

Trestle tutorials to be published on

Trestle website:

Transforms @degenaro (DONE)
Split / Merge @fsuits (DONE)
Markdown @butler54 (IN progress)
CICD @fsuits (TBC)

Republishing on medium (ALL TBC)

First video for split merge to be published @fsuits

Sample trestle repositories & scripts created and referenced in docs (see associated issues).

Any progress here? With 1.0.0 of OSCAL announced, I'd be really interested in kicking the tires here some. Thanks, Peter

Thank you for your interest @pburkholder - and we would welcome your kicking of tires.  Currently we are in the process of incorporating the recent changes to 1.0.0 and it will take a bit of reworking but we expect a release shortly.
In the meantime, have you seen the current workflow tutorials?  They are being updated periodically and the docs and tutorials were updated somewhat recently.  You can find them in the tutorials section here

This seems to be coming along nicely. I'll have to give it a spin.
On Tue, Jun 15, 2021 at 3:03 AM Frank Suits @.***>
wrote:

Thank you for your interest @pburkholder https://github.com/pburkholder

and we would welcome your kicking of tires. Currently we are in the
process of incorporating the recent changes to 1.0.0 and it will take a bit
of reworking but we expect a release shortly.

In the meantime, have you seen the current workflow tutorials? They are
being updated periodically and the docs and tutorials were updated somewhat
recently. You can find them in the tutorials section here
https://ibm.github.io/compliance-trestle/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IBM/compliance-trestle/issues/339#issuecomment-861236044,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJHWCSL6YFD7AB4TLL3P6LTS33J7ANCNFSM4XBRDZPQ
.

--
Peter Burkholder |  cloud.gov https://cloud.gov compliance & security
please use @.* for cloud.gov matters
202-709-2028 <(202)%20209-2028> | @.**
@.***> *
| pronouns he-him https://www.mypronouns.org/he-him
Free/Busy Calendar
@.*>

@pburkholder - would be great to get your feedback @fsuits and I are probably a few days away from 1.0.0 support (we are doing some optimisation in the process).
If you have any issues let us know - happy to connect off of github if you need help / have feedback so we can refine this.

I'm going to close this out based on the content in compliance-trestle-demos and that on the documentation website (as our backlog has got unwieldy).
If you have any specific demos you want to see let us know and we will add to the backlog.


The bundle size increases x4 when installing the next plugin in a monorepo

Hi there
We are trying the next-federation-plugin in a couple of nextjs projects that live in a monorepo. When using the plugin, nextjs adds binaries and other dependencies to the output build and because it exceeds the lambda limit it fails when deploying to vercel 😢
Traced Next.js server files in: 4.029s
Warning: Max serverless function size of 50 MB compressed or 250 MB uncompressed reached
Serverless Function's page: page1.js
Large Dependencies                                             Uncompressed size  Compressed size
node_modules/.pnpm/<EMAIL_ADDRESS>                    39.6 MB          13.6 MB
node_modules/.pnpm/<EMAIL_ADDRESS>                   39.3 MB          13.3 MB
<EMAIL_ADDRESS>                                        16.7 MB          7.26 MB
node_modules/.pnpm/<EMAIL_ADDRESS>                                  17.9 MB          6.68 MB
<EMAIL_ADDRESS>                15 MB          3.83 MB
<EMAIL_ADDRESS>                                      3.93 MB           954 kB
<EMAIL_ADDRESS>                                         4.09 MB           611 kB
node_modules/.pnpm/react-dom<EMAIL_ADDRESS>                       1.34 MB           335 kB
<EMAIL_ADDRESS>                            860 kB           316 kB
<EMAIL_ADDRESS>                                        1.03 MB           224 kB
<EMAIL_ADDRESS>                                    1.18 MB           213 kB
packages/ui/cjs                                                           660 kB           190 kB
<EMAIL_ADDRESS>                                        962 kB           185 kB
<EMAIL_ADDRESS>                                       637 kB           159 kB
<EMAIL_ADDRESS>           435 kB           147 kB
<EMAIL_ADDRESS>                                           397 kB           119 kB
<EMAIL_ADDRESS>                                           330 kB           109 kB

All dependencies                                                          156 MB          57.2 MB
Serverless Function's page: page2.js
Large Dependencies                                             Uncompressed size  Compressed size
node_modules/.pnpm/<EMAIL_ADDRESS>                    39.6 MB          13.6 MB
node_modules/.pnpm/<EMAIL_ADDRESS>                   39.3 MB          13.3 MB
<EMAIL_ADDRESS>                                        16.7 MB          7.26 MB
node_modules/.pnpm/<EMAIL_ADDRESS>                                  17.9 MB          6.68 MB
<EMAIL_ADDRESS>                15 MB          3.83 MB
<EMAIL_ADDRESS>                                      3.93 MB           954 kB
<EMAIL_ADDRESS>                                         4.09 MB           611 kB
node_modules/.pnpm/react-dom<EMAIL_ADDRESS>                       1.34 MB           335 kB
<EMAIL_ADDRESS>                            860 kB           316 kB
<EMAIL_ADDRESS>                                        1.03 MB           224 kB
<EMAIL_ADDRESS>                                    1.18 MB           213 kB
<EMAIL_ADDRESS>                                        962 kB           185 kB
<EMAIL_ADDRESS>                                       637 kB           159 kB
<EMAIL_ADDRESS>           435 kB           147 kB
<EMAIL_ADDRESS>                                           397 kB           119 kB
<EMAIL_ADDRESS>                                           330 kB           109 kB

All dependencies                                                          156 MB          57.2 MB
Serverless Function's page: page3.js
Large Dependencies                                             Uncompressed size  Compressed size
node_modules/.pnpm/<EMAIL_ADDRESS>                    39.6 MB          13.6 MB
node_modules/.pnpm/<EMAIL_ADDRESS>                   39.3 MB          13.3 MB
<EMAIL_ADDRESS>                                        16.7 MB          7.26 MB
node_modules/.pnpm/<EMAIL_ADDRESS>                                  17.9 MB          6.68 MB
<EMAIL_ADDRESS>                15 MB          3.83 MB
<EMAIL_ADDRESS>                                      3.93 MB           954 kB
<EMAIL_ADDRESS>                                         4.09 MB           611 kB
node_modules/.pnpm/react-dom<EMAIL_ADDRESS>                       1.34 MB           335 kB
<EMAIL_ADDRESS>                            860 kB           316 kB
<EMAIL_ADDRESS>                                        1.03 MB           224 kB
<EMAIL_ADDRESS>                                    1.18 MB           213 kB
<EMAIL_ADDRESS>                                        962 kB           185 kB
<EMAIL_ADDRESS>                                       637 kB           159 kB
<EMAIL_ADDRESS>           435 kB           147 kB
<EMAIL_ADDRESS>                                           397 kB           119 kB

All dependencies                                                          152 MB          56.4 MB
Max serverless function size was exceeded for 3 functions

Again, it only happens when I install or try to use the plugin. If I uninstall it from all microfrontends the problem is gone 🤔
Maybe, there is a problem when using the plugin with pnpm?
I'm trying to create a repository that reproduces that error. I'll edit the issue with the link to the repo when I have finished with the reproduction.
Finally, I have found this issue in the nextjs repository: https://github.com/vercel/next.js/issues/42641
But the issue is about nextjs v13 and I using nextjs v12.
Nextjs version: 12.3.4
pnpm version: 7.18.0
Operative System: MacOS
Thanks!

Tell serverless not to copy over swc?
In a future release ill be removing my use of SWC which might help drop the build size, but it looks like youre tracing and copying over build deps

hi @ScriptedAlchemy, the thing is that I have not control over what serverless is copying 😢
All the output in the issue is from the next build. I have reviewed the pages and I'm sure that I'm not importing @swc/core in any getServerSideProps or client code 😭

are you building standalone mode?
if so after build rm -rf .next/standalone/node_modules/ThingToRemove.
If i can repro this ill work on a solution

You should have control over serverless, via serverless.yml

@ScriptedAlchemy we are using the vercel CLI to deploy our apps. We are using the following commands:
vercel pull --env=production
vercel build --prod
vercel deploy --prebuilt --archive=tgz

After the deploy command, the nextjs app is uploaded to vercel.
I'm still working on the repro but I don't know why I can't reproduce the error in an isolated project 😢

@ScriptedAlchemy I have solved this issue by upgrading to nextjs v13.1.6
Closing for now, thanks!!!


KiCAD8 compatibility issues

Plugin doesn't work under KiCAD8. It gives the following error msg:

That was quick. Can you advise how to update?

Install v4.0.0 manually from the releases section.


Multiple streaming for bloggers with collection of all feedback from all platforms in one place (Deep instance)

Can be done with conjunction of OBS Studio + API of YouTube, VK, Twitch, etc.

https://streamlabs.com/multistream - similar, but not supports Discord


browser/jellyfish.umd.js to be exported as global mode

Moved out of scope from #17 due to waiting for upstream fixes and implementation.

What would you like to be added:
import jellyfish from 'jellyfish.ts'
window.jellyfish = jellyfish

Why is this needed:
To allow browser users to do this:
<script src="https://unpkg.com/<EMAIL_ADDRESS><script>
  var client = new jellyfish.Client('http://ocean.defichain.com')
  client.mining.getMintingInfo().then(info => {
    console.log(info)
  })
</script>

When completed:

[ ] Update docs website
[ ] Update README.md

Still waiting for support from parcel: https://v2.parceljs.org/getting-started/migration/#--global

Hey @fuxingloh!
Are there any updates?
I really look forward to using the library in a browser. If now there is some opportunity to do this, please tell me :)

Hey @fuxingloh!
Are there any updates?
I really look forward to using the library in a browser. If now there is some opportunity to do this, please tell me :)

Hey @IhorSavva
The jellyfish.umd.js output bundles together jellyfish-api-core & jellyfish-api-jsonrpc which are only 2/15 of the total modules available. They are created merely for the browser for interacting with the API, not very useful other than doing just that. Still waiting for upstream library to support it as switching to another bundler (e.g. rollup) is quite a pain.
https://github.com/DeFiCh/jellyfish/blob/015ec69bf41c73376ff1f659645e0095aceb7a42/packages/jellyfish/package.json#L37-L45
To use jellyfish packages, I suggest you import what you need via the esm exports. E.g. if you need the wallet component, you import import { generateMnemonic } from '@defichain/jellyfish-wallet-mnemonic'. If your target is a website then you can use webpack to do it.
We do that for our wallet and explorer project: DeFiCh/wallet package.json
Some examples for reference:

Vue Website: https://github.com/fuxingloh/defi-txn-composer
React Website: https://github.com/DeFiCh/explorer
React Native: https://github.com/DeFiCh/wallet

@fuxingloh, thanks for the answer!
See, I need the createRawTx and signRawTx functionality.
Tell me, can I somehow use this functionality in the browser (html + js)? Or, at least just for example in VueJS
And another question. Do you execute create and sing on the client, or is it the same request to the RPC itself?

@fuxingloh, thanks for the answer!
See, I need the createRawTx and signRawTx functionality.
Tell me, can I somehow use this functionality in the browser (html + js)? Or, at least just for example in VueJS
And another question. Do you execute create and sing on the client, or is it the same request to the RPC itself?

Yup, you can, this is a full end-to-end test with creating and signing a raw transaction.
https://github.com/DeFiCh/jellyfish/blob/main/packages/jellyfish-transaction-builder/__tests__/raw_crafting.ts
Yes, it will work on any platform (batteries not included), it's ubiquitous and written platform-agnostic with fully-featured implementation. (key signing only support Bech32 only for now)
It's non-custodial, so the keys for creating and signing a transaction do not have to be on the node. So no it's not on the RPC, this library does everything manually from scratch, the RPC is just a small part of what jellyfish do.
E.g. this is a classic single key wallet https://github.com/DeFiCh/jellyfish/blob/main/packages/jellyfish-wallet-classic/tests/wallet_classic.test.ts

@fuxingloh Thanks for your answers, it helps a lot!
Tell me how can I use testnet when creating a transaction?
And another question, does your library already know how to work with hd-wallets? (If so, could you show me how I can work with this?)

For creating a transaction, the tests cases are written exactly as such to show you how to create a transaction:
https://github.com/DeFiCh/jellyfish/blob/main/packages/jellyfish-transaction-builder/__tests__/raw_crafting.ts
Yes, the hd-wallet code is here:
https://github.com/DeFiCh/jellyfish/blob/main/packages/jellyfish-wallet/__tests__/wallet_hd_node.test.ts#L4-L9

@fuxingloh About tx creation. I mean: do you library support creating a transaction in the testnet network? If so, how can I do this?
How can I create a transaction without knowing the recipient's private key?(how I can get public_key having only an address)
https://github.com/DeFiCh/jellyfish/blob/559eab98e9f9270a1d0f167942e5c4f167d9141a/packages/jellyfish-transaction-builder/__tests__/raw_crafting.ts#L40-L43
https://github.com/DeFiCh/jellyfish/blob/559eab98e9f9270a1d0f167942e5c4f167d9141a/packages/jellyfish-transaction-builder/__tests__/raw_crafting.ts#L65-L79

Sorry, it's really difficult to answer some of your questions. Transactions are network agnostic. So are addresses, publicKey is address agnostic.

Hey, @fuxingloh!
Okay, let me rephrase.
Could you show me some sample code so that I can create and sign a transaction knowing:
1.about the sender (utxo and private keys)
2.about the recipient (address, amount)

Hey, @fuxingloh!
Okay, let me rephrase.
Could you show me some sample code so that I can create and sign a transaction knowing:
1.about the sender (utxo and private keys)
2.about the recipient (address, amount)

both 1 & 2
https://github.com/DeFiCh/jellyfish/blob/ad6e42290aca0719ee4f27cb7e9c212bcd9a2a2f/packages/jellyfish-transaction-builder/__tests__/raw_crafting.test.ts#L78-L102

@fuxingloh, but in order to create a transaction, you need to know the public key for output (this can be seen from the attached link). However, how can we get this from just an address?
https://github.com/DeFiCh/jellyfish/blob/559eab98e9f9270a1d0f167942e5c4f167d9141a/packages/jellyfish-transaction-builder/__tests__/raw_crafting.ts#L72
Or am I misunderstanding something?

Please open a discussion thread instead, this issue is created purely to track the implementation of global mode.

/close as we are not doing UMD anymore, it's hard to maintain long term.
That's the secondary reason, the primary reason is a full node should never be exposed to the web browser via the RPC protocol. Pretty dangerous to allow your web browser (untrusted runtime) to execute wallet RPC. We will focus on producing ts,esm,cjs builds for the foreseeable future.


When my program runs for a while, it crashes after displaying the "value too large for defined type" error .

Required information
Operating system:
Ubuntu 20.04 LTS
Compiler version:
gcc version 9.4.0 (Ubuntu 9.4.0-1ubuntu1~20.04.1)
Observed result or behaviour:
iceoryx version 2.0.2
When my program runs for a period of time, the following error occurs, what is the reason, thank you!

Of course this is most likely my problem.
Expected result or behaviour:
not crash

@byrnexu So the problem you encounter originates in the semaphore which is used to signal events like when the publisher has delivered a sample to the subscriber.
If the subscriber is attached to a waitset and you never call waitset->wait() those notifications pile up. The semaphore uses internally an integer and this has a maximum value. If you exceed the value by calling to often sem_post (the function the publisher calls when a sample was delivered) without calling sem_wait (which is done by waiset->wait()) the number will reach its maximum value and the system prints this nice little error message.
So depending on the system the max semaphore value is 65535 (mac os),<PHONE_NUMBER> (linux) or even greater or smaller.
So a first idea would be:

Do not use a waitset without handling the received events, see waitset->wait()

If this does not work or you didn't use a waitset at all I would require a little bit more context.

Is this error originating in iox-roudi or in an app
Does this error occur in an app with a publisher?
Do you use other constructs UserTrigger in combination with a waitset?

thank you for your reply! From the code below, I have to call waitSet.wait.
 void SHMIPCBase::init() {
   keepRunning_.store(true);
   beforeInit();

   instanceWithIdentity_ = instance_;
   if (clientChannel_ != std::nullopt) {
     instanceWithIdentity_ = fmt::format("{}-{}", instance_, *clientChannel_);
   }
   subscriberName_ = fmt::format("{}{}{}{}{}", service_, SEP_OF_SHM_SVC,
                                 instanceWithIdentity_, SEP_OF_SHM_SVC, event_);

   subscriber_ = new iox::popo::UntypedSubscriber(
       {iox::capro::IdString_t(iox::cxx::TruncateToCapacity, service_),
        iox::capro::IdString_t(iox::cxx::TruncateToCapacity,
                               instanceWithIdentity_),
        iox::capro::IdString_t(iox::cxx::TruncateToCapacity, event_)},
       makeSubscriberOptions());
   subscriber_->subscribe();
   LOG_I("Create subscriber. {} [{}]", appName_, subscriberName_);

   shutdownTrigger_ = new iox::popo::UserTrigger();
   waitset_ = new iox::popo::WaitSet<>();
   waitset_->attachEvent(*shutdownTrigger_).or_else([this](auto) {
     LOG_E("Failed to attach shutdown trigger. {} [{}]", appName_,
           subscriberName_);
   });

   waitset_
       ->attachEvent(
           *subscriber_, iox::popo::SubscriberEvent::DATA_RECEIVED,
           iox::popo::createNotificationCallback(dataInSHMRecvCallback, *this))
       .or_else([this](auto) {
         LOG_E("Failed to attach subscriber. {} [{}]", appName_,
               subscriberName_);
       });

   afterInit();
}

 void SHMIPCBase::dataInSHMRecvCallback(
     iox::popo::UntypedSubscriber* const subscriber, SHMIPCBase* self) {
   while (self->subscriber_->hasData()) {
     self->subscriber_->take()
         .and_then([&](const void* userPayload) {
           auto chunkHeader =
               iox::mepoo::ChunkHeader::fromUserPayload(userPayload);
           if (self->dataRecvCallback_) {
             self->dataRecvCallback_(userPayload,
                                     chunkHeader->userPayloadSize());
           }
           subscriber->release(userPayload);
         })
         .or_else([self](auto& result) {
           if (result != iox::popo::ChunkReceiveResult::NO_CHUNK_AVAILABLE) {
             LOG_E("Error receiving chunk. {} [{}]", self->appName_,
                   self->subscriberName_);
           }
         });
   }
 }

 void SHMIPCBase::start() {
   init();
   isReady_.store(true);
   LOG_I("Start SHM IPC Channel. {} [{}]", appName_, subscriberName_);
   futureDataInSHMRecv_ =
       std::async(std::launch::async, [this]() { startDataInSHMRecvThread(); });
 }

 void SHMIPCBase::startDataInSHMRecvThread() {
   while (keepRunning_.load()) {
     auto notificationVector = waitset_->wait();
     for (auto& notification : notificationVector) {
       (*notification)();
     }
   }
 }

Is this error originating in iox-roudi or in an app
error originating in an app.
Does this error occur in an app with a publisher?
yes.
Do you use other constructs UserTrigger in combination with a waitset?
From the code above, I have a shutdownTrigger

I found that it might be my problem. There is a compilation task in the background that deletes the running app, which may be caused by this reason.

I found that it might be my problem. There is a compilation task in the background that deletes the running app, which may be caused by this reason.

@byrnexu This is at least not supported by iceoryx ;)
I also looked over your code and you use the waitset correctly. So if you encounter this issue again, ping me here and we have to dig deeper into this.


Don't panic if gateway sender can't send

Codecov Report
Base: 66.86% // Head: 66.84% // Decreases project coverage by -0.01% :warning:

Coverage data is based on head (dcc0691) compared to base (fd22bdc).
Patch coverage: 0.00% of modified lines in pull request are covered.

:exclamation: Current head dcc0691 differs from pull request most recent head 8c2c724. Consider uploading reports for the commit 8c2c724 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #716      +/-   ##
==========================================
- Coverage   66.86%   66.84%   -0.02%
==========================================
  Files          87       87
  Lines       12690    12690
==========================================
- Hits         8485     8483       -2
- Misses       4205     4207       +2

Impacted Files
Coverage Δ

ln-gateway/src/lib.rs
50.76% <0.00%> (ø)

fedimint-server/src/net/peers.rs
91.40% <0.00%> (-0.27%)
:arrow_down:

fedimint-server/src/net/connect.rs
94.94% <0.00%> (-0.26%)
:arrow_down:

modules/fedimint-mint/src/lib.rs
90.94% <0.00%> (-0.19%)
:arrow_down:

fedimint-server/src/lib.rs
86.71% <0.00%> (+0.34%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


clang 17

llvm released version 17. Can the new version be added?
https://github.com/llvm/llvm-project/releases/tag/llvmorg-17.0.6

I lost interest in this project and will instead use nix.


Discrepancy between content page and tenant portal

This page shows VM series available in Stack and their IOPS number.
As an example, if I look at F2s_v2 VM series,  throughput (IOPS) documented is 4X2300. Firstly, it is not clear if it 4X2300=9200 or each disk having throughput of 2300.
Secondly, when a user is trying to add this VM from tenant portal, max IOPS shown in the table on "choose a size" step of wizard page is 4000.
Can you please clarify values shown in these 2 places?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ab1df1a4-dba7-3a66-dce4-88028f9b663d
Version Independent ID: f7ec02a5-b2db-3759-935f-a4f3f1f57649
Content: Virtual Machine sizes supported in Azure Stack
Content Source: articles/azure-stack/user/azure-stack-vm-sizes.md
Service: azure-stack
GitHub Login: @mattbriggs
Microsoft Alias: mabrigg

@Mahesh-MSFT Found out some of what is going on. For your first question, Each disk has 2300 iops (2300 would be the limit). The second item in the wizard, that number is an error in the portal. The portal team has been notified, and they will fix as soon as they are able to. Sorry for the confusion about that. Thanks for letting me know. #please-close

@Mahesh-MSFT We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Remove NamedDimGraph, refactor and simplify

This removes the NamedDimGraph type, along with slicing syntax g[:, 1] in favor of the more verbose but general subgraph(v -> v[2] == 1, g) syntax. The slicing syntax might return, but it causes some syntax conflicts with DataGraphs metadata access through getindex so for simplicity I am removing it for now.

@leburgel @JoeyT1994 heads up about some interface changes. I'm updating DataGraphs.jl and ITensorNetworks.jl accordingly. Over all this should simplify things (like adding support for Graphs.jl functionality, not having too many graph types floating around, etc.) and over all shouldn't change high level ITensorNetworks functionality.


fix(deps): Update dependencies

:tada: This PR is included in version 4.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


configuration

[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: a136f4a7-8a19-0e6c-2411-72457ad1ab7e
Version Independent ID: 42106787-5016-d2b6-2577-b86d09769c9d
Content: Upgrade a failover cluster instance - SQL Server Always On
Content Source: docs/sql-server/failover-clusters/windows/upgrade-a-sql-server-failover-cluster-instance.md
Service: sql
Sub-service: failover-cluster-instance
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

@Solutionkw -- thank you for your feedback.  Please consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.