Card continues to crash

Hi again! So after your plug in was working very well for a number of days I started getting the dreaded SSL error again. They became more frequent after I reset the card from the security panel and now it's to the point that the card locks itself into SSL mode every few seconds. I downgraded to v1.22 and the same thing happens so I suspect it's a hardware thing. The only thing I haven't done is take the power off the card which I will probably try later, it's just awkward. Which card do you use? I remember you saying you had a different version to mine. I wonder if I can buy that and install it.

That's very unfortunate to hear 🤬 I have an NX6v2 system with the NX595E network module installed with the ComNav 0.108O firmware revision installed on the latter.
SSL errors theoretically occur when attempting to connect via https to the web interface (and are caused by a deprecated TLS/SSL cipher suite that it uses). The installer manual explicitly states that SSL access is not supported anymore as its implementation is too old; the old manual mentions Location 19, option 2 as the SSL setting (has to be turned off to allow standard http connections), which can be set using the standard programming using the control panel of the alarm installation (typically starting with *8 -> [master PIN] -> 191#, and then changing the option value as usual). Please do note that tampering with the control panel and system settings can mess up your installation severely, so it would be best to do this with your security technician's help.
That being said, the plugin explicitly uses http:// calls, so other than the idea described above, I really have nothing else to suggest, and there is no practical way to force an SSL connection, short of installing a custom-build version of both node and the plugin itself to allow for the ancient cipher suite; needless to say, this is both dangerous and impractical, sadly.
I'll leave the issue open for a while; feel free to get back in touch with any kind of updates 🙏🏻✌🏻

Hi mate. I have become quite familiar with resetting the module. What happens is that everything is working fine then the module crashes and turns on the SSL by ITSELF. The only way to access the webpage then becomes by resetting the whole thing and disabling SSL (and redoing the options properly). I can crash the module now by just refreshing the webpage too many times. I suspect the polling from your plug in has the same effect.

On 18 Nov 2022, at 9:50 am, Spyros Smparounis @.***> wrote:
That's very unfortunate to hear 🤬 I have an NX6v2 system with the NX595E network module installed with the ComNav 0.108O firmware revision installed on the latter.
SSL errors theoretically occur when attempting to connect via https to the web interface (and are caused by a deprecated TLS/SSL cipher suite that it uses). The installer manual explicitly states that SSL access is not supported anymore as its implementation is too old; the old manual mentions Location 19, option 2 as the SSL setting (has to be turned off to allow standard http connections), which can be set using the standard programming using the control panel of the alarm installation (typically starting with *8 -> [master PIN] -> 191#, and then changing the option value as usual). Please do note that tampering with the control panel and system settings can mess up your installation severely, so it would be best to do this with your security technician's help.
That being said, the plugin explicitly uses http:// calls, so other than the idea described above, I really have nothing else to suggest, and there is no practical way to force an SSL connection, short of installing a custom-build version of both node and the plugin itself to allow for the ancient cipher suite; needless to say, this is both dangerous and impractical, sadly.
I'll leave the issue open for a while; feel free to get back in touch with any kind of updates 🙏🏻✌🏻
—
Reply to this email directly, view it on GitHub https://github.com/flareman/homebridge-caddx-interlogix/issues/18#issuecomment-1319310044, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOFXUYSEWNI4VW7MHGL26WTWI2ZCPANCNFSM6AAAAAASCUARF4.
You are receiving this because you authored the thread.

It really shouldn't lock up like this; all I can suggest is reducing the polling frequency (increasing the polling interval), to reduce calls to the web interface. The NX-595E is a hot mess coding-wise, the integrated HTTP server that the module serves respects close to no common practices, and its behavior is erratic outside the web interface (and, sometimes, within the web interface as well). It's unfortunate that there are no firmware upgrades to be expected too. What versions are you running? You could see this in the web interface -> Feature Setup (firmware, hardware, bootload, and web version fields).

Hi. The module just has become increasingly sensitive to poll requests. It is quite brittle now. Firmware 2.0k, hardware 1.0, bootload 4.0, web version CN_0.106-j. I increased the polling time to 10 secs but it still crashes it. Surely the hardware is now faulty?

On 19 Nov 2022, at 4:17 am, Spyros Smparounis @.***> wrote:
It really shouldn't lock up like this; all I can suggest is reducing the polling frequency (increasing the polling interval), to reduce calls to the web interface. The NX-595E is a hot mess coding-wise, the integrated HTTP server that the module serves respects close to no common practices, and its behavior is erratic outside the web interface (and, sometimes, within the web interface as well). It's unfortunate that there are no firmware upgrades to be expected too. What versions are you running? You could see this in the web interface -> Feature Setup (firmware, hardware, bootload, and web version fields).
—
Reply to this email directly, view it on GitHub https://github.com/flareman/homebridge-caddx-interlogix/issues/18#issuecomment-1320306477, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOFXUYQWXTIY7ZOYF4M6R6LWI62ZJANCNFSM6AAAAAASCUARF4.
You are receiving this because you authored the thread.

My module is running firmware A55P003001-03, hardware 206144IX12, bootloader B0402002, web version CN_0.108-O; maybe this older version works bad with multiple polling requests? Then again it was working in the past, so maybe it is just going faulty. If you had access to a second network module that's known to work, you could replace the present one with the good one and check for crashes, but I'm guessing this is taking it a bit too far 🤷 I just don't see how or why the module should crash when polled multiple times; even the web interface polls every 5500 ms, and that's by design from Interlogix. If you log in the web interface and leave it logged in like that for an extended period of time, will it crash too?

If I leave the webpage open it doesn’t to crash. If I reload the webpage a
few times it does crash. I agree that because it used to work, that the
card is probably going faulty. I’ll see if I can get another card to check
it out. Will keep you updated. You can probably close this issue as it
seems related to the hardware not your software. Thank you.
On Sat, 19 Nov 2022 at 19:39, Spyros Smparounis @.***>
wrote:

My module is running firmware A55P003001-03, hardware 206144IX12,
bootloader B0402002, web version CN_0.108-O; maybe this older version works
bad with multiple polling requests? Then again it was working in the
past, so maybe it is just going faulty. If you had access to a second
network module that's known to work, you could replace the present one with
the good one and check for crashes, but I'm guessing this is taking it a
bit too far 🤷 I just don't see how or why the module should crash when
polled multiple times; even the web interface polls every 5500 ms, and
that's by design from Interlogix. If you log in the web interface and leave
it logged in like that for an extended period of time, will it crash too?
—
Reply to this email directly, view it on GitHub
https://github.com/flareman/homebridge-caddx-interlogix/issues/18#issuecomment-1320835553,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOFXUYR6PWNLHF6YQ3A4TKTWJCG23ANCNFSM6AAAAAASCUARF4
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks so much for your continued support debugging the plugin 🙏🏻


GridSpacing: Per-Component UnitSI & UnitDimension

The gridSpacing should get a unitSI and a unitDimension for each of its components.
This allows to describe grid such as x, ux or x,y,t (although the "time" definition per iteration would then be "for the first row").

For gridUnitSI this seems quite simple: we just enter an array of floats (one element per dimension) instead of a single float.
For gridDimension, it seems that the most natural thing would be to have a 2d array of dimension Nx7, where N is the number of dimensions. However, I am not sure that HDF5 (and others) support 2d arrays as attributes. Therefore, I guess that the best choice would be a flattened 2d array. @ax3l Any opinion?

Sorry I missed the question. Yes, HDF5 does support 2D arrays but flattened to the same index order as the data (from fast varying index to slowest) is fine for me as well.


Economic Retirement

Closes #66 #130
Changes proposed in this Pull Request

In this PR I add the function correct_extendable_generators() to the add_extra_components module. This function adds logic for economic retirement of generators. Specifically, it:

Creates duplicate generators for any that are tagged as extendable. For example, an extendable CCGT generator will be split into CCGT and CCGT new
capital_costs of existing extendable generators are replaced with fixed costs
p_nom_max of existing extendable generators are set to p_nom
p_nom_min of existing and new generators is set to 0

Checklist

[x] I tested my contribution locally and it seems to work fine.
[x] Code and workflow changes are sufficiently documented.
[x] Changed dependencies are added to envs/environment.yaml.
[x] Changes in configuration options are added in all of config.default.yaml.
[x] Changes in configuration options are also documented in doc/configtables/*.csv.

Getting this weird issue where it is cheaper to sell off some existing renewable generators and rebuild in places with better potential.

This also happens at a bus level for solar, but at a system level it ends up building way more

Oh. Just kidding. Im an idiot and forgot to set the capacity factors of the new solar and wind plants.

@ktehranchi So I couldn't actually find where pypsa-eur implemented economic retirement, so I implemented it as we discussed yesterday to keep this moving forward. Two questions for thoughts tho:

I have the option to select economic or technical retirement in the config file. You think this is okay, or should we simplify this and only give the option for one retirement method? I lean towards keeping as an option (as thats whats implemented), if we choose to force economic retirement, it may be good to revert the change of setting p_nom_min as p_nom earlier in the workflow for clarity
Currently, all extendable generators (including wind and solar) have dummy generators implemented. I guess in reality, it probably wont make sense to retire wind and solar, so I am wondering if in an effort to reduce number of decision variables the economic retirement should only be applied to conventional generators?

Still a draft as docs and plotting need to be updated to reflect the above questions/discussion

Forgot to add in the marginal cost of new generators. Will need to add that still

@ktehranchi So I couldn't actually find where pypsa-eur implemented economic retirement, so I implemented it as we discussed yesterday to keep this moving forward. Two questions for thoughts tho:

I have the option to select economic or technical retirement in the config file. You think this is okay, or should we simplify this and only give the option for one retirement method? I lean towards keeping as an option (as thats whats implemented), if we choose to force economic retirement, it may be good to revert the change of setting p_nom_min as p_nom earlier in the workflow for clarity
Currently, all extendable generators (including wind and solar) have dummy generators implemented. I guess in reality, it probably wont make sense to retire wind and solar, so I am wondering if in an effort to reduce number of decision variables the economic retirement should only be applied to conventional generators?

1: Yes i like the idea of having this in the configuration file.
2: We already had dummy generators for wind and solar generators at each eligible node in the network (barring land use restrictions), so no need to add those again.

Right. I guess more what I was thinking that we are preventing the model from selling off existing wind/solar generators, even though we tag this as economic retirement. But I agree, that probably wouldnt make too much sense in reality. Might just add a note in the docs to note that economic retirement does not apply to solar/wind. Thanks!

Pretty sure this is ready to get merged, I will just do the following checks tomorrow:

[ ] Run in retirement mode technical
[ ] Run in retirement mode economic
[ ] Compare technical retirement mode results to main branch results, as they should be the same

There is a real minor difference in production results. I wait to discuss during our next meeting before merging in.

I just noticed that since pypsa version 0.25.0 there is a new field for generators called stand_by_costs.... which we could be using instead to assign our FOM cost to!
If we assigned fixed OM to the stand-by cost, and just capex for new capex build, we would still need to implement the split for new and old generators in order to track retirements separately. but we should update out pypsa version and add the standby costs for FOM split from CAPEX

@ktehranchi, Im not sure this will totally work? Stand by costs are only tracked if the generator/link is tagged as commitable? Therefore, we wouldn't actually be adding in any FOM since we arnt running unit commitment?

Where do you see that its only read if its committable? I can't find that.

@ktehranchi I think this is ready to get merged now. Can you please just do a quick run to make sure everything works okay, as per our conversation earlier this week?
Also, I left out the stand_by_costs conversion for now. We can revisit this in a new issue ticket if needed.

Looks good, just changed the docs description a little and default config. hit the merge when you're ready.


:seedling: Add support for CA/certificate rotation

Mounted secrets are automatically updated into pods, but...

It doesn't work with subPath mountings
When subPath is not used, then a bunch of directories are mounted
And one of those directories is a symlink, so IsDir() returns false
And a watch is needed to notice the change

So, update the certificate volume patch, which requires a change in how we look for certificates in the CA cert directory.
Add a watch, so when the certs do change, we update the cert pool.
Also look at validity dates of certificates, and error on expired certs.
The default cert-manager certificates have 90 days validities.

Description

Reviewer Checklist

[ ] API Go Documentation
[ ] Tests: Unit Tests (and E2E Tests, if appropriate)
[ ] Comprehensive Commit Messages
[ ] Links to related GitHub Issue(s)

This also adds a unit-test for an empty cert dir, and for an expired certificate. And also adds an e2e to make sure secret updates and rotations actually work.

TLDR: it looks like there is no equivalent of GetClientCertificate/GetCertificate, but for root CAs in TLS config in standard library, but it is possible to implement a workaround with VerifyPeerCertificate. This would require setting InsecureSkipVerify to true and re-implementing the standard verification in VerifyPeerCertificate as far as I understand - see this example from Go docs.

And we really don't want to do that (set InsecureSkipVerify) by default. And reimplementing the verification code is probably not worth it, when we can just offer up the updated certificate pool on demand.

And we really don't want to do that (set InsecureSkipVerify) by default. And reimplementing the verification code is probably not worth it, when we can just offer up the updated certificate pool on demand.

Setting InsecureSkipVerify to true and VerifyPeerCertificate means that we take over the responsibility of verifying the certs from standard library. That's not great, but it is not a lot of code (see standard library code, and client example from the docs). It doesn't sounds too terrible to me, so I'm open to consider this.
As far as I understand in this implementation we are re-creating a client each time and will have to establish a connection each we use it which is not optimal.
I don't know how significantly it will affect our performance. One one hand - creating new connections every time is not great, but on the other hand - not sure if we are going to benefit from re-using the connection in these use cases.


Added array example when using data attribute

tokenSeparators expects an array of characters when using javascript to initialize select2. When using the data-* attribute data-token-separators , you don't write an array, you simply write the characters one after the other, in quotes.

Any particular reason why this works? I would normally expect jQuery to pick up the array (written as a string in data-token-separators) and convert it to a JS array.
In theory,
<select data-token-separators=";,"></select>

Should be equivalent to
$('select').select2({
  tokenSeparators: ';,'
});

Which isn't exact an array (though it is array-like).

Hi @kevin-brown , thank for looking into it, I tried several array like syntaxes and select2 kept just ignoring my separators and would use anything that was inside quotes, like, if I used: data-token-separator="[',', ';']" , then typing a space triggered a new tag work, instead of just the command and semicolon. As it took me a while to get this working, I thought it would be good to have an example on the docs.
Let me know if there is anything else I can do.
Thank you.

It's because you are using single quotes instead of double quotes in your array. JSON requires double quotes, so jQuery isn't parsing it as valid JSON.
Related Stack Overflow question

Hi Kevin, sorry for the delay, you are right, if I use single quotes around the array syntax, it works, unfortunately that doesn't work for me because I'm using Scala to generate the html and their parser forces me to use double quotes on any attribute.
Would you accept the PR for documentation if I add both syntaxes, with  a note that if your server side language forces double quotes, this is the alternative syntax that also works?
Thanks

Would you accept the PR for documentation if I add both syntaxes, with a note that if your server side language forces double quotes, this is the alternative syntax that also works?

There's a problem with this: technically the alternative syntax is wrong. It's not just that it's wrong in the "it technically produces the equivalent data" sense, it only works because strings are indexable similar to how arrays are. Select2 expects an array for the tokenSeparators option, and passing a string is not officially supported and may break in the future.
It works, yes. Only if you need to provide an array of characters, but not if you need to provide an array of strings (or other data, like numbers). So I'm not comfortable with us recommending it as an alternative syntax for arrays when it doesn't actually produce an array.
I would be comfortable with documenting the official way for providing an array (using JSON) in the attribute.


Convert MUI tables inside panels

Tables are not preferred since we are dealing with lists.
Convert this, possibly to Material-UI Grid components.

Done and merged into develop

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


Port matching.coffee to matching.ex

You can do a separate PR for each item, but you need to test it:

[ ] omnimatch/1
[ ] dictionary_match/2
[ ] reverse_dictionary_match/2
[ ] l33t_match/3
[ ] spatial_match/2
[ ] repeat_match/1
[ ] sequence_match/1
[ ] regex_match/2
[ ] date_match/1

https://github.com/sevenshores/zxcvbn-ex/blob/master/lib/zxcvbn/matching.ex

Interested in starting off with this one, but looks daunting..
Looking at some reference implementation primarily to get some valid test cases to work with - suggestions are welcome :)

I don't know helpful it is, but here are the original tests in coffeescript:
https://github.com/dropbox/zxcvbn/tree/master/test

starting repeat_match/1, I find the scala project has very readable test cases
(https://github.com/tekul/szxcvbn)


Flutter_driver.waitFor impacts performance during UI performance testing

Affects customer: dream and Fuchsia.
https://b.corp.google.com/issues/168731654.

Likely not a Flutter issue. No action needed at present from the Flutter team; keeping open until upstream issue is closed.

Upstream priority lowered, and determined not a Flutter issue. Closing for now.


Remove duplicate variable assignment.

Nagging Assignee @drpngx: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @drpngx: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@larrytin sorry could you pull rebase and push again?

@larrytin could you please resolve the conflict?

It has been 14 days with no activity and the awaiting response label was assigned. Is this PR still valid? Assigning the stalled label. Please comment to reassure me that this is still being worked on.

closing this PR due to lack of activity.


update pack to install psr-18 client

do we need to add something for the psr-17 factories or where do those come from?

Bump.. @Nyholm can this be merged?

I tested and it does not work with guzzle/psr17.
I added nyholm/psr7 because it supports psr17.
Thank you @MrNicodemuz

And thank you @dbu


Is it possible to autocomplete when using dep?

When using https://github.com/golang/dep in a project all dependencies are placed in the project root under vendor/[url]/... is it possible to get gocode to see these and autocomplete them?
I realize dep is still an experiment, but I thought there might be a way since it's the official one. Still, I haven't been able to make it work either using the lib-path or package-lookup-mode

I also encountered this problem. My temporary solution is: go build -i in the project path.

I just encountered this in my own project and even though I have .a files generated under the /pkg path gocode still seems to fail with the autocomplete.

emacs gocode dep  not work

This issue isn't emacs specific, but are the arguments being passed incorrectly to gocode in the emacs gocode autocomplete? go build -i does fix it temporarily (or running go install on the 3rd party deps that I added using dep ensure) but that's not a solution I'd like to support long term. The arguments are being passed in https://github.com/nsf/gocode/blob/master/emacs/go-autocomplete.el#L101 . I'm not super familiar with the arguments that gocode takes.

I managed to get this working by setting this flag:
gocode set autobuild true
The docs says this is an experimental feature.
My setup was as follows:

Clean go directories
Create a project and install a dependency:
echo -e "package main\nfunc main(){}" > main.go && dep init && dep ensure -add github.com/gorilla/mux
Add main.go and try auto completion for router (does not work):

package main

import (
	"net/http"

	"github.com/gorilla/mux"
)

func main() {
	router := mux.NewRouter()
	router.hand // Autocomplete here
	http.Handle("/", router)
}

Enable autobuild:
gocode set autobuild true
Try autocompletion (works):

package main

import (
	"net/http"

	"github.com/gorilla/mux"
)

func main() {
	router := mux.NewRouter()
	router.Handle(path, handler) // Autocomplete here
	http.Handle("/", router)
}

@palsivertsen try to run go install ./vendor/... between 2 and 3 steps

@kovetskiy That seems to work as well, even with gocode set autobuild false


Single key launch

Hi,
I'm using the latest cerebro version on archlinux(gnome). I'd love to map the Super key to quickly launch cerebro- is that possible? Trying to set a shortcut via config or the settings menu didn't work.

@navdeepio hey! In UI you can set only keys with modifiers (so, you can't set single key). When you edited config, did you try to restart? It should work, if OS doesn't limit this usage somehow. What OS/version do you use?

@KELiON Hi, I tried to restart but got this error
Uncaught Exception: TypeError: Error processing argument at index 0, conversion failure from Super     at TypeError (native)     at exports.default (/usr/lib/cerebro/resources/app.asar/main.js:231:29)     at App._electron.app.on (/usr/lib/cerebro/resources/app.asar/main.js:99:44)     at emitTwo (events.js:111:20)     at App.emit (events.js:191:7)

@navdeepio I've tried to test it on my machine and looks like it is not possible to register global shortcut with Electron, that uses only modifier key. So, along with modifier should be defined another key with +. I'll check later source code to be sure:)


Genomescope2.0 release

Hi,
Do you see any possibility to release genomescope2.0 so it can be added to bioconda?

Added genomescope2 to bioconda using git_url without a release.


Can't start the debug process successfully

(1) The version of delve:
    [root@Fedora ~]# delv -v
    delv 9.10.2-RedHat-9.10.2-1.fc22

(2) The version of Go:
    [root@Fedora ~]# go version
    go version go1.4.2 linux/amd64

(3) OS and processor architecture:
[root@Fedora ~]# uname -a
Linux Fedora 4.0.4-301.fc22.x86_64 #1 SMP Thu May 21 13:10:33 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

(4) Issue description:
I doubt the this issue is related to my computer changes network IP, but not sure.

Debug the program using full path:
[root@Fedora ~]# delv /root/syncWait
;;   validating ./NSEC: got insecure response; parent indicates it should be secure
;;   validating ./NSEC: got insecure response; parent indicates it should be secure
;; no valid RRSIG resolving '/root/syncWait/DS/IN': <IP_ADDRESS>#53
;;   validating ./NSEC: got insecure response; parent indicates it should be secure
;; no valid RRSIG resolving '/root/syncWait/DS/IN': <IP_ADDRESS>#53
;; no valid DS resolving '/root/syncWait/A/IN': <IP_ADDRESS>#53
;;   validating ./NSEC: got insecure response; parent indicates it should be secure
;; validating /root/syncWait/A: bad cache hit (/root/syncWait/DS)
;; broken trust chain resolving '/root/syncWait/A/IN': <IP_ADDRESS>#53
;; resolution failed: broken trust chain

Debug the program using relative path:
 [root@Fedora ~]# pwd
 /root
 [root@Fedora ~]# delv ./syncWait
 ;; failed to convert QNAME ./syncWait: empty label

delv is not the correct command you need to install dlv as per https://github.com/derekparker/delve/wiki/Building
The delv binary you are running is part of the bind package
http://pkgs.fedoraproject.org/cgit/bind.git/tree/bind.spec?h=f22#n857
Man page ftp://ftp.isc.org/isc/bind/cur/9.10/doc/arm/man.delv.html

@pnasrat is correct. Invoke with dlv.


cni: fix plugin build failure

Recently, there is change for upstream plugins project of containernetworking,
the original build.sh is splitted into two OS-specific scripts build_linux.sh
and build_windows.sh, this patch will synchronize this change and make cni
plugin build happy.
Signed-off-by: Alex Jia<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

@chuanchang This is a duplicate pr.
https://github.com/alibaba/pouch/pull/2450

@chuanchang This is a duplicate pr.
#2450

oh, it seems we met the same issue :)

Thanks @chuanchang . This pr is duplicate and I close it.


Overview Comment is not being reset with '-Dsonar.stash.comments.reset=true'

I was surprised the -Dsonar.stash.comments.reset=true only resets the earlier comments in the diff, but not the overview comment.
Is this the desired behaviour, or is there something wrong on my side?
I find it surprising at least, and the the README also doesn't mention that it would relate to the diff comments only.

Hi,
do you have an idea when this will be integrated ?

I was working on it today.
The problem is, that the overview comments has to be requested with a new API endpoint, which we currently don't use.
I used this to also implement a generic accessor for paged API endpoints.
A first prototype should be done tomorrow.

Awesome, thank you!

@sebastian-r Can you take a look at #162

Hi @t-8ch ,
sorry I am responding so late. Had holiday and trainings. I will have a look at it tomorrow. Thanks already for your engagement :)

You can look at the change in this PR and manually apply it and build your own jar with the change included: https://github.com/AmadeusITGroup/sonar-stash/pull/162
Unfortunately it was for an older codebase (I believe 1.3.0 and current release is 1.6.0) and as such you'll manually have to apply that fix against the latest codebase you'd like.   I've used the fix provided and it works nicely against 1.51.


pythonPackages.vobject: 0.9.3 -> 0.9.5

Motivation for this change
I ran nox-review but it failed. Not sure if it's because of this change or whether those packages were broken anyway.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Hmm.. I'll rebase to master.

Oh, it was already updated in master but just wasn't moved to a separate file. Ok. this pull request is useless.


[VS Code] Starting collaboration session: Invalid root scheme: vsls, expected 'file'

Error:
Starting collaboration session: Invalid root scheme: vsls, expected 'file'
Steps to Reproduce:

Version Data

extensionName
VSLS

extensionVersion
0.3.1151

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.30.2

platformName
Linux

platformVersion
4.15.0-44-generic

@srohimah We're tracking this issue via #1568, so I'm going to close this as a duplicate. If you could run the Live Share: Export Logs command, and attach the resulting zip to #1568, that would be hugely appreciated. Thanks!


[CU-8686ppvry] Prevents user from navigating if post is not published

Task linked: CU-8686ppvry If a user tries to leave a post page and it's unpublished, show a confirmation alert


Macspileup

Without mates:

With mates:

Notice the increase in pileup!!

Related to #64

@siebrenf ready for review
PR adds the option keep_macs_mates which allows the mates to be used for pileup, and some general code cleanup.

Yes, I'll add a wider variety.
Anyways the tests should include different configurations, which they don't (yet)....

@siebrenf I'll do it properly tomorrow, for now you can accept this one. See $144


implement scrolling speed setting

a general scroll speed setting in the same manner sublime "scroll_speed" (http://docs.sublimetext.info/en/latest/reference/settings.html)
https://github.com/atom/atom/issues/5359
https://github.com/atom/atom/issues/1666

I'm new to electron, but have experience in nodejs, wanna try!

@alvarolm Please update this Issue with a more detailed explanation of what it is you're requesting. External documentation links can be moved or change. We need to have a more permanent record of what it is you're asking for so that we can a) make a determination as to whether this is something we want to do and b) ensure that if it is something we want to include that we have actually achieved the goals when we make a change.

What is wrong with the current setting that changes scroll speed?

@lee-dohm  @Ben3eeE updated

https://www.facebook.com/notes/daniel-colascione/buttery-smooth-emacs/10155313440066102/


ClassAliasBead.js:47 Uncaught TypeError: app.info is not a function

Run Time Error Screenshot:

Go To : Inventory form menu and then click Balance Transfer
Please check this issue on our VDI. Thanks in Advance
Project Details are :
https://paste.apache.org/cpo8y
Filename : dbzBalanceTransfer and dbzBalanceTransferDetail
available in D:\DBIZSOL\Main\Royale\src\com\dbz\modules\INV

Issue not Resolved
Go To : Inventory form menu and then click Balance Transfer
Please check this issue on our VDI. Thanks in Advance
Project Details are :
https://paste.apache.org/cpo8y
Filename : dbzBalanceTransfer and dbzBalanceTransferDetail
available in D:\DBIZSOL\Main\Royale\src\com\dbz\modules\INV

ClassAliasBead is being used on a BorderContainer.  It can only be used on an Application or Module.


Relicense under dual MIT/Apache-2.0

This issue was automatically generated. Feel free to close without ceremony if
you do not agree with re-licensing or if it is not possible for other reasons.
Respond to @cmr with any questions or concerns, or pop over to
#rust-offtopic on IRC to discuss.
You're receiving this because someone (perhaps the project maintainer)
published a crates.io package with the license as "MIT" xor "Apache-2.0" and
the repository field pointing here.
TL;DR the Rust ecosystem is largely Apache-2.0. Being available under that
license is good for interoperation. The MIT license as an add-on can be nice
for GPLv2 projects to use your code.
Why?
The MIT license requires reproducing countless copies of the same copyright
header with different names in the copyright field, for every MIT library in
use. The Apache license does not have this drawback. However, this is not the
primary motivation for me creating these issues. The Apache license also has
protections from patent trolls and an explicit contribution licensing clause.
However, the Apache license is incompatible with GPLv2. This is why Rust is
dual-licensed as MIT/Apache (the "primary" license being Apache, MIT only for
GPLv2 compat), and doing so would be wise for this project. This also makes
this crate suitable for inclusion and unrestricted sharing in the Rust
standard distribution and other projects using dual MIT/Apache, such as my
personal ulterior motive, the Robigalia project.
Some ask, "Does this really apply to binary redistributions? Does MIT really
require reproducing the whole thing?" I'm not a lawyer, and I can't give legal
advice, but some Google Android apps include open source attributions using
this interpretation. Others also agree with
it.
But, again, the copyright notice redistribution is not the primary motivation
for the dual-licensing. It's stronger protections to licensees and better
interoperation with the wider Rust ecosystem.
How?
To do this, get explicit approval from each contributor of copyrightable work
(as not all contributions qualify for copyright, due to not being a "creative
work", e.g. a typo fix) and then add the following to your README:
## License

Licensed under either of

 * Apache License, Version 2.0, ([LICENSE-APACHE](LICENSE-APACHE) or http://www.apache.org/licenses/LICENSE-2.0)
 * MIT license ([LICENSE-MIT](LICENSE-MIT) or http://opensource.org/licenses/MIT)

at your option.

### Contribution

Unless you explicitly state otherwise, any contribution intentionally submitted
for inclusion in the work by you, as defined in the Apache-2.0 license, shall be dual licensed as above, without any
additional terms or conditions.

and in your license headers, if you have them, use the following boilerplate
(based on that used in Rust):
// Copyright 2016 diesel Developers
//
// Licensed under the Apache License, Version 2.0, <LICENSE-APACHE or
// http://apache.org/licenses/LICENSE-2.0> or the MIT license <LICENSE-MIT or
// http://opensource.org/licenses/MIT>, at your option. This file may not be
// copied, modified, or distributed except according to those terms.

It's commonly asked whether license headers are required. I'm not comfortable
making an official recommendation either way, but the Apache license
recommends it in their appendix on how to use the license.
Be sure to add the relevant LICENSE-{MIT,APACHE} files. You can copy these
from the Rust repo for a plain-text
version.
And don't forget to update the license metadata in your Cargo.toml to:
license = "MIT OR Apache-2.0"

I'll be going through projects which agree to be relicensed and have approval
by the necessary contributors and doing this changes, so feel free to leave
the heavy lifting to me!
Contributor checkoff
To agree to relicensing, comment with :
I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to chose either at their option.

Or, if you're a contributor, you can check the box in this repo next to your
name. My scripts will pick this exact phrase up and check your checkbox, but
I'll come through and manually review this issue later as well.

[x] @derekprior
[x] @mcasper
[x] @mfpiccolo
[x] @palango
[x] @sgrif

I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to chose either at their option.
Whatever @sgrif decides if fine by me.

@crackofdusk, @mathroc, @tohou: You've only touched documentation, so no action is required from you on this, and I've removed you from the checkbox.
@mcasper, @mfpiccolo, @palango: I am in favor of this licensing change, but as contributors I cannot make this change unless you all agree.
@mikeastock: Looks like the only code change you've made is changing some variable names, so legally I don't think we need any action (I'm not a lawyer), but confirmation that you're fine would still be welcome.

I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to chose either at their option.

I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to chose either at their option.

I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to chose either at their option.

I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to chose either at their option.

Thank you for the quick replies, everyone


Web app not loading.

Describe the bug
Web app doesn't load for me anymore. Tried it on Chrome and Edge browsers. I see blank page with "Loading" word in the circle but nothing is happening.
Not sure if its something wrong on my side or with the server. Can't use other commputer to check it atm.

Hi,
this is likely related to the fact that I changed my GitHub handle, and thus the associated generated github.io URLchanged as well.
https://stepkie.github.io/MtgCsvHelper/ should work.
I will update the README  to point to the URL asap.


How to change the mvc action argument type?

I have a mvc action like this:
public async Task AddBatchDevice(string[] deviceNos ,ObjectId? customerID=null)
but the ObjectId in mongodb is the struct,I write the jsonconverter to change the string to objectId,but nswag can't mapper the objectid to string when it on method argument?
how to resolve it?thk

I try to use:
1: opt.TypeMappers.Add(new PrimitiveTypeMapper(typeof(ObjectId?), (s) =>
{
s.Type = JsonObjectType.String;
s.Title = "ObjectId";
}));
2.[JsonSchema(JsonObjectType.String)]ObjectId? customerID=null
above all can't work

Did you try [JsonSchemaType(typeof(string))] on the parameter?

Did you try [JsonSchemaType(typeof(string))] on the parameter?

yes,I try it,,but no work

and I find out a way to fix it ,but it look like so.......bad


SSO via OIDC Support

Maybe we can add this feature for administration
https://github.com/zhaow-de/pam-keycloak-oidc

@victorrodriguez1984 this is PAM module to use in Linux. For SSO it's enough standard golang.org/x/oauth2 package.

Good Thank you!

why is this marked as completed? can you please link the merge request or documentation that shows how the SSO works.

This should be reopened and proper oidc support implemented


[BUG]

Description
@GarimaSingh0109
Hi, I am Simran Shaikh. I’ve been reviewing the project as part of my contribution to GSSoC Extended 24 and noticed that the color combination for website is not proper . It is look like not related to each other.
I would like to work on this.
If you like this idea, please assign the task to me and add the corresponding charts to it.
Thank you for your time and consideration!

Steps to Reproduce
To change background color.
Expected Behavior
It will be more efficient for end user to used this website in easy way.
Actual Behavior
It will give proper UI/UX for end user.
Additional Information
No response
Browser
Chrome
Checklist

[X] I have checked the existing issues.
[X] I have read the Code Of Conduct
[X] The changes don't break the code
[X] Please attach screenshots and videos of the development work once done

@GarimaSingh0109  ?


Mapper: Call variants (BAM -> VCF)

In the Mapper module, supporting taking BAM input data and converting it to VCF output.
@dauss75, @chris-owen, this is one of the issues we discussed.

Here is the pipe to generate a quality filtered VCF file from a BAM file.  The example below generates the  the VCF using Freebayes and filters the VCF by quality score (e.g., phred 30).
/home/ubuntu/freebayes/bin/freebayes -f /home/ubuntu/refs/fasta/GCF_000001405.30_GRCh38.p4_genomic.fna <dummy.bam> | /home/ubuntu/vcflib/bin/vcffilter -f "QUAL > 30" > results.vcf


[feat] expose tao::Window::set_cursor_position

Describe the problem
I want to reposition the mouse pointer within the tauri window.
I have read that winit has functions to do that, so I would assume tao can also do that.
Since tao is not exposed by tauri, I cannot use those capabilities.
Describe the solution you'd like
I would like an API function where I can pass coordinates relative to the top left corner of the tauri window which would reposition the mouse pointer.
For example, if I would do set_pointer_position(0,0), the cursor would jump to the top left corner.
Alternatives considered
I have created a custom tauri command using winapi, which looks like this:
#[tauri::command]
fn set_pointer_position(window: tauri::Window, x: i32, y: i32) {
  let inner_size = window.inner_size().unwrap();
  let outer_size = window.outer_size().unwrap();
  let pos = window.inner_position().unwrap();
  unsafe {
    if winapi::um::winuser::SetPhysicalCursorPos(pos.x + x, pos.y + y + (outer_size.height as i32 - inner_size.height as i32)) == 0 {
      panic!("SetCursorPos failed");
    }
  }
}

Additional context
No response

I have tried using the exposed API function.
unfortunately, it hasn't worked for me.
I am on arch linux, kernel 5.17.3-arch1-1 build, and
npm run tauri info yields this:
><EMAIL_ADDRESS>tauri
> tauri "info"

Environment
  › OS: Arch Linux Unknown X64
  › Node.js: 17.9.0
  › npm: 8.7.0
  › pnpm: Not installed!
  › yarn: 1.22.18
  › rustup: 1.24.3
  › rustc: 1.60.0
  › cargo: 1.60.0
  › Rust toolchain: stable-x86_64-unknown-linux-gnu

Packages
  › @tauri-apps/cli [NPM]: 1.0.0-rc.7(outdated, latest: 1.0.0-rc.8)
  › @tauri-apps/api [NPM]: 1.0.0-rc.3(outdated, latest: 1.0.0-rc.3)
  › tauri [RUST]: git+https://github.com/tauri-apps/tauri#8e00f0904c9d7385a5fc79a32b3bb2397699c305 (1.0.0-rc.6),
  › tauri-build [RUST]: git+https://github.com/tauri-apps/tauri#8e00f0904c9d7385a5fc79a32b3bb2397699c305 (1.0.0-rc.5),
  › tao [RUST]: 0.8.2,
  › wry [RUST]: 0.15.0,

App
  › build-type: bundle
  › CSP: unset
  › distDir: ../build
  › devPath: http://localhost:3000/
  › framework: React

my tauri command looks like this:
#[tauri::command]
fn set_pointer_position(window: tauri::Window, x: i32, y: i32) {
    let result = window.set_cursor_position(tauri::Position::Physical(tauri::PhysicalPosition{x, y}));
    if result.is_err() {
        panic!("setting cursor failed");
    }
    println!("cursor set {} {}", x, y);
}

and the console output is as expected.
// relevant cargo.toml data
[build-dependencies]
tauri-build = { git = "https://github.com/tauri-apps/tauri", features = [] }

[dependencies]
serde_json = "1.0"
serde = { version = "1.0", features = ["derive"] }
tauri = { git = "https://github.com/tauri-apps/tauri", features = ["api-all"] }
winapi = { version = "0.3.9" }
mouse-rs = { version = "0.4" }

[features]
# by default Tauri runs in production mode
# when `tauri dev` runs it is executed with `cargo run --no-default-features` if `devPath` is an URL
default = [ "custom-protocol" ]
# this feature is used used for production builds where `devPath` points to the filesystem
# DO NOT remove this
custom-protocol = [ "tauri/custom-protocol" ]

what is the problem here?
I am running the app with cargo run --no-default-features as was recommended in the discord server after npm run start.
I haven't tried it on Windows yet, planning to do that this weekend.

It is not implemented yet on Linux, see https://github.com/tauri-apps/tao/blob/edf6e766704a3f10d62673e6be6f168a1daa0b15/src/platform_impl/linux/window.rs#L565
I documented this in the function but you'd need to build the docs manually to see it.

@CaptainTux I plan to implement this for linux soon. In a week maybe.

I lied 😉, here it is https://github.com/tauri-apps/tao/pull/373 if you want to test it and give feedback.

set_cursor_grab doesn't exist on Linux too, maybe we could just set the cursor icon to grab?

You can try it now by adding this to your Cargo.toml:
[patch.crates-io]
tao = { git = "https://github.com/tauri-apps/tao" }

Don't forget to run cd src-tauri && cargo update.

Cursor grab means to restrict the cursor movement from escaping the window coordinates, not just the icon change.
I wanted to implement it too in the PR but that one would need some trial and error to match the behavior of other platforms or close to it.
Relevant gtk apis:

https://gtk-rs.org/gtk3-rs/stable/latest/docs/gdk/struct.Seat.html#method.grab
https://gtk-rs.org/gtk3-rs/stable/latest/docs/gdk/struct.Device.html#method.grab

I guess I lied grin, here it is tauri-apps/tao#373 if you want to test it and give feedback.

works!

set_cursor_grab doesn't exist on Linux too, maybe we could just set the cursor icon to grab?

Cursor grab means to restrict the cursor movement from escaping the window coordinates, not just the icon change.
I wanted to implement it too in the PR but that one might need some trial and error to match the behavior of other platforms or close to it so I delayed it for now.

tao does not compile at the moment because of a type mismatch in set_cursor_grab (window.rs, line 581).
I inserted an Ok(()) which fixed it.

It's not in window coordinates, it's in overall monitor coordinates (I think) so you're not limited to the window.

My bad for the compilation error, can't belive I didn't see it.

It's not in window coordinates, it's in overall monitor coordinates (I think) so you're not limited to the window.

That is correct, but the tooltip (I assume also the docs) - from rust-analyzer - claims the repositioning is in window coordinates, which is incorrect

Ahh you're right. We'll need to change that.

Opened a PR to fix it: https://github.com/tauri-apps/tao/pull/375 thanks for catching this!


Query's distinct field should be prepared

Fixes #1148

LGTM :+1:

:+1:

Merged manually in https://github.com/doctrine/mongodb-odm/commit/0ddaafd35c7120ea8328cd438fdee0a8003ea896


feat(Dynamics): add TopologicalEntropy.DynamicalUniformity

First PR in a sequence to implement the notion of topological entropy for maps using Bowen-Dinaburg's formalism. This file describes a notion of dynamical uniformities, which generalizes dynamical balls in metric spaces.

[x] Dynamical uniformities
[ ] Topological entropy via covers
[ ] Topological entropy via nets
[ ] Behaviour for subsets
[ ] Behaviour under morphisms
[ ] Behaviour under union
[ ] Behaviour under products
[ ] Full shift

This file is pretty short; the next ones are more involved.
Paging @pitmonticone @sgouezel

Thanks Sébastien for this review!
With respect to naming convention: I have some doubts about a couple of lemmas. These are Monotone.dynEntourage_entourage and Monotone.dynEntourage_time. On the one hand, a positional naming convention looks more standard to me: Monotone.dynEntourage_middle and Monotone.dynEntourage_right, for instance. On the other hand:

I'm not quite sure how to make it work when there are more variables (which happens in the next file...). How to state that something is monotone with respect to the third of four variables? Even middle feels slightly dubious.
this can be annoying when passing arguments from one function to another. For instance, when I am working with the time variable, it makes sense to me to only call lemmas about time, instead of calling sometimes something about the third variable, sometimes about the fourth, etc.

The newer version still have my old naming convention in this respect, but I'm open to any suggestion.

Can you resolve the conversations you have dealt with? In this way, it's easier to spot what remains to be done.

@D-Thomine Finished golfing

bors r+
Thanks!

Can you merge master, fix the build, and then send it back to bors?
bors d+

bors r+


Additional file extensions

Hello, rubocop-md currently supports .md and .markdown extensions,
https://github.com/rubocop-hq/rubocop-md/blob/38addb370ca65283ce9d8d9770d6fe6ce64c65b7/lib/rubocop/markdown/rubocop_ext.rb#L5
also, what about adding the support of the following extensions according to linguist?
https://github.com/github/linguist/blob/96ca71ab99c2f9928d5d69f4c08fd2a51440d045/lib/linguist/languages.yml#L3065-L3083

.mdown
.mdwn
.mdx
.mkd
.mkdn
.mkdown
.ronn
.workbook

Great suggestion!
Would you like to propose a PR?

Yes! I'll do it soon. 👍


Remove custom styles icon to work well with new profiler design

Broken:

Fixed:

Thanks for your PR, but I've just merged PR #215 which contains a fix for this, too.


Method touch() doesn't resave the session

Renew expiries please!
Hey! Hello code! Seems like touch() is not working. No matter where I place it in code. Session puts new data in store with save() but does not renew expiries of key and cookie.
I guess touch() is the method I looking for, it suppose to be same as express-session's one
app.decorateRequest('authorize', async function (user: UserAttributes) {
    const req: FastifyRequest = this
    await req.session.touch() // not working
    req.session.data.user = user
    req.session.data.auth = true
    await req.session.save()
    await req.session.touch() // pathetic attempt
  })

I do it with decorator, cool thing.
Please Dear @mgcrea help me with renewing sessions! I don't want to switch to other package((

Hey! Thanks for pointing that out, indeed the touch functionality was missing and I think I properly did add it along with unit tests to prevent any regression.
Current code should also now properly auto-renew sessions maxAge on every request (touched when restored from cookie). Tell me if the current behaviour works for you.
Just released v0.14.0.

@mgcrea Thank you very much, everything is working fine now!


CELO native transfer push notification

Expected Behavior
"As a user I expect to get push notification when I received CELO transfer to my account."
Current Behavior
What is the current behavior?

@jeanregisser , in another issue @gnardini commented that this issue is already part of the https://github.com/celo-org/celo-monorepo/issues/6533

@jeanregisser , in another issue @gnardini commented that this issue is already part of the https://github.com/celo-org/celo-monorepo/issues/6533

Yes as discussed on Slack, we want to keep these 2 issues separate.


Some characters are not rendered

Attach (recommended) or Link to PDF file here: Can be reproduced on the demo page:
https://mozilla.github.io/pdf.js/web/viewer.html
record.pdf
Configuration:

Web browser and its version: chrome 118.0.5993.89
Operating system and its version: win11 22000.556
PDF.js version:
Is a browser extension: No

Steps to reproduce the problem:

Open the attached pdf on the demo page

What is the expected behavior? (add screenshot)

What went wrong? (add screenshot)
Some text is not displayed：

This is similarly broken in Adobe Reader (i.e. the PDF reference implementation) as well, and there's even a warning message about a broken font, hence this is clearly an issue with the PDF document itself.


[README] Update Django commands in Docker container

README reflects the sourced Python3 venv in the bashrc.

True, I missed one bash command. I also removed the installation of dependencies from the requirements.txt


bug: Unable to Scan Glitter Insemination Barcode

Plugin(s)

[X] Barcode Scanning
[ ] Face Detection
[ ] Translation

Did you test the latest version?

[X] I use the latest version

Platform(s)

[ ] Android
[X] iOS
[ ] Web

Current behavior
I'm struggling to scan these barcode labels, and it only works very rarely. I'm using an iPad Air. I have another app that can scan them without any issues.

Expected behavior
To scan the codebar correctly
Reproduction
https://github.com/FVolral/.capacitor-app
Steps to reproduce
$ npm run build && npx cap sync && npx cap open ios
The reproduction should try to scan the barcode use readBarcodesFromImage() but it doesn't work, not sure if it's me or a bug from the plugin. a working repro that use it may help.
Other information
My reproduction is not working, and I haven't been able to find a valid reproduction scenario. So, I did what I could, but it leads to a 'File does not exist' error when I'm trying to use the 'readBarcodesFromImage()' method. As far as I can tell, the path should be correct since the images load correctly in the DOM, right?".
Also, on my iPad, I can scan these insemination glitter barcodes almost instantly using an app called 'QR Reader' developed by Tap Media.
Those barcode use CODE_128 formats.
Capacitor doctor
💊   Capacitor Doctor  💊
Latest Dependencies:
@capacitor/cli: 5.3.0
@capacitor/core: 5.3.0
@capacitor/android: 5.3.0
@capacitor/ios: 5.3.0
Installed Dependencies:
@capacitor/android: not installed
@capacitor/cli: 5.3.0
@capacitor/core: 5.3.0
@capacitor/ios: 5.3.0
[success] iOS looking great! 👌
Before submitting

[X] I understand that incomplete issues (e.g. without reproduction) are closed.

Hi @FVolral,
thank you for your request.

As far as I can tell, the path should be correct since the images load correctly in the DOM, right?".

No, you need to provide a native path, not the web path. Here you can find an example: https://github.com/robingenz/capacitor-mlkit-plugin-demo/blob/main/src/app/modules/barcode-scanning/barcode-scanning.page.ts#L86:L98
I tried it myself and i can reproduce your issues. But I am not sure if the plugin can do much here. After all, the mlkit sdk is responsible for detecting the barcodes.
It's best to create an issue in this repository: https://github.com/googlesamples/mlkit
But I will leave this issue open for now. Please update this issue if you have any news.

Yep, that's what I thought. I will open an issue on mlkit. Thank you for your response.


Preparation or improvement of the guide for presentation of DynPy environment to scholars of InterCars foundation (issue #975 related)

The goal is to prepare or improve a guide or report that is generated by the following code:
from dynpy.utilities.documents.guides import IntroDynPyProjectGuidelines
IntroDynPyProjectGuidelines()

Minor changes introduced. The code was checked with the following call:
from dynpy.utilities.documents.guides import IntroDynPyProjectGuidelines
IntroDynPyProjectGuidelines()


Problems with callbacks functions

Hello,
I started using the callback functions (which are an awesome add) but I have a problem with it.
Even when using different urls, only the first one is always used (also if I try to go to /admin, it redirect to /)
If I comment the login header redirection, there is just no redirections at all (even on logout and session_unset() is not ran)
get('/', 'app/index/index.php');

get('/user/login', function(){
    $_SESSION['logged'] = true;
    header('Location: /');
});

get('/user/logout', function(){
    session_unset();
    header('Location: /');
});

get('/admin', 'app/admin/index.php');

Any idea why this happen?
EDIT : I added the test callback (the same one as website show) before the login callback and when I go to /user/logout, it just show Awesome.

I have updated the code. Multiple callbacks and multiple routes should work now.

Updated router.php. Now it is possible to have multiple routes and multiple callbacks.


ValueError: No JSON object could be decoded

After conversion and invoking CouchPotato renamer.scan, script outputs the following in SABnzbd:
Traceback (most recent call last):
File "/home/plex/sickbeard_mp4_automator/SABPostProcess.py", line 74, in
autoProcessMovie.process(path, settings, nzb, sys.argv[7])
File "/home/plex/sickbeard_mp4_automator/autoprocess/autoProcessMovie.py", line 93, in process
rstate = r.json()
File "/usr/lib/python2.7/dist-packages/requests/models.py", line 741, in json
return json.loads(self.text, **kwargs)
File "/usr/lib/python2.7/json/init.py", line 338, in loads
return _default_decoder.decode(s)
File "/usr/lib/python2.7/json/decoder.py", line 366, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/usr/lib/python2.7/json/decoder.py", line 384, in raw_decode
raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded

Possibly fixed, I am using Ubuntu and didn't realize the instructions for configuring CouchPotato via the guide here on GitHub specified double backslashes for Windows OS using backslashes in paths. Changed path in main.py to have single forward slashes. Will confirm correction on next completed download.

Check your couch potato settings in autoProcess.ini. This error usually pops up when they're wrong and it's failing to talk to CP
Sent from my iPhone

On Mar 27, 2016, at 12:56, enz1ey<EMAIL_ADDRESS>wrote:
After conversion and invoking CouchPotato renamer.scan, script outputs the following in SABnzbd:
Traceback (most recent call last):
File "/home/plex/sickbeard_mp4_automator/SABPostProcess.py", line 74, in
autoProcessMovie.process(path, settings, nzb, sys.argv[7])
File "/home/plex/sickbeard_mp4_automator/autoprocess/autoProcessMovie.py", line 93, in process
rstate = r.json()
File "/usr/lib/python2.7/dist-packages/requests/models.py", line 741, in json
return json.loads(self.text, **kwargs)
File "/usr/lib/python2.7/json/init.py", line 338, in loads
return _default_decoder.decode(s)
File "/usr/lib/python2.7/json/decoder.py", line 366, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/usr/lib/python2.7/json/decoder.py", line 384, in raw_decode
raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub


KFLUXINFRA-1163: Updating Host Config for MPC

With Addition of kflux-prd-rh02 in the list, this pull request will retructure the production overlay by further creating base, stone-prd-rh01 and kflux-prd-rh02 overlays.

@gbenhaim could you please review the changes?

@manish-jangra This PR now has a merge conflict with latest main, you need to manually rebase it


[Hub Generated] Publish private branch 'roni93-security-Microsoft.Security-2023-02-01-preview'

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

What's the purpose of the update?

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

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

Contribution checklist (MS Employees Only):

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

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

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

Change to data plane APIs
Adding new properties
All removals

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

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

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

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

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

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

The PR is created based on the updates in the private branch. The updates in the PR has already been reviewed and approved with this PR Azure/azure-rest-api-specs-pr/11328

Hi @XiaofeiCao,
This PR were created from private PR and all the reviewers approved it (ARM, SDK), you can see it here:
https://github.com/Azure/azure-rest-api-specs-pr/pull/11328#issuecomment-1463117106
About the falling checks:
Swagger LintDiff failed - I do have operation for top level subscription:
"/{scope}/providers/Microsoft.Security/healthReports"
Get a list of all health reports inside a scope. Valid scopes are: subscription (format: 'subscriptions/{subscriptionId}'), or security connector (format: 'subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Security/securityConnectors/{securityConnectorName})'
We have in MDC similar endpoint:
https://learn.microsoft.com/en-us/rest/api/defenderforcloud/governance-rules/list?tabs=HTTP
Swagger BreakingChange failed
I'm not sure why this test is falling. My API is a new one and there isn't any breaking change because it's a new entity.
Maybe it's because I started the PR in February and now it March?

@roni93 Thanks for the explanation, I'll go merge.

Signed off by ARM:
https://github.com/Azure/azure-rest-api-specs-pr/pull/11328#issuecomment-1462786219


add favicon to index.html

yup , sorry i didnt notice that , i've already add it to gitagnore

Can you also remove the .DS_Store files?  Then they disappear here from the Pull Requests chagnes


Integration Test - Release - 2019/02/21 17:38:31

Auto create for integration test. IssueCommandFlow

#pri1

Just type '#backlog'

#pri2

#backlog

#please-close


添加卡密有错误啊

导入卡密点击提交出现：Whoops, looks like something went wrong.怎么回事啊？真难搞

导入卡密点击提交出现：Whoops, looks like something went wrong.怎么回事啊？真难搞

如果强制https访问，那么.env里面配置需要开启https，具体看.env里面配置说明

大佬，能给我说下，我都升级了最新版本了，为什么添加商品还是提示需要添加图片。这个怎么搞的。
ashang<EMAIL_ADDRESS>于 2020年7月8日周三 21:34写道：

Closed #52 https://github.com/assimon/dujiaoka/issues/52.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/assimon/dujiaoka/issues/52#event-3525252317, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AQGZPX6SBGGR4NYZLWS3MQ3R2RYU7ANCNFSM4OUIVZHQ
.


Change from trading_calenders to exchange_calendars

trading_calenders has been marked unmaintained by developer. It is producing errors during build process on 3.9. Removed trading_calenders and replaced it with exchange_calenders

trading_calenders has been marked unmaintained by developer. It is producing errors during build process on 3.9. Removed trading_calenders and replaced it with exchange_calenders

@yoshishima
Please add erl to requirements.txt as well. The build is broken
ERROR: finrl.test (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: finrl.test
Traceback (most recent call last):
  File "/opt/conda/lib/python3.6/unittest/loader.py", line 428, in _find_test_path
    module = self._get_module_from_name(name)
  File "/opt/conda/lib/python3.6/unittest/loader.py", line 369, in _get_module_from_name
    __import__(name)
  File "/home/finrl/test.py", line 1, in <module>
    from elegantrl.agent import *
ModuleNotFoundError: No module named 'elegantrl'

----------------------------------------------------------------------
Ran 9 tests in 3.290s

FAILED (errors=1)
Error: Process completed with exit code 1.

cc @BruceYanghy @XiaoYangLiu-FinRL


Wildcard support for server names

if you have several servers in a type:
survival-1
survival-2
creative-1
creative-2

you may wanna use survival-* and creative-* to support both server groups maybe?
or you can create a new feature called server groups. I strongly recommend the server group way.
then we can have server groups like: survivals and creatives

See my replies here (since they kinda relate to this):
https://github.com/lucko/LuckPerms/issues/255
Wildcard servers were previously allowed, however, they were removed as they were hurting performance. An alternative was added in the form of static contexts.
You should use static contexts, to add a "server-type" context for each of your servers, and then set that to "creative" or "survival".
These static contexts are defined in the contexts.json file.

nice, it helped a lot! thanks!


ManagedIdentity for VMSS is not shown as SystemAssigned

What steps did you take and what happened:
Created an AKS cluster with Managed Identity, Kubernetes versions 1.16.9. Below are the environment details. Is there anything else I need to do to assign managed Identity to the nodepool?
What did you expect to happen:
Tried to get the identity assigned to the nodepool as specified in the link. Expected the type of identity as SystemAssigned, but actual result is UserAssigned
Anything else you would like to add:
NA
Which access mode did you use to access the Azure Key Vault instance:
System Assigned Managed Identity

Environment:

Secrets Store CSI Driver version: (use the image tag):
Azure Key Vault provider version: (use the image tag):
Kubernetes version: 1.18.2 and kubectl get nodes -o wide):
Cluster type: AKS

@jluk Should we update the doc?
This is what it currently says, but that doesn't match the output @NileshGule posted.
For AKS clusters, system assigned managed identities can be created on your behalf. To learn more read here about AKS managed identity support.

AKS does not support user assigned managed identities yet, only system assigned managed identities. Until this gap is covered, use a service principal or system assigned identities with AKS.

@tomgeske / @mlearned can you update the MSI doc based on this feedback, I agree the terminology is misleading. Perhaps clarifying custom BYO identities are what are not supported.

Yes, agree, we should align terminology in CSI driver docs with our MI docs https://aka.ms/aks/mi

Perhaps clarifying custom BYO identities are what are not supported.

I think in the context of CSI driver this is not a limitation that the Azure Key Vault Provider doc needs to call out as the objective here s for the user to assign the MSI manually to the VMSS or use pod-identity to assign the MSI.

@aramase I don't think user need to manually assign the MSI to VMSS. When I create the cluster using assign-managed-identities option the VMSS pool is assigned an Identity. I granted the access to AKV using access policy to this identity and was able to pull the secrets at the time of starting the pod.
I demonstrated this in a YouTube video https://youtu.be/MHm4IVGVO1w
Let me know if my understanding is wrong.

@aramase I don't think user need to manually assign the MSI to VMSS. When I create the cluster using enable-managed-identities option the VMSS pool is assigned an Identity. I granted the access to AKV using access policy to this identity and was able to pull the secrets at the time of starting the pod.

@NileshGule That's correct. Cluster with enable-managed-identities will contain a cluster User MSI. You can use the cluster MSI to access Key Vault contents or create another managed identity with scoped permissions only for Key Vault. If you create another MSI, then that needs to be manually assigned.
We have that documented here - https://github.com/Azure/secrets-store-csi-driver-provider-azure/blob/master/docs/user-assigned-msi-mode.md#vmss-user-assigned-managed-identity
You can create AKS with managed identities now and then you can skip steps 1 and 2. To be able to get the CLIENT ID, the user can run the following command

az aks show -g <resource group> -n <aks cluster name> --query identityProfile.kubeletidentity.clientId -o tsv

But I think we need to modify this snippet in the doc to align with https://aka.ms/aks/mi  -
AKS does not support user assigned managed identities yet, only system assigned managed identities. Until this gap is covered, use a service principal or system assigned identities with AKS.

I just made a docs PR.

Closed with #154


Working as expected on first line diagnostic.start

If Typescript diagnostic starts on line 0, fork-ts-checker-webpack-plugin as of 0.5.1 fails and throws an error, which it shouldn't. This humble and small PR fixes it :) I'll update package.json and changelog.md if needed.

@johnnyreilly looks like the tests were not passing in master as well due to fs.existsSync changes in node v10. Given that existsSync is deprecated, I took the liberty of creating a helper to emulate it :)

Should be released in 30 mins or so - thanks for the contribution! 🌻♥


How do you install irkernel to work with Hydrogen?

I'm trying to run R in atom. I have installed the language-r and hydrogen packages. and have also downloaded both R and Anocanda. However I have no idea how to install the IRkernel. The instructions here https://github.com/IRkernel/IRkernel makes no sense to me.

https://irkernel.github.io/installation/

@kylebarron  running the first command in the R terminal executes but

IRkernel::installspec()
gives the following error

Error in IRkernel::installspec() :
  jupyter-client has to be installed but "jupyter kernelspec --version" exited with code 127.
In addition: Warning message:
In system2("jupyter", c("kernelspec", "--version"), FALSE, FALSE) :
  '"jupyter"' not found

isn't jupyter installed by default anocanda?

Usually yes, but you may have to install it manually

How do I go about doing that?

If you use Anaconda terminal, it's already installed

I figured out the problem. It wasn't installing because I was executing the commands in the R terminal rather than running R from the anaconda terminal and then executing the commands.


@rollup/plugin-virtual does not work

Opening this issue as an enhancement following the relabelling of https://github.com/vitejs/vite/issues/1428
Is your feature request related to a problem? Please describe.
@rollup/plugin-virtual is currently not working. Checking the implementation, it looks like it may be workable.
I create a repro for this issue starting from the vanilla create app:
https://github.com/matias-capeletto/vite-rollup-plugin-virtual
It is using the example from the plugin docs (batman and robin are from there)
I am also using it with { enforce: 'pre', plugin } so it is applied before the rest of vite core plugins:
  plugins: [
    {
      enforce: 'pre', ...virtual({
        batman: `export default 'na na na na na'`,
        'src/robin.js': `export default 'batmannnnn'`
      })
    },
  ]

Additional context
It looks like there are two unrelated issues. Vite doesn't like the
import robin from "./robin.js";

$ vite build
building for production...
Could not resolve './robin.js' from main.js
error during build:
Error: Could not resolve './robin.js' from main.js

When removing this line, the other virtual file also fails. I think it is related to the prefix used by the plugin
const PREFIX = `\0virtual:`;

In Dev the error reads:
Uncaught TypeError: Failed to resolve module specifier "/@id/

During build it correctly compiles and runs through.
System Info
vite version: 2.0.0-beta.19
Operating System: Windows 10 Home, 10.0.18363 N/A Build 18363
Node version: v15.5.0
Package manager and version: yarn v1.22.10

Note: only the null byte needed to be fixed. The src/robin.js fails simply because robin.js isn't actually inside a src directory.

Note: only the null byte needed to be fixed. The src/robin.js fails simply because robin.js isn't actually inside a src directory.

Sorry for that second non-issue, and thanks for the null byte fix 🙌🏼
Updated @rollup/plugin-virtual to compatible in http://vite-rollup-plugins.patak.dev

Sorry for that second non-issue, and thanks for the null byte fix 🙌🏼
Updated @rollup/plugin-virtual to compatible in http://vite-rollup-plugins.patak.dev

Im trying to use virtual plugin set it in vitejs.dev but i dont know how it have to works, I put that plugin in my vite.config.js
function myPlugin() {
	const virtualFileId = '@my-virtual-file'

	return {
	  name: 'my-plugin', // required, will show up in warnings and errors
	  resolveId(id) {
		console.log('resolveId', id);
		if (id === virtualFileId) {
		  return virtualFileId
		}
	  },
	  load(id) {
		if (id === virtualFileId) {
		  return `export const msg = "from virtual file"`
		}
	  }
	}
  }
export default {
	// build: {
	// 	rollupOptions: {
	// 		input: 'entry'
	// 	}
	// },
	entry: 'src/main.js',
	plugins: [
		vuePlugin(),
		{
			enforce: 'pre',
			...myPlugin({
				batman: `export default 'na na na na na'`,
			})
		},
	]
}

Even I tried to set a different input like rollup docs, but does not works
Vite version: 2.0.0-beta.30

I dont know if I'm doing something wrong or if I dont understand how it would be working

Im trying to use virtual plugin set it in vitejs.dev but i dont know how it have to works, I put that plugin in my vite.config.js
function myPlugin() {
	const virtualFileId = '@my-virtual-file'

	return {
	  name: 'my-plugin', // required, will show up in warnings and errors
	  resolveId(id) {
		console.log('resolveId', id);
		if (id === virtualFileId) {
		  return virtualFileId
		}
	  },
	  load(id) {
		if (id === virtualFileId) {
		  return `export const msg = "from virtual file"`
		}
	  }
	}
  }
export default {
	// build: {
	// 	rollupOptions: {
	// 		input: 'entry'
	// 	}
	// },
	entry: 'src/main.js',
	plugins: [
		vuePlugin(),
		{
			enforce: 'pre',
			...myPlugin({
				batman: `export default 'na na na na na'`,
			})
		},
	]
}

Even I tried to set a different input like rollup docs, but does not works
Vite version: 2.0.0-beta.30

I dont know if I'm doing something wrong or if I dont understand how it would be working

@kasec, given that you commented in this issue, if you are trying to use @rollup/plugin-virtual, there are instructions for it in http://vite-rollup-plugins.patak.dev. There is no need to use enforce pre with it.
I updated the list to use 2.0.0-beta.30, and added a snippet for the app code usage of this plugin. Press the + in virtual to check it out.
The page is using this plugin, configured here, and the virtual files are used here.
But I see that you are defining your own plugin, following the example in Vite plugin docs. To use that particular example, you need to import msg from '@my-virtual-file' somewhere in your code and it should work. If it doesn't, please create a repro and open a new issue, as this would be unrelated to this one.

@kasec, given that you commented in this issue, if you are trying to use @rollup/plugin-virtual, there are instructions for it in http://vite-rollup-plugins.patak.dev. There is no need to use enforce pre with it.
I updated the list to use 2.0.0-beta.30, and added a snippet for the app code usage of this plugin. Press the + in virtual to check it out.
The page is using this plugin, configured here, and the virtual files are used here.
But I see that you are defining your own plugin, following the example in Vite plugin docs. To use that particular example, you need to import msg from '@my-virtual-file' somewhere in your code and it should work. If it doesn't, please create a repro and open a new issue, as this would be unrelated to this one.

Ok, thank you buddy, I really misunderstood about how to implement it, and yesterday before my comment https://vite-rollup-plugins.patak.dev had not the implementation code in virtual-code.
Excellent job!

Ok, thank you buddy, I really misunderstood about how to implement it, and yesterday before my comment https://vite-rollup-plugins.patak.dev had not the implementation code in virtual-code.
Excellent job!


Expandable item collapses the wrong section

https://drive.google.com/open?id=119dnaFUKtRZ0VgedsaOow-4aKCXkDxeX
Please check this video to understand the issue. When I click on 'Folder Pictures', the section above it 'Folder Movies' gets collapsed automatically for some reason.
I have set these options on the adapter
flexibleAdapter.expandItemsAtStartUp()
.setAutoCollapseOnExpand(false)
.setRecursiveCollapse(true)
.setAutoScrollOnExpand(true);
Removing them didn't help either. In the OnClick of the ViewHolder, I'm just calling toggleExpansion.
The items having folder in the title are expandable and have sub-items on them. The items having Album at the beginning don't have any sub items.

Ah, true...some titles ended up being the same. Thanks.


This project is dead

If I'm wrong, please let me know...but if I'm right, a notice stating so would be community focused.

Take a look at https://github.com/humanmade/network-media-library/issues/74, I'm not 100% sure of the state of the repo but I do know for new things we'd look towards AMF

That looks promising, thanks.


Streaming Solution: add VAD

accounts to #64
Activate the VAD setting for Whisper transcription and see what it does to the system.

We are using FasterWhsiper VAD


How to retain EventHandlerId

Consider this:
<button class="btn btn-primary"
        onclick="@(e => IncrementCount(1))">
    Click me +1
</button>

@for (int t = 2; t != 4; t++)
{
    <button class="btn btn-primary"
            onclick="@(e => IncrementCount(t))">
        Click me +@t
    </button>
}

@functions {
  int currentCount = 0;

  void IncrementCount(int t)
  {
      currentCount += t;
      this.StateHasChanged();
  }

blazor will generate three EventHandlerId.
When I press the button, the first EventHandlerId is retained (just because the ladba "point" at the same address) but the others are simple new labda expression, so blazor will delete and recreate the handler. In a list with many elements and many handler, it's a performance problem.
a way to retain can be:
@for (int t = 2; t != 4; t++)
{
    <button class="btn btn-primary"
            onclick="@Action(t, e => IncrementCount(t))">
        Click me +@t
    </button>
}

  Dictionary<MethodInfo, Dictionary<object, Action<UIEventArgs>>>
    _retainedActions = new Dictionary<MethodInfo, Dictionary<object, Action<UIEventArgs>>>();

  public Action<UIEventArgs> Action<T>(T args1, Action<UIEventArgs> action)
  {
      if (_retainedActions.TryGetValue(action.Method, out var list) == false)
      {
          list = new System.Collections.Generic.Dictionary<object, Action<UIEventArgs>>();
          _retainedActions[action.Method] = list;
      }

      if (list.TryGetValue(args1, out var value) == false)
      {
          value = (e) => action(e);
          list[args1] = value;
      }
      return value;
  }

other ideas?

I've found another way: blazor can only change the delegate for eventBindings dictionary, keeping same EventHandlerId.
I do not think there are contraindications: @SteveSandersonMS  what do you think?
here https://github.com/uazo/Blazor/tree/retain-eventhandlerid code and samples
...
                if (oldFrame.AttributeEventHandlerId > 0)
                {
                    // DONT REMOVE, SIMPLY CHANGE
                    //diffContext.BatchBuilder.DisposedEventHandlerIds.Append(oldFrame.AttributeEventHandlerId);
                    diffContext.Renderer.AssignEventHandlerId(ref newFrame, oldFrame.AttributeEventHandlerId);
                    newFrame = oldFrame;
                }
...

    internal void AssignEventHandlerId(ref RenderTreeFrame frame, int id)
    {
        if( id <= 0 ) id = ++_lastEventHandlerId;

        if (frame.AttributeValue is MulticastDelegate @delegate)
        {
            if (_eventBindings.ContainsKey(id))
                _eventBindings[id] = new EventHandlerInvoker(@delegate);
            else
                _eventBindings.Add(id, new EventHandlerInvoker(@delegate));
        }

        frame = frame.WithAttributeEventHandlerId(id);
    }

that code doesn't compile...

Making it static all is working as expected.
It's because EventHandlerId is a marker for the javascript code for the current delegate (c# side), regardless of what it is.
Maybe a problem happens for nested events, with multiple StateHasChanged(), just because is StateHasChanged (when calling ProcessRenderQueue)  to assign the current EventHandlerInvoker. So, if a code change the delegate in a event, and in the browser there is another event that what to call the older delegate, my code doesn't works.
I try to code an example.

there is no way (for now :) to generate nested events.
I tried with webcomponents but also the current code returns unexpected results

Trying with https://github.com/aspnet/Blazor/issues/802#issuecomment-387953426, it works very well.


Navigation with j/k/h/l breaks, when lazyredraw is set

Normal navigation does not work when I have lazyredraw set in my Neovim config file.
I normally don't use vscode, but currently due to my nightly build of Neovim lagging behind some plugins were breaking my Neovim. So I was using Vscodium (I have not tried on normal vscode, as I suppose they are mostly the same).
Can anyone else reproduce this?
EDIT: Tried on the latest version, still the same issue.

Normal navigation does not work when I have lazyredraw set in my Neovim config file.

What kind of navigation?

I normally don't use vscode, but currently due to my nightly build of Neovim lagging behind some plugins were breaking my Neovim. So I was using Vscodium (I have not tried on normal vscode, as I suppose they are mostly the same).

You can just check for extension in your configuration.

Normal navigation does not work when I have lazyredraw set in my Neovim config file.

What kind of navigation?

I normally don't use vscode, but currently due to my nightly build of Neovim lagging behind some plugins were breaking my Neovim. So I was using Vscodium (I have not tried on normal vscode, as I suppose they are mostly the same).

You can just check for extension in your configuration.

What kind of navigation?

Navigations with j/k/h/l when in Normal Mode. The cursor just doesn't move.

What kind of navigation?

Navigations with j/k/h/l when in Normal Mode. The cursor just doesn't move.

No, can't reproduce.

No, can't reproduce.

I tried on vscode too. And can still reproduce it :confused:

I tried on vscode too. And can still reproduce it :confused:

Try in clear environment. Maybe the issue in your plugins / settings.

Try in clear environment. Maybe the issue in your plugins / settings.

Clear the whole init.vim. Only thing left is:

set lazyredraw

Still having the issue.
@Shatur95 which OS are you on?

Clear the whole init.vim. Only thing left is:

set lazyredraw

Still having the issue.
@Shatur95 which OS are you on?

@Shatur95 which OS are you on?

Archlinux with latest neovim nightly.

@Shatur95 which OS are you on?

Archlinux with latest neovim nightly.

I am on Arch Linux too, I tried it on a different machine. With a fresh install of neovim-nightly-bin and vscodium-bin both from AUR. With no other extension/plugin in either vscodium/neovim.
I was able to reproduce it.

I am on Arch Linux too, I tried it on a different machine. With a fresh install of neovim-nightly-bin and vscodium-bin both from AUR. With no other extension/plugin in either vscodium/neovim.
I was able to reproduce it.

Need to force nolazyredraw in options

Need to force nolazyredraw in options

I have a similar problem. I am on windows nvim. Let me try nolazyredraw first.


feat(expr): switch to fancy-regex crate & update the original version

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
resolve #12119

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] 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

[ ] 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.

The expression (regex) in fancy-regex will be essentially parsed to a ExprTree when building by RegexBuilder, thus we do not need to explicitly specify -i flag, but to integrate it with the input pattern.
Also, we support back-reference and positive/negative lookahead in the regex now, which means the following query should be valid at present:
query T
select regexp_replace('foobarbaz', 'a(?=r)', 'X');
----
foobXrbaz

Part of Regex::new_options is as below:
fn new_options(options: RegexOptions) -> Result<Regex> {
        let raw_tree = Expr::parse_tree(&options.pattern)?;

        // wrapper to search for re at arbitrary start position,
        // and to capture the match bounds
        let tree = ExprTree {
            expr: Expr::Concat(vec![
                Expr::Repeat {
                    child: Box::new(Expr::Any { newline: true }),
                    lo: 0,
                    hi: usize::MAX,
                    greedy: false,
                },
                Expr::Group(Box::new(raw_tree.expr)),
            ]),
            ..raw_tree
        };
...
}

Are there some inputs can make the expression panic?

Are there some inputs can make the expression panic?

Yes, like if the input pattern can not be parsed when trying to RegexBuilder::build(), an Error will be returned.
And in the current implementation we just simply call unwrap() on it, which will cause panic in this case.
We could definitely make a wrapper to handle this if necessary.
The below is all the Error that may be triggered during the backtracking regex approach.
/// An error as the result of parsing, compiling or running a regex.
#[derive(Debug)]
pub enum Error {
    /// An error as a result of parsing a regex pattern, with the position where the error occurred
    ParseError(ParseErrorPosition, ParseError),
    /// An error as a result of compiling a regex
    CompileError(CompileError),
    /// An error as a result of running a regex
    RuntimeError(RuntimeError),

    /// This enum may grow additional variants, so this makes sure clients don't count on exhaustive
    /// matching. Otherwise, adding a new variant could break existing code.
    #[doc(hidden)]
    __Nonexhaustive,
}


browser & version about this plugin support.

[ ] Bug
[ ] Feature Request
[x] Question
I want to know the browser & version about this plugin support.

@STRML, what browser features are you using if any? Since jQuery isn't used I'm assuming it's just straight JavaScript but I'm also curious about the inner workings.