a CC BY NC version?

This seams to have stalled for many years. What if you had a noncommercial version?
This way, it will be easier to harvest assets that are already created. It would be legally compatible with +90% of games. This is meant to be a second version, not a replacement to this one. It's a more pragmatic solution, in order to actually have something usable in a timely manner.

NonCommercial will not happen.
This has too many problems due to NonCommercial being not properly legally defined.
When you make a mapping competition with prices it could be already violating the non-commercial aspect depending on which lawyer you ask so this is all highly problematic.
Just look at the RPG Atelier CC-BY-SA-NC ORTP: Nobody can use this because of this terrible license.


Unary ! and Logical not

The unary ! operator seems to be using not:
https://github.com/serqlane/serqlane/blob/1d86647f7cc261e882adeafc6b6f1b8008966bcb/serqlane/grammar.lark#L95-L98
If we are going with C style unary operators they should be as follows:

- unary minus
++ increment
-- decrement
! not
etc..

Additionally, it seems like we do not have a logical way to specify not, which is possible the cause of unary not being assigned as not. These are the only two I believe:
https://github.com/serqlane/serqlane/blob/1d86647f7cc261e882adeafc6b6f1b8008966bcb/serqlane/grammar.lark#L84-L85
The prevent us from being able to do something like:
let x = 10
let y = 20
if not x < y {
    dbg("x is not less than y")
}

As it causes the following error:
ValueError: tree=Tree(Token('RULE', 'unary_expression'), [Tree(Token('RULE', 'not'), []), Tree(Token('RULE', 'expression'), [Tree(Token('RULE', 'identifier'), [Token('__ANON_5', 'x')])])])

Addressed by #96


How do we package client secrets for OAuth?

Get the secrets from obfuscated URL
If abuse seen, blacklist offending IP addresses and republish


Show time progress in center of screen while scrubbing video?

Please do what VLC, mpv, Newpipe and Antennapod do and show the current time in the center of the screen with a bigger font so we can clearly see what time we are on when scrubbing the video forward and back.
Something like this.

I agree. This is something in my backlog that I want to try.

I agree. This is something in my backlog that I want to try.

Latest release (v0.14) prints time difference when using horizontal gesture to seek. I'm not sure/decided on showing more info.

Latest release (v0.14) prints time difference when using horizontal gesture to seek. I'm not sure/decided on showing more info.

Latest release (v0.14) prints time difference when using horizontal gesture to seek. I'm not sure/decided on showing more info.

I think most people would like to see [scrub time] (time difference).
Scrubbing 45 seconds ahead from 1:00:00 would be [1:00:45] (45s)
Scrubbing 90 seconds ahead from 1:00:00 would be [1:01:30 (1m30s)
I think that looks compact and informative.

Latest release (v0.14) prints time difference when using horizontal gesture to seek. I'm not sure/decided on showing more info.

I think most people would like to see [scrub time] (time difference).
Scrubbing 45 seconds ahead from 1:00:00 would be [1:00:45] (45s)
Scrubbing 90 seconds ahead from 1:00:00 would be [1:01:30 (1m30s)
I think that looks compact and informative.

Has this been implemented? then close..
@moneytoo I see a lot of open issues in issue tracker which have been referenced in commits have been implemented (if satisfactorily according to user's feedback then may be closed.


Cannot install PT

Microsoft PowerToys version
UNKNOWN
Installation method
Microsoft Store, WinGet
Running as admin
Yes
Area(s) with issue?
General
Steps to reproduce
PT not listed as installed,
System Win 10 , Ver 22H2, OS 19045.4651
✔ Expected Behavior
Tried to install via MS Store, and PS using Winget, both result in failure to install.
Log shows that it detects a previous version
❌ Actual Behavior
[1164:0FFC][2024-08-14T18:13:24]i101: Detected package: TerminatePowerToys, state: Absent, cached: None
[1164:0FFC][2024-08-14T18:13:24]i101: Detected package: WebView2, state: Present, cached: None
[1164:0FFC][2024-08-14T18:13:24]i101: Detected package: PowerToysUserSetup_0.83.0_x64.msi, state: Absent, cached: None
[1164:0FFC][2024-08-14T18:13:24]i052: Condition 'MinimumVersion >= DetectedPowerToysVersion' evaluates to false.
[1164:0FFC][2024-08-14T18:13:24]e000: PowerToys is already installed on this system for all users. We recommend first uninstalling that version before installing this one.
[1164:0FFC][2024-08-14T18:13:24]e000: Error 0x81f40001: Bundle condition evaluated to false: MinimumVersion >= DetectedPowerToysVersion
[1164:0FFC][2024-08-14T18:13:24]i199: Detect complete, result: 0x0
Other Software
No response

The error is it is saying you have a machine wide install of PowerToys already.  Could it have been installed via a different user maybe?  Does C:\Program Files\PowerToys exist?  /needinfo
this line is why i believe a machine wide was done and Store is a per-user only.
PowerToys is already installed on this system for all users. We recommend first uninstalling that version before installing this one.
WinGet should detect user vs machine wide however

Does C:\Program Files\PowerToys exist? / no
WinGet should detect user vs machine wide however . Winget downloaded PT but install fails with same error

When trying to install from link above, error
The older version on PowerToys Preview cannot be removed. Contact your technical support group.
Using reg scanner, no entries found relating to powertoys.

Fatal error during installation 0x80070643

Any help appreciated

on PowerToys Preview cannot be removed. Contact your technical support group.
Using reg scanner, no entries found relating to powertoys.

Do you have further details on the error message?
I had similar issues before where the installation got messed up. In such cases, I had to download the installer for the supposedly installed version again, extract the embedded MSI using dark.exe from WiX3, and then uninstalling using the MSI. (using: dark.exe <path to PT installer exe> -x . -> MSI is within folder AttachedContainer)
In these cases the installer log would show it could not find a MSI somewhere in %localappdata%\Package Cache\{some uuid}.
I guess this only works when there is at least some sort of trace left of the other version.
The Windows' installed apps screen does not show an entry for PT, right?

The Windows' installed apps screen does not show an entry for PT, right? Correct.
I tried the most up to date V and still the same, also tried installing via Choco, same result


DataProviderDelegate

Hi Peter
Is there any reason the DataProviderDelegate is defined with a generic method rather than an associated type?
https://github.com/linkedin/RocketData/blob/master/RocketData/DataProvider.swift#L196-L206
Let's say I want to create a Reactive extension that emit RocketDataDelegate updates
Right now I can write something like this
// given some kind of Generic Emitter class that emit T elements
class Emitter<T> {
  func on(data: T) { /* implementation */ }
}

class RocketDataEmitter<T: SimpleModel>: DataProviderDelegate {
  let emitter = Emitter<T?>()

  func dataProviderHasUpdatedData<V>(dataProvider: DataProvider<V>, context: Any?) {
    guard let data = dataProvider.data as? T? else { fatalError() }
    emitter.on(data)
  }
}

With associated type I could avoid the weird casting
public protocol DataProviderDelegate: class {
  associatedtype T: SimpleModel
  func dataProviderHasUpdatedData(dataProvider: DataProvider<T>, context: Any?)
}

class RocketDataEmitter<T: SimpleModel>: DataProviderDelegate {
  let emitter = Emitter<T?>()

  func dataProviderHasUpdatedData(dataProvider: DataProvider<T>, context: Any?) {
    emitter.on(data)
  }

There were a few reasons...and honestly, can't really remember all of them. But anyway, this was the original design, but it got so complicated, that I decided it wasn't worth it.
The main thing stems from:
weak var delegate: DataProviderDelegate?

This line is illegal if you use an associated type. Instead, you need to do:
class DataProvider<DataProviderType: DataProviderDelegate> {
   weak var delegate: DataProviderType
   var data: DataProviderType.T?
}

Which means at the call site, instead of:
let dataProvider = DataProvider<UserModel>()

you need to do:
let dataProvider = DataProvider<UserViewController>()
func dataProviderHasUpdatedData(dataProvider: DataProvider<UserModel>, context: Any?) {}

Anyway, as you can see, not only does this make the API really weird, but you end up with a bunch of weird behavior. There were some consequences for how it was implemented too, but can't really remember them off the top of my head.

The way you're doing it is how I'd suggest doing it :/
The cast will never fail.

Personally, I sort of chalk this up to a limitation in swift's generics (meh, maybe that's a cop out...). Maybe this will be solved with the generics manifesto? But not really familiar enough to be confident there.

Ok make sense. thanks for the explanations


Debug flag causes uncaught type error

When using Elm 0.18 with debug flag my code crashes at runtime with Uncaught TypeError: Cannot read property 'nativeBinding' of undefined
I stripped down my code to barebones in order to reproduce this bug: https://github.com/igotlegs/elm-debug-bug
Using OSX El Capitan 10.11.5 with Chrome 54.0.2840.98 (64-bit)

I need you to get the example down smaller. It is really hard to know if this is definitely caused by Elm with all the other code in there.

Not sure what is going on and cannot debug based on whole repos.
Maybe it is something here? https://github.com/elm-lang/virtual-dom/issues/98
If not, please open an issue on that repo with a single Elm file that can reproduce the error.


Update Gradle wrapper to 4.0

Updated to the latest stable version 4.0.

Updated again to the latest stable version 4.0.1.

Updated to latest stable version 4.1.

Updated to latest stable version 4.7.

will it ever be merged?

Updated to latest stable version 5.6.

Updated to latest stable version 5.6.2.

Updated to latest stable version 5.6.3.

Updated to 5.6.4.


Feat/add attributes

Closes #10

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


Fix user setting query

Fixes #1864
FYI @matmair iexact query on primary key/integer seems to make the PostGreSQL backend choke:
https://stackoverflow.com/a/60555361/12794913

@eeintech good catch. I often use iexact match with user-input as it ensures a better match on mysql but does not make sense in this case as id is a number. I would have assumed that Upper() of a number is just the same number.

Would have been nice agree, glad it is fixed now it was pretty annoying :sweat_smile:


Can't import files from Firefox

If you clicks Import > svg (or vector drawable) nothing happens. The file picker never appears. It works in Chrome.

Arg, sorry about that... will look into it today.

TLDR I should really just rewrite these enormous file picker hacks using the strategy described here: https://tympanus.net/codrops/2015/09/15/styling-customizing-file-inputs-smart-way/
Sincerely, a web noob

Should be fixed now! Let me know if you have any issues.

It is. Thank you for this wonderfull app :)


Question: interface the plugin with an external server

-----What I would like to do:
I would like to have the component rails managed by something, with a led for each rail. When I hoover the mouse over a component (or list of components of the same type) I would like to see the corresponding led to switch on.
-----What I THINK I need to do:
Since InteractiveHtmlBomDemo runs on client-side my something that manage the rails should be a server, to which the client make requests, with a fixed ip or something like that
-----And finally: ASSUMING that I think right, where should I put my hands? (on the code, I mean)
Where is the code that manage the highlighting of the component, when the mouse is over them?
Is it possible to make a request to a third-party server without suffering too much?
-----Thank you for your time and your attention, I am really a noob when we talk about web-clients-and-servers

Good news is you don't have to modify plugin code at all, this use case is covered by the customization system already. There is a hook specifically for highlight event, see https://github.com/openscopeproject/InteractiveHtmlBom/wiki/Customization#custom-javascript
All you have to do is define a callback function and register it for highlight event, ibom will call it whenever highlight changes and pass it a list of highlighted footprints. From there you can make a http call to another server or do anything your browser's js engine allows you to do.

Thank you very much, I am now working on this.
I have also learned that I must pay attention to the "wiki" button on github :)
You can manage this issue as you wish now

Hello, I have found one more question:
My "rail manager" knows the component code and the comopnent package if we talk about simple resistors.
In the event that I can log there is only the name of the component (es. R3 or U2)
How can I get the value and/or the package?


Supporting Elasticsearch 8.0 and its secure by default mode

With the release of Elasticsearch 8.0 a major change has been done to provide a secure implementation by default - namely enabling TLS by default as well as requiring user/password authentication. As this is the supposed mode of operation in production I think it would be great to provide the same experience within integration tests.
There are a couple of caveats though and before doing any PRs I'd appreciate some guidance. So, let's get started with what works and how.
I have a test up and running that initializes like this
private static final ElasticsearchContainer container =
            new ElasticsearchContainer("docker.elastic.co/elasticsearch/elasticsearch:8.0.0")
                    .withExposedPorts(9200)
                    .withPassword("s3cret");

This is the container and HTTP client setup
      // this is required because the wait strategy uses HttpsURLConnection and does not yet
      // know about the certificate to be copied out of the container, see below
      HttpsURLConnection.setDefaultSSLSocketFactory(SslUtils.trustAllContext().getSocketFactory());

      // remove from environment to have TLS enabled
      container.getEnvMap().remove("xpack.security.enabled");

      // custom wait strategy for tls and basic auth
      container.setWaitStrategy(new HttpWaitStrategy()
              .forPort(9200)
              .usingTls()
              .forStatusCode(HTTP_OK)
              .forStatusCode(HTTP_UNAUTHORIZED)
              .withBasicCredentials("elastic", "secr3t")
              .withStartupTimeout(Duration.ofMinutes(2)));
      container.start();

The Elasticsearch Client setup looks like this and requires extracting the self signed certificate out of the docker container
        // extract the ca cert from the running instance
        byte[] certAsBytes = container.copyFileFromContainer("/usr/share/elasticsearch/config/certs/http_ca.crt", InputStream::readAllBytes);

        HttpHost host = new HttpHost("localhost", container.getMappedPort(9200), "https");
        final CredentialsProvider credentialsProvider = new BasicCredentialsProvider();
        credentialsProvider.setCredentials(AuthScope.ANY, new UsernamePasswordCredentials("elastic", "s3cret"));
        final RestClientBuilder builder = RestClient.builder(host);

        builder.setHttpClientConfigCallback(clientBuilder -> {
            clientBuilder.setSSLContext(SslUtils.createContextFromCaCert(certAsBytes));
            clientBuilder.setDefaultCredentialsProvider(credentialsProvider);
            return clientBuilder;
        });

Let's cover the issues with this and how to resolve it:
First, ElasticsearchContainer.withPassword() sets xpack.security.enabled - this should probably be done with a specific method, as I need to explicitely remove it from the environment to make sure TLS is enabled. I think it makes to have proper methods to enable/disable TLS plus some checks if this is running against an 8.0 release in order to keep BWC compatibility?
Second, the wait strategy uses HttpsURLConnection, which at the point of its first run, has no access on the generated cert of the docker container. I do not consider a trust all cert manager a valid solution here, so the question is, whether it makes sense to have a custom wait strategy that extracts the cert from the docker container as well, and cleans up the UrlConnection to use the default ssl socket factory after the test. This would still mean that there might be issue with parallel running tests that may be setting this as well. Maybe you have a better idea to work around that.
Any help and guidance much appreciated!

Hey, thanks for bringing this to our attention @spinscale.
In some cases, we found that LogMessageWaitStrategy can provide the best experience with the least amount of workarounds, it might be the case here as well. Of course, it has some drawbacks, like tightly coupling against the logging side-effect rather than to the actual container behavior, but often this is a pragmatic tradeoff.
Why does disabling xpack.security enable TLS? This is a bit confusing to me.
Since users need the self-signed cert, I think it makes sense to provide an API for extracting it and document the usage.

Fantastic, I will take a look at the LogMessageWaitStrategy then as well as adapting the ElasticContainer class then!
Regarding xpack.security. This is not about disabling, but not setting at, as the default options mean that TLS is enabled. The moment you set options manually, you have to set all of them. Hope that clears it up.
Thanks for your help!


Depth Errors on Semi-Translucent Objects

Description
I'm trying to find the height of stacks of lab wellplates.  These plates are made of clear poly-polypropylene plastic and are covered with a semi-translucent silicone mat:

However, the stacks of plates look sunken into the ground in the depth image:

When I cover each of the stacks with an index card, the depth image looks as expected:

Do you have any tips for improving the depth camera performance in this case?  The depth image looks great when looking at other objects, so I was surprised about how poorly it worked on the wellplate.  I submitted a report to the Microsoft's support line and was asked to post here.
After some further reading, it looks like silicone is relatively transparent to IR.  I assume that is what's causing this, but wanted to see if anyone had any further ideas.
Desktop (please complete the following information):

OS with Version: Windows 18362
SDK Version:
Current Firmware Versions:
RGB camera firmware:      1.6.110
Depth camera firmware:    1.6.79
Depth config file:        6109.7
Audio firmware:           1.6.14

@thane what you are seeing is a combination of multipath and subsurface scattering. This is a limitation of current ToF depth technology.


Add data parity tests for epoch details

@oneEdoubleD Would you mind refactoring your tests to follow this pattern?

Yep! I agree, this is better. I'll do it the next time I look at GraphQL :)


[REQ] Component Request: DS18B20-compatible temperature sensors

What's the full name of the component you'd like to see in Adafruit.io WipperSnapper?
Waterproof 1-Wire DS18B20 Compatible Digital temperature sensor
What type of component is this (i.e.: Pin, I2C)?
1-Wire
Product URL or Datasheet
https://www.adafruit.com/product/381
https://www.adafruit.com/product/642
https://www.adafruit.com/product/374
Additional context
Requires https://github.com/adafruit/Wippersnapper_Protobuf/issues/89 to be implemented first.

https://github.com/adafruit/Wippersnapper_Components/pull/43

i think this can be closed :)

Closing, supported by the latest WipperSnapper firmware!


feature request for setting base directory config on source files before concatenation

Hello,
I'm using templates "<%= baseDirectory %>" and appending them before each source file directory on the file list before concatenation.
could you create some config for setting a base directory in the concat task?
or is this already available?

Sounds like the job for a separate task.


Add support for opensearch-benchmark's compare feature

Description
Adds support for developers to use the compare feature from opensearch-benchmark to compare their changes against a baseline.
Issues Resolved
#4837
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

The proposed changes would require you to update benchmark_test_runner_opensearch to get the respective command class.
benchmark_test_suite = BenchmarkTestSuiteRunners.from_args(self.args, cluster.endpoint_with_port, self.security, cluster.fetch_password()) for execute-test. For compare you would just need to pass self.args.

LGTM, great job on your first PR @OVI3D0!
@peterzhuamazon Let us know if you have any comments or concerns.

I have some offline conversation with @rishabh6788, and now understand the requirement of using docker, due to barebone OSB binary is preventing multiple processes run in parallel.
However, I would suggest these things if you must use docker this way:

Add a docker check before the run so it can inform user if it does not detect docker
Verify it is on either mac or linux, hard fail on any other related systems for now (Not hard blocker)
Add corresponding requirements to the readme file, such as docker, cpu/ram, system for this specific implementation since the code is not generalized to run on any system

We need to make sure users are aware of the limited condition of this compare feature. Since this is a rather big change, would appreciate if we can get these checks in the same PR or in a quick followup later.
Thanks.

Thanks @OVI3D0 / @rishabh6788 👍


Chart Title separate padding for top and bottom

Feature Proposal

Right now it is only possible to change one padding value for the Title, which is then applied as padding on the bottom and on the top of the Title.
It would be an improvement if we would have control over this per side (left, right, top, bottom).
 I have registered a subtitle plugin too, which needs the title to have padding on the bottom (so padding between the chart title and the actual chart). This space is where the subtitle sits.
Conclusion: I would like to be able to only specify a bottom padding, without any (or less) top padding, which is currently not possible.
Possible Implementation

I suggest that the implementation will have to be consistent with something like the layout padding currently:
padding: {
    left: 0,
    right: 0,
    top: 5,
    bottom: 15
}

Instead of the current:
padding: 15

Whether the left and right padding values are necessary, I'm not sure.

The title somewhat handles padding for each direction separately:
https://github.com/chartjs/Chart.js/blob/c8bdca62e89b4d6d9f334cec7a8bfa673725014c/src/plugins/plugin.title.js#L89
But you're right, that I think there's no way to set them to be different values. Your proposal seems very consistent with what we do elsewhere, so if you want to send a PR for this feature it seems very likely to be accepted ;-)

Hi Ben,
The properties paddingLeft, PaddingTop, etc. don't seem to be doing anything when changed. Is it possible that these properties are never accessed?

Oh, you are right. They don't appear to be used at all. That's disappointing that there's unused code like that present :-/

Yes that is indeed unfortunate. I removed the unused code in my PR.
Also, I added the ability to pass separate top and bottom padding for the title (but a single number for padding is still possible). Left and right padding didn't seem very useful in my eyes, so I didn't include that.
I see that some builds are currently failing in the PR, so I'm going to have a look at that.

PR is merged.


Allow specifying relative paths in .rsp files

This would allow for a scenario where you want to specify a logger for all builds in a repo. Example:
Directory.Build.rsp
-ConsoleLoggerParameters:Verbosity=Minimal;Summary;ForceNoAlign
-MaxCPUCount
-NodeReuse:false
-Restore
-logger:%MSBuildThisFileDirectory%\build\MyLogger.dll

This will work assuming that build\MyLogger.dll is checked into the repo.
I can add tests, just wanted feedback on the name. Not thrilled that it uses the MSBuild property syntax, but couldn't think of something better :)
Edit: Switched to % instead.

Environment variable style %MSBuildThisFileDirectory%, maybe? Or actually plumb through env vars and provide a magic one?

Expanding environment variables in the file is the next line, so this would be the place to do it. But yeah using % might be better to not give the impression anything else works.

@jeffkl FYI

So I thought of a snag with this approach. If someone uses this feature, MSBuild prior to 16.3 would be completely broken :(. Since the reason we want this is to attach loggers to giant repos, we just can't guarantee everyone will be on 16.3+ and will largely negate the point of this feature.
So, I have a proposed fix. I don't feel super happy with this, but it does solve the issue in a forward/backwards compatible way. From my previous example:
-ConsoleLoggerParameters:Verbosity=Minimal;Summary;ForceNoAlign
-MaxCPUCount
-NodeReuse:false
-Restore
# Only attach loggers in 16.3.300+
#v16.300 -logger:%MSBuildThisFileDirectory%\build\MyLogger.dll

Previous versions of MSBuild skip lines that start with #, and we can make newer versions honor this syntax. Thoughts? Better ideas? It was easy to implement so I'll push that, but I can always change it.

If someone uses this feature, MSBuild prior to 16.3 would be completely broken :(. Since the reason we want this is to attach loggers to giant repos, we just can't guarantee everyone will be on 16.3+

I don't think I understand. The concern is that someone would add an .rsp file to a repo, (presumably they've already updated the CI and official build infrastructure to support that), but some user is on an older version and still needs to be supported?

@livarcocc @BenVillalobos
As discussed in triage this only includes the change to support %MSBuildThisFileDirectory% and not the versioning stuff. Good to go on this one?

Does response files support actual env variables?
I have checked and it does not seem to support it with %abc% or $(abc) or $abc syntax!

Seems it also supports env variables but %CD% it spews out Í% garbage character.


Update jackson-module-scala to 2.18.0

About this PR
📦 Updates com.fasterxml.jackson.module:jackson-module-scala from 2.17.2 to 2.18.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
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.fasterxml.jackson.module", artifactId = "jackson-module-scala" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.fasterxml.jackson.module", artifactId = "jackson-module-scala" }
}]

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

Superseded by #96.


404s in BigQueryIO don't get output to Failed Inserts PCollection

Given that BigQueryIO is configured to use createDisposition(CREATE_NEVER),
and the DynamicDestinations class returns "null" for a schema,
and the table for that destination does not exist in BigQuery, When I stream records to BigQuery for that table, then the write should fail,
and the failed rows should appear on the output PCollection for Failed Inserts (via getFailedInserts().

Almost all of the time, the table exists before hand, but given that new tables can be created, we want this behavior to be non-explosive to the Job, however, what we are seeing is that processing completely stops in those pipelines, and eventually the jobs run out of memory. I feel that the appropriate action when BigQuery 404's for the table, would be to submit those failed TableRows to the output PCollection and continue processing as normal.
Imported from Jira BEAM-14364. Original Jira may contain additional context.
Reported by: svetak.

Unable to assign user @svetakvsundhar. If able, self-assign, otherwise tag @damccorm so that he can assign you. Because of GitHub's spam prevention system, your activity is required to enable assignment in this repo.

@reuvenlax @johnjcasey is this still happening? There is a lot of context on the Jira comments that I haven't brought over. It sounds like fundamental bugs in the connector.

I suspect it does happen. Some context:
This feature was intended for known persistent failures at the row level, and is implemented using the per-row status returned by BigQuery. An example would be a row that does not match the BigQuery schema or a row that exceeds BigQuery's size limit. This feature does not capture potentially ephemeral failures, such as when the RPC to BigQuery itself fails. In that case we simply retry the RPC.
While it seems that the RPC returning a 404 is a persistent error, this generally isn't the case. For instance a temporary outage might cause the RPC to return 404, yet on retry the RPC succeeds. We didn't want a situation in which a temporary BigQuery outage caused all data to be sent to the dead-letter output for a period of time.
However I would like to understand the use case more. Is this a case in which records destined for a specific table are sent to the Dataflow pipeline before the BigQuery table is created? Is there some offline process creating those tables, and is that process simply delayed?

@svetakvsundhar are you still looking at this?

Is there a particular reason to not use a pattern of outputting to failure after N retries?

In general, that pattern is avoided in streaming, since the system
prioritizes data consistency. Temporary problems (e.g. temporary issues
with the sink, the network, or anything else) could easily cause N retries
to exceed, causing 100% of output going to the failure collection.
A second issue is that there isn't a good way of determining retry count.
We can only know retry count in the current bundle. If the bundle fails and
retries, our count gets set back to 0.
On Mon, Sep 19, 2022 at 8:36 AM johnjcasey @.***> wrote:

Is there a particular reason to not use a pattern of outputting to failure
after N retries?
—
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/issues/21713#issuecomment-1251190989, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFAYJVMQ6RMY6N4A66IOEZTV7CCA3ANCNFSM5X4IOSHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Based on all of this, I think we can close this as working as intended.


Added offsets for Social Club & Epic Games version

Added a check if game is steam version or not (GetModuleHandleA("steam_api64.dll")), if it's not it uses the offsets for the Social Club version (which are the same offsets as the Epic Games version).

Sigs:
Call of netcat_insert_dedupe_addr
E8 ? ? ? ? 84 C0 74 39 8B 47 08
strlen_addr
48 8B C1 48 F7 D9
Call of netcat_insert_direct
E8 ? ? ? ? 8B 15 ? ? ? ? 0F B7 0D ? ? ? ?

have no way of verifying atm, but looks good to me!


Bot doesn't join server?

Sorry in advance, I'm a super noob when it comes to this stuff. As far as I know, I set it up right, but when I run the bot, it doesn't show up in online users and also doesn't take commands. This is what the cmd window looks like http://i.imgur.com/NXuvnQM.png . I tried to read through all the notes in the files as best I could and couldn't find a solution.

Can you show your entire traceback? You seem to have cut off parts after the bottom of your screenshot.

http://i.imgur.com/Eduwkaq.png

:inbox_tray: Updating your MusicBot dependencies
You will need to update your dependencies to resolve most common issues, and to be able to run the bot efficiently. Try and keep them updated as much as possible.
Windows - Open update_deps.bat
Linux - In your console, run python3.5 -m pip install --upgrade -r requirements.txt
Mac - Open update_macdeps.command in Terminal

opened update_deps.bat and ran it, it finished, I closed it, and still got http://i.imgur.com/YxmiBP1.png

Your bot has no permission to join the voice channel you're trying to make it autosummon to. You can clearly see in the screenshot. Can't join channel 'Overwatch', no permission
Give it all of the permissions on the server.

Could you explain how to do that? From what I read, it looked like when I have it my owner ID, it gave the bot all the permissions that I had but I now assume that is not the case.

You literally need to just give it a role that has permissions on the server e.g

How do I give it a role without it joining the server?

It easy go to what Jayden said and push that + and add the role and make sure to check the box I just checked all of them xp hope this works

Oh and don't forget to give the bot the role by right clicking it and push + and five the role what you put

Okay so update, my issue seemed to be with discord itself, not the bot. I had to go to the server members list to add a role because discord wasn't displaying online users with no role.


Configurable table style

simple = only separate header
strict = separate each line

https://discordapp.com/channels/592103645835821068/592118040179507207/769537630353227788


Feature/storage/appendblockoptions null

Looking to resolve https://github.com/Azure/azure-sdk-for-net/issues/29062

Also looks like a script ran and cleaned up unused TestRecords. Most of these test records have moved to our OpenWrite folder

API change check
API changes are not detected in this pull request.

Looks good!


PR: Add fallback validation for package wheels (Windows installer)

Description of Changes

[ ] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[ ] Included a screenshot or animation (if affecting the UI, see Licecap)

Workaround to check if wheels are available due to https://github.com/pypi/warehouse/issues/11949
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: dalthviz

Hello @dalthviz! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file installers/Windows/installer.py:

Line 196:80: E501 line too long (85 > 79 characters)
Line 209:80: E501 line too long (103 > 79 characters)

Seems like pynsist also depends on yarg to download and check wheels availability so this doesn't work and we need to wait for CDN PyPI purge


Binance perpetual - Memory leak happens after bot stopped

Describe the bug
Running out of memory by using Binance-perpetual connector within 60 minutes and then crash.
Here's the docker stats monitoring screenshots.

And screenshot of config command.

It happen both version of  1.4.0 and 1.6.0.
Thanks for your wonderful work.
Steps To Reproduce
// A concise description of the steps to reproduce the buggy behavior:
Here are the steps to reproduce the issue (see attachments in the section below):

Set up your configuration file with these parameters: ...
Start the bot using this command: ...
...

Screenshots
// If applicable, add screenshots to help explain your problem.
Release version
// Include your bot's version number (Can be found at the upper left corner of your CLI).
Attachments
// Please attach your config file and log file to help us debug. It would be difficult for us to help you without those!
WARNING: Do NOT publish any exchange API keys or your wallet's private key. Whoever has access to them may steal your assets!
Optional: your discord username:

I am affected with this as well. Any workaround till this gets fixed?

@pulkitsharma Could you please tell the version of the bot and sent your config file/screenshot?

@nikspz running version dev-1.8.0

Does not fail in 1 hour if you have high ram. Keep it running for a few hours. You will see memory usage going up until it fails with low memory most probably.

@pulkitsharma Could you also tell your installation type (Mac/Ubuntu, Docker/Source etc.)?
Bot also has some system requirements, depending on your\your cloud machine type.

I am using t2.large so 8gb+2vcpu with Ubuntu 20. I did not track after how many days the bot got killed. Twice it happened. But I know it increased over 1gb which is definitely a problem.

@pulkitsharma Thank you so much for the info, that help us investigate.
Do you run multiple bots when it happened?

I do run 2-3 other binance spot bots too. But they never go above 100mb each.

Noticed that bot have memory leak issue that could lead to bot crashed:
Steps to reproduce:
clone and install latest development branch
create/start perpetualMM strategy using binance_perpetual
Actual:
When you stopped the instance (or it stopped with killswitch) bot will slowly start to increase memory usage

version - 1.10
same issue.

Hi @Aaroncheongyuk @pulkitsharma @oliveagle good day,
We have a updated version of Binance perpetual connector on development branch and will be available on the upcoming version 1.12.0, just checking if its possible you guys can have a look if this fix the issue of the connector. Thank you

We have been monitoring our long term bot for a week and there were no signs of client crashed or sudden spike on memory using the updated binance perpetual connector on development branch. This version will be available on the upcoming version 1.12.0. We will be closing this ticket for now, else please let us know if issue re-occur. Thank you


COSMO does not compile on Daint

Since a couple of Daint COSMO does not compile on Daint once again because of the cray-libsci module:
generating compile information
make[1]: Leaving directory '/scratch/snx3000/jenkins/spack-stages/admin-daint/spack-stage-cosmo-c2sm-features-au4nhsndcea3ta5rer72afvhqqx7kgca/spack-src/cosmo/ACC/obj'
make[1]: Entering directory '/scratch/snx3000/jenkins/spack-stages/admin-daint/spack-stage-cosmo-c2sm-features-au4nhsndcea3ta5rer72afvhqqx7kgca/spack-src/cosmo/ACC/obj'
compiling kind_parameters.f90
Error:
Unable to find cray-libsci/20.09.1 libraries compatible with pgi/20.1.1.

This happened already a few times and could be solved by rearanging compiler precedence. But nothing changed in the yaml-files of Daint.
Strange is as well that a local instance of Spack is able to build COSMO, but the preinstalled as well as the admin-instance running on Jenkins cannot!

With the pre-installed spack instance it is working again to build cosmo. Tested master and c2sm-master. Currently building with Jenkins...


Improve hint format defaults

Depending on user settings ( $TERM, etc, etc ), default hint format is kind of unreadable

True, is there anyone who doesn't use "@fingers-hint-position: right"?

True, is there anyone who doesn't use "@fingers-hint-position: right"?

:point_up:

fixed on 0.6.3


how can i get values when multiple?

is there a way to get values when multiple?

Hi Gustavo,
I'm not really sure what you mean, would you mind sharing more details?

when you set a MultiStateToggleButton to multiselect, how can you get all values selected in an array?

Oh, sure, getStates returns a boolean array with the status of each button.


Max instance lifetime

Related to PR https://github.com/buildkite/elastic-ci-stack-for-aws/pull/836
If this number is set to 86400 or greater, the instance will terminated.

@nitrocode there's a merge conflict, would you mind updating your branch?

@keithduncan @yob anything blocking this? Just curious if this needs changes or is good to ship.

anything blocking this?

Better support for ASG initiated instance termination is the blocker I’m afraid.

Couldn't this still be added? The default behavior would be the same, and if a user enabled this, they are doing so at their own risk...

They could! And they are the approach I’m expecting us to use for this.
Unfortunately I’m not comfortable merging this without a reliable way to handle ASG initiated instance termination. The absence of this today already causes strife so I’m very hesitant to add another source of it. For those who are comfortable with the risk, it’s possible to apply a forked copy of this template to your stacks.
I’d also be thrilled to collaborate on a Lambda +SSM based lifecycle hooks implementation for the ASG events 😄 I believe it could solve several problems at once (e.g. #927 #838) if proven to be reliable, and enable features like agent instance quarantine/cordoning.

I'm not sure why #964 was closed, but we'd like to see this feature merged


fix ruby 2.7 warning with correct double splat usage for gitlab.rb

Identical to #1264 but for gitlab.rb
Cf. https://github.com/danger/danger/issues/1250#issuecomment-686392009

          1 Error

:no_entry_sign:
Please include a CHANGELOG entry.
You can find it at CHANGELOG.md.

          1 Warning

:warning:
Tests were not updated

  Generated by :no_entry_sign: Danger

👍🏻  - will ship with the next PR


Nodes not working since last nodered update

The Nodes worked fine until a few days ago.
Node-Red Version 2.2.2
Running on Home Assistant OS 8.1

https://github.com/hassio-addons/addon-node-red/releases/tag/v12.0.0


Remove escape of underscore in service file

Escaping the underscores in service files is not necessary

@lheimbs Hi, could you sign off the PR if it's ready for review? Thanks.

#sign-off

@lheimbs: I'm sorry - only the author of this article, @ahmetmithat, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@lheimbs Thanks for your contribution! Will merge the PR.


deposit processing using queues

Signed-off-by: umbrel<EMAIL_ADDRESS>

check build
trustlines monitoring missing
assets config example


[Question] Is it possible to combine 2 types of transitions?

Hi, Sorry for asking this here but I couldn't get an answer in the forum or SO.
Is there an easy way to combine 2 types of slide effects?
My page is divided into 2 sides and I would like each side to slide with a different effect.
Thanks!

No it is not possible

Ok thanks, are there examples how to set a custom slide effect? all the examples I find are related to old versions with outdate api


Bryton Gardia R300L not compatible

Hello, similarly to #20, Bryton uses it's own bluetooth characteristic for radar data.
It looks like there's no specific GATT characteristic for radar in the first place, so it's up to manufacturer to choose where to publish the radar data. Bryton uses characteristic f3641401-00b0-4240-ba50-05ca45bf8abc and the packet format is very similar, if not equal, to the ANT+ RDR standard.
It's unlucky that there's no common service.

Hey @OndrejBakan! Thanks for mentioning this, I agree it's a shame it's not standardised like some of the other protocols. Perhaps I should update the README to make it clear that the current Radar class probably only works for Garmin.
As always, you're welcome to implement the protocol for your device and I will review and merge it into the project. Tacx actually has something similar where it uses the ANT FE-C standard over BLE.

Hello again @zacharyedwardbull!
Could you, please, take a look at my code before I commit to finish it? Would you be OK with an implementation like this?
https://github.com/OndrejBakan/pycycling/commit/1429e4fb7393502e71cbda8cc039fcc804d0ab5e#diff-a8b9990e2e24f5af1f7e4b6b0b16951471db24c3f58c8ddf0fbe7d71cc61c0f8

Hey! Thanks for putting this together, looks really nice!
I like the way you've done it, but I think my preference would be to separate it out into a new file/service rather than combine with the Garmin one. My reasons are:

Unless you have a Garmin radar device we can't test the code easily to check it still works (unfortunately I don't have one either)
We're changing the number of elements in the RadarMeasurement namedtuple, which means if people are referring to elements using indexes then they would get different values, so it's not a backwards compatible change
Easier to reason about with less abstraction even if it means a little bit of duplicated code

Open to discussion though of course! Also @tensorturtle feel free to share your opinion since you wrote the Garmin support.
Cheers!


dedupe

I just spotted that there's a broken link in Connections -> Connect networks -> Private networks at "3. Run your Tunnel"

Some pages got deleted. Do we know if we have links to them? Can we replace deleted pages with redirects to new versions?

Some pages got deleted. Do we know if we have links to them? Can we replace deleted pages with redirects to new versions?

We should be alright here. Those pages were up for about 12 hours before decision was made to deduplicate.


chore: bdd-nh-1587383466 to 0.0.1

chore: Promote bdd-nh-1587383466 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Automatically blocking connection from untrusted peers

Once V2Ray knows the number of failed connections (both TCP and UDP based), it can actively block the peers that sends such connections.
Benefits:

Possible to reduce UDP-based attacks, which is hardly to do externally.
Simplified settings.

Yeah sure


Reset tasks details pane when tasks is deselected after a task is edited

Resolves #88

Pull Request Test Coverage Report for Build 227

0 of 11 (0.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 64.042%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/address/commons/events/ui/TaskPanelDeselectionEvent.java
0
2
0.0%

src/main/java/seedu/address/ui/TaskListPanel.java
0
3
0.0%

src/main/java/seedu/address/ui/TaskDetailsPane.java
0
6
0.0%

Totals

Change from base Build 225:
-0.3%

Covered Lines:
1578

Relevant Lines:
2464

💛  - Coveralls


ocamlPackages.cppo: 1.6.{5,6} → 1.6.7

Motivation for this change
Use Dune 2
Support for OCaml 4.02 and 4.12.0~α: https://github.com/ocaml-community/cppo/releases/tag/v1.6.7
Things done

[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

[ ] 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"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 107445 1

35 packages built:

acgtk
beluga
dot-merlin-reader
dune-release
eff
flow
framac
fstar
google-drive-ocamlfuse
jackline
libbap
libguestfs
liquidsoap
ocaml-crunch
ocaml-top
ocamlformat (ocamlformat_0_15_0)
ocamlformat_0_11_0
ocamlformat_0_12
ocamlformat_0_13_0
ocamlformat_0_14_0
ocamlformat_0_14_1
ocamlformat_0_14_2
ocamlformat_0_14_3
python37Packages.bap
python37Packages.guestfs
python38Packages.bap
python38Packages.guestfs
python39Packages.bap
python39Packages.guestfs
reason
satysfi
stog
vagrant
virt-top
why3


Error when using Hot Reload

Hello! I'd like to start off by saying this is my first Issue creation, and it is an honor to do it on something I am clearly finding much use of. That being said, this is a first, so bear with me as a get a hold of the nuances of Issue submission.
I implemented the example code into my dart, and it all works well, but any time I use Hot Refresh, like changing code and saving, I get a red screen with a stack error that only further breaks down the more I change code. The only fix is restarting the application.
test device: iOS 13.2 iPhone X
////////////////////////////////////////////////
Continuous Loop Break-Down Error (Restart Application to Fix)
Root Screen
class RootScreen extends StatefulWidget {
RootScreen();
@override
_RootScreenState createState() => _RootScreenState();
}
class _RootScreenState extends State {
GlobalKey _one = GlobalKey();
GlobalKey _two = GlobalKey();
@override
void initState() {
super.initState();
}
@override
Widget build(BuildContext context) {
return Scaffold(
body: Showcase(
key: _one,
description: ' Press and hold to create a new appointment ',
showcaseBackgroundColor: Colors.blueAccent,
textColor: Colors.white,
child: Container(
child: GestureDetector(
onLongPress: () {
ShowCaseWidget.of(context).startShowCase([_one, _two,]);
},
child: DayView(
...
),
),
),
),
);
}
@override
void dispose() {
super.dispose();
}
}
NavScreen
class NavScreen extends StatefulWidget {
@override
_NavScreenState createState() => _NavScreenState();
}
class _NavScreenState extends State {
final List _children = [
HomeScreen(),
RootScreen(),
];
@override
void initState() {
super.initState();
}
@override
Widget build(BuildContext context) {
return Container(
child: new Stack(children: [
new Container(
child: new Image.asset('assets/image.png', fit: BoxFit.contain),
),
Scaffold(
drawer: NavDrawer(),
appBar: appBar,
endDrawer: Drawer(),
body: ShowCaseWidget(
builder: Builder(
builder: (context) => _children[_currentIndex],
),
),
),
],
)
);
}
}
//////////////////////////////////////////////////////
Error:
════════ Exception caught by widgets library ═══════════════════════════════════════════════════════
The following assertion was thrown building Listener:
'package:flutter/src/rendering/box.dart': Failed assertion: line 1705 pos 18: 'debugDoingThisResize || debugDoingThisLayout ||
(RenderObject.debugActiveLayout == parent && _size._canBeUsedByParent)': is not true.
Either the assertion indicates an error in the framework itself, or we should provide substantially more information in this error message to help you determine and fix the underlying cause.
In either case, please report this assertion by filing a bug on GitHub:
https://github.com/flutter/flutter/issues/new?template=BUG.md
The relevant error-causing widget was:
GestureDetector file:///Users/home_screen.dart:710:18
When the exception was thrown, this was the stack:
#2      RenderBox.size. (package:flutter/src/rendering/box.dart:1705:18)
#3      RenderBox.size (package:flutter/src/rendering/box.dart:1711:6)
#4      _DayViewState.createEventsDrawProperties (package:flutter_week_view/src/day_view.dart:257:76)
#5      _DayViewState.didUpdateWidget (package:flutter_week_view/src/day_view.dart:109:5)
#6      StatefulElement.update (package:flutter/src/widgets/framework.dart:4694:58)
...
════════════════════════════════════════════════════════════════════════════════════════════════════
Reloaded 0 of 869 libraries in 247ms.
════════ Exception caught by widgets library ═══════════════════════════════════════════════════════
The following assertion was thrown building Builder:
'package:flutter/src/widgets/framework.dart': Failed assertion: line 4977 pos 14: '_dependents.isEmpty': is not true.
Either the assertion indicates an error in the framework itself, or we should provide substantially more information in this error message to help you determine and fix the underlying cause.
In either case, please report this assertion by filing a bug on GitHub:
https://github.com/flutter/flutter/issues/new?template=BUG.md
The relevant error-causing widget was:
Builder file:///Users/home_screen.dart:972:22
When the exception was thrown, this was the stack:
#2      InheritedElement.debugDeactivated. (package:flutter/src/widgets/framework.dart:4977:14)
#3      InheritedElement.debugDeactivated (package:flutter/src/widgets/framework.dart:4979:6)
#4      _InactiveElements._deactivateRecursively. (package:flutter/src/widgets/framework.dart:1944:15)
#5      _InactiveElements._deactivateRecursively (package:flutter/src/widgets/framework.dart:1946:6)
#6      ComponentElement.visitChildren (package:flutter/src/widgets/framework.dart:4560:14)
...
════════════════════════════════════════════════════════════════════════════════════════════════════
flutter: Brightness.light
════════ Exception caught by widgets library ═══════════════════════════════════════════════════════
The following assertion was thrown building _OverlayEntryWidget-[LabeledGlobalKey<_OverlayEntryWidgetState>#0dc3d](dirty, state: _OverlayEntryWidgetState#a7163):
'package:flutter/src/rendering/object.dart': Failed assertion: line 2330 pos 12: 'attached': is not true.
Either the assertion indicates an error in the framework itself, or we should provide substantially more information in this error message to help you determine and fix the underlying cause.
In either case, please report this assertion by filing a bug on GitHub:
https://github.com/flutter/flutter/issues/new?template=BUG.md
The relevant error-causing widget was:
MaterialApp file:///Users/lib/main.dart:51:12
When the exception was thrown, this was the stack:
#2      RenderObject.getTransformTo (package:flutter/src/rendering/object.dart:2330:12)
#3      RenderBox.localToGlobal (package:flutter/src/rendering/box.dart:2271:39)
#4      AnchoredOverlay.build.. (package:showcaseview/layout_overlays.dart:68:34)
#5      _OverlayEntryWidgetState.build (package:flutter/src/widgets/overlay.dart:177:34)
#6      StatefulElement.build (package:flutter/src/widgets/framework.dart:4623:28)
...
════════════════════════════════════════════════════════════════════════════════════════════════════

Can Confirm that this is directly an issue with another package I am using, flutter_week_view, my apologies


Operation Id and ParentId are rearranged or wrong

Repro Steps

Make a HTTP request to Kestrel with header: "Request-Id" : "CCCCC"
Kestrel will fill in the Activity.Current object:

Application Insights are configured (UseApplicationInsights()) so all requests are sent there. Additionally custom event is being reported. What I see in Azure -> Logs (Analytics) is following:

For request:

For custom event:

Why is this like that? For request both operationId and parentOperationId are the same. For customevents reported withing this context, these two are flipped. Totally can't understand this, this makes correlating events really hard. Is this a bug or I don't know something? What I would expect is Id and ParentId to be consistend across everything.
Version Info
SDK Version : "2.2.6"
.NET Version :	netcoreapp2.2
How Application was onboarded with SDK(VisualStudio/StatusMonitor/Azure Extension) : None
OS : Ubuntu
Hosting Info (IIS/Azure WebApps/ etc) :Kestrel

This is expected behavior.
When you send a request with "Request-Id" header "CCCCC", then "CCCCC" is used by hosting layer to generate its own ids which will be |CCCCC.random_ . And application insights respects it as well.
Why is this difficult to correlate? Application Insights UI should show this correlated correctly, and so can you if you use another tool for correlation.
All telemetry produced in the context of this Request will have same operation id. - "CCCCC", so they can be logically linked together.
Every telemetry generated in the context of this Request will have their operation.ParentId set to the ID of the RequestTelemetry.
The RequestTelemetry itself has parentID of "CCCCC" and its parent is external (extracted from request-id header)
This is the doc about correlation:
https://docs.microsoft.com/en-us/azure/azure-monitor/app/correlation

Every telemetry generated in the context of this Request will have their operation.ParentId set to the ID of the RequestTelemetry.

My screenshots show that it isn't true. For requests the operation_ParentId is different than for customEvents despite both being generated in the same context.


Chebyshev Polynomials of the third and fourth kinds

Perhaps these could be used for mixed boundary conditions, as they have different square root endpoint singularities and they are each others' Hilbert transforms.

The singularity structure for mixed boundary conditions is more complicated, but forthcoming work may allow for solution in spaces of direct sums of multiple weighted Jacobi polynomials and functions of the second kind.


Improve log entry for http connect result errors

This change uses BufferWriter to show the error name and number instead of just showing the error number for http connect result.

HttpSM.cc:5126:16: error: format string is not a string literal (potentially insecure) [-Werror,-Wformat-security]
    Log::error(lbw()
               ^~~~~
HttpSM.cc:5126:16: note: treat the string as an argument to avoid this
    Log::error(lbw()
               ^
               "%s",
1 error generated.

Blech. But it needs to be changed, there are other potential issues I overlooked with that. Just use the "%s" format and pass the lbw.

Just updated @SolidWallOfCode .

[approve ci autest]

Here is an example output for this:
'''
20180801.15h26m05s CONNECT: could not connect to <IP_ADDRESS>:1234 for 'http://localhost:1234/' (setting last failure time) connect_result=EIO: Input/output error [5]
'''

Can't cherry pick to 8.x because it doesn't have buffer writer.


showModal throws exception when dialog is inside a custom-element

I have a large web-components (using lit-element) application.  It will be used in a very closed environment, so I am able to control the browser support.  So far its been used exclusively with chrome.
I do use the <dialog> element inside a few components, and built in the use of the polyfill from the beginning.  These elements exclusively use showModal() to open dialogs when needed.
I have been asked to check for IOS support on ipads - but with them it has been impossible to debug why things didn't work, so I just starting using Safari on the mac.
However using the debugger on my mac with Safari I have discovered that showModal() tries to check whether the dialog exists by doing a document.body.contains(this.dialog_) call.  Node.contains() doesn't include elements inside the shadowRoot of a contained element, so the call throws an exception.

Thanks for the report. The polyfill should work inside a shadow root (even if automatically injecting the CSS is up in the air). I'll look at this soon.

Removing those checks, it seems to work for me (CDN dist here), except on form submission the page reloads, but maybe that's unrelated.

@samthor @nalply can the nalply/dialog-polyfill fork be merged to make the dialog usable inside shadow DOM?

I'll look at this soon.
My concern is that when you're in a shadow root, it's very likely that
something is causing layout, so the position: fixed code won't work. But
we detect that anyway and complain if you're doing something wrong :)
On Thu, 27 Feb 2020 at 18:30, Adrian Nita<EMAIL_ADDRESS>wrote:

@samthor https://github.com/samthor @nalply https://github.com/nalply
can the nalply/dialog-polyfill fork be merged to make the dialog usable
inside shadow DOM?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleChrome/dialog-polyfill/issues/186?email_source=notifications&email_token=AAA5DEFNBW3PKL2LWLZY7MLRE5T3BA5CNFSM4IWWUSZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENDIO2Q#issuecomment-591824746,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA5DED5QHDHSPHBWIULNIDRE5T3BANCNFSM4IWWUSZA
.

@samthor @nalply I think that any layout issue inside a shadow root is acceptable compared to not working at all in that context. Would be nice and helpful to see that fix merged. Thanks :)

Whats the best way forward? Looks like this is not on anyone's list to fix.
Should we use the  nalply/dialog-polyfill fork? Or is another polyfill out there?

If there's a PR that fixes this I can take a look very easily.
On Tue, 7 Apr 2020, 00:57 Karl Waclawek<EMAIL_ADDRESS>wrote:

Whats the best way forward? Looks like this is not on anyone's list to fix.
Should we use the nalply/dialog-polyfill fork? Or is another polyfill out
there?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleChrome/dialog-polyfill/issues/186#issuecomment-609846606,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA5DEEUZURMOGJRLIXHN23RLHUXHANCNFSM4IWWUSZA
.

I just forked and will try to do the minimal changes for this issue to be fixed. @samthor so you know why the dist files in the repo seem not to be up-to-date with index.js?

I'm not sure, but I can build it with Closure if needed. IIRC the non-dist
version should be fine for testing.
On Wed, 8 Apr 2020 at 22:43, Adrian Nita<EMAIL_ADDRESS>wrote:

I just forked and will try to do the minimal changes for this issue to be
fixed. @samthor https://github.com/samthor do you know why the dist
files in the repo seem not to be up-to-date with index.js?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleChrome/dialog-polyfill/issues/186#issuecomment-610936635,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA5DEAYEU42WQQ7BWOUMA3RLRWP3ANCNFSM4IWWUSZA
.

@samthor please check pull request https://github.com/GoogleChrome/dialog-polyfill/pull/194 that fixes this issue. Thanks!

I've published 0.5.1 which includes this.
On Fri, 10 Apr 2020 at 20:48, Adrian Nita<EMAIL_ADDRESS>wrote:

@samthor https://github.com/samthor please check pull request #194
https://github.com/GoogleChrome/dialog-polyfill/pull/194 that fixes
this issue. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleChrome/dialog-polyfill/issues/186#issuecomment-611981317,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA5DEDIO6TTTYZXNIT7DJLRL32RVANCNFSM4IWWUSZA
.

@samthor I believe this issue can be closed as per 0.5.1 ?

I bumped up package.json to 0.5.1.


[auto-merge] branch-23.02 to branch-23.04 [skip ci] [bot]

auto-merge triggered by github actions on branch-23.02 to create a PR keeping branch-23.04 up-to-date. If this PR is unable to be merged due to conflicts, it will remain open until manually fix.

SUCCESS - auto-merge


Update: en-gb.json

Added keys for:

"Settings.AudioInputFilteringSettings.UseVoiceNormalization"
"Settings.AudioInputFilteringSettings.NormalizationThreshold"

As they were using the fallback, American English, keys.
(Also, I don't know if I should add my own name to the author's list as it's only two keys)

Anyone with any level of contribution should be in the authors list, afaik, so you're fine to be there

I mean I've removed myself anyway because I don't really want credit for something so petty (i literally just changed "normalization" to "normalisation") but yeah I can see both POVs, even for a spelling fix

You're fine to add your name to the list! I'm merging this now, if you do want credit, then feel free to open another PR and reference this (even if you don't add any new keys in that).

Thank you for the update by the way!


problem with documentation for immersive engineering

I have been trying to add custom minerals and the documentation doesn't seem to be correct.
here what i have
import mods.immersiveengineering.Excavator;
mods.immersiveengineering.Excavator.addMineral("Africite", 50, 0.0005, ["ore:orevibranium", "ore:oreDiamond","ore:orepalladium","ore:oreTitanium"], [35, 15, 25, 25]);
i also tried the ore dictionary names
which didnt work

As far as I know you either use the item, so mod:item or the ore dictionary directly, so oreDiamond

Is this still an issue?

Closing this because no answer received.
If this is still an issue, please respond here and I will reopen this issue.


seccomp_init() failed

Describe the bug

I cannot install Okd using crc on fedora 37 libvirtd/qemu.
[root@darkstar qemu]# tail -f crc.log
-device '{"driver":"isa-serial","chardev":"charserial0","id":"serial0","index":0}' \
-audiodev '{"id":"audio1","driver":"none"}' \
-vnc <IP_ADDRESS>:0,audiodev=audio1 \
-device '{"driver":"cirrus-vga","id":"video0","bus":"pcie.0","addr":"0x1"}' \
-object '{"qom-type":"rng-random","id":"objrng0","filename":"/dev/urandom"}' \
-device '{"driver":"virtio-rng-pci","rng":"objrng0","id":"rng0","bus":"pci.5","addr":"0x0"}' \
-msg timestamp=on
2023-04-04T14:08:33.267526Z qemu-system-x86_64: -device {"driver":"vhost-user-fs-pci","id":"fs0","chardev":"chr-vu-fs0","tag":"dir0","bus":"pci.1","addr":"0x0"}: Failed to write msg. Wrote -1 instead of 12.
2023-04-04T14:08:33.267547Z qemu-system-x86_64: -device {"driver":"vhost-user-fs-pci","id":"fs0","chardev":"chr-vu-fs0","tag":"dir0","bus":"pci.1","addr":"0x0"}: vhost_backend_init failed: Protocol error
2023-04-04 14:08:33.333+0000: shutting down, reason=failed
---
[root@darkstar qemu]# tail -f crc-fs0-virtiofsd.log
seccomp_init() failed
virtio_session_mount: Waiting for vhost-user socket connection...
virtio_session_mount: Received vhost-user socket connection
seccomp_init() failed
virtio_session_mount: Waiting for vhost-user socket connection...
virtio_session_mount: Received vhost-user socket connection
seccomp_init() failed
virtio_session_mount: Waiting for vhost-user socket connection...
virtio_session_mount: Received vhost-user socket connection
seccomp_init() failed

Version

IPI
crc_okd_libvirt_4.12.0-0.okd-2023-02-18-033438_amd64
How reproducible

$crc setup
INFO Using bundle path /home/jj/.crc/cache/crc_okd_libvirt_4.12.0-0.okd-2023-02-18-033438_amd64.crcbundle
INFO Checking if running as non-root
INFO Checking if running inside WSL2
INFO Checking if crc-admin-helper executable is cached
INFO Checking if running on a supported CPU architecture
INFO Checking minimum RAM requirements
INFO Checking if crc executable symlink exists
INFO Creating symlink for crc executable
INFO Checking if Virtualization is enabled
INFO Checking if KVM is enabled
INFO Checking if libvirt is installed
INFO Checking if user is part of libvirt group
INFO Checking if active user/process is currently part of the libvirt group
INFO Checking if libvirt daemon is running
INFO Checking if a supported libvirt version is installed
INFO Checking if crc-driver-libvirt is installed
INFO Checking crc daemon systemd service
INFO Checking crc daemon systemd socket units
INFO Checking if systemd-networkd is running
INFO Checking if NetworkManager is installed
INFO Checking if NetworkManager service is running
INFO Checking if dnsmasq configurations file exist for NetworkManager
INFO Checking if the systemd-resolved service is running
INFO Checking if /etc/NetworkManager/dispatcher.d/99-crc.sh exists
INFO Checking if libvirt 'crc' network is available
INFO Checking if libvirt 'crc' network is active
INFO Checking if CRC bundle is extracted in '$HOME/.crc'
INFO Checking if /home/jj/.crc/cache/crc_okd_libvirt_4.12.0-0.okd-2023-02-18-033438_amd64.crcbundle exists
Your system is correctly setup for using CRC. Use 'crc start' to start the instance

then
crc start
INFO Checking if running as non-root
INFO Checking if running inside WSL2
INFO Checking if crc-admin-helper executable is cached
INFO Checking if running on a supported CPU architecture
INFO Checking minimum RAM requirements
INFO Checking if crc executable symlink exists
INFO Checking if Virtualization is enabled
INFO Checking if KVM is enabled
INFO Checking if libvirt is installed
INFO Checking if user is part of libvirt group
INFO Checking if active user/process is currently part of the libvirt group
INFO Checking if libvirt daemon is running
INFO Checking if a supported libvirt version is installed
INFO Checking if crc-driver-libvirt is installed
INFO Checking crc daemon systemd socket units
INFO Checking if systemd-networkd is running
INFO Checking if NetworkManager is installed
INFO Checking if NetworkManager service is running
INFO Checking if dnsmasq configurations file exist for NetworkManager
INFO Checking if the systemd-resolved service is running
INFO Checking if /etc/NetworkManager/dispatcher.d/99-crc.sh exists
INFO Checking if libvirt 'crc' network is available
INFO Checking if libvirt 'crc' network is active
INFO Loading bundle: crc_okd_libvirt_4.12.0-0.okd-2023-02-18-033438_amd64...
INFO Creating CRC VM for okd 4.12.0-0.okd-2023-02-18-033438...
INFO Generating new SSH key pair...
INFO Generating new password for the kubeadmin user
INFO Starting CRC VM for okd 4.12.0-0.okd-2023-02-18-033438...
 We will look into this, but since this is using the okd preset, we can not actually help.

The machine hasn't started at all, so its probably not related to OKD contents


xformer not found

I am having some xformer related errors when I start the webui. It doesnt seem to be installed. Current GPU used is A4000. It doesn't seem to keep me from running webui but I don't think it s optimized like that.

Try building xformers using the block in the tools section.


Arrays hashkeys?

I think I found a bug and a solution. if the object has an array it will give this error
Exception while invoking method '/contents/insert' Error: key $$hashKey must not start with '$'

solution
# cleanup hashkeys
item = angular.copy(scope.content)
angular.forEach item.images, (image) ->
 delete image.$$hashKey
$rootScope.contents.save(item)

@berkaey yes you are right, we had that code and removed it because of efficiency (and because we forgot about the $$hashkey...)
I've added it here:
https://github.com/Urigo/angular-meteor/commit/86cde85f30f9033ce51d29ba85acac6f3d93c015
We don't need to do 'foreach' and remove the $$hashkey because angualr.copy does that for us.
But, the AngularMeteorCollection.save function has to be very efficient and angular.copy is a little expensive.
the reason we have to do angular.copy is that otherwise we are keeping the reference and angular bring back the $$haskey even before we save it...

We know we don't need to check for $$hashkey if the save came from the autobind as $watch remove that from the results, so I've added a flag to the end of the parameters
Checking if we need to do angular.copy by running over all the nested object looks to me exactly as efficient as doing angular.copy (the only difference is the copying action which is not expensive).

What do you think about it @netanelgilad @barbatus ?
do you have any suggestions how to make it more efficient?

First of all angular.copy is a deep copy which means it might be really slow.
Secondly, Angular seems to add $$hashkey when an array is used in ng-repeat only which means it's not all the time. As a result, I'd suggest to leave it as a responsibility of user to clean an object before saving.

As a second I thought, I had potentially even more useful idea: since in most cases objects to save will have simple structure, it makes sense to check and clean only two first levels (object itself and objects in any subarray) and only otherwise throw an exception on the collection.insert if it's somewhere deeper.

I came across this issue myself and the problem is bigger when you are using autosave: you cannot clean the $$hashKey manually. As a quick fix I looked for the $$hashKey on the setDiff and just deleted it, but I don't think is the best approach to it.
I like @Urigo approach, but I'm also concerned abut the performance, though for my case it wouldn't be a big deal as the objects & collection are very small.
I did some tests and Angular adds the $$hashKey to the root of the object and not the inner objects, though I still think that in some cases the inner objects can get such key.
How about this: AngularMeteorCollection.save()'s 3rd param (hashKeysMightExist) is a multi-value param:

true (default): removes the $$hashKey from the root only using a simple delete item.$$hashKey
"recursive": uses angular.copy()
function(item): have a custom function that will be the one that eliminates any undesired keys
false: do nothing

@auterium If it's explicit save -- it's ok but what about autosave. How do you suppose to pass all that params to autosave? Also, how that function (3d in the list) is supposed to be used? Should it be called on failure and then collection insert will restart or before calling insert? I guess the latter - then it's the same as cleaning it before passing to save, isn't it.

@barbatus sorry I didn't explained myself correctly.
I forgot to mention that those options should be passed to the $meteor.collection() or $meteor.object() methods, so when autosave is invoked, it's done so with such options.
The idea behind the cleaning function is to be used with the autosave.

Just a small issue. @Urigo if you remember we didn't remove the angular.copy because of performance (though it was also considered). We removed it because it caused problems with CollectionFS object. If you try to save a File object (like CollectionFS does) then angular.copy causes problems when copying a File object.
So anyway, angular.copy should be avoided for those situations..


please update issue template regarding maintenance

as it shows "PiVPN is no longer maintained, see the README" which is different from readme

done ...
https://github.com/pivpn/pivpn/commit/e4fd3703788a1095e3b4f6c186eefc94e82b2a78
closing.

thanks


Rechunk Polars

Polars should be rechunked before doing the benchmarks. Currently a rechunk is triggered on benchmarking of the operations, whereas for instance pandas starts from a single chunk.

All done, with updated runs and plots


ACM-2758

/retest

/retest

@chenz4027 Just a question about the cards, the design https://marvelapp.com/prototype/1920h2h9/screen/90598599 has this:

Did we change it at the last minute?

Yep @fxiang1 I think so, as per latest comment in the Jira

@chenz4027 Sorry meant to ask your text doesn't match the design. Is it suppose to match?