post update ibi

solves #676
In fact, this could also be used to have non-bonded interactions be updated with IBI.
Aditionally I have simplified how one can exclude interactions from imc. Before you had to provide an extra settings.xml file. Now you can just set the interaction.imc.group to 'none' and it will be excluded.
I prepare a csg-tutorial for propane, where this is used to do imc for nb and ibi for bonded.
One thing that is awkward is that this post-update overwrites potential updates from other methods, but I print a warning in that case.
I have not yet changed the iie interface to use this (see line 111 in update_iie.sh) but will do so later, since I work on iie atm anyway.

@votca-bot format

@votca-bot changelog: add ability to run ibi as a postupd method

@marvinbernhardt I am a bit confused that this update doesn't break the spce/hncgn
2021-06-23T17:13:08.8420439Z         Start 209: regression_spce_hncgn
2021-06-23T17:13:26.6095206Z 136/159 Test #209: regression_spce_hncgn ..............................   Passed   17.77 sec

Do we need a PR in csg-tutorials as well?  If so can you add the naphtalne examples as well?

@votca-bot copyright

@marvinbernhardt I am a bit confused that this update doesn't break the spce/hncgn
2021-06-23T17:13:08.8420439Z         Start 209: regression_spce_hncgn
2021-06-23T17:13:26.6095206Z 136/159 Test #209: regression_spce_hncgn ..............................   Passed   17.77 sec

Do we need a PR in csg-tutorials as well? If so can you add the naphtalne examples as well?

IIE and IBI still do IBI on all bonded interactions by default I belive. Question is if we want to keep that behavior or want the user to explicitly use post_update IBI. I would say it should only be implicit for IBI itself, other methods should use post_update.

@marvinbernhardt I am a bit confused that this update doesn't break the spce/hncgn
2021-06-23T17:13:08.8420439Z         Start 209: regression_spce_hncgn
2021-06-23T17:13:26.6095206Z 136/159 Test #209: regression_spce_hncgn ..............................   Passed   17.77 sec

Do we need a PR in csg-tutorials as well? If so can you add the naphtalne examples as well?

IIE and IBI still do IBI on all bonded interactions by default I belive. Question is if we want to keep that behavior or want the user to explicitly use post_update IBI. I would say it should only be implicit for IBI itself, other methods should use post_update.

Yeah that makes sense!

ping @marvinbernhardt

I want to test it again with the propane system from the csg_tutorials

I had to modify a perl script (update_ibi_pot.pl) to get rid of nasty artifacts in my angle potential. Now it looks much better, but the perl code is surely redundant. But I am not sure how to improve it.

I had to modify a perl script (update_ibi_pot.pl) to get rid of nasty artifacts in my angle potential. Now it looks much better, but the perl code is surely redundant. But I am not sure how to improve it.

What were the artifacts and what is redundant?

I had to modify a perl script (update_ibi_pot.pl) to get rid of nasty artifacts in my angle potential. Now it looks much better, but the perl code is surely redundant. But I am not sure how to improve it.

What were the artifacts and what is redundant?

Artifacts were building up on the right side of angle potentials.

There is redundancy in these two for loops. Unfortunately I'm not so good at Perl. https://github.com/votca/csg/blob/7bc89c062154848b22a6e48714dfc11c23a5a057/share/scripts/inverse/update_ibi_pot.pl#L76-L110

There is redundancy in these two for loops. Unfortunately I'm not so good at Perl.

I guess you could make it in a function, but don't bother if it is too complicated.

I think this could be merged now.


added mongo ktor backend for sampoorna

rn we are using my personal mongo server, will soon update with openlake's

@BudhirajaMadhav please have a look


Bumped versions of taggable

If this PR is merged we should bump the minor version on this library.
Related to: https://github.com/php-cache/predis-adapter/pull/7

Current coverage is 27.21%

Merging #6 into master will not affect coverage as of 8883a62

@@            master      #6   diff @@
======================================
  Files            2       2
  Stmts          147     147
  Branches         0       0
  Methods         21      21
======================================
  Hit             40      40
  Partial          0       0
  Missed         107     107

Review entire Coverage Diff as of 8883a62

Powered by Codecov. Updated on successful CI builds.


handgun_items call to engine crashes the game

Simply doing the following without any return value (void) or anything else causes a crash (no error):
host::functions.invoke_raw_unary(client::__sqf::unary__handgunitems__object__ret__array, unit_);

Used in the wrapper: https://github.com/intercept/intercept/blob/master/src/client/intercept/client/sqf/uncategorized.cpp#L10377

https://github.com/intercept/intercept/commit/3704c601f94710ab3103f1ac0b6f701e69dfa222#diff-e601d05fc5e8be313d378ddf06550f59R10377

Works fine in latest develop.
{silencer="muzzle_snds_acp" laser="" optics="optic_MRD" }


New version: Microsoft.OneDrive.Internal.Slow version 22.225.1026.0001

Result: Installation Failed

Manifest
Add/Remove Programs

Name
Microsoft OneDrive Internal Slow

Version
22.225.1026.0001

Publisher
Microsoft Corporation

ProductCode

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 3487
Microsoft Reviewers: Open in CodeFlow

@wingetbot waivers Add Validation-Installation-Error


chore: yet more unused variables

Thanks!  I combined the lines and will wait for CI to finish before merging!

bors merge


Date property is one day behind selected date

I created a Date field and selected March 2nd, resulting in the frontmatter showing this:

In source mode, the YAML still shows March 2nd.

Is this intended behaviour or a time zone issue or a bug?

I am having the same issue.


Cant attach images to songs

When using --embed-thumbnail I get this error when the image is in the same file as the mp3 and named "Glue-187193381.mp3.jpg".
Is this an error with ffmeg or what?
[soundcloud:user] playlist DailyBreed: Collected 18 video ids (downloading 18 of them)
[download] Downloading video 1 of 18
[download] Destination: Glue-187193381.mp3
[download] 100% of 1.54MiB in 00:00
[ffmpeg] Adding thumbnail to "Glue-187193381.mp3"
ERROR:   Reason: image not found
nicks-mac-mini:dailybreed nick$

Post the full output you get when running with --verbose option.

Sorry dude didn't expect you to be that fast, heres what I got and thanks in advanced.
nicks-mac-mini:dailybreed nick$ youtube-dl https://soundcloud.com/dailybreed --embed-thumbnail --verbose
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'https://soundcloud.com/dailybreed', u'--embed-thumbnail', u'--verbose']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.04.28
[debug] Python version 2.7.9 - Darwin-13.3.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg present, ffprobe present
[debug] Proxy map: {}
[soundcloud:user] playlist DailyBreed: Collected 18 video ids (downloading 18 of them)
[download] Downloading video 1 of 18
[debug] Invoking downloader on u'https://ec-media.sndcdn.com/likVlNiQCyOZ.128.mp3?f10880d39085a94a0418a7ef69b03d522cd6dfee9399eeb9a522029d6dfabd3435da9a4ec35d7f09e36f12218d8d754cbae7385afd30a812062d0a4dd50207a1edc2573921'
[download] Glue-187193381.mp3 has already been downloaded
[download] 100% of 1.54MiB
[ffmpeg] Adding thumbnail to "Glue-187193381.mp3"
[debug] ffmpeg command line: ffmpeg -y -i Glue-187193381.mp3 -i Glue-187193381.mp3.jpg -c copy -map 0 -map 1 -metadata:s:v 'title="Album cover"' -metadata:s:v 'comment="Cover (Front)"' Glue-187193381.temp.mp3
ERROR:   Reason: image not found
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1512, in post_process
files_to_delete, info = pp.run(info)
File "/usr/local/bin/youtube-dl/youtube_dl/postprocessor/embedthumbnail.py", line 45, in run
self.run_ffmpeg(filename, temp_filename, options)
File "/usr/local/bin/youtube-dl/youtube_dl/postprocessor/ffmpeg.py", line 152, in run_ffmpeg
self.run_ffmpeg_multiple_files([path], out_path, opts)
File "/usr/local/bin/youtube-dl/youtube_dl/postprocessor/ffmpeg.py", line 148, in run_ffmpeg_multiple_files
raise FFmpegPostProcessorError(msg)
FFmpegPostProcessorError

Try updating to the latest version.

Updated and got what looked to the the same error so I restarted terminal and tried again to re-update the session but still got this same error:
[download] Downloading video 1 of 18
[debug] Invoking downloader on u'https://ec-media.sndcdn.com/likVlNiQCyOZ.128.mp3?f10880d39085a94a0418a7ef69b03d522cd6dfee9399eeb9a522029d6dfab83d7d0030515862090539e4c53865c0919f3a9d00689eba48a2005487d4354741f7792b2a7ce4'
[download] Destination: Glue-187193381.mp3
[download] 100% of 1.54MiB in 00:00
[ffmpeg] Adding thumbnail to "Glue-187193381.mp3"
[debug] ffmpeg command line: ffmpeg -y -i Glue-187193381.mp3 -i Glue-187193381.mp3.jpg -c copy -map 0 -map 1 -metadata:s:v 'title="Album cover"' -metadata:s:v 'comment="Cover (Front)"' Glue-187193381.temp.mp3
ERROR:   Reason: image not found
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1529, in post_process
files_to_delete, info = pp.run(info)
File "/usr/local/bin/youtube-dl/youtube_dl/postprocessor/embedthumbnail.py", line 45, in run
self.run_ffmpeg(filename, temp_filename, options)
File "/usr/local/bin/youtube-dl/youtube_dl/postprocessor/ffmpeg.py", line 152, in run_ffmpeg
self.run_ffmpeg_multiple_files([path], out_path, opts)
File "/usr/local/bin/youtube-dl/youtube_dl/postprocessor/ffmpeg.py", line 148, in run_ffmpeg_multiple_files
raise FFmpegPostProcessorError(msg)
FFmpegPostProcessorError

Run with --verbose to ensure the version.

Looks like an error with the way home-brew installed ffmpeg wrong but I will run that now
nicks-mac-mini:~ noah$ ffmpeg
dyld: Library not loaded: /usr/local/lib/libx264.144.dylib
Referenced from: /usr/local/bin/ffmpeg
Reason: image not found
Trace/BPT trap: 5
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'https://soundcloud.com/dailybreed', u'--embed-thumbnail', u'--verbose']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.05.10
[debug] Python version 2.7.9 - Darwin-13.3.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg present, ffprobe present
[debug] Proxy map: {}

So, not a youtube-dl issue. Closing.

Thanks


MissingPluginException(No implementation found for method Query#observe on channel plugins.flutter.io/firebase_database)

trying to use the plugin in Flutter for web i am getting error
MissingPluginException(No implementation found for method Query#observe on channel plugins.flutter.io/firebase_database)
I am trying to read data from firebase in flutter for web . for ios and android firebase database read work well but for web giving error whille reading firebase database on db.once().then((

Hi @shinde-vrushali
From what I can see, the issue is related to a FlutterFire plugin rather than to Flutter itself. Please open the issue in the dedicated repository.
Closing, as this isn't an issue with Flutter itself.
If you disagree, please write in the comments,
providing your flutter doctor -v, your flutter run -v,  your pubspec.yaml
a minimal reproducible code sample that does not use 3rd party plugins,
and I will reopen it.
Thank you


Suggest adding darknet-19 backbone to Yolov3

Suggest adding darknet-19 backbone to Yolov3

Hi, sorry to inform: Since MMYOLO already supports different sizes of the models of YOLOv5&v6&v7, we will not provide more resources for yolov3. But PRs of YOLOv3 are still welcome.


VSTest Task Does Not Rerun Failed Tests Properly (GoogleTestAdapter)

Question, Bug, or Feature?
Type: Bug
Enter Task Name: VSTest@2
Environment

Server - Azure Pipelines

Account Name: Microsoft
Team Project Name: Analog
Build Definition Name: SpatialServices-CI-BuddyBuild (20190627.5)

Agent - Private: Windows Agent Version 2.153.2 (Package ES Lab E)

Issue Description
We are using the VSTest@2 task to run a suite of tests built with the Google Test framework. When running the task with the rerunFailedTests option set to true, the task is unable to find the failing test when attempting to rerun it. It attempts to find the failing test 3 times (rerunMaxAttempts) and then gives up. The task is marked as Succeeded even though it shows "Test Run Failed". The rerun shows the following issue No test matches the given testcase filter 3 times (full error below and in attached zip file).
Also, it should be noted that we are using GoogleTestAdapter version 0.16.1. The rerun options for the VSTest task are as follows:
rerunFailedTests: True
rerunMaxAttempts: '3'
rerunType: 'basedOnTestFailurePercentage'
rerunFailedThreshold: '75'
Please met know if there is any other information I can provide.
Thanks,
Tushar
Error logs
2019-06-27T23:32:50.6076049Z Total tests: 98. Passed: 93. Failed: 1. Skipped: 4.
2019-06-27T23:32:50.6076908Z ##[error]Test Run Failed.
2019-06-27T23:32:50.6077371Z Test execution time: 3.1360 Minutes
2019-06-27T23:32:50.6312770Z **************** Completed test execution *********************
2019-06-27T23:32:50.6316117Z Test results files: C:\BA_temp\TR_512310af-a64c-409d-a936-ef95de59f9e3\issblder_PKGES-LABE176_2019-06-27_23_29_46.trx
2019-06-27T23:32:51.4876094Z Created test run: 115859256
2019-06-27T23:32:51.4948978Z Publishing test results: 98
2019-06-27T23:32:51.4949449Z Publishing test results to test run '115859256'.
2019-06-27T23:32:51.4949680Z TestResults To Publish 98, Test run id:115859256
2019-06-27T23:32:51.4950077Z Test results publishing 98, remaining: 0. Test run id: 115859256
2019-06-27T23:32:51.8218387Z Published test results: 98
2019-06-27T23:32:51.8218582Z Publishing Attachments: 1
2019-06-27T23:32:53.1731967Z Failed tests: 1; Total tests: 98;
2019-06-27T23:32:53.3338707Z Total failed tests as return by TCM:1
2019-06-27T23:32:53.3379822Z Found 1 filtered test case sources.
2019-06-27T23:32:53.3519598Z **************** Rerunning failed tests for Test run 115859256 *********************
2019-06-27T23:32:53.3520146Z Max attempts: 3; Current attempt: 1; Failed test cases threshold: 75; Failed test cases max limit: 0
2019-06-27T23:32:53.3520652Z [command]C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\Common7\IDE\CommonExtensions\Microsoft\TestWindow......\Extensions\TestPlatform\vstest.console.exe "@C:\Users\issblder\AppData\Local\Temp\tmp4E3C.tmp"
2019-06-27T23:32:55.8157798Z Microsoft (R) Test Execution Command Line Tool Version 15.9.1
2019-06-27T23:32:55.8158016Z Copyright (c) Microsoft Corporation.  All rights reserved.
2019-06-27T23:32:55.8171248Z vstest.console.exe "C:\BA\20\s\SpatialServices\bin\x64\Release\SpatialServicesApiTests.exe"
2019-06-27T23:32:55.8173584Z /Settings:"C:\BA_temp\rezqdkiye3w.runsettings"
2019-06-27T23:32:55.8174728Z /Logger:"trx"
2019-06-27T23:32:55.8175207Z /TestAdapterPath:"C:\BA\20\s\SpatialServices\packages\GoogleTestAdapter.0.16.1"
2019-06-27T23:32:55.8175808Z /TestCaseFilter:"FullyQualifiedName=SpatialServicesApiTests.SpatialServicesRestClientWithServiceTests.CreateNeighborhoodAnchorAndSpatialAnchorBasic"
2019-06-27T23:32:56.1101929Z Starting test execution, please wait...
2019-06-27T23:32:58.0233518Z Google Test Adapter: Test execution starting...
2019-06-27T23:33:00.1165170Z Found 98 tests in executable C:\BA\20\s\SpatialServices\bin\x64\Release\SpatialServicesApiTests.exe
2019-06-27T23:33:00.1526477Z Google Test execution completed, overall duration: 00:00:02.1674060.
2019-06-27T23:33:00.3187265Z No test matches the given testcase filter FullyQualifiedName=SpatialServicesApiTests.SpatialServicesRestClientWithServiceTests.CreateNeighborhoodAnchorAndSpatialAnchorBasic in C:\BA\20\s\SpatialServices\bin\x64\Release\SpatialServicesApiTests.exe
2019-06-27T23:33:00.4482427Z Results File: C:\BA_temp\TR_512310af-a64c-409d-a936-ef95de59f9e3\issblder_PKGES-LABE176_2019-06-27_23_33_00.trx
log_120_16707792.zip

@tusharb86 Is the failing test same as the one being used during rerun?
FullyQualifiedName of failing test seems to be SpatialServicesApiTests.SpatialServicesRestClientWithServiceTests.CreateNeighborhoodAnchorAndSpatialAnchorBasic

@singhsarab The FullyQualifiedName is wrong. The beginning of the name ("SpatialServicesApiTests.") is actually the name of the executable... The real name of the test is just "SpatialServicesRestClientWithServiceTests.CreateNeighborhoodAnchorAndSpatialAnchorBasic"

@tusharb86 Interesting! that seems to be the issue here. Will have to debug to see what is the fullqualifiedname reported by the test adapter.
Can you try running locally which trx logger enabled and see what is the fullyqualifiedname in the trx file ?

@singhsarab What exactly should I be looking for in the TRX file? I don't see a field called "FullyQualifiedName" only "name". Here's an example from the TRX file for this test:
<UnitTest name="SpatialAnchorRestClient_UseRealService/SpatialServicesRestClientWithServiceTests.CreateNeighborhoodAnchorAndSpatialAnchorBasic/0 [true]" storage="g:\sdk1\spatialservices\bin\x64\release\spatialservicesapitests.exe" id="dd1ae5fb-8d91-c86e-a3c4-7e34f9af9a32">
You'll see that there is no SpatialServicesApiTests in the test's name itself, only in the binary executable's name.
(Note that we've changed our tests since I first opened this issue and the test now has a parameter which is why there is a /0 [true] here).

@tusharb86 You can actually list the fullyqualifiednames using the /ListFullyQualifiedTests argument

Thanks @singhsarab! I did not know about that flag. I tried running with that and I get the following error:
Target Path should be specified for listing FQDN tests!
When I run with the "/lt" or "/ListTests" flag, everything works as expected and all test names are printed. Since the "/ListFullyQualifiedTests" flag does not work, does this point to an issue in the GoogleTestAdapter?
Here are the arguments and the full log:
vstest.console /logger:trx /platform:x64 /testadapterpath:G:\SDK1\SpatialServices\packages\GoogleTestAdapter.0.16.1 SpatialServicesApiTests.exe /ListFullyQualifiedTests
Microsoft (R) Test Execution Command Line Tool Version 15.9.1
Copyright (c) Microsoft Corporation.  All rights reserved.

No settings file provided through env variable GTA_FALLBACK_SETTINGS
Warning: Using default settings.
Google Test Adapter: Test discovery starting...
Found 359 tests in executable G:\SDK1\SpatialServices\bin\x64\Release\SpatialServicesApiTests.exe
Test discovery completed, overall duration: 00:00:11.2028399
Target Path should be specified for listing FQDN tests!

You'd have to give another argument for a file path to dump all the names. So your commandline would now look like:
vstest.console /logger:trx /platform:x64 /testadapterpath:G:\SDK1\SpatialServices\packages\GoogleTestAdapter.0.16.1 SpatialServicesApiTests.exe /ListFullyQualifiedTests /ListTestsTargetPath:Testsfile.txt

Thanks! (As a side note, it would be great to add these command line options to the documentation for VSTest.Console.exe since they are not documented there)
I was able to list the fully qualified names for the tests and found that there is no SpatialServicesApiTests in the fully qualified test name either:
SpatialAnchorRestClient_UseRealService/SpatialServicesRestClientWithServiceTests.CreateNeighborhoodAnchorAndSpatialAnchorBasic/0
I was also able to run the individual test with the following command line options:
vstest.console /logger:trx /platform:x64 /testadapterpath:G:\SDK1\SpatialServices\packages\GoogleTestAdapter.0.16.1 SpatialServicesApiTests.exe /TestCaseFilter:"FullyQualifiedName=SpatialAnchorRestClient_UseRealService/SpatialServicesRestClientWithServiceTests.CreateNeighborhoodAnchorAndSpatialAnchorBasic/0"
Microsoft (R) Test Execution Command Line Tool Version 15.9.1
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...
No settings file provided through env variable GTA_FALLBACK_SETTINGS
Warning: Using default settings.
Google Test Adapter: Test execution starting...
Found 359 tests in executable G:\SDK1\SpatialServices\bin\x64\Release\SpatialServicesApiTests.exe
Running 1 tests...
Google Test execution completed, overall duration: 00:00:18.3061090.
Passed   SpatialAnchorRestClient_UseRealService/SpatialServicesRestClientWithServiceTests.CreateNeighborhoodAnchorAndSpatialAnchorBasic/0 [true]
Results File: G:\SDK1\SpatialServices\bin\x64\Release\TestResults\tushab_STUDIOMAGIC_2019-08-06_09_29_05.trx

Total tests: 1. Passed: 1. Failed: 0. Skipped: 0.
Test Run Successful.
Test execution time: 19.9784 Seconds

Please let me know if you need any other information on my end. It would be great if you are able to setup a repro as well.
Thanks,
Tushar

@tusharb86 this is an issue with XUnit as well.
Basically XUnit as well as Google Test Adapter report the FQDN wrongly in the trx. When we use the same name to run the test again using the test case filter it does not work.
One option is to try and get to change the way these adapters behave but that's not an easy task.
We do however have a workaround where we use the translation layer in the vstestplatform to pass around testcaseref objects rather than perform invocations via command line. This ensures the tests get discovered since we forward the full objects to the adapter and not one piece of information.
Take a look at the github issue below for the workaround. We do hope to make the translation layer the default mode of execution in the vstest task in the future.
https://github.com/microsoft/azure-pipelines-tasks/issues/10709

Closing this due to no response, Please re-open if problem persists after attempting the workaround.

@ShreyasRmsft Sorry for the delay. The workaround does not work, unfortunately :-( I tried this a long while back and saw that batching wasn't working at all. The logs for those builds have expired, so I've kicked off new builds and will update when I have them.
Google Test Adapter is an open source project that we can contribute to directly. Also, we should file an issue for that project if the issue is in the adapter to track this.

@tusharb86 could you also provide me with a minimal sample repro on github? I can fiddle around and get more insights on what exactly is causing the mismatch. Thanks.

Thanks for reopening the issue @ShreyasRmsft! Unfortunately, I don't think I'll have time to come up with a minimal repro on GitHub for at least a week or so. Below are logs where I attempted to use the workaround suggested. Both attempts at batching tests produced similar results.
distributionBatchType: 'basedOnTestCases'
batchingBasedOnAgentsOption: 'customBatchSize'
customBatchSizeValue: '100'

And
distributionBatchType: 'basedOnAssembly'

Results:
IsValidServiceResponse: Received None command..Service Workflow is not active
Creating run for selected test assemblies with following parameters
SourceFilter: ThisWontBeUsed TestCaseFilter:
Run title: Google *Tests.exe (with service) (non-gating) - Windows Release-x64
Build location: C:\BA\2584\s\SpatialServices\bin\x64\Release
Build Id: 17601396
Test run with Id 117123290 associated
Received the command : Start
TestExecutionHost.ProcessCommand. Start Command handled
Slice with id = 240681, of type = 'Discovery' received.
Count of test sources found: 2
=================================================================
Discovering tests from sources
Using new test platform for test execution
##[warning]DiscoveryMessage : Could not find extensions: C:\BA\2584\s\SpatialServices\packages\GoogleTestAdapter.0.16.1
##[warning]DiscoveryMessage : No test is available in C:\BA\2584\s\SpatialServices\bin\x64\Release\SpatialServicesApiTests.exe C:\BA\2584\s\SpatialServices\bin\x64\Release\SpatialServicesCoreTests.exe. Make sure that test discoverer & executors are registered and platform & framework version settings are appropriate and try again.
Number of testcases discovered : 0
Discovered tests 0 from sources
=================================================================
##[error]The slice of type 'Discovery' is 'Aborted' because of the error : System.Exception: No test is available in C:\BA\2584\s\SpatialServices\bin\x64\Release\SpatialServicesApiTests.exe C:\BA\2584\s\SpatialServices\bin\x64\Release\SpatialServicesCoreTests.exe. Make sure that test discoverer & executors are registered and platform & framework version settings are appropriate and try again.

I should note that these results are very different from the results I got about a month or two back. It seems that the test adapter can't be found in these cases. Previously, the test distribution wasn't working at all - it showed many retry attempts at distributing the tests and then eventually failed out. Unfortunately, those builds have aged out and I don't have the logs any longer :(

@tusharb86 can you please provide the full zip file that you get when you click on "Download all logs"?
Also please set system.debug = true

Please do re-open with the request information.

Similar issue has been reported for NUnit, the reporter of https://github.com/microsoft/azure-pipelines-tasks/issues/16187 has provided a minimal example of the issue.


Fix build and lint

About the PR

ferret and pigs were missing state declarations in their yml for their crit states from https://github.com/space-wizards/space-station-14/pull/32175 so they go invisible
gas pump and whatever had their namespace changed which causes master to no longer build successfully after https://github.com/space-wizards/space-station-14/pull/28995

Thanks for the fix!


CTC-57 Media clip player

Description

Implement media clip player
Url changes when selecting video
Handle played videos in local storage

Issue(s)
Fixes CTC-57
How to test

Go to https://deploy-preview-3042--oak-web-application.netlify.thenational.academy
Go to https://deploy-preview-3042--oak-web-application.netlify.thenational.academy/teachers/programmes/design-technology-secondary-ks3-l/units/core-design-skills-a819/lessons/graphic-communication-cdhkat/media
Test the page

Checklist

[ ] Added or updated tests where appropriate
[ ] Manually tested across browsers / devices
[ ] Considered impact on accessibility
[ ] Design sign-off
[ ] Approved by product owner
[ ] Does this PR update a package with a breaking change

@weronika-szalas looking good. A few comments from testing:

When in mobile view its difficult to select the bottom media clip in the list - I've tried this on a 'real' mobile and in the browser. It can take a few clicks to hit the correct element
A few issues with performance (not sure if its related to the data). It can take a second or two for the file to be shown so we get a brief view of an empty image file. See video below, also the lighthouse performance report is below 70 (not sure if anything can be done around that but it would be good if it was 80+)

https://github.com/user-attachments/assets/873b9f0a-4200-4a2a-90cd-1537a60ca4e0

And just to check @Monika-Koziol when the section is collapsed the width should be reduced like below?

[ ]   Can the copy and transcript buttons be visible for each clip?
Updated visibility of copy button. Transcript button visibility depends on whether there is a transcript available for that video (we work on fixture data at the moment until real data is ready)

[ ]  Can the header and footer not refresh for each clip? It would be a bit smoother for the user
I need to investigate this further as it shouldn't refresh with the current implementation (but I can confirm that it does). Are you happy for it to be done in the separate ticket so this chunk of work can already be merged?

[ ]   After a clip finishes playing, could the state automatically switch to "played" and move to the next clip (but without autoplay)?
This one definitely needs a separate ticket

[ ]   For keyboard navigation: the clip header minimise chevron currently lacks visual feedback. Could we add a focus state, similar to the one used for the "back to lesson" button?
I will implement this in a separate ticket

When in mobile view its difficult to select the bottom media clip in the list - I've tried this on a 'real' mobile and in the browser. It can take a few clicks to hit the correct element

I removed the shadow for now and will implement this in a separate ticket so it only shows when the list is scrollable and disappears when user gets to the end of the list

A few issues with performance (not sure if its related to the data). It can take a second or two for the file to be shown so we get a brief view of an empty image file. See video below, also the lighthouse performance report is below 70 (not sure if anything can be done around that but it would be good if it was 80+)

I can investigate it in a separate ticket - it might be because we make quite a few requests for the thumbnails. Video loads for me in similar time as in lesson page where it is lower in the page so you can't see it the black screen before thumbnail loads.

@weronika-szalas I'm ok with all the other requests being separate tickets. The copy link button is working now, so it's all good by me.

:tada: This PR is included in version 1.641.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Improve type annotations

Codecov Report
Attention: 11 lines in your changes are missing coverage. Please review.

Files
Coverage Δ

src/sdr/_conversion.py
92.64% <100.00%> (ø)

src/sdr/_data.py
88.63% <100.00%> (ø)

src/sdr/_detection/_correlator.py
35.55% <ø> (ø)

src/sdr/_detection/_theory.py
100.00% <100.00%> (ø)

src/sdr/_farrow.py
85.96% <100.00%> (ø)

src/sdr/_filter/_fir.py
66.26% <100.00%> (ø)

src/sdr/_link_budget/_capacity.py
96.42% <100.00%> (ø)

src/sdr/_loop_filter.py
92.30% <100.00%> (-0.20%)
:arrow_down:

src/sdr/_measurement/_energy.py
91.66% <ø> (ø)

src/sdr/_measurement/_errors.py
86.66% <100.00%> (ø)

... and 18 more

:loudspeaker: Thoughts on this report? Let us know!


Meta.id.like is not working

Here is my code to fetch all records with having email passed as parameter(emId) & ID starts with "login_".
public static ResultSet queryLoginModelEmail(String emId, Database database) {         Query query = QueryBuilder.select(SelectResult.all())                 .from(DataSource.database(database))                 .where(Expression.property("email").equalTo(Expression.string(emId))                         .and(Meta.id.like(Expression.string("login_%e%"))));         ResultSet result = null;         try {             result = query.execute();         } catch (CouchbaseLiteException e) {             e.printStackTrace();         }         return result;     }

But it is fetching all details with that email-id, "login_" condition is not working
Please help

The expression above, LIKE login_%e% doesn't mean that the ID starts with "login_" but it means that the ID has 'login_' as the prefix AND has anything that contains an 'e' character after that. I think you would need LIKE login_% to achieve the 'login_` prefix checking. Please check your query.
I have also tested both LIKE "login_%" and LIKE "login_%e%" against meta.id and both worked as expected.


[MLIR] Enable OpenCL backend

This PR enabled OpenCL backend for MLIR solver:

Added a testing stage for OpenCL test for MLIR as a sanity test

Since rest of MLIR kernels are already covered by hip, I avoided add duplicated tests

Amended mlir_impl_gemm.cpp to include OpenCL required sections

OpenCL do not implicitly linearize all kernel arguments as hip: I make it such that argument are manually expanded
In OpenCL there's no easy way to get the kernel address: I placed the ocl memory object such that a clSetKernelArg will be invoked on each kernel args

Updated clhelper.cpp to include mlir build

After this PR, state of OpenCL backend will be an orthogonal topic with MLIR. The release of MIOpen with MLIR will not be limited by its support for OpenCL backend.
Tested locally and make sure MIOpen behave the same between hip/ocl backend.

Gentle ping @junliume @atamazov, all CI tests have passed please let me know if you have any feedback.


koishi-plugin-hitokoto needs Chinese-Japanese translations

Add Japanese translations in koishi-plugin-hitokoto.
Localization files is at src/i18n/<locale>.yml, see Chinese version for more detail.
Do not push to master branch directly, you can create a new branch and use Pull Request to submit your translation.

We have already host on Crowdin, please click the link below to translate.
https://crowdin.com/project/koishi-plugin-hitokoto

Track translation process from crowdin, so this can be closed.


Billing FE - Invoice Payment - Confirm Card payment

I created a separated branch based on the PoC from Caroline

approved but don't forget it works with saving the pm - lets merge the back before

too old - closed!


Port to 1.20.2

A surprisingly trivial port

Derp... I should've tested the existing release first. Only change actually needed is to mark the latest release as compatible with 1.20.2 on CurseForge


Support for Mongoose methods and statics

Is there a way to wire up the existing model methods and statics with resolve functions?
i.e. if my model User has a method User.statics.findSimilar() is there a way for me to expose this as a GraphQL type?
If this was a new application I would use pre and post hooks but theres TONNES of existing business logic within the mongoose statics that I would rather not replicate by hand!
Any thoughts?

Are you talking about static (model level) or method (instance level)?
I'm not sure how much sense exposing static would make
There isn't at the moment (as far as I know), but I'd imagine it wouldn't be too hard to implement.

@burkhardr Take the example of Animal.search from http://mongoosejs.com/docs/2.7.x/docs/methods-statics.html
In a lot of cases existing Mongoose schemas have business logic implemented as schema statics which I think become instance methods by default when a model is instantiated.
I think it makes sense for graffiti-mongoose to automatically add a Animal.search as a resolve function instead of requiring me to re-implement Animal.search manually, to give you some idea the schema I'm working on has 50+ models and most models have 30+ methods / statics with very specific logic and I'm sure I'm not the only one!

See #64 for a real use case


Add catboost

I've added in two commonly used Catboost models with this PR (CatBoostRegressorand CatBoostClassifier). Please take a look at the changes and let me know if any changes are needed!

@PragyanSubedi
Looks good. Can you add tests?

@PragyanSubedi
Looks good. Can you add tests?

I've added it. Please have a look.

@PragyanSubedi
Sorry, I probably missed your comment.
Merged. Thanks!


esp32_psram.c: Remove a seemingly stale comment

Summary
Impact
Testing

@acassis do you remember what this comment meant?

the ci failure looks interesting. (one bit corruption?)
but it doesn't seem related to this PR.
====================================================================================
Configuration/Tool: sim/rc
2024-09-02 06:05:32
------------------------------------------------------------------------------------
  Cleaning...
  Configuring...
  Building NuttX...
make[1]: *** No rule to make target `/Users/run~er/work/nuttx/nuttx/sources/nuttx/include/unistd.h', needed by `pthread_mutextimedlock.o'.
make[1]: Target `libsched.a' not remade because of errors.
make: *** [sched/libsched.a] Error 2
make: Target `all' not remade because of errors.
/Users/runner/work/nuttx/nuttx/sources/nuttx/tools/testbuild.sh: line 378: /Users/runner/work/nuttx/nuttx/sources/nuttx/../nuttx/nuttx.manifest: No such file or directory
  Normalize sim/rc

the ci failure looks interesting. (one bit corruption?) but it doesn't seem related to this PR.
====================================================================================
Configuration/Tool: sim/rc
2024-09-02 06:05:32
------------------------------------------------------------------------------------
  Cleaning...
  Configuring...
  Building NuttX...
make[1]: *** No rule to make target `/Users/run~er/work/nuttx/nuttx/sources/nuttx/include/unistd.h', needed by `pthread_mutextimedlock.o'.
make[1]: Target `libsched.a' not remade because of errors.
make: *** [sched/libsched.a] Error 2
make: Target `all' not remade because of errors.
/Users/runner/work/nuttx/nuttx/sources/nuttx/tools/testbuild.sh: line 378: /Users/runner/work/nuttx/nuttx/sources/nuttx/../nuttx/nuttx.manifest: No such file or directory
  Normalize sim/rc

let's ignore the false alarm.


Update changelog for release v1.5.0

Codecov Report

Merging #799 (a41cb6e) into master (3d351c0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #799   +/-   ##
=======================================
  Coverage   92.23%   92.23%
=======================================
  Files          66       66
  Lines        5102     5102
=======================================
  Hits         4706     4706
  Misses        234      234
  Partials      162      162


README switch-case example does not work with webpack env object

Using switch(env) when env is an object and the cases are strings will always go to the default.  It seems either the example is out of date or it should just be an if block.
module.exports = env => {
  switch(env) {
    case 'development':
      return merge(commonConfig, developmentConfig);
    case 'production':
      return merge(commonConfig, productionConfig);
    default:
      throw new Error('No matching configuration was found!');
  }
}

This is what I used instead:
module.exports = env => {
    if (env['development']) {
        return merge(commonConfig(env), devConfig);
    } else if (env['production']) {
        return merge(commonConfig(env), prodConfig);
    } else {
        throw new Error('Please specify webpack "--env development" or "--env production"');
    }
};

webpack 5 had a breaking change here but I forgot to update the readme. Thanks for the heads up.

On 19.03.2021, at 15:39, Cameron Avelis @.***> wrote:

Using switch(env) when env is an object and the cases are strings will always go to the default. It seems either the example is out of date or it should just be an if block.
module.exports = env => {
switch(env) {
case 'development':
return merge(commonConfig, developmentConfig);
case 'production':
return merge(commonConfig, productionConfig);
default:
throw new Error('No matching configuration was found!');
}
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

This was solved in #180.


Clean up code

Having all code in one file might not be the way to go lol
Also, there's duplicated code here and there.
The program (.py) itself is not very edit-friendly at the moment.

Hello. I realize this issue is pretty old but since it's still open I thought it would be ok to use it.
Would you be interested in refactoring the repo to be able to create a "standard" python package that could be distributed via PyPI?

The code has been extremely cleaned up since when this was created going through 4.X and 5.X with a lot more polish than the 3.X codebase, with the potential of even more clean-up in the future. TSH has come a long way since this Shino lol.


Tooltip issues on charts using Triton theme

Found: 4.2.1
Ext.NET forum thread: Stacked bar chart tooltip
The tooltips in all charts examples have foreground color too dim, hard to read.
This is happening due to an override of the background color for tooltips in all examples through the StyleSpec property of the tooltips.

Respective MVC issue is extnet/Ext.NET.Examples.MVC#5.


Warning: Prop style did not match. Server: "left:0;color:#ffbc0b;width:2%" Client: "left:0;color:#ffbc0b;transition:;width:2%"  in nextjs 13

Warning: Prop style did not match. Server: "left:0;color:#ffbc0b;width:2%" Client: "left:0;color:#ffbc0b;transition:;width:2%"

add transition={true} to

When readonly={true}, then following error occurs:
client.js:1 Warning: Prop `style` did not match. Server: "null" Client: "cursor:"
    at span
    at span
    at exports.Rating (webpack-internal:///./node_modules/react-simple-star-rating/dist/index.js:1:2743)

When readonly={true}, then following error occurs:
client.js:1 Warning: Prop `style` did not match. Server: "null" Client: "cursor:"
    at span
    at span
    at exports.Rating (webpack-internal:///./node_modules/react-simple-star-rating/dist/index.js:1:2743)

This can be resolved with following props
style={{
          cursor: readonly ? 'default' : 'pointer',
        }}


ValueError: Error when checking target: expected concatenate_1 to have shape (20,) but got array with shape (10,)

I want to train two networks meanwhile, and merge(concatenate) the last layer as one to output.
But,there is something wrong that i could not debug.
The code as follow:
import keras
from keras.models import Model
from keras.layers import Input, Dense, Dropout, Activation, Concatenate
from keras.optimizers import SGD
import numpy as np
x_train = np.random.random((1000, 20))
y_train = keras.utils.to_categorical(np.random.randint(10, size=(1000, 1)), num_classes=10)
x_test = np.random.random((100, 20))
y_test = keras.utils.to_categorical(np.random.randint(10, size=(100, 1)), num_classes=10)
input_1 = Input(shape=(20, ))
x_1 = Dense(64, activation='relu')(input_1)
x_1 = Dropout(0.5)(x_1)
x_1 = Dense(64, activation='relu')(x_1)
x_1 = Dropout(0.5)(x_1)
x_1 = Dense(10, activation='softmax')(x_1)
input_2 = Input(shape=(20, ))
x_2 = Dense(64, activation='relu')(input_2)
x_2 = Dropout(0.5)(x_2)
x_2 = Dense(64, activation='relu')(x_2)
x_2 = Dropout(0.5)(x_2)
x_2 = Dense(10, activation='softmax')(x_2)
concat = Concatenate(axis=-1)([x_1, x_2])
model = Model(inputs=[input_1, input_2], outputs=concat)
sgd = SGD(lr=0.01, decay=1e-6, momentum=0.9, nesterov=True)
model.compile(loss='categorical_crossentropy',
optimizer=sgd,
metrics=['accuracy'])
model.fit([x_train, x_train], y_train,
epochs=20,
batch_size=128)
score = model.evaluate([x_test,x_test], y_test, batch_size=128)
ValueError: Error when checking target: expected concatenate_1 to have shape (20,) but got array with shape (10,)

I am also getting similar issue.
Any developments?


Add Example Usage for JSON Output

Summary
Add Example Usage for JSON Output
Background
Provide an example on the main readme page illustrating how to generate an output report in JSON format.
Tasks

Add example command line usage showing how to generate a JSON report
(Optional) Provide a sample (sanitized) JSON report and store it in this repository (allows consumers to see a working example report)

Acceptance Criteria

Example usage displayed on the main readme page.
(optional) users can view publically accessible example JSON report

Related: #68.

Part of this issue should definitely be to add --json support, so you can repo-report --json > report.json.

@ljharb
does this one still needs work or it's already done?
if not can you assign it to me and @KhatiaIvanova

It definitely still needs work :-)

(I can't assign @KhatiaIvanova until they've commented on the issue)

@ljharb assign this issue to me as well


Add William Gough to Contributors list

First contribution!

Hi  @winglish, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added social share  to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


Not working with bootstrap 5

I'm trying to integrate with bootstrap 5 but I not working dropdown with optgroup. Am I missing something? Here is the link https://codepen.io/yurenlimbu/pen/xxXqyoK

Closing as duplicate of https://github.com/snapappointments/bootstrap-select/issues/2607


Buchheim plotting

Added a more robust plotting method. Implemented some defaults, that I found most sensible. Please feel free to come with input, if so pleased.

This implementation depends on the GridSearchCV pull request. It must be merged after that one.


Python Version

linked to:https://editor.p5js.org/codingtrain/full/0Ud-XsaYb

Community Contribution
Link to live project, video, or image:

Sharing

[x] I would be happy to have my project shared on The Coding Train's social media!

Instagram:
Twitter:

Python pyglet version


Implemented simple PDF generation through weasyprint

Added the possibility to generate PDFs from the HTML reports through weasyprint.
TODO:

[ ] add CSS format for prettier reports

@kurosch Thanks for the contribution!
Introducing code and a dependency increases complexity and reduces maintainability. At least the dependency should be optional. It's much more logical to apply the conversion step after generating the report. The html -> pdf functionality itself could be an extension.
With the current style I also expect the PDF output to not be useful, for the reason that you already mention (CSS not being optimized for static output). However, If you're planning on optimizing the CSS for print then that would be an amazing contribution.


JS: continuous speech recognition not invoking recognised event handler.

I'm writing yet-another speech transcription/translation app in javascript a browser. I have imported the "microsoft-cognitiveservices-speech-sdk" (v1.14.1) to achieve this and correctly configured the speech API in my Azure Resource Group along with the text translation cognitive services. So good so far. I have translation working like a dream. Transcription (speech-to-text, or STT) has been more difficult, but I'm finally getting there.
However, I have a problem with it. I am using the continuous recognition method directly from the default microphone, initialising the process with the startContinuousRecognitionAsync() method. Whilst the recognizing method is invoked during the 'session' I have found that the recognized event doesn't fire for me.
import { SpeechConfig, AudioConfig, SpeechRecognizer, ResultReason, NoMatchReason, NoMatchDetails } from "microsoft-cognitiveservices-speech-sdk";
const speech = {
  subscriptionKey: "...",
  region: "westeurope",
  speechConfig: null,
  audioConfig: null,
  recognizer: null,
  register: () => {
    console.log(`Starting...`);
    speech.speechConfig = SpeechConfig.fromSubscription(speech.subscriptionKey, speech.region);
    speech.audioConfig = AudioConfig.fromDefaultMicrophoneInput();
    speech.recognizer = new SpeechRecognizer(speech.speechConfig, speech.audioConfig);
  },
  rec: () => {
    speech.recognizer.recognized = speech.done;
    speech.recognizer.recognizing = speech.recognizing;
    speech.recognizer.startContinuousRecognitionAsync();
    console.log("Recording...");
  },
  done: (s, e) => {
    console.log(`SSS: ${JSON.stringify(s, null, 4)}`);
    console.log(`EEE: ${JSON.stringify(e, null, 4)}`);

    if (e.result.reason === ResultReason.NoMatch) {
      const noMatchDetail = NoMatchDetails.fromResult(e.result);
      console.log("DDD (recognized)  Reason: " + ResultReason[e.result.reason] + " | NoMatchReason: " + NoMatchReason[noMatchDetail.reason]);
    } else {
      console.log(`PPP (recognized)  Reason: ${ResultReason[e.result.reason]} | Duration: ${e.result.duration} | Offset: ${e.result.offset}`);
      console.log(`Text: ${e.result.text}`);
    }
  },
  recognizing: (s, e) => {
    console.log(`BBB: ${e.result.text}`);
  },
  stop: () => {
    speech.recognizer.stopContinuousRecognitionAsync(r => {
      console.log(`TTT: stopped | ${r}`);
    });
  }
}

The logs are filled with unique search strings to help me search for the code that generated them, e.g. 'TTT'.
I won't bore you with the code needed to invoke this object -- it sits in a react app with all sorts of complexities. It basically calls speech.register() once, then when the user wants to start STT it calls speech.rec(), followed sometime later by speech.stop(). I expect recognizing() to be invoked during the session and recognized() (the done() method in my object) to be called when the operation has completed  because speech.recognizer.stopContinuousRecognitionAsync() has been invoked. I have also registered an error handler, that's not being invoked either but that is not important for this conversation. I have paused my code in the Chrome debugger and confirmed that my handlers are properly registered with the recognizer.
This sort-of works. I see the results of the recognizing method (and they're quite impressive -- even transcribing Jabberwocky quite well). But I really need to hook into the final event when the recording is stopped and the full text becomes available. It would be a lot of hard work to use the output of the partial in-progress transcription event (recognizing()). I expect the recognized() event handler to be invoked with the full contents of the transcription from start to finish, but it isn't being invoked at all.
Any help would be great. I have a deadline for completion of the project in a fortnight and this is the final part to get working. The translation works great and the transcription seems really promising. Thanks, Bryan.

Sorry -- this wasn't a bug -- it was my fault entirely. The problem was that in trying to understand the SDK I had stringified the first parameter of the recognized() event handler, documented simply as s. It turns out that this is the recognizer itself and stringifying it causes a silent error. As this was the first line of the event-handler it was simply failing silently. I have fixed my code and I'm moving on.

Sorry -- this wasn't a bug -- it was my fault entirely. The problem was that in trying to understand the SDK I had stringified the first parameter of the recognized() event handler, documented simply as s. It turns out that this is the recognizer itself and stringifying it causes a silent error. As this was the first line of the event-handler it was simply failing silently. I have fixed my code and I'm moving on.


i18n(fr) Update French translation markdown-content.mdx

Description (required)
Update French translation markdown-content.mdx file with PR #7462

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

fr
guides/markdown-content.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


what is the proper way to implement analytics per page view (per step) ?

I haven't really looked into this use-case at all, but you might be able to leverage the useDeck hook to pick up the context/state in a custom provider component. Page views should probably work since this uses Reach Router

Is there any example of useDeck besides the doc? Thanks @jxnblk


[WIP] [1.11] Move CTM from internal code to CTM-mod.

Currently WIP as not everything is working as before.
Todo:

[ ] Fix textures Induction Cell, Induction Provider, Induction Port & Thermal Evap. Controller (BlockBasic getOverrideTexture())
[ ] Fix textures Reactor Controller & Reactor Port (BlockReactor getOverrideTexture())
[ ] Connect the texture of "same" blocks (Thermal Evap. tower blocks, Boiler blocks)
[ ] Cleanup some leftover code

Should fix #4488

Current PR is on hold, waiting on some features in CTM to check the third point


Security: permissions management

We need to investigate of how we can manage the users and their permissions. The goal is to be able to link a user with Okta. Maybe we can do a cron which push the users and their roles.

In the future, Okta will be plugged to GH. We keep this ticket in the backlog for now.

Bob Moghimi
Thibault Jamet
@bmoqimi @tjamet
we are talking about that on slack #github-com-adevinta-owners private channel


Send/Receive message draft

@lawrence-forooghian - I was wrong in that the macOS and iOS features diverged... I'm actually targeting iOS 16 but not macOS 16 hence the need to check available OS version... ChatAPI is the class to check (where I'm passing in a Protocol as a parameter instead of a concrete implementation)
Will merge main and resolve conflicts but this draft is good enough to get a gist of what is going on. I'll also refactor some of the access given to certain classes/structs, as well as improve general organisation
https://ably.atlassian.net/browse/ECO-4942

@umair-ably what was your aim in using existential types (i.e. the any Foo<T> which is giving you the OS support issue)? Is there a reason you avoided just using the concrete types i.e. https://github.com/ably-labs/ably-chat-swift/commit/71270cf3909ccf413a1f0dfd6b2cbc08b09670c0?

Also, just to check — are you intending to add tests?

@umair-ably what was your aim in using existential types (i.e. the any Foo<T> which is giving you the OS support issue)? Is there a reason you avoided just using the concrete types i.e. something like 71270cf?

I started off with a generic implementation hence the protocol itself being passed in. I didn't want the base network "layer" to know about any concrete types in case we add additional implementations of the protocol later. But on second thoughts (and from seeing your solution), I'd already implemented the concrete type to also be fairly generic so I achieve my initial aim too - I just became blind to tying all of it together from staring at this too long 😅
Thanks for this - it perfectly solves the problem!

Also, just to check — are you intending to add tests?

Yeah for sure, ideally I want to put the completed implementation of the message spec in this PR... hoping you can review this first to point out any obvious errors/deviations and then it'll be a good base to finish the rest off

Before removing from draft please could you also tidy up the commits; i.e. in addition to what I mentioned here also make sure to remove the merge commit and to incorporate linting fixes into the commit that introduced the errors. Thanks!

Marking as ready for review to see what code-rabbit has got to say... will respond to other PR comments shortly

I've tried this branch without preconcurrency import, and as I said on a meeting, you just need to replace all occurrences of  let properties: ARTChannelProperties with var properties: ARTChannelProperties { .init() } (it's a getter in cocoa anyways) and also replace let defaultChannelOptions... with var defaultChannelOptions... (for now at least). FYI @lawrence-forooghian
So yeah, no need for https://github.com/ably/ably-cocoa/pull/1973 atm

@coderabbitai pause

@coderabbitai pause

@umair-ably please could you also update the commit message to mention the commit hash of the spec that these were based on (see example in 25e5052739212b9ce058a677d906d60ee56cf820)? This is important because the spec is very much still in flux (i.e. spec points that you have implemented here might get changed), so in the future we might need to update some of the implementation added here in order to bring it in line with the latest version of the spec, and the easiest way to do that is to diff the latest version of the spec against the version of the spec that you based your implementation on.

@umair-ably please can you address #45 (comment)?

done

@lawrence-forooghian Getting that linting error again...
Error: packageLockfilesHaveDifferentContents(paths: ["Package.resolved", "AblyChat.xcworkspace/xcshareddata/swiftpm/Package.resolved"])

@lawrence-forooghian Getting that linting error again...
Error: packageLockfilesHaveDifferentContents(paths: ["Package.resolved", "AblyChat.xcworkspace/xcshareddata/swiftpm/Package.resolved"])

Also, do we need Package.resolved in the root directory? I can see the revision it is pointing to for ably-cocoa hasn't updated despite the packages in the left pane showing the updated version. I've removed this file locally and everything still seems to build fine?

Left a few small questions. Didn't check the validity of tests either, @umair-ably couldn't you assure that all implemented spec points are included in the tests of this PR?

A chunk of them have been but it's not exhaustive. I want to implement more chat features first, I'll add more tests for this later

A chunk of them have been but it's not exhaustive. I want to implement more chat features first, I'll add more tests for this later

In this case it's better to have an issue to track all unimplemented spec points IMHO. I'm approving regardless.

thank you!
added an issue here


Make more fixes for moving GenericStub from ::grpc_impl to ::grpc

Artifact Build Windows (internal CI): https://github.com/grpc/grpc/issues/23513


Why is it that the monitoring camera that the apple tablet sees is sometimes slow or stalled ?

Why is it that the monitoring camera that the apple tablet sees is sometimes slow or stalled ?

Probably ffmpeg failed to encode video fast enough?


Document.from_batch_process_operation() method failing due to sharding made by batch process documents

Hi,
I've been facing the following issue when I try to use the Document.from_batch_process_operation() method:
I expected that this method, given a succeeded operation (like mine), would get all the output files from GCS and serialize them into document objects. Instead, it's failing with the bellow described exception, and the thing is: the output of the operation was done through a BatchProcessing operations with the Python SDK, so I expected that both methods of the Python SDK would match output and input.
Note 1:
The same thing happens if I use the from_gcs() method, passing the root directory (<root_operation> dir let's call it) of that operation output as gcs_prefix. If I use the <root_operation_dir>// as gcs_prefix, the method succeeds.
Note 2:
The from_gcs() method raises
ValueError: Invalid Document - shardInfo.shardCount (1) does not match number of shards (1942).

and the from_batch_process_operation() method raises:
ValueError: Invalid Document - shardInfo.shardCount (1) does not match number of shards (1053).

Why the difference in shards (?) between the methods?
Environment details

OS type and version: MacOS 13.0 (22A8380)
Python version: 3.8.0
pip version:  19.2.3

Steps to reproduce

Requirements:

cachetools==5.3.3
certifi==2024.2.2
charset-normalizer==3.3.2
Deprecated==1.2.14
google-api-core==2.17.1
google-auth==2.28.1
google-cloud-bigquery==3.17.2
google-cloud-core==2.4.1
google-cloud-documentai==2.24.0
google-cloud-documentai-toolbox==0.13.0a0
google-cloud-storage==2.14.0
google-cloud-vision==3.7.1
google-crc32c==1.5.0
google-resumable-media==2.7.0
googleapis-common-protos==1.62.0
grpc-google-iam-v1==0.12.7
grpcio==1.62.0
grpcio-status==1.62.0
idna==3.6
immutabledict==3.0.0
intervaltree==3.1.0
Jinja2==3.1.3
lxml==4.9.4
MarkupSafe==2.1.5
numpy==1.24.4
packaging==23.2
pandas==2.0.3
pikepdf==8.13.0
pillow==10.2.0
proto-plus==1.23.0
protobuf==4.25.3
pyarrow==15.0.0
pyasn1==0.5.1
pyasn1-modules==0.3.0
python-dateutil==2.9.0
pytz==2024.1
requests==2.31.0
rsa==4.9
six==1.16.0
sortedcontainers==2.4.0
tabulate==0.9.0
tzdata==2024.1
urllib3==2.2.1
wrapt==1.16.0

Execution:
python3 main.py

Code example
main.py:

from google.cloud import documentai
from google.cloud.documentai_toolbox import document

wrapped_document = document.Document.from_batch_process_operation(
    operation_name=operation_name
    location=location
)

wrapped_document.entities_to_bigquery(
        dataset_name=dataset, table_name=table, project_id=project
)

Stack trace
Traceback (most recent call last):
  File "main.py", line 4, in <module>
    wrapped_document = document.Document.from_batch_process_operation(
  File "<script_location>/venv/lib/python3.8/site-packages/google/cloud/documentai_toolbox/wrappers/document.py", line 620, in from_batch_process_operation
    return cls.from_batch_process_metadata(
  File "<script_location>/venv/lib/python3.8/site-packages/google/cloud/documentai_toolbox/wrappers/document.py", line 576, in from_batch_process_metadata
    return [
  File "<script_location>/venv/lib/python3.8/site-packages/google/cloud/documentai_toolbox/wrappers/document.py", line 577, in <listcomp>
    Document.from_gcs(
  File "<script_location>/venv/lib/python3.8/site-packages/google/cloud/documentai_toolbox/wrappers/document.py", line 507, in from_gcs
    shards = _get_shards(gcs_bucket_name=gcs_bucket_name, gcs_prefix=gcs_prefix)
  File "<script_location>/venv/lib/python3.8/site-packages/google/cloud/documentai_toolbox/wrappers/document.py", line 133, in _get_shards
    raise ValueError(
ValueError: Invalid Document - shardInfo.shardCount (1) does not match number of shards (1053).

Thanks for the feedback @gabrielboehme. Can you clarify what is in the GCS directory that you're sending to the API? Can you share the output of gsutil ls on the directory (with any PII removed)?
It seems that there's many more JSON files in the directory than should be based on the Document.shardInfo.shardCount field. Both Document.from_batch_process_operation() and Document.from_gcs() expect that the directory contains Document.JSON files from a specific operation.

Hey @holtskinner, thanks for the reply.
Sure, let me clarify:
I used the following code to execute the batch operation on my custom extractor --successfully:
gcs_output_uri = 'gs://bucket_a/output/'
gcs_prefix = 'gs://bucket_a/input/' # This is a folder with many subfolders and files.
gcs_output_config = gcs_output_config = documentai.DocumentOutputConfig.GcsOutputConfig(
    gcs_uri=gcs_output_uri, field_mask=None
)

input_config = documentai.BatchDocumentsInputConfig(gcs_prefix=gcs_prefix)
output_config = documentai.DocumentOutputConfig(gcs_output_config=gcs_output_config)

request = documentai.BatchProcessRequest(
    name=name,
    input_documents=input_config, #
    document_output_config=output_config,
)
    # BatchProcess returns a Long Running Operation (LRO)
operation = client.batch_process_documents(request)

And separately I used the following code to push to try pushing to BQ:
from google.cloud import documentai
from google.cloud.documentai_toolbox import document

wrapped_document = document.Document.from_batch_process_operation(
    operation_name=operation_name # projects/project_id/locations/location/operations/operation_id
    location=location
)

wrapped_document.entities_to_bigquery(
        dataset_name=dataset, table_name=table, project_id=project
)

Which failed with:
Traceback (most recent call last):
  File "main.py", line 4, in <module>
    wrapped_document = document.Document.from_batch_process_operation(
  File "<script_location>/venv/lib/python3.8/site-packages/google/cloud/documentai_toolbox/wrappers/document.py", line 620, in from_batch_process_operation
    return cls.from_batch_process_metadata(
  File "<script_location>/venv/lib/python3.8/site-packages/google/cloud/documentai_toolbox/wrappers/document.py", line 576, in from_batch_process_metadata
    return [
  File "<script_location>/venv/lib/python3.8/site-packages/google/cloud/documentai_toolbox/wrappers/document.py", line 577, in <listcomp>
    Document.from_gcs(
  File "<script_location>/venv/lib/python3.8/site-packages/google/cloud/documentai_toolbox/wrappers/document.py", line 507, in from_gcs
    shards = _get_shards(gcs_bucket_name=gcs_bucket_name, gcs_prefix=gcs_prefix)
  File "<script_location>/venv/lib/python3.8/site-packages/google/cloud/documentai_toolbox/wrappers/document.py", line 133, in _get_shards
    raise ValueError(
ValueError: Invalid Document - shardInfo.shardCount (1) does not match number of shards (1053).

@gabrielboehme Thanks for that extra information. I'll try to reproduce on my end. Can you also specify how you're getting operation_name? This should be the way to get the operation name:
operation = client.batch_process_documents(request)
operation_name = operation.operation.name

wrapped_document = document.Document.from_batch_process_operation(
    operation_name=operation_name
    location=location
)

Note - I got a different error when running on Python3.8 that has been fixed in 0.13.1a0, so I ran it on that version.
I'm able to reproduce the behavior. Seems like the fetching of json files from GCS isn't working as expected and too many JSONs are being imported as a single Document. I'll investigate futher.

My working theory is that batch_process_metadata.output_gcs_destination looks like this:
output_gcs_destination: "gs://bucket-name/toolbox-test-output/15721623642972401514/1"

And without the trailing slash at the end, other folders like:
gs://bucket-name/toolbox-test-output/15721623642972401514/11
gs://bucket-name/toolbox-test-output/15721623642972401514/12
etc

Are all being read in as well. It's a quirk of how GCS prefixes work.
I'll also add an integration test for this edge case.

PR #274 should fix the error message. Although, your code to upload entities to BigQuery won't work as is. from_batch_process_operation() returns a List of Documents, so you'd need to do this:
wrapped_documents = document.Document.from_batch_process_operation(
    operation_name=operation_name, # projects/project_id/locations/location/operations/operation_id
    location=location
)

for wrapped_document in wrapped_documents:
    wrapped_document.entities_to_bigquery(
            dataset_name=dataset, table_name=table, project_id=project
    )

@holtskinner I see. Thanks for the help and for solving it quickly!


Deploy CRDs in the hive-operator.

Due to an OLM bug where CRDs are only created new, never updated, we
need a temporary workaround until a fix can be delivered. Rather than
having humans kubectl apply all the CRDs we'll do so with our operators.
The CRDs still are referenced in the OLM bundle, but we're re-applying
them here.
Change also adds hive admin and reader permissions to view CRDs, as this
is coming up a lot given the current bugs.

/lgtm


Detect that iOS app is running on Mac hardware

Solve #26 by adding support for iOS apps running on Mac hardware. The iOS 14 API NSProcessInfo.ProcessInfo.IsiOSApplicationOnMac is used to determine if the app is running on a Mac.
When running on a Mac the following values are returned:

Version: mac
Model: Mac
ChipType: AppleSiliconMac

The more generic chip name AppleSiliconMac was selected as I do not think we ever will be able to determine the actual underlying hardware for an iOS app executing on Mac hardware (the hw.machine returns ipad8,6 as its fallback/compatibility value).
One could argue that this PR should also keep ipad8,6 as its version string, or not try to be smart about the model name. But at the same time, it does feel similar to how the library handles Simulator as a special case.

Thank you @follesoe and apologies for the delay. Will be reviewing and likely merging this into the next release. Waiting on Apple for latest device devices just announced.

@dannycabrera A suggestion may be to change chip type enum from AppleSiliconMac to M1, as you will actually need that value for the latest iPad Pro models (which run M1). So we could consider using that value for Mac's running iOS, as we know that all Mac's running iOS devices (as of now) are running M1. This may change in the future, and I doubt there will be APIs to differentiate a MacBook Pro 13" with M1 from say a 16" with a future M2.


When the upload fails, remove image preview

This commit for preventing to submitting failed upload previews which are sources as base64 encoded img hash.

I don't think this is a universal behavior for every use case. Sometimes you might want to leave the preview images there. Maybe you want to implement some sort of auto retry after failed images and you don't want to remove previews. Or I even saw projects where they don't have upload API at all for some reason. Even our demo page doesn't have it.
I think more universal approach would be to remove preview images on save so you prevent sending those huge data URI to your server. But in that case you want to remove it in serialize function, not on failed upload.


Change watch_loop not to use wait_for_events method because the previ…

…ous implementation of watch_loop() has a race condition problem.
watch_loop() calls fetch() and then calls wait_for_events(). However before
wait_for_events() is called this situation can happen:
ExistsRequest -> (ExistsRequest is proccessed by server) -> SetDataRequest ->
SetDataResponse -> ExistsResponse -> GetDataRequest -> WatchEvent ->
GetDataResponse
Successive call to wait_for_events() awaits watch-event indefintely because
watch-event is already fired and forgot.
To solve this problem, modify watch_loop() to keep watch-callbacks always.

@rhdxmr thank you for the contribution. Could you please add tests for this fix?

@cybergrind OK I will do that next week. Because I am travelling Taipei in Taiwan until next Tuesday so I don't have time to write some code now. :) I will let you know when I finish the code. thanks.

@rhdxmr no hurry, have a good time!

@cybergrind Hello I just added a test code. The test succeeds with new implementation of watch_loop but fails with the previous one.

@cybergrind I guess you missed new comment before. So I remind you again. Could you review my code?

@rhdxmr actually I'm not sure if we need to fix it with such big patch, because your test for it can be fixed with 2 lines change https://github.com/micro-fan/aiozk/commit/03dbcc160f1c7b012ba6c12e8e83c8e220e8cc3c
Probably I don't fully understand it or you're missing something

Hello, @cybergrind
I appreciate your review.
I read your commit https://github.com/micro-fan/aiozk/commit/03dbcc160f1c7b012ba6c12e8e83c8e220e8cc3c
And I think it is more concise and neat than mine. And I think it solves race condition problem addressed by this PR.
So it would be better to reject my code.
But what do you think about the test code?  I just thought that I might oversee something and needed to get your confirmation that everything looks good for you. then will merge your test + my fix.
Thank you for your contribution!

Released as 0.19.0


Does my library, violations-lib, belong here?

I develop this library: https://github.com/tomasbjerre/violations-lib
I would happily have it added, if you feel it belongs here?

Hey @tomasbjerre,
this library absolutely belongs here!
I didn't even know that something like this existed, very useful.
May I suggest the Supporting Tools category? (https://github.com/mre/awesome-static-analysis#supporting-tools).
Looking forward to your PR.


Edit attribute label button is dead

In the Data Model Editor, hovering over an attribute shows an edit button that does nothing when clicked.

Should we add some logic to it or remove it?
Adding logic would lead to the question what should happen when the user switches the attribute type or if it should only be possible to assign a new label

If the button is dead, I would prefer to remove it.

But do we want such a button or not? If we want it, I think it is easier to implement it, then removing it...

Editing the datatype of an attribute comes with some potential problems, e.g. what happens to existing data? It feels simpler to remove the button for now 🔪

Is editing the datatype a feature we want or not? If not, we could add edit label functionality to the button. If we want it, I'd suggest that we try to cast existing values to the new datatype (e.g. double to integer or string to stringf should be possible).

💯


Resource routing overlaping when they are devided

Laravel Version
11.9
PHP Version
8.3
Database Driver & Version
No response
Description
Problem: Route Overlapping in Resource Routes with only and except Clauses
When dividing resource routes using the only and except clauses (e.g., separating the index and show routes under auth middleware and create, store, edit, update, and destroy routes under auth:admin middleware), an unexpected route overlapping issue occurs.
Scenario

When defining routes like this:
Route::middleware('auth')->group(function () {
    Route::resource('/areas', AreaController::class)->only(['index', 'show']);
});

Route::middleware('auth:admin')->group(function () {
    Route::resource('/areas', AreaController::class)->except(['index', 'show']);
});

The create route (GET /areas/create) overlaps with the show route (GET /areas/{area}). Laravel attempts to resolve the create string as an {area} parameter, leading to an SQL error when it tries to find a record with id = 'create'.

Outcome

Instead of resolving the correct route for create, Laravel interprets "create" as an {area} parameter and tries to find a record with that ID, causing an SQL error when it fails to find an id with the value "create".

Solution Found
A workaround for this issue is to add a whereNumber() constraint to the {area} parameter in the routes to ensure that Laravel only considers numeric values as valid IDs for the show, edit, update, and destroy routes.
Updated Route Definition
Route::middleware('auth')->group(function () {
    Route::resource('/areas', AreaController::class)->only(['index', 'show'])->whereNumber('area');
});

Route::middleware('auth:admin')->group(function () {
    Route::resource('/areas', AreaController::class)->except(['index', 'show'])->whereNumber('area');
});

Why This Works

The whereNumber() constraint ensures that Laravel treats the create string as a separate route, preventing it from being mistaken as an id in the show route. This prevents the overlapping issue between create and show.

Expected Behavior:
When defining resource routes with only and except clauses, routes should not overlap, and create should not be misinterpreted as a resource identifier for the show route.
Additional Notes:

This issue does not occur when the resource route is defined without only and except clauses (i.e., all routes are defined under a single route line) like this :

   Route::resource('/areas', AreaController::class);

It only occurs when routes are split across different middleware groups using only and except.

Proposed Fix or Improvement:
Consider adding a note to the documentation or improving the route model binding mechanism to handle such cases more gracefully when splitting resource routes using only and except. Alternatively, suggesting the whereNumber() constraint as a best practice when dealing with this scenario could help other developers avoid this issue.
Credit

@YounesAmalou

Steps To Reproduce
just try this :
Route::middleware('auth')->group(function () {
    Route::resource('/areas', AreaController::class)->only(['index', 'show'])->whereNumber('area');
});

Route::middleware('auth:admin')->group(function () {
    Route::resource('/areas', AreaController::class)->except(['index', 'show'])->whereNumber('area');
});

and the create route will not be accessible, insteade an sql error will occur saying that id = create is not allowed or something similar.

This is actually the expected behaviour. The first defined route is the one that'll catch the incoming request. Just swap the route sequence for it to work:
Route::middleware('auth:admin')->group(function () {
    Route::resource('/areas', AreaController::class)->except(['index', 'show']);
});

Route::middleware('auth')->group(function () {
    Route::resource('/areas', AreaController::class)->only(['index', 'show']);
});

Oh my bad :') Thanks ^.^


Add skip CD feature to workflow and update documentation for Go versi…

워크플로우  skip cd 기능 추가
go version 업그레이드에 따른 문서 수정

@heedaeshin
참고 바랍니다 :-)
https://docs.github.com/en/actions/managing-workflow-runs-and-deployments/managing-workflow-runs/skipping-workflow-runs


feat: [Drizzle] Convert MySQL timestamps to datetimes

Is your feature request related to a problem? Please describe.
For date + time related fields, MySQL's timestamp field is only 32 bits, and will run into issues once 2038 comes around. This isn't a giant issue per se, and users can certainly correct this on their own, but IMO there really is no harm in switching to datetime columns instead, since they can hold dates from 1000-01-01 00:00:00 to 9999-12-31 23:59:59, which will last a lot longer.
Describe the solution you'd like to see
Prisma already stores datetimes like this (reference), and switching drizzle is as simple as:
// Example model schema from the Drizzle docs
// https://orm.drizzle.team/docs/sql-schema-declaration

import { sql } from "drizzle-orm";
import {
  bigint,
  index,
  mysqlTableCreator,
-  timestamp,
+  datetime,
  varchar,
} from "drizzle-orm/mysql-core";

/**
 * This is an example of how to use the multi-project schema feature of Drizzle ORM. Use the same
 * database instance for multiple projects.
 *
 * @see https://orm.drizzle.team/docs/goodies#multi-project-schema
 */
export const mysqlTable = mysqlTableCreator((name) => `project1_${name}`);

export const posts = mysqlTable(
  "post",
  {
    id: bigint("id", { mode: "number" }).primaryKey().autoincrement(),
    name: varchar("name", { length: 256 }),
-   createdAt: timestamp("created_at")
-     .default(sql`CURRENT_TIMESTAMP`)
+   createdAt: datetime("created_at", { fsp: 3 })
+     .defaultCurrentTimestamp()
      .notNull(),
-    updatedAt: timestamp("updatedAt").onUpdateNow(),
+    updatedAt: datetime("updatedAt", { fsp: 3 }).onUpdateCurrentTimestamp(),
  },
  (example) => ({
    nameIndex: index("name_idx").on(example.name),
  })
);

This is using a sub-second precision of 3 points, which is in-line with what Prisma does, just for consistency.
Describe alternate solutions
N/A
Additional information
The syntax for onUpdateCurrentTimestamp requires https://github.com/drizzle-team/drizzle-orm/pull/1114 to be merged, and I don't know that Drizzle currently supports the syntax necessary for auto-updates directly, though I believe you can just insert it into the default SQL like so:
updatedAt: datetime("updatedAt", { fsp: 3 }).default(sql`CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP`),

seems reasonable. do you want to open a pr for this?

seems reasonable. do you want to open a pr for this?

I'm watching the Drizzle issue which will introduce the change for the default timestamp, since I don't want to rely on semantics that are kind of undefined behavior (the on update logic being in the default statement) in code that will likely ship to production applications. Once that goes through, I will probably make a PR if no one else has.

hey @Wundero this line
updatedAt: datetime("updatedAt", { fsp: 3 }).default(sql`CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP`),

throws this error when trying db:push


fields2cover: 1.2.1-2 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository fields2cover to 1.2.1-2:
upstream repository: https://github.com/Fields2Cover/fields2cover.git
release repository: https://github.com/Fields2Cover/fields2cover-release.git
distro file: noetic/distribution.yaml
bloom version: 0.11.2
previous version for package: 1.2.1-1

Holding for Noetic sync


Notes of a tutorial workthrough

Some notes form the workthough of http://wiki.ros.org/psen_scan/Tutorials/SettingUpPSENscanHW:

description of PilzDe/pilz_tutorials needs to be updated
Introduction

replace "using our Windows ..." with "using the Pilz configuration tool"

Prerequisites:

When restrict the configurator to 3.0.0, which scanner versions work with that configurator.
Manuel is inked to PsenScan sc M 3/5 are those the only supported onse?
State exactly which scanner versions are supported (to be consistent with manual an shop)
The Ethernet Cable/power cable is it shipped with the scanner or can we add a link to the shop?
State the required operation system
"An Ethernet interface to communicate from your PC via Ethernet to the PSENscan laser scanner." simplify e.g. "PC with Ethernet"
installation tutorial (after point "A 24V DC power supply. ") could be a separate section
image showing the psen scan bottom with the connection probably has the ethernet on the wrong adapter and the power slot is not clearly identifiable
second image PSENscan_sample_setup.png is only supposed to show ethernet has to be connected to the psenscan? Don't see the benefit of this image.

3.0

Is this Note really necessary? The scanner is also found from another subnet and the ip can be changed still.

3.1

the configurator is already a prerequisite, does it need another section?

3.2

"Device → Search device" is actually "Scanner -> Search device"

3.3

Why change the Ip Address?
no admin pw is required to change the ip
being in another subnet opens this ip change window on double click
"...connecting to the network and click OK." and "Click OK to continue" is one OK to much
*3.4
"Click on new configuration. " its "Create new configuration"
maybe add a note on where to find the scanner version on the product
"n the general properties "
"the right number of pins for your connection" which connection? Ethernet?
"Click at next and choose your scanner. " => "Click next ... "
the "Device not available warning " should not be here. For one we did already fix the ip in previous section and if this doesn't happen often it should be in troubleshooting anyway.
"During download the PSENscan laser scanner screen becomes black. " Its first black, then it shows a File with the text DL DNC below.
"The finish dialog appears, the PSENscan laser scanner screen comes back to life." It was "back to life" until i clicked "accept" and went black again during the Finish process.
"to switch to the Monitoring tab. In this tab an image of the ..." => "to display the 2d surroundings of the scanner."

Notes to http://wiki.ros.org/psen_scan/Tutorials/ConfiguringPsenScanParametersWithYaml

Note: This tutorial assumes ... link is connected to wrong tutorial

Notes for http://wiki.ros.org/psen_scan/Tutorials/ConfiguringPsenScanParametersWithYaml:

Link to previous tutorial is wrongly set
Page name and description are inconsistent with title of tutorial
1 Introduction:

Strange sentences. Suggestion: "This tutorial teaches you to set up and launch a ROS driver for the PSENscan laser scanner. This includes creating a new ROS package to read and display laser scanner data."

3 Required parameters

"the following mandatory parameters are required" -> "the following parameters are required" or "the following parameters are mandatory"
"ROS driver for the PSENscan laser scanner", we think "for the PSENscan laser scanner" can be omitted from here on.
It would make more sense to create the package first and then describe the parameters

4 Create a ROS package:

catkin_create_pkg command needs the --rosdistro argument!
Do we need the copyright/license header in the yaml-file/launch-file?
Why do the yaml/launch-file include tags? (e.g. %Tag(FULLTEXT)%)
It should be mentioned that the user has to change the ip addresses accordingly (The previous tutorial showed how these could be altered)

6 Conclusion:

Should we mention further steps?
Suggestion for the text: "In this tutorial you created a basis for an application using the PSENscan laser scanner."

Notes for http://wiki.ros.org/psen_scan/Tutorials/LaunchingMultipleScanners:

3 Creating the config files:

"Here´s an example" -> "Here is an example"

4 Creating the launch file:

What is the use of the tags in the launch-file?
Add a sentence at the end about what is different to the previous launch-file.

5 Start the application:

In order to show the data of both scanners simultaneously, the location of each scanner should be defined in a given environment (define base links in urdf-file like in the pilz_robots tutorials).


Option to trigger custom emails to the applicants at any time

Currently, the application allows emails for specific actions. There's no way we can resend, in case there is an error, or send a custom email if we want. We need an option to send a custom email.

@rathorevaibhav explained this :
There is a card for each round (Resume screening, telephonic interview, etc). The card-footer contains three items:

A drop-down for selecting the next action
A Reject button for rejecting the application
A Send Mail button for sending a mail

Currently, the footer will disappear as soon as a mail is sent to the applicant in any round.
This works fine but in case the application is rejected and a mail is sent, we would still like to see the drop down for taking further actions.

@rathorevaibhav where is the data related to this custom mail (sender, mail subject, mail message, etc.) going to be saved in the database?


Test

new pull request

Hi, We have accidentally closed our old PR: https://github.com/Somdotta07/To-do-List/pull/7.
We have tried and looked up every possible solution to undo this but to no avail.if there is a way to this, please let us know in the comments before your review, excuse us for the inconvenience.
We have fixed all the issues the last reviewer asked for.


docs(guides): french translation(forms)

Thank you for submitting this PR...
Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines: https://github.com/angular/angular/blob/master/CONTRIBUTING.md#commit-message-format
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Please provide the issue number?
issue #??
Did you finish translating the whole issue?

[ ] no (pr_state: WIP)
[ ] yes (pr_state: PLEASE_REVIEW)

What kind of change does this PR introduce? (check one with "x")

[ ] Translation
[ ] Other... Please describe:

Other information:

I signed it!

@aradjdi peux tu compléter le template de la PR ?

C'est fait @manekinekko


Tests fail with PyICU==2.8

Copied from : https://github.com/NixOS/nixpkgs/pull/143838#pullrequestreview-793587170
regressions with normality tests, they look legitimate
https://github.com/NixOS/nixpkgs/pull/143838

6 packages failed to build:
python38Packages.fingerprints python38Packages.normality python38Packages.scancode-toolkit python39Packages.fingerprints python39Packages.normality python39Packages.scancode-toolkit

6 packages built:
gramps python38Packages.PyICU python38Packages.slob python39Packages.PyICU python39Packages.slob xdxf2slob

Upgrading normality to 2.2.3 doesn't help
__________________________ NormalityTest.test_german ___________________________

self = <tests.test_normality.NormalityTest testMethod=test_german>

    def test_german(self):
        text = u"Häschen Spaß"
>       self.assertEqual("Haschen Spass", ascii_text(text))
E       AssertionError: 'Haschen Spass' != 'Haschen Spa<-ss->'
E       - Haschen Spass
E       + Haschen Spa<-ss->
E       ?            ++  ++

tests/test_normality.py:48: AssertionErro

I was able to reproduce the issue against PyICU 2.8 locally, not quite sure what these brackets are doing there. Since I cannot find any mention of it in the commit logs, I'll try and ping the PyICU maintainers to see if they know what's going on.

Ok, I've added more duct tape but this all a bit gnarly.


Flow Type Support

Having types in JavaScript is a nice feature. TypeScript is one option, however a better one is to user flowtype.
I was trying to integrate flow type with vuestatic based on this tutorial:
https://alligator.io/vuejs/components-flow/
My question is can flow be integrated into vuestic-admin officially?
It has no overhead such as typescript and improves the dev workflow.

Will consider adding this to our roadmap. Thank four your feedback!


AzCopy Does Not Report File Creation Errors Well

Which version of the AzCopy was used?
10.2.1
Which platform are you using? (ex: Windows, Mac, Linux)
Windows
What command did you run?
./azcopy.exe copy "https://md-rsnfv3h3tr2q.blob.core.windows.net/g5lfclt1jpcd/abcd?REDACTED" "C:\Users\REDACTED\Downloads\letsseewhathappens.vhd" --overwrite=false --from-to=BlobLocal --output-type=json
What problem was encountered?
I tried downloading a file too big for my hard drive. When on Windows, the download immediately failed with a non-helpful error message:
"FailedTransfers":[{"Src":"https://md-rsnfv3h3tr2q.blob.core.windows.net/g5lfclt1jpcd/abcd","Dst":"C:/Users/REDACTED/Downloads /letsseewhathappens.vhd","TransferStatus":"Failed","ErrorCode":0}]
In the log file I can see this:
File Creation Error truncate C:/Users/REDACTED/Downloads/letsseewhathappens.vhd: There is not enough space on the disk.
And it looks like maybe AzCopy also tries to clean up the created file but fails? As this error is seen on the next line:
Delete File Error -remove C:/Users/REDACTED/Downloads/letsseewhathappens.vhd: The process cannot access the file because it is being used by another process.. X-Ms-Request-Id:
On macOS, the download does not immediately fail. I assume it fails once enough data has been written to disk that it'll fail, but I didn't have time to test to see what happens. My macOS hard drive has a lot of free space on it.
How can we reproduce the problem in the simplest way?
Download a file which is too big for your disk.
Have you found a mitigation/solution?
Nope.

Job will not fail immediatly as one file may have failed in local creation but other files with smaller size can still succeed. At the end of the job a summary will be printed which will inform exactly how many files were successfully copied and how many failed. From the logs you can also figure out what all files failed to copy. In addition there is a resume job feature which will only try to copy whatever failed last time. If you do not have enough of disk space then you can create that space first and then resume job to continue.


<Cylinder />

Description
As a developer using Tres, I would like to have a <Cylinder /> component in order to easily add a customizable cylinder to scenes.
Suggested solution
Implement a Cientos component with an underlying THREE.CylinderGeometry and accepting the props of the same, with the same defaults.
https://threejs.org/docs/#api/en/geometries/CylinderGeometry
Alternative
No response
Additional context
No response
Validations

[X] I agree to follow this project's Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

Will create a PR for this


⚠ Cioos Pacific CKAN has degraded performance

In b867c47, Cioos Pacific CKAN (https://catalogue.cioospacific.ca) experienced degraded performance:

HTTP code: 200
Response time: 5747 ms

Resolved: Cioos Pacific CKAN performance has improved in e2ddfcd after 5 minutes.


Updating Function primary script determination logic

This change is for the same change made in the Functions runtime (see https://github.com/Azure/azure-webjobs-sdk-script/pull/352).
The plan is for these two changes to go out together in the same release, otherwise things will get out of sync.

:shipit: But hold off on merging to master until we have a clear plan to get the new Functions Runtime out. Let's start this conversation.

Yes, the Functions Runtime won't be ready for another release for at least a week or more. I won't check this in until we're ready on that side. Then we can coordinate the releases.

Rebased and merged.  81bebe79e7ba08c680b168aeb29326a255e9ef14


TimeSeries chart does not render correctly.

Azure Data Studio Version: 1.0.0

Steps to Reproduce:

Run the following query

select 'Counter', getdate(), 100
go

Choose from the chart options TimeSeries

Nothing is displayed.
This has worked all the way up to August Public Preview 0.32.8. It appears to have broken starting with Sept Insiders Build 0.33.7

 ERR "timeSeries" is not a chart type.: Error: "timeSeries" is not a chart type.
    at Chart.<anonymous> (/Users/mairvine/code/sqlopsstudio/node_modules/chart.js/src/core/core.controller.js:336:13)
    at Object.each (/Users/mairvine/code/sqlopsstudio/node_modules/chart.js/src/helpers/helpers.core.js:108:9)
    at Chart.buildOrUpdateControllers (/Users/mairvine/code/sqlopsstudio/node_modules/chart.js/src/core/core.controller.js:318:12)
    at Chart.update (/Users/mairvine/code/sqlopsstudio/node_modules/chart.js/src/core/core.controller.js:391:28)
    at Graph.set [as data] (file:///Users/mairvine/code/sqlopsstudio/out/sql/parts/query/editor/charting/insights/graphInsight.js:108:34)
    at Graph.set [as options] (file:///Users/mairvine/code/sqlopsstudio/out/sql/parts/query/editor/charting/insights/graphInsight.js:190:27)
    at Insight.set [as options] (file:///Users/mairvine/code/sqlopsstudio/out/sql/parts/query/editor/charting/insights/insight.js:52:46)
    at ChartView.buildOptions (file:///Users/mairvine/code/sqlopsstudio/out/sql/parts/query/editor/charting/chartView.js:157:38)
    at Object.set (file:///Users/mairvine/code/sqlopsstudio/out/sql/parts/query/editor/charting/chartView.js:67:34)
    at file:///Users/mairvine/code/sqlopsstudio/out/sql/parts/query/editor/charting/chartView.js:227:63
    at Emitter.fire (file:///Users/mairvine/code/sqlopsstudio/out/vs/base/common/event.js:148:38)
    at HTMLSelectElement.<anonymous> (file:///Users/mairvine/code/sqlopsstudio/out/vs/base/browser/ui/selectBox/selectBoxNative.js:26:36)

@anthonydresser when I run the repo with #2985 merged I see the below output.  I would expect the x-axis to have the date value.  Is this correct?

Now I see the below chart from the repro SQL.  @rgward please reactive if this doesn't look as expected.  I'll publish an updated build on Friday for further validation (or you can build the fix from master in the meantime).