SqlServerDatabaseReplicationLinkResource.DeleteAsync throws exception on 202 status code

Calling SqlServerDatabaseReplicationLinkResource.DeleteAsync generates a response with a http status code of 202, which results in an exception being thrown. When I go to the Azure portal, the link has been removed, so it appears that this operation works, but just incorrectly throws an exception.
Azure.RequestFailedException : Service request failed.
Status: 202 (Accepted)

Content:
{"operation":"ForceStopDatabaseCopy","startTime":"2022-09-06T19:47:17.453Z"}

Headers:
Cache-Control: no-cache
Pragma: no-cache
Location: REDACTED
Retry-After: 10
Azure-AsyncOperation: REDACTED
x-ms-request-id: cf62923e-56bc-452c-bd2c-e4ba08fee468
Server: Microsoft-HTTPAPI/2.0
x-ms-ratelimit-remaining-subscription-deletes: REDACTED
x-ms-correlation-request-id: REDACTED
x-ms-routing-request-id: REDACTED
Strict-Transport-Security: REDACTED
X-Content-Type-Options: REDACTED
Date: Tue, 06 Sep 2022 19:47:18 GMT
Content-Length: 76
Content-Type: application/json; charset=utf-8
Expires: -1

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'ARM:0.20359786,Storage:0.07202666,SQL:0.06551232'

@jhabkohrs:  Would you please let us know the package and version that you're using so that we can triage to the team member best able to assist?

I am using the 1.0.0 version of Azure.ResourceManager.Sql.

@jhabkohrs we checked our code and seems can take 202 as a valid response. Would you please share your code and let us know what type of SQL and replicationlink you created so that we can try reproduce on our side?

It does appear that this commit (https://github.com/Azure/azure-sdk-for-net/commit/936baab7398867eba11c127294ed5dd2ecbf9467) should fix the issue. However, when I decompile the 1.0.0 version of Azure.ResourceManager.Sql, the 202 case is not there. Is this just a matter of the fix hasn't been pushed in a released NuGet package?

I think https://github.com/Azure/azure-sdk-for-net/commit/936baab7398867eba11c127294ed5dd2ecbf9467#diff-4d50f7c27ae2112b63a1a2708cbef1e62f8b03359a7d47b6bef64acea34c137d this is to fix the 202 response code. @ArcturusZhang is this commit released to nuget?

@jhabkohrs  Yes I believe the current released version does not consider 202 as a valid response that's why there is Service Request Failed exception.
CC: @ArthurMa1978 , kindly help check and release a new version to address this issue
Any other feedback to Azure SDK, feel free to let us know in this survey

Any update on a potential release with this fix?

@jhabkohrs we are still fixing some other issues regarding this package, ETA of latest version release date in Oct

Is there an updated ETA on the package release?

We shall release a fix package by the end of this week, cc @fengzhou-msft and @ArthurMa1978 for awarness

Azure.ResourceManager.Sql 1.1.0 is released. Please have a try.

close this issue since we have released new packages to fix it


Fix bug in plot, loglog: let within if was capturing rest of closure

plot, loglog: added parentheses around let in 'if marker'
In the closure definition in plot, because the let in scope of if marker ... wasn't wrapped in parentheses, the let extended to the rest of the closure, and nothing else got executed if the if test failed. It's also clear from the indentation that this isn't what was intended.  Plot settings weren't getting rrestored to their previous values at the end of the closure, which caused some of the settings to carry over to the next closure when it was executed.
This fixes issue #113.  I don't know why the PR includes some old commits from my previous PR.  Let me know if I need to redo this one.
It seems clear that the same problem exists in loglog, so I fixed that. (EDIT: At first I couldn't run loglog just to test that I hadn't broken something, but now I figured out how to do that.)
(I haven't noticed other functions in owl_plot.ml that have this problem.  I searched through all 'if' expressions that use 'marker'.  Difficult to search for an if followed by a let on the next line.)
(This is the loglog error:
# let xs = M.sequential 1 10;;
val xs : M.mat =
   C0 C1 C2 C3 C4 C5 C6 C7 C8 C9
R0  0  1  2  3  4  5  6  7  8  9

# let h = Owl.Plot.create "foo.pdf" in Owl.Plot.(loglog ~h xs ; output h);;
Exception: Gsl__Error.Gsl_exn (Gsl__Error.EDOM, "domain error").
Raised by primitive operation at unknown location
Called from file "src/owl/owl_maths.ml", line 89, characters 14-21
Called from file "array.ml", line 91, characters 21-40
Called from file "src/owl/owl_plot.ml", line 1236, characters 41-68
Called from file "//toplevel//", line 1, characters 47-59
Called from file "toplevel/toploop.ml", line 180, characters 17-56

I'll try to figure out whether I'm doing something wrong and file an issue if need be.)

@mars0i Thank you so much for finding out and fixing this problem! I wrote the loglog function based on plot and I think that's why they share the same problem.

This is definitely a bug, many thanks for fixing it @mars0i
For the second issue, I think the reason is the 0 in xs, which loglog will call Maths.log 0 which will throw that exception.

Ah, I see.  That makes sense.

you can instead try with let x = Mat.sequential ~a:1. 1 10;;

Thanks.  I must have noticed the optional arguments to sequential at some point, but completely forgot about them.


Potential updates

In relation to #54, I'm trying to update things where possible. Would love your feedback @bryphe and @ManasJayanth!

I'm trying to figure out why the --env test is failing

Ah, finally figured it out. It's that the mount command now prints the values to stdout when you run the command.

I reverted the esy lockfile and dependency changes because newer deps caused a problem that I couldn't solve, but I kept all the process improvements in place.
If these changes are merged and released, I believe it will pick up the newest cygwin stuff that will also solve #54

I've also tested a fresh git-pull and ran all the scripts and everything works in my Windows 10 VM


please show full demo

hi i saw docs and cant find any demo
please build online demo
thanks

Right click, view source.

You mean this? http://blog.reactandbethankful.com/angular-multi-step-form


[Query] Implementation Model

Hello,
I went through the Orbit tutorials and the Orleans paper. I understand JGroups & Infinispan are used to manage distributed state. I have couple of questions:

How are virtual actors implemented on JVM ? using green thread like concept ? bytecode manipulation ?
Is there a concept of supervisor actor ? A single actor can have multiple children, for example, multiple requests on the same service URL will spawn multiple actors having different IDs.
How do you manage Actors ? Is there a management UI to control cluster - JMX ?

Thanks

Hi,
Thanks for your questions.
Virtual actors in Orbit are basically just a class instance at the most basic level. All execution of messages on that actor is handled by the framework. There is effectively a threadpool and orbit spreads the message processing across that pool (guaranteeing the same actor won't run 2 messages at once) as required. This is what the defaultExecutionPoolSize in Stage is doing.
The way Orbit actually implements this is fairly complex and uses bytecode generation & manipulation as well as routing everything through the orbit runtime to make sure it's all consistent and to ensure the guarantees are respected.
In the virtual actor model there is really no concept of supervisor actors. One of the key concepts is that actors aways exist conceptually so there is really no concept of an actor being alive/dead etc. You can address an actor at any time and the framework will figure out how to make that happen.
Similar to above, there is no concept of actor lifetime in the traditional sense so you don't need to manage actors manually at all. The framework deals with retrieving actor state from backing storage, activating the actor and then paging it out (and back in) as required.
The default behavior is that on the first message to an actor, state will be retrieved (or if no state exists, it will be default state) and the actor will be activated on the cluster, it will then process any further messages to that actor until it has been idle for 10 minutes, at which point it will deactivate and exist only in the backing storage again.
Hopefully that answers your questions. One of the hardest things to adapt to coming from a traditional actor framework when you come to the virtual actor framework is the lack of explicit lifetime and ownership and the idea that an actor always exists from the point-of-view of someone interacting with it.
Please let me know if you have any other questions.
Thanks,
Joe

Going to close this one. Please let me know if you have any further questions.


React Native Upgrade Breaking Tests

Describe the bug
After upgrading to RN 0.60.3 the tests just get stuck. The tests worked fine on version 0.59.9. The metro-bundler builds but the app is just idle and the test times out.
To Reproduce

[x] I have tested this issue on the latest Detox release and it still reproduces

Environment (please complete the following information):
Detox: 12.8.1
React Native: 0.60.3
Node: 10.15.3
Device: iPhone 7
Xcode: 10.2.1
iOS: 12.2
macOS: 10.14.3
Device and Verbose Detox Logs
errors.txt

Thanks, I'll have a look. I don't think we've tested RN .60.3 yet.

The test suite seems to run locally. Could you please follow the guide here and post back the log you are getting:
https://github.com/wix/Detox/blob/master/docs/Troubleshooting.Synchronization.md#lower-level-idling-resources-rebug-ios-only

@LeoNatan I added launchArgs: { 'detoxPrintBusyIdleResources': 'YES' } but when I run the command to obtain the log I get
Error from getpwuid_r: 0 (Undefined error: 0) Filtering the log data using "category == "EarlGreyStatistics"" and it just hangs

That's odd.
In your log, I see
detox[14367] TRACE: [DetoxServer.js/MESSAGE] role=tester action=isReady (sessionId=569d5a25-0d01-5c35-bfb3-6f981d5c84e4)
detox[14367] DEBUG: [DetoxServer.js/CANNOT_FORWARD] role=testee not connected, cannot fw action (sessionId=569d5a25-0d01-5c35-bfb3-6f981d5c84e4)
detox[14367] DEBUG: [DetoxServer.js/LOGIN] role=testee, sessionId=569d5a25-0d01-5c35-bfb3-6f981d5c84e4
detox[14367] DEBUG: [DetoxServer.js/LOGIN_SUCCESS] role=testee, sessionId=569d5a25-0d01-5c35-bfb3-6f981d5c84e4
detox[14367] DEBUG: [DetoxServer.js/DISCONNECT] role=testee, sessionId=569d5a25-0d01-5c35-bfb3-6f981d5c84e4
detox[14367] TRACE: [AsyncWebSocket.js/WEBSOCKET_MESSAGE] {"type":"testeeDisconnected","messageId":-49642}

It says that /testee/ (your app) connects to the detox server but then disconnects. I wonder, what is happening on the simulator while the test is hung. Is that last message printed before the timeout wait or after.

Before the timeout wait right when the app launches.

If the app closes, that means it is crashing. Please follow this guide and post here what you see in Xcode once the crash happens:
https://github.com/wix/Detox/blob/master/docs/Guide.DebuggingInXcode.md
Thanks

Getting Error: Could not find build script in detox.configurations["ios.none"].build  when running the build command with -c ios.none. My config is
"ios.none": {
"binaryPath": "ios",
"type": "ios.none",
"name": "iPhone 8 Plus",
"session": {
"server": "ws://localhost:8099",
"sessionId": "guru.fan.app"
}

You are not supposed using Detox but using Xcode (from the UI).

Getting this
xcode.txt
Is it not connecting or something? The app doesn't open it just hangs on
detox[25147] DEBUG: [AsyncWebSocket.js/WEBSOCKET_OPEN] opened web socket to: ws://localhost:8099
detox[25147] TRACE: [AsyncWebSocket.js/WEBSOCKET_SEND] {"type":"login","params":{"sessionId":"guru.fan.app","role":"tester"},"messageId":0}
detox[25147] TRACE: [AsyncWebSocket.js/WEBSOCKET_MESSAGE] {"type":"loginSuccess","params":{"sessionId":"guru.fan.app","role":"tester"},"messageId":0}
detox[25147] TRACE: [AsyncWebSocket.js/WEBSOCKET_SEND] {"type":"isReady","params":{},"messageId":-1000}

That log doesn't include Detox messages. Please follow the guide.

I've followed it. I set this up:

Then ran the server, built the app as I would normally do to an iPhone 7 with the run button, then ran the tests.

Just found this error
dyld: Library not loaded: @rpath/COSTouchVisualizer.framework/COSTouchVisualizer
Referenced from: /private/var/containers/Bundle/Application/D76AEF94-EE01-4F5C-AFF5-51D061A0891A/Fanguru.app/Frameworks/Detox.framework/Detox
Reason: no suitable image found.  Did find:
/private/var/containers/Bundle/Application/D76AEF94-EE01-4F5C-AFF5-51D061A0891A/Fanguru.app/Frameworks/Detox.framework/Frameworks/COSTouchVisualizer.framework/COSTouchVisualizer: code signature in (/private/var/containers/Bundle/Application/D76AEF94-EE01-4F5C-AFF5-51D061A0891A/Fanguru.app/Frameworks/Detox.framework/Frameworks/COSTouchVisualizer.framework/COSTouchVisualizer) not valid for use in process using Library Validation: mapped file has no cdhash, completely unsigned? Code has to be at least ad-hoc signed.

That's quite odd. Not sure what is going on.

The app that builds doesn't work it just closes and the buttons don't work. If I remove the embedded binary the app works fine.
Is this the right config for it?

Yes, that is the way to configure.
Let's try something different. Please clone Detox from source (this repo) recursively, then instead of integrating the Detox.xcodeproj from your node modules, integrate the project from <clone dir>/detox/ios and try to run your project.

Do you mean add the embedded binary Detox.framework from the cloned directory's Detox.xcodeproj?

Yes. Remove the Detox project you previously added, add the one from the cloned dir, and finally add it to embedded binaries. Now try to build your project.
Just make sure you recursively clone or update submodules recursively after cloning.

Tried that but the built app still doesn't function properly. The build logs are different. With the Detox binary:

Without:

Both builds succeed but with the binary the app looses all functionality.

I am sorry, I don't understand. What does "looses all functionality" mean? Is the app stuck?

This has turned into Xcode support, which I don't like here. We normally direct people to Stack Overflow.
Please provide a demo project which reproduces this issue or consult with Stack Overflow how to debug this issue further.

The app just shows the start screen without the buttons work or api calls being made. This issue goes away as soon as I remove the Detox embedded binary.
I can try to figure out how to get the logs by myself but looks like something in the docs is outdated (https://github.com/wix/Detox/blob/master/docs/Guide.DebuggingInXcode.md).

The Detox embedded binary seems to be the issue.

On an old working project with Detox, https://github.com/ReshmiCode/Detox-Test following the same steps just results in the build crashing sooner with errors like
:-1: /Users/QAintern/Desktop/Detox/Detox.xcconfig: unable to open file (in project "Detox") (in target 'Detox') :-1: Unable to resolve build file: XCBCore.BuildFile (namedReferencesCannotBeResolved) (in target 'Detox')

It's something on your machine. Detox, with that embedded framework, work correctly for everyone else. Not sure why signature is breaking for you.

 Are there any logs on Android that could be useful? Fixed the Android build and the same problem occurs with the tests.

If you have the same problem on Android as iOS, it's a 99.999999% problem in your project. I am less familiar with Android Detox. @d4vidi @rotemmiz Any logs on Android to help with sync issues?

In the process of making a video to showcase the error, the Android tests started working. But I noticed that when the tests are normally run (detox build then detox test), the iOS version looses functionality the same way as when adding the embedded binary. https://www.loom.com/share/520cb8e01fd24adabc5f2bcf7fd033aa
If I open the build by myself the app works but if it opens through the detox test command the app's buttons don't work (as shown in the video at the end).

When you npm install for the first time, Detox is built from source. It makes sense that it will fail the same way as it does when embedded in the project. But this is unrelated to RN version as you claimed originally.

Yeah sorry about that, I assumed it was due to the RN version and until Android worked I assumed it was. The video is taken from when Detox was launched normally (through detox build).

What is shown in the video is the app being launched. That means that it has successfully loaded Detox. So the problem there is different.
OK, in your ~/Library/Detox there should be one or more folders. If one, proceed to step 2.
Step 1 (for convenience)
Delete all folders in ~/Library/Detox
Delete your node_modules and npm install—this will generate a new folder under ~/Library/Detox
Step 2
Don't follow the guide—if you have, remove Detox from your project
Instead, drag the Detox.framework from ~/Library/Detox/{...} into your Xcode project
Add the newly added framework into Embedded Binaries (like it shows in the guide)
Step 3
Add an Objective C exception breakpoint before running (Google how to do this)
Run your project from Xcode
Start the test
Once your project crashes (which we suspect it does), head into the bottom section in Xcode, where there should be a console output and you can type things
Type bt and press return
Copy the entire log, post it to https://gist.github.com/ and link here
Thanks

https://www.loom.com/share/d6283756c8e3468695a0e87ee42e3f38
The beginning of the video shows that without the embedded the app works but after adding it, the buttons don't work. The breakpoint wasn't hit so I believe I couldn't type bt in the log section.

Oddly, you don't have the pause button. Perhaps debugging is disabled in the scheme? In the scheme editor for your app, make sure "Debug executable" is enabled.

Here's the log
https://gist.github.com/ReshmiCode/1d14943ecbf9a35db4c56ec104919018
I'm getting this after I build the app, before I can run the tests.

Great, this has been solved in 13.2.0. Please upgrade.


Tokenizing linter doesn't work

The following file is invalid, yet it passes the "fast" linter: https://3v4l.org/ij0AH.
<?php

class Foo
{
    public $bar;
    public $bar;
}

See how no exception is thrown: https://3v4l.org/UIF8C.

Not much use for syntax checking files. :/

I would say it works fine for CS usage.
What's the issue with this file to be CS fixed ?

The issue is one of our fixers could break a file and we'd never know because the new linter passes, but the old one fails. I don't know how many other cases this is true for as well. This is just one I noticed.

So let say the initial file is:
<?php class Foo{public $bar; public $bar;}

and we would fix it to:
<?php

class Foo
{
    public $bar;
    public $bar;
}

Both file passes tokenizer-based linter, but cannot be executed.
File was not damaged by our fixer.
In general, I do agree with you that it would be great if tokenizer-based linter would fail in the first place, but sadly, it didn't. Great to have it in mind and observe, but there is not much we could do except forcing process-based linter, for which I'm against.

Why we introduced linting before fixing?
Because a lot of our fixers assume proper token sequence, and then for file with invalid syntax they tries to do invalid operation, and crash for that. To not let that happen we lint the file before fixing it.

Why we introduced linting after fixing?
Because sometimes our fixers may introduce not-wanted change. That bad change may lead to file with invalid syntax. Linting after fixing is aiming us to prevent saving that not-wanted changes.
But that changes, which break syntax are an edge case, there may be other changes that will make file somehow not-nicely changed. Some of that changes we do on purpose, but warn users to be extra careful using that fixers (risky fixer, eg strict). Above that, there could also be unwanted changes that will change code meaning without our approval (eg fixer that will clean whole tokens collection due to some bug) - linter would pass that "new" file, but change wasn't nice.

Linter is not ultimate decision maker if the change is good or no. If it detects violations in syntax after fixing - for sure we did sth wrong and we shouldn't save it. But if it doesn't detect a violation there may still be some not-willing change introduced into code behaviour. For that we have +100k tests.
I'm ready to live with that edge case in mind, at least until somebody will show a real case when we break a syntax in file that was not broken before but tokenizer-based linter pass it.

Ok, here's a major issue for you then:
<?php

return [

    'connections' => [

        'pusher' => [
            'driver' => 'pusher',
        ]
        //,

        'redis' => [],
    ],
];

Passes the tokenization, but is clearly invalid syntax.

As you can see here, PHP-CS-FIXER broke the file:

D:\GIT\fork\PHP-CS-Fixer (master)
λ php --version
PHP 7.0.10 (cli) (built: Aug 18 2016 10:03:00) ( NTS )
Copyright (c) 1997-2016 The PHP Group
Zend Engine v3.0.0, Copyright (c) 1998-2016 Zend Technologies

D:\GIT\fork\PHP-CS-Fixer (master)
λ cat test-1.php
<?php

$code = "<?php

return [

    'connections' => [

        'pusher' => [
            'driver' => 'pusher',
        ]
        //,

        'redis' => [],
    ],
];";

try {
    var_dump(token_get_all($code, TOKEN_PARSE));
} catch (ParseError $e) {
    var_dump($e);
}
D:\GIT\fork\PHP-CS-Fixer (master)
λ php -f test-1.php
object(ParseError)#1 (7) {
  ["message":protected]=>
  string(78) "syntax error, unexpected ''redis'' (T_CONSTANT_ENCAPSED_STRING), expecting ']'"
  ["string":"Error":private]=>
  string(0) ""
  ["code":protected]=>
  int(0)
  ["file":protected]=>
  string(0) ""
  ["line":protected]=>
  int(12)
  ["trace":"Error":private]=>
  array(1) {
    [0]=>
    array(4) {
      ["file"]=>
      string(35) "D:\GIT\fork\PHP-CS-Fixer\test-1.php"
      ["line"]=>
      int(19)
      ["function"]=>
      string(13) "token_get_all"
      ["args"]=>
      array(2) {
        [0]=>
        string(165) "<?php

return [

    'connections' => [

        'pusher' => [
            'driver' => 'pusher',
        ]
        //,

        'redis' => [],
    ],
];"
        [1]=>
        int(1)
      }
    }
  }
  ["previous":"Error":private]=>
  NULL
}

D:\GIT\fork\PHP-CS-Fixer (master)
λ cat test-2.php
<?php

return [

    'connections' => [

        'pusher' => [
            'driver' => 'pusher',
        ]
        //,

        'redis' => [],
    ],
];

D:\GIT\fork\PHP-CS-Fixer (master)
λ php php-cs-fixer fix test-2.php --dry-run --diff --rules=strict_comparison
Loaded config from "D:\GIT\fork\PHP-CS-Fixer\.php_cs.dist".

Checked all files in 0.079 seconds, 4.000 MB memory used

Files that were not fixed due to errors reported during linting before fixing:
   1) D:\GIT\fork\PHP-CS-Fixer\test-2.php

Sadly, closing as cannot duplicate and lack of reporter involvement :(


unclear how to use additional classpath

Hi would it be possible to explain how I can add additional classpath entries form jars. It's unclear how this exactly works.

@joshtynjala That's what I don't understand how to set this setting. Could you explain that part?

It's an array of file system paths. You set it with the same values that you might pass to the -cp/-classpath compiler option.

I am feeling pretty stupid I still don't understand your answer.
tried this but it didn't work
java -cp libs/ -jar groovy-lang-orignal-all.jar

if you are using visual studio code
Open workspace settings
and then append this to the json
{
"groovy.classpath": [
"io/2.11.0/a2503f302b11ebde7ebc3df41daebe0e4eea3689/commons-io-2.11.0.jar",
"/Users//.gradle/caches/modules-2/files-2.1/org.apache.commons/commons-csv/1.9.0/b59d8f64cd0b83ee1c04ff1748de2504457018c1/commons-csv-1.9.0.jar",
],
}

@himanshutripathi01 not using vscode but codemirror languag server

I think it starts making sence now. Could it be I have to add this settings file to the workspace of the language server?

yes


Update README and PULL REQUEST TEMPLATE

📘 Issue Description
The Installation section of the README is incorrect, and it seems that a section is missing in the PR template to request screenshots or videos to aid in the review.
🔍 Steps to Reproduce

Review the Installation section of the README.
Notice that the instructions are incorrect.
Check recent PRs and observe the absence of images or videos in the descriptions.

✅ Acceptance Criteria

Correct the Installation section in the README.
Add a section in the PR template requesting screenshots or videos to aid in the review.

🌎 References

📜 Additional Notes

Hi @derianrddev can I quickly work on this?


use gci to sort imports

Description
This PR only adds gci to ibc-go, so that we can sort imports with ease.
Commit Message / Changelog Entry
chore: add gci to ibc-go

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

Note:  I could refactor this in the same manner that I did the sdk, if desired:
https://github.com/cosmos/cosmos-sdk/pull/16605

Note: I could refactor this in the same manner that I did the sdk, if desired:
cosmos/cosmos-sdk#16605

I am personally not strongly opinionated: to me this grouping works fine. If others think some tweaking is needed, I will be also fine with it.

I'm good with either, as well :)


If we can specify a folder as the init directory of the file management?

If we can specify a folder as the init directory of the file management shows instead of the folder homepage root folder located. Because sometimes we use this class file as a third party library in the application system.

Please change $root_path and $root_url or FM_ROOT_URL and FM_SELF_URL if using as embed library.
Example:
$root_path = $_SERVER['DOCUMENT_ROOT'] . '/foo/bar';
$root_url = 'foo/bar';


feat: add support for HTTPRouteTimeoutBackendRequest

What this PR does / why we need it:

Which issue this PR fixes:

fixes: https://github.com/Kong/kubernetes-ingress-controller/issues/4914
Special notes for your reviewer:

PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[x] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

https://github.com/Kong/kubernetes-ingress-controller/issues/4914#issuecomment-1815026623
This is my consideration before implementing this feature.

After discussing with @czeslavo and @mflendrich, we have reached a consensus.
This feature is experimental and in an early stage. Both the upstream specification and KIC may make modifications in the future. More thought is needed to change the default behavior of KIC for it.
Currently, we can keep this implementation but add a restriction that users can only set them to the same value, otherwise, it will be rejected, so there will be no overlapping situations.
e.g.
apiVersion: gateway.networking.k8s.io/v1
kind: HTTPRoute
metadata:
  name: timeout-example
spec:
  ...
  rules:
  - backendRefs:
    - name: some-service
      port: 8080
    timeouts:
      backendRequest: 5s
  - backendRefs:
    - name: some-service
      port: 8080

this one will be rejected.
apiVersion: gateway.networking.k8s.io/v1
kind: HTTPRoute
metadata:
  name: timeout-example
spec:
  ...
  rules:
  - backendRefs:
    - name: some-service
      port: 8080
    timeouts:
      backendRequest: 5s
  - backendRefs:
    - name: some-service
      port: 8080
    timeouts:
      backendRequest: 5s

this one will be allowed
I will increase this restriction through admission and create an issue for tracking. Once we have other modifications, the admission can be removed.


v-tea-CI-102594

Archive & redirect
CI 102594

Docs Build status updates of commit 7cb4279:
:warning: Validation status: warnings

File
Status
Preview URL
Details

mixed-reality-docs/mrlearning-asa-ch4.md
:warning:Warning
View
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

mixed-reality-docs/reset-or-recover-your-hololens.md
:white_check_mark:Succeeded
View

mixed-reality-docs/mrlearning-asa-ch4.md

Line 172: [Warning] Invalid file link: 'images/Module2chapter4step24im.png'.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 54d4c60:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

mixed-reality-docs/reset-or-recover-your-hololens.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Note: External dependency (publish together with windows-docs-pr PR #1388

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

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

mixed-reality-docs/reset-or-recover-your-hololens.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

#sign-off

Corresponding PR in windows-docs-pr has gone live.


Allow user to configure which flavor of Markdown is input

Putting this in 0.2 since it's a lot of new config routing code

Hi, I'm quite interested in this project.
I saw your were looking to integrate MyST as an input option.
May I ask wether it was the myst-parser version or the mystmd version that you had in mind?

After looking into MyST, I've since decided not to support it for now. The remark ecosystem is very opinionated in a way that is not conducive to this side project of mine. The remark AST is also less rich than the Djot AST in some important ways, like where you can specify anchors.

I see, I understand.
Thank you for responding. ^^

No problem at all! I really do think MyST is a great language, it's just not quite easy enough to plug in with the time allocation I can give to this project.

That's perfectly reasonable.
The language is also a bit early in its life, and from what i hear, still has a bit of road ahead to fully stabilize the spec on JS vs. Python.
So, trying to implement it right now might do more headaches than good. ^^


Updated format number to use numbro instead of numeral.

This PR has a corresponding 'numbro' PR in toolbox, where a 'Formats' tab has been added to test number formatting.
This PR just replaces numeral.js with numbro.js, and rewrites buildFormatPattern to buildFormatConfig, and replaces fmtNumber option 'formatPattern' with 'formatConfig'.
This is a breaking change if any apps were using 'formatPattern', but I searched client apps and did not find any (though I do not have access to all).
Note:  this is a minimal drop in replacement with the minimal code changes needed to make it work.
numbro has other features we could take advantage of in later refactors, such as 'average' for smart number compacting, 'percentage', and maybe 'currency'.

We could  take this change, just to get the new library in.

We could go further and simplify our implementation by using new numbro features to implement our 'ledger' and 'withPlusSign' features

We could go even further by removing  'ledger'/'precision'/ 'zeroPad'/'withPlusSign' arguments in favor of raw numbro

We could go in the other direction by removing the new "formatPattern/formatConfig" property from the public API -- where it doesn't seem to be greatly used -- and just relying on it for our internal implementation.

Thoughts?

Does anybody understand the difference between 'trimMantissa'? and 'optionalMantissa'? They seem to do the same thing -- feel like I need to understand this better to be confident this implementation replicates what we have/want.

Does anybody understand the difference between 'trimMantissa'? and 'optionalMantissa'? They seem to do the same thing -- feel like I need to understand this better to be confident this implementation replicates what we have/want.

explanation for optionalMantissa:  https://github.com/BenjaminVanRyseghem/numbro/blob/eac3859d2feec557eafed8bc2b25b2d3ca096c28/src/formatting.js#L476
explanation for trimMantissa:
https://github.com/BenjaminVanRyseghem/numbro/blob/eac3859d2feec557eafed8bc2b25b2d3ca096c28/src/formatting.js#L478

We need to put this merge on hold until this numbro bug is fixed:
https://github.com/BenjaminVanRyseghem/numbro/issues/372

I made this change.  If you test and feel comfortable with this, am happy
to merge it.
On Tue, Oct 16, 2018 at 5:25 PM Colin Rudd<EMAIL_ADDRESS>wrote:

We could work around this bug (BenjaminVanRyseghem/numbro#372)
http://BenjaminVanRyseghem/numbro#372
by changing line 308 in FormatNumber.js
from:
pattern.trimMantissa = true;
to:
pattern.trimMantissa = mantissa != 0;
Doesn't seem too unreasonable.
mantissa = 0 and trimMantissa = true don't really make sense together
anyways.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/exhi/hoist-react/pull/702#issuecomment-430406653, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACM16SH_jxNGmAf-pIN3_GjaoRZ3PtXtks5ulk7bgaJpZM4XaNKv
.

Excellent - glad this made it in!


div -> span in PercentageSymbol/iconNode

This helps prevent a react's validateDOMNesting warning

Yes. It makes sense. It will allow us to have Rate elements inside inline elements like <p>. :+1:
You don't need to commit the generated files under the lib directory. They are automatically generated on release. If you remove those files I will merge this pull request.
Thanks!

OK, done!

Thanks for your contribution.
I will release a new version with this change and probably issue #17 later today.


Assessment of the difficulty in porting CPU architecture for armnn

Hello everyone! I am working on implementing a tool to assess the complexity of CPU architecture porting. It primarily focuses on RISC-V architecture porting. In fact, it can be estimated on average across various architectures.My focus is on the overall workload and difficulty of transplantation in the past and future.As part of my dataset, I have collected the armnn project. I would like to gather community opinions to support my assessment. I appreciate your help and response! Based on scanning tools, the porting complexity is determined to be simple, with a small amount of code related to the CPU architecture in the project. (referring to the overall workload from adapting the project to a specific architecture to achieving full functionality on that architecture). Is this assessment accurate? I look forward to your help and response.

Closed due to inactivity.


Alt-enter doesn't invoke Peek Definition

Right-click -> Peek Definition works for types within the same project, but hitting alt+Enter does nothing.

When is Alt+Enter ever bound to Peek Definition?

Whoops, mistyped - what I meant was alt+f12.

We have this functionality implemented in VFPT https://github.com/fsprojects/VisualFSharpPowerTools/blob/master/src/FSharp.Editing.VisualStudio.v2015/PeekDefinition.fs
I can try to port it, but not right now since I failed to build VFT behind a proxy.

I cannot reproduce it on master. Alt+F12 works for symbols defined in current project.

It also works cross projects:

Interesting...I'll take a look again this afternoon.

Hmm.  Maybe it's my keyboard/keybindings, since alt+f12 also fails on C# projects for me.


document pull-request merging guidelines/process

See https://github.com/reactiveui/ReactiveUI/pull/1196#issuecomment-259000212 as well

This is done!
See https://docs.reactiveui.net/en/contributing/framework/merging-pull-requests.html and https://docs.reactiveui.net/en/contributing/framework/commit-message-convention.html


Add scrolling of the dropdown to the selected country

What has been done

Added scrolling of the dropdown to the selected country
Added relevant tests

:tada: This PR is included in version 1.6.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Question about Got_OSINT?

Hey, I was wondering how the port number 379 was derived for LDAP? From my research I haven't been able to find anything related to port 379 for LDAP, the closest I've come to is port 389 that it's listed as using.

Correct, looks like a mistake.


Url "routing" to group methods

(if I could, I'd mark this low priority).  One of the things I like about this library is that it doesn't overcomplicate things like forcing you to define routing paths for everything.
However, I have a circumstance where I'm trying to simulate another system which does have multiple paths at the end of which support different methods... so:
/foo/bar/baz supports all "baz"-object related methods where as /foo/bar/quz supports all "quz"-object related methods.  Ignoring, for a moment, that this makes it look more like REST than JSON-RPC, is there any way to do this, or would there be any interest in having this feature (besides me at least)?

Interesting idea. I think the biggest issue to think though would be what happens to the json-rpc method name vs the route. Which one would take priority in determining which method to invoke. Also you can already use route type of info to have the method be handled by another instance of the handler.
See CanCreateAndRemoveSession for an example.
https://github.com/Astn/JSON-RPC.NET/blob/c0a6d05741c6197a3ffaccbe2ee414b049d7d7d1/AustinHarris.JsonRpcTestN/Test.cs

Based on what I have seen in the other solution (Java based), it treats the URL like a namespace and class, and then the method name lives on the class.
So, /foo/bar/baz  method quz
would equate to a namespace of "foo.bar" a class of "baz" and a method "quz".
Given that, the route would take priority, then the method would take over.
Thoughts?
I looked at the sample, but I didn't follow it.  I'll have to spend more time reviewing it.
On Monday, February 29, 2016 9:57 PM, Austin Harris<EMAIL_ADDRESS>wrote:

Interesting idea. I think the biggest issue to think though would be what happens to the json-rpc method name vs the route. Which one would take priority in determining which method to invoke. Also you can already use route type of info to have the method be handled by another instance of the handler.
See CanCreateAndRemoveSession for an example.
https://github.com/Astn/JSON-RPC.NET/blob/c0a6d05741c6197a3ffaccbe2ee414b049d7d7d1/AustinHarris.JsonRpcTestN/Test.cs—
Reply to this email directly or view it on GitHub.

After looking at this again I think you can do this via:
Given a "/foo/bar/baz" -> Any portion of that can be mapped as the session id. Then you could use some other portion as the Context.
// Setup routes like:
Handler.RegisterInstance("/foo/bar", new Foo(new Bar))
// Process are request like:
Handler.GetSessionHandler("/foo/bar").Handle( ..Rpc.., "/baz" /*Per Invocation Context*/ , ...)


Piwik PRO and Matomo should be separated

Currently, we mix data from both Piwik PRO and Matamo in one entry, even though they are different entities:
https://www.whotracks.me/trackers/piwik.html
There should be two separate entries that are not linked: one for Matamo and one for Piwik PRO.

With the March release of the February data, there are now two separate pages:

searches for piwik.org link to Matomo: https://whotracks.me/trackers/matomo.html
searches for piwik.pro link to Piwik PRO: https://whotracks.me/trackers/piwik_pro_analytics_suite.html

The old page is still generated, even though it no longer received updates (it has data from October 2020 to January 2022, but February 2022 is no longer included):
https://www.whotracks.me/trackers/piwik.html
So, currently the old page still is present and the information is incorrect (Piwik Pro mapped to piwik.org owned by Matomo). Ideally, it should be removed (if possible) or the text should be changed.

(As a reminder) Another observation is that Piwik PRO is currently shown with a long list of subdomains, while for other trackers, the list is mostly limited to a domain level, or to a selective group of subdomains (e.g. analytics.tiktok.com). At the moment, it is not consistent how the recently added trackers are handled compared to the older ones.
To be consistent with the existing trackers, the *.piwik.pro entries could be shown as one piwik.pro entry to improve consistency with the existing data. The same observation also applies to the recently added entry for MS Clarity (https://github.com/whotracksme/whotracks.me/issues/265).

The April release is out. Verified that the outdated page was not re-introduced.


Customizable status bars width

Hello, I would like to suggest the option of making the width of the bars configurable, with the main idea being to make them thinner.
How is it now:

As I suggest having an option:

When using some UI configurations the bars are on top of the columns and looks very nice, but with this Resizable Mode configuration the double bars to the left of the inventory take up a lot of space and perhaps giving a width option would make the settup cleaner.
I got this idea while using the skills progress bar plugin, as this plugin have exactly this option (*but for height).
Thanks and sorry for bothering

Yes please! And/or allowing stacking the bars on top of the inventory to avoid collision entirely with stock ui elements and runelite ones moved to this poor unusable dock u_u ui element

I'm working on this now

#14138 implements this feature with being able to resize the width of the bars from 3 to 40 pixels, default is 20


I have issuse to cantent root path

Problem encountered on https://dotnet.microsoft.com/en-us/learn/aspnet/blazor-tutorial/run
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Can you give more details about your issue?


requirements.txt does not specify versions

The requirements.txt should specify compatible versions.  For example the current code is for motor 1 and not compatible with motor 2.
With the current setup motor 2 is installed but not working.

Archiving the project


Predictions from keras h5 file is different than its tflite version

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug, a feature request, or a significant problem with documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04.1 LTS
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: No
TensorFlow installed from (source or binary): Binary
TensorFlow version (use command below): 1.10.1
Python version: 3.6.5
Bazel version (if compiling from source): NA
GCC/Compiler version (if compiling from source): NA
CUDA/cuDNN version: NA
GPU model and memory: NA
Exact command to reproduce: See source code

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the problem
I am following this video to convert my ML model to TensorFLow Lite:
https://www.youtube.com/watch?v=MZx1fhbL2q4
I noticed that if I am making predictions by reading the saved h5 file, the result is correct. This is not the case when I make predictions by reading the saved tflite file. I expect the predictions from both files to be the same.
Source code / logs
import numpy as np
from tensorflow import keras
from tensorflow.contrib import lite

model = keras.Sequential([keras.layers.Dense(units=1, input_shape=[1])])
model.compile(optimizer='sgd', loss='mean_squared_error')

xs = np.array([-1.0, 0.0, 1.0, 2.0, 3.0, 4.0], dtype=np.float32)
ys = np.array([-3.0, -1.0, 0.0, 3.0, 5.0, 7.0], dtype=np.float32)

model.fit(xs, ys, epochs=500)

# Write out to keras save file
keras_file = 'linear.h5'
keras.models.save_model(model, keras_file)

# Convert the keras file to TensorFlow Lite
converter = lite.TocoConverter.from_keras_model_file(keras_file)
tflite_model = converter.convert()
open('linear.tflite', 'wb').write(tflite_model)

# input to predict
model_input = np.array([10.0], dtype=np.float32)

# make predictions from the h5 file
h5_model = keras.models.load_model(keras_file)
h5_prediction = h5_model.predict(model_input)

# make predictions fromm the tflite file
interpreter = lite.Interpreter(model_path='linear.tflite')
interpreter.allocate_tensors()
input_details = interpreter.get_input_details()
output_details = interpreter.get_output_details()
interpreter.set_tensor(input_details[0]['index'], [model_input])
interpreter.invoke()
tflite_prediction = interpreter.get_tensor(output_details[0]['index'])

# h5_prediction will always be around 19, which is correct
print('Prediction from h5 model: {}'.format(h5_prediction))

# tflite_prediction is some random value
print('Prediction from tflite model: {}'.format(tflite_prediction))

This should be resolved in master (try the nightly). @gargn can comment further.

There was a bug in TocoConvert.from_keras_model_file in TensorFlow 1.10. It will be fixed in TensorFlow 1.11 (when that is released). For now, as @aselle mentioned, it was fixed in August 9th's nightly build.


DNS Adblocker

We should either use AdGuard Home or Pi-Hole, preferably in docker

Pi-Hole implemented in f7fc6fa and documented in 048caa0


server: correctly route PATCH requests when using RouteHTTPToGRPC

What this PR does:
Tempo relies on the RouteHTTPToGRPC option in server to handle HTTP and gRPC requests from a single endpoint. This options configures a multiplexer (soheilhy/cmux) to match and route HTTP and gRPC requests to their respective listener.
For HTTP matching it uses cmux.HTTP1Fast which matches the method of the request with a predefined set of methods. This change adds PATCH to the list to ensure PATCH requests are also sent to the HTTP listener.
More details here: https://github.com/grafana/tempo/issues/2756
For context, this configuration option was added a couple of months ago to support Tempo: https://github.com/weaveworks/common/pull/288
Which issue(s) this PR fixes:
Fixes https://github.com/grafana/tempo/issues/2756
Checklist

[ ] Tests updated
[ ] CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

FYI draft PR to show this change works on Tempo: all tests succeed except vendor-check.

https://github.com/grafana/tempo/pull/2770

Now that this in dskit we may want to restructure this code, but I think this is fine for now. At the time I was trying to write in the style of the library, but I think we should adjust this to just allow for some functions to be passed in as hooks where the caller can just inject the behavior necessary.


Fix missing / misleading statements, and formatting in CREATE and ALTER CERTIFICATE

CREATE CERTIFICATE

Fixed description of "WITH PRIVATE KEY" to account for "BINARY =" option introduced in SQL Server 2012.

Fixed specification of BINARY for certificate (public key):

added "BINARY ="
moved above private key options to be properly grouped with public key options
added "Applies to"

Updated remarks to indicate that certificates can also be loaded from a binary constant (as of SQL Server 2012).

Added note about "CLR strict security" server config option at the bottom of "Example C" (as of SQL Server 2017).

Added links to CERT_ID and CERTPROPERTY functions in "See Also" section.

ALTER CERTIFICATE

Fixed main description as it was both incomplete and misleading.

Fixed syntax

Fixed formatting in "Arguments" section to be consistent with formatting found in CREATE CERTIFICATE page (and I have been told to specifically not escape underscores unless they are causing formatting issues).

Changed "password" in DECRYPTION BY PASSWORD and ENCRYPTION BY PASSWORD clauses to be "current_password" and "new_password", respectively, to hopefully be more intuitive to those who are less experienced with certificates.

Removed "from a file" wording in "Remarks" section since the private key can be imported from multiple sources

Changed "Example A" to be "Removing a private key". Previously it was "Changing the password of a certificate", which is identical to "Example B", except the title is slightly different and the order of the ENCRYPTION BY PASSWORD and DECRYPTION BY PASSWORD clauses is different. No need for redundant examples.

Reversed the order of the ENCRYPTION BY PASSWORD and DECRYPTION BY PASSWORD clauses in "Example B" because it is probably more intuitive to have decryption first since that is actually what happens first.

Fixed typo in "Example C": removed extraneous backslash in file path.

Added links to  CERTENCODED, CERTPRIVATEKEY, CERT_ID, and CERTPROPERTY functions in "See Also" section.

Both
For details /  testing, please see:  https://sqlquantumleap.com/2019/04/22/can-a-certificates-private-key-be-imported-restored-from-a-binary-literal-hex-bytes/

@srutzky : Thanks for your contribution! The author, @VanMSFT, has been notified to review your proposed change.

@VanMSFT : Thanks for your contribution! The author, @VanMSFT, has been notified to review your proposed change.

@VanMSFT : Thanks for your contribution! The author, @VanMSFT, has been notified to review your proposed change.

Hi @srutzky! Thanks for the detailed update. We really appreciate your testing and knowledge on this. I went through details, and everything looks applicable. Besides a slight change to syntax, I'll go ahead and approve. Thanks again!
#sign-off

Cool. Good catch on the syntax change. And you are quite welcome 😺
There are one or two minor additions I just thought of for ALTER CERTIFICATE, partially related to the PR I just submitted for BACKUP CERTIFICATE : #1976  . Is it too late to add them to this PR?

#hold-off
@srutzky - Go ahead and add to this PR if needed.

Thanks...won't take long..

@srutzky : Thanks for your contribution! The author, @VanMSFT, has been notified to review your proposed change.

Additional changes consist of:

Moved the "REMOVE PRIVATE KEY" clause to the top to be in the same order as shown in the syntax (that is how most other pages are done)
Added clarification regarding what private keys are used for in "Remarks", in the paragraph about removing the private key.
Added note to explicitly state how to accomplish changing the password rather than importing, else it was only mentioned in the description and shown in an example.
Added CERTPRIVATEKEY link to "Important" note regarding backing up the private key before removing it (this became an option in SQL Server 2012).

oops...sorry, one more quick thing...

@srutzky : Thanks for your contribution! The author, @VanMSFT, has been notified to review your proposed change.

Final edit:

Added info about default file location. Test case is in script linked in PR for BACKUP CERTIFICATE: #1976

@VanMSFT all done now...thanks..

@VanMSFT - if you approve, can you #sign-off ? After the PR is signed off, we'll check it against the PR review criteria before it gets merged. thanks.

Reviewed. Thanks @srutzky!
#sign-off

Yer welcome.

@GitHubber17 : Thanks for your contribution! The author, @VanMSFT, has been notified to review your proposed change.

@GitHubber17 : Thanks for your contribution! The author, @VanMSFT, has been notified to review your proposed change.


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>chore(scripts): cover@#overwrite c8 --temp-directory build/coverage ava && c8 report -r lcov --temp-directory build/coverage
chore(package): add nyc from template
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/-test.js,tests/-test.mjs' as in template
chore(package): set $.ava.extensions='js,mjs' as in template

Coverage remained the same at 91.489% when pulling 2acc78552c8aa4afa0df5fb059fb077677f3f442 on template-sync-1 into db4e153e1329c2f2ffb48cffc5700733367fcff6 on master.


Publish 0.9.0 spec tests to npm

<EMAIL_ADDRESS>


Change hover to build without Docker

Trying to build Hover without Docker.

Runing with nix-shell --pure:
[nix-shell:~/dev/nu/mini-meta-repo]$ hover run
hover: Missing config: file hover.yaml not found: open go/hover.yaml: no such file or directory
hover: Failed to lookup `flutter` executable: exec: "flutter": executable file not found in $PATH. Please install flutter or add `--docker` to run the Hover command in Docker container.
https://flutter.dev/docs/get-started/install

So we probably need to pass flutter as the buildInput also. But I think I will do this in another PR.


SystemJS tries to load ng2-slider.component.html.js

After setting the ng2-slider component up in SystemJS:
var map = { 	... 	'ng2-slider-component': 'node_modules/ng2-slider-component', 	'ng2-slideable-directive': 'node_modules/ng2-slideable-directive', 	'ng2-styled-directive': 'node_modules/ng2-styled-directive', 	... };
var packages = { 	... 	'ng2-slider-component': { main: 'ng2-slider.component.js'}, 	'ng2-slideable-directive': { main: 'slideable.directive.js'}, 	'ng2-styled-directive': { main: 'ng2-styled.directive.js'}, 	... };
and trying to import it in one of my components:
import * as slider from 'ng2-slider-component';
I get the following error on the console log:
http://localhost:4000/node_modules/ng2-slider-component/ng2-slider.component.html.js
It seems I can't set any sort of exceptional rule for .html files in SystemJS to load them without .js extension.
How did you get it working? Am I missing something?

Im publish new version with SystemJS compatible files to npm
*.js files compiled for WebPack
If you use SystemJS, you have to use *.system.js files, they are compiled for SystemJS.
Fragment of SystemJS config:
packages: {

    ....

    'node_modules/ng2-slider-component': {
            main: 'ng2-slider.component.system.js',
            defaultExtension: 'system.js'
    },

    ....

}


Allow export of classes inside object

In Akka classes are, most of the time, encapsulated in an outer object when they aren't exposed to the user:
object Outer {
  class Inner {}
}

It's a fairly standard pattern, but unfortunately the current implementation of @JSExport prevents from exporting nested classes.
If we could improve the annotation system to allow for such a behaviour, it would help tremendously!
Thanks :)

:+1:

This is IMO a very valid use case to have a separate way to make things reachable without exporting them to JS. The main issue with nested classes (not your example, truly nested classes, see #354) is that making them available from JavaScript is hard. Not eliminating them is trivial.

my use case is a bit more complex.. take this for instance
I need to @JSExport it exclusively so I can dinamically instantiate it using js.Dynamic

That use case is actually much more related to #1744 than to the ability to export a nested class. In fact, solving the "export a nested class" feature would not help your use case, as you could probably not derive the right way to instantiate the class just from its full class name, as it appears in is classOf.

Thanks guys, probably this allow us to remove a couple of workarounds in Akka.Js :-)


Layer improvements

Fixes I noticed when checking/answering questions posed by @bradgrantham-lunarg

ok @bradgrantham-lunarg this is ready to review, fixed ci issue

ok, no idea why the CLA is not running, but I've definitely done it


Add Google Authentication Feature

I would like to propose adding Google authentication to the project. This would allow users to sign in using their Google accounts, enhancing user experience and security.

Thank you for highlighting this issue. As it has been requested by multiple contributors, we are treating it as a priority. The contributor who resolves this issue first will be given priority for their efforts. We appreciate everyone's involvement and look forward to seeing the solution.

Label is added

level 1,2, 3 like that

I have Already did that
On Thu, Oct 17, 2024 at 9:02 PM simmi verma @.***>
wrote:

level 1,2, 3 like that
—
Reply to this email directly, view it on GitHub
https://github.com/ChandelAnish/fusionFLOW/issues/45#issuecomment-2419873168,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGLQQXL7SWOBGOQNBBF44F3Z37KCDAVCNFSM6AAAAABP2AG32SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJZHA3TGMJWHA
.
You are receiving this because you commented.Message ID:
@.***>


Python 3 Support

Hello @deepanprabhu, I'm interested in implementing python3 support for this project. Before beginning, I wanted to double-check whether this work has already been started by someone else. Please let me now.
Thanks!
Jimmy

see #7. you can fork from that branch and try to match with latest master branch

Thanks @rachmadaniHaryono ,
@jvleta - Thanks for sharing your interest to implement python3 support. Your contribution is valuable and would be merged with appropriate credits.


Fix for when out_of_salt == 'Today' or 'Tomorrow'

Added a simple if-statement to not strip the JSON of a date when the output of the JSON is simply "Today" (in case of 0 days until out of salt) but to just set the attribute to the string "Today".

I noticed there was also a case of the date being 'Tomorrow', so I included this as well

Fixed with Pull Request #10.
Thanks 👍


FancyZones' Custom Layout gets deleted out of nowhere

Microsoft PowerToys version
0.53.3
Running as admin

[X] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
No Specific Steps
✔ Expected Behavior
I was expecting to see my saved custom layouts under custom layouts section inside Layout Editor
❌ Actual Behavior
All my custom layouts were missing from Layout Editor
Other Software
No response

Just realized this was already reported ealier


Add possibility to cancel deletion and set the response

Add deletion behat test
This should fix this issue

Any idea why the test suite doesn't start on your PR? 🤔

No idea 🤷


Test SSDF workflow locally

Do a test run of SSDF workflow locally on laptop (or in a Linux environment).
Acceptance Criteria

[ ] Able to run SSDF with cwl-runner on local host to produce same output as in the Jupyter notebook.

Moving this to backlog since it may not be possible to do a local run because of the dependency on stage-in/out containers.  We need to think about any local tests that could be done before a full run on U-SPS.


Policy rename should not redirect to list

Currently when you rename a policy, it takes you back to the main list instead of at the policy you just renamed

okay changed to remain policy edit page.


Bug with definition of last on Vectors

The following example:
data Vec (n : Nat) (a : Type) where
  Nil : Vec 0 a;
  Cons : a -> Vec n a -> Vec (n + 1) a

last : forall {a : Type, k : Nat} . Vec (k + 1) (a [0..1]) -> a
-- when k = 0
last (Cons [x] Nil) = x;
-- when k = k' + 1
last (Cons [_] xs)  = last xs

Doesn't type check, it results in the following error:
The following theorem associated with `last` is falsifiable:
	∀ n0.1 : ↑Nat,a1.1 : Type,t2.1 : Type,t3.1 : ↑Nat . ((0..0 ≤ 0..1) ∧ (n0.1 + 1 = k + 1) -> (0..1 ≤ 0..1))

Ouch.

For this we need to be able to propagate information between equations which we used to do, but realised we were doing it in not a fine-grained enough way. This would make a very useful project...


UI enhancement for SignIn, SignUp,auth/signin/index.html and auth/signup/index.html pages

Add hover effects to input fields, shadows to borders, and title. Also, implemented password visibility features.
SignIn:

SignUp:

auth/signin/index.html :

auth/signup/index.html :

Closes #23

@ChandelAnish Could you check if this works for you?

I have changed color as you told. Could you check, if it's fine?

@ChandelAnish Could you please check this PR, as I have changed color as you suggested?


Fix brew doctor with HOMEBREW_INSTALL_FROM_API

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Fixes https://github.com/Homebrew/brew/issues/12356
I could have sworn I fixed this in #12333 but I must have made some change and forgotten to test it out fully. Either way, this should fix the check_deleted_formulae check in brew doctor so that if HOMEBREW_INSTALL_FROM_API is set, kegs with their tap set to homebrew/core and kegs with a blank tap but which are available via the API won't be listed as being deleted.
This also raised another interesting issue which is that installing with HOMEBREW_INSTALL_FROM_API (when homebrew/core isn't tapped) leaves the tap entry in INSTALL_RECEIPT.json blank. This should probably be changed to be homebrew/core. However, since people have been using it, it's probably too late to not have the blank tap condition here.
I'm way too tired to think this through completely right now, though, so I'll just leave this as-is and maybe re-visit in the morning.

Review period skipped due to critical label.


Release planning - 0.11.0 and beyond

Here is a summary of the plan for upcoming ACA-Py releases. Details to be added, but here is the short version:

ASAP - 0.11.0-rc0 from the main branch
0.11.0-rc1 and 0.11.0 final from the main branch plus the PR tagged with the 0.11.0 label
Add into the main branch code to enable ledger agnostic AnonCreds

A manual merging of the code in the anoncreds-rs branch of ACA-Py.

As needed, 0.11.x patch releases of items that must be published.
A breaking change release -- likely called 1.0.0 of ACA-Py from main with support for ledger agnostic AnonCreds

It is almost certain that in this release support for the Indy SDK will be dropped entirely

This will be presented/discussed at the ACA-Pug meeting this coming Tuesday, Oct. 31 🎃 at 8:00 Pacific / 16:00 Central Europe.

Shared with the community and updated.


Update sbt-updates to 0.5.2

Updates com.timushev.sbt:sbt-updates from 0.5.1 to 0.5.2.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.timushev.sbt", artifactId = "sbt-updates" } ]

labels: sbt-plugin-update, semver-patch

Superseded by #26.


Feedback for: compose/compose-file.md

File: compose/compose-file.md
Under update_config, the last three items (update_config, parallelism, and delay) should be bullets but aren't:

Thank you for the feedback! In fact, they should be in a code block, and this has been fixed: https://docs.docker.com/compose/compose-file/#updateconfig .


Fix for test to work locally with any project_id

At the moment if someone were to want to run tests locally we are requiring to have a project with project_id="dask-bigquery" but as mentioned on #7 by @bnaul project_id's are unique so this wouldn't be possible.
This PR fixes that problem and adds instructions on the readme about it. I'm not sure if this is the best way to solve this, but it's a starting point.
cc: @jrbourbeau
Closes #7

Closing in favor of a simpler and more consistent solution offered in #10


[MATLAB] Add arrow.type.StructType MATLAB class

Describe the enhancement requested
Before we add the arrow.array.StructArray class (#37653), we first need to add arrow.type.StructType as a class.
Component(s)
MATLAB

take


Use getState to access auth from thunk action creators.

We have prettier support now. Try it out, let's see if the config is correct. Our goal is to change as little lines as possible from our current formatting, but might also just try to align with what the React project does.


Issue with underscored engine names

Thanks for the gem, has worked great following the modular rails book.
I've now come up against a problem generating engines with more than one word names.
Such as:
bundle exec rails g modular:engine engines/blah_images --namespace=Blah

It will output folder structure and module name like:
app/controllers/blah/blahimages/application_controller.rb

module Blah
  module Blahimages

instead of what I would expect:
app/controllers/blah/blah_images/application_controller.rb

module Blah
  module BlahImages

Am I doing something wrong here?
Thanks in advance

I've tracked it down to this pull request. Seems to work fine for my use case but let me know if any issues

@ChrisLusted I will give it a try tomorrow and if everything works fine, I will get it merged. ;)


V1.1.0 - TypeError: Cannot read properties of undefined (reading 'inherit')

After updating to version 1.1.0 i'm not able to run my app anymore. I use AngularJS 1.8.3.
TypeError: Cannot read properties of undefined (reading 'inherit')
    at StateParams.$inherit (factory.js:63:9)
    at StateService.href (factory.js:63:9)
    at BaseUrlRule.handler (factory.js:63:9)
    at UrlService.sync (factory.js:63:9)
    at factory.js:63:9
    at factory.js:63:9
    at Array.forEach (<anonymous>)
    at factory.js:63:9
    at Scope.$broadcast (factory.js:63:9)
    at afterLocationChange (factory.js:63:9)
    at factory.js:63:9
    at Scope.$digest (factory.js:63:9)
    at Scope.$apply (factory.js:63:9)
    at bootstrapApply (factory.js:63:9)
    at Object.invoke (factory.js:63:9)
    at doBootstrap (factory.js:63:9)
(

I did not find any changelog and/or migration instructions. Any help is really appreciated.

Hello, thanx for the answer. I did it, i added "@uirouter/core": "^6.1.0" dep into package.json but still have the same error. Do i need to inject it into the angolarjs scope?

Are you using any other packages from uirouter?

Yes:
import { RejectType } from '@uirouter/angularjs';

Which version did you update from?

1.0.30

I can't see anything that'd make it break. Can you try uirouter/core 6.0.8?

Just tested using core 6.0.8 and that fixed the problem, works like a charm!

6.0.9 breaks as for 6.1.0

I think I found out what the problem is. Do you have params declarations as strings instead of objects?

Ok, i'm going to clenaup my code! Thank you!!!!

Well, I think your case is still supported so I'll add a fix for your case!

@p3pp8 I'm having some issues reproducing the issue in the unit tests. Would you be able to send me a basic state declaration structure that makes this fail? So that I can test my fix.

Hello @wawyed, the example in the post above is extracted from my project (with some renaming) and is a basic state declaration with default params set to null. I think they should've been declared like this:
$stateProvider
        .state('myStateName', {
            url: '/myPage',
            params: {
                a: { value: null },
                b: { value: null }
            }
        });

Thank you for the support!

If you change it as above the error disappears?

We have unit tests that test such functionality but they pass

I'm going to try!

Nothing to do, i've changed all params to have { value: null } instead of null but the error is still there...

Okay... so it's not that.. would you be able to put a breakpoint where it errors and see which param is trying to access?

The project is very complex and loads many angular components with states, i'll try to do my best but it's not as simple as put a breakpoint

The loop in stateParams.js kept adding the word 'chunk' in parentParamsKeys.
<EMAIL_ADDRESS>I don't know what causes that, but I changed the loop from
for (let j in parentParamsKeys) {     if (parentParams[parentParamsKeys[j]].inherit == false || inheritList.indexOf(parentParamsKeys[j]) >= 0)         continue;     inheritList.push(parentParamsKeys[j]);     inherited[parentParamsKeys[j]] = this[parentParamsKeys[j]]; }
to
parentParamsKeys.forEach(j => {     if (parentParams[j].inherit == false || inheritList.indexOf(j) >= 0)         return;     inheritList.push(parentParamsKeys[j]);     inherited[parentParamsKeys[j]] = this[parentParamsKeys[j]]; });
now it works.

I think i have to wait for a new release, but that's not a problem. Thank you guys!

@christ182 can you inspect parentParamsKeys, check Object.keys(parentParamsKeys) wondering if for some reason there's some enumerable properties added to that array?

@wawyed Object.keys is working as expected. It's the loop that does not behave properly.

I found the reason for mine.
                        Array.prototype.chunk = function(n) {
                            if (!this.length) {
                                return [];
                            }
                            return [this.slice(0, n)].concat(
                                this.slice(n).chunk(n)
                            );
                        };```

```for(var i in parents)``` includes functions as keys.

Although, I have this Array.prototype.chunk code in for some time now. I don't understand how it suddenly broke.

The reason I said Object.keys() it's because it retrieves the enumerable properties of the array. I wasn't referring to the existing Object.keys.
Regardless I think I have a fix based on your latest comment.

I do suggest though that you use Object.defineProperty() instead and make chunk non enumerable to avoid further issues.

@wawyed Can V1.1.0 be released as soon as possible?

@ckLocal  is this not what you need<EMAIL_ADDRESS>

@wawyed  Yeah, this is what I want, that's great.

Hi, I'm running into this same error and I don't see a similarity to @p3pp8's configuration.
Note the value of j is 'clear' here. Seems like that's referencing the clear prototype method of the parentParamsKeys array... but why?

Here's where my states are defined. Is there an incompatibility here? I'm upgrading from angular-ui-router 0.4.2 to @uirouter/angularjs 1.1.0 (yes, this is a legacy app that I must support)
(function () {
    'use strict';

    var theModule = angular.module('ai.fieldData', [
        // third-party modules
        'ng',
        'ngAnimate',
        'ngAria',
        'ngResource',
        'ngMessages',
        'ui.router',
        'ui.bootstrap',
        'ui.bootstrap.modal',
        'kendo.directives',

        // modules from common app
        'ai.core',
        'ai.directives',
        'ai.directives.upload',
        'ai.filters',
        'ai.services',
        'ngGrid',
        'ai.common.templateCache',
        'ai.directives.expander',

        // modules from field-data app
        'ai.fieldData.templateCache',
        'ai.fieldData.application.constants',
        'ai.fieldData.files',
        'ai.fieldData.monitoring',
        'ai.fieldData.activities',
        'ai.fieldData.activityTree',
        'ai.fieldData.locations',
        'ai.fieldData.navigation',
        'ai.fieldData.visits',
        'ai.fieldData.shared',
        'ai.fieldData.applicationConfiguration',
        'ai.fieldData.exceptionConfiguration',
        'ai.fieldData.corrections',
        'ai.fieldData.levelSurvey',
        'ai.fieldData.verticalReferences'
    ]);

    theModule.config(function ($stateProvider, $urlRouterProvider) {
        $stateProvider
            .state('app', {
                url: '',
                templateUrl: 'source/application/application.html',
                controller: 'FieldDataController',
                resolve: {
                    applicationData: function(ApplicationInitializationService) {
                        return ApplicationInitializationService.getApplicationData();
                    }
                }
            })
            .state('app.formContainer', {
                abstract: true,
                url: '/locations/:locationId/visits',
                templateUrl: 'source/application/form-container.html',
                controller: 'FormController',
                resolve: {
                    channels: function(ApplicationInitializationService) {
                        return ApplicationInitializationService.getChannels();
                    }
                }
            })
            .state('app.formContainer.form', {
                abstract: true,
                views: {
                    'navigationPanel@app': {
                        templateUrl: 'source/navigation/navigation-panel.html'
                    },
                    'visitHeader@app': {
                        templateUrl: 'source/shared/partials/visit-header-form.html'
                    },
<EMAIL_ADDRESS>{
                        templateUrl: 'source/directives/notification/ai-notification-template.html'
                    }
                }
            })
            .state('app.nonformContainer', {
                abstract: true,
                url: '/locations/:locationId/visits/:visitId',
                templateUrl: 'source/application/nonform-container.html'
            })
            .state('app.nonformContainer.nonform', {
                abstract: true,
                views: {
                    'navigationPanel@app': {
                        templateUrl: 'source/navigation/navigation-panel.html'
                    },
                    'visitHeader@app': {
                        templateUrl: 'source/shared/partials/visit-header-nonform.html'
                    },
<EMAIL_ADDRESS>{
                        templateUrl: 'source/directives/notification/ai-notification-template.html'
                    }
                }
            });

        $urlRouterProvider.otherwise('');
    });
}());

Indeed that 'clear' is an array prototype method... how could I have possibly misconfigured my states to make this happen?

@EoinGriffin-AI  You shouldn't make custom methods into prototype enumerable. Can you show me the code that adds those methods?

@EoinGriffin-AI  You shouldn't make custom methods into prototype enumerable. Can you show me the code that adds those methods?

Ah you're right, thanks for pointing out that they're non-standard functions. It's some old custom code added many years ago to our codebase. I can change their definitions to not be enumerable.


Async Executor

Opening this thread to start the discussion for a custom Async Executor for Tauri.
Why Async?
For Tauri, async means being able to continue core operations while user operations are still pending. This is a large boon to responsiveness in programs that have heavier background tasks.
Why not Tokio or AsyncStd?
Two main reasons: bloat and tailoring. Both runtimes are fairly liberal in how much they import, which is both heavy and a potential security issue. Additionally, we can fine-tune our executor to do only what we need, how we need it.
What do we need to do to get started?
First a few questions:
Q: What common tasks (ie Networking, any thread::spawn(||loop {...})s) will Tauri want to use threads to drive?
Q: What needs to be adjusted, internally and externally, to allow user code to propagate into our executor? Where should the executor sit in our design?
Once these questions are answered, we can start designing the executor. It can be broken up into the following design segments:

Relationship between the executor and the rest of the Tauri framework.
Design of the reactors (one general purpose and a variable number of specialized ones) and how they get work.
How the executor schedules work for the reactors.

Right now we use async operations here:
https://github.com/tauri-apps/tauri/blob/dev/tauri/src/lib.rs
https://github.com/tauri-apps/tauri/blob/ae51d9573e2c1b19e8e6216cb9d870cabdce8af0/tauri/src/app/runner.rs#L17
The main one we should update is the lib.rs because that's where every Rust API call goes.

Id also recommend that you look again at event.rs.  Got a thread_local! call in there. It may be worth implementing task_local! if we are going to go with Tokio.

As I mentioned, a custom executor would be a more sound decision for us, despite the work involved. An executor, generally speaking, is a way to centralize background work, so it makes sense that we'd want to keep that tailored to what we need. Tokio, while convenient for a lot of use cases, is a bit on the bloated side.
The event dispatcher is a prime candidate to be turned into a custom reactor. It would essentially get the Futures from the user code on each iteration and push them to the Futures reactor (which we would probably have 2-4 of, and could be a user configuration) to be handled as futures.
The update command could be done as a Future, but eventually that will need to be more complex for fancy update processes. Regardless of the complexity, we will be able to put that on a general purpose reactor.
This will definitely be a bigger-picture change, and I would like to encourage we think of this as a long-term project. Axiom's refactor took me over 150 hours of work (guesstimating), and it still needs adjustments.

I did consider using Actix at one point for the entire platform but it would require a major rewrite.  Just implementing futures and green threads in the event manager is a good idea and definitely work looking into.  It would make the library much more preformant and help with distributing the tasks.

As discussed on Discord, we'll not be using Tokio.

We could use something like fibers rather then writing an entire executor given how little async work we do in the project.  Native threading is fine but green threads might be better for our event system.

An option: https://github.com/dwango/fibers-rs

Nice @khionu. Would you consider looking at:

https://github.com/tauri-apps/tauri/blob/feature/iframe/tauri/src/event.rs

and making a proposal implementation with fibers and futures?

What's the current state of this issue?

I'm not sure at the moment. We just moved to the new webview bindings, and Khionu has left the project. @lucasfernog - is this something we can address or should it be closed?

well we have that lib that allows tauri commands to be async, and Tensor had some ideas about moving it to core.

I'm closing this issue since we started working on async support here: #1169
We're going to start with Tokio for simplicity but more async runtimes will be implemented in the future.

I'm closing this issue since we started working on async support here: #1169
We're going to start with Tokio for simplicity but more async runtimes will be implemented in the future.


Add stake module

Description

This PR is the stake module part of #15.
Checklist

[ ] Targeted PR against correct branch.
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Wrote unit tests.
[ ] Re-reviewed Files changed in the Github PR explorer.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ad5e001). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #21   +/-   ##
=========================================
  Coverage          ?   37.22%
=========================================
  Files             ?       11
  Lines             ?      642
  Branches          ?        0
=========================================
  Hits              ?      239
  Misses            ?      397
  Partials          ?        6

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 ad5e001...5f80138. Read the comment docs.


Text next to input gets added multiple times after update in IE11

http://jsfiddle.net/u9m5x0L7/90/
I changed the clock demo so the time is in an input and has a " %" suffix. This works as expected in Chrome, Firefox, Edge, but in IE11 after a few seconds the suffix will be doubled to " % %" and this will keep growing. I don't know what triggers this, but when you leave the window open the amount of % will grow.
When the time is not in a input it works as expected, when you put the suffix in a span it also works as expected.

I see you are using 7.1.0 in your fiddle.
Can you change it to use version 7.2.0 and check if the problem still happens? This might be related to the fixes we shipped in 7.2.0 that is currently in pre-release.

That does fix the issue indeed!

@mkxml think it'd be good to roll 7.2 over to latest?

@developit 👍 let's do it!


Add 'search_query_latency_mean' and 'search_index_latency_mean' metrics [JIRA: RIAK-1610]

I suspect the search-related _mean metrics got accidentally left out from this:
https://github.com/basho/riak_kv/pull/855
We have all the others (median, 95, etc), but not mean.
Can we add them?

Whoops, opened an issue on the wrong repo.
Moving this to https://github.com/basho/yokozuna/issues/468  and closing this one.


Conversion creates invalid JSON

hello.spdx.json.txt
hello.cdx.json.txt
Convert hello.spdx.jsonfrom SPDX to CycloneDX:
cyclonedx convert --input-file hello.spdx.json --input-format autodetect --output-file hello.cdx.json --output-format json

Try to convert the result back to SPDX:
cyclonedx convert --output-file hello2.spdx.json --output-format spdxjson --input-file hello.cdx.json --input-format json

Result:
Unhandled exception: System.Text.Json.JsonException: '}' is invalid after a single JSON value. Expected end of data. Path: $ | LineNumber: 182 | BytePositionInLine: 3.

@vargenau Could it be that hello.cdx.json already existed and you tried to overwrite it?
Then, I would assume that it is a duplicate of https://github.com/CycloneDX/cyclonedx-cli/issues/255.

I tested again with version 0.25.0 of cyclonedx-cli
The hello.spdx.json file is valid SPDX:
pyspdxtools -i hello2.spdx.json

When converted to CycloneDX then concerted back to SPDX, the resulting file hello2.spdx.json is invalid SPDX:
pyspdxtools -i hello2.spdx.json
ERROR:root:The document is invalid. The following issues have been found:
license_info_in_file is mandatory in SPDX-2.2
license_info_in_file is mandatory in SPDX-2.2
license_info_in_file is mandatory in SPDX-2.2
there must be at least one relationship "SPDXRef-DOCUMENT DESCRIBES ..." or "... DESCRIBED_BY SPDXRef-DOCUMENT" when there is not only a single package present

hello2.spdx.json.txt


Adding certificates on successful quitting of bad habits

Hey! I'm a GSSOC contributor and I'm suoer thrilled to contribute to this repo! Its amazing to work on a repo that has a real life implication and the idea is wonderful! I would like to add on this idea to motivate the people to quit their bad habits! I'm a sophomore amd would like to contribute to this one

I want to work on this issue as a contributor.
Pleasure Assign so that I can show my skills under GSSoC'23.

I have deep experience on the same and would highly appreciate if I were asssigned this one!

Hello, I'm a contributor to GSSoc'23 and I want to work on this issue. Can u please assign this issue to me?

I want to work on this Project. Plzz assign it to me.

Hey! I'm a GSSOC contributor and I'm suoer thrilled to contribute to this repo! Its amazing to work on a repo that has a real life implication and the idea is wonderful! I would like to add on this idea to motivate the people to quit their bad habits! I'm a sophomore amd would like to contribute to this one

Hey..Could you elaborate your approach and explain where you would incorporate the changes?


Invalid class when querying Win32_Process

As part of troubleshooting an issue encountered with wmi_exporter, I tried running the provided code example which queries Win32_Process:
package main

import (
	"github.com/StackExchange/wmi"
	"log"
)

type Win32_Process struct {
	Name string
}

func main() {
	var dst []Win32_Process
	q := wmi.CreateQuery(&dst, "")
	err := wmi.Query(q, &dst)
	if err != nil {
		log.Fatal(err)
	}
	for i, v := range dst {
		println(i, v.Name)
	}
}

It fails on an instance of Windows Server 2012 Standard, 64-bit with the following error:
2019/09/05 10:32:37 Exception occurred. (Invalid class )

However the following PowerShell snippet works:
Get-WmiObject -Query "SELECT Name FROM Win32_Process"

The output looks like:
__GENUS          : 2
__CLASS          : Win32_Process
__SUPERCLASS     :
__DYNASTY        :
__RELPATH        :
__PROPERTY_COUNT : 1
__DERIVATION     : {}
__SERVER         :
__NAMESPACE      :
__PATH           :
Name             : System Idle Process
PSComputerName   :
...

In addition, if I run
winmgmt /verifyrepository

the result is
WMI repository is consistent

Also, if I run wmimgmt.msc and open the properties for WMI Control (Local), no errors are shown.
Any ideas of how to proceed?

As the same, but I got an error message "invalid class when querying Win32_ComputerSystem".


Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)

[ ] iOS
[x] Android
[ ] UWP
[ ] .NET Standard

Version of Library
6.2.6
Version of OS(s) listed above with issue
Android v9
Steps to Reproduce
Try to connect to a BLE device with multiple Gatt services.
Expected Behavior
Multiple serviceUuids in the scanResult with the correct service uuids.
Actual Behavior
The scan result (IScanResult) does returns two service Uuids where the the first has both the services  somewhat concatenated.
In my scan result i get theese service uuids:
[0:] 1816180d-0000-1000-8000-00805f9b34fb
[0:] 00000000-0000-0000-0000-000000000000
From a btsnoop_hci log on the phone I get "heart rate" and "cycling speed and cadence" services as seen in the picture

So I would expect theese two service uuid's to appear in the list
0000180d-0000-1000-8000-00805f9b34fb
00001816-0000-1000-8000-00805f9b34fb

I have now tested this with the same Bluetooth peripheral on iOS.
The result is as expected, so this seems to be an Android only issue.

We are currently working on a BLE implementation for a BLE Device with custom GUIDs and are experiencing the same problem. We need to test against two GUIDs to identify the device. The ScanResult holds all GUIDs as expected under iOS while the it fails to retrieve more than one GUID under Android.
Changing the actual amount of advertised GUIDs in the BLE Device itself to only one works fine under Android.

I'm also experiencing this bug while scanning for heart rate sensors. Only Android seems to be affected. On iOS, everything works as expected.

I am experience the same problem; two service uuids are not transmitted correctly. I can confirm that with Android 6.0 and Android 8.1


Add Census GeoID and GeoType columns to Output CSV

Addresses #234 Review
Match the existing acs.csv format more closely by reordering and adding null columns

Tested on my machine, column order matches what Amanda laid out in #234


fix some pulse edge cases

Closes #328 and #357.

We’re moving out of our place today, so probably won’t get to look at this until the weekend or early next week.

On 23 Jul 2021, at 4:27, Jerome Kelleher @.***> wrote:

@jeromekelleher requested your review on: #362 fix some pulse edge cases.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub, or unsubscribe.


Relaxed layout naming requirements

Channels in Vixen profiles sometimes include slashes, spaces, dashes, and underscores. These are safe characters to have present (and most are present in this year's profile), so the constraints on valid characters in channel names have been relaxed. Some constraints on other names have also been relaxed while I was there.
Also, channels with empty names are present in this year's profile, and represent unsequenced channels. These channels are dropped.
Side note, the unique constraint on channel names has been removed.

Channel names can be changed to whatever you need.  Remaking the profile is
fairly easy
On Mon, Nov 20, 2017 at 2:14 PM, Ryan Fredlund<EMAIL_ADDRESS>wrote:

@bookdude13 commented on this pull request.
In src/project_types/file_layout.rs
https://github.com/ProtonLights/proton-cli/pull/66#discussion_r152099192
:

@@ -86,24 +90,24 @@ impl FileLayout {
if channel.channelName.len() > 40 {
return Err(Error::InvalidLayout(String::from("Channel name cannot be longer than 40 characters")));
}

       if !channel.channelName.chars().all(|c| c.is_alphanumeric() || c == ' ') {

           return Err(Error::InvalidLayout(String::from("Channel name has to be alphanumeric: ") + &channel.channelName));

       if !channel.channelName.chars().all(|c| c.is_alphanumeric() || " -/_".contains(c)) {

Not sure; these whitelists came out of what Josh organically used in
channel names, and he didn't use a \
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ProtonLights/proton-cli/pull/66#discussion_r152099192,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQJFq6mDXOTA0DWdsYFZZ3B8A6hMSKeiks5s4d2ZgaJpZM4Qkx_W
.

Added a quick fix, needed since I changed the database schema. @computergeek125 still approve?

Can we just make execute order 66 a thing for whatever issue or PR is #66?

That works for me :)


bug(MatSpinner): The UI for mat-spinner component is changed after the update to Angular 15

Is this a regression?

[X] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
14..
Description
After the update to Angular 15, it seems the structure for mat-spinner was updated and now the mat-spinner doesn't look like a complete circle (as on version 14).
This issue is more evident for smaller diameters (like 30).

Actual Behavior
The gap element seems to be outside of the circle.

Environment

Angular: 15.2.9 or 16..
CDK/Material:  15.2.9 or 16..
Browser(s): Chrome Version 114.0.5735.199 or Firefox 115.0.2
Operating System (e.g. Windows, macOS, Ubuntu): Windows

Hello,
Thank you for updating to Angular 16. I see that the spinner is not working as you expect it to.
I'm looking at the spinner in the provided stackblitz, and I don't see anything wrong with it. It has a width and height of 30px as specified by the diameter.

Unfortunately, I'm not sure I understand what "doesn't look like a complete circle" means. I was wondering if you could help us understand if this issue report requires action. Could you clarify what complete circle means and what it is expected to look like?
Best regards,
Zach


refactor(multiple): simplify ngSkipHydration expressions

We don't need to specify a value for ngSkipHydration.

cc @jessicajaniuk

Hi there!
Do you have a plan to support hydration for mat-sidebar component?


Notifications are disabled on Fdroid after application update

Presently the notifications are restored only when the user relaunches the app.

Fixed by the rebase on vector-im/riot-android (https://github.com/dinsic-pim/tchap-android/commit/04080b6b4aaca603c34352ea33e32b5132c7a6af)


Don't add the first pkcs12 certificate multiple times

This fixes a regression introcued by commit 1d6c867. When exporting a set
of certificates to a PKCS12 file we shouldn't add the first one twice. Also
we restore historic behaviour with respect to the canames option where we
have no ee certificate with key.
Fixes #15983

Note, this PR is draft - it still needs a test.

When exporting a set of certificates to a PKCS12 file we shouldn't add the first one twice.

The fix for this looks good.

Also we restore historic behaviour with respect to the canames option where we have no ee certificate with key.

I don't see how this is supposed to be achieved with the changes proposed so far?

When exporting a set of certificates to a PKCS12 file we shouldn't add the first one twice.

The fix for this looks good.

Also we restore historic behaviour with respect to the canames option where we have no ee certificate with key.

I don't see how this is supposed to be achieved with the changes proposed so far?

Meanwhile I understand why the friendly name was missing for the first copy:
the X509_dup() had been called before adding the canamesto the list of certs.

@mattcaswell, still wanna add a regression test, or shall we mark this as approval: done?

Approved with or without a test case from my perspective.

I've added a commit to add a test for this, and taken this PR out of draft. Please reconfirm!

Typo in one commit message: introcued

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Updating labels, as this seems to have gotten the necessary reviews on the latest version.

I had been doing the same, but then realized that there is a request to fix at least a typo in the commit message.

Merged after fixng typo but not adjusting test case.