macos m1 run sample failed with segmentation fault

Describe the bug
segmentation fault,
To Reproduce
Steps to reproduce the behavior:

this is my cmake

cmake .. -DCMAKE_BUILD_TYPE=Release -DHAS_QT5=OFF -DHAS_CAPNPROTO=ON -DBUILD_STANDALONE_PY_WHEEL=ON -DHAS_HDF5=OFF

run sample

./ecal_sample_minimal_snd_c
eCAL::Config - Could not load ini file, using defaults. File name : ecal.ini
[1]    86023 segmentation fault  ./ecal_sample_minimal_snd_c

lldb

lldb ./ecal_sample_minimal_snd_c
(lldb) target create "./ecal_sample_minimal_snd_c"
Current executable set to '/Users/xixioba/Desktop/gitlabs/ecal/build/bin/ecal_sample_minimal_snd_c' (arm64).
(lldb) r
Process 87656 launched: '/Users/xixioba/Desktop/gitlabs/ecal/build/bin/ecal_sample_minimal_snd_c' (arm64)
eCAL::Config - Could not load ini file, using defaults. File name : ecal.ini
Process 87656 stopped
* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x78)
    frame #0: 0x00000001004c03c8 libecal_core.5.dylib`asio::detail::scheduler::init_task() + 20
libecal_core.5.dylib`asio::detail::scheduler::init_task:
->  0x1004c03c8 <+20>: ldrb   w20, [x0, #0x78]
    0x1004c03cc <+24>: cbz    w20, 0x1004c03d8          ; <+36>
    0x1004c03d0 <+28>: add    x0, x19, #0x38
    0x1004c03d4 <+32>: bl     0x1004edc98               ; symbol stub for: pthread_mutex_lock
Target 0: (ecal_sample_minimal_snd_c) stopped.
(lldb) bt
* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x78)
  * frame #0: 0x00000001004c03c8 libecal_core.5.dylib`asio::detail::scheduler::init_task() + 20
    frame #1: 0x0000000100405890 libecal_core.5.dylib`asio::execution_context::service* asio::detail::service_registry::create<asio::detail::reactive_socket_service<asio::ip::udp>, asio::io_context>(void*) + 120
    frame #2: 0x00000001004bf114 libecal_core.5.dylib`asio::detail::service_registry::do_use_service(asio::execution_context::service::key const&, asio::execution_context::service* (*)(void*), void*) + 88
    frame #3: 0x00000001004090a8 libecal_core.5.dylib`asio::basic_socket<asio::ip::udp, asio::any_io_executor>::basic_socket<asio::io_context>(asio::io_context&, asio::ip::udp const&, asio::constraint<is_convertible<asio::io_context&, asio::execution_context&>::value, asio::defaulted_constraint>::type) + 68
    frame #4: 0x00000001004075a0 libecal_core.5.dylib`eCAL::CUDPSenderImpl::CUDPSenderImpl(eCAL::SSenderAttr const&) + 132
    frame #5: 0x0000000100408254 libecal_core.5.dylib`eCAL::CUDPSender::Create(eCAL::SSenderAttr const&) + 96
    frame #6: 0x0000000100474df0 libecal_core.5.dylib`eCAL::CLog::Create() + 1000
    frame #7: 0x0000000100473680 libecal_core.5.dylib`eCAL::CGlobals::Initialize(unsigned int, std::__1::vector<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > >*) + 788
    frame #8: 0x000000010045ae7c libecal_core.5.dylib`eCAL::Initialize(int, char**, char const*, unsigned int) + 2332
    frame #9: 0x0000000100003ddc ecal_sample_minimal_snd_c`main + 80
    frame #10: 0x000000010001108c dyld`start + 520
(lldb)

I tried updating the submodule and it doesn't cause a crash now
git submodule update --remote

Copy ecal/core/ecal.ini to  bin and modified the configuration  network_enabled           = trueand tested the sample,
only the service-related applications can work normally,
Terminal 1:
bin git:(master) ✗ ./ecal_sample_minimal_server
Could not load eCAL time sync module ecaltime-localtime
Method called : echo
Request       : Hello

Terminal 2:
bin git:(master) ✗ ./ecal_sample_minimal_client
Could not load eCAL time sync module ecaltime-localtime
Method blocking call failed ..

Method callback call failed ..

Method 'echo' called with message : Hello
Received response for method echo : Hello from host mbp.local

but minimal_rec_cb/minimal_rec can't receive data, it's too strange
Terminial 1:
bin git:(master) ✗ ./ecal_sample_minimal_rec
-------------------------------
 HELLO WORLD RECEIVER
-------------------------------
Could not load eCAL time sync module ecaltime-localtime

Terminial 2:
bin git:(master) ✗ ./ecal_sample_minimal_snd
-------------------------------
 HELLO WORLD SENDER
-------------------------------
Could not load eCAL time sync module ecaltime-localtime
Sent "Hello World (1)"
Sent "Hello World (2)"
Sent "Hello World (3)"
Sent "Hello World (4)"
Sent "Hello World (5)"

In fact, I only want to test the inter-process communication on a single host,
the phenomenon is that the service-client mode can communicate, but the one-way transmission does not work.
By the way, is ecal fully compatible with DDS, I may need interoperability between them

If you want to test locally, in general we would recommend to use local mode.
If you're communicating locally, UDP is used only to figure out which participants are there, e.g. the so called monitoring layer.
If you're in local mode, localhost is used, whereas in network mode the addresses as configured in ecal.ini are used for Multicast traffic.
So I would first check, if network_enabled=false works, and then move on to network_enabled=true.
On Windows (and here I am really unfamiliar with MacOS) we sometimes had the issue, that either Microsoft Defender / other virus protection blocks the packages, or they might get lost in Nirvana in case of VPN connections, whenever network_enabled=true.
As for the DDS question: no eCAL is not compatible with DDS. It's not a DDS implementation, so if you need interoperability, you will either need gateways to foward between eCAL / DDS.

Closing due to inactivity. We did fix a couple of things since then for macOS.


Updated change  to use ansible module for ssh key generation

Updated change  to use ansible module for ssh key generation
Giving comment for key as Ansible-OpenShift-Provisioning SSH key

Thanks @AmadeusPodvratnik

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


Partial tutorial is broken.

It looks like the ability to edit is broken.
I click on the item and nothing happens.
http://learn.ractivejs.org/partials/1

I've crossposted this to the learn repo, so I'm going to close the issue here.

The tutorials have been on the list for some time to be worked on. They're pretty woefully out of date and don't cover any of ractive's more recent features. Thanks for reminding us about it!


[UWP] Element fallback support

Fixes #2125. Implement fallback support in UWP renderer.

HRESULT AdaptiveActionElementBase::SetSharedElementProperties(std::shared_ptr<AdaptiveSharedNamespace::BaseActionElement> sharedCardElement)

Copy the fallback content back to the shared model

Refers to: source/uwp/Renderer/lib/AdaptiveActionElement.cpp:97 in 6284407. [](commit_id = 62844078d2f9fefd567bf9541383be64de801959, deletion_comment = False)

                            }

Can we make this chunk of code a function or otherwise refactor so that there's only one copy?

Refers to: source/uwp/Renderer/lib/XamlBuilder.cpp:964 in 6284407. [](commit_id = 62844078d2f9fefd567bf9541383be64de801959, deletion_comment = False)

                            // TODO: something

What does this TODO represent? Any reason not to combine with the None/default case for the time being?

Refers to: source/uwp/Renderer/lib/XamlBuilder.cpp:972 in 6284407. [](commit_id = 62844078d2f9fefd567bf9541383be64de801959, deletion_comment = False)

                // throw exception so that parent's fallback can be invoked

Update comment, we're not throwing

Refers to: source/uwp/Renderer/lib/XamlBuilder.cpp:991 in 6284407. [](commit_id = 62844078d2f9fefd567bf9541383be64de801959, deletion_comment = False)

    boolean ancestorHasFallback;

Looks like this is unused

Refers to: source/uwp/Renderer/lib/XamlBuilder.cpp:2763 in 6284407. [](commit_id = 62844078d2f9fefd567bf9541383be64de801959, deletion_comment = False)

            while (!renderer)

Do we support the case where if show card has something unsupported the action could fallback to a different action?

Refers to: source/uwp/Renderer/lib/XamlBuilder.cpp:1464 in 6284407. [](commit_id = 62844078d2f9fefd567bf9541383be64de801959, deletion_comment = False)

HRESULT AdaptiveActionElementBase::SetSharedElementProperties(std::shared_ptr<AdaptiveSharedNamespace::BaseActionElement> sharedCardElement)

nice catch!

In reply to: 470313901 [](ancestors = 470313901)

Refers to: source/uwp/Renderer/lib/AdaptiveActionElement.cpp:97 in 6284407. [](commit_id = 62844078d2f9fefd567bf9541383be64de801959, deletion_comment = False)

                            // TODO: something

yeah, I thought I was going to have to do something extra, but it turned out not to be the case.

In reply to: 470316547 [](ancestors = 470316547)

Refers to: source/uwp/Renderer/lib/XamlBuilder.cpp:972 in 6284407. [](commit_id = 62844078d2f9fefd567bf9541383be64de801959, deletion_comment = False)

    boolean ancestorHasFallback;

it's actually getting passed in to our IterateOverVector helper below

In reply to: 470318180 [](ancestors = 470318180)

Refers to: source/uwp/Renderer/lib/XamlBuilder.cpp:2763 in 6284407. [](commit_id = 62844078d2f9fefd567bf9541383be64de801959, deletion_comment = False)

            while (!renderer)

hoo boy. I'm not sure. This is probably something best answered by @andrewleader or @matthidinger :)

In reply to: 470318921 [](ancestors = 470318921)

Refers to: source/uwp/Renderer/lib/XamlBuilder.cpp:1464 in 6284407. [](commit_id = 62844078d2f9fefd567bf9541383be64de801959, deletion_comment = False)

                            }

yeah, sorry :)

In reply to: 470316170 [](ancestors = 470316170)

Refers to: source/uwp/Renderer/lib/XamlBuilder.cpp:964 in 6284407. [](commit_id = 62844078d2f9fefd567bf9541383be64de801959, deletion_comment = False)

@paulcam206 I have no clue about that while loop, @RebeccaAnne ?

@paulcam206 I have no clue about that while loop, @RebeccaAnne ?

think there was some missing context. @RebeccaAnne originally said this:

Do we support the case where if show card has something unsupported the action could fallback to a different action?

Do we support the case where if show card has something unsupported the action could fallback to a different action?

Ah, is the question whether we SHOULD support that? I believe we should (both an Action itself should be able to have a fallback, and the showCard card within the Action.ShowCard should be able to have a fallback).
I guess the only problem there is the showCard card having a fallback, since we didn't add the fallback property to the AdaptiveCard itself, correct?

Ah, is the question whether we SHOULD support that? I believe we should (both an Action itself should be able to have a fallback, and the showCard card within the Action.ShowCard should be able to have a fallback).

I believe showcard cards support fallback (i.e. if a showcard card has an unknown element and specifies valid fallback, the fallback will be used).

I guess the only problem there is the showCard card having a fallback, since we didn't add the fallback property to the AdaptiveCard itself, correct?

I think that's its own distinct issue that we should probably also talk about. I think what @RebeccaAnne was asking (correct me if I'm wrong) is if a showcard action has fallback content and the card supplied to the showcard has unknown elements but no fallback strategy, should the showcard action's fallback be invoked?

Ah, so I think we're talking about the following payload then...
"actions": [
  "type": "Action.ShowCard",
  "title": "Click me",
  "card": {
    "type": "AdaptiveCard",
    "body": [
      {
        "type": "UnknownElement"
      }
    ]
  },
  "fallback": {
    "type": "Action.OpenUrl",
    "title": "Fallback click me",
    "url": "https://msn.com
  }
]

And the question is whether that ShowCard's fallback (falling back to an OpenUrl) should be used since the ShowCard Card has an unknown element in its body.
I think YES, it should engage the fallback (to the OpenUrl in this case). The error (the unknown element) bubbles up.

            XamlHelpers::CreateXamlClass<IGrid>(HStringReference(RuntimeClass_Windows_UI_Xaml_Controls_Grid));

This throws. We'd probably ideally want it to update it to return HRESULT and have the created class as an out parameter, although it's used all over this file...

Refers to: source/uwp/Renderer/lib/XamlBuilder.cpp:1345 in 32619b5. [](commit_id = 32619b5fbc728904f8c73c4213f6039a4d88b3a6, deletion_comment = False)

            XamlHelpers::CreateXamlClass<IGrid>(HStringReference(RuntimeClass_Windows_UI_Xaml_Controls_Grid));

I've filed #2534 to track reviewing and fixing up our error handling strategy in XamlBuilder.cpp

In reply to: 471076906 [](ancestors = 471076906)

Refers to: source/uwp/Renderer/lib/XamlBuilder.cpp:1345 in ed60768. [](commit_id = ed60768c8ff16e802513aa023e162357e1ec46fa, deletion_comment = False)

And the question is whether that ShowCard's fallback (falling back to an OpenUrl) should be used since the ShowCard Card has an unknown element in its body.
I think YES, it should engage the fallback (to the OpenUrl in this case). The error (the unknown element) bubbles up.

I've filed #2535 to track implementing this.


GCD of two numbers

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Article on gcd of two numbers added
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#213

Please review @HarshCasper @ankitaggarwal23 @iamrajiv @atarax665


[GHSA-rxwq-x6h5-x525] Malicious code distributed via tarball archive of xz >= 5.6.0

Adding some info that may help in refining this advisory. Today, there's no supported ecosystem info. But other info can be added which could possibly be helpful.

Hey @navhits, thanks for the contribution, but Debian packages are currently out of our scope. For the time being you should refer to the Debian org advisory https://security-tracker.debian.org/tracker/CVE-2024-3094 for debian specific details.
That said, if you’re aware of any packages in the ecosystems we support which bundle a copy of xz at 5.6.0 or 5.6.1 please do share. For the moment though, I’m going to close this PR as there is no action we can take.

Understood. Thanks. There are language specific bundles. Let me take a look. Will send a fresh PR if so.


login not working after deploying to heroku, still works in localhost:4200 though

Angular: 7.2.3
"firebase": "^5.5.8",
"angularfire2": "^5.1.0",
Other (e.g. Ionic/Cordova, Node, browser, operating system):
"express": "^4.17.1",
Node: 10.9.0
Windows 10
Google Chrome
How to reproduce these conditions
make your angular app an express app and deploy to heroku and then make your app be served on yourDomain.com instead of yourDomain.herokuapp.com? Sorry I'm a beginner programmer.
** Screenshots **

Expected behavior

Facebook and Google login should work
Actual behavior
Both Facebook and Google Logins dont work and instead show the following errors in the google Chrome console:
Uncaught (in promise): Error: Cannot match any routes. URL Segment: '/auth/handler'
Error: Cannot match any routes. URL Segment: '/auth/handler'
and
Uncaught (in promise): Error: Cannot match any routes. URL Segment: '/auth/iframe'
Error: Cannot match any routes. URL Segment: '/auth/iframe'
and
Uncaught Error: Network Error
at main.115259c96fcb9456163d.js:1
at o (main.115259c96fcb9456163d.js:1)
at e.invokeTask (polyfills.ef2a861f18352d657dcf.js:1)
at t.runTask (polyfills.ef2a861f18352d657dcf.js:1)
at t.invokeTask (polyfills.ef2a861f18352d657dcf.js:1)
at invoke (polyfills.ef2a861f18352d657dcf.js:1)
at n.args. (polyfills.ef2a861f18352d657dcf.js:1)
FYI the message telling me to do a development build is normal. and the CORS thing came up just now but before it wasn't coming up

ok i figured it out. In my firebase config in my prod environment, i had to change 'authdomain' back to 'myProjectId.firebaseapp.com' instead of 'myDomain.com'. This got it working again


Fix & refactor market prices in wallet-list

To be honest, I'm not exactly sure, why the prices were not updated for @zillionn . It's possible that it was a timing problem (e.g. too late subscription to the update of the prices or something like that).
However I found the fetching of the prices & the history wasn't done entirely correctly / optimal anyway.
The problem was, that it could happen that we made too many calls to the API and to our local subscription methods.
I changed the process to something like this:

Try get the data from the cache (and ONLY from the cache )
If we have data in the cache, display this data immediately
Subscribe for future changes in the data
Fetch the up-to-date data from the api

Codewise this looks like this.
Also note that only the prices are updated in a regular interval, the chart data isn't. That's probably something for @luciorubeens  to decide.
@zillionn Can you test if this works for you now?
Fixes: #84

I can't test, sorry.

Nice! 👍🏻


Reduce log verbosity

I'm running stressdisk as a systemd service and am finding the stats very useful.
Bytes read:           144 MByte (   8.73 MByte/s)
Bytes written:         78 MByte (   2.81 MByte/s)
Errors:                 0
Elapsed time:  1m0.038864945s

Especially when comparing to the numbers monitoring tools give. i.e. iostat
But the constant logging is to verbose:
Error while writing "/data/logs/TST_0004"
Removing incomplete file "/data/logs/TST_0004"
Reading file "/data/logs/TST_0002", "/data/logs/TST_0003"
Reading file "/data/logs/TST_0001", "/data/logs/TST_0002"
Reading file "/data/logs/TST_0000", "/data/logs/TST_0001"
Reading file "/data/logs/TST_0003", "/data/logs/TST_0000"
Removing 4 check files
Removing file "/data/logs/TST_0000"
Removing file "/data/logs/TST_0001"
Removing file "/data/logs/TST_0002"
Removing file "/data/logs/TST_0003"
....

Would be a great feature to be able to quiet those.

Do you fancy sending a pull request adding a -q flag?


Microsoft Teams integration is not supported using Power Apps player ?

Hello,
In the documentation it says that Team Integration is not supported for "Mobile devices" but it's not clear if we talk about real mobile devices because it uses the Power Apps player.
I tested it myself and it looks like it's not working on Power Apps using a laptop.

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

ID: 8882c54d-281c-31fa-be77-e05f74029c91
Version Independent ID: 67733d0b-0185-f50c-e485-0322a635a8e9
Content: Overview of Microsoft Teams integration with Dynamics 365 apps
Content Source: ce/teams-integration/teams-integration.md
Service: dynamics-365
Sub-service: dynamics-365-teams
GitHub Login: @sbmjais
Microsoft Alias: shjais

Thanks for your comment, @allandecastro. This issue has been assigned to the doc author for review.

Thanks for the feedback. I checked with the engineering team. Teams integration works on Power Apps using a laptop.
In case issue still exist, please contact Support, using one of the options described here: https://docs.microsoft.com/dynamics365/get-started/support/
Closing this issue.

Hello @sbmjais
I got the answer for the support team that it's not supported. I think it's better to update the documentation.
Can you reopent this issue please ?


Support for Angular 1.5

Does angular2-now support angular 1.5 and its new components?

@robhicks What do you mean by it?
There are no breaking changes so angular2now should work with v1.5 (which is a RC atm).
But if you're talking about angular.component().
It is basically just a wrapping function for angular.directive().
Here is a commit with this new feature https://github.com/angular/angular.js/commit/54e816552f20e198e14f849cdb2379fed8570c1a
So I think you can easily use just @Component decorator.

@kamilkisiela Thanks that's what I needed to know.


Update perf log layout

@benchwang : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@tracych Thanks for PM approval; merging.


Docs are totally down

I can't load the API docs at all. Also looks like the ability to view the cache from Google is disabled, so I currently don't even know of any mirrors I can use to view the docs.

+1

Here's a mirror http://devdocs.io/

+1

Docs are up! can any one close this issue ?


style: fix lint issue

This fixes the lint issue that is breaking the builds.

Pull Request Test Coverage Report for Build 1116

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 34.675%

Totals

Change from base Build 1115:
0.0%

Covered Lines:
357

Relevant Lines:
897

💛  - Coveralls


Cardioid Shape Calculator

cardioid is a heart-shaped curve
I would like to add a calculator that calculates the diameter, perimeter and area of Cardioid given radius
please assign this to me

I have already created issue for the same #2367.

I have already created issue for the same #2367.

Sorry I didn't notice the issue


Failing tests using non-amd64 generic tag

I am suspecting there is a bug in the kryptology/pkg/core/curves/native/bls12-381/arithmetic_fallback.go, since there are a bunch of test failures in the kryptology libarary itself when running with generic build tag.
We at Obol implemented bls threshold signature splitting and verification using pkg/sharing/felman.go and pkg/signatures/bls/bls_sig and our tests fail when running with -tags=generic (it passes when running with GOARCH=amd64).
➜  kryptology git:(master) ✗ git rev-parse HEAD
71ffd4cbf01951cd0ee056fc7b45b13ffb178330

➜  kryptology git:(master) ✗ go version
go version go1.17.6 darwin/arm64

➜  kryptology git:(master) ✗ GOARCH=amd64 go test github.com/coinbase/kryptology/pkg/signatures/bbs github.com/coinbase/kryptology/pkg/accumulator
ok  	github.com/coinbase/kryptology/pkg/signatures/bbs	(cached)
ok  	github.com/coinbase/kryptology/pkg/accumulator	(cached)

➜  kryptology git:(master) ✗ go test -tags=generic github.com/coinbase/kryptology/pkg/signatures/bbs github.com/coinbase/kryptology/pkg/accumulator
--- FAIL: TestBlindSignatureContext (0.04s)
    blind_signature_context_test.go:56:
        	Error Trace:	blind_signature_context_test.go:56
        	Error:      	Received unexpected error:
        	            	invalid result
        	Test:       	TestBlindSignatureContext
.... (redacted for brevity)
FAIL
FAIL	github.com/coinbase/kryptology/pkg/signatures/bbs	0.426s
--- FAIL: Test_Membership (0.02s)
    witness_test.go:70:
        	Error Trace:	witness_test.go:70
        	Error:      	Received unexpected error:
        	            	invalid result
        	Test:       	Test_Membership
.... (redacted for brevity)
FAIL
FAIL	github.com/coinbase/kryptology/pkg/accumulator	0.373s
FAIL

Should be fixed with #49


[RELEASE] console v0.21.3

Updates to the docs for Console v0.21.3

[x] Additional info displayed in the Monitoring > Metrics > Info tab (PR #2446)

Closing as already complete with a different pass.


Missing Git tags

Looks like versions 13.2.2 and 13.2.3 are missing git tags.

Yep, it is manual process. I do not tag it each time as really do not see any benefit.

@btd : hello, tags are useful for packagers (I'm packaging should.js for Debian): our rules needs a build from source, not from npmjs registry.
Cheers,
Xavier

@Gaelan @guimard i added missing tags, thanks for explanation


Add thread-safe way to interrupt waitForSubscriptionEvent().

This is an attempt to solve #59. It closes the socket in order to interrupt p_websocket_->receiveFrame(), and in turn waitForSubscriptionEvent().
A separate mutex is now used for setting p_websocket_ and for using it. I believe this way there is no unnecessary blocking, but also no unsafe invalidation of p_websocket_.
From my understanding, closing the socket before a full frame has been received will make receiveFrame() throw a WebSocketException, which seems fine to me.

Note: earliest I can test this is friday. I won't have access to a controller before then.

Thanks for the PR :+1: . Unfortunately reviewing and merging it is going to take some time, as both me and @jontje have a very busy few weeks ahead of us.

Understood. Testing has shown that close() doesn't interrupt a running recvmsg(). on Linux. Plus, it could introduce a race with the fd number being reused before receiveFrame() continues, unless poco properly handles that internally (which I don't think it does).
However, shutdown() does interrupt the call and doesn't release the fd.

Polite bump. Would a review be possible?
By now I have tested this (manually) against a real IRC5 compact. It worked as intended during those tests.

Thanks for the PR @de-vri-es, it's much appreciated 👍
My calendar has finally calmed down a bit, and I intend to review this and your other PRs this week.
In the meanwhile, could you make sure your PRs are up-to-date with the base branch?

Good to hear! I updated the branches. Note that they probably have merge conflicts with each-other (because of the commit in #58 that removed trailing whitespace). I can fix them when they appear :)

Thanks for the review. I'll try to get round to processing the feedback on Friday.

I think I processed the feedback. I added a slightly longer explanation that also includes when it is useful to call the new forceCloseSubscription() function.
For now, I believe endSubscription() still can't safely be called while waitForSubscriptionEvent() is running, but I'll see if that can be fixed with a follow-up PR (as also being discussed in #59). Until then the comment suggesting to call endSubscription() instead of forceCloseSubscription() is a bit misleading.
Either way, this function will still be useful if endSubscription() and waitForSubscriptionEvent() are thread safe, for the case the controller isn't responding :)

Thanks @de-vri-es 👍

For now, I believe endSubscription() still can't safely be called while waitForSubscriptionEvent() is running, but I'll see if that can be fixed with a follow-up PR (as also being discussed in #59). Until then the comment suggesting to call endSubscription() instead of forceCloseSubscription() is a bit misleading.

Yes, I understand what you are getting at. Currently, endSubscription() can only safely be called in-between calls to waitForSubscriptionEvent(), which isn’t very flexible.

Either way, this function will still be useful if endSubscription() and waitForSubscriptionEvent() are thread safe, for the case the controller isn't responding :)

That is true 😄


[protobuf and grpc] The installation location of protoc.exe, grpc_cpp_plugin.exe, etc. is not correct

Is your feature request related to a problem? Please describe.
I have noticed that the protoc, as well as grpc_cpp_plugin are installed into $ROOT/tools/{protobuf/grpc}, rather than $ROOT/bin. If I understand it correctly the program protoc has assumption about the relative path of protoc.exe and include/google/protobuf/..., otherwise if user's protobuf includes those proto files, e.g., google/protobuf/any.proto, the code generation would fail.
Proposed solution
Install those binaries to $ROOT/bin.
Describe alternatives you've considered
Currently we have to copy those binaries to bin/ directory to make everything work as expected.

In vcpkg, according to our policy, tools should be installed in VCPKG_ROOT/installed/TRIPLET/tools, and only dynamic libraries and pdbs should be placed in the folder bin.

otherwise if user's protobuf includes those proto files, e.g., google/protobuf/any.proto, the code generation would fail.

If this issue exists, then we should use other methods to fix it.

For more information about the issue, please see the logic of adding default path in protobuf:
https://github.com/protocolbuffers/protobuf/blob/64dd751868d633421563e53e68da831a0a02c923/src/google/protobuf/compiler/command_line_interface.cc#L234-L273

It will be resolved tomorrow.


Remove apostrophe from Junyou

隼鷹 and 隼鷹改 translated to Junyou and Junyou Kai, but 隼鷹改二 translated to Jun'you Kai2

Closing this PR. The apostrophe must be present to differentiate んよう from にょう.
Old appearances will be fixed to include the apostrophe.


image-rendering: pixelated does not work on WebGL canvas in Safari and iOS Safari

Per https://bugs.webkit.org/show_bug.cgi?id=193895
This should be reflected in the Notes or Known Issues section on https://caniuse.com/css-crisp-edges

Thanks! Has been added to known issues.


Building KNIFT knift_index.pb on mobile device

System information

Have I written custom code (as opposed to using a stock example script provided in Mediapipe): Yes. #2406
OS Platform and Distribution (e.g., Linux Ubuntu 16.04, Android 11, iOS 14.4):  Ubuntu 20.02 Android 11 Android Studio with Gradle
MediaPipe version:v0.8.6
Bazel version:4.1.0
Solution (e.g. FaceMesh, Pose, Holistic): KNIFT
Programming Language and version ( e.g. C++, Python, Java): Java

Describe the expected behavior:
I made progress in regards to #2406 and am now able to extract the bounding box pixel locations.
I wanted to know if there is a way to create the knift_index.pb on the mobile device itself. How could I go about doing this?
The idea would be to take a picture on the mobile phone and then make the knift_index.pb file and the supply it to the KNIFT solution to perform detection. I would need some guidance to go about this.
Thank you in advance! :)

Hi! @yegenzhi
Any update on this?

We don't have an official supported way to build index on mobile device yet.
However, you can try to build the binary and adb push both the binary and graph to you device, and test if you can run it there.
Hope it helps.

Thank you for your quick response :) I will try this sometime in the upcoming weeks. I will keep my progress updated in this issue 👍

Unfortunately, I could not work on this for the moment. If I do work on this and run into any issues, I will re-open :)
Thanks! @sgowroji @yegenzhi


Incorrect compare columns with DEFAULT

Hello.
I compare two databases, synchronized through my own script.
I get a lot of "ALTER TABLE...CHANGE ... DEFAULT ..."
I execute SQL on the second database. And with repeated DBDiff, the differences are again visible.
Perhaps the DBDiff algorithm is not entirely correct.
Since I compare myself and see no differences.
Also, DbDiff does not distinguish between renaming tables / columns, but I really want to see it.

I understand why this happens, in one database DEFAULT is specified as '0', in another as 0 for Integer.
But SQL doesn’t change anything (mysql sees this as the same thing)


connection pool closing

i am accessing my cosmos db with mongoose. i create a connection pool when i start my (very simple) app. i then get intermittent (but frequent) error messages from the API when searching the database (very simple query, {} on a collection with objects in it). the error is {"name":"MongoError","message":"pool destroyed"}. i search around and the suggestions i got were related to closing the connection pool with db.close(), but i am only closing the connection when exiting the app....
any hints?

Hi @lucarv,
This seems to be a mongodb issue and I think you are not using our documentdb nodej sdk.
So please follow your question here:
https://feedback.azure.com/forums/599059-azure-cosmos-db-mongodb-api/suggestions/19361521-fix-bug-that-destroys-nodejs-mongodb-connection-po


Add a status bar to show test progress

Added an enhanced status bar during the testing stages for both download and upload.
This replaces the behaviour of print_dots with update_status_bar, however print_dots is still present and could possibly be enabled with a flag at a later date.
Additionally, 2 calls to printer have been modified to prevent text from being overwritten by the status bar.

I am a Python beginner.   Does it do  os.popen('stty size', 'r').read().split() every update?
If so might it be better to change it.


CASCI scanner core electrons not computed

The example for the casci scanner in pyscf/mcscf/casci.py:
from pyscf import gto, scf, mcscf
mf = scf.RHF(gto.Mole().set(verbose=0))
mc_scanner = mcscf.CASCI(mf, 4, 4).as_scanner()
mc_scanner(gto.M(atom='N 0 0 0; N 0 0 1.1'))
mc_scanner(gto.M(atom='N 0 0 0; N 0 0 1.5'))

does not work, saying that there are negative core electrons. I believe this issue occurs because when mc_scanner is created, no mol is used, so it assumes there are no electrons. When you give it the active space, the number of core electrons is computed assuming no electrons are in the system.
I fixed this issue by changing the CASCI_Scanner's __call__ function to:
class CASCI_Scanner(mc.__class__, lib.SinglePointScanner):
        def __init__(self, mc):
            self.__dict__.update(mc.__dict__)
            self._scf = mc._scf.as_scanner()
        def __call__(self, mol_or_geom, mo_coeff=None, ci0=None):
            if isinstance(mol_or_geom, gto.Mole):
                mol = mol_or_geom
            else:
                mol = self.mol.set_geom_(mol_or_geom, inplace=False)

            if mo_coeff is None:
                mf_scanner = self._scf
                mf_scanner(mol)
                mo_coeff = mf_scanner.mo_coeff
            if ci0 is None:
                ci0 = self.ci
            self.mol = mol

            # Added this section

            ncorelec = mol.nelectron - (self.nelecas[0]+self.nelecas[1])
            assert(ncorelec % 2 == 0)
            self.ncore = ncorelec // 2

            # end of section added

            e_tot = self.kernel(mo_coeff, ci0)[0]
            return e_tot
    return CASCI_Scanner(mc)

the added section comes from how ncore is computed in the CASCI.__init__ function.
The main problem is that the number of electrons is computed incorrectly when using the CASCI_Scanner class.

Problem fixed in commit 21ff742


Fix visible scrollbar in drawer-end animation

overflow-hidden on the drawer-side removes the scrollbar while animating in the drawer-end example.

I see the issue.
Thanks for the PR but this should be fixed in the component itself, not the example.
I will work on it.


[ENV] update runtime setting default values

Description
update runtime setting default values for resource copies, mem pool type
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at https://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the best of my knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@sxjscience @zhreshold it would be great to have some data on how such defaults work for the cv and nlp models.

I think CNNs are generally static shape while models in NLP are generally dynamic shape. Do we have any plan for improving the memory usage?
Get Outlook for iOShttps://aka.ms/o0ukef

From: Sheng Zha<EMAIL_ADDRESS>Sent: Monday, August 24, 2020 6:30:41 PM
To: apache/incubator-mxnet<EMAIL_ADDRESS>Cc: Xingjian SHI<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [apache/incubator-mxnet] [ENV] update runtime setting default values (#18987)
@szha commented on this pull request.

In src/storage/storage.cchttps://github.com/apache/incubator-mxnet/pull/18987#discussion_r476040201:

@@ -67,8 +67,9 @@ StorageManager *CreateStorageManager(const Context &ctx, const char *context,
int num_gpu_device, std::string *pStrategy) {
const auto env_var = env_var_name(context, pool_type);
const char *type = getenv(env_var.c_str());

if (type == nullptr)
type = "Naive";   // default pool

if (type == nullptr) {
type = "Round";   // default pool

My hope is of course to provide a good out-of-the-box usage experience to mxnet users. From what I observed, there seems to be more models with dynamic shape inputs than static ones, and many of the static-shape models can still run in this setting, hence the proposal.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/apache/incubator-mxnet/pull/18987#discussion_r476040201, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABHQH3TMASLXDEBXVJXNACLSCMH4DANCNFSM4QI2O3KQ.

I think CNNs are generally static shape while models in NLP are generally dynamic shape.

I don't think we can generalize like this. For example, object detection and segmentation are based on CNN and are usually not static-shaped.

Do we have any plan for improving the memory usage?

Of course we do. I think @ArmageddonKnight is currently fixing some missed allocation entries in memory profiler, and plans on developing a memory usage visualization tool later this week to help narrow down the focus for memory optimization. We also intend to add mirror option to cached op to allow training for larger model.

thanks for the reviews. @zhreshold how about setting the pool strategy to be naive when shapes are static?

thanks for the reviews. @zhreshold how about setting the pool strategy to be naive when shapes are static?

This is actually fine when shapes are static, my major concern is that with round enabled by default, in most use cases mxnet can be faster but consumes more memory than expected

@zhreshold we could consider adding an interface to allocate the exact size and use it in cached op for static shape only.

I reverted the round pool change first to merge the rest of the changes. I will work on a cached op path to enable exact size allocation to avoid the memory waste in the static graph case.


ADR 005 Plugins

This ADR has a pending implementation in #407 and an example plugin at #55
rendered

@evan-forbes pls add changelog :)

wait, should ADRs add to the changelogs? I couldn't find other adrs in the logs, so I added the no changelog label, but happy to add a log if that's needed

Oh did we not add any ADRs? Maybe we can make a section specifically for ADRs bc I think they're important to note.

There's one option that was not discussed in this ADR and I think would be simpler than this. Instead of supporting plugins, we would merge a slightly more generalized dalc grpc server (replace optimint.Blocks with core.Messages) into celestia-node. Then optimint can copy and paste what ever protobuf definitions it needs instead of importing celestia-core. Then we should be able to archive the dalc repo.


[11.x] Removed null safe operator from whereNotMorphedTo

Issue Summary:
We identified an issue with the whereNotMorphedTo method in the Laravel framework. The issue stems from the use of the null safe operator <=>, which is not compatible with SQLite and PostgreSQL databases. This operator, while functioning correctly in MySQL, causes errors when used with other database systems.
The following error message was encountered during execution:
PDOException: SQLSTATE[HY000]: General error: 1 near ">": syntax error in vendor/laravel/framework/src/Illuminate/Database/Connection.php:407
Solution:
To ensure cross-database compatibility and to avoid unexpected errors, we propose removing or replacing the null safe operator <=> in the whereNotMorphedTo method. This change will enhance the framework's reliability across different database systems.
References:
This issue was discussed and partially addressed in a previous PR: #42878, which initially proposed the use of the null safe operator.

Seems like just removing code could break other use cases?

This would just reintroduce another bug?

I believe there is no bug in the current implementation of the whereNotMorphedTo method.
The whereNotMorphedTo method's first parameter does not receive null models. The previous discussion about null values https://github.com/laravel/framework/pull/42264#issuecomment-1158825654 was a hypothetical scenario that does not occur in practice.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If applicable, please consider releasing your code as a package so that the community can still take advantage of your contributions!

@mo7ameds7awky I've run into this issue on Postgres. Since this PR was closed, did you ever come up with a work around?

@chrisreedio Yes, there are two workarounds here, first you can revert your condition for example:
->whereNot(fn (Builder $query) => $query->whereMorphedTo('relation-name', $model))
, second you can convert this one function into two checks for example:
->where('model_type', 'type')->where('model_id', 'id')


build(cmake): hide FreeBSD ctest preset on Linux

Hide the FreeBSD ctest preset on Linux and other non-FreeBSD operating systems.
Although building for FreeBSD might make sense on Linux, (e.g. for cross-compiling), testing FreeBSD on Linux should never work, so it's safe to remove it.
Notes to reviewers
Please only merge this PR once the FreeBSD tests pass.

Impacted Files
Coverage Δ

src/ap/ap_service.c
50.56% <0.00%> (+3.40%)
:arrow_up:

Again, we run into #394. That src/ap/ap_service.c file is a bit annoying.


Config file is not found ./data/config/config.json

Hi,
I'm using the following docker-compose.yml file on my Raspberry Pi 4:
version: '3.9'

services:
  kerberos_agent1:
    image: "kerberos/agent:latest"
    ports:
      - "8080:80"
    env_file:
      - ./.env
    volumes:
      - /home/raspberry/kerberos/config:/home/agent/data/config
      - /home/raspberry/kerberos/recordings:/home/agent/data/recordings

and when I run docker compose up I get:
kerberos_agent1_eufy  | {"level":"error","msg":"Config file is not found ./data/config/config.json, trying again in 5s.","time":"2023-06-02T12:17:47+02:00"}

so it seems to be a permission related error.
I tried to run on the host the following commands (that were suggested on another issue):
# tried also 777
sudo chmod -R 755 kerberos
sudo chown 100:101 -R kerberos

but no luck!
How can I fix that?
Thanks for your work!

hello @seth100, do you have a config.json file in that directory /home/raspberry/kerberos/config ?


Allow setting build and launch cache paths

This pull request makes the build cache and launch cache volume paths configurable.

@cmoulliard: I think that this is something that you wanted to use the other day in order to cache the m2 repository. I think that this is prerequisite.

Can you please update the README file to add as new example, this one able to configure the cache ? @iocanel

Not sure if this is doing what you expect, build & launch cache locations are used by buildpacks lifecycle binaries during execution to store things that should be reinflated if a subsequent build of the same app is performed.. so if you mapped the .m2 dir into one of those, it might get totally nuked by buildpacks..
We could do a bind mount of m2 to the container, but from previous containerized build projects I remember being told that .m2 should not be shared concurrently, as it has no protection against concurrent updates (eg, buildpacks building using it, while you use it locally in another shell)..
I'm leaning toward closing this one, but I am curious to know if the intent was to allow sharing of a dev's m2 dir into the build, if so, I don't think we can (safely) do that, but we could offer an extra api to allow additional bind mounts to the lifecycle phases configured by the end-user (at their own risk).
If so, we can raise a feature for that, and address it appropriately

Closing this as not planned.


About Leader Change Signal/Hook

When router is out of service, or part of etcd node is down, leader node often change among machines. When leader machine is changed, is it possible to touch an event from new machine who just became leader?
This notification signal is efficient and really needed in many backend system maintaining.
Currently, an alternative choice is to let every machine to run a check that looply posts a client query to check whether self-node is leader. The weakness of this approach is that it not only takes too much latency to catch leader change signal ( sleep 1s every time ??), but it also brings plenty of client query (checking leader) from every machines which is not good to reduce etcd working load.

You can try watch API, to get notification whenever the data is ready, if I understand your use case correctly.

touch an event from new machine

What do you mean by this?

Does watch API must use a another daemon as a client to request? I prefer the etcd daemon itself to notify a event when it is newly updated as a leader.
For example,
node1(peer)  node2(peer)  node3(peer)
node1(peer)  node2(leader)  node3(peer)  -> etcd daemon at node2 constructs an event to its host (e.g. "touch /tmp/leader-change-event", thus users can use 'inotifyd' to catch this signal efficiently and then special scripts can be executed: node2 call a DNS server to map a domain to node2 itself)
node1(peer)  node2(crash)  node3(peer)
node1(peer)  node2(crash)  node3(leader) -> ... node3 call a DNS server to map a domain to node3 itself)
An ugly implementation is to use another node as a client to looply request http://<IP_ADDRESS>:2379/v2/stats/leader and check whether the value is changed. If changed, this client node will call the DNS server to map a domain to the new leader).

@ghostplant the watch api is for monitoring updates to keys, not the status of the etcd cluster.
Maybe have whatever would listen for an inotify instead monitor the etcd server logs for cluster election messages? This DNS scheme (inotify or not) seems like it could race since there's no way to control for consistency, though. Consider:

node1 elected; begin updating DNS with node1
node1 is partitioned, connection to cluster lost
node2 elected; updates DNS with node2
node1 finishes updating DNS with node1, then crashes
dns has node1 as leader, but cluster has node2 as leader

@heyitsanthony
Thanks. Maybe my previous example is not a good one.
We can consider letting the leader to listen on a constant IP (instead of requesting to update DNS):

node1 elected; -> node1 added a constant IP address
node1 is partitioned, connection to cluster lost -> node1 drop this constant IP address
node2 elected; -> node2 added a constant IP address
node1 finishes updating DNS with node1, then crashes -> (node1 should not be a leader again since step-3 is performed, but whatever, we assume node1 becames a leader, thus node2 must be not a leader, this result in: node1 added a constant IP address + node2 dropped this constant IP address)

This also need leader-changed notification.

OK, is there a reason why the daemon can't listen on the etcd server logs? It's unlikely etcd will support inotify signaling when the leader changes since the notification is already available in the logs...

@ghostplant ping?

Okay, maybe piping logs to an analyzer can solve it. I'll close this issue.


chore: Update llama.cpp license name in artifacts

Closes #79

#80  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tazarov and the rest of your teammates on  Graphite


9/20/2022 Publish

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

File
Status
Preview URL
Details

docs-conceptual/azureadps-1.0/install-msonlinev1.md
:white_check_mark:Succeeded
View (azureadps-1.0)

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


[BUG] Method BlobContainerClient.CreateIfNotExistsAsync() randomly returns 401

Library name and version
Azure.Storage.Blobs 12.13.0
Describe the bug
Our service is using the .NET SDK to access the storage account resource. However, we randomly get 401 when trying to invoke some operation method in BlobContainerClient, e.g., BlobContainerClient.CreateIfNotExistsAsync(). We suspect there is an issue related to credential or token renew.
Here are our logs:
Azure.RequestFailedException: Server failed to authenticate the request. Please refer to the information in the www-authenticate header.
RequestId:9c3d8202-801e-001b-7beb-706b5a000000
Time:2024-03-08T00:00:00.1073565Z
Status: 401 (Server failed to authenticate the request. Please refer to the information in the www-authenticate header.)
ErrorCode: InvalidAuthenticationInfo

Additional Information:
AuthenticationErrorDetail: Lifetime validation failed. The token is expired.

Content:
<?xml version="1.0" encoding="utf-8"?><Error>InvalidAuthenticationInfo<Message>Server failed to authenticate the request. Please refer to the information in the www-authenticate header.
RequestId:9c3d8202-801e-001b-7beb-706b5a000000
Time:2024-03-08T00:00:00.1073565Z</Message><AuthenticationErrorDetail>Lifetime validation failed. The token is expired.</AuthenticationErrorDetail></Error>

Headers:
Server: Microsoft-HTTPAPI/2.0
x-ms-request-id: 9c3d8202-801e-001b-7beb-706b5a000000
x-ms-error-code: InvalidAuthenticationInfo
WWW-Authenticate: Bearer authorization_uri=https://login.microsoftonline.com/<redacted>/oauth2/authorize resource_id=https://storage.azure.com
Date: Thu, 07 Mar 2024 23:59:59 GMT
Content-Length: 404
Content-Type: application/xml

at Azure.Storage.Blobs.ContainerRestClient.CreateAsync(Nullable`1 timeout, IDictionary`2 metadata, Nullable`1 access, String defaultEncryptionScope, Nullable`1 preventEncryptionScopeOverride, CancellationToken cancellationToken)
at Azure.Storage.Blobs.BlobContainerClient.CreateInternal(PublicAccessType publicAccessType, IDictionary`2 metadata, BlobContainerEncryptionScopeOptions encryptionScopeOptions, Boolean async, CancellationToken cancellationToken, String operationName)
at Azure.Storage.Blobs.BlobContainerClient.CreateIfNotExistsInternal(PublicAccessType publicAccessType, IDictionary`2 metadata, BlobContainerEncryptionScopeOptions encryptionScopeOptions, Boolean async, CancellationToken cancellationToken)
at Azure.Storage.Blobs.BlobContainerClient.CreateIfNotExistsAsync(PublicAccessType publicAccessType, IDictionary`2 metadata, CancellationToken cancellationToken)
at Microsoft.<redacted>
at Microsoft.<redacted>in C:\__w\1\s\src\Microsoft.<redacted>\<redacted>.cs:line 312

Expected behavior
The request got 2XX
Actual behavior
The request got 401
Reproduction Steps

Prepare a TokenCredential (e.g., WorkloadIdentityCredential), which implements method GetTokenAsync() and can provide valid credential when it's invoked.
Use this TokenCredential to create a new BlobServiceClient
Call BlobServiceClient.GetBlobContainerClient().CreateIfNotExistsAsync() a lot of times to ramdonly reproduce this issue

Environment
framework: netstandard2.0

@CsCherrYY, the explanation is in the logs you provided:
AuthenticationErrorDetail: Lifetime validation failed. The token is expired.
Your auth token expired.

@seanmcc-msft I'm curious why the token is expired here. The token is controlled by the BlobServiceClient and we only implement the token provider functions. Besides, we randomly encounter the issue, so I'm thinking if there is any scenario the Client will get a expired token.


Add demo gif on README

here's a video of the date picker in action
captured from an emulator running api16 (Android 4)
react-native-persian-calendar-picker.zip


Overview strategy with point samples

Instead of putting clusters to pixel grid centres use one of the points as a sample. This gives nicer look if you zoom in map quickly using vector data (in mobile-sdk) - with default strategy you see regular grids of points as pixels until more detailed zoom is loaded.

The strategy currently used for automatically created overviews on imports (grid centre) was chosen for browser tiled visualisation mimicking the approach of torque, and assuming the grid size would be smaller than a pixel in normal visualisation.
When creating overviews manually (i.e. calling CDB_CreateOverviews) we can provide a different clustering strategy (in the form of a PostgreSQL function).
I think we could some additional strategies to the cartodb extension so they're available without having to define new functions, and we could add also an optional parameter to the Import API to be able to choose the overviews strategy (and grid size).
Example: if we add a function named _CDB_GridClusterSample_Reduce_Strategy to the extension (which could implement a clustering strategy that places the aggregated cluster points at the location of some of the original points) we can use it with (choosing also a grid size of 8 pixels):
select CDB_CreateOverviewsWithToleranceInPixels('table', 8, '_CDB_Feature_Density_Ref_Z_Strategy(REGCLASS,FLOAT8)'::regprocedure, 'CDB_GridClusterSample_Reduce_Strategy(REGCLASS,INTEGER,INTEGER,FLOAT8)'::regprocedure);

Here are a couple of functions that implement different clustering strategies: https://github.com/CartoDB/cartodb-postgresql/commit/5992304b475232a3f95b9a628b9f83e82eae1d43

Maybe even the default strategy could be replaced, as for rasters there is probably no visible difference if we put the aggregated point to the center of pixel or to some other point within same pixel? For vector tiles (which is the future anyway) Sample-based strategy would be much better, especially for quick zoom in and overzoom use cases. I have used 8 and 16-pixel tolerances for vector tile size control with good results.

The current default was carefully adjusted to act like torque clustering to a 1-pixel grid, and in particular to avoid visual artifacts (at pixel level) due to the use of TWKB (see #240).
If we do an efficient implementation of _CDB_GridClusterSample_Reduce_Strategy (current one has some issues) I think it should look the same with 1-pixel gridding and would also look nicer with larger grids, so I'd consider it for the default.
cc @javisantana

The new version of the cartodb extension, 0.18.5, which is in production, includes:

 _CDB_GridClusterSample_Reduce_Strategy to create overviews clustering by a grid, aggregating columns except for the cartodb_id and the geometry, taken from a sample point in the grid (actually, the one with the minimum cartodb_id).
_CDB_GridClusterCentroid_Reduce_Strategy which is similar, but the geometry is set to the centroid of all the geometries in its cluster

Example of use:
select CDB_CreateOverviewsWithToleranceInPixels('table', 8, '_CDB_Feature_Density_Ref_Z_Strategy(REGCLASS,FLOAT8)'::regprocedure, '_CDB_GridClusterSample_Reduce_Strategy(REGCLASS,INTEGER,INTEGER,FLOAT8)'::regprocedure);

cc @jaakla


Expected a Ghost Expression

I want to define a buffer that ensures its objects inside do not exceed, in this case signed integers that do not exceed the maximum range. The code below came back with a  (Error 58) Expected a ghost expression; got an expression "let _ = ...
open FStar.HyperStack.All
open FStar.HyperStack
open FStar.HyperStack.ST
open Lib.IntTypes
open Lib.Buffer

val exp:
	buffer1: (lbuffer int64 3ul) {forall (index:size_t{v index<3}). (v buffer1.(index) + v buffer1.(index) < maxint S64)}
//forall (index:nat{index<3}). range(v buffer1.(index) + v buffer1.(index))(Lib.IntTypes.S64)
	-> Stack unit
	(requires fun h -> live h buffer1)
	(ensures fun h0 r h1 ->True)

b.(i) expands to Lib.Buffer.index that has Stack effect. All expressions in refinements (assertions, pre-conditions, etc.) need to be Ghost.
You can use Seq.index (as_seq h b) (v i) instead of b.(i) in the requires clause. Obviously, you can't use this in the refinement of buffer1 because you don't have access to a initial heap.


Enable deserialization of old Akka cluster messages (mixed pekko/akka cluster)

Forming a cluster with Akka nodes requires the deserialization of cluster messages sent by the Akka. This commit fixes the following exception that occurs during deserialization.
[akka://HybridCluster@<IP_ADDRESS>:2551] with serializer id [5] and manifest [akka.cluster.InternalClusterAction$InitJoinAck].
java.lang.IllegalArgumentException: Unknown manifest [akka.cluster.InternalClusterAction$InitJoinAck]
    at org.apache.pekko.cluster.protobuf.ClusterMessageSerializer.fromBinary(ClusterMessageSerializer.scala:156)

I have updated https://cwiki.apache.org/confluence/display/PEKKO/Pekko+Akka+Compatibility and include the fact the we only support forming clusters with Akka nodes of version 2.6.5 and above.


Currently bumping org.apache.sling.feature.launcher to 1.2.4 causes an error.

Description
Currently bumping org.apache.sling.feature.launcher to 1.2.4 causes an error.
Apache Sling SLING-10956 has been closed in 2021.
I have generated a project from archetype and trying to update its org.apache.sling.feature.launcher to get the latest sling HTL features.
Steps to reproduce
If I update following pom files lines:
              <!-- 1.1.28 and newer versions don't work out-of-the-box due to SLING-10956 -->
              <artifact>org.apache.sling:org.apache.sling.feature.launcher:1.1.26</artifact>

I get the following error when I rebuild and run the new container.
local-cms-1    | [INFO] Selected cache/io/typerefinery/websight/typerefinery-distribution/1.0.0-SNAPSHOT/typerefinery-distribution-1.0.0-SNAPSHOT-typerefinery-websight.slingosgifeature for launching
local-cms-1    | [INFO] Automatically appended cache/io/typerefinery/websight/typerefinery-distribution/1.0.0-SNAPSHOT/typerefinery-distribution-1.0.0-SNAPSHOT-docker.slingosgifeature
local-cms-1    | Enabling Java debug options: -agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=*:5005
local-cms-1    | Listening for transport dt_socket at address: 5005
local-cms-1    | Error: Unable to initialize main class org.apache.sling.feature.launcher.impl.Main
local-cms-1    | Caused by: java.lang.NoClassDefFoundError: org/apache/commons/cli/ParseException
local-cms-1 exited with code 1

Current result
CMS container does not start.
Expected behavior
Updating version of launcher should allow container to build and run with success.
WebSight CMS version
Latest
Desktop (please complete the following information):

OS: Windows
Browser: Chrome
Version: Latest

Looks like it requires more changes. Based on https://github.com/apache/sling-org-apache-sling-starter I prepared:
https://github.com/websight-io/starter/compare/main...feature/GH-112-update-feature-launcher
Probably we will update it in archetype, but meanwhile, you can follow the above changes.

Pawel, I also noticed that we don't need this prepare launcher plugin in tests:
https://github.com/websight-io/starter/blob/main/tests/end-to-end/pom.xml#L126
As we are not using docker in e2e tests. It looks like a leftover.

Looks like it requires more changes. Based on https://github.com/apache/sling-org-apache-sling-starter I prepared: main...feature/GH-112-update-feature-launcher
Probably we will update it in archetype, but meanwhile, you can follow the above changes.

I've made a small update to your feature. https://github.com/websight-io/starter/pull/124


Dialyzer output

Environment

Elixir 1.2.5
PostgreSQL 9.5.1
Ecto 1.1.6:
postgrex 0.11.1
Operating system: OSX 10.11.3
dialyxir 0.3.3
phoenix 1.1.4

Current behavior
New Phoenix project `Library`
mix phoenix.gen.model Book books name:string
mix dialyzer

book.ex:26: Function changeset/1 has no local return
book.ex:26: The call 'Elixir.Library.Book':changeset(x0@1::any(),'empty') will never return since it differs in the 2nd argument from the success typing arguments: (#{},#{})

line 26 in the book.ex file:
  def changeset(model, params \\ :empty) do

Expected behavior
New Phoenix project `Library`
mix phoenix.gen.model Book books name:string
mix dialyzer
No dialyzer output

The first warning is fixed by writing a typespec in your model. The second warning is fixed in Ecto master, thank you!


Change PowerToys VCM shortcut as 0.53 update

As of release 0.53 for PowerToys, the default shortcut for VCM is changed.
This resolves #3615
cc: @crutkas

@franky920920 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"


route-less engine "crashes" on component property mutation

I'm developing a route-less engine. It has a main parent component that handles opening and closing some child components. That main parent component keeps the data for its children in an array property and shows one child for each element in the array. The biggest issue i discovered around this is that if I mutate that array, adding or removing an element causing a child component to be added or removed, the engine gets removed from the DOM entirely, with no warnings or errors in the console. Additionally the component lifecycle event handlers like willUpdate or willDestroyElement are called. Example code:
// parent-component.js
import Ember from 'ember';
export default Ember.Component.extend({
  init() {
    this.openChildren = [{name: 'John'}, {name: 'Paul'}, {name: 'George'}]
  },

  willUpdate() {
    // i'm never called
    console.log('updating...');
  },

  willDestroyElement() {
    // i'm never called
    console.log('destroying...');
  },

  actions: {
    openChild(child = {name: "Ringo"}) {
      let openChildren = this.get('openChildren');
      openChildren.addObject(child);
    },

    closeChild(child) {
      let openChildren = this.get('openChildren');
      openChildren.removeObject(conversation);
    }
  }

});

{! parent-component.hbs }

Parent

{{#each openChildren as |child|}}
  {{child
    child=child
    closeSelectedChild="closeChild"
  }}
{{/each}}

// child-component.js
import Ember from 'ember';
export default Ember.Component.extend({
  actions: {
    close() {
      this.sendAction('closeSelectedChild', this.get('child'));
    }
  }
});

{! child-component.hbs }

{{name}}

  <button {{action="close"}}>X</button>

Simpler component property mutations are also not working, though without the catastrophic results. Just a simple iterator like <button {{action "increment"}}>{{value}}</button> also does not update, and throws this error in the console on the first mutation: Uncaught Error: Assertion Failed: A helper named 'parent-component' could not be found

I feel like the root issue here is probably similar to the one in #85.
@thousand could you put together a simple app to clone down so we can debug further? Or, even better, would be if you could set up a failing test in the dummy app inside this addon.

@trentmwillis Sorry for the delay on this. I've not been able to create a simple test app with an engine. Following the instructions in the docs for ember-engines, i get an dependency resolution error for the ember-metal/streams/utils dependency in the mount keyword definition. Copying the ember version from my currently working app and engine, "ember": "kratiahuja/ember-preparse-patch#5d2926e" at least solves that dependency issue, but now my engine is not found when using {{mount}} for the routeless engine. I haven't tried a routed version yet, but will soon.
Current state is living here: https://github.com/thousand/engine-test

Ok, i was able to resolve my issues with getting a basic engine running. After much head scratching I discovered you need to use the ember-engines resolver as the app's resolver as well >.<. My engine-test repo from before now contains a "working" app and engine. Additionally, after looking through some of the issues here, discovered that ember-engines works with the ember beta channel, so the repo has been updated to reflect that more ideal state. However, the engine is now not rendering components at all.

Ok, now that i've thoroughly established that i'm an idiot, my test repo now demonstrates the behavior.

@thousand could you please update your test repo to use ember-engines 0.2.5 and ember 2.6.0 (final) and verify that your problem is still present. If it is, please describe the steps to reproduce or add a failing test and I will investigate.

@dgeb versions have been updated as suggested. The issue is still there, but slightly different from my original description, it indeed looks like the same issue that @trentmwillis noted in #85 and fixed in #92. When a property changes that causes the component to re-render, the component (not the engine, as i improperly assumed before) disappears on re-render. I didn't get around to writing a test, but in my example, clicking any of the buttons there will reproduce it. There's two examples in there. Each is a component called directly from the application.hbs template of the engine. One mutates an array, and the other is a simple counter with a button to increment. I did (briefly) try to port @trentmwillis's fix in PR #92 to the mount keyword but haven't gotten it working yet. I'll continue looking at it as well, now that i understand a little better about how ember is resolving and rendering components.

@dgeb @trentmwillis  Ok, here's the results of my investigations thus far.
I'm not sure WHY exactly this is a problem, but i'm pretty sure the renderEnv being generated for components (and other render nodes) used in the engine is wrong as those nodes re-render. In the routed engines, the engineInstances are kept around inside the route setup. In the routeless engines, they're created inside the mount keyword's render method, but not stored anywhere to be used on subsequent child nodes' renders. When the child nodes of the engine are re-rendered and mount's childEnv method is called to build render env objects, it cannot regain a handle on the engineInstance and more importantly the engine's toplevel/application view. This leaves no way to build the proper env for child nodes of the engine. Unless i've really missed something, this appears to be an existential problem with routeless engines, or at least with the current implementation where the mount keyword's render method spins up the engineInstance without stashing it anywhere.


Fixerror

updated package.json.
sass-loader to ^9.0.2
webpack-merge to ^5.0.9
which removes<EMAIL_ADDRESS>Chokidar 2 will break on node v14+"
and one more v14+ breaking error
Removed error occuring after running npm start
from webpack.dev.js => merge is not a function

Hello my friend!
 @vikrantuk
Regarding updating the deps, I will do that by myself just to prevent conflicts between deprecated versions.
Hence I will close it.


Testing with flutter_secure_storage

Good day,
How do you go about testing secure storage code using flutter test package?
I am getting the MissingPluginException?
Thank you

sorry i seem to have missed the one issue logged earlier


Except for StrictUndefined

Is it possible to add the specific variable(s) to exception list for current undefined mode?
I use StrictUndefined mode for Environment, but It's needed to convert specific variable {{unsubscribe_link}} to {{amazonSESUnsubscribeUrl}} for email template ({{amazonSESUnsubscribeUrl}} value will be converted by AWS SES on email sending).

Hi @sergei-sss,
You could just define these variables as Environment and/or Template globals, where their value is the AWS SES placeholder string, including {{ and }}.
from liquid import Environment
from liquid import StrictUndefined

passthrough = {
    "unsubscribe_link": "{{amazonSESUnsubscribeUrl}}",
    # Add more here.
}

env = Environment(undefined=StrictUndefined, globals=passthrough)

template = env.from_string('Unsubscribe')
print(template.render())

output
Unsubscribe

An alternative using a custom Undefined type can get a little messy.

Another option is to subclass liquid.Context and override _resolve(). This has the advantage that template authors can never redefine these variables using {% assign %}.
from typing import Any

from liquid import BoundTemplate
from liquid import Context
from liquid import Environment
from liquid import StrictUndefined

class MyRenderContext(Context):
    passthrough = {
        "unsubscribe_link": "{{amazonSESUnsubscribeUrl}}",
        # Add more here.
    }

    def _resolve(self, name: str, default: object = ...) -> Any:
        try:
            return self.passthrough[name]
        except KeyError:
            return super()._resolve(name, default)

class MyTemplate(BoundTemplate):
    context_class = MyRenderContext

class MyEnvironment(Environment):
    template_class = MyTemplate

env = MyEnvironment(undefined=StrictUndefined)

template = env.from_string('Unsubscribe')
print(template.render())

@jg-rp Thanks a lot for the detailed answer!


Recommendation for controller

Can you or anyone here recommend a usb controller or a usb joypad controller that would work well? Thanks in advance.
2x is preferred. Also a link to the website and item from which  I can buy it from.
Such as like amazon, ebay, etc...
All buttons must work or if its an adapter, it must fully work.
Currently my raphnet v3 n64 one doesn't work for the Z, L or R buttons.

My 8bitdo controllers and PS5 controller work especially well for modding: I can leave them plugged in for weeks and they never disconnect.

I have seen other people configure raphnet controllers to work with the port. There seems to be like 200 variations of them in the wild though.

I have seen other people configure raphnet controllers to work with the port. There seems to be like 200 variations of them in the wild though.

Yeah... its weird.  I have the dual n64 version 3 one.  The regular dual n64 v3 one.

Really depends on your tastes, though anything picked up by SDL (which is a huge variety of controllers) and of course has enough buttons and at least one stick should work fine. I personally just stick to KB+M, though I can confirm from some experimentation that my DualSense works fine on Windows.
I notice you were using a Raphnet adapter so presumably you were using a real N64 controller, maybe you could try one of the wireless repros Nintendo made for the Switch instead? They actually work on PC as well.

What about for linux? and which one specifically would you recommend. Can you show me one that is good on amazon or ebay or something? I kind of need one that works...

Actually, do you know of any adapters/controllers that will work that have the control stick in the traditional spot?
like here: https://www.amazon.com/Classic-Controller-iNNEXT-Joystick-Raspberry/dp/B07449HLKJ/ref=sr_1_3?crid=2YKSMGT9IJY8T&dib=eyJ2IjoiMSJ9.sQ-ZDIxSgv2F49ryHqoAdeOQC2ZYnDPNWhqtMDLf0L7omTrcWyczQOJtLgr_VO2oel_4zhf06lDwO2vDEJlCP0mOD5H0r6fD_R5vy1ebzJzDSaNJ0Bn7s6rOQH_zOu3smoez1aP5Xex2fVcw92exm24nTpy3DR8N40PuC1zPTYM3IbJhseORW6g9GH4c4qVejmlLfgp7uQnBIkHlXSBFceYVx6kwXfjfsh-WXzBQRhE.UMuHqDjjlvgCHvdtbtALkbf7dCf7s5eB6U6M4YZuk8Y&dib_tag=se&keywords=adaptoid+n64+controller&qid=1734945972&s=videogames&sprefix=adapton64+controller%2Cvideogames%2C323&sr=1-3
And not here:
https://www.amazon.com/sspa/click?ie=UTF8&spc=MTo4NDQ1MjY5OTU4ODQyMzExOjE3MzQ5NDU5NzI6c3BfYXRmOjIwMDAzODg1Nzc4Njk3MTo6MDo6&url=%2FWarrior-Controller-Original-Nintendo-Console%2Fdp%2FB08GJMGRXV%2Fref%3Dsr_1_1_sspa%3Fcrid%3D2YKSMGT9IJY8T%26dib%3DeyJ2IjoiMSJ9.sQ-ZDIxSgv2F49ryHqoAdeOQC2ZYnDPNWhqtMDLf0L7omTrcWyczQOJtLgr_VO2oel_4zhf06lDwO2vDEJlCP0mOD5H0r6fD_R5vy1ebzJzDSaNJ0Bn7s6rOQH_zOu3smoez1aP5Xex2fVcw92exm24nTpy3DR8N40PuC1zPTYM3IbJhseORW6g9GH4c4qVejmlLfgp7uQnBIkHlXSBFceYVx6kwXfjfsh-WXzBQRhE.UMuHqDjjlvgCHvdtbtALkbf7dCf7s5eB6U6M4YZuk8Y%26dib_tag%3Dse%26keywords%3Dadaptoid%2Bn64%2Bcontroller%26qid%3D1734945972%26s%3Dvideogames%26sprefix%3Dadapton64%2Bcontroller%252Cvideogames%252C323%26sr%3D1-1-spons%26sp_csd%3Dd2lkZ2V0TmFtZT1zcF9hdGY%26psc%3D1

Yes, does any adapter you suggest supported with this type of controller?

This goes for everyone who responds btw.

I have seen other people configure raphnet controllers to work with the port. There seems to be like 200 variations of them in the wild though.

Do you know any n64 adapters that work with the controller picture I attached? just now above this new comment I am sending you?
For linux preferably. Although windows configs might also work, I dont know.


A couple of more examples on adding/removing connection groups needed

Would you please together a couple of examples on adding/removing connections to/from a group? For e.g. the following code is from .netcoreapp3.1 which can no longer be compiled in .NET 5.0:
[FunctionName("addtogroup")]
		public static Task Run(
			[HttpTrigger(AuthorizationLevel.Function, "get", Route = "addtogroup/{customerKey}/{userId}")]
 			HttpRequestData req,
			string customerKey,
			string userId,
			[SignalR("myhub"]
 			IAsyncCollector<SignalRGroupAction> signalRGroupActions,
			FunctionContext executionContext)
		{
			return signalRGroupActions.AddAsync(new SignalRGroupAction
			{
				UserId = "123",
				GroupName = "groupName",
				Action = GroupAction.Add
			});
		}

Also, another example of how to send a message to a group of users will be great too.

Is there an update on this? I still can't find any information about this. Is it even possible at the moment with .NET 5 Azure Functions?

Here is an example for add to group that worked: https://github.com/Azure/azure-functions-dotnet-worker/issues/347#issuecomment-858133212
The class SignalRGroupAction is defined as, to use the code in that example.
public class SignalRGroupAction
    {
        public string connectionId { get; set; }

        public string userId { get; set; }

        /// <summary>
        /// Required
        /// </summary>
        public string groupName { get; set; }

        /// <summary>
        /// Required. Must be one of "add", "remove", "removeAll"
        /// </summary>
        public string action { get; set; }
    }


Switch to the next lens, the audio played by the current lens will not be closed automatically

Version: phaser3
Question: After I switch a lens, the audio of the previous lens is still playing, not automatically closed
Code：
``

I used scence.stop(), but I can’t turn off the audio automatically

That's normal. The sound manager is a global system.

That's normal. The sound manager is a global system.

This is the correct behavior. Audio is global and changing a Scene will not stop the audio unless you specifically tell it to stop in the Scene shutdown handler.

This is the correct behavior. Audio is global and changing a Scene will not stop the audio unless you specifically tell it to stop in the Scene shutdown handler.


Skippable tests

While working #306 I noticed that it might be beneficial for learners to skip single tests, enabling a workflow where learners implement and enable one test after the other (as seen e.g. on the Rust and F# track)
Also skipping all tests by default might improve the overall track setup as executing dune runtest from the track root can serve as a basic sanity check for all exercises and test harnesses.
An example of how this might look like can be found here
brainpower-org/ocaml-exercism-cubicle/acronym/test.ml#L4-L6
I also thought it might be nice to have a CI step that copies examples.ml and runs the (then enabled) unit tests against it. I'm not sure what the best way to do this would be, thoughts?
I tried to contributes this in the test-generator templates but failed to compile the test-generator.

The main drawback of commenting out tests by default is that lazy students will get the impression that their program works because they haven't enabled all tests. At work we have a BAIL_ON_FAIL=1 parameter that, when set, will stop on the first test that fails. Would it not be preferrable to achieve the same workflow of "one test at a time", but avoid manually commenting in tests?
This is handled in the Haskell track with HSpec's configFastFail = True.
I wonder if OUnit2 has an equivalent.
I don't think it does.

I thought that maybe
val run_test_tt_main : ?exit:(int -> unit) -> test -> unit

means that we can exit the test suite given a failure.
But trying
 let () =
  run_test_tt_main ("connect tests" >::: tests)
    ~exit:(fun i ->
        print_endline ("I got " ^ string_of_int i ^ "!");
        exit i
    )

still runs all tests.
So I don't think so.

I thought about a solution along these lines:
let ae ?(skip=false) exp got _test_ctxt  =
  skip_if skip "Skipped";
  assert_equal exp (got ())

let () =
  run_test_tt_main ("tests" >::: [
    "skipped - would fail" >:: ae ~skip:true true (fun _ -> false);;
    "not skipped - succeeds" >:: ae ~skip:false true (fun _ -> true);;
  ])

Running this creates output like this:
.S
Ran: 2 tests in: 0.10 seconds.
OK: Cases: 2 Skip: 1


[BUG] tensorrt package missing leading to onnx-to-trt failure

# morpheus tools onnx-to-trt \
>             --input_model /common/models/sid-minibert-onnx/1/model.onnx \
>             --output_model /common/models/sid-minibert-trt_b1-8_b1-16_b1-32.engine \
>             --batches 1 8 \
>             --batches 1 16 \
>             --batches 1 32 \
>             --seq_length 256 \
>             --max_workspace_size 4000
Traceback (most recent call last):
  File "/opt/conda/envs/morpheus/bin/morpheus", line 11, in <module>
    sys.exit(run_cli())
  File "/opt/conda/envs/morpheus/lib/python3.8/site-packages/morpheus/cli.py", line 1381, in run_cli
    cli(obj={}, auto_envvar_prefix='CLX', show_default=True, prog_name="morpheus")
  File "/opt/conda/envs/morpheus/lib/python3.8/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/opt/conda/envs/morpheus/lib/python3.8/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/opt/conda/envs/morpheus/lib/python3.8/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/opt/conda/envs/morpheus/lib/python3.8/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/opt/conda/envs/morpheus/lib/python3.8/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/opt/conda/envs/morpheus/lib/python3.8/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/opt/conda/envs/morpheus/lib/python3.8/site-packages/morpheus/cli.py", line 136, in new_func
    return f(ctx, *args, **kwargs)
  File "/opt/conda/envs/morpheus/lib/python3.8/site-packages/morpheus/cli.py", line 233, in onnx_to_trt
    gen_engine(c)
  File "/opt/conda/envs/morpheus/lib/python3.8/site-packages/morpheus/utils/onnx_to_trt.py", line 30, in gen_engine
    import tensorrt as trt
ModuleNotFoundError: No module named 'tensorrt'

In the mean time, updating the documentation with these stipulations/requirements will reduce issues for developers tasked with becoming familiar with this tool. If you'd like this to just be a docs update, I can do that.

stale


Incorrect value for True

Need to use the value $true rather than True for this to take effect.

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

ID: 2b2fe730-a95d-d12d-adb8-49d1397c77c7
Version Independent ID: 1f6077c3-6114-cb5e-8b1a-7587188f2cdf
Content: Set up for webinars in Microsoft Teams - Microsoft Teams
Content Source: Teams/set-up-webinars.md
Service: msteams
GitHub Login: @KarliStites
Microsoft Alias: kastites

@sdbyrd thank you for your feedback.
I've been managing other issues with the same feedback, and it'll be fixed once the following pull request is merged:
#7303
(really a couple of hours after it's merged)


teams fx samples migration

@Prithvi-MSFT - Please run build pipeline before merging the PR.

Build status - All builds passed
https://github.com/OfficeDev/Microsoft-Teams-Samples/actions/runs/5688535018


Duplicate PR  - Add EntityMut::remove_by_id

Objective

This is a duplicate of the awesome work by @mateuseap, last updated 04/09/23 - #9663
I'm opening a second one to ensure the feature makes it into 0.14
Fixes #9261

Solution
Almost identical to #9663 with three exceptions

Uses a closure instead of struct for commands, consistent with other similar commands
Does not refactor EntityCommands::insert, so no migration guide
EntityMut::remove_by_id is now safe containing unsafe blocks, I think thats what @SkiFire13 was indicating should happen in this comment

Changelog

Added EntityMut::remove_by_id method and its tests.
Added EntityCommands::remove_by_id method and its tests.

Seeing how this is only slightly different from EntityMut::remove_bundle, and given that both need to handle the same tricky details, would it make sense to just reuse EntityMut::remove_bundle here? Then this method would just need to initialize the BundleId with init_component_info and call remove_bundle with it (basically the same as EntityMut::remove except it initializes the BundleId in a different way).

Great idea, thats much simpler :)


Enable pbjson-build supprot no-std-serde

when i cargo check ics23, occur this. do your have some advice?
error[E0433]: failed to resolve: could not find `ContentRefDeserializer` in `de`
  --> /Users/davirain/.cargo/git/checkouts/pbjson-8ee13d6e78642cde/98fdb7b/pbjson/src/lib.rs:38:14
   |
38 |     #[derive(Deserialize)]
   |              ^^^^^^^^^^^ could not find `ContentRefDeserializer` in `de`
   |
   = note: this error originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0412]: cannot find type `Content` in module `_serde::__private::de`
  --> /Users/davirain/.cargo/git/checkouts/pbjson-8ee13d6e78642cde/98fdb7b/pbjson/src/lib.rs:38:14
   |
38 |     #[derive(Deserialize)]
   |              ^^^^^^^^^^^ not found in `_serde::__private::de`
   |
   = note: this error originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info)
help: consider importing this enum
   |
26 +     use private::Content;
   |
help: if you import `Content`, refer to it directly
   |
38 |     #[derive(Deserialize)]
   |

error[E0425]: cannot find function `borrow_cow_str` in module `_serde::__private::de`
  --> /Users/davirain/.cargo/git/checkouts/pbjson-8ee13d6e78642cde/98fdb7b/pbjson/src/lib.rs:38:14
   |
38 |     #[derive(Deserialize)]
   |              ^^^^^^^^^^^ not found in `_serde::__private::de`
   |
   = note: this error originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info)

Some errors have detailed explanations: E0412, E0425, E0433.
For more information about an error, try `rustc --explain E0412`.
error: could not compile `pbjson` (lib) du


feat(examples): optimize bullet examples

optimize bullet examples, add legend

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.478%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
9964

Relevant Lines:
10898

💛  - Coveralls


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_76wpvhjm5g/clone/recipe/meta.yaml:17: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_76wpvhjm5g/clone/recipe/meta.yaml:23: missing_wheel: For pypi packages, wheel should be present in the host section
Errors were found


Feature request: Filter or display a list workloads supported by device

There isn't an easy way to tell which workloads will run on Android, Linux, or both.
So far, workloads such as dhrystone, memcpy, cyclictest, and sysbench have standalone binaries that will run on both Linux and Android devices. Otherwise, it's assumed that the rest depend on Android SDK capabilities and have no Linux support. It isn't as obvious until users look further into the wlauto/workload directory.

Implemented by commit e7fae25821aa099733985e35da2cbd5392df6253


DEV2-1691 - add "Getting Started" functionality

open getting-started webview after installation
remove the hub welcome opening
add "Getting Started guide" action to the side panel

https://user-images.githubusercontent.com/10290661/203800126-54b67e13-d7a2-41b4-9193-3ad9a725d5b5.mp4

Read your replies - lgtm.
You can resolve them all


swallow errors inside of a template

ToToml should always return a string, even on marshal error (empty string).

Hi @carlory
Can you please explain more about the intent of this PR? Usually swallowing errors is a very bad idea.

Perhaps some unit tests around ToToml would be a great way to explain the necessity of this change. The function is documented as
https://github.com/kubernetes/helm/blob/d815310928b0ea77382e34b3b49a425adcea5f77/pkg/chartutil/files.go#L196-L199
So adding some unit tests to confirm this change works as described would be great.

@bacongobbler  Thanks for your feedback.  Has done!

I went and looked at how some of the Sprig encoders work. They do return errors on failure... but only for real encoding errors.
In this case, could we check the input string to make sure it's not an empty string? Then we could return "" immediately for that case. And we wouldn't have to swallow errors that might be legitimate (like malformed data).
Is that a workable alternative to just swallowing errors?

ping @carlory RE: comment above ^


Dependency on package tk

When I installed the package I noticed that it depends on tk.  I'm guessing that it was intended to be depending on tkinter instead.
Since tkinter is part of the Python standard library I would propose to delete the dependency.
For reference: the description of the project tk on pypi is "TensorKit is a deep learning helper between Python and C++." (tk on pypi.org).

Good catch. I think I clearly mixed mixed something up when writing the setup.py. I will fix it in the next release.

@marnikow This should hopefully have been fixed now in v1.3.0


[#5374] Let OpenSslContext take pre-encoded pkcs#8 private key/cert b…

[#5374] Let OpenSslContext take pre-encoded pkcs#8 private key/cert bytes
Motivation
OpenSslContext is expecting Java's PrivateKey and X509Certificate objects as input
(for JdkSslContext API compatibility reasons) but doesn't really use them beyond
turning them into PEM/PKCS#8 strings.
This conversion can be entirely skipped if the user can pass in private keys and
certificates in a format that Netty's OpenSSL code can digest.
Modifications
Two new classes have been added that act as a wrapper around the pre-encoded byte[]
and also retain API compatibility to JdkSslContext.
Result
It's possible to pass PEM encoded bytes straight into OpenSSL without having to
parse them (e.g. File to Java's PrivateKey) and then encode them (i.e. PrivateKey
into PEM/PKCS#8).
File pemPrivateKeyFile;
byte[] pemBytes = readBytes(pemPrivateKeyFile);
PemPrivateKey pemPrivateKey = PemPrivateKey.valueOf(pemBytes);

SslContextBuilder.forServer(pemPrivateKey)
    .sslProvider(SslProvider.OPENSSL)

@rkapsi - Can you fix the compile error:
[Compiler] Compilation failure
  where T is a type-variable:
    T extends Object declared in method <T>attr(AttributeKey<T>)
/var/lib/teamcity-agent/work/a00bdf77e621d005/handler/src/main/java/io/netty/handler/ssl/PemPrivateKey.java:[178,4] error: method does not override or implement a method from a supertype

@Scottmitch @normanmaurer PTAL

Using ByteBufAllocator all the way (including for the Base64 encoder)
Use direct buffers and approximate the initial capacity (something that wasn't done before)
Gave the PemEncoded interface a deeper meaning
Use retain/release to control a PemEncoded value's lifecycle (especially in respect to when to zeroout its bytes)

As for PrivateKey#getAlgorithm() and #getEncoded() (same for the X509 object). It can be done but IMHO we shouldn't do it. The idea is that we can obtain a PEM/PKCS#8 blob from some datasource (say DB) and pass it through into OpenSSL w/o having to ever parse it (and encode it). The only reason to support these methods would be if we wanted to pass PemPrivateKey instances into JDK's SSL impl but I don't know if it'd even work because there may be dependencies on RSAPrivateKey and such.

@rkapsi - After my comments are addressed LGTM ... good work!
@normanmaurer - Want to take another look?

@Scottmitch yep will check shortly

@rkapsi please address last comment, then squash ..
Thanks again for your hard work!

@normanmaurer done.

Cherry-picked into 4.1 (cc580e3ba162af434928f835befee85f0f6b9cba) and 4.0 (cc580e3ba162af434928f835befee85f0f6b9cba)

@rkapsi thanks!


useEditorState to allow shouldRerenderOnTransaction:false to work

Fix for #2055
Uses Tiptap's useEditorState with a selector to properly select the editor state when updates happen.

@CTNicholas i also updated the guide here

@CTNicholas i also updated the guide here

Remember we have a React guide too!

https://github.com/liveblocks/liveblocks/pull/2063


add the new menu connected component

add actions for the menu connected
add reducers for the menu connected
add uts
Add examples for the Menu connected

Codecov Report

Merging #740 into master will decrease coverage by 0.01%.
The diff coverage is 98.92%.

@@            Coverage Diff             @@
##           master     #740      +/-   ##
==========================================
- Coverage   99.77%   99.76%   -0.02%
==========================================
  Files         266      269       +3
  Lines        6999     7092      +93
  Branches     1641     1662      +21
==========================================
+ Hits         6983     7075      +92
- Misses         16       17       +1

Impacted Files
Coverage Δ

src/components/menu/MenuActions.ts
100% <100%> (ø)

src/ReactVaporReducers.ts
100% <100%> (ø)
:arrow_up:

src/components/menu/MenuReducers.ts
100% <100%> (ø)

src/components/menu/MenuConnected.tsx
98.27% <98.27%> (ø)

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 0e87bc5...36e2210. Read the comment docs.


added support for AWS CloudWatch Observability Managed Addon

Issue #, if available:
Fixes #150
Description of changes:
added support for AWS CloudWatch Observability Managed Addon
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

AWS CodeBuild CI Report

CodeBuild project: IDFModuleTestsInfrastructur-59JoFoVH64CI
Commit ID: 467c0fcc5b0d4d9dc4f3e7f742529edfc7ec0cea
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

We should probably add a list of all the configurable items in a condensed list....(ex. for the node groups - possible entries, for the addons  - possible entries, for the drivers - possible entries....just a lookup list)


mention the timestamp in FILES

Unsure about the wording, tbh., do think that files should maybe be an option:)

Looks good - I do think its location should be optional. Thanks


Remove flow-archive dependency & storage/remote package

Closes #???
Description

Remove flow-archive dependency & storage/remote package. See discussion on Discord and related issue.
Update flow-core-contracts version to v0.15.0

For contributor use:

[ ] Targeted PR against master branch
[ ] Linked to GitHub issue with discussion and accepted design OR link to spec that describes this work
[ ] Code follows the standards mentioned here
[ ] Updated relevant documentation
[ ] Re-reviewed Files changed in the GitHub PR explorer
[ ] Added appropriate labels

The tests are failing due to an upgraded version of RandomBeaconHistory contract. The new API should be used in the tests instead:
let sorAt1 = RandomBeaconHistory.sourceOfRandomness(blockHeight: 1)
let sorAt2 = RandomBeaconHistory.sourceOfRandomness(blockHeight: 2)

The above lines should replace these ones: https://github.com/onflow/flow-emulator/blob/master/emulator/random_source_history_test.go#L83-L84

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

Comparison is base (7adb012) 50.51% compared to head (c13f59b) 52.62%.
Report is 14 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #524      +/-   ##
==========================================
+ Coverage   50.51%   52.62%   +2.10%
==========================================
  Files          30       29       -1
  Lines        3951     3795     -156
==========================================
+ Hits         1996     1997       +1
+ Misses       1802     1646     -156
+ Partials      153      152       -1

Flag
Coverage Δ

unittests
52.62% <100.00%> (+2.10%)
:arrow_up:

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.


Write some new reference shaders that use GLSL ES 3.1 features and interesting built-in functions

The sample shaders that come with GraphicsFuzz currently are a bit limited.  First, they are essentially limited to GLSL ES 1.00 features (even though they may use a newer shading language version in the #version string).  Second, there just aren't that many of them.
Let's write some more!

This has been addressed via various PRs.


Add flow-api-translator compatibillity test

Summary:
Motivation
We want to make the react-native package parsable by modern Flow tooling (both flow-bundler, flow-api-translator). This will enable us to create a better public API signature, as well as auto-generate TypeScript definitions for every source file.
This diff
Adds a simple Jest test to track which files in packages/react-native/ are currently unparseable by modern Flow tooling. This will be used to support the incoming stack modernising our module syntax — where 1/ the migration will not be immediate and 2/ it's extremely useful to see the subset of files that were changed.
Changelog: [Internal]
Differential Revision: D65483279

Abandoning for now


Prepare Release v1.3.4

/cc @containous/traefik

CI passed! (sorry for breaking in guys, just hoping to get double-encryption fixed before leaving work today :smiley: )

Hi again guys,
Just curious what's the rough ETA for the release. I'm in the UK and our workday is almost over. Trying to figure out if it's worth waiting for an hour or two to pull 1.3.4 or simply go home and wait till tomorrow :–) What would you suggest?

Many thanks guys! You rock!!!


Accordion menu sticking open

In the layers window the accordion menus have a tendency to want to be open after they have been clicked on once before - we need to have them stay closed once they've been closed
To get this to happen:
Click on a feature or map layer in the layers menu
Click on Style to open the style menu
Click on Style again to close it
Now click on a different layer
The style menu magically opens again

@justin-bits, @wallw-bits this behavior seems intentional since it's saving which tab is open to settings here: https://github.com/ngageoint/opensphere/blob/master/src/os/ui/layer/defaultlayerui.js#L459
Do we want to change that behavior? Is this actually a problem?

I find it annoying as a user, so I'd say to change it.

@welchyd if it is saving the open selection then it should also save the fact that you closed it.

Okay, so it sounds like we could keep the behavior of remembering that a section is open, but also remember when a section has been closed