Nullable for the solution

Implies usage of C# 8.0, instead of 7.3

see #184

I did not make the command line functionality nullable aware, as it has a "huge" complexity and some functionality which is not being used.
Is the source of command line functionality this repo dotnet/command-line-api?
Unfortunately that code base is not nullable aware.

Changing API shape (CompilationGenerator) also doesn't quite feel like it belongs in here. But that I'll leave up to @AArnott.

I have no problem to change it back,

I think, if this PR is true to it's name,  should be enabled in Directory.Build.props and then in projects that weren't/couldn't be migrated, disable it. Currently that'll be only Tests and Tool it seems?

I activated nullable globally. I found some interesting situations 🙂

To get rid of the build error

we could update the nuget packge StyleCop.Analyzers to 1.2.0-beta.164. see dotnetanalyzers/stylecopanalyzers#3098 or
suppress the warning or
disable nullable for the corresponding project

Once NRT is activated, I omit/remove null checks on all internal methods.
But as @amis92 says, I keep null checks present on public entrypoints.


Repo sync for protected CLA branch

The pull request is created from public637919669772847305 to public to fix git push error for protected CLA branch

  Too many files - PR blocked
This public repo applies limits on the number of files that can be merged in a single pull request (PR):

Warning Warning message and ability to unblock PR using the Merge large pull request* label - 30
Block Block message and PR is blocked - 100

This pull request will change 137 files, which exceeds the block limit of 30. This PR will not be merged.
To merge this pull request
You must either reduce the number of files being changed to below the 100 file limit or split the changes into multiple PRs. PRs must not have more than 100 files and fewer than 30 files if possible.
To reject this pull request
If you want to abandon these changes and not merge this PR, click Close pull request at the bottom of this page to close it without merging. Work with your team to revert any changes that aren't correct.
* You must have write or triage access to this repo to add labels.


Do not retry vmImage when checksum is bad

Problem:
When backing image CR checksum is bad, virtualmachine image still retry 3 times
Solution:
Make virtualmachine image not retry to download when checksum is bad.
Related Issue:
#5288
Test plan:

Build a Version CRD Object, but provide the incorrect isoChecksum
Upgrade the cluster
Check that upgrade failed in first round and not retry again.

From the issue statement, I think @irishgordo asks to display the checksum error when retrying. Not to eliminate the retries.
After all, a checksum error might indicate that there is something wrong during data transfer and worth retrying.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 23.99%. Comparing base (3f0705f) to head (0063e87).
Report is 3 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #5375   +/-   ##
=======================================
  Coverage   23.99%   23.99%
=======================================
  Files          94       94
  Lines       11652    11652
=======================================
  Hits         2796     2796
  Misses       8552     8552
  Partials      304      304

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

superseded by https://github.com/harvester/harvester/pull/5386


types.hpp issue when building

Hi Johan,
I'm trying to build your ICP version of RPCS3 from source (I get that I may be in the wrong thread for that, GH is unintuitive for dummies like me). I installed all the dependencies, made sure the environment variables were right, and the issue I'm running into all comes down to whatever 'bit' is and the file rpcs3\rpcs3\util\types.hpp . I know nothing about C++. Here's the log:
https://pastebin.com/s2rQDcWn
Thank you!

Looks like you are using an old version of visual studio which isn't compatible with RPCS3. In any case there is a windows build here under artifacts https://cirrus-ci.com/task/5704510594088960 which comes from the IPC pull request https://github.com/RPCS3/rpcs3/pull/10522. Just make sure you build the current KAMI code and you should be fine.


Clarification of sub_filters description for zmq4 and nanomsg

When going through the docs I initially confused sub_filters with being synonymous with an array of filter processors specific to zmq4

Nice one, thanks @ollieparsley!


python3Packages.ismartgate: init at 4.0.1

Motivation for this change
Python module to work with the ismartgate and gogogate2 API
https://github.com/bdraco/ismartgate
This is a Home Assistant dependency.

Target Home Assistant release: 2021.6.0

Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 125364 run on x86_64-linux 1

2 packages built:

python38Packages.ismartgate
python39Packages.ismartgate

Result of nixpkgs-review pr 125364 at 42c01efb run on aarch64-linux 1

2 packages built successfully:

python38Packages.ismartgate
python39Packages.ismartgate

I have cherry-picked this change into my staging tree, thank you!


Allow to register Domains in GCP

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details
When using Pulumi with TS for GCP, I want use "Cloud Domains" from the "Network services" to register a new custom domain. Right now I cannot do this. I have to manually go to the Google GCP Console and register the domain and connect it via UI with the DNS Zone (which I can create with Pulumi).
It would be great if we can add the feature of registering domains to Pulumi GCP.

Hi @besserwisser and thanks for reporting this enhancement. This appears to be a duplicate of #1061 so will be closing this in favour of consolidating all discussions about this on that issue.


⚠ Bofrost.lu has degraded performance

In 943b534, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2124 ms

Resolved: Bofrost.lu performance has improved in fd9f089 after 15 minutes.


Fix integer digit time bug

Digitized hit times are being floored to int in the trigger code, effectively being "rounded" to the previous nanosecond. This bug is present in the latest three tagged versions of WCSim v1.5.0, v1.5.1 and v1.6.0.
Below are distributions of digitized hit times WCSimRootCherenkovDigiHit::GetT() for 600 MeV electron particle gun MC generated with WCSim v1.4.0 (before the bug was introduced), the current state of the develop branch, and the code in this pull request.

Ok, looks like this came in with PR #89. @tdealtry and @stewartt1982, I also see that the timing windows for the trigger are integers (NDigitsPreTriggerWindow, for example) where they used to be floats. Is there functional reason to do this or was this unintentional?

I think this was a confusion between how digit time & trigger time were handled with the old code - trigger times are rounded (floored to 5 ns). This commit does the correct thing to line it up with what the old code did. But of course, in reality, electronics don't give digit timing to floating point precision, but are limited by clock ticks (e.g. to 2.5 ns). Whether we want to handle this correctly with some appropriate flooring, I'm unsure.
In terms of the user inputs (NDigitsPreTriggerWindow etc.) these could be made floats of course. (The values they were set to were all floating point representations of integers, so I guess that's why I made them int in #89)

Ok, I see. @tdealtry, do you know what the appropriate precision is in the actual electronics? Is it 5 ns? Or is it 2.5 ns? I am not opposed to moving away from floats in favour of making things more realistic, but then we should aim to round things in the most realistic way.

I think moving to floats is the appropriate thing to do, but also add a digitizer-specific rounding. According to Section 5.1 of the SK NIM paper, the timing resolution of a hit is 0.4 ns.

Great, allowing for digitizer-specific rounding is a better idea. Do you have time to implement this @tdealtry? If so, I'll close this pull request. Do you agree @cvilelasbu?

Fine by me.
On September 12, 2016 3:15:52 PM GMT+01:00, eosullivan<EMAIL_ADDRESS>wrote:

Great, allowing for digitizer-specific rounding is a better idea. Do
you have time to implement this @tdealtry? If so, I'll close this pull
request. Do you agree @cvilelasbu?
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/WCSim/WCSim/pull/188#issuecomment-246360959

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

Yeah, I'll be able to get to this within a couple of weeks

Great, I'll close this out then. Just for an idea of the timeline, I'm holding off on merging anything until  #178 is merged (because I want people to test their PRs against the new Geant version), but once that's in I hope to get back to merging the outstanding PRs here.


L2 finite element maptype errors & inconsistencies

I am interested in solving electromagnetics problems in 2D, with E field in H(curl) and B field in L2. The CurlInterpolator requires that the L2 space use MapType==INTEGRAL. But other parts of MFEM e.g. GridFunction::GetValue require  that the L2 space use MapType==VALUE. And some functions such as VectorFECurlIntegrator do not check the map type at all (it should check the MapType, and divide by |J| for MapType==INTEGRAL).

@v-dobrev, can you help fixing this? It could be something that affects other physics...

PR #1931 adds support for using CurlInterpolator with MapType == VALUE.  It also adds support for MapType == INTEGRAL in GridFunction::GetValue.
This leaves VectorFECurlIntegrator, and possibly other oversights, to be dealt with.

PR #1931 adds support for using CurlInterpolator with MapType == VALUE.  It also adds support for MapType == INTEGRAL in GridFunction::GetValue.
This leaves VectorFECurlIntegrator, and possibly other oversights, to be dealt with.

PR #1995 adds several unit tests to verify that the L2 basis functions with MapType == INTEGRAL work as expected.  These tests identified at least one issue which was also fixed in that PR.
The trouble with VectorFECurlIntegrator is that it is only designed to work in 3D.  In 2D, where the Curl is scalar-valued, the correct alternative is to use the MixedScalarCurlIntegrator (this only works in 2D).
It is possible that we still don't have unit tests for every integrator and interpolator.  To my knowledge we have not performed a comprehensive audit of these unit tests.  If any more problems arise please let us know.

PR #1995 adds several unit tests to verify that the L2 basis functions with MapType == INTEGRAL work as expected.  These tests identified at least one issue which was also fixed in that PR.
The trouble with VectorFECurlIntegrator is that it is only designed to work in 3D.  In 2D, where the Curl is scalar-valued, the correct alternative is to use the MixedScalarCurlIntegrator (this only works in 2D).
It is possible that we still don't have unit tests for every integrator and interpolator.  To my knowledge we have not performed a comprehensive audit of these unit tests.  If any more problems arise please let us know.


fix: compatible with TS 5.6

Porting changes to module specifier cache for typescript 5.6. Module specifier cache seems to change the most so I changed it to use the one in ts.server when possible, similar to createSymlinkCache. This should also work in the "typescript" import since it now includes the server namespace. So the port for 5.6 is mostly just a fallback.

Thanks for your hard work!


[OpenCL] Disables forward optimization with SYCL

Fixes test tensorflow/python/kernel_tests/unstack_op_test.py

I can confirm that this became useless. The test unstack_op_test passes without these changes even when we re-enable testGradientsAxis0 (which we should do by the way).


Adding a simple webhook plugin

@pilhuhn I've moved the previous webhook plugin as "webhook-global" as it was designed as pure gateway.
I've added a new one that I think it can suit on your needs.
Please let me know if this works for you, I've tested on standalone without issues.
@jshaughn for some reason, the itest profile doesn't work fine with last changes, I didn't receive any travis error, perhaps we need to take a look. Does it works for you (as I changed my machine, I don't discard that I have something wrong locally).
Thanks

@jshaughn strange. Yesterday I can't execute a mvn clean install -Pitest and I remember you commented me that it was an issue as well in your machine.
Today I ran another round and it passed.
Also travis doesn't show anything wrong.
So perhaps it's something related to the environment.


tileview in  fragment

Hello sir ,i want to use tileview as a fragment,can u please help me how to use ,i try but fragment not extend  tow classes at the same time.

a TileVie is a View, like any other.  there's nothing specific about TileView that would make any difference.  if you can use a View, you can use a TileView


Portal alias settings not available in multisite

Description of bug
If you have multiple sties within 1 DNN host, you can not access site alias settings of other sites than the one you're in.
Steps to reproduce
List the steps to reproduce the behavior:

Have more than 1 site
Login as host in portal 0
Manage - sites
Try to add an alias to another site than the one, you're logged in.
I think the second or third icon should give access to functionality like site aliasses and import / export but neither of the buttons work.

Expected result
Provide a clear and concise description of what you expected to happen.
Screenshots
Error log
Paste the error log that is related to this issue.
Additional context
Add any other context about the bug that may be helpful for resolution.
Affected version

[x] 9.2.2

Affected browser

[ ] Chrome
[ ] Edge
Other versions not tested

I can reproduce the issue in 9.4.0 RC, clicking the settings icon for site 1 brings you to the settings of site 0, but if you click back to sites and do it again, then the second time it works. Strange, but moving this issue to AdminExperience


feat(jwt): remove "token_name" and use the document.entry_path instead. Replace also format field by signing_type.

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

v2.2.0
GitHub release

Your semantic-release bot :package::rocket:


Issues from testing AudiAnnotate against Cookbook Recipe Manifests

Describe the bug
This ticket lists the issues found when testing the AudiAnnotate tool against the IIIF manifests in AV related cookbook recipes.
To Reproduce
Steps to reproduce the behavior:

Create AudiAnnotate project
Import the following manifests with "Import Existing Manifest"

Expected behavior
Import the manifest successfully and display the resources in the manifest and perform any functionality extracted from the manifest (e.g. use table of content to navigate the player)
Cookbook Recipes

https://iiif.io/api/cookbook/recipe/0015-start/manifest.json: Manifest import successful. Does not work: Playback starts at 0 not at the specified time.
https://iiif.io/api/cookbook/recipe/0017-transcription-av/manifest.json: Manifest import successful. Does not work, cannot play the media file. Video tag's src attribute is set to the text file listed under the rendering property in the manifest.
https://iiif.io/api/cookbook/recipe/0219-using-caption-file/manifest.json: Manifest import breaks with the error: "no implicit conversion of String into Integer".
https://iiif.io/api/cookbook/recipe/0074-multiple-language-captions/manifest.json: Manifest import breaks with error: "undefined method `[]' for nil:NilClass".
https://iiif.io/api/cookbook/recipe/0064-opera-one-canvas/manifest.json: Manifest import successful. Does not work. No media playback is available for the second act (the second item on the canvas).
https://iiif.io/api/cookbook/recipe/0065-opera-multiple-canvases/manifest.json: Manifest import successful. Does not work. Displays table of content in the viewer, but cannot use it to navigate the player to desired section.

Notes
Testing matrix: https://docs.google.com/spreadsheets/d/1QBz7tJ1S-AnSpnAWZpNRm-FTOy-ZPcTfi1MVsRtHM44/edit#gid=0

On 1, there is a start element of the manifest which we do not respect.
We need to decide whether we should honor that element or whether it's out of scope.
More here: https://iiif.io/api/presentation/3.0/#start


Don't render star in signature before keyword-only params if signature has variadic positional param

Describe the bug
See https://github.com/machow/quartodoc/issues/117.
Quote:

What we could improve here (in quartodoc and mkdocstrings) is that in case of both variadic positional and keyword-only parameters, we shouldn't add a single star to the signature, as doing so in the code raises a SyntaxError and therefore is not possible:

def f(a, *b, *, c): ...
             ^
SyntaxError: * argument may appear only once

Expected behavior
No star in signature when it has both varpos and kwonly parameters.

It's actually already handled correctly.


expose a private field to Hibernate Reactive

needed to support Criteria queries

I think we need this on 5.4 as well

@DavideD done

Thanks a lot
On Tue, May 26, 2020 at 9:55 AM Andrea Boriero<EMAIL_ADDRESS>wrote:

@DavideD https://github.com/DavideD done
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hibernate/hibernate-orm/pull/3419#issuecomment-633898343,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEIQ5IBG5SWKFCD7VU4BELRTN7XJANCNFSM4NI6ROSA
.

Thanks @dreab8


Can't play video from card

Describe the bug
Whenever I press on the play button toast notification pops up: Failed to call service rest_command/pmha_playmedia. Service not found.
Checking the browser dev tools I can see that the following request to the server returns with 404 - Not found from the server.
http://<IP_ADDRESS>:32400/player/playback/playMedia?type=video&commandID=1&providerIdentifier=com.plexapp.plugins.library&containerKey=/playQueues/10&key=/library/metadata/2&offset=0&machineIdentifier=63b870c298b4bb8fa259f365fcee923d324b9b4b&protocol=http&address=<IP_ADDRESS>&port=32400&token=xxxxxxxx&X-Plex-Token=xxxxxxx.
Searching the library has no problem, showing details for the selected media has no problem, but playing the media does not work.
Expected behavior
The video should play on the configured android tv client.
[ x] I am running Home Assistant on HTTP protocol
[ ] I am running BOTH Home Assistant AND Plex on HTTPS protocol
[x ] My Plex server is accessible through protocol, port and ip/hostname I have specified for the card.
[ ] My Plex server is running on HTTPS and has a valid certificate.
Card Configuration
type: custom:plex-meets-homeassistant
protocol: http
sort: titleSort:asc
playTrailer: true
showExtras: true
showSearch: true
ip: <IP_ADDRESS>
token: xxxxxxxxxxxx
port: '32400'
entity:
  - >-
    plexPlayer | BRAVIA VH1 | <IP_ADDRESS> |
    cb21fc709ba53b31-com-plexapp-android
libraryName: Konzerte
maxCount: ''
maxRows: ''
useHorizontalScroll: 'No'
useShuffle: 'No'
displayTitleMain: 'Yes'
displaySubtitleMain: 'Yes'
minWidth: ''
minEpisodeWidth: ''
minExpandedWidth: ''
fontSize1: ''
fontSize2: ''
fontSize3: ''
fontSize4: ''
minExpandedHeight: ''
title: ''
runBefore: ''
runAfter: ''

Card Version
3.6.8
Plex Version
<IP_ADDRESS>49 running on Windows 11 Build 22621.2283
Device information
Surface Laptop Studio running Chrome Browser with Version 116.0.5845.188

I have this same problem, and I can.t figure it out.

I have this same problem

I have the same issue

Same issue here

Experiencing the same issue, tried a number of different configurations and couldn't make (a previously working) setup work again. I am configured with HTTPS, and confirmed valid tokens etc.; however, it would appear that the endpoint must have changed here on Plex's end in a recent server update.

Same issue for me. Using HTTP and Rokus

https://github.com/JurajNyiri/PlexMeetsHomeAssistant/issues/32#issuecomment-908614426


Introduce hashed passwords on login response & implement "reconnect" packet

I have recently confirmed my theory of packetId 2 being a login request with the custom password that the server set, during the login response of packetId 1. This means that the password sent back by the server was very likely hashed in the original.
Someday it would be very beneficial to implement this in hexagon as well.

Update: It turns out that the client authenticates normally over packetId 1, once it has done authentication through the reconnect packet. So that means we will have to get a bit creative here...
[2024-10-11 17:08:30] - <<IP_ADDRESS>:56856> DEBUG: -> 1: LoginRequest{Username: "Levi", Password: "afe99137967ee6f5ec8f08df89dd8076db932e13d4ca8d7b32feda3dc919ae3e1657cf30616523d28500727755eddad5a6e18fe255b407a9c5f6dd8de1a43b99", ...}

Forget what I just said, I was just logging the wrong packetId


Insert Title Here

Hi! Steps to reproduce:

new notebook
click "speech bubble" in upper right ("Chat notebook settings")
click "Code Assistant" persona
use single-quote to create new chat-enabled notebook cell
type "which version of GPT is this?"
see error message

Describe the issue in detail here. Attach any relevant screenshots or files. The section below was automatically generated. Remove any information that you do not wish to include in the report.

Debug Data

Property
Value

Name
"Wolfram/Chatbook"

Version
"1.0.4"

EvaluationEnvironment
"Session"

FrontEndVersion
"Desktop: 13.3 for Mac OS X x86 (64-bit) (July 24, 2023)"

KernelVersion
"13.3.1 for Mac OS X x86 (64-bit) (July 24, 2023)"

SystemID
"MacOSX-x86-64"

Notebooks
True

DynamicEvaluation
False

SynchronousEvaluation
False

TaskEvaluation
False

Settings

Property
Value

Assistance
True

Stack Data
catchTop
checkResponse
throwFailure
trimURL

Failure Data
<|
  "Evaluation" :> Wolfram`Chatbook`Actions`Private`toolFreeQ[ProgressIndicator[Appearance -> "Percolate"]]
  "Arguments" -> {}
|>

This was fixed in #379.
Evaluate the following to get the latest updates:
PacletInstall["Wolfram/Chatbook", UpdatePacletSites -> True]


Raise when trying to fetch response header with multiple values

Hi! 👋
I came across this library on Twitter, started using it and wanted to see if I could start contributing somehow.
This seemed like the only missing part for #194 so I wanted to create a PR, unless I misunderstood something. :)

Thank you!

We still need to document the feature but let's hold off because I have heard from some people that it might not be such a good idea after all. No rush!


Fixed parsing of matpower files that don't end in newline

The function parse_matlab_string created line indices based on how many
"\n" characters were present in the file. There was an off-by-one error
whenever the last character wasn't "\n".  The code would have not worked
at all if a file used line endings of "\r" (pre Mac OS X).

Please check my changes.  If you agree then I can also submit a PR to PowerModels.

Codecov Report

Merging #187 into master will decrease coverage by 0.06%.
The diff coverage is 55.55%.

@@            Coverage Diff             @@
##           master     #187      +/-   ##
==========================================
- Coverage   58.06%   57.99%   -0.07%
==========================================
  Files          44       44
  Lines        3737     3738       +1
==========================================
- Hits         2170     2168       -2
- Misses       1567     1570       +3

Impacted Files
Coverage Δ

src/parsers/pm_io/matpower.jl
51.27% <100%> (ø)
:arrow_up:

src/parsers/im_io/matlab.jl
90.06% <33.33%> (-1.94%)
:arrow_down:

Continue to review full report at Codecov.

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


getPrimaryKeys now working for views

Description
getPrimaryKeys in sqlite-jdbc raises SQLException with message "Table not found: '...'". In other jdbc (mysql, postgres) it returns empty list. Metabase (BI app) depends on this, so I'd like to fix this in driver.
Metabase stacktrace
 [{:type java.sql.SQLException    :message Table not found: 'view_table'    :at [org.sqlite.jdbc3.JDBC3DatabaseMetaData$PrimaryKeyFinder <init> JDBC3DatabaseMetaData.java 1886]}]

Fixed in #239. THanks


VM creation script

Vagrant ruby action: https://github.com/tomjn/vagrant-tart-ruby/blob/main/lib/vagrant/tart/action/create_instance.rb
driver function: https://github.com/tomjn/vagrant-tart-ruby/blob/main/lib/vagrant/tart/driver.rb#L42
shellscript: https://github.com/tomjn/vagrant-tart-ruby/blob/main/lib/vagrant/tart/scripts/create_vm.sh

To do this we'd need to run the tart import command and give it the vagrant box file. Currently tart doesn't have this command at the time of writing but it was added in https://github.com/cirruslabs/tart/pull/386 and should be released soon.
Previously, the plan was to copy the VM from inside the box file into Tarts VM folders with the vagrant ID and a prefix, then use tart run to start it. I've successfully cloned Tart VMs manually this way. There's also tart clone which can duplicate VMs, but this runs the risk of users manually starting box VMs and modifying them, then all subsequent VMs being broken or changed.

Similar to the Hyper-V scripts, the shellscript should get called with the VM's intended ID as decided by vagrant as the parameter


Plug.Conn.chunk does not send the chunk to client

Hello my phoenix app has an endpoint for user to subscribe to and get a stream of data back over time, it worked before using cowboy adapter but it does not work after switching to use bandit adapter, I logged the return of chunk/2 and it looks like it is working but it is not.
After the client connects to the endpoint, it waits indefinitely and receives no data back. Below is the log
2024-03-22T15:38:36.460Z,[INFO]: Chunked 200 in 103ms
{:ok,
 %Plug.Conn{
   adapter: {Bandit.HTTP1.Adapter, :...},
   assigns: %{},
   body_params: %{},
   cookies: %Plug.Conn.Unfetched{aspect: :cookies},
   halted: false,
   host: "localhost",
   method: "GET",
   owner: #PID<0.5005.0>,
   params: %{
     "merchant_id" => "1",
     "transaction_id" => "f5726640-1226-446a-9fc3-39e101e7ecce"
   },
   path_info: ["api", "user", "get_notify_on_progress"],
   path_params: %{},
   port: 4000,
   private: %{
     :user => %{},
     MyAppWeb.Router => [],
     :plug_session_fetch => #Function<1.76384852/1 in Plug.Session.fetch_session/1>,
     :before_send => [#Function<0.54455629/1 in Plug.Telemetry.call/2>],
     :phoenix_endpoint => MyAppWeb.Endpoint,
     :phoenix_router => MyAppWebWeb.Router,
     :phoenix_format => "json"
   },
   query_params: %{
     "merchant_id" => "1",
     "transaction_id" => "f5726640-1226-446a-9fc3-39e101e7ecce"
   },
   query_string: "merchant_id=1&transaction_id=f5726640-1226-446a-9fc3-39e101e7ecce",
   remote_ip: {127, 0, 0, 1},
   req_cookies: %Plug.Conn.Unfetched{aspect: :cookies},
   req_headers: [
     {"sec-fetch-site", "same-origin"},
     {"sec-fetch-mode", "cors"},
     {"sec-fetch-dest", "empty"},
     {"cookie",
      "csrftoken=k7c26Fow3gCkeScDkoZaFXVMyGTvP6B5; rl_anonymous_id=RudderEncrypt%3AU2FsdGVkX19LNG05sNFFBqAXXNCIlNn%2Fx85AdL1hQWnQN2KI3PLjY2ntovdTzct7MyTsDywCwROomNp0QlX%2FCg%3D%3D; rl_page_init_referrer=RudderEncrypt%3AU2FsdGVkX1%2BYfPZGNTNZs6lsJSo0NTdXrgL9wtDXdVk%3D; rl_page_init_referring_domain=RudderEncrypt%3AU2FsdGVkX19aCHDTVKdRvpIMh6FC%2FWXFGiFDY9cridQ%3D; MMUSERID=ohjntuxmg7gwxn8z9ztu1dzpfh; MMCSRF=a6gqxg1qgtb6pkdibkny1f84za; MMAUTHTOKEN=bbbhphunqjy3dg4unjueedn88y; rl_user_id=RudderEncrypt%3AU2FsdGVkX1%2FMMutUQ86kPj3bBwNCV%2B4Wy%2FHTNcxjNsGJ67WAOtt7hw2%2Fpngdzh3K; rl_trait=RudderEncrypt%3AU2FsdGVkX19hYLEpncOhCyQYcmeMQQFeYl5G6Qt%2FVc4%3D"},
     {"connection", "keep-alive"},
     {"dnt", "1"},
     {"x-user-id", "1c1d6cda-e8f0-4937-a94a-ddf594321b72"},
     {"accept-encoding", "gzip, deflate, br"},
     {"accept-language", "en-US,en;q=0.5"},
     {"accept", "*/*"},
     {"user-agent",
      "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:123.0) Gecko/20100101 Firefox/123.0"},
     {"host", "localhost:4000"}
   ],
   request_path: "/api/user/get_notify_on_progress",
   resp_body: nil,
   resp_cookies: %{},
   resp_headers: [
     {"cache-control", "no-store"},
     {"x-request-id", "F78JZdKGNMG1mHwAAHCC"},
     {"content-type", "text/event-stream; charset=utf-8"}
   ],
   scheme: :http,
   script_name: [],
   secret_key_base: :...,
   state: :chunked,
   status: 200
 }}

here is the handler for the endpoint
defmodule MyAppWeb.SubscribeForProgress do
  alias MyApp.Account

  def subscribe_for_progress(conn, %{"user_id" => user_id, "transaction_id" => transaction_id}) do
    with :ok <- Account.get_notify_about_progress(user_id, transaction_id),
      conn <- prepare_sse_conn(conn),
      {:ok, conn} <- wait_for_progress_notify(conn)
    do
      conn
    end
  end

  defp prepare_sse_conn(conn) do
    conn
    |> Plug.Conn.put_resp_content_type("text/event-stream")
    |> Plug.Conn.put_resp_header("cache-control", "no-store")
    |> Plug.Conn.send_chunked(200)
  end

  defp wait_for_payment_progress_notify(conn) do
    # Plug.Conn.chunk(conn, "hello") |> IO.inspect()
    # Plug.Conn.chunk(conn, "world") |> IO.inspect()

    receive do
      {:transaction_done, payload} -> Plug.Conn.chunk(conn, Jason.encode!(payload))

      {:plug_conn, :sent} -> wait_for_payment_progress_notify(conn)
    end
  end
end

sorry there was a typo bug on my side, closing this

Curious; were you missing a place
Where you should have been grabbing the conn from a function return?

Curious; were you missing a place Where you should have been grabbing the conn from a function return?

hello, so the get_notify_about_progress function just call phoenix pubsub to subscribe a topic and there was typo in the topic name that's y I did not getting any messages


[Bug]: bigger size of mushroom card in sections type of dashboard

Requirements

[X] I checked the troubleshooting section in the README to verify that I have the latest Mushroom version.
[X] I did a search to see if there is a similar issue or if a pull request is open.
[X] I have read the state of card mod support (in pin issues) and understand card mod support is not provided in this repository.

Current Behavior
I use Mushroom cards for quite a while. With HA 2024.3 release I wanted to take a look at their new type of dashboard ("sections"). Updated Mushroom cards to v3.5.2 and started copying items.
Things work as expected but sizes differ in a visible way. I do not know is it HA fault or Mushroom cards.
Masonry type of dashboard:

Sections type of dashboard:

Expected Behavior
I would rather expect the same size. But 'sections' dashboard type is marked as experimental so probably this is the reason.
Steps To Reproduce

create 'sections' dashboard
add Mushroom card
compare with same card on 'masonry' dashboard

Context
YAML state

Environment
- Browser: Firefox
- HA Version: 2024.3.0
- Mushroom: v3.5.2

Anything else?
No response

Same issue for me.
It is most visible when using two chips-stacks below each other.

Seeing the same with HA 2024.3 + Mushroom 3.5.2 and sections layout.

It looks like in the sections layout, there is a parent div wrapper around the mushroom card container, which has a fit-rows and a card class applied to it. That selector defines only one attribute: height
.card.fit-rows {
    height: calc((var(--row-size, 1)*(var(--grid-row-height) + var(--grid-gap))) - var(--grid-gap));
}

When this style is disabled in dev tools, the card renders as expected, and same as in the masonry layout.

@ptashek I fixed your issue with when using multiline_secondary options with https://github.com/piitaya/lovelace-mushroom/pull/1402.
It's available in 3.5.4 release.
@hrw The sections view is still experimental and cards are forced to fit the grid for now so it's not a Mushroom issue.
As I'm working in the HA front-end team, I can tell you that we are thinking about a way to resize card using the UI.
For now, using this yaml code on the card should fix your issue.
layout_options:
   grid_rows: auto

@piitaya your suggestion about the grid rows, would this also fix the gap we now have while using chips and the new sections layout? cfr example below.
Because adding
card_mod:
  style: |
    ha-card {
      margin-bottom: -66px;
    }

to the chips card has no effect.
But I do not understand your  ayout_options suggestion, where and how should this be added?

i have the same issue

i  found a hacky way to fix it by going into developer tools and override the --row-height variable to 33px on the .container element that contains the chip cards but i havent found a way to fix this using the card mod (because it's outside the mushroomcard element)

the result

Maybe correlated with the fact that the card do not fill orizontally the column even if "fill container" is selected?

I just came here to report the problem with single-row chip cards, and noticed that layout_options: { grid_rows: auto } doesn't solve the issue (it's also not a solution for empty auto-entites, although both are related to the variable below).
I did indeed notice that --row-height seems to be the culprit - I'm not sure why it is so big, as changing it doesn't seem to affect other cards, since most of them are usually bigger than the 66px in height. That said, that variable should go back to 66px when in edit mode, otherwise the user might have a hard time hitting the edit button.

A workaround for this issue with chips is adding them to a vertical-stack with the previous/next card; but this is not feasible in my scenario since I'm using the new Visibility option, and that's not available for sub-cards right now.

As a workaround for this, you could use a conditional card for each 'sub-card' in the vertical-stack (which, as I understand, works the same as the new visibility option).

Just a quick follow-up just in case somebody arrives here with the stably released sections. The following is NOT working anymore:

For now, using this yaml code on the card should fix your issue.
layout_options:
   grid_rows: auto

you need to use the following syntax now:
grid_options:
  rows: auto

(or just use the layout UI to change the value to let the UI generate the code and then change the number to "auto" in the YAML editor of the card)


Gowin: deal with active-low tristate

Now that we're adding support in Apicula for inout and tristate buffers we're finding that they are actually active-low, causing inverted behavior.
This PR matches the behavior of Nexus explained in #2916 by using an intermediate cell.
It also moves the iopadmap earlier into the process, so that there is more room to optimize away the inverter.

Hm so on CI it errors on the LUT1 which is easy enough to fix, but locally it fails on the opt_equiv assertion, which I'm not sure how to deal with.


Update subkey in extension INFs to proper value per documentation.

Testing of this change is still ongoing.


check for null after use

Version

Phaser Version: 3.52.0
Operating system: Linux
Browser: Chrome

Description
While attempting to address an issue in my code, I stumbled into this Phaser code:
var DOMElementCSSRenderer = function (renderer, src, camera, parentMatrix)
{
    var node = src.node;
    var style = node.style;
    var settings = src.scene.sys.settings;

    if (!node || !style || !settings.visible || GameObject.RENDER_MASK !== src.renderFlags || (src.cameraFilter !== 0 && (src.cameraFilter & camera.id)) || (src.parentContainer && !src.parentContainer.willRender()))
    {

Here !node is checked only after node is dereferenced in node.style. This seems incorrect.
Example Test Code
In my case, src.node was undefined, as I was calling const element = this.add.dom(400, 600).createFromCache('') (note the empty string) instead of specifying the key for the cached html. The surprising thing is that I copied that code from the example here, which seems to work.

Thank you for submitting this issue. We have fixed this and the fix has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.


⚠ RPC server - Harmony has degraded performance

In 22fd7cc, RPC server - Harmony ($HARMONY_RPC_URL) experienced degraded performance:

HTTP code: 0
Response time: 0 ms

Resolved: RPC server - Harmony performance has improved in 9da2498.


Clean-up binary field skipping.

This code was never updated to replace raw math with FieldTag when
that type was introduced.

Looks good!


Fix deadlock isssue with concurrent.futures.wait()

Ran into this issue with using concurrent.futures.wait(): https://bugs.python.org/issue20319. To avoid this bug, wait() is reimplemented by iterating over futures and calling result().
We ran into this issue on one of our automated test boxes, but I was able to reproduce this on a Windows EC2 instance that I have running and can confirm this PR no longer results in the deadlock.
I added no tests because they were already tests covering the use cases involving pending_main_kwargs which is where the wait() is used: https://github.com/boto/s3transfer/blob/develop/tests/unit/test_tasks.py#L431-L581 and for the SubmissionTask: https://github.com/boto/s3transfer/blob/develop/tests/unit/test_tasks.py#L81
I looked into adding a stress unit test: https://github.com/kyleknap/s3transfer/commit/4b743ad00a6e3663b3610ab086ffc277b4f60c76 but that was not reliably causing the repo case unless I created like 10,000 tasks which then caused it to hang like 1/5 times and took like over 10 seconds which was not ideal.
So I stuck with adding performance scripts for detecting hangs.
cc @jamesls @JordonPhillips

Can we move the script you added out of scripts/performance into either scripts/ or scripts/stress or something else?  This isn't really a perf test, so I don't think it makes sense in performance/.
Otherwise, :shipit:

Yeah I can move it.


Feature/working branch

Having completed the mandatory work for this assignment, the authors request to merge our work into development. The following work has been done:

Todo App created using Webpack and ES6.
Projects and tasks can be created using a form.
Tasks belongs to a project.
Local storage is used to save data across the user's sessions.
Live server demo.
A descriptive README has been written to accompany the project.

With regards to @mikenath223's requested changes:
Tasks do belong to projects.
When creating a task, the task is automatically assigned to the currently selected project. A project can be selected by clicking on its card in the left hand column, which is represented visually by a change in background/border.
Any task which is created while that project is selected is automatically inserted into that project's tasks array, and all tasks displayed in the task column belong to the currently selected project.
I include here the two screenshots in our README.md file which show how selecting a project displays that project's tasks.


Support macOS Big Sur and Apple Silicon hardware platform

This issue is to track the progress of the Apple Silicon hardware support.
Tensorflow recently released it with v2.5.0.
Tfjs is tracking this hardware support with this issue and will probably get an update soon.
Danfojs is currently using "@tensorflow/tfjs-node": "3.3.0".
As soon as we get an update for @tensorflow/tfjs-node, danfojs can version bump the dependency and be used on new Apple devices.

@chomatdam, are there any updates on this issue? I still get a Unsupported system: cpu-darwin-arm64 message.

@chomatdam can you show the full error response you recieved.

Have there been any updates here? Given that apple silicon isn't going away, there's a lot of potential impact in fixing the issue

Same here, the full error is this:
Exit code: 1
Command: node scripts/install.js
Arguments:
Directory: /*path-to-the-project*/node_modules/danfojs-node/node_modules/@tensorflow/tfjs-node
Output:
CPU-darwin-3.6.1.tar.gz
* Downloading libtensorflow
/*path-to-the-project*/node_modules/danfojs-node/node_modules/@tensorflow/tfjs-node/scripts/install.js:102
    throw new Error(`Unsupported system: ${libType}-${platform}-${os.arch()}`);
          ^

Error: Unsupported system: cpu-darwin-arm64

It seems to be related to the installation scripts of @tensorflow/tfjs-node see here.
Can we make this library a peer dependency so that can be updated as needed? My proposition makes sense?

Great suggestion, but making this a peer dependency does not solve the current issue.

Have there been any updates here? Given that apple silicon isn't going away, there's a lot of potential impact in fixing the issue

Sadly no official fix yet, but I keep tracking this issue for any.

Have there been any updates here? Given that apple silicon isn't going away, there's a lot of potential impact in fixing the issue

Sadly no official fix yet, but I keep tracking this issue for any.

It seems to be fixed now


Install sharp for Image Optimization

What feature would you like to have?
Install sharp so Image Optimization works better.
What problem will this feature solve?
From Next.js docs:

For a production environment, it is strongly recommended that you install the optional sharp library by running yarn add sharp in your project directory.

Do you want to help make the feature?
Yes
Is there anything else we need to know?
No response

I think this is finally not necessary. They have clarified this is automatically done when deploying on Vercel platform: https://github.com/vercel/next.js/pull/28476/files

They have clarified this is automatically done when deploying on Vercel platform: https://github.com/vercel/next.js/pull/28476/files

Even better, we get all of the benefits in our production environment, without having to deal with dependency updates or configuration.


Fix a typo in URL

The last PR has a typo in URL, sorry about that

Codecov Report

Merging #393 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #393   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          42     42
  Lines         481    481
=====================================
  Hits          481    481

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 3e0ee48...30690a2. Read the comment docs.


Bug in Conv1D conversion when padding type is same

Hi Team,
I was trying to convert a model with conv1d and padding type "same" when I countered below error:
Traceback (most recent call last):
  File "test_model.py", line 11, in <module>
    onnx_model = keras2onnx.convert_keras(model)
  File "/usr/local/lib/python3.7/site-packages/keras2onnx/main.py", line 102, in convert_keras
    return convert_topology(topology, name, doc_string, target_opset, channel_first_inputs)
  File "/usr/local/lib/python3.7/site-packages/keras2onnx/topology.py", line 309, in convert_topology
    cvt(scope, operator, container)
  File "/usr/local/lib/python3.7/site-packages/keras2onnx/ke2onnx/conv.py", line 226, in convert_keras_conv1d
    convert_keras_conv_core(scope, operator, container, is_transpose, n_dims, input_perm, output_perm, weight_perm)
  File "/usr/local/lib/python3.7/site-packages/keras2onnx/ke2onnx/conv.py", line 175, in convert_keras_conv_core
    len(input_shape))) if channels_first else input_perm_axes)
  File "/usr/local/lib/python3.7/site-packages/keras2onnx/ke2onnx/conv.py", line 40, in _calc_explicit_padding
    print((input_size[i] - 1))

Changing to
 if count_dynamic_dim(input_shape) > 0: # instead of 1

in below line fixes the issue:
https://github.com/onnx/keras-onnx/blob/383e431583d9a67da67d5a01f401b9f4df26ae4a/keras2onnx/ke2onnx/conv.py#L146
Here is a small code snippet, that demonstrates the problem:
import tensorflow as tf
import keras2onnx
import onnx

x = tf.keras.Input(shape=(None, 4), batch_size=1, name="input") # input of shape (n,d,c)
y = tf.keras.layers.Conv1D(32, 3, strides=1, activation='relu', padding="same", name="output")(x)

model = tf.keras.Model(inputs=[x], outputs=[y])
model.summary()
model.save("just_conv.h5")
onnx_model = keras2onnx.convert_keras(model)
onnx.save(onnx_model,  "just_conv.onnx")

Is it deliberate to keep count_dynamic_dim(input_shape) > 1 or this is a bug?

Thanks for pointing this out! I feel that count_dynamic_dim(input_shape) > 1 is for Conv2D case, because the first dim of input_shape is always None (it is a batch dim). Looks Conv1D should use 0 here, not 1.


RuntimeError: Set changed size during iteration

Information

qiskit-ibm-provider version: 0.5.3
Python version: 3.8
Operating system: Ubuntu Linux Focal

What is the current behavior?
Trying to run a simple circuit job against alt_ithaca_stretch failed with this:
        # Create name to Qiskit instruction object repr mapping
>       for name in all_instructions:
E       RuntimeError: Set changed size during iteration

.tox/production/lib/python3.8/site-packages/qiskit_ibm_provider/utils/json_decoder.py:136: RuntimeError
------------------------------------------------------------------------------------ Captured log setup -------------------------------------------------------------------------------------
INFO     /home/osboxes/ibmq/iqp-channel-e2e-tests/tests/conftest.py:conftest.py:15 getting provider
INFO     /home/osboxes/ibmq/iqp-channel-e2e-tests/tests/conftest.py:conftest.py:17 got provider
----------------------------------------------------------------------------------- Captured stderr call ------------------------------------------------------------------------------------
json_decoder.target_from_server_data:WARNING:2023-05-31 14:51:23,846: Definition of instruction ym is not found in the Qiskit namespace and GateConfig is not provided by the BackendConfiguration payload. Qiskit Gate model cannot be instantiated for this instruction and this instruction is silently excluded from the Target. Please add new gate class to Qiskit or provide GateConfig for this name.

That's in this for loop over a set:
https://github.com/Qiskit/qiskit-ibm-provider/blob/114a216034a1186c1833b04753fd2a30eb162d09/qiskit_ibm_provider/utils/json_decoder.py#L135
And the set can be modified here:
https://github.com/Qiskit/qiskit-ibm-provider/blob/114a216034a1186c1833b04753fd2a30eb162d09/qiskit_ibm_provider/utils/json_decoder.py#L158
Which results in the failure.
For example:
$ python
Python 3.8.10 (default, Mar 13 2023, 10:26:41)
[GCC 9.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> test = {1,2,3}
>>> for item in test:
...     test.remove(item)
...
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
RuntimeError: Set changed size during iteration
>>>

Steps to reproduce the problem
Run a job against alt_ithaca_stretch.
This is the test program script:
qc = QuantumCircuit(2)
    qc.h(0)
    qc.cx(0, 1)
    qc.measure_all()

    LOG.info("start")
    backend = provider.get_backend(device, instance=PROVIDER)
    LOG.info("get_backend")
    qc = transpile(qc, backend=backend)
    LOG.info("transpile")
    job = backend.run(qc)
    LOG.info("submitted job")
    job.wait_for_final_state()
    LOG.info("wait_for_final_state")
    counts = job.result().get_counts()
    LOG.info("got results")
    assert set(counts.keys()) == set(['00', '01', '10', '11'])

What is the expected behavior?
RuntimeError shouldn't happen.
Suggested solutions
Don't manipulate a set while iterating over it.

@nkanazawa1989 Can you take a look at this? Is there an issue with the backend target code or does the config of alt_ithaca_stretch need to be updated?
I was able to run the sampler program (qiskit-ibm-runtime) from this backend without any issues


Implement SCTEP

Check Singlet_fission.ipynb

[ ] new projection op

S1x, S1y, S1z = ry.spinops(0, 2, spin=3)
S2x, S2y, S2z = ry.spinops(1, 2, spin=3)

[ ] new hamiltonian

HZFSA = ry.HamiltonianZFS(D, E, 0)
HZFSB = ry.HamiltonianZFS(D, E, 1)

Note: ZFS = Zero Field Splitting (more complete dipolar hamiltonian)

[ ] mary has no time evol
[ ] (maybe) new kinteics op (but probably daijoubu)
[ ] New kinetics

Ks = ry.Kinetics(2, ks, model="Haberkorn-RTPsinglet")
Kd = ry.Kinetics(2, kd, model="Haberkorn-RTPfree")

Triplet exciton pair.
HZ, HZFS1, HZFS2, HJ*.
Check where self.radicals is not used (i.e. self.particles).
Think about just implementing HZFS().
Custom molecule "Triplet".


chore(test): refactor tests to use ESM syntax

Changes

Checklist

[ ] Added markup tests, or they don't apply here because...
[ ] Updated the changelog at CHANGES.md

@joshgoebel note that I haven't updated test/api/index.js, and the test suite is completing with code 0 instead of returning an error, we probably have to change mocha configuration for this kind of behaviour is reported as an error.

No idea, is it an async issue?

Hum apparently that was because I was using await import() instead of static import in some files. I think this is ready now.

Now highlight.js and highlight.cjs both seem to run?

Oh wait because you included both... is that intentional?  I thought we only needed the CJS one for the one thing?

It was intentional, I think we want to make sure both ESM and CJS users are able to run the library. It's more an integration test than a unit test spirit.

But when I merge our work (and rename the files back to .js) I get errors:
https://github.com/joshgoebel/highlight.js/tree/esm-tests-more-work
I find the error entirely opaque:
Writing highlight.js

><EMAIL_ADDRESS>test
> mocha test

SyntaxError[ @/Users/jgoebel/work/highlight.js/test/index.js ]: Unexpected identifier
    at Loader.moduleStrategy (node:internal/modules/esm/translators:147:18)

I agree the error message is unclear. From what I understand, this is due to syntax errors in the build/lib/index.js file:
import hljs from './core.js';

hljs.registerLanguage('1c', import L_1c from './languages/1c.js';);
hljs.registerLanguage('abnf', import L_abnf from './languages/abnf.js';);

It seems the require calls are being replaced by import in a "dumb" way.

You need to replace https://github.com/joshgoebel/highlight.js/blob/22ec6ffbf67092b91eb612c526a0183669dce36b/tools/build_node.js#L21
With
    return `${require};hljs.registerLanguage('${lang.name}', ${importName});`;

But then we run into the issue that the module doesn't have default export…

https://github.com/joshgoebel/highlight.js/blob/22ec6ffbf67092b91eb612c526a0183669dce36b/tools/build_node.js#L21
This is not very robust, it replaces only the first dash. It would be worth doing instead:
    const importName = "L_" + lang.name.replace(/\W/g,"_");

If I import first then use the identifier as you suggest then I just hit:
Error [ERR_REQUIRE_ESM]: Must use import to load ES Module: /Users/jgoebel/work/highlight.js/build/lib/index.js
require() of ES modules is not supported.
require() of /Users/jgoebel/work/highlight.js/build/lib/index.js from /Users/jgoebel/work/highlight.js/test/api/highlight.cjs

Yes, highlight.cjs test needs to be updated to use import('#hljs') instead of require('#hljs'). Do you want me to update #2 with this change?

I thought the only reason cjs existed was to test requiring the library?  But that's not possible?  I'm confused, sorry.

require() of ES modules is not supported.

I thought you said this "just worked"... how can we publish an ES6 library if it can't be required by people?  I feel like I'm missing something.

CJS users would need to import() it rather than require() it. Take a look at 2d8c9b7 to see what I'm talking about.

Hmmm... If I remove the async/await I'm stuck with:
   1) .highlight()
       should works without continuation:
     TypeError: Cannot read property 'highlight' of undefined
      at Context.<anonymous> (test/api/highlight.cjs:7:25)
      at processImmediate (node:internal/timers:464:21)

CJS users would need to import() it rather than require()

So does this mean users must also be forced to make sure their call site is async?

They can either use a bundler, switch to ESM, or make their API async.

They can either use a bundler, switch to ESM, or make their API async.

Ugh. I believe somewhere along the line I got confused (and maybe I didn't do enough testing) that since ES6 modules were "fully supported" now this would be more of a drop-in change... that sounds potentially like a LOT of work for a lot of people.   It seems like a lot less effort for the ES6 people to deal with a library being CJS than for CJS people to deal with a library being ES6.
I'm not sure I see the advantage of making this change for Node users on the server-side - and seems to force a lot of work on them.  Is there something here I'm missing?

Sorry I failed to set the correct expectations for the move to ESM. It is indeed a limitation with require, because it is sync it cannot load ESM modules reliably. There is an open PR on the Node.js repo to try to fix this (https://github.com/nodejs/node/pull/30891), however I don't think it's gonna happen any time soon.

It seems like a lot less effort for the ES6 people to deal with a library being CJS than for CJS people to deal with a library switching to ES6.

Let me disagree, working with CJS in the browser is a pain in my experience, you have to load a bunch of code to make it work somehow, or to use a tool that do that for you, or use the UMD bundle – either way you're pushing more JS to your users and have a (subjective) inferior developer experience.
Is it really a lot of work? It's not unheard-of that a library switches to a Promise-based API on a semver-major bump, and we are not even doing that here: for folks who already use a bundler, it's going to be completely transparent.
For folks who are using CJS and can't/don't want to switch to an async model, I guess they need to keep using v10 for the time being (same as people that need/want to keep supporting Node.js v10 after it's EOL).

I'm not sure I see the advantage of making this change for Node users on the server-side (is there any??)

The two systems are very different, I think it's very subjective which are advantages and which are limitations. The key feature for me is the ability for me to use the same syntax/module loading mechanisms for both browser and server development without setting up any tool.

if the ecosystem was truly ready for a "painless" upgrade from CJS to module, but it's starting to feel like it isn't.

I tried to find surveys that would help us finding out how spread out ESM is in the ecosystem, but couldn't find anything recent enough. If I was following my guts, I'd say we should do it, CJS is clearly a legacy system that is probably going to fade out.

Anyway, if we are not happy with the import() experience with CJS, here's a couple of thing we can do to help smooth the transition if we stick with distributing ESM:

Document how to transpile the library to CJS and let users deal with it. (I thought that was the way we've agreed upon).
Provide a CJS bundle in the npm module (E.G.: highlight.js/compat). That's something I've done with one of my package and I haven't received complains.
Publish as a dual-CJS+ESM package. As we discussed previously, this is not ideal.

I feel quite strongly that ESM is the correct way forward, however I realize I'm the only one who actually requested it, so I would understand if highlight.js as a project chose a different path. However I'll still try to use my humble influence of mere contributor of the project to guide it towards the light path of ESM 😉

Provide a CJS bundle in the npm module (E.G.: highlight.js/compat). That's something I've done with one of my package and I haven't received complains.

Link please, I would take a look at what that looks like.

https://github.com/aduh95/viz.js#synchronous-api

Note that it's not exactly the same thing, as the main API is promise-based, and I'm providing a different synchronous API, transpiled to CJS.

Note that it's not exactly the same thing

Yeah...

Provide a CJS bundle in the npm module (E.G.: highlight.js/compat)

I was looking for an example of a library that's taken this approach that I could have a look at.  Do you know of any?

I was looking for an example of a library that's taken this approach that I could have a look at. Do you know of any?

Rollup does something like that, they use package.json "exports" field to hide it from end users – that's a path we could explore if that help smooth out the transition.
I've created #3 to demonstrate what it would look like for this library.


Update README.md

Forgot the period!
"to skip this action If the system is running Windows 10"
Added period at the end of action.

Ha ha, great catch, thanks @99ytrewq911


Tests in server

A user might want to test if chapters are working before leaving the server running. Add a way to test an example entry.

README added, closing issue.


Click (touch) inside items doesn't work on iOS

Hi.
I'm trying to use Slick to display offers. Every offer has a button, that can be clicked (touched).
If  property mobileFirst set to false - on desktop everything works fine, but on iOS 7 Safari and Chrome jQuery $('hot-tour-button').on('click' function(){...}) doesn't work.
If  property mobileFirst set to true - iOS starts work with click event, but desktop borwsers don't catch this.
$(document).ready(function() {
    var SLICKconfig = {
        infinite: true,
        slidesToShow: 3,
        slidesToScroll: 1,
        autoplay: true,
        mobileFirst:false,
        autoplaySpeed: 5000,
        prevArrow:"

",
        nextArrow:"

",
        responsive: [
            {
              breakpoint: 1200,
              settings: {
                slidesToShow: 3,
                slidesToScroll: 1,
              }
            },
            {
              breakpoint: 1024,
              settings: {
                slidesToShow: 2,
                slidesToScroll: 1
              }
            },
            {
              breakpoint: 768,
              settings: {
                slidesToShow: 1,
                slidesToScroll: 1
              }
            }
        ],
    };
    $(".carousel-contaner").slick(SLICKconfig);
});

$(".hot-tour-button").on('click',function(){
        toggleShadow();
    });

How should I configure Slick to make click work both, on desktop and mobile devices?

please, jsfiddle

codepen (as like as jsfiddle) examle (http://codepen.io/anon/pen/jEYymW) crashes on my iPad mini iOS7.1.2 in both browsers (Chrome and Safari) after first scrolling (and I have no idea why it happens)
You can check live demo at http://vvtur.ru . Try tap green buttons in slider items, it should show you modal form. On desktop it is OK, but on mobile it fails

Ok, little investigation helped me to get some info. Problem is in responsive property. I get this bug when mobileFirst:false number of visible elements differs from the number in responsive.settings.slidesToShow property. Unsetting responsive property makes thing work good, but it provides design problems

@undestroyer - sorry i'm too sick right now to analyse this problem, but i'm also experiencing some problems with responsive options (didn't have time to check it also) - you could try 1.3.15 if it works, unless you need some options from 1.4.
P.S. by jsfiddle i mean any code playground (like "google that" === "check in search engine" or very old "walkman" === "cassette player [not necessary made by sony]").

You might want to try Flickity. Flickity has a custom staticClick event to resolve this tricky situation detecting clicks/taps within carousels. Try the demo: http://codepen.io/desandro/pen/XJZgzM

:+1: I am having a similar issue with not being able to click anchors within a slide if the responsive property is set, in fact no events on anchors seem to fire. I'll try and confirm @undestroyer's fix, or otherwise force it to work somehow and post back.

Yup, can confirm that @undestroyer's fix works as long as the event handlers are registered after slick is (re)initialised, but it's obviously not ideal to re-slick every resize.

It seems that Slick calls init() after the resize event calls checkResponsive() which removes all event handlers on the slides, probably somewhere around here: https://github.com/kenwheeler/slick/blob/1.4.1/slick/slick.js#L709
My current solution is to listen for the init event and rebind any event handlers (Marionette in this case):
this.$childViewContainer.on('init', function() {
  console.log('delegating child events');
  self.children.each(function(child) {
    child.delegateEvents();
  });
});

My solution was to use event delegation (jquery's ".on()" method) from outside of the carousel's top element (with the "slick-slider" class), targeting the elements inside the carousel. This way the event handler is unaffected due to the bug.

Also a good solution @mikerogerz
I had some issues with infinite looping of the slider as the "cloned" slides don't retain the events, so ended up setting data- attributes on the slide anchors so I could detect which one had been clicked.
@tomaszpoliszuk, @kenwheeler I would suggest these valid workarounds are probably enough to close this issue.

@mikerogerz Thanks for the solution.

For mikerogerz's solution need some additional thinking if you add anchors dynamically.
$('.carousel-container').on(..) vs $(document)

@mikerogerz solution works perfectly. In my experience it doesn't even appear that I need to bind this to a container element. I just use this and it works perfectly:
$('.slick-slider').on('click', '.slick-slide', function(){
  // This click event will not trigger when you drag the carousel
})


'Validate username' Endpoint not documented properly

Describe the issue:
It was observed that the documentation of the "Validate Username" endpoint [1] was not properly documented.
The document does not contain a description for statusCodes [2] received with the response.
[1]. https://docs.wso2.com/display/IS510/apidocs/self-registration/#!/operations#SelfRegister#validateUsernamePost
[2]. https://github.com/wso2-support/carbon-identity-framework/blob/v<IP_ADDRESS>/components/identity-mgt/org.wso2.carbon.identity.mgt/src/main/java/org/wso2/carbon/identity/mgt/constants/SelfRegistrationStatusCodes.java#L28

On hold, as the product implementation is not complete yet.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Update all config

Choosing the default options like com.mycompany etc, are not good
It should be discussed and changed

@pascalgrimaud : could it be tech.jhipster.lite.extension?

Why not, need to find a project name too
And waiting @murdos 's idea too

How about tech.jhipster.lite.extensions?
Unless we only plan to host one extension module (the banner one), using plurals seem to be a good idea.

How about tech.jhipster.lite.extensions?
Unless we only plan to host one extension module (the banner one), using plurals seem to be a good idea.

It could be a place to let contributors create modules that don't fit in the jhipster-lite. Does it sounds good?

So, it could be:
PackageName
tech.jhipster.lite.extensions
Project Name
JHlite Extensions Application
Base Name
jhliteExtensionsApplication
Let me know, WDYT 😃👍

No, I don't want additional modules here, I want this project as sample to create a custom jhlite, that's all

No, I don't want additional modules here, I want this project as sample to create a custom jhlite, that's all

So the singular is the better choice here:
PackageName
tech.jhipster.lite.extension
Project Name
JHlite Extension Application
Base Name
jhliteExtensionApplication
WDYT?

Base name: jhliteExtension would be enough

Project name: JHLite Extension Application (L in upper case)

Thanks 👍

No, I don't want additional modules here, I want this project as sample to create a custom jhlite, that's all

Fine. Then I guess we should include the word "sample" somewhere.
E.g. in the description: "JHLite Sample Extension Application".

I think application should reserved to real generated app.
What about :

JHLite Sample Extension
JHLite Sample Extension Engine
JHLite Sample Engine

Another idea?

I think it could be JHLite Extension Sample

To resume:
PackageName
tech.jhipster.lite.extension
Project Name
JHLite Extension Sample
Base Name
JHLiteExtensionSample


Add Octarine to the community page

@ybz can you tell me a little bit how you use Envoy?

Hi Chris,
I think you had an email conversion about it with William Choe, our VP
product.
I'm quoting from an previous email response:
'.... Its mentioned on the product page, you can search for Envoy.
Basically, we use Envoy as our proxy for TCP connections, gRPC, and http,
whereas we use libraries for kafka, nats, and rabbit mq.'
Thanks
Yaniv Ben-Zaken
On Thu, 12 Jul 2018 at 16:10, Chris Aniszczyk<EMAIL_ADDRESS>wrote:

@ybz https://github.com/ybz can you tell me a little bit how you use
Envoy?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/envoyproxy/envoyproxy.github.io/pull/57#issuecomment-404678586,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdEWW9tXycTm1bjFydcfPDu8KIzOYd8ks5uF9dvgaJpZM4VN3I_
.

Thank you for jogging my memory :)
On Thu, Jul 12, 2018 at 6:17 PM, Yaniv Ben Zaken<EMAIL_ADDRESS>wrote:

Hi Chris,
I think you had an email conversion about it with William Choe, our VP
product.
I'm quoting from an previous email response:
'.... Its mentioned on the product page, you can search for Envoy.
Basically, we use Envoy as our proxy for TCP connections, gRPC, and http,
whereas we use libraries for kafka, nats, and rabbit mq.'
Thanks
Yaniv Ben-Zaken
On Thu, 12 Jul 2018 at 16:10, Chris Aniszczyk<EMAIL_ADDRESS>wrote:

@ybz https://github.com/ybz can you tell me a little bit how you use
Envoy?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/envoyproxy/envoyproxy.github.
io/pull/57#issuecomment-404678586>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AAdEWW9tXycTm1bjFydcfPDu8KIzOYd8ks5uF9dvgaJpZM4VN3I_>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/envoyproxy/envoyproxy.github.io/pull/57#issuecomment-404679651,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAD5IVCBUV-jsFJTHIyvVIb_zbTVG9r2ks5uF9kOgaJpZM4VN3I_
.

--
Cheers,
Chris Aniszczyk
http://aniszczyk.org
+1<PHONE_NUMBER>


Added equality/comparison overloads for Duration and TimeSpan

Added equality/comparison overloads for Duration and TimeSpan. This makes it easier to work seamlessly with both types in the same code.

Other than that, looks good. Let me know when fixed and I will merg it in.

Should be fixed now.

Thanks. Also seems github finally introduced email notifications on new commits in a PR :-)

Nuget out.


Error when model itself is the carry in jax.lax.scan

Code
import jax, jax.random as jrnd
import equinox as eqx

key = jrnd.PRNGKey(5)
X = jrnd.normal(key, (20, 3))
y = jrnd.normal(key, (20, 1))

scanner = lambda carry, _: (carry, 5.0)

# 1. This works fine:
model1 = eqx.nn.Linear(X.shape[1], y.shape[1], key=key)
# NOTE: `model1` is the carry!
res1 = jax.lax.scan(eqx.filter_jit(scanner), model1, X)

# 2. This produces an error:
model2 = eqx.nn.MLP(X.shape[1], y.shape[1], 6, 3, key=key)
# NOTE: `model2` is the carry!
res = jax.lax.scan(eqx.filter_jit(scanner), model2, X)

Why this code?
I'm trying to implement the training loop inside jax.lax.scan, so the carry is the model updated by optax. The result of scan is supposed to be the model from the last epoch and the loss history.
Error
Traceback (most recent call last):
  File "_question.py", line 14, in <module>
    res = jax.lax.scan(eqx.filter_jit(scanner), model2, X)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/forcebru/.pyenv/versions/3.11.4/lib/python3.11/site-packages/jax/_src/traceback_util.py", line 166, in reraise_with_filtered_traceback
    return fun(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^
  File "/Users/forcebru/.pyenv/versions/3.11.4/lib/python3.11/site-packages/jax/_src/lax/control_flow/loops.py", line 254, in scan
    init_flat, carry_avals, carry_avals_out, init_tree, *rest = _create_jaxpr(init)
                                                                ^^^^^^^^^^^^^^^^^^^
  File "/Users/forcebru/.pyenv/versions/3.11.4/lib/python3.11/site-packages/jax/_src/lax/control_flow/loops.py", line 239, in _create_jaxpr
    carry_avals = tuple(_map(_abstractify, init_flat))
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/forcebru/.pyenv/versions/3.11.4/lib/python3.11/site-packages/jax/_src/lax/control_flow/common.py", line 40, in _abstractify
    return core.raise_to_shaped(core.get_aval(x))
                                ^^^^^^^^^^^^^^^^
  File "/Users/forcebru/.pyenv/versions/3.11.4/lib/python3.11/site-packages/jax/_src/core.py", line 1371, in get_aval
    return concrete_aval(x)
           ^^^^^^^^^^^^^^^^
  File "/Users/forcebru/.pyenv/versions/3.11.4/lib/python3.11/site-packages/jax/_src/core.py", line 1363, in concrete_aval
    raise TypeError(f"Value {repr(x)} with type {type(x)} is not a valid JAX "
jax._src.traceback_util.UnfilteredStackTrace: TypeError: Value <jax._src.custom_derivatives.custom_jvp object at 0x126e2e9d0> with type <class 'jax._src.custom_derivatives.custom_jvp'> is not a valid JAX type

The stack trace below excludes JAX-internal frames.
The preceding is the original exception that occurred, unmodified.

--------------------

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "_question.py", line 14, in <module>
    res = jax.lax.scan(eqx.filter_jit(scanner), model2, X)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: Value <jax._src.custom_derivatives.custom_jvp object at 0x126e2e9d0> with type <class 'jax._src.custom_derivatives.custom_jvp'> is not a valid JAX type

I think the problem is that activation functions in eqx.nn.MLP are Callable: https://github.com/patrick-kidger/equinox/blob/62c8aefd09aaf25ea3ecd17b71ef40694cd6c8e6/equinox/nn/_composed.py#L24-L34
The documentation says:

This error happens because a model, when treated as a PyTree, may have leaves that are not JAX types (such as functions). It only makes sense to trace arrays. Filtering is used to handle this.
Instead of jax.jit, use equinox.filter_jit.

However, I'm already passing eqx.filter_jit(scanner) to scan, but it doesn't help. Perhaps JAX doesn't support functions (the MLP's activations in this case) as part of the carry?
Is there any way around this?

An obvious fix is to use a Python loop instead of jax.lax.scan:
losses = []
for ep in range(epochs):
    (self.model, self.optim_state), loss_val = optim_step((self.model, self.optim_state), ep)
     losses.append(loss_val)
self.history_ = np.array(losses).flatten()

...instead of:
(self.model, self.optim_state), losses = jax.lax.scan(
    optim_step, (self.model, self.optim_state), jax.numpy.arange(epochs, dtype=np.uint32)
)

But with the Python loop the code that used to run for 2 minutes has now been running for 20 minutes, even though optim_step is jitted.

Right. So what's going on here is that jax.lax.scan requires that the carry be a PyTree of JAX arrays. But model2 isn't a PyTree of JAX arrays!
It's a PyTree, but it includes a non-JAX-array amongst its leaves. (Namely model2.activation and model2.final_activation.)
The appropriate thing to do is exactly as in Option 1 of https://docs.kidger.site/equinox/all-of-equinox/#2-filtering: by partitioning and combining the JAX arrays can get updated in the scan, and all the non-arrays can be captured via closure. You should be able to write a filter_scan operation in this way. I've actually been contemplating adding filter_scan, filter_cond, filter_while_loop etc -- they just don't come up as frequently, so I haven't gotten around to it.

Ah, seems like I didn't think of partitioning and then recombining the model. I got too used to treating eqx.filter_jit and friends as "magic that makes stuff work". Thanks!


Add icons to catalog.bom files

I've deliberately only focused on the .bom files in each module. I've not touched karaf/catalog/src/main/resources/library-catalog-classes.bom. At some point soon'ish, we should look at removing that duplication so it gets the .bom files from the bundles, rather than from a separate single .bom file. As such, I didn't want to spend my time updating the file that we'll likely subsequently want to delete!

Massive +1 @aledsage, thanks for this, LGTM.
Just one question though: Is the icon applied on the running instance if iconUrl is on the catalog item level (beside the catalog id like you did rather than down to one level, i.e within item) ?

Thanks @tbouron. I think the icon url needs to be at the top-level to be known about by the catalog, and that should be sufficient for it to be known by the entity instances.
I added two catalog items:
brooklyn.catalog:
    version: "1.0.1"
    itemType: entity
    items:
    - id: pr-128-top-level
      iconUrl: classpath:///ansible-logo.png
      name: PR 128 Icon at top
      item:
        type: org.apache.brooklyn.entity.stock.BasicApplication

And:
brooklyn.catalog:
    version: "1.0.1"
    itemType: entity
    items:
    - id: pr-128-inside-item
      name: PR 128 Icon inside item
      item:
        type: org.apache.brooklyn.entity.stock.BasicApplication
        iconUrl: classpath:///ansible-logo.png

I queried the catalog for these (e.g. with curl -v -u xxxx:xxxx http://localhost:8081/v1/catalog/entities/pr-128-top-level/1.0.1).
The response for the top-level .bom included: "iconUrl":"/v1/catalog/icon/pr-128-top-level/1.0.1", but the inside-item .bom didn't (in the "planYaml" section it did show the iconUrl though).

I deployed two apps:
services:
- type: pr-128-top-level

and:
services:
- type: pr-128-inside-item

I then queried the rest api about each entity (e.g. with curl -v -u xxxx:xxxx http://localhost:8081/v1/applications/zlhibxdxxz/entities/zlhibxdxxz).
The app response for "top-level" included "iconUrl":"/v1/applications/zlhibxdxxz/entities/zlhibxdxxz/icon", and the "inside-item" app had "iconUrl":"/v1/applications/jyda9wjmx0/entities/jyda9wjmx0/icon". I downloaded both of those icons from the server. I confirmed that both were the icon defined in the original .bom files.

@tbouron would you expect the icon to be defined/referenced anywhere else for the running instance? Anything else I should check?

@aledsage No I wouldn't. Having theiconUrl field returned when querying http://localhost:8081/v1/applications/zlhibxdxxz/entities/zlhibxdxxz is what we want.
Thanks for the tests BTW, LGTM 👍

@aledsage I forgot but does the endpoint http://localhost:8081/v1/applications/zlhibxdxxz/entities/zlhibxdxxz/children also returns the iconUrl?

@tbouron yes, the rest endpoint /children also includes the iconUrl (for both the pr-128-top-level and pr-128-inside-item). I deployed the app shown below:
services:
- type: org.apache.brooklyn.entity.stock.BasicApplication
  brooklyn.children:
  - type: pr-128-top-level

and then queried (via curl) for the children of the basic app.

retest this please

Great news! Then it's +1 from me :)


Reduce added

It resolves #4
Please tell me if something is off. I'm quite new to c++

@mateosss please also add a test in testing.cpp

Done, is that an acceptable test format? Or should I remove the "Reduce" print?

Thanks a lot @mateosss . This one was open for quite some time.


add e2e tests

use playwright???

@infomiho can share his experiences with it.
Last time I was doing e2e tests, which was couple of years ago, I was making a choice between Cypress and Testcaffee. I chose Testcafe, but I think Cypress got more popular in the meantime.

Ok, playwright has ~59k stars, cypress has ~49k, testcafe has ~9k.

Playwright worked for us in the main repo, let's keep using it 👍

https://github.com/wasp-lang/open-saas/pull/69


Added minty theme

Screenshot of my A11y card:

For assistance: https://help.github.com/en/articles/file-attachments-on-issues-and-pull-requests

Hi! This is a great start, but currently doesn't pass the WCAG requirements to be accessible.
If you could adjust the colors so that the A11y card displays "AA" or "AAA" compliant we can merge it in.

Made the theme WCAG compliant.

Looks great!
Thanks for your contribution! 🎉


[halide] fix core build

The core build did not work. The jit feature was the default and required to get a successful build => merge jit feature into core. This also adds the target-opengl-compute target/feature.

All features are tested successfully in the following triplet:
x64-windows

The current version has been upgraded to 15.0.0, please deal with the conflict.

Feature target-opengl-compute is tested successfully in the following triplet:
x86-windows
x64-windows


Container fails to start: unable to open database file: out of memory (14)

Running a minimal container with no volume mounts results in
docker run --rm ghcr.io/tailscale/golink:main
2022/11/14 14:34:02 NewSQLiteDB("/root/golink.db"): unable to open database file: out of memory (14)

or
version: "3"
services:
  golink:
    image: ghcr.io/tailscale/golink:main
    container_name: golink
    restart: unless-stopped
    command: "-verbose -sqlitedb /root/golink.db"

2022/11/14 14:30:55 NewSQLiteDB("/root/golink.db"): unable to open database file: out of memory (14)

However, running and building the binary directly works without an issue:
go run github.com/tailscale/golink/cmd/golink@latest -verbose -sqlitedb /persist/golink/golink.db

Hey, this happened for me too. It's due to the static / scratch image not playing nice with the Go bindings for sqlite.
To fix, I created a new Dockerfile which used a basic image:
FROM ghcr.io/tailscale/golink:main as golink

FROM debian:stable-slim

COPY --from=golink /golink /golink

I ran docker build -t golink . and docker run -it -e "TS_AUTHKEY=tskey-auth-...." --rm --name golink -v /opt/golink/data:/root/.config/tsnet-golink golink /golink -sqlitedb golink.db -verbose and it worked for me.
I did have an issue where my authkey was invalidated accidentally, but that's not relevant to this issue.

@parkr is that really the cause?  Interesting.  I think I recall seeing that also if I didn't map a volume for /root.  If that's the case, I can update the docs to make it clear that the volume mounting is required.

I can't claim that this is the only cause of this failure, but in my experience this stemmed from the fact that golink runs in cgr.dev/chainguard/static as uid:gid 65532:65532 (as documented), but a Docker named volume mount is, by default, owned by root inside the container.  Creating a local directory owned by 65532:65532 and mounting that as a volume at /root inside the container worked for me, as well as using a one-off utility container to change the permissions of a named volume.

hmm... @dlorenc @imjasonh what's the recommended way to handle this? I'd very much like to continue using wolfi images.

okay... I finally have a reproduction case.  I think it was because (as @parkr noted above) this is an amd64 issue, and I was testing on arm64.  Now that I can reproduce, I'll experiment more in the morning.

I just had a quick look at this. @epk if you add user: root to your compose file, it should work:
version: "3"
services:
  golink:
    image: ghcr.io/tailscale/golink:main
    container_name: golink
    restart: unless-stopped
    user: root
    command: "-verbose -sqlitedb /root/golink.db"

I've submitted a PR that should fix this without requiring setting the user to root.

This also happens on non-container systems with no root (i.e. impermanent nixos).


pfsense-api environment crash can still submit a post but the gui can't see the result

My colleague through the pfsense API to submit article about 1.1 million aliases to until after the firewall, the firewall ERROR: "/ API/v1 / firewall/alias/index. The PHP: New alert found: PHP ERROR: Type: Then add any aliase to the firewall again, and the request will appear successful, but from the perspective of the gui, it is not generated at all. Then I uninstall the api and reinstall the api and the problem is not solved
The attached.txt is the instruction they submit

response_1710212289247.txt

Screenshot of an error message

Screenshot of an error message

This error is coming from pfSense itself. After your API call, pfSense detects a problem with your config, but an error is thrown while pfSense is attempting to restore the previous config. Your configuration is in a bad state. You can try to look at the bad config in /conf/config.xml.bad after you get the error and see if there is anything obviously wrong. It's also possible your configuration has just become too large for pfSense/PHP to read on your system.


TEP-0090: Get Names of TaskRuns from ChildReferences only

Changes

In https://github.com/tektoncd/community/pull/736, we decided that Matrix will use ChildReferences only. Therefore, it is unnecessary to check the TaskRuns field of PipelineRun status when getting the names of TaskRuns for matrixed PipelineTasks.
Related PRs: https://github.com/tektoncd/pipeline/pull/5019, https://github.com/tektoncd/pipeline/pull/4958
/kind cleanup
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[n/a] Docs included if any changes are user facing
[x] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in (if there are no user facing changes, use release note "NONE")

Release Notes

NONE

@abayer @dibyom please take a look

/lgtm

/test pull-tekton-pipeline-integration-tests

/test pull-tekton-pipeline-build-tests


Update User Guide

Minor amendment

Coverage remained the same at 88.943% when pulling 906d7c46be6eb93100f8135c17bacca9da80379b on AyushChatto:updateUserGuide into 55053436c8a01cd428be9a1c0a964965c1baf627 on CS2103-AY1819S1-W13-1:master.


When saving a specific level file there will be a list of required header keywords that need to be populated

We can keep the list of the required keywords in constants.py and they can be populated with nans and comments that indicate they were auto-populated. This is for archiving purposes: it is easier to have a fixed data format.

This issue is now superseded by Issues #140, #141 and #142.


Stack and Sort

Basic Information
Mission version: 0.96
Map used: Tanoa
Mods used: Arma, RHS
Dedicated or local game? Ded
GitHub or Workshop mission file? GitHub
Individual Things
Did you edit anything within the mission files?
No, fresh download from Master.
If yes, what did you edit exactly?
/
Short Issue Description
Using Stack and sort on the storage gives 1 error per box and the boxes go airborne.
Screenshots, Serverlog or any other helpful Information
rpt.txt

Y our fob would unload them from the storage right in fron of your nose.. did't lose any, but had to store them again.


Update mev-inspect docs to be about mev-inspect-py instead of mev-inspect-rs

This adds sections for:

overview
install
quick start
inspecting blocks
exploring with postgres
specs for each database table

It also copies over architecture and contributing sections from the WIP GUIDE.md in mev-inspect-py, but doesn't add these to the sidebar yet
Here are links to the new docs:

overview
install
quick start
inspecting
exploring
data

classified_traces
swaps
transfers
arbitrages
miner_payments

And here's what the sidebar looks like:

Looking at tests now


MiniMagick

Hi Joseph,
I’m not sure if this is even desired, but I thought I open a PR anyway. Feel free to close this, if this is not of interest, or not good enough. But here we go:
I’ve made this gem use the MiniMagick gem instead of the RMagick gem.
Motivation
RMagick seems to currently be incompatible with ImageMagick 7. At least most solutions to installing the RMagick gem successfully suggest to explicitly install or downgrade to ImageMagick version 6.
I didn’t want to do that. And MiniMagick did support the latest version. Plus, it claims to have a smaller memory footprint.
And since the underlying library – ImageMagick – is the same with those two gems, I figured it wouldn’t make a difference functionally anyway.
Explanations
While changing from the Rmagick to the MiniMagick API, I tried to stay as closely as possible to your original code. I only touched the tests in so far as to accommodate for the different API.
What MiniMagick seems to do differently, though, is handling how to open and save copies of an original image. While with RMagick you passed the opened original around and create a copy on write/save, MiniMagick requires to open the original each time you want to make a resized copy of said original. (At least that’s what I did, following the MiniMagick documentation.)
Now, I don’t know what that did to performance, because I didn’t compare this library with using RMagick and then MiniMagick. But I guess it didn’t improve things in this regard.
(Partly) Fixing the performance issue
Regardless, I reckon it was wasteful to always “fully” open the original file, while the ResizeHandler is only figuring out if it should create a downsized copy of an image. Because at this stage, it’s only trying to figure out if a file exists at a certain path. None of the actual image needs to be loaded for this.
So that’s why I added another commit that (alas) adds another dependency to the ImageSize gem, but in turn gives much greater performance, if – and that’s a big if – you’re not using the auto_rotate configuration.
In my case though – a project without auto_rotate and close to 2000 original images and ~8000 in total for the _site (I generate 3 versions for each original) – the initial Jekyll build went from ~423 seconds to ~23 seconds. That is for when no new images need to be generated.
I hope that all makes sense! ;-)

Whoops. Tests fail. Hm… works on my machine (haha ;-) ). Damn, no time to check this now…

Hey, awesome PR! I have glanced through the code but haven't had a chance to really read and understand it. I did check it out locally and use it to build my blog which generates 524 resized images from 74 original images. I'm running ImageMagick 6.9.7-4 Q16 x86_64 20170114.

Build time with the master branch: 20.178 seconds
Build time with your minimagick branch: 64.374 seconds

Out of curiosity, I ran the build using the latest stable version of ImageMagick (7.0.8-10 Q16 x86_64 2018-08-25).

Build time with the master branch: 20.708 seconds
Build time with your minimagick branch: 149.366 seconds (?!)

This is a worst-case scenario, since (hopefully) most people have setups that only require new images to be resized. Even so, I need to think about this a bit before considering whether to merge it - I would hate to publish a new version of the Gem that made people's build times unbearable.
I should also mention that you are more than welcome to publish your branch as a separate Gem - all of the code is MIT-licensed so there are pretty much no restrictions there.
The tests pass locally for me as well, btw. There must be something weird with the Travis CI environment.

Thanks for replying so quickly!

I ran the build using the latest stable version of ImageMagick (7.0.8-10 Q16 x86_64 2018-08-25)

Are you able to run RMagick with ImageMagick 7? As far as I found out, it’s not compatible.
Regardless, this performance regression is not good!
If you are using the auto_rotate config setting, I’ve now added another commit, which improves performance for that case: the code no longer opens an original using MiniMagick until a resized version of an original has to actually be generated.

Even so, I need to think about this a bit before considering whether to merge it - I would hate to publish a new version of the Gem that made people's build times unbearable.

Absolutely! As I said: my motivation was so that I can the latest ImageMagick version (Version 7).
I found out RMagick is not compatible with that version, so I made these changes. If you can actually run IM 7 using RMagick, or if using MiniMagick is simply not desired at all, I’m happy for you to close this one.
No need for me to publish a whole separate gem, either. I can happily use my fork.
So, maybe run this branch again, and see if things improve. If not, then it’s probably best for most people to discard this PR.
(Locally, tests run successfully, contrary to what the Travis-CI build says.)

Argh, I just realised that I didn't rebuild rmagick after building IM7, so it was still using IM6. I ran my build again with your latest commit and it took 65 seconds, so I'm guessing the 149 second build was just an outlier since it wasn't actually any different to the first build.
I don't use auto_rotate, btw, but when I enabled it the build took 72 seconds so that optimisation is definitely working :)
I think this PR will be good to leave open for the time being, in case anyone else is searching for a way to run the plugin with ImageMagick 7.

Alright! Well, happy to leave it open.
But, yeah: still disappointing performance compared to master. If I feel like it, I might have another stab at this (in my particular case, though, performance is now 😀👍).
Anyhow: thanks for the quick and helpful feedback! And thanks for creating this gem!


Move provider includes

This merges two include directories, 'providers/common/include/provandproviders/implementations/include/provinto one new directoryinclude/providers. Luckily there was no overlap in the names, so a simple script like #173 was used and then hand-fixing the build.info` files.
One more massive change, turning include "crypto/foo.h" into include <crypto/foo.h> (and same for internal) which is not consistent right now.
Checklist
Thank you for your contribution. Please answer the following:

[X] I acknowledge that I am authorized to submit this code under
the terms of the Apache License
[ ] All new public APIs are documented
[ ] All new public functions have tests
[ ] All new public functions have fuzzing tests

I think I need a new label, trivial but tedious for review hints :)


Better docs for observability in Universal

Description
I would like to follow up how to make the most out of my kuma installation. It would be benifitial if commond kinds of setups (from entry point to x) would be documented, since the kuma documentation is complex and more like a rule book, not a guide book.
Is there any chance OSing the complete configuration of this application shown in your screencast?

Specificly I am trying to get nested traces working. Currently, when my frontend requests a resource via https://api.my.app, this naturaly happens from the users side, so the request to the frontend (user requesting app.my.app), and the users request from the frontend to the API (users browser requesting api.my.app) goes via the internet & both requests originate from Kong, and are not grouped together in a trace.
Actual
Internet -> Kong (app.my.app) -> frontend (react)
frontend (browser request) -> internet -> Kong app.my.app -> api
Desired
Internet -> Kong (app.my.app) -> frontend (react) -> api.my.app
Is this possible through some kind of header forwarding from my frontend, since the request from the browser has to be somehow connected to the previous request requesting the (react) frontend. Thanks!
Update
I found out that my react (next.js) app was swallowing the corresponding headers. Nonetheless I think all of these things should be documented or referenced to.
For anyone trying to fix the same issue, your application will need to take care to forward these headers to the subsequent call
let traceHeaders = {
 'x-b3-traceid': headers.req.headers['x-b3-traceid'],
 'x-b3-parentspanid': headers.req.headers['x-b3-parentspanid'],
 'x-b3-spanid': headers.req.headers['x-b3-spanid'],
 'x-b3-sampled': headers.req.headers['x-b3-sampled']
  }

After doing so the traces are collectively shown

Maybe for this to be easy in the demo. A universal zone should be able to reach the kubectl install metrics stack?

🏓🤖

Hey @chris-aeviator the obs docs were recently updated and might be more understandable for Universal: https://kuma.io/docs/1.7.x/explore/observability/
In any case I'd love to understand more precisely what parts of observability you are struggling to setup in Universal so we can make the docs more understandable.

When trying to go back to the time I had these issues understanding universal mode, the main thing coming back to my mind is being unclear who's installing the services (grafana, jaeger). Put different: do I need to install them seperately or does kuma start them for me. I ended up setting up everything by hand (which is ok, just wasnt too clear). In general the docs seemed way more targeted towards k8s. Thanks for taking the effort in updating and widening the documentation, I'll soon start setting up another machine from scratch and will be able to give a much more detailed reply. Should we keep the issue open o. close?

Let me have a read through the doc with your extra context and see how it looks now.
Thanks!


Add image management docs

Signed-off-by: Usha Mandya<EMAIL_ADDRESS>Add docs for Advanced image management dashboard

Preview - https://deploy-preview-12502--docsdocker.netlify.app/docker-hub/image-management/

Thanks for the review @mark-dr. Didn't want to use the dummy data, so added this screenshot instead. Is there an example repo I can log into to get a screenshot?

LGTM, thanks for the changes :slightly_smiling_face:

Removed the 'Image Management API' section.


Downgrade epoch to Initial if remote peer can't decrypt yet

Motivation:
We should downgrade the epoch to Initial if the remote peer cant decrypt yet, failing to do so might cause the remote peer to drop the frames on the floor and trying to resend.
Modifications:
Check if we need to downgrade to Intial and if so do it.
Result:
Remote peer can always handle frame

Will work on a unit test

@ghedo this has a unit test now which failed before the change. I used the #[cfg(test)] macro to expose stuff only for testing. If you have an other preference on how to do this let me know.

Merged, thanks @normanmaurer!


Children drag

The parent node should be expanded when one drags and drop inner nodes

Keep there position on innernodes

This has been fixed, now when a child changes its position also affects the parent size and position

Done


Add "responses" library to setup.py.

This library is necessary for the unit tests but wasn't included in setup.py.

Coverage remained the same at 89.284% when pulling 685bac796433c772b9a0f4a768229558a809cefa on stephen-mw:Add-pygenie-library into efc61d1667b49f257c1541e99b1fca1d1868e183 on Netflix:develop.