Creating a custom range facet with connectRange AND using routing fires an incorrect warning

🐛 Bug description
Using any form of routing with a custom range facet created using connectRange will fire a warning about range needing either rangeInput or rangeSlider.
🔍 Bug reproduction
Steps to reproduce the behavior:

Go to https://g2vyw.csb.app/?instant_search[range][price]=22%3A321321
Open your console
See incorrect warning

Live reproduction:
https://codesandbox.io/s/objective-darwin-g2vyw?file=/src/app.js
💭 Expected behavior
No warning should be fired
Environment

OS: macOS
Browser: Chrome
Version: 83
InstantSearch.js Version 4.7.0

It seems to come from the fact that connectRange always set the $$type to ais.range.
https://github.com/algolia/instantsearch.js/blob/ada37ae054275d46cc2edaee0081562b6c35cbc8/src/connectors/range/connectRange.js#L74-L75
and checkIndexUiState assumes that a range can only have ais.rangeInput or ais.rangeSlider NOT ais.range
https://github.com/algolia/instantsearch.js/blob/ada37ae054275d46cc2edaee0081562b6c35cbc8/src/lib/utils/checkIndexUiState.ts#L57-L60

I've also tried the version with the virtual widget and the same warning is fired.
See the sandbox: https://codesandbox.io/s/modern-mountain-8ggjc?file=/src/app.js
or the live example: https://8ggjc.csb.app/?instant_search[range][price]=22%3A321321

Ah this is a great catch! in the widgets we provide the $$type directly, but ais.range should be accepted too https://github.com/algolia/instantsearch.js/blob/master/src/widgets/range-input/range-input.js#L170


Rails admin assignment quick fix

Fixes #416
Describe the changes you have made in this pr -
Removed unused code from circuit_elements
Screenshots of the changes (If any) -

Coverage decreased (-0.03%) to 83.922% when pulling 0f78fb0c10445dee8f8a94ccc439b1516b605eb9 on ankitkataria:fix/admin-assignment into 42eec0abf09f9cd15a10b4c4a52327a20c904c82 on CircuitVerse:master.


Add Graceful Shutdown

Shutdown gracefully shuts down the server without interrupting any active connections. Shutdown works by first closing all open listeners, then closing all idle connections, and then waiting indefinitely for connections to return to idle and then shut down. If the provided context expires before the shutdown is complete, Shutdown returns the context’s error, otherwise it returns any error returned from closing the Server’s underlying

Hey @anonychun, thanks for this PR! Is it possible to combine this functionality into a single function to be shared between api and router to reduce the code duplication?

Closing this PR since there is no interest on it from the contributor.


Support nonce for Apple login

Describe the feature you would like to see
When performing login with Apple, there are cases where a nonce is required in the response body to associate a client session and an ID token. I don't believe this option is yet supported by this library.
An implementation of this can be in my fork here. However it was written a while back and not compatible with the current codebase.

Thanks @chyt! This seems pretty straight forward thanks to your fork, I'll try to get it to the main branch shortly 🤞

Hey @chyt, could you give https://github.com/kreait/firebase-php/pull/658 a try and confirm that it works as expected? Unfortunately I can't really test this myself and can only work on the assumption that I implemented the requirements correctly 🤞

@jeromegamez I will give this a try in the next few days and report back.


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/instalator/ioBroker.vis-players

[ ] [E150] No common.connectionType found in io-package.json
[ ] [E152] No common.dataSource found in io-package.json
[ ] [E605] No actual year found in copyright. Please add "Copyright (c) 2021 instalator<EMAIL_ADDRESS>at the end of README.md
[ ] [E701] No actual year found in LICENSE. Please add "Copyright (c) 2021 instalator<EMAIL_ADDRESS>at the start of LICENSE

Thanks,
your automatic adapter checker.
P.S.: There is a community in Github, which supports the maintenance and further development of adapters. There you will find many experienced developers who are always ready to assist anyone. New developers are always welcome there. For more informations visit: https://github.com/iobroker-community-adapters/info

Do you need help fixing the bugs?


:intro shows different version then other pages

In Vim 8.1 it says 8.1.1 on the :intro page.
The other pages like :version etc. says 8.1.
At least to me this is confusing.
I would expect to see the same version number on all pages.

On my present Vim, :intro says version 8.1.92 and the first two lines of :version are:
VIM - Vi IMproved 8.1 (2018 May 17, compiled Jun 21 2018 14:52:47)
Included patches: 1-92

The patchlevel (i.e. the third number of what :intro reports) is (when nonzero) at the end of :version's second line. (When the patchlevel is zero, i.e. no additional patches since the minor release, that second line is omitted.) It has been so since as long as I knew Vim (6.1) and probably earllier. I see nothing contradictory or confusing in that. It's just that version 8.1.92 is shorthand for:
Major version: 8
Minor version: 1
Patchlevel: 92

What concerns the help pages, which say For Vim version 8.1, the patchlevel is irrelevant for them because most of them don't change from one patchlevel to the next.

If i go to https://www.vim.org/ and it says "Vim 8.1 is released!" i would expect it to say "version 8.1" when i start the program not 8.1.1.

You are just confused because your version is actually 8.1.1. Update to the latest version and you'll see 8.1.95 (as we speak).

Why not say the same version throughout the program?

Version 8.1 refers to all patchlevels from 8.1.0 to the current 8.1.95 and beyond. Similarly version 8.0, which isn't the latest one anymore, refers to all patchlevels from 8.0.0 to 8.0.1850. So it is the same version throughout, except that :intro gives you a more detailed info (the patchlevel) and :version is still more detailed (on its second line, when present, it gives you the list of all included patches in this build of this major.minor version).
"Version 8" could even be used to mean all versions and patchlevels from 8.0.0 (included) to some future 9.0.0 (excluded). For instance the helpfile version7.txt gives info about everything new and changed in version 7, from the changes in 7.0.0 relative to version 6 up to the 22 patchlevels of 7.4b beta. For some reason patchlevels 7.4.1 to 7.4.2367 are not detailed there (yet?), but IMHO they ought to be.
Best regards,
Tony.

For some reason patchlevels 7.4.1 to 7.4.2367 are not detailed there (yet?), but IMHO they ought to be.

They are listed in version8.txt. (I feel it is confusing.)
(I fixed a typo in the title.)


How to use the Sphere24 and Sphere54 templates to generate Coons patches

Hello,
As the title suggests, I'm trying to generate patches using the Sphere templates as mentioned in the paper. I can see in the scripts/run.py file that by default the category specific templates are used. Replacing the template paths to use the sphere24/54 folder, raises Unable to load checkpoint error.
For example, changing the following line in main method of scripts/run.py
edge_data = [line.strip().split(' ') for line in open(
        os.path.join('templates', args.category, 'edges.txt'), 'r')]

to
edge_data = [line.strip().split(' ') for line in open(
        os.path.join('templates', 'sphere24', 'edges.txt'), 'r')]

and similarly changing paths of variables junction_order, topology, adjacencies, vertex_idx, and processed_vertices
raises
Unable to load checkpoint
torch.Size([3, 128, 128])

Please let me know if any additional information is required. Any guidance will be appreciated. Thank you!

On further inspection I noticed that the edges.txt and junction_order.txt of both sphere templates are empty. I assume this is the reason for the error?


vendor: Consolidate onto github.com/munnerz/goautoneg

bitbucket.org/ww/goautoneg is only used by one dependency, and
glide is having trouble cloning it. We want to be using munnerz
anyway.

/lgtm


EVP_MAC ctrl numbering duplicate removal.

Both EVP_MAC_CTRL_SET_MD and EVP_MAC_CTRL_SET_CIPHER were numbered 4.
This would preclude any future MAC from using both.

Merged, thanks.

(Just noting that this only came in with the GMAC commit which has never been in a release, so renumbering in the public header is okay.)


HTML 5

is there any plan to update for HTML 5 ?

Can you describe what from HTML 5 is missing in our library?
Also this is more of a discussion so I am moving it there.


Multi select option for the file explorer

I think multi select option in the file explorer similar to visual studio would be helpful and save lot of time for the developers in file management.

@Umamaheswaran1990 please take the time to search for duplicates first.

@bpasero I did tried to find the open issues with similar feature request before creating this but i couldnt find any, If there is already a ticket for this issue, please feel free to close this ticket. Also it would be helpful if you could share the other tickets id.

@Umamaheswaran1990

@bpasero That's a helpful animation to see how you searched, but you didn't link the actual duplicate. I came here to post the exact same thing as @Umamaheswaran1990 and also had problems finding duplicate items. The filter labels definitely helped, but it still wasn't easy. That's probably because as of right now there are over 1000 issues open, lol.
For the record, this is a duplicate of #1023.

Closing as duplicate of https://github.com/Microsoft/vscode/issues/1023


GH-33750: [GLib] Add garrow_table_batch_reader_set_max_chunk_size()

Rationale for this change
This is a missing feature.
What changes are included in this PR?
This adds a binding.
Are these changes tested?
Yes.
Are there any user-facing changes?
Yes.

Closes: #33750

+1


[Bug]: // in Strings and String Literals is failing the workflow

Bumping this issue up to prod because it should have made it's way there by now if it was not fixed

Hey team! Please add your planning poker estimate with Zenhub @ayushpahwa @nsarupr


Unitialized Variable Warnings

AudioKit/Common/Internals/Sporth/ugens/t.c:146:22: Variable 'ft' is uninitialized when used here
AudioKit/Common/Internals/Sporth/ugens/argtbl.c:151:22: Variable 'at' is uninitialized when used here
Not sure how to fix these. Initializing to NULL will likely cause a crash.

Where are you compiling this, ie for iOS, macOS, or tvOS?  In any case, I do not get any such warnings in Xcode.

Compiling for iOS using the Xcode 8.3 beta 4 tools. If you look at sporth_tblsize(), it does seem pretty obvious by inspection that ft is unitialized.

This type of issue probably belongs on the https://github.com/PaulBatchelor/Sporth issues page since we would want any important updates to be done at the source.  We keep a slightly modified version of Sporth, mostly just to not include things we don't need or haven't implemented.  @PaulBatchelor what is your take on this?

I'll look into it. I usually catch these warnings. Hmmm...

Yup, I indeed see uninitialized behavior happening here with "tblsize"
https://github.com/audiokit/AudioKit/blob/master/AudioKit/Common/Internals/Sporth/ugens/t.c#L146
However, this has been fixed now:
https://github.com/PaulBatchelor/Sporth/blob/master/ugens/t.c#L146
The other initialized variable is "argtbl":
https://github.com/PaulBatchelor/Sporth/blob/master/ugens/argtbl.c#L151
Still an issue with my Sporth:
https://github.com/PaulBatchelor/Sporth/blob/master/ugens/argtbl.c#L151
Actually, tbh whole ugen is an issue if you ask me. I thought I removed it long ago...

All these warnings should be gone in the upstream version of Sporth: https://github.com/PaulBatchelor/Sporth/commit/163cb93562e6126b9051c923b50e15bf2c303b08

Okay, I will update Sporth on AudioKit shortly.

We're fully up to date with newest Soundpipe and Sporth as of https://github.com/audiokit/AudioKit/commit/3cf43e2d5a5be85538c10957c1db4fdc91853b80 so I am going to assume this is taken care of, but of course feel free to reopen as necessary.


simple bash install

Just creates a bash script that can be used to run and install. Has fail-safe in case someone doesnt read install guide and uses from root directory.
wget https://raw.githubusercontent.com/lnbits/lnbits/snapcraft/lnbits.sh &&
chmod +x lnbits.sh &&
./lnbits.sh

I am wondering if this method should not be the number two method - while keeping poetry as recommended option one.
Also I think we should not install soon-to-be unsupported Python 3.9. We should just make sure that LNbits run with all supported versions (up to 3.12 - PR https://github.com/lnbits/lnbits/pull/2619 takes care of this).

I am wondering if this method should not be the number two method - while keeping poetry as recommended option one.
Also I think we should not install soon-to-be unsupported Python 3.9. We should just make sure that LNbits run with all supported versions (up to 3.12 - PR #2619 takes care of this).

Sure, lets move to option 2, sqlalchemy seems like a big job tho and will require a good few weeks worths of work.


Error displaying speed limit on certain roads

Display 0 km/h at Avenue des grenouillères, Nice FR for example instead of 50 km/h

Supposed to be fixed by using unknown road type in commit e9794a83


Stretch Cluster Replica example is wrong regarding computernames.

The example for Stretch Cluster is bad:
Set-SRNetworkConstraint -SourceComputerName "SR-SRV01" -SourceRGName "Group01" -SourceNWInterface "Cluster Network 1","Cluster Network 2" -DestinationComputerName "SR-SRV03" -DestinationRGName "Group02" -DestinationNWInterface "Cluster Network 1","Cluster Network 2"
[Enter feedback here]
-SourceComputerName and -DestinationComputerName should be the cluster's name, not individual cluster nodes.

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

ID: 33bdd435-62e9-3ecf-0cdd-e0e78166a116
Version Independent ID: 5be2ee2d-01b4-cbde-b91c-72ad7ea0c57b
Content: Set-SRNetworkConstraint (StorageReplica)
Content Source: docset/winserver2019-ps/storagereplica/Set-SRNetworkConstraint.md
Product: w10
Technology: windows
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

@Mihaly7 Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/windows-powershell-docs/pull/2811
Thanks
Sri


Hide Folding icons in Live Preview

now "Hide Folding icons" only works in preview and edit mode

I believe this is solved as of Minimal 4.2, can you confirm?

Thanks, it's solved! Can you tell me please which css attributes control it? "collapse-indicator"?


page is out of date

Tried to follow along, did not match what I was seeing on my end. I think it's out of date

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

ID: 8751595d-4839-d2ee-1e2c-16d07ef84a2d
Version Independent ID: e66f96b3-5b88-9304-1ec0-9e3bb8a2cfbe
Content: How to manage app passwords - Azure Active Directory
Content Source: articles/active-directory/user-help/multi-factor-authentication-end-user-app-passwords.md
Service: active-directory
Sub-service: user-help
GitHub Login: @curtand
Microsoft Alias: curtand

@grantog Thanks for the feedback. We will engage the content development team for further investigation, they will verify and perform necessary changes in the document as needed.
@curtand Can you have a look at this ?

hey @grantog
Thanks for providing feedback that helps improve our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response.  We are currently updating all the end-user content and will be migrating to the Microsoft Support site. We are closing the issue as we are migrating the content to a new site. #please-close


Add forum module.

Add a forum that users can use for discussion. Will also need to add formCategory and formPost @leapon what is the standard for multiple words - nocaps, camelCase, snake_case, or-dashes

The choice is to use snake_case for module name. The module name is also used as module's filename. This way, the filename is not case sensitive.

Thanks

Why was this closed? I'm not done with this feature.

I thought your question is answered so closed. Didn't realize this is for forum module as whole.


Use system package

Description
My organization requires us to lock the utilities on our build applications to specific versions, and not install utilities through DevOps pipeline tasks. We would like to use the Trivy Azure DevOps Pipelines Task for its UI functionality, but the lack of option to use the package on the system prevents us from doing so.
This PR adds an 'useSystemInstallation' input, that runs the Trivy tool from the system PATH instead of installing Trivy as a temporary tool.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Amelia Wheeler seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


TouchableOpacity elements in previous screens in navigation stack are still accessible in next screen

I have a simple navigation stack with 3 screens. To simplify, lets say screen 1 has 3 buttons named button1, button2, etc. Screen 2 has 3 buttons as well. Now, when I go to the second screen, if I am highlighting the first button and I go left, it will go and highlight button3 from screen1 off-screen. I have confirmed this by adding a console.log to each button on every page to identify itself ("button3 on screen one is highlighted" etc).
Is the solution to replace all the buttons with an empty array in screen 1 when I go to screen 2, or is there a more correct way?

We are having the same issue. We have it with stack navigators like you @oscadev and found it is also an issue with material tab navigators as well. If the next tab has touchable elements, it will let me continue to focus onto the next tab from my active tab.
I did the same workaround you mentioned but run into a significant performance issue when switching tabs or going back in a stack as the app has to re-render all the imagery I am using in my touchable elements. I think I could cache the imagery and that would help, but that solution still seems pretty hacky to me. I noticed that RCTVView has a property called isTVSelectable but this property isn't exposed on touchable elements in react-native. If I'm understanding everything right (I'm only 1 year new with react-native), couldn't a property called isTVSelectable be added to touchable elements so when the screen they are on isn't active we can set it to false? I am exploring that solution myself now, but figured I'd post about it here incase I am on a wrong path.

@colin-oos Yes, I was going to recommend either caching/hiding without re-rendering. Just add a conditional to render the items or render null depending on a boolean like "showElements" that turns to "off" when you click to leave the page.

@colin-oos Yes, I was going to recommend either caching/hiding without re-rendering. Just add a conditional to render the items or render null depending on a boolean like "showElements" that turns to "off" when you click to leave the page.

Yeah, thats essentially what I'm doing now only without caching the images. The issue is, conditionally rendering either null or the touchable element will affectively re-render the touchable element and all of its children

@colin-oos if it is only re-rendering (without having to refetch data or do other tasks that take a long time) it should re-render instantaneously, I think.

@oscadev In my case I do need to refetch data. My app is loading the uri's of the imagery on child components. There is also potential that my app may require other API related tasks in the children of my touchable components. I believe I found a fix to the problem though, and this might be a more elegant solution for yourself too.
I opened up the property isTVSelectable on TouchableHighlight and TouchableOpacity. This way, instead of rendering null or another View when  the screen isn't active, you can set on the touchable components isTVSelectable to false. This disables the ability to focus on the views without re-rendering the view and its children. In fact, this property I feel like can be useful in many other ways than just react-navigation. I could see the potential case for wanted to grey out or disable touchable components in a TV app. So I may also make a PR to see what @dlowder-salesforce thinks.
For now, you can find my changes on the forked branch colin-oos/react-native-tvos#tvos-v0.61.5 I made if you want to give it a try and see if it works well for you.

@colin-oos thank you VERY much, I will take a look at your branch.

@colin-oos I have some cycles to work on this now... would you be able to share your test project where you are using the new isTVSelectable property on Touchables to fix the issue?

@colin-oos @oscadev I was able to put together a new project using the latest react-navigation that reproduces the issue.  See https://github.com/dlowder-salesforce/ReactNavigationTVDemo.  It uses a forked react-native-screens that fixes its tvOS compilation issues.  Now I will be able to try different approaches to fix the issue.

I do see that using isTVSelectable does fix the issue and doesn't require rerender.  However, in the simulator, I am seeing a weird behavior where when navigating back, it takes a long time (about 1 second) for the left most button in a screen to pick up the change to isTVSelectable and get the default focus that it should have.  Investigating.

Solution for me was initially to replace TouchableOpacity with normal View when screen is blurred and switch back when screen is focused. But due to this there were performance issues and focus issues. Then I sought to subclass RCTTVView to create a custom Selectable Button Element, exposing the isTVSelectable property and handles focus, onPress events

@keshavkaul @colin-oos I have invited you to the repo; once you accept, please review PR #62 .

#62 is merged, and appears in https://github.com/react-native-community/react-native-tvos/releases/tag/v0.61.5-1 .  Closing.

We had this same problem with react-navigation. Our solution was similar to the one you suggested @oscadev. When we navigate to a new screen, we render an empty View component for the screen underneath in the stack.

Hi, can you provide an example? I have the same problem and can't figure it out.


Snapshot ID syntax to select parent snapshot

What should restic do differently? Which functionality do you think we should add?
instead off  diff 1483848 3747374 it would be nice if we could do something like
diff 12345-1 12345 or better diff -1 12345
so we could also show the diff for the last snapshot with diff -1 latest
-2, -3, ... should work of course too. 🙂
calculation should only checks the parents.
maybe with another notation because the - will conflict with regular arguments.
maybe something like prev, prev-1, prev-2, ...
not sure if also next, next+1, ... would be helpful

I'm not sure if that's all well defined, which is why we do not have something like this yet. Before this is implemented, we need a spec (at least an informal one) what the notations should mean. I know that the calculation in git is very complex, although git manages trees (so ancestry information is more complex), so restic should be a bit easier.

Don't you think it can be "calculated" like the parent during backup with a notiation like git?
so find parent-snapshots with group_by host,path and use ~to specify the "revsion"
https://git-scm.com/docs/git-rev-parse#Documentation/git-rev-parse.txt-emltrevgtltngtemegemHEADmaster3em
https://stackoverflow.com/questions/2221658/whats-the-difference-between-head-and-head-in-git
restic diff latest~ latest
restic diff latest~1 latest
restic diff latest~3 latest~2
restic diff 6547812~3 6547812
....

Don't you think it can be "calculated" like the parent during backup with a notiation like git?

The big problem is that unlike git, restic will over time remove some of the intermediate snapshots. That is, to calculate the parent snapshot, we will first have to group snapshots like in the forget command and then move back / forwards within that group.

this sounds fine to me because I think this is that what users currently do.

Without #4026 or something similar (e.g. adding --group-by to every command which accepts snapshots), most commands won't know how to group snapshots. And therefore be unable to make that calculation.


Add whisper test cases #776

Test cases for whisper recognizer:
Raise a SetupError when the API key is missing.
Raise a ValueError when invalid audio data is provided.
Raise a SetupError with a specific error message when the openai module is not found.

As requested, here is the new pr. @ftnext

Thanks. I will review this after #783

Currently I adopted RESPX.
https://github.com/Uberi/speech_recognition/blob/3.13.0/tests/recognizers/test_openai.py
FYI: Don’t Mock What You Don’t Own
https://hynek.me/articles/what-to-mock-in-5-mins/


[release-1.26] update k8s.io/kubernetes to v1.26.11 in go.mod

What this PR does / why we need it:
Update dependencies to v1.26.11 go version to 1.20.12
Which issue this PR fixes(if applicable):
fixes #2490
Special notes for reviewers:

Release note:

NONE

/approve

/lgtm

/test pull-cloud-provider-openstack-check


BLUFI is BREAK -> Error - Custom Data and Others - Example (IDFGH-3893)

Hi,
The BluFi example is broken, cant work.
I (0) cpu_start: App cpu up.
I (572) cpu_start: Pro cpu start user code
I (572) cpu_start: Application information:
I (572) cpu_start: Project name:     blufi_demo
I (577) cpu_start: App version:      v4.3-dev-907-g6c17e3a64-dirty
I (584) cpu_start: Compile time:     Aug 25 2020 16:43:05
I (590) cpu_start: ELF file SHA256:  362b9a16f2835f76...
I (596) cpu_start: ESP-IDF:          v4.3-dev-907-g6c17e3a64-dirty
I (603) heap_init: Initializing. RAM available for dynamic allocation:
I (610) heap_init: At 3FFAFF10 len 000000F0 (0 KiB): DRAM
I (616) heap_init: At 3FFB6388 len 00001C78 (7 KiB): DRAM
I (622) heap_init: At 3FFB9A20 len 00004108 (16 KiB): DRAM
I (629) heap_init: At 3FFC9280 len 00016D80 (91 KiB): DRAM
I (635) heap_init: At 3FFE0440 len 00003AE0 (14 KiB): D/IRAM
I (641) heap_init: At 3FFE4350 len 0001BCB0 (111 KiB): D/IRAM
I (647) heap_init: At 4009D024 len 00002FDC (11 KiB): IRAM
W (655) flash_encrypt: Flash encryption mode is DEVELOPMENT (not secure)
I (661) spi_flash: detected chip: gd
I (665) spi_flash: flash io: dio
W (669) spi_flash: Detected size(8192k) larger than the size in the binary image header(2048k). Using the size in the binary image header.
I (684) cpu_start: Starting scheduler on PRO CPU.
I (0) cpu_start: Starting scheduler on APP CPU.
I (788) wifi:wifi driver task: 3ffcd670, prio:23, stack:6656, core=0
I (788) system_api: Base MAC address is not set
I (788) system_api: read default base MAC address from EFUSE
I (808) wifi:wifi firmware version: 8d94a49
I (808) wifi:wifi certification version: v7.0
I (808) wifi:config NVS flash: enabled
I (808) wifi:config nano formating: disabled
I (818) wifi:Init data frame dynamic rx buffer num: 32
I (818) wifi:Init management frame dynamic rx buffer num: 32
I (828) wifi:Init management short buffer num: 32
I (828) wifi:Init dynamic tx buffer num: 32
I (838) wifi:Init static rx buffer size: 1600
I (838) wifi:Init static rx buffer num: 10
I (848) wifi:Init dynamic rx buffer num: 32
I (848) wifi_init: rx ba win: 6
I (848) wifi_init: tcpip mbox: 32
I (858) wifi_init: udp mbox: 6
I (858) wifi_init: tcp mbox: 6
I (858) wifi_init: tcp tx win: 5744
I (868) wifi_init: tcp rx win: 5744
I (868) wifi_init: tcp mss: 1440
I (878) wifi_init: WiFi IRAM OP enabled
I (878) wifi_init: WiFi RX IRAM OP enabled
I (978) phy: phy_version: 4370, 4e803b3, Aug 11 2020, 14:18:07, 0, 0
I (978) wifi:mode : sta (4c:11:ae:db:1d:b0)
I (978) BTDM_INIT: BT controller compile version [270f54d]
I (1308) BLUFI_EXAMPLE: BD ADDR: 4c:11:ae:db:1d:b2
I (1308) BLUFI_EXAMPLE: BLUFI VERSION 0102
I (1308) BLUFI_EXAMPLE: BLUFI init finish
trying to send Custom Data return eternal loop of
E (136678) BT_BTC: btc_blufi_recv_handler seq 117 is not expect 1
E (136678) BLUFI_EXAMPLE: BLUFI report error, error code 0
E (136688) BT_L2CAP: l2ble_update_att_acl_pkt_num not found p_tcb
W (136688) BT_BTC: btc_blufi_send_encap wait to send blufi custom data
E (136708) BT_L2CAP: l2ble_update_att_acl_pkt_num not found p_tcb
W (136708) BT_BTC: btc_blufi_send_encap wait to send blufi custom data
E (136718) BT_L2CAP: l2ble_update_att_acl_pkt_num not found p_tcb
W (136718) BT_BTC: btc_blufi_send_encap wait to send blufi custom data
E (136738) BT_L2CAP: l2ble_update_att_acl_pkt_num not found p_tcb
W (136738) BT_BTC: btc_blufi_send_encap wait to send blufi custom data
If trying to request Version returns infinite loop of:
E (30858) BT_L2CAP: l2ble_update_att_acl_pkt_num not found p_tcb
W (30858) BT_BTC: btc_blufi_send_encap wait to send blufi custom data
If trying to request ScanWifi returns infinite loop of:
E (49298) BT_L2CAP: l2ble_update_att_acl_pkt_num not found p_tcb
W (49298) BT_BTC: btc_blufi_send_encap wait to send blufi custom data
If trying to send CUSTOM DATA it returns infinite loop of:
E (106428) BT_BTC: btc_blufi_recv_handler seq 32 is not expect 1
E (106428) BLUFI_EXAMPLE: BLUFI report error, error code 0
Please advise as it was working correct before update sdk.

Thanks for reporting and sorry for the inconvenience, we will look into.

@Alvin1Zhang  We have found some ways to trigger the issue, if the callbacks are outside the main file it will return unexpectedly behaviors.
We have test to build in v4.1 and it works but functions must be in the same file, in v4.3 doesnt matter it doesnt work.

@filzek
Can you describe in more detail how to reproduce this problem? It would be even better to have example code.

The example doesn't work as intent, we have found out some errors but we
cant recreate where they happen, but if the callbacks are not in the same
file the error occurs, just move the callbacks and calls to the same file
and it will work. But custom data dont.
Em qui, 3 de set de 2020 13:06, hb-kaertech<EMAIL_ADDRESS>escreveu:

Hello @Alvin1Zhang https://github.com/Alvin1Zhang & @xiewenxiang
https://github.com/xiewenxiang, I am hitting the same issue.
I have bisected it to fa729f3
https://github.com/espressif/esp-idf/commit/fa729f3805433ff1aa262978515a49a17f1246df
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/espressif/esp-idf/issues/5792#issuecomment-686594737,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG7JIU6AUJS5ZJYEDDSIOH3SD65G7ANCNFSM4QLBGFPQ
.

@filzek
How long is the custom data you are sending? Can you provide a simple demo that can be reproduced? If not, could you  describe the operating steps, and we will try to reproduce this scenario

@xiewenxiang : I have spend some time to make a reproducer:

Use esp-idf/examples/bluetooth/bluedroid/ble/blufi as is (I am using master branch)
Patch EspBlufiForAndroid with the following: (this adds random long custom data just after wifi provisioning)

diff --git a/app/src/main/java/com/espressif/espblufi/ui/BlufiActivity.java b/app/src/main/java/com/espressif/espblufi/ui/BlufiActivity.java
index bc7b052..0ef760b 100644
--- a/app/src/main/java/com/espressif/espblufi/ui/BlufiActivity.java
+++ b/app/src/main/java/com/espressif/espblufi/ui/BlufiActivity.java
@@ -144,6 +144,32 @@ public class BlufiActivity extends BaseActivity {
                 BlufiConfigureParams params =
                         (BlufiConfigureParams) data.getSerializableExtra(BlufiConstants.KEY_CONFIGURE_PARAM);
                 configure(params);
+
+                String dataLongStr = "AAAAAAAAAABBBBBBBBBB" +
+                                     "CCCCCCCCCCDDDDDDDDDD" +
+                                     "EEEEEEEEEEFFFFFFFFFF" +
+                                     "CCCCCCCCCCDDDDDDDDDD" +
+                                     "EEEEEEEEEEFFFFFFFFFF" +
+                                     "CCCCCCCCCCDDDDDDDDDD" +
+                                     "EEEEEEEEEEFFFFFFFFFF" +
+                                     "CCCCCCCCCCDDDDDDDDDD" +
+                                     "EEEEEEEEEEFFFFFFFFFF" +
+                                     "CCCCCCCCCCDDDDDDDDDD" +
+                                     "EEEEEEEEEEFFFFFFFFFF" +
+                                     "CCCCCCCCCCDDDDDDDDDD" +
+                                     "EEEEEEEEEEFFFFFFFFFF" +
+                                     "CCCCCCCCCCDDDDDDDDDD" +
+                                     "EEEEEEEEEEFFFFFFFFFF" +
+                                     "CCCCCCCCCCDDDDDDDDDD" +
+                                     "EEEEEEEEEEFFFFFFFFFF" +
+                                     "CCCCCCCCCCDDDDDDDDDD" +
+                                     "EEEEEEEEEEFFFFFFFFFF" +
+                                     "CCCCCCCCCCDDDDDDDDDD" +
+                                     "EEEEEEEEEEFFFFFFFFFF" +
+                                     "CCCCCCCCCCDDDDDDDDDD" +
+                                     "EEEEEEEEEEFFFFFFFFFF" +
+                                      "";
+                mBlufiClient.postCustomData(dataLongStr.getBytes());
             }

             return;

Build & flash everything
On the Android app, connect, then Configure Wifi.
Boom, I hit the issue

My guess about why https://github.com/espressif/esp-idf/commit/fa729f3805433ff1aa262978515a49a17f1246df is bad: btc_blufi_send_encap is being called from the btuT (aka bluedroid) task. This exact same task is the one able to process the queue. Thus calling vTaskDelay won't allow btuT to run and process the queue.

Hi, I'm facing the same issue when sending custom data from ESP32 to Android APP.
Endless looping the following two lines:
E (10789) BT_L2CAP: l2ble_update_att_acl_pkt_num not found p_tcb
W (10794) BT_BTC: btc_blufi_send_encap wait to send blufi custom data

I just checked out 4.1
C:\esp\esp-idf\examples\protocols\mqtt\tcp>git status
On branch release/v4.1
Your branch is up to date with 'origin/release/v4.1'.
It dies like this with the original example (and I'm using a WROVER board)

@xiewenxiang : When do you expect to look into this? Thank you!

Just to note: when using the code at SHA bd72a9ab2 I had other issues.
I decided I'd rather put efforts into trying release 4.2, and that appears to be all good.

Espressif folk, you might want to see if you can use that as a basis to fix 4.1 or just decree 4.2 ready sooner rather than later.

@jimwhitfield : I am currently using the master branch (so v4.3) and I still see this issue. I have a local patch in esp-idf to circumvent this.

We have fix it already with a lot of pain, making some modifications we
make it to work on v4.1 release again, but solution is complete weird and
makes no logic at all.
But I can say that blufi continue break.
Em ter, 22 de set de 2020 18:59, hb-kaertech<EMAIL_ADDRESS>escreveu:

@jimwhitfield https://github.com/jimwhitfield : I am currently using
the master branch (so v4.3) and I still see this issue. I have a local
patch in esp-idf to circumvent this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/espressif/esp-idf/issues/5792#issuecomment-697004500,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG7JIU3YXIBDHQP25UKVN33SHEM2JANCNFSM4QLBGFPQ
.

@hb-kaertech @jimwhitfield
ESP32 sends customer data to the APP is through notify.  If the MTU size doesn't exchange, it will uses the default 23 bytes, at this point, if sends a lot of data to the phone, such as a few thousand bytes, or more, the queues will soon fill up, So a mechanism has been added here to detect the number of remaining sendable packets.
I retest it, send 2k or 5k cuntomer data to APP, did not encounter such a problem, so I want to know, Do I need to do anything other than send customer data to APP?

@xiewenxiang : Did you follow my test plan in https://github.com/espressif/esp-idf/issues/5792#issuecomment-688252081 ?

@hb-kaertech  @filzek @jimwhitfield
If it is urgent, can you make such modification temporarily? We are doing further analysis.
components/bt/host/bluedroid/btc/profile/esp/blufi/blufi_prf.c
esp_ble_get_cur_sendable_packets_num(blufi_env.conn_id)  -> esp_ble_get_sendable_packets_num()

@xiewenxiang : I have reverted the offending commit and I'm quite fine with it right now.
However, as we are many people affected by this, we would like to make sure you can reproduce the issue for it will be fixed one day.

Same issue here.
ESP-IDF master branch (blufi example from there)

I wrote on the Android Git repository that the problem could be fixed by reverting the commit for the file blufi_prf.c but the problem persists using a Samsung Galaxy S10+ phone.
Any news about this, someone?

I have confirmed that it is working in esp-idf v3.3.1-133-g817efe8aa

Just to clarify, the problem is in the 4.0 branch. I have verified if it works in 4.1/4.2/master

I traced the problem to function gatt_get_tcb_by_idx in components/bt/host/bluedroid/stack/gatt/gatt_utils.c
When called  returns a NULL pointer for p_tcb when the input argument tcb_idx is 3.
For clariity tcb_idx is the same as blufi_env.conn_id in the code indicated by @filzek.
the if statement is not entered because there is no connection id 3 allocated in gatt_cb (in my case only connection id 0 is allocated) hence NULL is returned.
I have this problem using the BLUFI APP from Android and iOS and esp-idf 4.1/4.2/4.3.

Hey,
I also have the same issue with ESP-IDFv4.2.
The issue is 100% reproducible by sending custom data from Blufi IOS app version 1.3.1 and BluFi Library version  2.1.0
I have tried the fix  suggested by @xiewenxiang
 components/bt/host/bluedroid/btc/profile/esp/blufi/blufi_prf.c esp_ble_get_cur_sendable_packets_num(blufi_env.conn_id) -> esp_ble_get_sendable_packets_num()
Thanks it worked for me :)

This issue has been fixed and it is recommended to update the IDF.

Can you share the commit#?

This issue has been fixed and it is recommended to update the IDF.

confirmed, thanks!
on a general note when running menuconfig after updating idf check that BT_BLE_BLUFI_ENABLE=y for unknown reasons mine switched to 'n'.

@xiewenxiang upgrade to wich version ?
An issue is still observed on v4.2:
Refs: [release/v4.2], {origin/release/v4.2}, v4.2-47-g2532ddd

@xiewenxiang upgrade to wich version ?
An issue is still observed on v4.2:
Refs: [release/v4.2], {origin/release/v4.2}, v4.2-47-g2532ddd

RELEASE V4.2 still break, cant work blufi!

RELEASE V4.2 still break, cant work blufi!

RELEASE V4.0.2 already have this problem , only RELEASA V4.0 can work well

@filzek  @daichengqian
Please use the latest release/4.2 and release/v4.0 branch. If it still does not work, please provide the accurate commit.

@filzek @daichengqian
Please use the latest release/4.2 and release/v4.0 branch. If it still does not work, please provide the accurate commit.

yeah，i use latest  release/4.2 and my app version is 1.3.1 still have problem

@daichengqian Please upgrade the app version. I tested it with the app version of Android 1.5.3 and the latest release/v4.2, and did not encounter this problem again

@daichengqian Please upgrade the app version. I tested it with the app version of Android 1.5.3 and the latest release/v4.2, and did not encounter this problem again

I can t see IOS new version , did you upgrade?

At present, the latest IOS version on the official website is 1.2.0, I think you used 1.3.1 before, isn’t it an Android app?

iOS version is 1.3.1 in appstore upgraded three months ago.  you should check it.
发自我的iPhone

在 2021年3月22日，17:50，xulongzhuang @.***> 写道：

At present, the latest IOS version on the official website is 1.2.0, I think you used 1.3.1 before, isn’t it an Android app?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

sorry,I changed an iPhone, the version is indeed 1.3.1, but the latest release/v4.2 SDK used still does not show the problem described in the title, so, can you describe the phenomenon of your problem, whether it is Must appear?

Hello I have test v4.3 with Android. App v1.6.0 the demo blefi is not works (with error E (136708) BT_L2CAP: l2ble_update_att_acl_pkt_num not found p_tcb
W (136708) BT_BTC: btc_blufi_send_encap wait to send blufi custom data)we want to use this demo for esp32-c3 and esp32 do you have a temporary patch for fixed this pb? Thanks

@hb-kaertech @filzek @jimwhitfield
If it is urgent, can you make such modification temporarily? We are doing further analysis.
components/bt/host/bluedroid/btc/profile/esp/blufi/blufi_prf.c
esp_ble_get_cur_sendable_packets_num(blufi_env.conn_id) -> esp_ble_get_sendable_packets_num()

This fixed issue for me! But please add this fix to your next release!!!

@yangxinzhe90 Did you test it on release/v4.3 and use the esp32-c3 chip? Can you provide the commit_id, we will find the problem and solve it as soon as possible

iOS version is 1.3.1 in appstore upgraded three months ago. you should check it. 发自我的iPhone
…
在 2021年3月22日，17:50，xulongzhuang @.***> 写道：  At present, the latest IOS version on the official website is 1.2.0, I think you used 1.3.1 before, isn’t it an Android app? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

@daichengqian , Will you still encounter this problem in the current IDF release/4.2 version or other IDF versions? What is the iPhone iOS version number you are using?

Finally fixed in v4.4 and newer

Thanks for sharing updates.

Of course, my callback function and calling function are not in the same file, is it caused by this problem? But this problem is very difficult to reproduce, it just happened for the first time today


Test upgrade to v6

What changes were proposed in this pull request?
Upgrade actions/github-scripts to v6
Why are the changes needed?
Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/
Does this PR introduce any user-facing change?
No, dev only
How was this patch tested?

Notify works well:

and update status set to failed as expeceted:

And See test results set right.
https://github.com/Yikun/spark/pull/181/checks?check_run_id=9029035780


Optimize numeric lucene queries for precision <= 18

Benchmark results (https://github.com/crate/crate-benchmarks/pull/319):
Q: select count(*) from tcompact where x = (select x from tcompact limit 1)
C: 10
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |       59.875 ±  137.763 |     10.921 |     13.305 |     15.347 |    471.476 |
|   V2    |       22.116 ±   45.993 |      1.497 |      4.907 |      6.936 |    158.329 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               -  92.10%                           -  92.23%
There is a 99.00% probability that the observed difference is not random, and the best estimate of that difference is 92.10%
The test has statistical significance

Q: select count(*) from tcompact where x in (select x from tcompact limit 50)
C: 10
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |      160.998 ±  303.177 |     50.281 |     58.980 |     67.956 |   1083.673 |
|   V2    |       19.225 ±   43.597 |      3.061 |      4.101 |      5.698 |    149.498 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               - 157.33%                           - 174.00%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 157.33%
The test has statistical significance

Q: select count(*) from tcompact where x >= (select x from tcompact limit 1)
C: 10
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |       30.008 ±   44.464 |     10.361 |     13.481 |     16.808 |    164.047 |
|   V2    |        5.127 ±    4.118 |      1.638 |      3.442 |      4.892 |     15.436 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               - 141.63%                           - 118.65%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 141.63%
The test has statistical significance

Q: select count(*) from tlarge where x = (select x from tlarge limit 1)
C: 10
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |      183.532 ±  303.415 |     62.992 |     83.096 |     92.215 |   1119.045 |
|   V2    |        1.483 ±    1.487 |      0.670 |      0.974 |      1.170 |      6.002 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               - 196.79%                           - 195.36%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 196.79%
The test has statistical significance

Q: select count(*) from tlarge where x in (select x from tlarge limit 50)
C: 10
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |      443.935 ±  674.227 |    169.934 |    222.613 |    243.601 |   2534.794 |
|   V2    |        0.930 ±    0.422 |      0.584 |      0.772 |      0.928 |      2.312 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               - 199.16%                           - 198.62%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 199.16%
The test has statistical significance

Q: select count(*) from tlarge where x >= (select x from tlarge limit 1)
C: 10
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |      166.315 ±  254.805 |     64.558 |     82.372 |     89.977 |    981.660 |
|   V2    |        2.641 ±    0.726 |      1.086 |      2.662 |      3.170 |      3.887 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               - 193.75%                           - 187.48%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 193.75%
The test has statistical significance

@romseygeek pushed a fixup, I think this should be good to go now


swagger-ui middleware redirects without trailing slash

Whenever you access /docs, it redirects (or attempts to) to /docs/ and the initially displayed page never fully renders.  I know that the node-static middleware we're using has a configuration to disable the redirect but we also could benefit from figuring out why this is happening.  This originally was found while working on #339.

+1

I cannot reproduce this with Connect or Express but with Restify, I see the issue.  I think the redirect that the swagger-ui middleware is performing is not handled by Restify since Restify's res.redirect is different than Connect/Express.  That being said, I cannot fix this in swagger-tools.  But I was able to get this working using a simple Restify redirect (modified from #339):
'use strict';

var restify = require('restify')
var server = restify.createServer();
var swaggerTools = require('swagger-tools');

var serverPort = 3000;

// swaggerRouter configuration
var options = {
  controllers: './controllers',
  useStubs: process.env.NODE_ENV === 'development' ? true : false // Conditionally turn on stubs (mock mode)
};

// The Swagger document (require it, build it programmatically, fetch it from a URL, ...)
var swaggerDoc = require('./api/swagger.json');

// Initialize the Swagger middleware
swaggerTools.initializeMiddleware(swaggerDoc, function (middleware) {
  var middlewares = [
    middleware.swaggerMetadata(),
    middleware.swaggerValidator(),
    middleware.swaggerRouter(options),
    middleware.swaggerUi()
  ];

  // Required since swagger-ui uses CORS to fetch the Swagger document
  server.use(restify.CORS());
  // Use Restify versions of body/query parsing middlewares
  server.use(restify.queryParser());
  server.use(restify.jsonp());
  server.use(restify.bodyParser());

  // Fix issue with swagger-ui redirect not working with Restify (Not sure who is to blame here)
  server.get('/docs', function (req, res, next) {
    res.redirect('/docs/', next);
  });

  // Supported Swagger HTTP verbs (options and patch are not Restify server APIs)
  // https://github.com/OAI/OpenAPI-Specification/blob/master/versions/2.0.md#path-item-object
  ['get', 'put', 'post', 'delete', 'head'].forEach(function (verb) {
    if (verb === 'delete') {
      verb = 'del';
    }

    // Register the middlewares with Restify
    server[verb](/.*/, middleware.swaggerMetadata(), middleware.swaggerValidator(), middleware.swaggerRouter(options), middleware.swaggerUi());
  });

  // Start the server
  server.listen(serverPort, function () {
    console.log('Your server is listening on port %d (http://localhost:%d)', serverPort, serverPort);
  });
});

I think at the end of the day, the middleware in swagger-tools is written to be Connect compatible, which Restify is not.  So while I'm sure we can make swagger-tools work just fine with Restify, there may be some hoops you have to jump through to do so.  (The code above was able to make all functionality of the swagger-tools 2.0 example work using Restify.)

      server.get(`${BaseUrl}/docs`, (request, response, next) => {
        response.redirect(`${BaseUrl}/docs/`, next);
      });

      [`get`, `put`, `post`, `del`, `head`].forEach((verb) => {
        server[verb](/.*/, middleware.swaggerUi({
          swaggerUi: `${BaseUrl}/docs`,
          apiDocs: `${BaseUrl}/api-docs`
        }));
      });

Not working. getting Error 310 (net::ERR_TOO_MANY_REDIRECTS) - because /docs redirects to /docs/, than /docs/ redirects to /docs because of swaggerUi: `${BaseUrl}/docs

I'm sorry but this is an issue with Restify and the serve-static middleware not working with this.  swagger-tools cannot work around this.  I'm not sure if the options you're passing are causing the issue but since you're passing in the default values, you might as well just pass no options.  Does that help?  Also, it seems you're only using swagger-tools for its swagger-ui middleware.  If that's the case, save yourself the trouble and just use Restify's built-in static file serving and serve swagger-ui yourself.  I only suggest this because I'm sure using Restify to do this would be much simpler than trying to work around it.


fix(schedule): multi-cloud oss config error when download log

What type of PR is this?
type-bug

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Additional documentation e.g., usage docs, etc.:

If it's just to resolve OBCloud issues, there's no need to merge into the 4.3.2

I know


make separator configurable

it's still comma by default but you can pass { separator: ' ' } or { separator: ':' } as an option argument when creating tags-input

Thought: what about having it be an attribute on the input?
<input type="tags" data-separator=":" value="foo:bar:baz">

Yup. Much better. Let me rework it.

Some fixes:

implemented data-separator
fixed char detection - Safari does not actually support neither key nor char on keydown events - we need to parse keyIdentifier to properly detect separator char

How can one specify multible seperators? E.g. "," and enter?

@luckydonald by pasting

@luckydonald - enter and tab is always a separator, as for others you can just say specify a string with multiple charactes - see: https://code42day.github.io/tags-input

Ah now I see what was being asked haha.
Nice demo @pirxpilot

Detecting keys is tricky, not all documented properties are set on all events. For example charCode is not set in most browsers on keyDown.
see: https://dvcs.w3.org/hg/d4e/raw-file/tip/key-event-test.html

🙏

@0xc00010ff check @pirxpilot 's fork
https://github.com/pirxpilot/tags-input


CSS

It’s not a good practice to style an iframe with attributes you should make a separate style sheet.

ok


🐛 Props first tab renamed to 'Story' instead of using main component name

Storybook bug, we should find a workaround and/or update if/when Storybook has released a fix
Applies to these components:

[ ] Accordion
[ ] Breadcrumbs
[ ] Card
[ ] Popover
[ ] Tabs

is this still relevant @pomfrida ?

Looks like this has been fixed 🎉


Use flake schemas?

https://determinate.systems/posts/flake-schemas

https://github.com/NixOS/nix/pull/8892


Rework of the bridge with cmake

Now uses CMake as build tool
All arcdps events are sent in a different thread and arcdps' threads are not used
All Events are also sorted before they are send over the callback

Not yet implemented are callbacks for every single event directly.

Closing this as it's not been touched in 4 months.


in PushwooshAndroid.js file, GOOGLE_PROJECT_ID should be GOOGLE_PROJECT_NUMBER

In PushwooshAndroid.js file, the comment about the google project id and the key name, "projectid", are very misleading.  Since they refer to the project number in the google developer console and there is a project id in the developer console, it would be helpful if the comment and the key name referred to project number.  Thanks.

Thanks

Hi, we don't really track the issues on this repo and we ask that folks having problems with PhoneGap please use our forum:
https://forums.adobe.com/community/phonegap


ErrorException: Array to string conversion

Describe the bug
 Array to string conversion {"userId":42,"exception":"[object] (ErrorException(code: 0): Array to vendor/kreait/firebase-php/src/Firebase/Messaging/MessageData.php:23)"}

To Reproduce
$message = CloudMessage::fromArray([
                'topic' => 'appEvent'
                'data' => [
                    'event' => 'servicesAreUpdated',
                    'data'  => [
                        'id'   => $service->id,
                        'name' => [
                            'EN' => $service->name,
                        ],
                        'icon' => $service->icon
                    ]
                ],
            ]);

            \FirebaseMessaging::send($message);

Expected behavior
It used to works well, but after I updated the packages I got above problem.
Environment (please complete the following information):

OS:  MacOS 10.14, Ubuntu 18.04
PHP version: 7.1
Firebase SDK Version: 4.37.0

Are you sure this worked before? The last change concerning this was introduced with https://github.com/kreait/firebase-php/commit/c8dfdcd75a5a61cf419a62bed2ba55c7c3d2c0aa, but the code example you showed would have thrown an error before the change as well 🤔. From which version have you updated from?
The SDK currently doesn't support multi-dimensional data in the data field, because I understood the respective section in the reference as if the data field should be a one-dimensional list of key/value pairs as strings.
If you're sure this worked before or that Firebase supports multi-dimensional data payloads, I will try to confirm this and remove the constraint in the SDK.

Tried sending multi-dimensional data to API, but got an error. Working around by storing the data as JSON string value and decoding on client.

@go24dev I just tested the functionality with a multidimensional array - when I remove the check in the SDK and let the multidimensional array go to the Firebase API, the API returns an error like Invalid JSON payload received. Unknown name "key_xxx" at 'message.data[3].value': Cannot find field.
So I will keep it as it is. If you're convinced that the Firebase APIs support this, please provide me with a reproducible code example 🙏

Beginning with 4.38, an InvalidArgumentException will be thrown when the given data is not a one-dimensional array with string(able) values. This should make the issue a little more transparent.


Integrating with React-Spring is cumbersome

I was trying to integrate the Material-UI components with the React-Spring animation library.
You can find it here: https://www.react-spring.io/
When I try to use any imported Material-UI component to a spring component,  for example:
<animated.Typography>Example</animated.Typography>

I am no longer rendering the imported Material-UI element so React complains.
This forces me to add inline  elements that will create turmoil in my code.
Am I missing a secret better way to do this?
Above all, I'm very grateful and proud of such an amazing job for each and every one of you in creating my favorite React library!

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Summary 💡
If I try to do something like:
<animated.Typography style={props}>Example </animated.Typography>

will produce an ERROR because we are not rendering the imported element

Examples 🌈
Please here's a code sandbox: https://codesandbox.io/s/material-ui-vs-react-spring-2mi6k
Also, a simple demo code of what I am trying to accomplish from the same sandbox:
import React from "react";
import ReactDOM from "react-dom";
import Typography from "@material-ui/core/Typography";
import { useSpring, animated } from "react-spring";

function App() {
  const props = useSpring({
    opacity: 1,
    from: { opacity: 0 }
  });
  return (
    <div className="App">
      <Typography variant="h2">
        <animated.div style={props}>Example</animated.div>
      </Typography>

      {/* UNCOMMENT THE LINE BELOW AND SEE */}
   {/* <animated.Typography variant="h2">Example</animated.Typography> */}
    </div>
  );
}

const rootElement = document.getElementById("root");
ReactDOM.render(<App />, rootElement);

Motivation 🔦
I would love if I could do something like:
 <animated.Typography style={props}>Example </animated.Typography>

without having to wrap things in more spans,divs,etc.
What do you experts recommend?

Tip: const AnimatedTypography = animated(Typography)

Thanks @oliviertassinari


Mistify needs a Code of Conduct

 Issue by nshalman
Tue Mar 22 20:18:13 2016
Originally opened as https://github.com/mistifyio/mistify/issues/62

Issue Description
We need a Code of Conduct in place before the hordes of community contributors show up.
I'll be investigating a few options with notes in this issue and will follow up with a pull request of my recommendation.

 Comment by nshalman
Tue Mar 22 20:18:52 2016

http://contributor-covenant.org/ is one option that I like.


Hyperparameter settings

The pair ray selection in the paper is 2048 but the config in the code is 1024, why is that? Hope you can answer, thanks.

Hi, the code does not fully align with the paper, as it has been updated multiple times. Experimentally, we’ve found that using 1024 achieves similar performance, so we’ve chosen to use it for efficiency.

Thank you for your answer.
On Sat, Dec 21, 2024 at 7:29 AM Ruyi Zha @.***> wrote:

Hi, the code does not fully align with the paper, as it has been updated
multiple times. Experimentally, we’ve found that using 1024 achieves
similar performance, so we’ve chosen to use it for efficiency.
—
Reply to this email directly, view it on GitHub
https://github.com/Ruyi-Zha/naf_cbct/issues/28#issuecomment-2557878986,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BMPEQKERDYV35SG5KSD4GUD2GSR5NAVCNFSM6AAAAABT7CRGAOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJXHA3TQOJYGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Collect allocated storage for container groups

Depends on https://github.com/ManageIQ/manageiq/pull/15023
Take sum of all persistent volume storage capacity as the derived_vm_allocated_disk_storage field in metric rollups. This will be filled in hourly rollups.
@yaacov @zakiva @cben Please review
cc @simon3z

@miq-bot add_label wip

conceptually looks good to me (pending tests and the PV PR)

cc @moolitayer @nimrodshn

Closing in favor of #15973


$ionicLoading without any text, just animated icon

Is there a way to make $ionicLoading display with just an animated icon (e.g. - spinning wheel).  I can sorta make it work by adding a html content to show the icon.  But, it still displays the box around it.

now you can use ion-spinner directive:
$ionicLoading.show({
    template: '<ion-spinner icon="dots"></ion-spinner>',
    hideOnStageChange: true
});

also, you can change the style of the spinner in your css:
.spinner svg {
  width: 64px;
  height: 64px;
  stroke: #FFF;
  fill: #FFF;
}

ion-spinners simply don't display in my app, checked out the DOM, and they aren't there as if they don't get rendering by angular or something, halp! My code is essentially as above:
$ionicLoading.show({
template: '',
hideOnStageChange: true
});

how can i customize icon spinner. ?
        template: '<i class="icon-dots"></i>',
doesnst work .. -- i have nt seen that template in .loading  ,,

how can i disable swipe on ionicLoading is showing???


feat(richtext-lexical): add indent and align converter utilities

Description
Add utilities for indent and align converter.

Added getElementNodeDefaultStyle.ts in features\shared folder because it seems to be used only inside the features folder. If I'm misinterpreting this, you can change the folder location.
Fixed EditorTheme.scss because --lexical-indent-base-value compiles to base(2) now

[x] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] I have added tests that prove my fix is effective or that my feature works
[x] Existing test suite passes locally with my changes
[x] I have made corresponding changes to the documentation

Fixes #5146 , Related #5814

Okay, I'll add a test for this within 2 days and update this PR.

Add tests with some fixes.

Delete indent for listitems(I forgot align is needed for listitem. I'll add it soon).
Add QuoteNode/HeadingNode wrapper because of no logic for HTML to Lexical Data.

Ok I've done my research and I think we need to reconsider whether this should be addressed in the Lexical repository.
The Lexical team has decided not to support highly opinionated styles such as text background-color, but they seem to be on the same page with us regarding text-align and indent. If it hasn't been fully implemented by now, it's probably due to lack of time.
Here I have documented some of the milestones in this issue.

Date
PR / Issue
Description

Aug 2022
PR 2785 (closed)
Export text-align for Paragraph (closed)

May 2023
PR 4445
Export text-align for Heading y Quote

June 2023
PR 4625
Import text-align for Heading y Blockquote.

Mar 2023
Issue 4243
text-align for list-items (headings done)

May 2024
Issue 6082
Indent for ParagraphNode

Note that the two issues (4243 and 6082) are still open.
If the solution was a small patch it would be easier for me to consider addressing it in Payload, but I think we can agree that overwriting the original nodes is a big deal.
Sorry for the time it took me to come to this conclusion, but on the bright side I think the time you've spent on this can be partially transferred to Lexical if you decide to address the problem there.

It seems wrong to implement Lexical's issue detection first, when it should have been done first. sorry.
If we don't overwrite the node, I think we should implement to block AlignFeature/IndentFeature in quote and list-item until lexical teams are implemented about quote/list-item in lexical.
I agree it's a big deal, but I think the issue that AlignFeature/IndentFeature is not working from the output HTML will come up constantly, and I think it should be documented or some other workaround.
Once payload teams decided on a workaround, I'll delete the wrapper node and change the code in that direction.

Indeed it is an issue with a lot of demand so it is very high on my priority list.
The correct plan of action would be to do the PR in Lexical, making sure that all nodes properly convert indents and text-align.
 Otherwise, I think I can do it on Monday.

I deleted the Wrapper Node.
I'll be working on inserting Indent logic into the importDOM logic (and maybe exportDOM logic) of QuoteNode, HeadingNode, and ListItemNode in Lexical starting tomorrow local time, but I'm not sure if I create PR and merged quickly.

I've opened a PR fixing the indent issue here: https://github.com/facebook/lexical/pull/6693
I'll do the same for text-align soon.
If you're interested, contributions are still welcome to make HTML serialization take into account custom indentation values in editorConfig.theme.indent

I have a question:
I think this PR is the PR that implements exportDOM and importDOM, but currently @payloadcms/richtext-lexical does not implement lexicalHTML via exportDOM, so will you change the implementation of lexicalHTML?
Additonal: I'm sorry I didn't get back to you sooner, I'm not making PRs.

You are correct that Payload is not using Lexical's serializer. I've spoken to @AlessioGr about this, and we'll probably change it.
If not, we'll revisit an approach like the one in this PR again.


chore: add memo copy button

Nice feature! Since it's already approved, is there anything against merging this PR right now?

@boojack what is with this?


feat(sbb-autocomplete): favourites PoC

Changes
Implemented a POC for the autocomplete with favourites component
Known a11y problems
The ARIA pattern we are using is not broadly used and presents some problems:

VoiceOver + Chrome => Reads the content of the option only if the previous one has finished speaking (it is needed to navigate slowly)
VoiceOver on Safari and iOS is not supported
VoiceOver + Chrome => Groups labels are not read
JAWS + Firefox => Groups labels are not read
Android + TalkBack => Groups labels are not read

Continued in #2512


feat: add svelte/no-svelte-internal rule

svelte/internal is deprecated in Svelte 5 and will be removed in Svelte 6 according to our current plan.
The Svelte compiler only compiles .svelte files and .svelte.[jt]s files, so the compiler can not check normal .[jt]s files. Therefore, the ESLint plugin checks this.

I forgot to write a document.😇

Finished ~~ @ota-meshi


refresh bugs

In normal use, the refresh method has some problems.
v1.14.0-beta3
test.zip

Yes. Please make sure to search for an existing report before creating a new issue, and rather than providing a zip file, use code snippets or online code editors to provide a use case. Closing as dupe of #2724.


Some data-sets seem to have missing header information

When calling parser.parse_model_metadata(), some of the data-sets could not be parsed on my machine, with the error message that the header information was missing. The data-sets that were affected are:
70000.csv
140000.csv
330000.csv
510000.csv
590000.csv
670000.csv
820000.csv
After removing these data-sets from the data folder, the parsing worked fine.

I cannot reproduce the issue with the dataset that I have locally (and see headers in 70000.csv) @sap-dianasola can you double-check? In any case, we could add code that adds headers if they are missing to mitigate this issue.

Unfortunately, I cannot reproduce the issue either. @CC90, is this still an issue? If you were able to mitigate the issue with @TimKam's suggestion, it would be great if you could start a pull request.


PoC: separate tarballs from bundle and test them separately

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

Needs approval from an approver in each of these files:

OWNERS [alenkacz]

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

@alenkacz: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

test
78dd1663dcb8c9fcfc4302fd86979c710e8b0cf6
link
/test test

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Is possible use Playwright instead of Puppeteer?

Is possible to use Playwrigth instead of Puppeteer?

I too may be interested in this possibility!
I was considering  moving to Playwright but Cluster is so good I don't think I can make the move without it.

Yes, this looks interesting. It probably won't be too much work to implement it.


MAISTRA-718 | Make Jaeger Elasticsearch spec fully configurable

Resolves: https://issues.jboss.org/browse/MAISTRA-718
This makes Jaeger's Elasticsearch spec fully configurable.
@objectiser could you please review?
Signed-off-by: Pavol Loffay<EMAIL_ADDRESS>

I have tested this with various CRs. For instance:
    tracing:
      enabled: true
      jaeger:
        # simple, all-in-one strategy
        template: production-elasticsearch
        elasticsearch:
          nodeCount: 3
          redundancyPolicy: FullRedundancy
          storage:
            storageClassName: testing
            size: 5Gi
          nodeSelector:
            beta.kubernetes.io/os: "linux"
        # production strategy, utilizing elasticsearch
        #template: production-elasticsearch
        # if required. only one instance may use agentStrategy=DaemonSet
        #agentStrategy: DaemonSet

LGTM - although I would prefer to actually just copy the whole .Values.jaeger.elasticsearch node into the template, rather than deal with the individual child nodes.

LGTM - although I would prefer to actually just copy the whole .Values.jaeger.elasticsearch node into the template, rather than deal with the individual child nodes.

Then we can't pass the ES spec directly because we are overriding the defaults. For instance by default we use 3 nodes and specific resources. Once https://github.com/jaegertracing/jaeger-operator/issues/560 is resolved and assuming the defaults are the same we could pass the ES spec directly

@pavolloffay Or the maistra CR could define the defaults?

The CR defines what will be created. The defaults here defines what will be created if the value is missing in the CR. So if you just create an empty spec it will still create a 3 node ES cluster.
If you agree with this, I am fine to pass there directly the whole ES spec.

@pavolloffay Fine to wait for feedback from the maistra team - but that may be better - so if elasticsearch node not present, then may need to set nodeCount but also the default resources.

@rcernich @knrc could you please review?

@rcernich can this be merged?


question - Create a numeric column in postgresql

Hi, just downloaded and run for 1st time. I have troubles to create a column with numeric type: the "scale" property is not editable.
Also, is there a way to select a datatype besides scrolling the select box with the mouse wheel? I'd like to write a couple of characters and then trigger the  "intellisense" and choose from 2-3 options rather than scroll in hundreds of different datatypes
many thanks

Scale and precision will be editable in the next version, thanks for reminder.
You can just enter type name and press Enter.

I just ran into the same issue, except that I was trying to change the datatype of an existing column, from varchar. When I toggle numeric, precision and scale are not editable (length is). Is it possible you fixed this only for column creation? I'm running 3.8.5 on Ubuntu.

This will be fixed in 4.0, thanks for noticing.


Can't add new project

Type: Bug
I'm trying to add a new project in my solution folder, a class library project, but the command pallette doesn't even appear.
VS Code version: Code 1.86.2 (903b1e9d8990623e3d7da1df3d33db3e42d80eda, 2024-02-13T19:42:12.210Z)
OS version: Darwin arm64 23.3.0
Modes:

System Info

Item
Value

CPUs
Apple M1 (8 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
10, 10, 11

Memory (System)
16.00GB (0.40GB free)

Process Argv
--crash-reporter-id 5dfc4a38-4832-4d4d-a2e7-698a8b71aedd

Screen Reader
no

VM
0%

Extensions (27)

Extension
Author (truncated)
Version

vscode-sqlite
ale
0.14.1

auto-rename-tag
for
0.1.10

codespaces
Git
1.16.10

copilot
Git
1.165.0

copilot-chat
Git
0.12.2

remotehub
Git
0.62.0

vscode-docker
ms-
1.29.0

csdevkit
ms-
1.3.10

csharp
ms-
2.15.30

vscode-dotnet-pack
ms-
1.0.13

vscode-dotnet-runtime
ms-
2.0.1

vscodeintellicode-csharp
ms-
0.1.26

vscode-kubernetes-tools
ms-
1.3.15

remote-containers
ms-
0.338.1

remote-ssh
ms-
0.108.0

remote-ssh-edit
ms-
0.86.0

remote-wsl
ms-
0.86.0

vscode-remote-extensionpack
ms-
0.25.0

mono-debug
ms-
0.16.3

remote-explorer
ms-
0.4.1

remote-repositories
ms-
0.40.0

remote-server
ms-
1.5.0

resharper9-keybindings
ms-
1.1.0

vscode-nuget-gallery
pat
0.0.24

vscode-yaml
red
1.14.0

LiveServer
rit
5.7.9

vscode-icons
vsc
12.7.0

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscorecescf:30445987
vscod805cf:30301675
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:30899288
c4g48928:30535728
azure-dev_surveyone:30548225
962ge761:30959799
pythongtdpath:30769146
welcomedialogc:30910334
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
h48ei257:30968133
pythontbext0:30879054
accentitlementst:30887150
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
8082a590:30971561
edj9j583:30969295
fegfb526:30961430
7j2b6412:30969109
bg6jg535:30958357
ccp2r3:30958157

Just a clarification, the issue of creating for example a Class Library is within the Solution Explorer window in VS Code for Mac.
The command palette appears, I choose a project like Class Library, but then nothing happens.

/extC#

It looks like this is caused by the C# extension. Please file the issue to the C# extension repository. Make sure to check their issue reporting template and provide them relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!


Use a constant for totals rollup instead of NULL

Coverage increased (+0.02%) to 90.933% when pulling 6e0c4f320aa6ac0ac332285c1993404f1ebb49d7 on gl3nn:not-null-for-totals into ca0ad5b8f29d3201d653d9264044db61a1e482c2 on kayak:master.


Invalid check for classification size

Currently, if the number of classes expected by QuPath is higher than the classes provided by the model, QuPath will take the missing classes in the rays channels as the number of rays is computed from the number of expected classes. The warning for invalid classification size is then never triggered.

I agree that this information might not be available from the model and then need to be provided from the builder by the user (as it is done right now), but in that case, the current check just below does not make sense as it will never fail:

Sorry, it has been a long time since I wrote or used that code so I still don't understand what exactly is wrong, and at this point on a Friday evening I'm not at my sharpest....
The StarDist code has to deal with a lot of scenarios:

3 outputs, 2 outputs or channels concatenated
classifications required or classifications not required
classifications specified in the builder or classifications not specified in the builder
a different numbers of rays from the default (since it can be specified when training StarDist)

where pretty much any combination is possible.
For concatenated output, there's no way for QuPath to distinguish between rays and channels if the information isn't given to the builder. So in this case it certainly could do the wrong thing, but it doesn't have enough information to do the right thing - because it needs to handle the case where there aren't any classifications anyway.
Can you describe specifically what combination of events isn't handled properly by the current code? Or can you say what the code should be?
The current code is quite involved, and I don't know what problem I should be looking for.

Note that the code you quote in the first post isn't always called:
https://github.com/qupath/qupath-extension-stardist/blob/main/src/main/java/qupath/ext/stardist/StarDist2D.java#L1102
QuPath supports StarDist models that have 3 outputs. In that case, the dimensions of the outputs are used to estimate the numbers of classifications. And in that case, the warning can be logged... I think.
But I'm getting this from reading the code / remembering, I don't have any classification models right now to test with.
It may well be the case that the warning can't be logged if the model concatenates outputs and the user doesn't specified the classifications... but in that case, I'd say QuPath has tried it's best and it's the user's fault :)
At least I think that's how it's meant to work, and why I yet see any bug.
As far as I'm aware, the original StarDist notebooks concatenate channels in the output using CSBDeep because that's needed for compatibility with the Fiji plugin - but QuPath can handle output without this channel concatenation. You can save a regular TensorFlow saved model without the CSBDeep export functions and use that in QuPath instead. This is better because it avoids most channel guesswork, and avoid needing to handle upsampled probability maps.

Indeed I did not took into account the output size, I was not expecting StarDist model to have this kind of shape as I'm always exporting them using the default export with CSBDeep, the check make sense for this situation so no bug :).
I was working with a binary classification and did not understood why I got only one classes as result all the times (it was related to the classes with id 0 being deleted, see keepClassifiedBackground), also StarDist starts the classes ids from 1 when training instead of 0 so there was some misunderstandings on the results I obtained since I used the same id system. To try to understand the problem, I increased the number of expected classes and started having a lot of results in multiple classes that I was not classifying in the model without any warning. I totally understand this information is not available and you cannot do anything on your side so that's the user's (me) fault.
Sorry for bothering you, have a nice weekend!


haxlib fails to resolve with ipv6?

after I setup ipv6 on my computer, I now get Failed with error: std@host_resolve errors trying to interact with haxelib, and, sure enough, there is no AAAA dns record set up, so ipv6 fails to resolve...
but I've even setup ipv4 to be preferred on my system...other apps prefer picking ipv4 as a result...it shouldn't even be picking ipv6 in the first place...
not sure whether it should be fixed on haxelib side, or haxe's stdlib side, but I think the sensible options are either
A) if either ipv4 or ipv6 fails to resolve, it should try to resolve the other one before erroring out
B) just explicitly make haxelib only use ipv4 and pretend that nothing is wrong
and yes, if I manually update /etc/hosts to point to ipv4 for lib.haxe.org then it works a little bit better, but still broke...
before host file workaround:
[jan6@Teartix ~]$ haxelib update
Checking hashlink
Failed with error: std@host_resolve
Checking hxcpp
Failed with error: std@host_resolve
All libraries are up-to-date
[jan6@Teartix ~]$

after host file workaround:
[jan6@Teartix ~]$ haxelib update
Checking hashlink
Checking hxcpp
Update hxcpp to 4.3.2 [y/n/a] ? y
Downloading hxcpp-4,3,2.zip...
Download complete: 0.24KB in 0.3s (0.6KB/s)
Download complete: 0KB in 0s (0KB/s)
Failed to download https://lib.haxe.org:443/files/3.0/hxcpp-4,3,2.zip. (1/3)
std@host_resolve
Download complete: 0.24KB in 0.2s (0.9KB/s)
Download complete: 0KB in 0s (0KB/s)
Failed to download https://lib.haxe.org:443/files/3.0/hxcpp-4,3,2.zip. (2/3)
std@host_resolve
Download complete: 0.24KB in 0.3s (0.6KB/s)
Download complete: 0KB in 0s (0KB/s)
Failed to download https://lib.haxe.org:443/files/3.0/hxcpp-4,3,2.zip. (3/3)
std@host_resolve
Error: [file_open,/usr/share/haxe/lib/hxcpp-4,3,2.zip]
[jan6@Teartix ~]$

for some reason, I need to ALSO override haxelib-files.haxe.org to ipv4 in my hosts file, DESPITE it having an ipv6 address that works just fine via curl???
if I do that, then it works, but, y'know, it's very fragile, having manual overrides
(using haxelib 4.1.0 and haxe 4.3.6 of course)

weirdly enough haxelib source has a giveErrorString() which is SUPPOSED to handle the error and give a more human-friendly message, but that isn't triggered?
I'm on an Arch-based distro


Here's an updated example for using exoplanet + pymc3

Yo! I played around with your exoplanet + PyMC3 example because I think that it's probably the best way to go for now. Here's what I got...

very nice! cheers


Absolute path 지원

https://github.com/EveryAnalytics/web-analytics-handbook/pull/32/files#diff-736fb2f00bdf28de6a5e870b49dfad7e19429dc0f7011711833e0af5f7ff72b4R6

relative path만 지원해서 개발 생산성 저하

duplicate https://github.com/EveryAnalytics/web-analytics-handbook/issues/28


Alerts are not sent when exactly the same message was sent before

Hi @honoki,
Spamming with issues :)
I'm not sure if this was your intention but when two alerts are sent via bbrf alert with the same message e.g.
bbrf alert test
bbrf alert test

only first message will be stored in CouchDB. Second attempt results in CouchDB returning error:
b'[{"id":"test","error":"conflict","reason":"Document update conflict."}]\n'
That's because document _id is set to message value that is not always unique.
Will it break something when alert _id will be auto-generated? (UUIDv4, timestamp etc.)

It was a deliberate design choice to use the alert strings as the unique identifier, so as to avoid adding duplicate findings when a tool is triggering the same findings more than once. However, I'm not convinced that it was a good choice :)
I'll leave this issue open to invite some thoughts about the matter; maybe it is worth changing this to work with CouchDB's auto-generated ids instead as you suggest.


Add documenation for queryOptions groupByMode and responseFormat

Adding documentation for query options groupByMode and responseFormat, since we have exposed them in the query console.

https://github.com/apache/incubator-pinot/issues/4450

Codecov Report

Merging #4708 into master will increase coverage by 0.16%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4708      +/-   ##
============================================
+ Coverage     64.72%   64.89%   +0.16%
  Complexity       16       16
============================================
  Files          1061     1061
  Lines         56629    56629
  Branches       8414     8414
============================================
+ Hits          36655    36751      +96
+ Misses        17246    17151      -95
+ Partials       2728     2727       -1

Impacted Files
Coverage Δ
Complexity Δ

...a/manager/realtime/RealtimeSegmentDataManager.java
50% <0%> (-25%)
0% <0%> (ø)

...pinot/core/operator/docidsets/OrBlockDocIdSet.java
84.9% <0%> (-9.44%)
0% <0%> (ø)

...n/java/org/apache/pinot/core/transport/Server.java
59.25% <0%> (-7.41%)
0% <0%> (ø)

...e/impl/dictionary/LongOnHeapMutableDictionary.java
63.41% <0%> (-6.1%)
0% <0%> (ø)

...impl/dictionary/FloatOffHeapMutableDictionary.java
60.21% <0%> (-5.38%)
0% <0%> (ø)

...thandler/SingleConnectionBrokerRequestHandler.java
84.09% <0%> (-4.55%)
0% <0%> (ø)

...nsport/netty/PooledNettyClientResourceManager.java
85.41% <0%> (-4.17%)
0% <0%> (ø)

...lix/core/realtime/PinotRealtimeSegmentManager.java
77.94% <0%> (-2.06%)
0% <0%> (ø)

...impl/dictionary/StringOnHeapMutableDictionary.java
86.2% <0%> (-1.73%)
0% <0%> (ø)

...va/org/apache/pinot/common/data/TimeFieldSpec.java
92.59% <0%> (-1.24%)
0% <0%> (ø)

... and 23 more

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 7540702...f1cd0a9. Read the comment docs.


Events after the payement process for pro plan

after successful payement for the pro plan its redirecting to /settings and there is no plan update
this is what im getting
INFO: Clerk: The request to /webhook is being protected (401) because there is no signed-in user, and the path is included in apiRoutes. To prevent this behavior, choose one of:

To prevent Clerk authentication from protecting (401) the api route, remove the rule matching "/webhook" from the apiRoutes array passed to authMiddleware
To make the route accessible to both signed in and signed out users, add "/webhook" to the publicRoutes array passed to authMiddleware
To prevent Clerk authentication from running at all, pass ignoredRoutes: ["/((?!api|trpc))(_next.*|.+\.[\w]+$)", "/webhook"] to authMiddleware
Pass a custom afterAuth to authMiddleware, and replace Clerk's default behavior of redirecting unless a route is included in publicRoutes

For additional information about middleware, please visit https://clerk.com/docs/nextjs/middleware
(This log only appears in development mode, or if debug: true is passed to authMiddleware)
And my gpt says
It looks like you're receiving multiple log messages indicating that the request to "/webhook" is being protected (401) because there is no signed-in user, and the path is included in apiRoutes. This typically indicates an issue with authentication middleware configuration, specifically with Clerk authentication.
Here are some possible solutions based on the provided log messages:
Remove "/webhook" from apiRoutes: If you don't want the "/webhook" route to be protected by Clerk authentication, you can remove it from the apiRoutes array passed to the authMiddleware.
Add "/webhook" to publicRoutes: If you want the "/webhook" route to be accessible to both signed-in and signed-out users, you can add it to the publicRoutes array passed to the authMiddleware.
Ignore "/webhook" route for authentication: If you want to prevent Clerk authentication from running at all for the "/webhook" route, you can pass ignoredRoutes: ["/webhook"] to the authMiddleware configuration.
Customize afterAuth behavior: You can pass a custom afterAuth function to the authMiddleware and replace Clerk's default behavior of redirecting unless a route is included in publicRoutes.
Choose the solution that best fits your authentication requirements and modify your middleware configuration accordingly. Make sure to test your changes to ensure that the authentication behavior is as expected.

@Vaibhav-nk-git, Please, take a look at this video. This is how it was working for me the last time I checked. No new changes have been made.
Anyway, I'll take a look whenever I get the chance. Big thanks for notifying me. Would you like to checkout my other latest projects?

P.S. The api/webhook is already being considered as the public route. So that might not be the root of this issue.
https://github.com/aftabrehan/jarvis-ai/blob/f591e39468013d6ce55c0a8adea95c6a20191e08/middleware.ts#L4


Specify graphql as a dependency for @graphql-codegen/typescript

Describe the bug

The typescript plugin uses isIntrospectionType from graphql as part of its main plugin function. This imported function from graphql is therefore required for usage of the package (not just for dev, as the graphl dependency is currently specified).
The problem is that isIntrospectionType was introduced in<EMAIL_ADDRESS>making the typescript plugin incompatible with older versions of graphql. The package.json should be updated to reflect this, otherwise this can cause issues when trying to use older versions of graphql as a consumer of @graphql-codegen.
To Reproduce
Steps to reproduce the behavior:
In your project that is consuming @graphql-codegen, use the typescript plugin along with a version of graphql older than v0.12.0. Notice errors about the missing isIntrospectionType function.

My codegen.yml config file:

overwrite: true
schema: ${SCHEMA_PATH}
generates:
    ${CODEGEN_GQL_PATH}/codegen.ts:
        plugins:
            - 'typescript'
            - 'typescript-operations'

Expected behavior

I expect to be able to use older versions of graphql (that fit in with the peer dependency requirements specified in @graphql-codegen, which is currently<EMAIL_ADDRESS>and up). If the plugin has a dependency on newer versions of graphql, then this should be reflected as a dependency or peer dependency of that package.

Done in 1.4.0 🎉

Thanks!


[Mobile. Challenge]  Links have black font color

Environment: Windows 10, Google Chrome 92.0.4515.107 (Розробка) (64-розрядна версія)
Reproducible: always
Build found: the last commit
Steps to reproduce

Go to burger menu >'Челендж' >'Мовомаратон'
2.Go to 'Мовомаратон' page: 'Крок2', 'Крок3' 'Крок4'
3.Review the Link font color

Actual result
Links have black font color

Expected result
Links have blue font color

User story and test case links
"User story #692

The bug is not reproducible

The bug is reproducible on "Крок 6" in mobile version.

The bug is reproducible on "Крок 11", "Крок 12" in mobile version.

Everything works correctly.
Windows 11 Enterprise, Google Chrome 108.0.5359.125 (64), notebook.

Android 11 RKQ1. Google Chrome, MI 9SE.


SSR support for SIX Webcomponents

What problem does this feature solve?
In order to support SSR we need to configure the library so the components can be properly server-side rendered.
React for example recommends to use NextJs, Remix or other meta frameworks which support SSR for new projects instead of plain React: https://react.dev/learn/start-a-new-react-project
What does the proposed API look like?
According to the documentation Angular Universal uses the same functionality for Ionic
Hydrate App - Stencil (stenciljs.com)
Sources:
Combining Server Side Rendering and Static Site Generation - Stencil (stenciljs.com)
Gomah/nuxt-stencil: Easy Stencil.js component library integration with Nuxt.js. (github.com)
javascript - How to do SSR with stencil.js in a project with Nuxt.js (SSR as well) - Stack Overflow

Closed for now as there is no one ussing SSR


Can Contacts and Faults Type combobox also search on the description?

The Type combobox on the Contacts And Faults pane displays the symbol and key and shows the description in the tooltip popup. It allows the user to begin typing a key value and autofills to a valid key. Can we make it also search the description?

I tried a few permutations of this, and none are acceptable.
The TextSearch. TextPath property of the combobox can only be set to one field. Setting this to anything other than the field that is displayed in each item causes weird behavior.
I could use the description as the text in each item, but it is too long, in most cases, to look good. It's also too long when it autofills for the user as they type. It overflows by pushing text to the right. This is disconcerting.
So, in general, I don't think this is a viable feature.


Can not bind a style class using a Multibinding

Is your feature request related to a problem? Please describe.
I'm trying to bind a style class using a Multibinding:
 <Button>
     <Button.Classes.Checked>
         <MultiBinding Converter="{StaticResource EqualsConverter}">
             <Binding Path="$parent[ItemsControl].((vm:ChannelListViewModel)DataContext).SelectedChannel" />
             <Binding Path="." />
         </MultiBinding>
     </Button.Classes.Checked>
</Button>

(supposed to be used here instead of a ToggleButton)
This causes the error Unable to resolve suitable regular or attached property Classes.Checked on type Avalonia.Controls:Avalonia.Controls.Primitives.Button Line 36, position 22
Also asked here: https://stackoverflow.com/questions/77235308/binding-classes-in-avalonia-using-a-multibinding
Describe the solution you'd like
Make it possible to bind a style class using a multibinding
Describe alternatives you've considered
I can use Classes.Checked={Binding .}, but that does not support Multibinding.
Someone also mentioned Markup Extensions, but I have no clue how to do that.
Additional context

Complete Sample
Window.xaml
<Window xmlns="https://github.com/avaloniaui"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        xmlns:vm="clr-namespace:Turbulence.Desktop;assembly=Turbulence.Desktop"
        mc:Ignorable="d" d:DesignWidth="800" d:DesignHeight="450"
        x:Class="Turbulence.Desktop.Window"
        x:DataType="vm:WindowViewModel"
        Title="Window">
    <Window.DataContext>
        <vm:WindowViewModel />
    </Window.DataContext>
    <Button>
        <Button.Classes.Checked>
            <MultiBinding Converter="{x:Static BoolConverters.And}">
                <Binding Path="Bool1" />
                <Binding Path="Bool2" />
            </MultiBinding>
        </Button.Classes.Checked>
    </Button>
</Window>

WindowViewModel.cs
namespace Turbulence.Desktop;

public class WindowViewModel
{
    public bool Bool1 { get; set; } = true;
    public bool Bool2 { get; set; } = false;
}

can you test:
 <Button>
     <Classes.Checked>
         <MultiBinding Converter="{StaticResource EqualsConverter}">
             <Binding Path="$parent[ItemsControl].((vm:ChannelListViewModel)DataContext).SelectedChannel" />
             <Binding Path="." />
         </MultiBinding>
     </Classes.Checked>
</Button>

can you test:
 <Button>
     <Classes.Checked>
         <MultiBinding Converter="{StaticResource EqualsConverter}">
             <Binding Path="$parent[ItemsControl].((vm:ChannelListViewModel)DataContext).SelectedChannel" />
             <Binding Path="." />
         </MultiBinding>
     </Classes.Checked>
</Button>

Yes this worked. I will close this issue and we will pretend this never happened.


How to set environment variables in the container

Using the maven plugin.  How can I set environment variables within the container?
Also, when is 0.9.10 likely to be released as I need the label feature?

One workaround for now would be to build your own custom base image that preloads the environment variables you want and make Jib point to that base image. (Jib will propagate the env vars in the base image when building an image (https://github.com/GoogleContainerTools/jib/issues/595#issuecomment-410264838).)

Ok thanks, any idea when that feature might be added and when 0.9.10 is likely to be released?

@matt-shaw 0.9.10 should be released within the next day or two if all goes well.

@matt-shaw 0.9.10 has been released with labels support.
We were also wondering why you'd like to set environment variables through the plugin at build time rather than during deployment?

We change the behaviour of the spring boot application using environment variables.  Some of these variables are set when the app is run via docker and others when it’s run as a stand-alone java app.  To be honest it’s also in the docker spec so would be nice to be able to set them.  We use the alpine-jre base image.

That's an interesting use case and we're currently discussing adding this in because it's in the docker spec.
Just out of curiousity:

You don't really change the environment between container runs?
Is there a reason you wouldn't set the environment when you run the application?

docker run -e WHATEVER=whatever
on the kubernetes yaml (if you're using kubernetes).

No, it’s not really runtime dependent, more build time, I.e. it’s set because it’s run in docker and doesn’t change at runtime.

@matt-shaw We have released version 0.9.11 with the <container><environment> configuration for setting environment variables!

Hey @coollog. Thanks a lot for making that nice tool! But have u tested the <container><environment> configuration cause I have the following config:
...
<container>
  <environment>
      <var-1>123</var-1>
      <var-2>456</var-2>
  </environment>
</container>
...

And after executing mvn jib:exportDockerContext in the resulting Dockerfile I don't see ENV declarations.

Update, the docker context problem has been fixed and will be available in v0.9.13 (already released for Gradle, working on Maven and should be released either later today or tomorrow).

@dzmitryh jib-maven-plugin version 0.9.13 is released with the fix for adding the environment definitions to the exported Dockerifle!


dev: Autogenerated files using get_codes

As you know some of the existing files are like: ast.rs, kind.rs, key_fields.rs are autogenerated right now. In my opinion, this method lacks transparency and requires consistent monitoring of cairo_spec.rs. Additionally, it introduces a dependency on nightly features, which may not be ideal.
I would like to propose an approach that salsa uses with salsa::query_group procedural macros.
Where instead we would declare structs directly in ast.rs the following way:
#[generator::generate_node(node(modifiers, ModifierList), node(arg_clause, ArgClause))]
pub struct Arg;

or
#[generator::generate_node(Arg, node(modifiers, ModifierList), node(arg_clause, ArgClause))]

Thus user will have access to new nodes right away without caring about updating cairo_codes and generating new files. As for kind.rs, key_fields.rs they can be generated using the same approach as salsa::database marco does does eg generator::kind(Arg, ArgClauseUnnamed, ...) or left for users(devs) to fill since without that compiler will throw error that Arg implementation uses unknown SyntaxKind

The current method is easy to use and maintain, from my experience. I don't see a need to do something else.
This approaches are also taken in other projects (such as rust-analyzer).