Incorrect vendor prefix identification

Describe the bug
Error Separate different vendor prefixes into their own rules. is showing on SASS that has no vendor prefixes:
.menu-item {

	&.menu-item-has-children::after,
	&.menu-item-has-grandchildren::after {
		content: '';
		position: absolute;
		top: 24px;
		right: 30px;
		width: 18px;
		height: 18px;
		background-size: contain;
	}

}

To Reproduce
Steps to reproduce the behavior:

Add the SASS code above into a sass/scss file
Run Stylelint with use-defensive-css plugin
Notice errors

Expected behavior
No vendor error(s).
Desktop (please complete the following information):

OS: macOS
Browser: NA
Version: Sonoma 14.2

Additional context
package.json
{
  "devDependencies": {
    "stylelint": "^15.10.2",
    "stylelint-config-standard": "^34.0.0",
    "stylelint-config-standard-scss": "^11.1.0",
    "stylelint-high-performance-animation": "^1.9.0",
    "stylelint-order": "^6.0.3",
    "stylelint-plugin-defensive-css": "^0.9.1",
    "stylelint-stylistic": "^0.4.3"
  }

Thank you for reporting this. I will dig into it as soon as I can 🤘

@crstauf Could you share your full package file? I want to set up a project in the same way as yours to make sure I am testing correctly.

@yuschick The package.json I shared above is the pertinent info (only other thing is a custom command).

I'm interested in seeing how you're bringing in SCSS. I don't work with it often, so I'd like to see your project setup there, so I can make sure I am testing correctly.

Oh. I have Dart Sass installed globally. That shouldn't impact Stylelint's review of files though, correct?

I'm not sure off hand, but that at least gives me a place to start. From what it seems, I may need to write tests specific to SCSS. This is a bit new for me, so bare with me as I try to figure out what is needed.

@crstauf Okay. I can recreate the error. Seems like I just need to update the RegExp I am using. Will get an update out soon.

This has been fixed and deployed in v0.10.3.
https://github.com/yuschick/stylelint-plugin-defensive-css/pull/27

Will check it out ASAP to confirm.

@yuschick I ran npm update stylelint-plugin-defensive-css, and now I get some output that is not correct:

package.json
{
  "devDependencies": {
    "stylelint": "^15.10.2",
    "stylelint-config-standard": "^34.0.0",
    "stylelint-config-standard-scss": "^11.1.0",
    "stylelint-high-performance-animation": "^1.9.0",
    "stylelint-order": "^6.0.3",
    "stylelint-plugin-defensive-css": "0.10.3",
    "stylelint-stylistic": "^0.4.5"
  }
}

@crstauf sorry for that! I have an idea and will update today.

@crstauf Thanks for catching this and reporting it so quickly. I have published 0.10.4 which will no longer have those logs.

Thanks, @yuschick! Will try it in the morning.

Ugh, I completely forgot about this until now. Will test and report back latest by Monday.

Managed to get to my desk tonight. Looks like it requires Stylelint 16, which not all my dependencies support yet, so I'll come back 'round once that happens.

Updated to Stylelint 16 and v1.0.0 today: works well. Thanks!


Jesse: Watch oberhausen-umzugsunternehmen.de

Jesse requests the watch of the watch_keyword oberhausen-umzugsunternehmen\.de. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by cigien in Charcoal HQ


[11.x] Fix prompting for missing array arguments on artisan command

This PR fixes the bug below:
When an Artisan Command has array arguments configuration, like:
protected $signature = 'app:foo {args*}';

promptForMissingArgumentsUsing() doesn't work,
because $input->getArgument($argument->getName()) returns empty array, and judgement in promptForMissingArguments() method of Illuminate\Console\Concerns\PromptsForMissingInput uses is_null(), which should be replaced with empty() .
empty() method also covers string argument, like:
protected $signature = 'app:foo {arg}';

I'm leaving in draft until @jessarcher reviews it.

I understand.

OK.That's right.
Thank you for your suggestion.

Personally I think this needs more thought. If you provide the argument on the CLI, you would get the argument as an array using $this->argument. With this, the argument would be a string. That feels inconsistent.

Personally I think this needs more thought. If you provide the argument on the CLI, you would get the argument as an array using $this->argument. With this, the argument would be a string. That feels inconsistent.

Good catch. I've updated it to wrap the response in an array. There isn't a good prompt for entering multiple arbitrary values, so with the default behaviour, you'd always get an array of one item.
It's possible for someone to provide their own prompt callback that can return multiple values using whatever implementation they like (multiselect of known options, multisearch against the DB, text prompts in a loop that continues until an empty response, or some sort of space/comma/newline separated answer).

Thanks!


.cs files clicked in Godot that launch Visual Studio fail to load the project.

Godot version
4.1.1
System information
Windows 10
Issue description
Visual Studio opens but fails to load the project correctly. Attempts to right click and reload fail to load project.
Steps to reproduce
Unity Mover. Download new Godot 4.1.1 .net version. Download the .net 6.0 sdk. Visual Studio Installer Desktop environment and development with C++ modules. Project Settings > Tools > C# > Create C# Solution. Create a C# script.
When I double click the script within Godot, Visual Studio launches but the project says unloaded. If I try to right click, Reload Project it fails with an error about SDK resolver fails, can not find System.Runtime Version.... , can not find xxx.csproj file.
What I have to do is open the file explorer, and double click either the .csproj or the .sln and Visual Studio opens and things appear to be working correctly.

This is the IDE after just opening the .csproj or .sln from file explorer.

Minimal reproduction project
New C# installation as listed in steps to reproduce.

I tried to reproduce with the 4.1.1 stable.mono.official but for me it opens just fine the file and i didn't have to do generate a C# project, seems it generated by itself as soon as i created a new script.

I created a new project
right click in the file explorer section on the res:// tree item => create_new => script
select C# and create
it generates the CS file and the .csproj and the .sln

Then tested from the script tab

from main editor view click on top "script" button tab next to 3D view
from the script files browsing view click on file => new script
select C# and create
it generates the CS file and the .csproj and the .sln

In case I do twice one of these creation processes or the first then the second so ending up with several scripts, it still manages to update the .csproj and .sln and also allows me to open the script directly in visual studio when i double clic a script to open

Thank you for your testing. I had the same experience with the .csproj and .sln files being created automatically.
I couldn't find anything online and it took me a bit so I thought I was being more thorough including the C#>Create C# Solution command... I have also definitely had the Visual Studio option selected as Godot did open Visual Studio.
The files created appear correct, the project will build. But what I do notice is that in the images I attached, the paths and the name displays are capital, but when it is opened directly and things work, the paths are normal casing. Which I think normally wouldn't matter... For some reason, whatever data is being sent to Visual Studio is off for some reason... The solution is in quotes 'SECONDTESTPROJECT' vs 'SecondTestProject' assuming visual studio just inserts what it is given...
Just to include, I now have code completion, I can run in debug mode. Everything appears to work. As long as I open the project from the file explorer.

Could you show the content of the .csproj and .sln that didn't work/generated an error with all uppercase text from your first post?

Below is what I was writing... I had changed modules, uninstalled old versions, thought I ran repair. But anyway, I had a minor version update for VS 2022. I just did the update, no other changes, and now Godot opens the .cs fine...
I have multiple projects for testing and they all do it.  Here you go:
.sln

net6.0
true

.csproj
Microsoft Visual Studio Solution File, Format Version 12.00
Visual Studio 2012
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "SecondTestProject", "SecondTestProject.csproj", "{AB48087E-154A-4B23-89AF-5B72D9A91BC1}"
EndProject
Global
GlobalSection(SolutionConfigurationPlatforms) = preSolution
Debug|Any CPU = Debug|Any CPU
ExportDebug|Any CPU = ExportDebug|Any CPU
ExportRelease|Any CPU = ExportRelease|Any CPU
EndGlobalSection
GlobalSection(ProjectConfigurationPlatforms) = postSolution
{AB48087E-154A-4B23-89AF-5B72D9A91BC1}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
{AB48087E-154A-4B23-89AF-5B72D9A91BC1}.Debug|Any CPU.Build.0 = Debug|Any CPU
{AB48087E-154A-4B23-89AF-5B72D9A91BC1}.ExportDebug|Any CPU.ActiveCfg = ExportDebug|Any CPU
{AB48087E-154A-4B23-89AF-5B72D9A91BC1}.ExportDebug|Any CPU.Build.0 = ExportDebug|Any CPU
{AB48087E-154A-4B23-89AF-5B72D9A91BC1}.ExportRelease|Any CPU.ActiveCfg = ExportRelease|Any CPU
{AB48087E-154A-4B23-89AF-5B72D9A91BC1}.ExportRelease|Any CPU.Build.0 = ExportRelease|Any CPU
EndGlobalSection
EndGlobal

Any update on this? I still have this problem


Wrong type is generated by typescript

🔎 Search Terms
generated type
🕗 Version & Regression Information

This changed between versions ______ and _______
This changed in commit or PR _______
This is the behavior in every version I tried, and I reviewed the FAQ for entries about _________
I was unable to test this on prior versions because _______

⏯ Playground Link
https://www.typescriptlang.org/play/?moduleResolution=99&target=99&jsx=0&module=199#code/KYDwDg9gTgLgBAMwK4DsDGMCWEV0wWwApgAbALjhSXwCNgo4AfSpEkgSjgG8AoOPBHGIk4AXlEs2nXv35ocAZ3gBDFArHc+s-qExKFFBMpILgAGi38AvgG5LcKMBhIouVQrvWt8tSrUAmDRlZADdjJGAKAHYLbV19ChgoCNi4Wy1HZ1c4d387Kx4gA
💻 Code
declare function get_el(): number | null;
var el=get_el()
export const im=function(){
  if (el==null){
    const ans= {
      exists:false
    }
    return ans
  }
  const ans2={
    value:7,
    exists:true,

  }
  return ans2
}()

🙁 Actual behavior
const im: {
value: number;
exists: boolean;
} | {
exists: boolean;
}
🙂 Expected behavior
export declare const im: {
exists: boolean;
};
Additional information about the issue
see description in https://stackoverflow.com/questions/79017053/wrong-type-is-generated-by-typescript/79017129

oops the Actual behavior and Expected behavior are reversed

This is working as intended and is due to "subtype reduction". If you don't want this you can add an explicit type annotation.

@MartinJohns @RyanCavanaugh I get it that it works as intended and that the user can "fix it" by changing the code, but the result is objectively wrong/lossy.

the result is objectively wrong/lossy and there's room for improvement.

It's not objectively wrong, and the proposed behavior is actually not sound.

In plain English: the function may return an object with a key named value
In TypeScript: no
I'm sure I lack the understanding of why you would say that, but to me TypeScript currently does not match the reality.

Ok, maybe not "wrong" but certainly lossy. Having a return type {x?: unknown} would also be "correct" according to your definition, but we'd lose two keys instead of one.
I don’t think anyone looking at that code won't be surprised of the current inferred type.


Memory exhaustion running lifetimes package for over 1 million transactions

I was fiddling with the lifetimes package and tried to look into why it took so long when the number of transactions increases significantly. I could not get 1 million transactions to work, so I started to use less number of transactions. (around 100k)
And when I try to do cProfile over the method calls, this is what I get.

I realised that if I ran the BetaGeoFitter.fit and BetaGeoFitter.fit, the method took 50 seconds to compute, which is a big issue if the number of transactions get high enough.
Is there anyway we can improve the performance of minimize() (line 308) in utils.py?
P.S. I already tried to reduce maxiter from 2000 to 400. It helps a little but it does not help much in the context of 1 million transactions.
Thanks loads!

Thanks for profiling this - a few thoughts:

I've found a series bottleneck is in the transformation from transactions -> summary data (one row per customer) using the built in summary_data_from_transactions_data - are you using this? If so, I would avoid this and try to do the transformation in SQL database or a Spark job instead.

We at Shopify treat shops with large number of customers by subsampling ~50k. We noted that the time-perf increased linearly with number of customers, but parameters became stable "enough" beyond that.

You can also try setting iterative_fitting=1 in the call to fit, but this may run the risk of bad parameters being found (though less likely with more data).

We are fitting BG/NBD model for about 15mln customers with pandas DataFrame. Fitting from scratch takes about 45 minutes with a Nelder-Mead algorithm (which is default) and about 5-6 from a warm start (with setting initial_params from first fit). SLSQP is used as minimizing algorithm for a warm start, because for Nelder-Mead you need to also pass initial simplex.

Due to the size of our data, we also transform the data from transaction to customer using SQL. FWIW, we found that fitting time increases by ~6 minutes per 1 million additional customers.

I think v0.11.0 will sign. improve performance for large datasets. Let me run some perf tests.

For ~50k rows in BetaGeoFitter, it's about 3x faster in v0.11.0.
For 236k rows, it's about 5x faster in v0.11.0.
And v0.11.0 has by default smaller tol parameter. The major speedup is being about to drop Neader-Mead and instead use BFGS.


Documentation errors and small change to the custom distribution

Smalls changes to the custom distribution to prevent an edge case where the random number generated is equal to or bigger than the highest value in the cumulative probability vector.
I also find errors in the documentation. In a previous Pull request, I forgot to change the old parameter names in the text below the code example.
Bonne année!

Ci is crashing for something that's not your fault. I will merge this :)


Add star-rail notes to cli

Please confirm these are actual names of the star-rail thingies.

genshin didn't have it and it seemed like a pain, I'll add it


Allow choosing pwsh on windows (or any other shell by path if possible)

Description of your problem
I use pwsh on my windows machine nowadays, and I don't see a way to use that from spyder terminal. The settings menu has a dropdown with only cmd and powershell.
What steps will reproduce the problem?

Go to spyder settings -> Terminal
Find that there is no option for pwsh in the Terminal shell dropdown

What is the expected output? What do you see instead?
I expect to see pwsh also in the dropdown, or another option to manually specify the location of shell where I can give the path to pwsh.exe
Versions and main components

Terminal Version: 1.2.2
Spyder Version: 5.3.0
Python Version: 3.10
Operating system: Windows 10

Hi @saaketp! Thanks for opening this issue, I will add pwsh for our next release.


Unify storage tests

This PR improves the test coverage of the Storage by moving most of the tests into "backend tests". Storage backend tests are shared between backend implementations, and executed on both backends (database and S3) when running cargo test. This reduces the amount of tests that needs to be written for the storage (as we don't have to duplicate tests between the database and S3 anymore) and increases the confidence in switching backends.
Along with those changes:

The compression code was split in its own finme
The S3 bucket name is configurable through Config
TestS3 is removed, and it's behavior is moved to the normal S3 backend when used during tests
Deleting prefixes was moved from db/delete.rs to Storage
Use a runtime in the S3Backend struct instead of a global one

This PR is best reviewed commit-by-commit.
r? @jyn514
cc @Kixiron

@pietroalbini do you need this change for build tests? Or is it a general refactor?

Kinda, it started for buildtests and then I started on a tangent :sweat_smile:

Rebased and did this other change:

Use a runtime in the S3Backend struct instead of a global one

Addressed most of the feedback. The few items left are things I think can be done in a followup PR.

Use a runtime in the S3Backend struct instead of a global one

Can you expand on the rationale for this? Are we sure the backend is only ever constructed once? Tokio is still leaking file descriptors so we need to be careful not to create too many runtimes.

Yeah we're only building a single instance of the backend across the whole application.


Update alternate.is-local.org.json

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.
[x] Your website is not hosting on the following due to SSL issues: Vercel, Netlify

Description

Link to Website

alternate.is-local.org is improperly configured
Your site's DNS settings are using a custom subdomain, alternate.is-local.org, that is set up as an A record. We recommend you change this to a CNAME record pointing to saksham.thedev.id. For more information, see documentation (InvalidARecordError).


Dockerfile: Use single docker image

Building multiple docker images with different dependencies consumes more time
and space. And even in deployment, Node hast to download all three different
images.
Instead, this change builds single docker image that holds all three binaries.
Did also needed deployment changes.
Now {build,push}-%-image(s) make targets are replaced with {build,push}-image
target.

Let's merge this before PR #265.


Loading ready event

Hi there, first of all excellent work with creating such an interactive, smooth and powerful visualisation.
I am trying to use the globe with a 16k jpg texture of the earth so I can have more detail in the texture as to when i zoom in.
The globe rendering takes a while though ( couple of seconds ) when the component mounts but I currently have no way of displaying a loader/mesh placeholder before the initial rendering is complete. Any insight to this?

Hi thanks for checking this out!  The library currently doesn't handle rendering a loader/mesh placeholder.  I would be happy to add this in as an option in globeOptions.  I don't really know how to do this in ThreeJS (I can explore and check out how to do this), but if you have a really simple example that works in Codesandbox.io, I can look at it and add this in.
Or feel free to submit a PR if you know how to add this in!

Hey thanks for the quick reply!
I am currently working on finding out how to do it.
Three JS supports this feature here https://threejs.org/docs/#api/en/loaders/managers/LoadingManager
I am not familiar with your library as to how its constructed or TS though, maybe you will be able to solve it. In the meantime I am trying to find a simple workaround for it with some sort of HOC or conditional rendering with react redux.

Awesome, I might have time to get to this next week and maybe support loader/mesh placeholders for both globe and backgrounds!

Hi @Jehutty, sorry I wasn't able to get to this earlier.  I included this enhancement in 3.1.1 (codesandbox example).  The docs in Textures also cover this in detail.
tldr;  I exposed a onTextureLoaded method that will allow you to pass a callback when the texture successfully load.  A few things I concerned while working on this:

I looked at LoadingManager and I did not want to support all the functionality start, error, loaded and increase the complexity of the component.  I think you bring a valuable point about having some control in determining if the component is loading.  If you look at the codesandbox example, you can see how you can achieve the effect outside of the component with the help of onTextureLoaded and React hooks.
This only applies to the globe texture and does not work for background and cloud textures.  In the future, I can include that functionality as well, but I'll leave it to the future to add it if more people would like that control.  I still want to prioritize keeping the component as simple as possible!


Log instead of error MissingOrMalformedExtensions in rustls_native_certs::load_native_certs

Only log errors because rustls-native-certs doesn't work with certificates that don't define any extensions, this is caused by the webpki crate: https://github.com/briansmith/webpki/blob/17d9189981a618120fd8217a913828e7418e2484/src/error.rs#L78-L83
However, such certs might be present in the trusted root store, like for Windows systems in an enterprise domain. So, reqwest should not throw reqwest::Error { kind: Builder, source: MissingOrMalformedExtensions } in that case, but rather log only a warning.
See also this comment https://github.com/rustls/rustls/blob/3c390ef7c459cc1ef2504bd9d1fefdcb7eea1c20/rustls/src/anchors.rs#L115-L125

Thanks for doing this for us https://github.com/seanmonstar/reqwest/issues/1355#issuecomment-953133794 1376

Hi @seanmonstar, what's required for this PR to come in?
I'm happy to deal with the conflicts if that would help?

Ah, this looks interesting! I suppose it's a good idea, I admit I haven't given it too much thought. Are there any decent instances where someone would prefer or be relying on this to error and NOT log? I guess, is there any reason we shouldn't just merge this? I don't have the time to investigate that answer very well.

In the meantime, pull request #1388 already made a change so that ClientBuilder::build() no longer panics but returns an error instead. So, this pull request needs to be renamed and rebased.

Are there any decent instances where someone would prefer or be relying on this to error and NOT log?
@seanmonstar

I can imagine a situation like this: A software project uses OCI containers that contain the reqwest library with the 'rustls-tls-native-roots' feature, and also a fixed version of the ca-certificates installed with the package manager. In the future, a new root certificate appears in ca-certificates incompatible with the used rustls version. The developers would like to spot the incompatibility during automated testing of the container image, instead of encountering crashes months later in production when a server moved to a certificate issued by the Root CA incompatible with the used rustls version. So, this imaginary developer would like to keep the error an error, so that it fails already during testing.
However, for many other users of this library, the native root store is updated independently of the application. Let's say, in the future a Windows update adds a new root certificate that is incompatible, while the rust application using reqwest is not updated. The next time the application starts, it crashes because of the error. In such cases, the imaginary developer would have liked that the error was only a warning in the logs instead, because the Windows updates are hard to account for.

💯
The error we're encountering is specifically the second one. We distribute a CLI to users, and can't control the runtime environment. Erroring with a really nice warning may not be sufficient either, in our testing (this is second hand, I can probably dig up the details) it can fail on a valid certificate, removing which may not be desirable. This happened to be on Windows, but I suspect it isn't unique to Windows. I suspect it may have been a development or self-signed certificate, but that shouldn't matter.
Logging a warn is probably better, because it can communicate the problem without forcing a user action.

Right on, thanks for the write-up, I agree with the assessment. And thanks for the code, as well! Let's go!

Amazing, thanks @seanmonstar !


Added google signin(#27)

@Manzee1609 Can you attach a gif/video and also follow the issue template at most if you can.

Ok
On Sat, 20 Mar 2021, 6:40 pm HIMANSHU SHARMA, @.***>
wrote:

@Manzee1609 https://github.com/Manzee1609 Can you attach a gif/video
and also follow the issue template at most if you can.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/himanshusharma89/relic_bazaar/pull/146#issuecomment-803316269,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APPSNLMBCKA3MVAL4MN6QF3TESNENANCNFSM4ZQDK4XQ
.

Related Issue
Fixed #146
Proposed Changes
Added google signin functionality in login page
Added google signout functionality in dashboard drawer
Additional Information
Used firebase authentication
Screenshots of added work

https://user-images.githubusercontent.com/64956077/111875495-168d0980-89c0-11eb-91d5-d494de1b8b4e.mp4


feature: implement the dfgetTaskMgr

Signed-off-by: Starnop<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Implement the dfgetTaskMgr
Ⅱ. Does this pull request fix one issue?

fixes part of #366
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Added.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews
It hasn't been implemented for the List interface yet.

Codecov Report

Merging #458 into master will increase coverage by 0.1%.
The diff coverage is 69.04%.

@@            Coverage Diff            @@
##           master     #458     +/-   ##
=========================================
+ Coverage   60.51%   60.62%   +0.1%
=========================================
  Files          64       65      +1
  Lines        3229     3271     +42
=========================================
+ Hits         1954     1983     +29
- Misses       1145     1153      +8
- Partials      130      135      +5

Impacted Files
Coverage Δ

supernode/daemon/mgr/dfget_task_mgr.go
69.04% <69.04%> (ø)

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 578ac41...6373e69. Read the comment docs.

LGTM


Reintroduce the lit test as needed

At the moment, lit tests are disabled. Consider reintroducing.

This is no longer needed. We use a direct compares between files instead managed from Python.


Bug: monitorsleep putting pc to sleep/standby

Describe the bug
When I use the montitorsleep command the pc goes to sleep and hassagent loses connection.
To Reproduce
Steps to reproduce the behavior:

Go to 'HA > hass.agent device view'
Click on 'button for command to "sleep monitor"'

Expected behavior
Montor to go to sleep instead of pc and to be able to turn monitor back on.
Misc info (please complete the following information):

Windows build (ideally screenshot/info of winver.exe output):  windows 11 64bit professional 2200.318
Windows' UI language: english
HASS.Agent version: 2022.14.0

Please check what's applicable (multiple answers possible):

[x] Installed via installer
[ ] Installed manually
[x] Problem occurs in HASS.Agent
[ ] Problem occurs in Satellite Service

Additional context
Add any other context about the problem here.

I have the same issue.
Workaround I use in the meantime:
Instead of monitorsleep command, I use the following Powershell command: powercfg -change -monitor-timeout-ac 1
(it changes the screen sleep when plugged to 1 minute)
To wake up the screen, use the monitorwake command and use the following Powershell command: powercfg -change -monitor-timeout-ac 0
(it changes the screen sleep when plugged to never)


Fix duplicate block scoped declaration not reported in the same scope

Duplicate let / const / class declaration is a hard error in any scope,
but VarCheck was only checking this in the global scope.

This change is

Let's not merge this just yet. I'm in the process of making a related change to VariableReferenceCheck, and it requires cleanup internally. And we're trying to do a release as well. After the release and my change have landed, then we can come back to this.

@dimvar This PR fixes a crash in the rewriting pass: https://closure-compiler-debugger.appspot.com/#input0%3Dfunction%2520f(x)%2520%257B%250A%2520%2520switch%2520(x)%2520%257B%250A%2520%2520%2520%2520case%2520'a'%253A%250A%2520%2520%2520%2520%2520%2520let%2520z%2520%253D%2520123%253B%250A%2520%2520%2520%2520%2520%2520break%253B%250A%2520%2520%2520%2520case%2520'b'%253A%250A%2520%2520%2520%2520%2520%2520let%2520z%2520%253D%2520234%253B%250A%2520%2520%2520%2520%2520%2520break%253B%250A%2520%2520%257D%250A%257D%26input1%26conformanceConfig%26externs%26refasterjs-template%26includeDefaultExterns%3D1%26CHECK_SYMBOLS%3D1%26CHECK_TYPES%3D1%26LANG_IN_IS_ES6%3D1%26MISSING_PROPERTIES%3D1%26PRESERVE_TYPE_ANNOTATIONS%3D1%26PRETTY_PRINT%3D1%26TRANSPILE%3D1
Will your change in VariableReferenceCheck catch this error?

@dimvar This is the crash you reported internally as b/28248708 . Sorry for the internal/external bug tracker/pull request confusion. I'm fine with holding off on landing this for a little while though, since it should be pretty easy for people to work around.

@Dominator008 Do you expect that this PR just turns crashes into errors, or do you also expect new errors that were not being detected before? In the latter case, we may have to delay the fix.

@dimvar It should just turn crashes into errors, but I can't guarantee that. My guess is that this kind of error is usually easy to spot.

sgtm


Upgrade protoreflect

Context
Upgrading dep protoreflect from v1.14.1 to v1.15.6.
This allows protoreflect to be updated in the infra repo
More context:
https://canva.slack.com/archives/CC6NN1GU9/p1709598824895179

🤞 nothing breaks

Tests are passing, build works. But let's see 😬


Exception on EnableReferences extension only when debugging

I don't know how to exactly reproduce it as it happens in specific circumstances. But when those are met, the exception always happens, eg. some of our tests always fail when debugging.
XmlSerializer is defined as follows:
private static readonly IExtendedXmlSerializer _Serializer = new ConfigurationContainer()
	.EnableReferences()
	.Create();

Full stacktrace:
DefaultActivators.Reference() in ExtendedXmlSerializer.ReflectionModel, ExtendedXmlSerializer.dll
DefaultActivators.Create() in ExtendedXmlSerializer.ReflectionModel, ExtendedXmlSerializer.dll
ConditionalWeakTable<Type, IActivator>.GetValue() in System.Runtime.CompilerServices, mscorlib.dll
ReferenceCacheBase<Type, IActivator>.Get() in ExtendedXmlSerializer.Core.Sources, ExtendedXmlSerializer.dll
FixedSource<Type, IActivator>.Get() in ExtendedXmlSerializer.Core.Sources, ExtendedXmlSerializer.dll
Lazy<IActivator>.CreateValue() in System, mscorlib.dll
Lazy<IActivator>.LazyInitValue() in System, mscorlib.dll
Lazy<IActivator>.get_Value() in System, mscorlib.dll
SingletonSource<IActivator>.Get() in ExtendedXmlSerializer.Core.Sources, ExtendedXmlSerializer.dll
Activators.Activator.Get() in ExtendedXmlSerializer.ExtensionModel.Types, ExtendedXmlSerializer.dll
Extensions.New<ExtendedXmlSerializer.ExtensionModel.RootInstanceExtension>() in ExtendedXmlSerializer.ReflectionModel, ExtendedXmlSerializer.dll
Extensions.New<ExtendedXmlSerializer.ExtensionModel.RootInstanceExtension>() in ExtendedXmlSerializer.ReflectionModel, ExtendedXmlSerializer.dll
Extensions.Add<ExtendedXmlSerializer.ExtensionModel.RootInstanceExtension>() in ExtendedXmlSerializer.Configuration, ExtendedXmlSerializer.dll
Extensions.Add<ExtendedXmlSerializer.ExtensionModel.RootInstanceExtension>() in ExtendedXmlSerializer.Configuration, ExtendedXmlSerializer.dll
Extensions.With<ExtendedXmlSerializer.ExtensionModel.RootInstanceExtension>() in ExtendedXmlSerializer.Configuration, ExtendedXmlSerializer.dll
Extensions.EnableRootInstances<ExtendedXmlSerializer.Configuration.IRootContext>() in ExtendedXmlSerializer.ExtensionModel, ExtendedXmlSerializer.dll
Extensions.EnableReferences() in ExtendedXmlSerializer.Configuration, ExtendedXmlSerializer.dll
Extensions.EnableReferences() in ExtendedXmlSerializer.Configuration, ExtendedXmlSerializer.dll
--- static xml serializer initialization is here
--- our test is here
[Native to Managed Transition]
RuntimeMethodInfo.UnsafeInvokeInternal() in System.Reflection, mscorlib.dll
RuntimeMethodInfo.Invoke() in System.Reflection, mscorlib.dll
MethodBase.Invoke() in System.Reflection, mscorlib.dll
TestMethodRunner.DefaultTestMethodInvoke() in Microsoft.VisualStudio.TestPlatform.MSTestFramework, Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration.dll
TestMethodRunner.RunTestMethod() in Microsoft.VisualStudio.TestPlatform.MSTestFramework, Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration.dll
TestMethodRunner.ExecuteTest() in Microsoft.VisualStudio.TestPlatform.MSTestFramework, Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration.dll
TestMethodRunner.ExecuteInternal() in Microsoft.VisualStudio.TestPlatform.MSTestFramework, Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration.dll
TestMethodRunner.Execute() in Microsoft.VisualStudio.TestPlatform.MSTestFramework, Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration.dll
UnitTestRunner.RunInternal() in Microsoft.VisualStudio.TestPlatform.MSTestFramework, Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration.dll
UnitTestRunner.RunSingleTest() in Microsoft.VisualStudio.TestPlatform.MSTestFramework, Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration.dll
[Application Domain Transition]
MixedModeExecutor.ExecuteTestViaMsTestV1Runner() in Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration, Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration.dll
MixedModeExecutor.ExecuteTests() in Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration, Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration.dll
MixedModeExecutor.ExecuteDurontoTestsInternal() in Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration, Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration.dll
MixedModeExecutor.ExecuteDurontoTests() in Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration, Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration.dll
MixedModeExecutor.ExecuteTests() in Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration, Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration.dll
MixedModeExecutor.ExecuteTests() in Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration, Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration.dll
MixedModeExecutor.RunTests() in Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration, Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration.dll
MSTestExecutor.RunTests() in Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration, Microsoft.VisualStudio.TestPlatform.Extensions.VSTestIntegration.dll
TestExecutionManager.RunTestInternalWithExecutors() in Microsoft.VisualStudio.TestPlatform.Core, Microsoft.VisualStudio.TestPlatform.Core.dll
TestExecutionManager.RunTestsInternal() in Microsoft.VisualStudio.TestPlatform.Core, Microsoft.VisualStudio.TestPlatform.Core.dll
TestExecutionManager.RunTests() in Microsoft.VisualStudio.TestPlatform.Core, Microsoft.VisualStudio.TestPlatform.Core.dll
TestExecutionManager.RunTests() in Microsoft.VisualStudio.TestPlatform.Core, Microsoft.VisualStudio.TestPlatform.Core.dll
ThreadHelper.ThreadStart_Context() in System.Threading, mscorlib.dll
ExecutionContext.RunInternal() in System.Threading, mscorlib.dll
ExecutionContext.Run() in System.Threading, mscorlib.dll
ExecutionContext.Run() in System.Threading, mscorlib.dll
ThreadHelper.ThreadStart() in System.Threading, mscorlib.dll
[Native to Managed Transition]

Where the exception happens exactly:

Ah... I'm thinking threading issue, perhaps?  Are other tests running concurrently when you get this exception?  Sounds like you are debugging a single test.  Perhaps there might be another test (or functionality/context) that might be inadvertently running alongside for the ride.
That has been my experience, at least.  Might be worth putting a lock around it (in another initialization method) to see if that subsides it.

Ah... I'm thinking threading issue, perhaps?  Are other tests running concurrently when you get this exception?  Sounds like you are debugging a single test.  Perhaps there might be another test (or functionality/context) that might be inadvertently running alongside for the ride.
That has been my experience, at least.  Might be worth putting a lock around it (in another initialization method) to see if that subsides it.

Ah, OK... so maybe I am misunderstanding here.  Are you saying that it happens consistently or it happens periodically?  If you are able to make it happen consistently then that would be most helpful.
As far as threading, I got wrapped around.  It's actually re-entry on a static field/property that leads to null exceptions which does not appear to be the case here, so please disregard.
It does look like the object type it is wanting to construct is an extension, which is not at all what it should be, obviously.
My bandwidth is limited during the holiday season, but I will try to see if I can figure out something here for you when I get the chance.

I'm not so sure anymore if this is actually a bug. When continuing after the exception the serializer works fine, so that exception must be caught somewhere. I'm only seeing it on my setup because I have "catch all exceptions" enabled in my IDE.
Here's a simple test case that causes this exception (if you have "catch all exceptions" enabled):
using System;
using ExtendedXmlSerializer.Configuration;
using ExtendedXmlSerializer.ExtensionModel.Xml;
using NUnit.Framework;

namespace TestProject1
{
	[TestFixture]
	public class Tests
	{
		private static readonly IExtendedXmlSerializer Serializer = new ConfigurationContainer()
			.EnableReferences()
			.Create();

		public class FooBar
		{
			public string Foo { get; set; } = "Bar";
		}

		[Test]
		public void Test1()
		{
			var stringified = Serializer.Serialize(new FooBar());

			Assert.NotNull(stringified);

			Console.WriteLine(stringified);
		}
	}
}

I tested older versions of the serializer and it seems this exception was introduced in 2.1.2.

Seems ok, the exception is not thrown anymore for me.


Fixed overflowed scroll sidenav on desktop

Proposed changes
There's additional 60px because it's problem with mobile devices. Without it, when hiding the address bar, there's some kind of jumpy effect, or rather re-rendering effect that makes UX not really smooth. But this fix comes with another issue, see https://github.com/Dogfalo/materialize/issues/4719. It's fixed on mobile. But in desktop, the scrollbar doesn't look good.
So, instead of using calc(100% + 60px), I think using 100vh is more appropriate. Here is an article explaining this.
I also deleted a temporary fix (I can't find the documentation about -moz-calc thing)
I hope this didn't create another new issue... so tell me if you found one 😉
Screenshots (if appropriate) or codepen:
Before : https://codepen.io/smankusors/pen/VwKdwPG
After : https://codepen.io/smankusors/pen/NWRzWjq
Types of changes

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

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

-moz-calc is an old / deprecated vendor-prefixed version of calc for old Mozilla Firefox versions.

-moz-calc is an old / deprecated vendor-prefixed version of calc for old Mozilla Firefox versions.

-moz-calc is an old / deprecated vendor-prefixed version of calc for old Mozilla Firefox versions.

and.... this is.... Firefox at 2010? I think we can safely delete this, right?

-moz-calc is an old / deprecated vendor-prefixed version of calc for old Mozilla Firefox versions.

and.... this is.... Firefox at 2010? I think we can safely delete this, right?

Correct, see also https://caniuse.com/calc

Correct, see also https://caniuse.com/calc


feat(orchestrator): add endpoint for retrieving single workflow insta…

…nces
FLPATH-1694
https://issues.redhat.com/browse/FLPATH-1694
we added /v2/workflows/{workflowId}/instances

@pkliczewski @gciavarrini @masayag @mareklibra I will rebase after PR https://github.com/janus-idp/backstage-plugins/pull/2247 is merged. fro now just review and we'll merge later

@gciavarrini @batzionb


[2/n] Critical Path analysis for GPU events and CPU->GPU and GPU->CPU dependencies

What does this PR do?
This is a follow up to #67 that adds the nodes in the critical path analysis graph for GPU kernels
and in addition we also add the kernel launch delay and synchronization events.
Details
We update the graph with the following

GPU kernels have two nods - begin, end and an edge connecting them.
Kernel launch delay - if the CUDA runtime launched a kernel when there was no outstanding kernel on the same stream, we add an edge for the delay in the critical path with the weight = launch delay.
Kernel kernel delay - the else case of 2, we add an implicit dependency on the last CUDA kernel executed on the stream.
Synchronization - using CUDA sync events we track
a. Context/Device sync= the last kernel on all streams are a dependency for the Context Sync call on CPU.
b. Stream Sync= the last kernel on the specific stream being synced is a dependency for the Stream Sync runtime call on CPU.

Event sync is handled in the next PR.
Test/Examples
Running this on simple add test.
We can see the edges for kernel launch and kernel-kernel delay.

The example below shows Context Sync event (pink on the right), there are two edges coming in, one each from the last kernel on each CUDA stream.

The final critical path shifts from CPU to GPU in the end of the iteration - highlighted events are on critical path.

Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)

[x] N/A

[x] Did you write any new necessary tests?

[ ] N/A

[ ] Did you make sure to update the docs?

[x] N/A

[] Did you update the changelog?

[x] N/A

Codecov Report
Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (8084627) 90.78% compared to head (06d5e31) 90.94%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #68      +/-   ##
==========================================
+ Coverage   90.78%   90.94%   +0.15%
==========================================
  Files          30       30
  Lines        2441     2506      +65
==========================================
+ Hits         2216     2279      +63
- Misses        225      227       +2

Files
Coverage Δ

hta/analyzers/trace_counters.py
96.59% <ø> (ø)

tests/test_trace_analysis.py
99.57% <100.00%> (+0.03%)
:arrow_up:

hta/analyzers/critical_path_analysis.py
90.53% <92.06%> (+1.30%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Unable to delete Work Item

Unable to delete Work Item

No Confirm dialog shows up after clicking on the Delete menu from Kebab

@sanbornsen please look into it

https://github.com/fabric8io/fabric8-planner/pull/1111

Is #1103 is the same issue?

yes !


publish<EMAIL_ADDRESS>

Requested by: @brustolin
Merge target: (default)
Quick links:

View changes
View check runs

Assign the accepted label to this issue to approve the release.
Leave a comment containing #retract under this issue to retract the release (original issuer only).
Targets

[x] github
[x] cocoapods[sentry-cocoapod]
[x] registry

Targets marked with a checkbox have already been executed.  Administrators can manually tick a checkbox to force craft to skip it.

fail to merge bach to main expected - closing


Where did code coverage go?

@Microsoft/pvsc-team
We used to code code coverage on our extension and it used to be integrated with GitHub PRs.
I can no longer see this info.

@gramster
I know you're looking at something related.

It looks like it was working until this PR: https://github.com/Microsoft/vscode-python/pull/2431

Code coverage is linked from our CONTRIBUTING.md file.
And the PR comments were turned off in August by request of everyone else since the results were not consistent enough to warrant relying on (yet). All coverage settings are controlled via https://github.com/Microsoft/vscode-python/blob/master/.github/codecov.yaml .

request of everyone else since the results were not consistent enough to warrant relying on

This at least helps us know if coverage drops.
E.g. we add a tonn of new code, and very little tests, this helps us determine easily whether there's sufficient (lack of) coverage for those changes.
In fact, I've used this in the past (when reviewing others PRs as well as my own).
E.g. I can easily how well the new code is covered in the tests...,
I guess going away for a month long holiday made me forget my best practices... hence never raised it until now.
Personally I would still like to see this, even if its not going to be used by others.

consistent enough to warrant relying on

What do you mean not consistent enough?


Run amplify pull

I ran amplify pull and it changed the local files. Thus, you should be able to use these instead of needing to run amplify pull yourself

"gqlSchemaPath": "C:\Users\Taco\Desktop\OffshoreWindJobNavigator\amplify\backend\api\oswjn\schema.graphql",
does using Tyler's local file for this have no issues with graphQL on other machines?


Use stable version of Nokogiri

Nokogiri will be cutting an official release soon after more testing. This
hacked together release is causing some developers issues with building
Nokogiri, and the security vulnerabilities are no longer thought to be too
severe. Whilst we wait for an official release, we should revert as far as the
most recent stable release, <IP_ADDRESS>, as this does address some security
vulnerabilites.

This merge was rebased out of master in favour of #451


[skeleton] variant=text wrong size

I suppose that text skeleton should be 1:1 size to real text that he is replacing. Right now, size is way off.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Size of text skeleton is wrong.
Expected Behavior 🤔
Size of text skeleton should be equal to size of replaced text.
Steps to Reproduce 🕹
https://codesandbox.io/s/create-react-app-7y24f
To work as expected, there has to be 2 changes:

Add   as skeleton children
Change component CSS to:

height: 'auto',
marginTop: 0,
marginBottom: 0,
transformOrigin: '0 65%',
transform: 'translateZ(0) scale(1, 0.65)',
textIndent: -999,
borderRadius: theme.shape.borderRadius,

Alternatively,   could be added in pseudoelement:
text: {
	height: 'auto',
	marginTop: 0,
	marginBottom: 0,
	transformOrigin: '0 65%',
	transform: 'translateZ(0) scale(1, 0.65)',
	textIndent: -999,
	'&:empty:before': {
		content: '" "'
	}
}

I suppose that text skeleton should be 1:1 size to real text that he is replacing.

@Mangatt Why should it follow this rule? The component was designed to be a rough approximation of the content.

@oliviertassinari You don't agree? Different size results in percieved jumps after content is loaded. What are benefits of different size really?

@Mangatt I had a closer look, actually, it looks pretty slick. I would be curious to try it out with our implementation :).

@oliviertassinari I'm glad that you like it. I think that as well :)

@Mangatt Alright, do you want to submut a pull request 💪?

@oliviertassinari I'll try to in next week 👍

@Mangatt Did you had the chance to work on it? :)

But what about other examples of usage? For example, if user provides disabledTypography prop. That way a rendered skeleton would shrink even harder compared to text content.
Demo here.

@macfire10 the <p> elements add its own margin. If you replace it with a <div>, we would still be good with disabledTypography set to true. @Mangatt's proposal seems to be solid. Given he has other priorities right now, would you be interested in contributing his idea? :)

@oliviertassinari sure!

This appears to have created an odd issue:

@josh-feldman Thanks for the report. It was taken care of today.


Allow CentOS nodes to manage EBS volumes

PR #507 only updated the Ubuntu profile. This updates CentOS as well.

CI is :green_apple: :shipit: Thanks for the PR!


Running ChaNGa on latest build of charm gives a crash

Original author: Harshitha Menon
Original issue: https://charm.cs.illinois.edu/redmine/issues/1026

I used the following command to build charm
./build ChaNGa gni-crayxe hugepages smp --with-production --enable-tracing -j8

latest commit on master is 7c03bc21b3a975ae764ca03e62ce7fa4344e2cd5
I used cosmo25 dataset on 512 nodes
It throws  the following error
Wed Apr  6 14:00:27 2016: [PE_0]:inet_recv:inet_recv: recv error (fd=-1543438960) Bad file descriptor
Wed Apr  6 14:00:27 2016: [PE_0]:_pmi_network_barrier:_pmi_inet_recv from target 0 failed pmi errno -1
Wed Apr  6 14:00:27 2016: [PE_0]:_pmi_barrier:network_barrier failed
Wed Apr  6 14:00:27 2016: [PE_1]:_pmi_barrier:_pmi_smp_barrier failed
Application 37193257 is crashing. ATP analysis proceeding...

ATP Stack walkback for Rank 0 starting:
  start_thread`0x2aaaaacd4805
  call_startfn`machine-smp.c:415
  ConverseRunPE`machine-common-core.c:1268
  _initCharm(int, char**)`init.C:1267
  CmiBarrier`machine-common-core.c:1578
  LrtsBarrier`machine.c:4306
ATP Stack walkback for Rank 0 done
Process died with signal 11: 'Segmentation fault'
Forcing core dumps of ranks 0, 2, 3
View application merged backtrace tree with: stat-view atpMergedBT.dot
You may need to: module load stat

_pmiu_daemon(SIGCHLD): [NID 14607] [c23-7c0s7n1] [Wed Apr  6 14:00:45 2016] PE RANK 355 exit signal Killed
_pmiu_daemon(SIGCHLD): [NID 14677] [c23-7c0s5n3] [Wed Apr  6 14:00:45 2016] PE RANK 859 exit signal Killed

Original date: 2016-04-07 14:33:39

Have you run this on the 6.7.0 release? It'd be good to pin down when it started failing...

Original date: 2016-04-14 04:28:58

Can you retry this with current master now?

Original author: Harshitha Menon
Original date: 2016-04-14 19:13:37

It doesn't crash anymore with the current master (046e8f9856f8618f6b4fa845c0a16840f9f252dc).

Original date: 2016-04-19 17:09:35

What was the resolution of this?

Original author: Harshitha Menon
Original date: 2016-04-19 17:13:20

It was not reproducible with the latest master


chore: segment-bridge on authentication component

since segment-bridge needs to create resources on
the toolchain-host-operator, which doesn't have a component of its own, those resources are created through the authentication component.
While this will probably change later on, during the last change the connection between the resource yaml and the component was not made. this fix it.

/lgtm


[BUG] MobileNet with local video file not working

Hello,
I'm trying to run a MobileNet model on a video file located on my computer, according to this example provided in the documentation. It works with the video provided by default, though the detections are a little bit laggy. Then, I wanted to try with an other video, this one: https://www.youtube.com/watch?v=MNn9qKG2UFI.
The model detects nothing except false positives (like a boat, a tv monitor). I've managed to be sure that the image is correctly resized by opening the resized image. I've also tried with a YOLO model, and I've got the same issue.
Here are some screenshots of the problem:

MRE
You can download the complete MRE here since it is too large to upload all files here: https://filetransfer.io/data-package/oCBNoI32#link . I've attached the MRE without the MobileNet mobilenet-ssd_openvino_2021.4_8shave.blob (which is the one coming frome the depthai-python examples models). Since I've also tried with latest MobileNet version and YOLO models, I suppose that this issue is not model related.
MRE_wo_model.zip
Version used
Camera: tried both on OAK-D USB and OAK-1 PoE (flashed with the latest bootloader, 0.22)
Host: tried from Mac OS, Ubuntu on a computer, Ubuntu on a Raspberry (I must highlight that the program is very slow on Raspberry despite no code executed on host)
Detphai Version: 2.17.3
Code
#!/usr/bin/env python3

from pathlib import Path
import sys
import cv2
import depthai as dai
import numpy as np
from time import monotonic

import blobconverter

# Get argument first
nnPath = 'mobilenet-ssd_openvino_2021.4_8shave.blob'
videoPath = 'traffic_5mn.mp4'
if len(sys.argv) > 2:
    nnPath = sys.argv[1]
    videoPath = sys.argv[2]

if not Path(nnPath).exists() or not Path(videoPath).exists():
    import sys
    raise FileNotFoundError(f'Required file/s not found, please run "{sys.executable} install_requirements.py"')

# MobilenetSSD label texts
labelMap = ["background", "aeroplane", "bicycle", "bird", "boat", "bottle", "bus", "car", "cat", "chair", "cow",
            "diningtable", "dog", "horse", "motorbike", "person", "pottedplant", "sheep", "sofa", "train", "tvmonitor"]

# Create pipeline
pipeline = dai.Pipeline()

# Define sources and outputs
nn = pipeline.create(dai.node.MobileNetDetectionNetwork)

xinFrame = pipeline.create(dai.node.XLinkIn)
nnOut = pipeline.create(dai.node.XLinkOut)

xinFrame.setStreamName("inFrame")
nnOut.setStreamName("nn")

# Properties
nn.setConfidenceThreshold(0.5)
nn.setBlobPath(nnPath)
nn.setNumInferenceThreads(2)
nn.input.setBlocking(True)

# Linking
xinFrame.out.link(nn.input)
nn.out.link(nnOut.input)

# Connect to device and start pipeline
with dai.Device(pipeline) as device:

    # Input queue will be used to send video frames to the device.
    qIn = device.getInputQueue(name="inFrame")
    # Output queue will be used to get nn data from the video frames.
    qDet = device.getOutputQueue(name="nn", maxSize=6, blocking=True)

    frame = None
    detections = []

    # nn data, being the bounding box locations, are in <0..1> range - they need to be normalized with frame width/height
    def frameNorm(frame, bbox):
        normVals = np.full(len(bbox), frame.shape[0])
        normVals[::2] = frame.shape[1]
        return (np.clip(np.array(bbox), 0, 1) * normVals).astype(int)

    def to_planar(arr: np.ndarray, shape: tuple) -> np.ndarray:
        return cv2.resize(arr, shape).transpose(2, 0, 1).flatten()

    def displayFrame(name, frame):
        for detection in detections:
            bbox = frameNorm(frame, (detection.xmin, detection.ymin, detection.xmax, detection.ymax))
            cv2.putText(frame, labelMap[detection.label], (bbox[0] + 10, bbox[1] + 20), cv2.FONT_HERSHEY_TRIPLEX, 0.5, 255)
            cv2.putText(frame, f"{int(detection.confidence * 100)}%", (bbox[0] + 10, bbox[1] + 40), cv2.FONT_HERSHEY_TRIPLEX, 0.5, 255)
            cv2.rectangle(frame, (bbox[0], bbox[1]), (bbox[2], bbox[3]), (255, 0, 0), 2)
        # Show the frame
        cv2.imshow(name, frame)

    cap = cv2.VideoCapture(videoPath)
    while cap.isOpened():
        read_correctly, frame = cap.read()
        if not read_correctly:
            break

        img = dai.ImgFrame()
        resized = to_planar(frame, (300, 300))
        img.setTimestamp(monotonic())
        img.setType(dai.RawImgFrame.Type.BGR888p)
        img.setSize(300, 300)
        img.setData(resized)
        cv2.imshow("ImgFrame output", img.getCvFrame())

        qIn.send(img)

        inDet = qDet.tryGet()

        if inDet is not None:
            detections = inDet.detections

        if frame is not None:
            displayFrame("rgb", frame)

        if cv2.waitKey(1) == ord('q'):
            break

Pipeline Graph

Thank you for your help, being able to infer on local video file is absolutely crucial for us.
Pierre

Update: bug was due to syncing, solved by @erik on Luxonis Forum: https://discuss.luxonis.com/d/868-detection-problem-with-mobile-net-on-video-from-host/11


Feature: Support multiple runtimes other than Node.js

Closes #515, #252
✅ Checklist

[x] I have followed every step in the contributing guide
[x] The PR title follows the convention.
[x] I ran and tested the code works

Testing

Created canary packages and tested on Cloudflare Workers with node_compat, Deno, and Bun
Additionally, added some runtime tests in the repo to automatically test the SDK on multiple runtimes

Changelog
Supports for the following runtimes:

Cloudflare Worker with node compat turned on (https://developers.cloudflare.com/workers/runtime-apis/nodejs/)
Bun
Deno using npm: specifiers

This is amazing. Is this usable yet? I'd be happy to test it out for you!

@threepointone Yup, it's been out for a little bit now: https://trigger.dev/changelog/cross-runtime-support


Improve RowMetadata for presence/absence check of columns

RowMetadata currently exposes getColumnMetadata(…) and getColumnMetadatas(). If a client wants to check if a column is part of the result, then it has to obtain getColumnMetadatas() and iterate over ColumnMetadata.
For a discovery-based client, that does not know the structure of a result, this mapping causes a significant overhead as the client cannot cache a previous metadata object. getColumnMetadata(…) on the other hand, throws an exception of the column at identifier (e.g. name of the column) is absent.
How about introducing a SortedSet<String> getColumnNames() as short-cut? This would solve at least three issues:

Discovery of the number of columns in the row.
Fast retrieval of column names without additional mapping.
Clients can easily call getColumnNames().contains(…) to check whether a column exists.

A nice side effect is that drivers can implement naming rules themselves (e.g. via TreeSet and a Collation) and drivers can cache the result metadata based on vendor-specific rules.
See also spring-projects/spring-data-r2dbc#69 for a use-case.

@mp911de created a PR for this change https://github.com/r2dbc/r2dbc-spi/pull/58. Please review and let me know if any changes are required

A nice side effect is that drivers can implement naming rules themselves (e.g. via TreeSet and a Collation)

@mp911de Can you please elaborate on this a little?

Databases typically implement their own rules how they compare field names (case-sensitive, case-insensitive) and they allow modifiers to (like escape chars) to enforce the former or the latter comparison method. By using an appropriate Comparator, drivers can implement column name equality rules, might resemble functionality of what Spring's LinkedCaseInsensitiveMap does.

Understood.  I'm good with this proposal then and am OK merging #58.


gh login

what do i need to put in ghlogin ?

what do i need to put in ghlogin ?

ghLogin is the login of the user whose graph you want to download


Create Dijkstra.c

Added => Algo => Graph => Dijkstra
Assigned Issue Number ?
#684
Language Opted ?
C
Number of files added or updated ?
1
Name of these files ?
Algorithms => Graph_Algorithms => Dijkstra => Dijkstra.c
Checklist

[x] I've read the contribution guidelines.
[x] I've referred the correct issue number.
[x] I've fill up this entire template correctly.

@ahampriyanshu , Is something wrong?


Modify/fork Initializr extension in IDE so we can add features

...maybe even turn it into proper checkboxes?

The general idea of forking the initializr integration for VSCode makes sense as a next step. In the longer run, we could think about owning that extension ourselves and moving that forward at a broader scope, maybe including those UI/UX changes like checkboxes, etc.

@dsyer The forked VSCode initializr extension is added in the branch vscode-initializr-fork. The initializr extension from the fork has the fix for #3
If it looks good, I'll look into properly integrating the forked initializr extension into https://github.com/eduk8s/pkgs-java-tools probably

How do I try it out if it's in a branch? If you put it on master it will show up in the lab right?

I've been testing it with our own instance of eduk8s... I can fold it in master and then we shall see if it shows up automatically...

It's in master now


Added ReferenceFile trait, modified TwoBitFile to extend it

Closing previous pull request as it was based on my master branch

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/630/
Test PASSed.

Thanks @ananth-pallaseni! Manually committed via https://github.com/bigdatagenomics/adam/commit/13d12b1b1595b2af85b4b4b46e9e569f5e6b4693.


[Bug] Error in documentation

Version
1.2.0
Reproduction link
https://demos.creative-tim.com/now-ui-dashboard-react/#/documentation/alerts
Operating System
N/A
Device
N/A
Browser & Version
N/A
Steps to reproduce
Navigate to https://demos.creative-tim.com/now-ui-dashboard-react/#/documentation/alerts
What is expected?
Each alert type should have a different name:
primary, secondary, success, etc.
What is actually happening?
Each alert has the same name (primary), although the colors are correct.

Solution
Fix the names of the alerts.
Additional comments

Hello there, @ElijahSink ,
Thank you for your interest in working with our products and for noticing this issue.
I'll make an update later this week. Please leave this issue open until then.
Best,
Manu


Image Saving

[x] Command for image saving in game controller

[x] Save image function in SaveAndLoad

Primary: @joperezc

Secondary: @nrhoopes

Size: MEDIUM

Actual Size: MEDIUM

Actual Time: 2HR

Command and function complete,
might need tweaking later for CLI save image


[PRW 2.0]  Moved rw2 proto to the full path (both package name and placement)

Changes:

New path: github.com/prometheus/prometheus/prompb/io/prometheus/write/v2, so io/prometheus/write/v2 package. This is what buf recommends https://buf.build/docs/reference/protobuf-files-and-packages#packages and i shortened it (we will likely never have "local write").
WriteRequest -> Request.
Updated commentary + grammarly.
Change behaviour for samples and histograms. It makes sense more I think, given metadata will be different for the known case this happens too.

Change behaviour for samples and histograms. It makes sense more I think, given metadata will be different for the known case this happens too.

Looks fine to me but I'm not seeing :point_up: this change, which I think you intended to commit (based on your note there but also the changes to the comments in the proto files).

Those changes are here @cstyan 🤗 https://github.com/prometheus/prometheus/pull/13973/files#r1592219265

Merging this, we can revert the commentary about samples vs histogram sample semantics later on, adding to our agenda


Production Merge

Closes/Fixes/Resolves

Explain the feature/fix

Does this PR introduce a breaking change

Screenshots

View Screenshots

Ready-for-merge Checklist

[ ] Expected files: all files in this pull request are related to one feature request or issue (no stragglers)?
[ ] Does the change have appropriate unit tests?
[ ] Did tests pass?
[ ] Did you update or add any necessary documentation (README.md, WHY.md, etc.)?
[ ] Was this feature demo'd and the design review approved?

Please hold back from merging this before merging #166 ⚠


Skip schedule to first day of school

No one cares about knowing that there's no school during summer break. When someone pulls up EPSchedule during the summer, it should automatically jump to the first day of school.

Already fixed.


Jobs Reborn for Forge 1.12.2 ?

Hi @Zrips , I've been setting up additional servers running on Sponge Forge 1.12.2 and trying to get some of the plugins I've used for the Spigot servers also added as Mods for Forge.  Did you ever create a forge version of this?  Any plans to if you haven't?
Thanks -Steve

Hey, in general i have 0 experience with moded servers. Most plugins are working with servers like that. But to make special support for it and add special features, well i don't have so much time to invest in stuff like this currently. I could fix some small compatibility issues, if there is any.

Thanks - I guess I'll need to find out from someone what the real difference is between them which I'm assuming is how classes are loaded or some meta data Forge expects that isn't there for Spigot.  I'll dig into that and go from there to see if there is something that's simple to do.


Fix Package Documentation and Enable Documentation Check in CircleCI

Many packages were not correctly formatting package documentation either because the documentation did not precede the package statement as required or the first line did not correctly reference the package name. A few packages were missing documentation. To prevent regressions this PR enables staticcheck ST1000 to check that every v2 package has valid package docs.

thanks!


Fix #669.

Don't introduce another let to ref for base calls if ref is represented as a local var.
Fixes #669.

Let me add some explanation for review.
The code that is being generated from F# code like:
type Parent(o:obj) = class end
type Broken() as self = inherit Parent(self)

is:
[CompilationMapping(SourceConstructFlags.ObjectType)]
[Serializable]
public class Broken : Parent
{
	internal int init@4;

	public Broken()
	{
		FSharpRef<Broken> self = new FSharpRef<Broken>(null);
		FSharpRef<Broken> self2 = self2;
		this..ctor(LanguagePrimitives.IntrinsicFunctions.CheckThis<Broken>(self2.contents));
		self.contents = this;
		this.init@4 = 1;
	}
}

And it's clear that this always dereferences null, see the original bug report.
However, the scope of the bug is likely not as large as initially anticipated. The problem is that in the bit of code I corrected a local var (self in the above code) is again being wrapped in a local var (self2) in the implicit ctor type checker code. In other words if the self ref is represented as a field, which you can force by e.g. doing:
type Parent(o:obj) = class end
type Broken() as self =
    inherit Parent(self)
    member __.MySelf = self

Everything worked.
To be honest I'm not sure why creating the extra let self = self in TypeChecker.fs isn't harmless and the code that's generated as a result of it:
		FSharpRef<Broken> self = new FSharpRef<Broken>(null);
		FSharpRef<Broken> self2 = self;

But that's apparently not what happens, and for variables that are already local it's clearly superfluous to generate it so I put in this extra check.

Great to have this fixed, thank you


[19.03] nixos/modules/networking/wg-quick Add wg-quick options support

Backport wg-quick module from https://github.com/NixOS/nixpkgs/pull/53043.
This is an implementation of wireguard support using wg-quick config
generation.
This seems preferrable to the existing wireguard support because
it handles many more routing and resolvconf edge cases than the
current wireguard support.
It also includes work-arounds to make key files work.
This has one quirk:
We need to set reverse path checking in the firewall to false because
it interferes with the way wg-quick sets up its routing.

Motivation for this change
Things done

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

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

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

Enabling this module changes the firewall behaviour silently.
That's not ideal in general, and worse to backport since it would change the default and expected behaviour of a stable release.

Agreed, let's close this.


mojito-webapp: Calling depends_on :java is disabled! Use "depends_on "openjdk@11", "depends_on "openjdk@8" or "depends_on "openjdk" instead.

To Reproduce
Run brew tap box/mojito fails
Error: Calling depends_on java: "1.8+" is disabled! Use depends_on "openjdk@11", depends_on "openjdk@8" or depends_on "openjdk" instead.
Environment:
OS: macOS Catalina
Version 10.15.7

I'm something similar when running operation brew tap box/mojito:
Error: Invalid formula: /usr/local/Homebrew/Library/Taps/box/homebrew-mojito/mojito-cli.rb
mojito-cli: Unsupported special dependency :java
Error: Invalid formula: /usr/local/Homebrew/Library/Taps/box/homebrew-mojito/mojito-webapp.rb
mojito-webapp: Unsupported special dependency :java

Any advice?

Fixed in https://github.com/box/homebrew-mojito/commit/90ff866bf0c6229dd563ac19f2724ef96f10c700. @OlleMattsson let me know if it working fine now.
(@muzammil-convo sorry for the late fix)

@aurambaj Brilliant! It's working nicely now. Cheers!

@aurambaj Thank you for the fix. Cheers 🍻


Do not execute the minimal-cuda-solver example

I have a problem when I run the minimal-cuda-solver example. It does not work but it also does not report an error.
I build with: cuda 10.2, g++ 7.5.0. I think It is not an error. Thank you!

minimal-cuda-solver requires input from the user, so you need to concatenate the inputs together and pipe them to the example, see the example documentation:
cat data/A.mtx data/b.mtx data/x0.mtx | ./minimal_solver_cuda

Thank you very much!


Add method to set meta indicators from data

It would be useful to have a generic df.set_meta_from_data() function, e.g.,
df.set_meta_from_data(name, method, column='value', **kwargs)

where

kwargs can be additional filters (for example apply only where variable='Temperature' or region='World')
method can be min, max, mean, median, and maybe even other pyam functions like pyam.timeseries.cross_threshold() or cumulative()
column would be what to set as meta - the actual value (default) or the year/region/... column where the value is reached (e.g., year of peak temperature)
name is the column name in meta (similar to the use in df.set_meta())

So the use case suggested by @znicholls for setting the peak temperature (see #237) could be implemented as
df.set_meta_from_data(variable='Temperature', name='Peak temperature', method='max')

closed via #241


Azure OpenAI: 2.0.0-beta.4 release (compatibility bump to OpenAI 2.0.0-beta.10)

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


Overlay: Image overlay on map

Can i do Image overlay on Map and put marker on top of it?
I dont have Tile based mechanism with my application.
I have a use case where on Selection on particular marker, i show floor map and need to put some marker on it.

You can Probably use two markers with zIndex, one with the floor map image and one with the actual marker(only if I understood your question correctly, an sample image of what are you trying to achieve  is helpful)

I think he is referring to this kind of image: https://developers.google.com/maps/documentation/javascript/examples/groundoverlay-simple
Which is not a Marker (it's size will change with the zoom level)
Is that possible?

@anupkale1984 Any luck on this?

@markmssd not yet

I don't know how to modify the react-native-maps properly, but I have hacked and hard coded it locally in my code for iOS only to make this happen. Can someone familiar with react-native-maps development please modify the codes to incorporate into react-native-maps the following:

A parameter for the image for the groundoverlay e.g. adelaidemap.jpg
The coordinates for the 2 corners of the ground overlay: south-west & north-east
See the groundOverlay parameter in the example usage below

`
    <MapView
         style={styles.mapContainer}
         provider={PROVIDER_GOOGLE}
            initialRegion={{
            latitude: -34.911703,
            longitude: 138.579694,
            latitudeDelta: 0.01,
            longitudeDelta: 0.01,
         }}

         showsUserLocation={true}

         groundOverlay={{
             groundImage: 'adelaidemap.jpg',
             south: -34.952547,
             west: 138.579217,
             north: -34.905822,
             east: 138.614837
         }}>

         <MapView.Marker
            coordinate={coordinate}
            title={'test'}
         />
      </MapView>

`

Then in the AirGoogleMaps/AIRGoogleMap.m, make the following changes:

(instancetype)init
{
if ((self = [super init])) {
_reactSubviews = [NSMutableArray new];
_markers = [NSMutableArray array];
_polygons = [NSMutableArray array];
_polylines = [NSMutableArray array];
_circles = [NSMutableArray array];
_tiles = [NSMutableArray array];
_initialRegionSet = false;
[self groundOverlay];
}
return self;
}

(void) groundOverlay
{
// use the input boundary from javascript
CLLocationCoordinate2D southWest = CLLocationCoordinate2DMake(south, west);
CLLocationCoordinate2D northEast = CLLocationCoordinate2DMake(north,east);
GMSCoordinateBounds *overlayBounds = [[GMSCoordinateBounds alloc] initWithCoordinate:southWest  coordinate:northEast];
UIImage *icon = [UIImage imageNamed:groundImage];
_overlay = [GMSGroundOverlay groundOverlayWithBounds:overlayBounds icon:icon];
_overlay.zIndex = 1000;
_overlay.map = self;
}

In the meantime, if you need to hardcode it then, you just need to grab the AIRGoogleMap.m and do what I did. Not ideal but a hack to make it work.
Unless someone can show me on how to extend the react-native-maps interface to make this happen

+1 on this issue. Any chance it can be integrated into the package for both android and iOS? Thank you very much!!
and thanks @geek2050 for the temp workaround!

+1

+1

+1

+1

+1

+1

Jun 30 — when I was last here, still trying to solve this 🤕

Duplicate of #103


Wrong domain in Locales()->Link

I have a website on multiple domains /aliases. All domains point to the same website (with a locale switcher on the website), but keep their entrance domainname in urls. I want visitors to stay on the same domain they arrived at when switching locale. But that's not happening right now:
When using $Locales inside the template to loop through the locales to create a language switcher (like the default LocaleSwitcher.ss template) $Link returns an url of the current page for that locale, but with a different domainname.
The method domain_for_locale() (line 332 on fluent.php) returns the first domainname supporting this locale it finds in fluent.yml. To me this is undesired behaviour. When a user choses to view a different language for a page, it should look for this new locale in the same domain. Only the language prefix/locale-alias/url-segments in the url should change.

After changing domain_for_locale (line 332 on fluent.php) to return $_SERVER['HTTP_HOST'], the links and everything else I can see seems to work. I don't have an overview over this module like you guys, so please let me know if I'm missing something:
public static function domain_for_locale($locale) {
        return $_SERVER['HTTP_HOST'];
}

@Friksel at a quick glance I'd assume the purpose of the domain_for_locale method is to pass in a locale string and return the domain for it. Your change will only return the domain for the locale that is currently active.

@robbieaverill  I know. The issue isn't about that function, wich obviously works as expected. But is about the locale switcher that is using that function, where I don't want it to look for a domainname, just to switch locale. The domain should remain the same, only the locale prefix in the url should change (or be added).
So let's say a user comes in at domain B. Then he tries to switch the language from English to let's say Spanish. Suddenly the user isn't on domain B anymore, because Fluent sends the user to domain A.... only because domain A is the first domain in the config having Spanish configured (while even B has Spanish too!!!)
I could be wrong, but can't think of any reason why anybody wants it to act this way.
At least it should be configurable then, I'd say.
Another thing is that in my fluent config I also configured localhost as a domain, for testing purposes on Wamp. I try to keep my config files on production server/live server the same as local to prevent mistakes. So far that's always working fine. And also here it should work that way. But it's not...
If I'm viewing the website on the production server and switch to another locale, it always switches to localhost.... wich obviously cannot be reached. That's not right. Again, I cannot think of any reason why it should look for other domains while just switching a locale.

@Friksel can you please post your entire config yml for me to review? That way I can better understand the structure of your website.
In your case, given you never want users to change domains when switching locales, I would say that you want no entries at all in your domains configuration. That way all links generated for locale-alternate versions of each page stay in the current domain.

---
Name: myfluentconfig
After:
  - #fluentconfig
---
Fluent:
  default_locale: en_US
  locales:
    - en_US
    - es_ES
  domains: [] # Leave empty
---
Name: myfluenti18nconfig
After:
  - #fluenti18nconfig
---
i18n:
  default_locale: en_US

@tractorcow thanks for your reaction and your time. I understand that if the domains-array in the config file will be left empty users stay on the same domain while switching locale. But as far as my understanding of this module config goes then it’s not possible to set different locales and default locales per domain. Or am I missing something and is it possible some way?
What I want to accomplish is setup different domains to have their own set of locales and default locales configured.
So for example: www.domain.es have some locals available for users to switch to with the locale switcher, but the website loads per default in Spanish.
The www.domain.com domain has locals available for users to switch too, but this domain defaults to English. The same with .nl to dutch and so on.
That is actually what I want. And this is working now with my workaround modification I did in the  module, as I described before. But it's obviously not very clean and I'd rather fix this without modifications to the module, to stay easely in sync with future versions of the module and not having to apply this modification to all future updates.
Would that be possible?
These are the settings I have right now for fluent:
fluent.yml inside modules _config folder:
---
Name: fluentconfig
---
Fluent:
  # Don't forget to set i18n->default_locale too!
  default_locale: nl_NL
  locales: []
  aliases: []
  field_include: []
  field_exclude:
    - 'Theme'
    - '/^LocaleFilter_/i'
    - 'Priority'
  data_include: []
  data_exclude: []
  search_adapters:
    MySQLDatabase: FluentMySQLSearch
  remember_locale: false
  persist_id: 'FluentLocale'
  persist_id_cms: 'FluentLocale_CMS'
  query_param: 'l'
  detect_locale: false
  force_domain: false
  domains: []
---
Name: fluenti18nconfig
---
i18n:
  default_locale: nl_NL

fluent.yml in mysite_config:
---
Name: myfluentconfig
After: '#fluentconfig'
---
# MAIN SETTINGS
Fluent:
  default_locale: nl_NL
  locales:
    - nl_NL
    - en_US
    - es_ES
  aliases:
    nl_NL: nl
    en_US: en
    es_ES: es
  field_include: []
  field_exclude: []
  data_include: []
  data_exclude: []
  search_adapters:
    MySQLDatabase: FluentMySQLSearch
  remember_locale: false
  persist_id: 'FluentLocale'
  persist_id_cms: 'FluentLocale_CMS'
  query_param: 'l'
  detect_locale: false
  force_domain: false
  domains:
    localhost:
      locales:
        - nl_NL
        - en_US
        - es_ES
      default_locale: nl_NL
    mytestserver.nl:
      locales:
        - nl_NL
        - en_US
        - es_ES
      default_locale: nl_NL
    mylivedutchdomain.nl:
      locales:
        - nl_NL
        - en_US
        - es_ES
      default_locale: nl_NL
    myliveenglishdomain.com:
      locales:
        - nl_NL
        - en_US
        - es_ES
      default_locale: en_US
    mylivespanishdomain.es:
      locales:
        - nl_NL
        - en_US
        - es_ES
      default_locale: es_ES
    someshadowdomainA.org:
      locales:
        - nl_NL
        - en_US
        - es_ES
      default_locale: en_US
    someshadowdomainB.nl:
      locales:
        - nl_NL
        - en_US
        - es_ES
      default_locale: nl_NL
---
Name: myfluenti18nconfig
After: '#fluenti18nconfig'
---
i18n:
  default_locale: nl_NL
---

But as far as my understanding of this module config goes then it’s not possible to set different locales and default locales per domain. Or am I missing something and is it possible some way?

I'm sorry but fluent doesn't support multiple content across different domains, as it violates the DRY rule for SEO, which is to (ideally) avoid duplicating content at multiple physical urls. If you have one piece of content, it should have one canonical URL.
If you run the validation task at https://github.com/tractorcow/silverstripe-fluent/blob/master/code/tasks/FluentValidateTask.php you'll get the appropriate error about some locales being assigned to multiple domains.
A possible solution for your use case is to simply remove all domain configuration, and have a custom script that redirects users to the correct home page URL based on domain.
This script could be in your _config.php, your home page controller, or even in a request filter to handle requests before routing.
E.g. pseudocode
if(REQUEST_URI === '/'){
  switch(domain) {
  	case 'domaina':
  	case 'domainb':
  	  return redirect('/nl');
	case 'domainc':
  	case 'domaind':
  	  return redirect('/es')
 	default:
  	  return redirect('/en');
  }
}

Ah, okay, I see now. You're right. I don't know that much about duplicated content in SEO, but you got me re-thinking the setup. I want to stay DRY. I'll probably stick to two main domains, each with unique languages then. Good to know now before the website will go live. Thanks for pointing that out!

Thanks @Friksel and hope that my feedback actually does help your progress. :)
I'll focus my attention on https://github.com/tractorcow/silverstripe-fluent/issues/200 for now, so that we can help get your site live in time. Do you have a date for this?


v0.12.1 -> v0.13.3 migration guide?

Hi, v12.1 was working perfectly with the following config:
{
  nodeIntegration: true,
  optimizeDeps: {
    include: ['fs-extra', 'obs-websocket-js', 'upath'],
  },
}

When I try upgrading to v0.13.3 there's a bunch of errors. How would I correctly migrate to the latest version?

The v0.13.x is easy to use the v0.12.x, the latest version is v0.13.14 :)
{
  nodeIntegration: true,
- optimizeDeps: {
-   include: ['fs-extra', 'obs-websocket-js', 'upath'],
- },
}

Looks like from React rather than Vite.

Sorry! upgrade to v0.13.6 to resolve it!


Update congratsbot workflow

What kind of changes does this PR include?

Something else!

Description

This PR updates the congratsbot action to use withastro/automation

size-limit report 📦

Path
Size

/index.html
13.18 KB (0%)

/_astro/*.js
16.6 KB (0%)

/_astro/*.css
9.09 KB (0%)


Issue with 00035221.npy

Hi @yuliangguo @oppo-us-research

Unable to load 00035221.npy file

Could you please help me with this file? This is stopping me from training my model with the dataset.

Hi, we are working on this, I will upload this file today, and you could continue your training. We would fix the dataset and reseed it this week.

Hi @yuliangguo @oppo-us-research

Unable to load 00035221.npy file

Could you please help me with this file? This is stopping me from training my model with the dataset.

Here is the original file.
00035221.zip

Hey guys, it seems that I don’t have any local copy of the data. I am not
sure if you can find the undamaged file on the server machine or some
desktop. If there is no luck, it would not hurt to just remove this sample
and it’s associated label from the training set. Since the training samples
are not equally sampled, where only keep those qualified ones are kept,
deleting one more does not hurt.
On Mon, Sep 12, 2022 at 5:22 PM calvindu95 @.***> wrote:

Hi @yuliangguo https://github.com/yuliangguo @oppo-us-research
https://github.com/oppo-us-research
[image: image]
https://user-images.githubusercontent.com/58805235/187569025-4635d084-abd5-4e08-bf1b-5d102ee3a260.png
Unable to load 00035221.npy file
[image: image]
https://user-images.githubusercontent.com/58805235/187569301-2fc46430-63ee-4d15-93c6-46fcb1494448.png
Could you please help me with this file? This is stopping me from training
my model with the dataset.
Here is the original file.
00035221.zip
https://github.com/oppo-us-research/PoP-Net/files/9552539/00035221.zip
—
Reply to this email directly, view it on GitHub
https://github.com/oppo-us-research/PoP-Net/issues/5#issuecomment-1244737975,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADFMXC7C52FVWYFDGEQ3I2TV57CMLANCNFSM6AAAAAAQA5AB7A
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Shreyas-NR  Hi. Could I ask whether you still have a copy of this dataset locally? Since the company has no maintainess of the downloading, could I ask you a favor to share the dataset back to us?

@yuliangguo, Sorry for the delayed response.
I have the copy of dataset with me.
I'm unsure of the best way to share it. Could you please let me know the same?
Best regards,
Shreyas

Hi Shreyas,
Thanks a lot for your reply. Since the dataset has Apache Licence 2.0,
which allows it to be distributed, the best way should be uploading the
dataset to huggingface directly.  When you have uploaded the dataset there,
you could either let us know the link, so we can directly refer to it for
everyone else to download from. Or you could transfer the ownership to me,
my huggingface account name is "yuliangguo". As long as we know,
huggingface can host dataset free, as long as it is made public.
We will truly appreciate your help.
On Sun, Oct 20, 2024 at 4:59 PM Shreyas Narasimhiah Ramesh <
@.***> wrote:

@yuliangguo https://github.com/yuliangguo, Sorry for the delayed
response.
I have the copy of dataset with me.
I'm unsure of the best way to share it. Could you please let me know the
same?
Best regards,
Shreyas
—
Reply to this email directly, view it on GitHub
https://github.com/oppo-us-research/PoP-Net/issues/5#issuecomment-2425302193,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADFMXC7EL2B4IBQTS3QTPODZ4Q7XRAVCNFSM6AAAAABQEUQVYSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRVGMYDEMJZGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

@yuliangguo
Each file size is 22.5GB,
I'm unable to upload them on huggingface
Any alternative solution?

@yuliangguo Each file size is 22.5GB, .zip file type
I'm unable to upload them on huggingface Any alternative solution?

Then it seems really complicated. Based on their rule https://huggingface.co/docs/hub/en/repositories-recommendations
They can host 300GB with 20GB per file. Could you try emailing<EMAIL_ADDRESS>as they said it might give you some alternative way.  I am really sorry to put such extra effort on your side.


Offline app: Constantly says "network request failed"

Happens regardless of if you are offline or online. I think it happens more often when offline. Might be related to the simulator or the /api/pkg endpoint.


cannot retrieve image in blade

Laravel Version: #.#.#
Voyager Version: #.#.#
PHP Version:
Database Driver & Version:

Description:
Steps To Reproduce:

@foreach($homeslides as $homeslide)

{{$homeslide->title_white}}
{{$homeslide->title_red}}
{{$homeslide->description}}
DONATE NOW

@endforeach
I got all the data expect $homeslide->cover_image. There is no mistake in spellings

Please fill the requested versions.
What happens if you debug the attribute like $homeslide->cover_image ?

Thanks for replying. I forgot to add storage/  . Previously it was   . The issue was solve with  ""

I think you are looking for something like <img src="{{ Voyager::image($homeslide->cover_image) }}" />

Thanks. that worked too & seems better. I'm new to Voyager and starting to love it and its community.

The next time please fill out the version numbers and describe your problem in more detail.


Dropdown - allow for animation duration customization

@juujisai Any update

@RabbiIslamRony We'll try to add this with the next release (04.12.23) but there are still some tasks that we have to take care of.


KMS region incorrectly defaults to us-east-1 when using a profile

When the credstash region is set through an AWS profile, the KMS key region is incorrectly set to the default value of us-east-1 instead of the region in the profile.

This turned out to be a configuration error.


Feature/footer

Adds a generic Footer to the web-app. This does not have a ClickUp for it.
Description
How has this been tested?
Types of changes

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

Checklist:

[ ] The related ClickUp task has been linked to this PR
[ ] The person creating the pull request is listed in "Assignees"
[ ] Change requires updated documentation

Screenshots or Loom Video (optional):

Task linked: CU-86cua9w09 UI: Footer


Unable to connect to the service using curl commands outside kubernetes cluster using Ingress

I have created a Helm chart with 2 services where one services using other as a back-end data base.
I have deployed the same and the installation went fine but i am not able to check the content in the service using curl command when connected outside the kubernetes cluster (from my CLI where the Kubeconfig and Helm commands are executed)
My ingress.yaml file looks like
spec:
rules:

host: {{ .Values.ingress.hostname }}
http:
paths:

path: /
backend:
serviceName: {{ .Values.ingress.serviceName }}
servicePort: 5003

Hostname and service name are defined as below:
hostname: x-postgresql-y.a.b.c.com (This is the address)
serviceName: x-postgresql-y
Curl command used to access the service is as below
curl -D - -o - -sS -X GET http://x-postgresql-y.a.b.c.com -- I am always getting the error connection refused on port 80..
Not sure if some configuration is wrong or something is missing.

Are you sure that your ingress controller is up and working? What's in its logs? Are there any errors? In which way is it being exposed to the outer world?

Hi Thanks for the response.
Currently I don't have the required info.. will quickly get back with the logs..

kubectl get all,ingress -n cm-kamesh-namespace
NAME                                   READY     STATUS    RESTARTS   AGE
po/app-postgresql-75c6c9cbff-9cmqh      1/1       Running   0          16h
po/app-postgresql-cm-59789c8764-k2xfk   1/1       Running   0          16h
po/app-postgresql-cm-59789c8764-lzjf8   1/1       Running   0          16h
NAME                            CLUSTER-IP       EXTERNAL-IP   PORT(S)          AGE
svc/app-postgresql               <IP_ADDRESS>           5432/TCP         16h
svc/app-postgresql-cm            <IP_ADDRESS>          5003:32161/TCP   16h
svc/app-postgresql-cm-headless   None                     5003/TCP         16h
NAME                      DESIRED   CURRENT   UP-TO-DATE   AVAILABLE   AGE
deploy/app-postgresql      1         1         1            1           16h
deploy/app-postgresql-app   2         2         2            2           16h
NAME                             DESIRED   CURRENT   READY     AGE
rs/app-postgresql-75c6c9cbff      1         1         1         16h
rs/app-postgresql-app-59789c8764   2         2         2         16h
NAME                   HOSTS                                             ADDRESS   PORTS     AGE
ing/app-postgresql-app   app-postgresql-app.messenger.seli.gic.io.com             80        16h
Hi,
Above is the ingress details and all the pods are up and running.I am deploying a chart that contains 2 subscharts.One is postgresql and other is my application.
This is the address app-postgresql-app.messenger.seli.gic.io.com  i am using for accessing the REST calls and the same is mentioned as the hostname in my ingress.yaml file.
I dont see any specific error above.
Please can you help me out..

Closing due to inactivity. Please open a new ticket against Helm 3 if this issue persists. Thanks!


Oak report fixes.

1. Overview
Updating some logs and optimisations for gas consumption.
2. Implementation details

3. How to test/use

4. Checklist

[ ] Does the Readme need to be updated?

5. Limitations (optional)

6. Future Work (optional)

can you add the report numbers that are fixed in this PR?


Is there a way to run a test for a set amount of time at a constant rate?

Hi,
I was looking to do a sustained load test and I was wondering if there is a way to set a constant stream for a defined period?
Sorry if this is basic, I am very new to K6.

please disregards, i found out how to complete this


Fix/change c2d orders order

Fixes # .
Changes proposed in this PR:

order asset with provider fees last so that transaction is the last and we can use the block to validate c2d resource
add proper digest values to our default algorithm container images

closing this PR since it's handled in #1615


Store the protocol headers if they need initialized

Currently, if SamlHeaderOutInterceptor finds no protocol headers in the message, it still proceeds to create the assertion, but it stores it in a new map which is not stored anywhere. The assertion doesn't appear in the HTTP headers output by CXF...
Simply storing the created map as the message's protocol headers fixes this.

Patch has been applied, thanks


Trying to fix windows-install.sh to fix #115

Was trying it out today, but got some line error issues. Turns out windows-install.sh is from a recent PR #113.
Detailed my issue in #115. Basically some lines seem to be replaced by > angle bracket? Possibly merging issue? But then @notAperson535 confirmed the PR to be working on his side.
So all in all, I was unsure why the original PR included that, but here's a version that worked perfectly on my side.
Also fixed the fallback for tagging, since for example: https://github.com/microsoft/WSL2-Linux-Kernel/releases/tag/<IP_ADDRESS> is invalid. If successful, latest produce the full tag: linux-msft-wsl-<IP_ADDRESS>, so the fallback should follow the same format.
Sorry if I'm wrong.

Summary by Sourcery
Fix errors in windows-install.sh by removing an incorrect angle bracket in the package installation command and updating the fallback version tag format to ensure compatibility with the expected kernel versioning.
Bug Fixes:

Correct the installation command in windows-install.sh by removing an erroneous angle bracket that was causing line errors.
Fix the fallback version tag format in windows-install.sh to ensure it matches the expected format for kernel versioning.

@ArsenicBismuth It worked for me because I already had previous dependencies installed, so it still worked even with the messed up angle bracket. Thanks for catching this so quick, I’ll merge it now

@notAperson535 I actually found and tried your script just today, so I didn't even know it's caused by a recent PR. Good to know it's working!


Request: Add NSTextField class

Please add support for NSTextField. Thank you.

Why not just everything? This might be a good time to remind you that you can use any class or API without bindings using the objc package.

You are probably talking about using objc.NewClass(). I did make this example program as a start but for some reason it keeps panicking. Do you see anything wrong with it?
package main

import "github.com/progrium/macdriver/cocoa"
import "github.com/progrium/macdriver/objc"
import "fmt"

func main() {
	app := cocoa.NSApp()
	cls := objc.NewClass("NSTextField", "NSControl")
	cls.AddMethod("labelWithString:", func(message string) {
		fmt.Println("Message:", message)
	})
	objc.RegisterClass(cls)
	app.Run()
}

Here is the panic:
panic: unable to AllocateClassPair
I traced the error to objc/class.go to this code:
func NewClass(classname string, superclass string) Class {
	superClass := GetClass(superclass)

	ptr := allocateClassPair(unsafe.Pointer(superClass.Pointer()), classname)
	if ptr == nil {
		panic("unable to AllocateClassPair")
	}

	// Register the dealloc method to be able to properly remove the classInfo
	// reference to our internal pointer.
	classAddMethod(ptr, "dealloc", encVoid+encId+encSelector)

	lazilyRegisterClassInMap(classname)
	return object{ptr: uintptr(ptr)}
}

It looks like allocateClassPair() is the problem. For some reason it doesn't like NSTextField.


Serverless documentation

Continues work done in https://github.com/Qiskit/documentation/pull/1770

Preview link

Frank is helping me sort out the problems with the nb test failures. Thanks Frank!

Marking as "ready to review" because the content is ready for feedback. Still getting test-notebook check fails.

FYI - Please don't merge yet - I'm integrating Sanket's other changes and have a note to incorporate. Once I've done that I will merge. Thank you!


rename versioning folder

Since the utils and folder include functions that are useful outside of the versioning, we can rename the package and path and the filenames related to versioning to accurately represent their purpose.
rename scipts/ modify paths and paths in yml files pointing to those as neccessary

Suggested names: package-utils, packaging

@KarishmaGhiya, Can you share details on which utils/scripts folder are you referring to here?

Closing this because I don't think there is any outstanding work


[Migrated] Retries and timeout for dependencies.

Originally from: https://github.com/Miserlou/Zappa/issues/1316 by Tipuch
Description
Added retries and timeout optional parameters in the cli, a retry scheme when downloading dependencies and a different exception handling when downloading packages.
GitHub Issues
https://github.com/Miserlou/Zappa/issues/1235
https://github.com/Miserlou/Zappa/issues/1040

not enough info.
closing.