Export /etc/hosts from host to velum-dashboard pod, bsc#1062728

Docker copies /etc/hosts from host during starting container,
we would like to keep /etc/hosts file updated since it is managed by salt.
The reason for this is that velum has to resolve Kube API fqdn.
Problem was discovered and discribed in bug 1062728

@mjura are you sure this works? /etc/hosts is managed by Docker, so I guess you cannot just mount it...

yes, it works, I'm still testing it

Fix will be done in another way, moving to
https://github.com/kubic-project/salt/pull/265

This should work just fine, on my host docker handles that in the expected way:
docker run -ti --rm -v /etc/hosts:/etc/hosts:ro opensuse /bin/bash

That should work also with kubernetes managing the container.


Create feature-card-22.html

Made new PR and close the previous PR. I think the conflict was generated because there is one same-name component generated before.
#836

This one is great. @nik132-eng


influxdb cluster: tcp.Mux: handler not registered: 71

I have a problem with setting up a influxdb cluster. I did everything from the manual and on the node0 i've got error in logs:
[tcp] 2016/02/26 09:55:00 tcp.Mux: handler not registered: 71
And on my joining node1:
[metaclient] 2016/02/26 09:57:59 failure getting snapshot from influx0.gem.lan:8088: Get http://influx0.gem.lan:8088?index=0: read tcp <IP_ADDRESS>:8088: connection reset by peer
node0 (main) listenings:
tcp        0      0 <IP_ADDRESS>:8083       <IP_ADDRESS>:*               LISTEN      10231/influxd
tcp        0      0 <IP_ADDRESS>:8088       <IP_ADDRESS>:*               LISTEN      10231/influxd
tcp        0      0 <IP_ADDRESS>:8091       <IP_ADDRESS>:*               LISTEN      10231/influxd
tcp6       0      0 :::8086                 :::*                    LISTEN      10231/influxd
Tried to delete everything from /var/lib/influxdb/ on both servers, didn't worked.
My node1 /etc/default/influxdb:
INFLUXD_OPTS="-join <IP_ADDRESS>:8088"
Please help me solving it out.

Hi @keksior,
Can you provide more information, including steps to reproduce? How many nodes? How have you configured them? What version of Influx?

Ubuntu 14.04.03, influx 0.10.1-1. 2 nodes: node0,node1. I configured them like this:
node0:
/etc/influxdb/influxdb.conf
http://pastebin.com/fLmyDx5y
node1:
/etc/influxdb/influxdb.conf
http://pastebin.com/4SgW3hqz
on the node1 i also added into file /etc/default/influxdb this:
INFLUXD_OPTS="-join <IP_ADDRESS>:8088"

@keksior you need to join to the meta node's HTTP service. Port 8088 is used for internal node communication. Nodes use the HTTP service to discover these ports and hosts.
So, when starting node1 (assuming you already started node0), change the env var to: INFLUXD_OPTS="-join <IP_ADDRESS>:8091".

Now it added my node1 to meta_nodes, but not the data_nodes:
data_nodes
id	http_addr	tcp_addr
1	"localhost:8086"	"<IP_ADDRESS>:8088"
meta_nodes
id	http_addr	tcp_addr
1	"<IP_ADDRESS>:8091"	"<IP_ADDRESS>:8088"
2	"<IP_ADDRESS>:8091"	"<IP_ADDRESS>:8088"

Can you provide the logs for both nodes please?

node0:
http://pastebin.com/x441UmSC
node1:
http://pastebin.com/vWa06JVN

Can someone help me solving it out?

I found that setting ":8086" in config make localhost in "show servers" data_nodes. When i changed it to ip address and cleared on node0 /var/lib/influxdb/meta/* it worked everything. Thanks for help.

I still have a problem. I've got whole cluster running:
data_nodes
id	http_addr	tcp_addr
1	"<IP_ADDRESS>:8086"	"<IP_ADDRESS>:8088"
3	"<IP_ADDRESS>:8086"	"<IP_ADDRESS>:8088"
meta_nodes
id	http_addr	tcp_addr
1	"<IP_ADDRESS>:8091"	"<IP_ADDRESS>:8088"
2	"<IP_ADDRESS>:8091"	"<IP_ADDRESS>:8088"
retention:
name	duration	replicaN	default
default	"0"	2	true
But the data is not send to the secondary server. How can i debug this?

Hi @keksior,
Are you saying you're writing to <IP_ADDRESS>:8086 and the data is not propagating to <IP_ADDRESS>:8086? How are you verifying data is not being written to the second node>

I checked disk usage:
node0:
root@influx0:~# du -h --max-depth=1 /var/lib/influxdb/data
7.1G	/var/lib/influxdb/data/telegraf
1.4G	/var/lib/influxdb/data/_internal
8.5G	/var/lib/influxdb/data
node1:
root@influx1:~# du -h --max-depth=1 /var/lib/influxdb/data
408K	/var/lib/influxdb/data/_internal
1.4G	/var/lib/influxdb/data/telegraf
1.4G	/var/lib/influxdb/data

Can you reproduce this from startup with a fresh cluster?
If not can you provide the influx logs for a period where you know writes
have come in and not been propagated to the other node?
On Thursday, 3 March 2016, keksior<EMAIL_ADDRESS>wrote:

I checked disk usage:
node0:
root@influx0:~# du -h --max-depth=1 /var/lib/influxdb/data
7.1G /var/lib/influxdb/data/telegraf
1.4G /var/lib/influxdb/data/_internal
8.5G /var/lib/influxdb/data
node1:
root@influx1:~# du -h --max-depth=1 /var/lib/influxdb/data
408K /var/lib/influxdb/data/_internal
1.4G /var/lib/influxdb/data/telegraf
1.4G /var/lib/influxdb/data
—
Reply to this email directly or view it on GitHub
https://github.com/influxdata/influxdb/issues/5846#issuecomment-191650209
.

--
Edd Robinson
InfluxDB Engineer
▼▴
InfluxData.com http://influxdata.com/
Github http://www.github.com/e-dard / LinkedIn
https://www.linkedin.com/in/eddrobinson / Personal http://eddrobinson.io

I have data coming in to my influx every few seconds. The logs for influx0:
[http] 2016/03/03 09:54:08 <IP_ADDRESS> - telegraf [03/Mar/2016:09:54:08 +0100] POST /write?consistency=&db=telegraf&precision=s&rp= HTTP/1.1 204 0 - InfluxDBClient 7da0659c-e11d-11e5-8f99-000000000000 15.159911ms
[http] 2016/03/03 09:54:09 <IP_ADDRESS> - telegraf [03/Mar/2016:09:54:09 +0100] POST /write?consistency=&db=telegraf&precision=s&rp= HTTP/1.1 204 0 - InfluxDBClient 7e45acef-e11d-11e5-8f9a-000000000000 36.631127ms
[http] 2016/03/03 09:54:10 <IP_ADDRESS> - telegraf [03/Mar/2016:09:54:10 +0100] POST /write?consistency=&db=telegraf&precision=s&rp= HTTP/1.1 204 0 - InfluxDBClient 7ed5115d-e11d-11e5-8f9b-000000000000 38.597214ms
[http] 2016/03/03 09:54:10 <IP_ADDRESS> - telegraf [03/Mar/2016:09:54:10 +0100] POST /write?consistency=&db=telegraf&precision=s&rp= HTTP/1.1 204 0 - InfluxDBClient 7edcca3a-e11d-11e5-8f9c-000000000000 8.666437ms
[http] 2016/03/03 09:54:10 <IP_ADDRESS> - telegraf [03/Mar/2016:09:54:10 +0100] POST /write?consistency=&db=telegraf&precision=s&rp= HTTP/1.1 204 0 - InfluxDBClient 7ef6d669-e11d-11e5-8f9d-000000000000 24.970879ms
[http] 2016/03/03 09:54:10 <IP_ADDRESS> - telegraf [03/Mar/2016:09:54:10 +0100] POST /write?consistency=&db=telegraf&precision=s&rp= HTTP/1.1 204 0 - InfluxDBClient 7f126f4e-e11d-11e5-8f9e-000000000000 42.305598ms
[http] 2016/03/03 09:54:11 <IP_ADDRESS> - telegraf [03/Mar/2016:09:54:11 +0100] POST /write?consistency=&db=telegraf&precision=s&rp= HTTP/1.1 204 0 - InfluxDBClient 7fb33837-e11d-11e5-8f9f-000000000000 35.105411ms
[http] 2016/03/03 09:54:11 <IP_ADDRESS> - telegraf [03/Mar/2016:09:54:11 +0100] POST /write?consistency=&db=telegraf&precision=s&rp= HTTP/1.1 204 0 - InfluxDBClient 7fc7f7e2-e11d-11e5-8fa0-000000000000 24.820672ms
[http] 2016/03/03 09:54:11 <IP_ADDRESS> - telegraf [03/Mar/2016:09:54:11 +0100] POST /write?consistency=&db=telegraf&precision=s&rp= HTTP/1.1 204 0 - InfluxDBClient 7fd6cbe2-e11d-11e5-8fa1-000000000000 14.126165ms
[http] 2016/03/03 09:54:12 <IP_ADDRESS> - telegraf [03/Mar/2016:09:54:11 +0100] POST /write?consistency=&db=telegraf&precision=s&rp= HTTP/1.1 204 0 - InfluxDBClient 7fd84234-e11d-11e5-8fa2-000000000000 16.669366ms
And the last logs from the node1:
[tsm1] 2016/03/03 09:52:57 beginning level 3 compaction of group 0, 4 TSM files
[tsm1] 2016/03/03 09:52:57 compacting level 3 group (0) /var/lib/influxdb/data/telegraf/default/3/000000372-000000003.tsm (#0)
[tsm1] 2016/03/03 09:52:57 compacting level 3 group (0) /var/lib/influxdb/data/telegraf/default/3/000000376-000000003.tsm (#1)
[tsm1] 2016/03/03 09:52:57 compacting level 3 group (0) /var/lib/influxdb/data/telegraf/default/3/000000380-000000003.tsm (#2)
[tsm1] 2016/03/03 09:52:57 compacting level 3 group (0) /var/lib/influxdb/data/telegraf/default/3/000000384-000000003.tsm (#3)
[tsm1] 2016/03/03 09:53:28 compacted level 3 group (0) into /var/lib/influxdb/data/telegraf/default/3/000000384-000000004.tsm.tmp (#0)
[tsm1] 2016/03/03 09:53:28 compacted level 3 group 0 of 4 files into 1 files in 30.971793057s
[tsm1] 2016/03/03 09:53:29 beginning full compaction of group 0, 2 TSM files
[tsm1] 2016/03/03 09:53:29 compacting full group (0) /var/lib/influxdb/data/telegraf/default/3/000000368-000000005.tsm (#0)
[tsm1] 2016/03/03 09:53:29 compacting full group (0) /var/lib/influxdb/data/telegraf/default/3/000000384-000000004.tsm (#1)
I have nothing more than tsm1 logs in my node1 log file.

 Any ideas?

This is from an old version and we no longer support clustering in the open source version, but in case you're still trying to figure this out, you're probably using the wrong port somewhere in your configuration file. This error message happens when you try to make an HTTP (8086) request against the RPC port (8088). I can't remember which configuration option it was exactly, but that would be a good place to start with.
I'm going to close this as it's an issue we're not going to fix (the clustering code is dramatically different from what 0.10 had), but I hope this helps a bit with trying to figure out your configuration issue. I'm sorry we were not able to get to responding to this issue in a more timely manner.


Added update status actions for stealers.


Update tokens to 4.0.0

This PR is to

Update the Material Design tokens to 4.0.0

Since the last token update(#145356), only Slider tokens have been updated. The M3 Slider has significant visual changes, compared to the original M3 Slider, so a separate PR would be needed. I commented out SliderTemplate and added a TODO when we are ready to add new widgets again.
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] I followed the [breaking change policy] and added [Data Driven Fixes] where supported.
[x] All existing and new tests are passing.

FYI @TahaTesser regarding the slider changes

FYI @TahaTesser regarding the slider changes

Ohhh... I was using hard coded values with TODOs to use tokens later in https://github.com/flutter/flutter/pull/147783.

FYI @TahaTesser regarding the slider changes

Ohhh... I was using hard coded values with TODOs to use tokens later in #147783.

I don't know we have #147783🤩 Then looks like we can just remove this todo once your PR is ready! Thanks!

This PR removes th tokens for current M3 detaults so I will update https://github.com/flutter/flutter/pull/147783 to use new the tokens and restore delete values as we plan to keep the current M3 shapes under the new flag.


[consensus] fix the leaking quorum cert in consensusdb

Motivation
This was introduced in https://github.com/libra/libra/pull/2449, we're
not able to prune the last QC it introduced because we don't have
corresponding blocks.
Have you read the Contributing Guidelines on pull requests?
Y
Test Plan
Added asserts.
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

@davidiw the comments are on the function

@bors-libra r=davidiw

:pushpin: Commit f6dc86c has been approved by davidiw

:hourglass: Testing commit f6dc86c2dce6f7aa9e6409f644cafb83a76c918d with merge b603a366b809cf98087d0d7fe7dc87649467f590...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: davidiw
Pushing b603a366b809cf98087d0d7fe7dc87649467f590 to master...


POD(Hostnetwok) can't use LB service

Describe the bug
I have two pods, one pod is a httpd(server) and has been deployed on node1 and another pod(client) which is been deployed on node2 , the client pod  used hostNetwork.  the client pod can't visit server pod.
To Reproduce
Steps to reproduce the behavior:

login client(hostnetwork) pod
use command curl or wget  to visit server pod

Expected behavior
the client(hostnetwork) can visit the server pod.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS/Distro: [e.g. Ubuntu 1804]
Kubernetes Version: [e.g. v.1.18]
Kube-vip Version: [e.g. 0.2.3]

Kube-vip.yaml:
If Possible add in your kube-vip manifest (please remove anything that is confidential)
Additional context
Add any other context about the problem here.

As mentioned in the template, can we get the kube-vip manifest or any logs? You also don't specify how you're using kube-vip for the LB service.


Revert "add cache clean before test run"

revert of #51

I'll wait a little, I've been doing tests since yesterday.
Purging all my docker images and containers to try and replicate the env of GH actions.
I'll continue to investigate since the changes in #51 were minimal, there's no reasons it's not working, plus it worked when you created the PR, so that's another weird thing.

strangely enough looks like GH actions are working now. I only added debug output to understand after 2 push GH actions started to work, I'm clueless about this one, but we can close this PR as your original changes are good.
Thanks


A warning in SLSQP shows up with correct code

Environment: Windows 7 64-bit, Python 3.4 64-bit, SciPy version 0.15.1, NumPy version: 1.9.2.
Code to repeat the bug:
import numpy as np
from scipy.optimize import minimize

def func(x):
    return (x ** 2).sum()

bounds = ((-5, None), (-5, None))
res = minimize(func, np.ones(2), method='SLSQP', bounds=bounds)

Reported warning:

C:\WinPython3\python-3.4.3.amd64\lib\site-packages\scipy\optimize\slsqp.py:338: RuntimeWarning: invalid value encountered in greater
bnderr = where(bnds[:, 0] > bnds[:, 1])[0]

Thought:
In line 330 of slsqp.py, it says:
if bounds is None or len(bounds) == 0:
    xl, xu = array([-1.0E12]*n), array([1.0E12]*n)

This check would fail when only part of the tuple is None.

The check on line 330 is fine.  The warning comes from comparing against NaN.  The fix here is to either filter the warning or fix this code to stop relying on comparison to NaN in the call to np.where on line 338 (looks like 337 in master).

Note that the best fix to the infinite bounds issue in SLSQP would be to
skip adding the constraint entries to the matrix if the bounds are
infinite, as I think it was done in NLopt
http://ab-initio.mit.edu/wiki/index.php/NLopt

Fixed in https://github.com/scipy/scipy/pull/7704 ?

Yes, this was fixed then.


Error while compiling

After cloning the repository, when trying to compile I get the following error:

with python:
python setup.py install

You may try "g++ -std=gnu++11 -pthread -O3 fastBPE/main.cc -IfastBPE -o fast", which solves my problem using cygwin for win10. However i still failed with python wrapper.


Update 0000-01-01-intro.md

Added text to line#5

Closing the pull request

Closing the pull request


We can provide the information in collective way to the public regarding the different schemes in the covid 19 Hotspot areas, like direct income support related information

Is your feature request related to a problem? Please describe.
Due to Covid 19 pandemic , most hit is the poor families and by providing this feature we can help them in getting their survival
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
The poor people have the mobiles but they dont know , what programs are run by the govt which can help them monetary and material wise .
We can have a dropdown based based on the district which are the hotspot for the pandemic and can provide the info regarding the schemes what government is doing there , how to get that support in regional language so that they can use this information for there survival.
Describe the solution you'd like
Dropdown with the hotspot districts with the relevant info from the govt for the downtrodden.
Describe alternatives you've considered
NA
Additional context
Not available , but will try to make some mock up screens .

Related to #793 Please add a comment on that issue. Closing this as duplicate.


Feat(eos_designs,eos_cli_config_gen): Add support for MLAG port-channels to endpoints with PTP

Change Summary
Add support for connected endpoints running PTP on MLAG switches
Related Issue(s)
Fixes #3627
Component(s) name
arista.avd.eos_designs
Checklist
Repository Checklist

[x] My code has been rebased from devel before I start
[x] I have read the CONTRIBUTING document.
[x] My change requires a change to the documentation and documentation have been updated accordingly.
[x] I have updated molecule CI testing accordingly. (check the box if not applicable)

@pvinci-arista please test this PR which should address your request.


Any way to add more than 5 items?

It would be helpful and the best library if you can modify it to have more than 5 items in bottom bar.

I have try to change the max count by reflecting. But failed.
I guess that is because of compiler. It replace the int constant with number. So it is no use to change constant.
I am sorry to tell you, it can't be done by reflecting.


Unable to configure  Ingress rules with Arango DB in the kubernetes cluster.

My Environment

ArangoDB Version:        3.8.6 or 3.9.0
Deployment Mode:       Single server
Deployment Strategy:   Kubernetes
Configuration:
Infrastructure:               AWS
Operating System:        Ubuntu 20.04
Total RAM in your machine:      32Gb.
Disks in use:        HDD
Used Package:

Component, Query & Data
Affected feature:
Foxx
AQL query (if applicable):

AQL explain and/or profile (if applicable):

Dataset:

Size of your Dataset on disk:

Replication Factor & Number of Shards (Cluster only):

Steps to reproduce

deploy the Arango DB application in the Kubernetes cluster
setup the Ingress rule ( apiVersion: extensions/v1beta1
kind: Ingress
metadata:
annotations:
cert-manager.io/cluster-issuer: letsencrypt
kubernetes.io/ingress.class: nginx
nginx.ingress.kubernetes.io/enable-global-auth: "false"
nginx.ingress.kubernetes.io/rewrite-target: /$2
name: dataops-ingress
namespace: cndataops
spec:
rules:
- host: xyz.com
http:
paths:
- backend:
serviceName:
servicePort: 8529
path: /_db(/|$)(.*)$)

Problem:
{"error":true,"errorNum":11,"errorMessage":"not authorized to execute this request","code":401}
Expected result:
https://xxxxxxxx.com/_db/_system/_admin/aardvark/index.html#login
Login Page appear on the screen.

Hi, if this issue is still relevant, please re-open it at https://github.com/arangodb/kube-arangodb .


Testcontainers create two containers in a multi module project

Hello,
I am trying to use testcontainers for integration tests with postgres in a multi module project.
Here is the scenario:
Module A is a library and contains the services, repositories and entities (I am using JPA). The entitymanager factory and datasource are configured in a @configuration class.
Module B is a spring boot application that contains the controller and a dependency to module A to call the services/connect to database.
From module A I am using mockmvc to test the whole functionality (from controller to service).
I tried two approaches:

Using jdbc containers (https://www.testcontainers.org/modules/databases/jdbc/)
With this approach TWO containers are created for one test. First container is removed immediately after initializing and a new one is created when Module B is hit.

[Test worker] docker[postgres:latest]                  : Starting container with ID: d32556d8b2f355feea8ed61a66611db54e2199d48aa20eaca6e1f068ee5bd573
[Test worker] docker[postgres:latest]                  : Container postgres:latest is starting: d32556d8b2f355feea8ed61a66611db54e2199d48aa20eaca6e1f068ee5bd573
[...]
 [    Test worker] o.t.containers.output.WaitingConsumer    : STDERR: 2020-09-26 11:04:55.529 UTC [1] LOG:  database system is ready to accept connections
[    Test worker] org.testcontainers.ext.ScriptUtils       : Executing database script from file:src/test/resources/integration/init_database.sql
[...]
[    Test worker] org.testcontainers.ext.ScriptUtils       : Executed database script from file:src/test/resources/integration/init_database.sql in 245 ms.
 [    Test worker] o.testcontainers.utility.ResourceReaper  : Removed container and associated volume(s): postgres:latest
[...]
[    Test worker] docker[postgres:latest]                  : Creating container for image: postgres:latest
[    Test worker] o.t.utility.RegistryAuthLocator          : Looking up auth config for image: postgres:latest at registry: index.docker.io
[    Test worker] o.t.utility.RegistryAuthLocator          : No matching Auth Configs - falling back to defaultAuthConfig [null]
[    Test worker] o.t.d.a.AuthDelegatingDockerClientConfig : Effective auth config [null]
[    Test worker] docker[postgres:latest]                  : Starting container with ID: 5e9a660c0df6bd93c139bc05ef986a1360ceefdd7ede8c1438d28512978abf47
[    Test worker] docker[postgres:latest]                  : Container postgres:latest is starting: 5e9a660c0df6bd93c139bc05ef986a1360ceefdd7ede8c1438d28512978abf47

Using static containers
With this approach only one container gets created but when module B is hit it tries to connect to the real database, not to the containers. So it seems that the spring datasource properties are not overridden.
Caused by: org.postgresql.util.PSQLException: Connection to localhost:5432 refused. Check that the hostname and port are correct and that the postmaster is accepting TCP/IP connections.  at org.postgresql.core.v3.ConnectionFactoryImpl.openConnectionImpl(ConnectionFactoryImpl.java:285)  at org.postgresql.core.ConnectionFactory.openConnection(ConnectionFactory.java:49)  at org.postgresql.jdbc.PgConnection.<init>(PgConnection.java:217)  at org.postgresql.Driver.makeConnection(Driver.java:458)  at org.postgresql.Driver.connect(Driver.java:260)  at java.sql/java.sql.DriverManager.getConnection(DriverManager.java:677)  at java.sql/java.sql.DriverManager.getConnection(DriverManager.java:189)  at org.springframework.jdbc.datasource.DriverManagerDataSource.getConnectionFromDriverManager(DriverManagerDataSource.java:155)  at org.springframework.jdbc.datasource.DriverManagerDataSource.getConnectionFromDriver(DriverManagerDataSource.java:146)  at org.springframework.jdbc.datasource.AbstractDriverBasedDataSource.getConnectionFromDriver(AbstractDriverBasedDataSource.java:205)  at org.springframework.jdbc.datasource.AbstractDriverBasedDataSource.getConnection(AbstractDriverBasedDataSource.java:169)  at org.hibernate.engine.jdbc.connections.internal.DatasourceConnectionProviderImpl.getConnection(DatasourceConnectionProviderImpl.java:122)  at org.hibernate.internal.NonContextualJdbcConnectionAccess.obtainConnection(NonContextualJdbcConnectionAccess.java:38)  at org.hibernate.resource.jdbc.internal.LogicalConnectionManagedImpl.acquireConnectionIfNeeded(LogicalConnectionManagedImpl.java:104)

Can you please point me to a possible solution to this?

It is currently not possible to "share" a container between multiple JVMs unless you're using something like the reusable containers, although it would keep the container running even after Module B.
Regarding the 2nd solution, it seems that you have a misconfigured test setup because Testcontainers always randomizes the ports and localhost:5432 can only appear if you hardcode it and/or not setting it correctly. For that, please refer to your framework's docs.


Update python tools & CI.

Use googletest version 1.11.0.
Add __name__ checks for tools.
Use gcc-11 & g++-11.

Codecov Report

Merging #11 (99cac3b) into main (50d4975) will decrease coverage by 14.28%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##              main      #11       +/-   ##
============================================
- Coverage   100.00%   85.71%   -14.29%
============================================
  Files            2        2
  Lines            6        7        +1
============================================
  Hits             6        6
- Misses           0        1        +1

Impacted Files
Coverage Δ

source/test/unittests/test.cpp
50.00% <0.00%> (-50.00%)
:arrow_down:

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 50d4975...99cac3b. Read the comment docs.


notes: add openstackclient antelope

Signed-off-by: Christian Berendt<EMAIL_ADDRESS>

We'll add this after the next SCS release.


Fixed a bug on demo cart

The function names in the cart were not correctly cased. Causing the demo to break

"Causing the demo to break" - Could you expand on this? The demo runs completely fine, as the PICO-8 is case insensitive. Unless there's been an update to the console since this was originall written, changing this?

It didn't let me to change to the other functions with the arrow keys and it just kept in the first one. Because you try to search the array by its index. Pico8 is not case sensitive but lua array names still are :(

Hmm, interesting that you run into that issue, as it works fine for me and has done in the past on previous devices I've tested with too. I'll merge the request anyway for sake of compatability, but if you wouldn't mind letting me know what version of PICO-8 you're running, and platform? Thanks!

This is the version I tested it with. On windows 10!

This is what happens when I press the arrow keys. It just doesn't change to the next function and restarts the current one.

Thanks for the info! I'll check through the version notes when I get a chance and see when the bug was introduced. The last tested working version was on 0.1.11G.


Object doesn't support property or method 'xhr'

You have on the main page the following:
}).xhr(function(xhr){
  //access or attach event listeners to the underlying XMLHttpRequest
  xhr.upload.addEventListener(...)
});
/* return $http promise then,catch or finally.
Note that this promise does NOT have progress, abort or xhr functions */
var promise = upload.then(success, error, progress);
              upload.catch(errorCallback);
              upload.finally(callback, notifyCallback);

I try to use .xhr and get error "Object doesn't support property or method 'xhr'". So is there xhr or not? - that is the question!

This was documentation error. You should call it on the original upload. I have updated the documentation. After you call then() the promise returned is angular promise without any overriden methods (abort/xhr).


[stable/minio] Pod topology spread specification in statefulset.yaml

I am using Azure host Kubernetes service and using azure-disk as a default storage class. All my Kubernetes worker nodes are spread across three availability zones. I want to make sure the minio statfulset pods spread equally across these zones and use the PV from the respective zone. I see that topologySpreadConstraints was introduced in 1.16 and is available in 1.18. The current statefulset.yaml spec does not have the option to specify this constraint.

I have a simple patch to address this
--- a/kubernets/miniostablechart/templates/statefulset.yaml
+++ b/kubernets/miniostablechart/templates/statefulset.yaml
@@ -73,10 +73,12 @@ spec:
 {{ toYaml .Values.podAnnotations | trimSuffix "\n" | indent 8 }}
 {{- end }}
     spec:
+{{- if .Values.podTopologySpread.enabled }}
       topologySpreadConstraints:
-      - maxSkew: 1
-        topologyKey: zone
-        whenUnsatisfiable: DoNotSchedule
+{{- with .Values.podTopologySpread.spec }}
+{{ toYaml . | indent 8 }}
+{{- end }}
+{{- end }}
       {{- if .Values.priorityClassName }}
       priorityClassName: "{{ .Values.priorityClassName }}"
       {{- end }}
diff --git a/kubernets/miniostablechart/values.yaml b/kubernets/miniostablechart/values.yaml
index e8d311e..e542998 100755
--- a/kubernets/miniostablechart/values.yaml
+++ b/kubernets/miniostablechart/values.yaml
@@ -17,6 +17,23 @@ image:
   tag: RELEASE.2020-04-28T23-56-56Z
   pullPolicy: IfNotPresent

+## Pod Topology Spread specification for statefulset
+podTopologySpread:
+  enabled: false
+  spec: []
+#  - maxSkew: 1
+#    topologyKey: topology.kubernetes.io/zone
+#    whenUnsatisfiable: DoNotSchedule
+#    labelSelector:
+#      matchLabels:
+#        app: {{ template "minio.name" . }}
+#  - maxSkew: 1
+#    topologyKey: kubernetes.io/hostname
+#    whenUnsatisfiable: DoNotSchedule
+#    labelSelector:
+#    matchLabels:
+#      app: {{ template "minio.name" . }}
+
 ## Set default image, imageTag, and imagePullPolicy for the `mc` (the minio
 ## client used to create a default bucket).
 ##


Paginate balances endpoint

What is needed?
Currently we have some long balances lists, the idea of this issue is to paginate this list to split the list in several pages.
Related issues
https://github.com/safe-global/safe-transaction-service/issues/1408
Current endpoint
/api/v1/safes/{safe-address}/balances/
New endpoint
GET /api/v2/safes/{safe-address}/balances/
Response
Paginated list of tokens
{
 "count": <num_tokens>,
 "next": "...",
"previous": "...",
"results": [...]
}

@Uxio0 after this PR https://github.com/safe-global/safe-transaction-service/pull/1446 this doesn't looks necessary, what do you think?

@Uxio0 after this PR https://github.com/safe-global/safe-transaction-service/pull/1446 this doesn't looks necessary, what do you think?

Not for now, but I think we should paginate balances at some point so I would leave this issue open for the future


Synchronization with TV cursor when mouse acceleration != 1

Hi there,
many thanks for your nice app!
I had some discussion on the server project (Wouterdek/magic4pc#25) about the position of the TV cursor (which seemingly cannot be turned off) and the mouse pointer. It turned out that I had mouse acceleration at factor 2 which made the two pointers moving at different speed.
I found this a bit unexpected as your code emulates a touchpad and sets absolute positions. But I assume that this has something to do how the mouse acceleration is implemented in the Desktop / X11 window system.
So this is rather not a bug. Maybe this could be documented in the README.md.

Interesting, that's precisely why I chose to go with the touchpad interface to avoid that issue 😅. Thanks for reporting. It's been a while since I touched this so don't have enough context to document it right now, but I'll try to look at it soon (and if something can be done about it).


RuntimeError: invalid hash value

RuntimeError: invalid hash value (expected "ce0d4300", got "a0a8c5ad919066a582f73746eb7f2ee742e282baab9130860ca03fe1f17ead75")
the code I used was
model = pretrainedmodels.dict['se_resnet50'](num_classes=1000, pretrained='imagenet')

I don't not what happened, I tried it again and it's OK

I also encounter this problem.Maybe there's something wrong with the server.

I met the same problem. How did you solve it，


[Feature Request] Support for ChromeOS

Hey again :)
As the tittle says... Would be nice to make an arm compatible version for Chromebooks, specially for those with Rockchip RK3399 2.0GHz CPU, wich is the most common on that computers becuase of the direct Google support. Now a days there are no reliable ssh clients for ChromeOS. I can give support and test/debug or provide logs if its needed. Should be really helpful for people which uses their Chromebooks for developing/IT proposes.
Thanks in advance :)

I'm not familiar with ChromeOS - is the arch armel, armhf or arm64?

And also, what kind of packaging is expected

I'm not familiar with ChromeOS - is the arch armel, armhf or arm64?

All three, there are different models. Most common i think its armhf

And also, what kind of packaging is expected

.deb as its natively supported on ChromeOS :)


Add forgotten notify action on Makefile

Summary
Add forgotten notify action on Makefile
Release Note

Add forgotten notify action on Makefile

@stafot Is this ready for review?

@hanzei Yeap, just wasn't urgent thing.


DConf 2016 logo (full width orange)

Alternative to #94.
The included version of the logo doesn't depend on a local font (uses <path>s instead of <text>s).

thx!


Support policy owner

Summary
Transactions will now have a distinct owner within the transaction policy owner field. For more context, see https://github.com/FuelLabs/fuel-specs/issues/617.

This new policy would specify an index indicating which transaction input is considered the "owner" of the transaction. This feature would enable developers to identify the transaction initiator.

Blocked by:

https://github.com/FuelLabs/fuel-specs/pull/618
https://github.com/FuelLabs/fuel-vm/pull/872

Closed await release of new fuel-core version.


Right click to add components to the inspector

Currently all components are available in the inspector window, consuming most of the space. By using the right click to create a new window we could categorize and expand the options for each component to be added

Thanks for @naomijub and #69


Some apps not compatible

Some apps that are compatible with factory OS are not compatible in GammaOS, for example Netflix. I don’t plan on watching Netflix but they have some games available through their app that perform a handshake authentication. Netflix forces portrait mode, and freezes on the logo, and errors out force closing itself.
Sorry if this is a nonissue issue.
Device: rg353v
GammaOS full v1.3.3

To add more information. Netflix didn’t show available for install from Play Store and used aurora app store

Another app is forced into portrait

V1.5.1 still has the same issue, but Netflix now shows in the play store but still crashes.

something might be wrong with the internet connection because i can't install apks downloaded on the device by using browsers but i can instal the same files downloaded on pc

something might be wrong with the internet connection because i can't install apks downloaded on the device by using browsers but i can instal the same files downloaded on pc

have you tried netflix, only for the games?


Add DM.SITEID to LB checks

Consider adding SITEID to LB checks:
check_lb_lbstresn_missing (LB missing standard value), with variables:
USUBJID | LBTESTCD | LBDTC | LBORRES | LBORRESU | LBSTRESN | LBSTRESC | RAVE | VISIT
check_lb_lbstresu (LB missing standard unit), with variables:
USUBJID | LBTESTCD | LBORRES | LBSTRESU | LBSTRESC | LBDTC | RAVE | VISIT

@AlcJ123 can you please add the DM dataset as a new parameter to the two functions Sara listed about.  If DM$SITEID exists  please left_join it into LB by USUBJID.  DM %>% select_any(c("USUBJID","SITEID"))) before left_join-ing should work well.
Please make sure SITEID is returned if any issues are flagged, again potentially using select(any_of())


No defmodule keyword found

Hi,
today I realized that if at the beginning of a module, defmodule is not defined, for example:
(foo
  (export (fun-name 1)))

the compiler just ends, the file is not compiled (of course, because no module is defined) and no message is shown, just like if the compilation process was fine. I don't know if this is done on purpose but I think it should show a message, what do you think?

Perhaps the trouble is that not all LFE source is destined to be a module, e.g. include files. Not sure how best to address this.

it just returns () or #(ok ()) indicating that no modules were generated. Seeing there were no modules then nothing was really compiled, hence no errors.
Compiling has 2 phases: first the file is split into separate modules; and then each module is compiled separately. Macros before the fir st module are global to the whole file, while macros defined "inside" each module is local to it.
Anything else defined before the first module is copied into each module, so any function defs and other forms go into the modules are processed there.
In this case as there are no modules then nothing happens and everything is just discarded. It is not an error to have an "empty" file.
I wrote a bit about it in a blog on lfe.io

Interesting thing, thanks for this detailed explanation.


Bump CMakeList version

also try supporting bios module on freebsd

For some reaason Windows binaries were not packaged into a zip file. Can you have a look?


Fix confusion around how public add() method works

Rename _prepSource, change add() method signature

Renames this._prepSource to this._buildIndexFromSource, and also
changes it to always use this.index and this.source vs. allowing
passed in values. All calls to the method were passing those values
already anyways.
Remove keys argument to public add() method. Now we just use
this.keys, which is set to the value for keys set on
instantiation of the FuzzySearch instance.
Rename internal variable names in public add() method to prevent
confusion between an un-prepped item and source.
Consistent use of variable name source_item when were dealing with an un-prepped item (previously found in _prepSource(), now propagated to add() and prepItem().
Adds docs to README
Adds tests using Mocha, Chai, and JSDom

Discussion on why these changes were made can be found here:
https://github.com/jeancroy/FuzzySearch/issues/9
In summary, the public add() method had some ambiguity on how it
behaved which sometimes led to unexpected results (i.e. item missing
in search results after you add it).

How do you plan to use source together with add ?
I ask because the use case in my head is that server will suggest entries but slower than our instant search. Unless the server keep an history, it's very likely to send duplicate. Do we care about avoiding duplicate ? If so, an unconditional this.source.push is not going to suffice.

Also I keep babbling about a private _add.
The reason I keep babbling about a private add is this
for (var item_index = -1; ++item_index < nb_items;) {
            var source_item = this.source[item_index];
            this.add(source_item);
}

// ...

add: function(source_item){
   //...
   this.source.push(source_item);
}

There's some kind of "circular reference" if we use the same add internally.
As you currently have setup things, source will double length each time we recompute index, I think.

Add tests! Uses Mocha, Chai,

Yay !

and JSDom (because access to window is presumed, but this could go away eventually)

No ...
I first test for the existence of window and window performance timer. - Only if both exist I use them, else simply date.now(). I never presume window exist, I check for it, does it make sens ?.
If people keep tripping over that detail, I may remove it.
I use it to measure how long a search take & adjust debounce time.
I used performance counter because the thing we are trying to measure is similar of the incertitude of date.now(). I also did not used nodeJS high resolution time, because I do not think server has to debounce key presses.

Maybe the most simple thing is to rename current add index
then

add: function(source_item)(){
     this.source.push(source_item)
     this.index(source_item)
},

index: function(source_item){
  // current add
},

_buildIndexFromSource: function () {
    // ...
   // foreach item
       this.index(source_item)
}

that way if anyone has a special need I'll direct them to build their own add function using index.


add Adress

add class Adress
add methods to class Record for working with the user address
add commands to adress_book

Add adress


How did you overcome the error "Parameter name 'file' is invalid?

Hi there, I'm implementing my own package for the OCR Space api, but whenever I made a request I get the following error:
{
  "OCRExitCode": 99,
  "IsErroredOnProcessing": true,
  "ErrorMessage": [
    "Parameter name 'file' is invalid. Valid parameters: apikey,url,language,isoverlayrequired,base64image,iscreatesearchablepdf,issearchablepdfhidetextlayer,filetype,addressparsing,scale,detectorientation,istable,ocrengine,detectcheckbox,checkboxtemplate,checkboxtemplateregex",
    "Please check if you need to URL encode the URL passed in request parameters."
  ],
  "ProcessingTimeInMilliseconds": "2652
}

Have you faced it before? If that where the case, what did you do? Thanks in advance.

Show us the code of your request.
You are probably passing a file parameter instead of calling the OCR API like this:
const res = await ocrSpace('/path/to/file.pdf', { apiKey: '<API_KEY_HERE>' })

Nvm, I was sending both file and url in the request, that was the reason of the error. I'm closing this, thank you.


Fractional seconds not presented correctly

It appears that fractional seconds are being treated as milliseconds, which is not what they are.
There also seems to be a difference in how fractional seconds are shown in Arrow class representation (incorrect) and with using a format string.
The Arrow class representation treats fractional seconds as 'number of microseconds.'
Problems with representation:
200 milliseconds represented as 2 microseconds

arrow.get('2013-05-05T12:30:45.2+0030')
<Arrow [2013-05-05T12:30:45.000002+00:30]>

2 milliseconds represented as 2 microseconds

arrow.get('2013-05-05T12:30:45.002+0030')
<Arrow [2013-05-05T12:30:45.000002+00:30]>

Problems with format:
Each of the examples of displaying 2 milliseconds is incorrect.

arrow.get('2013-05-05T12:30:45.002+0030').format('YYYY-MM-DD HH:mm:ss.SSS ZZ')
'2013-05-05 12:30:45.0 +00:30'

arrow.get('2013-05-05T12:30:45.002+0030').format('YYYY-MM-DD HH:mm:ss.SSSSS ZZ')
'2013-05-05 12:30:45.0 +00:30'

arrow.get('2013-05-05T12:30:45.002+0030').format('YYYY-MM-DD HH:mm:ss.SSSSSS ZZ')
'2013-05-05 12:30:45.2 +00:30'

arrow.get('2013-05-05T12:30:45.002+0030').format('YYYY-MM-DD HH:mm:ss.SSSSSSS ZZ')
'2013-05-05 12:30:45.20 +00:30'

This was just fixed :)


[BUG] Project always rebuilds when using Content + PreserveNewest

Using the reference project
https://github.com/bradphelan/msbuild-incremental-build-fail
which includes the simple project file
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>netstandard2.0</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <Content Include="Foo\TextFile1.txt" CopyToOutputDirectory="PreserveNewest" />
  </ItemGroup>

</Project>

every time build is initiated it rebuilds.
My build log is
https://gist.github.com/bradphelan/c6bd0f32d62c09d7cb1e7a164b0f00af
and my visual studio information is

Microsoft Visual Studio Professional 2017
Version 15.4.0
VisualStudio.15.Release/15.4.0+27004.2002
Microsoft .NET Framework
Version 4.7.02046
Installed Version: Professional
Visual Basic 2017   00370-10200-08842-AA207
Microsoft Visual Basic 2017
Visual C# 2017   00370-10200-08842-AA207
Microsoft Visual C# 2017
Visual F# 4.1   00370-10200-08842-AA207
Microsoft Visual F# 4.1
Application Insights Tools for Visual Studio Package   8.9.00809.2
Application Insights Tools for Visual Studio
ASP.NET and Web Tools 2017   15.0.30925.0
ASP.NET and Web Tools 2017
ASP.NET Core Razor Language Services   1.0
Provides languages services for ASP.NET Core Razor.
ASP.NET Web Frameworks and Tools 2017   5.2.50921.0
For additional information, visit https://www.asp.net/
Azure App Service Tools v3.0.0   15.0.30915.0
Azure App Service Tools v3.0.0
Common Azure Tools   1.10
Provides common services for use by Azure Mobile Services and Microsoft Azure Tools.
JavaScript Language Service   2.0
JavaScript Language Service
JetBrains ReSharper Ultimate 2017.2    Build 109.0.20170824.131346
JetBrains ReSharper Ultimate package for Microsoft Visual Studio. For more information about ReSharper Ultimate, visit http://www.jetbrains.com/resharper. Copyright © 2017 JetBrains, Inc.
JustDecompilePackage2017 Extension   1.0
JustDecompilePackage2017 Visual Studio Extension Detailed Info
Microsoft Azure Tools   2.9
Microsoft Azure Tools for Microsoft Visual Studio 2017 - v2.9.50719.1
Microsoft Continuous Delivery Tools for Visual Studio   0.3
Simplifying the configuration of continuous build integration and continuous build delivery from within the Visual Studio IDE.
Microsoft JVM Debugger   1.0
Provides support for connecting the Visual Studio debugger to JDWP compatible Java Virtual Machines
Microsoft MI-Based Debugger   1.0
Provides support for connecting Visual Studio to MI compatible debuggers
Multilingual App Toolkit   4.0
Multilingual App Toolkit helps you localize your Windows Store app by providing file management, pseudo and machine translation, translation editor, and build integration. http://aka.ms/matinstall
NuGet Package Manager   4.4.0
NuGet Package Manager in Visual Studio. For more information about NuGet, visit http://docs.nuget.org/.
Project File Tools   1.0.1
Provides Intellisense and other tooling for XML based project files such as .csproj and .vbproj files.
RunCustomToolPackage Extension   1.0
RunCustomToolPackage Visual Studio Extension Detailed Info
SQL Server Data Tools   15.1.61707.200
Microsoft SQL Server Data Tools
TargetFrameworkMigrator   1.0
Bulk Change .Net Projects Target Frameworks
TypeScript   <IP_ADDRESS>
TypeScript tools for Visual Studio
Visual Studio Code Debug Adapter Host Package   1.0
Interop layer for hosting Visual Studio Code debug adapters in Visual Studio
VSColorOutput   2.5.1
Color output for build and debug windows - http://mike-ward.net/vscoloroutput
VsVim   <IP_ADDRESS>
VsVim is a Vim emulator for Visual Studio
WebJobs Tools v1.0.0   15.0.30915.0
WebJobs Tools v1.0.0

An interesting note is that if the project is changed to
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>netstandard2.0</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <Content Include="Foo\TextFile1.txt" CopyToOutputDirectory="Always" />
  </ItemGroup>

</Project>

It builds the first time and the second time and doesn't rebuild the third time. This seems to be the opposite of the advice from the internet.

I have now replaced PreserveNewest with CopyAlways across 25 projects and now my builds work perfectly. That is if nothing has changed then the build take 1.5 seconds to check and then finishes.


Fix test instability for trigonometric functions

@kmaehashi thanks for this fix.
Running 10k repetitions, Arcsin's forward_gpu still seems a bit unstable. Could you fix this too?

Thanks!
To adjust forward tolerance I added forward_options to unary_math_function_test decorator.
Confirmed forward 10k repeats.

LGTM! Will merge after tests!

jenkins test this please

jenkins, test this please.

LGTM!


update jp locale

Translated many and remove unused translation keys.
Plurals should be translated like them in japanese:
Factory:工場
1 factory:1つの工場
2 factorys:2つの工場
But I can't find how to do in Localisation tutorial
what it should be?

Thanks a lot for the effort! Translating plurals like that is not currently supported by the mod, it's not a limitation of the translation system but of the mod I think.


[QUESTION] Multiple collection listeners

Hey guys,
Sorry if this is not the right place to ask this but:
Let's assume that I have two different screens presenting the data from the same collection, but presenting it differently. I have two different view models that will process the data differently to make it ready for each screen to present.
In this case, would it be recommended to have only one listener and write a mechanism to notify both view models using this firebase listener? Or is it fine to setup two different firebase listeners for each view model?
Thanks so much for all the hard work.

Can you clarify if this is Firestore or RTDB? Based on your mention of collections and listeners I suspect it's Firestore, but it would be good to confirm.

@morganchen12 yes, it's Firestore

From a billing perspective, each query is distinct. In the local cache and for network purposes updates to documents that match both queries are deduplicated.
So in short: it would be slightly cheaper to use a single listener and perform your own filtering but not any more performant.

@filipealva Just as a point of clarification, are both listeners listening to an identical query (i.e. the same collection, the same "where" filters, and the same OrderBy)?  Or are they performing distinct queries against the same collection?

So in short: it would be slightly cheaper to use a single listener and perform your own filtering but not any more performant.

Cool. I would imagine that it would be slightly cheaper, but I think that it compensates not having to deal with the code to perform our own filtering.

Just as a point of clarification, are both listeners listening to an identical query (i.e. the same collection, the same "where" filters, and the same OrderBy)? Or are they performing distinct queries against the same collection?

The queries are exactly the same.
Just to give more context: we are deciding some details about dealing with Firebase internally and, as I said before, for me it worth it to avoid adding more code to "re-handle" listeners if the only disadvantage is that it would cost a little bit more for us. I mean, one of the points of using Firebase is to speed up the process and get rid of complexities, so...

The Firestore SDK will completely deduplicate identical queries. The previous answer covers the case where you're running two queries over the same data with different filters or orderBys.

So it wouldn't even be more expensive to use two listeners for this case?

For completely identical queries we only issue the query to the backend once. That's what you get billed for--so yes, it doesn't even cost you more.


[core] Incorrect decompiling

classes5.zip
Use jadx latest unstable.

class: com.xiaomi.hm.health.bt.device.HMDeviceSource

Instead of lines:
put(Integer.valueOf(HMDeviceSource.a(50, 256)), HMDeviceSource.OTHER_MOZART);
Must be code using method b, not a:
put(Integer.valueOf(HMDeviceSource.b(50, 256)), HMDeviceSource.OTHER_MOZART);

Sample from Luyten:

It looks like there the method HMDeviceSource.a(int,int) is missing in the Jadx decompiled code. I can see it in the smali code:
.method public static synthetic a(II)I
    .registers 2
    .line 1
    invoke-static {p0, p1}, Lcom/xiaomi/hm/health/bt/device/HMDeviceSource;->b(II)I
    move-result p0
    return p0
.end method

As you can see this method just calls HMDeviceSource.b(int,int). If this method would exists the code would be equivalent to the code generated by Luyten. I don't know why Luyten does not show this indirection through the a method (may be optimized in a way by dex2jar?).

may be optimized in a way by dex2jar?

Not. I have not used dex2jar. Just pulled the dex file from apk.
Earlier, jadx did "not stumble" on this code. In which version it worked correctly, I no longer remember.

I tracked it down, the method is not decompiled because it has the flag synthetic and is a method of a enum class:
https://github.com/skylot/jadx/blob/master/jadx-core/src/main/java/jadx/core/dex/visitors/EnumVisitor.java#L306
@skylot I am not very familiar with synthetic methods in enum classes but it seems to me like that we can't just hide all synthetic methods. Especially if they are referenced by other code parts.

Thank you so much for responding to my message. I hope @skylot will hear you.  Just removing isSynthetic from condition resolve issue and also allow jadx to replace a call with b. Looks like I made such removing due to possible obfuscation of standard methods, see TODO on this method :(

@skylot Do you remember which type of methods should be removed by the isSynthetic check?
And that the current version is not able to hide the obfuscated methods of valueOf and values is in my opinion not that big problem. When Jadx has to deal with obfuscated code errors are inevitable, we can't foresee all the strange modifications the obfuscators can apply.

Do you remember which type of methods should be removed by the isSynthetic check?

I tried to found one yesterday but looks like there are no such methods.

And that the current version is not able to hide the obfuscated methods of valueOf and values is in my opinion not that big problem

Agree, except values which uses removed $VALUES field, so it must be also removed.
I think it is safe to remove all methods which are uses $VALUES field. I will make such a change shortly.
Also looks like valueOf it is not safe to remove by name because it can be overriden and have different code. From java doc:

Note that for a particular enum type T, the implicitly declared public static T valueOf(String) method on that enum may be used instead of this method to map from a name to the corresponding enum constant.

I made a fix. Hope it doesn't break anything :)


[Feature]: quantum utils docstrings and tests

Feature Description
We are looking for someone to write docstrings and tests for all of the functions in the file src/quantum_utils.jl.
Some docstrings are in place, but a lot are not. The extant ones, if necessary, should be expanded to include a description of the function and the arguments, in line with the blue style guidelines.
Implementation details

Write docstrings for all of the functions in src/quantum_utils.jl
Add tests for the functions in test/quantum_utils_test.jl

Importance
1 (lowest)
What does this feature affect?

[ ] quantum system construction
[ ] problem setup
[ ] problem solution
[ ] problem performance
[ ] solution analysis
[ ] plotting
[X] documentation
[X] tests
[ ] other (please specify below)

Other information
No response

Sounds interesting! I would like to work on this.

@Fe-r-oz It's yours to pursue! Check the contribution guide for some notes regarding testing.
Feedback on the naming and interpretability of the utilities is appreciated at this time. We also welcome suggestions for missing utilities, filling gaps or improving what has currently been made available.

Hi, @aarontrowbridge, Kindly please review the PR https://github.com/aarontrowbridge/QuantumCollocation.jl/pull/111. All the tests passed passed.
Since it has both documentation and testing, it's a little long.  But I followed the best practices from the guide See also, this guide..  If there is something wrong, please let me know in the review.
I also included comments to better expain each and every test. I hope the PR satisfies the requirements. Thank you.

Hi, Please close this issue as it's resolved! Thanks!


Allow to run 'check' and detect new files created (with 'recursive' option)

Just a thought to consider.
When I run rhash -r dir -o output.file and later check it rhash -c output.file, it won't detect new files created in this directory.
Would be nice if one could combine  -c option with -r like rhash -r dir -c output.file, so it would check hashes from output.file and also detect if new files were created in that directory.

As a workaround you should run the following commands:
rhash -u output.file
rhash -c output.file

It probably would be useful to allow combining of -u and -c options to avoid re-checking of newly added files.

FR is implemented by commit

60528db61a0201a99f99e122f3bae36dbf550388 Allow simultaneous usage of --update and --check options.

It got included into the RHash v1.4.3 release.


Feature Request: HEVC / h265 support

Since you are using the nvEncodeAPI for encoding, a good thing to have would be HEVC encoding. This is much faster and allows for higher output resolutions (up to 8k at near realtime speeds).

we are using OS-provided encoder because it is an only practical solution for us. there is no plan to work on hardware encoder for now. ( https://github.com/unity3d-jp/FrameCapturer/issues/17 )


Monorepo support [design suggestion]

The most straightforward and probably the easiest to implement approach for monorepos is probably adding a prefix to all models in a subdir equal to that subdir, i.e.:
gto annotate foo/bar/a-model --path foo/bar/data/model.pkl

Will write the stuff to foo/bar/artifacts.yaml. Helpfully / is not an allowed symbol now, so this is not ambiguous and backwards compatible.
Design choices to make:

may cut name and path to a-model and data/model.pkl before writing to artifacts file or not
may restrict referring outside of the subdir or not, or treat it as external/whatever
may support addressing by short name while inside dir or not, i.e. gto register a-model --bump-major

Regardless of whether name is cut in artifacts.yaml or not the real model name will stay foo/bar/a-model, which might be used to create tags the usual way with not clashes.
P.S. I still think whether this feature should be added or not should be seriously discussed first.

We're getting back to this discussion in the context of merging artifacts.yaml part into DVC #337
I'll start from another question to discuss, but it's quite related to the one Alex posted above. A big question to me is whether we want to allow artifacts with the same ID in different dvc.yaml files (NOT a monorepo case). So far my understanding was that we shouldn't, but since it's allowed for stages and plots, I'm not that sure.
Initially we had all artifacts in artifacts.yaml in the root and that worked. For user and Studio connection was straightforward: name of artifact <> name used in Git tag. However, now we can get:
# dvc.yaml
nn:
  path: models/nn.pkl

# dvclive/dvc.yaml
nn:
  path: models/my-neural-net.h5

Now if I create a Git tag<EMAIL_ADDRESS>which artifact does it register version for? It's unclear.

The first option is to disallow same names. It's simpler. And if I reorganize my folders and instead of dvclive I'll be using live folder, we won't lose connection between old Git tags and new ones.
The second option here is to use different Git tags<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(makes it harder to read, and can feel somewhat overcomplicated to user after running dvclive.log_artifact). The "pro" here is more flexibility.

Now, back to monorepo case and multiple DVC subrepos. We have this issue for that.
In this issue Alex suggested to simply use the above mechanics ^ (option 2) for that case. To properly support monorepos, I guess we need to use it here (don't see any other solution now). This can be still combined with Option 1 or Option 2 from above for the first question. I prefer Option 1 there, but please let me know WDYT. @dberenbaum @shcheklein

So far my understanding was that we shouldn't, but since it's allowed for stages and plots, I'm not that sure.

I'm not sure this is intended behavior and is probably more of an oversight. I agree that IDs should be unique.

Good points @daavoo. I'm fine with option 2.

We can't know if its a monorepo or a dvclive scenario, I think we should treat them the same.

Oh yes, you're right. I assumed monorepo case to be the case when you do dvc init in subrepos, but monorepo can exist without it.
Then in Studio it will look somewhat like this (I made up this example). We have three models with the same name in a single repo, but written to different dvc.yaml files. So they appear as 3 different models wrt their dvc.yamls:

So, following the possible artifact reference in dvc get . dvclive:models/def-detector, we need to register/promote artifact named dvclive:models/def-detector.
Posting some options to replace : delimiter in Git tags (since : isn't allowed):
$ git tag<EMAIL_ADDRESS>-m "dwda"
$ git tag<EMAIL_ADDRESS>-m "dwda"
$ git tag<EMAIL_ADDRESS>-m "dwda"
$ git tag dvclive<EMAIL_ADDRESS>-m "dwda"
$ git tag<EMAIL_ADDRESS>-m "dwda"
$ git tag<EMAIL_ADDRESS>-m "dwda"

@daavoo If I have a monorepo with multiple .dvc dirs, how will/should we handle the artifacts and other items? Are they referenced from the root of the git repo or the dvc dir/repo?

In Git tags we should reference them from the root of the Git repo - otherwise there is no way to distinguish them from each other.
Just tested this: if you have

.
├── .git
├── .dvc
├── subrepo
│   └── .dvc
│   └── dvc.yaml
└── detection
    └── dvc.yaml

then running dvc.repo.Repo(".").metrics.show() won't return metrics you have in subrepo/.
Also I'm not sure if it's possible to do anything with subrepo/dvc.yaml (download outs, run exp) while your're in in Git repo root.

Nested DVC repos (where you have a top level .dvc dir and then another .dvc dir inside a subdirectory) are not supported.
Having multiple DVC repos within a single Git repo is supported though, like:
.
├── .git
├── dvcrepo1
│   └── .dvc
│   └── dvc.yaml
└── dvcrepo2
    └── .dvc
    └── dvc.yaml

This was actually implemented, so closing.


Basic Editor functionality

Now has the ability to add circles, rectangles, and lines to the canvas while also mutating their preportions. Save has been implemented client-side but not yet connected to the database

Additional commits contain the delete functionality along with the ability to create text boxes

@Battletortoise It looks like there are some conflicts for this one! When you get a chance I'll happily review and merge afterwards


1>CSC : error SI0000: Exception: The hintName contains an invalid character ':' at position 13.

Sometime, at the second compilation , it gives me
1>CSC : error SI0000: Exception: The hintName contains an invalid character ':' at position 13.
I think that is from
context.AddSource(GetSourceFileName(namespaceName, jsonFile.Path),
SourceText.From(generatedCode, Encoding.UTF8));
Could you modify
private string GetSourceFileName(string namespaceName, string path)
{
var fullPath = Path.GetFullPath(path)
.Replace(Path.GetFullPath("."), string.Empty)
.Replace("/", "")
.Replace("\", "")
.Replace(".", "_");
        return $"{namespaceName}_{fullPath.Trim('_')}.gen.cs";
    }

to take just the file name of the jsonFile  ?
Something like
var fullPath = Path.GetFileNameWithoutExtension(path)
We do not need namespace and others - it is just a hint.
And , if we want to see, we can put in csproj
<PropertyGroup>      <EmitCompilerGeneratedFiles>true</EmitCompilerGeneratedFiles>     <CompilerGeneratedFilesOutputPath>$(BaseIntermediateOutputPath)Generated</CompilerGeneratedFilesOutputPath>   </PropertyGroup>

Thanks, that's a good idea. Fixed and released as v0.8.1

Does not work  Same error .  Contains still :
( I do not see the code modified in what I say)
Please at least make
private string GetSourceFileName(string path)
{
var fullPath = Path.GetFullPath(path)
.Replace(Path.GetFullPath("."), string.Empty)
.Replace("/", "")
.Replace("\", "")
.Replace(".", "")
//add this line
.Replace(":", "");
        return $"{fullPath.Trim('_')}.gen.cs";
    }

.Replace(".", "_");

Ah, sorry. I thought removing the namespace argument would be enough. I'll take another go at it.
Honestly, it would be better if I could reproduce the issue. I suspect you are using an absolute path to include a file in the compilation (instead of a relative path). Am I right?

No. I could show you the code, if you want

In the meantime, could you modify and re-deploy?

I did, but the CI build is now failing. I'll look into it soon (a bit busy right now).

Do not rush. Better safe than sorry :-)

Should work now. Please use v0.8.2.
Please reopen if the issue is still not fixed.

Now a different error:
1>CSC : error CS0016: Could not write to output file 'E:\ignatandrei\Presentations\2021\RoslynSourceCodeGenerators\code\JsonToClass\src\JsonToClass\JsonToClass\obj\Generated\JsonByExampleGenerator.Generator\JsonByExampleGenerator.Generator.JsonGenerator\E__ignatandrei_Presentations_2021_RoslynSourceCodeGenerators_code_JsonToClass_src_JsonToClass_JsonToClass_Persons_json.gen.cs' -- 'Could not find a part of the path 'E:\ignatandrei\Presentations\2021\RoslynSourceCodeGenerators\code\JsonToClass\src\JsonToClass\JsonToClass\obj\Generated\JsonByExampleGenerator.Generator\JsonByExampleGenerator.Generator.JsonGenerator\E__ignatandrei_Presentations_2021_RoslynSourceCodeGenerators_code_JsonToClass_src_JsonToClass_JsonToClass_Persons_json.gen.cs'.'
This occurs just from VS , with rebuild all ( ! ), not from dotnet build
I have attached the project. ( the sln is at localized at E:\ignatandrei\Presentations\2021\RoslynSourceCodeGenerators\code\JsonToClass\src\JsonToClass )
JsonToClass.zip

And I do not have righs to open the issue ;-)

Now the name of the namespace is dependent of the folder path. Could you not pluralize and just put into the namespace of the assembly ?

For example, now the namespace is
using JsonToClass.Json.C.D.JsonToClass.JsonToClass.Persons;
if the .csproj is at
C:\d\JsonToClass\JsonToClass


goguru looking in sublime bundle for code

I just installed and am using stock configuration. For some reason its not taking my GOPATH environment variable, or something. From the sublime console:
GoGuru [ERROR]: ('open /Applications/Sublime Text.app/Contents/hail/src/common/app: no such file or directory\nguru: no initial packages were loaded\n',)

GOPATH is /Users/rk/go/third_party:/Users/rk/go/hail

actually nevermind, the GOPATH env variable isn't making it through


size optimization

before and after:
$ docker images
REPOSITORY            TAG                 IMAGE ID            CREATED             SIZE
<none>                <none>              4bec0e324b07        3 minutes ago       746MB
<none>                <none>              f26c696d3cee        28 minutes ago      915MB

169MB profit.

That's great. I was wondering why the last change did not produce any result.
Thank you!


Separating Dataset and Pipeline configuration yaml files

Separating the existing yaml configuration into two files to keep both logic separated and avoid any accidental changes to each other.

Draft 1.0

active_recipe: standard # Selecting the recipe user wants to execute
pipeline:

recipe_name: standard # Name of the recipe which can be used for a recipe and decide which recipe needs to be executed by setting it in active_recipe.
generateGrpah: false # To generate png of the DAG using networkx library
stages: # Actual pipeline stages

stage: open_raw # name of the stage which will be the same as the prefect flow name
saveOutput: true # Option to save the intermediate output, need to decide on the location of where to store the output or we can use the config from Dataset.yml
parallelize: true # Option to parallelize the operation, most of the functions we have currently, we can decide on whether and how to parallelize or not. This option will be more useful for custom flows.
externalParams: # To set external params. for example, bypassing some values or XML config files in scenarios of different sonar models.
sonar_model: EK60
xml_path: s3//
stage: update_platform
stage: compute_Sv
stage: add_lat_lon
stage: compute_MVBS
stage: bottom_detection
stage: apply_mask
stage: NASC

Hey @Sohambutala , how about using fenced code blocks to show the above? It's kinda hard to read as a bullet list, and the indentation did not render correctly.
Also, I assume the above is a draft recipe for the pipeline without the dataset part? When you build it, it'll be great if you could incorporate the use of pydantic so that things can be checked automatically!

Hi @leewujung, I have created pydantic classes for both and will add validations as required.
As of now, I haven't changed the dataset configuration part which is as below.
Dataset.yaml
name: Bell_M._Shimada-SH1707-EK60
sonar_model: EK60
raw_regex: (.*)-?D(?P<date>\w{1,8})-T(?P<time>\w{1,6})
args:
  urlpath: s3://ncei-wcsd-archive/data/raw/{{ ship_name }}/{{ survey_name }}/{{ sonar_model }}/*.raw
  parameters:
    ship_name: Bell_M._Shimada
    survey_name: SH1707
    sonar_model: EK60
  storage_options:
    anon: true
  transect:
    file: ./x0007_fileset.txt
output:
  urlpath: ./combined_files
  overwrite: true

Minor Update in pipeline config yaml structure. Made variable conforming to snake case and added name and module for importing the correct function in case of multiple implementations of a given function in different modules.
Will attempt to expose the predefined functions but module and name of function still would be relevant in case user wants to add their own logic.
Draft 1.1
active_recipe: standard
pipeline:
- recipe_name: standard
  generate_grpah: false
  stages:
  - name: open_raw
    module: echoflow.stages_v2.subflows.open_raw
    save_output: true
    parallelize: true
    external_params:
      sonar_model: EK60
      xml_path: s3//
  - name: combine_echodata
    module: echoflow.stages_v2.subflows.combine_echodata
    save_output: true
    parallelize: true
    external_params:
      sonar_model: EK60
      xml_path: s3//

Here is the latest stable version for both pipeline and datastore yaml files.
Pipeline YAML Configuration
Datastore YAML Configuration


DLL load failed

Issue Type: Bug
DLL load fails for numpy and h5py packages nstalled in Ananconda if opening VScode directly. Opening VScode from anaconda shows no problem. What is the point of having ana anaconda extension if the envirnments and paths are properly configured?
See here
https://stackoverflow.com/q/56622152/3054385
https://stackoverflow.com/q/56786224/3054385
Extension version: 1.0.1
VS Code version: Code 1.46.1 (cd9ea6488829f560dc949a8b2fb789f3cdc05f5d, 2020-06-17T21:13:20.174Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz (8 x 2592)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.86GB (6.69GB free)

Process Argv

Screen Reader
no

VM
0%

The proper place to open issues against the Python extension is https://github.com/microsoft/vscode-python. Do know, though, that getting this right is very hard and we are aware of the issue (it's an unfortunate clash with conda wanting to own the entire environment it works in and VS Code wanting to also manage the environment it works in.


Add Installation Guide to README

@bakerwho pointed out in #772 that we do not have a clear and easy-to-spot installation guide. With this PR I basically move the little installation guide we had in CONTRIBUTING to the README so that it is harder to miss.

Maybe we don't need CONTRIBUTING at all?

@norvig: I think the CONTRIBUTING does serve a purpose. It is a good place for potential contributors to get a gist of the direction they can help contribute in. Maybe it needs a bit of a refocus though. The "Running the Suite" section should probably be removed (since the section now lies in the README). Also, maybe the "Contributing a Patch/Reporting an Issue/Patch" rules sections are a bit redundant (since most people already follow these rules).
I will set aside some time to work on it this weekend.


Consider adding Container.square or BoxDecoration to SizedBox

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
This came up during the design review. When building a custom design system, there may be cases where developers want to create a decorated square container. Currently, there is SizedBox.square - but this doesn't allow for a BoxDecoration
Proposal
We could consider adding Container.square(...) or a boxDecoration property to SizedBox

SizedBox is an elementary component. Its decoration equivalent is DecoratedBox. IMO, we shouldn't mix them.
Adding .square to Container would make more sense, as Container is more like a wildcard component.

Container like used for the decoration and shadow for other widget but sizedbox only used for the height and width nto like behave the container

I don't understand what exactly this feature request is proposing:

a widget that attempts to force a 1:1 aspect ratio during layout?
a widget that takes a single dimension and uses it both for width and height during layout?
a widget that paints a square decoration into whatever dimension it has?
a widget that combines two or more of the above?

Hello everyone, how can I make a square container at the moment?

Hello everyone, how can I make a square container at the moment?

There are many ways of doing it. Here are two suggestions:

By passing equal width and height to your Container;
By using the AspectRatio with a ratio of 1.


Question: Screenrecording

Just a quick question, what tool did you use to record the terminal interaction? Looks awesome.

Thanks, it was taken by LICEcap with cho45/KeyCast.

Seems like kind of FAQ, included in the README in  acb226f9fa7532a0e68806547796af4448c6152c.


Detect / Point To Already Installed Software

Suggested new feature or improvement
I have already installed software via Scoop / WinGet. I would like to point to already installed software.
Scenario
I have already setup my machine, and installed all necessary applications. I installed Dev Home after the fact.
Additional details
I've setup my machine already. Once Dev Home is installed, point to previously installed software.

When you say point to installed software, do you mean in the machine configuration -> install applications tool? Or would you like to see them somewhere else?

Related to:

#20

I've been thinking about having essentially "full" support of WinGet and the various features exposed by package managers. The ability to install specific versions of tools, pinning, etc.

When you say point to installed software, do you mean in the machine configuration -> install applications tool? Or would you like to see them somewhere else?

Yes, when I see the "Machine configuration" section, it is telling me to install Git 2.41.0 through Winget. However, I've already installed Git into my user space, via scoop.

Looks like it can't differentiate between VS editions either:


Ads: now banner ads is loading very slow, maybe in future we can optimize it

Not applicable in react native version anymore


Design and usability help opportunity

I don't intend to rub anyone the wrong way by suggesting this, but Simply Secure is offering "free assistance with user-experience (UX) design as well as usability research". As the definitive free software email client, I think K-9 Mail could benefit from this.
If you are interested in applying, more information is here.
Thank you for your awesome software work!

Thanks. Sound awesome.

The "here" link is dead, so I guess somebody can close this issue.


[Website] Apache Hop Case Study

Reworked Apache Hop Case Study.
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

this looks good to me. I'd like to get the website staged to see it before merging, so we need to wait for Jenkins to be working again

LGTM as well. @AKosolapov can we get this staged on the website and merge it?

Retest this please

here's the staging. It looks great: http://apache-beam-website-pull-requests.storage.googleapis.com/16824/case-studies/hop/index.html


Aspnet Core 1.1.7 Memory Leak

Describe the bug
We have a memory leak problem which occurs in at least once a day, sometimes in a half hour, sometimes in a couple of hours. We do not the root cause so need help to narrow down the issue.
To Reproduce
Steps to reproduce the behavior:

We are using this version of ASP.NET Core 'Microsoft.AspNetCore.Mvc 1.1.7'
Deployed on Windows Server 2012 R2 and IIS 6.2 Build 9200

Expected behavior
The memory usage of our API project spikes suddenly occasionally which eats all available memory in a couple minutes. We do not know the root cause but we spotted that when it occurs, starts from ~140MB RAM (stable) and increase in ~50-100 MB/s.
We took a memory dump and analysed with DebugDiag and took a report. Since it is a long report, I will share some selected parts of it. If necessary, I will provide more.
Additional context
PS > dotnet --version
Microsoft .NET Core Shared Framework Host

  Version  : 1.1.0
  Build    : 928f77c4bc3f49d892459992fb6e1d5542cb5e86

DebugDiag Report (some selected parts)

Warning
The following threads in BA.ECommerce.Api.DMP are waiting in a WaitMultiple( 11 13 15 18 20 22 23 24 25 28 30 )26.19% of threads blocked
Typically threads waiting in WaitMultiple are monitoring threads in a process and this may be ignored, however too many threads waiting in WaitOne\WaitMultiple may be a problem. Review the callstack of the threads waiting to see what they are waiting on

Warning
The following threads in BA.ECommerce.Api.DMP are waiting in a WaitOne( 0 7 8 9 10 34 )14.29% of threads blocked
Typically threads waiting in WaitMultiple are monitoring threads in a process and this may be ignored, however too many threads waiting in WaitOne\WaitMultiple may be a problem. Review the callstack of the threads waiting to see what they are waiting on

--
--
--

Warning
The following threads in BA.ECommerce.Api.DMP are waiting for .net garbage collection to finish.The current set of scripts were not able to determine which thread induced GC( 35 36 )4.76% of threads blocked
When a GC is running the .NET objects are not in a valid state and the reported analysis may be inaccurate. Also, the thread that triggered the Garbage collection may or may not be a problematic thread. Too many garbage collections in a process are bad for the performance of the application. Too many GC's in a process may indicate a memory pressure or symptoms of fragmenation. Review the blog ASP.NET Case Study: High CPU in GC - Large objects and high allocation rates for more details

--
--
--

Warning
Detected possible blocking or leaked critical section at 0x000000a3`c4337db0 owned by thread 37 in BA.ECommerce.Api.DMPImpact of this lock2.38% of threads blocked(Threads 40)The following functions are trying to enter this critical section0x00000000
The following vendors were identified for follow up based on root cause analysisPlease f

Possible Blocking or Leaked Criticical Section Details

Critical Section
0x000000a3`c4337db0

Lock State
Locked

Lock Count
1

Recursion Count
1

Entry Count
0

Contention Count
19

Spin Count
33556429

Owner Thread
37

Owner Thread System ID
3592

Entry point
ntdll!RtlUserThreadStart+34

Create time
12/24/2018 12:06:09 AM

Time spent in user mode
0 Days 00:00:00.00

Time spent in kernel mode
0 Days 00:00:00.00

Function
Source

ntdll!NtWaitForSingleObject+a

ntdll!RtlpWaitOnCriticalSection+e1

ntdll!RtlpEnterCriticalSectionContended+a4

coreclr!CrstBase::Enter+7a

coreclr!ThreadSuspend::LockThreadStore+b8

coreclr!ThreadStore::AddThread+29

coreclr!SetupUnstartedThread+50

coreclr!ThreadpoolMgr::CreateUnimpersonatedThread+99

coreclr!ThreadpoolMgr::MaybeAddWorkingWorker+19e

coreclr!ThreadpoolMgr::GateThreadStart+6e0

kernel32!BaseThreadInitThunk+22

ntdll!RtlUserThreadStart+34

@sebastienros - can you provide info on how to diagnose this?
BTW support for ASP.NET Core 1.1 is ending in the near future, so I recommend moving up to a newer version. Also, lots of great new features, performance improvements, and bug fixes!

@guneysus

which eats all available memory in a couple minutes

Does it reach the point where you get an OutOfMemoryException and the application crashes? Or is it just taking a lot of memory?
If so, would you be able to share the memory dump I send you a link to a onedrive folder to put the file on? Ideally when it's already taking a lot of memory so I can see what is using it.

Hi @sebastienros.
I did not remember OutOfMemoryException exception in production but in test environment yes, there was OutOfMemoryException.
It did not stop eating all available RAM on the server, and page faults starts to happen.
Sure, where can I upload the memory dump file, which takes about 50-60MB in zip but takes ~16GB if extracted.

Please send me an email at sebastien.ros at microsoft.com and I will share a onedrive folder with you.

Closing this as we haven't heard from you and generally close issues with no response. Please feel free to comment if you're able to get the information we're looking for and we can reopen the issue to investigate further!
Also, as @Eilon mentioned, .NET Core 1.1 is now out of support. If you still see this issue in a supported version of .NET Core, let us know!


home-assistant dependencies

Motivation for this change
More home-assistant dep updates
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

The octoprint build was broken on master, due to a version constraint for watchdog==0.10.4, and additionally on this branch due to emoji==0.6.0. Awesome version constraints ignore infrastructure. Kudos to the octoprint maintainers!

 It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 111661 run on x86_64-darwin 1

7 packages marked as broken and skipped:

poezio
python37Packages.bellows
python37Packages.poezio
python38Packages.bellows
python38Packages.poezio
python39Packages.bellows
python39Packages.poezio

1 package failed to build and already failed to build on hydra master:

ntfy: log https://termbin.com/hcqp

24 packages built:

home-assistant
mpdris2
octoprint
python37Packages.emoji
python37Packages.influxdb-client
python37Packages.mpd2
python37Packages.pysonos
python37Packages.pywilight
python37Packages.xknx
python38Packages.brother
python38Packages.emoji
python38Packages.influxdb-client
python38Packages.mpd2
python38Packages.pysonos
python38Packages.pywilight
python38Packages.xknx
python39Packages.brother
python39Packages.emoji
python39Packages.influxdb-client
python39Packages.mpd2
python39Packages.pywilight
python39Packages.xknx
rofi-mpd
terraform-compliance

That's it for now.

 It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 111661 run on x86_64-darwin 1

10 packages marked as broken and skipped:

poezio
python37Packages.bellows
python37Packages.bt_proximity
python37Packages.poezio
python38Packages.bellows
python38Packages.bt_proximity
python38Packages.poezio
python39Packages.bellows
python39Packages.bt_proximity
python39Packages.poezio

1 package failed to build and already failed to build on hydra master:

ntfy: log https://termbin.com/jddr

30 packages built:

home-assistant
mpdris2
octoprint
python37Packages.emoji
python37Packages.fixerio
python37Packages.influxdb-client
python37Packages.libsoundtouch
python37Packages.mpd2
python37Packages.pysonos
python37Packages.pywilight
python37Packages.xknx
python38Packages.brother
python38Packages.emoji
python38Packages.fixerio
python38Packages.influxdb-client
python38Packages.libsoundtouch
python38Packages.mpd2
python38Packages.pysonos
python38Packages.pywilight
python38Packages.xknx
python39Packages.brother
python39Packages.emoji
python39Packages.fixerio
python39Packages.influxdb-client
python39Packages.libsoundtouch
python39Packages.mpd2
python39Packages.pywilight
python39Packages.xknx
rofi-mpd
terraform-compliance

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:
python37Packages.fixerio:

Please consider this feature to be alpha.
A substituteInPlace with an unmatched pattern got detected:
substituteStream(): WARNING: pattern 'requests==2.10.0' doesn't match anything in file 'setup.py'

Please check the offending substituteInPlace for typos or changes in source.

python38Packages.fixerio:

Please consider this feature to be alpha.
A substituteInPlace with an unmatched pattern got detected:
substituteStream(): WARNING: pattern 'requests==2.10.0' doesn't match anything in file 'setup.py'

Please check the offending substituteInPlace for typos or changes in source.

python39Packages.fixerio:

Please consider this feature to be alpha.
A substituteInPlace with an unmatched pattern got detected:
substituteStream(): WARNING: pattern 'requests==2.10.0' doesn't match anything in file 'setup.py'

Please check the offending substituteInPlace for typos or changes in source.

substituteStream(): WARNING: pattern 'requests==2.10.0' doesn't match anything in file 'setup.py'

removed.


Collection of containers

The new SharpHound collects Containers as well as GPLinks to them and ACEs on them.
This would be a nice addition to BloodHound.py!
Cheers

Implemented in the latest version in #103


adds ActivatorProvider to allow custom type activation (Resolves LOG4NET-565)

This refactors type activation into an ActivationProvider class, which provides wrapper methods for the IActivator interface. The default IActivator implementation (DefaultActivator) derives from the current logic which uses Activator.CreateInstance. Clients of log4net can set the IActivator instance to a custom implementation if they so choose. For an example use case, this will allow for dependency injection in custom appenders and pattern converters.
I researched how to do dependency injection in custom log4net appenders and pattern converters without making changes to log4net itself and it proves to be increasingly difficult, since it would require custom classes in cases where Activator.CreateInstance is used. I tried to keep my changes to a bare minimum, as I recognize this project is one of the most popular logging frameworks in .NET and any changes must be heavily scrutinized.
I also recognize that there have been other attempts to add this capability to log4net (#26, #72), but this implementation is merely a refactoring of the existing Activator.CreateInstance logic. With all due respect to the authors of the previous implementation attempts, this implementation provides one other key component that the other attempts were missing. The IsTypeConstructible method of XmlHierarchyConfigurator has been refactored into a CanCreateInstance method on the IActivator interface. This is necessary to allow custom activation of types without a default constructor.
As this is my first contribution to this project, any feedback is definitely appreciated!
Thanks!
Travis Schettler

@tschettler I think next steps would be to raise a PR with a mvp implmentation of the described pattern in log4net. I don't know when I'll get time to look at an implementation, so if you want to crack on, go for it.

Sounds good, I’d be up for that, thanks!


Yaki Mobile - PWA user should add picture or use camera as Avatar

As PWA user, i should be able to take a picture as avatar.
Yet as its a PWE the functionnality of using the -galery- will be disabled

duplicated with #1019


Synthesis: Document composite identifiers

Relevant information
Document composite identifiers
Allow them on the validators
Extend the collision check to detect collisions when using the composite identifiers too.
Checklist

[ ] I've read the guidelines and understand the acceptance criteria.
[ ] The value of the attribute marked as identifier will be unique and valid.
[ ] I've confirmed that my entity type wasn't already defined. If it is I'm providing an
explanation above.

❤


Add auto_redirect_to_identity to access organizations

Adds auto_redirect_to_identity  to access organizations. This field has been supported for a while now, but undocumented.
Description
Describe your changes in detail through motivation and context. Why is
this change required? What problem does it solve? If it fixes an open
issue, link to the issue using GitHub's closing issues keywords1.
Has your change been tested?
automated test should be enough
Screenshots (if appropriate):
Types of changes
What sort of change does your code introduce/modify?

[ ] Bug fix (non-breaking change which fixes an issue)
[ X ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ X  ] My code follows the code style of this project.
[ X  ] My change requires a change to the documentation.
[  X ] I have updated the documentation accordingly.
[  X ] I have added tests to cover my changes.
[ X  ] All new and existing tests passed.
[ X  ] This change is using publicly documented (api.cloudflare.com or developers.cloudflare.com) and stable APIs.

Codecov Report

Merging #1260 (b60b857) into master (0c3db86) will increase coverage by 0.07%.
The diff coverage is 48.27%.

:exclamation: Current head b60b857 differs from pull request most recent head ae7ec32.  Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

docs: https://developers.cloudflare.com/api/operations/zero-trust-organization-create-your-zero-trust-organization


Laravel 11 can't get the UTC timestamp

Laravel Version
11.0.0
PHP Version
8.2.16
Database Driver & Version
MySQL 5.7.44 for Ubuntu 22
Description
I've tried three methods and I can't get the UTC+0 timestamp.
The time value is correct until it is converted to a timestamp.
Laravel 10 was valid code, upgrading to 11 caused problems. No server environment has been changed.
Steps To Reproduce
$now = now('UTC');
$nowTimestamp = $now->getTimestamp();

$nowTimestamp = time();

$now = now('UTC');
$nowTimestamp = strtotime($now);

@jevantang what Carbon version are you using?

@jevantang what Carbon version are you using?

name     : nesbot/carbon
versions : * 3.0.2
released : 2024-02-06, 1 month ago

now()->timestamp

@Muetze42

You'll have to report this to the carbon repo as we don't control that code sorry.

The timestamp is correct, isn't it?

The timestamp is correct, isn't it?

@Muetze42
1710276559 -> 2024-03-13 04:49:19
Current UTC+0 Time 2024-03-12 20:49:19

1710276559 ->2024-03-13 04:49:19

@jevantang How do you convert it?
1710276559 is equals to 20:49:19.
Timestamp has no timezone it always UTC+0000. where is the 8?

You'll have to report this to the carbon repo as we don't control that code sorry.

For the record, this issue is not related to Carbon since time() is giving the same value.


ActivityPub integration

How cool would it be if through ActivityPub the song currently being played could be shared on the Fediverse?

That would indeed be lovely


Update Prompted Generator - Add Box Prompts

The prompt generator now gives box prompts on top of point prompts too!

Question:
(There are two ways I thought of approaching the prompt generator class, I'll mention both below)

One could be two separate classes (one for PointPromptGenerator and another for PointAndBoxPromptGenerator) (the differences are super slim, but might be an open utility for all purposes)
Other could be (current PR) one class (PointAndBoxPromptGenerator) which returns box prompts subjected to get_box_prompts argument else stays consistent to just prompts (returning "None" for boxes - see here)

Which one do you recommend?

Your current solution is fine. But I would also add get_point_prompts=True to the function signature, so we can only get the boxes if needed.

I have also started to write tests for the point prompts in #16; we should first fix and merge the tests before working on this PR.

I have also started to write tests for the point prompts in #16; we should first fix and merge the tests before working on this PR.

Nice, okay. So currently I first updated the prompt generator class to utilise the possibility of all kinds of prompting. Now the accessibility would increase (I'll look at the referred PR as well now). Thanks!

Spotted a bug (small yet vital) in the prompt generator (mentioned here). It's fixed here now as well.


If value is zero then background circle doesn't show up

Hi,
I am using following code to show guage:

Majority of times the value "perc" is 0 in my case, so nothing shows up on the UI. I want to show the background circle (#ccc0bc) to show up without any filling with foreground color for value 0. Is this possible?
Please help.

Hi @samdanishaik ,
Make sure you are using the latest version 2.0.0. If not, then take an update and read the README document to slight changes in API. This would remove the major problems occurring in 1.x
In case, if it is still a problem, then as per my experimentation, when i bind a scope variable to value property of <ng-gauge> and it is undefined, then the gauge would not show up. However, if you initialize it with 0 then background rim of gauge will show up.
angular.controller('MyController', myController);

function myController() {
var vm = this;
vm.currentValue = 0;
}

<body ng-controller="MyController as m">
   <!-- this will show a rim with 0 value -->
   <ng-gauge value="m.currentValue"></ng-gauge>

<!-- this will not show the rim at all, because secondCurrentValue is undefined -->
    <ng-gauge value="m.secondCurrentValue"></ng-gauge>
</body>

Let me know if it is still an issue? May be paste some code to understand.

You are right.. This works.. Thanks for the quick response

Thanks.

Hi,
One more query on this.
I modify the guage value dynamically on the same screen/page on selection of an element from dropdown. Lets say by default, I have a process in 100%/50% completion state. Now if I change the dropdown to an element which has 0% completion then this doesnt refresh the guage.

In my controller this is the code I have:
$scope.perc = 0;
   onboardServices.getOpenOnboardingStatus($scope.process.processkey).then(function(response) {
        console.log("Status: ", JSON.stringify(response.data));
        $scope.obcase = response.data.items[0];
        $scope.perc = Math.round(($scope.obcase.complete_count/$scope.obcase.total_count)*100);
    },function(error) {
        alert("OB Status request failed. Please try again.");
    });

In html:

This happens only when binding the guage with 0% from any higher value. How can I fix this?


SSDP Support in Physical Web Android App

This PR contains first SSDP support for Physical Web Android App. Please refer to documentation/ssdp_support.md for more details. A Physical Web SSDP URL advertiser is also available to test this feature.

Hi Larry!
Thanks for your pull request. Good stuff! Below are instructions for signing the Contributor License Agreement. Once that's taken care of, we can start the review process.

It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA) at https://cla.developers.google.com/.
If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check the information on your CLA (https://cla.developers.google.com/clas) or see this help article on setting the email on your git commits (https://help.github.com/articles/setting-your-email-in-git/)
Once you've done that, please reply here to let us know. If you signed the CLA as a corporation, please let us know the company's name.
<<<

@matthewsibigtroth  @scottjenson  finally I get approval from our legal department and the CLA is signed. I pushed a new version and issues mentioned in the comments of this PR are solved. Once the PR is merged we can talk about next steps.
Louay

I pushed a new version that solves all the formatting issues you mentioned and other similar issues.


Missing module when calling getters, mutations, actions

Hi, Paleo!
I have followed your example but when calling getters, actions, mutations, the following problem occurs:
State: OK
store.state.mod1.p1 ---> OK
Getters: missing module
store.getters.p1OrDefault -----> call OK
But, I want it to be like this:
store.getters.mod1.p1OrDefault
How I can do?

I have fixed it. Thanks anyway


Fix error message for viewing tasks

Codecov Report

Merging #187 into master will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #187      +/-   ##
============================================
- Coverage     61.72%   61.68%   -0.04%
  Complexity      842      842
============================================
  Files           134      134
  Lines          3112     3114       +2
  Branches        349      349
============================================
  Hits           1921     1921
- Misses         1041     1043       +2
  Partials        150      150

Impacted Files
Coverage Δ
Complexity Δ

...ain/java/seedu/momentum/commons/core/Messages.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

...va/seedu/momentum/logic/commands/ClearCommand.java
41.66% <0.00%> (-8.34%)
2.00 <0.00> (ø)

...ain/java/seedu/momentum/model/project/Project.java
49.20% <0.00%> (ø)
12.00 <0.00> (ø)

Continue to review full report at Codecov.

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


Add console commands for index settings

Adds scout/index/update-settings and scout/index/dump-settings.
I wanted to have these settings sourced, and also needed the ability to forwardToReplicas, which you can't do via the Algolia admin, only the API.

Hrmmmm looks like the StyleCI failed, but I don't see a .styleci.yml, so I don't know what standards I'm up against…my whitespace stuff should just be standard PSR2.

This could just as easily be in it's own indexSettings config, instead of nested within the mappings…which might make more sense.

@timkelty If you have time, could you look this over again, solve the merge conflicts and add an example use case to the README?
Seems like a great feature to have

@timkelty Merged it in, works great!

@Rias500 oh good! I never updated this to use a separate indexSettings key, which I think is probably still the better choice. Next time I'm in there I'll try and push something.