Usage with immer.js

First of all thank you for this really nice to use, innovative library :).
This issue is mainly for reference and to document our hacks to make proxy-memoize work with immer.
Over at audapolis we wanted to use proxy-memoize to write selectors that compute derived data from our redux-state. We also want to use these selectors inside reducers that get an immer-draft.
Currently, this breaks with upstream proxy-memoize for the following reason:
import { produce } from 'immer';
import memoize from 'proxy-memoize';
test('upstream proxy-memoize breaks with immer', () => {
  const state = { someNumber: 0 };

  const memoizedSelector = memoize((draft: typeof state) => {
    return draft.someNumber;
  });

  produce(state, (state) => {
    memoizedSelector(state);
  });

  expect(() => {
    memoizedSelector(state);
  }).toThrow("Cannot perform 'get' on a proxy that has been revoked");
});

proxy-memoize stores the draft when executing memoizedSelector. At the end of produce immer revokes this draft. Later, proxy-memoize tries to compare the revoked draft with the next value, which throws the Cannot perform 'get' on a proxy that has been revoked error.
To work around this, we added the following snippet to the memoize function:
[...]

+import { isDraft, original } from 'immer';

const memoize = <Obj extends object, Result>(
  fn: (obj: Obj) => Result,
  options?: { size?: number }
): ((obj: Obj) => Result) => {
  const size = options?.size ?? 1;
  const memoList: {
    [OBJ_PROPERTY]: Obj;
    [RESULT_PROPERTY]: Result;
    [AFFECTED_PROPERTY]: Affected;
  }[] = [];
  const resultCache = new WeakMap<
    Obj,
    {
      [RESULT_PROPERTY]: Result;
      [AFFECTED_PROPERTY]: Affected;
    }
  >();
  const proxyCache = new WeakMap();
  const memoizedFn = (obj: Obj) => {
+    if (isDraft(obj)) {
+      const orig = original(obj);
+      if (orig !== undefined) {
+        obj = orig;
+      }
+    }
    const origObj = getUntracked(obj);
    const cacheKey = origObj || obj;
    const cache = resultCache.get(cacheKey);

[...]

Sadly, this hack is rather specific to immer and adds a dependency to it. Thus, it seems that this is not really a viable option for upstream proxy-memoize. Feel free to just close this issue and just keep it as a reference for other people with similar questions :).

Hi, thanks for using the library.
As you guess, we don't want to change the library for this usage. (I'm not even 100% sure if that hack works stably.)
Can you do something like this?
import { produce, isDraft, original } from 'immer';
import memoize from '../src/index';

describe('issue #37', () => {
  it('works with immer', () => {
    const state = { someNumber: 0 };

    const memoizedSelectorOrig = memoize((s: typeof state) => {
      return s.someNumber;
    });
    const memoizedSelector = (draft: typeof state) =>
      memoizedSelectorOrig((isDraft(draft) && original(draft)) || draft);

    produce(state, (draft) => {
      memoizedSelector(draft);
    });

    expect(() => {
      memoizedSelector(state);
    }).not.toThrow("Cannot perform 'get' on a proxy that has been revoked");
  });
});

That would probably also work :). I am closing this now as there is nothing to be done about this here apart from this issue for documenting the usage with immer for other people :)

fwiw RTK has a draft-safe version of reselect's createSelector, which is fairly easy to replicate with memoize:
/**
 * "Draft-Safe" version of `proxy-memoize`'s `memoize`:
 * If an `immer`-drafted object is passed into the resulting selector's first argument,
 * the selector will act on the current draft value, instead of returning a cached value
 * that might be possibly outdated if the draft has been modified since.
 */
export const draftSafeMemoize: typeof memoize = (...args) => {
  const selector = (memoize as any)(...args);
  const wrappedSelector = (obj: unknown) => selector(isDraft(obj) ? current(obj) : obj);
  return wrappedSelector as any;
};

/**
 * "Draft-Safe" version of `proxy-memoize`'s `memoizeWithArgs`:
 * If an `immer`-drafted object is passed into the resulting selector's first argument,
 * the selector will act on the current draft value, instead of returning a cached value
 * that might be possibly outdated if the draft has been modified since.
 */
export const draftSafeMemoizeWithArgs: typeof memoizeWithArgs = (...args) => {
  const selector = (memoizeWithArgs as any)(...args);
  const wrappedSelector = (obj: unknown, ...rest: unknown[]) => selector(isDraft(obj) ? current(obj) : obj, ...rest);
  return wrappedSelector as any;
};

it's worth noting that this intentionally forces a recalc when passed a draft (current will return a new object with any changes reflected in it), to avoid the selector using stale data.


refactor(set-field hooks): Handle 'as' field startsWith data and data type array

To handle the use case where multi:true, create method, the data is in array format. E.g
Inside hook
before: {
  all: [],
  find: [],
  get: [],
  create: [setField({ from: "params.user.id", as: "data.user_id" })]
  update:[],
  patch: [],
  remove: []
}

When POST array of data, currently the merge result will become
[
  { key: data, key2: data },
  { sample: data, sample2: data },
  user_id: "sample_id"
]

Expected result:
[
  { key: data, key2: data, user_id: "sample_id" },
  { sample: data, sample2: data, user_id: "sample_id" }
]

Thanks for the quick reply! I agree with you, setData is more straightforward and declarative. Maybe we can make a note at the setField documentation, e.g when dealing in multi:true service , the result may not work as expected, we will recommend check on setData  ......
By the way, I will close this pull request. Because my code didn't cover nested arrays and not handling well in the as field format.

Very good point to add it to the docs. Would you be up to add a PR for that? Any help is highly appreciated.

Ok, sure. But, the setData not yet included in this library. I will write a note about handling multi:true may not work as expected.


No support for Xiaomi scales and Mi Band?

Hello!
No support for Xiaomi scales and Mi Band?
On older versions, experimentally, you added this.
It worked the same in the https://github.com/AlexxIT/gw3.

You right. It was supported earlier with gw3 app. But this requires a huge effort and support for each of the Multimode Gateway 1 firmware.
Today's version of openmiio_agent works on all firmware versions of three gateway models at once. But it does not have as cool BLE support as gw3 had.
I guess with all my projects, I just don't have time to follow the way of gw3


Cache ziplineFunctions

ziplineFunctions fetches all of the serializers necessary, but currently it can be called a lot, meaning we end up doing a lot of work repeatedly. This PR caches the result of ziplineFunctions, enabling us to only do the work once.

This is wonderful


roles is undefined

This document is missing the roles property of the userSettings object. This is useful to show the combination of both the ID and Name of the current user's security roles. This will impact performance and maintainability by reducing calls back to the API to determine name for roles.

@aliyoussefi Can you point us to the docs that you're talking about.

Hello, https://docs.microsoft.com/powerapps/developer/component-framework/reference/usersettings references securityRoles property. If this is for a different docs article, please reopen or submit a docs issue on the respective page.


Implementation of morphological transformation techniques

Implement Morphological Transformations for images like
1.Erosion
2.Dilation
3.Opening
4.Closing

Implement Morphological Transformations for images like
1.Erosion
2.Dilation
3.Opening
4.Closing

I am GSSoC'21 participant and I want to work on this issue.
Kindly assign this to me.

Go ahead @ainy-123.
PLease make specific folders for each one of them.

Go ahead @ainy-123.
PLease make specific folders for each one of them.

thanks for assigning.


增加多卡部署

1.增加用于多卡部署的文件，multi_gpu_model.h
2.修改demo支持多卡多batch
3.修复det_model中的use_cpu_nms问题 + 删除多余代码

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: heliqi:x: paddle-cipaddle-ci seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Flicker on initial page load, first time

Do you want to request a feature or report a bug?
bug
What is the current behaviour?
there is a flicker on low end devices for initial page load
If the current behaviour is a bug, please provide the steps to reproduce.
on a machine with low end cpu, simulate 6x slowdown in chrome
clear all data, unregister service worker
visit https://clidemo.preactjs.com/
what happens is the following -

html and css load and you see SSR home component
bundle.js loads and it clears the screen and starts async load of home chunk
home chunk loads and preact renders the home component again

What is the expected behaviour?
component for the path requested by browser should be in bundle.js, all others can be async
If this is a feature request, what is motivation or use case for changing the behaviour?
Please mention other relevant information.

Node version
npm version
Operating system
CLI version
Browser

what version of preact-cli are you on?

I guess the demo site was never re-deployed with latest changes.
https://material.preactjs.com/ does not have this problem.
This was fixed as a part of: https://github.com/developit/preact-cli/issues/274

thanks, looks like the exact same issue. let me try that link

i tried https://material.preactjs.com/component/chips/, and it works better but in this case the menu icon doesnt show up until route chunk has loaded, is that due to the same reason as above?


Rewrite the Q and V updates equation (in backward pass) in square-root form

In GitLab by @cmastalli on Apr 16, 2019, 17:56

I will close this issue as I don't think anyone is interesting in this feature. It is also not clear if this will be useful

I will close this issue as I don't think anyone is interesting in this feature. It is also not clear if this will be useful


{Compute} Fix tests, test_vmss_disk_encryption_e2e, test_image_build_managed_image, test_image_build_shared_image

Description

{Compute} Fix tests, test_vmss_disk_encryption_e2e, test_image_build_managed_image, test_image_build_shared_image
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

/azp run

Compute

/azp run

Loading AKS extension fails.


Improved arduino analog voltage conversion

Hello,
testing the arduino analog with multiple analog channels I've noticed that read() [voltage conversion read] decreases its sampling frequency for multiple channels or increasing samples.
Looking in the code I've noticed that a floating point division is V_REF / 65536 is repeated for every read of every channel.
Moving it to the main before the mailbox command loop extensively increased the conversion performances.

done

@LucaCerina  Thanks for the changes you made. I just noticed that the binary file (arduino_analog.bin) file is not updated. Is it possible to update this file as well, so this pull request will be a complete one?

done

@LucaCerina  I guess this pull request will be redundant because of #234. So I will close this and only review #234 . Let me know if there is any problem.


s2u-46 5.3.1: Spanish translation 01

Good job so far @NuriaQS !
Do you think this is complete now?


SQLSTATE[HY000] [2002] php_network_getaddresses: getaddrinfo for mysql failed: Temporary failure in name resolution

Sail Version
1.29
Laravel Version
11.1.1
PHP Version
8.3.4
Operating System
Windows (WSL)
OS Version
Ubuntu 22.04
Description
On a fresh Laravel 11 installation, the main page is not reachable showing the following error:

It coulb be a bug, the reasolution I found un an old post is to change the .env file adding this line:
DB_HOST=host.docker.internal
I changed the following things in the docker-compose.yml before the sail build command:
service name form laravel.locale to project-name.locale
laravel service port from 80 to 8280
mysql service name from mysql to projectname-mysql*
Steps To Reproduce

Install a fresh Laravel project with the following command:

composer create-project laravel/laravel:^11.0 project-name

Install sail inside project-name directory using the following commands:

composer require laravel/sail --dev

php artisan sail:install

sail up -d

Using your exact steps, together with sail artisan migrate, I get a running Laravel app. Most likely this was a temp hiccup.


Sustain active/last tab when navigating between modules

Issue:
When a user opens multiple tabs and is actively working in one tab -> upon navigation to another module or option from the left-hand menu, the focus automatically goes to the tab that was opened last even when it might not be the one that the user was working in
Scenarios to test:

When user opens multiple tabs, and navigates away to another module then the user should be on the last tab/Configuration opened
When a user opens multiple tabs, clicks on any one of them, and navigates to a different module, then the focus should be retained on the tab that the user was on.

Bug resolved


Add 2.6.0 release notes

Description
Add 2.6.0 release notes
Issues Resolved
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

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

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

Test failures are unrelated to this change.


reset options not working well

Hello,
I've using GitLens these days, and I noticed that the reset --soft option is working just as reset, so the staged files are not kept.
I know that the source control extension that comes with vscode has an undo option that works really well: it doest a git reset --soft HEAD~ and keep the commit message

Fixed in GitLens 11


where could i found the password for params "MYSQL_ROOT_PASSWORD" and "MYSQL_PASSWORD"

where could i found the password for params "MYSQL_ROOT_PASSWORD" and "MYSQL_PASSWORD"
i checked the instruction doc, but found nothing.

also i found "CHEVERETO_DB_PASS=user_database_password" in the Dockerfile of repo.
but when i wanna backup the database, i can't found the password


at should throw if the key is missing

ask {
    const marta = { firstName: 'Marta' }
    marta:at('firstName3')
}

returns null (empty) but an error was expected (playground with AskVM v. 1.2.0)

@czerwinskilukasz1 what's your opinion about this?

I'd really like not to have two empty values like in JavaScript

I'm fine with throwing as soon as we have hasKey: https://github.com/xFAANG/askql/issues/400

hasKey is implemented, this is unblocked now.

Place to fix that in code: https://github.com/xFAANG/askql/blob/master/src/askvm/resources/core/at.ts

I have created a pull request with a unit test for this case. I will appreciate if someone could verify if the test is correct 😄
Next, I will work on changing the implementation of at to work as expected by the test

My pull request is ready for review 😃


ADT 3.0 - Rename sales diagnostics topic in markdown file

@mariusw Is there still something pending here, or can this issue be closed?


Open Buffer Command Opens Two Snipe Buffers

In db1b2af "fix indent" you didn't remove the original bad indent line resulting in two calls to opening the buffer. This completely breaks functionality as it will open the target buffer in the other snipe buffer once you select one.
vim.keymap.set({'n', 'x'}, 'go', function ()
    require("snipe").open_buffer_menu()
    vim.api.nvim_feedkeys(vim.api.nvim_replace_termcodes('<C-w>q', true, false, true), 'n', false)
end)

As a temp fix for anyone else having this problem.

ohh shit, that is embarrassing. Could someone open a pr Ill merge it. Not at my pc for a while.

tiny fix just did it on my phone :)


Use java.io.tmpdir in KafkaStarterUtils

Shall we use temp directory or use current directory with better structure?
I remember @kishoreg mentioned some issue users facing about the temp dir?

Shall we use temp directory or use current directory with better structure?
I remember @kishoreg mentioned some issue users facing about the temp dir?

What kind of issues? This code is appending a random suffix, so there shouldn't be an overlap


RPC Multi-Version

Insert RPC request header into ThreadLocal RpcInvokeContext for later per-thread processing.
See issue: https://github.com/sofastack/sofa-rpc/issues/828

描述一下问题背景

建议拷贝一份，否则后面会影响这个map，只允许读

A read only shallow clone of the request header is done.

Codecov Report

Merging #827 into master will decrease coverage by 0.1%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #827      +/-   ##
============================================
- Coverage     71.92%   71.81%   -0.11%
+ Complexity     1092     1089       -3
============================================
  Files           376      376
  Lines         15418    15420       +2
  Branches       2474     2474
============================================
- Hits          11089    11074      -15
- Misses         3050     3063      +13
- Partials       1279     1283       +4

Impacted Files
Coverage Δ
Complexity Δ

.../java/com/alipay/sofa/rpc/common/RpcConstants.java
100% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ipay/sofa/rpc/codec/bolt/SofaRpcSerialization.java
74.74% <100%> (+0.25%)
24 <0> (ø)
:arrow_down:

...m/alipay/sofa/rpc/module/FaultToleranceModule.java
63.63% <0%> (-31.82%)
4% <0%> (-1%)

...pay/sofa/rpc/transport/ClientTransportFactory.java
72.3% <0%> (-4.62%)
0% <0%> (ø)

...va/com/alipay/sofa/rpc/server/bolt/BoltServer.java
74.48% <0%> (-1.03%)
24% <0%> (-1%)

...ipay/sofa/rpc/server/bolt/BoltServerProcessor.java
64.17% <0%> (-0.75%)
23% <0%> (-1%)

...va/com/alipay/sofa/rpc/client/AbstractCluster.java
67.47% <0%> (-0.61%)
0% <0%> (ø)

...alipay/sofa/rpc/registry/zk/ZookeeperRegistry.java
62.65% <0%> (-0.61%)
0% <0%> (ø)

...ay/sofa/rpc/client/AllConnectConnectionHolder.java
62.59% <0%> (-0.27%)
0% <0%> (ø)

...pay/sofa/rpc/message/bolt/BoltInvokerCallback.java
84.48% <0%> (ø)
8% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


WithMultiThreadedApplicationMessageInterceptor always performs publish

Describe the bug
A clear and concise description of what the bug is.
Which project is your bug related to?

Server

To Reproduce
Steps to reproduce the behavior:

Using this version of MQTTnet 'latest'.
Run this code 'WithMultiThreadedApplicationMessageInterceptor(v => v.AllowPublish = false)'.
With these arguments '....'.
The broker always publish the call

Expected behavior
broker should not publish the message if specified
note: WithApplicationMessageInterceptor - this works as expected
Screenshots
see added png please

I don' think that this will work because the multi threaded interceptor will just start a new task for every message. So the server will continue with the unaltered event arguments and thus ignore everything. If you want to change the behavior of the server you must use the other interceptor. But you can start a new task on your own including your custom code which can run on another task.

Closing because I assume that the question is answered.


zig: provide stdenv from argument

Description of changes
Using darwin.apple_sdk_11_0.callPackage or overriding stdenv to darwin.apple_sdk_11_0.stdenv enables building zig on macOS with now merged #176661.
For example (flake.nix):
{
  inputs = {
    nixpkgs.url = "github:NixOS/nixpkgs/nixpkgs-unstable";
    flake-utils.url = "github:numtide/flake-utils";
    flake-compat = {
      url = "github:edolstra/flake-compat";
      flake = false;
    };
  };

  outputs = {nixpkgs, flake-utils, ...}: flake-utils.lib.eachDefaultSystem (system:
    let
      pkgs = nixpkgs.legacyPackages.${system};
      callPackage = if pkgs.stdenv.isDarwin then pkgs.darwin.apple_sdk_11_0.callPackage else pkgs.callPackage;
    in
      {
         devShells.default = pkgs.mkShell {
           packages = [
             (callPackage <nixpkgs/pkgs/development/compilers/zig> { llvmPackages = pkgs.llvmPackages_13; })
           ];
         };
      }
  );
}

I'm not sure it's okay to mark it as not broken because just installing it with nix-env is still failing. The users must modify the stdenv to build it.
And I don't understand why @AndersonTorres moved stdenv from argument to let-in expression. Is this for ensuring LLVM13? I don't know why the current package definition runs without specifying LLVM versions.
Anyway I'm a newbie for packaging nix. Please let me know of better implementation, or I'll just close the PR so please continue it if you have better idea.
Things done

Built on platform(s)

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

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I'm not sure it's okay to mark it as not broken because just installing it with nix-env is still failing.

Well, if it is still broken even after your fixup, you should keep it broken, I believe.

The users must modify the stdenv to build it. And I don't understand why @AndersonTorres moved stdenv from argument to let-in expression.

Redundancy.
Zig is built via llvm/clang, and including an stdenv when the only thing stdenv was being used was to set meta.broken looks awkward to me. I asked if there was any subtle reason to do this, and obtaining a negative answer, I proposed to change stdenv to the llvmPackages one.

Is this for ensuring LLVM13? I don't know why the current package definition runs without specifying LLVM versions.

Usually a specific version or flavor of an input is specified in all-packages.nix.

Nonetheless, your package is still failing on Ofborg Darwin.

@AndersonTorres Thank you for your clarification! I understood clearly. It really helped.
I got used to the Nix packaging system better than the day I opened this PR. I'll try to fix it to remove broken flag.


Can we have fluent API for new Requests

So the code can actually look like,
// get a connection object
RestClient::Connection* conn = new RestClient::Connection("http://url.com");
// configure basic auth
conn->SetBasicAuth("WarMachine68", "WARMACHINEROX")->SetTimeout(5)->SetUserAgent("foo/cool")->FollowRedirects(true);

what's the advantage of changing the API? I'm not opposed to it if this would improve the project. However in my experience fluid APIs are more likely to make the code more complicated, obscure failure and return paths, and add overhead in debugging.
I'm closing this since the issue is fairly old already and hasn't seen any activity. But feel free to reopen if you're still interested in having a discussion about it.


Transfer Transaction

BigchainDB version: 1.0.0
Operating System: ubuntu 16.04
Deployment Type: 3 istances of BCDB each one using mongoDb replica
BigchainDB driver: java

Description
e00ecb037536274e443995238874285e2d0aafff15c92f90bf4ae86f8466a657: id of a valid CREATE transaction
I'm sending the following ( yet to be fullfilled Json transfer transaction )
{
  "asset": {
    "id": "e00ecb037536274e443995238874285e2d0aafff15c92f90bf4ae86f8466a657"
  },
  "id": "cad8fd2532395b97bd63caec270a1fd94596ea2f3cb9ab46793433fa36863a0d",
  "inputs": [
    {
      "fulfillment": null,
      "fulfills": {
        "output_index": "0",
        "transaction_id": "e00ecb037536274e443995238874285e2d0aafff15c92f90bf4ae86f8466a657"
      },
      "owners_before": [
        "BPozbKVyzNHwMW3dVBR4atfcQyLYBw7cDEyD7mUWSuwU"
      ]
    }
  ],
  "metadata": {},
  "operation": "TRANSFER",
  "outputs": [
    {
      "amount": "98",
      "condition": {
        "details": {
          "public_key": "BPozbKVyzNHwMW3dVBR4atfcQyLYBw7cDEyD7mUWSuwU",
          "type": "ed25519-sha-256"
        },
        "uri": "ni:///sha-256;Wn7JBKb_lox7bMmnnlc-ioHsGWgvYF-CP59ugsKk4-M?fpt=ed25519-sha-256&cost=131072"
      },
      "public_keys": [
        "BPozbKVyzNHwMW3dVBR4atfcQyLYBw7cDEyD7mUWSuwU"
      ]
    },
    {
      "amount": "2",
      "condition": {
        "details": {
          "public_key": "GGzFxrMqighPNMtysyB2D34NNKPfKYVAijTQwusKDK6Y",
          "type": "ed25519-sha-256"
        },
        "uri": "ni:///sha-256;GfO9lqW8upjz0pyBrHxm1dVGCB472ZB4zXWTaFX0kAM?fpt=ed25519-sha-256&cost=131072"
      },
      "public_keys": [
        "GGzFxrMqighPNMtysyB2D34NNKPfKYVAijTQwusKDK6Y"
      ]
    }
  ],
  "version": "1.0"
}

and obtain after send:
{
  "message": "Invalid transaction schema: {'output_index': '0', 'transaction_id': 'e00ecb037536274e443995238874285e2d0aafff15c92f90bf4ae86f8466a657'} is not valid under any of the given schemas",
  "status": 400
}

BAD REQUEST
I ask if the Json transaction has somethink wrong, it is my construction so it is possible.
The transfer transaction has as input a CREATE transaction with amount 100, and 2 output with amount 98 and 2 to two different subjects, myself and other. It's only a test btw.
Many thanks, Stefano

'output_index': '0'  should be int


fix(test): drop subscription when test completes

Problem
This pull request mitigates https://github.com/neondatabase/neon/issues/6969, but the longer-term problem is that we cannot properly stop Postgres if there is a subscription.
Summary of changes
Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

It's strange that test_logical_replication doesn't have this issue

It's strange that test_logical_replication doesn't have this issue

test_logical_replication use Vanilla Postgres as subscriber

I have no idea, and we still need to investigate the root cause in the future 😅


[WIP] fix left string

Why I'm doing:
What I'm doing:
Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[x] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.3
[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.5 out of 6 committers have signed the CLA.:white_check_mark: zombee0:white_check_mark: silverbullet233:white_check_mark: stephen-shelby:white_check_mark: decster:white_check_mark: srlch:x: mergify[bot]You have signed the CLA already but the status is still pending? Let us recheck it.


Adding left, mid, right button support

Still needs wheel support

Could you separate this PR into two parts, one fix the button issue and one the other code? And please fix the conflict.

Could you separate this PR into two parts, one fix the button issue and one the other code? And please fix the conflict.


[Feature] Spike triggered population response (stPR)

Feature
Implementing the spike triggered population response (stPR) in elephant. This is a measure of spiketrains, given a list of simultaneously recorded spiketrains it is the measure of the spike triggered response from one spiketrain against all other spiketrains in the list. It is a sort of proxy for how locked to the population activity every individual spiketrain is. It was (as far as I know) presented by Okun et al 2015.
Motivation
I want to use this metric in one of my projects, since it is a single unit property it works ideally with the rest of my analysis.
Pitch
I want to implement stPR in elephant as long as this does not already exist.
Alternatives
Someone else might have implemented similar things that are not yet in elephant.
Additional context
Okun, M., Steinmetz, N., Cossell, L. et al. Diverse coupling of neurons to populations in sensory cortex. Nature 521, 511–515 (2015). https://doi.org/10.1038/nature14273
In the paper they describe how to calculate the stPR (also something they call stLFP, which is similar by using one spiketrain and one LFP signal):

Population rate (for example, in Fig. 1c) was computed by accumulating all the detected spikes (both well-isolated units and multi-unit activity) with 1 ms resolution, and smoothing the resulting vector with a Gaussian of half-width 12 ms. The population rate used to compute the stPR for any individual unit did not include the spikes of that unit. The baseline level of each stPR (which reflects the mean population rate) was subtracted.
For stLFP computations, raw extracellular signals were first digitally band-pass filtered offline between 0.1 and 200 Hz to isolate the LFP. For units recorded on a particular shank, the LFP was taken from an adjacent shank (200 μm away), to avoid contaminating the stLFP by the spike waveform itself. The size of stLFP was taken as the ordinate value of the negative peak of the cross-correlation in a 1 s interval around 0 lag. stLFPs were normalized similarly to stPRs (see below).
The size of the stPR was quantified as the value of the spike-triggered population rate at 0 time lag. Thus, the population coupling of unit is given by:

Here, represents the smoothed firing rate of a neuron (Gaussian kernel of half width  ms), is its mean firing rate, and represents its norm (that is, the number of spikes fired). To compare the sizes of stPRs across recordings, they were normalized by the median size of the stPR of the shuffled data in each recording (see next paragraph).
Spike shuffling was performed according to the previously described raster marginals model14. In more detail, the recording was first divided into non-overlapping 1 ms bins. A binary matrix was then constructed with one column for each time bin and one row for each isolated unit as well as additional rows for the multiunit spiking on each shank. Each matrix element contained a 1 if the corresponding unit spiked in the corresponding time bin. To shuffle, random 2-by-2 submatrices were repeatedly chosen with each row and column of the submatrix containing a 0 and 1; the positions of 0s and 1s were then exchanged in the submatrix, which leaves the summed values of each row and column identical. As we have discussed previously14, such a shuffling procedure produces in the limit a uniform sample from a distribution subject to the constraints on the mean firing rate and population rate distribution of the original data.

Hi Aitor, I see you become an engaged user of Elephant! We're currently in the middle of discussing my last PR, we'll come to your feature requests in two weeks.
Would you be interested in implementing the stPR?

Hi Danilo!
Yes, I would implement it if nobody else has done it. Since I will most likely use this metric anyway, I will have to implement it at some point, and elephant seems the right place for it to be :)

We have agreed on adding this feature in Elephant. It should be close to spike_triggered_xx functions.
Do you have the source code already? If yes, open a PR and provide either unit tests or an example you use to run the functional.

That is cool!
I will work on the implementation, since it is still non-existent.
Will open a PR when it is ready.


Automate docker image releases

docker pull asyncapi/generator:latest gives you version 0.22.0 now, this version is over 2 months old ( https://hub.docker.com/r/asyncapi/generator/tags ).
Ideally new release also create new docker images

Yea agree, should be automated 👍

after some cleanup on local, I've just pushed to docker latest 0.28.3 version with tag 0.28.3 and latest. Now will work on automating it.

Ok, my job is done, ready for review
Once it is merged I will set proper secrets for the GitHub Action flow in Settings/Secrets

Docker image will now always be published at the same time NPM package, once we tag the repo.
You cannot see any action here https://github.com/asyncapi/generator/actions because it won't show up until the first run of the Action


assets paths contain username address

Assets' paths in build folder are set to absolute paths,
and when deployed, the assets are all not found 404
so how to make the build return this
"main.css": "/static/css/main.150b2e95.chunk.css",

instead of this
"main.css": "/Ahmed-E-Elswerkey/domino/static/css/main.150b2e95.chunk.css",

Are you building with homepage set in your package.json?

yeah, set to the git repository

I tries setting it to nothing, and it worked,
thanks, heyimalex, for hinting this


Proposal for Makefile with GNU Make 4.4.1 Built for Windows

Install Make in Windows with chocolatey
>choco install make
--Rename folder LWN-Simulator with LWNSimulator

SUFFIX := ".exe"
clean:
	@rmdir /s /q bin

install-dep:
	@go install github.com/rakyll/statik
	@go get -u -d ./...

build:
	@echo "Building LWN Simulator"
	@echo "Building User Web Interface"
	@cd webserver && statik -src=public -f 1>NUL
	@mkdir bin
	@copy config.json bin\config.json
	@echo "Building from the source.."
	@go build -o bin/lwnsimulator${SUFFIX} cmd/main.go
	@echo "Build Complete."

build-platform:
	@echo "Building LWN Simulator"
	@echo "Building User Web Interface"
	@cd webserver && statik -src=public -f 1>NUL
	@mkdir bin
	@copy config.json bin\config.json
	@echo "Building from the source.."
	@go build -o bin/lwnsimulator${SUFFIX} cmd/main.go
	@echo "Build Complete."

build-x64:
	@make build-platform GOOS=windows GOARCH=amd64 SUFFIX="_x64.exe"

build-x86:
	@make build-platform GOOS=windows GOARCH=386 SUFFIX="_x86.exe"

run:
	@go run cmd/main.go

This issue has been addressed with https://github.com/UniCT-ARSLab/LWN-Simulator/commit/2649658d77db7afb84495ed05034d710afaefd86
Wait for the next release!


Disable AODH and Ceilometer

Closes: #821
RPC-O does not support either of these two products and should
therefore not deploy these services.  By removing the associated env.d
files we disables these services and prevent their deployment.

that didn't seem to work...
01:20:59 TASK: [lxc_container_create | Create container] *******************************
01:24:14 changed: [jrpcaioiad-f0f_ceilometer_collector_container-cad27350 -> jrpcaioiad-f0f]
01:24:20 changed: [jrpcaioiad-f0f_aodh_container-51b7ef9f -> jrpcaioiad-f0f]
01:24:20 changed: [jrpcaioiad-f0f_ceilometer_api_container-e510ab76 -> jrpcaioiad-f0f]

also started it

I could use the method defined in #815 but that'd only work if using the scripts

recheck - unrelated fail in swift
21:00:06 jrpcaioiad-7e5_swift_proxy_container-f33b69b1 : ok=55   changed=33   unreachable=0    failed=1

+1 in concept - I don't see a problem with removing these.

To add - rpc-openstack isn't supporting these two services, so I'm for their removal. If they remain, support and customers will be getting extra containers made they're not expecting.

ah, this is why
22:40:54 TASK: [os_swift | Add ResellerAdmin role to the service tenant and ceilometer user] ***

turns out you don't have the user if you don't enable ceilometer, osa bug most likely

neat, same
23:58:47 failed: [jrpcaioiad-d25_swift_proxy_container-135aec05] => {"attempts": 5, "err": "user [ ceilometer ] was not found.", "failed": true, "rc": 2}

Those two errors should only appear if ceilometer is specifically enabled for Swift. Check the conditionals on the tasks or facts set ahead of time. You may find that the openstack_user_config.yml, conf.d or user_*.yml entries are still around that shouldn't be there.

shouldn't be there, as they are explicitly removed, but I'll check tomorrow.

@prometheanfire Ceilometer is explicitly removed, but it could be some Swift vars are still set to use it

Looked like swift's ceilometer support is disabled by default in the openstack-ansible liberty vars. https://github.com/openstack/openstack-ansible/blob/liberty/etc/openstack_deploy/user_variables.yml#L21

The swift role is also set to check the variable appropriately. https://github.com/openstack/openstack-ansible/blob/liberty/playbooks/roles/os_swift/tasks/swift_service_setup.yml#L137

I think I see the problem. The gate jobs are configured to deploy ceilometer, but this PR specifically removes it.
23:01:44 + sudo DEPLOY_AIO=yes DEPLOY_HAPROXY=yes DEPLOY_TEMPEST=yes DEPLOY_CEPH=no DEPLOY_SWIFT=yes DEPLOY_CEILOMETER=yes DEPLOY_MAAS=yes ANSIBLE_GIT_RELEASE=ssh_retry ANSIBLE_GIT_REPO=https://github.com/hughsaunders/ansible ADD_NEUTRON_AGENT_CHECKSUM_RULE=yes BOOTSTRAP_OPTS= bootstrap_host_ubuntu_repo=http://mirror.umd.edu/ubuntu/ bootstrap_host_ubuntu_security_repo=http://mirror.umd.edu/ubuntu/ scripts/deploy.sh

deploy.sh sets the default to no, but the jenkins call sets it to yes.

I updated the job and removed the deploy_ceilometer env.
recheck

recheck - (has to be the firist thing in the comment I think) - thanks for looking at the gate job.

Failure on finding keepalived:
16:47:52 + openstack-ansible --forks 4 haproxy-install.yml
16:47:52 Variable files: "-e @/etc/openstack_deploy/user_extras_secrets.yml -e @/etc/openstack_deploy/user_extras_variables.yml -e @/etc/openstack_deploy/user_secrets.yml -e @/etc/openstack_deploy/user_variables.yml "
16:47:52 ERROR: cannot find role in /jenkins/workspace/RPC-AIO/openstack-ansible/playbooks/roles/keepalived or /jenkins/workspace/RPC-AIO/openstack-ansible/playbooks/keepalived or /etc/ansible/roles/keepalived

Earlier on:
16:46:21 - downloading role 'keepalived', owned by evrardjp
16:46:21 - sorry, evrardjp.keepalived was not found on galaxy.ansible.com.

Checking Galaxy to see if the role's still there or if they're having issues.

This is currently broken because the role author removed it from Galaxy. A change is incoming for OSA.

If we don't want to bump the OSA SHA to consume that change, we could also add the repo URL to this repo's role requirements until we consume the updated OSA SHA.
I'd like input from @prometheanfire and @d34dh0r53  on that, though.

https://review.openstack.org/#/c/280822/ is currently proposed. The author claims he didn't pull the role, but that galaxy is no longer showing it. https://galaxy.ansible.com/evrardjp/ doesn't list it, either.

https://github.com/ansible/galaxy-issues/issues/121 has also been opened.

recheck

@erikmwilson @robb-romans For docs, this is worth pointing out.
In summary:
openstack-ansible supports ceilometer and its dependency, aodh, in liberty. However, rpc-openstack does not. Therefore, on deploy, the aodh and ceilometer environment files need to be removed.

Still broken, but https://github.com/ansible/galaxy-issues/issues/121#issuecomment-184966737 appears to indicate this will be fixed when Jean-Phillipe is able to re-import the role.

recheck

same error, neat

ok, verified that they aren't installed still though
here's the cause...
https://github.com/openstack/openstack-ansible/blob/liberty/tests/roles/bootstrap-host/templates/user_variables.aio.yml.j2#L19-L29
07:40:06 PLAY [Install the ceilometer components] **************************************
07:40:06 skipping: no hosts matched
07:40:06
07:40:06 PLAY [Install the aodh components] ********************************************
07:40:06 skipping: no hosts matched

Ok, I like it. :+1:

Still +1, was waiting on checks to pass.


Google Analytics verification for Google Webmaster Central

Details

Project URL: http://neo4jrb.readthedocs.io/
Build URL (if applicable):
Read the Docs username (if applicable): cheerfulstoic

Hopefully informative history / explanation
I'd like to be able to link to my RTD documentation on a YouTube video that I made because there is some information in the video which is out of date.  In order to link to a website from a YouTube video, you apparently need to be able to prove that you are the webmaster.  There are various ways to do that, but they only one that I figured would work since my docs are generated via rst files is through Google Analytics verification.  I was able to add my GA code into my RTD admin page and after a day or so it started working (which is awesome, by the way.  That by itself is wonderful to have).  But when I try to verify the site via Google's Webmaster Central it won't verify.  On the page to verify it says the following:
* You must be using the asynchronous tracking code .
* Your tracking code should be in the <head> section of your page.
* You must have the "edit" permission for the Analytics web property.

I have the "edit" permission in Analytics, so I'm guessing perhaps one of the first two things isn't true.  I looked at the source of the docs page and I don't see anything about Google Analytics / ga in the source, so maybe the tracking is done via server side events?  Any insight help would be welcome.
Thanks for readthedocs, BTW!

I don't know too much about how ReadTheDocs works with Analytics, but I'd think that your issue may lay with the analytics.js script not being in the <head> section of the site.
The asynchronous tracking code, as far as I know of, is a UA-XXXXXXXX-X format, X being a random number, so if your tracking code is in that format, that shouldn't be the issue. Google Developers on Website Analytics explains it all in more depth, but I'm not sure if that helps.
First attempt at helping with RTD.... Probably failed.

You're welcome to include GA in your own HTML with the normal Sphinx way of doing it. We also have a setting in the project Admin > Advanced Settings that outputs it on the page, along with ours.

Thanks to both of you ;)
I set the "Analytics code" under the "Advanced Settings", but I didn't see it on the page.  But I'll try the Sphinx way of doing it (I hadn't thought of that).  Thanks!

FYI for anybody coming along, I was able to get the GA code into the header by editing the docs/_templates/layout.html file.  See here:
https://github.com/neo4jrb/neo4j/blob/8.1.x/docs/_templates/layout.html


Ethan: Watch duckos.tk

Ethan requests the watch of the watch_keyword duckos\.tk. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

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

Approved by Spevacus in Charcoal HQ


You guys should figure out what engine you want this running on before anyone starts coding

Just using JFrame would be severely limiting for this project, would basically eliminate the ability to have a custom looking user interface, or animations and would make networking more complex. It is probably best for a game engine/library to be chosen before any code gets done as the entire main class would have to be built around it. This decision would be mostly on the GUI guys as they are the ones who would have to deal with it. I already have experience with OpenGl so if you go with that I could definitely be helpful with that. With whatever Engine/Library you go with I should be able to help with the lower level setup, making the basics of rendering, texturing, and animation if necessary and leave the GUI guys to deal with the event based programming, and actual design.

OpenGL may be overkill. I agree that just JFrame would be too limiting. See #11 for my idea for a graphics library. The @XBHSCS15/graphical-user-interface Team would certainly appreciate your help though.

Closing due to our seemingly de facto choice of Processing


The "node app.js" problem

All steps was proceeded to "node app.js" step.
Inside myJsBots directory the bot directory created.
I change to this bot directory and run "node app.js".
I get:
module.js:471
throw err;
^
Error: Cannot find module 'botbuilder'

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

ID: dfaad3f2-75c5-e958-4734-8925d03d95c6
Version Independent ID: 8d98b390-5368-1fa4-8cce-3b0cff252eaa
Content: Create a bot using Bot Builder SDK for JavaScript - Bot Service
Content Source: articles/javascript/bot-builder-javascript-quickstart.md
Product: bot-framework
GitHub Login: @JonathanFingold
Microsoft Alias: jonathanfingold

@MishaVigdarov - did you use @preview tag to get the correct version?

Instructions also does not appear to account for TypeScript (you have to either run tsc to compile the TypeScript code or use ts-node)


Repeatedly claim Redis messages during processing

If director catastrophically fails (oom killed, preempted, etc) then the only signal we have that something has gone wrong is that the prediction never finishes and the message in the queue is never acked. To make use of that signal we have to wait longer than the prediction could possibly take.
This changes the behaviour of director so that it uses XCLAIM to continuously update the idle time of the message it's processing. If the idle time of any message is longer than the claim_interval used (with a resonable margin for error) then we can assume the director responsible for it has somehow died, and we can either retry it or fail it from whatever system monitors the queue.
This is done in a thread so that issues with, eg, webhook delivery don't stop the idle time from being updated at the interval.

Hmmmm, I'm not sure how I feel about doing this in another thread. It seems to me that opens up possibilities of (e.g.) the main thread getting stuck in a loop somewhere while we happily claim the message every few seconds as if nothing is wrong.
I had imagined that we would bake this into the main while not tracker.is_complete(): loop somehow. Perhaps you'd be up for pairing with me on this tomorrow to see if that's possible?
Alternatively, if we're keen to get something in to mitigate possible issues for ThumbnailAI, I can probably stomach it for now :)

Hmmmm, I'm not sure how I feel about doing this in another thread. It seems to me that opens up possibilities of (e.g.) the main thread getting stuck in a loop somewhere while we happily claim the message every few seconds as if nothing is wrong.
I had imagined that we would bake this into the main while not tracker.is_complete(): loop somehow. Perhaps you'd be up for pairing with me on this tomorrow to see if that's possible?

Happy to pair on it! I first tried to put it into the main loop, but the way we're currently sending webhooks risks false positives on dropped predictions. If we can work out how to get around that, then I agree that having it in the main loop is much safer.
I figure the thread is OK, at least for now, because the main things we're guarding against are the node being preempted or the director being oom killed (or similar), both of which will destroy this thread at the same time.


Release 2.0.0

Ping @jtimberman

Thanks for merging my stuff earlier.
My main concern here is the removal of the service not_if. On the server side, the empty client.keys issue is very annoying. I have seen confused posts on the mailing list about it so we should guard against it. I am trying to get it fixed upstream at ossec/ossec-hids/issues/662. On the agent side, agent_server_ip is determined by a search, which might come up blank if the server isn't ready yet. The Chef run would then die when trying to start the service.
If we're going to break things now then I really hope you would look at #48 and perhaps even consider dropping the existing SSH stuff as authd is very much safer. It's a shame OSSEC 2.9 isn't out yet as that adds certificate verification.

Closing this out as it did more harm than good


Node Fetch: Cannot cancel a stream that already has a reader

Reproduction
export const loader: LoaderFunction = async ({ context, request }) => {

  const controller = new AbortController()
  fetch('http://google/api.com', {
    signal: controller.signal,
  })
  controller.abort()

  return defer({
  })
}

System Info
node v18.12.0
remix-run/node 1.15.0

Used Package Manager
pnpm
Expected Behavior
The fetch request can be cancelled normally, but an error was caught in the catch.
Actual Behavior
#9216
I used fetch in Remix, but encountered the following issue：

I'm not sure how I should use fetch on the server to initiate request processing and interrupt requests after timeout.
export const loader: LoaderFunction = async ({ context, request }) => {

  const controller = new AbortController()
  fetch('http://google/api.com', {
    signal: controller.signal,
  })
  controller.abort()

  return defer({
  })
}

This may seem like a bug, but I do need to retrieve some data through fetch on the server. Is there no other way for Remix to do this?

see here on the tag of feat, would like to ask the next question for the time being no hack ending plan ？

I am not sure about the current progress of this issue, but I have used a hack approach to address it
const nodeFetch = require("node-fetch");
let fetchOverlay = false;
export function setupFetch() {
  if (!fetchOverlay) {
    globalThis.fetch = nodeFetch as any;
    fetchOverlay = true;
  }
}

I am not sure about the current progress of this issue, but I have used a hack approach to address it
const nodeFetch = require("node-fetch");
let fetchOverlay = false;
export function setupFetch() {
  if (!fetchOverlay) {
    globalThis.fetch = nodeFetch as any;
    fetchOverlay = true;
  }
}

https://github.com/remix-run/remix/issues/9222#issuecomment-2053949821

I am not sure about the current progress of this issue, but I have used a hack approach to address it
const nodeFetch = require("node-fetch");
let fetchOverlay = false;
export function setupFetch() {
  if (!fetchOverlay) {
    globalThis.fetch = nodeFetch as any;
    fetchOverlay = true;
  }
}

I'm not sure what problem this will bring, but it did solve the problem temporarily


Module Version 2.3.0 - Update SaaS and other Help docs

Module Version 2.3.0 - Update SaaS and other Help docs
Mostly just lots of small changes centered around the SaaS API endpoints, and other miscellaneous help documentation

Changed $saasCustomerId parameter from <string> to <int> type, as documented in Datto's Swagger file.
Updated $daysUntil parameter range to accept '0' value as this is also accepted by the endpoint
Update URL link to project repo and removed reference to deprecated dev branch in GitHub templates, and contributing docs
Corrected endpoint references of 'bulkSeatAssignment' to 'bulkSeatChange'
Miscellaneous formatting and changes in help docs

Contributes: https://github.com/Celerium/Datto-PowerShellWrapper/issues/4
Type of change

[x] General code cleanup (non-breaking change which improves readability)

Checklist

[x] I have reviewed the Contributing guide
[x] I am pulling to the main branch
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] New and existing unit tests pass locally with my changes

How Has This Been Tested?
PS \GitHub\Datto-PowerShellWrapper\tests> .\DattoAPI.Tests.ps1 -version 2.3.0 -buildTarget notBuilt

Starting discovery in 1 files.
Discovery found 23 tests in 46ms.
Running tests.
[+] \GitHub\Datto-PowerShellWrapper\tests\DattoAPI.Tests.ps1 623ms (428ms|152ms)
Tests completed in 626ms
Tests Passed: 23, Failed: 0, Skipped: 0 NotRun: 0
PS \GitHub\Datto-PowerShellWrapper\tests> .\HelpComment.Tests.ps1 -version 2.3.0 -buildTarget notBuilt

Starting discovery in 1 files.
Discovery found 320 tests in 1.22s.
Running tests.
[+] \GitHub\Datto-PowerShellWrapper\tests\HelpComment.Tests.ps1 2.99s (616ms|1.16s)
Tests completed in 3s
Tests Passed: 316, Failed: 0, Skipped: 4 NotRun: 0
PS \GitHub\Datto-PowerShellWrapper\tests> .\PSScriptAnalyzer.tests.ps1 -version 2.3.0 -buildTarget notBuilt

Starting discovery in 1 files.
Discovery found 2108 tests in 1.33s.
Running tests.
[-] Testing the [ notBuilt ] version of [ DattoAPI ] with [ PSScriptAnalyzer.tests.ps1 ].[ DattoAPI ] module PSScriptAnalyzer tests.[ DattoAPI.psd1 ].Should pass rule [ PSUseToExportFieldsInManifest ] 300ms (295ms|6ms)
 at ($invoke_Results | Measure-Object).Count | Should -Be 0, \GitHub\Datto-PowerShellWrapper\tests\PSScriptAnalyzer.tests.ps1:149
 at <ScriptBlock>, \GitHub\Datto-PowerShellWrapper\tests\PSScriptAnalyzer.tests.ps1:149
 Expected 0, but got 2.
Tests completed in 44.89s
Tests Passed: 2107, Failed: 1, Skipped: 0 NotRun: 0

[x] Ran all local unit tests
[x] Validated -saasCustomerId int type documented in Swagger for API properly calls endpoint as expected with change from <string>

Thank you very much again!


Rename Core service to Orchestrator

Motivations

Today, the Core service only runs the dispatcher after some checking (config and version).
The name Core is not meaningful. People don't know what is done behind the scenes. On the contrary, the term Orchestrator is explicit.
Running the service is only necessary for task orchestration, not for scenario mgt or data integration. You can have some data nodes and scenarios, and you can read, write, and compare them without submitting anything. In this case, you don't need to run Core service. Having it named Orchestrator makes it easier to document.

Note that the package core must remain unchanged.
TODO

Rename the class to Orchestrator. The Orchestrator class would be the public facade for the _Orchestrator, the _Dispatcher, and their Factory.
Rename the corresponding variables
Rename the tests and their variables
Change the documentation (User man, Ref man, tutorials)
Change the demos and gallery

do we also want to rename CoreSection to OrchestratorSection and CoreCLI to OrchestratorCLI?

The CoreCLI only contains options related to the version mode. It's not Core nor Orchestrator CLI. I'm not sure about the naming now.
We already have a different _VersionCLI for version management, which is different though.

do we also want to rename CoreSection to OrchestratorSection, CoreEvent to OrchestratorEvent and CoreCLI to OrchestratorCLI?

I would not.
-> The CoreSection is dedicated to the core package and not to the job orchestration.
-> The events are not related to orchestration either.

The CoreCLI only contains options related to the version mode. It's not Core nor Orchestrator CLI. I'm not sure about the naming now.
We already have a different _VersionCLI for version management, which is different though.

I would keep CoreCLI since it is the related to the way we run the core package. :/

I reopened the issue because of the documentation part.

Another PR related to this issue:
https://github.com/Avaiga/taipy-doc/pull/1098


With resource group contributor role i can't see nothing in security center

Hello,
with test user who has contributor role on one of resource group, i can't see nothing in security center, even when there are alerts for some resources from this resource group. I don't want to give users security reader over subscription.
Thank you.

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

ID: 8c61e7b5-619a-fbd4-c7de-a1d04a33e61f
Version Independent ID: c35c73d4-42d5-eb01-bb54-6c9c2ad9377a
Content: Permissions in Azure Security Center
Content Source: articles/security-center/security-center-permissions.md
Service: security-center
GitHub Login: @rkarlin
Microsoft Alias: rkarlin

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

@hadr10 I am not having any difficulties seeing the recommendations with Resource Group Contributor Role.  When you go to the Security Center blade with the test user account you may see this message over the screen.

Once you click on message and security center recommendations and other details will start appearing on the portal for your resource group only.

@SaurabhSharma-MSFT you are correct, after clicking to message it shows everything in Azure security center how it should. Thank you

@hadr10 We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Support localization

suggest making the object valid linked data, and handling localization with JSON-LD.

Can be address in v2


Mutations triggered in a lambda function don't push subscriptions to clients.

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
yarn
If applicable, what version of Node.js are you using?
No response
Amplify CLI Version
9.2.1
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No
Amplify Categories
api
Amplify Commands
Not applicable
Describe the bug
I have a number of subscriptions.
If I open the app in, say, Chrome and open the app in another browser, say, Firefox, mutations push subscriptions just fine.
Meaning, when I call a mutation created with useMutation, the opposite browser gets a notification.
However, if I call the same mutation from a lambda, no subscription update is pushed to the client.
I'm doing it like this:
const appsync = new AWSAppSyncClient({
  url: GRAPHQL_ENDPOINT,
  region: "us-east-1",
  disableOffline: true,
  auth: { type: AUTH_TYPE.AWS_IAM, credentials: () => AWS.config.credentials },
});

appsync
        .mutate<
          TriggerUserWagerUpdateMutation,
          TriggerUserWagerUpdateMutationVariables
        >({
          mutation: triggerUserWagerUpdate,
          variables: {
            input: {
              id: "9e94c65f-c61d-40e4-9678-0aae28a33014",
              amount: 400,
            },
          },
        })

Expected behavior
The lambda update pushes a notification just like the client update does.
Reproduction steps
See description
GraphQL schema(s)
No response
Project Identifier
Project Identifier: e32d9afe4d588c8820816299672c37e3
Log output
No response
Additional information
No response

Hey @CodySwannGT :wave: thanks for raising this! To better understand the context of the issue would you mind posting the contents of triggerUserWagerUpdate?
From the project identifier we were able to view the schema where there does not appear to be a subscription tied to this mutation (assuming the mutation is pointing to updateUserWager)
For reference https://docs.aws.amazon.com/appsync/latest/devguide/aws-appsync-real-time-data.html


Autocomplete: Add position prop

Description

Added position prop for the options container
Used popper library for the options container's position

References
Fixes #2801
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation update

How Has This Been Tested?

Checklist

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

@ElishaSamPeterPrabhu Once I select an option from the bottom of the list, I'm not able to change it.
User.Inputs._.Autocomplete.-.Default.Storybook.-.Google.Chrome.2024-09-12.00-17-54.mp4

Added animation change as fix


Move macro definitions into SwiftGodot target

This should avoid all the complications discussed in #148, leaving SwiftGodot the only dependency and only import to worry about for the users. And it'll keep the initial composition of libDriver.dylib+libSwiftGodot.dylib as the build output.
But it'll require additional workarounds for Windows, since macros aren't supported there yet.

I don't think we should merge the two targets together because then that would imply that you would need both to operate. Additionally, this can become much harder to test, and I imagine this could break how macros work in the future should Swift change how it operates. Rather, we should look into creating a new target that imports both so that you can still have the single import statement while also having the smaller, more composable targets.

But currently you do need macro definitions target SwiftGodotMacros (not macro implementations target SwiftGodotMacroLibrary) to operate. It was just hidden by static dependency in the initial implementation.
Look at the case with static dependency on SwiftGodotMacros in #148.
Why would driver's dylib jump in size from 100 Kb to 70 Mb after adding macros? Because driver's dylib also embedded SwiftGodotMacros target due to static dependency, which itself depended on SwiftGodot statically, making driver also embed SwiftGodot.
Why don't you need to copy libSwiftGodot.dylib to the /bin directory anymore after implementing macros? Same thing, because both SwiftGodot and SwiftGodotMacros are already in the driver's dylyb due to static dependency.
I'm fine with the static solution and embedding everything into a single dylib if that's what everyone prefers, just make SwiftGodot product also static to match it.

But currently you do need macro declarations target SwiftGodotMacros (not macro implementations target SwiftGodotMacroLibrary) to operate. It was just hidden by static dependency in the initial implementation.
Look at the case with static dependency on SwiftGodotMacros in #148. Why would driver's dylib jump in size from 100 Kb to 70 Mb after adding macros? Because driver's dylib also embedded SwiftGodotMacros target due to static dependency, which itself depended on SwiftGodot statically, making driver also embed SwiftGodot. Why don't you need to copy libSwiftGodot.dylib to the /bin directory anymore after implementing macros? Same thing, because both SwiftGodot and SwiftGodotMacros are already in the driver's dylyb due to static dependency.
I'm fine with the static solution and embedding everything into a single dylib if that's what everyone prefers, just make SwiftGodot product also static to match it.

Ideally, you can/should be able to still use the SwiftGodot project without using the macros. My issue here is that we're now bundling everything together in this singular, monolithic package with extra bloat, and it's no longer modular. By making the macros a core part of SwiftGodot in order for it to function, we lose this capability.
In my testing and usage, I've always had to copy over libSwiftGodot.dylib (or, in my case, SwiftGodot.framework), and my binary sizes have never ballooned up to 70+ MB. I suspect the reasoning as to why the file sizes are getting larger is because everything is being bundled together into a single package, rather than being broken up into modular, composable parts.

Single "import SwiftGodot" for users, that allows them to use the API and the macros (see the threads I posted on the forums.swift.org, that sadly have gone unanswered)

I've found a way to do this that works really well and still keeps the modular structure, which you can see at https://gitlab.com/indexing-your-heart/engine/swiftgodot. I fear that combining everything into a single structure will also affect the build sizes and other quirks for the modular system I have set up in place, and it will take considerable effort to keep it modularized.

In my testing and usage, I've always had to copy over libSwiftGodot.dylib (or, in my case, SwiftGodot.framework), and my binary sizes have never ballooned up to 70+ MB. I suspect the reasoning as to why the file sizes are getting larger is because everything is being bundled together into a single package, rather than being broken up into modular, composable parts.

I went and tried this patch, for both an extension and the general purpose library, and the size difference are as follows:

Current Code
With this Patch Applied
Difference

SimpleExtension
60,217,376
60,216,048
1,328

SwiftGodot
60,117,176
60,117,176
0

So the growth is marginal, I do not see any problems with this.
I am happy to have a separate discussion on how to reduce the executable size, and other strategies that we could use to cope with the growth over time, but last night 20-minute hack reduced the library by about six megabytes, so unless there are stronger objections, I think that this patch achieves the ease of use goals that I want to get to.
SIDEBAR: the ballooning of the library size is driven to a large extent to the API surface changes from 4.0 to 4.1, and in part by some code generation changes.

I am happy to have a separate discussion on how to reduce the executable size, and other strategies that we could use to cope with the growth over time, but last night 20-minute hack reduced the library by about six megabytes, so unless there are stronger objections, I think that this patch achieves the ease of use goals that I want to get to.

I still don’t think it’s a good idea to have everything merged into a single monolithic binary like this, and I don’t think having a single import statement is worth throwing out the current modularity. Sure, having a single import statement is easy; however, simple and easy aren’t necessarily the same thing. Loris Cro does a good job at explaining the differences here: https://kristoff.it/blog/simple-not-just-easy/
I do believe we can strike a good balance between simple and easy, but I don’t think merging everything into a single binary with extra bloat developers didn’t ask for is the most efficient way of doing so.

The macro implementations are not part of this library, only the macro definitions are, and they are an integral part of the experience that I would like to expose to users, because the alternative is riddled with boiler plate and things that are too easy to get wrong.
I would have loved to have explored these alternatives using the reflection proposal (which would increase the executable size much more than these macros do), but that proposal seems to be stuck, while the macros are not.

An additional bonus to the points 1..4 before:
For the first time since macros were introduced, consuming this library from Xcode projects, or standalone SwiftPM projects work.   I am beyond psyched about it.


Handle posts without categories

Avoid null reference exceptions if there's a post without any categories

This tweak is freakishly simple, @stuartleeks

:-D
C# does have some quite nice constructs these days!


Create Dashboard Route

Create Dashboard Route
Description
A new route for the dashboard needs to be created in the application. This route should initially render an empty page, serving as a placeholder for future content and features.
Requirements

Route Creation: Implement a new route for the dashboard (e.g., /dashboard).
Empty Page: The initial page should be blank with minimal content.

Technical Considerations

Follow existing project conventions for file structure and naming.

Tasks

Create a new file for the dashboard route in the appropriate directory.
Implement a simple functional component that renders an empty page.

Notes
This page will be updated in future iterations as additional features and content are developed.

Can I take care of this issue?
My background and how it can be leveraged
I'm a Software Engineer with over 4+ years of  demonstrable expertise across modern web and blockchain technologies. My tech stack centers on Next.js, TypeScript, React, and Node.js for web development and Solidity, and Cairo/Rust for blockchain solutions.
What sets me apart is my proven track record on OnlyDust and external Projects like Passage, Stellar, etc. This extensive involvement reflects my ability to deliver quality solutions under tight deadlines while adapting to diverse project requirements. With my deep experience in hackathon environments, I bring both technical excellence and practical development skills to the table. I'm confident I can contribute meaningfully to pushing the boundaries of what's possible in blockchain development while maintaining a focus on user-centric solutions.
I'm excited to bring this experience and drive for innovation to your team.
Implementation Plan:
Route Creation
Introduce a new route for /dashboard in the routing configuration.
Ensure seamless integration with the existing route structure.
File Structure and Naming
Create a new file (e.g., Dashboard.js or Dashboard.tsx) in the appropriate directory for page components.
Follow project conventions for organization, naming, and file structure.
Component Implementation
Implement a simple functional component to render an empty page.
Include minimal styling (if applicable) to indicate the placeholder state.
Future-Proof Design
Write clean and modular code that is easy to extend for future dashboard features.
Add comments to guide future development.
I estimate completing the task in 1 day.

Hello I am Joseph I am a frontend developer and blockchain developer and I am also an active contributor here on only dust here is my profile https://app.onlydust.com/u/josephchimebuka. This is my first time to contribute to this repo ill appreciate the opportunity to contribute. this is a couple of frontend and blockchain projects i have built https://metacrypt.vercel.app/ https://mattedsgn.com/
May I try this one?

Why i think i'm the right person for this
I am a software engineer(frontend) with 4 years of hands on experience and i am an avid open source contributor, i also participated in the just concluded hacktoberfest,
the ETA for this if i'm assigned to it would be 48hrs

Could I be assigned to this?
I am a Web3 front-end developer skilled in technical documentation and software development. I greatly understand technologies like JavaScript, Typescript, Cairo, Rust and Solidity. I have contributed successfully to open-source projects like Stark Quest and Speedrun scaffold-stark.
I will tackle this issue by following the routing used in the codebase and render the dashboard page

I would like to work on this issue

Can I take care of this issue?
i'm a frontend dev and a blockchain dev please kindly assign :)

For a simple issue like this, I can provide a solution within an hour. I would like to work on this as soon as possible.

Can I work on this, please?

i would like to take this issue
i am a new contributor to this repo and want to start my contribution journey with Paystell, also i have knowledge of nextjs and page router
ETA : 2 days
Thank you

Step 1: Create the Dashboard Component
I’ll create a new file for the dashboard route in the proper directory. For instance, if it's a React project, I'll place it in src/routes or src/pages.
Example: Basic Functional Component
Here’s what I’ll add for the placeholder dashboard:
// src/pages/Dashboard.tsx or src/routes/Dashboard.tsx

import React from 'react';

const Dashboard = () => {
  return (

Dashboard

Welcome to the dashboard! This page is under construction.

  );
};

export default Dashboard;

Step 2: Set Up the Dashboard Route
I’ll then add this new route to the application’s routing configuration.

React Router Example:
If the project uses React Router, I’ll modify the routing in `App.js` or `App.tsx`:

````import React from 'react';
import { BrowserRouter as Router, Routes, Route } from 'react-router-dom';
import Dashboard from './routes/Dashboard';

const App = () => {
  return (
    <Router>
      <Routes>
        {/* Other routes */}
        <Route path="/dashboard" element={<Dashboard />} />
      </Routes>
    </Router>
  );
};

export default App;```

Next.js Example:
If the project is built with Next.js, I’ll create a new file named dashboard.tsx in the pages directory:

```// pages/dashboard.tsx

const Dashboard = () => {
  return (

Dashboard

Welcome to the dashboard! This page is under construction.

  );
};

export default Dashboard;

I'd like to handle this task.
I am a Full Stack Developer with a strong background in blockchain and extensive experience in Next.js/React, TypeScript, and Rust. I’ve made over 45 contributions to over 15 projects in the OnlyDust ecosystem, solving complex issues and delivering efficient, scalable solutions.
MY ETA is 48hrs. Thanks

Can I contribute to this one?

May I handle this issue? I am an experienced frontend developer.

I am a front-end developer specializing in responsive web design,
device-compatible email templates, 3D web integration,
and technical writing on programming topics. I also contribute
to open-source documentation and help build documentation sites.
I will create a new route that will lead to the dashboard if the user is an admin using the if statement.
glad to work on this
Thanks

Hi, i am Chiboka Emmanuel  a full-stack web developer who enjoys creating solutions and applications that look great and work smoothly. I would really love to be assigned this issue and work on this project.

May I be assigned to this?

Hey! I'm Gerson. 👋
Member of Dojo Coding. I have contributed before to other projects like:

kakarot-rpc
kakarot-ssj
ef-tests
Raito
Starknet-Phone
Madara
stark-cairo-nodes

Can I work on this issue?
Check my OnlyDust profile, I've contributed to many projects: Profile

Plan to Solve the Issue:

Study the codebase to understand the current structure and functionality.
Understand the problem that needs to be solved by thoroughly reading the issue details.
Leverage all provided resources within the issue to get a better grasp of the task at hand.
If I encounter roadblocks, I will ask questions ASAP in the Telegram group to move forward effectively.
I will open a PR and wait for ur feedback


Add NEWS and README

Description
Added NEWs and README.
@ThomasDelteil @piiswrong @zheng-da @eric-haibin-lin @nswamy @lebeg @lanking520 @andrewfayres
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@ThomasDelteil Thanks! I have fixed it.

@piiswrong @ThomasDelteil cherry picked your changes. Please help review as  i had to resolve some conflicts.

I think https://github.com/apache/incubator-mxnet/pull/11267/commits/7632100d8f3ced6375df6d3907c4688fa0979899 is too verbose. It would only confuse users. A simple pointer to export/import should be enough

@piiswrong I think we can remove this sentence:
Such model can then be loaded back in any language binding
or even in Gluon using a :py:class:`SymbolBlock`.

The rest I think is necessary to disambiguate the current API.

Merging now to bring back 1.2 to the right commit.

I agree that we need the rest of the content to inform users about the save_parameters being the new API to use instead of save_params and to use export to serialize the model.

Closing this PR and will a new PR since I have messed up my git history.


Added files for the concept

Just for now and maybe as a starter for the final documentation I uploaded the initial documents we had for the conception of MeMoMe.

included in #41


Problems of evaluating on executables offered by the MMWHS organizer

Hello, could you please tell us more about the evaluation? When I tried to evaluate the prediction results using the executables offered by the MMWHS organizer, unfortunately the results demonstrated that the evaluation was failed with Dice and Jaccard being 0 for all predictions.
Could you please tell us how you use the executables? Is anything I should pay special attention to? Thanks a lot!

I find the where the problem is. The original lable was swapped in the preprocessing course, so we need to swap the labels back to the originals. Thanks.


Compute player score programmatically

Stub:
score = ((player.getInventoryValue()) + (player.getSteps())) * global.getDifficulty();

Closed in PR #40


Unimplemented method toJSON/Symbol

Description
First off, thanks for the great Jupyter extension!
I have a bit of a strange issue and hoping you might be able to give some pointers. When I install in a Conda environment I have no problems, everything works fine.
My organization has a very large Python environment with a lot of dependencies (including many Jupyter extensions - I know this means you won't be able to provide much help).
Opening JupyterLab in this environment with JupyterLab-lsp installed (and python-lsp-server[all] but I don't think this the problem) results in the browser hanging and eventually crashing. The dev console shows some sort of infinite loop happening on this line: https://github.com/jupyter-lsp/jupyterlab-lsp/blob/ed88a08288a8f203131b67d59aa98e04124a6af1/packages/jupyterlab-lsp/src/virtual/codemirror_editor.ts#L137

i.e. the code is stuck in a loop and produces so much output here that it is unable to do anything else.
Since it works fine in a clean Conda environment I know it's a problem with my environment. I know you can't provide much help without access to it but I'm wondering if anything seems suspicious based on what I've described so far? I've removed pretty much  every jupyter/notebook related extension that I can think of...
I'm on Python 3.7:
jupyter --version
Selected Jupyter core packages...
IPython          : 7.34.0
ipykernel        : 6.16.2
ipywidgets       : 7.7.2
jupyter_client   : 7.4.4
jupyter_core     : 4.11.2
jupyter_server   : 1.21.0
jupyterlab       : 3.5.0
nbclient         : 0.5.13
nbconvert        : 6.4.5
nbformat         : 5.2.0
notebook         : 6.5.2
qtconsole        : 5.3.2
traitlets        : 5.5.0

Since this happens in the browser is it fair to say that the Python environment can't be the problem and it must be some other extension that's messing with CodeMirror?

Minor update. It seems like it's not enough to jupyter labextension disable or pip uninstall an extension, I had to delete: share/jupyter/labextensions. This got LSP working. Now to figure out which extension is conflicting...

Major update: it was an internal extension. The code causing the issue is:
import * as Sentry from '@sentry/browser';
import { CaptureConsole } from '@sentry/integrations';

...

// jupyter lab extension stuff

...

      Sentry.init({
         enabled: true,
         autoSessionTracking: false,
         integrations: [new CaptureConsole()]
       });
       Sentry.setUser({ userEmail });
       Sentry.setTag('appGroup', 'jupyterlab-browser');

I guess this isn't surprising as Sentry probably does a bunch of proxying to enable it to capture logs and stacktraces...

Specifically it's the CaptureConsole integration: https://docs.sentry.io/platforms/javascript/configuration/integrations/plugin/#captureconsole

This integration captures all Console API calls and redirects them to Sentry using the SDK's captureMessage or captureException call depending on the log level. It then retriggers to preserve default native behavior.

If I had to guess I imagine it's something to do with this:

It then retriggers to preserve default native behavior.

and how it is interacting with the proxy here: https://github.com/jupyter-lsp/jupyterlab-lsp/blob/ed88a08288a8f203131b67d59aa98e04124a6af1/packages/jupyterlab-lsp/src/virtual/codemirror_editor.ts#L129


Golden image testing

Our customers have asked for an automated way to test the correctness of their UI. There are various internal systems for how to do this.
Internal issue: 31017525
Potential way to do this for external: https://github.com/flutter/flutter/issues/3227

Will defer to tracking internally.


thunder.jit fails with nn.Softmax raising got an unexpected keyword argument '_stacklevel'

🐛 Bug
thunder.jit fails with nn.Softmax raising:
TypeError: softmax() got an unexpected keyword argument '_stacklevel'

pointing to: https://github.com/pytorch/pytorch/blob/a21327e0b03cc18850a0608be2d9c5bd38fd4646/torch/nn/functional.py#L1883
To Reproduce
import torch
from thunder.examine import examine

examine(torch.nn.Softmax(1).cuda(), torch.randn(1, 10, device="cuda"))
#TypeError: softmax() got an unexpected keyword argument '_stacklevel'

Environment

PyTorch Version (e.g., 1.0): 2.4.0.dev20240423+cu121
thunder build from source using at https://github.com/Lightning-AI/lightning-thunder/commit/e0ab64867a5be914d0548c195a3f850a76c8c397

CC @Fuzzkatt as you've encountered this issue first.

Attaching full stacktrace below:
Traceback (most recent call last):
  File "/patwang-space/thunder_transform_pass/explore.py", line 250, in <module>
    explore(model, jfunc_in, fwd_extrace_in, bwd_extrace_in)
  File "/patwang-space/thunder_transform_pass/explore.py", line 8, in explore
    jfunc(*jfunc_in)
  File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1541, in _call_impl
    return forward_call(*args, **kwargs)
  File "/opt/pytorch/lightning-thunder/thunder/__init__.py", line 201, in forward
    res = self._forward_fn(*args, **kwargs)
  File "/opt/pytorch/lightning-thunder/thunder/__init__.py", line 648, in fn_
    cache_entry, inps, pro_to_epi = get_computation_and_inputs(*args, **kwargs)
  File "/opt/pytorch/lightning-thunder/thunder/__init__.py", line 269, in cache_info_wrapper
    res = fn(*args, **kwargs)
  File "/opt/pytorch/lightning-thunder/thunder/__init__.py", line 527, in get_computation_and_inputs
    jit_results: TraceResults = interpreter(fn, args, kwargs, sharp_edges=cd.sharp_edges)
  File "/opt/pytorch/lightning-thunder/thunder/__init__.py", line 182, in _general_frontend
    return thunder_general_jit(fn, args, kwargs, sharp_edges=sharp_edges)
  File "/opt/pytorch/lightning-thunder/thunder/core/jit_ext.py", line 1439, in thunder_general_jit
    result = jfn(*args, **kwargs)
  File "/opt/pytorch/lightning-thunder/thunder/core/interpreter.py", line 6673, in fn_
    raise e
  File "/opt/pytorch/lightning-thunder/thunder/core/interpreter.py", line 6641, in fn_2
    return fn(*args, **kwargs)
  File "/opt/pytorch/lightning-thunder/thunder/core/interpreter.py", line 6040, in _impl
    return fn.__func__(fn.__self__, *args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/opt/pytorch/lightning-thunder/thunder/core/interpreter.py", line 6040, in _impl
    return fn.__func__(fn.__self__, *args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1541, in _call_impl
    return forward_call(*args, **kwargs)
  File "/opt/pytorch/lightning-thunder/thunder/core/interpreter.py", line 6040, in _impl
    return fn.__func__(fn.__self__, *args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/activation.py", line 1545, in forward
    return F.softmax(input, self.dim, _stacklevel=5)
  File "/opt/pytorch/lightning-thunder/thunder/core/interpreter.py", line 1252, in wrapping_wrapper
    res = ufn(*uargs, **ukwargs)
  File "/opt/pytorch/lightning-thunder/thunder/core/symbol.py", line 257, in __call__
    result = self.meta(*args, **kwargs)
  File "/opt/pytorch/lightning-thunder/thunder/core/langctxs.py", line 124, in _fn
    result = fn(*args, **kwargs)
TypeError: softmax() got an unexpected keyword argument '_stacklevel'

Is it possible that thunder is getting confused between regular torch.nn.Softmax (which doesn't have a _stacklevel arg) and torch.nn.functional.softmax (which does have a _stacklevel arg)?

could you try 282?

Just tried 282, hitting
TypeError: _softmax() got an unexpected keyword argument '_stacklevel'
Seems like it's still passing on _stacklevel to _softmax somehow?


Add threat management worldmap for IDS/IPS logs

Currrently in the Unifi Controller a Threat Map is seen which should show you the locations of incoming threats. Sadly this doesn't work properly and only displays your own location, as it sees this as its 'source'.
Since UNPoller supports the addition of IDS/IPS data through Loki as a datasource, it would be neat to have a worldmap or some kind of visualization in a dashboard to see the actual sources of incoming attacks etc like below:

This isn't something I'm going to build, sorry. The data is there, so you're welcome to build it yourself.

added additional info, but agreed, the viz in loki would probably need to be done elsewhere.


Translate Glacier Simulator experiments into SERC-documented activities

The list of possible experiments in the Glacier Simulator page is fantastic.  The web page formatting may not be the most effective way of putting them across for instructors.  I suggest pulling most of the detailed experiments into one or more SERC labs -- probably Lab 3 in the current numbering? -- and streamlining the web page to link out to them, with just a brief description of the science questions.

:+1:


Fix bower.json main param: define release css and js location

When installing as a bower package, wiredep dependency manager automatically reads bower.json, then look into main param of installing package and include js and css in right places in parent projects.
But, when I've install likely via Bower, i've seen that lekely's main section looks like:
"main": "source/index.js",

Currently in main param missed css file, and index.js file isn't work properly.
So, main param should looks like:
"main": [
    "release/likely.css",
    "release/likely.js",
  ],

You can read more about main section in bower.json and how it should be defined here
Cheers!

Thanks for reporting the issue. I updated bower.json, I'll test it with wiredep and then close the issue in case of success.

It seems that everything works like a charm, Likely's main files are injected into HTML without any problems with wiredep.


feat: add model for completions

This PR adds a model for completion events:

@Ian2012 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.


chore(master): release 1.25.1

:robot: I have created a release beep boop
1.25.1 (2023-09-21)
Bug Fixes

优化菜单管理 (d72bac5)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/KYX1234/Element-Admin/releases/tag/v1.25.1 :sunflower:


[Feature] Camera Shake Vertical ++ Control

Note: for support questions, please use one of the following channels.
Reddit: /r/gbstudio
Discord: Join Chat
This repository's issues are reserved for feature requests and bug reports.

I'm submitting a ...

[x] feature request

What is the current behavior?
Camera Shake is horizontal and, we only have the time control.

What is the expected behavior?
I hope to choose if the shake is vertical and, the offset and speed controls.

What is the motivation / use case for changing the behavior?
I've looked at all Pokémon Generation I attacks (Red, Blue, Yellow), most attacks have horizontal shake, but have offset, velocity, and time differences, and in one attack the shake is vertical.

Please tell us about your environment:

Version: 1.1.0
Platform: [Windows]

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

00:31 until 00:45, you can see the shake variations.
https://youtu.be/4aBT7rjZoIE?t=31
10:00 here's the vertical shake.
https://youtu.be/4aBT7rjZoIE?t=600

I'm working on a PR to add this feature now! 🎉
This involves some (kind of important?) design decisions. Specifically, what are the desired parameters for the screen shake script?
I'm thinking should_x_shake, should_y_shake, shake_length(seconds), and intensity.
Does this sound agreeable?

Not super trivial as we added a work around for incorrect tile loading during a screen shake, I believe this separated scroll x and screenshake x to have a third x variable in v2 beta?
Exposing more options, it may be ideal to have a "should shake" or just use "shake length" is not 0 to determine if we should be shaking the screen? Then further features like speed/intensity/axis won't have a performance impact unless the screen is shaking, apart from the extra variables that would be required in y axis.
Screen shake operations might currently be done with 16bit operations to the overall map position, it might be worth investigating if that could be done on the final 8 bit screen scroll instead, wrapping from 255 to 0 should look no different if we're within the loaded tiles.

@RichardULZ That's interesting. I hadn't thought to account for tile wrapping, but you're absolutely right! If the screen is shaking near a "seam" in the tiles, we'll likely see some graphical artifacting. I'm interested to learn more about how the work around you mention was implemented.
I've gone ahead and implemented an improved screen shake which allows the user to select which axes they want to shake. Please take a look and let me know what you think!


Remove extra print when duplicate pokemon are being exchanged

@OpenPoGo/maintainers

:+1:

:+1: thanks!

:+1:


AttributeError: 'NoneType' object has no attribute 'tag'

Reproduction scenario:
Open a XML
select an entry which is of type execute
open another xml
watch log output

Traceback (most recent call last):
File "./obmenu2", line 1178, in on_label_changed
widget.get_text())
File "./obmenu2", line 372, in set_label
self.set_label(self.get_parent(item), label)
File "./obmenu2", line 366, in set_label
if self.strip_ns(item.tag) == "item":
AttributeError: 'NoneType' object has no attribute 'tag'


Rename ParamsAttribute

better name will be ParamsAttribute –> KfAttr
@h1alexbel wdyt

@l3r8yJ It's better to name interfaces without any prefixes, so I think ParamsAttribute is ok, maybe can be cut to ParamsAttr

@h1alexbel yes, i like your suggestion more


Can't get product title at checkout, Angular app

Hi, I'm working in the angular storefront, I'm having an issue with the cart, I can't get the product title.  {{lineItem.variant.title}} displays: Default title and {{lineItem.variant.product.title}} does not display anything at all.
Anything help will be very appreciated and thank you for the code!

Yes you're correct about the variants, but the product title doesn't not display  on the line item, this is a bug

@dcharua Open up the developer tools and go to the "Networks" tab. There should be a "graphql" network request that returns a Checkout object. You should be able to see the response that Shopify is returning there. This would be a good place to check if (1) Shopify is not returning a product title or (2) Angular is not properly surfacing the returned product title.

The problem is that the variant product doesn not have the title property

hasNextPage: false
hasPreviousPage: false
id: (...)
image: (...)
nextPageQueryAndPath: ƒ ()
price: (...)
product: GraphModel
id: (...)
refetchQuery: (...)
type: (...)
I ended up adding to title to the lineitem model and pass it when I create the lineitem


Remediation Steps in Output are Formatted as Double Numbered List

TL;DR
After running a scan, the remediation steps for a particular finding are double numbered. For example:

Remediation Steps:

Note: The remediation steps apply to legacy branch protections, rules set-based protection should be updated from the rules set page

Make sure you have admin permissions

Go to the repo's settings page

Enter 'Branches' tab

Under 'Branch protection rules'

Click 'Edit' on the default branch rule

Check 'Require a pull request before merging'

Check 'Require approvals'

Set 'Required number of approvals before merging' to 1 or more

Click 'Save changes'

Expected behavior
Only a single numbered list (like in v1.0.7)
Observed behavior
Remediation steps are double numbered
Version
1.0.8
On which operating system are you using legitify?
Linux
Relevant log output
No response
Additional information
Did not experience this in v1.0.7

Thanks @ajh-

Did this happen for only one policy?
Can you paste a few lines above the remediation steps (starting from the policy title)
Can you attach logs?

@ajh- thanks to @sagic-orca your bug was fixed in version 1.0.9.

@royb-legit @sagic-orca thanks for taking this on. I should've been more specific upfront. This happens for all policies when outputted as markdown. Unfortunately, just tested v1.0.9 and still have the same issue.
I'm assuming applying @sagic-orca's fix to formatter_markdown.go should also remediate my current issue?

@ajh- thanks for resurfacing this.
Version 1.0.10 should fix it, could you please check it?

@royb-legit Perfect, just tested it and it looks like it is fixed. Thanks!
I will add, with this formatting update, the blockquote section is now displayed as multiple sections instead of being one contiguous section. Not the end of the world, but just wanted to let you know.
For example:
Default Branch Should Require Code Review :no_entry:

In order to comply with separation of duties principle and enforce secure code practices, a code review should be mandatory using the source-code-management system's built-in enforcement. This option is found in the branch protection setting of the repository.
Policy Name: code_review_not_required
Namespace: repository
Severity: HIGH :no_entry:
Threat:

Users can merge code without being reviewed, which can lead to insecure code reaching the main branch and production.

Remediation Steps:

Note: The remediation steps apply to legacy branch protections, rules set-based protection should be updated from the rules set page

Make sure you have admin permissions
Go to the repo's settings page
Enter 'Branches' tab
Under 'Branch protection rules'
Click 'Edit' on the default branch rule
Check 'Require a pull request before merging'
Check 'Require approvals'
Set 'Required number of approvals before merging' to 1 or more
Click 'Save changes'

Violations:
Link to repository: https://test.com/test
Auxiliary Info:

Entity Id: 77
Entity Name: test


Allow the player to lock Coordinate Selectors

Nothing too fancy for this suggestion, I just feel like it could avoid quite a few mishaps. Not sure about the way you could implement that, maybe a simple craft with just the Coordinate Selector to alternate between Locked and Unlocked.
Keep up the great work and thank you for reading !
(EDIT: Speaking of the Telepad, could we get a capacitor slot in order to speed it up a bit ?)

Hi, I agree; it is extremely easy to accidentally lose coordinates in the coordinate selector.
I think I'd actually even make that default behavior: Once you select coordinates, the selector will automatically be locked.
To unlock it, you could craft it by itself on an empty grid to reset the coordinates stored.


Does it support PHP8?

Does it support PHP8?
The official PHP SDK is not supported. Now you are the only one who has written it.

I have never tried running it with PHP8 as I am no longer maintaining it, maybe you can try and see if it works.

Now only secp256k1 on PHP8 has a problem and may need your help, secp256k1 may be able to use https://github.com/kornrunner/php-secp256k1
I'm not very familiar with encryption

Everything else I can change has been changed.