Fix IntPtr overload code gen when native pointers are ref/out parameters

The new generated code for the method is below. Note the care taken to handle out and ref parameters.
public static unsafe int StrongNameGetPublicKey(string szKeyContainer, System.IntPtr pbKeyBlob, int cbKeyBlob, out System.IntPtr ppbPublicKeyBlob, out int pcbPublicKeyBlob)
{
    byte* pbKeyBlobLocal = (byte*)pbKeyBlob.ToPointer();
    byte* ppbPublicKeyBlobLocal;
    int result = StrongNameGetPublicKey(szKeyContainer, pbKeyBlobLocal, cbKeyBlob, out ppbPublicKeyBlobLocal, out pcbPublicKeyBlob);
    ppbPublicKeyBlob = new System.IntPtr(ppbPublicKeyBlobLocal);
    return result;
}

Remind me again why the code gen generates method bodies rather than just generating another [DllImport] overload?

This was reviewed by virtue of @jmelosegui's review of #121, which included this change. I meant to merge this PR first to keep #121 simpler but forgot.


couldn't find msrvtt-32frm.zip

when I run <bash scripts/download_models.sh>,it has this error:
--2022-05-23 21:26:20--  https://datarelease.blob.core.windows.net/swinbert/models/msrvtt-32frm.zip
Resolving datarelease.blob.core.windows.net (datarelease.blob.core.windows.net)... <IP_ADDRESS>
Connecting to datarelease.blob.core.windows.net (datarelease.blob.core.windows.net)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 404 The specified blob does not exist.
2022-05-23 21:26:21 ERROR 404: The specified blob does not exist..
Archive: ../SwinBERT/models/32frm/msrvtt-32frm.zip
End-of-central-directory signature not found.  Either this file is not
a zipfile, or it constitutes one disk of a multi-part archive.  In the
latter case the central directory and zipfile comment will be found on
the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of ../SwinBERT/models/32frm/msrvtt-32frm.zip or
../SwinBERT/models/32frm/msrvtt-32frm.zip.zip, and cannot find ..SwinBERT/models/32frm/msrvtt- 32frm.zip.ZIP, period.

Thanks for pointing it out. We don't have msrvtt-32frm.zip. Will remove it from the script.


Further arguing with mitaka

Deal with missing packages in venvs, provider networks, ironic netblocks, and adding an ironic bridge. What times we live in!

This test build resulted in a deployment with a non-functional keystone. I'm not 100% sure why right now. http://molokai.stillhq.com/lab/logs/20170323-1521-mitaka-7b2698fb647438e47993a3c24692fd58a9ee31e6/

Log for second test run is at http://molokai.stillhq.com/lab/logs/20170323-2239-mitaka-40217e1b843646bc5b90fe98d36eadcd7004e153/

I am not sure why keystone wasn't working post install. Its a couple of hours later when I get to debugging and its now working fine. A possible timing issue?

This now works as much as any other install has, so I'm going to merge before doing anything more.


Set MSYS_NO_PATHCONV through python script

Fixes #222.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-3 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 44.563%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pynxtools/nexus/nexus.py
0
3
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
4905

Relevant Lines:
11007

💛  - Coveralls


type_mappings documentation

The example is based on https://github.com/omniscale/imposm3/blob/master/test/single_table_mapping.json

@olt ,
Could you have a look at this?


Creating Models and Schemas

Creating User, Balance, Budget, Income, Expense models. Schema and typeDefs


Surface Query Run Time in VS Code UI

it will be nice to see how long a Query has took

Agree this would be useful to have properly surfaced; for now you can see it if you go to Terminal > New Terminal, then select the Output tab, and then "Malloy" from the dropdown on the right.

that's fantastic, thank you


Full copy of Feature Service not working

I am attempting to copy Feature Service files to another ArcGIS account using Full Copy; however, the copy status gets stuck on "Creating Service," never actually copying. Is there anything specific I should be looking for to troubleshoot this? Thanks for your time.

There will be some additional messaging in the developer console. Can you share what you see there?

@sptm1107 does your service have related tables/layers within it?

@slibby This should be closed due to inactivity.

In case it helps anyone in the future who finds this issue: I encountered the same problem, and in my case it was because the target user account did not have permission to create a Feature Service.


Go function with amplify function build is wrong

Which Category is your question related to?
function
Amplify CLI Version
4.27.3
What AWS Services are you utilizing?
Lambda
Provide additional details e.g. code snippets
I have a two Lambda function in Go. I am developing on Windows PC.  When I use
amplify function build it create main.exe file in /bin folder of the Lambda function and /dist folder contains latest-build.zip with main.exe in it.
After this, when I use amplify push --yes command, it push my function to the cloud without any issue.
But when I try to run the lambda function from the console, it always ends up with the following error.
{   "errorMessage": "fork/exec /var/task/main: no such file or directory",   "errorType": "PathError" }
I assume when amplify function build command runs or amplify push happens, it should build the Go lambda function with GOOS=linux.
Someone, please help to resolve this.
Thanks.

Thanks for reporting this. I am marking this as bug.

FYI:
amplify function build is the issue. Local invoke is not an issue.  It is already creating windows .exe for when we build function.
It should build with GOOS=linux on windows PC, so on lambda environment it can work with amplify push --yes.


proposal: cmd/go: breakout testmain generation functionality into it's own tool

When go test is ran there is logic in go/src/cmd/go/internal/load/test.go that will scan a  package for it's given tests and generate the code for the entry point a test binary.  The go tool then compiles and executes that binary.
External build tools like Bazel (https://bazel.build/) and Buck (https://buckbuild.com/) bypass the go tool and execute the compiler and linkers directly to get control over the output archives, etc to use for distributed caching and to unify building across multiple languages.  To deal with tests both Buck[1] and Bazel[2] have done a copy and paste of the testmain generation code with some modifications and embedded them.
Being an internal package, obviously it is not a stable API.  Every Go version there can be minor changes to how tests are discovered or how the testmain code generation is code.  These have to be painstakingly backported into Bazel and Buck to keep working while also maintaining comparability with older Go versions.
Since the generated testmain code is linked with the testing library found in the stdlib it would make sense that there is a clear way to generate testmain code that is tied to the version of Go toolchain.
I am proposing abstracting out the test discovery and testmain generation code into it's own external tool (e.g  go tool testmaingen) that can be called from external tools.  The go tool could call either a shared library or the testmaingen tool directly.
This would be similar to how cgo code generation is done.
[1] https://github.com/facebook/buck/blob/master/src/com/facebook/buck/features/go/testmaingen.go.in
[2] https://github.com/bazelbuild/rules_go/blob/master/go/tools/builders/generate_test_main.go

cc: @bcmills @rsc @ianlancetaylor

CC @jayconrod @matloob @ianthehat

Somewhat related: https://github.com/golang/go/issues/29430

You can kind of already do this. If you run go list -test -f '{{if eq .Name "main"}}{{.GoFiles}}{{end}}' *.go, go list writes the testmain source to the build cache, and you can read it out from there. Kludgy though.
Something that complicates this change is that we generate the testmain source when loading packages, not during execution as you'd expect. The generated testmain is listed in .GoFiles, not .CompiledGoFiles because of this. If we were going to generate the testmain in a subprocess, we should move the code generation into a proper action first so that we can benefit from caching. Actually, we should do this anyway so go list can do less work.
Even if we did this, I'd be very surprised if the generated file could be used as-is. Bazel needs to do some other stuff in main before running tests, like changing directories, sharding, filtering, and coverage post-processing. Every build system is going to need some custom code, and I'm not confident we can factor all that out into a separate file.

That is pretty interesting.  I didn't know it was stored in the cache.  Digging through the cache seems like it could be more fragile than what we are dealing with here.

I'd be very surprised if the generated file could be used as-is. Bazel needs to do some other stuff in main before running tests, like changing directories, sharding, filtering, and coverage **post-processing.

In Buck we change directories , do test discovery, do sharding through a "test runner" which is just a wrapper around the compiled testmain which seems like a much cleaner solution with better abstractions, but that isn't really the topic here.
Does Bazel/Blaze do this for other languages as well like Python or C++?

Every build system is going to need some custom code, and I'm not confident we can factor all that out into a separate file.

There is very little that Buck does here intentionally as it's already too hard to maintain the testmain generation code with just changes to the testing package
The core problem we have had w/ Buck is that the  testing package and the generated code that uses has changed with nearly every major Go release.  We have been able to hack around this by adding reflection, implementing an interface, etc  Even if some of the testmaingen code was available as an external library that could be imported it would help, but it would require a stable API between it and the testing package which may be less than ideal.
Here are some of the changes that has been required.

https://github.com/facebook/buck/commit/4271ef7dd33c2b5fd4abdd184793cbc41c1055bc#diff-d7a1345bc79619677ed4da43c65e7540
https://github.com/facebook/buck/commit/2b454ad05c3d5c08361d2b461b3a4bd952701851#diff-d7a1345bc79619677ed4da43c65e7540

In fact when we upgrade to new Go versions this is really the only consistent blocker we have had over the years.

I think @jayconrod's answer here is the right one. There shouldn't be a separate API for generating testmain.go by itself. Even knowing that testmain.go is a thing is too much detailed knowledge about tests. What we've exposed is how to ask 'tell me the exact way the test binary should get built' instead. That's a much more general query.

Per discussion with @golang/proposal-review, looks like this is resolved. Please comment if that’s not the case.

I guess then the problem is there is no spec on how other build tools should interact with the testing package.  It would be beneficial to have a spec on how that interaction would look so that other building tools could just build their own test main generation (or whatever).
The original problem is that the interaction between the the testing package and the tool is very much intertwined leading to a lot of breakage when you step slightly outside of it.


Added configuration for Notification demo Android

@kaa-leeroy test this fully please

The commits chain has already passed Jenkins as https://github.com/kaaproject/sample-apps/pull/322 before merge to this branch.


doc: Add rsync to install package

Purpose of pull request
emlinux-k510's sdk installer requires rsync command.
So, added rsync in apt command line example.
  INSTALL
/home/build/sdk-test/sysroots/aarch64-emlinux-linux/usr/include
/bin/sh: 1: rsync: not found
Makefile:1302: recipe for target 'headers_install' failed
make: *** [headers_install] Error 127
SDK has been successfully set up and is ready to be used.

Thanks!


Commands can sometimes fail to return on the next frame from the ActionFeatureLayer

This caused TestFeatureLayers and TestObservationActions to occasionally fail as the Point select would occasionally return 2 frames later instead of 1. Camera move commands are also returned 2 frames later.

I've seen this issue happen with camera move commands as well, including in the feature layer using pysc2 when step_mul is set to 1.

@TommyTran93 I see your issue was referenced in a PR which is already merged.
Was this solved?
If yes, can you please close this issue?


Note: suggest the name split if over 25 chars with no -

New packages only? Oh wait, that would be a requirement


Nextjs refactor: Normalize contextual H1 list view language when selecting filters

iirc H1 "popular" attribute for list results was intended post-feedback lists. Until implementing vicinity, location size and recency sorting as per streetlives/yourpeer.nyc-nextjs#164, Distinct H1 language would presumably only be linked to category sub-filters, becmoing 'All soup Kitchens, not 'All food locations' etc,

 Found an old Figma with this header styling, I think it looks better than the screenshot, let me know if it's not the right one https://www.figma.com/design/hE0RcUd4nK4NUe6D62OKR8/YourPeer-V1?node-id=734-15509&t=OugOJL5cdkfgRTSL-4

This is the right link https://www.figma.com/design/hE0RcUd4nK4NUe6D62OKR8/YourPeer-V1?node-id=591-163&t=2BvyUh7gAhwScjGc-1


False positive: hardcoded SSL/TLS version (DS440010) in custom enum

Using version: v0.4.131

Create test.cpp with the following content

enum ErrorCode { EC_ACME_BADHANDSHAKE };

Run analyze

> devskim.exe analyze test.cpp
file:test.cpp
        region:1,26,1,37 - DS440010 [Important] - OpenSSL: Do not hardcode SSL/TLS versions within an application.

Issues found: 1 in 1 files
Files analyzed: 1
Files skipped: 0

Thanks for your report. Looks like a potential false positive to me.

It looks like this regex needs to be rewritten because it is catching ADH.*SHA
{
    "pattern": "(TLS_)?(AES|DH|DHE|ADH|CAMELLIA|EDH|EXP|DES|IDEA|RC4|NULL|GOST|EXP|ECDH|ECDHE|AECDH|PSK)[A-Z0-9\\-_]+-?(SHA|MD|GOST)[A-Z0-9\\-]*",
    "type": "regex",
    "scopes": [
        "code"
    ],
    "_comment": "OpenSSL cipher suite"
}


small typo

I'm 95% sure the word should be marker not maker

@swanderz : Thanks for your contribution! The author(s) have been notified to review your proposed change.


[NInputNumber] Strange loop when @update:value

TuSimple/naive-ui version (版本)
2.25.3
Vue version (Vue 版本)
3.2.37
Browser and its version (浏览器及其版本)
Tested on Chrome and Firefox
System and its version (系统及其版本)
Linux Fedora 36
Node version (Node 版本)
Reappearance link (重现链接)
https://codesandbox.io/s/gallant-bassi-d5cr5i?file=/src/Demo.vue
Reappearance steps (重现步骤)
Source:
https://codesandbox.io/s/gallant-bassi-d5cr5i?file=/src/Demo.vue
To reproduce:

Open link [ https://d5cr5i.csb.app/ ]
Open devtools (sometimes it doesnt work with dev tools not present)
Fast click to increase or decrease
Loop starts

Video:
https://i.imgur.com/6RU9Vto.mp4
Expected results (期望的结果)
Expected no loop behaviour
Actual results (实际的结果)
Loop starts when fast click
Remarks (补充说明)
Bug happens from version 2.25.3 to the latest.

I can't reproduce it but it may be relevant with
https://github.com/TuSimple/naive-ui/pull/3333

I managed to repod it, with a macro that presses 30 times (every 2ms).

I confirm it's not fixed, version 2.32.1

Confirmed: Fixed


SQLSTATE[HY010]: [Microsoft][ODBC Driver Manager] Function sequence error (again)

I am close to getting buffered queries working in the Drupal database abstraction layer, with performance gains of ~30% (that's huge) on a vanilla install.
But... I am seeing again the Function sequence error :(
It took me ages to isolate the issue this time into a sample script.
Download schema and sample script from this dropbox link:
https://www.dropbox.com/s/3nd8xdf32a7dmyz/sample_sqlsrv_may.zip?dl=0
For the error to go away, simply remove the transaction related statements:
$cnn->beginTransaction();
$cnn->commit();
So it looks like you cannot have a transaction nested in a loop while iterating on a client buffered result set.

Not sure if directly related, but I am seeing the function sequence error appear randomly on different usage scenarios.
All of them when trying to iterate over a client buffered result set.

Hi david-garcia-garcia can you explain some of the usage scenarios that are causing the sequence error. I am with the team at Microsoft that own's this driver. I might be able to help you out.

The case was too complex and would need investigation to really isolate the steps to reproduce. But it did share something similar with the sample script I provided in the same comment:

The error came and go intermitently (one request with error, the next one without, and so on...)
The error appeared after previous operations on the same connection dealt with failing transactions and rollbacks.

If you could get to reproduce the error with the provided script there's a chance that both of them are indeed the same bug.

I am seeing this error consistently on Drupal 8 - MSSQL integration (and is preventing any further development because it breaks man automated tests...).
Unluckily, the circumstances that lead to the error are so complex that I was unsuccessfull at isolating a test script. I even tried to PROXY all operations on Drupal's database abstraction layer to later reproduce every single operation done against the PDO driver without luck, the error simply won't reproduce.
If you are interested in helping solve this out to get D8 working on MSSQL Server, I can provide the exact steps to reproduce on a clean D8 install. You will need the -dev version of the SQL Server driver for Drupal - that I can only share privately at the moment.

I’d love to take a look at this.
Can you share the repro steps and the dev version driver?
Thanks
From: Davvid<EMAIL_ADDRESS>Sent: Monday, July 20, 2015 6:36 AM
To: Azure/msphpsql
Subject: Re: [msphpsql] SQLSTATE[HY010]: [Microsoft][ODBC Driver Manager] Function sequence error (again) (#49)
I am seeing this error consistently on Drupal 8 - MSSQL integration (and is preventing any further development because it breaks man automated tests...).
Unluckily, the circumstances that lead to the error are so complex that I was unsuccessfull at isolating a test script. I even tried to PROXY all operations on Drupal's database abstraction layer to later reproduce every single operation done against the PDO driver without luck, the error simply won't reproduce.
If you are interested in helping solve this out to get D8 working on MSSQL Server, I can provide the exact steps to reproduce on a clean D8 install. You will need the -dev version of the SQL Server driver for Drupal - that I can only share privately at the moment.
—
Reply to this email directly or view it on GitHubhttps://github.com/Azure/msphpsql/issues/49#issuecomment-122886221.

Steps to reproduce:

Download a fresh copy of D8: git clone --branch 8.0.x http://git.drupal.org/project/drupal.git
Setup the site on IIS (see attached phpinfo() for enabled extensions and overal configuration)
Unzip the SQL Server driver to the root of drupal, this will show you SQL Server as an available target during the install process.

Set a breakpoint in line 183 of \core\modules\dblog\src\Controller\DbLogController.php (this is the loop where the error happens)
Run Tests. When the breakpoint is hit use F10 (Visual Studio + PHP Tools to debug) to step through about 4 times, the exception is thrown.

I found out that completely removing transaction management from the Drupal driver prevents the error from happening.
https://drive.google.com/file/d/0B2bjxiZ1Bb4BZmxNWnd1YmFpdDg/view?usp=sharing
(zip with sqlsrv driver, phpinfo() dump and screenshot of the exception)
Good luck. At least the exception is consistently reproducible.
Greetings

Any luck with this? D8 is to be released before the end of the year, it would be nice to have it working on MSSQL shortly before that happens.

I'm looking into it.  I'll,keep you posted on that.  Give me a week, since I was out of town.
Thanks
Sent from my iPad
On Aug 4, 2015, at 11:03 PM, Davvid <notifications<EMAIL_ADDRESS>wrote:
Any luck with this? D8 is to be released before the end of the year, it would be nice to have it working on MSSQL shortly before that happens.

Reply to this email directly or view it on GitHubhttps://github.com/Azure/msphpsql/issues/49#issuecomment-127882620.

Do not use the latest D8 HEAD, download something from the date of my original post. Recent changes in D8 have broken the MSSQL driver for Drupal.

Updated driver to fix compatiblity with latest head:
https://www.dropbox.com/s/j98z4fd8jh7wr0q/drivers2.rar?dl=0
Verified that the issue is still reproducible under the same conditions.

I’ll take a look at this tonight, in my Mac
I’ll keep you posted
From: Davvid<EMAIL_ADDRESS>Sent: Thursday, August 6, 2015 10:57 AM
To: Azure/msphpsql<EMAIL_ADDRESS>Cc: Luiz Fernando Santos<EMAIL_ADDRESS>Subject: Re: [msphpsql] SQLSTATE[HY010]: [Microsoft][ODBC Driver Manager] Function sequence error (again) (#49)
Updated driver to fix compatiblity with latest head:
https://www.dropbox.com/s/j98z4fd8jh7wr0q/drivers2.rar?dl=0
Verified that the issue is still reproducible under the same conditions.
—
Reply to this email directly or view it on GitHubhttps://github.com/Azure/msphpsql/issues/49#issuecomment-128458978.

When I’m trying to create the SQL Server database in Drupal 8, I’m getting the following error in the PHP56 error_log file:
[11-Aug-2015 22:04:27 America/Los_Angeles] PHP Fatal error:  Call to undefined function Drupal\Driver\Database\sqlsrv\wincache_ucache_get() in C:\inetpub\wwwroot\drupal\drivers\lib\Drupal\Driver\Database\sqlsrv\Connection.php on line 607
Any clue?
From: Davvid<EMAIL_ADDRESS>Sent: Thursday, August 6, 2015 10:57 AM
To: Azure/msphpsql<EMAIL_ADDRESS>Cc: Luiz Fernando Santos<EMAIL_ADDRESS>Subject: Re: [msphpsql] SQLSTATE[HY010]: [Microsoft][ODBC Driver Manager] Function sequence error (again) (#49)
Updated driver to fix compatiblity with latest head:
https://www.dropbox.com/s/j98z4fd8jh7wr0q/drivers2.rar?dl=0
Verified that the issue is still reproducible under the same conditions.
—
Reply to this email directly or view it on GitHubhttps://github.com/Azure/msphpsql/issues/49#issuecomment-128458978.

Hey Davvid,
I used the very first connector you sent me (used both versions in total) and the error below happens:
[13-Aug-2015 20:36:28 America/Los_Angeles] PHP Fatal error:  Declaration of Drupal\Driver\Database\sqlsrv\Schema::addIndex() must be compatible with Drupal\Core\Database\Schema::addIndex($table, $name, $fields, array $spec) in C:\inetpub\wwwroot\drupal\drivers\lib\Drupal\Driver\Database\sqlsrv\Schema.php on line 41
From: Luiz Santos <lfsantos<EMAIL_ADDRESS>Date: Tuesday, August 11, 2015 at 10:09 PM
To: Azure/msphpsql <reply<EMAIL_ADDRESS>Azure/msphpsql <msphpsql<EMAIL_ADDRESS>Subject: RE: [msphpsql] SQLSTATE[HY010]: [Microsoft][ODBC Driver Manager] Function sequence error (again) (#49)
When I'm trying to create the SQL Server database in Drupal 8, I'm getting the following error in the PHP56 error_log file:
[11-Aug-2015 22:04:27 America/Los_Angeles] PHP Fatal error:  Call to undefined function Drupal\Driver\Database\sqlsrv\wincache_ucache_get() in C:\inetpub\wwwroot\drupal\drivers\lib\Drupal\Driver\Database\sqlsrv\Connection.php on line 607
Any clue?
From: Davvid<EMAIL_ADDRESS>Sent: Thursday, August 6, 2015 10:57 AM
To: Azure/msphpsql <msphpsql<EMAIL_ADDRESS>Cc: Luiz Fernando Santos <lfsantos<EMAIL_ADDRESS>Subject: Re: [msphpsql] SQLSTATE[HY010]: [Microsoft][ODBC Driver Manager] Function sequence error (again) (#49)
Updated driver to fix compatiblity with latest head:
https://www.dropbox.com/s/j98z4fd8jh7wr0q/drivers2.rar?dl=0
Verified that the issue is still reproducible under the same conditions.

Reply to this email directly or view it on GitHubhttps://github.com/Azure/msphpsql/issues/49#issuecomment-128458978.

call to undefined function Drupal\Driver\Database\sqlsrv\wincache_ucache_get()

You need the Wincache PHP extension for the MSSQL driver to work.

Declaration of Drupal\Driver\Database\sqlsrv\Schema::addIndex() must be compatible with Drupal\Core\Database\Schema::addIndex($table, $name, $fields, array $spec)

I already published the driver on Drupal.org (https://www.drupal.org/project/sqlsrv).
You can see that the latest commit has this issue fixed and the method has the correct signature.
http://cgit.drupalcode.org/sqlsrv/tree/drivers/lib/Drupal/Driver/Database/sqlsrv/Schema.php?id=28c5fef
(line 1601)

Got any troubles to repro the issue? The latest dev from drupal.org should be working flawlessly to get past install.
https://www.drupal.org/project/sqlsrv

Hi Davvid,
I was out in a family trip until Sunday.  I’m back to work today.  Before going out, I set  up the whole environment, but I could not run the test successfully.
It just do not run.  I did not have time to investigate the issue, which I’m planning to do today.
I’ll let you know.
From: Davvid<EMAIL_ADDRESS>Sent: Thursday, August 20, 2015 6:44 AM
To: Azure/msphpsql<EMAIL_ADDRESS>Cc: Luiz Fernando Santos<EMAIL_ADDRESS>Subject: Re: [msphpsql] SQLSTATE[HY010]: [Microsoft][ODBC Driver Manager] Function sequence error (again) (#49)
Got any troubles to repro the issue? The latest dev from drupal.org should be working flawlessly to get past install.
https://www.drupal.org/project/sqlsrvhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.drupal.org%2Fproject%2Fsqlsrv&data=01|01|lfsantos%40microsoft.com|aed3e59f63794c02353b08d2a9657296|72f988bf86f141af91ab2d7cd011db47|1&sdata=G9VWBWYBpYx6HtsGAo0dJBFtrDpCdqegOSBsoetFqOg%3D
—
Reply to this email directly or view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fmsphpsql%2Fissues%2F49%23issuecomment-133011874&data=01|01|lfsantos%40microsoft.com|aed3e59f63794c02353b08d2a9657296|72f988bf86f141af91ab2d7cd011db47|1&sdata=YHIlHM4SOlCokQLg5GA6IFMARt%2FCJtLMHMNWUuFnkPU%3D.

I verified that this is still reproducible on lateset dev of driver and core.

It just do not run

That is weird, at least you should be seeing an error of some sort. I've sometimes experienced the progress bar getting stuck forever, but those bugs were already fixed.

That was exactly the behavior I’ve got.
I’ll continue investigating on my end.
Thanks
From: Davvid<EMAIL_ADDRESS>Sent: Wednesday, September 2, 2015 7:06 PM
To: Azure/msphpsql<EMAIL_ADDRESS>Cc: Luiz Fernando Santos<EMAIL_ADDRESS>Subject: Re: [msphpsql] SQLSTATE[HY010]: [Microsoft][ODBC Driver Manager] Function sequence error (again) (#49)
I verified that this is still reproducible on lateset dev of driver and core.
It just do not run
That is weird, at least you should be seeing an error of some sort. I've sometimes experienced the progress bar getting stuck forever, but those bugs were already fixed.
—
Reply to this email directly or view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fmsphpsql%2Fissues%2F49%23issuecomment-137301281&data=01|01|lfsantos%40microsoft.com|ebe800f891614c31d06508d2b4044817|72f988bf86f141af91ab2d7cd011db47|1&sdata=uvwgyq5Qcho6vWQNwUmYo7BIzX6YT5onop7xVhDlFrY%3D.

I just posted a repro script for another issue in #50.
I have the feeling that both #49 and #50 might be the same thing as they both seem to exhibit themselves when a PDO exception is thrown (and managed by the user!) while inside a transaction.
So it might be worth taking a look at #50 first given that it is an "easy" stand alone single file repro script.

I sent it to my team for investigation.
As soon as they find something, I'll reply back to you.
Sent from my iPad
On Sep 4, 2015, at 8:52 AM, Davvid <notifications<EMAIL_ADDRESS>wrote:
I just posted a repro script for another issue in #50https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fmsphpsql%2Fissues%2F50&data=01|01|lfsantos%40microsoft.com|26ca8d5cc4c24e91a81608d2b540c531|72f988bf86f141af91ab2d7cd011db47|1&sdata=LJ03hTOp1NdUUt2R1%2BR0fIG5FuaAkGNRbTDkXNFjwQ8%3D.
I have the feeling that both #49https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fmsphpsql%2Fissues%2F49&data=01|01|lfsantos%40microsoft.com|26ca8d5cc4c24e91a81608d2b540c531|72f988bf86f141af91ab2d7cd011db47|1&sdata=18fJyDJ3C7%2F210IE6MCGM3Qu7Tp3VDaaZpr6zLk6Uls%3D and #50https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fmsphpsql%2Fissues%2F50&data=01|01|lfsantos%40microsoft.com|26ca8d5cc4c24e91a81608d2b540c531|72f988bf86f141af91ab2d7cd011db47|1&sdata=LJ03hTOp1NdUUt2R1%2BR0fIG5FuaAkGNRbTDkXNFjwQ8%3D might be the same thing as they both seem to exhibit themselves when a PDO exception is thrown (and managed by the user!) while inside a transaction.
So it might be worth taking a look at #50https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fmsphpsql%2Fissues%2F50&data=01|01|lfsantos%40microsoft.com|26ca8d5cc4c24e91a81608d2b540c531|72f988bf86f141af91ab2d7cd011db47|1&sdata=LJ03hTOp1NdUUt2R1%2BR0fIG5FuaAkGNRbTDkXNFjwQ8%3D first given that it is an "easy" stand alone single file repro script.

Reply to this email directly or view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fmsphpsql%2Fissues%2F49%23issuecomment-137774580&data=01|01|lfsantos%40microsoft.com|26ca8d5cc4c24e91a81608d2b540c531|72f988bf86f141af91ab2d7cd011db47|1&sdata=eoeDFNBhL2S8RZ2F9g0qIo%2BNjY%2F67inihRoRnjziyp8%3D.

I am also getting the [Microsoft][ODBC Driver Manager] Function sequence error.
$q = "SELECT * FROM ".TBL_USERS." WHERE username = '$username'"; $stmt = sqlsrv_query($this->connection, $q, array(),array( "Scrollable" => 'static' )); if(!$stmt || (sqlsrv_num_rows($stmt) < 1)){ 	error_log("getUserInfo failed for $username: $q",3,'error.log'); 	sqlsrv_free_stmt($stmt); 	return NULL; }  else { 	/* Return result array */ 	sqlsrv_next_result($stmt); 	$dbrow = sqlsrv_fetch_array($stmt, SQLSRV_FETCH_ASSOC); 	if($dbrow === false) { 		error_log("dbrow is false\n",3,"error.log"); 		if( ($errors = sqlsrv_errors() ) != null) { 			foreach( $errors[0] as $key=>$error ) { 				error_log("$key -> $error\n",3,"error.log"); 			} 		} 	} 	elseif (is_null($dbrow)){ 		error_log("dbrow is NULL\n",3,"error.log"); 	} 	else { 		error_log("dbrow is OK\n",3,"error.log"); 	} }
It appears whenever I call the sqlsrv_fetch_array function.

@agrarian What version of driver are you using?

v4.0.1https://github.com/Azure/msphpsql/releases/tag/v4.0.1
Terry
From: Meet Bhagdev<EMAIL_ADDRESS>Sent: Tuesday, March 15, 2016 3:14 PM
To: Azure/msphpsql<EMAIL_ADDRESS>Cc: Carnes, Terry (RTIS<EMAIL_ADDRESS>Subject: Re: [msphpsql] SQLSTATE[HY010]: [Microsoft][ODBC Driver Manager] Function sequence error (again) (#49)
@agrarianhttps://github.com/agrarian What version of driver are you using?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHubhttps://github.com/Azure/msphpsql/issues/49#issuecomment-196979695

Hi @agrarian, I'm trying to understand your script. Why did you call sqlsrv_next_result() before sqlsrv_fetch_array()? Did you want to test for errors? If you drop sqlsrv_next_result() then there shouldn't be any problem.
Hi @david-garcia-garcia do you still have function sequence error problem? I looked at the other related issue #50 and your repro script is gone.

I am getting this error:
[unixODBC][Driver Manager]Function sequence error
On both the 4.3.0 release of sqlsrv and 5.1.0 preview. I am running a different framework that is mostly mentioned here (CodeIgniter rather than Drupal). I am afraid I don't know where to look for the error. Is it an error in the framework, the driver, or unixOBDC? What is the bottom line as far as fixes or workarounds? Thank you.

Hello @rkevinburton
Function sequence error can be caused by various factors, so more info is required in addition to telling us you get this error. Therefore, please create a new issue by providing repro script(s), the platform and/or version of the SQL Server, etc. The more you provide the closer we can get to help answer your questions.

I will try to put together a repro script. But it happens infrequently so it may take me some time to put together a relaiable repro. In the meantime I am connecting to
Microsoft SQL Server Developer (64-bit)
Microsoft Windows NT 6.1 (7601)
NT x64
11.0.6020.0
English (United States)
8192 MB
2 (processors)
I have tried the PHP sqlsrv driver version 4.3.0 and 5.1.0preview, and I get the error on both versions. Though, the error seems to occur less frequently with 5.1.0preview.
From: Jenny Tam<EMAIL_ADDRESS>Sent: Tuesday, October 10, 2017 10:26 AM
To: Microsoft/msphpsql<EMAIL_ADDRESS>Cc: Kevin (Ronald) Burton<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/msphpsql] SQLSTATE[HY010]: [Microsoft][ODBC Driver Manager] Function sequence error (again) (#49)
Hello @rkevinburtonhttps://github.com/rkevinburton
Function sequence error can be caused by various factors, so more info is required in addition to telling us you get this error. Therefore, please create a new issue by providing repro script(s), the platform and/or version of the SQL Server, etc. The more you provide the closer we can get to help answer your questions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/msphpsql/issues/49#issuecomment-335510658, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AdmtTVpIWEm_hVPqAzkdDEAoYykj5Gilks5sq4yegaJpZM4EPo38.

Hello @rkevinburton regrettably it's still a bit vague and we won't be able to reproduce the problem without a concrete repro scenario. For example, when you encountered this error were you doing a simple select, or update, or insert, with or without using stored procedure, transactions, etc.?

I mostly get errors that look like this:

A Database Error Occurred
Error Number: HY010[unixODBC][Driver Manager]Function sequence errorDELETE FROM users.dbo.users_groups WHERE group_id =  1 AND user_id =  685Filename: models/ion_auth_parent_model.phpLine Number: 1222

I am using CodeIgniter so admittedly the database code is buried in the MVC handling and is specific to our application.
From: Jenny Tam<EMAIL_ADDRESS>Sent: Tuesday, October 10, 2017 12:07 PM
To: Microsoft/msphpsql<EMAIL_ADDRESS>Cc: Kevin (Ronald) Burton<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/msphpsql] SQLSTATE[HY010]: [Microsoft][ODBC Driver Manager] Function sequence error (again) (#49)
Hello @rkevinburtonhttps://github.com/rkevinburton regrettably it's still a bit vague and we won't be able to reproduce the problem without a concrete repro scenario. For example, when you encountered this error were you doing a simple select, or update, or insert, with or without using stored procedure, transactions, etc.?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/msphpsql/issues/49#issuecomment-335542316, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AdmtTYDLshjJzuGIHoxCnSN9amX9KhzXks5sq6Q4gaJpZM4EPo38.

Thanks @rkevinburton. The problem likely lies in the file models/ion_auth_parent_model.php, line 1222, as indicated in the error message. Looks like something went wrong when trying to delete a certain user? I'm guessing that the user with id 685 and group 1 doesn't exist.
Please bring this problem to your application developers. FYI, function sequence error often occurs when ODBC functions are called out of the order required. You can check this page for different scenarios that caused HY010 error

If a record doesn’t exist then I get an error that the record doesn’t exist. I am not clear on what ‘function sequence error’ is?
This is the code that is an line 1222
$this->db->delete($this->tables['users_groups'], array($this->join['groups'] => (int)$group_id, $this->join['users'] => (int)$user_id));
From: Jenny Tam<EMAIL_ADDRESS>Sent: Tuesday, October 10, 2017 3:17 PM
To: Microsoft/msphpsql<EMAIL_ADDRESS>Cc: Kevin (Ronald) Burton<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/msphpsql] SQLSTATE[HY010]: [Microsoft][ODBC Driver Manager] Function sequence error (again) (#49)
Thanks @rkevinburtonhttps://github.com/rkevinburton. The problem likely lies in the file models/ion_auth_parent_model.php, line 1222, as indicated in the error message. Looks like something went wrong when trying to delete a certain user? I'm guessing that the user with id 685 and group 1 doesn't exist.
Please bring this problem to your application developers. FYI, function sequence error often occurs when ODBC functions are called out of the order required. You can check this page https://docs.microsoft.com/en-us/sql/odbc/reference/syntax/sqlfetch-function for different scenarios that caused HY010 error
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/msphpsql/issues/49#issuecomment-335594624, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AdmtTX9i40NUU3qG6TMqFZ50ApxZOcf3ks5sq9DTgaJpZM4EPo38.

@rkevinburton this is beyond our support as the error is clearly specific to your application and how the class(es) work(s) underneath. Please bring it to your application developers.

Closing due to inactivity. Please reopen if necessary.


Feature Request: Configurable maximum number of displayed options in Select

feature request:
An input property for defining maximum number of options displayed in the dropdown.
What is the expected behavior?
Currently the maximum number is always 5. If I could define 10, maximum 10 items could be displayed, while total number could be 8 or 20.
What is the use-case or motivation for changing an existing behavior?

When the vertical real-estate is tall enough, displaying more options means less scrolling and quicker select.
In some devices, the vertical scrollbar is not displayed until finger touching. This had caused confusion to users who may not know there may be more options hidden. Having more options displayed could reduce such confusion.

Which versions of Angular, Material, OS, TypeScript, browsers are affected?
As per Material 6.2.1 and older releases.

With HTML, I may use this:
 <select onfocus='this.size=10;'
onblur='this.size=1;'
onchange='this.size=1; this.blur();'>
    <option>1</option>
    <option>2</option>
    <option>3</option>
    <option>4</option>
    <option>5</option>
    <option>6</option>
    <option>7</option>
    <option>8</option>
    <option>9</option>
    <option>10</option>
    <option>11</option>
     <option>12</option>
     <option>13</option>
<option>14</option>
<option>15</option>
      <option>16</option>
    <option>17</option>
    <option>18</option>
    <option>19</option>
    <option>20</option>
  </select>

However, I would prefer to use mat-select overall.

Closing as you can control the max number of items that are visible in the select by setting a max-height style value for the mat-select-panel.
For example
.mat-select-panel {
  max-height: calc(48px * 10);
}

I should had mentioned I had tried what you @josephperrott had suggested. But it changes nothing. I had also tried other mat-XXXX classes, but at best got fixed height of scrollable area, not more items to be displayed. Please do re-open the case.


Create a test workflow cronjob

what test data to use --> /gpfs/gsfs10/users/CCBR_Pipeliner/testdata/rnaseq/old-demo-data
human data
run every week
save data for each month ... autodelete older files.

It would be great to launch two separate runs -- one with paired and one with single end data. There are a lot of if/else statements in the pipeline to control which rules are run depending on PE vs SE.


[WIP] add Cudnn mode as optional param to batch norm and softmax

split to https://github.com/apache/incubator-singa/pull/540 and https://github.com/apache/incubator-singa/pull/541


schemavalidator:421:65: executing "schemavalidator" at <.>: wrong type for value; expected string; got interface {}

Problem statement
Not sure if this a problem with go-swagger or the swagger spec I'm trying to use.
2018/02/09 20:37:57 name field AddLabel
2018/02/09 20:37:57 package field models
2018/02/09 20:28:57 creating generated file "add_label.go" in "models" as definition
failed rendering template data for definition: template execution failed for template definition: template: schemavalidator:421:65: executing "schemavalidator" at <.>: wrong type for value; expected string; got interface {}```

## Swagger specification
https://quay.io/api/v1/discovery

## Steps to reproduce
`swagger generate client -f swagger.json -A client -c quay -a quay`
## Environment
swagger version: dev
go version: go version go1.9.3 linux/amd64
OS: VoidLinux

I am seeing this error locally when the types in an enum field don't match the declared type.
The log in the report above shows add_label just before the error. This is the stanza from the quay.io source:
    "AddLabel": {
      "required": [
        "key",
        "value",
        "media_type"
      ],
      "type": "object",
      "description": "Adds a label to a manifest",
      "properties": {
        "media_type": {
          "enum": [
            "text/plain",
            "application/json",
            null
          ],
          "type": [
            "string",
            "null"
          ],
          "description": "The media type for this label"
        },
        "value": {
          "type": "string",
          "description": "The value for the label"
        },
        "key": {
          "type": "string",
          "description": "The key for the label"
        }
      }
    }

I think in this case, the error could be caused by the combined string and null types?
My swagger has ints trying to go in to a string field, so it's more clearly a problem with the input.

I made a patch to handle null in enum (this another issue involving quay.io).
The current issue I test too with this patch and works.
This patch is still under development, since fixing this particular issue is just a by-product of another feature.

Hey I left a message on slack about this, but figured I should put it here to. We are running into the same issue when it is trying to parse a definition with a null in option in an enum like this:
installs:
        type: array
        items:
          properties:
            id:
              type: string
              format: uuid
              example: 123456789-abcd-4321-dcba-123456789ab
            environment:
              type: string
              x-nullable: true
              enum:
              - production
              - staging
              - development
              - null

YAML has a list of "magic" words that have types that aren't strings, and null is one of those.  You need to put it in quotes to make it a string: - "null".  The same thing happens with yes,no,on,off,true,false which are booleans.


Iridium quality displays as (unknown)

Fixed: 5b827403aee592e81b4b6bc19693e21062046a1e

Note: quality goes 0, 1, 2, 4


XamForms editor issues

we need to leave out IsReadOnly for now so that you can select text (https://github.com/xamarin/Xamarin.Forms/issues/12930)
there's no vertical scrolling. Possibly related to https://github.com/xamarin/Xamarin.Forms/issues/9550, but I think it's because it's hardcoded to use NSTextField rather than NSTextView inside a scroll view.

For now, we use a custom renderer. It at least fixes vertical scrolling.


OSError: [Errno 2] No such file or directory

I try to run aetros on windows and get the following error. I did not have any issues when running it from linux. I have keras==1.2.2 installed. I do not know if the error is aetros related or not. Can you provide any feedback ?
Exception in thread Thread-1:
Traceback (most recent call last):
File "c:\python27\lib\threading.py", line 801, in __bootstrap_inner
self.run()
File "c:\python27\lib\threading.py", line 754, in run
self.__target(*self.__args, **self._kwargs)
File "c:\python27\lib\site-packages\aetros\commands\ServerCommand.py", line 131, in thread
if not self.connect():
File "c:\python27\lib\site-packages\aetros\commands\ServerCommand.py", line 82, in connect
self.event_listener.fire('registration', server_id)
File "c:\python27\lib\site-packages\aetros\backend.py", line 87, in fire
callback(parameter)
File "c:\python27\lib\site-packages\aetros\commands\ServerCommand.py", line 388, in registration_complete
self.server.send_message({'type': 'system', 'values': self.collect_system_information()})
File "c:\python27\lib\site-packages\aetros\commands\ServerCommand.py", line 405, in collect_system_information
values['disks'][name] = psutil.disk_usage(disk[1]).total
File "c:\python27\lib\site-packages\psutil_init.py", line 2017, in disk_usage
return _psplatform.disk_usage(path)
File "c:\python27\lib\site-packages\psutil_pswindows.py", line 249, in disk_usage
raise OSError(errno.ENOENT, msg)
OSError: [Errno 2] No such file or directory: 'X:'
Traceback (most recent call last):
File "c:\python27\lib\runpy.py", line 162, in run_module_as_main
"main", fname, loader, pkg_name)
File "c:\python27\lib\runpy.py", line 72, in run_code
exec code in run_globals
File "C:\Python27\Scripts\aetros.exe_main.py", line 9, in
File "c:\python27\lib\site-packages\aetros_init.py", line 58, in main
code = command.main(cmd_args)
File "c:\python27\lib\site-packages\aetros\commands\ServerCommand.py", line 301, in main
self.server.send_message({'type': 'utilization', 'values': self.collect_system_utilization()})
File "c:\python27\lib\site-packages\aetros\commands\ServerCommand.py", line 434, in collect_system_utilization
values['disks'][name] = psutil.disk_usage(disk[1]).used
File "c:\python27\lib\site-packages\psutil_init_.py", line 2017, in disk_usage
return _psplatform.disk_usage(path)
File "c:\python27\lib\site-packages\psutil_pswindows.py", line 249, in disk_usage
raise OSError(errno.ENOENT, msg)
OSError: [Errno 2] No such file or directory: 'X:'

Drive X: is my CDROM drive. I inserted a cd and now I get the following error:
Exception in thread Thread-1:
Traceback (most recent call last):
File "c:\python27\lib\threading.py", line 801, in __bootstrap_inner
self.run()
File "c:\python27\lib\threading.py", line 754, in run
self.__target(*self.__args, **self.__kwargs)
File "c:\python27\lib\site-packages\aetros\commands\ServerCommand.py", line 13
1, in thread
if not self.connect():
File "c:\python27\lib\site-packages\aetros\commands\ServerCommand.py", line 82
, in connect
self.event_listener.fire('registration', server_id)
File "c:\python27\lib\site-packages\aetros\backend.py", line 87, in fire
callback(parameter)
File "c:\python27\lib\site-packages\aetros\commands\ServerCommand.py", line 38
8, in registration_complete
self.server.send_message({'type': 'system', 'values': self.collect_system_in
formation()})
File "c:\python27\lib\site-packages\aetros\commands\ServerCommand.py", line 40
8, in collect_system_information
if 0 != id.find('lo') and net.isup:
AttributeError: 'tuple' object has no attribute 'isup'
Traceback (most recent call last):
File "c:\python27\lib\runpy.py", line 162, in run_module_as_main
"main", fname, loader, pkg_name)
File "c:\python27\lib\runpy.py", line 72, in run_code
exec code in run_globals
File "C:\Python27\Scripts\aetros.exe_main.py", line 9, in
File "c:\python27\lib\site-packages\aetros_init.py", line 58, in main
code = command.main(cmd_args)
File "c:\python27\lib\site-packages\aetros\commands\ServerCommand.py", line 30
1, in main
self.server.send_message({'type': 'utilization', 'values': self.collect_syst
em_utilization()})
File "c:\python27\lib\site-packages\aetros\commands\ServerCommand.py", line 47
4, in collect_system_utilization
values['loadavg'] = os.getloadavg()
AttributeError: 'module' object has no attribute 'getloadavg'

It seems that the client does not run on Windows, since the command os.getloadavg() in only available on Unix systems based on the documentation https://docs.python.org/2/library/os.html#os.getloadavg

All those issues should be fixed in version 0.9.9

Thanks for reporting this to us! Currently, we do not officially support Windows (since we do not have any windows developer in the team), however there should not many open issues left.

I updated to 0.9.9 and it seems OK now. Thank you !!


multiple floating ip pools

Is there a way to create multiple floating ip pools using hiera.
openstack::network::external::ippool::start: <IP_ADDRESS>
openstack::network::external::ippool::end: <IP_ADDRESS>

Looks like not. However, it seems like it would be reasonably easy to add a parameter or change these parameters to accept an array of ['start=x.x.x.x,end=x.x.x.x', ...] instead of just specifying one start and one end.

I couldn't figure it out. I'm not sure how to pass these array ['start=x.x.x.x,end=x.x.x.x', ...] and where exactly I can add parameters.
Here is what I'm trying to achieve.
Private (Tenant network) - <IP_ADDRESS>/24
Public (Internet) - <IP_ADDRESS>/25
And, now I'm trying to add another public network which is
Public (Internet) <IP_ADDRESS>/24
-Sakib

In https://github.com/cmurphy/puppetlabs-openstack/commit/308643afd87f08f0a249f1da21582648474aa7be we changed the network resource creation to be more free-form so that you can just use a hash of neutron_subnets in your hiera data. The example hiera data should give you an idea of how to do this now.


Removing App

I got pock and gave it permission but I miss my regular touch bar. I deleted the app and now the configuration that pock did to my touch bar won't disappear. How do I remove it? Nothing I do works.

Have you tried rebooting your Mac ?


Missing features

There is no clay, no animations, the water is at the same level that the other blocks... It seems to be an old version (even older than that shown in the video).

This is not the case - the world generator just changed. The game is up to date.


Toolbar colour doesn't visibly/practically update

Works better with left/right clicking squares now. Still need to make it work with right-clicking rectangles (and maybe other things too)?

I'll leave that for a separate issue


PI Swaps: Swap with multiple paths is broken

Following most recent PI decision I think we should consider removing PI for swaps completely @brunoguerios LMKWYT.
Seems to break because we build reveresed path incorrectly. Here swapQueryOutput.pathAmounts does not exist so we only add a single value to array ([swapQueryOutput.expectedAmountOut.amount]).
Some debug output:
Original swapInput:
[
  {
    inputAmountRaw: '617283945500000000',
    outputAmountRaw: '694318289054249432',
    pools: [
      '0x72f4de7ef3e2cebf98ae2aff38533fe899629ebf000100000000000000000ae3',
      '0xf3312968c7d768c19107731100ece7d4780b47b2000200000000000000000a50'
    ],
    vaultVersion: 2,
    tokens: [ [Object], [Object], [Object] ]
  },
  {
    inputAmountRaw: '617283945500000000',
    outputAmountRaw: '688262793068237442',
    pools: [
      '0xcd78a20c597e367a4e478a2411ceb790604d7c8f000000000000000000000c22'
    ],
    vaultVersion: 2,
    tokens: [ [Object], [Object] ]
  }
]

swapQueryOutput:
{
  swapKind: 0,
  expectedAmountOut: TokenAmount {
    token: Token {
      chainId: 137,
      address: '0x0d500b1d8e8ef31e21c99d1db9a6444d3adf1270',
      decimals: 18,
      symbol: undefined,
      name: undefined,
      wrapped: '0x0d500b1d8e8ef31e21c99d1db9a6444d3adf1270'
    },
    scalar: 1n,
    decimalScale:<PHONE_NUMBER>000000000n,
    amount:<PHONE_NUMBER>122485691n,
    scale18:<PHONE_NUMBER>122485691n
  }
}

reverseSwapInput (note inputAmountRaw: undefined,):
{
  chainId: 137,
  paths: [
    {
      inputAmountRaw:<PHONE_NUMBER>122485691n,
      outputAmountRaw: '617283945500000000',
      pools: [Array],
      vaultVersion: 2,
      tokens: [Array]
    },
    {
      inputAmountRaw: undefined,
      outputAmountRaw: '617283945500000000',
      pools: [Array],
      vaultVersion: 2,
      tokens: [Array]
    }
  ],
  swapKind: 0
}

With the decision of relying on USD prices to calculate priceImpact for swaps on the FE, no one else is consuming priceImpact for swaps from the SDK, right?
If that's indeed the case, I'd be ok on removing it to avoid confusion.
On the other hand, if we expect this to be used by anyone in the future, it should be ok to leave as a low priority task to be fixed in the long term. I doesn't seem like something that would take too long to get done.


Error 400: invalid_scope

Has been working perfectly for a long time and now I get:
Error 400: invalid_scope
Some requested scopes cannot be shown: [https://www.googleapis.com/auth/drive, https://www.googleapis.com/auth/compute, https://www.googleapis.com/auth/appengine.admin, https://www.googleapis.com/auth/cloud-platform]
Request Details
redirect_uri=urn:ietf:wg:oauth:2.0:oob
prompt=select_account
response_type=code
client_id=32555940559.apps.googleusercontent.com
access_type=offline
display=page
scope=https://www.googleapis.com/auth/userinfo.email https://www.googleapis.com/auth/cloud-platform https://www.googleapis.com/auth/appengine.admin https://www.googleapis.com/auth/compute https://www.googleapis.com/auth/accounts.reauth https://www.googleapis.com/auth/drive https://www.googleapis.com/auth/plus.me

I don't find https://www.googleapis.com/auth/accounts.reauth among the accepted scopes: https://developers.google.com/identity/protocols/googlescopes. Maybe that's the one causing the issue.

I also discovered that the last line,
apt-get -y install -qq google-drive-ocamlfuse fuse
results in
E: Unable to locate package google-drive-ocamlfuse
even though the repository has been added. Maybe this could be the cause of the issue?

I am also having this issues today

@JEF1056 @darylfung96: Which version of Ubuntu are you on? (You can get it running: lsb_release -a). I've removed the package for Ubuntu versions that reached end of life (zesty and artful).

Distributor ID:	Ubuntu
Description:	Ubuntu 17.10
Release:	17.10
Codename:	artful

@andrewkhimichuk: if you cannot upgrade to bionic, can you try downloading the package (amd64) from bionic and install it with sudo dpkg --install?

@astrada: I tried to install the package with sudo dpkg --install but it says that another package is needed:
 Preparing to unpack google-drive-ocamlfuse_0.7.0-0ubuntu1_ubuntu18.04.1_amd64.deb ... Unpacking google-drive-ocamlfuse (0.7.0-0ubuntu1 ~ ubuntu18.04.1) over (0.7.0-0ubuntu1 ~ ubuntu18.04.1) ... dpkg: dependency problems prevent configuration of google-drive-ocamlfuse:  google-drive-ocamlfuse depends on libfuse2 (>= 2.8); however:   Package libfuse2 is not installed.
dpkg: error processing package google-drive-ocamlfuse (--install):  dependency problems - leaving unconfigured Errors were encountered while processing:  google-drive-ocamlfuse

Please have a look at https://github.com/astrada/google-drive-ocamlfuse/issues/493#issuecomment-422385109. Thanks to @daniel-leonard-robinson

@DonPex Changing my code to this:
# Install a Drive FUSE wrapper.
# https://github.com/astrada/google-drive-ocamlfuse
!apt-get install -y -qq software-properties-common python-software-properties module-init-tools
!add-apt-repository -y ppa:alessandro-strada/ppa 2>&1 > /dev/null
!apt-get update -qq 2>&1 > /dev/null
!wget https://launchpad.net/~alessandro-strada/+archive/ubuntu/ppa/+files/google-drive-ocamlfuse_0.7.0-0ubuntu1~ubuntu18.04.1_amd64.deb
!apt-get install libfuse2
!dpkg --install google-drive-ocamlfuse_0.7.0-0ubuntu1~ubuntu18.04.1_amd64.deb
#!apt-get -y install -qq google-drive-ocamlfuse fuse

Based off astrata's answer seemed to fix the issue.


Refactor managers, permissions etc.

It is extremely big change. I'm not sure if it is possible to review well. Ask me about any doubts.
Main changes:

Implemented SimpleObjectManager that serves set of objects and links to users (Calendars)
Implemented ExtendedObjectManager that also serves links to devices and groups (Geofences, Drivers, Attributes)
Implemented getObjects, addObject, updateObject, removeObject database functions, they used Class name to select request string.
Some database.* names was changed to consistence
Changed permissions model hierarchy to fit new managers model. Added interface with getSlaveId function.
Combined permissions resources. Unfortunately had to combine devices and groups permissions in its main resource, because @Delete regex matching failed if it is in different resources.
Implemented smart functions to check and refresh permissions

PS: I believe new managers model might be used to optimize users, devices and groups management.

Hope did not missed any of your comments.

Totally removed any permissions classes, works with them as Map
Combined link/unlink API to one /api/permissions
Make type constants and helpers to work with type names.

As consequence checkPermissions(object,...  and linkObject become tolerate to names. They correctly react on device/Device/deviceId names.

Used classes everywhere and fix other comments

Added permission helper class and reorganized checking

Sorry missed that. Also made some clean up after PMD report.

Merged, looks good. I hope you thoroughly tested it ;)

Am I understanding correctly that we need some modifications to the web app because API paths changes? Also, can you please update swagger file.

I'll try to describe it in swagger, But  not sure if it is possible to do it clear.

Worst case you can just have some long comment describing the functionality.


'Long' only refers to a type, but is being used as a namespace here.

Facing this error for all use of Long inside this directory
node_modules/cosmjs-types/tendermint/
example :

that is not this repository.


BSAG trams not showing correctly

i'm on traveldroid and bremen trams aren't showing up correctly.
the HafasLineId is correct - so...did i miss something?

vbn-bsag,1,,8-webbtr-1,#00a54f,#ffffff,,rectangle
vbn-bsag,6,,8-webbtr-6,#feca0a,#000000,,rectangle

gonna move that issue over to Träwelldroid, I think the .csv is correct.

Quick look on this tells me, that you might have forgotten to add the "operatorCode" (operatorID) to the CSV file. This should always been included except when it is "nahreisezug"

Ahh, lovely. I assumed it wasn't necessary because it's just a tram line.
Gonna fix it asap, thank you!!!

Seems fixed now ;-)

that's great!!! i'm gonna start a pr for the other networks.
thank you so much!!!


chore(app): Loosen swift-support version

I merged the PR but left you as reviewers so you will know what I did


Reference type for primitives allowing interfacing with POSIX

Some POSIX functions require parsing a reference to a primitive type which is currently unable to be expressed within uniview.
Implement a template type that can only apply to non-linear types to allow lending of primitive values and thus parsing references.
let a = Reference#[i64]();
time(@a); // POSIX function
let unix_time = Read(a);

Implemented as part of #39


Create Docker image from the files after transformation

Create a Docker image based on the passed in tag
Added switch to copy content into CD folder
Updated readme with example usage

Thanks @jammykam , I just tested everything and merged in the PR, huge thanks for the big PR!


Update README to show the latest version

The README shows an example with v2.0.0. This is confusing for people trying to use this action. When you copy and paste the example into your workflow it does not work because v2.0.0 does not exist. Updating this to v1.4.3 allows users to copy and paste the example, and not have it complain about the version.

Thanks @robandpdx! Would be great to have a way to keep the readme example and marketplace version in sync somehow.


Museum of Art data sets are 404 (missing)

There used to be RSS feeds for Art After 5 and Art Exhibits - https://opendataphilly.org/organizations/philadelphia-museum-of-art/
They have either moved or no longer exist.
Contact Art Museum webmaster to determine fate<EMAIL_ADDRESS>

Followed up with<EMAIL_ADDRESS>


[BUG][Kernel] Delta kernel read checkpoint parquet file failed with exception "list not found in optional group readerFeatures (LIST)"

Bug
Which Delta project/connector is this regarding?

[ ] Spark
[ ] Standalone
[ ] Flink
[x] Kernel
[ ] Other (fill in here)

Describe the problem
When use delta kernel to read delta lake table encountered such an error
list not found in optional group readerFeatures (LIST) {
  repeated group bag {
    optional binary array (STRING);
  }
}

and the schema of parquet file is

And I think delta kernel currently does not supports reading this legacy array type.
Steps to reproduce

Observed results

Expected results

Further details

Environment information

Delta Lake version:
Spark version:
Scala version:

Willingness to contribute
The Delta Lake Community encourages bug fix contributions. Would you or another member of your organization be willing to contribute a fix for this bug to the Delta Lake code base?

[ ] Yes. I can contribute a fix for this bug independently.
[ ] Yes. I would be willing to contribute a fix for this bug with guidance from the Delta Lake community.
[ ] No. I cannot contribute a bug fix at this time.

@Youngwb Thanks for reporting. Currently, the default Parquet reader is not handling the legacy list physical format. Will post a PR shortly.


Final preparations for 0.6.3

Add release notes and minor docs improvements.
Use default timeout in method connect(path='...') (#380).
Bump version.

Codecov Report

Merging #383 into master will increase coverage by <.01%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master     #383      +/-   ##
==========================================
+ Coverage   94.49%   94.49%   +<.01%
==========================================
  Files          49       49
  Lines       19771    19772       +1
==========================================
+ Hits        18682    18684       +2
+ Misses       1089     1088       -1


Fixing bootstrap script

@kris-nova Looks like there's the same problem in the node script as well.


Setting datacenter_id in knife.rb has no effect

According to the documentation of knife-profitbricks

the data center ID can be added to the knife.rb which reduces the need to include the --datacenter-id parameter for each action within the data center.

Therefore I would expect that executing
knife profitbricks server list
will return a list of servers in a datacenter if I set it's ID by
knife[:datacenter_id] = '9b43cffa-d348-40f8-88ca-e4d58da5a26e'
in knife.rb.
Instead the call fails with the error message
You must supply -D DATACENTER_ID!
When I change the call to
knife profitbricks server list -D '9b43cffa-d348-40f8-88ca-e4d58da5a26e'
the servers within the datacenter are displayed.
That my knife.rb is used, is confirmed by the fact that my profitbricks_username is also set in knife.rb

@jbuchhammer This has been resolved in #10 and v1.2.1 has been released. The --datacenter-id parameter was explicitly set to required. Once that was removed, the ID is now properly read from the knife.rb.


Readme file update about toolbox new installation option

pip install marvin-python-toolbox is working !!!

Job already included in marvin-plataform-book#11


Examples showing non-existent class

Several examples show passing a new instance of Auryn\ReflectionPool into Injector constructor. However, that class does not seem to be defined anywhere in the library.
The Injector constructor itself creates a new CachingReflector if no dependency is passed -- is the documentation out of date in this respect?

This is addressed by https://github.com/rdlowrey/Auryn/commit/be3807b62a6991e61151b55b6974dc1289afabd5
Thanks :+1:


Can't make it work

Hi,
I love the ide-python add-on on Max and Windows. It works great! But when I tried to install it on my Ubuntu laptop. It stopped working. Even after I changed the default Python Language Server Path to './home/liangchen/.local/bin/pyls', it still does not work. What am I doing wrong here?
I am using Ubuntu 17.10. Python 3.6.
Thank you for your help in advance.
Cheers,
Frank

./home/liangchen/.local/bin/pyls as a path is probably wrong, since it looks for a directory called home/ in the current directory (because you started with ./ where the dot is the current working directory). You probably want this to be /home/liangchen/.local/bin/pyls.
However, if your PATH environment variable is properly set up and pyls is correctly installed, then you don't need to specify this explicitly.
Can you invoke pyls from a terminal? If yes, what does which pyls and echo $PATH say?

Checkout the release notes of version 1.0


XSS vulnerability

Raw user content is included into the results page, leading to potential issues with XSS for keyword queries like:
1"><script>alert(150)</script>
A quick solution (not requiring code changes) is to use the following guide: http://www.servletsuite.com/servlets/xssflt.htm

Solved it via the web server.


set default state and image for simulatorPump

This fixes an issue with "Add Pump" not displaying pump after Simulator Pump is added per @dnzxy.

Both with and without this PR applied, I notice with a fresh install on a simulator, adding a simulator pump still shows the "add pump" icon after selecting it. Sending a bolus changes the "add pump" icon to a "100% battery" icon, though, both with and without this PR. Removing the pump instantly changes the icon to "add pump", and if I then select "simulator" again for pump it shows the "100% battery" icon as soon as it's selected without having to issue another bolus.
Am I testing the wrong thing? What steps should I take?

Am I testing the wrong thing? What steps should I take?

No, you are correct.  There is still an issue with this PR that needs further investigation.  I'll convert this PR to a draft 'til we isolate the additional issues.


PLACEHOLDER_LICENSING_TEXT is missing a colon

Fontbakery wants the text to be:
This Font Software is licensed under the SIL Open Font License, Version 1.1. This license is available with a FAQ at http://scripts.sil.org/OFL

But I think it should be:
This Font Software is licensed under the SIL Open Font License, Version 1.1. This license is available with a FAQ at: http://scripts.sil.org/OFL

(Notice the colon after at).

I suspect you are right. It should be whatever the majority of existing
ttfs in google/fonts use.

Of fonts at google/fonts that use similar format, 922 files have a colon after at and 52 don’t.

Does it also apply to the apache licensing text?

@khaledhosny Can you please verify if the apache licensing text also needs to have the missing colon? I also wonder about the missing "http://" in the apache text. Which is the most common wording for the Apache license ?

Ttx -t name fonts/apace/*/*ttf
And grep
On Jul 18, 2016 10:22 AM, "Felipe Corrêa da Silva Sanches" <
<EMAIL_ADDRESS>wrote:

@khaledhosny https://github.com/khaledhosny Can you please verify if
the apache licensing text also needs to have the missing colon? I also
wonder about the missing "http://" in the apache text. Which is the most
common wording for the Apache license ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/googlefonts/fontbakery/issues/843#issuecomment-233342935,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAP9y3A_zboRvk2xOVe0ZXpgQJ4Get9Jks5qW4wPgaJpZM4JOVYo
.

Most Apache-licensed fonts on google/fonts just say Licensed under the Apache License, Version 2.0, but the few that provide a URL has a colon.

@felipesanches how many fonts on google/fonts/apache say
Licensed under the Apache License, Version 2.0
and how many say,
Licensed under the Apache License, Version 2.0: <URL>
?

I found roughtly 300 fonts with Licensed under the Apache License, Version 2.0 in google fonts github repo (by grepping their name tables extracted with ttx)
I found none with the colon and a URL.

Ok then I guess this is fine :)

I fixed the reference workding the check expects so that the hundreds of apache licensed fonts at the google fonts git repo will pass this check. This wording is, thus, our normative reference.


stdout maxBuffer length exceeded (diff.js)

I ran into a error while comparing two branches with large amount of changes.
events.js:174
      throw er; // Unhandled 'error' event
      ^

RangeError [ERR_CHILD_PROCESS_STDIO_MAXBUFFER]: stdout maxBuffer length exceeded
    at Socket.onChildStdout (child_process.js:348:14)
    at Socket.emit (events.js:189:13)
    at addChunk (_stream_readable.js:284:12)
    at readableAddChunk (_stream_readable.js:261:13)
    at Socket.Readable.push (_stream_readable.js:220:10)
    at Pipe.onStreamRead (internal/stream_base_commons.js:94:17)
Emitted 'error' event at:
    at /home/user/git/projectname/node_modules/gulp-git/lib/diff.js:87:31
    at ChildProcess.exithandler (child_process.js:301:5)
    at ChildProcess.emit (events.js:189:13)
    at maybeClose (internal/child_process.js:970:16)
    at Socket.stream.socket.on (internal/child_process.js:389:11)
    at Socket.emit (events.js:189:13)
    at Pipe._handle.close (net.js:600:12)

I found similar problem with solution issue#68.
In diff.js:86 I changed
exec('git diff --raw -z ' + cmd, {cwd: opt.cwd}, function(err, stdout) {

to
exec('git diff --raw -z ' + cmd, {cwd: opt.cwd, maxBuffer: Infinity}, function(err, stdout) {

Now everything works.

Added option to pass it in, thanks for reporting


Move to Roslyn-based StyleCopAnalyzer

We currently consume StyleCop via the StyleCop.MSBuild NuGet package. The project indicates that there is unlikely to be major new development on this, and that the Roslyn-based StyleCopAnalyzers project is recommended for developers who use VS 2015 or later.
There are some existing pain points with the StyleCop.MSBuild component, including:

Not being as reliable as desired on non-windows platforms (e.g. on the Mac)
Not supporting some newer language features (e.g. declaring out parameters in line).

The Roslyn based StyleCopAnalyzer:

Is supported on .NET Framework, Core, and Standard projects.
It will also show violations as you type, at design time, and at compile time.
Supports actions to automatically fix up violations.

Transition
The StyleCop.MSBuild component used a Settings.StyleCop configuration file that is not used by StyleCopAnalyzer. The severity of certain rules can be customized with rulesets. After setting up the StyleCopAnalyzer with default rules, I went through the reported violations in the projects and resolved them as follows:

For rules that the project did not follow previously, I added an exception to GVFS.ruleset
For violations that it looks like should have been flagged previously, I fixed up the code in place
For violations in projects that might not rules might not checked, I resolved violations in place.

Now projects use our custom ruleset instead of the Microsoft Recommended Managed Rules. The Microsoft Recommended Managed rules included rules that controlled static analysis. However, this analysis was not run by default previously, and is not supported for .NET Framework or .NET standard projects. As we are moving to .NET Core / .NET Standard, and we didn't utilize these rules previously, I left them out of our custom ruleset.
Open Issues

[ ] I am still playing with this configuration a bit more to make sure it is stable. I have observed some funny behavior with VS 2019 (but not with VS 2017)

The Microsoft Recommended Managed rules included rules that controlled static analysis. However, this analysis was not run by default previously, and is not supported for .NET Framework or .NET standard projects. As we are moving to .NET Core / .NET Standard, and we didn't utilize these rules previously, I left them out of our custom ruleset.

Do you have a pointer to the docs that says this isn't supported on .NET Standard? I would be surprised if it worked on .NET Core but not on Standard. Perhaps the issue is that it can't run on .NET Standard projects that are being consumed by Framework projects?
In the long term, I think we'll want to move everything to .NET Core 3 (if we're able to consume C++/CLI there), should we file a work item to track turning on the Microsoft Recommended Managed Rules when we're fully on .NET Core?

@derrickstolee - Thanks for trying this out.
What I think is happening is that analysis of the XML documentation is not enabled for builds, but when you have the file open in Visual Studio, the (design-time?) analyzer does do analysis of the XML documentation. Can you confirm that you only see this error when the document is open in Visual Studio (you might have to close the file and do a build)?
We could fix this by:

Enabling the checking of XML documentation during compile time, which would entail making sure the projects generate XML documentation files

Suppressing the existing documentation violations for now, and then deciding later on which ones we want to enable and whether we want to tweak the build to also generate XML documentation files.

I removed all the XML documentation rules for now. I will go through and try to make sure I didn't miss any. If you encounter others I have missed, please let me know.

@nickgra

Do you have a pointer to the docs that says this isn't supported on .NET Standard? I would be surprised if it worked on .NET Core but not on Standard.

I mistyped that. I have updated the text in the PR description, it should state that the static analysis is not supported on .NET Core and .NET Standard. It is only supported on .NET Framework.

should we file a work item to track turning on the Microsoft Recommended Managed Rules when we're fully on .NET Core?

Possibly. I figure we can follow up with other analysers. There exists a NuGet package with FXCop analysers that we could look at, and they should apply accross all of our projects. We don't have to wait until we are on .NET Core. Maybe when someone starts that work they can open an issue to discuss it.

I'll do some manual testing on Windows and MacOS this afternoon, but this looks great. Thanks for putting this together Jameson!

Thank you everyone for taking the time to test this and provide feedback!
@kewillford - I like your suggestion. I don't look at this change as the final goal, but a step in keeping the repository in good shape. I will follow up with further issues to continue to improve this.
If anyone hits issues after updating, please let me know. @nickgra had indicated that he had experienced some oddities and had to delete his .vs folder. At one point while switching between this branch and branches without this change I had to delete my BuildOutput folder as these Analyzers were not removed from the solution.


KAFKA-14367; Add OffsetDelete to the new GroupCoordinator interface

This patch adds OffsetDelete to the new GroupCoordinator interface and updates KafkaApis to use it.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

cc @jolshan @jeffkbkim

Things are starting to fall into a pretty common pattern here. :) Looks pretty good, but I'll give Jeff a chance to take a look as well.

@jolshan @jeffkbkim This one is ready for the last round.


Set RoleAssignmentProperties to be a direct paramter of RoleAssignmen…

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

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.

adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

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

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

Summary of Changes
Prior to this change, request body from generated clients for RoleAssignments_Create was unnecessarily wrapped in an additional properties object, resulting in 400 errors for those requests.
Request shape before:
{
    "properties":
    {
        "roleDefinitionId": "<someRoleDefinition>",
        "principalId":"<somePrincipalId>"
    }
}

Request shape after:
{
    "roleDefinitionId": "<someRoleDefinition>",
    "principalId":"<somePrincipalId>"
}

Can one of the admins verify this patch?


how can I get the extrinsics of particular picture?

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
D435

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
win10

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC

SDK Version
}

Language
{C++}

Segment
{others }

Issue Description
<Describe your issue / question / feature request / etc..>
Hello, I want to ask some questions about calibration.
I downlowd the CalibrationToolAPI , and it returns the extrinsic between Right camera to Left camera and RGB camera to Left camera. I got the  RotationLeftRight TranslationLeftRight, RotationLeftRGB, TranslationLeftRGB.
What I want is the extrinsic of  pratical picture in left camera or in rgb camera.
For example, I am taking  15 pictures of the checkerboard in infrared camera1 , I want to know the 15th picture's extrinsic. The Calibration Userbook said that the left camera is located in world origin, so I can get the 15th checkerboard_left camera Transformation matrix, including RotationLeft_15thPicture, TranslationLeft_15thPicture.
Or I  take 15 pictures of the checkerboard in rgb camera , so I can get the  15th  checkerboard_rgb camera Transformation matrix, including RotationRGB_15thPicture, TranslationRGB_15thPicture.
Anyway, how can I get the extrinsics of particular picture instead of camera sensor?
I try to put the RGB pictures in matlab calibrator, it returns  3315 ratation and 3*15 translation, so I can get the 15th picture's extrinsic. But the intrinsic calculated in matlab is deferent in DyCalibRusult. For example, the FocalLenthRGB is 1392.89, 1393.84. But the  FocalLenthRGB in matlab is 1384. And the DistortionRGB=0, but  matlab has the Radial distortion. Also the PrincipalPointRGB is different.

Due to the wrong instrinsic, so i think extrinsic is also wrong.
When i put the infrared pictures in matlab stereo calibrator, it refuses me a lot pictures（input 22 pairs remain 2 pairs pictures）and it shows the wrong cross point in checkerboard.

Due to the high rejection rate,  It's very difficult to get extrinsic and instrinsic.
So that is my problem, if anyone can tell me how to  I get the extrinsics of pratical picture ? Thank you!

[Realsense Customer Engineering Team Comment]
Hi @anranx,
You use below configuration for D435 custom calibration, right?
1280x800, Y16 format, 15 FPS for Left/Right
1280x1080 YUY2 format, 15 FPS for RGB
From your picture, the IR emitter is on. Suggest to disable the IR emitter and you can use the environment light for the calibration. And, make sure auto exposure is on.
By the way, I temporarily have the access problem of the intel community link, so far, not able to see if any further info there.


Cloned rooms in rooms list

I don't know how to reproduce it but it happens sometimes when I resume the phone after several hours. (When I unlock it the first time the morning).
For the moment it only happened on encrypted rooms.

The issue is related to https://github.com/vector-im/riot-ios/issues/913 and this comment https://github.com/vector-im/riot-ios/issues/913#issuecomment-271942945
At first, in this scenario, the first sent message failed to be decrypted by the room data source. But when the room list updates itself, it will attempt to decrypt the event. The decryption will be successful this time and will send a kMXEventDidDecryptNotification which will refresh the room data source which will refresh the room list which is already being refreshed. Here is the race.


WIP: Approvers Category UI

For

https://github.com/ProfessionalWiki/PageApprovals/issues/4

@waximabbax the user ID should not be shown. MediaWiki lists user names all over the place and has standard HTML and CSS classes. We should not deviate from this unless there is good reason to do so.

@waximabbax the user ID should not be shown. MediaWiki lists user names all over the place and has standard HTML and CSS classes. We should not deviate from this unless there is good reason to do so.

Yes. Makes sense. I wasn't planning to show userId since the username does the job anyway. I'llclean up.

Since you are doing the post request to the special page for modifying the categories of an approver, what do you have in mind for adding new approvers?

Since you are doing the post request to the special page for modifying the categories of an approver, what do you have in mind for adding new approvers?

So for testing purposes in this PR, I am simply creating a User Group "Approvers" and anyone added to it is an approver.

If this doesn't work we already have an issue to deal with this.
Add approver administration right

Admin can now add Approvers: Here is a manual test:
https://github.com/ProfessionalWiki/PageApprovals/assets/15813104/7a9b4849-3518-4ea6-b640-0aa6fa14add7


Add better support for finding Ant

Searches through the PATH variable looking for the ant executable. It will fail if it is not found.
On Windows, it looks for ant.exe, ant.bat or ant, on Unix just ant.
Closes #730.

Closing due to not working outside Mac and Windows.. need to investigate a bit.


Markdown messages can now be rendered and added footers to dialogues

To owner: You may need to correct author and url inside manifest.json and manifest-beta.json before merging
Changes

Added center participant

c: or center: for name of centered participant

Leading and trailing whitespace are now trimmed
Added footer content

Use the :: prefix after a dialogue to add a footer (see README)
Default footer content can be modified with new parameters leftFooter, rightFooter, and centerFooter
Use footerMode to disable or enable default footers (see README)

Render markdown in messages

New parameter renderMarkdownTitle for rendering markdown in the title
New parameter renderMarkdownContent for rendering markdown in the content
New parameter renderMarkdownFooter for rendering markdown in the footer
New parameter renderMarkdownComment for rendering markdown in the comment

Hide or show unparsable text with clean parameter

Install
Until this gets merged, you may use BRAT to install the plugin from my repo.
Use this link with BRAT: https://github.com/twiddli/obsidian-dialogue-plugin
Closes #11
Closes #6
Closes #5
Closes #2

@holubj still no new release with new changes. Can you do this please?
I really like the plugin and would love to see some updates.


Ability to install via composer

Right now there's no way to install SDK by using composer, which is basically a norm right now in PHP world. I have to use hack which is to manually define package in my composer file. See this for reference: https://getcomposer.org/doc/05-repositories.md#package-2.
It would be much better to be able to just run:
composer require tpay-com/tpay-php

And have the package installed automatically. See how your competition does it: composer.json. 😜

there's a way but if admins will be changing whole lib without notice this is pointless. Now I have to refactor bunch of code -.-
to use composer simply do:
"autoload": { 		"psr-4": { 			"": "src/" 		}, 		"files": [ 			"vendor/tpay-com/tpay-php/src/_class_tpay/paymentBasic.php", 			"vendor/tpay-com/tpay-php/src/_class_tpay/lang.php" 		] 	}, 	"repositories": { 		"tpay-com/tpay-php": { 			"type": "package", 			"package": { 				"name": "tpay-com/tpay-php", 				"version": "1", 				"source": { 					"url": "https://github.com/tpay-com/tpay-php", 					"type": "git", 					"reference": "master" 				} 			} 		} 	},
and then in require section:
"tpay-com/tpay-php": "*",

Thanks for the reply! Yes, that's exactly what I'm doing right now — defining local package in my composer.json. I just thought that it would be good idea to add it to this repository, so other developers don't need to spent time and do it manually like I had to.

I don't understand what's the problem here. The point of creating public repository with library to integrate your payment system with your client's application is to ease this process, right? Thanks to such library clients don't need to go through the docs and manually write code to handle each request & response to the payment system. It saves time and maybe even is increasing number of your clients thanks to easy process of integrating your service with their app.
If you're providing the library like this — do it right. Otherwise, you're just testing patience of the developers by serving half-baked & not-so-tested-it-seems library.

Thanks for the reply! Yes, that's exactly what I'm doing right now — defining local package in my composer.json. I just thought that it would be good idea to add it to this repository, so other developers don't need to spent time and do it manually like I had to.

I don't understand what's the problem here. The point of creating public repository with library to integrate your payment system with your client's application is to ease this process, right? Thanks to such library clients don't need to go through the docs and manually write code to handle each request & response to the payment system. It saves time and maybe even is increasing number of your clients thanks to easy process of integrating your service with their app.
If you're providing the library like this — do it right. Otherwise, you're just testing patience of the developers by serving half-baked & not-so-tested-it-seems library.

there's a way but if admins will be changing whole lib without notice this is pointless.

Why would anyone want to modify code of library if it's well designed and easily configurable through its API?

@Anatejms Sorry for this first comment! I have misunderstood it and took you as one of the maintainers of this library. 🙃
Yes, and that's exactly the problem with lack of versioning and composer support. You cannot just lock version of the library at some specific one and be safe. With manually created approach you're always at risk that maintainers will make some breaking changes and break your application.

Your issue has been resolved.
Please type the following commands:
composer update
composer require tpay-com/tpay-php


bundle version v1.16.1 prints strange "fatal: Not a git repository ...' message

I see strange 'Not a git repo' messages when running bundle version command. All other commands (e.g. bundle install) do not print this message.
Terminal session:
$ \bundle version
fatal: Not a git repository (or any of the parent directories): .git
Bundler version 1.16.1 (2018-03-23 commit )
$ \bundle install
Using rake 12.3.1
Using concurrent-ruby 1.0.5
Using i18n 0.9.5
... omitted
$ which bundle
/Users/laise/.rvm/gems/ruby-2.5.0@my-project/bin/bundle

RVM: rvm 1.29.3 (master)
Ruby: ruby 2.5.0p0 (2017-12-25 revision 61468) [x86_64-darwin17]

The source of the problem is this line which calls BuildMetadata.git_commit_sha. But the gem folder has no .git dir.

I found that packaged Bundler gem (bundler-1.16.1.gem) simply doesn't include .git dir. So I think that bundle version should avoid to try to extract any git-related info (BuildMetadata.git_commit_sha).

Thanks for catching this! We definitely shouldn't print that message if there is no .git directory.

The released version (1.16.1) has the following (which wont call git at all, since the ivar is already set)
# frozen_string_literal: true

module Bundler
  # Represents metadata from when the Bundler gem was built.
  module BuildMetadata
    # begin ivars
    @built_at = "2017-12-21".freeze
    @git_commit_sha = "0034ef341".freeze
    @release = true
    # end ivars

    # A hash representation of the build metadata.
    def self.to_h
      {
        "Built At" => built_at,
        "Git SHA" => git_commit_sha,
        "Released Version" => release?,
      }
    end

    # A string representing the date the bundler gem was built.
    def self.built_at
      @built_at ||= Time.now.utc.strftime("%Y-%m-%d").freeze
    end

    # The SHA for the git commit the bundler gem was built from.
    def self.git_commit_sha
      @git_commit_sha ||= Dir.chdir(File.expand_path("..", __FILE__)) do
        `git rev-parse --short HEAD`.strip.freeze
      end
    end

    # Whether this is an official release build of Bundler.
    def self.release?
      @release
    end
  end
end

@segiddins You're right! I debugged a little and found that my ruby 2.5.0 installation already have a copy of the Bundler 1.16.1 and this "preinstalled" Bundler is used by bundle version command instead of the one I installed manually (using gem install bundler).
Preinstalled Bundler path is:
~/.rvm/rubies/ruby-2.5.0/lib/ruby/site_ruby/2.5.0/bundler
Let's see on the BuildMetadata class from there:
# ~/.rvm/rubies/ruby-2.5.0/lib/ruby/site_ruby/2.5.0/bundler/build_metadata.rb
module Bundler
  # Represents metadata from when the Bundler gem was built.
  module BuildMetadata
    # begin ivars
    @release = false
    # end ivars
    ...

No ivars listed!
I think that this is because RubyGems repo uses a plain copy of the Bundler in its own source code. At the time of writing it has bundler @ 0034ef3 submodule (see rubygems repo).
I guess that metadata ivars are only injected into packaged version of Bundler. This approach is not compatible with using Bundler source code as is (like RubyGems do).

I was doing a quick trawl of old issues this morning and saw this. I've created https://github.com/bundler/bundler/pull/6664 to stop it happening with future releases.


Only the first request are displayed in the pinpoint web ui

I am new to pinpoint.  Today I have met a strange issue, I have added pinpoint to my spring boot application,here is my startup cmd:
java -javaagent:/Users/zack/Downloads/pinpoint-agent-1.6.2/pinpoint-bootstrap-1.6.2.jar  -Dpinpoint.agentId=vm11 -Dpinpoint.applicationName=pinpointtest -jar demo-0.0.1-SNAPSHOT.jar
When I call the api the first time ,I can see the request number increased, but I call it again, the number keeps the same.
The interesting thing is when I stop the application, and restart it again, the first request is recorded again, but the rest keeps not recorded. Anyone met the same issue before? Thank you!!

I have found the root cause, profiler.sampling.rate is set  to 20 in the pinpoint.config, I should increase


Performance improvements in SHAPInterpretation

I removed link keyword argument as it wasn't used and was preventing to pass this keyword to SamplingExplainer. T
The new max_train_samples limits train data. Example: Rossmann dataset with 800k rows needs almost 24h to compute 128 shap values. However, if you limit to 100k (default value) takes 82s per test sample (around 2h30) and 10k rows takes 8,2s per test sample (around 20min).

That does sound much more reasonable, thanks!


Feature/chap 256 update select component

@smenz-any Really cool implementation of Select but why not use an existing react library that we just wrap with our needs  like react-select (so we not reinvent the wheel)?
On another hand it's not a good practice to manipulate the DOM directly in a React component (ex: manipulate the class). Better would be to manipulate the className in the JSX 👍


only first name server attempted in finding the puppet server

Describe the Bug
I recently noticed all my puppet agents were failing when the primary DNS resolver was out. This was on FreeBSD.
Expected Behavior
I would expect that puppet would use the next nameserver in the list to look up the address of the puppet server.
Steps to Reproduce
Steps to reproduce the behavior:

Configure a host with two resolvers, the first one nonfunctional
Run the agent, I used puppet agent -t -v --logdest console
observe an error similar to the following

puppet-agent[29201]: Could not retrieve catalog from remote server: Host is down - sendto(2) for "<IP_ADDRESS>" port 53
Environment

Version 8.7.0
Platform FreeBSD 14.1

Additional Context
Add any other context about the problem here.

@rtprio can you provide more details about how you configured a "host with two resolvers, the first one nonfunctional"?

sure thing;
the resolv.conf of the host was like this:
nameserver <IP_ADDRESS>
nameserver <IP_ADDRESS>

In this case, .2 crashed and was no longer answering DNS. These hosts only have ipv4 at the moment, I have not tested the ipv6 side of this condition.

I suppose I should have investigated deeper when it was happening; It seems to be properly using the next nameserver now. 🤷

Thanks for letting us know. I'm going to close this, but if you can reproduce feel free to reopen.