Query with math returns field with no name

Using math on a field returns a field with an empty name.  If this used used in a continuous query or select into, it the points can't be inserted and causes some of the issues seen in #5697.
To repro:
> select value from cpu
name: cpu
---------
time			value
1455686412499134533	1
1455686413602752226	2
1455686415066866003	3

> select value * 10 from cpu
name: cpu
---------
time
1455686412499134533	10
1455686413602752226	20
1455686415066866003	30

Notice the second example is missing a value column header.
Selecting multiple columns doesn't fix it.
> select value * 10, value * 2 from cpu
name: cpu
---------
time
1455686412499134533	10	2
1455686413602752226	20	4
1455686415066866003	30	6

Using an as alias does work though:
> select value * 10 as scaled from cpu
name: cpu
---------
time			scaled
1455686412499134533	10
1455686413602752226	20
1455686415066866003	30

@jsternberg

I found where the error is coming from. What should the column name be where there are multiple fields in the binary expression?

For something like?
select user / total from cpu

Maybe concatenate the fields w/ _?  So that query would return user_total.

Sure! Should we remove duplicates? If we have SELECT (user + total) / total FROM cpu should it be user_total_total, total_user, or maybe user_total?

I would probably go with user_total_total or user_total since they are in the order that they were defined in the query.  No strong opinion though.

I'll go with user_total_total for now since the implementation is much easier.


Moneropedia links (created by _plugins/moneropedia.rb) are always lower case

What's the expected behavior? Should they all be capitalized? From what I can tell, the Moneropedia plugin just preserves the capitalization of the terms provided in each Moneropedia entry, and the majority of them are lowercase. Several of them (Locally-unique host, Canonically-unique host, and Kovri) are capitalized and are displayed as such.

Should they all be capitalized?

They should be capital when at the beginning of a sentence or after a dot, lowercase if the word is in the middle of a sentence.

What do you think about using the matched term instead of the entry term in the gsub call? The capitalization could then be decided on a case-by-case basis instead of being dictated by the Moneropedia entry or any preceding punctuation.
content = content.gsub(/(\@#{term})((?=-#{lookahead})|(?![\w-]))/i) { |match_term| '' + match_term.gsub(/[@-]/, '@' => '', '-' => ' ') + '' }

This would help with situations where the Moneropedia links aren't at the beginning of a sentence such as the video section under the FAQ:

The downside to this, however, is that any existing capitalization errors would need to be fixed manually. For instance, changing @ring-ct to @Ring-CT in the example above.

Good idea. I was slightly worried about how the other languages would have reacted (especially the ones not using latin characters), but i tested your patch quite extensively and there seem to be no problems.
Would you PR your patch? As you said, capitalization errors are not a big deal and we can fix them when we find them.


Add support for reference types in the API

Bind the new support in Wasmtime for reference types in the Go API as
well.

"Some checks were not successful" seems like an understatement :)
Very excited for this!


Add Steam installation

Game developers may use it.
-> This is an extra install.

Decision to just add FOSS extras.


Could you store config file in ~/.kube or ~/.config instead?

Hello! I don't like having too many dotfiles in my home directory.
I see that you only read the config from ~/.kubesafe.yaml:
https://github.com/Telemaco019/kubesafe/blob/3406ce756969374ea6faf12c2004581fa3afcbad/internal/repositories/repository.go#L34-L43
Could you change it to instead store it in the ~/.kube directory? Such as ~/.kube/kubesafe.yaml
To not break compatibility, your tool could look if ~/.kubesafe.yaml already exists, and then use that first and foremost. But then otherwise default to ~/.kube/kubesafe.yaml

My bad, I started with the home directory for simplicity and then forgot to switch it to ~/.kube 😅.
In the next release, I'll set ~/.kube as the default path for the settings file, with the home directory as a fallback for backward compatibility.
Thanks a lot for catching that!

There is also the XDG standard for home directory config files.  This article gives a gentle intro to the standard.

There is also the XDG standard for home directory config files. This article gives a gentle intro to the standard.

That's interesting, thanks for sharing it @chicks-net!

Kubesafe now follows the XDG standard, as highlighted by @chicks-net. For backward compatibility, it first checks for the old file at ~/.kubesafe.yaml. If not found, it defaults to the new path: ~/.config/kubesafe/config.yaml.

Btw in Go there's the os.ConfigDir which has better crossplatform support. Such as XDG config on Linux, AppData on Windows, etc.

That's even better, thanks for the heads up @applejag! I've just refactored it using os.ConfigDir, so now the path ~/.config/kubesafe is used only for UNIX systems.


Persist failed payment amount

Persists the last failed payment amount after the sync as the Greenlight amount_msat is 0 until status is completed.
Fixes #551

Think I need to consider how this will be remotely synced also

Think I need to consider how this will be remotely synced also

If the amount is needed to be synced then it should be written to the sync db such as when we write to payments_external_info table


FancyZone ERROR

Microsoft PowerToys version
0.45.0
Running as admin

[X] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
FancyZones  "win + ctrl + alt"** isn't working
✔ Expected Behavior
No response
❌ Actual Behavior
I want use FancyZones "win + ctrl + alt" but it's not working
Other Software
No response

Hi,
I have the same issue.
The shortcut WIN + CTRL + ALT + Number to quickly switch between layouts is not working anymore.
My version is 0.47.0

Thanks for reporting!
This is a duplicate of https://github.com/microsoft/PowerToys/issues/13585.


Unable to load Flux.1's T5 tokenizer

When I try to load the FLUX.1's T5 tokenizer, I keep getting an error: Tokenizers/Normalizer.swift:56: Fatal error: Unsupported Normalizer type: Strip.
Here is the code to reproduce the error:
import Hub
import Tokenizers

let repo = Hub.Repo(id: "black-forest-labs/FLUX.1-schnell")
let hub = HubApi()
try await hub.snapshot(from: repo)
let directory = hub.localRepoLocation(repo)

let tokenizerConfig = try? hub.configuration(fileURL: directory.appending(path: "tokenizer_2/tokenizer_config.json"))
let tokenizerVocab = try hub.configuration(fileURL: directory.appending(path: "tokenizer_2/tokenizer.json"))

let tokenizer = try AutoTokenizer.from(tokenizerConfig: tokenizerConfig!, tokenizerData: tokenizerVocab)

let inputIds = tokenizer("Today she took a train to the West")

I am wondering if swift-transformers supports the T5 tokenizer, or if I have missed something.

It looks like the normalizer is missing a Strip type, which the Flux.f tokenizer has in its configuration.


[release-v1.15] Create configmap for starting dispatcher pods (#4051)

Cherry-pick of https://github.com/knative-extensions/eventing-kafka-broker/commit/9730d529b2a9d26debbafc7f3f37da1a193d1c74

ERRO[2024-08-20T09:03:28Z]
  * could not run steps: step [release:latest] failed: failed to get CLI image: unable to wait for the 'cli' image in the stable stream to populate (waited for 5m5.013614732s): timed out waiting for the condition
  * ```

/test 415-images

/retest-required


bump cloudsql-proxy to latest patch version

bump cloudsql-proxy to latest patch version

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

/retest


Move testing related function to a dedicated lib + move unit tests

What does this PR do?
Move some method of the testwasher to a dedicated testing/flakes.py file, so that it can be reused elsewhere easily.
Use is_known_flaky_test method when determining the tag we put on junit uploads.
So we know that the logic for the test tagging is the same as the logic we use in testwasher to determine that a test is known as flaky
Motivation
Make sure tagging is more robust so the workflow does not notify test failure on already known flakes
Describe how to test/QA your changes
Possible Drawbacks / Trade-offs
Additional Notes

/merge


[Question] How to upgrade from List to this Nested List plugin?

We have already created some documents using the List feature but as our requirements evolved. We would like to implement this plugin but we apply the changes for it's config the previous data that we have resulted to "undefined"

We have different data structures for List and Nested-list plugins, we would implement convertor in Nested-list 2.0 version (we are working on it now), so it would be possible to use Nested-list with List data format

It would be great if List could be upgraded to NestedList for existing projects, keeping backward compatibility.
This would be useful mainly because List has some annoying bugs and it is no longer maintained


fix f767zi flash script

Every script in this branch is kinda wrong, they are copypasta of olimex version.

Where is broken the CI pipeline?

as you can see, yesterday when I made the pull request, and it didn't pass all the checks (the one for creating the microros client for the zephyr version of f743zi). then I commit a second time, adding the script for flashing the nucleo f746zg (cause it was again a copypast of the olimex setup), and all checks passed. thanks for the review @pablogs9


Bug: scipy.sparse.linalg.spsolve triangular: b with int dtype

scipy.sparse.linalg.spsolve triangular is very inaccurate if b is of dtype int.
The problem is that in this case an array of dtype int is used as solution vector. I change this to an array of dtype float. If b is of dtype int and overwrite_b is True instead a warning is issued.
(I also removed some PEP8 problems.)

I think an error should be raised with overwrite_b=True. The check also
probably should be not np.issubdtype(b.dtype, np.inexact).
.
astype(float) is suspect for complex-valued inputs. The tests probably
should be amented with complex-valued inputs.
.
The proper upcast is likely if not np.issubdtype(b.dtype, np.inexact):  b = b.astype(float).

Also, the result array dtype selection probably should consult
np.result_type, in case A and b have different types.

@pv: I incorporated your comments.
Maybe also an exceptions should be raised if A has dtype complex and b has dtype float and overwrite_b is True?

Yes, maybe raise exception if not np.can_cast(A.dtype, b.dtype) with
overwrite_b=True...

Any further comments? Should I squash the commits?

You can use np.can_cast(..., casting='same_kind'), which should be able to replace both dtype checks.
Don't worry about squashing commits, because Github allows auto-squashing when merging a PR.

@perimosocordiae: Your suggestions are now incorporated.

Thanks, merged.


Fix single source gen-anc frequencies on the Variant page

Resolves #1545

Tagging you in for review, @ch-kr, in addition to Phil, because I wanted to check on one thing display decision wise.
The v4 exomes and genomes have identical genetic ancestry groups with the exception of ami, which is included presumably because the data was already there from v3 genomes. When both boxes are selected, or just the genomes box is selected, the table will include ami with the respective values there.
When the genomes checkbox is unselected, and only the v4 exome data is shown, should the ami row in the table disappear? The alternative would be to include a row for ami and populate it with AC: 0, AN: 0, AF: -
I'm not sure there's any value to showing it if it's always AC and AN of 0, but also including it keeps it more consistent.
Either is equally doable, I wanted to check on your opinion of what is more clear/preferable.

Thanks for the ping!

When the genomes checkbox is unselected, and only the v4 exome data is shown, should the ami row in the table disappear? The alternative would be to include a row for ami and populate it with AC: 0, AN: 0, AF: -

Quick question before I answer: based on this variant, it looks like the ami row currently exists for variants only present in the exomes. Does this question apply to these variants in addition to when a user explicitly selects or unselects the data type checkboxes?

Thanks for the ping!

When the genomes checkbox is unselected, and only the v4 exome data is shown, should the ami row in the table disappear? The alternative would be to include a row for ami and populate it with AC: 0, AN: 0, AF: -

Quick question before I answer: based on this variant, it looks like the ami row currently exists for variants only present in the exomes. Does this question apply to these variants in addition to when a user explicitly selects or unselects the data type checkboxes?

Gah ok yeah I hadn't thought of that, good catch.
The question specifically was for behavior when the variant is present in both exomes and genomes.
Now I suppose there's another question of what to do for the variants that are not present in one data type, and thus don't allow de-selecting the type its not in. My initial thought there is leave it untouched, e.g. the example variant still includes the genome AN data? But also then if there's joint data maybe it makes sense to allow users to remove the ANs from the other data type that contribute to the overall AN per ancestry group.

thanks for clarifying! I think the most consistent behavior is to continue displaying the ami row both for variants only present in the exomes and for variants present in both data types (and where users can choose to unselect the genomes data). This reinforces that the v4 dataset is comprised of both exomes and genomes and also matches the behavior seen when toggling between the full dataset and a subset. For example, here's an example v3 variant -- the ami AC is 1 when all of v3 is selected and becomes 0 when the v3 non-TOPMed subset is selected.
On a slightly related note, this question sort of ties into this slack thread -- we should likely display the genetic ancestry group sample count/counts of each sex karyotype per group for v3/v4 on our stats page (across all groups, without any merged groups, which is what the table on the stats page currently contains)

thanks for clarifying! I think the most consistent behavior is to continue displaying the ami row both for variants only present in the exomes and for variants present in both data types (and where users can choose to unselect the genomes data). This reinforces that the v4 dataset is comprised of both exomes and genomes and also matches the behavior seen when toggling between the full dataset and a subset. For example, here's an example v3 variant -- the ami AC is 1 when all of v3 is selected and becomes 0 when the v3 non-TOPMed subset is selected.
On a slightly related note, this question sort of ties into this slack thread -- we should likely display the genetic ancestry group sample count/counts of each sex karyotype per group for v3/v4 on our stats page (across all groups, without any merged groups, which is what the table on the stats page currently contains)

Perfect, we'll go with that behavior, thanks so much for the reply and opinion/decision.

Alrighty, added some fixup commits to address comments from code review. Thanks, as always, for your review!
Re-requested you @phildarnowsky-broad


Migrate OpenStack playbooks to use "os_*" modules

Maybe we should wait until the os_* modules are flagged as stableinterface.


Fancier navigation

[ ] Display the page title in the navigation bar
[ ] Fuzzy search
[ ] Enter to submit

Is this a todo for launch, or something we can push to backlog?
Also, could you expand on fuzzy search?

Although, those features should not take tooo long

Is this a todo for launch, or something we can push to backlog?
Also, could you expand on fuzzy search?

We can totally push it to the backlog.
The simplest variant of fuzzy search basically just checks if words are similar enough using the https://en.wikipedia.org/wiki/Levenshtein_distance
Of course fancier, better variants are also an option https://stackoverflow.com/questions/23305000/javascript-fuzzy-search-that-makes-sense

Copied from Pull Request: I have not found a feasible way to make enter also navigate to the page thats selected.
For now I'm happy with how it work, if someone has an idea please let me know.
For now I'll extract it to its own issue and move it to backlog


typo: omited -> omitted

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-pnp/Set-PnPField.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


using polar transform for feature map

is it ok to use this library for multi-channel input other than images (eg: feature maps)?
thank you!

I can't comment specifically since I haven't used this library in a long time.
However, conceptually, it seems you should be able to do that. You'll need to make sure the feature map shape is such that the two Cartesian/polar axes are in the correct place.

thanks for noting, I will try it out!

Hello @addisonElliott, using the defaults, should we expect concentric circles in the polar image (first one)? This above has pieced / flattened circles.


[Settings] Shortcut guid: warning not greyed out

Microsoft PowerToys version
0.56.0
Running as admin

[ ] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce

Set activation mode for scg to win key.
Disable scg.

✔ Expected Behavior
Warning is hidden or disabled.
❌ Actual Behavior
Warning is visible:

Other Software
No response

cc: @niels9001

Good catch, thanks!

ping @niels9001 . do you missed this?

@Jay-o-Way , @niels9001
If you want me to fix this please let me know.
@Jay-o-Way
If you like to try fixing this and need some code examples you can look at #20184.

Related to #20173

Related to #20173

Sure. But I am not a friend of making things more complicated as needed. What I mean here is that we can fix this issue in a first step by hiding the infobar and then revisit the whole behavior/layout of the settings ui later.


seem having syntax error with nim version 1.5.x

hello,
while running it I immediately got error :
nimble run vim3
     Error: Could not read package info file in /home/francois/tmp/vim_cubed/vim3.nimble;
        ...   Reading as ini file failed with:
        ...     Invalid section: .
        ...   Evaluating as NimScript file failed with:
        ...     ../nimscriptapi_1323085986.nim(74, 3) Error: identifier expected, but got '('
        ... printPkgInfo() failed.

so I check my nim & nimble versions & ensure to get the last stable one then
nim --version
Nim Compiler Version 1.5.1 [Linux: amd64]

nimble --version
nimble v0.12.0 compiled at 2020-11-13 11:53:31

but error remain the same.

Did you build nim from source? The lastest stable version is 1.4.0

I'm having the same issue but on 1.4.2. I used snap to install the latest version.
nimble run vim3
     Error: Could not read package info file in /home/yang/Documents/Libraries/vim_cubed/vim3.nimble;
        ...   Reading as ini file failed with:
        ...     Invalid section: .
        ...   Evaluating as NimScript file failed with:
        ...     error: unknown command "e", see 'snap help'.
        ... printPkgInfo() failed.

nim version:
Nim Compiler Version 1.4.2 [Linux: amd64]
Compiled at 2020-12-02
Copyright (c) 2006-2020 by Andreas Rumpf

git hash: c5c9e12b5db6951342877feb3dae198c090c0ab6
active boot switches: -d:release

nimble version:
nimble v0.12.0 compiled at 2020-12-02 04:32:24
git hash: 8f7af860c5ce9634af880a7081c6435e1f2a5148

Update: my issue seems to be with snap and nimble, not this repo specifically.


Support restricting a function in certain modules

It's possible to restrict a function in all modules except certain modules:
- functions:
  - {name: [fromIntegral], within: 'modules_where_fromIntegral_is_ok'}

But it doesn't seem possible to restrict a function only in certain modules. This would be a useful feature to add.

So you declare a module where fromIntegral is banned, but by default it's allowed in most places? Seems a reasonable extension. Could be another field, or alternatively a prefix ! in the within field to indicate it?

So you declare a module where fromIntegral is banned, but by default it's allowed in most places?

Exactly. fromIntegral is generally OK, but in certain modules it is unsafe and we want people to use toIntegralSized, which returns Nothing upon overflow/underflow.

Could be another field, or alternatively a prefix ! in the within field to indicate it?

Yeah, something like that. Happy to open a PR

Cool, PR welcome - ! prefix seems reasonable to me, but it's probably not the only thing that is reasonable.

As part of ZuriHack I'm taking the :lock: on this issue :)

In process of researching it but: ! has special meaning in yaml for tags, so it might be that using it will require putting the module name(s) in quotes. Another way would be to add a separate key to put the exceptions in, e.g disallow.
We can bikeshed this while I look into where to add the part where the rule is enforced.

Quoting it should work, I think - as in within: "!Foo".
Or we can use a word that is the opposite of "within" - perhaps "outside"?

This would be nice, for large codebases you might have the capability to keep one part of your codebase clean of a particular function, and you want to list that subset of modules rather than listing out everything else.


made encryption config work

Made the encryption option in the config actually turn on and off encryption.
Also cleaned up some of the encryption response handling.

Looks good then, We should keep in mind that encryption is needed when online mode is enabled. Otherwise its Optional


Warning: A props object containing a "key" prop is being spread into JSX

hi!
I have issue in NextJS 13
this line:
{...gauge.getTickProps({ angle, length: showText ? 12 : 6 })}
in arced.tsx
this error:
Warning: A props object containing a "key" prop is being spread into JSX:
let props = {key: someKey, className: ..., strokeWidth: ..., x1: ..., x2: ..., y1: ..., y2: ...};
<line {...props} />
React keys must be passed directly to JSX without using spread:
let props = {className: ..., strokeWidth: ..., x1: ..., x2: ..., y1: ..., y2: ...};
<line key={someKey} {...props} />

Can you please post a reproduction or link to a repository where this is happening?

Thanks for your answer and for your interest in the problem. I created a new clean repository and implemeted the chart.
The issue have in console VScode. This is the link https://github.com/CodeCup7/chart-test
Thanks

The key issue (which is Next.js specific) should be fixed in 0.7.0.

Super, because this chart is awesome :)


Adding Linux setup instructions to Readme.md

Hi, I'm a Linux user, and would like to add Linux setup instructions under the mac instructions in the Readme. Would this be a helpful addition? Or do you find this unnecessary? I will admit the number of beginner programmers using Linux is pretty small.

I think it would be a helpful addition. With the exception of how you download Postgres, is there any difference from the Mac instructions?

Starting Postgres is different. You also have to enable Postgres as a system service, to make sure it will automatically start itself after a reboot.

Ok, so I agree that Linux users are not going to be super beginners that need to be hand-held through each setup step. If you want to write a Linux section, I would think it would be enough to just highlight the differences and point out the important bits like the environment variables.
Feel free to open a PR and I'll review it.

Ok, sounds like a plan. I'll just have them refer to the steps in the mac section, once the DB is created. I'll submit my PR sometime this evening/tonight.


docker-nydus-graphdriver: avoid `overlayfs: overlapping path'

For more details, see<EMAIL_ADDRESS>Signed-off-by: Gao Xiang<EMAIL_ADDRESS>

So current overlayfs mount behavior is not defined per as<EMAIL_ADDRESS>?
Can you help me understand how the overlapping happens around there?


Use This Issue for the thread of your Queries.

Write your queries here.
I'll try my best to Help you.

Please accept my PR

i have made a pr on https://github.com/Padmanabh82/Hacktoberfest-2021.git but my project has added as submodule how i don't know. will it work ??/ and also accept my pr

i have made a pr on https://github.com/Padmanabh82/Hacktoberfest-2021.git but my project has added as submodule how i don't know. will it work ??/ and also accept my pr

Delete your repo and fork again..and add your projects..let me know if you get any problem.

i have made a pr on https://github.com/Padmanabh82/Hacktoberfest-2021.git but my project has added as submodule how i don't know. will it work ??/ and also accept my pr

Delete your repo and fork again..and add your projects..let me know if you get any problem.

sir i don't know but it does't accpt folder to uplode. when i use git it made a submodule

i have made a pr on https://github.com/Padmanabh82/Hacktoberfest-2021.git but my project has added as submodule how i don't know. will it work ??/ and also accept my pr

Delete your repo and fork again..and add your projects..let me know if you get any problem.

sir i don't know but it does't accpt folder to uplode. when i use git it made a submodule

well for now try uploading files through GitHub site and generate pr from here .

Please accept my PR

done

Please accept my PR

done

thanks

i have made a pr on https://github.com/Padmanabh82/Hacktoberfest-2021.git but my project has added as submodule how i don't know. will it work ??/ and also accept my pr

Delete your repo and fork again..and add your projects..let me know if you get any problem.

sir i don't know but it does't accpt folder to uplode. when i use git it made a submodule

well for now try uploading files through GitHub site and generate pr from here .

Sir when I upload from web it doesn't accept folder only accept file but my project is in react

My project :- https://github.com/Padmanabh82/TextUtils-React

I will try from my mobile

I cannot upload folders

Only file

What to do sir ??

accept my PR #133
i am beginner in this can you please check that this is created or not..

accept my PR #133 i am beginner in this can you please check that this is created or not..

done

I cannot upload folders

well the try to methods that i have mentioned in readme section. You can upload folder by Drag and drop method.Google it for more info.

I cannot upload folders

well the try to methods that i have mentioned in readme section. You can upload folder by Drag and drop method.Google it for more info.

sir i have made a pull request with anther project:- https://github.com/TAbhay/Hactoberfest-2021/pull/446
it is working. please accept it

I cannot upload folders

well the try to methods that i have mentioned in readme section. You can upload folder by Drag and drop method.Google it for more info.

sir i have made a pull request with anther project:- TAbhay/Hactoberfest-2021#446
it is working. please accept it

that's not my repo. It's owned by someone else.Create a pr on my repo I'll Merge it.

I cannot upload folders

well the try to methods that i have mentioned in readme section. You can upload folder by Drag and drop method.Google it for more info.

sir i have made a pull request with anther project:- TAbhay/Hactoberfest-2021#446
it is working. please accept it

that's not my repo. It's owned by someone else.Create a pr on my repo I'll Merge it.
please accept:- https://github.com/vijaygupta18/Hacktoberfest-2021/pull/134
and issue was of other repo

I cannot upload folders

well the try to methods that i have mentioned in readme section. You can upload folder by Drag and drop method.Google it for more info.

sir i have made a pull request with anther project:- TAbhay/Hactoberfest-2021#446
it is working. please accept it

that's not my repo. It's owned by someone else.Create a pr on my repo I'll Merge it.
please accept:- #134
and issue was also of other repo

done

I cannot upload folders

well the try to methods that i have mentioned in readme section. You can upload folder by Drag and drop method.Google it for more info.

sir i have made a pull request with anther project:- TAbhay/Hactoberfest-2021#446
it is working. please accept it

that's not my repo. It's owned by someone else.Create a pr on my repo I'll Merge it.
please accept:- #134
and issue was also of other repo

done

now will counted in hactober fest ??

Can you plese accept my pr in bubble sort , I've made the code much shorter and it works fine

please merge #149 . in this all the sorting algorithm is implemented in c

@harshvardhan56  can you describe how to re merge it ? Also I made the code of heap sort.java lesser and also merged it

please merge #149 . in this all the sorting algorithm is implemented in c

done

If you have merged then why can't I see any progress on Hacktoberfest page ;)
try reloading the page..or create pr again I'll approve it

I'll create pr again as reloading the page still shows my status as 0 in progress

I'll create pr again as reloading the page still shows my status as 0 in progress

okay do it... avoid any conflicts

Even after you have merged my progress still shows 0 , I don't know what to do smh .

Even after you have merged my progress still shows 0, I don't know what to do smh.

it's a server error..wait for some time it will get updated

Please accept my PR.

In order to make a contribution, first i should create an issue ,work on it and then raise a pr? Is that it ? Or can i directly raise a pr without creating issue?

in my PR contribution it's showing that hacktoberfest accepted
but matures in 14 days?
what does it mean?

in my PR contribution it's showing that hacktoberfest accepted but matures in 14 days? what does it mean?

this means your pr will be verified by the e community after 14 days.

what are the chances of acceptances?
i have just contributed in dsa (c++)

what are the chances of acceptances? i have just contributed in dsa (c++)

if get merged in hacktober-accepted repo then valid.

what are the chances of acceptances? i have just contributed in dsa (c++)

if get merged in hacktober-accepted repo then valid.

Firstly Thank you @vijaygupta18 for accepting my PR.
But it seems that there is a lot of spam happening everywhere and many repositories are getting the message "Excluded from project".

Even after you have merged my progress still shows 0, I don't know what to do smh.

it's a server error..wait for some time it will get updated

Ya it takes much time to load

I would like to create a Dyanamic Programming folder using C++. Can in  start working on it.

I would like to create a Dyanamic Programming folder using C++. Can in start working on it.

sure

what are the chances of acceptances? i have just contributed in dsa (c++)

if get merged in hacktober-accepted repo then valid.

Firstly Thank you @vijaygupta18 for accepting my PR. But it seems that there is a lot of spam happening everywhere and many repositories are getting the message "Excluded from project".

Yeah.That's why i am merging only valid prs

I got hactoberfest batch in some repositories but those got removed as excluded project.

Key Pad Problem [Asked in FAANG Interview] I would like to make a folder for mostly frequently asked interview problems. Can I start working on it? I am beginner please let me know if I can contribute.

hello I would love to contribute to this by uploading a web application I have developed. Can I contribute?

hello I would love to contribute to this by uploading a web application I have developed. Can I contribute?

yeah sure.

Key Pad Problem [Asked in FAANG Interview] I would like to make a folder for mostly frequently asked interview problems. Can I start working on it? I am beginner please let me know if I can contribute.

yeah sure...do pr

@vijaygupta18  Kindly give hacktoberfest label to my pr #44

@drishyadamodaran  done!

@vijaygupta18  Kindly give hactoberfest label on my pr #210

@vijaygupta18  Kindly give hacktoberfest label on my pr #167 and #171 please.

@vijaygupta18 Kindly give hactoberfest label on my pr #210

done!

@Saumya2600  done!

my PR is #294 . i am new to hacktoberfest. what to do next?

my PR is #294 . i am new to hacktoberfest. what to do next?

i have merged your pr..wait for your pr to get matured

Kindly give hacktoberfest label to my pr #422

done

Hey, @vijaygupta18 please review pr #493 and Kindly give it hacktoberfest label :)
#hacktoberfest2021

#493


Why do I have to specify the version twice ?

The command to generate the project with the maven archetype is :
$ mvn archetype:generate -DarchetypeVersion=0.1.6 -Daadarchi-version=0.1.6 -DarchetypeGroupId=io.github.Riduidel.aadarchi -DarchetypeArtifactId=archetype

As far as I know, the archetype and other modules (like base) have exactly the same maven lifecycle.
So if you release a new version of aadarchi, the archetype version will be increased at the same time than base module for example.
So why do I have to specify the version of aadarchi dependencies ?
It could be inferred from -DarchetypeVersion=0.1.6 param and if I want to specify another version I could overwrite it with the -Daadarchi-version=0.1.6 param (even if I suppose there might have breaking change between two version).

Fixed in #313 : archetype version is the aadarchi version without any cumbersome parameter passing


Upgrade to ACK runtime v0.13.0.

Description of changes:
Upgrade ACK runtime to v0.13.0 and regenerate controller using code-generator v0.13.0
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

tests should pass after https://github.com/aws-controllers-k8s/code-generator/pull/173 is merged.

/retest

/test apigatewayv2-kind-e2e

/lgtm


Android: playback of audio muted when recording is ending

Version
master
Describe the bug
This applies to Android 10 and is platform-specific. It also applies to any web apps which is not using Web Chat.
In Chrome or Edge on Android 10, speech synthesis could be interrupted by speech recognition. For example, when speech recognition stopped and speech synthesis start immediately, the synthesis could be muted for 1-2 seconds.
Steps to reproduce

Go to https://addpipe.com/simple-recorderjs-demo/
Record an audio clip of 10 seconds
Click play
While playing the audio clip, click record again

Expected behavior
No stuttering when record start or end.
Instead, the playback is temporarily muted for 1-2 seconds when record start. After the mute, both playback and recording appears normally.
Additional context
This could be fixed by simultaneously playback and record at the same time and persisted. The microphone button would be a "soft" click. And the underlying audio subsystem always start playback and record regardless of whether it is currently synthesizing or recognizing.
[Bug]

Needs prioritization

Needs prioritization

Need to re-confirm bug - might be fixed in latest Android.

Need to re-confirm bug - might be fixed in latest Android.

It seems the issue is no longer repro on latest build of Android 10.


Support for GPU acceleration?

Hi, is there support for GPU acceleration in the OpenCV engine?
I tried the OpenCV engine on a GPU instance on AWS EC2, with CUDA drivers installed, but saw an actual drop in performance as compared to the default pil engine when doing simple resizes.

Bump

OoenCV is a computer imaging service that specializes in array math, not necessarily resizing. It also depends on your compiled version of openCV. It can be built with or without many flags including GPU acceleration. I personally upgraded to the openCV 2/3 C++ Api (import cv2 over import cv) and it out performs PIL for me.


Sync Labels

Please sync all labels from
https://github.com/orgs/octoposprime/discussions/14
Names and Colors

You have to sync all names, colors, and descriptions.
https://github.com/orgs/octoposprime/discussions/14#discussioncomment-9026048

All labels are synced.


doc: clarify BOARD_ROOT documentation

clarify BOARD_ROOT must point to dir ABOVE boards

Codecov Report

Merging #14021 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #14021      +/-   ##
==========================================
- Coverage   52.27%   52.24%   -0.04%
==========================================
  Files         307      307
  Lines       45467    45467
  Branches    10525    10525
==========================================
- Hits        23768    23752      -16
- Misses      16901    16913      +12
- Partials     4798     4802       +4

Impacted Files
Coverage Δ

subsys/testsuite/ztest/include/ztest_assert.h
38.88% <0%> (-38.89%)
:arrow_down:

subsys/testsuite/ztest/src/ztest.c
65.28% <0%> (-7.44%)
:arrow_down:

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 039d8ec...56e0c5e. Read the comment docs.

Please squash both commits together. In general, when fixing a pull request commit, instead of creating a new one, amend the previous one.
Also please address the issue reported by gitlint (you need to add a signed off message).

@hallo-alex : Please note that until the issues reported in https://github.com/zephyrproject-rtos/zephyr/pull/14021#issuecomment-469057614 are not fixed, and the commits are not squashed this commit PR cannot be accepted.


Allow adding dynamic jobs at runtime

This should be more of a plugin feature, for example cmd.RegisterJob(plugin, jobInfo)

I can't remember what I meant by "plugin feature" here. Great.

Closed by d3cd28aeaa8531b89f165779041743da2ec91f90 and d2bc0b2aa89edb95eeccabba15fb1e53d6dc3686.


fixes issue #1

:star: PR built and available in a preview environment cruk-jx-org-deleteme01-pr-2 here


Clean up to finish RC1

Description
Various small bug fixes and cleaning of code. Added email link functionality (a RC2 item)
Fixes # (issue)
Finalizes for RC1 sign-off
Type of change
Please delete options that are not relevant.

[x ] Bug fix (non-breaking change which fixes an issue)
[x ] New feature (non-breaking change which adds functionality)

Change Status

[x ] Complete, tested, ready to review and merge

How Has This Been Tested?

[x ] numerous user tests on different devices

Checklist

[x ] My code follows the style guidelines of this project
[x ] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[ x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x ] There are no merge conflicts

I’ve never used UIActivityVuewController and I’m open to all options. If that’s better then I’m all for it.
The reason I didn’t present an alert saying the message was sent was because in the Apple documentation it said the app has no way of knowing if the message was successfully sent. Like if there was no signal and the message couldn’t be sent or something like that. Probably an unlikely scenario though.
Sent with my mobile device.

On Jun 15, 2020, at 12:04 PM, Michael<EMAIL_ADDRESS>wrote:

@mrflowers22 commented on this pull request.
In DJHelper/DJHelper/EventPageViewController.swift:

// In a storyboard-based application, you will often want to do a little preparation before navigation

// Provides feature to send a URL via the Mail app.
// When testing, it only works on a device, not the simulator.
private func setupEmailForLink() {

   guard MFMailComposeViewController.canSendMail(),

I've never heard of this class - it's good to be exposed to new things! Did you look at UIActivityViewController? I think if we are unhappy with the MFMailComposeVC we can use the activityVC as a backup.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Material Checkbox become smaller size - After update to version 7.x

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Material Checkbox should look the same between version 6.x and 7.x
What is the current behavior?
Checkbox on version 7.x is SMALLER than in 6.x
What are the steps to reproduce?

Open this link (Material v6): https://v6.material.angular.io/components/checkbox/overview
and notice the Checkbox
Open this link (Material v7): https://material.angular.io/components/checkbox/overview
and notice the Checkbox
Compare the size (width x height) of the Checkbox

Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular 7.x
TypeScript 3.x
Node 10.x
OS: window 10 64bit
Browser: Chrome

This is intentional in order to bring in line with the latest Material Design spec.

@crisbeto - My application UI brake after this change! What CSS should I apply to display the Checkbox as before? I've try to give the "mat-checkbox-frame" a width and hight of 20px - which make the border of the checkbox bigger, however the label and the V mark appear not centered. Any quick fix to make it appear back as it was?

You can try to revert the changes from https://github.com/angular/material2/pull/12493, however it would be up to you to maintain it since it's not something that we support.

@crisbeto - thank you


Added support for Net6.0.

Fixes https://github.com/chkr1011/MQTTnet/issues/1296 (I hope, I didn't forget anything).

@SeppPenner
See https://github.com/chkr1011/MQTTnet/pull/1135#issuecomment-812466126, since net6.0 already has an proper sdk, you can replace all NET5_0 to NET5_0_OR_GREATER to avoid future changes for new targetFramework.

@SeppPenner See #1135 (comment), since net6.0 already has an proper sdk, you can replace all (excepted the one in TargetFrameworkProvider.cs) NET5_0 to NET5_0_OR_GREATER to avoid future changes for new targetFramework.

I will update this pull request soon with the changes :)

I will update this pull request soon with the changes :)

Is this done?

I will update this pull request soon with the changes :)

Is this done?

Not yet, I was quite busy (End of the year and all customers believe that I'm a kind of wizard or something) :D

No worries. I will close this pull request for now because I will adapt the changes you made in the branch which contains the 4.0.0 version of the library.

No worries. I will close this pull request for now because I will adapt the changes you made in the branch which contains the 4.0.0 version of the library.

Okay :)


Replacement for ISupportSequenceObject.Prefix in agnostic SequenceGenerator

Hi,
I'm currently trying to migrate from the old eXpand SequenceGenerator to the 'new' Reactive.XAF SequenceGenerator module.
I am using the ISupportSequenceObject.Prefix property to generate unique sequences based on a property of the target entities. I wonder how to archive the same with the new SequenceGenerator implementation. There is a new SequenceStorageKeyName attribute, but it's lacking documentation and I don't know if this is what I should use.
Any help is appreciated!

I am not sure either cause I presonally do not use the package and  for creating a sample it requires research time,
I see a related test that might be of help https://github.com/eXpandFramework/Reactive.XAF/blob/16fda8ed431d6a11b5e119f03def866fe2a761d1/src/Tests/SequenceGenerator/SequenceGeneratorTests.cs#L200
maybe @CWolffIF or @isatahiri can help on this one?

Thank you,
I also found this test, but either I don't understand the impl or it is is not sufficient for my requirement:
In the test SequenceStorageKeyName is used to change the 'Name' field of the SequenceStorage object from CustomSquenceNameTestObject to CustomSequenceTypeName at compile time.
I don't know how to create multiple SequenceStorage objects for CustomSquenceNameTestObject based on a property.
Any help of @CWolffIF or @isatahiri is very appreciated

Thank you,
I also found this test, but either I don't understand the impl or it is is not sufficient for my requirement:
In the test SequenceStorageKeyName is used to change the 'Name' field of the SequenceStorage object from CustomSquenceNameTestObject to CustomSequenceTypeName at compile time.
I don't know how to create multiple SequenceStorage objects for CustomSquenceNameTestObject based on a property.
Any help of @CWolffIF or @isatahiri is very appreciated


Fix PAL VS project and add newer files

This should stop VS from asking to upgrade pal.vcxproj the first time you open the solution (or after git clean)

This change is

Should I target master for this change ?

LGTM. Does the Windows build even depend on PAL? Whether to target here or master is a matter of: (1) how annoying is it to go unfixed; (2) how risky is this? I'd say not that risky -- if changing the project file doesn't break the build, there should be no other problems.

It is not included in the build at all. This is only to give visibility in the solution when making changes to function that are reimplemented in pal.
I think I will target 1.6 then and just call it a day


Lags when editing controller files and the open NesjJs File Generator panel

When editing a controller file, when the NesjJs File Generator panel is open with the List of HTTP Methods tab active, editor lags are observed (page scrolling, response to key presses), as well as a high load on RAM. When you close the sidebar or open any other tab other than NesjJs File Generator, the problems disappear.

Apple M1 16Gb.
Visual Studio Code 1.87.2 (Universal)
Electron: 27.3.2
ElectronBuildId: 26836302
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0
ОС: Darwin arm64 23.4.0
NestJS File Generator for VSCode v2.3.0

Oh, cool, you refreshed the list of methods only after saving the file and not every time after changing it. I think the problem has been solved for the most part, there are still small lags in page scrolling when refreshing, but I think this is no longer so critical. Thank you very much for your quick response.


[Feature] Add CONTRIBUTING.md

should I just add the md file will no content

Hey @issac5693, I've assigned #17 to you. Please complete that first. Thanks 👍

But how can I do that
On Thu, 6 Oct, 2022, 3:28 pm Aditya Gupta, @.***> wrote:

Hey @issac5693 https://github.com/issac5693, I've assigned #17
https://github.com/Aditya-Gupta1/job-application-cli/issues/17 to you.
Please complete that first. Thanks 👍
—
Reply to this email directly, view it on GitHub
https://github.com/Aditya-Gupta1/job-application-cli/issues/20#issuecomment-1269724577,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVDHXMTLXLTBUHAFFEN5TD3WB2PFZANCNFSM6AAAAAAQ6KO37U
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey @Aditya-Gupta1 , I can work on this issue!!

Hi! I would love to work on this issue. Can you explain what the file should contain? Thanks :)

Hey @AdarshCoder397 and @manjusha2912 👋
Great to hear you want to contribute to this project!
I'll be working on this issue and the other issues that are assigned to me.
This project is in early phases of development and will be open for a lot of contributions in a few days.
Please keep an eye out 🧐
Thanks.

Completed dude
On Thu, 6 Oct, 2022, 6:16 pm Adarsh kumar, @.***> wrote:

Hey @Aditya-Gupta1 https://github.com/Aditya-Gupta1 , I can work on
this issue!!
—
Reply to this email directly, view it on GitHub
https://github.com/Aditya-Gupta1/job-application-cli/issues/20#issuecomment-1269979716,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVDHXMQ7654TT2OENJKHNSLWB3C4FANCNFSM6AAAAAAQ6KO37U
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add project proposal for SoR'24

Add a project proposal by for the Summer for Reproducibility 2024, a collaboration between Technical University of Munich & University of Chicago

@gallenmu can you add your headshot to your biosketch? The rest looks great.


Handle empty typography.css file

Swagger's typography.css file is an empty file.

Pull Request Test Coverage Report for Build 141

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.5%) to 71.277%

Totals

Change from base Build 132:
0.5%

Covered Lines:
1072

Relevant Lines:
1504

💛  - Coveralls


Can't show *.ipynb files rightly

*.ipynb files can not be shown by nbsphinx.  And show as this:

Actually, if I use sphinx to build it locally, it can show in right format, just like this:

Thank you!

Please click the link if you want to see the html which haven't show *.ipynb files rightly. https://edunlp.readthedocs.io/en/docs_dev/build/blitz/sif/sif.html
Thanks!

I have used the method of #310https://github.com/spatialaudio/nbsphinx/issues/310, but the bug stills existence.

I had the same problem. Seems to have appeared with nbsphinx 0.8.7. I was able to build my docs correctly with sphinx 4.1.2 and nbsphinx 0.8.6.
The main change from 0.8.6 to 0.8.7, concerning this problem, is that the .ipynb extension is now added to sphinx's source suffixes ONLY if it is absent from the base config (the config as it is after executing  conf.py).
And it turns out I had this line in my conf.py:
source_suffix = [".rst", ".ipynb"]

Thus, nbsphinx was never registering notebooks to be parsed by the correct parser. Sphinx was reading them as if they were rst files.  I am not sure why I add that line before, but removing the ipynb part solved my problem.  @BAOOOOOM  maybe you have something similar in your code?

Yes, I have something similar in your code. Now the problem is sovled. Thank you very much.

source_suffix = [".rst", ".ipynb"]

Using a list as source_suffix does not do what you expect, see https://www.sphinx-doc.org/en/master/usage/configuration.html#confval-source_suffix
Since .rst is Sphinx's default and .ipynb is automatically added by nbsphinx, setting source_suffix is not necessary at all.


wip:  Blog post about Kubernetes Response Engine with Google Cloud Run

Signed-off-by: Batuhan Apaydın<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind content

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog

What this PR does / why we need it:
This is another part of the Kubernetes Response Engine series, this time we are using Cloud Run to set up it.
Which issue(s) this PR fixes:

Fixes #444
Special notes for your reviewer:
We have to wait falcosecurity/charts repository to release Google Cloud Run support first.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign mfdii after the PR has been reviewed.
You can assign the PR to them by writing /assign @mfdii in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: developer-guy
To complete the pull request process, please assign mfdii after the PR has been reviewed.
You can assign the PR to them by writing /assign @mfdii in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: developer-guy
To complete the pull request process, please ask for approval from radhikapc after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

/lgtm
/approve

LGTM label has been added.  Git tree hash: a606a80d63b29ae2e9697b4b8e09cbb1445414ba

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: danpopSD, developer-guy
To complete the pull request process, please ask for approval from radhikapc after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: danpopSD, developer-guy
To complete the pull request process, please ask for approval from radhikapc after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

thank you so much for helping me to improve this blog post, I will add you as a co-author to the blog post @radhikapc 🙏

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: danpopSD, developer-guy
To complete the pull request process, please ask for approval from radhikapc after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: danpopSD, developer-guy
To complete the pull request process, please ask for approval from radhikapc after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

Yesterday, Falcosidekick released a new version of its Chart and this release includes Google Cloud Run and Google Cloud Functions features, so I think we are ready to merge both PR's #446 and #453 now if everything is okay.
WDYT @radhikapc @Issif @Dentrax?

I think we can publish both at the same time when we're ready. If both of OK for you, we'll need to update the dates, but pushing a new commit removes the LGTM label.

LGTM label has been added.  Git tree hash: f87fbe8e9fa0d8786c8c8113e1560dc5c87aeec8

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: danpopSD, developer-guy
To complete the pull request process, please ask for approval from radhikapc after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: danpopSD, developer-guy, radhikapc
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [radhikapc]

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


Adding class to editor

This component didn't provide a way to set external class to the editor DOM node.

This can meet your needs？
<quill-editor class="my-editor" ref="myTextEditor"
              v-model="content"
              :config="editorOption"
              @blur="onEditorBlur($event)"
              @focus="onEditorFocus($event)"
              @ready="onEditorReady($event)">
</quill-editor>

Ah, yes, I forgot the truth that vue does it. Sorry for troubling you!


Improve warning about definePageMeta() outside setup

Environment

Operating System: Darwin
Node Version:     v16.13.1
Nuxt Version:     3.0.0-27408580.aece351
Package Manager<EMAIL_ADDRESS>Bundler:          Vite
User Config:      target, head, srcDir, css, plugins, components, buildModules, build
Runtime Modules:  -
Build Modules:    -

Reproduction
<script lang="ts">
definePageMeta({
  layout: 'hero'
})

export default defineComponent({
 // ...
})
</script>

Describe the bug
In a recent update, a new warn has been added when using definePageMeta() outside the setup() method. But it's a bit misleading, as it insinuates that it's only usable inside <script setup>. But it is indeed possible inside
export default defineComponent({
  setup () {
    definePageMeta({
      layout: false
    })
  },
  // ....
})

Additional context
Output of npm run dev
✔ Nitro built in 129 ms
definePageMeta() is a compiler-hint helper that is only usable inside <script setup> of a single file component. Its arguments should be compiled away and passing it at runtime has no effect.
✔ Vite server built in 416ms

Causing lines:
https://github.com/nuxt/framework/blob/68a227af037cc7c91b00787cadbd2653676e00ad/packages/nuxt3/src/pages/runtime/composables.ts#L27-L29

Any news on this issue? I get the same error regardless of where I position definePageMeta
<script lang="ts">
    import { defineComponent } from 'vue'

    definePageMeta({
        layout: 'sidebar-right',
        title: 'home.title',
        sidebar: {
            position: 'right',
            component: 'v-sidebar-static'
        }
    })

    export default defineComponent({
        setup() {
            return {}
        }
    })
</script>

<script lang="ts">
    import { defineComponent } from 'vue'

    export default defineComponent({
        setup() {
            definePageMeta({
                layout: 'sidebar-right',
                title: 'home.title',
                sidebar: {
                    position: 'right',
                    component: 'v-sidebar-static'
                }
            })

            return {}
        }
    })
</script>

same vibes

Nuxt version: 3.8
Node version: v20.5.0
Npm version: 9.8.0
I get the same error when set ssr true in nuxt.config.ts and page in browser is always loading, there is no error in console.
my code:
<script setup> definePageMeta({   middleware: ["category"],   keepalive: true }) </script>
page directory structure:

when access site root directly, there is no block, and I can click inspire/index.vue and any other link, without block
when access inspire/index.vue, it will block always.

@sigezhao If you think there is a bug, you should create a new issue with a reproduction, not comment on a closed issue from two years ago. 🙏

ok


unshare failed: Invalid argument

Microsoft Windows [Version 10.0.17107.1000]

I'm trying to get unshare -U to work but whenever I try to execute that command I get:
tim@localhost:/$ unshare -U
unshare: unshare failed: Invalid argument
tim@localhost:/$

it seems like WSL doesn't have User Namespaces implemented(CONFIG_USER_NS)
unshare.strace

You are correct, we have not yet implemented user namespaces.

Ref CLONE_NEWUSER in #2242, which was closed (entirely reasonably) because Chrome's suid sandbox now works, but still needs --disable-namespace-sandbox.
@tim241 - if you've got another use case ("I'm trying to....") it is worth citing.

@therealkenc well, I replaced the chroot with Void linux(because I nwant to write xbps package templates and the xbps-src script needs xbps-uunshare which is required for the way they build a package, so I /need/ the User Namespaces support. Here's an example of where it crashes:
> ~/void-packages ./xbps-src pkg linux
=> Using `/home/tim/void-packages/hostdir/binpkgs/wine-fixes' as local repository.
[*] Updating `https://repo.voidlinux.eu/current/x86_64-repodata' ...
[*] Updating `https://repo.voidlinux.eu/current/nonfree/x86_64-repodata' ...
[*] Updating `https://repo.voidlinux.eu/current/multilib/x86_64-repodata' ...
[*] Updating `https://repo.voidlinux.eu/current/multilib/nonfree/x86_64-repodata' ...
ERROR unshare (Invalid argument)
> ~/void-packages

@therealkenc, are you still working on it?

Wasn't the person "working on it". I take boypt at face value above, which would imply it was not implemented in WSL1. There is also #3282 which I suspect (but can't prove) is a side effect of chrome no longer supporting SUID sandbox (nor --no-sandbox last time I played with it, which was some time ago).
CLONE_ALLTHETHINGS is fully functional in WSL2.

@therealkenc, strange, I used WSL2. My issue is similar to @tim241's.

Tim241 didn't have a repro so that post was glossed in January; this was before WSL2, and thus not too surprising a fail at the time. But as it happens I'm working on something in a space (almost) related Void Linux at the moment, so you've made me look on the lark. I was able to reproduce Tim's scenario on WSL2. There is an easier repro than the ./xbps-src dance. Inside a Void chroot, just:

With the entirely predictable strace:
unshare(CLONE_NEWUSER)                  = -1 EPERM (Operation not permitted)
write(2, "unshare: ", 9unshare: )                = 9
write(2, "unshare failed", 14unshare failed)          = 14
write(2, ": Operation not permitted\n", 26: Operation not permitted

Which is nutty (at least in my somewhat sleep deprived state) because outside the chroot (back in plain 'ole Ubuntu):

Which I already knew worked because I had done it before. At a loss right this moment to guess the variable. On first glance it is hard to finger-point WSL, because user namespaces is enabled in the kernel and, well, your know, works. But it is also near-impossible to finger-point Void userspace, because I can't think of anything that Void userspace could do to make the unshare(2) fail. It isn't performing the chroot proper (that makes no sense).
Reason will probably be obvious once it comes to me; if someone else doesn't beat me to it. Appreciate the submission.

@therealkenc
I tested your repro on a native ubuntu 19.10 install (with a amd64 debian chroot)

it seems like intended behavior :)
some more research led to this:
https://unix.stackexchange.com/questions/442996/what-rule-prevents-entering-a-user-namespace-from-inside-a-chroot

There is an article of void-packages repo, which suggests to modify chroot's permissions.

it seems like intended behavior :)

I was afraid of that. Thanks.

This would be useful to automate https://github.com/microsoft/WSL/issues/5090#issuecomment-613945065 as per https://stackoverflow.com/questions/20553039/forcing-usermod-with-running-program/56338192#comment109450754_56338192


tmoreno.open-log-viewer version 1.5.1

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

close and reopen again since the bot had problems yesterday

Oh, this is not a installer, its just a portable app


Support for Django 1.10

Closes issue #119

Thanks! 🎉

Thanks for the merge, @dlo !

No problem! I'll push up to PyPi in an hour or two.


href.ly + anonymz + updated tests

Created href.ly and anonymz url expanders, remove adfoc.us regex, created tests and updated urls for existing tests

I rolled these changes into my branch.
I did entirely remove the INVALID_URL_ERROR_CODE thing, because it just made the API more confusing. Requiring the programmer to be aware of HTTP status codes is a reasonable requirement, I think, and the API needs to be able to disambiguate between things like a 404 failure because the referred-to URL has been removed, and a issue with the unshortening library.

Looks like #7 has these changes. Closing this PR


Support wasm api

Ⅰ. Describe what this PR did
Implement the higress wasmplugin API, doc: https://higress.yuque.com/qnr8w5/wlbt7k/rheasbbq2p1fe9gg#tQHr6
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #129 (3384de5) into main (b410fc9) will decrease coverage by 0.32%.
The diff coverage is 5.43%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #129      +/-   ##
==========================================
- Coverage   25.88%   25.57%   -0.32%
==========================================
  Files          26       26
  Lines        4524     4693     +169
==========================================
+ Hits         1171     1200      +29
- Misses       3277     3417     +140
  Partials       76       76

Impacted Files
Coverage Δ

pkg/ingress/kube/ingress/controller.go
3.66% <0.00%> (ø)

pkg/ingress/kube/ingressv1/controller.go
3.65% <0.00%> (ø)

pkg/ingress/config/ingress_config.go
18.01% <3.57%> (-3.47%)
:arrow_down:

pkg/ingress/kube/common/tool.go
48.30% <66.66%> (+0.66%)
:arrow_up:

pkg/bootstrap/server.go
62.97% <0.00%> (+7.63%)
:arrow_up:


Data preprocessing by Xiangnan

Staff and Course Enrollment Data (https://www.cde.ca.gov/ds/ad/ssctop.asp)


data.eventType() always returns "put"

Describe the bug
No matter what action is taken on the realtime database when printing the StreamData dataType() method it will always return "put"
To Reproduce
Steps to reproduce the behavior:

run stream_callback example
delete entry from your realtime database.
observe data.eventType() print
any action returns "put"

Expected behavior
It's very important to me to see a clear distinction between "put" and delete" events.
Screenshots
If applicable, add screenshots to help explain your problem.
IDE and its version:

Arduino
Version [1.8.13]

Please ask if you have the question or problem instead of set this issue to bug.
The event type is issued by Google not the library.
There are only 5 events that available from RTDB stream i.e. put, patch, keep-alive, cancel and auth_revoke.
Please read this doc for reference


(WIP) feat: claude integration

Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Type of change
Please delete options that are not relevant.

[ ] 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)
[ ] This change requires a documentation update
[ ] Documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration:

Test Configuration:

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

When will Claude's work be merged into the main branch?(✧∀✧)

When will Claude's work be merged into the main branch?(✧∀✧)

it's nearing completion now


Issue with k8s.io/docs/setup/production-environment/container-runtimes/

https://github.com/kubernetes/website/issues/22591
/language ko
/assign

According to #22859, I am going to close this issue. Thank you.


i18n(fr): Updating project-structure translation

What kind of changes does this PR include?

[x] New or updated content
[x] Translated content

Description

What does this PR change? Updates content from src/pages/fr/core-concepts/project-structure.md to reflect changes from withastro#761

It's time to merge some translations 🥳, thanks @Chrissdroid!!


Allow for an episode post to be staged ahead of release.

I want to be able to have the markdown for a given post added to the repository before it needs to go live.
In the "metadata header" section of the post, allow for an optional property that allows the post to be ignored or skipped by the templates.

Done as of #5


Update DFD to match current development status

Update DFD to match current structure of Git Scanner

Completed on WIKi


rake teaspoon unable to load teaspoon_env.rb

Following the instructions here: https://github.com/modeset/teaspoon/wiki/Testing-Engine-Assets
If I start the rails server in the dummy app, the web interface runs the tests that are outside the dummy directory propertly. If I run 'rake teaspoon' in the dummy directory everything is also good.
But if I run 'rake teaspoon' at the root of my engine it says
Teaspoon::EnvironmentNotFound: Unable to load Teaspoon environment in {spec/teaspoon_env.rb, test/teaspoon_env.rb, teaspoon_env.rb}
That file is in dummy/spec. How do I get it to be seen by my engine's rake task?

I had a similar issue where when my teaspoon_env.rb was in the dummy app the rake task would fail and when it was in the engine root the in browser specs would fail. I ended up setting up dotenv in my dummy app and set TEASPOON_ENV=/absolute/path/to/teaspoon_env.rb. That solved the issue for me.

Today I ran into this issue and played with a bunch of different things until I got this to work.
From the root of an engine you should be able to run the app:teaspoon rake task, which will fail because it can't find the spec/teaspoon_env.rb or config/environment.rb. Since we are able to hook into those couple of ENV vars though, we can get in front of the app:teaspoon rake task being called with something like this in <engine root>/Rakefile
desc 'Run the javascript specs'
task :teaspoon do
  ENV['TEASPOON_ENV'] = File.expand_path('../spec/dummy/spec/teaspoon_env.rb', __FILE__)
  ENV['TEASPOON_RAILS_ENV'] = File.expand_path('../spec/dummy/config/environment.rb', __FILE__)

  Rake::Task["app:teaspoon"].invoke
end

All looks good :) :+1:

Thanks for the heads up! :+1: Just as an aside, teaspoon is tested with teaspoon, and the teaspoon specs aren't in the dummy app -- they're in the engine top level specs. That's the official recommendation, but settings the envs works as well. I guess the reasoning behind putting them at the engine spec level, is because the dummy app is just the stub used to test your engine, and I don't put things like the rspec specs or capybara features in the dummy app either. That same logic seemed to apply to the javascript ones as well.

@jejacks0n yeah I would have to agree with that. My first attempt had it situated as such but I couldn't figure out what was going on so I went with the instructions in the wiki until I got it to work. Now that I have a better working knowledge of its internals I'm going to do just that - move the js specs from within the dummy app to the engine specs dir.

Glad it worked. Feel free to check out the dummy app from teaspoon, it's lightweight, and I think there's a few specific changes that are needed in general?

Jeremy Jackson

On Aug 26, 2015, at 8:16 AM, Joel Oliveira<EMAIL_ADDRESS>wrote:
@jejacks0n yeah I would have to agree with that. My first attempt had it situated as such but I couldn't figure out what was going on so I went with the instructions in the wiki until I got it to work. Now that I have a better working knowledge of its internals I'm going to do just that - move the js specs from within the dummy app to the engine specs dir.
—
Reply to this email directly or view it on GitHub.


Dynamic PSO states refactor

Fixes #2905
PR checklist:

[x] make succeeds (on *nix)
[x] make reftests succeeds
[ ] tested examples with the following backends:
[ ] rustfmt run on changed code

Side note on enum -> Option conversions. At some point in the past, I suggested to use enums for cleaner semantics at the type level. Now it's more clear that type level clarity is less important here as just being straight and simple. Option gives you that, without sacrificing the type safety.

Ok, I don't think there is a lot of controversy about the PR, other than the "Sided" name :)
Bors r=grovesNL

On Aug 8, 2019, at 08:13, Josh Groves<EMAIL_ADDRESS>wrote:
@grovesNL approved this pull request.
In src/hal/src/pso/output_merger.rs:

     }

 }

}
+/// A generic struct holding the properties of two sides of a polygon.
+#[derive(Clone, Copy, Debug, Default, Eq, Hash, PartialEq)]
+#[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
+pub struct Sided {
Maybe just PolygonFaces or similar?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Add init subcommand for yurtctl.

yurtctl add init subcommand.
What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind bug
/kind documentation
/kind enhancement
/kind good-first-issue
/kind feature
/kind question
/kind design
/sig ai
/sig iot
/sig network
/sig storage
/sig storage
/kind feature

What this PR does / why we need it:
Add init subcommand for yurtctl to create an OpenYurt cluster. Proposal:
https://github.com/openyurtio/openyurt/blob/master/docs/proposals/20210607-adding-subcommands-server-join-reset-for-yurtctl.md
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Does this PR introduce a user-facing change?

other Note

/lgtm

/approve


fix(snack-bar): announcing same message twice to screen readers

Currently we have role="alert" on the snack bar which will cause screen readers to announce the message automatically. On top of it, we also use the LiveAnnouncer to announce the same message, if the consumer hasn't set one. These changes clear the announcementMessage if it's the same as the main message.

cc @josephperrott
Shouldn't we always use the LiveAnnouncer and remove role="alert"? My understanding was that the announcementMessage replaced the actual content of the message for announcement purposes

@jelbourn if we were to remove the role="alert", content inside snackbars that are opened from a component or a TemplateRef won't be read out automatically. Given that it's the only way of using markup inside the snack bar, it may end up breaking some apps.


Create User Docs - Explanations

Following https://documentation.divio.com/explanation/, create concrete explanations for Hypper topics, such as:

[ ] the value propositions
[ ] how Hypper is different from Helm

isnt this covered on https://github.com/rancher-sandbox/hypper/blob/main/docs/design.md ??

I think you are right, those points are covered there! Ticking them off on the issue.
How about adding an explanation on the Hypper chart schema? From documentation.divio.com/explanation, I'm not thinking of a technical reference, but short explanations on why we:

Use the chart annotations: all Hypper charts must be valid Helm charts for backwards compatibility.
Embed a full yaml document in hypper.cattle.io/shared-dependencies: because of Kubernetes limitations. Configmaps only accept key-value pairs (not other yaml objects), therefore, we must store the yaml document in the value.

I would also like an explanation (or to expand) on why we are not pursuing Application CRDs and a more holistic approach to installing cluster services and apps.
E.g: cluster-wide CRD installations are fragile, as they are a global resource. Installing them system-wide means that all admins/users of the cluster can depend on them, and removal and update of CRDs is both opaque to all admins/users and breaks compatibility for them (see https://github.com/helm/community/blob/f9e06c16d89ccea1bea77c01a6a96ae3b309f823/architecture/crds.md).

chart schema sounds good as it seems like a basic piece of info that should be available on a mvp.
There rest seems like more advanced topics, not sure if they would fall into the mvp, but they sound good to me. New card to track those?

Sounds fine here. From https://documentation.divio.com/explanation, the chart schema would be part of  https://github.com/rancher-sandbox/hypper/issues/83 (as a "what"), and the explanation on "why" would be here, maybe in docs/design.md
Looking at the hypper lint implementation, I was expecting to use a schema to validate hypper charts, but doesn't seem to be the case. So maybe one doesn't need a full schema for hypper charts for now? I still think it's useful to have a sole document that you can point people to, listing exactly which key-values are supported in hypper charts.
About the Application CRD approach, maybe it's a task for Matt (if we should document it, in any case).


Compose file not parsable

Illegal offset with: https://github.com/siwapp/siwapp-ror

Thanks! This solved my ghost resources issue.


Migrate references to moby

resulting from docker project reorganization, we need to update references to
https://github.com/moby/moby

actually, they've not updated references in code yet, only the location of the code so we can't update references yet.


add logging to chase AttributeError: 'str' object has no attribute 'get'

I'm continuing chasing down errors in logs. I tried a few things but couldn't reproduce the error below. Therefore adding more logging to understand what's going on and what kind of data comes here.
[2022-04-13 23:55:13,287: ERROR/MainProcess] Task handler raised error: AttributeError("'str' object has no attribute 'get'")
Traceback (most recent call last):
  File "/opt/app-root/lib64/python3.8/site-packages/billiard/pool.py", line 362, in workloop
    result = (True, prepare_result(fun(*args, **kwargs)))
  File "/opt/app-root/lib64/python3.8/site-packages/celery/app/trace.py", line 555, in _fast_trace_task
    R, I, T, Rstr = tasks[task].__trace__(
  File "/opt/app-root/lib64/python3.8/site-packages/celery/app/trace.py", line 499, in trace_task
    I, _, _, _ = on_error(task_request, exc, uuid)
  File "/opt/app-root/lib64/python3.8/site-packages/celery/app/trace.py", line 338, in on_error
    R = I.handle_error_state(
  File "/opt/app-root/lib64/python3.8/site-packages/celery/app/trace.py", line 167, in handle_error_state
    return {
  File "/opt/app-root/lib64/python3.8/site-packages/celery/app/trace.py", line 220, in handle_failure
    task.on_failure(exc, req.id, req.args, req.kwargs, einfo)
  File "/app/ibutsu_server/tasks/__init__.py", line 53, in on_failure
    report_id = args[0].get("id")  # get the report ID from the task
AttributeError: 'str' object has no attribute 'get'

I think I know what is causing this. Let me see if I can track it down.

So what's happening here is that this method was created when the only thing that Celery was used for was the reports. We were experiencing task failures, and were trying to mark the report as error'd so that the user has a nicer experience. To do this, we tried to catch the failure and then get the report id out of the arguments.
Now we have other tasks, the first argument is no longer a dictionary, but could be a str or int, or something else.
So the solution is probably to just check if the first argument is a dictionary.

Thanks @rsnyman for the explanation! I will modify this PR


Loading "private" packages

Hello,
The documentation, which is generally excellent, makes it seem like what I'm hoping to do should be possible, but I'm not able to connect the dots.
I'm happy to get most of my packages from github, from melpa, etc. but I have a couple that aren't public. (I suppose, in practice, I could just install them by hand, but I'm trying to simplify my init by doing things in a more regular, less ad hoc fashion).
Can I change the protocol for a single recipe?
What I want to say is "you can get this package by checking out this git repo on that host over ssh".
If that's not possible or is impractical, can I say "you can get this package out of that directory over there?"

Thanks for your question!
Currently, you can't change the protocol for a single recipe (see https://github.com/raxod502/straight.el/issues/92). However, you have two options:

Just enable SSH globally by setting straight-vc-git-default-protocol. All GitHub repositories are accessible via SSH as well as via HTTPS.
Simply specify the entire URL, i.e. (:repo "git@github.com:yourname/private.git") instead of (:host github :repo "yourname/private").

I believe one of these options should solve your problem. However I will also note that you can indeed run a package from a local directory simply using :local-repo "full-or-relative-path". Note that you should put the repository inside ~/.emacs.d/straight/repos for the automatic modification detection to work the best.

Perfect. Thank you. I changed straight-vc-git-default-protocol to ssh and then this did the trick:
(straight-use-package
  '(my-package :repo "ndw@private.example.com:/home/ndw/git/my-package.git"))


Refactor ADTask and Related Components

Description
This PR includes several key refactoring changes:

Extracts common code from ADTask into TimeSeriesTask, creating ForecastTask for forecasting-specific logic.
Consolidates common code from ADTaskType into TaskType and introduces ForecastTaskType for forecasting-related purposes.
Renames ADTaskState to TaskState for consistent code reuse.
Renames the method getId in ADTask to getConfigId to differentiate it from other IDs like task id.

Note: The code refactored in this commit is subject to change when working on historical forecasting in the future.
Testing done:

Added unit tests for the new code to ensure functionality.
Executed a successful Gradle build.

Issues Resolved
https://github.com/opensearch-project/anomaly-detection/issues/866
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@sudiptoguha added comments and addressed compiler errors due to https://github.com/opensearch-project/OpenSearch/pull/9006
org.opensearch.common.component.LifecycleListener => org.opensearch.common.lifecycle.LifecycleListener
org.opensearch.common.unit.ByteSizeValue => org.opensearch.core.common.unit.ByteSizeValue
org.opensearch.indices.breaker.NoneCircuitBreakerService => org.opensearch.core.indices.breaker.NoneCircuitBreakerService
XContentFactory.contentBuilder => MediaTypeRegistry.contentBuilder

Have you done any testing with frontend for this branch?

I haven't. The frontend main branch CI will test backend main branch. If there is anything broken, I'll fix it. Also, since we won't release this branch until forecasting frontend is finished, I think we can leave the testing later.

Have you done any testing with frontend for this branch?

I haven't. The frontend main branch CI will test backend main branch. If there is anything broken, I'll fix it. Also, since we won't release this branch until forecasting frontend is finished, I think we can leave the testing later.

got it, was mostly referring to the comment regarding this file in the original issue: https://github.com/opensearch-project/anomaly-detection-dashboards-plugin/blob/710e3c4ddadf3689b7f014b5df88f2a82b8db999/public/pages/DetectorDetail/utils/constants.ts

Have you done any testing with frontend for this branch?

I haven't. The frontend main branch CI will test backend main branch. If there is anything broken, I'll fix it. Also, since we won't release this branch until forecasting frontend is finished, I think we can leave the testing later.

got it, was mostly referring to the comment regarding this file in the original issue: https://github.com/opensearch-project/anomaly-detection-dashboards-plugin/blob/710e3c4ddadf3689b7f014b5df88f2a82b8db999/public/pages/DetectorDetail/utils/constants.ts

created an issue https://github.com/opensearch-project/anomaly-detection-dashboards-plugin/issues/581 and will test later.

@sudiptoguha @amitgalitz new commit made the following replacement to address compiler errors due to https://github.com/opensearch-project/OpenSearch/pull/9073
org.opensearch.common.transport.TransportAddress => org.opensearch.core.common.transport.TransportAddress
org.opensearch.transport.TransportResponse => org.opensearch.core.transport.TransportResponse
org.opensearch.common.transport.BoundTransportAddress => org.opensearch.core.common.transport.BoundTransportAddress


Cow: Watch totalx.in

Cow requests the watch of the watch_keyword totalx\.in. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

totalx\.in has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Is https://appr.tc down?

Browsers and versions affected
Chrome V66
Description
Getting the following error;
WebSocket open error: WebSocket error.
WebSocket register error: WebSocket error.
Steps to reproduce
access https://appr.tc
Expected results
No errors
Actual results
Web socket failures
Messages:
WebSocket open error: WebSocket error.
WebSocket register error: WebSocket error.
Version:
gitHash:    20cdd7652d58c9cf47ef92ba0190a5505760dc05
branch:     master
time:       Fri Mar 9 17:06:42 2018 +0100

the certificate expired. someone is on it.

Same issue...

Thank you. Is it also the cause for iOS app failures when connecting to
https://appr.tc?
Thanks,
--Suman
On Thu, Jun 28, 2018 at 10:50 PM Philipp Hancke<EMAIL_ADDRESS>wrote:

the certificate expired. someone is on it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/webrtc/apprtc/issues/565#issuecomment-401254585, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AActvSLt21gd_k9gSNcpZhLJPVBvaXZAks5uBcAmgaJpZM4U8TH7
.

Most likely. See https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/discuss-webrtc/31d0BovQssE/iMaLsRMuBAAJ

seems to be resolved now. send #hugops to @MirkoBonadei

Resolving.

Yes, I am able to verify on AppRTCMobile sample and my own app.
Thanks a lot!
--Suman
On Fri, Jun 29, 2018 at 12:34 PM Philipp Hancke<EMAIL_ADDRESS>wrote:

seems to be resolved now. send #hugops to @MirkoBonadei
https://github.com/MirkoBonadei
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/webrtc/apprtc/issues/565#issuecomment-401452975, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AActveUHBVZq80mg_NRYoIxXU5tvO2x4ks5uBoE-gaJpZM4U8TH7
.

note that you should run your own server

The issue has been closed, but it's still not working for me using appRTCMobile and firefox.
to run my own server, where is  the certificate that need to be changed?

Yes, I am in the process of setting it up. It is a bit complicated than I
thought (for me) and taking more time than I expected.
Thanks,
--Suman
On Fri, Jun 29, 2018 at 1:06 PM Philipp Hancke<EMAIL_ADDRESS>wrote:

note that you should run your own server
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/webrtc/apprtc/issues/565#issuecomment-401460413, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AActvfN8-YTpsxAH9vf4uQ2zAhIjKjAQks5uBojPgaJpZM4U8TH7
.

Please, this issue is now appear, someone fixes that?


TranslateStaticLoader prefix

I'm submitting a ...  (check one with "x")
[ ] bug report => check the FAQ and search github for a similar issue or PR before submitting
[ ] support request => check the FAQ and search github for a similar issue before submitting
[x] feature request

Current behavior
Cant add  file following pattern 'prefix/customName-suffix;
Only possible prefix/suffix-cuctomName;
Expected/desired behavior
Should add possibility to add files that have name a file current following pattern 'prefix/customName-suffix;
Reproduction of the problem
If the current behavior is a bug or you can illustrate your feature request better with an example, please provide the steps to reproduce and if possible a minimal demo of the problem via https://plnkr.co or similar (you can use this template as a starting point: http://plnkr.co/edit/tpl:btpW3l0jr5beJVjohy1Q).
What is the expected behavior?
What is the motivation / use case for changing the behavior?
Let user decide how to name files
Please tell us about your environment:

ng2-translate version: 2.4.2
Angular version: 2.0.0-rc.X
Browser: [all ]
Language: [all]

Let me clarify that a bit. Right now, you can specify both a prefix and a suffix, which will result in the following URI:
`${this.prefix}/${lang}${this.suffix}`

Are you requesting something like this
`${this.prefix}${lang}${this.suffix}`

to be possible? Notice, that there's no / after prefix.
I think we can even make it a non-breaking change if we check for trailing slash in the given prefix.

For those of you following this in an email, I removed the "non-breaking" piece from my previous comment

The problem was that you cant specify name like i18n/locale-en.json.
You can only specify i18n/en-locale.json
It's not a problem for me just want to know why restrict choosing name?

That's true, but you can write your own loader based on the current loader, it's hard to satisfy everyone :)
I'm closing this for now, if you really need this in the core, then submit a PR with the change, thanks !


SCG Redis ratelimiter solution has unavailable issues when redis has problems

When I enabled the SCG Redis ratelimiter feature and encountered issues with my Redis cluster, I noticed that all requests passing through the gateway were pending for several seconds, similar to what was reported in PR-1634([2]).
To investigate, I tested removing this Redis ratelimiter configuration to see if the blocking issue persisted. Indeed, after removal, the problem disappeared.

The root cause lies here:
org.springframework.cloud.gateway.filter.factory.RequestRateLimiterGatewayFilterFactory#apply
It's important to carefully consider the implementation of the Redis-based ratelimiter feature:

Custom Redis keys are generated based on the request's IP address or user ID for rate limiting. With a high volume of requests from diverse IP sources or user IDs, there are performance, load, and capacity risks, especially with millions of users/ips.
It's advisable to have a dedicated and separate Redis cluster for the SCG rate limiter. Otherwise, it could impose significant load on Redis, leading to higher-than-expected tail latency.
If your scenario is not overly complex, consider implementing a local rate limiter solution instead of relying on a global rate limiter like this one.

References:
[1] https://spring.io/blog/2021/04/05/api-rate-limiting-with-spring-cloud-gateway
[2] https://github.com/spring-cloud/spring-cloud-gateway/issues/1634
[3] https://stripe.com/blog/rate-limiters
[4] https://gist.github.com/ptarjan/e38f45f2dfe601419ca3af937fff574d
[5] https://docs.spring.io/spring-cloud-gateway/reference/spring-cloud-gateway/gatewayfilter-factories/requestratelimiter-factory.html#redis-ratelimiter

What are you thinking we might do to the gateway?

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


[2.5] Fixed security analytics tests

Description
Fixed security analytics tests.
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Command used:
yarn cypress run --spec "cypress/integration/plugins/security-analytics-dashboards-plugin/*"  --config "baseUrl=https:///_dashboards" --env "openSearchUrl="
Passing tests:


chore(kit): InputSlider use new Slider inside

PR Checklist
Please check if your PR fulfills the following requirements:

[X] The commit message follows Conventional Commits
[ ] Tests for the changes have been added (for bug fixes / features)
[X] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[X] Refactoring
[ ] Code style update
[ ] Build or CI related changes
[ ] Documentation content changes

Does this PR introduce a breaking change?

[ ] Yes
[X] No

Aw, you have some problems, please rebase directly / or squash commits

also, you can change commit name to feat(kit): InputSlider use new Slider inside


update release modal styles

Description
Remove box-shadow and add hover states for modal elements
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

manual

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Now would be a good time to add tests for this feature

Now would be a good time to add tests for this feature


Enabling envoy.restart_features.upstream_http_filters_with_tcp_proxy breaks //test/extensions/upstreams/tcp/generic:config_test

Mock expectations are not correct.
Introduced by #32991
It looks like a just a config test problem. Does not affect any integration tests.
@vikaschoudhary16

@yanavlasov ooc why is this a problem?  AFIK there's no expectation for default-false flags to pass all tests when true.

How can you tell if a enabling a flag to true does not break Envoy in some horrible way if not all tests pass?
I think there should be an expectation that enabling flag to true does not break any tests. Otherwise it is not possible to reason whether this breakage is something that is just test related or something that truly breaks Envoy. If some tests depend on the 'false' value to work they should set it explicitly.

This particular test flag is for a WIP feature.  I think any false flag is high risk and it's up to the person deploying it to do risk assessment including deciding if the tests need to pass.


[AutoPR datacatalog/resource-manager] typo: Microsoft.DataCatalog

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4674

This PR has been merged into https://github.com/Azure/azure-sdk-for-js/pull/692


Update GetMergeRequestCommits URL to match the gitlab API spec

I noticed the URL was wrong for this method according to the spec:
Repo: https://github.com/xanzy/go-gitlab/blob/master/merge_requests.go#L176
Spec: https://docs.gitlab.com/ce/api/merge_requests.html#get-single-mr-commits

The result of this was this error:
2017/02/15 11:27:54 Unable to list commits on pull request: GET https://..../merge_request/195/commits: 404 {message: 404 Not found}
exit status 1

Whoops! Thanks @smeriwether!


Use path and path-io

That would make codebase a bit more robust.

There aren't many file paths in the hpack codebase - basically, just the directory to run in, package.yaml, and <name>.cabal. I don't know if it's worth bringing in the dependency.

Yes, same thought here. Low priority for me.
Sent from mobile

On 15 Aug 2016, at 12:39 AM, Mitchell Rosen<EMAIL_ADDRESS>wrote:
There aren't many file paths in the hpack codebase - basically, just the directory to run in, package.yaml, and .cabal. I don't know if it's worth bringing in the dependency.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I could do this myself, IMHO it's worth the dependency. As it's good creating a newtype for the filepaths, and DIY is more trouble than done and tested packages out there.

@phadej would you still be interested to open a pull request for this? I would like to have it for #119.

I could take a look next week


Create GPU framework for unit tests

As the GPU engine is more and more critical for applications using OCIO, it seems important to have unit tests validating the shader programs generated by the library. To have an easy way to write GPU unit tests, the framework is built on top of the CPU engine. The unit test defines the OCIO::Transform to be validated, and the framework automatically ensures consistency between CPU and GPU engine. So, the CPU implementation becomes the reference which means it should be fully validated using CPU unit tests. Using that approach GPU unit test are easy to write.
To demonstrate the framework and provide some examples, several unit tests were added: 1) some around MatrixOp and 2) some around Lut3DOp. It covers the two extreme cases.
The framework works well on Win7 & VS15, RH 6.8 & gcc 4.4.7. In addition, the branch was compiled on ubuntu 16.4 & gcc 5.4.0 (using docker).

Unfortunately, Travis & appVeyor could not execute GPU unit tests by default because they exclusively use headless machines. I tried to use Xvfb (Linux utility to create a virtual framebuffer) but a freeglut bug is crashing the app.

On a quick test build, I'm seeing a couple warnings and errors on the GPU Unit Test build possibly related to the OpenGL version used, I haven't looked into it too much yet.
warning: 'glutCreateWindow' is deprecated: first deprecated in macOS 10.9 [-Wdeprecated-declarations]
and
error: use of undeclared identifier GL_CLAMP_VERTEX_COLOR'
I'm also seeing that the GPU Unit Tests aren't built on TravisCI because GLUT isn't installed, we should probably modify the configuration to at least build the tests, even if they can't run.