Docker remote development on private network (no internet connection)

I'm trying to launch a vscode instance within a docker container.
I'm working on a Windows machine with no internet connection, connected to a remote Linux machine with no internet connection.
Using the following tutorial (Link), we were able to launch a vscode server instance on the remote machine.
Unfortunately, launching a vscode instance within a docker container on the remote server fails.

VSCode Version: 1.66.0
Local OS Version: Windows 10
Remote OS Version: RHEL 8
Remote Docker OS Version:  Ubuntu 18.04
Remote Extension/Connection Type: Docker
Logs:

[94 ms] Remote-Containers 0.232.4 in VS Code 1.66.0 (e18005f0f1b33c29e81d732535d8c0e47cafb0b5).
[93 ms] Start: Resolving Remote
[96 ms] Setting up container: /stupefied_cori
[97 ms] Host: ssh://linx107742t08
[102 ms] Start: Run: ssh linx107742t08 /bin/sh
[134 ms] Start: Run in host: id -un
[1003 ms] root
[1003 ms]
[1004 ms] Start: Run in host: cat /etc/passwd
[1010 ms] Start: Run in host: echo ~
[1013 ms] /root
[1013 ms]
[1014 ms] Start: Run in host: test -x '/root/.vscode-remote-containers/bin/e18005f0f1b33c29e81d732535d8c0e47cafb0b5/node'
[1017 ms]
[1017 ms]
[1017 ms] Exit code 1
[1018 ms] Start: Run in host: test -x '/root/.vscode-server/bin/e18005f0f1b33c29e81d732535d8c0e47cafb0b5/node'
[1020 ms]
[1020 ms]
[1021 ms] Start: Run in host: test -f '/root/.vscode-server/bin/e18005f0f1b33c29e81d732535d8c0e47cafb0b5/node_modules/node-pty/package.json'
[1023 ms]
[1024 ms]
[1024 ms] Start: Run in host: test -f '/root/.vscode-remote-containers/dist/vscode-remote-containers-server-0.232.4.js'
[1026 ms]
[1026 ms]
[1029 ms] userEnvProbe: loginInteractiveShell (default)
[1029 ms] userEnvProbe shell: /bin/bash
[1747 ms] userEnvProbe PATHs:
Probe:     '/var/lib/anaconda3/bin:/var/lib/anaconda3/condabin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/root/bin'
Container: None
[1751 ms] Start: Run in Host: docker inspect --type container /stupefied_cori
[1832 ms] Start: Check Docker is running
[1832 ms] Start: Run in Host: docker version --format {{.Server.APIVersion}}
[1909 ms] Server API version: 1.40
[1911 ms] Start: Inspecting container
[1911 ms] Start: Run in Host: docker inspect --type container bc35a261e7c584226c5e3afa26b0dbebf1c9bd0b47d819376e5f4d990f2d678f
[1997 ms] Start: Run in Host: docker exec -i -u root -e VSCODE_REMOTE_CONTAINERS_SESSION=842c41a8-0a7a-4b89-a6b7-8b749b6f62a11649854315923 bc35a261e7c584226c5e3afa26b0dbebf1c9bd0b47d819376e5f4d990f2d678f /bin/sh
[2004 ms] Start: Run in container: uname -m
[2139 ms] x86_64
[2140 ms]
[2140 ms] Start: Run in container: (cat /etc/os-release || cat /usr/lib/os-release) 2>/dev/null
[2148 ms] NAME="Ubuntu"
VERSION="20.04.3 LTS (Focal Fossa)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 20.04.3 LTS"
VERSION_ID="20.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=focal
UBUNTU_CODENAME=focal
[2148 ms]
[2148 ms] Start: Run in container: cat /etc/passwd
[2155 ms] Start: Run in container: test -d /home/jovyan/.vscode-server
[2160 ms]
[2160 ms]
[2161 ms] Start: Run in container: test ! -f '/home/jovyan/.vscode-server/data/Machine/.writeMachineSettingsMarker' && set -o noclobber && mkdir -p '/home/jovyan/.vscode-server/data/Machine' && { > '/home/jovyan/.vscode-server/data/Machine/.writeMachineSettingsMarker' ; } 2> /dev/null
[2166 ms]
[2166 ms]
[2166 ms] Exit code 1
[2166 ms] Start: Run in the container: cat /home/jovyan/.vscode-server/data/Machine/settings.json
[2171 ms]
[2171 ms] cat: /home/jovyan/.vscode-server/data/Machine/settings.json: No such file or directory
[2171 ms] Exit code 1
[2171 ms] Start: Run in container: test -d /home/jovyan/.vscode-server/bin/e18005f0f1b33c29e81d732535d8c0e47cafb0b5
[2174 ms]
[2175 ms]
[2175 ms] Exit code 1
[2175 ms] Installing VS Code Server for commit e18005f0f1b33c29e81d732535d8c0e47cafb0b5
[2176 ms] Start: Installing VS Code Server
[2176 ms] Start: Run in container: mkdir -p /home/jovyan/.vscode-server/bin/e18005f0f1b33c29e81d732535d8c0e47cafb0b5_1649854319453
[2181 ms]
[2181 ms]
[2182 ms] Start: Run in container: (dd iflag=fullblock bs=8192 count=2 2>/dev/null; dd iflag=fullblock bs=3822 count=1 2>/dev/null) | tar --no-same-owner -xz -C /home/jovyan/.vscode-server/bin/e18005f0f1b33c29e81d732535d8c0e47cafb0b5_1649854319453
[2195 ms]
[2195 ms]
gzip: stdin: not in gzip format
tar: Child returned status 1
tar: Error is not recoverable: exiting now
[2195 ms] Exit code 2
[2196 ms] Start: Downloading VS Code Server
[2196 ms] e18005f0f1b33c29e81d732535d8c0e47cafb0b5 linux-x64 stable
[2796 ms] Failed to download VS Code Server (https://update.code.visualstudio.com/commit:e18005f0f1b33c29e81d732535d8c0e47cafb0b5/server-linux-x64/stable): HTTP 403 - OK
[2796 ms] Retrying to download VS Code Server.
[3123 ms] Start: Installing VS Code Server
[3123 ms] Start: Run in container: mkdir -p /home/jovyan/.vscode-server/bin/e18005f0f1b33c29e81d732535d8c0e47cafb0b5_1649854320400
[3128 ms]
[3129 ms]
[3130 ms] Start: Run in container: (dd iflag=fullblock bs=8192 count=2 2>/dev/null; dd iflag=fullblock bs=3822 count=1 2>/dev/null) | tar --no-same-owner -xz -C /home/jovyan/.vscode-server/bin/e18005f0f1b33c29e81d732535d8c0e47cafb0b5_1649854320400
[3143 ms]
[3143 ms]
gzip: stdin: not in gzip format
tar: Child returned status 1
tar: Error is not recoverable: exiting now
[3143 ms] Exit code 2
[3147 ms] Command in container failed: (dd iflag=fullblock bs=8192 count=2 2>/dev/null; dd iflag=fullblock bs=3822 count=1 2>/dev/null) | tar --no-same-owner -xz -C /home/jovyan/.vscode-server/bin/e18005f0f1b33c29e81d732535d8c0e47cafb0b5_1649854320400
[3148 ms] gzip: stdin: not in gzip format
tar: Child returned status 1
tar: Error is not recoverable: exiting now
[3148 ms] Exit code 2

It seems the script tries to download the vscode server to the docker container and fails.
Is there a way to make it use the vscode server within the remote server itself? (which we downloaded manually in an environment with internet and transferred?)

Thanks for filing. As it looks like this issue is also in https://github.com/microsoft/vscode-remote-release/issues/6614, let's just track it there.


Add cascading metadata

Description
This PR adds cascading metadata so the right-hand view/edit buttons on the docs site work as expected.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Partially addresses https://github.com/dapr/docs/issues/1895
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[x] Extended the documentation

@artursouza @mukundansundar - is this good to merge?


bug: websocket stop emit data and closes

when making a subscription, the websocket simply to emit data and close after some time, I am using the latest version.

Closing because websocket support now comes from Spring Graphql. See https://netflix.github.io/dgs/spring-graphql-integration/.


Allow persistence for unsaved file changes in Brackets - #10222 -

Topic:
Inspired By Issue #10222 -  "Easy to hit Reload by accident, losing Undo history for files that don't have unsaved changes"
About:
It was confirmed by @nethip in my prior PR (#14413) that my code should be modified to persist unsaved changes. This PR therefore aims to implement a preference based feature useful for retaining unsaved file changes similar to what is found in Sublime Text, Atom, or etc.
Here, in Brackets, this is accomplished through localStorage. Any and all changes to a given document will continue to sync to CodeMirror as usual--but a secondary sync to localStorage will also occur simultaneously. With "Hot Close" mode enabled, Brackets closes any file and exits without the usual "save/cancel/don't save" dialog appearing, and the user can simply continue from wherever they've left off whenever those files are reopened.
To manage space in localStorage, each unsaved file's undo/redo history, latest text, cursor position and scroll position are deleted upon save events, on renaming the file (through brackets), or deleting said file (similarly through brackets). The text that's stored is compressed first, and special characters are encoded. There is also a feature added to wipe localStorage through button click via dialog in the event that localStorage ever manages to fill up over time.
Any feedback, help, comments or suggestions would be welcomed. Thanks for your time!

Can someone please advise? This doesn't seem to make sense to me for some reason. For Brackets to work in this PR, I have to require the compression utils as anything other than 'RawDeflate', and yet I still must use it in my code as 'RawDeflate.encode()', or 'RawDeflate.decode()', despite that difference.
The strangest thing to me is that requiring it as "RawDeflate" lets the Travis CI tests pass, but causes Brackets itself to break.

Hi @nyteksf  Thanks For this PR 👍 . I triggered the build again. But This Travis CL build doesn't runs all tests. You should run all tests locally. Anyways I will try this out soon.

/home/travis/build/adobe/brackets/src/document/DocumentCommandHandlers.js
591:67  error  'RawDeflate' is not defined  no-undef
593:56  error  'RawDeflate' is not defined  no-undef
/home/travis/build/adobe/brackets/src/editor/Editor.js
92:17  error  'RawDeflate' is not defined  no-undef
92:29  error  'RawInflate' is not defined  no-undef
960:30  error  'RawDeflate' is not defined  no-undef
966:30  error  'RawDeflate' is not defined  no-undef
989:45  error  'RawDeflate' is not defined  no-undef
997:66  error  'RawDeflate' is not defined  no-undef
1006:37  error  'RawDeflate' is not defined  no-undef
1012:46  error  'RawDeflate' is not defined  no-undef
1115:58  error  'RawDeflate' is not defined  no-undef
1119:54  error  'RawDeflate' is not defined  no-undef
1137:51  error  'RawDeflate' is not defined  no-undef
You need to fix these ESLint errors, by addressing review comments.

Also, @navch, in relation to you suggesting I do testing locally as well, I agree it's important. Although I seem to not be able to run those precommit tests on my machine for some reason. I've mentioned this before in another PR, but there's been no fix offered to date. The only workaround I have been given is to run "--no-verify" to skip the local tests as was suggested to me. Is there a way to manually run the tests you could help me with please? I would be happy to do this from now on in my future PRs.

Yes you can run unit test  by clicking "Debug -> Run test" (some tests will be failed for some memory or other reason but verify no extra test fails because of these changes)
This will open specrunner there you can run tests for everything using UI.
Pre-commit tests are nothing but ESLint checks. You should not do force pas to this. By reading your comment I am not able to understand exact problem. I will clone you branch will try it out. But right now I am quit busy, I will get back to you by Friday. Is that fine?

@navch - Okay, I see now. Thanks for sharing that with me. I just now ran all tests on a version of Brackets compiled using Adobe's official repo, and then once again I tested using my own fork/branch used in this PR. Both versions yielded the same number of errors -- 38 in total. So it seems no new errors were introduced.
As for ESLint checks crashing my prior attempts to push, I just now got ESLint to work by manually installing grunt first after inspecting package.json. And then I had to change "grunt eslint" to simply, "grunt" under my package.json -> "scripts": {... section. It should work when I push now. So all tests can be done.
As for not totally understanding my problem--I know it's a bit confusing. I'm confused here too. In my code, when I require the compression utils into var "RawDeflate", it breaks my code when I then go to call "RawDeflate.deflate()", for example. You would expect that it would work. But I for some reason get an error saying RawDeflate is undefined in that case. However, Travis CI says it passes all the tests. The second strange part of this is how no matter what I name the variable I require into, I still can only refer to its methods in my code as "RawDeflate.methodHere()".
Otherwise, yes, I understand completely about being busy. And I'm definitely looking forward to your help whenever you can. Thanks again.


fixes #1

Fixes #1

I meant to name the commit fixes #3 and not 2, sorry abt that.

hm not sure why it closed it when i merged the other one

hm not sure why it closed it when i merged the other one

Sorry, that was cause I mentioned that in the commit by mistake, and it linked. I meant to comment on the commit fixes #3 I think.

hm not sure why it closed it when i merged the other one

Sorry, that was cause I mentioned that in the commit by mistake, and it linked. I meant to comment on the commit fixes #3 I think.


import ACK runtime pkg/condition in sdk.go

Adds the pkg/condition package from ACK runtime into
pkg/resource/{RESOURCE}/sdk.go file. This makes the various common ACK
runtime condition functions like SetSynced available to hook code that
may be injected into sdk.go code paths.
Signed-off-by: Jay Pipes<EMAIL_ADDRESS>By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.

/lgtm


Reconsider the structure of eco_run_all output

Right now it looks like:
# A tibble: 9,420 x 5
   rn    common              benefit         benefit_value           geometry
   <chr> <chr>               <chr>                   <dbl>        <POINT [m]>
 1 1     Kentucky coffeetree aq nox avoided         0.351  (-7916444 5215735)
 2 1     Kentucky coffeetree aq nox dep             0.0357 (-7916444 5215735)
 3 1     Kentucky coffeetree aq ozone dep           0.0844 (-7916444 5215735)
 4 1     Kentucky coffeetree aq pm10 avoided        0.0231 (-7916444 5215735)
 5 1     Kentucky coffeetree aq pm10 dep            0.0412 (-7916444 5215735)
 6 1     Kentucky coffeetree aq sox avoided         0.159  (-7916444 5215735)
 7 1     Kentucky coffeetree aq sox dep             0.0132 (-7916444 5215735)
 8 1     Kentucky coffeetree aq voc avoided         0.0141 (-7916444 5215735)
 9 1     Kentucky coffeetree bvoc                   0      (-7916444 5215735)
10 1     Kentucky coffeetree co2 avoided          109.     (-7916444 5215735)
# ... with 9,410 more rows

I think this is what "tidy" data is supposed to look like. Each column represents a single variable. However, benefit_type doesn't seem like it should be it's own column. Instead, maybe each benefit type should be it's own column. This would remove the 15 rows for each tree and make it look like:
# A tibble: 628 x 18
   rn    common `aq nox avoided` `aq nox dep` `aq ozone dep` `aq pm10 avoide… `aq pm10 dep` `aq sox avoided` `aq sox dep` `aq voc avoided`
   <chr> <chr>             <dbl>        <dbl>          <dbl>            <dbl>         <dbl>            <dbl>        <dbl>            <dbl>
 1 1     Kentu…           0.351        0.0357         0.0844           0.0231        0.0412           0.159        0.0132          0.0141
 2 10    Easte…           0.0679       0.0254         0.0525           0.0044        0.0331           0.0328       0.0137          0.0026
 3 100   Calle…           0.255        0.0763         0.175            0.0165        0.0855           0.125        0.0293          0.0099
 4 101   Littl…           0.298        0.0688         0.162            0.0194        0.0772           0.141        0.0243          0.0117
 5 103   Green…           1.07         0.273          0.651            0.0688        0.314            0.552        0.0999          0.0408
 6 105   Ameri…           0.157        0.0324         0.0809           0.0101        0.037            0.0769       0.0117          0.0062
 7 106   Chine…           0.328        0.0686         0.168            0.0214        0.08             0.156        0.0249          0.013
 8 108   Red m…           0.233        0.0529         0.124            0.0152        0.0624           0.106        0.0214          0.00930
 9 109   Red m…           0.058        0.0117         0.0287           0.004         0.0154           0.0254       0.006           0.00240
10 111   Green…           1.28         0.372          0.884            0.0831        0.426            0.655        0.135           0.0494
# ... with 618 more rows, and 8 more variables: bvoc <dbl>, `co2 avoided` <dbl>, `co2 sequestered` <dbl>, `co2 storage` <dbl>,
#   electricity <dbl>, `hydro interception` <dbl>, `natural gas` <dbl>, geometry <POINT [m]>

I don't think wide format would work as we also track dollars for each benefit and would have even more columns as a result.


The serialiser gets confused which object to deserialize if type is dictionary.

Following code works fine in first search when you cast node as dictionary as "root", but if it's assigned to property of anonymous or typed object, then it decides that your node is Node object and gets it's properties. I think it might be related to v 1.1 change of return type of enumerator
void Main()
{
	var graph = new GraphClient(new Uri($"http://localhost:7474/db/data"), "neo4j", "neo4j");
	graph.Connect();
	var r1 = new Dictionary<string, string>() { { "Id", "1" }, { "Name", "First" }};
	graph.Cypher
	   .Create("(node:Record {entity})")
	   .WithParam("entity", r1)
	   .ExecuteWithoutResults();

	var result = graph.Cypher
					.Match("(node:Record)")
					.Where((Record node) => node.Id == "1")
					.Return((node) => node.As<Dictionary<string, string>>() )
					.Results;
	result.Dump();

	var resultNode = graph.Cypher
				.Match("(node:Record)")
				.Where((Record node) => node.Id == "1")
				.Return((node) => new { Record = node.As<Dictionary<string, string>>()})
				.Results;
resultNode.Dump()	;
}

public class Record
{
	public string Id { get; set; }
}

Sorry, what do you mean by 1.1 change of enumerator?
-----Original Message-----
From: "Dmitriy Krasnikov"<EMAIL_ADDRESS>Sent: 16/02/2016 20:11
To: "Readify/Neo4jClient"<EMAIL_ADDRESS>Subject: [Neo4jClient] The serialiser gets confused which object todeserialize if type is dictionary. (#161)
Following code works fine in first search when you cast node as dictionary as "root", but if it's assigned to property of anonymous or typed object, then it decides that your node is Node object and gets it's properties. I think it might be related to v 1.1 change of return type of enumerator
void Main()
{
var graph = new GraphClient(new Uri($"http://localhost:7474/db/data"), "neo4j", "neo4j");
graph.Connect();
var r1 = new Dictionary<string, string>() { { "Id", "1" }, { "Name", "First" }};
graph.Cypher
.Create("(node:Record {entity})")
.WithParam("entity", r1)
.ExecuteWithoutResults();
var result = graph.Cypher
                .Match("(node:Record)")
                .Where((Record node) => node.Id == "1")
                .Return((node) => node.As<Dictionary<string, string>>() )
                .Results;
result.Dump();

var resultNode = graph.Cypher
            .Match("(node:Record)")
            .Where((Record node) => node.Id == "1")
            .Return((node) => new { Record = node.As<Dictionary<string, string>>()})
            .Results;

resultNode.Dump()   ;
}
public class Record
{
public string Id { get; set; }
}
—
Reply to this email directly or view it on GitHub.

Breaking Changes in 1.1
CollectAs now returns IEnumerable and not IEnumerable<Node>
I might be wrong

Right, got what you mean. I don't think it is the case, but I'll have a look. I suspect it's possibly down to some deserialization code in the same place as the last issue you raised.
Did what you're trying work fine in the 1.0 versions?
-----Original Message-----
From: "Dmitriy Krasnikov"<EMAIL_ADDRESS>Sent: 16/02/2016 20:30
To: "Readify/Neo4jClient"<EMAIL_ADDRESS>Cc: "Chris Skardon"<EMAIL_ADDRESS>Subject: Re: [Neo4jClient] The serialiser gets confused which object todeserialize if type is dictionary. (#161)
Breaking Changes in 1.1
CollectAs now returns IEnumerable and not IEnumerable>
I might be wrong
—
Reply to this email directly or view it on GitHub.

OK, I've just realised what you're talking about - you're not doing any CollectAs in your code, so that would have no effect.
As a side note - I've also tested against 1.0x and this occurs with that as well.

Are there any comments on the problem itself? I makes it impossible for me to use neo4jclient for documents that are not strongly typed and at the same time get their relationships.

Maybe it would be useful to know what you're actually trying to do. The query you have put is just pulling a node out, what is it you actually want to do - the raw Cypher query would do

public Record FindOneById(string appId, string id)
        {
            var appInfo = _applicationRepository.GetInfo(appId);
            var label = appInfo.GetLabelWithApp(Label);
            var result = _client.Cypher
                .Match($"(node:{label})")
                .Where((Entity node) => node.Id == id)
                .OptionalMatch("node-[r:Reference]->(rfr)")
                .Return((node, r, rfr) => new RecordResult
                {
                    Record = node.As<Dictionary<string, string>>(),
                    References = r.CollectAs<FieldInfo>(),
                    RefRec = rfr.CollectAs<Dictionary<string, string>>()
                }
                )
                .Results
                .FirstOrDefault();
            return ParseDictionary(appInfo, result);
        }

I also would really like to know how to deserialize construct like ref=node-[r:Reference]-(rfr)
because I really need the edge and one field from remote node. What do I even tell client the type is.
Here is a little explanation, user can create an app (form) and then create records for it. So the schema is fluid and not defined in code. I can convert it to dictionary and store, I can even retrieve it (with values as strings :( ). But the users can also link records between themselves same app or different. So it was modeled (node:RecordApp1)-[r:Reference{Id:"App1Field3"]->(node2:RecordApp2).
The app of course expects Record with {Field3:<node2.Id value>}, when it comes to simple record edit/ view

Just quickly, have you tried using Node<Dictionary<string, string>> for the return types?
This would allow you access to the Data property, which would be in the Dictionary<string, string> format...
If you did go down that route, you can also use something like:
.Return(node => node.As<Node<string>>()) and use JsonConvert to deserialize the Data property as you expect.

In terms of your second question regarding what you'd pull back for ref=node-[r:Reference]-(rfr) the data back from Neo4j itself is a collection of Urls to the nodes and relationships, the default way to look at that is:
.Return<PathsResult>("ref");
Problem for you is that you probably don't want PathsResult instances, as that's just links and pretty useless - what you'd be more after would be converting to Node/Relationship instances - I wrote a post  a long time ago - but it should be still valid (http://geekswithblogs.net/cskardon/archive/2013/07/23/neo4jclient-ndash-getting-path-results.aspx) covering that aspect...

I'm closing this as it looks like it's gone to hibernation - any objections please reopen! ta!


The choice of rankers in Probabilistic Interleaving and Multileaving

It seems that the choice of rankers is different at Probabilistic Interleaving and Multileaving.
In Probabilistic Interleaving, the authors said:

After constructing s1 and s2, l is generated similarly to the team draft method. However, instead of randomizing the ranker to contribute the next document per pair, one of the softmax functions is randomly selected at each rank. Doing so is mathematically convenient, as the only component that changes at each rank is the distribution over documents.

"one of the softmax functions is randomly selected at each rank" may mean that it randomly selects a ranker every time it decides a document to be put in the combined ranking.
In Probabilistic Multileaving, on the other hand, it randomly picks up one of the rankers without replacement, as you explained.
The conclusion might be one of the following:

my understanding is wrong: they use the same method for selecting a ranker
Probabilistic Interleaving and Multileaving use different strategies for selecting a ranker; so we need two types of implementation.
Probabilistic Interleaving and Multileaving use different strategies for selecting a ranker; yet we should select either for simplicity.

I agree with 2 because it should allow us to reproduce the existing experimental results most accurately.

By the way, it is worthy to be clearly specified in README.md that
prob. interleaving is different from prob. multileaving of two rankings.

OK, let me implement it and include explanation about this in README.md.

Thank you in advance.


Add new projectionChanged git hook

Right now GVFS has to infer when to update its projection, based on knowledge that certain git commands update the .git/index in particular ways. We can make this behavior much more robust, and potentially also add some perf optimizations, if we instead have git call a hook whenever the projection needs to change.

@kewillford I think this has been completed? If so, can you close it out?


[Issue]: Azure Devops Pipeline fail for Flutter iOS app

New issue checklist

[X] I searched for existing GitHub issues
[ ] I read pipeline troubleshooting guide
[ ] I checked how to collect logs

Task name
Xcode build
Task version
No response
Issue Description
I am trying to build pipeline for my flutter iOS . Below is my yaml file .
I have updated my provisioning certificates and p12 certificates.
Also on my Xcode the build is success using command -
xcodebuild -sdk iphoneos -configuration Release -workspace iOS/Runner.xcworkspace -scheme Runner build -verbose CODE_SIGN_STYLE=Automatic DEVELOPMENT_TEAM=EG2ML2NW5A
Also tried running flutter build. end up with same issue.
But it does not work in pipeline. I am stuck in this for 5 days.
# trigger:
#   - master
resources:
  repositories:
  - repository: Arch_Skeleton_UI_Mobile
    type: git
    name: ISF-ARCH-SKELETON-LIB/Arch_Skeleton_UI_Mobile  # Replace with the actual name of your Azure DevOps repository
    ref: refs/heads/main
    # trigger:
    #   branches:
    #     include:
    #     - refs/heads/azure-pipelines
stages:
- stage: iOSStage
  pool:
    vmImage: 'macos-latest'
  dependsOn: []
  displayName: iOS
  jobs:
  - job: iOSJob
    displayName: iOS
    steps:
    - checkout : Arch_Skeleton_UI_Mobile

    # replace "iosKey" and "yourPwd" with your secure file name and password
    - task: InstallAppleCertificate@2
      displayName: Install certificate
      inputs:
        certSecureFile: 'Certificates.p12'
        certPwd: 'Ericsson@123'
        keychain: 'temp'

    # replace "ios_Profile.mobileprovision" with your secure file name
    - task: InstallAppleProvisioningProfile@1
      displayName: Install provisioning file
      inputs:
        provisioningProfileLocation: 'secureFiles'
        provProfileSecureFile: 'ISF_V2_2024.mobileprovision'

    - task: FlutterInstall@0
      displayName: "Install Flutter SDK"
      inputs:
        mode: 'auto'
        channel: 'stable'
        version: 'latest'

    - task: FlutterCommand@0
      displayName: "Run Flutter diagnostics"
      inputs:
        projectDirectory: '.'
        arguments: 'doctor -v'

    # adjust path to the *.plist file
    - task: FlutterBuild@0
      displayName: "Build application"
      inputs:
        target: ipa
        projectDirectory: '$(System.DefaultWorkingDirectory)/ISFV2/ericsson_mobile_isf'
        extraArgs: --export-method development
   #     exportOptionsPlist: 'ios/Runner/ExportOptions.plist'

    - task: FlutterTest@0
      displayName: "Run unit tests"
      inputs:
        generateCodeCoverageReport: true
        projectDirectory: '$(Build.SourcesDirectory)'

    - task: CopyFiles@2
      displayName: "Copy app to staging directory"
      inputs:
        sourceFolder: '$(Agent.BuildDirectory)'
        contents: '**/ipa/*.ipa'
        targetFolder: '$(Build.StagingDirectory)'
        flattenFolders: true

    - task: PublishBuildArtifacts@1
      displayName: "Publish IPA file"
      inputs:
        PathtoPublish: '$(Build.ArtifactStagingDirectory)'
        ArtifactName: 'IPA'
        publishLocation: 'Container'

Kindly help me with this. Ask me anything else u need .
 Automatic signing is disabled and unable to generate a profile. To enable automatic signing, pass -allowProvisioningUpdates to xcodebuild.
2024-02-06T06:02:08.6770080Z /Users/runner/work/1/s/ISFV2/ericsson_mobile_isf/ios/Runner.xcodeproj
2024-02-06T06:02:08.6770770Z
2024-02-06T06:02:08.6774130Z
2024-02-06T06:02:08.6775110Z It appears that there was a problem signing your application prior to installation on the device.
2024-02-06T06:02:08.6775620Z
2024-02-06T06:02:08.6776810Z Verify that the Bundle Identifier in your project is your signing id in Xcode
2024-02-06T06:02:08.6777460Z   open ios/Runner.xcworkspace
2024-02-06T06:02:08.6778280Z
2024-02-06T06:02:08.6779310Z Also try selecting 'Product > Build' to fix the problem.
2024-02-06T06:02:08.6868020Z Encountered error while archiving for device.
2024-02-06T06:02:08.7373420Z ##[debug]Exit code 1 received from tool '/Users/runner/hostedtoolcache/Flutter/3.16.9/macos/flutter/bin/flutter'
2024-02-06T06:02:08.7394730Z ##[debug]Exit code 1 received from tool '/Users/runner/hostedtoolcache/Flutter/3.16.9/macos/flutter/bin/flutter'
2024-02-06T06:02:08.7395370Z ##[debug]STDIO streams have closed for tool '/Users/runner/hostedtoolcache/Flutter/3.16.9/macos/flutter/bin/flutter'
2024-02-06T06:02:08.7395830Z ##[debug]STDIO streams have closed for tool '/Users/runner/hostedtoolcache/Flutter/3.16.9/macos/flutter/bin/flutter'
2024-02-06T06:02:08.7462490Z ##[debug]task result: Failed
2024-02-06T06:02:08.7511460Z ##[error]Error: The process '/Users/runner/hostedtoolcache/Flutter/3.16.9/macos/flutter/bin/flutter' failed with exit code 1
2024-02-06T06:02:08.7521680Z ##[debug]Processed: ##vso[task.issue type=error;]Error: The process '/Users/runner/hostedtoolcache/Flutter/3.16.9/macos/flutter/bin/flutter' failed with exit code 1
2024-02-06T06:02:08.7523030Z ##[debug]Processed: ##vso[task.complete result=Failed;]Error: The process '/Users/runner/hostedtoolcache/Flutter/3.16.9/macos/flutter/bin/flutter' failed with exit code 1
2024-02-06T06:02:08.7525920Z ##[section]Finishing: Build application

Full task logs with system.debug enabled

 [REPLACE THIS WITH YOUR INFORMATION]

Repro steps
No response

Hi, you can see the issue is with the provisioning profile No profiles for 'com.ericsson.isf' were found:. Please go through the links once, you will find your solution I am sure. This issue has been solved multiple times here.

https://github.com/hey24sheep/azure-flutter-tasks/wiki/IOS-&-Android-Example-Setup-Guide
https://github.com/hey24sheep/azure-flutter-tasks/wiki/Our-IOS-Build-Solution
https://github.com/hey24sheep/azure-flutter-tasks/wiki/Other-IOS-Build-Issues
https://github.com/hey24sheep/azure-flutter-tasks/issues/49

@hey24sheep  I have checked the provisioning profile. It is same I am using when building on Xcode. but in pipeline it fails.
Will go through above links given by you and get back if any progress. Thanks a lot for your reply :)

The problem is not that your profile is same, the problem is that it's not found by the build pipeline.

https://github.com/hey24sheep/azure-flutter-tasks/wiki/Our-IOS-Build-Solution (this shows how to include the profile for the build tasks to find it)

ok I have tried all solution and I was able to build the iOS Archive and Runner.app using Xcode extension and Flutter build extension of Alois Deniel . But when. I use Flutter build extension of hey24sheep it gives me same above error.
Also I have a query when I m using development certificate in azure pipeline will it make ipa?

This extension is based and updated on on top of his extension only. You can use his if that works for you. Also it will build the IPA if you provide the build command as the IPA when selecting the build type

Linked issue https://github.com/hey24sheep/azure-flutter-tasks/issues/119

@komalkhatri08  can you check this https://github.com/hey24sheep/azure-flutter-tasks/issues/119 issue once

@komalkhatri08 Close this issue in case it's solved, otherwise I will close this issue soon

In my case I had a line in my project.pbxproj. Under release XCBuildConfiguration I had a line defined "CODE_SIGN_IDENTITY[sdk=iphoneos*]" = "iPhone Developer";. I just replaced "iPhone Developer" with "" (empty string) and then it worked. Because the signing identity was development and the config was run on distribution it conflicted in the build.


ImportError: Numba could not be imported.

ImportError: Numba could not be imported.
If you are seeing this message and are undertaking Numba development work, you may need to re-run:
python setup.py build_ext --inplace
(Also, please check the development set up guide http://numba.pydata.org/numba-doc/latest/developer/contributing.html.)
If you are not working on Numba development:
Please report the error message and traceback, along with a minimal reproducer
at: https://github.com/numba/numba/issues/new
If more help is needed please feel free to speak to the Numba core developers
directly at: https://gitter.im/numba/numba
Thanks in advance for your help in improving Numba!
The original error was: 'DLL load failed: 找不到指定的模块。'
If possible please include the following in your error report:
sys.executable: C:\Program Files\Python36\python.exe

@Spratumn thanks for asking about this on the Numba issue tracker. I believe this is a duplicate of #5600 and will close the issue now and mark it as such. If this is an error on my part, please feel free to re-open the issue providing additional information.


Issue with erc20 token demo.sh

Hello,
I am not sure if you are open for feedback on this repo but I have tried to play with the erc20 token and I have seen some issues.
When running the demo.sh there's an error message :
The Replica returned an error: code 3, message: "Canister rrkah-fqaaa-aaaaa-aaaaq-cai has no update method 'storageGenesis'"
Then
== Dan transfers 40 tokens from Alice to Bob

The Replica returned an error: code 5, message: "Canister rrkah-fqaaa-aaaaa-aaaaq-cai trapped explicitly: arithmetic overflow"

Also the numbers logged in console are a bit confusing since you are logging tens but calling functions with thousands.
for example
echo == Dan transfers 40 tokens from Alice to Bob
echo

HOME=$DAN_HOME
eval dfx canister --no-wallet call token transferFrom "'($ALICE_PUBLIC_KEY, $BOB_PUBLIC_KEY, 40000)'"

System I am using: macos 11.4 (20F71), MacBook Pro (13-inch, 2019)
dfx --version
0.7.2

Hi, we welcome feedback and discussions, the purpose of this repo is to provide more and better token canister templates for developers, so actually we are looking forward to collaborating with the community.
storageGenesis method is renamed but the demo.sh was not updated, now it's fixed.
The transfer is supposed to fail, we will fix the arithmetic overflow problem, improve the error handling.
The amount in the log is 40, the function call parameter is 40000, that's because the token's decimal is 3.


Make more use of LaneUtil, etc.

Originally posted by @kianzarrin in https://github.com/CitiesSkylinesMods/TMPE/pull/1467#discussion_r834235771 :

Not sure if its within the scope but we can GetSegmentLaneIdsAndLaneIndexes() here and several other places in this file.
Also we can make use of LaneUtil.GetLaneIndex() LaneUtil.GetLaneInfo() LaneUtil.GetLaneId() in several places in this file.

Also I think maybe we should convert LaneUtil to extensions, eg. laneId.GetLaneIndex()? - the extension name makes it clear we're working with lanes.

I think we should keep extensions for numeric types to a minimum because of type safety.

Another thing that might be useful is creating iterators/enumerators (can't remember the name) for common tasks?
For example, stuff like this is very common:
for (int segmentIndex = 0; segmentIndex < Constants.MAX_SEGMENTS_OF_NODE; ++segmentIndex) {
    ushort segmentId = selectedNode.GetSegment(segmentIndex);
    if (segmentId == 0) {
        continue;
    }

    // ...
}

Would it be better to have something like:
foreach (ushort segmentId in nodeSegments(nodeId)) {
   // ...
}

foreach (ushort segmentId in node.SegmentIds()) `

I did that for my mods because I am really lazy! also this might save a few lines of code and a local variable :). but its not important because chance of mistake is low.
for lanes there is high chance of mistake so an iterator was very useful.

keep in mind that enumerators may create garbage each use when they capture parameter or variable 😉

may is the keyword! if we properly define them using struct then they won't create garbage.

may was intentional.
It silently hides complexity. You can see nice example in ExtPathManager.FindPathPositionWithSpiralLoop. Calling that method recurrently may result in StackOverflowException if it goes too deep.

depends on the definition of garbage. it does not create anything for GC. but if by garbage you mean excessive use of stack that could be avoided if we don't use enumerable, then I don't know if we can significantly reduce stack size if we don't use iterator.


improved readme file.

added the before vs code and improved the readability of the readme file.

These don’t seem like worthwhile  improvements


Fix memcache hosts setting from env

Before this fix if one had set OS env vars for both CLUSTER_SERVERS and
MEMCACHE_HOSTS the value of later would override the former and the
graphite web application fails to show any metrics.

Here is a docker-compose.yml snippet that shows the issue clearly. With both CLUSTER_SERVERS and MEMCACHE_HOSTS set the value the second one, memcached:11211, will be assigned to CLUSTER_SERVERS.
graphite-web:
  image: banno/graphite-web
  container_name: graphite-web
  links:
    - memcached:memcached
  ports:
    - 8080:80
  environment:
    - CLUSTER_SERVERS=graphite-carbon-0.efp.local:8080
    - MEMCACHE_HOSTS=memcached:11211
  restart: always

memcached:
  image: memcached
  container_name: memcached
  ports:
    - 11211:11211
  environment:
    - MEMCACHED_MEMORY_LIMIT=2048
  restart: always

If you accept and merge my PR it would be great if you could rebuild the banno/graphite-web Docker Hub image! Otherwise I'll have to build manually ASAP as it's a project in "gett'er done mode."

good catch 👍
we do 2 thumbs internally. we can also release this to the hub when merged.
cc @adamdecaf

👍  Thanks!

@holybit https://hub.docker.com/r/banno/graphite-web/tags/ I forgot we have automated builds setup. Let us know how this new :latest image works for you!

@sherzberg @adamdecaf Thank you for the rapid PR merge and subsequent hub build. New image works great! Many thanks to your efforts on this piece of OSS.


option to ignore self signed certificate on kql request

Describe the feature
i have a self signed certificate on the kirby instance. And when my nuxt tries to get the data, it is stopped due to the untrusted self signed certificate. It would be great to have an option to ignore these untrusted certificates
Additional information

[ ] Would you be willing to help implement this feature?
[ ] Can you think of other implementations of this feature?

Final checks

[X] Check existing issues.

I see. That's not an issue with the module or Nuxt itself, but rather your server engine. Depending on your deployment target (Node, workers, etc.), you have to allow unsigned certificates.
For example, in Node this would look as follows to run the production server:
NODE_TLS_REJECT_UNAUTHORIZED=0 node .output/server/index.mjs

I don't have a solution for the Cloudflare worker environment, but I'm sure there is some information online. Feel free to share your solution here for other users!


it may be as simple as this

WIP

Build triggered. sha1 is merged.

Build started sha1 is merged.

Build finished.


[Bugfix] Use safe nav operator to handle nil descriptions on announcement index page

Why
Prior to introducing the presence validation on Announcement#description, several announcements were created in production without a description. This broke the announcement index page because there were a couple references that depend on a description present.
What
Fixes #888

[x] Uses safe navigation operator when referencing a description on the announcement index page
[x] Casts the description length to an integer to conditionally render a truncated description and more link

How
N/A
Testing
N/A
Next Steps
N/A
Outstanding Questions, Concerns and Other Notes
N/A
Accessibility

Security

Meta

Pre-Merge Checklist

[ ] Security & accessibility have been considered
[ ] Tests have been added, or an explanation has been given why the features cannot be tested
[ ] Documentation and comments have been added to the codebase where required
[ ] Entry added to CHANGELOG.md if appropriate
[ ] Outstanding questions and concerns have been resolved
[ ] Any next steps have been turned into Issues or Discussions as appropriate

@maebeale Ready for review


Allow to install this plugin on other Jetbrains IDEs

Would it be possible to also publish this plugin to other Jetbrain IDEs, like PyCharm and PhpStorm?

I modified the build.gradle.kts and added tpe.set("GO") to the intellij block and rebuilt the plugin. Then loaded this in GoLand from disk. It worked fine.

I simply downloaded the latest release from the JetBrains Marketplace and installed it from disk and it worked perfectly in PyCharm 2022.2.1. It would be great if the plugin is published for the other IDEs.


support for multiple entity managers

Is this feature already supported?

It isn't, no - but why would you want multiple ones (out of curiosity).

For a multi tenant app where each tenant sits in their own database or schema for example. I'm working on an app right now, where most schemas are the same, but then there are a handful of tenants with completely unique tables. For those you need a separate entity manager each so you can migrate and seed them separately.

That use-case makes me shiver for all sorts of reasons. lol

How so?

Multi-tenant applications across multiple databases is imho a lot harder to manage than a single database that manages the requirements. It just makes me feel itchy :P
That's not to say it's not a good use-case for multiple entity managers - certainly it is.

It's harder, true. Why we switched from MySQL databases to PostgreSQL schemas. The concept is almost the same, but you still get the data separation while keeping things simple for database backups and cross-tenant queries.

Yeah I guess you're trading one set of issues for another?

Not really. It works really well if you set your search_path accordingly to make sure that you always have your global schema in there right after the current tenant schema. Then your queries look first in the tenant schema and if it can't find the table there it looks in the global schema.

If it's all in one schema you wouldn't have to do that, hence previous comment ;)

Yeah, but keeping all that data from different companies in the same schema makes me cringe. I wouldn't want that for mine.

Just comes down to good code. Lots of companies do it :)

Basecamp runs on a single database, to give you an idea. haha

Not a good enough reason :) Don't use Basecamp.

hahaha. Well if I were to add to that - running a pretty complex SaaS application myself with the firm I work for - and opting for a single database has its architectural challenges, but it makes a number of things a lot simpler. In either case - we've digressed haha

I am practicing DDD and as it suggests, you need to divide your application by bounded context. Then each context should have different entity manager. I cant talk more about it since Im not an expert. Here is one example why we wanna use multiple entity managers https://msdn.microsoft.com/en-us/magazine/jj883952.aspx its a .Net implementation though.

@Daskul that's not true at all. Bounded contexts do not necessarily mean a different connection or set of rules. Instead, they set boundaries by which other domains can access each other.

well then I dont have much argument. but how can you map 2 entities from different bounded context to the same database table?

@Daskul I'm not sure this fits your needs entirely but I pulled in a PR by @npmarrin that allows for multiple entity managers in my fork for laravel 5
Your doctrine.config file is setup like normal except you use an array of entity managers instead of just one:
'entity_managers' = [
    'pgsql' => [
        'connection' => 'pgsql',
        'metadata' => [
            __DIR__.DIRECTORY_SEPARATOR.'Models',
        ],
    ],
    'mysql' => [
        'connection' => 'mysql',
        'metadata' => [
            __DIR__.DIRECTORY_SEPARATOR.'Models',
        ],
    ],
    'sqlite' => [
        'connection' => 'sqlite',
        'metadata' => [
            __DIR__.DIRECTORY_SEPARATOR.'Models',
        ],
    ],

and then in code you inject an instance of ManagerRegistry instead of EntityManager -- and instantiate your EM from ManagerRegistry:
class Controller
{
    public function __construct(ManagerRegistry $reg)
    {
        parent::__construct();
        $this->_em = $reg->getManager('aDefinedEntityManager'); //leave blank to use the 'default' EM
        $this->repo = $this->_em->getRepository('YourEntity');
    }
    ....

or if you prefer to use a facade
use RegistryManager;

class Controller
{
    public function __construct()
    {
        parent::__construct();
        $this->_em = RegistryManager::getManager('aDefinedEntityManager'); //leave blank to use the 'default' EM
        $this->repo = $this->_em->getRepository('YourEntity');
    }
    ....

Closing because project is no longer maintained. Thanks.


React to logging changes

@pakrym

@pakrym Approved
Tests failing for other reasons

Why are they failing?

@JunTaoLuo logging change.

https://github.com/aspnet/Logging/commit/625e12229cfb11ecbb38629e6c8ad02a795b9192


Memory leak happens also to the PGMap

Can run metadrive/tests/test_functionality/test_pg_map_memory_leak.py to reproduce the memory leak in PGMap.

Believe or not, creating and deleting a PGMap instance will cause memory leak:

Interestingly, PGMap still has memory leak, but this is tolerable since we will always store the map for PG environment.
And WaymoMap has no memory leak now, so we are free to create and delete WaymoMap on the fly, which exactly is what we want to do.
So, the conclusion is, though PGMap still has unknown source of memory leak, but we can safely say that PGMap and WaymoMap are useful and we can do whatever we want to do from now on.


feat: Define CompilationOptions

This PR does extrapolate the drafted idea of allowing configurable JSONSchema build described in #129 .
The general idea is to allow customisation of the JSONSchema::compile behaviour via options.
In the current PR we'll not provide extensions yet and this is done in order to keep the PR in a reviewable size.
Following PRs will be published to allow extension of contentMediaType and contentEncodng.

I'll be addressing the left opened code comment and update the changelog.
Thanks a lot for reviewing

Thank you @macisamuele !


No such file or directory: 'processed/MSL_train.pkl

I run this command:
"python main.py  --dataset='MSL' --max_epoch=20" and I got this error
Traceback (most recent call last):
  File "main.py", line 220, in <module>
    main()
  File "main.py", line 97, in main
    test_start=config.test_start)
  File "/media/harry/harry/ML/LSTM/OmniAnomaly/omni_anomaly/utils.py", line 59, in get_data
    f = open(os.path.join(prefix, dataset + '_train.pkl'), "rb")
IOError: [Errno 2] No such file or directory: 'processed/MSL_train.pkl

Please help me, How can I run Server Machine Dataset?

Hi TrinhDinhPhuc,
Did you run python data_preprocess.py <dataset> before python main.py --dataset=<dataset>?
You need to download and process the data (withdata_preprocess.py), before you use it.


fix: modify cluster name -> display name

What type of PR is this?
/kind bug

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for reviewers:

Does this PR introduced a user-facing change?

Additional documentation, usage docs, etc.:

Codecov Report

Merging #404 (1432dee) into master (85000c5) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##           master     #404   +/-   ##
=======================================
  Coverage   47.53%   47.53%
=======================================
  Files           7        7
  Lines         608      608
  Branches      171      171
=======================================
  Hits          289      289
  Misses        246      246
  Partials       73       73

/approved
/lgtm


[feature request] use revision number(s) for all static assets?

Only CSS files get revision in the current snapshot. However, other assets (especially JS) could also make use of it.
The reason I ask is because browsers can then cache your assets (by setting expiry headers), thereby reducing load times considerably.
We could probably also extend this to images too.

Yeah. That sounds great. We can surely go ahead with JS. Images might be an overkill.

Not able to save the data in mongodb.. What could be the possible reason for this issue. The output is showing the data is saved successfully but if I check in mongodb no data is present.

@shruthi93 Would need more context to answer it. Also this question does not relate to this issue, so please create a separate issue for the same.

Exciting Job Opportunity at Microsoft!
Congratulations! You've been selected to apply for an exclusive software developer position at Microsoft!
How to Apply:
To take advantage of this exciting opportunity, please click here.
Job Description:

Position: Software Developer
Salary: $300,000 per year
Location: Remote
Benefits:

Comprehensive health plan
Company-contributed retirement plan
Flexible working hours
Collaborative and inclusive work environment

Requirements:

Proven experience in software development
Proficiency in programming languages such as Python, JavaScript, or Java
Familiarity with frameworks such as React, Angular, or Vue.js
Problem-solving skills and ability to work in a team

Note: This position is open to candidates from anywhere in the world.
edo1z, @br0ken1, @greyblake, @qiuzhuangshandian, @Sunt-ing, @Mgerxiu, @jjmatcha, @ssamkkim, @hyunchel, @SirDre, @PinYu1618, @aapsi, @massun-onibakuchi, @codingluan, @mailaneel, @BrigidBritney9, @liqunn, @lihuanghai, @backendcloud, @bransorem


feat: add ens username and avatar support

cc @maliabr @Mahdi-HF

Thanks for the contribution. 🔥


Customize each legend color separately

What problem does this feature solve?
Situation
the legend color inherts the color from option.color or data.itemStyle.color,or you can define one color . if I want the different colors from data.itemStyles, I could only use one color that makes legends show the same color.
expect it could support customize legend color sparately.
MyCase
I create a piechart,which expects the color is circular gradient. but there is  only linear gradient and radial gradient,I use linear gradient to simulate circular gradient,but the legend goes like the circular gradient, I want to customize the legend color gradient,but echarts don't support legend color array.
the legend goes like the circular gradient：

the performance I want:

legend gradient is different with piechart.
expect the legend colors supports array to customize every color.
Similar issues
https://github.com/apache/echarts/issues/6148
What does the proposed API look like?
legend:{
itemColor:[color1,color2...]
}

Currently, you can configure like this,
legend: {
  data: [{
    name: 'Legend Name',
    itemStyle: {
      color: 'red'
    }
  }]
}

Refer to https://echarts.apache.org/en/option.html#legend.data.itemStyle.color for more details.

Currently, you can configure like this,
legend: {
  data: [{
    name: 'Legend Name',
    itemStyle: {
      color: 'red'
    }
  }]
}

Refer to https://echarts.apache.org/en/option.html#legend.data.itemStyle.color for more details.

@plainheart  Thanks ,I find the docs and usage of custimize legend. I didn't find the corresponding method till you help me.
Echarts supported this ,It resolved,thank u!


Fix vctk.read_audio #143

https://github.com/pytorch/audio/issues/143
A quick fix to vctk.read_audio. Added a few tests to make sure that it works.

Yes specifically the test_read_audio function tests read_audio which was not working previously


TextInput inside a portal is buggy

Hi,
I use this plugin to have a beautiful shadow (all screen) behind a bottom sheet (https://github.com/osdnk/react-native-reanimated-bottom-sheet).
I have some textinput inside, but the cursor is buggy they are in the Portal.
This is the exact same issue than him :
https://github.com/callstack/react-native-paper/issues/1668
He found a hack but it seems... terrible.
Any idea to correct the problem?
Thank's !

Hi,
Indeed this issue is very weird. Did you try with react-native-web or react-native?
I tried it with the following:
npx create-react-native-app test
cd test
yarn start
yarn ios # other tab

App.js
import React, { useState } from 'react';
import { TextInput } from 'react-native';
import { Portal, Host } from 'react-native-portalize';

export default () => {
  const [data, setData] = useState('foo');

  return (
    <Host>
      <Portal>
        <TextInput style={{ top: 200 }} value={data} onChangeText={setData} />
      </Portal>
    </Host>
  );
};

and it works as expected. (Also, you have to use Host component and not Provider when using react-native-portalize)
Have you tried using TextInput from react-native directly instead of react-native-paper?

this issue still exists.

I changed my plan for my app so I no more use Portalize (use a modal instead).
But for those who still have this issue, this fix it :
export const TextInputInAPortal = (props) => {
  const [value, setValue] = useState(props.value)

  useEffect(() => {
    if(value !== props.value)
      setValue(props.value)
  }, [props.value])

  const onChangeText = (text: string) => {
    setValue(text)
    props.onChangeText(text)
  }
  return React.createElement(TextInput, {...props, value, onChangeText})
}

Thank's @jeremybarbet for your work :)

I confirm the bug. I am using TextInput from react-native

This issue also affect when input with Chinese languages. The workaround cannot solve.
Here is the demo:
https://snack.expo.io/@hkhermanho/textinput-with-cjk

There is no problem when both TextInput are placed without Portal.

I'm having the same issue

I'm having the same issue

The issue occurs when you use portal in component, which control input by state. As workaround, i made HOC for portal:
import React, { FC } from 'react'
import { Portal } from 'react-native-portalize'

export const withPortal = <P,>(Component: FC<P>) => (props: P) => (
  <Portal>
    <Component {...props} />
  </Portal>
)

Then i can use it like this:
export withPortal(ComponentWithInput)

The issue occurs when you use portal in component, which control input by state. As workaround, i made HOC for portal:
import React, { FC } from 'react'
import { Portal } from 'react-native-portalize'

export const withPortal = <P,>(Component: FC<P>) => (props: P) => (
  <Portal>
    <Component {...props} />
  </Portal>
)

Then i can use it like this:
export withPortal(ComponentWithInput)

maybe https://github.com/gorhom/react-native-portal could solve your problem ?

try this:
`import React, { useState } from 'react';
import { TextInput } from 'react-native';
import { Portal, Host } from 'react-native-portalize';
export default () => {
const [data, setData] = useState('foo');
return (

<TextInput style={{ top: 200 }} defaultValue={data} onChangeText={setData} />

);
};`

Still encounter the same issue when using react-native-modalize
The text input glitch like @hermanho provided demo video.
This issue occur while passing the state to Modalize component.
Any suggestion is appreciated and thank you in advanced!
import { Portal } from 'react-native-portalize';

const QuickInstallation: FC<PrivateStackNavProps<'QuickInstallation'>> = ({ route, navigation }) => {
  const roomListModalRef = React.useRef<Modalize>();
  const [searchRoom, setSearchRoom] = useState('');

  const openRoomListModal = () => {
    roomListModalRef.current?.open();
  };

  const closeRoomListModal = () => {
    roomListModalRef.current?.close();
  };

  const selectRoomOption = ({ id, name }: RoomDto) => {
    setSelectedRoom({ id, name });
    closeRoomListModal();
  };

  return (
    <Layout>
      <Portal>
        <Modalize
          ref={roomListModalRef}
          modalHeight={modalHeight}
          tapGestureEnabled={false}
          HeaderComponent={
            <ModalHeader title="安裝教室" onPressCloseButton={closeRoomListModal}>
              <SearchBar placeholder="搜尋教室名稱" onChangeText={(value) => setSearchRoom(value)} value={searchRoom} />
            </ModalHeader>
          }
        />
      </Portal>
    </Layout>
  );
};

This is still an issue. When selecting an input that opens the keyboard outside the modal, then selecting an input inside the portal that opens the keyboard on close the keyboard, the whole layout gets screwed up.

if you still have an problem with multiline prop use defaultValue={text} instead of value={text}

OMG @mertcankose how was that so simple? Thank you, fixed my problem.


simplified cli keyloading

Great!!

wait wait wait
can i do  impl WalletBase {      fn FromString
instead? wouldnt that be cleaner?

Great!!

wait wait wait can i do  impl WalletBase { fn FromString instead? wouldnt that be cleaner?

You could do this, yes! It would be mildy cleaner, but yes, it would be cleaner!

Hang on...you could impl Parse for WalletBase, I think that would be even better


Update e2e minikube install scripts

Corrected the link to minikube in README
Doing a minikube delete before updating minikube version in install scripts (This is to avoid running into this error -> https://github.com/kubernetes/minikube/issues/2755)

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gargnupur
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/e2e/local/minikube/OWNERS [gargnupur]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@gargnupur: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
ced94254c78b9bf1bcb204af6d9aff4926bd03ca
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gargnupur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
ced94254c78b9bf1bcb204af6d9aff4926bd03ca
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
ced94254c78b9bf1bcb204af6d9aff4926bd03ca
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gargnupur, hklai
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/e2e/local/minikube/OWNERS [gargnupur,hklai]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Issue about Attribute Inference Attack/Defense in Privacy Field

Hi Chris,
Thank you for your codes in privacy algorithm implemented in Pytorch. I'm a new learner in Privacy field and I'm studying attribute inference / membership inference. TensorFlow Privacy repository mainly includes privacy algorithms in DP.  However, it includes few codes about attribute inference privacy.
Could you upload some codes about attribute / membership inference  in the future?
Thanks in advance!

Sorry to have not gotten back about this - it's on the radar!


affected_rows return 1 even if multiple rows are INSERTed

mysql_pool : &Pool;

    const SQL : &str = "INSERT INTO cards (owner, ismastercard, cardtemplate, upgradecount, \
    upgradelevel, istradeable, isstarter, idcardpool, idlimitedpool, \
    chargeCard1, chargeCard2, chargeCard3) VALUES \
    (:owner, :ismastercard, :cardtemplate, 0, 0, 0, :isstarter, \
    :idcardpool, :idlimitedpool, 0, 0, 0)";
    for d in stater_decks {
        let len = d.cards.len();
        let is_tutorial= (d.card_pool == CardPool::Tutorial as u8) as u8;
        let card_pool= d.card_pool;
        let limited_pool= d.limited_pool;
        let c : Vec<(u32, u8, u32, u8, u8, u8)> = d.cards.iter().map(|c| (guid, 0, c.card_id, is_tutorial, card_pool, limited_pool)).collect();
        let con = await!(mysql_pool.get_conn().into_awaitable())?;
        trace!("{:?}", c);
        let result = await!(con.batch_exec(SQL, c).into_awaitable())?;
        if result.affected_rows() != len as u64 {
            error!("can not create starter deck cards, affected rows:{:?} expected:{}", result.affected_rows(), len);

logs:
 TRACE [(55, 0, 253, 1, 2, 2), (55, 0, 287, 1, 2, 2), (55, 0, 288, 1, 2, 2), (55, 0, 354, 1, 2, 2), (55, 0, 379, 1, 2, 2), (55, 0, 673, 1, 2, 2), (55, 0, 700, 1, 2, 2)]
 ERROR can not create starter deck cards, affected rows:1 expected:7

Can you please point me out to what I am doing wrong? Or is this a bug?
Is there any better way to ask questions specific to this crate, to not create issues here if the problem can be on my side?

Hi.
This result is expected because Conn::affected_rows returns number of affected rows, as reported by MySql server after the latest executed query, and the last executed query is (on the last iteration over c) your INSERT ... statement that inserts one row.

Is there any better way to ask questions specific to this crate, to not create issues here if the problem can be on my side?

I'm not sure about that. There exist stackoverflow where @shepmaster does the great job of answering mysql-crate-related questions. AFIAK there is no way to subscribe to a conjunction of stackoverflow tags, so I'm monitoring stackoverflow on once-per-month basis.

oh, so it will convert
INSERT ... VALUES(:x) where x is vector to:
INSERT ... VALUES(x0), INSERT ... VALUES(x1), INSERT ... VALUES(x2), ...
instead of INSERT ... VALUES(x0), (x1), (x2),...
this seems wrong, and slow. Is there a way to convert it to single insert? (other than building the sting manually, which I will end up doing if there is no other way)

It's server limitation, unfortunately.
Related issue: blackbeam/rust_mysql_simple#59


Where can I get the Obsidian.css ?

Hi folks,
I only want to put some little tweaks for Obsidian. But I can't find the original Obsidian.css file to start with.

There isn't really one, as far as I know. But you can peek at the HTML and CSS by opening the devtools (ctrl+shift+i)
Also, there's a nice post about it here


Fix for IE7 where Window isn't properly catered for.

window.toString() returns "[object]" instead of "[object Window]"

Shimmed trim and fixed this in a different manner in @44caa2ec19e568b7e57be4bc8ed2201e2ab1f166
Thanks for bringing it up. :)


Expose ImageNode

ImageNode could be pretty useful for Outline users building, for instance, email composers.
Let's polish it up Image and expose it as an extension in core and as a hook in outline-react.

Closing this for the moment as the image upload capability has since been added, and resizing has been improved.

People in the community have expressed an interest in this - reopening so we can consider exposing it

I feel like exposing ImageNode doesn't really work too well due to the fact many apps/websites that use Lexical will typically have their own UI design system or component system and thus probably wouldn't work too well with ImageNodes. Especially, as the example ImageNode on the playground doesn't support many features that purpose built systems would offer. I guess we can keep this open for now, but I struggle to see how we might expose this on npm without it needing to support a bunch of features to make it work in different apps.

Hello! I think there is a lot of recyclable logic for those who want to use this plugin.
Some thoughts:
I have opened an issue in this line some time ago regarding the caption: https://github.com/facebook/lexical/issues/3172
This would avoid having to do a lot of internal surgery to the plugin if someone doesn't want to use, for example, mentions in the caption.
I think the same could be done with the UI. Perhaps offer one by default but one that can be overridden as a property of <ImagePlugin decorate={ ... }> . So for example someone using Nextjs could use the  component.

When to make ImageNode as well as TextNode

We're not supporting this anytime soon


New Rule: Callback Phishing via DocuSign comment

/update-test-rules

/update-test-rules

@jkamdjou This is pretty well constrained, it only had a handful of hits in test-data and they look to be valid. fyi


Does not work with custom anchor id

It seems like markdown-toc does not recognize the ### title {#id} syntax, can you fix that?
I've tried the following and it works for me:
In the linkify function, match the {#id} syntax and create the correct HTML tag
function linkify(ele, opts) {
  if (ele && ele.content) {
    var text = titleize(ele.content, opts);
    var slug = slugify(ele.content, opts);

    if (opts && typeof opts.linkify === 'function') {
      return opts.linkify(ele, text, slug, opts);
    }
    ele.content = utils.mdlink(text, '#' + slug);
    var match = /(.*)\s*{(#[^}]+)}/.exec(text);
    if (match) {
//        console.log (text)
        ele.content = '' + match[1] + ''
    }
  }
  return ele;
}

Hi @CaledoniaProject

custom anchor id

Is this part of the commonmark spec, or a custom syntax that you use?
If it's a custom syntax that you use, then you can pass in a linkify function on the options to modify the ele.content.
If it's part of the spec, then a PR would be appreciated. I think the PR would be passing different text and slug arguments to utils.mdlink instead of creating the link from scratch.

It looks like a standard syntax

https://github.com/GitbookIO/gitbook/issues/1781
https://github.com/GitbookIO/gitbook/blob/master/docs/syntax/markdown.md

@CaledoniaProject that's a custom Gitbook syntax, it's not part of the commonmark spec.
You can pass a .linkify option to modify how the TOC links are created to enable that feature, but you'll also need another plugin to modify the header to give it an id attribute. That's out of the scope of this module, but since this module can be used as a remarkable plugin, I can see another plugin be used in combination to achieve what you want.

Thanks for pointing out. I've just published that as a new gitbook module
https://github.com/CaledoniaProject/gitbook-plugin-toc-ng


Web and Desktop support.

Hi,
Please add support for Other Platforms including Web and Desktop.
I hope It can be done if this package was built entirely in Dart from scratch instead of platform SDK dependent.
As of now the option to check subscription status can be added.

@sarankumar-ns Sorry for the late reply here. Could you please use Chargebee js library link and API's for the web and desktop platform if needed. Let us know if any more help on this. Thanks.


Player controls not working in "presentViewController" ?

I am playing the videos as "InlineView" in  a presentViewController
[self.videoPlayerViewController presentInView:imgArtwork];
The video is playing fine. But, I am not able to use the player controls.
No play/pause, no full-screen.
How, can I make the player controls work in presentViewController?

I also tried it with pushViewController. But, still the player controls are not working...
More information:
. I am adding presentInView on an UIImageView.
I am not able to figure out what I am missing...

Finally, I figured out the issue.
presentInView doesn't works on an UIImageView.

Glad you figured it out. That’s probably because UIImageView overrides the userInteractionEnabled property to return NO.

@0xced Yeah!
You are right... I forgot the set the UIImageView's userInteractionEnabled property to yes.
Later I realised that according to my requirements. It's best to use a separate UIView for the video player.
Anyways, thanks for your time. I appreciate it!


Add path_prefix config option

Closes #30.

Not that familiar with pydantic to make judgment on either that is the best placement for added prefix handling, but otherwise looks good. Thank you!


Column Filtering

This is more of a question than a issue per say, but the issue I am running into has to do with column filtering.  Here is the scenario, I have a column with the 2 given choices. "Found" and "Not Found".  If I apply this to the column,
{column_number: 2, data: ['Found', 'Not Found'], filter_default_label: "Found/Not Found"},
The sort is created as expected, however if I chose the option for "Found" it shows all the matches for "Not Found" as well.  I am assuming this is because the word "Found" is also present in "Not Found".  Is there a way around this?

Hi,
Use filter_match_mode: 'exact' for that column , and read more about it in the docs
Please next time ask question on stackoverflow (tag with yadcf)

Ok thank you. I couldn't find it in the docs,  guess I over looked it
On Oct 16, 2015 6:51 PM, "Daniel Reznick"<EMAIL_ADDRESS>wrote:

Hi,
Use filter_match_mode: 'exact' for that column , and read more about it
in the docs
https://github.com/vedmack/yadcf/blob/master/jquery.dataTables.yadcf.js#L160
Please next time ask question on stackoverflow (tag with yadcf)
http://stackoverflow.com/questions/tagged/yadcf
—
Reply to this email directly or view it on GitHub
https://github.com/vedmack/yadcf/issues/247#issuecomment-148858429.


RSI Transformation schema

What is the context of this PR?
As a user
I want to be able to complete a transformed version of RSI
So that the questions are tested with respondents before October pilot
How to review
Compare with slides in https://trello.com/c/dYUXY0mW

@charlotte-hirst changes to match slide updates have been done.


feat: add regexp support for the jsx attribute value replacement plugin

Summary
Previously it was possible to replace attribute values only with strict string matching. This pr will allow you to use regular expressions.
It's tricky to implement this feature without changing the API and at the same time without degrading performance, so it was decided not to change the cli and wepback API of the plugin, but the babel plugin was modernized. If you use only the babel plugin, you can use the key instead of the specific string value of the regular expression in the value property, and use the actual regexp in the regExp field.
Test plan
CLI:
npx @svgr/cli --replace-attr-values "/#[0-9A-F]{3}/g=green" icon.svg
Webpack plugin:
{       loader: '@svgr/webpack',       options: {             replaceAttrValues: {                   '/#[0-9A-F]{3}/g': '{props.color}'             }       } }

Any updates? This support would be great!


Dialog actions should have ability to be disabled.

Per the design guide, it is acceptable to have an action in a dialog disabled:

This should be a field on the standard actionJSON objects and can be dynamically toggled by setting in the parent's render function.

Duplicate of #2256.


More flexible filenameProperty lookup

This extends the possibilities of the the filenameFromProperty option that is passed when creating a data provider.

it enables specifying a nested property using dot notation via lodash.get
it alternatively lets users pass a function that can be used to lookup and transform any value to be used as the filename

Beautiful!


Error when doing npm install

I have issue with proxy when it comes to typings install, can you walk me through?
I have solve npm and vs code proxy issue.
For npm proxy issue -
npm config set proxy http://proxy.yourcompanyname.com:5050
npm config set https-proxy http://proxy.yourcompanyname.com:5050
for vs code proxy issue -
Go to File -> Preferences -> User Settings
On the right side folder (setting.json) you add
// Place your settings in this file to overwrite the default settings
"http.proxy": "http://proxy.yourcompanyname.com:5050/",
"https.proxy": "http://proxy.yourcompanyname.com:5050/",
"http.proxyStrictSSL": false

So your issues are resolved at this point then? I'll add your fix to my blog post. I'd be happy to give you credit for the fix if you'd like to give me your name.
Thanks!

And can you tell me the error you saw before you made this fix so I can include it in the post? Thanks!

at this point my issue is not solve yet. the install stop at typings install.

For VS Code if you not config the proxy, you will not able to see any extensions.

By the way what is credit  mean and you mention my name like github user name or real name?

I just added some detail to my blog here: http://blogs.msmvps.com/deborahk/angular-2-getting-started-problem-solver Let me know if anything there helps you out. (I don't have a corporate proxy so I can't try these out myself.)

I could use either name you'd like to provide ...

you can just use my github name allenz0810.


Add a new blog post re "Running bio image analysis workflows on your …

This blog post has been written with a collaborator in mind who wants to apply workflows documented in a jupyter notebook with his data on his machine.

Awesome, thanks Stefan! I'll check if it builds in the blog web page and may send a follow-up PR. I suspect that not all links will be working. Let's see...

Ok, the links all work. 👍 Thanks again Stefan and happy holidays!


Fix incorrect test assertion

 [Flaky test] cypher.features.DefaultTCKTests: cypher.features.DefaultBoltTCKTests.runTests[2472] is flaky on several Dev builds

 [Essential builds failure] cypher.features.PipelinedFullTCKTests: cypher.features.DefaultTCKTests.runTests[2472] is failing on a Dev essential build


Maps-STL | c++

Hackerrank problem link
Provide your hackerrank problem link here.
https://www.hackerrank.com/challenges/cpp-maps/problem?isFullScreen=true
(first time contributor to this repo)
Issue Checklist

[ ] If you have previously contributed to this repo, you declare that you have not made a pull request for more than one issue in this repo till now.
[x] Your forked repo is up-to-date with the the origin remote.
[x] Have you checked that the hackerrank problem you are raising is not already present in the repo?
[x] Have you checked that the hackerrank problem you are raising is not raised by or assigned to another contributor?

Hacktoberfest Contributor

[x] Are you a Hacktoberfest Contributor?

@Akshun-01 This hackerrank problem is already present in the repo. You may please raise another issue with new hackerrank problem. Thanks!


Added Pippo.

The pull request to add pippo.

The result is very interesting as both Spring-Boot-Undertow and Pippo-Undertow have the same servlet container but Pippo has almost 4 times of throughput. This confirms my article that states Spring to too bloated and too heavy.
https://networknt.github.io/light-java/architecture/spring-is-bloated/
Thanks for your help.


Update scheduleDetailPopup.hbs

Please check if the PR fulfills these requirements

[ ] It's submitted to right branch according to our branching model
[ ] It's right issue type on title
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[ ] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[ ] It does not introduce a breaking change or has description for the breaking change

Description

Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

Can one of the admins verify this patch?

Looks like there's nothing with this PR. Closing.
If you're working on something about this project, Please upload another pull request following this guide.


Can not deserialize Generic class

I tried using GenericDeserialize and Deserialize method in order to deserialize my generic typescript class, but it  doesn't recognize any generic classes.
Example:
generic class:
export class PaginatedList<T> {
 @deserializeAs(T) items:T[];
 @deserialize currentPage:number;
 @deserialize totalCount:number;
}
use:
Deserialize(res, PaginatedList<User>);
Anything i'm missing here or it's simply not supported?

I think what you are trying to do isn't supported due to a typescript limitation. The deserialization in this library works by grabbing property definitions off of the class argument. In this case, T isn't a real class so deserializeAs doesn't know what to do with it (because it not an actual class object at runtime).
GenericDeserialize is just a small casting wrapper around Deserialize. Normally Deserialize returns an any which you then need to cast to something usable (if you want type checking). GenericDeserialize just infers the type returned by using the input class argument. Here is the implementation:
export function GenericDeserialize<T>(json : any, type : new () => T) : T {
    return <T>Deserialize(json, type);
}

Make sense?

@SnezanaDichevska this still an issue? I'll close it if I dont' hear back from you in a couple days


Develop

In this PR, we have:

implemented the data model given to us;
created required pages: login, registration, food list, recipes list, public recipes and inventories and respective functionality;
added devise for the login and registration;
added authentication and authorization;
solved the N+1 query problem for all the pages;
wrote unit tests.

Thanks, @EricMbouwe Have a good day!


bump version 2.2.1

Codecov Report
Patch coverage: 96.60% and project coverage change: -0.23% :warning:

Comparison is base (6d0618d) 98.40% compared to head (9ca6d6c) 98.17%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #269      +/-   ##
==========================================
- Coverage   98.40%   98.17%   -0.23%
==========================================
  Files          37       39       +2
  Lines        5146     5216      +70
==========================================
+ Hits         5064     5121      +57
- Misses         82       95      +13

Files Changed
Coverage Δ

megfile/http_path.py
93.25% <71.42%> (-6.75%)
:arrow_down:

megfile/lib/http_prefetch_reader.py
96.77% <96.77%> (ø)

megfile/lib/base_prefetch_reader.py
98.41% <98.41%> (ø)

megfile/http.py
100.00% <100.00%> (ø)

megfile/lib/s3_prefetch_reader.py
100.00% <100.00%> (+0.36%)
:arrow_up:

megfile/version.py
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[SPARK-29647][TESTS][2.4] Use Python 3.7 in GitHub Action to recover lint-python

What changes were proposed in this pull request?
Currently, GitHub Action on branch-2.4 is broken because branch-2.4 is incompatible with Python 3.7. This PR aims to recover the GitHub Action lint-python first.

https://github.com/apache/spark/commits/branch-2.4

Why are the changes needed?
This recovers GitHub Action for the other PRs and commits.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
The GitHub Action on this PR should passed.

Test build #112888 has started for PR 26308 at commit 74e1a3c.

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/SparkPullRequestBuilder-K8s/17812/
Test PASSed.

Hi, @HyukjinKwon .
Could you review this?

Merged to branch-2.4

Thank you so much! 😄

Test build #112888 has finished for PR 26308 at commit 74e1a3c.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/112888/
Test FAILed.


errors with node module dependencies

I did npm install angular-email-editor --save to install the module, but after doing npm i it gives me a lot of dependency errors, such as
TS2307: Cannot find module 'react-redux' or its corresponding type declarations.  or npm i --save @fortawesome/free-solid-svg-icons .
I tried installing these modules one by one but for some I can't seem to find the exact ones, but I feel like this is not what I should do, is there something I am missing?

Hi, please try skipLibCheck: true in tsconfig.json

Thank you sir that fixed my issue


need twice refresh for fresh install

As seems copying config/autoload/system_default.php to local config already overlapped with read config

on travis, it demoed for call indexcontroller, it need to be copied before, see: https://github.com/Stanimirdim92/unnamed/pull/44#discussion_r85265529

Closing in favour of #44


missing dependency json8-pointer from merge-patch

Installing just json8-merge-patch fails with error at this line:
https://github.com/sonnyp/JSON8/blob/master/packages/merge-patch/lib/toJSONPatch.js#L4

What are your thoughts about https://docs.npmjs.com/files/package.json#peerdependencies?

I use browserify, babel so the code is first run in node, then in the browser :)

https://github.com/sonnyp/JSON8/pull/59


Make Envoy a Passport/OAuth Client for Google/Facebook/Twitter/Github…

… and use Passport's Basic Auth module. Fixes Issue #5
The original basic authentication mode is retained by default so that first-time users can try Envoy out quickly.
But by switching authentication modes (AUTH_STRATEGY=google), we can switch Envoy to use Google OAuth2.
This uses the PassportJS which has authentication modules for many third-party authentication brokers - Facebook, Twitter, LinkedIn, Twitter etc. I decided to modularise the authentication a bit - to create a new one just add a file to lib/auth and use AUTH_STRATEGY to select the one you want at run time.
The tests were all set up for basic auth so they remain largely unchanged.

Commits:
used the PassportJS adapter instead of our own middleware - also move some auth functions to utils
load auth module at runtime
added Google OAuth option
readme
all tests passing
no longer need basic-auth as a direct dependency
added facebook login, just to see how easy it would be to add - answer 15 minutes work
added facebook instructions to README
added github OAuth
updated docs for AUTH_STATEGY
twitter oauth
tidy up - return to /_auth at end of authentication process
tidy up
couple of fixes so that it works from a clean install in Bluemix

Was this functionality merged in some other PR?
This PR (issue5a) is probably the other one mentioned in #34 (issue5) / #5, but it seems neither were merged.


small fix and redundancies

Removed some redundant lines (this will prevent future compilation warnings with other architectures)
Previous line 160 was probably a bug (hi instead of low). Now fixed

I think the more proper fix would be to delete the ones at lines 133, 134.
The reason for having them inside the ifs was that not always both ups and downs are needed.
Giovanni

Agree (updated in the PR)


Ambiguity what should happen when no kid parameter is present in header when DID is used as iss value

The spec mentions that a kid is optional when the iss value is a DID. The same seems to be true when using JWT Issuer Metadata (although there it mentions it is RECOMMENDED at least). It doesn't describe however what should happen if no KID is present.
This has caused some interop issues with implementations not including a kid field when a DID is used when e.g. did:key or did:jwk is used as you can infer which key is meant. However this means you have to make exceptions for specific did methods instead of following a general pattern.
Is there a specific reason to make the kid optional if the iss value is a DID (and also similarly when JWT Issuer Metadata is used)? If a DID is used, it MUST be signed with a key in the did document right? And it also mentions that "If a recipient cannot validate that the public verification key corresponds to the iss value of the Issuer-signed JWT, the SD-JWT VC MUST be rejected."
So I don't see a very good reason here to make the usage of kid optional, as it results in ambiguity. If there is a good reason to make it optional, would it be possible to also add the RECOMMENDED to using a kid when iss is a DID, that way we can at least promote other implementation to always include a kid when using a DID.

I agree that there should not be any DID method specific exceptions. Even though we know that did:key and did:jwk basically encode a single public key, there should not be any logic in this specification which assumes this for all DIDs. DIDs resolve to DID documents that can potentially contain several public keys, therefore it is necessary to identify the specific public key using a DID URL, not just a DID.

The resolution described in #250 would address this.

In implementations of DIDs I have worked on, with JWTs, we have required that kid is always a fully-qualified key identifier such as did:example:abcd#key-1, where the iss would be did:example:abcd.
Handling all the variations of using fragments (#key-1, key-1, etc.) make things complex, so clearly requiring fully qualified key identifiers from DID Document Verification Methods is a simple choice.

I also believe that kid should be:

mandatory and perhaps most importantly
a fully-qualified DID URL (as per @decentralgabe  comment)

In addition, a check should be required to ensure that kid is a DID URL (sub-resource) of the DID in the iss claim.

The resolution described in #250 would address this.

This issue then has been overcome by events.
I encourage interested parties and proponents of DID usage in this context -- which, despite any potential misunderstanding, remains permissible -- to work on specifying it in a manner and forum befitting its importance.
In #250 @kimdhamilton pointed to a recent initiative that might be relevant for those interested in contributing to DID method standardization: https://identity.foundation/publications/LOI-DIDMethodStandardization.pdf


raise an exc when container creation fails

Fixes #34
Signed-off-by: Tomas Tomecek<EMAIL_ADDRESS>

jenkins/superCI comment

Status: failure, return code = 1
Result url: https://da.gd/uuGWm
Comment: See details for logs

jenkins/superCI comment

Status: failure, return code = 1
Result url: https://da.gd/Wi11p
Comment: See details for logs

jenkins/superCI comment

Status: success, return code = 0
Result url: https://da.gd/y4Ex
Comment: See details for logs


Releases count missing on some repos

Hello! One more rare bug (:
It looks like when opening a repository which has either only tags, or only pre-releases, release count stays 0.
I saw your comment about that, but actually original github UI did display that.
Sample repositories:
https://github.com/nuxt-community/eslint-module (tags only)
https://github.com/Cxbx-Reloaded/Cxbx-Reloaded from #10
Version 2.0
Here is web archive link from January 2020
Screenshots:

One more rare bug (:

Well, it's not so rare if there're 2 issues about it already)

...but actually original github UI did display that

Hm. It changes things completely. Ok, will show tags count if there're not any actual releases

Done:

Also I made this behavior optional, so you can disable it if you want:

Amazing! Thank you again! Now it is perfect (:


[Storage][Core] Wait for environment to become eventually consistent.

This is another attempt to solve https://github.com/Azure/azure-sdk-for-net/issues/17384 and https://github.com/Azure/azure-sdk-tools/issues/1567 .
Based on discussion in https://github.com/Azure/azure-sdk-for-net/pull/20559 .
The idea is that:

TestEnvironment exposes a overridable method that represents service provided sampling scenario
RecordedTestBase<TEnvironment> shall keep executing the sampling scenario until it signals it's ready (or fail fast if the error is severe)
Service will implement sampling scenario to call some API that fail if environment hasn't reach usable state

Out of scope:

Moving configuration from StorageTestBase to StorageTestEnvironment. However some work towards it is accomplished here.
The retry strategy and it's configuration in RecordedTestBase<TEnvironment> is simplified now. I think we should make it fancy when there's a need.
I'll cover other storage services in separate PR. Here just blobs as PoC.

/azp run net - storage - tests

/azp run net - storage - tests

This is really cool. Would you like to add a blurb to the Readme - https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/core/Azure.Core.TestFramework/README.md

/azp run net - storage - tests


Error : spawn gpg ENOENT

I try to run examples\encrypt.js and found the error results here
{ Error: spawn gpg ENOENT
at exports._errnoException (util.js:1018:11)
at Process.ChildProcess._handle.onexit (internal/child_process.js:193:32)
at onErrorNT (internal/child_process.js:367:16)
at _combinedTickCallback (internal/process/next_tick.js:80:11)
at process._tickCallback (internal/process/next_tick.js:104:9)
at Module.runMain (module.js:606:11)
at run (bootstrap_node.js:389:7)
at startup (bootstrap_node.js:149:9)
at bootstrap_node.js:504:3
code: 'ENOENT',
errno: 'ENOENT',
syscall: 'spawn gpg',
path: 'gpg',
spawnargs: [ '--batch', '--encrypt' ] }

any solution for the above issue?

Have you installed gpg cli tool? What is the output of $ gpg --version ?

I didn't installed gpg cli tool. Here is my code,var gpg = require('gpg');

I didn't installed gpg cli tool. Here is my code,
var gpg = require('gpg'),
...
gpg.encryptToFile({source:fileToWrite, dest: './export/test.dat' }, function(err, data){
                console.log(err);
 });

After running this file, I am getting the above error. But empty file is generated inside the export folder.
is there anything need to add.

I didn't installed gpg cli tool

You need to install this in order to use the npm package gpg. This package is a simple wrapper around the cli tool.

@freewil I am using windows 10 machine. Can you explain which one can i download and install. Also please explain after installation which steps i can follow for using gpg encrypt/decrypt?

@satheeshtcs, I am also facing the same error. Will you be able to find any solution for this?

I am facing this issue in the codecov-action v4, per https://github.com/codecov/codecov-action/issues/1279#issuecomment-2046096493. Same issue in GitHub hosted runners (macOS-12 and macOS-13), macOS-14 does not have the problem.
gpg --version
gpg (GnuPG) 2.4.5
libgcrypt 1.10.3
Copyright (C) 2024 g10 Code GmbH
License GNU GPL-3.0-or-later <https://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Home: /Users/runner/.gnupg
Supported algorithms:
Pubkey: RSA, ELG, DSA, ECDH, ECDSA, EDDSA
Cipher: IDEA, 3DES, CAST5, BLOWFISH, AES, AES192, AES256, TWOFISH,
        CAMELLIA128, CAMELLIA192, CAMELLIA256
Hash: SHA1, RIPEMD[16](https://github.com/LizardByte/Sunshine/actions/runs/8651443319/job/23722157891#step:5:17)0, SHA256, SHA384, SHA512, SHA224
Compression: Uncompressed, ZIP, ZLIB, BZIP2

https://github.com/codecov/codecov-action/pull/1371


Quicksettings for SD model and VAE stuck on "Loading..." even after completion, cannot change

This is a very annoying bug. I am using the web ui via colab.
When I frequently change SD models and VAEs, they will load but the quicksettings dropdown will be stuck on "Loading..." and I am unable to change them until I refresh the page.
I am running on a local URL on account of upscaling in extras breaking the output image when running on a gradio server.

Do you get the sane issue with gradio server or is it specific to local tunnel ?

So far, it's been with the local tunnel.

Use the gradio server to borrow down to the cause of the issue

Use the gradio server to narrow down to the cause of the issue

Upscaling in Extras is still broken when using a Gradio server.

I'll check it out, what type of upscaling ? or all of them ?


guard external testing against algorithm absence

This PR introduces a check for algorithm presence to avoid recurrence of CI failures due to bad liboqs configuration(s).

The Windows failures are a logical consequence of liboqs (prior to https://github.com/open-quantum-safe/liboqs/pull/1702 landing) not enabling any KEM algorithm considered STD by oqsprovider.


loader: Add support for WSI VK_KHR_display_swapchain extension

Since this is a device extension, normally loader support is not required. However, since the entry points for WSI extensions are documented as statically exported by the loader in the Vulkan spec. The loader needs to have trampoline code for this extension.

This change will require layer versioning.  That's being discussed separately for now.  I'll keep this open since once that issue is resolved, we'll add this set of entry-points first.

Moving this to issue #888 since there are a few more extensions now to add.


Use the access token to access /oauth2/authorize

Expected Behavior
OAuth2AuthorizationEndpointFilter should be after BearerTokenAuthenticationFilter
Current Behavior
Accessing /oauth2/authorize with access token failed
Context
I want to access /oauth2/authorize using access token

@knightdreams6

I want to access /oauth2/authorize using access token

The Authorization Endpoint does not accept access tokens. Please review the spec for details.


How does this library internally work?

Will the else conditions still be evaluated internally when the if condition was already true? This is based on my stackoverflow question: http://stackoverflow.com/questions/33540326/how-do-handle-if-else-cases-in-angular-ng-show-without-ng-switch?noredirect=1#comment54866854_33540326
if(x) {
}
else if(y){
}
Will this internally be converted to this?:
if(x) {
}
if(!x && y){
}
.. so will the x condition be evaluated twice?

Hey @akcasoy, what happens is that the expressions x and y are stored, and then each digest they are evaluated. So, x and y are each evaluated once. Note that this is much different than a "real" if-else in, say, Javascript, wherein when x is true y is not evaluated at all.

Then what is the advantage of this library when we have to evaluate the expressions of unneeded cases? So you basically convert the if else statement to this:
if(x) {
}
if(!x && y){
}
where you evaluate both x and y.  But the second 'if' never be true if the first one was true. So it will act like an if/else. But technically it is not sth special. Do i understand right?

No, we only evaluate x once and y once. You could think of it like this:
if(x){ ... }
else if(y){ ... }
else { ... }

Becomes:
var evaluatedX = x;
var evaluatedY = y;

if(evaluatedX){ ... }
else if(evaluatedY){ ... }
else { ... }

See https://github.com/zachsnow/ng-elif/blob/master/src/elif.js#L52-L61 for a clearer understanding.  It doesn't seem like it would be too hard to change it to be more like an if/else in a sequential programming language, I just haven't needed to yet.

@akcasoy see dev/short-circuit for a short-circuiting version of ng-elif; I will likely publish this today but in case I don't get a chance.

"I avoid writing conditions that are expensive to compute"
i would like to know how i can decide not to write complex conditions :) That is for me not a decision but a "need". I need complex conditions, i have complex conditions, i dont have any other option. So i would like to skip evaluating else conditions. Thank you for your time and answers though. I will have a close look at your project when i have time..


Move fd_advise on Linux to nix

Bases on #74.
We could actually enable posix_fadvise for all platforms supported by nix, but it's going to be messy. https://github.com/nix-rust/nix/blob/master/src/fcntl.rs#L15

Bases on #74.
We could actually enable posix_fadvise for all platforms supported by nix, but it's going to be messy. https://github.com/nix-rust/nix/blob/master/src/fcntl.rs#L15

Nice, looks good, thanks! As far as enabling for more platforms, sure, but I'd leave that for the (near) future. I'd rather we finished basic (experimental) support for Windows first, before diverging our focus on other things :-)


Use the actual UUIDv4 format

Hi, according to Wikipedia, a UUID is not mere series of random numbers ; a valid UUIDv4 is as follows :

xxxxxxxx-xxxx-Mxxx-Nxxx-xxxxxxxxxxxx
The four-bit M and the 1 to 3 bit N fields code the format of the UUID itself.

M represents the UUID version, so is supposed to be 4 here, and N the UUID variant (I don't know much more about that).

It seems that N is supposed to be one of 8, 9, a, b, according to Wikipedia again (these values are for a variant 1 UUID, as variant 0 is legacy and variant 2 is reserved by Microsoft).

This is intentional, but I must admit, this is not clear from the README.
The idea is that the library generates (cryptographically secure) random ids, similar to type 4-uuids, but generates even more randomness with the same byte size (at least in hex-string notation. one could of cause optimize away the static bits of the id, this is where yyids and uuids are not compatible).

Leaving this open as a reminder to improve the README


Showing search country instead of search state in state searchbar

the searchbar should say search state
And i am glad that i am the first person to open issues :)

Updated please use v0.0.8

@altafc22 Ok Thanks


ADD nacos-auth-multiple-database

在 nacos-plugin-default-impl 的基础上 增加 PostgreSql  、Oralce、DM的  PageHandlerAdapter

 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.