Fetch artifact on Maven with credentials

Hi,
summary
We are not able to fetch artifacts from a Maven repository (hosted by Jetbrain Space) using credentials.
steps
Sbt version 1.8.2 in a Playframework project.
build.sbt
lazy val root = (project in file(".")).enablePlugins(PlayJava)
scalaVersion := "2.13.10"
credentials += Credentials("space-maven", "https://maven.pkg.jetbrains.space/xxxx/p/main/space-maven", "user", "pass")
resolvers += "space-maven" at "https://maven.pkg.jetbrains.space/xxxx/p/main/space-maven"

libraryDependencies += "com.xxxx.hello" % "hello" % "1.0"

plugins.sbt
// The Play plugin
addSbtPlugin("com.typesafe.play" % "sbt-plugin" % "2.8.19")
libraryDependencySchemes += "org.scala-lang.modules" %% "scala-xml" % VersionScheme.Always

Run sbt compile
problem
We get Unauthorized. It's like credentials are not used (got the same issues with sbt 1.7.2) :
[error] sbt.librarymanagement.ResolveException: Error downloading com.xxxx.hello:hello:1.0
[error]   Not found
[error]   Not found
[error]   not found: /Users/maxence/.ivy2/localcom.xxxx.hello/hello/1.0/ivys/ivy.xml
[error]   not found: https://repo1.maven.org/maven2/com/xxxx/hello/hello/1.0/hello-1.0.pom
[error]   unauthorized: https://maven.pkg.jetbrains.space/xxxx/p/main/space-maven/com/xxxx/hello/hello/1.0/hello-1.0.pom
[error] 	at lmcoursier.CoursierDependencyResolution.unresolvedWarningOrThrow(CoursierDependencyResolution.scala:344)
[error] 	at lmcoursier.CoursierDependencyResolution.$anonfun$update$38(CoursierDependencyResolution.scala:313)
[error] 	at scala.util.Either$LeftProjection.map(Either.scala:573)
[error] 	at lmcoursier.CoursierDependencyResolution.update(CoursierDependencyResolution.scala:313)
[error] 	at sbt.librarymanagement.DependencyResolution.update(DependencyResolution.scala:60)
[error] 	at sbt.internal.LibraryManagement$.resolve$1(LibraryManagement.scala:59)
[error] 	at sbt.internal.LibraryManagement$.$anonfun$cachedUpdate$12(LibraryManagement.scala:133)
[error] 	at sbt.util.Tracked$.$anonfun$lastOutput$1(Tracked.scala:73)
[error] 	at sbt.internal.LibraryManagement$.$anonfun$cachedUpdate$20(LibraryManagement.scala:146)
[error] 	at scala.util.control.Exception$Catch.apply(Exception.scala:228)
[error] 	at sbt.internal.LibraryManagement$.$anonfun$cachedUpdate$11(LibraryManagement.scala:146)
[error] 	at sbt.internal.LibraryManagement$.$anonfun$cachedUpdate$11$adapted(LibraryManagement.scala:127)
[error] 	at sbt.util.Tracked$.$anonfun$inputChangedW$1(Tracked.scala:219)
[error] 	at sbt.internal.LibraryManagement$.cachedUpdate(LibraryManagement.scala:160)
[error] 	at sbt.Classpaths$.$anonfun$updateTask0$1(Defaults.scala:3687)
[error] 	at scala.Function1.$anonfun$compose$1(Function1.scala:49)
[error] 	at sbt.internal.util.$tilde$greater.$anonfun$$u2219$1(TypeFunctions.scala:62)
[error] 	at sbt.std.Transform$$anon$4.work(Transform.scala:68)
[error] 	at sbt.Execute.$anonfun$submit$2(Execute.scala:282)
[error] 	at sbt.internal.util.ErrorHandling$.wideConvert(ErrorHandling.scala:23)
[error] 	at sbt.Execute.work(Execute.scala:291)
[error] 	at sbt.Execute.$anonfun$submit$1(Execute.scala:282)
[error] 	at sbt.ConcurrentRestrictions$$anon$4.$anonfun$submitValid$1(ConcurrentRestrictions.scala:265)
[error] 	at sbt.CompletionService$$anon$2.call(CompletionService.scala:64)
[error] 	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
[error] 	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539)
[error] 	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
[error] 	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
[error] 	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
[error] 	at java.base/java.lang.Thread.run(Thread.java:833)
[error] (update) sbt.librarymanagement.ResolveException: Error downloading com.xxxx.hello:hello:1.0
[error]   Not found
[error]   Not found
[error]   not found: /Users/maxence/.ivy2/localcom.xxxx.hello/hello/1.0/ivys/ivy.xml
[error]   not found: https://repo1.maven.org/maven2/com/xxxx/peppol/hello/1.0/hello-1.0.pom
[error]   unauthorized: https://maven.pkg.jetbrains.space/xxxx/p/main/space-maven/com/xxxx.hello/hello/1.0/hello-1.0.pom

expectation
Artifact should be fetched with the credentials.
notes

We are looking to use the credentials like this to make it easier to provide the login and pass via environment variables for our CI.
The "password" is a token generated that looks like a JWT token and work with maven for publish and read artifacts
We tested the credentials with a maven project and it works as expected (we can fetch the dependency).

I tried multiple things to debug but I can't figure out if it's a bug or not.

Hi,
It looks like it's not related to Coursier.
Here is a test without Coursier. I added ThisBuild / useCoursier := false in build.sbt.
Here is the trace :

[error] Unable to find credentials for [ @ maven.pkg.jetbrains.space].
[warn] 	module not found: com.xxxx.hello#hello;1.0
[warn] ==== local: tried
[warn]   /Users/maxence/.ivy2/local/com.xxxx.hello/hello/1.0/ivys/ivy.xml
[warn] ==== public: tried
[warn]   https://repo1.maven.org/maven2/com/xxxx/hello/hello/1.0/hello-1.0.pom
[warn] ==== space-maven: tried
[warn]   https://maven.pkg.jetbrains.space/xxxx/p/main/space-maven/com/xxxx/hello/hello/1.0/hello-1.0.pom
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	::          UNRESOLVED DEPENDENCIES         ::
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	:: com.xxxx.hello#hello;1.0: not found
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::

....

[warn]
[warn] 	Note: Unresolved dependencies path:
[warn] 		com.xxxx.hello:hello:1.0 (/Users/maxence/dev/workspace/myproject/back/build.sbt#L128-129)
[warn] 		  +- com.xxxx:myproject_2.13:1.0-SNAPSHOT
[error] sbt.librarymanagement.ResolveException: unresolved dependency: com.xxxx.hello#hello;1.0: not found
[error] 	at sbt.internal.librarymanagement.IvyActions$.resolveAndRetrieve(IvyActions.scala:335)
[error] 	at sbt.internal.librarymanagement.IvyActions$.$anonfun$updateEither$1(IvyActions.scala:211)
[error] 	at sbt.internal.librarymanagement.IvySbt$Module.$anonfun$withModule$1(Ivy.scala:245)
[error] 	at sbt.internal.librarymanagement.IvySbt.$anonfun$withIvy$1(Ivy.scala:209)
[error] 	at sbt.internal.librarymanagement.IvySbt.sbt$internal$librarymanagement$IvySbt$$action$1(Ivy.scala:72)
[error] 	at sbt.internal.librarymanagement.IvySbt$$anon$1.call(Ivy.scala:82)
[error] 	at xsbt.boot.Locks$GlobalLock.withChannel$1(Locks.scala:113)
[error] 	at xsbt.boot.Locks$GlobalLock.withChannelRetries$1(Locks.scala:91)
[error] 	at xsbt.boot.Locks$GlobalLock.$anonfun$withFileLock$1(Locks.scala:119)
[error] 	at xsbt.boot.Using$.withResource(Using.scala:12)
[error] 	at xsbt.boot.Using$.apply(Using.scala:9)
[error] 	at xsbt.boot.Locks$GlobalLock.withFileLock(Locks.scala:119)
[error] 	at xsbt.boot.Locks$GlobalLock.ignoringDeadlockAvoided(Locks.scala:71)
[error] 	at xsbt.boot.Locks$GlobalLock.withLock(Locks.scala:59)
[error] 	at xsbt.boot.Locks$.apply0(Locks.scala:47)
[error] 	at xsbt.boot.Locks$.apply(Locks.scala:36)
[error] 	at sbt.internal.librarymanagement.IvySbt.withDefaultLogger(Ivy.scala:82)
[error] 	at sbt.internal.librarymanagement.IvySbt.withIvy(Ivy.scala:203)
[error] 	at sbt.internal.librarymanagement.IvySbt.withIvy(Ivy.scala:200)
[error] 	at sbt.internal.librarymanagement.IvySbt$Module.withModule(Ivy.scala:244)
[error] 	at sbt.internal.librarymanagement.IvyActions$.updateEither(IvyActions.scala:196)
[error] 	at sbt.librarymanagement.ivy.IvyDependencyResolution.update(IvyDependencyResolution.scala:22)
[error] 	at sbt.librarymanagement.DependencyResolution.update(DependencyResolution.scala:60)
[error] 	at sbt.internal.LibraryManagement$.resolve$1(LibraryManagement.scala:59)
[error] 	at sbt.internal.LibraryManagement$.$anonfun$cachedUpdate$12(LibraryManagement.scala:133)
[error] 	at sbt.util.Tracked$.$anonfun$lastOutput$1(Tracked.scala:73)
[error] 	at sbt.internal.LibraryManagement$.$anonfun$cachedUpdate$20(LibraryManagement.scala:146)
[error] 	at scala.util.control.Exception$Catch.apply(Exception.scala:228)
[error] 	at sbt.internal.LibraryManagement$.$anonfun$cachedUpdate$11(LibraryManagement.scala:146)
[error] 	at sbt.internal.LibraryManagement$.$anonfun$cachedUpdate$11$adapted(LibraryManagement.scala:127)
[error] 	at sbt.util.Tracked$.$anonfun$inputChangedW$1(Tracked.scala:219)
[error] 	at sbt.internal.LibraryManagement$.cachedUpdate(LibraryManagement.scala:160)
[error] 	at sbt.Classpaths$.$anonfun$updateTask0$1(Defaults.scala:3687)
[error] 	at scala.Function1.$anonfun$compose$1(Function1.scala:49)
[error] 	at sbt.internal.util.$tilde$greater.$anonfun$$u2219$1(TypeFunctions.scala:62)
[error] 	at sbt.std.Transform$$anon$4.work(Transform.scala:68)
[error] 	at sbt.Execute.$anonfun$submit$2(Execute.scala:282)
[error] 	at sbt.internal.util.ErrorHandling$.wideConvert(ErrorHandling.scala:23)
[error] 	at sbt.Execute.work(Execute.scala:291)
[error] 	at sbt.Execute.$anonfun$submit$1(Execute.scala:282)
[error] 	at sbt.ConcurrentRestrictions$$anon$4.$anonfun$submitValid$1(ConcurrentRestrictions.scala:265)
[error] 	at sbt.CompletionService$$anon$2.call(CompletionService.scala:64)
[error] 	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
[error] 	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539)
[error] 	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
[error] 	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
[error] 	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
[error] 	at java.base/java.lang.Thread.run(Thread.java:833)
[error] (update) sbt.librarymanagement.ResolveException: unresolved dependency: com.xxxx.hello#hello;1.0: not found

Seeing above logs I tried to change my credentials realm with maven.pkg.jetbrains.space because I saw Unable to find credentials for [ @ maven.pkg.jetbrains.space].
Here is my new config for the maven part :
credentials += Credentials("maven.pkg.jetbrains.space", "https://maven.pkg.jetbrains.space/xxxx/p/main/space-maven", "user", "pass")
resolvers += "maven.pkg.jetbrains.space" at "https://maven.pkg.jetbrains.space/xxxx/p/main/space-maven"

Unfortunatly I get the exact same error as above.
Note that I have the user and pass directly in my file for now to make sure I get them correct and they work 100% sure.
I guess something wrong with my config (or a bug?) when I see [error] Unable to find credentials for [ @ maven.pkg.jetbrains.space].
Feel free to ask any tests that could help.

Maybe something wrong because when i look on internet I see that the realm should be something like [Realm Name @ my.host.com]. In my case it likes the Realm name is not specified.

Yay, I found my issue looking at @mantovani comment here : https://github.com/sbt/sbt/issues/5492#issuecomment-614492826
My new working configuration is :
credentials += Credentials(null, "maven.pkg.jetbrains.space", "user", "pass")
resolvers += "space-maven" at "https://maven.pkg.jetbrains.space/xxxx/p/main/space-maven"

The trick is to set the Realm to null.
It works without and with Coursier (eg. with or without this line ThisBuild / useCoursier := false).
For those looking for environement variables it also work like this :
credentials += Credentials(null, "maven.pkg.jetbrains.space", sys.env.get("MY_MAVEN_USER").get, sys.env.get("MY_MAVEN_PASSWORD").get)
resolvers += "space-maven" at "https://maven.pkg.jetbrains.space/xxxx/p/main/space-maven"

This works with the sbt-dotenv plugin too (for dev mode), that could looks like :
.env
MY_MAVEN_USER=user
MY_MAVEN_PASSWORD=pass

We are using sbt-dotenv version 3.0.0 :
addSbtPlugin("nl.gn0s1s" % "sbt-dotenv" % "3.0.0")

Hope this may help someone trying to use Jetbrains Space or any Maven repo with Credentials.


Document on tab completion

I'm a Sublime Text user, and one of the remaining features that's stopping the switch to VSCode for me is the DocBlockr extension.
It's essentially the same as this plugin, but it can be invoked from pressing TAB after /**:
/**tab
function say(word) {
    return word;
}

becomes:
/**
 * @param word
 * @return string
 */
function say(word) {
    return word;
}

Would it be possible to add this 'tab to complete' functionality to this extension for VSCode?

@brendo can you try Document This? I was also a big fan of DocBlockr and I've found Document this to meet my needs.
@joelday is an excellent extension author and would be happy to hear your feedback if anything is missing from his extension.

Thanks @waderyan! @brendo, make sure to look at the available settings if the default behavior isn't to your liking. Let me know if you have any issues. Thanks!

Ah great! I tried Document This prior to the 0.3.0 release which looks like it added this functionality. Thanks for the reminder to check it out again.


Data Summary Function

Returns a problem: containing list of numbers
solution: containing mean,std,variance

Please remove pycache

Please remove pycache

Remove as in remove the pycache file itself?

Remove the folder, the files will be removed with it.

Hey what happened , i didnt understand the issue

Just a doubt, why wasnt this added the hacktoberfest label?


Feature idea: every number should be a constant in one single class

Magic numbers are evil, therefore a constant should be introduced for each. A codefixer should create a constant for each number that is not. All the  constants should be created in one single class called Constants and the created name should be the written name of the number.
E.g.
Console.WriteLine(5);

Becomes
Console.WriteLine(Constants.Five);

Instead of single class with multiple constants, I'd prefere one class per value.
Usage:
Console.WriteLine((new Universe.Mathematics.Numbers.Integers.Five()).IntegerOfFive);

// prints 5.0123
Console.WriteLine((new Universe.Mathematics.Numbers.Decimals.Five.Zero.One.Two.Three()).FloatOfFiveDotZeroOneTwentyThree);

The class shall implement own interface, so mocking and IoC could be easy.


Kubernetes/Agents: Create /etc/kubernetes/manifests directory

There are too many logs like this one:
Unable to read config path "/etc/kubernetes/manifests": path does not exist, ignoring
can be solved if /etc/kubernetes/manifests was created by default on all agents.

Thanks @khenidak for this bug report.  I have marked it as a P0.

This has now been fixed.  Closing issue.  Thanks again for the report.


[CARBONDATA-2240] Refactored TestPreaggregateExpressions  to remove duplicate test case to improve CI Time

Description: This PR includes the following improvements on Preaggregate expressions and selection scenario:

Refactor UT's to remove duplicate test scenarios to improve CI time.
Refactor test case for duplicate code in different class
Correcting test case for missing assert

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[x] Any interfaces changed?  No

[x] Any backward compatibility impacted? No

[x] Document update required? No

[x] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required? NA
- How it is tested? Please attach test report.
Ran all test cases.
 - Is it a performance related change? Please attach the performance test report. NA
 - Any additional information to help reviewers in testing this change.

[x] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA. NA

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2905/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4149/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3022/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4266/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5168/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3958/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5250/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6518/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5350/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5895/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/47/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8796/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/729/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/551/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1999/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2176/

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/10251/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/74/

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/73/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/73/


amine : Workflows : student bio

[ ] I wrote my markdown locally on a new branch
[ ] My bio file is named exactly the same as my GitHub username
[ ] My bio link in the class README opens to my student bio file
[ ] I pulled and merged main into my branch before pushing
[ ] I pushed my branch to the class repository and opened a PR
[ ] I went through the PR checklist
[ ] All the CI check pass
[ ] I assigned a coach to review my PR
[ ] I linked my PR to this issue

@Mamine777 You can delete this issue.


Support web adapter input to binder output pipeline in function based SCS

Related to: https://stackoverflow.com/questions/56517391/functional-programming-model-bean-definition-and-spring-cloud-function-spring
In short, provide a feature (using functional programming model) to have a web (http/rest) input and a binder output.
There are BinderAwareChannelResolver and
annotation based examples but these solutions are marked as deprecated in 3.0.0.
Provide some code examples (eg. in spring-cloud-stream-samples) and update the documentation.

It's already supported but not documented, so please see is ahttps://github.com/spring-cloud/spring-cloud-stream/issues/1867. It is slated for this SR1 which we'll be releasing next week.
You can also read Routing Sources section this post as well as my answer on Stack Overflow

@olegz thanks. will do.


USB device is displayed in Proxmox as "Global Unichip Corp." instead of "Google ..." and therefore does not work in Frigate.

Description
Hello everyone, I have been trying to get my Google Coral to run under Proxmox since yesterday. The stick should then be passed through to an LXC on which Docker -> Frigate is installed. I followed the official documentation from Google as well as various forums. But unfortunately I can't get the Google Coral driver to work properly. It is still not recognised as a Google device, but as ‘Global Unichip Corp.’.
I have already restarted the Proxmox Node several times and connected the USB device to other ports several times, unfortunately without success.
The link to the documentation: https://coral.ai/docs/accelerator/get-started/#runtime-on-linux
I have executed the following commands from Google:
echo "deb https://packages.cloud.google.com/apt coral-edgetpu-stable main" | sudo tee /etc/apt/sources.list.d/coral-edgetpu.list

curl https://packages.cloud.google.com/apt/doc/apt-key.gpg | sudo apt-key add -

sudo apt-get update

sudo apt-get install libedgetpu1-std

These commands (according to some forums, however, unnecessary, as Frigate probably already has everything possible on board) have brought me back errors:
sudo apt-get install python3-pycoral
Error:
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 python3-pycoral : Depends: python3-tflite-runtime (= 2.5.0.post1) but it is not going to be installed
                   Depends: python3 (< 3.10) but 3.11.2-1+b1 is to be installed
E: Unable to correct problems, you have held broken packages.

Many thanks in advance for your support!
Click to expand!
Issue Type
Build/Install
Operating System
Linux
Coral Device
USB Accelerator
Other Devices
No response
Programming Language
Other
Relevant Log Output
No response

Duplicate of #536
Possible workaround in comments: https://github.com/google-coral/edgetpu/issues/536#issuecomment-2378251821


arch: mips: Author Gregory Nutt: update licenses to Apache

Summary
Gregory Nutt has submitted the SGA and we can migrate the licenses  to Apache.
Impact
NONE
Testing
NONE

expected errors caused by TABS (please ignore for now)

this one finished


New version: oneTBB_jll v2021.5.0+0

Autogenerated JLL package registration

Registering JLL package oneTBB_jll.jl
Repository: https://github.com/JuliaBinaryWrappers/oneTBB_jll.jl
Version: v2021.5.0+0
Commit: 8ec8ff481fa78b8c583ead996ceab0465f4474d5
Revision on Yggdrasil: https://github.com/JuliaPackaging/Yggdrasil/commit/dec18d5a78b0ea6807ca801cbf6cea8579840e12
Created by: @Crghilardi

This is wrong, stop


JS不想写分号？那你得注意下 - Leonsux Blog

http://leonsux.top/2017/11/16/JS不想写分号-那你得注意下-2017/
Every failure is leading towards success.

自己占个一楼


feat(toggleButton): New ToggleButton & ToggleButtonGroup components

Issue: #904
What I did
Adding

ToggleButton component
ToggleButtonGroup component

Screenshots (if appropriate):

Checklist

[x] changeset has been added
[x] I have made corresponding changes to the documentation
[x] New and existing unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works

How to test

Navigate to the ToggleButton component in storybook. Test all the controls
Navigate to the ToggleButtonGroup component in storybook. Test all the controls. Confirm that enforced and exclusive work as expected
Verify the documentation pages for both ToggleButton and ToggleButtonGroup

AC:

The component uses the "Subtle" button styles, which can't be changed.
The component can consist of a single button on its own, or as a group of buttons. Styling of the buttons (space between,
border-radius) needs to update automatically.
Buttons may contain text only, icon only, or text + icon
Each button within the component needs to turn on and off via click/tap
Button states
5.1 When the button is "on", it uses the styling of the "pressed/active" Secondary button.
5.2 Regardless of whether the button is currently on or off, it gets the hover and pressed styling when triggered
5.3 A button can be disabled, but when it is in this component, the border styling needs to be consistent with the rest of the
buttons.
With exclusive selection set, selecting one option deselects any other.
With multiple selection set, multiple buttons can be on at the same time.
You can set any button to be turned on by default.
You can enforce that at least one button in the group must be on.
You can set the size small, medium (default), and large.
11  Accessibility
11.1 Some good accessibility rules can be found here: https://mui.com/material-ui/react-toggle-button/#accessibilit
11.2 Keyboard - Toggle buttons are in DOM order. Navigate between them with the tab key. The button behavior follows
standard keyboard semantics.

AC:

The component uses the "Subtle" button styles, which can't be changed.
The component can consist of a single button on its own, or as a group of buttons. Styling of the buttons (space between,
border-radius) needs to update automatically.
Buttons may contain text only, icon only, or text + icon
Each button within the component needs to turn on and off via click/tap
Button states
5.1 When the button is "on", it uses the styling of the "pressed/active" Secondary button.
5.2 Regardless of whether the button is currently on or off, it gets the hover and pressed styling when triggered
5.3 A button can be disabled, but when it is in this component, the border styling needs to be consistent with the rest of the
buttons.
With exclusive selection set, selecting one option deselects any other.
With multiple selection set, multiple buttons can be on at the same time.
You can set any button to be turned on by default.
You can enforce that at least one button in the group must be on.
You can set the size small, medium (default), and large.
11  Accessibility
11.1 Some good accessibility rules can be found here: https://mui.com/material-ui/react-toggle-button/#accessibilit
11.2 Keyboard - Toggle buttons are in DOM order. Navigate between them with the tab key. The button behavior follows
standard keyboard semantics.


The dashboard fails to load the element whose name contiains '/'

Apache StreamPipes version
dev (current development state)
Affected StreamPipes components
Backend
What happened?
The dashboard fails to load the element name contiains '/'
How to reproduce?

Creating a pipeline that contains a data lake named 'test/001'
Create a dashboard and add a widget for the data lake
The dashboard couldn't find the widget

Expected behavior
The dashboard could find the widget and display.
Additional technical information
The root cause is that the URL is corrupted with '/'. Here is the request URL for this widget:
http://localhost:80/streampipes-backend/api/v3/datalake/pipelines/1cbeb7f2e1da494a9b9f45e36f63e513/test/001

We need to use the escape character here. Or reject the use of the name containing '/' when creating the element.
Are you willing to submit a PR?
None

@RobertIndie good catch!
I added a method to replace forbidden characters with an underscore, similar as we are rewriting fields which contain reserved keywords.
(I first tried to configure Tomcat to allow URL-encoded slashs, but couldn't manage to get the configuration property working, so I went for this solution.)


support globstar

enabled via globstar runtime option

should support zsh like behavior

also add warning for meaningless (not recursively) ** pattern

initial support in commit 64ca6f3


Adjusting the phase banner

Summary | Résumé
We need to restyle the phase banner a little bit. The current banner is center aligned and we shouldn't be mixing text alignment like that. We also want to align text wherever possible.
TO DO

[x] The banner will align with the left edge of the content container
[x] Alpha badge will align with the flag in the goc signature
[x] Banner text will align with the text in the goc signature
[x] Outline styled badge
[x] Roughly scales down to mobile devices

Zenhub card
Adjusting the phase banner(#183)

Sorry Nick, clicked on the wrong PR to mark it as draft :see_no_evil:


Decouple the smart contract deployment mechanism from the Childchain

For all deployments of the Childchain we use dev_helpers() to deploy the rootchain contract to Ethereum. This makes automated service deployment difficult as dev_helpers() is only available with MIX_ENV=dev.
We need to:

Define the process for deploying the contracts outside of the application
Select tooling to use for contract deployment

Some discussion points:

@InoMurko suggested abstracting dev_helpers() into a separate tool
Sync up with the research guys on https://github.com/omisego/plasma-contracts/issues/103
I've flattened the contracts and tried to deploy via Remix (with no success so far)
The Childchain authority address is required to be a "fresh" account due to the nonces problem (@pdobacz could you add a bit more on this please?)

The Childchain authority address is required to be a "fresh" account due to the nonces problem (@pdobacz could you add a bit more on this please?)

with pleasure: see https://github.com/omisego/elixir-omg/pull/737. This question gets asked from time to time so I decided to put that in the docs.

I think this should be fairly easy with Truffle. As a quick test, I've been able to deploy the contracts with this Truffle snippet:
var PriorityQueueLib = artifacts.require('./PriorityQueueLib.sol')
var PriorityQueueFactory = artifacts.require('./PriorityQueueFactory.sol')
var RootChain = artifacts.require('./RootChain.sol')

module.exports = function (deployer, network, accounts) {
  deployer.deploy(PriorityQueueLib)

  deployer.link(PriorityQueueLib, PriorityQueueFactory)
  deployer.deploy(PriorityQueueFactory)

  deployer.link(PriorityQueueFactory, RootChain)
  deployer.deploy(RootChain)
}

That doesn't do the init() call by the authority address, but that should be easy to add, either by
creating and funding a new address here or by passing one in via config.

Nice one, thanks @kevsul! Reckon it's best to separate the creation and funding the authority address, then calling of init()? Or write all in the same module?
Also how build the project and get the contracts? truffle init then copy the contracts to ./contracts?

Nice one, thanks @kevsul! Reckon it's best to separate the creation and funding the authority address, then calling of init()? Or write all in the same module?

I would separate it. I imagine that at some point we'll need to get more careful about the authority private key (e.g. put it in a HSM or something), so let's plan for that.

Also how build the project and get the contracts? truffle init then copy the contracts to ./contracts?

Yeah, truffle init will create a truffle.js config file which may need tweaking. The contracts are already in ./contracts. Give me a shout offline and I'll walk you through the process

[Note] In our truffle meeting call, we make the decision to move forward using truffle for deployment.

Issue https://github.com/omisego/plasma-contracts/issues/115


Artboards fused

I don't think this script has multiple artboard support.
I have 4 Artboards (6000x4000 pixels) each with 22 layers (88 layers total) and there is transparency.
When I run the script it outputs 88 pngs, but each one has the incorrect canvas size because it fuses all 4 artboards together (12000x8000). The output file has the layer I want, but it is floating in a massively blank empty canvas.
Ideally each png is the size of the artboard it is placed on.
Thanks!

Same issue but even worse: I have 4 boards for 4 sizes of icons.  If the script could go per each Artboard, it would be just perfect and superior compared to any other option. But now there's almost no difference with the default export =((

Unfortunately most of this script was written before Photoshop had artboard support, so it is not something that's built in at the moment at all.
I can do some research and see what it might take, though off-hand it sounds really complicated to integrate, but I totally understand the use-cases here. Sorry for the poor workaround.

Thanks for making this plug-in hsw107

Thank you @hsw107 Hanna for your work on this script.
If you ever find the time and inclination, I'd also love to see artboard support in your script.


我一个200多k的apng文件，初始化播放后，内存变成好几百m，是什么问题？

APNGKit 2.0 was released with huge improvement on performance and memory optimization. I am closing it for now. If the same thing happens in version 2, please consider to report again.


Allow to use new version of rest-client

Fixes #16

@licianeandreatta Do you think we can have this merged? Travis failures are unrelated - looks like a misconfiguration of env variable.

Can we close this?


S3 Storage for Media Storage

Please use an existing file abstraction layer like Gaufrette or Flysystem and create storage classes against these. So you gain automatically the ability to store media on a whole bunch of local/remote filesystems.

Replaced by #1137.


Sexplore opens both Netrw and NERDTree

With NERDTreeHijackNetrw set to 1 NERDTree replaces the default file explorer except when using Explore and its variants. I have sometimes had the experience of NERDTree then opening once the Netrw window invoked by Explore is done. This happens whenever the selected file is opened in an window other than the current Netrw, such as happens when hitting t,s, or v. The ideal behavior would be for NERDTree to also replace Netrw when using Explore and all of its variants whenever NERDTreeHijackNetrw=1. Alternatively it would be almost just as useful to have NERDTree emulate Explore by opening NERDTree with the directory of the current file. I could do this with a line in .vimrc but I can't sort out how to pass anything but a literal string as an argument to NERDTree.
I'd like to use :NERDTREE =expand('%:p:h') but it doesn't like it. There may be a much better way of doing this but my knowledge of Vimscript is very limited.

You should be using :NERDTree [<start-directory> | <bookmark>], :NERDTreeToggle [<start-directory> | <bookmark>], or :NERDTreeFind commands to open a NERDTree window. let NERDTreeHijackNetrw = 1 is used to intercept the :e <directory> command, and show a NERDTree instead of the netrw directory listing.

@PhilRunninger is it possible to make NERDTree / NERDTreeToggle open in the current window (same behavior as :e .)?

No. NERDTree always opens in a separate window.


Adjust README.md to match code

The environment variables in the readme did not match what is used (and documented) in the code.

Hello @nijel, thank you for opening this PR, but this repo is just here for historical reasons, since I moved the code upstream in to https://github.com/munin-monitoring/contrib/tree/master/plugins/syncthing.
All contributions should be made directly there.
Thanks again !


Filter option for commented records

We could introduce a filter functionality for commented records because in some cases these could be the interesting ones a user wants to see

Is your feature request related to a problem? Please describe.
Especially, on views that allow many different categories of comments, it can happen that the number of comments becomes pretty large. In this case, finding a specific comment can be tedious work.
Describe the solution you'd like
Introduce filtering for comments. Options could be "Show all", "Specific type" and "specific type and instance". This way the number of displayed comments could be reduced to the ones in which the user is interested.
Additional context
Test findings v1.5.0

Backen integration is included frontend is planned:

Endpoint for a list of record ids to display in the databrowser:

query ($projectId: ID!, $recordIds: [ID]!) {
  recordComments(projectId: $projectId, recordIds: $recordIds)
}

example variables:
  "projectId": "25bae79e-a0d6-4dc1-b9d0-9cbb01ce53f6",
  "recordIds": ["ed59c85f-5970-4421-8f6b-fb1cbf31d069"]

Subquery for filtering if a record has a comment:

let element = {
    SUBQUERY_TYPE: 'BLACKLIST' || 'WHITELIST',
    SUBQUERIES: [{
        QUERY_TEMPLATE: 'SUBQUERY_HAS_COMMENTS',
        VALUES: [UserManager.getUser().id],
    }],
};

Steps to implement:

Add checkbox on the comment section to display all comments
Add new section in the data browser: Has comments
Implement queries and mutations for the tasks
Testing and review


Update dep version in README.md

README shows version 2.9 instead of 2.11.1. 🙂

Right below it directs you to the latest release, I thought it would be rather self-explanatory to use the latest version. Updating the readme now...
Closing issue.


Fix "Open folder" button on "Custom services" screen

Description

While trying to develop a recipe I noticed that the "Open folder" button on the "Custom services" screen to open the dev recipes folder doesn't seem to work anymore.
The error is "shell.openItem is not a function". Looking at the Electron docs (https://www.electronjs.org/docs/api/shell) it looks like "shell.openItem" is no longer part of the API and should be replaced with "shell.openExternal" and "file://".
This seems to work on macOS, maybe someone can test if this also works on other platforms?
Screenshots

Checklist:

[x] My pull request is properly named
[x] The changes respect the code style of the project ($ npm run lint)
[x] I tested/previewed my changes locally

@vantezzen thank you!
Let's ship this in the nightlies for user testing 👍


How to use <IP_ADDRESS>/16 as pod cidr with flannel

I followed https://kubernetes.io/docs/setup/independent/create-cluster-kubeadm/ to create 1 master+1 slave cluster.
There is a notice in the guide:

For flannel to work correctly, you must pass --pod-network-cidr=<IP_ADDRESS>/16 to kubeadm init.

I tried --pod-network-cidr=<IP_ADDRESS>/16, with below extra steps:
# before kubeadm init, clear earlier net interface on both nodes
sudo ip link del cni0
sudo ip link del flannel.1

# after create flannel
kubectl edit cm -n kube-system kube-flannel-cfg
... edit the configuration by changing network from <IP_ADDRESS>/16 to <IP_ADDRESS>/16 ...
kubectl delete pod -n kube-system -l app=flannel

But still fail to create dashboard, with error saying "api server is not reachable".
2 nginx pods are created successfully as 192.168.0.x and 192.168.1.x, but they can not ping each other.

It's resolved, though not sure th exact cause.
The above steps plus a reboot everything works.

FYI: https://stackoverflow.com/questions/52963102/use-192-168-0-0-16-or-other-iprange-as-pod-cidr-for-kubernetes-flannel

@zenglian Your solution worked for me. I just had to update the configmap with the correct network cidr, reboot, and everything is now working.
Thanks!


Standalone Hauntings Restoration only version?

As the titles says, a standalone .asi script that only restores the missing hauntings would be cool, as those looking forward to play the game for the first time may not be interested in randomizers and some folks find installing patches difficult. Thanks for the fix though!

This is planned for the next version. I will be releasing a version that only includes the hauntings fix with no configuration options. In the long term, I will likely create a "SH4 Enhanced" mod that aims to fix some other issues with the PC port that are still present and move the haunting fix over to that. Then the randomizer can stand alone as a randomizer and not have a crucial fix for the game bundled into it.

Ah that's pretty cool to hear, you should contact Steam006, as he has already released a fixes patch for SH4 and is working on a new version, and having this integrated in that would be neat.

This is planned for the next version. I will be releasing a version that only includes the hauntings fix with no configuration options. In the long term, I will likely create a "SH4 Enhanced" mod that aims to fix some other issues with the PC port that are still present and move the haunting fix over to that. Then the randomizer can stand alone as a randomizer and not have a crucial fix for the game bundled into it.

Hello.
Thank you so much for Restore Hauntings!
Tell me, if I want to use only RestoreHauntings and FastSplash, do I not need the Data folder?
Right?

This is planned for the next version. I will be releasing a version that only includes the hauntings fix with no configuration options. In the long term, I will likely create a "SH4 Enhanced" mod that aims to fix some other issues with the PC port that are still present and move the haunting fix over to that. Then the randomizer can stand alone as a randomizer and not have a crucial fix for the game bundled into it.

Hey can you also make a free saving option or saving by interaction with items in the game without going through the wall hole back in your room all the time? And can you make an option in the config file to make these immortal ghosts killable?
That would be important improvements for me making the game more comfortable to play.
Thank you so much!

As the titles says, a standalone .asi script that only restores the missing hauntings would be cool, as those looking forward to play the game for the first time may not be interested in randomizers and some folks find installing patches difficult. Thanks for the fix though!

Dropping by to let you know Hauntings fix is now included in the latest Steam006 patch, alongside the usual bugfixes.

But is the Steam006 patch also for the GOG version?
Am So., 7. Mai 2023 um 00:23 Uhr schrieb Arhum MK @.***

:

As the titles says, a standalone .asi script that only restores the
missing hauntings would be cool, as those looking forward to play the game
for the first time may not be interested in randomizers and some folks find
installing patches difficult. Thanks for the fix though!
Dropping by to let you know Hauntings fix is now included in the latest
Steam006 patch, alongside the usual bugfixes.
—
Reply to this email directly, view it on GitHub
https://github.com/HunterStanton/SilentHill4Randomizer/issues/4#issuecomment-1537233942,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRGGU25OZQUV3BRXWJWODDXE3FPVANCNFSM6AAAAAASS7CZ5A
.
You are receiving this because you commented.Message ID:
@.***>

Yes the patch is compatible with both versions.


Choose an apropriate react library for user feedback (toast library f.ex.)

I suggest we use react-toastify


SummaryWriter issues

Hello,
It seems like that num_of_processes tb.records will be saved when I use SummaryWriter.
It's obvious but If I only need one summarized tb.record, are there any ways to do that?
Thanks.
btw, one importerror occurred on my pytorch==1.4.0, 'start_processes' is not found in torch.multiprocessing.
Which was added after 1.5.0 according to official doc. So I replaced *.py under xxxx/multiprocessing/ with the 1.8.0 ones.
works fine so far.

Hi there! To only execute some code once, you just need to put it inside a
if accelerator.is_main_process:
   code_to_execute_once_only

For your second issue, Ì had not realized it was requiring v5, I thought that function was in v4 too. Will fix!

Hi there! To only execute some code once, you just need to put it inside a
if accelerator.is_main_process:
   code_to_execute_once_only

For your second issue, Ì had not realized it was requiring v5, I thought that function was in v4 too. Will fix!

Many thanks for your quick reply.


Build linux_amd64 target for PRs on Tekton

Reference: EC-540

/retest

Logs are gone already, but I could find the image here.
This confirms what platform/arches were built:
podman run -it --rm --entrypoint /bin/bash quay.io/redhat-user-workloads/rhtap-contract-tenant/ec-main-ci/cli-main-ci:on-pr-09e76e458c5a7fa91e83c683f8c5ed5e2dd5e896  -c "ls -l /usr/local/bin"
total 224452
-rwxr-xr-x 1 root root 104977672 Apr 19 13:26 cosign
-rwxr-xr-x 1 root root  95723520 Apr 19 13:26 ec
-rwxr-xr-x 1 root root  29115023 Apr 19 13:26 ec_linux_amd64.gz
-rw-r--r-- 1 root root       119 Apr 19 13:26 ec_linux_amd64.sha256.gz

Lgtm.


Demote emitted signal log message level from Critical to Info

Critical log messages are meant to indicate that the message emitted
is one that needs to be acted upon with high urgency. Syslog defines
critical level just below the alert level, which indicates that action
must be taken now but the system is still usable. In general critical
level represents a very severe logging level and often indicates that
the system is behaving in a unexpected manner that needs correcting
ASAP.
We monitor our logs and receive alerts for any message above the error
level and alert accordingly. This is logical since these types of
messages rarely to never show up unless something is wrong. So every
time we use a signal to shut down the datadog service and receive a
critical message, we have to know to ignore the alert that it causes.
Obviously this is not ideal.
Signals are a natural occurence in operation in software and as such
these log messages should be emitted at the informational level. This
patch makes this behavior a reality.

Hey... anyone there?  If I need to do something else what is it?  I can't set the simple things to pass the 3 failed tests so I'm stuck there.  Would really appreciate if the maintainer would help out a little here.

Right on.  Thanks
On Tue, Nov 17, 2020 at 9:07 AM Kevin Conaway<EMAIL_ADDRESS>wrote:

@kevinconaway approved this pull request.
Thanks for your contribution. Apologies about the delay in responding
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/DataDog/datadog-agent/pull/6531#pullrequestreview-532699118,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABCRT3KUVEHEOXLU2TZPBTSQLCVTANCNFSM4SJMZD6A
.

--
Andrew AV. Stiegmann


Use runtime tags specified in runtime manifest for YARNContainerFactoryTests

Description

Modifies YARNContainerFactoryTests to use the runtime tags provided in a deployment's runtime manifest instead of using the latest tag. Using the latest tag for runtimes is problematic as a deployment may use tags other than latest.
Failure example:
java.util.NoSuchElementException: key not found: ImageName(nodejs6action,Some(openwhisk),Some(latest))
	at scala.collection.MapLike.default(MapLike.scala:231)
	at scala.collection.MapLike.default$(MapLike.scala:230)
	at scala.collection.AbstractMap.default(Map.scala:59)
	at scala.collection.MapLike.apply(MapLike.scala:140)
	at scala.collection.MapLike.apply$(MapLike.scala:139)
	at scala.collection.AbstractMap.apply(Map.scala:59)
	at org.apache.openwhisk.core.yarn.YARNContainerFactory.createContainer(YARNContainerFactory.scala:131)
	at org.apache.openwhisk.core.containerpool.yarn.test.YARNContainerFactoryTests.$anonfun$new$8(YARNContainerFactoryTests.scala:138)

Related to: https://github.com/apache/incubator-openwhisk/commit/6e883f9392aeb8e1ceced92bf08eab517109281f
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[x] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[x] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

PG2 4251 👍

@SamHjelmfelt, btw.

Codecov Report

Merging #4305 into master will decrease coverage by 4.52%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4305      +/-   ##
==========================================
- Coverage   85.55%   81.03%   -4.53%
==========================================
  Files         168      168
  Lines        7839     7839
  Branches      519      519
==========================================
- Hits         6707     6352     -355
- Misses       1132     1487     +355

Impacted Files
Coverage Δ

...core/database/cosmosdb/RxObservableImplicits.scala
0% <0%> (-100%)
:arrow_down:

...core/database/cosmosdb/CosmosDBArtifactStore.scala
0% <0%> (-95.18%)
:arrow_down:

...sk/core/database/cosmosdb/CosmosDBViewMapper.scala
0% <0%> (-92.6%)
:arrow_down:

...whisk/core/database/cosmosdb/CosmosDBSupport.scala
0% <0%> (-84.62%)
:arrow_down:

...abase/cosmosdb/CosmosDBArtifactStoreProvider.scala
4% <0%> (-52%)
:arrow_down:

...in/scala/org/apache/openwhisk/common/Counter.scala
40% <0%> (-20%)
:arrow_down:

...nwhisk/core/database/cosmosdb/CosmosDBConfig.scala
94.11% <0%> (-5.89%)
:arrow_down:

...whisk/connector/kafka/KafkaProducerConnector.scala
87.5% <0%> (+15%)
:arrow_up:

.../scala/org/apache/openwhisk/utils/Exceptions.scala
40% <0%> (+20%)
:arrow_up:

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 fec91db...dbd7550. Read the comment docs.


Update: Block Native USDC on Optimism 1inch zap

Once this API PR get merged 1inch will try to zap with native USDC which leaks of liquidity on optimism, so will be temporally blocked until  it gets liquidity

I've marked don't merge as the addressbook will need updated once the API PR is merged. They'll both need done at same time.
(We should only need to block the 'new' USDC for a couple of hours or something to let the liquidity checker update in the API with the new USDC/USDCe.)


[Feature] provide a flink option to chose which log to read from

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
For now, a paimon table can keep log both in file store and external log system at the same time , by setting log.system=kafka and changelog-producer=input/full-compaction at the same time.
But at the source side, if log.system=kafka is set, the continues read is bound to kafka, and file store log will be ignored.
Since log of file store can support some feature that external log system doesn't support, like full-changelog, I think it's reasonable to provide a option for user to chose which log to read from.
Solution
Provide a flink option scan.log.from=external/internal, or sth like that to expose both choices.
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

I have interest to work for this issue. Please assign this issue to me.


MVP REST API - Jun 23

Example inputs to use:
Read user_id from username: /v1/user/id/johndoe0
Get user information from user_id: /v1/user/info/0
Use login with username, password: /v1/user/login
Read song information from song_id: /v1/song/info/0
Get user_preferences from user_id: /v1/user_preferences/id/0/
Insert record into user_preference: /v1/user_preferences/new
Updates record into user_preferences: /v1/user_preferences/update

@aagrawal05 Wanna take a look?

Ight lemme check.

Idk, found this token in ./node_modules/debug/.coveralls.yml repo_token: SIAeZjKYlHK74rbcFvNHMUzjRiMpflxve. Conclusion: It may be a positive, but an incredibly easy one to fix. I'll add it to .gitignore.


net: pkt: Add possibility to reserve space for link layer header

Allow network device driver to configure the system so that when a network packet is sent to it, the link layer header is stored just before the L2 payload in the same packet.
Currently the link layer header is stored in a separate net_buf that is linked in front of the L2 payload.
This option can typically save one net_buf when sending a network packet. Note that if you are using variable data size buffers (CONFIG_NET_BUF_VARIABLE_DATA_SIZE) then this embedding is not needed because one can allocate just the right size network buffers and not waste any memory.
Add support to Ethernet L2 to select at runtime whether to have a separate link layer header, or to embed it to first net_buf.

Updated according to comments

What about having an ethernet internal net_buf pool just for headers? Then allocating a net_buf just for the header as it is would not eat up too much memory. Could be made optional as well, etc...

That is one option of course. But then it would be specific to Ethernet, what is now in the PR can be used by any technology.

It would avoid having to change net_pkt etc..., moreover this seems to be pretty much an ethernet centric issue.

That was not the main point, it was just easy to implement this for Ethernet. The feature is totally optional and the old way is the default one.

The TCP commit was there as I noticed a bit field in net_pkt that was not used any more. I can certainly send a PR for that if needed.

That is one option of course. But then it would be specific to Ethernet, what is now in the PR can be used by any technology.

But is there actually a need for some tech other than ethernet?  Does not seem so, unless I am missing something

But is there actually a need for some tech other than ethernet?

That is correct, atm there are none.
I used this route (implementing having true headroom in first net_buf) so that we could avoid the extra net_buf just for the L2 header. I did not see any easy way to implement this without having this functionality implemented in net_buf. It seems to work just fine. If the change feels too controversial, we could add a Kconfig option for it that would be turned off by default, and user could select it if really needed.

Changes:

Upgraded to latest main
Added separate Kconfig option that depends on fixed data size and Ethernet to be able to use this

But is there actually a need for some tech other than ethernet?

That is correct, atm there are none.
I used this route (implementing having true headroom in first net_buf) so that we could avoid the extra net_buf just for the L2 header. I did not see any easy way to implement this without having this functionality implemented in net_buf.

I believe it's actually much easier than what you did with net_pkt. There would be no generic net_pkt API changes anywhere, just a dedicated pool in susbsys/net/l2/ethernet.c and one ethernet public function to allocate from it. And no existing logic changes besides using that allocator at the right place (in ethernet.c and of course in the drivers when relevant) and that's it. net_buf_unref() would do the job to deallocate to the right pool anyway.
Fact is that we use to have some sort of headroom/tailroom back then in net_pkt (if I am remembering well), and we removed it for the same reason that it was useful only in L2, and only in ethernet.

But is there actually a need for some tech other than ethernet?

That is correct, atm there are none.
I used this route (implementing having true headroom in first net_buf) so that we could avoid the extra net_buf just for the L2 header. I did not see any easy way to implement this without having this functionality implemented in net_buf.

I believe it's actually much easier than what you did with net_pkt. There would be no generic net_pkt API changes anywhere, just a dedicated pool in susbsys/net/l2/ethernet.c and one ethernet public function to allocate from it. And no existing logic changes besides using that allocator at the right place (in ethernet.c and of course in the drivers when relevant) and that's it. net_buf_unref() would do the job to deallocate to the right pool anyway.
Fact is that we use to have some sort of headroom/tailroom back then in net_pkt (if I am remembering well), and we removed it for the same reason that it was useful only in L2, and only in ethernet.

But with this we would end up using an extra buf (though it fixes the wastage issue with current design), no? The idea if to avoid an extra net_pkt_read to flatten the buffers, and use zero-copy approach for most use cases (network buffer sizes configure appropriatley).

@tbursztyka please take a look to the changes again.
The reason why I am proposing this change is to avoid having a separate net_buf for L2 in the first place. The driver in question does not support scatter/gather type interface so it needs to linearize the net_buf always. This helps to solve this issue. Also a nice benefit is to have zero copy when sending the data.
The code indeed needs to touch net_pkt internals but no API changes were needed. Also the changes are used only in Ethernet so no other type driver can try to use this by mistake. The changes also require config option to be set and also the driver needs sets a runtime flag to make this happen. Chances are quite small that this change would cause problems in other part of the system. We could mark the relevant Kconfig option as experimental to help describe the nature of the option.

I had started a reply, and forgot to press "comment", sorry for the delay.
I do understand the requirement, that's not the issue. The problem is that it brings back an implementation that we removed because it was not well separating l2 with upper layers (i.e.: bringing an l2 attribute into net_pkt structure)
So here is how it could be done:

have a generic struct net_l2 net_pkt allocator function pointer.
in net_pkt side: if such allocator exists for the given net_if's L2, use it (after it estimated the size in net_pkt_alloc_buffer)
also in net_pkt, perhaps rename pkt_alloc_buffer and make it public so l2 can use it directly (it's a very short function, so it could also be fully re-implemented in l2's allocator. either way)
in L2's allocator, get the alloc_len, add the headroom you want, call pkt_alloc_buffer - or reimplement - for allocating the net_buf, then use net_buf's headroom feature to tell about the headroom (net_buf_reserve I believe?).

It's a quick idea, I did not look in the details what could be the pitfalls. But:

it would not require any net_pkt structure attribute for such l2 attribute
as soon as an L2 has an allocator, it would be generalized for all devices that belong to it. So no need of a specific capability to expose. We could save a net_buf for all ethernet devices then.

Would that make any sense?

as soon as an L2 has an allocator, it would be generalized for all devices that belong to it. So no need of a specific capability to expose. We could save a net_buf for all ethernet devices then.

I have been looking this and I think this L2 needs to be device specific and needs to be able to select at runtime. We can have multiple network interfaces belonging to same L2, for example WiFi and Ethernet could be existing at the same time and they share the same L2. The separate L2 header vs embedding it to the first buf could confuse the driver if it is not prepared for it. With this in mind, the driver should be able to tell if it could use L2 header embedding.

@tbursztyka please take a look, was this what you had in mind?

Where would a confusion in the driver come from?

I faintly recall seeing some weird handling of the net_pkt in some driver, but did not spot anything suspicious in the upstream drivers when I just looked. Anyway, perhaps it is not really a problem after all.
I can then remove the separate Ethernet flag as it seems it is not really needed after all. Just wondering should we still make this optional feature or always enabled for Ethernet. WDYT?

Add a Kconfig option to Ethernet to enable reserving header vs. allocating separate net_buf for it (default)
Removed separate Ethernet capability flag, this means that all the Ethernet/Wifi interfaces in the system will use the same way to allocate L2 header

Ping @tbursztyka, please take a look.

I revisited and refactored this according to latest comments. The feature is only available for Ethernet based interfaces, also there is no runtime configuration so all Ethernet interfaces will honor the config option CONFIG_NET_L2_ETHERNET_RESERVE_HEADER for this.


Added another model -- the Carolina Coastal Plain Model (http://sc.water...

....usgs.gov/projects/gwavailability/).  The model can be run using the mf2k.exe program followed by the name of the name file.

Kyle, here is another model for you to test.  I don't have the exact lat/long coords.  Am waiting on those.  I get a bug at the end of the netcdf creation.  YOu will need to run the model (I only have the windows exe, which is included).  Or I can send the output.


Ecobee Heat mode disappeared

Home Assistant release with the issue:

0.98.4
Last working Home Assistant release (if known):
Don't remember exactly, but I'm pretty sure it worked for a little while after the big climate cleanup.
Operating environment (Hass.io/Docker/Windows/etc.):

Hass.io on RPi3
Component/platform:

Ecobee climate https://www.home-assistant.io/components/ecobee/
Description of problem:
I have an ecobee Smart SI thermostat to control a Daikin heat pump. Before, I was able to switch modes between Heat, Cool, Auto and Off. Now, I only see Cool and Off in the dropdown list.
From the dev tools / states, I can see this attribute on my ecobee climate component: hvac_modes: cool,off. So there is no way to set the heat mode...
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
ecobee:
  api_key: !secret ecobee_api_key

Traceback (if applicable):
N/A
Additional information:
I'm pretty sure the issue came from this commit. The operation_list is now built dynamically according to thermostat settings. But since my ecobee Smart SI thermostat is an older model, it may not have the same settings than the newer ones. In fact, the thermostat was sold by Daikin and was already configured to work with Daikin heat pumps. So maybe there are some slight differences in the settings.
I'm willing to help if you need to see the content that my thermostat provided for self.thermostat['settings']. I tried to load ecobee as a custom_component and add some debug messages in the log, but it failed to load the component so I'll need some help on how to get this done.
Thanks!

Ok, I managed to get the data from my thermostat, here are some values that I have in self.thermostat["settings"]:
'coolStages': 1,
'heatStages': 0,
'hasHeatPump': True,

So, for some reason there are no heat stage, however the hasHeatPump setting may be the key here to recognize that the device supports heating mode.

Same issue. Ecobee 3 + heatpump. HA 100.1
This used to work fine with HA for years.
@setvio where you able to figure out a workaround?
arch | armv7l
-- | --
dev | false
docker | true
hassio | false
os_name | Linux
python_version | 3.7.4
timezone | America/Toronto
version | 0.100.1
virtualenv | false

OK here is that works for Ecobee 3:
sudo docker exec -it home-assistant /bin/bash
vi /usr/src/homeassistant/homeassistant/components/ecobee/climate.py
		* search for ["heatSta
		* Modify the "if" line to:
			* if self.thermostat["settings"]["heatStages"] or self.thermostat["settings"]["hasHeatPump"]:
Restart HA

@arigit, great to hear that I'm not the only one with this issue. I did exactly the same correction and it fixed the issue for me (temporarily).
Thanks for submitting a PR!

@zetvio when you get a chance can you confirm in the PR that the change works for you?


Implement MusrRoot format for data files from PSI

As of 2023, PSI is saving all muSR data from all instruments in the MusrRoot format, described here: http://lmu.web.psi.ch/musrfit/user/html/musr-root.html . We should move away from the PSI-BIN format which we have been using and implement MusrRoot instead.

So I believe we need to use musrfit software to convert the files. They have code for creating programs for reading .root files or they have an any2many utility for converting them to a different type. I'd be more tempted to do the latter and convert them to something like psi-bin or the nexus hdf5 format, since I can't imagine we would lose data doing that? Plus then we could possibly just convert all our files to a common format and read them the same way (maybe), which would probably help with consistency.
musrfit has a GPL 3 license I believe, and I think that's the one where any code that uses it needs to be GPL3 right? Would we need to update our license?

So there is no easy way to read in the MusrRoot files directly? We can use any2many for the conversion if it looks like that will be the best way forward, and I'm totally fine changing our license to GPL3 if we need to. If we end up doing a conversion, I recommend hdf5, since I think that's the most flexible.

I like HDF5 as well, and we can read it with Python which is a nice feature. We could read muSR root, but if we don’t lose data converting it then it might be easier then maintaining another exe written in c++

Then again, perhaps not. Might be a pain using the any2many utility so I’ll try and see

I've set up a fork of their repo here, I'm going to start playing around with it and seeing how much I need to cut out. Most of the windows-incompatible code I imagine is beyond the scope of what we need anyways.


[New game]: Tic Tac Toe

🎮 Game Request
I would like to propose the addition of a Tic Tac Toe game to the GameZone website. This classic game would be a valuable addition to our collection of games, providing users with a fun and engaging experience.
Point down the features
Game points
A player wins the game by aligning three of their marks (either 'X' or 'O') in a horizontal, vertical, or diagonal row.
A player loses the game if the opponent aligns three of their marks (either 'X' or 'O') in a horizontal, vertical, or diagonal row before they do.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @ShrutiGuptaaa!
We are already having a similar game request in #3173 👀
Make sure you come up with a cool unique idea 😀
Waiting for your new game idea 💗.

Hey @ShrutiGuptaaa !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @ShrutiGuptaaa ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


PROPOSAL To add new functionality to Gonum's linear algebra packages, probably in or parallel to GONUM/mat

Background
LAPACK is of course the definitive package for linear equation solutions,
and is well integrated into Gonum (thanks to whoever did that!)
But lapack is quite old, and was defined before usable auto-regularizing methods were available. The package I am proposing provides a significant new capability for Go which would impact absolutely nothing else in the Go libraries. Logically it would just sit above gonum's svd decomposition.
I have supported a similar state-of-the-art C++ package for a decade.
Recently I have converted the primary functions from that package to Go (and Python).
Along the way I have had the opportunity to further "tune" the heuristic parameters in the package, based on further test  problems, to the benefit of all three version: C++, Python, and Go.
Please see the attached README file for some details.
README.txt
Since you probably have not heard of this auto-regularizing package please see my website:
http://www.rejtrix.net/
to see my currently released versions for C++, Python, and Go.
(The Python and C++ versions are more extensive, but those extensions may be too esoteric for most users.) There is also some tutorial material that will clarify what this package, called "Arls" there, can do. If you are not familiar with solving problematic linear systems, there is a brief tutorial there to introduce you to the issue.
Proposal
I propose to add to Gonum my "larls" package of  automatically regularizing solvers for linear systems of equations. (It will be a little different than the code offered by my website because the license will have to change.) This code is completely done, very well tested, and edited based on Go's golint and other code sifters. I also have a larls-test package, with excellent code coverage, ready to deliver, modelled after existing similar gonum packages.
Potential impact of proposal
This package is built on Gonum's mat package, including gonum's svd algorithm, which I believe just calls lapack. This is strictly an ADDITION to current gonum capabilities, so it should present no difficulties.
Exactly WHERE to put it is not my call... but I would suggest it  NOT be put it in LAPACK directly. It could go in gonum/mat, but I would suggest it would be much cleaner and less maintenance burden to go in a separate package in the gonum area, probably called "larls", to suggest its relation/reliance on lapack.
The main question for you might be whether you want any such current linear algebra capabilities, or whether you are happy with the current package, though there is no capability there for auto-regularization. Arls/larls/whatever also includes a solver than forces non-negativity, whether the system is in fact well-conditioned or ill-conditioned. This routine is a (much more reliable) update to the classic Lawson/Hanson NNLS.
Interested?
Ron Jones<EMAIL_ADDRESS>

Thanks, this does look like something that could fit into mat. I agree that it is not appropriate to go in the lapack implementation. I'll defer though to @vladimir-ch.
/cc @vladimir-ch

Thanks, this does look like something that could fit into mat. I agree that it is not appropriate to go in the lapack implementation. I'll defer though to @vladimir-ch.
/cc @vladimir-ch

So what is my next step?
Ron Jones
On Thu, Jan 7, 2021 at 10:13 PM Dan Kortschak<EMAIL_ADDRESS>wrote:

Thanks, this does look like something that could fit into mat. I agree
that it is not appropriate to go in the lapack implementation. I'll defer
though to @vladimir-ch https://github.com/vladimir-ch.
/cc @vladimir-ch https://github.com/vladimir-ch
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gonum/gonum/issues/1546#issuecomment-756550436, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ASLYHNMO3VYDDDA2I4VDVADSY2H7FANCNFSM4VZ6BKXA
.

So what is my next step?
Ron Jones
On Thu, Jan 7, 2021 at 10:13 PM Dan Kortschak<EMAIL_ADDRESS>wrote:

Thanks, this does look like something that could fit into mat. I agree
that it is not appropriate to go in the lapack implementation. I'll defer
though to @vladimir-ch https://github.com/vladimir-ch.
/cc @vladimir-ch https://github.com/vladimir-ch
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gonum/gonum/issues/1546#issuecomment-756550436, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ASLYHNMO3VYDDDA2I4VDVADSY2H7FANCNFSM4VZ6BKXA
.

Sorry @rejones7go, waiting for @vladimir-ch. Germany is in lockdown, so time is not highly available.

Sorry @rejones7go, waiting for @vladimir-ch. Germany is in lockdown, so time is not highly available.

HI... since I am still not busy due to Covid, I have now expanded this package to include all the features in my C++ and Python libraries. So now the user can combine the auto-regularization solver with both equality constraints, presented as arguments E (matrix) and f (RHS values), and inequality constraints (i.e., greater than constraints), presented as arguments G (matrix) and h (RHS values). One version accepts Ex==h in addition to.
Ax=b;  another accepts Gx>=h in addition to Ax=b, and a final one accepts all three: x = Arlsall(A,b,E,f,G,h).
(These solvers build nicely on each other, with no duplicated functionality in the code.)
These solvers handle redundant or conflicting constraints, and ill-conditioning among constraints (which should be very rare).
I have placed all this code and the associated test routines (for "Go test") in this portion of my web site for your review http://www.rejones7.net/ArlsGo/index.htm
(This has not been advertised to the public and Google hasn't scanned it.)
If you want me to place this code somewhere in Github, please tell me exactly where, as I only understand Github minimally.
Thank you for considering this package.
I think you will find it to be a unique and powerful resource.
(I have also submitted it to Python's SciPy, but the SciPy developers seem to be very busy with their own issues right now. They have promised to get to it.  The C++ version has been available for 10 years and used significantly.)

HI... since I am still not busy due to Covid, I have now expanded this package to include all the features in my C++ and Python libraries. So now the user can combine the auto-regularization solver with both equality constraints, presented as arguments E (matrix) and f (RHS values), and inequality constraints (i.e., greater than constraints), presented as arguments G (matrix) and h (RHS values). One version accepts Ex==h in addition to.
Ax=b;  another accepts Gx>=h in addition to Ax=b, and a final one accepts all three: x = Arlsall(A,b,E,f,G,h).
(These solvers build nicely on each other, with no duplicated functionality in the code.)
These solvers handle redundant or conflicting constraints, and ill-conditioning among constraints (which should be very rare).
I have placed all this code and the associated test routines (for "Go test") in this portion of my web site for your review http://www.rejones7.net/ArlsGo/index.htm
(This has not been advertised to the public and Google hasn't scanned it.)
If you want me to place this code somewhere in Github, please tell me exactly where, as I only understand Github minimally.
Thank you for considering this package.
I think you will find it to be a unique and powerful resource.
(I have also submitted it to Python's SciPy, but the SciPy developers seem to be very busy with their own issues right now. They have promised to get to it.  The C++ version has been available for 10 years and used significantly.)

Thank you @rejones7go. Sorry it's taking a long time to get to this; contributor time has been fairly significantly impacted by work at home arrangements due to COVID.
We could move forward with this by incorporating it into the gonum/exp repository. That repo holds things that are being worked on but have not been fully integrated into Gonum proper, with a more fluid development model. If you would like to send a PR adding ARLS there maybe that would work out. In the meantime, I think it's worth keeping this issue open.

Hi, sorry I haven't been able to take a look yet. @kortschak is right but in addition to work at home arrangements it's the work load itself. I haven't had free time for Gonum in months and I'm not sure when it will end. I will take a look but can't guarantee when.
@rejones7go , if you're new to Github and don't feel like discovering how to push your code to our https://github.com/gonum/exp, then don't worry, I'll get it from your website. Thanks for your patience.

Thank you for the response!
I have just placed the "Arls" code and test files in
github.com/gonum/exp/rejonesgo.
I hope I did it right. If not, please move things wherever they should be,
Thanks for your helpful offer to upload it for me.
I fully understand your being very busy. (Been there/ Done that! :-) )
I am happy to wait until whenever you have time to look at this,
as it is worth the wait to reach my goal of having Arls be "official".
(My Python version is also delayed for similar reasons, plus Python SCIPY
is in flux right now anyway.)
I did all this work recently because (being retired) COVID left me looking
for something to do..
Ron Jones
On Wed, Feb 24, 2021 at 6:54 AM Vladimír Chalupecký <
<EMAIL_ADDRESS>wrote:

Hi, sorry I haven't been able to take a look yet. @kortschak
https://github.com/kortschak is right but in addition to work at home
arrangements it's the work load itself. I haven't had free time for Gonum
in months and I'm not sure when it will end. I will take a look but
can't guarantee when.
@rejones7go https://github.com/rejones7go , if you're new to Github and
don't feel like discovering how to push your code to our
https://github.com/gonum/exp, then don't worry, I'll get it from your
website. Thanks for your patience.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gonum/gonum/issues/1546#issuecomment-785091313, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ASLYHNL4XBLGAJSTN2HW4I3TAUAJZANCNFSM4VZ6BKXA
.

@rejones7go Would you be able to redo the PR so that it's against gonum/exp rather than rejones7go/exp? Also, after that would you send a PR adding yourself to the AUTHORS and CONTRIBUTORS files in gonum/gonum.

Yes, I will do that.
On Wed, Feb 24, 2021 at 9:43 PM Dan Kortschak<EMAIL_ADDRESS>wrote:

@rejones7go https://github.com/rejones7go Would you be able to redo the
PR so that it's against gonum/exp rather than rejones7go/exp? Also, after
that would you send a PR adding yourself to the AUTHORS and CONTRIBUTORS
files in gonum/gonum.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gonum/gonum/issues/1546#issuecomment-785607818, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ASLYHNJTQBHOIPREPY2YLQ3TAXIPZANCNFSM4VZ6BKXA
.

OK, I think I did a correct pull of ARLS to gonum/exp instead of
rejones7go/exp.  Let me know.
As I mentioned my previous experience with Github was minimal, and with
Python, not Go, but I want to learn to do things right.
I am sure you could do the changes to AUTHORS and CONTRIBUTORS, but I want
to understand how to do it.
Do I fork a branch of gonum/gonum to my area, edit the files there, then do
a pull request based on the fork compare?
Or am I misunderstanding the process entirely?
Sorry to be such a pain.
Thanks for your managing all this.
Ron Jones
On Wed, Feb 24, 2021 at 9:43 PM Dan Kortschak<EMAIL_ADDRESS>wrote:

@rejones7go https://github.com/rejones7go Would you be able to redo the
PR so that it's against gonum/exp rather than rejones7go/exp? Also, after
that would you send a PR adding yourself to the AUTHORS and CONTRIBUTORS
files in gonum/gonum.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gonum/gonum/issues/1546#issuecomment-785607818, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ASLYHNJTQBHOIPREPY2YLQ3TAXIPZANCNFSM4VZ6BKXA
.

Thanks for that. The ARLS PR at gonum/exp is there and I'll take a look when I get back to a keyboard later today.
As far as the PR for adding yourself to A+C, yes you need to make a GitHub fork of the gonum/gonum repo and make the change adding yourself and then send a PR by pushing your changed branch back to GitHub and then using the same PR UI you used to send the ARLS PR. If you have any issues I'll try to explain more clearly (if you are using git on the command line, then it should give you a success output on pushing the branch that links to a PR set-up page).

Thanks for your guidance and patience.
I believe I just correctly modified AUTHORS and CONTRIBUTORS.
Ron
On Thu, Feb 25, 2021 at 9:28 PM Dan Kortschak<EMAIL_ADDRESS>wrote:

Thanks for that. The ARLS PR at gonum/exp is there and I'll take a look
when I get back to a keyboard later today.
As far as the PR for adding yourself to A+C, yes you need to make a GitHub
fork of the gonum/gonum repo and make the change adding yourself and then
send a PR by pushing your changed branch back to GitHub and then using the
same PR UI you used to send the ARLS PR. If you have any issues I'll try to
explain more clearly (if you are using git on the command line, then it
should give you a success output on pushing the branch that links to a PR
set-up page).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gonum/gonum/issues/1546#issuecomment-786403121, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ASLYHNPEOMGIZAAWNST62CDTA4PQBANCNFSM4VZ6BKXA
.

Thanks. If you're happy to fix up the license header in the gonum/exp PR, I'll approve and merge both of them.


File master is not used anymore by GitHub

First of all, well done it's a great app!
Anyway in the "Remote control" challenge, it requires to create the remote repository using master branch, but GitHub has replaced it with main now, so I wasn't able to accomplish it!
Keep up the great work

@jlord would you be open to this change? To change all references to master to main instead?
It may require some additional setup on some people's machines (to default in Git to main instead of master) - eg. git config --global init.defaultBranch main as in our System Setup Guide for our bootcamp.

@jlord would you be open to this change? To change all references to master to main instead?
It may require some additional setup on some people's machines (to default in Git to main instead of master) - eg. git config --global init.defaultBranch main as in our System Setup Guide for our bootcamp.

Hi! I am definitely open to this change, if someone can get to it and make a PR that would be great otherwise I will try to make some time in the next couple of week. Thanks for raising!

Hi! I am definitely open to this change, if someone can get to it and make a PR that would be great otherwise I will try to make some time in the next couple of week. Thanks for raising!

Hey Jessica,
nice to see you back here on the repo!
As nothing moved here for a while, and we didn't really reach you, in July i took a fork of the app, first had it on my account and later transferred it to an organisation (https://github.com/Git-it-App/git-it-electron). I updated all dependencies, moved translations to transifex for easier management, basically rewrote quite much of the code and even updated some content changes of GitHub. - Result is an up-to date version of the app, with translations beeing managed on transifex (some translations still need to be copied over). - So how do you think about joining there and keeping the app alive as a community project, instead of having parallel projects?

Respective this specific change discussed here: I thought about this already, even moved the Organisations Repo to main quite a while ago and started on preparing the content-change two weeks ago. However, Git itself has not changed the default branch yet.
-> As long as one does not initialise the Repo (as it is told in the app), GitHub does not produce the main branch. So normally completing the challenges should still be possible. Probably some other error happened to @buondevid? Did you probably init the repo on Github?
-> As the challenges are valid as they are, i thought until now to leave the master branch as it is and to wait until Git itself changes the default branch to main. This avoids the need of a setting, which users don't really understand at the beginning of the tutorial. However, i'm open to discuss this.

How do you think about?
Looking forward for your response! :)
Greets,
Jonas

Hey Jessica,
nice to see you back here on the repo!
As nothing moved here for a while, and we didn't really reach you, in July i took a fork of the app, first had it on my account and later transferred it to an organisation (https://github.com/Git-it-App/git-it-electron). I updated all dependencies, moved translations to transifex for easier management, basically rewrote quite much of the code and even updated some content changes of GitHub. - Result is an up-to date version of the app, with translations beeing managed on transifex (some translations still need to be copied over). - So how do you think about joining there and keeping the app alive as a community project, instead of having parallel projects?

Respective this specific change discussed here: I thought about this already, even moved the Organisations Repo to main quite a while ago and started on preparing the content-change two weeks ago. However, Git itself has not changed the default branch yet.
-> As long as one does not initialise the Repo (as it is told in the app), GitHub does not produce the main branch. So normally completing the challenges should still be possible. Probably some other error happened to @buondevid? Did you probably init the repo on Github?
-> As the challenges are valid as they are, i thought until now to leave the master branch as it is and to wait until Git itself changes the default branch to main. This avoids the need of a setting, which users don't really understand at the beginning of the tutorial. However, i'm open to discuss this.

How do you think about?
Looking forward for your response! :)
Greets,
Jonas

So how do you think about joining there (Git-it-App/git-it-electron) and keeping the app alive as a community project, instead of having parallel projects?

@jotoeri, what is it that you're looking to achieve here? More maintainers / velocity on the project?
Couldn't this also be achieved by leaving the project under the jlord username and:

opening a call for more maintainers here
potentially pulling over interesting commits from your project

So how do you think about joining there (Git-it-App/git-it-electron) and keeping the app alive as a community project, instead of having parallel projects?

@jotoeri, what is it that you're looking to achieve here? More maintainers / velocity on the project?
Couldn't this also be achieved by leaving the project under the jlord username and:

opening a call for more maintainers here
potentially pulling over interesting commits from your project

Hey Karl,

well, what i would appreciate, would be to merge the currently two projects back into one, having me and Jessica as maintainer from the two current branches, and yes, if you or others want to join - you're welcome!
More velocity is not necessary, the only thing, that needs to be cared for, are now dependency updates and changes on Git/GitHub. The app itself is in a good state, i think, it is not necessary, to have a fast evolution there.
(I thought about a second app already, covering some advanced topics of daily work, but that's different thing then.)

potentially pulling over interesting commits from your project

Well, that is around 300 commits? 😉 Indeed - one could do one massive merge or just pull everything, that would work. Picking just some single commits of it - could result in some more work to resolve that... - However - all of that would feel somehow like not appreciating the work i put into. It was quite much of a work, that i did. Now moving all of this back here, dissolving into which basically was a stale repository (i tried a few times to contribute here, at first!), that feels quite sad. That is relatable, no?
So this is why i think it would be nice, to have it as a community project, beeing equal maintainers and to avoid the loss of maintenace based on one person...
Greets,
Jonas :)

Hey Karl,

well, what i would appreciate, would be to merge the currently two projects back into one, having me and Jessica as maintainer from the two current branches, and yes, if you or others want to join - you're welcome!
More velocity is not necessary, the only thing, that needs to be cared for, are now dependency updates and changes on Git/GitHub. The app itself is in a good state, i think, it is not necessary, to have a fast evolution there.
(I thought about a second app already, covering some advanced topics of daily work, but that's different thing then.)

potentially pulling over interesting commits from your project

Well, that is around 300 commits? 😉 Indeed - one could do one massive merge or just pull everything, that would work. Picking just some single commits of it - could result in some more work to resolve that... - However - all of that would feel somehow like not appreciating the work i put into. It was quite much of a work, that i did. Now moving all of this back here, dissolving into which basically was a stale repository (i tried a few times to contribute here, at first!), that feels quite sad. That is relatable, no?
So this is why i think it would be nice, to have it as a community project, beeing equal maintainers and to avoid the loss of maintenace based on one person...
Greets,
Jonas :)

I don't have an opinion, personally - I don't have any investment in either outcome. If it stays here, that's ok with me. If it goes to a new organization, that's fine with me too.
Regardless of the way forward, jlord/git-it-electron has ~3700 stars and ~1000 forks. Probably the project shouldn't lose these metrics.
Two approaches come to mind:

Keep it under jlord. Easy and uncomplicated.

Transfer ownership of the repository to a community-led organization. It would need to be considered carefully who would be a maintainer, what rights they would have, etc.

By the way, in either of these options, if your commits are considered valuable to the project, they could be transferred in a way that they are properly attributed to you - you would show up under the Contributors list, eg https://github.com/jlord/git-it-electron/graphs/contributors

I don't have an opinion, personally - I don't have any investment in either outcome. If it stays here, that's ok with me. If it goes to a new organization, that's fine with me too.
Regardless of the way forward, jlord/git-it-electron has ~3700 stars and ~1000 forks. Probably the project shouldn't lose these metrics.
Two approaches come to mind:

Keep it under jlord. Easy and uncomplicated.

Transfer ownership of the repository to a community-led organization. It would need to be considered carefully who would be a maintainer, what rights they would have, etc.

By the way, in either of these options, if your commits are considered valuable to the project, they could be transferred in a way that they are properly attributed to you - you would show up under the Contributors list, eg https://github.com/jlord/git-it-electron/graphs/contributors

~3700 stars and ~1000 forks

Well stars, ok, some nice stuff. About the forks - how many of them are copied once and never touched again? 🤔 And tbh. - i'm not sure if i ever seroiusly looked onto this 'metrics' on any project.

could be transferred in a way that they are properly attributed to you

I know - but my text above was written having this knowledge already. 😉

So:

Adding jessica as full maintainer/owner to the current organisation? Easy and uncomplicated? 😉

=> Let's wait and see, what Jessica thinks about...

~3700 stars and ~1000 forks

Well stars, ok, some nice stuff. About the forks - how many of them are copied once and never touched again? 🤔 And tbh. - i'm not sure if i ever seroiusly looked onto this 'metrics' on any project.

could be transferred in a way that they are properly attributed to you

I know - but my text above was written having this knowledge already. 😉

So:

Adding jessica as full maintainer/owner to the current organisation? Easy and uncomplicated? 😉

=> Let's wait and see, what Jessica thinks about...

That's not actually an option that will retain the GitHub metrics. Metrics matter a lot.

That's not actually an option that will retain the GitHub metrics. Metrics matter a lot.

I am going to try and help out with a PR for this! I had the same issue and I'm happy someone else already pointed it out :)

I am going to try and help out with a PR for this! I had the same issue and I'm happy someone else already pointed it out :)

Thanks @izzyreiff , I would have done it myself if I was able to :)

Thanks @izzyreiff , I would have done it myself if I was able to :)

As a workaround, you can create a local master branch, switch to it, push it to GitHub and continues the Challenges :
git checkout -b master
git push origin master


Fix unused variable warning from clang

👍

Current coverage is 64.42% (diff: 100%)

Merging #1887 into develop will decrease coverage by 0.02%

@@            develop      #1887   diff @@
==========================================
  Files           685        685
  Lines         49009      49009
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits          31583      31573    -10
- Misses        17426      17436    +10
  Partials          0          0

Powered by Codecov. Last update 7688a97...e844510

Merged in db13ddf


list of files on commandline ignored

I'm running rustfmt 0.9.0-nightly (d50e361 2018-08-01) from rustup. When I invoke rustfmt with a list of files, it formats all the files recursively from my current working directory, instead of just the files I listed.
Minimal repro case: https://github.com/tkilbourn/rustfmt-test
Run rustfmt src/main.rs. Observe src/other.rs is modified.

https://github.com/rust-lang-nursery/rustfmt#running
... This runs rustfmt on the given file, if the file includes out of line modules, then we reformat those too. So to run on a whole module or crate, you just need to run on the root file (usually mod.rs or lib.rs). ...

You can use --unstable-features --skip-children to prevent formatting child modules.

Ah, so this is WAI. I find this unintuitive, based on the usage message. Now that I know, the --skip-children flag makes sense, but I don't know if I'd have noticed it before. I know it would be a breaking change, but will --skip-children ever become the default? I imagine many users will be using it with cargo anyway.

@nrc Has there been any more consideration of this? Is there a plan for stabilizing --skip-children soon? Personally I also would find this much more intuitive as the default, since folks wanting to format their whole crate will likely prefer cargo fmt.

I'm planning on moving stabilise a bunch more flags and options once we get Rustfmt 1.0 out the door (i.e., post-edition). That will probably include this.

Thanks for the update!


debugger in paper-code-mirror.html

Line 123 of the file paper-code-mirror.html has a debugger call. Was that left in intentionally?

Removed the "debugger" from there. No, it was by accidentally.


move Clangd* commands to on_attach function so they are only available after clangd attached ?

Sure, thanks

Fixed in https://github.com/p00f/clangd_extensions.nvim/commit/975ccb9d67011f3fae34bd7366da09252bc37a3f

Thank you~


add console support

adds readme
adds console support
todo: why are xsl's not cloned into egg, now throws file-not-found for xsl's

this works ok now, ready to merge

this works ok now, ready to merge


Performance issue typing in Chrome on Mac

We've gotten some reports of slowness when typing in Chrome on Mac.  Needs to be reproduced in a development environment, where we can easily profile what the cause is.

I too have noticed slowness when typing on Chrome on Ubuntu. It is most prevalent when I hold the backspace key for a few seconds.
I have only seen this problem on zulip.tabbott.net. I have never seen this in my development environment.

I've managed to reproduce this is in a development environment -- it appears to be caused by the recent upgrade of jquery from 1.7.x to 1.8.x.  The new version of jQuery makes initializing an object much more expensive (up to 60ms if the page has a lot of jquery objects in it), and we currently do that on each keypress.  It's not noticable in the development environment because there are less objects present.  I'm going to revert the upgrade for now; I suspect we can fix this problem eventually by upgrading to a yet newer jquery version, since according to the documentation at least, they seem to have resolved this performance problem since 1.8.x.


vagrant-cachier

Did you run the tests with vagrant-cachier installed or no? If you did, it might need to be clarified as well.

Hmm... I'm going to check.

It looks like it wasn't installed on the two 2016 models, but it was installed on my 2015 Pro. That could skew the test a little, as the 2015 model wouldn't have had to re-download all the apt packages (and composer too, right?) on each run.
That would also explain the huge delta in the average test speed!
I'll have to re-run the test with cachier uninstalled on my 2015 Pro tonight.

In any case, the 2016 model comparison is still valid, I just need to see if I get largely different results from the 2015. Good catch!

@oxyc - Thanks again for pointing this out! I ran the tests again without vagrant-cachier, and uploaded the results to a separate sheet so you can compare:
https://docs.google.com/spreadsheets/d/16H6TeKCOZRwzsd5bZJM2IHVqN9fU6GZhUrDiu_SK2zU/edit?usp=sharing
It performed one fewer run, and battery runtime was 5.6 hours instead of 4.9. Interesting result, but I don't know if I can keep sacrificing my entire night and part of a morning (draining the battery resets the PMU, which resets the clock, which causes all kinds of crazy with Apps that depend on the correct time) for these tests on one of my main laptops.


Update migration.md

Adding a note about exceptions to the migration guide
TODO:

Add page to list exceptions
FAQ update
Project Delete update

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

These docs need to be released. is this pr still valid?


Extend CommutativeInverseCancellation to cancel pairs of gates up-to-phase

Summary
Extending CommutativeInverseCancellation transpiler pass to cancel pairs of gates that are inverse up to a phase. Updates the global phase of the circuit accordingly.
Here is a small example:
import numpy as np
from qiskit.circuit import QuantumCircuit
from qiskit.transpiler import PassManager
from qiskit.transpiler.passes import CommutativeInverseCancellation

circuit = QuantumCircuit(1)
circuit.rz(np.pi / 4, 0)
circuit.p(-np.pi / 4, 0)

passmanager = PassManager(CommutativeInverseCancellation(upto_phase_optimization=True))
new_circuit = passmanager.run(circuit)

The pass is able to cancel the RZ and P gates, adjusting the global phase of the circuit to 15 \pi / 8.
Details:
The pass has two new arguments max_qubits and upto_phase_optimization. The new optimization is enabled by setting upto_phase_optimization = True. Generally this leads to more reductions at the expense of increased runtime. Internally, the check is based on constructing and comparing unitary matrices corresponding to various gates in the circuit. The argument max_qubits limits the number of qubits in matrix-based commutativity and inverse checks.

LGTM except I still think this does more than just ignoring global phase. If I define two custom adjacent gates who will cancel each other, this new PR will be able to cancel them -- no matter how these gates are called. This wasn't possible before, was it? If this is correct, how about renaming upto_phase_optimization to something like matrix_based?

Thanks @Cryoris, I did the renaming as per review comments and offline conversation.


Update README.md

Adding Start Blazoring.

Hi,
Is the source code available?

Hi Adrien, it is not. It's a commercial product my company is offering. I saw Syncfusion and Telerik in here and figure it was OK.

Released in https://github.com/AdrienTorris/awesome-blazor/releases/tag/v2020.W36.