Templating in policy

This PR enables using templating in Policy with data from the request when validating the request.
One use case would be to allow dnsNames with the pattern "*.{{ .Request.Namespace }}.svc" but there are certainly other use cases.
This PR does not yet include any documentation (only comments in code) but if you agree this is a good idea, I'll add some as well.
Closes #62

I assume that you'll want me to update the website docs as well at approver-policy.md but I'm going to wait for positive feedback before putting in any more work.

/ok-to-test

@inteon No problem! You'll look at it when you have the time. Thank you for enabling tests.

@smuda I'm wondering if CEL wouldn't be a better match for this kind of problem?

Could you list a couple of CertificateRequestPolicy examples?
For example, are my use cases in #62 met by this PR?

This is how one CertificateRequestPolicy would have looked with this PR:
apiVersion: policy.cert-manager.io/v1alpha1
kind: CertificateRequestPolicy
metadata:
  name: my-policy
spec:
  allowed:
    dnsNames:
      values:
      - "*.{{ .Request.Namespace }}.svc"

This would have handled my usecase and I think it should have handled yours as well.
But it doesn't really matter since this PR was not accepted by the cert-manager team and after a discussion I'll go down a different route with another PR.

This is how one CertificateRequestPolicy would have looked with this PR:
apiVersion: policy.cert-manager.io/v1alpha1
kind: CertificateRequestPolicy
metadata:
  name: my-policy
spec:
  allowed:
    dnsNames:
      values:
      - "*.{{ .Request.Namespace }}.svc"

This would have handled my usecase and I think it should have handled yours as well.
But it doesn't really matter since this PR was not accepted by the cert-manager team and after a discussion I'll go down a different route with another PR.

Indeed, we did conclude that adding a full templating engine to this project opens up many new possible bugs and vulnerabilities. We agreed that it would be better to implement this functionality in a plugin first before adding it to this repo.

We agreed that it would be better to implement this functionality in a plugin first before adding it to this repo.

Did someone start on a plugin? If not, I would like to take a shot at implementing a POC plugin based on CEL expressions declared as plugin values. With support for a limited set of CSR attributes. In our use case, we are interested in using this feature to validate dnsNames and uris, so I'll probably start with those.


With tabular input, the active field  not receive the model  real attribute for adding an has-error class

The method addErrorClassIfNeeded does not have an helper Html::getAttributeName to get the real attribute

Q
A

Yii version
<IP_ADDRESS>

PHP version
7.2

Operating system
windows 10

Duplicated  #15801

@FabrizioCaldarelli Sorry, did not check the search for this issue


segmentation fault in git spr update

Ran git spr update on a 5-long stack to update it. The last commit in the stack is WIP
$ git spr version
> git rev-parse --show-toplevel
0.12.2 : 2023-04-24T18:35:43Z : 85b71efd

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x0 pc=0x100f19bdc]

goroutine 22 [running]:
github.com/ejoffe/spr/github/githubclient.formatStackMarkdown({{0x14000366db5, 0x8}, {0x14000366cec, 0x28}, {0x14000366d73, 0x2e}, {0x100fa3a30, 0x0}, 0x0}, {0x14000180900, ...})
        /Users/runner/work/spr/spr/github/githubclient/client.go:399 +0x5c
github.com/ejoffe/spr/github/githubclient.formatBody({{0x14000366db5, 0x8}, {0x14000366cec, 0x28}, {0x14000366d73, 0x2e}, {0x100fa3a30, 0x0}, 0x0}, {0x14000180900, ...})
        /Users/runner/work/spr/spr/github/githubclient/client.go:419 +0x64
github.com/ejoffe/spr/github/githubclient.(*client).UpdatePullRequest(0x1400000d110, {0x1010dc2f0, 0x1400011a000}, {0x1010dc478, 0x1400000d0c8}, {0x14000180900, 0x5, 0x8}, 0x140002a0300, {{0x14000366db5, ...}, ...}, ...)
        /Users/runner/work/spr/spr/github/githubclient/client.go:510 +0x3c8
github.com/ejoffe/spr/spr.(*stackediff).UpdatePullRequests.func2(0x1400000c390, 0x140000770e0, {0x1010dc2f0, 0x1400011a000}, {0x14000180900, 0x5, 0x8}, 0x14000262540, 0x1)
        /Users/runner/work/spr/spr/spr/spr.go:229 +0xd8
created by github.com/ejoffe/spr/spr.(*stackediff).UpdatePullRequests
        /Users/runner/work/spr/spr/spr/spr.go:227 +0x6f8
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x0 pc=0x100f19bdc]

I just hit the same seg fault also with a WIP on the top of the stack. I didn't see a change in 0.13 that looks like it would resolve it but going to upgrade and see if that solves it.


stm32F4 port

Hi There
Im struggling to get this running on the stm32F401 Nucleo Board. Im using CubeMx to generate all the project files for True Studio and it looks like the F4 uses a unique DMA controller. Have you tried to run this on the f401 by any chance?
thanks

Hello,
I've opened reference manual for F303 and F401 and the DMA is a little different (not sure how ST HAL for F401 deals with that). You have to correct the TIM2 DMA Channels/Streams on F401. Compare the F303 reference manual Table 78 to F401 reference manual Table 28. You have to correctly set three DMA for TIM2_UP, TIM_2_CH1 and TIM2_CH2. You can also select completely different TIMer if you for example use TIM2 just for PWM.
I've created dirty image which would help you. If you'll have no luck, let me know. I try that on my F407.
https://dl.dropboxusercontent.com/u/2198455/stm32f303 STM32f401 ws2812b lib.png

Hi. Any luck to bring it running on stm32f4?

Hi, hit F5. I've just added port few minutes ago on my git :) STM32F407

https://github.com/hubmartin/STM32F4_WS2812B
https://twitter.com/hubmartin/status/814812529696407552

It's just a NY gift. Thank you!

Please let me know if you have any issues with running the code, the way the library is written or if you think something should be better documented. Thanks.

Just checked out https://github.com/hubmartin/STM32F4_WS2812B on nucleo stm32f446. Works like a charm, thank you!

Thanks for the direction! I haven't had a chance to try it out yet but ill let you know as soon as I have.

Recently I have been trying to learn your code PC0~4>> PA6-pa7 (TIM3_ch1ch2), could you help me have a look
WS2812B_STM32F4-master_try.zip


include_code Jekyll snippets don't work in non-WF templates

The {% include_code _code/dev-orientation.html devori javascript %} Jekyll snippets only work in the standard Web Fundamentals templates, they do not work in updates, shows, showcases, etc.
The include_code code expects the template to be formed in a specific way and closes expected divs, when they're not there, it screws up the existing template.
We'll aim to fix this in the MDL redesign.

Pete do you recall a scenario that shows this breakage?

Yep - try putting a _code folder in /updates and reference that.


[Bug] Enable Dependabot on the repository

Search before asking

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

Environment
Other
EventMesh version
Other
What happened
Since EventMesh has a lot of dependencies and the upgrades are done manually, the TAR distribution contains many libraries with known vulnerabilities.
For examples Jackson 2.13.0 is included multiple times in the distribution.
One way to solve these problems is to enable Dependabot (cf. documentation) or another dependency manager on the repository. Dependabot is highly configurable. You can upgrade every dependency or just those that are vulnerable.
How to reproduce
Check the contents of the distribution archive and look for Jackson 2.13.0.
Debug logs
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct *

We have many PRs submitted by dependabot, but we can't merge it directly, because it will cause dependency&license check fail. Do you have any suggestions?

We have a merge-dependabot-reusable.yaml Github Actions script that merges Dependabot PR's automatically if they pass the tests. @vy, any ideas how to adapt this to EventMesh?
The main problem, as I see it that EventMesh has a binary distribution with third-party dependencies, while Log4j doesn't. Therefore your merge Dependabot script must be more complex to adhere to the Apache "Assembling LICENSE and NOTICE' policy.
I can try to enhance your check-dependencies.sh so that it does not fail if a dependency version changes. Most of the time (i.e. if the new version does not have new dependencies) it should be enough to merge the Dependabot PR automatically.
IIRC Apache Airflow also has a complex CI deployment to upgrade several Python packages a day. @potiuk, is there some part of your infrastructure that can be adapted to a Java project?

If I understand it right, for a particular dependency (e.g., Log4j), these four locations must be aligned:

build.gradle
tools/third-party-licenses/NOTICE#L626
tools/dependency-check/known-dependencies.txt
tools/third-party-licenses/licenses/java/LICENSE-log4j-api.txt

[Please correct me if I am missing any above.]
You can follow a strategy as follows:

Break down the tools/third-party-licenses/NOTICE file into multiple tools/third-party-licenses/notices/*/NOTICE-*.txt files. This is similar to what you already have for licenses, i.e., tools/third-party-licenses/licenses/*/LICENSE-*.txt.
Enrich known-dependencies.txt to contain the link patterns formatted by version to external NOTICE and LICENSE files. For instance, you can use the following link pattern for the Log4j NOTICE file: https://github.com/apache/logging-log4j2/blob/rel/<version>/NOTICE.txt.
Implement a dependabot-specific GHA workflow such that if NOTICE and LICENSE files still do match and tests pass, update the version in build.gradle and known-dependencies.txt files.

If you are interested in, I can implement this as a contractor and contribute to EventMesh.
Disclaimer: I am a long time Log4j contributor and Apache Logging Services (log4j, log4cxx, etc.) PMC Chair.

@vy

these four locations must be aligned

another location shall be aligned: tools/third-party-licenses/LICENSE.
Thank you for proposing the solution. I find it feasible.
@xwm1992 @qqeasonchen , do you think we can further streamline the five files mentioned above?

For licensing problems there is also an Apache Creadur Whisker project, although I don't know what is it worth.

@vy
In your experience and knowledge, do you think our LICENSE and dependency management are excessively configured?
For example, we attach a txt file for every third-party dependency (e.g., tools/third-party-licenses/licenses/java/LICENSE-log4j-api.txt). Is it necessary to declare the licenses of these dependencies again in tools/third-party-licenses/LICENSE?
Regarding dependency management, I haven't seen any other projects using the tools/dependency-check/known-dependencies.txt file. The RocketMQ project doesn't perform dependency checks, while Kafka uses a Gradle plugin with minimal configuration. Does this mean that the tools/dependency-check/known-dependencies.txt file is redundant?

In your experience and knowledge, do you think our LICENSE and dependency management are excessively configured?

@Pil0tXia, I wouldn't say so. There are no plug-n-play tooling to make the life easier for 1) Gradle/Maven application developers 2) that need to provide a binary distribution 3) complying with the ASF distribution requirements. Airflow, Kafka, etc. they all roll out their own solutions. These efforts are mostly manual, that is, people occasionally check the validity of these files and update them as they see fit.
Note that point (2) – that is, the fact that you distribute binaries – is crucial. For instance, Log4j doesn't distribute binaries containing dependencies. Hence, there, we don't have this problem. We only maintain a NOTICE manually, which is almost fixed for decades.

For example, we attach a txt file for every third-party dependency (e.g., tools/third-party-licenses/licenses/java/LICENSE-log4j-api.txt). Is it necessary to declare the licenses of these dependencies again in tools/third-party-licenses/LICENSE?

No, I don't think so.

Regarding dependency management, I haven't seen any other projects using the tools/dependency-check/known-dependencies.txt file. The RocketMQ project doesn't perform dependency checks, while Kafka uses a Gradle plugin with minimal configuration. Does this mean that the tools/dependency-check/known-dependencies.txt file is redundant?

I don't think so. You have license check as a part of the build. This is great. You also partially implemented a framework to automatically include the LICENSE, etc. files while creating a distribution. But updates to this framework are not automated yet.
In conclusion, I don't think you are over-engineering. I think you are trying to do the right thing. I will consult to<EMAIL_ADDRESS>on this matter and see if we can simplify the process before getting down to implement an automation pipeline around it.

For libraries that don't bundle their dependencies there is some automatic tooling: maven-apache-resources. It contains Velocity templates that generate META-INF/LICENSE, META-INF/NOTICE and META-INF/DEPENDENCIES files.
For example the last one in log4j-core looks like this:
From: 'com.github.luben'

  - zstd-jni (https://github.com/luben/zstd-jni) com.github.luben:zstd-jni:jar:1.5.5-11
    License: BSD 2-Clause License  (https://opensource.org/licenses/BSD-2-Clause)

From: 'Conversant Engineering' (http://engineering.conversantmedia.com)

  - com.conversantmedia:disruptor (https://github.com/conversant/disruptor) com.conversantmedia:disruptor:jar:1.2.15
    License: The Apache License, Version 2.0  (http://www.apache.org/licenses/LICENSE-2.0.txt)

From: 'FasterXML' (http://fasterxml.com)

  - Woodstox (https://github.com/FasterXML/woodstox) com.fasterxml.woodstox:woodstox-core:bundle:6.5.1
    License: The Apache License, Version 2.0  (http://www.apache.org/licenses/LICENSE-2.0.txt)

I believe that for projects that bundle their dependencies a similar mechanism could be used. Notice files are more complex (but fortunately uncommon), but license file IMHO could be listed in a file like the example above (and bundled in a single copy).

I have shared this issue in<EMAIL_ADDRESS>(Note that it is only accessible by ASF members.)

@Pil0tXia, I would like to share some updates from the<EMAIL_ADDRESS>thread:

You are not over engineering, you are doing the right thing.
To my surprise, almost all projects manually collect and check the license information.
You can indeed massively ease dependency updates: hook up to dependabot PRs, auto-merge the PR iff build succeeds and license of the new version matches with the one of the old.  Otherwise, block the PR and add a PMC-review-needed label.

I figured that license information is available in pom.xmls and can easily be accessed/downloaded, no need for manually storing link patterns, etc. as I suggested earlier. No need to store licenses, etc. either. They can be download while creating the distribution.
I am still interested in this gig. Let me know what you think.

@vy
I agree with your idea. Thank you very much for the insights you've provided.
When EventMesh creates a distribution, we execute the dist and installPlugin tasks from build.gradle. We can create a new task that automatically downloads LICENSE files, and then utilize its outcome in the dist task.

May we make some progress here? ☺

@vy
I couldn't find the necessary configuration to allow modifications to the known-dependencies.txt file in dependabot.yml. Do you have any relevant reference materials on this? Thank you very much.

I think dependabot does not support editing known-dependencies.txt file in GitHub Actions after reading all dependabot documents. I've decided to weaken the version number determination of check-dependencies.sh.

@Pil0tXia,
No, it doesn't support your known-dependencies.txt format, but it supports the standard libs.versions.toml file.
My approach to this issue would be to:

Switch to the libs.versions.toml file for dependency versioning. This also requires removing the Spring dependency-management-plugin you are currently using,
Manually create a "bundle" (in libs.versions.toml) that contains all the third-party dependencies that end up in your binary distribution. E.g.:[libraries]
accessors-smart = { group = 'net.minidev', name = 'accessors-smart', version = '2.4.7' }
alibabacloud-gateway-spi = { group = 'com.aliyun', name = 'alibabacloud-gateway-spi', version = '0.0.1' }
...

[bundles]
dist = ["accessors-smart", "alibabacloud-gateway-spi", ... ]

Create a Groovy script that will use the data in the bundle to check that:

your binary distribution archive contains exactly the dependencies from the "bundle",
your binary distribution contains a license file for each dependency from the "bundle".

Implement automatic merging of Dependabot PRs, which @vy implemented in Apache Logging Services.

This should be a rather safe setup since:

The TOML file contains structured data, so a Groovy script can easily deduce that a file called accessors-smart-2.4.7.jar corresponds to the accessors-smart entry, which should have a license file in licenses/net.minidev/LICENSE-accessors-smart.txt,
Dependabot will only update the "libraries" section of your file, it will never add or remove libraries from "bundles". So if a new transitive dependency appears or disappears, the Dependabot PR will fail, so you can manually add or remove a license file.

Remark: without the last point

An alternative approach would be through SBOM generation:

You generate an SBOM for your binary distribution (there is a Gradle plugin for that),
The CycloneDX format has all the data you need to determine which license applies to which component,
You can create a script that generates your licensing data.
According to the INFRA guide on licensing you don't need a file for each dependency, but you need to modify the main LICENSE file in your binary distribution to mention the licenses of all the bundled dependencies. Therefore the structure of your binary distribution could look like:LICENSE
NOTICE
licenses/
    Apache-2.0.txt
    BSD-3-Clause.txt

where LICENSE is generated and looks like:<text of Apache 2.0 license>

This distribution contains the following third-party:

lib/accessors-smart-2.4.7.jar licensed under 'Apache-2.0'. For details see: licenses/Apache-2.0.txt
lib/super-widget-1.2.3.jar licensed under 'BSD-3-Clause'. For details see: licenses/BSD-3-Clause.txt

I found that the plugin https://github.com/jk1/Gradle-License-Report has been introduced in the project to generate plugin reports. However, the LICENSE file is exported to the build/reports/dependency-license path only if the third-party dependency jar package contains the LICENSE file. It exports 139 copies of LICENSE, far fewer than the 306 third-party dependencies the project is currently using.
While the html report it generates declares 300+ dependencies in full, it does not provide the contents of the LICENSE file for each one. Trying to generate a LICENSE file for a project using html as a data source is difficult.
The existing LICENSE file for the project has only 200+ dependencies in it, which is also incomplete. It seems that this plugin is hardly a help for generating LICENSE files.
However, the json file generated by cyclonedx-gradle-plugin is too huge. I am writing gradle scripts based on its data format.
https://github.com/apache/eventmesh/pull/4827 mainly addresses the dependencies problem. I'll address license problem in another PR.

@ppkarwasz

You have about 300 deps, which means around 10-20 Dependabot PRs daily! The automatic merge is really required.

Hi, I've made several attempts at https://github.com/apache/eventmesh-dashboard/issues/122, https://github.com/apache/eventmesh-dashboard/issues/128 and https://github.com/apache/eventmesh-dashboard/issues/130. It looks like the most we can do is auto-approve a Dependabot PR, but can't do merge it.
I looked at merge-dependabot-reusable.yaml. It deals with git, bypassing GitHub's PR mechanism in favor of committing directly on the master branch.
I'm not sure if the GPG_SECRET_KEY key exists in our repository. This key seems to need to be manually requested from infra before it can be used.

@Pil0tXia,
The GPG_SECRET_KEY most certainly does not exist in your repo, it must be generated and added manually by INFRA. See our INFRA-23996 for how our key was set up, you can reference it in your INFRA ticket. Our GPG key is assigned to the Apache Logging Services PMC. For security reasons we can only access it via Github Actions, but we can invalidate the key if it is abused. The key is somehow bound to the asf-rm GitHub user and the commits it creates appear to be generated by that bot account.
The automation of Dependabot merges is not so trivial: it took @vy a week or two of full-time work to figure out all the details (fortunately we had a sponsor for that). I was than able to reproduce it in half a day in copernik-eu/log4j-tomcat/.github/workflows/merge-dependabot.yaml.
The basic idea is that we follow the manual procedure to merge PRs:

We merge/cherry-pick/rebase ${{ github.head_ref }} (e.g. refs/heads/dependabot/gradle/org.mongodb-mongodb-driver-3.12.14) with ${{ github.base_ref }} (e.g. refs/heads/master),
We push the commit to ${{ github.head_ref }} (might require -f) to update the PR,
After a couple of seconds we push the same commit to the master branch ${{ github.base_ref }},
GitHub realizes that ${{ github.head_ref }} and ${{ github.base_ref }} now point to the same commit and marks the PR as merged.

There are some pitfalls you should be aware of:

The workflow must be triggered by the pull_request_target, because workflows triggered by pull_request don't have write access to the Git repo.
The pull_request_target event is special: the github.ref is the same as github.base_ref (e.g. refs/heads/master). Since most actions implicitly use github.ref, you could end up running your build and tests on the master branch instead of the PR branch. To fix this I had to add an explicit ref parameter to actions/checkout:- name: Checkout repository
  uses: actions/checkout@9bb56186c3b09b4f86b1c65136769dd318469633   # 4.1.2
  with:
    # When running on `pull_request_target` use the PR branch, not the target branch
    ref: ${{ github.event_name == 'pull_request_target' && github.head_ref || github.ref }}

The checkout action by default does a shallow clone: only the last commit is downloaded so Git can not always find the branching point where the master branch and PR branch diverged. Usual merge/cherry-pick/rebase commands might not work unless you change the depth parameter of the action. Volkan works around this by downloading the PR as patch and applies it to the master branch:- name: Download and apply patch
  shell: bash
  env:
    PATCH_URL: ${{ github.event.pull_request.patch_url }}
   run: |
     wget -O- "$PATCH_URL" | git apply

In the workflow of my personal repo I increase depth to an arbitrary 32 commit (I could also set it to 0 to make a complete clone.
The checkout action only clones one branch. If you need two branches for a merge/cherry-pick/rebaseoperation, you need to callgit fetch` explicitly.

Debugging tips
It is usually easier to test automatic Dependabot merges in your personal fork, since you can add your GPG key as repository secret.
If you run out of Dependabot PRs, you can:

downgrade a dependency,
go to Insights > Dependency graph > Dependabot (for this repo it will be at https://github.com/apache/eventmesh/network/updates) and trigger a Dependabot run manually


Feature request

Hello
This project is amazing!
I would just like to suggest a trailing similar to that of 3commas, with the possibility of configuring a small deviation, thus ensuring a greater profit. Take a look at what it would be like:
https://help.3commas.io/en/articles/3108982-trailing-take-profit-catch-the-rise
Thanks!

Hi thanks for you kind words, I will look into your suggestion and try to implement it once I have the time for it!

@xLinuxUserx How would you like to see this? I was thinking about doing it for OCO and Limit orders and having a set variable of the % of how big the trail should be. However that percentage could also be set automitcally by calculating the procentual difference between the coin at the time of setting the limit order and the limit order itself, like how it goes with the stop limit order now. Let me know what you think!

@StephanAkkerman
Thanks for answering me!
I would just like to be able to set a profit and a callback so that I don't lose much of the profit already made. Same as reported at: https://www.binance.com/en/support/faq/360042299292


Running sqla example.  email_validator dependency for sqlalchemy-utils missing

Topic.
Created pull request for this #2000

I just ran into the same issue.  Any chance this PR can be merged?

+1 to this. Looking at some old logs it looks like email-validator was causing some tests to fail and it was added to ./requirements-dev.txt, but strictly speaking following the README for the sqla example it fails by not being in ./examples/sqla/requirements.txt.

Fixed with #2094.


npm WARN deprecated<EMAIL_ADDRESS>

Hi,
You might want to update some of your dependencies ?
npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained. Please, upgrade to core-js@3 or at least to actual version  of core-js@2.

Please include the steps to reproduce this issue.

> npm ls core-js
├─┬<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   └─┬<EMAIL_ADDRESS>│     └──<EMAIL_ADDRESS>

We won't be using recompose in v4 anymore.


Parse ID3v2 timestamps

Summary
We could parse timestamps in the following frames: TDEN, TDOR, TDRC, TDRL, TDTG.
Instead of FrameValue::Text, they would now use FrameValue::Timestamp.
The structure is described in the ID3v2.4 frame overview:

The timestamp fields are based on a subset of ISO 8601. When being as precise as possible the format of a time string is yyyy-MM-ddTHH:mm:ss (year, “-”, month, “-”, day, “T”, hour (out of 24), ”:”, minutes, ”:”, seconds), but the precision may be reduced by removing as many time indicators as wanted. Hence valid timestamps are yyyy, yyyy-MM, yyyy-MM-dd, yyyy-MM-ddTHH, yyyy-MM-ddTHH:mm and yyyy-MM-ddTHH:mm:ss. All time stamps are UTC.

API design
pub struct TimestampFrame {
    // Year is required, and ID3v2 does not allow negative years
    pub year: u16,
    pub month: Option<u8>,
    pub day: Option<u8>,
    pub hour: Option<u8>,
    pub minute: Option<u8>,
    pub second: Option<u8>,
}

pub enum FrameValue {
    Timestamp(TimestampFrame),
    /// ...
}

Leading zeros in month/day fields are often omitted or replaced by space(s). The parser should tolerate those violations from the standard, because they are very common.
If these recoverable errors could be detected easily then ParsingMode::Strict might reject them. If that would result in 2 different implementations than it's probably not worth the effort.


add sqlfluff formatter for postgres

Here I'm contributing some of the local formatters I have configured, which could be interesting for anyone else.
Ceveat here, not sure how to handle this, sqlfluff needs the --dialect argument for formatting sql.
So either we would need to document how to adapt this, or keep it like it is right now.

@syphar sqlfluf dialect can be configured per project with a .sqlfluff file

I thought it would be confusing when the default formatter config just doesn't work, but I see you already added another config in 439c7138bd4f8fa583722ea956e915ca486d85e5.
I'll check how this looks like :)


Backport "HBASE-28837 Introduce row statistics coprocessor example (#6327)" to branch-3

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 43s

Docker mode activated.

-0 :warning:
yetus
0m  3s

Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck

_ Prechecks _

_ branch-3 Compile Tests _

+1 :green_heart:
mvninstall
2m 46s

branch-3 passed

+1 :green_heart:
compile
0m 21s

branch-3 passed

+1 :green_heart:
javadoc
0m 15s

branch-3 passed

+1 :green_heart:
shadedjars
4m 59s

branch has no errors when building our shaded downstream artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 43s

the patch passed

+1 :green_heart:
compile
0m 21s

the patch passed

+1 :green_heart:
javac
0m 21s

hbase-examples generated 0 new + 5 unchanged - 3 fixed = 5 total (was 8)

+1 :green_heart:
javadoc
0m 16s

the patch passed

+1 :green_heart:
shadedjars
4m 54s

patch has no errors when building our shaded downstream artifacts.

_ Other Tests _

+1 :green_heart:
unit
1m 55s

hbase-examples in the patch passed.

20m 24s

Subsystem
Report/Notes

Docker
ClientAPI=1.47 ServerAPI=1.47 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6554/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6554

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 4bd8c2906f5f 5.4.0-200-generic #220-Ubuntu SMP Fri Sep 27 13:19:16 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-3 / 4372e03c002824a1012be0c5be7b2d00e4d6e02e

Default Java
Eclipse Adoptium-17.0.11+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6554/1/testReport/

Max. process+thread count
1472 (vs. ulimit of 30000)

modules
C: hbase-examples U: hbase-examples

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6554/1/console

versions
git=2.34.1 maven=3.9.8

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 27s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
hbaseanti
0m  0s

Patch does not have any anti-patterns.

_ branch-3 Compile Tests _

+1 :green_heart:
mvninstall
3m  1s

branch-3 passed

+1 :green_heart:
compile
0m 27s

branch-3 passed

+1 :green_heart:
checkstyle
0m  9s

branch-3 passed

+1 :green_heart:
spotbugs
0m 28s

branch-3 passed

+1 :green_heart:
spotless
0m 44s

branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 50s

the patch passed

+1 :green_heart:
compile
0m 24s

the patch passed

+1 :green_heart:
javac
0m 24s

hbase-examples generated 0 new + 13 unchanged - 2 fixed = 13 total (was 15)

+1 :green_heart:
blanks
0m  1s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m  8s

the patch passed

+1 :green_heart:
xmllint
0m  0s

No new issues.

+1 :green_heart:
spotbugs
0m 34s

the patch passed

+1 :green_heart:
hadoopcheck
10m 39s

Patch does not cause any errors with Hadoop 3.3.6 3.4.0.

+1 :green_heart:
spotless
0m 41s

patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
0m 10s

The patch does not generate ASF License warnings.

27m 27s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6554/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6554

Optional Tests
dupname asflicense javac codespell detsecrets xmllint hadoopcheck spotless compile spotbugs checkstyle hbaseanti

uname
Linux 3ec5e345d02a 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-3 / 4372e03c002824a1012be0c5be7b2d00e4d6e02e

Default Java
Eclipse Adoptium-17.0.11+9

Max. process+thread count
86 (vs. ulimit of 30000)

modules
C: hbase-examples U: hbase-examples

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6554/1/console

versions
git=2.34.1 maven=3.9.8 spotbugs=4.7.3 xmllint=20913

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.


how to install?

i think i'm a big dumdum, i have no idea how to install this applet :/ probably its something wery easy but right now i'm stuck :D

Hi,
This is the source code, please download the binary’s here : http://lucaspardue.com/?wpdmdl=228.
Greetings.

oh, i see now :D do i have to open this app every time? or one opend it's always in g13?

Depending on your system. I suggest to add it to "Start at startup" programs.
Here : http://windows.microsoft.com/en-us/windows/run-program-automatically-windows-starts#1TC=windows-7
Add the EXE file `SpotifyStatusApplet.exe" at startup, and it should be start each time your keyboard start.

thanks ;) this will be helpful

Please close this issue if it is resolved.

Closing due to inactivity. If this is still a problem then please reopen.


chore(ci): add terraform checks job

Description

Add CI to the repo for Terraform checking

Issue(s)
ENG-966
How to test

The job will fail atm as there is no Terraform config to check

Checklist

[ ] Merge https://github.com/oaknational/oak-terraform-actions/pull/13

LGTM


[BUG] Invalid byte sequence in UTF-8

system: Windows 11
language: chinese
ruby 3.1.6p260 (2024-05-29 revision a777087be6) [x64-mingw-ucrt]
C:\>ruby -e "p RUBY_PLATFORM"
"x64-mingw-ucrt"

found the issue same with https://github.com/mcorino/wxRuby3/issues/302
C:\>ver
Microsoft Windows [版本 10.0.22631.4169]

https://github.com/mcorino/plat4m/blob/2e80faa90edfa35bdf793d2d96bc68dd9b406969/lib/plat4m/sniff/windows.rb#L38-L39
https://github.com/mcorino/wxRuby3/blob/3e922f789cd6ac40f6533fd4930574b26bd06fce/rakelib/lib/config.rb#L559
irb(main):001:0> require 'plat4m'
=> true
irb(main):002:0> Plat4m.current
C:/Ruby31-x64/lib/ruby/gems/3.1.0/gems/plat4m-1.1.2/lib/plat4m/sniff/windows.rb:39:in `match': invalid byte sequence in UTF-8 (ArgumentError)
        from C:/Ruby31-x64/lib/ruby/gems/3.1.0/gems/plat4m-1.1.2/lib/plat4m/sniff/windows.rb:39:in `match'
        from C:/Ruby31-x64/lib/ruby/gems/3.1.0/gems/plat4m-1.1.2/lib/plat4m/sniff/windows.rb:39:in `get_distro'
        from C:/Ruby31-x64/lib/ruby/gems/3.1.0/gems/plat4m-1.1.2/lib/plat4m/sniff/windows.rb:19:in `detect_system'
        from C:/Ruby31-x64/lib/ruby/gems/3.1.0/gems/plat4m-1.1.2/lib/plat4m/system.rb:163:in `detect_current'
        from C:/Ruby31-x64/lib/ruby/gems/3.1.0/gems/plat4m-1.1.2/lib/plat4m/system.rb:145:in `current'
        from (irb):2:in `<main>'
        from C:/Ruby31-x64/lib/ruby/gems/3.1.0/gems/irb-1.4.1/exe/irb:11:in `<top (required)>'
        from C:/Ruby31-x64/bin/irb:33:in `load'
        from C:/Ruby31-x64/bin/irb:33:in `<main>'

found a workaround with the issue, run below command in cmd first, this command alters the console's code page to 437 (United States English).
chcp 437

Ah, now it starts to make sense ;-)
However, I do not really like your workaround.  I would definitely prefer a more 'system independent solution'.
Could you please check what the output of the following Ruby code is on your system (I do not have any system with Chinese codepage available for testing).
s = `for /f "tokens=2 delims=[]" %s in ('ver') do @(for /f "tokens=2" %v in ('echo %s') do echo %v)`.strip
puts "Version = #{s}"

On my system this produces just the version number without any of the decoration text from the ver output.

@mcorino output of the code
c:\>irb
irb(main):001:0> s = `for /f "tokens=2 delims=[]" %s in ('ver') do @(for /f "tokens=2" %v in ('echo %s') do echo %v)`.strip
=> "c:\\>echo 10.0.22631.4169 \n10.0.22631.4169"
irb(main):002:0> puts "Version = #{s}"
Version = c:\>echo 10.0.22631.4169
10.0.22631.4169
=> nil
irb(main):003:0> `ver`
=> "\nMicrosoft Windows [\xB0汾 10.0.22631.4169]\n"

Ok, that's perfect (the code sample I sent you was just missing another '@' before 'echo %v' which is why the ouput was not correct yet).
I'll release an update to the gem implementing this.

This should be fixed in the 1.1.3 release.


Feature/0.40.0

add content missing from function removeConstraints

@CodeEagle I've tidied this up in feature/0.40.0, for future reference make sure you open up PR's against the correct branch ( this is against develop so I couldn't merge it ).
Cheers.

ok， sorry for that


Migrating postman rest client scripts to zerocode json stesps

It will be nice to generate zerocode json steps directly from postman test scripts.

Generate Zerocode steps from postman - this needs integration with Postman to support zerocode framework for code snippet generation.  It will be a advance feature for zerocode framework and it might take more time to have it

There is a possibility of converting the postman collections into zerocode scenarios. But it is not recommended.  Because postman already has multiple versions in their exported collection format. You will have to save the response as an example otherwise only request will be exported. I haven't seen an option to export the postman tests yet. Even if we do it, still it won't be making our life easier.
Let's assume we have converted Postman collection into zerocode compatible JSON, still, we need to insert the assertions and the file paths(This is a long debated postman issue which won't be fixed any time soon).
For maintaining the piece of converter code we need to be in sync with the postman and that's not a good idea. Still, there are people who are using alternative to postman like the open source one Insomnia will not be able to use it.

Since #172  is making progress let's close this.


User Error 1001: omp_set_num_threads

Running the windows build and during training/predicting on the latest push:

User Error 1001: omp_set_num_threads must be called with a positive integer
2018-05-14 14:35:02,451 INFO [default] #instances = 7136, #features = 4096
2018-05-14 14:35:03,938 INFO [default] #classes = 2
2018-05-14 14:36:02,830 INFO [default] working set size = 1024
2018-05-14 14:36:02,837 INFO [default] training start
2018-05-14 14:36:03,336 INFO [default] global iter = 0, total local iter = 512, diff = 2
2018-05-14 14:36:07,721 INFO [default] global iter = 48, total local iter = 16874, diff = 0.000993791
2018-05-14 14:36:07,728 INFO [default] rho = -0.00517282
2018-05-14 14:36:07,730 INFO [default] #sv = 7105
2018-05-14 14:36:10,845 INFO [default] #total unique sv = 7105
2018-05-14 14:36:10,912 INFO [default] training finished

Seem training program seems to be ignoring the OMP_NUM_THREADS set in my environmental variables, any idea how to resolve?

clf = SVC(verbose=True, gamma=0.5, cost=100, n_cores=4)
Setting the n_cores took care of the message, but unclear if CPU or GPU cores?


Create README.md

@ricardozanini @cdavernas I think some config is missing in this repo as you weren't assigned as reviewer by default.


DEP PHP Support in CMS5

Parent issue

https://github.com/silverstripe/silverstripe-framework/issues/10402

Requires https://github.com/silverstripe/cwp/pull/317

Passed tests in local env.
Requires https://github.com/tractorcow/silverstripe-proxy-db/pull/5 to be merged first


Build RTTOV-11 on Terra so that OPS and VAR can use it as an external library

keyword_RTTOV_RTTOV-9_RTTOV-11_radiative_transfer_RTM_OPS_VAR_external_library   |    by<EMAIL_ADDRESS>
Starting from ops-2016.11.0 and var-2016.11.1 RTTOV-11 replaced RTTOV-9. Also RTTOV-11 is now linked as an external library.
This ticket describes modifications to allow the build of RTTOV-11 on Terra.
This wiki page, https://accessdev.nci.org.au/trac/wiki/access/rttov-module contains information about RTTOV-11 build on Raijin.

Issue migrated from trac:303 at 2024-01-31 18:27:31 +1100

<EMAIL_ADDRESS>commented

From,
https://access-svn.nci.org.au/trac/ops/browser/branches/local/extras/RTTOV/rttov113_ops_branch_2121
I created a development branch,
https://access-svn.nci.org.au/svn/ops/branches/dev/jtl548/r2948_rttov113_2121_bom

fs5892 commented

As of the 2017.07 OPS builds, where RTTOV12 and RTTOV11 are both required, the version of RTTOV11 that is needed to compile OPS is this one:
https://access-svn.nci.org.au/svn/ops/branches/local/extras/RTTOV/rttov11-test-frwd-r2440_ops694-rev2448/
This is installed under the name rttov/11.3b_ifort???? where ???? denotes the compiler version.

<EMAIL_ADDRESS>commented

The changes committed at ![2952] (https://access-svn.nci.org.au/svn/ops/branches/dev/jtl548/r2948_rttov113_2121_bom@2952) seems to have resulted in a successful build.
Installed on,
terra02:/g/sc/projects_lustre/pr_nwp/access/apps/rttov/11.3_ifort<IP_ADDRESS>

<EMAIL_ADDRESS>commented

Hi Milton,
Would you be able to create a modulefile for the Terra RTTOV-11 installation,
/g/sc/projects_lustre/pr_nwp/access/apps/rttov/11.3_ifort<IP_ADDRESS>
Thanks.
Jin

<EMAIL_ADDRESS>set owner to jtl548


[4.2][Migrator] Use RewriteBuffer::InsertText() for applying insertion

Cherry-pick of #16171 reviewed by @nkcsgexi
Apparently, ReplaceText() on the same location replaces the replaced
text. Using InsertText() fixes the problem.
This problem used to cause a invalid migration. For example:
_ = FooBar(SomeThing)

// Renaming: FooBar(_:) → Foo.bar(someThing:)
_ = FooBar(someThing: SomeThing)

// Renaming: SomeValue → Some.val
_ = Foo.bar(Some.valg: SomeThing)

rdar://problem/39518867

@swift-ci Please test

@swift-ci Please test


Add examples using WithFileWithPermissions on Windows

What does this PR do?
Add examples on how to use WithFileWithPermissions on Windows
Motivation
Having examples
Additional Notes
Possible Drawbacks / Trade-offs
Describe how to test/QA your changes

/merge


dtc generates missing #address-cells in interrupt provider warning

Describe the bug
dtc version 1.6.1 produces a warning when building applications for ARMv7-m targets. This warning does not cause builds to fail. I recently upgraded my dtc so I'm not entirely sure what dtc version this was introduced with.
To Reproduce
Steps to reproduce the behavior:

Install dtc version 1.6.1
Run west build -b nrf52833dk_nrf52833 zephyr/samples/hello_world (hello_world path depends on workspace setup)

Expected behavior
No warnings produced when dtc is run.
Impact
This is  a small annoyance, but will be encountered frequently
Logs and console output
nrf52833dk_nrf52833.dts.pre.tmp:15.39-20.5: Warning (interrupt_provider): /soc/interrupt-controller@e000e100: Missing #address-cells in interrupt provider
  also defined at nrf52833dk_nrf52833.dts.pre.tmp:354.7-356.3

Environment (please complete the following information):

OS: MacOS
Toolchain: arm-gcc v 9.3.1
Commit SHA or Version used: v2.6.0

Additional context
Adding the following line seems to remove the warning but I am not knowledgeable enough on the DeviceTree to know if this
fix is valid or how to apply it to other architectures:
diff --git a/dts/arm/armv7-m.dtsi b/dts/arm/armv7-m.dtsi
index cd244f4594..9f2dc66734 100644
--- a/dts/arm/armv7-m.dtsi
+++ b/dts/arm/armv7-m.dtsi
@@ -11,6 +11,7 @@
                ranges;

                nvic: interrupt-controller@e000e100  {
+                       #address-cells = <1>;
                        compatible = "arm,v7m-nvic";
                        reg = <0xe000e100 0xc00>;
                        interrupt-controller;

It looks like this will affect other ARM cores but again I'm far from an expert.

@ejohnso49 this fix looks good.  Can you send a PR with this and add fix to all the files in dts/arm

@galak Researching a little bit more and I think this applies to any node that has the interrupt-controller property. I also read up on #address-cells. I think my prior suggestion is incorrect.
I do not see any dts files defining nodes inside of interrupt-controller nodes as far as I can tell. I also found this example in some Linux dts files<EMAIL_ADDRESS>Since these nodes don't have any children I think setting #address-cells to <0> is correct.

Further research seems to say that interrupt-controllers will only have non-zero #address-cells if interrupt-maps are used. I didn't find any of this usage in the code base except for edtlib testing.

@cfriedt I already approved the PR; we are waiting on @galak to approve it before merging. Why did you assign me and unassign him? Seems backwards...


Research reinforcement learning

Reinforcement learning techniques seem relevant to the bandit approach.
https://scikit-learn.org/stable/faq.html#why-is-there-no-support-for-deep-or-reinforcement-learning-will-there-be-support-for-deep-or-reinforcement-learning-in-scikit-learn

https://github.com/dancrew32/ab/issues/10 dupe


Version 0.4.20 doesn't work in the browser

I just upgraded from 0.4.5 to 0.4.20 and I get these errors using webpack:
./node_modules/javascript-lp-solver/src/External/lpsolve/main.js
Module not found: Error: Can't resolve 'child_process' in 'C:\Users\Daan\code\phantasmal-world\node_modules\javascript-lp-solver\src\External\lpsolve'
./node_modules/javascript-lp-solver/src/External/lpsolve/main.js
Module not found: Error: Can't resolve 'fs' in 'C:\Users\Daan\code\phantasmal-world\node_modules\javascript-lp-solver\src\External\lpsolve'

Is browser support no longer a feature?
Edit: 0.4.17 does work in the browser.

It is, and that’s on me...
I need better browser testing on my side.
Immediate solution?: Try using 4.16
I don’t have time to fix today but tomorrow I should.
What steps can I take to replicate its failure?
Are you bringing it in through a script tag?
Does it fail when you try and solve a model or when you call something else?
Are you building it yourself or using the one prebuilt in the package?
On Sat, Oct 26, 2019 at 10:35 AM Daan Vanden Bosch<EMAIL_ADDRESS>wrote:

I just upgraded from 0.4.5 to 0.4.20 and I get these errors:
./node_modules/javascript-lp-solver/src/External/lpsolve/main.js
Module not found: Error: Can't resolve 'child_process' in 'C:\Users\Daan\code\phantasmal-world\node_modules\javascript-lp-solver\src\External\lpsolve'
./node_modules/javascript-lp-solver/src/External/lpsolve/main.js
Module not found: Error: Can't resolve 'fs' in 'C:\Users\Daan\code\phantasmal-world\node_modules\javascript-lp-solver\src\External\lpsolve'
Is browser support no longer a feature?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JWally/jsLPSolver/issues/96?email_source=notifications&email_token=AAS6F53C4ZE2EAPECORLXB3QQRPTTA5CNFSM4JFNWXCKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HUROYVQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAS6F57ACKI54M2SNADORODQQRPTTANCNFSM4JFNWXCA
.

We're using it in this project: https://github.com/DaanVandenBosch/phantasmal-world (first line of this file: https://github.com/DaanVandenBosch/phantasmal-world/blob/master/src/hunt_optimizer/stores/HuntOptimizerStore.ts).
We use the package from npmjs (through yarn). And we use webpack to load it as a module. It's actually webpack that throws the error (before we can even load the module) because it can't find "child-process" and "fs", which are node built-in modules. The errors seem to stem from this file: https://github.com/JWally/jsLPSolver/blob/master/src/External/lpsolve/main.js
And 0.4.17 is the most recent version available on npmjs that does work and it's the version we're using at the moment.

I built out a better "test-suite" to make sure the library works in the browser.
1.) node server.js
2.) Aim a browser @ http://localhost:9000/test/test.html
However, I'm not sure if this helps you since it looks like webpack is what throws a fit when trying to compile everything...
Just curious, can you change what webpack compiles?
Right now, it looks like you're using /src/main.js.
What happens if you use /prod/solver.js?

@DaanVandenBosch , does this solve your issue?

I've actually applied a patch to mitigate this:
--- "a/node_modules/javascript-lp-solver/src/main.js"
+++ "b/node_modules/javascript-lp-solver/src/main.js"
@@ -25,7 +25,7 @@ var Constraint = expressions.Constraint;
 var Variable = expressions.Variable;
 var Numeral = expressions.Numeral;
 var Term = expressions.Term;
-var External = require("./External/main.js");
+var External = {};

 // Place everything under the Solver Name Space
 var Solver = function () {


Patch release of the azure-communication-identity [1.4.4]

Description
Patch release of the azure-communication-identity [1.4.4]
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

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


Usdc earn query fix

Test software upgrade ✅
4:30AM ERR UPGRADE "v0.29.10" NEEDED at height: 5789274:  module=server
4:30AM ERR CONSENSUS FAILURE!!! err="UPGRADE \"v0.29.10\" NEEDED at height: 5789274: " module=consensus stack="goroutine 130 [running]:\nruntime/debug.Stack()\n\truntime/debug/stack.go:24 +0x64\ngithub.com/cometbft/cometbft/consensus.(*State).receiveRoutine.func2()\n\tgithub.com/cometbft/cometbft@v0.37.2/consensus/state.go:732 +0x44\npanic({0x10580ff40, 0x1400493fa80})\n\truntime/panic.go:884 +0x204\ngithub.com/cosmos/cosmos-sdk/x/upgrade.BeginBlocker(_, {{0x105cf6630, 0x140002f2120}, {0x105d0c920, 0x140095de400}, {{0xb, 0x0}, {0x14003de2eb0, 0xd}, 0x58565a, ...}, ...}, ...)\n\tgithub.com/cosmos/cosmos-sdk@v0.47.4/x/upgrade/abci.go:78 +0xc28\ngithub.com/cosmos/cosmos-sdk/x/upgrade.AppModule.BeginBlock(...)\n\tgithub.com/cosmos/cosmos-sdk@v0.47.4/x/upgrade/module.go:161\ngithub.com/cosmos/cosmos-sdk/types/module.(*Manager).BeginBlock(_, {{0x105cf6630, 0x140002f2120}, {0x105d0c920, 0x140095de400}, {{0xb, 0x0}, {0x14003de2eb0, 0xd}, 0x58565a, ...}, ...}, ...)\n\tgithub.com/cosmos/cosmos-sdk@v0.47.4/types/module/module.go:562 +0x170\ngithub.com/elys-network/elys/app.(*ElysApp).BeginBlocker(...)\n\tgithub.com/elys-network/elys/app/app.go:1342\ngithub.com/cosmos/cosmos-sdk/baseapp.(*BaseApp).BeginBlock(_, {{0x1400220ec80, 0x20, 0x20}, {{0xb, 0x0}, {0x14003de2eb0, 0xd}, 0x58565a, {0x1bfca470, ...}, ...}, ...})\n\tgithub.com/cosmos/cosmos-sdk@v0.47.4/baseapp/abci.go:196 +0x674\ngithub.com/cometbft/cometbft/abci/client.(*localClient).BeginBlockSync(_, {{0x1400220ec80, 0x20, 0x20}, {{0xb, 0x0}, {0x14003de2eb0, 0xd}, 0x58565a, {0x1bfca470, ...}, ...}, ...})\n\tgithub.com/cometbft/cometbft@v0.37.2/abci/client/local_client.go:283 +0x108\ngithub.com/cometbft/cometbft/proxy.(*appConnConsensus).BeginBlockSync(_, {{0x1400220ec80, 0x20, 0x20}, {{0xb, 0x0}, {0x14003de2eb0, 0xd}, 0x58565a, {0x1bfca470, ...}, ...}, ...})\n\tgithub.com/cometbft/cometbft@v0.37.2/proxy/app_conn.go:101 +0xf4\ngithub.com/cometbft/cometbft/state.execBlockOnProxyApp({0x105cf7ba0?, 0x14005d330f0}, {0x105d08f40, 0x140049587f8}, 0x1400415a000, {0x105d0d4c0, 0x140049580c0}, 0x585659?)\n\tgithub.com/cometbft/cometbft@v0.37.2/state/execution.go:363 +0x3e8\ngithub.com/cometbft/cometbft/state.(*BlockExecutor).ApplyBlock(_, {{{0xb, 0x0}, {0x14005d25daa, 0x6}}, {0x14005d25dc0, 0xd}, 0x585652, 0x585659, {{0x14000e8bf80, ...}, ...}, ...}, ...)\n\tgithub.com/cometbft/cometbft@v0.37.2/state/execution.go:197 +0xe4\ngithub.com/cometbft/cometbft/consensus.(*State).finalizeCommit(0x14000229880, 0x58565a)\n\tgithub.com/cometbft/cometbft@v0.37.2/consensus/state.go:1700 +0x964\ngithub.com/cometbft/cometbft/consensus.(*State).tryFinalizeCommit(0x14000229880, 0x58565a)\n\tgithub.com/cometbft/cometbft@v0.37.2/consensus/state.go:1609 +0x28c\ngithub.com/cometbft/cometbft/consensus.(*State).enterCommit.func1()\n\tgithub.com/cometbft/cometbft@v0.37.2/consensus/state.go:1544 +0xa0\ngithub.com/cometbft/cometbft/consensus.(*State).enterCommit(0x14000229880, 0x58565a, 0x0)\n\tgithub.com/cometbft/cometbft@v0.37.2/consensus/state.go:1582 +0xb90\ngithub.com/cometbft/cometbft/consensus.(*State).addVote(0x14000229880, 0x14002f6e780, {0x0, 0x0})\n\tgithub.com/cometbft/cometbft@v0.37.2/consensus/state.go:2212 +0x1790\ngithub.com/cometbft/cometbft/consensus.(*State).tryAddVote(0x14000229880, 0x14002f6e780, {0x0?, 0x1022fe240?})\n\tgithub.com/cometbft/cometbft@v0.37.2/consensus/state.go:2001 +0x28\ngithub.com/cometbft/cometbft/consensus.(*State).handleMsg(0x14000229880, {{0x105cc9860, 0x1400063e230}, {0x0, 0x0}})\n\tgithub.com/cometbft/cometbft@v0.37.2/consensus/state.go:861 +0x37c\ngithub.com/cometbft/cometbft/consensus.(*State).receiveRoutine(0x14000229880, 0x0)\n\tgithub.com/cometbft/cometbft@v0.37.2/consensus/state.go:788 +0x350\ncreated by github.com/cometbft/cometbft/consensus.(*State).OnStart\n\tgithub.com/cometbft/cometbft@v0.37.2/consensus/state.go:379 +0xf4\n"
4:30AM INF service stop impl={"Logger":{"Logger":{}}} module=consensus msg={} wal=/tmp/elys/data/cs.wal/wal
4:30AM INF service stop impl={"Dir":"/tmp/elys/data/cs.wal","Head":{"ID":"PBUECDeZtsyJ:/tmp/elys/data/cs.wal/wal","Path":"/tmp/elys/data/cs.wal/wal"},"ID":"group:PBUECDeZtsyJ:/tmp/elys/data/cs.wal/wal","Logger":{"Logger":{}}} module=consensus msg={} wal=/tmp/elys/data/cs.wal/wal
2024/02/14 04:31:02 Block height 5789274 reached
4:31AM INF Ensure peers module=pex numDialing=0 numInPeers=0 numOutPeers=0 numToDial=10
4:31AM INF No addresses to dial. Falling back to seeds module=pex
4:31AM INF Timed out dur=3000 height=5789274 module=consensus round=0 step=3
2024/02/14 04:31:07 Process killed successfully
2024/02/14 04:31:07 Process was killed: signal: killed
2024/02/14 04:31:12 Waiting for node to start...
4:31AM INF Setting up upgrade handler for auth module=server
4:31AM INF Setting up upgrade handler for slashing module=server
4:31AM INF Setting up upgrade handler for assetprofile module=server
4:31AM INF Setting up upgrade handler for oracle module=server
4:31AM INF Setting up upgrade handler for gov module=server
4:31AM INF Setting up upgrade handler for commitment module=server
4:31AM INF Setting up upgrade handler for burner module=server
4:31AM INF Setting up upgrade handler for stablestake module=server
4:31AM INF Setting up upgrade handler for transferhook module=server
4:31AM INF Setting up upgrade handler for leveragelp module=server
4:31AM INF Setting up upgrade handler for transfer module=server
4:31AM INF Setting up upgrade handler for ibc module=server
4:31AM INF Setting up upgrade handler for icacontroller module=server
4:31AM INF Setting up upgrade handler for icahost module=server
4:31AM INF Setting up upgrade handler for amm module=server
4:31AM INF Setting up upgrade handler for perpetual module=server
4:31AM INF Setting up upgrade handler for clock module=server
4:31AM INF Setting up upgrade handler for bank module=server
4:31AM INF Setting up upgrade handler for staking module=server
4:31AM INF Setting up upgrade handler for mint module=server
4:31AM INF Setting up upgrade handler for crisis module=server
4:31AM INF Setting up upgrade handler for tokenomics module=server
4:31AM INF starting node with ABCI Tendermint in-process module=server
4:31AM INF service start impl=multiAppConn module=proxy msg={}
4:31AM INF service start connection=query impl=localClient module=abci-client msg={}
4:31AM INF service start connection=snapshot impl=localClient module=abci-client msg={}
4:31AM INF service start connection=mempool impl=localClient module=abci-client msg={}
4:31AM INF service start connection=consensus impl=localClient module=abci-client msg={}
4:31AM INF service start impl=EventBus module=events msg={}
4:31AM INF service start impl=PubSub module=pubsub msg={}
4:31AM INF service start impl=IndexerService module=txindex msg={}
4:31AM INF ABCI Handshake App Info hash="���$��ЙB�\x05�V�(4�ߡ\x1e�?e\x1e.���oq%n" height=5789273 module=consensus protocol-version=0 software-version=v0.29.10
4:31AM INF ABCI Replay Blocks appHeight=5789273 module=consensus stateHeight=5789273 storeHeight=5789274
4:31AM INF Replay last block using real app module=consensus
4:31AM INF applying upgrade "v0.29.10" at height: 5789274 module=server
4:31AM INF Running upgrade handler for v0.29.10 module=server
4:31AM INF migrating module stablestake from version 1 to version 2 module=server


Output HTML

Add support for outputting as HTML, both from CLI and from lib.

Interestingly enough, there already seems to be some HTML involved in the testing.
But yes, having the ability to generate proper, styled HTML would be great!

This is also necessary for supporting EPUB.
Generating an EPUB from LaTeX is pretty much impossible.
Hopefully a modern replacement for LaTeX will treat EPUBs as first-class citizen :D
Actually, I would go one step further, and say that it should support plain text output (which is also almost impossible in LaTeX).

+1 for outputting HTML. This will make integration into popular note taking apps like

Obsidian, which uses MathJax for partial LaTeX capability in HTML
Notion, which uses KaTeX for even weaker LaTeX capability in HTML

much easier, and could increase Typst's popularity.

Great looking project!  Am working on a combined book and website with math, and managing two environments is a huge waste of effort.  Being able to have the same project for both, with shared libraries and glossary, would make it worth redoing from Latex/MathJax!  Would want to replace Markdown in Jeckyll with it.

Maybe perhaps additional integration with GoHugo, GetZola, etc. can be added to be used instead of Markdown and AsciiDoc in static site generators. I posted this in initially in: https://github.com/typst/typst/issues/114

Not sure if you meant to link this issue @sirinath 😅
But I agree, integrations with all those would be great even from a performance standpoint. No clue how that could work but would be willing to help if an effort is made towards it.

@AntoniosBarotsis Because https://github.com/typst/typst/issues/114 is a partial duplicate I added other part.

The home page (typst.app) mentions HTML Export is coming soon, and they also mentioned it in their paper on Typst earlier this year.

Hi @pranabekka wondering if you could point me to their paper earlier this year? I'd be very interested to read but I was not able to find it online...?

Hi @Leedehai , you can find the pdf at https://www.user.tu-berlin.de/laurmaedje/programmable-markup-language-for-typesetting.pdf
I think it's a little outdated, and you can find the same information in the online docs.

Pandoc already has support for Typst markup, so everyone can use it to turn their Typst source files into HTML, EPUB, LaTeX, and a zillion other markups. You can even tell Pandoc how to handle equations in HTML, e.g. MathJax or KaTeX or MathML or whatever.
https://pandoc.org/
Please don't get distracted by supporting conversion to other markup languages. Trying to replace LaTeX (and the TeX ecosystem) is ambitous enough, never mind competing with Pandoc as well.

Pandoc has a Typst writer, not a reader.

everyone can use it to turn their Typst source files into HTML, EPUB, LaTeX, and a zillion other markups

Turning Typst *source* files to other formats is of very little use.
You would get HTML-formatted Typst source (containing all the #show, #text and so on), but then you would still need to interpret that Typst source to generate its output.
It is exactly the same thing with LaTeX.
Pandoc copies verbatim the LaTeX source to HTML, without interpreting it.
Or, better, Pandoc does implement a tiny subset of LaTeX, so that it can interpret trivial commands such as \emph{}, \subsection{}, or \cite{}, but everything else is unsupported (\citeauthor{}, \gls{}, \SI{}{}, \begin{tikzpicture}...).

Trying to replace LaTeX (and the TeX ecosystem) is ambitous enough

LaTeX does not support output formats other than PDF and DVI (if we exclude tex4ht, which does not really generate anything useable in my experience).
Since one of its (and Typst's) use cases is writing books, I would consider supporting one of the modern book formats (EPUB, which is just zipped HTML) a very big YES toward Typst.

Add support for outputting as HTML, both from CLI and from lib.

I've made a package which generates PDF and HTML and can tell you that this request is some kind of Pandora's box. The main problem is that PDF and HTML do not support the same functionality, so then you're left with three options:

Support all of PDF's possibilities and ignore things which don't work in HTML by suppressing those things there or adding a lot of workarounds via Javascript and CSS.
Support all of HTML's possibilities and ignore things which don't work in PDF by suppressing those things there.
Limit the possibilities to a subset of PDF and HTML which work on both.

All three options are feasible but will also require significant engineering resources. The first one seems the most feasible, but remember that it also requires handling lots of things like browser differences. The problem is that typst by picking this up would need to support everything they support for PDF. A more likely successful approach is to create stand-alone projects which limit themselves to a subset of typst and be very clear about that or projects like Bookdown which start with a limited set of capabilities in Markdown and then produce HTML and PDF from that.

So, in summary: If you want HTML, PDF, ePub, and Kindle books, then you can already write in Markdown and use Bookdown or Pandoc to generate all those different output formats.

Relevant discussion: #721.

@rikhuijzer Your statement "you can already ..." is simply not true yet because it misses the point. I have setup Markdown based workflows for several publishing houses and written typsetter software and Pandoc writers. I'm familiar with the moving parts here.
Others have explained in some, but your statement would not be true until such a time as Pandoc has a reader as well as writer and can has extensions to allow raw typst code embedded in Markdown or other ways of handling the transitions. Additionally it needs a read/write conversion for the math dialect.
Even with all that it still doesn't cover all the bases that HTML output from the compiler side (rather than just an input format reader) would handle, as others have explained.

Closing this in favor https://github.com/typst/typst/issues/721


Alternative geojson reader/writer from jts-io

Expected behavior
Alternative geojson reader/writer from jts-io ,  due to jts2geojson  is not a famous lib.

AFAIK, jts2geojson is the only library that handles GeoJSON Feature or FeatureCollection. JTS-IO only handles the geometry field of a geojson. Please correct me if I am wrong.


Fix exception from DOB month being a whitespace

Context
Entering a space instead of a valid month on the 'date of birth' form raises a Date::Error exception.
Sentry
Changes proposed in this pull request
Strip whitespace on the DOB month.
Guidance to review
...
Checklist

[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

Thanks @dankmitchell I'll let the day shift devs review before I merge so they have visibility 😂

Wise decision 😅

:pyjama:


Feature/mifareclassic readme

Changes

Added documentation  in README.md
Renamed getMifareClassicMessage to mifareClassicReadBlock to have a bit more consistency
Fixed a bug in mifareClassicAuthenticate (it was always authenticating to key A)
Some fixes on the example app
Added ByteParser.js in package.json

Hi @whitedogg13 ,
This is the last PR to make the Mifare Classic go to bèta. I added some documentation, and while I was doing so I noticed that it's better to rename the getMifareClassicMessage to mifareClassicReadBlock.
Please let me know your thoughts, I'm looking forward to release 7.0 if you're okay with it so we can announce we support Mifare Classic.
Best,
Nicolas

I think the rename is correct, and this feature is definitely worth a new version. Will release it soon!

New version 0.7.0 just published! Thanks you @poison !


error creating error stream during rad resource expose

Steps to reproduce
You need to check out samples repository to be able to reproduce this issue.

Run rad deploy ./samples/demo/app.bicep --application demo -p image=ghcr.io/radius-project/samples/demo:latest
Run rad resource expose containers demo -a demo --port 3000
Run Playwright tests for the demo application: npx playwright test tests/demo/demo.app.spec.ts --retries 3
See the attached errors.

Observed behavior

Desired behavior
We shouldn't see errors like error creating error stream and/or broken pipe
Workaround
None yet
rad Version
0.31
Operating system
macOS
Additional context
This is being discussed in the Kubernetes repo:

https://github.com/kubernetes/kubernetes/issues/107203
https://github.com/kubernetes/kubernetes/issues/74551

Would you like to support us?

[ ] Yes, I would like to support you

AB#11663

This is also affecting our samples repo tests.

@ytimocin Can you please elaborate why this is the bug? To me, it is an expected behavior. Do you see the same issue with the other container resource? I wonder if the issue is related to test sample and your test environment.

This is what I am running into in my local environment. I have also seen this in a few failed samples test runs. What happens basically is that when port-forwarding starts, I am able to reach localhost:3000 (assuming this is the port forwarded) but during the playwright test (running the command [playwright test demo.js] in my local) localhost becomes unreachable. I will be debugging further but this might be a reason why the samples tests sometimes fail @youngbupark.

This is what I am running into in my local environment. I have also seen this in a few failed samples test runs. What happens basically is that when port-forwarding starts, I am able to reach localhost:3000 (assuming this is the port forwarded) but during the playwright test (running the command [playwright test demo.js] in my local) localhost becomes unreachable. I will be debugging further but this might be a reason why the samples tests sometimes fail @youngbupark.

Did you see the status of pod which cli enables port-forward? it happens when the pod is unstable or crash and then cli retry to establish the connection. Please gather more info such as pods status and pod restart time when this issue happens. We need more info and data to decide whether this is bug or not.

@ytimocin Just checking have you been able to find a repeatable underlying cause?

Not yet. I am still investigating.

I couldn't reproduce this issue. Closing this for now.


[Storage] Add read to datalake StorageStreamDownloader

As a follow-up to #24275, this adds the read method to the StorageStreamDownloader for datalake. This was fairly trivial since datalake's downloader just calls into blob. Added one basic test to ensure functionality but more advanced testing is handled in blob tests.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-storage-file-datalake


Colorbar "Apply to All" is a no-op

The colorbar "Apply to all" button is a no-op
To reproduce:

load any data set and enable two 2D renderers
Enable color bars in both renderers. Change the background color in one of them and click "apply to all". Nothing happens

Duplicates #406


[Style] BinExploit | filtering a keyword in BinExploit

Describe the bug
the bug appears to be in the keyword filtering functionality of the web application. When you use a keyword to filter the results, the application is expected to highlight or color the keyword in a specific section(top section) of the page . However, the bug is causing the keyword to be highlighted multiple times across the entire page, not just in the intended section.
This could lead to confusion as it misrepresents the locations where the keywords are actually .
To Reproduce
go to 'Resources'
go to ' Tools'
click on 'BinExploit'
click on a keyword from the list provided at the top of the page
Expected Behavior
in my opinion the filtering section and the result section should've been separated maybe in the result section you can just align the keyword and in the filtering section it should remain the same this will help the users and it will improve user Experience
Screenshot/ Video

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@ZizouX0 it is not a bug it was intended to be in this way.
@kabir0x23 what do you think about changing the way it shows the active button in the main section?

yes good idea,
yes it was intended, but it is not good to do highlights in data i think

@kabir0x23 what do you mean? he/she means that only on top section the button should be orange and not in main section as it is now. If I understand it is right.
@ZizouX0 did you mean italic instead of align? or could you explain your solution? in addition, would you like to handle it?

@kabir0x23 what do you mean? he/she means that only on top section the button should be orange and not in main section as it is now. If I understand it is right. @ZizouX0 did you mean italic instead of align? or could you explain your solution? in addition, would you like to handle it?

yes top section


fix direction parameter bend_straight_bend

@thomaslima fixes #2389
    c = bend_straight_bend(direction="ccw")
    c.show(show_ports=True)

    c = bend_straight_bend(direction="cw")
    c.show(show_ports=True)

there is no test because it is not a component that is self-contained; we could force it to be tested

Very nice. Would you mind explaining the bug? I don't think I understand it...


DocumentDB support

Amazon DocumentDB is a document database service compatible with MongoDB.
https://docs.aws.amazon.com/documentdb/latest/developerguide/mongo-apis.html
But DocumentDB doesn't support change streams and has a limitation of 32 characters maximum in a compound index.
This pull request allows DocumentDB support in Tock with the environment property tock_document_db_on (false by default).

Push another commit to serialize Duration in String and not in Decimal128 (not supported by DocumentDB).
https://docs.aws.amazon.com/documentdb/latest/developerguide/mongo-apis-data-types.html

Thanks for the PR. Before I can merge it, we need to see together how to write an alternative way to listen db changes - things would be broken if we don't listen changes.
Does DocumentDB support Oplog? https://docs.mongodb.com/manual/core/replica-set-oplog/

AWS DocumentDB now supports change streams.
https://docs.aws.amazon.com/fr_fr/documentdb/latest/developerguide/change-streams.html
I will check that and update my PR.

DocumentDB is now fully compatible with change streams.
You have to manually enable them on your database with an admin command, like describe here.
The PR has been updated accordingly.

merged. Thank you!

Hello @FabienPennec
Can you please give more informations about how you use document DB?
I want to try it but i can see that documentDB uses ssl with a certificate. How did you add cert to differents tock containers that need a mongo database access?
Regards
Mat


OFFBOARD control(setpoint local position) not working

I just updated my firmware of my pixhawk to the newest firmware through qGroundcontrol.
Afterwards, the offboard control with setpoint local position stopped working.
I checked with my old quad which had the old firmware, and the exact same code worked fine on it.
Could you guys check if something went wrong with you updates?
Thanks guys.

Are you on master?
What's your connection to the pixhawk like on both quads for offboard control?
Did you update anything else? Mavros just got some toppic name updates that "broke" the code I was running before updates.

I had a similar issue with the latest master. I had data from my motion capture system streaming to the pixhawk, through ROS I was able to switch the system to off board control on a relatively recent master. On the very latest master the flight controller doesn't seem to be using the motion capture data into its position estimate. I have not tried off board control since, as the position data wasn't working. I thought it might be my ROS install was messed up from a series of modifications and was working on reinstalling everything. Thought I would mention it when I saw this issue.
I never actually flew the multi rotor in off board mode in the previous master, as the multi rotor was very unstable until a recent update.

Yes, I was on master(about a week ago).
I have connected my pixhawk and my offboard with USB to UART module which is plugged in to the telem2 port of pixhawk. Rate set to 921600.
Checking the log, Pixhawk seems to receive the setpoint local position coordinates correctly
(LPSP_X, LPSP_Y, LPSP_Z were all received).
Thanks.

I checked by installing the previously released firmwares and the offboard control  for setpoint localposition stops working after the second release of 1.0.0 beta.
First release of 1.0.0 beta works fine.

nevermind.
updating mavros solved the problem!


Need to scroll until a element is displayed

Hi Team,
I need to scroll until a element is displayed . Can you help me here .

WAD supports auto scrolling till 40th element. How long is your list?

@LisaAga you could also try some more creative approaches, like using the page down key press

Which UI framework are you targeting?
WinAppDriver will automatically scroll items into view when clicked and reporting IsOffscreen is true.
However, many desktop UI frameworks report that property wrongly or at least unintuitively.
Check that property for items that you want to scroll into view and you should have your answer.
Further, again depending on your UI framework, you can "fix" that behavior which should make the click-scroll work.


Lrz update

This is a PR that will merge when all fixes to LRZ update are complete and Cargo.toml is changed from main_zingolib branch to a final tag in zingolabs LRZ fork.
Please PR into this PR with each fix. full list of errors in cryptpad under "lrz update scrapbook"
UPDATE: this is ready to merge but blocked on backward compatibility tests for a change in USK derivation

Is this still not ready to merge into dev?

@Oscar-Pepper is this ready to merge into dev?

no this is blocked on @fluidvanadium's backward-compatibility tests. https://github.com/zingolabs/zingolib/pull/1413 I believe.

@fluidvanadium can you start by resolving conflicts?   @Oscar-Pepper seemed to think this was ready to land, but I blocked on the conflict.


fix(twitter): make twitter watcher to use rettiwt-api package instead of official one

:tada: This PR is included in version 1.0.0-dev.18 :tada:
The release is available on:

npm package (@dev dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Should we update MSRV to 1.42.0

With a few tweaks* we can support Rust version 1.42.0 (September 2020).
But should we?
Ensuring support for Rust released 2 years ago seems sensible for a library primarily used in dev environments.

*Changes:

println! macros in docs
println! in 1 example

The examples certainly need updating anyway.

Sure! Let's kick out the format strings.

While at it, I'd rather push MSRV more aggressively, maybe to 1.56.1 at least so we can specify the rust version in Cargo.toml and maybe look through the codebase and see if we can add any other features that were introduced

Yes, maybe 2 years is a bit too conservative. 1.56.1 is 1 year ago (n-8). After going down the rabbit hole(1 2) of other crate's policies n-8 seems like a good compromise.


CLOUDP-62719: Start publishing to the testing repo our rc versions

Proposed changes
Jira ticket: CLOUDP-62719
Checklist

[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added any necessary documentation (if appropriate)
[x] I have run make fmt and formatted my code

@tychoish this is our prep work to release to the stable repos, once we are ready we'll change the version to 4.4, let me know if you have any comments here

@tychoish friendly ping to give this a look, I'd really appreciate any insight on this

@tychoish I'm going to merge this as I'd like to qa the rc build, anyhow feel free to leave comments here and I can address them later


ImprovementSupport ODBC table for new planner.

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Add gitlab statistics similar to github

If crate's git repo is hosted on github docs.rs shows projects stats fetched from github.com. This ticket is for having similar arrangement for gitlab.com as well

The same would be nice for BitBucket as well (if it doesn't already exist), seeing as it another large hosting provider.

Triage: This is fairly low priority for us to do but if a PR were made we would accept it

I started to work on gitlab integration.

I started to work on gitlab integration.

Implemented in https://github.com/rust-lang/docs.rs/pull/1249.


[Consensus] Pacemaker implementation based on external TC aggregation

Summary:
Note: this is an intermediate step towards further refactoring.
As we're moving towards the solution, in which the TC is aggregated and managed by the block store, we can significantly simplify the Pacemaker.
This change introduces a new implementation that is not handling timeout certificates, instead it just takes the round being max{tc,qc} + 1.
The only real piece of logic in the pacemaker is the interval calculation and the generation of the local timeouts.
In order to have reasonably sized changes this diff introduces a separate "pacemaker_new" module with the new implementation, which is not used yet.
We're going to start using it in the followup change and will cleanup the old Pacemaker after that.
Testing:
added unit tests for the new implementation
Issues
ref #1048

:v: @dmitri-perelman can now approve this pull request

@bors-libra r=andll

:pushpin: Commit 39dadc5 has been approved by andll

:hourglass: Testing commit 39dadc5b424b2a9685ab195d084f0fadcd2defa6 with merge 0059fd062a6743083a770c358decdab2cb7ad336...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: andll
Pushing 0059fd062a6743083a770c358decdab2cb7ad336 to master...


Update dependencies

feature/update-dependencies

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.32%. Comparing base (dc094d0) to head (6167652).
Report is 11 commits behind head on 3.0-preview.

Additional details and impacted files
@@               Coverage Diff               @@
##           3.0-preview      #31      +/-   ##
===============================================
- Coverage        84.26%   82.32%   -1.94%
===============================================
  Files               35       65      +30
  Lines             2936     7311    +4375
===============================================
+ Hits              2474     6019    +3545
- Misses             462     1292     +830

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


File and filecontent should take filename regexes as inputs to scope search

Why would file need this? doesn't File just search for filenames?

Adding things here to make them look like structs like we talked about.

fixed by https://github.com/konveyor/analyzer-lsp/pull/185


AttributeError: 'AdaBeliefOptimizer' object has no attribute '_set_hyper'

I'm facing the problem with AdaBeliefOptimizer
AttributeError: 'AdaBeliefOptimizer' object has no attribute '_set_hyper'
optimizer = AdaBeliefOptimizer(learning_rate=1e-3, epsilon=1e-14, rectify=False)

Are you using pytorch or tensorflow version? I don't think I have written any function named _set_hyper

Tensorflow version with 2.15.0

Thanks i already used optimizer = optim.AdaBelief(model.parameters(), lr=1e-3) and worked but the accuracy doesn't greater than using Adam optimizer. My task is for classification


continue to get arecord: device_list:272: no soundcards found...

used a fresh image of Google AIY + raspberry buster from HACKING.md. However, after sudo apt-get upgrade, continue to get :
arecord: device_list:272: no soundcards found...
I have a feeling something wrong with compatibility with the Kernel version raspberrypi 5.4.51-v7+ , which is what I have

This is a known issue and a bug has been filed with the development team for this.

@manoj7410 .. thank you. Do you have the bug details? It's disappointing that Google is selling the AIY kit that doesn't perform it's basic sound function after updates?

I emailed their support team and they said that there's no defined ETA for a fix which is disappointing. Seems like this has been an issue since May

@thechamp21 interesting - I was told that a fix would probably be released by the end of this month. Hopefully this is the case!

@nmilenko I hope that is the case! Eager to get this thing built!

@manoj7410 - do you happen to know when the development team will be able to address this?

@thechamp21 @bravozulu1979 I had the same issue and was able to create a fix for the same.
To fix it, uninstall your aiy-voicebonnet-soundcard-dkms package and install the new package from here

@thechamp21 Since Google tries to keep one build for all the variants of AIY kits (Voice, Vision). The release will only happen once the build has been tested for all the platforms. I don't have any timeline available as of now.

@manoj7410 Google dooesn't need to release a new image to fix the issue, do they? All they need to do is update the aiy-voicebonnet-soundcard-dkms package in their debian repository so that people can do update their packages.
That doesn't need the whole image to be built and tested does it? After all, the driver will only load when the voicebonnet is attached to pi and people cannot really attach multiple aiy boards together to the pi, can they?

We have updated aiy-voicebonnet-soundcard-dkms packaged and also released new SD card image, please try them.


Collapse filter panel when view gets too narrow

Fixes:
At low resolution or when the nav is open and a quick start is open, the resources view gets too narrow to show much of the information for each resource.
Analysis / Root cause:
The addition of the filter panel decreases the width used by the view to show the resources.
Solution Description:
When the view width becomes narrow (under 600px) turn the filter panel into a slide out drawer controlled by a filter button in the header panel.
This PR also adds "Favorites" category.
Screen shots / Gifs for design review:

Browser conformance:

[x] Chrome
[x] Firefox
[x] Safari
[x] Edge

/cc @kywalker-rh

Can you tighten up the spacing of the Type to match the attached screenshot from OperatorHub?
Also putting everything left-aligned (the Types, titles, and checkboxes) makes it cleaner and easier to read.

Thanks!

@kywalker-rh Updated:


emails: Fix style regression for dynamically generated content.

Related CZO discussion

Ah I see. Those are still overbroad selectors—ideally, selectors should always target specific components and not their arbitrary descendants. But maybe we’re okay with that for now as a stopgap given the priority of the regression. I’ll let Tim decide.

I've applied the following changes to rename a few things and document this much more clearly.
diff --git a/templates/zerver/emails/email.css b/templates/zerver/emails/email.css
index 0e393657bd..4ae0fac4c0 100644
--- a/templates/zerver/emails/email.css
+++ b/templates/zerver/emails/email.css
@@ -1,10 +1,11 @@
 /* stylelint-disable color-no-hex, media-feature-range-notation */
-/* hex color codes are used instead of our standard of hsl colors,
-because hex color codes are rendered by all email clients
-consistently.  Whereas HSL colors don't work with Hotmail. */
+
+/* Hex color codes are used instead of our standard of hsl colors,
+   because hex color codes are rendered by all email clients
+   consistently; HSL colors aren't supported by Hotmail. */

 .illustration img,
-td.wrapper img {
+img.bottom-illustration {
     border: none;
     -ms-interpolation-mode: bicubic;
     max-width: 100%;
@@ -21,6 +22,9 @@ body {
     -webkit-text-size-adjust: 100%;
 }

+/* These .layout tables are used to create sections within the email;
+   see for example: https://templates.mailchimp.com/development/html/
+*/
 table.layout {
     border-collapse: separate;
     mso-table-lspace: 0;
@@ -34,6 +38,8 @@ table.layout > tbody > tr > td {
     vertical-align: top;
 }

+/* After email clients strip the actual  tag out of the email, the
+   .body element will be the equivalent of our outer element. */
 .body {
     width: 100%;
 }
@@ -45,10 +51,14 @@ table.layout > tbody > tr > td {
     padding: 10px;
 }

+/* Our newsletters/marketing emails use a wider content width, to
+   support more complex content with text to the side of images. */
 .container.wide-container {
     max-width: 700px;
 }

+/* Mystery: .container contains .content, but the max-widths for this
+   element are 80px wider than those for .container. */
 .content {
     box-sizing: border-box;
     display: block;
@@ -61,17 +71,22 @@ table.layout > tbody > tr > td {
     max-width: 780px;
 }

+/*
+    The main content region for the email; excludes the preheader and footer.
+*/
 .main {
     background-color: #fff;
     border-radius: 3px;
     width: 100%;
 }

+/* Right/left padding to keep content away from edges of email client winder. */
 .wrapper {
     box-sizing: border-box;
     padding: 0 20px 20px;
 }

+/* Footer section, containing unsubscribe links and the like. */
 .footer {
     clear: both;
     padding-top: 10px;
@@ -80,8 +95,12 @@ table.layout > tbody > tr > td {
     margin-top: 20px;
 }

-.content-block,
-.apple-link {
+/*
+  Apple mail and other popular email clients will convert the physical
+  address into a link, so we set link color on it.
+*/
+.footer-content-block,
+.physical-address {
     color: #5f5ec7;
     font-size: 12px;
     text-align: center;
@@ -110,12 +129,14 @@ p.digest_paragraph,
     margin-left: 15px;
 }

+/* Link color styling carefully tuned to not apply to user content
+   inside missed-message emails. */
 .body a,
 .digest_paragraph a,
 .sponsorship_request_link,
 .support_request_link,
 .email-preferences a,
-.missed_message.no_content a {
+.missed_message.no-content-explanation a {
     color: #5f5ec7;
     text-decoration: underline;
 }
@@ -156,6 +177,9 @@ a.button:hover {
     text-decoration: none;
 }

+/* The preheader is an invisible region that is used to control what
+   email clients display as the textual preview of the email's content
+   in inbox views. */
 .preheader {
     color: transparent;
     display: none;
@@ -169,12 +193,16 @@ a.button:hover {
     width: 0;
 }

+/* Illustrations appear over the top of the boundary between the
+   blank space at the top of the email and the content box; for
+   that reason they need a high z-index. */
 .illustration {
     pointer-events: none;
     cursor: default !important;
     z-index: 100;
 }

+/* CSS for digest email content. */
 #private-messages {
     width: 600px;
     font-size: 12px;
@@ -282,6 +310,7 @@ a.button:hover {
     font-weight: normal;
 }

+/* Text and link colors for missed-message emails. */
 .email-preferences {
     color: #666;
 }
@@ -303,6 +332,8 @@ a.button:hover {
     color: #15c;
 }

+/* Quoted block for text written by an organization administrator to
+   explain to the end user why their account got deactivated. */
 .deactivated-user-text {
     padding: 0 0 0 15px;
     margin: 0 0 20px;
@@ -366,6 +397,8 @@ a.button:hover {
 }

 @media all {
+    /* See https://templates.mailchimp.com/development/css/client-specific-styles/;
+       ExternalClass is something Microsoft Outlook adds to emails. */
     .ExternalClass {
         width: 100%;
     }
@@ -379,8 +412,10 @@ a.button:hover {
         line-height: 100%;
     }

-    /* iOS converts addresses in emails to links automatically */
-    .apple-link a {
+    /* iOS converts physical addresses in emails to links automatically,
+       so this CSS has an effect on iOS even though we don't actually
+       have a link inside the .physical-address element. */
+    .physical-address a {
         color: inherit !important;
         font-family: inherit !important;
         font-size: inherit !important;
@@ -390,6 +425,7 @@ a.button:hover {
     }
 }

+/* Mechanism for images alongside content, mainly for use in newsletters. */
 .float-right-image {
     float: right;
 }
diff --git a/templates/zerver/emails/email_base_default.html b/templates/zerver/emails/email_base_default.html
index 181dc50913..1e874609a1 100644
--- a/templates/zerver/emails/email_base_default.html
+++ b/templates/zerver/emails/email_base_default.html
@@ -28,15 +28,15 @@

-
+

-
-                                        {{physical_address}}
+
+                                        {{physical_address}}
                                         {% block manage_preferences %}{% endblock %}

diff --git a/templates/zerver/emails/email_base_marketing.html b/templates/zerver/emails/email_base_marketing.html
index cbf3ed95b2..47105ac1ea 100644
--- a/templates/zerver/emails/email_base_marketing.html
+++ b/templates/zerver/emails/email_base_marketing.html
@@ -35,8 +35,8 @@

-
-                                        {{physical_address}}
+
+                                        {{physical_address}}
                                         {% block manage_preferences %}{% endblock %}

diff --git a/templates/zerver/emails/missed_message.html b/templates/zerver/emails/missed_message.html
index 346e2c8f0a..53dbfe45fb 100644
--- a/templates/zerver/emails/missed_message.html
+++ b/templates/zerver/emails/missed_message.html
@@ -12,7 +12,7 @@
             {% endfor %}
         {% endfor %}
     {% else %}
-

+

     {% if message_content_disabled_by_realm %}
     {% trans help_url=realm_uri + "/help/hide-message-content-in-emails" %}This email does not include message content because your organization has disabled message content appearing in email notifications.{% endtrans %}
     {% elif message_content_disabled_by_user %}

I added a commit removing the CSS rules that I could prove are no longer effective, as well as the diff above; with those changes the code looks good to me.
I also had to essentially revert 40f0663d9d578409e7beceacbae35cf9805a3abb.
I have not tested as much as I would like, but I'm going to get this onto chat.zulip.org so we can get more natural testing there.


Can't get the node data type for mask_1/Cast

Hi,
I get the following error during conversion:
    onnx_model = keras2onnx.convert_keras(model, model.name, target_opset=7)
  File "/home/cbo/anaconda3/envs/keras-tf-py36/lib/python3.6/site-packages/keras2onnx/main.py", line 100, in convert_keras
    parse_graph(topology, sess.graph, target_opset, output_names)
  File "/home/cbo/anaconda3/envs/keras-tf-py36/lib/python3.6/site-packages/keras2onnx/parser.py", line 649, in parse_graph
    return _parse_graph_scope(graph, keras_layer_ts_map, topo, top_level, output_names)
  File "/home/cbo/anaconda3/envs/keras-tf-py36/lib/python3.6/site-packages/keras2onnx/parser.py", line 599, in _parse_graph_scope
    _convert_general_scope(nodes, varset)
  File "/home/cbo/anaconda3/envs/keras-tf-py36/lib/python3.6/site-packages/keras2onnx/parser.py", line 301, in _convert_general_scope
    subgraph, replacement = create_subgraph(sess.graph, node_list, sess, operator.full_name)
  File "/home/cbo/anaconda3/envs/keras-tf-py36/lib/python3.6/site-packages/keras2onnx/subgraph.py", line 128, in create_subgraph
    raise RuntimeError("Can't get the node data type for %s" % input_node.name)
RuntimeError: Can't get the node data type for mask_1/Cast

commenting K.set_learning_phase(0) in code solved the problem. strange

I have added *1. at the end so that the error does not occur, strange

The layer output is now  K.cast(K.greater(inputs, 0), K.floatx()) *1.


menu color

how can ı change menu color

Cheat dont have that option.

But you can change it in the source in the imgui_draw file.

You can edit imgui style. More info on that is in imgui docs.


If only entities are changed, changes are not saved

We need to model a dependency between pub-meta and articles.

Fixed via #401.


Updates quote-string regex in json2scssMap.js

Thanks for porting this lib!
Adding ")" to the regex value check. If a value contains parentheses, quote strings are not needed.
Came across this snag when trying to use css-variables from the generated scss map. css-variables will not work if stringified, e.g.:
.el {
...
background-color: "var(--color-bg-light)";
...
}

I don't think checking for only ")" will confirm that  string is going to be CSS variable string. Also, why "" checking for ? Please elaborate...properly.


Analysis of linked synthetized afi data

For now the analysis is not in the repo, as I’ve added the files in the gitignore. Might release the analysis after getting the right greenlights. @olivierabz could you please merge?

@olivierabz this can be merged now


Add contribution guideliness and code of conduct (#21)

Since the project is not very mature, basic contribution guidelines were added. Moreover, as suggested, we adhere to Covenant CoC.

Codecov Report

Merging #27 (49d81f6) into main (5e266c7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #27   +/-   ##
=======================================
  Coverage   86.66%   86.66%
=======================================
  Files           2        2
  Lines         330      330
  Branches       64       64
=======================================
  Hits          286      286
  Misses         28       28
  Partials       16       16

Flag
Coverage Δ

unittests
86.66% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


⚠ RPC server - Binance BSCT has degraded performance

In 0de09e8, RPC server - Binance BSCT ($BINANCE_BSCT_RPC_URL) experienced degraded performance:

HTTP code: 504
Response time: 10645 ms

Resolved: RPC server - Binance BSCT performance has improved in 97364ea.


CLOUD-725 gcp checker task improved

@keyki

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1203/


Circle updates unevenly when resizing

When resizing the circle, some parts of it lag behind and redraw with around half-a-second lag. Tested on iPhone XS.

This is an old code, let me check what I can do here.

hi Siddharth I have seen many facing problems while implementing this feature(the circle wouldn't resize smoothly), may I know what have you done differently in this project to overcome that?


Testing components which use a fetcher does not work with createRemixStub

What version of Remix are you using?
2.1.0
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce
Create a simple remix app with a component using useFetcher.
Then write a test case using createRemixStub to test that component.
Expected Behavior
The component should render and eventually receive the loader data from the fetcher.
Actual Behavior
The initial state is rendered, the loader is called, but the loader data never reaches the fetcher.
The component is not re-rendered and the test fails.

Is this the same issue as https://github.com/remix-run/react-router/issues/8715#issuecomment-1979666438?