Feature/support transaction

Closes #64
Closes #78

Codecov Report
Base: 93.32% // Head: 93.20% // Decreases project coverage by -0.13% :warning:

Coverage data is based on head (cc40424) compared to base (1c26a8c).
Patch coverage: 52.38% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #79      +/-   ##
==========================================
- Coverage   93.32%   93.20%   -0.13%
==========================================
  Files          76       77       +1
  Lines        4555     4576      +21
  Branches      422      422
==========================================
+ Hits         4251     4265      +14
- Misses        181      189       +8
+ Partials      123      122       -1

Impacted Files
Coverage Δ

src/NRedisStack/Transactions.cs
52.38% <52.38%> (ø)

src/NRedisStack/Search/Document.cs
86.00% <0.00%> (+6.00%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Terminal throws winpty.dll error and not responding

Whenever I try open it (no matter what shell I use) it gives exception from winpty.dll and terminal is not usable -
File: ..\..\..\deps\winpty\libwinpty\winpty.cc
Line: 194
Expression: "success && actual == sizeof(int32_t)

I'm getting this issue as well.

Yeah need help with this one too.

Don't know how, but installing and rebuilding @2.3.1 (I also tried some previous versions to check if the error was introduced in recent builds, I think 2.5.3 and 2.5.1) and then latest version fixed the issue. I've got this error after updating to<EMAIL_ADDRESS>

same here....

any updates on this ? It's a very annoying issue

You can also try with a fresh atom configuration as described in the debugging page for atom.
Or  by disabling non core packages one by one testing in between and close reopen a terminal to see if issue persists, until the issue is gone, or you run out of external extension to disable,
If the issue still persists after all that, youre SOL,
These me too reports that raise a year old report and dont provide ANY new information about OS being used, Atom versions or anything useful are the nightmare for anyone trying to help.

Please retest again with platformio-atom-ide-terminal 2.9.0 is using node-pty-prebuilt instead unmaintained and outdated pty.js


ACR: keep uploads with BinaryData only

We can offer an option to make content replayable in future, for now defaulting to replayable content only.

/azp run java - containerregistry - tests

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


Fixes to Argo Workflow Reference document

Change Overview
Fixed CR/CRDs and removed the part to submit workflows.
Pull request type
Please check the type of change your PR introduces:

[ ] :construction: Work in Progress
[ ] :rainbow: Refactoring (no functional changes, no api changes)
[ ] :hamster: Trivial/Minor
[X] :bug: Bugfix
[ ] :sunflower: Feature
[ ] :world_map: Documentation
[ ] :robot: Test

Issues

fixes #issue-number

Test Plan

[ ] :muscle: Manual
[ ] :zap: Unit test
[ ] :green_heart: E2E

Can merge once CI finished.


Handle initial project for embedded editor

Initial project code now handles case where editor is embedded. File system is not initialised until content from host is received in this case.
See #236.

@microbit-matt-hillsdon I've removed a couple of unused imports in host.test.ts. This looks good to me.


Twig 2.13.1 introduces a third parameter for twig_array_filter()

This issue was raised in https://wordpress.org/support/topic/args-passed-to-twig_array_filter-must-be-an-instance-of-twigenvironment/.
In this commit, they changed the parameters from
function twig_array_filter($array, $arrow) {}

to
function twig_array_filter(Environment $env, $array, $arrow) {}

Yes, they prepended a parameter. This now conflicts with the patch we introduced in Timber 1.14.0, in #2124 to be exact. The tricky thing would be to get the Twig environment in there. Or maybe we need to take a completely different approach.
@palmiak Any idea how we could solve this?

Prepending the parameter was a great idea...
TBH - the only thing that comes to my mind would be to check Twig version and based on it use 2 or 3 params. I think it sucks a bit, but can't think of anything better.

Ugggggggh. I think we might want to consider splitting the solution between 1.x and 2.x here. My proposal would suggest ...
Version 1.x

Introduce a kludge like what @palmiak suggests. Ugly but workable. BUT what about the Environment? is that available to the static function Helper::filter_array()? I don't believe it is. Hopefully it still performs if sent a null value.
If the above is not true (it fails when sent a null value for $env) — we'll need to issue some kind of error or warning when Helper::filter_array() is called with Twig >= 2.13.1

Version 2.x

Assume/require that Twig 2.13.1+ as the minimum version and go to 3 args. BUT what about the Environment? is that available to the static function Helper::filter_array()? I don't believe it is. Hopefully it still performs if sent a null value.
If the above is not true, I don't think it's worth doing heroics. Better instead to remove and require that users rely on either custom or native Twig filters for similar behaviors.

Update: just ran a test and sending null does NOT work. Without going through the history of that commit/issue, it's hard to understand why this breaking change suddenly landed in the code?
However, the good news is that we can bypass the call to Twig in the first place by replacing the line to ...
return array_filter( $list, $arrow, \ARRAY_FILTER_USE_BOTH );

In looking at the function definition I don't see what calling this function in Twig is really giving us besides the cleanliness of going through Twig (and some extra checks, etc. in there)

Version 2.x
1. Assume/require that Twig 2.13.1+ as the minimum version and go to 3 args. BUT what about the `Environment`? is that available to the static function `Helper::filter_array()`? I don't believe it is. Hopefully it still performs if sent a null value.

2. If the above is not true, I don't think it's worth doing heroics. Better instead to remove and require that users rely on either custom or native Twig filters for similar behaviors.

I thought we were going to remove Timber’s |filter anyways:
https://github.com/timber/timber/blob/ab8ae9e96a27127fca663a0546db0d32c72f766c/lib/Twig.php#L219-L225
As I see it, the remaining Todos for 2.x are to remove it and add it to the upgrade guide in https://timber.github.io/docs/v2/upgrade-guides/2.0/#deprecated-twig-filters.

However, the good news is that we can bypass the call to Twig in the first place by replacing the line to ...
return array_filter( $list, $arrow, \ARRAY_FILTER_USE_BOTH );

I think that’s a good approach and an elegant solution 👍 .

@gchtr good call, I had forgotten that it was slated for removal. So for now, let's bring this into 1.x and then make sure to give things the final 🔪  in 2.x!


Retry certificate request in case of failure

This adds retry logic for the certificate request in case of failure.
This addresses intermittent issues, such as limited network connectivity.

Still testing this.


Job created but doesn't progress when pipeline created

Provide a link to the page with an issue:
Describe what is wrong:
https://docs.pachyderm.com/latest/troubleshooting/pipeline_troubleshooting/#all-your-pods-or-jobs-get-evicted
Should include new information about how to find evicted pods, which is a common problem.
Describe the suggested changes:
The symptom as described in the attached ticket is that

After creating a pipeline a job started but never progressed through any datums. After deleting the pipeline and recreating the new job made it through 9 datums and froze.

This is how to diagnose if your pipeline is evicted, before it gets this symptom. I think this should be added as another troubleshooting step: "Your job starts but doesn't progress"
and the step should be to check for pod eviction using kubectl get pods.  Then perhaps point them to the pods getting evicted troubleshooting step above.
Anything else we need to know?
Environment:

Kubernetes version (use kubectl version):
Pachyderm CLI and pachd server version (use pachctl version):
Cloud provider (e.g. aws, azure, gke) or local deployment (e.g. minikube vs dockerized k8s):
OS:
Other:
(created from Zendesk ticket #54)gz#54

Agent comment from John Karabaic in Zendesk ticket #54:If you get a reproducible case around this, I'd like to get an issue created.

Agent comment from John Karabaic in Zendesk ticket #54:Have you ever reproduced this issue?

Agent comment from John Karabaic in Zendesk ticket #54:Going to close this.  If you ever get a reproducible case, we can reopen.


[TRTLLM Python backend]Fix the output format for client side batching in dynamic batch

Description
If the user sends, client side batch then the result is json array. If the user sends single input, then we send back json object.
Examples:
Request : "inputs":["The new movie that got Oscar this year"
Response :
[
  {
    "generated_text":"The new movie that got Oscar this year is The Shape of Water."
  }
]

Request: "inputs":"The new movie that got Oscar this year"
Response:
{
  "generated_text":"The new movie that got Oscar this year is The Shape of Water."
}

do we want to support client side batching? Maybe just server side is good enough

do we want to support client side batching? Maybe just server side is good enough

We already do support. This PR just addresses the output format. In our CI, we also test client side batching. Today's CI would fail because the format is wrong.


Still unable to use WHIP

v0.12.6, Ubuntu
Describe the bug
Throws the following error: [2024-04-28T00:25:45Z ERROR xwebrtc::webrtc] session run error, err: Utf8Error: invalid utf-8 sequence of 1 bytes from index 11
No matter what URL is supplied in OBS (with/without token, specifying /whip or not) it always throws Utf8Error
To Reproduce
Use any URL pointing to a xiu server in OBS (WHIP)
Expected behavior
Accepts incoming request from OBS

The error occurs for this line: https://github.com/harlanc/xiu/blob/7df491b69215e2182e8cca2748c14b2d75de88b5/protocol/webrtc/src/session/mod.rs#L97
...indicating that whatever data ends up in reader is not Utf8. I have not supplied any non-Utf8 text in my OBS settings, so it's possible that OBS is injecting some non-Utf8 characters or somewhere along the way the request is mutated by xiu to contain non-Utf8 characters

A similar problem mentioned here : https://github.com/harlanc/xiu/issues/82#issuecomment-1868472390.
Or refer to the doc https://www.rustxiu.com/docs/scenarios/webrtc ？

There is not sufficient information in the docs for me to troubleshoot with. An example push URL is provided in the form of http://localhost:8900/whip?app=live&stream=test&token=123 but there is no explanation of where to set app or whether token goes into the URL field or Bearer Token field in OBS. Is /whip always the served path? Do WHEP listeners really need the token too/is there no way to require authentication for only WHIP?

[2024-06-07T10:10:18Z INFO  xwebrtc::session] 1:whep,2:live,3:test
[2024-06-07T10:10:18Z INFO  streamhub] event data: {"Subscribe":{"identifier":{"WebRTC":{"app_name":"live","stream_name":"test"}},"info":{"id":"1717755018","sub_type":"PlayerWebrtc","notify_info":{"request_url":"","remote_addr":""}}}}
[2024-06-07T10:10:18Z ERROR streamhub] event_loop Subscribe error: no app or stream name
[2024-06-07T10:10:18Z ERROR xwebrtc::webrtc] session run error, err: event execute error: no app or stream nam

18:14:30.831: [FFmpeg libopus encoder: 'simple_opus'] bitrate: 160, channels: 2, channel_layout: stereo
18:14:30.831:
18:14:30.834: [obs-webrtc] [whip_output: 'simple_stream'] PeerConnection state is now: Connecting
18:14:31.097: [obs-webrtc] [whip_output: 'simple_stream'] PeerConnection state is now: Failed
18:14:31.097: Output 'simple_stream': stopping
18:14:31.097: Output 'simple_stream': Total frames output: 0
18:14:31.097: Output 'simple_stream': Total drawn frames: 8
18:14:31.097: [obs-webrtc] [whip_output: 'simple_stream'] PeerConnection state is now: Closed

webrtc::peer_connection::peer_connection_internal] Failed to start manager dtls: Alert is Fatal or Close Notify


Enable GraphQL Subscriptions

Due to how Serverless interfaces with API Gateway, maintaining subscriptions for GraphQL does not work out of the box and requires some additional configuration to get it working. @reconbot created graphql-lambda-subscriptions to solve this problem, but it does require DynamoDB which not everyone is interested in doing.
This package uses Apollo Server 3, which disables subscriptions out of the box and recommends using a custom PubSub engine if you are to enable subscriptions. Given the info available, I'm going to look into either a Redis or SQS style solution for this problem.

I think the last time I explored this, we ended up forking your project and shoving redis in place of ddb. I'd be curious what you think when I get something open for this. 😄

👀 well the next time I pick things up, let me give that a shot 😄


Hide simulator_state

The simulator state for simulators is an internal state used by the implementations for simulating state when stepping through a simulation.  It shouldn't be exposed "publically".  See #1658 for where callers where using this state instead of going to the public methods on step result or trial result.

Fixes #1658

I was using qubit_map to pass to cirq.pauli_string_expectation. Seems sortof important to have this tagging along (in a public way) when implicitly labeling qubits by e.g. row index. Is there a preferred workaround?

we should expose it in the result methods.


Fixing ext_proc integration test timeout issue.

This is to address https://github.com/envoyproxy/envoy/issues/28568.
https://github.com/envoyproxy/envoy/pull/27812 added a streaming body test in ext_proc_integration_test.cc, which is sending 100 chunks of body. This might cause the upstream timeout.    It is not needed to send that many chunks. Change it into send 10 chunks.

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/assign @phlax

Removing that trouble test cases for now.


display bug, not showing any titles only countdown with remaining time

as shown by both pics detection seems to be working fine but the display in discord is not, attached logs in zip format as well, hopefully you can get down to root of this issue
discordrichpresencelogs.zip

Duplicate of #58

Closed as duplicate of #58


Request for Weight Files in faceswap-GAN Project

I am currently working on a project utilizing the faceswap-GAN framework. I have been exploring the provided codebase and have encountered the need for the weight files associated with the following lines of code:
encoder.load_weights("models/encoder.h5") decoder_A.load_weights("models/decoder_A.h5") decoder_B.load_weights("models/decoder_B.h5") Unfortunately, I couldn't locate these weight files in the project repository or documentation. I understand the importance of these weights in the functioning of the model, and I was wondering if you could kindly provide or direct me to the appropriate source for obtaining these files.
Dear shaoanlu, your assistance in this matter would be greatly appreciated, as it would significantly contribute to the progress of my project. I am committed to adhering to any terms of use or licensing associated with the weight files.
Thank you for your time and consideration. I look forward to your guidance on this matter.


Add support for Laravel 8

I am unaware of any changes to models that require any modifications, but if there are some please let me know.
I tested this in a sample project, but my main project that relies heavily on JIT relationships can't be upgraded to Laravel 8 yet. I'm waiting for a few more packages still (which I am PRing today, so hopefully soon). If you want to hold off on a release until I test it fully on my project, that's fine. I can use dev-master until then.

Hi Liam,
Could you take a moment to look this over? I've had a chance to test this for a few days and everything seems to be fine. JIT relationships are still being logged as they were in Laravel 7 and I have had no errors related to this package.


PipelineRun delete workspace volume after completion

Feature request
We are using the Pipeline Workspace feature to share data among Tasks in a Pipeline. When attaching a volumeClaimTemplate to a PipelineRun the volume will only be deleted after removal of the PipelineRun. This is the current behaviour as documented here: https://github.com/tektoncd/pipeline/blob/3e906480a1c2907b97fc6cf7f7bd7a6c75bf2856/docs/workspaces.md#using-persistentvolumeclaims-as-volumesource
However this behaviour might not be wanted everytime. Keeping PipelineRuns resources as a reference is great, but keeping volumes with them might be not desirable. Therefore I would like to request the option to delete the volume that has been declared via volumeClaimTemplate automatically after the PipelineRun is finished (no matter if successful or failed).
Use case
I think of use cases where users want to keep PipelineRun resources to check if the latest runs are successful or with which parameter values a pipeline was triggered, but are actually not interested in the volume data as it was only used for sharing data between Tasks / speeding up build time at Pipeline runtime. Keeping the volumes would also mean occupying resource quota and space resources that might be needed elsewhere.

FWIW, I would really like this feature also. I currently am working on pipelines that use local storage for the PVC to optimize build times, which means I have to manually go in and delete pipelineRuns to release volume claims and prevent them all from being claimed since there are only a small number of local volumes attached.

+1


Type error for data attributes in attr (v6)

Environment
## Binaries:
 - Node: 16.15.0 - ~/.nvm/versions/node/v16.15.0/bin/node
 - Yarn: 1.22.19 - ~/.yarn/bin/yarn
 - npm: 8.5.5 - ~/.nvm/versions/node/v16.15.0/bin/npm
## npmPackages:
 - babel-plugin-styled-components: ^2.1.4 => 2.1.4
 - styled-components: ^6.0.7 => 6.0.7

Reproduction
const Box1 = styled.div`
  color: red;
`;

const Box2 = styled.div.attrs({
  'data-testid': 'box-test',
  //  ^ Argument of type '{ 'data-testid': string; }' is not assignable to parameter of type 'Attrs<FastOmit<DetailedHTMLProps<HTMLAttributes<HTMLDivElement>, HTMLDivElement>, never>>'.
  //       Object literal may only specify known properties, and ''data-testid'' does not exist in type 'Attrs<FastOmit<DetailedHTMLProps<HTMLAttributes<HTMLDivElement>, HTMLDivElement>, never>>'.ts(2345)
})`
  color: red;
`;

const test = <div data-testid="foo" />;

const test1A = <Box1 />;
const test1B = <Box1 data-testid="foo" />;

const test2A = <Box2 />;
const test2B = <Box2 data-testid="foo" />;

Expected Behavior
As data attributes are accepted as valid types when using the styled-component via JSX, this should probably also apply to the attrs usage as well.
Actual Behavior
Type error given for data attributes;
Argument of type '{ 'data-testid': string; }' is not assignable to parameter of type 'Attrs<FastOmit<DetailedHTMLProps<HTMLAttributes<HTMLDivElement>, HTMLDivElement>, never>>'.
  Object literal may only specify known properties, and ''data-testid'' does not exist in type 'Attrs<FastOmit<DetailedHTMLProps<HTMLAttributes<HTMLDivElement>, HTMLDivElement>, never>>'.ts(2345)

Hi,
I would like to work on this issue, can you please assign this issue to me?

Just though I'd share the workaround I'm using in the interim, in case it's of use to others;
type DataAttributes = {
  [K in `data-${string}`]: string;
};

const Box = styled.div.attrs<DataAttributes>({
  'data-testid': 'box-test',
  'data-foo': 'bar',
})`
  color: red;
`;

Just though I'd share the workaround I'm using in the interim, in case it's of use to others;
type DataAttributes = {
  [K in `data-${string}`]: string;
};

const Box = styled.div.attrs<DataAttributes>({
  'data-testid': 'box-test',
  'data-foo': 'bar',
})`
  color: red;
`;

Incidentally we have that DataAttributes type in core as well, I'll expose it.

Is the intention to update Styled Components so this workaround isn't required on usage? It would seem odd to have to specify types for valid native HTML attributes like this every time.

Is the intention to update Styled Components so this workaround isn't required on usage? It would seem odd to have to specify types for valid native HTML attributes like this every time.

I agree with you, the type was auto-widening and causing issues when index types are added (allowed unspecified props without a type error.) Maybe a future version of TS or some trick I haven't thought of yet will solve for it.


Have kube-state-metrics installed as an addon

Currently, kube-state-metrics nor heapster give all the metrics like PersistentVolume Usage. Having kube-state-metrics by default will be a nice feature.

Thanks @prat0318. Definitely something that would add value, but not something we are looking to prioritize in the short term.
Is there anything that prevents you from deploying it manually after installing the cluster? I just want to make sure that we are not blocking you from doing it.

@prat0318 just out of interest, if you would like kube-state-metrics would you also like Prometheus and Grafana as a default?

@alexbrand nothing blocking.
@swade1987 that would be great. we currently install them via helm charts.

Got it. Thanks @prat0318

@alexbrand as we have agreed that KET isn't going to be responsible for adding workloads to the cluster this issue can be closed.

@prat0318 due to the statements above are you happy for this issue to be closed?

Agreed @swade1987. Closing  this out.


[master]zvmguestconfiugre supports multi-attach/detach at same time

when detach/attach multiple volumes for a vm at same time, zvmguestconfiugre
will be restart too often to work well, and when detaching volume with previous
code, first to update database, then to punch scripts to vm and restart zvmguestcongure,
that may cause dirty data in the cases of concurrent detachment/attchment.

So I update code to make the zvmguestconfigure will be restarted only after the
last punch and reorder the sequence to first punch all scripts and then to update
database

Signed-off-by: wgxoyun<EMAIL_ADDRESS>

Two changes included in this PR:

only restart zvmguestconfigure at the last FCP in attach/detach
move the _add_disk operation after dedicate_fcp in attach flow, and move the _remove_disk before _undedicate_fcp

we will split the PR into two to make this clear.

@dongyanyang please help to review the latest change again, thanks


[OracleDB] Sub Organization switch not available on tenants

Describe the issue:
[OracleDB] Sub Organization switch not available on tenants.

How to reproduce:

Login to management console
Create tenant
Login to console with tenanted path
Create sub oraganization
Switch to the sub organization

Expected behavior:
There should be option to switch to sub organization.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [e.g., IS 5.10.0, IS 5.9.0]
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

@aaujayasena let's verify this in the latest pack

https://github.com/wso2/product-is/assets/32576163/8efc5c47-0b5f-4121-8aae-38d7254a768b
Verified in the latest pack by connecting into an oracle database, the issue is no longer reproducible


Crash on startup

https://crashy.net/uI2vk4lXteRgw989AFKW

Caused by: java.lang.NoClassDefFoundError: io/github/prospector/modmenu/api/ModMenuApi

This is a bit odd, since the 1.18.1 version should have no more references to the old io.github.prospector API...

@the-unheard Can you reproduce the crash with fewer mods installed? I'm guessing some other mod might bundle a bad version of the Mod Menu API.
If you only install ModSettings, or only install ModSettings and Mod Menu -- does it still crash?

I can confirm the Crash with the following Mods, every Mod + Fabric + Minecraft latest version:

https://www.curseforge.com/minecraft/mc-mods/capes
https://www.curseforge.com/minecraft/mc-mods/eggtab-fabric
https://www.curseforge.com/minecraft/mc-mods/enhanced-attack-indicator
https://www.curseforge.com/minecraft/mc-mods/auto-config-updated-api

They all use Auto Config Updated API (the one Mod is it), which uses in the newest version the old Mod Menu API, on their GitHub Repo it is already fixed (https://github.com/shedaniel/AutoConfig/pull/32/files)

Ok, so even though the old API is deprecated, some mods still use it, and this causes a crash. I could change Mod Settings to just not crash, but that seems pointless, so I have restored support for the old API. Guess we'll actually be stuck with it forever...


[WIP] Add validation to BestBid Phase

BestBid Phase bid shouldn't be less or equal to SealedBid Phase max bid.

Waiting for validation message translation.

Coverage increased (+0.09%) to 82.806% when pulling 912f5560229e959c46e96e22a26b20c10a2c71cc on Scandie:a484807863547476_best_bid_validation into 99d65681c69574daf617d5e9eb88f6f079f30c6c on openprocurement:sandbox.

Coverage increased (+0.09%) to 82.806% when pulling 8423d2ee27a9d7dd5259af2802d61f96ffba7afe on Scandie:a484807863547476_best_bid_validation into 5cd8deba7fec900caaecaebffc71f30232066568 on openprocurement:sandbox.


implements system-out and system-err output formating using example-metadata

This seems like a very specialised function. Does rspec implement capture like this in a standard way — are these metadata keys conventional?

Hey Samuel, thanks for checking my pull-request. Unfortunately, there is no such convention in Rspec, at least I didn't found anything that suited my needs. Before you close the pull request for that reason, let me explain why I did implement it.
We have a very long running test suite on Jenkins, which produces a lot of test output. We used to grep through the logs after Jenkins indicated a failure. This process was/is cumbersome, maybe you have made the same experience.  Because of that, I created this small hook capability, that will allow everyone that has similar issues, to put the output of stderr and stdout right next to the test results in Jenkins. I think the naming of the metadata keys is reasonable.
Let me know what you think. Threez

I quite like this, on review. Let's do it!


Roadmap

@react-dnd/gatekeepers - Should we start discussing on the road ahead?

Review and close issues which are just how-to / help-me types and prioritize on the ones which really needs to be fixed
Look into the PR's
Any feature improvements planned we want to take up
Also, an ongoing discussion about adding localization support for docs #679

Sure, maybe sometime in the next week we can have a discussion on Discord. We can set up some milestones and work through bugs; I don't think we need to big feature development, but i18n of the docs is something that would be compelling


Code Highlighting not working

What Happened?
The code highlighting is not working.
What Did You Expect to Happen
In the project site, pages includes some code of any language. It's expected to have code in language specific highlighting. Earlier when I visited this site, I saw code highlighting, but now when I visited, code highlighting was not there. It should be there for more understanding and good UI.
Screenshot

@PatilShreyas I have also noticed this is the case but only on mobile ... not sure what the difference is.

I spoke too soon.  Seems to be broken on desktop now too

Yes... I see..
I noticed this change long time ago, but raised issue now.
I think something happened wrong in recent PR.


[staging-next] calibre: fix build on staging-next due to missing setuptools

Motivation for this change
Targetting #152145
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Why so much timeout errors on ofborg?

No idea, :o

@fabianhjr got you fam


Envoy running short lived WebSocket load starts returning codes 503

Envoy running short lived WebSocket load starts returning codes 503
Description:
I have a WebSocket application using Envoy as the routing load balancer tool, and I'm doing some load tests to it to see how it behaves under load and what to expect on load scenarios from it. On high load with a persistent connection, I mean streaming messages through the same opened connection for 3 minutes I'm able to reach the 100% CPU in Envoy and measure the limits of requests I can handle but when I change to open the connection send a message and then after receiving an answer closing the connection, I start receiving errors 503 really fast instead of the expected 101 while trying to open new connections.
Repro steps:
Using the following k6 script:
import ws from "k6/ws";
import { check } from "k6";

const msg = "test message";

export default function () {
  const url = __ENV.WS_URL || "ws://localhost:8080/chat";
  const params = { headers: { Host: "example.com" } };

  const res = ws.connect(url, params, function (socket) {
    socket.on("open", function open() {
      socket.send(msg);
    });
    socket.on("message", (data) => {
      socket.close();
    });
    socket.on("error", (e) => {
      if (e.error() != "websocket: close sent") {
        console.error("An unexpected error occurred: ", e.error());
      }
    });
  });

  const result = check(res, { "status is 101": (r) => r && r.status === 101 });
  if (!result) {
    console.error(`Check failed: status ${res.status}, body ${res.body}`);
  }
}

Running it like this:
k6 run --vus 10 --duration 180s -e WS_URL=ws://mycompany/chat script.js

Starts returning 503 errors:

          /\      |‾‾| /‾‾/   /‾‾/
     /\  /  \     |  |/  /   /  /
    /  \/    \    |     (   /   ‾‾\
   /          \   |  |\  \ |  (‾)  |
  / __________ \  |__| \__\ \_____/ .io

     execution: local
        script: script-simple.js
        output: -

     scenarios: (100.00%) 1 scenario, 10 max VUs, 3m30s max duration (incl. graceful stop):
              * default: 10 looping VUs for 3m0s (gracefulStop: 30s)

ERRO[0006] Check failed: status 503, body upstream connect error or disconnect/reset before headers. reset reason: connection timeout  source=console
ERRO[0006] Check failed: status 503, body upstream connect error or disconnect/reset before headers. reset reason: connection timeout  source=console
ERRO[0006] Check failed: status 503, body upstream connect error or disconnect/reset before headers. reset reason: connection timeout  source=console
ERRO[0006] Check failed: status 503, body upstream connect error or disconnect/reset before headers. reset reason: connection timeout  source=console
......

Looking at a tcp Dump in Envoy we can see this happening while there are some retransmission errors on closing the websocket connection:
Dump against the client

Dump against the backend

This is how a successful websocket call looks like against the back:

Config:
The config is just using the router filter and enables the websocket upgrade
....
            "http_filters": [
             {
              "name": "envoy.filters.http.router",
              "typed_config": {
               "@type": "type.googleapis.com/envoy.extensions.filters.http.router.v3.Router"
              }
             }
            ],
....

....
            "upgrade_configs": [
             {
              "upgrade_type": "websocket"
             }
            ],
....

Logs:
[2024-09-19 20:58:36.656][3612][debug][conn_handler] [source/common/listener_manager/active_tcp_listener.cc:160] [Tags: "ConnectionId":"11628"] new connection from <IP_ADDRESS>:56980
[2024-09-19 20:58:37.363][3614][debug][pool] [source/common/conn_pool/conn_pool_base.cc:803] [Tags: "ConnectionId":"10029"] connect timeout
[2024-09-19 20:58:37.363][3614][debug][connection] [source/common/network/connection_impl.cc:149] [Tags: "ConnectionId":"10029"] closing data_to_write=0 type=1
[2024-09-19 20:58:37.363][3614][debug][connection] [source/common/network/connection_impl.cc:281] [Tags: "ConnectionId":"10029"] closing socket: 1
[2024-09-19 20:58:37.364][3614][debug][client] [source/common/http/codec_client.cc:107] [Tags: "ConnectionId":"10029"] disconnect. resetting 0 pending requests
[2024-09-19 20:58:37.364][3614][debug][pool] [source/common/conn_pool/conn_pool_base.cc:495] [Tags: "ConnectionId":"10029"] client disconnected, failure reason:
[2024-09-19 20:58:37.364][3614][debug][router] [source/common/router/router.cc:1354] [Tags: "ConnectionId":"11612","StreamId":"8531618248070408344"] upstream reset: reset reason: connection timeout, transport failure reason:
[2024-09-19 20:58:37.364][3614][debug][http] [source/common/http/filter_manager.cc:1077] [Tags: "ConnectionId":"11612","StreamId":"8531618248070408344"] Sending local reply with details upstream_reset_before_response_started{connection_timeout}
[2024-09-19 20:58:37.364][3614][debug][http] [source/common/http/conn_manager_impl.cc:1838] [Tags: "ConnectionId":"11612","StreamId":"8531618248070408344"] encoding headers via codec (end_stream=false):
':status', '503'
'content-length', '91'
'content-type', 'text/plain'
'date', 'Thu, 19 Sep 2024 20:58:36 GMT'
'server', 'Microsoft-Azure-Application-LB/AGC'
'connection', 'close'

[2024-09-19 20:58:37.364][3614][debug][http] [source/common/http/conn_manager_impl.cc:1950] [Tags: "ConnectionId":"11612","StreamId":"8531618248070408344"] Codec completed encoding stream.
[2024-09-19 20:58:37.364][3614][debug][http] [source/common/http/conn_manager_impl.cc:252] [Tags: "ConnectionId":"11612","StreamId":"8531618248070408344"] doEndStream() resetting stream
[2024-09-19 20:58:37.364][3614][debug][http] [source/common/http/conn_manager_impl.cc:1907] [Tags: "ConnectionId":"11612","StreamId":"8531618248070408344"] stream reset: reset reason: local reset, response details: -

Is there something that I'm missing in my configuration? What can I do to prevent this from happening? Is this the normal expected behavior?
Thanks

There is a connection timeout configuration for the upstream cluster that might need to be adjusted. There are also default circuit breaker settings that often need to be adjusted. You can see if you're triggering the circuit breaker by looking at metrics.
Since you're using web sockets, connections cannot be reused between the client and Envoy and between Envoy and the upstream. When opening and closing connections quickly between the same pair of hosts (say, envoy and upstream), it's possible to exhaust the available ephemeral ports used for outbound connections (the combination of protocol/source-ip/source-port/destination-ip/destination-port must be unique and when they're all constant except the OS-chosen source-port, it's easy to run out). You can usually tell this is happening if netstat or similar tools show a large number of connections in close-wait or fin-wait stats when the errors start occurring.

I will check my connection timeout and the circuit breaker settings, Thank you so much


Use native Stackdriver Monitoring Protos

see discussion in #137. Use native protos at the point of translation instead of ushering them into an intermediary format.

I started looking at this in more detail, because of @knyar 's comments on #156. The separation of Metric from MetricDescriptor in the API protos complicates matters somewhat, because we must derive the descriptors from firehose data.
I think (based on an hour or so of poking around, so don't take this as gospel) that the correct route forward is to respect this separation. Handwaving proto-design-doc discussion follows!
We should create a stackdriver.MetricRegistry (or DescriptorRegistry, or whatever) to encapsulate the creation and management of MetricDescriptors. This:

Removes this responsibility from the stackdriver.MetricAdapter, so it is solely responsible for batching and posting the timeseries data to Stackdriver.
Makes the error handling in MetricAdapter cleaner still, because we no longer have two types of error to be surfacing (descriptor creation and timeseries creation).

The Metric Type is the shared key tying a Metric to its Descriptor in the API; we can use that in the MetricRegistry for deduping and caching as is done in the MetricAdapter today. I don't think anything more than:
type MetricRegistry interface {
    // Checks descriptor.Type against internal cache
    // Returns error if cached and descriptors don't match
    // Creates if not cached
    // Adds to cache if successful
    // Returns error on failure
    // May not even need created bool, but probably useful for testing.
    Register(descriptor *metricpb.MetricDescriptor) (created bool, err error)
}

is needed, but I have not done enough due diligence to be 100% sure. An alternative API might create separate Register functions per MetricKind or maybe MetricKind+ValueType and take proto elements as args instead:
type MetricRegistry interface {
    RegisterGauge(metricType string, valueType MetricDescriptor_ValueType, labels []*LabelDescriptor,  otherStuff ...string) (created bool, err error)
    // ... or ...
    RegisterInt64Gauge(metricType string, labels []*LabelDescriptor, otherStuff ...string) (created bool, err error)
    // ... etc, etc.
}

I'm leaning towards the former since it allows more flexibility and is smaller, though it does push ~all of the creation code down into the metric sink. We could expose package-level functions outside of the interface to make some of the proto translation less of a chore, instead of having them in the registry interface.

The registry should populate its cache with all the currently-configured custom metric descriptors at nozzle startup so we can do client-side diffing, provide more meaningful errors, and possibly even forcibly override descriptors.

The nozzle's metric sink can derive both a metricpb.MetricDescriptor and a *monitoringpb.TimeSeries from the firehose event, call metricRegistry.Register() with the descriptor, then pass the timeseries off to the metricAdapter for deduping / batching / writing to SD. The metric pipeline can dedupe using the *metricpb.Metric proto from the timeseries, since that contains exactly the name and labelset we want to dedupe on.

Creating the TimeSeries in the metric sink also allows the possibility of creating MonitoredResource data for e.g. vm instances or container instances, if we can derive them in a consistent fashion.

Again, this is all very much handwaving and I'd love to hear your opinions on it!

The logging side of this change looks less controversial. Creating the Entry in the log sink allows us to potentially put more detail into the logs: Entry has an HttpRequest field which we could (partially) fill out from HttpStartStop events, for example. There's also the potential of filling out some of the Operation and Resource structs.
One key point is that we have to break the top level dictum that "a firehose event is either a metric or a log, it can't be both", since we already know from the MetricAdapter side channel that that is not true.

I'm all for the MetricRegistry and think the single Register method is the way to go.
From my point of view the MetricAdapter is responsible for translating between any internal representations and the final Stackdriver representation so it's appropriate for it to create a monitoringpb.MetricDescriptor. It would be reasonable to extract the specific Metric/MetricEvent -> MetricDescriptor translation into something like:
type MetricDescriptorAdapter interface {
    // Translate provides a monitoringpb.MetricDescriptor for a given messages.Metric.
    // The MetricDescriptor is unique for the name and labels on a given Metric but does not
    // depend on the specific value.
    Translate(messages.Metric) (monitoringpb.MetricDescriptor, error)
}

That way we can encapsulate caching and whatnot. I believe it's what you're eluding to as the package level function for translation.

Now breaking "a firehose event is either a metric or a log". I do think we still need an intermediary form from Firehose to Stackdriver given the different representations in Logging/Monitoring. But maybe we can drop messages.Log and have a single struct that represents both, goes through the exact same pipeline, and relies on MetricRouter to send it to the appropriate endpoint.

The whole point of me recommending this change was that there should not be any internal representations ;-)
As I said in #137, the translation to Stackdriver representation should happen at the point of entry to the nozzle, because the data model Stackdriver uses is a known-good way of representing timeseries, metrics and logs, and it's what you have to use eventually anyway.
You need to derive two things from a given events.Envelope in the MetricSink: (1) a MetricDescriptor, and (2) a TimeSeries (with a Metric inside it). If you want to keep MetricAdapter around and have the functions that perform these transforms in there rather than in MetricSink, I think that could work.

Descriptor registry behaviour discussed at length above.
Behaviour of "Metric Buffer" does not change.
"Metric Batcher" is the other half of the current "Metric Adapter" when translation is moved out.
We don't need a deduplicating buffer or a request batcher in the logs pipeline because the Stackdriver client automatically does that for us.
The "Client" packages at this point are going to be relatively thin wrappers around the SD client libs, mostly existing to export success/error metrics.


How can i  disable drag and drop on specific events  ?

I add multiple event with different types, Now I want to disable drag and drops for some events

That cannot be done at the moment.
We can add it, something like passing to each event that should not drag: disableDrag: true.
It should be straightforward to implement this

I did it in from node module In Event file line no 308
onDrag={onDragEvent && item.data.type != "workingHours" && item.data.type !="timeOff" && this.onDragEvent}`


Conditional has a dodgy Applicative instance

Conditional has a dodgy Applicative instance: (<*>) should be associative, but it is not technically because predicates are combined using makeAndPredicate (not associative). This is not unsound because makeAndPredicate is morally associative, but it would still be good to fix this up. Probably, we should make the predicate field of Conditional into a MultiAnd (Predicate _); that would be properly associative and it might help with some other things (e.g. #1391).

If we decide to change the representation of Conditional, we'd need to do https://github.com/runtimeverification/haskell-backend/issues/1530 first.

@goodlyrottenapple suggests the following investigation: remove https://github.com/runtimeverification/haskell-backend/blob/9ea642db346c529b75cc1f7963360064c5092860/kore/src/Kore/Internal/Conditional.hs#L139 and see how many changes we'd need to do.
If the code can be refactored easily to use other functions instead of the applicative instance then we can go forward with this. If it turns out it creates some other nested issues then we need to reassess the problem.


Uri solution https://www.urionlinejudge.com.br/judge/es/problems/view/1234

Hey there!, Thanks for submitting a PR to this repo
Please complete the following checklist for your PR to be approved. Mark the items by adding an x.
Example:  - [ ] Mark me as completed
To mark: - [x] Mark me as completed

[x] My Pull Request adheres to the Contribution guidelines.
[x] I have added readme.md and solution files in the appropriate folder.
[x] The solution I have given is my own and not plagiarized in any form.
[x] My solution passes all the testcases.
[x] I have properly indented and documented my solution using comments.

If any of the above is unmarked, please make the necessary changes and then mark them so that you're PR can be approved and merged.

Add comments done

Rename folder done


fix issue #1574

Seems not fixed.
Tried:

Delete the integration entry.
readd yunmi.waterpuri.lx11 using ip and token.
sensor.yunmi_lx11_bb89_total_dissolved_solids_for_output_water was not generated at all. Only sensor.yunmi_lx11_bb89_total_dissolved_solids_for_input_water

Any ideas?

https://github.com/al-one/hass-xiaomi-miot/blob/b6a13c75675e9b8458a244f82c2a3a41cf57921a/custom_components/xiaomi_miot/core/device_customizes.py#L1815-L1817
It has been fixed through custom option.
Thanks.


Remove coveralls mentions

The stats haven't been updated since 2017 so showing a 96% coverage badge is quite deceptive.

This have been release as part of version 1.14.0


[Troopl] -> [Other]

Resource Name - Troopl
Troopl is a free portfolio builder for web developers and data scientists. It helps people get a portfolio of work live on a customisable URL in minutes, instead of spending weeks building one from scratch.
Link: www.troopl.com
Checklist:

[ X ] I have performed a self-review of submitted resource and its follows the guidelines of the project.

@yeniv is it a SASS project? are you going to charge users for the service?

Hey @Alaev 👋
In time, Troopl may become a SASS project as we're exploring giving users the option of upgrading to a PRO portfolio for additional functionality. But it's important to us that there's always be a free version — which is the only version right now.


Hammad

after finish the the about page with new footer and profile pic

after add the logo in the new footer


[Bug Report] table selection carries out the full selection meeting when the data is large.

Element UI version
2.4.7
OS/Browsers version
10.13.6 / chrome 69
Vue version
2.5.17
Reproduction Link
https://jsfiddle.net/rfbu5jc2/2/
Steps to reproduce
点击全选 => 取消全选
What is Expected?
等待时间很短
What is actually happening?
等待时间很长，因为全选是遍历表格元素执行toggleRowSelection函数，400条时相当于执行400次，能不能加上一个事件是全选事件

this is the new link
https://jsfiddle.net/rfbu5jc2/2/

Duplicate of #12697

@masongzhi toggleAllSelection 方法应该可以满足你的需求吧。https://jsfiddle.net/b5taLfjc/


Remove the cranelift features

Remove the cranelift features since they aren't really needed at all, and (according to some unscientific tests) they add quite some time on the runs of our coverage CI run.

Added!


Add cloud-init support for the network-config file.

This allows creating static and custom networking configurations.
Re-opening #270 with documentation added.

This should be rebased; all tests passing and documentation added. Hopefully this is gtg.

My question to the reviewers is if we want to go this path, and starting to add support for all the cloudinit schema.

My question to the reviewers is if we want to go this path, and starting to add support for all the cloudinit schema.

Not sure about this. On one hand, the network configuration seems to be a top-level element of the cloud-init schema, and it is important enough for being managed by the terraform/libvirt provider. On the other hand, this could open a can of worms where we would end up having all the possible cloud-init stuff...
My personal view would be to go for the network configuration, but nothing else.

My understanding is that there are only 3 files that can exist in the ISO?
meta-data
user-data
network-config
I don't think you need to add the entire cloud-init schema. At least allowing folks to place those 3 files avoids having to implement the schema and the user can source those files as they see fit (I use terraform templates).

I would really like if we improve also the tests coverage for this feature

https://github.com/dmacvicar/terraform-provider-libvirt/pull/267

Closing this pr because it will be adressed on https://github.com/dmacvicar/terraform-provider-libvirt/pull/410, were we allow the new field, were the user can add cloud init network data. Thank you for contributing and your time spend on this pr ( reviewer and contributors). Have a nice day


Requesting an installation video tutorial

Hello
Please send the installation video about the system, this is the most important requirement for us.
Pls İnstall video

Which issue are you having?

I think start this new server is much more difficult than before especially for people like me, so may i ask for an instruction video or some pictures to explain how to start this server properly. Thanks a lot

Are the new instructions better?

@Ochii For smooth and good performance, you should prepare a video for the installation that shows us the way to us as a develop pls

@Ochii For smooth and good performance, you should prepare a video for the installation that shows us the way to us as a develop pls

I'm not going to create a tutorial video anytime soon.
Which steps do you find difficult?

after the new method came out, I solved all the problems, if you think you're doing something wrong, it currently only needs two core components: Node.JS and MongoDB, it makes no sense to think on the other three components.
do it first before you ask, google is your friend. if the program has an error, remember get error message before it exits, then go step by step to solve it.

How can I set up a database and how can I install the game exactly as I can help you, or if you provide me with my server information you can install.
do you have the server information to install the complete launcher also have a special launcher ?

i am a foreigner，i cant understand total，if you install video ，thank you so much

@enesvtn You just have to install MongoDB (if you're not using the Docker images) to prepare the database, no further configuration should be required.
You can use the same CSO2 launcher in Ochii/cso2-launcher.
@haohu0406 Which language do you speak? Maybe someone can translate the guide to your language.

@Ochii chinese and a little english，i‘ll try to translate，thanks a lot


ClassNotFoundException: OrderOpcControllerCore

Expected behavior
Configuration page shows up.
Actual behavior

Log

Level
Channel
Message

DEBUG23:39:09
doctrine
SELECT name FROM pr_module WHERE active = 1

CRITICAL23:39:09
php
Fatal Error: Class 'OrderOpcControllerCore' not found

DEBUG23:39:09
doctrine
SELECT name FROM pr_module WHERE active = 1

CRITICAL23:39:09
request
Uncaught PHP Exception Symfony\Component\Debug\Exception\ClassNotFoundException: "Attempted to load class "OrderOpcControllerCore" from the global namespace. Did you forget a "use" statement?" at /home/u47320p43168/domains/domainname.nl/public_html/vimexx/override/controllers/front/OrderOpcController.php line 2

Steps to reproduce the behavior
Migrated to <IP_ADDRESS> using 1-click-upgrade, but deïnstalled all modules (by deleting 'modules' folder) and reïnstalled the modules from clean <IP_ADDRESS> install. Then tried to reïnstall newest version of Mollie (3.3.5) but get 'Attempted to load class "OrderOpcControllerCore" from the global namespace.
Did you forget a "use" statement?' when trying to access the configuration page of the module.
If I install the older version that I also used in prestashop 1.6, version 3.1.0, then configuration page shows up fine.
Environment

PrestaShop version: <IP_ADDRESS>
Mollie module version: 3.3.5
PHP version: 7.0.33

Release 3.2.0 also works, the issue starts to appear from 3.3.0 onwards.

Hi @Torrentus,
What versions did you upgrade from?
Have you also checked the /overrides folder for the OrderOpcController.php file? This sounds like a leftover from PrestaShop 1.6. PrestaShop 1.7 does not have this file.

OrderOpcControllerCore turned out to be remains from an older version of the MyParcel module. Deleted the files in the overrides folder and that solved this issue.

Great to hear! Good luck with the migration!

@Torrentus Hi, i have the same problem, can u tell mi where i have to delete files? best regards


Train pretrained faster RCNN with resnet101 FPN as backbone

I want to use pretrain faster rcnn of resnet101 FPN to train with my own dataset.
But, it fails to download pretrain faster-rcnn model of resnet101 FPN.
This is my code of model part:
backbone =torchvision.models.detection.backbone_utils.resnet_fpn_backbone('resnet101','False')
model_urls ={'resnet101_fpn':'https://download.pytorch.org/models/fasterrcnn_resnet101_fpn_coco-258fb6c6.pth'}
state_dict = load_state_dict_from_url(model_urls['resnet101_fpn'])
model = FasterRCNN(backbone,num_classes=2)
The error looks like this:
Downloading: "https://download.pytorch.org/models/resnet101-5d3b4d8f.pth" to /home/user/.cache/torch/checkpoints/resnet101-5d3b4d8f.pth
100%|###############################################################| 178728960/178728960 [00:07<00:00, 23374182.30it/s]
Downloading: "https://download.pytorch.org/models/fasterrcnn_resnet101_fpn_coco-258fb6c6.pth" to /home/user/.cache/torch/checkpoints/fasterrcnn_resnet101_fpn_coco-258fb6c6.pth
Traceback (most recent call last):
File "detect.py", line 59, in
state_dict = load_state_dict_from_url(model_urls['resnet101_fpn'])
File "/home/user/miniconda/envs/py36/lib/python3.6/site-packages/torch/hub.py", line 433, in load_state_dict_from_url
_download_url_to_file(url, cached_file, hash_prefix, progress=progress)
File "/home/user/miniconda/envs/py36/lib/python3.6/site-packages/torch/hub.py", line 349, in _download_url_to_file
u = urlopen(url)
File "/home/user/miniconda/envs/py36/lib/python3.6/urllib/request.py", line 223, in urlopen
return opener.open(url, data, timeout)
File "/home/user/miniconda/envs/py36/lib/python3.6/urllib/request.py", line 532, in open
response = meth(req, response)
File "/home/user/miniconda/envs/py36/lib/python3.6/urllib/request.py", line 642, in http_response
'http', request, response, code, msg, hdrs)
File "/home/user/miniconda/envs/py36/lib/python3.6/urllib/request.py", line 570, in error
return self._call_chain(*args)
File "/home/user/miniconda/envs/py36/lib/python3.6/urllib/request.py", line 504, in _call_chain
result = func(*args)
File "/home/user/miniconda/envs/py36/lib/python3.6/urllib/request.py", line 650, in http_error_default
raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 403: Forbidden

We don't yet provide pre-trained detection models with resnet101 as a backbone, so this error is expected.

@icesohelrana, I trained the faster_rcnn model with the resnet101 backbone on the COCO dataset. You can see the performance here.
@fmassa, let me know if this is useful in any way.

Can I use torchvision.models.detection.backbone_utils.resnet_fpn_backbone('resnet101',pretrained=True) to load the pretrained resnet101 backbone? Following the same tutorial, should I also remove the last 2 layers with something like modules = list(resnet_net.children())[:-2]? Thanks

@tazu786 I can confirm that I've been using this to get around the absence of pretrained resnet101 backbone, and it's working :ok_hand:

Hello everyone, is there a new way to train a pretrained faster RCNN with resnet101 FPN as backbone ?

@NathaneBerrebi not exactly sure what you mean by "new way":

the team is gradually adding new pretrained weights for classification architectures. So you can use those are pretrained backbone to build your faster rcnn architecture.
however, there is no plan yet to provide a faster rcnn with resnet101 as backbone, pretrained on COCO to the best of my knowledge

Also, you might be glad to know that the training recipes have been updated, so that should improve your performances once trained :+1:


Doesnt work well with Webview

@HarshilShah
In my VC(embeded in NavigationController) I have a UIWebView.
I set my VC as UIWebView.scrollView.delegate.
However in setting the below scrollview delegate method and the code in it, the VC still dismisses when Im scrolling dispite having content at the top to scroll to.
public func scrollViewDidScroll(_ scrollView: UIScrollView) {
        if let delegate = transitioningDelegate as? DeckTransitioningDelegate {
            if scrollView.contentOffset.y > 0 {
                // Normal behaviour if the `scrollView` isn't scrolled to the top
                scrollView.bounces = true
                delegate.isDismissEnabled = false
            } else {
                if scrollView.isDecelerating {
                    // If the `scrollView` is scrolled to the top but is decelerating
                    // that means a swipe has been performed. The view and
                    // scrollviewʼs subviews are both translated in response to this.
                    view.transform = CGAffineTransform(translationX: 0, y: -scrollView.contentOffset.y)
                    scrollView.subviews.forEach {
                        $0.transform = CGAffineTransform(translationX: 0, y: scrollView.contentOffset.y)
                    }
                } else {
                    // If the user has panned to the top, the scrollview doesnʼt bounce and
                    // the dismiss gesture is enabled.
                    scrollView.bounces = false
                    delegate.isDismissEnabled = true
                }
            }
        }
    }```

Could you possibly upload a demo project showing your issue? Thanks

Closing due to inactivity. Feel free to reopen with more repro details


go.mod needs bumping

I noticed simpleblob go.mod is at 1.18.
Really this needs to be bumped in-line with Go's Release Policy (support for current plus two prior).
Seeing as we are on 1.21, go.mod should be bumped to 1.19.
Any reason not to ?
P.S. Also you might want to consider removing currently... from README.md otherwise you will be forever lagging behind. i.e. change We support the last two stable Go versions, currently 1.17 and 1.18. to We support the last two stable Go versions.

Hmm ... the NATS support I'm proposing in #55 (#54) is blocking because NATS is using strings.CutPrefix which only became available in 1.20.
So let me know, either:

Are you going to bump go.mod only to 1.19 ?  In which case I will have to submit my S3 SSE-C PR seperately, becasue that will work on 1.19; or
Can I tempt you to bump to 1.20 ? In which case my NATS submission will work. :wink:


Support for API Priority and Fairness per tenant

Describe the feature
Kubernetes introduced the API Priority and Fairness (APF) feature [beta in k8s 1.20] to control the behaviour of the Kubernetes API server in an overload situation. See here for details.
When multiple tenants are deployed, a buggy or malicious application running in a tenant can overload the APIs server or it can compete with other tenants and starve their applications.

Each request sent to the APIs server is matched by a FlowSchema. The FlowSchema states the PriorityLevel for requests that match it, and assigns a Flow identifier to these requests. Flow identifiers are how the system determines whether requests are from the same source or not.

The PriorityLevels may be configured to behave in several ways. Each PriorityLevel gets its own isolated concurrency pool.

To prevent any one tenant from monopolising a PriorityLevel, they may be configured to have multiple queues. The Shuffle Sharding algorithm is used to assign each flow of requests to a subset of the queues.

Finally, when there is capacity to service a request, a Fair Queuing algorithm is used to select the next request. Within each priority level the queues compete with even fairness.

It would be nice to implement support for APF in Capsule:

The Capsule Controller gets its own FlowSchema with priority just one step lower than other system controllers. This assures the Capsule Operator cannot be starved by default user generated requests.

apiVersion: flowcontrol.apiserver.k8s.io/v1beta1
kind: FlowSchema
metadata:
  name: capsule-system-service-account
spec:
  distinguisherMethod:
    type: ByNamespace
  matchingPrecedence: 1000  # <-- precedence just lower than kube-system-service-accounts
  priorityLevelConfiguration:
    name: workload-high  # <-- same priority of system controllers
  rules:
  - nonResourceRules:
    - nonResourceURLs:
      - '*'
      verbs:
      - '*'
    resourceRules:
    - apiGroups:
      - '*'
      clusterScope: true
      namespaces:
      - '*'
      resources:
      - '*'
      verbs:
      - '*'
    subjects:
    - kind: ServiceAccount
      serviceAccount:
        name: capsule # <-- Capsule ServiceAccount
        namespace: capsule-system # <-- Capsule Namespace

Each Tenant gets its own PriorityLevel with default priority. This assures each tenant is not starved by other tenants.

apiVersion: flowcontrol.apiserver.k8s.io/v1beta1
kind: FlowSchema
metadata:
  name: oil
spec:
  distinguisherMethod:
    type: ByUser
  matchingPrecedence: 9900
  priorityLevelConfiguration:
    name: global-default
  rules:
  - nonResourceRules:
    - nonResourceURLs:
      - '*'
      verbs:
      - '*'
    resourceRules:
    - apiGroups:
      - '*'
      clusterScope: true
      namespaces:
      - '*'
      resources:
      - '*'
      verbs:
      - '*'
    subjects:
    - kind: Group
       group:
          name: capsule.clastix.io
    - kind: User
       user:
          name: alice

What would the new user story look like?

When Capsule is installed, the related FlowSchema is created.
When a new tenant is created/modified/deleted, the related FlowSchema is created/modified/deleted

Expected behavior
Support for API Priority and Fairness per tenant

@prometherion do you have any concern about this enhancement?

How to handle fairness when an Owner owns multiple Tenant resources?
We would end up with duplicates matching different values, not sure how the Kubernetes controller would deal with them.


Wrong units string uploaded

In the Profiles/ProfileSwitches, "mgdl" is uploaded to Nightscout instead of "mg/dl". This will cause other systems that receive the data to default to mmol/l.
Severity Score
5 for now - can lead to wrong recommendations of bolus calculator and closed/open loop systems. (We should at the same time make other apps like AndroidAPS more tolerant to non-standard Strings).

OK but... the NS docs state "mmol" or "mgdl" for profile use:

sigh so does this actually need to be changed here?

BTW the "BG Check" treatment we send to NS contains "mg/dl" or "mmol" for the units.
I would tend to agree with you that a reader should probably for check both versions to be on the safe side.
I'm happy to tag this for the uploader to send "mg/dl" in the next version, just don't want to break something else given the docs.

AndroidAPS sends mg/dl "for ages" (since early 2016) now and it works very well.
It also receives and compares it with the profiles generated within Nightscout and those send "mg/dl".
So I assume internally Nightscout uses "mg/dl" and the documentation is mistaken.
In other parts of the documentation like for DISPLAY_UNITS it also want mg/dl and mmol (without the "per litre")

mg/dl should be used for NS upload

Closed - completed and available in 0.7.0 release


[rcore] Small issues on MacOS with SDL2

Issue description
When building raylib with SDL2 as backend I faced 2 issues:

The InitPlatform() method calls SDL_Init(SDL_INIT_EVERYTHING) which results in some SDL2 fatal exceptions, which I cannot reproduce now, but I found that SDL_Init(SDL_INIT_VIDEO) works fine. I found something on the internet that the SDL_INIT_AUDIO is the issue, a subsystem which is included by SDL_INIT_EVERYTHING, but not actually needed for raylib.
Another error I caught was Failed creating OpenGL context at version requested because on Apple if you are using OpenGL 3.3 the SDL_GL_SetAttribute(SDL_GL_CONTEXT_PROFILE_MASK, SDL_GL_CONTEXT_PROFILE_CORE) is not called, even though for OpenGL 4.3 it is.
There is no automatic way of building with SDL2? The PLATFORM variable only supports Desktop which defaults to GLFW.

Environment
MacOS Sonoma 14.1.1 M1,  OpenGL 3.3

Is certainly a bug, in GLFW implementation, rcore_desktop.c, the call is not missing.

@atticus5 Thanks for reporting! It has been reviewed 1. and 2. but about 3., Makefile supports PLATFORM=PLATFORM_DESKTOP_SDK.

@raysan5 thank you, but I'm using CMake and GLFW is hardcoded:
https://github.com/raysan5/raylib/blob/7ec43022c177cbf00b27c9e9ab067bd6889957a4/cmake/LibraryConfigurations.cmake#L11

@atticus5 Oh, I'm afraid I'm not the user neither maintainer of CMake build system. It was contributed by the community and I kept it for convenience but any expert on CMake should review and maintain it.


Move to binder-examples ?

Once we get a working version that's ready for others to use, mind if I copypaste this into the binder-examples organization?

Yeah, probably. I was thinking I'd like to avoid having to write and install an extension here (so that doesn't need to be duplicated across repos) before doing that, but we could fix it later.


Get rid of hardcoded 'bootstrap.crt', using k8sbundle plugin

Signed-off-by: Ilya Lobkov<EMAIL_ADDRESS>
Description

Motivation and Context

Issue https://github.com/networkservicemesh/networkservicemesh/issues/1661
How Has This Been Tested?

[ ] Covered by existing integration testing
[ ] Added integration testing to cover
[ ] Tested locally
[ ] Have not tested

Types of changes

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

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Only interdomain tests are failed and changes looks good.


In a project with resource quota , the used limit does not get updated when namespaces are deleted

Rancher versions:
rancher/server or rancher/rancher: Master
Steps to Reproduce:

Create a Project with resource quota and default namespace resource quota in UI
Project Resource quota:
{CPU limit = 4000m
Pods = 10
}
Default Namespace resource quota
{ CPU limit = 500m
Pods = 2
}

Create 5 namespaces successfully and creating the 6th namespace fails as pod limit of 10 is reached

Delete one namespace.

Create another namespace
Creating another namespace fails although we have room for 2 pods

The used limit shows 10 pods. It should be 8
resourceQuota": {
"limit": {
"limitsCpu": "4000m",
"pods": "10",
"type": "/v3/schemas/resourceQuotaLimit"
},
"type": "/v3/schemas/projectResourceQuota",
"usedLimit": {
"limitsCpu": "2100m",
"pods": "10",
"type": "/v3/schemas/resourceQuotaLimit"
}
},

Tested with master build from Sep 13.
Create a Project with resource quota and default namespace resource quota in UI
Project Resource quota:
{CPU limit = 4000m
Pods = 10
}
Default Namespace resource quota
{ CPU limit = 500m
Pods = 2
}
Create namespace N1 in P1 with default quota as defined above (CPU = 500m and pods = 2)
Create namespace N2 in P1 with resource quota CPU=500m and pods = 8
The used limit does not get updated.
"resourceQuota": {
"limit": {
"limitsCpu": "4000m",
"pods": "10",
"type": "/v3/schemas/resourceQuotaLimit"
},
"type": "/v3/schemas/projectResourceQuota",
"usedLimit": {
"limitsCpu": "500m",
"pods": "2",
"type": "/v3/schemas/resourceQuotaLimit"
}
},

The used limit gets updated only if the namespace uses the defaultquota defined.

Tested with master build Sep 13 1:30pm.
In a project with resource quota , the used limit gets updated when namespaces are deleted.
Also, when namespaces are created with resource values greater than the namespacedefaultquota, the used limit gets updated as expected.
It was also tested after editing  project resource quota and  adding more resources and creating more namespaces. Verified that used limits get updated accordingly


Document how to Create a Domain when the WebLogic image name is not store/oracle/weblogic:<IP_ADDRESS>

We need to document how to change the WebLogic and Operator image names and generate the correct yaml for the Domain creation job.  This is specially important for OCI and OKE where the image names must be taged to push into OCIR
docker tag store/oracle/weblogic:<IP_ADDRESS> phx.ocir.io/weblogick8s/weblogic:<IP_ADDRESS>
docker tag weblogic-kubernetes-operator:developer phx.ocir.io/weblogick8s/weblogic-kubernetes-operator:developer
The procedure is to edit the operator yaml create-weblogic-operator-inputs.yaml and change the parameter
weblogicOperatorImage: phx.ocir.io/weblogick8s/weblogic-kubernetes-operator:developer
For the domain the image name must change in two templates
cd ~/weblogic-kubernetes-operator/kubernetes/internal and
edit vi create-weblogic-domain-job-template.yaml & domain-custom-resource-template.yaml
Change :  image: "phx.ocir.io/weblogick8s/weblogic:<IP_ADDRESS>"
Now when the create-weblogic-domain.sh script is invoked the generated yaml have the correct image name and the domain is created properly.

I have had to do this as well because I want to use a local registry which is accessible by the k8s cluster.  It would be good if you can provide a create-weblogic-domain-inputs.yaml parameter where the image name can be specified.


Update README because Incorrect sample code(for Java) #548

close #548

@thagikura @AlexBalo
Please review this PR.

Thanks!


the file explorer do not work after use "go to definition(F12)" several times

VSCode Version:
OS Version:

Steps to Reproduce:

use VSCode with the github open source project "taurusai/kungfu"
view the codes with the "go to definition(F12)" function

Does this issue occur when all extensions are disabled?: Yes/No

@lfjbb Can you check to see if there is anything in the dev console? (F1 > Developer: Toggle Developer Tools)
What extensions do you have installed?

acctually, I do not know much how this issue happen


Deploy version 4.1.0

@elwills PR for version 4.1.0 is ready to be merged.
https://github.com/GridProtectionAlliance/osisoftpi-grafana/pull/125

@coderReview PR merged and 4.1.0 has been published. Thank you!


Game randomly crashes if extends GDscript file

Operating system or device - Godot version:
Windows 10, compiled with 81fd4598c1e436c9dfe7951b1f829ff68045103b
Issue description (what happened, and what was expected):
Game runs fine or randomly gets error one of following cases when run, if using extends "some.gd" and some.gd has func _init()

Runs fine
Invalid call to function '_init'. Expected 6 (or random number like 528, 720896) arguments.
Bug, call error: #1576692939
Game Crashes without debug info

Steps to reproduce:

Node2D (A.gd)

A.gd
extends "B.gd"

B.gd
extends Node2D

And run several times.
Link to minimal example project (optional but very welcome):
https://drive.google.com/open?id=0ByuhCIzeus36TURzY3RBMnVULUk

@reduz I guess it's related with profiler recently added.

@volzhs ~ Well, you should try running it in 2.0.3 to confirm that.

@bojidar-bg I tested with 4c4d79e3c98182faf348f41f98d1cc3e5d843e69 commit which is just before profiler commit.
And it didn't happen.

@bojidar-bg @reduz I found that it happens just when using extends "script.gd".
I deleted func _init() in B.gd, so B.gd has only extends Node2D.
And It happens same error as describe.

I can confirm this, using inheritance doesn't work at all. Debugger also shows what appears to be random errors like "Illegal opcode 71723344 at address 0" or "Invalid call to function '_init'. Expected 10 arguments."

I can confirm this, using a version before the commit who added the "profiler", the game runs fine. The error is "Bug, call error: #60507152" (running on Ubuntu 16.04 64b).

@reduz Can you have a look? It's a regression from this weekend's developments.

Sad onfirmed in b2a4908e9cc802a838a67f92dd66a17d3894c619
And it not random, game crashes all time that run, with message: Invalid call to function 'apply_json'. Expected 0 arguments
or nothing (crashes debuger)!
I make a clean clone of repo!
Affected classes:
ator.gd
extends Reference

func apply_json(ob):
   ...

personagem.dg
extends './ator.gd'
func apply_json(ob):
   ...
   .apply_json(ob) # <- raises the error

Quick PR to fix this, base function call was left unchanged after converting member_functions from GDFunction to GDFunction*


updating expired alias that had password with a new password does not work

Tried to update expired alias (which originally had a password) and changing password at the same time, received the following error message:

Also tried to renew alias without putting a password and it worked, then tried to update alias with password, received same error message.

Did you already use that password in that wallet? It would give that error if that password alias combination was already used.. ive since changed it to check and throw that only if password is not found in wallet

fixed close #118


Disable HTML escaping

I really like QTPL but I rarely use it for HTML generation but rather go code generation. So I often write code like this to avoid the automatic HTML escaping which is really inconvenient:
func writeunesc(iow io.Writer, s string) {
	qw := qt.AcquireWriter(iow)
	streamunesc(qw, s)
	qt.ReleaseWriter(qw)
}

func unesc(s string) string {
	qb := qt.AcquireByteBuffer()
	writeunesc(qb, s)
	qs := string(qb.B)
	qt.ReleaseByteBuffer(qb)
	return qs
}

func streamunesc(qw *qt.Writer, s string) {
	qw.N().S(s)
}

to use it like this:
{% func Foo(param *ParamType) %}
Some
{%= unesc(param.Text) %}
stuff
{% endfunc %}

So I wanted to ask if there is already a way to deal with this and if not how one could implement this maybe as some thing like {%r param.Text %}

You can already do it with {%s= params.Text %}.
However, I think it would be interesting to be able to disable escaping for a block of code.


Aruba POC Test Plan v0.6

What this PR does / why we need it:
Upload Aruba POC Test Plan v0.6.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

Coverage remained the same at 43.911% when pulling 172f34a30361e84764411c167d2b9b267e5b9f96 on xing-yang:aruba_poc into f4fc4f84c8c32eda335a35bd57b07f65e0bc2bba on opensds:master.


Fixes #1.

I haven't tested this extensively but it should work.

closing in favor of https://github.com/bcherny/draggable/commit/500cbd33d7438e8568ac6bd59648ed30870c2b25. thanks for the attempt, but please test your code before submitting a pr! (just because i didn't test mine doesn't make it ok :beer:)


Inconsistency in cubed sphere output in MAPL 1.x

Ricardo found an issue when using History with MAPL v1.x, not this is a non-issue in MAPL v2.x since that is no longer using CFIO in History and does not support the "old format" cubed-sphere output
For the purposes of the discussion the "old format" is the cubed output that does not have the face as an explicit dimension, "new format" does have the face as a dimension.
Ricardo found this oddity.
The MAPL_CFIO layer has this logic put in a long time ago when we first moved the new format that force any regridded output to be in old format.
However, there is an inconsistency.
If you request native cubed output in the old format you get this in the file:
dimensions:
        Xdim = 48 ;
        Ydim = 288 ;
        lev = 48 ;

but if you regrid to a different cube resolution you get this:
dimensions:
        lon = 48 ;
        lat = 288 ;
        lev = 48 ;

This is all because normally if you ask for native output there is a logical "isRectilinear" in MAPL_CFIOCreate that is set to true for the cube or tripolar grid. However, if it detects you are changing resolution at all, isRectilinear is false. and the use of the new cubeformat is set to false.
Ultimately this controls whether the call to little cfio forces the use xdim/dym or lon/lat for the horizontal (this is true even for non-cubed output).
That is the first thing
Testing this I discovered what appears to be a bug.
What I described above is what we see when you set VERSION: 1 in the history.rc file.
If you set VERSION: 0 something must go wrong as if ask for regridded cubed-sphere output you get native output.
Luckily the History.rc being used everywhere have VERSION: 1
This kluge to force the gridded files to be the old format may not even be necessary anymore has MAPL_CFIO can read the files in the new format. We could not remember exactly why the old format was necessary at the time for the DAS workflow.

@bena-nasa Have you fixed this?

Reexamined on 2021-Apr-12 with @bena-nasa and @tclune
Labeling wontfix and closing


Use bearer token for publishing API authentication

https://trello.com/c/tx50hSOx/492-add-bearer-token-value-to-all-apps-using-publishing-api

Tests so failed.

Lint failure; updated.


Update docs to use latest FreeBSD package name

With updates to the default version of Python, the package name has changed from py37-asciinema to py39-asciinema.
Fixes #404.

Thanks!


Rak11300 - OTAA join failed!

I'm trying to run the LoRaWAN_OTAA_ABP example for the Rak11300 core.
I'm getting the following error:
12:32:48.121 -> =====================================
12:32:48.121 -> Welcome to RAK11300 LoRaWan!!!
12:32:48.121 -> Type: OTAA
12:32:48.121 -> Region: EU868
12:32:48.121 -> =====================================
12:33:16.042 -> OTAA join failed!
12:33:16.042 -> Check your EUI's and Keys's!
12:33:16.042 -> Check if a Gateway is in range!

Check your EUI's and Keys's!: EUI and Keys are correct.
Check if a Gateway is in range!: The Gateway is 1meter from the device.
I've registred the device on the helium console and I receive 3 Join Request replied with 3 Join Accept.
Here is a screenshot: https://imgur.com/gallery/PExIFE6.
Note:
I've also tried the example LoRaWAN_OTAA_ABP example for the Rak4631 core. In this case I have no problem. Everything works fine and I receive packets succesfully.
If you need more info let me know.

I updated the LoRaWAN examples for RAK11310 two days ago. https://github.com/RAKWireless/WisBlock/tree/master/examples/RAK11300/communications/LoRa/LoRaWAN/LoRaWAN_OTAA_ABP
Do you use the changed example?
There was as well a bug in the SX126x-Arduino library. ==> https://github.com/beegee-tokyo/SX126x-Arduino/releases
Please update to latest version 2.0.11

I was using an old version of the example.
Now the device join succesfully :)
However, it should sends a frame every 20s. But it sends only one time (at the start).
Trying to find out why...
Thank you so much!

I needed to make little changes to your code. In particular to the tx_lora_periodic_handler function.
Here is my working version:
void tx_lora_periodic_handler(void)
{
  send_now = true;
  appTimer.attach(tx_lora_periodic_handler, (std::chrono::microseconds)(LORAWAN_APP_INTERVAL * 1000));
}

That is one of the problems with the RP2040 BSP. It needs sometimes such workarounds because it is based on mbed.
Glad you could solve it.


add except, update changelog

Description
Why was this PR created?
Development notes
What have you changed, and how has this been tested?
Checklist

[x] Read the contributing guidelines
[ ] Open this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Update the documentation to reflect the code changes
[x] Add a description of this change and add your name to the list of supporting contributions in the CHANGELOG.md file. Please respect Keep a Changelog guidelines.
[ ] Add tests to cover your changes

Notice

[x] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

@Galileo-Galilei I can't see what's failing in Snyk, no access to the org. Should I request access?

No don't worry. It's failing for a while because of mlflow dependencies issues.
Thanks for the PR, I'll have a look soon!

Okay, thank you. I can say that I tested with our internal project and building it with kedro-mlflow from this commit and I don't have any further error. So it looks that was the only issue from our perspective with version 0.18.1.
The only issue I can see is that using 0.18.1 would prevent users from configuring with something others than default. But I'm not sure how to proceed with this and since at least the basic functionality is fixed with this PR, adding config support seems less urgent and could be provided in a separate PR.

(And if I remember well, the kedro ==0.18.1 version is a but buggy, you should upgrade if possible)

Thank you for the merge!
Do you plan to release soon? I ask but it's not my goal to add any pressure, it's to know if you plan to add other commits in this release I will host the fix on a private pypi server in the meantime while if you plan to do it in the next few days I'll simply wait on the official version.
Thanks for the great work on this lib!

It's on PyPI!

Excellent, thanks a lot. Yes I'll also check about updating kedo, thanks for the tip!


pass url in Action Buttons is it possible?

I want to open especific url in my app with Action Buttons in android, is it possible ? is there a way to do this, any example, it's my code but doens't work, apreciate your help:
$msg = array
(
        'title'        => 'Example',
        'message'      => 'text',
        'url'          => 'inicio.html',
        'image'        => 'example.jpg',
'actions' => array(array('icon' => 'example','title' => 'view data','callback' => 'view.html?id=1000')),
);

@hmarina the callback is a JS function not a url. What you want to do is add the data to your push and read it in your callback as the callback is invoked with the notification data. For example send:
$msg = array
(
        'title'        => 'Example',
        'message'      => 'text',
        'url'          => 'inicio.html',
        'image'        => 'example.jpg',
        'viewdata'  => '100',
'actions' => array(array('icon' => 'example','title' => 'view data','callback' => 'view')),
);

and your view callback would be:
view = function(data) {
  document.location.href = 'view.html?id=' + data.additionalData.viewdata;
};


Can't deploy the Web to Azure APP

Does this occur consistently?
Repro steps:

Action: appService.Deploy
Error type: 403
Error Message: Error 403 - Forbidden
The web app you have attempted to reach has blocked your access.
Version: 0.24.9
OS: win32
OS Release: 10.0.19045
Product: Visual Studio Code
Product Version: 1.78.2
Language: en

Call Stack
new RestError extension.bundle.js:2:1709073
extension.bundle.js:2:1696547extension.bundle.js:2:1696547

Please add detailed steps to reproduce the issue.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


[Bug]: Discord & Slack links not working

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Describe the bug and reproduction steps
I couldn't find the links anywhere else, all lead to "expired" messages. Can you please update the links?
Currently not working:

Slack
Discord

OpenHands Installation
Docker command in README
OpenHands Version
No response
Operating System
MacOS
Logs, Errors, Screenshots, and Additional Context
No response

@UltraInstinct0x can you try the links in this PR: https://github.com/All-Hands-AI/OpenHands/pull/5773/files
And see if they work for you to join Discord and Slack please?

Hi @mamoodi! Thank you very much!!! Yes I could join the Slack via that link however Discord still seems invalid.
I think updating Readme with the slack link from the PR and a new Discord link will resolve this issue.
Or maybe updating them again and again, we can set up a flow for it.

@UltraInstinct0x how about this one: https://discord.gg/bm2gjaZYXK

Oh wait I think I'm doing something wrong.

Oh wait I think I'm doing something wrong.

Yup, its not working :(

@UltraInstinct0x one more try. How about this:
https://discord.gg/cVXaB6xKWD

@UltraInstinct0x one more try. How about this: https://discord.gg/cVXaB6xKWD

Still no luck :(

Hmmmmmm that's strange.
https://discord.gg/mBuDGRzzES
If this doesn't work, something may be up with the server? I'll ask

Cause I see it says 1349 people have already joined with that and it doesn't expire.

And the original link that is on the README, most people joined with that so there might be an issue on your side....

@mamoodi the link with "ESHStjSjD4" points to the moderator channel, please check that you create a link to the public channel? 😁

@tobitege that's what I thought it was at first!! But 3k+ people have joined using that link so it must be working? :D

@mamoodi @tobitege thank you for both of your help, I could finally join.
Here what was wrong:

Discord is banned in TR
WARP connects to many Discord API's without issues
However banners etc. cannot be loaded with WARP
Likewise invite link leads to a page which says "Invite is expired"
I tried another VPN, same link worked.

...most people joined with that so there might be an issue on your side....

Given that the link is valid, the only plausible explanation that comes to mind was the possibility of a network and it worked. Thanks a lot!!!


Custom font with spaces in name causes invalid PDF

When generating a PDF and a custom font title with a space in its name is used, the resulting PDF is not PDF/A compliant and even fails to open in some PDF readers (such as awful Adobe Acrobat Reader).
Here is a example with jsPDF v2.11 and the default windows fonts Calibri and Comic Sans. From my testing, any ttf font will do:
var doc = new jsPDF(); // create document

doc.text("This is the default font", 100, 20, "center"); // write text
doc.save("Standard.pdf"); // save pdf

doc.addFont("https://server/calibri.ttf", "Calibri", "normal"); // embed font
doc.setFont("Calibri"); // set font
doc.text("This is Calibri", 100, 40, "center"); // write text
doc.save("Standard with Calibri.pdf"); // save with a custom font embedded

doc.addFont("https://server/comicsans.ttf", "Comic Sans", "normal"); // embed font
doc.setFont("Comic Sans"); // set font
doc.text("This is Comic Sans with a space in its name", 100, 60, "center"); // write text

doc.save("Standard with Calibri and Comic Sans.pdf"); // save with a custom font embedded

Using https://www.pdf-online.com/osa/validate.aspx to test the invalid PDF shows the following:
Validating file "Standard with Calibri and Comic Sans.pdf" for conformance level pdf1.3
	The "endobj" keyword is missing.
	The key Encoding is required but missing.
	The key DescendantFonts is required but missing.
	The document does not conform to the requested standard.
The file format (header, trailer, objects, xref, streams) is corrupted.
The document doesn't conform to the PDF reference (missing required entries, wrong value types, etc.).
The document does not conform to the PDF 1.3 standard.

Simply removing the space, making it "ComicSans" instead, fixes this issue and neither the validator or Acrobat Reader will complain anymore.
It is probably related to issue #2593

I can confirm this scenario and that removing spaces fixes it. I came here looking for a solution and this provided it, relatively painlessly. Thanks @Ceremony64

Here, the font.fontName also needs to be escaped like in the other occurrences. @Ceremony64 could you provide a pull request?

I created a PR: #3018. @Ceremony64 could you verify that this fixes the issue? Note, that you need to run npm run build to update the files in the dist folder.

This is still an issue for me and I'm using version 2.3.1. I tested it out in much the same flow as the OP. I'd expect any font to work with a space in it but only spaceless names do:
Works:
      jspdf.addFileToVFS('Chalkduster.ttf', comic)
      jspdf.addFont('Chalkduster.ttf', 'Chalkduster', 'normal', 'normal')
      jspdf.addFont('Chalkduster.ttf', 'Chalkduster', 'normal', 700)

Does not work:
      jspdf.addFileToVFS('NotoSans500.ttf', regFont)
      jspdf.addFileToVFS('NotoSans700.ttf', boldFont)
      jspdf.addFont('NotoSans500.ttf', 'Noto Sans', 'normal', 'normal')
      jspdf.addFont('NotoSans700.ttf', 'Noto Sans', 'normal', 700)

Not even for system defined fonts:
      jspdf.addFileToVFS('ComicSans.ttf', comic)
      jspdf.addFont('ComicSans.ttf', 'Comic Sans MS', 'normal', 'normal')
      jspdf.addFont('ComicSans.ttf', 'Comic Sans MS', 'normal', 700)

This still isn't working and I'm using version 2.3.1. Is it expected we use the new fontFaces option for html()?

@cvharris this was fixed (again) in #3149 but is not yet released.


Extra space is added when escaping special characters.

looks like the library adds an extra space when escaping special characters.
($0.72/Ounce)
becomes
( $0 .72 /Ounce)

@martco we had a similar issue that may or may not help you.
In our case, it turned out we introduced the behavior in a custom link component.  We were map-ing over nodes content and eventually doing a join(' ') on the text.  The accidental space in join was the problem for us.


Can't install from pip

This packages is a lifesaver! Thank you.
I am getting this error when installing from pip.
Collecting django-sql-utils
  Using cached https://files.pythonhosted.org/packages/f1/ac/8ea2b8202de85c59560e3f7f5f4b57de461c387e512eb40574de11d4fa38/django-sql-utils-0.2.1.tar.gz
    ERROR: Complete output from command python setup.py egg_info:
    ERROR: Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/private/var/folders/y4/qbz9b6cx4z5b48vc_y6k_cbr0000gn/T/pip-install-iFRBHZ/django-sql-utils/setup.py", line 13, in <module>
        long_description=read_file('README.rst'),
      File "/private/var/folders/y4/qbz9b6cx4z5b48vc_y6k_cbr0000gn/T/pip-install-iFRBHZ/django-sql-utils/setup.py", line 5, in read_file
        with open(name) as fd:
    IOError: [Errno 2] No such file or directory: 'README.rst'
    ----------------------------------------
ERROR: Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/y4/qbz9b6cx4z5b48vc_y6k_cbr0000gn/T/pip-install-iFRBHZ/django-sql-utils/

I think the README.rst is not uploaded to pip.  Is there anyway to fix this?

Yes, looks like the README is not in the tar archive. I'll update setup.py to be more robust to that file missing.

This should be fixed if you try installing version 0.2.2. @blocher Can you confirm this?

Works brilliantly! Thank you.


make istio-system cluster local

As an alternative to https://github.com/istio/istio/pull/23625 for fixing https://github.com/istio/istio/issues/23591, we could make the entire system namespace cluster-local.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
d14c6d23af3c9e1755a51bce22db922f3ffbff6d
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
d14c6d23af3c9e1755a51bce22db922f3ffbff6d
link
/test unit-tests_istio

lint_istio
df964153548245ffc1c58884189848498483c580
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
6cd720cbf720f14efab50ee8ee4ac800ec8400ab
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl you closed this PR after the other implementation merged. Then I think you force pushed your repo without deleting the local branch. If this PR is open in error, would you be kind enough to close this PR out and delete your local branch of this work or use selective force pushes?
Cheers
-steve


fix: dont record various commands

When was the last time someone used sar_hud_inspection? Is it worth cheat-protecting? Certainly looks quite cheaty
Newly unrecorded commands:

sar_fast_load_preset
sar_hud_*
sar_ihud_*
sar_toast_*
sar_sr_hud_*
sar_echo[_nolf]

Modified help strings for:

help
sar_<x>_setpos
sar_hud_order_reset

LMK if there's anything else that deserves not recording :)

whoops: when flags are changed from 0/never_as_string on cvars they are given 0-1 range, because that makes sense.

more precisely, src/Variable.cpp@36
// Boolean or String
Variable::Variable(const char *name, const char *value, const char *helpstr, int flags, FnChangeCallback_t callback)
	: Variable() {
	if (flags != 0)
		Create(name, value, flags, helpstr, true, 0, true, 1, callback);
	else
		Create(name, value, flags, helpstr, false, 0, false, 0, callback);
}


[release-1.21] xds: push CDS on AUTO_PASSTHROUGH Gateway change

Follow-up to https://github.com/istio/istio/pull/51644.

@jewertow: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio_release-1.21
aa798da6250aac5a69c2356cc9edec52ce28c721
link
true
/test integ-security-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/retest


Consider removing usage of google/protobuf/wrappers and using an in-repo protocol for that

Currently, this library uses one proto file not present in the repo itself: google/protobuf/wrappers.
That protocol is completely trivial: https://github.com/protocolbuffers/protobuf/blob/master/src/google/protobuf/wrappers.proto
It's used directly as google-protobuf/google/protobuf/wrappers_pb.js in a number of places here, but there is a problem with that -- that file has been compiled in commonjs mode instead of commonjs_strict, which brings the same issues with globals pollution and CSP incompatibility. https://github.com/protocolbuffers/protobuf/issues/7778 is related.
#267 is related, and even after the fixes mentioned in #267, it would be still blocked against this one if wrappers.proto is not also recompiled in commonjs_strict mode.
A possible solution would be to replace all direct usage of that compiled js file with a proto file inside this repo, compiled in the same mode as all the other proto files. Perhaps even a subset of that file if not all imports are used.

It's a little unfortunate that the google provided library isn't compiled in strict mode but I agree a trivial fix is to inline the proto source.

Done in develop


capes float off the player's torso while running

What version mod/minecraft and are you on forge or fabric

1.20.4 fabric

Is it the latest serious player animation version 1.1.1 ?

yes


Adds PushOver key entry ui

Commits changes that add the new key entry ui element which stores the data to the "userdata".

Meant to create pull request for our fork not the original, figuring out what I did wrong xD


Create delivery application management page

Feature Description
Delivery managers should be able to review delivery applications, and mark them as active or rejected.
Expected solution

Need to add roles for admins and then pages only admins can interact with.
Testing

The delivery pages design prototype is up, going to close this to make specific issues for completing the functionality.


Detected Entities Report - Add support for BIQ Applications

Add support for:
Schemas inventory
Field inventory
Searches
Custom Metrics
Business Journeys
Experience Levels

Fixed by d26c4d64bb1dfb895a110ecf182ac7340e877d72