Are there any limitations with the input FG?

Thanks for your effort on this.
I am wondering if there are any limitations on the input FG in terms of size or shape? I modified your input FG to have few factors, it fails with null pointers at the point of loading the graph.
Also, would be able to tell me how you decide the factor values in the input FG for BP?

It loads the graph. I get the following exception:
17/02/06 20:08:10 INFO TaskSchedulerImpl: Adding task set 12.0 with 1 tasks
17/02/06 20:08:10 INFO TaskSetManager: Starting task 0.0 in stage 12.0 (TID 8, localhost, PROCESS_LOCAL, 2146 bytes)
17/02/06 20:08:10 INFO Executor: Running task 0.0 in stage 12.0 (TID 8)
17/02/06 20:08:10 INFO BlockManager: Found block rdd_18_0 locally
17/02/06 20:08:10 INFO Executor: Finished task 0.0 in stage 12.0 (TID 8). 2253 bytes result sent to driver
17/02/06 20:08:10 INFO TaskSetManager: Finished task 0.0 in stage 12.0 (TID 8) in 20 ms on localhost (1/1)
17/02/06 20:08:10 INFO DAGScheduler: ShuffleMapStage 12 (mapPartitions at VertexRDDImpl.scala:247) finished in 0.021 s
17/02/06 20:08:10 INFO TaskSchedulerImpl: Removed TaskSet 12.0, whose tasks have all completed, from pool
17/02/06 20:08:10 INFO DAGScheduler: looking for newly runnable stages
17/02/06 20:08:10 INFO DAGScheduler: running: Set()
17/02/06 20:08:10 INFO DAGScheduler: waiting: Set(ResultStage 13)
17/02/06 20:08:10 INFO DAGScheduler: failed: Set()
17/02/06 20:08:10 INFO DAGScheduler: Missing parents for ResultStage 13: List()
17/02/06 20:08:10 INFO DAGScheduler: Submitting ResultStage 13 (MapPartitionsRDD[32] at map at EdgeRDDImpl.scala:89), which is now runnable
17/02/06 20:08:10 INFO MemoryStore: ensureFreeSpace(5896) called with curMem=522206, maxMem=555755765
17/02/06 20:08:10 INFO MemoryStore: Block broadcast_11 stored as values in memory (estimated size 5.8 KB, free 529.5 MB)
17/02/06 20:08:10 INFO MemoryStore: ensureFreeSpace(3044) called with curMem=528102, maxMem=555755765
17/02/06 20:08:10 INFO MemoryStore: Block broadcast_11_piece0 stored as bytes in memory (estimated size 3.0 KB, free 529.5 MB)
17/02/06 20:08:10 INFO BlockManagerInfo: Added broadcast_11_piece0 in memory on localhost:36967 (size: 3.0 KB, free: 529.9 MB)
17/02/06 20:08:10 INFO SparkContext: Created broadcast 11 from broadcast at DAGScheduler.scala:861
17/02/06 20:08:10 INFO DAGScheduler: Submitting 1 missing tasks from ResultStage 13 (MapPartitionsRDD[32] at map at EdgeRDDImpl.scala:89)
17/02/06 20:08:10 INFO TaskSchedulerImpl: Adding task set 13.0 with 1 tasks
17/02/06 20:08:10 INFO TaskSetManager: Starting task 0.0 in stage 13.0 (TID 9, localhost, PROCESS_LOCAL, 2556 bytes)
17/02/06 20:08:10 INFO Executor: Running task 0.0 in stage 13.0 (TID 9)
17/02/06 20:08:10 INFO CacheManager: Partition rdd_30_0 not found, computing it
17/02/06 20:08:10 INFO BlockManager: Found block rdd_21_0 locally
17/02/06 20:08:10 INFO ShuffleBlockFetcherIterator: Getting 1 non-empty blocks out of 1 blocks
17/02/06 20:08:10 INFO ShuffleBlockFetcherIterator: Started 0 remote fetches in 1 ms
17/02/06 20:08:10 ERROR Executor: Exception in task 0.0 in stage 13.0 (TID 9)
java.lang.NullPointerException
at sparkle.graph.BeliefPropagation$$anonfun$2.apply(BeliefPropagation.scala:48)
at sparkle.graph.BeliefPropagation$$anonfun$2.apply(BeliefPropagation.scala:47)
at scala.collection.Iterator$$anon$11.next(Iterator.scala:328)
at org.apache.spark.graphx.impl.EdgePartition.map(EdgePartition.scala:185)
at org.apache.spark.graphx.impl.GraphImpl$$anonfun$7.apply(GraphImpl.scala:156)
at org.apache.spark.graphx.impl.GraphImpl$$anonfun$7.apply(GraphImpl.scala:155)
at org.apache.spark.graphx.impl.EdgeRDDImpl$$anonfun$mapEdgePartitions$1.apply(EdgeRDDImpl.scala:121)
at org.apache.spark.graphx.impl.EdgeRDDImpl$$anonfun$mapEdgePartitions$1.apply(EdgeRDDImpl.scala:118)
at org.apache.spark.rdd.RDD$$anonfun$mapPartitions$1$$anonfun$apply$17.apply(RDD.scala:710)
at org.apache.spark.rdd.RDD$$anonfun$mapPartitions$1$$anonfun$apply$17.apply(RDD.scala:710)
at org.apache.spark.rdd.MapPartitionsRDD.compute(MapPartitionsRDD.scala:38)
at org.apache.spark.rdd.RDD.computeOrReadCheckpoint(RDD.scala:300)
at org.apache.spark.CacheManager.getOrCompute(CacheManager.scala:69)
at org.apache.spark.rdd.RDD.iterator(RDD.scala:262)
at org.apache.spark.rdd.MapPartitionsRDD.compute(MapPartitionsRDD.scala:38)
at org.apache.spark.rdd.RDD.computeOrReadCheckpoint(RDD.scala:300)
at org.apache.spark.rdd.RDD.iterator(RDD.scala:264)
at org.apache.spark.scheduler.ResultTask.runTask(ResultTask.scala:66)
at org.apache.spark.scheduler.Task.run(Task.scala:88)
at org.apache.spark.executor.Executor$TaskRunner.run(Executor.scala:214)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)
I am using a very simple factor graph:
1 factor and 2 variables:
1
3
2
1  2
2 2
4
0   1
1    0.9
2     0.9
3    1

Hi @nabeelxy! We are using a slightly modified format with explicit factor IDs. Please refer to the example: https://github.com/HewlettPackard/sandpiper/blob/master/data/factor/graph7.fg.

Hi @nabeelxy, factor values are part of the input to BP. They are usually determined based on domain knowledge, or if enough data is available they can be learned from data.

Thank you @avulanov and @manish-marwah for your quick replies.
@manish-marwah, I noticed that you guys have two unused data files vetex100.txt and edge100.txt. They have fixed edge potentials and prior probabilities. Are they used in any why in your code? Also, since they are fixed, can we learn the factor functions from them? One more thing, am I correct that in your factor graph input files factor values are randomly assigned (i.e. not based on any domain knowledge of the variables)?
Thank you guys again for your replies and work.

Hi @avulanov, I think I used the same format as you pointed in graph7.fg. It may be a typo when I copied the input file. Just to be clear, my input factor graph is as follows:
1
3
2
1 2
2 2
4
0 1
1 0.9
2 0.9
3 1
Format of the file is as expected. In the logs, I see that vertices and edges are identified correctly:
Edge(1,3,())
Edge(2,3,())
(1,null)
(3,sparkle.graph.NamedFactor@5e1fc42f)
(2,null)
However, I still get the null pointer exception. Could it be a bug in libDAI?
....
17/02/07 08:31:22 INFO TaskSchedulerImpl: Adding task set 12.0 with 1 tasks
17/02/07 08:31:22 INFO TaskSetManager: Starting task 0.0 in stage 12.0 (TID 8, localhost, PROCESS_LOCAL, 2146 bytes)
17/02/07 08:31:22 INFO Executor: Running task 0.0 in stage 12.0 (TID 8)
17/02/07 08:31:22 INFO BlockManager: Found block rdd_18_0 locally
17/02/07 08:31:22 INFO Executor: Finished task 0.0 in stage 12.0 (TID 8). 2253 bytes result sent to driver
17/02/07 08:31:22 INFO TaskSetManager: Finished task 0.0 in stage 12.0 (TID 8) in 24 ms on localhost (1/1)
17/02/07 08:31:22 INFO TaskSchedulerImpl: Removed TaskSet 12.0, whose tasks have all completed, from pool
17/02/07 08:31:22 INFO DAGScheduler: ShuffleMapStage 12 (mapPartitions at VertexRDDImpl.scala:247) finished in 0.024 s
17/02/07 08:31:22 INFO DAGScheduler: looking for newly runnable stages
17/02/07 08:31:22 INFO DAGScheduler: running: Set()
17/02/07 08:31:22 INFO DAGScheduler: waiting: Set(ResultStage 13)
17/02/07 08:31:22 INFO DAGScheduler: failed: Set()
17/02/07 08:31:22 INFO DAGScheduler: Missing parents for ResultStage 13: List()
17/02/07 08:31:22 INFO DAGScheduler: Submitting ResultStage 13 (MapPartitionsRDD[32] at map at EdgeRDDImpl.scala:89), which is now runnable
17/02/07 08:31:22 INFO MemoryStore: ensureFreeSpace(5896) called with curMem=522262, maxMem=555755765
17/02/07 08:31:22 INFO MemoryStore: Block broadcast_11 stored as values in memory (estimated size 5.8 KB, free 529.5 MB)
17/02/07 08:31:23 INFO MemoryStore: ensureFreeSpace(3055) called with curMem=528158, maxMem=555755765
17/02/07 08:31:23 INFO MemoryStore: Block broadcast_11_piece0 stored as bytes in memory (estimated size 3.0 KB, free 529.5 MB)
17/02/07 08:31:23 INFO BlockManagerInfo: Added broadcast_11_piece0 in memory on localhost:42590 (size: 3.0 KB, free: 529.9 MB)
17/02/07 08:31:23 INFO SparkContext: Created broadcast 11 from broadcast at DAGScheduler.scala:861
17/02/07 08:31:23 INFO DAGScheduler: Submitting 1 missing tasks from ResultStage 13 (MapPartitionsRDD[32] at map at EdgeRDDImpl.scala:89)
17/02/07 08:31:23 INFO TaskSchedulerImpl: Adding task set 13.0 with 1 tasks
17/02/07 08:31:23 INFO TaskSetManager: Starting task 0.0 in stage 13.0 (TID 9, localhost, PROCESS_LOCAL, 2556 bytes)
17/02/07 08:31:23 INFO Executor: Running task 0.0 in stage 13.0 (TID 9)
17/02/07 08:31:23 INFO CacheManager: Partition rdd_30_0 not found, computing it
17/02/07 08:31:23 INFO BlockManager: Found block rdd_21_0 locally
17/02/07 08:31:23 INFO ShuffleBlockFetcherIterator: Getting 1 non-empty blocks out of 1 blocks
17/02/07 08:31:23 INFO ShuffleBlockFetcherIterator: Started 0 remote fetches in 0 ms
17/02/07 08:31:23 ERROR Executor: Exception in task 0.0 in stage 13.0 (TID 9)
java.lang.NullPointerException
at sparkle.graph.BeliefPropagation$$anonfun$2.apply(BeliefPropagation.scala:48)
at sparkle.graph.BeliefPropagation$$anonfun$2.apply(BeliefPropagation.scala:47)
at scala.collection.Iterator$$anon$11.next(Iterator.scala:328)
at org.apache.spark.graphx.impl.EdgePartition.map(EdgePartition.scala:185)
at org.apache.spark.graphx.impl.GraphImpl$$anonfun$7.apply(GraphImpl.scala:156)
at org.apache.spark.graphx.impl.GraphImpl$$anonfun$7.apply(GraphImpl.scala:155)
at org.apache.spark.graphx.impl.EdgeRDDImpl$$anonfun$mapEdgePartitions$1.apply(EdgeRDDImpl.scala:121)
at org.apache.spark.graphx.impl.EdgeRDDImpl$$anonfun$mapEdgePartitions$1.apply(EdgeRDDImpl.scala:118)
at org.apache.spark.rdd.RDD$$anonfun$mapPartitions$1$$anonfun$apply$17.apply(RDD.scala:710)
at org.apache.spark.rdd.RDD$$anonfun$mapPartitions$1$$anonfun$apply$17.apply(RDD.scala:710)
at org.apache.spark.rdd.MapPartitionsRDD.compute(MapPartitionsRDD.scala:38)
at org.apache.spark.rdd.RDD.computeOrReadCheckpoint(RDD.scala:300)
at org.apache.spark.CacheManager.getOrCompute(CacheManager.scala:69)
at org.apache.spark.rdd.RDD.iterator(RDD.scala:262)
at org.apache.spark.rdd.MapPartitionsRDD.compute(MapPartitionsRDD.scala:38)
at org.apache.spark.rdd.RDD.computeOrReadCheckpoint(RDD.scala:300)
at org.apache.spark.rdd.RDD.iterator(RDD.scala:264)
at org.apache.spark.scheduler.ResultTask.runTask(ResultTask.scala:66)
at org.apache.spark.scheduler.Task.run(Task.scala:88)
at org.apache.spark.executor.Executor$TaskRunner.run(Executor.scala:214)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)
....
Any pointers in this regard is highly appreciated.

@nabeelxy could you try running the unit test? it uses exactly the same file https://github.com/HewlettPackard/sandpiper/blob/master/src/test/scala/sparkle/graph/BeliefPropagationSuite.scala

@manish-marwah, I noticed that you guys have two unused data files vetex100.txt and edge100.txt. They have fixed edge potentials and prior probabilities. Are they used in any why in your code? Also, since they are fixed, can we learn the factor functions from them? One more thing, am I correct that in your factor graph input files factor values are randomly assigned (i.e. not based on any domain knowledge of the variables)?

@nabeelxy those files are based on a real graph, that is, the degree distribution is based on a real graph. All nodes are represent the same variable, and three different priors are used. (A prior of 0,5 implies no previous knowledge about that node). The edges in the model are homogeneous (since there is only one kind of node), and hence share the same edge potentials. The potential shows a common case of homophily - similar state pairs have high values, and dissimilar ones have low values.

@nabeelxy We've added a version of the BP algorithm that uses file format you mentioned: vertex100.txt and edge100.txt. It is called PairwiseBP and there is an example in the README.


AWS Lex bot, intent, and slottypes can not use autogenerated names with hyphens

The generated name is not acceptable for use in AWS Lex when using the order_flower_bot example at https://www.pulumi.com/docs/reference/pkg/aws/lex/bot/:
Diagnostics:
  aws:lex:Bot (order_flower_bot):
    error: aws:lex/bot:Bot resource 'order_flower_bot' has a problem: expected value of name to match regular expression "^([A-Za-z]_?)+$", got order_flower_bot-d885833

This also applies to intents and slottypes (I haven't checked other resources).

Should we keep auto naming but remove dashes? (Separator: "")


prediction difference between batch=1 and batch=16

Any ideas why I'm receiving different prediction values when running with batch_size=1,16?
find code below:
Thanks!
def predict(self,speech_input):
labels = np.empty(0, int)
labels = np.append(labels, range(speech_input.shape[0]), axis=0)
feature,logits,_ = self.session.run(
[self.features,self.logits,self.end_points_speech],
feed_dict={self.is_training: False, self.batch_dynamic: labels.shape[0],
self.margin_imp_tensor: 50,
self.batch_speech: speech_input})
#self.batch_labels: labels.reshape([labels.shape[0], 1])})
    # Extracting the associated numpy array.
    #print (feature[0])

    return  feature,logits

Are you running on your own dataset?

Yes
On Sun, May 13, 2018, 20:00 Amirsina Torfi<EMAIL_ADDRESS>wrote:

Are you running on your own dataset?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/astorfi/3D-convolutional-speaker-recognition/issues/16#issuecomment-388641016,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE_DSc68BlIm6FqR0HSmh25ZDkJCOEB9ks5tyGa3gaJpZM4T02CP
.

I do not get your question actually. Why are you expecting to get the same prediction values?

Because I'm in inference stage after I finished the training stage
look this pseudo code:
for i in batch16:
yi=model.predict(xi)
and I compare it to:
y=model.predict(batch16)
and this 16 length vector y look different...
and the results look different
On Sun, May 13, 2018, 20:09 Amirsina Torfi<EMAIL_ADDRESS>wrote:

I do not get your question actually. Why are you expecting to get the same
prediction values?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/astorfi/3D-convolutional-speaker-recognition/issues/16#issuecomment-388641585,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE_DSUtaa0xtwPCiiTDfms0D4OrUTynJks5tyGjcgaJpZM4T02CP
.

What do you mean by batch16?

Batch16 is a (16,80,40,20) tensor a opposed to yi which is (1,80,40,20)
signle example
On Mon, May 14, 2018, 04:23 Amirsina Torfi<EMAIL_ADDRESS>wrote:

What do you mean by batch16?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/astorfi/3D-convolutional-speaker-recognition/issues/16#issuecomment-388672100,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE_DSTwBRBfnPg8gVq0csGjtDKI5tuUyks5tyNybgaJpZM4T02CP
.

I think you should debug the values more carefully. It's weird. It's hard for me to have an idea about the details and the values that you are getting. Please see if the averaging is performing correctly. This might be due to the drastic changes happened in new TensorFlow versions.

Please open this issue if the problem has not been resolved!
Thanks


manual reload after uninstall

I have removed the plugin from the project, but the project still requires manual reload with Ctrl+R each time the change is made to the script. More so, the same is applied to other projects, without the plugin being imported at all.
How to fix this?

You can re-enable Unity auto reload in settings. Search for auto-refresh and change back to 'always'


Update google-auth-library-oauth2-http to 1.12.1

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

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.auth", artifactId = "google-auth-library-oauth2-http" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.google.auth", artifactId = "google-auth-library-oauth2-http" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #45.


I am using specflow, Easy Repro, Selenium and C#, facing issue while closing app,

I am using Specflow, Easy Repro, Selenium and C#, facing issue while closing app,
To createapp I am using the following method.
public static void AdfsLoginAction(LoginRedirectEventArgs args)
{
args.Driver.FindElement(By.Name("passwd")).SendKeys(args.Password.ToUnsecureString());  //Input webelemnt has been changed from Password to Passwd
args.Driver.ClickWhenAvailable(By.Id("idSIButton9"), new TimeSpan(0, 0, 2));
Thread.Sleep(500);
args.Driver.WaitForPageToLoad();
if (args.Driver.HasElement(By.Id("idSIButton9")))
{
args.Driver.ClickWhenAvailable(By.Id("idSIButton9"));
}
}
    public static void CreateApp(BrowserOptions options = null)
    {
        Trace.WriteLine($"Start ({CrmUrl})");
        options = options ?? TestSettings.Options;
        SetOptions(options);
        _client = new WebClient(options);
        _xrmApp = new XrmApp(_client);
        _xrmApp.OnlineLogin.Login(CrmUrl, Username1, Password2, null, AdfsLoginAction);
        Trace.WriteLine("Success");
       // return _xrmApp;
    }

where I see the driver is getting invoked through LoginRedirectEventArgs, after the test execution how to close the browser?
Can anyone please help.
Thanks,
Chandra.

@chandramohand
You can close the browser AfterScenario using the following dispose method.
xrmApp.Dispose();
Hope that helps

Hi @RaviYenumula, Thanks for your reply.
The issue has been resolved while initialling the webdriver xrmApp declared as Private, I was unable to quit that browser.
Once declared as public its working fine.
Thanks,
Chandra.

@chandramohand  Glad it sorted out.


regression: X connection to :0 broken (explicit kill or server shutdown).

Which operating system does the issue occur on?
Linux Debian 11 Bullseye
If on linux, are you using X11 or Wayland?
X11
When I try to maximize the window (by double clicking on the title bar or pressing maximization shortcut) alacritty crashes with: X connection to :0 broken (explicit kill or server shutdown) error.
I never experienced that before. Currently doing a git-bissect to figure out which commit would have introduced this but some previously working ones aren't working anymore. Would it be possible a dependency causes that crash?
How can I further help to debug?
$ target/release/alacritty -v
Created log file at "/tmp/Alacritty-140443.log"
[2019-11-22 13:27] [INFO] Welcome to Alacritty
[2019-11-22 13:27] [INFO] Configuration loaded from "/home/dv/.config/alacritty/alacritty.yml"
libEGL warning: DRI3: Screen seems not DRI3 capable
libEGL warning: DRI2: failed to authenticate
[2019-11-22 13:27] [INFO] Device pixel ratio: 2.399993896484375
[2019-11-22 13:27] [INFO] Initializing glyph cache...
[2019-11-22 13:27] [INFO] ... finished initializing glyph cache in 0.03561759s
[2019-11-22 13:27] [INFO] Estimated DPR correctly, skipping resize
[2019-11-22 13:27] [INFO] Cell Size: 28 x 54
[2019-11-22 13:27] [INFO] Padding: 0 x 0
[2019-11-22 13:27] [INFO] Width: 3836, Height: 1080
[2019-11-22 13:27] [INFO] PTY Dimensions: Line(20) x Column(137)
[2019-11-22 13:27] [INFO] Initialisation complete
[2019-11-22 13:27] [INFO] Width: 3836, Height: 1080
# Trying to maximize the window
X connection to :0 broken (explicit kill or server shutdown). # → instance crashes and window closes

$ git log -n1 --oneline
5ea4fb6 (HEAD, origin/master, origin/HEAD) Fix paste on Wayland inserting empty lines

No, you have a problems with video drivers, so please address these issues first. You are likely need a mesa-dri or something like this package if you're on an opensource driver.

Will do 🙂
Closing for now.


Feature/support rxxyyzz gate

Do we want this in the release?

Do we want this in the release?

Yes, if we are not in a hurry and this PR doesn't require too much change.

Yes we can do that. I would suggest that we merge this directly into the release branch as well?


DBZ-8275 Support for PostgreSQL 17

Taking inspiration from c32a5cff3294ff4a8bbc905298d41efd77128fe9, I've added files for postgres v17. It might be possible to use Debian bookworm instead of bullseye as base image, but that's maybe for another discussion.

Hi @dargmuesli, can you please file a Jira issue and prepend the Jira issue key to the commit message? Thanks!

There is https://issues.redhat.com/browse/DBZ-8275 but additional changes are needed. See description

Thank you for linking, I updated the PR and commit title. Not sure where to make the other changes, likely another repo, I suppose. Also I'm not into decoderbufs, so I cannot help there, I think. Please take this PR as a first step into PostgreSQL v17 support and feel free to build upon it :)

@jpechane ah, should build-all.sh haven been updates as well maybe? As I don't see 17-alpine on docker hub right now, just 17.

Oh, i just saw

https://github.com/debezium/debezium/pull/5937

So I switched to quay.io :+1:


Project opensource config

In this PR I update the project to include the required opensource configurations.
Included:

[x] Code of conduct
[x] Contributing guide
[x] Corrected Licence
[x] Coveralls to tests
[x] Coveralls report to CI for Pull Requests
[x] OpenSSF scorecards badge(wont be available until the project is public)
[ ] OpenSSD best practices badge will require to have the project public, and the gem publised to fill the respective form. To be tackled when project is ready to be published.
[x] Licence badge
[x] Build Badge
[ ] Rubgems downloads badge.
[ ] Rubygem version badge
[x] Basic information on README file.

Missing elements from the PR list, to be finished on a new one, after publishing the gem with a release


Removed files in src\cmd\vendor\golang.org\x\sys\unix\ required for port on new os

Go 1.16.7
Removed files in src\cmd\vendor\golang.org\x \sys\unix\ required for port on new os. These files are used in mkall.sh.
https://go-review.googlesource.com/c/go/+/172979
src/cmd/vendor/golang.org/x/sys/unix/mkasm_darwin.go Deleted
src/cmd/vendor/golang.org/x/sys/unix/mkpost.go Deleted
src/cmd/vendor/golang.org/x/sys/unix/mksyscall.go Deleted
src/cmd/vendor/golang.org/x/sys/unix/mksyscall_aix_ppc.go Deleted
src/cmd/vendor/golang.org/x/sys/unix/mksyscall_aix_ppc64.go Deleted
src/cmd/vendor/golang.org/x/sys/unix/mksyscall_solaris.go Deleted
src/cmd/vendor/golang.org/x/sys/unix/mksysnum.go Deleted
src/cmd/vendor/golang.org/x/sys/unix/types_aix.go Deleted
src/cmd/vendor/golang.org/x/sys/unix/types_darwin.go Deleted
src/cmd/vendor/golang.org/x/sys/unix/types_dragonfly.go Deleted
src/cmd/vendor/golang.org/x/sys/unix/types_freebsd.go Deleted
src/cmd/vendor/golang.org/x/sys/unix/types_netbsd.go Deleted
src/cmd/vendor/golang.org/x/sys/unix/types_openbsd.go Deleted
src/cmd/vendor/golang.org/x/sys/unix/types_solaris.go Deleted
Removed files in
SHA-1: d68ac59113e8c6d6a79abf9d608fd24c3566b91c

{,cmd/}vendor: rerun 'go mod vendor' to prune ignored files
Updates #31088
Change-Id: Ia126e4e83ac5cb12c2c4151d5e5c975497598f24
Reviewed-on: https://go-review.googlesource.com/c/go/+/172979
Reviewed-by: Brad Fitzpatrick<EMAIL_ADDRESS>

You'd want to work on the golang.org/x/sys repo directly, instead of the vendored copy in GOROOT.

You do not understand the essence of the questions. Take a close look at src\cmd\vendor\golang.org\x \sys\unix\mkall.sh.
There is a reference to these files, but they are not. Without them, I cannot create port goland to new architecture. These files are remembered in readmi.md in the  src/cmd/vendor/golang.org/x/sys/unix/ folder.

Take a close look at src\cmd\vendor\golang.org\x \sys\unix\mkall.sh.

You should not use this file at all. Use the one in golang.org/x/sys repository ( https://pkg.go.dev/golang.org/x/sys ) instead. After updating the golang.org/x/sys repository, vendor it into GOROOT.

cmd/vendor/golang.org/x/sys/unix is a copy of the repository hosted at golang.org/x/sys/unix.  All changes should be made to golang.org/x/sys/unix.  After the changes are made there, they can be copied into cmd/vendor/golang.org/x/sys/unix using go get and go mod vendor.  There is no reason for the files you mention to exist in cmd/vendor/golang.org/x/sys/unix.  They are only required in golang.org/x/sys/unix.


variable substitution in translated strings; can variables in "foo bar {0} and {1}" be substituted with "baz" and "poo" simply by doing $translate("my.key.to.string","baz","poo");

Subject of the issue
Describe your issue here.
Your environment

version of angular-translate
version of angular
which browser and its version

Steps to reproduce
Tell us how to reproduce this issue. Please provide a working demo, you can use this template as a base.
Expected behaviour
Tell us what should happen
Actual behaviour
Tell us what happens instead

@sandeep-instinct Well, please read your post again. It looks like you have mixed up something with title and the actual text.
So, I have a little problem identifying our actual problem. Yes, you can use translations with placeholders.

@knalli Wonderful. I found what I wanted and I think this issue can be closed. This was what I was looking for.
cheers!


Bug: Passing Line Chart null value(s) hides entire 'label' tooltip

In 1.0, if you passed multiple data-points to a Line Chart but included a null value, a label tooltip would still display if any other numeric data-point was hovered (simply removing the null data-point from the multi-tooltip).
In 2.0, one or more null values hides the entire label tooltip, even if there are one or more numeric values and one of those data-points is hovered.
I already have a fix for this on my local project (checking the meta data for the skip flag before adding an element to the returned getElementsAtEvent() array), but wanted to make sure this wasn't intended functionality before I made a PR.
Thanks!

@adamtLICOR I do not think this is intended behaviour. In my mind we should either eliminate the row represented by the null data or show the row with nothing for the value.

Well, in my particular use-case, removing the row is desirable.
This chart has four datasets -- when all four are present, the tooltip displays as normal.

If a dataset has a null value, it just doesn't show it in the tooltip (or technically 'select' it from the hover state, but that's a moot point since it's invisible)

If this is sufficient, I'll make another PR when I get the chance. My fix only works for label and the newly added x-axis tooltip options. I looked into fixing dataset briefly, but wasn't able, as it's significantly different from the first two options (and it's required by my particular use-case).

I support removing the row. I think in most (>90%) of cases this is what users will expect. Please PR these changes :smile:

How's the progress on this issue?

There's a PR open for it right now, actually. Not sure what else (if anything) needs to be done

@adamtLICOR @Regaddi merged the PR

Is this one only solves line chart? I have same issue for bar chart with v2.3.0


Wrong run-time type of the value returned by generator function

Why run-time type of the variable from the below is Future?
import "dart:async";

FutureOr<Stream<int>?> f1() async* {
  yield 1;
  yield 2;
  yield 3;
}

main() async {
  Stream<int>? res1 = await f1();
  List<int>? it1 = await res1?.toList();
  print(it1 is List<int>?); // Analyzer: Unnecessary type check; the result is always 'true'.
                            // Prints `false`
  print(it1.runtimeType);   // Future<List<dynamic>>
}

Ok, the fact that type of it1 is List<dynamic> instead of List<int> is a known issue described here. But why it is Future?
Note that if to remove nullability issue connected with the Future disappers
import "dart:async";

FutureOr<Stream<int>> f1() async* {
  yield 1;
  yield 2;
  yield 3;
}

main() async {
  Stream<int> res1 = await f1();
  List<int> it1 = await res1.toList();
  print(it1 is List<int>); // Analyzer: Unnecessary type check; the result is always 'true'.
                            // Prints `false`
  print(it1.runtimeType);   // List<dynamic>
}

Tested on Dart SDK version: 3.1.0-333.0.dev (dev) (Thu Jul 20 13:04:10 2023 -0700) on "windows_x64"
cc @eernstg @lrhn

This is the bug which was reported here, which gave rise to https://github.com/dart-lang/language/pull/3218.
With the old rules, f1 would have element type dynamic, so it returns a Stream<dynamic>, but the static type is FutureOr<Stream<int>?> (so this is already unsound). With the current rules it would have element type int, so it should return a Stream<int>, but that hasn't been implemented yet.
Next, we get the toList() and that turns out to be a Future<List<dynamic>>, with static type Future<List<int>?>. So the static type of the await is List<int>?, but the dynamic type is Future<List<dynamic>> (with the correct semantics of await) or List<dynamic> (if the runtime doesn't implement the required dynamic type check, cf. https://github.com/dart-lang/sdk/issues/50601). With DartPad we get the latter, so it prints JsArray<dynamic>.
Presumably you ran the experiment using dart on the command line (or some other non-dart2js runtime), so you got Future<List<dynamic>>.
So it's all working or failing as expected, considering the known issues.
The second example can be explained along the same lines, but in this case there is no doubt: We get a List<dynamic> at the end, because the type check is skipped (we don't check whether it's actually a Future<List<int>>, so we don't discover that it is actually not, it is a Future<List<dynamic>>, all because the program is unsound and it is only OK to skip that type test when the program is sound).

I tested command-line dart on Windows and Linux (freshly built) with no any experimental flags. Both give Future<List<dynamic>> as it1.runtimeType in the first example.
To clarify. We have an known issue here the List type is dynamic instead of int (if we don't have a SDK issue for that I'll file the new one later).
But if the fact, that variable declared as List<dynamic>? (let's have List<dynamic>? instead of List<int>? to separate the issues) has a run-time type Future<List<dynamic>> is an issue or not?

Both give Future<List<dynamic>> as it1.runtimeType in the first example.

Yes, that's because they are running the VM, and the VM has a fix for https://github.com/dart-lang/sdk/issues/49396, but dart2js does not (and that's the reason why https://github.com/dart-lang/sdk/issues/50601 is still open, at least for a couple of days yet ;-).
So the VM behavior is correct here, and the dart2js behavior is not. I think DDC has the fix as well.
But none of the tools have a fix (yet) for the generator element type bug (that was a spec bug), which is the reason why we get a Stream<dynamic> in the first place. That's also the reason why we get a Future<List<dynamic>?> from toList() later on, and then a List<dynamic>.
So we have one issue which is #49396, which is just about to be completely fixed. Next, we have one fault which is about the element type of a generator, and there is no github issue for that, yet (and the spec change just landed). I just created https://github.com/dart-lang/sdk/issues/53052 in order to address the updates needed now that the spec bug has been fixed.

Ok, thak you for clarification. Closing this issue then


Update ember-cli to 2.5.0

LGTM

@homu r+

:pushpin: Commit 6c1ce8d has been approved by poteto

:hourglass: Testing commit 6c1ce8d with merge bd20693...

:sunny: Test successful - status


Add sensirion scd30 v2

Add support for Carbon Dioxide (CO2) sensor types, and add support for the Sensirion SCD30 CO2 sensor

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 3 committers have signed the CLA.:x: winkj:x: andy2012zwj:x: Cheng-SGYou have signed the CLA already but the status is still pending? Let us recheck it.

    感谢您提出的宝贵问题，我们有7x24小时服务的“HaaS百事通”客服系统，
    您可以先尝试能否解决您的问题(https://haas.iot.aliyun.com/?ask=1&f=a2cre.b82925042)
    如果“HaaS百事通”没有解决您的问题，我们会在两个工作日回复您。


Put take too long time

➜  etcd ./etcd --version
etcd Version: 3.1.0-rc.1
Git SHA: 3d5ba43
Go Version: go1.7.3
Go OS/Arch: linux/amd64
➜  etcd ETCDCTL_API=3 ./etcdctl put /dlocker/aaa 1
when i run the etcdctl many times, there will be log this, and  i use clientv3 also have the problem.
2016-12-09 16:03:21.925822 W | etcdserver: apply entries took too long [202.238741ms for 1 entries]
2016-12-09 16:03:21.925846 W | etcdserver: avoid queries with large range/delete range!

Does etcd run on HDD?

yes, so it must run on SSD when using etcd v3

@johnnyluo586 https://github.com/coreos/etcd/blob/master/Documentation/faq.md#what-does-the-etcd-warning-apply-entries-took-too-long-mean should provide enough information on this question.


[BUG] DataLakeLeaseClient releaseLease is not changing the lease status

Describe the bug
After writing a file in datalake, I was calling the DataLakeLeaseClient's releaseLease method for releasing the acquired lease. After the release lease operation, the lease state is correctly updated as  'Available' but the  'Lease status' property of the file is showing as 'Locked'.
Expected behavior
After the releaseLease operation, the Lease Status property should be updated to Unlocked state.
Setup (please complete the following information):
Dependency version - azure-storage-file-datalake 12.1.0

Hi, @Anusarani. Thank you for reporting this issue. We'll do some work to verify this behavior, but I suspect this might actually be a service bug. In any case, lease state is the recommended way of working with leases at the moment, and lease status is mostly kept around for backwards compatibility, so I hope you aren't blocked on this?

We are not blocked due to this issue. We are able to perform read, delete and write operation on this file even if the lease status is shown as Locked. But not sure if any other datalake operations will be impacted due to this lease status. Can you please confirm?

We have confirmed the behavior you are seeing, and it does look like a service issue. We will engage with the appropriate teams and ensure this behavior does not negatively impact any other apis.

I am going to close this issue as it should be fixed for free when the service fixes its bug and therefore there is no active work for the client here.


Merging  outside of symbols ?

Hi! Can anybody tell me, what's the right way to use one defs to a several symbols (linear-gradient for example) ? When I'm add symbols to sprite, each symbol has its own defs, can I move and merge defs outside of symbols for multiple use ? I understand, that I can use it multiple through css, if it is in one symbol only, but I think, this is not usable to some tasks.

@alx-gk do you have any problems with multiple defs or just curious?

I think its not problem, but, for example, if I need two svg symbols with same linear-gradient, now I can create defs with gradient in each symbol ( but dont want to duplicate code ), or create defs in first symbol and use it for second symbol too ( symbols will be not independent ). I think the right way will be create one defs outside of symbols ( merge it from symbols with same gradient ) and use it for all necessary symbols ?

Actually it's not a loader issue, it is related to sprite optimization and minification, svgo for example. However, such optimizations in automatic way is rather dangerous and I don't think they are necessary.

@alx-gk svg-sprite-loader uses svg-baker for creating sprites under the hood, you can file an issue there.

Ok, thanks! Will read svgo documentation once more.


Unexpected behaviour in vector-valued custom_jvps

Description
I was running the example in custom derivatives. If we follow the same procedure for a function that outputs multiple values, it would be like this:
# For multi-output functions
import jax.numpy as jnp
from jax import custom_jvp

@custom_jvp
def f(x, y):
  return jnp.sin(x) * y, x*y, x**2

@f.defjvp
def f_jvp(primals, tangents):
  x, y = primals
  x_dot, y_dot = tangents
  primal_out = f(x, y)
  tangent_out1 = jnp.cos(x) * x_dot * y + jnp.sin(x) * y_dot
  tangent_out2 = x*1.0*y_dot + y * 1.0 * x_dot # Total differentail w.r.t inp = (x, y)
  tangent_out3 = 2*x*x_dot + 0
  return primal_out, (tangent_out1, tangent_out2, tangent_out3)

print(f(2., 3.))
val, x_dot = jvp(f, (2., 3.), (1., 0.)) # setting v = (1, 0) => we are only interested in x_dot
print(val)
print(x_dot)

This results in the following output:
(Array(2.7278922, dtype=float32, weak_type=True), 6.0, 4.0)
(Array(2.7278922, dtype=float32, weak_type=True), 6.0, 4.0)
(Array(-1.2484405, dtype=float32, weak_type=True), 3.0, 4.0)

However, if I run the same function without the custom_jvp decorator, the result is as follows:
(Array(2.7278922, dtype=float32, weak_type=True), 6.0, 4.0)
(Array(2.7278922, dtype=float32, weak_type=True), Array(6., dtype=float32, weak_type=True), Array(4., dtype=float32, weak_type=True))
(Array(-1.2484405, dtype=float32, weak_type=True), Array(3., dtype=float32, weak_type=True), Array(4., dtype=float32, weak_type=True))

The returns are all arrays while in the former case, we get one array and 2 scalars. Is this the expected behaviour ?
What jax/jaxlib version are you using?
JAX - 0.4.12
Which accelerator(s) are you using?
CPU
Additional system info
No response
NVIDIA GPU info
No response

I think this is expected. JAX generally isn't too careful about trying to distinguish JAX arrays from NumPy arrays from Python builtin bool/int/float/complex. In this case some of these arraylikes are being turned into JAX arrays via the custom_jvp machinery.
This is actually as-designed behaviour, in that it makes it relatively easy to interop between these three sets of arraylike objects, without needing to carefully track what's what.
If this bothers you, then wrap all inputs into JAX arrays (jax.numpy.asarray(x)) before calling your JAX program, and you should have consistent behaviour.


encode go errors as json in api handler

can you put an encodeErr in a defer in https://github.com/keybase/client/blob/master/go/client/chat_api_version_handler.go#L22-L29? also for the wallet and team api handlers


MGDOBR-914: Unable to list Processors for a FAILED Bridge

https://issues.redhat.com/browse/MGDOBR-914
PR owner checklist
Please make sure that your PR meets the following requirements:

[x] Your code is properly formatted according to this configuration.
Run mvn clean verify -DskipTests so that imports are correctly set.
[x] Your commit messages are clear and reference the JIRA issue e.g: "[MGDOBR-1] - $clear_explanation_of_what_you_did"
[x] The layers in the kustomize folder have been updated accordingly.
[x] All new functionality is tested
[x] Pull Request title is properly formatted: MGDOBR-XYZ Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains link to any dependent or related Pull Request
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket

PR reviewer(s) checklist

[ ] Check if code and Github action workflows have been modified and if the modification is safe
[ ] If the modification is safe, add the safe to test label

How to trigger pipelines and use the bots:

Run the end to end pipeline
Annotate the pull request with the label: safe to test. If you want to run the pipeline again, remove and add it again.

Rebase the pull request
Comment with: /rebase.

Deploy BOT
Comment with `/deploy <dev,stable> when the PR has been merged to deploy to a target environment.

@manstis it is minor thing, but I think bridge in failed state is not actionable, rather delete-able? in other warods, what are actions user can do with bridge in failed state?

Hi @jomarko various actions can be taken against a Bridge instance.

Delete it
Update it (not yet implemented)
List its Processor's
All of these should be possible if the Bridge is either READY or FAILED.

@lucamolteni @afalhambra Could you please review.


Inference failure, (Error 54) (?u1624) _ is not equal to the expected type

I ran into this error a few times already in different (big) contexts, but here's a standalone repro. Since the LHS is a uvar, my intuition is that we're creating it on the wrong context.
noeq
type mydep a b =
    | Mk : x:a -> y:(b x) -> mydep a b

TestSigmaProj.fst(6,23-6,24): (Error 54) (*?u1624*) _ is not equal to the expected type a (see also TestSigmaProj.fst(6,11-6,12))

Annotating (b : a -> Type) fixes it. Perhaps the uvar for b's type does not have a in scope? Is that expected? That rings a bell, from #291 and #1074, but it'd be nice to confirm (also given that #291 has changed a bit).

This works now, and my guess is https://github.com/FStarLang/FStar/pull/2473. Closing.


#144937469 czars should draw card by clicking

What does this PR do?

CZARs should be able to draw cards by clicking

Description of Task to be completed?

When a user is assigned the CZAR, a modal pops up and they are prompted to click to draw the card as opposed to just randomly selecting the card for the CZAR

How should this be manually tested?

N/A

Any background context you want to provide?

N/A

What are the relevant pivotal tracker stories?

#144937469 CZARs should be able to draw cards by clicking

Screenshots (if appropriate)

Questions:

Changes Unknown when pulling 4cdc3a3742ae65b00e423db8bfa6623e46c4a4bf on feature/144937469/czar-draw-card-by-clicking into ** on master**.

Changes Unknown when pulling d3c4b9aa5ec881f741e7dcb068225a1d3e9607a6 on feature/144937469/czar-draw-card-by-clicking into ** on master**.


SPEC: use the "tag" label instead of the "version" label

I was thinking on this tonight and I keep going back and forth. Putting
this up and sleeping on it.

Explain your rationale in a little more detail?

The rational is that people are familiar with using the term "tag" from git. However, it makes saying it outloud really annoying though: "I have the tag label set to version 1.0.0". I am going to just close this.


Fix deleting namespaces after running the e2e tests

Oops sorry about that, I have introduced this environment variable to allow keep
namespaces after running the e2e when we need to debug but the logic was wrong
and they were never actually deleted (which could be an issue on small memory
clusters)

Changes

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[ ] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
Describe any user facing changes here, or delete this block.

Examples of user facing changes:
- API changes
- Bug fixes
- Any changes in behavior

Closing this since I have submitted from my master 🙄


To go ds sslkeys generate

Branched from #2365 - recommend merging that first.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1791/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1809/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1851/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/1965/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2015/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2049/
Test PASSed.


Fixed Broken Image Links

Writer's Quality Checklist
Before merging your PR, did you:

[ ] Run spell checker
[ ] Run link checker to check for broken xrefs
[ ] Check for orphan files
[ ] Perform a local build and do a final visual check of your content, including checking for:

Broken images
Dead links
Correct rendering of partials if they are used in your content
Formatting issues, such as:

Misnumbered ordered lists (steps) or incorrectly nested unordered lists
Messed up tables
Proper indentation
Correct header levels

[ ] Receive final review and signoff from:

Technical SME
Product Manager
Editor or peer reviewer
Reporter, if this content is in response to a reported issue (internal or external feedback)

[ ] If applicable, verify that the software actually got released

Hey @alecdesjardins, thanks for contributing to our docs. It seems to me that the images are rendering properly on the docs site. Are you experiencing otherwise?
https://docs.mulesoft.com/db-connector/latest/database-connector-query-single#create-the-table-in-the-database

Hi @sathya0, yes they are working on there. Why do the GitHub docs not work for images?

Thanks @alecdesjardins for confirming. The pictures don't render on GitHub, we don't use GitHub to view the docs. We use GitHub to author and receive contributions for the docs, but to view the docs you should use the docs site for the best experience.
Since this is not an issue, I will close this PR. We appreciate you looking out for our docs.


Use full path for submodules

Full submodule paths should be used throughout the repo (e.g. from science_jubilee.tools.Tool import Tool instead of from tools.Tool import Tool)

Fixed in PR #25


Import data workflow docs - improve references to init_instance

Problem
On the 'Import data workflow' section in the docs, there are references to init_instance that are not clickable.
Solution
The PR adds a doc string to Resource.init_instance to make the references clickable.

Coverage remained the same at 96.134% when pulling bd7c229a466dce6398862dc058ac0e422a314c15 on kjpc-tech:docs-init-instance-link into d218fb5434cededb3f747a3cb2358d63b3475b6b on django-import-export:master.

Coverage remained the same at 96.134% when pulling bd7c229a466dce6398862dc058ac0e422a314c15 on kjpc-tech:docs-init-instance-link into d218fb5434cededb3f747a3cb2358d63b3475b6b on django-import-export:master.

Thanks!


Add vendoring of depenendent packages for specific commits at releases

It would be nice to add the ability to lock RR releases for dependent packages and tie the glide.yaml file to specific commits, preferably tied to other supported releases.

Hi @clintonskitson,
This will not be automated as it involves checking code back into the system as part of an automated build process. However, it's easy to accomplish by doing glide guess to get all of the project's dependencies. Place those in glide.yaml and then do a glide up to download them. Finally glide pin produces the glide.yaml output that contains all current reference IDs so you can lock things down.
This is handled for the 0.2.0 release in commit 9012ae3bbc5311.


Create LICENSE.md

Lägg gärna in denna, Pappa!

Pleas check that this licens is comatible with the licens mentioned in the protocol file!

The license provided by me is in compliance with the "protocol_lws_minimal.c" since the file license is marked as public domain (creative commons No Rights Reserved). "/*

ws protocol handler plugin for "lws-minimal"

Written in 2010-2019 by Andy Green<EMAIL_ADDRESS>
This file is made available under the Creative Commons CC0 1.0
Universal Public Domain Dedication.

This version holds a single message at a time, which may be lost if a new
message comes.  See the minimal-ws-server-ring sample for the same thing
but using an lws_ring ringbuffer to hold up to 8 messages at a time.
*/"

Great


Revert "Add automatic deploy to PyPI workflow"

Reverts equinor/configsuite-tui#17

Do you know how to fix it? It looked at lot cleaner then the configsuite one, but if it does not work perhaps that is an alternative?
Anyway, if you know how to fix it I think it is better to patch it then to revert it :)

Yes I think I do, will patch with new pull request.


[C++] Add secure option to the csm observability client, needed for calling a gRPC service on Cloud run.

release_notes: yes

I was advised to not use the Observability example for service-mesh userguide for cloudrun. Dropping this PR.


Exercise image_picker plugin

Exercise image_picker plugin

Closing this ticket to allow appropriate tickets to be opened and managed by the plugin team.


pretrained models with subsampling_rate=1

WeNet pretrained models with subsampling rate of 1 would be very useful for other speech tasks, for example, voice conversion. Yes we can still do VC with subsampling rate of 4, but there are other cases where subsampling rate of 1 is required.

There is no such plan, you can just train your own model with subsampling rate 1 on WeNet.


hotfix version of s3 update

Description
IMDSv2 support still seems broken despite a previous library update.
AWS JDK versions move fast so updating, also found a source that claims that the only reason we're accessing instance metadata is to get the region for the tooltester bucket which we already know, so hard coding is worth a try
Issue
https://github.com/dockstore/dockstore/issues/4359
See https://github.com/dockstore/dockstore/pull/4575 for develop counterpart
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that you pass the basic style checks and unit tests by running mvn clean install
[x] Follow the existing JPA patterns for queries, using named parameters, to avoid SQL injection
[x] Check the Snyk dashboard to ensure you are not introducing new high/critical vulnerabilities
[x] Assume that inputs to the API can be malicious, and sanitize and/or check for Denial of Service type values, e.g., massive sizes
[x] Do not serve user-uploaded binary images through the Dockstore API
[x] Ensure that endpoints that only allow privileged access enforce that with the @RolesAllowed annotation
[x] Do not create cookies, although this may change in the future

SonarCloud just seems bugged

Nice find, but confused, I could have sworn it worked.

Yeah, I find it weird too. But worth a try in develop first at least

(block until https://github.com/dockstore/dockstore/pull/4575 is reviewed on dev)


[DXCC Award] Started work on adding orbit

Added sat and orbit to selection in the GUI.
Queries need adjustment (joins).

* When clicking to SAT, SAT and orbit appears. When going back to "All" (which is my default) they won't disappear (Notice: At first loading with "All" there's no Orbit/SAT)

Fixed.


Fixed songID

Previously:
var songID = response.data
.split('"song":{"type":"')[1]
.split('","image":')[0]
.split('"')[4];
Now:
var songID = response.data
.split('"song":{"type":"')[1]
.split('","image":')[0]
.split('"')[8];

Btw all time you will not get song id from
.split('"song":{"type":"')[1]
.split('","image":')[0]
.split('"')[8];
you have to check its pid also . here is a referance code for you https://github.com/cachecleanerjeet/jiosaavnapi/blob/dfbce23c419112991bce905a13834a8546efc8ae/index.js#L189


Maps: Empty Dropdowns in "NFI: Volume by Production region and Stage of stand development "

Is your enhancement related to a problem? Please describe.
When I select the "NFI: Volume by Production region and Stage of stand development " and try to set the Interactive Filters for Evaluation Type and Inventory, per default the Dropdown seems to be empty. Then when I click on one of them I see its content.
I assume thats because nothing is in the data.
Describe the solution you'd like
Would it be possible that, when there is nothing in the date to add either "None" or "-" or something equivalent, so it doesn't look like an error?
Describe alternatives you've considered
None
Additi

onal context

As we recently added cascading filters behavior to interactive filters, this should no longer be the case.


Request Spanish translations for new test result copy

Background
There is new copy for the test results that did not have a Spanish translation from our CDC contact. The current, temporary translations in-app were provided by @johanna-skylight (Thank you!) We should reach out to our CDC contact to double check the translations.

Test result: COVID-19

For more information, please visit the Centers for Disease Control and Prevention (CDC) website or contact your local health department.

For flu A and B:

emailed on 06/08/2022

Translations received!
For translation to Spanish - 6.9.22-es.docx


Is this testable?

Hi. Is there a way to test this with Robolectric? Or with any test framework for that matter. I haven't been able to by just throwing an exception in the test method or by posting it to main looper.
BTW, I'm targeting and running in API 17 and MultiDex works :)
Thank you for your time.

Hi there, sorry for taking so long to reply. I don't have any experience with Robolectric nor testing in general, but I don't think it's easy to test this (at least, as a unit test). When a crash occurs, the OS kills the current process completely and a new process is launched (which shows CustomActivityOnCrash error activity when launched).


Docs: Wrong operator format

Based on the docs and mainly my experience with Yii, I believe the "and" operator format here is wrong. The condition parts which the "and" separates have to be each in an array, otherwise the where() doesn't know where the condition starts and where ends.

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Fixed issues
-

Thank you!


Add Pico Lipo SHIM example

I've just added Python linting to the examples, to make everyone's life that little more fun- and because Thonny seems to like to spew whitespace all over everything.
Could you fix the following please:
> flake8 micropython/examples/ --ignore E501
micropython/examples/pico_lipo_shim/battery.py:1:197: W291 trailing whitespace
micropython/examples/pico_lipo_shim/battery.py:2:76: W291 trailing whitespace
micropython/examples/pico_lipo_shim/battery.py:5:1: F401 'machine.ADC' imported but unused
micropython/examples/pico_lipo_shim/battery.py:14:8: F821 undefined name 'machine'
micropython/examples/pico_lipo_shim/battery.py:14:69: W291 trailing whitespace
micropython/examples/pico_lipo_shim/battery.py:15:12: F821 undefined name 'machine'
micropython/examples/pico_lipo_shim/battery.py:16:36: W291 trailing whitespace
micropython/examples/pico_lipo_shim/battery.py:24:24: E701 multiple statements on one line (colon)
micropython/examples/pico_lipo_shim/battery.py:25:1: W293 blank line contains whitespace
micropython/examples/pico_lipo_shim/battery.py:26:24: W291 trailing whitespace
micropython/examples/pico_lipo_shim/battery.py:36:1: W293 blank line contains whitespace
micropython/examples/pico_lipo_shim/battery.py:42:69: W291 trailing whitespace
micropython/examples/pico_lipo_shim/battery.py:46:19: W292 no newline at end of file

Sorry for moving the goalposts :grimacing:

LMK if I missed any of that pesky whitespace!

Looks like the tests passed, woohoo! Thank you.


Squeezelite player constantly goes to idle state

What version of Music Assistant has the issue?
2023.6.0b3
The problem
Squeezelite player goes to idle state after couple of seconds. It still continues to play and plays all the music from the queue, but the state changes to Idle after couple of seconds. If I do any operations on the player (like next song, increase / decrease volume, etc.) it moves back to state Playing and after couple of seconds moves back to state Idle again. This has an affect on my media player in Home Assistant as it doesn't show the correct song (the current song is shown as currently playing even though the player plays the net song in the queue). Same player normally works with LMS server and shows the correct status.
Squeezelite version is Squeezelite v1.9.9-1432, running on Raspberry Pi4 with DietPi v8.22.3.
How to reproduce
This happens constantly, don't know how to reproduce it.
Relevant log output
No errors in the log file that I could connect with this issue.
Additional information
No response
What version of Home Assistant Core are your running
2023.10.2
What type of installation are you running?
Home Assistant Container
On what type of hardware are you running?
Raspberry Pi

Please confirm you are looking at the MA entity in HA not a native HA one. I just tried this with Squeezelite running on Windows and it worked fine?

Yes, I am looking at MA entity. I also checked on the MA web interface directly as well and it shows the same behaviour. It starts and shows state 'Playing', after couple of seconds state switches to 'Idle'. If I do anything with the player, for example increase volume, it changes state to 'Playing' and then couple of seconds later, shows 'Idle' again.

Also, to make sure it's not caused by some networking issue, I installed shairport-sync on the same device where I run squeezelite and configured Airplay provider. I didn't have any issues with airplay provider, it never changes it's state to 'Idle' and works correctly.

Sounds like it might be a problem with that version of squeezelite then

I don't think that is the case since the same squeezelite client works normally with Logitech Media Server. It always shows correct state there.

All I could add to that is Marcel has in the past said LMS has a number of hacks within that to make things work that shouldnt according to the specs. What have you got this running on? If it is a Pi try running picoreplayer on it and see what happens.

I'll try with picoreplayer for testing, but I can't change my setup permanently. Unfortunately, I'll have to revert back to my old LMS setup which is too bad as I really like MA. Thanks for the help.

Yeah I have just tried picoreplayer and it also is working fine. So if yours isnt you could try different versions of whatever software you are using. Just confirm you have got the LMS shutdown on your network while trying to do this?

Im running out of ideas then. If picoreplayer works then defintiely points to something with the software you are using

Possibly. What version of picoreplayer you tested? I'll try to use the same one.

I just updated it to see if that made any difference. It is now on 1.9.9-1430 and working fine

How did you go with picoreplayer?

Yes, you were right. pCP works. Unfortunately, I was never able to make pCP work for me. I use FanShim to cool of my device and their python library just doesn't want to install due to some dependencies. But since my RPi4 sits in the closed cabinet with other equipment, it gets very hot, over 75C so I can't run it without fan. Anyhow, thank a lot for your help, this is clearly something with the squeezelite client.

Thanks.

Hi @OzGav,
opening this again as after upgrade of my Music Assistant server to 2.0.0b74, the same issue appears with piCorePlayer. Any ideas? I run pCP 8.2.0 with Squeezelite v1.9.9-1430-pCP.
Thanks

Where are you seeing the idle state? I see playing in the MA player list as well as the HA entity. Are you seeing idle everywhere?

I see idle state in both the HA entity as well as MA player in MA web interface. This started to happen after the upgrade for MA server. It worked before with pCP (but didn't work with my custom installation).

See if you can think of something else that is different with your setup. Maybe this is a networking issue? No one else has so far reported this type of issue.

Hi @OzGav, seems I found the issue but have no clue why is this causing squeezelite player to go to idle state. I was using internal stream and output buffer size switch (-b 1048576:1048576) for my squeezelite command. When I remove it, everything works ok and player correctly shows the status and never goes to idle state. I can reproduce the issue since it happens every time I start squeezelite player with -b switch.

Cool. Thanks for the fault finding. Hopefully this will help someone else in the future


init has no 'main' attribute

I am having issues getting the CLI to run on Ubuntu 14.04
$ wakatime
wakatime
Traceback (most recent call last):
  File "/usr/local/bin/wakatime", line 9, in <module>
    load_entry_point('wakatime==4.1.4', 'console_scripts', 'wakatime')()
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 351, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2363, in load_entry_point
    return ep.load()
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2093, in load
    raise ImportError("%r has no %r attribute" % (entry,attr))
ImportError: <module 'wakatime.__init__' from '/usr/local/lib/python2.7/dist-packages/wakatime/__init__.pyc'> has no 'main' attribute

$ python --version
python --version
Python 2.7.6

I installed using sudo pip install wakatime.
Any help would be much appreciated.

Sorry about that, renamed main to execute and forgot to update setup.py. It's fixed with v4.1.6 on pypi.

Thanks :)


False success reported when helm-diff has a network issue

We're experiencing an issue with 0.40.1 when helm-diff is run:
Comparing inf-rabbitmq stable/rabbitmq-ha
E1107 14:29:05.101635     301 portforward.go:316] error copying from local connection to remote stream: read tcp4 <IP_ADDRESS>:37167-><IP_ADDRESS>:53734: read: connection reset by peer
No affected releases
This has happened only sporadically so is probably due to transient network issues. But this should report as an error, and instead it shows as "No affected releases". This causes our deployment system to report success, when it was actually a failure.

@gorzek Thanks for the report! So you're running helmfile apply and it exits with a successful exit status even though the helm-diff command failed due to the transient error. Am I correct?
helm-diff --detailed-exit-code should just exit with an exit code other than 0 or 2 in that case. Assuming helmfile correctly handling exit codes, I doubt this is an issue in helm-diff.

I've seen this sort of error quite a few times on our internal K8s cluster, but just assumed it was something weird with our K8S deployment. Same error as in the description:
$ helmfile diff
...
E1108 09:28:42.217982     486 portforward.go:316] error copying from local connection to remote stream: read tcp4 <IP_ADDRESS>:42701-><IP_ADDRESS>:36704: read: connection reset by peer

E1108 09:28:47.789784     565 portforward.go:316] error copying from local connection to remote stream: read tcp4 <IP_ADDRESS>:38557-><IP_ADDRESS>:39020: read: connection reset by peer
...

$ echo $?
0

@mumoshu That is exactly right. I expect an appropriate exit code in a case like that, since there was clearly an error.
Looks like this has been reported to helm, as well: https://github.com/helm/helm/issues/3480


Beta Error: Invalid mapping during scss compilation

Hi,
I have a project that works with latest stable version of fuse-box 2.2.31 without any problem:
https://github.com/jezikk/fuse-box-foundation-starter

After updating to beta version 2.3.1-beta.19 (using fuse-box@beta), there are errors during SASS compilation:
Error: Invalid mapping: {"generated":{"line":4,"column":70397},"source":"/node_modules/foundation-sites/scss/components/_button.scss","original":{"line":150,"column":-8},"name":null}
    at SourceMapGenerator_validateMapping [as _validateMapping] (C:\Users\Michal\Documents\Projects\fusebox-foundation-starter\node_modules\source-map\lib\source-map-generator.js:289:13)
    at SourceMapGenerator_addMapping [as addMapping] (C:\Users\Michal\Documents\Projects\fusebox-foundation-starter\node_modules\source-map\lib\source-map-generator.js:101:12)
    at C:\Users\Michal\Documents\Projects\fusebox-foundation-starter\node_modules\concat-with-sourcemaps\index.js:60:28
    at Array.forEach (<anonymous>)
    at BasicSourceMapConsumer.SourceMapConsumer_eachMapping [as eachMapping] (C:\Users\Michal\Documents\Projects\fusebox-foundation-starter\node_modules\source-map\lib\source-map-consumer.js:155:14)

    at Concat.add (C:\Users\Michal\Documents\Projects\fusebox-foundation-starter\node_modules\concat-with-sourcemaps\index.js:58:18)
    at group.subFiles.forEach.file (C:\Users\Michal\Documents\Projects\fusebox-foundation-starter\node_modules\fuse-box\dist\commonjs\plugins\stylesheet\CSSplugin.js:51:20)
    at Array.forEach (<anonymous>)
    at CSSPluginClass.transformGroup (C:\Users\Michal\Documents\Projects\fusebox-foundation-starter\node_modules\fuse-box\dist\commonjs\plugins\stylesheet\CSSplugin.js:49:24)
    at promises.push.Promise (C:\Users\Michal\Documents\Projects\fusebox-foundation-starter\node_modules\fuse-box\dist\commonjs\core\ModuleCollection.js:168:59)

Error: Invalid mapping: {"generated":{"line":15,"column":70397},"source":"/node_modules/foundation-sites/scss/components/_button.scss","original":{"line":150,"column":-8},"name":null}
    at SourceMapGenerator_validateMapping [as _validateMapping] (C:\Users\Michal\Documents\Projects\fusebox-foundation-starter\node_modules\source-map\lib\source-map-generator.js:289:13)
    at SourceMapGenerator_addMapping [as addMapping] (C:\Users\Michal\Documents\Projects\fusebox-foundation-starter\node_modules\source-map\lib\source-map-generator.js:101:12)
    at C:\Users\Michal\Documents\Projects\fusebox-foundation-starter\node_modules\concat-with-sourcemaps\index.js:60:28
    at Array.forEach (<anonymous>)
    at BasicSourceMapConsumer.SourceMapConsumer_eachMapping [as eachMapping] (C:\Users\Michal\Documents\Projects\fusebox-foundation-starter\node_modules\source-map\lib\source-map-consumer.js:155:14)

    at Concat.add (C:\Users\Michal\Documents\Projects\fusebox-foundation-starter\node_modules\concat-with-sourcemaps\index.js:58:18)
    at BundleSource.addFile (C:\Users\Michal\Documents\Projects\fusebox-foundation-starter\node_modules\fuse-box\dist\commonjs\BundleSource.js:66:31)
    at files.forEach.f (C:\Users\Michal\Documents\Projects\fusebox-foundation-starter\node_modules\fuse-box\dist\commonjs\CollectionSource.js:18:33)
    at Array.forEach (<anonymous>)
    at CollectionSource.get (C:\Users\Michal\Documents\Projects\fusebox-foundation-starter\node_modules\fuse-box\dist\commonjs\CollectionSource.js:17:15)

If you need any other information, please let me know.
Thanks,
Michal

https://github.com/sass/libsass/issues/2312

Seems to be quite a problem...
@mixin FormField {
  position: relative;
  display: flex;
  line-height: $Pollicino-FormField-height;

  &-label {     // <---- that's where the error is coming from

It seems that it cannot be fixed at this stage, simply nothing we can do but wait. Node-sass states "bug confirmed" but when they are going to fix it - nobody knows.
Do you think you can use .css version for now?

As I use only one root app.scss file and all other sass files are included in this file, I don't need grouping at all. So this workaround works great. Thank you for help.


Coinbase verification fails, caption to long

When trying to verify my coinbase user coinbase generates a warning and is unable to save the verification message. I use the following command:
keybase prove coinbase gawin
After copy pasting the keybase client output in the caption part of my coinbase profile and hitting 'Save' I get the following error:

Caption is too long (maximum is 200 characters)

How to resolve this?

There is apparently a difference between
https://www.coinbase.com/gawin#settings
and
https://www.coinbase.com/settings
I already had the settings page open when doing the keybase verification, so that caused the confusion.
issue resolved


Types: Parsing incorrect volume

the FromJSON instances for Ticker types are incorrect.
It reads in the incorrect volume.
Update the ticker type to include volumes for both bid and ask, and alter the FromJSON appropriately

Bug fixed.
See: https://github.com/McGizzle/crypto-binder/commit/7d95c6e694c89569d6d03947abd73fcf0b2b5d14


Remove redundant 'Welcome to Nashville' file

A README for the Welcome to Nashville group project existed in both this repo and in the NSS org as a standalone repo. Since this is the only place where a project's README was stored in the curriculum itself, I have removed it from this repo and changed the link to the project to point to the repo in the NSS org

👍


clickhouse DateTime64 import and copy and export sql error

System information:

Operating system (distribution) and version
DBeaver version
Additional extensions

Connection specification:

Database name and version
Driver name
Do you use tunnels or proxies (SSH, SOCKS, etc)?

Describe the problem you're observing:
Steps to reproduce, if exist:
Include any warning/errors/backtraces from the logs

CREATE TABLE default.sites
(
id Int32,
last_order_time DateTime64(3) COMMENT '最后出单时间-新增',
dt DateTime COMMENT '时间'
)
ENGINE = MergeTree
PRIMARY KEY id
ORDER BY id
SETTINGS index_granularity = 8192;
insert into default.sites (id,last_order_time,dt) values (1,1655956585300,1655956585),
(2,'2022-06-22 00:00:00.020','2022-06-22 00:00:00'),
(3,'2022-06-21 00:00:00.001','2022-06-21 00:00:00');
import csv、excel、sync  error
"id","last_order_time","dt"
1,"2022-06-23 03:56:25.300","2022-06-23 03:56:25"
2,"2022-06-22 00:00:00.020","2022-06-22 00:00:00"
3,"2022-06-21 00:00:00.001","2022-06-21 00:00:00"
Error occurred during 批量插入
（您可以禁用批量插入以跳过特定行）。
原因：
Object parameter [2022-06-23 03:56:25.300] binding not supported
copy.sql is not Quotation marks
INSERT INTO select * from default.sites (id,last_order_time,dt) VALUES (1,2022-06-23 03:56:25.300,'2022-06-23 03:56:25');
INSERT INTO select * from default.sites (id,last_order_time,dt) VALUES (2,2022-06-22 00:00:00.020,'2022-06-22 00:00:00');
INSERT INTO select * from default.sites (id,last_order_time,dt) VALUES (3,2022-06-21 00:00:00.001,'2022-06-21 00:00:00');
dbeaver version 21 or 22

Enabling Skip bind values during insert results in another error:
 Code: 62. DB::Exception: Cannot parse expression of type DateTime64(3) here: 2022-06-23 03:56:25.300,'2022-06-23 03:56:25.000'): While executing ValuesBlockInputFormat. (SYNTAX_ERROR) (version <IP_ADDRESS> (official build))

Close as a duplicate of #17479


feat: support for integer coordinates

Here I am again with integer support.
I figured it was best to open a new PR, since the old one is very different at this point.
Support:
All tests pass except:

PCA transformations
Mesh refinement

I have modified some tests to only have one unambiguous solution. Previously the integer and float code picked different, but equally valid, solutions to some tests.
There are two new generic types:

TSquare: Used for squared distances and multiplication results. The integer implementation needs this to be able to use longs to avoid overflow.
TFloat: Used for angles and other unitless fractional values.

The local transform is supported, but it only supports translations. No scaling or rotation. Those are inappropriate for integer coordinates anyway. Even offsets shouldn't really be used with integer coordinates. But it was easy to add support for it, so why not.
Since the integer code requires some primitives, like circumCenter to be handled differently, I have unfortunately had to introduce some code duplication between the float and double implementations. But it's a very small amount of code (and it's a lot easier to read when one can actually use infix operators). So I think that's fine. I suspect, but have not yet measured, that the new code can perform better since it better preserves the vectorized forms of the coordinates.
I have tried to keep all algorithms as similar as possible. A second PR can be done to clean up and simplify some parts.
I have not measured the performance yet.

Thanks for contributing this! Well done!
Since this is quite a large PR, it will likely take me some time to review—probably by next week. I'll review this as soon as I can!

When I was working on generics, I considered that introducing int2 could be done without adding extra generic parameters. I’m not a fan of this approach, unfortunately. I’ll need to study it further. I believe TFloat could be easily eliminated. As I recall, the main issue was that the hash function requires floats by default, but I know how to implement it for int without using floats. Regarding TSquare, it's probably unavoidable due to overflow concerns as you mentioned. Without it, the approximate limit for int2 would be around <46,340.
Best,
Andrzej

I have pushed some updates which change some methods around so that we can avoid the TFloat type.
In doing so the implementation for AngleIsTooSmall got changed quite a lot, to avoid using atan2.

 If you'd rather keep those for consistency or whatever, I can revert it.

 I appreciate people who don't take themselves too seriously 😄

Hi @HalfVoxel ,
Thank you so much for all the hard work on the commit—great job! I'll push a cleanup and fixup commit to this PR sometime this week, so int2 support should be merged soon. However, your recent changes have introduced a small regression that needs to be fixed before merging this great PR into the main branch. I'll take care of the fix myself and will keep you updated in the thread about any changes.
Best,
Andrzej

Hi @HalfVoxel,
I've pushed a fixup and cleanup commit that addresses a few issues:

Cleaned up namespaces and whitespace.
Renamed TranslationInt32 to TranslationInt.
Removed TBig from the ITransform interface.
Restored T in the refine extension methods. Before the commit, your changes introduced a regression in that area. I designed this method to use T to allow setting values of the same type as the triangulation job, which is crucial for using fixed-point arithmetic.
Reduced PR changes by 30%.
Restored the previous implementation of the "angle" calculations. The performance improvement will be introduced in a separate PR.

I'm going to push a few more commits (in new PRs), and after that, I will continue my work on fixed points (in Q31.32 format). This is almost done; I just need to add some tests and benchmarks.
Best,
Andrzej

Hi @HalfVoxel
I appreciate your feedback, and I'm sorry that this has caused so much frustration. I can see how the lack of comments might make the code more challenging to understand, especially when you're diving into it deeply.
I think we might have different philosophies when it comes to commenting code. My aim is to keep the codebase clean and focused, but I realize this approach can make it harder for others to follow at times.
That said, I do recognize the value of clear documentation (but comments != documentation), and I’m open to finding a middle ground that works for both of us. If there are areas where additional comments would genuinely clarify the code, I’m happy to discuss how we can incorporate them effectively. I think it would be better to work on contributor notes instead of raw comments. Also, instead of adding comments, I try to document intentions in the tests.
I value your contributions and would love to keep working together to improve the project.
Thanks for your understanding, and I look forward to hearing your thoughts.
Best,
Andrzej


Fix issues with peerDependencies when ember-auto-import present.

When ember-auto-import is present, avoid app.import of QUnit itself. Without this change, users (when they have peerDependencies setup) would get an error like:
Uncaught Error: QUnit has already been defined.

This is because we were previously always app.importing QUnit itself, but then ember-auto-import sees: import { test, module } from 'qunit'; in a test file alongside the package.json having an entry for qunit so it attempts to bundle it again.
This is the path away from custom build/app.import shenanigans and helps us down the path of avoiding specialness in an Embroider focused world.

This makes sense to me. CI is failing though.

CI is failing though.

Ya, was because I forgot to remove the this.import for the old qunit-configuration file. Just updated (along with an ember-try scenario to confirm that things work with and without ember-auto-import).

@ef4 - FYI, this is roughly the culmination of the thing we had discussed a while back. As of 5.0 beta series (and this PR) the only this.imports needed for ember-auto-import users is for the QUnit styles (I couldn't figure out a better way, input definitely appreciated); qunit and @ember/test-helpers are peer dependencies; contentFor usage is gone.

Current failure with ember test (ember s + /tests works properly):
not ok 1 Chrome 85.0 - [undefined ms] - Global error: Uncaught Error: Testem was unable to detect a test framework, please load it before invoking Testem.hookIntoTestFramework at http://localhost:7357/testem.js, line 913
    ---
        browser log: |
            testContext: [object Object]
            ERROR: Uncaught Error: Testem was unable to detect a test framework, please load it before invoking Testem.hookIntoTestFramework at http://localhost:7357/testem.js, line 913

    ...
not ok 2 Chrome 85.0 - [undefined ms] - Global error: Uncaught Error: Could not find module `qunit` imported from `ember-qunit/adapter` at http://localhost:7357/assets/vendor.js, line 259
    ---
        browser log: |
            testContext: [object Object]
            ERROR: Uncaught Error: Could not find module `qunit` imported from `ember-qunit/adapter` at http://localhost:7357/assets/vendor.js, line 259

    ...
not ok 3 Chrome 85.0 - [undefined ms] - Global error: Uncaught Error: Assertion Failed: The tests file was not loaded. Make sure your tests index.html includes "assets/tests.js". at http://localho st:7357/assets/vendor.js, line 47077
    ---
        browser log: |
            testContext: [object Object]
            ERROR: Uncaught Error: Assertion Failed: The tests file was not loaded. Make sure your tests index.html includes "assets/tests.js". at http://localhost:7357/assets/vendor.js, line 47077

    ...
^C
Gracefully shutting down from SIGINT (Ctrl-C)
not ok 4 [undefined ms] - Error
    ---
        message: >
            Received SIGINT signal
    ...

1..4
# tests 4
# pass  0
# skip  0
# todo  0
# fail  4

Just pushed an update that should fix that issue with ember-auto-import + ember test.


[Android] Fix Crosswalk failed to build debug version.

This is simple cherry-pick for previous fix the blink debug build. Same as PR #43

The code already works fine in M42, so this PR should not be necessary.


Adina/gravatar command

Addresses #228.
Test changing your gravatar with /gravatar [emailAddress]. There should be a grey text pop-up notifying you you've changed your gravatar, and the change should be visible in your Account settings and in the side bar displaying users in room.
There is an issue with it displaying your gravatar on recent messages sent in the chatroom, but if you leave and come back it should display your updated gravatar on previously sent messages. It seems like this is a context issue because the client.hash is getting updated in the database for the user.  Will submit bug on backlog.

@ashanhol GravatarCommand works correctly with the email parameter. Account page shows gravatar and works as described in PR description. :shipit:


Fix #263: let let and const let declarations are supposed to be a syntax error

I was finally able to get a chance to finish this. I added a check to the parser that makes sure an identifier used in a let or const declaration is not let itself (see the links to the spec in issue #263 ). The included tests check a few of the different ways that an identifier may be used in this manner. Thanks!

Hi @AlexElting, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@AlexElting, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

There's a few minor things to touch up but otherwise looks great!  Thanks @AlexElting.
I was just reviewing this issue and wondering if it was coming soon when I noticed you had pushed a commit for it to your fork.  Weird.

Thanks, @ianwjhalliday!! I suspected there was a better way to do that check.

This is perfect.  Will merge in a moment.

Boom, it's in.  Thanks again @AlexElting!

Sweet, thank you!


ffmpeg-full: setup addOpenGLRunpath

Motivation for this change
addOpenGLRunpath was missing in pkgs.ffmpeg-full, preventing users to make use of hardware acceleration when encoding.
Things done
Backported addOpenGLRunpath from pkgs.ffmpeg.

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I think this fix should also be backported to 20.03.

Hmm - the change to ffmpeg causes a large rebuild, so this would need to go via staging.
Could you drop the ffmpeg commit, and open this one into staging? Happy to backport the ffmpeg-full one into 20.03 (and the ffmpeg one into staging-20.03, once its merged.

@flokli I've just split it up into another PR for ffmpeg (to staging), #87968.

I changed the addOpenGLRunpath $out/lib/lib….so* to addOpenGLRunpath $out/lib/lib….so, as there's only one "real" .so.X.Y.Z library, and the rest is less specific symlinks pointing to that file.

Backported to 20.03 in 885a6658073f6c5e9a4b37f131ce870a41a4ce7d.

Awesome, thanks!


FMC doc cleanup and PCR update

Cleaned up some outstanding TODO items in FMC doc and added more detail for PCR flows.

In creating this PR, I noticed some problems and inconsistencies in the PCR handling from ROM and FMC. Documented in this issue:
https://github.com/chipsalliance/caliptra-sw/issues/449


Fix CPUParticles3D to GPUParticles3D weird particle conversion behavior

This is A LIKELY fix to the problem #97621. (It needs to be assembled and tested)
Bugsquad edit:

Fix #97621

Please amend the commit name to the PR name. We don't use names like "Fix issue-link"

okay, 1 minute.

I can't do it. Can you help me?

Please amend the commit name to the PR name. We don't use names like "Fix issue-link"

I can't do it. Can you help me?

Please amend the commit name to the PR name. We don't use names like "Fix issue-link"

You can do interactive rebase, edit the comment and then do force push to your remote branch
Something like
git rebase -i HEAD~2
where 2 is the amount of commit that you have made, then amend the exact commit that you want

I am confused as to how the submitted code would work; you are changing the code that happens when you convert from GPU to CPU particles, when the bug is converting from CPU to GPU.
The issue is also more complex than was initially reported in the linked bug report


[Web] TextField: Cursor misplaced on word tap (mobile devices)

I'm using an expanded TextEdit widget as a text input for an app and so far I didn't have any issue on Android or Web (desktop). However, when using the Web version on mobiles (chrome/brave/etc) the cursor is totally broken when the text is larger than the screen (once you scroll), making it real difficult to interact with the text. This happens in Android and Ios devices.
This is the simplified version of the code:
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      home: Scaffold(
        body: Padding(
          padding: const EdgeInsets.all(24.0),
          child: TextField(
            scrollPhysics: const BouncingScrollPhysics(),
            cursorColor: Colors.black,
            expands: true,
            textAlignVertical: TextAlignVertical.top,
            decoration: InputDecoration(
              border: OutlineInputBorder(
                borderSide: BorderSide(),
              ),
            ),
            maxLines: null,
            showCursor: true,
            keyboardType: TextInputType.multiline,
            scrollPadding: const EdgeInsets.all(25),
          ),
        ),
      ),
    );
  }
}

You can access a deployed version for testing in https://yaymalaga.netlify.app/.
To reproduce the issue, just paste a long text, scroll a little bit and try to select a word. You will see that the cursor is misplaced with a huge offset, making the text to even scroll sometimes. Interestingly enough, it works fine just some on some words. Also, some times I can see two markers, one blue and another one red/orange.
In the following video you can see the behavior described. Even thought the red/orange marker seems to be in the correct place, the cursor is going all crazy.
https://user-images.githubusercontent.com/2735961/127692797-acfd533a-9324-4be4-933e-3bd7e84a10ce.mp4

This is tested in stable:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.2.3, on Linux, locale en_GB.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[✓] Chrome - develop for the web
[✓] Linux toolchain - develop for Linux desktop
[✓] Android Studio (version 4.2)
[✓] Connected device (2 available)

• No issues found!

However, same behavior in beta, dev and master channels. In this case, the app is built using flutter build web, but the same result is found when forcing both html or canvaskit renders.

Hi @yaymalaga
Thanks for filing the issue, I can reproduce the issue on iOS and Android on the latest stable as well as master using both renderers
Used the following minimal code sample

minimal code sample
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      home: Scaffold(
        body: Padding(
          padding: const EdgeInsets.all(24.0),
          child: TextField(
            scrollPhysics: const BouncingScrollPhysics(),
            cursorColor: Colors.black,
            expands: true,
            textAlignVertical: TextAlignVertical.top,
            decoration: InputDecoration(
              border: OutlineInputBorder(
                borderSide: BorderSide(),
              ),
            ),
            maxLines: null,
            showCursor: true,
            keyboardType: TextInputType.multiline,
            scrollPadding: const EdgeInsets.all(25),
          ),
        ),
      ),
    );
  }
}

Steps to reproduce

flutter build web --web-renderer html or flutter build web --web-renderer canvaskit
Paste the long scrollable text paragraph such as one below

sample text
The first version of Flutter was known as codename "Sky" and ran on the Android operating system. It was unveiled at the 2015 Dart developer summit[6] with the stated intent of being able to render consistently at 120 frames per second.[7] During the keynote of Google Developer Days in Shanghai, Google announced Flutter Release Preview 2, which is the last big release before Flutter 1.0. On December 4, 2018, Flutter 1.0 was released at the Flutter Live event, denoting the first "stable" version of the Framework. On December 11, 2019, Flutter 1.12 was released at the Flutter Interactive event.[8]

On May 6, 2020, the Dart software development kit (SDK) in version 2.8 and the Flutter in version 1.17.0 were released, where support was added to the Metal API, improving performance on iOS devices (approximately 50%), new Material widgets, and new network tracking.

On March 3, 2021, Google released Flutter 2 during an online Flutter Engage event. This major update brought official support for web-based applications with new CanvasKit renderer and web specific widgets, early-access desktop application support for Windows, macOS, and Linux and improved Add-to-App APIs.[9] This release included sound null-safety, which caused many breaking changes and issues with many external packages, but the Flutter team included instructions to mitigate these changes as well.

3. Scroll a little bit up and down, then tap a word you can select and tap somewhere else repeat, you will unexpected cursor placement in few tries.
Preview (same result on iOS and Android)
https://user-images.githubusercontent.com/48603081/127861408-f2356883-dd3c-401e-9574-f685cdcf5634.mp4

stable
master

❌
❌

Check flutter doctor -v outputs for each channel below

flutter doctor -v
[✓] Flutter (Channel stable, 2.2.3, on macOS 12.0 21A5294g darwin-x64, locale en-GB)
    • Flutter version 2.2.3 at /Users/tahatesser/Code/flutter_stable
    • Framework revision f4abaa0735 (5 weeks ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/tahatesser/Code/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Users/tahatesser/Code/SDK
    • Java binary at: /Users/tahatesser/Code/openjdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (AdoptOpenJDK)(build 1.8.0_292-b10)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode-beta.app/Contents/Developer
    • Xcode 13.0, Build version 13A5201i
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] VS Code (version 1.58.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.24.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.0 21A5294g darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 92.0.4515.107

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 2.5.0-2.0.pre.113, on macOS 12.0 21A5294g darwin-x64, locale en-GB)
    • Flutter version 2.5.0-2.0.pre.113 at /Users/tahatesser/Code/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 0abf02f855 (3 days ago), 2021-07-30 17:44:02 -0700
    • Engine revision 29f9df723f
    • Dart version 2.14.0 (build 2.14.0-368.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/tahatesser/Code/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Users/tahatesser/Code/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode-beta.app/Contents/Developer
    • Xcode 13.0, Build version 13A5201i
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] VS Code (version 1.58.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.24.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.0 21A5294g darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 92.0.4515.107

• No issues found!

✅: No Issue ❌: Issue reproduced
Related issues
https://github.com/flutter/flutter/issues/78031
https://github.com/flutter/flutter/issues/77083

Duplicate of https://github.com/flutter/flutter/issues/78031


Remove the internal file resources from the resource collection

This doesn't appear to do anything except create "dummy" file resources (with
action :nothing) on the stack.

I suppose I could use some feedback on this. Does adding the file resources to the context's resource collection serve some other purpose that I'm not seeing in my simple use case?

I think this seems to be required by ChefSpec when using step_into to test HWRPs instead of LWRPs.
You can use something like the following to avoid breaking the tests:
run_context.resource_collection << resource if defined?(ChefSpec)

@zuazo Thanks, I'll incorporate your feedback and open a new pull request.