TST: stats.ttest_1samp: fix xslow test

Reference issue
Closes gh-20641
What does this implement/fix?
Fixes the one test failure that occurs twice with each backend tested.
Additional information
[gh20641 aa257e0deb] TST: stats.ttest_1samp: fix xslow test
 1 file changed, 3 insertions(+), 3 deletions(-)
(scipy-dev) matthaberland@Matts-MacBook-Air scipy % git status
On branch gh20641
nothing to commit, working tree clean
(scipy-dev) matthaberland@Matts-MacBook-Air scipy % pytest scipy/stats/tests/test_stats.py::TestStudentTest::test_pvalue_ci -v
========================================= test session starts =========================================
platform darwin -- Python 3.11.6, pytest-7.4.3, pluggy-1.5.0 -- /Users/matthaberland/miniforge3/envs/scipy-dev/bin/python3.11
cachedir: .pytest_cache
hypothesis profile 'deterministic' -> database=None, deadline=None, print_blob=True, derandomize=True, suppress_health_check=[HealthCheck.data_too_large, HealthCheck.filter_too_much, HealthCheck.too_slow, HealthCheck.large_base_example, HealthCheck.function_scoped_fixture, HealthCheck.differing_executors]
rootdir: /Users/matthaberland/Desktop/scipy
configfile: pytest.ini
plugins: fail-slow-0.5.0, hypothesis-6.88.1, anyio-4.3.0, cov-4.1.0, skip-slow-0.0.5, xdist-3.3.1
collected 6 items

scipy/stats/tests/test_stats.py::TestStudentTest::test_pvalue_ci[less-numpy] PASSED             [ 16%]
scipy/stats/tests/test_stats.py::TestStudentTest::test_pvalue_ci[less-array_api_strict] PASSED  [ 33%]
scipy/stats/tests/test_stats.py::TestStudentTest::test_pvalue_ci[less-torch] PASSED             [ 50%]
scipy/stats/tests/test_stats.py::TestStudentTest::test_pvalue_ci[greater-numpy] PASSED          [ 66%]
scipy/stats/tests/test_stats.py::TestStudentTest::test_pvalue_ci[greater-array_api_strict] PASSED [ 83%]
scipy/stats/tests/test_stats.py::TestStudentTest::test_pvalue_ci[greater-torch] PASSED          [100%]

========================================== 6 passed in 8.73s ==========================================

@lucascolley, I've only been following the array API project peripherally, so I don't think I'll dig into it now to review this.
I hope all reviewers of PRs that touch XSLOW tests will be sure to run the tests themselves to ensure all is working as expected before merging.  Alternatively, perhaps the CI gurus could add a CI job that runs the XSLOW tests when a flag such as [run xslow] is provided in a commit message.

I tested locally again and things were still "good" so I merged it. I think the comments were addressed for the small patch here.


[stable/ndm]: update ndm charts to 1.0.1

update ndm charts to 1.0.1
update values.yaml
update README

Signed-off-by: Akhil Mohan<EMAIL_ADDRESS>

@prateekpandey14 @kmova  Haven;t updated the chart version in this PR. We already have 1.0.1 release in charts, because of some changes. But the actual 1.0.1 images are only available now. How will we handle such cases.?


Masonry not displayed on page load

Hi,
I've integrated this masonry into my Ionic React project. On the page load, cards are not displayed. When I adjust the page screen size, it will appear. I don't know what the was issue.
When I check the DOM it was empty.

I love this feature after resize, by default it didn't load. 😞

Hello! Thank you for reporting it. I can't help you without an example from your side - please provide one.
Thanks!

Hi Nikita Savchenko,
Thanks for your prompt reply.
I tried Stackblitz, Unfortunately, they support (Ionic+Angular), in my case is Ionic+ReactJs.
I've attached the GIF image. You can download and open it in chrome or firefox browser.
![Masonry-Issue](https://user-images.githubusercontent.com/414208/97961123-b57de400-1dd8-11eb-9410-ff495d0ba048.g

Thanks for the demo! Can you share your minimal reproducible example in Stackblitz please?

Hi Nikita Savchenko,
Thanks for your reply.
I tried with Stackblitz ReactJs+TypeScript Example It seems good, I can't create a reproducible example with Ionic+ReactJS, because there is no option in Stackblitz.
Ionic + ReactJS is the problem.

Got it. I've never used Ionic so it would be nice if you could make any single-command-launchable repository for me to see what's broken in XMasonry.

Hi Nikita Savchenko,
I've created a sample output Link for your reference. You can see the results when you can adjust the browser size.
Here is my Code:
App.tsx:
import React from 'react';
import { Redirect, Route, Switch } from 'react-router-dom';
import { IonApp, IonRouterOutlet } from '@ionic/react';
import { IonReactRouter } from '@ionic/react-router';

/* Core CSS required for Ionic components to work properly */
import<EMAIL_ADDRESS>
/* Basic CSS for apps built with Ionic */
import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>
/* Optional CSS utils that can be commented out */
import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>
/* Theme variables */
import './theme/variables.css';
import Home from './pages/Home';

const App: React.FC = () => {
  return (
    <IonApp>
      <Home />
    </IonApp>
  );
}

export default App;

Home.tsx
import { IonContent, IonHeader, IonPage, IonTitle, IonToolbar } from '@ionic/react';
import React from 'react';
import { useSelector, useDispatch } from 'react-redux';
import { XMasonry, XBlock } from "react-xmasonry";
const Home: React.FC = () => {
  return (
    <IonPage>
      <IonHeader>
        <IonToolbar>
          <IonTitle>Home</IonTitle>
        </IonToolbar>
      </IonHeader>
      <IonContent>

Masonry Content goes here!

        <XMasonry>
            <XBlock>

Simple Card

Any text!

            </XBlock>
            <XBlock width={ 2 }>

Wider card

Any text!

            </XBlock>
            <XBlock>

Card 2

Any text!

            </XBlock>
            <XBlock>

Card 3

Any text!

            </XBlock>
        </XMasonry>
      </IonContent>
    </IonPage>
  );
};

export default Home;

Thank you!
Hmm. That's definitely something's off with XMasonry (perhaps a race condition somewhere), however, there might also be a quick fix in when you initialize your Ionic react app. Try to do the initialization on DOM content load or just "onLoad" on body.
Test it yourself: right click at aruljayaraj.vethaexports.com, select "Save As" => have a locally saved copy of the webpage. Open it locally and boom, it renders:

Frankly speaking, I see more and more people using XMasonry, while it's been quite a hard from the maintenance standpoint project because of its initial design (it was a long while ago, when even flexbox wasn't that common!). Thus, I am thinking more towards rewriting it from scratch in a new major version release. This will bring a ton of improvements (the current internals of this library are very much tricky), however, don't expect it to happen very soon (I'm ultra-busy).
P.S. If vethaexports.com is your property and you can accept an advice to not to win the "best website design award", do at least these 2 simple things:

Remove custom scroll behavior from the page (you can keep the scrollbar itself but do not re-implement the scroll)
Remove music on page load

Hope this helps :)

Hi Nikita Savchenko,
Thanks for the prompt reply. Yes, It works when I save it locally. I don't know what the problem when I use it with Ionic+ReatJs. So I'll close this as of now.
Thanks for the suggestion about that website.

Thanks! The problem is indeed somewhere in XMasonry as far as I could see. I were just referring that it's better to rewrite the library than do fixes of this type: the library already has a lot of edge cases where its behavior is a bit weird. Thanks for bringing it up!


An unsed dependency on Encode::Locale

Build.PL declares a dependency on Encode::Locale, but it it is used nowhere:
$ git grep Encode::Locale
Build.PL:                 'Encode::Locale' => 0,

This is a vestige of b07014d2efbe9ac59f4cbf23970922a657c391f7 commit.

The same applies to Encode::EUCJPASCII, Encode::JIS2K, and Encode::HanExtra.

Also List::MoreUtils and List::MoreUtils::XS are not a run-time dependency. They are only used from dist/*/build.sh maintainer script.

And PerlIO::utf8_strict.

And Text::CSV_XS.


Error when run inference_example.sh

When I run inference_example.sh, I faced the following error. I have create a new folder called ckpt.
 File "3DFeatNet-master/venv/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 1470, in __init__
    self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access

NotFoundError (see above for traceback): Unsuccessful TensorSliceReader constructor: Failed to find any matching files for ./ckpt/checkpoint.ckpt
         [[Node: save/RestoreV2_3 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2_3/tensor_names, save/RestoreV2_3/shape_and_slices)]]
         [[Node: save/RestoreV2_24/_53 = _Recv[client_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device_incarnation=1, tensor_name="edge_172_save/RestoreV2_24", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:GPU:0"]()]]

The training step works well. By the way, what loss is recommanded to stop training?
Thank you so much for your help.

Hi, you need to place the training checkpoint in the ckpt folder. you can use the ones from your training or the one I provided (the link is in the readme).
For training, you can just let it run all the way as in my train.sh (which runs for 2+70 epochs). That should be sufficient for the model to converge.


Add continuous TSAN runs

run_tests -n inf -f -c tsan -l c c++
with errors flagged somehow
This should be run continuously on whatever is at head on master.

this seems to fall under the stress-testing umbrella.


fix(application): do not blow away cloudProviders when updating appli…

…cation
This has been a sneaky bug since forever. If you submit a partial update to an application, but don't include the cloudProviders field, it's gone. We happen to do this internally sometimes.

"Someday" we'll rewrite all this Application.groovy business...

Wacky.


Installing Mediawiki 1.30 created a window padding issue

Setup

Browsers and versions: Chrome Version 62.0.3202.94 (Official Build) (64-bit)
MediaWiki version: 1.30
Database version: MySQL 5.6.36-cll-lve
PHP version: 5.6.32
Pivot version: 1.0.5
Extension version (if connected to the issue): None

Issue
Hi,
I just installed Mediwiki 1.30 and the default window display is now reduced to 1024x768. See image attached. I searched but haven't found a parameter to allow the skin to be displayed full screen. Other skins are not creating that problem.

Thanks for reporting, I’ll investigate. It definitely should not be doing that.

Could you try changing this in your pivot features array? Set 'IeEdgeCode' => 1, to 'IeEdgeCode' => 0, Although I suspect MW 1.30 is just adding the meta tag without testing if IE = 9 or since the skin tests for IE=9 there is a conflict with the shim they added. That meta tag should NOT be inserted into head of a browser that is not IE 9 and it is there for Chrome and Firefox. 😞

Sorry for the delay, just had a chance to really look at this. Probably about time to have a real discussion on supporting IE 8. IE 8 support ended a while ago. Technically, IE 9 and 10 has ended too. Anyway, here is the fix.
Line 4 of /assets/scripts/pivot.js
Change
if (!jQuery.support.cssFloat) { jQuery('html').addClass('lt-ie9').addClass('no-js'); }

to
// if (!jQuery.support.cssFloat) { jQuery('html').addClass('lt-ie9').addClass('no-js'); }

I'm going to look at this further and probably just remove support for IE 8 completely. Or move it off to only load the CSS if IE 8 is detected properly. We have modernizr loaded too which would be a better way to detect.

Thank you very much, it works perfectly now.
Best Regards,
Phil

I was having this exact same problem, the real issue I had was that when viewing on mobile, it wasn't make the navigation sidebar go away. This fixed it all!

Since MW is no longer supporting IE 8 either this fix should imho go into 2.0

Closing, fixed in v2.0 found in develop branch


add licence

I'm watching your lambda days presentation and I noticed that while you had created this package you had not made it available.
Would you be interested in putting it on the Racket package catalog so others can use it.
This PR adds a simple licence declaration that is compatible with Racket.
I copied from the float/sci package https://github.com/soegaard/sci/blob/master/README.md
If you want to discuss I am a fellow Londoner and you can get me on the Racket discord https://discord.gg/6Zq8sH5 as Stephen or spdegabrielle on the racket discourse  https://racket.discourse.group/
best wishes and happy new year

I'd also encourage you to link your youtube video in the readme https://youtu.be/treqJHnl7lY
I'll do a followup PR

Hi Stephen,
Thank you for this - LGTM, and good idea about the link (will add in a moment).
I'm planning to put another AD project on the catalogue very soon as it happens. I may also slightly reorganize this package and put it up there too.
Have yet to join the Racket discourse/discord, but will look out for you there when I do!
Happy new year,
Oliver


Not able to delete a new file

Issue Type: Bug

create a new file (Ctrl + N)
click the X mark beside the Untitled name
click "Don't save" on the dialog
the file disappears and appears again

This happens even if I reload the window
Video sample:
https://user-images.githubusercontent.com/21330288/107715243-edbf8780-6d22-11eb-9e84-feb0f8042db1.mp4
VS Code version: Code 1.53.2 (622cb03f7e070a9670c94bae1a45d78d7181fbd4, 2021-02-11T11:48:44.518Z)
OS version: Linux x64 5.4.0-65-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8700T CPU @ 2.40GHz (12 x 3800)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_renderer: enabled_onvideo_decode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
2, 1, 1

Memory (System)
31.24GB (19.36GB free)

Process Argv
--no-sandbox --unity-launch --crash-reporter-id b9810d78-0af0-4474-be56-6bed3923ec23

Screen Reader
no

VM
0%

DESKTOP_SESSION
pantheon

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
pantheon

XDG_SESSION_TYPE
x11

Extensions (27)

Extension
Author (truncated)
Version

codesnap
adp
1.2.1

jest-snippets
and
1.8.0

atlascode
atl
2.8.5

turbo-console-log
Cha
2.1.6

path-intellisense
chr
2.3.0

bracket-pair-colorizer-2
Coe
0.2.0

vscode-eslint
dba
2.1.14

githistory
don
0.6.14

gitlens
eam
11.2.1

elm-ls-vscode
elm
2.0.3

prettier-vscode
esb
5.9.2

auto-rename-tag
for
0.1.6

codetogether
gen
3.2.0

vscode-pull-request-github
Git
0.23.1

Ionide-fsharp
Ion
5.3.2

vscode-styled-components
jpo
1.5.0

git-graph
mhu
1.28.0

vscode-docker
ms-
1.9.1

csharp
ms-
1.23.9

vsliveshare
ms-
1.0.3629

vsliveshare-audio
ms-
0.1.91

vsliveshare-pack
ms-
0.4.0

debugger-for-chrome
msj
4.12.12

javascript-test-runner
osh
1.2.4

git-file-history
pom
1.0.1

vscode-yaml
red
0.15.0

vscode-todo-highlight
way
1.0.4

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
pythonvspyt617cf:30253240
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30211401
vstry914:30256638
pythonvsdeb440:30248342
pythonvsded773:30248341
vstes627:30244334
pythonvspyt600cf:30251589

@AkshayHarti does the problem also happen after you run the command to reload with all extensions disabled? If not, please use the Extension Bisect command to investigate which extension is causing it. Then report the problem to the extension author and close this one.

@gjsjohnmurray , I found out that the problem occurred due to the codetogether extension. I disabled it and everything worked fine.
Closing the issue.

@AkshayHarti just wanted to let you know that we've been able to replicate this issue and will fix it in our next CodeTogether release. Thank you for the report!


UntaggedEnumVisitor::map already set

Hi, so I am working on a project using your library and I get this error when trying to deserialize an enum. I have replicated this issue in a git hub repo that you can download and see for yourself. Any help on this would be appreciated. Thank you.
https://github.com/EmekaLiberas/quick-xml-enum-deserialize-issue/tree/main/enum-deserialization/src

Hi just trying to follow up on my question to see if you could help me with this error. thanks


Screen gets stucked when using on iOS version above 10 & below 11

It happens so frequently page menu gets stucked i.e does not swipes on iOS version 10.1,10.2 & 10.3.

@siddharthsuneel Did you get any solution to that issue?

yeah, I'm also facing this issue the page menu doesn't shit to nxt option some time index changed but the indicator stuck on the index where it was...
have you find any solution ?

@JunaidVizz  No I haven't found any solution yet.
@aadilHaffez  What about you, is it working fine on your side ?
In case any of you finds the solution, please update here also. So that it helps everyone

@siddharthsuneel  actually I didn't find the exact place yet where the problem is sometimes it stuck on tap on notification sometime it stuck when the app comes from background and sometimes it just goes on normally for days :(

For all who are still struggling, the problem is in the CAPSPageMenu.swift file. This approach doesn't work on the newer models:
currentOrientationIsPortrait = UIDevice.current.orientation.isPortrait
Replace it with this:
currentOrientationIsPortrait = UIDevice.current.orientation.isValidInterfaceOrientation
            ? UIDevice.current.orientation.isPortrait
            : UIApplication.shared.statusBarOrientation.isPortrait

Thanks @iforaa it´s works fine.

THANKS @iforaa !!!


Please file a bug

Hi,
I got this
[CollectionView] An attempt to prepare a layout while a prepareLayout call was already in progress (i.e. reentrant call) has been ignored. Please file a bug. UICollectionView instance is (<UICollectionView: 0x7fe7cbdb0800; frame = (0 0; 414 774); clipsToBounds = YES; autoresize = W+H; gestureRecognizers = <NSArray: 0x600000d77960>; layer = <CALayer: 0x6000006a6d20>; contentOffset: {0, 0}; contentSize: {414, 0}; adjustedContentInset: {0, 0, 0, 0}; layout: <Xamarin_Forms_Platform_iOS_ListViewLayout: 0x7fe7cac3d560>; dataSource: <Xamarin_Forms_Platform_iOS_GroupableItemsViewController_1: 0x7fe7cac3dbc0>>)
Couldn't read view size for auto sizing

In the header template, I have an image and I am assigning the height of this image to a Binding.
HeightRequest="{Binding HalfScreenWidth}"
The collection view is displayed but not the image.
For information this value is 207, so not null or 0.
If I assign a fixed height (without binding), then I don't get this error and the headers images are displayed BUT the display is totally hazardous as some elements are behind the others.
Thanks

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

ID: db5e00a1-93a6-5965-70cc-cef81656dce0
Version Independent ID: 11361877-6f4b-b0f7-0ece-5dc6a44f9ce0
Content: Xamarin.Forms CollectionView Layout - Xamarin
Content Source: docs/xamarin-forms/user-interface/collectionview/layout.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @eli191
This needs logging in the engineering repo so the engineers can debug the problem and produce a fix. Thanks!


Peer registration failed in MacOS

It is rare issue, but happened several times and not on all Mac.

Nginx responded with 400 code error.
And we noticed that time differs
hmm, according to bazaar nginx logs, request happened at 09:57:50

Abdysamat Mamutov [2 hours ago]
but karaf log ends with 22:55:29,442

We could register after changing TZ on Mac. It was US tz and restarted console, after that were able to register peer.
But could not reproduce again.
Need to reproduce
On Mac change TZ than build peer. Register peer, if it fails set right TZ UTC and restart peer than try again to register
After succesfull registration - unregister and change to different TZ, restart peer and try to register peer

closed until reproduced


Update deps.

https://github.com/paulmillr/noble-curves/releases/tag/1.2.0
https://github.com/paulmillr/noble-hashes/releases/tag/1.3.2
https://github.com/paulmillr/scure-bip32/releases/tag/1.3.2

@alcuadrado this needs your review and merge

@alcuadrado ping


CalculateAverage_javamak

Check List:

[x] Tests pass (./test.sh <username> shows no differences between expected and actual outputs)
[x] All formatting changes by the build are committed
[x] Your launch script is named calculate_average_<username>.sh (make sure to match casing of your GH user name) and is executable
[x] Output matches that of calculate_average_baseline.sh

Execution time: 18s
Execution time of reference implementation: 9Mins
Submitter: Arun Kumar

Hey, as stated in the README and in the PR template, new entries should run in under 10 sec at this point, as it's just too hard to handle that large amount of submissions. This one comes in at > 40 sec. I'm still gonna add it, as you've put in the effort already. But should you plan to continue to work on it, please make sure it gets to that threshold for subsequent PRs. Thx for understanding!

00:42.893. Thx for participating!

Hi, Thanks for accepting the solution. I wasn't expecting it to be above 42 seconds as it was running around 18s in my 13th Gen i7 but the baseline took around 8 mins. I believe the multicore performance is better than the test machine but single core performance is weaker.


cirrus-cli: 0.133.0 -> 0.133.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/cirruslabs/cirrus-cli/releases.
meta.description for cirrus-cli is: CLI for executing Cirrus tasks locally and in any CI
meta.homepage for cirrus-cli is: https://github.com/cirruslabs/cirrus-cli
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.133.1 with grep in /nix/store/z6fayaxqn6byjbxxdmpl5rx6vy9h6j9m-cirrus-cli-0.133.1
found 0.133.1 in filename of file in /nix/store/z6fayaxqn6byjbxxdmpl5rx6vy9h6j9m-cirrus-cli-0.133.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

cirrus-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/z6fayaxqn6byjbxxdmpl5rx6vy9h6j9m-cirrus-cli-0.133.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cirrus-cli https://github.com/r-ryantm/nixpkgs/archive/2be29c721eb277feb3d875d3e7f72b1b1da98c8c.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/2be29c721eb277feb3d875d3e7f72b1b1da98c8c#cirrus-cli

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/z6fayaxqn6byjbxxdmpl5rx6vy9h6j9m-cirrus-cli-0.133.1
ls -la /nix/store/z6fayaxqn6byjbxxdmpl5rx6vy9h6j9m-cirrus-cli-0.133.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

cirrus-cli

Maintainer pings
cc @techknowlogick for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

.2 was just released, I've created a PR for it here: https://github.com/NixOS/nixpkgs/pull/363702


[BUG] AD does not work for interval > 1 hr

What is the bug?
When the detector interval is > 1hr, we experience long initialization and other unexpected behavior. We should test it out and fix.

@kaituo bumping to 2.10.0, can this be investigated for next release?

yes


[Codegen][CUDA] Fix: cuda codegen vectorize cast

Data types such as float32x8 and int32x8 are not supported in CUDA, which will result in errors like "Cannot convert type float32x8 to CUDA type!" in code generation. I tried to fix this by storing 2 32-bits values in 1 64-bits value.
Could you please help review this fix? @Laurawly

Thanks @kongroo @Laurawly


chore(package): update kronos-interceptor-decode-json to version 1.1.0

Coverage decreased (-8.5%) to 4.268% when pulling 9610ec8f6d4302c7a4063bcc910dcfe01022e2e6 on greenkeeper/kronos-interceptor-decode-json-1.1.0 into 0ea0743f9a7ceebaade071582e5e4ae5559b447e on master.


install bug

windows 10
node 4
atom 1.7

Thanks for the report. I'll have a look at that.

Thank you,look forward to that~

I have the same issue
windows 10
node 6
atom 1.7.3

Same for me
Windows 10
Node 6.2
Atom 1.7.4

Ditto.
windows 7
node 6.2
atom 1.7.4

Workaround for me on windows 10 was to open a command prompt (was not in Adminstrator Mode, not sure if it would make a difference)

cd %LOCALAPPDATA%\atom\packages
git clone https://github.com/Andifeind/atom-node-inspector.git
cd atom-node-inspector
npm install To bypass and use your own version of npm
apm install To then install the package into atom
apm link To make it appear in atom
Restart Atom

Hope this helps.

I pushed a new release. Hope this fixes the issue. node-debug tries to install node-inspector on its first start. This works for me on a Linux machine, I didn't tried it on Windows or Mac.


Simple Graph Exchanger Application

Benefit Hypothesis
Using the SGE Application, it will be easy to manually upload data supporting particular KA-based use cases
It will be possible to change and/or delete data
It will be possible to control under which policies and contracts the data is published
It will be possible to limit with which dataspace/internet resources the data can be merged/integrated

Beneficiaries
small and medium sized companies which want to directly upload/provide use case data

Acceptance Criteria
Either a separate application or a functionality integrated into the SDE
Should be publishable/compatible with the Catena-X Portal
Upload of data/creation of assets should be possible given use case templates
Changing of data and deletion of data should be possible.

Estimated Story Points:
30 SP

Story "Design/Prototype Graph Exchanger Pages"
Repository https://github.com/catenax-ng/knowledge-agents
Statement
As KA developer, I want to have a process diagram and (writeframe) prototype such that I can plan an API between frontend and backend.
Acceptance Criteria

process diagram
writeframe prototype
open API definition

Estimation
8 SP

Story "Implement Simple Graph Exchanger Backend"
Repository https://github.com/catenax-ng/knowledge-agents
Statement
As KA frontend developer, I want to have an implementation of the SGE API in order to bring my application alive.
Acceptance Criteria

API implementation on top of the agent plane

Estimation
8 SP

Story "Implement Simple Graph Exchanger Frontend"
Repository https://github.com/catenax-ng/knowledge-agents
Statement
As KA user, I want to have a ready-made frontend to the SGE API.
Acceptance Criteria

Implementation of the wireframe using the backend API

Estimation
8 SP

Story "Bundle SGE application"
Repository https://github.com/catenax-ng/knowledge-agents
Statement
As KA deployer, I want to have a ready-made chart of the SGE in order to deploy it.
Acceptance Criteria

Bundle deployment of frontend and backend

Estimation
4 SP


separate out channels

It is common when viewing many traces to apply offsets to each trace so that you can see them individually. It would be great to have a checkbox that allows for this view. See the corresponding logic in NWB Widgets here.

Once you have that, it would be great to be able to select a
It would also be great to be able to navigate up and down through channels. This will be needed for example in Neuropixel recordings, where there are more channels than can reasonably fit in a window all at once

https://flatironinstitute.github.io/neurosift/#/nwb?url=https://dandiarchive.s3.amazonaws.com/blobs/8d8/57c/8d857c1c-1bc3-48ac-ad59-266ea7eb9a79
This file has 384 ElectricalSeries channels, yet only 5 of them are visible in the viewer

Yes. There is a "group and order" controller widget on the left. This controls the grouping and the order within that grouping. Honestly, I have found that the grouping is far more useful than the ordering, so if you want to implement something quickly I'd recommend focusing on that. Though I suppose that could go in a separate issue

@bendichter How am I going to find the electrode information (e.g., the group)
Here's what I see:

Will it alway be in General -> extracellular_ephys -> electrodes? I don't see any pointer to that...

@magland this is where things get tricky.

Will it alway be in General -> extracellular_ephys -> electrodes? I don't see any pointer to that...

The pointer is ElectricalSeries/electrodes.table, which is a link to the electrodes table. It will almost always be at /general/extracellular_ephys/electrodes, though technically it could be other places. /general/extracellular_ephys/electrodes is a DynamicTable, and should be thought of as a table where each dataset represents a column. You'll need to infer the categorical variables for grouping. These columns are dynamic, and can change from dataset to dataset, so they cannot be hard-coded.
This general pattern will need to be repeated for RoiResponseSeries, which is associated with a PlaneSegmentation table through a dataset called .rois. The one difference is that PlaneSegmentation does not have a set place like the electrodes table does, so you are better off following the link in .electrodes and .rois than using the hard-coded path.

@bendichter Here's the current state of this widget

from https://flatironinstitute.github.io/neurosift/#/nwb?url=https://dandiarchive.s3.amazonaws.com/blobs/082/8f8/0828f847-62e9-443f-8241-3960985ddab3
You can control the number of visible channels (discrete set of options) and then use the left/right buttons to set the group of channels in the dataset. There's also a place to specify the degree of separation between the channels. I used the initial chunk of data to estimate the median local stdev of the data, and then used that (multiplied by a factor) to automatically determine a separation corresponding to a user-selected value of 1. The actual separation factor is pretty arbitrary - so I just put a.u.
Haven't yet handled coloring by group. I think I'll make a separate PR for that.
As mentioned in  #52, the above example loads much more efficiently than the other dataset I was working with. It's important that we get to the bottom of that.

Looks good!


Refactored and improved styling

Fixes #17, Fixes #15, Fixes #4, Fixes #5, Fixes #6

@yedhink @navaneethsdk
Please review.


predict_proba doesnot work?

predict_proba just does same thing like predict does

That is the intended behaviour. Since a hidden layer or a meta learner is trained on the predictions of the previous layer, each layer must be configured to predict probabilities or not at initialisation (i.e. in the add method).
Exposing a predict_proba method is just for compatibility with the scikit-learn API.


Allow marking certain layouts as favourite

This commit allows the user to mark certain layouts as favourite for use with new_window_with_favourite_layout command, which, much like new_window_with_current_layout, opens a new window; the difference is that said marked layout is used as a base instead of the current one.
If there are multiple layouts marked as favourite, only the first (per natural ordering in saved_layouts) is chosen.
Why this PR? Most of the time I want to open a new window with the same layout as always. Yet, all I have is that single one - using new_window_from_saved_layout for that feels counterproductive. Furthermore, I can't use neither it nor new_window_with_current_layout if I'm doing so from e.g. command line when no other windows exist. This is a partial remedy; I'd like to follow it up with a PR introducing a setting that would make Sublime automatically open windows using a certain layout. I haven't thought about the specifics yet - one idea is to use the marker introduced in this PR.

After some consideration this would be better served either as a companion plugin or a keymap using set_layout.


Protezionedati

Modifiche finite

Chiamami quando sei pronto che passiamo alla pagina successiva.
Andrea FabbIano
cell. + 39 (320).320.76.80
9 marzo 2017 14:49, "stefanomarty"  wrote:
Merged #20 (https://github.com/stefanomarty/cryptoteam/pull/20).
—

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub (https://github.com/stefanomarty/cryptoteam/pull/20#event-993147169), or mute the thread (https://github.com/notifications/unsubscribe-auth/AYekkP5BMySoTJxaoArirU7llNL9oGmeks5rkANggaJpZM4MX9cv).


Update Liquity Safe App

Name/Description
Liquity
Type
Update
Compatible Networks
Mainnet
Code for review
IPFS hash/App URL
test link: https://gateway.pinata.cloud/ipfs/QmYRyD91TT3aqtdNu7VsvhZSFmYNfCuDse3yFNbpNB2gt8/
ipfs link: ipfs://QmYRyD91TT3aqtdNu7VsvhZSFmYNfCuDse3yFNbpNB2gt8/
Desired date

Hey, I cannot get the app to load using a different IPFS gateway like https://cloudflare-ipfs.com/ipfs/QmYRyD91TT3aqtdNu7VsvhZSFmYNfCuDse3yFNbpNB2gt8/
Do you have an idea what can be the reason?
Unfortunately, pinata's gateway you shared has a very low rate limit, and we cant use it

Ok it seems like it needed some time to propagate

I confirm the app loads fine, and transactions are popping up. The app has been updated. Thank you


Grpc fiber integration with UPI in router

Description
This MR integrates the fiber grpc implementation within the router, to support the simplest configuration of a Turing router with grpc UPI routes.
Current router will not work with experiment engine or traffic rules yet. HTTP functionality are working as per normal.
*Note: Merging to a feature branch for this. CI aren't triggering since the target isn't a main branch. CI can be viewed under fork repo's action.
Changes

New UPI mission control that supports Route
engines/router/missionctl/mission_control_upi.go
Mission control contains the Route/Enrich/Ensemble logic. For now only route is added which is simply invoking the grpc fiber component.

New UPI server
engines/router/missionctl/server/grpc/server.go
Similar to the http handler. PredictValues of the UPI contract is actually similar to /predict endpoint. The UPI server will call the UPI mission control's route.

New env var in RouterConfig to indicate http or grpc router
engines/router/missionctl/config/config.go
This env var should eventually be created by API Server when deploying router.

Introduce ErrorProtocol for mapping Turing existing error into respective status code. Rename HttpError into TuringError
engines/router/missionctl/errors/errors.go
Referred to this doc as guideline.

Entrypoint to read envar and to create upi or grpc. Cmux library usage to serve http and grpc on same port.
engines/router/missionctl/server/application.go

Benchmarking for grpc/http UPI
This benchmark will use a common grpc/http client to call the Route directly versus calling through Turing
engines/router/missionctl/internal/benchmark/benchmark_e2e_test.go

Broken implementation (Traffic rules and experiment engine)

engines/router/missionctl/fiberapi/routing_strategy.go
Skipping grpc request for now. Experiment engine will be skipped. Eventually there should be a mapping into http payload to call the experiment engine, after extracting the experiment engine mapping and values from grpc request with protopath.
This will integrate with upcoming standard ensembler feature as well.

engines/router/missionctl/fiberapi/traffic_splitting_strategy.go and engines/router/missionctl/fiberapi/fan_in.go
not working with grpc request.
To be revisited with protopath and new trafic rules.

Minor changes

engines/router/missionctl/fiberapi/utils.go - minor refactor to separate fiber component and handler creation.
moved http directory to server/http
Added generics for proto metadata and http header since both are map[string][]string

Performance
Direct - Calling Route server directly
Turing - Calling Route server with Turing as proxy
For grpc, the generated service client is used for all.
For http, the route server uses protojson to convert request and response.
Benchmark_Upi_Grpc_Direct_Small-2   	    9782	    114621 ns/op	   17702 B/op	     312 allocs/op
Benchmark_Upi_Grpc_Direct_Large-2   	     247	   4813512 ns/op	 2428954 B/op	   27230 allocs/op
Benchmark_Upi_Grpc_Turing_Small-2   	    2696	    378148 ns/op	   60782 B/op	    1055 allocs/op
Benchmark_Upi_Grpc_Turing_Large-2   	     100	  12425686 ns/op	 88843 B/op	  108667 allocs/op
Benchmark_Upi_Http_Direct_Small-2   	    2917	    395544 ns/op	   71293 B/op	     656 allocs/op
Benchmark_Upi_Http_Direct_Large-2   	      54	  2204 ns/op	 5844320 B/op	  120654 allocs/op
Benchmark_Upi_Http_Turing_Small-2   	    2156	    554439 ns/op	   96649 B/op	     953 allocs/op
Benchmark_Upi_Http_Turing_Large-2   	      46	  24040678 ns/op	 9929781 B/op	  121016 allocs/op

Headers transmission
Passing of headers can be rather confusing as there is a need to state explicitly whether its an incoming or outgoing context. And Turing is a server to the user client, while acting as the client (through fiber) to the Routes (server).
The guidelines were followed and verified that headers sent by a Routes is indeed accessible to the user client.

Omg that's a lot of line changes 😶

merging this to grpc_upi feature branch. will open a WIP draft PR from grpc_upi into main containing all the upi features. While creating smaller PR to the grpc_upi feature by feature.


Refactor NetDaemonHost.HandleNewEvent

Breaking change

Proposed change

Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[ ] The code change is tested and works locally.
[ ] Local tests pass. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.
[ ] I have followed the [development checklist][dev-checklist]
[ ] The code compiles without warnings (code quality chek)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

@FrankBakkerNl thanks for this. Sorry for being late merging. Been busy times at work. I will check it tomorrow, there are some removed code I need to think about if it is ok. Looks good overall !


HOL 2 - Image not uploading

Image upload appears to be successful (no errors) yet no image file is observed in the storage account

Unable to repro. @stevenfollis Can you provide repro steps?

I've seen this intermittently as well.  We probably should have some error handling on the blob and queue upload.

This worked for the latest delivery. Re-open or create a new issue if we encounter this again.


Int8 Calibration  table creation for onnx model

Hi,
Is there a way to create the int8 calibration table with an onnx file as input instead of a .pth file?
That would be really usefull. or is there a simple workaround to this?

Hi, the C++ API does have this support: https://github.com/NVIDIA/retinanet-examples/blob/master/extras/cppapi/export.cpp#L88:

Uncomment 89
Comment 90
Add model name to 91 (like ResNet50FPN)
Set COCO_PATH in 10 correctly
make the executable again
Run as ./export model.onnx engine.plan <CALIBRATION_TABLE_NAME>


[Bug]: Known compatible PyBaMM and liionpack versions ?

liionpack Version
0.3.2dev (326b954169ba19a68acd2a8efae542c7f1458f0b commit)
Python Version
3.9.15
Describe the bug
Hi Dev team,
I really appreciate the effort the community is putting in PyBaMM and now liionpack.
I'm eager to run the example notebooks from the repo but I keep running into various runtime errors on multiples OSes and with multiple versions of liionpack (0.3.1, 0.3.2, 0.3.2dev) (see below).
I tried using Colab as a workaround for now but the pip install liionpack cell fails so it's a no-go.
I see bug reports for these issues already and some work has been done apparently suggesting it has been fixed in some commit.
Now, I would like to know if there are 100% known compatible versions of liionpack and PyBaMM that I can start with?
Can someone point me in the right direction please?
I can install either pip versions or specific commits.
Best regards,
Steps to Reproduce
print(lp.__version__)
print(pybamm.__version__)
print(python_version())

0.3.2
22.11.1
3.9.15

Expected behaviour
Able to run the 9 examples notebooks at:
https://colab.research.google.com/github/pybamm-team/liionpack/blob/main/
Relevant log output
Sample errors:
AttributeError: 'ElectrodeSOHSolver' object has no attribute 'get_initial_stoichiometries'
SolverError: Events ['Maximum voltage'] are non-positive at initial conditions
TypeError: solve() got an unexpected keyword argument 'htc'
KeyError: op['electric']

Additional context
Tried on Windows and Ubuntu 20.04.
Focusing on Ubuntu 20.04 at the moment.

Hi @arnauddevie thanks for your issue and sorry for the inconvenience caused. I am working to publish a new version of the code that should have all the fixes in place will let you know when complete. Hopefully tomorrow. By the way - I'd be interested to know what you are using our code for and if you are interested in finding out more about future projects. I'll send connection on linkedin if that's ok

The release is now on pip

@TomTranter Thank you very much.
I can confirm I was able to run all 9 examples notebooks without issues with the following configuration:
Ubuntu 22.04.1 LTS (GNU/Linux <IP_ADDRESS>-microsoft-standard-WSL2 x86_64)
from platform import python_version
print(lp.__version__)
print(pybamm.__version__)
print(python_version())

0.3.3
22.12
3.9.15


[REQUEST] Add a feature to only return search results when the search term is in the content of the tweet, not just in the username.

[REQUEST] Add a feature to only return search results when the search term is in the content of the tweet, not just in the username.
For example if I want to run a search to discover how many early bitcoin adopters tweeted about bitcoin before the year 2010 I would use this code:
import twint

c = twint.Config()
c.Search = "bitcoin"
c.Until = "2010-01-01"

twint.run.Search(c)

However, this returns thousands of unhelpful hits because so many people have changed their username to include the word bitcoin which causes this code to return anything and everything they tweeted before 2010, even though the tweets dont have to reference bitcoin at all.
For example if @exampleBob123 tweeted in 2009 "I like spaghetti" but later in the future he changes his username to be @bitcoinBob123 my search will return his tweet <bitcoinBob123> "I like spaghetti" because his username now includes the term bitcoin. It would be really convenient if I could restrict the output to only include tweets that are actually referencing bitcoin.

Hi,
thanks for creating this feature request.
I do think this is a big issue which pollutes your query results to some extent, (sometimes more, like in your case). But currently I don't think if it's possible to implement using the twitter's current search functionality. Perhaps twitter would add this when they fully migrate to the newer v2 API Endpoints (Although I am not to sure about that).
I'd like you to go through this thread to get a better understanding of what I am talking about here. This thread does provide you with a temporary solution (which works to some extent) but In my views it's not sustainable, as when you add more parameters to the query it'd break the twitter search (twitter only allows a certain number of parameters to be added to the query, I am not sure about the exact number though). For example, In the thread that I referenced above, if you exclude a ton of users from the search result, it'd certainly break the search functionality. I remember reading about this somewhere in the twitter documentation.
The above workaround does have a demerit though, that It'll exclude all the tweets from the filtered users even when those tweets contain the desired search parameter. Like in your case, if you are searching for bitcoin and you exclude a user named bitcoinhodler from your search query, then it will not return any tweet from this user, even when 90% of the tweets by this user contained the phrase bitcoin.
Finally, I'd suggest you to simply run the search using your current configuration :

import twint
c = twint.Config()
c.Search = "bitcoin"
c.Until = "2010-01-01"
twint.run.Search(c)

after that filter the resulting csv/dataframe using Pandas.
This would give you the best results in my opinion. This certainly requires some extra efforts on your part, but that depends on whether your use case requires that or not.
One more thing is, this can be implemented in twint but we'll have to see if this feature is actually needed. In my opinion it's better to leave the data cleaning part to the user. But then again I may be wrong, this needs a bit more discussion.

Oh thank you, that thread explained it really well.


question about cityscapes dataset preparation

Hi, @tinghuiz ,
I'm trying to prepare cityscapes dataset. According to the download page , generally we'll download "leftImg8bit_trainvaltest.zip" which are not consecutive frames and used for segmentation tasks. I guess we should download a dataset with consecutive frames, could you reply the name of exact dataset zip file to prepare? e.g. leftImg8bit_trainextra.zip / leftImg8bit_demoVideo.zip / leftImg8bit_sequence_trainvaltest.zip / etc  (which one ?)
Thanks!

Just updated the readme. Please take a look and let me know if you have further questions.


feat(release): Sign container images with Cosign

Proposed Change

The release process signs artifacts using Cosign. This PR extends this capability to container images and manifests. This is part of a larger effort to sign all BindPlane related artifacts.
The existing release workflow handles the placement of the private key and private key password. The public key already exists in the repo at signature/bp_agent_key.pub per out documentation here.
Testing
Testing is only possible when building a full release. I forked the repo and built a tag against my fork. The release passed and the resulting image can be verified with Cosign.
cosign verify \
  --key signature/bp_agent_key.pub \
  ghcr.io/jsirianni/bindplane-agent-arm64:0.0.2-minimal

Verification for ghcr.io/jsirianni/bindplane-agent-arm64:0.0.2-minimal --
The following checks were performed on each of these signatures:
  - The cosign claims were validated
  - The signatures were verified against the specified public key

[
  {
    "critical":{
      "identity":{
        "docker-reference":"ghcr.io/jsirianni/bindplane-agent-arm64"
      },
      "image":{
        "docker-manifest-digest":"sha256:820fd206415c6ddc1f95d3147ac7cefdeff634d518300a9d8d0328bbced7bd6c"
      },
      "type":"cosign container image signature"
    },
    "optional":null
  }
]

Checklist

[x] Changes are tested
[x] CI has passed

@dpaasman00, after further testing I realized I needed the --recursive flag to support signing container image manifests.
I learned that while testing this PR.
$ cosign verify \
  --key ~/git/bindplane-agent/signature/bp_agent_key.pub \
  ghcr.io/observiq/cosign-test-transform-agent:0.0.2-bindplane

Verification for ghcr.io/observiq/cosign-test-transform-agent:0.0.2-bindplane --
The following checks were performed on each of these signatures:
  - The cosign claims were validated
  - The signatures were verified against the specified public key

[{"critical":{"identity":{"docker-reference":"ghcr.io/observiq/cosign-test-transform-agent"},"image":{"docker-manifest-digest":"sha256:b90b67882d80b8744203a8d96fa1af1437721125dc8b9e872602c80bf21a5b9b"},"type":"cosign container image signature"},"optional":null}]

I think it is pretty solid now.


Add a class that represents ItemTypes with enchantments and other things

One of the reasons issue #1074 was submitted was because ItemStacks were the only way to hold enchanted sign-shop items. The class ItemStack should only represent the contents of an inventory slot, and be limited to max size the particular item supports. I suggested that a Map<ItemType, Integer> be used, but that did not hold the required enchantments. This is why there should be a ItemData class in between ItemStack and ItemType, and all of the DataManipulators of ItemStack be moved into it. ItemStack would be like a special kind of Pair<ItemData, Integer> that limits sizes, ItemData would be where all the DataManipulators would be, and ItemData would represent all of the items from vanilla / forge.

I'm sure that the issue is already closely related to just being able to set the ItemStack quantity greater than the ItemType's max defined stack size. Aside from that, there are things called ItemStackSnapshots that can represent an ItemStack without being able to modify said snapshot.
I have honestly no clue what function this ItemData would represent or how it would be used, let alone the fact that it's acting more as a cache of the possible ItemStacks, which to me is rather wasteful of resources.

Then I suppose that for shop-signs, storing item data would best be done by using ItemStackSnapshots. But then the other problem is, is there a way to (nicely) serialize ItemStackSnapshots, and then deserialize them? This would be for storing them into a Database..
ItemType, I "serialize" by it's getId/getName.
Amount is by Integer.
...And I suppose you could then serialize ItemEnchantment by it's id & level, and then each lore by a (text)-string, durability, and so forth for every DataSerializable object is has with it...
Would be quite a bit of work, but it could be done, by the plugin author.

@MichaelFedora ItemStackSnapshots are natively DataSerializables, meaning they can be serialized to DataContainer and likewise translated into ConfigurationNodes, etc. The advantage is that with this system, you'd be able dump it as a blob of text for the full ItemStackSnapshot with all data related to it, including enchantments and custom plugin data.
Closing as there's no real disagreement with using the snapshots to do this very utility functionality.


Fix: Unix-like systems correct path formatting

This fixes the baking of lut files on UNIX based operating systems by using the correct path seperator

Also apologies I have just noticed by white space formatter ran. Hope you dont mind. I can omit these changes if you'd rather


Speckle Connector duplicates rather than updates objects.

Really appreciate the great work you are doing!
Expected vs. Actual Behavior
Speckle Connector duplicates rather than updates objects.
Describe the problem here.
This has been described in the community forums and responded to, but it has not been addressed, nor is it mentioned in the documentation.  When a new person starts using speckle, they just think it's broken. Speckle without the ability to update objects is not much better than FBX.
I have also replicated it in the latest 2.19.1 version.
https://speckle.community/t/revit-to-blender-creates-duplicates-in-blender/8083
https://speckle.community/t/rhino-to-blender-not-updating-mesh-but-creates-a-new-one/8689
Reproduction Steps & System Config (win, osx, web, etc.)
https://speckle.community/t/revit-to-blender-creates-duplicates-in-blender/8083
https://speckle.community/t/rhino-to-blender-not-updating-mesh-but-creates-a-new-one/8689
Let us know how we can reproduce this, and attach relevant files (if any).
https://speckle.community/t/revit-to-blender-creates-duplicates-in-blender/8083
https://speckle.community/t/rhino-to-blender-not-updating-mesh-but-creates-a-new-one/8689

Would an update mode behaviour, similar to our Rhino connector fit your needs?
I.e. when you receive a newer version of a model, we simply remove all the old objects, and add the new ones in.
This wouldn't allow users to preserve any transform/UV/material changes on objects, since we would take the incoming geometry as is...

Yes!!  But you may want to create a new local "sync" object, copy the new geometry etc. to real object and then delete the sync object when you are finished.  This would preserve local changes that are not normally synced like custom shaders, geometry nodes, etc...
Even better would be a mechanism to track when an object last changed by computing an MD5-like signature on meaningful values (mesh geometry, materials, whatever else may be synced) and store that somewhere (in each object as a custom property, for instance).  That way when syncing the client can look up and see if the object hasn't changed locally, and automatically update it.  If it has changed the client could then prompt the user to either skip the object, overwrite, archive it, or create a new sync object collection that they can manually copy from to merge changes.
signature = hashlib.md5(b'object data')


C#: System.Random and other dotnet classes throw an exception on iOS

Godot version
v3.5.1.stable.mono.official.6fed1ffa3
System information
iOS 16.3
Issue description
This is similar to previous issues such as:
#38810
#40633
#40757
They were supposedly fixed by:
#49248
However, some of these classes are still not working. This seems to be contained only to the iOS platform when the app is submitted to App Store Connect, aka, not a simulator build. When simply instancing Random rand = new Random(); it throws an exception. This also seems to be connected to why this method- System.Text.Json.JsonSerializer.Serialize(); which comes from the System.Text.Json NuGet package also throws an exception because it gives this stack trace:
System.TypeInitializationException: The type initializer for 'System.Numerics.Hashing.HashHelpers' threw an exception. ---> System.TypeInitializationException: The type initializer for 'System.Random' threw an exception. ---> System.EntryPointNotFoundException: SystemNative_GetNonCryptographicallySecureRandomBytes
Things I've tried:

Using GD.RandRange() works, but only if I set Unhandled Exception Policy to LogError instead of TerminateApplication because it somehow still throws an exception while simultaneously returning the correct data. However, this doesn't solve the System.Text.Json.JsonSerializer() issue because it still seems to rely on System.Random specifically.
I verified that System.Text.Json is compatible with net472 and net48 even though it is normally packaged with dotnet 6.0 and so on.
I used a different source generator that does not use reflection because AOT on iOS prevents execution of dynamic code so the serialization part doesn't seem like the issue related to the System.Text.Json.

So that leads me to believe that it's probably not an issue with System.Text.Json. Perhaps symbols are being stripped when it's uploaded to App Store Connect, and when something that relies on native classes are called, an exception is thrown. Unfortunately, I'm not sure how to resolve something like that.
This symbol issue could be related to these:
#48701
#58454
Steps to reproduce

Create a default mono enabled project.
Use System.Random rand = new Random(); somewhere in a script.
Wrap new Random() in a try catch and set Unhandled Exception Policy to LogError from project settings to more gracefully see issue.
Export for iOS.
Upload to App Store Connect.
Download from TestFlight and run.

Alternatively, use the same steps but just use System.Text.Json.JsonSerializer.Serialize() after adding that as a NuGet package.
Make sure to change source generator from the default to metadata because of AOT restrictions on iOS.
Minimal reproduction project
N/A since described above.

Please see my comment on #58454. We're experiencing something similar, and I didn't want to duplicate what I wrote here, although this issue may be an even closer match.


✨ navigation bars

I've added navigation bars to Trends, Search and Ingredients pages. As for me, it looks miles better now. Hope you like it too!
Also I've changed a bit Ingredients page - now it looks a bit cleaner, as for me

Are we going to continue working on this project?


Add automation to publish releases to github

We want to make sure releases get github releases along with the sdist and wheels. That will also allow us to upload wheels for more platforms if we hit pypi resource usage limits.
@timkpaine shared https://github.com/finos/perspective/blob/aaff331a2ede998aeae28ea61c806766e1b4c453/.github/workflows/build.yml#L1809 as an example to crib off of.

Dixed by #90


Changed average_data to accept observable input in matrix form

Summary
As per discussion in #1322 I have altered the average_data function to accept an observable given as a sorted list.
Details and comments
When passed a list or matrix as an observable, average_data will now call make_dict_observable to convert into dictionary format. The function then continues as before.  This address the feature request in #1322 and has accompanying tests.

I have made the suggested changes to the documentation. I also wonder if make_dict_observable is of any use on its own, in which case it should maybe be moved to its own file?

I am good with this suggestion. Feel free to make it

I think this is ready to be merged now. Please let me know if there are any further changes that need to be made first.


qt5: upgrade to v5.15.13-lts-lgpl

SRCREVs updated to 5.15.13-lgpl
Removed patch on qtbase that has been integrated upstream

Packages successfully built against Yocto Hardknott: qtbase, qtmultimedia, qtwayland. Other recipes are untested for unavailability of machine build time.

Merged, thanks.


npm install fails with missing esprima-six

Hi there, just thought I'd let you know that when running npm install I was getting the following error messages:
npm ERR! Darwin 15.2.0
npm ERR! argv "node" "/usr/local/bin/npm" "install"
npm ERR! node v0.12.7
npm ERR! npm  v2.12.1
npm ERR! code E404

npm ERR! 404 Registry returned 404 for GET on https://registry.npmjs.org/esprima-six
npm ERR! 404
npm ERR! 404 'esprima-six' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404 It was specified as a dependency of 'astw'
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/r/code/js/webgl/webgl-lines/npm-debug.log

Basic on this issue, a solution is to switch glslify version from 1.6.0 to 1.6.1. Afterwards the demos are working great.

Fixed, thanks for the report! :+1:


[BUG] can't select table when tispark write table in TiDB 4.0.1 with useTableLock=false

Describe the bug
https://github.com/pingcap/tispark/blob/master/docs/datasource_api_userguide.md#tidb-options
useTableLock | false | true (3.x) false (4.x) | Whether to lock the table during writing

I set useTableLock to false, but select * from test.t limit 10 still freeze when TiSpark is writing.
What did you do
        Map<String, String> tidbOptions = new HashMap<>();
        tidbOptions.put("tidb.addr", config.getProperty("tidb.addr", "<IP_ADDRESS>").trim());
        tidbOptions.put("tidb.port", config.getProperty("tidb.port", "4000").trim());
        tidbOptions.put("tidb.user", config.getProperty("tidb.user", "root").trim());
        tidbOptions.put("tidb.password", config.getProperty("tidb.password", "").trim());
        tidbOptions.put("replace", "true");
        tidbOptions.put("useTableLock", "false");

             df.write()
                .format("tidb")
                .options(tidbOptions)
                .option("database", "test")
                .option("table", "t)
                .mode(SaveMode.Append)
                .save();

Run select * from test.t limit 10 when the spark job is writing the table (I'm pretty sure it's writing tidb).
What do you expect
The select clause shouldn't freeze.
What happens instead
The select clause does freeze.
Spark and TiSpark version info

TiDB: 5.7.25-TiDB-v4.0.1
TiSpark: 2.3.4
Spark: 2.4.7
JDK: 1.8

how many rows are there in table test.t before writing?

would you please provide tidb's log during The select clause does freeze ?

@Dieken i cannot reproduce this problem. would you please provide the script to reproduce this problem?
it will be nice if you can send your WeChat id to my email<EMAIL_ADDRESS>

The destination table is empty,  about 30w rows take 150s to insert with TiSpark,  8 x 4C/32G spark executors.
Seems our TiKV nodes are overloaded,  after adding one more TiKV node, the insertion above takes about 40s,  the freeze time decreases to 2~10s,  don't know how many seconds it would take ago,  I interrupted the select clause because it's too slow.
Although the freeze time is still too large, it's probably related to busy TiKV node, not table lock, so I close this issue now.

2~10s freeze time meets expectations.
in this scenario, tidb needs to do something extra in order to solve the read-write conflict.


[@azrure/identity] azureCliCredential just because there is content in stderr doesn't mean there's an error

Package Name: @azrure/identity
Package Version: all versions
Operating system: Windows
[x] nodejs

version: 14.17.1

[ ] browser

name/version:

[ ] typescript

version:

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
When using the AzureCliCredential getToken() calls the az cli tool. If there is any output in the stderr prop then it is assumed the command failed. This is not always the case as the stderr output is often used for deprecation warnings or information. For example, with az v2.13.0 the az CLI outputs a warning that the --tenant option is in preview, but the access token is still provided in the stdout.
To Reproduce
Steps to reproduce the behavior:

Install az v2.13.0
Attempt to get an access token via the CLI (which uses the --tenant argument, eg: a secret from a vault
Note that node throws an error Error: Argument '--tenant' is in preview. It may be changed/removed in a future release.

Expected behavior
If an az CLI command succeeds, the application should continue even if there is some output to the stderr
Screenshots
n/a
Additional context
https://github.com/Azure/azure-sdk-for-js/blob/037feadfe9fb35b442bc1403ccefd136dc57e377/sdk/identity/identity/src/credentials/azureCliCredential.ts#L124-L153
Instead of using the stderr prop to determine the branch of logic to proceed down, attempt to parse the stdout first, if this fails, attempt to handle error conditions

This is great feedback, @dhensby ! thank you!
We’ll follow up as soon as possible.


feature Load index data into index cache when writing data

Proposed changes
Issue Number: close #xxx
4.0-preview needs

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall


Code Link

Thanks for you summary. The code link of TSDiff (Predict, Refine, Synthesize: Self-Guiding Diffusion Models for Probabilistic Time Series Forecasting) is https://github.com/amazon-science/unconditional-time-series-diffusion.

Sorry, I have found it right.


BUG: Scrollbar present even from the loading page.

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?

Scrollbar present even from the loading page which is not needed there and it is a bad outlook.
Can u assign me this issue? I am a GSSOC'24 contributor. @anuragverma108
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?

Scrollbar present even from the loading page which is not needed there and it is a bad outlook.
Can u assign me this issue? I am a GSSOC'24 contributor. @anuragverma108
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Assigned!
Make sure to attach before & after screenshots in your PR and include the issue number.
Also, please star the repo before working on it. Happy open-source contributing!


Function to check login status

Is there a function to check if my session is still connected? I tried to login with wrong MFA and still was able to get position and account info etc.. However, when I tried to place an order, I got message Your session has expired please login again.
How long does the one session and trading token expire? Do we need to run get_trading_token every time before placing an order?

That's interesting. It depends on the behavior of Webull's own specification. However, I am strongly convinced that shouldn't be the case. (If the session is expired, you shouldn't be able to do get_account() any more.)
The trading password will expire depending on your setting on your webull account. Settings > Account Security > Trading Password. You should be able to choose 5 minutes, 15 minutes...etc.

That's interesting. It depends on the behavior of Webull's own specification. However, I am strongly convinced that shouldn't be the case. (If the session is expired, you shouldn't be able to do get_account() any more.)
The trading password will expire depending on your setting on your webull account. Settings > Account Security > Trading Password. You should be able to choose 5 minutes, 15 minutes...etc.

same problems here


Bug: X11 and Wayland should be treated mutually exclusive during swapchain creation

https://github.com/NVIDIAGameWorks/NRI/blob/919412b61114f737a7ac984cd909fd31bced3953/Source/VK/SwapChainVK.cpp#L103
This code is broken, if both, X11 and Wayland are available on a system.
NRWindow is a union, so writing to it via
        m_NRIWindow.x11.dpy = glfwGetX11Display();
        m_NRIWindow.x11.window = glfwGetX11Window(m_Window);

Will make the code in SwapChainVK::Create()  think, that we provided both, a X11 and a Wayland window handle.  On NVIDIA drivers this will later lead vkCreateSwapchainKHR() to crash deep in the callstack.
I think, making NRWindow not be a union and instead use #ifdef to control the platform-dependent members would be the right call.

Hi! You are right. Sorry for the lack of testing in this area. I will fix the problem in a few days (have other tasks to do, will try to prioritize).


Development/wpe 616 master

https://jenkins.code.rdkcentral.com/job/github-triggered/374/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/375/ started


chore: redo typo PR by satyambnsal

Thanks satyambnsal for https://github.com/AztecProtocol/aztec-packages/pull/5769. Our policy is to redo typo changes to dissuade metric farming. This is an automated script.

Docs Preview
Hey there! 👋 You can check your preview at https://6626867696715409bd44c2f5--aztec-docs-dev.netlify.app


NSSM wont install due to previous version running

Since upgrading to the latest version of the cookbook, it keeps trying to install a newer version of NSSM and since NSSM is currently running it gets an access denied error trying to overwrite the file failing the chef run.

since why run isn't supported i was able to fix this by modifying the default recipe and adding a notification on the remote_file resource to kill NSSM. i tried in another wrapper cookbook to subscribe to the resource but it did not work.

Can you please submit a PR with the bug fix? Thanks in advance.

its not elegant at all as most would not want to blindly force kill nssm in all cases just like that and potentially cause other issues. im not sure if theres a better solution to it, right now i needed to get it tot work for my use case but i will find some time to make a proper commit with a better solution that covers all use cases. just wanted to document here in case anyone else had ideas.
ideally fixing the whyrun issue in #15 would essentially allow us to kill it ourselves with a notification making this a closed will not resolve issue since fixing whyrun is the best solution.

given that whyrun is now supported and its easier to manage the services and kill them ourselves gracefully to allow for overwrites this issue is now irrelevant


Merge pull request #1 from tensorflow/master

Update on 9 Jan 2019

Closing this PR since no changes in files. Thanks!
cc @mihaimaruseac


feat: expose getHomeAzInfo api in cns to retrieve node home az infos from NMAgent

Reason for Change:

To enable AZR , dnc need to call CNS to get home az infos from NMAgent. This PR is for exposing the getHomeAzInfo api in cns to achieve that.
caching homeaz infos will be covered in separated PR
Issue Fixed:

Requirements:

[x] uses conventional commit messages

[x] includes documentation
[x] adds unit tests

Notes:
task link

do I understand correctly that nothing is currently populating these "cache" values? why not make them pull-through, so that the first attempt to read the cache does the look-up if the value has not been set? I prefer that to having an out-of-band cache put process.

Ya, currently nothing is populated to these caches. Based on the design we agreed on, we will have a thread to query home az from nmagent and cache the value when cns starts. @ashvindeodhar

/azp run

/azp run

@ashvindeodhar please leave a note explaining why this was merged bypassing checks

@rbtr  sorry, I added a note but seems didn't hit the 'comment' button.
Merging this bypassing the windows check which has been failing for last few days. We need this change for making progress on aquarius changes and testing the critical AZR feature in timely fashion. This change is not updating any existing code / functionality so risk of any windows regressions are very low. This code adds new functionality which is currently not used. After checking with pipeline v-team ( Esteban ), I came to know that this might need few more days to fix this windows pipeline issue.


Use generic invalid bundle state

Change this state name to something generic so we can avoid future crd changes

/approve
/lgtm


[FLINK-2148] [contrib] Exact and approximate countDistinct on streams

For the approximate calculation I used the HyperLogLog implementation in the Clearspring library.
Currently it operates only on the entire stream, but when the dust settles around the windowing rewrite, I will modify this code to work on windows instead. The Clearspring implementation has a merge method, which means that it is compatible with windowing aggregation optimizations like panes or B-Int.
I have also added a fromArray convenience method to StreamExecutionEnvironment.

I am wondering about two things, maybe I didnt catch something as I was just scrolling through:
-Why the output of the distinct methods are not DataStream
-Why don't we have a distinct method that does not take a field-position, so counting the true distinct elements in the stream

-Why the output of the distinct methods are not DataStream

They do output DataStream. ( is their generic parameter.)

-Why don't we have a distinct method that does not take a field-position, so counting the true distinct elements in the stream

That's a good idea, I have added it now.

I planned to add a generic there :p why not DataStream[Long]

Oh, I see. :) I'm actually glad that you brought this up, because I was not really sure how should this be done. At first I also thought to just return a DataStream[Long], but then it occured to me that if someone wants to do something with the result that also involves the original data, then it gets kind of awkward to reconnect them. Or would they just do a CoMap in that case? I don't know...
Note, that now the newly added overload that works with the entire record actually does return a DataStream[Long], so the user also has the option to do a project to one field and then call that overload to get a stream of Longs.
By the way, I think one could also ask the same question about WindowedDataStream.sum (and similar methods): why does that does its thing on one field and not just work with the entire records? Could you give some details of that design decision, and how it would or would not apply here?

This is an interesting issue, exactly because of the current API for aggregations. I think the main purpose of returning the aggregated values inside the original data is to use them when we have grouped aggregations. There this makes perfect sense.
Maybe someone knows better but I think we have this as we don't have explicit key-value pairs.

By the way, don't the maps need to have parallelism 1 to function properly?

You are right, I have now set the parallelism to 1 for the maps.
I have also moved the exact countDistinct functions to a new repository (https://github.com/ggevay/flink-statistics-extra), because they potentially use unbounded memory.

I'm closing this. It doesn't really make much sense to do this on an entire stream; users would probably want to do this on large windows instead, so this should probably be reimplemented for the new windowing API.


MINOR - Prepare AI chat feed

Describe your changes:
Required for https://github.com/open-metadata/openmetadata-collate/pull/373

We're adding the ai property to the threads so that we can pass AI generated queries information when chatting with AI

Type of change:

[ ] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

change from ai to chatbot


feat(sinsp/threadinfo): expose thread uid and gid as static fields

What type of PR is this?

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

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

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

/area API-version

/area build

/area CI

/area driver-kmod

/area driver-bpf

/area driver-modern-bpf

/area libscap-engine-bpf

/area libscap-engine-gvisor

/area libscap-engine-kmod

/area libscap-engine-modern-bpf

/area libscap-engine-nodriver

/area libscap-engine-noop

/area libscap-engine-source-plugin

/area libscap-engine-savefile

/area libscap

/area libpman

/area libsinsp

/area tests

/area proposals

Does this PR require a change in the driver versions?

/version driver-API-version-major

/version driver-API-version-minor

/version driver-API-version-patch

/version driver-SCHEMA-version-major

/version driver-SCHEMA-version-minor

/version driver-SCHEMA-version-patch

What this PR does / why we need it:
This PR enables access to thread uid and gid as static fields.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ekoops
Once this PR has been reviewed and has the lgtm label, please assign leogr for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

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

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

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

LGTM label has been added.  Git tree hash: 17218ace4eb86e69fbf7d45fde0d7fdde7012257

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

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

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

/hold

You need to update the corresponding test
2024-12-10T16:27:02.9330936Z [ RUN      ] thread_manager.table_access
2024-12-10T16:27:02.9331451Z /__w/libs/libs/userspace/libsinsp/test/state.ut.cpp:381: Failure
2024-12-10T16:27:02.9331926Z Expected equality of these values:
2024-12-10T16:27:02.9332533Z   newt->static_fields().size()
2024-12-10T16:27:02.9332834Z     Which is: 33
2024-12-10T16:27:02.9333103Z   s_threadinfo_static_fields_count
2024-12-10T16:27:02.9333628Z     Which is: 30
2024-12-10T16:27:02.9333924Z [  FAILED  ] thread_manager.table_access (1 ms)
2024-12-10T16:27:02.9334319Z [ RUN      ] thread_manager.fdtable_access

New changes are detected. LGTM label has been removed.

You need to update the corresponding test
2024-12-10T16:27:02.9330936Z [ RUN      ] thread_manager.table_access
2024-12-10T16:27:02.9331451Z /__w/libs/libs/userspace/libsinsp/test/state.ut.cpp:381: Failure
2024-12-10T16:27:02.9331926Z Expected equality of these values:
2024-12-10T16:27:02.9332533Z   newt->static_fields().size()
2024-12-10T16:27:02.9332834Z     Which is: 33
2024-12-10T16:27:02.9333103Z   s_threadinfo_static_fields_count
2024-12-10T16:27:02.9333628Z     Which is: 30
2024-12-10T16:27:02.9333924Z [  FAILED  ] thread_manager.table_access (1 ms)
2024-12-10T16:27:02.9334319Z [ RUN      ] thread_manager.fdtable_access

Done

LGTM label has been added.  Git tree hash: 19088f5758d15f82a5790a6a7a4ce5ea7f5cb5ba

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

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

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

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

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

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

/unhold


[Feature]: implement Multistore functionality

How the project can be improved?
Similarly to Magento, I have decided to not include the Multistore in the 1.0.0 version but in 1.1.0.
What are the acceptance criteria?

[ ]

Additional information
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Does this mean using Vendure's Channels to create a multi-vendor store? I am looking forward to this feature if so!

@rhclayto
To my knowledge yes. Would you be interested in developing that? :)
@michaelbromley could you please confirm and maybe add some useful info here about how it can be implemented?

Hi all,
Yes, multi-store is implemented in Vendure with Channels. There are some docs about it here: https://www.vendure.io/docs/developer-guide/multi-tenant/
For the storefront, the main thing that needs to be done is to attach a vendure-token header or query string to all requests which is used to specify which Channel to use: https://www.vendure.io/docs/developer-guide/multi-tenant/#the-storefront
Once that is set, you should just be able to use all the exact same queries & mutations, and Vendure will return the inventory etc only for the current Channel.

@rhclayto
To my knowledge yes. Would you be interested in developing that? :)
@michaelbromley could you please confirm and maybe add some useful info here about how it can be implemented?

I will start looking into the code structure & see if I can put a PR together.

Would you prefer using a header or a query string?

I would go for a header then. Lets keep it simple and close to Vendure approach :)

You can technically use either. My opinion is that headers are cleaner.

Just a note: I got something partially working, but it was ugly. All the abstraction in this code made my head hurt! My programming chops, not up to snuff.

Hey @rhclayto
I completely missed your response on this issue. My notifications for closed issues seems to be broken. As multistore is a functionality that can be used in some projects but does not have to work for all. I decided to only show in the docs how it works instead of imolementing it as for some users it may not be useful.
If you need any help implementing that, me or @michaelbromley can probably help 😉


Bazaar does not install

v0.1.0-beta.2
With this version, I have tested locally as well as on shared hosting and Bazaar does not install after the button is clicked. It is missing from the extensions page after the final installation phase.

Hi, thanks for submitting your bug. I am currently on vacation, and I will not be able to dig into the code just yet. I expect to be home on Sunday.
That said. I did not initially manage to reproduce the problem, alas, I decided to try under a different environment, which I believe reproduced it. This is the error I got:
`
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.
Problem 1
- doctrine/inflector v1.2.0 requires php ^7.0 -> your PHP version (5.6.31) does not satisfy that requirement.
- doctrine/inflector v1.2.0 requires php ^7.0 -> your PHP version (5.6.31) does not satisfy that requirement.
- doctrine/inflector v1.2.0 requires php ^7.0 -> your PHP version (5.6.31) does not satisfy that requirement.
- Installation request for doctrine/inflector (locked at v1.2.0) -> satisfiable by doctrine/inflector[v1.2.0].
Installation failed, reverting ./composer.json to its original content
`
@luceos - Since I am not home atm I can't verify, but, is the above only in require-dev?


Cross-file GoTo Definition

#1795

Intellisense also doesn't work across multiple R files. Are these two somehow connected (I assume they both depend on making ASTs of all the files, not just the one currently opened) or should I start a new issue?

We do not currently support cross-file analysis. It is #1364 actually.


Start-BitsTransfer cannot be leveraged in some instances

in Install-UnitySetupInstance
the line
Start-BitsTransfer -Source $downloadSource -Destination $downloadDest
fails if it's executed from a Visual Studio Team Services build agent with as the VSTS Agent runs as a service.
2018-08-03T17:31:03.2273217Z DEBUG:  472+              >>>> Start-BitsTransfer -Source $downloadSource -Destination $downloadDest
2018-08-03T17:31:03.7159554Z Start-BitsTransfer : The operation being requested was not performed because the user has not logged on to the
2018-08-03T17:31:03.7159834Z network. The specified service does not exist. (Exception from HRESULT: 0x800704DD)
2018-08-03T17:31:03.7160174Z At C:\Program Files\WindowsPowerShell\Modules\UnitySetup\4.0.97\UnitySetup.psm1:472 char:13
Either credentials need to be supplied to the call (and piped through as params) or it should use webclient to download the installer
(New-Object System.Net.WebClient).DownloadFile([string]$downloadSource, [string]$downloadDest)
I'm testing this E2E now.

Confirmed that (New-Object System.Net.WebClient).DownloadFile([string]$downloadSource, [string]$downloadDest) passes without issue.  can submit a PR if needed.

I found that Start-BitsTransfer is also not currently available on macOS. I think it would be great if you could get a PR up with WebClient, though I would prefer we keep bits as the default and have WebClient as the fallback. Bits transfers much faster, if available.

Still not available on macOS.


Question regarding Github and OIDC

I am relatively new to creating and managing authentication services and while this service appears to provide exactly the functionality I need, I am finding the examples to be a little lacking since they are almost entirely focused on using Google as a provider.
So, questions:
Question 1: I am planning on using Github as my authorization provider.  I have created the Github application and the associated Client Secret and Client ID.  What I can't seem to find is the correct URL on Github for the ODIC ISSUER URL.  Is this simply https://github/com/?
Question 2:  I assume that the Callback URL that I add in the Github application should be a link to the traefik-forward-auth container and port, is this right?
thanks in advance for answering these question.  Once I get this up and running I'll spend some time adding a few more examples and potentially updates to the docs to help out newer users like myself and submit a pull request.

Hi!
Unfortunately github don't currently support OpenID Connect, however I have (today!) added a patch which adds support for any provider that doesn't support OpenID Connect (#138). This needs a little more work, but is the last item pegged for v2.2 which I hope to release in the next couple of weeks.
I'll do some testing with github this week and add the setup docs to the new Provider Setup wiki page: https://github.com/thomseddon/traefik-forward-auth/wiki/Provider-Setup
I'll keep this issue open and ping you when this is ready to go.

wow.  thank you for the very swift response.  I think I was thrown by the following line in the readme which references GitHub.
Supports multiple providers including Google and OpenID Connect (supported by Azure, Github, Salesforce etc.)
I look forward to the release of #138 and definitely let me know if there's anything I can do to help....

Yes, I erroneously thought Github supported OpenID Connect 🙈
I've just cut the v2.2 release, which incldues the generic provider support - I've tested it with Github this afternoon and it works well (apart from the slight quirk that github won't return the users email unless it is "public"). I've documented the setup here: https://github.com/thomseddon/traefik-forward-auth/wiki/Provider-Setup#github
Let me know how it goes and please re-open or create a new issue if you have any problems!

I'm having issues with this, or maybe I don't fully understand the documentation? Here's what I configured as environment variables in my docker-compose.yml file:
    environment:
      - PROVIDERS_GENERIC_OAUTH_CLIENT_ID=$GITHUB_CLIENT_ID
      - PROVIDERS_GENERIC_OAUTH_CLIENT_SECRET=$GITHUB_CLIENT_SECRET
      - PROVIDERS_GENERIC_OAUTH_USER_URL=https://github.com/login/oauth/authorize
      - PROVIDERS_GENERIC_OAUTH_AUTH_URL=https://api.github.com/user
      - PROVIDERS_GENERIC_OAUTH_TOKEN_URL=https://github.com/login/oauth/access_token
      - SECRET=$OAUTH2_PROXY_COOKIE_SECRET

It seems like I have the auth_url value incorrect (I suspect I've taken the documentation too literally...), because when I try to navigate to a page that requires OAuth I only see this:
{
  "message": "Requires authentication",
  "documentation_url": "https://docs.github.com/rest/reference/users#get-the-authenticated-user"
}

I was very tired when I made that post, and can now easily see I mixed up my values.
    environment:
      - PROVIDERS_GENERIC_OAUTH_CLIENT_ID=$GITHUB_CLIENT_ID
      - PROVIDERS_GENERIC_OAUTH_CLIENT_SECRET=$GITHUB_CLIENT_SECRET
      - PROVIDERS_GENERIC_OAUTH_USER_URL=https://api.github.com/user
      - PROVIDERS_GENERIC_OAUTH_AUTH_URL=https://github.com/login/oauth/authorize
      - PROVIDERS_GENERIC_OAUTH_TOKEN_URL=https://github.com/login/oauth/access_token
      - SECRET=$OAUTH2_PROXY_COOKIE_SECRET

Now that I have them matched up properly it appears to be working fine! :laughing:


horizontal scrolling not working in safari 6?

Having an issue in safari 6 on os x mountain lion. For example, when trying to issue a scrollTo() command in a 2000/2000px html body, only the vertical component is scrolled. Is anyone else experiencing this? It works find in safari 5, ie9, firefox and chrome.
$.scrollTo({left:1000, top:1000}, 1000);

Hi,
On #101 we're testing a new beta version (currently 1.5.2-beta) that fixes scrolling in Chrome 40 (was working on most ends but not all), also works correctly on iOS and Android which wasn't previously working. I tested this version with my Safari 8.0 and it works.
I'd like you to join us on that other issue and run the tests on your available browsers (including Safari 6), I won't be releasing this until it gets enough confirmation from users.
Thank you

The new version looks good

You tested it on Safari 6?

I merged 2.0.0 to master via #109 and this issue might get fixed (indirectly).
If the problem persists please reopen the issue with details on the situation.
If you have any issue after the update, first check this link, If your problem is not solved then go ahead and report the issue.


[Storage] Fix duplicate sample tag in blob_samples_authentication.py

As raised in #26900 , this PR fixes the duplicate sample tag that appears and may cause confusion if customers refer to a sample case simply by the sample tag. Additionally, this adds the missing sample to the async sample file and some other small changes to unify the samples to match more closely.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-storage-blob


feat: create email verification

Implements email verification based on default implementation with laravel/ui, I've added a setting to enable/disable the requirement of verification defaulting to disabled.
Maybe there is a need to add the verified middleware to the api routes too.
Close #893

Epic!

Added the update on migration, defaulted to enabled and now when the first user is created, it will be verified.


initial attempt at wrapping adbc for database/sql

Here's my first shot at creating the wrapper. After this I'll look into actually creating an implementation to write tests with.
However, I did come across one thing: There are Commit and Rollback methods for a connection, but there's nothing to actually begin a transaction. Nor anything specific for defining transaction options such as Isolation level or Read-Only etc.
For Go's sql package there needs to be a BeginTx method on a connection which also shouldn't require creating a new connection (ie: start a new connection with autocommit turned off?) So i'm not sure of the best way to implement this in the current framework we have.

See https://github.com/apache/arrow-adbc/blob/1ca2d2df1e6daffcc9484aac88fd318254ab36fd/c/validation/adbc_validation.c#L268-L279
Disabling autocommit begins a transaction; commit/rollback also begin a transaction; enabling autocommit also runs a commit.

Gotcha, so the interfaces should allow setting options on an existing connection, not only setting options before initializing it. I'll make the adjustment.

@lidavidm Do we have any constants for setting the Isolation level of the transactions yet? If not, the golang values are:
LevelDefault
LevelReadUncommitted
LevelReadCommitted
LeveWriteCommitted
LevelRepeatableRead
LevelSnapshot
LevelSerializable
LevelLinearizable
With the documentation that if a driver does not support a given isolation level, an error may be returned. This is in addition to a ReadOnly boolean option. Should we add constants for these?

We don't, it probably makes sense to define them


Distinguish sounds with no soundTrackId from those in soundTrack 0.

Add a sound to sound track #0 and then put into a different track did not unregister it from sound track #0. This resulted in an infinite loop when the tracks were eventually disposed (since this.soundCollection.length never dropped to 0).
There are no existing SoundTrack unit tests. I can add soundTrack.test.ts and add a regression test just for this case if you feel like it's worthwhile.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14324/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14324/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14324/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14324/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14324/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14324/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14324/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14324/merge#BCU1XR#0

thanks so much for catching that!


Fix flags width in uber-trace-id header.

See https://github.com/jaegertracing/jaeger-client-go/issues/541.
Signed-off-by: Pavel Patrin<EMAIL_ADDRESS>

I don't think this change is necessary, see https://github.com/jaegertracing/jaeger-client-go/issues/541#issuecomment-704995964


fix(chat): fix custom auth pages initialization (Issue #2666)

Description:

fix custom auth pages initialization

Issues:

Issue #2666

Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review

/deploy-review

/deploy-review


Resuming training fails

So I removed the flag --overwrite_output_dir to be able to resume the training, and I'm getting the following error:
04/01/2024 00:30:01 - INFO - __main__ - max_steps is given, it will override any value given in num_train_epochs
04/01/2024 00:30:04 - INFO - __main__ - ***** Running training *****
04/01/2024 00:30:04 - INFO - __main__ -   Num examples = 4800000
04/01/2024 00:30:04 - INFO - __main__ -   Instantaneous batch size per device = 8
04/01/2024 00:30:04 - INFO - __main__ -   Gradient accumulation steps = 1
04/01/2024 00:30:04 - INFO - __main__ -   Total train batch size (w. parallel & distributed) = 8
04/01/2024 00:30:04 - INFO - __main__ -   Total optimization steps = 600000
Train steps ... :   0%|                                                                               | 0/600000 [00:00<?, ?it/s]04/01/2024 00:30:04 - INFO - accelerate.accelerator - Loading states from ./checkpoint-5000-epoch-0
Traceback (most recent call last):
  File "/home/vlad/distil-whisper/training/run_distillation.py", line 1682, in <module>
    main()
  File "/home/vlad/distil-whisper/training/run_distillation.py", line 1484, in main
    accelerator.load_state(checkpoint)
  File "/home/vlad/distil-whisper/.venv/lib/python3.10/site-packages/accelerate/accelerator.py", line 2966, in load_state
    load_accelerator_state(
  File "/home/vlad/distil-whisper/.venv/lib/python3.10/site-packages/accelerate/checkpointing.py", line 205, in load_accelerator_state
    models[i].load_state_dict(state_dict, **load_model_func_kwargs)
  File "/home/vlad/distil-whisper/.venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 2153, in load_state_dict
    raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
RuntimeError: Error(s) in loading state_dict for WhisperForConditionalGeneration:
        Missing key(s) in state_dict: "proj_out.weight".

At the same time, evaluation script works just fine with the same checkpoint.
I'm using Ubuntu 22, rtx 3090 ti.

I've also observed this in the log:
04/01/2024 00:35:47 - WARNING - accelerate.utils.other - Removed shared tensor {'proj_out.weight'} while saving. This should be OK, but check by verifying that you don't receive any warning while reloading

any updates on this ? I'm facing the same problem

Here's a temporary fix according to https://huggingface.co/docs/safetensors/torch_shared_tensors
Modify load_accelerator_state(): https://github.com/huggingface/accelerate/blob/main/src/accelerate/checkpointing.py#L153
-from safetensors.torch import load_file
+from safetensors.torch import load_file, load_model
...
    if input_model_file.exists():
-       state_dict = load_file(input_model_file, device=str(map_location))
+       load_model(models[i], input_model_file, device=str(map_location), **load_model_func_kwargs)
    else:
        # Load with torch
        input_model_file = input_dir.joinpath(f"{MODEL_NAME}{ending}.bin")
        state_dict = torch.load(input_model_file, map_location=map_location)
-   models[i].load_state_dict(state_dict, **load_model_func_kwargs)
+       models[i].load_state_dict(state_dict, **load_model_func_kwargs)