Add SEND example

Add SEND example files from the CBER pilot.

Lex, I think it will be better to have XPT/JSON files as releases (https://github.com/cdisc-org/DataExchange-DatasetJson/releases), rather then to keep them in the repo itself. Because this is data and as I understand we do not want to version control it. If we want to version control it, then it is logical to keep it in the repo.

Since we have SDTM and ADaM examples in the repo, I thought I would add SEND as well.

Where would you store this data then?

We can keep it in the release area. In this case when we update schema the next time, we can create a release and alongside with the source code put all of the datasets as zip archives. More on it is here: https://docs.github.com/en/repositories/releasing-projects-on-github/about-releases

I understand the release concept, but where would those example files be sourced from. Are you suggesting that the examples folder be part of .gitignore? Different developers may nog have the same example files then.
I would rather keep them in the remote repo.

Closing as SEND files will be added to dataset-JSON v1.1

Closing as SEND files will be added to dataset-JSON v1.1


Nokia lumia

it seems that this should be detected as mobile, but currently it shows desktop
http://useragent.mkf.solutions/?userAgent=Mozilla%2F5.0+(Windows+NT+10.0%3B+ARM%3B+Lumia+930)+AppleWebKit%2F537.36+(KHTML%2C+like+Gecko)+Chrome%2F46.0.2486.0+Safari%2F537.36+Edge%2F13.10586
https://github.com/serbanghita/Mobile-Detect/issues/516

WhichBrowser detection could be improved here, but this is a tricky user agent string.
This is a Lumia phone running in Continuum mode. Basically it is connected to an external monitor, keyboard and mouse and it runs the "desktop version" of Windows 10 with the desktop version of Edge. So I would actually say this is a desktop device.
But in any case, the model and manufacturer could be added.

Model and manufacturer are now properly detected. The operating system is now also detected as Windows Phone 10. But the device type is still desktop.

Fixed, will be included in the next update!


Unable to parse / missing tag value

Hello Jorge,
When I execute the script, I receive the following errors:
`HTTP/1.1 400 Bad Request
Content-Type: application/json
Request-Id: 08acbe87-7023-11e9-8325-000c294005bd
X-Influxdb-Build: OSS
X-Influxdb-Error: unable to parse 'veeam_vac_vcclicense,edition=,status=,type=,contactPerson=,backupServerName=,companyName= licenseExpirationDate=1557100800,supportExpirationDate=1557100800,licensedCloudConnectBackups=,usedCloudConnectBackups=,licensedCloudConnectReplicas=,usedCloudConnectReplicas=,licensedCloudConnectServers=,usedCloudConnectServers=,licensedCloudConnectWorkstations=,usedCloudConnectWorkstations=': missing tag value
X-Influxdb-Version: 1.7.6
X-Request-Id: 08acbe87-7023-11e9-8325-000c294005bd
Date: Mon, 06 May 2019 17:18:58 GMT
Content-Length: 433
{"error":"unable to parse 'veeam_vac_vcclicense,edition=,status=,type=,contactPerson=,backupServerName=,companyName= licenseExpirationDate=1557100800,supportExpirationDate=1557100800,licensedCloudConnectBackups=,usedCloudConnectBackups=,licensedCloudConnectReplicas=,usedCloudConnectReplicas=,licensedCloudConnectServers=,usedCloudConnectServers=,licensedCloudConnectWorkstations=,usedCloudConnectWorkstations=': missing tag value"}
HTTP/1.1 400 Bad Request
Content-Type: application/json
Request-Id: 08bc0cfa-7023-11e9-8326-000c294005bd
X-Influxdb-Build: OSS
X-Influxdb-Error: unable to parse 'veeam_vac_vaclicense,productName=,licensedTo=,licenseType=,licensesCount=,licensesUsedCount=,supportID= licenseExpirationDate=1557100800,supportExpirationDate=1557100800,licensevmCount=,licenseworkstationCount=,licenseserverCount=,licensecloudWorkstationCount=,licensecloudServerCount=,licensecloudconnectBackupVmCount=,cloudconnectBackupWorkstationCount=,cloudconnectBackupServerCount=,cloudconnectReplicationVmCount=': missing tag value
X-Influxdb-Version: 1.7.6
X-Request-Id: 08bc0cfa-7023-11e9-8326-000c294005bd
Date: Mon, 06 May 2019 17:18:58 GMT
Content-Length: 468
{"error":"unable to parse 'veeam_vac_vaclicense,productName=,licensedTo=,licenseType=,licensesCount=,licensesUsedCount=,supportID= licenseExpirationDate=1557100800,supportExpirationDate=1557100800,licensevmCount=,licenseworkstationCount=,licenseserverCount=,licensecloudWorkstationCount=,licensecloudServerCount=,licensecloudconnectBackupVmCount=,cloudconnectBackupWorkstationCount=,cloudconnectBackupServerCount=,cloudconnectReplicationVmCount=': missing tag value"}
`
Thank you
Paul

I found the issue, it was user error. Closing.

@pjcortes @jorgedlcruz
What was the issue and what did you do to resolve it. I'm stuck with the same output and
believe we using the latest VAC version.
Cheers

Hi @pjcortes @jorgedlcruz
Please help, I'm still not able to resolve.

Hi @pjcortes @jorgedlcruz
Please help, I'm still not able to resolve.

Hi @pjcortes @jorgedlcruz
Please help, I'm still not able to resolve.


Build with VS2017 on AppVeyor

Update appveyor.yml to use the latest Visual Studio image.

This will not work until #2404 is merged.


In China, how to get positioning  (No Google service)  for Android 6.0+ 。

From @zuohuadong on July 5, 2018 3:26
I try to use https://github.com/NativeScript/nativescript-geolocation  but not work (Google service).
Also, I can not find any Plug - ins, such as Baidu Map, a map.
Copied from original issue: NativeScript/NativeScript#6030

Closing as duplicate and related to https://github.com/NativeScript/nativescript-geolocation/issues/144#issuecomment-402992230


fix: 'in' type condition from Filter

From any doctype's Listview Filter if we use is "in/ Not in" throwing Error
Change in Condition builder for "in"
This error occurs in all doctype listview filter

Covers :

https://github.com/8848digital/erpnext/issues/393
https://github.com/8848digital/erpnext/issues/390


Replicating annotated papers

I made a 'normal' search, selected three results, and annotated one. Then, when I checked the selected papers list, I found 4 papers there in which the annotated one was replicated (and colored only once).

I was not able to replicate the issue.
Can you provide a specific example that make the issue occours?


Improve the structure of grease commands

Filing here as a v2 milestone item.

We also need to resolve the structure of grease configuration, greasi, and gear.

We also need to improve grease flag setting and comma notation.

We also need to support saving in greasi.

We also need to support code-specified and command-scoped defaults in grease.

This is very simple.

First pass that takes all the functions, creates the stuff for them.
Optional second pass where you can customize options for a given set of functions.
Final "Run" call or whatever to actually do it.


Use Twitter account, cancel Github news retweet

Again, part of our Q4 milestones.
@wfwalker @cgjones

Done.


SVGLocatable missing in lib.dom.d.ts

TypeScript Version:  2.2.1 / nightly (2.2.0-dev.201xxxxx)
Expected behavior:
type SVGLocatable should be available in some d.ts file.
Actual behavior:
type SVGLocatable (and a bunch of derived classes) are missing since 2.2.1. In Version 2.2.0 there are still there.

I believe it was removed from lib.d.ts also

This https://developer.mozilla.org/en-US/docs/Web/API/SVGSVGElement#Specifications makes it sounds like they are removing SVGLocatable interface and if you look at SVG 2 there is no mention of SVGLocatable. Could that be why?

So is the recommendation to use SVGElement instead?

I was looking for getBBox() and found it on SVGGraphicsElement


unnecessary rebuild of unrelated packages in monorepo on member version bump

Suppose I have crates/foo and crates/bar in a monorepo. Whenever I bump the version of crates/foo, the content of  Cargo.lock at the repo root also changes. That inevitably triggers rebuild of crates/bar and all of its deps, even if crates/bar does not depends on crates/foo and none of its files (except for the shared Cargo.lock) change.
Is there any way to avoid this?

Hi @iwinux thanks for the report!
Currently we copy over the Cargo.lock completely untouched when creating the dummified sources (to build only dependency crates). When a workspace package's version changes, it gets recorded in the Cargo.lock and as you've observed it leads to a rebuild...
I don't think there is a (good?) way to avoid this besides adding an additional Cargo.lock file (i.e. workspace) for crates/bar and telling crane to build that workspace separately (crates/foo can still refer to crates/bar, but that workspace will also get rebuilt whenever crates/foo or crates/bar's versions change)

Implementing https://github.com/ipetkov/crane/issues/288 may be able to help with this (or rather it will avoid having to rebuild all of crates/bar's dependencies. Changing any version of a workspace package would still end up rebuilding the workspace crates even if all external deps aren't fwiw)


Remove auth

 Remove views requiring auth

@m-kus conflict is fixed & diff is back, ready to merge


[qtbase] Present feature preset to disable openssl

Is your feature request related to a problem? Please describe.
I am shipping on windows so I can use the schannel TLS backend instead of the openssl backend.  Right now no matter what the OpenSSL libraries are shipped since you cannot disable the openssl feature due to #35694.  While not normally an issue I would prefer to not have OpenSSL libraries in the final package because it causes customers "automated vulnerability scans" to trip every few months.  Currently I have a step to manually delete the files from the install root before it is packaged.
Proposed solution
This could be attacked one of three ways:

Remove OpenSSL as a default on Windows platform similar to securetransport on iOS. (Windows would always have schannel and Qt will autodetect that fact and compile it as long as nothing else disables it.)
Add a schannel feature flag that will  disable OpenSSL and enable schannel similar to how the securetransport feature flag works. (At least would give developers an option to explicitly use one or the other.  Not sure if they would ever want both.)
Provide a "negative" feature flag that would disable the openssl option (Not ideal since both could be defined, but would allow to disable other default features such as testilb and sql  )

Describe alternatives you've considered
I just copied the port files locally and removed openssl.
Additional context
No response

you cannot disable the openssl feature due to #35694.

This is not right. You just have to be very explicit, and and there must be no reverse dependencies which prevent your choice.

Add a schannel feature flag that will disable OpenSSL ...
Provide a "negative" feature flag that would disable the openssl option ...

Features must be additive. Downstreams cannot take away what other downstream requested.

This is not right. You just have to be very explicit, and and there must be no reverse dependencies which prevent your choice.

I am not sure I follow here.  I have nothing explicitly asking for that dependency at least that I know of.  Is there a way to trace this?
If features must be additive then it seems to me that "Option 1" would be the best bet since on Windows it would seem that OpenSSL would be redundant.

By the way I am using manifest mode to declare my dependencies, so I am not sure how that would work with vcpkg depend-info

Here is how I came to that conclusion:
PS C:\Users\geiseri\vcpkg> .\vcpkg.exe depend-info qtactiveqt
vcpkg-cmake:
vcpkg-cmake-config:
vcpkg-tool-meson:
zlib: vcpkg-cmake
brotli: vcpkg-cmake, vcpkg-cmake-config
bzip2[tool]: vcpkg-cmake
egl-registry:
libpng: vcpkg-cmake, vcpkg-cmake-config, zlib
pkgconf: vcpkg-tool-meson
vcpkg-cmake-get-vars: vcpkg-cmake
freetype[brotli, bzip2, zlib, png]: brotli, bzip2, libpng, vcpkg-cmake, vcpkg-cmake-config, zlib
lz4: vcpkg-cmake, vcpkg-cmake-config
opengl-registry: egl-registry
openssl: vcpkg-cmake, vcpkg-cmake-config, vcpkg-cmake-get-vars
vcpkg-pkgconfig-get-modules: pkgconf
double-conversion: vcpkg-cmake, vcpkg-cmake-config
harfbuzz[freetype]: freetype, vcpkg-tool-meson
icu[tools]:
libjpeg-turbo: vcpkg-cmake, vcpkg-cmake-config
libpq[lz4, openssl, zlib]: lz4, openssl, vcpkg-cmake-get-vars, vcpkg-pkgconfig-get-modules, zlib
opengl: opengl-registry
pcre2[platform-default-features, jit]: vcpkg-cmake, vcpkg-cmake-config
sqlite3[json1]: vcpkg-cmake, vcpkg-cmake-config
zstd: vcpkg-cmake, vcpkg-cmake-config
qtbase[brotli, concurrent, default-features, sql-sqlite, freetype, jpeg, dbus, gui, doubleconversion, harfbuzz, icu, network, opengl, openssl, pcre2, png, sql, sql-psql, testlib, thread, widgets, zstd]: brotli, double-conversion, freetype, harfbuzz, icu, libjpeg-turbo, libpng, libpq, opengl, openssl, pcre2, sqlite3, vcpkg-cmake, vcpkg-cmake-config, zlib, zstd
qtactiveqt: qtbase

Is that understanding incorrect?

I am using manifest mode and adding qtactiveqt into the project, so that depends on qtbase as such it uses default features, and ignores my setting.

Show your manifest.
The point of default-features is that they are enabled if you don't ask for something else.
Here, you would also need to add qtbase with "default-features": false to your manifest (aka top-level manifest). It is under your control.

PS C:\Users\geiseri\vcpkg> .\vcpkg.exe depend-info qtactiveqt

depend-info used to be very restricted. My recent contributions, part of the next tool update, will allow a more elaborate request:
$ ./vcpkg.mine depend-info qtactiveqt qtbase[core] --host-triplet x64-windows
vcpkg-cmake:
vcpkg-cmake-config:
egl-registry:
zlib: vcpkg-cmake
brotli: vcpkg-cmake, vcpkg-cmake-config
bzip2[tool]: vcpkg-cmake
libpng: vcpkg-cmake, vcpkg-cmake-config, zlib
opengl-registry: egl-registry
double-conversion: vcpkg-cmake, vcpkg-cmake-config
freetype[png, bzip2, zlib, brotli]: brotli, bzip2, libpng, vcpkg-cmake, vcpkg-cmake-config, zlib
opengl: opengl-registry
pcre2[platform-default-features, jit]: vcpkg-cmake, vcpkg-cmake-config
qtbase[thread, gui, widgets, freetype, doubleconversion]: double-conversion, freetype, opengl, pcre2, vcpkg-cmake, vcpkg-cmake-config, zlib
qtactiveqt: qtbase

In the meantime, you can try
vcpkg install --dry-run qtactiveqt qtbase[core] --host-triplet x64-windows

Here is my current manifest:
{
  "name": "hive-windows-client",
  "version-string": "v1.3.5",
  "dependencies": [
    {
      "name": "qtbase",
      "default-features": false,
      "features": [
        "concurrent",
        "doubleconversion",
        "freetype",
        "gui",
        "harfbuzz",
        "icu",
        "jpeg",
        "network",
        "opengl",
        "pcre2",
        "png",
        "testlib",
        "thread",
        "widgets",
        "zstd"
      ]
    },
    "qtimageformats",
    "qtactiveqt",
    "qtsvg",
    "qtwebsockets",
    "atlmfc",
    "magic-enum",
    {
      "name": "spdlog",
      "features": [
        "wchar"
      ]
    },
    "cppwinrt",
    "fmt",
    "simonbrunel-qtpromise"
  ]
}

So if I run the vcpkg install with those deps I see:
PS C:\Users\geiseri\vcpkg> .\vcpkg.exe install --dry-run qtactiveqt qtbase[core] qtimageformats qtwebsockets qtsvg atlmfc magic-enum spdlog[wchar] cppwinrt fmt simonbrunel-qtpromise --host-triplet x64-windows
Computing installation plan...
The following packages will be built and installed:
  * atl:x64-windows@0
    atlmfc:x64-windows@0#3
  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS><EMAIL_ADDRESS>  *<EMAIL_ADDRESS>  * egl-registry:x64-windows@2024-01-25
<EMAIL_ADDRESS>  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS><EMAIL_ADDRESS>  * opengl:x64-windows@2022-12-04#3
  * opengl-registry:x64-windows@2024-02-10#1
  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS>  * vcpkg-cmake:x64-windows@2023-05-04
  * vcpkg-cmake-config:x64-windows@2022-02-06#1
  * vcpkg-cmake-get-vars:x64-windows@2023-12-31
  * vcpkg-pkgconfig-get-modules:x64-windows@2023-09-06
  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS>Additional packages (*) will be modified to complete this operation.

So after some mucking around I discovered that simonbrunel-qtpromise is the offender.
If I understand correctly it would be because it has
  "dependencies": [
    "qtbase",
    {
      "name": "vcpkg-cmake",
      "host": true
    }
  ]

and not
  "dependencies": [
   {
      "name": "qtbase",
      "default-features": false
    {
      "name": "vcpkg-cmake",
      "host": true
    }
  ]

Indeed, simonbrunel-qtpromise is a port which prevents the choice. And it shouldn't.


KAFKA-8613: KIP-633 New APIs for Controlling Grace Period for Windowed Operations

KIP-633 New APIs for Controlling Grace Period for Windowed Operations

Added API changes by KIP-633 for JoinWindows, SessionWindows, TimeWindows and SlidingWindows
Renamed Windows.DEFAULT_GRACE_PERIOD_MS to DEPRECATED_OLD_24_HR_GRACE_PERIOD
Added new constant Windows.NO_GRACE_PERIOD to avoid magic constants when 0 is specified as grace Period
Added preliminary Java unit test cases for new API methods
Replaced Deprecated calls with equivalent in Examples
Replaced Deprecated API calls in Scala tests with updated API method calls
Added Deprecation suppression in Tests for derecated API method calls in Java and Scala Tests

modified:   streams/src/main/java/org/apache/kafka/streams/kstream/JoinWindows.java
modified:   streams/src/main/java/org/apache/kafka/streams/kstream/SessionWindows.java
modified:   streams/src/main/java/org/apache/kafka/streams/kstream/SlidingWindows.java
modified:   streams/src/main/java/org/apache/kafka/streams/kstream/TimeWindows.java
modified:   streams/src/main/java/org/apache/kafka/streams/kstream/Windows.java
modified:   streams/examples/src/main/java/org/apache/kafka/streams/examples/pageview/PageViewTypedDemo.java
modified:   streams/examples/src/main/java/org/apache/kafka/streams/examples/pageview/PageViewUntypedDemo.java
modified:   streams/examples/src/main/java/org/apache/kafka/streams/examples/temperature/TemperatureDemo.java
modified:   streams/streams-scala/src/main/scala/org/apache/kafka/streams/scala/kstream/KStream.scala
modified:   streams/streams-scala/src/test/scala/org/apache/kafka/streams/scala/TopologyTest.scala
modified:   streams/streams-scala/src/test/scala/org/apache/kafka/streams/scala/kstream/KStreamTest.scala
modified:   streams/streams-scala/src/test/scala/org/apache/kafka/streams/scala/kstream/KTableTest.scala
modified:   streams/src/test/java/org/apache/kafka/streams/StreamsBuilderTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/TopologyTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/integration/AbstractResetIntegrationTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/integration/InternalTopicIntegrationTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/integration/JoinStoreIntegrationTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/integration/KStreamAggregationDedupIntegrationTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/integration/KStreamAggregationIntegrationTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/integration/KStreamRepartitionIntegrationTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/integration/MetricsIntegrationTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/integration/QueryableStateIntegrationTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/integration/RocksDBMetricsIntegrationTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/JoinWindowsTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/RepartitionTopicNamingTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/SessionWindowsTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/SlidingWindowsTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/TimeWindowsTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/KGroupedStreamImplTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/KStreamImplTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/KStreamKStreamJoinTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/KStreamKStreamLeftJoinTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/KStreamKStreamOuterJoinTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/KStreamRepartitionTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/KStreamSessionWindowAggregateProcessorTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/KStreamSlidingWindowAggregateTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/KStreamWindowAggregateTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/SessionWindowedCogroupedKStreamImplTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/SessionWindowedKStreamImplTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/SlidingWindowedCogroupedKStreamImplTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/SlidingWindowedKStreamImplTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/SuppressScenarioTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/SuppressTopologyTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/TimeWindowTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/TimeWindowedCogroupedKStreamImplTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/TimeWindowedKStreamImplTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/graph/GraphGraceSearchUtilTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/kstream/internals/graph/StreamsGraphTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/processor/internals/RepartitionOptimizingTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/processor/internals/StreamsPartitionAssignorTest.java
modified:   streams/src/test/java/org/apache/kafka/streams/tests/SmokeTestClient.java
modified:   streams/src/test/java/org/apache/kafka/streams/tests/StreamsOptimizedTest.java
modified:   streams/src/test/java/org/apache/kafka/test/GenericInMemoryKeyValueStore.java
modified:   streams/src/test/java/org/apache/kafka/test/GenericInMemoryTimestampedKeyValueStore.java
More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@ableegoldman @mjsax @showuon @cadonna
When you have a moment, please take a look.
This is an update based on the feedback from PR #10740

I will close this one and resolve the conflicts before opening the next one.


[RP Adapter] Create new DID for AuthZ credential

For AuthZ credential, the wallet needs to request for a new DID from RP adapter and pass it to the issuer.
Related to https://github.com/trustbloc/edge-agent/issues/334

Request:
{
   "@id":"089a0775-7e5f-4b96-912f-25532ec6853d",
   "@type":"https://trustbloc.dev/adapter/1.0/diddoc-req"
}

Response:
{
   "@id":"a8fb8f8f-4137-4e4e-9168-9b34f8d93fee",
   "@type":"https://trustbloc.dev/adapter/1.0/diddoc-resp",
   "~thread":{
          "thid":"089a0775-7e5f-4b96-912f-25532ec6853d"
   },
   "data":{
          "errorMsg":"<inCaseOfFailure>",
          "didDoc":{
                 <adapterDIDDoc>
          }
   }
}


test EG against multiple Kubernetes versions

We should test EG against multiple Kubernetes versions -- IMO at least the latest 3 minor versions.

Assigning myself, will pick this up soon.


MarginReportEntry bug fix

Description
Account.MarginEntryReport changed to also handle options on futures correctly.
Some fields were made Optional
Related issue(s)
Fixes #144
Pre-merge checklist

[x] Passing tests LOCALLY
[ ] New tests added (if applicable)

Please note that, in order to pass the tests, you'll need to set up your Tastytrade credentials as repository secrets on your local fork. Read more at CONTRIBUTING.md.

Thanks for this! Can you keep point_of_no_return_percent optional?


Plugin only renders first page

The plugin doesn't seem to be able to render multi-page documents. I'm rendering as SVG in the example, but it's the same with images!
Preamble
// typography
#set page(paper: "a4", fill: rgb("f7f9ff"), margin: 2.5cm, numbering: "1")
#set text(
	font: "Halant",
	size: 10pt,
	fill: rgb("5c6166")
)
#set par(justify: true, leading: 15pt)
#show heading: set text(
	font: "Montserrat",
	weight: "semibold"
	)
#show heading: set block(
	spacing: 15pt
	)

// citations
#set cite(style: "chicago-notes", brackets: false)
#set bibliography(style: "chicago-author-date")
#show "“": "‘"
#show "”": "’"

Codeblock
= h1
== h2
#lorem(100)

== h2
#lorem(200)

== h2
#lorem(300)

= h1
#lorem(100)

Intended result
Test.pdf
Actual Result

As this plugin is intended to only render code blocks and math blocks, it only makes sense to render the first page. Once a custom editor has been added multiple pages could be supported.

An implementation is planned but its waiting on HTML export from typst as that would remove a lot of the headache. I think @SekoiaTree started looking into the impl. then switched to looking into the html export and then I lost track :)

Haha yeah I've been relatively busy since sadly

hi! so just to clarify, I have a bunch of math blocks interspersed with standard Obsidian text. when I export to PDF, my content is longer than 1 page, and so only the math blocks on the first page are rendered, the rest are not. this is currently expected behavior, right? thank you!

@aczw I think your problem is closer to #29 than this one.

when I export to PDF, my content is longer than 1 page, and so only the math blocks on the first page are rendered, the rest are not. this is currently expected behavior, right?

In a way it is expected behaviour but it is still a problem that should be fixed. Its most likely because Obsidian expects the renderMath function to be blocking (when it returns it is completely done). However, the plugin overrides this and the function is async/nonblocking so it returns before it is "done rendering". I'm not entirely sure how to fix this and honestly I need to find a better way to override the default math blocks.
As a work around you try using pandoc to convert the markdown into Typst...


Switch to kubernetes stable release

Kubernetes unstable release breaks occasionally, this PR switches the deployment to stable version.
This PR also adds guide for setting subnet routes.

This change is


[MXNET-187] [WIP] fake shuffle functions

Description
(Brief description on what this PR is about)
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

https://issues.apache.org/jira/browse/MXNET-187

It may not be worth supporting Fermi architecture

I think makefile and cmake currently automatically list all archs. Would it list 2.0 and cause compile error here?

No, we call nvcc before the actual compilation happens to filter out unsupported cuda archs. In worst case, there won't be any pre-compilation happening but the kernels have to be compiled during runtime.

Took out Fermi from cmake...

See: https://github.com/apache/incubator-mxnet/pull/10486/files


Implement __repr__ for cupyx.profiler._time._PerfCaseResult

Description
Let's implement __repr__ for interactive benchmarking.
>>> cupyx.profiler.benchmark(lambda: numpy.asarray_chkfinite(z), n_repeat=10, n_warmup=0)
<cupyx.profiler._time._PerfCaseResult object at 0x7f3025808c40>

Additional Information
No response

Can I Give It A Try?

@Subhraneel77 Sure!

I want to contribute - is there any format that the returned string should have?

Fixed in #6617


Support Slack

Maybe create a Tekton channel in Kubernetes Slack?

There is a slack channel currently in the knative slack https://knative.slack.com/messages/CDCQ72D0R

You can see all of our docs about getting in contact at https://github.com/tektoncd/community/blob/master/contact.md
(And we'll update there once we have our own slack!)


proposal: wasm: Thread support

Problem
Most browsers (Chrome, FF, Safari, etc.) are shipping with thread support in Wasm. See https://webassembly.org/roadmap.
Based on discussions in #28360 and #28631, this proposal aims to create progress on Wasm thread support in Go. While standardization of threads is still in Phase 2 (with a vote to move to Phase 3 happening later this month), experimental thread support will improve performance for many Go Wasm applications. Wasm is a great tool for performing computationally intensive tasks, but without thread support, it is often more performant to use JavaScript alone.
Proposal
Until standardized, thread support should exist behind the GOWASM environment variable (e.g. GOWASM=threads).
A program using go routines can be compiled using the following:
GOOS=js GOARCH=wasm GOWASM=threads go build -o out.wasm

How is this issue different from #28631?

How is this issue different from #28631?

#28631 was not marked as a proposal, I figured filing specifically as a proposal may encourage more progress.

A proposal isn't a way to get someone else to do the work, only to accept and approve of work that other people may do.
Duplicate of #28631


Regional gas prediction endpoint

Admittedly the pickled models are predicting a little high right now, but they are predicting, and I laid down the groundwork to replace them super easily.  So I want to go ahead merge this to the main branch, and get it up on EB.
Once it is up, I plan on trying to replace the linear models with a GRU based NN. I've been having some success/higher accuracy with a GRU NN in the last few weeks.

Looks good from my viewpoint. I think once all the endpoints are up and running we can focus on refining the models. I'll try to keep a similar syntax for the Airbnb/hotel endpoints.
Random thought, joblib might be helpful if pickling isn't working. Might be worth a try if it's easier for uploading to AWS too.


[Flutter Web] I don't know where else to ask -- am I missing something here?

flutter pub run sentry_dart_plugin

[❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚] 0% reading config values..
sentry-cli is not available under the assets folder, using pre-installed sentry-cli
☑ reading config values
[❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚] 9% validating config values..
Auth Token is empty, check 'auth_token' at pubspec.yaml or SENTRY_AUTH_TOKEN env. var.
pub finished with exit code 1

Is there a way for me to 'login' at this pre-installed sentry-cli so I don't have to pass any configuration options?
I've installed sentry-cli via npm i -g @sentry/cli, but sentry_dart_plugin doesn't use that.
I'm on Windows 10 btw.
Thanks!

I figured out this part; sorry for the inconvenience :))


Fix Player initialization issues

The player initialization currently has some issues, including missing params and unsupported defaults. We need to make sure the setup is done with the following properties:

Set the ad schedule from the adSchedule value in the app config
Set casting from a custom param in the app config: webapp.enableCasting
Set sharing from a custom param in the app config: webapp.enableSharing

Only support link sharing at the moment
Ensure the URL is correct

Recommendations playlist from the app config recommendationsPlaylist

Otherwise defaults as follow:
  "autoPause": {
    "viewability": false
  },
  "autostart": false,
  "controls": true,
  "displayHeading": false,
  "displaydescription": false,
  "displaytitle": false,
  "floating": {
    "mode": "never"
  },
  "mute": false,
  "pipIcon": "disabled",
  "playbackRateControls": true,
  "related": {
    "autoplaytimer": 0,
    "displayMode": "shelf",
    "oncomplete": "autoplay"
  },
  "repeat": false,
  "stretching": "uniform",
  "width": "100%"

@dbudzins one comment here: we decided to remove enableCasting completely, so I will only add enableSharing.

Recommendations shouldn't be at player level. Just at app level as a "related videos' list


useSelection make input can't accept typing

A very odd bug: input element can't accept typing input when using useSelection hook.
Demo: https://codesandbox.io/s/reagraph-example-forked-hmdomr?file=/src/App.tsx

Found reason, code below prevent all keydown event
  const onKeyDown = (event: KeyboardEvent) => {
    event.preventDefault();
    setMetaKeyDown(event.metaKey || event.ctrlKey);
  };

  useEffect(() => {
    window.addEventListener('keydown', onKeyDown);

    return () => {
      window.removeEventListener('keydown', onKeyDown);
    };
  });

Good Catch! Should be able to detect target and handle.

Fixed.


refactor: committer node index

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#154
#160  👈
#156
#140
#133
main-v0.13.2

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @AvivYossef-starkware and the rest of your teammates on  Graphite

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#154
#160  👈
#156
#140
#133
main-v0.13.2

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @AvivYossef-starkware and the rest of your teammates on  Graphite

Merge activity

Aug 5, 6:34 AM EDT: @AvivYossef-starkware started a stack merge that includes this pull request via Graphite.


Examples/Examples under Schema Object wrongly documented

Should follow the same type documentation as in the Parameter Object.

No longer relevant.


Fix the height of the Challenges listing and Challenges detail screen

On Challenges listing and Challenges detail screens the scrollable view is too big, so when we scroll to the end of list, we see a lot of blank space.
Related to #8, #9

This was fixed in an earlier commit.


[Java] Exception when trying to load next batch while reading a parquet file

Describe the bug, including details regarding any error messages, version, and platform.
Getting this exception when trying to load next batch while reading a parquet file.
The parsing works if the batch size is big enough to process all the parquet contents in one shot. But if I try to give a smaller batch size , the code breaks giving the below exception
java.lang.IllegalArgumentException: should have as many children as in the schema: found 0 expected 8
at org.apache.arrow.util.Preconditions.checkArgument(Preconditions.java:282)
at org.apache.arrow.vector.VectorLoader.loadBuffers(VectorLoader.java:127)
at org.apache.arrow.vector.VectorLoader.load(VectorLoader.java:84)
at org.apache.arrow.c.Data.importIntoVectorSchemaRoot(Data.java:334)
at org.apache.arrow.dataset.jni.NativeScanner$NativeReader.loadNextBatch(NativeScanner.java:151)
Component(s)
Java

Hi @vinayvenk, what Arrow Java version are you on? Can you share the code you ran, ideally with code that can generate the data causing this?

GM @amoeba  it is pretty much standard code that i got from the example and I tried using 16.0.1, 16.0.0 and 15.0.1
String uri = parquetFile.toURI().toString();
ScanOptions options = new ScanOptions(/* batchSize */ 32768);
try (BufferAllocator allocator = new RootAllocator();
DatasetFactory datasetFactory = new FileSystemDatasetFactory(allocator, NativeMemoryPool.getDefault(),
FileFormat.PARQUET, uri);
Dataset dataset = datasetFactory.finish();
Scanner scanner = dataset.newScan(options);
ArrowReader reader = scanner.scanBatches()) {
int batchCount = 0;
while (reader.loadNextBatch()) {
try (VectorSchemaRoot root = reader.getVectorSchemaRoot()) {
//function to create csv data
createCSV()
}
it fails when it tries to load the second batch.


"Error: spawn UNKNOWN"

When trying to use the CLI version:
node_modules\.bin\babili.cmd -d .

internal/child_process.js:298
    throw errnoException(err, 'spawn');
    ^

Error: spawn UNKNOWN
    at exports._errnoException (util.js:856:11)
    at ChildProcess.spawn (internal/child_process.js:298:11)
    at Object.exports.spawn (child_process.js:367:9)
    at Object.<anonymous> (C:\temp\babilitest\node_modules\babili\lib\index.js:16:25)
    at Module._compile (module.js:413:34)
    at Object.Module._extensions..js (module.js:422:10)
    at Module.load (module.js:357:32)
    at Function.Module._load (module.js:314:12)
    at Module.require (module.js:367:17)
    at require (internal/module.js:16:19)

Windows 10, Node v5.7.1 (I should probably upgrade Node, guess)

It works if I run Babel directly:
node_modules\.bin\babel.cmd --presets=babili in\test.js

function helloWorld(){foo()}helloWorld();

import child from "child_process";

const args = process.argv.slice(2).concat(["--presets=babili"]);

const opts = {
  stdio: "inherit",
  env: process.env,
};

child.spawn(require.resolve("babel-cli/bin/babel"), args, opts);

must be an issue with child.spawn like in lerna?

Yeah... Windows doesn't use shebangs, so executing a JavaScript file directly won't work. Instead, you'll probably have to execute node directly:
import child from "child_process";

const args = [require.resolve("babel-cli/bin/babel")]
  .concat(process.argv.slice(2).concat(["--presets=babili"]));

const opts = {
  stdio: "inherit",
  env: process.env,
};

child.spawn('node', args, opts);

That is, run node babel-cli/bin/babel.js rather than just babel-cli/bin/babel. It should work the same as what it's currently doing.

Ok cool didn't know that - do you want to pr that one?


Add addNewAccountWithoutUpdate method

This adds a new method for adding accounts without updating identities in the preferences controller
this is currently being used in https://github.com/MetaMask/metamask-mobile/pull/1879

I see that createNewVaultAndRestore is used in a few places where new-account-detection would be undesired (e.g. upon changing your password), so maybe putting the logic there is less than ideal.
Maybe there should be two methods - createNewVaultAndRestore for this, and recreateVault for re-creating a vault with the same contents? :shrug: I think all of the uses are one of these two cases.
This might be a good idea for the extension as well :thinking:


Custom errors are not decoded (hex shown)

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (d431f74 2024-04-27T00:16:52.929844000Z)
What command(s) is the bug in?
forge test
Operating System
macOS (Apple Silicon)
Describe the bug
I would like to reopen the issue from here https://github.com/foundry-rs/foundry/issues/6579
I am not sure if the issue has been fixed because I am also getting hexadecimal custom errors.

To recreate the bug, please go to this repo here and use the command ->
forge test --match-path test/scWETHv2.base.t.sol --match-test test_invest_basic -vv

#6579 was done for custom errors within setup, will check the scenario you hit

managed to recreate, the error is ErrorNotEnoughBalance thrown by the ERC20Bridged contract https://basescan.org/address/0x69ce2505CE515C0203160450157366F927243309 which is not an artifact created during test so the abi to properly decode error is not available

yep, I managed to recreate with the project above, the error is ErrorNotEnoughBalance thrown by the ERC20Bridged contract https://basescan.org/address/0x69ce2505CE515C0203160450157366F927243309 which is not an artifact created during test so the abi to properly decode error is not available - you can check this by for example adding error ErrorNotEnoughBalance(); in contract PriceConverter which is created during test run, it will output
Failing tests:
Encountered 1 failing test in test/scWETHv2.base.t.sol:scWETHv2Base
[FAIL. Reason: ErrorNotEnoughBalance()] test_invest_basic() (gas: 852945)

I see. Thank you so much. Can you please tell me how did you decode the error is ErrorNotEnoughBalance() from the hex code please? That was really amazing.

I took the code for failing contract and run forge inspect errors on it, see https://book.getfoundry.sh/reference/cli/forge/inspect?#forge-inspect
You can also use forge selectors list https://book.getfoundry.sh/reference/cli/forge/selectors/list
(to get the code you can use forge clone or cast etherscan to download from verified contracts but you could only get the errors from failing contract, put in a local contract and run the command against)

I took the code for failing contract and run forge inspect errors on it, see https://book.getfoundry.sh/reference/cli/forge/inspect?#forge-inspect You can also use forge selectors list https://book.getfoundry.sh/reference/cli/forge/selectors/list
(to get the code you can use forge clone or cast etherscan to download from verified contracts but you could only get the errors from failing contract, put in a local contract and run the command against)

So cooooooooollll. Thank you so much man. That was very helpful.


Keyboard shortcuts for showing/hiding unchanged regions

Testing #186213

I've noticed that if I use cmd+down/up, it will unfold that region and move the cursor in. I can't find any other keyboard shortcuts for it and the commands don't show up in the command palette. Just one option that comes to mind

The cursor is on the line above this collapsed region
I press down arrow, it focuses the collapsed region indicator
I can press down arrow again to move the cursor to the region below the collapsed region indicator
Or, I can press enter to expand it

Can I submit a PR for this issue?

@weartist I think this requires more thoughts on how these commands should work.
@jrieken I know you do something similar with the inline chat, maybe I should copy your solution

I think first there should be simple commands that replicate the UI button actions:

Inside an expanded unchanged region: "Fold unchanged region", same as the button at the top of the region
Outside: "Expand next folded region" and "Expand previous folded region", same as the "Show all" button

Then, maybe this combined with context keys can achieve the keybind CTRL+up/down suggested. For example "isAboveCollapsedRegion" when the next line is collapsed.
On top of that there could be a command to expand/collapse line by line instead of the whole region. So you could hold/tap CTRL+up/down to expand a few lines like you can do now with dragging.

Expand/fold all command requested at https://github.com/microsoft/vscode/issues/188416 already with a PR


fix: console.error for input error message

A little code edit

Codecov Report

Merging #221 (d857e1b) into master (57f8273) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #221   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines           23        20    -3
  Branches         3         2    -1
=========================================
- Hits            23        20    -3

Impacted Files
Coverage Δ

src/inputVerifier.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add basic unit tests

Right now the project has no unit tests. We should add some basic tests :)

Hmm. We could probably split this into two problems, one for db tests and one for the dash app. The former should not be too hard - hard-code some queries and check data consistency. Dash app tests may be a little more involved (https://dash.plotly.com/testing). Am I missing anything?

Yeah I think those are the two major pieces right now. Probably something like a basic smoke test just to ensure the dash app runs for example would be key.
There are other things we could eventually test like that the plots auto update and the clicking behaviors work correctly, but those are of lesser value right now I think.

@all-contributors please add @abesto for his awesome test code!

I've added the ability to trigger the actions manually above and it works great:

Per our discussion in #67 @abesto, I've created #70.


Added first iteration of Shadow Fiend (IMBA)

Added the first iteration of Shadow Fiend (IMBA).
Only added tooltips for modifiers so far, no ability tooltips.

Great stuff! Added

when firesenpai merges your PR


Log Colored Sample to Ensure Collator Correctness

It may be useful to have a helper function which pretty prints a collated sample. I want to ensure the ignore index is working properly with DataCollatorForCompletionOnlyLM, so I'm using this function to verify correctness:
def print_colored_collated_example(trainer):
    print("Training Sample. Red = ignored (not included in loss calculation)")

    def colorized_decode(tokenizer, input_ids, labels, ignore_index):
        RED = '\033[91m'
        ENDC = '\033[0m'
        words = []
        for token_id, label in zip(input_ids[0], labels[0]):  # Assuming 2D tensors
            decoded_token = tokenizer.decode(token_id, skip_special_tokens=True).strip()
            if label == ignore_index:
                words.append(f'{RED}{decoded_token}{ENDC}')
            else:
                words.append(decoded_token)
        return ' '.join(words)

    example = random.choice(trainer.train_dataset)
    collated_example = trainer.data_collator([example])
    colorized_example = colorized_decode(
        trainer.tokenizer,
        collated_example["input_ids"],
        collated_example["labels"],
        trainer.data_collator.ignore_index
    )
    print(colorized_example)

It prints a decoded training sample with every token labelled with the ignore_index in red. It has slightly incorrect behavior which should be adjusted (e.g. space joining tokens, using 'print' instead of logger), but it's a good starting point.

Thanks for the response. If you think this is out of scope for trl that's understandable.

I think for now yes, but thanks a lot for outlining a solution, I am sure it will be helpful for others, too!


Image to 3D run out of RAM on colab

I have bought colab pro to run image to 3D. Things went well until I ran in training stage. I follow README.md to run the command using zero123 backend , but when it trained about 2 minutes , system RAM is not enough, and GPU RAM  seems not being used as picture showed below, how can I solve the problem?

Code
The code I ran :
https://colab.research.google.com/drive/1jP_eM8FMeyygv0NfmQsEcThjmy91sTvw?authuser=1#scrollTo=RH-8W-VT_XiU
Environment
google colab pro

same for me, adding --text for using SD backend fixes the issue

You can change your execution type to A100 and then you will have 83.5 GB system RAM and 40 GB GPU RAM. In this execution type, Image to 3D works really well !


FIAM: explicitly import modules used in Swift

Should fix the failure from https://github.com/firebase/firebase-ios-sdk/issues/8471#issuecomment-893449537.

LGTM. I wonder why these missing imports in FIAM qs and SDK didn't cause failures previously...

Good question. I don't know for sure, I guess UIKit was imported implicitly in some way before. But importing all required modules in a Swift file is a best practice so it seems like a reasonable change to me.


Changed @resource annotation to @group

This PR makes it possible to use the @group annotation instead of the current @resource one as stated in issue #338

Codecov Report

Merging #371 into master will not change coverage.
The diff coverage is 66.66%.

@@           Coverage Diff            @@
##             master    #371   +/-   ##
========================================
  Coverage      79.6%   79.6%
  Complexity      153     153
========================================
  Files             7       7
  Lines           407     407
========================================
  Hits            324     324
  Misses           83      83

Impacted Files
Coverage Δ
Complexity Δ

src/Generators/AbstractGenerator.php
77.6% <100%> (ø)
80 <0> (ø)
:arrow_down:

src/Commands/GenerateDocumentation.php
97.27% <50%> (ø)
34 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


F parameters

Ⅰ. Describe what this PR did
Change the data structure of ParameterHolder from 2 Dimension Array to Map so that we don't need to execute getParameterCount() while initializing PreparedStatementProxy,which will execute an additional business independent SQL query.
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #2242 into develop will not change coverage by %.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop    #2242   +/-   ##
==========================================
  Coverage      53.54%   53.54%
  Complexity      2619     2619
==========================================
  Files            491      491
  Lines          15369    15369
  Branches        1723     1723
==========================================
  Hits            8230     8230
  Misses          6330     6330
  Partials         809      809

Codecov Report

Merging #2242 into develop will decrease coverage by 0.09%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             develop    #2242     +/-   ##
============================================
- Coverage       53.8%   53.71%   -0.1%
+ Complexity      2622     2619      -3
============================================
  Files            491      491
  Lines          15385    15388      +3
  Branches        1760     1762      +2
============================================
- Hits            8278     8265     -13
- Misses          6325     6337     +12
- Partials         782      786      +4

Impacted Files
Coverage Δ
Complexity Δ

...io/seata/rm/datasource/PreparedStatementProxy.java
100% <ø> (ø)
5 <0> (ø)
:arrow_down:

...a/sqlparser/druid/oracle/BaseOracleRecognizer.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ata/sqlparser/druid/mysql/BaseMySQLRecognizer.java
90.47% <100%> (ø)
6 <0> (ø)
:arrow_down:

...va/io/seata/rm/datasource/exec/InsertExecutor.java
65.43% <100%> (ø)
39 <0> (ø)
:arrow_down:

...in/java/io/seata/server/session/SessionHolder.java
56.09% <50%> (-2.88%)
17 <1> (ø)

.../rm/datasource/AbstractPreparedStatementProxy.java
95.73% <80%> (-0.64%)
55 <3> (-1)

.../rm/datasource/undo/mysql/MySQLUndoLogManager.java
69.69% <0%> (-9.1%)
6% <0%> (ø)

...ata/rm/datasource/undo/AbstractUndoLogManager.java
53.75% <0%> (-3.75%)
17% <0%> (ø)

...o/seata/server/coordinator/DefaultCoordinator.java
54.35% <0%> (-0.52%)
28% <0%> (-1%)

...in/java/io/seata/server/session/GlobalSession.java
84% <0%> (-0.45%)
72% <0%> (-1%)

@CharmingRabbit please resolve conflict.

Codecov Report

Merging #2242 into develop will increase coverage by 0.59%.
The diff coverage is 77.77%.

@@              Coverage Diff              @@
##             develop    #2242      +/-   ##
=============================================
+ Coverage      50.03%   50.62%   +0.59%
+ Complexity      2954     2812     -142
=============================================
  Files            592      558      -34
  Lines          18998    17923    -1075
  Branches        2290     2093     -197
=============================================
- Hits            9506     9074     -432
+ Misses          8549     7964     -585
+ Partials         943      885      -58

Impacted Files
Coverage Δ
Complexity Δ

...io/seata/rm/datasource/PreparedStatementProxy.java
100.00% <ø> (ø)
5.00 <0.00> (ø)

...a/sqlparser/druid/oracle/BaseOracleRecognizer.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...ser/druid/postgresql/BasePostgresqlRecognizer.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

.../rm/datasource/AbstractPreparedStatementProxy.java
96.29% <100.00%> (-0.07%)
55.00 <3.00> (-1.00)

...va/io/seata/rm/datasource/exec/InsertExecutor.java
63.27% <100.00%> (ø)
45.00 <0.00> (?)

...ata/sqlparser/druid/mysql/BaseMySQLRecognizer.java
90.47% <100.00%> (ø)
6.00 <0.00> (ø)

...ing/annotation/GlobalTransactionalInterceptor.java
8.87% <0.00%> (-16.33%)
2.00% <0.00%> (-5.00%)

...main/java/io/seata/core/rpc/netty/RmRpcClient.java
30.76% <0.00%> (-10.90%)
9.00% <0.00%> (-1.00%)

...obuf/convertor/BranchRegisterRequestConvertor.java
90.47% <0.00%> (-9.53%)
3.00% <0.00%> (ø%)

.../rm/datasource/undo/mysql/MySQLUndoLogManager.java
63.88% <0.00%> (-8.34%)
6.00% <0.00%> (ø%)

... and 116 more

@CharmingRabbit please resolve conflict.
Resolved，please take a look.


Node owners are incorrect for testnet and previewnet

Description
Check a transaction including hbar transfers on testne or previewnett, the map of payments to nodes under "Hbar Transfers" list node owners from the mainnet list, on testnet and previewnet, nodes are not hosted by council members.
e.g. https://hashscan.io/#/testnet/transaction/0.0.11093-1652180465-382616732?t=1652180476.585125656
should not show Node 0 - LG - Seoul, South Korea for account 0.0.3
Steps to reproduce
navigate to https://hashscan.io/#/testnet/transaction/0.0.11093-1652180465-382616732?t=1652180476.585125656
Hbar transfers map should not show Node 0 - LG - Seoul, South Korea for account 0.0.3, the node description is incorrect for testnet and previewnet.
Additional context
No response
Hedera network
testnet
previewnet
Version
n/a
Operating system
No response

Does it still make sense to label 0.0.98 with « Hedera fee collection account » in that context ?


http2: add per connection(stream 0) HTTP/2 METADATA frames

Description:
Envoy added per stream metadata frame #2394
It's a naive requirement to extend the METADATA frame support to stream 0.
Below metadata fit per connection metadata category.

NodeInfo: the node info is supposed to be stable during the life of the connection.
Connection trace context.

Where to put the new config?
A ConnectionMetaData message and field within Http2ProtocolOptions. This option lives in both client and server http/2 connections.
Is anyone feel this useful?
CC @htuch @alyssawilk

CC @tbarrella This might affect the eventual shape of our upstream cluster config

I don't think any of the teams currently using metadata have a need for it.
Can you flesh out the use case a bit more? Who would receive that metadata and what would they do with it

I don't think any of the teams currently using metadata have a need for it.
Can you flesh out the use case a bit more? Who would receive that metadata and what would they do with it

stream 0 metadata is going to address issue that per stream metadata is sent again and again.
Assume the both end is Envoy,

The async client can encode the client node info into the stream 0. The server side HCM can create fake stream 0 to decode metadata and put in conneciton stream info, so that stream 1,3,... can get these per connection info without decoding.
For the metadata from server node should have the similar op. Although there is no http upstream filter yet, we can still persist the stream 0 metadata from server in the http async client. Ideally each upstream request could consume the metadata.

Alternatively we can specialize ConnectionImpl::onMetadataReceived if the passed in stream id == 0.
The metadata by definition is not a req/resp mode. It's not too hard to define a per connection metadata filter spec and only allow the metadata filter to access stream info and H2ProtocolOptions.
Does the above motivation reasonable?

"stream 0 metadata is going to address issue that per stream metadata is sent again and again."
Again, what stream metadata is sent again and again?  I don't think I understand the use case yet.
The use cases I know of metadata are to send things like RTT, which changes over the course of the stream and session lifetime so it's useful to resend.  Alternately if you want to send once and have it resent on all further streams, you could do something like have an HTTP filter which intercepted the metadata, added it to the connection's StreamInfo, and other streams would pull it from the connection's StreamInfo and forward it on as metadata.  I don't think we'd need HCM changes for that.  WDYT?

Alternately if you want to send once and have it resent on all further streams, you could do something like have an HTTP filter which intercepted the metadata, added it to the connection's StreamInfo, and other streams would pull it from the connection's StreamInfo and forward it on as metadata.

Thanks!  This kind of metadata is shared by all streams within the parent connection, ideally we want a dedicate stream 0 which follows how stream 0 is defined.
I think this should work: The first stream set and the rest of the rest of the streams consume to avoid HCM change

@lambdai per Alyssa, do you have an actual use case for this? If so, what is it? We don't want to add a feature like this unless there is a real use case that someone is going to implement.

The use case is to transmit connection attributes and node attributes bidirectionally.
I see node metadata and connection metadata can be delivered without metadata but using header.
stream 0 is easier to build "wait for certain metadata" before the normal http stream, but the first stream to carry and set should work. Please consider the stream 0 metadata as "nice to have" and we can wait for more use cases

I understand what it can theoretically do, but what I want to know is if someone has a production use case they are going to implement. Until that exists let's hold on this.

I understand what it can theoretically do, but what I want to know is if someone has a production use case they are going to implement. Until that exists let's hold on this.

Istio does use https://github.com/istio/proxy/tree/master/src/envoy/tcp/metadata_exchange as a workaround. Think of it as an extended PROXY protocol.
Istio also has http version which currently repeatedly exchanges metadata for each stream.
Also istio is pushing all tcp traffic to be tunnelled in http/2, so per connection metadata serve both http traffic and tcp traffic in istio world.
I create this issue to see if other people find per connection metadata a better abstraction to replace the individual workarounds in production

Is the per connection data something you want to send once at the beginning of the connection, or something you need optional updates on?  You said node info remains constant and I'm not sure if you'd be passing actual trace events, or just a trace ID.
While conceptually I like the idea of connection metadata on stream 0, one problem offhand with doing this at the HTTP/2 layer, is where we'd stick plug-ins. If we do something like proxy proto we could ideally have tcp upstream filters adding this metadata, and tcp downstream filters interpreting it and putting it in stream info, and folks can customize what they'd like to have added and removed.  But if the HCM is getting per connection data, it's unclear where we'd plug in the per-customer business logic of what data is added and how it is processed. Plus if we allow "proxy proto extensibility" it's useful for both HTTP/2-over-tcp and actual TCP (google has our own in-house proxy proto style header).

Yeah it does seem like we would need stream 0/connection filters for this kind of thing. Overall I like this idea, but we should flesh it out into a full design doc I think.

Is the per connection data something you want to send once at the beginning of the connection, or something you need optional updates on? You said node info remains constant and I'm not sure if you'd be passing actual trace events, or just a trace ID.

The trace id is what I was referring to. All of my existing production use cases are send-once-shared-by-all-stream.
Delivering occasional trace event during the full life cycle of h2 connection is attractive but I don't have a timeline for this requirement. The stream 0 filter api is obscure for this scenario. I believe a congestion control is a hard dependency.
I prefer build the "metadata exchange in h2" to "L4 transport" because h2 provides the concept of metadata frame and per connection stream(stream 0) and per business stream (1,3,...).
The PROXY protocol was a pain because I see PROXY protocol as degraded version of "metadata stream": It only allows transmit at beginning, it's require the receiver side aware of the header.
If we extend the PROXY protocol-ish thing we may eventually end up with a subset of h2.

Ok, so right now and for the foreseeable future you only need to transmit data at the beginning of the connection.
for the receive-side-aware argument, if you try to negotiate h2-with-medata and the peer rejects it, is Enovy savvy enough to not send metadata, or does it fail to connect entirely?  I would think it would fail to connect, or send invalid frames at which point both solutions have the same issue where the receive side needs data.
Either way if you really feel stream 0 metadata is the best solution can you do a one pager with suggested APIs?  I still think there's going to be a problem having a general extensible solution to L4 metadata being added and removed from L7 streams cleanly which I think we should sort out before going forward.

Ok, so right now and for the foreseeable future you only need to transmit data at the beginning of the connection.

Thank you for the help to clarify it!

for the receive-side-aware argument, if you try to negotiate h2-with-medata and the peer rejects it, is Enovy savvy enough to not send metadata, or does it fail to connect entirely? I would think it would fail to connect, or send invalid frames at which point both solutions have the same issue where the receive side needs data.

I wish the control plane can deliver if the metadata is expected at the peer to avoid handshake. The handshake adds roundtrip and complexity of state machine. Meanwhile,  either Envoy as client or Envoy as server should not crash (need to verify but I trust our existing code) when unexpected metadata receives.
I have two cases:
In one case the rest of the streams(1,3...) require the metadata, so the filter which consumes received metadata should reset the connection.
In the other case, the rest of the streams(1,3...)  see the metadata as optional, e.g. telemetry metadata.
It will be reflected in the design doc when the doc is ready.

Either way if you really feel stream 0 metadata is the best solution can you do a one pager with suggested APIs? I still think there's going to be a problem having a general extensible solution to L4 metadata being added and removed from L7 streams cleanly which I think we should sort out before going forward.

Yeah... I agree we can either lay the metadata down to L4 protocol. It is proved to be somewhat workable in most of the cases but failed on certain weird and picky L4 firewalls.
I will ping you when the doc is ready.
Thank you all!


Adding Chinese-Simplified localizations

This is a translation made possible by a teacher and his high school student.

Hi @hdcola and @Sichengthebest
Thank you very much for spending the time doing this 🙇
There is another PR #222 that was created before this one, so I'm going to work off of that one for Chinese-Simplified. It is still open, and would appreciated any comments you have on their translations if you have some time.


-n --name flag added

This is an optional parameter you can use to specify which function you want to test with aws-lambda-local.
My use case is that, sometimes I export other functions in the same module (even multiple handlers) for different purposes, so being able to specify which function I want to run would be helpful here.
This new flag is completely optional and it doesnt change the previous behaviour when omitted
// handler.js
module.exports = {
 otherFunction: function() { // whatever otherFunction does },
 handler: function(event, context) { // whatever handler does },
};

PS: If you agree to merge this PR, I can also update the README example with the -n flag if needed.

Hey there!
Thank you for this pull request. I have added --handler|-h CLI command input arguments.
Check the new version here: https://github.com/Max-Kolodezniy/aws-lambda-local/commit/7f3dece6c74790f13a9f1b698eefefab31d187f7


Send the plainTextValue and mentions array arguments to the handleCha…

…nge callback when adding a new mention
Currently these arguments are undefined in my handleChange action, where on keyDown they are not. This means when a user's last input was inserting a mention, it is not realized until they do an additional keyDown action. This change mimics the behavior done on keyDown, and sends the additional function parameters.

Cool, thank you!


Fix resolution of forward refs in dataclass base classes that are not present in the subclass module namespace

Change Summary
(Text mostly copied from #8730) When the base class of a dataclass has an annotation type that is not imported by the module the subclass is in, and from __future__ import annotations is in use (or in my case, Cython is in use), an error occurs.
The root cause is that when pydantic builds up the type namespace, it does not consider base classes (https://github.com/pydantic/pydantic/blob/main/pydantic/_internal/_generate_schema.py#L1474). This is fine for models, because each base class builds a schema immediately, but does not work for plain stdlib dataclasses. This PR adds code to iterate though all of the base classes of the dataclass that are also dataclasses, and push their namespaces as well, which should cover all of the fields which Pydantic will attempt to resolve.
Related issue number
Fixes #8730
Checklist

[x] The pull request title is a good summary of the changes - it will be used in the changelog
[x] Unit tests for the changes exist
[x] Tests pass on CI
[x] Documentation reflects the changes where applicable
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Selected Reviewer: @alexmojaki

The only job failing on CI is the tags check, so please review

All comments have been addressed, please review (looks like hooky doesn't react to rereview requests).

@alexmojaki Just to check, is there anything else I need to do to get this merged?

I've tested out using swapped orders for traversing the MROs, and I think the current approach makes sense. There were test failures for all other options re MRO traversal @dmontagu


[Bug] dbt_project.yml needs Components specified as a issue_field_history_columns vars if customer is using components

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the issue
We recently introduced new logic into jira__daily_issue_field_history that accounts for pulling components data directly into our model to avoid any overlap with field_option.  However, this is currently throwing an error because we did not add the necessary variable logic in dbt_project.yml for when it isn't present inside issue_field_history.
Adding
vars:
     jira:
      issue_field_history_columns:
          - "Components"

is necessary.
However, we will want this logic to compile only if the customer is using the components table. We might recommend using {% if var('jira_using_components', True) %} above issue_field_history_columns since we wouldn't want to call it if components wasn't a part of the model.  See examples in the newest version of jira__daily_issue_field_history.
Relevant error log or model output
00:11:01  Database Error in model jira__daily_issue_field_history (models/jira__daily_issue_field_history.sql)
00:11:01    Name components not found inside joined at [101:65]
00:11:01    compiled Code at target/run/jira/models/jira__daily_issue_field_history.sql

Expected behavior
The models should compile but it is not finding the components column inside daily_issue_field_history if it is not set as a field.
dbt Project configurations
vars:
     jira:
      issue_field_history_columns:
          - "Components"

Package versions
jira 0.12.1
What database are you using dbt with?
bigquery
dbt Version
dbt=1.3.0
Additional Context
No response
Are you willing to open a PR to help address this issue?

[X] Yes.
[ ] Yes, but I will need assistance and will schedule time during our office hours for guidance
[ ] No.

Thanks for opening this issue @fivetran-avinash.
As you mentioned in the issue description, I believe replacing the following lines with the same for/conditional logic that you used here should resolve the issue and ensure the logic only runs if customers have included components in their issue_file_history_columns variable:

https://github.com/fivetran/dbt_jira/blob/105300644f5b903172f9255ad1b0e9baf1c71e95/models/jira__daily_issue_field_history.sql#L132
https://github.com/fivetran/dbt_jira/blob/105300644f5b903172f9255ad1b0e9baf1c71e95/models/jira__daily_issue_field_history.sql#L150
https://github.com/fivetran/dbt_jira/blob/105300644f5b903172f9255ad1b0e9baf1c71e95/models/jira__daily_issue_field_history.sql#L169

This has been addressed in the merged rollback linked above. As such, I will close this issue.


Investigate using semantic-release

Semantic Release is a project for automating package publishing. It's a nice tool to not miss to publish a new version once new changes are in and I think it would fit very well to have automated via aegir.
It does a bunch of things, maybe even too much. If so, we could maybe just integrate the publishing part while leaving out the rest, but in general, it would be nice to try to automate the publishing so in the future, it'll be easy to integrate with the CI system.
https://github.com/semantic-release/semantic-release

I have looked at semantic-release before, but it does quite a lot and sometimes in a way that is not the way we would like to do it. I am curious which part of it you are missing, as integrating individual functionality is much cheaper in terms of speed and dependency size and much easier development wise as the core tool was not particular modular when I checked last time.
On 30. Oct 2017, 16:22 +0100, ᴠɪᴄᴛᴏʀ ʙᴊᴇʟᴋʜᴏʟᴍ<EMAIL_ADDRESS>wrote:

Semantic Release is a project for automating package publishing. It's a nice tool to not miss to publish a new version once new changes are in and I think it would fit very well to have automated via aegir.
It does a bunch of things, maybe even too much. If so, we could maybe just integrate the publishing part while leaving out the rest, but in general, it would be nice to try to automate the publishing so in the future, it'll be easy to integrate with the CI system.
https://github.com/semantic-release/semantic-release
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Something less automatic than Semantic Release would be Standard Version. From what I can tell it does almost exactly what aegir release is currently doing.

I've used semantic release on previous projects, it works well.


Pakt categories disappear when using tab

TODO:
Vanilla JS to add toggle className  open/close

only one is open

If I navigate via tab through the category headlines (e.g. "Bundesländer") they disappear and are replaced with an item from the category

Workaround or dirty bugfix with overflow hidden...


[staging] meson: fix build on darwin

Motivation for this change
Update fix-objc-linking.patch patch for meson 0.51.2
Things done

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

[X] NixOS
[X] macOS
[ ] other Linux distributions

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

Notify maintainers
cc @

This was done in https://github.com/NixOS/nixpkgs/pull/68105 (or some equivalent, I think)

I now merged staging-next to staging, so it's present in there as well.


beta4 is moving all API endpoints to /api/

https://github.com/blakeblackshear/frigate/pull/496

fixed this in a separate PR

fixed this in a separate PR


Product ID and F9R compatibility

Hi,
I'm trying your branch F9R and got issues with a custom product ID to differentiate from F9P.
I enforce it in the USB header but it doesn't want it and tell me "usb device not found".
Is there any restriction with the libusb about that ?
I got two udev rules to redirect F9R and F9P before that.
Could you add a parameter to change the productid ?
Thanks.

Presently it just detects the first device found using libusb ... have been thinking about a way to differentiate. I'm not sure of the correct approach as yet. Someone else wanted to be able to connect 3 x F9Ps to one machine. There is also the issue of the fact that there is different capabilities between the different models. Some messages should be turned off if they arent applicable to the model connected.

I also want to connect 2 to 3 F9 devices ... we will make a club :)
Would it be better to switch to /dev/tty mount ?

Tom, the current code looks for devices that match the USB Vendor ID (0x1546) and Product ID (0x01a9) values for the ZED-F9R.  As mentioned, it just detects the first device that matches these using libusb.
These USB ID's are defined on the device itself - using udev rules to change where the device is mounted doesn't change these values.   Switching to /dev/tty* likewise will not change the ID's.
Note that ublox appears to use the same product ID across multiple models of GPS.  As far as I can see, the ZED-F9P, ZED-F9R, and M8 all use 0x01a9 for their product ID.
You can actually change the USB ProductID via the CFG-USB configuration settings - see screenshot below from u-centre.  If you were desperate, you could modify these on your devices and add the product ID as a parameter (look here in the code).
I'm not sure this is a wise thing to do though, as it may have other unintended consequences.  There is however a 24-byte serial number string available which you can program to whatever you like.  Getting the ublox-dgnss driver to optionally find the matching device using this may be a better approach.

Hello there,
Is this code functional on a Ardusimple simpleRTK2B receiver?
https://www.ardusimple.com/product/simplertk2b-basic-starter-kit-ip65/
I've just bought it and doesn't support humble..
thanks!

@miguelao33 I don't have one of those devices. If its configurable via USB through ublox u-center it should work. There was a humble release, however the present driver version is primarily targeted now for ROS2 Iron

@miguelao33 we have an Ardusimple RTK2B working without problems on Humble.  What issues have you encountered?  Have you set a udev rule as mentioned in the readme?  Can you access and configure the device over USB using u-center2?

Hello Geoff,
I had no problem. I just received the ardusimple board and it works at the
moment I connected with your Code perfectly.
The only issue was having a rule applied to the same device (old test),
then the launch file crash and give me the error "cannot connect to usb".
Thanks!
El lun, 4 sept 2023, 18:26, Geoff Sokoll @.***>
escribió:

@miguelao33 https://github.com/miguelao33 we have an Ardusimple RTK2B
working without problems on Humble. What issues have you encountered? Have
you set a udev rule as mentioned in the readme? Can you access and
configure the device over USB using u-center2?
—
Reply to this email directly, view it on GitHub
https://github.com/aussierobots/ublox_dgnss/issues/5#issuecomment-1705512673,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BAWAGB6GMGAPQCOSPZSZH7LXYX6L3ANCNFSM6AAAAAAWGUXFAM
.
You are receiving this because you were mentioned.Message ID:
@.***>


Transfer labels to ERG repo

From @lfunkhouser on October 2, 2017 15:24
This issue is used to transfer issues to another repo
Copied from original issue: Esri/solutions-grg-widget#119

This issue was moved to ArcGIS/solutions-erg-widget#8


  This issue tracks the list, and any blockers that we need to do first.

[x] react-router-dom -- update to v6 blocked by converting the app to use functional components.  V6 requires hooks, so rather than wrapping every class-based component in a HOC, it is easier to just convert, then update the router version.  Once we update to functional components, it is easy to migrate from <Switch> to <Route> and the V6 useParams() hook.
[x] react-scripts (CRA )- update to v5 blocked by this CRA issue.  TL;DR - v5 implies Webpack v5, which no longer polyfills node API (eg, fs).  We have at least one package (d3-dag, which imports javascript-lp-solver) which somewhat carelessly imports fs and client_process.  The aforementioned GH issue has several work-arounds, but all are gross [an example].
[ ] react and react-dom v18
[x] Blueprint 4.X

Deferring React. Remainder are done.  See #29 and #30.


only-valid attribute is Not taking value From Any Variable

As I am trying to select date from particular Interval of days ,  It is working with  hard coded value to only-valid attribute And not with the value in variable. My code is as follows
woking==>

Not working==>

Hello! While we appreciate you opening an issue with us i am going to close this issue and ask that you open a new one that has the issue template filled out please. Thanks for using Ionic!


Fix memory leak bug

Thank you for the fix!
Could you rebase your work on the master branch and remove your contribution comment. I was thinking on adding contribution the way moveit_msgs does.

What would be even better is to wrap the shapes::createMeshFromResource into a unique ptr.

What would be even better is to wrap the shapes::createMeshFromResource into a unique ptr.

Sorry I screwed up the changes in your master branch. Thx for the fix.


fix: prioritize suite params over config

fix #297
We now deep merge the params passed from both config and suite params and always prioritize suite params over config params when a key is specified in both.

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: Pull request #298 opened

Start Time: 2021-06-03T19:49:22.331+0000

Duration: 13 min 52 sec

Commit: c23867f46938f450eea6a8c4ebbe55a5cc06e37b

Test stats :test_tube:

Test
Results

Failed
0

Passed
103

Skipped
0

Total
103

Trends :test_tube:


net: getaddrinfo: Query both IPv4 and IPv6 if hints is missing

We must query both IPv4 and IPv6 addresses if the hints parameter
is NULL i.e., user does not supply hints.
Fixes #16453
Signed-off-by: Jukka Rissanen<EMAIL_ADDRESS>

net: getaddrinfo: Query both IPv4 and IPv6 if hints is missing

Nitpick about commit message: fairly speaking, the matter should be that IPv4 and IPv6 should be both queried if hints has AF_UNSPEC address family (and that's default if hints is NULL at all).

Updated according to comments.

Removing platform_whitelist was a bit too brave and hits the same issue patch from @andrewboie hit earlier. Better to skip that change for now I guess.

I have been removing the whitelists in various PRs. Created an issue yesterday for this one #16778

Rebased against latest master.

As this no longer contains any changes not described in the topic of the PR, I'd suggest to merge it. (I guess @tbursztyka is on vacation, nor this is his known area of interest.)

recheck


PLP: Padding on the left disappears when filter is applied

Environment Details
Spartacus: develop branch
Browser: Chrome
Device: mac
Happens on: https://dev-com-2.accdemo.b2c.ydev.hybris.com:4200/electronics-spa/en/USD
Steps to Reproduce
viewport 1010x710
Search for "canon"
Apply filter:
store "Fukuoka Canal City Fukuoka Washington Hotel (44)"
Actual Result
Once the filter is applied, the padding on the left disappears.

Tested on dev17. Padding is fixed now.


make compute_psd work on a BaseEpochs instance

BaseEpochs can appear from the code base eg when concatenating Epochs.

I had no idea that BaseEpochs could occur in the wild.  To me it the right fix is to make concatenation return Epochs instead of BaseEpochs.  Is that not an option?

Or maybe EpochsArray? I agree it's weird to have it return BaseEpochs. But maybe we should do that in another PR

we agreed during meeting today that we should use EpochsArray
Dan accepted to have a look.
Let's not merge this one for now.

Message ID: @.***>

Coming back to this and thinking about it more -- internally I'm not sure why we wouldn't ever just check for BaseEpochs. If we ever add a new read_epochs_* that returns a EpochsSomething class that inherits from BaseEpochs and implements the right _get_raw_from_disk or whatever, it should work. For example, we already have this pattern with read_epochs_eeglab, which returns a EpochsEEGLAB that only inherits from BaseEpochs -- which seems like the right pattern to me -- and the epochs.compute_psd should be made to work with this object. If it works for this one, I'm not sure why it wouldn't work for anything that inherits from BaseEpochs...
@drammock could we check to see if it's BaseEpochs and then iterate over the object? Or if we require preloading just do _check_preload then use ._data?

Ohh I see, it should already work for anything that inherts from BaseEpochs but not a bare BaseEpochs instance itself. In that case, maybe we don't need this after #11309. Let's try without it, and resurrect this if needed...

Ohh I see, it should already work for anything that inherts from BaseEpochs but not a bare BaseEpochs instance itself.

Yep exactly!  so #11309 should be enough.


INSTALLATION FAILED

Error: INSTALLATION FAILED: unable to build kubernetes objects from release manifest: error validating "": error validating data: [unknown object type "nil" in ConfigMap.data.APP_API_URL, unknown object type "nil" in ConfigMap.data.CONSOLE_API_URL]
k8s version : 1.23

We could not replicate the issue with default Values.yaml. Please specify the Values.yaml you've applied and the installation command as well.

I have the same problem but don't know what the cause is

I have the same problem but don't know what the cause is

@TruongNguyen5395 Please submit a new issue with steps to replicate.


Write a implementation of GPA Calculator

Create a program to calculate GPA from grades of a student on the scale of 10.
Sample Input: "AAA+BCADE"
Sample Output: 7.87
Here:  A+ represents 10, A represents 9, B represents 8 and son on till E representing 5.

Hello, I can take this up. Are you assigning the issues?

Yes,you can!
On Wed, 30 Sep, 2020, 11:11 pm Nachiket Dhamankar<EMAIL_ADDRESS>wrote:

Hello, I can take this up. Are you assigning the issues?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Iltwats/hacktoberfest/issues/11#issuecomment-701540558,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AM5BX6B3L6CQ5UUTH4TH3ADSINUWNANCNFSM4R7FK2CQ
.

--
www.jecrcuniversity.edu.in http://www.jecrcuniversity.edu.in
ENGINEERING | SCIENCES | MANAGEMENT | HUMANITIES | LAW | AGRICULTURE |
HOTEL MANAGEMENT | JOURNALISM MASS COMMUNICATION | DESIGN |
*
*Important:
The information in this email is confidential and may be legally
privileged, It is intended solely for the addressee(s). Access to this
email by anyone else is unauthorized. If you are not the intended
recipient, any disclosure, copying, distribution is prohibited and may be
unlawful. If you are not the intended recipient, please contact the sender
by reply email and destroy all copies of the original message. Thank you,
JECRC University - Disclaimer
*"SAVE PAPER.  Good for your planet. Good for
your Business" *

I would like to work on this issue under hacktoberfest 2021 in C++

@spyd3r-byte Assigned

I would like to work on this issue under hacktoberfest 2021 .Can you assign it to me?

please approve #231


question: Specific resource to be found by custom check in tfsec

Can we write a custom rule to check if the specific resource(we know the name and specifications) is violating a rule?
For example, In custom check file we usually define required types(let say resources in our case) and required labels (let say aws_instance), so this custom rule would check for all resources which has label aws instance and but if we want to be more specific, we even know the name of that specific resource we will check this rule for.
Can we add that as well and only get the results for that specific resource and not all resources of this type?

That's right at the moment it is not supported at that granularity. The most we can do is by resource type as you know.


Fix underage accounts using password auth and xbl.verifyTokens logic

Allow password auth to gen XSTS tokens for accounts <18 years old
doSisuAuth now caches userToken and doesn't gen new tokens every run, resolves #17
Valid userTokens will now be used to gen new XSTS tokens instead of creating another userToken
Removed dependency on xboxlive-auth outside of doReplayAuth
Combine getXSTSToken and getXSTSTokenWithTitle as they both achieve the same thing

@extremeheat Could you have another look at this please?

This appears to be a breaking change and does several things at once. Going to need to review again at some point

Can we split this into multiple PRs? That will make this easier to understand/review what code changes correspond to what intentions of the PR


feat(log):add zap library and logging capabilities

fix write bug, mv don't need to delete temp file.
add some unit tests.
fix fs open bug, check and auto create dir.
refactoring space, add mutex.

/lgtm
/approve


What's the max allowed value of seriesCountLimit and valuesPerDimensionLimit?

I'm using MetricConfiguration to get metrics and track values. But it has two limits, seriesCountLimit and valuesPerDimensionLimit. We do have a vast number of dimensions and distinct values for every dimension. I want to know the upper limit of the two properties.
MetricConfiguration MetricConfig = new(seriesCountLimit: 100000, valuesPerDimensionLimit: 500, new MetricSeriesConfigurationForMeasurement(restrictToUInt32Values: false));

@zhangyazhong, did you ever find the answer to this?


Fix markdown math brackets render problem (#31371)

close #31371
compare
original text:
$i = \sqrt{-1}$ ($i = \sqrt{-1}$)

$i = \sqrt{-1}$ [$i = \sqrt{-1}$]

$i = \sqrt{-1}$ {$i = \sqrt{-1}$}

$abcd

(this is a test)

[$test]

$test$

$a$ ($b$) [$c$] {$d$}

before fix:

after fix:

Thank you for the fix!
ps: I will edit the PR title to remove the (# xxx) part: that part is only use for backporting in Gitea.


IE8 issue

$layoutRightPane.splitPane('lastComponentSize', 240)
There is an error :

function setLeft(firstComponent, divider, lastComponent, left) {
	firstComponent.style.width = left;

//Invalid argument.  left is NANpx
divider.style.left = left;
lastComponent.style.left = left;
}

The issue has been fixed in 0.7.1.


Move W3CTraceContextPropagator to API package

Which problem is this PR solving?
The propagator for W3C-format traces is present in the SDK portion of this package which does not conform to the OpenTelemetry specification.
Short description of the change
This PR moves W3CTracePropagator to the API portion of this package to conform to the OpenTelemetry specification.
How Has This Been Tested?
Unit tests demonstrate that these changes operate correctly.
Checklist:

[ ] Unit tests have been added
[ ] Documentation has been updated

Security Insights
No security relevant content was detected by automated scans.
Action Items

Review PR for security impact; comment "security review required" if needed or unsure
Verify aviary.yaml coverage of security relevant code

Questions or Comments? Reach out on Slack: #support-infosec.

Public API Changes
Recommendation: :bangbang: Major version bump (fyi @Workiva/semver-audit-group )
@@ line 10: package:opentelemetry/src/sdk/trace/propagation/w3c_trace_context_propagator.dart @@
-  class W3CTraceContextPropagator implements TextMapPropagator<dynamic>
// Removing a class is a major change.

-     void inject(Context context, dynamic carrier, TextMapSetter<dynamic> setter)
//    Removing a method is a major change.

-     W3CTraceContextPropagator W3CTraceContextPropagator()
//    Removing a constructor is a major change.

-     static RegExp get traceParentHeaderRegEx
//    Removing a field is a major change.

-     Context extract(Context context, dynamic carrier, TextMapGetter<dynamic> getter)
//    Removing a method is a major change.

      From dart:core/object.dart
-     Type get runtimeType
//    Removing a field is a major change.
-     bool ==(Object other)
//    Removing a method is a major change.
-     int get hashCode
//    Removing a field is a major change.
-     dynamic noSuchMethod(Invocation invocation)
//    Removing a method is a major change.
-     String toString()
//    Removing a method is a major change.

@@ line 10: package:opentelemetry/src/api/propagation/w3c_trace_context_propagator.dart @@
+  class W3CTraceContextPropagator implements TextMapPropagator<dynamic>
// Adding a class is a minor change.

Showing results for 9ef59433973f1068a1f92536156c17f327407796

Powered by semver-audit-service. Please report any problems by filing an issue.
Reported by the dart semver audit client 2.2.2
Browse public API.

@evanweible-wf This is in a better place and ready for your review.  Thanks!

Unit tests show these changes working properly.
QA +1

@Workiva/release-management-p


named plugin and consumer tag

Fixes: https://github.com/platformatic/rabbitmq-hooks/issues/39
Also, saves the consumer tag for a clean close.

The one to name is https://github.com/platformatic/rabbitmq-hooks/blob/4aaafe6229f37d4aff44ad5faee785e57e08b80c/index.js#L9.

The one to name is
https://github.com/platformatic/rabbitmq-hooks/blob/4aaafe6229f37d4aff44ad5faee785e57e08b80c/index.js#L9
.

Right @mcollina . Done, PTAL


.variance() [and therefore .sd()] calculated using N instead of N-1 [Bessel's correction]

Given that most likely uses of these particular functions (which operate on the data column) are to calculate the variance/sd of samples, you may wish to change it to being N-1 as a default or at least add the option.

Great point. Thanks for bringing it up.


[5.3] Improve Interacts with database testing methods

Right now it's a little bit annoying when seeInDatabase fails because there is not enough feedback. On the other hand, when it passes sometimes you want to grab the results and right now the only way is to execute a different query. This PR attempts to fix these 2 problems and improve the code as well.
With this upgrade you get more feedback:
Failed asserting that a row in the table "users" matches the attributes {"username":"Silence","first_name":"Duilio","last_name":"Palacios"}.

Found instead: [{"id":6,"username":"","first_name":"Duilio","last_name":"Palacios","email":"admin@styde.net","member":0,"remember_token":null,"created_at":"2016-10-09 15:52:54","updated_at":"2016-10-09 15:52:54"}].

You can also use seeAndGetInDatabase (Couldn't think of a better name)
So instead of this:
        $token = $this->seeInDatabase('tokens', [
            'email' =><EMAIL_ADDRESS>        ]);

        $token = Token::where('email', 'admin@styde.net')->first() //or Token::first();

You can use:
        $token = $this->seeAndGetInDatabase('tokens', [
            'email' =><EMAIL_ADDRESS>        ]);

I'd be happy to read some feedback or suggestions for other methods or improvements.

No plans on implementing this in the core.


TokenScript error show in dialog

Fixes #1449

Output of current implementation

Almost works 100%. Just a couple of issues I found:

When there's an error, the token name doesn't get blanked, so RecyclerView leaves the token name from the previous view:

When I click on the error, the leading text doesn't make sense within the context (TokenScript overriden by .....), use something like 'TokenScript Error: ':

@JamesSmartCell Feedback applied. Could you please check.


Playback speed

If the feature request is for a plugin or provider, specify it here.
YouTube videos
If the feature solves a problem you have, specify it here.
No response
Describe the proposed feature.
It's a very nice player but there are a small feature missing for YouTube player.
Please add playback speed features.
Describe alternatives you've considered
No response
Additional context
No response
Related Issues
No response

Hello @Sohanlalkush, thanks for the feature request!
Can you clarify in what context you need this feature? In my opinion, this does not seem very useful.
The idea of Vlite is also to stay focused on the core features, this is what makes it extremely light.

Closing due to inactivity. Thanks! Feel free to send a PR.


[9.3.0] Issue happening again "the following content is not available on this app".

Steps to reproduce
const yt = await ytjs.Innertube.create({ });
let info = await yt.getBasicInfo(id, "ANDROID");
Failure Logs
None

Expected behavior
Getting correct formats url.
Current behavior
Fall back to a default video.
Version
Default
Anything else?
ANDROID Client stoped working properly,  it showing "the following content is not available on this app" again from last 2 days.
Checklist

[X] I am running the latest version.
[X] I checked the documentation and found no answer.
[X] I have searched the existing issues and made sure this is not a duplicate.
[X] I have provided sufficient information.

same issue i meet, has any update?

anyone who can solve the issue?

Is this related? https://support.google.com/youtube/thread/269521462?hl=en

Yes it is. Also, if anyone still insists in using the android client, using an account bypasses this error.


Karabiner 10.9.0

Thank you for the contribution. It was merged directly as 6147e5b7f79652c670d8d9aa0eaad231090ae1bd to keep commit history cleaner. Your contribution is still credited to you.


[FR]: Able to specify mappings for two external classes

I have use case where I get two DTOs from separate libraries. Which I need to map to each other. I can't add annotation to either class. So a ability to specify a bunch of mappings between two types without annotations would be perfect.

This can probably be solved with @Konverter


Proposal: Change format of /v1/apps/<app id>/run

Currently run returns a dynamic array (Ewww!) [0, "hi\n"]. The docs said that run returned {"rc": 0, "output": "hi"} which seems much more sensible to me.
I would like propose to change the response of run to the format suggested to the docs:
{
  "rc": <return code>,
  "output": "<command output>"
}

This is a backwards incompatible change, so this would probably warrant a API version bump to version 2.
The good news is that for as long as we have had API docs nobody has caught this huge difference, so it seems unlikely that many people are consuming that part of the api.

ref #3983

Moved issue to deis/workflow#59, since this would be a V2 change.


Contracts: gateway and transfers

🚫 Not to be considered until after #32 is merged.
This PR:

adds gateway setter (resolves #9);
adds gateway set event (resolves #34)
completes transfer functions (resolves #6)
whitelists transferors (resolves #10)
adds whitelist event (resolves #35)
requires gateway to be set to accept a stake request (resolve #36)

✅ tested
✅ linted
✅ changelogged
Diff against branching point: https://github.com/jasonklein/openst-tokens/compare/jasonbanks/gh17/accept-stake-request...jasonklein:jasonbanks/gh9/gateway

Closing in favor of omnibus PR #48.