node v4.0 compatibility

I'm getting a lot of compilation errors when i compile with node 4.0
In file included from ../src/talib.cpp:29:
../node_modules/nan/nan.h:324:27: error: redefinition of 'NanEnsureHandleOrPersistent'
  NAN_INLINE v8::Local<T> NanEnsureHandleOrPersistent(const v8::Local<T> &val) {
                          ^
../node_modules/nan/nan.h:319:17: note: previous definition is here
  v8::Handle<T> NanEnsureHandleOrPersistent(const v8::Handle<T> &val) {
                ^
../node_modules/nan/nan.h:344:27: error: redefinition of 'NanEnsureLocal'
  NAN_INLINE v8::Local<T> NanEnsureLocal(const v8::Handle<T> &val) {
                          ^
../node_modules/nan/nan.h:334:27: note: previous definition is here
  NAN_INLINE v8::Local<T> NanEnsureLocal(const v8::Local<T> &val) {
                          ^
../node_modules/nan/nan.h:757:13: error: no member named 'smalloc' in namespace 'node'
    , node::smalloc::FreeCallback callback
      ~~~~~~^
../node_modules/nan/nan.h:768:12: error: no matching function for call to 'New'
    return node::Buffer::New(v8::Isolate::GetCurrent(), data, size);
           ^~~~~~~~~~~~~~~~~
/Users/oransel/.node-gyp/4.0.0/include/node/node_buffer.h:31:40: note: candidate function not viable: no known
      conversion from 'uint32_t' (aka 'unsigned int') to 'enum encoding' for 3rd argument
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate,
                                       ^
/Users/oransel/.node-gyp/4.0.0/include/node/node_buffer.h:43:40: note: candidate function not viable: 2nd argument
      ('const char *') would lose const qualifier
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate,
                                       ^
/Users/oransel/.node-gyp/4.0.0/include/node/node_buffer.h:28:40: note: candidate function not viable: requires 2
      arguments, but 3 were provided
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate, size_t length);
                                       ^
/Users/oransel/.node-gyp/4.0.0/include/node/node_buffer.h:36:40: note: candidate function not viable: requires 5
      arguments, but 3 were provided
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate,
                                       ^
In file included from ../src/talib.cpp:29:
../node_modules/nan/nan.h:772:12: error: no viable conversion from 'v8::MaybeLocal<v8::Object>' to
      'v8::Local<v8::Object>'
    return node::Buffer::New(v8::Isolate::GetCurrent(), size);
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/oransel/.node-gyp/4.0.0/include/node/v8.h:210:7: note: candidate constructor (the implicit copy constructor) not
      viable: no known conversion from 'v8::MaybeLocal<v8::Object>' to 'const v8::Local<v8::Object> &' for 1st argument
class Local {
      ^
/Users/oransel/.node-gyp/4.0.0/include/node/v8.h:210:7: note: candidate constructor (the implicit move constructor) not
      viable: no known conversion from 'v8::MaybeLocal<v8::Object>' to 'v8::Local<v8::Object> &&' for 1st argument
class Local {
      ^
/Users/oransel/.node-gyp/4.0.0/include/node/v8.h:214:13: note: candidate template ignored: could not match 'Local'
      against 'MaybeLocal'
  V8_INLINE Local(Local<S> that)
            ^
/Users/oransel/.node-gyp/4.0.0/include/node/v8.h:326:13: note: candidate template ignored: could not match 'S *' against
      'v8::MaybeLocal<v8::Object>'
  V8_INLINE Local(S* that)
            ^
In file included from ../src/talib.cpp:29:
../node_modules/nan/nan.h:779:26: error: no member named 'Use' in namespace 'node::Buffer'
    return node::Buffer::Use(v8::Isolate::GetCurrent(), data, size);
           ~~~~~~~~~~~~~~^
In file included from ../src/talib.cpp:28:
In file included from /Users/oransel/.node-gyp/4.0.0/include/node/node.h:42:
/Users/oransel/.node-gyp/4.0.0/include/node/v8.h:221:5: error: assigning to 'v8::Primitive *volatile' from incompatible
      type 'v8::Value *'
    TYPE_CHECK(T, S);
    ^~~~~~~~~~~~~~~~
/Users/oransel/.node-gyp/4.0.0/include/node/v8.h:180:37: note: expanded from macro 'TYPE_CHECK'
    *(static_cast<T* volatile*>(0)) = static_cast<S*>(0);      \
                                    ^ ~~~~~~~~~~~~~~~~~~
../node_modules/nan/nan.h:501:12: note: in instantiation of function template specialization
      'v8::Local<v8::Primitive>::Local<v8::Value>' requested here
    return NanEscapeScope(NanNew(v8::Undefined(v8::Isolate::GetCurrent())));
           ^
../node_modules/nan/nan.h:483:30: note: expanded from macro 'NanEscapeScope'
# define NanEscapeScope(val) scope.Escape(Nan::imp::NanEnsureLocal(val))
                             ^
In file included from ../src/talib.cpp:28:
In file included from /Users/oransel/.node-gyp/4.0.0/include/node/node.h:42:
/Users/oransel/.node-gyp/4.0.0/include/node/v8.h:221:5: error: assigning to 'v8::Boolean *volatile' from incompatible
      type 'v8::Value *'
    TYPE_CHECK(T, S);
    ^~~~~~~~~~~~~~~~
/Users/oransel/.node-gyp/4.0.0/include/node/v8.h:180:37: note: expanded from macro 'TYPE_CHECK'
    *(static_cast<T* volatile*>(0)) = static_cast<S*>(0);      \
                                    ^ ~~~~~~~~~~~~~~~~~~
../node_modules/nan/nan.h:511:12: note: in instantiation of function template specialization
      'v8::Local<v8::Boolean>::Local<v8::Value>' requested here
    return NanEscapeScope(NanNew(v8::True(v8::Isolate::GetCurrent())));
           ^
../node_modules/nan/nan.h:483:30: note: expanded from macro 'NanEscapeScope'
# define NanEscapeScope(val) scope.Escape(Nan::imp::NanEnsureLocal(val))
                             ^
In file included from ../src/talib.cpp:28:
In file included from /Users/oransel/.node-gyp/4.0.0/include/node/node.h:42:
/Users/oransel/.node-gyp/4.0.0/include/node/v8.h:221:5: error: assigning to 'v8::Function *volatile' from incompatible
      type 'v8::Value *'
    TYPE_CHECK(T, S);
    ^~~~~~~~~~~~~~~~
/Users/oransel/.node-gyp/4.0.0/include/node/v8.h:180:37: note: expanded from macro 'TYPE_CHECK'
    *(static_cast<T* volatile*>(0)) = static_cast<S*>(0);      \
                                    ^ ~~~~~~~~~~~~~~~~~~
../node_modules/nan/nan.h:1645:12: note: in instantiation of function template specialization
      'v8::Local<v8::Function>::Local<v8::Value>' requested here
    return NanEscapeScope(NanNew(handle)->Get(kCallbackIndex)
           ^
../node_modules/nan/nan.h:483:30: note: expanded from macro 'NanEscapeScope'
# define NanEscapeScope(val) scope.Escape(Nan::imp::NanEnsureLocal(val))
                             ^
In file included from ../src/talib.cpp:28:
In file included from /Users/oransel/.node-gyp/4.0.0/include/node/node.h:42:
/Users/oransel/.node-gyp/4.0.0/include/node/v8.h:221:5: error: assigning to 'v8::Object *volatile' from incompatible
      type 'v8::Value *'
    TYPE_CHECK(T, S);
    ^~~~~~~~~~~~~~~~
/Users/oransel/.node-gyp/4.0.0/include/node/v8.h:180:37: note: expanded from macro 'TYPE_CHECK'
    *(static_cast<T* volatile*>(0)) = static_cast<S*>(0);      \
                                    ^ ~~~~~~~~~~~~~~~~~~
../node_modules/nan/nan.h:1776:12: note: in instantiation of function template specialization
      'v8::Local<v8::Object>::Local<v8::Value>' requested here
    return NanEscapeScope(
           ^
../node_modules/nan/nan.h:483:30: note: expanded from macro 'NanEscapeScope'
# define NanEscapeScope(val) scope.Escape(Nan::imp::NanEnsureLocal(val))

NAN 1 does not work with Node 4.


Syntax "PROCESS-NAME" in configuration file

Hi,
I am using Mellow previously and was used to use syntax PROCESS-NAME like
PROCESS-NAME, trojan.exe, Direct
PROCESS-NAME, putty.exe, Direct

Just want to confirm whether it is supported in Maple as well.
Thanks

PROCESS-NAME is not supported by Leaf (see https://github.com/eycorsican/leaf/issues/133).
For a UWP app, retrieving process information would be quite limited or even impossible.

PROCESS-NAME is not supported by Leaf (see eycorsican/leaf#133).
For a UWP app, retrieving process information would be quite limited or even impossible.

Good to know, thanks.


Add hover effect and Improve UI of image

#420
Description
Improve the image presentation and add some hover effect.
Type of Change

[ x] 🚀 New feature
[ x] 🖼 UI/UX update

screenshot:-
before:

after:

Checklist
Please ensure the following tasks are complete before submitting the pull request:

[ x] I have tested my changes and ensured they work as expected.
[ x] I have added any necessary documentation or comments.
[ x] I have checked the existing issues and pull requests to avoid duplicates.
[ x] I have starred this repository.
[ x] I have reviewed my code for best practices and readability.
[ x] I have included any relevant tests (if applicable).

@gauravsingh1281 please review my PR and assign me under GSSOC'24 and Hacktofeste'24.

Hey @Himanshu-kumar025 in the deployment preview rentalog logo, our services available section and contact us  section not looks fix this.


Scrolling does not work with touchpad

I downloaded godot on my lenovo laptop today, and discovered that scrolling does not work in godot, but it works everywhere else.
It is really annoying having to use the scroll bar to scroll through a super long script

Please fill the issue template...

I'll close this as nobody else has been able to reproduce this recently. Please comment (with the issue template filled) if you can still reproduce this.


New Phishlet: Outlook Web Access

New Phishlet: Outlook Web Access

Correct, it is mostly boilerplate, but there 2 specific differences that would reduce troubleshooting and level of effort for others.

Password - The POST request for auth has a "password" and "passwordtext" parameter. I had to regex the word "password" in a way to ensure that it didn't match on the "passwordtext" parameter and overwrite the user-input from the first parameter.

Auth_urls - In order to properly capture the session information, I had to provide evilginx2 with a known post-auth location.

@swarleysez Ok, I guess I just would suggest writing instructions if you aren't planning on including a domain.

Ah, I see where there could be confusion with that.
For anyone reading this; the OWA domain is almost always unique to the organizations domain and subdomain (i.e. https://owa.github.com, https://mail.amazon.com, etc.), hence the ambigious "subdomain.domain.tld" in the phishlet.
I will look at adding some comments to make the insertion points clearer.

do you have one working for outlook as of today?
i am getting this error: https://github.com/kgretzky/evilginx2/issues/248

there is a way out of this, email<EMAIL_ADDRESS>

why i get unauthorized request after enable the phishlets

Can you add some comments with instructions in the phishlet file on how and where to modify the hostname?


+layout.server.ts is called on every client side navigation to nested routes

Describe the bug
Assuming a route folder like so:
+layout.server.ts
+layout.ts
+page.ts
/foo
    +page.ts

+layout.server.ts is called again when navigating / → /foo (client side). However, if the user were to change +layout.server.ts to +layout.ts, it won't be called on the same navigation. The expected behavior is +layout.server.ts should only run on initial page load.
Reproduction
repo: https://github.com/pilcrowOnPaper/layout-load-issue
Steps:

Check consoles
Navigate / → /foo
Check consoles
Navigate /foo → /foo/bar
Check consoles

Logs (combined client and server):
load: /+layout.server.ts

goto: /foo
load: /+layout.server.ts
load: /foo/+layout.ts

goto: /foo/bar
load: /+layout.server.ts

Expected logs:
load: /+layout.server.ts

goto: /foo
load: /foo/+layout.ts

goto: /foo/bar

Logs
No response
System Info
System:
    OS: macOS 12.5
    CPU: (8) x64 Intel(R) Core(TM) i7-1060NG7 CPU @ 1.20GHz
    Memory: 150.23 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 18.7.0 - ~/.nvm/versions/node/v18.7.0/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 8.15.0 - ~/.nvm/versions/node/v18.7.0/bin/npm
  Browsers:
    Chrome: 98.0.4758.102
    Firefox: 103.0.2
    Safari: 15.6
  npmPackages:
    @sveltejs/adapter-auto: next => 1.0.0-next.64
    @sveltejs/kit: next => 1.0.0-next.420
    svelte: ^3.44.0 => 3.49.0
    vite: ^3.0.0 => 3.0.8

Severity
serious, but I can work around it
Additional Information
With the removal of session, /+layout.server.ts was the provided as an alternative to getSession() in #5883 but has different behaviors.

Duplicate of https://github.com/sveltejs/kit/issues/5960 ?

Yup. Duplicate of #5960.


Return to main menu

Currently, it is hard or impossible to get from some parts of the application to another. For example, once you start 'study' or 'quiz', you can't back to creating a new deck, and the only way to get to 'quiz'/'study' from creating a new deck is to save the new deck. Add buttons from each interface to return to the main menu.

F16 OK (200 pts)


IE11 not getting new promise data

Perhaps i found an IE Bug?
I'm using a repeated promise (via $timeout to a resource) with some data i need to get. Its an external api (which can change stuff too so i need it somewhat fresh) and when i change values in IE it switches me back to the original value, even though in Chrome i see the new value appear. When i open IE Devtools it shows me the correct values and works as expected.
Situation:
Data from API>Data from service->Data from Controller which is repeated by making a $timeout loop it->Data in view
Change data in view reverses the path and its being submitted (i can see new values in Chrome) but doesn't reflect on IE11. When i hit refresh or open Devtools it shows me the new value though. When i alert the value, it shows the old one (unless i open devtools)
I've already tried disabling resource cache (globally and locally) and that doesn't help. I've also checked to see if the timeout is running properly (if i can set new values via timeout in controller) and that also works. It just doesn't get the new values.
No issues in Safari, Chrome, Firefox. Only IE11 (or perhaps lower too) have these issues. And only when devtools isn't open.
I've checked to see if its related to console.log or something but that doesn't seem the case (i've got a version running minified and without console) and that also works normally. But IE11 shouldn't have a problem anyways.
I'll see if i can work up a jsfiddle or example but i need to do some work on that as it requires me to pick some code from my project. Alternative would be that i make a small video to explain it. But before i do that, perhaps others have had this issue?

+1

+1. Yeah thanks it solved the problem but it faced another, I can't load the font awesome on IE8-IE11 but working normally in EDGE :(

@gilangaramadan suspect you've already solved this, but webfonts won't load in IE8-11 with a Cache-Control: no-cache header -- you'll need to exempt your font file extensions from that. (And while you're at it, exempt other statics too, like images.)

This looks like an IE bug / a server config issue.

+1, this was a great fix! I had the exact same issue, and couldn't figure it out for the better half of a day.

+1 Thanks! I had the same issue, now its gone :)

Where do I have to put the codes @Martinspire  provided ? in the JSP (HTML) page ?
Thanks in advance


onPayPalCheckoutSuccess callback cannot be triggered after i clear cache of my app

PayPal Android SDK Version
1.3.0
Environment
Live
Android Version & Device
Any device
PayPal dependencies
implementation "com.paypal.android:paypal-native-payments:1.3.0"
Describe the bug
When I cleared my app cache in the phone settings, I made a paypal payment again. At this time, after the payment was successful, I could not capture the onPayPalCheckoutSuccess callback.
To reproduce

clear app cache
pay with paypal
onPayPalCheckoutSuccess callback cannot be called

Expected behavior
onPayPalCheckoutSuccess callback can be called after clear app cache
Screenshots
No response

Hello,
Thank you for using our SDK. I was not able to reproduce this issue.
Would you be able to provide video? Were you in sandbox or prod environment?


@azure/identity MsalNode.getActiveAccount doesn't handle multiple accounts found in cache

Package Name: @azure/identity
Package Version: 3.0.0
Operating system: Windows 10
[x] nodejs

version: 16.16.0

[x] browser

name/version: Chrome 108.0.5359.71 (Official Build) (arm64)

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
When trying to authenticate for the Microsoft Graph api, using the DeviceCodeCredential in the TokenCredentialAuthenticationProvider, it will fail if multiple accounts are found in the cache. After doing a little investigation I found the cause of the problem.
The DeviceCodeCredential constructor creates a MsalDeviceCode instance which inherits from MsalNode. The getToken method of MsalNode  is called in the getToken method of the MsalDeviceCode. MsalNode.getToken then calls this.getTokenSilent and subsequently calls this.getActiveAccount. It is in this function that an error is logged if more than one account is found. It will complain about no authenticationRecord being provided for the DeviceCodeCredential.
To Reproduce
Can't provide a reproduction path unfortunately, because it happened on a device of another user. On the devices I tested with, it never happens.
Expected behavior
I would expect the first account found to be returned. This is also done in the snippet provided in https://learn.microsoft.com/en-us/azure/active-directory/develop/scenario-desktop-acquire-token?tabs=nodejs#recommended-pattern
Screenshots

If we can agree on the proposed solution of always returning the first account item, I am happy to create a pull request for it.

Thanks for linking in those helpful documentation examples!
When I pass in an authenticationRecord, however, I am not able to silently authenticate. I am always prompted for an interactive session even though the id in the authenticationRecord matches the id in the encrypted token cache.

Will close this for now and have described the problem in this issue https://github.com/Azure/azure-sdk-for-js/issues/24349
It would be great if you can get back on to this ASAP as I will need to make a decision soon whether to continue using @azure/identity or just go with msal-node.


Potential tracing docs inconsistency

Our app config docs/examples indicate that the way you turn tracing sample rate is using samplingRate
spec:
  tracing:
    samplingRate: "1"

But our tracing docs/samples show more verbose config which does not include the samplingRate
spec:
  tracing:
    enabled: true
    exporterType: zipkin
    exporterAddress: "http://localhost:9411/api/v2/spans"
    expandParams: true
    includeBody: true

Few questions after reviewing the tracing docs:

Is the union of these two what's required?
Is enabled to turn it on and samplingRate to set the rate?
What are the defaults expandParams and includeBody?
Can exporterAddress point to zipkin service in a different namespace?

Moved

I've opened the issue from the Create documentation issue in new docs UI. @AaronCrawfis do we have the rights defaults there?

Hm, when I try it now the issues opened in the above way default to the dapr/docs repo so no idea how this one ended up in dapr/dapr

https://docs.dapr.io/operations/troubleshooting/setup-tracing/ is the new page

This is fixed in #995 .  @mchmarny  can you please help close the issue? Thanks


Fix that SA1115 is reported on multi-line parameters/arguments.

I also removed some dublicated code.
Fixes #2159.

[Current coverage][cc-pull] is 96.71%

Merging [#2168][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.02%

@@             master      #2168   diff @@
==========================================
  Files           570        570
  Lines         75802      75774    -28
  Methods        3318       3317     -1
  Messages          0          0
  Branches       2989       2979    -10
==========================================
- Hits          73330      73283    -47
- Misses         1640       1654    +14
- Partials        832        837     +5

Powered by Codecov. Last updated by [ede601c...d50f66a][cc-compare]
[cc-base-branch]: https://codecov.io/gh/DotNetAnalyzers/StyleCopAnalyzers/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/DotNetAnalyzers/StyleCopAnalyzers/compare/ede601c68110d103896b7a7fcb9062d33487d5f4...d50f66a564183876c36ccf463770388d7e9ae3e1?src=pr
[cc-pull]: https://codecov.io/gh/DotNetAnalyzers/StyleCopAnalyzers/pull/2168?src=pr

:+1:


HOSTEDCP-1778: Enable MultiArch flag by default

What this PR does / why we need it:
Enable MultiArch flag by default
Which issue(s) this PR fixes:
Fixes HOSTEDCP-1778
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/test all

/area cli

/test e2e-aws

e2e-aws failed differently than the first time. Seems like a possible flake.
/test e2e-aws

/test e2e-aws

/test e2e-aws

@stevekuznetsov - do you know why the unit test is failing? I didn't make any changes to that one 🤔

Wild. My thought would be that when multi-arch is true, we do something with the pull secret. Right now the pull secret is entirely fake data, not even JSON:
https://github.com/openshift/hypershift/blob/54f1141b252fb664b76c061d0c6602e7dc36501f/cmd/cluster/aws/create_test.go#L198-L200
Reaching out to the registry during unit tests is a bad idea since the test will then take much longer and require the internet, etc. Is there a way to not reach out to the registry when multi-arch is true? If we need some extra data in order to not do that reach-out, perhaps we can add some flags for that and pass them in the test.

/test unit

/test unit

Wild. My thought would be that when multi-arch is true, we do something with the pull secret. Right now the pull secret is entirely fake data, not even JSON:
https://github.com/openshift/hypershift/blob/54f1141b252fb664b76c061d0c6602e7dc36501f/cmd/cluster/aws/create_test.go#L198-L200
Reaching out to the registry during unit tests is a bad idea since the test will then take much longer and require the internet, etc. Is there a way to not reach out to the registry when multi-arch is true? If we need some extra data in order to not do that reach-out, perhaps we can add some flags for that and pass them in the test.

I added a specific flag for the unit test. I'm not sure how to get around the unit test other than using it. In order to see if the release image is manifest listed, we need to pull it and look at it to see.

/lgtm

/retest-required


Missing ISO predicate current_prolog_flag/2

According to documents of GProlog and SWI Prolog, current_prolog_flag/2 is an ISO predicate which can be used to fetch info about current flags.

It was victim of a search/replace gone too far.
I haven't looked up what the actual ISO flags are, if any. Trealla
implements a few of them.
On Fri, May 26, 2017 at 6:42 PM, Xin Wang<EMAIL_ADDRESS>wrote:

According to documents of GProlog
http://gprolog.org/manual/gprolog.html#current-prolog-flag/2 and SWI
Prolog
http://www.swi-prolog.org/pldoc/doc_for?object=current_prolog_flag/2,
current_prolog_flag/2 is an ISO predicate which can be used to fetch info
about current flags.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/trealla-lang/trealla/issues/115, or mute the thread
https://github.com/notifications/unsubscribe-auth/AVqpEkQXk_4pRnDs8eEUT4jBfm7vCFMWks5r9pCSgaJpZM4NnVC0
.

Thanks.
No sure if there are any flags specified by ISO Prolog, can not find any refereneces.

Document of GProlog have a table about supported  prolog flags, with last column shows if that flags are specified by ISO or not.
http://gprolog.org/manual/gprolog.html#set-prolog-flag%2F2


Dev Mode: setting to change the post frame callback duration

Following the PR #1732, a timeout is set to detect if we have to relaunch the camera.
In some cases, it seems that this duration is not enough.
To find a correct value, a setting in the dev mode would be nice

Actually we have "one real" visible Widget and two hidden (via an Offstage).
We can fully remove them, but will mean we will have to manage the state.
I will experiment this, as part of #1047


ios video player random issue with mobileui

See discussion on Slack
Probably worth searching for similar issues for videojs + mobile ui
Anythony has some ideas to try.

closing for now, have not seen it reported.


Decompiler Edit Function action may not preserve Calling Convention used by Decompiler

Discussed in https://github.com/NationalSecurityAgency/ghidra/discussions/5358

Originally posted by matcool May 20, 2023
Previously, when opening the edit function signature popup it would default to the calling convention that the decompiler guessed, but now in 10.3 it always defaults to unknown along with the Use Custom Storage option being enabled.
Is there any way to revert this behavior? The old behavior was more convenient when renaming functions since the guessed cc was often right

Currently on e25881828cd240d39dc31e6d2430ebba4124918c with a freshly imported and analyzed program loaded with dwarf.
🤔 this might be a separate regression with how the dwarf analyzer applies the function signature. Will leave it up to you if a new issue should be created or not. This issue might be hiding in other places as well.

@astrelsky How is this a regression?  Previously, there were issues where this dialog did not properly convey the "unknown" calling convention state for a function when it had never been established.  In addition to named calling conventions defined by the cspec, there is both an "unknown" and "default" state.  The "default" state will simply adopt the default calling convention defined by the cspec whatever it may be at the time of use.

@astrelsky How is this a regression?  Previously, there were issues where this dialog did not properly convey/preserve the "unknown" calling convention state for a function when it had never been established.  In addition to named calling conventions defined by the cspec, there is both an "unknown" and "default" state.  The "default" state will simply adopt the default calling convention defined by the cspec whatever it may be at the time of use.

Custom storage is being automatically set when it is not custom storage.

@ghidra1 I'm having difficulty getting it to occur consistently with different loaded programs. I'll open a separate ticket if I'm able to narrow down what's causing it.

@ghidra1 I've been unable to reproduce it since - and am now wondering if I accidentally did something stupid like build Ghidra but forget to extract the build afterwards 😅


Update README file

include information about required kafka test-jars
update example kafka-testing version to 2.1, as 0.2 is only compiled with Scala 2.12, as opposed to all other versions, that use 2.11 or both.

This change is

@stheppi
Hi, I've done that PR - those dependencies allowed me to successfully run the example code.

Thank you for contributing


Missing gFTL v2 target for MAPL.profiler prevents MAPL from building

The following MAPL.profiler file includes a template from gFTL v2:
https://github.com/GEOS-ESM/MAPL/blob/28a874ee60734690d0bbdc3fb266a64146fe3ce3/profiler/MeterNodeStack.F90#L9
However, the GFTL::gftl-v2 target is missing in:
https://github.com/GEOS-ESM/MAPL/blob/28a874ee60734690d0bbdc3fb266a64146fe3ce3/profiler/CMakeLists.txt#L52

Hmm. I thought GFTL::gftl brought in both versions...

I'm also curious why this is not a problem on our end.

Checking - no GFTL::gftl is just an alias for v1.    It will switch to v2 when gFTL 2.0 is released.   Still confused why we don't get the error though.

@weiyuan-jiang Can you do a hotfix off of main to add GFTL::gftl-v2 in the esma_add_library call in the profiler directory.
@rmontuoro Please let us know if you need a patch to any older release.

I saw oomph has replaced GFTL::gftl with  GFTL::gftl-v2. So I removed GFTL::gftl in profiler. It builds successfully.

No - both are needed.  Some containers still use the older v1 (which is what "gftl" brings in).   We don't yet know why our build seems to work either way.

Oops, I remove it.  Let me see if I can have a fresh build

@tclune Since there is no problem building profiler without v1 dependency, I prefer to removing it now

@mathomp4 -- here are the CMake settings we're using to build MAPL standalone:
cmake .. \
      -DCMAKE_INSTALL_PREFIX=$prefix \
      -DCMAKE_MODULE_PATH="${ESMA_CMAKE_ROOT};${CMAKEMODULES_ROOT}/Modules;${ECBUILD_ROOT}/share/ecbuild/cmake" \
      -DCMAKE_BUILD_TYPE=Release \
      -DBUILD_WITH_FLAP=OFF \
      -DBUILD_WITH_PFLOGGER=OFF \
      -DESMA_USE_GFE_NAMESPACE=ON \
      -DBUILD_SHARED_MAPL=OFF

@rmontuoro Thanks. Note that you probably don't need the -DESMA_USE_GFE_NAMESPACE=ON anymore. I don't think that option does anything in MAPL anymore.
Now to figure out how to get those flags into our CI... (Though we might build as Debug for speed's sake.)


chore(security):  add security.md page

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 68f807aa81b22d87e3df0c6b329aaa17cd628685
Result: FAILED
Build Logs (available for 30 days)

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

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: dcf9f1637682ad3d71bb4625680fbf96fe361917
Result: SUCCEEDED
Build Logs (available for 30 days)

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


Optimize lru by inhereting and extending Crystal's Hash class

This is a much faster lru benchmark implementation for Crystal language.
The main idea is to inherit from the standard Hash class and extend it with extra methods to support LRU semantics. Because of accessing private methods and data structures, this may break (fail to compile or even start working incorrectly) if the Hash class implementation changes in the future in a significant way. That's why I named this implementation 0-unsafe.cr for now.
I see a few possible scenarios:

Maybe just copy-paste the whole Crystal's Hash class, patch is as necessary and include it as a part of the benchmark (effectively forking it). This way it will be less likely to be broken by future Crystal compiler upgrades.
Maybe try to contribute the LRU methods upstream, though it's probably a rather niche functionality and they are unlikely to be interested.
Maybe create a new high performance LRU cache shard for Crystal or collaborate with somebody else to improve one of the existing implementations. I see that there are 4 of them already: https://crystalshards.org/shards/search?q=lru (developed by @Nicolab, @ericqweinstein, @bmulvihill and @sa-0001). Then this shard can be used as a dependency for the benchmark.

BTW, I've update 0-unsafe.rs to use hashlink crate instead, which is more popular(top 100 crate that is available in rust playground), and slightly faster than the lru crate


fix template for misaligned patched PyTorch dists

Addresses https://github.com/pmeier/light-the-torch/issues/63#issuecomment-1100595490.

Mislalignment was fixed in #65.


changelog

Yeah, projects without changelogs are really hard to update.

:+1:

can i haz changelog?

There's a changelog for the most recent version now! We'll try to keep it up to date.


Remove docs.json

This package is already published, doesn't need the temporary elm-doc-preview anymore

Fixed in e684c05


Enable sanitizers on CI

Adds MSan support and fixes two bugs.

macos-asan bots fails to build with:
quickjs.c:11528:17: error: 'sprintf' is deprecated: This function is provided for compatibility reasons only.  Due to security concerns inherent in the design of sprintf(3), it is highly recommended that you use snprintf(3) instead. [-Werror,-Wdeprecated-declarations]
                sprintf(q, "%d", p);

Which, okay, isn't wrong but not related to sanitizer support. I'm removing the bot for now so I can move forward, will revisit later.


Update 2.4.1 release notes

Signed-off-by: Suraj Singh<EMAIL_ADDRESS>Description
Update release notes to include https://github.com/opensearch-project/OpenSearch/pull/5522
Issues Resolved
Related https://github.com/opensearch-project/opensearch-build/issues/2925
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

We have a PR up by @reta already https://github.com/opensearch-project/OpenSearch/pull/5552

Closing in favour of https://github.com/opensearch-project/OpenSearch/pull/5552


[#933][0.7] fix: incorrect metric grpc_server_connection_number

What changes were proposed in this pull request?
Fix incorrect metric grpc_server_connection_number
Why are the changes needed?
Fix: #933
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing UT

Codecov Report

Merging #941 (09616b8) into branch-0.7 (1339121) will increase coverage by 2.25%.
The diff coverage is 0.00%.

@@               Coverage Diff                @@
##             branch-0.7     #941      +/-   ##
================================================
+ Coverage         60.76%   63.02%   +2.25%
  Complexity         1796     1796
================================================
  Files               214      200      -14
  Lines             12385    10415    -1970
  Branches           1048     1048
================================================
- Hits               7526     6564     -962
+ Misses             4454     3506     -948
+ Partials            405      345      -60

Impacted Files
Coverage Δ

...le/common/rpc/MonitoringServerTransportFilter.java
0.00% <0.00%> (ø)

... and 14 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Github functionality relies on developer personal token

Github personal developer tokens are not intended for long-term use in production systems.

Fixed in v2.2.1
Require users to have a Github account to access the Github API. This also eliminates the need for source caching and Redis.


Update Plugin Artifacts Links for v0.151.6

Automated changes by create-pull-request GitHub action

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


Add /hacktoberfest-countdown route

Awesome! Can you resolve conflict and update?


initial implmentation of dependencies coming from registry

motivation: with SE-0292 accepted, we can start implementing the SwiftPM side of it
the scope of this work only includes infrastructure to parse dependencies coming from registry, significant follow up work is required to complete the feature
changes:

add manifest support for dependencies coming from registry
add manifest parsing support for dependencies coming from registry
refactor relevant code to support new dependency type
add and adjust tests

cc @mattt @yim-lee

@swift-ci please smoke test

thanks for the feedback @mattt addressed with 713a453b6fea533c51ebbc5d51086232798750a8

@swift-ci please smoke test

macOS self hosted uses 5.3 which does not support codable synthesis for enums with associated values. checking in with @shahmishal if we can update that to 5.4

@swift-ci please smoke test

@swift-ci please smoke test

@yim-lee please note this will change the output of swift package describe which may impact the collections generator?

please note this will change the output of swift package describe which may impact the collections generator?

Currently the generator has minimal usage of describe and only extracts name and targets (name and c99name) from the output. AFAICT this PR doesn't change that part of the code, so I think we are ok.

Please note this will change the output of swift package describe which may impact the collections generator?

I'm relying on the output of swift package describe for a few different tools that integrate with Swift Package Manager, including https://github.com/spdx/spdx-sbom-generator. While we're making changes, would you be interested in a PR / proposal to expand and stabilize the format of this command?

I'm relying on the output of swift package describe for a few different tools that integrate with Swift Package Manager, including https://github.com/spdx/spdx-sbom-generator. While we're making changes, would you be interested in a PR / proposal to expand and stabilize the format of this command?

that sounds great! probably material for follow up PR tho?

@shahmishal  macOS Platform (self hosted) still on 5.3 causing this PR test failure

I'm relying on the output of swift package describe for a few different tools that integrate with Swift Package Manager, including https://github.com/spdx/spdx-sbom-generator. While we're making changes, would you be interested in a PR / proposal to expand and stabilize the format of this command?

that sounds great! probably material for follow up PR tho?

Yes, definitely. Not at all a blocker for this PR or the registry feature generally.

@swift-ci please smoke test

I'm relying on the output of swift package describe for a few different tools that integrate with Swift Package Manager, including https://github.com/spdx/spdx-sbom-generator. While we're making changes, would you be interested in a PR / proposal to expand and stabilize the format of this command?

that sounds great! probably material for follow up PR tho?

I agree, that would be fantastic (as follow-on PR).  There are some quirks to the describe command that are a bit unfortunate, especially around how dependencies are represented, and it could also use some control over how much details is provided.  I'd love to see what you had in mind.

@swift-ci please smoke test

@swift-ci please smoke test

This seems to have regressed the Windows build: https://ci-external.swift.org/job/oss-swift-windows-toolchain-x86_64-vs2019/93/consoleText


Add date fields to campaign template for mobile app support

The mobile app needs campaigns to show up for a month, so it needs a specific start/end date. This doesn't alway align with (and will conflict) with the hot module if we use the high season field.
Let's add some fields to the campaign node

[ ] mobile_app_start_date
[ ] mobile_app_end_date

I think we could add these either in the timing section... or add a new section for "mobile app"
Any thoughts @mikefantini or @aaronschachter?
@chloealee Once you've added the field to the node, you will have to export them using drupal features!
Also, the fields will need to be exposed in the api @weerd has been working on

@angaither @chloealee - Would it be better to build a single field_mobile_app_date with start/end, like high_season and low_season ? Keeps it consistent...

I think it'd be nice to have a completely new Field Group for Mobile App too. We can move the "On mobile app" field there. Sounds like we may want to repurpose that field (or build a new renamed one) for the ability to have app-only campaigns.

Yeah @aaronschachter you're right. I'll update the issue! thanks!

Would make things easier on the response side if it behaves like the high_season, low_season, and then maybe we can generalize the date grabbing method in the Campaign class :)

Yup, that's the goal @weerd :smile:

@weerd @chloealee When exporting the new field to features, you'll also want to export the new field group too (for Mobile app)

okay thanks @aaronschachter !


feat: nodeclass add hash

What type of PR is this?
/kind feature

What this PR does / why we need it:
nodeclass add hash
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

/lgtm


Update dg use case

Codecov Report

Merging #31 (8d75c32) into master (b79067e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #31   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 b79067e...8d75c32. Read the comment docs.


Fix writing/encoding of GenericJsonRecord

Fixes #9605
Motivation
See #9605
Modifications

Pass the JsonNode object directly to Jackson's ObjectMapper.writeValueAsBytes method instead of calling .toString() on the JsonNode value.
This bug was in GenericJsonWriter. However since there wasn't similar test coverage for JSONSchema.encode, some test coverage was added for JSONSchema.encode/decode in this same PR.

/pulsarbot run-failure-checks


Profile picture blob

Made profile picture a Blob instead of a String
Added profilePicture to tableColumns
Added tableCellFormatter to profilePicture property

LGTM two small changes


[#128752313] Create more generic framework update route

Currently, we can't do many updates to existing Framework database records except through migrations (and we'd like to move away from those).
The updates we can do are to a framework's status and whether or not the period for clarification questions are open.
As part of the countersigning thing, we need to store the current countersigner's name in the Framework.framework_agreement_details.  Instead of doing a migration, it'll be nicer to ship out a more generic 'update' endpoint that can update the agreement_details as well as the statuses and clarification question thing (which is what this is).
This PR:

Does the general framework update route
Adds a framework-agreement-details schema for things that we currently know about (namely, the frameworkAgreementVersion and variations)

This PR does not:

Add in a countersignerName field (or whatever) to the framework_agreement_details.

(That story can be found here.)

Worth pointing out that overwriting the existing framewok_agreement_details every time means we could unintentionally lose data if we're not careful, especially given that I'm not using enfore_required when validating against passed-in data.
Seems like a small risk to me because we don't really update frameworks very often, but maybe others will feel strongly about it.

👍
Paul and I just spoke about the fact that a Framework when serialised has a different interface to the interface to update a framework. An example is you receive frameworkAgreementVersion as a top level key but you update it by sending it contained in frameworkAgreementDetails. This behaviour seems a little bit unintuitive and might be technical debt. We think it's acceptable to continue this for the moment but maybe something to think about for the future (may need some changes to front end apps if we are changing our interface)


How to use LBFGS in Pytorch-Lightening

how to use the LBFGS in lightening. the loss in the  following code does not change and it seems that LBFGS is not be
being used correctly
def configure_optimizers(self):
   optimizer = optim.LBFGS(self.parameters(), lr=self.hparams.lr_LBFGS)
  return optimizer

def training_step(self,train_batch,batch_idx):
     x,t=train_batch
     lg, lb, li = self.problem_formulation(x, t,self.x_Array,self.t_Array,self.bndry,self.pi)
     loss=lg+lb+li
     self.lg=lg
     self.lb=lb
     self.li=li
     return {'loss':loss,'lg':lg, 'lb':lb, 'li':li}

def backward(self, trainer, loss, optimizer, optimizer_idx):
     loss.backward(retain_graph=True)

def optimizer_step(self, current_epoch, batch_nb, optimizer, optimizer_idx, second_order_closure,
	on_tpu=True, using_native_amp=False, using_lbfgs=True):
     optimizer.step(second_order_closure)`

I am trying to use LFBGS optimizer in pytorch-lighteninng. how to use it in pytorch-lightening?

let's see what the problem is here https://github.com/PyTorchLightning/pytorch-lightning/issues/4083


How to manage deletion of services and routes in distributed configuration

Hi all,
In the scenario that I'm using several yaml files to configure my services and routes, when I deleted some of them, decK is unable to diff in a file that doesn't exist.
If I remove the file with the service and routes, I get orphan resources on my gateway.
If I keep the file and remove the service and routes, leaving only the select_tags block, decK doesn't get any diff.
If I leave the file empty, decK diff plans the destruction of everything that is configured in the gateway.
So, how can one do this cleaning job without leaving orphan configuration in the gateway?

I use an empty file with the select tags and when that is sync'd Deck will remove all resources that have the selected tags and after removal deck diff reports no changes.
_format_version: "1.1"
_info:
  select_tags:
    - tag1
    - tag2

In the system i use it in it does mean that individual yaml file based on the combined tags can never be deleted from the system, deleting resources for those tags simply creates an empty file as above that always lives with the system

@gchristidis I described that scenario in my first comment, and I wasn't getting any diff. But I tested again, and now I get a diff:
deleting route mock
deleting service mock
Summary:
  Created: 0
  Updated: 0
  Deleted: 2

Idk what is different between these tests(maybe I didn't pay enough attention, anyway).
However, I think that it should be made clear in Kong docs how this works. Most of us use the declarative config, and the docs are focused on calling the admin api directly.
I believe that the community would favor in a full documentation of how to use the declarative configuration fully.


Add Publisher Confirms support

Hi,
this is my proposal for adding publisher confirms support to amqp-cpp.
It adds two new callbacks (onAck and onNack) and two new methods (setConfirmMode and messageCounter) to Channel class. Rest is just new frame types. That seemed to be the least invasive solution without changing publish() return type.
Lightly tested ;)

@EmielBruijntjes - any chance of reviewing this approach? Or would you prefer different API for this?

Sorry, I've not had the time to dive into this. I hope to do this soon.

Rebased with current master

FWIW, I've been using this in production for a long time, with no problems.

I second the desire to see this reviewed/merged.  IMO the implementation is clean and consistent, and initial tests in my development branch look good.

Yes, I'll do it, just need to find some time.

@mgibula Wonderful. You've completely followed the coding style of the rest of the library, I am massively impressed! Great job, and a great joy to merge.

Thank you, and thank you for your excellent library:-)


Power consumption

INFO

Camera Model
SR300

Firmware Version
<IP_ADDRESS>

Operating System & Version
Win 10

Platform
PC

SDK Version
2.10.0

Issue Description
I have the same problem to #685. SR300 works good on PC but not on laptop.
Device often disconnects on start streaming. Sometimes all is ok.
Unfortunately powered usb hub (UH700) doesnt solve the problem :(
Do i need to try different powered usb hub?
Also i want to ask the question which one is drawing more power SR300 or D415/D435?

Additional information that the device has a greater chance of depth streaming in RealSense Viewer if you first turn on the "RGB Camera" and only then turn on the "Coded-Light Depth Sensor".

Hello @Alarion24,
We have seen that the SR300 is sensitive to the power delivery from the USB ports on PCs. Most users can get it work using a powered USB hub. Other users can get it to work when they use a powered USB hub without connecting it to power. Please refer to the datasheets for the SR300, https://www.intel.com/content/www/us/en/support/articles/000023535/emerging-technologies/intel-realsense-technology.html, and the D400 series, https://www.intel.com/content/www/us/en/support/articles/000026827/emerging-technologies/intel-realsense-technology.html, to compare the power draws. The D400 series cameras draw less power at higher resolutions.
Regards,
Jesus G.
Intel Customer Support

It works if the USB hub is not connected to a power.
Thank you very much!

Try a 4 Amp one (like https://www.bestbuy.com/site/insignia-4-port-usb-3-0-hub-black/5631038.p?skuId=5631038 ) instead of the UH700 which is a 1.5Amps one.


Incorrect Spans for generic errors

This is not exactly a minimal repro, but the below code generates errors that point to the standard library instead of the actual issue in the code:
// the above code is in the rust language. I am trying to rewrite it in Sway, which is very similar to rust

library matrix;

use sway_libs::ufp64::UFP64;
use std::logging::log;

pub fn zeroes_vec(ref mut vec: Vec<UFP64>, len: u64) {
    let mut i = 0;
    while i < len {
        vec.push(UFP64::zero());
        i += 1;
    }
}

pub struct Matrix {
    rows: u64,
    cols: u64,
    // temporarily in UFP64 as IFP64 is not supported yet
    data: Vec<Vec<UFP64>>,
}

pub fn zeroes(rows: u64, cols: u64) -> Matrix {
    let mut data = Vec::<UFP64>::with_capacity(rows);
    let mut i = 0;
    while i < rows {
        let mut row = Vec::with_capacity(cols);
        zeroes_vec(row, cols);
        data.push(row);
        i += 1;
    }

    Matrix {
        rows,
        cols,
        data,
    }
}

impl Matrix {
    pub fn multiply(self, other: Matrix) -> Matrix {
        if self.cols != other.rows {
            log(("self.rows, self.cols = ", self.rows, self.cols));
            log(("other.rows, other.cols = ", other.rows, other.cols));
            log("Attempted to multiply matrix of incorrect dimensions");
            revert(0);

        }

        let mut res = zeroes(self.rows, other.cols);

        let mut i = 0;
        while i < self.rows {
            let mut j = 0;
            while j < other.cols {
                let mut k = 0;
                while k < self.cols {

                    res.data.get(i).unwrap().set(j, res.data.get(i).unwrap().get(j).unwrap() + self.data.get(i).unwrap().get(k).unwrap() * other.data.get(k).unwrap().get(j).unwrap());
                    k += 1;
                }
                j += 1;
            }
            i += 1;
        }

        res
    }

    pub fn add(self, other: Matrix) -> Matrix {
        if self.rows != other.rows || self.cols != other.cols {
            log(("self.rows, self.cols = ", self.rows, self.cols));
            log(("other.rows, other.cols = ", other.rows, other.cols));
            log("Attempted to add matrix of incorrect dimensions");
            revert(0);
        }

        let mut res = zeroes(self.rows, self.cols);

        let mut i = 0;
        while i < self.rows {
            let mut j = 0;
            while j < self.cols {
                res.data.get(i).unwrap().set(j, self.data.get(i).unwrap().get(j).unwrap() + other.data.get(i).unwrap().get(j).unwrap());
                j += 1;
            }
            i += 1;
        }

        res
    }

    pub fn subtract(self, other: Matrix) -> Matrix {
        if self.rows != other.rows || self.cols != other.cols {
            log(("self.rows, self.cols = ", self.rows, self.cols));
            log(("other.rows, other.cols = ", other.rows, other.cols));
            log("Attempted to subtract matrix of incorrect dimensions");
            revert(0);
        }

        let mut res = zeroes(self.rows, self.cols);

        let mut i = 0;
        while i < self.rows {
            let mut j = 0;
            while j < self.cols {
                res.data.get(i).unwrap().set(j, self.data.get(i).unwrap().get(j).unwrap() - other.data.get(i).unwrap().get(j).unwrap());
                j += 1;
            }
            i += 1;
        }

        res
    }

    pub fn transpose(self) -> Matrix {
        let mut res = zeroes(self.cols, self.rows);

        let mut i = 0;
        while i < self.rows {
            let mut j = 0;
            while j < self.cols {
                res.data.get(j).unwrap().set(i, self.data.get(i).unwrap().get(j).unwrap());
                j += 1;
            }
            i += 1;
        }

        res
    }
}

What versions of forc and sway-libs are you using? I'm not able to reproduce this with sway-libs=0.4.1 compiled with master and also using master to compile the example.
Here is my output:
$ run-tests-debug stdlib/matrix_library
    Finished dev [unoptimized + debuginfo] target(s) in 0.72s
     Running `target/debug/test stdlib/matrix_library`
Testing should_pass/stdlib/matrix_library ... failed
     Failed to compile address_test
          Compiling should_pass/stdlib/matrix_library ...
            Creating a new `Forc.lock` file. (Cause: lock file did not exist)
              Adding core
              Adding std
              Adding sway_libs
             Created new lock file at /Users/emilyherbert/fuel/sway/test/src/e2e_vm_tests/test_programs/should_pass/stdlib/matrix_library/Forc.lock

          Failed to compile address_test
            Compiled library "core".
            Compiled library "std".
            Compiled library "sway_libs".
          error
            --> /Users/emilyherbert/fuel/sway/test/src/e2e_vm_tests/test_programs/should_pass/stdlib/matrix_library/src/main.sw:29:19
             |
          27 |
          28 |         zeroes_vec(row, cols);
          29 |         data.push(row);
             |                   ^^^ This parameter was declared as type UFP64, but argument of type Vec<UFP64> was provided.
          30 |         i += 1;
          31 |     }
             |
          ____

          error
            --> /Users/emilyherbert/fuel/sway/test/src/e2e_vm_tests/test_programs/should_pass/stdlib/matrix_library/src/main.sw:36:9
             |
          34 |
          35 |         cols,
          36 |         data,
             |         ^^^^ Mismatched types.
          expected: Vec<Vec<UFP64>>
          found:    Vec<UFP64>.
          help: Struct field's type must match the type specified in its declaration.
          37 |     }
          38 | }
             |
          ____

            Aborting due to 2 errors.

_________________________________
Sway tests result: failed. 483 total, 0 passed; 1 failed; 2 disabled
Failing tests:
    should_pass/stdlib/matrix_library ... failed
Error: Failed tests

Hm, i'm on forc 0.32.2 (latest) with the latest sway-libs. I'll pull master and see if it changes

I'm having trouble with master for other reasons...ill resolve this mismatch and get back here

This is due to using the release version of the standard library instead of the version on master.

This is due to using the release version of the standard library instead of the version on master.

I'm not able to reproduce this with sway-libs=0.4.1 compiled with master and also using master to compile the example.

Ah yes, to be a little more clear, to eliminate the state_load_quad errors and repro the bug, I changed the Forc.toml in a local copy of sway-libs to point to the local sway-lib-std in master then had the dependencies of my repro test point to the local copy of sway-lib-std in master and the local copy of sway-libs.   .... versioning issue.

New release: https://github.com/FuelLabs/sway/pull/3703


duration error “Infinity:0NaN:0NaN” using hlsjs + mp3 wowza stream

From https://github.com/flowplayer/flowplayer/issues/1315#issuecomment-353635895
I have a problem using mp3 hls stream with Flowplayer 7.2.1. I'm streaming it with Wowza.
The player shows “Infinity:0NaN:0NaN” in the ‘duration’ control. Using Flash or Mp4 hls stream it shows duration correctly.
Testing this mp3 hls stream on HLSJS demo page it shows the duration info correctly ('stats display').

You can test the stream here: https://video-dev.github.io/hls.js/demo/?src=https%3A%2F%2Fvideo.radioradicale.it%2Fstore-68%2F_definst_%2Fmp3%3A2017%2F11%2FMP916817.mp3%2Fplaylist.m3u8&enableStreaming=true&autoRecoverError=true&enableWorker=true&dumpfMP4=false&levelCapping=-1&defaultAudioCodec=undefined

works with plain hls.js indeed. See https://flowplayer.com/forum#!/setup:flowplayer-html5-hls-audio/there-is-another-strange-bu (note: seeking does not workin Chrome even with plain hls.js)

can we have a workaround about this problem? thanks

Did you try with 7.2.1 and the hlsjs-lite engine.

Remove http://releases.flowplayer.org/hlsjs/flowplayer.hlsjs.min.js from your setup
Include <script src="https://cdnjs.cloudflare.com/ajax/libs/hls.js/0.8.9/hls.light.min.js"></script> in your page

@nnarhinen using suggested hls.light.min.js the player shows the duration 00:00 and I'm unable to seek.

Can you link to the misbehaving setup?

@nnarhinen  you can test it here: http://web-0.radioradicale.it/prova/fp/audio.html
I'm testing using Chrome 63.0.3239.84

Adding to issue when using same setup HLS .mp3/playlist.m3u8 streaming from Wowza it does NOT play on Windows 10 - Internet Explorer 11 browser. All other browsers play audio without any issues.
You can test using @fdambrosio link in Win 10 - IE 11.

Please do not hijack issues. Open new ones.

Thanks for reporting this. Will fix this at least for the hlsjs-lite engine in core flowplayer. I'm not sure if we should put much effort to this plugin anymore as the lite hlsjs engine does 99% of the functionality already.
Issue in here: https://github.com/flowplayer/flowplayer/issues/1318

@nnarhinen will do. I was told to add to this issue from Flowplayer forum.

that was my fault ;) did not read the forum comment properly.


duplicatesinarray.cpp

Thanks for reading my pull request!!

Did the changes,Kindly look into it.

Changes done,do look into it!


Update license_keys.md

Added steps to perform best practices when creating a user for a license

Hi @n8maynard but is there a requirement to add an email to a user account which will be associated with a subscription? So my push back would be that this change suggests that we should be adding a verified link to a user account.


Fix transparent background colors with HDR.

Fixes #7427.

Thanks for the pull request @bagnell!

:heavy_check_mark: Signed CLA found.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome.
:earth_africa: :earth_americas: :earth_asia:

Nice! This fixes it.


Correct the config in the tsup.config.ts file

In the tsup.config.ts file, you need to correct the build. Namely, move the project assembly from the dist folder, where the files already exist, to the build folder, and also fix the issue that main.js is generated with a minified code.

@Ankush1oo8 Thanks!


ckeditor_uploader outside of django admin

Right now the ckeditor_uploader works fine outside of the django admin system.  However, the template currently loads "admin/css/base.css" from django admin which returns a 404 if django admin isn't installed.  The styling seems to be okay without it, so I'm wondering if it should either be removed or just the needed parts adapted into a new stylesheet included with ckeditor_uploader.

Here's what it looks like without the admin/css/base.css:

Here's what it looks like with it:

here's just the parts of admin/css/base.css being used (excluding the use of the Roboto font which is just used on the "Embed Image" link):
/* GLOBAL DEFAULTS */

form {
    margin: 0;
    padding: 0;
}

/* FORM DEFAULTS */

input {
    margin: 2px 0;
    padding: 2px 3px;
    vertical-align: middle;
    font-family: "Lucida Grande", Verdana, Arial, sans-serif;
    font-weight: normal;
    font-size: 13px;
}

input[type=text] {
    border: 1px solid #ccc;
    border-radius: 4px;
    padding: 5px 6px;
    margin-top: 0;
}

input[type=text]:focus {
    border-color: #999;
}

/* FORM BUTTONS */

input[type=submit] {
    background: #79aec8;
    padding: 10px 15px;
    border: none;
    border-radius: 4px;
    color: #fff;
    cursor: pointer;
}

input[type=submit]:active,
input[type=submit]:focus,
input[type=submit]:hover {
    background: #609ab6;
}

input[type=submit].default {
    float: right;
    border: none;
    font-weight: 400;
    background: #417690;
}

input[type=submit].default:active,
input[type=submit].default:focus,
input[type=submit].default:hover {
    background: #205067;
}

/* MESSAGES & ERRORS */

ul.errorlist {
    margin: 0 0 4px;
    padding: 0;
    color: #ba2121;
    background: #fff;
}

ul.errorlist li {
    font-size: 13px;
    display: block;
    margin-bottom: 4px;
}

ul.errorlist li:first-child {
    margin-top: 0;
}

/* OBJECT HISTORY */

table#change-history {
    width: 100%;
}

table#change-history tbody th {
    width: 16em;
}

/* PAGE STRUCTURE */

#container {
    position: relative;
}

Here's what it looks like (with an error message to show that it's styled properly):

I could provide a PR, but I'm not sure if it's better to just inline these styling into the one template it's used on, or make a separate .css file.  Any preferences?

it could be a separate css file as it's just for ckeditor part.

issue resolved due to #342


uint8 read and writing comes out as uint16

I'm developing an IoT device, so bytes are scarce.
When reading or writing the byte 0b11111000, which is encoded as w7g= in base64,
it comes out as 76CA.
That is equivalent to the same value in 16 bits. This happens for any 8 bit value I want to send.
Is there anything I can do to force uint8?
I would rather avoid having to do manual translation of values before writing and after reading, as I've got values of many different lengths. If they all have to be translated from another data type, that is a lot of logic that could go wrong

Binary 11111000 is encoded in Base64 as +A==, conversion in other direction leads to 1 byte: 0xF8. I am not sure how did you get these values.

Hi, this is embarrassing. I guess the binary to base64 converter I used was bugged or something. Thank you for your answer!


How to append history command instead of replace?

Category

[ ] fzf binary
[x] fzf-tmux script
[x] Key bindings
[ ] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

I would like to append a command from history instead of replacing the whole line.
For example:
$ first-command && [launch fzf, choose second command to be appended] second-command && [launch fzf, same..] third-command

How can I do this?
Maybe with --expect? or FZF_CTRL_T_COMMAND?

It's not supported by the key bindings this repository provides and I have no idea how to implement it. Feel free to share your solution when you find one, thanks.

This seems to work:
# In .zshrc

fzf-history-append-widget() {
  selected=( $(fc -il -30000 | eval "$(__fzfcmd) +e --tac +m -n2..,.. --tiebreak=index --preview='echo {}' --preview-window=up:30%:wrap --toggle-sort=ctrl-r $FZF_CTRL_R_OPTS") )
  num=$selected[1]
  command=$(fc -nl "${num}" | head -n 1)
  LBUFFER="${LBUFFER}${command} "
  local ret=$?
  zle redisplay
  typeset -f zle-line-init >/dev/null && zle zle-line-init
  return $ret
}

zle     -N   fzf-history-append-widget
bindkey '^O' fzf-history-append-widget

fzf-history-widget() {
local selected num lbuffer
lbuffer=${LBUFFER}
setopt localoptions noglobsubst noposixbuiltins pipefail no_aliases 2> /dev/null
selected=( $(fc -rl 1 | perl -ne 'print if !$seen{(/^\s*[0-9]+*\s+(.)/, $1)}++' |
FZF_DEFAULT_OPTS="--height ${FZF_TMUX_HEIGHT:-40%} $FZF_DEFAULT_OPTS -n2..,.. --tiebreak=index --bind=ctrl-r:toggle-sort,ctrl-z:ignore $FZF_CTRL_R_OPTS --query=${(qqq)} +m" $(__fzfcmd)) )
local ret=$?
if [ -n "$selected" ]; then
num=$selected[1]
if [ -n "$num" ]; then
zle vi-fetch-history -n $num
fi
fi
LBUFFER="${lbuffer}${LBUFFER}"
zle reset-prompt
return $ret
}
Did the trick for me !


Senna is missing as champion in the constants

Her ID is 235.
champion.json

Hello vinyz, feel free to add senna to constants and make a pull request!

https://github.com/Sansossio/twisted/commit/08f39d56a8d6e7235117ab54281bae3c62ec5284?diff=unified


Add ability to install and configure Agent but don't register it.

Hi team,
The ability to install Agents but don't register it is quite useful and requested by the community (see https://github.com/wazuh/wazuh-puppet/issues/106).
Adding that feature would give flexibility to the Agent installation and would ease the installation on cloud environments for example.
Tasks

[x] Modify code to allow user configure if he wants to register the agent or not

[x] Test changes with and without the flag enabled

[x] Update README.md

Best regards
Jose

Hi all,
Now we give the possibility to choose to register an agent after being configured and installed in a manager.
As @paulcalabro proposed this change in https://github.com/wazuh/wazuh-puppet/issues/106, he justified it by the following statement:

Sometimes it's necessary to install the Wazuh agent, but not register it with a manager. For example, when working with Amazon Machine Images (AMIs). You want to reduce the time it takes to install the agent by baking the installation into the image, however, you don't want to have a duplicate client.keys files by doing registration at the time of image creation.

In order to do so, it's only needed to assign the value yes to the variable default['ossec']['agent_auth']['register'] in the attributes file_ wazuh-chef/cookbooks/wazuh_agent/0attributes/authd.rb:
default['ossec']['agent_auth']['register'] = 'yes'

The default value is yes.In another case, we just assign a different value which is not yes.
Kind regards,
Rshad

Done here #50
Regards!


versioncomparisions are currently failing on Puppet 4 / strict

Failures:

  1) mcollective should contain Class[mcollective]
     Failure/Error: it { should contain_class('mcollective') }

     Puppet::PreformattedError:
       Evaluation Error: Error while evaluating a Function Call, 'versioncmp' parameter 'a' expects a String value, got Undef at /home/travis/build/puppet-community/puppet-mcollective/spec/fixtures/modules/mcollective/manifests/defaults.pp:8:6 on node testing-worker-linux-docker-54f46fad-3199-linux-2.prod.travis-ci.org
     # ./vendor/bundle/ruby/2.1.0/gems/puppet-4.3.1/lib/puppet/pops/functions/dispatcher.rb:37:in `dispatch'
     # ./vendor/bundle/ruby/2.1.0/gems/puppet-4.3.1/lib/puppet/pops/functions/function.rb:44:in `call'

and, zooming in:
Error while evaluating a Function Call,
  'versioncmp' parameter 'a' expects a String value, got Undef
  at mcollective/manifests/defaults.pp:8:6 on node testing-worker-linux-docker

Strict can be annoying. The simple fix is to change line 8 if versioncmp($::puppetversion, '4') < 0 { to if versioncmp("${::puppetversion}, '4') < 0 {  # lint:ignore:only_variable_string. puppetversion is no longer a core fact in facter 3, but it remains present in 2.4, so I'm not sure why it's empty in this case.

O_o
srsly??
why

The requirement is that we need to add facts for puppetversion and facter version:
  let :facts do
    {
      :puppetversion => Puppet.version,
      :facterversion => Facter.version
    }
  end

However, we then need to move all the code that actually is changed by those facts and put them into their own logic in the specs, defined on each version:
let :facts do
    {
      :puppetversion => Puppet.version,
      :facterversion => Facter.version
    }
  end

  let :mcollective_config_path do
    case Puppet.version
    when /4/
      '/etc/puppetlabs/mcollective'
    else
      '/etc/mcollective'
    end
  end

  let :mcollective_core_libdir_path do
    case Puppet.version
    when /^4\.$/
      '/opt/puppetlabs/mcollective/plugins'
    else
      '#{}'
    end
  end

  let :mcollective_site_libdir_path do
    case Puppet.version
    when /^4\.$/
      '/opt/puppetlabs/mcollective/plugins'
    else
      '/usr/local/libexec/mcollective'
    end
  end

etc.

@rnelson0 The puppetversion fact still exists, somehow it deaded when the original version of 4 came out, but seems resolved (https://tickets.puppetlabs.com/browse/PUP-4359).
The docs need updating for that it seems! :smile:

PR for change #267


support for creating polling guages with arbitary numbers

@jhaber
Adding methods for simply passing in Gauges that return a Number type instead of adding new methods for Floats. What do you think?

I'm like 75% sure there was a reason this method was private but I can't remember what it is

Seems ok to me


add pagination directive

Adding pagination directive.
You can change anything if you want and add some doc also don't worry ^^.

Me likey.
If no-one else does, i will properly add some documentation at some point in the future.

Ok nice.
I will add other directive when I need it.


Leverage Tropo File Storage

Extract from #6 : leverage Tropo Portal File Storage with a ftp endpoint for recordings updloads and an http endpoint to play recordings

Added configuration parameters to choose where recordings are uploaded and played from.

Test : ok on dev env
Closing


msvcrt and python dlls bundled twice

Is there a correct way to not to bundle msvcrt and python dlls twice (first one in bootstrap dir and second in version dir) into windows distribution?
I mean such a layout inside .zip:
./app-0.1.win32/Microsoft.VC90.CRT/<dlls+manifest>
./app-0.1.win32/python32.dll
./app-0.1.win32/<app files>
./Microsoft.VC90.CRT/<dlls+manifest>
./python32.dll
./app.exe
I suppose it's by design, but could you advise a workaround? E.g. copy dlls to selected version dir in bootstrap module or maybe some better alternatives?
(I use python 3.2 and cx_freeze)
Anyway, really thanks for esky, excellent work!

This double-bundling in an unfortunate artifact of the way windows looks for DLLs - they have to be in the same directory as the running executable to work correctly.  You could remove the ones inside the app-0.1.win32 directory and have it work most of the time, but it would make it hard to recover from certain corner-cases during a failed update.
There is a way around this, but it's a bit tricky to set up.  You might notice that the "bdist_esky" command takes an option called --compile-bootstrap-exes.  This uses the rpython compiler from the PyPy project to compile the bootstrapping environment as a native application, so that it doesn't need the python dlls.
If you're feeling game and want to give it a try, you will need to (1) set up a C compiler and development environment, (2) make sure you can build the latest pypy release from source, (3) copy the "lib-python", "pypy" and "py" directories from the pypy source distribution into your site-packages directory, (4) try building your esky with --compile-bootstrap-exes, and (5) bug me for help until you get it working :-)

Thanks for quick reply, Ryan. Yep, I've noticed the compile-bootstrap-exes option, but unfortunately I can't use pypy because my project written on py3k (pypy supports <=2.7.1). So I'll think about bundling the dlls on windows app installer level to deploy them once and don't bundle them in esky-generated updates.

... or may be will try to copy dlls from root dir to bestversion dir before chainload at bootstrap.


FFmpeg error for iPhone 4K 60fps videos

auto-editor versions attempted:
22w52a (22.52.1)' AND '23w04a (23.4.1)
Error received:
stderr: Input #0, rawvideo, from 'pipe:':
  Duration: N/A, bitrate: 5971968 kb/s
  Stream #0:0: Video: rawvideo (I420 / 0x30323449), yuv420p, 3840x2160, 5971968 kb/s, 60 tbr, 60 tbn
Stream mapping:
  Stream #0:0 -> #0:0 (rawvideo (native) -> hevc (hevc_videotoolbox))
[hevc_videotoolbox @ 0x7fc3c8707300] Error encoding frame: -12905
[hevc_videotoolbox @ 0x7fc3c8707300] popping: -542398533
Error initializing output stream 0:0 -- Error while opening encoder for output stream #0:0 - maybe incorrect parameters such as bit_rate, rate, width or height
Conversion failed!

Error! FFmpeg Error!

Command Used:
auto-editor {FileName}
This has only happened when trying to edit iPhone videos shot with 4k 60fps

Update: I installed ffmpeg and used --my-ffmpeg, and everything seems to be working. Might be a H.265 encoding issue based on built-in ffmpeg tags.

Since it's working for you when you're using your ffmpeg, I'll close this issue. At some point, I will update the built-in ffmpeg binaries and so it works by default, but this is a urgent task.


gk app: geo files don't have the sim name prefixed

The GK app is writing a bunch of geometry/field related files without the filename prefix
allxyz.gkyl
b_i.gkyl
bmag.gkyl
bmag_inv.gkyl
...
These should have the name of the sim as a prefix, like all our other output data.

This issue was solved by @akashukla in this commit https://github.com/ammarhakim/gkylzero/commit/3b35faf3a876805be02a4af1ad7de67592c08f7f
and now geometry is written to file with a matching name.


improving ConvertToTree extension method

closes #19

Codecov Report

Merging #23 (ac309ab) into main (63987e8) will increase coverage by 0.09%.
The diff coverage is 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main      #23      +/-   ##
==========================================
+ Coverage   91.76%   91.86%   +0.09%
==========================================
  Files          11       11
  Lines         328      332       +4
==========================================
+ Hits          301      305       +4
  Misses         27       27

Flag
Coverage Δ

PS_5.1
91.86% <100.00%> (+0.09%)
:arrow_up:

PS_7_Windows
?

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

Files Changed
Coverage Δ

src/PSTree/PSTreeExtensions.cs
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


How to debug?

I have set all the configurations as in the documentation (or at least this is what I think)
Still, the captcha is not working. There is nothing in the console. I don't know where to start debugging.
It would be nice if there is an option:
config.debug = true

And I can see some debugging in the console. Like if the gem is loaded, if the configuration is loaded. If the tags are rendered. What is the validation error.
I have activated:
config.on_failure = ->(verification) { Rails.logger.error("Captcha failure: #{verification.result}") }

And I see:

Captcha failure: #<Hashie::Mash error-codes=#<Hashie::Array ["missing-input-secret"]> messages=#<Hashie::Array []> success=false>

But it doesn't tell me much
Thanks for your work ;), I already starred the project

Just for the record. I fixed the issue. The credentials were not being loaded. Still, the suggestion remains, in this case, I would have expected something to show in the console.


How can I do the sum of multiple accessors?

{
Header:'Title',
accessor: 'SCADU_TOTA1P'+ 'RIMES_ASCA1P'+'PORFO_ASCA1P'
}
This method not works!

If you do
{   Header: 'Title',   Cell: props => <span>{props.original.SCADU_TOTA1P + props.original.RIMES_ASCA1P + props.original.PORFO_ASCA1P}</span> }
it should work out for you.

Github issues is no place for general support. We encourage you to use our public support forum at https://spectrum.chat/react-table.
NOTE: If a support/help/question issue is opened, it will be closed immediately and be redirected to the forum. Thanks for helping us keep our issues clean and productive!


Cannot find module './app.module.ngfactory'.ts(2307)

I have uploaded my project here: https://github.com/rezaee/conFusion, actually it was an NativeScript-Angular project written in 2017 with version 5 or 6 that I tried to update it to Angular 8, and I did all the job but one file named main.aot.ts remained. In that file that has this content:
// this import should be first in order to load some required settings (like globals and reflect-metadata)
import { platformNativeScript } from "nativescript-angular/platform-static";

import { AppModuleNgFactory } from "./app.module.ngfactory";

platformNativeScript().bootstrapModuleFactory(AppModuleNgFactory);

I get

Cannot find module './app.module.ngfactory'.ts(2307)

error message. So I tried npm installng-factory but it gives me the following error too:
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.

+<EMAIL_ADDRESS>updated 1 package and audited 11952 packages in 25.655s
found 169 vulnerabilities (13 low, 2 moderate, 154 high)
  run `npm audit fix` to fix them, or `npm audit` for details

As I am a new learner and just following a video tutorial I don't know how to solve the problem? Please help me to solve the last error message. Thank you in advanced!

Hello, we reviewed this issue and determined that it doesn't fall into the bug report or feature request category. This issue tracker is not suitable for support requests, please repost your issue on StackOverflow using tag angular-cli.
If you are wondering why we don't resolve support issues via the issue tracker, please check out this explanation.


Could you please make it an npm library?

[x] I'm sure this issue is not a duplicate.

More info:

Could you please make it an npm library that contains all the svgs, so that all the icons will be importable in js and can be used together with vscode-icons-js.
Thanks.

This extension is not meant to be an npm package. If you want to use it in your projects see the wiki on how you can use our docker image (https://github.com/vscode-icons/vscode-icons/wiki/Docker)


rescript-editor-analysis: 1.1.3 -> 1.3.0

The analysis executable was lagging behind the main rescript-vscode package,
due to which the vscode extension lost a few LSP capabilities, like semanticTokens.
See https://github.com/rescript-lang/rescript-vscode/issues/488 for more details.
This upgrade brings the analysis binary in sync with the main extension version.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Closing, because https://github.com/NixOS/nixpkgs/pull/181996 supersedes this PR.


Show less scary error when update check fails

When an update check fails (e.g. because the connection is offline), a scary red block of text is shown:

Show a much more user-friendly error message.

Fixed in the upcoming 1.6 release via 381c509b5abbb162ed8a16c873db72f402a5cf1f:


add a note according CONTENT IDEA REQUEST 100702

#sign-off

@jambirk,  CrystalThomasMS, kindly have a review and help merge the pull request.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

Meera will work on with the writer to update this article directly, close this PR


DRILL-4576: Add PlannerCallback interface for additional planner initialization.

Allow a storage plugin to be able to provide additional planner initialization.
Fix issue in StoragePluginRegistryImpl which deletes manually added plugins from the registry since they don't exist in the PersistentStore.
Add a simple test to ensure Callbacks are correctly registered and executed.

Reworked version of @jacques-n PR #466

Abandoned.


Revert "Optimize DirAccessUnix::get_next() for some file systems"

Reverts godotengine/godot#30657
Fix #30725

CC @hadrien-psydk

Thank you for letting me know. I will work on addressing this use-case when I have some time.

I'll revert for now to fix the regression, but feel free to propose a new PR that handles the optimization and the regression.


chore: hide search modal on back button press

This PR resolves
https://www.notion.so/artsy/Sliding-from-the-left-doesn-t-navigate-me-back-to-the-home-screen-after-opening-the-search-iOS-144cab0764a080e8956be0dfb69642cc?pvs=4
Description
This PR hides the search modal on back button press

Using Back Button
https://github.com/user-attachments/assets/615c5b14-fcd3-439b-9ade-9e5a2e5a67b5

Using the back gesture
https://github.com/user-attachments/assets/013f4b98-14f7-41da-b28d-40998834da35

PR Checklist

[X] I have tested my changes on iOS and Android.
[x] I hid my changes behind a [feature flag], or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an [app state migration], or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

#nochangelog

Warnings

:warning:

✅ No changelog changes

  Generated by :no_entry_sign: dangerJS against fc412d6bbd073a94dba3b465b79d969b45724893


Error when compiling on Mono - unknown.cake (28,94): Internal compiler error during parsingSystem.ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection.

Occurs only on Mono. Haven't tried different versions than v0.20.0 but I guess it might be in all versions.
It started to fail to compile after that commit:
https://github.com/PawelTroka/Computator.NET/commit/98d8e2313bfbbdbbfba109016773175f9a94229a
So I guess somewhere along those lines must be something that causes problems to compile build script on Mono.
See failing build:
https://travis-ci.org/PawelTroka/Computator.NET/jobs/246734398
Compiling build script...
unknown.cake (28,94): Internal compiler error during parsingSystem.ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index
  at System.ThrowHelper.ThrowArgumentOutOfRangeException (System.ExceptionArgument argument, System.ExceptionResource resource) [0x00029] in <9da04a6d1127416ab693e39fe1629be8>:0
  at System.ThrowHelper.ThrowArgumentOutOfRangeException () [0x00000] in <9da04a6d1127416ab693e39fe1629be8>:0
  at Mono.CSharp.Location.get_SourceFile () [0x0001c] in <e5610ebfe8c34526a8158db3c89186d4>:0
  at Mono.CSharp.Location.op_Subtraction (Mono.CSharp.Location loc, System.Int32 columns) [0x00000] in <e5610ebfe8c34526a8158db3c89186d4>:0
  at Mono.CSharp.Tokenizer.consume_string (System.Boolean quoted) [0x0000f] in <e5610ebfe8c34526a8158db3c89186d4>:0
  at Mono.CSharp.Tokenizer.xtoken () [0x00d72] in <e5610ebfe8c34526a8158db3c89186d4>:0
  at Mono.CSharp.Tokenizer.token () [0x00000] in <e5610ebfe8c34526a8158db3c89186d4>:0
  at Mono.CSharp.CSharpParser.yyparse (Mono.CSharp.yyParser.yyInput yyLex) [0x0010f] in <e5610ebfe8c34526a8158db3c89186d4>:0
  at Mono.CSharp.CSharpParser.parse () [0x0002e] in <e5610ebfe8c34526a8158db3c89186d4>:0
Error: Internal compiler error during parsingSystem.ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index
  at System.ThrowHelper.ThrowArgumentOutOfRangeException (System.ExceptionArgument argument, System.ExceptionResource resource) [0x00029] in <9da04a6d1127416ab693e39fe1629be8>:0
  at System.ThrowHelper.ThrowArgumentOutOfRangeException () [0x00000] in <9da04a6d1127416ab693e39fe1629be8>:0
  at Mono.CSharp.Location.get_SourceFile () [0x0001c] in <e5610ebfe8c34526a8158db3c89186d4>:0
  at Mono.CSharp.Location.op_Subtraction (Mono.CSharp.Location loc, System.Int32 columns) [0x00000] in <e5610ebfe8c34526a8158db3c89186d4>:0
  at Mono.CSharp.Tokenizer.consume_string (System.Boolean quoted) [0x0000f] in <e5610ebfe8c34526a8158db3c89186d4>:0
  at Mono.CSharp.Tokenizer.xtoken () [0x00d72] in <e5610ebfe8c34526a8158db3c89186d4>:0
  at Mono.CSharp.Tokenizer.token () [0x00000] in <e5610ebfe8c34526a8158db3c89186d4>:0
  at Mono.CSharp.CSharpParser.yyparse (Mono.CSharp.yyParser.yyInput yyLex) [0x0010f] in <e5610ebfe8c34526a8158db3c89186d4>:0
  at Mono.CSharp.CSharpParser.parse () [0x0002e] in <e5610ebfe8c34526a8158db3c89186d4>:0

Ok, this is actually partly my fault.
I have this foreach loop:
foreach(var monoVersion in monoVersions)
but I also have variable monoVersion defined above.
After changing monoVersion from foreach loop to for example monoVersion1 it does work correctly.
So this is not cake fault, more like Mono scripting should work better around this issue.

@PawelTroka at some point soon, the Mono Compilation will be completely removed in favour of Roslyn, which will make things better. For now, are we able to close this issue?

That's great to hear 👍
Yeah, I guess :)


Medication dispenses process render fix

https://github.com/edenlabllc/ehealth.api/issues/3342

Coverage remained the same at 88.43% when pulling dd8704dd64c9777a1441a347de6b11296fd94b56 on medication_dispenses_process_render_fix into e2378ece15b9b64e8a437cae21d1eafe082510ce on develop.


Creates SVG icon library for addition to revelation apps

What this pr does
This creates the base svg icon component library.  It wraps the necessary linear icons as inline svgs and exports as components usable by the consuming app.
How to test
Setup

[ ] clone and pull latest of repo git clone git@github.com:kscherling/ember-cli-revelation-icons.git
[ ] cd into directory
[ ] nvm use 4.4.4
[ ] ember s (may need to npm/bower install)

Tests pass

[ ] localhost:4220/tests

Dummy app Renders

[ ] Navigate to 4220

Test Addon

[ ] npm link directory
[ ] pull matching branch on participant app feature/ks/svg-icons
[ ] npm link ember-cli-revelation-icons
[ ] ember s
[ ] navigate to /cli/ep/1/todos where 1 is id of project you have a role in
[ ] you shoud see a paper plane icon

Related to https://invent.focusvision.com/Portland/revelation-frontend-participant/issues/155

If we make the svg files as partials rather than templates, then we can generate the name of the partial based on an input ('name' or 'icon').  This will allow us to have only one component.
example:
import lookupPartial from 'ember-views/system/lookup_partial'

...

iconPartialName: Ember.computed('name', function() {
  return `svg/-${this.get('name')`
}),

showIcon: Ember.computed('name', function() {
  return !!lookupPartial(this, `svg/${this.get('name')`)
}),

badIconAlert: Ember.observer('showIcon', function() {
  if(!this.get('showIcon') {
    Ember.Logger.error(`there is no icon named ${this.get('name'}`)
  }
})

then on the template for the component
{{#if showIcon}}
  {{partial iconPartialName}}
{{/if}}

pardon my pseudocode

That's awesome, I wasn't aware of lookupPartial functionality.  I wonder if we could actually just set the layout property directly?

awesome +1

+1


[feature request] Mapping HDFS into docker container per job

Hi Pai,
Thanks again for your great work. could you please add the feature that automatically mapping $some_dir to the hdfs so that any program could access hdfs without HDFS knowledge?
Qiwei

You can do that by customizing a docker image. here  is an example of using hdfs mount to do that.
However, our experience shows hdfs mount is not stable enough. It is recommended to use native hdfs reader/writer for the hdfs access.
An alternative is to deploy an NFS (or other file system like FS) server on your own and mount to the NFS from inside the docker instance.


⚠ SolarOscp has degraded performance

In 1613c21, SolarOscp ($SOLAROSCP_URL) experienced degraded performance:

HTTP code: 200
Response time: 412 ms

Resolved: SolarOscp performance has improved in 3329fdb after 9 minutes.


Too many logs by DefinePlugin

Bug report

What is the current behavior?

If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?
Less verbose output, less repetition, concise and impactful logs. This feels like a great candidate for using logger.debug instead.

Other relevant information:
webpack version: 5.89.0
Node.js version: 18.13.0
Operating System: Mac Sonoma on M1 pro
Additional tools: ¯\_(ツ)_/¯

Please read https://github.com/webpack/webpack/pull/18520

Please read https://github.com/webpack/webpack/pull/18520

Continuing the conversation in #18520 - there is no merit in repeating this log these many times with no additional information.


[BUG] flutter-tools.nvim/lua/flutter-tools/lsp/color/utils.lua:165: Invalid 'line': out of range

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When I tried to use shift-v to select multi lines of codes, and then delete them, the error throws:
Error executing vim.schedule lua callback: ...flutter-tools.nvim/lua/flutter-tools/lsp/color/utils.lua:165: Invalid 'line': out of range
stack traceback:
        [C]: in function 'nvim_buf_set_extmark'
        ...flutter-tools.nvim/lua/flutter-tools/lsp/color/utils.lua:165: in function 'color_virtual_text'
        ...flutter-tools.nvim/lua/flutter-tools/lsp/color/utils.lua:218: in function 'buf_color'
        ...flutter-tools.nvim/lua/flutter-tools/lsp/color/utils.lua:179: in function 'handler'
        ...w/Cellar/neovim/0.9.0/share/nvim/runtime/lua/vim/lsp.lua:1394: in function ''
        vim/_editor.lua: in function <vim/_editor.lua:0>

At the same time, make any change to a .dart file will break the code highlight too. But it can be restored by :e
Expected Behavior

Visual mode should be able to select and delete multi lines.
Edit code should not broke the code highlighting.

Steps To Reproduce

open a .dart file with Neovim
Enter visual mode by shift+v and select some lines of codes.
Press d to delete selected codes.
The error pop up.
The code highlight is not working.

Environment
- OS:macOS
- Flutter version:3.7.1
- Is flutter in $PATH:Yes
- neovim version:0.9.0

Anything else?
No response

@akring thanks for the report, I just tried this out and I can't reproduce this problem.
Can you give me a sample file and a sample minimal init.lua (please don't link your entire config as I can't use that to test, just a few lines long minimal example please)

@akinsho Thank you for your reply!
I tried it again and find out the error is gone 😂. So the only problem is code highlight now.
This is my init.lua
-- Basic
require("basic")
-- Packer
require("plugins")
-- Key
require("keybindings")
-- Theme
require("colorscheme")
require("transparent").setup({
	groups = { -- table: default groups
		"Normal",
		"NormalNC",
		"Comment",
		"Constant",
		"Special",
		"Identifier",
		"Statement",
		"PreProc",
		"Type",
		"Underlined",
		"Todo",
		"String",
		"Function",
		"Conditional",
		"Repeat",
		"Operator",
		"Structure",
		"LineNr",
		"NonText",
		"SignColumn",
		"CursorLineNr",
		"EndOfBuffer",
	},
	extra_groups = {
		"NvimTreeNormal",
		"TelescopeNormal",
		"BufferLineNormal",
		"DapUiNormal",
	},
})

-- Plugins
require("plugin-config.nvim-tree")
require("plugin-config.bufferline")
require("plugin-config.lualine")
require("plugin-config.dashboard")
require("plugin-config.project")
require("plugin-config.nvim-treesitter")
require("plugin-config.dap-ui")

-- LSP
require("lsp.setup")
require("flutter-tools").setup({
	decorations = {
		statusline = {
			app_version = false,
			device = false,
		},
	},
	widget_guides = {
		enabled = true,
	},
	debugger = {
		-- integrate with nvim dap + install dart code debugger
		enabled = true,
		run_via_dap = true, -- use dap instead of a plenary job to run flutter apps
		exception_breakpoints = {},
	},
	lsp = {
		color = {
			enabled = true,
			background = false,
			background_color = { r = 19, g = 17, b = 24 },
			foreground = false,
			virtual_text = true,
			virtual_text_str = "■",
		},
		settings = {
			showTodos = true,
			enableSnippets = true,
		},
	},
})
require("lsp.cmp")
-- Format
require("lsp.null-ls")

-- UI
require("lsp.ui")

-- Comment
require("plugin-config.comment")

I created a brand new initial flutter project via flutter create test, and I can reproduce the highlight problem with it. I record a short video to show what's going on.
https://user-images.githubusercontent.com/6525286/231173776-b8d668b4-8893-4be1-9137-d1a770b24047.mp4

@akring this plugin doesn't do any highlighting of your code, that comes from treesitter or if you're using old school vim highlighting. Essentially whatever is happening there isn't coming from this plugin. It might be semantic highlighting but that isn't controlled by this plugin. There are many threads about this on r/Neovim so have a look there about highlighting as well

I figured it out with some more search, the problem come with Neovim 0.9.0 version upgrade.
This is the workaround that fixed my problem: reddit
Thank you for your patience!


scans.py calls old version of selscan that rejects ten-column output for normalization

Output made from selscan with the --ihs-detail flag has ten columns. The old selscan normalization function throws an error at this, expecting six (normal, undetailed output). According to the selscan page (https://github.com/szpiech/selscan), norm v1.0.5 can handle this, but this version does not appear to be integrated into scans.py.

Resolved offline.


Set the horizontal angular velocity to zero

I think the angular velocity regarding to x and y axis is useless, in the case of 2d mapping.
Maybe  these values  are omitted in the latest cartographer if use_trajectory_builder_2d  is true.
However, we are trying to use the debianized cartographer in ros kinetic distro, which is based on version 0.2.0, and I found that the horizontal angular velocity effects significantly on the localization as shown in the following GIF.

After applying this commit, the localization is improved a lot as shown in the following GIF.

Please think about to merge this commit.

I signed it!

I signed it!
//////////////////////
東京大学大学院情報理工学系研究科
稲葉・岡田研究室 趙漠居
〒113-8656 東京都文京区本郷7-3-1
工学部8号館 3階326号室
Mail<EMAIL_ADDRESS>Tel:  03-5841-7 <javascript:void(0);>109
Fax:  03-5841-6285 <javascript:void(0);>
携帯： 090-4241-8671
//////////////////////
2018年7月3日(火) 16:01 googlebot<EMAIL_ADDRESS>
Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project (if not, look below for help).
Before we can look at your pull request, you'll need to sign a Contributor
License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I
signed it!) and we'll verify it.
What to do if you already signed the CLA Individual signers

It's possible we don't have your GitHub username or you're using a
different email address on your commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.

Corporate signers

Your company has a Point of Contact who decides which employees are
authorized to participate. Ask your POC to be added to the group of
authorized contributors. If you don't know who your Point of Contact is,
direct the Google project maintainer to go/cla#troubleshoot (Public
version https://opensource.google.com/docs/cla/#troubleshoot).
The email used to register you as an authorized contributor must be
the email used for the Git commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
The email used to register you as an authorized contributor must
also be attached to your GitHub account
https://github.com/settings/emails.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/googlecartographer/cartographer_turtlebot/pull/94#issuecomment-402033901,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADfwr9eimTijVyHsvoYigYIvtgdDd9Ztks5uCxa2gaJpZM4VAZUr
.