I got loss = nan .....

Solving VGG_VOC0712_SSD_300x300_train
I1112 16:14:25.373922 12274 solver.cpp:295] Learning Rate Policy: multistep
I1112 16:14:26.191864 12274 solver.cpp:243] Iteration 0, loss = 393.183
I1112 16:14:26.191922 12274 solver.cpp:259]     Train net output #0: mbox_loss = 463.676 (* 1 = 463.676 loss)
I1112 16:14:26.191949 12274 sgd_solver.cpp:138] Iteration 0, lr = 0.001
I1112 16:14:37.323004 12274 solver.cpp:243] Iteration 10, loss = 1.70709e+06
I1112 16:14:37.323065 12274 solver.cpp:259]     Train net output #0: mbox_loss = 1.00702e+07 (* 1 = 1.00702e+07 loss)
I1112 16:14:37.323082 12274 sgd_solver.cpp:138] Iteration 10, lr = 0.001
I1112 16:14:48.218691 12274 solver.cpp:243] Iteration 20, loss = nan
I1112 16:14:48.218749 12274 solver.cpp:259]     Train net output #0: mbox_loss = nan (* 1 = nan loss)
I1112 16:14:48.218765 12274 sgd_solver.cpp:138] Iteration 20, lr = 0.001
I1112 16:14:59.353508 12274 solver.cpp:243] Iteration 30, loss = nan
I1112 16:14:59.353610 12274 solver.cpp:259]     Train net output #0: mbox_loss = nan (* 1 = nan loss)
I1112 16:14:59.353627 12274 sgd_solver.cpp:138] Iteration 30, lr = 0.001
I only put the scripts to the /src and /include, then modified the train.prorotxt 。。。。but the loss
is 。。。。emmmm。。。
there must be something wrong....
shoule I do some other change????

I modified the base_lr=0.0001  then it gradual convergence.....

how to add multibux_focal_loss to the code ,this is no .cu file

@dubiwei  the multibux_focal_loss do not need .cu.....

after 12w iters, i got:
Iteration 120000, loss = 1.6058
I1114 06:50:20.753614 26071 solver.cpp:433] Iteration 120000, Testing net (#0)
I1114 06:50:20.753654 26071 net.cpp:693] Ignoring source layer mbox_loss
I1114 06:51:53.275323 26071 solver.cpp:546]     Test net output #0: detection_eval = 0.524345
I1114 06:51:53.275491 26071 solver.cpp:337] Optimization Done.
I1114 06:51:53.275498 26071 caffe.cpp:254] Optimization Done.
...........

I modified the max_iter to be 10w   but still get a lower map：.....  0.517319


reproduce problem

Thanks for your contribution. when i try to reproduce the experiment results with tusimple dataset, i found that 1)  the cls_loss is always 0 , is it normal? and the run test.py file with the 500th epoch .pth , it has a low accuracy,.can you give me some advices?

ok,  thanks for your recommend.


Outdated dependencies in the benchmark

Hi @dobbersc, I ran into errors while trying to run your benchmark, here is a small breakdown:
Numpy
Your script will not run for numpy versions >=2:
ValueError: numpy.dtype size changed, may indicate binary incompatibility. Expected 96 from C header, got 88 from PyObject
You need to explicitly pin numpy to an older version (e.g. by adding numpy==1.26.4 to your requirements) or to the current one by updating your evaluation.
LXML
Fundus blocks the other packages by requiring LXML versions < 5.
List of concerned packages:

justext (newest version: 3.0.1)
news-please (newest version: 1.6.12)
trafilatura (newest version: 1.11.0)

Could you please do something about it? Otherwise you will be running the evaluation using past versions of the packages.

Hi @adbar,
thanks for notifying us of the dependency issues. I restricted numpy to versions <2 in #5. We also plan on a minor release of Fundus to support lxml >=5. According to the project requirements of news-please and justext these packages should already support lxml >=5.
Whenever we wish to support a new version of a scraper for our benchmark or a user requests an update (e.g. via an issue), we create a release with the complete source and the updated scores. Thus, we ensure that the original scores from the paper and subsequent updates stay largely reproducible. In addition, the entire score history remains accessible through the releases page. We'll also add a new score column to the score table to indicate the scraper version from which each score originated.

Hi @adbar,
I've updated the evaluation results for the current latest versions of all supported news scrapers in #10. There's also a new release for the evaluation repository here. Trafilatura and news-please greatly improved :). If you have any further questions or issues feel free to reach out!

Nice, thanks for the update!


Spring boot with swagger can not run when package a jar

I used the example about the spring boot with swagger2 from url (http://www.tuicool.com/articles/Ibe6BzY) and download the source code from git (https://github.com/springframeworkguru/springboot_swagger_example) . After that I import the source  into Eclipse,everything is ok.
After I package the source code into a jar and run with " java -jar spring-boot-web-0.0.1-SNAPSHOT.jar" I get the error as following and the spring boot application can not be run.
org.springframework.beans.factory.BeanDefinitionStoreException: Failed to read candidate component class: URL [jar:file:/D:/sourcecode/springboot_swagger_example-master/springboot_swagger_example-master/target/spring-boot-web-0.0.1-SNAPSHOT.jar!/BOOT-INF/lib/springfox-swagger2-2.6.1.jar!/springfox/documentation/swagger2/mappers/LicenseMapper$License.class]; nested exception is java.lang.IndexOutOfBoundsException
How can I solve this problem? Thank you !

How are you generating the jar?

I make jar with eclipse .I right click the project then open a menu.I select run as --- maven build----enter package in the goal---- click run.After a while , eclipse shows build success.

You need to use the spring boot maven plugin. The goal should repackage I believe


Do we support multipart resources?

Overview
Specs-v1 has introduced concept of multipart resources:
{
    "resources": [
        {
             "name" : "name",
             "path": ["chunk1.csv", "chunk2.csv"]
        }
    ]

yes, it's supported
https://github.com/frictionlessdata/datapackage-php/blob/master/tests/DatapackageTest.php#L131
https://github.com/frictionlessdata/datapackage-php/blob/master/tests/fixtures/multi_data_datapackage.json

@OriHoch
Great!
So if we have:

resource.json

{
  "name" : "name",
  "path": ["chunk1.csv", "chunk2.csv"]
}

> chunk1.json

header1, header2
value1, value2

> chunk2.json

header1, header2
value3, value4
we could get:

```python
resource = Resource('resource.json')
resource.headers()
# header1, header2
resource.read()
# value1, value2
# value 3, value4

I've updated the title/description. Re-opening because I'm not sure I had a clearly formulated question at first place.


Envoy frequently resets http2 stream on doing downstream filters with NO ERROR

If you are reporting any crash or any potential security issue, do not
open an issue in this repo. Please report the issue via emailing
<EMAIL_ADDRESS>where the issue will be triaged appropriately.
Title: One line description
Description:
Envoy resets http2 stream on doing downstream filters on outbound traffic. The problem does not appear until the server has been running for a period of time.
I suspect there is a problem with the native jaeger tracing filter.
Repro steps:
Admin and Stats Output:

Include the admin output for the following endpoints: /stats,
/clusters, /routes, /server_info. For more information, refer to the
admin endpoint documentation.

Call Stack:

If the Envoy binary is crashing, a call stack is required.
Please refer to the Bazel Stack trace documentation.

I remove the native jaeger tracing filter, but the problem also happen.

@lizan Hi, can you help us? This problem bothers us very much.

It seems 'x-err-detail', 'stream+terminated+by+RST_STREAM+with+error+code%3A+NO_ERROR' is from upstream but not Envoy itself, or your log is from different Envoy.

It seems 'x-err-detail', 'stream+terminated+by+RST_STREAM+with+error+code%3A+NO_ERROR' is from upstream but not Envoy itself, or your log is from different Envoy.

@lizan Is it related with 13349?

@lizan Thanks for your reply. This log is from only one envoy. It has two incomplete calling process.  Sorry, It's not clear to see. If this problem reproduce again. I will put new log in here.

It seems 'x-err-detail', 'stream+terminated+by+RST_STREAM+with+error+code%3A+NO_ERROR' is from upstream but not Envoy itself

I once suspected that this problem was caused by upstream server. But the following log confused me. Please see this stream called S10909270796817708085.
2021-08-03T06:46:35.297256Z trace   envoy http  [C355666][S10909270796817708085] decode headers called: filter=0x55a9a32dccb0 status=0
2021-08-03T06:46:35.297262Z debug   envoy filter    override with 3 ALPNs
2021-08-03T06:46:35.297266Z trace   envoy http  [C355666][S10909270796817708085] decode headers called: filter=0x55a9a3b210a0 status=0
2021-08-03T06:46:35.297269Z trace   envoy http  [C355666][S10909270796817708085] decode headers called: filter=0x55a9a203efc0 status=0
2021-08-03T06:46:35.297272Z trace   envoy http  [C355666][S10909270796817708085] decode headers called: filter=0x55a99e458d90 status=0
2021-08-03T06:46:35.297276Z debug   envoy http  [C355666][S10909270796817708085] doEndStream() resetting stream
2021-08-03T06:46:35.297279Z debug   envoy http  [C355666][S10909270796817708085] stream reset
2021-08-03T06:46:35.297301Z trace   envoy wasm  [host->vm] proxy_on_log(510972)
2021-08-03T06:46:35.297301Z trace   envoy wasm  [host->vm] proxy_on_log(510972)
2021-08-03T06:46:35.297307Z trace   envoy wasm  [host<-vm] proxy_on_log return: void
2021-08-03T06:46:35.297310Z trace   envoy wasm  [host->vm] proxy_on_done(510972)
2021-08-03T06:46:35.297313Z trace   envoy wasm  [host<-vm] proxy_on_done return: 1
2021-08-03T06:46:35.297315Z trace   envoy wasm  [host->vm] proxy_on_delete(510972)
2021-08-03T06:46:35.297318Z trace   envoy wasm  [host<-vm] proxy_on_delete return: void
2021-08-03T06:46:35.297321Z trace   envoy main  item added to deferred deletion list (size=1)
2021-08-03T06:46:35.297326Z trace   envoy http  [C355666][S10909270796817708085] decode headers called: filter=0x55a9a32dcb60 status=1

This is an outbound request. The route filter is not being executed. Is it connect to an upstream server ?

@Patrick0308 from the config_dump and the log of the stream, I suspect this is due to the Lua filter, any script error will result a reset stream:
https://github.com/envoyproxy/envoy/blob/main/source/extensions/filters/http/lua/lua_filter.cc#L739-L744
You need to turn on log level to trace for lua and route as well to see if router is executed.

 For example
Http::FilterHeadersStatus FaultFilter::decodeHeaders(Http::RequestHeaderMap& headers, bool) {
  ENVOY_LOG(debug, "fault filter do decodeHeader");
  //...
}

config_dump file:
envoy-error-config-dump.log

It seems like wasm filter or fault filter due to reset stream.

It seems 'x-err-detail', 'stream+terminated+by+RST_STREAM+with+error+code%3A+NO_ERROR' is from upstream but not Envoy itself, or your log is from different Envoy.

I see "saw upstream close connection" in the envoy's log, like this,

Does it mean that the server Pod's Envoy terminate this stream?

It seems 'x-err-detail', 'stream+terminated+by+RST_STREAM+with+error+code%3A+NO_ERROR' is from upstream but not Envoy itself, or your log is from different Envoy.

I see "saw upstream close connection" in the envoy's log, like this,

Does it mean that the server Pod's Envoy terminate this stream?

This seems normal because upstream response is completed (one line above)

@lizan
I add some log on wam context decodeHeaders function.

I will not close this issue, because the issue is not be fixed.

Can you open an issue in Istio side as well to track? This is likely the wasm-based telemery implementation issue.

ACKed.


Make cards for participant blogs

Ian this done , we need it done asap

My laptop is at repair😔... Hard drive became corrupt!
Anyone else can take up the issue!

I'll do it

Do this fast. Add the blogs in the other issue.


multi spinner in one page

it would be nice to have more the one spinner in one page and load them on demand.
think of this scenario, you have a page with 2 async request, the second request starts after the first one depending on data provided from the first one. the first request fills the first table and the second one fills the second table. you do not want the page to wait until both requests finish to start using page. after the first request finish the loader must hide but the second loader must start within the second table container to show that the table is loading.
I don't know if this request or idea is achievable or not. but thank you for the nice work.

Hello @Muhanady ,
Thank you so much for your appreciation. :+1:
This is absolutely achievable. I'll think on it to provide a dynamic feature like you can have multiple instances of the spinner in the same page around a specific container component.

any news from this issue? or could i pull and try to build it my self after that we can review it together

Hack that you can use to display multiple spinners:
Instead of .hide(), set [timeout] to 0
and inseatd of just spinnerService.show()
set [timeout] to a vety large number and then call spinnerService.show()


Integrated examples/common/chip-app-server to the CHIP src/app/server.

Problem
examples/common/chip-app-server should is not a part of CHIP,
while is needed by most of CHIP devices and is based on CHIP API.
Summary of Changes

Created src/app/server directory to store all CHIP accessory device specific files.
Moved examples/common/chip-app-server content to the src/app/server.
Removed examples/common/chip-app-server.
Aligned READMEs to describe directories content.

Fixes #2927

this is great! but would you put these in src/app instead of src/accessory?

I mentioned why I did this in #2927, but we can also discuss it here. The thing is that to compile chip-app-server files, we need to have compiled src/transport and src/platform modules. Src/platform needs in indirect way through the src/lib to have src/app compiled before it can be compiled, so at the moment it's kind of impossible to compile src/platform before src/app. And that means that we can't compile files from chip-app-server inside of the src/app module. That's why I proposed creating new src/accessory module. Maybe it's not the only solution, but src modules already have some dependencies and chip-app-server was previously compiled after CHIP library, so now it's not so easy to find out how to integrate it in some of the existing modules to fulfill all dependencies.

this is great! but would you put these in src/app instead of src/accessory?

I mentioned why I did this in #2927, but we can also discuss it here. The thing is that to compile chip-app-server files, we need to have compiled src/transport and src/platform modules. Src/platform needs in indirect way through the src/lib to have src/app compiled before it can be compiled, so at the moment it's kind of impossible to compile src/platform before src/app. And that means that we can't compile files from chip-app-server inside of the src/app module. That's why I proposed creating new src/accessory module. Maybe it's not the only solution, but src modules already have some dependencies and chip-app-server was previously compiled after CHIP library, so now it's not so easy to find out how to integrate it in some of the existing modules to fulfill all dependencies.

You could potentially put it at src/app/server or src/app:server while we think about how to untangle this.

this is great! but would you put these in src/app instead of src/accessory?

I mentioned why I did this in #2927, but we can also discuss it here. The thing is that to compile chip-app-server files, we need to have compiled src/transport and src/platform modules. Src/platform needs in indirect way through the src/lib to have src/app compiled before it can be compiled, so at the moment it's kind of impossible to compile src/platform before src/app. And that means that we can't compile files from chip-app-server inside of the src/app module. That's why I proposed creating new src/accessory module. Maybe it's not the only solution, but src modules already have some dependencies and chip-app-server was previously compiled after CHIP library, so now it's not so easy to find out how to integrate it in some of the existing modules to fulfill all dependencies.

You could potentially put it at src/app/server or src/app:server while we think about how to untangle this.

@mspang that was first thing I tried, but I realized src/app/server files can't be compiled in the app static library, because of the reason described above. I have two ideas how to implement your suggestion:

do it similar to the src/app/util, so place files in the src/app/server and add it to sources in the external BUILD.gn (e.g. in lighting-common etc), but actually those server files still won't be integrated in the CHIP library, but only located in src/app.

place files in the src/app/server and create separate library from app, like app-server, having all dependencies set to the way to allow building it after src/platform and src/transport. That would allow integrating server to the CHIP lib.

I think that second option would be better, as using compiled CHIP library in the external projects would be much easier (currently I'm interested in doing it) without need of adding some sources manually outside of the library.
Another question is why would you like to place rendezvous server in the src/app, which contains data model specific files? I'm not claiming it is wrong, but it's quite confusing for me, as I can't find relation between those two. Is it some kind of intermediate step before finding out final shape?

this is great! but would you put these in src/app instead of src/accessory?

I mentioned why I did this in #2927, but we can also discuss it here. The thing is that to compile chip-app-server files, we need to have compiled src/transport and src/platform modules. Src/platform needs in indirect way through the src/lib to have src/app compiled before it can be compiled, so at the moment it's kind of impossible to compile src/platform before src/app. And that means that we can't compile files from chip-app-server inside of the src/app module. That's why I proposed creating new src/accessory module. Maybe it's not the only solution, but src modules already have some dependencies and chip-app-server was previously compiled after CHIP library, so now it's not so easy to find out how to integrate it in some of the existing modules to fulfill all dependencies.

You could potentially put it at src/app/server or src/app:server while we think about how to untangle this.

@mspang that was first thing I tried, but I realized src/app/server files can't be compiled in the app static library, because of the reason described above. I have two ideas how to implement your suggestion:

do it similar to the src/app/util, so place files in the src/app/server and add it to sources in the external BUILD.gn (e.g. in lighting-common etc), but actually those server files still won't be integrated in the CHIP library, but only located in src/app.
place files in the src/app/server and create separate library from app, like app-server, having all dependencies set to the way to allow building it after src/platform and src/transport. That would allow integrating server to the CHIP lib.

I think that second option would be better, as using compiled CHIP library in the external projects would be much easier (currently I'm interested in doing it) without need of adding some sources manually outside of the library.
Another question is why would you like to place rendezvous server in the src/app, which contains data model specific files? I'm not claiming it is wrong, but it's quite confusing for me, as I can't find relation between those two. Is it some kind of intermediate step before finding out final shape?

To be honest neither app nor accessory make very much sense to me.

this is great! but would you put these in src/app instead of src/accessory?

I mentioned why I did this in #2927, but we can also discuss it here. The thing is that to compile chip-app-server files, we need to have compiled src/transport and src/platform modules. Src/platform needs in indirect way through the src/lib to have src/app compiled before it can be compiled, so at the moment it's kind of impossible to compile src/platform before src/app. And that means that we can't compile files from chip-app-server inside of the src/app module. That's why I proposed creating new src/accessory module. Maybe it's not the only solution, but src modules already have some dependencies and chip-app-server was previously compiled after CHIP library, so now it's not so easy to find out how to integrate it in some of the existing modules to fulfill all dependencies.

You could potentially put it at src/app/server or src/app:server while we think about how to untangle this.

@mspang that was first thing I tried, but I realized src/app/server files can't be compiled in the app static library, because of the reason described above. I have two ideas how to implement your suggestion:

do it similar to the src/app/util, so place files in the src/app/server and add it to sources in the external BUILD.gn (e.g. in lighting-common etc), but actually those server files still won't be integrated in the CHIP library, but only located in src/app.
place files in the src/app/server and create separate library from app, like app-server, having all dependencies set to the way to allow building it after src/platform and src/transport. That would allow integrating server to the CHIP lib.

I think that second option would be better, as using compiled CHIP library in the external projects would be much easier (currently I'm interested in doing it) without need of adding some sources manually outside of the library.
Another question is why would you like to place rendezvous server in the src/app, which contains data model specific files? I'm not claiming it is wrong, but it's quite confusing for me, as I can't find relation between those two. Is it some kind of intermediate step before finding out final shape?

Not sure. App server would seem to belong with "app", but I'd agree the naming of the app directory seems a bit off. Maybe that should be called data_model.
I think CHIP speak for "accessory" is just "device".

this is great! but would you put these in src/app instead of src/accessory?

I mentioned why I did this in #2927, but we can also discuss it here. The thing is that to compile chip-app-server files, we need to have compiled src/transport and src/platform modules. Src/platform needs in indirect way through the src/lib to have src/app compiled before it can be compiled, so at the moment it's kind of impossible to compile src/platform before src/app. And that means that we can't compile files from chip-app-server inside of the src/app module. That's why I proposed creating new src/accessory module. Maybe it's not the only solution, but src modules already have some dependencies and chip-app-server was previously compiled after CHIP library, so now it's not so easy to find out how to integrate it in some of the existing modules to fulfill all dependencies.

You could potentially put it at src/app/server or src/app:server while we think about how to untangle this.

@mspang that was first thing I tried, but I realized src/app/server files can't be compiled in the app static library, because of the reason described above. I have two ideas how to implement your suggestion:

do it similar to the src/app/util, so place files in the src/app/server and add it to sources in the external BUILD.gn (e.g. in lighting-common etc), but actually those server files still won't be integrated in the CHIP library, but only located in src/app.
place files in the src/app/server and create separate library from app, like app-server, having all dependencies set to the way to allow building it after src/platform and src/transport. That would allow integrating server to the CHIP lib.

I think that second option would be better, as using compiled CHIP library in the external projects would be much easier (currently I'm interested in doing it) without need of adding some sources manually outside of the library.
Another question is why would you like to place rendezvous server in the src/app, which contains data model specific files? I'm not claiming it is wrong, but it's quite confusing for me, as I can't find relation between those two. Is it some kind of intermediate step before finding out final shape?

To be honest neither app nor accessory make very much sense to me.

this is great! but would you put these in src/app instead of src/accessory?

I mentioned why I did this in #2927, but we can also discuss it here. The thing is that to compile chip-app-server files, we need to have compiled src/transport and src/platform modules. Src/platform needs in indirect way through the src/lib to have src/app compiled before it can be compiled, so at the moment it's kind of impossible to compile src/platform before src/app. And that means that we can't compile files from chip-app-server inside of the src/app module. That's why I proposed creating new src/accessory module. Maybe it's not the only solution, but src modules already have some dependencies and chip-app-server was previously compiled after CHIP library, so now it's not so easy to find out how to integrate it in some of the existing modules to fulfill all dependencies.

You could potentially put it at src/app/server or src/app:server while we think about how to untangle this.

@mspang that was first thing I tried, but I realized src/app/server files can't be compiled in the app static library, because of the reason described above. I have two ideas how to implement your suggestion:

Sure, I meant put it in its own library, but in the app/server/BUILD.gn or app/BUILD.gn build files. There is no restriction that there be only one library per build file. It's more tidy to do that, but not required.

do it similar to the src/app/util, so place files in the src/app/server and add it to sources in the external BUILD.gn (e.g. in lighting-common etc), but actually those server files still won't be integrated in the CHIP library, but only located in src/app.
place files in the src/app/server and create separate library from app, like app-server, having all dependencies set to the way to allow building it after src/platform and src/transport. That would allow integrating server to the CHIP lib.

I think that second option would be better, as using compiled CHIP library in the external projects would be much easier (currently I'm interested in doing it) without need of adding some sources manually outside of the library.
Another question is why would you like to place rendezvous server in the src/app, which contains data model specific files? I'm not claiming it is wrong, but it's quite confusing for me, as I can't find relation between those two. Is it some kind of intermediate step before finding out final shape?

Not sure. App server would seem to belong with "app", but I'd agree the naming of the app directory seems a bit off. Maybe that should be called data_model.
I think CHIP speak for "accessory" is just "device".

this is great! but would you put these in src/app instead of src/accessory?

I mentioned why I did this in #2927, but we can also discuss it here. The thing is that to compile chip-app-server files, we need to have compiled src/transport and src/platform modules. Src/platform needs in indirect way through the src/lib to have src/app compiled before it can be compiled, so at the moment it's kind of impossible to compile src/platform before src/app. And that means that we can't compile files from chip-app-server inside of the src/app module. That's why I proposed creating new src/accessory module. Maybe it's not the only solution, but src modules already have some dependencies and chip-app-server was previously compiled after CHIP library, so now it's not so easy to find out how to integrate it in some of the existing modules to fulfill all dependencies.

You could potentially put it at src/app/server or src/app:server while we think about how to untangle this.

@mspang that was first thing I tried, but I realized src/app/server files can't be compiled in the app static library, because of the reason described above. I have two ideas how to implement your suggestion:

do it similar to the src/app/util, so place files in the src/app/server and add it to sources in the external BUILD.gn (e.g. in lighting-common etc), but actually those server files still won't be integrated in the CHIP library, but only located in src/app.
place files in the src/app/server and create separate library from app, like app-server, having all dependencies set to the way to allow building it after src/platform and src/transport. That would allow integrating server to the CHIP lib.

I think that second option would be better, as using compiled CHIP library in the external projects would be much easier (currently I'm interested in doing it) without need of adding some sources manually outside of the library.
Another question is why would you like to place rendezvous server in the src/app, which contains data model specific files? I'm not claiming it is wrong, but it's quite confusing for me, as I can't find relation between those two. Is it some kind of intermediate step before finding out final shape?

@mspang @rwalker-apple thanks for comments and remarks. I aligned structure according to your suggestions.

@rwalker-apple could you re-review it to verify if requested changes were done? If there is nothing else to fix I would be glad to push this PR forward.

@saurabhst @jelderton @BroderickCarlin @chrisdecenzo @hawk248 could you please take a look?


changefeed lag keep at 13.3s after restarting all pd、tikv、tidb

What did you do?
1、run tpcc with 1000 warehouse and 10 thread
go-tpc tpcc run -D tpcc --host tc-tidb.endless-ha-test-ticdc-tps-1815061-1-34 -P4000 --warehouses 1000 -T 10 --ignore-error '2013,1213,1105,1205,8022,8028,9004,9007,1062' --user root --password '' --interval '10s'
2、restart all pd、tikv and tidb
What did you expect to see?
after restarting all pd、tikv、tidb，changefeed lag can recover to be less than 3s
What did you see instead?
changefeed lag keep at 13.3s after restarting all pd、tikv、tidb

Versions of the cluster
version：v6.1.7
githash：33e9f9517e81396d5e1a94eefa777e9999efd587
current status of DM cluster (execute query-status <task-name> in dmctl)
(paste current status of DM cluster here)

/remove-area dm
/area ticdc

/severity critical

Running the scenario multiple times does not reproduce the problem


DBeaver creates a ".eclipse/configuration" directory in your profile for each version

Description
DBeaver creates a .eclipse//configuration directory in your home directory (e.g. c:\users\chris). It contains quite a lot of files, a new one is created each time you upgrade DBeaver, and they are never deleted.

Some of them seem pretty egregious, for example there is a PNG splash screen. Why is that part of my configuration anyway?

In any case, this causes significant profile bloat over time. shouldn't DBeaver either use the same directory (maybe backing up the old one after an upgrade, keeping only one such backup) or delete old version-stamped directories after a while?
I had 90 MB in old configuration directories (about 15 of them) after using DBeaver for a few years.
DBeaver Version
Community Version <IP_ADDRESS>308061552
Operating System
Windows 10
Database and driver
n/a
Steps to reproduce

Install and run DBeaver
Upgrade and run new version
Upgrade again and run new version
Look in %USERPROFILE%.eclipse

Additional context
No response

@qris thank you for the report.
It is a known issue, this happens when you install DBeaver in a write-protected folder (e.g. Program Files). DBeaver needs to extract several artifacts (including native binaries) native binaries during first start. When it is installed in a local user folder then all artifacts are extracted in the same folder where DBeaver was installed.
Otherwise they are extracted in user home .eclipse folder (because it is a part of Eclipse platform behavior).
Generally it is better to use different temp files location (e.g. %APPDATA%\DBeaverData). But we will need to patch core Eclipse libraries for that.
Workaround: install DBeaver for current user only (the default option).

Thanks, could the installer not extract these into a read-only system-wide directory, shared between all users?

Looks like a duplicate of #2118


linux/android add POSIX_SPAWN_USEVFORK (mostly no-op now) extension.

closes #2269

r? @Amanieu
(rust-highfive has picked a reviewer for you, use r? to override)

@bors r+

:pushpin: Commit 284472a0030336086d57f21525fd91c3484f02a6 has been approved by Amanieu

:hourglass: Testing commit 284472a0030336086d57f21525fd91c3484f02a6 with merge 9f0a87b9351eaf5b38d9ff108d308c71d70ef419...

:broken_heart: Test failed - checks-actions

@bors r+

:pushpin: Commit 8d38ada76dce067a59cbe52e6dfbef34c2bc5e4d has been approved by Amanieu

:hourglass: Testing commit 8d38ada76dce067a59cbe52e6dfbef34c2bc5e4d with merge f327cdf4bf767fca70441cd90781d95de14fdc98...

:broken_heart: Test failed - checks-actions

@bors r+

:pushpin: Commit ff29f2e2b7a6e0b4961534cdaa945a4e636335c8 has been approved by Amanieu

:hourglass: Testing commit ff29f2e2b7a6e0b4961534cdaa945a4e636335c8 with merge 0da7bba21582f0852875ce45ac02750ee87e7ab3...

:broken_heart: Test failed - checks-actions

@bors r+

:pushpin: Commit 7877856a2cd5ed45179f9db7148df105dfe19e38 has been approved by Amanieu

:hourglass: Testing commit 7877856a2cd5ed45179f9db7148df105dfe19e38 with merge bcfd392c6baf6e55cdcd65bd6d2e15d8b853dd94...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: Amanieu
Pushing bcfd392c6baf6e55cdcd65bd6d2e15d8b853dd94 to master...


Packaging Improvements

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


Add calculation reference inputs size function

Changelog
- description: |
    <insert-changelog-description-here>
# uncomment types applicable to the change:
  type:
  # - feature        # introduces a new feature
  # - breaking       # the API has changed in a breaking way
  # - compatible     # the API has changed but is non-breaking
  # - optimisation   # measurable performance improvements
  # - improvement    # QoL changes e.g. refactoring
  # - bugfix         # fixes a defect
  # - test           # fixes/modifies tests
  # - maintenance    # not directly related to the code
  # - release        # related to a new release preparation
  # - documentation  # change in code docs, haddocks...

Context
Additional context for the PR goes here. If the PR fixes a particular issue please provide a link to the issue.
How to trust this PR
Highlight important bits of the PR that will make the review faster. If there are commands the reviewer can run to observe the new behavior, describe them.
Checklist

[ ] Commit sequence broadly makes sense and commits have useful messages
[ ] New tests are added if needed and existing tests are updated. See Running tests for more details
[ ] Self-reviewed the diff

Included in https://github.com/IntersectMBO/cardano-api/pull/496


Bad image (and other) links in docs

In page https://github.com/optapy/optapy/blob/main/optapy-docs/src/modules/ROOT/pages/score-calculation/score-calculation.adoc#49-fairness-score-constraints (and probably others as well), all image links lead to a 404 page.
Some other links (e.g., https://github.com/optapy/optapy/blob/main/optapy-docs/src/modules/ROOT/pages/score-calculation/constraint-streams/constraint-streams.adoc#constraintStreams in Sec. 3.1) also lead to 404.

Thanks, I noticed this myself just now.  A quick solution could be to put a readme in the github page for the docs, with a link to optapy.org, for those people (like me) who got to the docs through github.


dart2js and VM don't agree about factory as a static method

Consider the code:
class Test {
  static factory() {
    return 50;
  }
}

main() {
  print(Test.factory());
}

Run this code on the Dart VM:
$ dart test.dart
'file:///home/ubuntu/test.dart': error: line 2 pos 17: factory method cannot be explicitly marked static
  static factory() {
                ^

Run this code in NodeJS with dart2js:
$ dart2js -o test.js test.dart
$ node test.js
50

In Dart, factory is a "built-in identifier". Those are not allowed in all places:

. It is a compile-time error if a built-in identifier is used as the declared name of a prefix, class, type parameter or type alias. It is a compile-time error to use a built-in identifier other than dynamic as a type annotation or type parameter.

Here factory is the name of a (static) method, which is not prohibited by the specification. It works in dart2js and is accepted by the analyzer, so this is a VM parser bug.

Change the method factory() into a dynamic method and dart2js fails as well:
class Test {
  factory() {
    return 50;
  }
}

main() {
  print(new Test().factory());
}

[d2 sdk]$ dart ./pkg/compiler/lib/src/dart2js.dart  ~/tmp/s.dart
/Users/hausner/tmp/s.dart:3:10:
Error: Expected identifier, but got '('.
  factory() {
         ^
/Users/hausner/tmp/s.dart:9:9:
Warning: No method named 'factory' in class 'Test'.
  print(new Test().factory());
        ^^^^^^^^^^^^^^^^^^^^
Error: Compilation failed.

Another example of how the language is unnecessarily baroque.

Saved the universe with 4ca841772589c193138a18ad807adc66e43cae21


Support for RX-V677?

I have an RX-V677 and know it's not on the support list, but everything around it is.  Hoping it "just works" I made an attempt and am getting this error when adding the integration:
Error
Config flow could not be loaded: 500 Internal Server Error Server got itself in trouble

I was able to power the receiver off and on with the entities found from the basic integration so at least I know HA can talk to the receiver.
What logs/etc. would you like to see?  Thanks

Hmm, that is kind of a generic error, so probably look in the Home Assistant logs.
You can either go to Settings > System > Logs and click on the error there to get more info and paste here. Or just attach the homeassistant.log file here.
Just to be sure on what version of  Home Assistant and the integration ar you using?

I just did a new docker pull last night:
Home Assistant 2023.1.7
Frontend 20230110.0 - latest

logs showing no permissions for /.local.  This would be in the container with processes running under a specific UID.  Guess that was bound to catch up to me sooner or later.
2023-02-01 17:57:47.315 ERROR (SyncWorker_9) [homeassistant.util.package] Unable to install package ynca==5.2.0: ERROR: Could not install packages due to an OSError: [Errno 13] Permission denied: '/.local'

I created the dir and made sure I had permissions, which looks like there was a pip install to that dir.  Now getting this error in UI:
Config flow could not be loaded: {"message":"Invalid handler specified"}

and in stderr:
2023-02-01 18:01:41.129 ERROR (MainThread) [homeassistant.config_entries] Error occurred loading configuration flow for integration yamaha_ynca: No module named 'ynca'

ynca is the only thing going in, is this the right location?
/.local
└── lib
    └── python3.10
        └── site-packages
            ├── ynca
            │   ├── __pycache__
            │   └── subunits
            │       └── __pycache__
            └── ynca-5.2.0.dist-info

It seems that your Home Assistant has some issues downloading/installing python libraries. That is kind of "core" for HA and is something that should just work.
It is unclear to me why HA is trying to install in .local (and I am not sure if that is in or outside the container).
This seems to be more of an Home Assistant issue than specific to this integration as the integration is not even being installed. I have never seen anything like this and I don't really know Docker, so maybe ask/search around on the forums/discord to see if other people had similar experiences.

ok that's fine. I'm ok with closing the issue

As an FYI. If you can solve the package installation issue your receiver should work according to the user in #129


Update index.html.md.erb

Doh! Totally missed removing this final temporary line. Corrected

Thanks!


⚠ ClamAV has degraded performance

In 06426be, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 51 ms

Resolved: ClamAV performance has improved in f145f38.


echart yAxis and xAxis label cutout

hi, so i am using echart in my bootstrap project and i use this styling to control the size of the chart
<style scoped>
	.echarts{
		width: 100vw;
		min-width: 50vw;
		height: 50vh;
	}
</style>

and i find there is some anoying problem that happen if only i have long yAxis or xAxis label like this

so how to fixed it? to make sure there is no cutout in yAxis and xAxis label?

Try http://echarts.baidu.com/option.html#grid.right please ~

well i tried it but it just have a weirds issue, i make grid left and right 10% and it just feels not right comparing with chartjs it have a responsive parameters that can automatically create grid size according to the container.

Any update?

Am alternate option is to adjust with grid margins and set a fixed width for labels while they can be represented with ellipsis for longer ones.

Am alternate option is to adjust with grid margins and set a fixed width for labels while they can be represented with ellipsis for longer ones.

https://stackoverflow.com/questions/38693662/label-cant-display-when-echarts-labels-length-is-too-long/60787785#60787785
i have try this links method, and it helps me.
the solution in the link is set property below
grid: { containLabel: true }
hopes this also helps u


Tellurium frontend becomes unresponsive when saving files with large variables

When doing complex calculations, the frontend of Tellurium often does not respond anymore and can also not be closed (using macOS). That usually happens when I do a simulation that produces variables with larger tables. Then Tellurium becomes unreactive, i.e., scrolling does no longer work, it does not save the file, and I cannot close the program anymore. In this situation, I can only kill the application entirely, hence also resetting my calculation. Is there a way to circumvent this?

Might be worth checking if this happens in the IDE version to see if it's a
tellurium problem or a notebook problem.
Herbert
On Wed, Nov 15, 2017 at 4:44 AM Andreas Dräger<EMAIL_ADDRESS>wrote:

When doing complex calculations, the frontend of Tellurium often does not
respond anymore and can also not be closed (using macOS). That usually
happens when I do a simulation that produces variables with larger tables.
Then Tellurium becomes unreactive, i.e., scrolling does no longer work, it
does not save the file, and I cannot close the program anymore. In this
situation, I can only kill the application entirely, hence also resetting
my calculation. Is there a way to circumvent this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sys-bio/tellurium/issues/305, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABAZDs2zoLyV-_bQIRI8WTT753GsEJPIks5s2tybgaJpZM4Qe3qP
.

Hi Andreas, there are several things which could cause this. If the output contains very large tables or plots with many traces then it may take a long time to save. You can clear the outputs by choosing Language -> Reset and Clear All Cells, but then the notebook will be saved without outputs. If you could send me an example notebook via email (with the outputs cleared), I could try to provide more specific advise.

If the model or output is really really large, it could cause a problem like this. From my experiences, if you wait a bit until the function executes, the program will become responsive again. This could be an issue with how Python deals with frontends in general.
However, this could also be a bug so we should probably check it.

Thanks! So, my previous comment was in the wrong thread https://github.com/sys-bio/tellurium/issues/304#issuecomment-344741801. When changing the plot library, I can even save very large outputs and Tellurium is not unresponsive. However, it might then take a bit before the plot is actually drawn, but that's fine. Maybe it would make sense to notify the authors of the default plot library about the problem with the unresponsiveness?

That's certainly a fair point. I have been meaning to do some testing to confirm whether the problem is with Plotly, or if this is just a consequence of trying to save large amounts of JSON data (in which case they wouldn't be able to do anything about it).


Fix libdir detection

This patch fix installation issue at least on Debian and Ubuntu.
In lua5.1.pc delivered in debian package, variable to be used to install lua
binary module is INSTALL_CMOD:
The following are intended to be used via "pkg-config --variable".
Install paths for Lua modules.  For example, if a package wants to install
Lua source modules to the /usr/local tree, call pkg-config with
"--define-variable=prefix=/usr/local" and "--variable=INSTALL_LMOD".
INSTALL_LMOD=${prefix}/share/lua/${major_version}
INSTALL_CMOD=${prefix}/lib/${deb_host_multiarch}/lua/${major_version}

It helped if I run
./configure LUA_CFLAGS=-I/usr/include/lua5.1 LUA_LIBS=-llua5.1


Remove duplicate translations from French file

These translations are already included in rails-i18n gem, which is necessary anyway when using another language other than English. This PR removes these translations from the French file. Note that they are not present in any other language in this gem (correctly)

@solidusio/core-team can we have another review here, please?


HTML

HTML
User Story
As a boot camp student,
I want a Prework Study Guide website,
so that I can review all the information I learned throughout Prework.
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I visit the website in my browser
THEN I see four boxes titled HTML, CSS, Git, and JavaScript with associated notes listed
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow
WHEN I open the console in my browser
THEN I can see the four topics I learned listed along with a suggestion on what I should study first
WHEN I visit the website using the URL
THEN I can access my website from any browser

I have finished this part of the pre-work study guide.


NEXT-RECORD feature can crash

when using the %VAR_NEXT-RECORD% feature, if the page consumes more records than the file has, it can crash:

02:06:18 - ScribusGenerator - ERROR:
error: Traceback (most recent call last):
File "/Applications/Scribus.app/Contents/share/scribus/scripts/ScribusGenerator-master/ScribusGeneratorCLI.py", line 148, in
generator.run()
File "/Applications/Scribus.app/Contents/share/scribus/scripts/ScribusGenerator-master/ScribusGeneratorBackend.py", line 186, in run
shiftedElts = self.shiftPagesAndObjects(tmpElt, pagescount, pageheight, vgap, index-1, recordsInDocument, groupscount, objscount, version)
UnboundLocalError: local variable 'pagescount' referenced before assignment

indeed. bug is now fixed, kindly update your SG from last commit. thanks for the bug report.


Update Kubernetes manifests

I run Plausible on AKS. I use the Kubernetes Terraform module for deployment, but my config is pretty much analog to the Kubernetes manifests in this repo. Here are the changes I've made to my config:
https://github.com/schnerring/infrastructure-core/compare/ccc8b1d2378979e2ae3e7ad7a42dd107716f70ba..32ca23afbf2b708215cb51075ff8e44c525cec9b
I thought it might be useful to consolidate the changes with you guys while I'm on it. This syncs up the manifests with the recent docker-compose updates you've made.
This should be good to go, but I haven't tested this myself. Maybe @oscartbeaumont can? Keep in mind that switchting to the Alpine image requires a chmod -R 70:70 on the PGDATA dir. This can be done by adding an init container like this:
initContainers:
- name: chown-pgdata
  image: alpine:3
  securityContext:
    runAsUser: 0
  command:
  - chown
  - -R
  - 70:70
  - /var/lib/postgresql/data/pgdata
  volumeMounts:
  - name: data
    mountPath: /var/lib/postgresql/data

As a side note, I think we could remove the following from clickhouse-user-config.xml:
<log_query_threads>0</log_query_threads>

It seems to be the default since ClickHouse v22.5:

Disable log_query_threads setting by default. It controls the logging of statistics about every thread participating in query execution. After supporting asynchronous reads, the total number of distinct thread ids became too large, and logging into the query_thread_log has become too heavy.

Also, if you have any more breaking changes on the roadmap, it's probably a good idea to use plausible:v1.5.1 instead of plausible:latest.

I agree with this PR, using latest tag can cause unexpected issues in the future.

@schnerring 👋
Thank you for the PR. I wonder if you'd like to take over k8s setup management? It'd be in your own repo to which we'd link from this repo's README. The reasoning is that I don't have k8s experience and as such can't really do proper reviews.

Sorry, but I'm closing this PR as the k8s setup has been removed: https://github.com/plausible/hosting/pull/71

@ruslandoga Sorry, I must have dismissed the notification of this issue! I'll put this in a separate repo and let you know when I'm done, but it might be a while. I'll send another PR that adds the repo link to the README, ok?


Removes iotedge Debian 9 instructions

The Debian 9 package has an issue with a dependency. As it is not technically a supported platform, I think we should remove the instructions until we can address the package issue.
Closes #10968

@myagley : Thanks for your contribution! The author, @kgremban, has been notified to review your proposed change.

#sign-off

@veyalla: I'm sorry - only the author of this article, @kgremban, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Warning requiring CardStackStyleInterpolator

Hi, in order to have StackNavigator with the horizontal animation (on Android), I followed the instructions on this issue https://github.com/react-community/react-navigation/issues/1759.
The problem is that by importing CardStackStyleInterpolator i got this warning
Requiring module 'react-navigation/src/views/CardStackStyleInterpolator.js' by name is only supported for debugging purposes and will BREAK IN PRODUCTION!
Is there another way to require CardStackStyleInterpolator?
Thanks

Sorry was a re-natal clojurescript problem


Permanent values for miracles

Is your feature request related to a problem? Please describe.
The miracle activation cannot be used with arguments as the positioning of the available miracles change.
Describe the solution you'd like
Only show found miracles as right now but in a permanently fixed order for automation to work.
Describe alternatives you've considered
Fixed list of all miracles, if a selected miracle is not found it should notify and ask to select another miracle or go back to the menu.

Added in this PR.


ONNX model connector

Are we considering creating a ONNX connector for MONAI Bundle? This potentially boost MONAI and MONAI Label usability.

Hi @diazandr3s ,
Thanks for raising it.
Actually, ONNX and TorchScript are optional models in a bundle, you can check:
https://docs.monai.io/en/latest/mb_specification.html#directory-structure
Thanks.

Superb! Thanks, @Nic-Ma I missed this part
Is there any tutorial that we can use as a reference? A MONAI Label user wants to export an ONNX model that also includes functions that are not in MONAI
Maybe this should be part of a discussion rather than an issue? :)

Hi @diazandr3s ,
@binliunls is developing some TorchScript -> ONNX script, will share it soon.
And @ericspod may also have some examples.
Thanks.

I had only briefly experimented with ONNX in the past when discussing Torchscript behaviour: https://gist.github.com/ericspod/65809935adf783c17bdcf19c5568f0fe

Thanks, @ericspod


Cause specific example

Add example of cause specific model

I think it will be best to remove the cascade steps and stick with the mortality pattern.


Niederlassungserlaubnis was broken because of change in the KVR website

Updated the description from NE for Blaue Karte

Hmm, I was looking there and I had a thought that it may be broken, but I think it worked for me... but let's see. Thanks!

Hum...I am not sure, I have done some tests and it is "loading" without providing results again. 😞


[Feature request] whitelist hostnames

Thank you for module.
Is there any way to whitelist only particular domains to allow proxy forwarding?
Any ideas on workarounds? Unfortunately I can't do this in Firewall (political reasons).

I need this feature as well. One possible solution I found is to use an if statement like this to check the host in the CONNECT request:
    if ($connect_host !~ ^www.google.com$) {
      return 403;
    }

This has to be placed outside the location / context.
When you try to connect to a host other than www.google.com the request is rejected:
> curl -sS -x localhost https://www.yahoo.com > /dev/null
curl: (56) Received HTTP code 403 from proxy after CONNECT
> curl -sS -x localhost https://www.google.com > /dev/null

map $connect_host $is_host_deny{
default 1;
"host1.com" 0;
"host1.com" 0;
"host1.com" 0;
}
if ($is_host_deny) {
return 403;
}

Thank you for your comments and solution. This solves my issue.

map $connect_host $is_host_deny{ default 1; "host1.com" 0; "host1.com" 0; "host1.com" 0; }
if ($is_host_deny) { return 403; }

Pay attention, It also restrict visit http:// website


css attribute line-height support?

I have following code to set text to HtmlTextView:
private String getHtmlText(Comment comment) {
        String nameString = getHtmlNameString(comment);
        String contentString = getContentString(comment);
        return "
" + nameString + ":" + contentString + "
";
    }

    private String getContentString(Comment comment) {
        return String.format("" + comment.getContent() + "",
                CONTENT_CLICK);

    }

    private String getHtmlNameString(Comment comment) {
        String userStr = String.format("" + comment.getUser().getUsername() + "",
                USERNAME_CLICK_0);

        if (comment.getToCommentUser() != null) {
            userStr += ("回复" + String.format("" + comment.getToCommentUser().getUsername() + "",
                    USERNAME_CLICK_1));
        }
        return userStr;
    }

However, I don't see line-height:200%; in getHtmlText() is working.
I've tried format like 2em, 40px, 200%, 1.5, but none of this works.

As written in the README: This is not supported.

How to  change the spacing between lines of one paragraph ?
They are very tight together.


Members Scrambled

I noticed when download both control and perturbed members with this, the members seem to be get scrambled. For example
client.retrieve(         date=20230127,         time=6,         step=72,         stream="enfo",         type=["cf","pf"],         param='tp',         target="tmp.grib2",     ) fieldset = ecdata.read("tmp.grib2")
and loop through forecast times, when I plot total precipitation, something like box and whiskers look good, but the data for each member over time does not increase or stay the same (in many cases, decreases). This is very noticeable with total precipitation, but would be much harder to know for something like temperature or height. This happens for every cycle I try. Not sure that it occurs if I give the entire list of forecast hours in one request, but that would result in a large file, and pretty crazy memory usage if done that way. When I spot check members with the multiple single forecast hour request, it shows the same order ([0,1,10,11,12,13,14,15,16,17,18,19,2,20,21,22,23,24,25,26,27,28,29,3,30,31,32,33,34,35,36,37,38,39,4,40,41,42,43,44,45,46,47,48,49,5,50,6,7,8,9]), so I am puzzled as to what leads to this.

There is no guarantee about the data ordering within the grib file.

But that assumption isn't made if one reads the "number" array in though, right? In any case, using xarray directly with cfgrib to open files downloaded by the opendata api works as expected - I only experienced this opening as a fieldset with metview in python.


RFC: Components to remove in version 2

As originally proposed, components that are added to GOV.UK Frontend should be removed from this project in a breaking release.
Summary card
An upcoming version of GOV.UK Frontend will provide a card variant for the existing summary list component. This means the ‘Summary Card’ component is no longer required, or recommended. It should be removed in version 2.
Task list
While the task list component has not been added to GOV.UK Frontend, the version being worked on for inclusion is now available as a plugin component.
Should the component in this project, which follows the older but familiar and existing design be retained until an officially agreed upon component is added to GOV.UK Frontend. Or should authors be encouraged to use the new version being worked on instead?
Deprecation
In the release for v1.1, we can include a deprecation notice for the affected component(s). Is a message in the release notes is enough?
Should a v1.x branch be maintained? Am thinking not (not sure what would change, and we should encourage authors to use official components from GOV.UK Frontend)

Agree with removing task list and summary card. The expectation is that the task list gets merged into govuk-frontend this quarter (subject to an accessibility audit that GDS are running).


Scan Canonicalization: Support for Scans in C++ Backend

Description
Enables support for scans in the C++ backends.
Still missing: standardized method of domain extension, passing/creating the domain in C++.
Requirements
Before submitting this PR, please make sure:

[x] You have run the code checks, tests and documentation build successfully
[x] All fixes and all new functionality are tested and documentation is up to date
[x] You looked at the review checklist

Additionally, if this PR contains code authored by new contributors:

[x] All the authors are covered by a valid contributor assignment agreement,
signed by the employer if needed, provided to ETH Zurich
[x] The names of all the new contributors have been added to an updated
version of the AUTHORS.rst file included in the PR

A reminder for me to check the result of using this branch in ICON: https://github.com/C2SM/icon-exclaim/pull/35

@havogt I implemented a much more robust extent analysis in https://github.com/fthaler/gt4py/pull/2
The changes are a bit more heavy than probably expected as it required a new shift/offset analysis. We can either first merge this one here and I will open a separate PR, or you can review the other one and merge into this one. I just kept it separate for now to simplify the review process.


Save Layer as a new Dataset

[ ] I added an entry to CHANGELOG.md if knowledge of this change could be valuable to users.

Here is a brief summary of what I did:

Moved some functionality of saving a workflow to a dataset to datasets module
Changed the result descriptor of datasets originating from workflows to match their query rectangle
Added a layer_to_dataset function to layers handler for saving a layer via the workflow functionality

 [RESPECT] 3.4 Overviews für Rasterdaten erzeugen

bors merge

bors retry

bors retry

bors retry

bors retry

bors merge

bors retry

bors retry

bors retry

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

522 of 567 (92.06%)  changed or added relevant lines in 3 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.02%) to 87.81%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

services/src/datasets/create_from_workflow.rs
154
167
92.22%

services/src/handlers/layers.rs
358
390
91.79%

Files with Coverage Reduction
New Missed Lines
%

services/src/pro/handlers/datasets.rs
1
92.35%

services/src/pro/handlers/drone_mapping.rs
4
94.66%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
85266

Relevant Lines:
97103

💛  - Coveralls

bors merge


HttpClient must use the http scheme for non SSL connections

When upgrading to Vert.x 3.5.3.CR1 and enabling HTTP_2 in HTTP client. The request scheme in the HTTP server is HTTPs even when the request from the HTTP client to HTTP server is HTTP.
Reproducer https://github.com/michaelpog/http2reproducer

Run HTTP server
Run HTTP client.
Observe HTTP server output:

Starting HTTP server on port 8881
REQUEST: hi from client - 1
Request Scheme : https
REQUEST: hi from client - 2
Request Scheme : https

Change line :

 HttpClientOptions options = new HttpClientOptions().setProtocolVersion(HttpVersion.HTTP_2);

in HTTP client  to
  HttpClientOptions options = new HttpClientOptions().setProtocolVersion(HttpVersion.HTTP_1_1);

Rerun HTTP client
Observe correct HTTP request scheme

REQUEST: hi from client - 1
Request Scheme : http
REQUEST: hi from client - 2
Request Scheme : http

it's actually a client problem and not a server problem

fixed by https://github.com/eclipse/vert.x/issues/2533

Awesome, thanks!


Question: Versioning inconsistent between NPM and Github

I'm getting confused here and maybe some one clear this up for me.
Why would is it that the version of laravel-mix on Github releases is 0.8.2 but on npm it's 0.8.8?

This question come about because I saw a new method mix.copyDirectory() in the docs but it can not be found in the master branch other than in this commit.

You're looking at the master docs, not the docs for 0.8.2 or 0.8.8. The copyDirectory method is here: https://github.com/JeffreyWay/laravel-mix/blob/master/src/Api.js#L265-L267
You can use that method once we tag 0.9.0 later this week.


Subscribe and Login buttons don't work

Hi there,
I've just tried the Dawn theme.
I have downloaded the .zip and upload it on my website, but it seems that the "Subscribe" and "logins" buttons don't show the popup.
I tried to upload the routes.yaml form the theme folder but nothing seems to work.
I also don't find where the .hbs files with the popup are in the theme folder.
Thanks

If this post: https://ghost.org/changelog/iveel/ is to be believed, the theme still needs a little work to be done. Specifically, it's missing the http://github.com/TryGhost/Portal reference that generates the popup. I would be adding the code yourself would take care of it, but can't say for sure.

Hey @MrTehes, the membership flow of the theme is powered by Portal - Ghost's new feature added in 3.37.0. If it's not working, it's most likely you haven't updated your Ghost instance yet. You don't need any custom code or routing.

Oh thanks ! My bad... I didn't see the changelog and the Portal feature. Everything works fine now.


checklist - description extension: content of pathway of introduction

Concerning the description extension, dwc term type: pathway
The content for pathway was now copied and cleaned from pathway_of_introduction in the raw Excel file. However, although the mapping on the CBD standard may seem quite straightforward, it is not, and there are some discrepancies with other registries that have mapped onto CBD (see #19).
Currently, the following terms are available for pathway

value

aquaculture

aquarium trade

corridors

other

shipping

@timadriaens : what approach should be taken concretely to map pathway correctly? Should every individual species be checked on potential pathways of introduction and verified against the available information here? Should all supplied references be checked?Other suggestions?

The way to do it would be to go to the original source publication, but I think I can make a good educated guess that can be checked afterwards by Pieter. So I'll prepare a mapping file, csv or other?

This  is a good source of information on how to interpret pathways for marine/brackish species.

@timadriaens yes, please make an educated guess. The mapping can be pasted here... it's fairly short anyway.

please find a .csv of my mapping with some remarks on how I interpreted them
AI_2016_Boets_etal_Supplement_MappingTim.zip

@LienReyserhove since the mapping of Tim is one for each species, I propose to just add that information as an extra column to the source Excel file (rather than managing this information separately) and use that column for mapping it in DwC.


Struggling to define schema overrides (on taps)

Hi again,
while working on https://github.com/MeltanoLabs/target-postgres/pull/251, I am struggling to define schema override rules. For development purposes, I am using tap-singer-jsonl.
Within meltano.yml, I've defined a schema override rule on the tap-singer-jsonl element.
schema:
  my_table:
    value:
      type: "foo"

After running meltano invoke like:
meltano --log-level=debug invoke tap-singer-jsonl

I can see in .meltano/run/tap-singer-jsonl/tap.properties.json:
{
  "streams": [
    {
      "tap_stream_id": "array_number",
      "key_properties": [
        "id"
      ],
      "schema": {
        "properties": {
          "id": {
            "type": "integer"
          },
          "value": {
            "type": "foo"
          }
        },
        "type": "object",
        "required": [
          "id"
        ]
      },
      "metadata": {},
      "selected": true
    }
  ]
}

NB: metadata has been compressed.
However, it looks like it does not have any influence on the output of the tap at all. The schema override rule is not being taken into consideration.
With kind regards,
Andreas.

I should have noted the I also needed to define the capabilities attribute in the tap-singer-jsonl section of meltano.yml.
capabilities:
  - catalog
  - discover

Otherwise, invoking meltano invoke would yield such a warning, and will not propagate the schema override rule into the tap.properties.json file at all.
[warning  ] A catalog file was found, but it will be ignored as the extractor does not advertise the `catalog` or `properties` capability

We converged this report into Learning Singer » Defining schema overrides on taps.


center and resize character green.GIF in play

possibly need to add the character sheet instead of the gif in order for the code to splice through and animate that way?

index.js inside the player constructor (crtl F "center")
gets the width and height of the ship
subtracts half of the width and height from the center point for a true center position


Keep the type of the services to be same.

Signed-off-by: Aisuko<EMAIL_ADDRESS>Description
As we discussed,

All services are the type of LoadBalancer.
Chart for CPX should be disable.
Chart for perf should be disable.

https://layer5io.slack.com/archives/CFGG6U10E/p1658193023302609?thread_ts=1658182545.355309&cid=CFGG6U10E
This PR fixes #
Notes for Reviewers
After we merge this PR, we need to upload it to the artifacthub.io
Signed commits

[x] Yes, I signed my commits.

You are welcome. I'm working on the rest part.

Oh, very good.


Added Empty Statement Check and enforced checkstyle

Description
No functional changes, Code formatting , indent fix and empty statement checktyle enforce :)
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[ ] Commit message starts with the issue number from https://issues.apache.org/jira/projects/FINERACT/. Ex: FINERACT-646 Pockets API.

[ ] Coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions have been followed.

[ ] API documentation at https://github.com/apache/fineract/blob/develop/api-docs/apiLive.htm has been updated with details of any API changes.

[ ] Integration tests have been created/updated for verifying the changes made.

[ ] All Integrations tests are passing with the new commits.

[ ] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the list.)

Our guidelines for code reviews is at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide

@thesmallstar this seems to include an (arbitrary?) number of "break lines up" changes - was that intentional? It would perhaps be clearer both for me to review and for you to do if in each PR you only include the changes strictly required to make the respective module (here EmptyStatement) pass? Just a thought.

@vorburger, agreed on your point,making changes.

Not sure why integration tests failed, probably due to other PR? If not, I will give a detailed look tomorrow.

Not sure why integration tests failed, probably due to other PR? If not, I will give a detailed look tomorrow.

@thesmallstar re. build failure, please subscribe to and watch (or help fix??) #764 and #761 for FINERACT-852...

@thesmallstar Following #764 and #761 this will pass the build (or not at least not fail because of integration tests) if you rebase this now.

@vorburger the build failed even after rebase, did I miss something? The build passes locally though.

@thesmallstar @awasum see https://issues.apache.org/jira/browse/FINERACT-886 for another test failure

@thesmallstar @awasum see https://issues.apache.org/jira/browse/FINERACT-898 for another test failure - and this one is very very weird...

@thesmallstar please rebase and fix merge conflict from #767.
The build error due to FINERACT-898 hopefully will disappear.

@thesmallstar this failed again due to an IntegrationTest, see FINERACT-857. Could I suggest that we please proceed as follows from now on: Any integration test failure you see in your (or anyone else's, for that matter...) PR, please from now on just search for the failing test in JIRA, comment on it like I do, and then raise a dedicated small PR to @Ignore the respective failing test, with a TODO for that JIRA, similar to e.g. my #761? Do that for you here for FINERACT-857, and also for me for FINERACT-855 for #779? Thanks!  /Cc @awasum FYI.
PS please ignore the failed Build - Summary, and only look at Travis (see FINERACT-907).

/rebase

@thesmallstar, any follow up on this?

hello @xurror , updating it.
-Thanks

if the build still fails, I will look into the "flaky" tests.

Does not look like our typical IT tests.

thank you @xurror!!
@vorburger you can give the final review :D
-thanks
Manthan

Sure. if this is good for @vorburger, I think this OK to merge.


when an error occurs i can't get the options

I suggest that when an error occurs can return options, in order to collect the wrong URI.
in line 380(lib/crawler.js)
modify:
options.callback(error);
to:
options.callback(error,options);

As I have stated in https://github.com/sylvinus/node-crawler/issues/142 you can retrieve the options from callback in case of error:
callback: function(error, result, $) {
  console.log(this.uri)
  console.log(this.mydata)
}


adds create

Docs: Create

Type of change
Doc/drawing

@afflom I am thinking that might may need to split up the diagrams. One showing the tasks that trestle-bot performs then another document to show possible workflows. Some ideas I am considering:

Default path (Assemble + Regenerate + Push Commit back to branch)
Regenerate only
Check Only
New imports/creation + trestlebot

Thoughts?

Would these share a container view and be differentiated in different component views?

@afflom Changing my feedback due to the updates given in https://github.com/RedHatProductSecurity/trestle-bot/pull/24. It would probably be more useful to provide snippets to show how each workflow could be done instead of working them into a diagram. The C4 model diagram code perhaps just be a component and code view of trestle-bot. Thoughts?


CX Privacy_Violation @ src/main/java/com/appsecco/dvja/controllers/Register.java [master]

Privacy_Violation issue exists @ src/main/java/com/appsecco/dvja/controllers/Register.java in branch master
Method getPasswordConfirmation at line 42 of src\main\java\com\appsecco\dvja\controllers\Register.java sends user information outside the application. This may constitute a Privacy Violation.
Severity: Medium
CWE:359
Checkmarx
Training
Recommended Fix
Lines: 42

Code (Line #42):
        return passwordConfirmation;

Issue still exists.


Image size issue

Hello,
I'm facing an issue setting captured image size in iOS & Android
I'm using capacitor 6 and plugin version 6.2.13
When i set the image size :
return CameraPreview.capture({
  quality: 66,
  width: 600,
  height: 1000
});

The captured image will never have the resolution i've chosen ! The only thing i can control is the quality
Any solution to this ?

That an issue I'm currently trying to solve.
The size should be a size available on the device, if not, we try to send the one who matches the ratio. If we don't have any, we return the biggest.
I think I will disable the width and height and let it only be chosen in the configuration, that will be less misleading

@riderx thank you
i've worked on something but i'm not sure if the right approach
(For iOS)
i've added a function called resizeImage that will resize the image before converting it the jpeg
func resizeImage(image: UIImage, targetSize: CGSize) -> UIImage {
    let size = image.size

    let widthRatio  = targetSize.width  / size.width
    let heightRatio = targetSize.height / size.height

    var newSize: CGSize
    if widthRatio > heightRatio {
        newSize = CGSize(width: size.width * heightRatio, height: size.height * heightRatio)
    } else {
        newSize = CGSize(width: size.width * widthRatio, height: size.height * widthRatio)
    }

    let rect = CGRect(origin: .zero, size: newSize)

    UIGraphicsBeginImageContextWithOptions(newSize, false, 1.0)
    image.draw(in: rect)
    let newImage = UIGraphicsGetImageFromCurrentImageContext()
    UIGraphicsEndImageContext()

    return newImage!
}

And i call the method once the image is captured
@objc func capture(_ call: CAPPluginCall) {
    DispatchQueue.main.async {

        let quality: Int = call.getInt("quality", 85)
        let width: Int = call.getInt("width", 650)
        let height: Int = call.getInt("height", 1000)

        self.cameraController.captureImage { (image, error) in

            guard let image = image else {
                print(error ?? "Image capture error")
                guard let error = error else {
                    call.reject("Image capture error")
                    return
                }
                call.reject(error.localizedDescription)
                return
            }

            let resizedImage = self.resizeImage(image: image, targetSize: CGSize(width: width, height: height))

            let imageData: Data?
            if self.cameraController.currentCameraPosition == .front {
                let flippedImage = resizedImage.withHorizontallyFlippedOrientation()
                imageData = flippedImage.jpegData(compressionQuality: CGFloat(quality/100))
            } else {
                imageData = resizedImage.jpegData(compressionQuality: CGFloat(quality/100))
            }

            if self.storeToFile == false {
                let imageBase64 = imageData?.base64EncodedString()
                call.resolve(["value": imageBase64!])
            } else {
                do {
                    let fileUrl=self.getTempFilePath()
                    try imageData?.write(to: fileUrl)
                    call.resolve(["value": fileUrl.absoluteString])
                } catch {
                    call.reject("error writing image to file")
                }
            }
        }
    }
}

This will break the aspect ratio of the image right?

@riderx "resizeImage" doesnt  break the aspect ratio of the image, it ensures that the aspect ratio is maintained by calculating the correct scaling factor based on the target size.

if the image is 19:9 and we want 4:3 where does the pixel go then ?
Or we break the pixel size and force the ratio or the image will have empty space?
Maybe can you share image output to explain if i miss something?

A common use case to want a preview at or around screen resolution and then to capture an image at a much higher resolution (but often the same aspect ratio).  I have a use case where I have a screen of 890x534px.  I want to take pictures at a 4:3 aspect ratio.  I can easily calculate the viewport for positioning a 4:3 preview. left: 89 and a 712x534 preview.  The plugin need not do that.
Now I need to capture at 1440x1080, but the max supported by my device is 8000x6000
Using the preview (video) size is not close to what I want, and I don't want to request larger video since this uses more resources and can be slow and jerky.  I also don't want max resolution since this is huge and slow to resize. So capture must be able to supply dimensions to meet that need.  There are a lot of the same problems as are faced by the web api MediaTrackConstraints.  So if you specify width and height and this is not available this should error as being over constrained. If instead we specify an aspect ratio and a width for example, it's more likely that we can deliver a  capture that closely matches the request.
It would be much easier to supply arguments of how big should the capture be, if the available sizes could be interrogated by the consuming code.   Then the 'matching code' in getOptimalPictureSizeForPreview doesn't have to guess what the user means

@WcaleNieWolny than something you can look up for

@eggbeard @riderx you want me to add a function like getSupportedPictureSizes?

Is implementing a function that returns following output sufficient to resolve this issue?
{
  "supportedPictureSizes": [
    {
      "facing": "Back",
      "supportedPictureSizes": [
        {
          "width": 4032,
          "height": 3024
        },
        {
          "width": 4000,
          "height": 3000
        },
        {
          "width": 4032,
          "height": 2268
        },
        {
          "width": 3840,
          "height": 2160
        },
        {
          "width": 4000,
          "height": 2000
        },
        {
          "width": 3264,
          "height": 2448
        },
        {
          "width": 3264,
          "height": 1836
        },
        {
          "width": 2560,
          "height": 1920
        },
        {
          "width": 2688,
          "height": 1512
        },
        {
          "width": 1920,
          "height": 1920
        },
        {
          "width": 2560,
          "height": 1280
        },
        {
          "width": 2048,
          "height": 1536
        },
        {
          "width": 1920,
          "height": 1440
        },
        {
          "width": 1920,
          "height": 1080
        },
        {
          "width": 4624,
          "height": 3472
        }
      ]
    },
    {
      "facing": "Front",
      "supportedPictureSizes": [
        {
          "width": 4032,
          "height": 3024
        },
        {
          "width": 4000,
          "height": 3000
        },
        {
          "width": 4032,
          "height": 2268
        },
        {
          "width": 3840,
          "height": 2160
        },
        {
          "width": 4000,
          "height": 2000
        },
        {
          "width": 3264,
          "height": 2448
        },
        {
          "width": 3264,
          "height": 1836
        },
        {
          "width": 2560,
          "height": 1920
        },
        {
          "width": 2688,
          "height": 1512
        },
        {
          "width": 1920,
          "height": 1920
        },
        {
          "width": 2560,
          "height": 1280
        },
        {
          "width": 2048,
          "height": 1536
        },
        {
          "width": 1920,
          "height": 1440
        },
        {
          "width": 1920,
          "height": 1080
        },
        {
          "width": 4208,
          "height": 3072
        }
      ]
    }
  ]
}

If getSupportedPictureSizes doesn't solve this issue, please reopen it

@riderx I agree with @WcaleNieWolny, we used getSupportedPictureSizes() to ensure we passed the correct width/height to CameraPreview.capture() but it ignores it on iOS and comes out as 1080x1920 whatever width/height we provide

I might be looking in the wrong place but on iOS capture() there doesn't seem to be any code relating to width/height
https://github.com/Cap-go/camera-preview/blob/main/ios/Plugin/Plugin.swift#L266-L304


Adds section to the docs about adding categories to posts auto-imported via saved search

See #38 (reopened).

Yep.


AttributeError: 'WorkbookItem' object has no attribute 'webpage_url'

Describe the bug
According to API, WorkbookItemclass has attribute webpage_url. Not sure if I need to enable something in my Tableau Server.
Versions
Tableau Server Version: 2021.1.10
Python Version: 3.8.5
TSC library version: 0.7
To Reproduce
tableau_auth = TSC.TableauAuth(tableau_user.password, tableau_pswd.password)
server = TSC.Server(config["server_address"], use_server_version=True)

with server.auth.sign_in(tableau_auth):
   all_wb, pagination_item = server.workbooks.get()
   for wb in all_wb:
       print(wb.webpage_url)

Results
AttributeError: 'WorkbookItem' object has no attribute 'webpage_url'

@ehg93 it looks like your TSC library version is somewhat old - could you update to the latest (0.17) and try again to see if that clears up your issue?
It looks like webpage_url was added in release 0.13.

I have updated to 0.17. The webpage_url issue is fixed. I had another question, out of the 32 workbook ids I provide, I only get 2 webpage_urls back, is there something in my Tableau Server settings that I am missing?

I am following on this here.

server.workbooks.get() does not get all workbooks, had to use TSC.Pager(server.workbooks) instead.


fix quick links and address text

hello,
I'm Suryakant,  in your project i want to improve/fix the not so good looking quick links and 'address' text, so that it could be more appealing and look better.
kindly create and assign this issue under GSSoC'21.

/assign`

can you elaborate the issue, maybe add some screenshots ?
looks repeated issue like #522


Flutter freezed v4.0.0

Description
The motivation for this version started when @endigo identified some bugs with the plugin.
As a quick resolution, I suggested this solution .
Then it hit me... I took a look at the schema @endigo  mentioned and realized how big it was, I tried implementing my initial suggestion on that schema and ended up with a bloated config file spanning close to 500 lines just for some minor configurations.
I was working on a project at that time and was writing pure CSS. It was then that I began to wonder how certain CSS properties have multiple syntaxes. Taking inspiration from the flexibility CSS offers, I began experimenting with RegExp to be used for configuring the plugin.
Initially, I thought I was going too far with this but I stubbornly persisted... +3 months of sleepless nights debugging and tons of refactoring later, comes this PR which hopefully might inspire other plugins too.
Documentation is still in progress and video tutorials will soon follow once this PR gets merged.
Looking forward to your usual collaboration @charlypoly, @Urigo  and all the team at the Guild
Related # (issue)
Replaces #8350 and #14
Type of change

[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

How Has This Been Tested?
All modules have a corresponding test file that verifies the generated output.

Install the dependencies with yarrn
run yarn test

Test Environment:

OS: Garuda Linux
@graphql-codegen/...: latest
NodeJS: latest

Checklist:

[x] I have followed the CONTRIBUTING doc and the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Sincere gratitude to @auaicn and @endigo and everyone who has helped along the way 🥂

@charlypoly @Urigo we have a green flag ... over to you

hey @Parables can you  create a changeset https://github.com/dotansimha/graphql-code-generator-community/pull/47#issuecomment-1383675987

Sure, I will that around 10pm GMT

@saihaj done as requested

@saihaj  I have to correct some typos and grammatical errors in the guide but I have to do that at school as I am already running late. Give me about 1 hour 30 mins to get to school so that we can continue

And please how do I fix the prettier error?

@saihaj I am back, what else do you need me to do?

@Parables can you run yarn prettier so CI is happy then I can merge and release this.

Done that but there are no changes to be committed here on my local repo

@Parables can you run yarn prettier so CI is happy then I can merge and release this.

Done that but there are no changes to be committed here on my local branch.
However, when I run yarn to install the dependencies, other packages were modified.
I didn't commit them because it was not my intention to modify them.
Any other solution?

weird I pulled this branch and ran prettier and it was just this plugin https://github.com/dotansimha/graphql-code-generator-community/pull/47/commits/b6c4b0159c53530251dc50040eeac282f6e128e9

Thanks very much @saihaj. I couldn't have figured out this issue on my own.
The guide for this plugin on the website to has been updated.
Everything is ready now for release.

hmm so something went wrong with CI publish flow. So will investigate it @Parables

Thanks very much for your dedication @saihaj

/theguild newsletter

released @Parables https://github.com/dotansimha/graphql-code-generator-community/releases/tag/release-1674417409150

Thank you all so much for the support.
This release wouldn't be possible without you:
@saihaj
@charlypoly
@Urigo
@endigo
@n1ru4l
@auaicn
@gabsn
@everyone
Cheers 🥂

hmm so something went wrong with CI publish flow. So will investigate it @Parables

I'm curious to learn why it failed. a little insight could help me learn from my mistakes

Thank you all so much for the support.
This release wouldn't be possible without you:
@saihaj
@charlypoly
@Urigo
@endigo
@n1ru4l
@auaicn
@gabsn
@everyone
Cheers 🥂


Fix ugprade dependencies/submodules script

knobots might use a non existing version branch, in that case, fetch main branch.
Tested with:
./hack/update-deps.sh --upgrade --release v1.10

/cc @matzew @creydr

Ping @matzew @creydr

/hold for https://github.com/knative-sandbox/eventing-istio/pull/16

/unhold


MM-12936 - Fixing alignment of post control menu

Summary
MM-12936 - Fixing alignment of post control menu
Ticket Link
https://mattermost.atlassian.net/browse/MM-12936
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] Ran make check-style to check for style errors (required for all pull requests)
[ ] Ran make test to ensure unit and component tests passed
[ ] Added or updated unit tests (required for all new features)
[ ] Needs to be implemented in mobile (link to PR or User Story)
[ ] Has server changes (please link)
[ ] Has redux changes (please link)
[x] Has UI changes
[ ] Includes text changes and localization file (.../i18n/en.json) updates
[ ] Touches critical sections of the codebase (auth, posting, etc.)

cherry pick 5c82bf3c3668df6ba9fa6288718b09e41431cf82 into release branch 5.5


Use IMDSv2 for instance metadata

Enforce the use of IMDSv2 for instance metadata.

Hash commit fe416e9d488de81372f1e45b39d6d93b58a6dd30 resolves this issue.


[resources] Adding extra resource dirs.

The resources (icons, ...) system functions (getIcon, ...) in silx are very practical, but they don't allow (AFAIK) for a user to add its own folder(s) to the lookup path, which means that a client project has to duplicate silx code into its own project to use these functionalities (well that's what I did anyway).
So maybe it would be interesting to allow the user to register a folder and a namespace (to avoid conflicts) to the list of paths into which silx looks for resources?
And then you could get files like resource_filename('myNs:foo/myFile').

Close by https://github.com/silx-kit/silx/pull/1054


Consider migrating tests from AppVeyor/Travis CI to Azure Pipelines

Consider migrating all tests from AppVeyor/Travis CI to Azure Pipelines.
Why migrate tests?

OSX tests on Travis CI are quite slow and have proven flaky from time to time (e.g. #4487,  #4448).
AppVeyor allows only a single job to run at a time; Azure Pipelines allows for 10 parallel jobs
Azure Pipelines can run both Windows and Mac OSX tests (note: no GPUs.)
Azure Pipelines is provided free of charge to open-source projects

TODO. Test Azure Pipelines for performance and stability
cc @sriramch

+10000 on getting rid of AppVeyor

I played with Azure Pipelines a bit, and so far it's been pleasure to work with.

Builds start almost instantaneously and run fast.
In a single YAML file, I can use Linux, Mac OSX, and Windows agents.
Jobs can run in parallel, but some jobs can be made dependent on other jobs.

See example: azure-pipelines.yml, build result

Can't wait.

I am also interested in seeing Azure Pipelines instead of AppVeyor

We've set up GitHub Actions (#5917) and moved some tests from AppVeyor to GitHub Actions (#5922). There is currently a nondeterministic test that's blocking a further migration away from AppVeyor.
We are keeping Travis CI because we need access to MacOS High Sierra to build XGBoost Python wheel. (Expectation is that it will be compatible with last three major releases, High Sierra, Mojave, and Catalina.) GitHub Actions only provides MacOS Catalina.
Closing this in favor of #5891.

@hcho3 I don't think we are running 3 versions of osx on travis right?

And another thing is, our test on Travis now take 1.5+ hour to run for each PR.  And multiple PRs cannot be run in parallel.

And multiple PRs cannot be run in parallel.

Travis CI runs multiple Linux jobs just fine. It's just that MacOS agent pool is really small, so our jobs get stuck in the global queue for a long time.

I don't think we are running 3 versions of osx on travis right?

No. We assume backward compatibility: if we can build and run XGBoost on MacOS High Sierra, then the generated binary should also be compatible with MacOS Mojave/Catalina.

Got it.  Thanks for the explanation.

Reopening, since Travis CI seems to be quite slow nowadays. The tests remain in queue for 2-3 hours. (Once they start, they are usually fairly quick.)
A proposal: Migrate all MacOS tests to GitHub Actions. Travis CI should only run on the master branch, to generate and upload nightly builds (Python wheels). Hopefully, discrepancy between MacOS releases isn't too big so that code that works in one release would break in another too often.  @trivialfis WDYT?

@hcho3 I'm not sure about the ecosystem on osx.  I will leave it to others to decide. ;-)  @CodingCat seems also fond of osx.

I think the backward compatibility of osx is generally guaranteed....
On Wed, Jul 22, 2020 at 6:43 PM Jiaming Yuan<EMAIL_ADDRESS>wrote:

@hcho3 https://github.com/hcho3 I'm not sure about the ecosystem on
osx. I will leave it to others to decide. ;-) @CodingCat
https://github.com/CodingCat seems also fond of osx.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/xgboost/issues/4498#issuecomment-662780619, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFFQ6HUV5ESUKI2WKT5TA3R46ITJANCNFSM4HPSVYFA
.

Travis CI seems fine now.


Merge from vscode 79a1f5a5ca0c6c53db617aa1fa5a2396d2caebe2

This PR fixes #

Coverage decreased (-0.1%) to 34.817% when pulling 8b0b0a029e6145095bb06e93fd3b39c7b6086d94 on ads-master-vscode-2020-05-31T19-47-47 into 84492049e8992e47a4dcaefdacb4fdcf3fcc291f on master.


Provide date range filter for ListArticles and ListReplies

From 20200212 meeting note, it would be great if we can provide date range search on createdAt for articles and replies, lastRequestedAt or having article reply createdAt for articles.
To achieve this, we plan to:

Replace current existing script queries with range queries so that the operators can compatible with date type
Simulate current EQ operator with gte + lte operator so that it is compatible with range queries.

最近 elasticsearch 的 docker container 有一些 warning 如下：
https://gist.github.com/MrOrz/d60b1fb33a89f240afa4147538c2688e
看起來下 /articles/doc/_count  request 時， doc['replyRequestCount'].value > params.operand 這個條件偶爾會觸發 runtime error，每小時會觸發個幾次的樣子。
希望 range operator 改下去之後可以讓這個 warning 消失

Partially fixed in #154 ; we can move other arithmetic operators like lastRequestedAt to range filters as well.


Set observedGeneration under condition Ready

Currently, observedGeneration is only set in status level for gitrepository resource. It would be helpful to set observedGeneration under conditions as well. To reliably rely on kubectl wait command for ready condition to be met.
If the previous condition was ready for gitrepository resource then, sometime kubectl wait will confirm to soon as condition met based on previous Ready condition.
observedGeneration  can be set to all conditions of all Kinds

This is being worked on in https://github.com/fluxcd/pkg/pull/101 and #361


add node_modules to .gitignore

Why? Kinda nice to be able to just run the thing from checkout, isn't it?

Why? Kinda nice to be able to just run the thing from checkout, isn't it?

if it's up to date, yeah. it's a lot of noise, and it can be difficult when going to revisions in the past, if the modules weren't updated correctly. if you want to keep them checked in, that's fine. most npm projects I've seen stop doing that. whatever's easiest for users/devs I suppose.

I prefer to keep as is for now.

okay, that's fine


Fixes breakage with fix model

Fixes breakage with fix model when hitting the fix button and the model armature is not under any collections. This patches that by adding a new collection and throwing everything under that.

Are you sure this fixes whatever the issue is here?
As far as I can tell, armature here comes from armature = Common.set_default_stage() which gets the armature through armature = get_armature(), which gets the first armature object in get_objects() with name matching bpy.context.scene.armature (or first armature object if bpy.context.scene.armature is a Falsey value such as ''). get_objects() finds only objects in bpy.context.view_layer.objects (for Blender 2.80+). From this, we can say that armature must be in the current view_layer and therefore, must be in a collection and therefore parentcollectionname = armature.users_collection[0].name will never raise an exception from being out of bounds (unless some of the previous code between here and armature = Common.set_default_stage() has removed it from all collections for some reason, in which case, why?).
Additionally, all of this code using collections needs to be surrounded with if not version_2_79_or_older(): as Blender 2.79 does not have collections.

Ok. If this is related to

then that's because the scene variable used in scene.collection.children.link(coll) has not been defined.
Though, from my previous comment, this appears odd, as this line of code is only executed when coll = bpy.data.collections.get(name) is a Falsey value, but I've just said that as far as I can tell armature will always be in a collection.
What can happen, and what I think is happening, is that armature is in the "Scene Collection". If you attempt to get the name of the "Scene Collection" you will get 'Master Collection'. There is no bpy.data.collections['Master Collection'] as the collection is part of the scene and is not a separate collection that can be found in bpy.data.collections.

People can import their model outside of a collection. This is only to fix an issue that they had which was solved by exactly just putting the armature inside a collection. That error message and stuff is unrelated to this fix.

If we need to fix another issue, then a new pull request would be easiest.

My point is that you're saying the issue is when an armature is not in a collection, but the armature must always be in a collection as per my first comment, unless some of the code after armature = Common.set_default_stage() is removing it from all collections and thus removing it from the scene, in which case, fix that problem instead of putting a band aid on top of what the actual problem is.

"Fixes breakage with fix model when hitting the fix button and the model armature is not under any collections. This patches that by adding a new collection and throwing everything under that."
This is what causes the issue, put an armature for a model not under a collection. Hit fix model, error occurs. I expect no error. And I haven't seen one before this issue

"Fixes breakage with fix model when hitting the fix button and the model armature is not under any collections. This patches that by adding a new collection and throwing everything under that."
This is what causes the issue, put an armature for a model not under a collection. Hit fix model, error occurs. I expect no error. And I haven't seen one before this issue

But if you remove the armature from all collections, it's no longer in the view layer of the scene and is effectively orphan data and it won't be available for Cats to detect, meaning you can't even hit the fix model button.

It's under scene collection and not in a collection under scenes. It's still in the scene. It happens with some importers like Blender Source Tools by default

It's under scene collection and not in a collection under scenes. It's still in the scene. It happens with some importers like Blender Source Tools by default

So the armature is still in a collection.
Yes, fix model is currently broken in the case that the Armature is in the Scene Collection because the current code tries to find the Scene Collection in bpy.data.collections, which does not contain scene collections, so it thinks the Armature is not in a collection (bug 1),
and then runs code to add it to a new collection, but that code contains an undeclared 'scene' variable (bug 2).
This PR fixes neither of these bugs, the code still tries to get the collection from bpy.data.collections (bpy.data.collections.get(parentcollectionname)), which will return None if the collection is a scene collection, and still contains the undeclared 'scene' variable (scene.collection.children.link(coll)), so what does it fix?
If you try to Fix Model with the code in this PR and an armature in the Scene Collection, it still gives you the NameError because the 'scene' variable is not defined. (This is run with the changed code in this PR)

yes, which is why your solution works