👷(deployment) restrict build-and-push github actions to merge

Purpose
The CI builds and pushes back-end and front-end images to DckerHub every time we push to the repo, including on branches.
I want to relax the CI a little, restricting images builds and pushes to what is necessary (merges on main only, from what I know)
Proposal
Description...

[x] restrict to merges, see this GitHub doc on events triggering workflow

Last we spoke, Antoine sounded surprised that images would build outside of tagging but, from what I understood, staging might still need a docker images for kubernetes deployement. But build and push after merging on main might suffice.
To be discussed.

As for sharding, this PR is stuck indefinitely by branch protection rules

Do you confirm I can safely remove protection rules on build-and-push-backend and build-and-push-frontend ?

Yes I think you can remove protections rules. As I said you have to keep in mind that maybe sometimes the build will failed on main and no new docker image will be deployed.


Removed dependency for man package

Q
A

Bug fix?
Yes

New feature?
No

BC breaks?
No

Deprecations?
No

Fixed tickets
N/A

The man package isn't present on our servers (space savings & speed-ups during apt-get actions). The same info could be retrieved with the --help attribute.
Without this fix Deployer won't use atomic deployments causing a small time frame in which our website returns 404 errors.

On macOS got this:
$ mv --help
mv: illegal option -- -
usage: mv [-f | -i | -n] [-v] source target
       mv [-f | -i | -n] [-v] source ... directory

I think it's better to add configuration option, similar to use_relative_symlink.

Would be better to just do like capistrano, they create release directory in other folder with the same name 'current', then at the end just move the symlink to the old symlink location and it works.
For example, create 'release' dir called 'current' inside .dep dir, then at the end move one directory out with '..'

I see what your are trying to implement, i will do it myself, okay? 🙂

Done 14a8f8f9c4ebbc7da45c2b6b7c3c00a51b563ccf


Overlap Threshold

Description
interval overlap is now capable of examining the amount of overlap between elements and passing them only when the threshold is met
Testing
testing passes
Checklist before requesting a review

[ ] I performed a self-review of my code
[ ] If it this a core feature, I added thorough tests
[ ] The command go fmt or make clean was used on all files included

I pushed a commit that doesn't change any logic, but tries to simplify it a bit. If you don't approve or would prefer it another way feel free to revert/edit.

Also it seems like thresholdOverlap is incompatible with merged output. Would it be possible to go through the answer slice and return the elements that pass the threshold instead of just returning an dummy slice?

Also it seems like thresholdOverlap is incompatible with merged output. Would it be possible to go through the answer slice and return the elements that pass the threshold instead of just returning an dummy slice?

I tried to do exactly that and it gave me a bunch of problems that i just didn't have the time to fix. Not really sure why it was happening but the new func may help, I'll try to reimplement.

I want to make sure to prioritize this PR in the next code review because something about it isn't sitting right and I'm not totally sure where it's coming from. Everything works fine so long as thresholdOverlap isn't used with MergedOutput so for now we will keep that intact.

I want to make sure to prioritize this PR in the next code review because something about it isn't sitting right and I'm not totally sure where it's coming from. Everything works fine so long as thresholdOverlap isn't used with MergedOutput so for now we will keep that intact.


MTSRE-348 | Add: addon metadata addOnParameters validator

Signed-off-by: Yashvardhan Kukreja<EMAIL_ADDRESS>

please merge #38 before merging this PR

@yashvardhan-kukreja please rebase

please consider creating wiki page: https://github.com/mt-sre/addon-metadata-operator/wiki/AM0005

Done @sugarraysam

lgtm
awesome, one last thing, please make sure to test this against prod addons in managed-tenants. https://gitlab.cee.redhat.com/service/managed-tenants-sops/-/blob/main/MT-SRE/sops/AMO-validators.md before merging
And since this code will replace tasks/check/40_test_addon_parameters.py, when integrating let's make sure to archive this task so CI doesn't run it twice -> tasks/archived/50_test_addon_parameters.py.
Thank you !

awesome, one last thing, please make sure to test this against prod addons in managed-tenants.

@sugarraysam  done this! Works well :)

And since this code will replace tasks/check/40_test_addon_parameters.py, when integrating let's make sure to archive this task so CI doesn't run it twice -> tasks/archived/50_test_addon_parameters.py.

Sure, I'll raise an MR for the above but let's block that MR until managed-tenants repo doesn't start making use of the latest mtcli consisting of the above validator.

@yashvardhan-kukreja are we ready to merge this?

yep :)


Wrong ORACLE_HOME when not below products

Somehow the wrong ORACLE_HOME is set when the binaries are not installed below $ORACLE_BASE/product

It seems that this is related to OUD 11g. Function get_oracle_home does get a wrong home based on install.path
OUD 12c => /u00/app/oracle/product/fmw<IP_ADDRESS>.0/oud
OUD 11g => /u00/app/oracle/product/fmw<IP_ADDRESS>.0

fixed in release v1.5.7. Identify OUD 12 and OUD 11 based on the Oracle home path.


Java9+ Generated annotation cannot be found for Android projects

AGP 7+ requires Java 11 according to the article, so I tested this java version ad well.
jsonschema2pojo library uses build tool Java version to determine which Generated annotation to use. The library sets this annotation as javax.annotations.processing.Generated for java 9+ which can't be found in Android Java STL (Google blocks certain packages to be included).
Please note, that Java used for build tool can differ from Java you targeting at.
To include this annotation into build I have to use an external library. While building using java8 I have to add library like javax.annotation:javax.annotation-api. Which library should I add for new annotation?

There are two options here:

Turn off the annotation using includeGeneratedAnnotation = false
If you're using jcenter, try com.github.pengrad:jdk9-deps:1.0

I think we can improve the logic here and attempt to use the target source version, but the above should help.

I've updated the Android gradle example here and I may change the default for the Android task.

Please, use the target source version as an indicator, because the this version is quite useless otherwise (I haven't noticed any differences on this parameter)

How do you indicate a default for an Android or for Java? Could it be a value inside library?
Changing this default in a plugin could lead to incompatibility problems

usage of libraries like com.github.pengrad:jdk9-deps:1.0 is less obvious than an official library I mentioned. BTW, javax.annotation:javax.annotation-api is highly used in enterprise applications, so if you'd use just one variant of Generated annotation for all versions of java, it'll be much easier to manage and have a security audit on.


FIX: non partition cv in gat/decoding

I hadn't realized that some sklearn cv were not partitions, i.e. a given test sample may be predicted by different folds.
IIUC, this is not supported by sklearn cross_val_predict. e.g. cross_val_predict(clf, X, y, cv=ShuffleSplit()
The previous behavior of mne.decoding.gat was to store the last prediction of each test (i.e. it just overwrite each prediction. Here we raise an explicit error if the user wants a cross-validation with a non partition cv.
Once again, we see the limit of internally handling the CV...
@teonbrooks you're not going to be happy.
closes https://github.com/mne-tools/mne-python/issues/3624

Current coverage is 87.36% (diff: 100%)

Merging #3626 into master will increase coverage by <.01%

@@             master      #3626   diff @@
==========================================
  Files           343        343
  Lines         60843      60852     +9
  Methods           0          0
  Messages          0          0
  Branches       9317       9318     +1
==========================================
+ Hits          53155      53165    +10
+ Misses         4927       4925     -2
- Partials       2761       2762     +1

Powered by Codecov. Last update b0629be...6b96a35

yes next release...
LGTM

This change has broken user code just using just defaults for me ... I.e.,
    gat = GAT()
    labels = np.in1d(epochs.events[:, 2], class1)
    gat.fit(epochs, labels)

Shouldn't defaults be made to work? Or am I missing something here?

yes it should not crash...

Let me see if I can do a reproducible example. So far, I'm using a prespecified CV,

What dtype is y? it seems to be bools; does it crash if it's ints?
JR
On 9 October 2016 at 15:38, jona-sassenhagen<EMAIL_ADDRESS>wrote:

Let me see if I can do a reproducible example. So far, I'm using a
prespecified CV,
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/pull/3626#issuecomment-252507465,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEp7DCx_g8Azbm66VP3M6Fe6OcDZYPN6ks5qyUKygaJpZM4KJKXD
.

@kingjr this one looks good to go, is there more left to do?
@jona-sassenhagen any luck reproducing the problem in master?

@Eric89GXL it's ok on my side

cf b48a9c3


Add npmjs.com as an example website.

There are already other threads to list example websites, e.g https://github.com/zeit/next.js/issues/727, https://github.com/unicodeveloper/awesome-nextjs

The search page is https://www.npmjs.com/search?q=next&page=1&ranking=optimal

@timneutkens the way I understood it, @vernondegoede means that the whole npmjs.com website is built with next.js...?

Afaik only the autocomplete is built with Next.js. I know @bcoe worked on it. So maybe he can answer ❤ cc @bcoe

@sedubois @timneutkens Maybe you guys are right. Maybe only search is build with it.
I noticed next.js in the source code when an internal server error occured (looks just like the default errors in Next.js). When you visit https://www.npmjs.com/search?q=slack and look at the source code, you can see this part is built with Next :)
Anyway, should I close this and just add it to https://github.com/unicodeveloper/awesome-nextjs ?

cc @nkzawa

@timneutkens @sedubois we're currently using next.js for the auto-complete search page; thanks for all the hard work 👍

@bcoe thanks for your answer ❤

@vernondegoede could you update to reflect this. Did you know https://www.npmjs.com` is running on Next.js as well?`
[npmjs.com](https://www.npmjs.com/)'s package search is using Next.js as well.

@tim shouldn't this text rather be added together with other prod examples, as is intended for the 2.0 release?

Yeah I talked about this on slack. Lets do that.


Use str#find in NamedParameter's match statement

Hi @fundon,
Thank you for creating and maintaining this tiny but awesome library! 💙
I stumbled upon an issue and also found a suitable solution, provided with this PR.
(in [impl Node<T> find])
https://github.com/viz-rs/path-tree/blob/4090e8da241c9a8eefda87f650fda555447de89d/src/lib.rs#L235-L245
tl;dr: not every char is represented with a single byte.
For the match value in NodeKind::Parameter we need to determine the byte index, not the char index, otherwise we will be off for multibyte characters in the slice further down (since such slices are basically byte slices). In worst case scenarios this could lead to panics due to the index not being on a char boundary for the slice.
The find methodon the other hand respects that and returns exactly what we will need for the slicing.
Example
// .chars().position(...) vs .find(...)

fn main() {
    let p1 = "sx/";
    let p2 = "ßx/";
    let p3 = "xß/";

    check(p1);
    check(p2);
    check(p3);
}

fn check(p: &str) {
    println!(
      "p: {:?} | {:?} | byte len: {}, char count: {}",
      p, p.as_bytes(), p.len(), p.chars().count());

    let i = p.find('/').unwrap();
    println!("find / => {:?}", i);
    println!("p[i..] => {:?}", &p[i..]);

    let i = p.chars().position(|x| x == '/').unwrap();
    println!("c.p? / => {:?}", i);
    if p.is_char_boundary(i) {
        println!("p[i..] => {:?}", &p[i..]);
    } else {
        println!("p[i..] => would panic (not at char boundary)!");
    }
}

# output

p: "sx/" | [115, 120, 47] | byte len: 3, char count: 3
find / => 2
p[i..] => "/"
c.p? / => 2
p[i..] => "/"
p: "ßx/" | [195, 159, 120, 47] | byte len: 4, char count: 3
find / => 3
p[i..] => "/"
c.p? / => 2
p[i..] => "x/"
p: "xß/" | [120, 195, 159, 47] | byte len: 4, char count: 3
find / => 3
p[i..] => "/"
c.p? / => 2
p[i..] => would panic (not at char boundary)!

References

https://doc.rust-lang.org/std/primitive.str.html#method.find

Notes
There are more &p[l..] slices, but they depend on the result of the .len() method, which does return the length in bytes and not chars, so such places are safe.
And the other match statements, where the position function is used are also fine, as they need the char based indices for the node lookup and would actually break on byte indices for multibyte strings in the router/tree side; when I tried to fix the issue just in the position function some other tests would have failed with panics or mismatches.
Sorry for the long text for such a tiny fix, but it took me a while to narrow down where the problem was, and this PR description should act as a write down of my thinking process.

Thank you very much for your detailed instructions. 👍

Bumped in v0.1.12


added support multiple-level subdomain

Use full domain like "myproject.tunnel.example.com" as client's id instead of "myproject".
Combine sub_domain and current hostname as client's id when creating new client.
Check the clients object with full hostname when new request comes in.

Oops, looks like this patch break the websocket. Working on it.

This pull request added multi-level subdomain support but didn't pass the websocket tests.
I tried to fix, but I found the original master branch cannot neither. So I think it's better not to fix websocket support in this pull request.
@defunctzombie what do you think?

Closing in favor of #65


Mention and deprecate InputEventJoypadButton's pressure

Kinda closes https://github.com/godotengine/godot-docs/issues/8549
TL:DR: This property has never been used by the engine since its conception and no one ever noticed. See linked issue for more details.

Thanks!

Cherry-picked for 4.2.2.
Cherry-picked for 4.1.4.


tiny tweaks to wording

(are you looking for more examples? I can do that, but I'm worried it will disrupt the flow of the page if you just have like 20 examples piled on top of each other)

Thanks for the word changes! Nah, these are enough examples for now on the main page -- but I'd love to add links to further examples in the Further Reading section if you know of any?


Add explicit dependency on webpack-dev-middleware

webpack-dev-server has a transitive dependency on ^1.0.11 of this package, but 1.11.0 breaks our development server for reasons unknown.
1.10.2 is a version that works, and strictly specifying it in package.json will get both yarn and npm to respect it when resolving dependencies.
Fixes #172.

@joelanman can you please confirm if this fixes your issue? 🌈

I get no WARN any more from npm install, and npm run dev works - however it still opens http://localhost:8080/undefined instead of http://localhost:8080 which seems odd

@joelanman cool! I’m happy to merge this for now and come back to investigate that later, as it seems harmless.


fix json typo

fix missing comma in logstash config json example
remove multiple unused whitespaces
and also apply 2 space indentation

Thank you! This will be reflected to the website soon.


Add supported languages to the readme

There was already a issue talking about this (kind of), but it was closed: https://github.com/qlaffont/fortnite-api/issues/68
It would be nice if in the readme there was all supported languages on the endpoints, some are easy to find out, but some (like brazilian portuguese) I couldn't figure out what it is.

Worth nothing that the API (not Epic's, this API) restricts the language, so if you want to try finding other languages, you need to remove the language checks from the repo.

Portuguese (Brazilian) language code: pt-BR
How I found it: I enabled Fiddler and clicked on the "Store" button on Epic Games' Launcher, there was a  /store/pt-BR/api/content/promotions/epic-mega-sale URL, so I tried using it and... it worked.
I also decompressed the pak files and technically these are all the supported languages for the store:

But I tried some of those language codes... and not every single one of them worked, so try them out, if it works, yay!

I have changed how the API use Language. You can see the change on the readme in the new release (v3.4.0).
You can provide an options object to ignore the language check.


Allow CTEs under set operation nodes

This PR moves the SelectStatement::cte_map to QueryNode::cte_map, so that queries like the following work:
SELECT 1 UNION ALL (WITH cte AS (SELECT 42) SELECT * FROM cte);

Before, an error would be thrown because the binder was not able to find cte.
This was found due to #1088 , which should now also be fixed.
Happy to receive any feedback.

Thanks

Thanks


REPL: Can't use console.log() in IE 9

console.log.apply(console, ...), which is apparently called internally by the REPL, causes an error in IE 9.

@webbedspace Can I get a stack trace?

I'll take a look at this in a VM when I get home tonight.

@sebmck It's repl.js, line 89.

@webbedspace I know what line it's on but it's difficult to isolate the issue without an actual error message or stack trace.

Looked closer - here's a solution you could use:

Thanks @webbedspace for that solution


Use 2 outcomes for Thailand in Marriage abroad

Trello card

Add two outcomes to Thailand in Marriage abroad

Description
Reduces to two outcomes Thailand
Factcheck
Factcheck URL
Expected changes

URL on GOV.UK

Before

After

@leenagupte this is embarrassing 😊 . I forgot to regenerate the artefacts.  I have addressed your comments. Thank you for your review.

👍 LGTM


Malformed URL Links - https://github.com/Azure-Samples/Synapse/tree/master/Notebooks/PySpark/Synapse Link for Cosmos DB samples/IoT

01-CosmosDBSynapseStreamIngestion: Ingest streaming data into Azure Cosmos DB collection using Structured Streaming - 404
02-CosmosDBSynapseBatchIngestion: Ingest Batch data into Azure Cosmos DB collection using Azure Synapse Spark - 404
03-CosmosDBSynapseJoins: Perform Joins and aggregations across Azure Cosmos DB collections using Azure Synapse Link - 404
04-CosmosDBSynapseML: Perform Anomaly Detection using Azure Synapse Link and Azure Cognitive Services on Synapse Spark (MMLSpark) - 404
Correct URL Links on this page - https://github.com/Azure-Samples/Synapse/tree/master/Notebooks/PySpark/Synapse Link for Cosmos DB samples

This was fixed long time ago. Sorry for the delay. Let me know if you find any other error. Tks!

This was fixed long time ago. Sorry for the delay. Let me know if you find any other error. Tks!


Missing Option Informaton in SCEP Profile

Hi, there's been an update to the SCEP Profile creation pane.  This update has added the Certificate Type option that can be User or Device.  It would be good to explicitly define the correct setting for this as it relates to a user but the article also refers to devices.
Thanks.

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

ID: 9f709fd4-0b5d-b238-4060-222d550cd538
Version Independent ID: fb3f29da-2236-c47d-6bea-8efbcec8d7d3
Content: Using Certificates for AADJ On-premises Single-sign On single sign-on - Microsoft 365 Security
Content Source: windows/security/identity-protection/hello-for-business/hello-hybrid-aadj-sso-cert.md
Product: w10
Technology: windows
GitHub Login: @mapalko
Microsoft Alias: mapalko

Hello @NRL-LEastham ,
certificate type is User.
I created a PR to add it to the article
Thank you

(Ref. PR update the profile creation instruction #6494)

@NRL-LEastham - Thank you for submitting feedback.
From our understanding, the issue has been resolved based on this merged commit https://github.com/MicrosoftDocs/windows-itpro-docs/commit/3c781945b8e8bc024d4f9f6276cfa18fc8d8652e.
Thank you for your contribution to make the docs better! Much appreciated!


Make sure I am using variables right!

Heard there can be unforeseen bugs when you use global consts in some functions that may affect other functions.
Go through all uses of const, let, and var and make sure I’m using them right.
Look up when to use each one and make sure I’m not accidentally making global variables that affect other functions or anything else bad.
Also make sure .css variables are not sharing names since these affect one another.

Var is pretty much deprecated
const is something you dont want to re-assign, but can be mutable! So dont think just bc its const the value cant change
let is every other case


Drosenbach/arm review

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[x] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
If updating an existing version, please select the specific language SDKs and CLIs that must be refreshed after the swagger is published.

[x] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that label "WaitForARMFeedback" will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

/azp run

      "202": {

async PUT should use 200/201 + ProvisioningState as it was before.  https://github.com/Azure/azure-resource-manager-rpc/blob/master/v1.0/async-api-reference.md#creatingupdating-using-put

Refers to: specification/datacollaboration/resource-manager/Microsoft.DataCollaboration/preview/2020-05-04-preview/DataCollaboration.json:659 in e688b75. [](commit_id = e688b7549a243c9559242405cd627e68bb8c0f69, deletion_comment = False)

      "202": {

async PUT should use 200/201 + ProvisioningState as it was before. https://github.com/Azure/azure-resource-manager-rpc/blob/master/v1.0/async-api-reference.md#creatingupdating-using-put
Refers to: specification/datacollaboration/resource-manager/Microsoft.DataCollaboration/preview/2020-05-04-preview/DataCollaboration.json:659 in e688b75. [](commit_id = e688b75, deletion_comment = False)

This is a bug in our swagger. We will include it in the next iteration.

/azp run

/azp run

/azp run


minor cli command fix

Need the istio-system namespace to run this command if the user followed the default install instructions.

I signed it.

Jenkins job istio.github.io/presubmit passed

Jenkins job istio.github.io/presubmit passed

I missed a couple.  Closing and will resubmit.


ptouch-print: init at 1.4.3

Motivation for this change
https://github.com/NixOS/nixpkgs/issues/101712
Things done
Packaged ptouch-print release 1.4.3

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

It looks like this homepage and source has fallen off the internet.

See https://github.com/NixOS/nixpkgs/pull/197981


Implement SparsePauliOp.assign_parameters

What should we add?
#8620 added support for Parameter objects to be used inside of SparsePauliOps, porting this functionality from opflow to quantum_info and paving the way for a switch of the algorithms and application modules to be using purely quantum_info in the future.
However, once key functionality for this to be useful is still missing; and that is the implementation of SparsePauliOp.assign_parameters. Without it, there is no convenient way of actually using a parameterized SparsePauliOp instance.

Alright, thank you! :+1:
Feel free to close this issue if you feel like this resolves it. I am open to whether or not this functionality should be added.

When opflow was done with Parameter support we had consistency in the way the object supported Parameters with the same the methods as a parameterized circuit for the parameterized operator. My take is that it would nice to have the same support here with the Parameter support/objects Terra has. If you are allowing other types as coeffs then I agree the onus is on an end user. But for Parameter support my take would be to have the SparsePauliOp support it for consistency and ease of end user use.

I also think it would be good to have proper support of Qiskit's Parameter objects in the SparsePauliOp, as it is replacing how we used parameterized operators in Opflow. In several algorithms (e.g. QBMs in ML or time-evolution with time dependent Hamiltonians) we would need to assign parameters to the operators.
As Steve said already, we of course don't support binding any variable type but since the Parameter is Qiskit native I would expect support for that 😄 Also, the parameters should be bound in the same order as in the circuit 🙂

Issue resolved by #9796.


NullPointerException, possibly triggered by embedded subchart?

Stack:
Caused by: java.lang.NullPointerException
	at org.microbean.helm.chart.StreamOrientedChartLoader.addFile(StreamOrientedChartLoader.java:271)
	at org.microbean.helm.chart.StreamOrientedChartLoader.load(StreamOrientedChartLoader.java:234)
	at org.microbean.helm.chart.StreamOrientedChartLoader.load(StreamOrientedChartLoader.java:187)
    	at org.microbean.helm.chart.AbstractChartLoader.load(AbstractChartLoader.java:87)

fileName is null, despite the assert (which probably wasn't on) saying it should never be.

Root issue is that this path:
chart-with-subcharts/charts/minio-0.3.0.tgz

…is not matched by this regular expression:
^.+/charts/[^._][^/]+/(.+)$

There's also a problem with the basename method.  Given a string like minio-0.3.0.tgz, I would expect it to return minio-0.3.0.tgz but it returns the empty string.

Fixed with cf5d1dd


Improve error logging

It would be nice to know which particular file the error occurred on.
Currently, when errors are thrown, it's not clear which file it errored on as shown below:
find * -name "*.docx" -exec ~/Applications/docxtools {} replace-links https://ABC/ https://DEF/ \;
thread 'main' panicked at 'called Result::unwrap()on anErrvalue: InvalidArchive("Invalid zip header")', src/bin/main.rs:79:45 note: run withRUST_BACKTRACE=1environment variable to display a backtrace thread 'main' panicked at 'unknown entity reference &', /Users/david/.cargo/registry/src/index.crates.io-6f17d22bba15001f/xml_dom-0.2.6/src/shared/text.rs:84:29 note: run withRUST_BACKTRACE=1 environment variable to display a backtrace

I believe this error is now fixed. In general (if possible) error messages now include the original file that was being processed. Closing as fixed.


Sidebar anomaly in documentation and example page

1 - Sidebar covers the page for both documentation and example page when on top of the website for large screen.
2 - Sidebar should be visible only after you scroll down.

Can I work on this?


Added some data retrieval methods to fetch data easily.

What kind of change does this PR introduce?
feature
What is the current behavior?
To fetch single latest or oldest data currently need to apply order() on created_at field and then single() method to get one also to fetch null, true or false we need to pass in isFilter.
What is the new behavior?
latest() and oldest()
Before
final data = await supabase.from('users').select().order('created_at, ascending: true).single();
Now
final data = await supabase.from('users').select().latest();
or
final data = await supabase.from('users').select().latest('id');
same for oldest.
isNull() , isTrue() and isFalse()
Before
final data = await supabase.from('users').select().isFilter('data', null);
Now
final data = await supabase.from('users').select().isNull('data');
or
final data = await supabase.from('users').select().isTrue('data');
or
final data = await supabase.from('users').select().isFalse('data');

Thanks for the suggestion, but unfortunately we are not adding these new methods.


Mac: 'wait' option isn't respected

From what i understand from the docs, If i set wait: true, the notification will wait until a user action.
But what really happen is that the notification is closed after 5 seconds and a "timeout" event is been fired..
This makes it a problem if i want to set replay: true either because the 'timeout' event will be fired after 5 seconds and the 'reply' event won't fire.

From https://github.com/mikaelbr/node-notifier/issues/147#issuecomment-274277899:

I think maybe there's some misunderstanding around what wait does. wait doesn't make a sticky notification, it forces things like terminal-notifier til wait until response is returned. Without wait, it will just "fire and forget" but with wait it waits until timeout or user interaction.

If any one need some help please take a look at:
https://github.com/mikaelbr/node-notifier/issues/147

@mikaelbr Yes, wait is totally misunderstood. Now reading that I understand it. I think the documentation should be updated to reflect that, and also perhaps renaming the option to something like waitForNotifierProcess or similar to be more explicit. I know that it's longer and not as succinct as wait, but wait is truly confusing and does not given insight into what it is for. I think many of us immediately think let the notification stay on the screen until the user interacts with it.


Question about GRF

Hi, Awesome work! You extended GRF into the pymarl famework. However when I run it with vdn_gfootball.yaml, there is a lot of debugging information. Could you please help me to fix it?
Detail: absl Dump "episode_done": count limit reached / disabled

Hello, these debug logs are printed by Google Football Environment and may need to refer to their documentation.

Thank you for your immediate reply. At present, it does not seem to affect the operation of the code. However, I do not know if there is any potential impact on the final result, which needs more verification.

These debug logs have no impact on performance. However, we did not use the CNN feature in the original  GRF paper, only the simple115 feature, which has a prominent impact on performance.


make unit tests work across different backends that are supported

first: figure out what backends are supported

duplicate of #21


Embed Data Explorers for other datasets

Part of #1457.
"Browse data" tooltip has "Export to Terra icon". This is out of date. Rather than update the tooltip, I deleted it, because Save button is pretty obvious.
These Data Explorers are private. Identity-Aware Proxy is used to restrict who can see them to a Google group (that corresponds with a SAM group).
Future PRs will handle different auth cases. Here's my current thinking.

If user is not signed into Google at all (not just not registered with Terra), "Browse data" button opens login window/account switcher for DE
If user has registered with Terra and is not in dataset's auth domain, "Browse data" button opens up login window/account switcher for DE. If applicable, also show "Apply for access" button. To detect this case, I will hard-code each dataset's auth domain and fetch the user's SAM groups.

(Pending 👍 from @bradtaylor)

FYI @bradtaylor AMP PD workshop is tomorrow, so it would be nice if this went in today

Also FYI @bradtaylor if you want to test out the flow, click on "Browse Data" for 1000 Genomes. I did 1000 Genomes in an earlier PR.

Actually, there's a problem. The embedded AMP PD Data Explorer works at first, but stops working after an hour, due to CORS errors. I believe https://issuetracker.google.com/119581489 needs to be fixed. I will reopen after that bug is fixed.


feat: Prototype Most Active contributors and contributions evolution graphs

Description
Prototype of the Most Active Contributors Bar chart and Contributions Evolution bar chart
https://www.figma.com/file/OpVX6WT7dmWqnwRuEvADMF/Design-Lab?type=design&node-id=9839%3A2205&mode=design&t=QOUxCo3TPUQxqblX-1

https://github.com/open-sauced/app/assets/555044/dc011293-25a6-4514-b032-c4a02e488ed1
closes #1600 #1599
What type of PR is this? (check all applicable)

[x] 🍕 Feature

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings
https://github.com/open-sauced/app/assets/555044/1a1a8a0a-9b04-4d9a-9b2a-0ff7bbc79d5b
Added tests?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[ ] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Thanks chris this is awesome!!
Is there a way to add labels to the barchart to show the details of the data?
Something like this:

where we highlight the text for the section of the bar we are hovering on, so for example if I'm hovering over the section of the bar that represents "commits", then the "commits" line on the label is semi-bold.

@isabensusan

@foxyblocks - this is looking sublime! Very nice.
I'm working on the API endpoints for this. We recently created a /v1/lists/:listId/contributors GET route that returns data like:
{
  "data": [
    {
      "id": "uuid-v4",
      "user_id": 237133,
      "list_id": "uuid-v4",
      "created_at": "2016-10-19 13:24:51.000000",
      "login": "bdougie"
    },
    {
      "id": "uuid-v4",
      "user_id": 54321,
      "list_id": "uuid-v4",
      "created_at": "2016-10-19 13:24:51.000000",
      "login": "jpmcb"
    }
  ],
  "meta": {
    "page": 1,
    "limit": 10,
    "itemCount": 100,
    "pageCount": 10,
    "hasPreviousPage": false,
    "hasNextPage": true
  }
}

https://beta.api.opensauced.pizza/#/User Lists service/getUserListContributors
I'm wondering if it'll work to extend that API endpoint to also include the user's contributions?
{
  "data": [
    {
      "id": "uuid-v4",
      "user_id": 237133,
      "list_id": "uuid-v4",
      "created_at": "2016-10-19 13:24:51.000000",
      "login": "bdougie",
      "contributions": {
        "commits": 42,
        "prsCreated": 42,
        "prsReviewd": 42,
        "issuesCreated": 42,
        "comments": 42,
      }
    },
    {
      "id": "uuid-v4",
      "user_id": 54321,
      "list_id": "uuid-v4",
      "created_at": "2016-10-19 13:24:51.000000",
      "login": "jpmcb",
      "contributions": {
        "commits": 1,
        "prsCreated": 2,
        "prsReviewd": 3,
        "issuesCreated": 4,
        "comments": 5,
      }
    }
  ],
  "meta": {
    "page": 1,
    "limit": 10,
    "itemCount": 100,
    "pageCount": 10,
    "hasPreviousPage": false,
    "hasNextPage": true
  }
}

Although that might not work great with your proposed filters (contributor type, sort by, etc.) I suppose that your original proposal works well separate from "get all contributors in this list". Let me know your thoughts!!

@jpmcb I haven't been doing any work in the API aspects of this particular project so it'd probably be best to consult with people who have such as @brandonroberts .
I have however sketched out a rough suggestion specification that would satisfy the needs of the frontend. In it I scoped each endpoint under the /lists/{listId}/stats/[endpointName] path so as to not have to worry about overloading the existing endpoints but I don't have a strong opinion on this.

Right right, I agree on not overloading existing endpoints. I'll just stick to your original design 👍

@isabensusan updated with line chart

@foxyblocks is this ready to merge so it can be added to a page?

@foxyblocks, just following up. Is this ready for a review? Anything blocking you?

Noting here that these have gotten feedback and are ready to merge. We'll forward any necessary fixes


Trigger smoketest ziti webhook dark jenkins

@plorenz None of these changes seemed super important to me, but I thought I'd suggest some refinements with the goal of becoming more familiar and involved with ziti-ci.

Now I see that my branch was sorely out of date, and the main workflow file that I'd modified has been deleted in the destination branch, so I'm left wondering if we're no longer using Actions at all. I'll chat up @plorenz to learn more.


exclude openshift namespaces via regex

Adding --included-namespace-regex option to the helm chart.
as mentioned in the docs there is only --included-namespace-regex or --excluded-namespace.
so in order to exclude all openshift.* namespaces I used the --included-namespace-regex as mentioned in this issue, this is added as an example in values.yaml

@mochizuki875 can you please tell if you like it?

@zfrhv
Thank you for your suggestion and PR!
There is just one thing to note.
Now, to generate Helm Chart, we need to execute make charts.
It is for keeping up-to-date with changes in HCN manifests(manifests/*.yaml generated by make manifests), which was  barrier to create Helm chart previously.
Then, all files in charts/hnc/templates are re-generated by make charts like this, so you should not edit charts/hnc/templates/xxx.yaml directly.
Sorry, I should left comment somewhere...
So in this case, it's good to update hack/helm_patches/update-helm.sh to update template like that.
https://github.com/kubernetes-sigs/hierarchical-namespaces/blob/master/hack/helm_patches/update-helm.sh#L176-L177

@mochizuki875
thank you for mentioning.
I updated the files accordingly, after running make charts it did updated the templates as expected.
i didnt added the make charts result to the PR because it also modified many other files
let me know if you want me to include the make charts result in the PR

@zfrhv
IMO, I think it's better to add make charts result if it works as expected, because I think latest charts are expected to be in charts dir of project and some users may use the chart without make charts.
Ideally, make charts should be executed automatically and the helm chart should be updated to charts dir and github pages as commented here, but I’ve not started working yet...

in values.yaml the image repository seems to be broken
am I supposed to dig in and figure out why?

Have you set some environment values? (see here)
By setting these values as below,
$ export HNC_REGISTRY=gcr.io/k8s-staging-multitenancy
$ export HNC_IMG_NAME=hnc-manager
$ export HNC_IMG_TAG=v1.1.0

values.yaml should be generated like this as a result of make charts.
image:
  repository: gcr.io/k8s-staging-multitenancy/hnc-manager
  tag: v1.1.0
  imagePullPolicy: {}
  ...

sorry for making you feed me with a spoon

/ok-to-test

@zfrhv
Thank you for your work!
This PR seems to make sense.
/lgtm
/cc @rjbez17
Related this PR, I have two questions.
If you have some information, please let me know.

Some ClusterRoles(admin, edit, view) templates are replaced from admin-role in this PR.
However, there doesn’t seems to be Binding resource to bind them.
Is this intentional?
It comes from #318.

(NIT) At first, I thought the range operation should be used for --included-namespace-regex like --excluded-namespace.
However, includedNamespacesRegex which comes from --included-namespace-regex is defined as string not arrayArg, so we can't.
How do you think?

@mochizuki875

those roles are the default permissions that users will get.
for example if someone has view on his namespace, then he will also be able to view the hnc resources on his namespace.
if someone has edit then he can edit and also admin.
the clusterRoles are not supposed to be bound to anything, they have the label rbac.authorization.k8s.io/aggregate-to-edit: "true" which kubernetes uses to merge the cluster role into the default clusterRole edit.
its just better permissions assignment instead of just giving admin to admin
im not good at go but when U checked how to use the includeNamespaceRegex I saw the pr that added it, and it mentions that its a string and not arrayArg here lines 69-70

@zfrhv Thanks your information;)


Restrict guest user role

This changes the guest user role so that it can only view the guest user
contract. This change goes hand-in-hand with updates to the worker
service that add special routes for login and password reset.
See https://github.com/product-os/jellyfish/pull/7774
Change-type: major
Signed-off-by: Lucian Buzzo<EMAIL_ADDRESS>

@balena-ci I self-certify!


No scrolling

From @JaneRuby

I think it shouldn't scroll in web view, it should all fit in the screen without a scroll

To fix that, I think the text at the bottom should be reorganized and not stacked as it is

Built by DevProgress | Design by Jane Ruby using Truck by John Caserta and Taco by Focus Lab, from the Noun Project | Want to help? It's open source!
^possible solution using pipes and keeping the text at the bottom on one line

There's just not very much room on mobile; the title and description takes up almost half of the height.  What do you think about putting the description in a dismissible popover?  It could close on click and/or automatically after some amount of time (10s?)

Yes I agree!

Can you also adjust the text at the bottom? I think there is an open issue for that 😊

Makes sense to me. Again, you guys are the experts on this, so I'll follow your lead.

see https://github.com/DevProgress/more-tacos/pull/105 for version with text adjustments

Looks good to me!

fixed by https://github.com/DevProgress/more-tacos/pull/105


Updates welcome modal text, per #153

Oops, accidentally branched from doi-zenodo, so that should be merged first!
Fixes #153

Closing to open a cleaner PR


Profile Chooser at startup

Feature request
I'm submitting a proposal to ...

[x] new feature
[ ] change an existing feature

Describe your request
It would be nice to have a setting to enable choosing my netflix profile whenever I start watching something.
So Multiuser setup would be more easy
Describe the use case
I currently have a setup where I use kodi profiles to switch to user specific stuff, but this so slow and when you just want to watch a show you need to wait a lot longer before beginning to watch. When I could have just one kodi profile where my shows are laid out in on one screen and someone else's shows on another screen, it wouldn't matter who is there as the profile can be chosen after clicking on the show. Like I'm watching Breaking Bad S1E3 and someone else is watching Breaking Bad S2E1 the records wouldn't be changed on the wrong profile.
Describe an alternative

When I could have just one kodi profile where my shows are laid out in on one screen and someone else's shows on another screen

this is not a proper way to use Kodi for multi-user purpose...
is not clear to me what means exacly with "on one screen"
but if you are talking of widgets, this addon dont support widgets ufficially and no support is provided for widgets,
as explained here: https://github.com/CastagnaIT/plugin.video.netflix/wiki/FAQ-(Audio%2C-Video%2C-Subtitle%2C-Other)#skin-widgets---problems-and-limits
if you are talking of regular browsing (by opening addon icon to be clear),
then you have to use the profile selection menu, is not possible browse lists with a specific netflix profile and play videos by using another netflix profile, this violates the way how the website works and also the addon,
making this type of mixture only leads to problems with browsing lists and playback
a kind of alternative solution is use Kodi library,
that is the only place where the addon allow to select a netflix profile every time you play a video.
To do this you have to export all video lists to Kodi library and/or enable library auto-sync (you can find some help on Wiki).
But due to its particular use this way leads to some disadvantages:

its not possible sync videos watched status in a "two way" mode
due to the forced selection of different profiles where play a video, playback problems/errors may be encountered

Okay thank you for the clarification. Unfortunately my specific use case won't be possible like this.
If my feature request to create a setting which can be enabled to always ask on which profile to watch is also not possible then you can close this issue.


Add automated testing

While the library does work as advertised and also there is a demo html file, it would be ideal to have automated tests so that they could be run to verify that the library is functioning properly. This would be especially helpful for when other people contribute code.

Fixed in 3b6616a66fcb20745406f4a67cacccab53b7fd30


#355 example refactoring of zebra

for multiple Devices (for each type of scan)
but hopefully without too much duplication

Fixes #355
To test:

Think about whether this is a good way to organise things
Check tests pass

On discussion with @DominicOram, decided to keep one zebra device and extract the varying logic into plans

Closed in favour of #365


deduplication of DefineOwnProperty() wrapper

This adds a new file called nan_define_own_property_helper.h that contains a deduplicated code block taken from very similar code between nan_maybe_43_inl.h and nan_maybe_pre_43_inl.h
To anyone looking and scratching their head as to how the code block in  nan_maybe_43_inl.h ever gets reached at all, this apparently happens in io.js 3 (see #725)
I wasn't sure what to name the common function, so I went with DefineOwnPropertyHelper(), but I'm open to other suggestions...
As discussed: https://github.com/nodejs/nan/pull/728#discussion_r167025257

Will try to take a look during the weekend.
On February 23, 2018 11:40:41 PM GMT+02:00, Michael Ira Krufky<EMAIL_ADDRESS>wrote:

@mkrufky requested your review on: nodejs/nan#741 deduplication of
DefineOwnProperty() wrapper.

Sorry, I must have forgotten about this PR when I went through the others on the weekend. LGTM.


Closing this PR with minimal review.


[New Snap] Web3 Security

Checklist
All items in the list below needs to be satisfied.

[x] Is the Snap repository publicly accessible and linked in this ticket? https://github.com/AnChainAI/web3-security-snap
[x] Is the Snap distributed on npm and linked in this ticket? https://www.npmjs.com/package/web3-security-snap
[ ] Has an audit been performed and the audit report attached or linked in this issue?
[ ] Is a complete list of discovered vulnerabilities from the audit documented in this issue?
[ ] For vulnerabilities that have been deemed necessary to be addressed, are the links to the fixes attached to this issue?
[ ] For vulnerabilities that have been deemed not necessary to be addressed, is a reason for each of them documented in this issue?
[x] The corresponding pull request in this repo has been merged. https://github.com/MetaMask/snaps-registry/pull/77

Audit report: Sayfer - 2023-08 Penetration Testing Report for AnChain Snap .pdf

Discovered vulnerabilities:

Snap Hosted Locally
Unexpected Success Message
Lack of Documentation
Wrong Fuction Name
Unused Parameter
Missing Unit Tests

All of the vulnerabilities are fixed as of this commit: https://github.com/AnChainAI/web3-security-snap/commit/d69cfe0cdb3f00778cba3efc996e1230d5046fef


Create rpm or binary release for simple install

Hi i love this tool but the need to compile it make it impossible to be use in server environement,
can we inplemant release to binary file in github action by exemple

This is done! Thanks for the suggestion.
https://github.com/orf/hq/releases/tag/v1.0.1


How to ise Tailwindcss classes

I ve tried to apply tailwindcss classes using tailwind's customised stylesheet. The classes get picked up in the markup  but are not applied. They are being overridden by other styles. Perhaps datatables stylesheet. I ve tried moving the tailwind inclusion in the base template to after the datatable css but it does not work.  Can you help me please?

I'm sorry ... .I don't know anything about tailwindcss


Add linters and react template

Step-by-step summary:

Create a new repo
Create a new branch
Set up linters
Set up react project
Fix linter errors
Create a new PR

Hi @matee-safi,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
Kindly add at least one Future Feature for this project.

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Creation of K8s "dummy" Node for BIG-IP Gateway keeps getting removed

What kind of request is this (question/bug/enhancement/feature request):
question
Steps to reproduce (least amount of steps as possible):

Bootstrap a k8s cluster that is managed by Rancher, with the following configurations:

custom cluster (3 nodes, each have all roles)
amazon provider
flannel (or canal) as network provider (although I think problem likely persists with Calico, too)

Create a yaml config for "dummy" node that will act as a gateway to BIG-IP device. Instructions found here.
Create the node using the yaml template.

Result:
Node goes from Status: Unknown, to Status: NotReady, and then is automatically removed.
Is there a way to ensure that the node is not automatically removed after going to the NotReady state?
Other details that may be helpful:
I am following the install docs for integrating f5-bigip-ctlr with kubernetes cluster (f5-bigip-ctlr install docs).
My yaml config for the node template is :
apiVersion: v1
kind: Node
metadata:
  name: bigip
  annotations:
    # Provide the MAC address of the BIG-IP VXLAN tunnel
    flannel.alpha.coreos.com/backend-data: '{"VtepMAC":"<BIG-IP_mac-address>"}'
    flannel.alpha.coreos.com/backend-type: "vxlan"
    flannel.alpha.coreos.com/kube-subnet-manager: "true"
    # Provide the IP address you assigned as the BIG-IP VTEP
    flannel.alpha.coreos.com/public-ip: <IP address assigned as the BIG-IP VTEP>
spec:
  # Define the flannel subnet you want to assign to the BIG-IP device.
  # Be sure this subnet does not collide with any other Nodes' subnets.
  podCIDR: <IP_ADDRESS>/24

If I try and create the "dummy" node in the local rancher cluster, the node succesfully remains in the cluster with Status: NotReady. This is what I want to happen in a k8s cluster that is managed by Rancher (i.e. not the local cluster). I am not sure why this behavior is seen for the local cluster, not seen in a k8s bootstrapped cluster.
Perhaps there is some taint/toleration that I can apply to ensure the node doesn't get deleted?
Environment information

Rancher version: v2.1.7
Installation option (single install/HA): HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Custom
Machine type (cloud/VM/metal) and specifications (CPU/memory): AWS EC2, t2.large
Kubernetes version (use kubectl version): v1.11.8
Docker version (use docker version): 1.12.6

this is specific to kubernetes and due to the fact that I select cloud-provider as AWS.
By selecting a cloud provider, the kube-controller-manager (node-controller, specifically) checks to see if the node is still available by contacting the cloud provider (aws). Since its a dummy node, it gets deleted automatically.
I will need to figure out a solution but this is not related to Rancher so I will close.


[BUG] UniGetUI start and closed

Please confirm these before moving forward

[X] I have searched for my issue and have not found a work-in-progress/duplicate/resolved issue.
[X] I have tested that this issue has not been fixed in the latest (beta or stable) release.
[X] I have checked the FAQ section for solutions.
[X] This issue is about a bug (if it is not, please use the correct template).

UniGetUI Version
3.1.1
Windows version, edition, and architecture
Windows 11 Enterprise 23H2 22631.4037
Describe your issue
20 sec after start closed UniGetUI automaticaly.
dotnet-sdk-8.0.402-win-x64.exe -> installed
dotnet-sdk-8.0.402-win-x86.exe -> installed
dotnet-sdk-6.0.425-win-x86.exe -> installed
Steps to reproduce the issue
No response
UniGetUI Log
[04.10.2024 11:22:59]    __  __      _ ______     __  __  ______
                        / / / /___  (_) ____/__  / /_/ / / /  _/
                       / / / / __ \/ / / __/ _ \/ __/ / / // /
                      / /_/ / / / / / /_/ /  __/ /_/ /_/ // /
                      \____/_/ /_/_/\____/\___/\__/\____/___/
                          Welcome to UniGetUI Version 3.1.1
[04.10.2024 11:22:59]
[04.10.2024 11:22:59] Version Code:  3,11
[04.10.2024 11:22:59] Encoding Code Page set to 437
[04.10.2024 11:22:59] Using bundled GSudo at C:\Program Files\UniGetUI\Assets\Utilities\gsudo.exe
[04.10.2024 11:22:59] Loaded language locale: en
[04.10.2024 11:22:59] Randomly-generated background API auth token for the current session: 3ac5ctsrt6kl8h9losku7b8jkzu0yhtv722p1csqmttvu0w9rbsqlo5l4wu9cm2l
[04.10.2024 11:23:00] Api running on http://localhost:7058
[04.10.2024 11:23:00] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: Chocolatey
                      █ Enabled: True
                      █ Found: False
                      █ THE MANAGER WAS NOT FOUND. PERHAPS IT IS NOT
                      █ INSTALLED OR IT HAS BEEN MISCONFIGURED
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[04.10.2024 11:23:00] Lang files were updated successfully from GitHub
[04.10.2024 11:23:00] UniGetUI is up to date
[04.10.2024 11:23:00] The restored geometry did not have exactly 5 items (found length was 1)
[04.10.2024 11:23:00] Downloaded new icons and screenshots successfully!
[04.10.2024 11:23:00] Command npm was not found on the system
[04.10.2024 11:23:00] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: Npm
                      █ Enabled: True
                      █ Found: False
                      █ THE MANAGER WAS NOT FOUND. PERHAPS IT IS NOT
                      █ INSTALLED OR IT HAS BEEN MISCONFIGURED
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[04.10.2024 11:23:01] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: Pip
                      █ Enabled: True
                      █ Found: True
                      █ Fancy exe name: pip
                      █ Executable path: C:\Users\o9001159\AppData\Local\Microsoft\WindowsApps\python.exe
                      █ Call arguments:  -m pip
                      █ Version:
                      █
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[04.10.2024 11:23:01] Command scoop was not found on the system
[04.10.2024 11:23:01] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: Scoop
                      █ Enabled: True
                      █ Found: False
                      █ THE MANAGER WAS NOT FOUND. PERHAPS IT IS NOT
                      █ INSTALLED OR IT HAS BEEN MISCONFIGURED
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[04.10.2024 11:23:02] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: .NET Tool
                      █ Enabled: True
                      █ Found: True
                      █ Fancy exe name: dotnet tool
                      █ Executable path: C:\Program Files\dotnet\dotnet.exe
                      █ Call arguments: tool
                      █ Version:
                      █   8.0.402
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[04.10.2024 11:23:02] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: Winget
                      █ Enabled: True
                      █ Found: True
                      █ Fancy exe name: winget.exe
                      █ Executable path: C:\Users\o9001159\AppData\Local\Microsoft\WindowsApps\winget.exe
                      █ Call arguments:
                      █ Version:
                      █   System WinGet CLI Version: v1.8.1911
                      █   Microsoft.WinGet.Client PSModule version: ""
                      █   Using Native WinGet helper (COM Api)
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[04.10.2024 11:23:04] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: PowerShell
                      █ Enabled: True
                      █ Found: True
                      █ Fancy exe name: powershell.exe
                      █ Executable path: C:\WINDOWS\system32\windowspowershell\v1.0\powershell.exe
                      █ Call arguments:  -NoProfile -Command
                      █ Version:
                      █   Name                           Value
                      █   ----                           -----
                      █   PSVersion                      5.1.22621.3958
                      █   PSEdition                      Desktop
                      █   PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
                      █   BuildVersion                   10.0.22621.3958
                      █   CLRVersion                     4.0.30319.42000
                      █   WSManStackVersion              3.0
                      █   PSRemotingProtocolVersion      2.3
                      █   SerializationVersion           <IP_ADDRESS>
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[04.10.2024 11:23:05] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: PowerShell7
                      █ Enabled: True
                      █ Found: True
                      █ Fancy exe name: pwsh.exe
                      █ Executable path: C:\Program Files\PowerShell\7\pwsh.exe
                      █ Call arguments:  -NoProfile -Command
                      █ Version:
                      █   PowerShell 7.4.5
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[04.10.2024 11:23:06] Microsoft.Powershell.Client detection has been forcefully skipped as the module is not required on the current context
[04.10.2024 11:23:06] Dependency WinGet PowerShell Module for manager Winget is present
[04.10.2024 11:23:06] Dependency .NET Tools Outdated for manager .NET Tool is present
[04.10.2024 11:23:06] LoadComponentsAsync finished executing. All managers loaded. Proceeding to interface.
[04.10.2024 11:23:06] Found 0 available updates from Pip
[04.10.2024 11:23:07] Found 0 installed packages from Pip

Package Managers Logs
Manager WinGet with version:
System WinGet CLI Version: v1.8.1911
Microsoft.WinGet.Client PSModule version: ""
Using Native WinGet helper (COM Api)

——————————————————————————————————————————

Logged native task on manager Winget. Task type is ListSources
Process start time: 04.10.2024 11:29:50
Process end time:   04.10.2024 11:29:50

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Logged subprocess-based task on manager Winget. Task type is RefreshIndexes
Subprocess executable: "C:\Users\xxxxx\AppData\Local\Microsoft\WindowsApps\winget.exe"
Command-line arguments: " source update --disable-interactivity"
Process start time: 04.10.2024 11:29:53
Process end time:   04.10.2024 11:29:56

-- Process STDOUT
 ...

Return code: SUCCESS (0)

——————————————————————————————————————————

Logged native task on manager Winget. Task type is ListInstalledPackages
Process start time: 04.10.2024 11:29:53
Process end time:   UNFINISHED

-- Task information
 ...

The task has not finished yet

——————————————————————————————————————————

Logged native task on manager Winget. Task type is ListSources
Process start time: 04.10.2024 11:29:54
Process end time:   04.10.2024 11:29:55

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Relevant information
No response
Screenshots and videos
No response

Are you using crowdstrike falcon?

Yes.

From: Martí Climent @.>
Sent: Friday, October 4, 2024 12:37
To: marticliment/UniGetUI @.>
Cc: feofan69 @.>; Author @.>
Subject: Re: [marticliment/UniGetUI] [BUG] UniGetUI start and closed (Issue #2815)
Are you using crowdstrike falcon?
—
Reply to this email directly, view it on GitHubhttps://github.com/marticliment/UniGetUI/issues/2815#issuecomment-2393396671, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A5BN3JNTKKZM3TFY7NGWZ7LZZZVYJAVCNFSM6AAAAABPLP5EWSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJTGM4TMNRXGE.
You are receiving this because you authored the thread.Message ID: @.***>

Duplicate of #2803


Can't seem to get upgrade to p2p to work

I've got the example chat up and running by following the instructions, but when I initiate the server and try to press "Go Private", I see "WebRTC connection established!" in the window but not in the other window that I've opened... however if I press "Go Private" on both of them, no messages get sent across at all. Has anyone else ran into this? Is there an actual hosted working example of the chat app somewhere?

Is this still an issue? I thought I may have seen on Twitter that you had solved it?

No longer an issue I solved it.
On Sep 14, 2015 3:15 AM, "tomcartwrightuk"<EMAIL_ADDRESS>wrote:

Is this still an issue? I thought I may have seen on Twitter that you had
solved it?
—
Reply to this email directly or view it on GitHub
https://github.com/socketio/socket.io-p2p/issues/10#issuecomment-140028415
.


How package the app to exe on MAC OS?

Insomnia Version: ___
Operating System: ___

Details

The only way to package it for windows is on a windows machine due to the native dependency on libcurl. Insomnia uses the Appveyor environment for its builds if you're curious
Sent with GitHawk


Storage: Harden systest 'tearDownModule' against 429 TooManyRequests.

Closes #5664.

Merging without review, since this is a testing-only change and the tests pass.

@tseaver to make sure I understand. The change is to allow retries on exceptions.TooManyRequests?

@frankyn Yes, during the system test module teardown.

Thanks!


AppState change event called for Android but not iOS on initial load

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[x] Use the latest React Native release: https://github.com/facebook/react-native/releases

Environment

React Native Environment Info:
System:
OS: macOS High Sierra 10.13.5
CPU: x64 Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz
Memory: 316.95 MB / 16.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 8.12.0 - /usr/local/bin/node
Yarn: 1.9.4 - /usr/local/bin/yarn
Watchman: 4.9.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 11.4, macOS 10.13, tvOS 11.4, watchOS 4.3
IDEs:
Android Studio: 3.1 AI-173.4819257
Xcode: 9.4.1/9F2000 - /usr/bin/xcodebuild
npmPackages:
react: 16.5.0 => 16.5.0
react-native: 0.57.1 => 0.57.1

Description
AppState.addEventListener('change', newState => {
  Alert.alert('new state', newState);
});

When the app is first loaded (eg. close from background and reopen), the above event is called on Android, but not on iOS. Should this inconsistency be fixed?
Reproducible Demo
https://snack.expo.io/@riwu/appstate

-stale

Hello there 👋 this issue seems to have been inactive for the past few weeks. Because of this, it's likely that the issue is not a high priority anymore or it has been solved by OP; for these reasons, we'll close it. But please, if it's actually still an issue with 0.59 please comment below and we can reopen it or please send us a Pull Request with a fix 😊

Duplicate of: https://github.com/facebook/react-native/issues/18836


⚠ Bofrost.at has degraded performance

In c3f5296, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 877 ms

Resolved: Bofrost.at performance has improved in 7d60d15.


Fix .env regex constants

Set .env regex list as constants.
This pull request fixes the nit related in this comment: https://github.com/flutter/flutter/pull/128668#discussion_r1253377454
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

manually set the google testing to green, this change shouldn't affect google targets


feat(nunit): support of nunit specific extensions

Coverage increased (+23.1%) to 97.347% when pulling 1f80ce5d38c03d374f13b62ba744b179c0aeb684 on nunit_support into 888226469e4887dad702d22a0dbdf305bc7fb347 on main.


Make homekit_controller a local push integration

Proposed change

Now that we are using aiohomekit for homekit_controller this PR enables events. This means that accessories can now push their state changes to Home Assistant over an open HomeKit secure session as they happen. In my testing with a Phillips Hue this is near instantaneous.
We added the framework for injecting state changes from events many cycles ago (HKDevice.process_new_events) so this ends up being quite a modest change now we aren't restrained by a blocking client.
Polling still occurs (for now) as not all characteristics support events and it seemed like a good idea to make sure we didn't "miss" any events. There are also edge cases where e.g. a device is unavailable and then becomes available - these might not trigger events. Hybrid push/poll  minimizes the impact of these sorts of issues.
~NOTE: This is currently behind #32212 in my patch queue. The intention is that it can be reviewed seperately to 5d31f95. I will rebase it and remove the WIP tags after that is merged.~
Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New integration (thank you!)
[x] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[x] The code change is tested and works locally.
[x] Local tests pass. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist
[x] The code has been formatted using Black (black --fast homeassistant tests)
[x] Tests have been added to verify that the new code works.

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly.
Updated and included derived files by running: python3 -m script.hassfest.
[x] New or updated dependencies have been added to requirements_all.txt.
Updated by running python3 -m script.gen_requirements_all.

The integration reached or maintains the following Integration Quality Scale:

[x] No score or internal
[ ] 🥈 Silver
[ ] 🥇 Gold
[ ] 🏆 Platinum

Doc PR: https://github.com/home-assistant/home-assistant.io/pull/12187

Cheers for merging #32212 @balloob!
Have rebased and removed the WIP tags from this PR.


Series not downloaded correctly

Hi,
i have been trying to download the latest episodes of a laracasts series but for some reason it doesn't work.
> Download started: 70 - Thread Views Design 3 - KISS . . . . Saving on series/lets-build-a-forum-with-laravel folder.
> Can't download this lesson! :( No download button
> Tring to find a Wistia.net video
> Elapsed time: 147ms, Memory: 4.00Mb
> Current: 6 of 6 total. Left: 0

A 70K file is downloaded instead.
-rw-r--r-- 1 root root  70K Aug 22 21:57 70-Thread Views Design 3 - KISS.mp4

I am using a valid laracasts premium account. Any idea why that's the case? Any help is appreciated.

Just tried now and is working.
 Works now. Thanks a bunch for this epic tool 👍


Spring Cloud Context breaks Camunda property management.

Application properties for Camunda section are not working properly when spring-cloud-starter is in the classpath.
There should be some issues with the context hierarchy.
As an example if I take example-cloud, and add configuration for admin user:
camunda:
  bpm:
    admin-user:
      id: admin
      password: admin

I got this error:
org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'simpleApplication': Unsatisfied dependency expressed through field 'historyService'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'processEngineFactoryBean' defined in class path resource [org/camunda/bpm/spring/boot/starter/CamundaBpmAutoConfiguration.class]: Unsatisfied dependency expressed through method 'processEngineFactoryBean' parameter 0; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'processEngineConfigurationImpl' defined in class path resource [org/camunda/bpm/spring/boot/starter/CamundaBpmConfiguration.class]: Unsatisfied dependency expressed through method 'processEngineConfigurationImpl' parameter 0; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'createAdminUserConfiguration': Invocation of init method failed; nested exception is java.lang.NullPointerException: missing field: camunda.bpm.admin-user.id
	at org.springframework.beans.factory.annotation.AutowiredAnnotationBeanPostProcessor$AutowiredFieldElement.inject(AutowiredAnnotationBeanPostProcessor.java:592) ~[spring-beans-4.3.4.RELEASE.jar:4.3.4.RELEASE]
	at org.springframework.beans.factory.annotation.InjectionMetadata.inject(InjectionMetadata.java:88) ~[spring-beans-4.3.4.RELEASE.jar:4.3.4.RELEASE]
	at org.springframework.beans.factory.annotation.AutowiredAnnotationBeanPostProcessor.postProcessPropertyValues(AutowiredAnnotationBeanPostProcessor.java:370) ~[spring-beans-4.3.4.RELEASE.jar:4.3.4.RELEASE]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.populateBean(AbstractAutowireCapableBeanFactory.java:1219) ~[spring-beans-4.3.4.RELEASE.jar:4.3.4.RELEASE]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:551) ~[spring-beans-4.3.4.RELEASE.jar:4.3.4.RELEASE]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:482) ~[spring-beans-4.3.4.RELEASE.jar:4.3.4.RELEASE]
	at org.springframework.beans.factory.support.AbstractBeanFactory$1.getObject(AbstractBeanFactory.java:306) ~[spring-beans-4.3.4.RELEASE.jar:4.3.4.RELEASE]
	at org.springframework.beans.factory.support.DefaultSingletonBeanRegistry.getSingleton(DefaultSingletonBeanRegistry.java:230) ~[spring-beans-4.3.4.RELEASE.jar:4.3.4.RELEASE]
	at org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:302) ~[spring-beans-4.3.4.RELEASE.jar:4.3.4.RELEASE]
	at org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:197) ~[spring-beans-4.3.4.RELEASE.jar:4.3.4.RELEASE]
	at org.springframework.beans.factory.support.DefaultListableBeanFactory.preInstantiateSingletons(DefaultListableBeanFactory.java:754) ~[spring-beans-4.3.4.RELEASE.jar:4.3.4.RELEASE]
	at org.springframework.context.support.AbstractApplicationContext.finishBeanFactoryInitialization(AbstractApplicationContext.java:866) ~[spring-context-4.3.4.RELEASE.jar:4.3.4.RELEASE]
	at org.springframework.context.support.AbstractApplicationContext.refresh(AbstractApplicationContext.java:542) ~[spring-context-4.3.4.RELEASE.jar:4.3.4.RELEASE]
	at org.springframework.boot.context.embedded.EmbeddedWebApplicationContext.refresh(EmbeddedWebApplicationContext.java:122) ~[spring-boot-1.4.2.RELEASE.jar:1.4.2.RELEASE]
	at org.springframework.boot.SpringApplication.refresh(SpringApplication.java:761) [spring-boot-1.4.2.RELEASE.jar:1.4.2.RELEASE]
	at org.springframework.boot.SpringApplication.refreshContext(SpringApplication.java:371) [spring-boot-1.4.2.RELEASE.jar:1.4.2.RELEASE]
	at org.springframework.boot.SpringApplication.run(SpringApplication.java:315) [spring-boot-1.4.2.RELEASE.jar:1.4.2.RELEASE]
	at org.springframework.boot.SpringApplication.run(SpringApplication.java:1186) [spring-boot-1.4.2.RELEASE.jar:1.4.2.RELEASE]
	at org.springframework.boot.SpringApplication.run(SpringApplication.java:1175) [spring-boot-1.4.2.RELEASE.jar:1.4.2.RELEASE]
	at org.camunda.bpm.spring.boot.example.cloud.SimpleApplication.main(SimpleApplication.java:29) [classes/:na]
Caused by: org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'processEngineFactoryBean' defined in class path resource [org/camunda/bpm/spring/boot/starter/CamundaBpmAutoConfiguration.class]: Unsatisfied dependency expressed through method 'processEngineFactoryBean' parameter 0; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'processEngineConfigurationImpl' defined in class path resource [org/camunda/bpm/spring/boot/starter/CamundaBpmConfiguration.class]: Unsatisfied dependency expressed through method 'processEngineConfigurationImpl' parameter 0; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'createAdminUserConfiguration': Invocation of init method failed; nested exception is java.lang.NullPointerException: missing field: camunda.bpm.admin-user.id
	at org.springframework.beans.factory.support.ConstructorResolver.createArgumentArray(ConstructorResolver.java:749) ~[spring-beans-4.3.4.RELEASE.jar:4.3.4.RELEASE]

The configuration values seems to be ignored.
If you run the main SimpleApplication, the workflow is deployed, even if there is camunda.bpm.auto-deployment-enabled: false.
Without spring-cloud-startereverything works well.
I'll try to investigate deeper. I'll keep you posted.

If I remove this line, it seems better.
It was introduced here.
I don't see any reason to add it, do I miss something?

Similar problem here. Camunda seems to ignore configuration properties. They are all null

What is the work around?

I just lost a day on this issue, before discovering this thread...
I tried upgrading to 2.1.0-SNAPSHOT, but my "camunda.bpm.database.schema-update" property are still ignored in SpringProcessEngineConfiguration.
As a workaround: Setting the property as a -D JVM flag or using bootstrap.properties, seems to work.
Is there another workaround/fix for this? I'd like to be able to trust my property files again ;-)

@lldata Can you provide some samples to the approaches you provided?

@gordonwho simply create a file called bootstrap.yaml and copy the camunda.bpm settings from application.yaml to that file.
Or when you start your program you can provide the flag that give you trouble as JVM system properties e.g. -Dcamunda.bpm.database.schema-update=create-drop
https://docs.spring.io/spring-boot/docs/current/reference/htmlsingle/#boot-features-external-config

tried to reproduce, suppose it's fixed with #206


Formatting error in Microsoft Edge browser

On a fresh installation of Windows 10 (latest) the "Kudu" logotype is formatted on top / behind other text on the pages. It doesn't look very nice :-)

Minor issue but nevertheless worth looking at.

This was just fixed yesterday and will be in the next build. See https://github.com/projectkudu/kudu/pull/1808

The change is now live in Azure.


Documentation update for mo.cpp

Description of Change

Checklist

[ ] Added description of change
[ ] Added file name matches File name guidelines
[ ] Added tests and example, test must pass
[x] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[x] Relevant documentation/comments is changed or added
[ ] PR title follows semantic commit guidelines
[ ] Search previous suggestions before making a new one, as yours may be a duplicate.
[ ] I acknowledge that all my contributions will be made under the project's license.

Notes:
mo.cpp file in range_queries folder didn't has proper documentation. And it was having c style of coding.

Seems like my forked is not upto date.
Closing PR.

fork is not up to date

Offtopic: See the Pull app, it'll help you with that. 🙂


[TTFS] Replace broadcast operators by explicit for loops in IPM kernels

Reduce drastically the precompile time.
On current master, we get
InferenceTimingNode: 3.715741/6.123850 on Core.Compiler.Timings.ROOT() with 1 direct children

On this branch:
InferenceTimingNode: 2.793325/3.842035 on Core.Compiler.Timings.ROOT() with 1 direct children

x-ref: #186

Codecov Report

Merging #231 (4a56f4b) into master (8260ac9) will increase coverage by 1.54%.
The diff coverage is 90.44%.

@@            Coverage Diff             @@
##           master     #231      +/-   ##
==========================================
+ Coverage   74.19%   75.74%   +1.54%
==========================================
  Files          38       38
  Lines        3697     3512     -185
==========================================
- Hits         2743     2660      -83
+ Misses        954      852     -102

Impacted Files
Coverage Δ

src/IPM/kernels.jl
93.39% <84.33%> (+9.22%)
:arrow_up:

src/IPM/restoration.jl
100.00% <100.00%> (ø)

src/IPM/solver.jl
91.70% <100.00%> (+0.30%)
:arrow_up:

src/nlpmodels.jl
100.00% <100.00%> (+3.22%)
:arrow_up:

src/LinearSolvers/linearsolvers.jl
0.00% <0.00%> (-20.00%)
:arrow_down:

src/Interfaces/utils.jl
66.03% <0.00%> (-2.69%)
:arrow_down:

src/utils.jl
83.78% <0.00%> (-1.94%)
:arrow_down:

src/Interfaces/MOI_interface.jl
69.53% <0.00%> (-1.78%)
:arrow_down:

src/options.jl
96.42% <0.00%> (-0.24%)
:arrow_down:

src/KKT/KKTsystem.jl
97.14% <0.00%> (-0.23%)
:arrow_down:

... and 21 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[zh]Update content/zh/docs/reference/access-authn-authz/rbac.md

please always keep your PRs in 1 commit. There are 2 now.

/label tide/merge-method-squash

/lgtm
/approve


Certificate (.cer) files are encoded as JSON

Thanks for providing the Azure Key Vault Explorer tool – it’s quite handy.
I came across an unexpected behavior today when adding public certificate (.cer) files to a key vault. It seems that the tool wraps the contents of the certificate in a JSON structure like the following:
{ Data: “….” }
Why is that? And is it something I can disable?

Azure does not explicitly support public keys with the limited thinking that since the keys are public that they are not secrets and therefore don't belong in a vault.  While that's technically true, from a pragmatic perspective, it only makes sense to store certificates in one place, right?
I say all that to say that the data structure used by the the Key Vault Explorer is added as a convenience and something of an afterthought.  I've posted some code on Stackoverflow that show how you can access the certificate from C# code:  https://stackoverflow.com/a/48106470/764307
Hope that helps!


Fix URL in README

Fix #3109

Thanks @dkhamsing, very quick, was just about to change this myself.
Thanks for your awesome work on frankenstein.

Thanks!

:smile:

Thx @kylef, very much a work in progress :grin:


Remove in-model as_json methods in favor of the updated OpenApi method.

Removing in favor of the as_json changes in https://github.com/ManageIQ/catalog-api/pull/152

@gmcculloug Isn't this going to send every column from a table down to the clients. Usually we only send out a subset of the attributes defined in the openapi.json.

cc @hsong-rh

@bzwei Approval side doesn't have such logic. We had changed to use https://github.com/ManageIQ/approval-api/pull/58

@gmcculloug If there are any secrets in these tables we would end up sending them to the client. Approval is thinking of adding passwords we had passwords for web hooks. We should atleast go with a static array of attributes that have to be sent to the client and use that, till we can get the logic to read the openapi.json and create the JSON object from that that.

Also if QE is validating the object definition they are going to be seeing extra attributes in the response.

@mkanoor Updated to use the updated OpenApi serializer to convert IDs and date and then use slice to limit properties.  This is still a temporary change until we process the openapi schema version.


Upgrade to ember-cli v3.6.0

I feel the creator of this addon is not very active anymore on github, so if this PR has no activity in some significant time I'll publish a public version of the updated addon.

Representando papa!
Yo creo lo mismo, ya va para dos años que no hacen commits en este repo :(
Send an email to the maintainer and see if you can publish a new version on npmjs

@vpassapera I am now a maintainer. When I get permissions to push to npm i'll publish the new version


renamed aaS files and adjust their titles to be consistent overall

the naming of FaaS, PaaS, IaaS files and their titles have not been consistent. Now they are :)

@CathPag what do we need to merge it?

@iamNoah1, almost. Per our workflow, we need a second approver. Once that is done, we'll merge :)

Great change! Thank you!


Improve readability of filter method, and update README.

Modifications for comments posted in Pull Request #3.
Please review if you don't mind.

Coverage remained the same at 100.0% when pulling c0cb7c2b45a3f9bf24f2a14cbd15939735ed020a on kogecoo:pr3_remainings into 779c00e05db5b3ecbd01478e34f35906afedc316 on gterzian:master.

Thanks a lot! I've made some comments, please let me know what you think.

Thank you for your good points. I've made response comments to all of it.
If you find my oversight or some misunderstanding, feel free to let me know.

@kogecoo Ok great, I agree with your comments and choices. Please make the changes and I'll merge them in!

Coverage remained the same at 100.0% when pulling 851c0a6705e563c649e7d7cc388246a9ef351b74 on kogecoo:pr3_remainings into 779c00e05db5b3ecbd01478e34f35906afedc316 on gterzian:master.

Thank you for your reviewing.I made additional commits and pushed them to this branch.

Great, thanks again @kogecoo Btw I think your English was really good! I hope I can learn Japanese some day, right now I am learning Chinese and living in Shanghai :smile:

Thank you again @gterzian for all of your patience! I enjoyed not only this implementation but also English communication.
I'm surprised at hearing that you are now in Shanghai and learning Chinese. I think Chinese is one of the most difficult language in the world. I guess you have very tough mind.And I'm glad to hear that you'd like to learn Japanese.  Please let me know the day coming, I'll be willing to support you!

@kogecoo thank you for your offer of support. Chinese is ok, once you get past the initial strangeness it's not so hard :smile:
I look forward to do more work with you, thanks a lot for your contribution! BTW I have released it on Pypi as 1.2.1 :+1:

Thank you, nice work :thumbsup:


Warning button color same as error color

Describe the bug
Warning button color same as error color, solid and outline versions.
How to reproduce
Can see within the v2 and v3 documentation pages for butttons.

Expected behavior
"btn-warning" should result in a "warning" color, similar to Alerts, etc
Versions
App

Angular: 8
Node: 12.16
Clarity: 2 or 3 (doc page)

Device:

Type: desktop
OS: Win 10
Browser Edge Chrome
Version

Additional notes

Hi @GaryHyland this is intentional. There hasn't been a clear use case for having a 'warning' button, in addition a yellow button is hard make accessible with contrast ratio requirements. The reason it is provided is we have success/warning/error in other places, but it is intentionally red.
I'm going to close this as we won't be making any changes for this.


DEVOPS: Fix the gcloud error on the release pipeline

Motivation
Fix the gcloud issue on the release and systest pipeline
Description

bump the gcloud to version 469.0.0

bors merge


feat: streamline dependency-review

Use inline configuration rather than a separate config file.

This requires options to be inline comma separated instead of lists. This makes it much harder to read and maintain. So lets just keep the existing system.


Fix broken gitref link in README.md

gitref.org points to services.github.com, and has been for some time[0].
[0] https://github.com/git/git-reference/issues/112

Thanks for the PR @Tharre ✨


issue 1 pull request

added functionality to the class IOUtils and changed access modifiers for fields in class Cell to public

Who was assigned for issue #1 (@jarc0der) , check please this pull request.

Formatter for output files basics.


[RFC] reduce logging output in the default case

Updates #47
Before:
% gb build .../sftp
2015/05/07 21:20:56 INFO project root "/home/dfc/devel/demo"
2015/05/07 21:20:56 INFO compile github.com/kr/fs
2015/05/07 21:20:56 INFO compile golang.org/x/crypto/ssh
2015/05/07 21:20:56 INFO install /home/dfc/devel/demo/pkg/linux/amd64/github.com/kr/fs.a
2015/05/07 21:20:57 INFO install /home/dfc/devel/demo/pkg/linux/amd64/golang.org/x/crypto/ssh.a
2015/05/07 21:20:57 INFO compile github.com/pkg/sftp
2015/05/07 21:20:57 INFO install /home/dfc/devel/demo/pkg/linux/amd64/github.com/pkg/sftp.a
2015/05/07 21:20:57 INFO build duration: 1.823214205s map[compile:1.854594245s]

After:
% gb build .../sftp
# compile github.com/kr/fs
# compile golang.org/x/crypto/ssh
# install /home/dfc/devel/demo/pkg/linux/amd64/github.com/kr/fs.a
# install /home/dfc/devel/demo/pkg/linux/amd64/golang.org/x/crypto/ssh.a
# compile github.com/pkg/sftp
# install /home/dfc/devel/demo/pkg/linux/amd64/github.com/pkg/sftp.a

This change reduces the amount of logging in the default case.
-q is still not the default, by default gb will still produce some output as it runs.
This is a deliberate decision based on watching new gophers use the go tool. Generally after replacing their Go install new gophers were confused that their builds suddenly take significantly longer (because the toolchain mismatch defeats incremental compilation). In those cases I wished that go {build,install} defaulted to -v, hence the motivation for gb being more talkative by default.
@wkennedy

yes please.  Too much output currently.

Thanks Brian. I want to keep working on this to get it down to one line per
package. Less than that will require the -q flag or some negotiation.
On Fri, May 8, 2015 at 12:35 PM, Brian Ketelsen<EMAIL_ADDRESS>wrote:

yes please. Too much output currently.
—
Reply to this email directly or view it on GitHub
https://github.com/constabulary/gb/pull/54#issuecomment-100077801.

+1 to one line per package. In my use of the go tool these days I habitually run go install -v so I can see which packages are getting rebuilt. IMHO that is a good default for gb.

Thanks for the feedback.  I hear you all loud and clear.
On Fri, 8 May 2015 14:02 Chris Hines<EMAIL_ADDRESS>wrote:

+1 to one line per package. In my use of the go tool these days I
habitually run go install -v so I can see which packages are getting
rebuilt. IMHO that is a good default for gb.
—
Reply to this email directly or view it on GitHub
https://github.com/constabulary/gb/pull/54#issuecomment-100095569.

I'm going to commit this now as an improvement to what exists currently. It is clear that people want even less output in the default mode, don't really care about the output in debug mode, and -q should of course shut up.


Removes a default option that is breaking deploys

to: @ryandeivert @chunyong-lin @blakemotl
cc: @airbnb/streamalert-maintainers
Background
This default option actually breaks deploys because it provides the name of an existing logging bucket that is invalid.
Notably, this is already documented here: https://www.streamalert.io/en/release-3-0-0/config-global.html#s3-access-logging

Coverage remained the same at 95.893% when pulling 266f2fdaac32fb4e442f60fefed18f4a9f1b97db on dw--fixupthing into 319fc115d3cf42dea6df10f3a9d9ed7583671f7f on release-3-0-0.


added weddings page

Issue
Fixes #10
Description
Created weddings.html and added new navbar in all other frontend files.
Type of change
You can write manually also

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

Screenshot
Screenshot of the fix if required.
Checklist:

[x] My code follows the style guidelines of this project
[x] The app runs properly
[x] I have performed a self-review of my own code
[x] I have formatted and commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

@ArchismanKarmakar I have removed the conflicts. Please review it. Thankyou


Target Python 3.11+

We should target a more modern version of Python as our minimum version. This gives us a few advantages:

3.11 has more type hinting features that will make it easier to work with static analysis tools.
We're planning on using static analysis tools on all future projects
We control the horizontal, we control the vertical too (i.e. this project is owned by us and there are few projects using it now; future projects should start targeting 3.11 by default)
(In theory) this potentially shields us from some deprecation work until 3.11 becomes deprecated for at least 5 years

@cbouss as already raised the point that abs-cli targets Python 3.9 and that may become an issue.

Being too aggressive in what we support can be a problem. I generally see no problem is supporting older version like 3.9. Most newer typing features as made available through https://pypi.org/project/typing-extensions.
Supporting older versions is usually not much of a burden. I'm ok if we drop support for 3.7 and 3.8, but dropping 3.9 and 3.11 is quite aggresive.

After further discussion, we are not doing this. Closing.


[ci] Update github action runner to macOS x86_64 instance

Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

There is no description on this.  The change involves version locking instead of using latest.  Also removing macos from the nightly?
Seems a valid explanation of why this is a good idea should be in the description.

@justin-octo This PR is intended to retain a number of our engine build scripts and the nightly tensorflow on OSX x86_64 rather than arm, mostly because these scripts were not supposed to be using arm. It was a change on github actions that latest now refers to arm rather than having x86 remain the default. In the future, these scripts may need to be migrated to further expand our osx arm support, especially as all osx support moves to arm. This change fixes them for the moment.
For nightly, this is only removing the tensorflow part of nightly for which we have never supported osx arm. As a temporary measure, we are going to stop testing tf on osx. If we do add the engine support, we will add this back to nightly as well.