VPC update is always enqueued leading to huge number of updates when creating subnet

Expected Behavior
When creating a subnet the VPC shouldn't be updated a large amount of times.
Actual Behavior
Creating a subnet after a VPC already has two subnets leads to a huge amount of updates.
I assume that this has to do with the following line:
https://github.com/kubeovn/kube-ovn/blob/60ef1ad1f31d2974cb9462670b0ad26b1015be6f/pkg/controller/vpc.go#L47
Before #3107 this was (note the missing ! in the line above):
https://github.com/kubeovn/kube-ovn/blob/68dc1c38023e4967e9644c89e72e63c605df8591/pkg/controller/vpc.go#L56
I'm unsure why the ! should be removed here, as that leads to the c.addOrUpdateVpcQueue.Add(key) basically being called every time, which leads to a huge amount of (unnecessary) updates.
Steps to Reproduce the Problem

Create a VPC

kind: Vpc
apiVersion: kubeovn.io/v1
metadata:
  name: test-vpc

Create a subnet and bind it to the VPC. Check the kube-ovn-controller logs. Shortly after you created the subnet the logs calm down again.

apiVersion: kubeovn.io/v1
kind: Subnet
metadata:
  name: subnet-138
spec:
  cidrBlock: <IP_ADDRESS>/24,fd50:52:129:138::0/64
  default: false
  disableGatewayCheck: true
  vpc: test-vpc

Create a second subnet and bind it to the VPC. Check the kube-ovn-controller logs. Shortly after you created the subnet the logs calm down again.

apiVersion: kubeovn.io/v1
kind: Subnet
metadata:
  name: subnet-139
spec:
  cidrBlock: <IP_ADDRESS>/24,fd50:52:129:139::0/64
  default: false
  disableGatewayCheck: true
  vpc: test-vpc

Create a third subnet and bind it to the VPC.

apiVersion: kubeovn.io/v1
kind: Subnet
metadata:
  name: subnet-140
spec:
  cidrBlock: <IP_ADDRESS>/24,fd50:52:129:140::0/64
  default: false
  disableGatewayCheck: true
  vpc: test-vpc

Now watch the log output of the kube-ovn-controller. Notice that it takes ~3 minutes (on my machines) until the logs stop.

Check the kubeovn-bug-report.log I attached.
I annotated where I started to create the subnets.
You should note that adding subnet-140 leads to an absurd amount of log lines.
Additional Info

Kubernetes version:
Output of kubectl version:
Client Version: v1.28.2
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.28.2

kube-ovn version:
v1.12.1

operation-system/kernel version:
Output of awk -F '=' '/PRETTY_NAME/ { print $2 }' /etc/os-release:
Output of uname -r:
Debian GNU/Linux 12 (bookworm)
6.1.0-13-amd64

@bobz965 can you explain this?

@bobz965 can you explain this?

yeah, this should be a bug, I think using the logic should be better.

@bobz965 can you explain this?

yeah, it is a bug which looks like introduced before. The vpc which is in deleting should not be updated again previously.
In the vpc update logic, if  a vpc is about to delete, the policy and static route, vpc peer routes should all be deleted in the deletion logic.


Better logging for named pipe

Add logging on the server side for why a project isn't being serialized.
Reduce some of the logging on the client side if it gets into a bad state. Log once for first bad read and once when it gets to a point where we think we returned to a good state.

I trust you that retrying makes sense, but it's not really clear to me why it wouldn't just immediately fail again :)

Because by reading data it moves the stream forward. That said I have a suspicion that it won't really recover but did some defensive programming around it. The updated server side is more important, but with VS Code this will flood logs pretty quickly if we don't stop. Maybe I should have just exited on fail... Or have some better retry logic to reconnect or something... but for now this is mostly to help figure out why Mac integration tests are failing in VS Code


Build fails on Linux

Arch Linux 4.3.3, GCC 5.3.0
Log:
[rob@Archie][~/dev/crown]% GENIE=`which genie` make linux-release64 -j128
/usr/bin/genie --file=genie/genie.lua --with-luajit --with-openal --with-bullet --with-tools --compiler=linux-gcc gmake
Building configurations...
Running action 'gmake'...
Generating build/projects/linux/Makefile...
Generating build/projects/linux/bgfx.make...
Generating build/projects/linux/shaderc.make...
Generating build/projects/linux/openal.make...
Generating build/projects/linux/bullet.make...
Generating build/projects/linux/crown.make...
Done.
make -R -C build/projects/linux config=release64
make[1]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.
make[1]: Entering directory '/home/rob/dev/crown/build/projects/linux'
==== Building bgfx (release64) ====
==== Building shaderc (release64) ====
==== Building openal (release64) ====
==== Building bullet (release64) ====
==== Building crown (release64) ====
sound_world_al.cpp
../../../src/audio/sound_world_al.cpp:11:22: fatal error: id_array.h: No such file or directory
compilation terminated.
crown.make:1039: recipe for target '../../linux64/obj/x64/release/crown/src/audio/sound_world_al.o' failed
make[2]: *** [../../linux64/obj/x64/release/crown/src/audio/sound_world_al.o] Error 1
Makefile:32: recipe for target 'crown' failed
make[1]: *** [crown] Error 2
make[1]: Leaving directory '/home/rob/dev/crown/build/projects/linux'
makefile:38: recipe for target 'linux-release64' failed
make: *** [linux-release64] Error 2

It looks like a file is entirely missing from the repo.

Yup, I'm rewriting that stuff. This should fix the build: a28185d1ee7d5884a20d382ef2e9572d53052323

It works! Although make[1]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule. is annoying, because I can't compile in parallel without trickery.


fix: fix build.sh warning by shellcheck

What's the purpose of this PR
fix build.sh warning by shellcheck
Which issue(s) this PR fixes:
Fixes #
Brief changelog
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Read the Contributing Guide before making this pull request.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit tests to verify the code.
[ ] Run mvn clean test to make sure this pull request doesn't break anything.

Codecov Report

Merging #3294 into master will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #3294   +/-   ##
=========================================
  Coverage     51.49%   51.50%
- Complexity     2306     2307    +1
=========================================
  Files           442      442
  Lines         13738    13738
  Branches       1398     1398
=========================================
+ Hits           7075     7076    +1
+ Misses         6175     6170    -5
- Partials        488      492    +4

Impacted Files
Coverage Δ
Complexity Δ

...rk/apollo/spring/property/SpringValueRegistry.java
83.33% <0.00%> (-5.56%)
10.00% <0.00%> (-1.00%)

.../framework/apollo/spring/property/SpringValue.java
87.71% <0.00%> (-1.76%)
20.00% <0.00%> (-1.00%)

...ework/apollo/internals/RemoteConfigRepository.java
88.75% <0.00%> (-0.63%)
24.00% <0.00%> (-1.00%)

.../apollo/internals/RemoteConfigLongPollService.java
78.31% <0.00%> (+0.60%)
29.00% <0.00%> (+1.00%)

...mework/apollo/openapi/service/ConsumerService.java
49.51% <0.00%> (+1.94%)
13.00% <0.00%> (+1.00%)

...framework/apollo/openapi/entity/ConsumerAudit.java
48.48% <0.00%> (+6.06%)
11.00% <0.00%> (+2.00%)

Continue to review full report at Codecov.

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


Add unit tests for BlockMasterClient

What changes are proposed in this pull request?
add more unit tests in BlockMasterClientTest that tests for expected grpc behaviors.
Why are the changes needed?
improve unit-test coverage of BlockMasterClient (20% -> 90%)
Does this PR introduce any user facing changes?
No

@dbw9580 May you take a look at my PR? Thanks!

@jiacheliu3 do you mind taking a look at this unit test?

The current unit tests are more about positive test cases, which are the fundamental guarantee like "Here are my APIs and this is how you are expected to use them". If you are to add more tests on them, you can add negative test cases where you verify what happens on exceptions and write down the assertions(expected behaviors) in the form of unit tests. That is beyond the scope of one PR, feel free to do that in a separate PR, now that we have the basics covered in this PR and feel more secure about the existing API.

@jiacheliu3 can we merge this?

alluxio-bot, merge this please


Error with running prepare.py

Running prepare.py e.g. with the command python prepare_data.py --target-dir "./train_data" --split-file "./omg_TrainVideos.csv" return the error The system cannot find the file specified as files such as https://www.youtube.com/watch?v=CROcsl0lIec are not available

Unfortunately it seems like some videos have been made private so the full dataset can't be accessed


more debugging info please

when I get this error: https://github.com/openseadragon/openseadragon/blob/cfc339edea0a65575c7904ec82e72e911c584825/src/iiiftilesource.js#L53
It would be great to have more context
I think its hitting this url and getting this response:
curl http://snapple.localhost:8080/images/4c8af5d5-9aa5-4385-841e-64315aa490e1%2Ffiles%2Fbb38721b-0beb-4af1-b93a-e3e90247910a/info.json
{"width":3264,"height":2448,"@context":"http://iiif.io/api/image/2/context.json","@id":"http://snapple.localhost:8080/images/4c8af5d5-9aa5-4385-841e-64315aa490e1%2Ffiles%2Fbb38721b-0beb-4af1-b93a-e3e90247910a","protocol":"http://iiif.io/api/image","profile":["http://iiif.io/api/image/2/level1.json",{"formats":["jpg","png"]}]}

Which looks valid to me, so if the error was:
IIIF required parameters not provided for  http://snapple.localhost:8080/images/4c8af5d5-9aa5-4385-841e-64315aa490e1%2Ffiles%2Fbb38721b-0beb-4af1-b93a-e3e90247910a/info.json

Got:
{"width":3264,"height":2448,"@context":"http://iiif.io/api/image/2/context.json","@id":"http://snapple.localhost:8080/images/4c8af5d5-9aa5-4385-841e-64315aa490e1%2Ffiles%2Fbb38721b-0beb-4af1-b93a-e3e90247910a","protocol":"http://iiif.io/api/image","profile":["http://iiif.io/api/image/2/level1.json",{"formats":["jpg","png"]}]}

It would be easier to understand what's wrong

Indeed, could you make a PR with this change?


Postscript for url in howpublished

Hi,
I try to use the postscript:
if (Translator.BetterBibTeX && item.itemType === 'webpage') {
    if (item.accessDate) {
      this.add({ name: 'note', value: "(accessed " + item.accessDate + ")" });
    }
    if (item.url) {
      this.add({ name: 'howpublished', bibtex: "{\\url{" + this.enc_verbatim({value: item.url}) + "}}" });
    }
  }

I found this here https://retorque.re/zotero-better-bibtex/scripting/ and added it in the Advanced tab of Better BibTex in the Zotero Preferences. However, I do not seem to get it to work as my Better Bibtex exports are still showing the howpublished field without embedded \url{}.
Any suggestions? Your help is much appreciated, as this is the first time I try to use a postscript.
Cheers,
T

[x] I have verified in the Zotero addons screen that I am on the latest release of BBT (https://github.com/retorquere/zotero-better-bibtex/releases/latest) and Zotero
[x] I am posting a single question, bug or feature request
[x] I have included a descriptive subject of the problem
[x] I am available for follow-up questions and testing

Bug classification
Exporting references
If your bug report/feature request/question concerns problems with exporting references, please specify whether you're exporting BibTeX or BibLaTeX. Please right-click the references where you experience the export problem, select "Send Better BibTeX Error", and post the resulting report ID (shown in red after you submit) here.
Non-export problems with BBT
If your issue is a bug report, but not for exports, restart Zotero with debugging enabled (Help -> Debug Output Logging -> Restart with logging enabled), reproduce your problem, and select "Report Better BibTeX error" from the help menu, and post the resulting report ID (shown in red after you submit) here.

Can you

From the help menu, restart with debugging enabled
Export one reference where you wanted the postscript applied
Right-click that reference, select "Report BBT error", and post the resulting debug ID here.

Hi retorquere,
Thanks for your reply. Here is the ID R7BVWQ5F.
T

:robot: this is your friendly neighborhood build bot announcing test build <IP_ADDRESS>43.issue-951 ("jurism does not wish to be downloadd...").

Please try again with 6343

Here is the new ID: 6MERCJ55. Not sure what you mean by 6343?
Thank you.
T

I meant the test build ("test build <IP_ADDRESS>43.issue-951") linked to above. I think that will solve the issue, if we have confirmation on that, I can cut a new release.

That worked. Thank you very much for your help!
T

My pleasure. This will be released in 5.0.128 in a few minutes


Site should pull data from Meetup.com

Requirements

[x] Should use Meetup.com API
[x] Should display latest meetup information on the homepage
[x] Should be performant enough to allow for quick loading
[x] Should not require a re-build of the site to update rendering

Implementation status
This was in place as part of an early dev build of the site, but I never completed the modifications necessary to make this work with the new UI.

Closed in b254e28


Issue with index.add_items() when building large indexes?

When building indexes of varying sizes I ran into some issues with some of the larger sizes..
Here's what my index creation code looks like:
# imagine `vectors` is an ndarray with multiple vectors of dimension 1728 ...
num_dimensions = vectors[0].shape[0]
index = Index(Space.Cosine, num_dimensions=num_dimensions)
index.add_items(vectors)
index.save(filename)

And my test code looks like this:
# imagine `vector` is a sample query vector of (matching) dimension 1728
index = Index.load(filename)
index.query(vector, k=200)

This works fine when vectors is of cardinality 10k, 50k, 100k, 500k, and 1M ...
but when vectors has 5M or 10M vectors in it, index creation runs fine, but upon querying ...
     index.query(vector, k=200)
RuntimeError: Potential candidate (with label '4963853') had negative distance -17059.800781. This may indicate a corrupted index file.

I tried creating the index with slices of the same vectors array of size 1M:
start = 0
while start < vectors.shape[0]:
    end = start + batch_size
    index.add_items(vectors[start:end])
    start = end

and it seems I can query this index just fine. Maybe some sort of limitation with the add_items() function?

hey @loisaidasam thank you for reporting, that result definitely shouldn't be happening! We've seen similar behavior when accidentally adding vectors with NaN values into the index, but I don't think the issue is related.  It's more likely due to a race condition somewhere so we'll investigate this and get back to you!

Any update on this tipoc?

Hi all. This error was removed in https://github.com/spotify/voyager/pull/80 and included in the 2.0.9 release.

Closing as this is stale due to this error not existing anymore. Please reopen if this is manifesting as another problem.


docs: updated with example on how to setup lazymc with minecraft container

I have created a docker container which wraps LazyMc add adds the functionality to control a minecraft container
I have updated the docs to add this as an example. Feel free to check out the project here: https://github.com/joesturge/lazymc-docker-proxy

Cheers! Happy to help

Hello itzg directed me to come here for support.
I am using a ubuntu server and trying to use lazymc proxy. I coped the example configuration from the readme and added my mc environment variables on top of it. Only the lazymc-proxy container is running but still can't connect to the server.
When I log the lazymc-proxy container it keeps saying this:

ERROR mc::lazymc::monitor             > Force killing server, took too long to start or stop
ERROR lazymc-docker-proxy::health     > Application is unhealthy.
WARN  mc::lazymc                      > Failed to force kill server

Hello itzg directed me to come here for support. I am using a ubuntu server and trying to use lazymc proxy. I coped the example configuration from the readme and added my mc environment variables on top of it. Only the lazymc-proxy container is running but still can't connect to the server. When I log the lazymc-proxy container it keeps saying this:

ERROR mc::lazymc::monitor             > Force killing server, took too long to start or stop
ERROR lazymc-docker-proxy::health     > Application is unhealthy.
WARN  mc::lazymc                      > Failed to force kill server

When I try to join from minecraft just keeps saying "Server is starting ... try to reconnect in a minute"

Hi @eyes-eyes I think someone else has expeienced this could you try my suggestion here: https://github.com/joesturge/lazymc-docker-proxy/issues/115#issuecomment-2448597328

Hello itzg directed me to come here for support. I am using a ubuntu server and trying to use lazymc proxy. I coped the example configuration from the readme and added my mc environment variables on top of it. Only the lazymc-proxy container is running but still can't connect to the server. When I log the lazymc-proxy container it keeps saying this:

ERROR mc::lazymc::monitor             > Force killing server, took too long to start or stop
ERROR lazymc-docker-proxy::health     > Application is unhealthy.
WARN  mc::lazymc                      > Failed to force kill server

When I try to join from minecraft just keeps saying "Server is starting ... try to reconnect in a minute"

Hi @eyes-eyes I think someone else has expeienced this could you try my suggestion here: joesturge/lazymc-docker-proxy#115 (comment)

Oh just realised that is infact also you... lol


Add AnnaCurly's file

created a branch, created a file and made a commit, and opened a pull request

this is reviewed

I coudn't do that


Enforce chunking in the Patching service

The current implementation might not be able to handle simultaneous clients well. Large files might block the event loop for too long.

investigate the performance of the current implementation
check wheter zero copy is available in this context
create a test to ensure that the implementation handles concurrency
implement and enforce chunked transfers if adequate

ensure the current method of storing binary files does not introduce unnecessary copying. (PatchKeeper)

actions taken in this issue must be applied to the website service.


For http client side, how to use tls.Config GetClientCertificate()

I'm using the http client to do mutual tls verify for different endpoints(or different client cert for different servers).
Not sure if GetClientCertificate() could be used for it. I'm a little confused about it. Because CertificateRequestInfo doesn't have server endpoint or host info.

crypto/tls

    GetClientCertificate func(*CertificateRequestInfo) (*Certificate, error) // Go 1.8

It would be great if someone could give such an example? or maybe any other method to meet this requirement?
What version of Go are you using (go version)?
go version go1.13.3 linux/amd64

Hi there,
Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For asking questions, see:

Stack Overflow with questions tagged "go"

The Go Forum, a web-based forum

Gophers Slack, use the invite app for access. The #general channel is a good starting point.

Go Community on Hashnode with questions and posts tagged with "go"

The golang-nuts mailing list

IRC channel #go-nuts on Freenode

Please ask the question on one of the above forums.
(Quoted from https://golang.org/wiki/Questions)


Auto promote concatenate.

This PR allows cube concatenation to promote candidate cube dimensionality in an attempt to match that of the proto-cube.
For example, given a 3D time-series, this PR will successfully concatenate the following:
iris.cube.CubeList([cube[-1], cube[1:-1], cube[0]]).concatenate_cube()

This PR replaces #1434, as it targets upstream/master rather than v1.7.x.

@rhattersley and @pelson I've been flip-flopping as to whether to include this PR for the v1.8.0 release ...
As much as I want to include it, the timing isn't quite right, for several reasons ...
Now that #1386 has been merged concatenate can maintain the deferred data payload of candidate cubes. Hoo-rah!
However, this PR makes use of iris.util.as_compatible_shape which forces a load of the data payload of candidate cubes, thus undoing the benefit of #1386. If only we had Biggus support for np.newaxis (https://github.com/SciTools/biggus/pull/116) and biggus.transpose ... @pelson :wink:
For that reason, I'm opting to bump this PR to a later date. When the appropriate Biggus infrastructure is in place, this PR will be ripe for a re-factor and inclusion into upstream/master.

When the appropriate Biggus infrastructure is in place, this PR will be ripe for a re-factor and inclusion into upstream/master.

Nudge: newaxis & transpose are available now. Is it time to update this PR, or just close it and let it come up again naturally?

It will come up again if it is needed.


Allow nullptr as data argument to constructor

This was requested in #103.
Currently, you can specify a model has no data by passing "". This also allows you to pass nullptr for the same thing.

@aseyboldt:  You picked the right team for pedantry.  We really do appreciate all the detailed comments.


Huge log messages

I started testing redisson with my app (Redisson 3.5.1), first issue i discovered:
Writing about ~23mb data to redis map.
Error message generated with size ~180mb when RedisTimeoutException accured.
I guess issue with line that blindly call toString:https://github.com/redisson/redisson/blob/8cad23cdcec19aff04725659cc0af9239053af6b/redisson/src/main/java/org/redisson/client/handler/CommandDecoder.java#L323

Message look like (with long long result field):
09-10 08:45:55     [rn3-11]  WARN o.r.c.handler.CommandDecoder    | response has been skipped due to timeout! channel: [id: 0x51f23a66, ...], command: CommandData [promise=org.redisson.misc.RedissonPromise@cc9939[Completed exceptionally], command=(HGET), params=[...], codec=org.redisson.client.codec.ByteArrayCodec@3e0ea2ee], result: [76, 90, 52, 66, 108, 111, 99, 107, 38, 8, 90, 0, 0, 0, 0, 1, 0, 104, -77, -17, 6, -16, 22, -84, -19, 0, 5, 115, 114, 0, 38, 99, 111, 109, 46, 102, 111, 114, 107, 115, 104, 117, 110, 116, 101, 114, 46, 115, 99, 97, 110, 101, 114, 115, 46, 99, 114, 97, 119, 108, 9, 0, -13, 4, 83, 116, 97, 116, 101, 47, 56, 36, 87, -39, -17, -50, -50, 2, 0, 9, 74, 0, 14, 28, 0, -9, 12, 86, 101, 114, 115, 105, 111, 110, 76, 0, 9, 98, 101, 116, 115, 67, 97, 99, 104, 101, 116, 0, 45, 76, 99, 111, 109, 47, 82, 0, 19, 47, 82, 0, 16, 47, 37, 0, 16, 99, 37, 0, 36, 47, 66, 47, 0, -1, 6, 59, 76, 0"

For big requests I would recommend to increase Config.timeout setting.

I already did it, but timeouts still happen sometimes. I think such huge log messages must be never generated anyway.

Fixed


Distribute Dispatch CLI as dotnet tool instead of npm package

Tool
Name: [Dispatch]
Describe the problem
I see Dispatch CLI is a .net core application. Could you please distribute it as a dotnet tool just like LUISGen?
Describe the solution you'd like
Let us download Dispatch CLI as dotnet tool
Describe alternatives you've considered
Let us build something ourselves using a Nuget package (see #345) or https://github.com/Azure/azure-sdk-for-net/issues/5278
[enhancement]

@hansmbakker , there's work going on to consolidate all the tools one or two tools so stay tuned.

@hansmbakker, checking on this at the moment.  Will update as soon as we decide on the solution. Thanks!

@tsuwandy thank you for checking!
If we could get a nuget package (with a .net library) then we could use this in azure functions or build our own tools - this is also requested in #345. This is a solution that would be flexible to all of us.

@hansmbakker, we are working on this.  Will update this issue when package is ready, should be in a few days. Thanks!

Thank you!

Hello, private package is here:
https://botbuilder.myget.org/feed/botbuilder-tools-daily/package/nuget/Dispatch
We are working to publish it to nuget.org.  Thanks for your patience!

@hansmbakker, thanks so much for your patience!  Unfortunately, after further discussion with the rest of the team, we have decided that we won't be releasing dispatch as a nuget package.   The rest of our tooling is NodeJS based and the next version of Dispatch (targeted for BUILD 2020) will also be in NodeJS.  The nuget package won't be maintained/supported post BUILD 2020 so it doesn't seem worthwhile to use it for a short time only.

Ok, that's clear. Thank you for your efforts, though!


Update docs to describe the conversion of an existing tqdm bar

This PR adds new documentation that describes how to convert a single existing tqdm bar running on the main process.

Replaced by #39


Using winnowmap

Attempted to run dipcall using winnowmap (by passing -W repetitive_k15.txt) and saw the following error in dipcall.hap1.paf.gz.log:
[M::mm_idx_gen::0.003*1.15] reading downweighted kmers
ERROR: input list of k-mers and winnowmap parameter k are inconsistent

My repetitive_kmer file is of the form:
CGGAATTGAATGGAA	18274
CTCAAAAACAAAAAA	4552
CGGAATTGAATGGAA	12469
...

Unless there is a way to pass a value for k to dipcall, what value of k does dipcall anticipate? I did not see it listed in the documentation, or in the error message. I know k=15 is the default for winnowmap, which is why I tried that first.

 Winnowmap2 uses 19-mer by default.


[Feature Request] testnet nodes list

🚀 Feature Request
testnet nodes list for CLI testing
Motivation
Is your feature request related to a problem? Please describe.
Due to the crazy amount of traffic / throttling
Pitch
Describe the solution you'd like
alternative instances to test
Describe alternatives you've considered
Run separate nodes
Are you willing to open a pull request? (See CONTRIBUTING)
Additional context
handicap the ability of some CLI to not connect due to :
Not able to connect to validator at ac.testnet.libra.org:80, error RpcFailure(RpcStatus { status: Unavailable, details: Some("Socket closed") })

One more complex workaround is to run your own node and test on your local network. If we add additional networks, we'll make an announcement. Keep an eye on https://developers.libra.org/blog/.


Fix goto Vue file from import source

Codecov Report

Merging #308 (2e15f85) into main (060b0c2) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #308   +/-   ##
=======================================
  Coverage   36.56%   36.56%
=======================================
  Files         165      165
  Lines       21050    21040   -10
  Branches      887      885    -2
=======================================
- Hits         7696     7694    -2
+ Misses      13354    13346    -8

Impacted Files
Coverage Δ

...cript-plugin-vue/src/features/DefinitionService.ts
0.00% <0.00%> (ø)

...ypescript-plugin-vue/src/managers/PluginManager.ts
0.00% <0.00%> (ø)

packages/shared/src/rpc/handlers.ts
97.42% <0.00%> (-0.52%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Error with graphql-upload multipart request

Argument "file" of required type "Upload!" was provided the variable "$file" which was not provided a runtime value

Looks like the file handle is not passed as part of the variables option when the queryComplexity validator is created. If you pass that in the variables, this should probably work...


Bit masks for different cache sizes

Can you please explain how are the bitmasks for the different cache sizes determined?
I want to vary the size of a partition from 2MB all the way to 15MB in increments of 1MB but can't seem to figure out what the bitmasks would be for each cache size. Thanks!

Hey. I realized this is straightforward. Thanks.

One bit in class of service corresponds to one cache way. Cache way size can be calculated manually (total LLC size divided by number of ways).
This information can also be retrieved from the utility. Run "sudo ./pqos -s -v" and look for the following lines:
...
INFO: LLC cache size 36700160 bytes, 20 ways
INFO: LLC cache way size 1835008 bytes
...
Note that these lines will only be logged if CAT support is detected.


Allow to edit oligos

I find it handy to be able to edit the sequence and name of the oligos in the oligo page. Do you think this could be implemented?
This would entail rechecking assemblies after editing or establishing some constraints i.e. allow only modifications on unused oligos or, if the oligo is used, only allow to modify the name.
I'm sure you've put some thoughts into this so, what do you think?

I decided not to allow it for the reasons you point out (need to check, etc) since it has only two fields. However, I agree that users probably expect to have it. I will add it next! Thanks for trying it!


Run VeriFast proofs in CI

This PR makes the VeriFast proofs run in CI along with the CBMC proofs. The results for both CBMC and VeriFast proofs are displayed on a single Litani dashboard.
This PR incorporates #836, which fixes whitespace changes in the VeriFast proofs.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

CI will pass once we actually install VeriFast in CI (soon). I also need to add a README explaining the proof-configuration file

Thanks Kareem, this looks good to me (from the point-of-view of getting the VeriFast proofs happy again)


IAC

file.zip

@MuhammadKhubaib9 BCS I am sorry, I have to close the request without merging, because you made a mistake in a request and did not specify any filepath. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for your understanding!


Collector's Bee Ring feature

Adding a bauble that simulates a Bee Collector's Jar using NBT stored on the item.
The main logic is located in ItemBeeRing.java and InventoryBeeRing.java. GUI and container are basically copy-pasted from the Collector's Bee Jar. RingHousing.java simulates an apiary just enough to get the effect to trigger.
I'm sorry if it's coded horribly; this is my first major Minecraft feature. :)

@OneEyeMaker please can you re check pr.
@Alastors oneEyeMake review it so not approve it until the review is done

@OneEyeMaker please can you re check pr. @Alastors oneEyeMake review it so not approve it until the review is done

Aight, the code itself looked fine and most of his requested changes were done besides a subjective "don't copy and paste code" request, so I thought it'd be fine, the code itself works so.

@Cardinalstars please run spotlessaply
@Alastors working code is ok but code quality need to be checked as well.
@OneEyeMaker maybe give tips and tricks how Cardinalstars improve the code if necessary

@Alastors I'm sorry, but your changes made code even more messy.
@Cardinalstars After more deep research of this implementation I found performance issue which I don't know how to fix right now. I'll reimplement this idea myself when I find solution. So I close this PR.

I think we should just leave this in limbo until someone fixes it instead of closing this out entirely, I don't think just saying "I don't know how to fix this, I'll reimplement this myself eventually, no one else can fix this" is a good enough reason to just nuke a feature pr

Either offer a new implementation as a justification to yeet this, or offer solutions to it's lone issue, of which multiple have been offered in the discord, such as it only checking like once every minute or such, or as I proposed, only one bee being allowed in it while the effect was happening, and not allowing any more until the bee was destroyed.

set to draft by wishes of cardinalstar and alastor

I'll fix this crapshow in the morning, I'm gonna have to examine the 9 changes and figure out which broke the camel's back

posted that on the wrong pr

I apologize for my previous statement. My mind was overclouded yesterday.
Summary of changes:

Simplified Container/UI/inventory code by using existing (Forestry) implementation.
Fixed crash with particle render on incorrect side.
Removed unnecessary setting of inventory slot.
Allowed wearing two rings at once with different bees providing different effects.

Probably needs to fix:

Ring reads inventory from NBT on every tick. Reducing tick rate N times is not a proper solution - because bee inside ring will work N times slower and live N times longer. And potion effects will update N time more rarely.
(I say "probably", because I haven't large and populate server to test.)

@Cardinalstars is this ready for review ?

@Dream-Master Yes, it should now be ready. I just needed to test on dedicated server.

@OneEyeMaker


fix: image property not propagated

:tada: This PR is included in version 1.2.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add sleep and retry when secondary rate limit impacts on PR search

We hit this error semi-regularly. GitHub's secondary rate limiting is easy to trigger. It often will happen if multiple requests to the API are made at the same time from different jobs resulting in a failure. The failure is easy to fix by rerunning the job, but a user has to notice.
There might be other places that we can hit a similar issue, but this commit addresses the known location.
Fixes #37

Hi @SeanTAllen,
The changelog - added label was added to this pull request; all PRs with a changelog label need to have release notes included as part of the PR. If you haven't added release notes already, please do.
Release notes are added by creating a uniquely named file in the .release-notes directory. We suggest you call the file 38.md to match the number of this pull request.
The basic format of the release notes (using markdown) should be:
## Title

End user description of changes, why it's important,
problems it solves etc.

If a breaking change, make sure to include 1 or more
examples what code would look like prior to this change
and how to update it to work after this change.

Thanks.


Hydrating a model with a casted attribute, re-casts that attribute upon reference

Laravel Version: 5.8.29
PHP Version: 7.3.9
Database Driver & Version: MySQL Community Server - GPL 8.0.17

Description:
When a model with a casted attribute is hydrated with already casted attributes, upon accessing the hydrated model, eloquent tries to re-cast those attributes.
Steps To Reproduce:

Make a model (SomeModel) with an attribute (some_column) casted to array or json.

/**
 * The attributes that should be cast to native types.
 *
 * @var array
 */
protected $casts = [
    'some_column' => 'array',
];

Make an array of SomeModel

$some_model_data = [
    [
        'some_column' => [
            'some_key' => 'some_data',
        ],
    ],
];

Use hydrate to assign current data to another variable.

$some_models = SomeModel::hydrate($some_model_data);

Access the said attribute (some_column) from the hydrated variable.

foreach ($some_models as $some_model)
{
    if (isset($some_model->some_column['some_key'])) // Throws ErrorException (E_WARNING)
    // json_decode() expects parameter 1 to be string, array given
    {
        echo $some_model->some_column['some_key'] . "\n";
    }
}

As a hotfix, I just commented the casting of the attribute that bugs, and use pure Mutator/Accessor on the model.
protected $casts = [
   //  'some_column' => 'array',
];

public function getSomeColumnAttribute()
{
    if (!isset($this->attributes['some_column']) || $this->attributes['some_column'] === null)
    {
        return null;
    }
    if (is_array($this->attributes['some_column']))
    {
        return $this->attributes['some_column'];
    }
    return json_decode($this->attributes['some_column'], true);
}

public function setSomeColumnAttribute(array $value)
{
    if ($value === null)
    {
        return $this->attributes['some_column'] = $value;
    }
    return $this->attributes['some_column'] = json_encode($this->attributes['some_column']);
}

There is no bug here, this is the expected behaviour.
Hydration happens with the "raw data from the database", that is the use case.
That's why the code (called through ->hydrate()) looks like this, emphasis on setRawAttributes:
    public function newFromBuilder($attributes = [], $connection = null)
    {
        $model = $this->newInstance([], true);

        $model->setRawAttributes((array) $attributes, true);

So if you want to make it work manually, you've to make sure your data is json encoded in this case already.

Thanks @mfn! I thought hydrate is tolerant. I think I should close this issue then and move it to laravel/ideas as having a more tolerant way of converting things (arrays/collections) to models.


[Feature request] Class name aliasing

Hello,
Just as Mongo field names can be aliased to reduce storage space, would it make sense to have a similar way to alias Class names?
I'm asking this because or application growing in size, we've namespaced parts of it. This in turn leads to long class names such as White::Monkeylearn::SampleTag that seem to take uneccesary space in db.

You mean, we we store the class name? like in the type field?
If thats what you mean, you could write some logic on top to handle some mapping look-up.
Or if what you mean is the collection name, you can use the store_in collection: 'foo' option to change the collection name.
That said, the first option I dont think we would want to have that on Mongoid TBH.


“swift ./-” does the same as “swift -”

Description
swift ./- should try and read the file named - in the current folder. Instead it reads from stdin, just like swift -.
Reproduction

Create a swift file name - in your cwd;
Run swift ./-

Expected behavior
Swift should execute the file just created.
Environment
swift-driver version: 1.87.3 Apple Swift version 5.9.2 (swiftlang-<IP_ADDRESS>.56 clang-15<IP_ADDRESS>.5)
Target: arm64-apple-macosx14.0
Additional information
No response

To solve this issue, try running swift run ./-
It works for me when I try this.

@sananthanarayan I get error: Could not find Package.swift in this directory or any of its parent directories. when doing swift run ./- (as expected…)

I have also tried with zero f these 2 commands, both work for me. Do they work for you?

swift ./-

swift "$(pwd)/-"

or with the swift run command try this modified version of that:
swift run -- ./-

Seems like a driver issue, swift -### ./- outputs a frontend command with the path -, which the frontend then interprets as the stdin. FWIW adding -working-directory="$PWD" avoids the issue, since the driver will convert it to an absolute path.


[Google-maps-flutter] UiKitView become unresponsive after interacting with image picker camera

Steps to Reproduce
flutter create bug
dependencies:
flutter:
sdk: flutter
amap_map_fluttify: ^0.25.1
image_picker: ^0.6.7+7
info.plist:
NSCameraUsageDescription
请求使用相机以拍照来修改头像，上传图片附件等
io.flutter.embedded_views_preview
YES
main.dart
import 'package:flutter/material.dart';
import 'package:google_maps_flutter/google_maps_flutter.dart';
import 'package:image_picker/image_picker.dart';

void main() async {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
        visualDensity: VisualDensity.adaptivePlatformDensity,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatelessWidget {
  MyHomePage({Key key, this.title}) : super(key: key);

  final String title;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      resizeToAvoidBottomInset: false,
      appBar: AppBar(
        title: Text(title),
        actions: [
          FlatButton(
              onPressed: () => Navigator.push(context, MaterialPageRoute(builder: (context) => ImagePickerPage())),
              child: Text('ImagePicker'))
        ],
      ),
      body: GoogleMap(
        initialCameraPosition: CameraPosition(
          target: LatLng(37.42796133580664, -122.085749655962),
          zoom: 14.4746,
        ),
        myLocationEnabled: true,
      ),
    );
  }
}

class ImagePickerPage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      resizeToAvoidBottomInset: false,
      appBar: AppBar(),
      body: Container(
        child: Center(
          child: RaisedButton(
              child: Text('ImagePicker'),
              onPressed: () =>
                  ImagePicker().getImage(source: ImageSource.camera, maxWidth: 1080, maxHeight: 1920, imageQuality: 90)),
        ),
      ),
    );
  }
}

before call ImagePicker camera, mapview could drag and swipe. but after call ImagePicker camera, mapview become unresponsive.

Hi @ykrank
Cna you try please with Google maps plugin : https://pub.dev/packages/google_maps_flutter

Hi @ykrank,
I see you are using this plugin amap_map_fluttify which is 3rd party. Can you try to run again without using that plugin and check you still see the issue ?
Thanks.

Hi @ykrank
Cna you try please with Google maps plugin : https://pub.dev/packages/google_maps_flutter

This app used in China, so google map plugin is not good answer.

Hi @ykrank,
I see you are using this plugin amap_map_fluttify which is 3rd party. Can you try to run again without using that plugin and check you still see the issue ?
Thanks.

perhaps you are right, this may be plugin's bug, but i can't find the cause.
I try to add gestureRecognizersBlockingPolicy:FlutterPlatformViewGestureRecognizersBlockingPolicyWaitUntilTouchesEnded, but nothing happen.

@ykrank, not sure what's your use case is. Are you also using map along with image picker ?
Can you try the official image_picker plugin example in your code and see if it works ?
Probably you need to raise this issue in 3rd party plugin repo.
Thanks.

@ykrank, not sure what's your use case is. Are you also using map along with image picker ?
Can you try the official image_picker plugin example in your code and see if it works ?
Probably you need to raise this issue in 3rd party plugin repo.
Thanks.

Yes, if I called image picker,  then ios amap became unresponsive.what else official dragable UiKitView plugin could I try? I tried google_maps_flutter, but could not connected.

@ykrank, not sure what's your use case is. Are you also using map along with image picker ?
Can you try the official image_picker plugin example in your code and see if it works ?
Probably you need to raise this issue in 3rd party plugin repo.
Thanks.

Yes, if I called image picker, then ios amap became unresponsive.what else official dragable UiKitView plugin could I try? I tried google_maps_flutter, but could not connected.

I tried google_maps_flutter, same error.
replace
AmapView(
        mapType: MapType.Bus,
      ),

With
GoogleMap(
        initialCameraPosition: CameraPosition(
          target: LatLng(37.42796133580664, -122.085749655962),
          zoom: 14.4746,
        ),
        myLocationEnabled: true,
        // mapType: MapType.Bus,
      ),

After called image picker camera, everything in map stun

flutter doctor -v
[✓] Flutter (Channel stable, 1.20.3, on Mac OS X 10.15.6 19G73, locale
    zh-Hans-CN)
    • Flutter version 1.20.3 at /Users/ykrank/Library/flutter
    • Framework revision 216dee60c0 (7 days ago), 2020-09-01 12:24:47 -0700
    • Engine revision d1bc06f032
    • Dart version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/ykrank/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/ykrank/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.6, Build version 11E708
    • CocoaPods version 1.9.3

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 48.1.2
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[!] VS Code (version 1.48.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    ✗ Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Proxy Configuration
    • HTTP_PROXY is set
    • NO_PROXY is localhost,<IP_ADDRESS>
    • NO_PROXY contains <IP_ADDRESS>
    • NO_PROXY contains localhost

[✓] Connected device (1 available)
    • iPhone (mobile) • 1e1ce461796bc73ebff49d5215dd80f0c4cce23d • ios • iOS
      12.4.5

@ykrank please provide a complete minimal reproducible code sample in one file that shows the issue you are facing which we can copy paste in IDE and run. Make sure it doesn't have 3rd party plugin / code in it.

@darshankawar
import 'package:flutter/material.dart';
import 'package:google_maps_flutter/google_maps_flutter.dart';
import 'package:image_picker/image_picker.dart';

void main() async {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
        visualDensity: VisualDensity.adaptivePlatformDensity,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatelessWidget {
  MyHomePage({Key key, this.title}) : super(key: key);

  final String title;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      resizeToAvoidBottomInset: false,
      appBar: AppBar(
        title: Text('地图'),
        actions: [
          FlatButton(
              onPressed: () => Navigator.push(context, MaterialPageRoute(builder: (context) => ImagePickerPage())),
              child: Text('ImagePicker'))
        ],
      ),
      body: GoogleMap(
        initialCameraPosition: CameraPosition(
          target: LatLng(37.42796133580664, -122.085749655962),
          zoom: 14.4746,
        ),
        myLocationEnabled: true,
        // mapType: MapType.Bus,
      ),
    );
  }
}

class MapPage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      resizeToAvoidBottomInset: false,
      appBar: AppBar(
        title: Text('地图'),
        actions: [
          FlatButton(
              onPressed: () => Navigator.push(context, MaterialPageRoute(builder: (context) => ImagePickerPage())),
              child: Text('ImagePicker'))
        ],
      ),
      body: GoogleMap(
        initialCameraPosition: CameraPosition(
          target: LatLng(37.42796133580664, -122.085749655962),
          zoom: 14.4746,
        ),
        myLocationEnabled: true,
        // mapType: MapType.Bus,
      ),
    );
  }
}

class ImagePickerPage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      resizeToAvoidBottomInset: false,
      appBar: AppBar(),
      body: Container(
        child: Center(
          child: RaisedButton(
              child: Text('ImagePicker camera'),
              onPressed: () =>
                  ImagePicker.pickImage(source: ImageSource.camera, maxWidth: 1080, maxHeight: 1920, imageQuality: 90)),
        ),
      ),
    );
  }
}

@ykrank thanks for the code sample. Could you also provide a video / gif that shows the issue ?
Thanks

No map data, but you can see shadow of location button and compass

Recording gif added @darshankawar

@ykrank,Have you set the Google maps key in ios/Runner, as mentioned in plugin's readMe:
https://pub.dev/packages/google_maps_flutter

@ykrank,Have you set the Google maps key in ios/Runner, as mentioned in plugin's readMe:
https://pub.dev/packages/google_maps_flutter

I said no data is network error,  please give attention to gesture but not no data.

@darshankawar Maybe this gif more clear

@ykrank,
in your code, the imagePicker code you are using seem to be deprecated.

Can you try the official image_picker plugin example code and see if it works for you ?

@ykrank,
in your code, the imagePicker code you are using seem to be deprecated.

Can you try the official image_picker plugin example code and see if it works for you ?

yes, i tried

@ykrank,
in your code, the imagePicker code you are using seem to be deprecated.

Can you try the official image_picker plugin example code and see if it works for you ?

you can see https://github.com/flutter/flutter/issues/65361#issuecomment-689937916

Issue replicable on latest master. Tested on iphone 6s. When moving around the map, if we open camera using image picker and then go back to the map and try to move around the location, it freezes.
RPReplay-Final1600090872.MP4.zip

flutter doctor -v
[✓] Flutter (Channel master, 1.22.0-10.0.pre.161, on Mac OS X 10.15.4 19E2269,
    locale en-IN)
    • Flutter version 1.22.0-10.0.pre.161 at
      /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision cba91c97d0 (9 hours ago), 2020-09-13 18:57:03 -0400
    • Engine revision 2fc054147e
    • Dart version 2.10.0 (build 2.10.0-121.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.5, Build version 11E608c
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 46.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.46.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.11.0

[✓] Connected device (4 available)
    • SM A260G (mobile)                   • 5200763ebcfa861f
      • android-arm    • Android 8.1.0 (API 27)
    • iPhone SE (2nd generation) (mobile) • 312977A7-2DB1-4737-AEBB-35FA6522A309
      • ios            • com.apple.CoreSimulator.SimRuntime.iOS-13-5 (simulator)
    • Web Server (web)                    • web-server
      • web-javascript • Flutter Tools
    • Chrome (web)                        • chrome
      • web-javascript • Google Chrome 85.0.4183.102

• No issues found!

@ykrank It also happened with webview_flutter plugin, below version 12.0.2,  it not happens.

Related: https://github.com/flutter/flutter/issues/66165
I have created a minimal repro sample.

@joniaiuser awesome!
Hoping fixing this ASAP~

@joniaiuser awesome!
Hoping fixing this as soon as possible~

😑

👌🏻

Are there any known workarounds to this issue other than removing the map or disabling the camera?

I tried on my iPhone XS with iOS 13.7 and could't repro. Could it be device or iOS specific? In this issue I can see it happened on iPhone 6s with iOS 13.5, is it reproducible on other devices or iOS versions?

I am experiencing this issue on

iPhone X iOS 13.7 & 14.0
iPhone 8 Plus iOS 13.7

Haven't tested this on any other devices or versions..

@darshankawar
Hi, May I know about in which version will this problem be solved?
We have an app where a key function has been affected by this issue, resulting in the version staying at v1.20.1.

@ykrank It also happened with webview_flutter plugin, below version 12.0.2, it not happens.

env: label_xcode11_6_flutter_1_20_4 , iOS UIImagePickerController was conflict with the webview_flutter plugin platform view.

It looks like the issue has been fixed in the latest stable.
The particular PR that fixed this issue was: https://github.com/flutter/engine/pull/21286
I'm going to close the issue.

same issue. How can I solve it?


Feature/che 377 beta 3 ds implementation netcetera

CHE-377
What this PR does
Adds 3DS functionality.
Add card number<PHONE_NUMBER>000008
Tap on the card row and select 'Perform 3DS'
See attached video
https://user-images.githubusercontent.com/5053148/121706771-9cf84980-cade-11eb-8241-de5582584f79.mov
Notable decisions & other stuff
n/a
Before merging
QA

[x] I manually tested this with a demo application (simulator)
[ ] I manually tested this with a demo application (real device)
[ ] I wrote unit tests for each new util-like function
[ ] I wrote at least basic e2e tests for the new functionalities or new paths
[ ] If this PR introduces UI changes, I manually tested this PR on iPhone 8 (iOS 10, 12, 14), iPhone 12, and iPhone 12 Max
[ ] If this PR modifies the behavior of an API, I did my best to make my new API backward compatible
[ ] If this PR modifies the API, I attempted to implement the feature with an example

Documentation

[ ] If this PR modifies the API, I updated the API Reference.
[ ] If this PR adds new options to the API, I wrote some documentation or guide in the online documentation
[ ] If the PR modifies the API, I wrote some documentation to deprecate it

After merging

Make sure a new release has been pushed to Cocoapods trunk

Nice job pulling this through 💪 ! Got a lot of questions :D
Main issue to me is that Primer and Netcetera are tightly coupled together. Would be great to have a bit of abstraction here!

Actually they aren't, I just forgot to remove the import statement. Netcetera is coupled with the ThreeDSecureService. Removed the import.


Episode Source endpoint returns status 500 for RAW server in Hianime

Describe the bug
When you try to call the episode source endpoint for an anime episode that only has the RAW server (on HiAnime) it returns 500 status code.
Expected behavior
Return the episode source objects.
Actual behavior
When using this server on HiAnime

It returns status 500.
Additional context
This error occurs in newer episodes that are recently released.

What does the error message say?

I'm not at my pc now but it was something along the lines of "can't find server"

Which anime has the raw server?
Then I can have a look at it myself.
:smiley:

So far this one and this one

I am not getting any errors on raw servers. Are you sure you are using the latest version of the api?

This is the endpoint I am trying to use: https://api-aniwatch.onrender.com/anime/episode-srcs?id=konosuba-gods-blessing-on-this-wonderful-world-3-19108?ep=123214

You're right! There seems to be some kind of edge case on that specific episode.
I will have a look into that.

I've created a pr that should fix the problems you mentioned. #43
A fix for the current version is adding &category=raw to the /episode-srcs endpoint.
Example: https://api-aniwatch.onrender.com/anime/episode-srcs?id=konosuba-gods-blessing-on-this-wonderful-world-3-19108?ep=123214&category=raw

@WBRK-dev thank for the quick fix

Great thanks!

BTW, is it possible to make it return the anime with RAW without the query parameter 'category'? Perhaps make it fallback to that if it gets a 500 by the default flow?

No, you need to check on your end what category you want to use.
The default is sub.
If there is a fallback then it would take much longer to get a response.

okay thanks, I can close the ticket now 💯


test(runctl): fix test_init_context() memory leak

test_init_context() in test/test_runctl.c creates ctx->config_ifinfo_array, but it's never deallocated.
In fact, it's even allocated twice to the same variable.
Allocated here:

https://github.com/nqminds/edgesec/blob/8e2f61a9287e16998998ab91fd0e4a9d983cccc2/tests/test_runctl.c#L152
https://github.com/nqminds/edgesec/blob/e940f3a613585f5be2a7ad4dbf6f3618ee1cf616/tests/test_runctl.c#L172

I've removed the extraneous allocation, and added a utarray_free(context.config_ifinfo_array); at the end of the test function to fix this memory leak.

Side-note, the cleanup code for a struct supervisor_context and struct app_config is currently a bit messy. Can we somehow refactor this into cleanup functions?

https://github.com/nqminds/edgesec/blob/8e2f61a9287e16998998ab91fd0e4a9d983cccc2/src/runctl.c#L567-L612
https://github.com/nqminds/edgesec/blob/8e2f61a9287e16998998ab91fd0e4a9d983cccc2/tests/test_runctl.c#L183-L196

Feel free to add PR for cleanup.


Verify that we can indeed open a connection to the newly found peer

https://github.com/diasdavid/node-ipfs-mdns/blob/master/src/index.js#L117
This needs: https://github.com/diasdavid/node-ipfs-swarm/issues/3

This was done long ago :)


Revert "remove python (#1518)"

This reverts commit bac97fa04ad8e9def814507b845d64038687f4a2.
Which issue does this PR close?

N/A
Rationale for this change

PR https://github.com/apache/arrow-ballista/pull/58 adds a Ballista Python client. It is based on datafusion-python which was originally in this repo but was deleted and moved to datafusion-contrib. This PR restores the original Python code so that the Ballista Python PR can be rebased on top of this.
What changes are included in this PR?

Revert the commit that removed the Python client.
Are there any user-facing changes?

No

@nl5887 fyi

@thinkharderdev are you good with this getting merged as a step towards getting the new Ballista Python bindings merged?

Yep, my Python is pretty rusty but it looks good to me :)


Playback

After starting and stopping a recording, the option to playback doesn't appear in the release version on UC, nor does the recorded path drawing work. I have no idea how to compile the code for the latest .dll, are you able to send to me?

uc version is outdated software works only partially Copy_User_Command that is responsible for recording isn't called so nothing is being recorded
"7. Off-site linking is strictly forbidden unless you are linking to a website that explicitly relates to game cheating or the topic at hand. All small articles and all supportive content should be posted on this website (with credits to the author where applicable). Leeway is given for large content where posting on this website would be a very time consuming process. Permission must be granted by a community staff member before you are allowed to link to such content. Linking to websites that do not relate to the UnKnoWnCheaTs community's goal, that sell goods or services (including but not limited to cheats), that are referral links, surveys, generate revenue, or that violate any of the other community rules (i.e. external cheat downloads) is strictly prohibited."
leave me your contact information

uc version is outdated software works only partially Copy_User_Command that is responsible for recording isn't called so nothing is being recorded
"7. Off-site linking is strictly forbidden unless you are linking to a website that explicitly relates to game cheating or the topic at hand. All small articles and all supportive content should be posted on this website (with credits to the author where applicable). Leeway is given for large content where posting on this website would be a very time consuming process. Permission must be granted by a community staff member before you are allowed to link to such content. Linking to websites that do not relate to the UnKnoWnCheaTs community's goal, that sell goods or services (including but not limited to cheats), that are referral links, surveys, generate revenue, or that violate any of the other community rules (i.e. external cheat downloads) is strictly prohibited."
leave me your contact information

Oh ok, I see..
My discord is ID is<PHONE_NUMBER>96653212
or just Voxina#3276


Android - Navigation back loses styles applyied programmatically

Environment
Provide version numbers for the following components (information can be retrieved by running tns info in your project folder or by inspecting the package.json of the project):

CLI: 8.2.0
Cross-platform modules: 6.6.0-next-2020-05-08-112112-01
Android Runtime: 6.5.3
iOS Runtime: 6.5.2
Plugin(s): nativescript-ng-shadow ^2.1.0
NativeScript-Angular: ~8.21.0
Angular:

Describe the bug
We have a directive to apply some styles programmatically (ex. backgroundColor, backgroundImage, ...) and shadow (using the plugin above) based on a JSON downloaded from a server. On Android to get the styles working we have to use a setTimeout with a non-empty time value (ex. setTimeout() => ... , 2) ).
The problem is only on Android when using the back button. Returning to a previous created page made all set styles lost. To avoid this I listen to router changes and, inside a setTimeout, I change the backgroundColor of the HTMLElement first to '' " then to its previous value. This seem to trigger the redraw of the page.
This solution causes flickering and, if the background is a remote image, the re-download of it, so it's far from perfect.
To Reproduce
On Android, apply a directive to a StackLayout component, setting the backgroundColor style of the HTMLElement. Then navigate to a page and return back.
Expected behavior
The programmatic styles are not lost on back, and backgroundImages are not reloaded like on iOS
Sample project
Currently I don't have time to set one up, I'll try in the weekend
Additional context

In addition the back stack is completely broken in my case. when I navigate to a page then I press back I see wrong styles, then when I press back the app again navigates to the page I was in and when I press back again I see the first page with correct styles.
I'm using "@nativescript/angular": "10.0.0" and "@nativescript/core": "rc"


Draggable

The underlying OrgChart object (looking at your ES6 examples) seems to support some basic ability to drag nodes to reorder the org chart, but I don't see how to do that in the Vue version.  is this supported?

Use this guy's: https://github.com/spiritree/vue-orgchart
https://github.com/spiritree/vue-orgchart/blob/c8de4152e2839e28af5561bcce9b4af0d3f046a4/docs/edit.md
It's a difference of using:
npm install vue-organization-chart -S
versus
npm install vue-orgchart -S
I think...

I try this too but it seems not work with vue 3. I've also tried this https://www.npmjs.com/package/vue3-organization-chart, it works with vue 3 but it doesn't have as much feature like the one you sent. Is there any solution?


Feature request: option to prepend style rules with component names

The displayName option is great and helps the DOM readability a lot. However, currently the actual style rules, the CSS, is outputted using only the hashed names. E.g., if I have a component, it's in the DOM as something like this:
<div class="myFile__MyComponent-asdf12 qwer98">

But the CSS will look like this:
.qwer98 {
  /* rules */
}

It would be easier to read the styles in the browser, if they'd also get prepended with the component name, as that's the one I've written in the code and that's what my eye tries to seek, not a random hash, when I'm debugging. For instance, when overriding Material UI components, the rule list is full of different classes and rules, so it's not always trivial to find the one you've put the with styled-components.
Naturally, this could be an opt-in option that you could set in the .babelrc, and only for development, if one so wishes (probably does).

I don't think this is something we'd particularly want to do. Generally, the names are always available and a class always corresponds with a single component, which you'll likely have available if you're looking at the rules. This is of course unless you're looking at CSS output. However, I believe looking at the CSS output of styled-components without the HTML/DOM contents isn't going to be a particularly pleasant experience.


treewide: Remove manpage generation

They clutter the repo and configs unnecessarily.
Since Isso is not being packaged in debian any more, we have little reason to keep them.
Improve docs and commmand line reference instead.

I am packaging isso for ArchLinux though: https://aur.archlinux.org/packages/isso
But ok, I removed the manpages as well in the PKGBUILD.


Training error

I'm trying to train with images from example
Tesseract 5.0
Leptonica 1.80.0
Error: https://pastebin.com/DdATKMxC

The error is reported: Error: missing ground truth for training.
So you did not provide enough ground truth (lines) for training. How many lines did you have?

Sorry, there were really few images


Remove includedAdded

Since we filter the before execution output snapshot, we don't need
includeAdded any more.

@lptr PTAL!


updated plan sharing to be case-insensitive

Addresses #550
Adds a where_case_insensitive method to User which takes as input the field to search for, and the value to match.
Updated the roles_controller to use this new method and respond with the correct case of the user's email in the message

Didnt realise this was still sitting in the PR's
This Now also includes an update to the default pdf settings. @sjDCC
All margins are set to 10, font-size is set to 10, and the font is changed to times new roman.
PDF demo below.
sam's plan.pdf

Looks good @xsrust


Use generated Open Liberty password rather than hard-coded

Signed-off-by: Scott Kurz<EMAIL_ADDRESS>Fixes #335.

@scottkurz we don't know whether this has broken the build or not because when I merged this PR, all of the Liberty tests were switched off. I tried  to fix the previous framework Error issue and could not recreate against the latest liberty build. I then switched on all of the tests. I only saw the security tests broken, so I thought what you have changed could be the cause.
Even now, I am not sure whether this PR is ok to go ahead as yet (even though it is unlikely), because we need to fix the framework issue first. I think we have to stop doing any liberty changes before the tests can be switched on.


Docker redo

Closes #56

Hi @SuperSandro2000,
I get following exception with your docker:
Step 6/11 : RUN go build -ldflags='-extldflags "-static" -s -w' -o /go/bin/compy
 ---> Running in a13eac30d4ee
# _/root/go/src/github.com/barnacs/compy
/usr/local/go/pkg/tool/linux_amd64/link: running g++ failed: exit status 1
/usr/lib/gcc/x86_64-alpine-linux-musl/9.2.0/../../../../x86_64-alpine-linux-musl/bin/ld: cannot find -ljpeg
collect2: error: ld returned 1 exit status

Any clue?

@lolobosse no clue, builds for me.

@lolobosse I got the same problem, looks like something to do with Alpine's glibc maybe? Switching to the non-Alpine Go image (i.e. 1.16 instead of 1.16-alpine) fixed it for me. Updated PR incoming.

@barnacs bump


View Component should not have it's own ViewModel

In #360, a dialog window was added with its own ViewModel. This should be adjusted so that the ViewModel is a part of any fragment that would want to launch the dialog, and the text of the dialog should be set with setters on the window.

5654bd34eb63bfa3ebcd975e43cf8f41f5a99032


[refactor]: migrate to use collect_env in mmcv

This PR migrates to use collect_env in mmcv.

Task linked: CU-4dnmvy MMDetection3D


[Kubernetes] Load Balancer may enter Failed state when updating deployment

Problem
Service could lose incoming connectivity after upgrade deployment or service. While checking the service from Kubectl, it does have a public IP address, but that connection to that IP simply timeout.
Investigation
We found out that the load balancer was configured correctly but with a 'Failed' provisioning state. This indicate the request to update load balancer (from master SPN) eventually failed. From activity logs, it appears the same, the last update to LB was when we update the deployment. Note the NRP operation is a long running operation. It appears that the controller on master node made the request to update LB, but did not follow the long running operation till completion. As a result when the operation fails, it leaves a broken LB and break all incoming traffic.
Mitigation
The mitigation is not difficult, simply GET the LB and PUT the exact JSON back (basically retry the last update operation). Once the operation succeed, the connectivity resumed.
Recommended Fix
the controller should follow the Azure-AsyncOperation header to track the operation, in case it fails retry the operation.

@colemickens I know the code is probably in Kubernetes, but since all ACS clusters could hit this issue potentially, I figured it's good to track it here.
Also given this is found from Livesite and has a relatively bigger impact, I recommend we prioritize this fix.

Could this issue be related to the fact that I'm currently unable to create a Service of type LoadBalancer? It seems that there's no public IP generated and the service stays in pending state. Tried it twice now and waited an hour, without success.

@timbrand Please file a separate issue and include logs from kube-controller-manager. And specify if this is with ACS or ACS-Engine.
@yangl900 Can you please check and/or provide the k-c-m logs? I would suspect it's still trying to create the LB, unless there is a glaring bug where we swallow errors (I don't see any from skimming the code quickly). Does the audit log, or the k-c-m logs say why the provisioning failed?
FYI, We just use the Go SDK and it definitely handles long polling (otherwise creating the service would never take 3-5 minutes and none of it would really work because there's an sequencing that only works via long polling).

@colemickens  I checked the kcm logs, it only said updating LB and there is one trace saying completed. I didn't find more details. Any keyword should I look at? Unfortunately the activity log also did not say why the LB enter failed state. I understand we handles long polling, but do we verify the terminal state is 'Succeeded'?

The Go SDK should surface any non 200s as an error and the code would react accordingly and surface an error in the logs and not complete successfully. But you have an interesting point, I've not specifically seen cases where the long poll eventually fails (more accustomed to it failing in the sync portion), nor have I looked at the code that does error handling on the long poll. We've certainly had bugs around polling behavior and handling in the past.
It might be worth filing a bug on the Azure SDK For Go (or go-autorest) to do a sanity check to make sure its not swallowing any 4xx/5xxs inadvertently before it propagates to me.
I can't remember if it's output at the default verbosity level, but when it completes it should've printed the external IP? I guess maybe you sent logs via corp email, I'll take a look on Monday.
Otherwise, I'm not quite sure what else to suspect; I've had problems finding historical stuff in audit log as well. Maybe we can look at internal logging/tracing to get an clue? I'm not even quite sure how to even instrument this right now for the future without a better idea of what went awry here...

well, the long polling always return 200. Usually the caller (Kubernetes, not the SDK) is responsible for checking the 'provisionState' property value, to verify if the operation succeed or not.
Also note that, in this case the external IP did not change, we updated an existing deployment, so only the node port changed (where the LB prob need change).
btw - where is the code for this azure specific driver/controller?

well, the long polling always return 200

Ah. It would be good to know the situations in which Azure LB will take a request in the sync portion but return a failure in the async portion. I've personally never seen this.

Also note that, in this case the external IP did not change, we updated an existing deployment, so only the node port changed (where the LB prob need change).

Oh, this is interesting. I'd be curious to know more about the exact change

btw - where is the code for this azure specific driver/controller?

https://github.com/kubernetes/kubernetes/blob/master/pkg/cloudprovider/providers/azure/azure_loadbalancer.go#L147 That's where we update the LB specifically. To add some context, can you tell us if this has happened repeatedly, or just once? Thanks.

(made some inline edits to the above post, posting this to trigger email notification to look at the current incarnation on GitHub)

The "Failed" state LB happened only once, but to 2 of our clusters. I think it was something wrong in NRP as an intermediate error, since retry the exact PUT fixed the LB.
Looking at the Kubernetes code, I don't think we check the end provisioningState. Should we fix that?
For your question - no, we didn't specify the NodePort, and update deployment changed the node port. I am guessing it is because the new pod was scheduled on same node. and that's why LB needs an update.

Looking at the Kubernetes code, I don't think we check the end provisioningState. Should we fix that?

I very much think that should be fixed in go-autorest so that all clients benefit from this. It's very unexpected that the long polling would swallow errors (if that is indeed the case), from an SDK perspective, in my opinion.

For your question - no, we didn't specify the NodePort, and update deployment changed the node port. I am guessing it is because the new pod was scheduled on same node. and that's why LB needs an update.

Hm, that's confusing. I've never seen that before (not to say it's impossible), but in k8s the Service and Deployment objects are totally independent. Changing the deployment should have absolutely zero effect on the Service object (or the LB behind it).

It looks like it checks for certain status codes when polling, rather than the provisioning status: https://github.com/Azure/go-autorest/blob/master/autorest/sender.go#L157

Briefly looked at the go-autorest code. I think this is the polling method should be used by the LB CreateOrUpdate() method:
https://github.com/Azure/go-autorest/blob/8a1913be6639bf0013293832bb0ccfe221287d9c/autorest/azure/async.go#L36
And it appeared been used too:
https://github.com/Azure/azure-sdk-for-go/blob/master/arm/network/loadbalancers.go#L103
now I'm a bit confused...Do you know if we have mock test for the LB failed to update case?
btw - we also do a service update (PUT, but no property change) during deployment, I should probably mention that, not sure if that caused the node port change...

Do you know if we have mock test for the LB failed to update case?

I know that we do not have a mock test for it. (and had I, I probably would've mocked the sync portion. I've asked the folks to generate interfaces/mock clients for testing, as it's a bit tedious to do today due to how the clients are generated.)

we also do a service update (PUT, but no property change) during deployment, I should probably mention that, not sure if that caused the node port change...

Interestingly, I think you're hitting a bug that we just spent a lot of time tracking down. (Issue) (Fix PR)
But that still doesn't answer:

Why did NRP hit a provisioning failure?

Why did this not surface up through the SDK and out of the service controller?

I had one other thought on what might be happening -- the LB port gets updated and the rules are updated with the probes being updated. But that's actually allowed anyway, and we update the LB all at one time.
I'll try to look at NRP logs on Monday.

Thanks.  Just fun facts for your question - one of the clusters got a node NIC left in 'Failed' provisioningState, and that blocked all further operations of the LB. I don't think anyone touched the NIC in control plane in anyway. So, maybe it's the same case that LB fall into 'Failed' state without a control plane operation.
I don't know if it's possible, but if the controller can reconcile all of the cluster resources into goal state 'Succeeded', it could be very helpful. Probably we need assume the resources esp network can fail any time.

Is this issue resolved? @colemickens

this was fixed in https://github.com/kubernetes/kubernetes/issues/43233

I think I am running into a similar issue. Being pretty new to Azure: how would you PUT the LB config?
This has given me the json of the LB:
az network lb show -n ... --resource-group ...
I'd like to see if that solves my problem as well or if I'd need to open a new issue.

Same issue, can somebody please advise me how to "GET the LB and PUT the exact JSON back"?
k8s is not getting IP and saying pending. When I see the Azure resource, I see multiple entries for static IP's.


Migrate seqr propmap calculation to metamist + add ar-guid support

Relies on: https://github.com/populationgenomics/metamist/pull/538

Moves the propmap calculation to metamist,

we can visualise it in the browser

implement support for the newer propmap format (with sequencing-group support)
Sneak in ar-guid mapping for seqr

Codecov Report
All modified lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (main@a705084). Click here to learn what that means.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #164   +/-   ##
=======================================
  Coverage        ?   88.04%
=======================================
  Files           ?        4
  Lines           ?      393
  Branches        ?        0
=======================================
  Hits            ?      346
  Misses          ?       47
  Partials        ?        0

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


Add Details about Testing the Project

Fixes #436
Please make sure the below checklist is followed for Pull Requests.

[ ] Checks are green
[ ] Tests are added where necessary
[x] Documentation is added/updated where necessary
[x] Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

Merging this, I'll commit directly on master to fix the typo

Thanks :smile:


Clarification of README

This is probably not a bug, but a request for clarification in README.

Point it to the base URL of your website followed by /actions/cloudinary/notifications/process?volume={VOLUME_ID}. Remember to replace {VOLUME_ID} with the relevant asset volume ID.

As far as I understand the volume Id is not something that is exposed anywhere, although the volume does have an Id. Only the handle is exposed, however replacing {VOLUME_ID} with the handle doesn't seem to work.

You can find a volume's ID by going to its settings in the admin panel. The ID will be displayed in the URL as follows: admin/settings/assets/volumes/{VOLUME_ID}. I agree that this could use some clarification in the README.


deepin.dde-api: init at 5.5.32

Description of changes
https://github.com/linuxdeepin/dde-nixos/issues/9

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofborg build deepin.dde-api

@ofborg build deepin.dde-api

@ofborg build deepin.dde-api

@ofborg build deepin.dde-api

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1811


Fix director default response and add HEAD as allowed verb

Short and sweet, this should allow us to issue HEAD requests against the API the /api/v1.0/director/<origin or object> endpoints. The tagged issue also mentioned making this compatible with our shortcut middleware in the director, but it looks like it already was -- the middleware was agnostic of verbs that weren't PUT. My addition to the middleware here explicitly ties GET/HEAD requests to the appropriate redirect functions so that other verbs we don't anticipate don't creep in.
One thing to note is that if a client who issues a HEAD for some resource chooses to follow the redirect, XRootD appears to hang with an open connection. Since that's in XRootD land, I'm considering it outside the scope of this small fix.
Closes #754

@jhiemstrawisc Congrats on opening the #1000 PR! ⭐


Add action dispatcher to runtime

Handle common things like exceptions,  inAction-Timing, etc. for all different call-sites

for later ...


Lists Windows commands, but not Linux commands

Should include both Windows and Linux commands.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b3ff94e6-3c1f-b9e1-1a46-b75fe2ffca1b
Version Independent ID: 8f3ca735-ddd4-f4a9-4ee0-189604018784
Content: Run PowerShell scripts in an Windows VM in Azure
Content Source: articles/virtual-machines/windows/run-command.md
Service: automation
GitHub Login: @georgewallace
Microsoft Alias: gwallace

@jtuliani Thanks for reaching out, this doc is specific to Windows, Linux has it's own version of Run Command for Linux. You can see the command for Linux here https://docs.microsoft.com/en-us/azure/virtual-machines/linux/run-command#available-commands

Closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.

I found the page via Google. Having done so, it easy to miss that this is a Windows-specific page, and that there's a separate page for Linux. And even harder to actually find the Linux page, since they're far apart in the left-nav.
I suggest there should be a banner at the top of all such Windows-only pages explaining this is the Windows page and linking to the Linux page. And vice-versa.


Unhelpful error message when struct name doesn't match

When deserializing a string that defines a RON struct with an optional class name that does not match the name of the Rust struct, ron::from_str returns an error that displays as Error: 1:1: Expected struct. It took me quite a long time to figure out what the issue was. It would be more helpful if the error message was something like Error: 1:1: Expected struct "Name" but got struct "ActualName".

I'm also getting this error "Failed to load config: 1:1: Expected struct" in the decode_file.rs example via example.ron even when using the exact same files, not entirely sure why it's asking for a Struct.

I just had a very brief look at the codebase to see how difficult it would be to implement this. Here are my thoughts:

A simple solution would be to also report the name of the struct that was expected. This would already give most of the debugging value as you could then manually look at the RON file to see what name you provided. Implementing this option is trivial.

To find out if a different identifier was given instead, ron would have to check if any valid identifier but wrong identifier is given and that the opening ( exists. In the case where the only error is the wrong struct name, this could certainly be achieved, though at the cost of a bit of extra code clutter. However, any further edge cases would still give the original error messages.

Personally, I think that option (1) could be a good start as it provides most of the error reporting benefit already while requiring very few code changes and no additional code complexity. Option (2) could then still be implemented later on as a refinement.


[OSD-19887] change the update manifestwork logic to increamental

for the clusters with manually added pull secret. we do not touch them at all.
it will only update the existing pullsecret with the matched auth in new pullsecret. And append anything new from the new pullsecret.

/lgtm

/approve


Fastclick removal regresses iOS cordova app responsiveness

Framework7 version: 5
Platform and Target: iOS 11+ Cordova app

Describe the bug
The recent removal of the embedded fastclick library in Framework7 v5 has re-introduced the 300ms click delay in Cordova iOS applications. Even with the most recent versions of cordova (9.0 with iOS platform version 5+) and iOS (13.1.3), the webkit versions in use are very old: AppleWebKit/605.1.15.
Without the fastclick library, the 300ms lag when clicking on buttons, links etc is re-introduced, making applications feel laggy to use.
Fortunately, the upstream library (https://github.com/ftlabs/fastclick) is still easily installable, so I've worked around this by packaging the fastclick library myself with the application to improve the speed once again.
Additional Context
I'm raising this defect however as Framework7 is specifically listing cordova & iOS as supported configurations, so thought perhaps it may be of interest to know.

the webkit versions in use are very old: AppleWebKit/605.1.15.

You need to use WKWebView to have latest web view where this is not the issue

Hi @nolimits4web ,
Apologies if I've missed the obvious, but I am using cordova-ios v5.0.1, with the https://github.com/apache/cordova-plugin-wkwebview-engine plugin, and have added the following into my config.xml:
  <plugin name="cordova-plugin-wkwebview-engine" spec="^1.2.1" />
  <platform name="ios">
        <feature name="CDVWKWebViewEngine">
            <param name="ios-package" value="CDVWKWebViewEngine" />
        </feature>
        <preference name="CordovaWebViewEngine" value="CDVWKWebViewEngine" />
    </platform>

When the app launches, it's clearly stating that the WKWebView is in use, though the navigator version is still listed as the one I inserted at the top.
Are you able to provide any tips as to why I'm not seeing a more recent browser as you'd expect?
Thanks for your help here and this wonderful framework.

More proof of this, here is the starting banner for the app:
2019-11-14 18:36:59.565798+1100 <....>[1128:696117] Apache Cordova native platform version 5.0.1 is starting.
2019-11-14 18:36:59.565919+1100 <....>[1128:696117] Multi-tasking -> Device: YES, App: YES
2019-11-14 18:36:59.578236+1100 <....>[1128:696117]

Started backup to iCloud! Please be careful.
Your application might be rejected by Apple if you store too much data.
For more information please read "iOS Data Storage Guidelines" at:
https://developer.apple.com/icloud/documentation/data-storage/
To disable web storage backup to iCloud, set the BackupWebStorage preference to "local" in the Cordova config.xml file
2019-11-14 18:36:59.706278+1100 <....>[1128:696117] CDVWKWebViewEngine will reload WKWebView if required on resume
2019-11-14 18:36:59.706336+1100 <....>[1128:696117] Using WKWebView
2019-11-14 18:36:59.706498+1100 <....>[1128:696117] [CDVTimer][console] 0.038028ms
2019-11-14 18:36:59.706569+1100 <....>[1128:696117] [CDVTimer][handleopenurl] 0.037074ms
2019-11-14 18:36:59.707412+1100 <....>[1128:696117] [CDVTimer][intentandnavigationfilter] 0.813007ms
2019-11-14 18:36:59.707483+1100 <....>[1128:696117] [CDVTimer][gesturehandler] 0.036001ms
2019-11-14 18:36:59.709140+1100 <....>[1128:696117] [CDVTimer][file] 1.620054ms
2019-11-14 18:36:59.735027+1100 <....>[1128:696117] [CDVTimer][splashscreen] 25.839090ms

I've triple-checked this today, and it seems that v605 of the browser is the latest on iOS 13, and the fastclick addition still has a noticeable improvement on the responsiveness.

Hrm. After cleaning everything (including node_modules etc), and rebuilding from scratch, I can confirm the fastclick module isn't required, as you said @nolimits4web . Seems like something was not correctly rebuilt in my earlier attempts.
Apologies for wasting your time on this! Thanks again 😃


fix(lib): Fix a bug in constructor functions and Object.assign

Codecov Report

Merging #9 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master     #9   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          15     15
  Lines         215    213    -2
=====================================
- Hits          215    213    -2

Impacted Files
Coverage Δ

lib/object/assign.js
100% <100%> (ø)
:arrow_up:

index.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[IDEA] Prevent Update Message for Plugins Require Newer TiddlyWiki

Each plugin declares the required TW core version
TiddlyWIki-plugins shows a message if it finds a newer plugin installed on the Wiki
Add this feature to not allow updating. not display the update message if the core version required by a plugin is higher than the host TiddlyWiki.

Reason: I am updating some kookma plugins like Shiraz, Utility, Favorites which require TW 5.3.3 above. Also some are NOT back compatible.

I think you means CPL. I think this is not difficult to add, but I'm working on other stuff, and I personally don't need this because TidGi app is always in latest tw version.

Yes, I mean CPL. No rush, add it when you find spare time.
Thank you

I think we already have this feature
https://github.com/tiddly-gittly/TiddlyWiki-CPL/blob/20ab462b21cfdba0c24994a839dde93d77bab3eb/tiddlers/CPLPlugin/core.js#L107


Feature Request - Paging: Jump to page n

Data table paging - Capability to jump to page n

@ishatd can you post a link or screenshot with an example of exactly how you & your team are envisioning this feature?

I asked around the group. Sounds like the request is relevant to the PrimeNg "PageLinks" feature.  https://www.primefaces.org/primeng/#/paginator

Bueno thx @thomaslennan


Show current configuration info/data as "pretty" rendered JSON in the util box

We may as well use it for something!

Implemented on: https://github.com/vkottler/conntextual/pull/24.


fix github actions AWS permissions

Staging post-deploy func tests failed due to GHA not having correct AWS perms:
Error: Failed to list store contents: AccessDeniedException: User: arn:aws:sts::***:assumed-role/cellxgene-github-actions-dev/GitHubActions is not authorized to perform: secretsmanager:GetSecretValue on resource: corpora/backend/staging/auth0-secret because no identity-based policy allows the secretsmanager:GetSecretValue action

Likely introduced by https://github.com/chanzuckerberg/single-cell-data-portal/pull/4702/files
An earlier perms fix was made here

It looks like the perms should exist:
https://github.com/chanzuckerberg/single-cell-infra/blob/main/terraform/modules/hosted-cellxgene/iam.tf#L248

Lower priority than fixing staging/prod deployment issues, but FYI, I just noticed that our happy stack reaper GHA is also failing with AccessDeniedExceptions. e.g.:

https://github.com/chanzuckerberg/single-cell-data-portal/actions/runs/4873370595/jobs/8692864143

[FATAL]: unable to write a workspace param: could not write parameter to /happy/data-portal/rdev/stacklist: operation error SSM: PutParameter, https response error StatusCode: 400, RequestID: 668eb4ba-9677-4199-bc37-ce7b6c7a97e4, api error AccessDeniedException: User: arn:aws:sts::***:assumed-role/cellxgene-github-actions-dev/HappyCleanupSingleCellDPRdevStacks is not authorized to perform: ssm:PutParameter on resource: arn:aws:ssm:us-west-2:***:parameter/happy/data-portal/rdev/stacklist because no identity-based policy allows the ssm:PutParameter action

Lower priority than fixing staging/prod deployment issues, but FYI, I just noticed that our happy stack reaper GHA is also failing with AccessDeniedExceptions. e.g.:

Thanks! Moved into a new issue, so that I can close this one and feel like I'm making progress in my life. 😄


using HTTP HEAD request to invalidate cache

As far as I can see, the image caching does not check if the image stored in the local cache is still up to date with the image on the server (in case its a downloaded image of course). I think it would be a good addition to use the HTTP HEAD request (this sends you the headers => last motif timestamp back without the actually large image data itself)  to verify if the image is outdated and in that case perform a new download from the server.

We likely don't want to do this manually but allow for some way to reissue the network request for a cached image so the server can tell us one way or another. One of the benefits of a non-persistent cache is that the app will redownload the images when it's relaunched anyway. We'll investigate this for the future.

Indeed needs investigation! It might be a good idea to not only support a HEAD request but also allow a custom closure to specify if the cache is still valid or if the image should be re-downloaded. I just had a project where the images ware downloaded via HTTPS (AlamofireImage) but the last modified timestamp of the image was checked via a Firebase API call...

It would be terribly inefficient to issue HEAD requests every time rather than load directly from the queue. Instead, we would want to expose the ability to ignore the cached image to force the image to re-download. As a non-convenient workaround, you could dig into the URLCache directly and remove the cached response before making the request if it's a blocker type issue.


Hi, could u fix some pages with special conditions

The rage is 2149-2386.  2208, 2279 are empty; 2227, 2307 no "Year"; 2365, 2383, 2385 are special (not a real film page).

cannot run at 2208

@Xiaoling123456 fixed with https://github.com/serv0id/sarajevo-scraper/commit/5634c7a82305fd1672b0ecd0a4aac1a54f82e5d0.
Can you please check and revert?


Add Carp

Add Carp language
Checklist:

[x] I am adding a new language.

[x?] The extension of the new language is used in hundreds of repositories on GitHub.com.

Search results for each extension:

https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3AFOOBAR+KEYWORDS+NOT+nothack

[x] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

[URL to each sample source, if applicable]

Sample license(s):

[ ] I have included a syntax highlighting grammar: https://github-lightshow.herokuapp.com/
[ ] I have updated the heuristics to distinguish my language from others using the same extension.

@wltsmrz Please fill out the pull-request template, and be sure to have familiarised yourself with the criteria for adding a new language. Note that if the language lacks sufficient real-world usage, it's  ineligible for addition until it does. Judging from the project's readme,

WARNING! This is a research project and a lot of information here might become outdated and misleading without any explanation. Don't use it for anything important just yet!

I'd say Carp has yet to achieve the minimum required distribution of hundreds of users.
Please consult CONTRIBUTING.md to learn more.

I'd say Carp has yet to achieve the minimum required distribution of hundreds of users.

I'll see if I can find out how many unique Carp repos there are on github, I'm curious myself.

My search topped out at 114 unique repositories.
As a reference, I compared the numbers I was getting for Carp to MoonScript, which tops out at 213 despite being added to linguist Jan 2013 and likely benefiting in terms of popularity from the free marketing of github tag. Although it may be a close call IMO, perhaps it doesn't meet today's popularity requirements.
See you down the road then?!

It was brought to my attention that there was already a PR to add Carp https://github.com/carp-lang/Carp/issues/972#issuecomment-728919176
Although there are currently 114 repos I'll close this in favor of bumping the popularity tracking issue (https://github.com/github/linguist/issues/4219)


Use GitHub URLs instead of local paths in SJSIR files

This replaces local file paths in Scala.js SJSIR files with GitHub URLs.
It enables to see circe sources in the browser in Scala.js projects and gets rid of warnings like these in projects that use scalajs-bundler and webpack:
[warn] Module Warning (from ./node_modules/source-map-loader/index.js):
[warn] (Emitted value instead of an instance of Error) Cannot find source file
  '../../../../../../../../../../../../home/travis/code/projects/circe/modules/core/js/target/scala-2.13/src_managed/main/io/circe/TupleDecoders.scala'

Note that the same is done in Cats.

Codecov Report

Merging #1703 (839ca4e) into master (f144acd) will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1703      +/-   ##
==========================================
- Coverage   87.08%   86.99%   -0.09%
==========================================
  Files          59       59
  Lines        2392     2392
  Branches      136      136
==========================================
- Hits         2083     2081       -2
- Misses        309      311       +2

Impacted Files
Coverage Δ

...re/shared/src/main/scala/io/circe/MapDecoder.scala
88.00% <0.00%> (-8.00%)
:arrow_down:

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
88.30% <0.00%> (+1.16%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks for catching this!