Calling statisticsQuery for QuantityType.restingHeartRate is never returning (future never completes).

Describe the bug
Calling statisticsQuery for QuantityType.restingHeartRate is never returning (future never completes).
To Reproduce
Steps to reproduce the behavior:
final QuantityType quantityType = QuantityType.restingHeartRate;
final Predicate predicate = Predicate(dateStart, dateEnd);
final List<PreferredUnit> preferredUnit = await HealthKitReporter.preferredUnits([quantityType]);

print('BEFORE'); // Does print

final Statistics statistics = await HealthKitReporter.statisticsQuery(
  quantityType,
  preferredUnit.first.unit,
  predicate,
);

print('AFTER'); // Never prints

NOTE: To my knowledge, I don't have any restingHeartRate data in my Health Kit data.
Expected behavior
The future should complete, returning the data, and 'AFTER' would print.
Desktop (please complete the following information):

OS: macOS 12.5.1
Flutter version: Channel stable, 3.3.1

Smartphone (please complete the following information):

Device: iPhone8
OS: iOS 15.1 19B74

Additional context
To my knowledge, I don't have any restingHeartRate data in my Health Kit data (and there is no way I know of to add some manually).

Actually, I'm getting an error (was not seeing it earlier but surrounding in try catch does show it):
PlatformException(StatisticsQuery, Error in statisticsQuery, Optional(Error Domain=com.apple.healthkit Code=11 "No data available for the specified predicate." UserInfo={NSLocalizedDescription=No data available for the specified predicate.}), null)
The error is easy to understand, but I find it odd this function is erroring if there is no data, would it not make more sense if the returned object just contained nulls? It would be more consistent with what happens if you do have data and query statistics for a time period that does not contain any data.

I'll close this as it is not actually a bug, and open a feature request.


Copy original HTTP request headers, #175

This is a mild redesign of how client requests are forwarded in the .NET proxy. It provides a chance for request headers to be set through the new "copyRequestHeaders" function. This creates some flexibility for future modifications as well.
By copying headers from the client's request to the proxy's request, the proxy can now forward headers such as User-Agent, Cookie, Range, and so forth, which were formerly dropped by the proxy.
My personal need for this was to be able to properly handle PDF files. Clients such as Chrome will try to make partial range queries of PDF files if allowed by the server. Because we were copying response headers but not request headers, this created an issue where Chrome was told that it MAY make partial range requests, when in fact the requested range would get dropped. I confirmed that the .NET proxy is now able to handle partial byte ranges.
Handling partial content also meant that the proxy needs to be able to return a status code 206 (Partial Content). To allow for this, I modified the fetchAndPassBackToClient function to copy the server's response code as the proxy's response code.

thanks for this @jwoyame .  we're a bit backlogged, but will review and provide more feedback shortly.

This one has been fixed by #134

#134 passes response headers, however this change is meant to pass request headers as well.

@jgravois @bsvensson I'm having an issue where I'm using the proxy and the request headers are not being passed. I assume this PR resolves that - is there a reason it was never merged? Do we just need to resolve conflicts?

@gavinr - If you/someone could resolve the conflicts, we can review again.
One thing to keep in mind is that not all headers should be passed along - see #362 - or not passed along as-is - see #384.

Closing this PR, but it's changes lives on.  See PR #433.


Fix Buffer support in Mongoose

Information

Type
Breaking change

Fix
No

This closes #1811.
Since propEntity.isClass returns true for Buffer typed property, so it is being treated as Schema instead of directly getting assigned to type like String or Number. So I have updated the isClass method to return false for Buffer.
Todos

[x] Tests
[x] Coverage

I may be wrong. So would be helpful if someone with experience in the project gives their review.

It’s seems to be good. Really good PR @Devil7DK  ;)
I let the CI run all test. If all is ok, I’ll merge it!
thanks for your contribution!
Romakita

:tada: This PR is included in version 6.107.3 :tada:
The release is available on:

v6.107.3
GitHub release

Your semantic-release bot :package::rocket:


Perfect results for stage 2 but then poor results for stage 3

Hi, thank you for the great code base and contribution, so far it was a pleasure to work with it ;). Now my problem is that im trying to fit a 3D motion capture dataset using humor. I simply adapted the configs from the fit_amass_joints.cfg as my data is very similar to it. When I look at the results of stage 2 the fit looks perfectly but then stage 3 becomes very unstable and is very poor. Do you might have an idea what the overall problem might be? I already tried to work with different loss weights but so far I did not manage to fix this problem. It looks a little bit as if the fitting destabilizes over time and the falls apart. Here is an example: Left Original, Middle Stage 2, Right Stage 3

It look like your original data skeleton is a bit different than SMPL -- how are you applying the 3D joint loss in this case? Since the joints between the two skeletons are not aligned it may somehow be destabilizing things in the 3rd stage. You may want to try only applying the 3D loss on a subset of joints that are well aligned (e.g. root, elbows, hands, knees, toe end) and see if that works better.

Thank you for the advise I will try to find a suitable subset and see of that works. Until now I tried to approximate some of the points that I had no clear match for in my dataset.

Fitting only a few, better matching points, did improve the results. However I guess I still need to do some more fine tuning.


For OpenShift, the OpenShift kube-rbac-proxy image must be used

See https://github.com/operator-framework/operator-sdk/issues/4684
I experience the error:
 Error: container create failed: time="2021-10-13T02:11:50Z" level=error msg="container_linux.go:366: starting container process caused: chdir to cwd (\"/home/nonroot\") set in config.json failed: permission denied"

@vadimeisenbergibm Thank you for reporting.

@jakub-dzon please take a look

With v0.1.0 released, the image for OCP is registry.redhat.io/openshift4/ose-kube-rbac-proxy:v4.9.0 and for k8s is gcr.io/kubebuilder/kube-rbac-proxy:v0.11.0


[Friday 10am] [Week 11] 3: Release as a jar file

The latest JAR that you released in v1.3 will be used in the PE Dry run (PE-D), and the deadline to release the v1.3 JAR file is Friday 10am. This is a very important deadline. See Week 11 project tasks for more info.

https://nus-cs2103-ay2324s1.github.io/website/schedule/week11/project.html#3-release-as-a-jar-file
Ideally, we lock in our features before Friday.

We currently only have a v1.3b trial JAR, initially intended for the tutorial demo. This is not the one we want to use for PE dry run and have the feature freeze apply to. We will need to release another before the deadline, once we have finalised features.


Add Zero Dark Matrix to extensions

Adds "UI" category to Extensions section
Add Zero Dark Matrix DevTools theme link

This should just be under UX if it is included.

Yeah, I was debating that as well. Just moved it into UX since one other person thought so too :smile:

:boom: awesome @tonybaroneee thanks a million!

Cool, thank you!


Vxlan operator

Description
This PR introduces a k8s operator used to configure the vxlan interfaces on each node of the cluster. At the same time it updates the fdb entries of the vxlan device when new nodes are adder or removed.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.
Unit tests.

/merge


Can't run (json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0))

I am trying to get this to work but so far without luck. I found out that this requires Python3 and I've installed DEAR via pip. I've tried this on both a Raspberry pi and a Ubuntu Server.
This is the output of the Ubuntu Server:
$ python3 evolution.py  using candlestick dataset {'from':<PHONE_NUMBER>, 'to':<PHONE_NUMBER>} DEBUG {'MIN_ VALUES': {'thresholds': {'high': 45, 'fibonacci': 0.1, 'persistence': 1, 'up': -0.125, 'down': -1.25, 'low': 10}, 'short': 1, 'long': 10, 'interval': 0, 'signal': 5}} DEBUG {'MAX_ VALUES': {'thresholds': {'high': 95, 'fibonacci': 1.0, 'persistence': 5, 'up': 1.525, 'down': 0.4, 'low': 60}, 'short': 21, 'long': 43, 'interval': 33, 'signal': 15}} Loading new date range; 	{'from': '2016-04-24 22:12:45', 'to': '2016-04-27 22:12:45'} multiprocessing.pool.RemoteTraceback:  """ Traceback (most recent call last):   File "/usr/lib/python3.5/multiprocessing/pool.py", line 119, in worker     result = (True, func(*args, **kwds))   File "/usr/lib/python3.5/multiprocessing/pool.py", line 47, in starmapstar     return list(itertools.starmap(args[0], args[1]))   File "evolution.py", line 74, in Evaluate     Score = runBacktest(Settings, DateRange)   File "/home/nils/gekkoJaponicus/gekkoWrapper.py", line 91, in runBacktest     RESULT = REQ.json()   File "/usr/lib/python3/dist-packages/requests/models.py", line 808, in json     return complexjson.loads(self.text, **kwargs)   File "/usr/lib/python3.5/json/__init__.py", line 319, in loads     return _default_decoder.decode(s)   File "/usr/lib/python3.5/json/decoder.py", line 339, in decode     obj, end = self.raw_decode(s, idx=_w(s, 0).end())   File "/usr/lib/python3.5/json/decoder.py", line 357, in raw_decode     raise JSONDecodeError("Expecting value", s, err.value) from None json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0) """ The above exception was the direct cause of the following exception: Traceback (most recent call last):   File "evolution.py", line 179, in <module>     GA = gekko_generations()   File "evolution.py", line 140, in gekko_generations     fitnesses = parallel.starmap(toolbox.evaluate, to_simulation)   File "/usr/lib/python3.5/multiprocessing/pool.py", line 268, in starmap     return self._map_async(func, iterable, starmapstar, chunksize).get()   File "/usr/lib/python3.5/multiprocessing/pool.py", line 608, in get     raise self._value json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
And this is the output of the Pi:
python3 evolution.py  using candlestick dataset {'from':<PHONE_NUMBER>, 'to':<PHONE_NUMBER>} Loading new date range; 	{'from': '2017-01-25 17:47:26', 'to': '2017-01-28 17:47:26'} Traceback (most recent call last):   File "evolution.py", line 205, in <module>     GA = gekko_generations()   File "evolution.py", line 167, in gekko_generations     fitnesses = toolbox.map(toolbox.evaluate, individues_to_simulate)   File "evolution.py", line 20, in progrBarMap     result.append(funct(array[w]))   File "evolution.py", line 111, in Evaluate     Score = runBacktest(Settings, DateRange)   File "/home/nils/gekkoJaponicus/gekkoWrapper.py", line 91, in runBacktest     RESULT = REQ.json()   File "/usr/lib/python3/dist-packages/requests/models.py", line 808, in json     return complexjson.loads(self.text, **kwargs)   File "/usr/lib/python3.5/json/__init__.py", line 319, in loads     return _default_decoder.decode(s)   File "/usr/lib/python3.5/json/decoder.py", line 339, in decode     obj, end = self.raw_decode(s, idx=_w(s, 0).end())   File "/usr/lib/python3.5/json/decoder.py", line 357, in raw_decode     raise JSONDecodeError("Expecting value", s, err.value) from None json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
The cli which runs Gekko UI outputs the following:
/home/nils/gekko/plugins/paperTrader/paperTrader.js:14   this.fee = 1 - (calcConfig['fee' + calcConfig.feeUsing.charAt(0).toUpperCase() + calcConfig.feeUsing.slice(1)] + calcConfig.slippage) / 100; TypeError: Cannot read property 'charAt' of undefined     at new PaperTrader (/home/nils/gekko/plugins/paperTrader/paperTrader.js:14:57)     at load (/home/nils/gekko/core/pluginUtil.js:100:22)     at /home/nils/gekko/node_modules/async/dist/async.js:1156:9     at replenish (/home/nils/gekko/node_modules/async/dist/async.js:1030:17)     at iterateeCallback (/home/nils/gekko/node_modules/async/dist/async.js:1015:17)     at /home/nils/gekko/node_modules/async/dist/async.js:988:16     at /home/nils/gekko/node_modules/async/dist/async.js:1158:13     at load (/home/nils/gekko/core/pluginUtil.js:68:14)     at /home/nils/gekko/node_modules/async/dist/async.js:1156:9     at replenish (/home/nils/gekko/node_modules/async/dist/async.js:1030:17)   xxx POST /api/backtest 500 247ms -   Error: non-error thrown: Child process has died.       at Object.onerror (/home/nils/gekko/node_modules/koa/lib/context.js:105:40)       at process._tickCallback (internal/process/next_tick.js:109:7)

It had major bugs XD all fixed at last commits. It should work now.
Which python3 is this? I run it on python3.6; don't know about other versions.

I re-cloned the repo and installed Python3.6, still the same issues though.

Both errors are the same, related to response.json() method. Thats related to the communication to gekko api.
So the script now bypasses this method.. it may work now :]
If it still doesn't, could you print the variable Request.text here for me? It's on gekkoWrapper.py line 21.

I'm getting a similar error, my first Request.text is fine but the second one returns an Internal Server Error. As far as I know, there are no errors in my gekkoWrapper.py as far as paths and URLs go and my config.js is configured accordingly to the IP of the node server. My gekko server is not set to run headless. Using python 3.
I get the same json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
My gekko process then prints out the following:
  xxx POST /api/backtest 500 450ms -
  Error: non-error thrown: Child process has died.
      at Object.onerror (/Users/me/Documents/gekko/node_modules/koa/lib/context.js:105:40)
      at process._tickCallback (internal/process/next_tick.js:103:7)

@Teecolz I'm working on this now... is your dataset from Poloniex (and USD/BTC)?

@Gab0 Awesome, so far I've been using Poloniex BTC/ZEC (and I've changed gekkoWrapper.py accordingly).

I'm having this issue as well. Could it be realted to the use of talib indicators? Getting no output from gekko process.

Locale1
first unevaluated: 30
0 individues removed due to equality
[30]
Error: config failure
http://localhost:3000/api/backtest
{'gekkoConfig': {'debug': False, 'info': False, 'watch': {'exchange': 'poloniex', 'currency': 'USDT', 'asset': 'BTC'}, 'paperTrader': {'fee': 0.25, 'feeMaker': 0.15, 'feeTaker': 0.25, 'feeUsing': 'maker', 'slippage': 0.05, 'simulationBalance': {'asset': 1, 'currency': 100}, 'reportRoundtrips': True, 'enabled': True}, 'tradingAdvisor': {'enabled': True, 'method': 'custom', 'candleSize': 30, 'historySize': 10}, 'custom': {'bbparameters': {'optInMAType': 3.56, 'optInNbDevDn': 1.6400000000000001, 'optInNbDevUp': 1.16, 'optInTimePeriod': 2.76}, 'interval': 8.02, 'long': 23.16, 'short': 8.8, 'signal': 6.04, 'stop': 9.260000000000002, 'thresholds': {'down': -0.316, 'high': 73.2, 'low': 18.3, 'persistence': 3.0, 'up': 0.14800000000000002}}, 'backtest': {'daterange': {'from': '2017-10-08 07:30:08', 'to': '2017-12-07 07:30:08'}}, 'performanceAnalyzer': {'riskFreeReturn': 2, 'enabled': True}, 'valid': True}, 'data': {'candleProps': ['id', 'start', 'open', 'high', 'low', 'close', 'vwp', 'volume', 'trades'], 'indicatorResults': True, 'report': True, 'roundtrips': False, 'trades': True}}. Not sure why I am getting a config failure.


[firebase_ml_vision] (textRecognizer)

Describe the bug
when i using the  textRecognizer.processImage(visionImage)  ,
android results is fine  but ios result is null, i don't receive any recognized string
Sample Code :
 File image = await ImagePicker.pickImage(source: ImageSource.gallery);
  visionImage = FirebaseVisionImage.fromFile(image);
  visionText = await textRecognizer.processImage(visionImage);
  print(visionText.text); // this line works on android platform bu ios platform dont return any string

** pubspec.yaml
firebase_ml_vision: ^0.9.2+2
**Podfile
target 'Runner' do
  system('rm -rf .symlinks')
  system('mkdir -p .symlinks/plugins')
  use_frameworks!

pod 'Firebase/Core'
pod 'Firebase/MLVisionTextModel'
pod 'Firebase/MLVisionBarcodeModel'
pod 'Firebase/MLVisionFaceModel'
pod 'Firebase/MLVisionLabelModel'
end

Hi @AtillaPehlivan
can you please provide your updated  flutter doctor -v
and your flutter run --verbose/flutter build --verbose ?
Also, to better address the issue, would be helpful
if you could post a self contained app on github
or the steps to reproduce it.
Thank you

hi @iapicca  thanks for your interest
i couldn't FirebaseVision.instance.textRecognizer().processImage(visionImage); code on ios
but i am using the
FirebaseVision.instance.cloudTextRecognizer().processImage(visionImage);  code temporarily on my ios app , cloudTextRecognition run on two platform as expected
flutter doctor -v
[✓] Flutter (Channel stable, v1.9.1+hotfix.6, on Mac OS X 10.14.6 18G103, locale en-TR)
    • Flutter version 1.9.1+hotfix.6 at /Users/atilapehlivan/Desktop/flutter1.5/flutter
    • Framework revision 68587a0916 (6 weeks ago), 2019-09-13 19:46:58 -0700
    • Engine revision b863200c37
    • Dart version 2.5.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.0)
    • Android SDK at /Users/atilapehlivan/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.0
    • Java binary at: /Users/atilapehlivan/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/191.5900203/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 10.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.2.1, Build version 10E1001
    • CocoaPods version 1.7.0

[✓] Android Studio (version 3.5)
    • Android Studio at /Users/atilapehlivan/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/191.5900203/Android
      Studio.app/Contents
    • Flutter plugin version 40.2.2
    • Dart plugin version 191.8580
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[!] Connected device
    ! No devices available

! Doctor found issues in 1 category.

flutter run --verbose
this command don't give any error , my app launched  as expected
[        ] Updating files
[ +227 ms] DevFS: Sync finished
[   +1 ms] Syncing files to device Android SDK built for x86... (completed in 2,477ms, longer than expected)
[        ] Synced 1.5MB.
[   +1 ms] Sending to VM service: _flutter.listViews({})
[   +2 ms] Result: {type: FlutterViewList, views: [{type: FlutterView, id: _flutterView/0xa178c510, isolate: {type: @Isolate, fixedId: true,
id: isolates/1273464644891715, name: main.dart$main-1273464644891715, number:<PHONE_NUMBER>891715}}]}
[        ] <- accept
[        ] Connected to _flutterView/0xa178c510.
[   +2 ms] 🔥  To hot reload changes while running, press "r". To hot restart (and rebuild state), press "R".
[   +1 ms] An Observatory debugger and profiler on Android SDK built for x86 is available at: http://<IP_ADDRESS>:51009/hUnKNFCzECo=/
[        ] For a more detailed help message, press "h". To detach, press "d"; to quit, press "q".

HI @AtillaPehlivan
I'm glad to see that you solved the issue,
thank you for letting me know

I am also facing same issue is it iOS 13 issue?

Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of 'flutter doctor -v'  please.
All system setups can be slightly different so it's always better to open new issues and reference related issues.

Hi,
I am also facing the same issue as @AtillaPehlivan .
[✓] Flutter (Channel stable, v1.17.0, on Mac OS X 10.15.4 19E287, locale en-SG)
• Flutter version 1.17.0 at /Users/hoongjun/Developer/flutter
• Framework revision e6b34c2b5c (2 weeks ago), 2020-05-02 11:39:18 -0700
• Engine revision 540786dd51
• Dart version 2.8.1
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
• Android SDK at /Users/hoongjun/Library/Android/sdk
• Platform android-29, build-tools 29.0.3
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.4.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.4.1, Build version 11E503a
• CocoaPods version 1.9.0
[✓] Android Studio (version 3.6)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 44.0.2
• Dart plugin version 192.7761
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
[!] IntelliJ IDEA Community Edition (version 2017.1.2)
• IntelliJ at /Applications/IntelliJ IDEA CE.app
✗ Flutter plugin not installed; this adds Flutter specific functionality.
✗ Dart plugin not installed; this adds Dart specific functionality.
• For information about installing plugins, see
https://flutter.dev/intellij-setup/#installing-the-plugins
[!] VS Code (version 1.44.2)
• VS Code at /Users/hoongjun/Applications/Visual Studio Code.app/Contents
✗ Flutter extension not installed; install from
https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter
[✓] Connected device (1 available)
• iPhone                    • 00008020-001D44A611EA002E            • ios         • iOS 13.4.1
It is quite easy to replicate. As mentioned by @AtillaPehlivan , this code return empty.

FirebaseVision.instance.textRecognizer().processImage(visionImage);


--list on a workspace should show if its published

This is already implemented, a (restricted) output will be appended if it wasn't published


Fix to properly assign schemas to content of all media types of all r…

It is an error to clean temp schema storage after processing single "produces" item since it's per response. Thus moving it outside by cleaning it only if there are any "produces" items to accomodate for scenario when "produces" is specified after "paths"

Thanks for the contribution @VitaliyKurokhtin. I'll check this in.


ISSUES-863 support predicate push down

#863
I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

@KochetovNicolai @alexey-milovidov Can you give me some advice about this issue.

I don't know how to write test for this Pull Request

Basic idea of implementation:
Look at expression in WHERE and PREWHERE.
(This should be done after expansion of aliases.)
It can be an AND of multiple elements or not. In latter case it is considered as single element.
In first case, you should consider cases when you have single AND function
and(a, b, c...) or a chain of nested functions: and(a, and(b, c))...
Then you should look at each of elements in condition (a, b, c).
For each element, you should check, that it is dependent only on columns from a subquery and constants. If that's true, this element is considered for push-down.
Then look at what kind of subquery do you have.
It can be single SELECT or UNION ALL.
For each SELECT we should decide, if the condition can be pushed-down.
If SELECT contains LIMIT or LIMIT BY or WITH TOTALS or FINAL, the condition cannot be pushed-down.
If SELECT contains GROUP BY, and condition is depend only on aggregation keys, the consition can be pushed to WHERE. Otherwise, the condition can be pushed to HAVING.
If you have multiple SELECTs with UNION ALL, there are multiple cases:
When condition cannot be pushed at all.
When condition can be pushed to all of SELECTs.
When condition can be pushed to some SELECTs but cannot be pushed to some other SELECTs.
In second case, you should push condition to all SELECTs and remove it from the outer query.
In third case, you should push conditions to some SELECTs but also leave it in outer query.
When adding conditions, take care to create new WHERE/HAVING expression if there was no such expressions before, or adding condition as an element of AND chain. When removing conditions, take care to remove WHERE/PREWHERE if all conditions was moved.
How to test:
Create a table of type MergeTree. Use settings force_primary_key and max_rows_to_read to check if the key was used. We expect, that when condition is pushed to subquery, the primary key will be used and number of rows will be limited.
Please take a look to other tests that use force_primary_key.

Sorry for the delay, I plan to continue this task after #2260

@zhang2014 don't be sorry, I'm very grateful your even working on it :)

@alexey-milovidov @KochetovNicolai @vavrusa Almost done, Could help review this?
BTW Sorry for the delay : )

TODO: verify that all test passed when the setting is enabled globally.

@alexey-milovidov There are some problems with the current implementation.

 DB::Exception: Received from localhost:9000, <IP_ADDRESS>. DB::Exception: Unknown identifier: concat(func.name, comb.name).


{CI} Hack solution to unblock CI first

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

add to S170


Downloads freeze randomly

Describe the bug
When running archive the downloads get stuck after some time. Currently downloading Boruto and it has froze a few times, downloaded 60 then froze, restarted downloaded 5 then froze, restarted downloaded 100 then froze. The last time I left it on all night where it got stuck to see maybe it'll panic out or continue downloading but it just stayed stuck at 94%.
To Reproduce
Steps / command to reproduce the behavior:
$ crunchy-cli --etp-rt "" archive --skip-existing --skip-specials -a ja-JP -a en-US -r best https://www.crunchyroll.com/series/GR75Q020Y/boruto-naruto-next-generations[S1E113-]

Expected behavior
When downloading takes to long IE gets frozen or stuck would be nice to have it error out that way I can have a cron job rerun it where it last failed using the filter command: https://www.crunchyroll.com/series/GR75Q020Y/boruto-naruto-next-generations[S1Exxx-]
Screenshots

Client (please complete the following information):

OS: Ubuntu
Version 3.2.4

Currently I have a Go script that I use to:

read in yaml of shows I want to download
exec the crunchy-cli for each show
update and rename shows/episodes and move files for plex usage
update yaml with current episode that I was able to download
I noticed my script would just freeze up on the download so thought it was my doing, tested running manually the command above and the same issue happens.

What would be nice to have is if it freezes up the download would retry or abort that way in my code it'll error out and I can handle that accordingly.
Right now I have to keep watch and once it freezes restart the script.

One thing I noticed is when it does get stuck it's always at 94%

I (re-)added a timeout for the download process in v3.2.5. If this issue is caused by a connection which gets never closes, it timeouts after 60 seconds and the program tries to re-request the timeouted url.
If this issue still exists for you in v3.2.5 (or later), please comment here


Add Laravel 10 support

This PR aims to add support for Laravel 10, which is expected to be released at February 7th, 2023.
Open problem: the update to Laravel 10 comes with an update to Monolog v3, which changes the signature on the AbstractProcessingHandler::write method. This can be circumvented on PHP ^8.0 by using a union type in the class extension, but this does not work on PHP 7.x as it has no union types.
Possible solutions:

drop support for older Laravel versions
drop support for PHP 7.x: see https://github.com/jnoordsij/sentry-laravel/pull/2
find some sort of polyfill to allow for union types on PHP 7.x

You can have a look how this was solved in the PHP SDK here https://github.com/getsentry/sentry-php/blob/master/src/Monolog/CompatibilityProcessingHandlerTrait.php

Also note that in https://github.com/jnoordsij/sentry-laravel/pull/2 I also added some package combos with Laravel 6/7 and PHP 8.0 that seem to be working; you may want to add those to the tests as well if you intend to support those.

This looks good to me so far, we'll have to see if we need to make any more changes on an actual 10.x install a little later down the line. The only actual thing missing is a changelog entry 👍
I don't think we need to drop support for anything, we try to support as much as possible without it being a total drag, but with the monolog thing solved I think we can leave the support as it is.

Thanks for reviewing! I think the Laravel 10 API should be more or less stable from here, as I've seen them adding and merging Laravel 10 support to many of their self-maintained packages. But I can't really find anything promising this, so I understand if you want to hold this back for a little longer.
Alternatively, is it possible to merge this into a (new) feature branch on this repo, e.g. feature/l10, like some other packages are doing? This allows people (and other packages) to add a dev-branchname dependency in their composer.json to test and prepare for the new release more easily.

Thanks again for working on this 💪


Fixed #117 #88

In api/processor/*, for every success we are clearing the respective tables of database and adding new values.So, for every onRefresh(),its will clear the tables only when we have completed fetching that data else retain it.

Please squash. Always!!

@mananwason  Squashed it.Please review it.

@mananwason  is there any other issues with the code?
Thank you.

No @opticod Merging.


Update free-programming-books-pt_BR.md

What does this PR do?
Add Resource(s)
For resources
Description
Add Go's hands on on Github :octocat:
Why is this valuable (or not)
How do we know it's really free?
For book lists, is it a book?
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

@eshellman :smile:


[WASM] Blank space has the link associated with it

Steps To Reproduce:

Visit WASM Calculator.
Observer the top black bar and click on anywhere in the blank space.

Actual behavior:

By clicking on the blank space opens the Windows Calculator App link.

Expected behavior:

User can open a link by clicking on the LinkText (Windows Calculator App). It does not open a link if user click somewhere else other than linktext.

Screenshot:
Blank_Space_Link_Video.zip
Device and Application Information:

OS Build: Windows 10
WASM Build Version: Version 0.1.0-uno.1+467
Environment: Firefox, Chrome and Microsoft Edge

Notes:

See the attached video to get better idea.

Checked with the latest build and it's no longer occurred.
Version: 0.1.0-uno.1+493


[Bug Report]: Embedded LibreOffice Calc spreadsheet in slide doesn't pick up templating fills

Environment
Carbone Version: Carbone 2.1.1
Node Version: Node 10.23.2
Desktop OS: Ubuntu 18.04
Other: LibreOffice: <IP_ADDRESS>
Expected Behaviour
Expect a table copied from a spreadsheet into another document (in this case a LibreOffice Impress slide) to work properly.
Actual Behaviour
Mustache template parameters in the table don't get filled.
Steps to reproduce

Open LibreOffice Calc, fill a cell with a mustache template
Copy the cell
In Impress -- Paste Special - choosing the LibreOffice Spreadsheet source
Attempt to replace the template parameter
Observe that replacement does not occur.

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

const fs = require('fs');
const carbone = require('carbone');

var data =
{
    title: "A title",
    fill_this: "Please work",
};

carbone.render('./CopiedCell.odp', data, function(err, result) {
    if (err) return console.log(err);
    fs.writeFileSync('FailedTemplateReplace.odp', result);
    process.exit();
});

Reproduction Link / Code example
I've included instructions above and sample files below.
EmbeddedCell.zip

Hello @RobotCaleb, thank you for your feedback!
When a spreadsheet element (ODS) is pasted in a presentation file (ODP), it is creating a vector image (SVM file). Carbone can't inject data in an image. If you want to insert markers in a presentation template, you should create text boxes.

Cool! Thanks for informing me, I had no idea.


JWK Set parsing fails if provider's set contains non-RSA keys

Our OIDC Provider supports multiple JWS algorithms for signing access tokens and ID tokens, and therefore its jwks_uri provides both RSA and EC keys. For example:
{
  "keys": [
    {
      "kty": "RSA",
      "e": "AQAB",
      "use": "sig",
      "kid": "B9ch",
      "n": "..."
    },
    {
      "kty": "EC",
      "use": "sig",
      "crv": "P-256",
      "kid": "J3DJ",
      "x": "...",
      "y": "..."
    },
    {
      "kty": "EC",
      "use": "sig",
      "crv": "P-384",
      "kid": "uNgL",
      "x": "...",
      "y": "..."
    },
    {
      "kty": "EC",
      "use": "sig",
      "crv": "P-521",
      "kid": "YW0I",
      "x": "...",
      "y": "..."
    }
  ]
}

Unfortunately, Spring Security OAuth is unable to handle such JWK Set since JwkSetConverter#createJwkDefinition throws JwkException the moment it runs into a non-RSA key, regardless of the fact that JWK Set actually does contain an RSA key.
IMO the behavior of the JWK Set parsing should be changed to ignore the keys of unsupported types, until #1159 gets merged, which is currently targeted at the 2.3.0 release. This affects 2.0.x/2.1.x/2.2.x.
I could put together a PR to address this in time for 2.2.1 release if we agree on the general approach on the changes to be made.

@vpavic Updating to ignore unsupported keys instead of throwing works for me. If you could put together a PR for this that would be great.


Cannot read package.json

Lapce Version
0.2.0
System information
Windows 10
Describe the bug
Cannot read package.json file, content of the file not visible
To reproduce :

npm init
press enter for everything
open package.json on lapce (empty)
open package.json on other editor (contains infos)

Additional information

screenshots

cleaning the "empty" package.json file in lapce will still remove the content, so it seems the content of the file is here but not displayed properly.

after restarting lapce I can now see the content of the package.json
Yet It still was invisible when created on the terminal with npm init and look in the file tree without restarting lapce


[feat] .asyncMap a Future to a Signal

(I'm new to the whole idea of signals, forgive me)
Imagine a scenario where we make a network call and then do a db query using that data.
With signals is there a better way than this?
final idSignal = futureSignal(() async => 1); // First async operation
final idPlusOneSignal = computedFrom([idSignal], (i) {
  return i.first.map(
      data: (id) {
        return Future.value(AsyncData(id + 1)); // Second async operation
      },
      error: () async => i.first,
      loading: () async => i.first);
});
final Computed<int?>result = computed(() {
  final result = idPlusOneSignal.value.map(
    data: (d) => d.map(
      data: (d) => d,
      error: () => null,
      loading: () => null,
    ),
    error: () => null,
    loading: () => null,
  );
});

The biggest issue here is (besides how ugly it is) is that there will be 2 clicks of the clock before any data is returned from the result signal

It depends on your logic really, for me I do that in one futureSignal with no hassle.
and I prefer using futureSignals with GET requests and any submitted POST requests or basically a request that needs some parameters you can do it in a normal future function

It depends on your logic really, for me I do that in one futureSignal with no hassle.

Yeah, the use case is for when more then one signal depends on the 1st one.

I'm not following really, however
i have a case where i have a signal containing a userModel (currentUser)
and a stream signal called isLoggedIn
streamSignal((){
   return Stream.value(currentUser.value != null);
},
  dependencies: [currentUser]
);

so is that what you're referring to?
or what?

so is that what you're referring to?

Not at all.
Here is a better explanation.
If we are comparing signals to streams, then there is no way to do a Stream's mapAsync or Future's then on signal.
Instead you get AsyncValue<AsyncValue<AsyncValue<...>>> as you add more signals

Yup

Found a way, @rodydavis , would you be interested in a PR that adds this?
M
/// A Stream that emits an int every 250 milliseconds
Stream<int> getInt([i = 1]) async* {
  for (var i in List.generate(50, (index) => index)) {
    yield i;
    await Future.delayed(const Duration(milliseconds: 250));
  }
}

/// A signal that will listen to the stream
final intSignal = streamSignal(() => getInt());

// A signal which will perform an async action on each item in the signal
final intSignalPlusOne = futureSignal(() async {
  final q = await intStreamSignal.future;
  return await Future.value(q + 1); // Async Action
}, dependencies: [intStreamSignal]);

// A signal which will return a single `AsyncState<int>`
final result = computed(() {
  return intSignalPlusOne.value.map(
      data: (d) => intSignalPlusOne.value,
      error: () => intSignalPlusOne.value,
      loading: () =>
          intSignalPlusOne.previousValue ?? const AsyncLoading<int>());
});

void main() async {
  effect(() => print(result.value.value));
  await Future.delayed(const Duration(seconds: 1000));
}

Result:
null
1
2
3
4
5
6
7
8
9
10
11
12

I'm fairly new to this, so it's very likely that there is an underlying issue here.
Let me know if there is something evil going on under the hood


support figsize

Adding figsize support to core plotting functions - same approach as for pdvega.plotting functions.
also, moved warn_if_keywords_unused to the end of each function

hm, not sure I understand why it failing here

note - I did't change doc strings as I don't want to encourage people to use figsize over width/height

Looks good – ready for merge?

I think so!
On Sat, Aug 25, 2018 at 14:57 Jake Vanderplas<EMAIL_ADDRESS>wrote:

Looks good – ready for merge?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/altair-viz/pdvega/pull/35#issuecomment-415989581, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACjTpbiFcG42DUBVoeFfkxQ30xglGx3Yks5uUZ40gaJpZM4WMbWb
.

great, thanks!


[master] ISPN-5890 NPE after a node joins a cluster and receives new views

Jira
https://issues.jboss.org/browse/ISPN-5890
8.0.x branch PR
https://github.com/infinispan/infinispan/pull/3829

Retest this please. The 2 failed tests have success rate of 93% and 97%, so sounds like intermittent failures.

I think it's fine, @pruivo wanna have a look as well?

integrated! thanks @rhusar !


下载train_2gpu.sh 返回403Forbidden

根据Local Tutorial 执行wget https://easyrec.oss-cn-beijing.aliyuncs.com/scripts/train_2gpu.sh 返回403Forbidden

Here is what you want:
https://github.com/alibaba/EasyRec/blob/master/scripts/train_2gpu.sh
https://github.com/alibaba/EasyRec/blob/master/scripts/train_ps.sh


New tested device

Please add to the list of tested models this one: Vertex D514
CPU - sc6531e
LCD: idk, but all works fine, if it needs, i can attach a mark from lcd, resolution 240x320
Boot key - center button

The log shows the LCD model, here is from F+ F256:
BSL_REP_VER: "SPRD3\0"
BSL_REP_VER: "Custom FDL1: CHIP ID = 0x65620001\0"
!!! doom_init
malloc heap: 3719992 bytes
scan_firmware: 303ms
keymap = 0x10084b10
pinmap = 0x10080eec
chip_id: num = 0x65620001
chip_id: ver = 0xa0002
chip_id: ahb = 0x609
chip_id: adi = 0x1161a000
LCD: clk_rate = 104000000
LCD: id = 0x009306
!!! doom_main
DOOM Shareware Startup - v1.10

In the log 0x6562xxxx means SC6531E, SC6531DA (and compatible) is 0x6531xxxx.
I will update the table along with the update on other model.


Users shouldn't have to use the web UI to obtain a token when using the local registry

The script is now run automatically in run_scrc_server:
$ .scrc/scripts/run_scrc_server
nohup: redirecting stderr to stdout
Local registry now accepting connections on http://localhost:8000
An access token for the REST API is available in the file /home/andrewlahiff/.scrc/token


"Vibrations" around the goal

Hi,
I'm a mechanical engineering student and I'm trying to use MuJoCo for Reinforcement Learning.
As first attempt, I created a simple environment with a spherical body moved by a mocap and a target site the body has to reach.
To move the sphere exactly with the mocap I created a weld joint with solimp='0.998 0.999 0.0001 0.1 6' solref='0.0015 0.7' (close to a hard constraint). The reward for the RL is considered to be the negative of the distance between the sphere and the target and the action is continuous between -0.04 and 0.04 (action = mocap delta position ).
Applying RL, I observe that the sphere reach the target but then it starts moving back and forth around the target at maximum action.
Is it a problem related to how the mocap works ?

Sounds like maybe this is just the noise injected to the controller during RL? That's how RL works after all...

Sorry, I probably missed the point. I can understand the movement of the sphere around the goal during learning since the agent has to explore the environment. But I can't understand why I have this problem of "vibrations" when I apply the trained agent (using 2 million learning timesteps). In theory, the agent should learn that, once the target is reached, the null action is the best one.

Most RL frameworks let you switch to a deterministic policy (taking the action with maximum probability) once finished learning since, depending on configurations, the agent may never converge to a fully zero variance policy on its own. Could you see if you have such an option available to you?

I'm using PPO from Stablebaselines3, so I can switch to a deterministic policy when the policy predicts the action.However, the change doesn't provide any effect (same problem with the body moving back and forth at maximum action possible).
self.action_space = spaces.Box(-0.04, 0.04, shape=(self.n_actions,), dtype="float32"), so the agent can choose an action between -0.04 and 0.04, the distance from the target is 0.45 and the distance threshold is 0.03. I don't think that delta position is too high since it's one order of magnitude lower than the distance.
Finally, the observations are :
self.observation_space = spaces.Dict(
dict(
desired_goal=spaces.Box(-np.inf, np.inf, shape=obs["achieved_goal"].shape, dtype="float64"), # target position
achieved_goal=spaces.Box(-np.inf,np.inf, shape=obs["achieved_goal"].shape, dtype="float64"), # gripper position
observation = spaces.Box(-np.inf, np.inf, shape = obs["observation"].shape,dtype="float64"),
contact = spaces.Discrete(2),
)
)
Inside "observation" I put the position and the velocity of the body, in "desired_goal" the target pos and in "achieved_goal" the position of the body again.


MudDataGrid: Group Change Doesn't affect

Bug type
Component
Component name
MudDataGrid
What happened?
Can not control Groupable state with MudSwitch.
Expected behavior
Should gropable with external field value change.
Reproduction link
https://try.mudblazor.com/snippet/QuQduTvKpmvFnKPt
Reproduction steps

Click on group switch several times.

...
Relevant log output
No response
Version (bug)
6.2.2
Version (working)
No response
What browsers are you seeing the problem on?
Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

I ran into the same issue. Refreshing the UI with StateHasChanged(); does not resolve the issue either.


aws: fix usage of reloadable_features.use_http_client_to_fetch_aws_credentials

Commit Message: aws: fix usage of reloadable_features.use_http_client_to_fetch_aws_credentials
Additional Description: This feature flag will be release in v1.29. In v1.30 we will flip this feature flag to be default enabled. Finally in v1.31 release this feature flag will be removed.
Risk Level:
Testing:
Docs Changes: Yes
Release Notes: Yes
Fixes: https://github.com/envoyproxy/envoy/issues/31119

/assign @JuniorHsu for first pass?

/retest

@suniltheta is there a plan to flip this true or would it make more sense to remove the flag?

I will check with @nbaws if we can flip this to true for upcoming release. And remove it in a release following that.

I would prefer to keep this default false for one more release. https://github.com/envoyproxy/envoy/pull/34138 was a fairly substantial change to async providers, whilst I'm comfortable it has fixed the original reported issues with this PR the fix has not had sufficient burn time.


Curated feeds don't filter out deleted packages

It is unclear whether there is user impact since package deletes are rare and curated feeds don't have many packages.
The code should still filter out deleted.

A couple clarifying questions:
1 - These are SOFT-deleted packages that are shown, correct?
2 - There is no threat of these packages being downloaded, right? Their metadata is shown but they are not downloadable?

Correct on both counts.
Hard deleted packages cannot be seen, by definition. They are gone.
Soft deleted packages have their package content removed so the content URL in the V2 feed would simply 404.


Init with git is broken

Describe the bug
using fleek init with a github repo is broken
To Reproduce
Steps to reproduce the behavior:
fleek init git@github.com:lamarios/fleek.git
  ERROR
  ERROR   Error: unknown command "git@github.com:lamarios/fleek.git" for "fleek init"
  ERROR

Expected behavior
A clear and concise description of what you expected to happen.
fleek to initialize properly
Environment

OS:  fedora silverblue
Shell: zsh
fleek version: 0.9.0

Please try fleek join<EMAIL_ADDRESS>instead. I'm doing a release today and the docs are behind the app by a few hours.

it worked well, thanks !

Thanks for letting me know!


Does not work on Android L?

The drawable for the progress bar does not change. Stays the default from the theme.
I used the exact same example as in the README file and the progress bar remained with the default material color. I also tried to create a new progress bar but the result was the same.

So it works. I had just to put the drawable here:
progressBar.setIndeterminateDrawable(drawable);
because I wanted an indeterminate drawable. :)


GHAS bootcamp request

Bootcamp Date
2024-04-16
Attendees
sumitmalik51
Facilitators
cloudlabs-enterprise

Provisioning errored 👎
Additional information on this error is available in the Actions logs.


Fix build for 15.6

Fix #3398
I added the assemblies that are needed to build so that build doesn't break on previous versions, though it won't work very well (exception when you try to use IntelliSense). At least that will give us some time to update the build and test VMs.
I got the result from the build machine, and that didn't work (I can build locally on 15.4.5).

C:\Program Files   (x86)\Microsoft Visual   Studio\2017\Enterprise\MSBuild\Microsoft\VisualStudio\v15.0\VSSDK\Microsoft.VsSDK.targets(979):C:\Program   Files (x86)\Microsoft Visual   Studio\2017\Enterprise\MSBuild\Microsoft\VisualStudio\v15.0\VSSDK\Microsoft.VsSDK.targets(979,5):   Error : CreatePkgDef : error : ReflectionTypeLoadException: Unable to load   one or more of the requested types. Retrieve the LoaderExceptions property   for more information. at
System.Reflection.RuntimeModule.GetTypes(RuntimeModule module) at
System.Reflection.RuntimeModule.GetTypes() at
   System.Reflection.Assembly.GetTypes() at
Microsoft.VisualStudio.Tools.CreatePkgDef.ProcessAssembly(String fileName,   Hive hive, PkgDefContext context, Boolean register, RegistrationMode mode) in
E:\A\_work\2\s\src\product\vssdk\tools\CreatePkgDef\CreatePkgDef.cs:line 327   at
Microsoft.VisualStudio.Tools.CreatePkgDef.DoCreatePkgDef(InputArguments   inputArguments) in
E:\A\_work\2\s\src\product\vssdk\tools\CreatePkgDef\CreatePkgDef.cs:line 201   at
Microsoft.VisualStudio.Tools.CreatePkgDef.Main(String[] arguments) in   E:\A\_work\2\s\src\product\vssdk\tools\CreatePkgDef\CreatePkgDef.cs:line 93   Could not load file or assembly 'System.Collections.Immutable,   Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of   its dependencies. The system cannot find the file specified. Could not load   file or assembly 'System.Collections.Immutable, Version=<IP_ADDRESS>,   Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies.   The system cannot find the file specified.
--
Process   'msbuild.exe' exited with code '1'.

Ok, that error also happens locally if you delete the BuildOutput folder before building. On subsequent build (even rebuild), there's no error. Also when the error happens, the assembly it complains about (system.collections.immutable.dll v<IP_ADDRESS>) is present in the binaries folder. Not sure what's going on.

System.Collections.Immutable is a dependency that we only need because you copied the assembly into the references folder. Ideally we can avoid having to copy the assembly entirely, otherwise we'll need to bring in its dependencies too.

I submitted https://github.com/Microsoft/PTVS/pull/3410 which disables deprecation warning.


add multiple encodings support to echo

Faced the problem of output of cp1251 (yes, it is still used) in the output of the echo.
https://github.com/illuminate/view/blob/master/Compilers/Concerns/CompilesEchos.php
89 line, when outputting the escaped variables, the form {{$ var}} - the text disappears.
574 line, function e() - escape only utf-8.
https://github.com/illuminate/support/blob/master/helpers.php
Is it possible to add support for different encodings in the template engine?

Please ask your question in https://github.com/laravel/framework . This repository is automatically split from there and is not used for support.
Do not forget to close this issue.


Add removal of Defender for Server P2

Please add that despite enablement of Defender for Server P2 enabling at the workspace automatically, this isn't automatic when removing, and this will need to be removed to fully disable the feature. We don't reference this publicly anywhere.

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

ID: a8ecb7bd-4b50-d484-2027-3dd84ff8bd4b
Version Independent ID: f090852a-9b8d-d9f9-0737-6309ef6330fa
Content: Common questions - Defender for Servers - Microsoft Defender for Cloud
Content Source: articles/defender-for-cloud/faq-defender-for-servers.yml
Service: defender-for-cloud
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

@louispdalton
Thanks for your feedback! We will investigate and update as appropriate.

#reassign:alizabernstein

#label:"backlog-item-created"
https://dev.azure.com/msft-skilling/Content/_workitems/edit/149331

@vapasula
Could you please review it?

#reassign:ElazarK


Occlusion Culling falsely culls large objects visible through small openings when touching the edge of the viewport

Tested versions
Reproducible in 4.3-beta2 and 4.2.2-stable.
System information
Arch Linux w/ X11 and  Windows 10 using Radeon RX6800XT
Issue description
Occlusioin culling falsely culls a large object that is mostly obsured, but still visible through an opening in a wall in certain spatial configurations.
Here the red block is visible (expected):

And here it is culled (unexpected):

Video demonstrations in Liblast project:
https://youtu.be/xQW2PNNUggw (Godot 4.2.2-stable)
https://youtu.be/kfxirWBtZQ4 (Godot 4.3-beta2)
Steps to reproduce

Open Blender
Model a big block (like 20x20x8 meters)
Model a wall with an opening in the middle (like 30x10, with the opening like 1x2)
Import both models into Godot
Enable occlusion culling and bake occluders
Look at the big block through the opening in the wall, so that the wall edges aren't visible in the viewport, but the opening touches the left or right edge of the screen (or even a corner).
At a certain combination of camera position and rotation the big block will be culled while still very much visible through the opening in the wall.

Minimal reproduction project (MRP)
Run the game to see an animated demo.
MRP.zip

The problem is the AABB is incorrect for the red box. If I set a custom AABB it works.

Running the MRP locally, it seems like this is also a self-occlusion problem. The red and blue boxes are getting occluded by their own baked occluders. Setting the red and blue boxes to invisible and re-baking solves the problem as well.
So likely some sort of precision issue that makes it so baked occluders are improperly occluding the object they were baked from. The issue also goes away when removing the wall in the centre of the scene. So there is some interaction between the angle of view, the size of the gap, and the AABB of the source mesh

I have started to try and resolve this

I have started to try and resolve this

I'd be very happy to see your solution someday. Overculling is a problem that I feel that not enough people are trying to address in this engine.


[BEAM-6583] Update minimal Python 3 requirements.


Add to Stackage

Hello,
would you mind adding this package to stackage? I could do this for you, if you want.
Thank you,
Dominik

@dschrempf I will add this package to stackage when I have time. I also don't mind if you do this for me.

In this case I suggest I add it right away, you can always take ownership when you have time!


Better Data Connector Initialization Error Message Without Intermediate Layer

This is more about a design question: which intermediate layers we want to keep, which also apply to error messages other than Data Connector Initialization Error

https://github.com/spiceai/spiceai/issues/856

If you can point to the error directly without displaying the underlying source error.  We should do that without showing this proprietary errors from each data source because this will look more consistent.
If we can't, then we should have a simple statemnt pointing to the error in our runtime, and then display the source error after.
Something like:
Failed to connect to S3 datasource {name}.  Error from datasource: {S3's error returned to the client}


TTS is not working

I tried removing the extension and adding again still TTS is not working on chrome, what could be the problem

Please mention your OS, judge name(codechef, codeforces or atcoder) and Chrome version.
If you are on Linux, there might not be a TTS engine and you might have to install one.

I am on a mac, codeforces chrome version:70.0.3538.77 and TTS was fine for the past 10 days but suddenly its not working and now its not even showing points and time/memory just showing accepted notification.

Are you solving questions from problemset or contests page? Currently, it only supports contests

Oh, well i am solving problems from problemset, it would be great if you could make it work for problemset also.


⚠ Bofrost.at has degraded performance

In 42770a5, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1569 ms

Resolved: Bofrost.at performance has improved in 0b171cd.


docs fix: remove api_key arg from docstring of TLM

This is not a valid arg, think it got accidentally copied in the docstring when moving from studio -> trustworthy_language_model module

@jas2600 after you merge this, could you release new version of the cleanlab-studio client, so we can get this bugfix live on the docs?

@jwmueller the doc fix is live now


feat: update deprecated husky command

Updated deprecated husky install command to husky as per https://github.com/code100x/cms/issues/630.

@kmartens27 Would it be possible for you to give this a review?

You are welcome @alyssat!


Result is null when current location button is not clicked

Here is the code I am using:
    return MapLocationPicker(       apiKey: "key",       currentLatLng: location,       onNext: (GeocodingResult? result) {         if (result != null) {           print("result");           print(result);         }       },
If you see the below image, if I press the round dot icon then if I search for any location in search bar, then its showing the result in onNext. But if I dont press current location icon(round dot icon) and if I search any location in search bar, then its showing the address in the bottom input box but when I click next, onNext is getting result as null.

I think forcing the user to select current location is not correct. Instead, I want to show the current location automatically. For that, I am using below code:
  getLocation() async {     LocationPermission permission;     permission = await Geolocator.requestPermission();     Position position = await Geolocator.getCurrentPosition(         desiredAccuracy: LocationAccuracy.high);     setState(() {       lat = position.latitude;       lng = position.longitude;       location = LatLng(lat, lng);     });     setState(() {       _currentPosition = location;     });   }
This way, I am able to show current location when screen is opened, but when next is clicked, onNext is showing result as null when round dot icon is clicked.
Is there anyway I can auto select current location button or someway so result will not be null when onNext is clicked without user clicking current location button.
Thanks.

Hi @bhaskar8088 I just added this to the package map_location_picker: ^1.2.5

@rvndsngwn Thanks, that worked. But there is another issue.
After I search for any address and after pressing next arrow button, it is showing address(with lat, lng) of current location in the onNext function instead of showing the address of the newly searched location. Can you please fix this too? Thanks.

Hi @bhaskar8088 the has been resolved


Alternate File command broken when file creation required

I've encountered a problem with the bundle version based on the most recent commit implementing TM_RSPEC_BASEDIR. The problem is while using the Alternate File command - it generates an error when it tries to create the alternate file:
[...]/Library/Application Support/TextMate/Managed/Bundles/RSpec.tmbundle/Support/lib/rspec/mate/switch_command.rb:145:in `write_and_open': undefined local variable or method `base_dir' for #<RSpec::Mate::SwitchCommand:0x10bf2ab78> (NameError)
from [...]/Library/Application Support/TextMate/Managed/Bundles/RSpec.tmbundle/Support/lib/rspec/mate/switch_command.rb:21:in `go_to_twin'
from Alternate File:4

The command still works fine if the alternate file already exists.
The problem stems from some code in the switch_command.rb file, which attempts to use a helper method (base_dir) from helpers.rb, but this file is not required in switch_command.rb, meaning this method is not available.
I thought it would be a simple matter of just requiring the file, but:
All the other files that use the base_dir helper method simply require the helpers file using require_relative, but seeing as the Alternate File command uses the ruby18 shim, require_relative is not available.
I tried requiring the helpers file using alternate methods I found while googling that are supposed to work with Ruby 1.8, but I still got errors saying that the base_dir method is not defined (I started using Ruby after 1.8 was already obsolete, so my knowledge here is admittedly a bit thin).
So I guess if someone knows how to get this file properly required in a ruby1.8 way, so that the helper method is available, then this should fix the problem. But I'm afraid I hit a brick wall.

Good catch and thorough examination, thanks!
Seems like we have to stick to Ruby 1.8 for Commands run from TextMate (see http://lists.macromates.com/textmate-dev/2014-August/014993.html – note that the ruby20 has not yet been implemented, as far as I can tell).
A backward compatible replacement for require_relative would be require File.dirname(__FILE__) + "/foo.rb", I think.
Ccing @infininight.

Yeah, I saw that TextMate wasn't ready for Ruby2.0 yet either, which is a bit of a bummer, but what can you do...
I actually tried a few different options, and I couldn't get any of them to work. Not sure if it's something to do with my setup though. Here's what I tried:-
require File.expand_path(File.dirname(__FILE__) + '/helpers.rb')
# or
require File.dirname(__FILE__) + "/helpers.rb"
# or
$LOAD_PATH << File.dirname(__FILE__) # followed by...
require 'helpers'

All of these should have worked correctly :) But you also need to add include Helpers to make the method available :)

Oh haha, of course!
Kinda doesn't work without being included 😅
Thanks


From 1.0.106 rustc_codegen_cranelift's users are blocked!

From cc-v1.0.106 we, users of the amazing https://github.com/rust-lang/rustc_codegen_cranelift, cannot build anymore (at least on Windows): https://github.com/rust-lang/rustc_codegen_cranelift/issues/1510.
Can we found an alternative way?

Hmmm... in that case, maybe we can revert that changes (and the msrv)?

Yeah, thanks. Anyway I can alert you here when the fix will be there.

Reverted in #1157 would cut a new release shortly

Thank you very much!


Deleting job raises KeyError

Pressing the DEL key to remove a job from the "Running jobs" dialog can raise KeyError:
Traceback (most recent call last):
  File "vistrails/run.py", line 144, in <module>
    main()
  File "vistrails/run.py", line 138, in main
    v = app.exec_()
--- Exception caught here ---
  File "/home/remram/Documents/programming/dat/vistrails/vistrails/gui/job_monitor.py", line 267, in keyPressEvent
    parent.parent().jobMonitor.deleteJob(item.job.id)
  File "/home/remram/Documents/programming/dat/vistrails/vistrails/core/vistrail/job.py", line 442, in deleteJob
    self.callback().deleteJob(id)
  File "/home/remram/Documents/programming/dat/vistrails/vistrails/gui/job_monitor.py", line 346, in deleteJob
    del workflow_item.jobs[id]
KeyError: u'5cee0ba1a068a06f67e0982b7f5689baea4c3991'

This should be fixed by yesterdays commit 69d8da4114f3a0f3920e6fa75d95146337129527?

Looks that way!


feat: added animated gcp-icon.svg

Added Animated Google Cloud Provider Icon
Work Content
Please check if everything is correct

Thansk

Fixed Animated Google Cloud Provider Icon + Fixed Type Errors.
I never used Gatsby so please I apologize in advance.
Please check if everything is correct.

Thanks


it takes lot of refresh attempts to connect to DB

i have postgres DB in AWS. I use this client to connect to it in VScode. I need to click refresh multiple times before it can establish a connection to my AWS DB. Then the connection breaks after some inactivity, I need to then do the same refresh button click again. Is there a way to increase time it waits for connection establishment to succeed? Is there a keep alive so that it doesnt break the connection often.

Hi, you can modify the connection timeout. The default time is 5 seconds. Is your connection to the AWS database very slow?

Regarding keep alive, I will try to support it in the next version.


NPE with IntelliJ version 14.1.3

Git flow plugin : 0.4.2
Git: <IP_ADDRESS>.msysgit
IntelliJ version: 14.1.3
Nothing is shown on the status bar below and when I click on the small arrows, I get this exception:
Stacktrace:
java.lang.NullPointerException
at gitflow.ui.GitflowWidget.getPopupStep(GitflowWidget.java:106)
at com.intellij.openapi.wm.impl.status.IdeStatusBarImpl$MultipleTextValuesPresentationWrapper$1.onClick(IdeStatusBarImpl.java:705)
at com.intellij.ui.ClickListener$1.mouseReleased(ClickListener.java:73)
at java.awt.Component.processMouseEvent(Component.java:6525)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3324)
at java.awt.Component.processEvent(Component.java:6290)
at java.awt.Container.processEvent(Container.java:2234)
at java.awt.Component.dispatchEventImpl(Component.java:4881)
at java.awt.Container.dispatchEventImpl(Container.java:2292)
at java.awt.Component.dispatchEvent(Component.java:4703)
at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4898)
at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4533)
at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4462)
at java.awt.Container.dispatchEventImpl(Container.java:2278)
at java.awt.Window.dispatchEventImpl(Window.java:2750)
at java.awt.Component.dispatchEvent(Component.java:4703)
at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:751)
at java.awt.EventQueue.access$500(EventQueue.java:97)
at java.awt.EventQueue$3.run(EventQueue.java:702)
at java.awt.EventQueue$3.run(EventQueue.java:696)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:75)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:86)
at java.awt.EventQueue$4.run(EventQueue.java:724)
at java.awt.EventQueue$4.run(EventQueue.java:722)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:75)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:721)
at com.intellij.ide.IdeEventQueue.e(IdeEventQueue.java:734)
at com.intellij.ide.IdeEventQueue._dispatchEvent(IdeEventQueue.java:565)
at com.intellij.ide.IdeEventQueue.dispatchEvent(IdeEventQueue.java:382)
at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:201)
at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:116)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:105)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:93)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:82)

What small arrows are you referring to?
Does this happen on every project?

See the screenshot for the small arrows I'm referring to.
I tried on multiple projects and I have the same issue.
Thanks for your help!

Can you please paste the contents of .git/config within your project?
I have a feeling gitflow is not properly configured

This is the current content of the .git/config for one of my projects:
[core]
repositoryformatversion = 0
filemode = false
bare = false
logallrefupdates = true
symlinks = false
ignorecase = true
hideDotFiles = dotGitOnly
[remote "origin"]
url = http://vappgithub.data.intra/JavaTeam/IAM-Licensing.git
fetch = +refs/heads/:refs/remotes/origin/
[branch "master"]
remote = origin
merge = refs/heads/master
[gitflow "branch"]
master = master
develop = develop
[branch "develop"]
remote = origin
merge = refs/heads/develop
[gitflow "prefix"]
feature = feature/
release = release/
hotfix = hotfix/
support = support/
versiontag =
[gitflow "path"]
hooks = c:/dev/Projects/IAM-Licensing/.git/hooks
[gitflow "branch.hotfix/3.1.10"]
base = master
[gitflow "branch.feature/part66"]
base = develop
[branch "feature/part66"]
remote = origin
merge = refs/heads/feature/part66
[gitflow "branch.feature/documentation"]
base = develop
[gitflow "branch.feature/Java8"]
base = develop

I think it's the same issue of #71
Please check if the new version 0.4.3 solves this and let me know

It was working again on 0.4.2 the last days. I have difficulties to reproduce the behaviour.
I installed the version 0.4.3 and will check if the NPE happens again.
Thanks a lot


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/48756

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/49087

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/50402

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/50679

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/65505

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/66516

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/80850

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/85203

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/89373

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/98410

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/102333

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/104952

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/107077


Update package.json

NPM installing the example results in an unmet dependency error:
UNMET PEER DEPENDENCY<EMAIL_ADDRESS>└── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
Updating to 15.6.2 gives the same error.  updating to 16.0.0 resolves the error (though gives, npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.)
App runs fine.

Didn't follow contributing guidelines.  Going back through properly


Add TypedArray::copy_from

This introduces TypedArray::copy_from(src), as the dual of TypedArray::copy_to(dst).
While this can already be achieved today with TypedArray::set(), it requires either unsafe code or an additional allocation:
let x = [1, 2, 3];
let mut array = Int32Array::new(&3.into());

unsafe { array.set(Int32Array::view(&x), 0); } // view() is unsafe
// or
array.set(Int32Array::from(&x), 0); // from() allocates a new Uint8Array to copy the contents of x

The new helper method provides a safe interface to perform this copy:
let x = [1, 2, 3];
let mut array = Int32Array::new(&3.into());
array.copy_from(&x);

Internally, it uses the unsafe implementation listed above. We know that this is safe, because:

the TypedArray view on the Rust slice lives only for the duration of the copy_from() call
set() does not grow the WebAssembly linear memory, so it cannot invalidate the created view

Looks good to me, thanks! I think that the method should probably take &self instead of &mut self as well since it's all JS, but otherwise seems good to go.


Remove Area from Application Artifact Identifier

@einari Should it be removed entirely from the project, or just from the AAI components?

We can completely remove it from everywhere.
This means also in the .NET SDK configuration, which becomes a bit interesting - as there are the .Domain('') type of extensions - which should then go and we only need to be able to map namespaces in general. We know from the ArtifactType what type it is and can deduct the area it belongs to  from that


Does lighterHTML.html treat native element differently than custom elements?  What am I doing wrong here?

https://codesandbox.io/s/parcel-sandbox-11t5g

this.innerHTML returns text, and text is safe by default so that what you see is exactly expected.
You can either explicitly inject html
  render() {
    const section = html`

${this._heading}

       ${{html: this.innerHTML}}

    `;
    //section.innerHTML = this.innerHTML;
    return section;
  }

or you can store all nodes once in the constructor, and append these within the render
import { render, html } from "lighterhtml";

class SectionElement extends HTMLElement {
  constructor(heading) {
    super();
    const fr = this.ownerDocument.createDocumentFragment();
    while (this.hasChildNodes())
      fr.append(this.firstChild);
    this._heading = heading;
    this._body = fr;
    this.render = render.bind(
      // used as update callback context
      this,
      // used as target node
      // it could either be the node itself
      // or its shadow root, even a closed one
      this.attachShadow({ mode: "closed" }),
      // the update callback
      this.render
    );
    // first render
    this.render();
  }
  render() {
    const section = html`

${this._heading}

       ${this._body}

    `;
    //section.innerHTML = this.innerHTML;
    return section;
  }
}

export default SectionElement;


client_id is missing from request body

I have the following code:
public class AccessTokenManager {
    private PrivateKey privateKey;
    private X509Certificate publicCert;

    private static final Log log = LogFactory.getLog(AccessTokenManager.class);

    public AccessTokenManager(PrivateKey privateKey, X509Certificate publicCert) {
        this.privateKey = privateKey;
        this.publicCert = publicCert;
    }

    public String retrieveAccessToken(String adfsEndpoint, String clientId, String resourceId) throws Exception {
        log.info("Retrieving Authentication token");

        PublicClientApplication a = PublicClientApplication.builder(clientId)
                .authority(adfsEndpoint)
                .build();

        IClientCredential credential = ClientCredentialFactory.createFromCertificate(privateKey, publicCert);
        ConfidentialClientApplication app = ConfidentialClientApplication
                .builder(clientId, credential)
                .authority(adfsEndpoint)
                .sendX5c(false)
                .build();
        ClientCredentialParameters parameters = ClientCredentialParameters
                .builder(Set.of(String.format("%s/openid", resourceId)))
                .build();

        IAuthenticationResult result = app.acquireToken(parameters).join();

        log.info("Retrieved Authentication token");
        return result.accessToken();
    }
}

I tried to acquire token like these:
        AccessTokenManager m = new AccessTokenManager(privateKey, cert);
        String token = m.retrieveAccessToken("https://my.server.com/adfs", "my-client-id","http://myResource" );

The server's response is something like "client_id is missing".
I checked the actual httpRequest and the body is like this:
client_info=1&client_assertion_type=urn%3Aietf%3Aparams%3Aoauth%3Aclient-assertion-type%3Ajwt-bearer&grant_type=client_credentials&scope=http%3A%2F%2FmyResource%2Fopenid+openid+profile+offline_access&
client_assertion=eyJ4NXQ<---removed--->vxTwXfw

The client_id could not be found in the request body. However, according to Microsoft documentation from here: https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-oauth2-client-creds-grant-flow, the request body should contains client_id in the body like this:
  POST /{tenant}/oauth2/v2.0/token HTTP/1.1           //Line breaks for clarity
  Host: login.microsoftonline.com
  Content-Type: application/x-www-form-urlencoded

  client_id=535fb089-9ff3-47b6-9bfb-4f1264799865
  &scope=https%3A%2F%2Fgraph.microsoft.com%2F.default
  &client_secret=sampleCredentia1s
  &grant_type=client_credentials

I am wondering why client_id is missing from the request body?

@siddhijain The version is 1.11.1.

@yongheyan Please use the latest version 1.11.2. Let me know if that resolves the issue.

Hello @yongheyan. You're right about client_id missing from the request body and we will work on fixing this in one of the next releases. However, I was still able to get an access token using the code above. Can you please check?

@siddhijain: Our ADFS server still responses with an error message that says: client_id is missing.

Any update on this issue?

Hello, we are working on a fix and will update the thread when it is available.

This issue was addressed in https://github.com/AzureAD/microsoft-authentication-library-for-java/pull/489, and should be fixed as of the recent 1.12.0 release.
If the issue is still unresolved or if there are any related issues feel free to re-open this thread or open a new one.


[INTEGRATION][SPARK] Change event types for JobStart and JobEnd events

In Spark SQL, a single QueryExecution can invoke one or more Spark Jobs, initiating multiple SparkListenerJobStart and SparkListenerJobEnd events. Currently, we emit a START event for every SparkListenerSQLExecutionStart and for every SparkListenerJobStart event. Similarly, we emit a COMPLETE event for every SparkListenerSQLExecutionEnd and SparkListenerJobEnd event. This is confusing to the backend, as a single QueryExecution always emits multiple COMPLETE events and in the case where a single QueryExecution invokes multiple jobs, the backend will receive a START event after the first COMPLETE event.
I propose changing the SparkListenerJobStart event to trigger a STAGE_START event and the SparkListenerJobEnd event to trigger a STAGE_COMPLETE event. This makes it clear to the backend that only a portion of the job is complete and that the full job is not completed until the SparkListenerSQLExecutionEnd event triggers the final COMPLETE event.
Note that this change must only effect Spark SQL QueryExecutions - RDD-only jobs should continue to emit the START and COMPLETE events as there is no other context with which to bind various RDD jobs together.

Actually, I forgot the spec dictates an enum for the type here: https://github.com/OpenLineage/OpenLineage/blob/main/spec/OpenLineage.json#L11-L17 . We can change the event types to OTHER, but I kinda hate to lose visibility into the actual lifecycle events.
@julienledem , @wslulciuc , what do you think about updating the spec to be more versatile WRT to lifecycle state changes? Should these just be a new facet? Or can we take advantage of the existing fields?

@collado-mike do you have some follow-on thoughts on how to resolve this ambiguity on the backend side, given that #599 did not actually lead to any changes in the Spark plugin?


lxd init invoked as normal user fails to find lvm thin-provisioning tools

Required information

Distribution: Debian
Distribution version: sid
The output of "lxc info" or if that fails:

Kernel version: 6.1.8-1
LXC version: 5.0.2
LXD version: 5.0.2
Storage backend in use: lvm -- 2.03.16(2) (2022-05-18) / 1.02.185 (2022-05-18) / 4.47.0

Issue description
Originally reported as Debian bug #1027321; I've waited a while to see if the original reporter would confirm my workaround as described below, but that hasn't happened yet and I wanted to get the bug forwarded.
The original reporter ran into an issue performing a lxd init as a normal user who had already been added to the "lxd" group. They were trying to configure a lvm storage backend, but LXD wasn't able to find the thin-provisioning tools even though they were properly installed. It appears this is caused by the thin_check binary existing under /usr/sbin/, which isn't in a normal user's path by default.
I have reproduced the issue, and verified that lxd init works correctly if I explicitly add /usr/sbin/ to my path when running that command. The logic for this check (https://github.com/lxc/lxd/blob/master/lxd/main_init_interactive.go#L903) relies on os/exec, and I'm not sure if there's an easy way to inject /usr/sbin/ when LookPath() is called.
Steps to reproduce
Without modifying $PATH
gibmat@tharkun:~$ echo $PATH
/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games
gibmat@tharkun:~$ id
uid=1000(gibmat) gid=1000(gibmat) groups=1000(gibmat),24(cdrom),25(floppy),27(sudo),29(audio),30(dip),44(video),46(plugdev),100(users),106(netdev),112(bluetooth),114(lpadmin),117(scanner),124(lxd)
gibmat@tharkun:~$ lxd init
Would you like to use LXD clustering? (yes/no) [default=no]:
Do you want to configure a new storage pool? (yes/no) [default=yes]:
Name of the new storage pool [default=default]:
Name of the storage backend to use (dir, lvm, zfs) [default=zfs]: lvm
Create a new LVM pool? (yes/no) [default=yes]:
Would you like to use an existing empty block device (e.g. a disk or partition)? (yes/no) [default=no]:
Size in GiB of the new loop device (1GiB minimum) [default=30GiB]:

The LVM thin provisioning tools couldn't be found. LVM can still be used
without thin provisioning but this will disable over-provisioning,
increase the space requirements and creation time of images, containers
and snapshots.

If you wish to use thin provisioning, abort now, install the tools from
your Linux distribution and run "lxd init" again afterwards.

Do you want to continue without thin provisioning? (yes/no) [default=yes]: ^C

Modified $PATH
gibmat@tharkun:~$ PATH=$PATH:/usr/sbin lxd init
Would you like to use LXD clustering? (yes/no) [default=no]:
Do you want to configure a new storage pool? (yes/no) [default=yes]:
Name of the new storage pool [default=default]:
Name of the storage backend to use (dir, lvm, zfs) [default=zfs]: lvm
Create a new LVM pool? (yes/no) [default=yes]:
Would you like to use an existing empty block device (e.g. a disk or partition)? (yes/no) [default=no]:
Size in GiB of the new loop device (1GiB minimum) [default=30GiB]:
Would you like to connect to a MAAS server? (yes/no) [default=no]:
Would you like to create a new local network bridge? (yes/no) [default=yes]: no
Would you like to configure LXD to use an existing bridge or host interface? (yes/no) [default=no]: yes
Name of the existing bridge or host interface: lxcbr0
Would you like the LXD server to be available over the network? (yes/no) [default=no]:
Would you like stale cached images to be updated automatically? (yes/no) [default=yes]:
Would you like a YAML "lxd init" preseed to be printed? (yes/no) [default=no]:

Ignoring the current PATH doesn't really feel right in general and may also affect our ability to test or limit some of those commands by altering PATH to use alternative versions of the binaries being looked for.

A reasonable fix would be to tweak the error a tiny bit to hint that it wasn't found in the normal lookup PATH so the user may get the hint that they need to include it in their PATH or run as root.

Ive not looked into this in great detail yet, but I was thinking we could check the lvm thin tools as part of checking which storage pool drivers were available on the server and send a hint back to the lxd init client.

Yeah, that'd be an alternative, having an extra field in the DriverInfo which is then plumbed all the way to be stored in memory and returned in /1.0 output. It's definitely possible though it's honestly more work than I'm willing to put into this at this point ;)
Especially for a driver that's as unpopular as LVM :)