Uncaught RangeError: Offset is outside the bounds of the DataView

While zooming in/out with mouse I get Uncaught RangeError: Offset is outside the bounds of the DataView on the following line of code:
https://github.com/melowntech/vts-browser-js/blob/1f3c4235c5ae4aefbbb64188e93867e89da834c9/src/core/map/metatile.js#L238
after this exception the map freezes.
here is the stack trace:
vts-browser.min.js:formatted:20137 Uncaught RangeError: Offset is outside the bounds of the DataView
at DataView.getUint8 ()
at n.parseMetatatile (vts-browser.min.js:formatted:20137)
at n.onLoaded (vts-browser.min.js:formatted:20120)
at j.processProcessingTasks (vts-browser.min.js:formatted:18504)
at j.update (vts-browser.min.js:formatted:18542)
at A.onUpdate (vts-browser.min.js:formatted:2423)
n.parseMetatatile @ vts-browser.min.js:formatted:20137
n.onLoaded @ vts-browser.min.js:formatted:20120
j.processProcessingTasks @ vts-browser.min.js:formatted:18504
j.update @ vts-browser.min.js:formatted:18542
A.onUpdate @ vts-browser.min.js:formatted:2423
requestAnimationFrame (async)
A.onUpdate @ vts-browser.min.js:formatted:2425

found the cause of issue: sometimes stream.data is empty and accessing first index raises the exception.

You are right. It seems to be caused by corrupted data. I can add more checks for data corruption. But unfortunately data can be corrupted in endless ways. Do you have URL of corrupted metatile?

Thank you for your response.
I cant find the URL following the call stack in the debugger.

You may try debug (unminified) version of the library

https://github.com/melowntech/vts-browser-js/releases/download/v2.22.5/compiled-libraries-and-demos.zip
Library file is stored in build directory.
URL value should in stored in MapMetatile object as .mapLoaderUrl or at least send me .id of the metatile.

As call stack is started by map.update event, URL is not present in current call stack, (only data related to the URL is passed through this call sequence)

In that case I have no way how to replicate this error. Can you send mapConfig URL and map position (or at least lat,lon) where error occur?


It would be nice if morph target names are exported as extras.

Blendshapes are exported as "targets" attributes of a primitive.
But they only have vertex data.
If blendshape names can be exported as well (maybe via "extras"?), it would be very helpful.
Thanks

They should already be exported as mesh.extras.targetNames.


Streamline all FTT modules

There's no reason why the FTT modules should use slightly different versions of similar mathematical equations. In most cases we can write one function for e.g. the market share dynamics which is called in each of the FTT modules.
List of functions that can potentially be streamlined:

Market share dynamics
Levelised cost calculation
Investment calculation
Leaning-by-doing calculation

For the regulation, Rosie says that the most convenient unit from a modelling standpoint (for corrections) is the same as demand drivers. So this would be GWh in FTT:P.


Needs Updated

Recent S&Box updates broke the tool... Unless I'm crazy.
https://i.imgur.com/s3gYatd.png
Can't get much of anything working.

I can't repro this, what are you doing to cause this?

Base install on the latest version of s&box
Here's a video:
https://streamable.com/i49xvn
Let me know if I'm missing something here.

Pushed a fix, download the repo zip and let me know if that fixes it for you

Decided to make an account for this since I am also interested in the tool
The issue seems to still persist. Same error as @XazeSyn provided

I did a bunch of testing and I think I got everything, this should be fixed now. Sorry for the delay


GH-35060: [C#][CI] Update dotnet download link regex

The download page changed and the existing regex was not correctly catching the link. This seems overall pretty brittle but it seems there is no better way to download the file directly.

Closes: #35060

@github-actions crossbow submit verify-rc-source-csharp-*


Does not work in Edge at all

Wavesurfer.js version(s):
v3.1.0
Browser and operating system version(s):
Edge 18, Windows 10
Code needed to reproduce the issue:
Go to https://wavesurfer-js.org/ in Edge
Use behaviour needed to reproduce the issue:
Check the player in the header and try to play it. Neither waveform or sound works. I do not get any errors in console.

that's weird because Edge has WebAudio support right: https://blogs.windows.com/msedgedev/2015/05/19/bringing-web-audio-to-microsoft-edge-for-interoperable-gaming-and-enthusiast-media/

@thijstriemstra … can you confirm the bug? I tested on two different Windows computers (same spec tho).

not working in Edge confirmed

Confirmed.Found following Error in Edge;
Microsoft Edge 44.18362.387.0
Microsoft EdgeHTML 18.18362
Tree on pic not full expanded !

supplement to the previous entry:
used:

Error Pic is from my site in which i used code parts from the following playlist example :
https://wavesurfer-js.org/example/playlist/index.html
I just tried the original playlist example in Edge. This one doesn't work either.

@Bopser I agree that it does not work
It seems to come from this line
return new Response(new ReadableStream(new ProgressHandler(instance, contentLength, response)), fetchOptions);
``` in `wavesurfer.js`

But I haven't found a solution yet. Have you ?

I made a (very) temporary patch because it's christmas :-) and I need to make it work NOW
So
Here is the commit on my fork https://github.com/cme-pro/wavesurfer.js/commit/601e9b584e94d172f0fd7a4821c366bf196c8b91
And you can use by
rm -Rf node_modules/wavesurfer.js &&  yarn add cme-pro/wavesurfer.js#fix/edge

I'm still interested to launch a PR if someone knows what is wrong with ReadableStream on Edge being here but not functional
Merry christmas to all of you

thanks for the feedback @martinratinaud. Maybe take a look at https://stackoverflow.com/questions/55294816/how-do-i-create-a-readablestream-in-microsoft-edge and https://github.com/mapbox/mapbox-gl-js/commit/c49dd3151c9e9e528e6eec339f54e033cd190e29

@martinratinaud can you make a pull request for your workaround?

@thijstriemstra sure I can but maybe you should validate the commit first https://github.com/cme-pro/wavesurfer.js/commit/601e9b584e94d172f0fd7a4821c366bf196c8b91 as

I wrote no tests
it downgrades the use of the progress indicator

Hey! @thijstriemstra is it possibly going to be merged to master soon? We're having the same issue and currently we're using @martinratinaud fix as a workaround.
Just wanted to let you know it's working properly.
Thanks for your work!

is it possibly going to be merged to master soon?

there is no pull request to merge. can you make one @Kucziii?

Any news for a fix with Edge browser ?

I was stucked with same problem, but I think this is done well after Edge Browser update.

Working with latest version in both edge and edge chrome. reopen if needed.

Working with latest version in both edge and edge chrome. reopen if needed.


Add rational Bezier curve functionality to BezierCurve

Summary

This PR is a feature
It enhances the BezierCurve class to allow for rational Bezier curves, which adds a weight to each control node.
It adds tests for rational Bezier curves, and ensures compatibility with winding number algorithms.

Looks very good.  I requested some tests and a rename of isValid.  Also, I think I caught a reversed test.  After these are addressed, I'll be happy to approve.


Update messages.json

补充汉化内容

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@xuefeng-bai 您好, 您可以把这个更新到原PR. 谢谢.

Thanks for the PR, but we should be editing the .po files, not the .json files, since the .json ones are generated automatically by the translation scripts.

I'll close this for now, but please feel free to update the PR with a rebase, and reopen if/when ready.


Index page updates

PR to shorten the user guide descriptions and to give each top level page an appropriate introductory paragraph.

@philippjfr I would like to add an introduction to the gallery and reference pages. Where can I do that?

I think this PR is mostly done. Note that https://github.com/ioam/ioam-builder/pull/13 needs merging too..

Looks good, merging.


Fix duplicate db references being allowed

This PR fixes #12794. Fixes two things:

Wasn't checking if project reference already exists before adding
On mac, duplicate references weren't being caught if a project already with a db reference was loaded because the uri read from the sqlproj had Window's slashes. Fix is to make sure slashes are all consistent when doing the comparison

Pull Request Test Coverage Report for Build 294197929

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 39.775%

Totals

Change from base Build 294184891:
0.01%

Covered Lines:
20047

Relevant Lines:
46151

💛  - Coveralls

opened #12809 to track adding more tests for this. There's already a test for duplicate refs, but it doesn't test after reloading a project and is only testing a dacpac without slashes in the filepath ('test.dacpac')


Broken link in Configure read-only routing for an Always On availability group

The article Configure read-only routing for an Always On availability group
contains a link to Calculating read_only_routing_url for Always On that is dead.
The article can still be found on the wayback machine at https://web.archive.org/web/20170512023255/https://blogs.msdn.microsoft.com/mattn/2012/04/25/calculating-read_only_routing_url-for-alwayson/ and the script inside is interesting (Someone posted an cleaned up version without the bad MSDN blog formatting here: https://gist.github.com/Echizendesu/1483916a232e9607650e627b00c13d7d/2827b23384b74301d412db34ca3fee66a8cc5f34 )

@vbfox fixing - thank you.

Fix will be live later today.


why can not Import the nodes:

(IMPORT FAILED): D:\comfy\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-Inspyrenet-Rembg

Same here!

same here

same here


fix: return blank string if we can't create htpasswd

to avoid stopping first time - the next secret generation loop we will populate it
Signed-off-by: James Strachan<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign jstrachan
You can assign the PR to them by writing /assign @jstrachan in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

/approve

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

Needs approval from an approver in each of these files:

OWNERS [jstrachan]

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


Add MVStore Encryption to NitriteStore through NitriteBuilder.

One of H2's great features is Encrypted files. This is provided in MVStore but isn't exposed by the NitriteBuilder.

Well I left it for a reason. MVStore does not encrypt MVMap entities, it encrypts the whole database file. So for every read/write operation, nitrite has to decrypt the whole database file and store it in memory everytime, which is a huge performance hit when the file size gets big and which I can't afford.
If you want to store data encrypted, you can implement Mappable in your pojo and encrypt/decrypt data in the Document level instead of encrypting the whole database file. It will  be way more convenient and performant.

Ok, that makes sense. Thanks.
Sincerely,
Brian Taylor
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On April 27, 2018 1:30 AM, Anindya Chatterjee<EMAIL_ADDRESS>wrote:

Well I left it for a reason. MVStore does not encrypt MVMap entities, it encrypts the whole database file. So for every read/write operation, nitrite has to decrypt the whole database file and store it in memory everytime, which is a huge performance hit when the file size gets big and which I can't afford.
If you want to store data encrypted, you can implement Mappable in your pojo and encrypt/decrypt data in the Document level instead of encrypting the whole database file. It will be way more convenient and performant.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Add row info to rowHeight function

Enable modifying row height based on row info.

@fritz-c hey, can you please check this out? This is a really small change that would help me out a lot.

Available now in v1.3.0.
Sorry for the delay. You were right - it was a small but meaningful change.
By the way, I tweaked your fix a little bit so the row info is included at the top level of the argument object, rather than nested inside the row property. See https://github.com/fritz-c/react-sortable-tree/commit/a906f21da438d0f8d28df869c4ffa2e38293b28c to understand what I mean.

hi,the changeNodeAtPath method is not called rowHeight function


Changed Friend Added dialog to a toast

Fixes #2507

This change is

:thumbsup:

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 1 of 1 files at r1.
Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable


How to change shape & color with programmatically?

Unfortunately, I couldn't find how to do this in the documentation.

Hello! Could you elaborate on what element’s color and shape you’d like to change?


Fix typo in lookback-delta parameter

While migrating from Prometheus v1 to v2, the staleness-delta parameter was renamed to lookback-delta.
This PR fixes a typo in the new name.

Thanks @aveyrenc!


metrics: Policy Attachment metrics (investigate/proof of concept)

How can/Does it make sense to support state metrics for Policies that have been attached via https://gateway-api.sigs.k8s.io/references/policy-attachment/?
What's the impact on:

the CustomResourceStateMetrics config
RBAC permissions for different CRDs

Would an example or 'how-to' make more sense if it's not reasonable to solve this in a generic way?

@guicassolato @alexsnaps Would you have an example AuthPolicy and RateLimitPolicy for reference here (v2 I guess)?
Even better if you had one with the status block showing some useful information.
Thanks, I'd really appreciate it.

@guicassolato @alexsnaps Would you have an example AuthPolicy and RateLimitPolicy for reference here (v2 I guess)?
Even better if you had one with the status block showing some useful information.

These are the most up-to-date examples of AuthPolicy and RateLimitPolicy v2 I can think of now, @david-martin:

https://github.com/Kuadrant/kuadrant-operator/blob/95f6c6344ac1fc7d2fb673139301c0560dd8cbe8/doc/user-guides/authenticated-rl-with-jwt-and-k8s-authnz.md
https://github.com/Kuadrant/kuadrant-operator/blob/95f6c6344ac1fc7d2fb673139301c0560dd8cbe8/doc/user-guides/authenticated-rl-for-app-developers.md

TBH, I'm not sure how helpful the status stanza of these CRs can be as of today. But we do have some work in the roadmap to get them enhanced soon – certainly along the lines of what's proposed in https://github.com/Kuadrant/architecture/pull/9, but also open to suggestions.

Meanwhile, and at least for Authorino only, maybe there's sth here that we could use:

https://docs.kuadrant.io/authorino/docs/user-guides/observability/
https://github.com/Kuadrant/authorino/blob/bffb49e40b68e13edc8e489a4cb21c10ddd20830/api/v1beta1/auth_config_types.go#L707-L772, along with https://github.com/Kuadrant/authorino/blob/bffb49e40b68e13edc8e489a4cb21c10ddd20830/api/v1beta1/auth_config_types.go#L47-L58

While the AuthConfigs and the AuthPolicies relate 1:1 one another, the status of the AuthConfig CRs, as well as their so-called associated "deep metrics", will certainly influence the status and the metrics of the AuthPolicies. Perhaps not so much regarding the Policy Attachment aspect of the AuthPolicies directly, but other observable aspects for sure. And, by transition, even the Policy Attachment part may end up affected, I reckon.
E.g.: "HTTPRouteRule X is generating too much latency due to an associated config in the auth policy that make requests to this route to fetch metadata from an external source, without setting proper caching." (If that makes any sense.)

I don't see metrics related to Kuadrant's AuthPolicy introduced in #36. Do we want to reopen this issue or capture what we missed in a new one?

Good catch @guicassolato
I'll reopen this. It closed automatically.
@Ygnas We can model the AuthPolicy metrics and dashboard on the RLP & TLSPolicy ones.


code requirements

Hi,
There are these two packages in the requirements:
pydicom
dicom-numpy
Since you have used python 3 and anaconda, I am wondering how you found them in anaconda. When I look for them, I find no available packages.
Thanks,
Amirtahà

I figured it out. Thanks!!


Add message field to lastRetry for conditional retries

Summary
In https://argoproj.github.io/argo-workflows/retries/#conditional-retries, lastRetry cannot access workflow error/failure messages such as Pod was terminated due to imminent node shutdown. This request is for adding a message on lastRetry containing this error message.
Use Cases
In https://github.com/argoproj/argo-workflows/issues/9880, we captured preemptions by setting OnTransientError and TRANSIENT_ERROR_PATTERN. However, we would also like to retry OnError. Using conditional retries, we cannot do both since lastRetry does not include a message field - e.g.
retryStrategy:
        limit: "3"
        expression: "lastRetry.status == 'Error' || lastRetry.message.contains('imminent node shutdown')"

Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

HI! I would like to work on this issue.

Please go ahead and submit a PR if you'd like to contribute.


issue about convert Pkcs8 keys to byte arrays

https://github.com/fudiwei/DotNetCore.SKIT.FlurlHttpClient.Wechat/blob/665efdfc9719029cf6f14cacab2bb35f457bebea/src/SKIT.FlurlHttpClient.Wechat.TenpayV3/Utilities/RSAUtility.cs#L230
A mistake? Do you intend to use Regex.Replace(publicKey, @"\s+", "") ？
https://github.com/fudiwei/DotNetCore.SKIT.FlurlHttpClient.Wechat/blob/665efdfc9719029cf6f14cacab2bb35f457bebea/src/SKIT.FlurlHttpClient.Wechat.TenpayV3/Utilities/RSAUtility.cs#L219
And here is the same issue

My mistake 😭
Thanks for your correction.
Anyway, there will be no problem now if you directly use the certificate files provided by Tenpay. So I will not release a new version to NuGet, only fix the code currently.
BTW, I'm from Dalian as also. Bro 😁

Fix this on release v1.4.0


Apouey index - iteration 1

(This iteration assumes the index initialization with a look-up table with $\alpha$ parameter. In the next iteration, the function estimating $\alpha$ will be defined, and the output's confidence interval will be calculated).
Definition
The index definition:
$$P_{2}(F) = 1 - \frac{2^{\alpha}}{C-1} \sum_{c=1}^{C-1}|F_{c}-\frac{1}{2}|^{\alpha}$$
where:

$P_{2}(F)$ - index value based on the cumulative proportions of c-th category,
$C$ - number of categories (we force users to use [3, 10] categories; see look-up table below),
$F(c)$ = $\sum_{i=1}^{c} p_{i}$ - sum of proportions $p$ up to the c-th category,
$p=(p_{1}, ..., p_{c-1}, p_{c})$ = $p_{c}$ - proportion for c-th category.

Look-up table for the $\alpha$ parameter

Number of categories
$\alpha$

3
$=\frac{ln2}{ln3}$

4
$=\frac{ln4 - ln3}{ln2}$

5
$\approx 0.73$

6
$\approx 0.66$

7
$\approx 0.78$

8
$\approx 0.75$

9
$\approx 0.82$

10
$\approx 0.81$

Example
We have a survey with ordered responses: {very poor, poor, medium, well, very well}. It is a 5-level grading. We get the following data:

Category
Category Level
Number of responses
Proportion

very poor
1
10
0.2

poor
2
10
0.2

medium
3
10
0.2

well
4
5
0.1

very well
5
15
0.3

We can calculate $F_{c}$ from the responses, up to the category $C-1$:

$F_{1} = 0.2$
$F_{2} = 0.4$
$F_{3} = 0.6$
$F_{4} = 0.7$

$\alpha$ is equal to 0.73 because we have 5 categories. Thus:
$$P_{2}(F) \approx 0.545$$
Implementation aspects

Please, consider using the info() function to get the proportions of each category.
For now, the look-up table should be hard coded into the script. An additional check of the number of categories is required, and if there are less than 3 or more than 10 categories the function should return ValueError.

Bibliography
Apouey B. Measuring health polarization with self-assessed health data. Health Econ. 2007 Sep;16(9):875-94. doi: 10.1002/hec.1284. PMID: 17705333 - PAGE 882 has the equation used in the package.

WIP https://github.com/hs3city/ordinal-scale-stats-py/pull/34


Add recommended build instructions to the README

Is it possible to add build instructions to the README? I'm not entirely familiar with Go but I'd like to use the agent in master so I can utilize the verbose flag that's been added. How are you guys currently building the agent for multiple platforms? I see the Makefile but make is failing when it tries to import in agent.go.

Hi Nick—
We're in the process of pushing out an update in the next day or so—when we do that, I will make sure we add a full set of installation instructions to the docs.

@mtabini sounds good, thanks!

Hi Nick—
The new release is out. You will find many changes and improvements, which are fully documented on the Telemetry documentation site.
If you still want to build from source, it should be as easy as explained in the docs. I'm going to close this issue for now—feel free to reopen if you're still having problems.
Thanks!

@mtabini Thanks for the update, really liking the changes in this release too.


tower-h2 unnecessarily depends on tokio-core

This appears to have been resolved by abb40873322af7c014d2cb3a4caf03f5f88175e9

Yeah, I believe this is fixed.


content: add public key encryption blog

This pull request creates a new blog "Public Key Encryption Explained"

@lukegalbraithrussell please rebase to ockam-network:develop before merge


The model makes up columns if the column is not present

Here is the schema:
CREATE TABLE person (
id INTEGER PRIMARY KEY NOT NULL AUTO_INCREMENT, -- A unique id for each person
name VARCHAR(255) DEFAULT NULL, -- The full name of the person
alias VARCHAR(255) DEFAULT NULL, -- Any known aliases or alternative names used by the person
age INTEGER DEFAULT NULL, -- The age of the person
);
Query: Person with age > 20 from haryana
Output: SELECT p.name FROM person p WHERE p.age > 20 AND p.state = 'haryana';
The model made up a column state even if it is not present in the schema.
Using the 7B model.
Tried to update the instructions still the same issue.

In issue #12, the authors specify that the schema used must be well-defined. Also, the model was tested only on definite queries (They have preprocessed data that have ambiguities) . Thus in these cases, like a usual LLM, it tends to hallucinate.
What was the instruction used. Maybe try including the instruction not to hallucinate right in the beginning.


TypeError: 'module' object is not callable

Steps to reproduce:
Install the package from pypi:
pip3 install earcut
Import and run example from the README.md file:
>>> import earcut
>>> triangles = earcut([10, 0, 0, 50, 60, 60, 70, 10])
Traceback (most recent call last):
    File "<input>", line 1, in <module>
        earcut([10, 0, 0, 50, 60, 70, 10])
TypeError 'module' object is not callable

Expected output:
[1, 0, 3, 3, 2, 1]

pip show earcut
Name: earcut
Version: 1.1.4
Summary: A pure Python port of the earcut JS triangulation library
Home-page: https://github.com/MetacitySuite/earcut-python
Author: Metacity
Author-email:
License: ISC
Location: /home/rtfm/.local/lib/python3.10/site-packages
Requires: numpy
Required-by:

The referenced Location discussed in #1 and introduced by @vojtatom is not available any more.

Damn. The solution was:
from earcut.earcut import earcut

Hey, the import structure is a bit unfortunate but I'm glad you got it. I could update it to make it simpler, but I'm deciding whether it's worth it - see https://github.com/joshuaskelly/earcut-python/issues/1


Remove more skipped ansible-lint rules

This patch removes the following skipped ansible-lint rules:

schema[meta]
schema[vars]

schema[meta] is fixed by removing the galaxy_info.platforms section. This was not maintained and has now been removed from Ansible Galaxy and soon to be removed from ansible-lint.[1]
schema[vars] was fixed by changing roles/openshift_provisioner_node/molecule/defaults to roles/openshift_provisioner_node/molecule/default, this was confusing ansible-lint
This patch also adds the following new opt-in rules as these are things we would like to avoid:

empty-string-compare
risky-shell-pipe

[1] https://forum.ansible.com/t/role-meta-data-platform-key-broken/4651/10

/lgtm


logging error occurred with the given example code

import logging
log = logging.getLogger('sknn')
log.basicConfig(
format="%(message)s",
level=logging.DEBUG,
stream=sys.stdout)
raises this error
AttributeError                            Traceback (most recent call last)
 in ()
2
3 log = logging.getLogger('sknn')
----> 4 log.basicConfig(
5         format="%(message)s",
6         level=logging.DEBUG,
AttributeError: 'Logger' object has no attribute 'basicConfig'

Thanks for reporting it! What version of Python?

Ah, it's the documentation.
Just do:
import logging
logging.basicConfig(*params)

It'll be fixed shortly.

Closed by [e82cba8].

still I cannot get any output from the code even I trigger verbose = 1

@erogol If you set verbose=1 in bench_mnist.py, do you see the output?


Validate route uri

Writeup and example: https://github.com/RyanBard/scg-npe
spring-cloud-gateway version: 2.0.0.M5
spring-boot-version: 2.0.0.M7
I'm getting a NullPointerException in NettyRoutingFilter on line 67 because scheme is null.  It's possible that my configuration in my application.yml is bogus (the uri and predicate Path do not match -- EDIT: I've verified that it still fails for the same reason even if they do overlap), however, it looks like the overwriting of scheme in the UriComponentsBuilder calls in RouteToRequestUrlFilter could be problematic in other situations.
I would appreciate if you could take a look.
Thank you

Can you try boo 2.0.0.M7?, M5 is meant to work with M7. If it still is a problem, please post the full stack trace.

It's still failing in the same places (the stack trace might be a little different in the middle).
I've updated the pom to 2.0.0.M7, fixed the mismatch on uri & path (and verified it still fails for the same reason), and updated the curl/stack/debug-info in the README.
I'll also update the spring-boot version in this ticket.

Here's the stacktrace if you don't want to see the notes in my README:
[INFO] --- spring-boot-maven-plugin:2.0.0.M7:run (default-cli) @ scg-npe ---

  .   ____          _            __ _ _
 /\\ / ___'_ __ _ _(_)_ __  __ _ \ \ \ \
( ( )\___ | '_ | '_| | '_ \/ _` | \ \ \ \
 \\/  ___)| |_)| | | | | || (_| |  ) ) ) )
  '  |____| .__|_| |_|_| |_\__, | / / / /
 =========|_|==============|___/=/_/_/_/
 :: Spring Boot ::             (v2.0.0.M7)

2017-12-12 18:28:24.940  INFO 39607 --- [           main] scg.example.ScgApplication               : Starting ScgApplication on jbard-M01.vmware.com with PID 39607 (/Users/jbard/projects/scg-npe/target/classes started by jbard in /Users/jbard/projects/scg-npe)
2017-12-12 18:28:24.945  INFO 39607 --- [           main] scg.example.ScgApplication               : No active profile set, falling back to default profiles: default
2017-12-12 18:28:24.988  INFO 39607 --- [           main] onfigReactiveWebServerApplicationContext : Refreshing org.springframework.boot.web.reactive.context.AnnotationConfigReactiveWebServerApplicationContext@53700d3c: startup date [Tue Dec 12 18:28:24 EST 2017]; root of context hierarchy
2017-12-12 18:28:25.798  INFO 39607 --- [           main] o.h.v.i.engine.ValidatorFactoryImpl      : HV000238: Temporal validation tolerance set to 0.
2017-12-12 18:28:26.608  INFO 39607 --- [           main] o.s.c.g.r.RouteDefinitionRouteLocator    : Loaded RoutePredicateFactory [After]
2017-12-12 18:28:26.608  INFO 39607 --- [           main] o.s.c.g.r.RouteDefinitionRouteLocator    : Loaded RoutePredicateFactory [Before]
2017-12-12 18:28:26.608  INFO 39607 --- [           main] o.s.c.g.r.RouteDefinitionRouteLocator    : Loaded RoutePredicateFactory [Between]
2017-12-12 18:28:26.608  INFO 39607 --- [           main] o.s.c.g.r.RouteDefinitionRouteLocator    : Loaded RoutePredicateFactory [Cookie]
2017-12-12 18:28:26.608  INFO 39607 --- [           main] o.s.c.g.r.RouteDefinitionRouteLocator    : Loaded RoutePredicateFactory [Header]
2017-12-12 18:28:26.608  INFO 39607 --- [           main] o.s.c.g.r.RouteDefinitionRouteLocator    : Loaded RoutePredicateFactory [Host]
2017-12-12 18:28:26.609  INFO 39607 --- [           main] o.s.c.g.r.RouteDefinitionRouteLocator    : Loaded RoutePredicateFactory [Method]
2017-12-12 18:28:26.609  INFO 39607 --- [           main] o.s.c.g.r.RouteDefinitionRouteLocator    : Loaded RoutePredicateFactory [Path]
2017-12-12 18:28:26.609  INFO 39607 --- [           main] o.s.c.g.r.RouteDefinitionRouteLocator    : Loaded RoutePredicateFactory [Query]
2017-12-12 18:28:26.609  INFO 39607 --- [           main] o.s.c.g.r.RouteDefinitionRouteLocator    : Loaded RoutePredicateFactory [RemoteAddr]
2017-12-12 18:28:26.644  INFO 39607 --- [           main] o.h.v.i.engine.ValidatorFactoryImpl      : HV000238: Temporal validation tolerance set to 0.
2017-12-12 18:28:26.747  INFO 39607 --- [           main] o.s.w.r.r.m.a.ControllerMethodResolver   : Looking for @ControllerAdvice: org.springframework.boot.web.reactive.context.AnnotationConfigReactiveWebServerApplicationContext@53700d3c: startup date [Tue Dec 12 18:28:24 EST 2017]; root of context hierarchy
2017-12-12 18:28:27.136  INFO 39607 --- [           main] o.s.j.e.a.AnnotationMBeanExporter        : Registering beans for JMX exposure on startup
2017-12-12 18:28:27.227  INFO 39607 --- [ctor-http-nio-1] r.ipc.netty.tcp.BlockingNettyContext     : Started HttpServer on /0:0:0:0:0:0:0:0:8080
2017-12-12 18:28:27.227  INFO 39607 --- [           main] o.s.b.web.embedded.netty.NettyWebServer  : Netty started on port(s): 8080
2017-12-12 18:28:27.231  INFO 39607 --- [           main] scg.example.ScgApplication               : Started ScgApplication in 2.731 seconds (JVM running for 6.43)
2017-12-12 18:28:48.785 ERROR 39607 --- [ctor-http-nio-2] .a.w.r.e.DefaultErrorWebExceptionHandler : Failed to handle request [GET http://localhost:8080/abc/def/xyz]

java.lang.NullPointerException: null
	at org.springframework.cloud.gateway.filter.NettyRoutingFilter.filter(NettyRoutingFilter.java:67) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.cloud.gateway.handler.FilteringWebHandler$GatewayFilterAdapter.filter(FilteringWebHandler.java:121) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.cloud.gateway.filter.OrderedGatewayFilter.filter(OrderedGatewayFilter.java:40) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.cloud.gateway.handler.FilteringWebHandler$DefaultGatewayFilterChain.filter(FilteringWebHandler.java:103) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.cloud.gateway.filter.RouteToRequestUrlFilter.filter(RouteToRequestUrlFilter.java:59) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.cloud.gateway.handler.FilteringWebHandler$GatewayFilterAdapter.filter(FilteringWebHandler.java:121) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.cloud.gateway.filter.OrderedGatewayFilter.filter(OrderedGatewayFilter.java:40) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.cloud.gateway.handler.FilteringWebHandler$DefaultGatewayFilterChain.filter(FilteringWebHandler.java:103) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.cloud.gateway.filter.factory.AddResponseHeaderGatewayFilterFactory.lambda$apply$0(AddResponseHeaderGatewayFilterFactory.java:47) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.cloud.gateway.filter.OrderedGatewayFilter.filter(OrderedGatewayFilter.java:40) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.cloud.gateway.handler.FilteringWebHandler$DefaultGatewayFilterChain.filter(FilteringWebHandler.java:103) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.cloud.gateway.filter.factory.AddResponseHeaderGatewayFilterFactory.lambda$apply$0(AddResponseHeaderGatewayFilterFactory.java:47) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.cloud.gateway.filter.OrderedGatewayFilter.filter(OrderedGatewayFilter.java:40) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.cloud.gateway.handler.FilteringWebHandler$DefaultGatewayFilterChain.filter(FilteringWebHandler.java:103) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.cloud.gateway.filter.NettyWriteResponseFilter.filter(NettyWriteResponseFilter.java:52) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.cloud.gateway.handler.FilteringWebHandler$GatewayFilterAdapter.filter(FilteringWebHandler.java:121) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.cloud.gateway.filter.OrderedGatewayFilter.filter(OrderedGatewayFilter.java:40) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.cloud.gateway.handler.FilteringWebHandler$DefaultGatewayFilterChain.filter(FilteringWebHandler.java:103) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.cloud.gateway.handler.FilteringWebHandler.handle(FilteringWebHandler.java:87) ~[spring-cloud-gateway-core-2.0.0.M5.jar:2.0.0.M5]
	at org.springframework.web.reactive.result.SimpleHandlerAdapter.handle(SimpleHandlerAdapter.java:62) ~[spring-webflux-5.0.2.RELEASE.jar:5.0.2.RELEASE]
	at org.springframework.web.reactive.DispatcherHandler.invokeHandler(DispatcherHandler.java:168) ~[spring-webflux-5.0.2.RELEASE.jar:5.0.2.RELEASE]
	at org.springframework.web.reactive.DispatcherHandler.lambda$handle$1(DispatcherHandler.java:160) ~[spring-webflux-5.0.2.RELEASE.jar:5.0.2.RELEASE]
	at reactor.core.publisher.MonoFlatMap$FlatMapMain.onNext(MonoFlatMap.java:118) [reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxSwitchIfEmpty$SwitchIfEmptySubscriber.onNext(FluxSwitchIfEmpty.java:67) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoNext$NextSubscriber.onNext(MonoNext.java:76) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxConcatMap$ConcatMapImmediate.innerNext(FluxConcatMap.java:271) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxConcatMap$ConcatMapInner.onNext(FluxConcatMap.java:803) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxMap$MapSubscriber.onNext(FluxMap.java:108) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxSwitchIfEmpty$SwitchIfEmptySubscriber.onNext(FluxSwitchIfEmpty.java:67) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.Operators$MonoSubscriber.complete(Operators.java:1092) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoFlatMap$FlatMapMain.onNext(MonoFlatMap.java:144) [reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxMap$MapSubscriber.onNext(FluxMap.java:108) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoNext$NextSubscriber.onNext(MonoNext.java:76) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxFilterFuseable$FilterFuseableSubscriber.tryOnNext(FluxFilterFuseable.java:129) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxIterable$IterableSubscriptionConditional.fastPath(FluxIterable.java:574) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxIterable$IterableSubscriptionConditional.request(FluxIterable.java:459) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxFilterFuseable$FilterFuseableSubscriber.request(FluxFilterFuseable.java:170) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoNext$NextSubscriber.request(MonoNext.java:102) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxMap$MapSubscriber.request(FluxMap.java:149) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoFlatMap$FlatMapMain.onSubscribe(MonoFlatMap.java:103) [reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxMap$MapSubscriber.onSubscribe(FluxMap.java:86) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoNext$NextSubscriber.onSubscribe(MonoNext.java:64) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxFilterFuseable$FilterFuseableSubscriber.onSubscribe(FluxFilterFuseable.java:79) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxIterable.subscribe(FluxIterable.java:124) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxIterable.subscribe(FluxIterable.java:61) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxFilterFuseable.subscribe(FluxFilterFuseable.java:51) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoNext.subscribe(MonoNext.java:40) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoMap.subscribe(MonoMap.java:55) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoFlatMap.subscribe(MonoFlatMap.java:60) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoSwitchIfEmpty.subscribe(MonoSwitchIfEmpty.java:44) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoMap.subscribe(MonoMap.java:55) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.Mono.subscribe(Mono.java:3008) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxConcatMap$ConcatMapImmediate.drain(FluxConcatMap.java:418) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxConcatMap$ConcatMapImmediate.onSubscribe(FluxConcatMap.java:210) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxIterable.subscribe(FluxIterable.java:128) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxIterable.subscribe(FluxIterable.java:61) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.FluxConcatMap.subscribe(FluxConcatMap.java:121) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoNext.subscribe(MonoNext.java:40) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoSwitchIfEmpty.subscribe(MonoSwitchIfEmpty.java:44) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoFlatMap.subscribe(MonoFlatMap.java:60) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoFlatMap.subscribe(MonoFlatMap.java:60) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoOnErrorResume.subscribe(MonoOnErrorResume.java:44) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoOnErrorResume.subscribe(MonoOnErrorResume.java:44) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoOnErrorResume.subscribe(MonoOnErrorResume.java:44) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.Mono.subscribe(Mono.java:3008) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.drain(MonoIgnoreThen.java:167) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoIgnoreThen.subscribe(MonoIgnoreThen.java:56) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoOnErrorResume.subscribe(MonoOnErrorResume.java:44) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.core.publisher.MonoPeekTerminal.subscribe(MonoPeekTerminal.java:61) ~[reactor-core-3.1.2.RELEASE.jar:3.1.2.RELEASE]
	at reactor.ipc.netty.channel.ChannelOperations.applyHandler(ChannelOperations.java:383) ~[reactor-netty-0.7.2.RELEASE.jar:0.7.2.RELEASE]
	at reactor.ipc.netty.http.server.HttpServerOperations.onHandlerStart(HttpServerOperations.java:359) ~[reactor-netty-0.7.2.RELEASE.jar:0.7.2.RELEASE]
	at io.netty.util.concurrent.AbstractEventExecutor.safeExecute(AbstractEventExecutor.java:163) ~[netty-common-4.1.17.Final.jar:4.1.17.Final]
	at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:403) ~[netty-common-4.1.17.Final.jar:4.1.17.Final]
	at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:463) ~[netty-transport-4.1.17.Final.jar:4.1.17.Final]
	at io.netty.util.concurrent.SingleThreadEventExecutor$5.run(SingleThreadEventExecutor.java:858) ~[netty-common-4.1.17.Final.jar:4.1.17.Final]
	at java.lang.Thread.run(Thread.java:748) ~[na:1.8.0_144]

Indeed, a uri needs to be fully formed, otherwise it doesn't know what host to send it to. What are you trying to do with a relative uri? We'll keep this open to do some validation and documentation work.

For ribbon/eureka (or consul, etc...) http://cloud.spring.io/spring-cloud-static/spring-cloud-gateway/2.0.0.M5/single/spring-cloud-gateway.html#_loadbalancerclient_filter

I think the problem is that I misunderstood what the uri is for.  I thought it was for purposes of matching the route, but it seems like uri is the destination uri.  Things are starting to make more sense now (I think).
I want to have a request come in for https://abc.com/path/stuff and based on some rules on the path, I will manipulate the request and forward it to another microservice, lets say at https://def.org/stuff.
The mappings of paths to destination URIs is going to be dynamic, so I'm assuming I can't really use routes in a yml (what I was pursuing was filters to manipulate information and eventually have a controller route the request, but this doesn't seem to match spring-cloud-gateway's purpose).
Is it problematic to want to change the RouteLocator of a running system? (Either by modifying the RouteLocator being used -- it looks like it is probably meant to be immutable, or by replacing it with a different RouteLocator.)

If my new understanding of uri is correct (it is the outbound uri), then the ticket is invalid, but it might help to add an error condition before checking !scheme.equals or to flip the comparisons to !"http".equals(scheme) && !"https".equals(scheme) if it makes sense to let the no-scheme uri fall through the filter chain (I'm not sure it does) .

Is it problematic to want to change the RouteLocator of a running system?

Nevermind, I shouldn't have to do that.  I'll be using consul or ribbon.  I let my small/simple examples distract me from my actual goal.

See d5f87dc for NPE fix. Will keep this open for validation of URI.

Hey @spencergibb,
We have already checked the scheme of the URL in the commit https://github.com/spring-cloud/spring-cloud-gateway/commit/d5f87dc6a08e2e5bf26697451bf9e37aca046874
What else should we validate here for which this ticket is still open? Should we check if the entire URI is a valid URL or not? we can do that by creating a new URL and return valid if there is no exception.
Have I understood the problem correctly?

yes, but not a URL, specifically a URI.

[2021-04-27 14:35:31.814][WARN][DefaultErrorWebExceptionHandler.java:215]trace_id=1386817415517048832||host=<IP_ADDRESS>||client_ip=||url=/erlang/views/getAggregData||Failed to handle request [HEAD http://<IP_ADDRESS>:11111/]: Response status 404
The request was successful  and The log level is specified INFO
How do I resolve this WARN log


TestCaseAnimation doesn’t work

WebRequests failing?

I can't reproduce this. Maybe a platform-specific issue?

Definitely a platform-specific issue on mono


Not working with Tailwind?

Hi,
maybe I'm doing something wrong here but I cloned this repo, switched into the Tailwind example (examples/intermediate/ViteNET.Tailwind) and tried to run it, but Tailwind isn't getting picked up. Is something missing here in the repo?
I also tried it to use with Blazor in .NET 8 but it didnt't worked. I mean I can let tailwind-cli run on build with the csproj file, but I was not able to find any solution to run it with dotnet watch.
Maybe someone can enlighten me? Maybe something has changed with .NET 8 (I'm using .NET 8 btw).

Hi, I tried and it works for me. What OS are you using?

It works fine.


Having multiple Password fields causes strange behaviour

Summary
Using the mime: true setting and having multiple key-value with the same key causes unexpected behaviour, with the ordering of the different key being lexicographical instead of as per the actual order in the entry.
Steps To Reproduce
Create an entry containing:
Password: rockyou
Password: 1
Password: 2
Password: test
Password: 0

(Make sure to include the new line at the end so that it's using the new native secret format.)
And then show it using :
$ gopass edit fo9
Warning: Password is too short
$ gopass show -f fo9
Password: 0
Password: 1
Password: 2
Password: rockyou
Password: test

$ gopass show -o fo9
0

As you can see the entries are display in another order than what we inserted in the edit mode.
And then test changing the config mime to false and show or edit it: all the other keys with "password" will have disappeared!
$ gopass config mime false
mime: false
$ gopass show -u fo9
0

$ gopass edit fo9

This is really not the behaviour we want with mime set to false, we'd like to have all the content displayed rather than the Password fields being squashed into a single one.
The worst is that if you start editing it or generating a new password for the Password field, it will misbehave even more !
Notice this can cause loss of data:
$ gopass show -f fo11
Password: 123
Password: 124
Password: 124

$ gopass generate fo11 password 4
Password for fo11:password generated
$ gopass show -u fo11
Password: A0Ni

Expected behaviour
We should clarify the expected behaviour.
IMO:

we should display entries in the order they are written in the secret file
we should error out or ask the user when there are multiple entries with the same key and they do a generate or insert on that key
we should make sure the mime setting is not causing strange stuff if we switch it on and off (maybe specify how the transition is supposed to happen for all the existing types of secrets?)

The workaround for 1.10.2 will be to be able to disable parsing altogether.
But we still need to implement proper support for key-valueS instead of key-value.

The workaround for 1.10.2 will be to be able to disable parsing altogether.
But we still need to implement proper support for key-valueS instead of key-value.

By definition we only exactly one password field (the first line) now.
This issue might still apply to the key-value part, but not the password line.

By definition we only exactly one password field (the first line) now.
This issue might still apply to the key-value part, but not the password line.

Yes, I kept it open wrt to the key-value part, in order to implement key-valueS

Yes, I kept it open wrt to the key-value part, in order to implement key-valueS


Debugging choose_impl parallel issue

Details:

item1
...

Tickets:

ticket-id

I think there will be one more problem for unique id for building body network of loop operation,
because the processing order will start from 0 again for body network..
i.e., if same kernel is being used from outer network and body network, the same id might be used again.
(very low probability to have same processing order but it might be !)
I think we need to consider about this..hmm

How about following ?

Modify the type of "id" to be string instead of size_t
append id as "[network_id]_[processing_order]"

seems that this part can be shared b/w parallel and sequential codes

Sorry I check this review at now. I will update it at the official PR. thanks.

@yeonbok's last comments are applied to the merged PR. thanks.


Add accelerometer data so it can be used in Yuzu

Despite using Cemuhook, it is not possible to use this service on Yuzu due to the fact that there is no accelerometer data provided. Please add

This repo only for cemu.
May be it's can help you https://github.com/iiibbbmmm/WebGyroForCemuhook

This repo only for cemu. May be it's can help you https://github.com/iiibbbmmm/WebGyroForCemuhook

I came across this a couple of days ago. Tried it and it works on Yuzu too! Thank you


Code completion for variable declararation not working

Am I the only one who don't get auto-completion for declared variables in the same file? Manually typing all variables is really bothering. I can see there is a feature Code completion for all contracts / libraries in the current file and all referenced imports in README, but it doesn't seem working.
I am using VS Code 1.22.2 in macOS High Sierra 10.13.4(17E202), with the latest extension provided(0.0.40) and I'm writing with solidity v0.4.23.

variables on the same function have not been implemented yet. So you will get autocomplete from attributes in the current contract and methods.

#58 reference autocomplete improvements

Hi Juan,
I am facing the same problem, for me autocomplete is not working in the current contract.

@as-ajitsingh are you using latest? I had the same issue as 24 is out.

This is an annoying issue indeed. A quick look seems to suggest because when we incrementally type variable name, the contract enters a compilation error state thus cannot auto-complete variable names in the current scope.
I suppose we could try complete without current line if completion fails but it feels too hacky and wouldn't work for multi-line statement. The proper solution is probably use a non-validating parser instead so it returns best effort result.
If I cmd + space then type y, I get

but if I type y then cmd + space

Yes, you are right, I was thinking about using the Antlr parser but still have seen issues.


Extract formatted text into a separate component

Right now if we want to inline some piece of code we're using the formatted class:
<span className="formatted"> type="" </span>

Let's extract into a separate component so we don't have to write this class over and over again.
Make sure the component has an as prop in case we want to render it as a different element (eg: <p>, <strong>, etc)

export default function Formatted(props) {
  return (
    <>
      <props.as>{props.children}</props.as>
    </>
  );
}

Do you want something like this ?

Yes, but I think it should also be a forwardRef component, so we can apply normal HTML attributes to it, like title:
<Formatted as="p" title="Greeting">  Hello, I am bob! </Formatted>

export default function Formatted(props) {
  return (
    <>
      <props.as {...props} className="formatted">{props.children}</props.as>
    </>
  );
}

just destructuring the props will do the work i hope ...

Yes, I think you might be right.
We're using TS so ideally we would have types. For instance, in you example, since you used as="a", it would be great to get suggestions for the correct props: href, target, just like when using the normal <a> element.

I found this thread in case it might help: https://stackoverflow.com/questions/55969769/typing-a-dynamic-tag-in-react-with-typescript

PS: I've assigned you. Thanks :D

Thank you so much for the guide !!  😊

No worries :D
You'll probably get some errors when trying to run the project, will try and fix that in the following days. Until then, I think to test you can just run yarn build and verify that the build is created.

Do i need to install exactly node v 16.1.0 ?  It is showing me this error ....

Yep. I think easier it would be to change this line "node": "16.1.0" to your version. Then before you create the PR just revert back that line.
It should work.
In the meantime I added this issue: https://github.com/FrontEnd-ro/frontend.ro/issues/234. I think it should allow everyone with a 16+ version

I have created the component and also separated the scss file and its working fine. Also replacing all the className="formatted" lines with the Component throughout the whole project

Great! I think you can open a Pull Request then.

I forgot to update our CONTIBUTING.md file but lately we've been adding the issue inside the commit message (in parantheses ())

I
'm getting only this error all over code while running yarn build
Should I ignore this ?

I think you can safely ignore it, yep. It might be actually a problem with my config...

Not a problem at all. I have adjusted my settings according to your config


Start db container using gitlab image from ecr

https://github.com/Shippable/pm/issues/10021

this is done, closing


Some fixes

Some fixes for JWM

Closing. This introduces a bug and memory leak in path expansion. It also replaces stack allocations with heap allocations without reason or an evaluation of the consequences.


kcp CLI: error out for an non existent workspace while changing ws

previously, cli used to proceed creating bad state even
if the workspace doesn't exist on the cluster.
now, it validate the workspace if it exist on the cluster.
Fixes #1190
Signed-off-by: Shivam Mukhade<EMAIL_ADDRESS>Changes applied on cluster:
Current workspaces in the cluster
 > ~ > k kcp ws
Current workspace is "root".
 > ~ > k kcp ws list
NAME      TYPE           PHASE   URL
default   Organization   Ready   https://<IP_ADDRESS>:6443/clusters/root:default
green     Organization   Ready   https://<IP_ADDRESS>:6443/clusters/root:green
 > ~ > k kcp ws use green
Current workspace is "root:green" (type "Organization").
 > ~ > k kcp ws list
NAME   TYPE        PHASE   URL
g1     Universal   Ready   https://<IP_ADDRESS>:6443/clusters/root:green:g1
g2     Universal   Ready   https://<IP_ADDRESS>:6443/clusters/root:green:g2 > ~ > k kcp ws
Current workspace is "root".
 > ~ > k kcp ws list
NAME      TYPE           PHASE   URL
default   Organization   Ready   https://<IP_ADDRESS>:6443/clusters/root:default
green     Organization   Ready   https://<IP_ADDRESS>:6443/clusters/root:green
 > ~ > k kcp ws use green
Current workspace is "root:green" (type "Organization").
 > ~ > k kcp ws list
NAME   TYPE        PHASE   URL
g1     Universal   Ready   https://<IP_ADDRESS>:6443/clusters/root:green:g1
g2     Universal   Ready   https://<IP_ADDRESS>:6443/clusters/root:green:g2

CLI output for invalid workspaces
 > ~ > k kcp ws junk
error: clusterworkspaces.tenancy.kcp.dev "junk" not found
 > ~ > k kcp ws ju:nk
error: invalid workspace name format: ju:nk
 > ~ > k kcp ws root:abc
error: clusterworkspaces.tenancy.kcp.dev "abc" not found
 > ~ > k kcp ws root:green:g3
error: workspaces.tenancy.kcp.dev "g3" not found
 > ~ > k kcp ws
Current workspace is "root".
 > ~ > k kcp ws root:default
Current workspace is "root:default".

Summary
Related issue(s)
Fixes #

Why should we do this in the CLI plugin as opposed to the admission chain in the server? If we send back useful error messages, all clients should be able to surface them.

I see, sorry - when I read that the validation was telling someone about an invalid format, I thought we were creating workspaces. In my mind, the check to see if a workspace exists is sufficient - if it's an invalid format, it won't exist. Since the user is not trying to create a new workspace, it's not as important to tell them about formats that can be valid.

@sttts  @davidfestal I have addressed the comments ptal :)
I am trying to add unit test but one of the current tests is failing
more details  https://github.com/kcp-dev/kcp/pull/1278#issuecomment-1157660387
if you could help me understand the use of prettyNames in the test

Perfect.
/lgtm
/approve
/ok-to-test

Unit test broken.

Unit test broken.
@sttts
yeah. I am not sure how to handle TestUse/absolute name Test
It is trying to find bar workspace (root:foo:bar)
but here https://github.com/kcp-dev/kcp/blob/060f5ec825c777c190cf27518ec33be1b8ea88a7/pkg/cliplugins/workspace/plugin/kubeconfig_test.go#L337
prettyNames is replacing {"bar": "baz"}
can you tell how can I fix this?

The pretty names go away soon. Doens't it help to just make that pretty-name map empty?

The pretty names go away soon. Doens't it help to just make that pretty-name map empty?

@sttts yep. Thanks. it does fixes it. added a couple of more tests. ptal :)


Cmdlet is not available on Exchange Online?

Just tried to use this cmdlet, and get the error that it is not a recognized cmdlet.  I am running an EXOPSession and have admin access.  Did this cmdlet get deprecated?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3665db68-7dd8-2e93-1797-505eb77a22ce
Version Independent ID: 9cf5b8f2-a6aa-e79b-70e7-cfe5c1aee69c
Content: Get-RecoverableItems (exchange)
Content Source: exchange/exchange-ps/exchange/mailboxes/Get-RecoverableItems.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

nvm - it was the role "Mailbox Input Output" that I needed, that and giving it about 15 min for the role permission to propagate and take effect.

@DigitalFrog007, OK. Closing.


Add Experts callout to bounces.md

Description of the change:
Reason for the change:
Link to original source:
Related to #4614

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hello @rosariopfernandes,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


link_to_remove_association breaks when floated

Hi!  I've been using cocoon on many projects for several years and I love it!
I have a project working fine.  link_to_remove_association and link_to_add_association work as expected, and all of the server side stuff works.
I would like to have the link_to_remove_association appear next to the field label.  I have tried:

adding the bootstrap float-left class using render_options;
passing in a style='float:left' block using the same;
adding css to the application.css like .remove_fields {float: left !important;};
adding a float-left class to the generated HTML (using Inspect).

All of these methods make the link_to_remove_association appear as I want it, but also make the link stop working.  Clicking on it does nothing, and shows nothing in the console.
Thanks!

EDIT:
I found a workaround by wrapping .nested-fields with a .nested-fields-wrapper, and putting the link_to_remove_association and .nested-fields within that wrapper and changing the wrapper_class option.  That seems to work ok.
Still curious if you have any idea why the previous way I described would make it stop working.

Great that you found it!
I guess your solution kind of explains why it didn't  work? The link_to_remove_association  searches for the nearest/closest surrounding wrapper_class (default nested-fields) and will remove it. However, probably, because of the float, the link was outside the nested-fields block, and so it was no longer found?


Django 1.11 to 2.2 support

I am looking for a few sponsors to work on these features:
https://opencollective.com/celery

Would like to contribute as a code contributor on this feature.

Would like to contribute as a code contributor on this feature.

That's not a sustainable solution IMHO and from previous experience.

@auvipy A fundraiser seems like a good idea.
As it stands, what's currently missing / would be a new feature from Django 1.11/2.2 support?  Is there list / issue thread of stuff that funding would go to?

work on celery 3.x branch for some new releases with numerous bug fix and some new features needed for smoother migration to celery 4.x and Django-celery working with them. in a nutshell

could you plz provide a sponsor link to pay?

https://opencollective.com/celery

@auvipy  Place where we can find the list of bugs/feature needed in Django-celery for celery 3.x for smoother migration to celery 4.x?

I'd be happy to support if you have a description of the project and estimate of the cost.

can you drop a line<EMAIL_ADDRESS>

Has any head way been made ?  I have some time I could contribute as well.  I am working on an old project using djcelery and would like to update it.

Also happy to chip in if needed.

Looks like the functionnalities have been incorporated into different projects:
https://github.com/celery/django-celery-results
https://github.com/celery/django-celery-beat
We have an internal build working with django 3+, python 3.7 and celery 4.4.0 using some of the latests commits from djcelery master branch as well.  Once I am satisfied with the quality of the solution, I will share it (we are not using a large portion of the code and we fix the bugs as they come instead of refactoring).
This is our relevent requirements.txt:
celery==4.4.0
Django==3.0.6
django-celery @ git+git://github.com/celery/django-celery@a78b3a5e0d3b28ee550c70cc509d608744c88337 # master branch
hopes this helps.

it's for celery 3.x series only. those two are for celery 4.x series+

we might add django 1.11 support to ease the migration

we might add django 1.11 support to ease the migration


[BUG] A resource failed to call close.

This is a critical bug. If files don't get properly closed, the app runs out of file descriptors and the previously opened files, are not retrieved when calling PhotoManager.getAssetPathList(); again.
Describe the bug
I get the console error and the images are no longer retrieved after this
W/System  (32145): A resource failed to call close.

To Reproduce
Steps to reproduce the behavior:
1.
List<AssetPathEntity> list = await PhotoManager.getAssetPathList();

AssetPathEntity data = list[0]; // 1st album in the list, typically the "Recent" or "All" album
List<AssetEntity> imageList = await data.assetList;

AssetEntity entity = imageList[0];

List<int> fileData = await entity.fullData; // image/video file bytes

See
W/System  (32145): A resource failed to call close.

Retrieve assets again and see that the picked asset is missing now.

Now, I can't see the log.


Merging v1-dev into main

See #186

Actually it's a lot simpler to just merge the single change we want from main to v1-dev and then rename that to main. Closed for 55e4854de63c5faf2a495453d7ed5399ff4322cf


Support seamless interop between ratio and rational Magnitude

We provide two new functions, numerator(m) and denominator(m), for a
Magnitude m.  They fulfill the following conditions:

numerator(m) and denominator(m) are always integer Magnitudes.
If m is rational, then m == numerator(m) / denominator(m).

If m is not rational, then the numerator and denominator are not
especially meaningful (there is no uniquely defined "leftover irrational
part").  However, we choose a convention that matches how humans would
write a mixed number.  For example, sqrt(27/16) would have a numerator
of 3, denominator of 4, and a "leftover part" of sqrt(3), matching the
"human" way of writing this as [(3 * sqrt(3)) / 4].  This has no use
yet, but it may later be useful in printing the Magnitude of an
anonymous Unit for end users.
To further reduce friction for the upcoming migration, we provide a
conversion from a Magnitude to a ratio.  We restrict this operation to
rational Magnitudes, and guard this with a static_assert.

The PR after this one is the "big one": the one that changes us over from "mostly ratio" to "mostly Magnitude".  I just got the builds to pass for the first time on that one, so I'm putting this one out for review!

I've just now split up the "big one" into a sequence of 7 commits, to reduce cognitive load for the reviewers.  Once this present PR is landed to master, I will put the "big one" out for review.

Was there anything else you wanted to see before merging?  Once this is merged, I'll be ready to post the next PR.

My fault, I forgot to merge when all checks have passed (too busy...).

No worries :sunglasses:

@chiphogg, I am a bit worried about the explosion of types here. Something that was units::ratio{1, 1, 3} now is units::magnitude<units::base_power<long int>{2, units::ratio{3, 1, 0}}, units::base_power<long int>{5, units::ratio{3, 1, 0}}>]. This makes user experience with compilation errors much worse (even if we remove exp component from ratio) 😢
I think that the main reason is the proposed solution is too generic. In real life, we will be mostly dealing with the magnitudes of 10 and sometimes with magnitudes of 2 and pi. If something else will appear it will be really rare. Can we optimize for those common cases to make such common types much shorter?

Please take into account that we may be forced to get rid of the downcasting facility before the standardization process.

Thanks for raising the question!  My intuition is that magnitude specializations should tend not to appear in end users' compiler errors, because we use opaque typedefs for our unit types.  Thus, for example, only inches should appear in a compiler error, not something like unit<dimension::length, magnitude<base_power<long int>{2, ratio{-3, 1}}, base_power<long int>{5, ratio{-4, 1}}, base_power<long int>{127, ratio{1, 1}}>.
What are some of the compiler errors you saw where magnitudes showed up?  It'd be interesting to see which unit-forming pathways are vulnerable to this problem.
In any case: yes, I think it makes sense to optimize for common use cases.  Magnitudes use a vector space representation, and we can choose whatever basis is most useful.  If we want 10 to be a basis vector, and also 2, then 5 can't be one.  Conceptually, it's simple enough to say "compute the prime factorization, then borrow as many powers of 2 as we need to turn the coefficient for 5 into one for 10".

My intuition is that magnitude specializations should tend not to appear in end users' compiler errors, because we use opaque typedefs for our unit types.

Well, this is true only for values assigned to variables by the user. If only we are dealing with the result of some dimensional calculation the resulting type may be really long. See here: https://godbolt.org/z/8qKW91ao5. In many cases, the downcasting facility can convert a long intermediate type into the short one provided by the user but it will work only if such type was predefined by the user. Specifically, it will not work for some temporary intermediate calculation results that may not have any dimensionally valid sense. Also, as I mentioned before, the downcasting facility may not be accepted by the ISO C++ Committee, which means that every result of the dimensional calculation will result in a long type (at least before it is being manually assigned by the user to the predefined short type).

Thanks for the examples.  So is your current thinking that we would be OK with a change of basis that "swaps out" 10 for 5, shortening most magnitude typenames we would expect to see in practice?  Or are the longer long types so bad that you're reconsidering switching to magnitude at all?
If the latter: I should say explicitly that it wouldn't hurt my feelings.  :slightly_smiling_face:  While I've put in a lot of work, I don't subscribe to sunk cost fallacies, and I want decisions to be driven by what is best for the project.
My own assessment of the tradeoffs is that becoming resilient to overflow, and gaining support for both rational powers and irrational scale factors, outweighs the price we pay in longer compiler errors, particularly if we change the basis to reduce that cost.  However, that's only my assessment, and I feel it's unavoidably biased, so I'd appreciate hearing your thoughts.

Something that was units::ratio{1, 1, 3} now is units::magnitude<units::base_power<long int>{2, units::ratio{3, 1, 0}}, units::base_power<long int>{5, units::ratio{3, 1, 0}}>].

Well, this is true only for values assigned to variables by the user. If only we are dealing with the result of some dimensional calculation the resulting type may be really long. See here: https://godbolt.org/z/8qKW91ao5.

Seems like CE's version isn't up-to-date. I don't see a mention of base_power in the error message.

I know that this cannot work on all the current compilers as they will not take a floating-point number as an NTTP for now. Maybe some workaround could be introduced like:
template<auto... Base>
struct base_power;

template<>
struct base_power<3, 14> {
  static constexpr auto base = std::numbers::pi;
  std::intmax_t num = 1;
  std::intmax_t den = 1;
};

to express pi?
Anyway, some food for thought...

Such change would replace
units::magnitude<units::base_power<long int>{2, units::ratio{3, 1, 0}}, units::base_power<long int>{5, units::ratio{3, 1, 0}}>
with
units::magnitude<units::base_power<10>{3, 1}> for kilo which looks quite nice to me and not that much worse than units::ratio{1, 1, 3}.


IDENTITY-3855 Can't login to Dashboard if external BPS is down

How fixed https://wso2.org/jira/browse/IDENTITY-3855 by above code changes ?

The issue has been occurred due to wrong method name, 'isDebugEnabeld' has used instead of 'isDebugEnabled',  This occurred since iDEA plugin shows wrong suggestions and jaggery issue has reported at https://wso2.org/jira/browse/JAGGERY-418


Test Release Drafter usage on backport branches

Currently the action assumes that there is a next pseudo-version for Release Drafter. This system likely does not make much sense when backport branches are involved. Should the action detect that it is running on a nondefault branch and just skip the usage of Release Drafter?

Does not seem to run automatically: unexpected commit from verify-ci-status-action.
Running manually on the backport branch seems to work, including picking the correct PRs for R.D.: https://github.com/jenkinsci/ssh-credentials-plugin/releases/tag/277.280.v1e86b_7d0056b_
It does use the next release name, though only temporarily, so this seems like a race condition (do not try to cut releases from both trunk and a backport branch around the same time).

It does use the next release name, though only temporarily, so this seems like a race condition (do not try to cut releases from both trunk and a backport branch around the same time).

This seems reasonable, otherwise your asking for trouble? 😅


Failed to start neoray due to uninstalled font

OS: macOS 13
Neoray version: v0.2.4
nvim version: v0.8.0

Describe the bug:
When I use a font that is not installed on my system, the following bug occurs.

Thanks TianYaX for both issue and pr. But pr does not fix the issue but hides it. Please try the 7ec4f6c23ba13de924e0cd5d1836783166723abf

I tried to apply this commit(https://github.com/hismailbulut/Neoray/commit/7ec4f6c23ba13de924e0cd5d1836783166723abf) but it doesn't seem to work...

Is it possible to use the default font and give an error? like this pr😄

Is it possible to use the default font and give an error? like the pr😄

😂
Actually pr is good but the problem is not using the default font, it's not our fault either.. We must give an error to the user while the Neoray is not launched with nofork and there is only way to do it, show an error message in neovim (actually not only way, we can show a system error dialog). User should know their font is not found by Neoray with this name. Also setting kit to nil is unnecessary here because the kit is already nil and font system will use default font while it is nil. There is no problem with that. Only problem here is neovim does not give a promise us the cursor position will be in the grid area. We can't assert that. The simplest solution is adding a SafeCellAt function to grid and check for bounds and return an empty cell if it out of bounds. Also I liked the system dialog idea (I thought of it while writing this). It will be safer than calling neovim api for showing error message because neovim sometimes does not write this echoerr/msg calls (this is also is not our problem and we can't fix this)
I'm also curious about your opinions

Is it possible to use the default font and give an error? like the pr😄

😂
Actually pr is good but the problem is not using the default font, it's not our fault either.. We must give an error to the user while the Neoray is not launched with nofork and there is only way to do it, show an error message in neovim (actually not only way, we can show a system error dialog). User should know their font is not found by Neoray with this name. Also setting kit to nil is unnecessary here because the kit is already nil and font system will use default font while it is nil. There is no problem with that. Only problem here is neovim does not give a promise us the cursor position will be in the grid area. We can't assert that. The simplest solution is adding a SafeCellAt function to grid and check for bounds and return an empty cell if it out of bounds. Also I liked the system dialog idea (I thought of it while writing this). It will be safer than calling neovim api for showing error message because neovim sometimes does not write this echoerr/msg calls (this is also is not our problem and we can't fix this) I'm also curious about your opinions

giving the user an error message is indeed a good interactive experience


zabbix plugin version 3.10.1 corruption

Describe the bug
After update Zabbix plugin from version 3.9.1 to 3.10.1 , we face a connection problem with Zabbix server. All the dashboard had a massage "t is null".
After this error, we rollback to version 3.9.1
We have Zabbix version 3.2.4 with Oracle Database , grafana version 5.3.4 and Zabbix plugin 3.9.1.
All installations is in the same server (Zabbix application, DB, grafana)
Any installation or config tips for new version , will be gladded.
Please find a screenshot with the errors.
Best Regards.

@nlioumis try to use Grafana 5.4.3 or 6.0.


Update ebs_volume to support new volume types #238

https://github.com/chef-cookbooks/aws/issues/238
Obvious fix.
https://aws.amazon.com/ebs/details/

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@bernardgardner

Please sign the CLA here.

Thanks!


fix: vastly reduces number of db queries

Problem:  The creation of some tables was causing a large number of
queries to be run which itself scaled with the size of the database,
i.e., larger databases (such as the one in production) caused more
queries to be made the the db itself (a problem which componds on
itself).  This issue was not apparent in development because of the
smaller size of the development db.
First, we can verify the above problem by increasing the size of the
development db.  We can do this by temporarily editing
scripts/entrypoint.sh, chanding line
    python make_fake_fixtures.py 1000 1000 1000 > /tmp/fake_agagd_data.json

with
    python make_fake_fixtures.py 100 10000 100 > /tmp/fake_agagd_data.json

and then deleting and recreating the docker images/containers/volumes.
Before the change, the development database was creating data for 1000
players, 1000 games, and 1000 tournaments.  Note that this meant that,
on average, each player only played one game and each tournament only
contained one game.  This was helping mask the primary issue.  After the
change, the development database is creating 100 players, 10_000 games,
and 100 tournaments, meaning that, on average, each player played 100
games, and each tournament contained 100 games.  We note that, although
this new development db is still significantly smaller than the one in
production, the local development app is already having trouble
responding to requests in involving player and tournament pages.
With the new larger development database in place, we observe the
following concerning facts:

a random player page requires ~1000 queries to be made.
a random tournament page requires ~100 queries to be made.

Solution:
This commit addresses the above issues by making 2 primary changes in
how some tables are computed or rendered:

The first involves the way player names and ids were rendered in the
game tables using the custom
agagd_core.tables.players.LinkFullMembersNameColumn.  This custom
column had a .render() method which internally made a query to the
db.  Then, for the rendering of the whole table, the .render()
method was being called for every entry in the table data, causing
many queries to be made.  To solve this issue, we removed
LinkFullMembersNameColumn altogether, and constructed the
appropriate queries to construct the "player-name (player-id)" label
once for all entries in the table.

The second involves the way opponent and tournament data was
collected to create the opponent and tournament tables in the
player_profile view.  This data was collected and manipulated in pure
python, using explicit python loops over the Games model, and at
least one db query per iteration.  To solve this issue, we created
appropriate queries which gathered and aggregated the required data
using a couple of large queries (rather than many small ones).

After the above changes:

the number of queries made in a random player page dropped from ~1000
to ~10.

the number of queries made in a random tournament page dropped from
~100 to ~5.

I will note that this commit gets rid of a complicated part of code which was constructing the data for some tables, and reimplements similar logic using more straightforward queries.  However, in doing so, some facets of the previous data-collection may have been changed.  I will add some comments in the places where I know this is happening, so that somebody can review the importance of those aspects.


Add Support to FAISS Retiever!

How we can use FAISS retriever in RAG pipeline

You have seen the documentation for it, right? https://dspy-docs.vercel.app/api/retrieval_model_clients/FaissRM

Thanks! @tom-doerr


Key files are intermittently not reloaded when hitting /reload endpoint

This is a particularly tricky issue to debug because it's intermittent, and so far I have not managed to reliably reproduce it, but I can say with certainty it happens every week or so and requires manual intervention when I receive alerts that attestations are being missed.
I'll outline the rough setup which causes the issue and hopefully you'll be able to (help me) figure out what's going on, and confirm whether there is an underlying caching bug in the Web3Signer key reload logic or not.

Using Web3Signer v23.3.1:

Write 30k individual key files to disk, each with a config similar to the following:

keyName: key
keyPath: /v1/secret/data/goerli/12345
keyType: BLS
serverHost: localhost
serverPort: 8200
timeout: 30000
tlsEnabled: false
token: unused_handled_by_vault_proxy
type: hashicorp

Hit Web3Signer's /reload endpoint to load the first batch of keys

Write another 30k individual key files to disk, each with a config similar to the above

Hit Web3Signer's /reload endpoint to load the next batch of keys

Hit Web3Signer's /api/v1/eth2/publicKeys endpoint and observe that less than 60k keys are actually loaded. In this example, 59957 keys are loaded and 43 are missing. This is the intermittent part and is hard to reliably reproduce.

Hit Web3Signer's /reload endpoint and note that it reports that there are no new keys to load, even though 43 are missing:

2023-08-22 09:32:39.146+00:00 | ForkJoinPool-117-worker-1 | INFO  | SignerLoader | Parsing configuration metadata files
2023-08-22 09:32:39.146+00:00 | ForkJoinPool-117-worker-1 | INFO  | SignerLoader | Total configuration metadata files processed: 0
2023-08-22 09:32:39.146+00:00 | ForkJoinPool-117-worker-1 | INFO  | SignerLoader | Total signers loaded from configuration files: 0 in 00:00:00.000 with error count 0
2023-08-22 09:32:39.146+00:00 | pool-2-thread-1 | WARN  | Eth2Runner | No BLS keys loaded. Check that the key store has BLS key config files
2023-08-22 09:32:39.146+00:00 | pool-2-thread-1 | INFO  | DefaultArtifactSignerProvider | Total signers (keys) currently loaded in memory: 59957

Identify the files for the keys that are not returned by Web3Signer's API (i.e. not loaded)

Manually touch the 43 keys that have not been correctly loaded to update the modification time

Hit Web3Signer's /reload endpoint again and note that it reports:

2023-08-22 09:45:27.281+00:00 | pool-2-thread-1 | INFO  | RegisteredValidators | Validators registered successfully in database:43

So it seems that somehow Web3Signer is picking up all the key files the first time around, failing to load a subset into memory, but incorrectly caching the fact the the keys have been loaded, and then failing to pick them up on the next /reload, until the modtime is manually changed.
I tried to follow the code through to see how this might be happening, but didn't get very far. But there definitely seems to be an issue related having a large number of keys, (re)loading, and caching.

Have also noticed that after touching the files as described above and successfully reloading them, the logs report that all 60k keys were loaded:
2023-08-22 10:49:05.476+00:00 | ForkJoinPool-121-worker-1 | INFO  | SignerLoader | Total configuration metadata files processed: 60000
2023-08-22 10:49:05.476+00:00 | ForkJoinPool-121-worker-1 | INFO  | SignerLoader | Total signers loaded from configuration files: 60000 in 00:01:41.635 with error count 0

However when I list the public keys via /api/v1/eth2/publicKeys, I get the following back:
$ curl -sS http://localhost:9000/api/v1/eth2/publicKeys | jq length
59986

So something fishy is definitely going on here.

Hi @gg718
Thank you for reporting this. I'll try to reproduce it locally. Meanwhile, can you enable serial processing of metadata config files and see if that makes any difference in regards to loading all the files (both at restart and /reload)? --Xmetadata-files-parallel-processing-enabled=false

@gg718 you should definitely upgrade to 23.6.0 (or possibly to 23.8.1 once it is announced as it uses Java 17 both for compiling and runtime). It also contains for hashicorp using http 1.1. 23.6.0 contains other fixes related to database contention as well. I suspect that you are still facing the local socket running out issue when connecting to hashicorp. Using serial processing would eliminate this possibility. You can then attempt to increase the local socket connections.

@gg718 any updates trying to observe this issue with latest web3signer release?

@usmansaleem Only managed to upgrade to v23.8.1 on Monday, so will monitor the situation and let you know. So far, I haven't noticed anything strange going on.

Closing the issue. Feel free to reopen this issue or create a new one if regression is noticed.


optional size limit

Is there a way to abort requests if the response content exceeds a predefined size?
I'd like to avoid that my NodeJS process screws up just because axios is requested to load a URL that returns some gigabytes of data...

This isn't currently possible, but could be done by adding a maxContentLength config option, then rejecting for requests that exceed that. I would be happy to accept a PR if you want to take a stab at this.

looking into this..

@mzabriskie Would we want reject with an err or with a response on this one?

I'd go with an Error object

I think we can close this issue.

Fixed by #275

Is there anything new regarding this topic?


feat: add Songmu/maltmill

https://github.com/aquaproj/aqua-registry/pull/6515 Songmu/maltmill: create and update Homebrew thrid party Formulae
$ aqua g -i Songmu/maltmill

How to confirm if this package works well
Reviewers aren't necessarily familiar with this package, so please describe how to confirm if this package works well.
Please confirm if this package works well yourself as much as possible.
Command and output
$ maltmill --help
maltmill - Update or create homebrew third party formulae

Version: 1.0.1 (rev: 9622920/go1.19)

Synopsis:
    % maltmill -w [formula-files.rb]

Options:
  -token string
        github `token
  -w    write result to (source) file instead of stdout

Commands:
    new            create new formula

If files such as configuration file are needed, please share them.
$ maltmill new -w aquaproj/registry-tool
[maltmill] created "registry-tool.rb"

$ cat registry-tool.rb
class RegistryTool < Formula
  version '0.1.4'
  homepage 'https://github.com/aquaproj/registry-tool'

  on_macos do
    if Hardware::CPU.arm?
      url 'https://github.com/aquaproj/registry-tool/releases/download/v0.1.4/registry-tool_darwin_arm64.tar.gz'
      sha256 '44e74fc2e7cf7d0ee9eaea860ddb40638002baaa1d43074d619d3119b3c23b10'
    end
    if Hardware::CPU.intel?
      url 'https://github.com/aquaproj/registry-tool/releases/download/v0.1.4/registry-tool_darwin_amd64.tar.gz'
      sha256 '202ea257c22f525ae1c9d0c16c6606b6f8a28407e2c6d2776e2265ceb6622df9'
    end
  end

  on_linux do
    if Hardware::CPU.arm? && Hardware::CPU.is_64_bit?
      url 'https://github.com/aquaproj/registry-tool/releases/download/v0.1.4/registry-tool_linux_arm64.tar.gz'
      sha256 '71530aabd832faead521ff48ab48cdabf49ab9ec3dccef017ae513ad503e9fb0'
    end
    if Hardware::CPU.intel?
      url 'https://github.com/aquaproj/registry-tool/releases/download/v0.1.4/registry-tool_linux_amd64.tar.gz'
      sha256 '532673490deb9459481e25e22c8e2a3143e802b1f51a18981bed8c5ab12a8198'
    end
  end

  head do
    url 'https://github.com/aquaproj/registry-tool.git'
    depends_on 'go' => :build
  end

  def install
    if build.head?
      system 'make', 'build'
    end
    bin.install 'registry-tool'
  end
end

Reference

README.md

https://github.com/Songmu/maltmill/blob/main/README.md

Thank you always!

Released. https://github.com/aquaproj/aqua-registry/releases/tag/v3.61.0


Page crashes when application name has  ""

For a long time I had problems seeing the report. When I tried to access any statistics, such as "top time comsuming queries", the page would break. So I decided to open the browser's development console and it showed an error in specific positions of two lines. I then realized that the broken data was the application_name and that it contained some peculiar information. The DBeaver client set the value to be: "app + version + a text between '<>' " . Something like: "DBeaver 12.3.7 <Some script.sql>".
I edited the index.html removing the value "" and the page reopened.
Steps to simulate the problem:
1 - Connect to database
2 - set application_name = 'test <some script.sql>';
3 - set log_min_duration_statement= '1s';
4 - select pg_sleep(2);
5 - generate pgBadger report;
6 - try to open a statistic page;

Commit 853eedf fixes this issue.


feat: add breadcrumbs and route for selected submission

This PR adds breadcrumbs to the navbar, i.e. navigating to lab7 of a course will display:
Dashboard / Course Name / lab7

where each section will take you to the corresponding routes.

It also adds functionality to remember which submission is selected in the Results view.
Selecting a submission with ID=541 will add ?id=541 to the URL.
Navigating to <domain>/course/1/results?id=541 will select the submission with ID 541 if it exists.
Although #409 refers to the old UI, this PR somewhat adresses the first point.

The only minor thing in the heading is the use of / between the sections. Perhaps another symbol could be used. Ideas include | or ◆ or some other symbol that signals separation.


[BUG] You can't change the template in v3

Describe the bug
Clicking on a template name in a resume doesn't do anything
Product Flavor

[x] Managed (https://rxresu.me)
[ ] Self Hosted

To Reproduce

Go to https://rxresu.me and log in
Create a new (blank) resume
Load the sample data by going to Settings > Load sample data
Scroll to the top of the right panel and click any of the template names

Expected behavior

The resume's template changes to the template name that was clicked
Screenshots

Desktop (please complete the following information):

OS:  NixOS 20.03 (Linux)
Browser  Brave
Version  1.15.76 Chromium: 86.0.4240.111 (Official Build) (64-bit)

Additional context

This works flawlessly in the v2 version of RxResume

I was not able to reproduce this issue on my system. Does this continue to happen even now?

Can confirm this issue can't be reproduced (upgrading Chromium from 86.0.4240.111 to 96.0.4664.110 fixed this issue)


[Bug] Entry Span does not override tag(AbstractTag, String) method

Search before asking

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

Apache SkyWalking Component
Java Agent (apache/skywalking-java)
What happened
In the EntrySpan, it only overrides tag(String, String)
    @Override
    public EntrySpan tag(String key, String value) {
        if (stackDepth == currentMaxDepth || isInAsyncMode) {
            super.tag(key, value);
        }
        return this;
    }

but not this method,
@Override
    public AbstractTracingSpan tag(AbstractTag<?> tag, String value) {
// ...
    }

while ExitSpan does for both.
What you expected to happen
We should add,
    @Override
    public AbstractTracingSpan tag(AbstractTag<?> tag, String value) {
        if (stackDepth == currentMaxDepth || tag.isCanOverwrite() || isInAsyncMode) {
            super.tag(tag, value);
        }
        return this;
    }

This cause confusion when we were investigating Spring 6.x test failure in https://github.com/apache/skywalking-java/pull/466#issuecomment-1452944810.
This latter issue is reported in #
How to reproduce
Run Spring 6.x scenario test case and debug locally
Anything else
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Close for now. No update in a month.


Decide how we want to store analysis stats like decorrelation times and starting frame

After processing a replica exchange simulation, we get the frame at which the production period starts and the spacing between uncorrelated data determined from the pymbar timeseries module. Similarly, from native contacts calculations, we get the spacing between uncorrelated native contact fraction data points, a list of native contact pairs, and the types of those pairs. It would be useful to save all of this information to be used in multiple different analysis scripts, perhaps as a serialized dictionary.
Right now, we have to manually enter the starting frame in the analysis functions.

I've been storing the trajectory analysis stats in a pickle file of a dictionary, at least for the signac workflows, and that seems to be working out fine.


Hash values do not match across architectures with different Int sizes

Because the result of hash is always Int, I can't make a Hashable instance that can produce string hashes that match across architectures.   I'd like to do this with ghc (64 bit Ints) and ghcjs (56 bit Ints.)  Anyone know a solution?

This is by design and documented.
https://github.com/haskell-unordered-containers/hashable/blob/master/src/Data/Hashable/Class.hs#L237-L239
Duplicate of https://github.com/haskell-unordered-containers/hashable/issues/139

I'm not so much asking for a change to Hashable as wondering if there is an existing Haskell solution to my problem.

There are libraries for cryptohashes, e.g. https://hackage.haskell.org/package/cryptohash-sha256 or cryptonite. I'm not aware of libs for "fast on some platform" hashlibs (i.e. stable but not-so-fast on non-primary platforms. Although GHCJS is usually slow, if there aren't any shims specifically for it).
Note: hashable is the driver for unordered-containers, where you shouldn't care about exact hash values. If you do, hashable is not a library for you.


KAFKA-5326:When resize the index file, maybe cause the content disappear

This modify to resolved the KAFKA-5326.

When the kafka server is running, someone removed the index file on the disk manually, if at this point, the function AbstractIndex.Resize(newSize: Int) is triggered, will create a new ".index" file which the size is "roundedNewSize", but the content is empty.
I think if the ".index" is not exist when the function AbstractIndex.Resize(newSize: Int) running , we should copy the contents from old "mmap" to new "mmap" to avoid the "empty file".

@rajinisivaram, maybe you can review this one. It's a tiny change.

@ijuma, maybe you can review this one. Thanks.

cc @junrao


[CERT-TEST-FAILURE] TC-LVL-2.3 - Test Case Fails Due to First Entry in reportedCurrentLevelValuesList Not Being Larger Than startCurrentLevel at step 11

Feature Area
Other
Test Case
TC-LVL-2.3
Description
The test case test_TC_LVL_2_3 fails because the first entry in reportedCurrentLevelValuesList is not larger than startCurrentLevel. According to the test plan, the first entry should be greater than startCurrentLevel when reportedCurrentLevelValuesList contains two or more entries.
PFA test plan screenshot :

https://github.com/CHIP-Specifications/chip-test-plans/blob/master/src/cluster/levelcontrol.adoc#tc-lvl-2-3-reporting-requirements-with-dut_server
Expected Behavior:
The first entry in reportedCurrentLevelValuesList should be larger than startCurrentLevel.
Actual Behavior:
The test case fails because the first entry in reportedCurrentLevelValuesList is equal to startCurrentLevel.
Reproduction steps
Command used to launch the DUT : ./chip-all-clusters-app
Command used to run the python script : python3 TC_LVL_2_3.py --commissioning-method on-network --discriminator 3840 --passcode 20202021 --storage-path admin_storage.json --paa-trust-store-path ../../credentials/development/paa-root-certs/

[MatterTest] 07-31 06:58:51.073 INFO 		***** Test Step 11 : If reportedCurrentLevelValuesList contains two or more entries, TH verifies the value of the first entry is larger than startCurrentLevel
[MatterTest] 07-31 06:58:51.076 ERROR Exception occurred in test_TC_LVL_2_3.
Traceback (most recent call last):
File "/home/ubuntu/Jul_Dut/connectedhomeip/no/lib/python3.10/site-packages/mobly/base_test.py", line 783, in exec_one_test
test_method()
File "/home/ubuntu/Jul_Dut/connectedhomeip/src/python_testing/matter_testing_support.py", line 1837, in per_endpoint_runner
_async_runner(body, self, *args, **kwargs)
File "/home/ubuntu/Jul_Dut/connectedhomeip/src/python_testing/matter_testing_support.py", line 1645, in _async_runner
return asyncio.run(runner_with_timeout)
File "/usr/lib/python3.10/asyncio/runners.py", line 44, in run
return loop.run_until_complete(main)
File "/usr/lib/python3.10/asyncio/base_events.py", line 649, in run_until_complete
return future.result()
File "/usr/lib/python3.10/asyncio/tasks.py", line 445, in wait_for
return fut.result()
File "/home/ubuntu/Jul_Dut/connectedhomeip/src/python_testing/TC_LVL_2_3.py", line 135, in test_TC_LVL_2_3
asserts.assert_greater_equal(e.value, last_value, "Values are not increasing")
File "/home/ubuntu/Jul_Dut/connectedhomeip/no/lib/python3.10/site-packages/mobly/asserts.py", line 236, in assert_greater_equal
_call_unittest_assertion(_pyunit_proxy.assertGreaterEqual,
File "/home/ubuntu/Jul_Dut/connectedhomeip/no/lib/python3.10/site-packages/mobly/asserts.py", line 52, in _call_unittest_assertion
raise signals.TestFailure(my_msg, extras=extras)
mobly.signals.TestFailure: Details=1 not greater than or equal to 5 Values are not increasing, Extras=None
[MatterTest] 07-31 06:58:51.091 INFO 		***** Test Failure :
[MatterTest] 07-31 06:58:51.091 INFO Finished test in 40803913.0ms
[MatterTest] 07-31 06:58:51.092 INFO [Test] test_TC_LVL_2_3 FAIL
[MatterTest] 07-31 06:58:51.104 INFO Summary for test class TC_LVL_2_3: Error 0, Executed 1, Failed 1, Passed 0, Requested 1, Skipped 0
[MatterTest] 07-31 06:58:51.106 INFO Summary for test run MatterTest@07-31-2024_06-58-08-503:
Total time elapsed 42.594418749038596s
Artifacts are saved in "/tmp/matter_testing/logs/MatterTest/07-31-2024_06-58-08-503"
Test summary saved in "/tmp/matter_testing/logs/MatterTest/07-31-2024_06-58-08-503/test_summary.yaml"
Test results: Error 0, Executed 2, Failed 1, Passed 1, Requested 2, Skipped 0

Bug prevalence
Everytime
GitHub hash of the SDK that was being used
86011093e730609a7dca462f8cc0dcf278019c56
Platform
raspi
PFA log Below :
TC-LVL-2.3.txt

Test case is passing in latest Master TOT, hence closing this issue
TC-LVL-2.3.txt


Media items do not inherit ACLs from collections

When ACLs are set on collections, they have not been imported as part of the SMS ingest procedure. Example: https://alpha.media.cam.ac.uk/media/fORYV3X7FIo belongs to the original SMS collection  https://sms.cam.ac.uk/collection/2665516 which has an ACL set to the lookup groupd "002258". This ACL has not been applied to the media item.

The work to fix this is to, when syncing SMS media items:

If the SMS media item has an ACL set, use it to set the media item view permission
If there is no ACL set on the SMS media item, and there is an ACL on the SMS collection use the collection ACL to set the media item view permission.
Otherwise, the media item view permission is reset. This should never happen in practice since all media items should have some visibility set through the visibility, ACL or collection ACL.

Fixed in uisautomation/sms2jwplayer@23b1217 and confirmed to fix the problem in the test instance. Should fix itself next time production syncs.