Broadcast incorrect target shape. Expecting either 1 or 126. Got 0

When I followed the tutorial at this notebook, and after executing model.compile(), I encountered an error while trying to run the model with the sample provided in the tutorial:
My Code
sample = """from typing import List

def has_close_elements(numbers: List[float], threshold: float) -> bool:
    \"\"\" Check if in given list of numbers, are any two numbers closer to each other than
    given threshold.
    >>> has_close_elements([1.0, 2.0, 3.0], 0.5)
    False
    >>> has_close_elements([1.0, 2.8, 3.0, 4.0, 5.0, 2.0], 0.3)
    True
    \"\"\""""

from transformers import TextStreamer

# Tokenize the sample
inputs = tokenizer([sample], return_tensors='pt')

# Call generate on the inputs
out = model.generate(
    **inputs,
    max_new_tokens=128,
    streamer=TextStreamer(tokenizer=tokenizer, skip_special_tokens=True),
    pad_token_id=tokenizer.eos_token_id,
)

Run ERROR
75 warnings generated.
75 warnings generated.
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
Cell In[39], line 7
      4 inputs = tokenizer([sample], return_tensors='pt')
      6 # Call generate on the inputs
----> 7 out = model.generate(
      8     **inputs,
      9     max_new_tokens=128,
     10     streamer=TextStreamer(tokenizer=tokenizer, skip_special_tokens=True),
     11     pad_token_id=tokenizer.eos_token_id,
     12 )

File ~/.conda/envs/ai_code/lib/python3.10/site-packages/torch/utils/_contextlib.py:115, in context_decorator.<locals>.decorate_context(*args, **kwargs)
    112 @functools.wraps(func)
    113 def decorate_context(*args, **kwargs):
    114     with ctx_factory():
--> 115         return func(*args, **kwargs)

[...additional stack trace...]

RuntimeError: Exception from src/inference/src/cpp/infer_request.cpp:245:
Exception from src/bindings/python/src/pyopenvino/core/infer_request.hpp:54:
Caught exception: Check 'DimType::broadcast_merge(result_shape[i], arg_shape_vec[i], target_input_shape[i])' failed at src/core/shape_inference/include/broadcast_shape_inference.hpp:149:
While validating node 'opset3::Broadcast Broadcast_1320379 () -> ()' with friendly_name 'Broadcast_1320379':
Broadcast incorrect target shape. Expecting either 1 or 126. Got 0

Please tell me what to do to fix it. Thanks!

can't reproduce in my setting. the code ran as expected after compilation:
Compiling the model to CPU ...
from typing import List

def has_close_elements(numbers: List[float], threshold: float) -> bool:
    """ Check if in given list of numbers, are any two numbers closer to each other than
    given threshold.
    >>> has_close_elements([1.0, 2.0, 3.0], 0.5)
    False
    >>> has_close_elements([1.0, 2.8, 3.0, 4.0, 5.0, 2.0], 0.3)
    True
    """
    for i in range(len(numbers)):
        for j in range(i + 1, len(numbers)):
            if abs(numbers[i] - numbers[j]) < threshold:
                return True
    return False

if __name__ == "__main__":
    import doctest

    doctest.testmod()

@SunriseB can you give more details about your setting and the versions you're using ?

Thx for reply.
test (1).ipynb.zip
You can download the ipynb file to see the detail and the entire process where the error occurred.

I see you're trying to run the model using device="gpu" on nvidia gpu ?

yes. so it can only accelerate the inference on cpu?

Fine. Thanks for your advice!!!


What would be the effort needed for having type completion in html files.

I understand that this question is related to specific frameworks used in the editor in conjunction with typescript (e.g. angular2, aurelia, ...)
So, let's focus on a case where a viewModel (a .ts) goes together with a view (a .html).
the viewModel is a typescript class and contains a definition for 'Customer' having firstName and lastName props.
When creating the view, I would like to have type completion on the specific bind statement (so, in the example here on the string inside value.bind.
///////here some magic to 'bind' the view to the viewModel with respect to type completion
<div class="col-sm-10">
<input type="text" placeholder="First Name"  class="form-control"
value.bind="customer.firstName" />
</div>

The above example is aurelia, but similar stuff can happen in other frameworks.
I think I saw similar things in visual studio in asp.net MVC.
How realistically would it be to make this happen also for sublimeText ?
Thanks.
paul.

I do not see any technical or architectural blockers. there are a lot of missing pieces that will need to be able to do that. first you will need the plugin to register for html, then we will need at least an html parser, then a way to connect an html file to a ts/js file based on the framework conventions and semantics, then all needed is to query the properties of an exported entity from the ts/js file with the right name and render the results. again there are no blockers just work to be done :D

Thanks for the prompt reply.
So good to hear that from technical perspective there are not real constraints.
May I ask an additional question regarding the architecture of sublime text plugins. (I'm not hindered by too much knowledge about subl. plugins)
Would there be a feasible way to implement this functionality for multiple SPA frameworks in such a way that no parallel versions of the plugin are necessary? So, the idea of 'sub-plugins", one for ng, aurelia, react, ...

That would be a question for @billti or @zhengbli.

@paulvanbladel I'm not sure I understand correctly what you refer to as "sub-plugins". Generally, the sublime plugin works mainly through implementing commands and subscribe to the editor's events. You can also add setting items to let the user choose which framework to use.

@zhengbli Great, these settings items could cope with the different frameworks.
I'll study a bit more the details of a sublime plugin.
Do you think there is somwhere a sample which could be used as a base for the functionality I have in mind (so the code completion in a view file and seeded from a viewmodel) ?
Do you guys believe that this feature would have "market-potential"?

This feature would definitely have its audience. As @mhegazy said, the major work might be in parsing the html correctly and communicating with the typescript compiler. Sublime only serves as the front end where content is retrieved.

Hi, can the lang service work in html file now?
I followed the official guide Angular Language Service to have code completion and type check. But I cannot have them in loose HTML files referenced in component ts.
I just searched in issues and stackoverflow, only find this one even seems a bit out of date. But can anyone confirm whether sublime plugin support this function?


Implement Components with new HTML parser

This change ports the existing Blazor component compiler features to
work on top of the Razor compiler. These new features are on once Razor
Language version is set to V3 or higher.
The tests and code for the old Blazor compiler were added to this repo
in previous changes. This change adds the necessary enabling features,
and gets the tests working.
Code generation for tests tests is identical with a few exceptions.
Design time directives now include line mappings (as they should). The
infrastructure seems to prefer windows style line-endings whereas the
Blazor code did the opposite. I plan to leave this alone unless it's a
problem for testing on *nix.
IR Generation will be cosmetically different due to different node names
and formatting. Structure should generally be the same.

In order to implement this support, I had to add the following core
features.
Configuring passes/engine-features based on the langauge version. This
required revisiting a lot of old tests and forcing them to use a
version appropriate Razor engine. A lot of our tests for legacy things
were using new versions of the language :(.
Configuring passes/engine-features to switch on/off depending on the
document kind.
Configuring directives depending on the file-kind. Directives are a
parser feature, so this decision needs to be made before the IR phase.
Directives that are specific to a file kind will override directives
that are not - this allows components to customize templates for
instance.
Adding functions/inherits/implements directive as mainline features that
are always on in V3. I'm not quite motivated to do this yet for
typeparam, but we might end up there. Remember that directives are
reserved words in Razor.

:shipit:


Create Rust version of the "Loggable Data Types" doc section

What
Create Rust version of the Loggable Data Type doc section
Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[ ] I've included a screenshot or gif (if applicable)

PR Build Summary: https://build.rerun.io/pr/2501

Docs preview: https://rerun.io/preview/b3832c6/docs
Examples preview: https://rerun.io/preview/b3832c6/examples

These examples were exceedingly interesting to write, and are IMHO very useful to identify the shortcomings of the current Rust API. In particular, I ran into the following gotcha, many of which required @teh-cmc's help to sort out:

RawMesh3D.vertex_colors want packed RGBA u32, but, porting from Python, I first provided a flattened array of u8 RGB components (felt natural since the other fields also take flattened arrays of coordinates/normals/etc.). This would happily compile, but nothing was displayed and no error was presented.
When implementing the step timeline of the scalar example, I first searched for set_time_sequence (used in Python API) in the Rust docs, which I found in RecordingStream. Turns out it's not in 0.7 and bugged on main (or rather ignored by MsgSender). Again, it compiled and displayed no error, but no timeline was created.
There is a redundancy between the ClassID key in AnnotationContext.class_map and the ClassDescription.info.id field. It's unclear if they much match, or what's the meaning of each of them. I always matched them and didn't run into any issue, but it's still confusing.
Likewise for ClassDescription.keypoint_map: the hashmap's KeypointID is redundant with AnnotationInfo.id field and generates confusion.
Btw, AnnotationInfo.id is u16 but documented as "either ClassID or KeypointID". How un-rusty! :)
Still on the topic of annotation context, specifically for the "connection" example. To work, the Rust version requires a ClassId(0) splat to be logged with the points, presumably to match the AnnotationContext.class_map hashmap key. Inferring that was possible thanks to classid being listed as secondary component of Point2D/3D, but it was confusing that there appears to be no such equivalent (or hidden default) in the Python API.


validationQueryTimeout in OracleValidConnectionChecker

issues: The oracle driver will call cancel when the query timeout, but if the timeout is caused by executing in a closed connection(closed by database), the cancel command would not work and the checker throws runtime exception until the socket read timeout(config by oracle.jdbc.ReadTimeout, default 600000 milliseconds).
So the time used to check the connection may be longer than the validationQueryTimeout configuration and will cause upper invoker's reading time out.
fix: Execute the connection check by submitting a asynchronous task to ExecutorService and retrieve the result from the Future instance.

I meet this problem too, but why this commit didn't be merged?

I meet this problem too, but why this commit didn't be merged?

Try other ojdbc driver.

oracle version 11.2  which ojdbc version should be choose？ can I use ojdbc6  <IP_ADDRESS>.0 solve this problem？


Kage update support for shift operators

What issue is this addressing?
Updates #2755
What type of issue is this addressing?
feature
What this PR does | solves

Add type validation for left and right shifts.
Add support for the shift assignments operation.
According to the docs, it should work everywhere:
HLSL
GLSL
MSL

Any updates? I'll take over this if you are not available. Thanks!

I am sorry, it took more time than I expected. I will upload updates today.

I'll review this later, but I feel like we don't have to introduce complex things such as delayedTypeValidator. We should be able to do with much smaller modifications.

I think I found an incosistent behavior in the Go compiler (https://go.dev/play/p/pCHHBoblbAY)
package main

import "fmt"

func main() {
	s := 1
	a := 2.0 << s
	b := 2.0<<s + s<<3.0
	fmt.Println(a, b)
}

I'll report this. TBH I don't know how to treat this.

I reported the above issue https://github.com/golang/go/issues/66071. Let's wait for the Go team's answer. @aoyako, your test catched this corner case. Great job!

I think I found an incosistent behavior in the Go compiler (https://go.dev/play/p/pCHHBoblbAY)
package main

import "fmt"

func main() {
	s := 1
	a := 2.0 << s
	b := 2.0<<s + s<<3.0
	fmt.Println(a, b)
}

I'll report this. TBH I don't know how to treat this.

I think it is an expected behaviour.

In the first case:
2.0 would be a float because it would be so in a := 2.0. Therefore, the left operand for shift is float for a := 2.0 << s and thus it will fail.
In the second case:
The result of s << 3.0 is int, so when guessing a type for 2.0 compiler should try b := 2.0 + int implying 2.0 to be converted to an integer type. Therefore, a constant 2.0 is treated as an integer.
However, I don't know if s << 3.0 should give a panic for go vet.

The result of s << 3.0 is int, so when guessing a type for 2.0 compiler should try b := 2.0 + int implying 2.0 to be converted to an integer type.

You meant 2.0<<s + int?

However, I don't know if s << 3.0 should give a panic for go vet.

This was another bug https://github.com/golang/go/issues/65939

I tested a + operator with an integer, and apparently you are right... 🤔
package main

import "fmt"

func main() {
	a := 1
	b := 1.0<<a + 0
	c := 1.0<<a + int(0)
	fmt.Println(b, c)
}

The result of s << 3.0 is int, so when guessing a type for 2.0 compiler should try b := 2.0 + int implying 2.0 to be converted to an integer type.

You meant 2.0<<s + int?

Yes, it is a final expression.
According to the docs:

If the left operand of a non-constant shift expression is an untyped constant, it is first implicitly converted to the type it would assume if the shift expression were replaced by its left operand alone.

The compiler will try to assume a type in the expression b := 2.0 + int first.

I tested a + operator with an integer, and apparently you are right... 🤔
https://go.dev/play/p/OHnUA3weL8w
package main

import "fmt"

func main() {
	a := 1
	b := 1.0<<a + 0
	c := 1.0<<a + int(0)
	fmt.Println(b, c)
}

It is funny you can't easily define a float variable equal to a power of 2 using shifts: https://go.dev/play/p/qvk1sHyIOhK
package main
import "fmt"

func main() {
	s := 1
	floatPowerOfTwo1 := float32(1<<s + int(0))
	floatPowerOfTwo2 := float32(1 << s) // Fail
	fmt.Println(floatPowerOfTwo1, floatPowerOfTwo2)
}

floatPowerOfTwo2 := float32(1 << s) // Fail

This blowed my mind! 🤯
Hmm, ok. Even if this is an expected behavior, I now doubt it'd be worth emulating the authentic Go behavior faithfully: I feel like it is OK to do a simpler type resolusion. Let me think more. I'd like to wait for the Go team answer anyway.

Sorry for keeping you waiting. Now I don't think we have to follow the Go's current specification in a very strict way. So, we can change the Kage behavior like
{stmt: "var a = 1; b := a << 2.0; _ = b", err: false},
{stmt: "var a = 1; b := 2.0 << a; _ = b", err: false},

so that we no longer need the delayed type determination. We can treat shift operators as the other binary operators especiallyl like a mod operator which takes only integers. Thoughts?

https://github.com/golang/go/issues/66071#issuecomment-1974744117
One possible reason why the shift operators behave like this is a natural type deduction for constant values, but as Kage doesn't have int32 and int64, we don't need this.
var n = 32
var y = int64(1 << n) // y should be int64 whichever the archtecture is 32bit or 64bit

Sorry for keeping you waiting. Now I don't think we have to follow the Go's current specification in a very strict way. So, we can change the Kage behavior like
{stmt: "var a = 1; b := a << 2.0; _ = b", err: false},
{stmt: "var a = 1; b := 2.0 << a; _ = b", err: false},

so that we no longer need the delayed type determination. We can treat shift operators as the other binary operators especially like a mod operator which takes only integers. Thoughts?

Thanks for the confirmation. It makes sense to use simpler rules since Kage uses the same integer type.
I also wondered whether allowing untyped float constants in shift operators is a good idea in the first place. However, keeping rules the same for all operators will be much simpler.
I will update the PR with these new rules then. It might also be worth adding a compile error when shifting by a constant causes overflow.

I also wondered whether allowing untyped float constants in shift operators is a good idea in the first place. However, keeping rules the same for all operators will be much simpler.

I confirmed that % works as following:
	println(1.0 % a) // works
	println(a % 1.0) // works
	println(1.0 % 1.0) // doesn't work

https://go.dev/play/p/R2MFM9f0Cxe
I think doing the same thing with % would make sense.

It might also be worth adding a compile error when shifting by a constant causes overflow.

As the integer size is not determined, can we detect overflow?

As the integer size is not determined, can we detect overflow?

In HLSL and MSL the max size for ints is 64 bits. 32 bits are not guaranteed in GLSL for non-high-precision numbers, and it won't apply here.
Also, it might be unnecessary since the API might change and cause outdated restrictions on 64 bits for integers.

Also, it might be unnecessary since the API might change and cause outdated restrictions on 64 bits for integers.

IMO the Kage compiler doesn't have to detect the overflow and we can let the drivers how to treat them.

I realized 1.0 << 1.0 worked (https://go.dev/play/p/wTSIXouoc-9), so this is different from the mod operation. I think all these cases should work
	a := 1
	println(1.0 << a)
	println(a << 1.0)
	println(1.0 << 1.0)

Sorry for going back and forth

These cases are included:
{stmt: "a := 1.0 << 2.0; _ = a", err: false},
{stmt: "var a = 1; b := a << 2.0; _ = b", err: false},
{stmt: "var a = 1; b := 2.0 << a; _ = b", err: false}, // PR: #2916

Thanks, I'll take a look later.


feat: add a way to use zoxide as an input field

I'd appreciate it if we discussed this, before you make any changes.
Missing:

[ ] New default keybind for jump fzf
[ ] Fix the hang on line 55
[ ] Replace Err(anyhow!("")) with the correct way to create an empty error

Breaking changes:

jump zoxide now uses an input field, and the old functionality is moved to jump zoxide_interactive
Default jump fzf keybind is changed

Thanks for your PR!
I plan to reimplement zoxide as a plugin to provide more flexibility, which will eventually deprecate the jump command, conflicting a bit with the content of this PR. I just haven't had the time to do it yet, but it should be soon ;)

Hi, I've now reimplemented zoxide as a plugin in https://github.com/sxyazi/yazi/issues/865, which will allow users to copy it as a template and easily modify the commands inside.
That is, you can copy https://github.com/sxyazi/yazi/blob/main/yazi-plugin/preset/plugins/zoxide.lua as your ~/.config/yazi/plugins/zoxide.yazi/init.lua, and then call ya.input() to receive the user's input, and passes it to the zoxide command - Yazi will load your own zoxide.yazi plugin automatically


swift 2.3, swift 3

Hello!
Do you have any plans on migrating library to newer versions of swift?

Hello!
For sure, please check out #76. Until Swift 3 is officially out, I can't make the main release use Swift 3. However, I can upgrade the library for Swift 2.3 if it's throwing warnings and errors?

Haven't tried to use it from project with 2.3 or 3
Maybe before official release you can create some swift3 branch and post updates there?

Makes sense, let's see where the PR contributor is on that front, and we can hopefully arrange something.

Swift 3 changes are in the swift-3 branch for now, and will hopefully be merged into master as soon as Xcode 8 comes out.

Blocked by a Cocoapods bug now .. once I can overcome that bug, will release a new version.


"Expected fetch contoller" invariant failure

Reproduction
https://stackblitz.com/~/github.com/gforro/remix-fetchers
Use cases which are probably bugs:
A: Fetcher in parent route gets to a for ever "loading" state

Open the application and wait for fetcher in the header to finish data loading
Submit the "Fetcher form" with 100ms response time
Wait for the loading state of the submission (1 second for example) and click on "Go to Page 2" => fetcher in header is cancelled and a new load is executed
Click again the "Go to Page 2" link => "Expected fetch controller" invariant failure is logged to the console
User is navigated to Page 2 and the fetcher in the Header stays in "loading" state for ever

B: Fetcher in parent route gets to a for ever "loading" state

Open the application and wait for fetcher in the header to finish data loading
Submit the "Fetcher form" with the default 2000ms response time
Click on the "Go to Page 2" immediately (in the submitting phase) => fetcher gets to "loading" state after the click, the application waits for the submission to be finished and navigates the user to Page 2
Click the "Go to Page 1" immediately (while the Header fetcher is in loading state) => "Expected fetch controller" invariant failure is logged to the console
User is navigated back to Page 1 and the fetcher in the Header stays in "loading" state for ever

Another use cases, which seems to be strange:
C: Fetcher in parent route blocks navigation after Form submission

Open the application and wait for fetcher in the header to finish data loading
Submit the "Form with navigation" with the default 2000ms response time
Click on the "Go to Page 2" when the form state changes to "loading..." => the fetcher in the Header is cancelled and revalidated
Click on the "Go to Page 2" again while the navigation state is "loading..." => the fetcher in the Header is cancelled and revalidated
Click on the "Go to Page 2" again and again and again.... => the fetcher is the Header is always cancelled and revalidated

D: Fetcher in parent route blocks navigation after fetcher Form submission

Open the application
Click on "Mount" in the fetcher fieldset => fetcher is mounted to Page 1
Wait for the fetchers to get idle
Submit the "Fetcher form" with 100ms response time
Wait for the loading state of the submission (1 second for example) and click on "Go to Page 2" => fetcher in header is cancelled and a new load is executed
Click again the "Go to Page 2" link => fetcher in the header is cancelled and a new load is executed
Keep clicking the "Go to Page 2" link => fetcher in the header is cancelled and a new load is executed and after a while (10-20 clicks) the "Expected fetch controller" invariant failure is logged to the console
User is navigated to Page 2 and the fetcher in the Header stays in "loading" state for ever

Interesting that the application in case D behaves differently like in case A - they are similar and difference is the present of a fetcher in the child route. If there is a fetcher in the child route where the Fetcher form submission happened, then the navigation to Page 2 is blocked for a longer time (there is more successful revalidations of the Header fetcher but at use case ends with the very same invariant failure - just later.
System Info
See Stackblitcz. The problem also present in my Mac:

System:
    OS: macOS 14.7
    CPU: (10) arm64 Apple M1 Max
    Memory: 54.81 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.14.0 - ~/.nvm/versions/node/v20.14.0/bin/node
    Yarn: 1.22.22 - ~/.nvm/versions/node/v20.14.0/bin/yarn
    npm: 10.7.0 - ~/.nvm/versions/node/v20.14.0/bin/npm
    pnpm: 8.15.9 - ~/.nvm/versions/node/v20.14.0/bin/pnpm
  Browsers:
    Chrome: 129.0.6668.90
    Safari: 18.0
  npmPackages:
    @remix-run/dev: * => 2.12.1
    @remix-run/node: * => 2.12.1
    @remix-run/react: * => 2.12.1
    @remix-run/serve: * => 2.12.1
    vite: ^5.1.0 => 5.4.7

Used Package Manager
npm
Expected Behavior
I have expected to not block navigation when a fetcher in the parent route is revalidated because of a form submission in a child route.
I have also not expected to get invariant failures when playing with fetchers and to not get a "for ever" loading state for a fetcher.
Actual Behavior
Navigation is blocked if there is a fetcher in the parent route which is revalidated because of a form submission in the child route. This problem is noticeable when there are longer response times (I know that response time should be around 200ms but it i not always the case unfortunately)

I believe these are duplicates of this issue in RR which has been fixed and will be available in an upcoming release: https://github.com/remix-run/react-router/issues/12049

Thanks @brophdawg11 for the reaction. It might be the same (I was not able to find that issue event I have tried to find a similar issue).
The issue mentioned no runtime error, but there is one - the sticked "loading" state of fetchers after the failed invariant. Also fetchers stops to be revalidated after that error.

@brophdawg11 any alternative fixes, until upcoming release ?

@gforro Remix 2.13.0-pre.0 is out - would you like to test your scenarios against that and let us know if that helps?

I have expected to not block navigation when a fetcher in the parent route is revalidated because of a form submission in a child route.

I think this expectation is incorrect - when a submission navigation occurs, it waits for all active loaders to be revalidated before proceeding - this includes all route loaders and all fetcher.load calls.
With that in mind, would you be able to create individual minimal repros of any remaining issues on 2.13.0-pre.0?  I'd be happy to take a look but the conglomerate app showing off 4 different scenarios is a little confusing when trying to hone in on a singular issue.


Chars missing from a document

Multiple chars are missing from the document below. Nothing is reported in the console.
URL: pos36.pdf

I can reproduce this with the latest master branch with Firefox on Arch Linux. Moreover, random "(" and ¨%" characters appear in the text, for example in the last line of page 1.

Text layer is of .js is ok.

Page 1
diff 🔬

.js 2.8.335
ium 4505

I've got a patch for this, so hopefully Calixte won't mind me stealing the issue :-)


Does it support spinner picker mode?

It does not support a spinner mode.
I don't intend on adding this myself.
You can take a look at this library instead: https://github.com/drawers/SpinnerDatePicker


Geometry positioning ignored, screen recording starts at top left no matter what

As the title says, doing something like wl-screenrec -g "$(slurp)" -f "$HOME/Videos/screenrec_$(date +'%s').mp4" --low-power=off and having $(slurp) be a region like 1528,156 1110x712 gets the size right but completely ignores the positioning, leaving it to record that size at the top left.

Actually, even after reverting it, the behavior still happens, so I'm not sure what's causing it...

That commit doesn't ignore x and y, it just removes the cast as it's not needed anymore

Ah. Still not sure why it's being ignored in practice though.

Yeah, it's super strange. My guess is an issue with your vaapi driver of sorts, but not entirely sure

Probably a dup of #47, what does messing with exact give?

I can try the suggested fix in a couple hours. Also on Hyprland fwiw

I'll move the discussion over there since it's definitely a duplicate


Delete local certificate files after dehydrated fails

For each issue that doesn't succeed we're still creating three files (cert-.csr, cert-.pem, privkey-*.pem) in dehydrated's cert directory that don't get deleted. While they are not big in size, a seizable number of (sub-) domains can this silently exhaust the number of available inodes.
Without this, or a similar patch to the same end, a single certificate that fails to renew (e.g. because the domain has been moved elsewhere) would add 90 files (three per day), which aren't ever deleted. This is compounded by having each subdomain its own certificate.
This is a follow-up to 49256f85c2753d27732a1cadaf7e94cf3d5c5cd8

@gohai: Many thanks, as always (and as always sorry for the delays too)! I've tweaked some of this in https://github.com/auto-ssl/lua-resty-auto-ssl/commit/aa018afaaf7118929742aaa175e5eb25961a6e52 before merging. The commit message attempts to explain the changes I made, but let me know if you have any feedback regarding the tweaks I made:

There was similar logic to cleanup dehydrated files both in the hook file (if the hook succeeded) and in the SSL provider (in case the dehydrated call failed). Since this means the files are deleted in either case (success or failure), try to simplify this by shifting the logic to always run in the SSL provider regardless of success status.
For the new delete logic introduced in the SSL provider, switch it to use non-blocking shell execution to prevent the rm from potentially blocking (even though this should be quick).
Since dehydrated's files are always being removed now, remove the logic we had in place to handle an edge-case of storage failing, but files remaining in dehydrated's cache, since this scenario is now moot.
Update tests to account for new delete logic.

Thank you @GUI for merging and improving this patch! Your changes look much cleaner to me, we'll pick them up next time we'll rebase our tree.


After disabling the plugin, it continues to run in the background.

Uninstalling it doesn't help either. Please advise how to solve this problem.

restart Obsidian

Yes, it worked, thank you so much))))))


feat: rename assign unassign #419

Resolves #419

Test
https://github.com/starlitnightsky/Real-Estate/issues/11#issue-1764931704
Test for skipping start
https://github.com/starlitnightsky/Real-Estate/issues/11#issuecomment-1604093932
Test for help menu
https://github.com/starlitnightsky/Real-Estate/issues/11#issuecomment-1604094225

You need to update the help menu and other related functions. @Draeieg you might be able to QA this if it works already.

I just updated and pushed.

The grammar doesn't make sense. You need to spend some time rewriting the messages instead of just doing a find-and-replace.

I assume this isn't ready for QA yet

Closed for inactivity.


Cannot find the port when running in docker

Hi there,
This works fine locally. However, when I try to move it into a docker container, the server cannot find the correct port. My docker file looks like this:
FROM node:18.12.1

ENV NODE_ENV=production
WORKDIR /fronted
RUN npm install -g local-web-server
COPY . .
CMD [ "ws", "--port 30" ]

Then, if I run docker run -dp 30:30 frontend-server, and then docker logs -f <container-id> it shows  Listening on http://8ab29995624f:null, http://<IP_ADDRESS>:null, http://<IP_ADDRESS>:null. I expect null to be 30.

Try changing this line
CMD [ "ws", "--port 30" ]

to this
CMD [ "ws", "--port", "30" ]

Try changing this line
CMD [ "ws", "--port 30" ]

to this
CMD [ "ws", "--port", "30" ]

Thank you so much for your quick reply, that works!
Now, in the docker it shows Listening on http://5f80a3af8967:30, http://<IP_ADDRESS>:30, http://<IP_ADDRESS>:30. But I can only open the page with http://<IP_ADDRESS>:30 but http://5f80a3af8967:30 and http://<IP_ADDRESS>:30 are not loading up, is it normal? If I run without docker, all three sites work perfectly.

http://5f80a3af8967:30 and http://<IP_ADDRESS>:30 are not loading up, is it normal? If I run without docker, all three sites work perfectly.

it could be normal - i don't know how you have networking/routing/DNS/WIN etc set up on your machine or container.. Closing as beyond the scope of ws..


[Feature Request] Intelligently run code based on file name

My ideal workflow for this plugin would allow me to mid development, select a function say, and use a hotkey to run that function (and surrounding code) based on the file name. So *.py would run python selected text. *.js would run node selected text etc.

I would be open to looking at a pull request if you want to take a shot at it. This would need to be a separate command from the existing ExecuteBuffer and ExecuteSelction as we would need to differentiate between using the first line of text to determine how to run the file and using some kind of a default configured by the user based on file type.


retryDelay can be bypassed with back button

The screen which is shown during the retryDelay can easily be bypassed by simply pressing the back button on an android device.
This could be fixed by wrapping the delay route in a WillPop that denies the pop if the screenLock cannot be cancelled.
But it might be nicer to disable all keypad buttons and
show an appropriate message on screen, with a countdown.

I think something like
Widget Function(BuildContext context, Duration delay) delayBuilder
= const HeadingTitle(text: 'Input is locked for ${delay.seconds} seconds');

as a parameter of ScreenLock, instead of the current new screen with delayChild, would be interesting.
It would then also have to disable the input buttons during the delay.


Feature Request: Unmatching shouldn't unmatch when eruptions close together

In current scheme, the unmatch step takes only the eruption closest to the SEP event as the likely cause of the event. Any older eruptions are unmatched. It will also unmatch CME parameters that are refit with a slightly changed start time. This is the case for sepster_20210528_2312_0824_Parker_Spiral_iss_20210529_1343.json. The CME was refit and had a start time one SOHO frame later. The later CME time was taken to indicate the correct forecast.
For March 7, 2012, to eruptions followed each other closely by less than 2 hours. It was likely the first eruption that caused the SEP event. In the current unmatching logic, if forecasts from both CMEs are there, only a forecast generated from the second eruption will be associated with the SEP event. May be better to associate all CMEs or flares within close succession with the SEP event. So within a window of the closest eruption - 2 hours, for example.

The unmatched status is recorded in SPHINX_dataframe output, correct?  It should be somewhat standard procedure to manually verify unmatching.
For Scoreboard validation, it would be nice if SPHINX could generate a Scoreboard URL to take the validation evaluator directly to the event period to inspect what happens.  This may imply a feature request to the Scorebaord to accept parameters with HTTP GET.

"it would be nice if SPHINX could generate a Scoreboard URL" - this is a fantastic idea and I think we will need to ask for the Scoreboard to accept parameters via URL.

SPHINX has been updated not to unmatch if an eruption is within 3 hours of the "best" eruption (selected because it is closest in time). Going to close this issue and make the Scoreboard URL features into its own issue.


User Story 60501: 101-aks-cluster

Part of POC to test generating sample code and articles using OpenAI.

@grayzu @lonegunmanb
In a more recent PR, I put the SSH-creation code in a separate file (ssh.tf). I like doing that as it's isolates the SSH creation and AzAPI usage. Should we use that pattern as the standard? If so, I'll make the change to this PR.

@grayzu @lonegunmanb
In a more recent PR, I put the SSH-creation code in a separate file (ssh.tf). I like doing that as it's isolates the SSH creation & AzAPI usage and reduces code in main.tf. Should we use that pattern as the standard? If so, I'll make the change to this PR.

As long as we've agreed on a working pattern I think it's ok to apply the pattern on this repo.

@stemaMSFT @lonegunmanb This PR is ready to review.


Limit the maximum granularity of the time interval

Discussed once in the daily meeting. We agreed to have minimum 1 ns per
pixel to avoid visual issues but after seeing that we could have some
function calls of 50ns duration and some discussions, I decided to limit
100ns to the whole visible capture. If we try to zoom to a smaller
interval, we will center that interval in the minimum time range.
This PR is moving that new limit to the SetMinMax function to be
consistent. Before we had:

1 ns limit for zooming with mouse wheel
no limit with ctrl + mouse right click

We have some issues when zooming-in a lot (Timers: http://b/168283058,
ThreadStates: http://b/200905941, and I even have a crash with ctrl + mouse
right click). They won't appear anymore after this PR.
PS: As a limitation, the overlay who is generated when holding mouse right
click is still rounded to an integer nanoseconds and this rounded is still
visible with 100ns total time range, as we can see in the screenshot:
http://screen/3x9qPTZWJsjnEQo. Anyway, it's not too drastic with 100ns and
I will consider that expected behaviour.
Test: several interactions in a loaded capture.

LGTM, seems reasonable!
Nit: The PR description is duplicated for some reason. You should fix this in the commit message when merging.

Thanks Ronald! I was confused by why the commit message was so big but I didn't realize that was because duplicated message. Thanks for pointing it out! It's solved.


Some uncommon resolutions (like 544x408) errors and causes a re-compilation

When using the SDXL pipeline with oneflow_compile (on onediff/oneflow Community Edition 0.12.0), for certain sizes like (544x408, 648x920, 984x680) the inference process prints an error (although it still continues, albeit much slower, like ~50 seconds per error).
This doesn't happen for more common resolutions like 1024x1024, 1024x576, 576x1024, 512x512, etc.
2024-01-24 01:41:49.820 [stdout   ] Warming up with 544x408 (t2i)
2024-01-24 01:41:49.878 [stderr   ]
2024-01-24 01:41:49.878 [stderr   ]   0%|          | 0/25 [00:00<?, ?it/s]
2024-01-24 01:41:49.954 [stdout   ] [ERROR](GRAPH:OneflowGraph_0:OneflowGraph) run got error: <class 'oneflow._oneflow_internal.exception.Exception'> Check failed: (51 == 52)
2024-01-24 01:41:49.954 [stdout   ]   File "oneflow/core/job/job_interpreter.cpp", line 307, in InterpretJob
2024-01-24 01:41:49.954 [stdout   ]     RunNormalOp(launch_context, launch_op, inputs)
2024-01-24 01:41:49.954 [stdout   ]   File "oneflow/core/job/job_interpreter.cpp", line 219, in RunNormalOp
2024-01-24 01:41:49.954 [stdout   ]     it.Apply(*op, inputs, &outputs, OpExprInterpContext(empty_attr_map, JUST(launch_op.device)))
2024-01-24 01:41:49.954 [stdout   ]   File "oneflow/core/framework/op_interpreter/eager_local_op_interpreter.cpp", line 84, in NaiveInterpret
2024-01-24 01:41:49.954 [stdout   ]     [&]() -> Maybe<const LocalTensorInferResult> { LocalTensorMetaInferArgs ... mut_local_tensor_infer_cache()->GetOrInfer(infer_args)); }()
2024-01-24 01:41:49.954 [stdout   ]   File "oneflow/core/framework/op_interpreter/eager_local_op_interpreter.cpp", line 87, in operator()
2024-01-24 01:41:49.954 [stdout   ]     user_op_expr.mut_local_tensor_infer_cache()->GetOrInfer(infer_args)
2024-01-24 01:41:49.955 [stdout   ]   File "oneflow/core/framework/local_tensor_infer_cache.cpp", line 210, in GetOrInfer
2024-01-24 01:41:49.955 [stdout   ]     Infer(*user_op_expr, infer_args)
2024-01-24 01:41:49.955 [stdout   ]   File "oneflow/core/framework/local_tensor_infer_cache.cpp", line 178, in Infer
2024-01-24 01:41:49.955 [stdout   ]     user_op_expr.InferPhysicalTensorDesc( infer_args.attrs ... ) -> TensorMeta* { return &output_mut_metas.at(i); })
2024-01-24 01:41:49.955 [stdout   ]   File "oneflow/core/framework/op_expr.cpp", line 602, in InferPhysicalTensorDesc
2024-01-24 01:41:49.955 [stdout   ]     physical_tensor_desc_infer_fn_(&infer_ctx)
2024-01-24 01:41:49.955 [stdout   ]   File "oneflow/user/ops/concat_op.cpp", line 55, in InferLogicalTensorDesc
2024-01-24 01:41:49.955 [stdout   ]     CHECK_EQ_OR_RETURN(in_desc.shape().At(i), out_dim_vec.at(i))
2024-01-24 01:41:49.955 [stdout   ] Error Type: oneflow.ErrorProto.check_failed_error
2024-01-24 01:41:49.955 [stderr   ] ERROR [2024-01-24 01:41:49] - Exception in __call__: e=Exception('Check failed: (51 == 52) \n  File "oneflow/core/job/job_interpreter.cpp", line 307, in InterpretJob\n    RunNormalOp(launch_context, launch_op, inputs)\n  File "oneflow/core/job/job_interpreter.cpp", line 219, in RunNormalOp\n    it.Apply(*op, inputs, &outputs, OpExprInterpContext(empty_attr_map, JUST(launch_op.device)))\n  File "oneflow/core/framework/op_interpreter/eager_local_op_interpreter.cpp", line 84, in NaiveInterpret\n    [&]() -> Maybe<const LocalTensorInferResult> { LocalTensorMetaInferArgs ... mut_local_tensor_infer_cache()->GetOrInfer(infer_args)); }()\n  File "oneflow/core/framework/op_interpreter/eager_local_op_interpreter.cpp", line 87, in operator()\n    user_op_expr.mut_local_tensor_infer_cache()->GetOrInfer(infer_args)\n  File "oneflow/core/framework/local_tensor_infer_cache.cpp", line 210, in GetOrInfer\n    Infer(*user_op_expr, infer_args)\n  File "oneflow/core/framework/local_tensor_infer_cache.cpp", line 178, in Infer\n    user_op_expr.InferPhysicalTensorDesc( infer_args.attrs ... ) -> TensorMeta* { return &output_mut_metas.at(i); })\n  File "oneflow/core/framework/op_expr.cpp", line 602, in InferPhysicalTensorDesc\n    physical_tensor_desc_infer_fn_(&infer_ctx)\n  File "oneflow/user/ops/concat_op.cpp", line 55, in InferLogicalTensorDesc\n    CHECK_EQ_OR_RETURN(in_desc.shape().At(i), out_dim_vec.at(i))\nError Type: oneflow.ErrorProto.check_failed_error')

This happens right after compiling the other resolutions, for which the logs are like the ones here: https://gist.github.com/isidentical/9b34b3a42f51a7b7919c2db39f1479b5

Looks like a duplicate of #527, if so, we can close this.

Thanks for your feedback.

Looks like a duplicate of #527, if so, we can close this.

It should be the same problem.


BAU Further DWP KBV debug logging

Logging staging (only) token and complete requests so can be reproduced locally

Sonarcloud is complaining because the new log method is not covered - but this is just temporary code to be removed soon


Provide a Pebble.js API for the WakeUp API

This new Pebble API is very interesting for pebble.js. We should have an API for it.

There is now documentation for the implementation!
Here is also a very dry example using all of the available functions:
https://gist.github.com/Meiguro/0deada2525b16ec91e15
There's also Clock.weekday for setting recurring alarms. Feel free to reopen or start a new issue if there are any problems.


Language error in spite being English UK or US.

Error: SCAN_LOG_CHECK_LOGIN_ERROR_LANGUAGE

I encountered it too. I wish I can help but I know nothing about coding, could I also be enlightened as to why this happens?
I tried changing English UK > US> UK > US but still to no avail.
Error message:
Script was able to scan all the months, but December.
Old Facebook api is used to scan December, please change Facebook language to English (UK), English (US), Русский, Українська or עברית, and scan again.

Hmm, I just checked again and Facebook itself is missing December...

Hello!,
I have the same problem. But in my execution of the APP in December 2021, the data of the birthdays of November 2022 disappears..

Please note that the error message: SCAN_LOG_CHECK_LOGIN_ERROR_LANGUAGE is alerting but the actual December data is extracted correctly.
I did change languages and the final output contains December.

Scanning issue was fixed in https://github.com/zagushka/birthday-calendar-extractor/pull/94


setting options in schema doc clarification

In the section of the README where you use a helper to "calculate" the options in the schema, and then use map, I found it more helpful to use [collection_name].find().fetch() instead of just [collection_name].find(). This is because fetch() returns an array of Objects, which then can easily be mapped to an object with a label and name. Here is an example of me doing so in my own code:
  workshops: {
    type: [String],
    label: 'workshops',
    autoform: {
      options: function() {
        return _.map(Workshops.find().fetch(), function(c) {
          return {label: c.name, value: c.name};
        });
      }
    }
  }

Consider slightly modifying README to include this example?

I usually use cursor.map, like here. I assume it's faster than converting to an array and using _.map, though I have no numbers to back that up.

Ahhh I see, got it, thanks. My bad, didn't realize that cursors have a built-in map method.


implemented oaep encryption

Coverage decreased (-0.03%) to 53.156% when pulling 728b5f56003a31deedc1b92e9c1d53b137255ee2 on oaep into a3f29cd180c555f5474ab835bfae70693e5ed2b1 on develop.


[Graph] Add support for any-depth nesting in graph UX

Motivation: Why do you think this is important?
We want to add support for any-depth nesting on the Graph UX.  This is applicable for branching and sub-workflow/dynamic nodes.
Comps:

Goal: What should the final outcome look like, ideally?
Provide users a means to dive deep into nested graphs.

Resolved, starting from flyteconsole v.0.44.0


.cropper-container inline styles width: 200px and height: 100px

Why is it that the .cropper-container is always set to 200x100 px? I cannot style it so it scales properly. I can see the styles are set inline to the element itself (meaning it is set in JS?).
This is how small it looks when simply initialising the cropper without any settings. I have not added any css to interfere with anything.

MORE INFO:
To be clear, I have also tried minCanvasWidth and minCanvasHeight, which partly solves the problem. I now have a bigger image, but the cropper still can't size bigger than 200x100 px.. Meaning this is not the solution (this is the max side of the cropped area, I cant drag it any bigger)

Please read the docs about the minContainerWidth and minContainerHeight options:
$('img').cropper({
  minContainerWidth: 800,
  minContainerHeight: 600
})

In my situation the solution was to apply cropper on image load:
$('img').load(function() {
  $('img').cropper();
});


Handle Chum repo adding/removing from GUI

As developers may want to swap between chum:testing and chum as well as to simplify use, let's add Chum repo management to GUI. This seems to be possible via SSU DBus API. Both, testing and regular repo should be supported, probably one in a time.
It would allow users to install Chum GUI, add repo from it and just use that. Maybe, haven't looked into official Store requirements, we can even submit GUI there.
Note that we would have to remove requirement for chum package from RPM SPEC then.

Good idea


controller recent changes

controller recent change: job version
bug fixes: namespace mutator capture non exist error to return empty list
ExtendEnumCases should not make http call when no enum cases

Codecov Report

Merging #215 into master will decrease coverage by 0.02%.
The diff coverage is 44.44%.

@@            Coverage Diff             @@
##           master     #215      +/-   ##
==========================================
- Coverage   73.82%   73.79%   -0.03%
==========================================
  Files         150      150
  Lines       20304    20311       +7
==========================================
  Hits        14989    14989
- Misses       4174     4178       +4
- Partials     1141     1144       +3

Impacted Files
Coverage Δ

controller/mutators/etcd/namespace_mutator.go
68.96% <0%> (-5.11%)
:arrow_down:

controller/client/controller.go
70.54% <0%> (-1.12%)
:arrow_down:

client/schema.go
72.8% <100%> (+0.24%)
:arrow_up:

controller/mutators/etcd/job_mutator.go
72.81% <100%> (+0.26%)
:arrow_up:

memstore/cuckoo_index.go
90.26% <0%> (-1.01%)
: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 c4ae44e...a137060. Read the comment docs.


Support new V2 configuration format for tbot

Closes https://github.com/gravitational/teleport/issues/27455

TODO: Check version and throw an error if being used with an old version of tbot.

I'll hold off merging this until Teleport 14.0 ships :)


Downloading data - no access

Hello,
I'm not sure if this is on purpose or a permission error when downloading the data:
HTTP request sent, awaiting response... 403 Forbidden
2019-11-11 11:23:18 ERROR 403: Forbidden.
It seems like I don't have access to the server here: http://ict.icrar.org/store/staff/cwu/rgz_data
Thank you for your time,
In case this isn't a mistake, is there anything I can do to get access to the data?
I am working on an university project, feel free to ask for as many details as you wish.
edit: the only thing that downloads when running the download script is annotations.tar, but the system does not recognize it returning "this does not look like a tar file" when trying to work with it.

Hi Valeria-alex,
Sorry about the link above not working.  The new URL from which the data can be downloaded is as follows: https://cloudstor.aarnet.edu.au/plus/s/agKNekOJK87hOh0
Hope that this works better for you.

Thank you so much! I got it working
I replaced the link in the script but still there were errors.
However it worked when I downloaded all .tar and .tgz manually. I don't know if it's a problem in my setup or in the script.
Using dowloand_data.py I would always get this error:
raise Exception("fail to extract %s: %s" % (tar_file, msg))

Exception: fail to extract $PATH/D3/D3_model.tar: tar: This does not look like a tar archive
tar: Skipping to next header
tar: Exiting with failure status due to previous errors
or
raise Exception("fail to extract %s: %s" % (tar_file, msg))
Exception: fail to extract $PATH/PNGImages/D3_images.tgz:
gzip: stdin: not in gzip format
tar: Child returned status 1
tar: Error is not recoverable: exiting now
so what I did was place the archive manually in the respective folder whenever the downloading script would stop and it was able to extract the files. Downloading it directly from the script results in a different type of file (?)

I am glad that the download worked manually.  I am not sure if it's your setup or the script. Just a guess but does the script work if you provided the full pathname rather than one with $PATH ?

I have the full path, it downloads files of seemingly the proper type but then they are not recognized as that type

Thanks for the confirmation valeria-alex. I think that you may be right and it might be the difference in setup then.

@ivyw hi, can you provide the d2 dataset , F dataset and W dataset ?  i didn't find from https://cloudstor.aarnet.edu.au/plus/s/agKNekOJK87hOh0            thank you very much

Hi @qiuchenjian
I just checked and it seems that our cloudstor server is currently down. If possible can you wait a few days until it is back up again?  I suspect that it is undergoing some maintenance. Sorry for the inconvenience.


fix: make kms_key_crn null by default in solution

Description
kms_key_crn was not marked as null by default in the solution so was required, as the observability portion is now optional this should be null and validation is done at the base level of the module to determine if it is needed.
Release required?

[ ] No release
[x] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

var.kms_key_crn is no longer a required variable in the solution
Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

allowed_ip addresses should also be set to default     = []

/run pipeline

:tada: This PR is included in version 1.2.1 :tada:
The release is available on:

GitHub release
v1.2.1

Your semantic-release bot :package::rocket:

Still not there...
2024/04/05 12:33:20 Terraform plan | Error: Invalid function argument
2024/04/05 12:33:20 Terraform plan |
2024/04/05 12:33:20 Terraform plan |   on .terraform/modules/account_infrastructure_base.account_settings/main.tf line 55, in locals:
2024/04/05 12:33:20 Terraform plan |   55:   concatenated_allowed_ip_addresses     = join(",", var.allowed_ip_addresses)
2024/04/05 12:33:20 Terraform plan |     ├────────────────
2024/04/05 12:33:20 Terraform plan |     │ while calling join(separator, lists...)
2024/04/05 12:33:20 Terraform plan |     │ var.allowed_ip_addresses is null
2024/04/05 12:33:20 Terraform plan |
2024/04/05 12:33:20 Terraform plan | Invalid value for "lists" parameter: argument must not be null.


This command does not work when attempting to update records that are stored in a zonescope.

Attempting to use this command to update records in a zonescope (both -zonename and -zonescope specified) does not work. It always returns an error:
set-dnsserverresourcerecord : Resource record in OldInputObject not found in scope <scopename> of zone <zonename> on server <server>

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

ID: 2335a3db-5fe1-404b-aa3d-a3b4f10086fa
Version Independent ID: 3dfd55d0-2210-626c-b9a6-f584316ec374
Content: Set-DnsServerResourceRecord (dnsserver)
Content Source: docset/windows/dnsserver/set-dnsserverresourcerecord.md
Product: w10
Technology: powershell-windows
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

@officedocsbot assign @e0i

@BearGFR
The issues section of this repository is for product documentation issues.
Please consider checking any relevant resources and asking the community about your problem via the following link.
PowerShell Community
Thanks.

@officedocsbot close


Improve flux-local test support for helm releases

Improve support for flux-local test command by rewriting the test collector plugin and adding proper helm release template expansion.

Codecov Report

Merging #71 (842127b) into main (eae4aa8) will decrease coverage by 1.06%.
The diff coverage is 87.02%.

@@            Coverage Diff             @@
##             main      #71      +/-   ##
==========================================
- Coverage   92.73%   91.68%   -1.06%
==========================================
  Files          15       14       -1
  Lines         964     1046      +82
==========================================
+ Hits          894      959      +65
- Misses         70       87      +17

Impacted Files
Coverage Δ

flux_local/tool/test.py
86.40% <85.34%> (-13.60%)
:arrow_down:

flux_local/git_repo.py
98.37% <100.00%> (ø)

flux_local/tool/flux_local.py
84.84% <100.00%> (-0.87%)
:arrow_down:

flux_local/tool/visitor.py
98.21% <100.00%> (ø)

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


_facet_array should work against views

I created this view: https://json-view-facet-bug-demo-j7hipcg4aq-uc.a.run.app/russian-ads-8dbda00/ads_with_targets
CREATE VIEW ads_with_targets as select ads.*, json_group_array(targets.name) as target_names from ads
  join ad_targets on ad_targets.ad_id = ads.id
  join targets on ad_targets.target_id = targets.id
  group by ad_targets.ad_id

When I try to apply faceting by array it appears to work at first: https://json-view-facet-bug-demo-j7hipcg4aq-uc.a.run.app/russian-ads/ads_with_targets?_facet_array=target_names
But actually it's doing the wrong thing - the SQL for the facets uses rowid, but rowid is not present on views at all! These results are incorrect, and clicking to select a facet will fail to produce any rows: https://json-view-facet-bug-demo-j7hipcg4aq-uc.a.run.app/russian-ads/ads_with_targets?_facet_array=target_names&target_names__arraycontains=people_who_match%3Ainterests%3AAfrican-American+Civil+Rights+Movement+(1954—68)
Here's the SQL it should be using when you select a facet (note that it does not use a rowid):
https://json-view-facet-bug-demo-j7hipcg4aq-uc.a.run.app/russian-ads?sql=select+*+from+ads_with_targets+where+id+in+(
++++++++++++select+ads_with_targets.id+from+ads_with_targets%2C+json_each(ads_with_targets.target_names)+j
++++++++++++where+j.value+%3D+%3Ap0
++++++++)+limit+101&p0=people_who_match%3Ainterests%3ABlack+(Color)
So we need to do something a lot smarter here. I'm not sure what the fix will look like, or even if it's feasible given that views don't have a rowid to hook into so the JSON faceting SQL may have to be completely rewritten.
datasette publish cloudrun \
    russian-ads.db \
    --name json-view-facet-bug-demo \
    --branch master \
    --extra-options "--config sql_time_limit_ms:5000 --config facet_time_limit_ms:5000"

It may be that some facet implementations (ArrayFacet in this case) need a way to detect if they are supported by the thing they are running against (must be a rowid table in this case) and avoid suggesting themselves if they are not compatible. This may require a change to the information we make available to the suggest() method (information passed to the Facet class constructor).

I think this SQL recipe may work instead:
select
  *
from
  ads_with_targets
where
  'people_who_match:interests:African-American Civil Rights Movement (1954—68)' in (
    select
      value
    from
      json_each(target_names)
  )
  and 'interests:Martin Luther King III' in (
    select
      value
    from
      json_each(target_names)
  )

https://json-view-facet-bug-demo-j7hipcg4aq-uc.a.run.app/russian-ads?sql=select
++*
from
++ads_with_targets
where
++'people_who_match%3Ainterests%3AAfrican-American+Civil+Rights+Movement+(1954—68)'+in+(
++++select
++++++value
++++from
++++++json_each(target_names)
++)
++and+'interests%3AMartin+Luther+King+III'+in+(
++++select
++++++value
++++from
++++++json_each(target_names)
++)&interests=&African=&Martin=

Applied that fix to the arraycontains filter but I'm still getting bad results for the faceting:

I think this code is wrong in the ArrayFacet class: https://github.com/simonw/datasette/blob/502c02fa6dde6a8bb840af6c4c8cf858aa1db687/datasette/facets.py#L357-L364

It looks like the problem here is that some of the tags occur more than once in the documents:

So they get counted more than once, hence the 182 count for something that couldn't possibly return more than 172 documents.

Given this query: https://json-view-facet-bug-demo-j7hipcg4aq-uc.a.run.app/russian-ads?sql=select
++j.value+as+value%2C
++count(*)+as+count
from
++(
++++select
++++++id%2C
++++++file%2C
++++++clicks%2C
++++++impressions%2C
++++++text%2C
++++++url%2C
++++++spend_amount%2C
++++++spend_currency%2C
++++++created%2C
++++++ended%2C
++++++target_names
++++from
++++++ads_with_targets
++++where
++++++%3Ap0+in+(
++++++++select
++++++++++value
++++++++from
++++++++++json_each([ads_with_targets].[target_names])
++++++)
++)
++join+json_each(target_names)+j
group+by
++j.value
order+by
++count+desc%2C
++value
limit
++31&p0=people_who_match%3Ainterests%3AAfrican-American+culture
select
  j.value as value,
  count(*) as count
from
  (
    select
      id,
      file,
      clicks,
      impressions,
      text,
      url,
      spend_amount,
      spend_currency,
      created,
      ended,
      target_names
    from
      ads_with_targets
    where
      :p0 in (
        select
          value
        from
          json_each([ads_with_targets].[target_names])
      )
  )
  join json_each(target_names) j
group by
  j.value
order by
  count desc,
  value
limit
  31

How can I return a count of the number of documents containing each tag, but not the number of total tags that match including duplicates?

This looks like it might work:
with inner as (
  select
    id,
    file,
    clicks,
    impressions,
    text,
    url,
    spend_amount,
    spend_currency,
    created,
    ended,
    target_names
  from
    ads_with_targets
  where
    :p0 in (
      select
        value
      from
        json_each([ads_with_targets].[target_names])
    )
),
items as (select
  distinct j.value, inner.*
from
  json_each([inner].[target_names]) j join inner
)
select value, count(*) from items group by value order by count(*) desc

It uses a CTE which were introduced in SQLite 3.8 - and AWS Lambda Python 3.9 still provides 3.7 - but I've checked and I can use pysqlite3-binary to work around that there so I'm OK relying on CTEs for this.

I tried this and it seems to work correctly:
        for source_and_config in self.get_configs():
            config = source_and_config["config"]
            source = source_and_config["source"]
            column = config.get("column") or config["simple"]
            facet_sql = """
                with inner as ({sql}),
                deduped_array_items as (
                    select
                        distinct j.value,
                        inner.*
                    from
                        json_each([inner].{col}) j
                        join inner
                )
                select
                    value as value,
                    count(*) as count
                from
                    deduped_array_items
                group by
                    value
                order by
                    count(*) desc limit {limit}
            """.format(
                col=escape_sqlite(column), sql=self.sql, limit=facet_size + 1
            )

The queries are very slow though - I had to bump up to 2s time limit even against only a view returning 3,499 rows.

Actually with the cache warmed up it looks like the facet query is taking 150ms which is good enough.

Also note that this demo data is using a SQL view to create the JSON arrays - the view is defined as such:
CREATE VIEW ads_with_targets as
select
  ads.*,
  json_group_array(targets.name) as target_names
from
  ads
  join ad_targets on ad_targets.ad_id = ads.id
  join targets on ad_targets.target_id = targets.id
group by
  ad_targets.ad_id;

So running JSON faceting on top of that view is a pretty big ask!

Tests are failing and I think it's because the facets come back in different orders, need a tie-breaker. https://github.com/simonw/datasette/runs/4219325197?check_suite_focus=true


Doctrine Common 2.5.0 Release compatibility

Error message:
Compile Error: Declaration of Doctrine\KeyValueStore\Mapping\ClassMetadataFactory::doLoadMetadata() must be compatible with Doctrine\Common\Persistence\Mapping\AbstractClassMetadataFactory::doLoadMetadata($class, $parent, $rootEntityFound, array $nonSuperclassParents)

my bad, was on 0.1 and not dev-master


Arm64 build

Hey guys,
Build be nice to have a binary for Raspberry Pi

Hey @kristianpaul,
Thanks for your interest in this project!
There are a couple of linux/arm builds on the releases page:
https://github.com/dennis-tra/punchr/releases/tag/v0.9.0
If you scroll down you should find some punchr_cli_linux_arm... CLI binaries.
Is that what you were looking for?

Already checked https://github.com/dennis-tra/punchr/releases but

Thanks, I was looking at  https://github.com/dennis-tra/punchr/releases initially


number of parameters at using update2() function

Hello,
I found one code error when I tried to reproduce code.

In my analysis, this TypeError raises at the situation when experience includes Coreference Resolution task. (only standalone-linking cases passed this error) And this occurs at the time when code tries to save trained model.
The reason of this error was here: the number of parameters in update2 was different at "/src/metrics/coref.py" and "/src/metrics/linker.py" (check following screenshots)
But I can't find the solution of this issue.

Can anyone try to reproduce this code by theirselves and check this issue? If this issue only occur on my PC, I'd like to know other PC setups.
Thanks.

Dear SEONHO AN,
Thanks for your interest in our work and for raising the issue! Indeed, there was a bug due to refactoring in our original code. It has to be solved now (we are still testing this), please let us know if the issue persist or any other problem/question you may have.
Kind regards,
Klim


add real data table titles and descriptions

Current titles and descriptions are placeholders, assigning this to you @sbeaulieu as you may have ideas about what to put in these metadata fields.

chl-transect-underway-discrete.csv
NES-LTER underway discrete chlorophyll data
Underway discrete chlorophyll data during NES-LTER Transect cruises
chl-transect-underway-post-cal.csv
NES-LTER post-calibrated underway fluorometer data
Post-calibrated underway fluorometer data during NES-LTER Transect cruises

@sbeaulieu feel free to kick this back to me if you'd like me to make these updates.


NullPointerException when starting KafkaMessageListenerContainer without setting ApplicationContext

Version: 2.5.6.RELEASE
Issue:
When providing fully-qualified class names as value for ConsumerConfig.KEY_DESERIALIZER_CLASS_CONFIG or ConsumerConfig.VALUE_DESERIALIZER_CLASS_CONFIG, KafkaMessageListenerContainer is no longer able to start without assigning an ApplicationContext first.
The reason is because it would need the ClassLoader from ApplicationContext in order to check whether deserializer class names are valid. However, if ApplicationContext is not set, it simply throw NullPointerException.
Steps to re-produce:

Setting up a similar example as https://docs.spring.io/spring-kafka/docs/2.5.6.RELEASE/reference/html/#a-very-very-quick-example.
Replace props.put(ConsumerConfig.KEY_DESERIALIZER_CLASS_CONFIG, IntegerDeserializer.class); props.put(ConsumerConfig.KEY_DESERIALIZER_CLASS_CONFIG, "org.apache.kafka.common.serialization.IntegerDeserializer");
Run the test.

Expected: Test ran successfully. or failed with a descriptive error.
Actual: Test threw NullPointerException
java.lang.NullPointerException
	at org.springframework.kafka.listener.KafkaMessageListenerContainer$ListenerConsumer.checkDeserializer(KafkaMessageListenerContainer.java:940)
	at org.springframework.kafka.listener.KafkaMessageListenerContainer$ListenerConsumer.<init>(KafkaMessageListenerContainer.java:668)
	at org.springframework.kafka.listener.KafkaMessageListenerContainer.doStart(KafkaMessageListenerContainer.java:302)
	at org.springframework.kafka.listener.AbstractMessageListenerContainer.start(AbstractMessageListenerContainer.java:338)

If there's no context, we should use the container's classloader.

I'm facing this issue. How can I locally resolve this?
Or do I need to wait for this to get released?

You should not be manually creating a listener container; it is designed to be managed by Spring; if there is not a proper application context, many features will not work. Define it as a Spring bean; if you are using Spring Boot, use the auto configured container factory to create it.
In a test case (such as referenced in the original post), it is ok to set it to mock(ApplicationContext.class).

@sarvesh05 I solved it by manually setting the application context to new GenericApplicationContext().

Hi, the issue is also present in spring-kafka 2.3.11 (part of Spring Boot 2.2.10.RELEASE). No backport to 2.3.x ?

I have observed that 2.4.9.RELEASE is also affected

As I said above, creating a container outside of a Spring context is not advised because a lot of functionality depends on it. It is not back-ported to all supported versions because there is a simple work around for test cases like this. I back ported to 2.5.x as a convenience since that was the version it was originally reported against.


code gen overwrites value of up for multiple up statements

Code:
sheesh(){
    whole i = 1#
    kung(i == 1){
        up("true bitch!")#
        up("i'm up in this bitch!")#
    }
}

Output:
"i'm up in this bitch!"
"i'm up in this bitch!"

bagoong issue now resolved.
Terminal output now resolved.
C:\Users\David\Downloads\sheesh#\Compiler-Sheesh>sheesh.exe
true bitch!i'm up in this bitch!


Is this Project Still Maintained?

Hello,
I am thinking of using this project in an industrial automation environment, so it's important it works properly.  I see that the last commit was on Sept 3rd, 2019 with 15 open pull requests.  Is this project still maintained by anyone?

Hey,
The project works well and has been used in several places around the world with great success.  I began doing a typescript rewrite about a year ago but have not yet completed it.  I do not have the time available at this point to fully support the project unfortunately though I think you will be able to use it with success if you choose to.  If you have any questions, feel free reach out and Ill do my best to anwser them.

@cmseaton42 OK.  Great.  I plan on using this in an express TypeScript project.

@xinthose I’m actively using a version from this pull request:
https://github.com/cmseaton42/node-ethernet-ip/pull/25
The only difference is support for UDT, which I need. Otherwise, the standard library works fine. I think everyone just got busy and stuff hasn’t been merged in.
For myself, once the library was working with enough features, I moved onto things that use the library. 🙂
I’ve had this running in lots of production environments for two years, with new ones added all of the time. Zero issues.
I also use with TypeScript.

@jhenson29 excellent; thank you for the feedback gentlemen


tsp-openapi3 - scope top-level parameters to Parameters namespace

Fixes #4151
This PR updates tsp-openapi3's model generation so that all top-level parameters (#/components/parameters) are nested in a Parameters block namespace.
Prior to this change, if top-level parameter had the same name as a top-level schema, we would attempt to merge the two. This worked OK if the schema was an object type, but led to broken results if the schema was anything else.
Note:
In the linked issue, it was suggested that top-level schemas not be scoped to their own namespace, so if a schema is referenced by a parameter, it will now qualify it with the file-level namespace. This PR introduces a context object that contains some state that can be passed around. This is useful for keeping track of the file-level namespace and using it when necessary, but the context will also be useful in cases where we need to look at the definition of a referenced schema from another schema.

All changed packages have been documented.

:white_check_mark: @typespec/openapi3

Show changes
@typespec/openapi3 - fix ✏

Fixes issue in tsp-openapi3 that resulted in component schemas and parameters with the same name being merged into a single TypeSpec data type.

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs


$(...).select2 is not a function

Hi everybody,
I'm having trouble with django-autocomplete-light.
I'm using the last version on django 1.11.7 with django-jet 1.0.6.
When I try to use django-autocomplete-light in the admin interface, in change_list template, I've got this error in console:
$(...).select2 is not a function
For showing the autocomplete I've created a custom form like this:
class EmailsAdminForm(ModelForm):     class Meta:         model = Emails         fields = '__all__'         widgets = {             'user': autocomplete.ModelSelect2Multiple(url='customer-autocomplete'),             'date': autocomplete.ModelSelect2Multiple(url='date-autocomplete'),         }
It seems a problem of jquery.
Someone know how to fix this?
Thanks in advance.

Is there anything in the browser error console ?

This is the complete code showed in console:
Uncaught TypeError: $(...).select2 is not a function     at HTMLSelectElement.<anonymous> (select2.js:102)     at HTMLDocument.dispatch (jquery.js:4737)     at HTMLDocument.elemData.handle (jquery.js:4549)     at Object.trigger (jquery.js:7819)     at HTMLSelectElement.<anonymous> (jquery.js:7903)     at Function.each (jquery.js:365)     at jQuery.fn.init.each (jquery.js:137)     at jQuery.fn.init.trigger (jquery.js:7902)     at HTMLSelectElement.initialize (autocomplete.init.js:47)     at Function.each (jquery.js:365)

What happens if you load the widget's media after jquery ?
http://django-autocomplete-light.readthedocs.io/en/master/tutorial.html#using-autocompletes-outside-the-admin

But I'm using it inside the admin, not outside.
This tutorial is for using it outside.

You're using it in django-jet, not django.contrib.admin.
For $.select2 to be loaded, we need the form.media to render correctly.
Could you paste the page html ?

Ok, this is the template used by django-jet:
`{% extends "admin/base_site.html" %}
{% load i18n admin_urls static admin_modify %}
{% block extrahead %}{{ block.super }}

{{ media }}
{% endblock %}
{% block extrastyle %}{{ block.super }}<link rel="stylesheet" type="text/css" href="{% static "admin/css/forms.css" %}" />{% endblock %}
{% block coltype %}colM{% endblock %}
{% block bodyclass %}{{ block.super }} app-{{ opts.app_label }} model-{{ opts.model_name }} change-form{% endblock %}
{% if not is_popup %}
{% block breadcrumbs %}

{% trans 'Home' %}
› {{ opts.app_config.verbose_name }}
› {% if has_change_permission %}{{ opts.verbose_name_plural|capfirst }}{% else %}{{ opts.verbose_name_plural|capfirst }}{% endif %}
› {% if add %}{% blocktrans with name=opts.verbose_name %}Add {{ name }}{% endblocktrans %}{% else %}{{ original|truncatewords:"18" }}{% endif %}

{% endblock %}
{% endif %}
{% block content %}
{% block object-tools %}
{% if change %}{% if not is_popup %}

    {% block object-tools-items %}

        {% url opts|admin_urlname:'history' original.pk|admin_urlquote as history_url %}
        {% trans "History" %}

    {% if has_absolute_url %}{% trans "View on site" %}{% endif %}
    {% endblock %}

{% endif %}{% endif %}
{% endblock %}
{% csrf_token %}{% block form_top %}{% endblock %}

{% if is_popup %}{% endif %}
{% if to_field %}{% endif %}
{% if save_on_top %}{% block submit_buttons_top %}{% submit_row %}{% endblock %}{% endif %}
{% if errors %}

    {% if errors|length == 1 %}{% trans "Please correct the error below." %}{% else %}{% trans "Please correct the errors below." %}{% endif %}

    {{ adminform.form.non_field_errors }}
{% endif %}
{% block field_sets %}
{% for fieldset in adminform %}
{% include "admin/includes/fieldset.html" %}
{% endfor %}
{% endblock %}
{% block after_field_sets %}{% endblock %}
{% block inline_field_sets %}
{% for inline_admin_formset in inline_admin_formsets %}
{% include inline_admin_formset.opts.template %}
{% endfor %}
{% endblock %}
{% block after_related_objects %}{% endblock %}
{% block submit_buttons_bottom %}{% submit_row %}{% endblock %}
{% block admin_change_form_document_ready %}
<script type="text/javascript"
id="django-admin-form-add-constants"
src="{% static 'admin/js/change_form.js' %}"
{% if adminform and add %}
data-model-name="{{ opts.model_name }}"
{% endif %}>

{% endblock %}
{# JavaScript for prepopulated fields #}
{% prepopulated_fields_js %}

{% endblock %}`

I was thinking about the rendered html, btw i don't see form.media in this template, perhaps that's an issue ?
{{ form.media }} should be after the script tag for jquery.js and then $.select2 should be defined ;)

You're right!
I don't know if is an issue of django-jet but I extended the template, included the jquery and then the form media and know it works.
Thank you very much!!
If someone else is using django-jet and face the same problem, you need to extend the change_list.html template of your model like this to get django-autocomplete-light works:
`{% extends "admin/change_form.html" %}
{% load static %}
{% block extrahead %}

{{ adminform.form.media }}
{% endblock %}`

Nice one !!
Would you like to contribute a short note about django-jet in the tutorial like there is grappeli ?
https://github.com/yourlabs/django-autocomplete-light/blob/master/docs/install.rst

Yes, no problem :)
When I have some free time I'll do it.


Error in docker ./build.sh in Step 3 of dockerfile

npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /interop/server/npm-debug.log
npm ERR! not ok code 0
The command '/bin/sh -c sudo ln -snf /usr/bin/nodejs /usr/local/bin/node &&     sudo npm install -g         karma         karma-chrome-launcher         karma-jasmine         karma-phantomjs-launcher         phantomjs' returned a non-zero code: 1
Experiencing this error on Ubuntu.

Are you following the docs? You shouldn't need to run ./build.sh unless
you're trying to run a custom version of the server.
On Tue, Nov 28, 2017 at 10:28 PM, Eli Minkin<EMAIL_ADDRESS>wrote:

npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR! /interop/server/npm-debug.log
npm ERR! not ok code 0
The command '/bin/sh -c sudo ln -snf /usr/bin/nodejs /usr/local/bin/node
&& sudo npm install -g karma karma-chrome-launcher karma-jasmine
karma-phantomjs-launcher phantomjs' returned a non-zero code: 1
Experiencing this error on Ubuntu.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/auvsi-suas/interop/issues/290, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACOBmCzOp7pUnG02PnoUIclgK_eSQPt_ks5s7M9xgaJpZM4QuXmO
.

--
Phillip Tischler
Software Engineer & Site Reliability Engineer
<EMAIL_ADDRESS>


Metrics on workload certificate expiry

Describe the feature request
Since istio 1.7 workload certificates are not stored as kubernetes secrets anymore, but rather on the sidecar proxy container. Viewing them works with the istioctl pc secret <pod> command. They are also exposed over the admin api under localhost:15000/certs, where the istioctl command seams to use that endpoint in combination with port-forwarding.
We want to be able to monitor the workload certificates for possible expiry, e.g. by scraping that information from istiod.
Describe alternatives you've considered
An option we considered was defining values.sidecarInjectorWebhook.templates: certificate-monitor and maybe set the values.sidecarInjectorWebhook.defaultTemplates: [sidecar, certificate-monitor]. An app in the container would basically query localhost:15000/certs and extract and exposes the certificate expiry, being scraped by prometheus on a regular basis. Got a working proof of concept, but the overhead on both prometheus and running a second sidecar for every istio enabled workload is unproportional, especially when having thousands of workloads.
Also one could think about opening the admin endpoint for a specific monitoring application that collects the information and exposes them to prometheus. This was discarded because of the obvious security concerns.
Affected product area (please put an X in all that apply)
[ ] Docs
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[x] Extensions and Telemetry
[x] Security
[ ] Test and Release
[] User Experience
[ ] Developer Infrastructure

This looks more like a metric at istio proxy instead of istiod? Envoy has a metric exposed envoy_server_days_until_first_cert_expiring although it will almost always be 0 since the default expiry 24 hours.

@bianpengyuan   Is there any way i can get this metrics from pilot-agent ? (In seconds )

The metric is not yet available, it needs to be added, probably via a periodic goroutine running against the secret cache at istio-agent: https://github.com/istio/istio/blob/284def0c93cc2dc37b07a1d7acd74f26eb24fb63/security/pkg/nodeagent/cache/secretcache.go#L93

@bianpengyuan IIRC, we can add metric here?
https://github.com/istio/istio/blob/199b33bf6d0515c995cde922973a2ec2a4dd4c20/security/pkg/nodeagent/cache/secretcache.go#L600

@bianpengyuan IIRC, we can add metric here?

This only happens when generating the cert? But we'd want periodic report of the current workload cert expiry?

@bianpengyuan
https://github.com/istio/istio/blob/199b33bf6d0515c995cde922973a2ec2a4dd4c20/pkg/security/security.go#L279
if the comments is right, GenerateSecret will called when cert expiry.

@zirain yeah, and I don't think we should only record expiry when cert is about to expire, but should happen periodically like every 1 min.

@bianpengyuan do you mean a Gauge metric which will renew every min?

Yeah, and I believe we have something similar at istiod for root cert expiry?

Yeah, and I believe we have something similar at istiod for root cert expiry?

sound good
BTW,prefer to export the cert metric by default (it will not cost too much), WDYT?

BTW,prefer to export the cert metric by default (it will not cost too much), WDYT?

sgtm.

Yeah, and I believe we have something similar at istiod for root cert expiry?

a metric(named citadel_server_root_cert_expiry_timestamp) exists in pilot-discovery right now, but the format is not friendly to ready.
# HELP citadel_server_root_cert_expiry_timestamp The unix timestamp, in seconds, when Citadel root cert will expire. A negative time indicates the cert is expired.
# TYPE citadel_server_root_cert_expiry_timestamp gauge
citadel_server_root_cert_expiry_timestamp 1.959728195e+09

Yeah, and I believe we have something similar at istiod for root cert expiry?

a metric(named citadel_server_root_cert_expiry_timestamp) exists in pilot-discovery right now, but the format is not friendly to ready.
# HELP citadel_server_root_cert_expiry_timestamp The unix timestamp, in seconds, when Citadel root cert will expire. A negative time indicates the cert is expired.
# TYPE citadel_server_root_cert_expiry_timestamp gauge
citadel_server_root_cert_expiry_timestamp 1.959728195e+09

Are you adding this changes as well in your PR.

this's already in master branch, nothing need to do.

this's already in master branch, nothing need to do.

I don't see this metric,  I am using cert-manager , disabled the istiodCA does that mean none of citedal metrics available?

this's already in master branch, nothing need to do.

I don't see this metric, I am using cert-manager , disabled the istiodCA does that mean none of citedal metrics available?

@zirain  How to get this root cert expiry ?


RoleDescriptor Step - Not Required

[Enter feedback here]
The points mentioned below in the document are no longer required to be performed as guided by the Qlik Technician :
a. Open the FederationMetaData.xml file, which you have downloaded from Azure portal in a text editor.
b. Search for the value RoleDescriptor. There are four entries (two pairs of opening and closing element tags).
c. Delete the RoleDescriptor tags and all information in between from the file.
d. Save the file and keep it nearby for use later in this document.

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

ID: 725a7b74-1b0f-1ba6-8d23-661dc31e0e38
Version Independent ID: 8f34c027-02af-98ef-ad1b-401883d727b0
Content: Tutorial: Azure Active Directory integration with Qlik Sense Enterprise
Content Source: articles/active-directory/saas-apps/qliksense-enterprise-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@ameymulay
Thanks for your feedback! We will investigate and update as appropriate.

Hi @ameymulay, could you please provide more context around the suggested change?

@ameymulay, Then what about the Configure Qlik Sense Enterprise SSO 7-e point, do we need to remove this context as well?
will you please provide some more information about this, before going to resolve?

@ameymulay, will you give reply to the above query?

we are working on the issue, will reflects the suggested changes into the main article soon.
#please-close


Set often needed parameters to app parameters

slaxweb/framework#119

Moved to Bootstrap.

Will be handled in router after all.

Resolved.


Bug/helpers for outgoing commands/GitHub#342

Block helpers for outgoing commands. This PR is needed for the non determinism issue. These changes will be needed by the CHIPClusters-src.zapt and CHIPClusters.zapt as per my understanding.
The way I understood those templates is that outgoing command signatures are generated for here.

This PR is required for https://github.com/project-chip/connectedhomeip/pull/13871

@tcarmelveilleux , @bzbarsky-apple , @vivien-apple :
Please review. This needs to get pushed into the master to unblock another PR @brdandu is working on on the main CHIP repo, which should be a first step towards solving the non-deterministic behavior.


Creating selectors

Hi @dalefrancis88, I'm coming to redux from a FP background and found this nice library.
One thing I'm wondering:
This library allows you to use lenses to drill down into the store for "writing" aka reducers.
Would it make sense to use the same lenses for reading (i.e. creating Redux selectors)?
Or is there something I'm missing that makes creating selectors from lenses unnecessary?

Nice, Welcome. So i've actually been playing around with this idea as part of a local implementation.
What I'd like to get to is a point where I could compose the lens nicely. The idea here is for when you have the multiple levels instead of creating one as a path all the way from the root reducer, you can just create it at the level that's relative. You could then "add" a level as needed. The idea then would be to be able to then be able to reference a collection of the lens to create the state for a component in a call to connect it would create the mapStateToProps func
Apologies for the late reply, you caught me right on the week that i was moving house

Was thinking, if you come from an FP background, you should check out Crocks

Hi @dalefrancis88, thanks for your response despite being busy!
We've also been thinking of composing lenses for drilling down into data structures for both the read and the write side.
In fact, I was so convinced that something like this must exist that I was quite surprised that there are only ~3 lens implementations for Redux and only yours seems to be actively maintained :open_mouth:
I'm still wondering if there's some obvious alternative that I'm missing, or JS people are just not aware of lenses (and potentially other beneficial FP constructs) and implement the same logic by hand over and over...
We were/are using reselect to do the drill-down on the read side. Would this also be an option of creating selectors?

I think when it comes to JS and FP its a hard won battle. The issue I see is there is so much focus on trying to "fix" javascript by developers that don't really quite grasp the freedom of a dynamic language and push to make it closer to something like C#.
Reselect is a good library, I don't use it though, not for any reason other than i'm using that Crocks library that has a bunch of ADT's and I rolled my own before that was ever released. It's also the reason I havn't moved it into this library just yet is that it relies on that other one which I don't want to do. So I'm trying to make sure i'm fully across the problem space first and to see any potential pitfalls.
That being said, the idea of an integration could be interesting, I'll take an in depth look at the reselect but to ask you, how do you think that'd work in your mind?


[Concurrency] Add syntax for the await expression.

Paired with https://github.com/apple/swift/pull/33199

@swift-ci please test


Cooperation

Hello @thisisthekap
as you may know I am the maintainer of Maui.RevenueCat.InAppBilling and based on your Xamarin bindings I was able recreate up-to-date .NET8 Android and iOS bindings. I try to describe in READMEs steps that were done for creating the binding so that also someone else could create binding himself.
I suppose when you maintain your Xamarin bindings more often than those MAUI ones that you did not migrate to MAUI yet. If you do would you be up to maintaining those binding+wrapper around them together ? So we don't have to do double-work. Currently mine are ~2days old and in case you would also start to use my MAUI InAppBilling wrapper it would simplify you the multiplatform access.
What do you think ?
Regards
Samuel

@Kebechet Can we take the discussion to some messenger app?

@thisisthekap in that case you can write me on messenger: https://m.me/Samuel.Sidor


fix IsPodWithPVC func notes.

typo update.

/retitle fix go doc for method IsPodWithPVC

/retitle fix go doc for method IsPodWithPVC
@a7i
Thanks every time for the retitle thing.
I am not a native english specker .
Thanks a lot.
🤭🤭

/lgtm

/retest


Deprecated registerBrickClass on BrickCollectionView

Closes #171

Codecov Report

Merging #172 into master will decrease coverage by 0.07%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #172      +/-   ##
==========================================
- Coverage   93.42%   93.34%   -0.08%
==========================================
  Files          39       39
  Lines        4107     4104       -3
  Branches      332      332
==========================================
- Hits         3837     3831       -6
- Misses        269      272       +3
  Partials        1        1

Impacted Files
Coverage Δ

Source/Bricks/Collection/CollectionBrick.swift
85.6% <ø> (-0.45%)
:arrow_down:

Source/ViewControllers/BrickViewController.swift
87.75% <ø> (ø)
:arrow_up:

Source/ViewControllers/BrickCollectionView.swift
94.77% <100%> (-0.65%)
: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 bb7128a...22a6224. Read the comment docs.

Closing due to inactivity


Implement TopLevel.Launcher

What does the pull request do?
Adds TopLevel.Launcher service to allow launching Uri and IStorageItem.
See https://github.com/AvaloniaUI/Avalonia/pull/14320/files#diff-dedbbea0fc7e00cbdde2962c6eb7ba6edc52d0af3894b74608e742000321963a for the public API changes.
How was the solution implemented (if it's not obvious)?
For desktop: simple Process shell execute.
For browser: window.open (only links).
For mobile: copy of essentials launcher with adjustments.
Both macOS and Windows platforms can be improved by using more specific APIs, like WinRT Launcher API on Windows. But I am going to leave it outside of this PR scope.
Checklist

[ ] Added unit tests (if possible)?
[x] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/avalonia-docs with user documentation

Fixed issues
Contributes to #7640
Fixes https://github.com/AvaloniaUI/Avalonia/discussions/13640

Great.
What do you think to add to "AppBuilder" the ability to override the behavior of ILauncher? May be useful in non-standard Linux distributions.

@workgroupengineering no, probably not now. If anything - we should support OpenURI and OpenFILE - https://github.com/flatpak/xdg-desktop-portal/blob/main/data/org.freedesktop.portal.OpenURI.xml.

@workgroupengineering no, probably not now. If anything - we should support OpenURI and OpenFILE - https://github.com/flatpak/xdg-desktop-portal/blob/main/data/org.freedesktop.portal.OpenURI.xml.

Ok, thanks I had already seen it. My proposal is a simple way to give those who use custom Linux distributions with DRM or FrameBuffer support the possibility to customize the behavior.
I was thinking of something like this:
          return AppBuilder.Configure<App>()
                      .UsePlatformDetect()
                      .WithInterFont()
                      .WithLaunchUriAsync( async uri=>
                      {
                          ...
                          return await Task.FromResult(false);
                      })

I can't say I am a fan of that. If it had to be done you would be better off somehow providing a custom implementation of ILauncher. But even that feels unnecessary.

This is great! I'll plan to resurrect the HyperlinkButton control and open a PR in a few weeks.


fix: Add trim to searchValue in OptionsUtils

Details

Added trim() for OptionsUtils so that leading/trailing spaces can be ignored when making the search calls.

Fixed Issues

$ https://github.com/Expensify/App/issues/6907
Tests

Tested the search with leading/trailing spaces for the input
Ensured the input doesn't change automatically

QA Steps

Click the Profile image to open settings
Navigate to any Workspaces
Go to Members >Manage Members
Invite any email with leading/trailing spaces
Ensure the search shows the result.


[Cognitive Services - Content Moderator] API Review

New API Review meeting has been requested.
Service Name: Cognitive Services - Content Moderator
Review Created By: Meng Ai
Review Date: 10/14/2022 7:00 AM PT
PR: https://github.com/Azure/azure-rest-api-specs/pull/20784
Hero Scenarios Link: Not Provided
Core Concepts Doc Link: Not Provided
Description: We would like to publish a new version of Content Moderator, we will provide new interfaces in this version. Currently we are working for the private preview, and we want to hear the advices of review board of our contract.
Detailed meeting information and documents provided can be accessed here

Meeting updated by Meng Ai
Service Name: Cognitive Services - Content Moderator
Review Created By: Meng Ai
Review Date: 10/14/2022 7:00 AM PT
PR: https://github.com/Azure/azure-rest-api-specs/pull/20784
Hero Scenarios Link: Not Provided
Core Concepts Doc Link: Not Provided
Description: Please refer to this PR, the original one was deleted: https://github.com/Azure/azure-rest-api-specs-pr/pull/8888
Detailed meeting information and documents provided can be accessed here

The PR was moved to: https://github.com/Azure/azure-rest-api-specs-pr/pull/8888

Meeting updated by Meng Ai
Service Name: Cognitive Services - Content Moderator
Review Created By: Meng Ai
Review Date: 10/14/2022 7:00 AM PT
PR: https://github.com/Azure/azure-rest-api-specs/pull/20784
Hero Scenarios Link: Not Provided
Core Concepts Doc Link: Not Provided
Description: Please refer to this PR, the original one was deleted: https://github.com/Azure/azure-rest-api-specs-pr/pull/8888
Detailed meeting information and documents provided can be accessed here

API Stewardship Board Review: 13-Oct-22

If you are not using fields diagnoses or modelOutputDetail then consider removing them until you need them. It is much easier to add fields than to change them and/or remove them later.

You may consider using polymorphism and the category as the discriminator.

POST /moderate/text should be modeled as an action, and should look like POST /contentmoderator:analyze-text
(See: performing an action )

Connect with @robch to determine what the right term should

Other Cognitive Services use the term language

It is a requirement to increment the new api version when new enums are added, even though it's not a breaking change.

categories should be required to avoid analyzing on categories that are not expected.

see: video on breaking changes

Regarding the API path, should they look like below? Or is any misunderstanding?
 Or is any misunderstanding?
 See the notes above and the section on performing an action. It should look like...
POST /contentmoderator:analyze-text


Expose original header case

Relates to amphp/http-client#242.

I'm seeing this in some code:
$request->setHeaders($originRequest->getHeaders());
we might need a setRawHeaders method to allow
$request->setRawHeaders($originRequest->getRawHeaders());?

Not sure how often that will happen. I guess the following is enough for now?
foreach ($originRequest->getRawHeaders() as [$key, $value]) {
	$request->addHeader($key, $value);
}

That's what I did, so yes!