Fix asyncResult use!, try..with and try..finally

Hi @swlaschin,
First of all, thanks for the great workshop @ NDC Oslo 2019 :)
We've been messing around with the workshop material with my colleagues and found this issue with asyncResult: use! immediately disposes the object, before the continuation is actually run. By extension, I checked and found try..with to behave in a similar manner.
I'm not sure if this is the best way to fix it, but it seems to work. The whole custom CE implementation in F# is still somewhat mystic to me.

BTW, This probably fixes #8

Oh, and apparently a similar fix has already been discovered in #2

Thanks for fixing this! Sorry for the delay in responding!


added note about calling catalog api for default catalog

Docs Build status updates of commit 02608e2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Advertising/shopping-content/code-example-manage-products.md
:white_check_mark:Succeeded

Advertising/shopping-content/get-started.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add caching for the most used public endpoints

Hey, I suggest using model caching for common models like Currency, Market, etc. This way we can update cache when the model is updated.
Ex.
https://github.com/GeneaLabs/laravel-model-caching

Hey, I suggest using model caching for common models like Currency, Market, etc. This way we can update cache when the model is updated.
Ex.
https://github.com/GeneaLabs/laravel-model-caching

I will use model callbacks to wipe the cache after commit.


Question: Does DDS-Router support data deduplication

Lets assume I have on one machine one publisher and on the other machine 10 subscribers. Those two machines are connected over ethernet. Is the data sent over ethernet once or 10 times?
And if deduplication is not supported yet, is it on the roadmap?

Hi @deeplearningrobotics,
Let's assume you have the following scenario:
1 Publisher <-> DDS Router 1 <-Ethernet-> DDS Router 2 <-> 10 Subscribers

The DDS Router 1 will only send the data once to DDS Router 2. Then the DDS Router 2 will behave as the publisher, sending the data to each subscriber. This is because the publisher and DDS Router 1 are agnostic of the 10 subscribers, i.e., it only sees the subscriber of DDS Router 1.


Identity 2965

IDENTITY-2965 CarbonRemoteUserStoreManger must extend AbstractUserStoreManager

This PR depends on https://github.com/wso2/carbon4-kernel/pull/267,
need to merge that and update kernel version in master pom to newer kernel version.


Current Build

Not really an issue but you should mention that it requires apcu

Ah! Sorry, I keep forgetting that.
The throttle package uses that.
Thanks!
I'll make a note of that in the readme.


⚠ SmartAtlantic ERDDAP has degraded performance

In e8474db, SmartAtlantic ERDDAP (https://www.smartatlantic.ca/erddap/index.html) experienced degraded performance:

HTTP code: 503
Response time: 21074 ms

Resolved: SmartAtlantic ERDDAP performance has improved in 86b0e1f after 5 minutes.


ack-reboot doesn't calculate the correct uptime

While testing this new feature in a cloud, we found that the ack-reboot action is incorrectly calculating the timestamp since the last reboot.
For a given server, we see this:
ubuntu@:/etc/nagios/nrpe.d$ uptime -p
up 46 weeks, 1 day, 10 hours, 8 minutes
running the ack-reboot action yields this:
message: known reboot time updated to 2021-03-30 06:08:14

(full output is here - https://pastebin.canonical.com/p/FcBwWhZwQJ/)
However, when comparing that against the check_reboot script, we see the last reboot was actually:
CRITICAL: unknown reboot at 2021-03-30 06:08:15
So i think there may be a rounding error or miscalculation for the ack-reboot action

Imported from Launchpad using lp2gh.

date created: 2022-02-16T16:18:52Z

owner: thogarre

assignee: guoqiao

the launchpad url

(by guoqiao)
The check_reboot.py script relies on output from uptime --since to get reboot time, and compared with the known_reboot_time arg.
It's noticed that the output is actually not stable/consistent:
ubuntu@juju-733f2f-8-lxd-9:/etc/nagios/nrpe.d$ sudo -u nagios /usr/local/lib/nagios/plugins/check_reboot.py "2021-03-30 05:30:54"
CRITICAL: unknown reboot at 2021-03-30 05:30:55
ubuntu@juju-733f2f-8-lxd-9:/etc/nagios/nrpe.d$ sudo -u nagios /usr/local/lib/nagios/plugins/check_reboot.py "2021-03-30 05:30:54"
OK: system is up since 2021-03-30 05:30:54   --> change from :55 to :54
ubuntu@juju-733f2f-8-lxd-9:/etc/nagios/nrpe.d$ sudo -u nagios /usr/local/lib/nagios/plugins/check_reboot.py "2021-03-30 05:30:54"
OK: system is up since 2021-03-30 05:30:54
ubuntu@juju-733f2f-8-lxd-9:/etc/nagios/nrpe.d$ sudo -u nagios /usr/local/lib/nagios/plugins/check_reboot.py "2021-03-30 05:30:54"
CRITICAL: unknown reboot at 2021-03-30 05:30:55  --> change again from :54 to :55
ubuntu@juju-733f2f-8-lxd-9:/etc/nagios/nrpe.d$

This seems because ntp may keep changing the system time.
A patch to allow 5s gap is created:
https://code.launchpad.net/~guoqiao/charm-nrpe/+git/nrpe-charm/+merge/415698


⚠ Fortnite has degraded performance

In 5afeb59, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1621 ms

Resolved: Fortnite performance has improved in 2dea5a2.


Improve performance and optimize memory

Reduce the consumption, improve performance

[ ] Optimize the use of Arenas
[ ] Add dispose() to all classes with native pointer
[ ] Add detach to the constructors with native pointer

Codecov Report
Attention: Patch coverage is 51.11111% with 330 lines in your changes are missing coverage. Please review.

Project coverage is 81.40%. Comparing base (a04f909) to head (187c828).

Files
Patch %
Lines

lib/src/core/cv_vec.dart
36.36%
70 Missing :warning:

lib/src/features2d/features2d.dart
60.19%
41 Missing :warning:

lib/src/core/point.dart
45.61%
31 Missing :warning:

lib/src/core/array.dart
20.00%
28 Missing :warning:

lib/src/core/vec.dart
29.62%
19 Missing :warning:

lib/src/core/rect.dart
30.00%
14 Missing :warning:

lib/src/video/video.dart
75.92%
13 Missing :warning:

lib/src/core/mat.dart
68.57%
11 Missing :warning:

lib/src/imgproc/imgproc.dart
81.81%
10 Missing :warning:

lib/src/core/dmatch.dart
35.71%
9 Missing :warning:

... and 18 more

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #67      +/-   ##
==========================================
- Coverage   88.97%   81.40%   -7.57%
==========================================
  Files          36       36
  Lines        4897     5147     +250
==========================================
- Hits         4357     4190     -167
- Misses        540      957     +417

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

Ready to merge.
Note:

Wrapper classes with large native resources, users now can free them manually with dispose() to free native resources, but note that the dart instances will still be available, users have to ensure not to access them if disposed, dart instances are small and believe dart VM/GC to manage them.

Example:

import 'package:opencv_dart/opencv_dart.dart' as cv;

void main() {
  const counts = 10000;
  final mat = cv.Mat.zeros(1000, 1000, cv.MatType.CV_8UC3);
  final sw = Stopwatch()..start();
  for (var count = 0; count < counts; count++) {
    final mat1 = cv.cvtColor(mat, cv.COLOR_BGR2YCrCb);
    // manually dispose will reduce the memory consumption
    mat1.dispose();
  }
  sw.stop();
  print(
      "All: ${sw.elapsedMicroseconds}μs, counts: $counts, per: ${sw.elapsedMicroseconds / counts} μs");
}

Without mat1.dispose(), memory: ~18G
All: 5527358μs, counts: 10000, per: 552.7358 μs

With mat1.dispose(), memory: ~240M
All: 7263633μs, counts: 10000, per: 726.3633 μs

package user will have to manage the object?

Ready to merge.
Note:

Wrapper classes with large native resources, users now can free them manually with dispose() to free native resources, but note that the dart instances will still be available, users have to ensure not to access them if disposed, dart instances are small and believe dart VM/GC to manage them.

Example:
import 'package:opencv_dart/opencv_dart.dart' as cv;

void main() {
  const counts = 10000;
  final mat = cv.Mat.zeros(1000, 1000, cv.MatType.CV_8UC3);
  final sw = Stopwatch()..start();
  for (var count = 0; count < counts; count++) {
    final mat1 = cv.cvtColor(mat, cv.COLOR_BGR2YCrCb);
    // manually dispose will reduce the memory consumption
    mat1.dispose();
  }
  sw.stop();
  print(
      "All: ${sw.elapsedMicroseconds}μs, counts: $counts, per: ${sw.elapsedMicroseconds / counts} μs");
}

Without mat1.dispose(), peak memory: ~18G
All: 5527358μs, counts: 10000, per: 552.7358 μs

With mat1.dispose(), peak memory: ~240M
All: 7263633μs, counts: 10000, per: 726.3633 μs

Does this mean the package user will have to manage the object? I another way do I have to dispose every native asset I use?

Not necessary, the above test is for dart, I didn't test in Flutter but Flutter will trigger GC more frequently than pure dart, so it tends to rely on GC to free, BUT, if users can know that the APP will process large images (e.g., 384021603), it's recommend to dispose manually.
Anyway, this PR just provides a potential to manage native resources more efficiently according to the weakness of the GC of dart VM, but not necessary.

Oh got it , so GC works too but if needed manual mode can be used for optimization

Oh got it , so GC works too but if needed manual mode can be used for optimization

bingo!


Unable to run your project. the "RunCommand" property is not defined.

I do follow this guid.
=> https://blazor.net/docs/get-started.html

At step 4 > Press Ctrl-F5 to run the app without the debugger. Running with the debugger (F5) isn't supported at this time.
When i do it , then dialog pop.

When try "dotnet run"
result is

How can i do for run blazor?

I experienced the same issue but turned out that I didn't add reference to Microsoft.AspNetCore.Blazor.DevServer


Message: Failed to deserialize textDocument/hover: .: missing field line;

I'm getting this output from Rust Analyzer Language Server. I do not have a concrete repro right now as I've just returned from vacay so I'll try and cook something up later.  Everything worked when I stopped working 10 days ago, only thing I did since I came back is updated vscode to latest 1.59.0 from whatever was 2 weeks ago
rust analyzer version: 0.2.702
remote sshing into ubuntu from windows 10 with latest updates
Version: 1.59.0 (system setup)
Date: 2021-08-04T23:13:12.822Z
Electron: 13.1.7
Chrome: 91.0.4472.124
Node.js: 14.16.0
V8: 9.1.269.36-electron.0
OS: Windows_NT x64 10.0.19043

[Error - 3:49:00 PM] Request textDocument/hover failed.
  Message: Failed to deserialize textDocument/hover: .: missing field `line`; {"textDocument":{"uri":"file:///home/mister/file.rs"},"position":{"start":{"line":27,"character":52},"end":{"line":28,"character":41}}}
  Code: -32602
[Error - 3:49:17 PM] Request textDocument/hover failed.
  Message: Failed to deserialize textDocument/hover: .: missing field `line`; {"textDocument":{"uri":"file:///home/mister/file.rs"},"position":{"start":{"line":29,"character":35},"end":{"line":29,"character":54}}}
  Code: -32602
[Error - 3:49:27 PM] Request textDocument/hover failed.
  Message: Failed to deserialize textDocument/hover: .: missing field `line`; {"textDocument":{"uri":"file:///home/mister/file.rs"},"position":{"start":{"line":38,"character":0},"end":{"line":38,"character":0}}}
  Code: -32602
[Error - 3:50:09 PM] Request textDocument/hover failed.
  Message: Failed to deserialize textDocument/hover: .: missing field `line`; {"textDocument":{"uri":"file:///home/mister/file.rs"},"position":{"start":{"line":38,"character":0},"end":{"line":38,"character":0}}}
  Code: -32602
[Error - 3:51:10 PM] Request textDocument/hover failed.
  Message: Failed to deserialize textDocument/hover: .: missing field `line`; {"textDocument":{"uri":"file:///home/mister/file.rs"},"position":{"start":{"line":21,"character":0},"end":{"line":21,"character":0}}}
  Code: -32602
[Error - 3:52:00 PM] Request textDocument/hover failed.
  Message: Failed to deserialize textDocument/hover: .: missing field `line`; {"textDocument":{"uri":"file:///home/mister/file.rs"},"position":{"start":{"line":44,"character":23},"end":{"line":44,"character":45}}}
  Code: -32602
[Error - 3:52:01 PM] Request textDocument/hover failed.
  Message: Failed to deserialize textDocument/hover: .: missing field `line`; {"textDocument":{"uri":"file:///home/mister/file.rs"},"position":{"start":{"line":44,"character":23},"end":{"line":44,"character":45}}}
  Code: -32602
[Error - 3:52:05 PM] Request textDocument/hover failed.
  Message: Failed to deserialize textDocument/hover: .: missing field `line`; {"textDocument":{"uri":"file:///home/mister/file.rs"},"position":{"start":{"line":44,"character":28},"end":{"line":44,"character":45}}}
  Code: -32602

That sounds like a mismatch between the language server and the extension version. Do you have a server path set in the remote Code config?

I've had this issue as well for the past 1-2 weeks. I finally figured out a way to reproduce it consistently.

Go into any .rs file - even just cargo new foo and open main.rs
Click ANYWHERE with the mouse cursor, so a text cursor is placed
Move the mouse cursor away
Move the mouse cursor on top (or really close) to the text cursor

This consistently posts the same error message as @doivosevic's to the "Output" panel for me.
GIF:

Here's 3 of the same errors:
[Error - 12:17:10 PM] Request textDocument/hover failed.
  Message: Failed to deserialize textDocument/hover: .: missing field `line`; {"textDocument":{"uri":"file:///home/vallentin/ra-issue/src/main.rs"},"position":{"start":{"line":1,"character":30},"end":{"line":1,"character":30}}}
  Code: -32602
[Error - 12:17:20 PM] Request textDocument/hover failed.
  Message: Failed to deserialize textDocument/hover: .: missing field `line`; {"textDocument":{"uri":"file:///home/vallentin/ra-issue/src/main.rs"},"position":{"start":{"line":1,"character":9},"end":{"line":1,"character":9}}}
  Code: -32602
[Error - 12:19:11 PM] Request textDocument/hover failed.
  Message: Failed to deserialize textDocument/hover: .: missing field `line`; {"textDocument":{"uri":"file:///home/vallentin/ra-issue/src/main.rs"},"position":{"start":{"line":3,"character":0},"end":{"line":3,"character":0}}}
  Code: -32602

I've tried reinstalling rust-analyzer, and that didn't change anything.
Versions:

rustc 1.49.0 (e1884a8e3 2020-12-29)
rust-analyzer version: e5c1c8cf2 2021-07-05 stable
VSCode - Version: 1.59.0 (user setup), Date: 2021-08-04T23:13:12.822Z, OS: Windows_NT x64 10.0.19043

For what it's worth, I'm using Ubuntu through WSL on Windows.

On the Windows machine itself, I don't have this issue. However, I'm also using Rust 1.54 and can see that I'm also using a different version of rust-analyzer as well.

rustc 1.54.0 (a178d0322 2021-07-26)
rust-analyzer version: 5664a2b0b 2021-08-09 stable

Not sure if the version difference is due to the older Rust version, or because it hasn't updated?

@vallentin

rust-analyzer version: e5c1c8c 2021-07-05 stable

Why are you using a rust-analyzer version that's a month older than the Code extension?

I honestly don't know? When rust-analyzer prompts to download an update, I do so. So I guess it haven't prompted an update for some time? Though I did reinstall the rust-analyzer extension just the other day. I just did it right now as well. Isn't it supposed to check for an update and prompt?

The extension should download the corresponding server binary unless another one has been set in settings.json. Can you check for the latter (on the remote side) and maybe reinstall the extension? There was a short period of time when the version checking was broken.

unless another one has been set in settings.json

I have never specified one manually, and the only thing in my settings are:
"rust-analyzer.cargo.allFeatures": true,
"rust-analyzer.diagnostics.disabled": [
    "unresolved-import",
    "macro-error",
],

maybe reinstall the extension

I just did before :)
When I check the installed extension version, it says<EMAIL_ADDRESS>which is the newest according to VSCode Marketplace.
However, I still haven't gotten any prompt to update rust-analyzer and it still says rust-analyzer version: e5c1c8cf2 2021-07-05 stable.

Try deleting the downloaded rust-analyzer binary from the extension.

Deleting the rust-analyzer binary and reloading VSCode, then it finally prompted it to download 🎉

rust-analyzer version: 5664a2b0b 2021-08-09 stable

I of course don't know if this is the same problem as @doivosevic.

I also ran into this issue and resolved it by deleting the downloaded rust-analyzer binary.
For anyone else who is confused like I was, the path is documented in the user manual: https://rust-analyzer.github.io/manual.html#vs-code

What worked for me is updating rust-analyzer. Apparently the vscode extension version used was through nix which was using an older version. Now I don't seem to have any problems

Let's close this, it was a version mismatch.


Pharos docker registry timeout (504)

What happened:
registry.pharos.sh report timeout with status code 504 and docker cannot pull pharos images.

What you expected to happen:
200 status code and successful docker pull.
How to reproduce it (as minimally and precisely as possible):
docker pull registry.pharos.sh/kontenapharos/kube-proxy:v1.14.9
Anything else we need to know?:
Environment:
$ docker version
Client: Docker Engine - Community
 Version:           19.03.5
 API version:       1.40
 Go version:        go1.12.12```

registry has been shut down

@madddi on purpose?

jes @jacub, the company shut down

Ah, that's sad news. Would be nice to be informed ahead on some official channel.
Anyway, I'll keep this open as this affects all users interacting with pharos-cluster tool and basically disables the usage of the whole solution (initialization of new cluster and managing the running clusters).

well they did publish a blogpost etc almost two months ago

Kontena is completely shut down. They announced it on their slack channel and website which is shut down now. Unfortunately there isn't any info on github.

can't we get the image source code and just push them to another repository?

That won't work. Because pharos cluster is integrated with their services. Authentication etc..

@shinebayar-g  AFAIK kontena's services are not mandatory for OSS version. We had problems only with downtime of pharos registry.

if anyone has an kontena version up and running we could extract the tar archives and push them to a new registry

All the images are available under docker.io/kontenapharos.

@jakolehm thank you, do you think they can be open sourced the ones used in pharos? So we can update them as time goes by?

There is very little to open source.. most of the stuff is built from upstream sources. Basically most were a build pipeline configs which were used to bake multi-arch images (because upstream was not always providing those).
Pharos 3.x (which is pure OSS) has switched to use upstream repos so this should not be a problem anymore.


Fix null replica in HelixClusterMap when decommissioning replicas.

Ambry frontend now allows GET operation to try on OFFLINE replicas. During replica decommission, the first step is to
disable replica. This will eventually bring replica to OFFLINE state and remove its entry from DataNodeConfig. All listeners
(both frontends and servers) will receive the DataNodeConfig change and remove this replica from in-mem clustermap. However, at this point of time, the IdealState in Helix hasn't been changed and this replica still exists in external view. So when router calls getReplicaIdByState(), the RoutingTableProvider still returns instance(server) associated with this replica. This causes a null replica when router queries in-mem clustermap by this instance and our code doesn't invalidate the null value in the return result, which triggers NPEs in the end.
This PR makes two changes:

Add a config to control whether to add down replicas to the replica pool in operation tracker. If yes, down(offline) replicas
will be added to the end.
Preclude null replicas in the method of getReplicaIdsByState().

Codecov Report

Merging #1705 (3d3e6c4) into master (c28a89b) will decrease coverage by 11.00%.
The diff coverage is 78.94%.

@@              Coverage Diff              @@
##             master    #1705       +/-   ##
=============================================
- Coverage     73.84%   62.83%   -11.01%
+ Complexity     8969     7488     -1481
=============================================
  Files           647      650        +3
  Lines         49212    49736      +524
  Branches       6163     6224       +61
=============================================
- Hits          36341    31253     -5088
- Misses        10967    16516     +5549
- Partials       1904     1967       +63

Impacted Files
Coverage Δ
Complexity Δ

...om/github/ambry/router/SimpleOperationTracker.java
92.59% <75.00%> (-0.81%)
80.00 <1.00> (-1.00)

...ain/java/com/github/ambry/config/RouterConfig.java
97.97% <100.00%> (-0.05%)
1.00 <0.00> (ø)

...ub/ambry/clustermap/HelixClusterChangeHandler.java
100.00% <100.00%> (ø)
3.00 <1.00> (ø)

...api/src/main/java/com/github/ambry/named/Page.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-3.00%)

...rc/main/java/com/github/ambry/store/TimeRange.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-8.00%)

...java/com/github/ambry/notification/UpdateType.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

...n/java/com/github/ambry/store/CostBenefitInfo.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-9.00%)

...m/github/ambry/config/AccountStatsMySqlConfig.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

...om/github/ambry/store/CompactAllPolicyFactory.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-2.00%)

...om/github/ambry/store/CompactionPolicyCounter.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-5.00%)

... and 170 more

Continue to review full report at Codecov.

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


Change the install type for crc so it will list differently to telemeter

https://github.com/code-ready/crc/issues/508 this is something we need to do as part of snc side.

Fixed via https://github.com/code-ready/snc/commit/f80d3842d713ca92a479d6f0ee69ec5ce77903e1


xboot.bin does not work!

Hello, I had visited almost all of your documents and tried to use xfel to boot the boot.bin which under the directory tools/windows on my Nezha-MQ.
BUT IT DOES NOT WORK!

I had also tried the rt_thread.bin which under the same directory and it works well.
So I think maybe your image has broken, could you just test it again and then tell me the result, please.

You are right, the current xboot.bin is only for testing whether the startup is normal, currently only for the nezha development board, if you want to develop applications based on xboot, please compile xboot yourself


update gokrb5 to v8.4.1

Hello!
This updates gokrb5 dependency to v8.4.1 which fixes an issue I ran into when a KDC server was unresponsive: https://github.com/jcmturner/gokrb5/pull/399

@bai I’m pretty sure I attempted this upgrade as part of #1626 and had to revert in #1630 because it broke some users.
Essentially:

Switching to go module based import paths for dependencies would require every consumer of the client to have also switched. We should wait for Sarama/v2 before doing that

Oh good point, I missed that one 👍

I’m happy for us to just start prepping a v2 though 😄

Looking forward to v2! I'll go ahead and close this PR and in the meantime I'll use my branch to handle this scenario


ACI support for DAPR

Are there plans to support DAPR for ACI instances?

Just to make sure:
Are you asking about Azure Container Instances OR Azure Kubernetes Service with Virtual Kublet/ACI underneath?

The latter one.
On Fri, Nov 15, 2019 at 3:18 PM Ricardo Niepel<EMAIL_ADDRESS>wrote:

Just to make sure:
Are you asking about Azure Container Instances OR Azure Kubernetes Service
with Virtual Kublet/ACI underneath?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dapr/dapr/issues/804?email_source=notifications&email_token=AFLWF3OFSEBYV2PRT2G2JDLQT37Y3A5CNFSM4JN6D7QKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEGTA7Q#issuecomment-554512510,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFLWF3PLMSP6QCPRESNAIPDQT37Y3ANCNFSM4JN6D7QA
.

This should all "just work" today. ACI and AKS + Virtual Nodes support sidecars, you'll just need to adjust the kubernetes instructions to enable deploying to ACI.
This isn't documented as well as it should but PRs to improve docs would be great!

I can do this, pls assign @brendandburns

I can do this, pls assign @brendandburns

Assigned. @RicardoNiepel feel free to reach out to me directly to work on this. We basically need docs on how to run Dapr and configure the parameters for daprd in containerized environments where you don't have the operator that does this for you.

Updated title to reflect what is being discussed here.

Hy @yaron2
I'd also be interested in contributing to that but would need a bit of guidance around where to start. Thanks!


remove deprecated dependency assert_cli

thanks for the fast feedback

the faults in ci is:
limited_independence_of_irrelevant_alternatives runs for a long time in cargo integration see rust-lang/cargo#6258 for tracing of this fault
and futures-test-preview do not compile

@nrc merged without bors?

Sorry thought I was still on /cargo


ODOMeter is wrong on XC40 Pure Electric 2023

The ODO Meter is displayed incorrectly. The decimal place seems wrong. My ODOMeter is 973km and it only shows 97km.
Very THX for u work!!!

What is the value shown in the app 970 or 973? This is the value from the API

for the Android System there are no values in the App.

and it shows only 97km in ioBroker not 970km

I thinks it is an issue on the api side. please check if its switching to 98km

I can confirm that this is an issue with the API itself. I've seen this when consuming the API directly, also for other values. E.g. the precision of tripMeter1 is off by 2 orders of magnitude.

Hello, something wrong with the ODOMeter. The value is now 2147483. The rest is still working fine. Does it come from the API?
THX


adding converters between DocumentKey

Part of the C++ migration.
Sending this PR for review and fix the design before using these converters everywhere.
This PR will eventually merge to master. For now, merging into the pending cpp/port_dockey for easier review.

Updated, PTAL.


[Coinpaprika] new feature: favorite coins

Description
New feature - Favorite Coins

Additional Section -> Favorite Coins
Add & Remove Favorite Coins
Action Show & Hide Favorite Coins in Coin List

Screencast

Checklist

✔ I read the extension guidelines
✔ I read the documentation about publishing
✔ I ran npm run build and tested this distribution build in Raycast
✔ I checked that files in the assets folder are used by the extension itself
✔ I checked that assets used by the README are placed outside of the metadata folder

Hi @gumbleliks 👋
Thanks for your contribution 🔥
I have some feedback for you 🙂

Your metadata images are not formatted correct, we now have a tool build into Raycast that can help you make some awesome good looking metadata images.
Maybe we should consider removing #1... in the title? Holding ⌘ down shows you some of the numbers - is there a reason for having them?

I'm looking forward to testing this extension again 🔥
Request a new review when you are ready. Feel free to contact me here or at Slack if you have any questions.

Hi @pernielsentikaer,
thanks for your feedback and the hints.

I created a new metadata image with your awesome screenshot tool. I'll use it for further development 😊
Should I also replace the media Images with screenshots out of the raycast screenshot-tool or are they fine?

Regarding the # in the  title: The reason why I'm showing the number is the marketrank (measured in total market value of a cryptocurrency's circulating supply.) of the coins. So it is a quite needed information.

Hi @gumbleliks 👋
Thanks for your contribution 🔥
I have some feedback for you 🙂

Your metadata images are not formatted correct, we now have a tool build into Raycast that can help you make some awesome good looking metadata images.
Maybe we should consider removing #1... in the title? Holding ⌘ down shows you some of the numbers - is there a reason for having them?

I'm looking forward to testing this extension again 🔥
Request a new review when you are ready. Feel free to contact me here or at Slack if you have any questions.

Hi @pernielsentikaer,
thanks for your feedback and the hints.
I created a new metadata image with your awesome screenshot tool. I'll use it for further development 😊
Should I also replace the media Images with screenshots out of the raycast screenshot-tool or are they fine?
Regarding the # in the title: The reason why I'm showing the number is the marketrank (measured in total market value of a cryptocurrency's circulating supply.) of the coins. So it is a quite needed information.


Feat: add kl social media to header

Added white social media icons centered to header. Omitting the twitch button so that you can add the button that you have commented whenever you want.
I have duplicated the "SOCIAL_NETWORKS" array, in order to give you the option of doing the refactor.
So that you extract them to the file that you think is convenient.


Peer to Peer Code Review

great job so far :+1: , I'd suggest a few code refactor changes.

I'd suggest using li.append(child,child,child)  .
remove unnecessary comments, the comment on form.reset().

@migo1 I kind of like the comments because they make finding things easy.
I also like your suggestion "li.append(child,child,child)" this will reduce repeation
😎


Support change pin functionality

I wonder if there is a change pin lock screen

Hi @lolgear
There is already a screen for supporting setting/updating a PIN. Just use ABPadLockScreenSetupViewController and configure it appropriately. The example demo project shows an example of how to use this.

I mean scenario:
input old pin -> set new pin -> set new pin agan.
I can't find it in Setup screen.


Can you keep the name the same

Can the name remain the same after compression

@gjmm123 the compression return the path to the file compressed. So use can use this path to rename the file if you want to.

I think this should be a issue.


[YouTube] Q&A message didn't have header

liveChatCallForQuestionsRenderer didn't have header in https://github.com/xenova/chat-downloader/blob/657e56eeec4ebe5af28de66b4d3653dbb796c8c1/chat_downloader/sites/youtube.py#L1759
Traceback (most recent call last):
  File "c:\users\test01\appdata\local\programs\python\python39\lib\runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "c:\users\test01\appdata\local\programs\python\python39\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\test01\AppData\Local\Programs\Python\Python39\Scripts\chat_downloader.exe\__main__.py", line 7, in <module>
  File "c:\users\test01\appdata\local\programs\python\python39\lib\site-packages\chat_downloader\cli.py", line 194, in main
    run(**args.__dict__)
  File "c:\users\test01\appdata\local\programs\python\python39\lib\site-packages\chat_downloader\chat_downloader.py", line 360, in run
    for message in chat:
  File "c:\users\test01\appdata\local\programs\python\python39\lib\site-packages\chat_downloader\sites\common.py", line 283, in __next__
    item = next(self.chat)
  File "c:\users\test01\appdata\local\programs\python\python39\lib\site-packages\chat_downloader\sites\youtube.py", line 1762, in _get_chat_messages
    parsed_header = self._parse_item(
  File "c:\users\test01\appdata\local\programs\python\python39\lib\site-packages\chat_downloader\sites\youtube.py", line 633, in _parse_item
    item_info = item.get(item_index)
AttributeError: 'NoneType' object has no attribute 'get'

Sorry for being off topic, but may you please accept my pull request at https://github.com/JC-Chung/yt-dlp/pull/1 ? I have been permanently banned from yt-dlp for rapid issue making, even though this contribution is completely valid. It simply adds the concurrent view count of the stream to the metadata and adds a comment

Sorry for being off topic, but may you please accept my pull request at JC-Chung/yt-dlp#1 ? I have been permanently banned from yt-dlp for rapid issue making, even though this contribution is completely valid. It simply adds the concurrent view count of the stream to the metadata and adds a comment

So should I accept your PR or just edit your updated part up if you are banned by yt-dlp?

Sorry for being off topic, but may you please accept my pull request at JC-Chung/yt-dlp#1 ? I have been permanently banned from yt-dlp for rapid issue making, even though this contribution is completely valid. It simply adds the concurrent view count of the stream to the metadata and adds a comment

So should I accept your PR or just edit your updated part up if you are banned by yt-dlp?

I'd just say accept it, however they discovered an alt of me that way, but you could just copy it up, and make sure to support the live events.

You now have consent to extract tiktok live chat and other events: https://github.com/isaackogan/TikTokLive/issues/89


Aot Complier Error

Hi I'm using this for my ionic3 project, but it show error message when I run command

ionic cordova build android --prod --minifyjs --minifycss --optimizejs

The error message is
Error<EMAIL_ADDRESS>Module build failed: Error: No element indexed by 1
I have the same error in another component and resolve after update the version, so I guess this is a bug in the module.
my ionic info
cli packages: (E:\bi-app\node_modules)
@ionic/cli-plugin-proxy : 1.5.7
@ionic/cli-utils        : 1.19.0
ionic (Ionic CLI)       : 3.19.0

global packages:
cordova (Cordova CLI) : 7.1.0

local packages:
@ionic/app-scripts : 3.1.7
Cordova Platforms  : android 6.3.0 ios 4.4.0
Ionic Framework    : ionic-angular 3.9.2

System:
Android SDK Tools : 26.0.2
Node              : v8.9.4
npm               : 5.6.0
OS                : Windows 10

Environment Variables:
ANDROID_HOME     : C:\Users\summer\AppData\Local\Android\sdk
HTTP_PROXY       : not set
http_proxy       : not set
HTTPS_PROXY      : not set
https_proxy      : not set
IONIC_HTTP_PROXY : not set
PROXY            : not set
proxy            : not set

Misc:
backend : pro

Any idea? Thanks

I'm experiencing this too :( Any ideas? Thanks!

I think its too late to reply,
use this
npm i @ionic-tools/emoji-picker --save

Use this instead https://www.npmjs.com/package/ngx-emoji-picker
What's New

Fixed the AOT (Ahead Of Time) Compile error or simply put the ---prod build compile error.
Upgraded the library from a Ng2 project to a Ng7 project.
Upgraded the library RxJs from v5 to v6 to make it support new angular versions.
Now compatible with Ng2 - Ng7 projects and as well as Ionic 3/4.


[RESOLVED] High ICMP ping response time

Problem started at 02:19:14 on 2023.04.12
Problem name: High ICMP ping response time
Host: ZO-FG
Severity: Warning
Operational data: Value: 165ms
Original problem ID: 16106
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23125&eventid=16106

Problem has been resolved in 1m 1s at 02:20:15 on 2023.04.12
Problem name: High ICMP ping response time
Host: ZO-FG
Severity: Warning
Original problem ID: 16106


errors for superfluous use of boolc

boolc( contains( plus there are more combinations like line_exists
I think this is not needed, https://github.com/larshp/abapGit/blob/f9318305e969799cef8c5b1f2f8f427d556d1970/src/apack/zcl_abapgit_apack_writer.clas.testclasses.abap#L23

hmm,

not sure, closing


Mobx-React does not work with React.CreateContext

Versions
Newest versions as of May 2018
mobx: 4.2.1
mobx-react: 5.1.2
react: 16.3.2
react-dom: 16.3.2

Bug
It seems like it is not possible to create an observing context consumer.
// A simple store which counts up the count value:
class CounterStore {
  @observable count: number = 0;
}
const counterStore = new CounterStore();
setInterval(() => counterStore.count ++, 1000);

// A simple context for the store
const Context = React.createContext<CounterStore>(null);

Expected behaviour
All context.consumers which rely on an observed value should rerender once an observed value changes.
Actual behaviour
The following screenshot contains two components which rely on the same observable value.
The first component does not access the observed value the second one does.
As you can see the first component does not rerender.

Demo
The following demos show two scenarios for the same problem:
ReactContext which holds a mobx store
https://stackblitz.com/edit/react-ts-q5ktj1?file=index.tsx
ReactContext which does not hold a mobx store
https://stackblitz.com/edit/react-ts-rjk13r?file=index.tsx

Thank you very much - your example contains a small error but helped me to find the solution.
The following code works for me:
(store) => (
        <Observer>
          {() => (
            <span>{store.count}</span>
          )}
        </Observer>
      )}
)

I also created a working demo for reference:
https://stackblitz.com/edit/react-ts-2pvbmx?file=index.tsx

The following code would create a observable Consumer:
  const Context = React.createContext<{ theme: string}>(null);
  const ObservableConsumer = ({children}) => {
     return <Context.Consumer>{(value) => (
    <Observer>{() => children(value)}</Observer>
  )}</Context.Consumer>

Do you think an abstraction like the following could be added to mobx-react?
function createObserableContext<T>(defaultValue: T, calculateChangedBits?: (prev: T, next: T) => number) {
  const Context = React.createContext<T>(defaultValue, calculateChangedBits);
  const ObservableConsumer = ({ children }) => {
    return <Context.Consumer>{(value) => (
      <Observer>{() => children(value)}</Observer>
    )}</Context.Consumer>
  }
  return { Provider: Context.Provider, Consumer: ObservableConsumer }
}

Demo: https://stackblitz.com/edit/react-ts-at3ucs?file=index.tsx

I suspect that const ObservableConsumer = observer(Context.Consumer)
would do the trick already.
Not sure if it belongs to this package, there is a more concise context
mechanism already in the form of Provider / inject, and two mechanisms to
offer the same thing is often confusing for others
Op ma 7 mei 2018 om 11:08 schreef Jan Nicklas<EMAIL_ADDRESS>
The following code would create a observable Consumer:
const Context = React.createContext<{ theme: string}>(null);
const ObservableConsumer = ({children}) => {
return <Context.Consumer>{(value) => (
{() => children(value)}
)}</Context.Consumer>
Do you think an abstraction like the following could be added to
mobx-react?
function createObserableContext(defaultValue: T, calculateChangedBits?: (prev: T, next: T) => number) {
const Context = React.createContext(defaultValue, calculateChangedBits);
const ObservableConsumer = ({ children }) => {
return <Context.Consumer>{(value) => (
{() => children(value)}
)}</Context.Consumer>
}
return { Provider: Context.Provider, Consumer: ObservableConsumer }
}
Demo: https://stackblitz.com/edit/react-ts-at3ucs?file=index.tsx
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/mobxjs/mobx-react/issues/471#issuecomment-387005617,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABvGhHy1goPw0H2GnG1Te9NOfpUPCxxeks5twA75gaJpZM4Tz_4Y
.

Unfortunately this does not work:
https://stackblitz.com/edit/react-ts-heai4b?file=index.tsx

ok, was worth trying
Op ma 7 mei 2018 om 16:11 schreef Jan Nicklas<EMAIL_ADDRESS>
Unfortunately this does not work:
https://stackblitz.com/edit/react-ts-heai4b?file=index.tsx
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/mobxjs/mobx-react/issues/471#issuecomment-387077370,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABvGhKm_0-Q4YEI_C1915jtpllC2SPFFks5twFYbgaJpZM4Tz_4Y
.

I like the idea of createObserableContext and using a version of what @jantimon provided. But a question here from @mweststrate. Is <Observer /> component going to be deprecated? I've seen a few comments about it in the source code e.g. https://github.com/mobxjs/mobx-react/blob/master/src/observer.js#L410
This is the helper function I am using:
export function createObserableContext<T>(defaultValue: T, calculateChangedBits?: (prev: T, next: T) => number): React.Context<T> {
  const Context = React.createContext<T>(defaultValue, calculateChangedBits);
  return {
    Provider: Context.Provider,
    Consumer: (props: React.ConsumerProps<T>) => (
      <Context.Consumer>{(value: T) => <Observer>{() => props.children(value)}</Observer>}</Context.Consumer>
    )
  }
}

No
Op wo 27 jun. 2018 21:30 schreef Ali Malekpour<EMAIL_ADDRESS>
I like the idea of createObserableContext and using a version of what
@jantimon https://github.com/jantimon provided. But a question here
from @mweststrate https://github.com/mweststrate. Is
component going to be deprecated? I've seen a few comments about it in the
source code e.g.
https://github.com/mobxjs/mobx-react/blob/master/src/observer.js#L410
This is the helper function I am using:
export function createObserableContext(defaultValue: T, calculateChangedBits?: (prev: T, next: T) => number): React.Context {
const Context = React.createContext(defaultValue, calculateChangedBits
);
return {
Provider: Context.Provider,
Consumer: (props: React.ConsumerProps) => (
<Context.Consumer>{(value: T) => {() => props.children(value)}}</Context.Consumer>
)
}
}
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mobxjs/mobx-react/issues/471#issuecomment-400801883,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABvGhIpZ-93qS2vfcVBwcULwq2Jfdy0Hks5uA91ZgaJpZM4Tz_4Y
.

Not sure if it belongs to this package, there is a more concise context
mechanism already in the form of Provider / inject, and two mechanisms to
offer the same thing is often confusing for others

yep it might be confusing for beginners, but IMHO the helper @jantimon created does indeed belong to that package.
It's fundamentally different from inject.  Context on the other hand is dynamically applied in your tree based on what you provide it with.
I'll open up a PR so we can maybe discuss there. I'd love for @urugator to also express how he feels about the createObserableContext helper.

I think the actual problem is the original Consumer API. It doesn't compose very well and it's somewhat cumbersome to use. I think I have read somewhere that we're supposed to extract Provider/Consumer logic into seperate components/HOCs so that our "normal" components doesn't depend on context directly, which makes sense from reusability and testability perspective I guess.
So we end up with something like ThemeProvider/WithTheme(Something). Such design would limit the problems with observers I presume. However it can be PITA if you use context extensively.
I have personally ended up using this abstraction:
consumer({
  prop1: Context1,
  prop2: Context2,
})(MyComponent);

The impl for anyone interested:
function consumer(ctxs) {
  return (component) => {
    const keys = Object.keys(ctxs);
    const consume = keys.reduceRight((inner, key) => {
      return props => React.createElement(ctxs[key].Consumer, null,
        value => inner({
          ...props,
          [key]: value,
        }),
      )
    },
      (props) => React.createElement(component, props)
    )

    function Consumer(props) {
      return consume(props);
    }

    Consumer.displayName = `Consumer(${component.displayName || component.name || 'Component'})`;

    return Consumer;
  }
}

That being said I think this will become no-issue with hooks as they make consuming context much easier.
So I would personally wait for hooks before introducing any workarounds in mobx.
About createObservableContext itself:
I am not sure about the name - it doesn't make anything observable, it makes part of the context - the consumer - observer.
I don't like that it introduces abstraction over the whole context object (technically it's no longer context), while we only need a consumer (I see it's convenient, but it just seems wrong to me).


tf.contrib.data.Dataset outputs have only partial shape

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04.2
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 1.3.0
Python version: 3.6
Bazel version (if compiling from source): N/A
CUDA/cuDNN version: 8.0 / 6
GPU model and memory: Titan X 12GB
Exact command to reproduce: see below

Describe the problem
Dataset outputs should have the first shape dimension specified when dataset.batch is specified. Instead, "?" is given.
Source code / logs
import tensorflow as tf

dataset = tf.contrib.data.Dataset.range(100).map(lambda x: [x,x])
dataset = dataset.batch(32)
iterator = dataset.make_initializable_iterator()
inputs = iterator.get_next()
print(inputs.shape)

Output: (?, 2)
Expected: (32, 2)

This is intended behavior: if you evaluate inputs 4 times, you will notice that the shapes of the returned tensors are (32, 2), (32, 2), (32, 2), and (4, 2).

@mrry is there a way to prevent the dataset from producing the final batch which has incomplete batch size (in your example, the (4,2))?

Yes! In TF 1.4 (just RC'd yesterday) you can replace:
dataset = dataset.batch(batch_size)

dataset = dataset.apply(tf.contrib.data.batch_and_drop_remainder(batch_size))

I've been struggling with this problem.
I've put
dataset = dataset.repeat()
before
dataset = dataset.batch(batch_size)
and expected to see fully defined TensorShape.

@tomwesolowski Did you try the workaround I suggested in my previous comment?

@mrry Yes and it worked. But my question is if it's a intended behaviour?

@mrry I have struggle with this problem
first, when i deal with dim = reshape.get_shape()[1].value     weights = _variable_with_weight_decay('weights', shape=[dim, 384],                                           stddev=0.04, wd=0.004)

Thank you to @mrry for the workaround!
I would like to point out that this also affects the more dynamic model definitions (not just the last batches of training), for example when modifying the example models/research/resnet/resnet_model.py to use a Dataset instead of the queues, building the model will choke on the fully connected layer at resnet_model.py#L290.

Hi, "InvalidArgumentError: ConcatOp : Dimensions of inputs should match" , I meet the same problem, Did you have some solutions? thanks!!!

Thanks @mrry , I used the drop_remainder=true flag and it fixed this issue.


ignore collection count

Thanks for the pull request submission.  This implementation is going to confuse people later.  If the counts are different and one list is longer than the other then the longer list items will not be compared.  Please make a IReadOnlyCollectionComparer and ignore the count when CompareReadOnly == false

I am thinking that maybe the IReadOnlyCollection would not be compared at all if CompareReadOnly == false

I just realised that your last comment suggests ignoring the entire collection, but I am hoping to have the functionality of just ignoring the Count property specific, do you have any suggestions for doing that?

Now this is just a defect that is specific to a read only collection.
What I would do is simply this:

Set IgnoreCollectionOrder = true
Just not use the count property in your output:

var differencesWithoutCount = Differences.Where(o => o.ChildPropertyName != "Count" && !o.Object1TypeName.Contains("ReadOnlyCollection"));
string result = differencesWithoutCount.Aggregate(string.Empty, (current, difference) => current + (difference + Environment.NewLine));

Ye that is my current approach, but I'm not  a huge fan because I then need to specify the same for every type of collection I might be using in my classes
I just thought that it made sense to exclude Count when setting CompareReadOnly = false, since Count is in fact a readonly property
I can always just add the filter for any collection type I happen to be using

Updated the PR to simply ignore REadOnlyCollections when CompareReadOnly = true

Deployed:  https://www.nuget.org/packages/CompareNETObjects/4.77.0


Request: connected underscores __

BEM web naming convention uses a lot of double underscores.
As of now Operator shows a little gap between two undercores. Would be nice to have it eliminated as in 7.2 Stylistic Sets Operator Mono (see “connected underscore”).

That is actually built into the font.

<!DOCTYPE html>

Without Stylistic Set 08 Enabled

      .block__elem { color: #042; }
      .block__elem--modifier { color: #420; }

With Stylistic Set 08 Enabled

      .block__elem { color: #042; }
      .block__elem--modifier { color: #420; }

The issue is that there isn't an easy to way to enable font-feature-settings directly in VS Code. There are some issues in GitHub asking for this feature.
https://github.com/Microsoft/vscode/issues/6918
https://github.com/Microsoft/vscode/issues/10435
The current workaround is to update the workbench.main.css file.

@kiliman could you please tell me what to add to worbench.main.css? I tried ↓ to no avail:
.monaco-editor { font-feature-settings:"ss08"; }

I got it to work in VS Code. I also used the Custom CSS and Javascript Loader extension. https://github.com/be5invis/vscode-custom-css
This way I didn't need to modify the main css and can add a custom user file. Follow the instructions for the extension.
Make sure you add !important.
.monaco-editor {
font-feature-settings: "ss08" !important;
}
Looks really nice

You’re messing with me right? :-)
Your code strips any stylistic sets from the original file. "ss08" only works with the original file but is pointless with the “lig” version.
Or is there any other version of your code (unmerged branch maybe)?

Ah, that could be. Try the features/contextsubst branch. I rewrote the entire table generation to support contextual substitution. It only supports Operator Mono SSm Book (and part of OM Light) currently. That's why I haven't merged to master yet, since I need to redo all the ligatures :(

Yep, that has done the trick, thanks. But maybe you could make this a default behaviour withouth the need of switching to stylistic set? That would be nice.
To avoid installing any extension: find every mention of the font-feature-settings: in the workbench.main.css and add after a comma "ss08".  I’ve reached out guys from Hoeffler. The response was, basically, no current plans for an update:

We’ve had a few other people request these as well and while we don’t have any immediate plans to add them, it’s certainly on our radar.

Added the __ ligature. Here's how it looks in VS Code. You no longer need to enable the stylistic set.

Closed by 95503e21c690ff369a5cb6d988ef6223f9e05549

Please use the develop branch. I've merged the feature branch and deleted it.


Blueprint collision is too high

Probably increase the model's y coordinates on vertices, and remove item rando correction

Thanksgiving report


[TEVA-4163] Replace leadership with senior leader

Jira ticket URL
https://dfedigital.atlassian.net/browse/TEVA-4163

Hey @cesidio, looks good but noticed one thing worth looking at.
The job role on the listing does not appear as a link when Senior Leader is selected like it does for other roles like Teacher


New code for governance program (Stations)

A submitter for Continuous Underwater Noise asked for us to add WOZEP as a program governance to https://vocab.ices.dk/?codetypeguid=dc896ec4-e236-4a4c-a900-e4bcafcf0cde
WOZEP is a Dutch national program funding ecological monitoring concerning windfarms. While it is national, it's not exactly the same as "national" in PRGOV.
https://www.noordzeeloket.nl/en/functions-and-use/offshore-wind-energy/ecology/offshore-wind-ecological-programme-wozep/

@pcrjoana to contact Pepjin and discuss using "national" instead.

Pepjin agreed to use "national".


Enable configuration of PersistentKeepalive in client configuration

https://unix.stackexchange.com/questions/714145/wireguard-not-able-to-connect-after-a-few-minutes

Thanks for the suggestion! As a workaround, you can set the Persistent Keepalive in the server config. But it will apply globally to all clients. It should be moved to a per-client config option.

am I right to say that if you said it manually in the config file for the clients that it stands to be overwritten the next time any change is made via the GUI?

Yes you are right. If you press the Save button in either the Server or Client config, it will rewrite the WireGuard files according to what it thinks they're supposed to be.

Hey @brandonros, this feature should be added in the latest release, v2.0.11. Feel free to check it out and let me know if this works for you.

@micahmo thank you for all of the hard work you do for the community. will test and report back if there are any issues. highly appreciate it.

@micahmo hey, sorry to bother you. i'm 99% sure on latest release this doesn't work as expected 100%. in clients_wg folder the [Peer] is missing PersistentKeepalive = 25 for example and it gets overridden everytime a change is made in the UI

https://github.com/micahmo/WgServerforWindows/blob/55e9197cfeb9f60c6def32ff268ddbc97916e20a/WgServerforWindows/Models/ClientConfigurationsPrerequisite.cs#L238
If I had to guess it was getting dropped there


If user selects only one MOS instrument then emchain will fail

At the XMM mission level, if only one MOS is selected, and then emchain is run, this is the result:
ValueError                                Traceback (most recent call last)
Cell In [10], line 1
----> 1 emchain(arch)
File ~/code/DAXA/daxa/process/xmm/_common.py:230, in sas_call..wrapper(*args, **kwargs)
227 obs_archive: Archive  # Just for autocomplete purposes in my IDE
229 # This is the output from whatever function this is a decorator for
--> 230 miss_cmds, miss_final_paths, miss_extras, process_message, cores, disable, timeout = sas_func(*args, **kwargs)
232 # Converting the timeout from whatever time units it is in, to seconds - but first checking that the user
233 #  hasn't been daft and passed a non-time quantity
234 if timeout is not None and not timeout.unit.is_equivalent('s'):
File ~/code/DAXA/daxa/process/xmm/assemble.py:232, in emchain(obs_archive, process_unscheduled, num_cores, disable_progress, timeout)
225 miss_extras[miss.name] = {}
227 # This method will fetch the valid data (ObsID, Instrument, and sub-exposure) that we need to process. When
228 #  given M1 (for instance) as a search term only MOS1 identifiers will be returned. As emchain needs to
229 #  process MOS1 and MOS2 data, I just run it twice and add the results together
230 # I prefer this to how I originally wrote this, as it saves multiple layers of for loops/if statements, which
231 #  can be a little tricky to decipher
--> 232 rel_obs_info = obs_archive.get_obs_to_process(miss.name, 'M1') + obs_archive.get_obs_to_process(miss.name, 'M2')
234 # Don't just launch straight into the loop however, as the user can choose NOT to process unscheduled
235 #  observations. In that case we clean that rel_obs_info list.
236 if not process_unscheduled:
237     # Select only those sub exposures that have an ident that doesn't start with a U
File ~/code/DAXA/daxa/archive/base.py:984, in Archive.get_obs_to_process(self, mission_name, search_ident)
980     # I try to check that the search_ident value is legal - either an instrument for the selected mission, or
981     #  an ObsID
982     if search_ident not in self._missions[mission_name].filtered_obs_ids
983             and search_ident not in self._missions[mission_name].chosen_instruments:
--> 984         raise ValueError("The passed search ident ({si}) should be either an ObsID associated with the "
985                          "mission, or a mission instrument enabled for this archive; otherwise it must be"
986                          "set to None.".format(si=search_ident))
988 # Using dict_search and iterating (as it as a generator) - I know this method isn't super well explained, but
989 #  in theory it should work for _use_this_obs which are upto four levels deep (mission-obsid-inst-subexp), and
990 #  support the other possible depth where there aren't subexposures. Its disgusting and janky but I can't be
(...)
993 #  steps is to construct a list of lists of [ObsID, Instrument, SubExp] that CAN be used, and the return
994 #  from this function can be iterated through.
995 for res in dict_search(search_ident, rel_use_obs):
996     # This will catch when the dictionary is three deep (ObsID-Instrument-SubExp) and the search_ident was
997     #  on the second level
ValueError: The passed search ident (M1) should be either an ObsID associated with the mission, or a mission instrument enabled for this archive; otherwise it must beset to None.

This should be sorted at every point now, as well as the similar problems with selecting only a subset of the expected instruments for XMM


Updated langpacks to be pushed in

This PR adds the current langpack source files that I have verified to work on ADS in the current build, currently the vast majority of the core strings and bundled extensions are translated with the exception of strings that are hardcoded in ADS, are not currently translated/added to the English XLF, or are carbon edits to vscode files.
These are only a few strings though..
There is currently a bug where the report issue window from vscode is currently not localized, even though the paths are correct in the lang packs and the vscode version of the lang pack works fine. This is not an issue with the lang packs, as other strings from vscode are translated fine.
This is to be pushed in conjunction with #15489 to create the langpacks.

Pull Request Test Coverage Report for Build 857901106

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 43.411%

Totals

Change from base Build 857808873:
0.01%

Covered Lines:
26555

Relevant Lines:
55643

💛  - Coveralls


3.17.2 Release

@justinedelson @ireasor @bstopp @badvision - Any issue w/ me releasing 3.17.2 now w/ whats in master?

Too late now! released!


Freshly created table is not recognized by DBeaver

open DBeaver and ER-diagram view
create a table outside of DBeaver
refresh ER-diagram

Table of step 2 is not found. You have to refresh tables in database navigator manually and then you can refresh DBeaver table.
System: DBeaver 3.8.0 with MariaDB.

This was made on purpose - diagram refresh doesn't refresh metadata but just re-render diagram.

I'll make active object refresh. It won't make much sense in case of individual tables diagram (because referenced tables won't be refreshed). But this can be used in schema diagrams.


health: calculate distance from steps

multiplies step counts with stride length
distance menu under step counting is only available if stride length is
set

TODO: interface.html

Should we make step length global setting? At least "pebble clock with distance" and "sixths" depend on that value already.

ok, I was not aware of that, this brings back the "My Profile" idea.
@hughbarney How far are you with this?

thyttan marked this pull request as ready for review 1 hour ago

uhm why?

Sorry, don't know how that happened!

I have added an initial version of "myprofile" to store personal data and use that instead.

I'm done for now.

Since "health" is installed by default I assume @gfwilliams  wants to review this personally.

This looks great! Thanks for your work on this!
I guess at some point in the future we could have an interface.html so it can be set from the app loader too, but this seems great for now.
... also moving step goal into this app would be great! It's currently in the health app (but also some pedometers) I think and would be really good to move to one place.
Merging now...


Use baseurl property for css

Without this change the template doesn't work for baseurls
that aren't at the root of the domain. E.g.
'http://example.com/foo/bar' leads to a 404 looking for
http://example.com/css/style.css.

@srsudar merged,thanks 👍


The following asset(s) exceed the recommended size limit

To reproduce, run yarn build-storybook. There are two asset size limit warnings that can impact web performance.
yarn build-storybook
yarn run v1.15.2
$ build-storybook -c .storybook -o .out
info @storybook/react v5.0.6
info
info clean outputDir..
info => Copying prebuild dll's..
info => Building manager..
info => Loading manager config..
info => Loading presets
info => Compiling manager..
info => manager built (12 s)
WARN asset size limit: The following asset(s) exceed the recommended size limit (244 KiB).
WARN This can impact web performance.
WARN Assets:
WARN   vendors~main.587e785d3b39ea07a8b4.bundle.js (1.19 MiB)
WARN entrypoint size limit: The following entrypoint(s) combined asset size exceeds the recommended limit (244 KiB). This can impact web performance.

WARN For more info visit https://webpack.js.org/guides/code-splitting/
info => Building preview..
info => Loading preview config..
info => Loading presets
info => Loading custom webpack config (extending mode).
info => Using base config because react-scripts is not installed.
info => Compiling preview..
 10% building 1/1 modules 0 active(node:1679) DeprecationWarning: Extend-mode configuration is deprecated, please use full-control mode instead.

See https://storybook.js.org/docs/configurations/custom-webpack-config/#full-control-mode
info => Preview built (19 s)
WARN asset size limit: The following asset(s) exceed the recommended size limit (244 KiB).
WARN This can impact web performance.
WARN Assets:
WARN   vendors~main.233f5970e9ab16c4055e.bundle.js (1.66 MiB)
WARN   iframe.html (385 KiB)
WARN entrypoint size limit: The following entrypoint(s) combined asset size exceeds the recommended limit (244 KiB). This can impact web performance.

WARN For more info visit https://webpack.js.org/guides/code-splitting/

Solution: implement Code Splitting.

Hi, I'm trying and trying to eliminate the size warnings, and nothing I've tried in the main.js config is working. Can someone show an example in the 'new' and better main.js config?
//.storybook/main.js
const path = require('path')
const BundleAnalyzerPlugin = require('webpack-bundle-analyzer').BundleAnalyzerPlugin
module.exports = {
stories: [
'../src//*.stories.mdx',
'../src//.stories.@(js|jsx|ts|tsx)',
'../src/components/**/.stories.js',
],
addons: [
'@storybook/addon-links',
'@storybook/addon-essentials',
'@storybook/preset-scss',
'@storybook/theming',
'@storybook/addon-storysource',
'@storybook/addon-notes/register',
'@etchteam/storybook-addon-status/register',
'@whitespace/storybook-addon-html',
],
presets: ['storybook-addon-deps/preset'],
webpackFinal: async (config, { configType }) => {
// configType has a value of 'DEVELOPMENT' or 'PRODUCTION'
// You can change the configuration based on that.
// 'PRODUCTION' is used when building the static version of storybook.
    // Make whatever fine-grained changes you need
    config.module.rules.push({
        test: /\.scss$/,
        use: ['style-loader', 'css-loader', 'sass-loader'],
        include: [path.resolve(__dirname, './')],
    })

    // analyze webpack bundle size. To enable run storybook with `cross-env analyze=true`
    // not needed, but helps debug the problem :)
    const shouldAnalyze = process.env.analyze === 'true'
    if (shouldAnalyze) {
        config.plugins.push(new BundleAnalyzerPlugin())
    }
    // split into more chunks
    config.optimization = {
        splitChunks: {
            chunks: 'async',
            minSize: 30 * 1024, // 30KB
            maxSize: 64 * 1024, // 1MB
        },
    }
    // Return the altered config
    return config
},

}

Hi, I'm trying and trying to eliminate the size warnings, and nothing I've tried in the main.js config is working. Can someone show an example in the 'new' and better main.js config?
//.storybook/main.js
const path = require('path')
const BundleAnalyzerPlugin = require('webpack-bundle-analyzer').BundleAnalyzerPlugin
module.exports = {
stories: [
'../src//*.stories.mdx',
'../src//.stories.@(js|jsx|ts|tsx)',
'../src/components/**/.stories.js',
],
addons: [
'@storybook/addon-links',
'@storybook/addon-essentials',
'@storybook/preset-scss',
'@storybook/theming',
'@storybook/addon-storysource',
'@storybook/addon-notes/register',
'@etchteam/storybook-addon-status/register',
'@whitespace/storybook-addon-html',
],
presets: ['storybook-addon-deps/preset'],
webpackFinal: async (config, { configType }) => {
// configType has a value of 'DEVELOPMENT' or 'PRODUCTION'
// You can change the configuration based on that.
// 'PRODUCTION' is used when building the static version of storybook.
    // Make whatever fine-grained changes you need
    config.module.rules.push({
        test: /\.scss$/,
        use: ['style-loader', 'css-loader', 'sass-loader'],
        include: [path.resolve(__dirname, './')],
    })

    // analyze webpack bundle size. To enable run storybook with `cross-env analyze=true`
    // not needed, but helps debug the problem :)
    const shouldAnalyze = process.env.analyze === 'true'
    if (shouldAnalyze) {
        config.plugins.push(new BundleAnalyzerPlugin())
    }
    // split into more chunks
    config.optimization = {
        splitChunks: {
            chunks: 'async',
            minSize: 30 * 1024, // 30KB
            maxSize: 64 * 1024, // 1MB
        },
    }
    // Return the altered config
    return config
},

}

duplicity https://github.com/storybookjs/storybook/issues/6391


chore(weave): Inline pydantic and dataclass objects

Resolves:

https://wandb.atlassian.net/browse/WB-20357

closing in favour of: https://github.com/wandb/weave/pull/2147


research other LLM methods

User story

As a Product Owner
I want to know possible methods for document search using LLMs
So that we can decide which methods to explore and implement

Acceptance criteria
documentation for each method containing the following information or estimations as bullet points is created

short method description
advantages of the method (information security, privacy, price and quality of response)
disadvantages of the method (information security, privacy, price and quality of response)
comments
links / sources

Definition of done (DoD)

Acceptance criteria are met.
Work products are uploaded to the Github repository.
A pull request is created for each related branch.
The work products in the pull requests are reviewed.
The corresponding branches are merged and closed.
The bill of materials section of the planning documents is updated.
All defined conventions are complied with.

DoD general criteria

Feature has been fully implemented
Feature has been merged into the mainline
All acceptance criteria were met
Product owner approved features
All tests are passing
Developers agreed to release

not relevant any more


Fix gcs build issue when vcpkg build is enabled

Fixes #6003.

Please pick this commit to bump the google-cloud-cpp version in vcpkg
https://github.com/zhouyuan/gluten/commit/36b9a50799520e630feac677194face49ad72661

@zhouyuan, thanks so much for your help. Will merge the patch soon.


Feature request: add a severity threshold to the .env file

Context
Many organizations' security teams are outnumbered by developers, sometimes as drastically as a 1:100 ratio. Also, different organizations have different risk appetites when it comes to dismissing security alerts (e.g. focus security review on critical & high severity alerts, over medium & low severity alerts). This makes it undesirable to review the dismissal of every alert as opposed to just those that surpass a given severity threshold.
Suggestion
Allow for an optional parameter in the .env file to specify the severity threshold that requires scan-manager dismissal. For example:
SEVERITY_THRESHOLD=high would reopen only those alerts dismissed by someone other than a scan-manager that have a severity of high or critical. Not specifying a SEVERITY_THRESHOLD would reopen alerts of any severity.
cc @kenmuse

Planning to include this in the next version. I'll include the following settings:

SECURITY_ALERT_CLOSE_TEAM: The original setting and the default for all of the values. Default value is scan-managers
DEPENDABOT_APPROVER_TEAM: If provided, requires Dependabot issues to be closed by a member of this team
DEPENDABOT_SEVERITY: Anything over this level requires the team approval. Defaults to All. It uses the higher value of security_advisory.severity or security_vulnerability.severity (although they appear to always be the same)
CODE_SCANNING_APPROVER_TEAM: If provided, requires Code Scanning issues to be closed by a member of this team
CODE_SCANNING_SEVERITY: Anything over this level requires the team approval. Defaults to All.
SECRET_SCANNING_APPROVER_TEAM: If provided, requires Code Scanning issues to be closed by a member of this team

The severity levels will be case-insensitive and accept any of the following:

None: Disables approvals
All: The default. All alerts must be approved in order to be closed.
Note: Note (and higher) alerts must be approved. Code-scanning only.
Warning: Warning (and higher) alerts must be approved. Code-scanning only.
Error: Error (and higher) alerts must be approved. Code-scanning only.
Low: Low (and higher) alerts must be approved
Medium: Medium (and higher) alerts must be approved
High: High (and higher) alerts must be approved
Critical: Critical (and higher) alerts must be approved


parallel_nn related crash

If I turn on --parallel_nn, the translation demo (./train.sh) will crash:
#0  __memcpy_sse2_unaligned () at ../sysdeps/x86_64/multiarch/memcpy-sse2-unaligned.S:33
#1  0x0000000000740f90 in paddle::BaseMatrixT<float>::assign(paddle::BaseMatrixT<float>&) ()
#2  0x00000000005b4fb5 in paddle::GruStepLayer::forward (this=0x16875e80, passType=<optimized out>) at /home/byzhang/src/github/Paddle/paddle/gserver/layers/GruStepLayer.cpp:98
#3  0x000000000062639e in paddle::NeuralNetwork::forward (this=0x16866ac0, inArgs=..., outArgs=0x7fff86369b10, passType=paddle::enumeration_wrapper::PASS_TRAIN) at /home/byzhang/src/github/Paddle/paddle/gserver/gradientmachines/NeuralNetwork.cpp:242
#4  0x00000000006164af in paddle::RecurrentGradientMachine::forward (this=<optimized out>, inArgs=..., outArgs=<optimized out>, passType=paddle::enumeration_wrapper::PASS_TRAIN) at /home/byzhang/src/github/Paddle/paddle/gserver/gradientmachines/RecurrentGradientMachine.cpp:546
#5  0x000000000057257a in paddle::RecurrentLayerGroup::forward (this=<optimized out>, passType=<optimized out>) at /home/byzhang/src/github/Paddle/paddle/gserver/layers/RecurrentLayerGroup.cpp:42
#6  0x0000000000621854 in paddle::ParallelThread::computeThread (this=0x168c59b0) at /home/byzhang/src/github/Paddle/paddle/gserver/gradientmachines/ParallelNeuralNetwork.cpp:174
#7  0x00007ffff6486c30 in ?? () from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#8  0x00007ffff78c2184 in start_thread (arg=0x7fff8636a700) at pthread_create.c:312
#9  0x00007ffff5c0e37d in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:111

The commit is https://github.com/byzhang/Paddle/commit/de9d6c6f39ed8068ffb30e5dde7e3eda6ed889bd
I tried to fix the aforementioned crash, but it crashed somewhere else related with unassigned realLayer_ of ScatterAgentLayer. So I need your help on debugging the crash.

Thanks for your attention, we repeat above crash on our machine, and are debugging now.

👍

I look at the commit byzhang@de9d6c6, there is a doubt. Do you just want to use GPU training? If so, just configure --use gpu = true and --gpu_id = 0 on the command line. Do not need parallel_nn and default_device(0) in the config.
PS: translation demo use RecurrentGradientMachine and RecurrentGradientMachine is not support --parallel_nn=true, this is why paddle run crash.

I am building another network which uses both gru and parallel_nn. But to
make it simpler, I repro the crash using translation demo as an example.
What's the key reason why RecurrentGradientMachine doesn't support
parallel_nn?
On Oct 25, 2016 2:29 AM, "hedaoyuan"<EMAIL_ADDRESS>wrote:

I look at the commit byzhang@de9d6c6
https://github.com/byzhang/Paddle/commit/de9d6c6, there is a doubt. Do
you just want to use GPU training? If so, just configure --use gpu = true
and --gpu_id = 0 on the command line. Do not need parallel_nn and
default_device(0) in the config.
PS: translation demo use RecurrentGradientMachine and
RecurrentGradientMachine is not support --parallel_nn=true, this is why
paddle run crash.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/baidu/Paddle/issues/241#issuecomment-255984587, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA1O1S3k4NVgjj2mr4X5mpZZMg86wvu1ks5q3cvigaJpZM4KfbuU
.

At the moment, should change ParallelNeuralNetwork to report error when recurrent_nn is used. In the longer term, should consider make ParallelNeuralNetwork support recurrent_nn.

The key reason is that ParallelNeuralNetwork design earlier, RecurrentGradientMachine design in the post. They are each in order to solve the training of different models. However, compatibility between the two is not considered. ParallelNeuralNetwork pays more attention to using different computing devices to train a model(like CPU+GPU, multi-GPU..). RecurrentGradientMachine pays more attention to how to train the RNN model.
PS: At present, we will follow emailweixu's comments, add some checks in the code.


js: isso.js: Disable Postbox submit button on click, enable after response

Checklist

[x] All new and existing tests are passing
[x] I have added an entry to CHANGES.rst because this is a user-facing change or an important bugfix
[x] I have written proper commit message(s)

What changes does this Pull Request introduce?
Disable the submit button in Postbox to prevent double posting upon click. Enable the button after receiving a response from the API endpoint.
Why is this necessary?
Fixes https://github.com/posativ/isso/issues/913

Also, you have 3 identically named commits with the same commit body.

Is there something missing that prevents you from removing the draft status?

Is there something missing that prevents you from removing the draft status?

I've noticed that E2E tests are failing due to timeout on checks, and it's happening in different places. I thought it was due to changes in PR and was looking for a solution.
So for now, I'm marking PR as a draft to look for a solution.

I hope the usability of the screenshot tests wasn't too bad for you. I found them useful at the time when I created them because we were messing up some of the CSS/layout with a lot of attempted cleanups.
At least the tests don't seem to be flaky any more ;)

I hope the usability of the screenshot tests wasn't too bad for you. I found them useful at the time when I created them because we were messing up some of the CSS/layout with a lot of attempted cleanups.
At least the tests don't seem to be flaky any more ;)

It took me a little while to get the hang of them, but I found them very helpful.
Thank you for creating them. They're definitely useful for moving forward!


RateController: drop deferred fetch of image

Hi,
I keep having the following warning message in Apache log for every few minutes
RateController: drop deferred fetch of https://[IP]/images/01.jpg on shutdown
I enable mod_speedpage for one of my virtual domains on the server so far,
whereas the image is not accessable from the IP base path because of web server setting, but it available on domain base path like https://[VIRTUAL_DOMAIN_NAME]/images/01.jpg
It looks like mod_speedpage try to get the image for optimize from the server IP itself.
And my question is how can I configure mod_speedpage to access the image in domain base path instead of IP path or use local path.
Thank you very much.
Anthony

Hi
What´s the value of ModPagespeedRewriteRandomDropPercentage Percent?
Percent is a value between 0 and 100.
https://www.modpagespeed.com/doc/config_filters#RewriteRandomDropPercentage

Hi
The parameter is disabled in my conf file
#ModPagespeedRewriteRandomDropPercentage 90

Hi
At the moment I can't have access to the GitHub web, so I send this email
directly.
Uncomment and change the 90 to -1, no rate limits.
I think that the parameter have a default value, so you must change to
overwrite the default.
El lun., 12 ago. 2019 6:59, Atycyber<EMAIL_ADDRESS>escribió:

Hi
The parameter is disabled in my conf file
#ModPagespeedRewriteRandomDropPercentage 90
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-pagespeed-mod/issues/1937?email_source=notifications&email_token=AFKOZCICVHEY637EWIYLYOLQEDU4HA5CNFSM4IKRG532YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4BSHIQ#issuecomment-520299426,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFKOZCLMSPJ57CGIHUGV5ETQEDU4HANCNFSM4IKRG53Q
.

Hi Lofesa,
Thanks for your suggestion, I changed the setting to be -1 and monitor it for a couple of hours,
the warning message still appending to log file quite frequently. If this is not serious message how can I turn it off or ignore warning message?

Hi @Atycyber
Sorry for coming late, but I was out on holidays.
Can you post or attach you config (pagespeed and apache)?

HI Lofesa,
Sorry for my very late reply and I was back from business trip.
The pagespeed conf:
<IfModule !mod_version.c>
LoadModule version_module /usr/lib64/httpd/modules/mod_version.so

<IfVersion < 2.4>
LoadModule pagespeed_module /usr/lib64/httpd/modules/mod_pagespeed.so

= 2.4.2>
LoadModule pagespeed_module /usr/lib64/httpd/modules/mod_pagespeed_ap24.so

<IfModule !mod_deflate.c>
LoadModule deflate_module /usr/lib64/httpd/modules/mod_deflate.so

ModPagespeed on
ModPagespeedInheritVHostConfig on

AddOutputFilterByType MOD_PAGESPEED_OUTPUT_FILTER text/html

ModPagespeedFileCachePath "/var/cache/mod_pagespeed/"

ModPagespeedLogDir "/var/log/pagespeed"

ModPagespeedSslCertDirectory "/etc/pki/tls/certs"
ModPagespeedSslCertFile /etc/pki/tls/cert.pem

ModPagespeedCreateSharedMemoryMetadataCache "/var/cache/mod_pagespeed/" 51200
ModPagespeedEnableFilters prioritize_critical_css
ModPagespeedEnableFilters defer_javascript
ModPagespeedEnableFilters sprite_images
ModPagespeedEnableFilters collapse_whitespace,remove_comments
ModPagespeedEnableCachePurge on

ModPagespeedEnableFilters lazyload_images
ModPagespeedEnableFilters rewrite_images
ModPagespeedEnableFilters responsive_images,resize_images

ModPagespeedFetchHttps enable

ModPagespeedDisallow "*/wpcallback.php"
ModPagespeedFileCacheInodeLimit        500000

ModPagespeedRewriteRandomDropPercentage -1
ModPagespeedPrivateNotVaryForIE on
ModPagespeedMaxSegmentLength 250
<Location /pagespeed_admin>
    Order allow,deny
    Allow from localhost
    Allow from <IP_ADDRESS>
    Allow from XXX.XXX.XX.XX
    SetHandler pagespeed_admin
</Location>
<Location /pagespeed_global_admin>
    Order allow,deny
    Allow from localhost
    Allow from <IP_ADDRESS>
    Allow from XXX.XXX.XX.XX
    SetHandler pagespeed_global_admin
</Location>

ModPagespeedStatisticsLogging on

ModPagespeedMessageBufferSize 100000

And the apache is config in standard way, and the log file is under R/W directly and only by pagespeed module so I only show the above config here.
Thanks a lot for review.

Hi @Atycyber
My bad, the ModPagespeedRewriteRandomDropPercentage parameter must be set to 0 to not discard any rewrite, valid values are from 0 to 100.

HI Lofesa,
OK, I change to 0 for the parameter now, will let it keep running for 1-2 days, get back to you about the result later

Hi Lofesa,
Just checked the log, it looks like the same to the old setting. It keeps writing the same warning to log for every 2-3 minutes. :-(

Hi @Atycyber
Maybe you have a redirect/rewrite from [IP] to [SUB-DOMAIN]? In the html code the requests are https://[VIRTUAL_DOMAIN_NAME]/images/01.jpg ?
Try to auth the domain:
ModPagespeedDomain http*://*:example.com
As stated in the doc, pagespeed try to fecht resources that belong to the auth domain it found in the html.


IPC doesn't work?

hyprctl hyprpaper preload wallpaper.png
hyprctl hyprpaper wallpaper ,wallpaper.png

And the wallpaper doesn't change, it keep displaying the initial one which is set in the config
Here's hyprpaper output after issuing those two commands:
[LOG] Accepted incoming socket connection request on fd 5
[LOG] Received a request: preload wallpaper.png
[LOG] Preloaded target wallpaper.png in 40.26ms -> Pixel size: [1920, 1080]
[LOG] Buffer created for target wallpaper.png, Shared Memory usage: 8.3MB
[LOG] Total SM usage for all buffers: 16.6MB
[LOG] Accepted incoming socket connection request on fd 5
[LOG] Received a request: wallpaper ,wallpaper.png

hyprpaper 0.4.0

Only thing I can note of in my setup is that initial image is symlinked
But I don't see why that would be a problem, it shows real path in the logs, so it seems that it's resolved correctly

cwd isn't shared, use an absolute path.

Thanks, absolute path works


[7.0] Remove node 0.10 and 0.12 from travis

We are going to drop the support for node 0.10 and 0.12 in version 7.0.0 and a branch for 7.0 does already exist here https://github.com/babel/babylon/tree/7.0.
We can now safely remove node 0.10 and 0.12 from .travis.yml. Also the whole custom logic with nvm in this file can be removed. It was added because rollup only supports node 4 and up.
https://github.com/babel/babylon/blob/7.0/.travis.yml#L11-L14
https://github.com/babel/babylon/blob/7.0/.travis.yml#L20-L21
PR welcome :)

What about Node 5 which is also deprecated?

I think we can close this issue :)


Fix thread race condition found with Thread Sanitizer

While running KSCrash in my application using Thread Sanitizer, it reported several thread race conditions in three files (mostly related to accessing some global static variables in the files).
This PR solves those issues by adding locking through pthread mutexes around the variables that were reporting the issues. It is probably done in a too gross way, and probably a more granular solution can be found.
Similar issues might also be happening in other files of KSCrash, but Thread Sanitizer only reports on exercised code, and my usage of the library is not very high.
With the changes in this PR my app can run under thread sanitizer without any issues.

Unfortunately, we can't use locks here because the app could crash after entering a critical section but before unlocking the mutex. When the app in the crashed state tries to read the thread list, for example, it will deadlock.
In general, the unsafety of this code shouldn't be an issue, because only one thread should be triggering the code (the main thread).


Limit SQL Server memory to half of the containers memory limit

While checking some memory issues with my container I discovered that the SQL Server does not have any memory limit. This can lead to problems where the SQL Server will consume all memory from the container.
If you pass a memory limit for the container, SQL Server memory limit will be set to half of the limit of the container. Up to a maximum of 10G. If no container memory limit is passed, SQL Server will not be limited either.
PS: I do not unterstand why there are changes without anything changed.
The relevant changes are line number 1134 and 1475 to 1503

Thanks a lot for this contribution.
I probably won't include it as is - I have learned that every time I change something fundamental, some pipeline somewhere in the world breaks.
I am considering how I can release things like this without adding parameters and without breaking people.

This is perfectly okay for me. I just think that an unlimited SQL Server can lead to problems and its best practice to set a limit.
I wanted to provide a solution which works as far as i can tell. If you can reuse any of this I am okay if it wont be merged as it is 👍

This was included.
I learned afterwards though that SQL Express can maximum take 1Gb of memory, so this is really only for limiting SQL below that.


webpack minified bundle

I wanted to use apollo-local-query as a dependency from create-react-app, but since apollo-local-query exports ES6 code, npm run build runs into this error: https://github.com/facebookincubator/create-react-app/issues/984. So I repackaged apollo-local-query based on react-webpack-starter, to export a transpiled minified bundle file.
Now npm run build works, see it deployed: http://cdetrio.github.io/apollo-graphiql/.

I'm not sure I understand the use case here- why would you need this module client-side?

The use case I have in mind is experimenting with custom resolvers or graphql adapters to back-end data sources, e.g. a client-side REST wrapper.
Btw, webpack complained about the debug module so its commented out.

Ok, makes sense. This is a pretty small module, focused on server-side applications, so I don't think this PR fits the direction of the project. However please keep your branch up on github; I'm sure it'll be a useful reference for anyone who is trying to build something similar.


Update date issued metadata tag label

Story
Updates metadata tag label for date_issued to display 'Date' on show and index pages.
Refs

#169

Expected Behavior Before Changes
Date issued metadata tag displays 'Date issued'
Expected Behavior After Changes
Date issued metadata tag displays 'Date'
Screenshots / Video

Before

After

Notes


Cannot override kami address in 2.6.x

Describe the bug
Lmctl v2.6.0/2.6.1 assume Kami is on the same host as TNCO API gateway, this is not always true in OCP environments.
Proposed Fix
Add configuration option for lm environments to set the kami address

Already fixed on master but now fixed on 2.6.x branch


[4.14] Add an OVN overlay (flat l2 topology) NAD option (#1330)

Short description:
More details:
What this PR does / why we need it:
Which issue(s) this PR fixes:
Special notes for reviewer:
Bug:

/verified

/verified

/lgtm

/verified