isReusable should return a constant value

isReusable should return a constant value.
Or  ClickHouseHttpClient may close on flying connection when creating new connection.
        if (connection != null && connection.isReusable()) {
closeConnection(connection, false);
}

When connection is on fly by a request  the others wait it released?

Thanks for testing and raising the PR. ClickHouseHttpConnection.isReusable() suggests if the connection instance can be reused among requests. Each time when there's a new request made to the same server, we have two cases to deal with:

reuse the same connection if it's not busy on something else
create a new connection so that we can process requests in parallel

The change I made was for the latter one, which also fixed the test failures. Yes, it may cause connection leak but I thought the connections are all managed by Apache connection manager. Did you run into memory issue using latest code on develop branch? Will run stress test tonight to see how it goes.

@zhicwu We can use PoolingHttpClientConnectionManager and keep isReusable return true in which way connections will be managed by Apache connection manager.

Thank you @JackyWoo, the changes look good. I'm closing this PR as the changes are now merged into #1174, where I cleaned up code for consistency and improved compression/decompression support.


Downsampling of reads

We are using Biodalliance genome browser with high-coverage bam-files (up to 10,000 reads per base pair).
By default, the limit of reads to be displayed is set to 100. (And there has to be a limit, because it gets terribly slow, if not)
Problem is, the genome browser seems to just take the first 100 reads then. In a recent case, there was not one singe read displayed for the locus in question, but only reads which started right from the current position. In other cases you may have only wildtype-reads being displayed while the mutated ones get clipped.
Could you implement some form of statistical downsampling? e.g. selecting the reads to be displayed per random? Or just taking every 100th read or something like that?

There isn't a built in option for this, but if you configure your tracks programatically, you can do this via a plugin.
Something like:
function readDownsampler(featureSets) {
    const reads = featureSets[0];
    const sampledReads = [];
    for (var i = 0; i < reads.length; i += 10) {
         sampledReads.push(reads[i]);
    }
    return sampledReads;
}

...then configure your source with...
{
    name: 'Downsample test',
    bamURI: '/path/to/data.bam',
    merge: readDownsampler
}```

A slight downside is that if you return to almost but not quite the same region of the genome, you'll end up seeing a different subset of the reads.  If this matters, it might be better to sample based on,  e.g., MD5 of the read ID instead.

Having said all that, I think it's a great idea to have something along these lines this built into the core -- so will leave this issue open for now.

Thank you for this great solution!!
Unfortunately something goes wrong here...
When I copy this code exactly as described here, the function readDownsampler never gets called (I inserted some debug log output - which never gets printed).
When I write it with brackets, i.e. merge: readDownsampler(), then the function gets called, but featureSets is undefined.
What to do about this issue?

And another question: Is it possible to access the user-defined variable limit from within this function?
With this the downsampling could be adapted to the limit of reads to be shown, as defined by the user in the config dialog.

Sorry for the confusion -- the example I sent is something that really ought to work, but currently doesn't because of the way two features (combining multiple data sources in one track, and applying arbitrary filters to data) are coupled together.
The following version is actually tested :-)
{
    name: 'Downsample test',
    overlay: [{bamURI: '/path/to/data.bam'}],
    merge: readDownsampler
}```

(The readDownsampler function itself is fine).  I'm going to tweak things so that the example as I originally wrote it does actually work -- but might not happen right away.

Great! This solution works indeed for filtering the read data.
But... sorry that I have to ask questions again...
We have use this in combination this with
(a) a bam index file (bai)
(b) a style sheet configuration that enables "Highlight mismatches and strands" by default
(c) a readDownsampler() function that considers the user configured read limit
I could not figure out by now, where to correctly place the style sheet information in order to work correctly with the overlay command. When we put it below the merge command, the checkbox "Highlight mismatches and strands" gets checked, but the corresponding style seems not to be applied. Even with manually unchecking and re-checking the checkbox, the style gets not applied.
Do you have an idea, how to solve this?
Additionally, is there a possibility to read out the user configured read limit in order to use it in the downsampling-function?

Re: mismatch colouring...
Thanks for spotting this.  It sounds like your config is fine, but some logic that's used to determine whether reference sequence data needs to be threaded through to a given track's renderer was failing when your custom filter was applied.  This has been fixed in the git-latest version.
Re: user-configurability of the the custom filter.
Do you want to be able to configure this at run time (via a custom field in the track editor).  Currently no way of doing this, but I'd certainly agree it would be nice!

Thanks again!
Re: mismatch colouring...
After having built the latest version from git, the mismatch colouring actually works fine.
BUT, there has appeared some new issue with the latest version...
The "cursor" indicating the middle position gets not correctly positioned after applying the API function SetLocation().
Furthermore, the cursor "jumps" back and forward, when the user opens the configuration dialog. That is kind of ... weird.
Re: custom filter
Actually I do not really need to let the user configure the read downsampling limit in the track editor.
BUT, there already IS a field "limit" in the track editor, which is pre-configured with 100, and which will be  out-of-function, if we cannot read out its value from the downsampling function.
Furthermore, the limit of 100 is not very suitable for us. If it is a hard limit that cannot be changed by the user anymore, then it would be nice, if we could at least pre-configure it with a higher value, like 500.
Does there exist a style-option for changing this value?

I'm concerned about what you say regarding the "cursor" (do you mean the vertical position indicator in the middle of the browser area).  Could you send a screenshot or two to illustrate this (offline to<EMAIL_ADDRESS>is fine if you prefer).
Regarding the "bumping limit", it can be configured as a top level (not stylesheet) option on a track configuration:
        {
               name: 'my track',
               bamURI: '...',
               subtierMax: 500
         }


WindowsAppSDK-VersionInfo.cs and MddBootstrapAutoInitializer.cs cause warnings in analyzers

Describe the bug
Setting <GenerateDocumentationFile>True</GenerateDocumentationFile> in the project causes the following warnings to appear:

Another issue is reported here:
DotNetAnalyzers/StyleCopAnalyzers/#3441
Steps to reproduce the bug
1.) Create a new WinUI 3 app with the template: Blank App, Packaged (WinUI 3 in Desktop)
2.) Enable documentation generation
3.) Build the app
Expected behavior
Warnings from nuget packages should not appear in any project that uses them.
Screenshots
No response
NuGet package version
1.0.0
Packaging type
Packaged (MSIX), Unpackaged
Windows version
Windows 10 version 2004 (19041, May 2020 Update)
IDE
Visual Studio 2022
Additional context
No response

This bug still exists in v1.0.1

In version 1.1.0-preview2 I don't get warnings but I do get several messages. I think this is caused by me having many of my VS code style settings set to "Suggestion" instead of "Refactoring Only". Is there a way to exclude this file from being checked against the VS code style settings (ie. Tools | options | Text Editor | C# | Code Style)

Still not fixed in v1.1.0

@DrusTheAxe It appears that there are additional StyleCop warnings that aren't covered by #2533; is that expected?

Still not fixed in v1.1.2

I'll be looking at it next week

Still not fixed in v1.1.3

I did some digging...
Workaround here => https://github.com/dotnet/roslyn/issues/53729#issuecomment-850016824
Official word is it's a feature => https://github.com/dotnet/roslyn/issues/53729#issuecomment-879400025

Closing because this behavior is "By Design" and I don't really see us changing it here. The compiler itself has very little understanding of generated code in terms of how it handles diagnostics. That is true for the types of files that we register as generated as well as the behavior we tweak for those files (think it's pretty much just nullable diagnostics at this point).
The solution here is to either generate the correct code or to disable the warning in editorconfig for that pattern of files (as you showed in the sample above)

Another discussion with the full menu of options => https://stackoverflow.com/questions/7982525/visual-studio-disabling-missing-xml-comment-warning

To summarise, the ways around the warning would be to:

Suppress the warning by changing the project Properties > Build > Errors and warnings > Suppress warnings by entering 1591
Add the XML documentation tags (GhostDoc can be quite handy for that)
Suppress the warning via compiler options
Uncheck the "XML documentation file" checkbox in project Properties > Build > Output
Add #pragma warning disable 1591 at the top of the respective file and #pragma warning restore 1591 at the bottom

and a little later in that thread a C# specific answer which looks most promising

You can also modify your project's .csproj file to include a 1591 tag inside of the first . Originally from Alexandru Bucur's Article Here

    ...
   1591

 ...

@DefaultRyan is it possible to declare that <PropertyGroup> in one of our nuget's .props to address this?

How about you just document the stuff...

We do e.g. https://docs.microsoft.com/en-us/windows/apps/api-reference/cs-bootstrapper-apis/cs-bootstrapper-apis-constants.
This isn't about documenting our stuff, but a matter of how, or more precisely, where.
Is it just the warning noise that's an issue? Does having doccomments in source bring any additional value (or issues)? Just for files like MddBootstrapAutoInitialize.cs that are compiled into your binaries or others as well e.g. WinAPI APIs available to C# developers via C#/WinRT like `Microsoft.Windows.ApplicationModel.DynamicDependency'?

Imho it is noise. Noise that should not be there as i as developer have no control over those files. I should not have to do anything to work around those warnings.

Imho it is noise.

Gotcha. Thanks for the input.

I should not have to do anything to work around those warnings.

No disagreement, just trying to figure out the needs and best ways to squash the issue. I have a simple fix for the surface issue and we're discussing if a bigger change is warranted for the larger picture. I've teed up a patch for main and will bring it for 1.1-servicing consideration for our next servicing update.

I'm using:
[*WindowsAppSDK-VersionInfo.cs] # Workaround for https://github.com/microsoft/WindowsAppSDK/issues/2032
dotnet_diagnostic.SA1201.severity = none
dotnet_diagnostic.SA1402.severity = none
dotnet_diagnostic.SA1403.severity = none
dotnet_diagnostic.SA1633.severity = none
dotnet_diagnostic.SA1649.severity = none

I can see it too, also on MAUI.

WindowsAppSDK-VersionInfo.cs was missed in the pass previously fixing this. in other files.
That's generated out of another repo so I'll cook a fix over there. Thanks for reporting this.

I'm still seeing this. Maybe because  isn't on the very first line?


Dockerimage fails to build

unable to build docker image because of some missing dependencies
2.283 Get:6 http://deb.debian.org/debian buster-updates/main amd64 Packages [8788 B]
3.157 Fetched 8700 kB in 3s (2839 kB/s)
3.157 Reading package lists...
3.662 Reading package lists...
4.186 Building dependency tree...
4.326 Reading state information...
4.378 libpq-dev is already the newest version (11.16-0+deb10u1).
4.378 Some packages could not be installed. This may mean that you have
4.378 requested an impossible situation or if you are using the unstable
4.378 distribution that some required packages have not yet been created
4.378 or been moved out of Incoming.
4.378 The following information may help to resolve the situation:
4.378
4.378 The following packages have unmet dependencies:
4.431  dnsutils : Depends: libbind9-161 (= 1:9.11.5.P4+dfsg-5.1+deb10u7) but 1:9.11.5.P4+dfsg-5.1+deb10u9 is to be installed
4.431             Depends: libdns1104 (= 1:9.11.5.P4+dfsg-5.1+deb10u7) but 1:9.11.5.P4+dfsg-5.1+deb10u9 is to be installed
4.431             Depends: libisc1100 (= 1:9.11.5.P4+dfsg-5.1+deb10u7) but 1:9.11.5.P4+dfsg-5.1+deb10u9 is to be installed
4.431             Depends: libisccfg163 (= 1:9.11.5.P4+dfsg-5.1+deb10u7) but 1:9.11.5.P4+dfsg-5.1+deb10u9 is to be installed
4.431             Depends: liblwres161 (= 1:9.11.5.P4+dfsg-5.1+deb10u7) but 1:9.11.5.P4+dfsg-5.1+deb10u9 is to be installed
4.442 E: Unable to correct problems, you have held broken packages.
------
Dockerfile:8
--------------------
   7 |     # dependencies for psycopg2
   8 | >>> RUN apt-get update && apt-get install --no-install-recommends -y dnsutils=1:9.11.5.P4+dfsg-5.1+deb10u7 libpq-dev=11.16-0+deb10u1 python3-dev=3.7.3-1 \
   9 | >>>  && apt-get clean \
  10 | >>>  && rm -rf /var/lib/apt/lists/*
  11 |
--------------------
error: failed to solve: process "/bin/sh -c apt-get update && apt-get install --no-install-recommends -y dnsutils=1:9.11.5.P4+dfsg-5.1+deb10u7 libpq-dev=11.16-0+deb10u1 python3-dev=3.7.3-1  && apt-get clean  && rm -rf /var/lib/apt/lists/*" did not complete successfully: exit code: 100

Looks like its not able to install neither dnsutils or libpq-dev in versions selected in dockerfile.
I did quick check and I am able to build image without specifying particular versions of this two packages, but not quite sure does it "broke" any functionalities

same here, is this fixed yet?? what branch/tag is working??

still working to fixing

Any update on this or a temporary fix?

Still fails to build:
 => ERROR [3/9] RUN apt-get update && apt-get install --no-install-recommends -y dnsutils=1:9.11.5.P4+dfsg-5.1+deb10u7 libpq-dev=11.16-0+deb10u1 python3-dev=3.7.3-1  && apt-get clean  && rm -rf /var/lib/apt/lists/*                                                               4.5s
------
 > [3/9] RUN apt-get update && apt-get install --no-install-recommends -y dnsutils=1:9.11.5.P4+dfsg-5.1+deb10u7 libpq-dev=11.16-0+deb10u1 python3-dev=3.7.3-1  && apt-get clean  && rm -rf /var/lib/apt/lists/*:
0.623 Get:1 http://deb.debian.org/debian buster InRelease [122 kB]
0.635 Get:2 http://security.debian.org/debian-security buster/updates InRelease [34.8 kB]
0.898 Get:3 http://security.debian.org/debian-security buster/updates/main amd64 Packages [598 kB]
0.902 Get:4 http://deb.debian.org/debian buster-updates InRelease [56.6 kB]
1.032 Get:5 http://deb.debian.org/debian buster/main amd64 Packages [7909 kB]
1.830 Get:6 http://deb.debian.org/debian buster-updates/main amd64 Packages [8788 B]
2.850 Fetched 8729 kB in 3s (3268 kB/s)
2.850 Reading package lists...
3.478 Reading package lists...
4.165 Building dependency tree...
4.303 Reading state information...
4.356 libpq-dev is already the newest version (11.16-0+deb10u1).
4.356 Some packages could not be installed. This may mean that you have
4.356 requested an impossible situation or if you are using the unstable
4.356 distribution that some required packages have not yet been created
4.356 or been moved out of Incoming.
4.356 The following information may help to resolve the situation:
4.356
4.356 The following packages have unmet dependencies:
4.411  dnsutils : Depends: libbind9-161 (= 1:9.11.5.P4+dfsg-5.1+deb10u7) but 1:9.11.5.P4+dfsg-5.1+deb10u11 is to be installed
4.411             Depends: libdns1104 (= 1:9.11.5.P4+dfsg-5.1+deb10u7) but 1:9.11.5.P4+dfsg-5.1+deb10u11 is to be installed
4.411             Depends: libisc1100 (= 1:9.11.5.P4+dfsg-5.1+deb10u7) but 1:9.11.5.P4+dfsg-5.1+deb10u11 is to be installed
4.411             Depends: libisccfg163 (= 1:9.11.5.P4+dfsg-5.1+deb10u7) but 1:9.11.5.P4+dfsg-5.1+deb10u11 is to be installed
4.411             Depends: liblwres161 (= 1:9.11.5.P4+dfsg-5.1+deb10u7) but 1:9.11.5.P4+dfsg-5.1+deb10u11 is to be installed
4.423 E: Unable to correct problems, you have held broken packages.
------
Dockerfile:8
--------------------
   7 |     # dependencies for psycopg2
   8 | >>> RUN apt-get update && apt-get install --no-install-recommends -y dnsutils=1:9.11.5.P4+dfsg-5.1+deb10u7 libpq-dev=11.16-0+deb10u1 python3-dev=3.7.3-1 \
   9 | >>>  && apt-get clean \
  10 | >>>  && rm -rf /var/lib/apt/lists/*
  11 |
--------------------
ERROR: failed to solve: process "/bin/sh -c apt-get update && apt-get install --no-install-recommends -y dnsutils=1:9.11.5.P4+dfsg-5.1+deb10u7 libpq-dev=11.16-0+deb10u1 python3-dev=3.7.3-1  && apt-get clean  && rm -rf /var/lib/apt/lists/*" did not complete successfully: exit code: 100

Removing the version numbers from the apt-get install does get the image to build, but the image doesn't start. It appears it needs gunicorn but that was removed some time ago: https://github.com/adeyosemanputra/pygoat/commit/5ae6b4d28ef3fdcd8244e53e397384cdd820fd5c .

hello dears,
I've been having the same problem here since last year. I really would like to have help with that because would be very useful in my studies.
Thanks

@lucasferreira94 why you till using docker? can you install with pip or other?

@lucasferreira94 why you till using docker? can you install with pip or other?

@adeyosemanputra I was setting up in a Lab to use with Kind K8s Cluster with other vulnerable apps. And by docker is easier to set up and maintain

you can try install by see this video
https://youtu.be/a5UV7mUw580?si=rkWL6tl65T-qNP4A

@lucasferreira94 there is any guide for you

same here, is this fixed yet?? what branch/tag is working??


Detection by TikTok

Whenever I post with this tool on TikTok, my video gets 0 views (stuck in fantom mode), whatever the video, whatever the account.
(I also tested with some apps that allow to post automatically and they also got stuck on 0 views.)
Am I the only one who doesn't have view when I use this tool?
I think Tiktok has some verification that you're using the Tiktok app/website and not just the API.
Maybe try to replicate some requests that are made like these ones.

Here's a guy that I found on GitHub who explored a bit the Tiktok API: https://github.com/xtekky?language=&page=1&q=Tiktok&sort=stargazers&tab=repositories&type=source .
Maybe we will find something interesting to improve our uploader.

all the api this script use is based on TikTok Website.
this is account, all the videos upload by this script

me too i tested the script but the videos gets 0 vues im asking if i can use proxy or i must every time change the rdp where im hosting the script or there is anything i must do it like undetectable browser...

all the api this script use is based on TikTok Website. this is account, all the videos upload by this script

Oh okay, amazing for you! Do you use any IP or MAC adress modification before launching the script ? Or you just launch it normally like any script with your home internet connexion ? Or maybe the account that you're using, is it aged ?
We are having the same problem with @bilalazza and you don't have it @546200350 so it must have a solution 😄

i want run it into another rdp but i get this error
user_id = r.json()["data"]["user_id_str"]
KeyError: 'user_id_str'
can you please advice me . i look into the folder there is no json file  what can i do please

Update :
As @546200350 is in Chengu (China) he used the singapore server to upload. Whereas I'm in France, so I did set up a proxy with Charles for my smartphone to see which server i'm using when on Tiktok.
As result I found this url : tnc16-platform-useast1a.tiktokv.com .
So for the uploader I changed all the url = "https://vod-ap-singapore-1.bytevcloudapi.com/" by
url = "https://vod-us-east-1.bytevcloudapi.com/" (and some changes in the AWS signature function) and I it managed to upload.
I'll keep you updated if this works or still locked with 0 views.

@UnMars did the same, but still getting 0 views if video uploaded by tool

@ViktorKravtsov Any solutions found ?

@UnMars no :(

Update :
As @546200350 is in Chengu (China) he used the singapore server to upload. Whereas I'm in France, so I did set up a proxy with Charles for my smartphone to see which server i'm using when on Tiktok.
As result I found this url : tnc16-platform-useast1a.tiktokv.com .
So for the uploader I changed all the url = "https://vod-ap-singapore-1.bytevcloudapi.com/" by
url = "https://vod-us-east-1.bytevcloudapi.com/" (and some changes in the AWS signature function) and I it managed to upload.
I'll keep you updated if this works or still locked with 0 views.

Are your modifications open sourced?

Update :
As @546200350 is in Chengu (China) he used the singapore server to upload. Whereas I'm in France, so I did set up a proxy with Charles for my smartphone to see which server i'm using when on Tiktok.
As result I found this url : tnc16-platform-useast1a.tiktokv.com .
So for the uploader I changed all the url = "https://vod-ap-singapore-1.bytevcloudapi.com/" by
url = "https://vod-us-east-1.bytevcloudapi.com/" (and some changes in the AWS signature function) and I it managed to upload.
I'll keep you updated if this works or still locked with 0 views.

Are your modifications open sourced?

Yes I mean I just replace the server url everywhere in the code as I said. If you have any solutions, I take it 😄

Update :
As @546200350 is in Chengu (China) he used the singapore server to upload. Whereas I'm in France, so I did set up a proxy with Charles for my smartphone to see which server i'm using when on Tiktok.
As result I found this url : tnc16-platform-useast1a.tiktokv.com .
So for the uploader I changed all the url = "https://vod-ap-singapore-1.bytevcloudapi.com/" by
url = "https://vod-us-east-1.bytevcloudapi.com/" (and some changes in the AWS signature function) and I it managed to upload.
I'll keep you updated if this works or still locked with 0 views.

Are your modifications open sourced?

Yes I mean I just replace the server url everywhere in the code as I said. If you have any solutions, I take it 😄

Do you have a discord so we can discuss easily?

Update :
As @546200350 is in Chengu (China) he used the singapore server to upload. Whereas I'm in France, so I did set up a proxy with Charles for my smartphone to see which server i'm using when on Tiktok.
As result I found this url : tnc16-platform-useast1a.tiktokv.com .
So for the uploader I changed all the url = "https://vod-ap-singapore-1.bytevcloudapi.com/" by
url = "https://vod-us-east-1.bytevcloudapi.com/" (and some changes in the AWS signature function) and I it managed to upload.
I'll keep you updated if this works or still locked with 0 views.

Are your modifications open sourced?

Yes I mean I just replace the server url everywhere in the code as I said. If you have any solutions, I take it 😄

Do you have a discord so we can discuss easily?

Here's UnMars#2668

I get between 70k and 200k views a day with this, so I don’t think there’s any detection by tiktok tbh

@TheTTCoder are you from China? Or you modified this script somehow?

@TheTTCoder are you from China? Or you modified this script somehow?

I’m from France


Compute 'A', 'D', 'U' matrices

Hello! I want to try use other human template(SMPLX) in your work,  I need precompuute 'A', 'D', 'U' matrices according this , so I test using the smpl template first, but  I can't get the same result as you. Please point out my mistake, Thank you !
I changed part of the code of ‘coma’
from __future__ import print_function
import mesh_sampling
from psbody.mesh import Mesh, MeshViewer, MeshViewers
import numpy as np
import json
import os
import copy
import argparse
import pickle
import time

print("Loading data .. ")
reference_mesh_file = 'data/smpl_neutral_vertices.obj'
reference_mesh = Mesh(filename=reference_mesh_file)

ds_factors = [4,1]	# Sampling factor of the mesh at each stage of sampling
print("Generating Transform Matrices ..")

# Generates adjecency matrices A, downsampling matrices D, and upsamling matrices U by sampling
# the mesh 4 times. Each time the mesh is sampled by a factor of 4

M,A,D,U = mesh_sampling.generate_transform_matrices(reference_mesh, ds_factors)

print(type(A))

np.savez('mesh_downsampling_test.npz', A = A, D = D, U = U)

How different are the 2 results?

File size is different

There are also some differences in the content of the file,I output the contents of the file with the following code
import sys
import numpy as np
from psbody.mesh import Mesh

data = np.load('mesh_downsampling_test.npz',encoding='latin1')
print(type(data['A_test']))

np.savetxt('out_A_test.txt',data['A_test'],fmt='%s')
np.savetxt('out_D_test.txt',data['D_test'],fmt='%s')
np.savetxt('out_U_test.txt',data['U_test'],fmt='%s')
#np.savetxt('out_F.txt',data['F'],fmt='%s')
#np.savetxt('out_M.txt',data['M'],fmt='%s')

output.tar.gz

I'm sorry to disturb you. I hope you can give me a reply. Thank you very much

Hi @moondabaojian, can you try ds_factors = [4, 4] instead of [4, 1] and get back to me?  FYI, I used pytorch_coma to reproduce the work.
First of all, I checked the original mesh_downsampling.npz in GraphCMR/data and that is like following:
# default['A']
array([<6890x6890 sparse matrix of type '<class 'numpy.float64'>'
	with 41328 stored elements in Compressed Sparse Column format>,
       <1723x1723 sparse matrix of type '<class 'numpy.float64'>'
	with 10326 stored elements in Compressed Sparse Column format>,
       <431x431 sparse matrix of type '<class 'numpy.float64'>'
	with 2574 stored elements in Compressed Sparse Column format>],
      dtype=object)

# default['D']
array([<1723x6890 sparse matrix of type '<class 'numpy.float64'>'
	with 1723 stored elements in Compressed Sparse Column format>,
       <431x1723 sparse matrix of type '<class 'numpy.float64'>'
	with 431 stored elements in Compressed Sparse Column format>],
      dtype=object)

# default['U']
array([<6890x1723 sparse matrix of type '<class 'numpy.float64'>'
	with 20670 stored elements in Compressed Sparse Column format>,
       <1723x431 sparse matrix of type '<class 'numpy.float64'>'
	with 5169 stored elements in Compressed Sparse Column format>],
      dtype=object)

However, the new npz result after applying ds_factors = [4, 1]  to basicModel_neutral_lbs_10_207_0_v1.0.0.pkl that we can download via ./fetch_data.sh is different. I suppose this lead to the different size that you mentioned.
# template['A']
array([<6890x6890 sparse matrix of type '<class 'numpy.float64'>'
	with 41328 stored elements in COOrdinate format>,
       <1723x1723 sparse matrix of type '<class 'numpy.float64'>'
	with 10326 stored elements in COOrdinate format>,
       <1723x1723 sparse matrix of type '<class 'numpy.float64'>'
	with 10326 stored elements in COOrdinate format>], dtype=object)

# template['D']
array([<1723x6890 sparse matrix of type '<class 'numpy.float64'>'
	with 1723 stored elements in COOrdinate format>,
       <1723x1723 sparse matrix of type '<class 'numpy.float64'>'
	with 1723 stored elements in COOrdinate format>], dtype=object)

# template['U']
array([<6890x1723 sparse matrix of type '<class 'numpy.float64'>'
	with 20670 stored elements in COOrdinate format>,
       <1723x1723 sparse matrix of type '<class 'numpy.float64'>'
	with 5169 stored elements in COOrdinate format>], dtype=object)

Each A, D, and U matrix size is different from the original .npz, so that I try making it the same by using ds_factors = [4, 4]. This produces the same matrix size like the below result.
# test['A']
array([<6890x6890 sparse matrix of type '<class 'numpy.float64'>'
	with 41328 stored elements in COOrdinate format>,
       <1723x1723 sparse matrix of type '<class 'numpy.float64'>'
	with 10326 stored elements in COOrdinate format>,
       <431x431 sparse matrix of type '<class 'numpy.float64'>'
	with 2574 stored elements in COOrdinate format>], dtype=object)

# test['D']
array([<1723x6890 sparse matrix of type '<class 'numpy.float64'>'
	with 1723 stored elements in COOrdinate format>,
       <431x1723 sparse matrix of type '<class 'numpy.float64'>'
	with 431 stored elements in COOrdinate format>], dtype=object)

# test['U']
rray([<6890x1723 sparse matrix of type '<class 'numpy.float64'>'
	with 20670 stored elements in COOrdinate format>,
       <1723x431 sparse matrix of type '<class 'numpy.float64'>'
	with 5169 stored elements in COOrdinate format>], dtype=object)

However, the inside of matrix result is slightly different. This makes me not certain I did in a correct way and curious the sentence in the paper;

subsample by 4 factor and upsample it again the original scale.

The above sentence seems to mean [4, 1] but [4, 4] gives me similar result. If @nkolot can help us, it will be very nice.

Hello @syenpark , Thank you for your reply to this question. According to your suggestion, I tried using  ds_factors = [4, 4] , Here are my results

and
output.tar.gz
I still haven't got the same result as the original one mesh_downsampling.npz. But as you suggest, we have at least the same matrix structure

@moondabaojian, I suppose some format or header information of .npz leads to the slight different size that you point out. For instance, the discrepancy in the format such as Compressed Sparse Column format in the original and COOrdinate format in mine. This difference doesn't cause any issues when I run the pretrained model of this GitHub on my own mesh.


auro-pane: separate FIXED CSS

Is your feature request related to a problem? Please describe.
The recently released FIXED feature is creating a block of CSS that if not needed in a use case, is bloat to the bundle.
Describe the solution you'd like
Updating the fixed attribute to be a property that when used will load either a complete set of REM or FIXED PX CSS.
Describe alternatives you've considered
None
Additional context
This issue ticket is a conversation starter. See https://github.com/AlaskaAirlines/auro-pane/pull/3 for more details.

See https://github.com/AlaskaAirlines/WebCoreStyleSheets/issues/55 for notes.


Update README URLs based on HTTP redirects

Created with https://github.com/dkhamsing/frankenstein
GitHub Corrected URLs

Was
Now

https://github.com/stretchr/graceful
https://github.com/tylerb/graceful

HTTPS Corrected URLs

Was
Now

http://en.wikipedia.org/wiki/Second-system_effect
https://en.wikipedia.org/wiki/Second-system_effect

http://github.com/yvasiyarov/gorelic
https://github.com/yvasiyarov/gorelic

http://nathany.com/go-packages/
https://nathany.com/go-packages/

Other Corrected URLs

Was
Now

https://godoc.org/code.google.com/p/go.tools/cmd/goimports
https://godoc.org/golang.org/x/tools/cmd/goimports

+1

+1 This README is auto-generated, from various sources (READMEs from examples, ...) I'll update the sources, regenerate it and close the PR.


Allow searching near-by

If you travel abroad without changing your phone's locale, you will still see results from your home country.
For example, my phone is set in British English, so my search only show results in the UK, no matter where I am at the moment.
A good workaround would be to authorize the use of:
PlacesSearchResponse response = await places.searchNearbyWithRankby(new Location(31.0424, 42.421), "distance");
which is available in the https://pub.dev/packages/google_maps_webservice package.

Originally posted by @Dereumaux-Adrien in https://github.com/martin-braun/google_maps_place_picker_mb/issues/32#issuecomment-1384640201

Maybe an option is to offer a flag that allows you to search near-by? I assume the issue with searchNearbyWithRankby is that you cannot find stuff far away, so it shall be optional.

Originally posted by @martin-braun in https://github.com/martin-braun/google_maps_place_picker_mb/issues/32#issuecomment-1385560033

After looking more in details, I understood that using places.searchNearbyWithRankby would not work as it doesn't provide the same kind of output as the actually used places.autocomplete (respectively PlacesSearchResponse and PlacesAutocompleteResponse).
I also looked in the autocomplete documentation and found that there should be a way to make these with autocomplete as well, it mentions rankby=distance here but this one is weirdly not mentionned in the parameters. But the parameters' category locationbias seems to be offering many ways to control which results come first and are favored. These sadly don't seem to be readily available google_maps_service, they may have been added to places later on.
Not wanting to update all that by myself, I tried to "play" around with the values that are already available and sound alike.
I did some more digging and found that setting autocompleteRadius is a good temporary workaround not depending on the locale of the device. But it sadly doesn't order by proximity (For example autocompleteRadius:5000 will  first give me results less than 5000m away, but the top one might not be the closest one).
And then I tried lowering the value to see what happens if the closest result is much further than my minimum radius.
It seems that setting autocompleteRadius:5 orders the results by order of proximity and I guess that if many results are present less than 5 meters away from the user, they can be ordered in any way without it impacting the user.
I think that the most important at the moment is to add in the documentation that autocompleteRadius can be used to order the results by proximity. If you still want to add the locationbias (as I am not sure if rankby=distance is really possible) I could maybe guide you in the process for the google_maps_service side (I think I understood where the changes are needed).

@Dereumaux-Adrien Thanks for your analysis and great work so far, it helps to find the shortcut, as I'm currently rather maintaining than enhancing this library due to time constraints. autocompleteRadius seems to be sufficient for the problem and I think the caveat in regards of the lack of proximity ordering for the specified radius is not even a problem and rather a feature.
If I have a radius of 5km that I'm accepting to travel for the best destination, I want the best/largest/most known location of my search within that radius, not the closest one. If I want to travel the least possible, autocompleteRadius:1 should be the way to go.
If you feel confident, I would highly appreciate if you could update the README to document the property properly. Otherwise I will update it later myself.
Thanks again, brother.


WIP: bst fmt: Add basic functionality

See original merge request on GitLab
In GitLab by [Gitlab user @coldtom] on Nov 15, 2018, 16:55
Description
Adds a bst fmt command, which modifies the format of a projects element files into a canonical yaml format. This has the main benefit of automating files into the format bst track will dump them into when run.
Currently the functionality is somewhat stinted by #767, and there are a couple of other features I would like to add to this - namely an option which doesn't modify the yaml, but creates a diff, and a way for a user to define the order of top-level nodes in a .bst file, into which bst fmt sorts the nodes.
Changes proposed in this merge request:

Add a bst fmt command which formats elements into a canonical format

This merge request,  when approved, will close #485

In GitLab by [Gitlab user @jjardon] on Nov 15, 2018, 17:14
changed the description

In GitLab by [Gitlab user @coldtom] on Nov 15, 2018, 17:59
added 1 commit
1d2d0de4 - bst fmt: Add basic functionality
Compare with previous version

In GitLab by [Gitlab user @knownexus] on Nov 19, 2018, 14:21
Commented on buildstream/_frontend/cli.py line 374
Shouldn't there be a requirement for --all to be enabled before we allow --except?
I suppose it doesn't break anything if you do it without, but it seems to have no value in other cases from what i can see

In GitLab by [Gitlab user @knownexus] on Nov 19, 2018, 14:25
Commented on buildstream/_scheduler/queues/formatqueue.py line 12
Is this called anywhere? I can't see it but i may be missing it

In GitLab by [Gitlab user @coldtom] on Nov 21, 2018, 12:56
Commented on buildstream/_scheduler/queues/formatqueue.py line 12
As I understand it this is where one implements the process performed by the queue, and is called by the scheduler. It's an abstract method in the Queue class. However I was wrong to give a return value for the feature implemented here, I'll fix that presently.

In GitLab by [Gitlab user @coldtom] on Nov 21, 2018, 13:47
Commented on buildstream/_frontend/cli.py line 374
Other elements with similar functionality use the same method of implementing --except, personally I'm of the opinion that adding a guard against it would make the code more awkward for little benefit, but I'm not opposed to it.

In GitLab by [Gitlab user @tlater] on Nov 21, 2018, 16:16
Commented on buildstream/_scheduler/queues/formatqueue.py line 12
[Gitlab user @knownexus] it's part of the Queue API here. It's what keeps us from creating new job types for every queue type.

In GitLab by [Gitlab user @tlater] on Nov 21, 2018, 16:19
Commented on buildstream/_stream.py line 297
While I understand that you're just reusing code here, I'd prefer if you changed the name of the kwarg to do so. Perhaps modify_selection or somesuch?

In GitLab by [Gitlab user @tlater] on Nov 21, 2018, 16:26
Commented on buildstream/_frontend/cli.py line 374
I think that the help string should at least mention that it has no effect unelss --all is specified. Otherwise this is probably fine.

In GitLab by [Gitlab user @tlater] on Nov 21, 2018, 16:27
Commented on buildstream/_scheduler/queues/formatqueue.py line 15
I'd be pretty impressed if you could do this. Any suggestions on how this would be implemented? Surely you'd run into the halting problem here.

In GitLab by [Gitlab user @tlater] on Nov 21, 2018, 16:38
So, the ML was pretty happy with this, but as much as I see it being useful, I worry about feature creep.
We have been adding a lot of new commands to BuildStream in this cycle, but they have mostly been there to expose missing bits of API and clean up inconsistencies. This adds an entirely new feature (tracking partially does this, but that's by accident, not design). I would be unhappy landing this without having [Gitlab user @tristan] or [Gitlab user @juergbi] at least approve the feature (neither of them seems to have replied to the ML topic).
Before we land this we should probably also define what "canonical" means, document it, and ensure that our implementation here actually applies that format through tests. I doubt that it currently is particularly stable, or matches what the larger yaml community thinks should be "canonical". In my mind this aims to be an opinionated formatter, and if we're writing one of those, we should do it right. Hence I also think that this entire feature probably should be a separate project. bst-lint is something that has been discussed before, and probably more suitable.
Also, on that note, please add some tests beyond ensuring that completion works.

In GitLab by [Gitlab user @coldtom] on Nov 22, 2018, 09:48
I understand concerns about feature creep, I don't think BuildStream would need this feature if not for bst track changing the formatting on being run. As this is due to ruamel.yaml, there isn't much we can do but mitigate the damage.
Currently canonical format is "whatever the ruamel.yaml dumper spits out by default" but this could be configured to be something defined and consistent. The format we consider "canonical" will have to be consistent with what the dumper does when run, otherwise we will have the same issue when running bst track. This may limit how opinionated we can be.
I'm not opposed to the formatter being a separate tool, but if it is we need to document that bst track will modify your yaml format and we strongly recommend using the tool.

In GitLab by [Gitlab user @coldtom] on Nov 22, 2018, 09:54
Commented on buildstream/_scheduler/queues/formatqueue.py line 15
While considering the implementation details I realised that this would offer little to no optimisation given the feature as it stands. The quickest way I can think of to check if an element is formatted correctly is to dump the node and compare the files, but obviously this is slower than just dumping the node.

In GitLab by [Gitlab user @coldtom] on Nov 22, 2018, 16:44
Commented on buildstream/_stream.py line 297
changed this line in version 3 of the diff

In GitLab by [Gitlab user @coldtom] on Nov 22, 2018, 16:44
Commented on buildstream/_scheduler/queues/formatqueue.py line 15
changed this line in version 3 of the diff

In GitLab by [Gitlab user @coldtom] on Nov 22, 2018, 16:44
added 74 commits
1d2d0de4...abef70fe - 72 commits from branch master3bf4ed98 - bst fmt: Add basic functionality8d77677e - bst fmt: Add tests for core functionality
Compare with previous version

In GitLab by [Gitlab user @tristanvb] on Dec 3, 2018, 09:45
Commented on buildstream/element.py line 1342
This probably needs to have a lot more control over the ordering of keys in the output.
I.e. see https://gitlab.com/BuildStream/buildstream/issues/485#note_121782763

In GitLab by [Gitlab user @coldtom] on Dec 13, 2018, 11:31
marked as a Work In Progress

In GitLab by [Gitlab user @coldtom] on Dec 13, 2018, 13:46
added 157 commits
8d77677e...224aa4c2 - 154 commits from branch master4d3c032a - bst fmt: Add basic functionality6b0a2e1a - bst fmt: Add tests for core functionality778f1db9 - bst-fmt: Allow greater control over order
Compare with previous version

In GitLab by [Gitlab user @coldtom] on Dec 17, 2018, 12:37
added 15 commits
778f1db9...b23bec55 - 12 commits from branch master95307bb4 - bst fmt: Add basic functionality574e5a78 - bst fmt: Add tests for core functionalityc1cef0cc - bst-fmt: Allow greater control over order
Compare with previous version

In GitLab by [Gitlab user @coldtom] on Dec 20, 2018, 11:41
added 12 commits
c1cef0cc...aae5e4b3 - 9 commits from branch master67d8d0a1 - bst fmt: Add basic functionality9b5e2133 - bst fmt: Add tests for core functionalityecaf433f - bst-fmt: Allow greater control over node order
Compare with previous version

In GitLab by [Gitlab user @coldtom] on Dec 20, 2018, 11:43
Commented on buildstream/element.py line 1342
[Gitlab user @tristanvb] I have reworked this to have greater control over the order of dumping, but it has introduced an issue in stripping comments in some situations. I still need to add tests but want feedback on the approach used first.

In GitLab by [Gitlab user @LaurenceUrhegyi] on Mar 26, 2019, 18:15
Hi, thanks for the contribution! Unfortunately this WIP MR is over a month without an update from the author, so as per our policy I'm going to close this to keep our queue tidy. Please do re-open the MR if you come back to work on it. Cheers!

In GitLab by [Gitlab user @LaurenceUrhegyi] on Mar 26, 2019, 18:15
closed

In GitLab by [Gitlab user @jjardon] on Mar 26, 2019, 19:32
This MR is waiting for feedback from the maintainers; reopening

In GitLab by [Gitlab user @jjardon] on Mar 26, 2019, 19:32
reopened

In GitLab by [Gitlab user @jjardon] on May 4, 2019, 03:54
[Gitlab user @coldtom] can you please rebase this? For the comments seems It's (almost) ready to merge

In GitLab by [Gitlab user @danielsilverstone-ct] on Jul 4, 2019, 09:44
Hi, thanks for the contribution! Unfortunately this WIP MR is over a month without an update from the author, so as per our policy I'm going to close this to keep our queue tidy. [Gitlab user @coldtom] please re-open the MR if you come back to work on it. Cheers!

In GitLab by [Gitlab user @danielsilverstone-ct] on Jul 4, 2019, 09:44
closed

In GitLab by [Gitlab user @jjardon] on Dec 9, 2019, 09:15
Again, This MR was waiting for feedback from the maintainers; reopening

In GitLab by [Gitlab user @jjardon] on Dec 9, 2019, 09:15
reopened

In GitLab by [Gitlab user @BenjaminSchubert] on Dec 9, 2019, 10:13
I'm not sure what we would gain with that command:

The ruamel formatter in theory should not change the format of the file at all. In practice, if it does, that's a bug that should be reported upstream and fixed.
Other tools (like yamllint) are very good at linting your yaml, and I believe that having them as a precommit or part of the pipeline might be easier and less work to maintain in BuildStream itself.

In GitLab by [Gitlab user @jjardon] on Dec 12, 2019, 04:34
[Gitlab user @BenjaminSchubert] does yamllint convert to a canonical format or only errors/warns if something is not following the format? This is about the former

In GitLab by [Gitlab user @BenjaminSchubert] on May 11, 2020, 20:46
[Gitlab user @jjardon] woops seems I missed that one. No, yamllint doesn't convert to a canonical format. Seems like we'd still need that one then.


Rename to meet standard conventions

What should we do?

[x] Rename it from element-template[s]-icon[s]-renderer to element-template-icon-renderer
[x] Re-publish + deprecate original

Why should we do it?
Ensure this repository adheres to standard naming conventions, cf. also element-template-chooser.

Published new + deprecated old package: https://www.npmjs.com/package/@bpmn-io/element-templates-icons-renderer.


on import  link extensions make absolute urls relative

WHY are these changes introduced?
The current process to create legacy link extensions in the partners dashboard enables to create non secure and external pointing absolute URLs for embedded apps. Those URLs are later on overwritten by core to point to the shop embedded app URL.
This behaviour fills the DB with incorrect URLs and the new CLI managed admin link extension is avoiding this pattern.
WHAT is this pull request doing?
To reduce friction on the partners migration, this code automatically ignores the domain of the embedded app legacy link extensions and transforms them into relative URLs.
How to test your changes?

From the CLI repo, checkout this branch.

Create a link extension in the partners dashboard with an absolute URL pointing to a random external domain (http://example.org/product).

In the CLI repo run the import extensions command: pnpm shopify app import-extensions --path path-to-your-app

Select the admin links option

Check the the generated extension toml contains a url like app://product

[x] I've considered possible cross-platform impacts (Mac, Linux, Windows)

[x] I've considered possible documentation changes

/snapit


Make SSH Signage permissions configurable

Discussed in https://github.com/quexten/goldwarden/discussions/96

Originally posted by benni347 February  5, 2024
Hey, I wanted to ask if it is possible to deactivate the window which asks to allow ssh signage, if it is possible, I would also like to know if I can activate/deactivate it on a per app basis?
I am talking about these windows:

SSH key prompts should rarely occur now, doing even less would lower the security.

Hi, I still have it for every single ssh connection.


Losing control of player

I get the following error (below) inconsistently when using the seekTo function. It seems that what I am trying to do works some percentage of the time when the page loads and the other times it just pauses the video and then eventually throws the error when I change the playing prop to false after the attempted seekTo which stalls the video.
Uncaught (in promise) DOMException: The play() request was interrupted by a call to pause().
I'm finding it hard to think of what could be causing it as it works fine some of the time.
I'm just using a normal url to host the webm video file currently.

Similarly when I try to change loop after the video has started it doesn't take effect. Is that the intended behaviour?

Nevermind it was something to do with an error being throws due to a specific type of webm encoding.


[PBDEV-1479] add support for multi domain in fabric map handling

updated the fabricMap to be categorized by domain name
modified the GetNextAttachableHost() function to return attachable nodes according to the domain-specific fabricMap
added integration test scenarios to validate the functionality of multi domain support

Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
[ ] Reviewed the developer guide on Submitting a Pull Request
[ ] Pending release notes updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

무엇을 하기 위한 (이슈 수정?) PR 인가요?

single domain support를 온전히 완료한 후 다시 open 하겠습니다.


Update cms-rare.toml

Description
All previous genesis validators should name their PRs "Update {validator_alias}.toml for tesntet-15" and provide links to previous PRs merged.
If this is an UPDATE for a previous genesis validator
Checks are made against your net_address. If this has changed since the previous testnet, make sure you provide links of previous prs merged from your previous git username.
Thanks in advance!
Checklist before merging

[x] Only one toml is added in this PR
[x] The file being added is indeed a .toml file
[ ] The toml being added is to the correct folder, and only to the correct folder
[ ] The eth_hot_key and eth_cold_key are present
[ ] The email, discord, elements telegram, and twitter fields are present and valid

Successfully validated transactions.toml file for commit hash 8c390c2

Actually this is a lie, it does not comply with the correct metadata formulation. Please see the example PR in the docs.

@bengtlofgren
Can you give a bit more detail? The docs were not clear about this matter. I used version 0.28.0 as directed, but
namadac utils init-genesis-validator
has a flag for:
--discord-handle
and no flag for elements or twitter so I added them manually... Is it supposed to be discord and not discord_handle? this is what the binary set in the toml...

Successfully validated transactions.toml file for commit hash 61032a6

Uh oh, you've just made me aware of something quite important thank you. I didn't realise that all those fields had now become part of the cli.

OK, I put it back to discord_handle as per the instructions in discord

Successfully validated transactions.toml file for commit hash cfdf58b

Successfully validated transactions.toml file for commit hash cfdf58b


Add thermal modeling capabilities to the lead-acid model

Summary
Add parameters to the lead-acid model to allow for use of the thermal models.

note that the thermal models were written with only li-ion in mind. in particular, the heat source terms in base_thermal won’t all be correct for lead-acid, so you may need an if statement for calculating those based on chemistry


chore: further optimize logo.svg

The changes are lossy, however, they should not be noticeable to the human eye.

Size before
Size after
Size Reduction
Machine image diff

3716 B
1990 B
1726 B (46.45%)

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


Fix spelling error

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

Spelling fix verified.


[WIP] Allow attach to reload the world

Description
After the initial compile, if the user attached without building null out the build time. This has the effect of forcing all used sources into the next reload. This allows a user that continually attaches to an older app to  always reload changes into the app without restarting, at the cost of a much slower first reload time.
Also makes the ProjectFileInvalidator an injectable interface and async to allow alternative implementations.
Related Issues
Fixes https://github.com/flutter/flutter/issues/38320

I'm going to refocus this a bit: We can provide an option by which users can reset the compilation time.


Package name and Product name

Hi, thank you for great tool.
Is there a way to package with a name that is not productName?
electron-packager-6.0.0 seems to naming with productName like productName.app or productName.exe.
I want to specify a human-readable product name that might be long and it might include space, comma, etc,.
As actual example, "Microsoft Office Word" is specified as product name to WINWORD.EXE.
I specified name option and version-string.ProductName option for Windows, then name.exe that has ProductName as file property was created. This result has come up to my expectations.
But, CFBundleDisplayName and CFBundleName fields for OS X were set name. I want to set productName to these.

I'm not clear on what you're asking. Could you please write some explicit steps to reproduce the behavior you're seeing (vs. what you expect)?
This is what I did:
git clone https://github.com/atom/electron-quick-start && cd electron-quick-start
$EDITOR package.json # Added `"productName": "Electron Quick Start",` (minus the backticks) under the "name" key
npm install
npm install --save-dev electron-packager
$(npm bin)/electron-packager . --all

Expected: CFBundleName and CFBundleDisplayName are set to Electron Quick Start, not electron-quick-start.
Actual: In Electron Quick Start-darwin-x64/Electron Quick Start.app/Contents/Info.plist, the relevant key-value pairs look like this:
    <key>CFBundleDisplayName</key>
    <string>Electron Quick Start</string>
    <key>CFBundleName</key>
    <string>Electron Quick Start</string>

Yes, productName is set to CFBundleName and CFBundleDisplayName, but productName is used to name of files and directories, name is ignored.
Then Electron Quick Start.app, Electron Quick Start.exe, etc., are created.
These include spaces.
Product name might include more special characters, for example ".Net foo bar" (a dot at first), and ® is commonly used. http://unicode.org/cldr/utility/character.jsp?a=00AE
I hope that name option or name key of package.json (e.g. electron-quick-start) is used to name of executable file and other folders, and productName (e.g. Electron Quick Start) is used to property of executable file and fields of plist such as CFBundleDisplayName.
I can do that by name option and version-string.ProductName option for Windows (don't set productName), but I don't know a way to set to fields of plist.

One of the things that we could do is strip out any non-word-or-whitespace character from generated filenames.
What I'm trying to avoid is a pages-long list of parameters and/or possible package.json keys. It's bad enough already.

I think that name (i.e. short lowercased name) should be used as name of files and folders, and productName (i.e. application's full capitalized name) should be set to file property or field of plist.
For example, the original electron executable file is named electron.exe and its Product Name property is Electron. (for Windows. but folders for OS X are named Electron.app.)
Anyway, I can already control the name of generated files and folders by name option.
And I can specify a product name by version-string.ProductName option for Windows.
But, specific plist fields with extend-info option for OS X are overridden by name.
So, I wrote a build task script that changes those fields to proper product name.

I am not opposed to this idea, but it will require a major version bump.

Ok, I understand that.
For the time being, I use task script.
I think that a note should be added to document, that say: productName or name of package.json or name option should be safety string that is used to name of files and folders.
Thank you for your time :smile:

At least on OS X, .apps very often have title-cased names.  I'd be wary of electron-packager suddenly becoming magically opinionated about that.

To be clear, I was not endorsing forcing Mac .app folders to always be set to package.json's name. I do forsee bugs being filed in the future about how certain names cannot be generated because the filesystem is not Unicode-based.

Someone could file a bug like that today.
I agree that it's desirable to permit (not require) a split between human-readable name fields (CFBundleName, CFBundleDisplayName) and filesystem names (CFBundleExecutable). And I'd rather do it with an additional (optional) option than have the packager try to transform my input into a "safe" format.

And also, a name that includes spaces of an executable file may make problem.
The important point is that we don't make productName include space and special characters when we use electron-packager. In many cases, name is safety (because that is used to repository name, directory name, etc.), but productName may include spaces.
I think that this should be added to document.

And also, a name that includes spaces of an executable file may make problem.

When would spaces in an executable name be a problem?

Spaces in a path that is not quoted are recognized as separator according to context that starts the application.
For example, adding a path to an application launcher, specifying a path of favorite editor such as Atom to git, setting-up a task scheduler, and specifying a path in command line like shell (but in this case, user will find spaces), etc. Users will not perceive that in GUI operation.
path/to/Electron Quick Start.exe is split to path/to/Electron and Quick Start.exe as two options.

It's more likely that /path/to/Electron, Quick, and Start.exe would be the options, but when would you be using a shell in Windows (or any other OS) and not know to quote arguments with spaces?
(By the way, the version of electron-builder I'm using generates installer files for Windows that are named $APP_NAME Setup.exe, which wouldn't surprise me if that was the default for NSIS.)

Nearly all Mac apps are not meant to be started up from the command line. If you're building one that is, I think it's in your lap to either not use spaces or document for your users how to quote spaces.

Nearly all Mac apps are not meant to be started up from the command line. If you're building one that is, I think it's in your lap to either not use spaces or document for your users how to quote spaces.

Agreed.

Sorry, my English is very poor.
Did you ask me about cases that users don't quote a path?
For example, some file selectors with GUI dialog box don't make user think a path, such as adding a path to an application launcher,  task scheduler, etc.
Anyway, characters that have meaning for file system (e.g. :, \, /, *, ?, ", space, etc.) should not be included in productName.
I think that installer files are usually started in Windows explorer by user operation, therefore it may be not problem.
@erkyrath, as I said, path in command line is ok, because user will find spaces.
Some application launchers require paths of applications. Before, that threw an error about space included folder name.
Anyway, for the time being, I use task script to fix productName. And an issue in my case is solved.

So, don't you think adding the information about productName to document?
electron-builder's document says clearly: name doesn't allow spaces and special characters, and productName allows those.
https://github.com/electron-userland/electron-builder/blob/master/docs/options.md#application-packagejson
grunt-build-atom-shell's document also says clearly that the productName is used to name of generated files.
https://github.com/paulcbetts/grunt-build-atom-shell#options
(not package.json)
I think that the note should be added to document, that says: productName and name of package.json or name option should be safety string that doesn't include spaces, /, \, :, . at first, etc.
The rule about name of package.json is defined, but since electron-packager handles productName like name, that should be shown in document.

Related: https://github.com/electron-userland/electron-builder/pull/204
I don't see anything wrong in the current electron-packager behaviour.
electron-packager packs your app and produce one artefact. App file. If you specify product name, you expect that it will used in all places. As @tmm1 said and I am agree.
electron-builder produces nupkg, deb/rpm/etc and other deployment artefacts (in a distributable format). Publish to GitHub releases. In this case we must use only name and it is expected. Also, for example, electron-builder uses name in the icon path on Linux instead of product name to avoid unsafe chars (because icon layout is under the hood and is not visible to end user).
So, I don't think we should change current behaviour. Option to change exe name may be added (but I don't like it — yet another option + "Nearly all Mac apps are not meant to be started up from the command line" + it is ok to use Google Chrome/App Store/LEGO Digital Designer on OS X).

As actual example, "Microsoft Office Word" is specified as product name to WINWORD.EXE

And what? Do you start MS Office using terminal (cmd)? Or using Spotlight (I mean poor windows alternative Cortana/Start)? How does it affect end user? In any case you should create desktop shortcuts on Windows and do not distribute electron-packager result (instead, you should create Squirrel.windows app file using electron-builder or tool like that).

Hi @develar, thank you for your advice.
I could not understand your advice completely, but those may help me.
Anyway, as I said, an issue in my case was already solved by task script or electron-builder.
And I suggested adding the information. (https://github.com/electron-userland/electron-packager/issues/297#issuecomment-204307916) Because those unsafe characters (e.g. /, \, :, .) often be used for product name.
p.s. I gave WINWORD.EXE as an example of the product name that differs from the executable file name. Not an example of problems. and of course I don't start MS Office from terminal.
I say again, an issue in my case was already solved. I just thought that the information about productName may help someone. Someone might use /, \, :, etc., for product name.

FYI, there's a different bug to strip special path characters from generated filenames: #308

Hi. So, is it possible to set the Original filename? Now it's set as electron.exe in built *.exe Properties.
And having the filename without spaces would be also sweet.

@z-ax See: https://github.com/electron-userland/electron-packager/blob/master/docs/api.md#version-string

@malept Oops, missed ¯_(ツ)_/¯. Exuses, all works as required.
Thanks for very fast reply!

Another reason to strip spaces from your file name:
MakeAppx : error: Error info: error C00CE169: App manifest validation error: The app manifest must be valid as per schema: Line 6, Column 13, Reason: 'File Name' violates pattern constraint of '[-.A-Za-z0-9]+'.
MakeAppx that is needed to get your app on Windows store doesn't like a filename with spaces.
Microsoft (R) MakeAppx Tool Copyright (C) 2013 Microsoft.  All rights reserved.

This might be useful -- I did finally have success with this.
https://github.com/electron-userland/electron-builder/issues/4287


Added Volley Library Exemplar App issue #46

Volley Library Exemplar App
It's a Exemplar App for showing about How Volley Library for calling/fetching data from API and how it's work.
What's the purpose of Volley Library in Android.
Best way to learn Volley Library through this App.
What is Volley Library ?
Volley is an HTTP library that makes networking for Android apps easier and most importantly, faster.
Volley is available on GitHub.
Fixes #46
Type of Change:

Android application added

Checklist:

[ ✔ ] My PR follows the style guidelines of this project
[ ✔ ] I have performed a self-review of my own code or materials
[ ✔ ] My branch is up-to-date with the Upstream development branch
[ ✔ ] I have added necessary images/screenshots of my changes

Hey @aadityamp01 I have added the fixed/stable screenshots in Readme.md
Branch is upto date too. Parallel with main repository
Thanks


EACCES permission denied with multiple users

Describe what happened:
When multiple users on the same machine try to use this extension to join sessions, only the first user to do so will succeed. The rest will get permission errors:
joining collaboration session: EACCES: permission denied,  mkdir '/tmp/vsliveshare/tmp-...'

I suspect this is because the first user to join a session will own the /tmp/vsliveshare directory.
What was your system configuration?
Product and Version [VS/VSCode]: VSCode 1.52.1
OS Version[macOS/Windows]: Ubuntu 18.04 (Remote SSH Extension v0.63.0)
Live Share Extension Version: v1.0.3577
Target Platform or Language [e.g. Node.js]: n/a (any)
Steps to Reproduce / Scenario:

User A connects to a session (this creates /tmp/vsliveshare, owned by A)
User B attempts to connect to a session from the same machine as A, gets permission denied error

Please attach logs to this issue: The log was 0 bytes in size.

Hi @galaharon, thanks for reaching out to Live Share! We're sorry about the issue you're having. We faced a similar issue with logging permissions for the remote extension scenario (#3252) and fixed it by allowing there to not be logging, which is why your logs are 0 bytes. However, the host machine (User A) should have logs in the scenario you described above. Can you please check for those and attach them, if possible?
Live Share needs the contents of tmp/liveshare to run, so bypassing them isn't an option like we did for logging. I will mark this issue for triage.
Thanks!

Hi @galaharon, thanks for reaching out to Live Share! We're sorry about the issue you're having. We faced a similar issue with logging permissions for the remote extension scenario (#3252) and fixed it by allowing there to not be logging, which is why your logs are 0 bytes. However, the host machine (User A) should have logs in the scenario you described above. Can you please check for those and attach them, if possible?
Live Share needs the contents of tmp/liveshare to run, so bypassing them isn't an option like we did for logging. I will mark this issue for triage.
Thanks!

I believe this issue is resolved with recent changes.


[dart2js] Breaking change: remove support for compiling sources from http

Context: The dart2js compiler internally allowed compiling inputs provided via an HTTP client connection. Support for this was experimental and never documented. We don't believe this functionality is in use at this time, and we intend to delete it with the goal of reducing technical debt in our system.
What can break?
If you run dart2js and provide http URIs as inputs, the compiler will no longer download the files and compile them. Instead it will provide a compile-time error indicating that the compiler doesn't support http sources.
We don't expect this to be an active pattern used by the community today, but we are using this notice to make sure we didn't miss any use cases.
Mitigation
If by chance there is a use of this feature, the mitigation is to add a tool chain step to first download resources and then run the compiler with inputs from local disk.
/cc @itsjustkevin @kevmoo @mit-mit @vsmenon

SGTM!

There aren't active uses today, but it is potentially useful for running dart2js against a development server (like webdev, or the internal equivalent). Experiments have shown it to be pretty slow though compared to loading files from disk, even with a local server.

@Hixie @grouma @vsmenon @mit-mit thoughts on this breaking change?

SGTM.

fine by me

SGTM

Marking this change as approved


Fix typos in config.toml

Noticed that some of the comments for password field attributes refer incorrectly to the username field.

Sorry for the delay. Thank you for the fix.


fix(webpack): support import.meta; new URL(./, import.meta.url)

closed in favor of https://github.com/open-wc/open-wc/pull/64


Typo in KeyVaultClient causes invalid authorization request

I got this error while trying out the KeyVaultClient
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Azure("AuthorizationError(Failed to authenticate to Azure Active Directory
Caused by:
   Generic error: AADSTS70011: The provided request must include a \'scope\' input parameter. The provided value for the input parameter \'scope\' is not valid.
   The scope https://vault.azure.net.default is not valid

adding a forward slash fixes the crash

@MindFlavor , Is there anything else I need to do to get this change merged in?

Sorry @jefshe, it's my fault, I have missed rylev's approval.
Thank you for you help! 👍

No worries! Thanks for merging it in!


chore(ci): simplify and cleanup

Thanks for the PR!
Deployments, as required, will be available below:

Frontend
Backend

Please create PRs in draft mode.  Mark as ready to enable:

Analysis Workflow

After merge, new images are deployed in:

Merge Workflow

On hold while working on a Helm deployer action.
Set up bcgov-nr/action-deployer-helm

Most of this has been replaced by other recent PRs.  Closing.


[ALLUXIO-3164] Add profile for selective HDFS ACL support

https://alluxio.atlassian.net/browse/ALLUXIO-3164

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19885/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19887/
Test PASSed.


added new /homepage route

curl http://localhost:3060/homepage returns
{
  collectives: {
    meetup: [{group1}, {}, ...],
    opensource: [{group1},{}, ...]
  },
  stats: {
    totalCollectives: 60,
    totalDonations: 30124,
    totalDonors: 293
  }
}

Can you also add a test for this new route with data? Since this will be on our homepage, let's make sure we'll know if it breaks.

Waiting on a test to merge this. You can probably copy the leaderboard tests with some dummy data. Don't let perfect (real data for tests) prevent this from being merged, when good enough (dummy data) is fine. :)


Intercooler's CDN doesn't support crossorigin/integrity checks

I'm trying to add crossorigin and integrity attributes to all of the CSS and JS resources that my project gets from CDNs. In the case of Intercooler, that looks like this:
<SCRIPT src="http://cdn.intercoolerjs.org/intercooler-1.1.2.min.js" integrity="sha384-NLFLtSEhkAcJ3fF4aBB6yNp4U2BmG+eRjKv9qKYfdGEC4ofH+1W4X81KL18xiBBx" crossorigin="anonymous"></SCRIPT>
This fails in Chrome with the following error:
Access to Script at 'http://cdn.intercoolerjs.org/intercooler-1.1.2.min.js' from origin 'http://localhost:8000' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:8000' is therefore not allowed access.
For reference, both Bootstrap's and jQuery's CDNs return a catch-all CORS policy header like this:
Access-Control-Allow-Origin: *
Can you enable that header on Intercooler's CDN?

I think I've enabled it:

Yep, it's working now! Awesome.
By the way, the integrity value I originally put in the description was incorrect due to being mid-debugging (changed one character to see if I got different errors). I've fixed it in the description. I had been getting the original error both with the correct and incorrect integrity values.


Improve query parameters '?' handling.

Currently, query parameters (?) are handled in various places across the planner. The process is as follows:

When query is first sent to backend, it is parsed and the parse tree (AST) is (is it?) then cached for further use
AST is reanalyzed with Parameter index -> expression bindings
Query plan is rebuild based on analysis and Parameter index -> expression bindings

Parameter nodes are replaced with expressions by ParameterRewriter

The problem with this approach are:

Only AST parse tree can be cached. The rest (analysis, query plan) have to be rebuild/reanalyzed
ParameterRewriter is used across the planner.

Places where ParameterRewriter is used in planner are often unrelated to query parameters (e.g: in AggregationAnalyzer, TranslationMap). The impression is that ParameterRewriter and Parameter are leaking to unrelated code

With the goal to cleanup potentially unrelated ParameterRewriter usages I am proposing the following refactor:

Analyze EXECUTE parameter bindings separately from query. This will result in Parameter index -> type bindings
Analyze query with Parameter index -> type bindings.
Build query plan based on analysis (no Parameter specific code)

Such query plan can be cached for further use with the same parameter types

Introduce some kind of ParameterDesugarer which would replace Parameter nodes witch appropriate constants or expressions (which can be evaluated to constants).

The benefits of this approach are:

Query plan can be reused with parameters of the same type
ParameterRewriter is not handled explicitly throughout the planner. The code significantly simplifies.

This is somehow related to: https://github.com/prestodb/presto/issues/7398

This is copied from my old issue https://github.com/prestodb/presto/issues/7989.
I think it's in much better state thanks to @martint. ParameterRewriter is only used in analysis in AggregationAnalyzer (I removed it in https://github.com/prestosql/presto/pull/1144).
Parameters are now used in initial query building only.
It should be fairy easy to move them out of there so that initial plan can be cached.

One question about this: Lets say I have a query like this: select count(1) from table where time_col between cast(to_unixtime(<param1> - interval '1' hour) as bigint) and cast(to_unixtime(<param1>) as bigint)
If you do this planning without 's actual expression, how would we do the equivalent of tuple domain extraction or pushdown for that matter. The question is: Do we expose the rest of the planning logic to the parameters ?
I think if we do this properly, we can totally fix #1141. I think it would be unavoidable to truly expose parameters to the rest of the planning framework including expression simplification, pushdown etc. And I think that is okay, but just increases the scope of the work.

If you do this planning without 's actual expression, how would we do the equivalent of tuple domain extraction or pushdown for that matter. The question is: Do we expose the rest of the planning logic to the parameters ?

There are 3 planning stages:

ANALYSIS
IR BUILDING (INITIAL PLANNING)
OPTIMIZATION

We could potentially cache results of ANALYSIS and IR BUILDING stages for prepared statements. Predicates would still be pushed down in OPTIMIZATION stage.
However, first we need to know if caching would make a difference for low-latency queries. For that we need statistics what takes the most time in such queries and whether additional complexity is really worth it. It could be that planning stages 1) and 2) take little time compared to 3) stage.

Hi @sopel39, I replied on the other issue #1141 with statistics on where the planning time is going currently :-). But to summarize: The majority of time is going in the Optimization phase (running the list of rule optimizers), ie bullet 3 in your description. Of that the majority of the time is in expression related optimizers, pruning etc.
I will try to profile and get some numbers on low hanging fruits.
But basically making Presto truly be aware of parameters, ie truly implementing late binding would be perhaps more complex than it meets the eye because a lot more downstream planners would need to change to support the notion of parameters and this late binding.
For example, how will you do partition pruning or predicate pushdown when the expression is late bound ? Sounds complicated :-)

For example, how will you do partition pruning or predicate pushdown when the expression is late bound ? Sounds complicated :-)

You could potentially inject parameters after some of the optimizers already fired, but before predicate pushdown. I would start with optimizers benchmarking first though.


CDS-79431 google-functions env option note

https://harness.atlassian.net/browse/CDS-79431
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ x] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

https://harness.atlassian.net/browse/CDS-79431

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Fixed documentation link for storybook in README.md

Simple documentation fix for the storybook documentation link which returns a not found page

Thanks.


USHIFT-2966: ensure the output of git push is in the traceback if there is an error

/assign @pmtk

/lgtm


time-range filter should persist when selecting different dashboard in repo pages

When I'm on the Overview dashboard for repo pages and change the range, I expect that range to persist when I click the contributors tab. Instead, it goes back to the default of 30days.
 Issues get the needs triage label because the team still needs to see what needs to be done or if it's ready to be worked on.


_HEX_SYMBOLS doesn't seem to be used in DittoMachine.sol

https://github.com/ditto-lab/ditto/blob/6d04912f15957206d33bb39e8d9f8a2c02bee4a5/src/DittoMachine.sol#L42
I think this variable can be removed it is no longer being used in DittoMachine.sol

I have this in one of my draft PRs.


Windows 10 upgrade failed

Steps to Reproduce
A:\flutter\bin\flutter.bat --no-color upgrade
Upgrading Flutter from A:\flutter...
From https://github.com/flutter/flutter
49c87d010..171f94d17  master     -> origin/master
Already up to date.
Upgrading engine...
Oops; flutter has exited unexpectedly.
Sending crash report to Google.
Crash report sent (report ID: 39c834a69d345530)
Crash report written to A:\flutter\flutter_01.log;
please let us know at https://github.com/flutter/flutter/issues.
Process finished with exit code 1
Logs
command
flutter --no-color upgrade
exception
ArgumentError: Invalid argument(s): Cannot find executable for bin\flutter.
#0      _getExecutable (package:process/src/interface/local_process_manager.dart:113)
#1      LocalProcessManager.start (package:process/src/interface/local_process_manager.dart:41)
#2      runCommand (package:flutter_tools/src/base/process.dart:115)
#3      runCommandAndStreamOutput (package:flutter_tools/src/base/process.dart:133)
<asynchronous suspension>
#4      UpgradeCommand.runCommand (package:flutter_tools/src/commands/upgrade.dart:61)
<asynchronous suspension>
#5      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:290)
<asynchronous suspension>
#6      FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:228)
<asynchronous suspension>
#7      CommandRunner.runCommand (package:args/command_runner.dart:194)
<asynchronous suspension>
#8      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:286)
<asynchronous suspension>
#9      CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:109)
#10     new Future.sync (dart:async/future.dart:222)
#11     CommandRunner.run (package:args/command_runner.dart:109)
#12     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:166)
#13     run.<anonymous closure> (package:flutter_tools/runner.dart:90)
<asynchronous suspension>
#14     AppContext._run (package:flutter_tools/src/base/context.dart:76)
<asynchronous suspension>
#15     AppContext.runInZone.<anonymous closure> (package:flutter_tools/src/base/context.dart:66)
#16     _rootRun (dart:async/zone.dart:1126)
#17     _CustomZone.run (dart:async/zone.dart:1023)
#18     runZoned (dart:async/zone.dart:1501)
#19     AppContext.runInZone (package:flutter_tools/src/base/context.dart:65)
#20     run (package:flutter_tools/runner.dart:61)
<asynchronous suspension>
#21     main (package:flutter_tools/executable.dart:48)
<asynchronous suspension>
#22     main (file:///A:/flutter/packages/flutter_tools/bin/flutter_tools.dart:16)
#23     _startIsolate.<anonymous closure> (dart:isolate-patch/dart:isolate/isolate_patch.dart:277)
#24     _RawReceivePortImpl._handleMessage (dart:isolate-patch/dart:isolate/isolate_patch.dart:165)

Flutter Doctor
PS> flutter doctor -v
[√] Flutter (Channel beta, v0.2.3, on Microsoft Windows [Version 10.0.17127.1], locale en-US)
• Flutter version 0.2.3 at A:\flutter
• Framework revision 5a58b36e36 (2 weeks ago), 2018-03-13 13:20:13 -0700
• Engine revision e61bb9ac3a
• Dart version 2.0.0-dev.35.flutter-290c576264
[X] Android toolchain - develop for Android devices
X ANDROID_HOME = C:\Program Files\Android
but Android SDK not found at this location.
[√] Android Studio (version 3.0)
• Android Studio at C:\Program Files\Android\Android Studio
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)
! Doctor found issues in 1 category.

The problem might be that we're not handling Windows here:
https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/commands/upgrade.dart#L83
It's not clear if this is actually failing on Windows but we definitely need to check before we do another release.

I'm not able to reproduce this on my machine (both cmd and powershell).
Initial run:
PS C:\Workspace\flutter> .\bin\flutter.bat --no-color upgrade
Checking Dart SDK version...
Downloading Dart SDK from Flutter engine e61bb9ac3a3fd789754e2e54220bcfc27076a857...
Unzipping Dart SDK...
Updating flutter tool...
Upgrading Flutter from C:\Workspace\flutter...
Already up-to-date.

Upgrading engine...
Downloading package sky_engine...

Flutter 0.2.3 • channel beta • git@github.com:flutter/flutter.git
Framework • revision 5a58b36e36 (2 weeks ago) • 2018-03-13 13:20:13 -0700
Engine • revision e61bb9ac3a
Tools • Dart 2.0.0-dev.35.flutter-290c576264

Running flutter doctor...
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel beta, v0.2.3, on Microsoft Windows [Version 10.0.10586], locale en-US)
[X] Android toolchain - develop for Android devices
    X Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).
      If Android SDK has been installed to a custom location, set $ANDROID_HOME to that location.
[√] Android Studio (version 3.0)

! Doctor found issues in 1 category.
PS C:\Workspace\flutter> .\bin\flutter.bat --no-color upgrade
Upgrading Flutter from C:\Workspace\flutter...
Already up-to-date.

Upgrading engine...
Already up-to-date.

Flutter 0.2.3 • channel beta • git@github.com:flutter/flutter.git
Framework • revision 5a58b36e36 (2 weeks ago) • 2018-03-13 13:20:13 -0700
Engine • revision e61bb9ac3a
Tools • Dart 2.0.0-dev.35.flutter-290c576264

Running flutter doctor...
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel beta, v0.2.3, on Microsoft Windows [Version 10.0.10586], locale en-US)
[X] Android toolchain - develop for Android devices
    X Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).
      If Android SDK has been installed to a custom location, set $ANDROID_HOME to that location.
[√] Android Studio (version 3.0)

! Doctor found issues in 1 category.

Run when we're already up-to-date:
PS C:\Workspace\flutter> .\bin\flutter.bat --no-color upgrade
Upgrading Flutter from C:\Workspace\flutter...
Already up-to-date.

Upgrading engine...
Already up-to-date.

Flutter 0.2.3 • channel beta • git@github.com:flutter/flutter.git
Framework • revision 5a58b36e36 (2 weeks ago) • 2018-03-13 13:20:13 -0700
Engine • revision e61bb9ac3a
Tools • Dart 2.0.0-dev.35.flutter-290c576264

Running flutter doctor...
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel beta, v0.2.3, on Microsoft Windows [Version 10.0.10586], locale en-US)
[X] Android toolchain - develop for Android devices
    X Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).
      If Android SDK has been installed to a custom location, set $ANDROID_HOME to that location.
[√] Android Studio (version 3.0)

! Doctor found issues in 1 category.
PS C:\Workspace\flutter> flutter --no-color upgrade
Upgrading Flutter from C:\Workspace\flutter...
Already up-to-date.

Upgrading engine...
Already up-to-date.

Flutter 0.2.3 • channel beta • git@github.com:flutter/flutter.git
Framework • revision 5a58b36e36 (2 weeks ago) • 2018-03-13 13:20:13 -0700
Engine • revision e61bb9ac3a
Tools • Dart 2.0.0-dev.35.flutter-290c576264

Running flutter doctor...
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel beta, v0.2.3, on Microsoft Windows [Version 10.0.10586], locale en-US)
[X] Android toolchain - develop for Android devices
    X Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).
      If Android SDK has been installed to a custom location, set $ANDROID_HOME to that location.
[√] Android Studio (version 3.0)

! Doctor found issues in 1 category.

Unlabelling TODAY since we're unable to reproduce locally. We have some further checking to do on flutter upgrade on Windows.

I reinstall flutter (with removing flutter folder and AppData dart folders)
And again get this error, it seems I found the reason but I do not understand why so

Windows 10 Enterprise Insider
Version: 1803
Build: 17127.1

@xeurun -

I assume you have rebooted since modifying the Path...   (If not, this is probably why it is failing)
In your cmd prompt; type echo %PATH% and see what it says.

I just upgraded both the beta and master channels fine on Windows 7.     But you might have found a issue on Windows 10 Insider if you have rebooted.

I restarted the computer several times

Wow, that is very interesting.   Can you try clicking into Properties->options on the console window; and then choose Use Legacy Console; restart the console and see if you can type "flutter" alone w/o the .bat file

This is very very interesting

generate broken log file

i open in notepad and resave

That is very interesting..
So if you don't have use legacy console - flutter.bat actually works.   But flutter (without .bat) doesn't.   If you do have legacy console enabled flutter and flutter.bat run and just crash while trying to upgrade check.

Can you verify you are still on the "beta" branch..  git status should tell you the branch.
If you run flutter from inside powershell does it crash?    Or only the cmd w/ legacy?

I wish I had windows 10 to test this on...  So if you don't mind testing this...
I have one more idea that I would be curious to know if it works.
Remove the use legacy console, restart the cmd window.
Create a file called "flutter.cmd" in the flutter\bin folder.
In this file put flutter.bat %* as it's only line...
Then try typing flutter and see if it works...

version.dart
version.dart.txt
proof beta branch

test no legacy (flutter.cmd copy flutter.bat)

test legacy (flutter.cmd copy flutter.bat)

test no legacy (flutter.cmd is flutter.bat %*)

log

Flutter crash report; please file at https://github.com/flutter/flutter/issues.

## command

flutter upgrade

## exception

ArgumentError: Invalid argument(s): Cannot find executable for bin\flutter.

#0      _getExecutable (package:process/src/interface/local_process_manager.dart:113)
#1      LocalProcessManager.start (package:process/src/interface/local_process_manager.dart:41)
#2      runCommand (package:flutter_tools/src/base/process.dart:115)
#3      runCommandAndStreamOutput (package:flutter_tools/src/base/process.dart:133)

#4      UpgradeCommand.runCommand (package:flutter_tools/src/commands/upgrade.dart:61)

#5      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:290)

#6      FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:228)

#7      CommandRunner.runCommand (package:args/command_runner.dart:194)

#8      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:286)

#9      CommandRunner.run. (package:args/command_runner.dart:109)
#10     new Future.sync (dart:async/future.dart:222)
#11     CommandRunner.run (package:args/command_runner.dart:109)
#12     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:166)
#13     run. (package:flutter_tools/runner.dart:90)

#14     AppContext._run (package:flutter_tools/src/base/context.dart:76)

#15     AppContext.runInZone. (package:flutter_tools/src/base/context.dart:66)
#16     _rootRun (dart:async/zone.dart:1126)
#17     _CustomZone.run (dart:async/zone.dart:1023)
#18     runZoned (dart:async/zone.dart:1501)
#19     AppContext.runInZone (package:flutter_tools/src/base/context.dart:65)
#20     run (package:flutter_tools/runner.dart:61)

#21     main (package:flutter_tools/executable.dart:48)

#22     main (file:///A:/flutter/packages/flutter_tools/bin/flutter_tools.dart:16)
#23     _startIsolate. (dart:isolate-patch/dart:isolate/isolate_patch.dart:277)
#24     _RawReceivePortImpl._handleMessage (dart:isolate-patch/dart:isolate/isolate_patch.dart:165)

## flutter doctor

[✓] Flutter (Channel beta, v0.2.3, on Microsoft Windows [Version 10.0.17127.1], locale en-US)
• Flutter version 0.2.3 at A:\flutter
• Framework revision 5a58b36e36 (2 weeks ago), 2018-03-13 13:20:13 -0700
• Engine revision e61bb9ac3a
• Dart version 2.0.0-dev.35.flutter-290c576264
[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
• Android SDK at S:\AndroidSDK
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-27, build-tools 27.0.3
• ANDROID_HOME = S:\AndroidSDK
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b02)
• All Android licenses accepted.
[✓] Android Studio (version 3.1)
• Android Studio at C:\Program Files\Android\Android Studio
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b02)
[!] Connected devices
! No devices available
! Doctor found issues in 1 category.

Awesome, thanks for trying that.   That is very interesting.

Delete the flutter.cmd it didn't do what I was hoping it would do; and it might have caused the last crash for the upgrade from inside powershell.   Since it says powershell couldn't find the executable; it is possible it confused it...   I was really hoping that windows would see the .cmd extension since it wasn't seeing the .bat extension and would allow us to redirect to the batch file.   But according to all your screen snap shots; windows insider is ignore both .bat and .cmd files unless you manually put the extension on them.   ;-(

Just to confirm -- does flutter.bat upgrade work fine from the cmd prompt without the legacy and without my flutter.cmd that I had you create for testing?

flutter.bat upgrade not work since call bin/flutter, see last log or first log

@NathanaelA magic, i moved the flutter from A: drive to S: drive and... works
`
PS C:\Users\XeuRun> flutter upgrade
Upgrading Flutter from S:\flutter...
From https://github.com/flutter/flutter
cab8242f2..f69d12595  master     -> origin/master
Already up to date.
Upgrading engine...
Already up-to-date.
Flutter 0.2.3 • channel beta • https://github.com/flutter/flutter.git
Framework • revision 5a58b36e36 (2 weeks ago) • 2018-03-13 13:20:13 -0700
Engine • revision e61bb9ac3a
Tools • Dart 2.0.0-dev.35.flutter-290c576264
Running flutter doctor...
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel beta, v0.2.3, on Microsoft Windows [Version 10.0.17127.1], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[√] Android Studio (version 3.1)
[!] Connected devices
! No devices available
! Doctor found issues in 1 category.
PS C:\Users\XeuRun>
`


feat: [GALI-1966] DatabaseService.executQuery() method

Issue #, if available:
GALI-1966
Description of changes:
Introduce an interface named DatabaseService that will handle codify required database functionality.
Checklist:

[x] Have you successfully deployed to an AWS account with your changes?
[x] Have you written new tests for your core changes, as applicable?

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

A few high level things I want to discuss with you today:

Is having the method and returned data type too general as it is now? As we discussed, Query is just one of the DDB calls we make that has this problem of mutating the return object's type. Do you have plans for how the other DDB calls would get their versions of executeQuery()? I worry it might be a lot of reused code so wonder if it might be worth trying to converge on a single method for all mutations of the responses from DDB.
I do not think the current executeQuery method is either in the right service or doing the right thing exactly. The point of DDB Service is that you can create a Query object with this and then still have to call .execute() afterwards. This is so that developers can choose to hand write their Query objects and params (using Query methods) or use the DDB Service to create the params and objects (using .query()). Either way, the .execute() comes later and is triggered by the developer explicitly. I would either like to see a DDBService.format() method that takes in any response from DDB (kind of tying to Tim's comment above) OR a Query.executeQuery() method that would be used in place of Query.execute() (think 1:1 replacement). I don't think this method should be in DDBService as it is not currently a simple replacement for .execute(). Happy to sync online today!


Fix typos, remove unused imports, and grammar check

In this commit:

Fixed various typographical errors found throughout the codebase.
Removed unused imports to improve code efficiency and readability.
The conversion of README.md into AsciiDoc format for better documentation standards.

Appreciate the team's great work on this project! Hope these small tweaks and the AsciiDoc conversion make the code and documentation even better."

much appreciated,merged into 7aad51da7f9ba28676c7d19860518bc3c61de12a


Package versions

Hello,
Can someone please let me know what versions of torch, torchvision, opencv and matplotlib worked for you?
The owner(s) of this repo have not provided adequate information regarding what versions they used to run this model in the requirements section.
Thanks in advance!


No module named 'parseNetParams'

Hello,
Whenever I'm trying to build the example with make hdl I have this error:
python3 ../lib/haddoc2.py \  --proto=./caffe/lenet.prototxt \ --model=./caffe/lenet.caffemodel \ --out=./hdl_generated \ --nbits=5 Traceback (most recent call last):   File "../lib/haddoc2.py", line 34, in <module>     import parseNetParams ModuleNotFoundError: No module named 'parseNetParams' Makefile:11: recipe for target 'hdl' failed make: *** [hdl] Error 1
Do you know what causes it?

Even am having same error

Hi all,
Sorry for the long response time. I'm no longer maintaining this project ...
Have you set the HADDOC2_ROOT variable correctly ?


Don't accept invalid certificates

Signed-off-by: Michael Pankov<EMAIL_ADDRESS>

This change is

This is not needed / would forbid using self-signed certs


can I install the SDK using linux mint?

For some reason, I have to use Linux mint for my project. But I also have to use realsense depth camera D435i, is there a way yo install the relsense SDK on Linux mint? I can not figure it out myself.
I've checked about this Linux mint, they say the mint is based on Ubuntu 16, I'm confused.
Thank you in advance!

There are a few different ways to install the SDK on any flavor of Linux.
You can build the SDK from source, or do a 'bypass':
https://github.com/IntelRealSense/librealsense/issues/1132#issuecomment-363572666
Or you can do a LibUVC-Backend installation.
https://github.com/IntelRealSense/librealsense/blob/master/doc/libuvc_installation.md?language=en_US

Hi @yuan0623
To correct @MartyG-RealSense answer a bit - you cannot use libuvc method with the D435i (yet).
However, the SDK should be mostly functional with default build on recent Linux kernel versions.
Do you know what kernel version is used by the distro you are using?

Hi @yuan0623
To correct @MartyG-RealSense answer a bit - you cannot use libuvc method with the D435i (yet).
However, the SDK should be mostly functional with default build on recent Linux kernel versions.
Do you know what kernel version is used by the distro you are using?

Thank you so much. I've installed it and running it on the Mint!

There are a few different ways to install the SDK on any flavor of Linux.
You can build the SDK from source, or do a 'bypass':
#1132 (comment)
Or you can do a LibUVC-Backend installation.
https://github.com/IntelRealSense/librealsense/blob/master/doc/libuvc_installation.md?language=en_US

Thank you so much, it's running on the Mint!

Awesome news - thanks so much for letting us know  :)