Removed all references to DLL

Closes https://github.com/corona-school/project-user/issues/427

@realmayus Can you check for this building issue? This is the corresponding log from Heroku:
-----> Building on the Heroku-18 stack
-----> Using buildpack: mars/create-react-app
-----> React.js (create-react-app) multi app detected
=====> Downloading Buildpack: https://github.com/heroku/heroku-buildpack-multi.git
=====> Detected Framework: Multipack
=====> Downloading Buildpack: https://github.com/heroku/heroku-buildpack-nodejs.git
=====> Detected Framework: Node.js

-----> Creating runtime environment

       NPM_CONFIG_LOGLEVEL=error
       USE_YARN_CACHE=true
       NODE_ENV=production
       NODE_MODULES_CACHE=true
       NODE_VERBOSE=false

-----> Installing binaries
       engines.node (package.json):  unspecified
       engines.npm (package.json):   unspecified (use default)
       engines.yarn (package.json):  unspecified (use default)

       Resolving node version 16.x...
       Downloading and installing node 16.13.2...
       Using default npm version: 8.1.2
       Resolving yarn version 1.22.x...
       Downloading and installing yarn (1.22.17)
       Installed yarn 1.22.17

-----> Installing dependencies
       Installing node modules (yarn.lock)
       yarn install v1.22.17
       [1/4] Resolving packages...
       error Your lockfile needs to be updated, but yarn was run with `--frozen-lockfile`.
       info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
-----> Build failed
 !     Outdated Yarn lockfile
       Your application contains a Yarn lockfile (yarn.lock) which does not
       match the dependencies in package.json. This can happen if you use npm
       to install or update a dependency instead of Yarn.
       Please run the following command in your application directory and check
       in the new yarn.lock file:
       $ yarn install
       $ git add yarn.lock
       $ git commit -m "Updated Yarn lockfile"
       $ git push heroku main

       https://help.heroku.com/TXYS53YJ
 !     Push rejected, failed to compile React.js (create-react-app) multi app.
 !     Push failed

It's probably simply merging master into the branch and then executing yarn install.


Update README.md

"Think Different" :smile: :fast_forward: :+1:

Well it is quite like it but i do like to point out that advertising(don't take it on :heart:) in the guidlines is seriously ill sort of stuff. If you really wanna point users to your site then we can create a CONTRIBUTIONS.md for that. Or better a THANKSTO.md

Neither is it a good idea. Well did you recieve the invitation to join CPP-Coders
:question:

so what should I do. Should I withdraw Myself...... :anger:

No improve the code and then I will merge it :smile: :+1:


Update hook scripts

Hook scripts 'pre_gen_project.py' and 'post_gen_project.py' need to be replaced for a similar in copier

Copier does not seem to have hooks, but input validation can be performed via e.g. the following:
  type: str
  help: Name of the package author (Used to kickstart a few files)
  validator: "{% if AuthorName | length == 0 %}Can't be empty{% endif %}"```


Bug in loading GMSC

When using the load_gmsc function and using a value for n that is arbitrarily large, the function will oversample the dataset resulting in more data points than that exist in the dataset. This is not a desirable result. Instead, the function should just return the full dataset when n is larger than the amount of data points.

I believe this is no longer relevant to CounterfactualExplanations.jl, since that function has been moved to TaijaData.jl package.


Rendering on VR devices

Hello, may I ask how do you realize rendering on VR devices, whether through streaming or others.

Implemented in C++/OpenGL, through the official SDK provided by HTC.

Thanks for answering. Could you provide the related code?

fovnerf_vs.zip
Please take a look at view_vr in the attached zip file. The code only supports the Windows platform and HTC Vive Pro Eye.


Create a BGZIP index (BGZF) on the fly

See: http://www.htslib.org/doc/bgzip.html.  This should not be too hard if you keep the uncompressed and compressed byte offset of each block you are processing.  Right now, I have to use bgzip to create the index.

I do really want this and just haven't gotten around to it / needed it myself yet.

Maybe I’ll make a PR


Element move up after remove

Hey,
I'm currently developing a tool with gridster JS in it. When I have 2 sticky elements with space between them and I remove the one above the other one will pull up to the top automaticly. Seems kinda buggy.
Anyone know a fix?
KR Xarf

fixed it. Documentation didn't say sth about the silent parameter :D

Great,,, would you like to submit a PR for the doc change


Front page news

Added appropriate pictures to the showcased news articles on the front page (index.html). I also removed descriptions since it seemed repetitive to have them both places, but we can add them back.

complete #35


Setup travis / buddy suite

Update tests to use CI. Ideally multiple versions of hxcpp / Haxe. Maybe docker :)

Travis implemented for Linux64 across 3 combinations of Haxe / hxcpp versions, running a single test! :tada:
Still TODO:

[ ] MacOS
[ ] See about Windows - example repo
[ ] A more significant test suite


Github actions & clean up to be able to do linting

Checklist

[X] I have read the Contributor Guide
[X] I have read and agree to the Code of Conduct
[X] I have added a description of my changes and why I'd like them included in the section below

Description of Changes
This PR introduces building with Github Actions instead of using Buildkite. The goal is for this to streamline how we build open source projects.
What does the PR contain:

Removed the old buildkite pipeline
Added one workflow for PR's and one for master. Master will only build and push.
Added linting, etc using gosec, golint, errcheck, staticcheck and go vet to ensure that we are following best practices.
Fixed issues raised by the linters above

What's left to do before it's ready for merge:

Need to add Dockerhub credentials & make it possible to push a docker image using buildah in the push_master.yaml

This looks like a huge PR, it's actually not that big if you disregard the static assets that I decided to check in (as opposed to having them built every time). As of right now we are skipping some staticchecks (ST1003,U1000,ST1005) and also golint is run on specific files as opposed to the package due to us using go-bindata-assetfs to bundle the static assets in the binary. There is a PR in the go-bindata repo (https://github.com/go-bindata/go-bindata/pull/37) that aims to fix this and once it's merged we will start linting the whole package again. Next steps after this PR also includes building for ARM and IPv6 support, but that's for later.
For a later discussion, it's probably time to tag another release once this is merged as latest contains some nice fixes like following the RFC1918 specification.
Feel free to start reviewing, I only put it as draft to finish up the last thing to push to Dockerhub.
Related Issues

I'd say stop using dockerhub and instead push to Github Packages instead. That would take care of the credentials issue, no?

I'd say stop using dockerhub and instead push to Github Packages instead. That would take care of the credentials issue, no?

I agree with this, let's ship packages instead!

Yeah, I'm currently working on it! :shipit:


update wanted ebooks list (2 released books)

2 released books are still on the list, remove them

Thanks!


Metadata trait_type

How do you include the attribute trait_type in the metadata? currently, only the name of the layer element shows up and not the name of the layer. This removes the ability to add property tags to the nft when minting.

Am wondering how to fix this also

use v3 branch.
you can find the code on line 63 here.
https://github.com/HashLips/generative-art-opensource/blob/v3/index.js

Thank you for reply i tried this and it will not return the name of the layer ie the folder name

use v3 branch.
you can find the code on line 63 here.
https://github.com/HashLips/generative-art-opensource/blob/v3/index.js

Thanks for the reply but I tried copy pasting that into the main branch and it does not return the trait still. I don't wanna use the v3 branch as it does not have enough rarity configuration. Can someone please modify the main branch so that it outputs the trait type along with its name? That would be much appreciated!

Ok Im a complete noob but I managed to solve this issue after a lot work. Basically what I did is add
var path = require('path')
to the beginning of the index file and changed the const getAttributeForElement to
const getAttributeForElement = (_element) => {
let selectedElement = _element.layer.selectedElement;
var parentDir = path.dirname(_element.layer.selectedElement.path);
var baseDir = require('path').resolve(parentDir, '..');
var traitName = path.basename(baseDir);
let attribute = {
trait_type: traitName,
value: selectedElement.name,
rarity: selectedElement.rarity,
};
return attribute;
};
This now outputs the trait_type and value for each nft.

Lol thanks guys i appreciate it. Just trynna help everyone out as much as I can


Bug? EditorSpinSlider: Change input via ´down´ or ´up´ arrow key

Godot version
v4.1.1.stable.official [bd6af8e0e] and v4.2.dev4.official [549fcce5f]
System information
macOS 13.4.1 - Vulkan (Forward+) - integrated Apple M1 - Apple M1 (8 Threads)
Issue description
If the input is set to -1 and you want to change the value back to 0 by pressing the arrow key up, the value is increased by 0.001 only. This looks like a rounding error (see video below).  In version 3.5.2 it was still ok.
Steps to reproduce

create Node2D
Inspector -> Transform -> x - Position
enter -1
press the arrow keys up several times, then down again.

https://github.com/godotengine/godot/assets/41921395/2ff82b7f-b24d-4039-88b3-30328258791d
Minimal reproduction project
N/A

Can't replicate, and the video doesn't show the same behavior as your description

now? new video and Steps to reproduce

I cannot replicate this no, not without pressing other keys, like Alt

no, only up and down, on my mac keyboard

Looks like you are accidentally dragging when selecting the edit, which modifies the value
Can you confirm this by having the mouse outside the area, and select it with tab instead of clicking on it?

new video, now with keyboard events (bottom left)
mouse outside the area, and select it with tab
alt (option) changes the value by 0.1, but from -1 and alt + arrow up also from time to time by 0.001

https://github.com/godotengine/godot/assets/41921395/ac921b2c-6180-4f29-a795-4ef8db1241aa

Probably something either hardware specific on your end or Mac specific, as I cannot replicate this on Windows

I'll also check it later under WindowsOS.
I wanted to try to catch mouse wheel events and change the values. I could not yet find the logic for arrow keys (up / down). Does anyone know in which file? Via debugging I found out that the variable text is only adjusted much later after "_process". Not yet in the function 'LineEdit::gui_input'.
https://github.com/godotengine/godot/blob/fa3428ff25bc577d2a3433090478a6d615567056/scene/gui/line_edit.cpp#L241

It's not a LineEdit it's an EditorSpinSlider under editor/gui

partially reproduced under Windows v4.1.1.stable

leave input at 0
alt + key down reduced by -0.001
alt + key down reduced by -0.001  to -0.002
alt + key down reduced by -0.1  to -0.102
alt + key down reduced by -0.1  to -0.202
alt + key down reduced by -0.001  to -0.203

that looks strange, or is it intentional?

So you've now moved away from the original issue?

ok thx, I will check it more closely. I think macOS also has a "floating point problem", even with -1

I found out that the CLAMP condition is true even though the value is within min / max limit.
When I comment out the lines, I no longer see the "floating point problem". I think this is a real error set_value(last_value + real_step) and set_value(last_value - real_step) are executed unnecessarily. With double, the floating point problem should only occur after the 10th decimal place and not after 0.001.
https://github.com/godotengine/godot/assets/41921395/5b59f2b8-9224-46cd-9dbb-7f608fc110e5

The floating point problem isn't related to floating point rounding, but to representing as text, as you can see it only represents to a fixed number of digits

I may already have a solution.
more questions:
get_min() returns -99999 and get_max() +99999
but is it possible to enter larger values, should I still execute CLAMP after the normal input or remove it from key up / key down?
Note: if the values are too large, the input can no longer be changed via "key up" / "key down".

It is based on the range provided, for Node2D the range is -99999 to 99999, it depends, other properties have different ranges


Add Github MoJ GitHub standards badge

User Story
As a CloudOps Engineer
I want to have visual guidance on the status of my repos
So that I get visual feedback on status of MoJ GH standards
Value / Purpose
Repository badge for our repositories that will display if meets the MoJ github repository standards.

Additional Information
Instructions from Operations Engineering about adding badge here
https://github.com/ministryofjustice/operations-engineering/issues/642
Definition of Done (DoD)
All repo's have badges

[ ]  staff-device-dns-dhcp-admin
[ ]  staff-device-docker-base-images
[ ]  staff-device-logging-infrastructure
[ ]  staff-device-shared-services-infrastructure
[ ]  staff-infrastructure-monitoring
[ ]  staff-infrastructure-monitoring-blackbox-exporter
[ ]  staff-infrastructure-monitoring-deployments
[ ]  staff-infrastructure-smtp-relay-server
[ ]  network-access-control-admin
[ ]  network-access... repos

Checklist for definition of done and acceptance criteria, example below (optional).
Example

[ ] Documentation has been written / updated
[ ] Another team member has reviewed
[ ] Tests are green

Reference
How to write good user stories

Hey team! Please add your planning poker estimate with ZenHub @ASTRobinson @Gary-H9 @kyphutruong @mitchdawson1982 @moontune @mTouhid @wanieldilson

all public NVVS DevOps repos have now had the moj compliance badge added.


[gardening] Fix headers. Remove "REQUIRES: asserts" from fixed crashers.

@swift-ci please smoke test and merge


deleting all node-modules files with .gitignore and fixing jsx render issue in webpack.config.js

waiting to test changes on Gareth laptop


Add Local Partners and Sponsors section to Events page

Heading
Local Partners and Sponsors
Paragraph types:

Paragraph. Text area with restricted HTML editing.

Subheading. Can be used as headings for different levels of sponsors, e.g. Platinum/Gold/Silver/Bronze

Sponsor with image. Support JPG, GIF, PNG, SVG?
-- Sponsor Name
-- Sponsor Link
-- Sponsor Logo + Alt Text

Sponsor with details.
-- Sponsor Name
-- Sponsor Link
-- Sponsor Logo + Alt Text
-- Sponsor Description

Display
Display at most 4 across in a row. Logo image is linked.

Note from Yury:

...[I]n some locations conferences like WIAD are held by communities - local professional groups with known name, logo, web site, etc. It would be convenient (at least here in Saint Petersburg) to be able to mention our community. But, as an option, we could include it in the local sponsors list


Lower threshold for SSL verification to Python 2.7.5

This avoids errors with SSL verification on Python version shipped with RHEL 7

Maybe with the ability to turn on/off SSL verification with a parameter introduced in https://github.com/splunk/splunk-sdk-python/commit/5807d9076e708f21f3949fafea304485396b5d99, this check should be removed. If the server is running a valid certificate, there's no reason to disable verification unless it's needed. Now people will just be able to disable it where they need it.

This should resolve the problem with being able disable SSL verify.  https://github.com/splunk/splunk-sdk-python/pull/233


Fix code scanning alert - Compilation error

Tracking issue for:

[x] https://github.com/michelin/i4i-iot-hub-portal/security/code-scanning/2

Code scanning alert manually closed as false positive


Add ScopeRangeProvider

Required by #1653
Depends on #1647

Checklist

[ ] I have added tests
[ ] I have updated the docs and cheatsheet
[ ] I have not broken the cheatsheet

@codiumai-agent /describe


how to close "open in windows terminal"

Description of the new feature/enhancement
How to close the right-click menu of "open in windows terminal"

Right now, there's not a way to do that. We're tracking adding a setting to remove it in #8105. Thanks!
/dup #8105

Right now, there's not a way to do that. We're tracking adding a setting to remove it in #8105. Thanks!
/dup #8105


Workload Identity Support

Feature request
To simplify GCP Bucket access, support Workload Identity as it's the Google recommended way to authenticate.
Use case
Simplify GCS bucket access and reduce complexity.

Can you describe what doesn't work about WI today? This may just be a documentation issue.
I wrote some unofficial docs here: https://github.com/ImJasonH/tekton-gke/blob/master/workload-identity.md

We had a call with a Google Tekton SME and they stated more work was required. If it's a case of just documentation then it would be great to get something more official.

@ImJasonH - The issue is not around deployment of pipelines. Currently the installation directions require a kubernetes secret containing GCP service account key json.
We are currently working with inelegant flows where we:

Create a service account
Create a service account key associated with it
Push that key into vault
In a deployment pipeline extract that key from vault, creating it as a file
Pushing that file into a named kubernetes secret
Telling the tekton backend storage ConfigMap the name of the secret

It would be better to be able to give the ConfigMap a service AccountID that has been pre-configured for WLI, and then most of the above workflow goes away.

@ImJasonH For reference, it's this part of the documentation: docs/install.md#example-configuration-for-a-gcs-bucket

Thanks, I'll take a look at those instructions and try to spruce them up to take WI into account.
In the meantime, that configuration is really only necessary to support PipelineResources, which are considered "alpha" and are likely to undergo some changes in the future. You might want to consider not relying on them for now, and instead simulate the same functionality by having a Task that fetches from GCS then a dependent Task that operates on that source, with a shared workspace between them.
The benefit of this is that workspaces don't require any GCS-specific setup, and Tasks should already support WI today, as described in the unofficial docs
cc @sbwsg for artifact-storage and workspaces discussion visibility

( I think @wlynch was looking into this at one point as well )

DID NOT MEAN TO CLOSE 🙏

Until optional workspaces are available, I think you need to fork the gcs-download Task, remove the credential file stuff, then run your Task with the appropriate serviceaccount.

Ah, good catch @coryrc .  You might also be able to work around this by just passing an emptyDir: {} workspace for the credentials workspace.  I haven't tested this but it appears that the Task YAML shouldn't (🤞 ) break if an empty creds dir is provided.

Ah, good catch @coryrc . You might also be able to work around this by just passing an emptyDir: {} workspace for the credentials workspace. I haven't tested this but it appears that the Task YAML shouldn't ( ) break if an empty creds dir is provided.

And if it does, I bet a fix is easy and welcome!

/cc @NavidZ who's been working on https://github.com/tektoncd/pipeline/issues/2316 (which I think is the same issue).

the problem with this approach is if tekton-operator is used to install tekton-pipelines
tekton-operator will create ksa tekton-pipelines-controller and even if you annotate it with google service account, reconciler will remove the annotation


cli(test): support outputting TAP format

Add the --tap flag to compas test to output TAP.

Don't need this at this point. Feel free to open a new issue if you need this with your usecases.


same for membership card no more an array

It is a breaking changes.
This will be deployed later in  August.


WIP CircleOPs

This adds OPs on the Unit circle w.r.t. the weight y^α. I think we get quartic OPs from this.

Codecov Report

Merging #3 into master will decrease coverage by 1.17%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master       #3      +/-   ##
==========================================
- Coverage   36.44%   35.26%   -1.18%
==========================================
  Files           7        8       +1
  Lines         568      587      +19
==========================================
  Hits          207      207
- Misses        361      380      +19

Impacted Files
Coverage Δ

src/OrthogonalPolynomialsAlgebraicCurves.jl
50.00% <ø> (ø)

src/circle.jl
0.00% <0.00%> (ø)

src/hermlaurent.jl
0.00% <0.00%> (ø)

src/wedge.jl
0.00% <ø> (ø)

Continue to review full report at Codecov.

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


[SVCS-249] Purge failed osfstorage uploads from pending directory

Purpose
In osfstorage, a copy of a file that fails to be successfully uploaded to the remote data storage is
left behind in /tmp/pending/ and will continue to take up disk space until manually removed. This PR makes sure that file is deleted before the error is thrown to prevent pending files from accumulating.
Changes
Simple Python magic, and new unit tests.
Side Effects
Partially uploaded files will be deleted.
Ticket
https://openscience.atlassian.net/browse/SVCS-249

Coverage increased (+0.02%) to 75.748% when pulling 485580eda0d9b6527f8e4ed42f367e4eff9e97df on Johnetordoff:purge-pending into 2c5d8c3d30772840436ad20fad95d236b7f55f3a on CenterForOpenScience:develop.

Coverage increased (+0.01%) to 78.576% when pulling 019aa8b78a38fedab021cc0d1744d6abaf1f594e on Johnetordoff:purge-pending into 761c430396730b72e600f8def98c06e4a5d0316b on CenterForOpenScience:develop.

Coverage increased (+0.01%) to 79.358% when pulling 143a929ffe13807ae3211f36632383a151bc8223 on Johnetordoff:purge-pending into 1d9a756f2ff3b87298eacf3e5f86fb53963ace2f on CenterForOpenScience:develop.

Coverage increased (+0.01%) to 79.332% when pulling 9ade8937cca7ea1dc52caae9190bff5d20b93975 on Johnetordoff:purge-pending into 44d3c9a63c1511fd1b8d2fff3c8b8cce0a2c29f6 on CenterForOpenScience:develop.

I keep forgetting to write my top level comment before submitting a review.... anyway
Couple line lengths/formatting changes
One small functionality question relating to exceptions thrown (but probably not super relevant)
LGTM after formatting stuff.
Oh also there is a conflict with develop to fix.

Coverage increased (+0.009%) to 89.128% when pulling a009f2e055b896efa2b64f912f8b98dda6a11fd6 on Johnetordoff:purge-pending into ba5573122292d5baa4d62635cab773278e2ee876 on CenterForOpenScience:develop.

Coverage increased (+0.009%) to 89.113% when pulling a7c007b40d82105ad85a021e27ce4fa7e16946c4 on Johnetordoff:purge-pending into 473191c78c36b6ee63d4609d3952a317ee4ab63e on CenterForOpenScience:develop.

Coverage increased (+0.008%) to 89.95% when pulling f47fe45f9a41a15622b47d129b8e8b04eb4e904b on Johnetordoff:purge-pending into 26bf2093c15af333e634f14372709e7bf014ccb4 on CenterForOpenScience:develop.

Update: consider a rebase on the latest develop as well. Thanks.

Coverage increased (+0.008%) to 89.78% when pulling 5870eb9d4680cf8fb5f75a7f97628841e3457585 on Johnetordoff:purge-pending into 3ec764ae879c107c405d67b1e2d60d1018eafa96 on CenterForOpenScience:develop.

Coverage increased (+0.006%) to 89.917% when pulling e9332955f0c57b5a5e86fb311b042e52fc99b281 on Johnetordoff:purge-pending into 1b60f4017fc7fa52824988d2bc687e070b4d586b on CenterForOpenScience:develop.

Pending secondary PR: https://github.com/Johnetordoff/waterbutler/pull/6


Use GITHUB_OUTPUT envvar instead of set-output command as the latter is deprecated

save-state and set-output commands used in GitHub Actions are deprecated and GitHub recommends using environment files.
This PR updates the usage of set-output to $GITHUB_OUTPUT
Instructions for envvar usage from GitHub docs:
https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#setting-an-output-parameter

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

分支指向 develop 签一下 cla

Sorry, I deleted my repository by mistake which in turned closed this PR. A new PR is here:
https://github.com/alibaba/lowcode-engine/pull/2818

Sorry, I deleted my repository by mistake which in turned closed this PR. A new PR is here:
https://github.com/alibaba/lowcode-engine/pull/2818


Output of all the report coming out corrupt

Hi Bob,
Nice job you have done with this, I have finally installed the solution and getting it to work, but the returns a corrupt document. I'm I doing anything wrong ?

It may be how your Flow is configured.  Are you emailing the document, or saving it somewhere?

@bguidinger emailing it to myself and its the out of the box excel and word docs as well.

@bguidinger

Fine the attached pls. Thanks

Can you try removing one of the "Output" parameters from the "Attachments Content - 1"?  You should only need one.

@bguidinger i had only one there initially. But after about the 10th trial, it produced a good doc. Thanks again Bob.


Trinkets and pull timer

Would it be possible to have a setting where you can unequip trinkets before pull so you don't instantly proc some trinkets. Thinking about Comets Trail and getting it to proc after gargoyle has been summoned so you would gain the most benefit from the proc. I don't know much about sims and would this really even work.

Duplicates #2007 #1664


Domain operations

Hi @jorge07!
Your boilerplate is fantastic! I have a question regarding the domain operations.
For example, after UserWasCreated, who should be responsible for sending a notification (email, push, SMS) after the persistence layer successfully finished his work? Should we have the Handler class that will work with the event bus as a part of the domain layer, which will expect the UserWasCreated event in the __invoke method? I spied this approach in botilka demo
Also, it would be nice to add more advanced examples to the boilerplate. Currently, I am working on investigating DDD for my organization and I might contribute if you don't mind.
Thanks!

Hi @mxkh
Your approach makes totally sense to me in terms of workflow.
An event listener class / interface that implements/extends Broadway\EventHandling\EventListener in your domain layer and the implementation for the transport (mail, push, sms...) in infrastructure. Keep in mind I use Broadway and method to implement here is handle instead __invoke as you may expect but there're a reason and it's in the read model event handling side: https://github.com/broadway/broadway/blob/master/src/Broadway/ReadModel/Projector.php#L36
Happy to add more advanced examples and improve documentation. Open for suggestions 👍

The component Messenger of Symfony isn't sufficient ?  Because it do that  no ?

@xfifix the messenger component is used for the Command, Query and AsyncEventBus but Broadway has his own even bus too but ONLY for synchronous tasks. So yes, you're right and I miss the topic here that if you want to to this in an async way you've to implement this interface and will be automatically executed in the workers
That' makes me realise there's a lot of room for improvement in the documentation. Thanks!

@jorge07 Thanks for your reply! It was very helpful!
So, if I want to send a notification where I should do that according to your boilerplate? I really want to clarify it to myself. Should I work with the read model events? Because I want to make sure that the persistence layer was finished successfully and only then send UserWasCreated notification to some transport

You've an aggregate root call User, when the user is created it generates a UserWasCreated event. Then it get persistent in the write model (events gets persisted). Then this event it's published in the event bus and you've some projections like the UserView in this repo that generates the read model. This projection handlers extend Broadway Projection so they will be executed synchronously. You can put your SMS handler on the async event bus to guarantee it gets extended when read model is already applied. Otherwise you'll need to deal with eventual consistency

You've an aggregate root call User, when the user is created it generates a UserWasCreated event. Then it get persistent in the write model (events gets persisted). Then this event it's published in the event bus and you've some projections like the UserView in this repo that generates the read model. This projection handlers extend Broadway Projection so they will be executed synchronously. You can put your SMS handler on the async event bus to guarantee it gets extended when read model is already applied. Otherwise you'll need to deal with eventual consistency

Thanks for the detailed answer! Your words confirm what I was thinking about but I wanted to get proof of my thoughts 😅  Thank you a lot! I am going to fork your boilerplate and add more examples.

I'm refactoring few things and playing with advance things with symfony messenger.
It's all in #187 PR
There's an example of how to deal with async events here with some notes about the things i found and work around
https://github.com/jorge07/symfony-5-es-cqrs-boilerplate/pull/187/files#diff-7a5e0582aaf2f0e794df52345439b794


Bug when resetting game, the grade stays the same after resetting the game

Hey, I noticed after you reset the game that you get the same grade from the last game you play.
-You play the game until the monster comes
-Then you reset to the main menu and play the game again
-The same grade you have from the last game you played is still there, instead of having a 100% as the grade
You can fix if you have time

Rachelle Badua

Fixed it

Nevermind, Doreen wasn't able to fix it and now the whole scoreboard isn't working.

Nevermind, fixed it again


CMake gets rebuilt for cross because of run-time dependency on pkg-config

I try to do some weird cross and notice that cmake is being rebuilt. That's odd.
Here's why:

pkg-config now has a target-specific wrapper.
CMake has long had a run-time dep on package config

So either we give CMake a wrapper too (ugh...) or we just remove pkg-config from buildDepends of CMake. I learn towards the latter.
CC @matthewbauer and @LnL7, who I think added the dep?

I only made some dependencies optional at some point to reduce the number of things needed to bootstrap. What's the issue exactly? I you mean moving pkgconfig to nativeBuildInputs the that sounds fine to me.

What's the issue exactly?

CMake is rebuilt when it's target platform changes..

If you mean moving pkgconfig to nativeBuildInputs then that sounds fine to me.

Oh, I assume it had some nativeBuildInputs and so it was in buildInputs in purpose, but maybe it wasn't!
Also, do we know what the setup hook is a buildInput?


replace oc with kubectl

replace oc with kubectl as it is erroring out in non OCP envs

https://github.com/ibm-apiconnect/v10-postmortem/issues/65

@geraintwjones any chance you can merge this - I do not seem to have access -we keep getting pmrs on this.

@louiseohalloran Done.

Thanks @geraintwjones


Question: is it possible to read modbus data over wifi?

hi there @CJNE ,
thanks for this integration :).  i have been using it very successfully for nearly two years now.
my solar edge inverter is currently using a wired connection to my router.  however, i had to make a change to this and connect the solar edge inverter  to my wifi instead - eveything was configured to remain the same (IP address, modbus ports etc).
in doing so, i was not able to read the modbus data in home assistant.  so i reverted back to the wired connection.
is this expected behaviour for all inverters or perhaps simply a limitation of my own?
many thanks.

Hi there!
This is a bit hard to tell, from the home assistant side of things it shouldn't make a difference wether the connection is wired or WiFi.
Perhaps your home assistant box don't have access to the WiFi?
Or maybe the inverter doesn't support modbus over the WiFI connection, that's also a possibility.

Thanks for the response. I think it's a limitation of the inverter. Will close case.


A contextual approach towards more accurate duplicate bug report detection

Paper link: http://ieeexplore.ieee.org/xpl/articleDetails.jsp?arnumber=6624026

Data from bug reports of proprietary softwares


Update README.md

Update MySQL server download information under the installation in readme file

As discussed, @ananafrida will add the point about turning off the VM and @PattonYin will review this PR.


RTSP "No supported authentication protocol was found"

I tried to integrate to test gstreamer with (https://github.com/awslabs/amazon-kinesis-video-streams-producer-sdk-cpp) through RTSP from openipc but I got "No supported authentication protocol was found" issue as follow log. The VLC player worked well for the RTSP endpoint. Do you know why below log raised? RSTP server of openipc firmware needs some changes or configuration update?

[INFO ] [27-06-2024 07:08:42:185.345 GMT] Streaming from rtsp source
Error received from element source: Could not open resource for reading.
Debugging information: ../gst/rtsp/gstrtspsrc.c(6427): gst_rtspsrc_setup_auth (): /GstPipeline:rtsp-kinesis-pipeline/GstRTSPSrc:source: No supported authentication protocol was found

I found the root cause. Mejestic UI show the endpoint as default password and I just copied it but I changed the password..
<EMAIL_ADDRESS>Please close this issue.


All home-related pending logic

This mainly focuses on stitching up all the data-related operations with the pending home functionality (that was in TODO with mocked data).

Codecov Report

:exclamation: No coverage uploaded for pull request base (collection-execution+services@e0bbab3). Click here to learn what that means.
The diff coverage is n/a.

@@                       Coverage Diff                        @@
##             collection-execution+services      #49   +/-   ##
================================================================
  Coverage                                 ?   70.46%
================================================================
  Files                                    ?       22
  Lines                                    ?      413
  Branches                                 ?        0
================================================================
  Hits                                     ?      291
  Misses                                   ?      122
  Partials                                 ?        0

Continue to review full report at Codecov.

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

All updates made and merged in the last PR (#56) in this nested chain of collection execution PRs


Add ability to expose CSP nonces/hashes to server hook for header based CSP

Describe the problem
I cannot have a dynamic environment variable control a CSP directive.
This is needed to configure the CSP on startup of the sveltekit app in a docker container, the image cannot compile the application on the deployment machine because it has a relativley weak cpu and will take a while to spin up the instance then.
This problem makes it so that people that want to self-host the frontend either need to completely re-build it and re-publish their own docker image, or the project needs to get changed to not support inline script injection protection.
Describe the proposed solution
It would be really nice to be able to get the CSP hash/nonce for a intercepted page in the hooks.server.ts to be able to append those to the header of the response.
Alternatives considered
Considered reading the file in the hook, locating the CSP meta element, reading it, removing it, and then adding the hash to the header.
Importance
i cannot make my SvelteKit app selfhostable without it
Additional Information
No response

This is also useful for other types of CSP manipulation.
form-action could be restricted to the server form actions of the given page.
A Typebot widget e.g. TypeBotIntegration.svelte could add dynamic values to the page only if it's rendered, so only those pages can load a Typebot that have the widget component rendered.
So, there could be 2 levels of configurability: route based (e.g. if route is /posts/, allow loading certain resources), or component-based, e.g. (add this to the style-src if this component is rendered anywhere on the page).


Taskbar Groups

Description of the new feature / enhancement
It would be great to have a wait to group multiple icons/apps in the taskbar (similar to what we can do on the start menu
I've been using this free app to achieve this (https://github.com/tjackenpacken/taskbar-groups), but it would be a nice feature to add natively to windows
Scenario when this would be used?
Daily as a way to organize apps and have everything grouped together in "themes" or "interests" like Games/Tools/Browsers etc
Supporting information
https://github.com/tjackenpacken/taskbar-groups

/feedbackhub


Remove legacy notifications sending

Since new way of sending notification is through GDS notify, the configuration in Mail Notifier seems redundant. This needs validating in Integration.
Test is renamed so it runs in CI, previously CI was not picking up this test since it wasn't named correctly.
https://trello.com/c/r8eG9T6o/2240-remove-bootstrap
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
Follow these steps if you are doing a Rails upgrade.

Annoyingly, would like to remove the default email default from<EMAIL_ADDRESS>, but the tests fail on the error SMTP From address may not be blank: nil
Will test this branch on integration to make sure email notifications still happen

Tested this in Integration with Laura - the email sending still works and looks the same as previous


No Internet Access, SimpleDNS crash, service not starting.

I'm frequently losing network access with error: No Internet Access - your device is connected and can access other devices on your local network, but you may not be able to reach the internet.
When I open SimpleDNS I find the resolvers page empty and after a few seconds SimpleDNS becomes unresponsive and crashes. Other times I have found the DNSCrypt service fails to start when Windows starts. I use Malwarebytes Firewall Control and don't know if the issues I'm experiencing are linked.
Can anybody help with this?

did u tried disable malwarebytes firewall and see if solve ur issue?
i use malwarebytes but i dont use their firewall and no problem with resolver
try take a look :\Program Files\bitbeans\Simple DNSCrypt x64\dnscrypt-proxy
public-resolver.md and public-resolver.md.minisig is updated quite often
there is possibility malwarebytes blocking access

did u tried disable malwarebytes firewall and see if solve ur issue?
i use malwarebytes but i dont use their firewall and no problem with resolver
try take a look :\Program Files\bitbeans\Simple DNSCrypt x64\dnscrypt-proxy
public-resolver.md and public-resolver.md.minisig is updated quite often
there is possibility malwarebytes blocking access

Malwarebytes firewall has a feature called secure boot which basically enforces inbound and outbound connections being blocked until the profile is changed to a less restrictive profile. I suspected this may be the issue but I have sometimes randomly lost internet access even when inbound connections are allowed. I'm unsure what to look for in the public-resolver.md... Are there any logs I can look for? I don't know if you can disable MB firewall but I will try disabling the start automatically at user logon option.

in malwarebytes dashboard, you should see Real-Time Protection
so just disable web-protection (firewall) and leave other (malware, ransomware, exploit) enable
you can open public-resolver.md with notepad and you can see the content
but you cant freely edit it, as it will verify the file with .minisig
public-resolver.md content basically same to this : https://download.dnscrypt.info/dnscrypt-resolvers/v3/public-resolvers.md
the things i am not sure is to where simplednscrypt updating that, as there are few url available to fetch those
in my mind, probably we can whitelist those url in malwarebytes so it can fetch/update

Malwarebytes WFC is a separate program to Malwarebytes. There is no real time protection. It is like an alternative GUI and way to control the Windows firewall. More info here: https://www.binisoft.org/wfc.php
site is currently down

i thought you using "malwarebytes"
based what i see on the sites, even it using malwarebytes names, it completely separate from malwarebytes
with those firewall control, then u need extra work with its filtering to work fine with various programs.
for me, its just cause troublesome and not really worth putting time setting those, thus i disable web-protection in malwarebytes
few things to note is:

dnscrypt -> need to be allowed to send and receive connection freely
dns-requests from various server listed in public-resolvers (or those u select/using)-> probably also need to be allowed

I've recently installed Windows 11 and are trying the included DNS over HTTPS built in Win 11 instead of SimpleDNSCrypt. I'm occasionally still getting unidentified network so I think this has to be an adapter issue or something.

I used simpleDnsCrypt for more than a year and managed to setup a working configuration with DoH server.
However, starting from today i cannot start dnscrypt service when ethernet adapter is enabled. If i try to manuall start it from services.msc, i get error "1067: The process terminated unexpectedly". If I disable the ethernet adapter, the service starts but of course I cannot bound it to that interface.
My wild guess is that a recent windows update broke something...

Update: I moved to Acrylic DNS Proxy
This project looks dead.

fusionneur Looking at  Acrylic DNS Prox as well. Ideally would like to config it, DNSSEC using DNSCrypt? or DOH?. To be as secure as Simple DNS encrypt.

I use cloudflare's doh. Acrylic's documentation is done quite well and if read carefully, you can set it up in less than a couple of hours. Basically you will want to edit the ini file which guides you through the setup.

I used simpleDnsCrypt for more than a year and managed to setup a working configuration with DoH server.
However, starting from today i cannot start dnscrypt service when ethernet adapter is enabled. If i try to manuall start it from services.msc, i get error "1067: The process terminated unexpectedly". If I disable the ethernet adapter, the service starts but of course I cannot bound it to that interface.
My wild guess is that a recent windows update broke something...
Today Windows update downloaded but for some reason failed to install:
Update for Windows 10 Version 21H1 for x64-based Systems (KB4023057)
After reboot, dnscrypt stopped working

i  got KB4023057 update back on 5th august and got no issue with dnscrypt at this point (https://imgur.com/vDMk9Tq)
probably try update ethernet driver

The problem can be solved by using dnscrypt-proxy+proxifier. The application can break down because once the application start, the dnscrypt-proxy will download server list files through system default dns first. If your dns recordings have been polluted,such as you are in the china, dns will return a fake ip, leading to dnscrypt-proxy breaking down. You should open proxy client first, then use proxifier to proxy the network flow of dnscrypt-proxy, then shut down proxifier, the dnscrypt-proxy will work correctly.

I doubt that is the case.. I never had the internet connection censored, but it still caused me those issues. Using acrylic for almost a year now with no issues.


Replace eslint-config-standard with standard

Is there a reason why koajs and current/future middleware can't all use standard instead of eslint-config-standard?

@rynz
i guess it's just a traditional style guided by @tj with a long history, maybe @dead-horse can tell u why.

With standard you can´t use custom .eslintrc files needed for example jest lint rules: https://github.com/koajs/koa/blob/master/test/.eslintrc

every project has their own style guide, we just need to ensure the code style is unified in this project.


[DCOS-14199][1.10] Write strings to files atomically in pkgpanda.util

High-level description
This is a backport of https://github.com/dcos/dcos/pull/2730 which has a Ship It label
This changes pkgpanda.util.write_string to avoid a race condition as described in DCOS-14199.
I have also applied this change to some related functions as a precaution.
I have stopped returning data from the functions as it appears that those return values are never used.
This also adds a test for write_string to enshrine the permission requirements and basic functionality of write_string (thanks @mhrabovcin for help). That test passes on master and with the changes here.
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS-14199 exhibitor.py try_shortcut() failed due to an empty file

Checklist for all PRs

[ ] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change: See https://github.com/dcos/dcos/pull/2730#issuecomment-380506106.
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here: This code is already called by the test suite. We could test the race condition with something like the following with a timeout but I'd rather not. I'm happy to discuss this with a reviewer:

# This will trigger the error from the issue if run by two processes in parallel.
# Calling the new function in the loop will not.
filename = 'foo.txt'
while True:
    with open(filename, 'w+') as f:
        f.write('hello')

    with open(filename) as f:
        stripped = f.read().strip()
        assert stripped != ''

[X] Read the DC/OS contributing guidelines
[X] Followed relevant code rules Rules for Packages and Systemd

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label Ready For Review


CVE Reported in Version v1.28.7

CVEs found in v1.28.7
Our Security Scanning tools have identified CVEs in v1.28.7. Can you please review this and help us with an update on following:
Documentation that explains the mitigation strategy that we can apply to reduce the severity level
Details on when is this going to be fixed with the expected version number and if its already fixed which version number is it fixed in.
Issues found in build: v1.28.7
CVE-2022-41717
CVE Description
An attacker can cause excessive memory growth in a Go server accepting HTTP/2 requests. HTTP/2 server connections contain a cache of HTTP header keys sent by the client. While the total number of entries in this cache is capped; an attacker sending very large keys can cause the server to allocate approximately 64 MiB per open connection.

If the prometheus metrics server is not running, the operator will not listen to HTTP requests at all. Prometheus metrics are disabled by default, so I believe you should be good. (See main.go)
This should be addressed in the next release. This commit updates the go version to v1.18.9: https://github.com/tigera/operator/commit/4ef0c6205e00df1edebaa5212efe69a16e50f003
cc: @Behnam-Shobiri


OpenPGP: Warn on bad subkeys

Had some trouble running engine test locally, will depend on CI.
@maxtaco @oconnor663

OK I tried to make it like Max suggested here: returning Warnings, contextifying Warnings.

One more try. Not contextifying Warnings after all. Added a bad subkey test.

:+1:


Version 0.12.0?  used 12 because 11 was last used.

Based off of 0.8.0, this version successfully implements the Linux special keys including the ESC.

added the TAB character, which was missing.

For both OS types, the new readkey() now wraps a get_char method, which is separate from the readchar methods (which only return a character byte).  The get_char handles the reading need to get the full keystroke codes.
for Linux:

At the heart of this is a new reachar_linux.readchar() that successfully implements a real blocking bahavior, allowing us to get all queued up characters after an ESC.
This means if Escape was the only character, we can detect that now.
Users can also use the Linux (and Windows) versions to call readchar() in a non-blocking manner (the default is to block, i.e. wait for a keystroke to be typed).

for Windows:

I re-implemented Guiweber's windows methods, with blocking, however I don't have a windows testing platform to test or improve this section.
It differs from his version because the readkey() returns the standard constants from key.py that are used by Linux versus the originally proposed string values (so separate comparisons are not required between OSes).
for convenience, key.py contains a conversion function convertchar() to convert these back to string names like "ctrl_a", if desired.
The constants for windows will need to be updated and improved due to being incomplete.  New items should only be added if we can create a valid byte-string constant for them.

Gonna close this until I find the problem.

I think the issue is that the tests are not reading characters after the \x1b, even though they actually do read on my machine:
>       self.assertEqual(char, result)
E       AssertionError: '\x1b[3^' != '\x1b'
tests/unit/test_readkey.py:53: AssertionError

On these travis builds, the code is not obtaining the non-blocking reads, I believe:
        if charbuffer in key.ESCAPE_SEQUENCES:
            char1 = readchar(False)

Two of these failing keys, I can't even test:

'\x1b\x5b\x33\x5e` (ctrl- alt -SUPR/DELETE) will attempt to log me off my system.
'\x1b\x01` (ctrl-alt-a) only produces a x01 on my system (same as ctrl-a).

\x1b\x5b\x41 works in my code for key.UP:
    x = readchar.readkey()
    if x == readchar.key.UP:
        print("terminated.")
        break

press UP:
terminated.
surest@surest-HP700-047c ~/github/readchar $

(No other key breaks the loop)

I have given up my tests to attempt to get this to pass Travis checks.  I suspect that the way my (and guiwebers) code operates, it cannot be tested using the provided mechanism.  This is because my method reads the ESC key and Sequences not as separate strokes, but a "sequence".  When my code sees an ESC, it immediately collects any available pending characters.  if there are none, this signals (in a "real" terminal) that the key was simply an "ESC"; nothing else.  If it was an escape sequence, the buffer will have the remaining (however many) codes.  However, the test routine "feeds" items one-by-one.
I think if you test this in an actual terminal, you will find they work properly.

The only alternative is to attempt to do it the way curses does, by implementing a "timer" that will only acknowledge a single escape key after a "timeout" (usually a second or so).  If more characters enter the buffer in that timeout period, then they are added to the escape sequence (which can be annoying to a user who must wait a full second after pressing the ESC key to see it acted upon..)

I flat out do not get this!  it says the function (in the tests) is returning a \x1b:
    def test_special_key(self):
        char = '\x1b\x5b\x41'
        getchar_fn = readchar_fn_factory(char + 'foo')

        result = readkey(getchar_fn)

>       self.assertEqual(char, result)
E       AssertionError: '\x1b[A' != '\x1b'

This is flatly not possible because I just wrote a line where it will not return a key that is just \x1b!:
            if (charbuffer + char1) != '\x1b':
                return charbuffer + char1

@magmax .. Can we discuss??  I found the problem.  The test suite is substituting its get_char function for the actual function in our code... which works when YOUR readkey was processing the keystrokes (not real keystrokes, but just a stream of values).  As a result, when the test suite passes "\x1b ....", the stream returns x1b and the factory function returns that to the test code.  The internal functions for get_char for linux or windows never get run.
This test suite does not properly test our code.

If you don't like this whole thing, feel free to close it.  I admit it is a bit of a mess and maybe you want to start from scratch with what you were doing.  I don't like the hacks I implemented around the Travis testing, which created "passing" build, but horrible code...
However, on my own project, I did do a lot of windows testing and mapped most of the windows keys.  I would suggest you take advantage of the mappings I made for windows in my other repo:
https://github.com/suresttexas00/readkey

I see you already realized about the mess we have to do here. But I think it is better to do the mess here than in a lot of projects.
I started this project thinking it was much easier, but... well, now I am even more happy to have started it, despite maybe it is not the best solution in the world.
Thank you very much for your contributions. I will have a second look as soon as I can.

Not sure where we are with this now... Did you want me to make these changes first, or did you want to approve the pull request and then make the changes yourself?

I've changed the way to use versions. Do not care about them any more.
This PR contains a lot of changes. Please, use smaller ones.

Wow, I had forgotten about this.  I don't use it anymore and I have no idea where I left off, code-wise.  I won't be issuing any more PR's at this time.


Feature Request: Remove \ when 2 spaces in documentation or comment

Hello,
I came across some robot code which had the following construction in documentation and/or comments
[documentation]  this function \ adds two numbers, and return  \ sum
The only reason these \ are here is to prevent that part of documentation (or comment) is seen as an extra argument. I think in 99.9 % of the cases only a single space is required(and the \ is obsolete).
There could be a transformator(RemoveObsoleteSpacesInDocumentsOrComments) that removes these \ signs. Making documentation and comment more readable.
In the case the backslash is required, there is already some mechanics in place to skip transforming files. The only problem remains when 1 file contains 2 keywords for instance. In 1 method there is a required \ character, and in another method it can be removed.
What do you think ?

I Would like to add, this is a minor issue in the code.
Other transformers should get attention first. No hurry at all

Thank you for feature request. It looks simple to implement. I will do some research to see if it will be easy to find cases when \  is necessary and when \  is not needed and can be removed.


Fix: Remove dependency on unmaintained atty

atty is unmaintained for a while. And GitHub dependabot says there is a detected vulnerability in it.
So let's use new IsTerminal trait from stdlib instead

LGTM. Thank you for the contribution!
Best Regards
Mydayyy


Instructions: Use anonymous clone URL, use pip3

pip install git+https://github.com/PaulMcInnis/JobFunnel.git is standard for packages not yet uploaded to PyPI.
See here: https://adamj.eu/tech/2019/03/11/pip-install-from-a-git-repository/
Not every user wants to clone this package as it is meant to be installed via pip.  Hence the MANIFEST.in.
Perhaps we should look into uploading the package to PyPI.
As for pip3 vs. pip this is often disputed online.  Some think python -m pip install, python3 -m pip install, pip3 install, pip3.7 install.  I'd rather not get into the specifics.  The readme.md indicates Python 3.6 or later in it's dependencies.

Readme was updated recently to use https,
Does
’pip install git+https://github.com/PaulMcInnis/JobFunnel.git’
suffice?

It only wanted to change it so that you don't need to have a GitHub account with an public ssh key to clone the repo.

python -m pip install

This load the pip module installed for python2.

python3 -m pip install

This load the pip module for python3 only.

pip3 install

This is basically a wrapper script which load the pip module for python3.
Content is:
import sys
from pip._internal import main
if __name__ == '__main__':
sys.exit(main())

pip3.6 install

This does not exist under Debian so I don't really care. There is python3.6 which only load Python 3.6.X

I'd rather not get into the specifics.

There is nothing to get into. If you use a proper Python distribution that follows PEP 394 pip3 is always the pip for python3 and pip is always the pip for python2.

The readme.md indicates Python 3.6 or later in its dependencies.

I can bet with you that a not to little amount tries  to launches/install this project with python2 cause pip is under normal circumstances python2 not 3. I know one exception and that is conda which also does not create a python3 binary so it is non standard.

What if we add a version check for python?


ip route missing

Rancher versions:
rancher/server:v1.6.5
rancher/agent:v1.2.5
Infrastructure Stack versi.ons:
healthcheck:v0.3.1
ipsec:
net: v0.11.3
net: holder
network-services:
network-manager:v0.7.4
metadata:v0.9.2
dns:v0.15.1
scheduler:
rancher/scheduler:v0.8.2
kubernetes (if applicable):
Docker version: (docker version,docker info preferred)
1.12.6
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
ubuntu 16.04.2 LTS
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
VMware Fusion
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node rancher
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
Results:
When Executing command to add Rancher-agent node, the error prompted: Failed to connect to xxx.xxx.xxx.xxx port 8080 :No route to host
I executed the command （ip route）to see the routing table, No routing information about docker0 was found. When I restart host, the problem will disappear

@hongxiaolu
The routing table for docker0 is controlled by docker itself.
Agent registration is running in a docker container, if docker0 routing table is missing, agent process cannot access rancher server.
Please find the reason for docker0 routing table missing, when you have a chance.

thanks!


url bar does not default to safe://

Operation System: Linux
Beaker Version: safe-browser-v0.3.5-linux-x64
urls appear to still require safe:// is typed in the address, rather than just defaulting to that.
Also the list of favorites makes no sense, since they are not accessible as http://

yup. Both on the list 😄
favourites will come with safeSync, as I'll be overriding this behaviour entirely.
safe:// as a default should be something I can drop in a little sooner, I think.


Builds fail on Windows with 2.089 and later

At least unittest builds.
ERROR: This is a compiler bug.
Please report it via https://issues.dlang.org/enter_bug.cgi
with, preferably, a reduced, reproducible example and the information below.
DustMite (https://github.com/CyberShadow/DustMite/wiki) can help with the reduction.
---
DMD v2.089.1-dirty
predefs   WithAdminPlugin WithAutomodePlugin WithBashQuotesPlugin WithChanQueriesService WithChatbotPlugin WithConnectService WithCTCPService WithHelpPlugin WithNotesPlugin WithOnelinersPlugin WithPersistenceService WithPipelinePlugin WithPrinterPlugin WithQuotesPlugin WithSedReplacePlugin WithSeenPlugin WithWebtitlesPlugin WithPlugins Colours Web TwitchSupport WithTwitchBotPlugin TwitchWarnings OmniscientAdmin PrintStacktraces ExplainReplay ProfileGC AssertsGeneration Have_kameloso Have_arsd_official_dom Have_dialect Have_lu Have_requests Have_arsd_official_characterencodings FlagAsUpdated Have_lu_core Have_lu_meld Have_cachetools DigitalMars Windows CRuntime_Microsoft CppRuntime_Microsoft LittleEndian D_Version2 all D_SIMD D_InlineAsm_X86_64 X86_64 Win64 D_LP64 unittest assert D_ModuleInfo D_Exceptions D_TypeInfo D_HardFloat
binary    C:\dmd2\windows\bin\dmd.exe
version   v2.089.1-dirty
config    C:\dmd2\windows\bin\sc.ini
DFLAGS    -IC:\dmd2\windows\bin\..\..\src\phobos -IC:\dmd2\windows\bin\..\..\src\druntime\import -L/OPT:NOICF
---
core.exception.OutOfMemoryError@src\core\exception.d(647): Memory allocation failed
----------------
0x0063839A
0x00598637
0x0059D79C
0x0058DFE5
0x00523E3C
[...]

Currently attempting reduction.

Bisected and reported.
https://issues.dlang.org/show_bug.cgi?id=20562

This only affects dmd. The workarounds are to either use LDC, or to build with --build-mode=singleFile. Neither can be enforced in a build configuration; they are only opt-in at the command line. As such we can only warn when the compiler is dmd and __VERSION__ is 2089L (2.089.0 seems to be the closest tag) and hope the user does the right thing.
DFLAGS=-lowmem does not currently work but I have not yet reported it upstream. It could be enforced in dub.sdl, it just needs to work first.

Fixes for AppVeyor were pulled in 9163c96a9da6d571958bff24f800edadb7f2e015. It also includes compile-time messages suggesting to use singleFile. We may have to promote it further in the readme.
Leaving this bug for visibility but it's technically a NOTOURBUG.


[CLI] wand.run.dir points to root (/) when running in disabled mode

When wandb mode is set to disabled the wand.run.dir attribute points to /.  Is this by design or is it overlooked?
I mostly use wandb.run.dir to store artifacts and saving them to root is almost never what I want (and fails due permissions anyway) Doesn't it make more sense to fallback to whatever was passed to wand.init()s dir argument or CWD?
Environment

OS: Linux 5.10.52-1
Environment: Conda
Python Version: 3.8.10

Hey @mil-ad when running in disabled mode we create a mocked run object that returns dummy data.  Each run actually creates it's own sub directory within the dir specified to wandb.init.  Because the run is disabled, we do not automatically create this directory.
I've created a PR that sets this to the generated directory, however that directory won't exist so if you wanted to use it you would need to create it first.
Can you share a little more about your usecase?  What's the scenario where you're completely disabling wandb and still reading writing to the wandb directory?  It could be that mode="offline" is better suited to your usecase.

I think it makes sense that wandb is not creating that dir in disabled mode, I just think that returning / makes this hard to catch as it's a valid path. Perhaps returning None makes more sense?

Hmm, I don't like the idea of dir not always being a string.  What if we set it to the system temp directory?

I mean, if I've specifically disabled wandb then should there even be a wandb.run object? let alone the dir attribute.  I understand that internally you might want to have a shim thing with dummy data but in terms of what gets exposed to the user it might not be the best thing.

The primary usecase for WANDB_MODE=disabled are users testing their ML projects in CI or CD.  In this case they want all of their code to execute properly and must depend on core wandb sdk methods to continue functioning, wandb.log, if wandb.summary["acc"] > 0.5 etc.  If we didn't return a run object at all or broke any of our existing SDK methods the user code wouldn't be able to run.

Thanks for the explanation -- that makes sense. I think in case of wandb.run.dir it's difficult for the code to continue functioning unless you return something in /tmp

Cool, I've changed it to use tempfile.gettempdir() and will release next week.


fix(gha): official input is a string

Summary
GitHub Action inputs are always strings.
So this comparison, was "(string) == boolean" which was always false. This is just part of the release process we don't exercise often.
Sister EE PR: https://github.com/Kong/kong-ee/pull/4627
Checklist

[ ] The Pull Request has tests
[ ] There's an entry in the CHANGELOG
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Full changelog
Issue reference

Closing in favor Tyler's fix.


gtkmm3: 3.24.7 -> 3.24.8

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for gtkmm3 is: C++ interface to the GTK graphical user interface library
meta.homepage for gtkmm3 is: https://gtkmm.org/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.24.8 in filename of file in /nix/store/h89y3w477djj780vgzw0cz3bxx4s4kwg-gtkmm-3.24.8

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

89 total rebuild path(s)

89 package rebuild(s)

First fifty rebuilds by attrpath
abaddon
adapta-gtk-theme
adriconf
arc-theme
arx-libertatis
asciidoc-full
asciidoc-full-with-plugins
astroid
ayu-theme-gtk
blueberry
capitaine-cursors
cherrytree
cinnamon.cinnamon-screensaver
cinnamon.iso-flags-png-320x420
cinnamon.iso-flags-svg
clevis
coreboot-configurator
dablin
dblatexFull
disorderfs
emojione
fim
gImageReader
glom
gnome.gnome-system-monitor
gnote
gobby
goocanvasmm2
gparted
gsmartcontrol
gtkmm3
gtksourceviewmm
gtksourceviewmm4
gtkspellmm
guitarix
hackneyed
hikounomizu
horizon-eda
hypr
inkscape
inkscape-with-extensions
iso-flags
jstest-gtk
k40-whisperer
kabeljau
kakounePlugins.rep
lifeograph
litebrowser
litemdview
luksmeta

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/288c66cca7429b5f6c1c97a165107a2ac4b6d815#gtkmm3

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

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 45m and timed out

Maintainer pings
cc @7c6f434c for testing.

@ofborg build inkscape


Allow per question feedback even the whole question is hallucination

if allowing per question feedback when whole question is hallucination, do we still want to mark the question as completed when the question is marked as hallucination? If not, update here:
https://github.com/datacommonsorg/website/blob/17d68b742e5b4cd26adc08d57020f02baee268bd/static/js/apps/eval_rig/eval_list.tsx#L55-L59

I see. Removed that check now.


Implement Cassandra DB & PHP Extension

I completed the 3 steps below:

[X] I've read the Contribution Guide.
[X] I've updated the documentation. (refer to this for how to do so).
[X] I enjoyed my time contributing and making developer's life easier :)

This PR implements the feature of having the cassandra database system and also the PHP driver.
Find the updated .env Variables:
WORKSPACE_INSTALL_CASSANDRA=false
PHP_FPM_INSTALL_CASSANDRA=false
PHP_WORKER_INSTALL_CASSANDRA=false
CASSANDRA_VERSION=latest
CASSANDRA_TRANSPORT_PORT_NUMBER=7000
CASSANDRA_JMX_PORT_NUMBER=7199
CASSANDRA_CQL_PORT_NUMBER=9042
CASSANDRA_USER=cassandra
CASSANDRA_PASSWORD_SEEDER=no
CASSANDRA_PASSWORD=cassandra
CASSANDRA_NUM_TOKENS=256
CASSANDRA_HOST=
CASSANDRA_CLUSTER_NAME="My Cluster"
CASSANDRA_SEEDS=
CASSANDRA_ENDPOINT_SNITCH=SimpleSnitch
CASSANDRA_ENABLE_RPC=true
CASSANDRA_DATACENTER=dc1
CASSANDRA_RACK=rack1

I chose to use the Bitnami Image instead of the official image, because you can easily set a default user & password.
I hope everything is fine and i didn't forget anything. I also found in correlation with other DB Engines (RethinkDB, CouchDB, etc.) that there are no explicit guides to how install them in the documentation. So, FYI, to use cassandra just use docker-compose up -d cassandra.

I noticed that CI failed, because packagecloud.io fails at apt-get update, which has nothing to do with my changes.


Any kotest/pitest version combination above 5.3.2/1.8.1 crashes

Any version combination above Kotest 5.3.2 and pitest 1.8.1 crashes with Exception in thread "main" org.pitest.util.PitError: Coverage generation minion exited abnormally!.
Here is a small reproducer, if you run the pitest task as is, it works, but if you try to bump the versions on line 12 or 26, the task fails.
pitestPlusKotestBug.zip

@SeekDaSky could you try using hte latest snapshot with kotest 5.5?

Tried it with the snapshot <IP_ADDRESS>-SNAPSHOT and kotest 5.5.4. The linked reproducer is still throwing the same error sadly.

The extension is now built to use Pitest 1.9+, which also removes support for Pitest 1.8 and below. Snapshot build out

1.2.0 released with a fix. Please re-open the issue of it persists.

Confirmed that snapshot <IP_ADDRESS>-SNAPSHOT with pitest 1.9.0 and kotest 5.5.4 works now, thanks a lot 🎉


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 11:45:29 on 2023.08.18
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 4, errors out: 0
Original problem ID: 85067
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=85067

Problem has been resolved in 21m 0s at 12:06:29 on 2023.08.18
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 85067


Fix broken link in blog article

Remove some spurious Unicode code points [preview]
My version of PR #32684
/area blog
/priority important-longterm

Duplicates #32686
OK to merge, but not needed.
/close


Date.prototype.toJSON has incorrect return type

Bug Report
Date.prototype.toJSON() can return a string or null according to the spec https://tc39.es/ecma262/#sec-date.prototype.tolocaledatestring. For example new Date("").toJSON() returns null.
Currently, the return type of toJSON() is specified as a string:
https://github.com/microsoft/TypeScript/blob/131875bb849c0a9c56c26d09881453aace0cbfed/lib/lib.es5.d.ts#L879
🕗 Version & Regression Information
Typescript 4.2.3
💻 Code
new Date("").toJSON().slice(0,5);

produces an error at runtime since new Date("").toJSON() is null, but doesn't produce a typescript error because new Date("").toJSON() is assumed to be string.
🙂 Expected behavior
TypeScript should warn about the potentially null value when an arbitrary string is passed into Date. However, if the Date object can be guaranteed to be valid (for example new Date()), the return type of toJSON() should be string.

You’re right, but making these kinds of changes often breaks a lot of people for not a whole lot of payoff. (People may have other validation systems in place that ensures their Dates are valid, which would make it a huge pain for them if all of a sudden they had to pretend that all their Dates might not be valid.) This is the sort of thing we want to hear a lot of demand for first.

Is there a supported way for overriding the built-in types that have this sort of issue?

Unfortunately, there’s not a super clean, easy way to make a built-in type wider than it’s already declared 😕. This sort of thing makes me want to revisit the idea of having “strict” variants of lib files that people could opt into with the lib compiler option, so we could make these kind of changes without breaking the world.
I guess the easiest option I would recommend is just writing a wrapper function around built-ins whose types are too loose, and always use your wrapper functions instead of the built-ins (you could enforce this with an ESLint rule):
export function dateToJSON(date: Date): string | null {
  return date.toJSON();
}

I think a --strictLibs style option would be a good compromise. Is there currently a list of non-spec-compliant types that are not being corrected due to compatibility claims?

There’s no list, but the biggest category is that every any in a covariant position would become unknown. I think the barrier to doing it now is really just a matter of figuring out how to automate everything so we aren’t actually maintaining 2x the number of lib files. In particular, the DOM libs are generated from IDL files, so inserting a bunch of manual editing into that process is kind of a non-starter. And changing unknown to any or vice versa is easy, but this string | null return type is a good example of something we’d have to think about. Not every nullable type should be made non-null in the non-strict libs. There are a lot of open questions about how the whole process should work.

Would be really nice to have correct return types in toJSON() and JSON.strigify(). Today we run into a really bad bug on production because of this. I would expect that TS warned us. Sadly that was not the case. 😢


Improvements on Thanos Compactor

Description of the Problem
Thanos Compactor has a large scope of improvement, particularly scaling out and running Compactor concurrently, so this umbrella issue tracks all the expected improvements, proof of concepts and design proposals for improving the Compactor.
Describe the solution you'd like
There are a few important issues/discussions that are relevant to this issue -
Design Docs

Scalable Compactor Doc - https://docs.google.com/document/d/1xi0V8DB0hE54XgkogJRnNL6yH7C5JThJywlLFoC6dCQ/edit
LFX Mentoring Proposal - Vertical Block Sharding - https://github.com/thanos-io/thanos/pull/4191
Vertical Block Sharding - https://github.com/thanos-io/thanos/pull/3390

Pull Requests/Issues

Compaction Grouper Concurrency - https://github.com/thanos-io/thanos/pull/3807
Streamed Block Reading - https://github.com/thanos-io/thanos/issues/3416
Delete old data - https://github.com/thanos-io/thanos/issues/4046
Progress Calculation for Compactor - https://github.com/thanos-io/thanos/issues/3985
Upper Bound - https://github.com/thanos-io/thanos/issues/2340
Penalty based deduplication (offline deduplication) - https://github.com/thanos-io/thanos/pull/4239

P.S. Feel free to add your Proof of Concepts/Suggestions/Discussions
cc @thanos-io/thanos-maintainers

Not stale.


Systest through gateway

Wondering if i should still include workflow client to do those three tests i removed (admin stats, elasticsearch test and version test)

I added a sleep timer after we delete 2 events and before we query for other events. Also added a sleep timer after deleting triggers but before deleting the eventType.  This seems to have fixed the problem (working for me) but maybe we should use the refresh=true option on our last delete call instead of each sleep timer (if the go elastic library supports it)

Rather than just wait a random amount of time, we should poll to see when the database is consistent (like we do elsewhere).


feat(bank_redirects): modify api contract for sofort

Type of Change

[ ] Bugfix
[ ] New feature
[x] Enhancement
[ ] Refactoring
[ ] Dependency updates

Description

Adding required billing details field for Sofort bank transfer
Changes in the api contract for Sofort

Sofort

{
    "payment_method": "bank_redirect",
    "payment_method_type": "sofort",
    "payment_method_data": {
        "bank_redirect": {
            "sofort": {
                "billing details": {
                    "billing_name": "Example"
                },
                "country": "AT",
                "preferred_language": "en"
            }
        }
    }
}

Additional Changes

[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

Billing details in Sofort bank transfer are needed for other connectors like PayPal.
Also discussed this with @bernard-eugine and made the change.
How did you test it?

Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed submitted code
[ ] I added unit tests for my changes where possible
[ ] I added a CHANGELOG entry if applicable

@bernard-eugine Should we send these details to other connectors if available?


(node:9941) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 exit listeners added to [Bus]. Use emitter.setMaxListeners() to increase limit

1. Versions

What is your wechaty version?
Answer: 0.40.8

Which puppet are you using for wechaty? (padchat/puppeteer/padpro/...)
Answer: padplus

What is your wechaty-puppet-XXX（padchat/puppeteer/） version?
Answer: 0.7.18

What is your node version? (run node --version)
Answer: 13.5.0

What os are you using
Answer: macOS 10.14.6 (18G5033)

2. problem
Got this sometimes after node is restarted. Still trying to figure out what causes it and gain more info about it.
(node:9941) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 exit listeners added to [Bus]. Use emitter.setMaxListeners() to increase limit

discussion on SO: https://stackoverflow.com/questions/9768444/possible-eventemitter-memory-leak-detected
maybe have to do with using .on instead of .once over new WeChaty for onMessage callbacks, etc.

Maybe related to #2019
BTW: This warning is just telling you that the Wechaty have more than 11 listeners for one event.
Sometimes we do need to have lots of listeners, especially when we are working with the RxJS & Redux, like wechaty-redux subsystem.

I see! I will .setMaxListeners to a higher number for now! wechaty-redux looks interesting :)


add textbox to highlight a continued slide

Writing text 'continued (x)' to a given slide's title placeholder fails when title placeholder doesn't exist. Would be a common fail since users are allowed to use their own templates. Text 'continued (x)' now added to the question label shape, aka header shape, which is created and added from scratch. This approach removes the need to search for certain shapes.

Resolved by #400.


consoleme-celery yarn error during docker-compose installation

Describe the bug
When installing consoleme locally with docker and running the following command:
docker-compose -f docker-compose.yaml -f docker-compose-dependencies.yaml up -d

I receive the following error:
Step 20/22 : RUN yarn --cwd ui build:prod
 ---> Running in 049ad792edf1
yarn run v1.22.10
$ yarn run clean:prod && yarn build && yarn run copy:prod
$ rm -rf build && rm -rf ../consoleme/templates
$ react-app-rewired build
Creating an optimized production build...
events.js:292
      throw er; // Unhandled 'error' event
      ^

Error: write EPIPE
    at ChildProcess.target._send (internal/child_process.js:832:20)
    at ChildProcess.target.send (internal/child_process.js:703:19)
    at ChildProcessWorker.send (/apps/consoleme/ui/node_modules/jest-worker/build/workers/ChildProcessWorker.js:299:17)
    at WorkerPool.send (/apps/consoleme/ui/node_modules/jest-worker/build/WorkerPool.js:32:34)
    at Farm._process (/apps/consoleme/ui/node_modules/jest-worker/build/Farm.js:147:10)
    at Farm._enqueue (/apps/consoleme/ui/node_modules/jest-worker/build/Farm.js:176:10)
    at Farm._push (/apps/consoleme/ui/node_modules/jest-worker/build/Farm.js:183:12)
    at /apps/consoleme/ui/node_modules/jest-worker/build/Farm.js:106:14
    at new Promise (<anonymous>)
    at Farm.doWork (/apps/consoleme/ui/node_modules/jest-worker/build/Farm.js:69:21)
    at JestWorker._callFunctionWithArgs (/apps/consoleme/ui/node_modules/jest-worker/build/index.js:182:23)
    at /apps/consoleme/ui/node_modules/terser-webpack-plugin/dist/index.js:304:45
Emitted 'error' event on ChildProcess instance at:
    at internal/child_process.js:836:39
    at processTicksAndRejections (internal/process/task_queues.js:75:11) {
  errno: -32,
  code: 'EPIPE',
  syscall: 'write'
}
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
ERROR: Service 'consoleme-celery' failed to build : The command '/bin/sh -c yarn --cwd ui build:prod' returned a non-zero code: 1

To Reproduce
Steps to reproduce the behavior:

Clone the repo
Cd into the project
Run the following command: docker-compose -f docker-compose.yaml -f docker-compose-dependencies.yaml up -d
See error

Expected behavior
Consoleme should be properly installed.
Screenshots
N/A
Desktop (please complete the following information):

OS: MacOS Catalina 10.15.7
Docker Desktop version 3.0.4

Smartphone (please complete the following information):
N/A
Additional context
None.

I re-ran the docker-compose command and received a new error:
Creating an optimized production build...
The build failed because the process exited too early. This probably means the system ran out of memory or someone called `kill -9` on the process.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
ERROR: Service 'consoleme' failed to build : The command '/bin/sh -c yarn --cwd ui build:prod' returned a non-zero code: 1

Although my computer is a bit old (Mac Mini 2012), I have 16GB of ram so I'm not sure how I ran out of memory.

Could you try bumping up your Docker memory and swap? I was able to build (albeit on a newer Mac) with these settings:

That did it @castrapel! Thanks for the help!


WIP: Upgrade to latest nightly

So I know this library is supposed to be pegged to the version of Rust that Servo uses, but I thought maybe it would be worth having a branch that is updated separately?
Changes are broken up into a few logical commits. The last two are just trivial fixups, but the first two are important.
The first adds PhantomData to bind the variance of the otherwise unused lifetimes. I don't believe master is actually valid without this change (s/PhantomData/ContravariantLifetime should be the equivalent servo-version-friendly change), as the lifetimes are otherwise bivariant, as I understand it.
This actually revealed a lifetime bug in the code that the second patch attempts to resolve. I say attempts because I'm not 100% sure what semantics you were going for. I went for the one that seemed to match up with usage.
TBD: updating the example code. This requires updating the sdl2 dependencies, which are currently hardcoded to some commit. Need to look into what the appropriate version to update to is. Any insight on this matter would be appreciated.

Ah, sdl2 master has undergone a major API refactor. Gonna take me a bit to learn the new API and migrate things (also found another lifetime issue in libavcodec).

Alright everything works on the nightly now :)

I'll be happy to merge this into a new branch if you squash everything into one commit. Thanks!

squashed

This is now in the nightly branch.


I changed the word salientes with enviados

is a better expression for spanish speakers to say 'enviados' meaning 'sent/going out' rather than saying 'salientes'.  Users who read the documentation will have a better understanding now of the maning.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for your contribution.
After reviewing your pull request we have decided not to merge it
On this article, the term saliente matches the proper translation for outgoing. I think saliente is a well known when talking about email, specifically when talking about set up (like in this article).
We would welcome future pull requests from you.
Thank you and happy coding!


Add Additional Image MIME Types, Dynamic Per OS

Issue Link :link:
#428 #47
Goals :soccer:
This PR updates our default MIME types to be properly dynamic by OS version. It also adds support for JPEG2000 on all OSes, HEIC and HEIF on 2017 OSes, and WebP on macOS 11 and iOS 14 (doesn't seem to work on tvOS 14).
Implementation Details :construction:
Simply constructs the default accept types dynamically.
Testing Details :mag:
Tests updated for additional MIME types and image type support.

@maciej-letgo Yes, it just needs test updates. I just need to be less busy to finish it up.


[Bug]: Alignment of buttons in Hotels and Resorts Section

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Hello PA,
Buttons in Hotels and Resorts section are not aligned properly and there is a huge gap between cards and buttons.
Expected behavior
Buttons are aligned properly with not much gap between cards and buttons.
Add ScreenShots
Before:

On which device are you experiencing this bug?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[ ] I'm a VSOC'24 contributor
[X] I have starred the repository

@apu52 @Ghanshyam89 @KrishnenduDG @itsekta
Kindly assign this to me, also label the issue.
Will work on this.
Thank you.


Update Get-CsTeamsCallHoldPolicy.md

Formatting fixes per build report

Docs Build status updates of commit 714003f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Get-CsTeamsCallHoldPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Npm publish of nightly release is failing since 2022-05-21

There're no nightly npm packages newer than 2022-05-18 on NPM, and it seems that the nightly release action is failing to publish.
https://github.com/ruby/ruby.wasm/runs/6527649310?check_suite_focus=true#step:6:31
Probably similar to this https://github.com/semantic-release/npm/issues/209 issue, there're something wrong with the NODE_AUTH_TOKEN. Would you check it?

I haven't realized that situation. Thank you for letting me know!
I've updated the token and I hope it makes the automation back. Will close after today's nightly release.

Thank you!

OK, it works now again 👍


Fix absolute file path error in DiskCache - Swift 2.3

Added the absolute file path as a separate variable in the let
expression so its value can be safely unwrapped and used within
NSFileManager.fileExistsAtPath(...) (in order for Mattress to compile
properly in Swift 2.3).

Aside from maybe spelling out "absolute" (nitpick) LGTM.


fix: enforce defaults in OCP form view

Description of the change
OCP Console is acting strangely when we want to default to [], it assumes it's null and pulls data from the schema. This PR should ensure the schema is always consistent with default values in the chart, including values for subcharts.
Existing or Associated Issue(s)
Fixes: #151
Additional Information
https://github.com/janus-idp/helm-backstage/assets/7453394/e2709f02-0665-4954-ac15-2beb327e37ab
Checklist

[x] Chart version bumped in Chart.yaml according to semver.
[x] Variables are documented in the values.yaml and added to the README.md. The pre-commit utility can be used to generate the necessary content. Use pre-commit run -a to apply changes.
[x] JSON Schema template updated and re-generated the raw schema via pre-commit hook.
[x] List tests pass for Chart using the Chart Testing tool and the ct lint command.

cc @kadel

CI failing due to https://github.com/janus-idp/backstage-showcase/issues/813 please retrigger once new build for next finishes.


Docker image not working with Yarn PnP - cannot find module

What is your Scenario?
I would like to run my repo set up with Yarn PnP through the testcafe docker image
What is the Current behavior?
When I attempt to run docker run --rm -v D:\path\to\testcafe-docker-yarn-pnp:/app -w /app -it testcafe/testcafe:2.1.0 chromium build/tests --config-file=.testcaferc.json I get the following error:
ERROR Cannot prepare tests due to the following error:

    Error: Cannot find module 'node-fetch'
    Require stack:
    - /app/build/tests/lifecycle.test.js
    - /usr/lib/node_modules/testcafe/lib/compiler/test-file/formats/es-next/compiler.js
    - /usr/lib/node_modules/testcafe/lib/compiler/compilers.js
    - /usr/lib/node_modules/testcafe/lib/compiler/index.js
    - /usr/lib/node_modules/testcafe/lib/runner/bootstrapper.js
    - /usr/lib/node_modules/testcafe/lib/runner/index.js
    - /usr/lib/node_modules/testcafe/lib/testcafe.js
    - /usr/lib/node_modules/testcafe/lib/index.js
    - /usr/lib/node_modules/testcafe/lib/cli/cli.js
    - /usr/lib/node_modules/testcafe/lib/cli/index.js
        at Object.<anonymous> (/app/tests/lifecycle.test.ts:2:1)

    Type "testcafe -h" for help.

What is the Expected behavior?
The tests should run without any errors around dependencies
What is your public website URL? (or attach your complete example)
https://github.com/aboe026/testcafe-docker-yarn-pnp
What is your TestCafe test code?
import { Selector } from 'testcafe'
import fetch from 'node-fetch'

fixture('Lifecycle').page('https://google.com')

test('Search bar exists', async (t) => {
  const result = await fetch('https://swapi.dev/api/people/1')
  const text = await result.text()
  let json
  try {
    json = JSON.parse(text)
  } catch (err) {
    throw Error(`Invalid JSON response '${text}': '${err}'`)
  }
  await t.expect(json.name).eql('Luke Skywalker')
  await t.expect(Selector('input[title="Search"]').exists).ok()
})

Your complete configuration file
{
  "browsers": ["edge"],
  "screenshots": {
    "path": "screenshots",
    "takeOnFails": true
  },
  "src": "build/tests"
}

Your complete test report
No response
Screenshots
No response
Steps to Reproduce
Please see https://github.com/aboe026/testcafe-docker-yarn-pnp#steps-to-reproduce
TestCafe version
2.1.0
Node.js version
v16.13.2
Command-line arguments
docker run --rm -v D:\path\to\testcafe-docker-yarn-pnp:/app -w /app -it testcafe/testcafe:2.1.0 chromium build/tests --config-file=.testcaferc.json
Browser name(s) and version(s)
chromium
Platform(s) and version(s)
Windows 11
Other
No response

I think I found a workaround: https://github.com/aboe026/testcafe-docker-yarn-pnp#workaround
Needed to alter the docker image like so:
FROM testcafe/testcafe

USER root

RUN npm install -g yarn && \
    sed --in-place --regexp-extended "s/^node /yarn node /" /opt/testcafe/docker/testcafe-docker.sh

USER user

Hi @aboe026

I think I found a workaround: https://github.com/aboe026/testcafe-docker-yarn-pnp#workaround

I am happy to hear that.
We will consider supporting Yarn PnP since it has its own module resolution mechanism that is not fully compatible with the standard way used in npm.

Duplicate of https://github.com/DevExpress/testcafe/issues/7710


Closing HTML tag not highlighting in PHP files.

VSCode Version: 1.18.0
OS Version: Mac OS

Steps to Reproduce:

Open PHP file  with HTML
Put cursor on tag to see closing tag

All tags will be highlighted. Expected result: only closing tag should be highlighted.

Reproduces without extensions: Yes

#2915 ?

duplicate of #2915


Unable to infer type of union of sets

Environment data

Language Server version: 2021.6.0
OS and version: macOS 11.4
Python version (& distribution if applicable, e.g. Anaconda): 3.9.5

Expected behaviour
Infer the type of the union of a list of sets of the same type
Actual behaviour
Type information lost after union
Logs
Type of "union" is partially unknown
  Type of "union" is "(self: set[Unknown], *s: Iterable[Unknown]) -> Set[Unknown]"

Code Snippet / Additional information
bar: list[set[int]] = [{1, 2}, {3, 4}]
foo = set.union(*bar)

foo should have type set[int].

Pyright (the type checker that underlies pylance) does not currently support the unpacked arguments that map to multiple parameters in the target function. In this case, you're doing something very unusual. You're calling set.union as though it were a static method but then using the unpack operator to provide one instance of set to the self parameter and expecting the remainder to map to the s parameter.
Adding support for this very unusual case would add significant complexity to an already very complex and performance-critical part of the type checker. Unless we see evidence that this is a common pattern, I think adding support for this case has more downsides than upsides.
It should be straightforward to work around this limitation.

I see. Thanks for your answer! The main problem is that set.union, which is not a static method, is the only function that takes an iterable of sets as input and output their unions. Totally agree that support for this case should not be added.
Personally, I use the following workaround which is more symmetric with respect to the inputs:
foo: set[int] = set()
foo.union(*bar)

Thank you again!

I like your solution better than mine. Much easier to read. :)


Map transducing

As a PoC, I've implemented transducing on map!
In this PR:

Chained map operators will collapse on themselves. This results in about a 20% performance boost when only two map operations are chained. That should grow with the number chained.
map().mergeAll() will flatten to .mergeMap() via magic. This results in about a 20% perf improvement over using .map().mergeAll() prior to this commit, and is ~15x faster than .map().mergeAll() in Rx4.

While I think we can land this here in master, this is a PoC to see what needs to be done.
One thing I learned while working on this is the thisArg makes creating this a little uncomfortable. As if I needed another reason to hate the thisArgs.

attn @trxcllnt (for sanity).
attn @benjamingr, @robwormald and @jeffbcross  as this relates to discussion in https://github.com/angular/http/issues/56.

Awesome! Thanks so much for exploring this, %20 performance boost for map operators collapsing sounds like a useful optimization. What about filter and its friends?
Optimally this optimization for single subscriber is something that should be done more generally.

@benjamingr if this PR passes the smell test then scan, reduce, filter can get the same treatment and:  map().concat() can be concatMap, map().switch() can be switchMap(), etc scan().mergeAll() can be mergeScan().
Another possibility is collapsing resultSelectors from operators like zip or combineLatest with map... (e.g.  .zip(a, b, (a, b) => a + b).map(x => x + '!') is really just .zip(a, b, (a, b) => (x => x + '!')(a + b)))

In the case of zip above, it will take a different type of approach, because we can't skip to the source operator, rather we're augmenting output.

@blesh Did you see if there is any significant change in performance when map is not chained (e.g. only a single map)?

@luisgabriel I did not... but the map perf test was already chaining them.  Given that it's really just a quick assertion at the moment map() is called, I doubt we'd see any noticeable difference in any perf test. Especially, since we're generally not calling map in the perf tests, rather we're testing the usage of the Observable result of the map call.

@luisgabriel looking at the source code change - when no chain collapsing is performed the same path is hit - it should perform exactly the same. I'm not sure if these two extra variables are needed but realistically the runtime can take care of that for us.

@blesh is there a method we can add to the Operator interface to support this more generically (and move the conditional transducing logic into the Operator implementation instead of the public method)?

@trxcllnt definitely. Perhaps call could just return a different Subscriber?

@blesh It could, but don't the operators need to know about each other? I'm thinking something like:
export function map(project) {
  return this.lift(this.operator.transduce(new MapOperator(project)));
}

And the Operator base class could have a no-op transduce implementation:
class Operator {
  call(subscriber) { return subscriber; }
  transduce(operator) {
    return operator; // return the operator we were given if we can't collapse it with ourself.
  }
}

@blesh though I'm not sure about how much I like the word transduce for the public API... I know it's well understood in certain circles, but another term might make more sense for newcomers trying to implement their own operators... collapse perhaps?

+1 for collapse but for the opposite reason, transduce is well understood enough to trigger bikeshed later on.

@blesh @benjamingr Hmm. Makes sense.

Looking at this design, I think it falls apart a bit with operators like filter and scan. I'll leave this open, but I don't think it'll make it into this release.

Ping @blesh status?

@benjamingr I've been working on other things, but there's a new design in the works for this that (hopefully) will be able to collapse almost all sync map-type operations in a chain.

@blesh Is there an issue where you're discussing/tracking this?