rtools43 now has grpc

seems like rtools42 not supported?

No, they won't update rtool42. So if we want to support that, which would be good, then we still need to download grpc for that. But I think that's ok. Or we can require R 4.3. on Windows.

No, they won't update rtool42. So if we want to support that, which would be good, then we still need to download grpc for that. But I think that's ok. Or we can require R 4.3. on Windows.

I wonder if the submission will be declined if it includes code to download lib from the internet like last time.

Yeah, probably better not to download this time.


License information

Hi, just found this Rust application, which I would like to add to https://github.com/pka/awesome-georust
Would you mind clarifying the license?

I had forgotten, it's not a super well-organised package (yet).
It's MIT: 85e6f0bc4faf1dd0ee5f174de521cfc5305beb0b
Thanks for adding it to this page!

Added, thanks a lot!


Conditional Fetch requests support

Overview
Introduces support for conditional request parameters when fetching a value from Riak bucket.
Riak Fetch Object (HTTP API) parameters are described in here:
http://docs.basho.com/riak/kv/2.1.4/developing/api/http/fetch-object/#request
Supported conditions:

IfMatch - fetch a value if it has the same ETag.
IfNotMatch - fetch a value if it has a different ETag.
IfModifiedSince - fetch a value if it has been modified (Last-Modified attribute) since a specific time.
IfUnmodifiedSince - fetch a value if it has been modified (Last-Modified attribute) since a specific time.

Conditions can be combined together.
Compatibility
This change is backwards compatible with existing code that uses the library.
Tests
Tested with integration specs (RiakBucketSpec.scala) against Riak 1.4 (1.4.10) and Riak 2 (2.1.1).
Future improvements
Conditional parameters support can be expanded also to Store Object API:
http://docs.basho.com/riak/kv/2.1.4/developing/api/http/store-object/#request
It works the same way in Riak.
Support of other API backends
Note that the conditional requests support in Protobuf API is a bit different:
http://docs.basho.com/riak/kv/2.1.4/developing/api/protocol-buffers/fetch-object/#request

No support of conditions based on Last-Modified attribute.
If(Not)Match conditions are based on VClocks, not ETags.

So HTTP API and Protobuf API are not on par in that sense.

Coverage increased (+0.5%) to 81.308% when pulling 11ac7d02f3519871b623c4fcb5b6a708eec37281 on ajantis:conditional-fetch-support into 54f742a19ab6a1bf5d40f7b4a5ef788e8d2e9138 on agemooij:master.

Looks good!
I made some minor comments but nothing really important. Feel free to merge at any time.

Hey Age,
Thanks for your swift review! ;)
I am applying the comments.
Didier's remark makes sense, too: I also was thinking that it might be needed for a client to be able to distinguish 'Not Found' and 'Found but Precondition Failed' cases.
BR,
Dmitry

I agree with @didierliauw too.
I recommend using Spray's StatusCodes (or akka-http's StatusCodes) as inspiration since Http doesn't have 404 and 304 for nothing 😄

I put this change on hold for now, as there is one substantial piece missing. In case of siblings, all ETag-based conditions don't work with resolved RiakValue's ETag, only with the parent ETag.
Currently, library doesn't expose those parent ETags anyhow, so this needs to be fixed as part of this PR, I guess.


Updated travis to use docker CouchDB

Thanks for your hard work, please ensure all items are complete before opening.

[x] Tick to sign-off your agreement to the Developer Certificate of Origin (DCO) 1.1
[x] You have added tests for any code changes - test only changes
[x] You have updated the CHANGES.md - N/A test only changes
[x] You have completed the PR template below:

What
Changed .travis.yml to use local docker apache/couchdb
Testing
CI passing.
Issues
N/A

I could update it to couchdb:2 which I think gets the latest 2.x. I didn't see an equivalent tag documented for apache/couchdb.
I'm not sure on the status between couchdb (apparently official) and apache/couchdb (apparently unofficial). I thought all the Apache projects were putting official images in apache/ now.

@ricellis Maybe it's better to use library/couchdb which I believe is official.

The CouchDB docs say use apache/couchdb though. I've asked for clarification from the community.

Merging for now, we can update the docker image to couchdb:2 later if I get any clarification on it.


NavLink and RouterLink

Codecov Report

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

@@           Coverage Diff           @@
##           master      #42   +/-   ##
=======================================
  Coverage   31.48%   31.48%
=======================================
  Files          29       29
  Lines         378      378
  Branches       43       43
=======================================
  Hits          119      119
  Misses        237      237
  Partials       22       22

Impacted Files
Coverage Δ

src/components/Header.js
95.23% <ø> (ø)

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 bca4ed8...23a451c. Read the comment docs.


"If you want the called method to change the value of the parameter, you must pass it by reference"

The original sentence:

If you want the called method to change the value of the parameter, you must pass it by reference, using the ref or out keyword.

I'm not sure but I think it should be "the argument" instead of "the parameter".

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

ID: c4dcb4da-bdb8-853d-678a-9c5130921ecb
Version Independent ID: 1dbe0544-77c2-c70f-8406-2bca85e914e8
Content: Passing Value-Type Parameters - C# Programming Guide
Content Source: docs/csharp/programming-guide/classes-and-structs/passing-value-type-parameters.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

It means the same thing, but parameter is actually the correct term. Since in the implementation of the method, the argument may undergo change. The argument at the end of the method may not be the same as it was at the input. Parameter is more appropriate since it may be dynamically changing.


Will on-prem traffic to StorageAccount, Azure SQL route trough ExpressRoute by default?

We have ExpressRoute created between on-prem and Azure, question is if we try to access Azure PaaS services   which are created with public endpoints from on-prem, such as StorageAccount, Azure SQL, will the traffic route through ExpressRoute by default or it will still goes through internet untill MS peering is in space

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

ID: 3cdb32c0-628d-0be2-f514-477d46dfa1b8
Version Independent ID: fa096059-2d16-30a2-1e8b-8b9671afcaa4
Content: FAQ - Azure ExpressRoute
Content Source: articles/expressroute/expressroute-faqs.md
Service: expressroute
GitHub Login: @jaredr80
Microsoft Alias: jaredro

@CharmanderJieniJieni This depends upon what kind of Peering you have associated with the circuit. With no peering and connecting to the Public endpoints, traffic will take the public internet.
If you have Microsoft Peering enabled for the circuit, traffic will travel across the ExpressRoute.
It is also possible to set up an Azure Private Link in your Virtual Network, which allows VNET level connectivity to Azure Storage or Azure SQL that does not go across the public internet.

Thanks for the quick response. From what I understand both Service endpoint and Private Link enables private connection between traffic initialized from the VNet and the destination Azure PaaS service. Is this correct?
If so the traffic from on-prem does not fall into the scope because on-prem network does not have a Azure VNet. Would you please kindly clarify that?
Saying that, what if our on-prem advising traffic to an Azure Firewall which is hosted in an Azure VirtualNetwork through Express Route and then we enable Service Endpoint/Private Link between the Firewall VNet and Azure PaaS service, will this ensure all traffic between on-prem and Azure PaaS service routing through private networks?

@CharmanderJieniJieni Service endpoint traffic must originate from the Virtual Network, but Private Link can come over ExpressRoute or VPN to the Virtual Network.
Using an Azure Firewall for a NAT as you described above will work for Service Endpoints, but it sounds like a Private Link is the best fit for your scenario.

@CharmanderJieniJieni We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Mbasiuk/couchbase tests

What is the current behavior? (You can also link to an open issue here)
Small test coverage.
What is the new behavior?
Test coverage 75+%
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[x] The code coverage on new code >75%
[x] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: failure :x:

Pull request test result: failure :x:


Auto-fix when in the editor

How could I use the automated fix after logging in to Grammarly?
I got the information shown and I could only ignore the issue. Thank you for developing the extension.

Also having the same issue.


main skeleton

This adds a simple http handler in main.go as a starting point.

@eduncan911 @alaska @preillyme @erikstmartin reviews please?

@peterbourgon if you have an opportunity, I'd appreciate your guidance here too.

Are we going to have a single binary running both the API and public frontend?

@eduncan911 It's a possibility, but undecided at this point.


This version of Microsoft Edge WebDriver only supports Microsoft Edge version 122

Hello!
I am trying to run my usual test cases in wedriverIO using Microsoft Edge browser.
Since the latest Microsoft Edge update, I keep getting this error:
'2024-05-08T07:56:37.941Z ERROR @wdio/runner: Error: Failed to create session.
[0-0] session not created: This version of Microsoft Edge WebDriver only supports Microsoft Edge version 122
[0-0] Current browser version is 124.0.2478.80 with binary path C:\Program Files (x86)\Microsoft\Edge\Application\msedge.exe'
I have tried all sort of solutions found online but I am unable to make this work.
Any idea what might cause this or how to deal with it ?
If you need more details, please let me know.
Thanks in advance.

For our use cases, when we are two versions behind we get the same error.
Update your Edge driver to be the same as 124 or only 1 version behind.
(Same issue will happen for chrome)
The error says your browser is 124, yet your webdriver is 122.
Update to 124 and you should be good. (or update to whatever version your current edge version is)

@Stefanczc Are you using Selenium, by any chance? They recently released v4.21.0. I updated its Maven dependency in my Java project to reflect this and I do not encounter the error shown in your post.

The Edge browser is getting an update like every week, the driver should be more tolerant and accept all version until a breaking change happened - or add a "ignoreVersion" option to the driver.

I got it running without version check exception like this:
`var service = EdgeDriverService.CreateDefaultService(localPath);
service.DisableBuildCheck = true;
var options = new EdgeOptions();
options.AddArgument("--allow-running-insecure-content");
options.AddArgument("--ignore-certificate-errors");
var driver = new EdgeDriver(service, options);`
I tried it before with option "--disable-build-check" but that didn't work.


Make pylint parallel

On my local machine this took pylint from 3 minutes to less than one.

Looks to shave about 2+ minutes off the CI runs.


Hard to differentiate bold text

When making the text bold in the editor, it's hard to tell that any text has been boldened, both in the editor and on the site. My browser is google chrome, Version 65.0.3325.181.
The first screenshot is from the editor.

The second screenshot is from the website.

The difference is very apparent now in all browsers and in the editor, perhaps because of the font-smoothing or changes applied to address #93 .


PyMJCF Attachment frame extra body breaks compilation of freejoint object attachments

Description:
Attaching one PyMJCF model (the "source") to another (the "destination") adds the content of the source's worldbody into a newly generated body element in the destination worldbody. So if we want to add a robot whose model name is "robot1" to some scene, then all the contents of its worldbody will be inside a body named  "robot1/" and all its children's names will be "robot1/{original_name}". The issue is that if "robot1" has a freejoint in its root body (the one direct child of worldbody), the resulting merged model cannot be loaded with MuJoCo because the freejoint element is now inside the base element which is inside the newly generated body "robot1/".
Error Message:
Error: free joint can only be used on top level
Reproduce:
A simple example to recreate the error message using the MuJoCo menagerie. This works with any free-moving robot (has a freejoint in the base). In the "scene.xml" file, we comment the <include/> line to avoid loading the robot this way, and only attach it with MJCF
from dm_control import mjcf

scene_model = mjcf.from_path('mujoco_menagerie/robotis_op3/scene.xml')  # <include/> tag is commented
robot_model = mjcf.from_path('mujoco_menagerie/robotis_op3/op3.xml')

scene_model.attach(robot_model)

physics = mjcf.Physics.from_mjcf_model(scene_model)

Suggested Solution:
My preferred solution would be to add an argument in the attach method that will exclude the outer "wrapper" body. We can still avoid name ambiguity by renaming the source model's components as before ("robot1/base", "robot1/wrist", etc).

Thanks for the quick response. Is your long-term solution development watchable on GitHub?

No, unfortunately.

I've been trying to implement this. My current solution is to edit the _AttachableElement  class's attach method such that it extends the frame's siblings list with all the attachment children instead of inserting the frame itself. I want to be able to preserve relative pose according to the attachment site. That is, if i attach an object whose root body position ("pos" attribute in XML) is [1, 0, 0] and the site base position is [0, 1, 0], then final position relative to the attaching frame would be [1, 1, 0]. This can be done manually by editing the properties directly on the root body of the attachment, and is easy for position (simple addition). Are there any helper functions in MuJoCo that can help me do this for rotation?
So far I found some conversion functions such as mju_axisAngle2Quat (in mujoco) and quat2euler (in dm_control). Are there more conversion functions that will help me convert between all 5 types of rotations? Is there an easier way that does not require conversion?

Update
I've implemented the required changes. Please see the pull request here. I ended up implementing some required rotation functions as well.


add download speed to progreesbar

Description of the new feature/enhancement
i would like to see download speed,time remaining while downloading a package

Proposed technical implementation details (optional)

H


Enable Selection for disabled Textfield

Hi,
I am using the CodeField mainly for displaying code, rather then writing it. So mainly I have it "enabled = false". Nonetheless I'd like to select some code snippets here and there, but as I see, the selection is only possible on enabled/writeable Textfields.
Is this possible?
Thanks in advance,
Mark

@S-Man42 readOnly is the property you're looking for.
Mark enabled:true and readOnly: true and you should be good to go!

Hi,
When using the last stable version 1.0.2, there is no readOnly...
So, I need to use the Git repo directly? Are there any plans for a stable release? Can it be used within a productive project?
Next question:
I need to modify the selection controls. Normal TextFormField and SelectableText have a hook selectionControls for it. Is there anything similar in CodeField (the idea is, that I need to do something with a copied text directly)

Hi,
When using the last stable version 1.0.2, there is no readOnly... So, I need to use the Git repo directly? Are there any plans for a stable release? Can it be used within a productive project?
Next question: I need to modify the selection controls. Normal TextFormField and SelectableText have a hook selectionControls for it. Is there anything similar in CodeField (the idea is, that I need to do something with a copied text directly)

Hey @S-Man42 , reference the package on pubspec the package like this while @BertrandBev  don't publish the package.

Yes, I did this. However it feels not good to do so in a ready for launch project, that's why I asked for stability. Which commit it preferred and can be used for it? The one you mentioned or simething later?

Yes, I did this. However it feels not good to do so in a ready for launch project, that's why I asked for stability. Which commit it preferred and can be used for it? The one you mentioned or simething later?

Yes I agree it's better a pub.dev version. I used the lastest commit from this repo.

Next question: I need to modify the selection controls. Normal TextFormField and SelectableText have a hook selectionControls for it. Is there anything similar in CodeField (the idea is, that I need to do something with a copied text directly)

Can you say something to this one?

Next question: I need to modify the selection controls. Normal TextFormField and SelectableText have a hook selectionControls for it. Is there anything similar in CodeField (the idea is, that I need to do something with a copied text directly)

Can you say something to this one?

Created https://github.com/BertrandBev/code_field/pull/69


Adding CNCF logo to footer

Addressing logo requirements for CNCF onboarding: https://github.com/cncf/toc/issues/1299

 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.


TypeError: Cannot modify readonly property: constructor

Test case:
            "use strict";
            var e = Promise.resolve(1);
            e.constructor = {};
            console.log('done');

In real browser, the script runs fine with both strict and non-strict mode. In htmlunit, the script runs with non-strict mode but fails with strict mode.

Will this ever end? :-)
As always thanks for the report - will require some time to nail down the problem.

I'm sorry it won't end shortly. :)
But please always take your time and no hurry.

have added some tests for this
ScriptableObjectTest
    ctorChangeableHasNoEffectForSealed()
    ctorChangeableHasNoEffectForSealedStrict()
    ctorChangeableHasNoEffectForTypeOf()
    ctorChangeableHasNoEffectForTypeOfStrict()
    ctorNotChangeableForPrimitives()
    ctorNotChangeableForPrimitivesStrict()

Thanks for pointing at this again. The original case works already, another improvement will land soon.

Yes, I confirm this works now.

@RuralHunter i guess your special case works since the switch to the Rhino promises.

Reopened to track the general change of the upcoming property adjustment of constructors.
https://github.com/mozilla/rhino/pull/1285

hope this is fixed for other cases also


Typo with QuickfixJ Server Configuration Filename Corrected

Corrected from quickfix-server.cfg to quickfixj-server.cfg

Codecov Report

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

@@          Coverage Diff          @@
##           master    #19   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      2
  Lines           4      4
=====================================
  Hits            4      4

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 5aa0e54...559fba1. Read the comment docs.


Vk pipeline initialization

This PR is primarily focused on the creation of the Vulkan pipeline, but there is also some small additional add ons bundled with the PR for the future.

Putting this as blocking until restructure is finished.


AF 3.0 在iOS7 上调用 uploadTaskWithRequest 上传文件，总是会报错，有人遇到了吗

AF 3.0 在iOS7 上调用 uploadTaskWithRequest 上传文件，总是会报错，有人遇到了吗

顺便问一下，AFNetwork3.0如何实现队列请求？

Would you be able to translate your request into English?

can i  use "uploadTaskWithRequest" mothed in ios7 ? when i update AF 3.0, i could not upload imagedata with  uploadTaskWithRequest.

Please provide reproducible code example.
请给出可以重现问题的示例代码。

This is probably best suited for stackoverflow. We use Github to track bugs and feature requests.
:beers:

From our Contribution Guidelines
Asking Questions
We don’t use GitHub as a support forum. For any usage questions that are not specific to the project itself, please ask on Stack Overflow instead. By doing so, you’ll be more likely to quickly solve your problem, and you’ll allow anyone else with the same question to find the answer. This also allows maintainers to focus on improving the project for others.


#2330_fix

Description
An extra variable was added to include the author name.
Related issues and discussion
#2330
Screen-shots, if any
Checklist
Make sure you've done all the following (You can delete the checklist before submitting)

[ ] Apply the AndroidStyle.xml style template to your code in Android Studio.
[ ] Code is well documented
[ ] Include unit tests for new functionality
[ ] All user-visible strings are made translatable
[ ] Code passes Travis builds in your branch
[ ] If you have multiple commits please combine them into one commit by squashing them.
[ ] Read and understood the contribution guidelines .

Change these lines too accordingly.
https://github.com/openfoodfacts/openfoodfacts-androidapp/blob/master/app/src/main/java/openfoodfacts/github/scrachx/openfood/fragments/ContributorsFragment.java#L72
https://github.com/openfoodfacts/openfoodfacts-androidapp/blob/master/app/src/main/java/openfoodfacts/github/scrachx/openfood/fragments/ContributorsFragment.java#L62

Edited the fragments. Please check.

The changes were not done by me. It's not required, i think.
On Mon, 14 Oct, 2019, 11:12 AM Huzaifa Iftikhar<EMAIL_ADDRESS>wrote:

@huzaifaiftikhar commented on this pull request.
In .idea/codeStyles/Project.xml
https://github.com/openfoodfacts/openfoodfacts-androidapp/pull/2903#discussion_r334331154
:

@@ -12,6 +12,9 @@

Why is this added? Is this required?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openfoodfacts/openfoodfacts-androidapp/pull/2903?email_source=notifications&email_token=AHVHNWULKURJBD5WPKZXXJDQOQBETA5CNFSM4I7W43I2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCHZJHAI#pullrequestreview-301110145,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHVHNWSNURWCNLCSDZTK3ODQOQBETANCNFSM4I7W43IQ
.

The changes were not done by me. It's not required, i think.
…
On Mon, 14 Oct, 2019, 11:12 AM Huzaifa Iftikhar, @.> wrote: @.* commented on this pull request. ------------------------------ In .idea/codeStyles/Project.xml <#2903 (comment)> : > @@ -12,6 +12,9 @@    +  Why is this added? Is this required? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#2903?email_source=notifications&email_token=AHVHNWULKURJBD5WPKZXXJDQOQBETA5CNFSM4I7W43I2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCHZJHAI#pullrequestreview-301110145>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHVHNWSNURWCNLCSDZTK3ODQOQBETANCNFSM4I7W43IQ .

@Abhisek2090 Can you remove the changes then? I guess it would be good to be merged then!

Merged, thanks @Abhisek2090!


flag: remove dependencies on shared variables

What type of PR is this?

bug fix

What this PR does / why we need it:
Remove the dependencies of shared variable from slice type flags parsing, and instead use a separatorSpec for parsing. This should not fail downstream dependencies that may be running multiple *App concurrently with race detector on.
Which issue(s) this PR fixes:
Fixes #1670
Special notes for your reviewer:
This commit creates a new type separatorSpec that will get passed into flag parser, instead of reading from shared variables. I'm uncertain that if I may have missed some parts of the code to be injected. Please let me know if I overlooked anything.
With HideVersion and HideHelp, the integration test mentioned in the issue is now passing with race detector on with this patch applied.
Testing
Should be passing with separatorSpec, assuming that the call site has the configured separatorSpec from caller.
Release Notes
flag: remove dependencies on shared variables when parsing slices

Minor comments. Thanks for the PR @zllovesuki

Any thoughts on failing tests?


cant stdin to clickhouse-client ( not | )

i use syslog-ng to start an external application to stdin query's Into clickhouse-client. this way i dont have to reopen the connection for every query.
but this wasnt working. i think i know why.
to test stdin to clickhouse-client i did echo "query\n" > /proc/pid/fd/0 the line appeared in the clickhouse-client as "query\n" so my guess is that clickhouse-client does not accept STDIN correctly.
can you fix this in clickhouse-client so i can use syslog-ng to STDIN data to clickhouse-client?
https://www.balabit.com/documents/syslog-ng-ose-latest-guides/en/syslog-ng-ose-guide-admin/html/configuring-destinations-program.html

Could you provide your queries?
The following examples work correctly for me:
vludv@vludv-ub14:~/distcc-ClickHouse$ echo "select 1" | clickhouse-client
1
vludv@vludv-ub14:~/distcc-ClickHouse$ echo "select 1, 2, 3" | clickhouse-client
1	2	3
vludv@vludv-ub14:~/distcc-ClickHouse$ echo -e "select 1, 2, 3;\n select 4, 5, 6;\n\n" | clickhouse-client -n
1	2	3
4	5	6

@ludv1x you are piping and not STDIN. piping works fine. only STDIN doesnt cause clickhouse-client does not understand \n

@yctn https://stackoverflow.com/questions/5374255/how-to-write-data-to-existing-processs-stdin-from-external-process


Keypresses are send to focussed client in sway

Version Information:
Running arch linux, swhkd version 1.1.0 and sway version 1.7
Describe the bug:
When doing setup keybinds, e.g. super + p, the 'p' still gets send to the focused client. I have not tested another compositor yet.
Expected behavior:
Keybinds are captured by swhkd and only send to client when swhkd has not 'consumed' them
Actual behavior:
See above
To Reproduce:
Set up keybinds in swhkd;
Run swhkd;
Run alacritty and set focus to it;
Press one of the keybinds set up in swhkd;

Hi I merged a patch for this an hour back, can you recompile from main and try again?

Just for future issues, swhkd is compositor independent. If something is broken in sway then it's broken everywhere, x11 too.

First off: thank you for the quick response! Second off it is indeed fixed now that I updated, thank you! And no I hadn't noticed, but I am loving this application! And I am actually thinking about using it in vikingwm :p

Glad that fixed it :) and thanks for using swhkd!!


iio: Node type code refactor for macros

Combine these similar macros
Signed-off-by: Yong Li<EMAIL_ADDRESS>

@edersondisouza As discussed in PR #2102, I changed these macros, please help to review

@edersondisouza I combined all these macros into one, please help to review

I like the way it is now and would merge it once minor comment is fixed. However, I think you could change ADD_CHANNEL to an inline function returning the newly added channel. Then, error checking would be left to the caller. Reasons: eases error handling and caller doesn't have to declare a sol_iio_channel_config that it's not seem to be used (it is, hidden inside the macro). What do you think?

@edersondisouza I agree with you that we can change the macro to an inline function. Actually I found we can refactor these create_channels functions too.  Is it okay to merge this PR first, then I create another PR for the new change?

Sure, it is =D

Integrated, tks!


Automate Env. setup for Azure DevOps Fundamentals training

┆Card is synchronized with this Github issue by Unito

Jan Krag commented in Trello:

@jankrag2 has the requirements. Monika might be interested in helping


⚠ Bofrost.fr has degraded performance

In 542b636, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1589 ms

Resolved: Bofrost.fr performance has improved in 9903fa9 after 6 minutes.


add a favicon

Use a square light themed logo from brand manual, convert it to an ico file and then add it as a favicon.

I would like to take this issue up.

Issue has been completed


sparse: check for not recursive matches when finding dirty files

Resolves #1604
The previous file path matching was only matching against recursive
sparse paths, and incorrectly flagging children of non-recursive folders
as dirty.  This matching has been updated to match both recursive and
non-recursive parents, and unit tests were added to validate the
matching behavior.

Next commit will address @kewillford's comment here:
https://github.com/microsoft/VFSForGit/pull/1607#discussion_r350968841


Centering N Badge

AB#16415391

Need to figure out how to put the badge in front of the edge component.

Pushing through


Missing space in data-versioning.md

It is the tiniest possible change, but it would make the project more professional.

thanks @JonasVerbickas !


Make updates to contribution doc

I tested out these instructions on a Windows 10 machine and noted a few spots that might need updating.

@beeankha, funny enough this is exactly what I recommended to a friend the other day that was having difficulty using Windows and git repositories together.
Might as well recommend GitHub Desktop too while we're at it.


DOC: don't mention elements that can't be included

Closes: https://github.com/ignitionrobotics/sdformat/issues/704
//model/include can not include a //actor or //light. This PR removes them being mentioned in the spec of this element.

This pull request has been mentioned on Gazebo Community. There might be relevant details there:
https://community.gazebosim.org/t/new-ignition-releases-2022-03-25-fortress-edifice-citadel/1343/1


xarray backend: tracking issue

https://github.com/pydata/xarray/issues/7446
https://github.com/conda-forge/staged-recipes/pull/21742
https://github.com/conda-forge/staged-recipes/pull/24105

Hidefix has now been added as a conda package!
I think after fixing #25, it's ready to add some docs to xarray.


repeating monitoring

How to judge whether you have listened to a certain time and avoid repeating monitoring

Can you show an example of what you are trying to do?


identical samples generated

In the following code, the samples generated x_train, are identical for each instance 0...N. The samples are identically distributed but usually different.
# required pacakges
import inferpy as inf
import tensorflow as tf

@inf.probmodel
def linear_reg(d):
    w0 = inf.Normal(0, 1, name="w0")
    w = inf.Normal(0, 1, batch_shape=[d], name="w")

    with inf.datamodel():
        x = inf.Normal(5, 2, batch_shape=d, name="x")
        y = inf.Normal(w0 + x @ tf.transpose(w), 1.0, name="y")

m = linear_reg(d=2)

N = 1000
data = m.sample(size=N, data={"w0": 0, "w": [2, 1]})
x_train = data["x"]
y_train = data["y"]

print(x_train)

output:
[[3.7095456 4.9971905]
 [3.7095456 4.9971905]
 [3.7095456 4.9971905]
 ...
 [3.7095456 4.9971905]
 [3.7095456 4.9971905]
 [3.7095456 4.9971905]]

Temporal fix: restored previous code with intercept. TODO: implement with contextmanager
    @util.tf_run_allowed
    def sample(self, size=1, data={}):
        """ Generates a sample for eache variable in the model """
        expanded_vars, expanded_params = self.expand_model(size)
        with ed.interception(util.interceptor.set_values(**data)):
            expanded_vars, expanded_params = self.expand_model(size)
        return {name: tf.convert_to_tensor(var) for name, var in expanded_vars.items()}

You are right. Because of sample size, it cannot be used observe to fix the values. It is better to use expand and intercept in this sense. I also do not see any trouble in using this version of sample. I propose to close this issue with this code.


ZBL short-range potential

Contributor (creator of pull-request) checklist

[x] Tests updated (for new features and bugfixes)?
[x] Documentation updated (for new features)?

📚 Documentation preview 📚: https://metatrain--335.org.readthedocs.build/en/335/

Looks good to me, I don't see any potential issues with PET at first glance
As the tests are fixed, we can merge this I think
But please wait until #344, it is once step from being merged

IMO, since multiple people need this, we should merge as is, without worrying about handling different units or neighbor lists. We can open issues however to keep track of those

Okay, so I tried to run PET + ZBL, and there are a few things to note.

ZBL only supports the cutoff which is 2x larger than the largest covalent radius in the dataset. In the case of MAD dataset, it results in the minimal cutoff of ~4.9 A, which is already large than our current 4.5 A cutoff. Maybe we should change the factor to at least 1.5? In this case the minimal cutoff will be 3.66, at it seems more reasonable.
This check for ZBL minimal cutoff happens only during the evaluations, i.e. after the model was trained. Perhaps it should happen in the beginning of the training.
There is a large block of converting the NL from half to full format in the ZBL code:

# convert to full NL
half_nl = system.get_neighbor_list(nl_option)
half_nl_samples = half_nl.samples.values
half_nl_values = half_nl.values
nl = TensorBlock(
    samples=Labels(
        names=half_nl.samples.names,
        values=torch.concatenate(
            [
                half_nl_samples,
                torch.concatenate(
                    [
                        half_nl_samples[:, 1].unsqueeze(-1),
                        half_nl_samples[:, 0].unsqueeze(-1),
                        -half_nl_samples[:, 2:5],
                    ],
                    dim=1,
                ),
            ]
        ),
    ),
    components=half_nl.components,
    properties=half_nl.properties,
    values=torch.concatenate(
        [
            half_nl_values,
            -half_nl_values,
        ],
    ),
)

Maybe it makes sense to transfer this code to utilities? It might be useful to have something like this as a utility for other applications, and clean-up the code a bit.

Thanks a lot @abmazitov for testing this out. Unfortunately the factor of 2 is not something arbitrary but it's given by the necessity of making the potential continuous while re-using the neighbor list of the ML potential. I thought the values were pretty safe in a general case, but you have large atoms and a small cutoff so it goes over by a bit... I think the best solution is to make ZBL request its own NL, as painful as it might be, which would also spare us the NL conversion step that you highlighted.
Finally, the fact that the error is only raised after training is finished is a huge bug and I will be investigating it

Things to be done:

[ ] ZBL needs to be able to request its own NL in case it's larger than the one from the model or a different kind (ZBL always wants full, right now we're using a slow converter)
[ ] Improve the covalent radii design that we're inheriting from ASE, i.e., the elements for which ASE devs didn't find a covalent radius are listed as "0.0". And find a solution in case the user wants to use ZBL with these elements
[ ] Carefully check the units. The lengthscales are reasonable but the height of the repulsive wall is suspicious


G2Config.py dying when loading libG2.so

When running code like
g2_config = G2Config()

The code dies on this line:
https://github.com/Senzing/g2-sdk-python/blob/358ccc762b4a0da629227694dae7240fa60cb5ed/src/senzing/G2Config.py#L108

Moved to proper repository:  https://github.com/Senzing/g2-sdk-python/issues/104


Implement CustomerStatementExport .get .list .delete

Pull Request Test Coverage Report for Build 173

30 of 30 (100.0%) changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 99.582%

Totals

Change from base Build 171:
0.02%

Covered Lines:
715

Relevant Lines:
718

💛  - Coveralls

Pull Request Test Coverage Report for Build 173

30 of 30 (100.0%) changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 99.582%

Totals

Change from base Build 171:
0.02%

Covered Lines:
715

Relevant Lines:
718

💛  - Coveralls


Added Courses

What does this PR do?
Add resource(s)
For resources
Description
Why is this valuable (or not)?
It Covers all topic from beginner to advance
How do we know it's really free?
Available in youtube
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Please review and merge it

Thanks, and welcome to free-programming-books!


Allow spacing out the inner lines on Line Chart

When there are a lot of data points, it starts to look cluttered:

It would be nice to be able to specify a number of vertical lines to show, similar to the segments property.

hi @JBaczuk , I have created a props yAxisInterval .
And thanks to @Hermanya, it has been  merged and available in v5.0.0 :tada:


Dark Mode - Search query text not visible in search results

I'm currently using GPMDP 4.2.0 on macOS 10.12.3, and when Dark Mode is enabled search query text remains the default dark grey color and is therefore not visible; I tried switching back to Light Mode and back to Dark Mode, quitting and reopening GPDMP, and even uninstalling/reinstalling the app, but the issue seems to persist no matter what.
Here is what I'm referring to, the other unhighlighted results should show the search query on the first line:

After looking in Google Play Music, it appears this text answers to the CSS classes style-scope sj-entity-suggestion, but when I tried making a custom stylesheet to change the color for either of those classes and loaded it into GPMDP it didn't seem to take, so there seems be more at play than my limited CSS knowledge can grasp...

See #2341


Performance problems of the initial rendering of various editor modes

Describe the bug
The performance of the initial rendering of various editor modes seems to significantly differ.
By initial rendering I mean the moment a user switches between modes using the toolbar.
According to our observations and some measurements, the winner, interestingly enough, is the "edit" mode, while the absolute loser is, interestingly enough, the "resize" mode.
We clearly notice the difference with some huge pages we have here in our internal app, which has a bunch of custom plugins. So to rule out the plugin interference, I'll try to demonstrate using a standard example page.
The measurements show that the difference in performance is not caused by how long it takes for the component tree to reconcile (render), but how long it takes to commit it, which seems to be a function of the number of state altering side effects.
The best way to demonstrate the problem is by using the Timings feature of the Performance tab of Chrome Dev Tools. It uses the Timing API which was removed from the latest versions of React, so in order to reproduce the attached measurements you'll need to downgrade your React to 16.x (I used 16.14).
In the below screenshots I'm using cellSpacing example and simply switching between editor modes.
It makes a difference which mode you are switching from, so I took all measurements switching from "preview" to the target mode. When measuring "preview" performance I was switching from "edit".
 However, when compared to each other they seem to make sense.
The reconciliation phase is about the same in all the measurements, but the commit phase differs greatly.
The "edit" mode is the only one that doesn't seem to have any state altering side effects. Here is the graph for it:

"Insert" and "preview" modes seem to be about 50% slower, most likely because of some state altering side effect (cascading update warning):

The "move" mode is usually about 2 times slower, again with the cascading update warning:

And here's our champ, the "resize" mode:

It's usually 10-15 times slower than "edit"!
It shows the same mammoth number regardless of whether you enter or exit it.
The more plugins you have on the page, the bigger hit the "resize" mode sluggishness is going to take. Here is what we see in our case on a huge page with lots of plugins:

Another way to look at it is using a more standard React DevTools profiler. For 17+ React it is the only option. It lacks the cascading update warning, it doesn't clearly show you the time it took to perform the final stages of the commit, but you can tell the unhealthy cycle by the number of commits.
cellSpacing example page shows the difference of about 3x times more commits for "preview to resize" switch than "preview to edit" switch. This is a large number, but it's not that staggering. In our application, on a page with 120 plugins we have this:
Switching from "preview" to "edit":

And here is switching from "preview" to "resize":

It looks like a bunch of state altering size effects somewhere in or around ResizableRowCell component.
Expected behavior
Resize mode clearly needs fixing, but it would be nice get rid of cascading update warnings across the board.

thanks for the detailed analysis.
is the worse performance of the resize mode a problem or is it just an observation?
is "resize in edit mode" enabled (its enabled by default)?
ResizableRowCell is always mounted (if not readonly), so its probably that Draggable fires an event very often.
What we also should look at is useMeasure, maybe it has to recalculate very often due to some reasons.

Ah, glad you mentioned this - I was thinking this was my imagination, but I distinctly noticed a bit of sluggishness when switching to edit from preview on a heavy duty page. This is after upgrading to v2.0.0 from v1.7.2.

This is actually quite serious - this page (https://www.scoopcitizen.net/home) was taking 2 secs to move from

preview to edit and
revert from editing to normal

Now taking 5s for # 1 and 15 - 21 seconds for # 2. The browser tab is completely unresponsive during this duration.
Even allowing for react debug vs production mode, this is really long.

@macrozone,
When it takes over a second to perform a commit, then I'm afraid it's a real problem. The plugins are all in their read-only state, so it has no right to be slower than "edit", especially given the fact that we have "resize in edit" on (which answers your other question).
Ok, I looked at useMeasure implementation. It very much looks like our culprit!
Sets the state every time the observer fires. Yikes!
Since those calls will be performed in "outside of react" context, all those setState calls will be unbatched -- each of them will cause their own commit which is exactly what I'm seeing. Gotta batch them somehow and apply all at once.

@macrozone,
Okay, so we got three independent areas of concern. The last one is by far the most impacting.

react-draggable causes instances of "cascading updates" and "forced reflow". These have little effect, but are trivial to fix. I'll file an issue with them.

removing useMeasure does improve performance to an extent. It doesn't help the resize mode phenomenon, but it improves everything else. Right now it is used proactively and the associated observer etc is attached regardless of whether they even going to use the resize handles. The more cells there are, the costlier the whole thing becomes. It might be a good idea to consider enabling the associated logic only after they mouse down on the resize handle (or something like that).

and finally this: the whole problem with sluggish resize mode switch goes away if this block is commented out inside Cell/Draggable/index.tsx:

if (isResizeMode) {
  return <>{children}</>;
}

This is the current performance graph with all code intact (cellSpacing plugin):

And this is the graph with the above block commented out:

It turns out that the problem is related to the fact the "resize" is the only mode that bails out of building the "long" version of the component tree, so React has to tear the old structure down, unmount everything, run a ton of associated lifecycles, them mount everything and run another bunch of the associated lifecycles. Here is switching from "edit" to "resize":

Interestingly enough, even the "preview" mode keeps the long version of the tree.
Here's switching from "preview" to "edit": (notice all those "mounts" are "updates" now).

An interesting case of how making the component structure lean can lead to devastating results!
We trim the component structure and speed up to a negligible extent on rendering, but loose bit time on lifecycles.

It turns out that the problem is related to the fact the "resize" is the only mode that bails out of building the "long" version of the component tree, so React has to tear the old structure down, unmount everything, run a ton of associated lifecycles, them mount everything and run another bunch of the associated lifecycles. Here is switching from "edit" to "resize":

thats an interesting observation and makes kinda sense. So we just remove the
 if (isResizeMode) {
    return <>{children}</>;
  }

?

@macrozone,
I didn't make sure it works with this removed. I was just looking for an explanation for the phenomenon.
Some of the drag-n-drop logic that this early return circumvents might kick in and break that mode.
Some of it might need to be disabled.

@avoskresensky just checked. It works when removed, but you can drag cells around in resize mode then (which might be actually useful).
there are some other early returns in the component there. I can try to streamline them a bit

@macrozone,

It works when removed, but you can drag cells around in resize mode then (which might be actually useful).

I would rather have resize handles in the layout mode, lol !

@macrozone,

It works when removed, but you can drag cells around in resize mode then (which might be actually useful).

I would rather have resize handles in the layout mode, lol !

In earlier versions the modes were more separated. I think we could actually merge them and remove resize mode completly

@macrozone,

I think we could actually merge them and remove resize mode completly

I agree. It seems rather superfluous now. We found a good use for the layout mode however. We make those overlays that currently show on hover permanently visible via CSS and this mode turns into a sorta map of what's where:

Used some styling from v0.xxx to make those overlays contained.
The only thing we're missing there is resize handles from the resize/edit mode.

we could also remove allowMoveInEditMode option and just always enable it. It was an experimental feature at some point, but i think there is little reason to disable it.

No objections!

@macrozone,

I think we could actually merge them and remove resize mode completly

I agree. It seems rather superfluous now. We found a good use for the layout mode however. We make those overlays that currently show on hover permanently visible via CSS and this mode turns into a sorta map of what's where:

Used some styling from v0.xxx to make those overlays contained.
The only thing we're missing there is resize handles from the resize/edit mode.

it actually used to look exactly like that. I move the labels up, so that it won't overlap the content. But it would be nice to have them inside the cells again at least in layout mode

removing the options would be a breaking change though (strictly speaking). So i first clean it up a bit and fix the early returns, push it under v2

@avoskresensky see https://github.com/react-page/react-page/pull/968, what do you think? (it does not remove and change any functionality)

@macrozone ,
looking real solid. Tried it in our app and it works as expected.
Thanks!

:tada: This issue has been resolved in version 2.0.2 :tada:
The release is available on:

v2.0.2
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 2.1.0-beta.4 :tada:
The release is available on:

v2.1.0-beta.4
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 2.1.1 :tada:
The release is available on:

v2.1.1
GitHub release

Your semantic-release bot :package::rocket:

This is actually quite serious - this page (https://www.scoopcitizen.net/home) was taking 2 secs to move from
1. preview to edit and

2. revert from editing to normal

Now taking 5s for # 1 and 15 - 21 seconds for # 2. The browser tab is completely unresponsive during this duration.
Even allowing for react debug vs production mode, this is really long.

There's a remarkable difference on # 2. This is back to 2s. But # 1 is still 5secs or so.


[OFFICIAL/TV]* —Brighton vs Man City LiVe 𝐒𝐭𝐫𝐞𝐚𝐦𝐬 FrEe BrOaDcAsT oN TV Guide & 𝐒𝐭𝐫𝐞𝐚𝐦𝐢𝐧𝐠 09 November 2024

Brighton will host Manchester City in an EPL match at American Express Community’s Stadium in Falmer, England on Saturday (November 9).
The current season for the defending champions is not going smoothly right now; they lost the last three matches in three different competitions. Brighton’s last win was also three matches ago. Currently, the visitors are placed at the second position with 23 points in 10 matches while the hosts are placed at eighth with 16 points playing the same number of matches.
Manchester City face Brighton on Saturday with Pep Guardiola's team looking to avoid experiencing four defeats in a row for the first time under the Spanish manager.
A win for City would see them return to the top of the English Premier League, but the visitors will be without stars Jack Grealish, Rúben Dias and John Stones.
The home team, meanwhile, are in the midst of a tough run of fixtures, coming into this game off back-to-back defeats to Liverpool in the EFL Cup and the Premier League. Now may nevertheless be a great time to face the defending champs, and Brighton have the added incentive of knowing that win here will push them into the top four.
A heavily injury-stricken Manchester City will try their best to avoid losing four matches straight for the first time in Pep Guardiola’s managerial era. Brighton will also try to make a comeback after being defeated by the table topper Liverpool in the last match.
Pep Guardiola's side has gone off the boil after a bright start to the season. They come into the match on the back of three straight defeats. After Tottenham Hotspur knocked City out of the EFL Cup, Bournemouth handed them the first league defeat of the season. In a UEFA Champions League clash mid-week, City was stunned 4-1 by Sporting.
While the defending champions have slipped to No 2 in the league standings, an inconsistent Brighton is currently eighth with four wins and four defeats from their 10 matches so far.
Under Pep Guardiola, City have rarely seen a streak of defeats, and they're determined to avoid a fourth consecutive loss for the first time since his tenure began. While City have faced some rare stumbles, a trip to Brighton could be the perfect opportunity to reestablish their rhythm and regain their place near the top of the Premier League standings.
But the Seagulls, led by Roberto De Zerbi, are no easy target. Despite losing their last two matches to Liverpool in both the EFL Cup and the Premier League, Brighton has had an impressive season, showcasing a vibrant and organized playing style that has earned them respect across the league.
Manchester City are in unchartered waters right now - having lost three games running in separate competitions - but outsiders simply cannot write Pep Guardiola and his star-studded squad off just yet. City are still sitting pretty in second in the Premier League standings and there are mitigating circumstances for their struggles.
City have been hit harder than most when it comes to injuries this season and, while one or two first-team names are starting to return, Guardiola will still be without John Stones, Ruben Dias and Jack Grealish this weekend. That's without even mentioning Kevin de Bruyne (thigh) and long-term absentee Rodri (knee).
Without the Ballon d'Or winner for the rest of this season, City have had to be flexible in midfield after such a key blow, but there's still more than enough quality in their squad to make them favourites for a fifth straight title win. Here, with Brighton next up on Saturday evening, we take a closer look at how to follow that match-up;
Despite the losing streak, Manchester City remains sitting second in the Premier League, trailing just Liverpool. That’s because two of Manchester City’s defeats came in non-Premier League games.

as


Ignore telemetry events in tests

This solves part of the telemetry events not being matched in the expected text for some tests.
I have a lot of unit test failures when I run the tests, so it's difficult to see whether this addresses all the telemetry issues in the tests or not.
I added a .only option to itIntegrates

Thanks!


Unused variables

nginx-rtmp-module/ngx_rtmp_eval.c
nginx-rtmp-module/ngx_rtmp_handler.c
Both of these files contain unused variables which prevent nginx from installing smoothly.  Had to remove the offenders manually.

nginx-rtmp-module/ngx_rtmp_eval.c looks like it was fixed in https://github.com/arut/nginx-rtmp-module/commit/c56fd73def3eb407155ecebc28af84ea83dc99e5


Future region us-west8 (unknown)

The future region us-west8 is reported during execution of coverage-published test case. This issue will track that region's progress to GA and support in this module.

Not stale

Not stale

Not stale

Not stale

Not stale

The region us-west8 is still present in GCP metadata but is not yet online.


Add visualize_sample in main.py

Add a function that visualizes original, positive, and negative data samples so that users can check the data transformation.

Thanks for adding this feature!


I'm overwhelmed by the jobs overview page

I need to delete or hide old jobs.  If that isn't possible, I'd like to group job status by the job directory in the workspace.

@jimdavis1 Does the "running" filter on the alpha site help with this issue?

The main issue with this is that the jobs list is still limited to the last 10k jobs.  We should be able to increase this to 20k or so, but there's not much else that can be done without server side changes.

this is on alpha

whoops... I misread.  @jimdavis1 do you think this "fixed" on alpha?  It isn't exactly what you asked for, but may suffice for time being.

The new filters are helpful in finding jobs by types or status.
However, both me and Jim, submit hundreds/thousands of assembly annotation jobs in batch mode, often in the same workspace folder, because they are part of single study.
The problem is finding a specific job for follow up / looking at results. Currently the way I do it is by manually adding specific job id in the URL and go to that directory. It would be good if we have a way to search for jobs and / or workspace files by keyword.
@jimdavis1 do you agree?

Searching would be another task. I think people all liked current solution and this is already deployed on production last November 2017 release. Please create another ticket if you want extra feature.


Oasis dynamic mode?

Why not use oasis dynamic mode for development¹ instead of commiting again and again the generated files (setup.ml, _tags, myocamlbuild.ml...)
¹ Not for the final tarball for OPAM which then will be uploaded and associated to a Git tag instead of being the content of the Git tree.

To the first question, I had no particular reason for not choosing development mode so I'll give it a try.  To the second, oasis2opam wasn't released when I first started development and I haven't had time to switch to it yet.  Happy to accept changes ;)

For oasis2opam nothing has to be changed. In the directory packages/ of an OPAM-repository checkout, issue oasis2opam URL_of_your_tarball and the OPAM files will be created (just make sure that _oasis — and _opam if any exists — are present in the tarball).


Document the E2e setup details.

Document the way OpenEBS E2e setup is built and run.
https://github.com/openebs/e2e-tests/wiki/On-premise-E2e-Setup-using-GitLab

Oenshift 4.2 setup details: https://docs.google.com/document/d/1C4TXdd-ZSror66F_AdQNBP9q3fxN9_vaoYUDuoIMIoQ/edit

Gitlab setup and troubleshooting DOC: https://docs.google.com/document/d/150m696Kc6n85dppeFKMK6fwCTac3M84sCyP64k6mJeA/edit#heading=h.az0qx7ucyh1k


5735 set up uat

Code changes required for UAT environment:

Adding option to select UAT in github workflow
Making keycloak client-id as configurable parameter.

@weskubo-cgi : Can we remove the .env. files now with application reading custom frontend properties from config.js?

I'd prefer to leave the env files in as the code supports both ways of configuration and they will be used during local development and will support standard builds using vite build.

I think there's just the one issue with updating .env to address. Also, just wanted to make sure that application-dev.yaml will be updated in svn.
@weskubo-cgi Configurations for backend are saved in application.properties in ArgoCD repo. I feel we do not need to maintain application-dev.yaml file in SVN.


wrong item icon size reproduction

Description
found a way to reproduce the wrong item icon size thing using banana
Reproduction

pick up a banana peel
pick up a banana and eat it
new banana peel is small
switch hands and drop other banana peel
the new banana peel gets correct size

Screenshots

(ignore item cooldown thing its defib bug)

Repro steps don't seem to work
Uploading Content.Client_xcGUnhcxth.mp4…

maybe it was an old thing in lizard idk


defer to base retry check when response is nil

We use a custom retryable http client to retry GET requests. Since it is intended to only retry GET requests we check that the request was a GET at the beginning of the method.  The only way to access the request at this point in the code is through the response object, however, there are some cases where this  object is nil.  This was resulting in a nil pointer exception.
After this change, any errors that are seen without a set Response object will fall through to the default handler.
Fixes a Nil pointer exception reported here:
https://cockroachlabs.slack.com/archives/C01J7MGJ3S7/p1696613858625329?thread_ts=1696613850.476459&cid=C01J7MGJ3S7
Commit checklist

✔  Changelog
[ ] Doc gen (make generate)
[ ] Integration test(s)
[ ] Acceptance test(s)
[ ] Example(s)

Yeah, I'm not sure.   I looked through the httpclient.Do code and found that there are a few places that can return a nil response.  It could be that this change is just kicking the can up the road but we'll see.
here is one example:
https://cs.opensource.google/go/go/+/refs/tags/go1.21.3:src/net/http/client.go;l=674


add pip upgrade on Dockerfile

I added a pip upgrade on Dockerfile.
Without this modification, wheeling grpcio is too long to complete first step.
By upgrading pip version from 9 to 20 (or more), we can access grpc's .whl file directory.
Reference

https://github.com/grpc/grpc/issues/22815

@googlebot I signed it!


Fixing Locale issue in UTest

Fixing UTest on computer where default locale is not US.

Wrong commit name


Write a page on String, DString, ZString

How to use each, how to use with temporary allocator

coppamocha — Today at 09:36
do we need to free Strings or are they automatically freed.
asking this cuz, i tried .free() method and the program aborts:
free(): invalid pointer
[09:37]
is this because maybe the string is allocated on stack?
[09:38]
if so, is there a way to detect if the string was allocated on stack?

Christoffer L — Today at 10:36
No, you need to know where this String comes from @coppamocha. That is one of the drawbacks of using String. You can use the DString if you want to have something that knows how to free itself.

string.iterator() iterates over codepoints, so you get Char32 back

For additional info see: https://github.com/c3lang/c3c/issues/1612


ORC-87: [C++] Handle missing timezone conversion for timestamp statistics

I think we should probably add new methods for timestamp statistics for getLowerBound and getUpperBound that will be the same for the new version of the statistics, but be a looser bound in the case of the older files.

@omalley I have added methods for timestamp statistics for getLowerBound an getUpperBound


How to reproduce the demo video results.

First of all, thanks for you great efforts on maintaining this project.
The outpaining results on my own images are relatively bad, so I am trying reproduce the reuslts of the demo video to make sure my implementation is correct.
However, I find that when using the same init image and prompt (with SD 1.4 ckpt), I cannot get results as good as the authors, see the figure below:

So I would like to ask, for the demo results in the video, did the author get pretty good results after each "outpainting click", or try outpainting for each place many times, then cherry pick the best one to make the video?
Many thanks.

The video is edited to remove safety-checker affected results (e.g. I intended to outpaint a wooden closet on the left, but the safety-checker was triggered. so you may notice a selection box shift at the second outpainting) and reduce the length to 30s (e.g. chandelier was tried multiple times). However, the girl and the right part of the image are generated continuously.
Please note the all generations are random, so it may be difficult to reproduce a certain generation sequence.


Android sdk update

Can you please move your code from the 1.0.0 branch back to master, and just add a simple git tag with 1.0.0 as the release? This'll make it much easier for you to keep track of releases and to accept contributions / Pull Request for your step in the future.

Done.

Hey Viktor,
Can you check it now ? I made all the changes that you suggest.
Thanks

I'll check it ASAP, hopefully tomorrow I'll have the time.

And of course thanks for the changes & for your time! This step will be fantastic once released ;)

Note: this'll still overwrite the value provided by the user: https://github.com/guitcastro/steps-update-sdk/blob/master/step.sh#L3

FYI to test a Step easily you should run it with bitrise (CLI), not just the step.sh directly. Alternatively you can provide all the inputs as environment variables, but every variable you define in your step.sh will override the value defined by the user, either through bitrise or through an environment variable.

One more thing, you don't need to call ls or any loop for checking the existence of a directory (like you do here: https://github.com/guitcastro/steps-update-sdk/blob/master/step.sh#L9 ) - you can use the Bash built in operator in an if : if [ -d "$DIR_PATH" ] ; then -> the -d option will only be true if the path exists and it is a directory. For files you can use -f. So, in case of the checkIfIsInstalled it can be as simple as:
function checkIfIsInstalled {
  if [ -d "$ANDROID_HOME/$1/$2" ] ; then
    return 0
  else
    return 1
  fi
}

@viktorbenei You are right, testing using biirise cli is more reliable and prevents me from forget to remove the variables.
A changed and tested your function (using -d instead of ls) and it's working like a charm.
A think it's finally ready to go :)

These are the things I mentioned on Slack & the ones I found:

https://github.com/guitcastro/steps-update-sdk/blob/master/step.yml#L23 -> a default value should be set, because this input has value_options
https://github.com/guitcastro/steps-update-sdk/blob/master/step.yml#L33 -> shouldn't the default be "none"?
https://github.com/guitcastro/steps-update-sdk/blob/master/step.yml#L49 -> is_required should be false
https://github.com/guitcastro/steps-update-sdk/blob/master/step.yml#L59 -> is_required should be false

Finally you changed your code, so you'll have to create a new PR here - because this PR now points to an older commit hash (easiest way is to simply delete this one and just share the current version again).

Sent a PR with these fixes ;)

Hey Viktor, Thanks for the PR.
My HD broken, I'm only with my notebook (which does not have linux neither os x).
I was waiting for the repair to fix it, thanks for the PR =)

Sure, no hurry ;)

I just submitted another PR. I am closing this one ;)


First changeing after Pipewire started lower output volume

This is how it is,

Starts pipewire and wireplumber
Plays some audio
pw-volume status says percentage is 1. Definitely, current audio output is not 1%
any change,  cause actual output to lower.

I think, the percentage is wrong at 3rd step, actually it might some higher, say 30%, while at 4th step, when I changed for 1%+, the actual output lowers. Means the sound really becomes 2%.
So, I guess, we need to have a look how pw-volume got incorrect percent inititialy, after launching pipewire.
I'm using Artix with OpenRC as init.

How was it to check volume level with pw-cli?

Thanks for providing such detailed information in the issue! I agree, it seems like the volume that pw-volume is reporting on your system doesn't match what it should be.

To clarify, you are now running the latest binary I uploaded (v0.2.0) correct?
If you want to retrieve volume information from pw-cli, you can run

$ pw-cli enum-params <node-id> Props

Here's what the output looks like on my system:
$ pw-cli enum-params 44 Props
  Object: size 680, type Spa:Pod:Object:Param:Props (262146), id Spa:Enum:ParamId:Props (2)
    Prop: key Spa:Pod:Object:Param:Props:volume (65539), flags 00000000
      Float 1.000000
    Prop: key Spa:Pod:Object:Param:Props:mute (65540), flags 00000000
      Bool false
    Prop: key Spa:Pod:Object:Param:Props:channelVolumes (65544), flags 00000000
      Array: child.size 4, child.type Spa:Float
        Float 0.750000
        Float 0.750000
...

As you can see, channelVolumes is an array containing [0.75, 0.75].
And pw-volume status reports
$ pw-volume status
{"percentage":8, "tooltip":"7.5%"}

To clarify, you are now running the latest binary I uploaded (v0.2.0) correct?
Yep.

pw-cli enum-params 39 Props

Just after pipewire and wireplumber,

...
    Prop: key Spa:Pod:Object:Param:Props:channelVolumes (65544), flags 00000000
      Array: child.size 4, child.type Spa:Float
        Float 0.164012
        Float 0.164012
...

After change +1%

...
  Prop: key Spa:Pod:Object:Param:Props:channelVolumes (65544), flags 00000000
    Array: child.size 4, child.type Spa:Float
      Float 0.164012
      Float 0.164012
...

So, it's Pipewire that reports incorrect parameter.

Yup, I notice this on my system as well now.

I restarted it.
Played a song.
pw-volume status reported 3.9999200000000004%
Ran pw-volume change 0% to ensure volume did not change.
However volume is now perceptibly lower, even though pw-volume still reports 3.9999200000000004%.

I'm wondering if there's some other way to get the true volume.

Edit: It seems that others have experienced issues with pipewire reporting inaccurate volume too? jaor/xmobar#535

 pamixer --get-volume, pavucontrol and Waybar volume module displays the real outpu percent if pulsewire-pulse is running

Yeah pamixer reports 74% unless I use pulseaudio tools to change volume. It seems that each of these engines (alsa, jack, pulseaudio) maintain their own state about volume, that pipewire doesn't know about. And some range from 0-150% of volume capacity, where pw-volume stays strictly in 0-100.
Looking at this issue: https://gitlab.freedesktop.org/pipewire/pipewire/-/issues/1475, my guess is that this comes from running pulseaudio and pipewire at the same time? I'll investigate more.

I uninstalled pulseaudio (and disabled pulseaudio emulation) and rebooted. I confirm that I don't see this weird behavior anymore.
I'd like to have the pulseaudio emulation still running, but I suspect that applications default to going through pulseaudio, so there needs to be a way to opt out of that in order to use pipewire volume accurately.

Maybe we can ask this issue on #pipewire matrix group

Just so you know, I haven't forgotten about this. I just recently joined a new job, and it'll take some time before I can wind down and attend to this. Hope you can understand!

I think I ended up figuring out the issue. I've been setting the stream volume instead of the output device volume all this time.
In pulseaudio terminology, I've been setting the sink input volume, instead of the sink volume.
The correct way to do it is https://gitlab.freedesktop.org/pipewire/pipewire/-/wikis/Migrate-PulseAudio#sinksource-port-volumemuteport-latency.
I'll have to rewrite a bunch of the logic to address this. After getting the node id, I have to look up its device id and card profile device.
With the device id, I look up its active routes, and execute the command
pw-cli s <device-id> Route '{ index: <route-index>, device: <card-profile-device>, props: { mute: false, channelVolumes: [ 0.5, 0.5 ], save: true } }'

I'll try to send out a change for this this week, along with other mic-related updates.

And another thing, pipewire saves its volume state in the ~/.local/state/ directory. Although, with the approach in the previous comment, we don't need that local state anymore.

The latest release https://github.com/smasher164/pw-volume/releases/tag/v0.3.0 should have the fix. Let me know if it works for you.


feature: select message key from kafka external stream

PR checklist:

Did you run ClangFormat ?
Did you separate headers to a different section in existing community code base ?
Did you surround proton: starts/ends for new code in existing community code base ?

Please write user-readable short description of the changes:
Allow to select _message_key from kafka external streams. refs #400

(Jove Github Bot) added it to the current sprint.


manpages.debian.org should be a psuedo package for bugs.debian.org

It would be nice/great if manpages.debian.org was a psuedo package/service like so many which are listed at https://www.debian.org/Bugs/pseudo-packages . It would make it easier to reporter and track down things.

This is already in progress: https://bugs.debian.org/851885

"in progress"? seems stalled to me. ;) i pinged the issue to try and figure out the next step. in the meantime, i documented the (eventual) pseudo-package in the nascent FAQ section so that needs review before merging.

"in progress"? seems stalled to me. ;) i pinged the issue to try and figure out the next step. in the meantime, i documented the (eventual) pseudo-package in the nascent FAQ section so that needs review before merging.

this should also be fixed on the next rebuild.


The project defaults to using mkdocs for documentation but many in the community use sphinx. Could we have a docs option? sphinx vs mkdocs?

I'm torn between two possibilities:

Set up the template only with the sphinx configuration as that is what this community recommends. Limit choices and thereby complexity of using the template.
Create different options:

Sphinx
mkdocs
no documentation

I like the idea of offering both, but I think we should have a short-circuit option up front that asks if you want to use the default settings for everything that doesn't strictly require input.

There is one more side-effect of this decision in my opinion. When the documentation is sphinx-based, I probably want to keep the contributing guide, code of conduct, etc. as .rst files, such that I can easily include them in the documentation built. Otherwise, I would probably leave them as markdown.
They can certainly exist as duplicates, either in the project instance as both md and rst, one for the repo and one for the docs, or as duplicates in the template where files get included based on user choices. Both come with the usual caveats of duplicating sources.

I would keep the files as .md and use myst_parser

This is all great discussion. i've definitely kept those files as md files as @blink1073 suggests. I've also included them in my docs using the sphinx include directive
I think we should ensure that these files are included even if the user doesn't use git / github. that brings us to placing them either:

in the docs dir or
in the root.

given these are two plaes that are common for them to live.
What do you both think about

Let's have options for docs  as described above 1) sphinx 2) mkdocs 3) no docs (least preferred)
Let's for now keep the contributing, COC, etc files in the project root and see how it goes. I can see positives and draw backs for docs vs root and i don't have a strong opinion. However i did have a look at both numpy and scipy and both have contributing at the root. Pandas doesn't! But maybe we follow the core packages here?
I think @blink1073 idea of a short cut is excellent. i'll turn that into a new issue. let's come up with some set of defaults that we think is good and use that too. <new issue - not related here>!!


Fatal error: Uncaught exception 'Exception' with message 'getAttestation Exception: HTTP Status Code != 200'

Fatal error: Uncaught exception 'Exception' with message 'getAttestation Exception: HTTP Status Code != 200' in C:\xampp\htdocs\src\snapchat.php:217 Stack trace: #0 C:\xampp\htdocs\src\snapchat.php(470): Snapchat->getAttestation('password',<PHONE_NUMBER>273) #1 C:\xampp\htdocs\config\config.php(54): Snapchat->login('password') #2 C:\xampp\htdocs\index.php(3): require_once('C:\xampp\htdocs...') #3 {main} thrown in C:\xampp\htdocs\src\snapchat.php on line 217
This is the error returned by my program. It was working perfectly last night, nothing was changed and today it simply isnt working and I cant figure out why. I see this is a common issue because people have opened topics before, Im curious as to how you fixed it.
Here is a test dump:
http://pastebin.com/68kcYXNK

#201

Completely useless answer, thanks. I'm glad to hear other users are having the same issue and it's not being addressed.

@TGEJesse if you don't read you will never get the answer
`
Hiya,
My API gateway server was undergoing an upscale. This is because of the increased traffic.
It's all back up and running again.
Cheers,
Liam Cottle
`
Useless answer, sigh...

I dont understand how the casper app has to do with your api? Do you use his gateway server? (Im assuming).  YES WE DO USE LIAM'S SERVER.
NOW...do you know how you could have figured that out all on your own without me telling you here??
    STEP 5: LOOK AT THE CAUSE OF THE ERROR
$ch = curl_init();
curl_setopt($ch, CURLOPT_URL, "https://www.googleapis.com/androidcheck/v1/attestations/attest?alt=JSON&key=AIzaSyDqVnJBjE5ymo--oBJt3On7HQx9xNm1RHA");
curl_setopt($ch, CURLOPT_RETURNTRANSFER, TRUE);
curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, FALSE);
curl_setopt($ch, CURLOPT_HEADER, FALSE);
curl_setopt($ch, CURLOPT_POST, TRUE);
curl_setopt($ch, CURLOPT_POSTFIELDS, $postData);
curl_setopt($ch, CURLOPT_HTTPHEADER, array(
	'Accept:',
	'Expect:',
	'User-Agent: SafetyNet/7899000 (WIKO JZO54K); gzip',
	'Content-Type: application/x-protobuf',
	'Content-Length: ' . strlen($postData),
	'Connection: Keep-Alive'
));
curl_setopt($ch, CURLOPT_ENCODING, "gzip");
$return = curl_exec($ch);

if(curl_getinfo($ch, CURLINFO_HTTP_CODE) != 200)
{
	throw new Exception("getAttestation Exception: HTTP Status Code != 200");
}

OH HEY THERE'S A REQUEST.  THAT MEANS. THAT IM SENDING SOMETHING. THAT IS NOT RIGHT. TO GOOGLE. SO I SUPPOSE. THAT MEANS. THAT THE PROBLEM. WOULD BE WITH ATTESTATION. AND IF IM NOT MAKING THAT DATA MYSELF. THEN IT MUST BE EITHER LIAM'S SERVER DATA. OR THE DATA THAT IM GIVING TO LIAM'S SERVER.

STEP 7: COOL COOL NOW THAT WE HAVE THAT FIGURED OUT, LET'S CONCLUDE

OH WAIT. THAT MEANS. THAT IM SENDING SOMETHING. THAT IS NOT RIGHT. TO GOOGLE. SO I SUPPOSE. THAT MEANS. THAT THE PROBLEM. WOULD BE WITH ATTESTATION. AND IF IM NOT MAKING THAT DATA MYSELF. THEN IT MUST BE EITHER LIAM'S SERVER DATA. OR THE DATA THAT IM GIVING TO LIAM'S SERVER.

TADAAAAAAAAAAAAAAAAAAAAAA

rip in peaches

Hi, mgp25
I got this issue . how did you get Apk_digest ?
Snapchat team updated as version 9.15.0.
if you have any solution , let me know it
Happy working .

Guys, plz chill. The issue is with my servers. Google made some server side changes, and I am yet to fix this issue on my servers.
You'll all need to wait until I fix this problem.
Until then, anyone using the Casper API for attestations, such as this PHP Lib, will not be able to login.
Thnkz :)

Hello, sorry to add a new comment to a resolved issue, but even after a fresh git pull and deleting every .dat files in the src/authData directory, I'm still not able to get the attestation and get the error :
Fatal error: Uncaught exception 'Exception' with message 'getAttestation Exception: HTTP Status Code != 200' in /_MY_PATH_/SC-API/src/snapchat.php:181
... which corresponds to the curl call on https://api.casper.io/droidguard/attest/binary
Is liamcottle still in the vicinity ?
Just in case, I made sure that both my snapchat and gmail passwords only contain alphanumerical chars.
I also tested it from another PC and another IP, but it didn't change anything.

@plopinou same here

It's working today, so I suppose Liam did something on his end. Thanks :)


add open props

Added support to handle if drawer is open or closed through props
I need this so I can control if drawer is open from anywhere in the app using flux, no need to pass this.drawer reference

nvm forget about this, at the end I am using this.drawer.open


unable to update lead using requestPatch()

jsforce: v1.11.1
node: v20.11.0
I'm trying to update lead using Composite API, but i'm seeing METHOD_NOT_ALLOWED: HTTP Method 'PATCH' not allowed error. And the code that i used to make this call is as shown below
async function updateLead(leadId) {
  const leadUpdatePayload = {
    "allOrNone": false,
    "compositeRequest": [
        {
          "method": "PATCH",
          "url": `/services/data/v60.0/sobjects/Lead/${leadId}`,
          "referenceId": "updateLead",
          "body": {
            "LastName": "Composite API updated",
            "Phone": "1234"
          },
        },
      ],
  };

  try {
    const result = await conn.requestPatch(`/services/data/v60.0/composite`, leadUpdatePayload);
    console.log("Lead updated successfully:", result);
  } catch (error) {
    console.error("Error updating lead:", error);
  }
}

Can someone help me why i'm seeing this error. Thank you in advance.

the PATCH method in the leadUpdate payload is ok but you are using conn.requestPatch which is trying to do a PATCH request (the composite endpoint only accepts GET and POST).
Try using conn.request and make it do a POST request, no need to change the payload.

Hi @cristiand391, i made changes as per your suggestion and didn't see any errors. but i don't the new changes in salesforce.
here is the code that i used, and i tried passing the payload as it is and also by using JSON.stringify()
async function updateLead(leadId) {
  const leadUpdatePayload = {
    "allOrNone": false,
    "compositeRequest": [
        {
          "method": "POST",
          "url": `/services/data/v60.0/sobjects/Lead/${leadId}`,
          "referenceId": "updateLead",
          "body": {
            "LastName": "Composite API updated",
            "Phone": "1234"
          },
        },
      ],
  };

  try {
    const result = await conn.request(`/services/data/v60.0/composite`, JSON.stringify(leadUpdatePayload));
    console.log("Lead updated successfully:", result);
  } catch (error) {
    console.error("Error updating lead:", error);
  }
}

response:
Lead updated successfully: {
  tree: '/services/data/v60.0/composite/tree',
  batch: '/services/data/v60.0/composite/batch',
  sobjects: '/services/data/v60.0/composite/sobjects',
  graph: '/services/data/v60.0/composite/graph'
}

@raghu64 the payload was correct, it should specify PATCH not POST.
The conn.request call should be a POST, by default it does GET which gets you those endpoints:
https://developer.salesforce.com/docs/atlas.en-us.api_rest.meta/api_rest/resources_composite_composite_get.htm

@cristiand391 i tired with PATCH as well, but i'm still not able to see the new changes in salesforce.

can you share the code you are using to make the request?

i tried to pass the json object as it is to the conn.request and also by using JSON.stringify()
async function updateLead(leadId) {
  const leadUpdatePayload = {
    "allOrNone": false,
    "compositeRequest": [
        {
          "method": "PATCH",
          "url": `/services/data/v60.0/sobjects/Lead/${leadId}`,
          "referenceId": "updateLead",
          "body": {
            "LastName": "Composite API updated",
            "Phone": "1234"
          },
        },
      ],
  };
console.log(JSON.stringify(leadUpdatePayload,0,2))
  try {
    const result = await conn.request(`/services/data/v60.0/composite`, leadUpdatePayload);
    console.log("Lead updated successfully:", result);
  } catch (error) {
    console.error("Error updating lead:", error);
  }
}

@raghu64 conn.request is doing a GET request, that's why you get this response:
Lead updated successfully: {
  tree: '/services/data/v60.0/composite/tree',
  batch: '/services/data/v60.0/composite/batch',
  sobjects: '/services/data/v60.0/composite/sobjects',
  graph: '/services/data/v60.0/composite/graph'
}

see the example I linked above to make conn.request do a POST request, the response should be something like this:
https://developer.salesforce.com/docs/atlas.en-us.api_rest.meta/api_rest/responses_composite.htm
see: https://developer.salesforce.com/docs/atlas.en-us.api_rest.meta/api_rest/resources_composite_composite_post.htm

@cristiand391 that worked. Thank you for your help.


merge from template-arlac77-github

package.json

chore(scripts): add ava --timeout 4m tests/-ava.mjs tests/-ava-node.mjs (scripts.test:ava)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
61

Relevant Lines:
61

💛  - Coveralls


Support for Server Side Encryption specification

There is no issue open for this problem, but if you want to turn on server side encryption for your tables you cannot use offline development anymore, because CloudFormation syntax is different than the JSON object required by createTable function.
Current error:
MultipleValidationErrors: There were 2 validation errors:
MissingRequiredParameter: Missing required key 'Enabled' in params.SSESpecification
UnexpectedParameter: Unexpected key 'SSEEnabled' found in params.SSESpecification
at ParamValidator.validate (/Users/myuser/project/path/node_modules/aws-sdk/lib/param_validator.js:40:28)

CloudFormation docs: https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-dynamodb-table-ssespecification.html#cfn-dynamodb-table-ssespecification-sseenabled
SSESpecification object: https://docs.aws.amazon.com/amazondynamodb/latest/APIReference/API_CreateTable.html#DDB-CreateTable-request-SSESpecification

@raducoti Thank you very much for the addition. 🥇

sls dynamodb migrate
Unexpected key 'SSESpecification' found in params

@raducoti I'm still getting errors even after you patch, is there anything else I need to update?

nvm it was a missing bump #155


ART-2960 release:calc-upgrade-tests

Codecov Report

Merging #519 (b20b9b0) into master (633736b) will increase coverage by 30.40%.
The diff coverage is 8.92%.

@@             Coverage Diff             @@
##           master     #519       +/-   ##
===========================================
+ Coverage        0   30.40%   +30.40%
===========================================
  Files           0       42       +42
  Lines           0    10180    +10180
  Branches        0     2170     +2170
===========================================
+ Hits            0     3095     +3095
- Misses          0     6889     +6889
- Partials        0      196      +196

Impacted Files
Coverage Δ

doozerlib/cli/__main__.py
0.00% <0.00%> (ø)

doozerlib/cli/release_calc_upgrade_tests.py
0.00% <0.00%> (ø)

doozerlib/util.py
49.11% <15.15%> (ø)

doozerlib/rhcos.py
71.42% <0.00%> (ø)

doozerlib/pushd.py
90.47% <0.00%> (ø)

doozerlib/state.py
50.00% <0.00%> (ø)

doozerlib/cli/scan_sources.py
16.48% <0.00%> (ø)

doozerlib/cli/config_plashet.py
0.00% <0.00%> (ø)

doozerlib/cli/release_gen_payload.py
17.57% <0.00%> (ø)

doozerlib/dblib.py
37.09% <0.00%> (ø)

... and 35 more

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 633736b...b20b9b0. Read the comment docs.


feat: sieve

I'm OK with it as stands. It aligns with how other exercises did it (all-your-base if I recall).


APP-1315: AppId not supported by SBE saveToken API.

Codecov Report

Merging #131 into dev will decrease coverage by 0.29%.
The diff coverage is 100%.

@@           Coverage Diff           @@
##             dev    #131     +/-   ##
=======================================
- Coverage   88.7%   88.4%   -0.3%
=======================================
  Files        141     171     +30
  Lines       2788    3510    +722
  Branches     261     326     +65
=======================================
+ Hits        2473    3103    +630
- Misses       256     321     +65
- Partials      59      86     +27

Impacted Files
Coverage Δ

...ation/pod/api/client/IntegrationAuthApiClient.java
84.21% <100%> (+0.28%)
:arrow_up:

...ioning/exception/AppRepositoryClientException.java
100% <0%> (ø)

...ning/service/ConfigurationProvisioningService.java
93.75% <0%> (ø)

...tegration/provisioning/service/KeyPairService.java
91.66% <0%> (ø)

...ation/provisioning/IntegrationProvisioningApp.java
50% <0%> (ø)

...sioning/exception/CompanyCertificateException.java
100% <0%> (ø)

...ation/provisioning/service/ApplicationService.java
95.65% <0%> (ø)

...visioning/exception/UsernameMismatchException.java
100% <0%> (ø)

...ion/provisioning/client/model/AppStoreWrapper.java
94.11% <0%> (ø)

...roperties/ConfigurationProvisioningProperties.java
0% <0%> (ø)

... and 21 more

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 3b21f4d...30c463b. Read the comment docs.


Fallback to MSBuild 14.0

On my machine this the MSBuild 15.0 path doesn''t have a bin folder. Don't ask me why.
This makes the build  work for me again.
[I wished this project would use FAKE and it's MSBuild detection]

reinstalled VS2017 and it works. damn I wished we had working software.


cytnx'c ncon() contraction order

In the original ncon, there is an optional argument "sequence" which can be used to specify contraction sequence. It seems that cytnx does not support this? and always try to find the (sub)-optimal contraction order?

There’re options “optimize” and “cont_order”
Which can be used to specify contraction order


No support for PATCH verb

To use the MailChimp 3.0 you have to use the method PATCH to do updates. Any reason this has not been implemented?

I see you've already made the changes in this branch: https://github.com/hgoebl/DavidWebb/tree/method_PATCH any reason it wasn't merged?

I tried the branch and the first thing I noticed is that PATCH is not supported by any JDK that uses HttpURLConnection.class. This workaround does work even though it's a big hack:  https://stackoverflow.com/a/46323891/142824 and I got it working in your branch.
other than that there are some tests that fail now. Testing against a remote service like google maps is going to fail sooner or later. The reflection hack above requires java 1.8 so some of the plugins in maven stopped working. Had to remove them to build successfully

If you're just starting a new project, you might consider using OkHttp instead. AFAIK some Android versions use this as an HttpUrlConnection implementation. This lib could use this natively as well, but what's the point of wrapping a very powerful and easy to use library by another and losing features?
Still, it would be an alternative to use OkHttp instead of HttpUrlConnection under the hood. Currently, I've no time to change this.


Add urltransform as trusted nework filter modifier

https://github.com/gorhill/uBlock/commit/2e4525fe3c

Duplicates https://github.com/AdguardTeam/CoreLibs/issues/1364


make chef install for busser optionally

added option to make chef install optionally as it is is in kitchen-puppet and kitchen-ansible.

Working on merge of multiple PRs. Considering rename the option to require_chef. Btw. there is an similar (but not implemented) option on kitchen docker: https://github.com/test-kitchen/kitchen-docker/search?utf8=✓&q=require_chef_omnibus.
In both cases, require_chef better describes the result.

I support the require_chef flag!

i support the toggle too, as long as busser still works at the end of the day.

👍


[Feature] Cover CNF Test Suite vs Clusters where Pod Security Standard restricted is enforced

Is your feature request related to a problem? Please describe.
CNF Test Suite cannot run today vs Clusters where Pod Security Standard restricted is enforced.
To work vs hardened clusters in production, Functest Kubernetes has to patch all 3 namespaces before running CNF Test Suite.
https://github.com/opnfv/functest-kubernetes/blob/master/functest_kubernetes/cnf_conformance/conformance.py#L67-L86
Describe the solution you'd like
I would like to see github actions verifying CNF TestSuite under the following configuration:
https://kubernetes.io/docs/tutorials/security/cluster-level-pss/
apiVersion: apiserver.config.k8s.io/v1
kind: AdmissionConfiguration
plugins:
- name: PodSecurity
  configuration:
    apiVersion: pod-security.admission.config.k8s.io/v1
    kind: PodSecurityConfiguration
    defaults:
      enforce: "restricted"
      enforce-version: "latest"
      audit: "restricted"
      audit-version: "latest"
      warn: "restricted"
      warn-version: "latest"
    exemptions:
      usernames: []
      runtimeClasses: []
      namespaces:
        - kube-system
        - local-path-storage

**How will this be tested? aka Acceptance Criteria (optional) **
The gate job would work and any new issue in CNF Test Suite would be detected premerge

A simple way to setup an hardened cluster as described in https://kubernetes.io/docs/tutorials/security/cluster-level-pss/
$ cat kind.v1.29.yaml
kind: Cluster
apiVersion: kind.x-k8s.io/v1alpha4
nodes:
- role: control-plane
  image: kindest/node:v1.29.0
  kubeadmConfigPatches:
  - |
    kind: ClusterConfiguration
    apiServer:
        extraArgs:
          admission-control-config-file: /etc/config/cluster-level-pss.yaml
        extraVolumes:
          - name: accf
            hostPath: /etc/config
            mountPath: /etc/config
            readOnly: false
            pathType: "DirectoryOrCreate"
  extraMounts:
  - hostPath: /home/opnfv/K8S
    containerPath: /etc/config
    # optional: if set, the mount is read-only.
    # default false
    readOnly: false
    # optional: if set, the mount needs SELinux relabeling.
    # default false
    selinuxRelabel: false
    # optional: set propagation mode (None, HostToContainer or Bidirectional)
    # see https://kubernetes.io/docs/concepts/storage/volumes/#mount-propagation
    # default None
    propagation: None
- role: worker
  image: kindest/node:v1.29.0
- role: worker
  image: kindest/node:v1.29.0

$ cat /home/opnfv/K8S/cluster-level-pss.yaml
apiVersion: apiserver.config.k8s.io/v1
kind: AdmissionConfiguration
plugins:
- name: PodSecurity
  configuration:
    apiVersion: pod-security.admission.config.k8s.io/v1
    kind: PodSecurityConfiguration
    defaults:
      enforce: "restricted"
      enforce-version: "latest"
      audit: "restricted"
      audit-version: "latest"
      warn: "restricted"
      warn-version: "latest"
    exemptions:
      usernames: []
      runtimeClasses: []
      namespaces:
        - kube-system
        - local-path-storage


Creates sdcore management interface

SD-Core Webui is used to configure SD-Core.
In Charmed-5G SD-Core NMS Operator provides a user interface that leverages SD-Core Webui Operator to configure the network based on user input.
This spec describes the relation interface that is needed that Webui to pass the needed url to the NMS.

Code formatting is not checked for interfaces folder and we can change this line https://github.com/canonical/charm-relation-interfaces/blob/34782a04276a4874cb0b243cd61ca88463ae8b26/tox.ini#L14 as follows:
all_path = {[vars]tst_path} {[vars]docs_path} {toxinidir}/run_matrix.py {toxinidir}/interfaces

Code formatting is not checked for interfaces folder and we can change this line
https://github.com/canonical/charm-relation-interfaces/blob/34782a04276a4874cb0b243cd61ca88463ae8b26/tox.ini#L14
as follows:
all_path = {[vars]tst_path} {[vars]docs_path} {toxinidir}/run_matrix.py {toxinidir}/interfaces

If we start to check interfaces, several issues appear for linting.

I fully agree, however I would say that change is out of scope for this PR, as it will make CI fail because of other interfaces.

@simskij @PietroPasotti Any ideas why pytest-interface-tester-1.0  is being installed in this action in the CI then causing it to fail? In previous PR I saw pytest-interface-tester-0.0.3  was being installed, like here. I don't see any changes in dependencies that would cause this, or are there any? could the version of pytest-interface-tester in discover.yaml be pinned like it is in pyproject.toml?

Oh I see that 1.0 and above was released just a few days ago an 0.3.3 was the latest before that.
But why isn't it getting 1.0.3 then?

@simskij @PietroPasotti Any ideas why pytest-interface-tester-1.0  is being installed in this action in the CI then causing it to fail? In previous PR I saw pytest-interface-tester-0.0.3  was being installed, like here. I don't see any changes in dependencies that would cause this, or are there any? could the version of pytest-interface-tester in discover.yaml be pinned like it is in pyproject.toml?

it's installing the outdated (and buggy) 1.0.1. It should install 1.0.3 instead. Looking into why...

I think the culprit is this line https://github.com/saltiyazan/charm-relation-interfaces/blob/2e911a55845cb6635c7fd05d462b13182935337c/.github/workflows/discover.yaml#L21
causing a dependency conflict, resulting into 1.0.3 being unviable and forcing pip to fall back to earlier less constrained versions.
Merging https://github.com/canonical/charm-relation-interfaces/pull/101 will fix it: https://github.com/canonical/charm-relation-interfaces/pull/101/files#diff-02685a2eb98c9c2a8e69958ee993ee5f900b1be78b6911e014bddb6b533c1748

I think the culprit is this line https://github.com/saltiyazan/charm-relation-interfaces/blob/2e911a55845cb6635c7fd05d462b13182935337c/.github/workflows/discover.yaml#L21
causing a dependency conflict, resulting into 1.0.3 being unviable and forcing pip to fall back to earlier less constrained versions.
Merging #101 will fix it: https://github.com/canonical/charm-relation-interfaces/pull/101/files#diff-02685a2eb98c9c2a8e69958ee993ee5f900b1be78b6911e014bddb6b533c1748

Thank you @PietroPasotti

@Gmerold @ghislainbourgeois @gruyaume
I can't request your reviews on this PR, please one of you take a look :)


fix typo.

Fix typo in ingress cluserrolebinding
Moved from https://github.com/istio/istio/pull/7475

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ostromart
If they are not already assigned, you can assign the PR to them by writing /assign @ostromart in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

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

/cc @costinm
Any comments for this? You mentioned some upgrade issues here https://github.com/istio/istio/pull/7475#pullrequestreview-141333835

@costinm I think this will not break Istio upgrade, since we just change the reference name of clusterrole, instead of clusterrole itself.

@ymesika If you mean test upgrade from 0.8.0
Helm upgrading for istio currently broken because of an helm issue: helm/helm#1193
So I think helm upgrade from 0.8.0 to 1.0+ will be broken for other reasons.
But upgrading Istio with kubectl apply ... should be OK.

@sdake Thanks OK to close this.

@morvencao I wouldn't give up all hope - maybe someday kubernetes will permit the modification of the name field there.  There are several bugs of this type in Kubernetes and they receive alot of traffic and complaints.
Cheers
-steve

Maybe add a comment there to avoid the 6th?