[OR-1481] Add code to predeploy USDC and L2UsdcBridge contract in L2 and create genesis file

Address allocation for predeploy (addresses.go)
Create binding file about USDC contracts and add library code in foundry.toml
Add constructor and stroges related code to config.go and Immutables.go
Logic updates to proxy to setters.go

In this work, the purpose is to predeploy L2UsdcBridge.sol and FiatTokenV2_2.sol
I added the code necessary to predeploy these two contracts, and if you run make devnet-up, you can see that genesis-l2.json is changed.
+) If L1UsdcBridge is later deployed to L1, the address pre-assigned to L2UsdcBridge have to be changed.
Thank you!! 😄

Is there an option if I don't want to deploy UsdcBridge?
I think provide usdc bridge could be an option when service on-demand L2.
@rlgns98kr  What do you think of this?

Is there an option if I don't want to deploy UsdcBridge?
I think provide usdc bridge could be an option when service on-demand L2.
@rlgns98kr  @boohyung @praveen263112 What do you think of this?

Great!
I think so. I think we need a flag for it as well. 🙇🏻♂

Have you tried upgrade a those smart contracts? @xxeonge
We are using our own proxies, so maybe we need to check the admin slot for the new Proxies as well.

You are correct!!  Due to issues with the proxies we're using, so I considered upgrading the contracts.
However, I've been advised against upgrading the USDC and L2UsdcBridge contracts hastily, as they conform to the Circle standard. So, instead, we've updated the proxy logic for the USDC contract in setters.go, with assistance from Steven. Thank you!😊

You are correct!! Due to issues with the proxies we're using, so I considered upgrading the contracts. However, I've been advised against upgrading the USDC and L2UsdcBridge contracts hastily, as they conform to the Circle standard. So, instead, we've updated the proxy logic for the USDC contract in setters.go, with assistance from Steven. Thank you!😊 @nguyenzung

Thank you
I think it looks good right now.
I think we can check this line later to make sure it can work with our own proxy: https://github.com/tokamak-network/tokamak-thanos/blob/OR-1481-Add-some-USDC.e-and-USDC-Bridge-code-to-create-genesis-file/op-chain-ops/genesis/setters.go#L77


Propagate non-ready consumers top level condition in ConsumerGroup

When a Consumer wasn't ready, we would get a generic error
on the user-facing API which isn't optimal to debug the issue.
Signed-off-by: Pierangelo Di Pilato<EMAIL_ADDRESS>

/retest

/retest

/retest


Add destroy api?

yea i think khalti is not good at the moment for things like subscription etc.. Esewa has got so many api and support for many programming language. I think we will need to write on and opensource it .

Any details on this ?

Sorry for late reply. Not sure if its worth having it but the idea was to have a method which removes checkout iframe from merchant's webpage.


fix metadata resolution

wrong branch, see https://github.com/NativeScript/android-runtime/pull/213


"Cojasca", "Fântânele", "Iazu" are listed in the drop-down menu

Description:
When user writes ”Cojasca”, "Fântânele" and "Iazu" the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Cojasca"

Expected results
The location appears in the drop-down menu as "Sat Cojasca, DÂMBOVIȚA (Cojasca)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Fântânele"

Expected results
The location appears in the drop-down menu as "Sat Fântânele, DÂMBOVIȚA (Cojasca)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Iazu"

Expected results
The location appears in the drop-down menu as "Sat Iazu, DÂMBOVIȚA (Cojasca)".


Bug: On stealjs.com, there is a link redirecting to a unexciting page

Always reproduces on all browsers
Steps:

go to http://stealjs.com/
click on the CSS icon from the "works with your favourite tech"
notice that it redirects to a 404 page

Hi @AlexGandu, thanks for filing this issue! The StealJS website has it’s own repo, so I’ve created an issue there to track this.


Can't apply Bijectors.ordered to TDist(), says it's constrained

The following fails with the error ArgumentError: ordered transform is currently only supported for unconstrained distributions.:
using Turing

Bijectors.ordered(filldist(TDist(1),2))

However, the t-distribution is supported on the real line and thus in fact unconstrained in the sense of this package.

Could you file this to Bijectors.jl?

The reason seems to be that TDist's support result in the wrong type:
julia> bijector(TDist(1))
Bijectors.TruncatedBijector{Float64, Float64}(-Inf, Inf)

On a side note, Distributions also has MvTDist for this purpose, but this doesn't work either, which I think it should. So this is probably a bug.

In the meantime, you can bypass the check by calling the constructor directly:
julia> Bijectors.transformed(filldist(TDist(1), 2), Bijectors.OrderedBijector())

MultivariateTransformed{Product{Continuous, TDist{Float64}, FillArrays.Fill{TDist{Float64}, 1, Tuple{Base.OneTo{Int64}}}}, Bijectors.OrderedBijector}(
dist: Product{Continuous, TDist{Float64}, FillArrays.Fill{TDist{Float64}, 1, Tuple{Base.OneTo{Int64}}}}(v=Fill(TDist{Float64}(ν=1.0), 2))
transform: Bijectors.OrderedBijector()
)

But please report this to Bijector when you can since there seem to be at least two to three bugs here!

Thanks a lot for sharing the method to bypass!
I have filed the issue at Bijectors.jl: https://github.com/TuringLang/Bijectors.jl/issues/287.


Footnotes messed up in text export

Observe last lines in https://ambuda.org/proofing/kushakumudvatiya/download/text
[^1]M omits नाम [^2]T1, T2 add- ओंकारश्चाथशब्दश्च द्वावेतौ ब्रह्मणः पुरा। कण्ठं भित्त्वा विनिर्यातौ तस्मान्माङ्गलिकावुभौ॥ओं।। T2 adds- इदं कुशकुमुद्वतीयं नाम नाटकं रामाभट्टेन लिखितम्। obviously this is the name of the scribe. M- इत्थं श्रीमत्तञ्जनगरविराजमानं श्रीमद्राजकीयं श्रीसरस्वती महाल ताळपत्रग्रन्थानुसारेण लिखितं समवलोकितं च सद्विजयतेतराम्।

Contrast with what was actually typed at https://ambuda.org/proofing/kushakumudvatiya/178/ :

We observe that the footnote definitions were typed in separate lines but the backend mashes them up together without considering the fact that they start with the string [^.
Also, given that different pages use identical footnote numbers, it would be very useful for the text export to have page numbers, so that the footnote definition may be easily matched with anchor intra-page.

I count this as another point (like #503) in favour of the grand vision of rich editor + preview. :-) Either one would help on its own:

If the editor natively supported footnotes in its structure (rather than relying on markup), everything would be stored with a uniform structure,
If the editor had a preview (and the input was still markup like currently), the proofreader would have noticed something was amiss and inserted blank lines.

But in the meanwhile, I guess for this text separating the footnotes is part of the manual post-processing work required (or hack it in the code like you suggested: if a line in the markup starts with [^ then treat it as a separate footnote).
cc @suhasm fyi

I count this as another point (like #503) in favour of the grand vision of rich editor + preview. :-) Either one would help on its own:
* If the editor natively supported footnotes in its structure (rather than relying on markup), everything would be stored with a uniform structure,

साधु।

But in the meanwhile, I guess for this text separating the footnotes is part of the manual post-processing work required

नैतत् साधु - at the very least the download offered should not be worse than whatever was input by the proofreader.

(or hack it in the code like you suggested: if a line in the markup starts with [^ then treat it as a separate footnote).

युक्तम्।

But in the meanwhile, I guess for this text separating the footnotes is part of the manual post-processing work required

नैतत् साधु - at the very least the download offered should not be worse than whatever was input by the proofreader.

Agreed: I meant that someone needs to make another pass on the proofreader's work, so that the backend's interpretation matches of the text matches what was intended. (So I meant one-time post-processing by a tech-savvy Ambuda person/volunteer, not by each person who downloads the text export.)
I also agree with offering a download that is the “raw” text input by the proofreader, with no “smart” processing line collapsing line breaks: I think (not sure) the issue #25 is about this.


Make the plugin releasable

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:
/kind bug
/kind enhancement

What happened:
There is no release process. no artifacts available.
What you expected to happen:
First release is done, and plugin artifacts can be downloaded and installed.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

CDI version (use kubectl get deployments cdi-deployment -o yaml):
Kubernetes version (use kubectl version):
Cloud provider or hardware configuration:
Install tools:
Others:

https://github.com/kubevirt/project-infra/pull/1687

done: https://github.com/kubevirt/kubevirt-velero-plugin/pull/10
First release (marked as pre-reelase) is now ready:
https://github.com/kubevirt/kubevirt-velero-plugin/releases/tag/v0.1.0
https://github.com/kubevirt/kubevirt-velero-plugin/tags


SW-4881 Search and filtering within internal deliverables table

Pass the following through into the requestDeliverablesSearch call from the internal deliverables table:

Fuzzy search
Table featured filters
Participant dropdown filter

This just shows the SearchNodePayload being created

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

#2206  👈
#2205
main

This stack of pull requests is managed by Graphite.

Merge activity

Feb 27, 11:22 AM EST: @nickgraz started a stack merge that includes this pull request via Graphite.


Add logging binary support when terminal is true

Currently the shims only support starting the logging binary process if the
io.Creator Config does not specify Terminal: true. This means that the program
using containerd will only be able to specify FIFO io when Terminal: true,
rather than allowing the shim to fork the logging binary process. Hence,
containerd consumers face an inconsistent behavior regarding logging binary
management depending on the Terminal option.
Allowing the shim to fork the logging binary process will introduce consistency
between the running container and the logging process. Otherwise, the logging
process may die if its parent process dies whereas the container will keep
running, resulting in the loss of container logs.
Signed-off-by: Akshat Kumar<EMAIL_ADDRESS>
Code to start logging binary referenced from pkg/process/io.go

@cpuguy83 @fuweid PTAL?

@cpuguy83 PTAL?

Thanks for the reviews, all. Is this ready to be merged? @AkihiroSuda

removing the cherry-pick labels for now, per https://github.com/containerd/containerd/pull/4616#issuecomment-710746986 and https://github.com/containerd/containerd/pull/4615#issuecomment-730001554


Fix streaming url conversion

This solves #157 and also many of the URLs posted in #124
I think it was me who broke the code where URLs are converted from platform string to UTF8.

I thought this was semantically equivalent, but then I realized it was a c++ gotcha.

I am pretty sure that I had originally tested this after switching this to use our StringUtils class. But probably after my testing, I further "optimized" the code and removed the variable, which turned out to be a bad idea in C++ :-S


Windows 10 support

Hi,
I'm currently toying around with an Windows 10 implementation (using winrt-rust) of rumble.
If you are interested in it I could create a PR!
For now connecting, discovering characteristics, reading and getting notified about a value change are working.
The rest is still to do as I wasn't able to understand the correct working of these API parts..
Thanks!

That's great! Please let me know if there's anything in the common API that's too linux/bluez specific. I tried to keep the functionality generic, but I'm not very familiar with other bluetooth stacks. And definitely happy to accept PRs and answer questions.


DOC: Improve docstrings for connecting to Ibis

Fixes #444.

@wesm Let me know about the deprecation warning issue referencing ibis.impala.connect

I added the deprecation warning on ibis.make_client, is there anything else?

grep for any references to ibis.impala.connect.


add support for Claude 3

Hi,
Is there any plan to add support Claude 3, thanks a lot!

Hello, I haven't tested it, but the example app should already support Claude 3. It already uses the new Claude messages API required for Claude 3.
To update the model used by the demos in your fork, update the following lines:

https://github.com/aws-samples/amazon-bedrock-serverless-prompt-chaining/blob/main/stacks/util.py#L45-L48
https://github.com/aws-samples/amazon-bedrock-serverless-prompt-chaining/blob/main/stacks/util.py#L111
https://github.com/aws-samples/amazon-bedrock-serverless-prompt-chaining/blob/main/stacks/util.py#L208
https://github.com/aws-samples/amazon-bedrock-serverless-prompt-chaining/blob/main/functions/most_popular_repo_langchain/index.py#L79
https://github.com/aws-samples/amazon-bedrock-serverless-prompt-chaining/blob/main/functions/most_popular_repo_langchain/index.py#L132

Note that Bedrock Agents does not yet support Claude 3, so that demo will not be updated.

Thank you Clare.
Yes, I have noticed that, I am not familiar with CDK, so I manually changed model id in StepFunction.
Most demo works well, however, I found in Trip plan demo, it doesn't use message API still "Human, Assistant", just for your heads up.
For Bedrock Agents, I noticed it as well, so I will stay tuned. However, for Langchain demo, I tried to manually modify related two Lambda code (lookup, summary), just change model id and "max_token_sample" to "max_token", but not worked, I will debug it later.

Where in the trip planner demo are you seeing it use Human/Assistant?  I thought I found all the places in this commit, but maybe I missed something


PostgresConnection should connect via hostname, not IP

I've been looking to upgrade an app from the community wrapper, but our database server expects a connection to a specific hostname, not the resolved IP. Because of this, I get the following error message (actual values [redacted]):

no pg_hba.conf entry for host "[IP address]", user "[username]", database "[dbname]"

This may be a TLS issue. The community wrapper around libpq must've been smarter about this.


Shrink width on Language Dropdown/Desktop

Narrows the language option dropdown on 992px + devices

@faxblaster For my laziness, would you mind sharing some screenshots at various sizes?
Thanks!

@faxblaster For my laziness, would you mind sharing some screenshots at various sizes?
Thanks!

Edited my initial comment to include. Let me know if there's a better place for them?


EmaLink doesn't appear to be polled in intervals for battery alarm

There's an option to have the app alert you when the battery of your EmaLink goes below a user adjustable threshold. (Eg: I have it set to < 30% = alarm.)
However, the app never seems to query the EmaLink (for that variable) unless you go to Pump and tap your EmaLink to view status and reveal the settings. It will only at that moment ask for the battery level, show it and trigger the notification/alarm - at which point it's a bit late.
It'd be nice if the app queries the EmaLink battery once every x hours. Maybe every 3 hours or so?
Note: had the same issue in the previous iteration of the app, where it was mentioned that it did work fine for OrangeLink.
Medtronic 754 + EmaLink, Trio Dev, iOS 17.5.

hey 👋 - silence for 30 days 🤐 ... anybody? triage is required!

Not sure it's very handy to auto-close bug reports just because they haven't gotten attention yet, auto-discarding something is probably not the best way forward to eventually work towards an app of which the features are working as expected in a stable manner. I see multiple issues including more severe bugs being moved out now. (This one is mostly just annoying as you cannot rely on the app to warn you about the battery state of an important part of the setup, but not directly dangerous.)

Not sure it's very handy to auto-close bug reports just because they haven't gotten attention yet, auto-discarding something is probably not the best way forward to eventually work towards an app of which the features are working as expected in a stable manner. I see multiple issues including more severe bugs being moved out now. (This one is mostly just annoying as you cannot rely on the app to warn you about the battery state of an important part of the setup, but not directly dangerous.)

Commenting on an auto-closed issue will bring attention to it, and may get it reopened. So no big deal if stakeholders look out for these stale issue/closed issue alerts.


[EditorFileDialog] Do not update file selection on filter change if nothing was previously selected.

Fixes https://github.com/godotengine/godot/issues/100077

Thanks!

@bruvzg  This issue is still present in dev7 :/

was the patch uneffective?

The fix was merged after dev7 was released, so that's normal, you'd need to test the master branch or wait for beta1.


How to convert the Dom into html elment

let html = r#"
hello world

hi paragraph
"#;
let nodes = Dom::parse(html).unwrap_throw();

// add the html
let doc = window().unwrap_throw().document().unwrap_throw();
let my_dom_element = &doc.get_element_by_id("1").unwrap_throw();
my_dom_element.append_child(&nodes.children[0]).unwrap_throw();

erorr
31 |         my_dom_element.append_child(&nodes.children[0]).unwrap_throw();
   |                                     ^^^^^^^^^^^^^^^^^ expected struct `web_sys::Node`, found enum `html_parser::Node`

This library was just created to parse html. It would be nice to do these kind of mutations etc. But I've not had the time to work on the library or even think about future use-cases.


unrecognized configuration parameter "datestyle"

Describe the bug
The SQL will be sent to the server if we use psycopg2
SET datestyle TO 'ISO';

To Reproduce
Expected behavior
Additional context

How about adding a no-op configuration mode to ignore these unrecognized ones?


Added ability to export trade history using saved settings

I've updated this a fair bit today, but I think this is now stable and ready for merging if you'd like to take a look, @markrickert

Awesome! Thanks for fixing that merge conflict!


php: add a dev ouptut & strip all the binaries

These commits introduce stripping to all the binaries, and create a dev output to separate php's generated include files which contain references to build inputs.
Also update nixos to pick the right output.

Other than the comment I made, looks good to me.

Closing while we are working on #15339


shutdown manager image not possible to override from envoyproxy/gateway-dev

Description:

What issue is being seen? Describe what should be happening instead of
the bug, for example: Envoy should not crash, the expected value isn't
returned, etc.

I am trying to develop EG and building own gateway controller. However, when I try to start own gateways it will use always envoyproxy/gateway-dev prefix images as shutdown image. It is not possible to override shutdown image as own image. Which makes it really difficult to develop envoy gateway. This shutdown image is defined in https://github.com/envoyproxy/gateway/blob/3f8d05b3bb5e5dce9644d756870459fdba811431/internal/infrastructure/kubernetes/proxy/resource.go#L279-L284 and as you can see, its not possible to override that value.
Repro steps:

Include sample requests, environment, etc. All data and inputs
required to reproduce the bug.

build own envoy gateway controller image GOARCH=amd64 IMAGE=jesseh/envoy-gateway make image
deploy controller with custom image
create new gateway

envoy-eg-internal-7f4ff7e4-5874dd88d9-pprgg   1/2     ImagePullBackOff   0          17m
envoy-eg-internal-7f4ff7e4-7dc49f9c4f-zxhcx   1/2     ImagePullBackOff   0          4m47s
envoy-eg-internal-7f4ff7e4-d9d76b974-p6746    1/2     ImagePullBackOff   0          10m

events:
  Normal   BackOff    3m34s (x6 over 4m56s)  kubelet            Back-off pulling image "envoyproxy/gateway-dev:3f8d05b3"
  Warning  Failed     3m34s (x6 over 4m56s)  kubelet            Error: ImagePullBackOff
  Normal   Pulling    3m22s (x4 over 4m58s)  kubelet            Pulling image "envoyproxy/gateway-dev:3f8d05b3"

in this case the correct image is available in jesseh/gateway-dev:3f8d05b3, but its not possible to override the image in anyway

Note: If there are privacy concerns, sanitize the data prior to
sharing.

Environment:

Include the environment like gateway version, envoy version and so on.

current main branch
Logs:

Include the access logs and the Envoy logs.

I manually now modified https://github.com/envoyproxy/gateway/blob/main/api/v1alpha1/shared_types.go#L30 row to jesseh/gateway-dev:latest and everything works fine after new compilation. However, this is not really developer friendly currently

seems that there is PR https://github.com/envoyproxy/gateway/pull/3269 for this already


Firebase Phone MFA not working with some numbers

I found this similar issue.

@ibragimazizli if you're still experiencing this problem and want to continue the discussion just leave a comment here and we are happy to re-open this.
Originally posted by @google-oss-bot in https://github.com/firebase/flutterfire/issues/12594#issuecomment-2095037520

I've been using Firebase Authentication with Multi-Factor Authentication (MFA) in my Flutter app (web) without issues for the past 6 months. Recently, I started encountering a 500 Internal Server Error when trying to initiate MFA. No code changes were made to the authentication logic. It was working before, some days ago I startet getting this 500 error on the mfa:start endpoint.
Packages:
dependencies:
  firebase_auth: ^4.14.1
  firebase_storage: ^11.0.10
  firebase_core: ^2.4.1
  cloud_firestore: ^4.3.1

Error Message:
{error: {code: 500, message: "Internal error encountered.", status: "INTERNAL"}}

Checked the Endpoint calls, this throws 500:
https://identitytoolkit.googleapis.com/v2/accounts/mfaSignIn:start?key=MY_API_KEY
In the body request it also contains all the neccessary info:
{
    "mfaPendingCredential": "AMzJoSk ... MFA PENDING CREDENTIAL",
    "mfaEnrollmentId": "db86e89e-b3c6-47cb ... ENROLLMENT ID ",
    "phoneSignInInfo": {
        "recaptchaToken": "03AFcWe... RECAPTCHA"
    }
}

The signInWithEmailAndPassword and recaptcha endpoint runs to 200, and returns back the mfa-id and all the other info related and needed.
What I've Tried:

Retrying the request: Multiple times with no success.
Checking Firebase Status: No reported outages or issues.
Reviewing API Documentation: No updates or deprecations found related to my issue.
Updating Dependencies: Cant really update it easily, because this is an old project, and have dependency conflict with the latest Flutter, but as I said, this project has been running since 6 month without issues.

Additional Info:
I found a note about cross-origin redirect sign-in on Google Chrome M115+ stopping on June 24, 2024. However, I don't think this is related as my issue is with the signInWithEmailAndPassword and it should not affects phone auth MFA process itself?!
Questions:

What could be causing this sudden 500 Internal Server Error?
Has there been any recent changes to Firebase Auth that could affect MFA?
How can I debug or resolve this issue?

Any help or insights would be greatly appreciated!
FIREBASE PROJECT ID: marketprog-bf7d3

UPDATE:
I talked to the support, I experience this issue with Hungarian phone numbers (we are operating here in Hungary...), but I tried with the added test phone number it worked, also tried some free USA and Finnland based phonenumber and the sms were received, but the hungarian "carriers affected with consistently low SMS success rates are temporarily unavailable" - the support told me.
This auth feature is crucial for our production application, where we need a second layer after the sign in email+password also type the SMS code they receive.
I found a similar stackoverflow reference, but the author deleted the post, but it looks like we have the same issue (production app, eastern european country, phone auth/mfa, sms OTP):

Stackoverflow link: https://stackoverflow.com/questions/78718291/firebase-multi-factor-authentication-suddenly-failing-with-500-internal-server-e

Hello @radikris,
Thank you for pointing this out. Since only some numbers (Hungarian) are affected but not others, I'm afraid that this issue cannot be solved/handled by the FlutterFire team. You can continue contacting Firebase support to reach a solution.


Use of slashes "/" as separators for Copyright/Species/Character/etc

What steps will reproduce the problem?
Using / ot \ as a separator for Copyright
What is the expected behavior? What do you get instead?
i would like to make an embedded folders for more than one copyright tag, but instead, it just puts a blannk space ' '
How often does this problem occur?
everythime
What version of the program are you using? On what operating system?
5.3.3 on Windows 10
Please provide any additional information below
I know that it's because how slashes are interpreted, but there is a way to bypass that? I dont know how to use the filenaming by Javascript for conditiionals and can't get the "regular" expression to work with slashes used as separators.

looks like you use it wrong, for example, I use "%search%%md5%.%ext%" and all works

@MasterPetrik Literally studied the page xD. I think I don't explain myself very well. I have this expression for file naming: "%rating%</!%search_1% /Other/%copyright%></%search_1%>/%artist%<%copyright%><%character%>-%md5:maxlength=8%.%ext%_  "
This works if I'm looking for a specific things (used search) it will use search 1  as a folder, else it will create a folder named Other  and put inside another folder for copyright  tags.
For example, if I looks for pokemon . Inside the rating  will be pokemon  folder and inside the image in question.
But if I find a pokemon  pictures by looking on the main page it will create the Other  folder and inside a Nintendo  folder and then the picture inside.
But I'm looking to is for grabber create "rating/Other/Nintendo/pokemon/image"
Instead of:
"rating/Other/Nintendo+Pokemon/image"
Since it has more than one Copyright tag, it separate it with a + or whatever you use, and since copyright doesn't have a way to select specifically one tag of them (like search 1 2 3) I can't use the filenaming method. But in the copyright section, the separate character can't be "/" to force it to create another folder. "/, ", " since this characters are not allowed.
Thanks for the reply btw!

Now that's what I call advanced usage of the filenaming feature :smile:
Indeed, the tags merging is done before forbidden characters are removed. And / is a "forbidden" character because we wouldn't want tags such as fate/stay_night to create a new folder. The solution is to change the code and remove forbidden characters before merging tags. This way, even if the separator is "forbidden" as in your case it won't be replaced by a space.
However, with what you want to do, you could have one copyright in different places, with different depths. Even more so as tag order is not really guaranteed. Let's say you have "Mario Nintendo", "Nintendo Pokemon", "Pokemon Pokemon_X", "Nintendo Pokemon Pokemon_X", it will result in this tree:

Mario

Nintendo

Nintendo

Pokemon

Pokemon_X

Pokemon

Pokemon_X

Oh you are right, i didn't think in the order. What a problem. I had it restringed to "if more than 1" tags in copyright but as you said, it will make a mess. Maybe with a "if less than" and "if more than" then can be restringed to the use of two folders and no more, and in case that there is more tags, then it will use the "Crossover" folder but i think that there is no way to do that yet right? I didn't found it. And even in that case, the order will still be messed up (pokemon/nintendo | nintendo/pokemon)
The only think i can think of is the use of "if" statement or customs rules like "if more than 1 copyright and one of them is nintendo then put nintendo first" but that will be a nonsense for literally everything that's not nintendo xD. Not including that diferent pages has diferents ways to put copyrights. (Like danbooru that uses pokemon, pokemon sm, pokemon x as part of the copyright)
I'll try with custom tokens and see what can i do. btw i couldn't find info about the "conditional filename" option in the filenaming wiki thou.
Thanks for the reply! I'll experiment a little more with the naming and the options and see if i can get a better configuration. Also what an incredible work you do here by the way! I really think this could be the ultimate app for image collectors as myself. Thank you a lot!

Maybe with a "if less than" and "if more than" then can be restringed to the use of two folders and no more, and in case that there is more tags, then it will use the "Crossover" folder but i think that there is no way to do that yet right?

Isn't it already feasible though?

Not including that diferent pages has diferents ways to put copyrights. (Like danbooru that uses pokemon, pokemon sm, pokemon x as part of the copyright)

If you use the "Use shortest if possible", any copyright which is another copyright with something appended (example: pokemon_x = pokemon + _x) will be removed, only keeping the shortest copyright.

And even in that case, the order will still be messed up (pokemon/nintendo | nintendo/pokemon)

Usually tags are sorted alphabetically. It's not enforced by the program though. I could add a sort option to force it.

I'll try with custom tokens and see what can i do. btw i couldn't find info about the "conditional filename" option in the filenaming wiki thou.

Well it's not very complicated, so let me explain here: the condition can be either a tag or a %token%. In your case, you could use %search_1% as a condition to generate your filename:

With %search_1%: %rating%/%search_1%/%artist%<%copyright%><%character%>-%md5:maxlength=8%.%ext%
Default: %rating%/Other/%copyright%/%artist%<%copyright%><%character%>-%md5:maxlength=8%.%ext%

It's just a convenience to not use <> conditions in some basic situations. In your case, it just makes it more complicated. It can be useful if for example you want to treat some images differently depending on some tag, and don't want to ruin your filename for it. Let's say you want to have "Nintendo"-tagged images in a totally different location from other files, you can use the "Nintendo" as a condition to change both the filename and path for these images.

Thanks for the reply! I'll experiment a little more with the naming and the options and see if i can get a better configuration. Also what an incredible work you do here by the way! I really think this could be the ultimate app for image collectors as myself. Thank you a lot!

If you really want to do something complicated, you could always invest some time converting your filename to JavaScript. Then you'll be able to do whatever you want. As long as you know JavaScript of course.

Thanks so much for let clear the "Shortest if possible" I thought it works on some other way. I will make some test with the "if more than" and then try to learn some Javascript.
Thanks again for the help!


Marks EffectorResourceTest.testInvokeEffectorWithTimeoutTimesOut as WIP

I assume this only fails sometimes?  Can you add a comment with some details on the failure?  So that someone can fix this in the future.

Please close due to #1323


Cannot read property 'nativeElement' of undefined when using with Ivy

I've looked around here about ivy-related issues but I haven't found much.
Is there any desire to make this library compatible with Ivy?

I would also love to keep using this library with Ivy!

Just upgraded to Angular 9 and I get the same error. I turned off Ivy and I was still getting the error, so I'm not sure if it's an Ivy issue or not...

Same with me, just upgraded to Angular 9 and  got this error:
TypeError: Cannot read property 'nativeElement' of undefined
    at LottieAnimationViewComponent.push../node_modules/ng-lottie/__ivy_ngcc__/dist/esm/src/lottieAnimationView.component.js.LottieAnimationViewComponent.ngOnInit

Hello
I had the same problem with angular 9, we must replace ngOnInit by ngAfterViewInit to solve the problem
https://github.com/Jrbebel/ng-lottie/blob/master/src/lottieAnimationView.component.ts

I think this project is on life support, it hasn't had a commit in almost  year and a half, and the issues are piling up. Thanks @Jrbebel ,  I'll have to fork the project and try it myself.

@ErikHDev any news ?

@ErikHDev I would suggest taking a look at ngx-lottie before you attempt fork this project.
ngx-lottie seems maintained and works with ivy, and most importantly, it's highly compatible with this project, API-wise.
It literally took me only a few minutes to switch since almost all events and APIs had equivalents either in the same name (so no change is required) or with a similar one (event animCreated is animationCreated in ngx-lottie).
Also, ngx-lottie has better TypeScript typings as opposed to this project where the emitted event data is any.
I hope this helps someone.


Remove CloudEvents code from up-java

CloudEvents were send and received over transports in earlier versions of uProtocol (ex. 1.3.x. For 1.5.x we use UMessage (and UAttributes) as the message schema (contenttype). The current cloudevent code in up-java is trying to convert a 1.5.x version of uMessage into a Cloudevent however this version of CloudEvent is not compatible with old production deployments of the protocol.
After carefully studying the implications of our cloud infrastructure supporting 1.5.x uMessages and 1.3.6 CloudEvents simultaniously, we found that we can support both version independently through the new open source up-java library, and the old GM closed source sdk for 1.3.6. This then means that the onus is on the cloud uEs to know what version of the protocol the vehicle (and other deployments) talk, and build the appropriate message type (UMessage or CloudEvent) that is sent down to a transport that can support both (in the cloud only).
Having said all that, and given that there is no need to convert 1.3.6 Cloudevents to uMessages (as uEs will be responsible to send/receive in the correct message type), we do not need anymore ANY of the CloudEvent code in up-java and it can all be removed.
This issue is to blow away all the old cloudevent code in up-java, yey....

merged


[libIDE] When printing type interface, ensure we translate 'Self' to the type under printing. rdar://27306890

[libIDE] When printing type interface, ensure we translate 'Self' to the type under printing. rdar://27306890

Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci please test


feature suggestion: continuously read /run output

Issue
If I'm working on an application that's intended to keep running indefinitely (say, a webserver), it would be nice for aider to be able to continuously watch its log output and latch onto issues. Maybe that could be a background task for a very lightweight, cheap model.
When something obviously untoward happens, such as an [ERROR] log message flying past, it could ask the user whether to investigate (or ignore once or ignore forever), and then go from there as if the user had manually pasted the offending log messages.
Similar to the awesome new feature of aider pre-populating "please fix" after a /run concludes with an error, but for applications that don't return.
I realise that this would potentially mean hogging the CLI forever, but maybe instead aider could e.g. grow a /watch-log keyword which would continuously read the contents of a given file. The application could then be started in a separate shell and instructed to log to that file, so that aider's interface would remain accessible just like now, yet be able to observe the application in an even tighter loop.
Version and model info
No response

maybe my feature request #2740 could achieve this.
You would use my requested piping feature to pipe the output into aiders command?


spacewarp.modlist info

Nice mod! Thanks for putting it out there.
When I bring up the Mod menu from the loading screen there is no version or source info, and the KSP2 version looks like it's stuck with the default text "... - ... (can be * for no max version)"
Also the dependencies list is blank, should probably be none?

Which version are you using? In the latest (0.1.3] I tried to resolved this (for spacewarp 0.4.0). You are right though, there are no dependencies.

Fixed in latest.


[ISSUE-385] Fix compile warning

[ISSUE-385] Fix compile warning

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


TPUEstimator does not update meta-weights

System information

OS Platform and Distribution: Debian GNU/Linux 9.11 (stretch)
TensorFlow version: 2.1.0
Python version: 3.5.3

I train a neural network using TPUEstimator. In model_fn, losses are stored in losses list. After the end of an epoch, meta-weights theta should be updated using losses. But for the entire epoch, only one tensor is added to losses (len(losses) = 1) and theta remains the same (theta==theta_prev = True).
Here is gist with the complete code. Part of model_fn:
...

    def update_theta():
        global losses
        if len(losses) > 1:
            losses_by_updates = [[] for _ in range(num_updates)]
            for i in range(num_tasks):
                for j in range(num_updates):
                    losses_by_updates[j].append(losses[i*num_updates+j])
            losses_by_updates = [tf.reduce_sum(loss) / tf.to_float(num_tasks) for loss in losses_by_updates]
            gradients = theta_optimizer.compute_gradients(losses_by_updates[-1])
            with tf.variable_scope(tf.get_variable_scope(), reuse=tf.AUTO_REUSE):
                theta_op = theta_optimizer.apply_gradients(gradients)
            weights = copy_theta_to_weights()
            return theta_op
        else:
            return tf.no_op()
...

    loss = 0.0
    for key in logits_train_dic:
        logit_train = logits_train_dic[key]
        loss += tf.square(labels[key]-logit_train)
    loss_op = tf.reduce_mean(loss)
    losses.append(loss_op)

...

Is there any way to write loss to a list inside model_fn? Or maybe there is some other way to update the meta-weights using TPUEstimator?

@KirillAI
Could you please let us know if this is still an issue in latest stable TF v2.6.0 ?Thank you!


Return localhost instead of the contigured domain in virtual host

Hi I tried this package, but during the implementation of the helper function route() it returns http://localhost/..
In my project I set up a Virtual Host which point to localhost address I've been working on, The OS I'm using currently is windows 10,
For Example
  getData(route('ajax.get.event.list'), id, function(data) {
    data = data[0];
    data = editEventData(data);
    ...

the expected output should be
http://liz.dev/users/org-adviser/get/event
the return instead is
http://localhost/users/org-adviser/get/event

@filjoseph1989 The app is using Laravel's app.url config setting, which is filled from your APP_URL key in your .env file. Check that and see whether you have it set to localhost instead of liz.dev.
https://github.com/tightenco/ziggy/blob/master/src/BladeRouteGenerator.php#L20

@mattstauffer That worked for me!
I was just wondering if this wouldn't be better solved if the package were to detect the URL the same way Laravel does (as in Laravel's route() facade, which never fails)

Huh. Good point. Let me see where we can go there. :)

@mattisbones @DanielCoulbourne I have the same problem in Laravel 5.6 and Ziggy 0.7.1.
What route function {{ route('permissions.create') }} returns in blade:
http://sogl.lar/manage/permissions/create

With Ziggy:
http://localhost/manage/permissions/create

Just to help those who want it to work in vue3 i used a mixin and it works:
 createApp({
mixins: [
	{
		methods: {
			route: function(name, params, absolute) {
				return route(name, params, absolute, Ziggy);
			}
		}
	}
],
render: () =>
	h(app, {
		initialPage: JSON.parse(el.dataset.page),
		resolveComponent: (name) => import(`@/Pages/${name}`).then((module) => module.default)
	})

})
.use(plugin)
.mount(el);

Try to run:
php artisan optimize:clear
php artisan ziggy:generate
Also if you have a deploy script, remember to run the above before you run your frontend build


Comment block is not working when the line has preceding whitespace

So this works:
###
Some block comment
###
class ComposerExtension extends ContenteditableExtension

And this doesn't:
class ComposerExtension extends ContenteditableExtension
  ###
  Some block comment
  ###
  @sendActionConfig: ({draft}) ->

I've found that changing the begin regex for punctuation.whitespace.comment.leading.coffee from (^[ \t]+)?(?=#) to (?=#) makes indented block comments to be highlighted as comments.
But I don't know what the side effects of this change are..


Discrepancy between SDK models and API models

Hi team,
There seems to be a discrepancy between the SDK models and the API models. I found several models in the SDK where some fields are not available which are available in the API.
For example:

https://developers.klarna.com/api/#payments-api__session__authorization_token: In the GET /payments/v1/sessions/{session_id} response an authentication_token field is present, but in the PaymentCreditSession object (the return object of the klarnaApi.Payment.GetCreditSession) this is not available.
https://developers.klarna.com/api/#order-management-api__create-a-refund__reference: In the POST /ordermanagement/v1/orders/{order_id}/refunds request input a reference field is present, but in the OrderManagementRefund object (the input object of the klarnaApi.OrderManagement.CreateRefund action) this is not available. Also, in the response headers there should be a Refund-Id header, but I have no access to this in the SDK.
https://developers.klarna.com/api/#order-management-api__create-capture__reference: In the POST /ordermanagement/v1/orders/{order_id}/captures request input a reference field is present, but in the OrderManagementCreateCapture object (the input object of the klarnaApi.OrderManagement.CreateCaptureaction) this is not available. Also, in the response headers there should be a Capture-Id header, but I have no access to this in the SDK.

I may be missing some more examples. Could you please look into this?
Many thanks in advance.

Hi @JustinAren
Thanks for pointing this out. Yes, we know about this issue and currently we are working on migrating to the auto-generated models architecture. Such models will simplify a process of adding new fields and updating the models.
We already migrated the Klarna Payment and Order management APIs.
These changes are still not released, but we plan to release them in a short time.
Best wishes!

All the changes are available in the latest v3.1.10 release:
https://www.nuget.org/packages/Klarna.Rest.Core/3.1.10
The SDK Ref can be found here: https://klarna.github.io/kco_rest_dotnet/v3.1.10/api/index.html


Feature request:  Introducing “sudo” functionality into Granted

From Slack
Context:
From Iain Elder

Hi, all. I've been a lurker for a while but I just thought of a use case that would give me a compelling reason to switch to using granted from my current janky custom setup.
Does granted have some sort of "sudo" capability for AWS profiles?
By default I set my CLI to the "read-only" profile for the account I work in.
Maybe 1 command in 100 actually needs to modify something in the account.
Guess which command fails with "access denied" before I realize I have to use the "admin" profile!
To switch to the admin profile I need to set its whole explicit name in the command or in the script, which is more typing and perhaps makes the commands less reusable across accounts.
Obviously typing sudo aws doesn't work, but maybe granted would define a command such as sudoaws or similar that would call the aws command with the correct elevated privilege for the context.

From @chrnorm

I love this idea. At the moment we don’t do this, but would love ideas from everyone as to how this could work. The use case here is introducing “sudo” functionality into Granted, to elevate into an admin profile for a particular AWS command:
 granted sudo aws s3 rm <object>
As you’ve mentioned, Granted would need to know about the default and elevated role for an account. Perhaps we could do this with an attribute on the profile:

> [profile account-a/default] # profile naming convention doesn't matter
> ... regular profile config here
> granted_default_profile_for = account-a

> [profile account-a/sudo] # profile naming convention doesn't matter
> ... regular profile config here
> granted_sudo_profile_for = account-a

When you run assume, you’d just see account-a appear, and if you run assume account-a, you get the default profile for that account (account-a/default). The default role should be something like ViewOnlyAccess, which would mean a command like aws s3 rm  for example wouldn’t work.
If you want to use the elevated role for an aws command, you could run granted sudo aws s3 rm , and Granted would swap your profile to the elevated one just for that command.
Reposting in the channel here for visibility, would anyone else be interested in this sort of functionality being built-in to Granted? Personally I’d use it myself for when I am working against prod accounts!

Thank you @shwethaumashanker for tracking the feature idea.
Once I have refamiliarized myself with Granted's current features, I'll be able to provide more specific feedback on how I would expect a "sudo" feature to work.


Methods with a new this/new this.constructor don't break given a child class ctor that breaks subtyping

Using new this(...) under static methods and new this.constructor(...) under nonstatic methods, Flow lets me sneak by a couple of false positives:
class A {
  x:number;
  constructor(k:number) { this.x = k; }
  static build(k:number): this { return new this(k); }
  clone() { return new this(this.x); }
}

class B extends A {
  s:string;
  constructor(s:string) {
    super(1);
    this.s = s;
  }
  // static `build` is no longer okay--must override
  // `clone` is no longer okay--must override
}

var b = new B("some string");
var staticFalsePositive = B.build(3);
var nonstaticFalsePositive = b.clone();

Closing since this issue hasn’t seen a lot of activity. Sorry about that 😣
Please reopen with a reproduction on https://flow.org/try if you are still having issues!


Replace curl with built-in Invoke-WebRequest on Windows

Recently, the curl is not working in github workflow and failed to build base Windows images. Replace the curl with Invoke-WebRequest to bypass the issue.

/skip-all


Option to make the Android touch screen controls disappear when a physical controller is detected

Thanks for developing this app, it has made the game accessible for me to play on my android gaming device!
Device: Retroid Pocket 4
Android Version: 13
Latest Version of Pokerogue-App
I believe this is a bug because there is a touch screen settings that will not toggle off for me. OR, it's not supposed to go opaque.
The biggest gripe is that it puts an opaque white filter on the game and I don't think there is a way to remove the controller filter from the screen.
Here are some screenshots attached.
The first two are using touch screen controls. This is not touching the physical keys at all.

When any physical key is pressed then the opaque filter comes on

Physical keys are detected, but that filter just doesn't go away unless you fully quit the app and only use touch controls and never touch the physical keys.

This is an issue that needs to be fixed on the game devs' end I believe, see #23 and #34


Replace Newtonsoft.Json with System.Text.Json

(This was discussed in #2316 but deserves to be its own issue, as the work necessary to close this issue would be different to the work to close that one.)
I'm tired of typing JsonSerializer in my test projects and getting it autocompleted to Newtonsoft.Json.JsonSerializer because Microsoft.Net.Test.Sdk (via Microsoft.TestPlatform.TestHost) references (a very old version) of the now-essentially-deprecated Newtonsoft.Json, especially when my greenfields .NET Core app under test has zero references to Newtonsoft.Json.
This would also help people who do have explicit dependencies on Newtonsoft.Json and are being bitten by #2279.

We would love to do this, but we need to keep our dependencies compatible with net452 and System.Text.Json is not compatible with that.

It's actually much worse, the dependency on Newtonsoft.Json is not properly declared in the NuGet package.
I was trying to replace Newtonsoft.Json with System.Text.Json in Stryker.NET and the dependency on Newtonsoft.Json hit me hard. After some work to replace all JSON handling everywhere in Stryker.NET I was finally able to remove <PackageReference Include="Newtonsoft.Json" Version="13.0.1" /> from its csproj. But thanks to JetBrains Rider great feature where you can see transitive packages (Implicitly Installed Packages in Solution), I saw that Newtonsoft.Json was still referenced.
It turns out it was referenced through Buildalyzer/3.2.2 → Microsoft.Extensions.DependencyModel/2.1.0 → Newtonsoft.Json/9.0.1. So in order to completely get rid of Newtonsoft.Json I explicitly updated Microsoft.Extensions.DependencyModel to version 5.0.0 which has zero dependencies for the net5.0 target framework. I added <PackageReference Include="Microsoft.Extensions.DependencyModel" Version="5.0.0" /> to Stryker.Core.csproj and Newtonsoft.Json was definitely gone.
But then… this happened at runtime:
System.IO.FileNotFoundException: Could not load file or assembly 'Newtonsoft.Json, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed'. The system cannot find the file specified.

File name: 'Newtonsoft.Json, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed'
   at Microsoft.VisualStudio.TestPlatform.CommunicationUtilities.JsonDataSerializer..ctor()
   at Microsoft.VisualStudio.TestPlatform.CommunicationUtilities.JsonDataSerializer.get_Instance()
   at Microsoft.VisualStudio.TestPlatform.CommunicationUtilities.SocketCommunicationManager..ctor()
   at Microsoft.TestPlatform.VsTestConsole.TranslationLayer.VsTestConsoleRequestSender..ctor()
   at Microsoft.TestPlatform.VsTestConsole.TranslationLayer.VsTestConsoleWrapper..ctor(String vstestConsolePath, ConsoleParameters consoleParameters)
   at Stryker.Core.TestRunners.VsTest.VsTestRunner.PrepareVsTestConsole()

This is because Microsoft.TestPlatform.TranslationLayer is lying about its dependencies. Its JsonDataSerializer class depends on Newtonsoft.Json on .NET Standard 2.0 but Newtonsoft.Json is not declared as a dependency.
Q.E.D.

almost 2 years later. perhaps it is time to drop support for net452? support is ending in 2 months anyway

@nohwnd can u point to where/how u use data annotations?

Another issue with System.Text.Json is that it is not built-in in .NET Framework, and serializers are missing for common types that we use.

System.Text.Json is available as a NuGet package which multi-targets across .NET 6/7, .NET Standard, and .NET Framework 4.6.2, so that shouldn't be a problem?

@michael-hawker One of the reasons is that we get complaints about newtonsoft.json being required to test code that is not relying on it. We keep using very old version of the library to avoid upgrading the tested code, but ideally we would not use any such dependency that does not come with runtime.

Any progress on this?

perhaps u could alias newtonsoft https://github.com/getsentry/dotnet-assembly-alias/ and use the internal option
to avoid conflicts for consuming libs

Wouldn't using PrivateAssets="all" on the dependency help at least a bit for the autocompletion problems? Some users would need to explicitly reference Newtonsoft when upgrading, but that doesn't seem much compared to the benefits.

PrivateAssets only impacts whether or not the DLL gets copied to the output directory. It won't impact code-completion or the ability to reference symbols from the Newtonsoft namespace. There's an open issue here that requests a way to disable/remove implicit dependencies at the package-level. I recommend folks go there to upvote that issue. Which ever issue gets addressed first (that one or this one) is the one I will use!

Any status update?

There is no update, the reasons in https://github.com/microsoft/vstest/issues/2488#issuecomment-1155072927 are still true, and why we don't work on replacing newtonsoft.json.

Data annotation is a bad pattern because the tight coupling.

Can you be more specific why this is a bad practice? From what I see in our code we use a in-built attribute that is general purpose for data annotation, and newtonsoft.json is picking that up.  So to me that is the opposite of coupling.
While if we use fluent mapping we either need to let a "serializer" library know about all the shapes of all our messages. Or we need to add references to serializer to the dll that has our messages.

you could use compile flags that let you compile newtonsoft specific code

Yes that is a way forward, but at that point we can also just move to STJ fully and maintain it in a single way. But we don't because the newtonsoft signature is part of our public api and we don't have good opportunity to break when it works.

Data annotation is a bad pattern because the tight coupling.

Can you be more specific why this is a bad practice? From what I see in our code we use a in-built attribute that is general purpose for data annotation, and newtonsoft.json is picking that up. So to me that is the opposite of coupling.
While if we use fluent mapping we either need to let a "serializer" library know about all the shapes of all our messages. Or we need to add references to serializer to the dll that has our messages.

you could use compile flags that let you compile newtonsoft specific code

Yes that is a way forward, but at that point we can also just move to STJ fully and maintain it in a single way. But we don't because the newtonsoft signature is part of our public api and we don't have good opportunity to break when it works.

Datannotation are bad practice because it favours high coupling.
Problem with newtonsoft is that if you have a unit test project in your SLN and a less careful developer in the team uses resharper to auto reference newtonsoft dependency from mstest instead of correctly using System.Text.Json. If newtonsoft is not present in SLN resharper will not suggest using it but instead reference System.Text.Json. Right now Im seriously thinking of moving to nunit just to get rid of newtonsoft.

@AndersMalmgren

Right now Im seriously thinking of moving to nunit just to get rid of newtonsoft.

that wont help. nunit still requires  Microsoft.NET.Test.Sdk  to work. and that refs  Microsoft.TestPlatform.TestHost  which refs Newtonsoft.Json

Data annotation is a bad pattern because the tight coupling.

Can you be more specific why this is a bad practice? From what I see in our code we use a in-built attribute that is general purpose for data annotation, and newtonsoft.json is picking that up. So to me that is the opposite of coupling.
While if we use fluent mapping we either need to let a "serializer" library know about all the shapes of all our messages. Or we need to add references to serializer to the dll that has our messages.

you could use compile flags that let you compile newtonsoft specific code

Yes that is a way forward, but at that point we can also just move to STJ fully and maintain it in a single way. But we don't because the newtonsoft signature is part of our public api and we don't have good opportunity to break when it works.

Datannotation are bad practice because it favours high coupling. Problem with newtonsoft is that if you have a unit test project in your SLN and a less careful developer in the team uses resharper to auto reference newtonsoft dependency from mstest instead of correctly using System.Text.Json. If newtonsoft is not present in SLN resharper will not suggest using it but instead reference System.Text.Json.

This is what code reviews are for.

How is the debate about attributes & tight coupling helping to move us closer to a solution? If it's not, we should limit discussion to on-topic updates. I'd prefer to remain subscribed to replies so that I can keep up with meaningful conversation on the topic. Asking for status updates and debating attributes are not what I consider meaningful.
The issue is in open status and repository maintainers are aware of it. Let's patiently wait on a response, or if you're impatient, I'm sure they would appreciate a pull request.

Would an approach based on STJ contract customization be considered acceptable? (see https://github.com/dotnet/runtime/issues/29975#issuecomment-1187188015 )
(aka does it make sense for me to try and work on a PR in that direction?)

I know this will be unpopular but we would have to break the public API to do this, and we still would have to ship STJ for .NET Framework, so this would require significant changes to vstest. For this reasion it won't be implemented, we are focusing on adding new features to Testing.Platform instead, where we avoided this problem since the beginning. https://aka.ms/testingplatform


scipy.sparse.csr_matrix slicing: SIGSEGV

I catch SIGSEGV on https://github.com/scipy/scipy/blob/master/scipy/sparse/sparsetools/csr.h#L1168 while row-slicing a large (>1^31 nnz) csr_matrix. indptr.dtype is int64, however the template parameter is chosen wrong and I has type long which is 32-bit on all Linux-es => we get integer overflow.
Tested on 17.1 and 18.1. I would like to fix this myself - pls wait for my PR.

long is actually int64 so no problem with overflow, got overflow earlier in vstack()

Sorry, have to close this - a series of 100 negative values inside 9M rows in indptrs is related to some nasty bug in vstack(), or with my curved hands in general.


feat: Don't generate OpenAPI

https://github.com/getappmap/appmap-js/pull/1316 must be merged and published first - and the install action must be picking up the new appmap binaries.

As a test ran under this project and returned an error
Generating OpenAPI...

/home/runner/work/_actions/getappmap/archive-action/feat/dont-generate-openapi/dist/webpack:/archive-appmap-action/src/executeCommand.ts:65
                reject(new Error(`Command failed with code ${code}`));
^
Error: Command failed with code 134
    at ChildProcess.<anonymous> (/home/runner/work/_actions/getappmap/archive-action/feat/dont-generate-openapi/dist/webpack:/archive-appmap-action/src/executeCommand.ts:65:1)
    at ChildProcess.emit (node:events:513:28)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:293:12)

Will be provided by https://github.com/getappmap/archive-action/pull/35


build / release makeover

By request from @jeremyrickard
Highlights:

Tracks vendored code (it's a modest 36 MB or so)
All dev tasks (tests, lint checks, etc) occur within a consistent, containerized environment
Multi-stage image builds
DinD image builds (don't require the host's docker socket to be mounted into containers)

cc @vdice

Isn't the purpose of a lockfile to pin dependencies at particular versions which can rebuild the vendor tree? I'm not following why the vendor directory needs to be checked into source control.

I'd like a quick/memorable way to make bin/duffle on macOS or linux.

See doc changes. There is a way, but perhaps not adequately quick or memorable. I would be happy to add a few OS-specific convenience targets.

Isn't the purpose of a lockfile to pin dependencies at particular versions which can rebuild the vendor tree? I'm not following why the vendor directory needs to be checked into source control.

This is a classic debate and from what I have observed, dep users have mostly been erring on the side of tracking vendored code for some time now. You are absolutely correct that the .lock for enables a dependency tree to be rebuilt... as long as no one has deleted any library from github...
Beyond that, when vendor code isn't tracked, there's a need for some additional dependency resolution step (e.g. make bootstrap) to be run. For good measure, that should be run every time you check out a given branch, else you're potentially building with the wrong dependencies. So tracking vendored code has the advantage of being a little more foolproof in terms of the moment anyone clones the repo or checks out a branch, they already have the exact desired dependencies and are ready to build or test with no further adieu.

I'd like a quick/memorable way to make bin/duffle on macOS or linux.

See doc changes. There is a way, but perhaps not adequately quick or memorable. I would be happy to add a few OS-specific convenience targets.

That would be really helpful. I'd like something like make duffle or make darwin to produce bin/duffle (without any os/arch naming).

That would be really helpful. I'd like something like make duffle or make darwin to produce bin/duffle (without any os/arch naming).

@glyn I added a build-% target. You can build for your OS of choice with make build-<OS>. It assumes amd64 as the architecture.

without any os/arch naming

That makes me uneasy. The same build script that produces binaries on-demand for you also produces deployable artifacts in our pipeline. I think it's best if those artifacts are named unambiguously. This being said...
Elsewhere we discussed make install going away and I suggested creating a symlink instead. If you do so, there's no inconvenience to binaries having OS and architecture embedded in their name.
Does that seem ok?

That would be really helpful. I'd like something like make duffle or make darwin to produce bin/duffle (without any os/arch naming).

@glyn I added a build-% target. You can build for your OS of choice with make build-<OS>. It assumes amd64 as the architecture.

without any os/arch naming

That makes me uneasy. The same build script that produces binaries on-demand for you also produces deployable artifacts in our pipeline. I think it's best if those artifacts are named unambiguously. This being said...
Elsewhere we discussed make install going away and I suggested creating a symlink instead. If you do so, there's no inconvenience to binaries having OS and architecture embedded in their name.
Does that seem ok?

Yes. I can use a symlink duffle -> duffle-darwin-amd64.

I re-tested and SKIP_DOCKER=true make build-darwin works as expected as does make build-darwin. I wonder if you would mind documenting this in the developer's guide, for posterity?

Looks great now. Ideally, I'd like make duffle-darwin et al. described in the developer's guide, for posterity.

Yes. I will add that.

Just tested on Windows:
PS C:\projects\go\src\github.com\deislabs\duffle> $env:SKIP_DOCKER
true
PS C:\projects\go\src\github.com\deislabs\duffle> make -f .\Makefile build
bash -c "LDFLAGS=\"-w -s -X github.com/deislabs/duffle/pkg/version.Version=de5062b\" scripts/build.sh"
building Windows_NT-amd64
cmd/go: unsupported GOOS/GOARCH pair Windows_NT/amd64
make: *** [build-all-bins] Error 2

Of course, this assumes WSL is configured on the machine, which is not default.

@radu lemme look at that. This is really odd. I'm not sure how Windows_NT got set as.the OS here. Do you have an existing env var by that name?

Ok. Apparently the OS environment variable (which I am using) is pre-defined as Windows_NT on Windows (but not in WSL). So, if not executing within a container, in a place where I expected OS to either be a valid OS for Go cross-compiling or be undefined, it is neither of those things. I can fix this.

@radu-matei the latest issue you reported should be fixed by the latest commit.

This currently has 4 LGTMs by core maintainers, so I am merging.


All outcomes are duplicated for the dec 21 election market

Describe the bug
On the disputing page, for the "Who will win the 2020 U.S. Presidential election?" market 0x1cafEd8337Fb45a891b30Bd5B3C486D5FfF7A9DA, all the outcomes show up twice, and all of them say "make tentative winner" and the button says "dispute tentative winner. See screenshot.
To Reproduce
Steps to reproduce the behavior:

Go to https://bafybeiebluevb7w6qmwp5vnhwgzloyxg44vqbuzfoq4wcqyomco6oslxu4.ipfs.dweb.link
Click on "disputing"
Click on "2 more outcomes"
See error

Expected behavior
Each outcome should show up once, only one should say it's the tentative winner, with the correct button text.
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser: Firefox
Version: 84.0.1

And this looks to be just on this one market?
@justinbarry can you see if this is an issue with the data returned from the graph?

And this looks to be just on this one market?
@justinbarry can you see if this is an issue with the data returned from the graph?

there are currently two on that page, this is the only one that has this
problem.
On Fri, Jan 1, 2021 at 10:37 AM Paul Gebheim<EMAIL_ADDRESS>wrote:

And this looks to be just on this one market?
@justinbarry https://github.com/justinbarry can you see if this is an
issue with the data returned from the graph?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/AugurProject/augur/issues/9944#issuecomment-753362143,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPATJF3PH3I76G35PJBJ3SXYI5NANCNFSM4VQMWMRQ
.

there are currently two on that page, this is the only one that has this
problem.
On Fri, Jan 1, 2021 at 10:37 AM Paul Gebheim<EMAIL_ADDRESS>wrote:

And this looks to be just on this one market?
@justinbarry https://github.com/justinbarry can you see if this is an
issue with the data returned from the graph?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/AugurProject/augur/issues/9944#issuecomment-753362143,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPATJF3PH3I76G35PJBJ3SXYI5NANCNFSM4VQMWMRQ
.


GOMAXPROCS

I'm not closely following the event, but one detail that even the most experienced Go developers often forget in containerized environments is to correctly set the GOMAXPROCS, as mentioned in the article below. In certain situations, it can harm performance.
https://www.ardanlabs.com/blog/2024/02/kubernetes-cpu-limits-go.html

Obrigado pela contribuição.


[eCBIS] Update the login page

Name of feature to enhance

Currently, we have the login page as the default login page with FHIR core logos.
We need to update this page to have the eCBIS branding

Description of feature

[ ] Update the login logo to use the eCBIS logos
[ ] Update the app naming to eCBIS

are we going to use the app config to have this loaded from a fhir-based remote config? or is there another strategy?

are we going to use the app config to have this loaded from a fhir-based remote config? or is there another strategy?

We'll use the application configurations.

@dubdabasoduba The logo on the app can be found here - linked to this issue with details of the app login page that currently exists. The idea is to keep this the same - maybe with a change of name? @rowo


Make "Get a Sprig" UX more consistent w/ rest of app

Currently the "Get A Sprig" page uses the same green background and different style as the main landing page.  We want to move away from the different style for all pages except the main page.  So a few things need to happen here:

Make the top nav bar exactly the same as in the gallery.
Do away with the green background image.  Change color scheme of whole page to be more cohesive/aligned w/ rest of app.
Wrap the top copy (Share your game, etc, the whole paragrph there) in some eye-catching nice UI element.

fixed in #2068


CtrlH works like backspace on some terminal emulators

Description of the problem or steps to reproduce
CtrlH is equivalent to backspace on some terminal emulators.
Expected behavior: Pressing CtrlH will execute whatever action is bound to it.
Actual behavior: Pressing CtrlH executes a backspace.
This is a "feature" of many terminal emulators, where hitting Ctrl+H sends the backspace keycode. This should be possible to fix, because other terminal-based text editors (vim for example) are able to tell the difference.
Specifications
Commit hash: d41f0bb
OS: Arch Linux
Terminal: GNOME terminal

CtrlH is still bound by default to backspace, but you can now actually rebind it if you have a terminal that differentiates between CtrlH and Backspace.


Update Supply to use google-api-client 0.12.0 or above

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context

Description

I signed it!

I signed it!


fix: multiple share grants

closes #1459. cleans up some of the logic when enabling multiple grants . Previously if you had two grants that managed the same resources, then one would overwrite the other during subsequent apply.

Test Plan

[ ] acceptance tests

References

@sfc-gh-swinkler there is a bug introduced with version 0.56,3 that is causing any plans with grants to have a perpetual diff

This line was removed to check privs
} else if privileges.hasString(priv) && (existingRoles.Contains(roleName) || existingRoles.Len() == 0) && multipleGrantFeatureFlag {


Add cummax, cummin and cumsum in Series

This PR adds cummax, cummin and cumsum in Series by deduplicating existing logic.

Codecov Report

Merging #534 into master will decrease coverage by 3.03%.
The diff coverage is 96.55%.

@@            Coverage Diff             @@
##           master     #534      +/-   ##
==========================================
- Coverage   93.85%   90.81%   -3.04%
==========================================
  Files          31       31
  Lines        4263     4268       +5
==========================================
- Hits         4001     3876     -125
- Misses        262      392     +130

Impacted Files
Coverage Δ

databricks/koalas/missing/series.py
100% <ø> (ø)
:arrow_up:

databricks/koalas/generic.py
94.44% <100%> (+0.24%)
:arrow_up:

databricks/koalas/frame.py
91.9% <100%> (-2.41%)
:arrow_down:

databricks/koalas/series.py
92.9% <90.9%> (-0.3%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
23.14% <0%> (-74.08%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
76.31% <0%> (-7.9%)
:arrow_down:

databricks/conftest.py
90.69% <0%> (-6.98%)
:arrow_down:

databricks/koalas/utils.py
97.29% <0%> (-2.71%)
:arrow_down:

... and 4 more

Continue to review full report at Codecov.

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

This is basiaclly deduplication. I am merging this.


Steam power mod interaction unusable

When I try to create a steam power setup in my instance, steam will not enter the turbine/radiator, and the boiler will eventually stop outputting steam into the pipes. I have not tested this bug with other fluids, but I believe that this is unique to the steam power blocks. With ONLY forge, crossroads, and dependencies in an instance, newly placed steam turbines and radiators work just fine. Said turbines continue to function when other mods are added to the instance, without disturbing the steam setup. I apologize for not being able to say which mods are responsible for this bug, but I found out that already placed turbines would continue to function while trying to discover that.
I am using:
Minecraft 1.10.2
Forge <IP_ADDRESS>5
Crossroads 1.3.2
full mods list: mods.txt

When you say "the boiler will eventually stop outputting steam into the pipes", do you mean the pipe is filling up, or that that the boiler stops producing steam? If the boiler stops producing steam, then you are likely using normal water instead of distilled water, thus clogging the boiler with salt. Also, make sure you are trying to insert into the correct sides. Turbines and radiators only accept steam from the bottom. Nothing in your mod list should be interfering with this at all.

When I say the boiler will eventually stop outputting steam into the pipes, I mean that the pipe stays mostly empty, and the boiler fills up with steam. This occurs even with distilled water, so it's not the salt mechanic, and the pipes are feeding into the bottom. The same setup that works with only crossroads installed, will NOT work if I build it in an instance that has all of my mods installed.

Could you post a picture of the exact setup you are using so I can try to reproduce the bug with those mods?

Yes, here is a picture a setup that I believe should work. If there is something wrong with it, let me know. (The red output of the turbine is closest to the camera)

I've discovered this is an incompatibility with Mekanism. I'll keep looking into this.

Thanks. Do you want me to make an issue with the Mekanism github?

No, I need to keep looking into this to find out exactly what the cause is.

I found the problem, and it's on Crossroad's end. It's caused by Mekanism and Crossroads both adding steam. I'll fix it immediately.


False positive?

The executable isn't code signed.
If you really want to use this project badly, just compile from source.


DOCS: typos, rewording, updated links.

How very eloquent!

Thanks @willmorgan. I aim to please ;) Though I suppose what is more important is whether or not the end result is a bit more readable!

On line 22 you updated the version to 3.2 but reference master as being 3.2 also, when it's actually 4.0

OK...will fix :)

:blush:


Bar chart sparklines

Resolves https://github.com/studentinsights/studentinsights/issues/593.
First pass. Design feedback wanted!

 How would it look with a solid line on the bottom so those rectangles are a bit more grounded?

@awalker4: I agree, here's how it looks now.

@erose What are the dividing units on the X-axis?

@alexsoble: The two big vertical lines? Those are starts of school years. (August 15th 2015, August 15th 2014).
I don't think that's clear. Should we try to indicate it better, or just lose them? (since this is just a sparkline not a full graph).

@erose Nice!  This seems like a great improvement to me, and the suggestions of clarifying the time sound 👍 too.

@erose I think some way to indicate that is important. Maybe a mini-legend at the bottom?

Hey @erose. I think this is excellent work, I apologize if my suggestions/comments about the X-axis have slowed us down from merging it. Why don't we merge it in now (solid improvement from the line graphs) and we can think more about the X-axis labeling as a next step?

@alexsoble: Dude, that's not on you! I agree, (merged), I'll think about it more + iterate more when I see it in production.

Pushed to production! These look nice!
On 28 August 2016 at 20:18, Eli Rose<EMAIL_ADDRESS>wrote:

@alexsoble https://github.com/alexsoble: Dude, that's not on you! I
agree, (merged), I'll think about it more + iterate more when I see it in
production.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/studentinsights/studentinsights/pull/602#issuecomment-243008851,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADD5He2Xpakai9tKQSgwAXstF8cr1RU5ks5qkiVggaJpZM4JBjft
.


Empty icon is shown in GNOME Shell Favorites panel and Window list

When I launch Kube Forwarder under GNOME Shell
Then the Kube Forwarder icon is not visible in the Favorites dock
And an empty space is shown instead

The same empty icon is shown when I bring up the window switcher via Alt + Tab.
The icon is showing when I search for the application in the Activities screen however:

I used the AppImage package from https://github.com/pixel-point/kube-forwarder/releases/download/v1.5.1/kube-forwarder.AppImage
When I first launched the application I was asked if I want to integrate it via the system and I clicked Yes.
I was able to reproduce the issue on my daily-driver Manjaro installation and on a fresh install of Ubuntu 20.04 inside a VM.

In my case, I have icons visible in both the app menu and the favorites pane. But when it's not favorited, I see the default gear icon on the launch bar while running. Digging into the issue, I realized the StartupWMClass in the .desktop file has the wrong WM_Class value of Kube Forwarder while using xprop I get kube-forwarder as the correct value. Changing this fixed the issue for me...


[Improvement]: Slow LS functionalities caused by resolving dependencies in PackageContext.java

Description
The getResolution method in PackageContext.java which resolves all the dependencies, results in high CPU usage. We tested this on a low performance machine and found no significant memory consumption, instead, the CPU usage spiked. As a result of this issue, noticeable slowness in LS functionalities can be observed.
The specs of the laptop were as follows,
Windows 10
i5-10210U CPU
RAM - 16GB
Describe your problem(s)
No response
Describe your solution(s)
No response
Related area
-> Compilation
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

Related release issue : https://github.com/ballerina-platform/ballerina-release/issues/2629


Fixing Directive Retry Scenario

Previously an exception was thrown was for unreachable/invalid DNS and no retry was attempted on the IP address (for RVblob ).
Fixing this bug by adding a proper exception handling structure to retry with IP address in case of invalid DNS.
Signed-off-by: Davis Benny<EMAIL_ADDRESS>

When we say "Multi RV Support",  I have the following question:

Exactly how with how many RV has it been tested with?
Is there an upper limit to how many RVs we can specify? If so please update the documentation for the same.
Does the code handle the scenario when the number of RVs specified exceed the max threshold?

When we say "Multi RV Support", I have the following question:

Exactly how with how many RV has it been tested with?
-- Tested with 2,3 & 4 RV directives. It works with n RV directives.
Is there an upper limit to how many RVs we can specify? If so please update the documentation for the same.
-- There is no upper limit on the number of RV's.
Does the code handle the scenario when the number of RVs specified exceed the max threshold?
-- There is no upper limit as the spec;

When we say "Multi RV Support", I have the following question:

Exactly how with how many RV has it been tested with?
-- Tested with 2,3 & 4 RV directives. It works with n RV directives.
Is there an upper limit to how many RVs we can specify? If so please update the documentation for the same.
-- There is no upper limit on the number of RV's.
Does the code handle the scenario when the number of RVs specified exceed the max threshold?
-- There is no upper limit as the spec;

Ok. Thanks


Article count design test round 2

What does this change?
Adds a second round of article count design tests. Additionally, we want to track how many users opt out of article counts. To do this the platform now passes in a submitComponentEvent function as a prop, much like for the banners. This requires https://github.com/guardian/dotcom-rendering/pull/2747 to be merged. I was thinking we wouldn't bother implementing this in frontend as so few (if any?) articles are rendered by frontend.
How to test
UK/AUS

force-epic=EpicSeparateArticleCountTestR2__UK_AUS:control
force-epic=EpicSeparateArticleCountTestR2__UK_AUS:above
force-epic=EpicSeparateArticleCountTestR2__UK_AUS:inline

EU/ROW

force-epic=EpicSeparateArticleCountTestR2__EU_ROW:control
force-epic=EpicSeparateArticleCountTestR2__EU_ROW:above
force-epic=EpicSeparateArticleCountTestR2__EU_ROW:inline

Images
control

above

inline

I was thinking we wouldn't bother implementing this in frontend as so few (if any?) articles are rendered by frontend.

I just asked dotcom - currently, of articles which can be displayed by DCR, we're choosing to show the frontend version to 10% of the audience. But this is likely to change this week.
So I agree with not making the frontend change. But we must track view events for the article count, so that we can have an accurate click rate (which excludes frontend epic views).

Yeah agreed 👍


Use Buffer from buffer

Unfortunately I couldn't figure out a TS configuration that wouldn't include Node types. It seems to me that @types/bn.js reference them explicitly in https://github.com/DefinitelyTyped/DefinitelyTyped/blob/5919cb6bae26f700ea7f66ff39f46ea4ad2e00e3/types/bn.js/index.d.ts#L8 and… they just get pulled in.

@sjchmiela could we use pnpms patch to remove that line? Maybe that would make it easier to maintain?

Good idea! I haven't fully noticed pnpm supports patches now, that was my main complaint about pnpm haha!

removed node references from bn.js and tape types, added Buffer to bn.js
configured TS to:

emit esnext code following glow-js example
exclude test from typechecking
not import any @types/ automatically (lets us have both @types/node which make test work and not include them when compiling src)


hmmFit & generativeClassifierFit chol->matrix is not positive definite

From<EMAIL_ADDRESS>on March 14, 2011 10:03:33
What steps will reproduce the problem? 1.Fitting a HMM on observations and setting the nr of States higher than 2
2. This is a Gauss or GaussMixTied
3. It works normal with 1xT(i)  sequences What is the expected output? What do you see instead? An error is generated at the next line with  generativeClassifierFit. Chol seems to be the problem in gaussLogprob at line 52 What version / revision of the product are you using? On what operating system? Edition pmtk3 28feb 2011, Matlab R2007b , Ubuntu Please provide any additional information below. File is based on http://pmtk3.googlecode.com/svn-history/r2517/trunk/docs/tutorial/html/tutGenClassif.html
Attachment: ARenPCA2.m
Original issue: http://code.google.com/p/pmtk3/issues/detail?id=20

hi,
i'm also getting the same error from the PTMK3 toolbox on Matlab 2014a on 64 bit linux.
'Error using chol
Matrix must be positive definite.'
any help would be appreciated!
f

For me, what helped was to change the matrix with initial probabilities 'transmatrix0' with another probabilities.
This version of PMTK has a tridiagonal (as I remember) transmatrix0 but if I use another matrix it does not get me the error.
For example, by using a function mk_stochastic of the previos version HMMall it works well. However, another option is to make one matrix by hand. For example if you are using 5 states.
=[.2 .2 .2 .2 .2;.2 .2 .2 .2 .2;.2 .2 .2 .2 .2;.2 .2 .2 .2 .2;.2 .2 .2 .2 .2]  %with all rows summing one.


GCE Flags Error causing unit tests to fail

I pulled the latest commits from master, and currently I'm getting 2 unit tests errors which is blocking my commits since I have the pre-commit hook setup.
Here's a link to stack traces:
https://gist.github.com/meteorfox/64934edbc07733a0ba53

According to git blame
(pkb)carl6796@t450s:~/programming/github.com/GoogleCloudPlatform/PerfKitBenchmarker$ git blame -L110,115 /home/carl6796/programming/github.com/GoogleCloudPlatform/PerfKitBenchmarker/perfkitbenchmarker/providers/gcp/util.py
85a34c9d (Skyler Schneider 2015-11-17 21:40:25 -0800 110)     self.flags['quiet'] = True
85a34c9d (Skyler Schneider 2015-11-17 21:40:25 -0800 111)     if resource.project is not None:
85a34c9d (Skyler Schneider 2015-11-17 21:40:25 -0800 112)       self.flags['project'] = resource.project
85a34c9d (Skyler Schneider 2015-11-17 21:40:25 -0800 113)     if hasattr(resource, 'zone'):
85a34c9d (Skyler Schneider 2015-11-17 21:40:25 -0800 114)       self.flags['zone'] = resource.zone
85a34c9d (Skyler Schneider 2015-11-17 21:40:25 -0800 115)     self.additional_flags.extend(FLAGS.additional_gcloud_flags)

It looks it comes from one of @skschneider commits, but it could be a side effect of another change.
Thanks ahead of time!

This is likely because #695 was merged between the time I opened and submitted #675, and the tests weren't rerun. I will work on a fix.

#724 has been submitted. tox and hooks/check-everything are clean on master.

Thanks!


Allow popper modfiers to be updated

Currently if a consumer updates the modifiers object prop, unless another prop also changes, the new modifiers are not applied. For example, if the passed modifier flip.enabled were to change from true to false, the popper doesn't pick up on that fact and continues allowing the popover contents to flip.
This PR allows for the Popper component to construct a new popper instance with the new modifiers if and only if the modifiers object changes (note: this is a reference comparison and not a value comparison).
I'm not that familiar with Popper.JS so if there is a more efficient way to make sure modifier changes get propagated to the underlying popover, please let me know!
As it stands, this could cause some unnecessary popper instance creation/destruction if the consumer isn't properly memoizing the modifiers object they pass. This could be partially solved by doing something like shallow comparing each modifier option object one by one instead of a top level reference comparison. Alternatively, if there is a more performant way to imperatively update an existing popper instance, that may be better to use to avoid as much churn. In fact, it would be even possible to combine both of these solutions as well.

I wonder if we could add a check in developement mode to warn in case the modifiers object has the same content but changed identity?

done!

Cool thanks! I'm gonna test it out and merge it, it may take a few days unfortunately so I say sorry in advance!


Update post-ti-indicator.md

added Generatealert parameter

@JohanFreelancer9 please copy review

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/7305

@mjcaparas Please merge this PR. Thanks


Support deploying Edge stacks on Edge Docker standalone endpoints

At the moment, Edge stacks for Docker only works with Swarm endpoints.
We need to enhance it to support Edge Docker standalone endpoints, this will require the use of the docker-compose binary on the agent (related to #3750).

shouldn't this be open on the agent repo?

There should be an equivalent ticket in the agent, yes. I'll open it.

There should be an equivalent ticket in the agent, yes. I'll open it.


Bump log4j to 2.17.1

Is your feature request related to a problem? Please describe.
Coming from https://github.com/opensearch-project/opensearch-build/issues/1417, Log4j 2.17.1 is available with CVE-2021-44832.
Describe the solution you'd like
Bump log4j to 2.17.1.

https://github.com/opensearch-project/performance-analyzer-rca/pull/113


reverselist

We reverse the list by changing the links between nodes.

Great work @IshanBk
Keep it up


What are the settings that are valid here?

Same goes for TabletSettings and PhoneSettings.
Is any XML valid here? What are the source location and control settings referenced in the description? Please link to or create additional documentation.

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

ID: 7590a7b5-a498-d258-3258-0283df814ad4
Version Independent ID: 1dd3170b-63ce-9b28-bb88-5b6370802a41
Content: DesktopSettings element in the manifest file - Office Add-ins
Content Source: docs/reference/manifest/desktopsettings.md
Product: non-product-specific
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

@exextoc Can you provide the missing element information please?

@DennisGaida thank you for reporting this issue.

The Form element supports three elements in the following order. Each of those elements can have a SourceLocation element which must be a valid, fully qualified HTTPS address and TabletSettings and DesktopSettings also support a RequestedHeight element that can have a minimum height of 240px and a maximum height of 800px. These three extension points will appear in older versions of OWA and Outlook 2013 for Windows. The DesktopSettings extension point not supported in OWA for Office 365, or Desktop Outlook. TabletSettings and PhoneSettings will appear in Tablet and Mobile versions of OWA but is deprecated and will be removed from OWA in Office 365 soon (It will still be available in older versions of Exchange On-premise). TabletSettings and PhoneSettings are not supported in Outlook Mobile for Android or iOS however we support add-ins in these clients using the following extension points
<Form xsi:type="ItemRead">
          <DesktopSettings>
              <SourceLocation DefaultValue="https://website.html" />
              <RequestedHeight>360</RequestedHeight>
          </DesktopSettings>
          <TabletSettings>
              <SourceLocation DefaultValue="https://website.html" />
              <RequestedHeight>360</RequestedHeight>
          </TabletSettings>
          <PhoneSettings>
              <SourceLocation DefaultValue="https://website.html" />
          </PhoneSettings>
      </Form>


[ROCm] Add run_xla script to fecilitate running XLA unit tests

I'd suggest at least trying to add such scripts there first.

I'd suggest at least trying to add such scripts there first.

Pushed https://github.com/openxla/xla/pull/12948 Let's see what they say


Implementing hand_shake phase as a state machine

No issue to close
Description
introduce hand_shake rust module to handle transitions of the hand shake with Startup messages from the client
Is it a feature that change user experience?
no
Client output
no

Related to #377