Remote read is failing with ‘http: multiple response.WriteHeader calls‘ in remote node

Proposal
Use case. Why is this important?
I have used shard data with three prometheus server to collect more than 2000+ nodes ,  And Use a prometheus server used remote_read as a proxy to support promsql from grafana. Architecture like follow:
Grafana  ->  Prom(with remote read)(i will call it proxy for this page)  ->  Prom( three node with shard)   ->    node-exporter
Nice to have' is not a good use case :)
Bug Report
What did you do?
I curl a url to proxy  :  http:// proxy.domain.com//api/v1/series?match[]=up
What did you expect to see?
Return all active node with their labels.
What did you see instead? Under which circumstances?
Sometimes !!! just Somethings .Got a error return ：  context deadline execeeded
Environment

System information:
X86_64 RHEL 7.4

Prometheus version:

level=info ts=2018-06-22T05:39:17.089870496Z caller=main.go:220 msg="Starting Prometheus" version="(version=2.2.1, branch=HEAD, revision=bc6058c81272a8d938c05e75607371284236aadc)"

Alertmanager version:
Do not use Alertmanager

Prometheus configuration file:

Proxy (Prom with remote_read)
global:
  scrape_interval: 15s
  scrape_timeout: 10s
  evaluation_interval: 1m

remote_read:
- url: 'http://<IP_ADDRESS>:9090/api/v1/read'
  remote_timeout: 5s
  read_recent: true
- url: 'http://<IP_ADDRESS>:9090/api/v1/read'
  remote_timeout: 5s
  read_recent: true
- url: 'http://<IP_ADDRESS>:9090/api/v1/read'
  remote_timeout: 5s
  read_recent: true

Prom ( one of the node with shard)
global:
  scrape_interval: 15s
  scrape_timeout: 10s
  evaluation_interval: 1m
  external_labels:
    slave: "<IP_ADDRESS>"

scrape_configs:
  - job_name: 'idc_ping'
    scrape_interval: 60s
    metrics_path: /probe
    params:
      module: [icmp]  #ping
    relabel_configs:
      - source_labels: [__address__]
        modulus: 3
        target_label:  __tmp_hash
        action: hashmod
      - source_labels: [__tmp_hash]
        regex: ^2$
        action: keep
        action: keep
      - source_labels: [__address__]
        regex: (.*)(:80)?
        target_label: __param_target
        replacement: ${1}
      - source_labels: [__param_target]
        regex: (.*)
        target_label: ping
        replacement: ${1}
      - source_labels: []
        regex: .*
        target_label: __address__
        replacement: <IP_ADDRESS>:9115  # Blackbox exporter.
    file_sd_configs:
      - files:
        - /app/conf/targets/idc_switch/*.json

  - job_name: 'idc_snmp'
    scrape_interval: 300s
    scrape_timeout: 30s
    metrics_path: /snmp
    params:
      module: [if_mib]
    relabel_configs:
      - source_labels: [__address__]
        modulus: 3
        target_label:  __tmp_hash
        action: hashmod
      - source_labels: [__tmp_hash]
        regex: ^2$
        action: keep
        action: keep
      - source_labels: [__address__]
        target_label: __param_target
      - source_labels: [__param_target]
        target_label: instance
      - target_label: __address__
        replacement: <IP_ADDRESS>:9116
    file_sd_configs:
      - files:
        - /app/conf/targets/idc_switch/*.json
  - job_name: 'idc_jcpu'
    scrape_interval: 300s
    scrape_timeout: 30s
    metrics_path: /snmp
    params:
      module: [JuniperCPU]
    relabel_configs:
      - source_labels: [__address__]
        modulus: 3
        target_label:  __tmp_hash
        action: hashmod
      - source_labels: [__tmp_hash]
        regex: ^2$
        action: keep
        action: keep
      - source_labels: [__address__]
        target_label: __param_target
      - source_labels: [__param_target]
        target_label: instance
      - target_label: __address__
        replacement: <IP_ADDRESS>:9116
    file_sd_configs:
      - files:
        - /app/conf/targets/idc_switch/*.json

Alertmanager configuration file:

	Do not use Alertmanager

Logs:

Proxy (Prom with remote_read)
level=info ts=2018-06-22T05:25:56.159754372Z caller=main.go:220 msg="Starting Prometheus" version="(version=2.2.1, branch=HEAD, revision=bc6058c81272a8d938c05e75607371284236aadc)"
level=info ts=2018-06-22T05:25:56.159865588Z caller=main.go:221 build_context="(go=go1.10, user=root@149e5b3f0829, date=20180314-14:15:45)"
level=info ts=2018-06-22T05:25:56.159902798Z caller=main.go:222 host_details="(Linux 3.10.0-327.el7.x86_64 #1 SMP Thu Oct 29 17:29:29 EDT 2015 x86_64 c27ad4e6d7b9 (none))"
level=info ts=2018-06-22T05:25:56.15993398Z caller=main.go:223 fd_limits="(soft=65536, hard=65536)"
level=info ts=2018-06-22T05:25:56.164939994Z caller=web.go:382 component=web msg="Start listening for connections" address=<IP_ADDRESS>:9090
level=info ts=2018-06-22T05:25:56.164954328Z caller=main.go:504 msg="Starting TSDB ..."
level=info ts=2018-06-22T05:25:56.175147161Z caller=main.go:514 msg="TSDB started"
level=info ts=2018-06-22T05:25:56.175188157Z caller=main.go:588 msg="Loading configuration file" filename=/app/conf/prometheus.yml
level=info ts=2018-06-22T05:25:56.176519009Z caller=main.go:491 msg="Server is ready to receive web requests."
level=error ts=2018-06-22T07:22:34.386435013Z caller=engine.go:529 component="query engine" msg="error selecting series set" err="error sending request: context canceled"

Prom ( node with shard)
level=info ts=2018-06-22T05:39:17.089870496Z caller=main.go:220 msg="Starting Prometheus" version="(version=2.2.1, branch=HEAD, revision=bc6058c81272a8d938c05e75607371284236aadc)"
level=info ts=2018-06-22T05:39:17.090013649Z caller=main.go:221 build_context="(go=go1.10, user=root@149e5b3f0829, date=20180314-14:15:45)"
level=info ts=2018-06-22T05:39:17.090054321Z caller=main.go:222 host_details="(Linux 3.10.0-327.el7.x86_64 #1 SMP Thu Oct 29 17:29:29 EDT 2015 x86_64 37aa86dcc533 (none))"
level=info ts=2018-06-22T05:39:17.090090703Z caller=main.go:223 fd_limits="(soft=65536, hard=65536)"
level=info ts=2018-06-22T05:39:17.094020099Z caller=main.go:504 msg="Starting TSDB ..."
level=info ts=2018-06-22T05:39:17.094025856Z caller=web.go:382 component=web msg="Start listening for connections" address=<IP_ADDRESS>:9090
level=info ts=2018-06-22T05:39:21.813964411Z caller=main.go:514 msg="TSDB started"
level=info ts=2018-06-22T05:39:21.814052973Z caller=main.go:588 msg="Loading configuration file" filename=/app/conf/prometheus.yml
level=info ts=2018-06-22T05:39:21.815212701Z caller=main.go:491 msg="Server is ready to receive web requests."
level=error ts=2018-06-22T05:54:51.526192719Z caller=stdlib.go:89 component=web msg="http: multiple response.WriteHeader calls"
level=info ts=2018-06-22T07:00:00.246050978Z caller=compact.go:393 component=tsdb msg="compact blocks" count=1 mint=1529640000000 maxt=1529647200000
level=info ts=2018-06-22T07:00:06.669934675Z caller=head.go:348 component=tsdb msg="head GC completed" duration=484.65759ms
level=info ts=2018-06-22T07:00:08.029114057Z caller=head.go:357 component=tsdb msg="WAL truncation completed" duration=1.359096044s
level=error ts=2018-06-22T07:14:30.287165508Z caller=stdlib.go:89 component=web msg="http: multiple response.WriteHeader calls"
level=error ts=2018-06-22T07:14:30.306147777Z caller=stdlib.go:89 component=web msg="http: multiple response.WriteHeader calls"
level=error ts=2018-06-22T07:14:30.326146722Z caller=stdlib.go:89 component=web msg="http: multiple response.WriteHeader calls"
level=error ts=2018-06-22T07:14:30.331301576Z caller=stdlib.go:89 component=web msg="http: multiple response.WriteHeader calls"
level=error ts=2018-06-22T07:14:30.486377958Z caller=stdlib.go:89 component=web msg="http: multiple response.WriteHeader calls"
level=error ts=2018-06-22T07:14:30.625925566Z caller=stdlib.go:89 component=web msg="http: multiple response.WriteHeader calls"
level=error ts=2018-06-22T07:14:49.192145902Z caller=stdlib.go:89 component=web msg="http: multiple response.WriteHeader calls"
level=error ts=2018-06-22T07:14:49.227607287Z caller=stdlib.go:89 component=web msg="http: multiple response.WriteHeader calls"
level=error ts=2018-06-22T07:14:49.316431716Z caller=stdlib.go:89 component=web msg="http: multiple response.WriteHeader calls"
level=error ts=2018-06-22T07:14:59.750539448Z caller=stdlib.go:89 component=web msg="http: multiple response.WriteHeader calls"
level=error ts=2018-06-22T07:15:00.04012553Z caller=stdlib.go:89 component=web msg="http: multiple response.WriteHeader calls"
level=error ts=2018-06-22T07:15:00.070867002Z caller=stdlib.go:89 component=web msg="http: multiple response.WriteHeader calls"
level=error ts=2018-06-22T07:15:00.104882808Z caller=stdlib.go:89 component=web msg="http: multiple response.WriteHeader calls"
level=error ts=2018-06-22T07:15:00.32962463Z caller=stdlib.go:89 component=web msg="http: multiple response.WriteHeader calls"
level=error ts=2018-06-22T07:15:00.335144468Z caller=stdlib.go:89 component=web msg="http: multiple response.WriteHeader calls"

Thanks for the report!
Hmm, I have tried to reproduce this by pointing a 2.2.1 Prometheus at the remote read endpoint of another 2.2.1 Prometheus, but it is working for me. Proxy config:
global:
  scrape_interval:     10s
  evaluation_interval: 10s

remote_read:
- url: "http://localhost:9090/api/v1/read"
  read_recent: true
  remote_timeout: 5s
- url: "http://demo.robustperception.io:9090/api/v1/read"
  read_recent: true
  remote_timeout: 5s

And http://localhost:9091/api/v1/series?match[]=up works...
Do you still have the problem with Prometheus 2.3.1 by the way?

Thanks for your reply
Most of the time it works, but sometimes it goes wrong, but I don't know what caused it. If so, is it possible to list some of the reasons for this error and I can help with some testing?

Ah sorry, I didn't understand earlier that http: multiple response.WriteHeader calls also only happens sometimes.
Hmm, first I thought the proxy Prometheus ran into a query timeout, but that should read context deadline exceeded rather than context canceled normally. But still something causes it to give up on the query, maybe it's still a timeout, but the error message ends up being slightly different for some reason. On the shard Prometheus, it is running this code: https://github.com/prometheus/prometheus/blob/master/web/api/v1/api.go#L592-L667
The error message indicates that we write HTTP headers twice, which can happen if you first set some headers, then start writing the body (which automatically writes out the headers first), then try to set some more headers. The only place where I currently see that could happen is in https://github.com/prometheus/prometheus/blob/master/web/api/v1/api.go#L663-L666. EncodeReadResponse() already writes out headers, but if it then fails to write the body back to the requesting Prometheus server (because it timeouted / canceled the request for some reason), then we still do http.Error() in the caller, which tries to write out headers again. To handle this more cleanly and give a better error message, we should probably differentiate the error that happens while writing back a body (https://github.com/prometheus/prometheus/blob/master/storage/remote/codec.go#L66) and not try to do an http.Error() if it happened.
Your remote timeout is pretty low with 5s btw. Have you tried setting it to something higher to see if the problem goes away?

Thank you very much for helping me analyze the reasons，But I still don't quite understand the mechanism of writing headers twice
I also think the possibility of timeout is relatively high. My current approach is to optimize the query statement and reduce the response time and content size. Moreover, I changed the timeout period on the proxy to 30s, and then observed for a period of time

Oh,yeah.
Now I know it Very clearly
Thank you very much
It works greatful after increase timeout value
I think this issue could be closed


Drop plan is not aligned with Sprint dates.

Team,
This Monday we have started the Sprint. In Drop Plan Friday date is not showing. And also observed that 2 sundays are coming/showing in Dropplan. Please find attachment 1
Further next i have created another sprint now Sprint is starting from Sunday but it suppose to start from Monday. Please find attachment 2.
Please give us a fix for this. Drop plan our teams are very widely using extension. This dates misalignment are confusing us.

We got the same concern and our version is 1.6.9 (Latest)
There are even dates that are hidden and some date which are not marked off are showing off.
This issue totally defeats the purpose of this application. I hope a resolution be submitted soon.

@AMARAVATHIPRASAD @tbauto
Is this still an issue?


Expiry time/date

When setting the expiry time / date I can only do times after 09.59 by using the up and down arrows. If I try and type 14 (for 2pm ) it defaults to 04. I also do not get a drop down and Dan does on his Browser (Firefox on Mac)
MacOS Big Sure v 11.4 / Brave Version 1.43.89 Chromium: 105.0.5195.102 (Official Build) (arm64)
and Chrome Chrome is up to date
Version 105.0.5195.102 (Official Build) (arm64)
https://user-images.githubusercontent.com/42541523/189326902-55596985-7ea3-465d-9393-06ccafc59a87.mov

@rebeccajsimmonds The date picker uses your native settings, so if you want 24hr you'll need to change it on your system. Additionally brave doesn't open the date picker UI just by focusing the input. You should have a clickable calendar displaying on the right of the input, which you don't. Can you try disabling your extensions?
Screenshot from brave:


vtk_fps input must be integer not decimal in .fst-file

Description
I have had an issue in which I was able to run acdc with one set of input files, but it crashing with another set of input files. Specifically, acdc failed to import the input files in one of the cases. After some systematic comparison, I found that if the variable "VTK_fps" in the .fst-file was set to "15.0", acdc failed to read .fst-file and import input file, whereas if the variable was set to "15", acdc is able to import the input files. I don't see why the frame rate has to be an integer, and there was no error message, so it was difficult to track down what was wrong.
Steps to reproduce issue
The following input made the simulations crash:
---------------------- VISUALIZATION ------------------------------------------
0                      WrVTK       - VTK visualization data output: (switch) {0=none; 1=initialization data only; 2=animation}
2                      VTK_type    - Type of VTK visualization data: (switch) {1=surfaces; 2=basic meshes (lines/points); 3=all meshes (debug)} [unused if WrVTK=0]
False                  VTK_fields  - Write mesh fields to VTK data files? (flag) {true/false} [unused if WrVTK=0]
15.0                    VTK_fps     - Frame rate for VTK output (frames per second){will use closest integer multiple of DT} [used only if WrVTK=2]
Current behavior
In the "Model"-tab, there is no list of input files after opening the .fst-file. Furthermore, acdc crashes when trying to run simulations.
Expected behavior
When importing an .fst-file, the imported input files should be listed and the simulations should run without crashing.

@verlivkra Thank you for tracking this down and giving a detailed explanation of what was happening. I apologize for taking so long to respond, several other projects had higher priority. I agree that 15.0 should parse into an integer field since it is an integer. I've modified the code to allow this to work; however, I added the check that the number has to be an integer so 15.1 would still be an error. On the plus side, the code to display the error messages has finally been added, so it should be obvious if this is the case.
The binaries with these changes are available at https://github.com/OpenFAST/acdc/actions/runs/10014059895.


Handle redirect to sub menu when click parent menu

Fixes #805
@rhtyd cc @svenvogel Please review it. Thanks.

@blueorangutan package

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/876 (JID-3701)

I still see the same old behaviour @utchoang can you check the test/QA env.

@rhtyd Currently, it only activity for the tablet and mobile mode.

@utchoang can you do this for desktop as well?

@blueorangutan  package

@blueorangutan package

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/876 (JID-3757)

Thanks @utchoang I tested does not seem to work, is there something I'm missing (I'm expecting the VM to be selected/open when I click compute)?

@rhtyd Sorry about my mistake. Please review it again. Thanks.

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

@blueorangutan package

@utchoang a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/876 (JID-3761)

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/876 (JID-3762)

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/876 (JID-3763)

Tested few screens for mobile and desktop - lgtm.

@utchoang It intermittently works on desktops. Just navigate from top to bottom and back up again and sometimes it does not open the first sub-menu. Could you fix it ?

@blueorangutan package

@utchoang a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/876 (JID-3765)

@blueorangutan package

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/876 (JID-3768)

LGTM, @davidjumani are you lgtm on it for merging?

@utchoang Looks like navigation sometimes breaks now if anything other than the first sub-menu item is selected :(

@blueorangutan package

@utchoang a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/876 (JID-3771)

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/876 (JID-3772)

LGTM - on latest testing on desktop and mobile; the only case it doesn't work is when you click on right on the nav title (not on the expand button but between the title/label and the right expand button, it expands than open the first child) - still better than what we've now.
@davidjumani can you test your cases?

@utchoang @rhtyd Works well on mobile and tablet. Clicking the icon / text while on desktop opens the first submenu, so users who'd just like to brows the navbar can click the blank space


fix bug where top-level typedefs were mishandled by zng

This commit fixes a bug where the zng reader and writer were
not properly handling alias encodings when the top-level
record had an aliased type.
Fixes #2187

@mccanne: TestZq/zio/zngio/ztests/outer-alias failed during make test-system.


feat(GuildScheduledEvent): add support for image

Summary
Discord pushed it into canary last night, the cdn endpoint is the one that the client uses.

note: there isn't any PR in discord-api-docs, but the image field has been there for a while.
ref:
https://github.com/Discord-Datamining/Discord-Datamining/commit/d633cd1d95e097d9c97beaef42926d958395bc7c

Checklist

[x] If code changes were made, then they have been tested

[x] I have updated the documentation to reflect the changes
[x] I have formatted the code properly by running task lint or pre-commit run --all-files

[ ] This PR fixes an issue
[x] This PR adds something new (e.g. new method or parameters)
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

Speaking of the test_bot changes, this folder is going to be removed soon
Are we going to use a different repo?

audit log keys are missing, do those just not exist yet?

Oops, didn't think about that, I'll check it

merged now as discussed on Discord, seems like audit log events are still uncertain/TBD


Feature/tornado loop

This pull request adds a TornadoConsumer class, that utilises the TornadoConnection object in Pika.

Codecov Report

Merging #11 into master will increase coverage by 1.05%.
The diff coverage is 91.35%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
+ Coverage   80.38%   81.44%   +1.05%
==========================================
  Files           8        7       -1
  Lines         413      512      +99
  Branches       15       20       +5
==========================================
+ Hits          332      417      +85
- Misses         81       95      +14

Impacted Files
Coverage Δ

sdc/rabbit/test/test_consumer.py
100% <100%> (ø)
:arrow_up:

sdc/rabbit/__init__.py
100% <100%> (ø)
:arrow_up:

sdc/rabbit/consumers.py
50% <78.46%> (ø)

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 bb9d284...89bebea. Read the comment docs.


Do Not Block the Event Loop

Currently, the implementation of Puppet has bugs that will block the event loop, because the array is too big and we have a sync loop to deal with lots of the data.
Should split the Array to Chunks with a max size like 64 or 128.

Merge to https://github.com/Chatie/wechaty/issues/1450


compare getcontent to new value

This is about https://github.com/PowerShell/platyPS/issues/393
Content check is done before testing for -Force switch. This means these all three commands work fine, while only the first one updates file.
New-MarkdownHelp -Command Get-Date -OutputFolder MyFolder
New-MarkdownHelp -Command Get-Date -OutputFolder MyFolder -Verbose # file exists, no changes
New-MarkdownHelp -Command Get-Date -OutputFolder MyFolder -Verbose -Force

Second option is to move this check after -Force check (6 lines below), which would cause 2nd command to fail, but 3rd one would still work as expected.

That's great, good job!


Use gene symbols a node labels

Hi Colin,
we have user ticket:
https://github.com/Complex-Portal/complex-portal-view/issues/191
From Suzie at SGD.
Current behaviour
In the Participant diagram, SOME subunits are showing a PRO ID (P00410-PRO_0000006048) instead of the human readable/standard-ish name (COX2_yeast). Looks like in the righthand key, these subunits display the PRO ID instead of the UniProt accession- but clicking the PRO ID goes to UniProt anyway.
Expected behaviour
Subunit names in the moving diagram be uniform and recognizable by humans, eg COX2_yeast
Minimal reproduction of the problem with instructions
Go to https://www.ebi.ac.uk/complexportal/complex/CPX-1721
Look at Participants diagram

What is the motivation / use case for changing the behaviour?
Hard to read the diagram (minor bug as righthand list is readable)
Please tell us about your environment:
Mac OS 10.12.6, FF 66.
__
I thought we discussed it before as to most users the UniProt_name (= our shortlabel) or gene symbol is more human readable - but I can't find any tickets.
I think this need looking at from your side?

Hi Birgit,
I'm not sure, but I think the viewer is displaying what its asked to.
If you look at the JSON it recieves (https://www.ebi.ac.uk/intact/complex-ws/export/CPX-1721) some proteins are labelled with the "cox3_yeast" form and some with the "p00410-pro_0000006048" form.
So, assuming https://www.ebi.ac.uk/intact/complex-ws/export/CPX-1721 is displaying the JSON the viewer receives, then the problem is in that JSON?
There is a proposal for me to come down to Cambridge for a week to help with ComplexViewer issues, but I don't know when this will be.
Colin

I know the viewer is displaying what it's supposed to as we query the "shortlabel". However, we had various discussions about displaying the "gene symbol" which is somewhere in the enriched data.
Yes, we are looking forward to hosting you for another week or 2. Just let us have the date :)

discussions about displaying the "gene symbol" which is somewhere in the enriched data.

Um, I'm a bit confused, I don't know what the enriched data you refer to is.
If the input JSON to ComplexViewer is searched for the the string "cox2" it doesn't appear in it - so it still seems there's not a lot ComplexViewer can do about this? (Still seems this would need fixed in the thing that produces the JSON which ComplexViewer receives?)

Ok, you are right, we do not supply the gene symbol in the JSON. In that case we need to do it in the JSON first...
In short, we can't do anything about the dodgy display when we have PRO chains.
Ce la vie ;-)


Set GLTFQuadImporter's global subivision to allow baking

Set res://addons/godot_subdiv/quad_import_plugin.gd 's subdivision from 0 to a integer number to allow baking in the options menu or hard coded.

May use convert_meshinstance_to_quad(i, subidivsion_level)

Can expose https://github.com/tefusion/godot-subdiv/blob/c059952d75ab9e1ae770c12914edc8dc5db27b87/src/subdiv_types/subdivision_mesh.cpp#L308 this array as an ArrayMesh using surface tool.

new surface tool
begin trimesh
create_from(existing: Mesh, surface: int)
Maybe ImporterMesh.add_surface(primitive: PrimitiveType, arrays: Array, blend_shapes: Array [] = [], lods: Dictionary = {}, material: Material = null, name: String = "", flags: int = 0)

Can expose
https://github.com/tefusion/godot-subdiv/blob/c059952d75ab9e1ae770c12914edc8dc5db27b87/src/subdiv_types/subdivision_mesh.cpp#L308
as an ImporterMesh
ImporterMesh.add_surface(primitive: PrimitiveType, arrays: Array, blend_shapes: Array [] = [], lods: Dictionary = {}, material: Material = null, name: String = "", flags: int = 0) then ImporterMesh.get_mesh() as an ArrayMesh

Seems like a good idea to me. Cause I think baking at import will be done not that often (only benefit I see is not having to keep a large glb/gltf file that already is subdivided around) an import setting would be better here (bool bake and below int level). We should also make a seperate function for this in subdivision_mesh like bake(p_mesh, int level, bool generate_lods, bool bake_skinning) cause there might be a need to bake bones and weights as well. I'll likely seperate the last part https://github.com/tefusion/godot-subdiv/blob/c059952d75ab9e1ae770c12914edc8dc5db27b87/src/subdiv_types/subdivision_mesh.cpp#L272-L306 into an extra function to make that easier for quads.

@tefusion I had some trouble enabling lod generation. It appears to be an order of import operations problem.

I'm currently working on a SubdivisionBaker class for the baking (which will then be called within the importer for arraymesh). The functions I currently have are:
Ref<ArrayMesh> get_array_mesh(const Ref<ArrayMesh> &p_base, const Ref<TopologyDataMesh> &p_topology_data_mesh, int32_t p_level, bool generate_lods);

and
Ref<ImporterMesh> get_importer_mesh(const Ref<ImporterMesh> &p_base, const Ref<TopologyDataMesh> &p_topology_data_mesh, int32_t p_level);

where the arraymesh method will call the importermesh one and then call generate_lods.
Another question: Do you remember if you got baking the bone/weights data working, it should work just like UV's or no? Haven't got it working yet.

Note!
If you leave a ImporterMesh and ImporterMesh3D without lods, it will get auto generated by the scene importer system

Baking for skinning added in 7879bb9cbc52d2e062b5ddadfded22176276e34f and baking for blendshapes added in 7ed344dc4853a637ff4501cc0bd131f62272fe2b. So this is pretty much done now. The only issue left for this is LOD generation. Imo the importer should produce an ArrayMesh, so lods need to be generated by code.

Lods are now working with 39847c796bc3255f058229bc10f392829d1f924f. The normal scene importer now has a subdivision setting section that bakes before the normal importer stuff starts


added my name to guestbook.md

Hey Sean,
Thank you for this wonderful Coursera course. I am simply going through the steps in the 'Forking' part of Week 4.
Thanks,
Eric

Thank you!


Fail to build falco module in air-gapped environment

When deploying falco in an air-gapped environment, it can't compile the falco probe by itself even though kernel headers/devel packages are installed on the host.

How to reproduce it
Deploy falco 0.21.0 deployment manifests in a kubernetes cluster with strict egress restrictions (no Internet connectivity).
Expected behaviour
falco-probe.ko can be compiled at container startup using host kernel headers.
Screenshots
* Setting up /usr/src links from host
* Unloading falco-probe, if present
* Running dkms install for falco
Error! Could not find module source directory.
Directory: /usr/src/falco-latest does not exist.
* Running dkms build failed, couldn't find /var/lib/dkms/falco/latest/build/make.log
* Trying to load a system falco-probe, if present
* Trying to find precompiled falco-probe for 3.10.0-1062.12.1.el7.x86_64
Found kernel config at /host/boot/config-3.10.0-1062.12.1.el7.x86_64
* Trying to download precompiled module from https://s3.amazonaws.com/download.draios.com/stable/sysdig-probe-binaries/falco-probe-latest-x86_64-3.10.0-1062.12.1.el7.x86_64-eab5dab83905e02403bc54f09c828b8a.ko
curl: (7) Failed to connect to s3.amazonaws.com port 443: Connection timed out
Download failed, consider compiling your own falco-probe and loading it or getting in touch with the Falco community
Fri Apr  3 16:28:25 2020: Falco initialized with configuration file /etc/falco/falco.yaml
Fri Apr  3 16:28:25 2020: Loading rules from file /etc/falco/falco_rules.yaml:
Fri Apr  3 16:28:26 2020: Loading rules from file /etc/falco/falco_rules.local.yaml:

Environment

Falco version: 0.21.0

System info:

{
  "machine": "x86_64",
  "nodename": "infra-stack-falco-dbsg7",
  "release": "3.10.0-1062.12.1.el7.x86_64",
  "sysname": "Linux",
  "version": "#1 SMP Tue Feb 4 23:02:59 UTC 2020"
}

Cloud provider or hardware configuration:
OS:

NAME="CentOS Linux"
VERSION="7 (Core)"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="7"
PRETTY_NAME="CentOS Linux 7 (Core)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:centos:centos:7"
HOME_URL="https://www.centos.org/"
BUG_REPORT_URL="https://bugs.centos.org/"

CENTOS_MANTISBT_PROJECT="CentOS-7"
CENTOS_MANTISBT_PROJECT_VERSION="7"
REDHAT_SUPPORT_PRODUCT="centos"
REDHAT_SUPPORT_PRODUCT_VERSION="7"

Kernel: 3.10.0-1062.12.1.el7.x86_64

Installation method: Kubernetes

Additional context
Kernel related RPMs installed on all Kubernetes nodes:

kernel-3.10.0-1062.12.1.el7.x86_64
kernel-headers-3.10.0-1062.12.1.el7.x86_64
kernel-devel-3.10.0-1062.12.1.el7.x86_64

When I try to compile manually the probe, it succeeds in the pod, but somehow the dmks build is failing (no logs found)
# cd /usr/src
# ls
debug  falco-be1ea2d9482d0e6e2cb14a0fd7e08cbecf517f94  kernels
# cd falco-be1ea2d9482d0e6e2cb14a0fd7e08cbecf517f94
# ls
Makefile  dkms.conf	   dynamic_params_table.c  fillers_table.c  main.c  ppm_compat_unistd_32.h  ppm_events.c  ppm_events_public.h  ppm_fillers.h	   ppm_ringbuffer.h  ppm_version.h
bpf	  driver_config.h  event_table.c	   flags_table.c    ppm.h   ppm_cputime.c	    ppm_events.h  ppm_fillers.c        ppm_flag_helpers.h  ppm_syscall.h     syscall_table.c
# make
make -C /lib/modules/3.10.0-1062.12.1.el7.x86_64/build M=/usr/src/falco-be1ea2d9482d0e6e2cb14a0fd7e08cbecf517f94 modules
make[1]: Entering directory '/host/usr/src/kernels/3.10.0-1062.12.1.el7.x86_64'
  CC [M]  /usr/src/falco-be1ea2d9482d0e6e2cb14a0fd7e08cbecf517f94/main.o
  CC [M]  /usr/src/falco-be1ea2d9482d0e6e2cb14a0fd7e08cbecf517f94/dynamic_params_table.o
  CC [M]  /usr/src/falco-be1ea2d9482d0e6e2cb14a0fd7e08cbecf517f94/fillers_table.o
  CC [M]  /usr/src/falco-be1ea2d9482d0e6e2cb14a0fd7e08cbecf517f94/flags_table.o
  CC [M]  /usr/src/falco-be1ea2d9482d0e6e2cb14a0fd7e08cbecf517f94/ppm_events.o
/usr/src/falco-be1ea2d9482d0e6e2cb14a0fd7e08cbecf517f94/ppm_events.o: warning: objtool: .altinstr_replacement+0x0: call without frame pointer save/setup
  CC [M]  /usr/src/falco-be1ea2d9482d0e6e2cb14a0fd7e08cbecf517f94/ppm_fillers.o
  CC [M]  /usr/src/falco-be1ea2d9482d0e6e2cb14a0fd7e08cbecf517f94/event_table.o
  CC [M]  /usr/src/falco-be1ea2d9482d0e6e2cb14a0fd7e08cbecf517f94/syscall_table.o
  CC [M]  /usr/src/falco-be1ea2d9482d0e6e2cb14a0fd7e08cbecf517f94/ppm_cputime.o
  LD [M]  /usr/src/falco-be1ea2d9482d0e6e2cb14a0fd7e08cbecf517f94/falco-probe.o
  Building modules, stage 2.
  MODPOST 1 modules
  CC      /usr/src/falco-be1ea2d9482d0e6e2cb14a0fd7e08cbecf517f94/falco-probe.mod.o
  LD [M]  /usr/src/falco-be1ea2d9482d0e6e2cb14a0fd7e08cbecf517f94/falco-probe.ko
make[1]: Leaving directory '/host/usr/src/kernels/3.10.0-1062.12.1.el7.x86_64'
# ls
Makefile	dkms.conf		dynamic_params_table.o	falco-probe.ko	   falco-probe.o    flags_table.c  main.o	  ppm_compat_unistd_32.h  ppm_events.c	ppm_events_public.h  ppm_fillers.o	 ppm_syscall.h	  syscall_table.o
Module.symvers	driver_config.h		event_table.c		falco-probe.mod.c  fillers_table.c  flags_table.o  modules.order  ppm_cputime.c		  ppm_events.h	ppm_fillers.c	     ppm_flag_helpers.h  ppm_version.h
bpf		dynamic_params_table.c	event_table.o		falco-probe.mod.o  fillers_table.o  main.c	   ppm.h	  ppm_cputime.o		  ppm_events.o	ppm_fillers.h	     ppm_ringbuffer.h	 syscall_table.c
# cat /var/lib/dkms/falco/latest/build/make.log
cat: /var/lib/dkms/falco/latest/build/make.log: No such file or directory
# cd /var/lib/dkms
# ls
dkms_dbversion	falco
# cd falco
# ls
be1ea2d9482d0e6e2cb14a0fd7e08cbecf517f94
# cd be1ea2d9482d0e6e2cb14a0fd7e08cbecf517f94
# ls
build  source

looks like /usr/src/falco/latest and /var/lib/dkms/falco/latest are both not created

Hello @EppO thanks for reporting this.
Anyway, this issue should have been fixed by https://github.com/falcosecurity/falco/pull/1111
Would you please try using falcosecurity/falco:master image?
Otherwise, you can wait the end of this week in Falco 0.22.0 that will contain that fix.

fixed in 0.22.0!!! thanks :)


Transfer ownership of ansible-sensu

Details

Project URL: https://readthedocs.org/projects/ansible-sensu/
Build URL (if applicable): https://readthedocs.org/projects/ansible-sensu/builds/
Read the Docs username (if applicable): jaredledvina

Expected Result
Hi! Currently, the ansible-sensu repo has moved under the Sensu organization on Github. It appears, that ever since that move, the builds have never been kicked off even though looking at the Github integration, things look legit. I think, we just need an Owner of the RTD site to update the config for the new repo and/or relink the repo.
I have already opened https://github.com/sensu/sensu-ansible/issues/135 but have been unsuccessful in getting the current owner to take a look. Is there any chance you'd be able to add the current Sensu Ansible maintainers as owners of the ansible-sensu docs repo?
Current list of Sensu Ansible maintainers: https://github.com/sensu-plugins/community/#maintained-areas
I'm happy to provide any information that might be helpful here, just let me know!

Any chance someone might have a few to take a look here? If I need to provide any additional information I'm happy too. (likewise if there's a better avenue to accomplish this, let me know!)

cc @humitos or @agjohnson?

@jaredledvina hi! I just send and email to @cmacrae. Let's see if we have any sucessful reply.

Hi guys 👋 Sorry about the delayed response. Yes, I'm absolutely happy for ownership/permissions to be granted for this. Let me know if there's any action I need to take!

To grant permissions to @jaredledvina so he can update the docs, you have to go to https://readthedocs.org/dashboard/ansible-sense/users/ and add @jaredledvina user. Thanks a lot!

Thanks! All done - @jaredledvina

@cmacrae thanks a lot!
I'm closing the issue since I can see @jaredledvina as a member of the project now. In any case, feel free to reopen if you consider.


'Typing' speed of BrowserSession.Fillin.With massively variable

I'm finding that the speed at which Coypu fills in text boxes can vary massively from one field to another in the same test.
So I have a test which fills in a few fields, then moves on to another section and fills in a few more.
The first fields, it's almost as if the fields are filled in one go - ie the whole word appears almost instantaneously, wheras for other fields, it's as if it's being typed letter by letter so that the whole field can take a few seconds to to fill in.
Any ideas what might be causing this, or what I can do.
I have other projects where none of the tests exhibit this behaviour, wheras all of the tests in this project do, with the same version of Coypu, ChromeDriver, etc,
I've attached a zip file which has a recording showing that the first two fields are filled in almost instantaneously wheras the next fields are much slower.
Using Coypu 2.14
Selenium.WebDriver.ChromeDriver 2.40.0
IRAMS.zip

@bwmodular are those other projects and tests performed against the same form as presented on that video?
Maybe there is some weird implementation of those inputs fields on that specific page because if it works fine everywhere else..


Project name public/private report fix

This is to fix the issue of year and ID being appended to Project Name in Public and Private Reports .


NIFI-4072 replace keyword RELEASE with actual version number

Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit? single

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?

[ ] Have you written or updated unit tests to verify your changes?

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
dependency license is MIT. And this dependency is not new, only version is specified explicitly now.

[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?

[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?

[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Confirmed that's the only place to update and confirmed full build is OK. Merging to master, thanks @markobean


Fix script class icons looking for paths at runtime

I believe this closes #25171. At the very least, I made this change, re-compiled, re-generated export templates, exported the project, and encountered no errors.

Thanks!


[WIP] .travis.yml: Detect if 'make update' updated something

If it did, it really is something that should be checked in, and should therefore make a CI build fail.

There's an extra commit that introduces a fake added function, to check that this modification does the job right.  That commit will be removed before merging.

  I'll take it, though, provided the CI shows that this works as I intended.

Ok, the builds seem to do what I expect.  Merging.

Merged.
master: 46e5b661d435b11652b90cd9e06cbf6606d3b61a
1.1.0: 85bbb9e6189e158647b1cb15c5c1dea32882240e


[Task] Missing analytics events for Git Import

@vuiets is this sorted?

@vuiets @hiteshjoshi Can we please add details to this issue? What events are missing? What needs to be added? This will make it easier for us to grok this issue & estimate it properly.


CAMEL-18047 Upgrade jakarta.mail to 1.6.7

Issue: https://issues.apache.org/jira/browse/CAMEL-18047
Cherry-picked d04a8ee1d6bc98c997bd4895e449de427a52f4ab

I will merge it as soon as the upstream PR is  merged. Until that time I'm changing this PR into draft to avoid accidental merge,


bug(cdk-virtual-scroll-viewport): Content not present in SSR load

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
When rendering a page in SSR mode, the content for the virtual scroll is not rendered. It's blank. The page works fine when rendered in the browser. Screenshot attached for the SSR load response

:
Reproduction
Steps to reproduce:

Clone the repo https://github.com/bivs06/angular-universal-ssr-bug
Run npm run dev:ssr
Check the response in the network tab

StackBlitz link: https://stackblitz.com/edit/angular-ivy-kfyv87?file=src%2Fapp%2Fapp.component.html
Expected Behavior
The items for virtual scroll should load in SSR load
Actual Behavior
The items for the virtual scroll are not loading in the SSR load
Environment

Angular:
CDK/scrolling:
Browser(s):
Operating System (e.g. Windows, macOS, Ubuntu):

Hi all, news for this bug?

I'm having the same issue.
Is it even possible to get *cdkVirtualFor to render something in SSR? Shouldn't it at least be possible to prerender an item list which has a fixed item size on every viewport width?

Any update on this issue?

Any update on this issue?

Please provide some updates or any help with any workaround or alternative is highly appreciated. Thanks!!!

As per the code CDK virtual scroll will not support SSR, did I miss spotting this anywhere on the docs page? If not why it is not explicitly mentioned anywhere?

Same error in my case when i need add Virtual CDK Scroll throw error with SSR and in doc not mention only working in browser


Can Jasper be used on the Raspberry Pi A+?

I understand that installation images for the B+ are available, is it possible to install Jasper on the Raspberry Pi A+? In particular, how much memory will Jasper use in normal operation, given the A+ limit of 256 Mb?

I believe that with using the slimmer STT and TTS engines, it would be fine. The base chip is the same, just the stacked RAM is smaller. Also, the images probably would work fine. Try them out first and see.


Slowdown training LSTM on TensorFlow 2.4+

We are noticing a significant slowdown during training with TensorFlow when using the Tensorflow-Privacy LSTM and GRU optimizers on TF 2.4. In tetsing, this only happens when using the TF-privacy optimizers, but it appears related to a change in the recurrent_v2 module of TensorFlow.
Doing some testing, it looks like the slowdown was introduced in between these two tf-nightly builds.
Describe the expected behavior
Training can go from 15 sec/epoch to 2 mins+ per epoch with the latest TF release candidate (tensorflow==2.4.0rc1).
tf-nightly==2.4.0.dev20201019 - 15 sec/epoch
Describe the current behavior
tf-nightly==2.4.0.dev20201020 and TensorFlow RC1 - 2 mins+/epoch
System information
GCP, running on Tesla V100, 16GB RAM, Ubuntu, 8 vCPU, Python 3.8, cuda11, TensorFlow 2.4.0rc0 and nightly installed via PIP.
Standalone code to reproduce the issue
https://gist.github.com/zredlined/72305ab04670197869e470b232d22ed4
In tensorflow/python/keras/layers/recurrent_v2.py
I think this TensorFlow commit is the culprit-- changing use_new_code() back to True speeds the code back up. The only reference I can find is in the issue above for what looks like an internal Google issue? Any help would be hugely appreciated, on most datasets we have tested with slowdowns are 10-20x. Thanks!
tensorflow/tensorflow@73b7097.
def _use_new_code():
  return False  # NOTE: changed to False in @73b7097. Changing back to True speeds training up.

Other notes/logs
Originally posted at https://github.com/tensorflow/privacy/issues/141, opening an issue here as it appears to be an issue within TensorFlow.

Hi @zredlined, thanks for reporting the issue! Currently, the new code path is disabled due to some other internal issues. In theory, enabling the new code of LSTM/GRU will boost eager performance a lot, but should have little affect to other modes.
Do you run the training with pure eager mode?

Yes, we are running the training in eager mode.
Can you help me understand what the other internal issues might be? We use this for a production library, and need to use TF 2.4.x+ to support differentially private training. Any options to re-enable the codepath (as an optional argument, or whatever you recommend) would be awesome. Thanks!

@amahendrakar Is there any way we can make the codepath for _use_new_code() a user facing option, defaulted to "off" like it is now, but that we can enable to true to speed up eager mode training?

@yhliang2018 the training is run via a call to model.fit which IIUC should execute the actual training in graph mode. However, the function optimizer is reporting the failure function_optimizer failed: Invalid argument: Input 0 of node RMSprop/gradients/loop_body/PartitionedCall_1/pfor/PartitionedCall/gradients/zeros_like/pfor/ZerosLike was passed float from sequential/lstm/PartitionedCall:6 incompatible with expected variant. so some of the graph mode optimizations seem to not get applied. The above optimizer failure message doesn't appear when monkey-patching _use_new_code() to return True.

Andreas Terzis from the TF-Privacy team @aterzis-google

@zredlined The new code path with new Case op causes some failures internally, so it's disabled for now. We are working on the fix, and will enabled the new code path eventually.
In terms of the training mode, as @misberner mentioned, in your repro, model.compile with run_eagerly=None is used, so the model training with model.fit will be wrapped within tf.function which is in graph mode. The main change in the new code path is to warp the layer call on CPU into tf.function, which in theory only affects pure eager performance on CPU, and should have no effect on graph performance.
I see there is a recent change on apply_gradients of DPKerasAdamOptimizer, and this might affect model performance as well. @schien1729 any insights into it?

@yhliang2018 the training is run via a call to model.fit which IIUC should execute the actual training in graph mode. However, the function optimizer is reporting the failure function_optimizer failed: Invalid argument: Input 0 of node RMSprop/gradients/loop_body/PartitionedCall_1/pfor/PartitionedCall/gradients/zeros_like/pfor/ZerosLike was passed float from sequential/lstm/PartitionedCall:6 incompatible with expected variant. so some of the graph mode optimizations seem to not get applied. The above optimizer failure message doesn't appear when monkey-patching _use_new_code() to return True.

@misberner Where is this error message from? Could you provide a colab link for a mini repro?

@yhliang2018 unfortunately I am unable to reproduce this in a Colab notebook even with GPU enabled (or the native library messages are suppressed). I am able to reproduce it on my box by running the code from the gist linked in the original issue w/o any further modifications.
$ pip list | grep -E '^(tf-nightly|tensorflow-privacy)'
tensorflow-privacy                 0.5.1
tf-nightly                         2.4.0.dev20201020
$ curl 'https://gist.githubusercontent.com/zredlined/72305ab04670197869e470b232d22ed4/raw/4d9cd113fe38cce4d0b6494595fea143096541d4/tf_privacy_basic.py' -o tf_privacy_basic.py
$ python tf_privacy_basic.py
[...]
WARNING:tensorflow:Converting IndexedSlices(indices=Tensor("gradient_tape/sequential/embedding/embedding_lookup/Reshape_1:0", shape=(1600,), dtype=int32), values=Tensor("gradient_tape/sequential/embedding/embedding_lookup/Reshape:0", shape=(1600, 256), dtype=float32), dense_shape=Tensor("gradient_tape/sequential/embedding/embedding_lookup/VariableShape:0", shape=(2,), dtype=int32)) to a dense representation may make it slow. Alternatively, output the indices and values of the IndexedSlices separately, and handle the vectorized outputs directly.
WARNING:tensorflow:Converting IndexedSlices(indices=Tensor("gradient_tape/sequential/embedding/embedding_lookup/Reshape_1:0", shape=(1600,), dtype=int32), values=Tensor("gradient_tape/sequential/embedding/embedding_lookup/Reshape:0", shape=(1600, 256), dtype=float32), dense_shape=Tensor("gradient_tape/sequential/embedding/embedding_lookup/VariableShape:0", shape=(2,), dtype=int32)) to a dense representation may make it slow. Alternatively, output the indices and values of the IndexedSlices separately, and handle the vectorized outputs directly.
2020-11-19 11:46:07.364349: I tensorflow/compiler/mlir/mlir_graph_optimization_pass.cc:116] None of the MLIR optimization passes are enabled (registered 2)
2020-11-19 11:46:07.428010: I tensorflow/core/platform/profile_utils/cpu_utils.cc:112] CPU Frequency:<PHONE_NUMBER> Hz
2020-11-19 11:46:07.480035: E tensorflow/core/grappler/optimizers/meta_optimizer.cc:592] function_optimizer failed: Invalid argument: Input 0 of node Adam/gradients/loop_body/PartitionedCall/pfor/PartitionedCall/gradients/zeros_like/pfor/ZerosLike was passed float from sequential/lstm/PartitionedCall:6 incompatible with expected variant.
2020-11-19 11:46:07.707187: E tensorflow/core/grappler/optimizers/meta_optimizer.cc:592] function_optimizer failed: Invalid argument: Input 0 of node Adam/gradients/loop_body/PartitionedCall/pfor/PartitionedCall/gradients/zeros_like/pfor/ZerosLike was passed float from sequential/lstm/PartitionedCall:6 incompatible with expected variant.
2020-11-19 11:46:07.795195: W tensorflow/core/common_runtime/process_function_library_runtime.cc:805] Ignoring multi-device function optimization failure: Invalid argument: Input 0 of node Adam/gradients/loop_body/PartitionedCall/pfor/PartitionedCall/gradients/zeros_like/pfor/ZerosLike was passed float from sequential/lstm/PartitionedCall:6 incompatible with expected variant.

Note that I do see the first two warnings when running it in Colab as well, but they don't seem to come from native code. Another possibly relevant piece of information is that the errors disappear when running with CUDA_VISIBLE_DEVICES=-1.

@amahendrakar can we make the codepath selection for _use_new_code() user selectable and defaulted to "false"?

Is the proposed solution above of making use_new_code() user selectable and defaulted to False acceptable? Happy to submit a pull request. Thanks!

@zredlined Sure, making it as a private flag which users can enable it accordingly seems fine to me. Feel free to submit the PR! Thank you!

@misberner Which TF version do you use in your box? Could you try tf-nightly to see if the error still exists? Feel free to open another issue if it still doesn't work.

@misberner Sounds great. Thanks! Does the error happen only on GPU? Given #42957, How about trying the latest tf-nightly + CPU?


App not working in IE11, getting a blank page

Type of issue

[x] Bug
[ ] New feature
[ ] Improvement

Description
When loading the app on IE11 (11.15.16299.0) I get a white empty page. The solution works fine in other browsers (Edge, Firefox, Chrome).
The console shows an error, which might be the root cause of the blank page:

Object doesn't support property or method 'endsWith'

likely the code at line 113:

"endsWith" is not supported by IE: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/endsWith

This looks like the @dluc  fix on 10/20/2017 fixed it... or a subsequent IE update did.  The UI came up just fine on IE Version 11.125.16299.0.


Extended SetParsedStack to handle stack traces of inner exceptions.

Fix Issue Microsoft/ApplicationInsights-dotnet#1012.
The existing API allows to modify stack traces of logged exceptions in a very limited way. For example there's an option to replace the stack trace of the top level exception, but stack traces of inner exceptions cannot be modified at all. This simple change extends the existing SetParsedStack method to accept an additional parameter which specifies whether stack trace of the top most exception or it's selected inner exception should be replaced.
It would be nice to bring more of improvements described in dotnet/corefx#24627 but to do it a bigger change would be needed. Till then this modification should still help a lot.

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

CC: @tokaplan for review

@tokaplan when you are back from holidays, could you please review?

Also cc @cijothomas @MS-TimothyMothra


feat: update screenshots for catalog

Update screenshots to fit catalog requirements.

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


SoftBody support in GodotPhysics 3D

List of changes:

Fixed SoftBody surface update with new rendering system
Added GodotPhysics implementation for SoftBody
Added support to get SoftBody rid to interact with the physics server
Added support to get SoftBody bounds from the physics server
Removed support for unused get_vertex_position and get_point_offset from the physics server
Removed SoftBody properties that are unused in both Bullet and GodotPhysics (angular and volume stiffness, pose matching)
Added RenderingServerHandler interface to PhysicsServer3D so the physics servers don't need to reference the class from SoftBody node directly

Notes:

SoftBody creation and simulation is mostly based on Bullet soft bodies and internal soft body queries use the dynamic BVH implementation from Bullet introduced in #44531.
The solver for Body / Soft Body pairs is based on what is done for Body / Body pairs with a few specific changes to work with soft body nodes. There's probably room for optimization but the collision response seems to work well for now.
SoftBody collision with ConcavePolygonShape is currently buggy. GodotPhysics allows backface collision and that doesn't play well with small particle collision (they can be pushed through faces by the solver and sometimes along the face plane because of an extra bug). This will be addressed in a separate PR.

Just pushed a fix in BodySoftBodyPair3DSW to calculate impulses applied on rigid bodies properly according to their angular velocity.

Thanks!


LoadFromGitHub : C:\baseline_15.0.516.30.zip not found...

Describe the issue
CompareExchangeHashes.ps1 fails when trying to download baseline_15.0.516.30.zip
Script Output
PS C:> .\CompareExchangeHashes.ps1
[03/11/2021 11:56:36] Started...
Found exchange version: 15.0.1497.2
Loaded baseline for 15.0.1104.5, hashes number 7353
Loaded baseline for 15.0.1497.2, hashes number 7395
Loaded baseline for 15.0.1497.12, hashes number 7464
Can't find local baseline for 15.0.516.30
Downloading baseline file from GitHub to C:\baseline_15.0.516.30.zip
LoadFromGitHub : C:\baseline_15.0.516.30.zip not found... please open issue on
https://github.com/microsoft/CSS-Exchange/issues, we will work on it
At C:\CompareExchangeHashes.ps1:647 char:13

        LoadFromGitHub -url $zip_file_url -filename $zip_file -installed_ver ...

  + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
  + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,LoadFromGitHub

I am still experiencing this issue:
[PS] C:\utilbats\haf>.\CompareExchangeHashes.ps1
Do you want to run software from this untrusted publisher?
File C:\utilbats\haf\CompareExchangeHashes.ps1 is published by CN=Microsoft Corporation, O=Microsoft Corporation,
L=Redmond, S=Washington, C=US and is not trusted on your system. Only run scripts from trusted publishers.
 Help (default is "D"): r
[03/11/2021 23:09:56] Started...
Creating a new session for implicit remoting of "Get-ExchangeServer" command...
Found exchange version: 15.0.1497.2
Can't find local baseline for 15.0.1395.4
Downloading baseline file from GitHub to C:\utilbats\haf\baseline_15.0.1395.4.zip
Loaded baseline for 15.0.1395.4, hashes number 7395
Can't find local baseline for 15.0.516.30
Downloading baseline file from GitHub to C:\utilbats\haf\baseline_15.0.516.30.zip
LoadFromGitHub : C:\utilbats\haf\baseline_15.0.516.30.zip not found... please open issue on
https://github.com/microsoft/CSS-Exchange/issues, we will work on it
At C:\utilbats\haf\CompareExchangeHashes.ps1:647 char:13

        LoadFromGitHub -url $zip_file_url -filename $zip_file -installed_ver ...

  + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
  + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,LoadFromGitHub

[PS] C:\utilbats\haf>

Created a single task for tracking all missing baselines, please follow: https://github.com/microsoft/CSS-Exchange/issues/313


Type aliases using Union syntax incompatible with Python 3.9

Describe the bug
A basic Python script bombs when importing from expression.collections.
To Reproduce

Install expression into Python 3.9 virtual environment using pip install.
Create a basic top-level script that imports expression.collections.
Run script with Python 3.9 from virtual environment.

Expected behavior
Python script runs without issue.
Code or Screenshots
Here is  the code from my Python script:
from expression.collections import seq, Seq

if __name__ == '__main__':
    print("All done!")

Here is the traceback I get:
Traceback (most recent call last):
  File "/home/atonally/develop/sandbox-py39/main.py", line 3, in <module>
    from expression.collections import seq, Seq
  File "/home/atonally/develop/sandbox-py39/.venv/lib/python3.9/site-packages/expression/__init__.py", line 11, in <module>
    from . import collections, core, effect
  File "/home/atonally/develop/sandbox-py39/.venv/lib/python3.9/site-packages/expression/collections/__init__.py", line 4, in <module>
    from . import array, asyncseq, block, map, seq
  File "/home/atonally/develop/sandbox-py39/.venv/lib/python3.9/site-packages/expression/collections/array.py", line 28, in <module>
    from expression.core import (
  File "/home/atonally/develop/sandbox-py39/.venv/lib/python3.9/site-packages/expression/core/__init__.py", line 20, in <module>
    from .fn import TailCall, TailCallResult, tailrec, tailrec_async
  File "/home/atonally/develop/sandbox-py39/.venv/lib/python3.9/site-packages/expression/core/fn.py", line 24, in <module>
    TailCallResult = _TResult | TailCall[_P]
TypeError: unsupported operand type(s) for |: 'TypeVar' and '_GenericAlias'

Additional context

Kubuntu 22.04
Expression 4.2.2
Python 3.9.16

I can't make a pull request, but here's the patch that I'm applying locally to get expression 4.2.2 to work with Python 3.9:
diff --git a/expression/collections/array.py b/expression/collections/array.py
index d1f27f6..399f57b 100644
--- a/expression/collections/array.py
+++ b/expression/collections/array.py
@@ -23,6 +23,7 @@ from typing import (
     Tuple,
     TypeVar,
     cast,
+    Union
 )

 from expression.core import (
@@ -44,7 +45,7 @@ _TState = TypeVar("_TState")
 _TSourceSortable = TypeVar("_TSourceSortable", bound=SupportsLessThan)

 _TSourceSum = TypeVar("_TSourceSum", bound=SupportsSum)
-_Array = List[_TSource] | MutableSequence[_TSource]
+_Array = Union[List[_TSource], MutableSequence[_TSource]]

 class int8(int):
diff --git a/expression/core/fn.py b/expression/core/fn.py
index 4461f27..80a75b9 100644
--- a/expression/core/fn.py
+++ b/expression/core/fn.py
@@ -1,7 +1,7 @@
 from __future__ import annotations

 import functools
-from typing import Awaitable, Callable, Generic, TypeVar
+from typing import Awaitable, Callable, Generic, TypeVar, Union

 from typing_extensions import ParamSpec

@@ -21,7 +21,7 @@ class TailCall(Generic[_P]):
         self.kw = kw

-TailCallResult = _TResult | TailCall[_P]
+TailCallResult = Union[_TResult, TailCall[_P]]

 def tailrec(fn: Callable[_P, TailCallResult[_TResult, _P]]) -> Callable[_P, _TResult]:

Just curious if you're abandoning support for Python 3.9.  Your documentation still states support for Python 3.9+

So the break started from version 4.2.0 (in #77). Haven't seen any discussion or content in documentation about dropping python 3.9.
Hope the compatibility is fixed as I'm using Anaconda which supports python 3.9 (version 2022.10).

Closing this issue since the issue was fixed in #117. However, just giving a heads-up that Expression v5 just took a hard dependency on dataclass_transforms in Python 3.11, so we are now Python 3.11+.


How to restore both model and feature names

I used
bst.save_model(file_path) # to save
bst1 = xgb.Booster(model_file=file_path) # to restore

But I noticed that when using the above two steps, the restored bst1 model returned None
with bst1.feature_names
but with bst.feature_names did returned the feature names I used.
So is there anything wrong with what I have done? or is there another way to do for saving feature _names
thanks

It seems I have to manually save and load feature names, and set the feature names list like:
bst.feature_names = feature_names_list

for your code when saving the model is only done in C level, I guess:
    def save_model(self, fname):
        """
        Save the model to a file.

        Parameters
        ----------
        fname : string
            Output file name
        """
        if isinstance(fname, STRING_TYPES):  # assume file name
            _check_call(_LIB.XGBoosterSaveModel(self.handle, c_str(fname)))
        else:
            raise TypeError("fname must be a string")

You can pickle the booster to save and restore all its baggage.

@khotilov, Thanks. Yes, I can. But I think this is something you should do for your project, or at least you should document that this save method doesn't save booster's feature names

Agree that it is really useful if feature_names can be saved along with booster.
For example, when you load a saved model for comparing variable importance with other xgb models, it would be useful to have feature_names, instead of "f1", "f2", etc.

Other than pickling, you can also store any model metadata you want in a string key-value form within its binary contents by using the internal (not python) booster attributes.
E.g., to create an internal 'feature_names' attribute before calling save_model, do
if hasattr(bst, 'feature_names'): bst.set_attr(feature_names = '|'.join(bst.feature_names))

Then after loading that model you may restore the python 'feature_names' attribute:
if bst.attr('feature_names') is not None: bst.feature_names = bst.attr('feature_names').split('|')

The problem with storing some set of internal metadata within models out-of-a-box is that this subset would need to be standardized across all the xgboost interfaces. Thus, it was left to a user to either use pickle if they always work with python objects, or to store any metadata they deem necessary for themselves as internal booster attributes.

Hi, If using the above attribute solution to be able to use xgb.feature_importance with labels after loading a saved model, please note that you need to define the feature_types attribute as well (in my case as None worked).


Exposed updateCombinational and updateSequential logic in interpreter

Dont merge this yet. I need to change some reset related definitions

Ok. This can be merged now


Why there's a few pixels gap, between WeekBar and CalendarView?

Previously, I use the following code, so that I can have rectangle border around every date, in month view.
    // Code for CustomMonthView
    @Override
    protected void onDrawText(Canvas canvas, Calendar calendar, int x, int y, boolean hasScheme, boolean isSelected) {
        if (WeNoteOptions.isFullscreenCalendar()) {
            canvas.drawLine(x, y, x + mItemWidth, y, this.calendarRectBorderPaint);
            if (x != 0) {
                canvas.drawLine(x, y, x, y + mItemHeight, this.calendarRectBorderPaint);
            }
        }

I use 1dp for stroke width. But, if you notice carefully, the 1st horizontal line, the visible drawn line width seems to be only 0.5dp.
To avoid such problem, I change the code to the following
    // Code for CustomMonthView
    @Override
    protected void onDrawText(Canvas canvas, Calendar calendar, int x, int y, boolean hasScheme, boolean isSelected) {
        if (WeNoteOptions.isFullscreenCalendar()) {
            if (y != 0) {
                canvas.drawLine(x, y, x + mItemWidth, y, this.calendarRectBorderPaint);
            }
            if (x != 0) {
                canvas.drawLine(x, y, x, y + mItemHeight, this.calendarRectBorderPaint);
            }
        }

Yet. I would like to still have a horizontal line between WeekBar and CalendarView. Hence, this is what I did.
The following is the layout of my custom WeekBar (I add a 1dp divider at the end of WeekBar)
<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:orientation="vertical"
    tools:ignore="HardcodedText">

    <LinearLayout
        android:id="@+id/ll_week"

        android:layout_width="match_parent"
        android:layout_height="0dp"
        android:layout_weight="1"
        android:orientation="horizontal">
        ...
    </LinearLayout>

    <View
        android:id="@+id/bottom_divider"

        android:layout_width="match_parent"
        android:layout_height="1dp"
        android:background="?attr/calendarRectBorderColor"
        />

</LinearLayout>

However, the end result isn't perfect still. It seems there's is a small pixel gap, between WeekBar and CalendarView?

Is that a normal behavior? How I can avoid such small pixel gap? Thank you.

I notice that, when I try to drag-n-move the month view toward the top most (Before releasing my finger), there are few pixel gaps between month view (not week view) and weekbar.
Please see my screenshot.
Before drag-n-move

After drag-n-move to top most, not before releasing my finger

As you can see

There is few pixel gap between week bar, and top of calendar view.
Few pixel is missing from the bottom of calendar view (The horizontal green line looks thinner)

Any idea?
I look at the code https://github.com/huanghaibin-dev/CalendarView/blob/67bbee1c73ceca4b0a86f9a51d94c06a79fd47cc/calendarview/src/main/java/com/haibin/calendarview/CalendarLayout.java#L352
Not sure changing
     if (dy < 0 && mContentView.getTranslationY() + dy <= -mContentViewTranslateY) {
to
    if (dy < 0 && mContentView.getTranslationY() + dy < -mContentViewTranslateY) {
will help anything?


how can i find

NanoTimeKit.framework, RelevanceEngine.framework, and RelevanceEngineUI.framework
in xcode simulator file

They are under /Applications/Xcode.app/Contents/Developer/Platforms/WatchOS.platform/Library/Developer/CoreSimulator/Profiles/Runtimes/watchOS.simruntime/Contents/Resources/RuntimeRoot/System/Library/PrivateFrameworks.

发自我的iPhone
------------------ Original ------------------
From: Khaos Tian @.>
Date: Sun,Mar 13,2022 11:54 AM
To: KhaosT/NTKExperimentalDigitalFaceBundleWatch @.>
Cc: xzk @.>, Author @.>
Subject: Re: [KhaosT/NTKExperimentalDigitalFaceBundleWatch] how can i find (Issue #2)
They are under /Applications/Xcode.app/Contents/Developer/Platforms/WatchOS.platform/Library/Developer/CoreSimulator/Profiles/Runtimes/watchOS.simruntime/Contents/Resources/RuntimeRoot/System/Library/PrivateFrameworks.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


python311Packages.aiofile: 3.8.5 -> 3.8.6

Diff: https://github.com/mosquito/aiofile/compare/refs/tags/3.8.5...3.8.6
Changelog: https://github.com/aiokitchen/aiomisc/blob/master/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 236349 run on x86_64-linux 1

9 packages built:

home-assistant-component-tests.bmw_connected_drive
python310Packages.aiofile
python310Packages.aiofile.dist
python310Packages.bimmer-connected
python310Packages.bimmer-connected.dist
python311Packages.aiofile
python311Packages.aiofile.dist
python311Packages.bimmer-connected
python311Packages.bimmer-connected.dist


[#43] setup buildx and update gh actions

[!IMPORTANT]
Update GitHub Actions for Docker and fix typo in GoogleSignInButtonProps in google-signin.tsx.

GitHub Actions:

Update docker/login-action to v3, docker/metadata-action to v5, and docker/build-push-action to v6 in build-push.yml.
Add Docker Buildx setup using docker/setup-buildx-action@v3 in build-push.yml.
Add platforms: linux/amd64,linux/arm64 to Docker build step in build-push.yml.

Frontend:

Rename GitHubSignInButtonProps to GoogleSignInButtonProps in google-signin.tsx.

This description was created by  for 1dfc4b278e07686e5bf583bb3e0c24e53be5fb28. It will automatically update as commits are pushed.

addresses #43


add keywords.json

Sorry for these confusing commits, I made a spelling mistake and tried to correct it and it got messy, but basically, it is just the json file we talked about.

Sorry for these confusing commits

No worries. At some point I may show you how to "squash" the commits that you make in your branch, so that things are as simple as possible (perhaps as little as one commit) when you go to merge into the master branch. That can help to keep the Git history of the repository better organized.


Multi-Node SLURM Support

Adds a SlurmRunner class for multi-node jobs SLURM clusters, without requiring OpenMPI or special permissions. Pairs with this GPT-NeoX PR.

@Quentin-Anthony or @ShivanshuPurohit can you review this and give it a thumbs up?

LGTM

@Quentin-Anthony You're now a CODEOWNER of this repo. Can you resubmit your review and merge it?


Case sensitivity of channels/claims?

Is it intended behavior that claims and channels are case sensitive? i.e. lbry://@oscopelabs <> lbry://@OSCOPELABS

I described this promlem earlier in slack channel. wiil find later and past here

This was a poor design choice that needs to be fixed at the blockchain level.


Remove "productized" comments from community branch #220

Signed-off-by: Aurélien Pupier<EMAIL_ADDRESS>

@rhuss hope that this is OK for you.

Actually, I'd prefer fix that issue ;-) Ass mentioned in #220 the RHEL variation is really meant for production so should refer to productised environments. We don't build or offer RHEL variatons on Docker Hub, so that should be fine.


HAWKULAR-918 Reformat of screen to clean up a bit. Remove experimental status

Not complete as design is not complete yet.

Merged #786


XlsStream.ReadStream hangs on corrupted file

migrated from codeplex https://exceldatareader.codeplex.com/workitem/13090
"When I try to read a corrupted .xls file (with a part of file accidentally lost), my application hangs in an infinite loop, while I expect it to fail with some exception. As I use the library in a web application, such behavior can slow down the server greatly. It also creates a vulnerability to DDOS attacks. An example corrupted file is attached.
This issue can only be reproduced in "Any CPU" or "x64" build mode, as in "x86" OutOfMemory exception is thrown.
Issue reason:
I have discovered that the apllication loops in a "do while" cycle in XlsStream.ReadStream. The fat.GetNextSector never returns FATMARKERS.FAT_EndOfChain, so the loop never stops.
Fix proposal:
Infinite loop can easily be avoided by analyzing the return value of m_fileStream.Read call (the number of bytes read from file). When the file is read to end, m_fileStream.Read returns zero, and it means that we can exit the loop. So I added additional check to the "while" statement.
With such fix, an attempt to read a corrupted file results in an "ArgumentOutOfRange" exception thrown somewhere further in code. This is way better than a hang; however, the thrown exception is not informative. I guess we can throw an exception about file corruption explicitly when we get "zero" from m_fileStream.Read, but I'm not sure that in normal operation such situation can never happen (I just don't know the ExcelDataReader inner structure well enough). I have attached both variants of the fix.
Can the fix be added to the trunk version? Should I upload the patch in the "Source Code" section?
Thanks!"
XlsStream issue.zip | uploaded via ZenHub

Fixed with #210


add "Get all references" route

The Get all references and Get a reference are technically the same route
GET /repos/:owner/:repo/git/refs/:ref

Depending on whether :ref exactly matches e.g. a branch, or whether it is a prefix that matches multiple brunches, the response is an object or an array of objects.
This is not ideal and the API team is aware of it. For the time being, I want to bring the "Get all references" route back as preparation for https://github.com/octokit/rest.js/pull/1043 and then implement a workaround within octokit/rest

lol 🤦♂ we already have that nevermind nothing to see here


CMake: Review external suffixes to add architecture as well as platform

See https://github.com/flucoma/flucoma-pd/issues/40#issuecomment-1106373068
Apparently best practice is to have something like .pd_linux.l_ia64 to help disambiguate different architectures. Need to find out is this applies to all OSes, and what the magic strings are.

The gospel seems to be
https://github.com/pure-data/pure-data/blob/18c9695ba4de543c51c2c013b45b99b8f1c35a19/src/s_loader.c#L32-L38
According to which .l_ia64 is accepted but wrong and would actually be .l_amd64 for linux, .d_amd64 on mac and so forth.


zmq.socket is not a function

I installed zeromq in Raspberry Pi. And when I run the nodejs example for zeromq it gives error as:
TypeError: zmq.socket is not a function.

Same on my side.
OSX
Node v10.15.0
zeromq@5

I am also having this issue, have either of you found a solution? Running on Windows 10

I fixed this problem using the 5 version
npm i zeromq@5

I have the same issue

Same issue for me. On Windows10 + electron 8.0.0
Have rebuilt using npm rebuild zeromq --runtime=electron --target=8.0.0
ZeroMQ version 6.0.0-beta.6
Didn't work.


Suggested correction to explanatory text for exercise 14

The current text is technically false. As of today, it is written:
// Problem:
// For statements can also have init and post statements in the clause.
//
//  for i := 0; i <= 10; i = i + 1 {
//      fmt.Println(i)
//  }
//
// The above for loop will iterate from 0 to 10 and is
// identical in functionality to the following
// single-condition for statement:
//
//  i := 0
//  for i <= 10 {
//      fmt.Println(i)
//      i = i + 1
//  }

which would leave i in scope after the loop block, which is true in some languages but not here.
Proposed change
The exercise text could instead be
// Problem:
// For statements can also have init and post statements in the clause.
//
//  for i := 0; i <= 10; i = i + 1 {
//      fmt.Println(i)
//  }
//
// The above for loop will iterate from 0 to 10 and is
// identical in functionality to the following
// single-condition for statement:
//
// {
//      i := 0
//      for i <= 10 {
//          fmt.Println(i)
//          i = i + 1
// }

I'm a bit on the fence with the idea of adding anonymous scopes as that is a feature that may catch some off guard. Maybe we can change the comment:
// The above for loop will iterate from 0 to 10.
// The following single-condition for statement
// will also perform the same operations:

This sound good?

It does


Fix sim import

Note: the code also requires python 3.8+, replacing from functools import cached_property by property makes it compatible with 3.7

Thanks for fixing the bug!

I found another incorrect import.
It should be good now.

true, i actually corrected that one locally...
btw, would be nice to release a requirement file too ;) (i managed to run the code but with some flag hacks ...)


Suggestion: typescript module resolution support

TypeScript module resolution
https://github.com/Microsoft/TypeScript/issues/5039
Configs
Add baseUrl and, optionaly, paths to tsconfig.json
{
  "compilerOptions": {
    "baseUrl": ".",
    "paths": {
      "*": [ "modules/*" ]
    }
  }
}

Add modules to angular-cli.json apps
{
  "apps": [
    {
      "modules": [
        "src/modules"
      ]
  ]
}

Realisation
https://github.com/ValeryVS/angular-cli/commit/c812fe81f6770c7f9d881e2fcb3a610350fb1b21
Pros
Then we will be able to write
import { MyModule } from 'my';
and it will load module from src/modules/my/index.ts
We aslo can write src/app or src/app/shared to angular-cli.json and get rid of relative paths when we load services in app.

This also seems convenient to me from a syntax point of view. But I think there are some "cons" to go with the "pros" above - in particular around ease of understanding.
As developers first enter the typescript and Angular ecosystem, among 1000 other things they need to understand import statements. In the current default behavior, this can be explained in just a few seconds: import paths which start with "." are importing other modules in the application being worked on. Other imports (from paths that don't start with ".") are importing libraries, code written in other projects or by other people.
The typescript configuration described above would add more complexity to the current easy-to-understand default situation. So while I like it for my own convenience, I think this should be weighed as a trade-off before such a feature is put in to the default configuration CLI creates.

I agree, that this feature can add mode complexity.
But we can provide minimal, but extendable, configuration.
Probably, the most frustrating is import {} from '../../../shared' (how many ../) while import shared service from complicated module.
Also, sometime we want to make shared module. With a few services. We can make it, as a separate package and import from node_modules. But it is simpler to leave it in main repository, if we don't need that module in other projects.
Wrap in.
I like the approach of #2210 with TsConfigPathsPlugin.
Main question is default configuration.
For example:

relative import - start with "."
node_modules import - no prefixes
import from shared - suggest some prefix: # @ ^

Then, we be able to place shared modules into shared folder and import them with our prefix.
app/
    shared/
        api-module/
        ui-module/
    heroes/
        heroes-list.component.ts

// heroes-list.component.ts
import { Component } from '@angular/core';
import { HeroesService } from './heroes.service.ts';
import { ApiService } from '#api-module';
import {
    UINotyficationService,
    UISelectService
} from '#ui-module';

First post updated.
Actually, new developer can simply use relative paths until he will encounter the tsconfig.json's path option.
All we need is support of this ability in standard webpack configuration.

It seems I miss older similar issue #1465

Fixed by https://github.com/angular/angular-cli/pull/2470


[WIP] Docs and more

More improvements (mostly docs) in advance of the paper release.

Thank for the cleanup.
My only concern here is removing all the links from the sidebar. This is a bit of a matter of taste I think. Not everybody wants to click through many pages to get to the information they want.
Can we get some additional feedback from some users?

I had hoped to add the tutorials in a dropdown within the sidebar but wasn't sure how to do it at the time ... but I've managed it just now. I think (?) the only other page I removed from the sidebar was the Pkg info.

merge?

Just made a few tweaks to better incorporate the Molly tutorial. Merging.


Increase ExamineDetail Range from 3 to 8 tiles

About the PR
Increased the range of Character Description inspects from 3 to 8, allowing players to view others' character descriptions from much farther away.
Why / Balance
This is purely a Quality of Life change to make it easier to see other's character descriptions without having to awkwardly move super close to them, especially given many situations both RP-related and combat-related demand you to be standing in a certain spot away from the character you wish to learn more about.
This does not affect balance at all, as RMC gameplay does not have an element of deception and disguises/identity hiding like standard SS14 does, which is why the range was so low to begin with (as viewing someone in greater detail would always give the same description per-character, you can't fake this, ruining your disguise). You can not hide your identity at all as a marine, and the ability to do wouldn't be important to the gameplay anyway, so being able to see flavor text from afar should cause no balance issues.
Media
https://github.com/user-attachments/assets/fedbaff2-d071-4197-b18d-067703a4c3d7

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
:cl:

tweak: You can now view player character descriptions from much further away.

I think you should PR this upstream to wizden. This would be a nice feature for salamander.

I think you should PR this upstream to wizden. This would be a nice feature for salamander.

I have said in this PR description why the range is so low in upstream (it is exploitable in their gameplay loop) so I am not interested in upstreaming this at this time. If you wish you are free to make an attempt there yourself, but I do not believe it would be accepted there personally.

The reason why I mention it is because looking at the git blame that number was picked over 4 years ago and seems to be untouched. I don't think it is as exploitable as you say it is because many people leave their descriptions blank. I might make a PR myself if you are not interested- it really sucks to not be able to read people's flavor text.


[Release] Bump to 1.4.0

Summary
Upgrades the Recorder to v1.4.0.
Implementation details
Increases the Playwright and Synthetics deps.
How to validate this change
Standard testing. Run the recorder, run the test browser, test code gen.
Video that I recorded testing this:
https://github.com/elastic/synthetics-recorder/assets/15065076/f91c0832-cd3c-4af1-80c8-a1a92098c2af

@justinkambic thanks for the review!
I'm checking why the E2E step fails.
Locally I can reproduce the same error:

Is this situation familiar to you?

Locally I can reproduce the same error:

did you check if selectors have changed in anyway?

it seems the e2e tests have an issue related to "waiting for locator('[placeholder="Enter a starting URL"]')"
perhaps it is because the Electron window shows up empty, I'm checking 🧐

hmmm something is going on with one of the dependencies, I deleted package-json.lock and installed it everything again and it works now... let's see if I'm able to pinpoint the difference with the package-lock in this PR


add metrics to gossip

https://github.com/MystenLabs/sui/issues/2042

@gdanezis Sure, any plans for what kind of things would we like to trace?


IncomingMessage local and remote

Separates incoming messages by ChatActivity source to clean up testing code and help prevent races and CI flakes.
This also re-enables the TestChatSrvSetConvMinWriterRole, @patrickxb lmk if this still fails locally for you after the patch

I checked out your branch and the test still fails locally.

passed!


feat: patient segmentation

Review Checklist

Summary*

[ ] fix: Bug fix for Add the items tackled here as checklists
[x] feat: Completed task
[ ] chore: Incomplete task

[ ] test: Sub-task 1

Structure*

[ ] The Pull Request has a proper title that conforms to our MR title standards
[ ] The Pull Request has one commit, and if there are more than one, they should be squashed
[ ] The commit should have a proper title and a short description
[ ] The commit must be signed off
[ ] Unused imports are not present
[ ] Dead code is not present
[ ] Ensure dry running library to confirm changes

Tests

[ ] Proper and high quality unit, integration and acceptance(if applicable) tests have been written
[ ] The coverage threshold should not be lowered

Sign off*

[ ] All comments have been resolved by the reviewers
[ ] Approved by Czar {replace_with_czar_name}
[ ] Signed off by second reviewer {replace_with_name}
[ ] Ensure all checks are done before merging :warning:
[ ] All PRs needs to be signed before merging :warning:

N/B:

Add a checklist if more than one item is done.
Add screenshots and/or images where necessary
Indicate any breakages caused in the UI :exclamation:
Where necessary, indicate which issue the Pull Request solves (Closes #)
Any new files are updated in the folder structure in the README

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

-56 of 122 (54.1%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 81.718%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/clinical/infrastructure/services/pubsub/publisher.go
0
3
0.0%

pkg/clinical/usecases/clinical/questionnaire_response.go
14
18
77.78%

pkg/clinical/infrastructure/services/advantage/service.go
29
36
80.56%

pkg/clinical/presentation/rest/handlers.go
20
27
74.07%

pkg/clinical/presentation/config.go
1
15
6.67%

pkg/clinical/application/dto/enums.go
0
21
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
6812

Relevant Lines:
8336

💛  - Coveralls


Optimize dict encoding for non nullable dtypes

Right now we always get optional iterator, however, the optional value can be elided if we know the type is non nullable.

Branch predictor gets this right and there's no speedup given we are bottleneck on deduplication and its hashmap lookups


update habitat_lateral scripts to support additional species

omg this repo is BOSS
Current hack changes model/habitat_lateral/Makefile call to watershed groups list to exclude LEUT
WSG = $(shell $(PSQL) -AtX -c "SELECT watershed_group_code FROM bcfishpass.param_watersheds where watershed_group_code NOT IN ('ELKR', 'LEUT')")

Working again.  error was:
make[1]: Entering directory '/Users/airvine/Projects/repo/bcfishpass/model/habitat_lateral'
mkdir -p data/temp/LEUT
python valley_confinement.py LEUT -o data/temp/LEUT/valleys.tif -d data/temp/LEUT
2023-02-16 09:41:34,482 __main__     INFO     LEUT - Processing extent 960134.118,919154.936,1052995.159,965742.825
2023-02-16 09:41:34,623 __main__     INFO     LEUT - rasterizing streams
Traceback (most recent call last):
  File "/Users/airvine/Projects/repo/bcfishpass/model/habitat_lateral/valley_confinement.py", line 728, in <module>
    cli()
  File "/opt/homebrew/Caskroom/miniconda/base/envs/newgraph/lib/python3.9/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/opt/homebrew/Caskroom/miniconda/base/envs/newgraph/lib/python3.9/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/opt/homebrew/Caskroom/miniconda/base/envs/newgraph/lib/python3.9/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/opt/homebrew/Caskroom/miniconda/base/envs/newgraph/lib/python3.9/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/Users/airvine/Projects/repo/bcfishpass/model/habitat_lateral/valley_confinement.py", line 724, in cli
    valley_confinement(watershed_group_code, db_url, out_file, workdir, dem_path=dem, **config)
  File "/Users/airvine/Projects/repo/bcfishpass/model/habitat_lateral/valley_confinement.py", line 351, in valley_confinement
    get_streams(db, bounds, minimum_drainage_area, DEM, dem_meta, data_path)
  File "/Users/airvine/Projects/repo/bcfishpass/model/habitat_lateral/valley_confinement.py", line 208, in get_streams
    A = features.rasterize(
  File "/opt/homebrew/Caskroom/miniconda/base/envs/newgraph/lib/python3.9/site-packages/rasterio/env.py", line 394, in wrapper
    return f(*args, **kwds)
  File "/opt/homebrew/Caskroom/miniconda/base/envs/newgraph/lib/python3.9/site-packages/rasterio/features.py", line 327, in rasterize
    raise ValueError('No valid geometry objects found for rasterize')
ValueError: No valid geometry objects found for rasterize
make[1]: *** [Makefile:108: data/temp/LEUT/valleys.tif] Error 1
make[1]: Leaving directory '/Users/airvine/Projects/repo/bcfishpass/model/habitat_lateral'
make: *** [Makefile:222: model/habitat_lateral/.make/habitat_lateral] Error 2

lateral habitat is purely salmon/steelhead at the moment, so LEUT is not going to work.
The valley confinement script should work if LEUT was added to the list of watersheds included in param_watersheds but the habitat lateral processing in habitat_lateral.py would need updating to work more generally.

I'm thinking the easiest way to do this is make the assumption that the user has only generated access/habitat outputs for the species they are interested in. Build unconfined valleys for all streams defined as accessible to any species, and build lateral habitat for all streams noted as having habitat for any species.

Aha. Thanks!
you are probably already on it but I needed to make a few very minor changes to make the model/habitat_lateral/sql/habitat_lateral.sql script run (val as value in first table and add schema to call to drop habitat_lateral_disconnected_rail at the end of the script.
A few target paths needed to be updated as well in main Makefile
Changes are here. https://github.com/NewGraphEnvironment/bcfishpass/tree/ng

yeah, I think I have captured a few of those already. Haven't taken time to figure out how to get the merge of the output tifs to work yet though.


Adds last executive report to active incidents status page

Do you have a screenshot? Should we put the reports into tabs?

@kevgliss see attached screenshot. I think it looks good w/o tabs for now.


Bridge.Html5.FormData isn't reflectable or serialisable

Currently when attempting to serialise a FormData object it throws an exception relating to it not being reflectable:
System.Exception: Uncaught System.InvalidOperationException: FormData is not reflectable and cannot be serialized.
I'm not sure whether the Bridge.Newtonsoft.Json library would be able to serialise / de-serialise them if it were reflectable, as it's a JavaScript class rather than a C# type.
Here's a repro of the issue: https://deck.net/46ff934fec2feb727ffc8b33a05a14bf
Would it be possible for this type to be serialisable / reflectable? It's quite common for us to serialise FormData in our generic HTTP request helpers, and would mean that we don't have to have any special code to handle this case.

Try to use filter
https://deck.net/f781ab07d34068b7eabdd541c1d83a63

Hi @vladsch - thanks for the help, this is good to know for the future. On second thought what I'm requesting here doesn't really make sense, as if we serialise FormData there won't be a type to deserialise out as on the server side.
I feel that I should be able to just adapt our HTTP request helpers to ensure that when we send POST data we avoid serialising if the type is a JavaScript class. I'll close this off now!


entity-services-examples mldeploy fails

Unable to deploy examples.  Server Message: XDMP-NOSUCHDB: No such database entity-services-examples-schemas
:entity-services-examples:mlDeployApp FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':entity-services-examples:mlDeployApp'.
> Unable to complete bulk load, cause: Role does not exist

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

hm.  doesn't look like our issue does it..

I am guessing if recent updates/delete to security/roles in last PR has anything to do with this.

Races runs OK. Tried on 03/07 nightly build.
Versions-Example
entity-services-examples$ ./gradlew -PexampleDir=example-versions mlDeploy
:entity-services-examples:mlDeleteModuleTimestampsFile
Deleting /Users/sbalasub/git/entity-services/entity-services-examples/build/ml-last-configured-timestamps.properties

:entity-services-examples:mlPrepareRestApiDependencies
:entity-services-examples:mlDeployApp FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':entity-services-examples:mlDeployApp'.
> Unable to complete bulk load, cause: Role does not exist

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Suppliers-Example
entity-services-examples$ ./gradlew -PexampleDir=example-suppliers mlDeploy
:entity-services-examples:mlDeleteModuleTimestampsFile
Deleting /Users/sbalasub/git/entity-services/entity-services-examples/build/ml-last-configured-timestamps.properties

:entity-services-examples:mlPrepareRestApiDependencies
:entity-services-examples:mlDeployApp
Unable to load schemas because no schemas database exists; cause: Local message: write failed: Bad Request. Server Message: XDMP-NOSUCHDB: No such database entity-services-examples-schemas
:entity-services-examples:mlDeployApp FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':entity-services-examples:mlDeployApp'.
> Unable to complete bulk load, cause: Broken pipe

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Indeed, I neglected to copy ml-confg around to the various examples.  Didn't notice the problem because it does not surface unless you undeploy and redeploy each example anew, so great catch.
The fix is simple fortunately.  I'll have a commit and PR shortly.

Well, given today's packed schedule, it will probably take until tomorrow.

I'll re-push this branch in a moment.

Updated comments in #268 mldeploy works OK. run gives errors for all but versions.

makeing sure i can deploy and run anything before giving you another runaround.

Working as expected post fix. Shipping!


Programmatic-style node tutorial outdated

Hello,
its me again, i have a problem with the programmatic-style node tutorial, 2 on 3 imports are outdated.
import {
	IExecuteFunctions,
} from 'n8n-core';

import {
	OptionsWithUri,
} from 'request';

I found that the first one is changed by n8n-workflow, but what about the second ? and how i should change the code in the exemple to match the right import :
// Make HTTP request according to https://sendgrid.com/docs/api-reference/
			const options: OptionsWithUri = {
				headers: {
					'Accept': 'application/json',
				},
				method: 'PUT',
				body: {
					contacts: [
						data,
					],
				},
				uri: `https://api.sendgrid.com/v3/marketing/contacts`,
				json: true,
			};
			responseData = await this.helpers.requestWithAuthentication.call(this, 'friendGridApi', options);
			returnData.push(responseData);
		}

Thanks by advance !

can you please link to the tutorial you are using? I can then try to get it updated 🙏🏽

Sure ! there it is : https://docs.n8n.io/integrations/creating-nodes/build/programmatic-style-node/

I have created DOC-961 as the internal ticket to update the guides.

Thank you !