Exoplayer

The aim of this workload is basically to test YouTube on devices like hikey960 where we can't really get real Youtube working.
All of the stuff to get videos is basically stolen from the old 'video' workload, so if/when that gets ported I suppose we'll want to factor that out into some sort of base class?
One thing I'm not sure about is the way I've overridden Parameters from the ApkWorkload class (package_name, and force_install) that aren't really user-definable. Not sure how else to achieve that.
There's also the problem with runtime permissions as noted in the comment  - this only works with force_install because 'pm clear' clears runtime permissions but 'pm grant' doesn't seem to give them back.

cc @valschneider who wrote the LISA worlkload this is stolen from.

I've updated the commit but there are a couple more issues to iron out - I'll get back to you once it's ready for a second round of review.

Given the similarity between workloads, would it make more sense fore the player to be a configurable parameter? (note: I'm not suggesting you add support for the built-in player as part of this pull request)

Hmm, perhaps, although I can imagine that the different players could end up with different sets of tunable parameters...

Hmm, perhaps, although I can imagine that the different players could end up with different sets of tunable parameters...

Yes, true; also, given that in run you're parsing output specific to this player... OK, let's keep this exoplayer for now. I'll revisit this when I get to porting video to WA3; maybe it'd make more sense to keep  them separate and split out common functionality into a utility class...

OK I think I've ironed out the issues now. Two points remain

the validate vs init_resources thing we discussed. I was thinking the solution might be to say validate comes before init_resources, but if you need the resources in validate you are free to call self.init_resources yourself?

I've put a grant_app_permissions call in setup. Do you think that belongs in the superclass?

Forgot to mention - I worked around the validate vs init_resources thing but just having the latter call the former, with a comment. I think this is actually a pretty acceptable solution - the workload knows about dependencies and idempotency of those methods.


Make each RegisterTagEventOpts optional

... so that you can supply a value for one option while keeping the defaults for others.

Also should note that I'm just adjusting the typing, rather than any functionality!

Thanks for the contribution!


Validation appears on top of all controls.

Hello,
The validation border and triangle is showing above of all controls in the window. In my case I'm using a custom alert control when it showin and a validation error occurs, the border and the triangle are shown above the alert control.

Can someone help me to solve this? The Panel.ZIndex property is not working.
Thanks.

@aBerna You can wrap your content with the control which validates inside an AdornerDecorator.
<AdornerDecorator>
  <Grid>
    ...
  </Grid>
</AdornerDecorator>

Great, it works!
Thank you for your help!


Step 6 URL

First off, holy cow - thank you! There's no way an advanced beginner could do this in 40 minutes, but even if it took 2 hours it'd be OK. This took some serious effort and a lot of attention went into really laying it out well.
I did notice that a link is broken on this page (see screenshot here)

Thanks a ton for pointing that out! Fixed.


readability: review the repo, document and comment for readability

Repo has many areas that could use readibility improvements, particularly in folding. Document function stubs, and attempt to improve the readability of the codebase, with further references documented either in readme or some other reasonable location.
Augments repo cleaning performed in #96. Est 2-4h work.

This might be addressed in #103, maybe take a look and let me know what you think.


[eta-tour]: Bug - typo in description

Source Code:
main :: IO ()
main = do
  print $ [1, 2..10]
  print $ take 5 [1,5..100]
  print $ drop 5 [1,2..10]
  print $ replicate 5 "AB"
  print $ head [1..10]
  print $ tail [1..10]
  print $ init [1..10]
  print $ last [1..10]

Output:

numers -> numbers

Implemented this - will be visible on the next deployment tonight. We will include you in a credits list for your valuable feedback. Thanks for your time!


Sanity label check uses output from skopeo inspect task

@dirgim @Michkov @jsztuka @Dannyb48 PTAL.
It can pass by ./hack/test-builds.sh hacbs
Here is my test pipeline https://console-openshift-console.apps.hongliutester.lab.pnq2.cee.redhat.com/k8s/ns/hongweiliu/tekton.dev~v1beta1~PipelineRun/java-builder-run-2gmst/logs/build-container

@hongweiliu17 can you also remove the IMAGE_URL param from sanity-label-check.yaml?
That parameter won't be needed anymore.


Only copy Docker images to Kind if they have changed

Restart the etc-cluster-operator deployment if the image has changed

Closing this because I think it'd be better if we used the kind command line tool to create clusters and load images into the clusters, and it already checks for an existing image ID.


AirVPN: IPv6 only vs IPv4 only but not both?

URL to the Wiki page
https://github.com/qdm12/gluetun-wiki/blob/main/setup/providers/airvpn.md
What's missing?
First, from the AirVPN config page:

⚠ WIREGUARD_ADDRESSES may only contain an IPv6 address (such as ff:ff.../128) if your container setup supports IPv6. To do so, see the IPv6 document. Otherwise, remove the IPv6 address from WIREGUARD_ADDRESSES.

And second, from the IPv6 page:

Wireguard: modify the WIREGUARD_ADDRESSES value to have both an IPv4 and IPv6 address. Note if you only set an IPv6 Wireguard address, all IPv4 traffic won't go through which is undesirable.

Does this mean that gluetun either only supports IPv4 or IPv6 when it comes to AirVPN, but not both?

Does this mean that gluetun either only supports IPv4 or IPv6 when it comes to AirVPN, but not both?

From my tests both IPv4 and IPv6 can be enabled at the same time, and I can see both IPv4 and IPv6 peers in qbittorrent.


Add deployment APIs for Microsoft.Resources

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

Can one of the admins verify this patch?

Automation for azure-sdk-for-js

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/5906
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/5906/commits/1c95204b18f18663ee6c2eafec779296f5ba7eeb

Automation for azure-sdk-for-java
Encountered a Subprocess error: (azure-sdk-for-java)
Command: ['/usr/local/bin/autorest', '/tmp/tmp93ts6kth/rest/specification/resources/resource-manager/readme.md', '--azure-libraries-for-java-folder=/tmp/tmp93ts6kth/sdk', '--java', '--multiapi', '--use=<EMAIL_ADDRESS>'--verbose']
Finished with return code 1
and output:
AutoRest code generation utility [version: 2.0.4283; node: v8.12.0]
(C) 2018 Microsoft Corporation.
https://aka.ms/autorest
   Loading AutoRest core      '/root/.autorest/@microsoft.azure_autorest-core<EMAIL_ADDRESS>(2.0.4302)
   Loading AutoRest extension<EMAIL_ADDRESS>(2.1.85->2.1.85)
   Loading AutoRest extension<EMAIL_ADDRESS>(2.3.38->2.3.38)
Processing batch task - {"tag":"package-features-2015-12"} .
Processing batch task - {"tag":"package-locks-2016-09"} .
Processing batch task - {"tag":"package-policy-2018-05"} .
Processing batch task - {"tag":"package-policy-2018-03"} .
Processing batch task - {"tag":"package-policy-2016-12"} .
Processing batch task - {"tag":"package-resources-2019-05"} .
FATAL: System.InvalidOperationException: Sequence contains no matching element
   at System.Linq.Enumerable.First[TSource](IEnumerable1 source, Func2 predicate)
   at AutoRest.Java.Azure.Fluent.Model.ARMUri.Init() in /home/vsts/work/1/s/src/azurefluent/Model/FluentCommon/ArmUri/ARMUri.cs:line 249
   at AutoRest.Java.Azure.Fluent.Model.ARMUri..ctor(MethodJvaf method) in /home/vsts/work/1/s/src/azurefluent/Model/FluentCommon/ArmUri/ARMUri.cs:line 36
   at AutoRest.Java.Azure.Fluent.Model.SegmentFluentMethodGroups.InnerMethodGroupToSegmentFluentMethodGroups(CodeModelJvaf codeModel) in /home/vsts/work/1/s/src/azurefluent/Model/FluentCommon/FluentMethodGroup/SegmentFluentMethodGroup/SegmentFluentMethodGroups.cs:line 90
   at AutoRest.Java.Azure.Fluent.CodeGeneratorJvaf.<Generate>d__6.MoveNext() in /home/vsts/work/1/s/src/azurefluent/CodeGeneratorJvaf.cs:line 55
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at AutoRest.Java.Program.<ProcessInternal>d__3.MoveNext() in /home/vsts/work/1/s/src/Program.cs:line 114
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at NewPlugin.<Process>d__20.MoveNext() in /home/vsts/work/1/s/autorest.common/src/Plugins/NewPlugin.cs:line 163
FATAL: java/generate - FAILED
FATAL: Error: Plugin java reported failure.
Process() cancelled due to exception : Plugin java reported failure.
Failure during batch task - {"tag":"package-resources-2019-05"} -- Error: Plugin java reported failure..
  Error: Plugin java reported failure.

  "name": "groupId",

This should be called managementGroupId to make it obvious.

Refers to: specification/resources/resource-manager/Microsoft.Resources/stable/2019-05-01/resources.json:3884 in dff86dc. [](commit_id = dff86dcb7f32d5b58147c195161d217933bfce85, deletion_comment = False)

  "description": "The name of the deployment to delete.",

The description doesn't match what this is.

Refers to: specification/resources/resource-manager/Microsoft.Resources/stable/2019-05-01/resources.json:3888 in dff86dc. [](commit_id = dff86dcb7f32d5b58147c195161d217933bfce85, deletion_comment = False)

      },

I wonder if we should refactor this into a parameter we reference like the others. If you agree to do it, we probably should only do it in 2019-05-01.

Refers to: specification/resources/resource-manager/Microsoft.Resources/stable/2019-05-01/resources.json:117 in dff86dc. [](commit_id = dff86dcb7f32d5b58147c195161d217933bfce85, deletion_comment = False)

  "name": "groupId",

I was referring to what MG group team did for their APIs. They use "groupId" for this parameter..

In reply to: 485941229 [](ancestors = 485941229)

Refers to: specification/resources/resource-manager/Microsoft.Resources/stable/2019-05-01/resources.json:3884 in dff86dc. [](commit_id = dff86dcb7f32d5b58147c195161d217933bfce85, deletion_comment = False)

  "description": "The name of the deployment to delete.",

will fix this one.

In reply to: 485941319 [](ancestors = 485941319)

Refers to: specification/resources/resource-manager/Microsoft.Resources/stable/2019-05-01/resources.json:3888 in dff86dc. [](commit_id = dff86dcb7f32d5b58147c195161d217933bfce85, deletion_comment = False)

@Tiano2017, the PutDeploymentAtManagementGroup.json file appears to be empty. And you've also got some checks failing right now.

  "name": "groupId",

ok

In reply to: 485942694 [](ancestors = 485942694,485941229)

Refers to: specification/resources/resource-manager/Microsoft.Resources/stable/2019-05-01/resources.json:3884 in 561b51c. [](commit_id = 561b51cebda50c09a05e7bf1719163a74f0caa47, deletion_comment = False)

Yeah. I'll get a sample for that.

In reply to: 485943610 [](ancestors = 485943610)

      },

I think we can do that.

In reply to: 485942143 [](ancestors = 485942143)

Refers to: specification/resources/resource-manager/Microsoft.Resources/stable/2019-05-01/resources.json:117 in dff86dc. [](commit_id = dff86dcb7f32d5b58147c195161d217933bfce85, deletion_comment = False)

@Tiano2017 does this PR has the new operation that we discussed offline?
Is this ready to be reviewed?

@Tiano2017 does this PR has the new operation that we discussed offline?
Is this ready to be reviewed?

Not yet.. Let me close on that and will let you know once it's ready for review.

@shahabhijeet could you have a look one more time.

@Tiano2017 lost entire context :), will sync with @sergey-shandar and will get back on this PR by EOD today.


Create 0000-01-02-subtitleuk.md

Go me...

Very easy to follow and I feel confident to start using github via the web interface.  Job well done, thank you


CIAO use Keystone for projects/tenants handling

CIAO is using sqlite to store tenants/projects in its own side, projects being a common asked resource should be stored only in one database, so any service calling/requesting data about them must 100% accurate for any service.
CIAO projects handling should be delegated to keystone.

We cache tenant/project information to provide the level of performance that we require.  Can you please let me know what specific problem you are seeing?

Rally benchmarking simulates real user load.
Boot scenario creates a temporal project through Keystone API, then sends the workload to with project-id to compute API, which CIAO tries to verify it using its own sqlite database, so it does not make match because  CIAO's database does not have the project created through Keystone API, therefore rally cannot continue because of that.

The flow you described is not correct.  If a workload is requested to be started and the tenant information is not in ciao's cache, the tenant information will be added and the start workload will be allowed to proceed. So there is another reason why this call is failing, and we should debug that. Can you tell me what the exact api rally is using and what the response code is?

The ciao-controller doc.go should describe the caching, how coherency is maintained, that our API support isn't yet complete, and how to debug a problem.

@kaccardi  you are right, I tried to make a whole new test, restarting everything in my environment, and now it shows the adding process as you mentioned.
Now I can see the
exec: INSERT or IGNORE into tenants VALUES ('7cbb7a272bbe4e4a9c6a30b2be946dd2','','',$
02:c7:a0:40:0b:f1','');

My environment must had gone into a wrong state because it wasn't happening, but glad it works now. Now I'm into https://github.com/01org/ciao/issues/166, I'm taking a look into it, see if I can contribute on that.
I'm closing this issue,  +1 for @tpepper  comment
Thanks guys


"Backward" and "reverse" shouldn't both be available

Just the other day I was tagging some railroad signals  and the prompt system suggested "reverse", so I picked it.
But then I noticed that other people's signals had this nice kind of flashlight effect on them. Oh they had tagged their signals "backward".
I looked in the selection list menu, and lo and behold backward was just a little bit further down than reverse. But that's only on some days.
Anyway you want to get the bad one out and leave the good one in, not let users just accidentally pick the wrong one.

I still haven't checked the wiki to see which one is correct. All I know is if it gets special effects on the map (in iD) and the other one doesn't,  that's the one I'll use.
Yup, "editing for the (editor) renderer," that's me!

The direction field for railway signals (railway/signal/direction, which is used by the Railway Signal preset) only has forward,  backward and both as allowed options. Do you mean a different preset perhaps?

He is talking about the suggestions from taginfo which appear in the tag editor. I've already fixed most of the instances of that tag and they shouldn't appear.

OK. I'm however still wondering which field or preset is affected by this issue. Because the direction field of the "Railway Signal" preset does not trigger a taginfo lookup as far as I can see. I'd be interested in knowing which field might still have this bug.
Or did you got this in the raw tags editor @jidanni?

I apologize that it's hard to explain / remember exactly what happened. And another difficulty is it seems to learn with what one uses  too.
So maybe all the places that one might encounter the wrong values have now been plugged up by...
So I'll just close this.


[CI] llama: add more CI jobs

Hello, I am a code review bot on flows.network.
It could take a few minutes for me to analyze this PR. Relax, grab a cup of coffee and check back later. Thanks!


[fix](mark join) fix bug of mark join with other conjuncts

Proposed changes
Issue Number: close #xxx
Problem summary
Fix bug that probe_index is not increased for mark hash join with other conjuncts.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.72 seconds
stream load tsv:          476 seconds loaded<PHONE_NUMBER>9 Bytes, about 149 MB/s
stream load json:         36 seconds loaded<PHONE_NUMBER> Bytes, about 62 MB/s
stream load orc:          68 seconds loaded<PHONE_NUMBER> Bytes, about 15 MB/s
stream load parquet:          28 seconds loaded 861443392 Bytes, about 29 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230213043607_clickbench_pr_94979.html


105 Notebook Error

When I run this on Windows, I got this error.
'unzip' is not recognized as an internal or external command,
operable program or batch file.

I would recommend use the Python library and so we don't depend on the user's environments.
(e.g., import zipfile)
@ryanloney - what's your setup on Windows?

Also, we shall standardize all 'curl' downloads with the progress bar UI. That was one of the key UX requirements we discussed much much earlier.

Thanks for catching this @raymondlo84, and for the fix @Debskij .


Bonding breaks secure connection

Hey, I am trying to use bonding and secure connection on 2 esp32-s3's.
Using your examples of secure server and secure client everything works the first time. But after I reload the client, they connect, but the client hangs on secure connection, and the server prints, that an unhandled event happens (esp32_nimble::server::ble_server: unhandled event: 17) (If i checked correctly, 17 is BLE_GAP_EVENT_REPEAT_PAIRING). After that client just times out.
Without bonding everything works just fine.
I used your examples without changing them, so I don't think I have to post the code.
I am using esp-idf v4.4 with the help of esp-idf-template, if that's in any way helpful.

Thanks for the report.
I fixed the bug.
You can check the operation by adding the following to your Cargo.toml.
[patch.crates-io]
esp32-nimble = { git = 'https://github.com/taks/esp32-nimble.git', rev = "63015eec2cef5575abf491f1e3795ffe69f6cc04" }

Yes, everything works as expected now, thank you!


Add new file type

Add new type (sql and chart)
fixes #91

Please make sure that all the checks pass. Please add here any additional information regarding this pull request. It's highly recommended that you link this PR to an issue (please create one if it doesn't exist for this PR)

@roll it is ready for the review. Thanks!


Unresponsive 'Publish Idea' button

This is a Bug Report
:tophat: Description
For bug reports:

What went wrong?
'Publish Idea' button is unresponsive when creating a proposal. It simply doesn't become active after filling out required fields. I created a fresh git clone, npm install, and got meteor v1.5.1
What did you expect should have happened?
What was the config you used?
Meteor v1.5.1, running a regular local deploy
What stacktrace or error message from your provider did you see?

:pushpin: Related issues
:clipboard: Additional Data

Sovereign deployment where you found the issue:
Browser & version:
Latest Chrome
Screenshot:
Error messages:
URL to reproduce the error:

@santisiri do you see this same behavior from your end? Checked in with @virgile-dev and he also sees the same behavior. Seems the alpha version is working fine. Also, I was seeing this behavior before commit 7d9b756a413b7380a3ea47216566112a0e08fa50

I think I had the same issue, and fixed it in #198 .
It's a desktop-only issue, if memory serves.

Hey @domi41 yeap, was literally just looking into that when you commented. Looks like rightToVote is causing the issue, which is involved in #198 as you mention. Will continue debugging

rightToVote is never getting set because the code that does so was commented out in commit 0c6c0982a654b25b075e8d6b655636016560ae82.
imports/startup/client/functions.js
// status of action button
    if (contract.kind === 'DELEGATION') {
      Session.set('rightToVote', verifyDelegationRight(contract.signatures));
    } else if (contract.kind === 'VOTE' && contract.stage === 'DRAFT') {
      Session.set('rightToVote', true);
      Session.set('alreadyVoted', false);
    } else if (contract.kind === 'VOTE' && contract.stage === 'LIVE') {
      Session.set('rightToVote', true);
      Session.set('alreadyVoted', verifyVotingRight(contract._id));
    } else if (contract.kind === 'VOTE' && contract.stage === 'FINISH') {
      Session.set('rightToVote', false);
    }

@santisiri will wait to hear from you because I don't want to interfere with what was started in that commit. But FYI this is sort of a blocker because it inhibits testing for development. I will uncomment and test forth, but probably best to wait before a PR?

Not an issue anymore as code has been refactored


⚠ Samsung has degraded performance

In ecf720a, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 3202 ms

Resolved: Samsung performance has improved in 6809f08.


flutter install error  command flutter doctor

Flutter crash report; please file at https://github.com/flutter/flutter/issues.
command
flutter doctor
exception
VersionCheckError: VersionCheckError: Command exited with code 128: git log -n 1 --pretty=format:%ad --date=iso
Standard error: fatal: bad config line 1 in file C:\ProgramData/Git/config
#0      _runSync (package:flutter_tools/src/version.dart:506:5)
#1      FlutterVersion._latestGitCommitDate (package:flutter_tools/src/version.dart:130:12)
#2      FlutterVersion.frameworkCommitDate (package:flutter_tools/src/version.dart:121:37)
#3      FlutterVersion.frameworkDate (package:flutter_tools/src/version.dart:83:31)
#4      FlutterValidator.validate (package:flutter_tools/src/doctor.dart:452:127)
<asynchronous suspension>
#5      Doctor.startValidatorTasks (package:flutter_tools/src/doctor.dart:116:52)
#6      Doctor.diagnose (package:flutter_tools/src/doctor.dart:189:41)
<asynchronous suspension>
#7      DoctorCommand.runCommand (package:flutter_tools/src/commands/doctor.dart:48:39)
<asynchronous suspension>
#8      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:545:18)
#9      _asyncThenWrapperHelper.<anonymous closure> (dart:async/runtime/libasync_patch.dart:77:64)
#10     _rootRunUnary (dart:async/zone.dart:1132:38)
#11     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#12     _FutureListener.handleValue (dart:async/future_impl.dart:126:18)
#13     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:639:45)
#14     Future._propagateToListeners (dart:async/future_impl.dart:668:32)
#15     Future._complete (dart:async/future_impl.dart:473:7)
#16     _SyncCompleter.complete (dart:async/future_impl.dart:51:12)
#17     _AsyncAwaitCompleter.complete.<anonymous closure> (dart:async/runtime/libasync_patch.dart:33:20)
#18     _rootRun (dart:async/zone.dart:1124:13)
#19     _CustomZone.run (dart:async/zone.dart:1021:19)
#20     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:947:23)
#21     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#22     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#23     _runPendingImmediateCallback (dart:isolate/runtime/libisolate_patch.dart:115:13)
#24     _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:172:5)

flutter doctor
encountered exception: VersionCheckError: Command exited with code 128: git log -n 1 --pretty=format:%ad --date=iso
Standard error: fatal: bad config line 1 in file C:\ProgramData/Git/config

#0      _runSync (package:flutter_tools/src/version.dart:506:5)
#1      FlutterVersion._latestGitCommitDate (package:flutter_tools/src/version.dart:130:12)
#2      FlutterVersion.frameworkCommitDate (package:flutter_tools/src/version.dart:121:37)
#3      FlutterVersion.frameworkDate (package:flutter_tools/src/version.dart:83:31)
#4      FlutterValidator.validate (package:flutter_tools/src/doctor.dart:452:127)
<asynchronous suspension>
#5      Doctor.startValidatorTasks (package:flutter_tools/src/doctor.dart:116:52)
#6      Doctor.diagnose (package:flutter_tools/src/doctor.dart:189:41)
<asynchronous suspension>
#7      _doctorText.<anonymous closure> (package:flutter_tools/runner.dart:188:26)
#8      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142:29)
<asynchronous suspension>
#9      _rootRun (dart:async/zone.dart:1124:13)
#10     _CustomZone.run (dart:async/zone.dart:1021:19)
#11     _runZoned (dart:async/zone.dart:1516:10)
#12     runZoned (dart:async/zone.dart:1463:12)
#13     AppContext.run (package:flutter_tools/src/base/context.dart:141:18)
<asynchronous suspension>
#14     _doctorText (package:flutter_tools/runner.dart:187:19)
<asynchronous suspension>
#15     _createLocalCrashReport (package:flutter_tools/runner.dart:165:32)
<asynchronous suspension>
#16     _handleToolError (package:flutter_tools/runner.dart:120:33)
<asynchronous suspension>
#17     run.<anonymous closure> (package:flutter_tools/runner.dart:64:20)
<asynchronous suspension>
#18     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142:29)
<asynchronous suspension>
#19     _rootRun (dart:async/zone.dart:1124:13)
#20     _CustomZone.run (dart:async/zone.dart:1021:19)
#21     _runZoned (dart:async/zone.dart:1516:10)
#22     runZoned (dart:async/zone.dart:1463:12)
#23     AppContext.run (package:flutter_tools/src/base/context.dart:141:18)
<asynchronous suspension>
#24     runInContext (package:flutter_tools/src/context_runner.dart:48:24)
<asynchronous suspension>
#25     run (package:flutter_tools/runner.dart:51:10)
#26     main (package:flutter_tools/executable.dart:52:9)
<asynchronous suspension>
#27     main (file:///C:/flutter/flutter/packages/flutter_tools/bin/flutter_tools.dart:8:3)
#28     _startIsolate.<anonymous closure> (dart:isolate/runtime/libisolate_patch.dart:298:32)
#29     _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:171:12)

Duplicate of #28877

Oops, not a dup. The error code is the same, but the problem is a local git misconfiguration:

Standard error: fatal: bad config line 1 in file C:\ProgramData/Git/config

You'll need to fix your git config if you haven't already.
It seems like in general the first step for all of these problems would be to gate all git calls through a helper function that will tool-exit with the output from git instead of crashing. In many cases (here, several variants of Xcode-not-configured-breaks-git) the output already explains how to fix the problem.
Then for common specific cases like #28877, they could be added as special cases in that wrapper, providing a custom error message.
@jonahwilliams Is there a bug tracking something like that already?

No, but it sounds like a good idea.

I'm fixing the doctor crash at https://github.com/flutter/flutter/pull/44868.  I don't think this is tracking anything unique since this was specific to doctor, and there was https://github.com/flutter/flutter/issues/28939 and  other issues open for flutter upgrade https://github.com/flutter/flutter/issues/42565 and flutter --version https://github.com/flutter/flutter/issues/34142.


Make root option on IntersectionSignal able to use a signal.

the Root options currently takes a value.  It could be that the root changes.  This will probably require a refactor to the base dom-observer-base since that options is a constructor option.

I doubt there's any demand for this.


⚠ GitLab Server has degraded performance

In 63a3e26, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11576 ms

Resolved: GitLab Server performance has improved in 81583f0.


Fix #15656: Error in the documentation of widget theming

Q
A

Is bugfix?
no

New feature?
no

Breaks BC?
no

Tests pass?
no

Fixed issues
#15656

[ci skip]

Merged. Thank you!


bug fix pls

there is this bug called "no documentation"
pls fix it asap

there is nothing to document until now


user docs: Write a "zulip for slack users" guide.

We can put it as the third guide under here, on /help

No, it's just for people using Zulip for the first time, but whose first/main experience with team chat was Slack.

@zulipbot claim


Minor update to the clang-format file for better readability

Modified .clang-format and applied it to src/ and tests/ directory files. Highlights are as follows

Added block indent for alignment after open bracket
Increased the column limit to 101
Fixed the include categories to improve grouping

@ddement Not sure if you wanted to review this some more, if not, I can go ahead and merge this one.


Tried to run it in Android Studio 1.0 and failed to build

when trying to import it into Android studio 1.0, there comes an error :
Error:(15, 0) Could not find property 'Compile' on project ':Packet Sender'.
and
Error:The project is using an unsupported version of Gradle.

This project was originally built with early Android Studio. I recommend using the code as a reference instead of a base.


openai-whisper-cpp: 1.2.0 -> 1.2.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/ggerganov/whisper.cpp/releases.
meta.description for openai-whisper-cpp is: Port of OpenAI's Whisper model in C/C++
meta.homepage for openai-whisper-cpp is: https://github.com/ggerganov/whisper.cpp
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.2.1 with grep in /nix/store/1v2jzybqhzcbznllnawilmz779z5p78g-whisper-cpp-1.2.1
found 1.2.1 in filename of file in /nix/store/1v2jzybqhzcbznllnawilmz779z5p78g-whisper-cpp-1.2.1

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
openai-whisper-cpp

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1v2jzybqhzcbznllnawilmz779z5p78g-whisper-cpp-1.2.1
ls -la /nix/store/1v2jzybqhzcbznllnawilmz779z5p78g-whisper-cpp-1.2.1/bin

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

1 package built:

openai-whisper-cpp

Maintainer pings
cc @dit7ya for testing.


Add larger-scale batch testing

Passes fine locally.

No longer valid, as integration testing is getting re-worked in https://github.com/influxdb/influxdb/pull/1788


Fix some memory leakage found till now

Memory allocated in asn1p_l.c and asn1p_y.c are freed.
Remaining two leaked memory blocks are allocated inside function asn1f_printable_value() and asn1c_make_identifier() hold by static variables. Please refer to Valgrind log.
asn.1 modules under examples directories and some more asn.1, ex. LTE-RRC, S1AP, RANAP, DSRC, NBAP are used for verification.
When the parser ends with unexpected token or grammar error, memory allocated by asn1_module_new().... etc are not able freed because asn1p_parse() does not return a valid asn1p_t * pointer to caller. Perhaps adding a suitable hook to call asn1p_free() inside parser is needed.
Of course, there might still be un-found leakages exist in other asn,1 syntax cases.

Coverage increased (+0.05%) to 77.905% when pulling 3dcf05bbb066f6c553beed1aee51e896532599e4 on brchiu:vlm_master_fix_most_memory_leakage into 0eca8c322749a501f1a10f47f58f99a793bbe8b8 on vlm:master.


Test 167 (pytorch/rcfr_pytorch_test.py) failed when building from source on mac os

Building from source from master (haven't done it in a while, so I could definitely be doing something wrong?)
Get the error:
FAIL: test_rcfr_functions (__main__.RcfrTest)
RcfrTest.test_rcfr_functions
----------------------------------------------------------------------
Traceback (most recent call last):
  File ".../open_spiel/python/pytorch/rcfr_pytorch_test.py", line 470, in test_rcfr_functions
    self.assertLess(pyspiel.nash_conv(_GAME, average_profile), 0.91)
AssertionError: 1.4048300405772975 not less than 0.91

What I did:

pull latest master
./install.sh
pip install -r requirements.txt
./open_spiel/scripts/build_and_run_tests.sh

and that's when the test fails.
Anyone else run into this before?  All other tests pass just fine.

This was one test in a series that I changed recently when upgrading pytorch versions.
Can I check which version of pytorch you are using?

Here is the commit with those changes, if it helps:
https://github.com/deepmind/open_spiel/commit/5111a3591df9f26a7b6c11d68e3660b1aedd1293

I'm apparently still on pytorch 1.7.0

confirmed that upgrading to the expected pytorch passes the test (and it was consistently failing before):
pip install --upgrade torch==1.11.0
./open_spiel/scripts/build_and_run_tests.sh --test_only="rcfr"
passes.

So is there a bug where running the install scripts should have updated pytorch and didn't? (I didn't set any env vars when running the scripts, so they are all on auto.) Or if that's expected behavior, then is there a bug where the pytorch tests shouldn't have run for me but they did?

This is all intended behavior, it's just a rare case because the change in the commit above caused an incompatibility with the old version of PyTorch.
Last year, we made most of the Python packages optional dependencies. This is because most of the python functionality is "extra" and we didn't want to cause python dependency nightmares for any projects that depend on us for mostly optional features (we were pegging versions quite strongly). So now we only have 4 real required python packages, and the "canonical versions" for the optional ones are listed in scripts/python_extra_deps.sh.
Our build and run test script do not install these optional extra deps, it (mostly) detects them and enables the right python tests based on what's found. It's only on github Actions CI that we actually install and upgrade everything to those python versions listed in the extra deps.
I think this is the currently the best compromise, because we don't want to be forceful about upgrading user's python package versions locally, but it does mean breakage in these rare cases.

ok, that sounds reasonable, thanks for the reponse! Is it not desirable or not feasible to check the user's pytorch version before deciding whether or not to run the pytorch tests?
My 2 cents: It feels to me like the behavior I encountered is pretty bad (since there's no error message or anything indicating what was wrong) and I don't see why it would be rare: anytime OpenSpiel upgrades a dependency, there's a good chance there can be backwards-incompatible changes. It's fortunate that the test failed here, otherwise the issue would have been much more insidious if I was running pytorch code and it wasn't working.
I guess this means that running and failing the test was actually good, so my solution above is actually undesirable.  Maybe the best and easiest solution is just to add a line to install.md about manually running pip install --upgrade the extra dependencies if using python?

Yeah I thought it would be more common myself, but turns out most projects tend to be backward-compatible. This is only the second time something like this has happened since August 2019, and we upgrade the versions 2-3 times per year (and the other time wasn't even python packages, it was that we upgraded the version of abseil).
I like the idea of mentioning the versions and pointing to that file in the install docs. Do you want to submit a PR for it?


Fix runtime exceptions for "newtmgr conn add/delete" and update help text for newtmgr

Fixed runtime exception when no argument is given to the  newtmgr conn add and the newtmgr conn delete command.
Updated help texts.

Seems like you're fixing some of the go formatting. Could you fix the other things 'gofmt' is not happy with in runtest.go as well? Thanks!

Hi Aditi
I added "remote" to the newtmgr help text

Hi Marko,
Umm, my original commit should have the gofmt cleanup for runtest.go.  I just did a gofmt -d runtest.go on my local branch to verify that I committed the changes, and gofmt -d doesn't indicate any differences. Are you still seeing things that gofmt is not happy with for runtest.go?

Maybe our gofmt are of different version? Thanks for doing the changes.


How to restore original packages for Ubuntu 18.04

Microsoft Windows [Version 10.0.17134.648]
Ubuntu 18.04 bionic from Win store
After installation, I did the "update & upgrade" for packages. But when I tried to install AFNI and SUMA, it turned out that a lot of needed shared libaries were gone. It is hard to install all the lost libaries one by one.
How can I restore the packages to the original ones (while keeping AFNI and SUMA)?

Ubuntu forums are here. They will probably want to know what a AFNI and a SUMA are when you post.


TDR-2317 - Get consignment by user id

If the user has reporting access then return all the consignments else return the current user’s consignments only
Please have a look and let me know your thoughts. I'm not sure if this is good enough to validate and return the consignments.

I think we can still use ValidateHasReportingAccess check, and:

pass the user id into the query;
use the user id to then check it matches the user id on the token. There is access to the arguments passed into a query on the context object in the validate functions. See for example: https://github.com/nationalarchives/tdr-consignment-api/blob/master/src/main/scala/uk/gov/nationalarchives/tdr/api/auth/AuthorisationTags.scala#L39

See the psuedo code below. That way, we're still using a validation check, and a user should only have access to their own consignments through the query.
...
consignments(... userId: UUID) {
  ...
  tags = List(ValidateHasReportingAccess)
}

...

//probably rename this to something like: ValidateHasConsignmentsAccess
object ValidateHasReportingAccess extends SyncAuthorisationTag {
  override def validateSync(ctx: Context[ConsignmentApiContext, _]): BeforeFieldResult[ConsignmentApiContext, Unit] = {
    val userId = ctx.arg("userId")
    val token = ctx.ctx.accessToken
    val reportingAccess = token.reportingRoles.contains(reportingRole)
    if (reportingAccess || userId == token.userId) {
      continue
    } else {
      val tokenUserId = token.userId
      throw AuthorisationException(s"User $tokenUserId does not have permission to run reporting")
    }
  }
}

...

ctx.arg


New version: AIMP.AIMP version 5.40.2653

Pull request has been created with komac v2.8.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

@wingetbot run

[Policy] Needs-CLA
[Policy] Needs-Author-Feedback


Library fails when account has no refunds

I'm trying to use the YNAB script, and it fails with the following error:
{"level":"info","message":"Retrieving reports from Amazon","metadata":{},"timestamp":"2021-01-05T02:26:13.784Z"}
(node:23752) UnhandledPromiseRejectionWarning: Error: Invalid Record Length: columns length is 18, got 1 on line 2
    at Parser.__onRow (C:\snapshot\amazon-ynab-sync\node_modules\csv-parse\lib\index.js:762:9)
    at Parser.__parse (C:\snapshot\amazon-ynab-sync\node_modules\csv-parse\lib\index.js:635:40)
    at Parser._flush (C:\snapshot\amazon-ynab-sync\node_modules\csv-parse\lib\index.js:453:22)
    at Parser.prefinish (_stream_transform.js:142:10)
    at Parser.emit (events.js:315:20)
    at prefinish (_stream_writable.js:618:14)
    at finishMaybe (_stream_writable.js:626:5)
    at Parser.Writable.end (_stream_writable.js:570:5)
    at ReadStream.onend (_stream_readable.js:677:10)
    at Object.onceWrapper (events.js:421:28)

Looking at my Amazon order history reports, the only reports generated by the library were of type Refund. This is what one of them looks like:
Order ID,Order Date,Title,Category,ASIN/ISBN,Website,Purchase Order Number,Refund Date,Refund Condition,Refund Amount,Refund Tax Amount,Tax Exemption Applied,Refund Reason,Quantity,Seller,Seller Credentials,Buyer Name,Group Name
No data found for this time period

Fixed in v3.3.2

Fixed in v3.3.2

This should now be fixed in https://github.com/starsprung/amazon-ynab-sync, I've updated it to the latest version of this library.

This should now be fixed in https://github.com/starsprung/amazon-ynab-sync, I've updated it to the latest version of this library.

Awesome. Thanks!

Awesome. Thanks!


Support dynamic batch size in online_norm for tensorflow

Currently in online_norm (not batch_online_norm) we store intermediate
outputs from forward pass for backward pass in a tensorflow variable. Tensorflow variables have fixed shape which makes it impossible to use one OnlineNorm layer in two places with different batch sizes.
In my use case (and I assume generally in Reinforcement Learning) I am using one tensorflow layers
with different batch sizes in two places - in policy rollouts and in policy updates.
I noticed that outputs can be passed from forward to backward via reference (and not via mutable tensorflow variable).
In this PR I implement that change making OnlineNorm layer batch-size agnostic and getting rid of unnecessary b_dim argument.

ON is created to run at batch size one. If the batch size is larger than batch size one then you would need to loop through the samples in the batch. If you know the batch size apriori, you can play some tricks to get around having to loop through the batch. This is implemented in batch_online_norm.py
While this code does run it will not be doing what the ON paper advertises.

but passing output from forward to backward via reference is a good idea.

Thanks for reply!
So batch_size in 'online_norm.py ' must be equal to 1?
And for B>1 I should use batch_online_norm.py?
It the 'batch_size=1' parameter kept for documentation purposes? Or is there some use-case for batch_size > 1 in 'online_norm.py' ?

So batch_size in 'online_norm.py ' must be equal to 1?
And for B>1 I should use batch_online_norm.py?

is correct.
By definition Online Normalization operates on one sample at a time. online_norm can be used on a batch of data if you put it into a loop which processes one sample at a time. This loop is then slow, so we vectorized the operation in batch_online_norm using a FIR filter to replace the loop. This FIR filter approach is much faster but needs to know the batch size apriori. Another alternative would be to write a CUDA kernel for online norm.
In the online_norm code batch_size=1 was for different norm technique we were looking into. It was similar but different. I think I left it there on accident when I was cleaning up the code for publishing and could probably be removed (especially if we use a reference for output).

Support for a dynamic batch sizes requires a loop to loop over the samples in the batch. In TF / PyTorch / C this is slow. The best solution is to write a CUDA kernel which does the looping. I have done a first pass at a CUDA implementation and while it is fast our organization has decided not to pursue perfecting the kernel for publishing.

Updt:
I wrote a CUDA kernel that accelerates OnlineNorm. It has been incorporated into PyTorch and will be integrated into the TensorFlow implementation shortly. I'll update you again when this is done.
Once integrated, this should enable dynamic batch sizes.
-Vitaliy

Updt:
With https://github.com/Cerebras/online-normalization/pull/12 dynamic batch sizes are fully supported.


Unable to open dbeaver

Description
Hi team,
I'm unable to open dbeaver app in my VD, but when I run it as different user it does work normally.
Version:-DBeaver Community DBeaver 21.0.4
OS:- Windows 10
steps to reproduce the issue:- When click on application shortcut, nothing happens
DBeaver Version
DBeaver Community DBeaver 21.0.4
Operating System
Windows 10
Database and driver
PostgreSQL
Steps to reproduce

When click on application shortcut,
nothing happens

Additional context
No response

Hello @SagarKaro,
Try uninstalling existing installations and installing DBeaver for all users using the .msi installer.
Please let me know if it helps.

HI @@.> @@.>,
Out support team have tried this more than several times and the issue isn’t resolved.
As I mentioned, it work when it is accessed as different user. I guess it just happening with my access only, even though it’s a community edition which I think don’t require any purchase and opensource.
Please try to give me resolution asap.
Thanks And Regards,
Sagar Karokar
Infosys Ltd. | Contracted to Pfizer Inc.
@.***
From: ShadelessFox @.>
Sent: Friday, June 30, 2023 9:47 PM
To: dbeaver/dbeaver @.>
Cc: Karokar, Sagar @.>; Mention @.>
Subject: [EXTERNAL] Re: [dbeaver/dbeaver] Unable to open dbeaver (Issue #20440)
Hello @SagarKarohttps://urldefense.com/v3/__https:/github.com/SagarKaro__;!!H9nueQsQ!6kmRSoJBa65P74JMn0twaK_Z7WtJtqFHlnaukXp2IJvx6zhdjrVFAYLyPq__yVg3qw0C6lOHXkRAcy3qpf-FyBYHMjxb$,
Try uninstalling existing installations and installing DBeaver for all users using the .msi installer.
Please let me know if it helps.
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/dbeaver/dbeaver/issues/20440*issuecomment-1614882027__;Iw!!H9nueQsQ!6kmRSoJBa65P74JMn0twaK_Z7WtJtqFHlnaukXp2IJvx6zhdjrVFAYLyPq__yVg3qw0C6lOHXkRAcy3qpf-FyFb52h0A$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/BA6RLULAZ766OCYETMTXYYDXN33V5ANCNFSM6AAAAAAZZZPMUA__;!!H9nueQsQ!6kmRSoJBa65P74JMn0twaK_Z7WtJtqFHlnaukXp2IJvx6zhdjrVFAYLyPq__yVg3qw0C6lOHXkRAcy3qpf-FyFbozk5I$.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Without the understanding on what is going on, we can't help, unfortunately.
But you might try the following:

Run the executable itself. It can be located via the properties of the shortcut
Run the executable as the administrator
Look for log files and, if present and they are recent, attach here (on GitHub, not by email, please)

Sure,
I’ll get back to you with details.
Thanks And Regards,
Sagar Karokar
Infosys Ltd. | Contracted to Pfizer Inc.
@.***
From: ShadelessFox @.>
Sent: Monday, July 3, 2023 5:17 PM
To: dbeaver/dbeaver @.>
Cc: Karokar, Sagar @.>; Mention @.>
Subject: [EXTERNAL] Re: [dbeaver/dbeaver] Unable to open dbeaver (Issue #20440)
Without the understanding on what is going on, we can't help, unfortunately.
But you might try the following:

Run the executable itself. It can be located via the properties of the shortcut
Run the executable as the administrator
Look for log fileshttps://urldefense.com/v3/__https:/github.com/dbeaver/dbeaver/wiki/Log-files*log-files__;Iw!!H9nueQsQ!_tRYrEgYlYHWqVEhOhyKDdthZvvq4dFtonQyPK0Q1PE_D77rKdfI6DanQSo3PzTUJUpVGOVt7NaBInD34mKKgtJYjgO3$ and, if present and they are recent, attach here (on GitHub, not by email, please)

—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/dbeaver/dbeaver/issues/20440*issuecomment-1618060478__;Iw!!H9nueQsQ!_tRYrEgYlYHWqVEhOhyKDdthZvvq4dFtonQyPK0Q1PE_D77rKdfI6DanQSo3PzTUJUpVGOVt7NaBInD34mKKgo_RNPM4$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/BA6RLUOTIQJ33UYFCR5WG23XOKWLPANCNFSM6AAAAAAZZZPMUA__;!!H9nueQsQ!_tRYrEgYlYHWqVEhOhyKDdthZvvq4dFtonQyPK0Q1PE_D77rKdfI6DanQSo3PzTUJUpVGOVt7NaBInD34mKKgiRUbsDg$.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Please Check above Screenshot, as I am using community edition is it fine I’m getting asked for authentication, can this be the cause of issue?
It pops up every time I restart my VD

Sorry, but I don't know what this is. This is not part of DBeaver. Please get in touch with your system administrator.

It's been a while since no update here. If the issue is still actual please let me know and provide additional information.

Uninstall and reinstall, make sure install for all user and reset settings while install


Makefile: fix DESTDIR environment variable behaviour

The DESTDIR environment variable is used in the wrong way: since 30+ years
it's common practise using it for specifying temporary target root
(where eg. packaging infrastructure picks up the install image), instead
of the installation prefix on the final target.
Fixing this by introducing PREFIX variable (with default /usr/local) and
using both variables according to the 30 years matured common practise.
That way, distro packagers and other standardized build/installation
systems can easily do correct deployments w/o individual hacks.
Signed-off-by: Enrico Weigelt, metux IT consult<EMAIL_ADDRESS>

Please add a documentation to explain that the behavior of DESTDIR was changed in this release (v1.6).

DESTDIR is used by scripts and downstream makefiles that we use for local and release installs. There are some release overrides for DESTDIR in the makefile.. and some missing for the local install-deps where the scripts are expecting either a local destdir that includes prefix.. or a override destdir for release...

Please add a documentation to explain that the behavior of DESTDIR was changed in this release (v1.6).

where / how exactly ?

where / how exactly ?

Just adding comment lines to Makefile would be fine.
In addition we may copy the text to https://github.com/containerd/containerd/blob/master/BUILDING.md#build-containerd .

/ok-to-test

/test pull-containerd-node-e2e


Allow fuzzy matching on test result

Fuzzy (regex) matching on errors

changed one old test case (the only one that needed regex)
changed the existing ---- error(regex) handling (unused by any test cases) to be recognized as a pattern

Example
we should now be able to run the example from issue (ExtensionErrorTest) but the behaviour of that test changed due to a code update.

Can u take a look at the following two tests and rewrite them with regex pattern too?
TinySnbExceptionTest.DeleteNodeWithEdgeErrorTest
TinySnbDeleteNodeTest.DeleteNodeWithConnectedRels

done!


Fix issue when loading a bad default config file

If the default config file is badly formed tfsec will not fail, instead it will continue to dereference a nil tfsecConfig pointer and crash.
To avoid this, first check if the default config file exists, if not we'll continue on happily; otherwise load the config and fail if there are any errors.

Very sorry.  Just found this bug while playing around.
Thanks!

Think we're good! Thanks again!


Google Chrome for mobile incorrectly identified as Safari

Expected Behaviour
Using the Browser Condition and selecting Mobile Safari as an exclusion blocks only Mobile Safari on iOS.
Actual Behaviour
Using the Browser Condition and selecting Mobile Safari as an exclusion blocks traffic from Chrome on Android as well. This is because Chrome reports 'Mobile Safari' in its User Agent string
Sample Code that illustrates the problem
https://github.com/adobe/reactor-extension-core/blob/master/src/lib/conditions/helpers/clientInfo.js#L30
L30 needs to be swapped with L31 so that Chrome is "passed" first and that, indeed does allow Chrome for Android to work.

I've logged this internally as DTM-15864.

The DTM browser detection rules were pretty far out of date. We did update the browser detection rules when we released Launch in 2017, and they were up to date at the time. We haven't updated them since then. Mobile Safari was designed to match against Safari on iPhones and iPads (again, back in 2017). We have not checked it against any other iOS browsers. And Safari has made changes with Version 13 and 14 that introduced some strangeness, and at the moment I'm honestly not sure what "Mobile Safari" would match against.
We have discussed internally as a team. Our conclusion is that we will not fix this bug and will mark both the Browser and Operating System conditions as being "no longer supported" (as the Device Type condition already is).
Reasons:
User-agent detection of browsers and operating systems is a never-ending task that has always been tricky, our team is not staffed to build an automated way to test this against new browser versions as they come out or to do the effort manually. The most honest way to communicate this is to mark them as no longer supported.
Introducing a 3rd-party library to do the browser detection would add bloat to the Launch library. And none of the popular ones are very up-to-date anyway.
The Browser and Operating System conditions are in use less than 1000 times each across the entire Launch user base, representing less than 0.02% of all conditions. As a related data point, we usually drop runtime support for older browser versions when we notice that they drop below 0.1%.
So we won't delete them, and they'll continue to function as they do now, but we will not be fixing bugs that are logged against the functionality.

Thank you for the clarification Ben.


Support for coordinates array

[lng, lat, elevation], e.g. [-110, 32, 0]

it should work out of the box
anything returned by getCoordinates will be added as geometry coordinates


Fix typo - "hoook"

FireScriptHook had a typo, instead of saying "hook" it said "hoook" in its description.

Looks great, thank youuu <3


Including BIDS in your project

Are you planning on implementing BIDS?
If so, here is a useful library that will organize your NIfTI files into BIDS:
https://cbedetti.github.io/Dcm2Bids/
You may/may not want to include it in your pipeline and/or resource list.

Thank you so much! Of course, I will.


1.1.0 cannot compile

After upgrading the nuget package from 1.0.3 to 1.1.0 I cannot compile!

Any errors at all?

CS0012	The type 'ActionCollection' is defined in an assembly that is not referenced. You must add a reference to assembly 'Microsoft.Xaml.Interactivity, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'.

Is the problem solved???
This problem comes when upgrading the Nuget package from 1.0.3 til 1.1.0. After downgrading I can compile again...

Have you upgraded all projects that are using it? Are you using something like Template10? Or another toolkit that might be using it as a dependency?

Yes - I'm using Template10

Ok, Template10 needs updating to 1.1 so that it's not dependent on the version of xaml behaviours that has strong naming. I'll try and do a PR for that later, but will require @jerrynixon to push an update when it's in.

I'm using template10 version 1.1.4

Ok. Well, for now, you'll have to stick with 1.0.3 of xaml behaviour until Template10 gets an update to point to the new version

No problem- thanks Scott. I'l create an Issue at Template10 :-)


[BUG] [Ubuntu] [PostgreSQL] [upgrade] Unable to upgrade after rollback has been performed

Describe the bug
Cannot upgrade PostgreSQL after rollback has been performed. The error is as follows:
Unable to start service postgresql@13-main: Failed to start<EMAIL_ADDRESS>Unit<EMAIL_ADDRESS>is masked
How to reproduce
Steps to reproduce the behavior:

Deploy an old (v1.1) PostgreSQL cluster (2 vms, replication enabled)
Add a task in the file data/common/ansible/playbooks/roles/postgresql/tasks/upgrade/upgrade.yml before the rescue block to fail the task and trigger a rollback:

    - name: PostgreSQL -- manual fail to perform rollback
      fail:
        msg: PostgreSQL upgrade failed

Execute epicli upgrade to v1.2
Upgrade should fail and rollback tasks should be performed

2021-09-27T18:29:24.5249324Z[38;21m18:29:24 INFO cli.engine.ansible.AnsibleCommand - TASK [postgresql : PostgreSQL -- manual fail to perform rollback] **************
2021-09-27T18:29:24.5825776Z[31;21m18:29:24 ERROR cli.engine.ansible.AnsibleCommand - fatal: [ec2-x-x-x-x.eu-west-1.compute.amazonaws.com]: FAILED! => {"changed": false, "msg": "PostgreSQL upgrade failed, trigerring rollback"}
2021-09-27T18:29:24.5908227Z[31;21m18:29:24 ERROR cli.engine.ansible.AnsibleCommand - fatal: [ec2-y-y-y-y.eu-west-1.compute.amazonaws.com]: FAILED! => {"changed": false, "msg": "PostgreSQL upgrade failed, trigerring rollback"}
2021-09-27T18:29:24.6370047Z[38;21m18:29:24 INFO cli.engine.ansible.AnsibleCommand -[0m
2021-09-27T18:29:24.6371129Z[38;21m18:29:24 INFO cli.engine.ansible.AnsibleCommand - TASK [postgresql : Rollback | Get service facts] *******************************
2021-09-27T18:29:27.6833954Z[38;21m18:29:27 INFO cli.engine.ansible.AnsibleCommand - ok: [ec2-y-y-y-y.eu-west-1.compute.amazonaws.com]
2021-09-27T18:29:27.6937203Z[38;21m18:29:27 INFO cli.engine.ansible.AnsibleCommand - ok: [ec2-x-x-x-x.eu-west-1.compute.amazonaws.com]
.
.
.

Upgrade again after disabling previously added task to force failure.
It fails:

2021-09-27T18:41:29.0659659Z 18:41:29 INFO cli.engine.ansible.AnsibleCommand - TASK [postgresql : PostgreSQL | Debian | Mask postgresql@10-main service to avoid port conflict] ***
2021-09-27T18:41:29.1675150Z 18:41:29 INFO cli.engine.ansible.AnsibleCommand - skipping: [ec2-x-x-x-x.eu-west-1.compute.amazonaws.com]
2021-09-27T18:41:29.9487271Z 18:41:29 INFO cli.engine.ansible.AnsibleCommand - changed: [ec2-y-y-y-y.eu-west-1.compute.amazonaws.com]
2021-09-27T18:41:29.9963235Z 18:41:29 INFO cli.engine.ansible.AnsibleCommand -
2021-09-27T18:41:29.9964741Z 18:41:29 INFO cli.engine.ansible.AnsibleCommand - TASK [postgresql : PostgreSQL | Start PostgreSQL 13 service] *******************
2021-09-27T18:41:30.0811215Z 18:41:30 INFO cli.engine.ansible.AnsibleCommand - skipping: [ec2-x-x-x-x.eu-west-1.compute.amazonaws.com]
2021-09-27T18:41:30.6920537Z 18:41:30 INFO cli.engine.ansible.AnsibleCommand - changed: [ec2-y-y-y-y.eu-west-1.compute.amazonaws.com]
2021-09-27T18:41:30.8360037Z 18:41:30 INFO cli.engine.ansible.AnsibleCommand -
2021-09-27T18:41:30.8361564Z 18:41:30 INFO cli.engine.ansible.AnsibleCommand - TASK [postgresql : PostgreSQL | Debian | Ensure that instantiated PostgreSQL 13 service started] ***
2021-09-27T18:41:30.9221936Z 18:41:30 INFO cli.engine.ansible.AnsibleCommand - skipping: [ec2-x-x-x-x.eu-west-1.compute.amazonaws.com]
2021-09-27T18:41:31.5320274Z 18:41:31 ERROR cli.engine.ansible.AnsibleCommand - fatal: [ec2-y-y-y-y.eu-west-1.compute.amazonaws.com]: FAILED! => {"changed": false, "msg": "Unable to start service postgresql@13-main: Failed to start<EMAIL_ADDRESS>Unit<EMAIL_ADDRESS>is masked.\n"}
2021-09-27T18:41:31.6288630Z 18:41:31 INFO cli.engine.ansible.AnsibleCommand -
2021-09-27T18:41:31.6295014Z 18:41:31 INFO cli.engine.ansible.AnsibleCommand - TASK [postgresql : PostgreSQL | Extensions | repmgr | Register primary node] ***
2021-09-27T18:41:31.6478961Z 18:41:31 INFO cli.engine.ansible.AnsibleCommand - skipping: [ec2-x-x-x-x.eu-west-1.compute.amazonaws.com]
2021-09-27T18:41:31.7819017Z 18:41:31 INFO cli.engine.ansible.AnsibleCommand -
2021-09-27T18:41:31.7822511Z 18:41:31 INFO cli.engine.ansible.AnsibleCommand - TASK [postgresql : PostgreSQL | Check if PostgreSQL 13 is running on primary node] ***
2021-09-27T18:41:32.6236278Z 18:41:32 ERROR cli.engine.ansible.AnsibleCommand - fatal: [ec2-x-x-x-x.eu-west-1.compute.amazonaws.com -> ec2-y-y-y-y.eu-west-1.compute.amazonaws.com]: FAILED! => {"changed": false, "msg": "unable to connect to database: connection to server on socket \"/var/run/postgresql/.s.PGSQL.5432\" failed: No such file or directory\n\tIs the server running locally and accepting connections on that socket?\n"}

Expected behavior
Upgrade should be successful on re-run.
Environment

OS: [Ubuntu]

epicli version: [v1.2]
How to fix:
Run sudo systemctl unmask<EMAIL_ADDRESS>command on each PostgreSQL node after rollback before running upgrade again.

DoD checklist

[ ] Changelog updated (if affected version was released)
[ ] COMPONENTS.md updated / doesn't need to be updated
[ ] Automated tests passed (QA pipelines)

[ ] apply
[ ] upgrade

[ ] Case covered by automated test (if possible)
[ ] Idempotency tested
[ ] Documentation updated / doesn't need to be updated
[ ] All conversations in PR resolved
[ ] Backport tasks created / doesn't need to be backported

:heavy_check_mark: Fixed. Tested upgrades on 1- and 2-nodes clusters.


Refactor to enable support for Amp in PyTorch and better support future phase2 work

Looks overall good to me. Thanks for getting this bug resolved while also improving the codebase. Since ScheduledOptimizer is deprecated, could we update one of our examples to use these changes or update this PR with a brief example of the new flow for AMP?
Also are there any docs that need updating?

I'm holding off on changing over examples/docs for a minor point release. So, trying to keep everything originally functioning and enabling this silently on the backend for the ultralytics integrations to unblock those for now. On 0.4 we'll update examples and docs


Resubmitting the AS3 declaration for TS profiles throws eerror

Environment

3.22.1
14.1

Summary
The declaration works on each call, but any call after the initial one will return the error below. The declaration does appear to be successful in its changes though.
"01070344:3: Cannot delete referenced virtual address /Common/Shared/<IP_ADDRESS>.",
Steps To Reproduce
Steps to reproduce the behavior:

Submit the following declaration:

{
    "class": "AS3",
    "action": "deploy",
    "persist": true,
    "declaration": {
        "class": "ADC",
        "schemaVersion": "3.22.0",
        "remark": "Bacon",
        "Common": {
            "Shared": {
                "class": "Application",
                "template": "shared",
                "telemetry_local_rule": {
                    "class": "iRule",
                    "iRule": "when CLIENT_ACCEPTED {\n  node <IP_ADDRESS> 6514\n}"
                },
                "telemetry_local": {
                    "class": "Service_TCP",
                    "virtualAddresses": [
                        "<IP_ADDRESS>"
                    ],
                    "virtualPort": 6514,
                    "iRules": [
                        "telemetry_local_rule"
                    ]
                },
                "telemetry": {
                    "class": "Pool",
                    "members": [
                        {
                            "enable": true,
                            "serverAddresses": [
                                "<IP_ADDRESS>"
                            ],
                            "servicePort": 6514
                        }
                    ],
                    "monitors": [
                        {
                            "bigip": "/Common/tcp"
                        }
                    ]
                },
                "telemetry_hsl": {
                    "class": "Log_Destination",
                    "type": "remote-high-speed-log",
                    "protocol": "tcp",
                    "pool": {
                        "use": "telemetry"
                    }
                },
                "telemetry_formatted": {
                    "class": "Log_Destination",
                    "type": "splunk",
                    "forwardTo": {
                        "use": "telemetry_hsl"
                    }
                },
                "telemetry_publisher": {
                    "class": "Log_Publisher",
                    "destinations": [
                        {
                            "use": "telemetry_formatted"
                        }
                    ]
                },
                "telemetry_security_log_profile": {
                    "class": "Security_Log_Profile",
                    "application": {
                        "localStorage": false,
                        "remoteStorage": "splunk",
                        "protocol": "tcp",
                        "servers": [
                            {
                                "address": "<IP_ADDRESS>",
                                "port": "6514"
                            }
                        ],
                        "storageFilter": {
                            "requestType": "all"
                        }
                    }
                }
            }
        }
    }
  }

Observe the following error response:

        {
            "code": 200,
            "message": "success",
            "lineCount": 16,
            "host": "localhost",
            "tenant": "Common",
            "runTime": 6313
        },
        {
            "code": 422,
            "message": "declaration failed",
            "response": "01070344:3: Cannot delete referenced virtual address /Common/Shared/<IP_ADDRESS>.",
            "host": "localhost",
            "tenant": "Common",
            "runTime": 4106
        }

Expected Behavior
"no change" returned
Actual Behavior
Error above.

I have confirmed the bug and added it to our internal product backlog as AUTOTOOL-2035.
I noticed your Common tenant is missing the class property. While the schema allows this, it causes a lot of strange behavior. I am still able to reproduce this bug when this property is present.

This issue has been resolved as part of the 3.25.0 release.

This issue has been resolved as part of the 3.25.0 release.


[Modal] Dim Each Modal When Showing Multiple

Hi,
I have multiple modal use case, where opening one modal from another modal with (allow multiple = true), should use a new dimmer which will dim the previous modal too. (All the dimmers must be page level dimmers)
As far as I checked, I haven't found any such option in the documentation. Could you please let me know if this feature is available and if so how can I implement it?.
If it is not available, are there any chances that it will land in future versions?

Any updates on this issues?

I'm too waiting for the same. Any update?

any updates...??

Hi, Is there an update regarding this issue?

Since Fomantic-UI 2.5.1 the previous modals are automatically dimmed when allowMultiple is iset to true by https://github.com/fomantic/Fomantic-UI/pull/119
https://fomantic-ui.com/modules/modal.html#multiple-modals


Update notebook API

Codecov Report

Merging #14313 into main will increase coverage by 0.10%.
The diff coverage is 63.76%.

@@            Coverage Diff             @@
##             main   #14313      +/-   ##
==========================================
+ Coverage   59.75%   59.85%   +0.10%
==========================================
  Files         697      709      +12
  Lines       38649    39366     +717
  Branches     5577     5706     +129
==========================================
+ Hits        23094    23562     +468
- Misses      14364    14562     +198
- Partials     1191     1242      +51

Impacted Files
Coverage Δ

src/client/activation/common/activatorBase.ts
14.41% <0.00%> (+1.19%)
:arrow_up:

...rc/client/activation/jedi/multiplexingActivator.ts
17.74% <0.00%> (ø)

src/client/activation/node/languageServerProxy.ts
26.58% <0.00%> (ø)

src/client/activation/refCountedLanguageServer.ts
41.30% <0.00%> (ø)

src/client/activation/types.ts
100.00% <ø> (ø)

src/client/common/application/notebook.ts
10.16% <0.00%> (-0.74%)
:arrow_down:

src/client/common/process/pythonDaemon.ts
14.28% <0.00%> (ø)

src/client/common/utils/localize.ts
96.24% <ø> (ø)

src/client/datascience/baseJupyterSession.ts
56.30% <ø> (-0.52%)
:arrow_down:

.../datascience/interactive-common/interactiveBase.ts
5.78% <0.00%> (+<0.01%)
:arrow_up:

... and 90 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c248197...740ee49. Read the comment docs.

@DonJayamanne we will close this in a week if this is not needed.


Learn enemy node compositions

So, when the compass prediction tells us where we are going, it also includes an enemy ID normally. Is that ID unique?
If it is, then this:
Have KC3 remember the enemy composition associated to that ID; next time you get a node prediction, it could show you a thumbnail of the enemy formation you're going up against.
Now, this seems like it's getting a bit cheaty to be honest, but I have other ideas for this kind of data for the future, this just seemed like an easier, quicker application of this data. The other idea is not cheaty and if you'd rather not have these kinds of cheats on KC3 then lemme know and I'll open an issue for the other idea

saw this on ElectronicObserver, will add this, probably after the Quest flowcharts


WiX: adjust installer images according to official Swift logo guidelines

Correct background orange color
Rounded corners instead of square logo shape

cc @compnerd


config: Fix timestamp when JSON logging is disabled

Hi,
I noticed the timestamp is not present in the logger when JSON logging is disabled. I noticed that during configuration the wrong field is used to configure the timestamp format. This PR fixes that issue. Also confirmed by testing it.
Old format:
level INF Request: GET ...

New format:
2022-05-06T11:04:44+02:00 INF Request: GET ...

This would make our logs much more consistent 👍🏽


trycmd: allow to set substitutable variables

I'm trying to define a variable with contains a random value and want to inject it somehow into tests executed by trycmd. I can mask the value with insert_var(..) and tried to use env(..) to make the variable accessible in my tests but something like $ echo $TEST_VALUE never substitutes the variable with the value given to env(..). Is there some way to access the value behind a variable?

yeahs exactly! A simplified code example looks like this:
let key = "some-randomly-generated-value";

trycmd::TestCases::new()
            .case("tests/cmd/secrets/*.md")
            .env("KEY", key)
            .insert_var("[KEY]", key)
            .unwrap();

and I want to run a test like this:
$ bin-command secrets add $KEY
key '[KEY]' added

my current workaround is to write the generated value into a file, add that file to gitignore (so that it won't get flagged by security scans..) and allow my add subcommand to read from files

As we have another concept of substitutions (also called redactions) in terms of insert_var, I've clarified the issue name.
This is another part in the overall story thats building up in issues of "it would be nice if we had an embedded shell to work with".  We have requests for pipes (#172) and some common unix built-ins (e.g. #168, #43)


Set level in console_file_logger

Initialize_logger() is not suitable as we don't want to replace the pattern in sinks.

Done in commit 7649b101660bac5fc7edd5a645b3f9f22a73255b


HDDS-6189. Intermittent failure in TestDirectoryDeletingServiceWithFSO

What changes were proposed in this pull request?
TestDirectoryDeletingServiceWithFSO was intermittently timing out at assertSubPathsCount.  This method has a waitFor, but the condition cannot change, since it compares two fixed long values.  It can only pass if wait is not necessary and the condition is true at the time of the call.
The fix is to get the actual value for each check during the wait.
https://issues.apache.org/jira/browse/HDDS-6189
How was this patch tested?
100x passed:
https://github.com/adoroszlai/hadoop-ozone/runs/6107144162
Regular CI:
https://github.com/adoroszlai/hadoop-ozone/actions/runs/2200070046

Thanks @sodonnel for the review.


Schedule not showing up

My schedule has not been showing up for the last year. after i changed my last name, i have not been in the system, so no one can see what classes I'm in and i don't have any classes listed
Submitted via Form

If you can let me know which student, I now know how to fix this. Ideally, I would have their old and new email addresses.

submitted by cwhitfield @JonBriggs

This has been fixed for cwhitfield.


[osx] pyrcc4 not found

OS: Mac OSX Mojave
PyQt version: 4

$ make qt4py2
pyrcc4 -py2 -o resources.py resources.qrc
make: pyrcc4: No such file or directory
make: *** [qt4py2] Error 1

fix
brew install cartr/qt4/pyqt@4

I will recommend using pyqt5 as well as python3.


404 error

I just pasted the code:

  <meta charset="UTF-8">
  <title>Lonely Globe Advisor</title>

Top Hotels in Rome

Villa Domus - Via Piacenza 9, Rome, Italy

Hotel Trivelli - Piazza Barberini 11, Rome, Italy

Got error
A bad HTTP response code (404) was received when fetching the script.
This 404 code is caused when UpUp.start is called
If I comment UpUp.start, everything works fine.
(upup.min.js is well loaded)
Any idea ?

Can you upload this somewhere where we can take a look at it and see why it breaks?


Create writer lazily only when there's record to write

Signed-off-by: Yinan Li<EMAIL_ADDRESS>

@zliu41 can you review?

LGTM.

LGTM


fix(ansible/docker): replace lineinfile with copy not to write duplicated entries

Resolves #65.
Did some refactoring at the same time.

CI: https://github.com/autowarefoundation/autoware/actions/runs/1874451925

CI passed.


Handle more and no mouse buttons.

This PR adds mouse support for more than just two buttons.
Update: It also changes the mouse events signature.

Okay I added MouseButton::Unknown and use it as a fallback on all the platforms.

After some further testing and thinking I also changed mouse events to contain Option<MouseButton>, because in some cases we know for sure that no button was pressed and that is different from an unknown button.
One slight downside of using an Option is that people now need to import the MouseButtonExt trait to get the convenience is_left etc methods.

Would it make more sense to have MouseButton::None?

An additional (probably useless?) thought: mouse buttons are really a mask, multiple can be down at a time etc. I'm probably over thinking this though, it definitely isn't what we would want to surface as the principle API (although we might want to surface it at some point, for advanced cases?)

I went with Option because that's the rust way but I don't consider that too important in this case.
How about a custom struct MouseButtons which is basically a type-safe bitmask. Something like this:
impl MouseButtons {
    fn has(&self, button: MouseButton) -> bool;

    fn has_left(&self) -> bool;
    fn has_right(&self) -> bool;
    // ....
    fn has_x2(&self) -> bool;

    fn has_any(&self) -> bool;
}

The problem is that these have different uses; so that would work, but would not replace MouseButton.
on MouseDown and MouseUp, it matters which button triggered the event; the mask would be supplementary at best. On MouseMoved, the idea of a MouseButton is meaningless, and something like MouseButtons makes sense.
For all this, though, what's most important is a good API that covers the 99% case; to me this means that MouseDown and MouseUp always have an associated mouse button, and that MouseMoved probably shouldn't.
One option:
MouseEvent does not have a MouseButton, it has only something like the MouseButtons you describe; and then MouseDown and MouseUp are a MouseEvent plus a MouseButton, but MouseMoved is only a MouseEvent? This is the most reasonable thing I can think of right now...

That sounds reasonable. During MouseDown you would have druid-managed state that tells you which other buttons are currently already held down, plus the one new button separately.
Event::MouseDown((MouseEvent, MouseButton)) // Something like this tuple?

After thinking about this some more, especially in relation to #842 which is headed towards adding a new focused bool to the event for clicks, I'm thinking of splitting up the event structs into MouseMoveEvent and MouseClickEvent.
Having the single button info and also focused info only makes sense for clicks. This would also allow for more easy divergence between the event data in the future. It should also break less code because there's no tuple and the following should still work:
Event::MouseUp(e) => {
    if e.button.is_left() { do stuff; }
}

a few options I think:
MouseDown { event: MouseEvent, button: MouseButton }

// or,
MouseClickEvent {
    mouse: MouseEvent,
    button: MouseButton,
    count: u32,
}
// with
Event::MousDown(MouseClickEvent),
Event::MouseUp(MouseClickEvent),
Event::MouseMove(MouseEvent),

(I realized in the course of writing this that the click count is also irrelevant for the purposes of MouseMove).
I think maybe one reason for the current design is that I was thinking about mac, and NSEvent is a monolith; the same underlying 'type' can represent tons of different events, so there are lots of fields that exist that aren't actually meaningful for a given specific event.

I pushed a new iteration of this work. Holding down multiple mouse buttons is now nicely tracked. It works perfectly on macOS and Windows and mostly on GTK. GTK has all sorts of issues due to fragmentation, which will need solving separately. This PR at the very least only improves the situation in druid-shell and doesn't make it worse.
For this iteration I went with ClickEvent and MoveEvent. It's not perfect but doesn't seem too bad either. The benefit is that the struct doesn't contain useless fields, however it does make match code a bit more verbose if it wants to do something on a common field.
// What used to be this ..
Event::MouseDown(e) | Event::MouseUp(e) | Event::MouseMove(e) => {
    self.last_mouse_pos = Some(e.pos)
}
// is now this.
Event::MouseDown(ClickEvent { pos, .. })
| Event::MouseUp(ClickEvent { pos, .. })
| Event::MouseMove(MoveEvent { pos, .. }) => self.last_mouse_pos = Some(*pos),

I'm not 100% convinced that splitting them is the way to go. Maybe the superior ergonomics in handling these three events together is worth the price of wasting those extra bytes of memory and having useless fields in the code editor autocomplete for MouseMove?

A new iteration has been pushed. I went back to a single MouseEvent struct and changed the union / extend naming for MouseButtons.
I also thought about your question of whether MouseButton::Other is useful. It can have a bit of value but it does also introduce a lot of problems.

It isn't really cross-platform in a portable way, so hot-key configurations made with Other in druid wouldn't be portable either.
It represents multiple buttons so events can get rather funky with up/down and we couldn't really perform double-click detection on it. #859
It isn't supported in our MouseButtons in more than one way. Not just the set itself, but even figuring out what to add to the set. There isn't a clear way to figure out the holding state on all platforms.

I think druid should strive for a consistent API that is reliable across platforms. Given these issues I don't think Other passes the quality bar for druid. For that reason I have removed it from this PR. Now druid-shell just ignores click events on buttons that we don't know.
In the long term, if we want to support more buttons, that can of course be done. However it should be done in a qualitative way and I don't think it can be achieved with the platform mouse APIs druid-shell is using right now. More lower level APIs would need to be used.


Use accessor for ClimaAnalysis units

https://github.com/CliMA/ClimaAnalysis.jl/pull/70 is very slightly breaking if units are accessed directly (the return value might no longer be a String, but can still be printed as a string). To be safe, I changed to use the getter function.

I think you need to update the Manifest as well?

I thought we were already using 0.5.7 (it was released 2 weeks ago), but clearly no. Updating it now.


Double encoding of entities.

When an string with HTML entities like   is loaded, the returned document has these ampersands encoded again to &nbsp; breaking the original HTML.
$payload = <<< 'HTML'

    <meta charset="utf-8">
    <title>
      Hello world!
    </title>

      Hello world!

HTML;
$doc = \Layershifter\Gumbo\Parser::load($payload);
var_dump($doc->saveHTML());

Output:
string(162) "<meta charset="utf-8"><title>
      Hello&nbsp;world!
    </title>

      Hello&nbsp;world!

"

@ajgarlag thanks for report, it's a one of known issues. It happens because libxml that saves document performs entity encode.
I have plans to solve this problem, however I don't that it will be in near future. As temporary solution you can perform entity decode after save.

@layershifter Thank you for your work!
In that case, a Known Issues section could be added to the README file.

Yes, you're right, I will add it today


WIP: Algorithms storage v2

This change is

core/api-server/lib/service/execution.js, line 82 at r3 (raw file):

            if (pipeline.flowInput && !alreadyExecuted) {
                const metadata = parser.replaceFlowInput(pipeline);
                const storageInfo = await storageManager.hkube.put({ jobId, taskId: 'flowInput', data: pipeline.flowInput },

What does this mean
"taskId":"flowInput"

core/api-server/tests/storage.js, line 11 at r3 (raw file):

let encoding;

describe.only('Storage', () => {

remove the "only" from describe.only


Create card get TypeError: (0 , navigation.useParams) is not a function

I add a card to my content/docs/index.mdx and I get
TypeError: (0 , navigation.useParams) is not a function
--
22:25:52.314 | at SafeLink (/vercel/path0/.next/server/chunks/9983.js:30641:42)
22:25:52.315 | at Be (/vercel/path0/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.browser.production.min.js:106:273)
22:25:52.315 | at Z (/vercel/path0/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.browser.production.min.js:112:91)
22:25:52.315 | at Be (/vercel/path0/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.browser.production.min.js:107:9)
22:25:52.315 | at Z (/vercel/path0/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.browser.production.min.js:112:91)
22:25:52.315 | at Ce (/vercel/path0/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.browser.production.min.js:114:98)
22:25:52.315 | at Ee (/vercel/path0/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.browser.production.min.js:113:424)
22:25:52.315 | at Z (/vercel/path0/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.browser.production.min.js:112:222)
22:25:52.315 | at Ce (/vercel/path0/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.browser.production.min.js:114:98)
22:25:52.315 | at Be (/vercel/path0/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.browser.production.min.js:107:215)

how to fix it

How do you add the card? Please provide sufficient information to reproduce your project.

this is my code: https://github.com/ssangyongsportsorg/Ssport-website/blob/main/content/docs/index.mdx

this is my code: https://github.com/ssangyongsportsorg/Ssport-website/blob/main/content/docs/index.mdx

I noticed that you're using a deprecated version of Next Docs, the SafeLink API has been removed since 4.0.0, can you upgrade and try again?

Also, the App Router feature in Next.js 13.2.4 is not stable yet, please update to Next.js 13.4+


Compilation problem

Hello,
Thank you for sharing your library to access Basler cameras.
I tried to compile under Linux with Pylon v.4 and v.5. In both cases I had the following error:
basler_set_get.cpp: In function ‘double BaslerHelper::get_float(Pylon::CInstantCamera*, const char*, bool)’:
basler_set_get.cpp:94:57: error: conversion from ‘GenICam::gcstring’ to non-scalar type ‘std::__cxx11::string {aka
std::__cxx11::basic_string}’ requested
std::string s_result = p_parameter->ToString();
Do you have any idea of what I am doing wrong here?
Thank you for your help,
L Denis

Hi Denis,
I have an idea, what the problem could be. I have created the whole driver with Pylon 4 and it appears that Pylon 5  has some differences. As I don't have a Linux installation with Matlab and Pylon 5, I couldn't try it out, so can you please try the following:
In the file basler_set_get.cpp, in each function get_float, get_int, get_string, and get_bool, there are the following two lines:
std::string s_result = p_parameter->ToString();
printf("Read Parameter \"%s\": %s\n",s_param_name, s_result);

Please try replacing them in each of the mentioned functions by the following:
GenICam::gcstring s_result = p_parameter->ToString();
printf("Read Parameter \"%s\": %s\n",s_param_name, s_result.c_str());

I'm afraid that the same happens in line 128 of the same file, then we would need some workaround there too. But can you try what I posted above first, and see if that works?
Best,
Hannes

Thank you very much for your answer. I tried the fix you suggested + changed the type of the output of function get_string in both the .h and .cpp file. Now, the compilation is OK. There are still problems during linking, maybe due to changes from Pylon4 to Pylon5 library (e.g., "undefined reference to
`Pylon::CTlFactory::GetInstance()"). I am still trying to fix them and will let you know when I succeed.

Hello Hannes and Denis,
I am trying to adapt the code for MacOS 10.12 using Pylon 5.
I have resolved all the compiler errors but I have several linking errors, all 'undefined symbols'
Including:
Pylon::CTlFactory::GetInstance(), other Pylon class symbols, GenericException logical exception etc.
I imagined this was because some header has changed between Pylon 4 and 5? But not sure where to go looking.
Wondering if you might have some insights into these linking errors.
Thanks,
Dev
ps @ Hannes, if this is successful, please let me know if you'd be interested to open a branch for MacOS development of your driver.


Added mcrypt to require composer.json

Hello.
I've just tried to run tests, and got this error:

PHP Fatal error:  Call to undefined function Thruway\Authentication\mcrypt_create_iv() in /projects/github/Thruway/src/Thruway/Authentication/WampCraAuthProvider.php on line 70

I added ext-mcrypt to composer.json

Thank you.
Since ext-mcrypt is only needed for WAMP-CRA authentication, I think I would be better to suggest it instead of requiring it.
    "suggest": {
       "ext-mcrypt":"If you want to use WAMP-CRA for authentication"
    },

@davidwdan, you're right. Thank you.


Allow tutors to add a set of template-answers per exercise

Is your feature request related to a problem?

In simple exercises, most students either give the correct answer or do similar mistakes. It takes a lot of time to always give the same answer without this feature
Describe the solution you'd like

For each exercise, every tutor can define a set of feedbacks (in the best case also associated with a point value) from which they can choose in a dropdown instead of writing a new feedback. This predefined text will then be pasted in the feedback field for further customization
Describe alternatives you've considered

Currently I use a Text-File where I store my default feedback texts and then copy&paste those in the field manually which takes a lot of time
Additional context

We have structured grading criteria which somehow support your idea of default feedback.
It would be nice, if tutors could store additional structured grading criteria for their own use or even for the use of other tutors (which would probably require an instructor approval)