bug: vim-sandwich stops working since 1.14

Check the following:

[X] I have checked the behavior after setting "vscode-neovim.neovimClean" in VSCode settings.
[X] I have read the vscode-neovim docs.
[X] I have searched existing issues.

Neovim version (nvim -v)
0.9.1
Operating system/version
Windows 7
Describe the bug
I have been using https://github.com/machakann/vim-sandwich in neovim and vscode-neovim since day 1. It has been working perfectly. But it stops working since v1.14 (the last version it is still working is 1.13.1).
When try to delete / replace pair of parentheses, it just alter the texts outside of the selected scope.
Steps To Reproduce

Install vim-sandwich => Plug 'machakann/vim-sandwich'
Create a *.md file with

"test_1"
"test_2"
"test_3"

Run sr"' in normal mode. It should replace " pair into ' pair.
Run sd" in normal mode. It should delete " pair

Expected Behavior
Run sr"' in normal mode. It should replace " pair into ' pair.
Run sd" in normal mode. It should delete " pair

Thanks for the report. I can repro this
Also, I've wanted a plugin like this for a while, so thanks for the rec :D

broken by #2035

Yep. Specifically f73eac673e72d0c3c62eeaadbbfc266486e7c2e5. Trying to track down why this would even affect it.

Also, I've wanted a plugin like this for a while, so thanks for the rec :D

Modern nvim replacements :): https://github.com/echasnovski/mini.nvim/blob/main/readmes/mini-surround.md https://github.com/kylechui/nvim-surround

Also, I've wanted a plugin like this for a while, so thanks for the rec :D

Modern nvim replacements :): https://github.com/echasnovski/mini.nvim/blob/main/readmes/mini-surround.md https://github.com/kylechui/nvim-surround

Most of the modules within mini.nvim look really cool (although it does not go along with my philosophy about one plugin should just do one thing XD
Do you know if vscode-neovim supports most if not all of the modules?

Do you know if vscode-neovim supports most if not all of the modules?

The simplest way to determine whether a plugin works correctly is to try it out.

although it does not go along with my philosophy about one plugin should just do one thing

You can install each plugin individually.

Do you know if vscode-neovim supports most if not all of the modules?

Yes, other than modules which provide extra UI elements.

@echaya Submit feature requests to these plugins.

If someone else wants to take this on, please feel free. I don't know when I'll have a free moment in the next couple of weeks

I saw it is fixed at v1.18.2. Will it be pushed to VScode soon (as now there is still v1.18.0), as it is a bug fix? Thank you!

@albert-ying I intend to do it as soon as I can, but it may be a bit. You can build the extension from source in the interim, if this is breaking your workflow.


Add DbContextOptionsBuilderExtensions

The new extension methods will perform the same tasks as your old code, but in a more reusable and extensible way. They allow for a DbContext to be configured from an environment variable or a secret connection string independently. This would make it easy to add more configuration sources in the future if needed.
Here's an example of how you could use these extension methods:
public class MyDbContext : DbContext
{
    protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
    {
        optionsBuilder.ConfigureFromActionConnection().ConfigureFromSecretConnection();
    }
}


[TS] Node and browser support requirements

Sort of an FYI, but here is my proposal for Node and Browser support for the TS SDK. Key points:

All Node LTS versions are supported. Node 6 is EOL as of today, so Node 6 no longer needs to be supported.
Edge is supported (as long as the version of Edge you're using is supported, some patch versions are not supported by Microsoft and thus don't have to be supported)
Firefox, Chrome, and Safari are successful at keeping people at latest, so last two versions seems sufficient.

No longer needed.


[zh] Sync kubeadm-join-phase.md and its dependent files

Updated the zh page:
content/zh-cn/docs/reference/setup-tools/kubeadm/kubeadm-join-phase.md

and its dependent files.
See preview.

/lgtm


1.9.8 bitkeep use error

Use version 1.9.8
npm install<EMAIL_ADDRESS>This PR at  https://github.com/WalletConnect/web3modal/pull/569

With stable version 2.0.0 of Web3Modal now released, we are officially dropping support for version 1.x
Due to this this issue/pr was marked for closing. It is highly recommended to upgrade as 2.x will be receiving further updates that will enable functionality for some of our newer sdks like auth and push as well as support for WalletConnect v2 (See this post about WalletConnect v1 being deprecated https://medium.com/walletconnect/walletconnect-v1-0-sunset-notice-and-migration-schedule-8af9d3720d2e)
If you need to continue using Web3Modal 1.x and require this feature/fix implemented, we suggest adding it via forking V1 branch.


Pylint integration

Is there a version of isort that is a Pylint plugin, that integrates into running only Pylint, directly?
Pylint is now more tightly integrated into PyCharm and we're leaning towards running only Pylint, instead of the existing solution of running Pylint + Flake8 + isort.

Pylint actually integrates isort directly, see: https://github.com/PyCQA/pylint/blob/master/pylint/pkginfo.py#L34.


Implement: files mkdir

Add a "files mkdir" command to tapis-cli

Reopened for testing

Working for me:
dhcp-146-6-176-225:tapis-cli-ng urrutia$ tapis files mkdir agave://data-tacc-work-urrutia/ new_dir
+--------------+---------------------------------------+
| Field        | Value                                 |
+--------------+---------------------------------------+
| name         | new_dir                               |
| uuid         |<PHONE_NUMBER>732757481-242ac113-0001-002 |
| owner        | urrutia                               |
| path         | /new_dir                              |
| lastModified | 2019-10-11T11:56:47.577-05:00         |
| source       | None                                  |
| status       | STAGING_COMPLETED                     |
| nativeFormat | dir                                   |
| systemId     | data-tacc-work-urrutia                |
+--------------+---------------------------------------+

Would be nice to support it w/o the positional spacing as well, ex:
dhcp-146-6-176-225:tapis-cli-ng urrutia$ tapis files mkdir agave://data-tacc-work-urrutia/new_dir

I will look into that, but will close the issue for now as the basic functionality has been validated. Thank you.


Feat: add unbrekable 2024 writeups

Just added my stuff. :)
Waiting on @Stefan-Radu and @SecioreanuStefanita


Mismatch rental_apps OPTIONAL vs CONDITIONALLY REQUIRED

What is the issue and why is it an issue
GBFSv2.3 as well as GBFS3.0 declare system_information.rental_apps as optional.
In contrast, the gbfs-json-schema instead deems it required in case the feeds contain rental_uris, as @testower pointed out to me in a private conversation. An example can currently be seen here in this example.
Please describe some potential solutions you have considered (even if they aren’t related to GBFS).
I suggest to declare rental_apps, rental_apps.android and rental_apps.ios explicitly CONDITIONALLY REQUIRED in the spec.
Is your potential solution a breaking change?

[ ] Yes
[ ] No
[x] Unsure

As the gbfs-json-schema is already stricter than the spec, I don't expect serious effects, but others may see this differently.

Does it actually make any sense not to provide rental_apps when you have rental_uris?


Compare technical demand response potentials against literature values

Idea
In order to get an impression of the available potential compared to some literature estimates, an idea is to create a combined stacked and group plot to compare the demand response potential to the one from selected literature sources that were evaluated in the course of my PhD project.
Suggestion
Include plot as described which can solely be build up from input data (parts of which needed from the "DR_Potentials" repository, yet / soon to be published.

For this feature, some preparatory steps are required in the above-mentioned "DR_Potentials" repo. I'll need to address them first.

Thinking about this, I think, it would make much more sense to address this in the "DR_potential" repo directly. I'll close the issue here and open one there.


iOS app crashes when uploading an image

Summary
I have a client who uses the iOS app and reports that the app crashes every time when he wants to upload an image.
Environment Information

Device Name: unknown
OS Version: unknown
Mattermost App Version: unknown, installed late december probably
Mattermost Server Version: Version: 4.4.0  (4.4.3)

Steps to reproduce
Open app, open private channel to a person, choose an image
Expected behavior
Image should be uploaded
Observed behavior
App crashes
Possible fixes
If you can, link to the line of code that might be responsible for the problem.

I can too confirm, but it seems to happen rather randomly. I had the phone on for some time, took a picture and tried to send it and the app crashed. The next time started it went just fine. I also had reports from other colleagues that the picture upload stuck (endless spinner on thumbnail).
ios 11.2.2
Mattermost Team Edition 4.6.1
App version 1.6.0

Wondering if this happen cause of a network glitch while uploading the image

Don't think so,it happened for various people on different ios devices/networks :-/

Hi @sils @hydrapolic
I'm not able to reproduce this issue on the latest version of the iOS app and Mattermost server v4.9 (released April 16th).
I'm going to close this issue off here for now but if you can still reproduce after upgrading to the latest versions, please feel free to open a new issue with reproduction steps.


EventDispatcher ignores nested events of same type

There is a problem with how EventDispatcher dispatches events of the same type.
If an event is fired with the same type as an event that is currently being dispatched, it gets ignored.
Unfortunately this breaks Event.ADDED in many circumstances, and as a result breaks several MVC libraries which are watching for views being added to the stage.
This is the problematic process flow:

DisplayObject gets added to stage
Event.ADDED is dispatched and gets caught by some system
Somewhere within the handler, a child DisplayObject is created and added to the first view
An ADDED event gets dispatched on this child, which then bubbles to the parent DisplayObject
The parent DisplayObject goes to dispatch the event, but as it is already dispatching a Event.ADDED the new event it is ignored.

The line which does the ignoring is 199 (in EventDispatcher.hx @ commit f68e4da).
Fixing this is not so trivial, as the __newEventMap/__dispatching collections are only designed to track a single dispatching event.

I intend to test these pull requests this week,
could you post a code sample showing this issue?

I probably cannot until Tuesday, apologies, but will try to then.  Would very much like to back it up with asserts (any suggestions about how to do asserts?) and tests.  Note #1105 is a different approach than #1101, I saw some other possible bugs and went with a deeper refactor. If you'd like I can base 1105 on 1101.
We had at least one instance where an addEventListener in a callback was enough to get a different unrelated type listener to fire when it wouldn't with the old code.  Will try to reduce that to a testcase.

No worries, there's a lot of bugs for me to see until then ;)

Okay, I'm having a bit of a time coming up with reliable testcases here -- hopefully @TomByrne may be able to help with a formal version of what he mentioned in the original post?
In the interim, I'm going to comment below with the points that I think might cause problems and attempt to derive some testcases from those, and revisit our most recent game codebase to see if I can extract a reliable exact pattern for a testcase from the issue we saw.
Roughly, in the original code, things that should be able to convert to tests:

Dispatching goes false too soon: The reset of dispatching at the tail of __dispatchEvent, namely the __dispatching.set (event.type, false); line, is unconditional.  Clearly we want to keep dispatching true if we're nested, so we should only unset that if we're the "outermost" dispatcher.
Possible early __newEventMap discard: In removeEventListener and the tail of __dispatchEvent, the original code can null out both __eventMap and __newEventMap at the same time based on only __eventMap being empty (!__eventMap.iterator ().hasNext ()).  It's reasonable to imagine cases where we have pending changes to __newEventMap for an unrelated type that are not committed back to a now-empty __eventMap.  I'm not sure this is actually possible in the original code, it takes some care to avoid cases where there is an empty __eventMap, but it might be given the first bullet (incorrectly dropping "are we dispatching" back to false once we exit the innermost nest).
Possible stomp over all of __dispatching: Similarly, where we null those two, we don't null out __dispatching at the same time.  The next addEventListener call will replace the current __dispatching with a new one, blasting over any dispatch-in-progress bools.
Use of not-yet-populated entry in __newEventMap: The first if (list == null) return false; return in __dispatchEvent, as pointed out by OP, is problematic: if you dispatch FooEvent, don't add or remove listeners, and dispatch it again within a listener (taking care not to infinitely recurse using a bool or somesuch), you'll get a null __newEventMap.  This is the only one I've really successfully gotten a NOT OK / ok test result from:

class Main extends Sprite
{
	var o:DisplayObject;
	var test01aCallCount = 0;
	private function test01a (e:Event):Void {
		++test01aCallCount;
		if ( test01aCallCount == 1 ) { // avoid infinite recursion, but we still should get a second call
			o.dispatchEvent( new Event("Test01Event") );
		}
	}
	public function new()
	{
		super();

		o = new DisplayObject();
		o.addEventListener( "Test01Event", test01a );
		o.dispatchEvent( new Event( "Test01Event" ) );
		trace ( (test01aCallCount == 2 ? "ok" : "NOT OK") + ": Expected call count of 2 for test01a, got " + test01aCallCount );
	}
}

It also strikes me that we shouldn't need this block at all:
			if (listener == list[index]) {

				index++;

			}

...and can just unconditionally bump index (e.g. in a for loop) if we're not accidentally mutating lists somewhere.  Full discards or replacements of lists don't trigger the behavior because the loop retains a reference to the original list, so the 1->0 and 0->1 size cases don't tend to exhibit drastically bad behavior here -- but I have a feeling there'd be a way to make that [index] out of bounds with a little trying.
Hopefully that helps to start.  I'll dive into testcases for the other issues above and the problem we saw in our codebase (up at http://fiscalship.org/ now btw, as a "yay team") tomorrow.

I'm having trouble exploiting the others, partly (fortuitously) due to the effects of "Use of not-yet-populated entry in __newEventMap".
Here's a test:
	var o:DisplayObject;
	var test02aCallCount = 0;
	var test02Sequence:String = "";
	private function test02a (e:Event):Void {
		test02Sequence += "a";
		++test02aCallCount;
		if ( test02aCallCount == 1 ) {
			test02Sequence += "(";
			o.dispatchEvent( new Event( "Test02Event" ) );
			test02Sequence += ")";

			// dispatching should still be true here, so this shouldn't modify the list we're traversing over
			// ...but it does...
			o.removeEventListener( "Test02Event", test02a );
			o.removeEventListener( "Test02Event", test02b );
			o.addEventListener( "Test02Event", test02a, false, 4 );
			o.addEventListener( "Test02Event", test02b, false, 5 );
		}
	}
	private function test02b (e:Event):Void {
		test02Sequence += "b";
	}
	private function test02c (e:Event):Void {
		test02Sequence += "c";
	}
	public function new()
	{
		super();
		o = new DisplayObject();
		// Test 02: Dispatching goes false too soon.
		// The reset of dispatching at the tail of __dispatchEvent,
		// namely the __dispatching.set (event.type, false); line,
		// is unconditional. Clearly we want to keep dispatching true
		// if we're nested, so we should only unset that if we're the
		// "outermost" dispatcher.
		o.addEventListener( "Test02Event", test02a, false, 3 );
		o.addEventListener( "Test02Event", test02b, false, 2 );
		o.addEventListener( "Test02Event", test02c, false, 1 );
		o.dispatchEvent( new Event( "Test02Event" ) );
		trace ( (test02Sequence == "a(abc)bc" ? "ok" : "NOT OK") + ": adds or removes should not mutate a list we're currently traversing; wanted a(abc)bc and got: " + test02Sequence );
	}

Note that it's tricky to get this to fail for "Dispatching goes false too soon" reasons, because that doesn't happen in this test due to the "Use of not-yet-populated entry in __newEventMap" (it'll return a()bc in that case, still not what is wanted, but for a different reason).  If you fix that naively, by just looking for an entry in __eventMap if there's none in __newEventMap, you might get something like a(abc)bac.  What we want is a(abc)bc -- the two dispatches should happen in abc order, with the inner dispatch between.
Should I be aiming for another commit that puts these tests in https://github.com/openfl/openfl/blob/develop/tests/test/openfl/events/EventTest.hx ?
Likely to get pulled off onto other projects imminently here.  =( May not be able to return to this for a bit.  Trying to get a bit more time in on it today before stopping, though.

Updated #1105 with some commented out asserts (ran our game and the above tests with these) and some more cleanups and dedupes.
Holler if you want EventTest.hx updates.

Is the following fix OK (see __newEventMap.set (type, list.copy());)?
private function __dispatchEvent (event:Event):Bool {

    // ...

    if (__dispatching.get (type) == true) {
        list = __newEventMap.get (type);
        if (list == null) return false;
        list = list.copy ();
    } else {
        list = __eventMap.get (type);
        if (list == null) return false;
        // The line below is the fix:
        __newEventMap.set (type, list.copy());
        __dispatching.set (type, true);
    }

    // ...

@nikolay-mamaev I believe it will fix (4) above in https://github.com/openfl/openfl/issues/1100#issuecomment-214902204 as it is listed there, but I worry that it hasn't gotten the other edgecases, and it's provably more work (it pessimistically assumes you need to make that list copy before anything has tried to modify the list of listeners).  JGranick mentioned that he wanted to avoid this kind of pessimistic overhead in https://github.com/openfl/openfl/pull/1105#issuecomment-221372862 .
I'm going to try to break up #1105 into a patch sequence, starting with the "don't set dispatching false until we leave the outermost dispatch" and working through the other fixes, then the refactors, but it's probably going to take a while before I get to this (I'm not even working with Haxe/OpenFL/Lime in any projects at the moment, although I am peripherally supporting a few).  =(  I've found it difficult to prove the behavior of small fixes the way it's written, hence the overhaul, deduplication, and such in #1105.

Hey guys
I fixed this use case, of two event dispatches in a row, nested. @Leander1P you raise some important points. I know that a list.copy at the beginning of a dispatch would solve all these problems, but I wanted to avoid doing so because of the burden it would put on garbage collection. That's where the "newEventMap" came from, but I know this is an imperfect system.
Do you think that using an Int for __dispatching (instead of Bool) would help solve problems of embedded dispatches inside of embedded dispatches, or we going to inevitably run into a problem of this eventually breaking if you use the right combination of add/remove event listener, plus dispatches?

@jgranick No real need to switch to Int; the nesting depth should be inherently kept in the arrangement of the stack frames.  All you need to know is when the outermost dispatch returns.  See https://github.com/openfl/openfl/pull/1105/commits/93b3e4a489d4e45b76426cf96189089a98f6cdd3#diff-093bdd3a872540af6c72eafb5386f796R261 -- one local Bool in the __dispatchEvent method, plus the Bool map (type key -> Bool value).  The introduction of this outermostDispatch bool would be the first small patch in the patch chain that leads up to #1105, if I get a chance to pull it apart, since I think it's the single most important piece.  I don't think it's the only piece, but it solves most of the actually observed problems after the cd17847 null-list thing.

Just as a for-what-it's-worth, your change at https://github.com/openfl/openfl/commit/cd17847fbaf4091374ae7da8446a98c3bffe42ee#diff-093bdd3a872540af6c72eafb5386f796R199 corresponds to this bit here in #1105: https://github.com/openfl/openfl/pull/1105/commits/93b3e4a489d4e45b76426cf96189089a98f6cdd3#diff-093bdd3a872540af6c72eafb5386f796R97


Hashing output files only when needed

I would like to hash all my js output files less the main.js file and the vendor.js file.
I have many entry points and everything is working as expected, but I would like to have a set of output of the type:
main.js
vendor.js
ldkqfjmlqjk.js
etc
my output is:
    output: {
        path: path.join(__dirname, 'dist'),
        filename: '[name].js',
        publicPath: '/'
    }

I tried something as:
    output: {
        path: path.join(__dirname, 'dist'),
        filename: '[name].js' == 'main.js' ? '[name].js' : '[hash].js',
        publicPath: '/'
    }

but this is not working. All the files are created less the main file!!
What is wrong with my approach?
Is there a proper way to achieve this?
Thanks

If you need more control per entry, consider using the multicompiler mode. You would end up with something like this:
module.exports = [{...}, {...}]; // configuration per entry

You can configure it per entry this way.
Feel free to re-open at Stack Overflow if that doesn't do the trick.


ERC721.burn is meant to be called by token owner

Currently burn can only be called by the contract owner (in ERC721_Enumerable and ERC721_Mintable), but generally if there is a burn function it's expected to be invoked by a token holder in order to burn their own token.
Some say this is the true way to experience an NFT:

In Klein’s 1958 exhibition, a collector who wished to purchase a Zone had to pay in solid gold. They were then issued a paper receipt to serve as proof of transaction. Klein also designed a ritual meant to empower the purchaser to obtain the true immaterial sense of the artwork, enabling the sensibility of the artwork to become part of the owner in a spiritual sense. During this ritual, the purchaser would burn this receipt and Klein would throw half the gold into the River Seine.

Also note that ERC721 does not have a burn function, maybe due to the misunderstanding that it requires Ownable.


README: Fix CI badge for the registry consistency tests

@DilumAluthge I couldn't quite tell what the updated link here should be. Is it ok to delete, or if update it, what should it be updated to?

Let me generate the new URL.

I've put in the updated links.


Add support to extra restraints

This PR adds support to the restraints used by HADDOCK2.4, it does so by writing them to the run.param

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (946c3cf) compared to base (ad341ba).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #31   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            6         6
  Lines          801       817   +16
=========================================
+ Hits           801       817   +16

Impacted Files
Coverage Δ

input/input.go
100.00% <ø> (ø)

dataset/dataset.go
100.00% <100.00%> (ø)

runner/jobs.go
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[GRAPH-MOD] [FEATURE] Graph based model-API

The current way of doing things such as RAG and similar models is restrictive. We might want to implement filters based on terms detected in the query string/ query image(s).
Example
query = (
    collection()
        .find({Variable('character'): {'$in': '_outputs.image.characters.0'}})
        .like({'text': Variable('question')})
)

ner = NER()
query_model = QueryModel(query, variables={'X': 'character', 'Z': 'question'})

class QueryModel:
    def predict(self, X, Z):
        ...

m = GraphModel()
m.add_edge(m.input, ner.X)
m.add_edge(m.input, query_model.X)
m.add_edge(ner, query_model.Y)

In this example:

the input is passed to the ner model (recognizes import names).
the output of the ner model is passed to the X parameter of the query_model
the input is passed to the Z parameter of the query_model

[ ] Bring created Node API inside _Predictor
[ ] #1555

We would need a way to set the outputs of the model (the node with the greatest path length?).

[ ] Discuss basic graph mode design with @jieguangzhou and @blythed
[ ] Create a basic graph mode with path planning algorithm
[ ] Add tests for graph mode
[ ] Revamp graph mode with recursive bottom up algorithm


[DOC]Add saslJaasServerRoleTokenSignerSecret parameter in security-kerberos.md

Master Issue: #15121
Preview looks good:

Documentation
Check the box below and label this PR (if you have committer privilege).
Need to update docs?

[x]  doc

Hi @liudezhi2098 , I noticed that the master code PR was merged two days ago. Does this doc PR need any updates?
If it is good to go, ping @Anonymitaet for a check and merge.

Hi @liudezhi2098 , I noticed that the master code PR was merged two days ago. Does this doc PR need any updates? If it is good to go, ping @Anonymitaet for a check and merge.

need review again, thx.

@liudezhi2098 I've renamed your PR based on the Pulsar PR Naming Guideline. Please follow it, thanks.


Unable to display data returned from Jenkins

Hello!
 I'm sure it's something stupid!

If I only have $.builds[*].number, it shows the build numbers, but somehow I cannot display result.

The JSON you posted seems to work for me:

The error you get tells me that the result of each query don't contain the same number of rows. I'm guessing that one of the number and result properties is sometimes missing from the builds.

Thanks for your reply. That's what I thought as well, but I just cannot get any result to show! Even if I do $.builds[0].result, I get nothing. I can get $.builds.[0].number. So really not sure what's happening here. I checked the response from the Jenkins API, and it has both number and result for all elements, the JSON I posted above is directly from the Jenkins API. So kind of strange it works for you, but not for me!

I'm not sure either why it's failing for you. Could you perhaps send me the result from the Query inspector. That way I can be sure we're looking at the same data.

Click the Query inspector button in the upper right corner of the query editor
Click Refresh to make a new request
Click Copy to clipboard
Paste it in this issue

Make sure that the request doesn't contain any sensitive information before pasting it

Thanks. But that's the other thing. In query inspector, when I click refresh, it keeps loading, which I again found to be strange.
Keeps being stuck on this:

Okay, figured it out. I need to pass the URL query parameters as URL encoded, else it doesn't work. Maybe something you can fix? :)

Oh! That's indeed something I can fix! 😅

Can I ask you to provide the query parameters that broke things for you, what you had to put in there instead?

Here you go:
URL: https://my-jenkins-instance.com/job/Cloud/job/attempt_iqa_promotion/job/develop/api/json
Query Parameters actual (which broke): pretty=true&tree=builds[number,result,url]{0,10}
URL Encoded (which work): pretty=true&tree=builds%5Bnumber%2Cresult%2Curl%5D%7B0%2C10%7D
Note that if I also encode '=' and '&', it doesn't work again. So only some parts need to be URL encoded.

Hey i have the same situation and my URL dose not need to be encoded (http://api.gios.gov.pl/pjp-api/rest/station/findAll)
Sample JSON:
[
    {
        "id": 114,
        "stationName": "Wrocław - Bartnicza",
        "gegrLat": "51.115933",
        "gegrLon": "17.141125",
        "city": {
            "id": 1064,
            "name": "Wrocław",
            "commune": {
                "communeName": "Wrocław",
                "districtName": "Wrocław",
                "provinceName": "DOLNOŚLĄSKIE"
            }
        },
        "addressStreet": "ul. Bartnicza"
    },
    {
        "id": 117,
        "stationName": "Wrocław - Korzeniowskiego",
        "gegrLat": "51.129378",
        "gegrLon": "17.029250",
        "city": {
            "id": 1064,
            "name": "Wrocław",
            "commune": {
                "communeName": "Wrocław",
                "districtName": "Wrocław",
                "provinceName": "DOLNOŚLĄSKIE"
            }
        },
        "addressStreet": "ul. Wyb. J.Conrada-Korzeniowskiego 18"
    }
]

@kubala156 I can reproduce it the JSON you've sent me. This is the result from the data source. There are two reasons why this doesn't work:

$.[*].id matches recursively (just learned this myself), which also adds the city ID to the id values. I fixed it by changing it to $.[:].id. We should document this in the README, but I need to read up on it a little more.
gegrLong gets parsed as a timestamp for some reason. This is a bug :)

[
  {
    "name": "id",
    "type": "number",
    "values": [
      114,
      117,
      1064,
      1064
    ]
  },
  {
    "name": "stationName",
    "type": "string",
    "values": [
      "Wrocław - Bartnicza",
      "Wrocław - Korzeniowskiego"
    ]
  },
  {
    "name": "gegrLat",
    "type": "string",
    "values": [
      "51.115933",
      "51.129378"
    ]
  },
  {
    "name": "gegrLong",
    "type": "time",
    "values": []
  }
]

Closing this with a fix for @kalpik's issue where query parameters aren't encoded properly.

Fixed in v0.6.2.

Thank you :)


Add dash full by MDK

Song Name
dash full
Artist Name
MDK
Source
Youtube
Youtube Link
https://youtu.be/ljEWEfSr0zQ
Direct File Link
No response
Song ID
10007220
Start Offset [ms]
0
Extra
No response

accept


:bug: Fixed range selection splicing text

closes Facebook/Lexical#4658
-root key range selections should not splice text, though this is appropriate for text range selections
-root key range selections seem to be the best way to handle selecting decorator + text node content, and this bug prevents that from being a good solution

If a text node is the last node in root, the content is spliced based on the root offset, which doesn't make sense.

Based on the issue you reported, having a leaf node such as TextNode immediately under RootNode is not a valid state. We do not handle invalid states on Lexical Core (this and many other cases). At some point we might introduce Schemas to identify and throw on these invalid states but we do not want to overcomplicate the logic to handle them all gracefully.

@zurfyx to be clear, it's root > paragraph > text, not text directly under root. Did you check the branch behavior? I can mock up the action in playground if needed.

https://github.com/9larsons/lexical/tree/root-selection-bug here's a branch with the behavior that you can see in this video:
https://github.com/facebook/lexical/assets/21961100/2ee3e9ae-e062-4482-96f0-1255d48fdf0b

@9larsons - Thanks for the detailed test plan! If I'm looking at this right your selection does not appear to be valid, element selection is only valid when the node is not a text node and only within the immediate parent whereas text selection is only valid on the text descendant. Hence, a root selection is only valid when you're targetting an element that's empty and an immediate descendant of root (includes DecoratorNode) or an empty RootNode
For some reason, CMD + A seems to be broken on Chrome for this specific case (it does not select) but Firefox shows a correct selection which will delete properly

See $normalizeSelection__EXPERIMENTAL

Yeah, that's interesting - I (clearly) haven't been testing with Firefox. With that, I'm also not sure how I'd implement $normalizeSelection, could you share some more details there?
It seems like the 'issue' with the selection is that a root key is used regardless of it being text or element - is that accurate? If there's a place you have more of this documented I'd very much appreciate having a reference. The concept of simply using the root offsets to create a selection that wrapped all kinds of nodes was from the other PR. This works really well except for that splice line that this PR is targeting.
It's a whole lot more difficult to look at what the first and last nodes of root are, see if their first/last descendant is text, and set the offset appropriately. I will give that a go, though, as that seems to be what's needed if I look at how the selection object is constructed in Firefox.

I think this is what is correct based on what you describe. Could you please take a look when able? https://github.com/facebook/lexical/commit/72601c2d905d25612a5650975a0e20c024beea6d

Sorry for the back and forth. I didn't realize that you were performing the selection from the youtube node 🤦♂ (I should've paid more attention to the anchor and focus)
This is the normalization bugfix I was referring to https://github.com/facebook/lexical/pull/4664 - this should also fix the issue pointed out above. It may be worth rechecking whether the insertText logic works as intended when the selection is normalized.
The reason why deletion was failing on select all was more actually more generic than just root, it turns out that our lastNode is correct, a TextNode, but the offset was not transferred to the children. Notice in the video below how offset is the number of the children under root instead of the TextNode offset
https://github.com/facebook/lexical/assets/193447/8a89f9a0-0964-4d80-be39-7212b634e82e

I think we're still a bit mixed up. I left a comment on that PR as it does fix coming from a NodeSelection but still doesn't work when the Youtubenode is first and you Cmd+A from a TextNode (ie coming from RangeSelection).
https://github.com/facebook/lexical/commit/72601c2d905d25612a5650975a0e20c024beea6d is fully functional for our uses and I think for Lexical's playground - haven't broken it yet, anyways.
With Select All it's much easier to handle whether we've got a text or decorator node so we can appropriately set the selection to text and not element. That seems to be another way to come at the issue, if we can't get the fix applied.
I figured we could use the root element selection for text nodes but that's where we're colliding with Lexical's handling. I don't think this would harm anyone and at any rate, sure has a very confusing output when you remove text from a node that isn't selected (in other words, even if it's not 'correct' selection handling, it's still buggy behavior).
I'm going to see where I can get by implementing additional adjacent node checking so I can know in a situation like:
[

](https://www.youtube.com/watch?v=b52pBaObiY0)

... I've bolded the text of what I want to select. Let's say I'm selecting going down the page. Here's the two possibilities:
in either case the anchor will be the same
anchor: {key: 'root', offset: 0, type: 'element'} (zero index of the root editor)
(this is what I presented, and to my understanding is incorrect selection handling)

Not sure why it closed.

Let me read this in detail tomorrow but happy to merge what you have for now. Mind adding some test for it to make sure it doesn't regress? Also, feel free to DM me directly on Discord

Thank you. I was looking around and am not seeing a good spot for the test - could you advise?
The trouble about creating an e2e test is that I can't recreate the range selection using root using keyboard inputs (without adding more code), but I also can't seem to use $setSelection in the e2e tests. To test, I would need to:

add two paragraph>text nodes with some content
select the first one with:
anchor: {key: 'root', offset: 0, type: 'element}
focus: {key: 'root', offset: 1, type: 'element}
delete the selection
verify the second paragraph's content is not mutated

Does https://github.com/facebook/lexical/pull/4671 solve your use case?

Getting the PRs a bit confused - copying this over (as well as the latest change to this commit) from #4586:
I know we had some conversations outside of this PR. I took another pass at this and had a few notes.
$normalizeSelection mutates the actual selection object, so using $setSelection(normalizedSelection) isn't necessary in the current state (I saw this here) - not sure if that's the intent.
You mentioned needing to do normalization - or that it'd be helpful - because following the browser behavior leads to issues/confusion. It looks like here the browser is using the leading position (e.g. if on the end of the line, it's grabbing the node ahead of it).
Without some serious changes to selection behavior (i.e. not including the subsequent node in the selection.getNodes() call when lastPoint.getNode() is the root node), I think it makes the most sense to just clean up the conditional statement.
I changed it to check if the endPoint was the root node as that's the only place we do splicing, and this accounts for the anchor & focus being reversed. Ultimately it's just more flexible.
The only other way I see to get around this is to lessen the number of nodes returned by selection.getNodes() but this will surely have impacts in a lot of places. I don't think there's a 'right' answer here as the cursor exists in two places at once in this kind of selection.

@zurfyx hey, could we revisit this or https://github.com/facebook/lexical/pull/4586? I think it would make sense to have some shift+up handling that supports decorators within Lexical, but if needed, we can handle that in our own code base. What's necessary at minimum is the change within this PR. I know it's been a while since we talked about this last so please reach out on Discord if that's easiest.


Fixed a non-critical bug with is_atom(kw[:foo]) == true

Follow-up of #201 :: bug fix.
Description
In #201 that I had provided and you have merged, there was a non-critical bug.
 This PR fixes this by using Keyword.get/3 instead, with the default value being a binary.
It probably did not bring any actual harm, because the value is being checked for not nil down the call stack, but it’s extremely inaccurate. Sorry about that.
Motivation and Context
Context: #201. Motivation: make the codebase better.
Types of changes

[x] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to
not work as expected)
[ ] Improvement/refactoring (non-breaking change that doesn't add any feature
but make things better)

Checklist

[x] Check other PRs and make sure that the changes are not done yet.
[x] The PR title is no longer than 64 characters.

@am-kantox Personally instead of "unset" I would simply check for nil, atom and for anything else raise an ArgumentError. What do you think?
        name = unquote(using_opts)[:name]

        opts =
          case name do
            nil -> opts
            mod when is_atom(mod) -> Keyword.put_new(opts, :name, mod)
            _ -> raise(ArgumentError, "option :name should be an atom got: #{inspect(name)}")
          end

@Eiji7 technically, these two approaches differ: opts[:name] returns nil both when the key is missing and when it’s explicitly set to nil.
But here it does not make any difference, yes, and I like your suggestion for being cleaner; pushed the update.

Thanks guys. Good catch. I'll review it too and get it in.

Yes. The final solution is cleaner. Thanks!


Add support for compressed path data

This adds support for compressed path data, such as used by the Lubluelu SL60D.
I haven't tested the changes to confirm they do not break on devices using non compressed data (I don't have one).
The way I handle the originX and originY offsets differs to the panel-demo reference code, but I think any other way would require more changes to the map rendering.

This should address issues #14, #15, #41 and #42

LuBlueLu SL60D
Firmware: 2.6.46
I tested this with my install, and it fixed my issues.
I can now set it up from the UI when selecting to show paths (I could set it up before without the paths or colours selected).
I have not done any additional live testing, but I now see the history of my last path.

Same here, works awesome, thanks

Can this be merged now?

many thanks, it morks perfectly <3


add name

Added my name to the README file.

Wonderful! I can't wait to participate in more open-source projects.


Update index.md

Checklist

[ ] Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
[ ] Any modified or new methods or classes have helpful JSDoc and code is thoroughly commented
[ ] The description lists all applicable issues this PR seeks to resolve
[ ] The description lists any configuration setting(s) that differ from the default settings
[ ] All tests and CI builds passing
[ ] The description lists all relevant PRs included in this release (remove this if not merging to master)
[ ] e2e tests are all passing (remove this if not merging to master)

Description
Please explain the changes you made here and, if not immediately obvious from the code, how they resolve any referenced issues (and PRs if merging to master). Be sure to include all issues being resolved and any special configuration settings that are need for the software to run properly with these changes.

sorry - trying to create polish version of documentation and for some reason, it went as PR to your repo, instead of my fork... can you please delete ?


[Fix #63] Add support for definitions and schemas

@harenson can you add a little bit of documentation about this (if possible, with an example) to the Readme?

Done.


Scroll to switch tabs: Reverse mode is strange when tabs overflow

Testing https://github.com/microsoft/vscode/issues/95952
I find the reverse setting value to be quite weird in this situation:

Have a bunch of open tabs, let's say 10
Scroll all the way to the right and open the first tab

Now, scroll up, next tab will be selected. Again, next tab will be selected. Repeat until you get to the last tab. Scroll up once more, you actually scroll up, but no tab changes. Again, you keep scrolling up. Repeat until you get to the far left of the scroll view.
It's very strange that always scrolling in the same direction will take me to the far right and then to the far left.
Is reverse really important to have as a setting? And if we reverse it, shouldn't the scrolling be reverse too? Those two should be identical right?

I like that idea of having a setting to change the scrolling direction which then has an impact on this too. I think I will go ahead and change the setting to be just a boolean.


add raven l7 dns controller

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind enhancement
/kind feature
/sig network

What this PR does / why we need it:
This controller maintains a configmap record that requests across network domains are forwarded to the Gateway node by the Raven Agent
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Does this PR introduce a user-facing change?

A configmap named edge-tunnel-nodes is mounted to CoreDNS using the hosts plugin, the DNS controller is responsible for maintaining this configmap and maps requests of acrossing pools to the x-raven-proxy-internal-svc address

other Note

/lgtm


exclude multiple fileds

my setup:
excludeFields: [$('select[name="name_select"]'), $('input[name="name_input"]')],
but does not exclude

Event though array is used, it doesn't work with array / multiple jquery objects.
You should try using single jquery object. In your case this would look like this:
excludeFields: $('select[name="name_select"], input[name="name_input"]'),


[microNPU] Fix incorrect comparison in schedulers

Bug fix in the copy_constants() and copy_luts() planning functions. The condition is meant to check whether the tensor is in the planned set but was incorrectly checking Object equality.

@manupa-arm @NicolaLancellotti @ekalda @lhutton1 @mbaret

Thanks @jacobbohlin for fixing this!
Should we add a test case too ?
I would suggest adding either a) network test using APIs b) lower_to_tir test to see if expected TIR is produced when using copy constant scheduler or c) use a TE-based test.
For a), use simliar API like this to create a small graph : https://github.com/apache/tvm/blob/aa99699e532ec8c60afd552e951d238b481c4b5d/tests/python/contrib/test_ethosu/test_codegen.py#L162-L192
and then use https://github.com/apache/tvm/blob/aa99699e532ec8c60afd552e951d238b481c4b5d/tests/python/contrib/test_ethosu/test_codegen.py#L389 this for comparison.
cc : @dchauhan-arm
For b) you could use relay snippets such as : https://github.com/apache/tvm/blob/aa99699e532ec8c60afd552e951d238b481c4b5d/tests/python/contrib/test_ethosu/test_legalize.py#L62-L79
and then use a test like this : https://github.com/apache/tvm/blob/aa99699e532ec8c60afd552e951d238b481c4b5d/tests/python/contrib/test_ethosu/test_compiler.py#L25-L44
For c) https://github.com/apache/tvm/blob/aa99699e532ec8c60afd552e951d238b481c4b5d/tests/python/contrib/test_ethosu/test_scheduler.py#L155-L166
I think for the given bug c) is smallest reproducer if provide a diamond graph while a) being a more end to end test.
cc : @lhutton1 @ekalda
What do others think ?

Thanks for fixing it @jacobbohlin! Yeah I agree with Manupa that (c) is probably the smallest unit test (though arguably also the most cryptic one), but considering that this bug showed itself only in the case of diamond graphs, it means we don't also have any integration tests for diamond graphs, so it would be good to have some in general, in case the NPU compiler struggles with it in any other stage as well (we might want to add a proper integration test in a different PR though).

@manupa-arm @ekalda I think c) sounds sensible for this PR, I will add it!

@jacobbohlin sounds good!
@ekalda agreed, we should look to add resnet style network test or a small custom diamond network.

@jacobbohlin Shall we retrigger the CI ? It seems to be stuck for unknown reason.

@manupa-arm Yes that would be good, I don't know how (or if) I can do that though.

Thanks! @ekalda @lhutton1. This is merged now!


Question: how can I use the API without linked calls?

Hello,
I have history with C and relatively new to C++ and your API is a pattern I have not seen before.  I have the test app working fine with the call like:
int ret = influxdb_cpp::builder() 			 .meas("imu") 			 .tag("track", "Home Testing") 			 .field("ax", ax) 			 .field("ay", ay) 			 .field("az", az) 			 .field("gx", gx) 			 .field("gy", gy) 			 .field("gz", gz) 			 .field("yaw", yaw) 			 .field("pitch", pitch) 			 .field("roll", roll) 			 .field("temperature", temperature) 			 .field("pressure", pressure) 			 .field("altitude", altitude) 			 .timestamp( std::chrono::duration_cast<std::chrono::nanoseconds>(clock.now().time_since_epoch()).count() ) 			 .post_http(si, &resp);
but I have a thread pool that handle a queue I have created.  The queue contain a generic payload for more than 1 type of measures.  The fields and tags are passed in as maps in the format { tag name, tag value}
So therefore I need to iterate over the map and call .field or .tag APIs.
I can't seem to figure out a calling mechanism to support this where I create the builder and then add details to it iterating over my map(s) and then calling the post_http.
Thought I would post this here for some help.  In the mean time I will dig deeper to see if I can understand the way your API works.
Thanks, Jeff

Hi,
Great to hear from you.
Your scene and suggestion is really good.
The cue I could give you is the member variant lines_.
What chain-able call finished is to build lines_ that to be send.
Apologize for my busy working these days, otherwise I'll add this feature ASAP.
Looking forward to your reply.
Thank you, Orca

Without even needing to look into the source for the datatypes, you could probably just use cpp's auto feature. In pseudocode, something along the lines of
auto& m = influxdb_cpp::builder().meas("imu").tag("track", "Home Testing");
foreach key, value in std::map {
    m = m.field(key, value);
}
...
m.post_http(si, &resp);

If you need a separate map for tags, you should create an additional loop before the fields loop as well.

I followed your suggestion, but I got an error message like this.
error: use of deleted function 'influxdb_cpp::detail::tag_caller& influxdb_cpp::detail::tag_caller::operator=(const influxdb_cpp::detail::tag_caller&)'
Could you help me?

Without even needing to look into the source for the datatypes, you could probably just use cpp's auto feature. In pseudocode, something along the lines of
auto& m = influxdb_cpp::builder().meas("imu").tag("track", "Home Testing");
foreach key, value in std::map {
    m = m.field(key, value);
}
...
m.post_http(si, &resp);

If you need a separate map for tags, you should create an additional loop before the fields loop as well.

I followed your suggestion, but I got an error message like this.
error: use of deleted function 'influxdb_cpp::detail::tag_caller& influxdb_cpp::detail::tag_caller::operator=(const influxdb_cpp::detail::tag_caller&)'
Could you help me?

Do some small modifications.
Try the code belong, remove the assignment when setting fields would be OK.
auto& m = influxdb_cpp::builder().meas("imu").tag("track", "Home Testing");
foreach key, value in std::map {
    m.field(key, value); // notice this line please
}
...
m.post_http(si, &resp);

I use the linked call, but compile error:
for (const auto& i: dataIdCountMap) {
     43             std::string dataId = i.first;
     44             map<std::string, int> countMap = dataIdCountMap[dataId];
     45
     46             auto& m = influxdb_cpp::builder().meas("gather")
     47                         .tag("system_name", systemName.c_str())
     48                         .tag("data_id", dataId.c_str());
     49             for (const auto& j: countMap) {
     50                 m.field(j.first, j.second);
     51             }
                     int ret = m.post_http(si, &resp);

error: 'struct influxdb_cpp::detail::tag_caller' has no member named 'post_http'
             int ret = m.post_http(si, &resp);
                         ^

I came up with a solution using std::optional (C++17). Rather messy, but it gets the job done.
void Test(influxdb_cpp::server_info& si)
{
   std::unordered_map<std::string, float> kvps;

   auto m = influxdb_cpp::builder();
   std::optional<std::reference_wrapper<std::decay_t<decltype(m.meas("").field("",""))>>> field;

   for (auto it = kvps.begin(); it != kvps.end(); ++it)
   {
      const auto& key = it->first;
      const auto& value = it->second;

      if (field.has_value())
         field = field->get().field(key.c_str(), value);
      else
         field = m.meas("imu").tag("track", "Home Testing").field(key.c_str(), value);
   }

   field->get().post_http(si);
}

And why tag_caller is private?

Same compilation error for me when building the following code:
 auto &m = influxdb_cpp::builder().meas("test");

    m.field("val", 10); // notice this line please
    //m.timestamp(1578569518);
    m.post_http(si, &resp);

error: ‘struct influxdb_cpp::detail::tag_caller’ has no member named ‘post_http’
m.post_http(si, &resp);
Any incoming fix for this issue ?

auto &m = influxdb_cpp::builder().meas("test");
auto & f = m.field("val", 10);
f.post_http(si, &resp);

I was having trouble with this part. To prevent using deleted references, I recommend using:
influxdb_cpp::builder builder;
auto &m = builder.meas("test");
auto & f = m.field("val", 10);
f.post_http(si, &resp);

`
auto & builder = influxdb_cpp::builder()
.meas(request->measure());
    for(int i = 0; i< request->tags_size(); i++)
    {
        auto & tag = request->tags(i);
        builder.tag(tag.key(),tag.val());
    }
//!!!!!!!!!!!!!!important, becasue field use ',' but tag use ' '
auto & field_builder = (influxdb_cpp::detail::field_caller&)builder;

    for(int i = 0; i< request->intfs_size(); i++)
    {
        auto & intf = request->intfs(i);
        builder.field(intf.key(),intf.val());
    }

    for(int i = 0; i< request->boolfs_size(); i++)
    {
        auto & boolf = request->boolfs(i);
        builder.field(boolf.key(),boolf.val());
    }
    for(int i = 0; i< request->stringfs_size(); i++)
    {
        auto & stringf = request->stringfs(i);
        builder.field(stringf.key(),stringf.val());
    }
    for(int i = 0; i< request->floatfs_size(); i++)
    {
        auto & floatf = request->floatfs(i);
        builder.field(floatf.key(),floatf.val());
    }

    std::string resp;
    int ret = field_builder
    .timestamp(request->timestamp())
    .post_http(si_, &resp);

`
After tag(), typecast to field_caller is important, because tag_caller and field_caller behave differently(delim is , or blank)

`
auto & builder = influxdb_cpp::builder()
.meas(request->measure());
    for(int i = 0; i< request->tags_size(); i++)
    {
        auto & tag = request->tags(i);
        builder.tag(tag.key(),tag.val());
    }
//!!!!!!!!!!!!!!important, becasue field use ',' but tag use ' '
auto & field_builder = (influxdb_cpp::detail::field_caller&)builder;

    for(int i = 0; i< request->intfs_size(); i++)
    {
        auto & intf = request->intfs(i);
        builder.field(intf.key(),intf.val());
    }

    for(int i = 0; i< request->boolfs_size(); i++)
    {
        auto & boolf = request->boolfs(i);
        builder.field(boolf.key(),boolf.val());
    }
    for(int i = 0; i< request->stringfs_size(); i++)
    {
        auto & stringf = request->stringfs(i);
        builder.field(stringf.key(),stringf.val());
    }
    for(int i = 0; i< request->floatfs_size(); i++)
    {
        auto & floatf = request->floatfs(i);
        builder.field(floatf.key(),floatf.val());
    }

    std::string resp;
    int ret = field_builder
    .timestamp(request->timestamp())
    .post_http(si_, &resp);

`
After tag(), typecast to field_caller is important, because tag_caller and field_caller behave differently(delim is , or blank)

auto &m = influxdb_cpp::builder().meas("test");
auto & f = m.field("val", 10);
f.post_http(si, &resp);

after above, it doesn't act,  can you help me?

I was having trouble with this part. To prevent using deleted references, I recommend using:
influxdb_cpp::builder builder;
auto &m = builder.meas("test");
auto & f = m.field("val", 10);
f.post_http(si, &resp);

As you mentioned, one field is ok, but two or more field is wrong, how to resolve it?

Hi, I have a similar need.
I noticed multiple insert is supported.
influxdb_cpp::builder()
    .meas("foo")  // series 1
    .field("x", 10)

    .meas("bar")  // series 2
    .field("y", 10.3)
    .send_udp("<IP_ADDRESS>", 8091);

I hope to implement a caching mechanism to save post_http overhead
For example:
class MyLogger {
    void Write2DB(...) {
        buffer
            .meas("foo")
            .tag("k", "v")
            .tag("x", "y")
            .field("x", 10)
            .field("y", 10.3, 2)
            .field("z", 10.3456)
            .field("b", !!10)
            .timestamp(1512722735522840439);
        bufferSize++;
        if (bufferSize > 10) {
            buffer.post_http(si);
            buffer = ...; // a new object
            bufferSize = 0;
        }
    }
    auto buffer = ...; // something like influxdb_cpp::builder
    int bufferSize = 0;
}

Is there any way to achieve this now?


Database error

I got the following errors

Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 2000, in call
return self.wsgi_app(environ, start_response)
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1991, in wsgi_app
response = self.make_response(self.handle_exception(e))
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1567, in handle_exception
reraise(exc_type, exc_value, tb)
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1988, in wsgi_app
response = self.full_dispatch_request()
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1641, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1544, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1639, in full_dispatch_request
rv = self.dispatch_request()
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1625, in dispatch_request
return self.view_functionsrule.endpoint
File "/bin/pokeminer/web.py", line 76, in pokemon_data
return json.dumps(get_pokemarkers())
File "/bin/pokeminer/web.py", line 109, in get_pokemarkers
pokemons = db.get_sightings(session)
File "/bin/pokeminer/db.py", line 143, in get_sightings
return query.all()
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/query.py", line 2613, in all
return list(self)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/query.py", line 2761, in iter
return self._execute_and_instances(context)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/query.py", line 2776, in _execute_and_instances
result = conn.execute(querycontext.statement, self._params)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/base.py", line 914, in execute
return meth(self, multiparams, params)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/sql/elements.py", line 323, in _execute_on_connection
return connection._execute_clauseelement(self, multiparams, params)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/base.py", line 1010, in _execute_clauseelement
compiled_sql, distilled_params
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/base.py", line 1146, in _execute_context
context)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/base.py", line 1341, in _handle_dbapi_exception
exc_info
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/util/compat.py", line 202, in raise_from_cause
reraise(type(exception), exception, tb=exc_tb, cause=cause)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/base.py", line 1139, in _execute_context
context)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/default.py", line 450, in do_execute
cursor.execute(statement, parameters)
OperationalError: (sqlite3.OperationalError) no such table: sightings [SQL: u'SELECT sightings.id AS sightings_id, sightings.pokemon_id AS sightings_pokemon_id, sightings.spawn_id AS sightings_spawn_id, sightings.expire_timestamp AS sightings_expire_timestamp, sightings.normalized_timestamp AS sightings_normalized_timestamp, sightings.lat AS sightings_lat, sightings.lon AS sightings_lon \nFROM sightings \nWHERE sightings.expire_timestamp > ? AND sightings.pokemon_id NOT IN (?, ?, ?, ?, ?, ?)'] [parameters:<PHONE_NUMBER>.47449, 13, 16, 19, 21, 41, 96)]

Fixed with: https://www.reddit.com/r/pokemongodev/comments/4tz66s/pokeminer_your_individual_pokemon_locations/d5lovb6.


Android Chrome omnibox and tap position issues with App-Drawer

DEMO site: https://www.teamatical.com/
If you go to home page and scroll down several screens, then use scrollup button then you should see well working button....
BUT if you use app-drawer (burger button or swipe left from right edge) and scroll down again than swipe a bit on page area to show omnibox of the browser ....BOOM then try to use scrollup button, you should see it does not work as expected and sometimes it needs to tap over...but sometimes it requires to tap under the button to make it works. Try tap under the button, so you can see positioning issue...
 FF works well because it does not hide omnibox (address bar)...
Are there any ideas what is it? How to fix?
Acually it repeats on https://shop.polymer-project.org/ too, you can swipe out the drawer and scroll page up to hide omnibox and links on drawer are missing it tap positioning...
Is it an issue of the Polymer or Browser?
Browsers Affected

[ X] Chrome Android
[X ] Edge Android
[Works well ] Firefox Android
[NOT TESTED] Safari 11
[NOT TESTED] Safari 10
[NOT TESTED] IE 11

Versions

Polymer: v2.6.0
webcomponents: v1.1.0

This issue was moved to PolymerElements/app-layout#562


cirrus: Check low-scale run in Cirrus CI.

Add a Cirrus CI configuration YAML file with the job to run a minimal
low-scale test.  It is practically not possible or extremely hard to
do the same in GitHub Actions for few reasons:

In GHA we're already inside of a container.
This container is ubuntu-based.
ovn-heater dosn't support debian-based distributions so we need to
run a nested container.
Using a 'container:' config we can run a fedora container, but the
container command is always overwritten, so no way to run a
container with systemd as init.
I failed to start any container with systemd as init manually.
'runner' is not a root, most of the operations require 'sudo',
but we will ssh to localhost without using sudo afterwards, even
if the user is replaced with 'runner' in ansible configuration.
Many more environment quirks that doesn't allow you to do what
you actually need.

Cirrus CI's default containers are similar to GHA, but with Cirrus CI
we're able to just start a plain fedora-cloud VM and have a full access
to it as root.
'runtime' is cached to speed up builds.  Command logging enabled to
simplify troubleshooting.  Container image is cached as well.
'runtime' is archived to avoid symlink breakage.
Since we're re-constructing and re-packing many parts of the 'runtime',
the cache will be re-uploaded every time.  But, at least, we're not
re-building it, which is the main part.
Added a check for already existing venv to allow re-using a cached
version.
Reported-at: https://github.com/dceara/ovn-heater/issues/126
Signed-off-by: Ilya Maximets<EMAIL_ADDRESS>

https://cirrus-ci.org/guide/quick-start/


Jupyter notebook cells are verrrrrry faint

I recently started using VSCode for Jupyter notebooks and realized that at present Rosé Pine's colors for cells and cell highlighting are barely legible. This is in both dark and light versions.
I've included screenshots of both, along with a screenshot of the Github Light theme for contrast of how legible and accessible the expectation would normally be.
I'm happy to toy with this if pointed in the right direction and submit a PR ☺, not a huge deal to switch themes while working in Jupyter for the time being, but I of course think Rosé Pine is the greatest theme of all time and would love to help make it work for this use case.

Thanks for the report! Happy to guide you through creating a PR:

Clone your forked copy of the repo, e.g. git clone<EMAIL_ADDRESS>rose-pine-vscode
Install dependencies via npm install or similar
Navigate to the notebook highlights in within _pinecone-color-theme.json
Changing this file effects all three variants. Build these variants via npm run build. Alternatively, you may npm run watch to automatically build on save.

Within _pinecone-color-theme.json you have access to our palette as variables prefixed with "$", e.g. $rose or $surface.
Ask away with any questions, this is motivation for me to put together a simpler contributing guide soon :)

awesome, makes sense, i'm on it! 🙏🏻

here's where i landed https://www.loom.com/share/4697650610dd40b591de18ce56746c79
i'm going to reimplement these changes quickly on a fresh branch because i just noticed:
a) you prefer all lowercase commit messages
b) vscode autoformatted a ton of bracket spacing that i'm sure you don't want changed


Implement ClearFeature(ENDPOINT_HALT)

This is in draft until https://github.com/greatscottgadgets/python-usb-protocol/pull/45 makes it into a release (and it probably needs a bit of cleanup), but it should work as-is for testing in the meantime.

https://github.com/greatscottgadgets/python-usb-protocol/pull/45 is now available in:  https://github.com/greatscottgadgets/python-usb-protocol/releases/tag/0.9.1

Thanks! I've made those changes.


Fixed Symfony 3.1 deprecation

This is the fix for https://github.com/okapon/DoctrineSetTypeBundle/issues/10

Included https://github.com/okapon/DoctrineSetTypeBundle/pull/11 to fix tests


File upload control : Replacing files causes duplicates

upload a file
click "Replace" and choose another file
Save and you will see a duplicate

Fixed: tested with manual tests


Feat: Add in an option to use the ajv-formats plugin

Hi vanekj
We have been working on a Workflow that needed an AJV validator to help validate our Yaml files. When searching for AJV actions which could do the job, we stumbled upon yours, which we liked. This action was easy to understand, and we thought it could be the base of what we needed.
With these Actions, we have added a few needed features to make it work within our project and could benefit others looking for the same thing. We have added in:

An option to use ajv-format plugin. Great for when you use formats not supported by AJV alone.
An option to allow matching properties. We needed this for our projects because we used "properties" and "patternProperties".
An option to show validation results in a workflow summary. This is for quick analysis when you want to avoid looking at logs.
An option allowing you to use referenced schemas within any file type to help validate.
We hope you like the changes we have made.

Hello @VirenMistrys, I am more than a pleased that somebody is using my package and contributed into it! I will go through the pull request and check on the changes. Thank you 👍🏻

Hey @vanekj. Thank you for getting back to me. There is a last-minute change which I may need to check. I will let you know when I have added it.

Hey @vanekj.
This is ready to review now. Thank you for holding the review while I fix a few things.

Hey @vanekj.
This PR is ready to be reviewed now. Can I have this review, please?
Thank you.

@VirenMistrys, sorry for the delays, I will review it by this week

Hi @VirenMistrys, thanks for your submission. I merged the pull-request and will review it locally as well to see if there is some space for refactoring. I should also add some project configurations such as linters, editorconfig, etc.


Use of zoomscreen breaks status_line

Describe the bug
Use of zoomscreen breaks status_line
Reproduction Steps

Launch the PS editor and load example game Fifteen
Add "zoomscreen 8x8" to prelude - save and run
Win the game - there is no Win message (but verbose logging indicates the status rule has triggered)

What did you expect would happen?
Win message should appear in the status line
OS & Browser version, desktop/mobile
All
Screenshots, example projects, other relevant issues, etc.
Notice that rule 127 has triggered but there is no message...

Can no longer repro after fixing #122, #123.

This is still broken on dev-release. Status line messages do not appear if zoomscreen is enabled (it doesn't work for flickscreen either). The steps above reproduce the issue. I've tested both Chrome and Firefox on MAC. There are no errors in browser console. I'm not sure how to re-open the bug.

Oops. Repro. Reopened.

It's a problem with my test program. The status line works just fine as long as the level fills the zoom/cflickscreen size, but if not it gets clipped out. Try zoomscreen 6x6.
Will fix.

Fixed.

I assume I can't test this yet. Let me know when I can. Have you also checked the status_line is consistently displayed below the play grid? I attach a video demonstrating the status_line overlaying the level itself when the H value for smoothscreen is set smaller than the level height (see the final SAVE/RUN in the video).
https://github.com/user-attachments/assets/ffb54123-5883-4250-9a52-37c3c082cf59

This appears to be a new issue specific to smoothscreen. Happy to look into it.


Is there any difference between node2vec-c and reference implementation of Node2Vec?

I was wondering if there is (algorithmically) any difference between your C++ implementation of Node2Vec and the original reference implementation from Stanford. For instance, I saw something about subsampling frequent nodes in the C++ code.
I'm asking because I tried a few other Node2Vec implementations, including the one in pytorch_geometric, and have had trouble replicating the good performance of your C++ code. So I was wondering if there is anything missing in those.

Can you link the original implementation you are mentioning explicitly? I have reimplemented the official Python code that used the Gensim library underneath. The subsampling actually comes from word2vec implementation in Gensim (for that matter, it's standard word2vec stuff). I have provided some subsampling analysis in Section 3.6 of the VERSE paper. I believe the subsampling is rather key, as it effectively shifts the positive pair distribution.

ok, that helps a lot! I didn't look into the Gensim implementation at all.
The two implementations I tried were this, based on Gensim, so it should be fine and the pytorch_geometric here which I'm pretty sure doesn't do any subsampling (unless that is done while sampling the walks).

Yeah, I tried to keep it close to Python implementation (to be honest, there is not much in Python code per se, I was just reimplementing Gensim). I believe this is still one of the fastest implementations available, since I generate the walks on-the-fly and store the precomputed walk probabilities efficiently.


Better form handling

Since diffing right now is just at the handler level, and always replaces everything beneath it, people could get a pretty jarring surprise when working with a form.  For example, they add an onBlur to an input in the form for validation, and other fields already filled in are blanked when the DOM updates with an error.
If they changed the position of the form in the DOM by some means, I also think it would be blanked.
Take an initial stab at it working for a very simple case:

A form with an email and password
The password has an onBlur validation that fails and displays an error message beneath the password field
The entered email should not be overwritten or blanked when the error message is added

It's kind of a weird space, for example in react (or in purview right now) I think you'd make each a controlled component that sets form values on blur.  I also recall this issue in liveview.  Forms are tricky!
Worth a shot anyways although I imagine there's some secret downside to diffing more.

It might also, maybe, somehow, just work?  With some browser magic?


Why promisse all instead of two awaits on navigation

I'm using pupperteer v1.19.0 and i had problems while navigating between pages.
I was trying use waitForNavigation after click, both with await, but it didn't work.
Example:
await page.click('#my-button');
await page.waitForNavigation({ waitUntil: 'networkidle0' });

However, i used this following code and it worked.
await Promise.all([
    page.click('#my-button'),
    page.waitForNavigation({ waitUntil: 'networkidle0' })
]);

The problem happened when i tried use the following code:
const text = await page.evaluate(() => {
    return document.querySelector('#my-div').innerHTML;
});

The return of that function null is.
Why it works the second way, but not in the first?

await Promise.all([
    page.waitForNavigation({ waitUntil: 'networkidle0' }),
    page.click('#my-button'),
]);
await page.waitForSelector('#my-div');
const text = await page.evaluate(() => {
    return document.querySelector('#my-div').innerHTML;
});

First, use click and waitForNavigation as the docs says:

Bear in mind that if click() triggers a navigation event and there's a separate page.waitForNavigation() promise to be resolved, you may end up with a race condition that yields unexpected results. The correct pattern for click and wait for navigation is the following:
const [response] = await Promise.all([
  page.waitForNavigation(waitOptions),
  page.click(selector, clickOptions),
]);

https://pptr.dev/#?product=Puppeteer&version=v1.19.0&show=api-pageclickselector-options

Then you should waitForSelector #my-div before your evaluate. You might avoid the waitForSelector if you use options like load or domcontentloaded in waitForNavigation, but it depends on how your page works (for example, see https://github.com/GoogleChrome/puppeteer/issues/598).

I hadn't found this documentation. Now I understand, thanks for the answer!


[release/1.6] Fix ambiguous tls fallback

Backport

Avoid TLS fallback when protocol is not ambiguous #9283
Check scheme and host of request on push redirect #9294

/retest

k8s-ci-robot now reruns GitHub Actions with the retest command?  That's new.  I just wanted the one aborted prow job to rerun...
/test pull-containerd-node-e2e

k8s-ci-robot now reruns GitHub Actions with the retest command?

@samuelkarp It was added so that the github workflows could be retriggered by the author of the PR. Otherwise, only org members/owners could retrigger a github workflow

Also, the aborted job is for the main branch. In release/1.6 we are using the pull-containerd-node-e2e-1-6 job.

Looks like it got triggered because of this?

dmcgowan changed the base branch from main to release/1.6

/test all

The integration test failure seems unrelated to the changes in this PR. Can the job be retriggered and checked?

/test all

pull-containerd-node-e2e is showing

Test started last Wednesday at 9:49 PM is still running after 42h38m9s. (more info)

So we may need to do something on Prow side. I don't think I can cancel the job. Can I?

pull-containerd-node-e2e is showing

Test started last Wednesday at 9:49 PM is still running after 42h38m9s. (more info)

So we may need to do something on Prow side. I don't think I can cancel the job. Can I?

@kzys It can be ignored, since that test is not supposed to run on this branch(release/1.6). I am not sure if github allows to merge by overriding the failing check.

I am not sure if github allows to merge by overriding the failing check.

I merged it anyway.  I don't think there's a way to clear pull-containerd-node-e2e and we can't rerun it (since it doesn't actually run for the release/1.6 branch).


Reduce bundle size (FilePicker)

Category
[ ] Enhancement
[ ] Bug
[x] Question
Version
Please specify what version of the library you are using: [ 2.10 ]
Expected / Desired Behavior / Question
How can I reduce the FilePicker footprint? Now it's bigger than whole @fluentui  for example (and any other library), actually it's size seems to be somehow ridiculous 😄 Is it really the case, and FilePicker is that big, or am I misinterpreting the results? Check out this picture (produced by webpack-bundle-analyzer).

Do you have any recommendations to optimize the bundle size?

Thank you @nbelyh for reporting it!
I will investigate if we can do some optimizations.

Hi @nbelyh,
I've updated the imports for the FilePicker. And now it looks much better.
The final size of the web part bundle that uses FilePicker decreased from 1.9 MB to 853 KB.
You can test new build using beta version

Sounds great. Will check it out 👍


fix: Master password autogen constraints to match AWS requirements

Description

The issue is mentioned in #71
Basically, when specifying the master_password, AWS has constraints that

Must be between 8 and 64 characters in length.
Must contain at least one uppercase letter.
Must contain at least one lowercase letter.
Must contain one number.
Can be any printable ASCII character (ASCII code 33-126) except ' (single quote), " (double quote), , /, or @.

Currently, we have the attribute min_special (sets as 1) which overrides the special (sets as false) and randomly assigns the constraints characters as mentioned above.
Motivation and Context

We want to use the built-in attribute create_random_password without having our custom random_password resource.
Breaking Changes

It does not break any backwards compatibility as master_password is currently under lifecycle's ignore_changes.
How Has This Been Tested?

[ ] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[ ] I have tested and validated these changes using one or more of the provided examples/* projects

[ ] I have executed pre-commit run -a on my pull request

This PR is included in version 5.0.0 :tada:


Remove async-http-client backends

async-http-client v2 is not maintained and has security vulnerabilities
async-http-client v3 lacks reactive-streams integration (see https://github.com/AsyncHttpClient/async-http-client/discussions/1925), hence the backends would either be significantly less useful, or much harder to implement; good alternatives (http-client, ok-http, armeria) exist
Closes https://github.com/softwaremill/sttp/issues/2372

Thank you!


Removing old versions from PyPi

First off, thanks for making Pushbullet.py, it's awesome.
At Home Assistant we were depending on PushBullet.py 0.7.1 as we had no time yet to upgrade to the latest version. Somehow version 0.7.1 has gone missing from PyPi resulting in an error when people tried to install PushBullet support in Home Assistant.
I don't know if you had a role in removing the version from PyPi. But if you did, could you please be aware in the future that there are probably other projects depending on older versions of your code, thanks!

Since when did it start happening? I just upload version 0.9.0 but I did not delete any old version.

We got the first report 3 days ago: balloob/home-assistant#594

If you have not upgraded yet I may be able to re-upload 0.7.1. I'll check if PyPi has some sort of automatic package removal

We have upgraded to 0.8.1 in our development branch. We should release it
within a week so I'm not too worried.
On Tue, Nov 10, 2015 at 12:27 PM, Roey Darwish Dror <
<EMAIL_ADDRESS>wrote:

If you have not upgraded yet I may be able to re-upload 0.7.1. I'll check
if PyPi has some sort of automatic package removal
—
Reply to this email directly or view it on GitHub
https://github.com/randomchars/pushbullet.py/issues/55#issuecomment-155555837
.

--
PaulusSchoutsen.nl
It's nice to be important but it's more important to be nice.


Include error context in expectStatus message

From time to time I stumble on a 500 error when running the tests locally. The message of a 500 error is often quite unhelpful and hard to debug. Having the context (if it is available) would save a lot of time and doesn't increase the test output that much.
Example test output after this change:
Expected statusCode 201, got statusCode 500 POST request on /ghost/api/admin/members/
Internal server error, cannot save member.
offer is not defined on the model.

Codecov Report

Merging #61 (04233c3) into main (c389da5) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #61      +/-   ##
==========================================
+ Coverage   89.91%   89.92%   +0.01%
==========================================
  Files          46       46
  Lines        2915     2919       +4
  Branches      406      406
==========================================
+ Hits         2621     2625       +4
  Misses        290      290
  Partials        4        4

Impacted Files
Coverage Δ

packages/express-test/lib/expect-request.js
100.00% <100.00%> (ø)

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 c389da5...04233c3. Read the comment docs.


error please help

error after logo apple shutdown
lenovo l390

I think that you forgot to edit your config.plist to add/remove the new values after an update. Maybe this guide could be useful for you: https://dortania.github.io/OpenCore-Post-Install/universal/update.html#_2-mount-your-efi
it shouldn't be anything impossible to solve, and in any case it doesn't depend on the platform (Lenovo) but from a problem in the config.plist file


RR-501 fix bug for checking for the step index on step view

Description
Fix bug for checking for the step index on step view
Bug comment: https://dsdmoj.atlassian.net/browse/RR-501?focusedCommentId=356513

Your fix here might be right, but I remember putting that code in place (in that order) specifically for the main create journey (I think); and I want us to be sure we dont introduce a regression in that by making this fix.
You are probably right in what you've done here, but let's look at it together on Monday 👍


Fix removeExporterChangeListener NPE

What is the purpose of the change
(16:54:21) WARNING:  [DUBBO] Unexpected error occurred when destroy invoker of ReferenceConfig(null)., dubbo version: <IP_ADDRESS>-aliyun, current host: <IP_ADDRESS>, error code: 5-3. This may be caused by , go to https://dubbo.apache.org/faq/5/3 to find instructions.
(16:54:21) java.lang.NullPointerException
(16:54:21) 	at org.apache.dubbo.rpc.listener.InjvmExporterListener.removeExporterChangeListener(InjvmExporterListener.java:125)
(16:54:21) 	at org.apache.dubbo.rpc.cluster.support.wrapper.ScopeClusterInvoker.destroy(ScopeClusterInvoker.java:101)
(16:54:21) 	at org.apache.dubbo.registry.client.migration.MigrationInvoker.destroy(MigrationInvoker.java:329)
(16:54:21) 	at org.apache.dubbo.config.ReferenceConfig.destroy(ReferenceConfig.java:246)
(16:54:21) 	at org.apache.dubbo.config.utils.SimpleReferenceCache.destroyReference(SimpleReferenceCache.java:269)
(16:54:21) 	at org.apache.dubbo.config.utils.SimpleReferenceCache.lambda$destroyAll$18(SimpleReferenceCache.java:256)
(16:54:21) 	at java.util.concurrent.ConcurrentHashMap.forEach(ConcurrentHashMap.java:1597)
(16:54:21) 	at org.apache.dubbo.config.utils.SimpleReferenceCache.destroyAll(SimpleReferenceCache.java:254)

Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

@MieAh PTAL

Codecov Report

:exclamation: No coverage uploaded for pull request base (3.2@c2b809a). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 6d693a5 differs from pull request most recent head fcf610b. Consider uploading reports for the commit fcf610b to get more accurate results

@@          Coverage Diff           @@
##             3.2   #12015   +/-   ##
======================================
  Coverage       ?   69.19%
  Complexity     ?        2
======================================
  Files          ?     1559
  Lines          ?    64335
  Branches       ?     9386
======================================
  Hits           ?    44514
  Misses         ?    15545
  Partials       ?     4276

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


IEEE 754 calculator

Feature request: add IEEE 754 calculator. Similar to http://www.weitz.de/ieee, but with more features (e.g. showing exceptions raised, etc.).

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

This appears to be a duplicate of #1681, so going to close this issue to focus the discussion there.


tab bar icon image size

image showing very small ..
https://www.dropbox.com/s/415v0uab4w0a8xf/Screenshot 2016-09-29 01.28.49.png?dl=0

@smn-khan just to confirm, the dropbox image is the undesired size tab bar, right?

@smn-khan - can you send me those pictures so I can try it out? You may need to use higher resolution images


Audio playing on tab looks blurry and overlaps with next tab

This doesn't look so great:

The default icon just isn't very good on different backgrounds. I've changed it to match Chrome's behaviour so you'll see just the icon (which is now Chrome-style) in pinned tabs, no favicon. There's basically no way to smush the icon in there without it overlapping something and looking cluttered so I think this should be fine unless you have multiple pinned tabs emitting sound, but that seems like a very unlikely scenario. It also now matches the close tab button's hover and pressed states.


WIP: handle no response for broadcast

Sending a broadcast command should result in a None response instead of panicking. This switches Result<Response> to Result<Option<Response>> to handle this edge case.
I'm looking for feedback on this approach. Another approach could be splitting call into call_with_response and call_without_response to push the choice of broadcast/non-broadcast farther up to the client

Fixed by #141.


fix: rest api body binary tab placeholder updated

Description
This PR updates the REST API body tab binary's placeholder text to indicate that this requires input as file contents in base64 format. This can help users in debugging in case REST API file upload does not work out for them.
New placeholder text changes are as below:

Fixes #33266
or
Fixes Issue URL

[!WARNING]
If no issue exists, please create an issue first, and check with the maintainers if the issue is valid.

Automation
/ok-to-test<EMAIL_ADDRESS>:mag: Cypress test results

[!CAUTION]
If you modify the content in this section, you are likely to disrupt the CI result for your PR.

Communication
Should the DevRel and Marketing teams inform users about this change?

[ ] Yes
[ ] No

/build-deploy-preview skip-tests=true


[BUG] Getting capacity leads to compilation error

Describe the bug
When calling capacity() a compilation error in line 214 is generated.
The type 'size't cannot be stored in T (in my case uint8_t)
The line needs to be changed to
m_data[0] = static_cast((s << 1U) | 1U);

should be fixed, can you give the latest version a try: https://github.com/martinus/svector/blob/main/include/ankerl/svector.h


Force update from Constraints

This PR establishes the pipelines discussed:

Assignment of form diagram constraints
Reflect constraints on force diagram
Update force diagram from constraints
Compute form from force.

It brings new additions to points 2 and 3. These new additions are summarised below:
The main function is compas_ags.ags.graphstatics.force_update_from_constraints.
This function calls a new version of the parallelise edges at compas_ags.ags.core.parallelise_edges.
The constraints in the force are managed by nicely-named attributes and the method force.constraints_from_dual().

@brgcode hi, can I give an up on this request for review?

will have a look on monday


More explaination

Hello.
I have tried to use your api as you have explained in your README but failed.
I have a iBox2 of mi-light as a wifi box connected to my network.
I have created a WifiBox with the ip of my iBox2.
When I try to do box.on() or any other command nothing happen.
I don’t have any error but no light turns ooh.
Can you explain me exactly how to install and use your api?
thanks

Thanks for trying my API! This was not tested with the iBox2, but only with the older version. The tested boxes look like this: https://cdn.webshopapp.com/shops/38765/files/27386857/mi-light-milight-wifi-empfaenger-controller-box.jpg
I'm not sure whether the interface is the same for both versions of the box, but could easily imagine that they changed the format. I would recommend to try connecting with the app using the same box first. If you can't get your lights to switch on/off after that it's likely to be an incompatible device. Calling box.on() is everything that should be required.

Thanks for answering me.
I have tried every configuration possible with the iBox2 without success.
A simple python library make the job so I oriented my choice with this solution
Le 4 mars 2021 à 00:02, Stefan Toman<EMAIL_ADDRESS>a écrit :

Thanks for trying my API! This was not tested with the iBox2, but only with the older version. The tested boxes look like this: https://cdn.webshopapp.com/shops/38765/files/27386857/mi-light-milight-wifi-empfaenger-controller-box.jpg
I'm not sure whether the interface is the same for both versions of the box, but could easily imagine that they changed the format. I would recommend to try connecting with the app using the same box first. If you can't get your lights to switch on/off after that it's likely to be an incompatible device. Calling box.on() is everything that should be required.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/stoman/MilightAPI/issues/2#issuecomment-790134919, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJSI3BEEZ2LMN5EJLTIDHFDTB25Z3ANCNFSM4W7DCWOQ.

It's great to hear you found another solution for your hardware. Have fun building with that!


Better styling for cds-plugins

Fixed external link icon position + spacing
Reusing classes instead of repeated inline styling
Subtle hover effect for plugin images

→ can discuss in UA sync

Also cap/docs/pull/458/ internally to align it.


add sqlx support

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 89.38053% with 12 lines in your changes missing coverage. Please review.

Project coverage is 89.38%. Comparing base (0378d29) to head (d8e6fc0).

Files with missing lines
Patch %
Lines

sqlx/nested_transactions_mssql.go
84.00%
3 Missing and 1 partial :warning:

sqlx/nested_transactions_oracle.go
84.00%
3 Missing and 1 partial :warning:

sqlx/nested_transactions_savepoints.go
85.18%
3 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##             main       #6    +/-   ##
========================================
  Coverage   89.38%   89.38%
========================================
  Files           5       10     +5
  Lines         113      226   +113
========================================
+ Hits          101      202   +101
- Misses          9       18     +9
- Partials        3        6     +3

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


st.svg.downsampling gives UFuncTypeError: Cannot cast ufunc 'add' output from dtype('float64') to dtype('uint32') with casting rule 'same_kind'

Hi,
thank you for this very interesting tool. I was trying to run spatially variable gene identification but I encounter an error in the first step of downsampling.
I am running
adata = st.svg.downsampling(adata_sm, downsampling=400)
and get following error:

UFuncTypeError                            Traceback (most recent call last)
Cell In[47], line 1
----> 1 adata = st.svg.downsampling(adata_sm, downsampling=400)
2 adata
File ~/.conda/envs/spateo/lib/python3.11/site-packages/spateo/svg/get_svg.py:204, in downsampling(adata, downsampling)
194 """Downsampling the cells from the adata object.
195
196 Args:
(...)
201     adata: adata after the downsampling.
202 """
203 adata = adata.copy()
--> 204 ind = sample(arr=np.array(range(adata.X.shape[0])), n=downsampling, method="trn", X=adata.obsm["spatial"])
205 adata = adata[ind].copy()
206 return adata
File ~/.conda/envs/spateo/lib/python3.11/site-packages/dynamo/tools/sampling.py:54, in sample(arr, n, method, X, V, seed, **kwargs)
52     sub_arr = arr[sample_by_velocity(V=V, n=n, seed=seed, **kwargs)]
53 elif method == "trn" and X is not None:
---> 54     sub_arr = arr[trn(X=X, n=n, return_index=True, seed=seed, **kwargs)]
55 elif method == "kmeans":
56     sub_arr = arr[sample_by_kmeans(X, n, return_index=True)]
File ~/.conda/envs/spateo/lib/python3.11/site-packages/dynamo/tools/utils.py:793, in timeit..timed(*args, **kw)
791     main_info_verbose_timeit("Time elapsed for %r: %.4f s" % (method.name, (te - ts)))
792 else:
--> 793     result = method(*args, **kw)
794 return result
File ~/.conda/envs/spateo/lib/python3.11/site-packages/dynamo/tools/sampling.py:210, in trn(X, n, return_index, seed, **kwargs)
197 """Sample method based on topology representing network.
198
199 Args:
(...)
207     The sample graph or the indices of the sub-sampled array.
208 """
209 trnet = TRNET(n, X, seed)
--> 210 trnet.run(**kwargs)
211 if not return_index:
212     return trnet.W
File ~/.conda/envs/spateo/lib/python3.11/site-packages/dynamo/tools/sampling.py:155, in TRNET.run(self, tmax, li, lf, ei, ef, c)
153 ep = ei * np.power(ef / ei, tt)
154 # run once
--> 155 self.runOnce(P[t - 1], l, ep, c)
File ~/.conda/envs/spateo/lib/python3.11/site-packages/dynamo/tools/sampling.py:125, in TRNET.runOnce(self, p, l, ep, c)
123 # move the nodes
124 if c == 0:
--> 125     self.W += ep * np.exp(-K[:, None] / l) * D
126 else:
127     # move nodes whose displacements are larger than the cutoff to accelerate the calculation
128     kc = -l * np.log(c / ep)
UFuncTypeError: Cannot cast ufunc 'add' output from dtype('float64') to dtype('uint32') with casting rule 'same_kind'

I already tried to adjust adata.X and adata.obsm['spatial'] to be uint32 or float64 but neither of the combinations work.
Thank you for any insights that might solve the problem!

Sorry, I must have made a mistake yesterday. Converting the .obsm['spatial] to float solved the problem.
Here my solution:

adata_sm.obsm['spatial'] = adata_sm.obsm['spatial'].astype('float64')
adata = st.svg.downsampling(adata_sm, downsampling=400)


chore: docker compose to standalone by default

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
After testing, the standalone mode has become relatively stable, so we plan to change docker-compose to standalone by default.

docker-compose.yml -> docker-compose-minio.yml

docker-compose-standalone.yml -> docker-compose.yml

change integration tests docker-compose to standalone mode. The following two cases cannot be simply changed to standalone mode, so I will still deploy using a distributed cluster.

doris-sink
feature-store
@xxhZs please help confirm if these two cases can be run in standalone mode.

https://github.com/risingwavelabs/risingwave/issues/13235
Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[x] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

"Doris Sink" is because we need to ensure that the entire cluster adopts the same set of network IP, otherwise it will result in RW being unable to access Doris. Is there any other way to solve the whole problem? The Feature Store needs to transfer UDF files to Docker, and I'm not sure if there is any other solution.

standalone mode integration test：
https://buildkite.com/risingwavelabs/integration-tests/builds/488
It seems that the data check of vector case fails in standalone mode. Please help to check. cc @neverchanje

"Doris Sink" is because we need to ensure that the entire cluster adopts the same set of network IP, otherwise it will result in RW being unable to access Doris. Is there any other way to solve the whole problem? The Feature Store needs to transfer UDF files to Docker, and I'm not sure if there is any other solution.

These two cases are not in the daily test list. We can temporarily not enable standalone mode in this PR. Maybe you can try to see if you can run them in standalone mode?

Did you run it using the nightly version? I ran on the nightly last time. Please use the cluster mode if it just cannot run.

I use ‘nightly-20231205’ version