how to deploy docker on app service by azure cli

I want to publish/update my bot service from git repo automatically.

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

ID: 7b1864e8-0c73-eaaa-94e6-5e8993a3bf9d
Version Independent ID: 9e246eaa-216b-c06e-1c11-4a5aa60453ff
Content: Quickstart: Run a custom Linux container - Azure App Service
Content Source: articles/app-service/containers/quickstart-docker.md
Service: app-service
GitHub Login: @msangapu-msft
Microsoft Alias: msangapu

Hi @zffocussss, Thanks for the feedback! We are currently investigating and will update you shortly.

Closing this legacy issue as we will not address it in our documentation. To ask a question regarding your specific scenario, go to Microsoft Q&A or Stackoverflow.
#please-close


Not able to get Node metrics in HA mode

Rancher server version:
HA setup with rancher server version - v2.2.0-alpha3
Steps to reproduce the problem:
Create a custom cluster with few nodes.
Enable monitoring at cluster level.
Go to node detail page and view "Node Metrics".
Following error is presented to the user:

Can be mitigated by scaling rancher server replicas down to 1, ie:
helm upgrade rancher-alpha/rancher --name rancher --namespace cattle-system --set hostname=my.rancher.com --set replica=1

I met the same bug. In my case, the Grafana icons showed but no live metrics.
Setup: v2.2.0-alpha3, HA

I have tried the same scenario several times, and it turns out the problem does not show up every time.

I tested and reproduced as well. It took six tries. When I did this I could not resolve by refreshing the browser. When I disabled monitoring again and re-enabled the issue persisted. Then on a third try charts came back.

Hi @davidnuzik , did you see the red error message get node from mgnt failed on the top right corner? If not, It should be another issue which might not related to HA setup.

@loganhz @jiaqiluo I did not get the red error messages. I replicated the same situation Jack posted where we just don't see any chart data. @jiaqiluo we may need a new separate issue for this situation where charts do not show up and no red error messages.


Code highlight for npm/js_package dependencies

While migrating from rules_nodejs I came across a limitation that affects the developer experience a lot. The previous yarn install rule I used can create and track the node_modules folder as a managed directory on Bazel.
I understand rules_js has another approach, but having the node_modules folder allowed all my external npm dependencies and local libraries (js_library) to be available to IDEs/text editors like VSCode. Right now, all the dependencies are seen as missing since there's no way for them to be found, removing access to definitions, syntax highlight, linters, etc. while coding.
Are there any alternatives or recommended approaches?

As you probably saw, there's a proposal likely to land to remove managed_directories.
The recommended way to teach TS language service about mapping package names to locations in the monorepo is with the paths property, aka. path mapping.
You can still run pnpm install to get a local node_modules folder for the IDE. How else is rules_js different from rules_nodejs without managed_directories?

The only difference I can think of now is that the rules_nodejs's js_library links local packages to the node_modules. Wondering how to achieve the same behavior with rules_js?
Example based on my use case: I have a lot of protobuf definitions and a target that outputs interfaces for them. They're local js libraries that need to be available at the IDE.

rules_js also links local packages into the node_modules. However the bazel-bin/path/to/package/node_modules tree is created lazily as actions run that require populating it with symlinks to the pnpm store.
Protobufs are a problem across all languages. See recent chatter about this in #go on bazel slack like https://bazelbuild.slack.com/archives/CDBP88Z0D/p1652721846297049.
The practical answer, though not "bazel-idiomatic", is to output the generated protos to the source tree for non-Bazel tooling to be able to locate them. Anything else is an uphill battle against the expectations of those tools.

Maybe I've missed something or the docs don't state it yet, but I just realized not only the js_package targets aren't linked to node_modules, but my application doesn't find the package at all.
If I have my js_package like:
js_package(
    name = "my_package",
    srcs = [":compile_ts"], # aspect swc rule that transpiles .ts to .js
    package = "my-package-name",
    replace_prefixes = {
        "src/": "",
    },
)

When I include the target to "my_package" in the data array on the swc or js_binary targets, it just doesn't work. I can't see anything linked in the node_modules inside the bazel-bin and  my application throws Error: Cannot find module 'my-package-name'.

Got it! I needed to use the link_js_package rule:
load("@npm//:defs.bzl", "link_js_packages", "package")
load("@aspect_rules_js//js:defs.bzl", "link_js_package")

link_js_packages()

link_js_package(
    name = package("@ingka/grpc-error").name,
    src = "//backend/packages/my-package:my_package",
    deps = [package("protobufjs")],
)

Then instead of providing my package target in the data, I pass package("my-package-name):
swc(
    name = "compile_ts",
    srcs = glob(["src/**/*.ts"]),
    data = [
        package("my-package-name"),
        "@npm//joi",
    ],
    source_maps = True,
    swcrc = ".swcrc",
)

I only noticed afterward that there was an example in this repo. My bad. Thank you for the nice discussion above. Like you said I'll use the paths for mapping dependencies in the IDE. The node_modules in the bazel-bin indeed has it all linked.

Got it! I needed to use the link_js_package rule:
load("@npm//:defs.bzl", "link_js_packages", "package")
load("@aspect_rules_js//js:defs.bzl", "link_js_package")

link_js_packages()

link_js_package(
    name = package("my-package-name").name,
    src = "//backend/packages/my-package:my_package",
    deps = [package("protobufjs")],
)

Then instead of providing my package target in the data, I pass package("my-package-name):
swc(
    name = "compile_ts",
    srcs = glob(["src/**/*.ts"]),
    data = [
        package("my-package-name"),
        "@npm//joi",
    ],
    source_maps = True,
    swcrc = ".swcrc",
)

I only noticed afterward that there was an example in this repo. My bad. Thank you for the nice discussion above. Like you said I'll use the paths for mapping dependencies in the IDE. The node_modules in the bazel-bin indeed has it all linked.

Ha, I was struggling with this too and your answer helped me in rules_ts. It's very new and needs more docs and better error messages.
Let me know if there's still something to fix (including where the error handling) could be more helpful)


remove number of .clone() calls

In many places we really do not need to clone the values and can just use references with lifetimes.
I also removed the mut declaration in the aardvark interface which is not needed and thus just causes confusion.
There is a lot more I could do but these ones were very simple to fix and I don't want to spend more time on this right now.

yah! LGTM

/lgtm
/hold

/hold cancel


DEV-10: Add more features for dev

Need more dev features supported for developers:
info: Function - prints informative data (show current venv, etc.)
enter off: Function - add new argument that deactivates the venv
package_update [dnn|api]: Function - update requirements.txt (should use --exclude-editable flag in pip freeze)
Write more details in env_setup.sh -instruction.
Add more if nessessary

renamed info -> status
renamed package_update -> update_package


Clarify guidance on UMDF v1

@tedhudek @wm1 another PR for your review. Thanks!

Learn Build status updates of commit 03a84fb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-driver-docs-pr/wdf/getting-started-with-umdf-version-2.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 721b939:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-driver-docs-pr/wdf/getting-started-with-umdf-version-2.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Error executing ./experiments/scripts/demo.sh $0

Error information is as follows:
Traceback (most recent call last):
File "./tools/demo.py", line 118, in
from networks.factory import get_network
File "/home/fiona/FromGithub/PoseCNN/tools/../lib/networks/init.py", line 8, in
from .vgg16 import vgg16
File "/home/fiona/FromGithub/PoseCNN/tools/../lib/networks/vgg16.py", line 2, in
from networks.network import Network
File "/home/fiona/FromGithub/PoseCNN/tools/../lib/networks/network.py", line 6, in
import backprojecting_layer.backprojecting_op as backproject_op
File "/home/fiona/FromGithub/PoseCNN/tools/../lib/backprojecting_layer/backprojecting_op.py", line 5, in
_backprojecting_module = tf.load_op_library(filename)
File "/home/fiona/.local/lib/python3.5/site-packages/tensorflow/python/framework/load_library.py", line 60, in load_op_library
lib_handle = py_tf.TF_LoadLibrary(library_filename)
tensorflow.python.framework.errors_impl.NotFoundError: /home/fiona/FromGithub/PoseCNN/tools/../lib/backprojecting_layer/backprojecting.so: undefined symbol: _ZN10tensorflow12OpDefBuilder4AttrENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE
Do you have any idea about what this might be caused and how to solve this kind of issue?
Thanks so much!!!
My configuration is:
Ubuntu 16.04
CUDA 8
Tensorflow 1.12.0
cmake 3.14.0-rc2
gcc 5.4.0

I solved it by downgrading Tensorflw to 1.2.0


Alternative version of callback-taking functions

I'm working out how to wrap egui's API for my scripting language. Something that would make this a lot easier is if there was an alternative API to the functions that take callbacks (like SidePanel::show, Window::show, and Ui::horizontal) which could be written in straight-line code. For example, an object that exposes a &mut UI to draw widgets, and has a finish method which drops it.
It looks like I can emulate this with some pairs of Ui::new and Ui::allocate_rect, but I'll have to rewrite all the useful stuff those functions do.

I might be wrong, but i think the point of closures is that they will run only if necessary (ui is shown).
With a straight sequence style of begin and end functions, we will have to throw around if shown else not shown kind of code. Or is there a more idiomatic code style?

egui passes closures to functions to call, like this:
ui.collapsing_header("Foo", |ui| {
    // …
});

These aren't really "callbacks" in the classical sense since they are called immediately (or never).
The reason for them is that the parent Ui must run some setup-code and then some tear-down code after the child has been layed out. In the code above there are two different Ui both with the name ui (parent/child), but since we only ever care about the innermost, this is a good use of shadowing.

One alternative could be this (used e.g. by Dear ImGui and Nuklear):
if ui.collapsing_header_begin("Foo") {
    // …
}
ui.collapsing_header_end();

This has two downsides:
A) it is way to easy to put the end call in the wrong place (or forget it completely), leading to a lot of subtle bugs that only show up a t runtime.
B) it requires the library (egui) to keep a manual stack of what is going on
We could of course consider adding support for a lower lever API like this, but due to B) above it would require a huge rewrite and redesign of egui.

Another alternative us using linear types (types you must manually close):
if let Some(child) = parent.collapsing_header("Foo") {
    // …
    child.end(parent); // must call, or it won't compile
}

Linear types aren't supported by Rust (though you can approximate them with hacks that yields linker errors if you fail to close them), but it also creates a lot of child/parent bindings, which can be confusing.

If Rust had something like Pythons with statements we could do:
with ui = ui.collapsing_header("Foo") {
   // …
}

and have the compiler insert the end call automatically. But Rust doesn't support anything like this.

Thanks for the explanation! What about something like RAII objects? This seems to be the pattern the standard library uses to run tear-down code, for example unlocking mutexes.

It might be ok to impl Drop on structs returned by parents. The only catch is that Rust doesn't guarantee destructors are always called. This is why e.g. crossbeam::scope takes a callable. For any cases where not calling the destructor would result in unsoundness, this is currently the best pattern offered by the language.

A RAII approach would also require a Ui to keep a reference to its parent Ui, leading to a lot of hassle with lifetimes.
I would be more interested in investigating a more low-level C-like API (with begin/end) and then build the current Ui interface on top of that as a convenience. But that's a lot of work. It also is not obvious how one would handle third-party containers.

Another downside of the current closure approach is code bloat and long compile times (due to excessive monomorphization)

+1, I want to bind egui to python but the callback approach is much worse than something I can wrap in a context manager
I'm imagining this to prototype it:
@ui.collapsing_header
def cb(ui):
    ...

but I'd really like to be able to do something like this (which I confirmed properly nests scope in Python without persistently shadowing the outer ui variable)
with ui.collapsing_header() as ui:
    ...

My vote is to try this with Drop:
if let Some(ui) = ui.collapsing_header() {
    ...
}

I looked into Drop a bit and it seems like it should be stable enough for RAII in most cases: https://aloso.github.io/2021/03/18/raii-guards.html#when-drop-isnt-called
I think Box::leak doesn't apply unless you're returning a boxed ui object. Same goes for reference counting cycles.

I'm working out how to wrap egui's API for my scripting language. Something that would make this a lot easier is if there was an alternative API to the functions that take callbacks (like SidePanel::show, Window::show, and Ui::horizontal) which could be written in straight-line code. For example, an object that exposes a &mut UI to draw widgets, and has a finish method which drops it.

Yes, a fluent API is great for programmers, but not friendly to automation, which is what a scripting system ultimately is.
There should be a low-level, imperative, step-by-step API for machine-controlled environments.

This is incomplete, but I'm working on a concept here:

Playing around with a Context Manager API: https://gist.github.com/rust-play/8667abe111edeb90bca9a44687ecdca4
Working on structurally wiring some of the ui api up to it: https://gist.github.com/rust-play/c618172551bebd4f93d929f870d4baa4

Basically, a method like ui.horizontal() would return an object conforming to the UiContextManager trait, which will need to run before and after the user's inner UI code. To make a context manager work, we really just need some state and an exit() method (or a Drop impl). Nested context managers will work fine, you just chain the exit() methods.
Children don't need a reference to their parent, the context manager can be a short lived object that has a reference to the parent, the child, the mutable Result object, and its internal state necessary for exit().
The ui.* api can be moved to a trait as demonstrated, so you can call ui methods like ui.horizontal() on the returned context manager.

This approach seems somewhat clean for defining using the context manager at call sites:
https://gist.github.com/rust-play/5dbf90f3c22fcec9ce0de00cb87f4273
If necessary the Context object could be either dynamic or templated, allowing each specific call site to stash arbitrary data.

If there is going to be a partial low level rewrite of egui api,  then we might as well check if the new api will be suitable to write bindings for at the very least in wasm/deno_core/pyo3 or rustpython/mlua.  If the new api is fine for one of these and not for others, it would be really sad. Ideally egui will have a simpler C based abi / ffi compatible as that will be the v⅘

we might as well check if the new api will be suitable to write bindings

Yes, I'm specifically doing this for PyO3. (Technically you could probably already bind the existing API, but it would require a really awkward rust -> language -> rust -> language stack to satisfy the egui callback API.)

I've prototyped a new context manager style here: https://gist.github.com/0b550112406de053660238627d3189d9
It allows you to write widgets like this, with both the setup and teardown code in the same function:
fn horizontal(&'a mut self, render: bool) -> Option<ContextManager> {
    if render {
        let child = Ui{id: self.id + 1, parent: self.id, data: 0};
        let local = 1;
        Some(ContextManager::new(move |parent, child| {
            println!("cm exit parent={:?} child={:?} local={}", parent, child, local);
            child.data += 1;
            parent.data += 1;
        }, self, child))
    } else {
        None
    }
}

It can be polyfilled to the old callback API style like this:
fn horizontal_cb<R>(&mut self, f: impl FnOnce(&mut Self) -> R, render: bool) -> Option<R> {
    if let Some(mut cm) = self.horizontal(render) {
        Some(f(&mut cm.child))
    } else {
        None
    }
}

@emilk thoughts?

I started a port here: https://github.com/talonvoice/egui/tree/no-callback
I ported everything in ui.rs to my context manager approach, with a _ctx method suffix. I modified all of the add_contents methods to use the equivalent _ctx method internally. At a glance, the egui demo app appears to completely work.
I have two notes:

I'm managing the lifetime of Response objects using an Option type. The callback compatibility methods end up calling unwrap() on the response (at a point in time where I know the Response has been set), because InnerResponse isn't optional.
My ContextManager assumes only one child UI object, and that assumption broke for columns(), so it's not properly wrapped for now.

Another alternative us using linear types (types you must manually close):
if let Some(child) = parent.collapsing_header("Foo") {
    // …
    child.end(parent); // must call, or it won't compile
}

Linear types aren't supported by Rust (though you can approximate them with hacks that yields linker errors if you fail to close them), but it also creates a lot of child/parent bindings, which can be confusing.

If you implement Drop for child then this could work: Playground


previewForm : enable scroll-bars in the modal dialog

hello,
Greetings!!
The plugin was awesome. but Sir My form size is big. so yes/ no button goes down side of the window and scrollbars is not appearing. please tell me how to enable scroll-bars in the modal dialog so that yes/ no button can come up.
my mail id is<EMAIL_ADDRESS>regards,
Dinesh

#previewBox{
height:300px;
overflow-y:scroll;
}


adding addLibertyMvnPlugin recipe

Recipe org.openrewrite.maven.liberty.AddOpenLibertyPluginDependency
What's changed?
Simple recipe to update pom.xml to add OpenLibertyPluginDependency as part of tWAS to Liberty General rule
Checklist

[X] I've added unit tests to cover both positive and negative cases
[X] I've read and applied the recipe conventions and best practices
[X] I've used the IntelliJ IDEA auto-formatter on affected files

hi @ranuradh ; I've done a quick polish and added a test to show that the plugin would now be added to all projects in a multi module Maven build; is that ok? If you'd rather see the plugin only added once then you can add a filePattern argument, or even a org.openrewrite.maven.search.FindPlugin precondition that checks that the plugin isn't already present.
I'll leave it up to you to decide when this is ready for a merge. Thanks again for the contribution here!

@timtebeek to me this looks ready will recheck..qq doesnt org.openrewrite.maven.AddPlugin already take care of checking if the plugin is present or not so we don't need to call org.openrewrite.maven.search.FindPlugin separately right?

Great suggestion; we've picked this up in:

https://github.com/openrewrite/rewrite/pull/4583

Feel free to share your thoughts and comments on that PR as well.


+Twitter feeds

Added Bruce and Kolton to Twitter feeds of Chaos Engineers

Great! Thanks for the contribution!


Implement getpeercert

NOTE: Please use stackoverflow for support questions.
This repository's issues are reserved for feature requests and bug reports.
I am submitting a …

[ ] bug report
[x] feature request

Description
I need to obtain the server's certificate after a successful handshake. In PEP there is a function documented getpeercert but I see in the code this is not implemented (commented out).
Would it be possible to obtain the server's certificate in some other way? If not, I'd like to ask you to add this feature.
Current behavior
getpeercert is not implemented
Expected behavior
getpeercert to be implemented
Steps to reproduce
N/A
Minimal demo of the problem
N/A
Other information
I don't need it in parsed form, binary DER is more than enough.

Hi! That seems reasonable, I will see what I can do.

The standard SSLSocket.getpeercert() returns a dict for binary_form=False but I have wrapped the x509 part of mbedTLS so I would rather return an mbedtls.x509.CRT instance. They can be converted to DER with bytes() and PEM with str().

I have got it working by
def getpeercert(self, binary_form=False):
        crt = _tls.mbedtls_ssl_get_peer_cert(&self._ctx)
        if binary_form is False:
            raise Exception('Not supported!')
        return crt.raw.p[0:crt.raw.len]

This is what is expected by PEP when binary_form=True. I'm just not sure what to do with the pointer. Should it be freed?

sorry I haven't noticed you already pushed, it's okay for me

sure, no problem.

I'm just not sure what to do with the pointer. Should it be freed?

I don't think so. I think it is a view on the peer certificate. At least, that is how I interpret the documentation and the fact that it is returned as a const *. So copying and returning looks good to me.


Thread-local lazy static

lazy_static and std's thread_local are very similar concepts, but lazy_statics interface (with Deref) is much nicer. I think it should be possible to implement a thread-local version of lazy_static.

Indeed, some consistency between the two would be nice. I've always thought of that more as changing lazy_statics API to be closer to thread_local, but providing a wrapper for it with a nicer API seems like a interesting idea as well.

I've made a crate that combined lazy_static, thread_local and RefCell.
https://docs.rs/ref_thread_local/0.0.0/ref_thread_local/


Edge attrs

I've added attributes for edges. I used https://www.graphviz.org/doc/info/attrs.html as a reference for which attributes applied to edges and which applied to nodes. To support different sets of attributes for edges and nodes, I created a module for edge attributes and node attributes. While this is a bit duplicative and required making several new modules, it seemed preferable to something like https://github.com/natefaubion/purescript-variant.

That is really interesting. I had stashed some basic work on attributes.
It looks like your works is more refined. I'll check it out!


Data structure

❓ Questions & Help
Hello
I do have a questions about the structure of data in the dataset class.
I'm creating my own dataset and so far i wanted to check if i'm doing things correctly, or if i'm on the wrong track.
I have 8 features per node. So far i have encoded them and created a tensor per node in the shape of.
torch.Size([233, 8])
Now if i understand the data structure correct, do i just have to stack them, to get a tensor of the shape:
torch.Size([number_of_nodes, 233, 8])
or is this an incorrect format?

Where does the 233 come from? IMO, the correct data structure would be [number_of_nodes, 8].

The 233 is due to the dimensionality of my data.
So one of my features, for example, is a text embedding, that is [100,1] and then i have another feature, that is categorical and has 233 categories. So i padded every other feature to this size and ended at [233, 8]

Ok, so the 8 features actually encode multi-dimensional features. In this case you should concat them to one multi-dimensional feature vector.

ok, thank you very much. Should i still pad them to the same length, or is this not necessary in this case?

This is not needed.


Track number of conections opened

Is any way to track how many conections of the pool is being used at the same time?

@GitDov With either JMX or Dropwizard.


Feature/add node scripts

I've added the renaming script in js.
https://github.com/infinum/wp-boilerplate/pull/76/files#diff-efac429a75f004416d69b7bbe9b6a72c
I'd like if the JS part is looked over, and if anything can be improved, suggest away :)
The rename of the theme is also made (which is basically a separate PR, but if all is ok, then this can either be dismissed once PR is merged, or the other PR can be dismissed).

Things to add:

[ ] Better color coding (test on white terminal just in case) - separate beginning statement (Theme name:), and end statement (My theme name)
[ ] Progress bar instead of the list of renamed files
[ ] Options to debug the renaming script in the event something isn't working (show full output or something alike)
[ ] Ability to already rename the theme once it's already renamed. This could be tricky because you'll need to know all the specific strings you've replaced them in the first place. Maybe all the initial replacement details could be written in some .json file so that it can be pulled from there for the future replacement


Stack and Tile (Enhancement)

Stack and Tile:
When using the default or Be Decorator the Tab will change color when Stacking or Tiling windows together. This lets the use know that the two windows are close enought to each other. With the Flat Decorator this doesn’t happen, that Tab remains the same color making Stack and Tile difficult to use.
Thanks
Donald

Added! try with the new binaries in repository if works ok!

Nice!!  :)
Please don't be too mad.  But, I did see a couple problems.  Can't help it.  I'm a software tester all day, everyday.
The gadgets in the tab itself stay a grey color while the rest of the tab turns yellow.
Also, and this is less of a deal, but just an observations.  Take a look at the color the tabs turn on the Be and Default decorator.  If possible I think it should stay consistent with the other decorators.  Currently the tabs turn the same shade of yellow as they would if they were active windows.  It took me a moment to realize it was actually working.
I attempted a video screen capture.  The quality of the recording is rather lacking.  But, I hope it is able to convey what I'm trying to say.
I am now able to use the Flat Decorator with Stack and Tile.  I'm very happy.
I am not able to attach the video file to this thread.  Hopefully the link will work.
https://drive.google.com/file/d/1qzfb7JVvlHioyyC-uyugpXJR6DF7UiGW/view?usp=sharing
Thanks


Improvement in Exercise Cards in Exerciser Activity

To open an activity we have to click on the play button itself. Intuitively, the exercise should open on clicking anywhere on the card.
https://user-images.githubusercontent.com/96700361/158371350-a0a38372-6885-45db-8396-85eae9f5870d.mp4
Also adding hover effect on particular card would help the user.
https://user-images.githubusercontent.com/96700361/158371369-673ab9eb-b56a-42b8-8a22-70a94c76f8c3.mp4
@llaske PTAL!
I would like to work on this.

Also adding hover effect on particular card would help the user.

@gitkaarma I feel like I have seen this issue before, is it that you have posted it before and reposted now or has it been repeated by mistake. can you please check previous issues to see if its already addressed or fresh one.

Yep! I posted a similar issue on exerciser git repo, but I got no reply there.

Yep! I posted a similar issue on exerciser git repo, but I got no reply there.

Oh that's why i felt like I have seen it somewhere.
Lets see what @llaske sir says about the feature.


Train / Validation / Test splits for million song dataset

Hi!
Thank you for releasing this repo :)
I was wondering where I can find the train/test/validation splits you used for MSD? My team and I are trying to reproduce this study but, unfortunately, we can't find the 201 680 / 11 774 / 28 435 splits and the corresponding tags from Last.FM. Would be very helpful for any assistance on this!
Kind regards,
Cody

Can add to this that we have been able to access the audio data itself, and it's really just used splits we're looking for. Curiously, the total number doesn't add up to one million so I guess some filtering/concatenation has been done as well.

@yiyiyi0817 don't know about those files specifically (@codyhesse and @SebastianLoef might know more), but we belive the splits used in CLMR were the ones by @keunwoochoi over in https://github.com/keunwoochoi/MSD_split_for_tagging at least.

Thank you very much. @carlthome

@yiyiyi0817 don't know about those files specifically (@codyhesse and @SebastianLoef might know more), but we believe the splits used in CLMR were the ones by @keunwoochoi over in https://github.com/keunwoochoi/MSD_split_for_tagging at least.

Where can I get the npy files?

@lix4 it's on the mentioned repo - https://github.com/keunwoochoi/MSD_split_for_tagging

@lix4 it's on the mentioned repo - https://github.com/keunwoochoi/MSD_split_for_tagging

I mean the original data files like "3/6/36122424.npy". Is there a place I can download it?

you meant the audio files. sorry you should ask around people who might have them as the crawling API doesn't work anymore. it's very problematic that i even wrote a short paper about it.
https://arxiv.org/abs/2308.16389

you meant the audio files. sorry you should ask around people who might have them as the crawling API doesn't work anymore. it's very problematic that i even wrote a short paper about it. https://arxiv.org/abs/2308.16389

All right, thank you for your explainition. I have searched them for a while.


How to implement tooltip in functional component?

How can I implement tooltip in a functional component. The demo only shows class based components.

For now you would use the HOC and class component. We're focused on TypeScript migration at the moment, hooks are coming after.

Hi,
I think I tried the above using another example from an early issue elsewhere in the repo and it turns out that as of this release (0.0.181) the tooltip stops showing on the Bar shape for some reason. I am still a newbie and not savvy enough to know why, but I tested each version of dependency until tooltip stopped working and it is this version.
Here is the CodeSandbox with a working tooltip with @vx/shape at 0.0.179. (also it generates a console error, but I don't know if that is my fault)
https://codesandbox.io/s/vx-responsive-tooltip-version-check-8s6kl
Hope this is helpful.

The demo here shows how to use withTooltip() and a functional component: https://vx-demo.now.sh/areas
and there is an example for useTooltip() here: https://codesandbox.io/s/github/hshoff/vx/tree/master/packages/vx-demo/src/sandboxes/vx-tooltip?file=/Example.tsx


mobile Safari: Touch scroll on elements page will cause the element to be selected

The following fixes should address this issue.
https://github.com/Polymer/polymer/commit/bdf191b80b7abd812bcd93ab2c211ec866182144
https://github.com/PolymerElements/iron-selector/pull/23

Just need polymer tagged to fix this...

Fixed with 1.0.1


Angular App - error plugin not installed

I am not able to see permission popup coming for camera. getting this error plugin not installed. though plugin is very much installed.
Its an angular 6 app and I am using ng-webcam camera module. Need to apply android permissions but it is not working. I tried with diagnostic plugin also, same result.
this warning msg keeps coming. and status error is saying plugin not installed.
deviceready did not fire within 5000ms. This can happen when plugins are in an inconsistent state. Try removing plugins from plugins/ and reinstalling them
any inputs, pls?

It means your app bootstrap bundle is too huge. That will need more time to load your SPA. Trying to reduce your app size might solve your issue.


fix IE complete/currentSrc bug

The code style issues do not belong to my changes and are fixed in my other PR. (https://github.com/scottjehl/picturefill/pull/510)

@aFarkas Looks like this is failing due to some code styling issues. If they've been fixed already, maybe merge in the updates. If not, mind fixing those to make sure everything else passes?

I simply merged the hole thing. should be green now.

+1


Minor enhancements to the AnimationPlayer's "add new track" dialog

Make the dialog adjust its size when the bézier toggle is hidden.
Minor rewrite of the message. No need to scream that the track is NEW.

@YeldhamDev Do you think its possible to create a default track which has a keyframe that represents all keyframes bellow like blender does in the animation track, because sometimes the user needs to rearrange a lot of frames at the same time, and if the AP has more than 10 tracks for example, its very hard to select each one:

As you can see in the image above, if the user moves the Armature track layer, all the keyframes below it moves along.

@nonunknown You may want to open a proposal in the proposals repo. for that.

Moved the changes to this PR: #38088, as this is too small to be standalone.


fix an error occurred while expanding utf8 path

ExpandEnvironmentStringsA can't handle utf8 string correctly, even I have change code page to utf8.
test code
setlocale(LC_ALL, ".65001");
std::string s1 = u8"对";  // e5 af b9
char buffer[255];
ExpandEnvironmentStringsA(s1.c_str(), buffer, 255);

// buffer : e5 af 00 00 .........

Which source code file encoding are you using(MBCS or UNICODE)?
Probably we may need to use ExpandEnvironmentStringsA or ExpandEnvironmentStringsW depending on source file encoding(using UNICODE ifdef)

Which source code file encoding are you using(MBCS or UNICODE)?
Probably we may need to use ExpandEnvironmentStringsA or ExpandEnvironmentStringsW depending on source file encoding(using UNICODE ifdef)

I use UNICODE and I think MBCS has the same problem.
WIN32 API has define the macro ExpandEnvironmentStrings  depend on UNICODE.  I think it would be better if we assume all strings(path/uri in gltf) are utf8 and then convert string to wstring and call the _W API.

I have tested the code with VS2019(Use utf-8 as an file encoding and UNICODE as a character set for MSVC compiler) and ExpandEnvironmentStringsA returns   e5 af b9.
I have also tested the code with some modification

Use S-JIS encoding(common multibyte character encoding in Japanese) since I don't know what is common encoding for Chinese multibyte characters
Use Japanese character "対"
Both use Unicode and MBCS when compiling

ExpandEnvironmentStringsA still works well.

@cwbhhjl Could you please write an unit test to reproduce this issue? Here is an example unit code(in tinyexr repo) : https://github.com/syoyo/tinyexr/blob/master/test/unit/tester.cc#L82

@syoyo I did some tests and found strange results.
https://rextester.com/RIJZM69582  online code, the result is correct.
Then I compiled an exe with the same code.
On my computer(Win10 Pro - simplified chinese) : wrong

On hyperV Win10 Pro - English: correct

Then I tested it on several different computers (all of them are win10 simplified chinese), all results were wrong, the last byte was missing.

@cwbhhjl Could you try setting the encoding of console to UTF-8(Using cmd.exe may be easier to try it on than PowerShell) in Simplified Chinese Win 10 environment and how it goes?
https://superuser.com/questions/269818/change-default-code-page-of-windows-console-to-utf-8
Probably system locale affects the locale and/or Win32 API behaviour of .exe binary.

@syoyo I found chcp 65001 does not affect the result.
I probably tested the options that affect the result.  open Control Pannel -> time & region -> region -> administrative -> language for non unicode program -> change system local...
If I set to English or Japanese, the result of the program will become correct.
If I set to Chinese(traditional or simplified), the result of the program will become wrong.
Is it convenient for you to try this test?

In addition, if I set to chinese and select the utf8 beta option, the result will also be correct.

@cwbhhjl Thanks! I can reproduce the issue with Chinese codepage(936). The binary is compiled with MBCS charset encoding.

So, in any case, we'll better to use wchar and W suffix API  for string where UTF-8 input is expected.
Basically, your patch would work both for MBCS and UNICODE binary on Chinese and UTF-8 locale encodings, but let me give some time to verify it.

I have added a unit test and verified the PR works well on  Windows 10 + codepage 936(tested both MSVC MBCS and UNICODE build) etc...
https://github.com/syoyo/tinygltf/blob/master/tests/tester.cc#L416

@syoyo Thanks! Maybe I should also report this api issue to Microsoft.

@cwbhhjl I think it won't be an API issue.


Fix #18 Add disabled prop

I added the disabled prop to the element and added the propType and defaultProp for it. I also updated the Jest snapshot so the tests run correctly. I did not add any new tests.

I apologize for not getting to the requested changes before now! Thanks for doing it and for merging my PR.

@thedeany no problem what so ever. Thank you for submitting the addition and improving the component.


Fix GPU sort for large arrays

Proposed changes
Fixes #1254 - GPU sort should now support arbitrarily large arrays (as long as their shape is contained in int)
Added relevant test
Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING document
[x] I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have updated the necessary documentation (if needed)

Thanks for the quick fix 🚀

That looks like an oversight. Will fix for the next release.

(Fixed in #1289 )


feat: allow signing git commits from scaffolder

Hey, I just made a Pull Request!
adds support to sign git commits with configured PGP key in scaffolder actions. configuration can be done either by integration or by using the default signing key in the scaffolder config. note that this pgp-plugin is used for signing and that it is limited to using RSA keys and signatures made with SHA1 hashing algorithm.
closes #25934
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@drodil Code-wise looks good. There are some conflicts atm.
Did you test all providers that it really works after the change? 😅

Thanks, will do a rebase. Actually, I could only test GitHub and Bitbucket that those work so probably have to get some more eyes and testers 🤔

Could the following people take a look  at this if it looks ok as you are one of the latest contributors to these modules:

@lukasfruntke-extern
@sanderaernouts
@tcardonne
@kenzliang
@cmoulliard

Sorry if I pinged the wrong people!

@tcardonne thanks for verifying!

@drodil some conflicts here too

@sanderaernouts thanks for the comment. I learned that this actually does not work with Github apps either. So closing this for now and will see if this can be done some other way. For gerrit/gittea/bitbucket the functionality would work so those could be done in a separate PR.


Error while processing rearrange-reduction pattern "(i j) c f h w -> c f (i h) (j w)".  Input tensor shape: torch.Size([3, 16, 256, 256]). Additional info: {'i': 1}.

same problem

same problem

Is this issue solved? I encounter the same problem. If we downgrade torch version from 2.0 to 1.13 <, xformers are not compatible with it :(

Is this issue solved? I encounter the same problem. If we downgrade torch version from 2.0 to 1.13 <, xformers are not compatible with it :(

I solved it by following #15 to downgrade torch and rebuild the environment. xformers will conflict with the downgraded torch but anyway the code can still process and output the gif.

In my case, the model could output a gif but it is just blacked out like this:

Is it OK in your case? @mumukawayi

In my case, the model could output a gif but it is just blacked out like this:  Is it OK in your case? @mumukawayi

In my case, the model can successfully produce the moving scene of the moonlight.

I finally find a way,
def _i(tensor, t, x):
    r"""Index tensor using t and format the output according to x.
    """
    shape = (x.size(0), ) + (1, ) * (x.ndim - 1)
    return tensor.to(x)[t].view(shape).to(x)

This works for me.
But I still wonder why the below code does not work:
def _i(tensor, t, x):
r"""Index tensor using t and format the output according to x.
"""
shape = (x.size(0), ) + (1, ) * (x.ndim - 1)
tensor = tensor.to(t)
return tensor[t].view(shape).to(x)

After I made modifications to the code, the problem was resolved
def video_tensor_to_gif(tensor, path, duration = 120, loop = 0, optimize = True):     tensor = tensor.permute(1,2,3,0)     images = tensor.unbind(dim = 0)     images = [(image.numpy()*255).astype('uint8') for image in images]     # imageio.mimwrite(path, images, fps=8)     # return images     imageio.mimsave(path, images, 'GIF', duration = duration)     return path

After I made modifications to the code, the problem was resolved
def video_tensor_to_gif(tensor, path, duration = 120, loop = 0, optimize = True):     tensor = tensor.permute(1,2,3,0)     images = tensor.unbind(dim = 0)     images = [(image.numpy()*255).astype('uint8') for image in images]     # imageio.mimwrite(path, images, fps=8)     # return images     imageio.mimsave(path, images, 'GIF', duration = duration)     return path

After I made modifications to the code, the problem was resolved

def video_tensor_to_gif(tensor, path, duration = 120, loop = 0, optimize = True):
tensor = tensor.permute(1,2,3,0)
images = tensor.unbind(dim = 0)
images = [(image.numpy()*255).astype('uint8') for image in images]
imageio.mimwrite(path, images, fps=8)
return images
imageio.mimsave(path, images, 'GIF', duration = duration)
return path

After I made modifications to the code, the problem was resolved
def video_tensor_to_gif(tensor, path, duration = 120, loop = 0, optimize = True):
    tensor = tensor.permute(1,2,3,0)
    images = tensor.unbind(dim = 0)
    images = [(image.numpy()*255).astype('uint8') for image in images]
    # imageio.mimwrite(path, images, fps=8)
    # return images
    imageio.mimsave(path, images, 'GIF', duration = duration)
    return path

This works for me.

I finally find a way,
def _i(tensor, t, x):
    r"""Index tensor using t and format the output according to x.
    """
    shape = (x.size(0), ) + (1, ) * (x.ndim - 1)
    return tensor.to(x)[t].view(shape).to(x)

This works for me. But I still wonder why the below code does not work:
def _i(tensor, t, x):
    r"""Index tensor using t and format the output according to x.
    """
    shape = (x.size(0), ) + (1, ) * (x.ndim - 1)
    tensor = tensor.to(t)
    return tensor[t].view(shape).to(x)

I met the same error, would you mind tell me where you change this function?


Consider adding ckeditor module

(internal planio 12118)
We should consider adding https://www.drupal.org/project/ckeditor_accordion module to Openfed but we'll need to add the patch https://www.drupal.org/project/ckeditor_accordion/issues/3124167 to make it accessible.
If we add this module, we should consider adding https://github.com/smillart/WAI-ARIA-Patterns-And-Widgets which is already part of openfed-libraries
TODO:

make sure the patch is retro-compatible with existing accordions.
make sure that if we remove the patch the accordion will still work.
make sure the module is fully D9 compatible (maybe not 100% yet)

I added the module and patch.
After some tests, I can confirm that:

Patch is retro-compatible, as it allows to alter the output of the original module on render time, so the initial structure is always the same in the database
If patch is removed, then the original behavior occurs


stream: one batch per batch consume routine

Currently, there is only one batch for the whole batch consumer entity, which prevents usage of multiple runners to process data efficiently.
Please introduce a batch for each runner.

Closing issue, as auto-scaling will be increasingly more complex to realize with multiple consumers per batch entity


Drop Kyuubi extension for Spark 3.1

:mag: Description
Issue References 🔗

This pull request is the next step of deprecating and removing support of Spark 3.1
VOTE: https://lists.apache.org/thread/670fx1qx7rm0vpvk8k8094q2d0fthw5b
VOTE RESULT: https://lists.apache.org/thread/0zdxg5zjnc1wpxmw9mgtsxp1ywqt6qvb
Describe Your Solution 🔧
Drop module kyuubi-extension-spark-3-1 and delete Spark 3.1 specific codes.
Types of changes :bookmark:

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

Test Plan 🧪
Pass GA.

Checklist 📝

[x] This patch was not authored or co-authored using Generative Tooling

Be nice. Be informative.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 61.11%.  Consider uploading reports for the commit 021532c to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #6125      +/-   ##
============================================
- Coverage     61.15%   61.11%   -0.05%
  Complexity       23       23
============================================
  Files           624      624
  Lines         37247    37247
  Branches       5042     5042
============================================
- Hits          22780    22763      -17
- Misses        12014    12018       +4
- Partials       2453     2466      +13

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


[perf] Add pipeline template and storage pipelines

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

/check-enforcer overrride

/check-enforcer reset

/check-enforcer evaluate


Expand template build infrastructure for Perl components

Simply add generic configuration to spawn additional Perl subprojects off from. Using a PR because I can't commit directly to main.

Removing skip ci because the tests need to run before I can merge...

Based on the failure, this was a bad idea. I don't want Makefile, dist.ini or cpanfile to be overwritten.


crashes on startup

crashes on boot
ERR) TypeError: Cannot read property 'clientId' of undefined
W20150207-14:18:44.129(-8)? (STDERR)     at Accounts.onCreateUser.user.emails (app/server/accounts.js:5:46)
W20150207-14:18:44.129(-8)? (STDERR)     at /Users/dc/dev/ref/meteorhunt/.meteor/local/build/programs/server/boot.js:182:5

Getting the same error here :(

the same for me.
regards
@jeangui

you should check out the demo of all widgets app
https://github.com/meteoric/demo
this seems to be better maintained.

Actually that one is the most outdated- I recently updated both meteorhunt and contacts

On Feb 27, 2015, at 8:35 AM, dc<EMAIL_ADDRESS>wrote:
you should check out the demo of all widgets app
https://github.com/meteoric/demo
this seems to be better maintained.
—
Reply to this email directly or view it on GitHub.

i noticed similar error unless i use meteor --settings settings.json

You are right meteor --settings settings.json works fine. :+1:
Now when I want to run thé application into thé IOS Simulator by running thé following command:
meteor run ios --settings settings.json .It does not work. The same error again. How do you include the settings in thé ios build ?
Have you an idea of what command line I should use instead ?
thanks.

i think there's a separate settings for cordova. check meteor docs

meteor run ios --settings settings.json worked for me.
You've edited settings.json right with details created at https://www.meteor.com/account-settings/apps right?
Note: I've patched to use Meteor 1.0.3.

Hi
Thanks for your help.
I did create an app in the meteor developper console.
And then i filled the settings.json with the correct info coming from the console.
Still not working.
Regards.
jeangui

running
meteor --settings settings.json
worked for me.


Possibility of generating intermediate values between data points?

Hi there, thank you for this library.
Please could you tell me if what I am trying to do is possible, and if so how I might accomplish it with this library?
I'm following the example here
I have 3 steps of hourly data. I've setup my code so that it follows pretty much how it is in that notebook but when I generating the forecast I'm running:
forecast_anvil = anvil.forecast(
    rainrate_field[-3:], velocity, 3, ar_window_radius=25, ar_order=2
)

My assumption is that this is passing in the 3 hourly steps of data which I have, and it's generating 3 more steps, predicting the motion/intensity of the rainfall at hours 4, 5 and 6. Is that correct?
I am currently able to display this data and as far as I can see it appears to be working.
What I would like to do is "train" the data using a sample size of say 3 hours. Then generate intermediate steps of say 5 minutes between hours 3 and 4 using the forecast.
Then repeat this using a rolling window now of hours 2,3,4 to train the data to give it an idea of the motion, and then generate the 5min increments between hours 4 and 5, etc etc.
Is this possible? If it is would it be possible to guide me through how to do it?
It seems to be possible as far as I can see when using something like:
motion = dense_lucaskanade(precip)

timesteps = [0, 0.1, 0.2, 0.5, 0.6, 6, 7]
thr = 0.5
slope = 1  # pixels / timestep

out = forecast(precip[0], motion, timesteps, thr, slope=slope)
for n, frame in enumerate(out):

    plt.subplot(2, 4, n + 1)
    frame[np.isnan(frame)] = 0
    plt.imshow(frame, interpolation="bilinear", vmin=0, vmax=1)
    plt.title(f"t={timesteps[n]}")

whereby it seems to support timesteps below 1, but that's just giving me probabilities I think. But the output does seem to make sense for the steps between 0 and 1.
Thank you in advance for any help, I'm far from an expert in this area so I hope I am making sense!

hello @atom5dev

My assumption is that this is passing in the 3 hourly steps of data which I have, and it's generating 3 more steps, predicting the motion/intensity of the rainfall at hours 4, 5 and 6. Is that correct?

Correct, the definition of a time step is given by the velocity field: was this estimated based on an hourly sequence of rainfall fields? If yes, then the the nowcasting method will predict with the same hourly resolution.

What I would like to do is "train" the data using a sample size of say 3 hours. Then generate intermediate steps of say 5 minutes between hours 3 and 4 using the forecast.
Is this possible? If it is would it be possible to guide me through how to do it?

Yes, it is possible and you are doing it right. All nowcasting methods can accept a list of floats as timestamps. You can pass a list of timestamps where the values are a fraction of the original hourly timestamp corresponding to 5 minutes (that is, 1/12), so for between hours 3-4: [3, 3 + 1/12, 3 + 1/6, 3 + 1/3, ..., 4].

whereby it seems to support timesteps below 1, but that's just giving me probabilities I think. But the output does seem to make sense for the steps between 0 and 1.

I think it's returning probabilities because you've been using the lagrangian_probability method, which is designed to predict exceedence probabilities. But if instead you use the plain extrapolation method you'll get the out the same units used for the input field. Remember to scale the units accordingly if you've been using something else than rainrate intensities for input.
I hope this helps. I'll close the issue. For usage questions, we'd recommend to use our slack channel. You can find invite link in the README.

Thank you for taking the time to reply @dnerini , very much appreciated.


Non-nullable parameter best practices

I'm writing my first binary PowerShell module and have C# 8 with non-nullable reference type checking enabled. I have a cmdlet with a string parameter called Name. The parameter has the ValidateNotNull attribute for PS to check if Name is null and error out. The compiler outputs a warning about the uninitialized string.
What are the best practices for non-nullable parameters where PS does a null check?
Steps to reproduce
[Cmdlet(VerbsCommon.Find, "Thing")]
public sealed class FindThing : PSCmdlet
{

    [Parameter()]
    [ValidateNotNull]
    // PS does a null check and errors.
    // Compiler produces this warning:
    // Non-nullable property 'Name' is uninitialized. Consider declaring the property as nullable.
    public string Name { get; set; }

    protected override void ProcessRecord()
    {
        // do thing
    }
}

In that case you can do one of two things:

Set a default value of string.Empty just to satisfy the compiler, or
"Assure" the compiler that it should never be null, effectively. I believe the pattern looks something like this:

public string Name { get; set; } = null!;

Thanks @vexx32. I was trying to use the null-forgiving operator but put it in the wrong spot.

I still put it in the wrong place all the time, it's very confusing! 😂


Windows Authentication for Azure SQL MI

Regarding

Yes - see Windows Authentication for Azure Active Directory principals (public preview).

Please clarify the preview/GA status.
According to this page it's in public preview.
However, the destination link has no mention of that, making the reader assume it's in GA.

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

ID: f7f71c75-4511-ed7d-6e3d-4c30e1c01781
Version Independent ID: 7be40f32-162d-1f6d-ff3c-163d0c4eb10d
Content: Compare the database engine features of SQL Database and SQL Managed Instance - Azure SQL Database & SQL Managed Instance
Content Source: azure-sql/database/features-comparison.md
Service: sql-db-mi
Sub-service: service-overview
GitHub Login: @danimir
Microsoft Alias: danil

Ok, answering my own question, feature went GA this August: https://techcommunity.microsoft.com/t5/azure-sql-blog/windows-authentication-for-azure-ad-principals-for-sql-managed/ba-p/3600690
So this page is out of date and needs to have "(public preview)" suffix removed.

@VitalyMCT Thanks for your findings, the document will be updated soon

@VitalyMCT  Document is updated and you will see the changes in the next 24hrs.


What database uses this project?

Hey i just met this project and i would like to test it by my self, what kind of data storage system is used in this project?
Thanks in advance.

Flat file, mainly in CSV and JSON formats

I would not like it to use redis and mongodb without detailed rationale. Flat files have plenty of advantages.
Also note that the login server uses MySQL.

Also, this is not an issue. Issues are planned features, bugs, or implemented changes in need of review.
We have several other forms of contact: http://pokemonshowdown.com/contact

If we switch to something that isn't flat file, it would make the transition to a proper multiprocess server easier

I'm opposed to Redis and MongoDB myself. There is no reason to use them, really, and MongoDB is broken by design. If you are going to use database, let it be SQL other than MySQL, but the issue is that @Zarel wants Showdown to just work on Windows without needing to use Visual C++ (needed for SQLite) or installing additional software.

SQLite doesn't really play that well with simultaneous access from multiple processes, which is the main reason for switching to a database in the first place

I actually don't care about Windows support too much, but I'm not a huge fan of external dependencies in general, especially without a good reason.
I think an in-memory database might be useful for our in-memory text data, though. Would be a replacement for node.js/io.js's lack of a resizable buffer class. :|


Plugins

Is your feature request related to a problem? Please describe.
Right now, there's a large number of modifications to .perseus/ (which all require ejecting) that could be modularized.
Describe the solution you'd like
Perseus should have a plugins system with three types.
Type 1 plugins should provide functions that can be provided with extra information through define_app!. These functions will the be executed at various places in Perseus' engine. For example, a plugin could be created with this system that would add new routes to the inbuilt server, adding a GraphQL API with almost no configuration and no separate systems.
Type 2 plugins would be more powerful, being able to modify files arbitrarily in addition to providing type 1 capabilities. An example use for this would be appending size optimizations to .perseus/Cargo.toml.
Type 3 plugins would be the most powerful, as they would have their own entirely customizable .perseus/ directories. Further, they'd issue instructions to the CLI that would set new commands to be run for different Perseus commands. For example, they might instruct perseus serve to also spin up extra servers and a database in parallel. These would be read from a specially named file in .perseus/, and this would be done by stages. Then, an extra stage of completely custom executions could be added, which would preserve the CLI's own loaders and progress reporting system.
Describe alternatives you've considered
Manual ejection in all cases, but this is very tedious, and doesn't lend itself to a highly customizable and collaborative open-source ecosystem.
Additional context
Types 1 and 2 should be ready for v0.3.0 when it goes stable (meaning more beta releases), but type 3 will be reserved until after that. However, CLI instructions would be very useful for ejections already, and that should be included.
This issue will track general progress for all aspects of the plugins system, which will improve Perseus dramatically by providing the customizability of v0.1.x with the convenience of v0.2.x and the power of x0.3.x.

[ ] Add type 1 plugins support
[ ] Add type 2 plugins support
[ ] Add support for custom CLI instructions
[ ] Add type 3 plugins support

Designing this, I can see a great deal of things that should be able to be modified by functions, but that aren't compatible with multiple plugins. For example, a plugin that modifies the call to sycamore::render_to would be type 1, but wouldn't be compatible with any other plugins, as it takes full control there.
So, there should be some functions that should be restricted to type 2 plugins.

We also need better names than types, because they will not be remembered well. Here's a proposal:

Functional plugins (always compatible with each other except in niche cases)
Control plugins (one plugin takes control of one or more critical functions, replacing the default behavior altogether)
Engines (custom .perseus/ directories)

This naming clearly separates the idea of plugins from custom engines, which it doesn't really make sense to even call 'plugins', they don't plug in, they replace the socket.


From Chrome 115.x, Google has changed ChromeDriver download Url

Google has recently changed download Url for ChromeDrivers.
WebEngine Framework need to be adapted for this change.
Reference: https://github.com/GoogleChromeLabs/chrome-for-testing#json-api-endpoints
Changes:

Url of "Latest driver version" has been changed
Url of "chromedriver" has been changed
Driver are now placed in a folder inside the zip.

Implemented with #40


KAFKA-10163; Throttle Create Topic, Create Partition and Delete Topic Operations (KIP-599, Part I, Broker Changes)

This PR implements the broker side changes of KIP-599, except the changes of the Rate implementation which will be addressed separately.
The PR changes/introduces the following:

It introduces the protocol changes.
It introduces a new quota manager ControllerMutationQuotaManager which is another specialization of the ClientQuotaManager.
It enforces the quota in the KafkaApis and in the AdminManager. This part handles new and old clients as described in the KIP.

The PR is a good shape. The tests coverage is good. I will add few more unit tests for the api layer later one.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@junrao @rajinisivaram @apovzner Whenever you have time, I would appreciate if you could check this one out. Thanks.

@rajinisivaram Thanks for your review. I have addressed all your comments module the two regarding the naming of the type and the classes which are still under discussions.

ok to test

@rajinisivaram I just pushed an update which addresses your comments.

retest this please

retest this please

retest this please

retest this please

retest this please

ok to test

ok to test

retest this please

ok to test

Merged to trunk, test failures were unrelated.


Fetch-Transform-InsertArbeitnow

Three functions need to be made
1.Fetch Data
2.Transform Data
3.Insert Data in DB

related to pr #140 #139 closed with pull request #153


Why I can't use decorator on property

QUESTIONS: When I use decorator which is defined by myself on property and typescript throw an error:Unable to resolve signature of property decorator when called as an expression.
Typescript version:2.5.3 Vue:2.5.2
Code
@AutoWired
private userService: IUserService;
const AutoWired = (target, name, descriptor) => {
    import(ioc[name]).then((service) => {
        descriptor.value = new service();
        return descriptor;
    });
   return descriptor;
};

Expected behavior:
no error
Actual behavior:
Unable to resolve signature of property decorator when called as an expression.

That is because the decorator's third parameter, the property descriptor, is only available when targeting members on the class prototype like methods or properties defined in terms of accessors . A decorator for a regular property only takes a target and a key.
Note you can create a decorator that works for both the 2 and 3 argument scenarios by making the third parameter optional.
I take it you are trying to set up a property that will eventually be given the value of the asynchronously resolved service but that will start out undefined. If so, you want something like
export const AutoWired = <T>(target: T, key: keyof T) => {
  let service: UserService;

  Object.defineProperty(target, key, {
    get: () => service,
    set: value => service = value,
    enumerable: true
  });

  import(ioc[key]).then(Service => {
    service = new Service();
  });
};

export class A {
  @AutoWired userService: UserService;
}

thank you


Mean Squared Error (MSE) Metric Document

Linked issue(s)
Requires #549 to be merged before. (Will need to rebase/add links)
What change does this PR introduce and why?
Please check if the PR fulfills these requirements

[ ] Include reference to internal ticket and/or GitHub issue "Fixes #NNNN" (if applicable)
[ ] Relevant tests for the changes have been added
[ ] Relevant docs have been added / updated

As it was discussed in this PR thread, let's move this doc to "Common Metrics".

As it was discussed in this PR thread, let's move this doc to "Common Metrics".

would this be where "Accuracy", etc. and others are currently located?

Correct.  It will be added to this section of the doc.


Implement aperio svs workloads (wip)

Here's a work in progress Aperio SVS reading workload.
One workload reads the file via tifffile's ZarrTiffStore the other workload creates a kerchunk json for the file and reads array data via FSStore and fsspec.ReferenceFileSystem.

Looks good to me - thanks! LMK when you're happy for me to merge this.

I'll change the Dataset subclass to generate the input tiff instead of downloading one.
Will mark this PR as ready when I added the changes.


SSLV3 Bad_certificate _ssl.c: 1091 Issue while using Locust FastHttpUser

I am setting Cert , Key and CA Cert and passing as **kwargs in my get request. I am running test on AWS.

Sorry, thats not enough info. Try googling the problem, or ask on stackoverflow. Or switch to HttpUser.


Don't add limit to params array if no valid limit is set.

When an invalid limit is set with .limit(<0), then when the params
are built, a non-falsy (i.e., non-zero) limit is seen and is added
to the params array.
Added code to update the value based on the validity of the limit arg.
Added tests to cover this case.
Thanks for a great, clean query builder library. This is very helpful.
-david

Nice catch! Thank you so much :)


#looping-and-branching id seems to be missing

The "Using dialogs" paragraph links to "https://docs.microsoft.com/en-us/azure/bot-service/bot-builder-concept-dialog?view=azure-bot-service-4.0#looping-and-branching" which is not linked to a paragraph / bookmark on the page.

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

ID: 6d405d76-b17a-dbe9-c2f2-734fa34b0e8b
Version Independent ID: 3acc3fa8-b98d-a5f1-8bea-e823c3efec14
Content: Dialogs within the Bot Framework SDK - Bot Service
Content Source: articles/v4sdk/bot-builder-concept-dialog.md
Service: bot-service
GitHub Login: @johnataylor
Microsoft Alias: johtaylo

Fixed in branch, will be published soon. Thanks!


Online sensor outputs

As a user I want to be able to wirelessly read all sensors data from the vehicle so that I can monitor everything remotely

[ ] Readings and current states of all measuring instruments on the vehicle can be uploaded to a private MQTT broker over the internet.
[ ] The client app is able to receive all the data from a private MQTT broker
[ ] Implementation is tested
[ ] Function is successfully added

Was closed automatically by mistake.


Using OllamaEmbedding as retriever ?

Hi,
I was trying to workout a solution with OllamaEmbedding as a retriever. However got the following error :

File ~/miniconda3/envs/RAG/lib/python3.11/site-packages/dsp/primitives/search.py:12, in retrieve(query, k, **kwargs)
10 if not dsp.settings.rm:
11     raise AssertionError("No RM is loaded.")
---> 12 passages = dsp.settings.rm(query, k=k, **kwargs)
13 if not isinstance(passages, Iterable):
14     # it's not an iterable yet; make it one.
15     # TODO: we should unify the type signatures of dspy.Retriever
16     passages = [passages]

TypeError: 'OllamaEmbeddings' object is not callable
Will there be any support for OllamaEmbedding in the future ?
Thanks in advance

I am facing the same error.


Add localhost, port, daemonset, and localnode namers

After some in-person discussion we will probably not pursue this.  Doing this kind of thing as a transformer instead of as a namer is appealing because it allows the initial naming to happen in namerd but still allows post-processing to happen in linkerd.  This is useful in the case where you want to resolve an id in namerd and then filter that set of addresses down based on some contextual information available in linkerd.  An example of this is the localnode namer/transformer.  In this use case we want to be able to resolve the destination service id in namerd but then filter that down to only endpoints which are node-local to the linkerd.  This is better accomplished using a transformer rather than a namer since the entirety of naming takes place in namerd (if namerd is used at all).


Try to wrap the messages appropriately to look better on screen

The Issue
Too much wrapping on messages sent to the user.
How This PR Solves The Issue
Try to calm them down.

Let's skip this and wrap in the code then, good idea.


Why isn't CALICO_IP=auto the default?

When browsing the source, I noticed that using -e CALICO_IP=auto will cause calico-docker to auto-assign an IPv4 address. Is there any reason this isn't the default?
When integrating with orchestrators, it's not always reasonable to expect every orchestrator to always pass a CALICO_IP environment variable. If an orchestrator is going to the trouble of using the Powerstrip endpoint for Calico (as opposed to just using Docker directly), I'd think the reasonable default would be to auto-assign an available IP from the Calico pool.

Right now the Calico adapter for Powerstrip looks for that variable to know which container requests to add Calico to.  The idea is that if that variable isn't present, we wouldn't set up Calico networking (if, for example, you needed to run a container with "host" networking).
As you hinted at, people can always bypass Powerstrip and go directly to the Docker API for that, but that approach won't work if you're using more than one Powerstrip adapter.  For example, you might connect both Flocker and Calico via Powerstrip and want to use Flocker but not Calico for a container.
Anyway, that's the reason it's that way now.  If it's a pain to tweak your orchestrator to pass the environment variable, we could probably add support to configure the Powerstrip adapter to assume CALICO_IP=auto as the default.  Let me know!
One final thing to mention is that Powerstrip is itself a temporary solution for prototyping.  Docker are working on a networking API right now.  We don't know exactly what that will be like, but we're expecting it give us a way to configure extensible Docker networking in a much cleaner way.

@spikecurtis Thanks for the in-depth explanation! I'm looking into Calico as a possibility for running Kubernetes. For now I'll just always specify the CALICO_IP variable for each pod. Thanks again.


Delay before device.onDisconnected is fired

My simplified code is as follows:
this.manager
.connectToDevice(id)
.then(device => {
device.onDisconnected((error, disconnectedDevice) => {
console.log("Disconnected ", disconnectedDevice.name);
});
However there is approximately a 10s delay between turning of the physical device and the onDisconnected listener being called. Is there any possibility of decreasing the delay?

It's better to monitor disconnection before connectToDevice call as otherwise there might be a race condition. Actual disconnection event depends on Connection supervision timeout, which is configured on the firmware side. Nothing can be done about it with current API on iOS or Android.


The type org.apache.http.HttpResponse cannot be resolved. It is indirectly referenced from required .class files

The type org.apache.http.HttpResponse cannot be resolved. It is indirectly referenced from required .class files
每次用到这个包，第一行package那里 就会报这个错误

This is related to combination of used Android SDK version and library version, these headers are in apache http legacy which was abandoned and later removed from SDK.
Anyway this is too vague issue report, If you provide more debug info, feel free to re-open this ticket.


feat(seed-repo): add red "priority 0" label

Priority 0 marks an issue that must be fixed in the next product release. Ping @Joshua-Anderson @kmala @vdice.

We should add the label to the list here: https://deis.com/docs/workflow/contributing/triaging-issues/#prioritizing-issues

We should have further discussion about this. I think the term showstopper and p0 will be confusing over time. We should probably drop the term showstopper and assign levels of priority to all issues. This will help us further prioritize the backlog and make sure we are making progress on all levels of issues.

We talked about this a bit at the triage meeting. We decided that showstopper and p0 have different meanings. showstopper is specifically for release blocking issues, whereas p0 is just a top priority issue. I think more priority levels can be added, but the immediate need was some way to flag important bugs for investigate.

I would say that a p0 issue should be relabled showstopper and p1 issue would be things that we will address in the next release. This allows us to have a common naming scheme for all priority levels

I agree with @jchauncey. At this point a showstopper means "specifically for release blocking issues" and p0 means "an issue that must be fixed in the next product release", which means they both accomplish the same thing.
I realize I wasn't there for the triage meeting so my vote doesn't count, just thought I'd throw in my 2 cents about any future confusion towards these labels. We can just label everything p0 and showstopper though ;)

I think my fear was that we would label everything as a p0/showstopper which makes 1 of them kind of pointless.

@jchauncey @bacongobbler of course your vote counts!
After more discussion, I think we decided on this:

"priority 0" means it's a 0-day, drop everything, maybe make a patch release type of fix
"priority 1" means it's a serious bug that will be fixed for the next planned release
"showstopper" is deprecated and will be removed eventually

I'll update this PR to add those labels, and create an associated docs PR clarifying their usage. But please continue the conversation here if I've missed something or if you think we should proceed in a different direction.

Nope! That sounds exactly like what we talked about in the standup. Sounds good to me :)


Default SceneInstance and Scene added for code-only approach

PR Details
Default SceneInstance and empty Scene is created programmatically if the game is run without the default assets (MainScene.sdscene) created by Game Studio.
Description
Discussed here https://github.com/stride3d/stride/discussions/1253#discussioncomment-1979625
else branch added
else
{
    SceneInstance ??= new SceneInstance(Services) { RootScene = new Scene() };
}

Related Issue
More details in the issue below.
Resolves #1290

Motivation and Context
We are working on code-only approach and to simplify this approach so a game can be run without a too much boilerplate. Default scene can be always updated/replaced as needed through the code.
This will help to run a game with a very few lines of the code.
This change shouldn't affect any existing approach through the Game Studio Editor.
Types of changes

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

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Thanks !

My pleasure! 👍😀


Add a hook to catch invalid messages

Currently there are hooks for reading messages off the wire (DecorateReader), checking if they comply with policy (MsgAcceptFunc), and generating responses (Handler).  However, there is no hook that notifies the server when a message is dropped or rejected due to a syntax error.  That makes it hard to monitor these packets without repeating the parsing process.
This PR adds a hook for notifications about invalid packets.

excellent idea. Small comment and making the names more consistent


Service Changed - 0x2A05

Is there any documentation on how to setup the Service Changed characteristic? I've found this Stack Overflow post, but I haven't been able to make that work.
It would be nice to not have to turn off bluetooth and reboot my phone to clear the cache every time I flash any changes involving BLE characteristics to the micro controller

Could you share what modifications you've tried? If you use the macro API, you have to set the characteristic value after creating the handle. If you use the builder api, you can use add_characteristic_ro with the value (handle range)

So perhaps I don't understand the value I need to create,  but I can certainly show what doesn't work!
So I've got a gatt_server, which also sets up a generic service 0x1801, which should hold the service_changed characteristic.
// GATT Server definition
#[gatt_server]
pub struct Server {
    pub generic: Generic,
    pub fault_service: FaultIdentifierService,
    pub relay_service: RelayService,
}

#[gatt_service(uuid = "1801")]
pub(crate) struct Generic {
    #[characteristic(uuid = "2A05", indicate, read)]
    service_changed: i32,
}

and then I can try and set the value for the service changed.
        //Note: A server may set the affected Attribute Handle range to 0x0001 to 0xFFFF to indicate to the client to rediscover the entire set of Attribute Handles on the server.
        let handle_start = 0x0001;
        let handle_end = 0xFFFF;
        let full_value = handle_start + handle_end;
        let test = server.set(&server.generic.service_changed, &full_value);
        info!("We were able to set service changed? {test:?}");

But scanning with NRF Connect, I can see the 0x1801 service, but it does not contain the service_changed characteristic.

Reading some docs , I've also tried concatenating the handle start & end, as well as having the characteristic be indicate only. I have not noticed any change though
#[gatt_service(uuid = "1801")]
pub(crate) struct Generic {
    #[characteristic(uuid = "2A05", indicate)]
    service_changed: u32,
}

        let handle_start = 0x0001_u16;
        let handle_end = 0xFFFF_u16;
        let concatenated: u32 = ((handle_start as u32) << 16) | handle_end as u32;
        info!("Concatenated: {concatenated:?}");
        let test = server.set(&server.generic.service_changed, &concatenated);
        info!("We were able to set service changed? {test:?}");

So, I believe each handle is little endian, and it should be:
    let start = 0x0001;
    let end = 0xFFFF;

    let mut value: [u8; 4] = [0; 4];
    value[0..4].copy_from_slice(&start.to_le_bytes());
    value[4..8].copy_from_slice(&end.to_le_bytes());

I think there might be another issue as well, namely that the macros will add the 1801 generic service automatically, and you might end up with 2 of that same service. I think you'd need to use the builder api for now to work around that. We should ideally modify the macros to allow adding optional characteristics to it somehow.
CC @jamessizeland

Alright, I believe I have the world's simplest POC for clearing the service cache.
It still does not list the service changed characteristic in nRF Connect, but it rediscovers the services / characteristics on every connect, so that's good enough to keep the ball rolling. Thanks for the help @lulf !
diff --git a/host/src/gap.rs b/host/src/gap.rs
index a0f0720..5f666d8 100644
--- a/host/src/gap.rs
+++ b/host/src/gap.rs
@@ -93,7 +93,9 @@ impl<'a> PeripheralConfig<'a> {
         gap_builder.add_characteristic_ro(characteristic::APPEARANCE, self.appearance);
         gap_builder.build();

-        table.add_service(Service::new(service::GATT));
+        let mut gatt_builder = table.add_service(Service::new(service::GATT));
+
+        gatt_builder.add_characteristic_ro(characteristic::SERVICE_CHANGED, &[1, 0, 255, 255]);

         Ok(())
     }

This will require you to set a custom attribute_table_size
#[gatt_server(attribute_table_size = 50)]

or it will crash on first boot after flash with
trouble/host/src/attribute.rs:310:13: no space for more attributes
If I understood the macro syntax better (or at all), I could probably get that to include our added characteristic in the table size calculation during compile time, but that's a problem for NOT the world's simplest POC.


Add Anilist Search Plugin

I created a plugin that uses the flow launcher to search for anime in the anilist.
https://github.com/Misakiix/flow_launcher_anilistplugin
It's a simple plugin, and there's no need to use credentials

Hi there @Misakiix, you accidentally created a pr for merging our API branch into old main branch. I will close this, please resubmit.
You need to fork this repo then add your plugin details into the plugins.json file. See readme.


[$500] New Feature: Create the UI for Actionable Mention Whispers

We're looking for a contributor to create the UI for actionable mention whispers.
IMPORTANT: we will NOT be including the "Let them know" button

Flow

Make sure you have a DM already between userA and userB and userC. This way their mentions will get autofilled.
Create a workspace as userA.
Create a room on that workspace with any visibility as userA.
As userA, mention userB and userC in a message sent in that room. This should trigger an actionable mention whisper to be sent to userA (see mockup above).
userA can click "Invite them" or "Do nothing". In both these cases, the actionable mention whisper should disappear after a choice is selected.

If "Invite them" was clicked, both userB and userC should have been invited to the room.
If "Do nothing" was clicked, nothing happens.

Technical details

You'll need to pass true to the shouldAllowActionableMentionWhispers param here to get the actionable mention whispers to send and appear in Onyx.
When userA clicks "Invite them" or "Do nothing", we'll call the new API command ResolveActionableMentionWhisper. It takes these parameters:

reportActionID: the reportActionID of the message that mentions the users to invite
resolution: a string that is either "invited" or "nothing" (for each button)–create constants for these

Once ResolveActionableMentionWhisper has been called, the actionable mention whisper should have been updated with a new resolution field in Onyx, set to either invited or nothing depending on what was passed. If the resolution field exists in Onyx, make sure that the actionable mention whisper is does NOT render.
Make sure that the resolution field is set optimistically in the actionable mention whisper.

Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01c01bba106bc8268e
Upwork Job ID:<PHONE_NUMBER>476602368
Last Price Increase: 2023-12-08

👍 Will follow along as this progresses.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Create the UI for Actionable Mention Whispers
What is the root cause of that problem?
New Feature.
What changes do you think we should make in order to solve the problem?
We will have the Buttons render in reportActionItem after the content here:
https://github.com/Expensify/App/blob/adbeba13ef409abb11024a2bf602a6625d3249fb/src/pages/home/report/ReportActionItem.js#L556-L567
The buttons will appear on the condition that isWhisper is true and isMentionResolved false.
{isWhisper && !isMentionResolved && (
  <View style={[styles.flexRow, styles.gap4]}>
      <Button
          style={[styles.mt2]}
          onPress={() => Report.resolveMentionWhisper(props.action.reportActionID, CONST.WHISPER_ACTION.INVITED)}
          text={translate('reportAction.inviteThem')}
          small
          success
      />
      <Button
          style={[styles.mt2]}
          onPress={() => Report.resolveMentionWhisper(props.action.reportActionID, CONST.WHISPER_ACTION.NOTHING)}
          text={translate('reportAction.doNothing')}
          small
      />
  </View>
)}

The resolveMentionWhisper function will look like this:
function resolveMentionWhisper(reportActionID, resolutionStatus) {

    const failureData = [
        {
            onyxMethod: Onyx.METHOD.MERGE,
            key: `${ONYXKEYS.COLLECTION.REPORT_ACTIONS}${reportActionID}`,
            value: {
                resolution: null,
            },
        },
    ];

    const successData = [
        {
            onyxMethod: Onyx.METHOD.MERGE,
            key: `${ONYXKEYS.COLLECTION.REPORT_ACTIONS}${reportActionID}`,
            value: {
                resolution: resolutionStatus,
            },
        },
    ];

    const optimisticData = [
        {
            onyxMethod: Onyx.METHOD.MERGE,
            key: `${ONYXKEYS.COLLECTION.REPORT_ACTIONS}${reportActionID}`,
            value: {
                resolution: resolutionStatus,
            },
        },
    ];

    API.write('ResolveActionableMentionWhisper', {reportActionID, resolution}, {optimisticData, successData, failureData});
}

Not the final code, just an outline
Result:

@jasperhuangg Hey, since even after setting the param, the actionable mention whispers were not appearing, I took a screenshot of a sample whisper message. I have replied on Slack here with a screencast on how its not appearing. I apologize for the wrong screenshot :bow:

Hey @neonbhai sorry I just checked and the PR that adds that as a param actually hasn't been deployed to staging yet 😓
I'll get back to you once it's out, thanks for your patience. Regardless, could you update your proposal to specify that you'll be creating a new component for the action?

@neonbhai it should be on staging now!

@jasperhuangg still the same for me! mentions getting directly invited:
https://github.com/Expensify/App/assets/64629613/f2d674cc-803b-4c76-9bad-4623e62f9024

Ah shoot, the PR still hasn't been deployed, let me look into what's going on with that

@ishpaul777 I like your proposal, it seems to fit all the requirements. Can you post a video of you trying it with the steps I provided in the OP? The code that we need to test the shouldAllowActionableMentionWhispers flag should have been deployed to staging and production already.

@ishpaul777 interesting, I've tested it recently and it seems to be working. Let me investigate.

Are you sure they don't appear in Onyx? They're not rendering at the moment, but the data should be in Onyx.
If the user isn't getting auto invited then that means you should be hitting the correct logic when passing the flag.

@ishpaul777 i can check again in a bit today.

@ishpaul777 are you able to get the actionable mention whisper to show up in Onyx?

That error looks like you were able to, and now you're trying to call ResolveActionableMentionWhisper, I assume?

@ishpaul777 I saw what the problem is, we are deploying something that should fix this, sorry for the confusion!

@ishpaul777 the changes should be on staging now!

@ishpaul777 that looks correct to me! Thanks, looks great!

@ishpaul777 Ah, let me see what's happening with that

Hmm there was a problem with how we were constructing the actionable mention whispers in the back-end, I'm getting a fix up.

@ishpaul777 the changes are on staging, can you take another look please?

If the request fails you should reset the resolution to be what it was before, i.e. null.

Left an initial review, thanks @ishpaul777!

Ooh, weird. Hmm, the automation didn't work to move this over to Daily. I'll work on this now.

This was a new feature so we don't need a reg test. Handling payment and closing out.

Paid, thanks everyone!

@strepanier03 what do you think here? It looks like there was a follow up PR that fixed a regression caused by the original PR that implemented the new feature.

Re-opening for payment summary

Payment Summary

C+ - $500 - @rushatgabhane paid via Manual Request

@JmillsExpensify payment request incoming from Rushat.

@rushatgabhane - Go ahead and submit the request. I'll check back Monday to close.

$500 approved for @rushatgabhane based on summary.


Add reading of transaction logs and unit tests

While we're not working on replication yet, we should add the necessary methods to read the transaction logs so that we can unit test the transactions.

Still need to add tests for the edge cases around the transaction query API


Cannot avoid TrackingDisposableTransients for Web

When using DryIoc with Asp Net, current implementation is changing my container rules forcing TrackingDisposableTransients
https://github.com/dadhi/DryIoc/blob/d2d4eb9f5f1448c51c8c939d89e6a352ee3bf323/src/DryIoc/Container.cs#L5407
My reason to use DryIoc (as opposed to e.g. the Microsoft DI) is because I can configure such behaviour to my preference. However the current implementation is forcing Disposing of Transient instances and hence introducing bugs in my code (undesirable disposing transient classes after each request) since using DryIoc in Web context behaves differently than using DryIoc without Web
Example use case

IMemoryCache implements IDisposable. I've registered it as Transient, so I can have different instance for each use case
2 of my classes - e.g. CacheA, CacheB both require IMemoryCache as constructor argument, and are both registered as Singletons (for obvious reasons)
when running in Web context, whilst CacheA and CacheB are alive across different requests, their underlying cache is disposed after first request, because it's transient. Instead if I wanted to dispose the underlying cache, CacheA,B can implement IDisposable and call IMemoryCache.Dispose() inside

If the reason for forcing this rule is such that Microsoft's transient dependencies are disposed after each request as if running with defualt DI, there is a different override that can be applied to each "microsoft" transient dependency on registering, rather than forcing this globally?

Hi, that was down to conform with MS.DI, and probably it mat be modified to apply the Setup.With(trackDisposableTransient: true) for the ServiceCollection registrations. I will give it a though. The only drawback I see now is the increased complexity of the adapter, but let's see.
Until then, you may prevent the disposal of your components like this c.Register<MyTransientDisposable>(setup: Setup.With(preventDisposal: true));

Given that this will be a breaking change and  there is a workaround, I will keep the current rules intact. Let's see if I have complaints.


Show Google Login Failure Error

When the StartActivityForResult returns a result which is not RESULT_OK, the app doesn't do anything. This PR allows the app to show a failure message
Easiest way to test this change is to delete your SHA-1 from Firebase, then build app and see if the error appears

@jumaallan please change your tabbing to use tabs instead of spaces so that the diffs aren't really crazy. We use tabs in all our projects.

I had the wrong codestyle set on my IDE 😄
Will fix this


word2vec via gensim?

Would be nice to provide word2vec support within Julia. IMO its natural home would be here.
One way, and hopefully both the easiest and fastest, would be to interface with gensim via PyCall rather than to wrap the original C lib.

@timClicks There are some tradeoffs from the C version and the gensim version. You can see them in Radim Řehůřek presentation:
https://www.youtube.com/watch?v=vU4TlwZzTfU
The gensim version is suppose to be optimize using Cython + BLAS + Sigmoid table for non distributed this is very good better than the C version.
However when you start using multiple threads, the gensim version will take up  more memory than the C version, since there is a larger overhead.
The best choice to write the whole thing in Julia and take advantage of the power in distributive programming. But it is a long term thing since it is a lots of work.

Word2Vec should be made native to TextAnalysis like LSA and LDA. Word2Vec is already available as an independent modules. But the expectation is the whole blob of text should be provided to Word2Vec. With the corpus interface there may be a much better interface for Word2Vec.

Yeah, happy to take PR's to that effect. I generally use Word2Vec separately, but I agree it would be nice to have a combined interface.

I think Word2Vec.jl provides the functionality we need. https://github.com/JuliaText/Word2Vec.jl


az aks get-versions output changed to crap

Describe the bug
Previously it was so easy to find the current default AKS version supported by Azure using following query:
az aks get-versions -l westeurope | jq -r '.orchestrators[] | select(.default == true).orchestratorVersion'
But now, since 2.50.0 we get this monster:
{
  "values": [
    {
      "capabilities": {
        "supportPlan": [
          "KubernetesOfficial",
          "AKSLongTermSupport"
        ]
      },
      "isPreview": null,
      "patchVersions": {
        "1.27.1": {
          "upgrades": [
            "1.27.3"
          ]
        },
        "1.27.3": {
          "upgrades": []
        }
      },
      "version": "1.27"
    },
    {
      "capabilities": {
        "supportPlan": [
          "KubernetesOfficial"
        ]
      },
      "isPreview": null,
      "patchVersions": {
        "1.26.3": {
          "upgrades": [
            "1.26.6",
            "1.27.3",
            "1.27.1"
          ]
        },
        "1.26.6": {
          "upgrades": [
            "1.27.3",
            "1.27.1"
          ]
        }
      },
      "version": "1.26"
    },
    {
      "capabilities": {
        "supportPlan": [
          "KubernetesOfficial"
        ]
      },
      "isPreview": null,
      "patchVersions": {
        "1.25.11": {
          "upgrades": [
            "1.26.6",
            "1.26.3"
          ]
        },
        "1.25.6": {
          "upgrades": [
            "1.26.6",
            "1.26.3",
            "1.25.11"
          ]
        }
      },
      "version": "1.25"
    }
  ]
}

which was announced with this tiny remark:

Related command
az aks get-versions -l westeurope
Errors
none
Issue script & Debug output
az aks get-versions -l westeurope
Expected behavior
it would be great to have a less complex json output where someone can get the default version easier, as it was before the change
Environment Summary
azure-cli                         2.50.0 *
core                              2.50.0 *
telemetry                          1.0.8 *
Extensions:
authV2                             0.1.2
automation                         0.2.1
azure-devops                      0.26.0
bastion                            0.2.4
interactive                        0.5.1
log-analytics-solution               0.1.1
ssh                                1.1.6
Dependencies:
msal                              1.22.0
azure-mgmt-resource             23.1.0b2
Additional context
No response

Thank you for opening this issue, we will look into it.

I try also [any supported location], it doesn't work.
az aks get-versions --location [any supported location]
where by [any supported location] is either of below:
'australiacentral, australiacentral2, australiaeast, australiasoutheast, brazilsouth, brazilsoutheast, canadacentral, canadaeast, centralindia, centralus, eastasia, eastus, eastus2, francecentral, francesouth, germanynorth, germanywestcentral, japaneast, japanwest, jioindiacentral, jioindiawest, koreacentral, koreasouth, northcentralus, northeurope, norwayeast, norwaywest, polandcentral, qatarcentral, southafricanorth, southafricawest, southcentralus, southindia, southeastasia, swedencentral, switzerlandnorth, switzerlandwest, uaecentral, uaenorth, uksouth, ukwest, westcentralus, westeurope, westus, westus2, westus3'

Sorry for this breaking change, the output change is caused by the update of the backend API used. Starting from 2023-08-01/2023-08-02-preview, the response will contain the isDefault field to identify the default version, still working on it.

Good day, is there any update with regards to this issue?

I try with azure-cli 2.54.0 it already work.


increase memory resource for the operator

Increased resources.limits.memory from 30Mi to 300Mi.
Increased resources.requests.memory from 20Mi to 200Mi.

LGTM

/lgtm


ERC-7540: Asynchronous ERC-4626 Tokenized Vaults

ported from https://github.com/ethereum/EIPs/pull/7876

linking https://github.com/ethereum/ERCs/pull/10#discussion_r1373721970 @lightclient @SamWilsn I think EIP-X should probably fail eipw

The commit 787983e (as a parent of 60895d8) contains errors. Please inspect the Run Summary for details.

@lightclient the Run Summary links EIP repo with a mismatched PR