Middleware should be passing EFFECT actions?

Hi,
I'm currently building a performance-critical app:

that fires many actions per second (~x60 per second)
that has many react-redux connect()ed components
that has many sagas running in parallel to take care of effects (~x5 effects per action)

Since I'm also relying heavily on multiple sagas to take care of effects, I get many saga-related actions running through my redux reducers (EFFECT_TRIGGERED, EFFECT_RESOLVED, EFFECT_REJECTED), but I don't think it is necessary for them to go normal redux reducers, because they are dealt only by sagas anyway.
So I implemented something like:
const no_saga_actions = () => next => action => {
    if (action.type != 'EFFECT_TRIGGERED' &&
        action.type != 'EFFECT_RESOLVED' &&
        action.type != 'EFFECT_REJECTED') return next(action);
};

where my redux configuration store looks like:
const store  = createStore(
    combined_reducer,
    applyMiddleware(
        createSagaMiddleware(...combined_saga),
        no_saga_actions,
    )
);

I know I have to minimize the number of effects and connect()ed components to increase speed eventually, but this has resulted in much greater performance already, simply because my redux store is receiving less actions and that results in less connect()ed wrapper renders.
I wonder if passing saga-related actions is by design, or if there should be an option somewhere to prevent it from flowing to redux reducers (or if you would like me to submit a PR for that)..?
Thanks!

Those are monitoring actions. They are only fired in development mode. You wont see them if you start your app in production mode

Thanks for the clarification. I've been using a build from npmcdn. Using the minified build also solves the problem, for anyone else interested!

@joonhyublee curious, what kind of performance are you seeing? We're in development of something similar and I'm curious what the throughput of the sagas is.

@ronanwize for reference, I have a page of about x10 custom-built graphs. Because each graph selectively fetches data from the server and automatically zooms to fit data on the fly when a user drags it, I have each of the mouse events handled as action to the redux store and various effects handled by redux saga. (without all these effects, I would just handle them as local component state updates)
So optimally, with proper throttling and all, I have ~x60 actions per second. Each scroll action may trigger 3~5 effects within the respective saga. So that is ~x300 actions per second. With x10 graphs on the page, each react-redux connect()ed to redux store, this leads to ~x3,000 connect() wrapper re-renders per second and more, leading to heavy jank (~5fps), even with all the shouldComponentUpdate optimizations. Granted, connect() wrappers are very efficient, but no user interaction should be firing x1,000's of function calls in the first place.
As you can see, things can get out of hand really easily when you handle mouse events through redux store / redux saga, but it was absolutely necessary in my case. To optimize, it is crucial to:

minimize actions passed to the store (all connect()ed wrappers will update for each action) &
minimize the number of connect()ed components at a given instance
make mapStateToProps functions as computation-free as possible

There is no trick here, but with aggressive effects batching, component restructuring, selective showing (rendering only the visible graphs using infinite-list-esque technique), I got the number of actions, number of connect()ed components and subsequent re-renders down to manageable level.
One useful technique is to implement a custom middleware similar to one shown above to filter out actions specific to redux-saga. These actions had the sole purpose of triggering a saga, and had no direct consequence on the store. Remember, even inconsequential actions will invalidate the store and trigger an update on all connect()ed components. This alone significantly cut down the number of actions. There are libraries that allow you to do this, like redux-ignore, but my use case was so simple that I just wrote my own.

Just to add, it can be difficult to use existing redux monitoring tools to see all the actions that are being passed to the store, simply because there are too many actions being fired per second and devtools designed to scrutinize individual actions can't keep up. So I wrote a simple aggregate analytics middleware to use in conjunction with react perf addon:

const make_analytics = () => {
    const perf = React.addons.Perf;
    let stats = {};
    let is_counting = false;

    const count_action = action => {
        if (is_counting) {
            if (!stats[action.type]) stats[action.type] = 1;
            else stats[action.type] = stats[action.type] + 1;
        }
    };

    const start = () => {
        is_counting = true;
        perf.start();
    };

    const stop = () => {
        is_counting = false;
        perf.stop();
        console.log(stats);
        perf.printWasted();
        stats = {};
    };

    const get_stats = () => {
        console.log(stats);
    };

    return {
        count_action,
        start,
        stop,
        get_stats
    };
};

window.analytics = make_analytics();

const analytics_middleware = store => next => action => {
    window.analytics.count_action(action);
    return next(action);
};

and using it looks like:
store  = createStore(
    combined_reducer,
    applyMiddleware(
        analytics_middleware,
        createSagaMiddleware(...combined_saga),
        ignore_middleware
    )
);

in case you are wondering what my ignore_middlware looks like:
const ignore_middleware = store => next => action => {
    if (action.type != 'REQUEST_NAVIGATION') return next(action);
};

@yelouafi sorry, this is not really redux-saga specific, but thought I'd share since somebody asked.. :)

@joonhyublee this might interest you: http://stackoverflow.com/questions/34782249/can-a-react-redux-app-really-scale-as-well-as-say-backbone-even-with-reselect/34788570

@slorber food for thought. thanks!


[Doc] Documentation Needed for eventmesh-connector-kafka

Search before asking

[X] I had searched in the issues and found no similar issues.

Documentation Related
Currently, most connectors lack documentation. The following details must be added to provide better clarity to the developers:- 1. functions of the current connector
2. descriptions of configuration items
3. precautions (if any),
4. relevant official reference links (if any),
5. Other specific content that needs to be explained in the documentation.
Having these details can increase the clarity for the users who are using the product and can also help in onboarding the new developers easily
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

its closed by https://github.com/apache/eventmesh-site/issues/204


Added JPMorgan files

Checked to determine all code added from JPMorgan repository was linted and all test passed. All checks have passed so the code is ready to merge to main.


adds structure test to ensure no java 7 runtime gets pulled in

closes #55.

Nice! Testing logic looks good.
Did you consider running apt-get update before running the test? While building the openjdk docker container I believe we clear the apt caches, so it's possible that the apt-cache dump  command that you use to find java 7 packages might not be fully populated.

I rewrote the check for an extensible blacklist checking test rather.
The list of java/jre/jdk 7 packages are part of it now (maybe too much?).
But this is much more understandable and safe.
Thus, apt-get update is unnecessary. I also switched to dpkg, which removed the annoying warning from the apt CLI.


Error Mitogen was disconnected from the remote environment while a call was in-progress

We running the ansible as a container for our jenkins job and we got this error
[2022-06-21T23:18:06.030Z] fatal: [<IP_ADDRESS>]: UNREACHABLE! => {"changed": false, "msg": "Mitogen was disconnected from the remote environment while a call was in-progress. If you feel this is in error, please file a bug. Original error was: the respondent Context has disconnected", "unreachable": true}

Which version of Ansible are you running?
ansible:2.10-alpine-3.13

I got the same message while launching a docker container.
After that the playbook to that node would never complete getting a different error.
failed: [<IP_ADDRESS>] (item=docker-registry-default.gsm-test.lab.c-cor.com/gsm/memcached:1.6) => {"changed": false, "item": "docker-registry-default.gsm-test.lab.c-cor.com/gsm/memcached:1.6", "msg": "Failed to import docker or docker-py - u'chardet' is present in the Mitogen importer blacklist, therefore this context will not attempt to request it from the master, as the request will always be refused.. Try pip install docker or pip install docker-py (Python 2.6)"}
Ansible version being run is old:
ansible-playbook --version
ansible-playbook 2.6.2
config file = /etc/ansible/ansible.cfg
configured module search path = [u'/var/lib/awx/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
ansible python module location = /usr/lib/python2.7/site-packages/ansible
executable location = /usr/bin/ansible-playbook
python version = 2.7.5 (default, Jul 13 2018, 13:06:57) [GCC 4.8.5 20150623 (Red Hat 4.8.5-28)]
Host node has pip docker installed as part of the playbook
Running: mitogen-0.2.9

Did you try this fix?
https://github.com/mitogen-hq/mitogen/issues/593#issuecomment-498094053


Release google-cloud-java v0.108.0

This pull request was generated using releasetool.
09-05-2019 09:50 PDT
Bigtable

Bigtable: Add an integration test for large rows (#6204)
Bigtable: Adding ABORTED as default retry code (#6214)
Bigtable: RowMutationEntry#create for unsafe mutation creation (#6211)
fix Bigtable Emulator README formatting (#6213)
Bigtable: instrument client with Opencensus metrics (#6143)
Bigtable: Adding Integration Test of BigtableInstanceAdminClient (#6186)
Added deleteForcedAppProfile InstanceAdmin operation (#6193)

Spanner JDBC

Spanner JDBC driver is now GA! (1.0.0)
spanner-jdbc: Fix possible IllegalArgumentException in SingleUseTransaction (#6175)

Speech

Regenerate speech client - adds SpeakerDiarizationConfig (#6217)

Translate

Translate: fix sample output (#6197)

Dependencies

Update gax to 1.48.1 (#6221)
Update dependency org.apache.avro:avro to v1.9.1 (#6207)

Release is at https://github.com/googleapis/google-cloud-java/releases/tag/v0.108.0

The release build has started, the log can be viewed here. :sunflower:

The release build failed! Please investigate!

The release build has started, the log can be viewed here. :sunflower:

The release build failed! Please investigate!


7 Ways to Deal with Application Secrets in Azure

The article was already reviewed but there's no publish date yet, so I'm opening the PR for a future date

@ericrudder Thanks for the suggestions! I've updated the text.


Enhancement: support lifespan state

Latest uvicorn release adds support for this. From asgi spec:

2.3.1 Scope
The lifespan scope exists for the duration of the event loop.
The scope information passed in scope contains basic metadata:
• type (Unicode string) – "lifespan".
• asgi["version"] (Unicode string) – The version of the ASGI spec.
• asgi["spec_version"] (Unicode string) – The version of this spec being used. Optional; if missing defaults to "1.0".
• state Optional(dict[Unicode string, Any]) – An empty namespace where the application can persist state to be used when handling subsequent requests. Optional; if missing the server does not support this feature.
If an exception is raised when calling the application callable with a lifespan.startup message or a scope with type lifespan, the server must continue but not send any lifespan events.
This allows for compatibility with applications that do not support the lifespan protocol. If you want to log an error that occurs during lifespan startup and prevent the server from starting, then send back lifespan.startup.failed instead.

Lifespan State
Applications often want to persist data from the lifespan cycle to request/response handling. For example, a database connection can be established in the lifespan cycle and persisted to the request/response cycle. The lifespan["state"] namespace provides a place to store these sorts of things. The server will ensure that a shallow copy of the namespace is passed into each subsequent request/response call into the application. Since the server manages the application lifespan and often the event loop as well this ensures that the application is always ac- cessing the database connection (or other stored object) that corresponds to the right event loop and lifecycle, without using context variables, global mutable state or having to worry about references to stale/closed connections.
ASGI servers that implement this feature will provide state as part of the lifespan scope:
The namespace is controlled completely by the ASGI application, the server will not interact with it other than to copy it. Nonetheless applications should be cooperative by properly naming their keys such that they will not collide with other frameworks or middleware.

so for now at least we are not going to support this. The reason is - we already have a state injected during lifespan, and this created unnecessary complications.


Opening script from Godot into VS Code un-maximizes VS Code

Godot version
4.0 RC 2
System information
Linux Mint 20.3 (Cinnamon)
Issue description
I've set Godot to open scripts in VS Code, since I'm using C#. When I open a script through Godot, it pulls it up in VS Code. However, VS Code will no longer be maximized, and instead pops the window to be square, in the center of the screen (every time).
Steps to reproduce

Create a Godot project that uses C# scripts.
Set Godot to use VS Code as the script editor.
Create a C# script in Godot.
Open the script (in Godot) and watch VS Code open up.
Maximize VS Code.
Switch back to Godot.
Open the script (in Godot) again, and watch VS Code open up, again.
Notice that VS Code is no longer maximized, and is now a free-floating window again.

Minimal reproduction project
pokemon-like-4-cs.zip

I can't reproduce, it sounds more like an issue with VSCode than Godot. Can you take a look at your settings? Maybe check window.openFilesInNewWindow, window.openFoldersInNewWindow, and window.newWindowDimensions.

Closing per the above comment.


RFE: Add cli only functionality for TOTP generation?

Is it possible to add some sort of CLI only functionality for generating TOTP codes?  I don't need the GUI/systray functionality and I'm having a hard time getting PySide to run in RHEL7.
I [wrote a quick PoC] (https://github.com/jyundt/yubioath-desktop/blob/cli-only/yubicoauthenticator/yubico_authenticator_cli.py) and and it seems to work fine on RHEL7:
[jyundt@rhel7 yubicoauthenticator]$ python yubico_authenticator_cli.py
Failed to load symbol for: SCardCancelTransaction, /lib64/libpcsclite.so.1: undefined symbol: SCardCancelTransaction!
list of readers
<function readers at 0x7f42c7080140>
using reader
Yubico Yubikey NEO OTP+CCID 00 00
Password:
success
github.com/jyundt  => ######
[jyundt@rhel7 yubicoauthenticator]$

The new rewrite (2.x.x) contains a CLI tool. For now it's quite basic, but the plan is to improve it, adding more functionality. That said, it should already do what you're looking for.


firemaking filter

Add firemaking level filter:

33 needed for Harpie Bug Swarms

13 February 2014(update) | Slayer masters no longer assign Earth warriors or Spiritual creatures to players who lacked the Agility or quest requirements to access those targets.


Fix parsing of 8-bit characters after an escape sequence.

Get upstream fix by upgrading to jawn-0.8.4.
Fixes #514.

Is this ready to go? If so, LGTM.


[MPWG Page] Add under the Legal section the EF Membership Agreement

Eclipse Foundation Membership Agreement

@rstjames,
This agreement is missing from legal :) thanks you for adding it!

done


build docs on each PR

It is minimal extra CI time to build docs on each PR.

Related: https://github.com/go-gitea/gitea/pull/24877

Likely conflicted by #24877.

Yeah, this is a stopgap until that could be merged.

I think https://github.com/go-gitea/gitea/pull/24877 is ready for a merge and a few test runs, but happy to merge this as well if it takes longer.


Internal Server Error 500 by using "Billing Period - List "

Hi,
I'm using the endpoint try to get the billing period but got the Internal Server Error 500 on some subscription. My subscription is MSDN/PAYG/AIO. Any idea or workaround to get the billing period? Thanks.
Document: https://docs.microsoft.com/en-us/rest/api/billing/billingperiods/list

@wilcobmsft would you know someone who can take a look at this one?

There was a known issue in one of our downstream systems. It should be resolved now. Can you please try again and let us know if you're still able to reproduce the issue?

@wilcobmsft
It works now.
May I know is there any other API also can get the billing period ?
(Because I found this issue happened several times this year, so I'm looking for other API which could be more stable)
Thanks,


OSX master branch unit test errors

These are as of b03ab1ca452b0e613d3c3f495f38d78937aa52df, split by Chrome and FireFox. Please let me know how I can contribute.
Chrome: https://pastebin.com/eJNMZNUa
Firefox: https://pastebin.com/9CsAiWgD

@andig is this one you'd be willing to send a pull request for?
also, quick note, but make sure to test against the latest commit in master. there were a few recent test fixes like https://github.com/chartjs/Chart.js/commit/c6bda02468475fc9b2548e143b64b6a8fff4139c

I have unfortunately no clue how to fix these errors. I can submit test resulta for latest master if that helps.

Sure, it'd be helpful to confirm if this is still an issue after recent changes
Unfortunately I'd have a hard time fixing this one too since I develop on Linux

I'm still seeing 40 unit test errors on ff and chrome each on my local machine. To make this a little more accessible- would you mind if I tried to beef the travis integration to automatically run the tests on OSX, too? Not sure this is entirely feasible but I could at least try?

That would be great if only unit tests (gulp test) could be ran on each platform but I'm not sure that's possible without tricky Travis configuration. We don't want Travis to execute everything multiple time, it will take too much time but more importantly, it will break the release process since it will try to deploy multiple time to GH and npm (not good).

Interesting that it's still broken on OSX. I tested the latest stuff locally and the errors I was seeing are gone.
@andig what version of OSX, Chrome, and FF do you have on your system?

OSX: Sierra 10.12.6
Chrome:  59.0.3071.115
Firefox: 47.0.1 (rarely used, I'll check later version, too)
I'll check if I can wip up a simple, unit-test only OSX build to run by default.

Here is a hacked-up version of the travis build for linux plus osx: https://travis-ci.org/andig/Chart.js/builds/260713745
The osx build only runs the unit test. It is unfortunately much slower than linux as resources are scarce. It should be good enough to get a first impression though. The deployment part is currently missing but can easily be added back in.

Though the goal is not to maintain a tricky Travis config ;) We spent lot of time to get this build/deploy/release process stable, I really don't want to deal with a broken config for future release.

Thats up to you. Unfortunately travis has no straightforward way to configure this buidl :/ I could offer to run osx from my repo and post results from time to time but that would make catching errors harder.

I'm definitely down for unit tests on other platforms, I'm just a bit worry about making the Travis logic even more complex that it is now and take the risk to break the current setup. An alternative idea: we could use another CI service to run unit tests only and reserve Travis for build and deploy only?

I checked the build logs you posted in your first message and it seems that unit tests based on images comparison are failing, but we can't see the error output (image diff). Can you post a screenshot of one of these tests?

Can you post a screenshot of one of these tests?

I've look through the various config files but couldn't figure out where to find those?

You need to run gulp unittest --watch which will keep the browser(s) open. On the top/right part of the page, there is a "DEBUG" button, clicking on it will display the tests details. You should see failing image diff as described in this comment

You need to run gulp unittest --watch

Great, didn't know that. Here we go:

The other failures on chrome are more or less the same. Failed image is always showing a zoomed section of the excepted image. As if this was some case of resizing and the resize hasn't finished?

Looking at the other failing tests, it seems that all expect a canvas with half the render size, so that might be because your device pixel ratio is 2. Can you try to enforce the pixel ratio via the options at this line:
config.options.devicePixelRatio = 1;

I'll check. Funny enough I just got a working build on travis without this change: https://travis-ci.org/andig/Chart.js/builds/261027889. Makes it look more like a timing issue?

Update: you're on the right track after all.
Travis OSX: 100% success. Might be a non-retina device.
Local build without your fix:
Chrome 60.0.3112 (Mac OS X 10.12.6): Executed 574 of 578 (39 FAILED) (skipped 4) ERROR (18.529 secs / 16.01 secs)
Firefox 54.0.0 (Mac OS X 10.12.0): Executed 574 of 578 (40 FAILED) (skipped 4) (23.813 secs / 22.728 secs)

With fix:
Chrome 60.0.3112 (Mac OS X 10.12.6): Executed 574 of 578 (7 FAILED) (skipped 4) (28.539 secs / 26.799 secs)
Firefox 54.0.0 (Mac OS X 10.12.0): Executed 574 of 578 (8 FAILED) (skipped 4) (34.715 secs / 31.8 secs)

The failed tests are any variety, here is some examples:
Chrome 60.0.3112 (Mac OS X 10.12.6) Chart Retina scale (a.k.a. device pixel ratio) should scale the render size but not the "implicit" display size FAILED
        Expected render height 240 to be equal to 720
            at Object.<anonymous> (test/specs/core.controller.tests.js:606:18)
Chrome 60.0.3112 (Mac OS X 10.12.6): Executed 111 of 578 (3 FAILED) (0 secs / 4.351 secs)
Firefox 54.0.0 (Mac OS X 10.12.0): Executed 4 of 578 SUCCESS (0 secs / 0.854 secs)
Chrome 60.0.3112 (Mac OS X 10.12.6) Chart Retina scale (a.k.a. device pixel ratio) should scale the render size but not the "implicit" display size FAILED
Chrome 60.0.3112 (Mac OS X 10.12.6) Chart Retina scale (a.k.a. device pixel ratio) should scale the render size but not the "explicit" display size FAILED
        Expected render height 240 to be equal to 720
            at Object.<anonymous> (test/specs/core.controller.tests.js:623:18)
Chrome 60.0.3112 (Mac OS X 10.12.6): Executed 112 of 578 (4 FAILED) (0 secs / 4.355 secs)
Firefox 54.0.0 (Mac OS X 10.12.0): Executed 4 of 578 SUCCESS (0 secs / 0.854 secs)
Chrome 60.0.3112 (Mac OS X 10.12.6) Chart Retina scale (a.k.a. device pixel ratio) should scale the render size but not the "explicit" display size FAILED
Chrome 60.0.3112 (Mac OS X 10.12.6) Chart config.options.devicePixelRatio 3 should scale the render size but not the "implicit" display size FAILED
        Expected render height 240 to be equal to 720
            at Object.<anonymous> (test/specs/core.controller.tests.js:654:18)
Chrome 60.0.3112 (Mac OS X 10.12.6): Executed 113 of 578 (5 FAILED) (0 secs / 4.364 secs)
Firefox 54.0.0 (Mac OS X 10.12.0): Executed 4 of 578 SUCCESS (0 secs / 0.854 secs)
Chrome 60.0.3112 (Mac OS X 10.12.6) Chart config.options.devicePixelRatio 3 should scale the render size but not the "implicit" display size FAILED
Chrome 60.0.3112 (Mac OS X 10.12.6) Chart config.options.devicePixelRatio 3 should scale the render size but not the "explicit" display size FAILED

Actually, I'm also getting these failures if I apply the suggested changes locally. I'm not sure if it's possible to change the browser device aspect ratio manually, but let's try to set window.devicePixelRatio = 1 before this line.

Better:
Chrome 60.0.3112 (Mac OS X 10.12.6): Executed 574 of 578 (4 FAILED) (skipped 4) ERROR (10.216 secs / 9.661 secs)
Firefox 54.0.0 (Mac OS X 10.12.0): Executed 574 of 578 (5 FAILED) (skipped 4) (19.207 secs / 18.197 secs)

And here's the complete remaining list:
Chrome 60.0.3112 (Mac OS X 10.12.6) Chart Retina scale (a.k.a. device pixel ratio) should scale the render size but not the "implicit" display size FAILED
	Expected render height 240 to be equal to 720
	    at Object.<anonymous> (test/specs/core.controller.tests.js:606:18)
Chrome 60.0.3112 (Mac OS X 10.12.6) Chart Retina scale (a.k.a. device pixel ratio) should scale the render size but not the "explicit" display size FAILED
	Expected render height 240 to be equal to 720
	    at Object.<anonymous> (test/specs/core.controller.tests.js:623:18)
Chrome 60.0.3112 (Mac OS X 10.12.6) Chart config.options.devicePixelRatio 3 should scale the render size but not the "implicit" display size FAILED
	Expected render height 240 to be equal to 720
	    at Object.<anonymous> (test/specs/core.controller.tests.js:654:18)
Chrome 60.0.3112 (Mac OS X 10.12.6) Chart config.options.devicePixelRatio 3 should scale the render size but not the "explicit" display size FAILED
	Expected render height 240 to be equal to 720
	    at Object.<anonymous> (test/specs/core.controller.tests.js:672:18)
....
WARN: 'Deprecation warning: value provided is not in a recognized RFC2822 or ISO format. moment construction falls back to js Date(), which is not reliable across all browsers and versions. Non RFC2822/ISO date formats are discouraged and will be removed in an upcoming major release. Please refer to http://momentjs.com/guides/#/warnings/js-date/ for more info.
Arguments:
[0] _isAMomentObject: true, _isUTC: false, _useUTC: false, _l: undefined, _i: A, _f: undefined, _strict: undefined, _locale: [object Object]
Error
    at Function.createFromInputFallback (http://localhost:9876/absolute/var/folders/w5/_c0nb6n90fn96tzw04dtc6240000gn/T/3076a0bd546f5802dfa49ee68748f329.browserify?9e87072d9a5367e5dc5355bd5e852e2b1a4ad816:1978:94)
Firefox 54.0.0 (Mac OS X 10.12.0) Chart Retina scale (a.k.a. device pixel ratio) should scale the render size but not the "implicit" display size FAILED
	Expected render height 240 to be equal to 720
	@test/specs/core.controller.tests.js:606:4
Firefox 54.0.0 (Mac OS X 10.12.0) Chart Retina scale (a.k.a. device pixel ratio) should scale the render size but not the "explicit" display size FAILED
	Expected render height 240 to be equal to 720
	@test/specs/core.controller.tests.js:623:4
Firefox 54.0.0 (Mac OS X 10.12.0) Chart config.options.devicePixelRatio 3 should scale the render size but not the "implicit" display size FAILED
	Expected render height 240 to be equal to 720
	@test/specs/core.controller.tests.js:654:4
Firefox 54.0.0 (Mac OS X 10.12.0) Chart config.options.devicePixelRatio 3 should scale the render size but not the "explicit" display size FAILED
	Expected render height 240 to be equal to 720
	@test/specs/core.controller.tests.js:672:4
Arguments:
[0] _isAMomentObject: true, _isUTC: false, _useUTC: false, _l: undefined, _i: A, _f: undefined, _strict: undefined, _locale: [object Object]
TOTAL: 8 FAILED, 1140 SUCCESS
[17:03:37] 'unittest' errored after 23 s
[17:03:37] Error: Karma returned with the error code: 1

A bit confusing that the moment.js stuff is only in my local test?

Did you remove config.options.devicePixelRatio = 1;?

Ehrm no. Doing so limits it to one failure locally:
Firefox 54.0.0 (Mac OS X 10.12.0) Platform.dom event handling should notify plugins about events FAILED
        Expected 255 to be 256.
        @test/specs/platform.dom.tests.js:398:4

Still don't get why momentjs should depend on that?

That looks like to a rounding issue, you could change these tests for:
expect(notifiedEvent.x).toBeCloseToPixel(chart.width / 2);
expect(notifiedEvent.y).toBeCloseToPixel(chart.height / 2);

I don't know about the moment stuff, that's just warnings.

Success. This makes the OSX travis build obsolete as it didn't catch the retina errors anyway. Do you want me to send a PR for the changes or update yourself?

Sounds good, can you PR these changes?
Most unit tests assume a DPR  of 1 (when comparing with pixel values), same for the image based tests which compare with bitmaps acquired with a DPR of 1. There is unit tests to make sure that the DPR is correctly taken in account, so I think these changes are totally fine. I would simply add a comment above the window.devicePixelRatio = 1 to explain why.

Agree for the OSX Travis builds: we ship a JS library (not binaries) so for a given browser/version, our code should run and render the same whatever the platform is (I guess Chrome/FF have their own cross platform unit tests). If there is a difference between Chrome 59 under Linux and OSX, I think we can assume that's an issue with the browser itself, not with Chart.js.

Learned so much again, THANKS!

It makes our tests more reliable, thank you for the time spent on it.

Great find @simonbrunel @andig. This explains why I didn't see these failures on my MBP as it's old and doesn't have a retina display.


Google Plus - Account settings

Hi Pavel,
I was wondering if there is a way where I can define in the accounts settings to only display posts with a #hashtag from specific user accounts?
For example, only pull posts with #greatday from @+johnsmith
Right now, if I define @userpage and #hashtag, it will pull everything from that user's page and all posts from any user with that hashtag.
Please let me know your thoughts.  Thanks in advance! :)
NJ

Hi @njtam,
I believe that you can select posts from a given account and then use the moderation function to display only those containing a given hashtag.

Thanks Pavel!  I'll check it out.


Pass in path to bicepconfig.json as an argument

Is your feature request related to a problem? Please describe.
To customise linting rules, currently I need to place a copy of bicepconfig.json in the same folder as the bicep file. If I have many bicep files in different folders but wish to use the same rules, I will need to have a copy of the same json file in each folder which isn't great if I just want to use a common set of rules.
Describe the solution you'd like
Please can you add an argument to bicep build such as :
bicep build --file main.bicep --config \configs\bicepconfig.json

Thanks Marcin, I didn't realise bicep traverses the parent folders. I must have missed that when reviewing the documentation. I'm happy for this issue to be closed unless you can see any benefit to keeping it open?

Sounds good. I'll close it...


Can't interact with code

Challenge Create a Form Element has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.133 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

<link href="https://fonts.googleapis.com/css?family=Lobster" rel="stylesheet" type="text/css">

CatPhotoApp

Click here for cat photos.

Things cats love:

cat nip

laser pointers

lasagna

Top 3 things cats hate:

flea treatment

thunder

other cats

<input type="text" placeholder="cat photo URL">
<form action="/sum"

I cant write anything in the code , I can write only after I press backspace once but then the page on the phone image crashes saying Chrome detected unusual code

What happens if you hit Reset and start again with fresh code?
Also, make sure you text input is inside a form.

@StarixCrown Closing as a duplicate of  https://github.com/freeCodeCamp/freeCodeCamp/issues/13727.


dockerization problem

I am working on a Windows 10 pro m/c.
I have used the latest OpenCVSharp libraries (4.3.0...)
My, for instance,   webapi with aspnetcore version 3.1 works perfectly.
However when I create a linux docker container from the app and run it,  it explodes spitting out
"System.TypeInitializationException' occurred in OpenCvSharp.dll: 'The type initializer for 'OpenCvSharp.NativeMethods' threw an exception.'" message,
on the first line it meets an OpenCVSharp call  such as:
new Mat(100, 100, MatType.CV_8UC1, new Scalar(0));
Has anybody been able to run a linux container successfully  so far  from their OpenCVSharp4 application?

I think there is something wrong with the library dependencies, you can use ldd command to check dependencies of libOpenCvSharpExtern.so.
The Dockerfile I created may be helpful.
https://github.com/shimat/opencvsharp/blob/master/docker/ubuntu.18.04-x64/Dockerfile
I am aware of the following problem, but I don't have a good solution.
https://github.com/shimat/opencvsharp/issues/920
I run an application with OpenCvSharp on Linux Docker on Google AppEngine Flexible.


Correcting sample query

New query works with parser as of version 1.2.65405.4 (current version in engine)

@Benjin : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Benjin : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Remove trailing commas + rules.

Trailing commas add no "value" for correctness, and their absence is "always correct" for thing like older version of Node. Now that we actually lint all our JS files in Radium, it is easier to just remove trailing commas in lint than have different lint for transpiled src vs. "real node" root *.js files like webpack configs, etc.
Fixes #932
/cc @alexlande

👍  🚢  :shipit:

Coverage remained the same at 94.006% when pulling 9b8f6e8f04df7f581b3f43fea1b46f9d62294c8b on chore/no-trailing-commas into 95e7a4d5826bf871a3fedbf783707204efbda7b7 on master.


Performance issues on mobile device when editing polyline with many points

I'm not sure if anything can be done about it. I use Leaflet.draw to allow the user to edit GPS tracks. These tracks can be long with hundreds and thousands of points. Editing works well on desktop, but is really slow on mobile device (iPad). Dragging a handler is actually ok, but zooming in/out and panning is very slow on mobile device when all the edit handlers are showing along the polyline.
I'm using LL 1.0.3 with LL.draw 0.4.9 .
Does anyone have any optimization tips?

Sounds like you should use sampling so that your polygon is not actually
hundreds of thousands of points.
On Thu, May 11, 2017 at 10:08 AM, nhinze<EMAIL_ADDRESS>wrote:

I'm not sure if anything can be done about it. I use Leaflet.draw to allow
the user to edit GPS tracks. These tracks can be long with hundreds and
thousands of points. Editing works well on desktop, but is really slow on
mobile device (iPad). Dragging a handler is actually ok, but zooming in/out
and panning is very slow on mobile device when all the edit handlers are
showing along the polyline.
I'm using LL 1.0.3 with LL.draw 0.4.9 .
Does anyone have any optimization tips?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Leaflet/Leaflet.draw/issues/727, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAzNEpOTS7n_MgHZ1x003bltQ39cIACwks5r40BzgaJpZM4NYS2K
.

The issue is that all points on these GPS tracks need to be editable.
This project seems to have the right idea. Editing a 20,000 point polyline works smoothly on iPad. However, it looks abandonned:
https://github.com/tkrajina/leaflet-editable-polyline

Why do all 20,000 need to be editable? I can't believe that a
200,000-300,000 point polyline can't be downsampled, at least for which
vertices the the edit markers are available. In the
leaflet-editable-polyline example, you need to zoom in 9 times from a world
view before the edit markers even appear.
If you truly can't downsample for some reason, I suggest that you try to
render the edit markers on the fly somehow. E.g. only place icons that are
currently in on screen (or are within a screen or some such). A combination
of this technique with the above downsampling may be required at low zoom
levels. Its hard to say for sure without understanding your exact use case.
At any rate, Leaflet.Draw has no built-in optimization for handling your
use case, so you'll have to add something yourself.
Joe
On Thu, May 11, 2017 at 11:22 AM, nhinze<EMAIL_ADDRESS>wrote:

The issue is that all points on these GPS tracks need to be editable.
This project seems to have the right idea. Editing a 20,000 point polyline
works smoothly on iPad. However, it looks abandonned:
https://github.com/tkrajina/leaflet-editable-polyline
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Leaflet/Leaflet.draw/issues/727#issuecomment-300875414,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAzNEikd1IYTO9mvftWXIM7yEyhX9i66ks5r41HfgaJpZM4NYS2K
.


Support end to end encryption for node client

After this pr is apache/pulsar#9074 merged, we can consider enabling end-to-end encryption for the node client.
Enabling end-to-end encryption
Add producer, consumer examples and test certificates

Because this PR needs the PR apache/pulsar#9074, I will use the latest pulsar release to release a new version of the client-node at the branch e2e.

Is this merged pull request published in latest pulsar-client npm module?
I do not see these changes in pulsar-client 1.3.0

https://github.com/apache/pulsar-client-node/pull/138 Yes, it has been merged, but I think it will be released in 1.4.0 @bdkanand

Thanks a lot.
Do we have any ETA on when the 1.4.0 verison will be released?
Also, i see in Index.ts consumerConfig do not have any property of
publickey though i see you have added that in the Consumer.cc
Can you please elaborate on that?
On Mon, 12 Jul, 2021, 5:53 pm Guangning E, @.***> wrote:

apache#138 https://github.com/apache/pulsar-client-node/pull/138 Yes,
it has been merged, but I think it will be released in 1.4.0 @bdkanand
https://github.com/bdkanand
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/streamnative/pulsar-client-node/pull/86#issuecomment-878235426,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADHEZ5QX4B3HQQREQQXDBETTXLNFLANCNFSM4VP5CH7Q
.

I'm not sure when 1.4.0 will be released, but I think you could try sending an email to<EMAIL_ADDRESS>to discuss this
You don't need to configure the publicKey in consumeConfig, because the publicKey is used on the producer side to encrypt and the privateKey is used on the consumer side to decrypt, of course, you can add this configuration, but it won't be used.


When saving a customer "Account Is Active" switches back to No

It works all fine, but when I try to activate an account, the setting will not persist. When I manually set the db value to 1 and save the customer, it automatically switches back to 0. And there's no way to get it back to 1using the admin.
Running on Magento 2.2 CE

Hi.
Thanks you for you report.
I am going to check very soon and keep you updated.

Hi Enrico,
It's version 2.2.0. I've solved it in the end by saving the value in a controller of my own, but it's merely a workaround.

Hi @jurjens
OK I am going to check it on a recent version of the CE.


Stetting orgs and posts avatar icon to only 1 initial or 2 initials

What does this pull request aim to achieve? 💯
This PR just fixes the initials in posts and org profile depending on name. If name only 1 word, then the avatar will be 1 letter, but if the name is 2 words or more, then the name will be 2 letters
Please be concise and link any related issue(s):
This PR resolves #918
Here is the outcome of this PR:

Yeah, I was looking at one that we already have actually 🤔 It is on the User info utils. I might just implement it there so that way we can use it here and also on the org profile

Validate #918
Profile logo is displayed as first initials  of the first and second names.


set_3d_view doesn't set view in docs

Describe the bug
I think that set_3d_view does not work any more, all the examples in https://mne.tools/stable/generated/mne.viz.set_3d_view.html#mne.viz.set_3d_view look incorrect to me. I don't know what some looked like before but the fNIRS one is definitely now not correct.
Steps to reproduce
https://mne.tools/stable/auto_examples/visualization/plot_3d_to_2d.html#sphx-glr-auto-examples-visualization-plot-3d-to-2d-py
https://mne.tools/stable/auto_tutorials/preprocessing/plot_70_fnirs_processing.html#id1
Expected results
I expect the function to set the 3d view, but it docent seem to
Actual results
See doc links
Additional information

@GuillaumeFavelier can you have a look?

Thanks for reporting this, I'm on it!

Thanks


runtime: Caller returns different result after we upgrade go1.11.6 to go1.12.1

What version of Go are you using (go version)?

$ go version
go version go1.12.1 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/weicheng/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/weicheng/Documents/bilibili/code/go:/Users/weicheng/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/Cellar/go/1.12.1/libexec"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.12.1/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/dz/yrw4n9b168n2_w8dj8cjzdz00000gn/T/go-build579051166=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

run this demo code on go1.11.6 and go1.12.1 will get different result.
package main

import (
	"runtime"
	"strconv"
)

type Handler interface {
	Log()
}

// Handlers a bundle for hander with filter function.
type Handlers struct{}

var h Handler = Handlers{}

// funcName get func name.
func funcName(skip int) (name string) {
	if _, file, lineNo, ok := runtime.Caller(skip); ok {
		return file + ":" + strconv.Itoa(lineNo)
	}
	return "unknown:0"
}

// Log handlers logging.
func (hs Handlers) Log() {
	println(funcName(4))
}

func Error() {
	h.Log()
}

func main() {
	Error()
}

What did you expect to see?
{filepath}:35

What did you see instead?
go1.11.6 print
{filepath}:35

go1.12.1 print
/usr/local/Cellar/go/1.12.1/libexec/src/runtime/proc.go:200

looks like if skip  greater than 3 will get different result.

This is a result of eliding wrapper functions in tracebacks.
There's a wrapper function main.(*Handlers).Log which is required for the interface call. That function used to be reported in 1.11 and is no longer reported in 1.12.
Upgrading your example to print all the skips:
package main

import (
	"fmt"
	"runtime"
	"strconv"
	"strings"
)

type Handler interface {
	Log()
}

// Handlers a bundle for hander with filter function.
type Handlers struct{}

var h Handler = Handlers{}

// funcName get func name.
func funcName(skip int) (name string) {
	for i := 0; true; i++ {
		_, file, line, ok := runtime.Caller(i) // line 22
		if !ok {
			break
		}
		fmt.Printf("%d %s:%d\n", i, file[strings.LastIndex(file, "/")+1:], line)
	}
	if _, file, lineNo, ok := runtime.Caller(skip); ok {
		return file[strings.LastIndex(file, "/")+1:] + ":" + strconv.Itoa(lineNo)
	}
	return "unknown:0"
}

// Log handlers logging.
func (hs Handlers) Log() {
	println(funcName(4)) // line 36
}

func Error() {
	h.Log() // line 40
}

func main() {
	Error() // line 44
}

If you look at all the locations reported for different skips, in 1.11 you get:
0 issue31185.go:22
1 issue31185.go:36
2 issue31185.go:40
3 issue31185.go:44
4 issue31185.go:44
5 proc.go:201
6 asm_amd64.s:1333
issue31185.go:44

Notice the duplicate :44. One of those is for the call to the wrapper function main.(*Handlers).Log and one of those is for the call from the wrapper function to the user's function main.Handlers.Log.
From the user's perspective, though, the duplicate :44 is nonsense. What called itself at line 44? There's only one call there. To avoid this confusion, we decided to remove all the wrapper functions from tracebacks because their existence is implicit - there's no callsite to point to in the user's code where wrapper calls happened.
In 1.12, you get:
0 issue31185.go:22
1 issue31185.go:36
2 issue31185.go:40
3 issue31185.go:44
4 proc.go:200
5 asm_amd64.s:1337
proc.go:200

Which correctly corresponds to the callsites present in the user's code.

Thanks. This behavior change was mentioned in the release notes: https://golang.org/doc/go1.12#runtime
Closing for now given that @randall77 has provided a solution for you that takes the new behavior into account.


[copilot chat] Cannot copy to clipboard when code/text is still streaming

VS Code Version: 1.87.2
OS Version: Darwin arm64 23.0.0
Logs:

Steps to Reproduce:

Ask copilot to add comments to the open file
Try to select some code and copy it, it fails and the selection gets reset many times.

https://github.com/microsoft/vscode-copilot-release/assets/10356126/aa7f55bf-5af8-40e6-809e-f312141dc002
I have attached a screen-grab of what it looks like.

Here are some debugging traces which might be helpful, I think whats happening is that the list-renderer is replacing the DOM node without diffing it properly at this code location: https://github.com/microsoft/vscode/blob/dd42b9fda56ee3c2f37cf94941a1220c213aca4f/src/vs/workbench/contrib/chat/browser/chatListRenderer.ts#L642
So I tried using DomDiff to apply just the diff to the element, but turns out the ID of the whole Dom node is changing causing a re-render which is triggered from here: https://github.com/microsoft/vscode/blob/dd42b9fda56ee3c2f37cf94941a1220c213aca4f/src/vs/workbench/contrib/chat/browser/chatWidget.ts#L317-L332 here the ID changes because on every update to the stream we increment the dataid here: https://github.com/microsoft/vscode/blob/dd42b9fda56ee3c2f37cf94941a1220c213aca4f/src/vs/workbench/contrib/chat/common/chatViewModel.ts#L353-L355
I am not sure how to solve this proper, but I think for the code blocks being generated (since they are just monaco editors in disguise) we can avoid the re-rendering in a smart way.
I stopped debugging at this point, because I did not want to spend more time understanding the re-rendering logic. Hopefully this helps!

https://github.com/microsoft/vscode/issues/208191


Search existing customer using phone number in connect an order with an account dialog

What I'm trying to achieve
I want to attach an order to the existing customer,  where existing users are searchable using phone number. As of now I can search only using E-mail Address.
CHECKBOX: Set billing and shipping address in order to customer details, should work in similar way with phone number as it is working for E-mail.
Describe a proposed solution
Is there a way where I could simply replace or duplicate the E-Mail validation with a Customer's Phone Number which is similarly searchable like an E-Mail address!
Screenshots or mockups

The function responsible for searching users is ajax_users_list. There is a queryset of users that we filter using provided search_query. Filtering by phone number would require adding the following query:
Q(addresses__phone__icontains=search_query)

The above would search the phone number in all user's addresses.

Hey thank you for that, but one little help,
In the drop down its still showing E-mail, how can i get phone numbers there.

@Vinit-Koshti
To customize the line, you need to edit get_ajax_label of the model Account:
https://github.com/mirumee/saleor/blob/232a743bc63f5dcc4455bb025bd66733eb218670/saleor/dashboard/customer/views.py#L97-L99
https://github.com/mirumee/saleor/blob/232a743bc63f5dcc4455bb025bd66733eb218670/saleor/account/models.py#L148-L153

https://github.com/mirumee/saleor/blob/232a743bc63f5dcc4455bb025bd66733eb218670/saleor/account/models.py#L110
In models.py I have added the phone field.
phone = PossiblePhoneNumberField(unique=True, blank=True, default='')
https://github.com/mirumee/saleor/blob/232a743bc63f5dcc4455bb025bd66733eb218670/saleor/dashboard/customer/views.py#L88
My Edited Code: I have added below query in ajax_users_list
Q(phone__icontains=search_query)
After above changes, I got the below results. but I want phone number in the drop down.

now I am trying to return self.phone from below in line 153, but getting nothing in the ajax drop down.
https://github.com/mirumee/saleor/blob/232a743bc63f5dcc4455bb025bd66733eb218670/saleor/account/models.py#L148-L153


Set a descriptive name for discovered participants if none is given by the user [#11907]

For the review: Is better to go commit by commit.

The tests added in this PR are issuing GMOCK WARNINGs. Please, try to get rid of these warnings.


”Botiz” is listed in the drop-down menu

Description:
"Botiz" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Botiz" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Botiz, SATU MARE".


Events Tracked Immediately with useEffect Fail to Send

I'm was to track 404 page URLs using Plausible's suggested approach. I modified this for Next.js as follows:
function Custom404() {
  const analytics = useAnalytics();
  useEffect(() => {
    analytics('Page Not Found', {
      props: { page: document.location.pathname },
    });
  }, [analytics]);
 // ...

However, this fails to send the event when the page is loaded directly (if the user arrives at the 404 page via client side routing, it works fine). My theory is that Plausible isn't ready yet, so the event is ignored.
Possible Solution
In the 404 page tracking docs referenced above, it shows a version of the Plausible script tag that includes a .q parameter for holding a queue of events that may have fired before the script loads:
<script defer data-domain="<yourdomain.com>" src="https://plausible.io/js/plausible.js"></script>
<script>window.plausible = window.plausible || function() { (window.plausible.q = window.plausible.q || []).push(arguments) }</script>

I wonder if next-plausible could adopt such an approach to ensure the events are processed. The Plausible analytics script itself consumes this queue, so just defining it on the window may be enough. Note that I may be wrong about the root cause of this issue and the above approach may not resolve it.
Temporary Solution
I did confirm that delaying my 404 tracking code resulted in a successful tracking of the event, even when the 404 page is directly loaded from SSR.
function Custom404() {
  const analytics = useAnalytics();
  useEffect(() => {
    setTimeout(() => {
      analytics('Page Not Found', {
        props: { page: document.location.pathname },
      });
    }, 300);
  }, [analytics]);
 // ...

Hi @mutewinter,
Thanks for reporting this.
I fixed it in https://github.com/4lejandrito/next-plausible/commit/7f261e9e1abe864a11d8100ffb2a555ec4a5f1c7 and published as<EMAIL_ADDRESS>Thanks again!

Fixed! Thanks for the incredibly fast update. Really appreciate how well written this library is, especially the TypeScript event types.

Thanks for your words @mutewinter!


Integrate training pages into docs

In conjunction with the launch of our online training platform and first interactive online course, we need to move the current content at https://www.cockroachlabs.com/docs/stable/training/ into our docs, or otherwise figure out what to do with it. Mostly, the pages are tutorials, some duplicative of pages already in the docs, others valuable additions.

Decision: We'll just leave these pages live, for possibly use in classroom training, but we'll remove references to them from the site. They're already being blocked from search indexing.


randomly skipping a few shorts on it's own

the add on often skips 4 to 8 shorts before it allow one to plays to the end
default settings

Yo can you check if you extension is updated (same version number as image below.

Cause I believe old version had that issue

Hi, The extension has been updated and looks all good on my end.
If face any other issues check if updated to v 2.2.2

Hi, Thank you for fixing this and for your reply
Keep up the good work
Kind Regards
Amr
On Mon, Apr 3, 2023 at 1:27 PM Tyson3101 @.***> wrote:

Hi, The extension has been updated and looks all good on my end.
If face any other issues check if updated to v 2.2.2
—
Reply to this email directly, view it on GitHub
https://github.com/Tyson3101/Auto-Youtube-Shorts-Scroller/issues/12#issuecomment-1494149481,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZRODFHRBIVBDCVR7RZOYWLW7KX25ANCNFSM6AAAAAAWOQWHC4
.
You are receiving this because you authored the thread.Message ID:
@.***>


File Picker action button issue

Looks like the File Picker action button's are not clearing correctly when moving from the trash to the workspace tab.

This has been fixed.


[BUG] Wrong CSS variable used in degree

Description
The wrong variable is used for the degree input element width, which makes it out of uniform width with the other input elements
here's where needs to change:
https://github.com/Dun-sin/Code-Magic/blob/83576a5857bf51b843098f9b5b6e3787e9bb3c32/src/style.css#L355
here's the variable it should be replaced with
https://github.com/Dun-sin/Code-Magic/blob/83576a5857bf51b843098f9b5b6e3787e9bb3c32/src/style.css#L14
Screenshots
No response
Additional information
Tasks:

[ ] replace the content container variable with the generator width variable

🥦 Browser
Mozilla Firefox
👀 Have you checked if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read and understood the rules in the Contributing Guidelines

Are you willing to work on this issue ?
No response

can you assign me this issue?

can you assign me this issue?

Definitely, thanks for wanting to contribute, Make sure you follow the rules 👉🏽here👈🏽, else your PR will not be accepted and will be closed.
Good Luck

:tada: This issue has been resolved in version 2.22.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Null pointer when testing credentials

When testing the credentials for access to the repository, to make comments, or to update commit statuses a null pointer is occurring because the queryParameter is incorrect.

Affects 1.26

Fixed in 1.26.1


Image upload

This issue can be used to upload images to GitHub. Dragging an image into the comment box will automatically upload the image and generate a link. You can then subsequently use the link in other documents.


v2.6 flagged as malware on Windows

I noticed this on my work computer out of all places; v2.6 [35ee6cb] is flagged as malware by Windows.
From Windows Defender:
Threat detected: Program:Win32/Uwamson.A!ml
Alert level: Medium
Date: 2021-09-16 13:57
Category: Potentially Unwanted Software
Details: This program has potentially unwanted behavior.

v2.5 does not trigger Windows Defender, so I have reverted back to that version.

Hm, that doesn’t sound good… 🤔 (“Win32/Uwamson.A!ml” seems to be a trojan.)
I’ve quickly tried on Windows 10 with both the v2.5 and v2.6 binaries, and it seemed to work there without issues. But then again, I don’t know too well about all potential security features of Windows, so maybe it’s only a problem with specific security configurations? @vladdeSV is it correct that you only see this on one machine but not on another?
It’s actually odd that this is dependent on the version, because there were no substantial changes between v2.5 and v2.6, especially no new sys-calls but purely functional changes. (Change set: https://github.com/jotaen/klog/compare/v2.5...v2.6)

Right-clicking the v2.6 binary and selecting "Scan with Microsoft Defender" (on both my home PC and my work PC) gave the warning.
No warnings appeared for the v2.5 version.
Could there possibly be changes with the dependencies, or any of the tools which generated the executable?

(Sorry, forgot to mention)
I am using Windows 10, build 1909 at work, and I believe the same at home. When I get home I will verify this.

The computer I tried yesterday had Avira installed instead of Defender (apparently you can only have one scanner installed), that’s why I couldn’t reproduce it. I just tried again with Defender, and I also see the “Win32/Uwamson.A!ml” alarm now – however, for both v2.5 and v2.6 🤔.
Some background info about the build: the dependencies haven’t changed in quite a while. The binaries are build on Github in clean and disposable VMs (see here for the build of v2.6, for example). The Windows binary is build in an Ubuntu VM, not in a Windows VM, so the build VM cannot have been infected.
I am pretty sure this is a false positive – the question is just how to convince Defender of this, because it would be cool to get rid of this alarm. One way I found was to submit the file for analysis at Microsoft, but I’m not sure how successful that’ll be, and I probably would need to repeat that process on every release. I might try it anyway…

both v2.5 and v2.6

Huh. That's odd. I scanned v2.5 on my work PC just now, and on my personal PC yesterday, which both gave the result of it being clean.

I might try it anyway…
Let's hope the pattern which is used to identify that type of malware gets tweaked, so further binaries are not misidentified in the future.

The incorrect malware detection from has now been resolved, with Defender version 1.349.1263.0.
You need to update Defender as described here. I had some troubles resetting the caches, though, to make the existing alert eventually go away.
Even though I only submitted v2.6 for analysis, the alert also went away for v2.5. That’s a hopeful sign for future releases, I nevertheless try to remember to check on the next release.


World within scenario

I wish to add something like the world in cucumber.js (https://github.com/cucumber/cucumber-js/blob/main/docs/support_files/world.md)
This would allow for state to pass between steps within a scenario.
I would recommend the addition of a flag that either enables or disables this. In the scenario you then create an object that contains a Map<String, dynamic> that is passed along in the scenarios.
By wrapping this map in an object it will be able to expand it later (with for example logs like the cucumber.js has).
I think of using a key value map since this seems the most flexible for sharing whatever state you need.

Hey @daniel-deboeverie-lemon 👋
First of all, thanks for the proposal and the PR!
I see that the proposed change adds some complexity to the package, so it could make it harder to maintain, hence I'd love to understand the use case better before merging the PR.
bdd_widget_test was created as a dumb converter from human to Flutter's widget tests. To keep it simple, I'm trying to keep it as close as possible to plain tests one could write, as a developer, manually. From the top of my head, I see the following options to pass some data between steps in my tests without Gherkin/BDD:

Pass the data as a parameter to function (similar to the solution you implemented). With this option, I'm getting pure functions and predictability. Unfortunately, if we go the World route, parameters will be stored in Map<String, dynamic> which is not type-safe. The alternative is to create different Worlds, but that would make steps non-reusable between projects, as there might be conflicts because of different implementations.
Use global singleton objects to hold the data. While these singletons provide type safety, they make steps harder to maintain. Plus, now developers are responsible for clearing holders between tests, which is easy to forget or miss.
Use DI. Frankly, conceptually that's similar to option 2, but a bit more robust. In my tests, I use get_it. That allows me to reuse steps between different projects, keeping the implementation type-safe. As I use get_it in my apps as well, reset cleans global states the same moment it resets the state of other singletons that potentially might be implemented in the app.

While option 1 is elegant, as I said, it adds some complexity to the package, plus introduces a new concept that might be useful in the future or might become redundant if there are no other use cases for it.
Option 2 is poorly scalable, but option 3 works for me quite well. The final solution looks quite similar to what is described here for Java/Kotlin.
Do you think option 3 might always be used instead of 1, or maybe I'm missing some edge cases? What is your vision of the World class future, do you think it may evolve into something more than just a map of parameters?

Outside of the fact that you would need to call GetIt.I.reset() at the end of every test, it is indeed a clean solution. I was worried it might run into problems with multiple tests running concurrently, but I wasn't able to break it.
Something I imagine there might be a request for later is the ability to add logs into the world, but I think you could also do this in GetIt.
I might use this instead of the world variable I proposed. The type safety was indeed a big issue.
Thx for the input!

the ability to add logs into the world

Would you please give more details on this?

In cucumber js the logs parameter of the world is basically just a string that logs can be dumped into. With the framework we used in our project, this log also got filled by said framework, thus allowing us to use their logs in our rapport.
Since we want to keep this package as dumb as possible, and I don't see an immediate use case for us to log from the code, I don't think this would be truly used.

Hey @daniel-deboeverie-lemon,
How everything is going with your experiment of using DI instead of World? Have you found any issues with this approach? If not, do you think we may close this ticket?

Hey @olexale,
DI has worked. Thank you for the hint :)

Alrighty, as we have a solution that does the trick and is simpler to maintain, I'm closing the issue and the appropriate PR.
@daniel-deboeverie-lemon, thank you so much for the proposition!


Update back-end-server.md

Added SQL Server 2016 and a note about that it's the only version supported by SfB 2019.

@jambirk @CarolynRowe @MicrosoftHeidi Can I merge this?


Lib: SMF: Add initial transitions to HSMs and add smf_set_handled()

Brings SMF framework closer into alignment with accepted Hierarchical State Machine operation by:

Allowing 'programming by difference' by having some child states handle events and prevent propagation up to the parent run actions while others propagate events up to a common handler in a parent state.
Optionally allow initial transitions within a parent state to determine the most nested child state to transition to.
Adding a test case for CONFIG_SMF_INITIAL_TRANSITION and smf_set_handled()
Updating documentation for the new API (and fixing some references)

There was discussion in https://github.com/zephyrproject-rtos/zephyr/issues/55344 about not making the initial transition a Kconfig option, but I'm not sure of any way else of doing it without permanently adding a pointer to each smf_state entry, which is a problem for resource-constrained devices.
This does not fix https://github.com/zephyrproject-rtos/zephyr/issues/66341 but documentation has been updated to warn users of the issue.

Anything else I should do?

It would be great if you could split in several commits so that the additions of new Kconfigs and APIs are not mixed with what are "simple" documentation enhancements.

I'm not entirely sure how such a split would work. We've got about 8 instances where I replaced double-backticks with the appropriate sphinx directives, all the rest is directly relevant to the API being added. I could revert the changes to the double-backticks, then artificially add them again, but that seems silly.
Can you give me a clearer idea of what is required? I feel I'm missing the intent of this request.

I've pushed a couple commits to your branch adding what was missing for the functions to properly render as links.

I'm terribly sorry, but I think I managed to nuke your changes by not pulling them before I amended my commit message. I fixed the smf_terminate one, but can't remember where the doxygen one goes.

How's this looking now? Any more action items for me?

https://github.com/zephyrproject-rtos/zephyr/pull/66753#discussion_r1510029400
Updated. Does the new wording seem okay?

Thank you all so much for reviewing this!


"FSM failed" is not under normal condition for me - with that message it has a failure

Which node-red-contrib-modbus version are you using?
5.31.0
What happened?
"FSM failed" is not under normal condition for me - with that message it has a failure

Server
Modbus-Server Node
How can this be reproduced?
need your help to fix it
What did you expect to happen?
node warning doesn't appear anymore again
Other Information
Previously I tried to make a modbus program using npm using a code script, and succeeded in reading the data, but it had an impact on the modbus read palette

Hi, also here. It needs much more information and not just like "doesn't work". Please, follow the issue template to provide information, otherwise we can not reproduce that issue you had and can not help you.


Best Approach for CI/CD implementation

Hello Everyone,
I would like to build Jenkins pipeline which performs code checkout to till system monitoring, i need your thought process for this scenario.
any pre-requisite stuff required?
How should i start ?
How many servers do i require to achieve this ?
i also welcome any smart/quick way of establishing this pipeline through docker.
please input your answer.
when ever i make changes to petlclinic code it should go through all the above mention stages.
Thanks in advance,
Example flow should be like this : SCM + BUILD+SCAN+ARTIFACTORY +DOCKER+TERRAFORM (IF POSSIBLE)+ANSIBLE+KUBERNETES+NAGIOS.
Dont' forget to share if you have any reference tutorial/video.

This flow works for the first time (by the way that is Iac)
When it comes to regular docker world, you gotta make sure pipe line works this way:
SCM+Jenkins>dockerhub>Jenkins>K8s> Nagios
if you need this sort of architecture in Iac
You need to work on bits first and link them with Jenkins.
Firstly, how do you want to do,
Do you want to spin up all the servers by provisioning with code or just deployment.
 I’m working on it all the infra in one single click.
If you need reference better go with github repos: split your work and make it.
All the best

@sunil3456  Thanks for your input.
i agree, terraform script required for the first to sPin the servers then most of work will be on BUILD & DEPLOY. whenever there is scm change.
I need some extra clarity here why do we need jenkins again after docker hub.
SCM+Jenkins>dockerhub>Jenkins>K8s> Nagios
i assume once the build images are pushed to dockerhub(or any private registry) i will specify that app. image in  k8s yaml file. K8's will pull the image & deploy it on one of its node. if i miss anything please
correct me .
thanks,
pradeep

Ok you can specify in manifest but someone has to call kubectl command!!

@sunil3456  thanks
i need small help & have few questions on devops
how can i reach you? what's your email.
thanks in advance.


Pin go version used by gogradle

In some environments where no go binary is installed, gogradle will attempt to download it. If no version is specified, the latest is used. The latest version 1.11 has sharpened the compiler so some code changes in the cli are required to support it, otherwise it won't compile, see https://github.com/apache/incubator-openwhisk-cli/issues/380.
With this change the go version is pinned to the officially supported - 1.9

@mdeuser do you mind merging it?


[BugFix] skip fill_null_with_default for ConstColumn

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Case: select * from xxx order by NULL.

@Mergifyio backport branch-2.3

how about add down_cast check for as_column?
     template <typename Type>
     static inline typename Type::Ptr as_column(ColumnPtr value) {
+        down_cast<Type*>(value.get());
         return std::static_pointer_cast<Type>(value);
     }

Good advice, I would add a check at the next PR.


use treesitter

You could use treesitter which imo would be way cleaner to detect math areas.
this could be done with a function like this
https://github.com/max397574/nabla.nvim/blob/390d1a2a12df22e3f797c629aa82fe98ac9aa5d2/lua/nabla/utils.lua#L2-L67
you could also use get_node_range and nvim_buf_get_text or directly get_node_text to get the actual math

https://github.com/max397574/nabla.nvim/blob/3704e556f8880c843da5257714c2baa327ce1943/lua/nabla/init.lua#L935-L957
here is an example of how I implemented this and it works really goo with all the different math environments

Thank you for your suggestion. This is indeed a good idea, I've used treesitter in other of my plugins for similar purposes and it works well.
I will look into this.

Can the code sample you provided be directly pasted in the code? Or do I need other things as well?
I am asking this because without the proper treesitter parser, it won't give any results back I assume. Maybe do you know of a parser which would properly parse the file to detect math areas?

I just use the parser from the nvim-treesitter repo
I'd have to check again if the sample could be pasted directly

diff --git a/lua/nabla.lua b/lua/nabla.lua
index ee67f98..630c280 100644
--- a/lua/nabla.lua
+++ b/lua/nabla.lua
@@ -1,6 +1,9 @@
 -- Generated using ntangle.nvim
 -- local parser = require("nabla.parser")
 local parser = require("nabla.latex")
+local ts_utils = require("nvim-treesitter.ts_utils")
+local utils=require"nabla.utils"
+

 local ascii = require("nabla.ascii")

@@ -333,26 +336,50 @@ local function gen_drawing(lines)
 end

 local function popup(overrides)
-  local buf = vim.api.nvim_get_current_buf()
-  local row, col = unpack(vim.api.nvim_win_get_cursor(0))
-  local line
+    if not utils.in_mathzone() then
+        return
+    end
+    local line

-  if not row then
-    row, col = unpack(vim.api.nvim_win_get_cursor(0))
-  end
+    cur_line = line
+
+    local math_node = utils.in_mathzone()

-  local srow, scol, erow, ecol, del = find_latex_at(buf, row, col)
+    local srow, scol, erow, ecol = ts_utils.get_node_range(math_node)
+    local lines = vim.api.nvim_buf_get_text(0, srow, scol, erow, ecol, {})

-  if srow then
-    local lines = vim.api.nvim_buf_get_lines(buf, srow-1, erow, true)
-    lines[#lines] = lines[#lines]:sub(1, ecol)
-    lines[1] = lines[1]:sub(scol+1)
     line = table.concat(lines, " ")
+    line = line:gsub("%$", "")
+    line = line:gsub("\\%[", "")
+    line = line:gsub("\\%]", "")
+    line = line:gsub("^\\%(", "")
+    line = line:gsub("\\%)$", "")
+    local whitespace = string.match(line, "^(%s*)%S")
+    line = vim.trim(line)
+    if line == "" then
+        return
+    end

-  else
-    vim.api.nvim_echo({{"Please put the cursor inside an inline latex expression and try calling this function again.", "ErrorMsg"}}, false, {})
-    return
-  end
+  -- local buf = vim.api.nvim_get_current_buf()
+  -- local row, col = unpack(vim.api.nvim_win_get_cursor(0))
+  -- local line
+  --
+  -- if not row then
+  --   row, col = unpack(vim.api.nvim_win_get_cursor(0))
+  -- end
+  --
+  -- local srow, scol, erow, ecol, del = find_latex_at(buf, row, col)
+  --
+  -- if srow then
+  --   local lines = vim.api.nvim_buf_get_lines(buf, srow-1, erow, true)
+  --   lines[#lines] = lines[#lines]:sub(1, ecol)
+  --   lines[1] = lines[1]:sub(scol+1)
+  --   line = table.concat(lines, " ")
+  --
+  -- else
+  --   vim.api.nvim_echo({{"Please put the cursor inside an inline latex expression and try calling this function again.", "ErrorMsg"}}, false, {})
+  --   return
+  -- end

   local success, exp = pcall(parser.parse_all, line)

and adding what I sent above in utils.lua did the job for me

the only problem I encountered was that sometimes I'd get

and there really is a call to a function which doesn't exist
perhaps sth went wrong with ntangle there

I was able to figure that out
the function exists but it's defined below the call
this could be fixed like this:
diff --git a/lua/nabla/ascii.lua b/lua/nabla/ascii.lua
index f78ffd1..8dd34e2 100644
--- a/lua/nabla/ascii.lua
+++ b/lua/nabla/ascii.lua
@@ -7,6 +7,8 @@ local put_if_only_sub

 local put_if_only_sup

+local to_ascii
+

 local style = {
 	plus_sign = " + ",
@@ -1352,7 +1354,7 @@ function put_if_only_sup(exp, g)
   return g
 end

-local function to_ascii(exp)
+to_ascii=function(exp)
 	local g = grid:new()
 	if not exp then
 		print(debug.traceback())

the last two things here are unrelated to treesitter
I think they just happened because other math things like $$..$$ or [...] were also parsed
I can open a separate issue if you'd like to

the last two things here are unrelated to treesitter
I think they just happened because other math things like
or [...] were also parsed

Is it not? I thought using a treesitter approach would eliminate all these delimiters problems with proper parsing.

I just use the parser from the nvim-treesitter repo

Oh I didn't read the code you provided carefully but now I see it's using a latex parser. Didn't know there was one! That's wonderful, we can use that, probably the utils code can be added here without any modifications. I still need to find a way so that people not using ntangle can contribute because this is non-optimal...

the last two things here are unrelated to treesitter
I think they just happened because other math things like
or [...] were also parsed

Is it not? I thought using a treesitter approach would eliminate all these delimiters problems with proper parsing.

this has nothing todo with these delimiters
I just think that somehow other parts of the ascii code are reached which cause this error
because I don't recall ever seeing it before
and it obivously is a mistake in the code that isn't related to using treesitter or not

I just use the parser from the nvim-treesitter repo

Oh I didn't read the code you provided carefully but now I see it's using a latex parser. Didn't know there was one! That's wonderful, we can use that, probably the utils code can be added here without any modifications. I still need to find a way so that people not using ntangle can contribute because this is non-optimal...

yes ntangle is also the reason why I just didn't add those things in a draft pr xD

yes ntangle is also the reason why I just didn't add those things in a draft pr xD
understandable, although if I didn't have ntangle, I would have never written such a big plugin in the first place. So it's a sad tragedy, hopefully I/somebody comes up with something better to manage this.

this has nothing todo with these delimiters
I just think that somehow other parts of the ascii code are reached which cause this error
because I don't recall ever seeing it before
and it obivously is a mistake in the code that isn't related to using treesitter or not

Yes, maybe open a new issue for that. We can leave this issue here for the treesitter implementation in nabla.nvim.

just tested
with small adjustments to what I sent above we could also support
\begin{equation}
    ...
\end{equation}

understandable, although if I didn't have ntangle, I would have never written such a big plugin in the first place. So it's a sad tragedy, hopefully I/somebody comes up with something better to manage this.

would it be an option for you to develop in lua?

it would be quite a big step to get there since the lua code is really really messy atm
but perhaps it would be worth it because more people would/could contribute

Actually I think that using the ntangle tools, it would be quite manageable to refactor the code. Give it 1-2 days, and it would be done.
But I'm just interested by ntangle and this type of programming. It is beyond me. Like some people like to use fennel or teal. Basically, if the code would be converted to Lua, I would not have any interest in contributing to it anymore. That's my point.
I feel like ntangle allows me to program at 3x speed, so basically what you're saying is go back to 1x speed. I might just be stupid but that's how I feel about the whole thing.

But I think people contributing with normal Lua code to this repository is still not out of the question. I have some ideas I need to test.

so about the treesitter thing
should I look into how ntangle works to try to contribute it myself or will u do (I could look at the pr to see if it is what I imagined)?

I will do it if you are not particularly akin to try ntangle. I will leave a PR for you to check yes.
Just give me some time, I'm currently refactoring the whole thing, and fixing some serious issues I found through the other issues you opened. Thank you for that btw.

you're welcome
thank you for your fast responses and the great discussions
since I have quite a lot of time the next week I could look into ntangle
it's something I've never done before so it could be interesting to get into
can I hit you up if I'm stuck with the pr?

Likewise,
of course feel free to hit me up anytime.

I give up
I rly don't like this style of programming and it's not worth for me spending so much time to learn the syntax if you can do it so much faster

Fair enough, I will see if I can push some commits on top of your PR to fix that

should we perhaps make a post about this or mention it in this-week-in-neovim?
(I think the second one would be quite a good option)
since a lot of people wanted more types of math to be supported which is now the case

I'll add it to this section
https://this-week-in-neovim.org/2022/Oct/03#updates

ok for you @jbyuki ?


Updating Readme file for better documentation

Hi @neerajrathore  I would love to take on the documentation aspect of this project, detailing what the project is about. Additionally, I can create a logo something like "JavaBook".
You can view my documentation work in one of my repositories. This version maintains your original intent while enhancing clarity and professionalism.
https://github.com/priyamaggarwal18/Expense-Tracker

@priyamaggarwal18 you can start your contribution.
Before you start working on your PR, please make sure to:

⭐ Star the repository if you haven't already.
Pull the latest changes to avoid any merge conflicts.
Include the issue number in your PR description for better tracking.

I'm excited to see your contribution as part of Hacktoberfest 2024! 🎉
Don't forget to follow me on GitHub for more updates.
Happy open-source contributing! 🚀

Yes sure


Tracebacks running code

I pip installed (pip install starbars) but example code in https://pypi.org/project/starbars/1.3.0/ and this repo throw tracebacks.
example in https://pypi.org/project/starbars/1.3.0/:

example in https://github.com/elide-b/starbars/blob/cd987a90ca8c537de540eba73d89867132bbc6bf/examples/hue.py:

Hi there! I'd like to help you!. Could you please post the full command and entirety of the output (build logs, error logs, ...) as pure text formatted code blocks?
Could you also give tell me which version of Python, IPython and Jupyter notebook you're using? This might be a duplicate of https://github.com/elide-b/starbars/issues/5, where versions of seaborn under 0.13.0 don't work.


datetime stamp should be receive in 24 hour format

I have hosted my database on AWS (Singapore Region) where timezone set UTC. One of my table has timestamp datatype for a column,  and it is hold value like '2010-08-02 11:53:39.000000'.
When I run this query in workbench
select CONVERT_TZ('2015-10-28 08:02:27.062120','+00:00','+05:30') ist
its return me '2015-10-28 13:32:27.062120' which is perfect.
But when I receive data using your drive in javascript, its shows me "2015-10-28T02:32:27.062Z", which is wrong (As per my opinion), my expected value should be "2015-10-28T01:32:27.062Z", which is one hour less. and does it have any option to receive date time in 24 hours format?
EDIT:
I have set timezone = '+05:30' and  dateStrings = false in my pool option.
If I have set dateStrings = true, so its not converting date as per timezone, which is bed.
It is very urgent and very serious problem for me. Please guide me.
Thanks.

If it's still not working if you have done mysql.createConnection({ ... , timezone: '+00:00' }) to match that your server is in UTC, I'm not sure what else to do, as there isn't really enough information here for me to determine.

I want data as per my local timezone, with is +5:30 (IST, Indian Time). So, I must have to set mysql.createConnection({ ... , timezone: '+05:00' }) in driver. I am ready to set this. but when result comes in javascript, It should be in 24hour format instead in 12hour format.


feature: support time range parameters in snapshot API

Proposal
Use case. Why is this important?
Current snapshot API only supports snapshotting all TSDB blocks in the disk and head. But sometimes I just want to backup the blocks within a certain time range.
Maybe we can support start and end params in the snapshot API.
“Nice to have” is not a good use case. :)

Snapshots are really cheap as they are hardlinks, the expensive part is the head which you can already ask not to have. Is this really needed?

I don't see a reason why we don't want to support this. The point is that it is not always necessary to snapshot all the blocks. I just need to snapshot some blocks
The code change is trivial I think. Does it break something?

I think this is not needed. If we do so, our users will expect to only and exactly have from the start to the end, which we can't do with snapshots.

During the bug scrub, we felt that this would gain us very little but adds some complications prone to be misunderstood.
If anyone feels this would be useful, please speak up. Otherwise, we'll close this.


Fix Sequin boot in self-hosted

sequin-1           | ** (File.Error) could not read file "/config/playground.yml": illegal operation on a directory
sequin-1           |     (elixir 1.17.2) lib/file.ex:385: File.read!/1
sequin-1           |     (sequin 0.1.0) lib/sequin/yaml_loader.ex:40: Sequin.YamlLoader.apply!/0
sequin-1           |     nofile:1: (file)
sequin-1           |     (stdlib 6.0.1) erl_eval.erl:904: :erl_eval.do_apply/7
sequin-1           |     (elixir 1.17.2) lib/code.ex:572: Code.validated_eval_string/3
sequin-1 exited with code 1

Looks like we have a migration out of order which is causing an invalid schema name error
sequin-1           | ** (Postgrex.Error) ERROR 3F000 (invalid_schema_name) schema "sequin_config" does not exist
sequin-1           |     (ecto_sql 3.11.3) lib/ecto/adapters/sql.ex:1054: Ecto.Adapters.SQL.raise_sql_call_error/1
sequin-1           |     (elixir 1.17.2) lib/enum.ex:1703: Enum."-map/2-lists^map/1-1-"/2
sequin-1           |     (ecto_sql 3.11.3) lib/ecto/adapters/sql.ex:1161: Ecto.Adapters.SQL.execute_ddl/4
sequin-1           |     (ecto_sql 3.11.3) lib/ecto/migration/runner.ex:348: Ecto.Migration.Runner.log_and_execute_ddl/3
sequin-1           |     (elixir 1.17.2) lib/enum.ex:1703: Enum."-map/2-lists^map/1-1-"/2
sequin-1           |     (ecto_sql 3.11.3) lib/ecto/migration/runner.ex:311: Ecto.Migration.Runner.perform_operation/3
sequin-1           |     (stdlib 6.0.1) timer.erl:590: :timer.tc/2
sequin-1 exited with code 1

https://github.com/sequinstream/sequin/commit/f9b6792ee76467fb068111fcd3983d8c52fa3757

https://github.com/sequinstream/sequin/commit/bd2e99228df90d4f17cfc88d921daba1a1deaf1f


implements weighted shuffle using binary tree

Problem
Optimize weighted shuffle.
Summary of Changes
The commit implements weighted shuffle using binary tree instead of Fenwick tree.

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

Project coverage is 81.9%. Comparing base (e027a8b) to head (9558d6d).
Report is 14 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff            @@
##           master     #185    +/-   ##
========================================
  Coverage    81.8%    81.9%
========================================
  Files         838      838
  Lines      226389   226570   +181
========================================
+ Hits       185342   185564   +222
+ Misses      41047    41006    -41

This is partial port of firedancer's implementation of weighted shuffle:
https://github.com/firedancer-io/firedancer/blob/3401bfc26/src/ballet/wsample/fd_wsample.c
The rest is implemented in https://github.com/anza-xyz/agave/pull/259.

is it worth periodically caching the weighted shuffle in push_active_set.rs since it takes longer to build?

The following commit https://github.com/anza-xyz/agave/pull/259 will speed up WeightedShuffle::new as well.


VCGenerator should annotate decomposed invariants

Is your feature request related to a problem? Please describe
VCGenerator decomposes state invariants like Inv == A /\ B into smaller pieces such as Inv$0 == A and Inv$1 == B. These pieces are propagated into InvariantViolation in the counterexamples. While it works, usability is not great. It's actually quite hard to see, which part (A or B) has been violated.
Describe the solution you'd like
Annotate the violated subformula with the definitions that it includes, e.g., Inv/A or Inv/B.

Describe the impact on your work
It will definitely make it easier for me to debug invariant violations.

This has always been a hard nut. I think I have found a solution that would work with TLA+, but then what about Quint.
After several hours of trial and error, I have an amazing solution for TLA+ that has always been there. It's TLA+ labels! Look at this one:
----------- MODULE t ---------------
EXTENDS Integers

VARIABLE
    \* @type: Int;
    x

Init == x = 0

Next == x' = x + 1

Inv ==
    /\ Lemma1 :: x >= 0
    /\ Lemma2 :: x < 10
====================================

Let's check this spec:
$ apalache-mc check --inv=Inv t.tla
...
Check the trace in: [redacted]/_apalache-out/t.tla/2024-09-30T20-39-07_11757565786153457186/violation1.tla

$ tail [redacted]/_apalache-out/t.tla/2024-09-30T20-39-07_11757565786153457186/violation1.tla

(* Transition 0 to State10 *)
State10 == x = 10

(* The following formula holds true in the last state and violates the invariant *)
InvariantViolation == Lemma2 :: (x >= 10)

================================================================================

This is so simple and beautiful at the same time.
@bugarela any ideas how we could produce labels in Quint? It's a huge usability unlock, for real. We could do the same for actions, as people have been constantly asking about something like labels.

Hi, this looks amazing! What do you mean by produce here?

Have something equivalent to TLA+ labels in the Quint syntax; or
Have Quint introduce something when combining multiple invariants into q::inv that can get the translation to Apalache IR to contain the labels for each original invariant


problem about spliceColumns

there is a problem about spliceColumns function.the line 209 in this file worksheets.js .the 'r' is not array. so it can't be splice

Version: 1.11.0
.spliceColumns() does not work for me too.

+1


remove transitive dependency to webjar

I am in a company network with centralized nexus. I do not have access to camunda-public-nexus. I can not use the starter-webapp because the ce webjar is not on central.
I can not even use the "custom ee webjar" workaround since we cannot even exclude the webjar from the starter-webapp without access to the repo.
We have to remove the dependency and document how to add a central or custom webjar accordingly

this will be a breaking change ... so I switch the milestone to 2.0

No breaking change ... I introduced starter-webapp-core (without webjar) and let the old starter-webapp use it. push soon