Exception : Index was out of range. Must be non-negative and less than the size of the collection. on VSTest\2.0.17\TestSelector\TestSelector.exe UpdateTestResults

Hi,
Today I was playing with the 'Run only impacted tests' feature of the 'Run Tests using Visual Studio task'. My  test run succeeds, but I get the following error when VSTest\2.0.17\TestSelector\TestSelector.exe UpdateTestResults is executed (see below).
2017-03-07T15:55:38.0425386Z Total tests: 1206. Passed: 1200. Failed: 0. Skipped: 6.
2017-03-07T15:55:38.0425386Z Test Run Successful.
2017-03-07T15:55:38.0425386Z Test execution time: 11.8204 Minutes
2017-03-07T15:55:38.9516587Z Results File: C:\tfsagent\_work\19\s\TestResults\tfsonlinebuild_VTFS-DBS02 2017-03-07 15_44_13.trx
2017-03-07T15:55:39.1116416Z ##[debug]rc:0
2017-03-07T15:55:39.1116416Z ##[debug]success:true
2017-03-07T15:55:39.1116416Z ##[debug]rm -rf C:\Users\TFSONL~1\AppData\Local\Temp\bbaf38b0-034c-11e7-a552-4390675c08db.runsettings
2017-03-07T15:55:39.1116416Z ##[debug]removing file
2017-03-07T15:55:39.1116416Z ##[debug]defaultRoot: 'C:\tfsagent\_work\19\s\TestResults'
2017-03-07T15:55:39.1116416Z ##[debug]findOptions.followSpecifiedSymbolicLink: 'true'
2017-03-07T15:55:39.1195144Z ##[debug]findOptions.followSymbolicLinks: 'true'
2017-03-07T15:55:39.1195144Z ##[debug]matchOptions.debug: 'false'
2017-03-07T15:55:39.1195144Z ##[debug]matchOptions.nobrace: 'true'
2017-03-07T15:55:39.1195144Z ##[debug]matchOptions.noglobstar: 'false'
2017-03-07T15:55:39.1195144Z ##[debug]matchOptions.dot: 'true'
2017-03-07T15:55:39.1195144Z ##[debug]matchOptions.noext: 'false'
2017-03-07T15:55:39.1195144Z ##[debug]matchOptions.nocase: 'true'
2017-03-07T15:55:39.1216417Z ##[debug]matchOptions.nonull: 'false'
2017-03-07T15:55:39.1216417Z ##[debug]matchOptions.matchBase: 'false'
2017-03-07T15:55:39.1216417Z ##[debug]matchOptions.nocomment: 'false'
2017-03-07T15:55:39.1216417Z ##[debug]matchOptions.nonegate: 'false'
2017-03-07T15:55:39.1216417Z ##[debug]matchOptions.flipNegate: 'false'
2017-03-07T15:55:39.1288242Z ##[debug]pattern: 'C:\tfsagent\_work\19\s\TestResults\*.trx'
2017-03-07T15:55:39.1288242Z ##[debug]findPath: 'C:\tfsagent\_work\19\s\TestResults'
2017-03-07T15:55:39.1288242Z ##[debug]statOnly: 'false'
2017-03-07T15:55:39.1288242Z ##[debug]findPath: 'C:\tfsagent\_work\19\s\TestResults'
2017-03-07T15:55:39.1288242Z ##[debug]findOptions.followSpecifiedSymbolicLink: 'true'
2017-03-07T15:55:39.1288242Z ##[debug]findOptions.followSymbolicLinks: 'true'
2017-03-07T15:55:39.1288242Z ##[debug]  C:\tfsagent\_work\19\s\TestResults (directory)
2017-03-07T15:55:39.1316419Z ##[debug]  C:\tfsagent\_work\19\s\TestResults\e8ca5e74-0e5e-4b9a-9669-ae42f540193d (directory)
2017-03-07T15:55:39.1316419Z ##[debug]  C:\tfsagent\_work\19\s\TestResults\e8ca5e74-0e5e-4b9a-9669-ae42f540193d\tfsonlinebuild_VTFS-DBS02 2017-03-07 15_43_51.coverage (file)
2017-03-07T15:55:39.1316419Z ##[debug]  C:\tfsagent\_work\19\s\TestResults\f82cb885-fd7f-461c-a1ec-97ade1afe849 (directory)
2017-03-07T15:55:39.1316419Z ##[debug]  C:\tfsagent\_work\19\s\TestResults\tfsonlinebuild_VTFS-DBS02 2017-03-07 15_44_13 (directory)
2017-03-07T15:55:39.1316419Z ##[debug]  C:\tfsagent\_work\19\s\TestResults\tfsonlinebuild_VTFS-DBS02 2017-03-07 15_44_13\In (directory)
2017-03-07T15:55:39.1384786Z ##[debug]  C:\tfsagent\_work\19\s\TestResults\tfsonlinebuild_VTFS-DBS02 2017-03-07 15_44_13\In\VTFS-DBS02 (directory)
2017-03-07T15:55:39.1384786Z ##[debug]  C:\tfsagent\_work\19\s\TestResults\tfsonlinebuild_VTFS-DBS02 2017-03-07 15_44_13\In\VTFS-DBS02\tfsonlinebuild_VTFS-DBS02 2017-03-07 15_43_51.coverage (file)
2017-03-07T15:55:39.1384786Z ##[debug]  C:\tfsagent\_work\19\s\TestResults\tfsonlinebuild_VTFS-DBS02 2017-03-07 15_44_13.trx (file)
2017-03-07T15:55:39.1384786Z ##[debug]9 results
2017-03-07T15:55:39.1384786Z ##[debug]found 9 paths
2017-03-07T15:55:39.1384786Z ##[debug]applying include pattern
2017-03-07T15:55:39.1384786Z ##[debug]1 matches
2017-03-07T15:55:39.1384786Z ##[debug]1 final results
2017-03-07T15:55:39.1384786Z ##[debug]C:\tfsagent\tasks\VSTest\2.0.17\TestSelector\TestSelector.exe arg: UpdateTestResults
2017-03-07T15:55:39.1384786Z ##[debug]Build.BuildId=22122
2017-03-07T15:55:39.1416422Z ##[debug]System.TeamFoundationCollectionUri=https://xxx.visualstudio.com/
2017-03-07T15:55:39.1416422Z ##[debug]System.TeamProject=MFS
2017-03-07T15:55:39.1416422Z ##[debug]SystemVssConnection auth param AccessToken = ********
2017-03-07T15:55:39.1478419Z ##[debug]exec tool: C:\tfsagent\tasks\VSTest\2.0.17\TestSelector\TestSelector.exe
2017-03-07T15:55:39.1478419Z ##[debug]Arguments:
2017-03-07T15:55:39.1478419Z ##[debug]   UpdateTestResults
2017-03-07T15:55:39.1478419Z [command]C:\tfsagent\tasks\VSTest\2.0.17\TestSelector\TestSelector.exe UpdateTestResults
2017-03-07T15:55:39.1478419Z ##[debug]Deleting the response file C:\Users\TFSONL~1\AppData\Local\Temp\d847ee40-034c-11e7-a552-4390675c08db.txt
2017-03-07T15:55:39.1478419Z ##[debug]rm -rf C:\Users\TFSONL~1\AppData\Local\Temp\d847ee40-034c-11e7-a552-4390675c08db.txt
2017-03-07T15:55:39.1478419Z ##[debug]Deleting the discovered tests file C:\Users\TFSONL~1\AppData\Local\Temp\bd966f40-034c-11e7-a552-4390675c08db.txt
2017-03-07T15:55:39.1478419Z ##[debug]rm -rf C:\Users\TFSONL~1\AppData\Local\Temp\bd966f40-034c-11e7-a552-4390675c08db.txt
2017-03-07T15:55:39.1478419Z ##[debug]removing file
2017-03-07T15:55:39.1478419Z ##[debug]Deleting the baseline build id file C:\Users\TFSONL~1\AppData\Local\Temp\bb56b871-034c-11e7-a552-4390675c08db.txt
2017-03-07T15:55:39.1516430Z ##[debug]rm -rf C:\Users\TFSONL~1\AppData\Local\Temp\bb56b871-034c-11e7-a552-4390675c08db.txt
2017-03-07T15:55:39.1516430Z ##[debug]removing file
2017-03-07T15:55:41.4382356Z PERF: GetTestCaseResults: took 1738.5298 ms
2017-03-07T15:55:41.6221835Z PERF: ReadResultsFromFile: took 173.4023 ms
2017-03-07T15:55:42.2365842Z PERF: UpdateResultsAndSignatures: took 623.0815 ms
2017-03-07T15:55:42.2614184Z Exception : Index was out of range. Must be non-negative and less than the size of the collection.
2017-03-07T15:55:42.2614184Z Parameter name: index
2017-03-07T15:55:42.2614184Z  Stack :    at System.ThrowHelper.ThrowArgumentOutOfRangeException(ExceptionArgument argument, ExceptionResource resource)
2017-03-07T15:55:42.2614184Z    at System.Collections.Generic.List`1.get_Item(Int32 index)
2017-03-07T15:55:42.2614184Z    at Microsoft.TeamFoundation.DistributedTask.Task.TestResults.TestRunPublisher.UpdateResults(TestCaseResultData[] testResults, List`1 tcmTestCaseResults, TestHttpClientBase testImpactClient, Int32 runId, Int32 definitionRunId, DefinitionType definitionType) in D:\v2.0\A1\_work\4\s\DistributedTask\Agent\Task.TestResults\TestRunPublisher.cs:line 152
2017-03-07T15:55:42.2614184Z    at Microsoft.TestFoundation.TestImpact.TestSelector.CommandUpdateTestResults.Run() in D:\v2.0\A1\_work\4\s\Ta\Tasks\TestImpact\TestSelector\Commands\Command\CommandUpdateTestResults.cs:line 133
2017-03-07T15:55:42.2614184Z    at Microsoft.TestFoundation.TestImpact.TestSelector.Program.RunCommand(String commandName, String[] args) in D:\v2.0\A1\_work\4\s\Ta\Tasks\TestImpact\TestSelector\Program.cs:line 41
2017-03-07T15:55:42.2964179Z ##[debug]rc:4294967295
2017-03-07T15:55:42.2964179Z ##[debug]success:false
2017-03-07T15:55:42.2964179Z ##[debug]Test Run Updation failed!
2017-03-07T15:55:42.2964179Z ##[debug]task result: Failed
2017-03-07T15:55:42.3064182Z ##[error]Vstest exited with return code: 1.
2017-03-07T15:55:42.3364192Z ##[debug]Deleting the response file C:\Users\TFSONL~1\AppData\Local\Temp\d847ee40-034c-11e7-a552-4390675c08db.txt
2017-03-07T15:55:42.3364192Z ##[debug]rm -rf C:\Users\TFSONL~1\AppData\Local\Temp\d847ee40-034c-11e7-a552-4390675c08db.txt
2017-03-07T15:55:42.3364192Z ##[debug]Deleting the discovered tests file C:\Users\TFSONL~1\AppData\Local\Temp\bd966f40-034c-11e7-a552-4390675c08db.txt
2017-03-07T15:55:42.3364192Z ##[debug]rm -rf C:\Users\TFSONL~1\AppData\Local\Temp\bd966f40-034c-11e7-a552-4390675c08db.txt
2017-03-07T15:55:42.3364192Z ##[debug]Deleting the baseline build id file C:\Users\TFSONL~1\AppData\Local\Temp\bb56b871-034c-11e7-a552-4390675c08db.txt
2017-03-07T15:55:42.3364192Z ##[debug]rm -rf C:\Users\TFSONL~1\AppData\Local\Temp\bb56b871-034c-11e7-a552-4390675c08db.txt
2017-03-07T15:55:42.3364192Z ##[debug]Deleting the run id fileC:\Users\TFSONL~1\AppData\Local\Temp\bb56b870-034c-11e7-a552-4390675c08db.txt
2017-03-07T15:55:42.3364192Z ##[debug]rm -rf C:\Users\TFSONL~1\AppData\Local\Temp\bb56b870-034c-11e7-a552-4390675c08db.txt
2017-03-07T15:55:42.3364192Z ##[debug]removing file

Am I missing something?

Hi @wouterroos,
Answer to a few questions will help us:

Did you see this error only in your first run or are you seeing this in every run? Are you able to see only impacted tests running? Please share the logs with us after setting system.debug to true in the build variables.

The error log says "Test Run Updation Failed". Can you verify if you are not able to see run information in "Summary" tab in the build details or Runs tab under the Test tab?

Thanks

Hi,

I saw the error multiple times, but sometimes the run finished with some 'not impacted' tests in the results. I'm currently trying to reproduce the issue at the moment, but I'm having some issues with the 'Path to custom test adapters' field (set to 'packages' for our repo, since we're using xunit) which causes the test run to fail with errors similar to: The test source file "(187a4a28b02d5c896372aaea0625e2440ffa3315),<<InsertSensitiveTestNameHere>>" provided was not found. Any ideas on that?

For this run, the 'Summary' tab looks like this:

@wouterroos can you share the build logs (after enabling system.debug = true) for "test source file ..." issue

For "Index was out of range. Must be non-negative and less than the size of the collection" issue let me check the code to see where the problem could be

@acesiddhu see the attachment for the logs, I've anonymized them so there could by 'duplicate' file references which can be disregarded. If you need anything else just ask.
log.txt

@acesiddhu @prawalagarwal I tried to reproduce the issue on my own machine based on the logging I sent you and it seems that the tests will run fine when the (187a4a28b02d5c896372aaea0625e2440ffa3315) part of the test names are not appended. It seems to only append that part for xunit tests but I'm unable to trace where these test names are coming from (my guess is from the Test Impact Analysis feature). Is there anything else I can do to help? We're really exited to start using this feature!

@wouterroos we got a repro today at our end using xunit tests. in the mean time can you tell if you are using VS 2015?
187a4a28b02d5c896372aaea0625e2440ffa3315 should not come from Test Impact Analysis feature. Did this test case name come when you disable test impact analysis?
Also can you ping me ur account name? you can mail me at devops_tools AT Microsoft DOT com. you might get mail delivery failure but I will get the mail

@acesiddhu we use Visual Studio 2015 Update 3 on our build server. If I look at older, successful logs I don't see these kind of numbers in the test execution logs. Which account name do you mean exactly?

@wouterroos your vsts accountname. .visualstudio.com

@acesiddhu Great, I sent it to the email address provided earlier.

@wouterroos I got a repro and I am working on a fix. Will keep you posted

Hi @wouterroos
Just to update. I saw that test name while using xunit.runner.visualstudio version 2.2.0 comes as fullyqualified name of the test where as in older version of adapter I saw test name was coming as just name (which is correct behavior)
Due to this there is a mis match while uploading test results to TFS. Can you confirm if you are also using 2.2.0?

@acesiddhu Yes, we indeed use xunit 2.2.0.

@acesiddhu Do you have any update on this or is there something else I can do to help?

@wouterroos we know the root cause of the issue. we can have a temporary fix on our side but we are looking at long term and correct fix. I am discussing this internally on this. Will post back on this. in the mean time is it possible for you to use xunit .99 till we roll out a fix or may be disable run only impacted tests? I will keep this thread posted as we get a fix out

@acesiddhu we have currently disabled the feature in order to pass our builds and we will wait until you guys come up with a fix. Thanks for all the help so far!

@wouterroos just an update. one part of the fix (the fix is ensure we publish results properly to server and get rid of the exception you reported) is fixed in this PR https://github.com/Microsoft/vsts-tasks/pull/4002. the fix will roll out in next deployment.
The next issue we are investigating is about test impact meta data that is used to run only impacted test wasn't working for me if I use xunit 2. so essentially if you check run only impacted tests we will always be running all the test now after the above fix. Will let you know about that investigation.

@acesiddhu good to hear, when will the next deployment take place? We could then enable the feature again to see of the first part is fixed.

It will start next week. But it depends on which scale unit your account is in. Will keep you updated

@acesiddhu Any updates on when either one of the issues will be fixed on our VSTS environment?

@acesiddhu running a build now, will post back the results when finished.

@acesiddhu just tried running our tests twice, but I'm still getting the same error message as before.

Can u share build logs to me on email address i shared before in the thread. Also r u using theories by any chance?
Sent from my Windows 10 phone
From: Wouter<EMAIL_ADDRESS>Sent: 01 May 2017 23:47
To<EMAIL_ADDRESS>Cc: Siddhartha<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [Microsoft/vsts-tasks] Exception : Index was out of range. Must be non-negative and less than the size of the collection. on VSTest\2.0.17\TestSelector\TestSelector.exe UpdateTestResults (#3744)
@acesiddhuhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Facesiddhu&data=02|01|siddhartha.pandey%40microsoft.com|2591b982a29f4e5ba5d108d490be5de6|72f988bf86f141af91ab2d7cd011db47|1|0|636292594666920615&sdata=BGzefKsY4uzuwCm2SZUHm%2FxxuRDMfcl9emTOE47r51s%3D&reserved=0 just tried running our tests twice, but I'm still getting the same error message as before.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fvsts-tasks%2Fissues%2F3744%23issuecomment-298392553&data=02|01|siddhartha.pandey%40microsoft.com|2591b982a29f4e5ba5d108d490be5de6|72f988bf86f141af91ab2d7cd011db47|1|0|636292594666920615&sdata=lXw%2Flx7Ysou9HAmERnm%2F9lZ%2FpOBX8CJItYGqpRGDpGg%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABHVt-81i22ZTxCavLTl8qM-pMx707zyks5r1iHIgaJpZM4MVxq8&data=02|01|siddhartha.pandey%40microsoft.com|2591b982a29f4e5ba5d108d490be5de6|72f988bf86f141af91ab2d7cd011db47|1|0|636292594666920615&sdata=qZMmBMiOcvaeokaaDGVFTuza%2FrYAUGyLye%2FNq0N22Ko%3D&reserved=0.

@acesiddhu I sent you the logs as requested. If you need any more information, just let me know. Regarding your question about the xunit Theory tests, we do use them, but as far as I can see from the logs, both Fact and Theory tests are causing errors.

@wouterroos we got the logs. we will look into this.

@acesiddhu Any update on this issue?

@wouterroos
Sorry for the delay. we were checking with xunit folks on this one.
There are overall two issues

Looks like you are using theories. Theories or data driven test cases wont work with run only impacted test cases. We should make a fix to atleast run all theories and not fail like this. This will ensure that if the test task is running mix of theories and facts we should atleast apply this feature in facts and run all theories.

Xunit 2 by default enables parallelism. we rely on certain events from xunit for this functionality. To make this feature work you need to have a runsettings file with this content. This is for Facts
<RunSettings> <RunConfiguration>  <DisableParallelization>true</DisableParallelization> </RunConfiguration> </RunSettings>

We can have a quick call on this and see how we can unblock you on this one?

@wouterroos I have added support for theories and data driven test cases for this feature to work in our backlog. I am closing this issue. Let me know and we can have a call to see if we can unblock you in the mean time

@acesiddhu It's not blocking for us at the moment, but we would love to have this functionality to improve our build times. If the feature doesn't work properly, disabling parallelism for xunit would only slow it down, or would this cause it to work as expected for Facts?

@wouterroos do let us know if Facts works by disabling parallelism.

@acesiddhu I tried running our tests again with DisableParallelization but I still get the same error as before.

Does it have theories? Whats the full log saying? If the log has 187a4a28b02d5c896372aaea0625e2440ffa3315 (some random number )they do have theories

@acesiddhu Yes it does, I was under the impression that you already created a fix that excludes [Theory] tests, but that doesn't seem to be the case. At the moment I have no easy way to skip all [Theory] tests in our solution so I can't really test if it would work for just [Fact] tests.

@wouterroos hmm. yes it wont work if you have theories. Let me see what can be done to atleast enable a fix scenario like urs before we have solution for theories

We are experiencing what seems to be the same issue: checking the "Run only impacted tests" box causes our build to fail with lots of "The test source file ... provided was not found" errors. We are also using theories in xUnit, although we are on v2.3.0 beta 1. Please let me know if there is anything we can do to help with testing or a fix. Since this bug is closed, is there somewhere we can track progress for always running theories or for handling them fully?
In the meantime, I just uncheck that box, and it works fine. It would be very nice to skip that step -- it takes about half an hour for around 3000 tests to run.

Next deployment will atleast get rid of this exception we see ("The test source file ... provided was not found")
Note that there isn't a way to associate the original discovered test with test case that were eventually executed because of data rows and hence you will end up seeing duplicates tests (one whos results are marked properly and one which will be marked not impacted. For instance assume there is a single test case named "Foo" with three data rows 1 2 and 3
The results will show these test with their outcomes eventually
Foo 187a4a28b02d5c896372aaea0625e2440ffa3315           NotImpacted
Foo kqjhjkefjgkhsdgfiuernbkhfea0625e2440ffa3315            NotImpacted
Foo 575a4a28b02d5c896372aaea0625e2440ffa3315           NotImpacted
Foo 1                                                                                Passed
Foo 2                                                                                Passed
Foo 3                                                                                Passed
Supporting data driven test cases fully (by fully I mean select that are impacted properly and have proper test results) is in backlog. we just ensured folks are not blocked in cases there is a mix of data driven test cases/theories and normal test cases and they have Run only impacted tests is switched on.


Added files parameter to define resources quickly

This patch allows for the creation of tftp::resources via a class parameter - possibly from hiera or an ENC.

I looked over the snippet ghoneycutt included and looked over the official puppetlabs modules (to ensure a consistent style) and didn't see hiera_hash in use for any of the Supported puppetlabs modules....

I use it in ghoneycutt/pam which is an approved module. Without the hiera_hash() lookup, the functionality you added will not work if you need to specify data at multiple levels in the hierarchy.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Dev/keysightdsox1102g

Adds Instrument KeysightDSOX1102G

Implemented suggested modifications, re-added the tests (skipped by default). Sorry for the delay, I didn't have the instrument to run the tests by myself these past weeks. New authors are colleagues who contributed to testing and final touches.

Updated the docstrings per suggestion. The Keysight-specific unit tests are all passing on our side.

I think we are done here. Thank you for your contribution, and your patience during iteration on this PR!

Thanks Christoph for your great comments and review! Glad to see these changes merged.


create potion object

[ ] Create tests for the Potion object.
[ ] The Potion object will be used to give the player stat boosts.
[ ] If no stat is provided, the stat should be randomly selected.

Closed by #6


Address feedback on RancherD docs #2935

https://github.com/rancher/docs/issues/2935
@rancher-max Should we call out any more environment variables than the rancherd version one? That's the only rancherd environment variable I know of.

@rancher-max Should we call out any more environment variables than the rancherd version one? That's the only rancherd environment variable I know of.

That is the most relevant one, and I like it the way you have it so I would say let's only include that.
For reference, the other environment variables come from https://github.com/rancher/rancher/blob/master/cmd/rancherd/install.sh and the available options for channel are from https://update.rancher.io/v1-release/channels. I assume at some point we will have a stable channel as well that may become default, and at that point it might be useful to document this variable.

@rancher-max Should we call out any more environment variables than the rancherd version one? That's the only rancherd environment variable I know of.

That is the most relevant one, and I like it the way you have it so I would say let's only include that.
For reference, the other environment variables come from https://github.com/rancher/rancher/blob/master/cmd/rancherd/install.sh and the available options for channel are from https://update.rancher.io/v1-release/channels. I assume at some point we will have a stable channel as well that may become default, and at that point it might be useful to document this variable.


SITL installation script: drcsim installation fails on ubuntu

@AndreasAntener  Can you maybe give this a test when you run the installation anyway for some reason? It's not high priority though.

From @mverzilli:

The px4_ros_installation_ubuntu.sh works great except for the last line: sudo apt-get -y install drcsim. It gives “Unable to locate package drcsim”.

By inspecting the corresponding Packages list, I can see that indeed drcsim isn’t there (http://packages.osrfoundation.org/drc/ubuntu/dists/trusty/main/binary-i386/Packages).

But Gazebo’s own installation instructions do suggest installing drcsim just like your installation script does, so I’m a bit clueless as to what happened to that package. My guess is it either got subsumed but something bigger (maybe gazebo itself?) or it got renamed.

Thanks for taking care of this! I've got some more info now:
In https://bitbucket.org/osrf/drcsim/wiki/DRC/ChangeLog, for the last version they mention:
Debian distributions moved to new server (http://old.gazebosim.org/assets/distributions/).

There's a bunch of .deb packages over there for different versions of drcsim, so I could try dpkg installing the appropriate one.

I just noticed that the repository for i386 doesn't have drcsim but the one for amd64 does. @mverzilli are you installing on a 32bit system?
Just found this:
"Note: as of drcsim 2.2.0, binary packages are available for 64-bit systems only. A 32-bit version is still available as source, but 64-bit is required to run the BDI walking controller."
from here
I'm not sure what this means for us, but ATM we actually don't need drcsim right? @thomasgubler ;)

Otherwise, the setup still works for me (docker container built successfully)

Indeed. I was attempting all of this on a 32-bit Ubuntu 14.04 VirtualBox image. It actually was an accident, I intended to download a 64-bit image :/. I'll give it a try on a 64-bit VM and see if it works (it looks like it will :)). Thanks for the quick response, pretty impressive!

Great :)
But just so you know: I'm having trouble compiling the whole thing at the moment. There might be really a problem there somewhere.

Ok, it seems that compilation for me failed because my VM didn't have enough memory (only 1GB). Got it working with 4GB.
Looking forward to your results ;)

Hey! Sorry for the delay, it took me a while to download a 64-bit image and I switched to work on other things. So, I followed the tutorial again and this time everything went smoothly :). Now I'm blocked because it seems I don't have access to a couple of repos:
https://github.com/PX4/euroc_simulator.git
https://github.com/PX4/mav_comm.git
Could you grant me access?
By the way, there's a broken reference to the script px4_workspace_setup.sh at https://pixhawk.org/dev/ros/sitl (I still grabbed it from https://github.com/PX4/Firmware/blob/master/Tools/ros/px4_workspace_setup.sh).

You already have access. See my email from Feb 5. You need to accept the
invitation.
-Thomas
On Mon, Feb 9, 2015 at 2:42 PM, Martin Verzilli<EMAIL_ADDRESS>wrote:

Hey! Sorry for the delay, it took me a while to download a 64-bit image
and I switched to work on other things. So, I followed the tutorial again
and this time everything went smoothly :). Now I'm blocked because it seems
I don't have access to a couple of repos:
https://github.com/PX4/euroc_simulator.git
https://github.com/PX4/mav_comm.git
Could you grant me access?
By the way, there's a broken reference to the script
px4_workspace_setup.sh at https://pixhawk.org/dev/ros/sitl (I still
grabbed it from
https://github.com/PX4/Firmware/blob/master/Tools/ros/px4_workspace_setup.sh
).
—
Reply to this email directly or view it on GitHub
https://github.com/PX4/Firmware/issues/1750#issuecomment-73509827.

closing since the original issue was resolved/present only on i368

@thomasgubler @AndreasAntener thanks for your help! I was able to build and run it.


Xcode 10.0 & Swift 4.2

Migrates developer project to use Swift 4.2, Update CI configuration to add additional Xcode 10 test and updates iphone & apple tv tests to xcode 10.
This PR includes zero code changes, the project was already compatible with Swift 4.2, and thus no changelog entry is required.

Codecov Report

Merging #374 into master will decrease coverage by 0.56%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #374      +/-   ##
==========================================
- Coverage     100%   99.43%   -0.57%
==========================================
  Files           5        5
  Lines         175      176       +1
==========================================
  Hits          175      175
- Misses          0        1       +1

Impacted Files
Coverage Δ

ReSwift/Utils/Assertions.swift
83.33% <0%> (-16.67%)
:arrow_down:

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 b531ce2...271c49c. Read the comment docs.

Interesting how the LastSwiftMigration setting behaves. But anyway, LGTM :) 👍


Rails 4 fix report serialization

There were additional serialized objects being stored as binary blobs that are not report objects. They are actually PDF and CSV contents. The original migration code did not account for this and blew up with an error indicating an unknown format. This change detects these other serializations of reports and skips converting them. They are fine as is.
I also added error handling around the conversion so that if an error is encountered it will output to the screen and continue on.

@Fryguy Please review

I also added error handling around the conversion so that if an error is encountered it will output to the screen and continue on.

Why? It seems to me that having this blow up loudly, and force us to identify and fix the problem, is precisely the right thing -- as shown by this very change.
The alternative is to happily continue, leaving data in the DB post-migration that we currently have no idea could be there... is it better to get a firm error during a migration, in a maintenance window, or to leave the unanticipated data to strike later when someone happens upon a particular button in the UI?

I struggled with that question when I originally created the migration and ultimately decided to have it blow up loudly. I guess I got swayed by the issues that people were having. I'm fine with going back to allowing it to blow up. Hopefully there all of the variations of serialized reports have now been accounted for.


max-width with sideNav

I have this layout.
!!!
%html{:lang => "en"}
  = render "layouts/head"
  %body
    .max-container
      %header
        = render "layouts/header"
      %main
        = yield
      = render "layouts/footer"

And this class to restrain the width of my site.
.max-container
  margin-right: auto
  margin-left: auto
  max-width: 1200px

How to merge A and B as illustrated on the below image such that both parts will stick together and centre horizontally?

As this is not an issue/bug this is better asked on Stack Overflow.


Compile to file stopped working (?)

After updating to Atom 1.13.0-beta3 sass-autocompile (0.13.1) "Compile to file" stopped working. Nothing happens when running that command. "Compile direct" still works fine.

Can you open Developer Tools (View -> Developer -> Toggle Developer Tools) and use "Compile to file"? Perhaps an error occurs which is printed to console, but normally my package should show an exception then...

Nope. No errors in the Console.
But the Deprecation Cop in Atom is reporting this for sass-autocompile:
Argument to path.extname must be a string
Object.extname - C:\Users\emin\AppData\Local\atom\app-1.13.0-beta3\resources\app.asar\src\electron-shims.js:20:10
Function.hasFileExtension - C:\Users\emin.atom\packages\sass-autocompile\lib\helper\file.coffee:149:28
Object.isSassFile - C:\Users\emin.atom\packages\sass-autocompile\lib\sass-autocompile.coffee:508:19
Object.compileToFile - C:\Users\emin.atom\packages\sass-autocompile\lib\sass-autocompile.coffee:395:16
atom-workspace.subscriptions.add.atom.commands.add.sass-autocompile:compile-to-file - C:\Users\emin.atom\packages\sass-autocompile\lib\sass-autocompile.coffee:311:26
CommandRegistry.handleCommandEvent - C:\Users\emin\AppData\Local\atom\app-1.13.0-beta3\resources\app.asar\src\command-registry.js:259:29

Thanks for your report. I will have a closer look to that! I suppose that this is the problem!

I fixed the bug with version 0.13.2


We need a way to automatically label bugs as "Triage"

Right now there is no way to know whether a bug has been triaged or not. We need a way to automatically label a bug as "Triage needed" so we know we need to triage it.

Already done.


Add link rel=canonical on rewritten images and pdfs

See https://support.google.com/webmasters/answer/139066?hl=en

fixed in 0ea2124fcdb0ae94e2

I'm eager to deploy this feature, but it is 11 months old now and still unreleased. Is this planned for the next minor release, and if so when is that?

@ademarre As far as I can tell this feature was released in <IP_ADDRESS>, however:
What seems a bit odd is that when I check the commit that adressed this (https://github.com/pagespeed/mod_pagespeed/commit/0ea2124fcdb0ae94e25383ac8f689b18edbedd26) to see on which branches it was included, for 1.11.3.x it only shows up for <IP_ADDRESS>, which seems odd (what about <IP_ADDRESS> and <IP_ADDRESS>?).
What version of mod_pagespeed are you currently running?

I'm currently running <IP_ADDRESS>. I must have been wrong about it not being released...
$ git branch -r --contains 0ea2124
  origin/<IP_ADDRESS>-scratch
  origin/<IP_ADDRESS>
  origin/<IP_ADDRESS>
  origin/<IP_ADDRESS>
  origin/<IP_ADDRESS>
  origin/<IP_ADDRESS>
  [...]

The commit is in the <IP_ADDRESS> branch. (I don't see a tag for that release, so I'm assuming that branch is correct.) But I'm not seeing the canonical link header on rewritten images. I'm an outsider on this codebase, but might it be related to the fix in 1ddc05c which has not been released?
My config is essentially this:
ModPagespeed on
ModPagespeedRespectVary on
ModPagespeedPreserveUrlRelativity on
ModPagespeedMapProxyDomain https://www.example.com/pagespeed/proxy/cdn https://cdn.example.com
ModPagespeedRewriteLevel CoreFilters
ModPagespeedEnableFilters prioritize_critical_css,insert_dns_prefetch

Is the rel=canonical link not added for proxied images?

According to John Mueller, Google doesn't use rel=canonical for images:

On the one hand, for images we don't use the rel=canonical. So if you have the rel=canonical header in the images themselves, we don't use that.

I still feel it's correct to set the header if possible though. In principle it seems like an accurate signal to broadcast, and one that might become more meaningful to Google in the future.

@ademarre please read the comments on that link you gave.  In particular:
"I wondered the same thing when configuring maxcdn a while ago. They add the canonical source to the http header sent from the CDN, which is not the same as adding a rel="canonical" to an html file. Google started supporting this method in 2011: https://webmasters.googleblog....
I think John must have meant canonical within an image file is not recognized. I have several sites using CDNs with canonical headers and the CDN versions of images don't show up in search while the originals do."

@ademarre Testing, I don't see the bleeding edge of mod_pagespeed adding a Link header for requests routed via MapProxyDomain. I think that would be a new feature - so you could consider opening a new issue with a feature request for that.

@jmarantz, considering the context in that Google Hangout video, I doubt John Mueller was referring to setting canonical links within image files. I've never heard of such a thing. Perhaps John is simply mistaken, but I'm more likely to believe his explanation than that commenter's anecdotal account. The cited blog makes clear that the rel=canonical link header is supported for PDFs, but says nothing about images.
Nonetheless, this probably isn't the right forum for this discussion.
Thanks for your help, guys!


Add new toc

Describe changes
Synced docs: https://zenml-io.gitbook.io/hamza-test
Pre-requisites
Please ensure you have done the following:

[ ] I have read the CONTRIBUTING.md document.
[ ] I have added tests to cover my changes.
[ ] I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.
[ ] IMPORTANT: I made sure that my changes are reflected properly in the following resources:

[ ] ZenML Docs
[ ] Dashboard: Needs to be communicated to the frontend team.
[ ] Templates: Might need adjustments (that are not reflected in the template tests) in case of non-breaking changes and deprecations.
[ ] Projects: Depending on the version dependencies, different projects might get affected.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

@htahir1 fixed the links from the broken links feature on gitbook. it still shows some but I can't find them on those pages. looks all fixed to me.


rename config.auto_include_turbolinks to config.turbolinks.auto_include

I'll do a PR if the change makes sense to the repo collaborators

:+1:


Issues + request

These issues can be reproduced on a brand new installation of Spotify + Spicetify + Marketplace with just Bloom installed from the Marketplace.
Still loving this Spotify theme. Thanks for it!
Issues:

1. Play and Pause buttons are overlayed onto themselves.

2. Some elements have the same color no matter the color scheme. Nothing in color.ini will affect them

3. Minor issue: there is no blur when clicking the user menu, unlike other dialogs

4. Play bar obstructs what Friends are playing as well as Lyrics

5. Weird glitch when scrolling through sidebar playlists
It scrolls fine until it gets about halfway through, then that happens...

[Screencast from 2023-03-28 18-03-48.webm](https://user-images.githubusercontent.com/81482539/228366728-90c559f7-e386-4039-abc4-27ccb1995e90.webm)

6. Liked Songs + Your Episode + New Play list buttons won't remain hidden once Spotify is restarted.

Request

1.  Any chance we could get support for custom images to be displayed instead of the folders? Ideally like Dribbblish, but anything similar really. Personally I am not a fan of how they look, and they even look broken on some dark color schemes... like Comfy and Dark Mono

@stpnwf thanks for reaching out and providing a well-organized issue. we will do our best to resolve these issues in the near future

Minor issue: there is no blur when clicking the user menu, unlike other dialogs

Can't reproduce

Can't reproduce

I am on Linux, maybe that is it. Tried it on both Fedora 37 and Vanilla OS with Bloom from the marketplace and Spotify installed via Flathub.
Although some of those issues may not be relevant anymore given Spotify's redesign. Unsure if it will influence Bloom's interface going forward.

3 is definitely a spicetify issue, not a theme

Can't reproduce

I am on Linux, maybe that is it. Tried it on both Fedora 37 and Vanilla OS with Bloom from the marketplace and Spotify installed via Flathub.
Although some of those issues may not be relevant anymore given Spotify's redesign. Unsure if it will influence Bloom's interface going forward.

Same here, pause and play buttons overlay each other, this is definitely a linux problem, as i'm running fedora 38 + GNOME

Can't reproduce

I am on Linux, maybe that is it. Tried it on both Fedora 37 and Vanilla OS with Bloom from the marketplace and Spotify installed via Flathub.
Although some of those issues may not be relevant anymore given Spotify's redesign. Unsure if it will influence Bloom's interface going forward.

Same here, pause and play buttons overlay each other, this is definitely a linux problem, as i'm running fedora 38 + GNOME

Try updating Spicetify and Bloom now. That issue is resolved.
Tested environments: ChromeOS Linux Development Environment (Debian Bullseye), EndeavourOS GNOME, ZorinOS 16.2

Play bar obstructs what Friends are playing as well as Lyrics

fixed in https://github.com/nimsandu/spicetify-bloom/commit/be77059df1c1bfe5f5d7ae0c6ecbdc08c15ebcc2

Weird glitch when scrolling through sidebar playlists

Fixed in https://github.com/nimsandu/spicetify-bloom/commit/a4ef0b5fc14ba19360624ece033c581efe442014

Some elements have the same color no matter the color scheme. Nothing in color.ini will affect them

and more can't be fixed:

Some elements have the same color no matter the color scheme. Nothing in color.ini will affect them

fixed completely in https://github.com/nimsandu/spicetify-bloom/commit/47bb62196dbc5a20358f13f772da72204628b51f

@stpnwf so all issues are fixed now?
nice color scheme you have btw, do you want to make pr with it?

Any chance we could get support for custom images to be displayed instead of the folders? Ideally like Dribbblish

added in https://github.com/nimsandu/spicetify-bloom/commit/a3b751bd4732f7a1df05d43a67ee671a9961a181
@stpnwf are we ready to close the issue?

Minor issue: there is no blur when clicking the user menu, unlike other dialogs

Can't reproduce

Still can't, but I now I know the reason. As karimawii has in #259 you have a menu appears right after user-widget box element, not at the very end of the body. Karimavi, as I see, has no blur either, which confirms my theory.
For the same reason, for example, this menu does not have transparency:

And this is what happens if we move it to the end of the body:

Backdrop filters is working!
I don't know how to fix this right now.

fixed

@stpnwf are we ready to close the issue?
nice green color scheme you have btw, do you want to make pr with it?

I won't be able to check it until hopefully this weekend, but I will close this issue and open another if there are still issues.
Do you have any idea when the updated version will hit Marketplace?

nice green color scheme you have btw, do you want to make pr with it?

Thanks, it is heavily inspired by the Zorin OS Dark Green color scheme. I really love it.
If you'd like I can make a PR, but I have never made one before, so I'd have to look up how to do it. LMK
These are the colors though:
text=BBF1DD
subtext=6b8a7f
alt-text=6b8a7f
main=1B2421
sidebar=1B2421
player=101411
player-border=33433d
player-bar-shadow=BBF1DD
player-bar-bg=323336
card=101411
shadow=101411
selected-row=33433D
button=28c48b
button-active=00ac6f
button-disabled=33433D
tab-active=33433D
notification=6b8a7f
notification-error=d43353
misc=6b8a7f
not-selected=6b8a7f
accent=BBF1DD
layer-shadow=1B2421
contour=1B2421

Sadly, I don't have a screenshot for it ATM bc I am playing around with a frankeinstein theme rn (in the pic below), but the pic below should have the same feel as the above color scheme.

Do you have any idea when the updated version will hit Marketplace?

Usually it takes two days

If you'd like

Personally I definitely would

but I will close this issue and open another if there are still issues.

It's better not to close current issue until you are sure that all is fixed than to close and open a new one

Usually it takes two days

Awesome, then I will check that and file the request on Sunday.

It's better not to close current issue until you are sure that all is fixed than to close and open a new one

Alright, thanks for reopening it.

@stpnwf hello, just a quick reminder

Sorry, I forgot.
Most issues were fixed but I still encountered some:

pic 3

what's wrong?

@SunsetTechuila look at queue button

Thanks. I'll probably make all the buttons spice-text color

I'm waiting for the color scheme first of all)

I am having issues with SSH, will try pushing it once I get it working.

I'm waiting for the color scheme first of all)

Here you go!

@stpnwf Are there still any issues?

@stpnwf i closed this as this is inactive, be comfortable to reopen anytime if needed! thanks!


GenSoftware: vmg not present in Ubuntu repos

2-GenSoftware.sh installs vmg which is not present in the Ubuntu repositories. For installing vmg, the tar file has to be downloaded from
https://liquidtelecom.dl.sourceforge.net/project/magnifier/magnifier%20for%20Linux/3.5/magnifier-linux-3.5.tar.bz2
This is a very old version, last updated in 2011.
There are alternatives such as kmag which is present in the Ubuntu repos.

kmag seems to install a lot of libk* libraries and seems unnecessarily bloated. For now I'll just add a condition to install it if version >17


Tool bar thinks folders whose names end with ".zip" are archives

Description

As you can see in the screenshot, this folder named MSYH_V20230523.zip is recognized as a zip archive by the tool bar and shows the Extract button
Steps To Reproduce

Create a folder whose name ends by .zip
Enter the folder, don't select anything
Observe the bug

Requirements
The Extract button should not appear since it is not an archive and there's nothing to extract
Files Version
<IP_ADDRESS>
Windows Version
10.0.26100.1150
Log File
debug.log

Thank you for the report, we're tracking a similar issue in #15886. The same fix can be used so we'll continue tracking there.

A fix will be included in the next update.


Charging Station - Fix Amperage Limit

Update the method getChargingStationAmperage() to return the safe min/max value
For handle wrong amperageLimit values : when OCPP Status Notif is received, in method enrichChargingStationConnectorWithTemplate(), you can check the amperageLimit and update with the min/max safe value

@jerome-benoit
I think I found the part in the code where the amperageLimit is set to 0. My guess the difference is caused by different templates (or the handling of the difference):
The processing is done in the CheckChargingStationTemplateTask
There we have the following code snipped:
for (const connector of chargingStation.connectors) {
          // Amperage limit
          if (!Utils.objectHasProperty(connector, 'amperageLimit')) {
            connector.amperageLimit = connector.amperage;
            chargingStationUpdated = true;
          } else if (Utils.objectHasProperty(connector, 'amperageLimit') && connector.amperageLimit > connector.amperage) {
            connector.amperageLimit = connector.amperage;
            chargingStationUpdated = true;
          }

The template of SAP Mougins 15 does not have an amperage value on connector level right?
That iswhy the amperageLimit is set to 0 when the task runs.
For example SAP Mougins 12 has an amperage value on connector level. That is why the amperageLimit would be set this case.

@jerome-benoit
Is there something else needed?

@jerome-benoit
Is there something else needed?

There's no automated adjustment of the amperageLimit value if it's below the ev ready min for the charging station. I think it's checked a setting the value so it's maybe not necessary add an adjustment in the same code path.

Done in https://github.com/LucasBrazi06/ev-server/pull/1944


[Xcode 7.1]Failed to build workspace SwiftyJSON with scheme SwiftyJSON iOS

When i use cartheage update , get the following error:
xcodebuild: error: Failed to build workspace SwiftyJSON with scheme SwiftyJSON iOS.
Reason: The run destination iPad 2 is not valid for Running the scheme 'SwiftyJSON iOS'.
A shell task failed with exit code 70:
This problem was happened before?

Please upgrade to carthage version 0.9.2

Hao le, thanks!


Resolve Issue #1: Use API param key for compatibility with older OneFS versions

7.x versions of OneFS do not recognize the query param keys for the statistics current and history endpoints:

/statistics/current
/statistics/history

This PR updates the browser to use the key param which is respected in 7.x and 8.x versions of OneFS.
Resolves issue #1

This fix will go in with a new PR tomorrow.


Offline User Access

Hi there. Great library, I appreciate the detail and the docs a lot.
How do I use the access token for offline access? I am using the google auth flow and need to manage a
user's email.
Thanks.

But then you can't ask for offline access, so then you can use the refresh token to get a new token when the token has expired and the user is not online.

@sahat @asmerkin In the explicit grant flow in Satellizer, how would one get the refresh token and refresh the token on expiry? I would like to make server side offline calls pull Google user data.

Hi there. I just had that problem and i solved it using optional request url params.
$authProvider.google({
  optionalUrlParams: ['access_type'],
  accessType: 'offline'
});

Once your account has been authorised, you can store refresh_token to database and use it in your app.

This is what I did too.
On Thu, Jul 16, 2015, 06:37 dgrubelic<EMAIL_ADDRESS>wrote:

Hi there. I just had that problem and i solved it using optional request
url params.
$authProvider.google({
optionalUrlParams: ['access_type'],
accessType: 'offline'
});
Once your account has been authorised, you can store refresh_token to
database and use it in your app.
—
Reply to this email directly or view it on GitHub
https://github.com/sahat/satellizer/issues/205#issuecomment-121922366.

Thanks! Got it working now.

+1

@sahat this should be added to the docs. spent three hours researching this only to have google eventually turn up this issue!


Move HTML5 to a separate plugin

Move HTML5 to a separate plugin. Use SASS as CSS source.

Running ./gradlew dist from this feature branch successfully builds docs that are equivalent to docs output from develop, but related links are missing.

Running the ant build from the docs subrepo using the OT state from this feature branch fails with the following errors:
BUILD FAILED
~/docs/build.xml:122: The following error occurred while executing this line:
~/docs/build.xml:31: The following error occurred while executing this line:
~/dita-ot/src/main/build.xml:41: The following error occurred while executing this line:
~/dita-ot/src/main/build.xml:87: The following error occurred while executing this line:
~/dita-ot/src/main/plugins/org.dita.xhtml/build_dita2xhtml.xml:34: stylesheet ~/dita-ot/src/main/plugins/org.dita.xhtml/xsl/map2html5-cover.xsl doesn't exist.

@infotexture, I can't reproduce the issue. Could you create a Gist with the full log file?

Sure: https://gist.github.com/infotexture/b4ea7094713922ee5a0c.

Ok, the transtype was tocjs, not html5. I changed TocJS to use HTML5 at some point, but I think I'll revert it back to XHTML, to the original state.

Sorry, I was running the ant build from the docs subrepo out of habit.
Running dita -i userguide.ditamap -f html5 builds docs output successfully.

Test-merged infotexture/dita-ot@0f5e316 into dita-ot/dita-ot@1c28763, re-ran the ant build from the docs subrepo, BUILD SUCCESSFUL.


About Link Not Routing Correctly

Clicking on "How It Works" doesn't work as intended. Goes to 404.

Just needed a redeploy on Heroku :)


pd.to_datetime raises expection when converting date object with unit param

Hello, since pandas 0.18.1, converting to datetime a date defined as object, using the unit parameter, raises a ValueError exception.

I don't know why you would think this would work.
from the doc-string.
unit : unit of the arg (D,s,ms,us,ns) denote the unit in epoch
    (e.g. a unix timestamp), which is an integer/float number.

In prior versions this should have raised but didn't (as was a bug).

Thank you.
I was confused considering the prior version.
Didn't noticed the change in API.


Fix coverity 1506297 & 1506298: negative returns

Two more that had previously been ignored.

Ping @openssl/committers for second review

In dh_pub_encode() it frees the ptr also - not sure if there is a reason for this..

In dh_pub_encode() it frees the ptr also - not sure if there is a reason for this.

Huh?  This isn't about that function.

In dh_pub_encode() it frees the ptr also - not sure if there is a reason for this.

Huh? This isn't about that function.

dh_pub_encode != dh_priv_encode (Just saying they have different patterns in the err: blocks)

This pull request is ready to merge


Slim docker image

Using mesos image as base, removing caches and unneded files after sbt.
Image size is much smaller, 1.316gb vs 1.951gb for current image. Moreover, if you run marathon on the same node as mesos (which is common case), you get 1.172gb shared with mesos-master image. It makes 150mb on disk instead of 1951mb. Layer where marathon jar is built:
47f807a08209        23 seconds ago      /bin/sh -c sbt assembly &&     mv $(find targ   59.39 MB

And old one:
7fc8b0dc7d69        6 weeks ago         /bin/sh -c sbt assembly                         483.9 MB

However, ideal case would be (single RUN step):

Install jdk, scala, sbt
Build marathon jar
Remove jdk, scala, sbt
Install jre

This way image would only contain needed files to run marathon, not to build it. Disk and bandwidth savings compared to proposed version are not that big to do it now.
Bonus: mesos version is now more obvious.

I updated base image to mesosphere/mesos:0.21.1-1.1.ubuntu1404.

:ship:


Update fastlane to use new TestFlight endpoints

Looks like Apple has added some new features to TestFlight which have broken our current code. We were kind of expecting this would happen at somepoint since the last time they made changes in the spring. This will make sure that everything is up-to-date using the new testflight/v2 endpoints which are much nicer anyways :smile:

          1 Warning

:warning:
Pull Request is Work in Progress

  Generated by :no_entry_sign: Danger

This is ready for review/merge @taquitos! As we discussed in chat, we want to get this in so that people can try it out in nightlies and report back.

How do we want to deal with the tests that are commented out? Just merge it as is for now, and then do a follow-up PR early next week?

Yeah exactly @KrauseFx! Definitely not ideal, but thinking its the best way forward to make sure we are on the right track.


feat(tracing): Initial tracing experiments

This is the most basic hack I could come up with that gets us into the direction
of correlating errors.  It currently uses a made up trace context but i think
realistically speaking we probably want to use attributes for this.

(This is not intended for merging, just for experimenting)

I think we could infer span context from request headers serverside.

from flask import Flask
import requests
import sentry_sdk
import sentry_sdk.integrations.flask

sentry_sdk.init(
<EMAIL_ADDRESS>    propagate_traces=['http://localhost:5000'],
    integrations=[sentry_sdk.integrations.flask.FlaskIntegration()]
)

app = Flask(__name__)

@app.route('/')
def call_out():
    return requests.get('http://localhost:5000/broken').json()['foo']

@app.route('/broken')
def broken():
    1 / 0


Fix license query param to items.locations.license

https://github.com/wellcomecollection/docs/pull/26/files#r415723355

I think Jamie did this in the latest round of aggregations cleanup.


Is the technique applicable to Llama / Falcon models?

In other words, am I wrong to say that this will bring long context lengths to the various existing OSS models?
(Exciting work either way, congrats!)

+1


Error code 1501: Passwords must not exceed 8 characters

If your password is longer than 8 characters, you will always get a 1501.
However, your password can contain non-alphanumeric characters.

Okay, this seems like it might actually be a length limit. Currently trying to narrow down the exact limit, but I can use symbols if the password is 8 characters.

Just wanted to share my own experience regarding this issue.
I've just implemented a simple script for some home automation and my password is well beyond 8 characters.
I haven't run into any issues though. Maybe the update for my P100 plugs I installed yesterday fixed this on their end?
Went from 1.3.3 to 1.3.4.

My devices are on 1.3.4 as well, and I haven't updated them since before I tested this. How strange, there must be something else to it than just length or special characters.

Hi. Any solution?
I have changed my password to 1234567o but still not working...

Also get the 1501 error. Though notice that https://github.com/K4CZP3R/tapo-p100-python experiment does not have the same issue. Works fine even with my long password.

I am still working on a fix for this but thanks for all your help!

I get some problem and change password only 8 ch and this work

I believe I finally fixed this in the latest version (0.1.0) :)


Execution of ServiceCallout SCO-AddURLVault failed

ZnQC0gxUxpLzh2SLAsX0G5UCydmB{"Envelope":{"encodingStyle":"http://schemas.xmlsoap.org/soap/encoding/","Body":{"Fault":{"faultcode":"soap:Server","faultstring":"Execution of ServiceCallout SC-SearchURL failed. Reason: ResponseCode 503 is treated as error","faultactor":{},"detail":{"source":{"errorcode":"steps.servicecallout.ExecutionFailed"}}}}}}{"Envelope":{"encodingStyle":"http://schemas.xmlsoap.org/soap/encoding/","Body":{"Fault":{"faultcode":"soap:Server","faultstring":"Execution of ServiceCallout SC-SearchURL failed. Reason: ResponseCode 503 is treated as error","faultactor":{},"detail":{"source":{"errorcode":"steps.servicecallout.ExecutionFailed"}}}}}}

i have following your tutorials and i have happened to have this error ,kindly can you help?

Have you successfully initiated an stk push? I have an updated send stk code.

Hi, i  was doing a C2B integration  without initiating an stk push.
I have not succeded yet!
On Tue, 15 Nov 2022 at 17:45, Stephen Ndegwa @.***>
wrote:

Have you successfully initiated an stk push? I have an updated send stk
code.
—
Reply to this email directly, view it on GitHub
https://github.com/bnjunge/MPESA-API-Tutorial/issues/7#issuecomment-1315412883,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYPEBQ5HCUEQWJQSAHS546TWIOO2JANCNFSM6AAAAAAR6JLNCA
.
You are receiving this because you authored the thread.Message ID:
@.***>

This is the error message am getting
{"Envelope":{"encodingStyle":"http://schemas.xmlsoap.org/soap/encoding/","Body":{"Fault":{"faultcode":"soap:Server","faultstring":"Execution
of ServiceCallout SC-SearchURL failed. Reason: ResponseCode 503 is treated
as
error","faultactor":{},"detail":{"source":{"errorcode":"steps.servicecallout.ExecutionFailed"}}}}}}
On Wed, 16 Nov 2022 at 09:33, PATRICK WAMBUA @.***> wrote:

Hi, i  was doing a C2B integration  without initiating an stk push.
I have not succeded yet!
On Tue, 15 Nov 2022 at 17:45, Stephen Ndegwa @.***>
wrote:

Have you successfully initiated an stk push? I have an updated send stk
code.
—
Reply to this email directly, view it on GitHub
https://github.com/bnjunge/MPESA-API-Tutorial/issues/7#issuecomment-1315412883,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYPEBQ5HCUEQWJQSAHS546TWIOO2JANCNFSM6AAAAAAR6JLNCA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi,
Did you find  a way the error  could be solved?
On Wed, 16 Nov 2022 at 09:36, PATRICK WAMBUA @.***> wrote:

This is the error message am getting
{"Envelope":{"encodingStyle":"http://schemas.xmlsoap.org/soap/encoding/","Body":{"Fault":{"faultcode":"soap:Server","faultstring":"Execution
of ServiceCallout SC-SearchURL failed. Reason: ResponseCode 503 is treated
as
error","faultactor":{},"detail":{"source":{"errorcode":"steps.servicecallout.ExecutionFailed"}}}}}}
On Wed, 16 Nov 2022 at 09:33, PATRICK WAMBUA @.***> wrote:

Hi, i  was doing a C2B integration  without initiating an stk push.
I have not succeded yet!
On Tue, 15 Nov 2022 at 17:45, Stephen Ndegwa @.***>
wrote:

Have you successfully initiated an stk push? I have an updated send stk
code.
—
Reply to this email directly, view it on GitHub
https://github.com/bnjunge/MPESA-API-Tutorial/issues/7#issuecomment-1315412883,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYPEBQ5HCUEQWJQSAHS546TWIOO2JANCNFSM6AAAAAAR6JLNCA
.
You are receiving this because you authored the thread.Message ID:
@.***>

HI
On Thu, 17 Nov 2022 at 13:40, PATRICK WAMBUA @.***> wrote:

Hi,
Did you find  a way the error  could be solved?
On Wed, 16 Nov 2022 at 09:36, PATRICK WAMBUA @.***> wrote:

This is the error message am getting
{"Envelope":{"encodingStyle":"http://schemas.xmlsoap.org/soap/encoding/","Body":{"Fault":{"faultcode":"soap:Server","faultstring":"Execution
of ServiceCallout SC-SearchURL failed. Reason: ResponseCode 503 is treated
as
error","faultactor":{},"detail":{"source":{"errorcode":"steps.servicecallout.ExecutionFailed"}}}}}}
On Wed, 16 Nov 2022 at 09:33, PATRICK WAMBUA @.***> wrote:

Hi, i  was doing a C2B integration  without initiating an stk push.
I have not succeded yet!
On Tue, 15 Nov 2022 at 17:45, Stephen Ndegwa @.***>
wrote:

Have you successfully initiated an stk push? I have an updated send stk
code.
—
Reply to this email directly, view it on GitHub
https://github.com/bnjunge/MPESA-API-Tutorial/issues/7#issuecomment-1315412883,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYPEBQ5HCUEQWJQSAHS546TWIOO2JANCNFSM6AAAAAAR6JLNCA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Add rich text associations for each locale

When working with the Gem, I noticed that (given a translates :content), there was a rich_text_content association, which yields the associated model for the content in the current language. However, sometimes, not the current, but a specific translation is required. I therefore added new associations for each locale, e.g. rich_text_content_en and rich_text_content_fr.

Thanks!
It's possible to have rich text in a different locale by wrapping like so: I18n.with_locale(locale) { record.rich_text_content }, but Mobility does have an API similar to what you suggest, so it could be good to add it for coherence.

@sedubois Yes, using I18n.with_locale is possible for some cases - but not for e.g. a posts.includes(:rich_text_content_en).
And in my case, I was using the pg_search gem and was defining pg_search_scopes for different languages, with associated tables - where I explicitly wanted to query against _de or _en-associations 🙂

@Timmitry hello, for some reason this update breaks my app, since it attempts to generate stuff like rich_text_title_zh-TW :
SyntaxError: /home/crohr/.asdf/installs/ruby/2.7.4/lib/ruby/gems/2.7.0/gems/activerecord-6.1.5/lib/active_record/associations/builder/association.rb:102: syntax error, unexpected '-', expecting ';' or '\n'
        def rich_text_title_zh-TW

Any idea?

It might be because of the dash in the locale. Could you propose a PR with the line below changed to has_one :"rich_text_#{name}_#{locale.underscore}", ... with an extra test?
https://github.com/sedubois/mobility-actiontext/blob/ff68ef13b3ecc3b6e55028862ffe4f4d0ebd2322/lib/mobility/backends/action_text.rb#L87

This should be fixed in release 1.1.1.


fix: Remove invalid source-map comments

Coverage decreased (-0.4%) to 99.558% when pulling 2b334db7c2808979dfc46d68e861abf3e53e0064 on remove-source-maps into 2b22f54d837b07fd3fbc2109bd2042b3e6c04bb3 on master.

Coverage remained the same at 100.0% when pulling a294c084416799af4fb131f62ed2d6981a5d3190 on remove-source-maps into 2046114f79abab0b300960b5451630b77c9a4644 on master.


Allow to pass metaheuristics from CLI to solvers

As User,
I want to pass metaheuristic directly from CLI,
so i dont have re-compile everytime i want to experiment;

seems StructOpt gives required high-level around CLAP: https://github.com/TeXitoi/structopt


Config management problems + interest for PR ?

Hello,
I read a lot of users are wondering how to connect to different accounts, and also people wondering why there are heavy differences between sim and trade behavior.
There is a @zenbot:conf object that is loaded with either sample-conf.js or conf.js in boot.js, and then requested with the get('conf') method. The problem is that this object doesn't take into account command parameters (including the --conf one), which results in different configurations depending on the modules implementation.
Some only request the global object (exchanges), when others parse command options to update it (trade/sim). In this case there is also the problem of --conf file rewriting other command options, while NOT rewriting the ones in @zenbot:conf, hence NO multiple accounts).
I think the default behavior should be :

sample-conf.js => conf.js => --conf file => command options

What do you think, if you are interested I can clean up my local patch and submit a PR.
Best,

I think a PR is always welcome, it's also usually easier to reason about code than about words, at least for me.

@JonRiv Hey, could you see if PR #912 addresses this issue for you?

Can you check if that is still the case after the recent changes in unstable?


No MagickCore-config in $PATH on Debian Jessie

Hey guys. This bindings won't build anymore on Debian Jessie since there is no MagickCore-config in $PATH:
% apt-file search MagickCore-config
libmagickcore-6.q16-dev: /usr/lib/x86_64-linux-gnu/ImageMagick-6.8.9/bin-Q16/MagickCore-config

Any ideas how should we fix this?

You can use pkg-config instead:
pkg-config --cflags MagickCore
pkg-config --libs MagickCore

This bindings also doesn't compile anymore for me (i'm the original author of this "thing"). It was very "drafty" (from draft) because it was the first time i was trying to do something with C.
I found easier to make a new one from scratch rather than to fix this one:
https://github.com/fccm2/mgk-gen
(there is the minimum, something like 10 effects (filters), some draw primitives, and layers (called composite in im)).
there is now also a bindings for graphics-magick, the historical fork of image-magick:
https://github.com/fccm2/ocaml-graphicsmagick
i thought at the beginning that the binding don't compile anymore because IM's api changed, but it's probably because my first binding was quite lame.
I also tried to make another one:
https://github.com/fccm2/mgk-cmd
that tries to provide an interface similar than a binding, but which is only a "printf-interface" because it produces a command line for the command line tool of IM (and with some minimal compatibility with GM).
The 2 new ffi bindings are probably not better than the first one, but at least it compiles on my computer. (dunno if it will also do on someone else's computer too).


BareMetal - discovery url

I have problem with a baremetal CoreOS to connect to discovery url.
here is my etcd2 configuration :
etcd2:
    name: tdeheurles-bm-coreos
    listen-peer-urls: http://<IP_ADDRESS>:2380
    listen-client-urls: http://<IP_ADDRESS>:2379
    initial-advertise-peer-urls: http://<IP_ADDRESS>:2380
    advertise-client-urls: http://<IP_ADDRESS>:2379
    discovery: https://discovery.etcd.io/b9371c90aa85773de23c3c3eb52a60b9

What I don't understand is that this configuration work with a vagrant VM.
I can see the VM registred at the discovery url.
PS: this discovery url is out of date

Can you please follow the doc https://github.com/coreos/etcd/blob/master/Documentation/reporting_bugs.md and provide more information? Then we have a better understanding of your issue and help you with your problem.

I've got a response here


Improve reliability of facts.hardware.NetworkDevices

I ran into a few issues with NetworkDevices not properly parsing command output.
This PR fixes those issues, and adds a couple of large test cases to match.

Fixed the mypy issue, not sure why the unit test failed though :/


NotImplementedError for custom identifers

Expected behaviour:
No error occurs.
Actual behaviour:
An error occurs (see log)
Steps to reproduce:

Step 1: Execute a task with a series identified by an id

Config:
series:
  default:
  - <NAME_1>
  - <NAME_2>
  settings:
    default:
      identified_by: id
      id_regexp: (?:v|Vol ?)(\d+)
      tracking: backfill

Log:
<DATE> CRITICAL task          <IDENTIFIER>     BUG: Unhandled error in plugin series:
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/flexget/task.py", line 483, in __run_plugin
    return method(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/event.py", line 23, in __call__
    return self.func(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/filter/series.py", line 1513, in on_task_filter
    self.process_series(task, series_entries, series_config)
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/filter/series.py", line 1621, in process_series
    if entity < entity.series.begin:
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/filter/series.py", line 508, in __lt__
    raise NotImplementedError
NotImplementedError

Additional information:

Flexget Version: 2.10.31 (error occured before that, too)
Python Version: 2.7.12
Installation method: easy_install/setuptools
OS and version: Ubuntu 16.04

This shouldn't happen. The reason is that you set series begin to a type other than ID. Can you confirm that?

I actually haven't set begin for this task - apart from inputs, this is the whole configuration.
I do, however, have another, similar task (with identifier "sequence"), where begin is set to "01". That task works as supposed.

I spent some more time looking at this, and apparently commit f1045239209 added the error. Before, the method simply returned NotImplemented, but since then it raises an error.
Is my understanding correct? Would it be possible to return to the old implementation?

The old implementation does not make sense.

Actually, it does @cvium. I looked into and that's the correct way to return an Unhandled scenario with a custom comparison: https://dwieeb.com/2015/06/02/usages-of-notimplemented-and-notimplementederror-in-python/
I'll revert back to it. That being said those situations should not occur in the first place. I'll add some error logs to these scenarios that will enable further investigation.

I see. Still sort of applies in that it shouldn't happen.

I added some logs, pull or wait for next release and run again (any log level would be fine, the comparison errors should not log in error level)

Thanks for your work, @liiight !
I installed the latest version, and my log shows a very long list of
ERROR series <task> cannot compare id with sequence
ERROR series <task> cannot compare sequence with id
ERROR series <task> cannot compare id with sequence
ERROR series <task> cannot compare sequence with id

until finally
CRITICAL task <task> BUG: Unhandled error in plugin series: maximum recursion depth exceeded while getting the str of an object

I'm sorry, but that doesn't look too good...

This shouldn't happen. Something is off with your db. Did you change series identifiers after they were created? Paste full debug log please.
About the second issue, it's probably some mistake on my part, hopefully the log will let me see it

I don't think that I changed series identifiers, but it might have happened some months ago.
Debug log (the relevant parts, and anonymised) is at https://pastebin.com/ECwLkxfU . If this is not enough, just tell me.
Also, the full crash log is at https://pastebin.com/aiWD7VAc
And again, thank you!

Wow, wtf is going on there...

Please paste me the output of series show for that series

I had a look at series show and I think I know now what happened:
I have two tasks defined, one with the id identifier above, and a similar one with a sequence identifier, for different release types. Unfortunately, the same series is configured in both - I think I hoped that Flexget would track them separately, because they're in different tasks. But according to series show, there's both versions in the same db, one with the id, and others with sequence. And in that case, a comparison would run wild.
So, thank you very much, @liiight, for your trouble!

i imagined its a conflict between identifiers, need to think on how to protect against such a thing though

I had another look at Flexget's database to clean the mess up, and found that I had several such misconfigured series, with both sequence and id identifiers. However, this seems to have worked for some time, as Flexget had downloaded entries for both identifers - even if they were similar like "2" and "02".
For now, I have deleted all shared series, and also removed the entries from the database, and now it seems to work again. So thanks again for your help!


Please support the function of sending updates

At present, the web version of WhatsApp can send account updates, text, images, or videos. Please support it

At present, the web version of WhatsApp can send account updates, text, images, or videos. Please support it

sending status/stories? it can already be done, you just need to change the jid to status@broadcast and add statusJidList<EMAIL_ADDRESS>to the options
conn.sendMessage('status@broadcast', { text: 'Hello World' }, { statusJidList: [conn.user.id] })

after the WhatsApp application is updated
function of:
sock.sendMessage('status@broadcast', { text: 'I am here' }, { statusJidList: [sock.user.id] })
can not be used


Add Compliance check: githubOrgMFA

How the Check Works
Provide a clear definition
Pending Tasks

[ ] 1. Define a Good Implementation

[ ] Brainstorm how to implement this check.
[ ] Achieve an agreement on the implementation details before starting to work on this.

[ ] 2. Update Check Record

[ ] Update the compliance_checks row with the following fields: how_to_url, implementation_status, implementation_type and implementation_details_reference
[ ] Check the migration scripts using npm run db:migrate and npm run db:rollback
[ ] Update the database schema by running npm run db:generate-schema

[ ] 3. Implement the Business Logic

[ ] Add the alert row in the compliance_checks_alerts table.
[ ] Add the task row in the compliance_checks_tasks table.
[ ] Add the result row in the compliance_checks_results table.

[ ] 4. Ensure It Works as Expected

[ ] Add new test cases for this check.
[ ] Verify that all tests are passing.
[ ] Run the command check run --name {check_code_name} and verify the changes in the database. Update the seed script if needed (npm run db:seed)

After some research seems like two_factor_requirement_enabled is the best field that can provide us the information from the table github_organizations that we populate from the GitHub API.

When the value of two_factor_requirement_enabled is true, the organization requires all members, billing managers, and outside collaborators to enable two-factor authentication. GH API documentation

Logic
Main logic:

If the value is true then is considered as passed
If the value is false then is considered as failed
If the value is NULL then is considered as unknown

Edge cases (multiple GitHub organizations):

With a single failed organization, the global check is considered failed

With a single unkown organization (without failed ones), the global check is considered unknown

Only if all the orgs have passed the global consideration is passed

Both PRs are landed, so this check is considered completed at this stage :partying_face:


Overriding the Identity Error Messages

AFAIK, currently the error messages in identity library are not override-able. For example, we cannot override 'Name {0} is already taken.'. This will cause a big headache for multi-lingual application and also in some other cases,  http://forums.asp.net/t/1957899.aspx?How+to+override+validation+for+UserName

This is getting a second look. See https://aspnetidentity.codeplex.com/workitem/1947 and http://aspnetidentity.codeplex.com/discussions/638351.

I think that the word "Override" means "change the resources.resx file's value" not point to another neutral culture.


[IDEA] - Console.log colors in the console

Molarien colorets per cada console log

https://stackoverflow.com/questions/9781218/how-to-change-node-jss-console-font-color?page=1&tab=votes#tab-top

Added in a6cd33b77187cfc6c2ba9e65551dfae3ab760f9d


Implement Dynamo DAX on Edge

Due to the time based in-memory caching that was recently added to our Edge API, we are putting this on hold to gather more information about what benefits we would still gain from using DAX.
We also want to discuss the possibility of using redis here instead of DAX.

No longer relevant because extensions can be used with dynamodb streams instead.


Failed to load ensemble model cause it depends on model 'model_name' which has no loaded version

Hello, I'm trying to run model analyzer profile for my ensemble
I'm using 23.02 release, and running it with following command:
docker run -it --gpus all \
      -v /var/run/docker.sock:/var/run/docker.sock \
      -v /home/triton/dali_back/model_repository_dali_back:/home/triton/dali_back/model_repository_dali_back \
      -v /home/triton/output_dali_mod_analyzer:/home/triton/output_dali_mod_analyzer \
      --net=host model-analyzer

Here is model repository(model_repository_dali_back) tree:
model_repository_dali_back/
├── dali
│   ├── 1
│   │   └── model.dali
│   └── config.pbtxt
├── ensemble_dali_vit
│   ├── 1
│   └── config.pbtxt
└── vit_base_trt
    ├── 1
    │   └── model.plan
    └── config.pbtxt

And running model_analyzer with:
model-analyzer profile \
  --model-repository=/home/triton/dali_back/model_repository_dali_back \
  --profile-models=ensemble_dali_vit --triton-launch-mode=docker \
  --triton-http-endpoint=localhost:8000 --triton-grpc-endpoint=localhost:8003 --triton-metrics-url=localhost:8002 \
  --output-model-repository-path=/home/triton/output_dali_mod_analyzer/dali \
  --override-output-model-repository \
  --run-config-search-mode quick

And getting kind of error
[Model Analyzer] Model ensemble_dali_vit_config_0 load failed: [StatusCode.INVALID_ARGUMENT] load failed for model 'ensemble_dali_vit_config_0': ensemble 'ensemble_dali_vit_config_0' depends on 'vit_base_trt_config_0' which has no loaded version. Model 'vit_base_trt_config_0' loading failed with error: version 1 is at UNAVAILABLE state: Internal: unable to load plan file to auto complete config: /home/triton/output_dali_mod_analyzer/dali/vit_base_trt_config_0/1/model.plan;

Full Log here
@/opt/triton-model-analyzer# model-analyzer profile   --model-repository=/home/triton/dali_back/model_repository_dali_back   --profile-models=ensemble_dali_vit --triton-launch-mode=docker --triton-http-endpoint=localhost:8000 --triton-grpc-endpoint=localhost:8003 --triton-metrics-url=localhost:8002   --output-model-repository-path=/home/triton/output_dali_mod_analyzer/dali --override-output-model-repository --run-config-search-mode quick
[Model Analyzer] Initializing GPUDevice handles
[Model Analyzer] Using GPU 0 Tesla T4 with UUID GPU-e459443f-0c95-c1fa-5cca-a91d5818456c
[Model Analyzer] WARNING: Overriding the output model repo path "/home/triton/output_dali_mod_analyzer/dali"
[Model Analyzer] Starting a Triton Server using docker
[Model Analyzer] Pulling docker image nvcr.io/nvidia/tritonserver:23.02-py3
[Model Analyzer] No checkpoint file found, starting a fresh run.
[Model Analyzer] Profiling server only metrics...
[Model Analyzer]
[Model Analyzer] Starting quick mode search to find optimal configs
[Model Analyzer]
[Model Analyzer]
[Model Analyzer] Creating model config: dali_config_default
[Model Analyzer]
[Model Analyzer]
[Model Analyzer] Creating model config: vit_base_trt_config_default
[Model Analyzer]
[Model Analyzer] Model ensemble_dali_vit_config_default load failed: [StatusCode.INVALID_ARGUMENT] load failed for model 'ensemble_dali_vit_config_default': ensemble 'ensemble_dali_vit_config_default' depends on 'vit_base_trt_config_default' which has no loaded version. Model 'vit_base_trt_config_default' loading failed with error: version 1 is at UNAVAILABLE state: Internal: unable to load plan file to auto complete config: /home/triton/output_dali_mod_analyzer/dali/vit_base_trt_config_default/1/model.plan;

[Model Analyzer]
[Model Analyzer] Creating model config: dali_config_0
[Model Analyzer]   Setting instance_group to [{'count': 1, 'kind': 'KIND_GPU'}]
[Model Analyzer]   Setting max_batch_size to 1
[Model Analyzer]   Enabling dynamic_batching
[Model Analyzer]
[Model Analyzer]
[Model Analyzer] Creating model config: vit_base_trt_config_0
[Model Analyzer]   Setting instance_group to [{'count': 1, 'kind': 'KIND_GPU'}]
[Model Analyzer]   Setting max_batch_size to 1
[Model Analyzer]   Enabling dynamic_batching
[Model Analyzer]
	[Model Analyzer] Model ensemble_dali_vit_config_0 load failed: [StatusCode.INVALID_ARGUMENT] load failed for model 'ensemble_dali_vit_config_0': ensemble 'ensemble_dali_vit_config_0' depends on 'vit_base_trt_config_0' which has no loaded version. Model 'vit_base_trt_config_0' loading failed with error: version 1 is at UNAVAILABLE state: Internal: unable to load plan file to auto complete config: /home/triton/output_dali_mod_analyzer/dali/vit_base_trt_config_0/1/model.plan;

[Model Analyzer]
[Model Analyzer] Done with quick mode search. Gathering concurrency sweep measurements for reports
[Model Analyzer]
[Model Analyzer]
[Model Analyzer] Done gathering concurrency sweep measurements for reports
[Model Analyzer]
[Model Analyzer] Saved checkpoint to /opt/triton-model-analyzer/checkpoints/0.ckpt
[Model Analyzer] Profile complete. Profiled 0 configurations for models: []
[Model Analyzer]
[Model Analyzer] WARNING: Inference output field "perf_throughput", has no data
[Model Analyzer] WARNING: Inference output field "perf_latency_p99", has no data
[Model Analyzer] WARNING: GPU output field "gpu_used_memory", has no data
[Model Analyzer] WARNING: GPU output field "gpu_utilization", has no data
[Model Analyzer] WARNING: GPU output field "gpu_power_usage", has no data
[Model Analyzer] WARNING: Server output field "gpu_used_memory", has no data
[Model Analyzer] WARNING: Server output field "gpu_utilization", has no data
[Model Analyzer] WARNING: Server output field "gpu_power_usage", has no data
[Model Analyzer] WARNING: Requested top 3 configs, but none satisfied constraints. Showing available constraint failing configs for this model.
[Model Analyzer] WARNING: Requested top 3 configs, but found only 0. Showing all available configs for this model.
[Model Analyzer] WARNING: Requested top 3 configs, but none satisfied constraints. Showing available constraint failing configs for this model.
[Model Analyzer] WARNING: Requested top 3 configs, but found only 0. Showing all available configs for this model.
[Model Analyzer] WARNING: Requested top 3 configs, but none satisfied constraints. Showing available constraint failing configs for this model.
[Model Analyzer] WARNING: Requested top 3 configs, but found only 0. Showing all available configs for this model.
[Model Analyzer] WARNING: No data found for model ensemble_dali_vit, skipping export summary.
[Model Analyzer] WARNING: Requested top 3 configs, but none satisfied constraints. Showing available constraint failing configs for this model.
[Model Analyzer] WARNING: Requested top 3 configs, but found only 0. Showing all available configs for this model.
[Model Analyzer] WARNING: Requested top 3 configs, but none satisfied constraints. Showing available constraint failing configs for this model.
[Model Analyzer] WARNING: Requested top 3 configs, but found only 0. Showing all available configs for this model.
[Model Analyzer] To generate detailed reports for the 0 best configurations, run `model-analyzer report --report-model-configs  --export-path /opt/triton-model-analyzer`

Is it possible to run model_analyzer for this ensemble?
As I understand  ensemble support is available in 23.02 release)

Yes, ensemble support is now available. Don't have enough info to determine what has gone wrong. Can you rerun and with the Triton server output logging enabled --triton-output-path TRITON_OUTPUT_PATH?

Yes, the logging is telling you what has gone wrong. The plan file you are running for the vrt_base_trt composing model needs to be regenerated. This is why the ensemble failed to load.

@nv-braf Thank you!
After converting model with newer version of TRT, model-analyzer works fine!
But it gives warning about GPU metrics, I'm running container with --gpus 'device=1' so I'm sure that only one device
[Model Analyzer] WARNING: No GPU metric corresponding to tag 'gpu_used_memory' found in the model's measurement. Possibly comparing measurements across devices.
[Model Analyzer] WARNING: No GPU metric corresponding to tag 'gpu_utilization' found in the model's measurement. Possibly comparing measurements across devices.

And doesn't report any GPU and CPU metrics like "GPU Utilization", "GPU Memory Usage" and etc.
Is there any way to improve this? may be I should add something to my config file?
Now it looks:
model_repository: '/home/proevgenii1/triton/dali_back_new/model_repository_dali_back'
output_model_repository_path: '/home/proevgenii1/triton/output_dali_mod_analyzer/res/'
profile_models: 'ensemble_dali_vit'
override_output_model_repository: true

triton_launch_mode: 'docker'
client_protocol: 'http'
triton_http_endpoint: 'localhost:8000'
triton_grpc_endpoint: 'localhost:8003'
triton_metrics_url: 'localhost:8002'

run_config_search_mode: 'quick'
run_config_search_max_instance_count: 4
run_config_search_max_concurrency: 16
run_config_search_max_model_batch_size: 64
num_configs_per_model: 10
triton_output_path: '/home/proevgenii1/triton/triton_output.txt'
perf_analyzer_flags:
    percentile: 95
    input-data: '/home/proevgenii1/triton/input_triton.json'

GPU metrics should be returned only for the entire ensemble, but not the individual models. MA gets these metrics from PA. @matthewkotila what are the next steps in root causing why PA didn't return GPU metrics?

@proevgenii

But it gives warning about GPU metrics, I'm running container with --gpus 'device=1' so I'm sure that only one device
That can mean that it is not running on a GPU. Are you referring to running the tritonserver:23.02-py3-sdk container? Can you run nvidia-smi from inside that container to confirm it can see the GPU?

@proevgenii Nevermind, I was able to reproduce locally. The issue is that you are specifying a bad format for triton_metrics_url. If you changed it to triton_metrics_url: 'localhost:8002/metrics' it would work.
Note that there is almost no reason for you to specify those options unless you are running in remote mode. The defaults will work just fine unless those ports are already in use. I would recommend you remove the triton_*_endpoint and triton_metrics_url options from your config.
We will look into detecting this case and giving a better warning/error.


buildah inspect man page is inaccurate

The 'buildah inspect' man page says the command can be use used like:
buildah inspect ctrid
buildah inspect imageid
However the second variant fails saying the provided argument is not a container.  In order to inspect an image you need to do "bulidah inspect --type=image ".
The documentation needs to be touched up and/or the inspect command needs to try and inspect as an image if the container inspect fails and a type is not specified.

Closed via #277


Missing Step in Setup readme.

Hello!
I was trying to add this to my profile, and under the Vercel section, I think it should be said you need to fork your repo. I needed to find a video of someone else adding it to their profile to figure out what I was doing wrong.

Yeah I think that makes a lot more sense! When I tried to add this to my repo, I was under the impression I was supposed to fork my profile's readme.


fix: mask systemd-timesyncd in Mariner

What type of PR is this?
/kind fix

What this PR does / why we need it:
We should mask timesyncd, chronyd fills the role of time synchronization for the AzureLinux container host.
Which issue(s) this PR fixes:

Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[x] tested upgrade from previous version

Special notes for your reviewer:
pipeline run
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 61.643%

Totals

Change from base Build<PHONE_NUMBER>:
0%

Covered Lines:
2544

Relevant Lines:
4127

💛  - Coveralls


USDT pools have disappeared from pools list

Overview
USDT on ETH and USDT on BSC pools are no longer showing up in the available TC LP pools list.  There are currently users with active positions in these pools and they are unable to engage with them from the 'active pools' section either.
References and additional details
Jam of current list with no USDT pools available besides USDT on AVAX.
https://jam.dev/c/422ff3aa-47f7-42cc-a229-17db17bfee82
Acceptance Criteria
Our TC pools list reflects all available pools of chains/assets supported in ShapeShift.  Users are able to see and interact with current/new pool positions.
Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
No response

midgard syncing which should resolve this

Dev is synced, cut over all web environments to dev until snapshots finish and prod is back in order. Will cut back app and private at that time.


India - added and deleted some subs

/r/urdu - is generic
/r/indianpics - is dead and used for redirection
/r/ecigindia - drugs/vaping/illegal subs discontinued

Looks good.


Update migration-v4 docs for wrong import path

[X] I have followed (at least) the PR section of the contributing guide.

Reading up on what the v4 -> v5 migration is going to entail for us and noticed this typo. The change is from import { styled } from '@mui/material/styles'; to import { styled } from '@mui/styles'; the extra styled is incorrect.

@mnajdova is there anything else I need to do before this is good to go?


Global DNS UI: When you edit a DNS Provider or Entry page is a create page

Version: 2.2 master (1/22/19) HA setup
What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Create a Global DNS Provider and Global DNS Entry. Edit both.

Result:
The "Create" button for each should be a "Save" button just like when we update other areas in the UI such as clusters. You'll also see this looks like a normal "Create" page based on the title of the page!

I believe this was already fixed in another issue so I am moving to to-test status

Version: Master 2.2 (3/14/19) 6875782b6131
This was already fixed in a related issue, but I checked again. GDNS Provider/Entry have correct titles when editing and show the Save button now, not Create. This is fixed.


Should be able to change color and timeout for "enable_presenter_mode" argument in the "Library"

Is your feature request related to a problem? Please describe.
Yes, the default color for "enable_presenter_mode" argument in the "Browser Library" is blue and its getting mixed with html elements in our UI which are of same color
Describe the solution you'd like
I would like to be able to change the default color, change the border style as well (solid border for example) and time out for "enable_presenter_mode" argument in the "Browser Library"
Describe alternatives you've considered
I am using "Highlight Elements" keyword instead
As you could see in the screenshot, Yes button is in blue and cant see it highlighted when used "enable_presenter_mode"

@allcontributors please add @gvr-kumar for ideas.

Decided at same go support all the configuration options which are currently supported by the Highlight Elements keyword: duration, width, style and color. duration will also affect how long keyword will sleep after highlight.

Thankyou so much Aaltat


Mode HTMLEbmedded Doesn't Work

Trying to print a HTML Embedded file doesn't work and returns an error Uncaught TypeError: CodeMirror.multiplexingMode is not a function. To resolve include the addon/mode/multiplex library.

Thank you for your request.
However, I do not understand the relationship between "Mode HTMLEbmedde" and "multiplexingMode".
Please let me know the code you want to print.

You just need to include the addon/multiplex library.

Perhaps it will be the desired behavior.
However, I want to know the behavior before the modification and the behavior after the modification before adding the code.
I'd like to test with the code below, so please tell me the extended function name that will be mode == 'htmlembedded'. It seems that it can not be specified with standard VSCode.
<%
function hello(who) {
	return "Hello " + who;
}
%>
This is an example of EJS (embedded javascript)

The program says <%= hello("world") %>.

http://codemirror.net/mode/htmlembedded/

I'm not entirely certain of what you are asking me for so I'm just going to guess that you are wanting to know what file/mime type I'm seeing this behavior with. I'm editing Classic ASP with the file extension of .asp: It looks like all of the following extensions are detected as HTMLEmbedded by your code:
.ejs
.erb
.aspx
.asp
.jsp

I could not reproduce it.

I can help you solve if you can show actual code.

Sure, I've attached a sample.
nav_menu_online.asp.zip

This is what it looks like if I manually add the