Golang: fails to communicate with Google logs unless HTTP/2 disabled

Under Go 1.6, attempts to retrieve the STH from any Google log fail, unless I disable HTTP/2 by setting GODEBUG=http2client=0.  I don't have trouble with any of the non-Google logs.
Test program:
package main

import (
        "github.com/google/certificate-transparency/go/client"
        "fmt"
)

func try (url string) {
        c := client.New(url)
        _, err := c.GetSTH()
        if err != nil {
                fmt.Printf("%s: Error: %s\n", url, err)
        } else {
                fmt.Printf("%s: No error\n", url)
        }
}

func main () {
        try("https://ct.googleapis.com/pilot")
        try("https://ct.googleapis.com/aviator")
        try("https://ct.googleapis.com/rocketeer")
        try("https://ct.googleapis.com/submariner")
        try("https://ct1.digicert-ct.com/log")
}

Results under Go 1.5 OR Go 1.6 with GODEBUG=http2client=0 set:
https://ct.googleapis.com/pilot: No error
https://ct.googleapis.com/aviator: No error
https://ct.googleapis.com/rocketeer: No error
https://ct.googleapis.com/submariner: No error
https://ct1.digicert-ct.com/log: No error

Results under Go 1.6:
https://ct.googleapis.com/pilot: Error: invalid character '<' looking for beginning of value
https://ct.googleapis.com/aviator: Error: invalid character '<' looking for beginning of value
https://ct.googleapis.com/rocketeer: Error: invalid character '<' looking for beginning of value
https://ct.googleapis.com/submariner: Error: invalid character '<' looking for beginning of value
https://ct1.digicert-ct.com/log: No error

Specifically, the Google server is returning the error Error 400 (Bad Request)!!1 [sic].

This is probably a bug in the net/http library in Go 1.6 and should probably go over in the https://gihtub.com/golang/go issues.

https://github.com/google/certificate-transparency/blob/master/go/client/logclient.go#L127 appears to be the problem. Removing that line should work around it.

Removing the Keep-Alive header fixed it for me - thanks!

Go 1.6 fix is still needed. Keep-Alive (and other Connection-Specific headers) should be ignored/removed if the underlying transport is HTTP2, thanks to https://http2.github.io/http2-spec/#rfc.section.<IP_ADDRESS>:
"An endpoint MUST NOT generate an HTTP/2 message containing connection-specific header fields; any message containing connection-specific header fields MUST be treated as malformed. ... Such intermediaries SHOULD also remove other connection-specific header fields, such as Keep-Alive, Proxy-Connection, Transfer-Encoding, and Upgrade..."

I made a ticket for the net/http2 fix: https://github.com/golang/go/issues/15085

This got fixed in x/net/http2 https://go-review.googlesource.com/#/c/21482/ and was merged in as https://github.com/golang/net/commit/024ed629fd292398cfd43c9678a5bf004f7defdc

So we can close this, right?

Wellllll, the problem is going to be around until whenever net/http gets this commit dropped in it. I don't know when it'll go out. Not sure if it'll be in 1.6.2 (the release coming soon after the 1.6.1 security fix). We might be waiting until 1.7 and making folks include and config http2 themselves with mysterious errors before they do so.
Could totally be wrong and it's in the plans for 1.6.2 and maybe in a few weeks this is all moot.

Ah, thanks. I was confused about the relationship between x/net/http2 and net/http. Looks like Boulder will just have to disable HTTP2 temporarily till the fix is part of a Go release.

golang/go#15085 is tagged with the Go1.7 milestone, so I assume the fix is not going to make it into 1.6.2.  Thus I'm inclined to re-open this issue, though I'm not sure what the CT package should do.  Drop the Keep-Alive header? Disable HTTP/2?

I think the CT package should drop the Keep-Alive header. That way things continue to work, but less efficiently, in older versions, but get an efficiency improvement in newer versions of Go.


Clean up some existing varlink endpoints

Going through and adding options (like tls-verify, signature option, etc)
to some varlink endpoints (like push/pull) many of which had not been
updated since their original authoring.
Signed-off-by: baude<EMAIL_ADDRESS>

/approve

Changes LGTM other than @mheon's question.  Tests aren't hip atm.

ya, job is showing as 'triggered' but not actually doing anything: https://cirrus-ci.com/build/4570116530372608
Some other jobs are running fine though, and I don't see any google problems.  It looks like there was a temporary hiccup in Cirrus CI, though can't say for sure b/c I've never seen one until now :smile: I'll follow up with their support.
I bet if @baude re-pushes the Cirrus tests will start going.

fixed up now

can someone merge this please


feat(core): add app-region: drag to data-tauri-drag-region

Use app-region: drag on supported platforms (currently only WebView2 123+ on Windows)

I don't think we need to, we won't get events in non client region, so if app-region: drag is working, it should just skip the other handlers

We probably need to get the webview version and embed it into this script at startup then disable the custom JS handler on Windows when using WebView2 123+

I don't think we need to, we won't get events in non client region, so if app-region: drag is working, it should just skip the other handlers

That's enough I guess


NIFI-3361 Upgrade Jetty version to 9.4.2

Some of the existing tests for GetHTTP and PostHTTP needed to be updated or ignored; this is not because of any change to these processors, but rather the existing Jetty test server which was used to validate them.
Jetty 9.4.0+ only supports TLSv1.2, so we should be very clear in the release notes for NiFi 1.2.0 that TLSv1/1.1 are no longer supported.
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Will review...

Thanks @alopresto. This has been merged to master.


Migrating a game from the usual Launcher, is it possible?

Is it possible to use Collapse raw without keeping the Launcher of each game? What to do if I have the game fully installed and neither Hoyoverse app where it's bundled nor if Collapse can detect the install?
I could be able to execute Genshin raw like eating a fish without cooking it.

You should be able to oress Install/Locate Game on the game screen then go to that folder. If it doesn't detect it, make sure the app.info file inside GenshinImpact_Data is there and contains these lines
miHoYo
Genshin Impact

You should be able to oress Install/Locate Game on the game screen then go to that folder. If it doesn't detect it, make sure the app.info file inside GenshinImpact_Data is there and contains these lines
miHoYo
Genshin Impact

Also please make sure the Genshin impact game folder is not inside the same folder that Collapse Launcher is installed in, move it to another folder

I just done moving the game folder out of Collapse Launcher install directory, and whilst found the app.info file, it cannot detect.
Moreover, both apps don't work.

I just done moving the game folder out of Collapse Launcher install directory, and whilst found the app.info file, it cannot detect. Moreover, both apps don't work.  There is guaranteed two lines in the file.

I had to uninstall the typical Genshin Impact launcher and switch to yours to diagnose this. Still, everyone mandates me a game reinstall?

With Collapse console in the setting turned on, can you record the process on which you use the locate game function?

Status: not installed. Whilst there's all files + English (US) audio
It's still say the not-installed state.

Can you send me what the contents of your config.ini file?

[General]
uapc=ded3ab92053a

Ahhh, seems your config.ini file isn't complete
Here's mine, copy it to your installation. Make sure the version string is the same as the one you've installed right now (you can check by opening the game and see the string on the bottom left followed by OSRel)
[General]
channel=1
cps=mihoyo
game_version=4.0.0
sub_channel=0
sdk_version=
plugin_7_version=1.0.0

Confirmed, thanks.


Roll Flutter Engine from c40b0b602822 to d19098fca7b6 (4 revisions)

https://github.com/flutter/engine/compare/c40b0b602822...d19098fca7b6
2024-10-31<EMAIL_ADDRESS>Roll Skia from 3c62d4a94d78 to 7e79a516284b (1 revision) (flutter/engine#56252)
2024-10-31<EMAIL_ADDRESS>[Impeller] use primitive restart for faster tessellation: write directly into host buffer. (flutter/engine#56173)
2024-10-31<EMAIL_ADDRESS>Skwasm single threaded (flutter/engine#56206)
2024-10-31<EMAIL_ADDRESS>[Impeller] expose reference to tessellator instead of shared_ptr. (flutter/engine#56244)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC chinmaygarde<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Label - onRemove cannot be triggered independently of onClick

Steps

Create Label element with onRemove and onClick.
Click the onRemove icon.

Expected Result
onRemove is triggered.
Actual Result
onRemove is triggered, then onClick is triggered.
Version
latest
Original behavior in Label
handleClick = (e) => {
    const { onClick } = this.props

    if (onClick) onClick(e, this.props)
  }

  handleRemove = (e) => {
    const { onRemove } = this.props

    if (onRemove) onRemove(e, this.props)
  }

Proposed Change
Have handleClick double check if handleRemove has been called. Should I think of a better solution, or is this not a good idea?
handleClick = (e) => {
    const { onClick } = this.props

    if (this.removed) {
      this.removed = false
    } else if (onClick) {
      onClick(e, this.props)
    }
  }

  handleRemove = (e) => {
    const { onRemove } = this.props

    if (onRemove) {
      this.removed = true
      onRemove(e, this.props)
    }
  }

The same problem exists when you pass an Icon via children that has a onClick prop.
Handle the clicks:
 handleClick = (e) => {
    const { onClick } = this.props

    if (this.removed) {
      this.removed = false
    } else if (this.iconClicked) {
      this.iconClicked = false
    } else if (onClick) {
      onClick(e, this.props)
    }
  }

  handleRemove = (e) => {
    const { onRemove } = this.props

    if (onRemove) {
      this.removed = true
      onRemove(e, this.props)
    }
  }

  handleIconClick = (onClick) => (e) => {
    this.iconClicked = true
    onClick(e, this.props)
  }

Update the onClick prop passed to icon with our handleIconClick version, this was my solution (could be more elegantly written maybe?):
if (!_.isNil(children)) {
      let childrenWithIcon
      _.forEach([...children], (child, i) => {
        if(typeof child === 'object' && child.props.as == 'i' && child.props.onClick) {
          const onClick = this.handleIconClick(child.props.onClick)
          const props = { ...child.props, onClick }
          const iconChild = { ...child, props }
          childrenWithIcon = [...children]
          childrenWithIcon[i] = iconChild
          return false
        }
      })
      return <ElementType {...rest} className={classes} onClick={this.handleClick}>{childrenWithIcon || children}</ElementType>
    }

This is the expected and intended behavior.  The icon is a child of the Label, so a click on the remove icon will propagate up to the Label itself as a click.  They are both click events.
If you'd like to stop the event from propagating up to the click handler when the remove icon is clicked, you should call e.stopPropagation() in the remove handler:
http://codepen.io/levithomason/pen/qrydWz


"Similarity" search for 'a list -> 'a in FCS causes an exception

Thank you for the report.
I heard from @hafuu that the root cause of this issue has already been fixed in the latest version of FSharpApiSearch (FYI: this code).
So this will be fixed very soon.


Extract PIN protocol implementation

This patch extracts the implementation of PIN protocol one from the ctap2 and state modules into the ctap2.pin module.  This makes it easier to add support for PIN protocol two in the future.
Functionally, the extracted implementation should be mostly identical to the old implementation, expect for some smaller changes for improved compliance with the specification (error codes, explicit token resets).

Minor issue of spec compliance:

Minimum PIN Length: 4 code points.

In decrypt_pin_check_length we check 4 bytes, not 4 code points. The max is still 63 bytes, so the check should be:
        let pin_length = pin.iter().position(|&b| b == b'\0').unwrap_or(pin.len());
        let pin_data = &pin[..pin_length];
        let pin_utf8 = core::str::from_utf8(pin_data)?;
        let pin_len_codepoints = pin_utf8.chars().count();

        if pin_len_codepoints < 4 || pin_length > 63{
            return Err(Error::PinPolicyViolation);
        }

Good catch.  I’d prefer to fix that in a separate PR as it is not directly related to the changes in this one.


Nightly r-docs failing: maybe because OPENDP_LIB_DIR is undefined?

Describe the bug
Nightly is failing, and has been since last month, though it looks like the failure has changed over time, and is now near the end of the CI chain.
To Reproduce
ci logs:
using C compiler: ‘gcc (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0’
expected either a src/binary.tar.xz, src/source.tar.xz (run bash tools/r_stage.sh) or OPENDP_LIB_DIR to be set
make: *** [Makevars:16: rust_build] Error 1
ERROR: compilation failed for package ‘opendp’
* removing ‘/tmp/RtmpaKQVmk/devtools_install_17a51673c2d6/opendp’
Error in `(function (command = NULL, args = character(), error_on_status = TRUE, …`:
! System command 'R' failed
---
Exit status: 1
stdout & stderr: <printed>
---
Backtrace:
 1. devtools::document("R/opendp")
...

More broadly...
Do we need to move more of Nightly into the normal PR CI so that we don't find ourselves surprised by failures like this?

More debugging notes on:

#1613


Duplicated tags with v prefix

There are many duplicated tags with v prefix in this repo
They should be cleaned up. I think they were created when people worked on release drafts as it suggests tag starting with v.
People should use existing tag instead, but they need to wait for https://github.com/quarkus-qe/quarkus-test-framework/actions/workflows/release.yml action to finish.
Tags with v prefix should be removed, @mjurc @pjgg
git tag -l | sort
0.0.1
0.0.10
0.0.11
0.0.12
0.0.13
0.0.14
0.0.15
0.0.16
0.0.2
0.0.3
0.0.4
0.0.7
0.0.8
0.0.9
1.0.0.Final
1.0.1.Final
1.0.2.Final
1.0.3.Final
1.0.4.Final
1.0.5.Final
1.1.0.Beta.1
1.1.0.Beta2
1.1.0.Beta3
1.1.0.Beta4
1.1.0.Beta5
1.1.0.Final
v0.0.10
v0.0.11
v0.0.12
v0.0.13
v0.0.14
v0.0.15
v0.0.16
v0.0.2
v0.0.3
v0.0.4
v0.0.5
v0.0.6
v0.0.7
v0.0.8
v0.0.9
v1.0.1
v1.0.2
v1.1.0.Beta.1
v1.1.0.Beta4
v1.1.0.Beta5

It doesn't seem to happen anymore. We can carefully clear old tags as there is no advantage in keeping v1 and v0 (and imo also 0.x) tags


stamp local_xy_origin message picked from list with now()

LGTM; do you know if this is also an issue for the ROS2 version in the dashing-devel branch?  If so, could you cherry-pick this over there?

LGTM; do you know if this is also an issue for the ROS2 version in the dashing-devel branch? If so, could you cherry-pick this over there?

no idea. Currently don't have an environment for ROS2 setup.

LGTM; do you know if this is also an issue for the ROS2 version in the dashing-devel branch? If so, could you cherry-pick this over there?

no idea. Currently don't have an environment for ROS2 setup.

You should be able to just cherry-pick it onto it without trying to test or build it.

LGTM; do you know if this is also an issue for the ROS2 version in the dashing-devel branch? If so, could you cherry-pick this over there?

no idea. Currently don't have an environment for ROS2 setup.

You should be able to just cherry-pick it onto it without trying to test or build it.

Don't think we have an origin_manager.py for ros2


Add std.experimental.allocator files to win32.mak.

Closing to re-create the PR because I got latest, found typed.d was removed, fixed win32.mak, and subsequently messed up the merge of the PR.


Can't load another url when video is playing

Thank for create an awesome library!
I'm using your library in my Laravel project, it's work.
But, when i use stream function my web can't open another urls and resources.
And, in Safari on MacOS, it works well in localhost. But, when i publish to heroku hosting, it can't play video (the same direct link work well on Chrome).
This is stream function:
$youtube = new \YouTube\YouTubeStreamer();
$youtube->stream('https://r4---sn-n4v7knll.googlevideo.com/videoplayback?...');

The returning video urls are only for a specific client ip address. Your heroku app must parse a video url and then stream it. Try the following code, the $url variable is the full link to your YouTube video (the code is for the latest version):

$youtube = new \YouTube\YouTubeDownloader();

$links = $youtube->getDownloadLinks($url);

$best = $links->getFirstCombinedFormat();

if ($best) {
    $youtube = new \YouTube\YouTubeStreamer();
    $youtube->stream($best->url);
}


1718/bnd maven plugin

Update project to use bnd-maven-plugin

@adamcin when verifying my build, oakpal is complaining:
[ERROR] Failed to execute goal net.adamcin.oakpal:oakpal-maven-plugin:2.1.0:scan (oakpal-scan) on project acs-aem-commons-ui.apps: Failed to execute package scan. Failed to resolve cndNames.: Failed to find node type definition on classpath for cndName nodetypes/nodetypes.cnd -> [Help 1]

however the node this path exists (no changes to it from master) .. <project root>/src/test/resources/nodetypes/nodetypes.cnd
Any idea why? I'm guessing its something simple I'm overlooking... 🤦

@kwin / @joerghoh - if you get a moment cant you take a quick pre-review look at this? Your maven-foo is stronger than mine and want to make sure I didn't botch syntax (esp in CDATA) and/or there isn't a more efficient configuration.

@adamcin when verifying my build, oakpal is complaining:
[ERROR] Failed to execute goal net.adamcin.oakpal:oakpal-maven-plugin:2.1.0:scan (oakpal-scan) on project acs-aem-commons-ui.apps: Failed to execute package scan. Failed to resolve cndNames.: Failed to find node type definition on classpath for cndName nodetypes/nodetypes.cnd -> [Help 1]

however the node this path exists (no changes to it from master) .. <project root>/src/test/resources/nodetypes/nodetypes.cnd
Any idea why? I'm guessing its something simple I'm overlooking... 🤦

@davidjgonzalez Can you confirm that the cnd is being copied to target/test-classes/nodetypes/nodetypes.cnd?

@adamcin I do not see these target/test-classes/nodetypes/nodetypes.cnd in my ui.apps project.
I actually tried doing an mvn clean install from master and ran into the same issue.

@adamcin so this is strange (and maybe why i haven't seen this before). Everything seems to work in intellij, however when i run the command from the command line it fails.
My version of maven is the same 3.6.3 (command line is 3.6.3_1), and I'm using the same version of java. That help ring any bells?

Closing for now. To pick up post 5.0


Finetuning configuration

What is the parameter setting for fine-tuning? In particular, what learning rate did you use?

The pre-trained models are finetuned for additional 30 epochs with the cosine annealing learning rate from 1e-4 to 1e-5.

Thanks for the info. I was still unable to reproduce the finetuning results. Anything important to take care of other than learning rate, e.g., balancing the real / synthetic examples?


Axis workflow

points(1,2, side=c(1,2,3,4) %>%
axis(c(3,4), labels=FALSE)

Support many axes.

So, still need to:

[ ] Support many axis
[ ] Support multiple axis on the same side
[ ] Support single axis calls

So, say I want major and minor axis tickmarks on the x axis:
gs <- gsplot() %>%
      points(y=c(3,1,2), x=1:3, xlim=c(0,NA),ylim=c(0,NA),
             col="blue", pch=18, legend.name="Points", xlab="Index") %>%
     axis(side=c(3,4), labels=FALSE) %>%
     axis(side=1, at=seq(0,3,0.1), tcl=0.1)

Closing this and breaking it into individual issues.


Various extra USFM and TN checks

Pull Request Test Coverage Report for Build 403

16 of 73 (21.92%)  changed or added relevant lines in 3 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.1%) to 53.396%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

webhook.py
0
1
0.0%

callback.py
0
2
0.0%

preprocessors/preprocessors.py
16
70
22.86%

Files with Coverage Reduction
New Missed Lines
%

webhook.py
1
9.36%

preprocessors/preprocessors.py
2
33.89%

Totals

Change from base Build 395:
-0.1%

Covered Lines:
3066

Relevant Lines:
5742

💛  - Coveralls


Controlling partitions while reading xml files

Hi,
I am asking this in the perspective of providing no of partitions while reading from textFile
val rdd= sc.textFile(“file.txt”,5)
Is there any option available while reading files using spark-xml as well, so that instead of taking default hadoop block size for partitioning, I can control based up on cores available in cluster from code itself.
Thanks,
Charan.

Currently, no. I recall there is an (kind of hidden) option in Spark. I will leave another comment after searching it soon.

It would really be of great help , please let me us know as we  are  underutilizing our cluster and is taking hit on time and performance as well

I just took a look. Maybe I think I meant spark.sql.files.maxPartitionBytes for increasing the number of partitions and coalesce(...) to reduce. Maybe, we could consider adding this option.

did anyone got how to improve performance by increasing parallelism ? i tried by modifying options below but still no luck :(
spark.sql.files.maxPartitionBytes
spark.default.parallelism
spark.default.parallelism

even default hadoop properties:
mapreduce.input.fileinputformat.split.minsize
mapreduce.input.fileinputformat.split.maxsize

Any on this topic? I am getting lots of OOM errors when reading a single large file.. I've asked a question on SO: Out of Memory Error when Reading large file in Spark 2.1.0

Hi,
In case I am reading an xml file stored on hdfs, the start and end tags of each record are not on the same line, I am wondering if the XmlInputFormat is able to read the data without loss as the getSplits function doesn't check the content of the data.

It's fine if start/end tags are on different lines; that's the common case. It reads through text/lines until it finds the start and then end tag. What it won't do is find a tag split across partitions. That's a fairly fundamental limitation here, but typically is not a huge issue as commonly the input is many XML files that don't exceed the block size of storage. However it is important to note this as you can miss an input if you configure the Hadoop input format with a much smaller block size.

Hm, wait I correct myself. On re-reading the code, it will read past the end of the split to find the end tag if necessary. And it will actually read past the end of the split if it's currently matching the start tag, too. What it won't be able to do is handle the case where the start and end tag split across files, but, that's a fine assumption for XML.

In this case, I think it is too risky to read xml files that exceed block size (this is  common in most of the projects I've worked on) as data loss will (probably) occur. I suggest to document this limitation as a first step (the time to find a solution, if it exists) .

See my second note - that won't happen if the block size causes a file to be split. What (else) are you concerned about here?

I am facing a case where some data is missing when I change mapreduce.input.fileinputformat.split.maxsize
I've read the code of the version I am using and I found out it may be because of the condition of tracking the end of split which uses filePosition.getPos the function that count the chars in the buffer and are not  processed yet.
I checked the master branch and I found out it was fixed with PR https://github.com/databricks/spark-xml/pull/400
I'll update the lib version in my project. Thanks for your help!


[MGDOBR-440] - Create RHOSE Openshift Template

See: https://issues.redhat.com/browse/MGDOBR-440
Signed-off-by: Mauricio Magnani<EMAIL_ADDRESS>Insert JIRA Issue number and update link
PR owner checklist
Please make sure that your PR meets the following requirements:

[ ] Your code is properly formatted according to this configuration.
Run mvn clean verify -DskipTests so that imports are correctly set.
[ ] Your commit messages are clear and reference the JIRA issue e.g: "[MGDOBR-1] - $clear_explanation_of_what_you_did"
[ ] All new functionality is tested
[ ] Pull Request title is properly formatted: MGDOBR-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] 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.

@robpblake done! thanks.


Fix precheck newclient usage

Please provide a description of this PR:
There's no need to setup a new client for the kubesource, should just use CLIClient

@hanxiaop: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
083081e9639bd5d3ff77c53951f4894d9e696e13
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test release-notes


Request: Sending Images to Discord

Hey there,
at the moment, I'm having an option to send different pages or journal entries to Discord, wich works really well, except for images.
I'll just get the image link postet in Discord, which of course can't be resolved. Is this even possible to send images from Foundry to Discord?
If this is feasible, it would be nice if you could consider implementing it.
Cheers
Berytak

By images in journal pages, you mean images that are embedded as  on journal entries? If so, that's not supported, and you'll have to popout the image to Send to Discord. This is what it should look like:

Journal pages of "image" type should be supported, too.
There is a limitation, though, Discord only embeds images that are of format: JPG, PNG, WEBP, or GIF. If it's not any of these, it can't be embedded onto a message.

This seems odd, I guess than it is something Warhammer specific.
I clicked the image in the journal and after that I clicked "Send Image" on the new window (the yellow marked one).

In Discord it looks like this ...

Ah, I found your problem. Remove the /join/ from your module invite link. It should fix it.

Found the configuration, but sadly that didn't help.

The interesting part about it is, that in the module it's stated that I should test it with the defaultavatar.png. That works like a charm. Even the above postet URL to the image work in my browser ... so I might be missing something...

Oh, I've seen this problem before. It's probably your certificate. Discord doesn't recognize your webpage as secure, which might require adding a chain PEM file to your web certificate. Not sure how to do that with duckdns, but give it a shot.

No-IP offers my dynamic IP to URL service, because ISP only offers dynamic IPs. For their paid service they include a certificate chain which is issued by DigiCert, so no self signed certificate. This is a regular trusted SSL certificate and Foundry loads the whole chain.
Nevertheless, this was this a good hint. I'll try searching in this direction, maybe I can find something useful, because it's very likely a Discord Issue.
Thanks for your help 👍

Just found the culprit. When blocking the Discord server from accessing my web server due to a not whitelisted region, this error might occur ... lol
Appreciate your help again, because without you nudging me in the right direction, I wouldn't have found the solution.


Error when training a model

Hi!
I'm trying to train my model but I get this error:

./train.py --proto=prototype_encdec_state --state mydata.py

python-2.7/lib/python2.7/site-packages/theano/sandbox/rng_mrg.py:768: UserWarning: MRG_RandomStreams Can't determine #streams from size (Elemwise{Cast{int32}}.0), guessing 60*256
  nstreams = self.n_streams(size)
Traceback (most recent call last):
  File "./train.py", line 100, in <module>
    main()
  File "./train.py", line 81, in main
    enc_dec.build()
  File "build/bdist.linux-x86_64/egg/experiments/nmt/encdec.py", line 1385, in build
    c=self.sampling_c)
  File "build/bdist.linux-x86_64/egg/experiments/nmt/encdec.py", line 1252, in build_sampler
    name="{}_sampler_scan".format(self.prefix))
  File "/soft_orokorra_linux_x86_64_rhel6/python-2.7/lib/python2.7/site-packages/theano/scan_module/scan.py", line 1007, in scan
    scan_outs = local_op(*scan_inputs)
  File "/soft_orokorra_linux_x86_64_rhel6/python-2.7/lib/python2.7/site-packages/theano/gof/op.py", line 399, in __call__
    node = self.make_node(*inputs, **kwargs)
  File "/soft_orokorra_linux_x86_64_rhel6/python-2.7/lib/python2.7/site-packages/theano/scan_module/scan_op.py", line 370, in make_node
    inner_sitsot_out.type.ndim))
ValueError: When compiling the inner function of scan the following error has been encountered: The initial state (outputs_info in scan nomenclature) of variable IncSubtensor{Set;:int64:}.0 (argument number 2) has dtype float32 and 2 dimension(s), while the result of the inner function for this output has dtype float64 and 1 dimension(s). This could happen if the inner graph of scan results in an upcast or downcast. Please make sure that you use dtypes consistently

I'm new on neural networks and GroundHog and I don't understand the error message.
Could anyone help me with it?
Thanks!

Update theano. It is probably fixed. Be site to use the development version
of theano or the release candidate for 0.7
Le 18 mars 2015 09:29, "antxa"<EMAIL_ADDRESS>a écrit :

Hi!
I'm trying to train my model but I get this error:
./train.py --proto=prototype_encdec_state --state mydata.py
python-2.7/lib/python2.7/site-packages/theano/sandbox/rng_mrg.py:768: UserWarning: MRG_RandomStreams Can't determine #streams from size (Elemwise{Cast{int32}}.0), guessing 60*256
nstreams = self.n_streams(size)
Traceback (most recent call last):
File "./train.py", line 100, in
main()
File "./train.py", line 81, in main
enc_dec.build()
File "build/bdist.linux-x86_64/egg/experiments/nmt/encdec.py", line 1385, in build
c=self.sampling_c)
File "build/bdist.linux-x86_64/egg/experiments/nmt/encdec.py", line 1252, in build_sampler
name="{}_sampler_scan".format(self.prefix))
File "/soft_orokorra_linux_x86_64_rhel6/python-2.7/lib/python2.7/site-packages/theano/scan_module/scan.py", line 1007, in scan
scan_outs = local_op(*scan_inputs)
File "/soft_orokorra_linux_x86_64_rhel6/python-2.7/lib/python2.7/site-packages/theano/gof/op.py", line 399, in call
node = self.make_node(*inputs, **kwargs)
File "/soft_orokorra_linux_x86_64_rhel6/python-2.7/lib/python2.7/site-packages/theano/scan_module/scan_op.py", line 370, in make_node
inner_sitsot_out.type.ndim))
ValueError: When compiling the inner function of scan the following error has been encountered: The initial state (outputs_info in scan nomenclature) of variable IncSubtensor{Set;:int64:}.0 (argument number 2) has dtype float32 and 2 dimension(s), while the result of the inner function for this output has dtype float64 and 1 dimension(s). This could happen if the inner graph of scan results in an upcast or downcast. Please make sure that you use dtypes consistently
I'm new on neural networks and GroundHog and I don't understand the error
message.
Could anyone help me with it?
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/lisa-groundhog/GroundHog/issues/28.

I've updated Theano, but I get the same error:
Traceback (most recent call last):
  File "./train.py", line 100, in <module>
    main()
  File "./train.py", line 81, in main
    enc_dec.build()
  File "build/bdist.linux-x86_64/egg/experiments/nmt/encdec.py", line 1385, in build
    c=self.sampling_c)
  File "build/bdist.linux-x86_64/egg/experiments/nmt/encdec.py", line 1252, in build_sampler
    name="{}_sampler_scan".format(self.prefix))
  File "/soft_orokorra_linux_x86_64_rhel6/python-2.7/lib/python2.7/site-packages/Theano-0.7.0rc2-py2.7.egg/theano/scan_module/scan.py", line 1017, in scan
    scan_outs = local_op(*scan_inputs)
  File "/soft_orokorra_linux_x86_64_rhel6/python-2.7/lib/python2.7/site-packages/Theano-0.7.0rc2-py2.7.egg/theano/gof/op.py", line 507, in __call__
    node = self.make_node(*inputs, **kwargs)
  File "/soft_orokorra_linux_x86_64_rhel6/python-2.7/lib/python2.7/site-packages/Theano-0.7.0rc2-py2.7.egg/theano/scan_module/scan_op.py", line 350, in make_node
    inner_sitsot_out.type.dtype))
ValueError: When compiling the inner function of scan the following error has been encountered: The initial state (`outputs_info` in scan nomenclature) of variable IncSubtensor{Set;:int64:}.0 (argument number 2) has dtype float32, while the result of the inner function (`fn`) has dtype float64. This can happen if the inner function of scan results in an upcast or downcast.

Do you have the Theano flag floatX to float32 or float64? The problem is
that some initial value of the scan step are in float32, but the new value
have dtype float64.
You can use the theano flag warn_float64=pdb to get in the python debogger
when the upcast happen.
On Mon, Mar 23, 2015 at 6:56 AM, antxa<EMAIL_ADDRESS>wrote:

I've updated Theano, but I get the same error:
Traceback (most recent call last):
File "./train.py", line 100, in
main()
File "./train.py", line 81, in main
enc_dec.build()
File "build/bdist.linux-x86_64/egg/experiments/nmt/encdec.py", line 1385, in build
c=self.sampling_c)
File "build/bdist.linux-x86_64/egg/experiments/nmt/encdec.py", line 1252, in build_sampler
name="{}_sampler_scan".format(self.prefix))
File "/soft_orokorra_linux_x86_64_rhel6/python-2.7/lib/python2.7/site-packages/Theano-0.7.0rc2-py2.7.egg/theano/scan_module/scan.py", line 1017, in scan
scan_outs = local_op(*scan_inputs)
File "/soft_orokorra_linux_x86_64_rhel6/python-2.7/lib/python2.7/site-packages/Theano-0.7.0rc2-py2.7.egg/theano/gof/op.py", line 507, in call
node = self.make_node(*inputs, **kwargs)
File "/soft_orokorra_linux_x86_64_rhel6/python-2.7/lib/python2.7/site-packages/Theano-0.7.0rc2-py2.7.egg/theano/scan_module/scan_op.py", line 350, in make_node
inner_sitsot_out.type.dtype))
ValueError: When compiling the inner function of scan the following error has been encountered: The initial state (outputs_info in scan nomenclature) of variable IncSubtensor{Set;:int64:}.0 (argument number 2) has dtype float32, while the result of the inner function (fn) has dtype float64. This can happen if the inner function of scan results in an upcast or downcast.
—
Reply to this email directly or view it on GitHub
https://github.com/lisa-groundhog/GroundHog/issues/28#issuecomment-84964502
.

I ran into this too. Make sure your floatX flag is consistent between runs.

Right. Setting floatX=float32 in Theano everything works OK.
Thanks!


maybe add a single RwLock<GhostToken <'static>> for closure haters

There are some alternatives to ghost_cell that are partially based on the criticism that ghost_cell requires all usage to be within a closure.  Can there be a single static RwLock<GhostToken<'static>> that would enable usage without closures?

The goal of GhostCell was to demonstrate that a sound zero (runtime) overhead solution is possible -- indeed, GhostCell is first and foremost an academic paper demonstrating the soundness of the construct.
There are many variants of cells, starting with Cell and RefCell in the standard library, and following with an extensive ecosystem.
In particular, the singleton-cell (SCell) crate seems to fill exactly this usecase.
I see no reason to try and compete with SCell in this case:

It would result in duplicated maintenance.
It would water down the value proposition of GhostCell: that it is formally proven.


Link to county managed websites

Provide a link to the counties that manage their own tiesheets.
This is the current list. Should there be any updates?
{
    'UTAH': 'https://maps.utahcounty.gov/TieSheets/TieSheet.htm',
    'WASATCH': 'https://wasatch.maps.arcgis.com/apps/webappviewer/index.html?id=103db0251a5342f7bbd1462eb7a47440',
    'DAVIS': 'http://www.co.davis.ut.us/surveyor/default.cfm',
    'SALT LAKE': 'https://slco.org/surveyor/apps/surveymonument/map.html',
    'WEBER': 'https://www3.co.weber.ut.us/gis/maps/survey/index.html',
    'DUCHESNE': 'https://duchesne.utah.gov/surveyor/base&meridian.html',
    'UINTAH': 'https://www.co.uintah.ut.us/surveyor/UCTieSheet.htm',
    'CACHE': 'http://<IP_ADDRESS>/websites/surveyviewer/'
}

@seanfernandez the Cache link needs updated. Looks like we should either change it to the find-a-survey page, or go directly to the survey and section viewer. My opinion is to user the former of the two options since it provides some other important information
@steveoh, please change the Cache link to:
https://www.cachecounty.org/surveyor/find-a-survey.html
The rest look good to me.

Cache validated


Fixing labs() and themes to work with piping syntax

As I'm working on an online course, I discovered that labs() and themes don't work with a piping syntax. labs() doesn't work because of a bug in the current syntax, and themes just need a new method signature to allow for piping.
These are super easy fixes and I'll put them in today. Just wanted to open issue for awareness.

Fixed in https://github.com/TidierOrg/TidierPlots.jl/pull/46

Thanks! Merged the PR so closing this issue


Add serialization of typenode for null/undefined/never as part of metadata type

Fixes #12684 and #11933

@mhegazy Please take a look, I have handled "non identifier" serialized expressions with null/undefined/never in the update.

@sheetalkamat does this handel https://github.com/Microsoft/TypeScript/issues/13098 as well?

@mhegazy #13098 is different. This only deals with union/intersection. Will take a look at that one separately.


On registration check for labels that overlap with dimensions set in the builder.

Should emit an error to tracing when this happens and return a noop metric handle.

A bit concerned about runtime impact of something that shouldn't really happen with any sane design.


Update Tinder.vue

Wait for NextTick before getting size.
It allows the dev to use relative sizing (for example "height = 100%") when designing the app (without waiting for nextTick, the size is equal to 0)

I think the height need be set before create, or you can mount vue-tinder after page loaded.

@shanlh : the height is defined with css, but in my case it's a "flexible" height, not an absolute one, so the value is not computed by the browser before all the elements are rendered.
  width: 100%;   max-width: 400px;   height: 100%;   max-height: 600px;
That's why my solution is to wait for a first rendering of the page to get the correct size.
Considering your alternative solution, "mount vue-tinder after page loaded", could you help me with that? I do not know how to differ the mounting of a component.
What I have sometimes see in other libraries is that you can disable the "auto init" of the component, and call an "init" function when wanted (for example, in my case, in the "next Tick" event). But it implies to add this prop and this method to the component, which is not the case with vue-tinder.
Thanks in advance for your help

Maybe you can set height: 100vh instead of 100%

Sadly not, because my card is not 100vh, but is 100% of it's parent's height.
I believe that there is many situations where the size of the card cannot be declared as an absolute value. That's why I proposed to use the small trick of getting the size after the render occurred.

I made a demo which use flex:
https://codesandbox.io/s/vue-tinder-preview-forked-hxrie


Upgraded to NH5. Support for async.

The upgraded nuspec has not been tested.
The sample web project didn't start here, but it's a bit dated.

Hi @gliljas,
Thanks for your work on this! I'm interested in merging this but there a few issues that would need to be addressed before I can do so:

We must be able to support those that still depend on NHibernate < 5. Because NHibernate is such a fundamental library to applications, upgrading to NHibernate 5 isn't the easiest for everyone (I know this from personal experience). This means that the minimum version of which our Nuget package depends upon shouldn't have to change. I noticed in your nuspec changes that if targeting net461, use NHibernate >= 5.0. This makes sense from NHibernate 5 requiring .NET 4.6.1, but we need to make sure if someone is on .NET 4.6.1 that they can still use NHibernate < 5 (it's not immediately clear if this will work with Nuget). We need to make sure there is binary compatibility for NHibernate < 5. For example, if someone has version 3.0.0-beta5 of this library installed, would upgrading to the next version force NHibernate to also be upgraded (obviously if our Nuget dependency changed it would force it)?
Some of the code doesn't match the existing code style (e.g. we only use spaces, extra whitespace lines, I don't like the idea of having a separate file for async, etc).
There's a lot of superfluous changes to config files that aren't relevant to the NH 5 upgrade (e.g. what is AsyncGenerator.yml, why so many App.config changes, etc).

Thanks for replying. I will give it a bit of cleanup and an attempt to support NH<5 cleanly.

How would you like to handle the backwards compatibility? A separate Async package with >=5 dependency and changing the dependency of the standard package to <5?

Can we not do it in one package using binary compatibility?

I don't see how, or am I missing something? Or are you thinking of having a package version 5 (or 4) with a NH5 dependency, while still being able to maintain v3 packages for NH4?

The "standard" caches were just upgraded
https://www.nuget.org/packages/NHibernate.Caches.SysCache/5.1.0
..and they've chosen to just up the dependency. I suggest the same for this library.

NH 5 isn't binary compatible with previous versions.

People needing to use older NHibernate should just use the previous cache package version. NuGet allows this.
The drawback is for supplying new features to them along to the latest version: it will require to publish a new version of the old package along with a new version of the up-to-date package. As long as the up-to-date package has a major version greater than the packages targeting previous NHibernate, it is doable without confusion by just upgrading the minor part of the version of both.


Added support for updating and replacing axis properties

As I was investigating getting the MercatorTicker working in GeoViews I realized that the code that handles updating of axis properties never correctly handled updating of ticker and formatter models. I have now added a utility which handles recursive updates to bokeh models, preventing unnecessary updates and even handling cases where the model is being replaced (which we now allow in live and dynamic modes as well as on the server). Once I've figured out whether we can use the new bokeh protocol to embed data we may eventually be able to switch out models at will and use this update function everywhere, e.g. to replace a linear axis with a log or categorical axis on the fly.
Turns out this was also the reason why I had to filter ticker and formatter model updates from the static json patches before, allowing me to simplify that now.

Excellent!!!

Looks great! Especially together with #1949.
Is this ready to merge and test or do you think we should cut a dev release before merging these two PRs?

Go ahead and merge this. I'll do some more testing before #1949 is ready.

Tests are passing and I'm looking forward to seeing this in action and simplifying the code with #1949. Merging.


do not fail with twig 3

Q
A

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Related tickets
fixes #358

Documentation
-

License
MIT

What's in this PR?
Make sure our twig extension works with twig 3.
Why?
We did not declare a conflict with twig 3, so the bundle can be installed alongside twig 3.

failure are only about deprecations in the test setup with symfony 4.4. okay if we merge as is? i want to work on symfony 4.4/5 support in a separate PR.

deprecations seem triggered by old twig bundle version. should be fixed in #360

continued in #360


Signed/Unsigned warnings fixed, build system adjusted for scalable container builds

Can one of the admins verify this patch?

run test

Any idea why git is failing?

17:22:48 tests/jenkins/run_test_amzn_linux_gpu.sh: line 24: ./tests/cpp/*_test: No such file or directory

Rebuilding PR


Can't run carthage update (or tests) due to Bolts failure

Not a PromiseKit error, but I tried to be diligent by running the tests before submitting #262 and couldn't due to this error with Bolts:

error: cannot parse the debug map for "/Users/bgerstle/Code/PromiseKit/Carthage/Build/iOS/Bolts.framework/Bolts": The file was not recognized as a valid object file

Tried rolling back to Bolts 1.2.0, but no luck. Using Carthage 0.8.

Yes, Bolts apparently doesn’t build with Carthage currently. It affects me also. Hopefully Facebook will fix it.
Not building Bolts allows the rest of the Carthage deps to build, but then the iOS tests will fail.

I believe this is resolved now as I have no longer have an issue building Bolts with Carthage. But feel free to reopen if it persists.


Add new template tag: set

Django Template tag {% block name %}...{% endblock %} has one disadvantage - you can't use repeatable data twice on a page. It's why we need to create new template block {% set name %}...{% endset %} and use later as {{ name }} - as a regular variable.
It allow us to set some content to variable, and then use it in few places on this page, and even use this variable on parent's pages (on pages that we extend with {% extends 'base.html' %} tag).
Under the hood this set template tag should do simple thing - create new template variable and set to it content from inside the block. But, this content should be executes only at the time, when we show it. We can see it on the example below.
{% set menu_item %}
    {{ name }}
{% endset %}

{% for link in links %}
  {{ menu_item }}
{% endfor %}

If it's not possible to execute variable at time when we show this variable - we need to provide a template tag to get this variable and execute it: {% get name %}.

Would it be possible to use template partials with context? ie:
{% include 'your-template-partial.html' with menu_item=menu_item %}
Then just include that partial twice in the page's block.


ion-nav-back-button not shown on view Clientes

Hello friends,
In this case, what would be the navigation rule?
Thank you!

Greetings @condepatula!
Sorry but this issue may no longer be applicable to recent versions of Ionic.
If you are still experiencing this issue, please feel free to reopen and provide the following information so that I may assist you.

Can you provide a detailed description of this issue?
How would a robot go about replicating this issue?
What platform does this happen on? (iOS, Android, all, etc.)
What platform versions does this happen on? (Android 4.2, iOS 7, all, etc.)
Is this an issue with the Ionic Framework, or Ionic CLI?

Additionally, providing a codepen which replicates the issue is extremely helpful.
If you wish to get help using the framework itself, the recommended place is the forum.
Thank you for allowing me to assist you.


BigQuery client: cache results

Using Google client-side libraries, we should be able to manage BigQuery queries from the frontend
https://www.npmjs.com/package/googleapis
https://www.npmjs.com/package/@google-cloud/local-auth
For now, we can create a separate Supabase table for tracking and managing BigQuery jobs/results

Now that we have Clickhouse, closing in favor of https://github.com/opensource-observer/oso/issues/1268


fix: only include all components if no specific excludes

A suggested fix for https://github.com/primefaces/primevue-nuxt-module/issues/29.
A side effect of this change is that it assumes that if a user has included and excluded all components, including all should take priority.

Thanks a lot for your contribution!
Best Regards,


Derivation fixes: update derivation timestamp bounds, clear up payload attributes

This implements the timestamp derivation design as discussed in #251, going for the proposal of @trianglesphere here, with max-fill definition of @norswap here and sequencer drift (or "diff") like in https://github.com/ethereum-optimism/optimistic-specs/pull/264 but now with the fixed bounds definition from the discussion thread.
Additionally, this cleans up the rollup-node spec:

The min/max really affect the batch parsing early on. Other batches should be ignored right away when not within bounds.
The L1 input and sequencing window sections are unified
Payload attributes are not implicitly derived from parsed inputs anymore, but explicitly for a specific range of timestamps, including clarification of any gaps/trailing blocks.
Individual payload attributes are a sub-section of this, and now utilize the noTxPool field that the implementation has been using for a while now. (And exec-engine spec is updated to describe the field)

TODO:

update/test implementation to reflect spec changes

Codecov Report

Merging #315 (35ba98c) into jg/harden_seq_v3 (a9ac485) will increase coverage by 0.49%.
The diff coverage is n/a.

@@                 Coverage Diff                  @@
##           jg/harden_seq_v3     #315      +/-   ##
====================================================
+ Coverage             55.79%   56.29%   +0.49%
====================================================
  Files                    46       46
  Lines                  3613     3613
====================================================
+ Hits                   2016     2034      +18
+ Misses                 1358     1343      -15
+ Partials                239      236       -3

Impacted Files
Coverage Δ

opnode/rollup/driver/state.go
72.15% <0.00%> (+1.26%)
:arrow_up:

opnode/bss/submitter.go
62.29% <0.00%> (+4.91%)
:arrow_up:

opnode/internal/testlog/testlog.go
76.92% <0.00%> (+18.46%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a9ac485...35ba98c. Read the comment docs.

See last "TODO" commit: the E2E fails when you add the NEXT_L1_TIMESTAMP - L2_BLOCKTIME part to the MAXIMUM_L2_FILL.
Maybe that is crowding out the tx attempt with more instant early empty blocks? Not sure why E2E fails or what's wrong with this max.

Will rebase this onto main

@trianglesphere @norswap thanks for review, implemented the suggestions + fixed other minor stuff.
@trianglesphere the maxL2Time condition during sequencer block construction is now conditional on it not being the first block of the epoch, to ensure at least 1 block is output. But I am still looking at your first point, about L1 time between blocks being larger than the sequencer drift. Not sure what the best approach is there.

But I am still looking at your first point, about L1 time between blocks being larger than the sequencer drift. Not sure what the best approach is there.

I see two approaches here:

Keep the 2s block time and fill them out with empty blocks (must change block derivation)
Be willing to drop the 2s block time and have a cap on the number of L2 blocks in an epoch (must modify block time req)

I think that either is possible to implement.

TLDR of the solution to the sequencer drift vs L1 block time gap problem: the verifier case was already solved with the next_l1_timestamp - l2_block_time part of the new_head_l2_timestamp. The sequencer just needed to consider that, by continuing on the current epoch until it catches up to the start of the next epoch.


503 error with websocket

Description

Setup envoy with websocket has 503 error. I read an issue about this and get a suggestion that's turning on keepalive. However, I can't find this option in the docs

I tested without envoy (using direct `http://localhost:3001) and it worked.

Update Application return 503
Code
admin:
  access_log_path: /tmp/admin_access.log
  address:
    socket_address: { address: <IP_ADDRESS>, port_value: 8081 }

static_resources:
  listeners:
    - address:
        socket_address:
          address: <IP_ADDRESS>
          port_value: 80
      filter_chains:
        - filters:
            - name: envoy.http_connection_manager
              config:
                codec_type: auto
                stat_prefix: ingress_http
                route_config:
                  virtual_hosts:
                    - name: backend
                      domains:
                        - "localhost"
                      routes:
                        - match:
                            prefix: "/"
                          redirect:
                            path_redirect: "/"
                            https_redirect: true
                http_filters:
                  - name: envoy.router
                    config: {}
    - name: listener_0
      address:
        socket_address: { address: <IP_ADDRESS>, port_value: 443 }
      filter_chains:
        - filters:
            - name: envoy.http_connection_manager
              config:
                codec_type: auto
                stat_prefix: ingress_ws_to_ws
                upgrade_configs:
                  - upgrade_type: websocket
                route_config:
                  name: local_route
                  virtual_hosts:
                    - name: local_service
                      domains: ["*", "<IP_ADDRESS>", "localhost"]
                      routes:
                        - match:
                            prefix: "/"
                          route:
                            upgrade_configs:
                              upgrade_type: websocket
                            cors:
                              allow_credentials: true
                            cluster: ws
                      cors:
                        allow_origin_string_match:
                          - prefix: "*"
                        allow_methods: GET, PUT, DELETE, POST, OPTIONS
                        allow_headers: keep-alive,user-agent,cache-control,content-type,content-transfer-encoding,custom-header-1,x-accept-content-transfer-encoding,x-accept-response-streaming,x-user-agent,x-grpc-web,grpc-timeout
                        max_age: "1728000"
                        expose_headers: custom-header-1,grpc-status,grpc-message
                http_filters:
                  - name: envoy.grpc_web
                  - name: envoy.cors
                  - name: envoy.router
                    config: {}
          transport_socket:
            name: envoy.transport_sockets.tls
            typed_config:
              "@type": type.googleapis.com/envoy.extensions.transport_sockets.tls.v3.DownstreamTlsContext
              common_tls_context:
                tls_certificates:
                  - certificate_chain:
                      filename: certs/localhost.crt
                    private_key:
                      filename: certs/localhost.key
  clusters:
    - name: ws
      connect_timeout: 0.25s
      type: logical_dns
      http2_protocol_options: {}
      lb_policy: round_robin
      hosts:
        - socket_address:
            address: backend
            port_value: 3001

Thanks for any help!

@zuercher can you give me an example for getting access logging, I tried to follow docs to configure but it makes so many errors

This configures logging: https://github.com/envoyproxy/envoy/blob/a12869fa9e9add4301a700978d5489e6a0cc0526/test/config/integration/server.yaml#L53
It uses templates and is for testing log filter, so change path to an actual path on your system and remove the filters: section.

I received this message
 {"message":"upstream_reset_before_response_started{connection_failure}","status":503}

Do you have any idea? Following #14981, the upstream's connection might be closed. However, I don't think websocket will close the connection at any time.

Thanks for support


AsyncClient(app=asgi_app) does not run startup/shutdown functions

This might be by design, but it seems like the client, when wrapping an ASGI app, does not trigger the startup/shutdown lifespan events or wait for their event handlers.
If the only thing the startup/shutdown handlers do is to connect/disconnect to the database, I might as well solve this with additional fixtures in my test setup. My concern is keeping the startup/shutdown handlers and the test setup in sync once the handlers grows larger.
If the lack of lifespan support is by design, I'm OK with that, but if so we should document it.
If this isn't by design, there's an implementation based upon requests_async.ASGISession here that looks quite adaptable to http3.AsyncClient.

Requirements:
http3==0.5.0
pytest
pytest-asyncio
starlette

Test case:
import asyncio

import http3

import pytest

from starlette.applications import Starlette
from starlette.responses import PlainTextResponse

startup_was_called = asyncio.Event()

@pytest.fixture
def app():
    app = Starlette(debug=True)

    @app.on_event("startup")
    def startup():
        startup_was_called.set()

    @app.route("/")
    async def homepage(request):
        await asyncio.sleep(1)
        return PlainTextResponse("Hello, world!")

    return app

@pytest.fixture
def http_client(app):
    return http3.AsyncClient(app=app)

@pytest.mark.asyncio
async def test_async_test_client_triggers_app_startup(http_client):
    result = await http_client.get("http://www.example.com/")

    assert result.text == "Hello, world!"
    assert startup_was_called.is_set()

Result:
____________________________________________ test_async_test_client_triggers_app_startup _____________________________________________

http_client = <http3.client.AsyncClient object at 0x7f11a35b5dd8>

    @pytest.mark.asyncio
    async def test_async_test_client_triggers_app_startup(http_client):
        result = await http_client.get("http://www.example.com/")

        assert result.text == "Hello, world!"
>       assert startup_was_called.is_set()
E       assert False
E        +  where False = <bound method Event.is_set of <asyncio.locks.Event object at 0x7f11a39a3dd8 [unset]>>()
E        +    where <bound method Event.is_set of <asyncio.locks.Event object at 0x7f11a39a3dd8 [unset]>> = <asyncio.locks.Event object at 0x7f11a39a3dd8 [unset]>.is_set

test_http3_async_test_client.py:40: AssertionError
================================================ 1 failed in 1.09 seconds =================================================

So.
Yes, it is by design really.
Starlette's test client happens to bundle the test client up with the startup/shutdown functionality but really I think we ought to separate them, so that we've got AsyncClient() for making the requests, and Lifespan() for triggering startup/shutdown. (Lifespan would be something we'd provide in Starlette, rather than here.)

It would look a bit like this...
class Lifespan:
    def __init__(self, app):
        self.app = app
        self.lifespan_task = None
        self.loop = asyncio.get_event_loop()
        self.receive_queue = asyncio.Queue()
        self.send_queue = asyncio.Queue()

    async def startup(self):
        # Start up the lifespan task.
        scope = {'type': 'lifespan'}
        receive = self.receive_queue.get
        send = self.send_queue.put
        self.lifespan = self.loop.create_task(self.app(scope, receive, send))

        # Send the startup message, and wait until complete.
        await self.receive_queue.put({'type': 'lifespan.startup'})
        message = await self.send_queue.get()
        assert message['type'] == 'lifespan.startup.complete'

    async def shutdown(self):
        # Send the shutdown message, and wait until complete.
        await self.receive_queue.put({'type': 'lifespan.shutdown'})
        message = await self.send_queue.get()
        assert message['type'] == 'lifespan.shutdown.complete'

        # Wait for the task to exit.
        await self.lifespan_task

    async def __aenter__(self):
        await self.startup()

    async def __aexit__(self, exc_type, exc_val, exc_tb):
        await self.shutdown()

Then usage either with...
lifespan = Lifespan(app)
await lifespan.startup()
...
await lifespan.shutdown()

or...
async with Lifespan(app):
    ...


[docs] Material Icon Imports wrong in docs

Related page
https://mui.com/material-ui/material-icons/?query=playlist
Kind of issue
Broken demo
Issue description
Using the MUI Icons docs located here: https://mui.com/material-ui/material-icons/?query=playlist
And clicking PlaylistAdd and PaylistRemove, presents the following import:
import PlaylistAddIcon from '@mui/icons-material/PlaylistAdd';
import PlaylistRemoveIcon from '@mui/icons-material/PlaylistRemove';

However this is incorrect.  It should be:
import PlaylistAdd from '@mui/icons-material/PlaylistAdd';
import PlaylistRemove from '@mui/icons-material/PlaylistRemove';

I'm new to js/react/mui, but I'm pretty sure I have the most up to date version of Material Icons, from my package-lock.json:
        "@material-ui/styled-engine": "^5.0.0-alpha.11",
        "@mui/icons-material": "^5.15.19",
        "@mui/material": "^5.15.18",
        "@mui/x-charts": "^7.5.1",

Context
Just use icons on my buttons.
Search keywords: Material Icon Imports wrong in docs

Why is this incorrect? It's a default export and can be imported with any name.


[BUG] SendAsync performance degradation

Describe the bug
After updating the Microsoft.Azure.ServiceBus package from 4.1.1 to 5.1.2 our performance load tests started to fail. We believe there is a performance degradation issue that affects the queue client (SendAsync method).
Expected behavior
The performance is stable.
Actual behavior (include Exception or Stack Trace)
The performance is degraded.
Version 4.1.1 load tests performance (for different builds):
duration...................: avg=1274.325813 min=28.1346 med=1164.3777 max=5199.2588 p(90)=2152.46747 p(95)=2516.562035
duration...................: avg=1228.571845 min=16.9841 med=1098.56585 max=5672.2006 p(90)=2323.30299 p(95)=2639.88406
duration...................: avg=1402.146743 min=33.019  med=1268.26185 max=6997.9666 p(90)=2321.07433 p(95)=2763.291545
duration...................: avg=1258.797064 min=19.0035 med=1178.0737 max=5760.0523 p(90)=2046.12003 p(95)=2410.7208
Version 5.1.2 load tests performance (for different builds):
duration...................: avg=1674.858551 min=31.0022 med=1508.90515 max=7353.941 p(90)=2910.44202 p(95)=3372.70381
duration...................: avg=1740.276796 min=28.0038 med=1620.86505 max=7111.0786 p(90)=2783.84421 p(95)=3265.554415
duration...................: avg=1887.889699 min=14.9668 med=1774.9656 max=6988.9752 p(90)=3116.05071 p(95)=3584.0975
duration...................: avg=1725.125742 min=57.1496 med=1638.98715 max=8505.227 p(90)=2706.48648 p(95)=3043.7245
To Reproduce
To reproduce the issue, you can create a console application that sends several messages using the public Task SendAsync(Message message) method from the queue client and compare the performance using the versions 4.1.1 and 5.1.2.
Environment:

Name and version of the Library package used: Microsoft.Azure.ServiceBus 5.1.2
Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects): Azure AppService

//fyi: @JoshLove-msft

Thank you for your feedback.  Tagging and routing to the team best able to assist.

@evadelnuevo was this against a standard or premium namespace?

@axisc It's against a standard namespace

@evadelnuevo it's possible that the degradation may be due to reasons unrelated to the client library.
Can I request you to run the same workload (with old and new library) against a premium namespace?

@axisc Our load tests started to fail after a commit that only updated the azure service bus library. We tried deploying an older version and they succeeded. We also tested our latest version from master but reverting that commit, that's why we doubt that the problem is not related to the library.
Unfortunately the option to upgrade the namespace to premium is not available in the portal for us:

Checking against the premium tier is the best way to get a comparable dedicated setup each time.
How many times did you test it out against standard and see your load tests failing? It's best to try it multiple times before we consider digging further.
Also, and more importantly, we don't make any SLA guarantees on send or receive latency, so I'd recommend avoiding having your business logic depend on any observed latency.

Our load tests run daily after deploying master to our development environment, and we also tested this manually several times. According to the Test failure report from VSO, they have failed 20 times after we deployed the version with the latest service bus library:

@evadelnuevo  what is the criteria for failure?

The load tests make lots of calls to one of our API methods, simulating multiple callbacks from another API. That method only reads the request and puts a message in the azure service bus queue.
The criteria for the failure is that the median of the response time is greater than 1300 ms

I'll close the issue as we have requested help from Microsoft Premium Support. Many thanks for your time.


Touch events are not working in on my Android phone

Hi
I just opened the main page in on phone and dragging is not working.
I tested with Firefox Mobile (Chrome and Opera had time out error!)

Hi, Yes you are correct about touch events. I currently have not implemented this as it creates issues when trying to scroll on mobile devices. only mouse events are currently implemented.

You can enable dragging when the user long touched. It seems to be the expected behavior.
I think that you can use Pressure.js library to simulate this.

Thankyou for your thoughts, I have enabled touch events on the touch branch of this project. Thanks for the link regarding pressure.js ill have a read when i get a moment.

FYI ive started some testing of another drag/resize lib:
Initial testing is here: https://github.com/bensladden/vue-interact & https://vue-interact-test.netlify.app/
See what you think.

Version .4 now used interactjs. moveHold and resizeHold props enable you to delay moving/resizing thus allowing scrolling behavour


Relative Filter in many columns with one field

I created a Relative Filter thats does the filter in many columns with one only field, this is very helpful when you want to filter on the all fields and one filter at the same time.
names:

DeivisF
DeivisG

url?name=Deivis&search=DeivisF
    public function getFilters(): array
    {
        return [
            new RelativeFilter('name'),
            new RelativeFilter('lastname'),
            new FieldsRelativeFilter("search", ['name', 'lastname']),
            new Sort(),
        ];
    }

damm I cant belive I forgot to click Submit review, my bad then @@!


Improve compatibility for Ansible 2.9 and Python 3.6

Examples for those cases at the moment:

RHEL 7+8 with Ansible

fixes #49

@lbetz, whats your opinion on the support for python 3.6?
I guess many customers won't be able to use the collection if rhel 8 won't natively support it.
For Tower/AWX Users its possible to user PyVenv to get other python versions without conflicting with the os version.

Thanks again for the PR it's merged.


EPE-1577 rodeos getting stream addresses for queues or exchanges from environment variables

Change Description

Change Type
Select ONE:

[ ] Documentation

[ ] Stability bug fix

[X] Other

[ ] Other - special case

Testing Changes
Select ANY that apply:

[ ] New Tests

[ ] Existing Tests

[ ] Test Framework

[ ] CI System

[ ] Other

Consensus Changes

[ ] Consensus Changes

API Changes

[ ] API Changes

Documentation Additions

[ ] Documentation Additions

This change makes rodeos be able to get the rabbits stream addresses for queues or exchanges from environment variables.
  --stream-rabbits arg                  RabbitMQ Streams to queues if any;
                                        Format: amqp://USER:PASSWORD@ADDRESS:PO
                                        RT/QUEUE[/STREAMING_ROUTE, ...].
                                        Additional list of streams delimited by
                                        ::: can be specified in the environment
                                        variable EOSIO_STREAM_RABBITS.
  --stream-rabbits-exchange arg         RabbitMQ Streams to exchanges if any;
                                        Format: amqp://USER:PASSWORD@ADDRESS:PO
                                        RT/EXCHANGE[::EXCHANGE_TYPE][/STREAMING
                                        _ROUTE, ...]. Additional list of
                                        streams delimited by ::: can be
                                        specified in the environment variable
                                        EOSIO_STREAM_RABBITS_EXCHANGE.

Replaced by https://github.com/EOSIO/eos/pull/10841 .


Compute GEN-level free energy results

https://github.com/choderalab/covid-moonshot-infra/pull/34#discussion_r477653314

If we organize all the work values in a flat array, we can probably just have another flat int array that annotates which GEN each work value came from, e.g.
"work_forward": [1.234, 2.345, ...]
"work_reverse": [4.523, -1.235, ...]
"work_gen": [0, 0, 1, ...]

Makes sense, and then we could record the free energies by gen with something like
"delta_f": [1.234, 2.345, ...],
"ddelta_f": [0.12, 0.23, ...]

where there is one entry per gen.
In the PR I had started down the path of implementing a more nested structure, which looks like
{
  "analysis": {
    "complex_phase": {
      "fe": {
        "delta_f": 42.42,
        "ddelta_f": 4.24,
        "bar_overlap": 0.85,
        "num_work_values": 42
      },
      "gens": [
        {
          "fe": {
            "delta_f": 42.42,
            "ddelta_f": 4.24,
            "bar_overlap": 0.85,
            "num_work_values": 42
          },
          "forward_works": [42.4, 40.34, 44.83],
          "reverse_works": [40.55, 38.12, 37.2]
        }
      ]
    }
  }
}

I like this from the perspective of "making invalid states unrepresentable", but admittedly the nesting might be more annoying to deal with downstream.

Nesting shouldn't be a problem!
But let's use more descriptive, self-documenting names if possible. "fe" is a little too short to be unambiguous.
Eventually, we can overhaul this so that even the units are self-documenting, either forward_works_in_kT or using a units library like pint or unyt.

Implemented in #36


⚠ Bofrost.lu has degraded performance

In 8905a01, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2465 ms

Resolved: Bofrost.lu performance has improved in 519b4a9 after 6 minutes.


Attachment shows as text in body of email

This is the email body:
Content-Type: application/vnd.openxmlformats-officedocument.spreadsheetml.sheet;
 charset=UTF-8
Content-Transfer-Encoding: base64
Content-Disposition: attachment;
 filename="Daily Status 2014-09-02.xlsx"
Content-ID<EMAIL_ADDRESS>
UEsDBBQAAAgIAACYn+tjOGvU2QAAAEECAAALAAAAX3JlbHMvLnJlbHOtksFO
wzAMhu88ReT7mm5ICKFluyCk3RAqD2ASt43axJEToLw9gQNiaAgOO1r2//3f
wdv9Emb1QpI9RwPrpgVF0bLzcTDw2N2trmG/2z7QjKVe5NGnrGokZgNjKelG
62xHCpgbThTrpmcJWOoog05oJxxIb9r2Sst3BhwzVYcyUDGwzPqVZXpinpoK
A9W9JfpPFfe9t3TL9jlQLCcaf1yAOjgDcnBr0L+4OLb3wjVpWejcNoEKOiz4
CV+l2kNSPOUvr83fXpjSubVoKRQduVNGlx9G+ugVdhfvUEsDBBQAAAgIAACY
n+vvyqPEyAEAAAkFAAANAAAAeGwvc3R5bGVzLnhtbKVU22rcMBB971cIfUBl....

This is my mailer:
class StatusMailer < ActionMailer::Base
    default to: Proc.new {<EMAIL_ADDRESS>from<EMAIL_ADDRESS>
	def daily_coding_status

		# execute sp
		status_results = DailyStatus.execute_procedure("get_coding_status")

		# cache date
		@date = Time.zone.today.to_s

		xlsx = render_to_string handlers: [:axlsx], formats: [:xlsx], template: "status_mailer/status", locals: { status_results: status_results }
    	attachments["Daily Status #{@date}" + ".xlsx"] = { mime_type: Mime::XLSX, content: xlsx}
		mail(:subject => "#{@date}  Status Report")
	end
end

This is the template 'status.xlsx.axlsx':
# build excel file
wbook = xlsx_package.workbook
wbook.add_worksheet(:name => "Daily Status") do |sheet|
	sheet.add_row ["Parent Location RID", "Location RID", "Location Name", "Unique Tools", "Total Parts", "Total Coded", "Total Not Coded", "Total Questions", "Total Do Not Code",
		  "Total Ready To Code", "Unique Parts", "Unique Coded", "Unique Not Coded", "Unique Questions", "Unique Do Not Code", "Unique Ready To Code", "Percent Coded",
		  "1 Day Difference", "7 Day Difference", "30 Day Difference"]

	status_results.each do |result|

	data_row = [result['parent_location_rid'], result['location_rid'], result['location_name'], result['unique_tools'], result['total_parts'], result['total_coded'], result['total_not_coded'],
	 		    result['total_questions'], result['total_do_not_code'], result['total_ready_to_code'], result['unique_parts'], result['unique_coded'], result['unique_not_coded'], result['unique_questions'],
	 			result['unique_do_not_code'], result['unique_ready_to_code'], result['percent_coded'], result['one_day_difference'], result['seven_day_difference'], result['thirty_day_difference']]

      #add the row to the sheet
      sheet.add_row data_row, :types => [:string, :string, :string, :string, :string, :string, :string, :string, :string, :string, :string, :string, :string, :string, :string, :string, :string, :string, :string, :string]
	end
end

I can't see what I'm doing incorrectly.

Same thing happening on 3.2.18. Adding self.instance_variable_set(:@_lookup_context, nil)
fixes the problem.

Same on ruby 2.1.7, rails 3.2.12.
Adding self.instance_variable_set(:@_lookup_context, nil) fixes the problem.
Another solution?
Instead self.instance_variable_set(:@_lookup_context, nil) I could do use render_to_string again, this time to force the html format, adding formats: [:html] :
body_template = render_to_string(formats: [:html], :action => "path/to/action", :layout => false)
mail(subject: "Test", body:  body_template, to: email)

Though You'll probably want to use html_safe on the result, so make sure it is actually html safe

@Roko131 Thanks for this result. That makes sense now why the first call to render to string is showing up in the message if your second render to string is producing the message. I'll have to dig into the lookup context to see what is going on sometime.

Mail is going but in attachment, a sheet is not opening.

My rails version: 4.2
Ruby Version: 2.2.4
My mailer action:
def send_timesheet_summary_report(project_employee, projects_summary, employee_summary)
data_file = render_to_string(
layout: false, handlers: [:axlsx], formats: [:xlsx],
template: 'time_sheets/export_project_report',
locals: { project_employee: project_employee, projects_summary: projects_summary, employee_summary: employee_summary }
)
attachment = Base64.encode64(data_file)
attachments["Timesheet_summary_report.xlsx"] = {mime_type: Mime[:xlsx], content: attachment, encoding: 'base64'}
self.instance_variable_set(:@_lookup_context, nil)
mail(
subject: "Timesheet summary report",
to<EMAIL_ADDRESS>end
and my action name and template name is different


[Bug]: WindTerm's file name is not same as the tag

Prerequisites

[X] I have written a descriptive issue title.
[X] I have searched all issues/PRs to ensure it has not already been reported or fixed.
[X] I have verified that I am using the latest version of Scoop and corresponding bucket.

Package Name
windterm
Expected/Current Behaviour
404 Not Found
Steps to Reproduce
scoop install windterm
WARN  Purging previous failed installation of windterm.
ERROR 'windterm' isn't installed correctly.
Removing older version (2.6.0).
'windterm' was uninstalled.
Installing 'windterm' (2.6.0) [64bit] from extras bucket
远程服务器返回错误: (404) 未找到。
URL https://github.com/kingToolbox/WindTerm/releases/download/2.6.0/WindTerm_2.6.0_Windows_Portable_x86_64.zip is not valid

Possible Solution
When I install this software, scoop notified me that the download link throwed a not found error. After investigating, I found that the file names provided on the WindTerm release page do not match the tags, which affects the automatic matching in the autoupdate section.
Here is a valid link:
https://github.com/kingToolbox/WindTerm/releases/download/2.6.0/WindTerm_2.6.1_Windows_Portable_x86_64.zip
However, the link given in the manifest is:
https://github.com/kingToolbox/WindTerm/releases/download/2.6.0/WindTerm_2.6.0_Windows_Portable_x86_64.zip
This is definitely an issue on WindTerm's side, but how should we resolve this issue now? I'm not familiar with the structure of scoop, so I don't know if it's robust to only modify the architecture without changing the autoupdate part, so I opened this issue in hopes of an official solution.
Scoop and Buckets Version
Current Scoop version:
v0.3.1 - Released at 2022-11-15

'extras' bucket:
d49a410c3 (HEAD -> master, origin/master, origin/HEAD) wavebox: Update to version <IP_ADDRESS>

Scoop Config
last_update                       scoop_branch scoop_repo
-----------                       ------------ ----------
2024-04-04T17:36:54.4070222+08:00 master       https://github.com/ScoopInstaller/Scoop

PowerShell Version
Name                           Value
----                           -----
PSVersion                      5.1.22621.2506
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.22621.2506
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Additional Softwares
No response

Thank you for reporting this.
Should be ok now.


IED LN mapping onto LNode within Substation Section

Related to https://github.com/openscd/open-scd/issues/1128
@danyill suggested a pull request might be good after demonstrating earlier today. But I thought I'd outline what I propose first.
The SSD files we generate contain LNode's which are templates/typicals (iedName=None). They are clues/placeholders for a system integrator to map appropriate LNs from instantiated IEDs to show the secondary system connection to the primary system. Support for the substation section from IED vendors is somewhat lacking, but I feel it's the direction the standard is heading.
One metric to determine whether a substation has been "configured" is to see if there are any LNodes which remain unmapped after the engineering process. This can be beneficial for both the system integrator and the client to show that the specification has been met (in some aspects, but not all).
Here's an example CSWI which is intended to be mapped to a bay controller IED:

This CSWI is generated from a lnType template (using the Template Generator plugin which is very good).
Without too much effort and very quickly a specification SSD file for the substation section can be created.
I propose that a new action is provided to allow for a LN to be mapped from a list of IEDs within the SCD:

Which provides a dialog window to select a LN from an instantiated IED (filters based on the lnClass, and only allows one unmapped LN):

This then updates the LNode attributes from the selected LN:

If a mistake is made, then the LN can be remapped by carrying out the action again.
I thought about an unmap button and make use of the IEC 61850-6-100 LNodeSpecNaming private element to repopulate the original LNode from the template, but LNodeSpecNaming does not have a field to keep the lnType (maybe a deficiency in the proposed namespace). The alternative is to simply not care about lnType, but it feels a bit incomplete to offer a "undo" feature which isn't an exact replica of the original LNode.
This does require some development within the scl-wizarding plugin, so I am somewhat limited in what I can contribute. But I am happy to show what I did (to be honest, it's not pretty).
Prototype Substation editor with this functionality available below (ignore the other stuff, I've been doing a fair bit of experimenting/educating myself about Lit and Material web components).
https://jarradraumati.github.io/scl-editor/

And a SCD file you can use:
XAT_LLN0_mapped.zip


TV off stopped working, turn TV on still works

Installed and I'm still using 3.1.5, windows 11 64bits.
I didn't do any chance that I know of to either tv settings nor settings in the application, but the tv automatic turning off when Sleep or Shutdown is selected has stopped working. It still turns on when I turn the PC on.
I have tried changing the setting from default to early, but it didn't make a difference, any idea what I could do?
Thanks for the app!!

Same here...
Just out of nowhere the app does not seem to work anymore

Hiya, your issues are indicative of a network change which has made the configuration invalid, specifically the PC does not seem to be able to communicate with the TV properly. The first thing I would  like to verify is that the IP adress of the TV is correctly configured (it may have changed unless a static lease is configured in the router)

Mine is set to static, and my TV won't turn off. If I use the "Test" option, it turns off and on just fine. But if I set the Automatic user idle mode management, the TV shows a blank screen saver, but it does not actually turn off. And yes, my windows screen saver is disabled.

Mine is set to static, and my TV won't turn off. If I use the "Test" option, it turns off and on just fine. But if I set the Automatic user idle mode management, the TV shows a blank screen saver, but it does not actually turn off. And yes, my windows screen saver is disabled.

I solved the issue by dong this; control panel -> hardware and sound -> power options -> change powerplan plan setting -> setting the same time for the "Turn off the display" than I have set in the LGTV companion "Automatic user idle mode management."

Hi @FabledNinja great you figured it out how to workaround
Your issue is however related to user idle mode not triggering so it's a different issue than "the TV not turning off when PC goes to sleep". You can execute c:/program files/lgtvdaemon.exe -show to get a status window with more information about why user idle mode does not trigger

Hi @JonasFi and @Kenshokram any feedback wrt the suggestion above?

Hi @JonasFi and @Kenshokram any feedback wrt the suggestion above?

Well I reinstalled the application, set my TV IP adress to be static and that seemed to fix my problem. :)

Sorry I'm in the middle of moving and this went out of my mind, it worked for me just with updating the TV IP, since some ISP changes on my side must've changed them around, and I didn't realize only part of the features failing could still be due to this, thank you!!

Issue closed


Mix drawiing raw juce component and web view content.

https://html2canvas.hertzen.com/
Is better solution?

In solution of react native.
<EMAIL_ADDRESS>https://github.com/react-native-webview/react-native-webview/blob/master/docs/Guide.md#communicating-between-js-and-native

AltLayer Airdrop Season One Announcement

Exciting news! AltLayer token (ALT) is set to launch on Binance on January 25, 2024. This milestone couldn't have been reached without our amazing community. Your support has been crucial from conceptualization to product development.
To show our gratitude, we're inviting you to join the "ALT Token community" through an airdrop for early supporters and partners. Below is a summary:

Total Allocated ALT Tokens: 300,000,000 (3% of total supply)
Distribution Categories:

NFT Holders
Altitude Campaign Participants
EigenLayer Restakers
EigenLayer Ecosystem Partners
Celestia Stakers and others

For detailed criteria and information, check the full announcement here.
Claim opens from January 25 to February 25, 2024. Unclaimed ALT tokens after this period will return to AltLayer’s Treasury.
Thank you for being part of AltLayer's success! More to come in Season One, stay tuned.
Note: Certain regions are restricted from the airdrop. Please review the disclaimer before participating.
Winners: @bobbytonight, @sonpython, @lanweihong, @christian-fei, @MENGCHE-Wu, @sahongzhi, @TyJackson, @chisNaN, @deepblue-phoenix, @joaopmgd, @sulicat, @deju, @wpank, @miighty, @yushengme, @otariski, @covenz, @chenlie449, @rickarubio, @yinhangfeng


[Suggest]: Support work with windows.

Background
From K/pkg/basedir.go. I think we can use os.Getenv replace user.Current(). Because os.GetEnv can get clean infomation. And we only judge user platform that is 'win' or others.

Thanks for your suggestion. We will update in subsequent iterations.

I already saw you're refactor the part. I will close this issue.


Enhancement Request: Allow Options Parameter After Pipeline

Enhancement request: allow the caller to supply second-positional-parameter options.  I saw that there was an earlier request for this feature specifically to enable allowDiskUse, which you mentioned would not work.  I have more modest need: to enable explain.
I have hacked together a custom solution on my system:
explainResults = Meteor.wrapAsync(collection.rawCollection().aggregate.bind(collection.rawCollection()))(pipeline, { explain: true });

This seems to work, I'm getting back helpful explain results.
This could be a low-cost/low-risk change that would improve utility of this package.  IMHO explain results can be quite helpful when developing complex aggregate queries.

Would you mind send me a PR?
On Thu, Jul 23, 2015 at 3:24 AM David Lynch<EMAIL_ADDRESS>wrote:

Enhancement request: allow the caller to supply
second-positional-parameter options. I saw that there was an earlier
request for this feature specifically to enable allowDiskUse, which you
mentioned would not work. I have more modest need: to enable explain.
I have hacked together a custom solution on my system:
explainResults = Meteor.wrapAsync(collection.rawCollection().aggregate.bind(collection.rawCollection()))(pipeline, { explain: true });
This seems to work, I'm getting back helpful explain results.
This could be a low-cost/low-risk change that would improve utility of
this package. IMHO explain results can be quite helpful when developing
complex aggregate queries.
—
Reply to this email directly or view it on GitHub
https://github.com/meteorhacks/meteor-aggregate/issues/22.

Arunoda,
Please bear with me, because I don’t do many of these pull requests:
https://github.com/sotarules/meteor-aggregate/pull/1
I think this is right.   I tried the code locally and it seems to work.
Best regards,
Dave
David Lynch
Software Engineer
SOTA New Minimal 200 X 100
SOTA Enterprises, Inc.
690 South La Luna Avenue
Ojai, California 93023
Voice<PHONE_NUMBER> Skype<PHONE_NUMBER>
From: Arunoda Susiripala<EMAIL_ADDRESS>Sent: Wednesday, July 22, 2015 5:54 PM
To: meteorhacks/meteor-aggregate
Cc: David Lynch
Subject: Re: [meteor-aggregate] Enhancement Request: Allow Options Parameter After Pipeline (#22)
Would you mind send me a PR?
On Thu, Jul 23, 2015 at 3:24 AM David Lynch<EMAIL_ADDRESS>wrote:

Enhancement request: allow the caller to supply
second-positional-parameter options. I saw that there was an earlier
request for this feature specifically to enable allowDiskUse, which you
mentioned would not work. I have more modest need: to enable explain.
I have hacked together a custom solution on my system:
explainResults = Meteor.wrapAsync(collection.rawCollection().aggregate.bind(collection.rawCollection()))(pipeline, { explain: true });
This seems to work, I'm getting back helpful explain results.
This could be a low-cost/low-risk change that would improve utility of
this package. IMHO explain results can be quite helpful when developing
complex aggregate queries.
—
Reply to this email directly or view it on GitHub
https://github.com/meteorhacks/meteor-aggregate/issues/22.

—
Reply to this email directly or view it on GitHub https://github.com/meteorhacks/meteor-aggregate/issues/22#issuecomment-123919188 .  https://github.com/notifications/beacon/AFM5nxF6aapolCJsXg7OUTIue3TTbmU0ks5ogDK7gaJpZM4Fd9DP.gif


Sequencing Sagas example needs to yield

The Sequencing Sagas example reads as such:
function* playLevelOne() { ... }

function* playLevelTwo() { ... }

function* playLevelThree() { ... }

function* game() {
  const score1 = yield* playLevelOne()
  put(showScore(score1))

  const score2 = yield* playLevelTwo()
  put(showScore(score2))

  const score3 = yield* playLevelThree()
  put(showScore(score3))
}

If I understand Sagas correctly, then those put lines should be changed to yield put:
function* game() {
  const score1 = yield* playLevelOne()
  yield put(showScore(score1))

  const score2 = yield* playLevelTwo()
  yield put(showScore(score2))

  const score3 = yield* playLevelThree()
  yield put(showScore(score3))
}

Is that correct?

Yes, you are right. Sent a PR with a fix.


Adding support for Cmder

I don't really like how the logic for selecting/dispatching the different shells now look, but rather than blocking and delaying until I would find time to write a different solution, I decided to just merge this right away and add a comment as a reminder.
I'll release soon. Thanks for working on the feature!

Ok, great, thank you !


fix/article-url-in-readme.md

Fixing article URL in README.md file.

ah, didn't see the PR @hardikSinghBehl - I just fixed it - thanks for the proactive work!


No Sound from Caller when using WhatsApp on MacbookAir

Hi,
when I have a WhatsApp Call or VideoCall, I don't hear the Sound of the other Caller when EqMac is switched on.
If I quit the program or switch off the Main button, I can hear the sound.
Do I have to enable the sound for specific Apps?
Best regards
Jörg

Hi, currently, there are some issues around calling and WhatsApp. I would close this issue now since there is already one that is pretty similar to this. #648


Rebase to newest version of google's model-explorer repo

@madcampos We should do this at some point. Are we expecting a lot of conflicts?

I've synced main with Google's upstream a couple of times. There were not many conflicts, and I was able to resolve all of them.
This commit was the latest sync: https://github.com/tenstorrent/model-explorer/commit/146dd5ae987f66474cb99c2ce44c7683a54a6c78
Also pulled all tags from Google's repo: https://github.com/tenstorrent/model-explorer/tags


Cards under 'A little about me' aren't of the same height

Description: These 4 cards possess different heights, upsetting the overall user experience.

I can work on this issue. Please assign (under SWOC)

Also, make sure the content in the cards is loaded from a file home.js, similar to what we have for the project page

Also, make sure the content in the cards is loaded from a file home.js, similar to what we have for the project page

Can you please explain what exactly does 'Greeting Request Failed' means?

Can you please explain what exactly does 'Greeting Request Failed' means?

Can you please explain what exactly does 'Greeting Request Failed' means?

Please let me know the issue.

It's a bug with the GitHub action. Nothing wrong from your end😊

Can you please explain what exactly does 'Greeting Request Failed' means?

Please let me know the issue.

It's a bug with the GitHub action. Nothing wrong from your end😊


Travis builds sometimes cannot complete due to missing style scripts

Travis sometimes reports that it can't find the required clang formatting scripts.  This was seen originally on OS X (where it was arbitrarily disabled without investigating further for a fix) and it is now showing on GNU/Linux.
An example build failure shows as
./reproducible_build.sh: line 421: ./contrib/devtools/clang-format-diff.py: No such file or directory
+LINES_W_FORMAT_REQUIRED=0

An example log is here:
https://travis-ci.org/lbryio/lbrycrd/jobs/424021818

I think this is fixed via #241. I'll reopen this if I see the error again.


feat(Request): Expose the URI template that was matched for the request

This has been requested by several members of the community who would
like a way to access the URI template in middleware, etc. (e.g., for
logging).
Fixes #619

Current coverage is 99.89% (diff: 100%)

Merging #889 into master will decrease coverage by 0.10%

@@           master       #889   diff @@
========================================
  Files          30         30
  Lines        1897       1905     +8
  Methods         0          0
  Messages        0          0
  Branches      314        314
========================================
+ Hits         1897       1903     +6
- Misses          0          2     +2
  Partials        0          0

Powered by Codecov. Last update d9d1aed...c8c3ac0

:+1:

LGTM2!!!


Validation bugfix - do not run phone validation for empty strings

This is also applicable in case where there was previously phone number and user wants to delete it. Previously he couldn't - empty string (or +1) would be run through phone validation and $valid would be set to false.
See this example:

I agree with @lepipele the required attribute can be added to the directive if the developer so chooses. Great Idea. @lepipele Please add a test that verifies that the form in valid when empty.

@lepipele thanks for the PR!
Would you please add testing to verify that empty and emptied values are valid?
Also, would you please use conventional changelog format for your commit message? I'll add a contributing document for clarity.

@SyntaxStacks would this test work:
it('should set the field as valid with empty input', function () {
	angular.element(element).val('').trigger('input');
	$scope.$digest();
	expect(form.tel.$error.ngIntlTelInput).toBeUndefined();
	expect(form.tel.$valid).toBe(true);
});

@bighappyface I am not sure I am able to write proper messages since I've never used that conventional changelog - I can delete my repository and re-create pull request with this message:
fix(directive): do not run phone validation for empty strings
Also included test

@SyntaxStacks @bighappyface Any response on my previous post? OK to incorporate those changes and create new pull request?

@lepipele take a stab at both the message and the test and we are happy to review.
Some guidance:

For the commit message try something like fix(directive): skip validation for empty values
For the unit test, your proposal seems suitable. Give it a shot.

Created pull request #26 - I'll close this one


Remove user.meta-data config key

In [LXD] First class cloud-init support  @stgraber noted that user.meta-data "will just go away completely".
Is this still the plan? I don't have a specific use-case however I contemplated to use it pass additional data to cloud-init before deciding that I can just as easily modify the scripts or YAML.

user.meta-data is being replaced by cloud-init.meta-data. If the latter works for you, use that as it will be more future proof.
LXD itself doesn't have any particular handling of user.meta-data, it's normally handled by templates in the images. As we don't control all of those, we don't have a clear timeline for the full disappearance of user.user-data in favor of cloud-init.user-data.

I'm confused. First you talk about user.meta-data but then about user.meta-data, which are different config keys.
The user.meta-data config key has special handling: it is added to the container meta-data in the instance API.