Change PageSize property from string to int?

Description
Change PageSize property from string to int?
Fixes #134
Type of change
Please delete options that are not relevant.

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[X] I have checked my code and corrected any misspellings

@hognevevle Thank for your contribution. 🙌🏼


Drop Digits type

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (2ca907d) 80.92% compared to head (d51be3e) 81.59%.
Report is 1 commits behind head on main.

Files
Patch %
Lines

hotp.go
87.50%
0 Missing and 1 partial :warning:

totp.go
90.00%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
+ Coverage   80.92%   81.59%   +0.66%
==========================================
  Files           3        3
  Lines         194      201       +7
==========================================
+ Hits          157      164       +7
  Misses         29       29
  Partials        8        8

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


Clarify code styling in Documentation

Make explicit code styling in repo documentation.

@rafaelfsilva is there anything else that you think would be worth recording? I honestly didn't know that pep8 was the de-facto standard for python, it would be good to make explicit any other implicit assumptions, best practices.


UseMongoDb should two parameters

UseMongoDb(  this SerilogUiOptionsBuilder optionsBuilder, string connectionString,  string databaseName, string collectionName )
because the connectionString contains the databaseName, so the databaseName is redundant.

Sorry for the delay in getting back to you. For the next release, I will add another overload without databaseName field.


Try out CircleCI

Feature Request
Desired behaviour
Hedge against Travis-CI failure
https://circleci.com/
┆Issue is synchronized with this Jira Story
┆Issue Number: DOCK-528

Unit tests for both UI and webservice


Add pipeline trigger

It would be great to be able to trigger a specific pipeline to run by the press of a button. It should be easy to implement, it's just a POST: https://docs.gitlab.com/ee/ci/triggers/#triggering-a-pipeline

Do you mean to trigger a manual pipeline?
I was thinking about adding some kind of mechanism to retry a pipeline, but this sounds like a different action, isn't it?

Yes you're right this would be:
POST /projects/:id/pipelines/:pipeline_id/retry

triggering new build uses this endpoint:
POST /projects/:id/trigger/pipeline

@SantiMA10 do have any idea when you can work on this? At the moment I am using API Ninja to achive this, but a more "native" feeling with your Plugin would be great :)

@tuc0w sorry, but I'm a bit busy lately, I will work on it when I have some free time
meanwhile, if you want to work on it I will be happy to review a PR

Maybe we can add this a long-press action, following this example: https://github.com/elgatosf/streamdeck-counter/blob/master/Sources/com.elgato.counter.sdPlugin/code.html


Template has two nested vertically scrollable areas

Hi,
an other one :-).  The template creates two vertical scrollbars for two different areas, which confuses navigation.  Depending on the location of the cursor, one or the other scrollbar is used with the scrollwheel.  Also nested scrollable areas should probably be avoided in general :-):

Hi, again! I updated the style. Please re-install and try it.
It occurs when the browser width is narrow. The mobile header was causing the problem.

fastastic! Thanks for the quick fix.  It looks much better now :-)


Automatically add a PR comment with documentation link

When the HTML documentation is rebuilt, the link to generated documents is hard to find.  Automatically add a comment to the PR with the docs link.

I am not a big fan of every workflow adding their own comment -- it creates unnecessary noise and clutters the comment section.
This should not be necessary since we already provide a link to the built documentation in the form of annotation.

The annotation isn't easy to discover from the PR.  You have to find the Documentation Build in the list of checks, click "details", then click summary, and finally open the doc link.
I was hoping to make the doc link more visible on PRs that update docs.  I personally like reviewing the rendered HTML for large doc changes.
This PR is just a proof of concept (and a failing one at the moment).  My vision was that doc-build workflow would add one, and only one comment to the PR.  Force pushes to the PR would not generate a new comment.
I see other potential use cases for workflows adding a comment if the security concerns can be addressed.  For instance, adding a comment with a test coverage report for the PR.  We do this in a downstream repo (on gerrit), and it has been very effective at encouraging developers to write more tests.


Add props to customize (translate) aria-labels of buttons

Is your feature request related to a problem? Please describe.
For the lightbox to be accesible we need to translate aria-labels of prev, next and close buttons to different languages.
Describe the solution you'd like
Add "translations" prop to the lightbox.
Describe alternatives you've considered
For now I am doing this:

Which works, but is kind of verbose when only reason for it is changing aria-labels.
Additional context
No response

It sounds like you are looking for the labels prop.
labels={{
  Next: "Next slide",
  Previous: "Previous slide",
  Close: "Close the lightbox",
}}

https://yet-another-react-lightbox.com/documentation#Lightbox


Add missing perl-unicode feature to regex crate

PR #550 reduced the feature flags of the regex crate on only ["std"], however this is not enough, as compiling the used regexps requires the unicode-perl feature flag to compile the \s class (likely to match Unicode whitespaces).
Without this flag, prost-build panicks with the following message:
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Syntax(
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
regex parse error:
    https?://[^\s)]+
               ^^
error: Unicode-aware Perl class not found (make sure the unicode-perl feature is enabled)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
)'

How could this work before, I have no idea, but a minimal example with a blank crate and only the regex dependency panics with trying to compile the regex in prost-build/src/ast.rs:98: https?://[^\s)]+.

Seems like this got fixed in https://github.com/tokio-rs/prost/pull/604, so this PR is obsolete now.


Linker Errors VS2017. Not sure what's wrong?

LNK2019	unresolved external symbol "public: void __thiscall anax::World::SystemDeleter::operator()(class anax::detail::BaseSystem *)const " (??RSystemDeleter@World@anax@@QBEXPAVBaseSystem@detail@2@@Z) referenced in function "public: __thiscall std::unique_ptr<class anax::detail::BaseSystem,struct anax::World::SystemDeleter>::~unique_ptr<class anax::detail::BaseSystem,struct anax::World::SystemDeleter>(void)" (??1?$unique_ptr@VBaseSystem@detail@anax@@USystemDeleter@World@3@@std@@QAE@XZ)
Getting this whenever using anax::World. I've used both Visual Studio and mingw32-make and get the same results. I get the DLL and put it in the .exe directory. I feel like there should be a .lib file but I don't get any?

Within your Visual Studio project you have to change your linker settings to link to the DLL you compiled, otherwise you will get linker errors complaining that it can't find the implementation.

Everything I've found suggests there needs to be a .lib file for Visual Studio to import the library. That's how every other library I've used works. I'm not sure what to do here. Looks like someone else had the same problem #78


Fails to work with directories with foreign symbols

Really simple issue I think:
My name is Rafał (Ł is a polish letter), and jest runner cant open c:\Users\Rafał... When I move working folder just to c;\ it works fine. Jest runner parses Rafał as Rafa. Screenshot below.
To recreate just change a letter in working directory to Ł.

probably they need to be escaped and are not


Add Birthday Announcement Feature

Anyone who wants to look over my code feel free to, let me know ill add you as a reviewer

I went through some of the changes and did commit some of them, others i will probably hold off until i have more information on how to actually do it


template-including-parent:: false / true not working

I inserted a button and would like template-including-parent to work as well.
Currently the parent is not included.

+1


no longer works with latest tree-sitter

Did you check existing issues?

[X] I have read all the tree-sitter docs if it relates to using the parser
[X] I have searched the existing issues of tree-sitter-surrealql

Tree-Sitter CLI Version, if relevant (output of tree-sitter --version)
tree-sitter 0.22.6 (b40f342067a89cd6331bf4c27407588320f3c263)
Describe the bug
Until the yesterday's changes I was able to generate the parser using the latest tree-sitter version. After I updated to the latest main I can no longer do that as it goes OOM in the process (I have 16GB of RAM and it's just not enough).
Steps To Reproduce/Bad Parse Tree

using the latest tree-sitter;
run npx tree-sitter generate.

Expected Behavior/Parse Tree
I expected generate to be successful.
Repro
No response

Hey @alexaandru 👋🏻
I'll look into this today. Are you able to use the compiled version for the grammar that is committed to the repository? The latest version of the grammar aligns more with the Surrealist Lezer grammar. However, if it's causing issues, I'll have to look into someone more custom.
Thanks for testing it out 🙏🏼

Hello,
Yes, the compiled version works fine. FWIW, I use it here https://github.com/alexaandru/go-sitter-forest and when a parser stops generating I switch to pulling in the compiled version instead. I update it almost daily so that's why I find out as soon as something stops compiling ;-)
Cheers!
Alex

Hello,
Yes, the compiled version works fine. FWIW, I use it here https://github.com/alexaandru/go-sitter-forest and when a parser stops generating I switch to pulling in the compiled version instead. I update it almost daily so that's why I find out as soon as something stops compiling ;-)
Cheers!
Alex

Ah, interesting- I'll take a look at your project! I reckon I can minimise the amount of states that are generated, which should reduce the memory footprint when generating the parser.
Thanks for letting me know. I'll keep this issue open and post updates 👍🏻

You're most welcome! Have a great day!

Hey @alexaandru 👋🏻
I have significantly reduced the run time for tree-sitter generate. Let me know if this solved your issue. Thanks 🙏🏼

Hello,
Yes, it's back working now, already switched back to generating the parser :-)
Thank you & have a great day!


Accept {{token.name}} variable in effect description

Right now, if you use {{token.name}} in a Dfreds Convenient Effects effect, it renders properly in chat (shows the token's actual name) for the condition. But the text description in Effects Panel shows {{token.name}} instead of the creature's name, which is a bit ugly. Would be great if the effect description accepted that variable!

Interesting. I don't have official support for that first use case anyway, so I guess something is doing that. I'll see about adding it to the effects panel though

Do you have an example of using {{token.name}} in CE? It doesn't seem to be working for me

Hmm. I may have confused CE with the regular AE effect description box, which accepts {{token.name}} as a variable. Screenshot attached.


Custom nodeAffinity for fleet

What happened:

write nodeAffinity for fleet
try helm install custom helm chart
got helm error

Error from server (InternalError): error when creating "qq": Internal error occurred: failed calling admission webhook "mutations.stable.agones.dev": an error on the server ("") has prevented the request from succeeding

remove affinity section
success helm install

What you expected to happen:

GS pod's with affinity

How to reproduce it (as minimally and precisely as possible):

add to any fleet

apiVersion: "stable.agones.dev/v1alpha1"
kind: Fleet
metadata:
...
spec:
...
  template:
    metadata:
...
    spec:
...
      template:
...
        spec:
          affinity:
            nodeAffinity:
              requiredDuringSchedulingIgnoredDuringExecution:
              - preference:
                matchExpressions:
                - key: agones
                  operator: Exists
                weight: 1

Anything else we need to know?:
Environment:

Agones version: 0.8.1
Kubernetes version (use kubectl version): 1.11 (OKD 3.11)
Cloud provider or hardware configuration: bare-metal
Install method (yaml/helm): both
Troubleshooting guide log(s):

controller log:
{"error":"error with webhook handler for path /mutate: error unmarshalling original Fleet json: {\"apiVer.....
}}}: json: cannot unmarshal array into Go struct field NodeAffinity.requiredDuringSchedulingIgnoredDuringExecution of type v1.NodeSelector","message":"","severity":"error","source":"*webhooks.WebHook","time":"2019-05-30T15:02:46.370320387Z","url":{"Scheme":"","Opaque":"","User":null,"Host":"","Path":"/mutate","RawPath":"","ForceQuery":false,"RawQuery":"timeout=30s","Fragment":""}}

Others: scheduling == Distributed

i wrote bad template or controller has extra filters in webhook?

trace:
[
"agones.dev/agones/pkg/util/webhooks.(*WebHook).handle\n\t/go/src/agones.dev/agones/pkg/util/webhooks/webhooks.go:131",
"agones.dev/agones/pkg/util/webhooks.(*WebHook).AddHandler.func1\n\t/go/src/agones.dev/agones/pkg/util/webhooks/webhooks.go:99",
"net/http.HandlerFunc.ServeHTTP\n\t/usr/local/go/src/net/http/server.go:1964",
"net/http.(*ServeMux).ServeHTTP\n\t/usr/local/go/src/net/http/server.go:2361",
"net/http.serverHandler.ServeHTTP\n\t/usr/local/go/src/net/http/server.go:2741",
"net/http.initNPNRequest.ServeHTTP\n\t/usr/local/go/src/net/http/server.go:3291",
"net/http.(*initNPNRequest).ServeHTTP\n\t\u003cautogenerated\u003e:1",
"net/http.Handler.ServeHTTP-fm\n\t/usr/local/go/src/net/http/h2_bundle.go:5592",
"net/http.(*http2serverConn).runHandler\n\t/usr/local/go/src/net/http/h2_bundle.go:5877",
"runtime.goexit\n\t/usr/local/go/src/runtime/asm_amd64.s:1333"
]

Quick question, I assume error was on kubectl apply (or similar api command), not on helm install ?

Looking at the reference, looks like you are missing a nodeSelectorTerms under your requiredDuringSchedulingIgnoredDuringExecution
https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.11/#nodeselector-v1-core

kubectl apply or helm install - both, (helm install) + (helm template+kubectl apply -f)

my fault - in 1.11 absent "requiredDuringSchedulingIgnoredDuringExecution" without nodeSelectorTerms and not exist "requiredDuringSchedulingRequiredDuringExecution"

Work fine with:
apiVersion: "stable.agones.dev/v1alpha1"
kind: Fleet
...
    spec:
...
      template:
...
        spec:
...
          affinity:
            nodeAffinity:
              requiredDuringSchedulingIgnoredDuringExecution:
                nodeSelectorTerms:
                - matchExpressions:
                  - key: testnode
                    operator: Exists

Awesome - glad you got it working!


fix the value convert error in the rx operator

Thank you for contributing to Coraza WAF, your effort is greatly appreciated
Before submitting check if what you want to add to coraza list meets quality standards before sending pull request. Thanks!

Note: that go.mod and go.sum can only be modified for tested dependency updates or justified new features.
Make sure that you've checked the boxes below before you submit PR:

[x] My code includes positive and negative tests.
[x] I have an appropriate description with correct grammar.
[x] I have read Contribution guidelines, maintainers note and Quality standard.

Thanks for your PR :heart:
The convert function can't solve the problem of non-utf8 matching and causes the problem of normal data matching errors. So I remove the convert function and will open a new issue to discuss the rx operator.

Ok, so that's why we are removing the test, right?
Isn't better to have a test that fails? I mean, we keep a failing test, with the description of why it fails (but we make it pass). So solving the problem will make the test fail, and we require to change the test so it matches. But we still are covered by tests.
WDYT?

Is this still a problem in v3? could you add a failing test case for what you describe too to tackle an integral solution?

@fzipi I have updated this branch with the restoration of the test and a description of what's going on. While looking at @jcchavezs's other PR I deep dove into convert as indeed it looked strange (it is currently just converting each byte of the input into its ASCII code, which causes unicode content to get mangled when matching). And there's quite a bit of overhead to the operator used all throughout CRS. Hopefully we can get this fix in now and cherrypicked into v3

BTW while looking at this I also found some confusion in CRS itself - https://github.com/coreruleset/coreruleset/issues/2778

LGTM

It looks like 2 approvals are required (maybe only this branch) - @jcchavezs @jptosso can you approve? This will just revert the PR for v2 without seclang parser changes


#523: exclude files from wrong OS

fixes #523

exclude files from wrong OS in assembly files
fix XSD link to https

Also fix OS and arch classifier to follow our own standard (see ide-urls).
Was also specified in #427:

Actually I would expect that either in the workflow build or in our CLI POM or in the assembly we define that we want to have a release for win-x64, max-x64, mac-arm, linux-x64 (and potentially more)
See also #251

Fun fact is that even without this change, we got releases with windows-x64, mac-arm, etc. as specified so it worked.
The question is why did it work? At least the release artifact filenames in the target-folder are derived from these assembly IDs and have been somehow inconsistent, so for me this change makes sense.
UPDATE: I was blind. Each file is mapped in the POM. All non-sense because we never completed what I was trying to say in issues like #427
See also my comment on Teams:

Ich hatte ja mal vorgeschlagen, dass wir in den matrix.os VMs nur das native image bauen (ideasy[.exe]) und das danach ins release einbauen, d.h. wir bauen 4 Assemblies die jeweils ein eigenes natives image nutzen, was im matrix.os build hochgeladen und danach im Gesamt-Build wieder runtergeladen wurde.  Dann wird es m.E. auch wieder konsistent.

Created new issue #563 for that.
For the record: I had a long long discussion with @jan-vcapgemini if our arch classifiers are OK or not.
We could easily agree that arm is correct and should be preferred over aarch.
We can also agree that 32-Bit is to be considered dead.
That we use x64 what actually stands for x86-64 on the one hand and arm without any 64 is kind of inconsistent.
My argument was however, that this was specified long time ago in devonfw-ide and the entire ide-urls repo is based on this convention. Changing this in IDEasy would require to change or rebuild the entire ide-urls repo and that would even break devonfw-ide. So consider this as given by legacy if you do not like it.
Might be more logical if we would instead have

x64 and arm64
or x86 and arm
or x86-64 and arm-64

However, we simply have x64 and arm and that is somehow curved in stone so we better follow strictly our own conventions instead of doing something different in various places.

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

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

Totals

Change from base Build<PHONE_NUMBER>0:
0.0%

Covered Lines:
5743

Relevant Lines:
8507

💛  - Coveralls

Our builds now fail with this error:
Run coveralls report
coveralls: error while loading shared libraries: libevent-2.1.so.7: cannot open shared object file: No such file or directory
Error: Process completed with exit code 127.

Not related to the changes at all but some infrastructural problem of coveralls.
If that persists, we need to disable coveralls but first lets wait 24h or so.


fix ND RA test + added deviation for Arista

Pull Request Functional Test Report for #3399 / c005714292e46cbe8d8ead7d3fe5318cca9a3a2b
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

RT-5.9: Disable IPv6 ND Router Arvetisment
57bf617a
Log

Cisco 8000E

RT-5.9: Disable IPv6 ND Router Arvetisment
73055d51
Log

Cisco XRd

RT-5.9: Disable IPv6 ND Router Arvetisment
534bef21
Log

Juniper ncPTX

RT-5.9: Disable IPv6 ND Router Arvetisment
3e62e039
Log

Nokia SR Linux

RT-5.9: Disable IPv6 ND Router Arvetisment
11534159
Log

Openconfig Lemming

RT-5.9: Disable IPv6 ND Router Arvetisment
69e393bc
Log

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

RT-5.9: Disable IPv6 ND Router Arvetisment

Cisco 8808

RT-5.9: Disable IPv6 ND Router Arvetisment

Juniper PTX10008

RT-5.9: Disable IPv6 ND Router Arvetisment

Nokia 7250 IXR-10e

RT-5.9: Disable IPv6 ND Router Arvetisment

Help

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

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

Totals

Change from base Build<PHONE_NUMBER>5:
0.0%

Covered Lines:
1983

Relevant Lines:
3589

💛  - Coveralls


The loadfile option must be an integer: start=+104.792667

Reloading doesn't work and gives this error on console.

I have encountered exact same issue.
shinchiro's mpv-x86_64-v3-20240304-git-9ac791c build seems to break quite a few scripts.

The syntax for loadfile changed https://github.com/mpv-player/mpv/issues/13656


Load more in Search Results not working

Current behavior
When there are more search results then the initial batch a "Load more"-button is displayed, but clicking it does nothing. Seems vue-storefront-api response is not returning more products.
// EDIT: Problem only occurs, if you type the search term really fast (which triggers a lot of requests in short time). Request for more products then somehow has the wrong offset (e.g. from=36 instead of 18).
Expected behavior
Load more products and append them to the list.
Steps to reproduce the issue

Go to https://demo.storefrontcloud.io
Search for 'Shorts' (type fast)
Click 'Load more' button

Can you handle fixing this bug by yourself?
NO
Which Release Cycle state this refers to? Info for developer.
https://demo.storefrontcloud.io
https://test.storefrontcloud.io
https://next.storefrontcloud.io

Thanks @Flyingmana!


Cleaned up routes

Basically naming most of the routes so that we can easily do #867.

@joecohens can you check this please before merge. I think at least every GET route (not including the API) is now aliased.

This needs rebasing.

Done.

@jbrooksuk It seems very good to me :) :+1:

Yay! Now we can sort out #867


HTTP RPC, connection reset by peer

Discussed in https://github.com/paradigmxyz/reth/discussions/5172

Originally posted by bsl October 25, 2023
I'm doing what I consider to be non-abusive activity against my node's <IP_ADDRESS>:8545, and periodically I get "connection reset by peer", as if reth is upset about me connecting too quickly or too often. Is this due to rate-limiting?

@bsl how many concurrent requests?
what kind of requests?

@mattsse I think max concurrent requests is just the same as my number of threads, so let's say 10 at most. I'm doing JSONRPC requests, eth_getCode. It's Python requests doing the POST.

hmm,
ratelimiting should return a 429 error
but can you try increasing the max concurrent connections via:
--rpc-max-connections to something higher?

are you setting the content-type header?

I don't think I've ever seen a 429. I'm not getting JSON back, or even an HTTP-level error, I always just get "connection reset by peer". I set --rpc-max-connections to 100 and then to 200 and can't see any difference. I do set Content-Type: application/json.
If I nc -l -p 9999 and make requests do its POST there, here's how the request looks:
POST / HTTP/1.1
Host: <IP_ADDRESS>:9999
User-Agent: python-requests/2.31.0
Accept-Encoding: gzip, deflate
Accept: */*
Connection: keep-alive
Content-Length: 87
Content-Type: application/json

{"jsonrpc": "2.0", "id": 1, "method": "eth_getBlockByNumber", "params": ["0x01", true]}

As I was writing this just now, I saw that Connection: keep-alive, and it looked bad to me, because the code is currently written to connect, POST, get something back, and consider the connection dead (leaving aside whether this is good practice).
Changing the code to
with requests.Session() as s:
    s.headers["Connection"] = "close"
    ...

seems to have fixed it. Thank you. Sorry about the noise.


Made site work with latest hugo version

As title. I didn't update everything inside of content/pages as it seem to not be used anymore.

LGTM


Doc Feedback: Blank page

This page and the one below it are blank.
Location : https://apim.docs.wso2.com/en/latest/Reference/Guides/business-scenarios/

Fixed in https://github.com/wso2/docs-apim/pull/707


added clear_text

Created clear_text(), because while using if you return back, it shows previous written text.

Thanks for your contribution! 👍  :)


VS2019 .net core 2.1 or 3 import GraphEngine. LIKQ package errors

在vs2019上建立MVC项目，无论是.net core 2.1 或 3，导入GraphEngine.LIKQ包都报错
Create MVC project on vs2019, select. net core 2.1 or 3, and import GraphEngine. LIKQ package all report errors
error info
严重性	代码	说明	项目	文件	行	禁止显示状态	禁止显示状态
错误	NU1107	Microsoft.CodeAnalysis.CSharp 中检测到版本冲突。直接安装/引用 Microsoft.CodeAnalysis.CSharp 2.8.0 到项目 WebApplication2 可解决此问题。
WebApplication2 -> Microsoft.AspNetCore.Mvc 2.1.3 -> Microsoft.AspNetCore.Mvc.Razor.Extensions 2.1.1 -> Microsoft.CodeAnalysis.Razor 2.1.1 -> Microsoft.CodeAnalysis.CSharp (>= 2.8.0)
WebApplication2 -> GraphEngine.LIKQ 1.0.8784 -> Microsoft.CodeAnalysis.Scripting 1.3.0 -> Microsoft.CodeAnalysis.CSharp.Scripting 1.3.0 -> Microsoft.CodeAnalysis.CSharp (= 1.3.0).	WebApplication2	C:\Users\shy_l\source\repos\GEWebApplication\WebApplication2\WebApplication2.csproj	1

@shyleoking We going to need a new build that targets .NET 3.0; the current packaging is a bit out of date. I have a local build is working with .NET Core 3.0. We are going to need to have a PR in place to resolve the build; one of the repo members must update the build. You can clone the repro and execute a local build and all the bits will be on your machine.

looks like we should update the dependency for LIKQ.
it's probably a good idea to introduce dependabot in this repo.

I'm looking into dependabot setup

Oh wow, the dependabot setup is very simple; I don't have access to the GE Azure CD/CI DevOps environment, so a member of the GE Team will need to update it.

The toolchain has been updated; this issue is not relevant anymore. I will close this thread.


Nested list parsing doesn't work in Mynah UI

Problem
We've seen responses from LLM being answered in a wrong level when nested (see screenshots).

Steps to reproduce the issue
If I copy the file for nested examples in Mynah UI, add as an input response, it doesn't format/work as expected.

Expected behaviour
It should parse it as expected in Markdown spec: https://ref.coddy.tech/markdown/markdown-nested-lists
System details

OS:
Tested enviroment:

[X] Web (Demo app)
[X] VSCode Amazon Q Chat
[ ] JetBrains IntelliJ Amazon Q Chat

Enviroment extension version (if applicable):
[ ] Enviroment is remote

Formatting is also broken for other functionality. For example, if I "Send to prompt" selected code block, code indentation is lost:

The issue was caused by this change: https://github.com/aws/mynah-ui/pull/72/files.


Crashes under bad network condition

This exception raises when the network condition goes terrible. Is there a way to bypass the exception and force it to retry? Thank you in advance.
/usr/lib64/ruby/2.3.0/net/protocol.rb:44:in `connect_nonblock': SSL_connect SYSCALL returned=5 errno=0 state=SSLv2/v3 read server hello A (Faraday::SSLError)
	from /usr/lib64/ruby/2.3.0/net/protocol.rb:44:in `ssl_socket_connect'
	from /usr/lib64/ruby/2.3.0/net/http.rb:928:in `connect'
	from /usr/lib64/ruby/2.3.0/net/http.rb:863:in `do_start'
	from /usr/lib64/ruby/2.3.0/net/http.rb:852:in `start'
	from /usr/lib64/ruby/2.3.0/net/http.rb:1384:in `request'
	from /home/shou/.gem/ruby/2.3.0/gems/faraday-0.13.1/lib/faraday/adapter/net_http.rb:80:in `perform_request'
	from /home/shou/.gem/ruby/2.3.0/gems/faraday-0.13.1/lib/faraday/adapter/net_http.rb:38:in `block in call'
	from /home/shou/.gem/ruby/2.3.0/gems/faraday-0.13.1/lib/faraday/adapter/net_http.rb:85:in `with_net_http_connection'
	from /home/shou/.gem/ruby/2.3.0/gems/faraday-0.13.1/lib/faraday/adapter/net_http.rb:33:in `call'
	from /home/shou/.gem/ruby/2.3.0/gems/faraday-0.13.1/lib/faraday/request/url_encoded.rb:15:in `call'
	from /home/shou/.gem/ruby/2.3.0/gems/faraday-0.13.1/lib/faraday/request/multipart.rb:15:in `call'
	from /home/shou/.gem/ruby/2.3.0/gems/faraday-0.13.1/lib/faraday/rack_builder.rb:141:in `build_response'
	from /home/shou/.gem/ruby/2.3.0/gems/faraday-0.13.1/lib/faraday/connection.rb:387:in `run_request'
	from /home/shou/.gem/ruby/2.3.0/gems/faraday-0.13.1/lib/faraday/connection.rb:174:in `post'
	from /home/shou/.gem/ruby/2.3.0/gems/telegram-bot-ruby-0.8.5/lib/telegram/bot/api.rb:72:in `call'
	from /home/shou/.gem/ruby/2.3.0/gems/telegram-bot-ruby-0.8.5/lib/telegram/bot/api.rb:60:in `method_missing'
	from /home/shou/.gem/ruby/2.3.0/gems/telegram-bot-ruby-0.8.5/lib/telegram/bot/client.rb:30:in `fetch_updates'
	from /home/shou/.gem/ruby/2.3.0/gems/telegram-bot-ruby-0.8.5/lib/telegram/bot/client.rb:25:in `listen'
	from hejmo-bot.rb:98:in `block in <main>'
	from /home/shou/.gem/ruby/2.3.0/gems/telegram-bot-ruby-0.8.5/lib/telegram/bot/client.rb:18:in `run'
	from /home/shou/.gem/ruby/2.3.0/gems/telegram-bot-ruby-0.8.5/lib/telegram/bot/client.rb:8:in `run'
	from hejmo-bot.rb:95:in `<main>'

It shouldn't be a problem of my code, my code looks like just the standard message processing in the listen block:
token = 'SECRET_BOT_TOKEN'
Telegram::Bot::Client.run(token) do |bot|
  bot.api.get_updates(offset: -1) # eat up all previous update
  bot.listen do |msg|
    # do stuff with msg
  end
end

Where in the error message, hejmo-bot.rb:95 corresponds to the Telegram::Bot::Client.run(token) line and hejmo-bot.rb:98 corresponds to the bot.listen line.

hmmm are you using windows ?
anyway you have two way to fix this error:
1:
write this command in cmd set SSL_CERT_FILE=:c/path_to_certfile.pem
or try this one: #101

Thanks for the reply. I was using it under Linux, and I'm pretty sure it's not the certficate's problem.
Usually my script works all fine, but if I left it to run for few minutes, exceptions like this might occur and crashes the script.

hmmm weird but it look like a SSLError ... did you checked your ssl certification?

I don't really think it's about SSL certificate. I think it could be some packet losses happen when making requests so it failed the SSL handshake.

no idea ... but it's not bad to try the 2nd way to fix this ... try it, maybe it works

So temporary dirty fix towards this is writing code like this:
begin
    bot.listen do |msg|
      # do stuff with msg
    end
rescue
  retry
end

While it's very temporary and doesn't fix the problem completely. If the network error happens on other requests within the bot.listen block, it can't retry that specific request alone rather than restarting the whole listen process.

yea i had these problems too ... i usually use this method .. yea it's not so good but ... it works

Sounds like an issue with SSL version. Unfortunately, there is no way to configure it for now.

Catching the exception and further investigating what is causing the issue is probably the right thing to do.  What "properly" means depends on the specific context. An idea could be to store the messages in some data structure (array?) and process them later when the connection is stable again. In addition you could add a timeout instead of retrying immediately. Just be careful not to block the bot with too long timeouts.

hmmm are you using windows ?
anyway you have two way to fix this error:
1:
write this command in cmd set SSL_CERT_FILE=:c/path_to_certfile.pem
or try this one: #101

i wrote that on my cmd on window and it did nothing

i wrote that on my cmd on window and it did nothing

Did you replace c/path_to_certfile.pem with path to your cert file?

i wrote that on my cmd on window and it did nothing

Did you replace c/path_to_certfile.pem with path to your cert file?

how can i know the path to my cert file pls


Incorrect SharedPtr ref count with arm build.

Hello everyone.
I am trying to call shared library API (built from c++) from RUST using cxxbridge.
I am using shared_ptr in my implementation. But shared_ptr behavior is different w.r.to x86 build and ARM build.
Below is the base code.
CPP (The below code is built as shared library (.so))
class A {
	public:
	A() {}
	~A() {}
};

class testA {
	public:
	testA() {}
	~testA() {}

	void calculate(std::shared_ptr<A> ptr_A)
	{
		cout << "### [testA CPP] ptr_A ref count : " << ptr_A.use_count() << endl;
		// ...
		// calculation logic

	}
};

RUST
test.h
------
std::shared_ptr<A> create_shared_ptr_A();
std::shared_ptr<A> create_shared_ptr_testA();
void doCalculate(std::shared_ptr<testA> ptr_testA, std::shared_ptr<A> ptr_A);

test.cc
--------

std::shared_ptr<A> create_shared_ptr_A()
{
	return std::make_shared<A>();
}

std::shared_ptr<A> create_shared_ptr_testA()
{
	return std::make_shared<testA>();
}

void doCalculate(std::shared_ptr<testA> ptr_testA, std::shared_ptr<A> ptr_A)
{
	cout << "### [RUST] ptr_A ref count : " << ptr_A.use_count() << endl;
	ptr_testA->calculate(ptr_A);
}

main.rs
-------

#[cxx::bridge]
mod ffi {
        include!("path/to/test.h");
	type A;
	type testA;
	fn create_shared_ptr_A() -> SharedPtr<A>;
	fn create_shared_ptr_testA() -> SharedPtr<testA>;
	fn doCalculate(ptr_testA : SharedPtr<testA>, ptr_A : SharedPtr<A>);
}

fn main()
{
	let ptr_A = ffi::create_shared_ptr_A();
	let ptr_testA = ffi::create_shared_ptr_testA();

	ffi::doCalculate(ptr_testA, ptr_A);
}

The above code gives different shared_ptr count for x86 and arm builds.
shared_ptr ref count is fine with x86 output, but wierd output for arm build.
x86 output
### [RUST] ptr_A ref count : 1
### [testA CPP] ptr_A ref count : 2

ARM output:
### [RUST] ptr_A ref count : 1
### [testA CPP] ptr_A ref count : 0  ==> ref count is 0 ?

I used "cargo build" command for building with different target options for x86 and arm
Any thoughts on this ?
Thanks in advance.

Do you use the same compiler and same configuration of flags? Especially, do you use the same standard version (e.g., c++17) in both (you didn't post your build.rs and your Makefile or similar for C++ .so).
In any case, since the code compiled by the bridge does give you proper refcount of 1, but the code in the .so doesn't. I strongly doubt this has to do with cxxbridge, it seems like discrepancy between compilers/standard libs used by the bridge and the shared library.

c++14 version is used in both (build.rs and C++ library too).
Below is build.rs
fn main()
{
    cxx_build::bridge("src/main.rs")
        .flag_if_supported("-std=c++14")
        .file("src/test.cc")
        .compile("test");

    println!("cargo:rerun-if-changed=src/main.rs");
    println!("cargo:rerun-if-changed=src/test.cc");
    println!("cargo:rerun-if-changed=src/test.h");

    println!("cargo:rustc-link-lib=<cpp_library>.so");
    println!("cargo:rustc-link-search=<cpp_lib_path>");
}

We are cross-building for arm in x86_64 linux machine.
Our Cargo.toml has cxx dependency added like below.
[dependencies]
cxx = "1.0"

[build-dependencies]
cxx-build = "1.0"

We are cross-building for arm in x86_64 linux machine.

OK, that sounds like you have a high risk of compiler/stdlib mismatch.
You likely still did not check the versions of the C++ compiler/linker and C++ stdlib used, since you didn't post them. Version of cxx crate is irrelevant.


Update CCI Examples

Update the internal circleci orb documentation.

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/promiseofcake/workflow-queue?version=dev:0dd26b4ce9e8eb001d8e4bbda0500346bb78c898

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/promiseofcake/workflow-queue?version=dev:0dd26b4ce9e8eb001d8e4bbda0500346bb78c898

Your orb has been published to the CircleCI Orb Registry.
You can view your published orb on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/promiseofcake/workflow-queue?version=3.0.2


reducing time taken for creating wrapper object

Expected Behavior
when the wrapper object is not available for the element, it takes 6-7 seconds before moving on the next step which makes the overall script slow. Is there a way to decrease the time taken for this wrapper object creation\fail
Steps to Reproduce the Problem

I tried using the depth argument to reduce the depth of child_window to decrease the time taken but it's not working

Short Example of Code to Demonstrate the Problem
app = application.Application(backend="uia").connect(title=app_name, timeout=10)
wo = getattr(app, app_name).child_window(titlle="Submit", control_type="Button").wrapper_object()

@vasily-v-ryabov Hi, I would really appreciate if this problem can be solved.

Hello,
Since you know the element name and probably properties of the Submit Button, you can save it to a variable and use wait with a less timeout to eliminate the 6-7 seconds that it takes.
Something like below would be worth a try 😊
button = app.Submit
try:
    button.wait(timeout=2)
except pywinauto.timings.TimeoutError:
    pass


Improve stability when fracture occurs

Currently, FPS dips greatly when fracture begins. Need to search and apply optimizations to maintain stable FPS throughout the simulation

Try to implement active/asleep islands to localize the fracture more and reduce starting workload


Make sure that create.go propagates back validation errors correctly.

This is a unit test verifying that errors from Validations.go are returned.
cc @nikhiljindal @madhusudancs

This change is

/lgtm


Roll Flutter Engine from 1ae1d5318257 to e8f67d917bec (3 revisions)

https://github.com/flutter/engine/compare/1ae1d5318257...e8f67d917bec
2023-11-22<EMAIL_ADDRESS>Roll Skia from 23b9316efd20 to cebd44423589 (1 revision) (flutter/engine#48304)
2023-11-22<EMAIL_ADDRESS>[Impeller] make host buffer state internally ref counted. (flutter/engine#48303)
2023-11-22<EMAIL_ADDRESS>Roll Skia from b6f33389cefa to 23b9316efd20 (2 revisions) (flutter/engine#48302)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Refactor Format class

Description
Closes #3149 .
I tried to pull that PR and continuing from that but when I rebased to latest develop I got merge conflicts. I am not yet adept in solving merge conflicts, even IDE-aided, so I opted for the long road. This PR also adds tests for the new class addition and new inserted function to Services.
Checklist:

[x] Securely signed commits
[x] Component(s) with PHPdocs
[x] Unit testing, with >80% coverage
[x] User guide updated
[x] Conforms to style guide

Good job, my PR just missed tests, I asked for help with it but you know. ..

@paulbalandan is this ready?

@paulbalandan is this ready?

I dunno. Is there more to add here? It's ready regardless. Sorry, I'm weak at colloquial English so I don't quite get what Lonnie last meant. 😂

I think it is good! That conversation resolved.

"Bummer" = that is unfortunate
"Can't have it all" = you cannot get everything you want

Thanks to @paulbalandan and @mostafakhudair for this one!


Add SIMD suppport for some functions

SIMD optimized:

mat4.multiply
vec4.transformMat4

What did:

Saved backward compatibility
Saved legacy functions and added SIMD versions (required support it)
Added Edge browser SIMD support
Not support load/store by Edge, but added Firefox compatibility. Planned workaround for Edge.
Used new SIMD syntax (unlike Firefox) by default
Used standart capitalized letters
If slightly modify code (common.js), can be used other SIMD runtimes
Used glMatrix.FLOAT32X4 (runtime) and glMatrix.SIMD_SUPPORT (flag) variables
Used static SIMD detection and function choise (on init)

Later will integration of SIMD (will used SIMD objects). And TypeScript support.

I would recommend waiting until #172 lands, then refactoring this patch to work with the runtime detection and testing we've set up there.

Please add vec4.transformMat4


Picking in 3D scene

This PR provides an API to perform picking in a 3D scene from the items.
One can call item.pickItems(xInPixel, yInPixel, condition) on a group item to get an iterator of PickingResult objects.
A PickingResult object provides access to the picked item, the positions picked (in either scene or object frame of reference), indices in the item's data array, and picked data of corresponding data point if applicable.
related to #2022

Added a PositionInfoWidget displaying the closest picked item data point value and position upon double-click (used in plot3dSceneWindow.py example)
Updated documentation

Added tests + a few fixes

Ready for review.
There is still a few things to do, but this PR is already large and the main feature is there.
I'd prefer to continue to work on it in other PRs.
The plot3dSceneWindow.py sample code can be used to test the picking.
In the plot3d API, it adds:

silx.gui.plot3d.SceneWidget.SceneWidget.pickItems(x, y, condition) which yields silx.gui.plot3.items._pick.PickingResult objects.
silx.gui.plot3d.tools.PositionInfoWidget.PositionInfoWidget which displays picked position and data value upon double click in the associated SceneWidget.

Remaining things:

Provide alternative ways to handle the picking: a dedicated mode with a button, perform picking in whole scene/only in selected item(s)
Add picking of 3D meshes
Improve picking of scatter plots in wireframe/surface mode
Benchmark and optimizations


koka: 2.3.2 -> 2.3.6

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/koka-lang/koka/releases.
meta.description for koka is: "Koka language compiler and interpreter"
meta.homepage for koka is: "https://github.com/koka-lang/koka"
meta.changelog for koka is: "https://github.com/koka-lang/koka/blob/master/doc/spec/news.mdk"
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/6jqbcfsh02gmksv56zy95dyc9ck7h2jv-koka-2.3.6/bin/koka had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/6jqbcfsh02gmksv56zy95dyc9ck7h2jv-koka-2.3.6/bin/.koka-wrapped had a zero exit code or showed the expected version
0 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 2.3.6 with grep in /nix/store/6jqbcfsh02gmksv56zy95dyc9ck7h2jv-koka-2.3.6
found 2.3.6 in filename of file in /nix/store/6jqbcfsh02gmksv56zy95dyc9ck7h2jv-koka-2.3.6

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

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
koka

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

koka

Maintainer pings
cc @siraben @sternenseemann for testing.

New regression for x86_64-darwin according to ofborg?!

Yes: https://hydra.nixos.org/eval/1729064?filter=koka&compare=1728792&full=

I would recommend to just report it upstream.

Checked and it seems that the problem is clang 7.1.0. Since clang 11 works fine, I'm sending this (single rebuild change) to staging-next which uses clang 11 as the default.

https://github.com/koka-lang/koka/issues/222


[stable/parse] adds parse chart

Deploys a parse server and optionally the parse dashboard

Marking as stale. Please update within the next week.

Closing as stale. Feel free to re-open if/when you decide to work on this again.

Sorry, I missed your earlier comment.
I've applied the standard labels to the Deployment templates as and updated the image versions, etc.

applied the required changes. thanks for the review.

updated! thanks for the review.

updated!

/re-test


Corruption of const input string that use C.O.W.

GCC-4.6 seems to use C.O.W. (Copy-On-Write) for std::strings.
When calling std::string modp::b85_decode(const std::string& s), the input
string s is corrupted.
This is due to the fact that although 's' is "copied" into an internal copy
'x', the copy is not really done. A further call to the non-const b85_decode()
corrupts s, since it uses const_cast to strip the const from the .data()
returned results (presumably to avoid a copy at all costs on all
implementations).

What steps will reproduce the problem?
1. On gcc (with C.O.W.), pass an std::string 's' to modp::b85_decode(const
std::string& s);
2. 's' is now corrupted.

What is the expected output? What do you see instead?
A call to a function that accepts const should not alter the const argument.

What version of the product are you using? On what operating system?
gcc 4.6 on Linux.

Please provide any additional information below.

One possible fix is to change the implementation to:
     inline std::string b85_decode(const std::string& s)
     {
         // std::string x(s); // original impl.
         std::string x(s.data(), s.size()); // new impl. - force copy even with COW
         b85_decode(x);
         return x;
     }

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 1 Nov 2012 at 8:37

pull requests welcome


Setting new CLARISA project

Create documentation over CLARISA separation from MARLO, Architecture, Database, Components. etc

Repository created; MARLO and CLARISA are separated.


Express socket io session no longer sharing sessions with express

For some odd reason this library is no longer working with my nodejs configurations. The sessionID of the socket session is never the same generated after the HTTP login.
I login the user via http and start his session. Then, I connect to the sockets. However, the socket connection always generates a new socketID.
I tried to DEBUG=express-socket.io-session but still I could not find the problem.
Here is my code and info
"express-socket.io-session": "^1.3.2"
"socket.io": "^2.0.3"
"express": "^4.15.4"
"express-session": "^1.15.5"

    var io_session = require("express-socket.io-session");
    var e_session = require("express-session");

    var sessionFileStore = require('session-file-store')(e_session);

    var ee_session = e_session({
        store: new sessionFileStore({ path: './user_sessions' }),
        secret: "something-random",
        resave: true,
        saveUninitialized: true
    });

    var enableCORS = function(req, res, next) {
        res.header('Access-Control-Allow-Origin', req.headers.origin);
        res.header('Access-Control-Allow-Methods', 'GET,PUT,POST,DELETE,OPTIONS');
        res.header('Access-Control-Allow-Headers', 'Origin, Accept, Content-Type, Authorization, Content-Length, X-Requested-With, *');
        res.header('Access-Control-Allow-Credentials',true);

            // intercept OPTIONS method
        if ('OPTIONS' == req.method) {
            res.sendStatus(200);
        } else {
            next();
        };
    };

    app.use(function(req, res, next) {
      ///
       next();
    });

    //app.use(cookieParser());
    app.use(enableCORS);
    app.use(bodyParser.urlencoded({ extended: true }));
    app.use(bodyParser.json());
    app.use(ee_session);

    preparedApp = require("http").Server(app);

    var io = require("socket.io")(preparedApp);

    io.use(io_session(e_session,{
        autoSave: true
    }));

    preparedApp.listen(8080, function(){});

    io.on('connection', function(socket){

        var socket_session = socket.handshake.session,
            socket_session_id = socket.handshake.sessionID;

        console.log("SOCKET_SESSION:",socket_session); // NOTHING IN COMMON WITH THE HTTP SESSION
        console.log("SOCKET_SESSION_ID:",socket_session_id); // Different session ID from the http one.

        (...)

Other things I noticed:
1 - The http session is always the same and is maintained
2 - The socket sessions is always the same and is maintained
3 - They are both different. As if only the http sessions are being accessed in ./user_sessions and the socket ones are not.
Any idea?

@oskosk If you know something about an update in packages that could have broken this let me know please. Thank you so much for your work :) :+1:

@oskosk I've been debugging anywhere and everywhere but still it's not working. I just can't figure on why the socket is not catching the session

I've setup a testing dir with the code you put here and will be testing it in some time. Got it a few calls on the last hours .

Okok @oskosk thanks, let me know :)

This is, quite literally, draining the life force out of me

@oskosk the socket ids / sessions created by socket.io are not saved in any folder, or so it seems. When I send a socket packet and console.log the socket session in the server it is the same ID though. So it must be saved. It just is not in ./user_sessions or the default ./sessions in expressFileStore...

@oskosk I just noticed that despite the code being as I sent you, sessionStore in the ?
socket.handshake still comes as this (notice path and all)

One question. In the snippet you pasted before...
    io.use(io_session(e_session,{
        autoSave: true
    }));

looks like it should be ee_session instead. Like
    io.use(io_session(ee_session,{
        autoSave: true
    }));

Is this possibly causing some problem ?

@oskosk nope, I have ee_session. Sorry, bad paste but that was correct indeed...

@oskosk for some reason and even though socket.handshake sessionStore points to the same folder as the express session "./user_sessions" only the express session files are created. But the socket.io session is not destroyed. It is persitent.

@oskosk if you need some variable or info or dump let me know please

I haven't really used expressFileStore before, but testing locally now, it works for me. I get the .user_sessions folder with a file inside it named after the session Id. And this session Id is maintaned among refreshes of the page. You debuggin code, shows the same data after refreshes. in my console. I'm using the code you pasted here and just added what's missing in that snippet for leveraging an index.html.
I also tested removing that storage lib so sessions are kept in memory and sitll works.

Damn. Across the socket too @oskosk ? A simple io(url) is needed to connect to the sockets and use session correct?

Yes, across sockets. So I get this file which is created by the express middleware part...

And with your debug code, I get to see this which is coming from the socket.io middleware

A simple io(url) is needed to connect to the sockets and use session correct?

Yes, you don't even need to use url (just io()) if connecting to the same server that serves the html.
Have you tried running your js server like this to see if any package adds warnings to the run ?
$ NODE_ENV=development node index.js

This can also help because a lot of packages use the debug package to log about internals.
$ DEBUG='*' node index.js

@oskosk good idea, I found something, here is the trace:
This is what happens when I connect to the sockets after the login. "Cookie signature invalid"? Any idea?
engine intercepting request for path "/socket.io/" +19s
  engine handling "GET" http request "/socket.io/?EIO=3&transport=polling&t=LzP-MhI" +9ms
  engine handshaking client "LsXpFH0KxiunVcLOAAAA" +11ms
  engine:socket sending packet "open" ({"sid":"LsXpFH0KxiunVcLOAAAA","upgrades":["websocket"],"pingInterval":25000,"pingTimeout":60000}) +3ms
  engine:polling setting request +6ms
  engine:socket flushing buffer to transport +1ms
  engine:polling writing "97:0{"sid":"LsXpFH0KxiunVcLOAAAA","upgrades":["websocket"],"pingInterval":25000,"pingTimeout":60000}" +5ms
  engine:socket executing batch send callback +3ms
  socket.io:server incoming connection with id LsXpFH0KxiunVcLOAAAA +1ms
  socket.io:client connecting to namespace / +2ms
  socket.io:namespace adding socket to nsp / +0ms
  express-socket.io-session Using autoSave feature. express-session middleware will be called on every event received +19s
  express-session cookie signature invalid +12s
  express-session cookie signature invalid +0ms
  express-session no SID sent, generating session +0ms
  socket.io:socket socket connected - writing packet

I don't really know what's happening there. But, why don't you try to use the cookie-parser middleware explicitly? In your current case, it looks like you're not using it explicitly and express-socket.io-session just creates one on the fly for you. It's working for me, but what if this is causing any issue on your setup... Can you check it? When you try it, also check with the NODE_ENV and DEBUG env variables to see if you get extra messages too.
There's a short snippet in the README.md here under the title Using your own custom cookie-parser instance.

Alright @oskosk, I'll take a go at this but what would the point be exactly? Should I just open the cookieParser? What logic should go inside the cookieParser middleware to solve this?

@oskosk I did pass cookieParser manually to both express and the socket.io express session library, with the secret, but the same error persists. I just can't figure out why this is happening. Seems haunted.

@oskosk I finished the login after the first line, where express-session saves the string id. The rest of the output (from the second line) appeared after I clicked the button "Connect to sockets". Any idea on why this might be happening? Why does the angular2 environment send a socket connection request that is intercepted through /socket.io/ and with an invalid cookie signature?

@oskosk Check this out please
Here is the terminal from a regular http session / re-session. As you can see the session is found normally and the user logs in (check the HEADER variable that I logged):

Now, this one is the socket connection request, after the login, going through the same algorithm:

Any further ideas? I feel like I am getting somewhere now

@oskosk Could this be related to the fact that I don't refresh after the login in my Angular2 environment and so the cookies are not set yet when the socket.io client connects to the backend? Although the http session seems to be maintained without refresh so the cookies must be set...

@oskosk I solved it.
Http was reaching <IP_ADDRESS>:8080
Socket.io was reaching localhost:8080
Just FML and F cross-domain requests.
Thank you so much for your help anyway @oskosk. For real, thank you! I'll keep on using your library :)

niceeeeee! Thanks for sharing the solution!


Swedish locale not working since 4.0.0

Swedish locale not working. Day/month names defaults to English, and time-format defaults to 12-hour.
Tried other locales (fr,de,es), by setting the language configuration option, and they all work fine.
Expected behaviour
Day/month names should translate and time-format should be 24-hour, when HA language are set to Swedish or when explicitly set by the language configuration option.
Current behaviour
Day/month names defaults to english, and time-format defaults to 12-hour.
Steps to reproduce

Set up an Atomic Calendar Card in Lovelace.
Set both Home Assistant locale to Swedish and language configuration option to sv.
Nothing changes...
Manually install v3.4.1 from github and it's working again.

Context
Stopped working after upgrading from v3.4.1. I thought it was related to #450, but upgrade to v4.4.2 doesn't help.
Your environment

Home Assistant core-2021.7.1
Home Assistant supervisor-2021.06.8
Home Assistant OS 6.0
atomic-calendar-revive v4.4.2

Stack trace / logging
Nothing appears in log...
Checklist

[x] I searched the current issues and the issue is not known.
[x] I searched the troubleshooting and there's no fix available for this bug.
[x] I searched on Google, YouTube, Reddit and there's no fix.
[x] I am mostly sure that the problem is related to the code and isn't my fault.

Maybe related to change in Home Assistant described in #445?


update readme to mention windows support

As of #40, notifications work on windows (if I understand the PR correctly :)).

Huch! You're right, I have to admit I never properly tested it myself, so I can't guarantee it's stability at the moment, therefore I would argue "basic window support"

I can tackle proper windows support down after my exams :)


Path to plugin.js seems incorrect

I've just noticed the text plugin failing to load locally after it couldn't find plugin.js.
On inspecting the path locally I've got static/djangocms_text_ckeditor/ckeditor_plugins/cmsplugins/plugin.js
But in cms.ckeditor.js#L60 the path seems to miss out the djangocms_text_ckeditor directory;
CKEDITOR.plugins.addExternal('cmsplugins', settings.static_url + '/ckeditor_plugins/cmsplugins/');

Should that not be;
CKEDITOR.plugins.addExternal('cmsplugins', settings.static_url + '/djangocms_text_ckeditor/ckeditor_plugins/cmsplugins/');

Path should be correct (settings.static_url is set here https://github.com/divio/djangocms-text-ckeditor/blob/2.4.3/djangocms_text_ckeditor/templates/cms/plugins/widgets/ckeditor.html#L25 and includes djangocms_text_ckeditor part)
Have you recently updated from older version?

Don't think I've updated recently, and it only seemed to happen locally so maybe just an odd scenario on my machine I guess. If I see it on a remote machine I'll come back! Thanks @yakky


Apps crashes when centeredIndexPath is being called

Hi,
If number of items is zero, attemp to call collectionView?.centeredIndexPath will cause "divide by zero" crash instead returning nil
Thanks.

Any fix on this?

Hi!
Can you provide some additional context around how you are getting the divide by zero crash? I've been unable to replicate.
You can see a modified version of the BaseCollectionViewController.swift from the example project below. When you load this VC in the example app then centeredIndexPath returns nil as expected.

 BaseCollectionViewController.swift
override func viewDidAppear(_ animated: Bool) {
	print(self.infiniteCollectionView!.centeredIndexPath as Any)
}

override func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
    guard let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "cell", for: indexPath) as? Cell else {
        fatalError()
    }
    cell.update(index: self.infiniteCollectionView!.indexPath(from: indexPath).row)
    return cell
}

override func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
    return 0
}

override func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) {
    collectionView.scrollToItem(at: indexPath, at: self.infiniteCollectionView!.infiniteLayout.scrollDirection == .vertical ? .centeredVertically : .centeredHorizontally, animated: true)
}


Using the incorrect name for an API key throws exception: call of reflect.Value.SetString on zero Value

When setting up API keys if you specify the wrong key value an ambiguous exception is thrown:

➜  subfinder git:(master) ./main --set-config VirusTotalAPIKey=

/ _/ / /  / (_) / / ____
\ / // / _ / // / _ / _  / -) /
//_,/_.// /////_,/_/_/
SubFinder v0.1.0 	  Made with ❤ by @Ice3man
panic: reflect: call of reflect.Value.SetString on zero Value
goroutine 1 [running]:
reflect.flag.mustBeAssignable(0x0)
/usr/lib/go-1.8/src/reflect/value.go:221 +0x195
reflect.Value.SetString(0x0, 0x0, 0x0, 0xbfd7746e, 0x40)
/usr/lib/go-1.8/src/reflect/value.go:1529 +0x21
main.main()
/opt/subfinder/main.go:89 +0x208
➜  subfinder git:(master) ./main --set-config VirustotalAPIKey=

Note the misspelling of VirusTotal (should be Virustotal for successful addition).

That doesn't seems like an issue since reflect needs correct name to change values at run time. Also, the README.md document correctly states the names we need to modify. Any ideas?

I think updating this to show a more accurate exception would be the best move. If you don't put 2+2 together this can be confusing for the end user in its current form.


Elements not rendering

I love the library, you've done some really awesome work here.
However, I'm having a hard-to-reproduce issue with a presentation where a large number of the elements on screen stop rendering; possibly in order from the top (the first to go is invariably the background gradient).
This happens:
i) Only when the window size is above a certain point (I can resize the window to have the elements reappear)
ii) Only if there are a certain number of elements in the document (I can hide steps until everything else starts appearing)
iii) Only if there are a certain number of elements in the window (my first step is scaled to show everything on the window, but the background redraws once I navigate to another step and only a small proportion of the elements are onscreen).
iv) But the browser can definitely cope with rendering this number of elements: if I zoom out (in the browser) from one of those subsequent steps to put all the elements back in view, they render just fine.
I've made substantial use of SVG elements. So there's that.
I've reproduced with Chrome 18.0.1025.11 and 19.0.1042.0 across two Windows machines, but also had it working fine on a third machine and also on a Mac in Chrome + Safari. It also renders fine in Firefox.
I'm unsure if this is a webkit-windows bug or something with impress.js? Has anyone else encountered this? I did think there might be a canvas element limit, but that doesn't make sense if I can zoom out and see all of the elements.
I'll try to demonstrate a test case, but I need to anonymise my document first.

Hard to say anything without live example to test with.
Sometimes I've seen similar problems (with disappearing elements) on nightly builds of Chromium, so I think it's probably a browser bug. CSS 3D transforms are still quite fresh functionality, so they may behave (and break) in unexpected ways.

Ah , here I though I was alone :)
Yes, seeing the same thing, I have a somewhat large presentation and I started losing stuff. I do a particular "overview" of a few slides (slide as in the ones you have on impress-demo.css) as if it were a matrix and with each new sub-element, pieces of the "matrix" started to go away.
Browser breakdown:

Chromium 17.0.963.56 on Linux -  I have missing tiles, normal gradient background
Chrome 19.0.1041.0 on Linux - Missing tiles (but not the same back-to-front pattern) and the background is white, almost as if an element was covering the background (but there isn't one :) )
Firefox 10.0.2 on Linux - Everything is apparently fine, but it's so slow it's embarrassing :)
Safari (5 I think) on Mac/windows - perfectly ok

The mind boggles, but either way, I know it's bleeding edge stuff , yada yada :) . Would you want the code to debug, or is this mostly a sit-down-and-wait thing?
either way, impress.js is impressive :) , thank you for that

I've had it on Linux, too.
So it seems to be Chromium Linux issue. May even be hardware related.

It's definitely not a Linux only issue. I've seen it on Windows and Mac builds of Chrome.

See this bug report http://code.google.com/p/chromium/issues/detail?id=110457 Maybe it is the same problem you have.

Just switched over to Windows :

Chrome 19.0.1041.0 dev-m is slightly better than on Linux, at least it shows all the elements, but sometimes it will go all white on transitions
Safari 5.1.2 is, however, running flawlessly

god damn it :)

I think @bartaz and @mschaaf are right. It's a documented Chromium bug with 3D transforms. I'm seeing it with:

18.0.997.0 (Developer Build 116462 Linux) Built on Ubuntu 11.10, running on LinuxMint 12

Ok, thanks for confirmation that it's a browser bug.
CSS 3D are still an experimental feature, so such things will happen.
I did my best to make it work as good as possible, and can't do anything to 'fix' this issue.


[FEATURE] Better Errors in Failed Tests

Describe the solution you'd like
Make better errors for failed tests rather than the usual assertion errors, so it's clearer what specifically failed
Is your feature request related to a problem? Please describe.
It's unclear what errors are happening in some cases
Describe alternatives you've considered
None
Additional context
N/A

Moved to Tadukoo Util, since this project has been moved there


B2Bds-194 Adds Wizard Component

Worked on top of @brandtf2512 commit.

nice that this will be part of the library and thanks for implementing this so fast!


http3: Low Network Usage Efficiency with HTTP3 in quic-go

We have been using quic-go with HTTP/3 to fetch files in parts from CloudFront. During our tests, we varied latencies (100ms & 1000ms) and bandwidths (50KBps, 1.25MBps, 3.75MBps). Latency and bandwidth were emulated using tc and netem tools.
Each part size was set to 5% of the bandwidth, and we fetched 20 parts in parallel by making 20 HTTP/3 Range Requests concurrently.
For every transfer, we captured the network traffic using tshark (v3.6.2). We then calculated the network usage efficiency as File size in bytes / Total bytes on the wire (both sent and received bytes).
Observed Results:

For small file sizes (10KB, 100KB), we observed very low network usage efficiency (~65%) for low bandwidth (50KBps). This is understandable due to the high overhead relative to the small file size.
For larger file sizes (1MB, 10MB, 100MB), we still observed ~65% efficiency at low bandwidths.
For higher bandwidths, efficiency improved to ~85%-91%, potentially due to fewer HTTP/3 requests needed.

We still feel that 85% efficiency is low.
Questions:

Has anyone else performed similar testing? I found a related discussion here, but it does not use quic-go.
Is this level of efficiency expected with QUIC/HTTP/3?
Can you see any issues with our testing methodology?

Additional Context:
1-2 months ago, we conducted a similar calculation using qlog. We summed the length of all packet_sent and packet_received events and observed an efficiency of ~93%-95%. This might be because HTTP/3 qlog data was not accounted for, as it's not supported yet (an existing issue acknowledges this).
Let me know if you need any further details. Thanks!
PS: This might not be an issue with quic-go itself, but we are seeking insights from QUIC protocol experts to understand QUIC/HTTP/3 better.

Framing overhead should be pretty low. On the QUIC layer, the largest overhead is probably the QUIC packet header (depending on the connection ID length). There's also overhead from STREAM frames, and occasionally other control frames (such as ACK and MAX_DATA frames, for example). Framing overhead on the HTTP/3 layer should be very low, as it is possible to use DATA frames with a large size.
If you want to dig into this deeper, it would probably make sense to classify the overhead you're seeing into (at least) the 4 categories I mentioned above.

I forgot to mention another source of overhead you might be seeing, which is (spurious?) retransmissions.
Regarding methodology, going by qlogs is an option, but qlog wasn't really designed to capture the exact byte representation on the wire (e.g. how a varint was encoded), so you won't be able to get exact numbers. You can import an SSLKEYLOG file into Wireshark, allowing it to decrypt QUIC packet contents.

Yes, we were able to decrypt the QUIC packet contents in Wireshark. I was wondering how to identify retransmissions, I believe qlog does not emit an event for retransmission, is there a way to identify that?

QUIC doesn't retransmit packets. It retransmit frames. That makes identifying retransmissions more challenging. For STREAM frames, you can check if the same stream offset was already sent before. If I remember correctly, Wireshark even does that for you. Obviously, this means that you need to record the pcap close to the server side (i.e. before the network segment where the packet loss happens).


5.6.1在linux系统上报错

Bug Description

What did you expect to happen?
Please enter the details...
System, Browser, and Version

Operating System
Browser and Version

wangEditor-next Version
Please enter the details...
Can the demo reproduce the bug?
Yes/No

Demo: https://cycleccc.github.io/demo/

Online Demo

SandBox url: https://stackblitz.com/@cycleccc/collections/wangeditor

Steps to Reproduce at Minimum Cost

Step 1
Step 2
Step 3

重复 bug
https://github.com/cycleccc/wangEditor-next/issues/256
已在 5.6.2 解决。


Add helper to allow to set cookies in ‘ServantErr’

Awesome, thanks! Merged.

@zohl Can you upload a new release with this change included?

Sure, done. The new version is 0.3.1 and can be found at hackage.


VueDatePickerProps textInput doesn't use the TextInputOptions interface

I am writing a wrapper component to VueDatePicker, in expanding the props I have a TypeScript problem with the textInput prop, I noticed that you didn't use the TextInputOptions interface in the type declaration of VueDatePickerProps but you redefined the interface. For some reason, TypeScript isn't able to compute that correctly, and therefore when I try to redefine the props' defaults, it gives me the following:
export type BsdkNewDatePickerProps = Omit<VueDatePickerProps, 'uid'> & {
  id?: string
}

const _props = withDefaults(defineProps<BsdkNewDatePickerProps>(), {
  enableTimePicker: false,
  textInput: {
    enterSubmit: true,
    tabSubmit: true,
    openMenu: 'open',
    rangeSeparator: '-',
  },
})

Object literal may only specify known properties, and 'enterSubmit' does not exist in type '(props: LooseRequired<BsdkNewDatePickerProps>) => boolean | { enterSubmit?: boolean | undefined; tabSubmit?: boolean | undefined; openMenu?: boolean | ... 2 more ... | undefined; rangeSeparator?: string | undefined; selectOnFocus?: boolean | undefined; format?: string | ... 2 more ... | undefined; }'.ts-plugin(2353)

index.d.ts(191, 5): The expected type comes from property 'textInput' which is declared here on type 'InferDefaults<LooseRequired<BsdkNewDatePickerProps>>'

I believe this can be fixed by setting the type in  textInput prop to
boolean | TextInputOptions

instead of
textInput?:
        | boolean
        | {
              enterSubmit?: boolean;
              tabSubmit?: boolean;
              openMenu?: 'open' | 'toggle' | boolean;
              rangeSeparator?: string;
              selectOnFocus?: boolean;
              format?: string | string[] | ((value: string) => Date | null);
          };

let me know if this is the solution or if I am overseeing anything, thank you.
Desktop & mobile (please complete the following information):

Library version - 10.0.0

I find the problem, I forgot to wrap the textInput object in an anonymus function, now it works if I do this:
const _props = withDefaults(defineProps<BsdkNewDatePickerProps>(), {
  enableTimePicker: false,
  textInput: () => ({
    enterSubmit: true,
    tabSubmit: true,
    openMenu: 'open',
    rangeSeparator: '-',
  }),
})

But still, I think that it would be better if VueDatePickerProps use the exposed interfaces instead of redifining them, to avoid possible errors in the next versions, not only for the textInput prop, but for the following as well: filters, transitions, presetDates, ariaLabels, highlight and disabledTimes (if there are any other interfaces that are getting exposed)


Don't see any way to access menu

I notice that ability to see currently installed version number was added via about in the menu, but I don't see any way to get to an app-level menu.

Yeah, electron menu doesn't show at gnome. Let me check how to show that.
For now, you can get version: In app -> Resource Center -> Get app info

Hi @stevecj for menubar issue, can you try to press Alt key to see if you can get menubar? It seems some settings in system. In our Ubuntu 20.04, most apps don't have menubar. Only Firefox can get that by right click to show it.
BTW I only add about menu at Tray.

Alt key did not work for me, but app -> Resource Center -> Get app info does, and I'm happy with that.


support for sharing views

Trac Ticket #14
component: website
owner: somebody
reporter: martin
created: 2016-07-03 21:15:26
milestone:
type: task
version:
keywords:
currently it is not possible to share what I see. when I share the link the other user will arrive at the front page but not with the stuff I've selected.
instead create shareable links. for example using

the # symbol: http://localhost/stats-website/#tab=donut&timestart=2010-09-10&select=CellML
the above with a ?
some session id or something, similar to combinearchive-web:wiki http://localhost/stats-website/share/123abc456

see also #12

selected date range, filters and visualization are stored in url
TODO: store diff / graph tab


MGDSTRM-3724 Add the status code to the keycloak_request_duration_bucket metric

Motivation
https://issues.redhat.com/browse/MGDSTRM-3724
https://issues.redhat.com/browse/MGDSTRM-4318
What
Add status code and uri to the keycloak_request_duration_bucket metric
Why
This will allow filtering on specific statuses
Currently no status are returned in the metric
keycloak_request_duration_bucket{method="POST",resource="realms,realms/master/protocol/openid-connect",le="50.0",} 0.0 keycloak_request_duration_bucket{method="POST",resource="realms,realms/master/protocol/openid-connect",le="100.0",} 0.0 keycloak_request_duration_bucket{method="POST",resource="realms,realms/master/protocol/openid-connect",le="250.0",} 0.0 keycloak_request_duration_bucket{method="POST",resource="realms,realms/master/protocol/openid-connect",le="500.0",} 0.0 keycloak_request_duration_bucket{method="POST",resource="realms,realms/master/protocol/openid-connect",le="1000.0",} 1.0 keycloak_request_duration_bucket{method="POST",resource="realms,realms/master/protocol/openid-connect",le="2000.0",} 1.0 keycloak_request_duration_bucket{method="POST",resource="realms,realms/master/protocol/openid-connect",le="10000.0",} 1.0 keycloak_request_duration_bucket{method="POST",resource="realms,realms/master/protocol/openid-connect",le="30000.0",} 1.0 keycloak_request_duration_bucket{method="POST",resource="realms,realms/master/protocol/openid-connect",le="+Inf",} 1.0 keycloak_request_duration_count{method="POST",resource="realms,realms/master/protocol/openid-connect",} 1.0 keycloak_request_duration_sum{method="POST",resource="realms,realms/master/protocol/openid-connect",} 625.0
How
added status code and uri to the to the bucket output.
Verification Steps
Option 1

Build the code
Copy the artifact into $KC/standalone/deployments
Make sure the metrics logger is turned on
Play with Keycloak a bit
Look at the following url: "$KC/auth/realms/master/metrics"

Option 2
update keycloak cr to use the jar built and released from my fork
https://github.com/CathalOConnorRH/keycloak-metrics-spi/releases/tag/v2.4.1-beta
Metrics should look similar to
keycloak_request_duration_bucket{code="200",method="POST",resource="realms,realms/master/protocol/openid-connect",uri="realms/master/protocol/openid-connect/token",le="50.0",} 0.0 keycloak_request_duration_bucket{code="200",method="POST",resource="realms,realms/master/protocol/openid-connect",uri="realms/master/protocol/openid-connect/token",le="100.0",} 0.0 keycloak_request_duration_bucket{code="200",method="POST",resource="realms,realms/master/protocol/openid-connect",uri="realms/master/protocol/openid-connect/token",le="250.0",} 1.0 keycloak_request_duration_bucket{code="200",method="POST",resource="realms,realms/master/protocol/openid-connect",uri="realms/master/protocol/openid-connect/token",le="500.0",} 1.0 keycloak_request_duration_bucket{code="200",method="POST",resource="realms,realms/master/protocol/openid-connect",uri="realms/master/protocol/openid-connect/token",le="1000.0",} 1.0 keycloak_request_duration_bucket{code="200",method="POST",resource="realms,realms/master/protocol/openid-connect",uri="realms/master/protocol/openid-connect/token",le="2000.0",} 1.0 keycloak_request_duration_bucket{code="200",method="POST",resource="realms,realms/master/protocol/openid-connect",uri="realms/master/protocol/openid-connect/token",le="10000.0",} 1.0 keycloak_request_duration_bucket{code="200",method="POST",resource="realms,realms/master/protocol/openid-connect",uri="realms/master/protocol/openid-connect/token",le="30000.0",} 1.0 keycloak_request_duration_bucket{code="200",method="POST",resource="realms,realms/master/protocol/openid-connect",uri="realms/master/protocol/openid-connect/token",le="+Inf",} 1.0 keycloak_request_duration_count{code="200",method="POST",resource="realms,realms/master/protocol/openid-connect",uri="realms/master/protocol/openid-connect/token",} 1.0
Checklist:

[x] Code has been tested locally by PR requester
[ ] Changes have been successfully verified by another team member

Progress

[x] Finished task
[ ] TODO

Additional Notes
PS.: Add images and/or .gifs to illustrate what was changed if this pull request modifies the appearance/output of something presented to the users.

/lgtm

@pb82 Would like to take a look as well?

LGTM

@CathalOConnorRH if it is absolutely required to add those labels to the histogram, can we make that opt-in behind a flag?

@pb82 I've updated the ResourceExtractor to include a URI_METRICS_ENABLED boolean and a check for /token URI.
an example of the metrics output is attached
Let me know what you think
KeycloakMetrics.txt

Hey @pb82 Have you had any thoughts on the above changes ?

thanks @CathalOConnorRH I'll give it a final try and then merge

@pb82 did you find time to try it out? Just curious about the status. Thanks!

@pb82  I've implemented the detailed output and removed the uri when it's not enabled.
Some sample output of each of the three scenarios are attached.
Let me know what you think
URIEnabledURIDetailed.txt
URIDisabled.txt
URIEnabled.txt

@pb82 We need this change in MAS SSO soon, to build some metrics - dashboards and also to investigate and get precise information regarding our load (which end points are used most). Can you take a look and close this at the earliest?


fireplace: init at 0-unstable-2020-02-02

Description of changes
A cozy fireplace in your terminal
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

Also please change the PR title and commit message to fireplace: init at 0-unstable-2020-02-02

Thank you for a great review @eclairevoyant!

Thank you for a great review @eclairevoyant!

as per @eclairevoyant

Also please change the PR title and commit message to fireplace: init at 0-unstable-2020-02-02

Thank you for a great review @eclairevoyant!

as per @eclairevoyant

Also please change the PR title and commit message to fireplace: init at 0-unstable-2020-02-02

Done!


Failed to remotely connect to WinAppDriver - Error Adding url to url group

None of the recommended Issues or past solutions have solved my problem. I'm trying to simply initiate remotely, both machines are in the same network and this are the steeps I've followed:

Get IP Address from the remote.
Open cmd as Administrator.
Run the following from the cmd:
C:\Program Files (x86)\Windows Application Driver>WinAppDriver.exe xxx.xxx.x.xx 4723/wd/hub

RESULTS:
Windows Application Driver listening for requests at: http://xxx.xxx.x.xx:4723/wd/hub
Error adding url to url group
Failed to initialize: 0x80004005
Machines Configs:

Both machines with Developer Mode on
WinAppDriver v1.1 stable on both machines
Firewall exceptions are implemented as the readme implies.

Found the problem, the README is incorrect or misguiding user, I've sent  #1184 pull request to update it.
I've just done as described in my pull request and it worked =)


Loosen gem dependencies.

Hello, I loosened the gems dependencies a bit.
Newer prawn versions ( >= 0.13, <= 0.15) have a ~> 1.2 dependency on pdf-reader so I was unable to use pdf-reader-turtletext on my project.
I ran the specs with pdf-reader 1.2 against ruby 1.9.3 and 2.0.0 without any problem.
So I changed the pdf-reader dependecy to: >= 1.1.1, <= 1.2
Thanks. Great gem, btw.

thanks @oesgalha. closing this as it is probably dealt with by #10


fix(infra): lattice prefix ENV var

Feature or Problem

Related Issues

Release Information

Consumer Impact

Testing

Unit Test(s)

Acceptance or Integration

Manual Verification

Ahh thanks @joonas  for noting that this chart is deprecated -- can I delete it now?

Ahh thanks @joonas for noting that this chart is deprecated -- can I delete it now?

It was removed in https://github.com/wasmCloud/wasmCloud/pull/3646

Ah OK! If this was removed already then will close this PR! thanks


MMDM-1742 Add migration to remove Lee's CAC access

Description
Removes CAC access. This doesn't have to be a secure migration because we're only identifying the sha256 digest in dev/experimental/staging/prod. And if it doesn't exist in those environments this is a noop.
Reviewer Notes
NOTE: This needs to be deployed manually to experimental to apply in that environment.
Setup
make db_dev_run db_dev_migrate

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against ec68a99edc3cddcc4a6c2aa8f1ca8b17f5002942


Update sbt-bloop to 1.4.8-126-57329401

Updates ch.epfl.scala:sbt-bloop from 1.4.8-19-4d9f966b to 1.4.8-126-57329401.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.epfl.scala", artifactId = "sbt-bloop" } ]

labels: sbt-plugin-update, semver-pre-release

Superseded by #98.


e2e: fix some test issues

Fixes the few remaining issues from cloud run review the other day.

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.


Use rich for displaying error traceback in mlinfra cli

This PR proposes the following changes

Closes #96

PR Checklist

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Tests

@ara-25 have you tested these changes? I tried to see rich based stacktrace, but that didn't work.

@ara-25 have you tested these changes? I tried to see rich based stacktrace, but that didn't work.

That's strange. Are you testing by running the mlinfra cli or some other module?
For me, the stack trace is showing upon running mlinfra  estimate-cost abc.yml

I'm testing it by trying to run the pyinvoke command directly, so by calling pyinvoke estimate-cost abs.yaml.

So, with my current implementation, the rich traceback only appears in mlinfra command or if cli.py is run directly. This is because from the original issue, I understood that this feature only needs to be added for the mlinfra cli, instead of the entire codebase.
Please let me know if the formatted traceback needs to be displayed for the entire codebase and I will modify it accordingly.

Ah alright, that makes sense.
Yes, the idea was to have this enabled for the entire codebase as it would be easier to spot where an error is coming from. It would be really cool if you can adjust it for the repo, or if you want i can create a new issue for that. Both are fine with me.

@aliabbasjaffri would be great if a separate issue can be created for that :)

Just tested and verified. Thank you for your contribution. If you are interested in contributing further and extending rich to the rest of the codebase, do let me know.

Thanks @aliabbasjaffri
I am interested on working further on integrating rich in mlinfra

hey @ara-25 , i managed to create a new issue for rich integration across entire codebase. Please let me know if you need any more context of help regarding this. You can reach out to me via discord channel, i'm rather fast to reply back there than on the github issues.
Also, i would like to request a github ⭐ from you as that would really help the tool get more traction and users. Thank you!

Thanks a lot @aliabbasjaffri.
Also, I have already starred your amazing work


Using SwipeRow in ListView

I want to use a SwipeRow in a ListView rather than use a SwipeListView because i want different behaviour on different rows. Some for example should not be swipeable.
The behaviour I'm seeing is that the right-swipe works fine, but the left-swipe doesn't.  After releasing the left-swipe, the view immediately switches as if I'd done a right-swipe.

hi, i was wondering how i can make "rightOpenValue" value dynamic? i tried adding "ref=" to a "View" element, and then doing something like: https://github.com/facebook/react-native/issues/953
but that didn't work seems the "ref" is not available inside "renderHiddenRow". any ideas or examples would be appreciated.

Hey @npomfret. So I tried to replicate your issue but I can't. I think what may be happening is as you're swiping left you're also swiping up causing the ListView to scroll which causes the row to close. This is handled if you're using the SwipeListView since I have access to all scrolling behavior and can determine what is a scroll vs a swipe.
For this reason I wouldn't recommend using standalone SwipeRow's inside of Scrollable Views.
To achieve what you're talking about the SwipeListView needs to handle SwipeRow's with different behavior. This would also address #31. I have some ideas but haven't had time, I'll try to get to this asap. PR's are always welcome 😃 .
I tried to show what I mean below. Let me know if this makes sense or I'm missing the issue.
Here's the code and screengrab:
<View style={styles.container}>
	<ListView
		style={{flex: 1}}
		dataSource={this.ds.cloneWithRows(this.state.listViewData)}
		renderRow={ data => (
			<SwipeRow
				leftOpenValue={75}
				rightOpenValue={-75}
			>
				<View style={styles.standaloneRowBack}>
					<Text style={styles.backTextWhite}>Left</Text>
					<Text style={styles.backTextWhite}>Right</Text>
				</View>
				<View style={styles.standaloneRowFront}>
					<Text>I'm SwipeRow {data}</Text>
				</View>
			</SwipeRow>
		)}
	/>
</View>

Thanks for the reply.  If i get some time I'll take a look at implementing per row behaviours

@npomfret Npm install the latest version and check out the docs here for dynamic values on SwipeRows.
The example now shows how to disable swiping on only some rows.
Let me know if you have any further problems or questions!

Perfect, thanks!


fixed MathRNG - it now uses the initial seed properly

Before this fix, it was generating the same Uint8List over and over if the initial seed was specified.

Oh, fantastic catch and thank you for the fix. I will merge this and add some tests to make sure it doesn't regress in the future.


Name logo of FreeHit Website

Related Issue
Designed the name logo of the website.
Closes #issue_number
#804
Description
Added the logo.
Screenshots

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

@Tisha6661, let's first discuss the details, and then we can proceed with the implementation.

@JasonDsouza212 Sure, Is there any issue?

@JasonDsouza212 Are the colors alright, I only need to work on design right?

Hello, contributors! This is Jason speaking. Due to some workflow issues, we will be closing all the PRs today. Therefore, I kindly request all of you to create new PRs tomorrow, as the currently open ones will be closed by our team. I apologize for any inconvenience caused. Also, please make sure to check the titles when creating the PRs. You can refer to the README.md for more details. Thank you.


Proxy extra request options

I have a server which has only ipv6 access to api.telegram.org. So, I must pass family: 6 option for each request to telegram api.
It would be great to add an extra option to TelegramBot constructor which makes it possible.
@yagop what do you think?

This PR is awesome! I'm ready to merge this! Please rebase your branch!
However, the polling requests are not using these custom options.

Please don't use extend, useObject.assign.


Autocomplete doesn't work for newly created targets

Create a new cargo project
Check if autocompleting any std:: items works (it should).
Add a new bin or lib target.
In the source code of the new target, try autocompleting any std:: items. It only offers self and super.
Reload the ra-lsp extension. Now completion will also work in the new target.

rust-analyzer: 5202b0ec
Code: 1.40.2 (f359dd69833dd8800b54d458f6d37ab7c78df520)
Electron: 6.1.5
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Linux x64 5.4.2-arch1-1

This depends on #1968

And this one is fixed!


Update on changelog.md

Fixed

Turkish Translations #72
added.

@kaanhangithub the changelog changes need to be on the same PR as the changes it point!
Could you add this codes changes into https://github.com/CanHub/Android-Image-Cropper/pull/72 and close this PR?
Thanks!


Whether the golden programs available?

Thanks for the excellent work! I'd like to know if the golden program for each question is available. Since the questions in TheoremQA are a little difficult for people without expert knowledge. Thanks a lot!

Already noticed the PoT rationales of TheoremQA are annotated by GPT4 in your new work. Issue closed.

You can find these rationale in my solution/ folder, or under the outputs/ folder.


Inconsistency in "Vector Widening Integer Multiply-Add Instructions" examples

In chapter 12.14 Vector Widening Integer Multiply-Add Instructions, there are the examples code as follows:
# Widening signed-unsigned-integer multiply-sub, overwrite addend
vwmaccsu.vv vd, vs1, vs2, vm    # vd[i] = +(signed(vs1[i]) * unsigned(vs2[i])) + vd[i]
vwmaccsu.vx vd, rs1, vs2, vm    # vd[i] = +(signed(x[rs1]) * unsigned(vs2[i])) + vd[i]

# Widening unsigned-signed-integer multiply-sub, overwrite addend
vwmaccus.vx vd, rs1, vs2, vm    # vd[i] = +(unsigned(x[rs1]) * signed(vs2[i])) + vd[i]

The code comment says "multiply-sub". Is it correct? It does not seem to match the operation equation and the chapter description.

Thanks for finding this editing error; it should be multiply-add.


Add local Compiler Explorer feature

I'd like to run something like forc godbolt and have it launch a local instance of Compiler Explorer so I can inspect how the Sway source code maps to asm with different compiler options. Doesn't have to be Compiler Explorer, or even in the browser if IDEs can do it.
Maybe blocked by source mapping #70.

In a dream world, we would have a nightly build of the Sawy compiler accessible through a public godbolt link just like https://rust.godbolt.org/ - Imagine sway.godbolt.org where developers can easily look at the generated assembly
Do we know if this realistic? Some information on that here: https://github.com/compiler-explorer/compiler-explorer/blob/main/docs/AddingALanguage.md

We don't really need a nightly build, do we? We do tag releases, so simply having each release of forc should suffice methinks.

We don't really need a nightly build, do we? We do tag releases, so simply having each release of forc should suffice methinks.

True - as we're tagging releases often, nightlies are probably an overkill.


GEOMESA-567 Refactor AccumuloCoverageStore to use RasterStore apply method

Cleaned-up code in and around AccumuloCoverageStore
Exposed shards/writememory/writethreads/querythreads params in RasterStore apply

Committed as f45d986a905f6ee9f2f3a88431dee25474f00d8b on f_wcs_4.


Cant be written  to Excel on MAC(m2 clips)

hi,all
I cant be wrriten any type to Excel ,my os is Mac os 13.5.2 (22G91) (Apple M2 Max)

Thanks for your issue. Which version of the Go language and Excelize library are you using? Could you follow the issue template and provides more details about this?

golang 1.21.0
excelize v2.8.0 / v2.6.1

If you are reporting a new issue, make sure that we do not have any duplicates already exist. If it does not work with Go 1.21.0, please reference the issues #1465, #1595, #1603, #1608, #1614, #1619, #1620, #1621, #1623, #1633, #1637, #1641, #1642, #1648, #1651, #1652 and #1656. I have added notice on the README and documentation website for this.