About NW-UCLA

Hello, when i tried to use CTR-GCN to train NW-UCLA, i got 95.5 accurancy , 1% lower than the result CTR-GCN given.
Did you meet the same problem in your experiments.

You may need to tune the weights of different modals.

Hello, when i tried to use CTR-GCN to train NW-UCLA, i got 95.5 accurancy , 1% lower than the result CTR-GCN given. Did you meet the same problem in your experiments.

You may need to tune the weights of different modals.

Thank you for your reply. Is this how you handled your experiment


Feature/ajax based thumbnails

Building on what @KorvinSzanto did, moving the ajax into a separate frontend js that runs when the basic thumbnailer is called. I'm sure there are ways to improve this – looking for the absolute path of least resistance on getting something into 8.2.
Galleries WILL have to update themselves or have a substandard experience; the Fluid Gallery worked but still showed broken thumbnails until the image was eventually built via AJAX – seems like CSS could make that look a lot nicer. A different gallery actually displayed an error message until the page was reloaded because it's trying to manipulate the images with javascript and do custom stuff.

In addition to @KorvinSzanto I would love reviews by @mlocati and @Mnkras

I'll be back from holidays at the end of this week

First off, I have not tired this, so am only writing from a position of ignorance.  I like the concept, but am concerned this could have disastrous consequences addons and themes in the marketplace.
Imagine a site that uses a gallery that bugs when this update is installed. On the current core, they site owner has to refresh a page a few times to get the gallery to work. Now they update their c5 to include this, add a few pics to the gallery and it breaks. It won't work again until the gallery developer has made changes, and those changes depend on a third party JQuery plugin being updated. We have had too many breaking changes since 5.7 and anything broken by this could become a permanent break because the 3rd party jQuery plugins are outside a c5 developer's control.
Could the new thumbnailer provide a very processing cheap and correctly sized 'work in progress' thumbnail? That way jQuery plugins will have an image to work with, then the ajax thumbnailer can replace the WIP image once the thumbnail is built.
Failing such a strategy, can the new ajax thumbnailing be made selectable, with updating sites defaulting to the existing behaviour?

Can't we make this optional? Upgraded sites use the old logic and new installations the new and a config setting allows you to override it? Like John, I'm worried about potential regressions that put a bad light on concretet5/packages/marketplace.

These are both good suggestions. What do we think about this @Mnkras @KorvinSzanto ?
Some background on this functionality: for years we just built the thumbnails on-the-fly when rendering a page and the thumbnails didn't exist. However, with the advent of the non-local storage locations, checking to see whether a thumbnail existed on other servers started to get very slow. Additionally, programmatically generating these things at runtime was always a bit of a risky endeavor given memory constraints. Finally, on sites with full page caching enabled, clearing the cache would clear the thumbnails, and regenerating the thumbnails could sometimes create a situation where a page would be cached before its thumbnails were built, leading to pages that had missing thumbnails.
However, these are both good solutions. I could certainly see this being a config option, as many sites don't have the problems I've described above (honestly, we've seen this be a huge problem on one site in particular – but we could simply enable this functionality for that site.) Additionally, I like the idea of generating the thumbnails asychronously, but I would love it if we could put a temporary image in the thumbnails place the moment the thumbnail build was requested. I like both of these ideas.

FWIW, google recommends using an async background task to load images in Android listViews.

I agree with Remo here, that it would be great if existing thumbnail generation and caching isn't changed via upgrading. It's not just galleries and packages where thumbnails are generated, we use it in places like custom pages lists or page templates. In fact, I can't think of a project recently where we haven't used thumbnail generation in some way.
For example, this page here uses something like  $ih->getThumbnail($f, 940, 705, false); to generate both thumbs and the lightbox enlarged image:
https://www.glennhawkephotography.com/wedding-photography
This uses the List Files From Set block and a custom template.
First load took quite some time, but this shared server still handled it.
Is this change suggesting that for me to update this site in the future I'll have to recode this block template?
I do totally understand the idea of creating thumbnails via javascript here, and this would fantastic as an option for gallery blocks, but I think it should be something that can be utilised, not the default behaviour.  I've very occasionally seen the issue of a site timing out or running out of memory due to very large galleries, this has been pretty rare for me, even on shared hosting. Generally a refresh to finish off the processing fixes everything up. But normally it's used for things like blog thumbnails or product lists, where you might have no more than a dozen images to crunch.
Apologies if I've misunderstood what this PR is actually doing - I'm personally just a bit concerned about all the places where thumbnailing may stop working!

@Mensuva: The problem that currently exists with 8.2.0.RC2 is a serious one: only a handful of thumbnails are created at once, both on dashboard and frontend pages. https://github.com/concrete5/concrete5/issues/5618

my 2 cents here: I am in theory totally happy to modify my gallery to use thumbnail types instead of legacy thumbnailer so I don't have to deal with all this js ajax thumbnail creation stuff.
However... I don't see any way of doing this that doesn't put the user in a bad spot.
Let's say I write some code that runs on upgrade. I check in the block's table for all blocks and grab their thumbnails width and height. For each block that's 1 for the thumbnail and 1 for the full screen image.
Then for each I create a thumbnail type if none already exists with those dimensions.
Then I have to find a way to show a message to the user to please rescan all images after upgrade.
And then I might have to explain why all of a sudden a lot of new images just appeared on their server.
So, add thumbnail types, pray the user understands how to rescan all files, pray they actually understand they have to do it...
Using thumbnail types is great, moving an existing gallery from legacy thumbnailer to thumbnail type is not so great.

I have updated the pull request to be dependent on a configuration setting, which is defaulted to off. For the sites that we need to update the behavior on (due to static caching, external storage locations, etc...) we will change the setting manually, and will not update the default behavior (at least not yet.) This should be good for an 8.2 launch. No galleries will need to change to have the current behavior honored.
@africanforest just as an aside: in very recent builds of 8.2 RC3 we updated this to use an asynchronous javascript call to progressively load these missing thumbnails. You are correct that the current build of 8.2 available for download doesn't have this so it just shows missing thumbnails until you reload the page a bunch of times.
This behavior is now gone, replaced by the asynchronous JS loader – however at this point that behavior is now only enabled if you turn it on through a configuration setting. By default we've moved back to the same behavior as 8.1 and earlier.

This should get us through 8.2.

With 8.2 I'm seeing thumbnails getting successfully created as in 8.1 in
application/files/cache
But there is also an empty thumbnails folder:
application/files/cache/thumbnails
So when I clear the cache WITHOUT checking (Clear cached thumbnails), all thumbnails are lost again.
I thought there would be some switch to force thumbnails getting created in application/files/cache/thumbnails in order to protect them from being deleted when the cache is cleared (and Clear cached thumbnails unchecked)?

If thumbnails get cleared when not checking that box that definitely sounds
like a bug, and something we should have a separate report on. This issue
is currently closed.


Wrong item count

Wrong count for array items. You get //9 items if the json array has 10 objects

The "9 items" in your screenshot refers to how many keys there are inside each collapsed object, not the length of the parent array
If you collapse the array itself, then you should see the array length
Let me know if it's still a problem


Implement usp_ssiscatalog as a command similar to WhoIsActive

Is this a feature OR bug:
Enhancement
https://trello.com/c/gKznAWF0
Additional comments
https://ssisreportingpack.codeplex.com/wikipage?title=usp_ssiscatalog
the author appears to have invested a lot of time in the help, we should keep this in mind and ensure this information can also be passed on to the user.

@BartekR - you interested in this at all?

Definitely! I recently started investigating https://github.com/sqlcollaborative/dbareports/issues/52 so I can work both at the same time. I would gladly work on that.

excellent! thanks so much. Also, added you to dbatools.io/team, thanks so much for joining us, man. So happy to have you on board!

Because the codeplex project site has been archived adding the code to that stored procedure here before it is lost:
usp_ssiscatalog.sql.txt

Looking over the source code for the referenced project and this is out of scope to just make one command like whoisactive for SSIS. There is metadata in SSIS now that I think this project was trying to cover. It has a number of dependent objects (tables, functions, procs) that would have to be deployed out to a given system and in some cases it would duplicate names that SSISDB already have (I think).
I'm going to just close this one and we can visit all the SSISDB stuff later in a different way...think the schema that SSISDB has the data this is trying to show.


P | preservePaths option

tar has a -P option to allow you to prefix entries with a /.  (Otherwise, leading / is stripped.)
Right now, it's kind of a hodgepodge in node-tar v2.  Make it consistent so that the WriteEntry does the right thing here.

linkpath can be anything
path cannot be empty
cannot add tar file to itself
if opt.preservePaths, anything goes
if the path contains .. at all, then warn and ignore unless opt.preservePaths x d/i/r/../r/dirlink: Path contains '..'
if the path is absolute, then strip the / from the start of it.  tar: Removing leading '/' from member names
if the path is absolute on windows, we strip the path.parse(d).root
when extracting, check to make sure that no portion of the element's path or dirname is a symbolic link.  If opt.preservePaths, then allow it.  If opt.clobberSymlinks -U, then delete the symbolic link and create the dir.

Do this in extract/create, not parse.

Implemented in Pack and Unpack streams.
High-level API just needs to translate P into preservePaths.


resident evil 5 steamworks

will this new version work with resident evil 5 steamworks version?

Why would it matter if RE5 non Gold has the ability to natively play splitscreen?

because the non gold doesnt have the dlc to play coop on same pc, and will be interesting to maybe play versus on splitscreen too

Thats weird: i dont have a single dlc installed nor the 'untold story' one.
No is it gold..

Nor*

you have to buy the dlc to have it on steam.....

Let me see..  Also got dlc; still cant. ??

Apparently it likes to (instant black window in game) close itself... Hmm

right... i went back to try it; im wrong.. apparently it only works with gfwl :(

CAPCOM removed the splitscreen code that was in GFWL version...

no, you can still play splitscreen with gfwl version

so... is gonna be supported?


WIP: update logstash to 1.5

This updates logstash, and fixes the logstash test.
Unfortunately, this doesn't (yet) support plugins. Logstash now uses Bundler, and so their old way of allowing you to specify a path to directory full of plugins (by modifying $GEM_PATH) no longer works. They provide a bin/plugin install mechanism to modify their Gemfile (which, remember, is arbitrary Ruby code), and then drive Bundler to install the gem alongside logstash. Yeah. So that's not going to work for us.
I'll inevitably need to rewrite this to build logstash from source using the (as of yet unreleased) new Bundler builder.

Logstash was updated in 9146edd5be812eb681a9a61e1db9bc493691e585 and the test fixed in 93132d171784d0056b90919ffed7c4f2a51e5da5.


Oracle Text

Searchable oracle text
Improved mouseover tooltip and card image placeholder.

Improved mouseover tooltip and card image placeholder.

There are edge cases and inconsistencies with double faced and split cards here as well ;)


Component not rerendered in wasm-bindgen-test when using hooks and function_components

Problem
I've recently created a template to write full stack app with rust using yew. I tried to migrate to current master of yew to use hooks and functional components. A template includes basic yew example with button incrementing number:
#[function_component(Model)]
fn model() -> Html {
    let counter = use_state(|| 0);
    let onclick = {
        let counter = counter.clone();
        Callback::from(move |_| counter.set(*counter + 1))
    };
    html! {
        <>
            <Logo />
            <Heading />

{ *counter }

                <button {onclick}>{ "+1" }</button>

        </>
    }
}

I have there an example test which looks like this:
#[cfg(test)]
pub mod tests {
    use super::*;
    use wasm_bindgen::JsCast;
    use wasm_bindgen_test::wasm_bindgen_test;
    use web_sys::HtmlElement;

    // This macro should be only called once per binary in crate
    wasm_bindgen_test::wasm_bindgen_test_configure!(run_in_browser);

    #[wasm_bindgen_test]
    fn clicking_on_button_should_increment_value() {
        let document = gloo_utils::document();
        let body = document.create_element("div").unwrap();
        yew::start_app_in_element::<Model>(body.clone());

        let value = body.query_selector(".panel > p").unwrap().unwrap();
        let button = body.query_selector(".panel > button").unwrap().unwrap();
        let button = button.dyn_into::<HtmlElement>().unwrap();

        assert_eq!("0", value.text_content().unwrap());
        button.click();
        assert_eq!("1", value.text_content().unwrap());
    }
}

However last assertion fails. Everything works when running application in browser, the state is correctly updated. Also this test was working on the "0.18" version of yew without function components and hooks. You can check it here.
Steps To Reproduce
Steps to reproduce the behavior:

git clone https://gitlab.com/zwolnik/fullstack-rs.git
git checkout 0.2.0
wasm-pack test --headless --firefox --chrome front
required dependencies and so on are listed in readme

Expected behavior
State is updated and component is rerendered so it can be tested
Environment:

Yew version: master d81c49d1dec
Rust version:  1.55.0
Target, if relevant: wasm32-unknown-unknown
Build tool, if relevant: wasm-pack
OS, if relevant: latest archlinux
Browser and version, if relevant: Mozilla Firefox 94.0.1

Questionnaire

[x] I'm interested in fixing this myself but don't know where to start
[ ] I would like to fix and I have a solution
[ ] I don't have time to fix this right now, but maybe later

Looks to me like you just forgot to append the newly created div to the DOM (div element stored in body variable)
Let me know if this helps :thinking:

Indeed, much thanks! For some reason this has worked on 0.18.0 without appending element to body.
It works after:
let document = gloo_utils::document();
let body = document.body().unwrap();
let div = document.create_element("div").unwrap();
body.append_child(&div).unwrap();
yew::start_app_in_element::<Model>(div);


tegraflash no rootfs?

@madisongh i'm following the instructions at the end of this issue https://github.com/madisongh/meta-tegra/issues/12. The one difference being I'm using core-image-minimal
it seems to go ok but it doesn't appear to flash a kernel or rootfs. After it reboots I just get a u-boot prompt. the doflash.sh happens very quickly but does seem to end ok
[   3.2199 ] Flashing completed

[   3.2199 ] Coldbooting the device
[   3.2206 ] tegradevflash --reboot coldboot
[   3.2211 ] Cboot version 00.01.0000
[   3.2231 ]

Is there a next step I need to do?

Can you try core-image-base instead, or add ${MACHINE_EXTRA_RRECOMMENDS} to your core-image-minimal images?  I think I need to move a couple of the packages listed in that variable to MACHINE_ESSENTIAL_EXTRA_RRECOMMENDS instead, so the kernel image, device tree, and extlinux.conf file get installed in /boot.
You may also need to explicitly set IMAGE_ROOTFS_ALIGNMENT to "4096" as mentioned in that other thread.

If anyone needs this information... this could also happen if you forget to remove the following configuration:
TNSPEC_BOOTDEV = "nvme0n1p1"


Add full example for kubernetes.io/cinder provisioner

What this PR does / why we need it:
Add kubernetes.io/cinder example, make it easy for beginner to understand the difference with openstack.org/standalone-cinder provisioner.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged):
fixes #62
Special notes for your reviewer:
NA
Release note:
NA

Have signed CLA.

/assign @hogepodge

/ok-to-test

I am not fully sure is this really needed? We have similar example there already if people can combine two files

@zetaab you mean "cinder-storage-class.yaml" and "example-pod.yaml"? looks there is no PVC define.

https://github.com/kubernetes/cloud-provider-openstack/blob/master/examples/persistent-volume-provisioning/cinder/cinder-full.yaml https://github.com/kubernetes/cloud-provider-openstack/blob/master/examples/persistent-volume-provisioning/cinder/cinder-storage-class.yaml

@zetaab I know your means, I just think if we can quick start a full stack example about kubernetes.io/cinder provisioner in one step "kubectl create -f cinder-in-tree-full.yaml", that is better for beginner than copying piece code from several files.

/ok-to-test

+1 to full complete examples. please do update some doc too when you can that can explain the files.
/approve

Can we turn this, and some of the other yaml files into a helm chart? It would be great to be able to helm install the thing rather then tracking down all the individual bits, changing config, then uploading one by one.

@kfox1111 +1 added a issue to track the idea

/lgtm


import failed as actors.internal package not found

I had installed with pip, but failed on the import actors package.
There is no actors.internal module or sub package on the installed directory.
Could you guide me how to solve this case?
Thanks!

This is a bug in packages — it must list all subpackes too:
packages=['actors', 'actors.internal', 'actors.utils']


xml preview broken

https://inveniordm.web.cern.ch/records/6dv0m-6c552

I've tested and this will be fixed by https://github.com/inveniosoftware/invenio-previewer/pull/179

Fixed in https://github.com/inveniosoftware/invenio-previewer/pull/179


added tests.

Powered by testify.

ping @sonots ?

I am not sure which testimg library is good to use, but adding tests are good.

it is okay to merge

Thanks. The reasons I want to use testify are below.

Go has standard test package as testing. But it does not support assertion. OTOH, testify supports assertion. I like test with assertion.
I'm testify user for over many years. It has been useful and stable.

Thanks, it sounds good to me.


MAINTAINERS: misc updates

Update collaborators and maintainer of some orphaned areas. Also update
path for some of the areas to include tests and samples.
Signed-off-by: Anas Nashif<EMAIL_ADDRESS>

@dcpleung maintaining user-space is a challenging task :p

Review Changes -> Request for Changes -> typing 'Oh no...` :P


add siepic - like connectivity check macro

for connectivity checks you currently need to install siepic tools
it would be nice to have it as a klayout macro inside gdsfactory, so we can improve the flow:

write it so you can call it from a script (it should be a function) as well as a GUI macro button
don't require to have some layers defined in your klayout layer properties lyp Waveguide DEVREC PORT PORTE
don't require to install siepic tools

@tvt173
@jaspreetj
@thomasdorch

SiEPIC tools connectivity check is too slow for big layouts
How about transition to the netlist browser from klayout?
https://github.com/austinapatel/KLayout-Connection-Validator
https://www.klayout.de/forum/discussion/2089/connectivity-based-stamping-check
@thomasdorch
@mabrains
@klayoutmatthias


fix(AL-901): Updated README info

Jira Ticket: AL-901
Changes

Updated README info to reflect a more cohesive version of the same type of info. The step by step guides have been removed and they have been replaced by an explanation of the operations that the library provides, followed by clear snippets of code examples that can be copied and tested locally.

@ejtorresm @rroa
All the content has been adjusted as per my first example in my first commit to the PR. All the endpoints are documented, with clear copy paste examples and brief descriptions. An outsider should be able to follow the use of the library without issues.
Please give me all the feedback that you can, if this structure holds to our standard then its the template we are going to use to document the other 2 libraries.

@MiguelTapTagshelf
Thank you for the thorough effort in improving the documentation for this library, especially given that it is a public one. This is definitely a step in the right direction. I do have a couple of suggestions to enhance it further:

Remove Results from C# Example Executions: The results included in the C# example executions seem to distort the actual response from the API. Removing these will make the examples clearer and more accurate.

Add Reference to HTTP API Documentation: Since this library serves as a wrapper for our HTTP API, it would be beneficial to include a reference to our HTTP API documentation. This will provide users with a more comprehensive understanding of how the library interacts with the API.

Your efforts are much appreciated, and these adjustments will help make the documentation even more valuable to our users.

In my last commit I removed the examples in the blocks of code in the README file and added the reference to the http docs in the overview section.


Event loop bug

Describe the bug
bug 1:
nodejs run one task and then one microtask since node 11, here in the container with nodejs 16 we get an older behavior
setImmediate(() => {
  console.log('immediate 1');
  Promise.resolve().then(() => {
    console.log('micro-task');
  });
});
setImmediate(() => {
  console.log('immediate 2');
});

bug 2:
when running in microTaskQueu, it should exhaust this queue before jumping to nextTickqueue
queueMicrotask(() => {
  queueMicrotask(() => {
    console.log('microTask');
  });
  process.nextTick(() => {
    console.log('nextTick');
  });
});

Link to the blitz that caused the error
https://stackblitz.com/edit/node-b76fas?file=bug1.js
Steps to reproduce
run npm start in stackblitz terminal
Expected behavior
bug 1:
// output: immediate 1 -> immediate 2 -> micro-task
// expected: immediate 1 -> micro-task -> immediate 2
bug 2:
// output: nextTick -> microTask
// expected: microTask -> nextTick
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots

Platform

OS: macOS 13.2.1
Browser: Chrome 110

Additional context
No response

Hey there! Thanks for reporting this issue and I can confirm that the event loop is not entirely the same as on local especially when micro tasks are in the mix. Unfortunately this is really hard to be fixed on our side due to browser limitations, or without trading performance. Out of curiosity, did you run into a specific issue with the event loop or were you just trying to understand it?

Thanks for replying. I found this issue when I dug into how the event loop works and wanted to make a quick demo in https://node.new to prove my research, but the output confused me, so I compared it to my several local node.js results. Anyway, it's ok to close this issue if it's limited by browsers now. Webcontainer is awesome.

Oh gotcha. Yea it's a little unfortunate that the event loop is slightly different and if possible we do intend to close the gap as much as possible. It's just a little difficult without scarifying performance too much. Thanks for reporting the issue tho, and I hope you can still use WebContainer for other things. Usually relying on the exact behavior of the event loop is something that I would avoid, even if you can somewhat rely on it locally. But it could also be a source of bugs.
I am going to close this issue for now, but keeping this in mind and also tracking it internally so we can improve the event loop over time as browser APIs improve, or new APIs / capabilities are added.

Hey, any updates on this issue?
I also saw this on the website of webcontainers while studying event loop.
I would like to use WebContainers but the order being different than the node is a current blocker to me.

This is the code I'm using

This is the result when running on node


cassandra_2_1: 2.1.20 -> 2.1.22

Motivation for this change
Issue: #99742
Reason: Fixes CVE-2020-13946 Apache Cassandra RMI Rebind Vulnerability
Description:
It is possible for a local attacker without access to the Apache Cassandra
process or configuration files to manipulate the RMI registry to perform a
man-in-the-middle attack and capture user names and passwords used to access
the JMX interface. The attacker can then use these credentials to access
the JMX interface and perform unauthorised operations.
Users should also be aware of CVE-2019-2684, a JRE vulnerability that enables
this issue to be exploited remotely.
2.1.x users should upgrade to 2.1.22

Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

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

Result of nixpkgs-review pr 104838 1

1 package built:

cassandra_2_1

Result of nixpkgs-review pr 104838 run on x86_64-darwin 1

1 package built:

cassandra_2_1


Series offset strategy needs a bit of thought

The problem with using dataBoxLeftOffset and dataBoxThickness is that it isn't totally apparent what the offset should be for non-databox point: ie whiskers and anchors. As it stands, the whisker caps are centered based on their own length, and the midline of the whiskers is not offset at all, which is clearly a bug. A further limitation is that the offset cannot be different from the bar thickness (creating a gap or overlap, if that is desired).

Idea: replace dataBoxLeftOffset with seriesXOffset that shifts the whole data series either left or right. Normally this would have the same value (or the same function) as dataBoxThickness.
applySeriesStyles needs to change to deal in absolutes instead of multiples of dataBoxOffset.


debug + output complains about token

I am running with debug and output and yet coveralls complains:
coveralls debug --output=asdf
Missing .coveralls.yml file. Using only env variables.
You have to provide either repo_token in .coveralls.yml, or launch via Travis

If I set the COVERALLS_REPO_TOKEN env, the command works.

Which version of the package? I believe that is fixed in alpha:
coveralls==1.0a2

Confirmed


KAFKA-14524: Rewrite KafkaMetricsGroup in Java

Instead of being a base trait for classes, KafkaMetricsGroup is now an independent object. User classes could override methods in it to adjust its behavior like they used to with the trait model.
Some classes were extending the KafkaMetricsGroup trait, but it wasn't actually used.
Committer Checklist (excluded from commit message)

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

@ijuma @satishd what do you think about this approach to rewriting KafkaMetricsGroup in Java? Currently, classes-users of the KafkaMetricsGroup trait sometimes override its methods to tune its behavior (mostly, names). Instead, they can create anonymous subclasses. In this PR I only demonstrate the idea. If you think it's viable, I can spread it and get rid of the KafkaMetricsGroup trait.

Hi @ivanyu, thanks for looking into this.

Currently, classes-users of the KafkaMetricsGroup trait sometimes override its methods to tune its behavior (mostly, names).

If that's the case, why not directly use explicitMetricName?

There are four places where KafkaMetricsGroup methods are overridden, it's always metricName. Sometimes it comes with some logic like in UnifiedLog. We of course can change this in various ways, but probably it's a good idea to keep it as is for the initial refactoring.

@ivanyu Does this add value over the approach used by QuorumControllerMetrics? I haven't compared the approaches closely yet, but interested in your thoughts.

@ijuma I don't see much difference with QuorumControllerMetrics on the low level.
Tags are not used in QuorumControllerMetrics (this is situational I think); type names are set explicitly instead of being derived from the class names. Apart from that, the metric names are formed the same way.
The metric registry is passed explicitly to QuorumControllerMetrics, but it's the same KafkaYammerMetrics.defaultRegistry().
On a higher level, the big difference is that QuorumControllerMetrics encapsulates the metric machinery and gives the user class just some meaningful methods. This has a downside, too: QuorumControllerMetrics doesn't have access to the state of the user class, so it has to store some bits of this state internally (these bunch of volatiles and atomics) to create gauges. If this is convenient to apply universally probably depends on how much state duplication is needed (e.g. fields like active or metadataErrorCount don't exist in the user class SharedServer, so QuorumControllerMetrics is the primary holder of those). It may be a bit inconvenient to do in all the classes like Partition, however in "big" classes like SharedServer this makes sense.

I've finished porting KafkaMetricsGroup to Java and I'm opening this for review.
I decided to not implement the performance improvements @lbownik proposed above and stick to verbatim porting to limit the review surface. The methods are far from the hot path so their performance doesn't affect much.

I've fixed the conflicts. @ijuma could you please have a look if you have time?

@ivanyu Please resolve the conflicts and update the PR.

Comments addressed, conflicts resolved.

@satishd can you help review this?

@ijuma Sure, I will have cycles after this week. I will review it by next Monday.

@ivanyu Please have incremental commits instead of a single squashed/merged commit. This is helpful for reviewers to review the incremental changes done in the PR that address the review comments.

Thank you @satishd! I addressed your last comment.

Please have incremental commits instead of a single squashed/merged commit. This is helpful for reviewers to review the incremental changes done in the PR that address the review comments.

I relied on Github being able to show the diff on force-push, but sure, I'll do incremental commits and can squash before merge.

A couple of failed tests in the jobs do not seem to be related to this change, merging the changes to the trunk.


No specification for what options are available for 'appType'

When reading the documentation, it is very unclear what options are available for appType, since they are not listed. There are some options sprinkled throughout the page, but it would be very helpful if all the options are listed for the properties that have a limited amount of options.

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

ID: f82ad1fa-cee5-c5c6-7050-21aaec6b714a
Version Independent ID: 617316ff-2dab-d021-880f-c1c79313c320
Content: Azure Function App task - Azure Pipelines
Content Source: docs/pipelines/tasks/deploy/azure-function-app.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @azooinmyluggage
Microsoft Alias: atulmal

Investigating

@PontusMagnusson Thank you for your feedback! the updated article will go live shortly

Thank you for updating! :)

Hi there! Hope you don't mind me commenting on a closed issue @ramiMSFT. I found this mentioned in another document: https://docs.microsoft.com/en-us/azure/logic-apps/set-up-devops-deployment-single-tenant-azure-logic-apps?tabs=azure-devops appType: 'workflowapp'.
Could you confirm or deny that workflowapp is a valid option, and if it is, update the documentation accordingly?

@jgarplind Thanks for your feedback! workflow is not a valid option. Please see the task json below. I'll update the sample shortly.
  {
      "name": "appType",
      "type": "pickList",
      "label": "App type",
      "defaultValue": "",
      "required": true,
      "options": {
          "functionApp": "Function App on Windows",
          "functionAppLinux": "Function App on Linux"
      }
  }


Fix: add missing .meta file

My Unity complains about the missing .meta file for LISENCE.md :
Asset Packages/com.gabrielbigardi.savesystem/LICENSE.md has no meta file, but it's in an immutable folder. The asset will be ignored.
Just created one.

Thanks, merged!


Added Similarity() for FamilyNode. Fixes #51.

This change is

Codecov Report

Merging #55 into master will decrease coverage by 0.13%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master      #55      +/-   ##
==========================================
- Coverage   88.25%   88.11%   -0.14%
==========================================
  Files          20       20
  Lines         783      791       +8
==========================================
+ Hits          691      697       +6
- Misses         84       85       +1
- Partials        8        9       +1

Impacted Files
Coverage Δ

individual_node.go
85.15% <100%> (+0.23%)
:arrow_up:

family_node.go
88.23% <66.66%> (-4.63%)
:arrow_down:

Continue to review full report at Codecov.

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


Add extra fields on the same level with status field

Hi!
Is it possible to add to healthcheck custom fields on the same level with status field?
I want something like:
{
    "status": "UP",
    "appVersion": "1.0.0-ci.234",
    "branch": "master",
    "owner": "John Doe",
    "checks": [ ]
}

So, I have added extra fields appVersion, branch and owner. I need this in healthcheck. How can I add this in code? Or does it requires changes in smallrye-health?

Hi @StanislavMakhrov. This is not currently possible to be done directly in the HealthChecks. However, there is an easy possibility to extend this yourself (workaround basically) - https://github.com/xstefank/learning-tests/blob/master/sr-health-custom-top-level-fields/src/main/java/io/xstefank/CustomHC.java.
If we would implement this in SR, would you like these values to be dynamic or can they be specified in the config values?

Nice example! Thanks! I think it should be in healthcheck out of the box.
I get these values from build.properties file using class with @ConfigMapping annotation.
Regarding nice example above. I can't do this in reactive way because such code causes error:
import com.aligntech.cs.treatcanaryapi.config.BuildInfoProperties;
import com.fasterxml.jackson.core.JsonProcessingException;
import com.fasterxml.jackson.databind.ObjectMapper;
import com.fasterxml.jackson.databind.node.ObjectNode;
import io.smallrye.health.SmallRyeHealthReporter;
import io.smallrye.mutiny.Uni;
import javax.enterprise.context.ApplicationScoped;
import javax.inject.Inject;
import javax.ws.rs.GET;
import javax.ws.rs.Path;
import javax.ws.rs.Produces;
import javax.ws.rs.core.MediaType;
import javax.ws.rs.core.Response;

@Path("/healthcheck")
@ApplicationScoped
public class HealthCheckController {

    @Inject
    BuildInfoProperties buildInfoProperties;

    @Inject
    SmallRyeHealthReporter smallRyeHealthReporter;

    @GET
    @Produces(MediaType.APPLICATION_JSON)
    public Uni<Response> reportHealth() throws JsonProcessingException {
        return Uni.createFrom().item(Response.ok(
                new ObjectMapper().readValue(smallRyeHealthReporter.getHealth()
                        .getPayload().toString(), ObjectNode.class)
                        .put("appVersion", buildInfoProperties.appVersion())
                        .put("revision", buildInfoProperties.revision())
                        .put("branch", buildInfoProperties.scmBranch())).build());
    }
}

Error:
Internal Server Error
Error id 4290e94c-ec27-43a8-ae34-5ecfe7461f22-1, java.lang.IllegalStateException: The current thread cannot be blocked: vert.x-eventloop-thread-9

We have also async support in SR health so to do the workaround the reactive way you can try this - https://github.com/xstefank/learning-tests/blob/master/sr-health-custom-top-level-fields-reactive/src/main/java/io/xstefank/CustomHC.java.
I see no problem in adding this then as a feature that will read the values that should be added at the top level from the config directly. Thanks.

@xstefank thanks again! It works perfect!
One question: how can I disable standard healthcheck /q/health
Because now I have two healthchecks - by url /q/health and my custom url.
This doesn't work for me:
quarkus:
  health:
    extensions:
      enabled: false

@SMakhrov this property will only disable any health checks that are exposed in different Quarkus extensions. There is no way to disable the /q/health endpoint. You can only use path configs to move it somewhere else - https://quarkus.io/guides/smallrye-health#quarkus-smallrye-health_configuration. But I will work on this feature in SR so there should not be a need for this hopefully soon.

Thank you so much!
@xstefank if I want to add some check (for example DB connection), how can I do this?
I've used this example https://github.com/xstefank/learning-tests/blob/master/sr-health-custom-top-level-fields-reactive/src/main/java/io/xstefank/CustomHC.java and tried to do something like:
    @GET
    @Path("/health")
    public Uni<Response> reportHealth() throws JsonProcessingException {
        HealthCheckResponse.named("Database").down().build();
        return smallRyeHealthReporter.getHealthAsync().onItem().transform(health -> {
            ObjectMapper objectMapper = new ObjectMapper();

            ObjectNode objectNode = null;
            try {
                objectNode = objectMapper.readValue(health.getPayload().toString(), ObjectNode.class);
            } catch (JsonProcessingException e) {
                e.printStackTrace();
            }
            objectNode.put("customKey", "customValue");

            return Response.ok(objectNode).build();
        });

    }

I've added this string according example above:
HealthCheckResponse.named("Database").down().build();

But I don't see Database in output JSON;
I've awaited that overall healthcheck would pay attention that DB is DOWN then overall status also will be DOWN.

@StanislavMakhrov for that you need to define the HealthCheck (or AsyncHealthCheck) normally as you used to - https://github.com/xstefank/learning-tests/blob/master/sr-health-custom-top-level-fields-reactive/src/main/java/io/xstefank/NormalHC.java. ``smallRyeHealthReporter.getHealthAsync()` is what is called when you access normal health endpoint (/q/health) and will process all HealthChecks beans normally.


Are we allowed to directly assign permissions to users?

I was working on #9314 when I realised no where in the docs we have an example of directly assigning permissions to users.
I asked in forum, but no one answered and I think here is a more suitable place to ask.
in official guide we have:
        $auth = Yii::$app->authManager;

        $createPost = $auth->createPermission('createPost');
        $createPost->description = 'Create a post';
        $auth->add($createPost);

        $author = $auth->createRole('author');
        $auth->add($author);
        $auth->addChild($author, $createPost);

        $auth->assign($author, 2);

permissions are children of roles and roles are assigned to users. this quote from guide also promotes this:

A role represents a collection of permissions (e.g. creating posts, updating posts). A role may be assigned to one or multiple users. To check if a user has a specified permission, we may check if the user is assigned with a role that contains that permission.

But what if one assigns a permission directly to a user? no roles in between.
        $auth = Yii::$app->authManager;

        $createPost = $auth->createPermission('createPost');
        $createPost->description = 'Create a post';
        $auth->add($createPost);

        $auth->assign($createPost, 2);

the above code works fine and also checkAccess(2, 'createPost') method will return true.
Is it okay to do that?  if Yes then #9314 makes sense, if not then I think assign() method should throw an exception when a permission is passed to it.

Technically it will work. It's less flexible though.


"Undefined array key "multiple"" on line 103 of /application/plugins/winter/builder/classes/ModelFormModel.php

After save dropdown in form builder this error.

multiple not exists =)
^ array:4 [
  "label" => "Тип"
  "showSearch" => true
  "span" => "auto"
  "type" => "dropdown"
]

This was fixed in https://github.com/wintercms/wn-builder-plugin/commit/36075870087aef719ccfa394c1d603ed8a9b2665, load this plugin via composer targeting dev-main as your desired version for now and perhaps @bennothommo can fix the test suite and push a new version when he has a moment.

Duplicate of #33

thanks =)


Add search embeddings extraction as part of the age_estimation workflow

Linked issue(s):
Fixes KOL-3500
What change does this PR introduce and why?
Introduce a documentation example for extracting search embeddings on image data.
Please check if the PR fulfills these requirements

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

Hmmm... I can't merge this since everyone else part of kolenaIO/kolena-owners is OOTO 😅

Please message me on Slack if you want a kolena_studio_client_api.tar.gz tarball for testing

Should we mention this example somewhere from our docs? I wonder if there should be a page under advanced usage section for this.

Should we mention this example somewhere from our docs? I wonder if there should be a page under advanced usage section for this.

Planning on that as the next step!


Implement #29, add python3.9

Codecov Report

Merging #30 (65aa1a9) into master (f34c478) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #30   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           35        35
  Lines         1650      1650
=========================================
  Hits          1650      1650

Continue to review full report at Codecov.

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


add Sonatype Nexus OSS as Docker Registry

@gtonic Looks like a cool product. I will have to try it out as well. Once you adjust the order of the entries we can merge this. Thanks again!


Why are the IS INITIAL clauses indented in two different depths?
Is there an ad-hoc-configuration mechanism in the interactive cleaner that goes beyond "block rule locally", so that I can experiment with the rule configuration ad hoc for my code  and apply the results one-time/locally?
Best regards,
Edo

Hi Edo,
this is an effect of the following option in cleanup rule "Align logical expressions":

(it's 23 spaces in your example). You can set this to 999 spaces, then you will never see this effect again :-)

Is there an ad-hoc-configuration mechanism in the interactive cleaner that goes beyond "block rule locally", so that I can experiment with the rule configuration ad hoc for my code and apply the results one-time/locally?

If I understand this correctly, I'd say: create a copy of your profile, do experimental configuration changes (and cleanups) with that copied profile, later switch back to the original profile.
After some experiments, you can always use the "Highlight" feature to compare the configuration of two profiles:

The rule list then shows: green = rule was activated (while it is deactivated in the other profile), red = rule was deactivated, yellow = rule configuration was changed; if you select a 'yellow' rule, changed options will be highlighted in yellow.
Kind regards,
Jörg-Michael

Hi Jörg Michael,

(it's 23 spaces in your example). You can set this to 999 spaces, then you will never see this effect again :-)

ah, ok. Not sure whether I like the default of that rule. Maybe it has to do with the seemingly irregular sequence of those lines in my cases (several alternations between long and short indent). The limit 20 makes more sense if there are 10 short variable names, then 10 long ones. In my case though, I want to keep the sequence as it is.

If I understand this correctly, I'd say: create a copy of your profile, do experimental configuration changes (and cleanups) with that copied profile, later switch back to the original profile.

Maybe you did not understand me. I meant something like a one-time exception (or even a pseudocomment to avoid processing by a rule?)
This option seems to be similar:

Thinking about the issue again, I have a feeling that this would go too far: Pseudocomments would make the code less readable again. And the possibility of "interactive" one-time rule exceptions would only work once; the next developer would - without bad intention - revert the code back to standard ABAP Cleaner formatting.
Best regards,
Edo


Normalization is only for scRNA data

Hello,
It seems the normalization was only performed on scRNA data. Is there a special reason for that? For example, in Stereoscope pipeline:
https://github.com/QuKunLab/SpatialBenchmarking/blob/3586b9d81c479f939aa40adedfe1c95c5b496bcb/Codes/Deconvolution/Stereoscope_pipeline.py
Thanks.

Hello, There is no special reason, we just followed the tutorial of each methods.


python311Packages.huggingface-hub: 0.21.3 -> 0.21.4

Description of changes
Changelog: https://github.com/huggingface/huggingface_hub/releases/tag/v0.21.4

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 293786 run on x86_64-linux 1

32 packages marked as broken and skipped:

python311Packages.dm-sonnet
python311Packages.dm-sonnet.dist
python311Packages.farm-haystack
python311Packages.farm-haystack.dist
python311Packages.openllm
python311Packages.openllm.dist
python311Packages.tensorflow-datasets
python311Packages.tensorflow-datasets.dist
python311Packages.vllm
python311Packages.vllm.dist
python312Packages.accelerate
python312Packages.accelerate.dist
python312Packages.chromadb
python312Packages.chromadb.dist
python312Packages.fastembed
python312Packages.fastembed.dist
python312Packages.gradio-client
python312Packages.gradio-client.dist
python312Packages.gradio-pdf
python312Packages.gradio-pdf.dist
python312Packages.open-interpreter
python312Packages.open-interpreter.dist
python312Packages.optimum
python312Packages.optimum.dist
python312Packages.peft
python312Packages.peft.dist
python312Packages.stanza
python312Packages.stanza.dist
python312Packages.transformers
python312Packages.transformers.dist
python312Packages.txtai
python312Packages.txtai.dist

22 packages failed to build:

khoj
khoj.dist
libretranslate (python311Packages.libretranslate)
libretranslate.dist (python311Packages.libretranslate.dist)
python311Packages.anthropic
python311Packages.anthropic.dist
python311Packages.cleanlab
python311Packages.cleanlab.dist
python312Packages.anthropic
python312Packages.anthropic.dist
python312Packages.datasets
python312Packages.datasets.dist
python312Packages.evaluate
python312Packages.evaluate.dist
python312Packages.litellm
python312Packages.litellm.dist
python312Packages.tokenizers
python312Packages.tokenizers.dist
tts
tts.dist
wyoming-faster-whisper
wyoming-faster-whisper.dist

86 packages built:

open-interpreter (python311Packages.open-interpreter)
open-interpreter.dist (python311Packages.open-interpreter.dist)
openai-whisper (python311Packages.openai-whisper)
openai-whisper.dist (python311Packages.openai-whisper.dist)
python311Packages.accelerate
python311Packages.accelerate.dist
python311Packages.argos-translate-files
python311Packages.argos-translate-files.dist
python311Packages.argostranslate
python311Packages.argostranslate.dist
python311Packages.chromadb
python311Packages.chromadb.dist
python311Packages.ctranslate2
python311Packages.ctranslate2.dist
python311Packages.dalle-mini
python311Packages.dalle-mini.dist
python311Packages.datasets
python311Packages.datasets.dist
python311Packages.diffusers
python311Packages.diffusers.dist
python311Packages.effdet
python311Packages.effdet.dist
python311Packages.evaluate
python311Packages.evaluate.dist
python311Packages.fastembed
python311Packages.fastembed.dist
python311Packages.faster-whisper
python311Packages.faster-whisper.dist
python311Packages.gradio
python311Packages.gradio-client
python311Packages.gradio-client.dist
python311Packages.gradio-pdf
python311Packages.gradio-pdf.dist
python311Packages.gradio.dist
python311Packages.huggingface-hub
python311Packages.huggingface-hub.dist
python311Packages.k-diffusion
python311Packages.k-diffusion.dist
python311Packages.litellm
python311Packages.litellm.dist
python311Packages.llama-index
python311Packages.llama-index-cli
python311Packages.llama-index-cli.dist
python311Packages.llama-index-vector-stores-chroma
python311Packages.llama-index-vector-stores-chroma.dist
python311Packages.llama-index.dist
python311Packages.manifest-ml
python311Packages.manifest-ml.dist
python311Packages.open-clip-torch
python311Packages.open-clip-torch.dist
python311Packages.optimum
python311Packages.optimum.dist
python311Packages.outlines
python311Packages.outlines.dist
python311Packages.peft
python311Packages.peft.dist
python311Packages.pyannote-audio
python311Packages.pyannote-audio.dist
python311Packages.sentence-transformers
python311Packages.sentence-transformers.dist
python311Packages.spacy-transformers
python311Packages.spacy-transformers.dist
python311Packages.speechbrain
python311Packages.speechbrain.dist
python311Packages.stanza
python311Packages.stanza.dist
python311Packages.timm
python311Packages.timm.dist
python311Packages.tokenizers
python311Packages.tokenizers.dist
python311Packages.torchinfo
python311Packages.torchinfo.dist
python311Packages.transformers
python311Packages.transformers.dist
python311Packages.translatehtml
python311Packages.translatehtml.dist
python311Packages.txtai
python311Packages.txtai.dist
python311Packages.vqgan-jax
python311Packages.vqgan-jax.dist
python312Packages.huggingface-hub
python312Packages.huggingface-hub.dist
rclip
rclip.dist
whisper-ctranslate2
whisper-ctranslate2.dist

Result of nixpkgs-review pr 293786 run on aarch64-linux 1

52 packages marked as broken and skipped:

libretranslate
libretranslate.dist
python311Packages.argos-translate-files
python311Packages.argos-translate-files.dist
python311Packages.argostranslate
python311Packages.argostranslate.dist
python311Packages.ctranslate2
python311Packages.ctranslate2.dist
python311Packages.dm-sonnet
python311Packages.dm-sonnet.dist
python311Packages.farm-haystack
python311Packages.farm-haystack.dist
python311Packages.faster-whisper
python311Packages.faster-whisper.dist
python311Packages.libretranslate
python311Packages.libretranslate.dist
python311Packages.openllm
python311Packages.openllm.dist
python311Packages.tensorflow-datasets
python311Packages.tensorflow-datasets.dist
python311Packages.translatehtml
python311Packages.translatehtml.dist
python311Packages.vllm
python311Packages.vllm.dist
python312Packages.accelerate
python312Packages.accelerate.dist
python312Packages.chromadb
python312Packages.chromadb.dist
python312Packages.diffusers
python312Packages.diffusers.dist
python312Packages.fastembed
python312Packages.fastembed.dist
python312Packages.gradio-client
python312Packages.gradio-client.dist
python312Packages.gradio-pdf
python312Packages.gradio-pdf.dist
python312Packages.open-interpreter
python312Packages.open-interpreter.dist
python312Packages.optimum
python312Packages.optimum.dist
python312Packages.peft
python312Packages.peft.dist
python312Packages.stanza
python312Packages.stanza.dist
python312Packages.transformers
python312Packages.transformers.dist
python312Packages.txtai
python312Packages.txtai.dist
whisper-ctranslate2
whisper-ctranslate2.dist
wyoming-faster-whisper
wyoming-faster-whisper.dist

26 packages failed to build:

khoj
khoj.dist
python311Packages.anthropic
python311Packages.anthropic.dist
python311Packages.chromadb
python311Packages.chromadb.dist
python311Packages.cleanlab
python311Packages.cleanlab.dist
python311Packages.llama-index
python311Packages.llama-index-cli
python311Packages.llama-index-cli.dist
python311Packages.llama-index-vector-stores-chroma
python311Packages.llama-index-vector-stores-chroma.dist
python311Packages.llama-index.dist
python311Packages.torchinfo
python311Packages.torchinfo.dist
python312Packages.anthropic
python312Packages.anthropic.dist
python312Packages.datasets
python312Packages.datasets.dist
python312Packages.evaluate
python312Packages.evaluate.dist
python312Packages.litellm
python312Packages.litellm.dist
python312Packages.tokenizers
python312Packages.tokenizers.dist

60 packages built:

open-interpreter (python311Packages.open-interpreter)
open-interpreter.dist (python311Packages.open-interpreter.dist)
openai-whisper (python311Packages.openai-whisper)
openai-whisper.dist (python311Packages.openai-whisper.dist)
python311Packages.accelerate
python311Packages.accelerate.dist
python311Packages.datasets
python311Packages.datasets.dist
python311Packages.diffusers
python311Packages.diffusers.dist
python311Packages.effdet
python311Packages.effdet.dist
python311Packages.evaluate
python311Packages.evaluate.dist
python311Packages.fastembed
python311Packages.fastembed.dist
python311Packages.gradio
python311Packages.gradio-client
python311Packages.gradio-client.dist
python311Packages.gradio-pdf
python311Packages.gradio-pdf.dist
python311Packages.gradio.dist
python311Packages.huggingface-hub
python311Packages.huggingface-hub.dist
python311Packages.k-diffusion
python311Packages.k-diffusion.dist
python311Packages.litellm
python311Packages.litellm.dist
python311Packages.manifest-ml
python311Packages.manifest-ml.dist
python311Packages.open-clip-torch
python311Packages.open-clip-torch.dist
python311Packages.optimum
python311Packages.optimum.dist
python311Packages.outlines
python311Packages.outlines.dist
python311Packages.peft
python311Packages.peft.dist
python311Packages.pyannote-audio
python311Packages.pyannote-audio.dist
python311Packages.sentence-transformers
python311Packages.sentence-transformers.dist
python311Packages.spacy-transformers
python311Packages.spacy-transformers.dist
python311Packages.speechbrain
python311Packages.speechbrain.dist
python311Packages.stanza
python311Packages.stanza.dist
python311Packages.timm
python311Packages.timm.dist
python311Packages.tokenizers
python311Packages.tokenizers.dist
python311Packages.transformers
python311Packages.transformers.dist
python311Packages.txtai
python311Packages.txtai.dist
python312Packages.huggingface-hub
python312Packages.huggingface-hub.dist
rclip
rclip.dist


Tentative fix to errors in new versions of oF related to ofPixels

This PR addresses #5. It's hacked together with a machete and duct tape, but the test project seems to building correctly.

I should note that I'm on openFrameworks 0.9.8, Xcode 8.2.1. I'm not sure what kind of impact this will have on other environments....


Lambda function HTTP Error

Hi all!
I using this module to send all alerts of the AWS, but when this function run occurs this error.
What is the problem? Security Group?
Thanks!
[ERROR] HTTPError: HTTP Error 404: Not Found Traceback (most recent call last):   File "/var/task/notify_slack.py", line 84, in lambda_handler notify_slack(subject, message, region)   File "/var/task/notify_slack.py", line 77, in notify_slack urllib.request.urlopen(req, data)   File "/var/lang/lib/python3.7/urllib/request.py", line 222, in urlopen return opener.open(url, data, timeout)
[ERROR] HTTPError: HTTP Error 404: Not Found Traceback (most recent call last):   File "/var/task/notify_slack.py", line 84, in lambda_handler     notify_slack(subject, message, region)   File "/var/task/notify_slack.py", line 77, in notify_slack     urllib.request.urlopen(req, data)   File "/var/lang/lib/python3.7/urllib/request.py", line 222, in urlopen     return opener.open(url, data, timeout)   File "/var/lang/lib/python3.7/urllib/request.py", line 531, in open     response = meth(req, response)   File "/var/lang/lib/python3.7/urllib/request.py", line 641, in http_response     'http', request, response, code, msg, hdrs)   File "/var/lang/lib/python3.7/urllib/request.py", line 569, in error     return self._call_chain(*args)   File "/var/lang/lib/python3.7/urllib/request.py", line 503, in _call_chain     result = func(*args)   File "/var/lang/lib/python3.7/urllib/request.py", line 649, in http_error_default     raise HTTPError(req.full_url, code, msg, hdrs, fp)

Hi @rlinstorres !
Such HTTP errors are usually happening when you don't provide correct slack_webhook_url.
Please use v2.7.0 where HTTP errors and are being caught with more details.

Hi @antonbabenko , thanks for your reply!
Now I'm try with version v2.7.0 and this is the error:
START RequestId: c1433d02-7170-44f1-8a9b-accd03de9980 Version: $LATEST [WARNING]	2019-12-23T14:10:32.676Z	c1433d02-7170-44f1-8a9b-accd03de9980	Event logging enabled:
{
"Records": [
{
"EventSource": "aws:sns",
"EventVersion": "1.0",
"EventSubscriptionArn": "arn:aws:sns:OMITTED:OMITTED:slack-notification:15405ea9-77dc-40d4-ba55-3f87997e5abb",
"Sns": {
"Type": "Notification",
"MessageId": "7c9f6458-2b6c-55f4-aee9-3222e6745e5a",
"TopicArn": "arn:aws:sns:OMITTED:OMITTED:slack-notification",
"Subject": "RDS Notification Message",
"Message": "{"Event Source":"db-snapshot","Event Time":"2019-12-23 14:10:24.176","Identifier Link":"https://console.aws.amazon.com/rds/home?region=OMITTED#snapshot:id=MY-TEST\",\"Source ID":"MY-TEST","Event ID":"http://docs.amazonwebservices.com/AmazonRDS/latest/UserGuide/USER_Events.html#RDS-EVENT-0041\",\"Event Message":"Deleted manual snapshot"}",
"Timestamp": "2019-12-23T14:10:32.199Z",
"SignatureVersion": "1",
"Signature": "kPGKHJ9rWTgK0Lw/UJow59z4B6cjoPfbnYlwDCbO/Wk/IlPmqjQMib94+GqozIPw4F9QEwwzb7RyaQ4IC3/iBoPYM5shVXkxdl2I8a7fyYqer4QgJWCUijZ60HhYZ7m2WeO7NJei5/8ahtLtyIPoD+8rHNiGJ9JV2RXokdsgWzbXIhbsQ6xGmcbwNe5FkpiqTcw7/52uJUWyUUcRz1E/BZEC5kFAw///u8JlioRmIC95e0isq724+5hf3BEryab3HC+5+BlWMPGug4FQ8kS8rquiXLKTl/e4ubFqz1GEUjiNoNXHqOqm9Bq+WNcBrmKMGNGhzr6In8Kh4srr56oGfQ==",
"SigningCertUrl": "https://sns.OMITTED.amazonaws.com/SimpleNotificationService-6aad65c2f9911b05cd53efda11f913f9.pem",
"UnsubscribeUrl": "https://sns.OMITTED.amazonaws.com/?Action=Unsubscribe&SubscriptionArn=arn:aws:sns:OMITTED:OMITTED:slack-notification:15405ea9-77dc-40d4-ba55-3f87997e5abb",
"MessageAttributes": {}
}
}
]
}
`
[ERROR]	2019-12-23T14:10:33.937Z	c1433d02-7170-44f1-8a9b-accd03de9980	HTTP Error 404: Not Found: result
[ERROR]	2019-12-23T14:10:33.941Z	c1433d02-7170-44f1-8a9b-accd03de9980	Error: received status `Content-Type: text/html
Transfer-Encoding: chunked
Connection: close
Date: Mon, 23 Dec 2019 14:10:33 GMT
Server: Apache
Vary: Accept-Encoding
Strict-Transport-Security: max-age=31536000; includeSubDomains; preload
Referrer-Policy: no-referrer
X-Frame-Options: SAMEORIGIN
Access-Control-Allow-Origin: *
X-Via: haproxy-www-ki4w
X-Cache: Error from cloudfront
Via: 1.1 85fc1201a1918facbeb30836e7391661.cloudfront.net (CloudFront)
X-Amz-Cf-Pop: IAD89-C1
X-Amz-Cf-Id: zvJmMENfc9f3s3KFPLK9LfJXrI0bQ7IhDXbGgS4vW2lhA9IHFR4VeQ==
using event{'Records': [{'EventSource': 'aws:sns', 'EventVersion': '1.0', 'EventSubscriptionArn': 'arn:aws:sns:OMITTED:OMITTED:slack-notification:15405ea9-77dc-40d4-ba55-3f87997e5abb', 'Sns': {'Type': 'Notification', 'MessageId': '7c9f6458-2b6c-55f4-aee9-3222e6745e5a', 'TopicArn': 'arn:aws:sns:OMITTED:OMITTED:slack-notification', 'Subject': 'RDS Notification Message', 'Message': '{"Event Source":"db-snapshot","Event Time":"2019-12-23 14:10:24.176","Identifier Link":"https://console.aws.amazon.com/rds/home?region=OMITTED#snapshot:id=MY-TEST","Source ID":"MY-TEST","Event ID":"http://docs.amazonwebservices.com/AmazonRDS/latest/UserGuide/USER_Events.html#RDS-EVENT-0041","Event Message":"Deleted manual snapshot"}', 'Timestamp': '2019-12-23T14:10:32.199Z', 'SignatureVersion': '1', 'Signature': 'kPGKHJ9rWTgK0Lw/UJow59z4B6cjoPfbnYlwDCbO/Wk/IlPmqjQMib94+GqozIPw4F9QEwwzb7RyaQ4IC3/iBoPYM5shVXkxdl2I8a7fyYqer4QgJWCUijZ60HhYZ7m2WeO7NJei5/8ahtLtyIPoD+8rHNiGJ9JV2RXokdsgWzbXIhbsQ6xGmcbwNe5FkpiqTcw7/52uJUWyUUcRz1E/BZEC5kFAw///u8JlioRmIC95e0isq724+5hf3BEryab3HC+5+BlWMPGug4FQ8kS8rquiXLKTl/e4ubFqz1GEUjiNoNXHqOqm9Bq+WNcBrmKMGNGhzr6In8Kh4srr56oGfQ==', 'SigningCertUrl': 'https://sns.OMITTED.amazonaws.com/SimpleNotificationService-6aad65c2f9911b05cd53efda11f913f9.pem', 'UnsubscribeUrl': 'https://sns.OMITTED.amazonaws.com/?Action=Unsubscribe&SubscriptionArn=arn:aws:sns:OMITTED:OMITTED:slack-notification:15405ea9-77dc-40d4-ba55-3f87997e5abb', 'MessageAttributes': {}}}]}and context<bootstrap.LambdaContext object at 0x7f7248cd5e90> END RequestId: c1433d02-7170-44f1-8a9b-accd03de9980

The test case you have is working well when I run it inside notify_slack_test.py, so I would recommend taking a look at IAM and network connectivity (VPC, Security Groups, NACL...) where AWS Lambda function is running.

For anyone who is searching for an answer: try to send a simple message using curl:

curl -X POST -H 'Content-type: application/json' --data '{\"text\":\"test\"}' 'https://hooks.slack.com/services/xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx'

Also check is your Incoming Webhook is enabled.

I've got the same issue. The root cause was a typo in the slack channel name.

I've got the same issue. The root cause was a typo in the slack channel name.


set input processor on Stage only once, in :show.

This seems to fix the ui button non-responsiveness issue I noted in #75.
Maybe it was some concurrent modification thing on the inputMultiplexer.

Please try the latest commit. It attempts to fix the issue by checking if the screen exists already in the input multiplexer. Your version looked fine but I wanted to minimize the changes to prevent unintentional breakages.

OK, that seems to work as well as my patch.  I do still see unresponsiveness sometimes -- if I add the actors to the stage later rather than creating them in the show function perhaps but haven't narrowed it down properly...

I just noticed that, while this commit fixed the ui actors which is great, it did something to on-touch-down.  Specifically (at least in my game) it now only fires in the outermost 1 or 2 pixels around the stage. Which suggests that it may be masked by the ui touch down listener - though i haven't looked into it.

Does this problem exist in your change?
On Thu, Mar 12, 2015 at 10:34 AM, Felix Andrews<EMAIL_ADDRESS>wrote:

I just noticed that, while this commit fixed the ui actors which is great,
it did something to on-touch-down. Specifically (at least in my game) it
now only fires in the outermost 1 or 2 pixels around the stage. Which
suggests that it may be masked by the ui touch down listener - though i
haven't looked into it.
--
Reply to this email directly or view it on GitHub
https://github.com/oakes/play-clj/pull/76#issuecomment-78491177.

Actually, no. But I don't know why.

On second thought, your version seems pretty safe, and doesn't do any unnecessary removals of actors. So, I'll use it instead.


APP-463: [C SDK] GPIO control slave application

Add client application for GPIO control for posix "platform"
Review: @Slash32 and @AlexShev76

@kaa-leeroy test this fully please


Problem: simple-network-setup page getting long

Solution: Start refactoring it into a new section
This first pull request in the refactoring of that page mainly just moves it into a page in new section. I wanted to retain as much Git history (about who contributed what) as possible, and I think this will work.
Note: The old URL will break, but if enough people complain, we can create a redirect (using ReadTheDocs).

This is strange... GitHub says that the Travis checks haven't completed yet, but when I go over to Travis, it says it's done.


[Hub Generated] Review request for Microsoft.Batch to add version stable/2022-10-01

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM API Information (Control Plane)
Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.

Deperated Certificates
-- Marked all Certificate related api's as Deprecated
-- Updated descriptions of Certificate related apis to indicate that the api's will be deprecated by Feb 2024
Add api support for node communication mode setting
-- Added definition for NodeCommunicationMode supporting types default, classic, and simplified
-- Added targetNodeCommunicationMode and currentNodeCommunicationMode property to PoolProperties definition

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[x] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

This  PR is labeled with breaking change, if that's the case, please get it approved first and come back to ARM review. Thanks.

@tadelesh please help review the Go SDK breaking changes

Approved for Go breakings as removal of unreferenced types are acceptable.


Feature/documentation tab

Closes #929

Added Swagger viewer
Added Swagger Petstore json localy

TODO:

[ ] Conditionaly hide link or viewer block if they are not available.

Current look of the documentation view:

added panels

@frenchbread, we are changing how we handle Swagger files in this PR. We need your help in a couple of areas where you are familiar:

Loading an uploaded Swagger file into the 'Documentation' tab Swagger UI
Optionally validating the Swagger file (yaml or json) in the AutoForm hook

Are you available to help us with this task?

@brylie I'll have a look into this.

The upload dialogue should be dismissed when the upload is successful.

Lets add a toaster icon to indicate when the swagger settings are successfully changed.

When Document Link is updated in Swagger UI, show a s alert to indicate this
http://s-alert-demo.meteorapp.com/
"Scale" and "sidescales" are two good examples.
Make sure the alert appears close to the textbox

Is the file-upload still in progress? Swagger parser/validator can use URL or file path, could we go with link to Swagger file or do we need to provide hosting for Swagger files to accomplish this feature?

@elnzv Check out this: https://github.com/tomitrescak/meteor-uploads/issues/235
File upload directory should be outside of the app directory to make them persist on the server when we deploy new version of application.

Evening coder strikes again.. I got https://github.com/vsivsi/meteor-file-collection working.
I just definitely need better UI for this though..
I demo this for you tomorrow, to see what you guys think.
List of uploads updates reactively.

@elnzv @brylie Pushed file upload code integrated for Apinf.
What works:
File upload manage modal has buttons for upload/select file and delete + placeholder for showing filename.
Todo:
Relationships are not there yet, and delete gets called but needs documentationId.
File success hook: https://github.com/apinf/api-umbrella-dashboard/pull/967/files#diff-a78c46af64e803f13c7d78a53c3635c9R56
Delete event: https://github.com/apinf/api-umbrella-dashboard/pull/967/files#diff-a78c46af64e803f13c7d78a53c3635c9R89

Cool! I will take a look and start working in relationships.
On Tue, 31 May 2016 at 17:37 Ville Jyrkkä<EMAIL_ADDRESS>wrote:

@elnzv https://github.com/elnzv @brylie https://github.com/brylie
Pushed file upload code integrated for Apinf.
What works:
File upload manage modal has buttons for upload/select file and delete +
placeholder for showing filename.
Todo:
Relationships are not there yet, and delete gets called but needs
documentationId.
File success hook:
https://github.com/apinf/api-umbrella-dashboard/pull/967/files#diff-a78c46af64e803f13c7d78a53c3635c9R56
Delete event:
https://github.com/apinf/api-umbrella-dashboard/pull/967/files#diff-a78c46af64e803f13c7d78a53c3635c9R89
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apinf/api-umbrella-dashboard/pull/967#issuecomment-222708446,
or mute the thread
https://github.com/notifications/unsubscribe/AHJGFOOPcki5D9VYemgYXrAKPaXoX4kBks5qHEeegaJpZM4Imkme
.

--
Illya Nizyev ~ www.nzv.nz

@elnzv I work on this currently forward after yesterday, when you come to ofiice let's see this together.

@elnzv Uploading needs check if file already exists, let's check it before we try to insert on DB.
"MongoError: insertDocument :: caused by :: 11000 E11000 duplicate key error index: meteor.fs.files.$id  dup key: { : ObjectId('725955531b42b3e2004093e0') } [409]"

We have asked for upstream support to figure out how to initialize the FileCollection.on() event handler, so that we can link the file upload with the API backend.

When no swagger document is available, show the message in the usual place of the document link: "No swagger document is available. You can add one using Manage API feature."

We still need an s-alert for the AutoForm onSuccess. E.g. when changing the documentation URL. Cing @Nazarah

When no documentation link is available, display the following text:

No documentation link is available. You can add one using Manage API feature.

Reviewing


Log bad JSON received by Slack if Slack sends us bad JSON.

We recently hit an odd issue with Slack: Slack is sending us messages over RTM which are not valid JSON. In order to debug this and send a useful error report to the Slack platforms team, we needed the actual messages being received, to see if they're truncated, empty or something else. I've sent this PR because this change seems like it would be helpful to others, as well.
P.S. Unfortunately, I was unable to run the tests—when I tried, I got these errors:
 FAIL  __test__/lib/Botkit.test.js
  ● exports bot interfaces

    Cannot find module 'ltx' from 'JabberGroupManager.js'

      at Resolver.resolveModule (node_modules/jest-cli/node_modules/jest-runtime/node_modules/jest-resolve/build/index.js:179:17)
      at Object.<anonymous> (lib/JabberGroupManager.js:6:61)

  ● exports bot interfaces

    TypeError: Cannot read property 'core' of undefined

      at Object.<anonymous>.test (__test__/lib/Botkit.test.js:22:19)
          at new Promise (<anonymous>)
          at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)

These appear to be unrelated to my change, so I'm going ahead and sending this pull request anyway. My apologies.

@n-rook this is merged, and will be in the next published version of Botkit. In the interim you can install from Github instead of npm to get the latest.
Let me know what you learn!

Thanks a lot!
So far, it looks like every instance we've seen of this is the empty string. (Unfortunate, it would have been more interesting if it were something else :))

@n-rook so you are receiving what looks like an empty string FROM SLACK?


API Issues unresolved

Hi, it looks like this API is orphaned. API responses (with valid key) are always 'unauthorised' and a bunch of issues in the API dashboard are unanswered for months. Any hint? Thanks, Christian
(See here: https://admetadata.portal.azure-api.net/issues)

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

ID: 0e291bd6-f015-c848-acb5-2fad69b32ebd
Version Independent ID: 36f59c20-e0c4-a30f-20df-5ffa06bfbf89
Content: App metadata API for advertising networks - UWP applications
Content Source: windows-apps-src/monetize/app-metadata-api-for-advertising-networks.md
Product: uwp
Technology: publish
GitHub Login: @hickeys
Microsoft Alias: hickeys

Hi - I'm sorry it's taken so long to get to this issue. I believe this is related to the closure of the Advertising features:
"As a reminder, we have shut down the Microsoft Ad Monetization services for Windows UWP apps per the sunset announcement made January 2020: "
https://social.msdn.microsoft.com/Forums/windowsapps/en-US/db8d44cb-1381-47f7-94d3-c6ded3fea36f/microsoft-ad-monetization-platform-shut-down-as-of-june-1st?forum=aiamgr
If this is not the case, and your issue is still a problem, please re-open this issue. Thanks - john


enable recording of child spans when remote parent exists. fixes #135

Hi, this change enables the recording of traces when a parent trace is available.
fixes #135
i found the answer here:
https://stackoverflow.com/questions/70700163/opentelemetry-context-propagation-test
thanks!

Hi @craigatk. I think this (the OpenTelemetryInit class, the test is up to you) should be merged, I also delved into the code and found the same solution.
Investigation

I was trying to figure out if the plugin was getting the trace and span IDs - it was.

Then I went deeper into the code. Everything works correctly, but when I tried to use LoggingSpanExporter instead of
https://github.com/craigatk/opentelemetry-gradle-plugin/blob/8c002c7c20ed344a124c7990c1c17f4c42b327e7/src/main/kotlin/com/atkinsondev/opentelemetry/build/OpenTelemetryInit.kt#L83
, I found that I don't get spans in the console if the IDs are set, and vice verse, I get them if the values are null.

I looked at the OpenTelemetry test as an example:
private static final SpanContext remote =
   SpanContext.createFromRemoteParent(
       SECOND_TRACE_ID, SECOND_SPAN_ID, TraceFlags.getSampled(), TraceState.getDefault());

and tried using getSampled instead of getDefault and the problem went away.

Root cause
If we add project.logger.info("Root span is recording: {}", rootSpan.isRecording()) after
https://github.com/craigatk/opentelemetry-gradle-plugin/blob/8c002c7c20ed344a124c7990c1c17f4c42b327e7/src/main/kotlin/com/atkinsondev/opentelemetry/build/OpenTelemetryBuildPlugin.kt#L87
we will get:

The log Root span is recording: false with getDefault().
and Root span is recording: true with getSampled().

I think this correlates with information in the documentations:

IsRecording
Sampled Flag
Span Processor receives Span?
Span Exporter receives Span?

false
false
false
false

@robwhitby thanks for working on this fix and @progxaker thanks for confirming it!
I'll get this merged and cut a new release of the plugin.

I released plugin v1.12.0 with this fix, please let me know if it works for y'all! :)

@craigatk thanks for your quick response :) Yes, everything works as it should be.
P.S. One question - why 1.12 and not 1.11?)

Glad it worked - thanks for confirming!
On why 1.12 - I didn't fully set up the release for 1.11 so I had to cut another release haha

all good, thank you! 🚀


Update README.md

Add IMPORTANT note about the transition from Microsoft Azure RTOS.

@timlt Can you nudge @v-thepet about getting her Eclipse ECA situation in order, please? We cannot merge without that check passing. I provided instructions in https://github.com/eclipse-threadx/.github/pull/5. Thanks!


Mutate JsonNode using JsonPointer

Environment

Nuget Package:  JsonPointer.Net
Nuget Version: 2.0.0

Can this library make doing something simpler?
Allow JsonPointer to mutate a JsonNode.
Today the JsonPointer.TryEvaulate method can retrieve the value of a JsonNode, but usually JsonNode is used for mutation. Something like a bool JsonPointer.TrySetValue(JsonNode root, JsonNode value) method would allow direct mutation of JsonNode using JsonPointer.
Same requirement was brought up here: https://github.com/gregsdennis/json-everything/issues/251#issuecomment-1143649899
Describe alternatives you've considered
In Json.NET, the same functionality can be achieved using JObject.SelectToken followed by a JToken.Replace.

Thanks for the suggestion.  This seems pretty in line with the JSON Patch replace operation.
var patch = PatchOperation.Replace(pointer, value);
var result = patch.Apply(root);

var newJson = result.Result;

The only catch is that I haven't updated JsonPatch to support nodes yet.

The only catch is that I haven't updated JsonPatch to support nodes yet.

Now I have 😄

Thanks!


Adding Multiple PD Endpoints Does Not Provide Redundancy

I have a TiKv cluster running in 3 test VMs: <IP_ADDRESS>:2739, <IP_ADDRESS>:2739, <IP_ADDRESS>:2739
Starting a basic mount with the following:
./target/release/tifs -f tifs:<IP_ADDRESS>:2379,tifs:<IP_ADDRESS>:2379,tifs:<IP_ADDRESS>:2379 /mnt/tifs/

Dec 18 10:00:35.994 INFO connect to tikv endpoint: "<IP_ADDRESS>:20162"

If I pause, stop or kill VM <IP_ADDRESS>, there is no failover, the file system doesn't try one of the other endpoints. Is this expected?
Are there plans to add redundancy?
Thanks

Doing more research, looks like the TiKV_client library might not be working correctly as the docs states is should be working. https://docs.rs/tikv-client/latest/tikv_client/struct.TransactionClient.html#method.new_with_config
Going to have to dig a bit deeper to see why the client library is not connecting to multiple PD nodes


Some users not able to log in

Reported in Slack:

Is the login via GitHub currently working? If so, why does the suggestion to log in appear even after I've logged in?
https://user-images.githubusercontent.com/8326331/216413758-9f2d2206-f8a3-4054-a022-b806c8b4d704.mp4

Awaiting more details from the person who reported it (who is the only one experiencing the issue), but my best guess is that it is a bug with this line:
https://github.com/biothings/mygeneset.info-website/blob/main/src/store/index.ts#L34
This user might not have the name field defined for some reason.
This ternary was probably included to appease TypeScript, but shouldn't be needed.

Hotfixed with https://github.com/biothings/mygeneset.info-website/commit/065c6859b6ce76f3ea1fc4959541db2a6aa5e93b


Proxy: dynamic hosts

Proxy support for dynamic hosts using data providers,
Currently supports only Etcd.
Caddyfile Format
proxy / etcd://[username:password@]etcd_endpoint1,etcd_endpoint2,...[/directory]

Example
#basic
proxy / etcd://<IP_ADDRESS>:2039
#complex
proxy / etcd://user:pass@<IP_ADDRESS>:2039,<IP_ADDRESS>:4001/caddy_proxy_hosts

Key value format
/directory/alias = endpoint

e.g. setting with etcdctl
$ etcdctl set /caddy_proxy_hosts/local1 <IP_ADDRESS>:2016

values of keys under /caddy_proxy_hosts will be added to proxy.
$ etcdctl ls /caddy_proxy_hosts
/caddy_proxy_hosts/local1
/caddy_proxy_hosts/local2

Very nice. :+1: Going to ping @j-mcnally, @GeertJohan, @pastjean, and @kelseyhightower to see what they think of this since they'd have a better idea than I would. Also would encourage them to see the demo video you made: https://www.youtube.com/watch?v=vP6M23Fwj-g
Knowing how well this is working will be important going forward as we prepare an API. Even if this PR doesn't make it into master, it'll help guide our development in the very near future, so we want some feedback. :smile:

I would say docker support would be a great addition for the future, but perhaps an existing docker -> etcd solution already exists.

Having this feature is very nice indeed!

This is really nice, would make it easy to support others like eureka,consul, ...
This is something i was initially planning to mock using server restarts, ... if this is built in 😬
Implementation makes sense too, good job

@captncraig check out https://github.com/jwilder/docker-register

@abiosoft just watched the youtube demo, this looks great!

@mholt i'd like to see dynamic land in master, and then have etcd and anything else building on that in its own addon. Thoughts?

@j-mcnally I think that's where we're headed with this, ultimately. We're exploring a solution to make the configuration dynamic with pluggable providers (etcd, consul, etc.) -- this is mostly a PoC.


⚠ Bofrost.lu has degraded performance

In 8e852b3, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2274 ms

Resolved: Bofrost.lu performance has improved in 53c862b after 6 minutes.


[Menubar Calendar] Right-click open in reminders

Description

Right-click on the reminder to open it in Reminder

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Approved


Add items to hackathons section

hack.summit() and Yelp Dataset Challenge to Hackathons section

Thanks @codeeachday

👍  Thank you.


Pagination

Undo changes for the server.ts in our backend directory

Why? I also made some changes on server.ts

Rugh tnoe we are not fetching from our backend we are doing from the API directly and I am working on a clean structure for the backend


Fernet release

This release is mostly for propagating migrations squash down to every env's

Codecov Report

Merging #221 into master will increase coverage by 0.15%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #221      +/-   ##
==========================================
+ Coverage   91.62%   91.77%   +0.15%
==========================================
  Files          47       47
  Lines        1671     1703      +32
  Branches      146      149       +3
==========================================
+ Hits         1531     1563      +32
  Misses        119      119
  Partials       21       21

Impacted Files
Coverage Δ

datahub/core/constants.py
100% <ø> (ø)
:arrow_up:

datahub/core/serializers.py
91.48% <ø> (-0.18%)
:arrow_down:

datahub/v2/views/service_deliveries.py
100% <100%> (ø)
:arrow_up:

datahub/company/serializers.py
95.34% <100%> (+0.56%)
:arrow_up:

datahub/interaction/views.py
100% <100%> (ø)
:arrow_up:

datahub/investment/urls.py
100% <100%> (ø)
:arrow_up:

datahub/metadata/serializers.py
100% <100%> (ø)
:arrow_up:

datahub/metadata/admin.py
100% <100%> (ø)
:arrow_up:

datahub/core/viewsets.py
93.54% <100%> (+2.24%)
:arrow_up:

datahub/company/urls.py
100% <100%> (ø)

... and 2 more

Continue to review full report at Codecov.

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


index-discovery command NPE when running with option -i or -r on individual community/collection

Describe the bug
ClassCastException is thrown when attempting to run the index-discovery command on an individual community.
To Reproduce
Steps to reproduce the behavior:

Run command index-discovery [-f] -i community-uuid

Expected behavior
The re-indexing of the community happens successfully and no exception is thrown
Related work
Link to any related tickets or PRs here.

This is the incorrect line: https://github.com/DSpace/DSpace/blob/b956bcd3891e372cc0fe5b1595e422e5d059bced/dspace-api/src/main/java/org/dspace/discovery/IndexClient.java#L182

I am working on a PR to fix this issue, together with another issue also within the IndexClient class that affects the -r option for removing communities/collections where there is an NPE on https://github.com/DSpace/DSpace/blob/b956bcd3891e372cc0fe5b1595e422e5d059bced/dspace-api/src/main/java/org/dspace/discovery/SolrServiceImpl.java#L223
I'll open the issue separately but both issues will be fixed on the same PR if that's OK. Let me know if separate PRs are preferred


sidebar_class_name not applied for index pages

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Description
When using a category index page (e.g. docs/how-to/index.md) and providing front-matter meta data
---
sidebar_class_name: green
---

i would expect, that this is applied to the category's <li class="green"> element, but it is not.
Current workaround: _category_.json with
// _category_.json
{
  "className": "green"
}

Self-service

[ ] I'd be willing to address this documentation request myself.

I think that's a reasonable expectation. Would investigate

Closing in favor of #6254


Update CLI args for --web-renderer to explain new default and clarify "skwasm"

Skwasm is not experimental (any longer)
It's the only renderer used when compiled to wasm

We should explain this!
--web-renderer          The renderer implementation to use when building for the web.

   [auto]            Use the HTML renderer on mobile devices, and CanvasKit on desktop devices.
   [canvaskit]       Always use the CanvasKit renderer. This renderer uses WebGL and WebAssembly to render graphics.
   [html]            Always use the HTML renderer. This renderer uses a combination of HTML, CSS, SVG, 2D Canvas, and WebGL.
   [skwasm]          Always use the experimental skwasm renderer.

We should also remove html and auto from the docs. They can stay hidden for some time to give people enough time to migrate.

Based on a discussion at the triage we concluded that we should just hide this option entirely. The reason is because the defaults already pick the correct compiler+renderer combo, and any other combination is unsupported, is deprecated, or just doesn't work:

flutter build web builds in CanvasKit mode with dart2js, which is the new default and fully supported mode. There's no need to also add --web-renderer=canvaskit.
flutter build web --wasm builds in Skwasm mode with dart2wasm, which is also a fully supported mode. There's no need to also add --web-renderer=skwasm.

Picking html or auto is no longer supported for new apps, and existing apps are already aware of the --web-renderer option and choose accordingly. They don't need the docs.
Picking canvaskit in dart2wasm mode is not supported. The tool will try to build it. We use it for ad hoc debugging sometimes. But there are no guarantees that it will work. We will not accept issues that affect this mode alone.
Picking skwasm in dart2js mode simply doesn't work.


bug: Factory-resetting deck calibration leaves behind legacy calibrations from before v4.0

Overview
When you clear an OT-2's deck calibrations, it appears to leave behind any calibration files that were created in robot software versions prior to v4.0. (v4.0 overhauled the way that robot calibration works.) This causes a confusing user experience because the app will show the robot as having a deck calibration no matter how many times you factory-reset it.
In other words, it seems like our code for getting the current deck calibration is smart enough to find these legacy files, but our code for factory-resetting deck calibrations isn't.
Steps to reproduce

Start with a robot that has deck calibrations created under a robot software version prior to v4. You can check this by SSHing in and looking for files like /data/deck_calibration*.json. (I'm working from memory, so I might have the exact filename pattern wrong.) If they exist, those are from prior to v4.
Find the Factory Reset menu in the Opentrons App.
Select Clear Deck Calibration.
Select Clear data and restart robot.
Reconnect to the robot and go to its page.

Current behavior

On the robot's filesystem, and in the robot's HTTP API, the factory reset does not clear the deck calibrations from prior to v4.
The Opentrons App sees that there is still a deck calibration and does not prompt you to calibrate the deck before using the robot.

Expected behavior
After the factory reset and reboot, the Opentrons App should prompt you to calibrate the deck.
So, either:

The Opentrons App should specially treat a robot that has a legacy deck calibration as having no deck calibration, for the purposes of prompting users to calibrate the deck. It can detect this by looking for "source": "legacy" in GET /calibration/status.
The factory reset should clear all deck calibrations, including those prior to v4.0.

Closed by #10656


COMP: Add sources for ITK modules only if they are enabled.

Cc: @thewtex

Thanks for the review.


Double output

My page gets shown twice (see screenshot). I tried to reproduce the bug with a very simple file and still the same issue. Any idea what this could be?
Note: when i update my home.region, the top part updates, while the second part (duplicated?) doesn't change.

I'm getting the same issue too, it seems to render the local templates then load in the current public squarespace html (and headers) at the bottom of the document.

I've tracked it down to the {squarespace-footers} tag, removing this fixes the duplicate output for me.

Good find. I'm also having issues again with the caching.
@kitajchuk You resolved the caching problem, no? The issue seems to be back.

Good find @cwaring.
The double output seems to be "overriding" your current css. So the online site.css interferes with your local css. Pretty annoying. When I remove the squarespace-footers tag, I don't seem to have any problems anymore :)

Having the same issues here with the double output... Can replicate the temporary fix by removing the {squarespace-footers} tag also. Any thoughts on a fix?

What version of the server are you guys on?

The latest.

Okay, I'll take a look.

Replicated this locally. Looking into a fix.

Looks like they changed the imageloader code snippet, so the footers parsing is breaking. Unfortunately there's not a better way to do that so I'll just have to update the code. Funny enough, they spelled "bootstrapper" wrong, "imageloader-bootstraper".

Just merged a fix in for this issue, 06282425c3ca45bb8ad0abc9c1dbc8a7b22a730b. The latest package version is<EMAIL_ADDRESS>If this seems to work then I'll close this issue :)

Ha, great. Least we know where to look when everything explodes again if they fix the typo :)
Thanks for the patch :+1:

Thank you @kitajchuk !


Random crashes and running in the background

TrueCraft crash randomly and runs behind all other applications.
I am on Windows 10 64 bit

Is It a Thread issue because I have the same problem ?

Windows is not supported.


fix: add SelectedNetworkController setNetworkClientId useRequestQueuePreference guard

Explanation
Our previous SelectedNetworkController fix that added a guard to permission state changes was not sufficient. This PR properly addresses the underlying issue of setting networkClientId for domains when the useRequestQueuePreference flag is false by moving the guard into the setNetworkClientId() method itself
References

Fixes: https://github.com/MetaMask/metamask-extension/issues/25097

Changelog

@metamask/selected-network-controller

FIXED: setNetworkClientId() will now result in a noop if useRequestQueuePreference is false

Checklist

[x] I've updated the test suite for new or updated code as appropriate
[x] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
[x] I've highlighted breaking changes using the "BREAKING" category above as appropriate

apologies, accidentally comingled some changes intended for another branch


update the referrals URLs to match the implementation in the API

What does this pull request do?

changes the URLs in the interventions service to match the URL structure discussed here https://mojdt.slack.com/archives/C01DYKJUKDX/p1606925013029800
makes the id parameter more representative of the real IDs (in fact, passing plain integers to these endpoints will result in 400 responses - is this something we want/need to test in the UI layer?)

What is the intent behind these changes?
ensure the api calls succeed on the real interventions UI when it's deployed.

@tomsmyers is this ready for re-review?


I would like a parallelized decrypt_all and shred_all

Is this possible and within the scope of what Blackbox is trying to solve? I'd like to take a whack at it if that's alright.

Sure!
Look at the git history to see an earlier attempt.   It only worked in some OSs.   You might want to leave the old code in and use the parallel version for OSs that are tested.

FYI: I'm rewriting blackbox in Go.  See the Golang branch.  I could use help making the shred command parallel.

I've done a little bit of work in golang at my current job, but I am not anywhere near proficient. I'd love to get my hands a little dirty in that.

  There's also no packaging.   Certainly all the shred and decrypt stuff is stable.
I'd gladly accept the changes to either branch.  I guess it depends on if you want to work on something that is going away in 2-3 months, or something that won't be ready for everyone to use for 2-3 months :-).  (I'm using the new golang version for all my personal projects. No problems so far!)

My apologies for resurrecting an old thread, but is there a current working version of blackbox with a parallelized decrypt_all?

My apologies for resurrecting an old thread, but is there a current working version of blackbox with a parallelized decrypt_all?

No
Not to be a downer but... I would reject PRs to add that to the bash version of Blackbox.  The bash version is brittle enough without adding such complexity.  The go version is abandoned (unless @jose-bonilla picks it up).

No worries, thank you for the reply! Are you aware of any more basic solutions to the problem of long blackbox decrypt_all's?

My recommendation is to keep secrets in Conjur, AWS KMS, Azure Key Vault or GCP KMS. Then use Blackbox for encrypting the API keys that let you access that system.  Now you are simply encrypting one tiny file.