[BUG] The Microsoft Stream Tile is not appearing in the solution

Describe the bug
In the following video: https://youtu.be/-sLXl_rKGxQ?t=651   You can clearly see a Microsoft Stream tile for learning about Microsoft Stream.  This tile is not visible in the solution when it has been deployed.
To Reproduce
Steps to reproduce the behavior:

Go to https://tenantURL.sharepoint.com/sites/Microsoft365/SitePages/CustomLearningViewer.aspx
Scroll down to Products
See that Microsoft Stream is missing

Expected behavior
Microsoft Stream tile should appear as shown in the video
Screenshots
https://myitservices.adventhealth.com/wp-content/uploads/2020/02/Missing-Microsoft-Stream-Tile.png
Learning Pathways version number:
3.0.4
Additional context
Add any other context about the problem here.

Can you verify you didn't accidentally hide Stream in the Admin web part?

Good Afternoon,
Yes I can verify it is not in the list of products.  https://myitservices.adventhealth.com/wp-content/uploads/2020/02/Microsoft-Stream-Missing-in-List-of-Products.png
<EMAIL_ADDRESS>Eric Davis
AdventHealth
Systems Administrator, Enterprise Collaboration Services | AIT Technology Services
407-543-5826
From: Julie Turner<EMAIL_ADDRESS>Sent: Friday, February 14, 2020 10:26 AM
To: pnp/custom-learning-office-365<EMAIL_ADDRESS>Cc: Davis, Eric<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: [EXTERNAL] Re: [pnp/custom-learning-office-365] [BUG] The Microsoft Stream Tile is not appearing in the solution (#207)
Can you verify you didn't accidentally hide Stream in the Admin web part?
[image]https://user-images.githubusercontent.com/7570936/74544008-63c4bf00-4f14-11ea-9a0a-abdc1d8d4318.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/pnp/custom-learning-office-365/issues/207?email_source=notifications&email_token=AMK5ODO4ACD6J5QB6QFTZFDRC2ZY5A5CNFSM4KVJVRK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELZMJPQ#issuecomment-586335422, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMK5ODL25PHOEMD3NREPGILRC2ZY5ANCNFSM4KVJVRKQ.
This message (including any attachments) is intended only for the use of the individual or entity to which it is addressed and may contain information that is non-public, proprietary, privileged, confidential, and exempt from disclosure under applicable law or may constitute as attorney work product. If you are not the intended recipient, you are hereby notified that any use, dissemination, distribution, or copying of this communication is strictly prohibited. If you have received this communication in error, notify us immediately by telephone and (i) destroy this message if a facsimile or (ii) delete this message immediately if this is an electronic communication. Thank you.

If you're absolutely sure you're on version 3.0.4 of the web part (meaning you updated the web part after I released it yesterday?) then you clearly have some type of problem getting the updated content manifests from GitHub.  Please check out the network tab of the browsers developer console and verify that you're getting a valid response from the metadata.json, playlists.json, and assets.json calls when loading the admin page.

Good Afternoon,
I am quite sure I am running 3.0.4 (see screenshot below).
The metadata.json, playlists.json, and assets.json all responded with 200 OK.  See the attached .har file exported from the Network tab in the following link.  https://myitservices.adventhealth.com/wp-content/uploads/2020/02/Custom-Learning-Admin-Page-Network-Tab-Results.zip
<EMAIL_ADDRESS>Eric Davis
AdventHealth
Systems Administrator, Enterprise Collaboration Services | AIT Technology Services
407-543-5826
From: Julie Turner<EMAIL_ADDRESS>Sent: Friday, February 14, 2020 1:06 PM
To: pnp/custom-learning-office-365<EMAIL_ADDRESS>Cc: Davis, Eric<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: [EXTERNAL] Re: [pnp/custom-learning-office-365] [BUG] The Microsoft Stream Tile is not appearing in the solution (#207)
If you're absolutely sure you're on version 3.0.4 of the web part (meaning you updated the web part after I released it yesterday?) then you clearly have some type of problem getting the updated content manifests from GitHub. Please check out the network tab of the browsers developer console and verify that you're getting a valid response from the metadata.json, playlists.json, and assets.json calls when loading the admin page.
[image]https://user-images.githubusercontent.com/7570936/74555828-95e11b80-4f2a-11ea-9bcc-2990cfcc052c.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/pnp/custom-learning-office-365/issues/207?email_source=notifications&email_token=AMK5ODN4QLGVGUNKBNGLIU3RC3MO5A5CNFSM4KVJVRK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELZ43HY#issuecomment-586403231, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMK5ODOBYAJ4GLZXGYXQJFTRC3MO5ANCNFSM4KVJVRKQ.
This message (including any attachments) is intended only for the use of the individual or entity to which it is addressed and may contain information that is non-public, proprietary, privileged, confidential, and exempt from disclosure under applicable law or may constitute as attorney work product. If you are not the intended recipient, you are hereby notified that any use, dissemination, distribution, or copying of this communication is strictly prohibited. If you have received this communication in error, notify us immediately by telephone and (i) destroy this message if a facsimile or (ii) delete this message immediately if this is an electronic communication. Thank you.

First, let me remind you that you're exposing your PII when you respond via email. You might want to remove information from the email or respond via the web client.
Because you responded via email I can't see the screenshot, so you'd have to paste that into the browser, but, that said the HAR file shows that you're downloading the manifest from the right endpoint so I can confirm you're using version 3.
All this said I figured it out by going back. I'm pointing at a staging CDN. I didn't remember I was doing that when recording the videos but regardless that content has not been added to the public endpoints although I thought they had. I will research more and see if I can find out why and when that might happen.

Thank you for helping me find the root cause.  I appreciate your assistance.  Looking forward to any updates you may be able to share after you learn more.
From: Julie Turner<EMAIL_ADDRESS>Sent: Friday, February 14, 2020 2:29 PM
To: pnp/custom-learning-office-365<EMAIL_ADDRESS>Cc: Davis, Eric<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: [EXTERNAL] Re: [pnp/custom-learning-office-365] [BUG] The Microsoft Stream Tile is not appearing in the solution (#207)
First, let me remind you that you're exposing your PII when you respond via email. You might want to remove information from the email or respond via the web client.
Because you responded via email I can't see the screenshot, so you'd have to paste that into the browser, but, that said the HAR file shows that you're downloading the manifest from the right endpoint so I can confirm you're using version 3.
All this said I figured it out by going back. I'm pointing at a staging CDN. I didn't remember I was doing that when recording the videos but regardless that content has not been added to the public endpoints although I thought they had. I will research more and see if I can find out why and when that might happen.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/pnp/custom-learning-office-365/issues/207?email_source=notifications&email_token=AMK5ODIFVCWE2GDX6DTDP5LRC3WJRA5CNFSM4KVJVRK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEL2FX4Q#issuecomment-586439666, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMK5ODJQWMQMLHVBC5J5SCLRC3WJRANCNFSM4KVJVRKQ.
This message (including any attachments) is intended only for the use of the individual or entity to which it is addressed and may contain information that is non-public, proprietary, privileged, confidential, and exempt from disclosure under applicable law or may constitute as attorney work product. If you are not the intended recipient, you are hereby notified that any use, dissemination, distribution, or copying of this communication is strictly prohibited. If you have received this communication in error, notify us immediately by telephone and (i) destroy this message if a facsimile or (ii) delete this message immediately if this is an electronic communication. Thank you.

Ok, so the long and short of this is that yes I'm pointing at staging but the reason those categories aren't in production is that there's some issues with the docs site and the way it opens videos/URLs.  They're working on the fix and hopefully, that content will be pushed out soon.


DBZ-5591 Snapshot process setting source.ts_ms

Related https://github.com/debezium/debezium/pull/3855
https://issues.redhat.com/browse/DBZ-5591

Applied to main (=2.0) and branch 1.9.


Using an operator to deploy a complex microservices system

Type of question

General operator-related help
Question

Is it recommended to use operator framework to deploy an entire complex system?

I was going through some docs and Youtube videos about the purpose of k8s operators. Most of them says that perfect use case would be to deploy and operate a stateful application such as database or the monitoring system.
I wonder if the operator pattern can be applied to a complex web system which is composed of microservices, but I couldn't find any resources on the web about such approach.
Let's suppose that there are few applications in a system. Some of them depends on databases and message queues, may be exposed via ingress and there may be a monitoring system like Prometheus with Grafana plus ELK stack. In general, there are many k8s resources to manage, and the deployment process is complex.
I can imagine an operator which will install all the monitoring systems and databases first and then deploy stateless applications to a Kubernetes cluster. Hence, the operator will automate the system deployment process.
Questions:

Is this a proper way of using an operator pattern?
Is it possible, and recommended, to deploy the entire system using one operator and then manage stateful applications using dedicated operators? For example, I deploy the entire system with an operator, then I want to manage/scale a previously deployed database using another operator, assuming that the first operator was using database CRDs.
If this approach is implemented, is it possible to deploy/update a single application in a system without the need of calling other services?
Should I use operator framework for rather very complex systems or simple ones?

@piotrpersona Thanks for bringing up this issue. Answering the questions inline:
As you mentioned the idea of operators is to automate stateful application or even a bunch of such applications co-existing in an ecosystem.  Fortunately, operator framework does have a few projects that can support your use-case.

Imo, this system can be considered to be a use case to follow operator pattern. Reason being, each independent application's lifecycle is to be automated, and they are "stateful".
We can have each of microservices to be managed by an operator, and then have Operator Lifecycle Manager (https://olm.operatorframework.io) to manage the overall lifecycle of each of them. This way, we have OLM to manage the independent lifecycle of each of them, their upgrades and dependencies.
We do have a bunch of users who have multiple operators installed through OLM. Here are some examples, (though these may/maynot be exactly similar: https://operatorhub.io)
Operator Framework does provide a set of tools to make this easier. We have:
Operator SDK - tool to build your operator for each micro service.
Operator Lifecycle Manager - tool to manage those multiple operators.
Operator Package Manager - tool to package the operator content into bundle and send it to OLM.
The tools can serve multiple use-cases, whether a simple one - dealing with one operator, or a complex system - dealing with multiple microservices.

I've heard of very complex applications that use "top level" operator that uses APIs provided by other operators.
This makes sense and (IMO) fits with Kubernetes design principles. For instance, you might create a Deployment, and the deployment controller will create replica sets, which are then managed by the ReplicaSets controller. Operators offer a nice way of abstracting out the complexity of the smaller parts, and can simplify the whole system in that way.
One caveat is that each operator is another process that must always run on the cluster, so it might consume more resources when done this way.
Also, be very sure to keep the abstractions separate. Having multiple controllers managing a single resource is definitely an anti-pattern that causes all kinds of pain like infinite reconciliation.

@varshaprasad96 @asmacdo Thank you for response.
I think it's clear that operator framework with OLM would fit into my system's design.


Refactor validate signature.

Reorder code.
Avoid signature validation error when DS namespace declared at the root (don't remove signature element).

We are also running into the issue this resolves.  Any idea when it will be released to rubygems?

:+1: ship it

:+1:  sweetness!


[clinical-item-collection] update to consume the terra-icon upgrades

Summary
Update ItemCollectionExample.jsx for documentation purposes
This change was tested using:

[ ] WDIO
[x] Jest
[x] Visual testing (please attach a screenshot or recording)
[ ] Other (please describe below)
[ ] No tests are needed

Reviews
In addition to engineering reviews, this PR needs:

[X] UX review
[X] Accessibility review
[ ] Functional review

This PR resolves:
[UXPLATFORM-8888]

Thank you for contributing to Terra.
@cerner/terra

clinical-item-collection doc package has been deprecated with https://github.com/cerner/terra-clinical/pull/849
This PR will be closed and the changes will not be merged.


Use will-change: transform in elements with position: fixed

Doing so prevents unnecessary repaints as explained here:
https://www.youtube.com/watch?v=2vFrZXWiwIc

We've already tried applying transform: translate3d(0,0,0), which FWICT is an approximate moral equivalent of will-change: transform, and ended up having to revert it because of (a) Safari bugs (b) it creates a new stacking context, which is often undesirable.
See https://github.com/twbs/bootstrap/pull/14667 and https://github.com/twbs/bootstrap/pull/13649#issuecomment-57767763
So I don't think we'd be terribly eager to try this.

According to mdn (https://developer.mozilla.org/en/docs/Web/CSS/will-change#Values) it creates a new layer or a new stack, didn't know that..
(my navbar is very simple, so I didn't run into anything strange, but I can see the problems that it can cause to other people).
Ok then.


Add/edit identity file through web interface

Improvement

can you expand a bit? ssh keys? adding/editing them?

Sorry. I meant identity file path. I updated the title.

It should be possible with the new release. can you update it and try again?

I see. Can you update the version on brew?
https://github.com/Homebrew/homebrew-core/blob/master/Formula/stormssh.rb

@youtux, I belive you maintain the homebrew package of stormssh. Can you update it? Relevant tag: 0.6.9
If that's not the case, sorry for pinging you :)

@emre Sorry for the late response, I will be happy to update it, if nobody already did it.

thanks 🤗

See https://github.com/Homebrew/homebrew-core/pull/3394

thank you! :)


docs:(telemetry-utils): Add explicit release tags to exported API members and fill in / improve existing docs

Removes the rule override in the api-extractor config. Fills in explicit release tags for exported members per our common rules.
Also makes some improvements to existing documentation patterns to be more user-friendly in the IDE and in the generated docs using our TSDoc guidelines.
Finally, extracts an inline props interface from createMultiSinkLogger into its own interface to better encapsulate parameter documentation.
AB#5907

@markfields Our eslint configuration requires summary comment docs for exported API members. I filled in some missing docs in places, which could use review. Additionally, I left some TODOs where I wasn't sure how to document the semantics. Hoping you (or someone) can offer some guidance there.


Use a new _ParametricGeometry for specializations

Changes

Implements an internal _ParametricGeometry <: Meshes.Geometry type to serve as a simple wrapper for custom parametric functions.
Uses this new geometry type to replace specialization methods that used hard-coded domain transformations and analytic methods. The new methods simply generate a _ParametricGeometry with domain-transformed parametric function, and then use the standard _integral code paths. This new process was applied to specialization methods for:

BezierCurve
Line
Plane
Ray
Tetrahedron
Triangle

Deprecates jacobian(::BezierCurve, ts, ::Analytical) method - performance was inferior to the generic FiniteDifference method.
Changes to internal utility functions:

Removes now-obsolete _guarantee_analytical
Defines a _parametric(::Geometry, ts...) used for specializations
_zeros([T,] N) and _ones([T,] N) return an NTuple{N,T} of zeros/ones (faster than allocating a vector with Base.zeros/ones)

Added a new Specializations section to benchmarks

Moved out BezierCurve from normal section, reducing redundancy.
Added benchmark for Triangle.

Results

Normalizes support for the six specialization types listed above, i.e. they can now be expected to behave like any other curve/surface/volume.
Closes #40 by adding full support for integrating Tetrahedron types.
Mostly completes #128: will update to indicate that we’re waiting on a proper implementation upstream to replace _ParametricGeometry in the future.
Significant performance increases

Massive improvement (~75x faster) for integrals over a BezierCurve.
Significant improvement (up to 4x) for scalar integrands with HAdaptiveCubature rule.

Significantly reduces code and math complexity.
Incurs a performance regression (~4x slower) for integrals over a Triangle due to new requirement to call differential inside integrand (vs baked-in analytical), but simultaneously discovered and fixed a bug where Unitful functions couldn't be integrated with the old version.

TODO

Update Documenter specializations page

Whoa. The new method for BezierCurve is almost 70x faster!

Apologies for the surprise PR, @kylebeggs and @JoshuaLampert. I had a flash of inspiration over the weekend for how to implement Tetrahedron methods and it turned out to be pretty great at replacing all of the other complicated integral specializations too.

I was puzzling over why the recent commit to benchmarks failed. Turns out we had a bug in the HAdaptiveCubature integral method for Triangle - it lacked the Unitful wrapper and so would throw an error for any integrand function with Unitful output. Yet another benefit of having far fewer integral methods to think about.

I'm trying out a slightly (moderately (strongly)) hacky solution. I don't love it, but it gets the analytical jacobian for BezierCurve working, so let's see how performance compares.

Update: The hacky solution was able to get the jacobian(bezier, ts, ::Analytical) back in use through a _ParametricGeometry wrapper. Surprisingly, performance was not significantly different vs using the FiniteDifference method: ~80x faster than main with Analytical but ~85x with FiniteDifference. It seems like something else entirely about the old integral methods for BezierCurve was just plain slower. I'm just going to call this one dead and deprecate this analytical method (and un-roll the latest hacky test changes).

cc: @JoshuaLampert, @kylebeggs
Just need to update the docs and I think this one is otherwise just about done.

So this means has_analytical always returns false, which means we don't need the Analytical differentiation method anymore? Would it be the easiest to simply remove the option entirely? Then we would also not need default_method anymore, but simply make FiniteDifference always the default. What do you think?
The new method for integrating Triangles is ~5 times slower now. What do you think about that?

Sorry, I have not been able to keep up with this PR. However, if this is true:

The new method for integrating Triangles is ~5 times slower now. What do you think about that?

I would urge to fix this before merging. Just my opinion but performance should be near the top in priority in a package like this.

Performance with this change is oddly kind of a mixed bag. There's actually a couple of ideas contained in this PR, so I think I'm going to partition it into multiple smaller changes to make them easier to review and reason about.


Question: Remove files after emit?

Is it possible to remove certain files from dist folder after emit?

Try clean-after-emit-webpack-plugin with clean-webpack-plugin.

Thanks for this @flyyang . I installed the clean-after-emit-webpack-plugin and it's working exactly as wanted.
I don't really understand what beforeEmit: false on the clean-webpack-plugin is doing then though. I thought if we set beforeEmit to false that should execute cleaning on the end of emit, but that setting doesn't seem to do anything here. But I might be missing something or doing something wrong.
Anyway, your suggestion works fine, so thanks again!

@Friksel see here for more context: https://github.com/johnagan/clean-webpack-plugin/pull/72#issue-165399571
@flyyang thanks for the handy plugin!

Closed in v2.0.0 via #99. See johnagan/clean-webpack-plugin#usage. Please open a new issue if you continue to have problems.


IndexError

environment：

windows python3.6
CUDAv10.2 and Corresponding cdnn、paddle（2.4.0.port102），

where:
render_utils.py line 56：rush = paddle.to_tensor(meta_brushes_resize[index])
IndexError
index<PHONE_NUMBER>4 is out of bounds for axis 0 with size 2

Hi, have you solved this problem now, I got this too.

Hi, have you solved this problem now, I got this too.

I got this error too


docker failing to start when ports being mapped

Output of docker version:
Client:
 Version:      1.11.1
 API version:  1.23
 Go version:   go1.5.4
 Git commit:   5604cbe
 Built:        Tue Apr 26 23:43:49 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.11.1
 API version:  1.23
 Go version:   go1.5.4
 Git commit:   5604cbe
 Built:        Tue Apr 26 23:43:49 2016
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 3
 Running: 0
 Paused: 0
 Stopped: 3
Images: 21
Server Version: 1.11.1
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 90
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: host bridge null
Kernel Version: 4.4.0-22-generic
Operating System: Ubuntu 16.04 LTS
OSType: linux
Architecture: x86_64
CPUs: 8
Total Memory: 11.67 GiB
Name: ENVY
ID: 5E2B:2NGP:DART:EAJI:FXXR:QUTF:ZYOV:BACM:T2LZ:SLAU:YRHC:Z6JO
Docker Root Dir: /var/lib/docker
Debug mode (client): false
Debug mode (server): false
Username: matthb
Registry: https://index.docker.io/v1/
WARNING: No swap limit support

Additional environment details (AWS, VirtualBox, physical, etc.):

Ubuntu 16.04 running on a laptop.
uwf disabled
docker restarted
nothing is running on 3306

Steps to reproduce the issue:
$ docker run -e MYSQL_USER=mysql -e MYSQL_PASSWORD=mysql -e MYSQL_DATABASE=sample -e MYSQL_ROOT_PASSWORD=supersecret -it -p 3306:3306 mysql

Describe the results you received:

docker: Error response from daemon: driver failed programming external connectivity on endpoint small_brown (c45f5b44e5862d2f3a9d064352fe463a31b55b178c4c8a36460951aa3c903cc2): iptables failed: iptables --wait -t nat -A DOCKER -p tcp -d 0/0 --dport 3306 -j DNAT --to-destination <IP_ADDRESS>:3306 ! -i docker0: iptables: No chain/target/match by that name.

Describe the results you expected:
Additional information you deem important (e.g. issue happens only occasionally):
this happens whenever I run anything that maps a port

This sounds as https://github.com/docker/docker/issues/16816 (and https://github.com/docker/docker/issues/17320), and could be related to other software interacting with iptables.

urrg.  Ubuntu asked to install some updates this evening and now all is well.  I wish I could be more help.
thanks for reading.


Automatic scratch org generation

Branches starting with "dev/" are now excluded from scratch org generation. Just for me... :)
Closes #16

Maybe we should change the prefix from "dev/" to "no_scratch_org/", to be more clear. I think in many cases developers will want to use the scratch org creation to make use of the dependency management. What do you think?

Yes, that makes sense.
Since we like kebab case in this template, what about "no-scratch/"?


Onevnet multinet provider

onevnet and addressrange types and providers.
tested on opennebula virtualbox image

looks okay to me... but onevnet_addressrange has some commented code which seems to do nothing...

I assume you are referring to the onevnet_addressrange type.
At the moment this lacks validation. I will work on this next.


fix: Remove unused packages in lock file

#minor
Description
This PR cleans up the packages not used in the yarn.lock file.
Specific Changes

Removed unused packages in the root lock file.
Removed unused package versions in the root lock file.

Testing
The following image shows the unit tests working after the updates.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

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

Totals

Change from base Build<PHONE_NUMBER>8:
-0.3%

Covered Lines:
20431

Relevant Lines:
23091

💛  - Coveralls


Compile error https_server.crt.S not found

First of all, thanks @Jason2866 for your amazing work on espressif32 forks to support PIO.
I'm trying to switch from Jason2866/platform-espressif32 to pioarduino/platform-espressif32, as I think this new repo is a more general purpose one. Am I right?
I have no problem with Jason2866/platform-espressif32, it compiles fine.
However, I get this error with pioarduino/platform-espressif32:
Processing debugNoBle (platform: https://github.com/pioarduino/platform-espressif32/releases/download/51.03.04/platform-espressif32.zip; board: lolin_d32; framework: arduino, espidf)
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Verbose mode can be enabled via `-v, --verbose` option
CONFIGURATION: https://docs.platformio.org/page/boards/espressif32/lolin_d32.html
PLATFORM: Espressif 32 (51.3.4) > WEMOS LOLIN D32
HARDWARE: ESP32 240MHz, 320KB RAM, 4MB Flash
DEBUG: Current (cmsis-dap) External (cmsis-dap, esp-bridge, esp-prog, iot-bus-jtag, jlink, minimodule, olimex-arm-usb-ocd, olimex-arm-usb-ocd-h, olimex-arm-usb-tiny-h, olimex-jtag-tiny, tumpa)
PACKAGES:
 - framework-arduinoespressif32 @ 3.0.4
 - framework-arduinoespressif32-libs @ 5.1.0+sha.b6b4727c58
 - framework-espidf @ 3.50104.0 (5.1.4)
 - tool-cmake @ 3.21.3
 - tool-esptoolpy @ 4.7.5
 - tool-mklittlefs @ 3.2.0
 - tool-ninja @ 1.9.0
 - tool-riscv32-esp-elf-gdb @ 11.2.0+20220823
 - tool-xtensa-esp-elf-gdb @ 11.2.0+20230208
 - toolchain-esp32ulp @ 1.23500.220830 (2.35.0)
 - toolchain-xtensa-esp32 @ 12.2.0+20230208
Warning: the 'src_filter' option cannot be used with ESP-IDF. Select source files to build in the project CMakeLists.txt file.

Warning! Arduino framework as an ESP-IDF component doesn't handle the `variant` field! The default `esp32` variant will be used.
Reading CMake configuration...
LDF: Library Dependency Finder -> https://bit.ly/configure-pio-ldf
LDF Modes: Finder ~ chain, Compatibility ~ soft
Found 48 compatible libraries
Scanning dependencies...
Dependency Graph
|-- Core
|-- Imu
|-- Obelisk
Building in debug mode
*** [.pio\build\debugNoBle\.pio\build\debugNoBle\https_server.crt.S.o] Source `.pio\build\debugNoBle\https_server.crt.S' not found, needed by target `.pio\build\debugNoBle\.pio\build\debugNoBle\https_server.crt.S.o'.
===================================================================================== [FAILED] Took 8.14 seconds =====================================================================================

Environment    Status    Duration
-------------  --------  ------------
debugNoBle     FAILED    00:00:08.141
=============================================================================== 1 failed, 0 succeeded in 00:00:08.141 ===============================================================================

It is as you noted 100% the same as the espressif Arduino 3.0.4. So it includes Rainmaker and the insights stuff.
You have to add in your env for arduino/espidf projects (see example):
board_build.embed_txtfiles =
    managed_components/espressif__esp_insights/server_certs/https_server.crt
    managed_components/espressif__esp_rainmaker/server_certs/rmaker_mqtt_server.crt
    managed_components/espressif__esp_rainmaker/server_certs/rmaker_claim_service_server.crt
    managed_components/espressif__esp_rainmaker/server_certs/rmaker_ota_server.crt


Big list of things to check

Stack Exchange has a list of "What things should a programmer implementing the technical details of a web application consider before making the site public". This may be useful to double-check in case we missed something.
If this seems useful, we might want to create individual github issues for specific items on the list.

This kind of thing is definitely a good practice. I think there are two main things from this list that matter: 1) Mobile-friendliness, which we won't have at launch, and 2) Resource limits and throttling on the AI, which we should have at launch.
We have no user-inputted data and no user-facing communication (comments sections, etc.), no user accounts or passwords, most of the optimization stuff is done with HTTP2, we don't use cookies for anything we don't absolutely need to and store minimal data in those cookies (just a session ID for carts). We're not SEOed for individual votes right now but we also can't be until we get canonical names for votes.
I will make it a priority to try to implement the skeleton of an API throttle this weekend.


Sponsor deserves more visual real estate

Testing #150748
We already discussed this offline but creating an issue so we potentially revisit this in the future.
Currently we render a ❤ Sponsor button in the Extension details and it is great.
However I still think that we should add a simple ❤  in the extension viewlet if an author has on-boarded to the sponsor. Here's my reasoning:

It feels to me the same as verified publisher - and we currently render that
If a user is super happy with an extension he will never open the extension details and discover that the extension is asking for a sponsorship. However everybody opens the extension viewlet from time to time (to install / uninstall extensions) and this should be discoverable there
We want to make this just discoverable enough so extension authors do not use notifications to spam users for donations
There is enough real estate in the extension viewlet. I suggest to not show this for search results, but only for installed extensions (when we do not show ratings / install count)
Adding an icon will not bring significant noise to the view

Design proposal: similar that we have the verified checkmark before the publisher. We could render the heart after the publisher area.
fyi @daviddossett @misolori for thoughts

First two look great and just loud enough. We could add a blurb with link in the hover (similar like we have for verified).

I understand the interest behind making this feature visible to users.
But I am not convinced that this has to be shown as a primary action in the extensions view. Sponsoring is not something  users often do when compared to other actions like enable, disable or uninstall. This is the main reason why I did not render Sponsor action in the header in the extension editor. Primary focus of our UI shall be to make day to day activity easy for users. Please also consider that for those users who do not want to sponsor (which I think is majority) this is unnecessary noise. This is also noise for those users who already sponsored - because I do not think they would sponsor again for the same extension.
For example, GitHub also does not render if a project can be sponsored when browsing for projects. One can only sponsor when you open the project.

It feels to me the same as verified publisher - and we currently render that

I disagree. Verified publisher gives an impression to the user that the extension can be somewhat trusted, where as sponsor action does not - this is requesting user to sponsor. Imagine if the publisher is not verified but still has sponsor action? How can I take a decision to sponsor. There is no enough information in the extension view for this. We shall always drive users to open the extension and provide as enough information as possible before they decide to sponsor.

Adding an icon will not bring significant noise to the view

I am sorry that adding an icon adds noise because extensions viewlet has very less real estate. Please see following

If the intention is to make it discoverable, I would explore the badge area which is intended for similar purposes like this

pre-release
recommendation
sponsor

Or let's come up with some smart model that periodically checks if user is using an extension for a long time and ask if user would like to sponsor those extensions.

@sandy081 good feedback and ideas, thanks.
Let me clarify: what I was proposing is not to have a clickable action in the extension view, but just some kind of UI decoration. Something to convey additional meaning and tackle the user's imagination. If you think a badge area is a better way to convey this information this works for me. The only problem I see with this approach is what if there are multiple badges - pre-release, recommendation and sponsor at the same time. Who wins right now when a pre-release extension is recommended?
Your idea for a smart model that periodically checks might work, my worry is that we would have to use notifications for it - I do not see a better way to ask the user to sponsor?

I think the sponsoring probably brings more total value to the ecosystem than giving a rating. And for ratings users get introduced to them when they install the extension (we show ratings then). After that user can go to the extension details to rate.
Extension sponsor currently does not have this moment where users get introduced to the concept.
So when I said "tackle the user's imagination" I meant what I just said - just a way for users to discover this initially.
For multiple badges stacked on top - I live this to @daviddossett when he finds time (nothing urgent)
For the notification center idea - this might be just showing a badge in the extension view. The badge might go away once the user opens the view. I do not think we need something too heavy for this.

One idea that came out of discussion between @isidorn and me is to add Sponsor / Rating to hover.

Interesting idea. I can quickly sketch this at some point this week. Stay tuned.

@daviddossett we thought that the messaging should be something like "help make the extension ecosystem better by rating or sponsoring NAME_OF_EXTENSION"
This is not the exact wording, but that should be the underlying message, and then rating or sponsoring would be two links...
Of course we are open to other ideas :)

@daviddossett this might work great. I would hate to add too much noise.

When I implemented custom hovers for extensions, I was including rating in the hover and removed it later.  I pulled those changes now and added sponsor and this is how it looks:

It aligns with extension editor representation.
Note that I also moved Sponsor action to the header in the editor.
What do you think?

Reopening it for further discussions

@sandy081 I just tried this out from source and feels great to me. Both the hover, and the move to the header section. I am talking to Anup on the Marketplace side to get his thinking about the MP rendering - ideally we would also do it in the header section there.
My only feeling is that the color does not work for dark theme. Not enough contrast.

Unrelated to this. "This extension is enabled globally" - I suggest to not have a hover for this, only if the extension is disabled does this deserve a message. If we remove this the hover will be lighter. Is there some benefit of this message that I am missing?

My only feeling is that the color does not work for dark theme. Not enough contrast

My bad, I did not check this in Dark theme. @misolori I need your help again to tweak the colour. Also can we have filled heart icon?

Unrelated to this. "This extension is enabled globally" - I suggest to not have a hover for this, only if the extension is disabled does this deserve a message. If we remove this the hover will be lighter. Is there some benefit of this message that I am missing?

Since we have different enablements, it is helpful to show it.

@sandy081 as for full or empty heart. Both work for me. Whatever you, @daviddossett think is best.
I have also shared this issue with MP side so let's also hear what they prefer

@sandy081 previously the colors were being used as a background color in a button, do we envision this not being used anymore or do we need separate color tokens?

@misolori we envision those not being used anymore. We are leaning towards this design now it seems :)

@isidorn Closing this in favour of the improvements done.
Please verify the improvements done.


Marconi Utxo indexer return collateral

This PR is to fix the collateral processing of UTXO per PLT-656.  There are three commits in the PR:

Commit-1 adds property tests for :

return collateral
phase-2 validation failure
minor refactoring of the Indexser.Utxo module to support TDD
We expect both these tests to fail in this commit

Fixes to the Indexers.Utxo to get the above tests pass

Removes the SQL Unique constraint, and replace INSERT OR REPLACE with INSERT for the spent and unspent_transactions

Pre-submit checklist:

Branch

[ ] Tests are provided (if possible)
[ ] Commit sequence broadly makes sense
[ ] Key commits have useful messages
[ ] Formatting, PNG optimization, etc. are updated
[ ] Important changes are reflected in changelog.d of the affected packages

PR

[ ] Self-reviewed the diff
[ ] Useful pull request description
[ ] Reference the ADR in the PR and reference the PR in the ADR (if revelant)
[ ] Reviewer requested

Did you run your changes on mainnet without anything failing?

Did you run your changes on mainnet without anything failing?

@koslambrou my local instance of node is comletly sync . Marconi is at 48.02% currently. I run the script:

I transition the PR  to draft.
Although this PR is moved to marconi repo, there is a pending change on plutus-chain-index-core/src/Plutus/ChainIndex/Marconi.hs that may need to be applied to plutus-apps that we may need to consider


Fix two minor issues

#32 and #29

@Anaphory I have a rather big rewrite of the CLDF spec in the making, which will require quite a bit of refactoring of pycldf as well. I'll make sure to address the issues this PR solves as well. So I think it doesn't make sense to merge this now.

Makes sense! I just fixed them in my local installation and therefore it made sense to push and submit the request.


Fix IoT Device Update Readme issue #31589

Fixes #31589
@joshfree, @drwill-ms and @dpokluda for notification.

@joshfree, @drwill-ms and @dpokluda  Please help review this PR. Thanks in advance.

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


Chore: Parenthesis in shell

Changelog Description
Escape parenthesis for shell subprocess when bash or sh are set as shell applications.
Additional info
Content of parenthesis is "executed" in bash so it is necessary to escape them. This was discovered when thumbnail extraction should change resolution and command contained something like (1920-iw).
Testing notes:
I don't know. It requires linux machine and specific settings and case of input image.

The real question here is maybe also, why are we running this using shell=True instead of without that and passing it just via subprocess as separate arguments?
And is there a chance that shell on Linux may not be bash?
And, shouldn't shlex.quote be able to do this otherwise?](https://docs.python.org/3.4/library/shlex.html#shlex.quote) or doesn't that escape ()?

The real question here is maybe also, why are we running this using shell=True instead of without that and passing it just via subprocess as separate arguments?

Many times asked and aswered question. Because we're receiving ffmpeg arguments from settings, and to be able to "safely" use them as list, we would have to force TDs to fill them correctly (single arg as single string value ["-c:v libx264"] -> ["-c:v", "libx264"], or hare 100% reliable way how to split them. I didn't find one, shlex is not really good at it. We would have to have own OS specific implementation, and probably few iterations of the implementation. That would break productions few times before we get it correctly (not fun).

However, I personally dislike the code duplication for this. It's not crazy, yet at the same time still annoying.

Well, it probably can be moved to run_subprocess, not a big deal for me.


Add projects built with create-eth-app to README

Off the top of my head:

https://github.com/raid-guild/xdai-omnibridge
https://github.com/ensdomains/reclaim

One more:
https://github.com/PermanentLoss/permanentloss


Avoid SNS topic name collision

@zmully I was getting an error from CFN when two SNS topics were named the same thing.  This varies the names.

Superceded by #51


Problem view should show source information

VSCode Version: alpha
OS Version: all

Markers carry a diagnostic attribute (e.g eslint, ts,js, ...). If present it should be rendered in the problems view.

@dbaeumer See the attached screenshot. Wondering if this would be clear for the user to determine the source of error/warning.

After thinking again, I think its not a good idea to put owner information as part of the entry as not everybody is interested in this info. Another idea is to put it as part of the filters so that interested users can filter markers based on who generate them

@sandy081 sorry, I was not clear in my description. Corrected it. I was talking about the source attribute on a marker. And be aware that different markers on a file can have different source attibutes. So we can't make it part of the file name. Either we need a column or we integrate it into the message as we do in hovers (e.g [ts]. [js], ...)


Version Packages (beta)

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
⚠⚠⚠⚠⚠⚠
main is currently in pre mode so this branch has prereleases rather than normal releases. If you want to exit prereleases, run changeset pre exit on main.
⚠⚠⚠⚠⚠⚠
Releases
<EMAIL_ADDRESS>Patch Changes

41799f3: Pass alt to Image if fill or width and height are not defined

<EMAIL_ADDRESS>Patch Changes

Updated dependencies [41799f3]

<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 62.903%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1255

Relevant Lines:
1741

💛  - Coveralls


Bone layers needs to be updated for Blender 4.x

https://developer.blender.org/docs/release_notes/4.0/upgrading/bone_collections/

I was wrong about the source of the problem because proxy_picker.py is throwing an error in the same place:
Error in bpy.app.handlers.depsgraph_update_pre[0]:
Traceback (most recent call last):
  File "C:\Users\myuser\AppData\Roaming\Blender Foundation\Blender\4.1\scripts\addons\proxy_picker.py", line 190, in proxy_picker
    if bone_layer_active(b) == False:
       ^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\myuser\AppData\Roaming\Blender Foundation\Blender\4.1\scripts\addons\proxy_picker.py", line 22, in bone_layer_active
    if b.layers[i] == True and ob.data.layers[i] == True:
       ^^^^^^^^
AttributeError: 'Bone' object has no attribute 'layers'
=========================================================

Not my monkey not my circus


exported fetch is useless on client side

var fetch = require('isomorphic-fetch')

doesn't work on client side, because whatwg-fetch module exports nothing.
To make it work npm-client.js must look like
require('whatwg-fetch');
module.exports = fetch;

(instead of module.exports = require('whatwg-fetch');)
client.js has the same issue, I guess.

+1

+1

The documentation and example does state that requiring isomorphic-fetch modifies the global object for consistency with the native API. However being able to require/import it via the usual means makes it more obvious when reading the code.

Thank you :D.


forgejo: 9.0.2 -> 7.0.11

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for forgejo is: Self-hosted lightweight software forge
meta.homepage for forgejo is: https://forgejo.org
meta.changelog for forgejo is: https://codeberg.org/forgejo/forgejo/releases/tag/v7.0.11
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 7.0.11 with grep in /nix/store/nwgjs55dxmgf2ca1fyhqlrjbwy3bd2wk-forgejo-7.0.11
found 7.0.11 in filename of file in /nix/store/nwgjs55dxmgf2ca1fyhqlrjbwy3bd2wk-forgejo-7.0.11

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

forgejo

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/nwgjs55dxmgf2ca1fyhqlrjbwy3bd2wk-forgejo-7.0.11 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A forgejo https://github.com/r-ryantm/nixpkgs/archive/d838fee6f7c13a616bd407049cf775803cf858e9.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/d838fee6f7c13a616bd407049cf775803cf858e9#forgejo

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/nwgjs55dxmgf2ca1fyhqlrjbwy3bd2wk-forgejo-7.0.11
ls -la /nix/store/nwgjs55dxmgf2ca1fyhqlrjbwy3bd2wk-forgejo-7.0.11/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 2 packages built:

forgejo
forgejo.data

Maintainer pings
cc @emilylange @urandom2 @bendlas @adamcstephens @nycodeghg for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

lol, no.

https://r.ryantm.com/log/forgejo/2024-11-16.log

Fix submitted: https://github.com/NixOS/nixpkgs/pull/356593


[tensorflow] disable REPL (both integrated and LLDB)

Disables the integrated and LLDB REPLs in the S4TF toolchain.
When a user tries to run the REPL, it does this:
$ swift
<unknown>:0: error: The Swift for TensorFlow toolchain does not support the Swift REPL. Colab (https://github.com/tensorflow/swift/blob/master/Usage.md#colaboratory) and Swift-Jupyter (https://github.com/google/swift-jupyter) are supported alternatives.

Determined users who need to use the REPL despite the fact that we don't support it can still get the REPLs by invoking swift -frontend -repl (for the integrated REPL) or lldb -r (for the LLDB REPL).
Justification: The LLDB REPL keeps breaking in our toolchain, and we would rather spend effort maintaining Swift-in-Colab and Swift-Jupyer.
Corresponding documentation update at https://github.com/tensorflow/swift/pull/432.

Oops, I forgot to trigger the tests. I'll revert it and run the tests before remerging.


3265 long date boolean types

https://hibernate.atlassian.net/browse/HSEARCH-3265

After its very comprehensive description, the issues seems not be very difficult to solve. I've implemented the mapper and the backend part for a Long property type. I'll add the backend support for the other types in some subsequent commits. Thanks

Pull Request Test Coverage Report for Build 2

88 of 97 (90.72%)  changed or added relevant lines in 13 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 86.712%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

backend/elasticsearch/src/main/java/org/hibernate/search/backend/elasticsearch/types/codec/impl/LongFieldCodec.java
6
8
75.0%

backend/lucene/src/main/java/org/hibernate/search/backend/lucene/types/codec/impl/LongFieldCodec.java
15
22
68.18%

Totals

Change from base Build 60:
0.05%

Covered Lines:
11485

Relevant Lines:
13245

💛  - Coveralls

@yrodiere, as we were saying before, RangeSearchPredicateIT and SearchSortByFieldIT seem not suitable to handle some boolean values. So I'm going to create a specific test for boolean types, as a replacement.

Hi @yrodiere.
Shall I change even theAbstractStandardLuceneFieldSortBuilderFactory class?
In particular, updating:
protected final LuceneFieldConverter<F, ?> converter;
in to:
protected final LuceneFieldConverter<?, F> converter;
or not? Thanks

@fax4ever Yes you can. It seems more correct. If you do it, please rename the type parameter F to T in that class, too, to be consistent with the naming in LuceneFieldConverter.

Fine, I will change it in a subsequent commit.

Hi @yrodiere tests added. Let me know what I have to change. Thanks ;)

Looks good! Merged, thanks.


Makes zigzag encoding and decoding save wrt. undefined behavior, see #28

This changes the zigzag encoding and decoding implementation so that we never run into undefined behavior with signed integer overflows. See thread starting at https://github.com/lemire/streamvbyte/issues/28#issuecomment-873645608 cc @lemire

Please see PR https://github.com/lemire/streamvbyte/pull/36

Closing in favour of https://github.com/lemire/streamvbyte/pull/36


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_91uic0jla3/clone/recipe/meta.yaml:15: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_91uic0jla3/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_91uic0jla3/clone/recipe/meta.yaml:43: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_91uic0jla3/clone/recipe/meta.yaml:43: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found


Update README.md to show newly added support AWS APAC regions

Updated documentation to show APAC regions (Singapore, Mumbai, Tokyo, Sydney) are now supported. I have tested them to be working.

Thanks @nicholas-chia!


move examples/ to demo/

Closes #72

Those failures have anything to do with my changes?

@stjepang Docker images which are used by cross by default do not have OpenSSL in them (see https://github.com/rust-embedded/cross/issues/229).
These changes are failing CI now because [[bin]] targets are compiled when cargo test (or cross test) called and that causes an OpenSSL linkage error; [[example]] targets are only checked right now and not really built during the CI execution:
https://github.com/stjepang/smol/blob/74b8276e999201836ff5e5f8515c6ec026c6c970/.github/workflows/build-and-test.yaml#L26-L30
One possible solution to solve this issue is to stop compiling [[bin]] targets for cross workflow job in here:
https://github.com/stjepang/smol/blob/74b8276e999201836ff5e5f8515c6ec026c6c970/.github/workflows/build-and-test.yaml#L65-L66
Yet, I'm personally not sure what are the benefits of whole this change, so it maybe worth just to leave everything as is. Also, I'm not a maintainer here, so it is just my two cents on a situation :)

Yet, I'm personally not sure what are the benefits of whole this change, so it maybe worth just to leave everything as is.

Two reasons:

Adding examples is hard. I can't just create examples/foo.rs and then run it. I need to first define it inside examples/Cargo.toml, which is a bit annoying. I often create temporary examples like that for local testing.

Running examples is not a smooth process because we need to cd into examples/. This might be surprising to new users, and requires users to look into the readme to figure out what's actually going on.

@stjepang thank you for clarification! It seems that [[bin]] sections should be removed from the demos/Cargo.toml file then?
Regarding the CI failure. Since there is no OpenSSL in the cross containers, it is possible to tweak CI a bit to make it work at least somehow:

Stop compiling binary targets on cross test. Unfortunately, there is no smth like --exclude-target argument exist in cargo test target selection options, so all other targets should be set manually as in

cargo test --lib --tests --benches

Since binary targets are not compiled now, at least we can "check" them, so additional job step should be added with

cargo check --bins

It seems that [[bin]] sections should be removed from the demos/Cargo.toml file then?

I'm fine with [[bin]] sections there - those don't bother me because demos are not added very often.

Regarding the CI failure. Since there is no OpenSSL in the cross containers, it is possible to tweak CI a bit to make it work at least somehow:

Cool! That doesn't seem too bad.

It seems that [[bin]] sections should be removed from the demos/Cargo.toml file then?
How would one run them if [[bin]] were removed?

@svartalf are you able to do those CI changes, because I don't actually understand.

I just found a better solution to this problem, which does not requires any CI modifications. So, here is a dependencies branch, which causes the compilation problem:
smol-demos v0.0.0 (/home/svartalf/projects/smol/demos)
├── async-native-tls v0.3.3
│   ├── native-tls v0.2.4
│   │   ├── openssl v0.10.29
│   │   │   └── openssl-sys v0.9.55

Fortunately to all of us, it is possible to enable vendored feature of openssl-sys through all these dependencies chain with help of the async-native-tls feature with the same name.
@tshepang can you apply the following patch and push the changes so we can check the CI results?
diff --git a/demos/Cargo.toml b/demos/Cargo.toml
index c15e8e3..7560ca3 100644
--- a/demos/Cargo.toml
+++ b/demos/Cargo.toml
@@ -16,7 +16,7 @@ timerfd = "1.1.1"
 [dependencies]
 anyhow = "1.0.28"
 async-h1 = "1.1.2"
-async-native-tls = "0.3.3"
+async-native-tls = { version = "0.3.3", features = ["vendored"] }
 async-std = "1.5.0"
 async-tungstenite = { version = "0.4.2", features = ["async-native-tls"] }
 base64 = "0.12.0"

That build failure happens because the ~examples/~ demos/ are now checked in cross compile environment.  Let me push a change that removes that (though that means one would have to navigate into demos/ to run them).

Pull Request Test Coverage Report for Build 95835664

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.873%

Totals

Change from base Build 94462064:
0%

Covered Lines:
129

Relevant Lines:
146

💛  - Coveralls

It seems that [[bin]] sections should be removed from the demos/Cargo.toml file then?

@svartalf How would one run them if [[bin]] were removed?

Maybe move the source code for bins to demos/src/bin, then one would no longer need to annotate Cargo.toml each time they add a new one.

Just a question?? why are you not just using examples as they are supposed to be used? Is it to keep dev-dependencies OUTSIDE of main Cargo.toml?

I think it was to keep CI builds fast, but am not sure.

@tshepang There was an issue with an example requiring the cssparser crate that failed to compile

@leo-lb would it not be enough to add cssparser to [dev-dependencies]?

@tshepang I am not sure examples count as "dev"
See here for context: https://github.com/stjepang/smol/pull/13

@leo-lb just checked Cargo book...

Dev-dependencies are not used when compiling a package for building, but are used for compiling tests, examples, and benchmarks.

@tshepang We would need them to be used for examples but not tests, because tests are used for coverage, so it's not solving the issue.

this is made obsolete by #188


Admin ID and regular ID and cannot publish my Visual Studio, C#, SharePoint, app

I have successfully built an app using MS Visual Studio 2019 that I would like to publish. In my current capacity, I have two IDs one is my standard user that I do everything with, the other is my standard user with admin access. ID 1 = StandardUser, ID 2 = StandardUserA. When I start Visual Studio, I logon as StandardUser and start VS using StandardUserA. All the project files can be found in C:\Users\StandardUserA\source\repos\TQM4\TQM4\bin\Release\app.publish\<IP_ADDRESS>. I can deploy my app to SharePoint, I can also debug it. Everything appears to be working until I attempt to publish it.
When I click "Package the add-in" it fails with "C:\Users\StandardUser\source\repos\TQM4\TQM4\bin\Release\app.publish\<IP_ADDRESS> is unavailable. . ." It suggests making sure the device is connected. I am running visual studio as StandardUserA.
The file it is looking for is: C:\Users\StandardUserA\source\repos\TQM4\TQM4\bin\Release\app.publish\<IP_ADDRESS>. How do I tell Visual Studio to look there? DO I have to logon to windows as StandardUserA to publish?
Thank you, Duncan

Confirmed it works when signed in as StandardUserA but that user is not one I use for day to day activities.  Would like to know how to make it work with StandardUser.  The project was created when logged on has StandardUser but Studio was has an administrator using StandardUserA and all the project files are in C:\Users\StandardUserA\source\repos\TQM4.


Unable to build

During the follow step, I got an error building:
<EMAIL_ADDRESS>postinstall C:\Users\Me
> electron-builder install-app-deps

  • electron-builder version=19.56.2
  • loaded configuration file=package.json ("build" field)
  • rebuilding native production dependencies platform=win32 arch=x64

And in the app, when I try to execute a query, I get this error:

Failed to load gRPC binary module because it was not installed for the current system Expected directory: electron-v4.0-win32-x64-unknown Found: [node-v64-win32-x64-unknown] This problem can often be fixed by running "npm rebuild" on the current system Original error: Cannot find module 'C:\firestation\node_modules\grpc\src\node\extension_binary\electron-v4.0-win32-x64-unknown\grpc_node.node'

I tried npm rebuild as recommended but still got the same error.

Hm, not totally sure if the app will work on windows nowadays, been a while since I opened this.
Try running the projectRevamp branch and see if you have any luck with that:
git  checkout projectRevamp
npm start

Sorry forgot to mention those were the steps I followed:
git clone https://github.com/JoeRoddy/firestation.git
cd firestation
git checkout projectRevamp
npm install
npm start

Same problem....


goaccess doesn't seem to like hostnames like "localhost"

Parser gives errors like:
Token 'localhost' doesn't match specifier '%h'
Any way to avoid this?

If a valid IP isn't provided for the %h specifier, please utilize the --no-ip-validation option.

Thanks! That fixed the issue.

Happy to hear that it worked out. I'll go ahead and close this for now, but feel free to reopen it if needed.


Real time data causes render blinks

While working with real time data coming in through a websocket, the graph render becomes very jerky. It also causes blinks while rendering.

What is the behavior you expect?

The render needs to be smooth and without blinks. If there is any lower limit of refreshing beyond which the rendering becomes jerky, would be good to know.

I have not experienced this yet. Can you please create a simpler version of it without web sockets?
Or maybe paste your chart configuration and updateSeries logic?

Problem was caused due to high data rate at the client end. Works seamlessly at a lower data rate. Closing the issue.

@junedchhipa Cloud you please check this one?

I checked the data from WebSocket and it doesn't have a problem.
I'm using apex and vue3-apexcharts and my config is:
export const speedGaugeChartOptions = {
 plotOptions: {
    radialBar: {
      startAngle: -90,
      endAngle: 90,
      hollow: {
        size: "65%",
      },
      track: {
        show: true,
        strokeWidth: "20%",
      },
      dataLabels: {
        name: {
          offsetY: 25,
          color: "#6b7280",
          show: true,
        },
        value: {
          offsetY: -10,
          color: "#8a63a3",
          fontSize: "2rem",
          formatter(val) {
            return `${parseFloat(val)}`;
          },
        },
      },
    },
  },
  chart: {
    sparkline: {
      enabled: true,
    },
  },
  fill: {
    colors: [
      function ({ value }) {
        if (value <= SPEED_LIMIT) {
          return "#7ca363";
        } else {
          return "#AD6D6D";
        }
      },
    ],
    type: "solid",
  },
  labels: ["km/h"],
};


Source to cat-demon and logo?

Are the cat-demons from the header image or the logo somewhere accessible to download?
I would need one cat-demon for a folder icon for the VS Code vscode-icons extension.

I don't have an SVG source, sorry :(

Ahh, what a shame... They are really cute. I would have made a wallpaper for myself 😄 maybe someday some experienced Illustrator want to redraw them in Illustrator.

Best I have are these:
https://memcached.org/images/memcached_link_125.png
https://memcached.org/images/memcached_banner100.jpg
they're rabbits, btw. "fast, scalable, stupid"

Did u made those by yourself? I really like those demons/rabbits 😄
Sent with GitHawk

Friend of mine did the illustrations a long time ago (along with the site color scheme, logo, etc). going to close this out


importer: rename sst file with key manager to fix ingest sst failure

Signed-off-by: glorv<EMAIL_ADDRESS>
What problem does this PR solve?
Issue Number: close #xxx
Problem Summary:
Fix ingest lightning written sst files failure with TDE enabled.
What is changed and how it works?
In #8852, if tikv TDE is enabled, the sst file written by lightning will be encrypted, but the save action renames the encrypt file but not register this action in the key manager, thus, when ingesting this sst file, rocksdb will treat it as unencrypted and fail.
So this pr add an extra DataKeyManager::rename_files to the original file rename action and add an integration test to test write and ingest sst with TDE enabled.
Proposal: xxx
What's Changed:
Related changes
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects
Release note

Fix the bug that lightning fails to ingest sst files to tikv with importer/local backend when TDE is enabled

cc @yiwu-arbug  @hunterlxt

Didn't look through the code. Does this
https://github.com/tikv/tikv/blob/fed30f813f6ba017fe56f9ddcefc9516020344c9/components/sst_importer/src/sst_importer.rs#L734
need the change too?

No. This file is generated used for exporting sst files. And it's not managed by KeyManager? @overvenus please confirm this

/run-all-tests

/run-all-tests

/merge

/merge


Base: rgUSD, USD3, Vaya (Reserve tokens)

Please review the following token assets:
📑 Description

Token Name: Vaya
Project Website: https://app.reserve.org/
Explorer URI:

Vaya: https://basescan.org/address/0xC9a3e2B3064c1c0546D3D0edc0A748E9f93Cf18d
USD3: https://basescan.org/address/0xEFb97aaF77993922aC4be4Da8Fbc9A2425322677
rgUSD: https://basescan.org/address/0x8E5E9DF4F0EA39aE5270e79bbABFCc34203A3470

✅ Checks

[x] I created a new folder with the token address, all in lowercase
[x] I added the token's logo as a 32x32 PNG file, named logo-32.png
[x] I added the token's logo as a 128x128 PNG file, named logo-128.png
[x] I added the token's logo as a SVG file, named logo.svg
[x] My SVG logo is a proper SVG and not some base64 stuff
[x] My documentation/website clearly display the token address somewhere

cc @ethzoomer


STORM-2488: The UI user Must be HTTP

The UI user Must be HTTP. Otherwise, the UI page can not be authorized

+1


WER vs. Token Accuracy

Hello. We have obtained more than 90% (token) accuracy on our validation set. If I understood correctly, tokens are basically phonemes, plus disambiguation symbols and blanks (please, correct me if I'm wrong)? On the other hand, when we try to decode, we get WER of more than 15%. If we try to increase our language model weight, we get more than 50%. We are using a dictionary with more than 120000 words, and a trigram language model (for the Serbian language), with the default parameters. With such a large token accuracy, we were expecting much better WER (even with phoneme error rate of less than 60% we have obtained higher word recognition accuracy on some other systems). Can you please tell us how is that possible?
Thank you in advance.

Those statistics seem pretty good for Eesen. I just checked and training token accuracy for the tedlium experiment, which went as high as 99.4% but yet the best WER achievable was 15.8% on the test set. (with LM weight of 0.7, but that varies with language and with experiment, and usually a range of LM weights are tried)

Yes, I know. But I am still confused with such a large difference between WER and token accuracy. I am aware of the fact that we cannot compare them directly, but we usually have much better WER than PER, not the opposite. 0.7 was the best choice for Eesen (ACWT 0.7, LMWT 1.0, any other value for LMWT gives us worse results). We had somewhat better accuracy in Kaldi with ACWT of about 0.08 and LMWT between 10 and 15.

I'll wait on a more expert explanation, curious now, as well!

Did you check the beams? How fast are the respective systems? Does the Eesen system maybe prune away too many alternatives?
Another thing to try is the blank scale. You can scale down the blank symbol a bit, which gives you more phones (compared to blank). This can sometimes improve results if you have a high deletion right. Not sure if this is the case here?
F.

On Feb 14, 2017, at 6:09 PM, riebling<EMAIL_ADDRESS>wrote:
I'll wait on a more expert explanation, curious now, as well!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/srvk/eesen/issues/123#issuecomment-279770654, or mute the thread https://github.com/notifications/unsubscribe-auth/AEnA8aEWHCGg79MSMNE5Pfxm-PFZAuRnks5rcd_WgaJpZM4MArLz.

Florian Metze http://www.cs.cmu.edu/directory/florian-metze
Associate Research Professor
Carnegie Mellon University

Actually, we have too many substitutions.
beam=17.0
lattice_beam=8.0
max_active=5000
lstm_layer_num=4, lstm_cell_dim=320
TRAIN ACCURACY 92.8732%, VALID ACCURACY 89.2381%
%WER 16.96 [ 26911 / 158653, 2300 ins, 4759 del, 19852 sub ]
lstm_layer_num=4, lstm_cell_dim=1024
TRAIN ACCURACY 94.1744%, VALID ACCURACY 89.4255%
%WER 17.56 [ 27853 / 158653, 2493 ins, 4667 del, 20693 sub ]
Decoding time is about the same in Kaldi. Most of those substitutions should be solved by increasing the language model weight, but whenever we tried to increase that value (the default value is 1), we obtain higher error rate.
How can we adjust the blank scale?

These are the results for beam=30, lattice_beam=15, lstm_layer_num=4, lstm_cell_dim=320.
WER 16.41 [ 26041 / 158653, 2532 ins, 4587 del, 18922 sub ]
There was no significant improvement.

How can we adjust the blank scale?
This might be from code that hasn't yet been merged with the base branch of Eesen.
As a preview, to do this in a hard coded way, modify class-prior.cc around line 57, adding a line such that it looks like this:

  tmp_priors(0)*=blank_scale; #where the scale default value is 1.0
  double sum = tmp_priors.Sum();
  tmp_priors.Scale(1.0 / sum);

or of course, you could do it in a more elegant(?!) Kaldi-esque way adding a variable to class-prior.h and set it as an argument to net-output-extract in decode-ctc-lat.sh - this will be available in the next version of Eesen

(and in case you don't notice - the linked class-prior.h file actually contains the modifications necessary to do the more elegant way, so that you can specify blank scale on the command line from a script)

Thank you. I will try that as soon as I can.

Hi,
What LM are you using ? is this the very pruned LM trigram that fits in the FST graph ?
Also, just a quick question regarding tedlium results.
It seems v1 is based on Tedlium release1 = 110 hours of audio
and v2-30ms is based on release2 (210 jours even though script still says 110 hours)
can you confirm ?
did you try release 2 on both 10ms and 30ms frames ?
Also did you guys try some rescoring with bigger LM ?
many questions, sorry.
Vincent

I think (this is from memory), v2-30ms is 200h+ and 30ms, yes. We decoded with the regular LM, which worked fine. We compiled the large Cantab LM into a search graph, and were able to directly decode with that, and saw small gains - but nothing too dramatic. More as a proof of concept (with CI phones, the search graph stays small, even with a large LM). If I remember, there was a small advantage in going to 30ms training vs 10ms, yes. Let me know if you want me to dig up the old experiments.

Here are the results for different blank scale values.
lstm_layer_num=4, lstm_cell_dim=320
blank_scale=1
wer_7:%WER 16.96 [ 26911 / 158653, 2300 ins, 4759 del, 19852 sub ]
blank_scale=0.75
wer_7:%WER 16.79 [ 26640 / 158653, 2085 ins, 5061 del, 19494 sub ]
blank_scale=0.5
wer_8:%WER 16.65 [ 26417 / 158653, 2161 ins, 4905 del, 19351 sub ]
blank_scale=0.25
wer_8:%WER 16.77 [ 26613 / 158653, 1807 ins, 5708 del, 19098 sub ]
blank_scale=0.1
wer_9:%WER 17.75 [ 28160 / 158653, 1907 ins, 6405 del, 19848 sub ]
As you can see, the number of insertions improved a little bit, but we also had an increased number of deleted words.
About LM:
KN prune 0.0000001, ngram 1=121197, ngram 2=1279389, ngram 3=357721

@branislavpopovic are you on tedlium1 or tedlium2 (ie 100hours or 200+ hours ?)
also your LM is very small. you may need at least 4M n-grams to get better results.
@fmetze results file in v2-30 is not updated then, right ? does anyone have the actual results for this run?
or maybe it was only run at the phone level ?

@vince62s Those results are not for Tedlium, but for the Serbian language database (mobile speech + books + journalistic corpus). We have tried using some other language models with or without pruning, and that one was the best).

sorry I misread the begining of th thread.

Serbian is a highly inflected language and our results are a consequence of a large number of substitutions. In the mean time, we have calculated character error rate, and it was about 1.75% (short words, and also some longer words that differ in no more than one of two characters), which is an answer to our question. Blank scale also helps.


fix(userspace/engine): improve rule json schema to account for source and required_plugin_versions

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area engine
What this PR does / why we need it:
Now the validation passes against k8s_audit_rules.yaml.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

/milestone 0.39.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

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

/cc @leogr @LucaGuerra

/unhold
false positive.


JupyniumLoadFromIpynbTab hangs for large ipynb project

The function JupyniumLoadFromIpynbTab hangs when I try to use it with a large ipynb file (which I am not able to share), possibly an infinite loop is created. I am not entirely sure why that happens for larger files but not small test files I have created.

By large files do you mean there are lots of media etc? Maybe I need to process the input and only send the input. Currently it sends the whole ipynb and convert it to jupynium file.

Yes, I have lots of plots in this ipynb, it seems to be completely stuck loading this file.
Btw.: Thank you for creating this! I have searched for something like this for a long time

I think I'll need to process the inputs in javascript before sending the ipynb file, as currently it is sending all the outputs. I'm new to javascript and Jupyter Notebook doesn't have an API so I created the whole thing with just trial and error haha. I'll definitely improve this soon! And no worries, thanks a lot for much good feedback!

Hi, does #18 fix the issue?

I have installed jupynium via packer, so I will only receive the fix once it is merged I guess? How would I tell packer to install the dev version for testing?

Okay, I'll just merge and let's see

This just gets automatically closed when I merge. But feel free to reopen if you still face the same issue

Works perfectly now! Thank you!

Works perfectly now! Thank you!

It may work for loading the file, but when you save, it will automatically download a copy as well.
Does it hang when you save?


Join type with deep joins missing

While query builder allows defining join types for every table that relates directly to a model this seems not be the case for deep joins. Joins defined in models are always left joins.
To create an inner join between the second and third table in a join chain I had to write such a command:
// get only those buchungen having a journal and journal has a belegdatum >= $from
$data = Finanzkonto::find()
        ->joinWith(['buchungen' => function($query) use ($from) {
            $query->innerJoinWith('journal')
            ->andwhere(['>=', 'belegDatum', $from]);
             },
        ])
        ->andwhere (['owner'=>$owner))
        ->orderBy('id', 'belegDatum')
        ->all();

This looks very ugly and will get worse the more tables you need to join in a chain. Please also note that the condition for the journal table has to be defined in the subquery while the conditon for the finanzkonto (and buchung) tables are placed after the join operators.
To sove this issue I'd like to make three suggestions.

Enhance the hasOne() and hasMany() to allow specifying the join types. The join types need to be used only on deep joins.
In querybuilder allow dot syntax not only for tables but also for join types.
For deep allow an array to specify the join chain where each element has one link within the chain.

Any other hints are welcome.

What about ->joinWith(['buchungen.journal']) ... ?

@cebe, then the join between buchungen and journal is a left join. I need an inner join to get only those buchungen where journal meets some requirements (e.g.  'datecolumn' > $fromdate).
Due to the fact that yii2 runs a query for each table in the join chain you get all buchungen related to the primary table (finanzkonto) and then only those rows out of the journal table which meets the where condition.


FR: Ability to set new name automatically based on templates

Useful template variables could be

current date
note creation date
note modification date
fields from frontmatter

Don't see a good usecase


Add Hex badges

Add Hex badges and close #221.

We don't need to include them all, but I have for now to show all the options.


chore(ci): add arm64 worker

Pull Request Test Coverage Report for Build 432607288

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 39.351%

Totals

Change from base Build 432537376:
0.0%

Covered Lines:
2256

Relevant Lines:
5733

💛  - Coveralls

I can't set it up because Amazon is rejecting my credit card.
Amazon is so advanced that does not support CVC and my bank is not so advanced and rejects such $1 transactions.
So that's why there will be no testing for arm64.


Bump cilium/ebpf to latest

This includes [1] which should fix the following error:
Failed to load BTF spec: can't read types: type id 2036: unknown
kind: Unknown (19)

[1]: https://github.com/cilium/ebpf/pull/820
cc @shaneutt
Fix #112

Tested locally on Linux ceuse 6.0.8-arch1-1 - works.

Thank you!


[Doc] Add notes for S3 offloading configuration

This PR adds a workaround for https://github.com/apache/pulsar/issues/9084

@Renkai @addisonj could you please help review? Many thanks

@Renkai @addisonj could you please help review? Many thanks

also ping @gaoran10 to help review.

also ping @gaoran10 to help review.

Maybe we should guide users on how to set permission, too. https://docs.aws.amazon.com/AmazonS3/latest/dev/using-with-s3-actions.html

@Renkai thanks for your suggestion, I've added the reference link to the doc, PTAL.


listen should take a callback

.listen should take a callback as optional last argument, in the same way net/http server.listen(port, cb) does so you know if and when the server bound to the port.

No probs and I agree. Should be a quick fix.

Okay, this is now supported. The readme has been updated as well as tests.
You can pass it in as the 2nd or 3rd param, as the 2nd can be used for options as well.


Move factory definitions to a macro

This looks really good. I think there are some places in the readme that also need to be updated. This should probably be merged into a 0.3.0 branch as well.

Looks good :+1:

Merged into 0.3.0


daily checklist script

call vrf
call rr
call df
query latest 1000 submissions for each datafeeds

compare the latest timestamp with current timestamp (should not be that much diff)
check for failed submissions (graph-node doesn't support error handler, nor events are emitted for failed cases)
check timestamp gaps (less than 20-25 secs)

for submission information, grafana dashboard currently only shows last submission time offset, which is good to see current status at that moment, but doesn't show average submission frequency for certain amount of time. so this kind of checks could be meaningful

if using github action schedule trigger, it could be automated
https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#schedule

for submission information, grafana dashboard currently only shows last submission time offset, which is good to see current status at that moment, but doesn't show average submission frequency for certain amount of time. so this kind of checks could be meaningful

If we need to see a stats for longer periods, we can add it to dashboard as well.

Please consider this together with https://github.com/Bisonai/orakl-monitor, https://github.com/bisonai/bisontry and https://github.com/Bisonai/orakl/tree/master/monitor 🙏

@nick-bisonai Let's close this down and reopen for data feed only if you think it is necessary.


edit create room to make one read to the database, and not read all rooms and users in the rooms

What does it do?
it make one call to the db to find the exist room, the old code was read all the user rooms and all the users in this rooms, this code just read one room if it exist,
this will add the ability to use this function any where i user profile and open the chat with his id
Why is it needed?
the function create room was read all the user rooms, and read all the users in this rooms, too much reads with no reason
How to test it?
By open an old room with this code, and create new room .
Related issues/PRs
https://github.com/flyerhq/flutter_firebase_chat_core/issues/67#issue-1180562356
i found this one, but i make this edit for my app  with other customisation, i may make PR for some of them in the future.

Hey! Thanks for the optimisation changes! I see one problem with it though, you did a backwards compatibility support buy reversing the list of user ids, but user ids is not an array with two values, you can have group chats with tens of users, so in case of anybody using that, reverse will not help. Here I see 2 possibilities:

Drop back to the code I had initially, after checking sorted user ids, downside - 1 additional read for the old rooms.
We don't add backwards compatibility and ask everyone to sort their userIds (I assume not many people use it in important production code)

WDYT?

Hey! Thanks for the optimisation changes! I see one problem with it though, you did a backwards compatibility support buy reversing the list of user ids, but user ids is not an array with two values, you can have group chats with tens of users, so in case of anybody using that, reverse will not help. Here I see 2 possibilities:

Drop back to the code I had initially, after checking sorted user ids, downside - 1 additional read for the old rooms.
We don't add backwards compatibility and ask everyone to sort their userIds (I assume not many people use it in important production code)

WDYT?

Hey! Thanks for the optimisation changes! I see one problem with it though, you did a backwards compatibility support buy reversing the list of user ids, but user ids is not an array with two values, you can have group chats with tens of users, so in case of anybody using that, reverse will not help. Here I see 2 possibilities:

Drop back to the code I had initially, after checking sorted user ids, downside - 1 additional read for the old rooms.
We don't add backwards compatibility and ask everyone to sort their userIds (I assume not many people use it in important production code)

WDYT?

Hey, i know the problem of group chats with tens of users, this code is just for the one to one chat, for user to user chat, i dont add it to group chat, so i dont think it could be a problem.

We don't add backwards compatibility and ask everyone to sort their userIds
it can be done using a simple dart code run for one time to sort the old ids array in all rooms, but i try to use simple way dont need external work.

Thanks for the PR!


ent bretagne

hello, just a question, the ENT bretagne-rennes is not implemented in pronotepy? If this is indeed the case, is there a way to implement it?

Hello,
If your ent is not implemented yet, there is a good probability that its login system is similar to another one.
Take a look at the other ent in ent.py, and maybe you will be able to implement your ent as well.
Here is an usefull link :
https://github.com/bain3/pronotepy/issues/36

Try copy-pasting the ENT functions for your region and modifying the links (like the login page's) to yours.

@zazou-du22 This ENT was implemented in #96


Exclude products after completing the selection.

fixes #3210

Tests are failing https://travis-ci.org/django-oscar/django-oscar/builds/601893953

nu nie meer


Improve SIGTERM handling

I observed some issues where the exporter would keep the database pod from terminating due to not handling SIGTERMs. There is probably a cleaner way to do this, but here is an attempt.
Thanks for putting this together, it has been working well otherwise!

@callikai Thanks for the contribution. I will test it out.


feature: api support aliyun billing report export to OSS

这个 PR 实现什么功能/修复什么问题:
aliyuncli支持订阅账单详情到指定的OSS bucket
是否需要 backport 到之前的 release 分支:
NONE

Codecov Report

Merging #1064 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #1064      +/-   ##
=========================================
+ Coverage    7.89%   7.92%   +0.02%
=========================================
  Files         356     356
  Lines       56362   56362
=========================================
+ Hits         4450    4464      +14
+ Misses      51516   51504      -12
+ Partials      396     394       -2

Flag
Coverage Δ

#aFlag
7.92% <ø> (+0.02%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/util/rbacutils/rabc.go
77.02% <0%> (+4.53%)
:arrow_up:

Continue to review full report at Codecov.

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

/approve
/lgtm


Update prerelease version for preview 6

@ajcvickers master is failing, can you fix this in the preview5 branch and merge the fixes forward to master?


[otlLib] Emit empty ComponentRecords

@behdad, are these two equivalent? Currently, otlLib emits the former; see the unit test in https://github.com/behdad/fonttools/commit/036e2ce497da4468c3f1665b941b9cfb9727e438. If there was no semantic difference, it would be easy to emit the latter.
Variant 1, current output:
<LigatureAttach index="0">
    <!-- ComponentCount=2 -->
    <ComponentRecord index="0">
        <LigatureAnchor index="0" empty="1"/>
        <LigatureAnchor index="1" empty="1"/>
        <LigatureAnchor index="2" empty="1"/>
        <LigatureAnchor index="3" empty="1"/>
    </ComponentRecord>
    <ComponentRecord index="1">
        [actual content]
    </ComponentRecord/>
</LigatureAttach>

Variant 2:
<LigatureAttach index="0">
    <!-- ComponentCount=2 -->
    <ComponentRecord index="0" empty="1"/>
    <ComponentRecord index="1">
        [actual content]
    </ComponentRecord/>
</LigatureAttach>

I can't imagine why one would want to do such a thing.  They are equivalent as far as HarfBuzz is concerned, though, things like OTS might not like it...  Not sure about Windows, but I think it should be safe...  What does makeotf do?


SCRUM-245: add spock testing and compatible mocking frameworks

Ticket:
https://agr-jira.atlassian.net/browse/SCRUM-245
Evidence of integration tests running:

With class:

And test (not an integration, but just for running something):

Class and Test only added to my local, not in pull request.

@abecerra so typically tests go in the src/test/java directory and have some default package lets say org.alliancegenome.curation_api.testing do you think you could give an example of how the mocking could work... plus it wasn't clear how the tests are being run?

the tests go in 'src/test/groovy'
the execution of the tests is done when building the project

"the execution of the tests is done when building the project" thats the issue. The integration tests cannot be run while building the project, only unit tests, unless we are going to setup a mock infrastructure @cmpich? Integration tests need to be run separately via a mvn verify or some other command... see the linked article.

need

"the execution of the tests is done when building the project" thats the issue. The integration tests cannot be run while building the project, only unit tests, unless we are going to setup a mock infrastructure @cmpich? Integration tests need to be run separately via a mvn verify or some other command... see the linked article.

Related findings in the comment:
https://agr-jira.atlassian.net/browse/SCRUM-245?focusedCommentId=31473

Closing because spock framework is not integrated to Quarkus yet.


Use module concatenation plugin and set NODE_ENV to production

#115
Bundle size went down from 1MB+ to 621KB (65KB behind metro-bundler) for fresh RN project.
We should see more size decrease if we stop transpiling ES imports/exports to CJS.

Update the snapshots and we're good


Async reqwest slower than old reqwest

Hello,
I'm upgrading my tool using reqwest but the async 0.10 version is way slower than 0.9.20 i was using before :
here is the two version of the code, the "new" one is way slower :
// new
extern crate reqwest; //  0.10
#[tokio::main]
async fn main() {
    let _urls: Vec<String> = reqwest::get("http://web.archive.org/cdx/search/cdx?url=google.com/*&output=text&fl=original&collapse=urlkey").await
            .expect("Error GET request")
            .text().await;

    println!("{}", _urls;
}

// new
real	0m20.230s
user	0m0.526s
sys	0m0.542s

// old
extern crate reqwest;  // 0.9.20

fn main() {
    let _urls: Vec<String> = reqwest::get("http://web.archive.org/cdx/search/cdx?url=google.com/*&output=text&fl=original&collapse=urlkey")
            .expect("Error GET request")
            .text()
            .expect("Error parsing response");

    println!("{}", _urls);
}

// old
real	0m6.547s
user	0m0.169s
sys	0m0.118s

I tried the blocking reqwest and they are slower in the last version of reqwest too

Hm, when I try the exact same URL you mentioned in reqwest's examples/simple.rs file, with --release, it takes about 8 seconds, which is around your previous results (my test machine is weak). If you try again now, do you still see an issue? What if you run cargo update?

Hello,
I retried and the old version takes from 5 to 6 seconds and the new one from 8 to 18 seconds.
There is still a huge difference and I tried a lot of time

I'd be curious to know which version of tokio you're using. In #763 a couple people (including myself) were seeing dramatically long request times, but things tightened up again with a later release of tokio.

I use the last version :
[dependencies]
reqwest = "0.10.1"
tokio = { version = "0.2.11", features = ["full"] }

I suppose the problem in "gzip" feature. It is not on by default.
So if I run "0.10.3" vs "0.9.24" and gather logs:
RUST_LOG=debug ./target/release/test_new_reqwest >/tmp/log_new
RUST_LOG=debug ./target/release/test_old_reqwest >/tmp/log_old

Then I ran awk and got:
$ grep bytes /tmp/log_new | grep read | awk  '{sum+=$5;}; END {print sum}'
2673086
$ grep bytes /tmp/log_old | grep read | awk  '{sum+=$5;}; END {print sum}'
398752

so new reqwest download 6x times more bytes via network, from slow enough server.
Then I change:
reqwest = { version = "0.10.3", features = ["default", "gzip"] }

and new reqwest download the almost the same amount of bytes: 398744 .
And timing almost the same.

Oh right, thanks for noticing about gzip no longer being enabled by default! Seems solved then.


provide way to apply configuration to specific ceph clusters.

Today we have cephcsi command line arguments like kernelmountoptions,fusemountoptions, etc. These options can also be specific to the ceph clusters that cephcsi is managing. If we set these command line arguments, this will be applied to all the ceph clusters cephcsi is connecting to, which will not work in some cases. The request is to make it configurable and identifiable based on the clusterID or something else.
### Tasks
- [ ] https://github.com/ceph/ceph-csi/issues/4161
- [ ] https://github.com/ceph/ceph-csi/pull/4245

@Madhu-1, I think we can close this as the subtasks are completed. Or do you have anything on this?


Chinese translations

Please add Chinese Translations

Hi @babaric-dev !
If you can provide the translations, I can add it to the app. I'll need something similar to this Pull Request where French translations were added.
The base files in English are here:

https://github.com/xa17d/saveto/blob/main/assets/playstore/details/details.xml
https://github.com/xa17d/saveto/blob/main/app/src/main/res/values/strings.xml


CLIP-1636: Fix the setup-go github action

Pull request description
Update setup-go action to the latest as it became flaky.
Checklist

[ ] I have added unit tests
[ ] I have applied the change to all applicable products
[ ] I have added the change description to the changelog.md and Chart.yaml files
[ ] (Atlassian only) I have run the E2E test (if applicable)
[ ] (Atlassian only) Internal Bamboo CI is passing

Adding e2e label to this PR end up to run e2e test using main` branch which result a failure because the fix still is not merged to main.
To prove that the test is fixing the problem I ran e2e test manually through the action workflow.


fix: table crash when data is null or undefined

This PR provides a fix for situation when tabledata is undefined or null,the table will crash with error 'TypeError: Cannot read property 'forEach' of undefined'

@tannerlinsley This is actually a big problem that is a pretty simple fix. If there are any null values in a table right now, it crashes the whole component

Since neither version 12 or the alpha have the fix merged, I have to maintain my own separate fork for my projects currently where the ability to handle null values is a requirement

@tannerlinsley This still needs a fix

Sorry, why can't you just pass data: data ?? [] to useTable?

@KevinVandy Its not that the whole table is null, just that the array of data has a particular value that is null. Then if you try to sort or filter the data, the whole component crashes

Yeah, null or inaccessible row values should either be filtered out or remapped/transformed into a homogenous row shape.

@tannerlinsley I mean I kinda disagree. First off, I'm pretty sure it says in the docs that a cell supports values of any valid JSON datatype, of which null is a perfectly valid JSON datatype. Sure I could do some transforms beforehand and cast all null values to strings, but of course "null" and null are not the same thing and shouldn't be treated the same. A row in a table should be able to have a column in which does not have a value for that particular row when data is incomplete, which it very often is in a lot of applications. It would be pretty silly for like microsoft excel to not allow you to have empty cells in a row and not very user friendly if the whole excel app froze and crashed when trying to generate a graph from an incomplete dataset or filter a column that had an empty cell
Secondly for like a table that might take user input to complete incomplete data, it is definitely more human readable as a literal absence of a value if there is no apparent value rendered in that row, like would happen for the datatype null, but of course if that was a string then to a human it might appear as though the row is complete in its data when it really isn't. Plus it is also possible for a user to manually enter a cell as blank, save it, and then if they try to filter it at that point, it will also crash.
Lastly, it just makes more sense, both for dev quality of life using the lib and for performance reasons. I understand having to remap or write transformations in order to ensure a homogenous row shape, which is normal for tables, but I think a lot of the advantages in using some of the hooks this library exposes are moot if I have to filter, sort, and/or transform normal json data types ahead of time before I am able to utilize the tables internal sorting, filtering, etc without my whole client crashing due to one table component (which it will literally do).
Plus things are never perfect, an api might return homogenous perfect data 99.9% of the time and then that .1% of the time a null value just so happens to be included in the response, the users whole client crashes, and you shouldn't need to have a bunch of client side transforms running on every api request for table data to compensate for 1 in 1000 requests.
I hope that makes more sense, thanks for your response so far. Happy to discuss this further too and contribute as needed. Seeing as there are a couple of different issues that reference these same things, I certainly don't think I am the only react-table user out there that feels this way

@tannerlinsley There are still more issues being opened about this exact problem, I would be happy to work on an improved solution, but I still feel strongly aligned to some of the sentiments and issues that this causes above^
Perhaps this functionality might still be considered? Maybe this initial implementation isn't quite the right fix, but it still really needs some sort of fix. Else I guess our companies projects will either try to find an alternative or publish and maintain our own patched custom version that fixes these issues and a couple more

You’re more than welcome to try your hand at a PR.

Any update on this?

Lastly, it just makes more sense, both for dev quality of life using the lib and for performance reasons.

💯 agree with this, it's safer to handle null/ undefined values at table state as a safe default behavior

@nicolad No dice as far as I know, my original PR that would fix the crashing on null/undefined got rejected and I haven't had time to put out a new one. I can see if I can work something up if you are also running into the issue


Fixes for RFC 1214

This PR updates dependencies and some traits to prevent future hard error of RFC 1214.
chrono 0.2.15 - also used to have Self with unsized traits, but in 0.2.16 that had fixed.

Thanks :beers: