Add fuzzy string matching to quick open search

This is a continuation of https://github.com/godotengine/godot/pull/82200 to implement https://github.com/godotengine/godot-proposals/issues/7771. Rebasing the first PR against https://github.com/godotengine/godot/pull/56772 was pretty awkward so I ended up flattening the earlier changes with @samsface as a co-author. A bit more info and pre-flattened commits are available in this pr against sam's fork https://github.com/samsface/godot/pull/1.
Functionally this adds optional fuzzy searching and search highlighting in the results:

The highlighting on the grid items feels a bit hacky due to the centered text; I'd appreciate any tips to do it more cleanly. This is also my first time working on a nontrivial c++ codebase so I'd appreciate any general tips to be more idiomatic or if I'm using anything incorrectly. Also a few semi-related changes:

Fixes a bug where, if adaptive isn't used, on first launch neither result container will be visible. Now it uses project metadata to track the last mode, defaulting to list.
Grid items are slightly wider, showing 5 per row with the default window size, in order to show more of the filename.
In AcceptDialog, the vertical button margin isn't considered if the button hbox has min y size of 0 in order to reduce the "chin" on the quick open popup.

How does this new algorithm work? There seems to be a lot more to it than the first two approaches. But it works pretty well.
Also, is there a way to not match "res://" ?

Is it correct that we rank HAT.wav first here?

@samsface thanks for testing it out! The matching process is similar to the earlier ones, although there are definitely more helpers etc cluttering it up haha. It essentially is doing:

Match each token starting from 0
If a match is found, test to see if that match overlaps any previously matched sections and reject it if so
Otherwise, score the new match and potentially update the top scoring match
In either case of a match, move the offset from 0 to start + 1 and try again. If no match, quit

Scoring then prioritizes (in rough order of importance):

Exact token matches without breaks
Longer matched sections
Fewer missed query characters
Filename match
Word boundary match

Is it correct that we rank HAT.wav first here?

Results with equal scores are tie-broken first on length and then on their alphanumeric order, which is why HAT.wav is ordered before hat.wav -- I'll add a slight score deduction if the match relies on being case insensitive to break that in favor of the exact match.

Also, is there a way to not match "res://" ?

I hadn't even noticed that! Yeah I can add a way to the fuzzy search code to skip a given prefix, or we can change that part of the updated quick open popup to not include it to begin with.
Also thanks for mentioning the PR; I didn't get an email or notification about it so I would've missed it otherwise. TIL!

I just updated it to 1) add a slight tie-breaker penalty if it relies on case insensitive matching and 2) ignore res:// by setting a new starting offset value. Ironically, similar to last time, I'll be out of town this weekend so slow updates from me for a bit.

I wonder if FuzzySearch can be used for autocompletion 🤔 We used to have something similar, but it was reverted.
Although for that, it can't be coupled to QuickOpenDialog.

I wonder if FuzzySearch can be used for autocompletion 🤔 We used to have something similar, but it was reverted. Although for that, it can't be coupled to QuickOpenDialog.

Thanks for the thorough pass! Someone had linked the prior merged and reverted subsequence matching stuff; I guess it picked up way too many low-quality results. This could be used in other places - it is definitely intentional that the fuzzy search class is distinct from the quick open stuff, and the use of RefCounted was I think originally so that it could be exposed in the gdscript api in a later change. It is somewhat coupled to the use of filepaths since it considers the last / index, but that just wouldn't be a factor for scoring non-path items.

I tested the changes in my project with lots of files and the performance is fine.
The results are more or less as accurate, however for some weird reason, tokens are favored in reverse order sometimes:

(left is old search, right is new)
Also the way matches are highlighted is a bit confusing:

The path is positioned before the name, but in the dialog it appears below and gray. Makes the order unintuitive. I know the dialog was already like that and not sure how it can be improved. Technically it's correct, just pointing it out.
The two unlabeled buttons at the bottom look odd:

It was acceptable when there was one, but now... idk.
There is also a regression: when search string is empty, the dialog displays nothing. Before this PR it would display recently opened entries.
Aside from the regression and token order weirdness, it's an improvement overall.

The code is probably ok. The FuzzySearch code is quite complex, so I mostly checked the style. As I noted in one of the comments, the fuzzy search classes don't need to be RefCounted. You can make them plain C++ classes, so they can be managed on stack instead of heap. It's tricky to use though (you'll need to pass by reference everywhere to avoid copying). If you plan to eventually try exposing it, you can leave FuzzySearchResult as RefCounted, as it seems to be the only class used non-internally.
I didn't check the added tests.

Thanks again for all the feedback. I don't think I'll get through this in the next few hours so I'll try to have an update ready next week.

tokens are favored in reverse order sometimes:

I think in this case all of the results in that right screenshot have an equal score. I'll add another tie breaking rule to favor in-order matches.

It was acceptable when there was one, but now... idk.

I agree; I'll add short labels for each one in addition to the current tooltips.

There is also a regression: when search string is empty, the dialog displays nothing. Before this PR it would display recently opened entries.

I'll need to check but I don't think I touched that logic. Iirc it only kicks in when you're searching for a single type, like with an export, and not when searching generally across more files/types https://github.com/godotengine/godot/blob/master/editor/gui/editor_quick_open_dialog.cpp#L662-L663 but I'll test and fix it once I'm through the other comments.

As I noted in one of the comments, the fuzzy search classes don't need to be RefCounted

Yeah I think speed maters for search so would second this. Maybe best to do everything without Refs and then have a separate pr to expose this in gdsctipt that copied the results into a friendlier class for gdscript use.

Sounds good, I'll convert it to not use RefCounted. Out of curiosity, I'll benchmark it before and after, and maybe add a few more benchmark cases. It makes sense to consider a better gdscript api; would that be its own proposal? I remember seeing a comment asking for this to be exposed to gdscript, but I'm assuming people would want more discussion about how exactly it should be exposed (like maybe as a string method search/search_all?)

Sounds good, I'll convert it to not use RefCounted. Out of curiosity, I'll benchmark it before and after, and maybe add a few more benchmark cases. It makes sense to consider a better gdscript api; would that be its own proposal? I remember seeing a comment asking for this to be exposed to gdscript, but I'm assuming people would want more discussion about how exactly it should be exposed (like maybe as a string method search/search_all?)

Yeah a separate proposal makes total sense to me. Once the gdscript api goes in, we can’t break it without breaking everyone’s code using it. So best not delay this PR with a ton more discussion.

Alright updated the branch with the following changes: (screenshots in top post are also updated)

New classes are not RefCounted anymore and copies are generally minimized
Results vector dynamically resizes so the setting no longer requires restart on change. Also this allows the destructor to be removed
A small tie-breaking bonus is added when tokens are matched in order
Regression with showing history is fixed
Labels have been added to each toggle, and the default window width has increased to account for this
A few new test cases have been added to try and stress different points of the search

Overall, although the new tie breaker does incur a small cost, this version of the code is ~30% faster according to the benchmark, with the speedup slightly increasing as the number of files increases. Queries with many tokens or token matches really benefit with close to a ~50% speedup,

So uh

The path should be above buttons. There is not enough space now.
Also instead of separate labels, you can set text of the CheckButtons, so they are easier clickable.

Will this apply to the Command Palette too? I often find myself mis-spelling things or using alternative words in tools like VS Code and Rider, and still getting to the same result - but I have to type everything exactly in Godot. For instance, typing "options" for opening settings, and being allowed to spell it wrong

@btarg not as part of this PR (it's already too big) but the fuzzy searching logic was intentionally kept separate so that it could be used elsewhere, just as a part of a different PR.

There's one more optimization I'd like to apply (but in another PR) that in my testing cut most of the benchmarks by 50%. A cull during the search that discards results early if their score is considerably less than the current max. I noticed we spend a ton of time adding then removing results that had like, a score of 3 while the max sore is 100.

Ah yeah I was thinking about something similar but instead considering the future potential score a la an advent of code problem from last(?) year. If you have the branch ready, feel free to pr against https://github.com/a-johnston/godot/tree/fuzzy-search-rebase

There's one more optimization I'd like to apply (but in another PR) that in my testing cut most of the benchmarks by 50%. A cull during the search that discards results early if their score is considerably less than the current max. I noticed we spend a ton of time adding then removing results that had like, a score of 3 while the max sore is 100.

Ah yeah I was thinking about something similar but instead considering the future potential score a la an advent of code problem from last(?) year. If you have the branch ready, feel free to pr against https://github.com/a-johnston/godot/tree/fuzzy-search-rebase

It's already fast enough (imo)  so wouldn't want to delay the PR. Let me know if you need help with anything to get the PR merged.

It's already fast enough (imo) so wouldn't want to delay the PR. Let me know if you need help with any coding work to get the PR merged.

Haha well you piqued my curiosity so I wanted to try it before nuking the benchmark. I settled on a pretty tame early cull criteria of being 1) negative (so, bad and missing query characters) and 2) at least 50 away from the current max score because I noticed that although the top results were of course preserved, the combination of match length being squared and a 100 bonus being given to exact token matches resulted in a query of ie spawner having a huge delta between spawn.tscn and spawner.tscn although to me the former still seems very relevant as a secondary result. From logging the early/late cull counts, this seems to have the most pronounced effect early when typing the query, when many more results are "valid" but negative, but later in the query most targets filter out with invalid matches. In the benchmark with this early cull criteria I saw a ~5% speedup for a synthetic 100k case and ~3% speedup for a 1k case, although in terms of the delta and not ratio, the 100k case had an average delta of ~3ms and the 1k case never had a delta greater than a few percent of a ms. That said, the benchmark is pretty biased due to the way I pruned down your tree to 1k lines.
Although the changes are pretty small I won't include them here but yeah for a future pr that addresses this

Keep in mind the nonlinear score bonuses and secondary results, not just the top result matching the benchmark's expected result
I've been changing the list item's name to name->set_text(p_candidate.file_path.get_file() + vformat(" (%d)", p_candidate.result->score)); to get a better intuition for the relative scores of displayed results

Should I rebase these changes down to fewer total commits or is it fine as is?

Yes please squash your commits into one, with a clear commit message with the same title as the PR

Not planning on making any more changes unless things are requested here but just to add visibility

There is an alternative RichTextLabel impl here https://github.com/godotengine/godot/pull/98278#discussion_r1818243991 which avoids the Label changes
It would be good (and pretty easy) to add additional editor options for what colors to use in the highlight border/fill. This would especially be useful for a11y / custom theme stuff. If this wasn't already reviewed I probably would add it in here.

Thanks! Congratulations on your first contribution! 🎉

Thank you!

every single change we've done so far brought its wave of discontents. Let's hope this time's the One Algorithm that Satisfies Them All.

Hopefully having added some relevant editor options helps with that :P


How to use front or 2nd camera with vision API to scan QR Code

Hi,
We wish to use front / 2nd camera of android tablet and phone instead of default back one
is there easy way without fiddling with camera source class?
Regards,
Hemal

This is what you need.
barcodeCapture..setCameraFacing(CameraSource.CAMERA_FACING_FRONT);
But as I'm too using this app, I can suggest a better way to use this API.
Try this : Google Mobile Vision API

Use Camera.CameraInfo.CAMERA_FACING_FRONT instead of CameraSource.CAMERA_FACING_FRONT in
.setFacing() method. It work well in my application.


Provide examples for how namespace_above functions

As I digest the codebase, I am trying to understand how the discrete parts work. Personally, I find such "test-like" comments for pure functions very helpful as a form of documentation.

I might strip this out completely based on this comment. https://github.com/joeldrapper/phlex/issues/2#issuecomment-1185699710


System.Net.Http.HttpClient possibly causing "Unresolved P/Invoke" crash.

Introduction
In my UWP app, calling GetAsync (and possibly others) in the System.Net.Http.HttpClient class causes my app to produce the following exception Unresolved P/Invoke method 'kernel32.dll!LocalAlloc' from this method. Please look for this method in build warnings for more details. However, there is nothing mentioned in the build warnings.
This only happens after publishing my app to the Windows Store.
More Info
This error is happening when I target the Fall Creators Update and using version 6.0.1 of the Microsoft.NETCore.UniversalWindowsPlatform nuget package. I'm not sure which one started causing the error as I targeted FCU and used the latest nuget package at the same time.
Calling this GetAsync method from either my UWP app or a .NET Standard (v1.4) library cause the same error.
What is more interesting is that this issue only happens after I publish my app to the Windows Store (and download it through the Windows Store). Performing local DEBUG and RELEASE builds work as expected. Side-loading the app also works as expected.
I've also done testing with both HttpClient's available in UWP. The Windows.Web HttpClient works as expected.
Specifications
Windows 10 (16299.19)
App Target and Minimum SDK: 16299.15
.NETCore UWP: 6.0.1
Visual Studio: 15.4.2
My source code is below if anyone wants to check out the references and how my project is setup:
Note: You will have to download the Performing tests on HttpClients commit to test.
https://github.com/DominicMaas/SoundByte

CoreRT repo is not a good support channel for issues with .NET Native, especially if they're specific to Store compilation and WinRT networking stack. None of the people working on that watch this repo. Please use the email channel described at https://github.com/dotnet/core/blob/master/Documentation/ilcRepro.md.


Error on "empty recycle bin"

Describe the bug
It is not possible to empty the trash in the admin area.
Steps To Reproduce
Steps to reproduce the behavior:

Delete a page, a book or a shelf
Go to Settings ->   Maintenance -> Open Recycle bin
Click on empty recycle bin or remove an item in the list

Screenshots
If applicable, add screenshots to help explain your problem.

Your Configuration (please complete the following information):

Exact BookStack Version (Found in settings):BookStack v0.31.5
Hosting Method (Nginx/Apache/Docker): Docker

Additional context
Log:
[2021-02-01 21:10:49] production.ERROR: Return value of BookStack\Entities\Tools\TrashCan::destroyEntity() must be of the type int, none returned {"userId":1,"exception":"[object] (Symfony\\Component\\Debug\\Exception\\FatalThrowableError(code: 0): Return value of BookStack\\Entities\\Tools\\TrashCan::destroyEntity() must be of the type int, none returned at /var/www/html/app/Entities/Tools/TrashCan.php:304) #19 /var/www/html/vendor/laravel/framework/src/Illuminate/View/Middleware/ShareErrorsFromSession.php(49): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #20 /var/www/html/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(171): Illuminate\\View\\Middleware\\ShareErrorsFromSession->handle(Object(BookStack\\Http\\Request), Object(Closure)) [2021-02-01 21:11:08] production.ERROR: Return value of BookStack\Entities\Tools\TrashCan::destroyEntity() must be of the type int, none returned {"userId":1,"exception":"[object] (Symfony\\Component\\Debug\\Exception\\FatalThrowableError(code: 0): Return value of BookStack\\Entities\\Tools\\TrashCan::destroyEntity() must be of the type int, none returned at /var/www/html/app/Entities/Tools/TrashCan.php:304) #18 /var/www/html/vendor/laravel/framework/src/Illuminate/View/Middleware/ShareErrorsFromSession.php(49): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #19 /var/www/html/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(171): Illuminate\\View\\Middleware\\ShareErrorsFromSession->handle(Object(BookStack\\Http\\Request), Object(Closure)) [2021-02-01 21:11:57] production.ERROR: Return value of BookStack\Entities\Tools\TrashCan::destroyEntity() must be of the type int, none returned {"userId":1,"exception":"[object] (Symfony\\Component\\Debug\\Exception\\FatalThrowableError(code: 0): Return value of BookStack\\Entities\\Tools\\TrashCan::destroyEntity() must be of the type int, none returned at /var/www/html/app/Entities/Tools/TrashCan.php:304)

Possible pull request https://github.com/BookStackApp/BookStack/pull/2530

Thanks for reporting @loki-core, This has been addressed and covered with testing within cc5d0ef4cf899650e64b4d6f1adbec7beb96c369, 7843d8f05427ec80fbac8933072cb2b9bfbc4905 & 61a911dd39f71c914abe407705f690b33cf7cf08 and will be part of the next patch release due within the next day or so.


bos.pegtoken and unittests

Change Description

Add pegtoken contracts V2 with core functionalities
Fix the files structure issue in bos.pegtoken folder
Add bos.pegtoken_tests in tests for unittest purpose

Deployment Changes

[ ] Deployment Changes

API Changes

[ ] API Changes

Documentation Additions

[ ] Documentation Additions

@libhasheos  please recommit latest code


fix keyboard navigation for selectors

What changes are proposed in this pull request?

fix keyboard navigation for selectors
add tweak to auto-scroll to active selector option when navigating to hidden options

How is this patch tested? If it is not, please explain why.
Using the dataset selector in the app with both keyboard and mouse navigation
Release Notes
Is this a user-facing change that should be mentioned in the release notes?

[ ] No. You can skip the rest of this section.
[x] Yes. Give a description of this change to be included in the release
notes for FiftyOne users.

fix keyboard navigation for selectors in the app
What areas of FiftyOne does this PR affect?

[x] App: FiftyOne application changes
[ ] Build: Build and test infrastructure changes
[ ] Core: Core fiftyone Python library changes
[ ] Documentation: FiftyOne documentation changes
[ ] Other

Thanks for the tweaks @benjaminpkane! 🙂


Handle getSDK Identities

Background

I learnt that this API was implemented for customers to retrieve SDK identifiers stored locally by Analytics, Target, Audience, Identity and Configuration extensions for GDPR purposes.

These identifiers are inferred from the extension shared states. It follows that there may be a possibility that all the shared states are not be available (i.e are in PENDING) when customer calls this API.

In Android Core 1.0, this logic is implemented in Configuration extension.
When the API is invoked, we wait for all required states to be non pending. If any of the contributing shared states are pending we bail immediately. Since this api invocation has a 5 second response timeout, null result is returned as a response after 5 seconds of api invocation.
However, internally, a marker for the request is still queued and and for every getSDKIdentities call we keep trying to fetch non pending shared states to populate a valid result and send it as a response (for the second time). Once a successful result is sent the request marker is removed from the internal queue.

In iOS, this logic is implemented in Identity extension.
When the API is invoked and any of the contributing shared states are pending, all incident events are blocked until all required states are non pending. Once valid states are available for all necessary extensions we process the event and send a response.

Android 2.0 Situation
In Android Core 2.0 we intend to handle this API in the Configuration extension as Identity is optional. Implementing it by blocking all event processing until all necessary states are available (like iOS does in Identity) is detrimental because we risk missing configuration shared state that may be set after the getSDKIdentities call (and stay blocked forever).
As of now we see following options to solve this:

Provide all the sdk identifiers based on the non-pending states as of the time when the api was invoked.
Keep trying for a fixed timeout to fetch all required states become available (non pending). If timeout elapses return without any identifiers
Keep trying for a fixed timeout to fetch all required states become available (non pending). If timeout elapses return with available identifiers.
Keep trying for an unlimited amount/large of time (!!) and return results when all states become available.

Questions

What is the right way to honor this request from a legal/product perspective? Should the aim be to return

all the identifiers that we know for certain will be stored in the future?
all available identifiers at the time the api call is incident?
any available identifiers within a set timeout as a best effort.
all available identifiers without a set timeout.

any thoughts on other ways to handle the implementation or, suggestions to improve above implementation proposals?

We zeroed in on implementing such that  all available identifiers at the time the api call is incident are retrieved.


Docker log files contains docker hub password in clear after using Kitematic on Windows

Expected behavior
Kitematic should not use an authentication api where the password is put in the URL and any password should never be found in clear
Actual behavior
Docker log files contains docker hub password in clear after using Kitematic on windows:
Information about the Issue

Windows 10
Kitematic <IP_ADDRESS>
Docker 1.13.0 (9795)
file: C:\Users\${user}\AppData\Local\Docker\log.txt

[11:33:04.393][VpnKit         ][Debug  ] com.docker.slirp.exe: forwarding to server <IP_ADDRESS>:53
[11:33:21.206][ApiProxy       ][Info   ] proxy << POST /images/create?authconfig=%7B%22username%22%3A%22manuc66%22%2C%22password%22%3A%22------password is in clear here------%22%2C%22auth%22%3A%22%22%7D&fromImage=swaggerapi%2Fswagger-generator&tag=latest
[11:33:21.209][ApiProxy       ][Info   ] proxy >> GET /containers/swaggerapi/swagger-generator:latest/json
[11:33:21.210][ApiProxy       ][Info   ] Dial Hyper-V socket 841de5db-4e9a-44fb-9a8c-54c637309976:23a432c2-537a-4291-bcb5-d62504644739
[11:33:21.210][ApiProxy       ][Info   ] proxy >> GET /containers/json?all=true

Can you open this issue in https://github.com/docker/for-win ?

No problem. It's now https://github.com/docker/for-win/issues/440
Thanks.

PR also opened to fix at its root (within the node module):
https://github.com/apocas/docker-modem/pull/72


Term of CFD is not displayed in UI

Somehow incorporate:

CFD term length + end timestamp
Oracle event ID (+ link to olivia?)

Obsolete. If we want to display anything, it might be the time to the end of the next funding period.


WIP - Color validation and Color Picker

Fixes #85

cc @SylvainCorlay still not working with xgenerator / xmaterialize :/


Make appending carbon.super configurable for super tenant

In the password recovery flow, once the user submits the username, a message similar to below will be displayed.
Password recovery information has been sent to the email registered with the account   user1<EMAIL_ADDRESS>
However, carbon.super appended to the username (or the email, if the email as username is enabled) should be configurable to prevent appending it.
Password recovery information has been sent to the email registered with the account<EMAIL_ADDRESS>

This should also be considered in the user registration page (Error message displayed when an already existing username is entered) and all other scenarios wherever the username is appended with @carbon.super.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


not working...

plug-in is not working.
When you click on any readio channel it keeps loading and thats it, doesn't work

please put the plugin-log (../Application Support/Plex Media Server/Logs/PMS Plugin Logs/com.plexapp.plugins.tunein2017.log) here. I will take a look at it.
/R

There you go! :)
com.plexapp.plugins.tunein2017.log

could you please make a try with English as choosen language?
I'm not shure, but the reason may be a code-problem with the localisation-strings.
I don't need a new log for this try.

Great.
A fix for this bug comes with the nex update.
You are welcome to support the the translation project for this plugin (no supporter for arabic until yet).
Please take a look at the Plex forum:
https://forums.plex.tv/discussion/288260/rel-tunein2017
Contextes and pics you can find in the wicki:
https://github.com/rols1/TuneIn2017/wiki/Pictures-and-Contexts-for-the-Localisation-Project
/R


WIP - School overview page experimentation

@alexsoble Here's the branch we worked on yesterday, with some improvements to the filtering, the header visuals and adding an initial table.  The URL is /schools/1/overview.
There's a flag to use local fixture data now, and there's scripts for generating this.  There's also a hack to open production student and homeroom pages from the local overview page, which is useful for demoing or for understanding the data.
I probably won't get to work on this much more until later in the week, but this should be useful for getting another round of feedback.  Based on how that goes, I can help out with either making UX and design revisions, or cleaning this up to finish it off and get it in PR shape.
Potential TODOs:

proper import of React (potentially JSXify)
split up components in overview into separate files
show homeroom names
URL and back button support
some Jasmine smoke tests
cleanup pass on basically everything
improve students list

@alexsoble Closing this to move onto your branch on the main repo: https://github.com/codeforamerica/somerville-teacher-tool/tree/experiment/students-exploration.


Upon running the code example in main.py, I am getting only one output despite setting the setMaxTweets to 1000. Also I don't get any output using custom search terms. Please help. I have written the error which arises.

/Library/Frameworks/Python.framework/Versions/3.6/bin/python3.6 /Users/siddharth/PycharmProjects/GetOldTweets-python-master/Main.py
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/pyquery/pyquery.py", line 66, in fromstring
result = getattr(etree, meth)(context)
File "src/lxml/lxml.etree.pyx", line 3213, in lxml.etree.fromstring (src/lxml/lxml.etree.c:82934)
File "src/lxml/parser.pxi", line 1819, in lxml.etree._parseMemoryDocument (src/lxml/lxml.etree.c:124533)
File "src/lxml/parser.pxi", line 1700, in lxml.etree._parseDoc (src/lxml/lxml.etree.c:122964)
File "src/lxml/parser.pxi", line 1040, in lxml.etree._BaseParser._parseUnicodeDoc (src/lxml/lxml.etree.c:116705)
File "src/lxml/parser.pxi", line 573, in lxml.etree._ParserContext._handleParseResultDoc (src/lxml/lxml.etree.c:110510)
File "src/lxml/parser.pxi", line 683, in lxml.etree._handleParseResult (src/lxml/lxml.etree.c:112276)
File "src/lxml/parser.pxi", line 613, in lxml.etree._raiseParseError (src/lxml/lxml.etree.c:111124)
lxml.etree.XMLSyntaxError: Start tag expected, '<' not found, line 2, column 1
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/siddharth/PycharmProjects/GetOldTweets-python-master/Main.py", line 36, in
main()
File "/Users/siddharth/PycharmProjects/GetOldTweets-python-master/Main.py", line 25, in main
tweet = got.manager.TweetManager.getTweets(tweetCriteria)[0]
File "/Users/siddharth/PycharmProjects/GetOldTweets-python-master/got3/manager/TweetManager.py", line 26, in getTweets
scrapedTweets = PyQuery(json['items_html'])
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/pyquery/pyquery.py", line 226, in init
elements = fromstring(context, self.parser)
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/pyquery/pyquery.py", line 70, in fromstring
result = getattr(lxml.html, meth)(context)
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/lxml/html/init.py", line 867, in fromstring
doc = document_fromstring(html, parser=parser, base_url=base_url, **kw)
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/lxml/html/init.py", line 755, in document_fromstring
"Document is empty")
lxml.etree.ParserError: Document is empty

I can't reproduce the error you are describing. The examples work fine for me. Can you maybe attach your code?
Also make sure you remove the '[0]' from 'tweet = got.manager.TweetManager.getTweets(tweetCriteria)[0]' if you want to see more than one tweet

Done. Output file generated "output_got.csv".
Traceback (most recent call last):
File "/usr/local/lib/python3.6/site-packages/pyquery/pyquery.py", line 66, in fromstring
result = getattr(etree, meth)(context)
File "src/lxml/lxml.etree.pyx", line 3213, in lxml.etree.fromstring (src/lxml/lxml.etree.c:82934)
File "src/lxml/parser.pxi", line 1819, in lxml.etree._parseMemoryDocument (src/lxml/lxml.etree.c:124533)
File "src/lxml/parser.pxi", line 1700, in lxml.etree._parseDoc (src/lxml/lxml.etree.c:122964)
File "src/lxml/parser.pxi", line 1040, in lxml.etree._BaseParser._parseUnicodeDoc (src/lxml/lxml.etree.c:116705)
File "src/lxml/parser.pxi", line 573, in lxml.etree._ParserContext._handleParseResultDoc (src/lxml/lxml.etree.c:110510)
File "src/lxml/parser.pxi", line 683, in lxml.etree._handleParseResult (src/lxml/lxml.etree.c:112276)
File "src/lxml/parser.pxi", line 613, in lxml.etree._raiseParseError (src/lxml/lxml.etree.c:111124)
lxml.etree.XMLSyntaxError: Start tag expected, '<' not found, line 2, column 1
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "Exporter.py", line 70, in main
got.manager.TweetManager.getTweets(tweetCriteria, receiveBuffer)
File "/Users/tuhinchakrabarty/Desktop/GetOldTweets-python/got3/manager/TweetManager.py", line 26, in getTweets
scrapedTweets = PyQuery(json['items_html'])
File "/usr/local/lib/python3.6/site-packages/pyquery/pyquery.py", line 226, in init
elements = fromstring(context, self.parser)
File "/usr/local/lib/python3.6/site-packages/pyquery/pyquery.py", line 70, in fromstring
result = getattr(lxml.html, meth)(context)
File "/usr/local/lib/python3.6/site-packages/lxml/html/init.py", line 867, in fromstring
doc = document_fromstring(html, parser=parser, base_url=base_url, **kw)
File "/usr/local/lib/python3.6/site-packages/lxml/html/init.py", line 755, in document_fromstring
"Document is empty")
lxml.etree.ParserError: Document is empty
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "Exporter.py", line 79, in
main(sys.argv[1:])
File "Exporter.py", line 72, in main
except arg:
TypeError: catching classes that do not inherit from BaseException is not allowed
SAME ERROR

A proxy will instantly fix the issue. This is the most common problem reported, so there should probably be some mention in the README.
One of the classes you use has a default param for adding a proxy. You'll have to find which one it is as I don't remember.

You have been IP blocked. Use a proxy to fix this issue. There is support for a proxy with the getTweets function.


error loading script in fast stable diffusion colab

it worked in in stable diffusion local,but not in colab.

This is a reoccurring issue with fast-stablediffusion since they change the working directory mid-startup, which confuses the path comparison. As far as I'm aware I can't prevent that from my side, a small code edit is needed in the colab. Since the folder structure they use has changed multiple times since this issue first popped up and undone previous fixes, I'm not really sure what the current state is, so I can only tell you what worked for others before for now:
In the last code cell, try changing
with capture.capture_output() as cap:
  %cd /content/gdrive/MyDrive/sd/stablediffusion

to
with capture.capture_output() as cap:
  %cd /content/gdrive/MyDrive/sd/stable-diffusion-webui

at this part close to the end:

See #67, #79 and https://github.com/TheLastBen/fast-stable-diffusion/issues/409 for more info.

thankyou bro


The problem is traced to the use of scipy in the weight initialization of the two models:
https://github.com/pytorch/vision/blob/90645ccd0e774ad76200245e32222a23d09f2312/torchvision/models/inception.py#L122-L124
https://github.com/pytorch/vision/blob/90645ccd0e774ad76200245e32222a23d09f2312/torchvision/models/googlenet.py#L126-L128
The slow down is caused by a regression on scipy affecting versions 1.4.x: https://github.com/scipy/scipy/issues/11299. The reason why only Windows runs are affected is because our CI config is pinning the version of scipy to 1.4.1 on Windows but not on Linux. This was introduced at #2328 (first commit of the file) and it's probably by accident that the two files are not aligned.
By unpinning the version and allowing scipy to use newer versions on Windows, the issue is fixed. This is an alternative solution to #3195.

The failing tests are not related to this PR.

Awesome, thanks for the PR @datumbox !

Awesome, thanks for the PR @datumbox !


Add Ant property for location of Saxon jar

It's not clear to me why it is necessary to specify the location of saxon.jar (and xml-resolver.jar) on the Ant command line.  Especially since you're running the <java> task and providing <pathelement path="${java.class.path}"/>.
I don't want to have to remember where Saxon is on every computer where I run XSpec.  I would rather set a property for it in xspec.properties on each computer and have XSpec just work.

If you put Saxon and resolver jar into [ant]/lib, you don't have to specify the location on the command line. Does it help?

I have put Saxon and resolver.jar in the Ant lib directory before, back when the <xslt> task was particularly bad about using a specified classpath.  Even then, it was only half a solution.
You'll probably find that I was the one who added the saxon.jar property to the build file (and the now-defunct xspec macro).  I have in the past made build files that work out if they're being run under oXygen and use the oXygen version of Saxon if so.  I could look at that doing that here, but my first priority is getting XSpec 1.0.0 to work as a submodule of schematron/schematron-test.
The minimum required for this issue would be something like:
<pathelement location="${saxon.jar}" if:set="saxon.jar" />
<pathelement location="${xml-resolver.jar}" if:set="xml-resolver.jar" />

The if:set isn't strictly necessary, but it saves Java from looking for files that don't exist.

I once contacted the oXygen support, for saxon.jar didn't take effect at all on oXygen.
To resolve it, they suggested inserting <pathelement path="${saxon.jar}"/>. But then another problem happened. (The embedded license was invalidated.)
They ended up telling me to remove <pathelement path="${saxon.jar}"/> and just use ${java.class.path} with necessary jars included.
Also, in the current build file, not only <java> but <xslt> requires the resolver jar. And then we will need to add some test cases for the added properties on Travis and AppVeyor.
So I'm afraid inserting a <pathelement> might make the build file a bit error prone. Just relying on Ant classpath seems simpler.
Any reason schematron/schematron-test can't easily include the jar files in Ant classpath?

I want it to 'just work'.  I had XSpec testing schematron/schematron-test on a Linux box, but then I wanted to do some work on a Windows box.  I had the choice either of installing XSpec on the Windows box and making the schematron/schematron-test clone work with that or of adding XSpec as a submodule and making it 'just work' everywhere (possibly after setting a property or two before the first use).  We might have to agree to disagree, but adding multiple -lib every time (that are different on each platform) just for this use of Ant doesn't fit my definition of 'just work'.

I quickly tried to set classpath in <java> and <xslt> so that it can be overridden externally. Now

XSpec tests run much slower. (x3)
<xslt> fails to find the resolver.

I may be doing something wrong. I'll continue trying to resolve them.
Anyway, such complexity like this is what I was afraid.

@tgraham-antenna , I can't get <xslt> work with your suggested changes.
Were you able to change build.xml and get this test case work without including resolver.jar in Ant classpath?

*nix: https://github.com/xspec/xspec/blob/b5707e91a15a69c669daf6c0cd3875e63fa5340b/test/xspec.bats#L348
Windows: https://github.com/xspec/xspec/blob/b5707e91a15a69c669daf6c0cd3875e63fa5340b/test/xspec-bat.cmd#L426

By the way, as for the now-defunct xspec macro, I didn't object to removing it, not only because no test existed for it but also because some properties were added. Defining a common denominator macro seemed no longer possible. Letting each user define his own macro based on his specific need seemed to be the way to go.

I submitted a pull request for Ant last month. (#187)
If you're interested or find a concern about it, please jump in there as a reviewer.


Issues with IPFS routing

We have observed some unfortunate behaviour with a private IPFS cluster that means results are not downloadable. The issue appears to be that nodes that are in the same swarm as each other don't appear to discover content on each other.
An example setup:

Two separate machines – one running Bacalhau in requester mode, one in compute mode, both with kubo IPFS nodes
The IPFS nodes have each other listed in their peering config
Run a Bacalhau job which will generate some result in the compute node
Run Bacalhau get with the IPFS swarm addr of the requester node

Expected: results are downloaded successfully as requester IPFS node is able to get the data from the compute IPFS node
Observed: download times out as requester is seemingly unable to find the data
Observed: problem disappears if ipfs pin is used to explicitly and permanently cache the data on the node

And/or, Run Bacalhau get with the IPFS swarm addr of the compute node

Expected: results are downloaded successfully
Observed: download times out even though this should be directly where the data is stored (??)
Hints: IPFS node bootstrap config might be an issue, by default it points at public nodes. May be worth leaving it empty or setting it to the same as the peering list.
Context:

https://bacalhauproject.slack.com/archives/C055U60AJKE/p1697211889274979

embedded ipfs node has been deprecated in favour of connecting to your own node
https://github.com/bacalhau-project/bacalhau/pull/4061


Likely typo in memory_map.py

I just remembered that a while ago I noticed a very likely typo in memory_map.py.
The following line is probably missing a ::
https://github.com/scalesim-project/scale-sim-v2/blob/78919903a99d87824a106bedb54835dd8c557c00/scalesim/memory_map.py#L85
I.e., the line should be
self.ifmap_map_list.append(elems[:self.num_banks])

While I do not understand the code 100%, I am pretty sure that the : is missing because there are very similar lines that come after that have a : in this position:

https://github.com/scalesim-project/scale-sim-v2/blob/78919903a99d87824a106bedb54835dd8c557c00/scalesim/memory_map.py#L100
https://github.com/scalesim-project/scale-sim-v2/blob/78919903a99d87824a106bedb54835dd8c557c00/scalesim/memory_map.py#L115

Hi @datMaffin, thank you for pointing this out but currently we do not support memory mapping.


FutureExt::inspect_err() ?

In continuation of #700
Inspecting errors in a future could be quite useful. Since we have the same API in Stream perhaps we could include one in Future as well?

If i understand correctly, this method doesn't exist in the Futures-0.1 release, can/will this be backported?

It's a pretty trivial method to add, if you/someone opens a PR to add it to 0.1 I'm sure it'll be accepted.


Update zero-downtime-deployments.md

Fix typo "exists" -> "exits"

Codecov Report

Merging #612 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #612   +/-   ##
=======================================
  Coverage   55.00%   55.00%
=======================================
  Files          62       62
  Lines        5303     5303
=======================================
  Hits         2917     2917
  Misses       1959     1959
  Partials      427      427

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 c367e65...bf27aed. Read the comment docs.


False positive for RSpec/PendingWithoutReason in 2.16.0 when object has a method pending

Hi rubocop-rspec team,
I noticed that we had a false positive in our project (at least i think it's a false positive): There is a object with an enum state with the value pending. In our test we are querying all pending transations. I would expect that the spec does not consider this as an issue.
    it 'approves the transaction' do
      expect do
        request
      end.to change(Transaction.pending, :count).from(1).to(0)

(Ruby 2.7.7, rubocop 1.43.0, rubocop-ast 1.24.1, rubocop-rspec 2.16.0)

Thank you for your report!
This issue has been fixed in https://github.com/rubocop/rubocop-rspec/pull/1515 and https://github.com/rubocop/rubocop-rspec/pull/1516, but not yet released. Please wait for the next release.

My apologies for a temporary inconvenience.
@bquorning does it make sense to release bug fixes in a patch release before 2.17 with Capybara extraction?


Project UI - Update Experiments page to work with new navbar

Update experiments page to work with new navbar, including look and feel, resizing elements, showing/hiding experiment, clicking to switch experiments, etc...

Reworked pages under experiment to work with navbar. Updated scrolling behavior. Reworked some page layouts. Fixed nagging issues.


Features founded Issue #25

[1] To ignore files .pyc generated by executing the project, add .gitignore. [2] Modify URLs in chatbot.py, adding http:// to work in ubuntu. [3] Add pyjokes library to cmd3, this library return a random joke.

fixes #7


Fixed: settings undefined in core/src/download.js

This fixes #194
My linting may have modified the structure a bit, I can do another pull req. if you prefer.

Could you update this pull request to remove the changes regarding the linting and code restructuring?
It's better to have a commit diff with only what actually changed.
Also regarding the functionality itself, could you change the order a bit,
instead of job, asset, settings, -> job, settings, asset?
Thank you!


chore: Test sticky comment on the public functions report

Display results of https://github.com/AztecProtocol/aztec-packages/pull/8750 when we have changes in public function size.

Closing as this PR was simply for displaying what happens when public function bytecode sizes change after #8755.


Support an output array parameter.

The MKL interface supports passing the output array for a matrix-vector product. (for example, see: https://scc.ustc.edu.cn/zlsc/tc4600/intel/2017.0.098/mkl/common/mklman_f/GUID-1FAA1369-97A7-4F13-87B2-3FE4425D876D.htm)
This is helpful when we want to avoid allocating a new array only to add it to an existing array. For example in the pseudocode snippet:
y = np.zeros(...)
for m in list_of_matrices:
    y += dot_product_mkl(m, x)

That's no problem at all. The only limitation is that it'll have to be contiguous, so filling most array slices is right out.

Yeah, that seems like a reasonable restriction. You could take a similar attitude to numpy.dot: https://numpy.org/doc/stable/reference/generated/numpy.dot.html
This is a performance feature. Therefore, if these conditions are not met, an exception is raised, instead of attempting to be flexible.

I added out and out_scalar to dot_product_mkl such that out := matrix_a * matrix_b + out_scalar * outin 193e1182ef3e889684a71d30c2474ea7029b59ea. Theoutmatrix needs to match what would have been produced anyway or it'll raise aValueError`.
I'm probably gonna write more tests for some of the edge conditions on this before I package it and put it onto pypi.

Thanks! Makes sense that this would add a decent amount of new complexity and thus should be tested thoroughly. Really helpful that this package exists!

Aight I think I've gotten all the subtle bugs. I've packaged this and released it as v0.6.0 - if there are any issues that come up, please let me know.


Please suggest OrderBy usage for this Json

Hi,
This is not a Bug or issue. Please consider this as Info/Request
I have the below data with me in json format.
This is the list of comments. In this example 19 comments are there under items tag.
http://www.jsoneditoronline.org/?id=47b282c498505ed25869dc36f5f3bd58
Comments are sorted based on Date published.
Now on the Item Id 10th there are two replies to this comment.
I am looking for a way to order this reply based on publishedAt time.
Below is my html:
Here videoComments = the json data specified above
<ion-list>
  <ion-item *ngFor="let comment of videoComments">
    <!-- here i Print the details what is required , next if there are any reply i would like to show them-->
    <div *ngIf="comment.snippet.totalReplyCount != 0">
      <ion-list>
        <ion-item *ngFor="let reply of comment.replies.comments | orderBy: 'publishedAt'">
          <!-- Above orderBy pipe is not working , may be my syntax is wrong -->
           <!-- I would like to show replies in ascending order of when it was submitted-->
          <!-- Details of replies will be shown here -->
        </ion-item>
      </ion-list>
    </div>
  </ion-item>
</ion-list>

Your assistance is highly appreciated.

Hi, looks like you need to transform your date which is in string format into an instance of Date first. Otherwise a sort on the string will be done, which is probably not what you would expect.

@fknop Thank you for the very quick response. You are right I will definitely look at your suggestion.
I believe my way of providing the key for the orderBy pipe is not right. Let's assume publishedAt value is string, I tried both ' publishedAt' and '-publishedAt' this did not change the result. it should have changed the reault as both the values are not same for two replies of the comment.
Thanks

@fknop do you think this syntax is correct to fetch replies, based on my Json object?
<ion-item *ngFor="let reply of comment.replies.comments | orderBy: 'publishedAt'">
Thanks


Issue 255 - Fixing update/upload new file bug

Relevant to PR #249 where the full refactor to this functionality will take place. Changes here will ultimately be changed to the version implemented there. The old implementation of upload new file/update file has been buggy for a while, but has not been revisited. Now that we're starting to refactor session-core with uploadDocument being one of the targets, it's a good time to make sure the functionality under the hood is implemented correctly so that it could be used as the starting point for optimizations and subsequent refactors.
This PR fixes that issue by implementing what originally intended for that flow (see #249 (comment) for context).
First file upload works like before since the container initially starts empty, so updateDocument was not called when uploadDocument is triggered. A new Thing has been generated for document.ttl at #document. This Thing will simply be a reference to itself rather than the document files like with container.ttl. All subsequent Things will be named something else like #document1 or #document2, etc. But a more robust naming scheme would need to be flushed out in case we're going to keep this feature during the refactor in PR #249.

A second file upload triggers updateDocument since container is no longer empty. But unlike when the container is empty, all subsequent calls for updateDocument triggers confirmation window which enables the user to cancel if desired.
Assuming the user continues and not cancel the operation, since the file name for the second upload is different, uploadNewFile within updateDocument is triggered instead of updateFile. Compared to uploadDocument which generates a new document.ttl after uploading a file, uploadNewFile instead appends to the existing document.ttl.

Uploading an updated version of the first file will trigger updateFile when uploading and the old file will be overwritten by the new one. The Thing related to the first file will also be updated based on what was filled out and a new dateModified field is included to show that it has been updated.

Other changes include edits to the message for new file upload and file updates so that the message is now "Uploading new/updated content on Pod..." in FormSubmissionHelper and a new parameter for createResourceTtlFile so that a unique name could be used for the Thing being generated instead of being hard-coded to 'document'.

Had been following and reviewing the progress of #249. It seems to be handling the cases touched by this PR. Can close this in favor of #249.


chore: update Rust and all dependencies to allow wasm-pack to be installed again

What is it?

[x] Feature / enhancement
[ ] Bug
[ ] Docs / tests / types / typos

Description
Update Rust so wasm-pack can be installed again, it stopped working on older Rust versions. Also modernized all the dependency versions. This involves no breaking updates.
Use cases and why
cargo install wasm-pack now completes succesfully again.
Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[x] Added new tests to cover the fix / functionality

Elaborating: opened this because i was following the contribution instructions which prompted me to run "cargo install wasm-pack", which then failed because the latest version of wasm pack does not compile on the current 2022 version of Rust. Ran into some dependency problems so updated dependencies where possible, making sure not to introduce breaking changes. Let me know if i made any mistakes, this was a quick attempt 🙂

Having some trouble running the test suite, but cannot verify whether tests did work on my machine on main, because of errors with wasm-pack and wasm-bindgen, happens to be that is exactly what this PR aims to solve :)
><EMAIL_ADDRESS>test.unit /Users/jensmeindertsma/Development/qwik
> tsm node_modules/uvu/bin.js packages unit.ts --tsmconfig tsm.cjs

(node:62689) ExperimentalWarning: Custom ESM Loaders is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
ReferenceError: Cannot access 'dangerouslySetInnerHTML' before initialization
    at file:///Users/jensmeindertsma/Development/qwik/packages/qwik/src/core/render/dom/visitor.ts:755:4
    at ModuleJob.run (node:internal/modules/esm/module_job:192:25)
    at async CustomizedModuleLoader.import (node:internal/modules/esm/loader:228:24)
    at async Module.run (file:///Users/jensmeindertsma/Development/qwik/node_modules/.pnpm/uvu@0.5.6/node_modules/uvu/run/index.mjs:9:3)
    at async /Users/jensmeindertsma/Development/qwik/node_modules/.pnpm/uvu@0.5.6/node_modules/uvu/bin.js:26:5
 ELIFECYCLE  Command failed with exit code 1.
 ELIFECYCLE  Test failed. See above for more details.
 ```
 Does anyone with experience with this repository have any pointers?

Managed to work around the wasm-bindgen outdated issue on main by running cargo install wasm-bindgen-cli --vers "0.2.84" --locked. This could be a workaround, but this would still leave Rust severely outdated as is. This did allow me to confirm that the above error also occurs on main on my M1 Mac. Not sure where to look next though to resolve this test error

I would love to merge this, but it is currently not in a mergeable state. Can you have a look...

@jensmeindertsma are you still on this PR?

@jensmeindertsma thanks for your help. are you still on this PR?

i have arch installed on a system with this cooler but haven't been able to install this PR😬 not familiar enough with AUR

i have arch installed on a system with this cooler but haven't been able to install this PR😬 not familiar enough with AUR

me neither. Anything you need from me?

sorry for that weird response, was definitely more asleep than awake 😓
I have everything running BUT the e2e tests which are persistenly failing, not sure how to debug those.

Also, clippy is giving a hard error on these lines: https://github.com/BuilderIO/qwik/blob/6d0be5e2ba46dfbee76e11017abaa82124b65235/packages/qwik/src/optimizer/core/src/transform.rs#L2128-L2131
https://rust-lang.github.io/rust-clippy/master/index.html#/collection_is_never_read
I am not familiar enough with this codebase to understand what is going on in this 2000 lines+ of code file, is there anyone from the core team who could elaborate on these few lines of code?

I cannot take these any further, I can get stuff to compile on my local machine, but the e2e tests just fail massively. Closing because I don't have the capacity for this at the moment.


TypeError: Cannot read property 'avatar_url' of null

The github action is being marked as completed, but no message is appearing in teams
The steps is showing the following error
Post job cleanup.
Workflow run information: {
  "owner": "wcbs-ltd",
  "repo": "hub-ui",
  "ref": "3d36a7328a26420e1578759ce60f89d6ab9443d2",
  "branchUrl": "https://github.com/wcbs-ltd/hub-ui/tree/refs/heads/GithubActions",
  "runId": "131963184",
  "runNum": "2507"
}
(node:3692) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'avatar_url' of null
    at Object.formatCozyLayout (/home/runner/work/_actions/toko-bifrost/ms-teams-deploy-card/3.0.1/dist/post/index.js:1:365095)
    at Object.formatCompleteLayout (/home/runner/work/_actions/toko-bifrost/ms-teams-deploy-card/3.0.1/dist/post/index.js:1:4207)
    at Object.formatAndNotify (/home/runner/work/_actions/toko-bifrost/ms-teams-deploy-card/3.0.1/dist/post/index.js:1:16527)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
(node:3692) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:3692) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Hi @Shaun-Harrison, may I know what type of git action it is trying to process? For instance, push, pull_request?

Hello, this was a push

I understand now that both author and committer will be null if the committer is not associated with a Github user. https://github.community/t/request-for-commits-author-null-and-committer-null-cant-get-user-login/14409/2. I guess I'll just have to remove the author details in the card if its null.

This has been resolved on #12.


Clean up Xiaomi Air Quality Monitor support (cgllc.airmonitor.b1)

Description:
Fixes https://github.com/home-assistant/home-assistant/pull/27735
@fwestenberg Could you provide some testing? Especially is the unique_id properly generated?
Pull request with documentation for home-assistant.io (if applicable): https://github.com/home-assistant/home-assistant.io/pull/11027
Checklist:

[ ] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Thanks for your patience! :-)

Here's the unique_id of my device. Seems right!


The instruction of using the plugin is not clear!

Can you make the install more easier life, or explain step by step what to do.
Many thanks!

See #21.


Changing text on input does not save the changes with buttons = no

I don't want to use buttons and I want the value to be saved.
I have made this fiddle
Supplying buttons: 'no' and submit on blur
When I click on a button element that has ng-click, the value is not changed only after the ng-click has ended.
To replicate - try to change the value on the User.Name and click the button.
Added: If I add a $timeout(...), it will work.

The reason the select works is because when you select a new item in the list, that is the blur.
What are you trying to do with the button and why is timeout not an option?

The button is collecting data for later send the server, so if a user press this button, the value will still be the old value.
timeout is an option but I prefer not to use timeout if not needed.
Do you not consider this a bug? basically with button='no' I would expect that the value will be changed on change, not on blur. (the docs states on change)

The select input is setup so that it saves on change for no buttons, which works well for a select.  A text input is not setup the same way and I'm not sure it would work as well.
I think the best thing to do is use the timeout


There's no next tag in the npm package.

I tried to install with npm install koa-connect@next and gives me an error, using npm show koa-connect in tags section i can confirm there's no next tag as the documentation says, but there's is a latest tag which is compatible with koa 2, documentation should  be updated.

This is still an issue...

Thank you for the reminder @krini. I've removed references to needing the next tag for installation, and mentioning Koa v2 as something special since it's now the stable version. Published as 2.0.1


Implement dynamic facets

Do you want to request a feature or report a bug?
This is a feature request – both for instantsearch.js and react-instantsearch.
It might make sense to have something implemented in the Helper that would then be leveraged in instantsearch.js.
Feature: What is your use case for such a feature?
This is a need mainly stemming from e-commerce use cases, but could be useful for any use case that involves a wide variety of facets.
In large e-commerce selling many different types of products (from electronic devices to groceries), displaying the facets that make sense for any given search is a problem.
Besides the generic price, brand or categories facets, there's no way to know in advance what will make sense to display in the facets.
If a user searches for tv you'll want to display facets for connectivity, size of screen, screen technology, etc. — for a query like printer you'd display things like number of pages per minutes, included scanner, connectivity, and so on.
The way we solve this today is by doing the following:

Each record has two attributes: relevant_properties and properties. They'd look something like this:

  {
    "properties": {
      "Connectivity": ["HDMI", "WiFi"],
      "Screen Size": "50"
    },
    "relevant_properties": ["Connectivity", "Screen Size"]
  }

relevant_properties and properties are added to the attributes for faceting
at each keystroke, a first search query is sent with the query string (+ possible active facets), facets=['relevant_properties'] and hitsPerPage=0. This retrieves the properties that are the most represented in the query results.
Once the results come back for that first query, a second query is sent to actually fetch the hits, along with the facets that have been returned/extracted from the results of the first query.

I have built a fiddle to demonstrate this using the JS helper:
https://jsfiddle.net/rbbosLh3/
Unfortunately the only dataset I had handy for this usecase is a french one, sorry.
Feature: What is your proposed API entry? The new option to add? What is the behavior?
Today there's no (easy) way to dynamically remove/add widgets in an instant search implementation, which makes it hard to implement this solution.
I'm not sure what should be the API/modifications made to our libs to make this work, but it would be nice to have something friendlier than going back down to the helper for this.
What project are you opening an issue for?

react-instantsearch
instantsearch.js
algoliasearch-helper?

This actually sounds more like an API feature than a library feature to me, but does sound very useful.

@Haroenv yup indeed, but it's not clear yet that this would be implemented at the API level

I talked about this with @bobylito last week, and he suggested to search with facets:*, and then only render those that are deemed relevant X times. Although doing two requests isn’t a terrible idea, this alternative is having one bigger request (granted that you don’t have thousands of facets)

Could the relevant facet be used in the display of the hits too?

@Haroenv @bobylito searching with facets:* is what this methods is working around. This is terrible in terms of performance, as the total number of facet attributes are usually in the hundreds or thousands in that kind of scenario.
@bobylito I did not get your question

@olance Your method retrieves the most relevant facets right? And facets are attributes of the records, so could we use those to makes custom hits rendering?

I still don't get it 🙊

Yes 😅

@bobylito Sorry, holidays :)
So yeah now I have understood what you meant 🙌
It totally makes sense, however I'm not sure how you'd tie that with the requested feature?
Make the dynamic facet values available within the widget template?

@olance I had the opportunity to work on an actual implementation with @sepehrfakour. This uses the searchFunction, which can't be made into a widget. I'm thinking a guide for that.

@olance & @bobylito let me know if you think this guide is useful: https://github.com/algolia/dynamic-faceting-instant-search/blob/master/guide.md

Link results in 404 here?

The link is private right now. @sepehrfakour, should it be made public and iterated when needed?

Let's ensure we provide a public link soon or remove this private link (with explanations on why) for now (can raise frustration from users). cc @sepehrfakour

Updated above link with a public guide:
https://github.com/algolia/dynamic-faceting-instant-search-guide/blob/master/guide.md

done in #4687, more information in the doc to come


Fix s2i op when shard grid is larger than actual used grid

Ticket
https://github.com/tenstorrent/tt-metal/issues/15004#issuecomment-2476975904
https://github.com/tenstorrent/tt-metal/issues/14902
Problem description
Bug occured when running a matmul + s2i ops test, s2i op does not take account that some cores are unused in the passed in sharding grid, and writes extra data to output tensor, causing L1 corruption.
Fix is for s2i to take account for the unused cores.
Checklist

[ ] Post commit CI passes
[ ] Blackhole Post commit (if applicable)
[ ] Model regression CI testing passes (if applicable)
[ ] Device performance regression CI testing passes (if applicable)
[ ] New/Existing tests provide coverage for changes

Is this rebased onto Naif's changes for S2I/I2S from yesterday and have you tried BH post-commit with it?

@sjameelTT it should have Naif's commit.
Will test BH

I think in the future, we should make shard spec store the exact grid. Is it possible to add an assert to s2i op that shard grid is exact? If that doesn't break tests, I think it's better. You would need to update matmul to output exact grid in this case

@TT-BrianLiu the grid is not rectangle, I think if we want to make them exact , then there will be multple ranges, so matmul and s2i both needs to read multiple ranges.
It's also bad for user to use, as in height/width sharded case, use might just want to avoid the trouble specifying the exact grid?

@TT-BrianLiu the grid is not rectangle, I think if we want to make them exact , then there will be multple ranges, so matmul and s2i both needs to read multiple ranges. It's also bad for user to use, as in height/width sharded case, use might just want to avoid the trouble specifying the exact grid?

When ops iterate through the ranges, it's done with a helper which generates the cores so I think that shouldn't be an issue (maybe minor perf hit?). For users, we can still provide option to specify grid as essentially a boundary (which is what it is today), but we internally always store the exact grid. If we don't store exact grid, I have seen some use cases where it's confusing or error prone:

Two sharded tensors like this: [2, 1, 32, 64] and [10, 1, 32, 64] sharded onto "same" shard spec with shard shape [32, 64] and shard grid [8,8]. If we want to check if the two tensors are sharded onto the same cores, we compare the shard spec, which is equal but it shouldn't be if we stored the actual shard grid.
In cases like which you are fixing today, ops in general assume shard grid always contains data. With how it is today, they should actually always recompute actual shard grid, which is annoying.


geemap doesn't display attributions in my notebook

geemap version: 0.6.14
Python version: 3.6.9
Operating System: Linux

Description
Is it normal that geemap doesn't display attributions in my notebook? Is there a way to add them?
What I Did
I'm running :
import geemap

m = geemap.Map(center=[40,-100], zoom=4)
m.add_basemap('CartoDB.Positron')
m

And I get this map

Please update geemap from GitHub using the following command. geemap now adds attribution control by default.
pip install git+https://github.com/giswqs/geemap

thank you for your reactivity
It was impossible to go on production without that.
As I'm using custom controls for my map I've taken the liberty to look at your commit.
I will use m.add_control(AttributionControl(position='bottomright'))

Sure, the temporary solution would be to the add AttributionControl by yourself. Keep in mind that if you update geemap in the future, you might need to either remove this custom attribution control or set geemap.Map(show_attribution=False). Otherwise, you might get duplicate attribution controls.


Support video upload

I read this post on twitter blogs:
https://blog.twitter.com/2015/rest-api-now-supports-native-video-upload
And i would like know if now is possible post videos on twitter via tweepy?!

According to Twitter Dev docs, we need to use the https://dev.twitter.com/rest/reference/post/media/upload-chunked endpoint following the following implementation https://dev.twitter.com/rest/public/uploading-media#chunkedupload
Nothing in the code for now, hope this will coming quickly.

https://github.com/tweepy/tweepy/pull/655

I'd like to try to get #929 into Tweepy v3.9 or v4.0, depending on how quickly the review process can be completed.
cc @fitnr re:#1180

@Harmon758 #929 is a good commit, I've been using it in production for almost 2 years. Let me know if I can assist with the review in any way.

How can I gather tweet data using tweet just for one month's post from @Protestitas

@WaliyaYohannaJoseph That's not related to this issue at all.
See https://tweepy.readthedocs.io/en/latest/api.html#API.user_timeline and https://developer.twitter.com/en/docs/tweets/timelines/api-reference/get-statuses-user_timeline.

Hi, I get the same error

Invalid file type for image: video/mp4

I use version 3.8.0 of Tweepy
is there any solution?

Hi, I get the same error

Invalid file type for image: video/mp4

I use version 3.8.0 of Tweepy
is there any solution?

I can agree. I'm also receiving this error

Hey, I would just like to emphasize the desire to see PR #929 merged.
I would like to use it in an ongoing project and would appreciate any updates

v3.9.0 was released without this feature. I had intended to try to release it with v3.9.0.
I've since fixed and improved the changelog and release notes for v3.9.0.
Since v3.9.0 was also released without originally noting that it would be the last version to support Python 2 (#1253) in its changelog and release notes, a v3.10.0 is now planned for that and the goal is to include this in that release.
I've updated the milestones accordingly, but this might still be pushed back to v4.0, as I'd like to get started on dropping Python 2 support sooner rather than later.

Any ETA on when v4.0.0 is being released :)? Really looking forward to using this feature!

The goal is still to include this in v3.10.0, which will be released prior to and likely a lot sooner than v4.0.
You can check the milestones for both to see what still needs to be done for each version.
@fitnr #929 seems to be stale now. I wouldn't be adverse to a new updated PR from you that includes the original commits from #929 as well as the further improvements you made that haven't been included in it yet. If you're up for it, you'd also need to address the issues in my review of #929 though. Otherwise, I'll probably create a new branch and PR to do so myself (I'd still include the original commits and probably the additional improvements you made as well), but in that case, it'll be more likely that I drop Python 2 support for it and push it back to v4.0.

@Harmon758 I can work on a revised pull incorporating your notes.

What is the issue with this video upload thingy not being implemented for many years while such pulls already do exist?
Not trying to sound rude and I understand that everyone has a private life, but come on... 5 years, you gotta be kidding me

I'm pushing this back to v4.0.
See https://github.com/tweepy/tweepy/pull/1414#pullrequestreview-558777740 for further explanation.

The video-upload branch / pull request #1486 should be complete now.
Any feedback or review would be appreciated.

@Harmon758 Hi. Is now Video upload feature working? If yes then how to do it (or link to any such instructions?)
Thanks In Advance

#1486 has just been merged, so it should be working with the development version on the master branch.
For installation, see https://tweepy.readthedocs.io/en/latest/install.html and https://pip.pypa.io/en/stable/reference/pip_install/#git.


Extension breaks pasting in Google Drive Spreadsheets

When I have this extension enabled, I can't no longer Paste in Google Drive Spreadsheets. Neither Right-Click-Paste nor Menu Edit->Paste not Ctrl-V work.


Write a Rust program to find sum of n natural numbers

Description
Write a Rust program to find sum of n natural numbers
Input  : 5
Output : 15

How to contribute

Save the solution in find_sum_of_n_natural_numbers.rs file
Add find_sum_of_n_natural_numbers.rs file in find-sum-of-n-natural-numbers folder

!assign


new welcome email notification for new organizations

Fixes https://github.com/opencollective/opencollective/issues/1028

Coverage decreased (-0.1%) to 72.839% when pulling da7300e92f679c5b5a9db2bc67e4b6fe916cbb3d on fix1028 into d6b3161ee07a860d9b77db29c5355c1c9f017cc9 on master.


Autocomplete: Add feature flag to extend context language pool

Closes #3407
@valerybugakov: Is this an A/B test that you think we can/should run? 🙂
Test plan

No idea why the integration test is failing but look at the cool PR number wow. We came a long way folks!


PowerShell install script fails on Linux

PowerShell 7.2.5 on Ubuntu 20.04

Instructions should highlight where OS-specific steps are required
There's also a bug whereby install script looks under scripts/scripts rather than scripts; maybe that's because I executed from the scripts subdir, but Bash instructions suggest to run from that directory so it'd be better if instructions were consistent and if the PS script could handle it

Get-Content: /home/sean/code/Codex-CLI/scripts/powershell_setup.ps1:82
Line |
  82 |  (Get-Content -Path $plugInScriptPath) -replace "{{codex_query_path}}" …
     |   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Cannot find path '/home/sean/code/Codex-CLI/scripts/scripts/powershell_plugin.ps1' because it does not
     | exist.

This is also OS-specific and not handled properly. What's this path supposed to be?

PS /home/sean/code/Codex-CLI/scripts> # get current time UTC
>> # Please update the nl_cli_script path in the profile!
>>
>> notepad :/home/sean/.config/powershell/Microsoft.PowerShell_profile.ps1
>> # Please update the nl_cli_script path in the profile!
notepad:
Line |
   4 |  notepad :/home/sean/.config/powershell/Microsoft.PowerShell_profile.p …
     |  ~~~~~~~
     | The term 'notepad' is not recognized as a name of a cmdlet, function, script file, or executable program.

@scaleoutsean, thanks for the feedback. We'll take a look at PowerShell on Ubuntu.


Remove version: foxglove.studio version 1.17.0

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/foxglove/studio/releases/download/v1.17.0/foxglove-studio-1.17.0-win-arm64.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 720)


Rancher does not load images behind a proxy (Catalog issues)

With a simple nginx reverse proxy configured on port 80, the rancher management loads ok except for some images which do not load, for example in the Catalog or when choosing a Scheduler when you create an environment.
I think that the doc for running Rancher behind a proxy needs to be more detailed to address issues and correct usage of no_proxy directive.
Rancher Versions:
v1.5.3
Docker Version:
$ docker info
Containers: 1
 Running: 1
 Paused: 0
 Stopped: 0
Images: 1
Server Version: 1.12.6
Storage Driver: btrfs
 Build Version: Btrfs v4.4
 Library Version: 101
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: null bridge host overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: apparmor seccomp
Kernel Version: 4.4.0-64-generic
Operating System: Ubuntu 16.04.2 LTS
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 1.952 GiB
Name: rancher-server01
ID: xxxx
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Http Proxy: http://proxy.company.com:3128
Https Proxy: http://proxy.company.com:3128
No Proxy: localhost,<IP_ADDRESS>,.company.com,.company.local
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
Insecure Registries:
 <IP_ADDRESS>/8

Setup Details
Nginx config:
upstream rancher {
    server x.x.x.x:8080;
}
server {
	listen 80;

	root /var/www/html;

	server_name rancher.company.com;

	location / {
		proxy_pass http://rancher;
		proxy_set_header Host $host;
        	proxy_set_header X-Forwarded-Proto $scheme;
        	proxy_set_header X-Forwarded-Port $server_port;
        	proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
        	proxy_http_version 1.1;
        	proxy_set_header Upgrade $http_upgrade;
        	proxy_set_header Connection "upgrade";
        	# This allows the ability for the execute shell window to remain open for up to 15 minutes.
Log:
2017-03-29 04:10:20,313 ERROR [:] [] [] [] [qtp859417998-17] [i.g.i.g.r.handler.ExceptionHandler  ] Exception in API for request [http://rancher.company.com/v1/proxy/http://localhost:8088/v1-catalog/templates/library:infra*nfs]. Error id: [d71cd55c-666b-44c6-9300-754ef008bdaa]. org.apache.http.conn.HttpHostConnectException: Connect to localhost:8088 [localhost/<IP_ADDRESS>, localhost/0:0:0:0:0:0:0:1] failed: Connection refused

failed: Connection refused doesn't seem like it's a proxy issue.  Can you do a docker exec into the container and run
env -i curl http://localhost:8088

If all is swell you should get the below output
{"data":[{"actions":{},"id":"v1-catalog","links":{"self":"http://localhost:8088/v1-catalog"},"type":"apiVersion"}],"links":{"latest":"http://localhost:8088/v1-catalog","self":"http://localhost:8088/"},"resourceType":"apiVersion","type":"collection"}

I more suspect that catalog service isn't running.

Thanks @ibuildthecloud I confirm I get exactly the same output.
All images have the broken image sign but if I open the image in a new tab, it opens. I get all the errors in the log though.

I'd like to point out that it's not just the Catalog that doesn't load the images, but also when you choose a scheduler when adding a new host. Basically any image.

Even if I access Rancher UI directly on port 8080 over VPN, it fails to load half the images in the catalog.
Log:
2017-04-04 03:41:20,222 ERROR [:] [] [] [] [tp859417998-131] [i.g.i.g.r.handler.ExceptionHandler  ] Exception in API for request [http://rancher-internal:8080/v1/proxy/http://localhost:8088/v1-catalog/templates/community:infra*swarm]. Error id: [072d483d-86d1-456f-ac35-f95b1b253990]. org.apache.http.conn.HttpHostConnectException: Connect to localhost:8088 [localhost/<IP_ADDRESS>, localhost/0:0:0:0:0:0:0:1] failed: Connection refused

See attached photos:

Also when creating a new stack from the Catalog in K8s, no services are created:

I get a lot of connection refused in logs plus
time="2017-04-04T03:47:42Z" level=info msg="Starting Catalog Service (port 8088, refresh interval 300 seconds)"

The issue here is that each time you access the catalog-service it is just panicing and crashing, so not a proxy issue.  The reason it is dying is due to an odd CGO issue.  We just need to compile the catalog-service to no CGO (as done here https://github.com/rancher/catalog-service/pull/73)

Fixed in v1.5.5


add mini two-digit nixie tubes, data format change, bump version number

I love your mod and want to use it in a lot of small spaces around my factory. To that end, here's a patch that brings in a second kind of nixie tube, 1 tile instead of 2, with two small digits on it. I re-used your artwork, mostly just scaled down. I modified the nixie-map so that a nixie description can contain an arbitrary (equal) number of states and entities. I also added a migration script for the new recipe and data migration check  to handle the data format change.

Here's a screenshot:
https://i.imgur.com/Wdw49x0.png
(small and large nixies do not slave to each other)

Nice, merging. Did this by chance also fix issue #6 that you reported a couple of days earlier?

I am not sure what happened with #6. A few save/reload later I found myself able to remove the nixie tubes, and I applied entity.destroy() to the leftover vehicle entities from when the bug was happening.

hrm. Never ran into anything like that myself except during development, where it was a direct result of changes to the code between saving and loading, but I've not played any factorio in quite a while at this point. Lemme know if it crops up again.


Improve Logging Schema

The purpose of this issue is to improve our current logging schema in order to get better feedback on failing gmx commands.

Our current logging approach is handled primarily by the log_event function in folding/utils/logging.py file. The log_event function takes in an instance of the class it's being called from and an event dictionary. The function then prepares the event for logging by calling prepare_event_for_logging function from neurons/validator.py.

To improve logging

primarily focused on improving the visibility on gromacs simulation events, and making sure that we catch any exceptions and log accordingly.

(potential) Solutions

Make the run_cmd_commands function into a method of a larger running class, that is able to hold the state of the particular command.
Have a method that returns some type of dictionary with potential simulation error events
This provides flexibility such that we can add the information to the event dict that is being passed through something like event.update(...).
Partially getting inspo from the Store class

Deliverables

Pass commands
check the state of the running commands (success/fail). Some status market
Store the error(s) if there is one.
Take the class and turn into dictionary

The most complicated part of this will be pipelining, which includes how to use the class throughout the codebase


Cannot Install Because PowerShell Repeatedly Stops Working

Followed the installation steps and ensured prerequisites are in place. Run the PowerShell command 'Set-ExecutionPolicy -ExecutionPolicy Unrestricted -Force -Scope CurrentUser'. The old SF SDK and cluster have been uninstalled.
Used the Core SDK WebPI link. Always fails.
Unable to get Powershell ExecutionPolicy. ExitCode: -1073741819

Nearly another year has passed and the Web Platform Installer 5.1 still fails. Again, clean machine with Windows 10 Enterprise (18363.836).

This appears to be the same issue as
https://github.com/microsoft/service-fabric/issues/756
If you can ensure the powershell execution policy is set prior to running the installer, you can skip this step using the parameter /SkipPSExecutionPolicy.


Persistent Subscriptions/ Competing consumer

I'm looking to add this functionality. Are there any technical reasons why this hasn't been done? Or is it simply a missing feature?

No, there's no reason why it can't be added, and there are other people that want it too.
I simply haven't needed that feature, so I haven't added it.
Feel free to send in a pull request.
Thanks


Raise on ManifestFileNotFound

closes #7 - Missing file now raises error.
Raising the error looks like this:
iex(1)> raise Vite.ManifestReader.ManifestNotFoundError, manifest_file: "manifest.json"
** (Vite.ManifestReader.ManifestNotFoundError) Could not find static manifest at "manifest.json".
  Run "mix phx.digest" after building your static files
  or remove the configuration from "config/prod.exs".

Thanks!

Release here: https://hex.pm/packages/vite_phx/0.3.1


Priors, take 1

Let's start things off with the easy one, i.e. the mixing. Here is a working piece of code for the Gamma prior on the Dirichlet process, as well as a gtest. I figured I'd ask you if this is the right direction before going for the much wider change to the hierarchies. Last time I didn't ask for your approval it didn't go well for me... (jk)
The test uses a slightly roundabout way for checking due to the lack of a get_state() mehtod. This is because the Mixing class won't need it, at least as far as I can tell.
Feel free to suggest other names for the mixing proto members, I'm not 100% sold on these, even if they are as basic as it gets.
Speaking of renaming protos, I also added the bayesmix namespace for the Vector and Matrix protos, because reading lone Vectors and Matrices in the code was very confusing.

Good to go for me :) Merge at will


Code-Insiders hangs after paste shortcut or hovering "Edit" menu entry on macOS

VSCode Version:

Version: 1.39.0-insider
Commit: bdd1971f2cd9d08a8a90e02740f1850afe03da66
Date: 2019-09-17T05:59:55.803Z
Electron: 6.0.9
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0

OS Version:

Darwin x64 18.7.0

Steps to Reproduce:
Method 1:

Open Code-Insiders
Press ⌘ + v
Code Insiders hangs forever

Method 2:

Open Code-Insiders
Hover over the menu bar
When hovering over "Edit" entry in menu bar, Code Insiders hangs forever

Does this issue occur when all extensions are disabled?: Yes/No
Yes
Sometimes, restarting Code Insiders several times resolves the issue.
Happens since two days.

Two days ago I reported #80949 and the difference to this report seem to be the Node.js versions.

Can you please follow the steps in https://github.com/Microsoft/vscode/wiki/Native-Crash-Issues to get at more details around the crash and attach the result here? Thanks!

I couldn't reproduce since yesterday but I'll post it here as soon as it happens again.

Today it happened again.
Attached please find the log.
vscode-log.txt


chore(master): release 1.1.10

:robot: I have created a release beep boop
1.1.10 (2023-06-25)
Miscellaneous

deps: update dependency wayofdev/cs-fixer-config to v1.2.19 (f0bea44)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/laravel-symfony-serializer/releases/tag/v1.1.10 :sunflower:


React component props: mixing defaultProps and union types

TypeScript Version:  2.9.2

Search Terms: react component props union types defaultprops
Code
Using the new support for defaultProps in TypeScript 3.0 re. https://github.com/Microsoft/TypeScript/pull/24422
{
    type Props = { shared: string } & (
        | {
              type: 'foo';
              foo: string;
          }
        | {
              type: 'bar';
              bar: string;
          });
    class Component extends ReactComponent<Props> {
        static defaultProps = {
            shared: 'yay',
        };
    }

    <Component type="foo" />; // Expected error, got none
    // Unexpected error
    // Property 'foo' does not exist on type 'Defaultize<Props, { shared: string; }>'.
    <Component type="foo" foo="1" />;
}

https://github.com/Microsoft/TypeScript/pull/24422#issuecomment-403123876
/cc @mhegazy who requested that I open an issue to discuss/track this.

I'm assuming this is because the way "outer props" type of a component is calculated uses keyof somewhere along the line, and keyof only return the common properties in a union type? I've ran into this before when trying to use a withDefaults HOC that did something similar.

Here is a complete example that demonstrates the problem. I'm using functions here instead of components for simplicity. (Components are just functions.)
I think what we're missing is something like Diff (also known as Defaultize helper) that preserves the union properties. Is there a proposal somewhere for something like that?
// `Omit` and `Diff` copied from
// https://github.com/gcanti/typelevel-ts/blob/e1d718db887476f7b6b6e2c88542cc6ce3bd8265/src/index.ts#L9-L13
type Omit<A extends object, K extends string | number | symbol> = Pick<A, Exclude<keyof A, K>>;
type Diff<A extends object, K extends keyof A> = Omit<A, K> & Partial<Pick<A, K>>;

type Props = { shared: string } & (
    | {
          type: 'foo';
          foo: string;
      }
    | {
          type: 'bar';
          bar: string;
      });

const defaultProps = {
    shared: 'yay',
};
type DefaultProps = typeof defaultProps;

type InputProps = Diff<Props, keyof DefaultProps>;

const component = (props: Props) => {};

const componentWithDefaults = (inputProps: InputProps) => {
    /* Type '{ type: "foo" | "bar"; shared: string; }' is not assignable to type 'Props'.
        Type '{ type: "foo" | "bar"; shared: string; }' is not assignable to type '{ shared: string; } & { type: "bar"; bar: string; }'.
            Type '{ type: "foo" | "bar"; shared: string; }' is not assignable to type '{ type: "bar"; bar: string; }'.
            Property 'bar' is missing in type '{ type: "foo" | "bar"; shared: string; }'. */
    const props: Props = { ...defaultProps, ...inputProps }

    return component(props);
};

componentWithDefaults({
    type: 'foo',
    /* Argument of type '{ type: "foo"; foo: string; }' is not assignable to parameter of type 'Diff<Props, "shared">'.
        Object literal may only specify known properties, and 'foo' does not exist in type 'Diff<Props, "shared">'. */
    foo: 'foo',
})

Ah, this is possible to fix if we rewrite the Omit type to use conditional types, which distribute over unions:
type Omit<T, K extends keyof T> = T extends any ? Pick<T, Exclude<keyof T, K>> : never

https://github.com/Microsoft/TypeScript/issues/12215#issuecomment-378589263
Using this knowledge we can rewrite Defaultize to use conditional types:
type Defaultize<TProps, TDefaults> = TProps extends any
    ? { [K in Extract<keyof TProps, keyof TDefaults>]?: TProps[K] } &
          { [K in Exclude<keyof TProps, keyof TDefaults>]: TProps[K] } &
          Partial<TDefaults>
    : never;

Then we get the expected behaviour:
    <Component type="foo" />; // Expected error, got one :-)
    <Component type="foo" foo="1" />; // No error, as expected :-)

Fixed in the typings https://github.com/DefinitelyTyped/DefinitelyTyped/pull/27901/files#diff-96b72df8b13a8a590e4f160cbc51f40cR2292


HuggingFace Datasets support

Adding support for HuggingFace Datasets.
I add the --hf-datasets flag which indicates that the string passed to --train-set is a HuggingFace Dataset. I assume that there is a train split (as is most often the case), and the key for the images in the dataset is provided by --hf-datasets-key.
Hopefully this is fine, but let me know if you want me change this interface in any way...

I need to work on this, I want to add webdataset too and some special-cased datasets like MNIST and CIFAR-10, and I'm not sure how to support this level of flexibility yet...

What do you want me to do with this PR? Shall I close it and work on a more general dataset loader? We can brainstorm further on Discord.

I think I'm ready to look at HF Datasets again now that I have added a "dataset" section with a "type" key to the model config files. :)


qownnotes: 23.11.1 -> 23.12.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/pbek/QOwnNotes/releases.
meta.description for qownnotes is: Plain-text file notepad and todo-list manager with markdown support and Nextcloud/ownCloud integration
meta.homepage for qownnotes is: https://www.qownnotes.org/
meta.changelog for qownnotes is: https://www.qownnotes.org/changelog.html
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 23.12.3 with grep in /nix/store/a9h7rdxnivx93rkgvnm6hhiajjb0ffxl-qownnotes-23.12.3
found 23.12.3 in filename of file in /nix/store/a9h7rdxnivx93rkgvnm6hhiajjb0ffxl-qownnotes-23.12.3

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
qownnotes

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/a9h7rdxnivx93rkgvnm6hhiajjb0ffxl-qownnotes-23.12.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A qownnotes https://github.com/r-ryantm/nixpkgs/archive/592c90046e5e168480cef9a7b50cfcebad46b755.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/592c90046e5e168480cef9a7b50cfcebad46b755#qownnotes

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/a9h7rdxnivx93rkgvnm6hhiajjb0ffxl-qownnotes-23.12.3
ls -la /nix/store/a9h7rdxnivx93rkgvnm6hhiajjb0ffxl-qownnotes-23.12.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

qownnotes

Maintainer pings
cc @pbek @totoroot for testing.

LGTM


fix: update miniflare dependency to resolve HTTPS bug on Windows

Closes #3264
Closes #3218

[ ] Tests
[x] Changeset (Changeset guidelines)

Reviewer is to perform the following, as applicable:

Checked for inclusion of relevant tests
Checked for inclusion of a relevant changeset
Checked for creation of associated docs updates
Manually pulled down the changes and spot-tested

I applied this patch locally and can confirm it fixed #3264 for me!


MultiCellMarketSelection Not Plotting Summary

@JussanN @kesemadi
Hello,
I am using MultiCell function and I am not getting a summary output. This is not expected:

Secondly, I wanted to check if I can do an inverse Geolift by defining N=5, meaning I will use these as holdout (as suppose to treatment) and then the ATT will be flipped. Can I just flip the script or are there any implications?

Hi @titubs Please run Markets (object) in a new line after the function as explained in the walkthrough (link below) to see the summary of your results.
https://facebookincubator.github.io/GeoLift/docs/GettingStarted/MultiCellWalkthrough#2-finding-test-markets-with-multicellmarketselection
I'm closing this issue for now. Thank you.