Packages (Linux): Add Guix package manager.

Add the 3 main profiles for guix package manager.
The detection is based on the one for neofetch and only includes packages explicitly installed by the user or propagated from other packages.
I have not yet found a good way to implement a more sophisticated approach (i.e. something similar as what is done for nix package manager). Should this be in a second merge request once I have it ready, but when I have more time to look more deeply into it?
Hopefully I caught all position where to include the new package manager option in modules/packages/packages.c.
Also had some trouble with whitespace before in the commit, hopefully i caught it all.
I have built the package with this commit successfully on x64-linux-gnu and aarch64-linux-gnu (via qemu-binfmt).
Tested it on a sytem running Guix System and on Fedora running guix as additional package manager.
Closes https://github.com/fastfetch-cli/fastfetch/issues/970

I think it's pretty solid. LGTM


Prometheus not sending resolved notification when pods have restarted

What did you do?
I am using the Elasticsearch-cluster-health alert file and it sends the alert ok to the Alertmanager when I bring some of the Elasticsearch pods down. However when the pods come up again the alert does not clear from the Alertmanager
What did you expect to see?
The alert disappearing from Alertmanager when the Elasticsearch pods come up again
What did you see instead? Under which circumstances?
The alert is still showing in Alertmanager
Environment
Prometheus configured on Kubernetes cluster with IBM Cloud Private

System information:
insert output of uname -srm here

Alertmanager version:
insert output of alertmanager --version here

Prometheus version:
insert output of prometheus -version here (if relevant to the issue)

Alertmanager configuration file:

insert configuration here

Prometheus configuration file:

spec:
  data: |-
    groups:
      - name: elasticsearchClusterHealth
        rules:
          - alert: elasticsearchClusterHealth
            expr: elasticsearch_cluster_health_up != 1
            for: 1s
            annotations:
              description: Elasticsearch cluster's health status is not green.
              summary: Elasticsearch cluster not healthy
              type: Storage
  enabled: true

* Logs:

insert Prometheus and Alertmanager logs relevant to the issue here

Thanks for your report. It looks as if this is actually a question about usage and not development.
To make your question, and all replies, easier to find, we suggest you move this over to our user mailing list, which you can also search. If you prefer more interactive help, join or our IRC channel, #prometheus on irc.freenode.net. Please be aware that our IRC channel has no logs, is not searchable, and that people might not answer quickly if they are busy or asleep. If in doubt, you should choose the mailing list.
Once your questions have been answered, please add a short line pointing to relevant replies in case anyone stumbles here via a search engine in the future.

Hi @simonpasquier thanks for getting back to me. I tried to joing the IRC channel and submit a question but its giving me a 'Cannot send to Nick/Channel' error. I also see a message in the UI saying  Register with Nickserv to join the channel - https://promcon.io/2019-munich/schedule/ but it just takes me to a schedule of the upcoming PromCon EU event. Could you let me know what I should do to be able to submit a question in the channel please?

Yes you need to register your nickname to NickServ to be allowed to send messages on the IRC channel, see https://freenode.net/kb/answer/registration. The promcon link has nothing to do with that.

Thanks @simonpasquier I registered but it still won't allow me to send messages on the #prometheus channel. I can see others commenting but I still can't post mine, it's still saying 'Cannot send to Nick/Channel'. I tried registering again but its saying Guest_62 is already registered. Anything else I am missing? Thanks in advance

Hmm not sure what happens then. You can also try the CNCF Slack, there's a prometheus channel there.

Closing since it's a support request that it is better addressed on the usual channels (chats, mailing lists).


Cache buckets to speed up BytesRefHash#sort

Following https://github.com/apache/lucene/pull/12775, this PR tries another approach to speed up BytesRefHash#sort:
The idea is that since we have extra ints in this map, we can cache the bucket when building the histograms, and reuse them when reorder.  I checked this approach on intel chip, showing ~30% speed up. I'll check M2 chip and wikimedium data tomorrow.
BASELINE:  sort 5169965 terms, build histogram took: 1968ms, reorder took: 2132ms, total took: 5470ms.
BASELINE:  sort 5169965 terms, build histogram took: 1975ms, reorder took: 2133ms, total took: 5526ms.
BASELINE:  sort 5169965 terms, build histogram took: 1999ms, reorder took: 2157ms, total took: 5573ms.
BASELINE:  sort 5169965 terms, build histogram took: 1955ms, reorder took: 2138ms, total took: 5446ms.
BASELINE:  sort 5169965 terms, build histogram took: 1990ms, reorder took: 2161ms, total took: 5528ms.
BASELINE:  sort 5169965 terms, build histogram took: 1997ms, reorder took: 2175ms, total took: 5571ms.
BASELINE:  sort 5169965 terms, build histogram took: 2004ms, reorder took: 2119ms, total took: 5477ms.
BASELINE:  sort 5169965 terms, build histogram took: 1978ms, reorder took: 2155ms, total took: 5501ms.
BASELINE:  sort 5169965 terms, build histogram took: 2015ms, reorder took: 2169ms, total took: 5572ms.
BASELINE:  sort 5169965 terms, build histogram took: 1941ms, reorder took: 2138ms, total took: 5400ms.
BASELINE:  sort 5169965 terms, build histogram took: 2000ms, reorder took: 2155ms, total took: 5558ms.

CANDIDATE:  sort 5169965 terms, build histogram took: 1996ms, reorder took: 133ms, total took: 3734ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 1989ms, reorder took: 142ms, total took: 3655ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 2031ms, reorder took: 155ms, total took: 3762ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 2016ms, reorder took: 145ms, total took: 3739ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 1994ms, reorder took: 142ms, total took: 3667ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 2010ms, reorder took: 140ms, total took: 3651ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 2021ms, reorder took: 154ms, total took: 3731ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 2019ms, reorder took: 144ms, total took: 3727ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 2064ms, reorder took: 138ms, total took: 3784ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 2043ms, reorder took: 142ms, total took: 3727ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 1964ms, reorder took: 140ms, total took: 3630ms.

Even faster than the original approach on M2:
BASELINE:  sort 5169965 terms, build histogram took: 489ms, reorder took: 1359ms, total took: 2381ms.
BASELINE:  sort 5169965 terms, build histogram took: 449ms, reorder took: 1290ms, total took: 2249ms.
BASELINE:  sort 5169965 terms, build histogram took: 458ms, reorder took: 1279ms, total took: 2238ms.
BASELINE:  sort 5169965 terms, build histogram took: 462ms, reorder took: 1302ms, total took: 2260ms.
BASELINE:  sort 5169965 terms, build histogram took: 455ms, reorder took: 1282ms, total took: 2239ms.
BASELINE:  sort 5169965 terms, build histogram took: 449ms, reorder took: 1327ms, total took: 2282ms.
BASELINE:  sort 5169965 terms, build histogram took: 474ms, reorder took: 1320ms, total took: 2303ms.
BASELINE:  sort 5169965 terms, build histogram took: 448ms, reorder took: 1260ms, total took: 2227ms.
BASELINE:  sort 5169965 terms, build histogram took: 448ms, reorder took: 1307ms, total took: 2258ms.
BASELINE:  sort 5169965 terms, build histogram took: 464ms, reorder took: 1313ms, total took: 2280ms.
BASELINE:  sort 5169965 terms, build histogram took: 444ms, reorder took: 1294ms, total took: 2261ms.

CANDIDATE:  sort 5169965 terms, build histogram took: 549ms, reorder took: 84ms, total took: 1287ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 535ms, reorder took: 83ms, total took: 1205ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 531ms, reorder took: 65ms, total took: 1197ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 552ms, reorder took: 70ms, total took: 1217ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 576ms, reorder took: 73ms, total took: 1255ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 551ms, reorder took: 66ms, total took: 1207ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 551ms, reorder took: 68ms, total took: 1208ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 553ms, reorder took: 85ms, total took: 1240ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 562ms, reorder took: 66ms, total took: 1235ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 557ms, reorder took: 76ms, total took: 1233ms.
CANDIDATE:  sort 5169965 terms, build histogram took: 541ms, reorder took: 66ms, total took: 1204ms.

As "reorder" gets faster, I'm considering lowering the fallback threshold and letting radix sort do more of the work.
Benchmark result:
MAC Intel
BASELINE:  sort 5169965 terms, build histogram took: 1969ms, reorder took: 2103ms, total took: 5524ms.
BASELINE:  sort 5169965 terms, build histogram took: 1975ms, reorder took: 2118ms, total took: 5435ms.
BASELINE:  sort 5169965 terms, build histogram took: 1977ms, reorder took: 2140ms, total took: 5441ms.
BASELINE:  sort 5169965 terms, build histogram took: 1982ms, reorder took: 2168ms, total took: 5452ms.
BASELINE:  sort 5169965 terms, build histogram took: 1987ms, reorder took: 2164ms, total took: 5475ms.
BASELINE:  sort 5169965 terms, build histogram took: 1963ms, reorder took: 2157ms, total took: 5456ms.
BASELINE:  sort 5169965 terms, build histogram took: 1979ms, reorder took: 2162ms, total took: 5452ms.
BASELINE:  sort 5169965 terms, build histogram took: 1984ms, reorder took: 2172ms, total took: 5493ms.
BASELINE:  sort 5169965 terms, build histogram took: 1986ms, reorder took: 2166ms, total took: 5476ms.
BASELINE:  sort 5169965 terms, build histogram took: 2016ms, reorder took: 2287ms, total took: 5660ms.
BASELINE:  sort 5169965 terms, build histogram took: 1761ms, reorder took: 2109ms, total took: 5183ms.

CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 2018ms, reorder took: 125ms, total took: 3760ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 1998ms, reorder took: 118ms, total took: 3630ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 1987ms, reorder took: 115ms, total took: 3596ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 2001ms, reorder took: 111ms, total took: 3622ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 2017ms, reorder took: 110ms, total took: 3610ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 2021ms, reorder took: 118ms, total took: 3712ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 2006ms, reorder took: 108ms, total took: 3634ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 2015ms, reorder took: 106ms, total took: 3620ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 2006ms, reorder took: 101ms, total took: 3611ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 1993ms, reorder took: 115ms, total took: 3609ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 2024ms, reorder took: 106ms, total took: 3633ms.

CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 2972ms, reorder took: 199ms, total took: 3288ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 2941ms, reorder took: 211ms, total took: 3279ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 2991ms, reorder took: 178ms, total took: 3283ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 2932ms, reorder took: 195ms, total took: 3233ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 2954ms, reorder took: 202ms, total took: 3259ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 2938ms, reorder took: 201ms, total took: 3249ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 2949ms, reorder took: 195ms, total took: 3295ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 2961ms, reorder took: 208ms, total took: 3274ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 2965ms, reorder took: 201ms, total took: 3257ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 2957ms, reorder took: 188ms, total took: 3285ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 2975ms, reorder took: 182ms, total took: 3291ms.

MAC M2
BASELINE:  sort 5169965 terms, build histogram took: 478ms, reorder took: 1332ms, total took: 2334ms.
BASELINE:  sort 5169965 terms, build histogram took: 483ms, reorder took: 1351ms, total took: 2333ms.
BASELINE:  sort 5169965 terms, build histogram took: 462ms, reorder took: 1319ms, total took: 2284ms.
BASELINE:  sort 5169965 terms, build histogram took: 463ms, reorder took: 1272ms, total took: 2246ms.
BASELINE:  sort 5169965 terms, build histogram took: 466ms, reorder took: 1285ms, total took: 2257ms.
BASELINE:  sort 5169965 terms, build histogram took: 471ms, reorder took: 1318ms, total took: 2289ms.
BASELINE:  sort 5169965 terms, build histogram took: 465ms, reorder took: 1322ms, total took: 2291ms.
BASELINE:  sort 5169965 terms, build histogram took: 449ms, reorder took: 1288ms, total took: 2234ms.
BASELINE:  sort 5169965 terms, build histogram took: 448ms, reorder took: 1302ms, total took: 2257ms.
BASELINE:  sort 5169965 terms, build histogram took: 453ms, reorder took: 1305ms, total took: 2263ms.
BASELINE:  sort 5169965 terms, build histogram took: 456ms, reorder took: 1270ms, total took: 2229ms.

CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 586ms, reorder took: 79ms, total took: 1343ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 551ms, reorder took: 62ms, total took: 1212ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 588ms, reorder took: 72ms, total took: 1253ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 561ms, reorder took: 74ms, total took: 1227ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 572ms, reorder took: 67ms, total took: 1260ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 539ms, reorder took: 63ms, total took: 1208ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 542ms, reorder took: 74ms, total took: 1209ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 546ms, reorder took: 78ms, total took: 1222ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 532ms, reorder took: 70ms, total took: 1195ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 536ms, reorder took: 81ms, total took: 1224ms.
CANDIDATE( fallback threshold = 100 ):  sort 5169965 terms, build histogram took: 548ms, reorder took: 66ms, total took: 1207ms.

CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 660ms, reorder took: 108ms, total took: 822ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 645ms, reorder took: 120ms, total took: 815ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 648ms, reorder took: 124ms, total took: 829ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 654ms, reorder took: 145ms, total took: 835ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 652ms, reorder took: 137ms, total took: 837ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 631ms, reorder took: 144ms, total took: 816ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 654ms, reorder took: 120ms, total took: 821ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 684ms, reorder took: 111ms, total took: 841ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 652ms, reorder took: 146ms, total took: 836ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 658ms, reorder took: 127ms, total took: 837ms.
CANDIDATE( fallback threshold = 50 ):  sort 5169965 terms, build histogram took: 673ms, reorder took: 123ms, total took: 838ms.

I try this approach with wikimedium10m on the M2 mac, the sort took sum decreased ~60%.
 Details

flush round
sort field name
sort term count
baseline
candidate
diff

1
date
45925
32
15
-53.13%

1
groupend
1
0
0
#DIV/0!

1
titleTokenized
33250
10
6
-40.00%

1
id
989620
356
167
-53.09%

1
body
3444798
1991
698
-64.94%

1
title
46261
10
7
-30.00%

2
date
84517
52
21
-59.62%

2
groupend
1
0
0
#DIV/0!

2
titleTokenized
50134
15
7
-53.33%

2
id
1059900
680
212
-68.82%

2
body
3210318
1730
660
-61.85%

2
title
85009
22
13
-40.91%

3
date
86188
37
21
-43.24%

3
groupend
1
0
0
#DIV/0!

3
titleTokenized
60502
14
9
-35.71%

3
id
1011210
397
185
-53.40%

3
body
3389689
1865
692
-62.90%

3
title
86756
24
15
-37.50%

4
date
123222
76
39
-48.68%

4
groupend
1
0
0
#DIV/0!

4
titleTokenized
84573
23
16
-30.43%

4
id
1030670
533
205
-61.54%

4
body
3442602
1985
717
-63.88%

4
title
124156
41
24
-41.46%

5
date
150957
85
38
-55.29%

5
groupend
1
0
0
#DIV/0!

5
titleTokenized
100910
37
16
-56.76%

5
id
1045550
595
209
-64.87%

5
body
3267300
2031
689
-66.08%

5
title
152061
58
32
-44.83%

6
date
162374
77
41
-46.75%

6
groupend
1
0
0
#DIV/0!

6
titleTokenized
107464
30
17
-43.33%

6
id
1057430
385
194
-49.61%

6
body
3140161
1834
633
-65.49%

6
title
163347
68
36
-47.06%

7
date
177647
116
45
-61.21%

7
groupend
1
0
0
#DIV/0!

7
titleTokenized
120037
59
18
-69.49%

7
id
1062920
652
211
-67.64%

7
body
3173313
2373
638
-73.11%

7
title
178886
79
37
-53.16%

8
date
199744
106
50
-52.83%

8
groupend
1
0
0
#DIV/0!

8
titleTokenized
133801
42
21
-50.00%

8
id
1072970
409
193
-52.81%

8
body
3090582
1817
641
-64.72%

8
title
201130
84
45
-46.43%

9
date
209298
104
62
-40.38%

9
groupend
1
0
0
#DIV/0!

9
titleTokenized
142697
50
25
-50.00%

9
id
1084580
571
225
-60.60%

9
body
3032046
1642
631
-61.57%

9
title
210958
92
46
-50.00%

10
date
119014
47
30
-36.17%

10
groupend
1
0
0
#DIV/0!

10
titleTokenized
93308
26
16
-38.46%

10
id
585150
208
111
-46.63%

10
body
1909153
918
380
-58.61%

10
title
119806
50
27
-46.00%

sort took sum

24538
9086
-62.97%

Thanks for review @jpountz !
I'll merge this and close https://github.com/apache/lucene/pull/12775.

Did we see any bump in nightly benchmarks?  This should make initial segment flush when there are many terms in an inverted field faster?

Thanks for tracking in ! @mikemccand

Did we see any bump in nightly benchmarks?

I would expect this change more likely bring some improvements for flushing high cardinality StringField, or other places taking advantages of the BytesRefHash#sort like DeletedTerms.
However, I had not expected this change would bring a bump in nightly benchmarks - If you look at the CPU profile of the nightly indexing, you may find that most of CPU was used to tokenize / deduplicate terms before flushing. After deduplication the terms count will get reduced and the sort is not the bottleneck of the indexing speed (sort only use ~1% CPU).
Small difference may be found in the flame graph that reorder is not there any more. But the proportion of sorting overhead is too low to bring a noticeable E2E difference for nightly indexing.
Before this patch

After this patch


Redesign DataSourceXAResourceRecoveryHelper to be compatible with JBTM-3879 (based on org.jboss.narayana.jta.jms.JmsXAResourceRecoveryHelper)

Works with actual Narayana 7.0.1 release and Narayana 7.0.2 (including JBTM-3879) #jbosstm/narayana/pull/2268

Hi @geoand, this adjusted version of DataSourceXAResourceRecoveryHelper is needed to be compatible with JBTM-3879 changes in Narayana.
FYI @marcosgopen

So do we want to merge this?

Well, I think the implementation is better than the current one, and it is needed if upgrading to newer Narayana versions should be possible. ;)

cc @mmusgrov

Thanks @graben for your PR, it is greatly appreciated! Could you double check the PooledTransactionalIT test? It keeps failing with 7.0.2.Final: ''[ERROR] Errors:
[ERROR]   PooledRecoveryIT>GenericRecoveryIT.testCrashBeforeCommit:122 » ConditionTimeout Condition with alias 'Wait for the recovery to happen' didn't complete within 1 minutes because assertion condition with alias Wait for the recovery to happen [Test entry should exist after transaction was committed]
Expected size: 1 but was: 0 in:
[]."
I am going to have a look at that.

I got a failure after git cloning https://github.com/snowdrop/narayana-spring-boot and typing mvn clean install verify :

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-jar-plugin:3.3.0:jar (default-jar) on project narayana-spring-boot-starter-it: You have to use a classifier to attach supplemental artifacts to the project instead of replacing them. -> [Help 1]

and my env is

[mmusgrov@dev2 narayana-spring-boot] ((HEAD detached at pr/149)) $ ./mvnw -version
Apache Maven 3.9.3<PHONE_NUMBER>9f1ead511c958d89bd2f672198ae9f)
Maven home: /home/mmusgrov/.m2/wrapper/dists/apache-maven-3.9.3-bin/326f10f4/apache-maven-3.9.3
Java version: 17.0.11, vendor: Red Hat, Inc., runtime: /usr/lib/jvm/java-17-openjdk-<IP_ADDRESS>.9-1.fc39.x86_64
Default locale: en_GB, platform encoding: UTF-8
OS name: "linux", version: "6.8.11-200.fc39.x86_64", arch: "amd64", family: "unix"

and it similarly fails running just the relevant test case although the actual test passes:

[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 32.77 s -- in dev.snowdrop.boot.narayana.generic.GenericRecoveryIT

...

[mmusgrov@dev2 narayana-spring-boot-starter-it] ((HEAD detached at pr/149)) $ pwd
/home/mmusgrov/src/forks/spring/narayana-spring-boot/narayana-spring-boot-starter-it
[mmusgrov@dev2 narayana-spring-boot-starter-it] ((HEAD detached at pr/149)) $ ../mvnw clean install verify -Dit.test=GenericRecoveryIT#testCrashBeforeCommit

In fact now Narayana is not treating XAER_RMFAIL as a transient failure and would consider it a heuristic aligning to xa_rollback specification (https://publications.opengroup.org/c193)
From the test log you can read: "WARN 170836 --- [nsaction Reaper] com.arjuna.ats.arjuna : ARJUNA012117: TransactionReaper::check processing TX 0:ffffc0a82b22:a64d:66840d2a:1e in state RUN"
This is a issue that might have already been solved by Artemis team (maybe apache/activemq-artemis@269be13c5a). FYI @graben @tomjenkinson

Thank you for providing the summary @marcosgopen

@mmusgrov FYI the CI steps for snowdrop/narayana-spring-boot (which might give a hint as to how it's meant to run at least in one permutation) are in https://github.com/snowdrop/narayana-spring-boot/blob/main/.github/workflows/test.yml#L23

@marcosgopen the CI run is passing - are you saying you consider it should be considered failing due to the WARN?

Thanks @graben for your PR, it is greatly appreciated! Could you double check the PooledTransactionalIT test? It keeps failing with 7.0.2.Final: ''[ERROR] Errors:
[ERROR]   PooledRecoveryIT>GenericRecoveryIT.testCrashBeforeCommit:122 » ConditionTimeout Condition with alias 'Wait for the recovery to happen' didn't complete within 1 minutes because assertion condition with alias Wait for the recovery to happen [Test entry should exist after transaction was committed]
Expected size: 1 but was: 0 in:
[]."
I am going to have a look at that.
[UPDATE] the PooledRecoveryIT test fails with Narayana 7.0.2.Final. That is because of another issue (see https://issues.redhat.com/browse/JBTM-3838). In fact now Narayana is not treating XAER_RMFAIL as a transient failure and would consider it a heuristic aligning to xa_rollback specification (https://publications.opengroup.org/c193)
From the test log you can read: "WARN 170836 --- [nsaction Reaper] com.arjuna.ats.arjuna                    : ARJUNA012117: TransactionReaper::check processing TX 0:ffffc0a82b22:a64d:66840d2a:1e in state  RUN"
This is a issue that might have already been solved by Artemis team (maybe https://github.com/apache/activemq-artemis/commit/269be13c5a). FYI @graben @tomjenkinson

Hi @marcosgopen,
the pooled test fails because of an issue in H2 database. I'm actually waiting for an enhancement to Agroal agroal/agroal#109 actually under review by @barreiro to do a proper workaround for this

(sorry @marcosgopen now I see this is not upgrading to 7.0.2.Final :) )

Thanks @graben for your PR, it is greatly appreciated! Could you double check the PooledTransactionalIT test? It keeps failing with 7.0.2.Final: ''[ERROR] Errors:
[ERROR]   PooledRecoveryIT>GenericRecoveryIT.testCrashBeforeCommit:122 » ConditionTimeout Condition with alias 'Wait for the recovery to happen' didn't complete within 1 minutes because assertion condition with alias Wait for the recovery to happen [Test entry should exist after transaction was committed]
Expected size: 1 but was: 0 in:
[]."
I am going to have a look at that.
[UPDATE] the PooledRecoveryIT test fails with Narayana 7.0.2.Final. That is because of another issue (see https://issues.redhat.com/browse/JBTM-3838). In fact now Narayana is not treating XAER_RMFAIL as a transient failure and would consider it a heuristic aligning to xa_rollback specification (https://publications.opengroup.org/c193)
From the test log you can read: "WARN 170836 --- [nsaction Reaper] com.arjuna.ats.arjuna                    : ARJUNA012117: TransactionReaper::check processing TX 0:ffffc0a82b22:a64d:66840d2a:1e in state  RUN"
This is a issue that might have already been solved by Artemis team (maybe apache/activemq-artemis@269be13c5a). FYI @graben @tomjenkinson

Hi @marcosgopen,
the pooled test fails because of an issue in H2 database. I'm actually waiting for an enhancement to Agroal agroal/agroal#109 actually under review by @barreiro to do a proper workaround for this. The pooled test is not affected by this change. The helper is only used by non-pooled connections.

You are right @graben . The pooled test failure is not related to this PR and not related to JBTM-3879 (XA recovery scan cursor handling).
Having written my comment here is confusing so I will keep discussing about it in the Narayana update PR (https://github.com/snowdrop/narayana-spring-boot/pull/148)

Yes and no. The change in Narayana by JBTM-3879  is causing that the first workaround is not working any more. It's a bit complicated and confusing that changes in several projects are causing "strange" behaviours. It's keeping me busy. :-D

I got a failure after git cloning https://github.com/snowdrop/narayana-spring-boot and typing mvn clean install verify :

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-jar-plugin:3.3.0:jar (default-jar) on project narayana-spring-boot-starter-it: You have to use a classifier to attach supplemental artifacts to the project instead of replacing them. -> [Help 1]

and my env is

[mmusgrov@dev2 narayana-spring-boot] ((HEAD detached at pr/149)) $ ./mvnw -version
Apache Maven 3.9.3<PHONE_NUMBER>9f1ead511c958d89bd2f672198ae9f)
Maven home: /home/mmusgrov/.m2/wrapper/dists/apache-maven-3.9.3-bin/326f10f4/apache-maven-3.9.3
Java version: 17.0.11, vendor: Red Hat, Inc., runtime: /usr/lib/jvm/java-17-openjdk-<IP_ADDRESS>.9-1.fc39.x86_64
Default locale: en_GB, platform encoding: UTF-8
OS name: "linux", version: "6.8.11-200.fc39.x86_64", arch: "amd64", family: "unix"

and it similarly fails running just the relevant test case:

[mmusgrov@dev2 narayana-spring-boot-starter-it] ((HEAD detached at pr/149)) $ pwd
/home/mmusgrov/src/forks/spring/narayana-spring-boot/narayana-spring-boot-starter-it
[mmusgrov@dev2 narayana-spring-boot-starter-it] ((HEAD detached at pr/149)) $ ../mvnw clean install verify -Dit.test=GenericRecoveryIT#testCrashBeforeCommit

although the actual test passes:

[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 32.77 s -- in dev.snowdrop.boot.narayana.generic.GenericRecoveryIT

Why calling install and verify at once? verify is called by install goal.

@marcosgopen : Can you please make a review of this change set from a Narayana perspective. I think the guys able to merge will need expert advice. Thx!

@geoand: Hope to get positive feedback by @marcosgopen soon, to get this merged.

Would anyone be able to share an easy way to run PooledRecoveryIT in a debugger, please?

Would anyone be able to share an easy way to run PooledRecoveryIT in a debugger, please?

I would run ' mvn -Dmaven.failsafe.debug="-agentlib:jdwp=transport=dt_socket,server=y,suspend=y,address=localhost:8000" install -Dit.test=PooledRecoveryIT' from the narayana-spring-boot-starter-it folder

@marcosgopen : Can you please make a review of this change set from a Narayana perspective. I think the guys able to merge will need expert advice. Thx!

I am checking the PR. Do you @graben  have any updates about the Agroal/H2 workaround you mentioned above?

Thank you @marcosgopen ! A few other things that might be relevant to share are to consider adding  -Dmaven.surefire.skip=true -f narayana-spring-boot-starter-it/pom.xml and also I am using ./mvnw
Thank you again - debugging is working.

@marcosgopen : Can you please make a review of this change set from a Narayana perspective. I think the guys able to merge will need expert advice. Thx!

I am checking the PR. Do you @graben  have any updates about the Agroal/H2 workaround you mentioned above?

Has been merged right now by Luis.

Hi @geoand, may you be able to merge this?

Let us know when you need a new release - cc @jacobdotcosta

I'll do, but I think there will come an Agroal and Narayana release update before that. ;)

@tomjenkinson and @graben thanks for your suggestions related to resolving my build issue.


Import How-to related to Configuration Database

Import How-to related to Configuration Database:

https://wiki.egi.eu/wiki/HOWTO20: Registering a new service in GOC-DB: mandatory and optional information
https://wiki.egi.eu/wiki/HOWTO21: How to add URL service endpoint information into GOC-DB

I would be in favour of just moving this documentation under the section about the Configuration Database under https://docs.egi.eu/internal/configuration-database/, instead of keeping it as an "how to" (especially when it's just a list of required fields).

Some changes requested, since we are bringing the docs here, it's a good moment to improve them a bit.
I also think this may fit better under GOCDB documentation instead of general howto

@paolini78 OK for moving this under GOCDB documentation instead of as a general How to?

If left as HOWTO, they also miss some nicer representation at the left menu (now only showing HOWTO21 and not the whole title as others)

I feel that we should keep them in the HOWTOs section, regardless if they refer to a service or another, so these howtos are all collected in a single place. In case a link can be created into the service section.

I personally feel the opposite and that having such a generic section/category is confusing and messy and going the opposite way to what the goal of the docs.egi.eu was: having a documentation site organised around service-specific section that are coherent/complete and things clearly located.
I feel that people need to find the doc where it makes most sense and ease also the management: having the Configuration Database documentation discoverable from the Configuration Database-specific section, not having things spread in the service-specific section and in other places like an How-to/Man section that would be covering only a part of the things (as all other services won't have anything there as it's very HTC specific).
In fact most of the How To could be better placed under the HTC section as they are really linked to HTC and are just noise for people caring only about cloud or DataHub.

I see your point. My worry was that those manuals are mentioned into procedures, created to address an audit finding, and they could have been put in a place together other manuals.
Let's do as you suggest ;)

@paolini78 is this PR fine with you?


Update to actions/checkout@v2, fix checks error: reference is not a tree.

Please answer these questions before submitting pull request

Why submit this pull request?

[x] Bug fix

[ ] New feature provided

[ ] Improve performance

Related issues

Bug fix

Bug description.

Sometime ci and e2e test in github actions will tigger "fatal: reference is not a tree”.
https://github.com/actions/checkout/issues/23

How to fix?

Quote form "V2 fetches a specific SHA and retries with a few delays between before failing. Whereas V1 fetched the merge PR ref." https://github.com/actions/checkout/issues/23#issuecomment-572688577 ，seem problem above is solved in<EMAIL_ADDRESS>submodule function has been removed from V2 version https://github.com/actions/checkout/issues/81 . Whether subsequent submodule functions will be added is still unclear . https://github.com/actions/checkout/issues/81#issuecomment-585935021
From the official documentation of checkout, there is a way to provide submodule functionality through the attachment command line in checkout@v2 https://github.com/actions/checkout#Checkout-submodules ,

New feature or improvement

Describe the details and related test reports.

Agree with @kezhenxu94 , please fix this in the #4418, if this is the issue. Then the master commit log makes more sense.

test checkout@v2 about submodules.

I think mostly this PR works in actions/checkout@v2, @kezhenxu94 Recheck? Also about my comments.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #4419   +/-   ##
=======================================
  Coverage   25.42%   25.42%
=======================================
  Files        1211     1211
  Lines       28083    28083
  Branches     3869     3869
=======================================
  Hits         7140     7140
  Misses      20292    20292
  Partials      651      651

Impacted Files
Coverage Δ

...p/server/core/profile/analyze/ProfileAnalyzer.java
68.51% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

e2e seems influxdb test failed.

FYI @dmsolr

Once you think this PR is ready, remove the [WIP] prefix and ping the reviewers.

get it

@kezhenxu94 Need you recheck.


STAT v2 - Base Module Migration

Base Module migration to Azure Functions

[x] Account Entity Enrichment
[x] Account Enrichment by UPN
[x] Account Enrichment by Id
[ ] Account Enrichment by Sid
[ ] Account Enrichment by DN
[ ] Account Enrichment by Mail (failback if UPN enrichment fails)
[x] IP Entity Enrichment
[x] Host Entity Enrichment
[x] File Entity Enrichment
[x] File Hash Entity Enrichment
[x] Domain Entity Enrichment
[x] URL Entity Enrichment
[ ] STAT Update Check
[x] Add comments
[x] Disable Account comments option
[x] Disable IP comments option
[x] Disable Geo Enrichment Option
[ ] Error Handling - Invalid Account / Failed Enrichment
[ ] Error Handling - Invalid IP / Private IP
[ ] Error Handling - No Entities
[ ] Error Handling - Permissions Issue
[ ] Error Handling - Unexpected error
[ ] Run through validation tests

@piaudonn I like the idea but are any of these official? or any API endpoint?  The only thing I've seen is a json file for download... and the services I've seen are third party and sometimes unreliable (slow/down)

There are created by MSFTees we can check with them. Worst case the json can be downloaded and cached let's say once a day or something.


Hydration errors when using SSRMountStyleProvider with Suspense/Transition during SSR

I've encountered hydration errors when using the component introduced in #335 while implementing Suspense/Transition with Server-Side Rendering (SSR).
Below is a minimal reproducible example demonstrating the issue:
use leptos::prelude::*;
use leptos_meta::{MetaTags, provide_meta_context};
use thaw::ssr::SSRMountStyleProvider;
use thaw::{ConfigDirection, ConfigProvider, Spinner};

pub fn shell(options: LeptosOptions) -> impl IntoView {
    view! {
        <SSRMountStyleProvider>
            <!DOCTYPE html>
            <html lang="en">
                <head>
                    <meta charset="utf-8" />
                    <meta name="viewport" content="width=device-width, initial-scale=1" />
                    <AutoReload options=options.clone() />
                    <HydrationScripts options />
                    <MetaTags />
                </head>
                <body>
                    <App />
                </body>
            </html>
        </SSRMountStyleProvider>
    }
}

async fn fetch_cats(_: u32) -> Vec<String> {
    vec![]
}

#[component]
pub fn App() -> impl IntoView {
    provide_meta_context();
    let dir = RwSignal::new(ConfigDirection::Ltr);
    let (cat_count, _) = signal::<u32>(1);

    let cats = Resource::new(move || cat_count.get(), |count| fetch_cats(count));

    view! {
        <ConfigProvider dir=dir>
            <Transition fallback=move || {
                view! { <Spinner /> }
            }>
                {move || Suspend::new(async move {
                    cats.await
                        .into_iter()
                        .map(|src| {
                            view! { <img src=src /> }
                        })
                        .collect_view()
                })}
            </Transition>
        </ConfigProvider>
    }
}

I'm currently using the latest versions of both Thaw (0.4.0) and Leptos (0.7.2).

I fixed this issue in #347. Can you try it in your project?
thaw = { git = "https://github.com/thaw-ui/thaw", rev = "cfbcad1" }

Thank you for promptly addressing the initial issue; the fix appears to resolve the hydration errors successfully. However, I've encountered another concern. Upon page load, there is a noticeable period where the page renders without the applied, resulting in a flash of unstyled content(FOUC). This behavior suggests that styles are being inserted only after client-side hydration, rather than included during server-side rendering.
Ideally, it should be possible to determine the components that will be loaded at application that the can be and included during the initial server-side render This would prevent FOUC and ensure user experience without discrepancies during the load.

I have found yet another issue, SSR style rendering is appending ConfigProvider style tag, but after hydration this style tag is being duplicated by hydration process.

The problem comes from the fact that hydration and SSR cannot generate the same id. I added a theme_id parameter to the ConfigProvider component for users to specify an id themselves.

It appears to be functioning correctly now, thank you once again for the prompt resolution. I am pleased to report that it works as intended although it still briefly displays unstyled content for approximately 100ms. It seems that the hydrated styles replace generated before sending the HTML to client.


老是报错

运行 Inpaint Anything 总是报内存不足，没有一次成功的，R5 3600   3060ti  RAM16G 是不是这个卡无法运行这个插件，
2024-02-03 20:39:53,872 - Inpaint Anything - ERROR - Allocation on device 0 would exceed allowed memory. (out of memory)
Currently allocated     : 3.26 GiB
Requested               : 2.64 GiB
Device limit            : 8.00 GiB
Free (according to CUDA): 0 bytes
PyTorch limit (set by user-supplied memory fraction)
:<PHONE_NUMBER>4.00 GiB

试试看


Update "publish and subscribe" tutorial

Coverage remained the same at 94.106% when pulling 6c5c1452095de6babbbdd025b6280790aa52e1b8 on rahul-update-publish-and-subscribe into 09976c0fcb686cb86b07306220ae44c94397cfd0 on main.


💡 FR - New syntax support for alert conditions

Summary
Requesting support for more natural syntax options surrounding the creation of alert conditions.
Desired Behavior
Using the New Relic UI, creation of alert conditions allows you to copy both a NerdGraph Mutation snippet and a Terraform snippet.

The current workflow for creating a quickstart alert is to manually transcribe an alert from these natural snippets (or the UI itself) into the YAML syntax we require for the backend service to create an alert. Essentially we are having users create an alert in the UI, copy the NerdGraph mutation, transcribe that into YAML, and then we take that YAML and make a NerdGraph mutation 🙃
Possible Solution
Add direct support for users to simply upload a GraphQL file for alert conditions.


Disable non terra-devel envs till the alt spend profile is deployed to the other envs.

failing unrelated tests. merging to resolve failing test in the downstream.


Feature request: delete item from quickfix / location list.

Hi, one feature I'm missing with this plugin, that vim-qf offers, is:
Remove single lines or multiple lines from the list via :Reject. For example a single line can be removed via:
 vim.api.nvim_buf_set_keymap(0, '', 'dd', ':.Reject<cr>', { noremap = true, silent = true })

Is that something that you would consider for this plugin?

Yeah I think that's a reasonable idea. I've implemented :Keep and :Reject similarly to vim-qf (no pattern filtering, since if you need that I think :Cfilter serves that purpose). I also didn't add any keymap by default since it could potentially conflict with the behavior of other plugins (e.g. quickfix-reflector).


Fixed a typo in "consume"

@bernardosulzbach : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good; thanks @bernardosulzbach.


iptables: Don't use ip{,6}tables if unavailable

Fixes: https://github.com/cilium/cilium/issues/18513.
Fix agent crash when IPv6 is partially disabled in the host kernel.

/test

/test-runtime

/test-1.21-5.4

I'll revert and ask for more information.

I'll remove the needs-backport label then :+1:


Desktop- IOU - Notification is only received for the first edit of IOU request in workspace

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Issue found when executing PR https://github.com/Expensify/App/pull/27299
Action Performed:

Launch New Expensify app.
Create a workspace with admin and member.
[Workspace member] Send a money request to the workspace chat.
[Workspace admin] On desktop app, send a reply in thread to the money request.
[Workspace admin] Open other chat like Concierge and background the app.
[Workspace member] Open the IOU request and edit the amount, and then edit other fields.
Note that

Expected Result:
The workspace admin on desktop app will receive notification for each edit on the IOU request (amount, description, date etc)
Actual Result:
The workspace admin on desktop app only receives notification for the first IOU request edit
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.75-3
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/ccd54c89-0f10-447a-8f80-c93a8216930b
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

i wasn't able to tackle this today. I'll continue it on Monday if it hasn't been tackled yet.

This is a new feature, so I don think its valid blocker, I have also assigned Andrew coming from the original PR

Can't reproduce this on staging v1.3.75-8.
@lanitochka17 it looks like step 4 is incorrect. The admin needs to open the expense and reply in the the expense thread for the notifications to display. NOT in the report thread.
Closing this out. Please re-open/comment if anything else comes up

Issue reproducible on the latest build 1.4.9-2
https://github.com/Expensify/App/assets/78819774/f06b4d23-0307-4f56-ba6a-74199bfdf77e

@lanitochka17 this test isn't correct. It's the same issue as described above.
Employee needs to comment directly on the expense before they can receive notifications for the expense being edited. The comment you added here is on the report:

What regression test are you following? It probably needs to be updated

@lanitochka17 bump ^

@lanitochka17 assigning you since we're waiting on your response.

This issue found when executing PR
#27299.
We will double check if any TR steps need updating

@mvtglobally @lanitochka17 this test IOU Actions - Edit Amount is incorrect. Step 2 should be:

Navigate to the money request details, open the expense details and send a message (this will subscribe the user to notifications)

I also see that these tests do not have that step and they should have them

IOU Actions - Edit Merchant
IOU Actions - Edit Date
IOU Actions - Edit Description

@arosiclair just to confirm, is this even a bug, or are you saying that it's expected behavior?

@arosiclair just to confirm, is this even a bug, or are you saying that it's expected behavior?

Yeah it's expected behavior. Without commenting on an expense, you don't get notifications for edits on the expense (same way it works for commenting on threads).

@mvtglobally it still doesn't look like those tests we're updated

@mvtglobally bump also in slack

@arosiclair after a quick review we modified the test cases to the following:

Edited step 2 in  TC IOU Actions - Edit Amount to be clearer that the message should be sent in the expense/transaction details conversation for the user to be subscribed to notifications.

Before: Navigate to the request details conversation and send a message (this will subscribe the user to notifications)
After: Navigate to the expense/transaction details conversation and send a message (this will subscribe the user to notifications)
For the consequent TCs, to avoid repeating the step of creating a new request and subscribing the user each time, I added a note that it should be executed in the same money request as in the first TC in the IOU Actions series > IOU Actions - Edit Amount. This would save us a bit of time when executing these TCs. Let me know if it's valuable to create a new request and subscribing to each for each edit action TC.
Additionally, we will remind our testers the difference between the 3 conversation types (1:1 or parent conversation, report conv and request conv).

Thank you for the updates they look good to me 👍. In the future please, prioritize any test case updates like this since we'll continue to run tests incorrectly until the updates are done.


Does not remove branches using undefined

If there is a condition such as this:
if (undefined === 'development') {
  // some sensitive code only for development
}

It does not get removed. For reference, I am using the envify transform to do conditional paths based on NODE_ENV at build-time for client-side code

Because undefined could've been reassigned - it's not a constant in old environments. When using envify, try actually defining a value for process.env.NODE_ENV instead of using the "prune" option.

Oh boy...forgot about that.
You're workaround is the one we are currently using.
Before closing this issue, could we add a note/caveat for this case in the README?
Thanks!

Sure! I'd happily accept a PR for that :smile:


Added --no-hosts option

Codecov Report

Merging #4 into master will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
+ Coverage   95.39%   95.43%   +0.04%
==========================================
  Files          40       40
  Lines        2975     3003      +28
==========================================
+ Hits         2838     2866      +28
  Misses        137      137

Impacted Files
Coverage Δ

vnet_manager/actions/manager.py
94.87% <100.00%> (+0.08%)
:arrow_up:

vnet_manager/tests/actions/test_action_manager.py
100.00% <100.00%> (ø)

vnet_manager/vnet_manager.py
90.47% <100.00%> (+0.23%)
:arrow_up:

Continue to review full report at Codecov.

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


Prepare the analyze_once test for removal of analysis_options_user support

Fixes https://github.com/flutter/flutter/issues/138227. Work towards https://github.com/flutter/flutter/issues/82948
In https://github.com/dart-lang/sdk/commit/c033718da0502ed92609f8c6cee2f6a59ccc058e, the analyzer's support for analysis_options_user.yaml is dropped. So no lint is reported in this test, as expected. This PR changes the test to expect no lint, and also no write the edits that trigger the lint.
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.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

CC @goderbauer


1.18.2 backport

What it says on the tin. Tested on a server with a different time rate than vanilla, worked fine.

Thanks for this! It will be a while before I can take the time to properly test/merge this, I'm on a trip until early December. In the mean time, would you mind testing how far back/forward those changes work? I have a feeling that they might work as far back as 1.16 and as far forward as 1.19. When you find a limit, set the gradle properties to the oldest version it worked in. Basically, I'd like for this to cover as many versions as possible when I push it to modrinth.

just checked, it doesn't; prior to 1.18.2, RegistryEntry didn't exist

Sick! Thanks for looking into this


Fix issues with fact page layout.

Changes

Fixes #3771. Cleans up spacing issues on fact page. Also refactors template to remove excessive looping, instead printing two separate groups of facts.

For review: @DoSomething/front-end

:+1:


#general

Was not able to get it to work with "#general". Was receiving "ok": false,
"error": "channel_not_found" So went to https://api.slack.com/methods/channels.info/test to retrieve the channel ID, plugged it in and it worked.

Thanks for the tip!


Added Deep Link support

It appears that after the upgrade to swagger 3.0 we lost all deep linking support: https://github.com/swagger-api/swagger-ui/blob/master/docs/usage/deep-linking.md

I guess it's disabled by default in swagger-ui 3x. You should be able to explicitly enable it with the following:
app.UseSwaggerUI(c =>
{
    ...
    c.EnableDeepLinking();
});


.PHONY targets could all be defined in one line

What issue would you like to bring attention to?
.PHONY targets are not defined in one line at the top.
What is the impact of this issue?
Defining .PHONY targets all over the file makes the Makefile not easy to read.
Do you have a proposed response or remediation for the issue?
It's probably better practice to keep all .PHONY commands together at the top.

Closing due to not best makefile practice - see:  https://clarkgrubb.com/makefile-style-guide

Closing due to not best makefile practice - see:  https://clarkgrubb.com/makefile-style-guide


turn billing account info on project creation

providing a billing account info is optional for project creation. if the developer is not a billing account admin, they should still be able to create projects to try out CFT.

Have you tested all the other functionalities of the template without the billing account attached? For example creation of usage export bucket or enabling APIs?

Hi Tryon,
We really appriciate your contribute, please don't stop! However we do not wish to merge your changes to our codebase because of the following reasons:

Creating a project without a billing account is a really edge case since that project will not be usable at that stage.
Our current Project template supports multiple usefull features such as enabling services, shared VPCs. These would fail without a billing account.

Reflecting on your reasoning:

If the developer is not a billing account admin, they should still be able to create projects to try out CFT

If the developers can't get access to a dummy bulling account, they can't fully test project creation. However they are able to test the other templates of CFT in an existing project with an attached billing account.


Transaction per request / Exception indicator for aspnet mvc middleware

Our team leverages a standard db transaction per http request middleware.  If an unhandled exception bubbles up from lower down the stack, the transaction is rolled back for the entire request.  It seems that HotChocolate is not bubbling these exceptions up - which makes sense, in order to give back nice graphql error messages.  However, we don't see any way to check for these underlying exceptions.  I would be curious if this is something that we are missing, or if there is a different way we should be thinking about transactions in GraphQL.

In GraphQL mutations are considered to be top-level. Everything below is a query and has to be side-effect free.
The important thing to notice here is that query fields are executed isolated and in parallel.
Here are the relevant spec texts and their references:
https://spec.graphql.org/June2018/#sec-Mutation

It is expected that the top level fields in a mutation operation perform side‐effects on the underlying data system. Serial execution of the provided mutations ensures against race conditions during these side‐effects.

https://spec.graphql.org/June2018/#sec-Normal-and-Serial-Execution

Because the resolution of fields other than top‐level mutation fields must always be side effect‐free and idempotent, the execution order must not affect the result, and hence the server has the freedom to execute the field entries in whatever order it deems optimal.

This is a good talk to understand more Facebooks thoughts on GraphQL:
https://www.youtube.com/watch?v=etax3aEe2dA
Also, a good talk is this one from Nick Shock:
https://www.youtube.com/watch?v=zMa8rfXI6MM

If you still want to implement a top-level transaction with nested mutations (which would violate the spec) you can do that by implementing a request middleware. the request middleware can open the transaction path it down as scoped service, so it will scope the full tree. The request middleware has access to the collected errors and can then rollback the selection for the whole request.
Which version are you on? If your project is a new project I would suggest moving to version 11.0.0-dev.12. We have done some major changes with the request pipeline.
with V11:
service
    .AddGraphQLServer()
    .AddQueryType<Foo>()
    .UseRequest(next => context =>
    {
          await next(context);
          if(context.Result.Errors is not null)
          {
               // roll back
           }
     })
     .UseDefaultRequestPipeline();

something like that. If you need help ... join our slack channel.
We will release 11 at the end of October.

Thanks for the prompt response - let us digest these items and figure out the best path!

We have due to a lot of people wanting this added out-of-the box transaction support with version 11.

We have due to a lot of people wanting this added out-of-the box transaction support with version 11.


Telemetry?

Curious if a firestormforum.org telemetry package is going to be included.
A way to gather stats on deployments and usage.
obvious one is which browsers are people using to access Firestorm Forum sites and what size windows they are using.
details for implementation:

default off
any stats anonymised
ask admin if they want to participate

show exactly what is included
link to public page showing the stats gathered

show admin the stats of local firestorm forum even if not participating in sharing

have share on/off button on this page/view

What do you think, @knewter ?
@SoreGums do you have time to help us on this ?


feat: update migrate cli import

Tests need to be rerun after merging https://github.com/Avaiga/taipy-core/pull/786


Support toggle click in leaderboards to exclusively select a dimension value

closes #3848
This adds a simple cmd/ctrl + click handler to leaderboard and dimension table. Doing this on an unselected value will select just that value and deselect everything else. Doing this on a selected value will just deselect it similar to a normal click.

This interaction feels great! Only thing is we should probably have a line item in the tooltip to callout that this interaction is possible.

I'm seeing:

The tooltip in the expanded dimension table doesn't include the informational copy
The interaction doesn't work in the Time Dimension Detail table (but maybe we can keep this as future work)

Ya some other tool tips also do not show up in dimension table. Also we decided this feature will not be added to TDD for now.


OpenSSF Best Practices Badge for devfile/api

Which area/kind this issue is related to?

/area api
Issue Description

This task aims to complete the OpenSSF Best Practices Badge as part of a greater effort to improve our adherence to best practices across all repositories. In order to complete this task the badge should be completed to 100%. You can find the badge on the OpenSSF Best Practices site and it clearly outlines the necessary tasks that must be completed.
At the bottom of the link there should be an edit button available to anyone who has commit permissions to the repository.
Acceptance Criteria

[ ] Badge completed to 100%

Pulling into sprint 253 as all my current items in the sprint are in review/done. Issue should not take long and I don't expect it to carry over to 254.

Badge completed and can be viewed here as well as in the readme. Anyone who wants to make additions or changes to the badge can do so if they have commit access to the repo.


Support updating aqua-renovate-config in config preset(default.json)

For now updating aqua-renovate-config in default.json isn't supported.
default.json is recommended filename of config presets.

https://github.com/aquaproj/aqua-renovate-config/pull/449
https://github.com/aquaproj/aqua-renovate-config/pull/450
https://github.com/aquaproj/aquaproj.github.io/pull/695

1.13.0 is out 🎉
https://github.com/aquaproj/aqua-renovate-config/releases/tag/1.13.0

It works well.

https://github.com/suzuki-shunsuke/test-renovate/pull/239


pipenv couldn't display progress bar when it downloads package

I sometimes download big package such as pytorch.
When my network speed is slow, the progress of downloading will take long time to execute.
The output in the consolse just spin and don't give me more feedback like download speed
or complete rate.
I have try add --verbose but don't work as my exception.
I could use pip to isntall and use pipenv to lock, but this solution not sound good.
I am wondering if pipenv have feature like $ pip install torch --progress-bar=on
and something like this:
  Downloading https://files.pythonhosted.org/packages/69/43/380514bd9663f1bf708abeb359b8b48d3fabb1c8e95bb3427a980a064c57/torch-0.4.0-cp36-cp36m-manylinux1_x86_64.whl (484.0MB)
    0% |                                | 563kB 248kB/s eta 0:32:28

Hi! Thanks for the report. Unfortunately I feel I am not able to comprehend the situation well. Could you elaborate when you want the download progress to appear, and suggest how we could add this to our current output? It would help immensely if you could provide a fake output to illustrate how you imagine the end result would look like if we implement this.

Thanks for your reply.
I mean, sometimes when we want to install a big pypi package like pytorch (the pytorch's .whl package size is 400+Mb), and one's internet speed is slow (say 400Kb/s). In this situation, we need to wait about 17 minutes to finish downloading. During this time, the user cannot add options to pipenv to display package downloading speed or complete rate. The normal pipenv output to console during download pypi package is as follow:
$ pipenv install torch -v
Installing torch…
⠋Installing 'torch'
⠙$ "/home/lennon/.local/share/virtualenvs/test-yq5-nVUR/bin/pip" install   --verbose    "torch" -i https://pypi.org/simple --exists-action w
⠸

What I want is something like wget and pip's progress-bar function.
say command $ wget https://pjreddie.com/media/files/yolov3.weights
the wget will output to console the speed, complete rate, remain time to complete as follow:
--2018-07-02 20:19:50--  https://pjreddie.com/media/files/yolov3.weights
Loaded CA certificate '/etc/ssl/certs/ca-certificates.crt'
Resolving pjreddie.com (pjreddie.com)... <IP_ADDRESS>
Connecting to pjreddie.com (pjreddie.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 248007048 (237M) [application/octet-stream]
Saving to: ‘yolov3.weights’

yolov3.weights            0%[                               ] 455.70K  24.8KB/s    eta 2h 25m

And we can found the pip's additional options in https://pip.pypa.io/en/stable/man/commands/install/?highlight=progress-bar#cmdoption-progress-bar
when we use pip to download package, it will provide enough infomation for user about package downloading information.
the pip's output is something like wget:
$ pip install torch --progress-bar=on
Collecting torch
  Downloading https://files.pythonhosted.org/packages/69/43/380514bd9663f1bf708abeb359b8b48d3fabb1c8e95bb3427a980a064c57/torch-0.4.0-cp36-cp36m-manylinux1_x86_64.whl (484.0MB)
    0% |                                | 143kB 20kB/s eta 6:36:45

So, my suggestion is that maybe pipenv could provide a option mentioned previously. The user will know how long he could leave alone pipenv's installation and handle other things.

I realize that the pipenv finally call pip_install and delegate installation command to subprogress in https://github.com/pypa/pipenv/blob/master/pipenv/vendor/delegator.py#L152
Could we get more detail information output from subprocess and display it in parent progress's console, for example https://stackoverflow.com/questions/18421757/live-output-from-subprocess-command ?

any progress for this feature?

any progress on this?

any progress on this?

any progress on this?

any progress on this?

any progress on this?

no progress on this. but I do this to check my downloads: du -h ~/.cache/pipenv. It slowly grows when files have finished downloading, so at least you see that something is happening.

What version of pipenv and what version of bash?  This isn't an issue for me on linux, OS X or windows -- I see the progress bar rendering.

@matteius 2022.1.8 on zsh on Linux. And yeah I see the spinner with the text "Locking ...", but no download indicator that says X/YMB or time estimates.

Progress bar has been removed in an optimization to batch install which cut the average install time in half.


Add a cleanup file

It would be nice to have something like the "startup file", a script that runs when Symphony starts, for cleanup. A "cleanup file" sounds right.

Closed with 9e668aeb05ff49fecf377ca63db31c598491f868


Cannot launch Interface with detached launching (default) if launcher is running as admin

This is because it detects the path it is in as system32 instead of the actual launcher executable location.

Fixed in 0.0.1r


Setup the project

Gitignore
Parent pom
Checkstyle
Commitlint
Readme

Reopen to rename the modules (add the project name as a prefix)


Issue #345 fixed

Just updated the reference by adding type under monolog.handlers.redis
        redis:
            type: redis
            redis:
                #id: optional if host is given
                host: <IP_ADDRESS>
                password: null
                port: 6379
                database: 0
                key_name: monolog_redis

The type attribute have to be configured whatever the value inside the configuration (this is not just about redis, but all the handler types)
The purpose of this docbloc is to list the different options for each type. See comment on top of the docbloc you are updating https://github.com/symfony/monolog-bundle/blob/a9122a26bcc08ba11a8c9d8cbc6ce168adff1e86/DependencyInjection/Configuration.php#L25


Not able to open vs code on macos Catalina

i have  updated macos to Catalina
i have installed vscode from here https://code.visualstudio.com/docs/?dv=osx
now i am getting this issue. The application “Visual Studio Code 3” can’t be opened.
also in system preference i am not getting any option to open the editor

Can you paste the output of spctl --verbose --assess --type execute --v <path-to>/Visual\ Studio\ Code.app/ ?

I got the same error.
$ spctl --verbose --assess --type execute --v /Applications/Visual\ Studio\ Code.app
/Applications/Visual Studio Code.app: accepted
source=Notarized Developer ID
override=security disabled
origin=Developer ID Application: Microsoft Corporation (UBF8T346G9)

is there any advice to solve this question?


DS-2315: export Simple Archive Format without bitstreams

https://jira.duraspace.org/browse/DS-2315

+1, code looks correct and and passes tests.

Refactored patch to be compatible with new service-based api (DS-2701), please remove "merge conflict" tag.

I'm going to clean this up, please do not merge yet

Done the clean up, please remove "work in progress".

Broke this PR, closing and creating a new, clean one.

New clean PR instead of this one: https://github.com/DSpace/DSpace/pull/1210


Editor doesn't insert white spaces when formatting html files.

VSCode Version: 1.6.1
OS Version: macOS Sierra 10.12

Steps to Reproduce:
1.
Make sure that you set
"editor.insertSpaces": true
in your workspace settings
2.
Make sure you are using indentation with spaces and indentation size 2 in my case.
3.
Try to format html file
Expected result:
Editor should preserve white spaces as tabs.
Result:
Editor converts spaces to tabs. It should not.
Additional comment:
It works fine when saving typescript file for example.
The white spaces are preserved correctly.

Are you formatting with an extension, or the built-in vscode format?

Build in VS Code formatter.

This is #13563


[BUG] - Underestimation of Mem and Cpu units with transaction build

Area
Plutus    Related to Plutus Scripts (Alonzo).
cardano-cli Relaated to the Cardano-CLI
Summary
Creating a smart contract transaction with cardano-cli transaction build results in an inaccurate mem and cpu estimation when using a fee-paying input UTxO that contains a token that must be returned to the change address using the --change-address. The build function builds the transaction but fails at submission with the overspending budget error.
The machine terminated part way through evaluation due to overspending the budget.

In the example smart contract I am using, when the fee paying input is:
deb0785b9b65de8872e423180c3ee9e818f90118ec80b4efa248fc205380d1e2     1<PHONE_NUMBER> lovelace +<PHONE_NUMBER> 698a6ea0ca99f315034072af31eaac6ec11fe8558d3f48e9775aab9d.7444524950

The tx.draft assumes the cost is {5: [[0, 0, 121([]), [1649252, 510827641]]]} where the --calculate-plutus-script-cost parameter returns
[
  {
    "executionUnits": {
      "memory": 1678012,
      "steps": 519565179
    },
    "lovelaceCost": 134282,
    "scriptHash": "e606f584d5910a03eca95243ac489a37dcc040859b8a9b465533c215"
  }
]

The estimation using calculate-plutus-script-cost is correct but the estimation with transaction build is incorrect.
By removing the tokens from the input the transaction build calculates the correct fee and mem/cpu units.
This input lets build and --calculate-plutus-script-cost to return the same mem and cpu units.
86dd709f3740b81b3231799d4990aa526817c972c676b445514d0bbf43479dfb     1<PHONE_NUMBER> lovelace

This should be reproducible with any contract.
 Clearly, it can calculate it because --calculate-plutus-script-cost is correct but that information is not being put into the out-file resulting in the overspending of the budget error.
System info:
Ubuntu 22
cardano-node 8.1.1 - linux-x86_64 - ghc-8.10
git rev 6f79e5c3ea109a70cd01910368e011635767305a
cardano-cli 8.1.1 - linux-x86_64 - ghc-8.10
git rev 6f79e5c3ea109a70cd01910368e011635767305a

Any work on this in the new version of the cli?

Its more than a year now and this bug still exists. It was brought up https://github.com/IntersectMBO/cardano-api/issues/302 but it was closed.
What else is required to get some attention on this?

Ah good catch I was using the  cardano node 9.1.1.
Just confirmed that it exists in all cardano-node tagged releases before the cardano-api update to <IP_ADDRESS>. So cardano node tag  versions 9.2.0+ should work as far as I can tell from my testing but all older versions have the bug.
Pretty sure this commit fixed it.
If I run into it again, I'll make a new issue but I will close this for now assuming it is solved in cardano-api <IP_ADDRESS>. Thanks @carbolymer


🚀 - Padding 0 for decimals to precision

Which package(s) are relevant/related to the feature request?
@maskito/kit
Description
We need two modes, padding zeroes to specific precision and leaving decimals untouched. In Taiga UI leaving decimals untouched is currently done with Infinity passed as precision, but we might also need to be able to set particular precision without zero padding.

https://github.com/Tinkoff/taiga-ui/pull/4114


Wechat Miniprogram bind user phone number needs to be updated

Wechat Bind phone number API should be updated to use new target url, currently it is outdated.
See

@slarkerino 感谢通知，你可以为它做升级吗？

@gdlcf88 I have created a PR on abp.wechat repository. Plz check and merge so I can update this repo :P


Allow incrementals publishing for Stapler

See stapler/stapler#209 (comment). Stapler incrementals publishing regressed because github.index.json no longer has an entry for stapler/stapler (caused by #1806). I ensured that with this change github.index.json again contains an entry for stapler/stapler.

@daniel-beck ^^^?

I think it was along the lines of not showing issue trackers that we don't have some control over.
i.e. if objectionable content shows up, and we scrape it and show it on the website. We have control over our own Jira and GitHub org but not other ones.
Like we don't accept plugin docs hosted on GitHub outside of our org.
The isGitHubIssues shouldn't affect stapler as it's mostly for Jenkins core and the plugin site to know where the issue tracker is, I can't see stapler needing that

Note that @oleg-nenashev had some plan, long paused, to move Stapler and other components into @jenkinsci, which would make these sorts of things simpler.

I do not follow why the check for the org prefix was introduced in #1806 at all.

I think it was along the lines of not showing issue trackers that we don't have some control over.

Jenkins plugins are expected to be hosted in the jenkinsci org. We're not actively removing plugins that got in before this was actively enforced through the hosting process, but we're not supporting outside hosting either.


Option to keep submit errors on change

In my current project we have high accessibility requirements (e.g. blind people must be able to use the site) and a thing that has been reported as problematic is the disappearance of form validation error messages before the form is submitted.
Currently redux-form is hard-coded to clear the errors always on field change. I'd like to change that so that reduxForm can take a clearErrorsOnChange: false parameter which will disable this feature. I think that only the submitError property should be affected by this parameter, and not asyncError. What do you think? Would clearSubmitErrorsOnChange or some other name be better?
I have a prototype of the change already working locally. Would you like me to make a pull request? Should I make it for v5 and/or v6? We're still using v5 for now, so I can make a pull request for both versions.

I've created a pull request for v6. I also have a branch for v5, but that doesn't handle _error, so I'm not making a pull request for it, considering that we'll anyways sooner or later migrate to v6.

Fix published in v6.0.3.


MeteorCollection

hey @siegesmund
I'm getting two issue when trying to follow the dictionary way (Simple in-memory persistence)
below

Cannot assign value of type MeteorCollection<.MeteorDocument.>  to type 'MeteorCollection'
Generic parameter 'T' could not be inferred when using on table view functions like "collection.count"

Issue solved
On https://github.com/siegesmund/SwiftDDP/blob/master/Examples/Dictionary/SwiftTodos/Lists.swift
var collection:MeteorCollection = (UIApplication.sharedApplication().delegate as! AppDelegate).lists
I followed this way then I got that problem there was type mismatch... my mistake when removed ":MeteorCollection" it worked fine...

Glad you solved it!

thanks...but facing another issue like the collection list  it not initilized even after successful subscription. Like I declare a variable (a class subclassed from MeteorDocument) on appdelegate and use it on other VC and subscribe there just like as you demontrated but then later found out that value is not getting populated , do I need explicitly populate any how?

You must create a collection that will contain the documents, not a document. Here is an example, from the documentation:
class List: MeteorDocument {

    var collection:String = "lists"
    var name:String?
    var userId:String?

}

let lists = MeteorCollection<List>(name: "lists")   // As with Meteorjs, the name is the name of the server-side collection
Meteor.subscribe("lists")

You might declare the collection in your AppDelegate. It doesn't make any sense to declare a document there.

By the way, per your original question: you're running into the basics of Swift's strong typing.
You can write it as:
var collection:MeteorCollection = (UIApplication.sharedApplication().delegate as! AppDelegate).lists

or, more commonly, you can cast it:
var collection = (UIApplication.sharedApplication().delegate as! AppDelegate).lists as! MeteorCollection

This is something that you will run into repeatedly in Swift.

actually that was not my question but I'm clear now.

Great!


Intellisense is not working

Issue Type: Bug
Hi, code editor does not recommend me any function or variable names. Also, it doesn't let me to ctrl+click on paths to see files inside of these paths.
 Please provide us with a small example that shows the problem for you. Thank you.

/needsMoreInfo

I am using jsx @kieferrm
Here is the gif

same here

I am using : Visual Source Code Version :  1.41.1
IntelliSense is not working.
I search som many blogs and tried to fix it but no use.
I am working with ReactJs snippets, Please help me to fix this.
For me it should be like this 👍
When i open the file and try import, it is working. This is reproducible when when i come to second line of code.

Are you on Windows? If yes, is your project on a different drive to the one you installed VS Code on? If yes, try one of the three:

Move your project to the same drive and try again
Install the 'JavaScript and TypeScript Nightly' extension and use typescript@next
For your project, run npm i -D<EMAIL_ADDRESS>then opening VS Code, bringing up the command palette (F1) > select typescript version and using workspace version

i followed all steps but i can't resolve problem
its still loading.....
On Thu, Dec 26, 2019 at 6:31 AM Cellv2<EMAIL_ADDRESS>wrote:

Are you on Windows? If yes, is your project on a different drive to the
one you installed VS Code on? If yes, try one of the three:

Move your project to the same drive and try again
Install the 'JavaScript and TypeScript Nightly' extension and use
typescript@next
For your project, run npm i -D<EMAIL_ADDRESS>then opening VS
Code, bringing up the command palette (F1) > select typescript version and
using workspace version

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/87635?email_source=notifications&email_token=ANCRJNDBY52VXSM6C3YAXUTQ2P675A5CNFSM4J64MXA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHUWAZY#issuecomment-568942695,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANCRJNDAYMILG32J5NLA7N3Q2P675ANCNFSM4J64MXAQ
.

Me also same. If I try previous version, it works well (VS code : )
On Sat, Dec 28, 2019 at 2:12 PM sonivipul89<EMAIL_ADDRESS>wrote:

i followed all steps but i can't resolve problem
its still loading.....
On Thu, Dec 26, 2019 at 6:31 AM Cellv2<EMAIL_ADDRESS>wrote:

Are you on Windows? If yes, is your project on a different drive to the
one you installed VS Code on? If yes, try one of the three:

Move your project to the same drive and try again
Install the 'JavaScript and TypeScript Nightly' extension and use
typescript@next
For your project, run npm i -D<EMAIL_ADDRESS>then opening VS
Code, bringing up the command palette (F1) > select typescript version
and
using workspace version

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<
https://github.com/microsoft/vscode/issues/87635?email_source=notifications&email_token=ANCRJNDBY52VXSM6C3YAXUTQ2P675A5CNFSM4J64MXA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHUWAZY#issuecomment-568942695
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ANCRJNDAYMILG32J5NLA7N3Q2P675ANCNFSM4J64MXAQ
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/87635?email_source=notifications&email_token=AN7VLTIYL3QR6LQKUYSOGODQ24GP5A5CNFSM4J64MXA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHYFMQI#issuecomment-569398849,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN7VLTO2KS2W56V2F6PQS7DQ24GP5ANCNFSM4J64MXAQ
.

@emakina-efeuruk What driver letter is your code stored on?


Remove myself from the infrastructure team.

haven't had much time to do infra things for a few months, don't see that changing anytime soon, so removing myself from the roster

r? @aturon


Atom installed, doesn't show up

I'm running the latest version of chocolatey (installed in two days ago) on Windows 10.
I've installed Atom via choco, but it doesn't show up anywhere on my system.  When I try installing it in an admin powershell:
PS C:\Windows\system32> choco install atom
Chocolatey v0.10.15
Installing the following packages:
atom
By installing you accept licenses for the packages.
atom v1.58.0 already installed.
 Use --force to reinstall, specify a version to install, or try upgrade.

Chocolatey installed 0/1 packages.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).

Warnings:
 - atom - atom v1.58.0 already installed.
 Use --force to reinstall, specify a version to install, or try upgrade.

But I don't see an Atom folder in Program Files or similar.  I searched my entire computer for "Atom", and the only files are in C:\ProgramData\chocolatey.

It looks like this is an issue with the Atom package. You'll want to reach out to the package maintainer either through the comments on the package page: https://community.chocolatey.org/packages/atom or through the package source repository.


Test with julia-v0.4

Tests pass with julia-v0.4.0-rc1. The one issue I had was with PyCall.jl, but after checking out master (Pkg.checkout("PyCall"); Pkg.build("PyCall")), everything works.


Adding FreeAgent to the list of Lookbook users

Adding FreeAgent to the list of Lookbook users

Awesome - thank you :-)


Syncing with upstream

This PR merges the changes from bch-wallet-web3-android that switched from React Class components to Functional components.

:tada: This PR is included in version 1.5.0 :tada:
Your semantic-release bot :package::rocket:


Create code of conduct

It is encouraged by GitHub to comply with recommended community standards.

Thank you so much for this
Sent with GitHawk

My pleasure!

Thanks @rodrigobdz!
Sent with GitHawk


Assorted Jenkins CI Improvements

[x] Perform a merge from master before build/test
[x] Set up email reporting directed at people who submitted PR and maintainers
[x] Add clang builds to the build/test matrix
[x] Jenkins CI documentation
[x] Add a nightly build of OE with email reporting
[ ] Puppet or Ansible configuration management for all systems in the Jenkins CI domain
[x] Enable libc/libcxx tests run at a regular cadence with master and an optional parameter for PR builds
[ ] Add a testrun for inside a docker container on an ACC VM in hardware mode
[x] Backups of Jenkins master configuration
[x] Add a "build passing/failing" button to the main OE github page build never fails with Bors
[ ] Investigate whether it is possible to change the Jenkins Blue Ocean UI timing that quickly jumps you to the bottom of the page.
[x] Transition OAuth application to be owned by oeciteam when logging into Jenkins
[x] Investigate adding bors-NG to automate build-merging
[x] Replace read/write OECITeam key with read-only deply keys
[x] Transition all Jenkins CI resources to the same resource group/region so they can all be on the same private network, allowing better security

We have "configuration management" of sorts for the build environment in the sense that we have Dockerfiles describing the state required to build Open Enclave. While it would be nice to have Puppet or Ansible style configuration management of the Jenkins master and nodes (and Bors), we have extensive documentation on the manual setup of the nodes, and the Jenkins master and Bors (and its database) all run out of containers.

Considering this closed now, thanks @johnkord.


fix/Do not read the whole CSV file every time we want to write.

Before submitting this PR, please make sure:

[X] Your code builds clean without any errors or warnings
[X] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
There is no need to read the whole CSV file in memory (and spend time parsing it as well) if we are only checking the first row

LGTM
-- Review submitted in representation of the Hummingbot Foundation Technical DAO

Sorry, forgot to rebase with upstream dev earlier

Steps:

clone and install feature branch
create/start pureMM strategy with tight spreads
wait for multiple trades filled

Bot not recorded second trade filled and return that error:
csv5459.zip
2022-07-15 10:36:05,421 - 74338 - hummingbot.core.event.event_reporter - EVENT_LOG - {"timestamp":<PHONE_NUMBER>.0, "type": "OrderType.LIMIT_MAKER", "trading_pair": "ETH-DAI", "amount": "0.01000000", "price": "1217.48000000", "order_id": "x-XEKWYICXSEHDI182016e0a098cb05b", "creation_timestamp":<PHONE_NUMBER>.0, "exchange_order_id": "452529104", "leverage": 1, "position": "NIL", "event_name": "SellOrderCreatedEvent", "event_source": "binance"}
2022-07-15 10:36:13,048 - 74338 - hummingbot.connector.client_order_tracker - INFO - The SELL order x-XEKWYICXSEHDI182016e0a098cb05b amounting to 0.01000000/0.01000000 ETH has been filled.
2022-07-15 10:36:13,048 - 74338 - hummingbot.strategy.pure_market_making.pure_market_making - INFO - (ETH-DAI) Maker sell order of 0.01000000 ETH filled. [clock=2022-07-15 10:36:13+00:00]
2022-07-15 10:36:13,054 - 74338 - hummingbot.connector.exchange.binance.binance_exchange.BinanceExchange - ERROR - Unexpected error while processing event 107.
Traceback (most recent call last):
  File "hummingbot/core/pubsub.pyx", line 165, in hummingbot.core.pubsub.PubSub.c_trigger_event
    typed_listener.c_call(arg)
  File "hummingbot/core/event/event_listener.pyx", line 25, in hummingbot.core.event.event_listener.EventListener.c_call
    self(arg)
  File "/home/ubuntu/csv5459/hummingbot/core/event/event_forwarder.py", line 24, in __call__
    self._to_function(self.current_event_tag, self.current_event_caller, arg)
  File "/home/ubuntu/csv5459/hummingbot/connector/markets_recorder.py", line 274, in _did_fill_order
    self.append_to_csv(trade_fill_record)
  File "/home/ubuntu/csv5459/hummingbot/connector/markets_recorder.py", line 319, in append_to_csv
    if (os.path.exists(csv_path) and (not self._csv_matches_header(csv_path, field_names))):
  File "/home/ubuntu/csv5459/hummingbot/connector/markets_recorder.py", line 303, in _csv_matches_header
    return tuple(df.iloc[0].values) == header
AttributeError: 'TextFileReader' object has no attribute 'iloc'

Hi @cheemeng, can you please fix the issues stated by QA? And also please fix github unit tests to pass. Thanks


Fix incorrect time measurement by using monotonic time

Statsd#time uses the Time class to measure the difference between
elapsed time. This is incorrect because Time.now uses realtime,
which is meant to be used for system clock.
clock_gettime from time.h supports monotonic time. Let me quote
the man page of clock_gettime man to demonstrate the difference
between them:

CLOCK_REALTIME
the system's real time (i.e. wall time) clock, expressed as the
amount of time since the Epoch.

CLOCK_MONOTONIC
clock that increments monotonically, tracking the time since an
arbitrary point, and will continue to increment while the system is
asleep.

Since we measure elapsed time, we should be using monotonic time,
since it more accurate and meant to be used for that reason.

The errors in the build seem to be unrelated to the changes here.

Thank you!


Subscribe_position to Display Drone Position

Hi,
am trying to receive drone position and display it on a QT GUI qlineedit.
I have the code for position, but the problem when I test it using px4 I received the position once, and I need a continuous movement. The code is same as the takeoff and landing example.
`    Mavsdk mavsdk;
ConnectionResult connection_result = mavsdk.add_any_connection("udp://:14540");
if (connection_result != ConnectionResult::Success) {
    std::cerr << "Connection failed: " << connection_result << '\n';
    return 1;
}
auto system = get_system(mavsdk);
if (!system) {
    return 1;
}

// Instantiate pluginis.
auto telemetry = Telemetry{system};
auto action = Action{system};

// We want to listen to the altitude of the drone at 1 Hz.
const auto set_rate_result = telemetry.set_rate_position(1);
if (set_rate_result != Telemetry::Result::Success) {
    std::cerr << "Setting rate failed: " << set_rate_result << '\n';
    return 1;
}

telemetry.subscribe_position([&](Telemetry::Position position) {
    emit updateAvAlt(position.relative_altitude_m);
    emit updateAvLatLong(position.latitude_deg, position.longitude_deg);
});`

and the emit call the display function.
could you please help?

Your callback in telemetry.subscribe_position([&](Telemetry::Position position) { should be called for each position update. Is that not the case?

telemetry.subscribe_position

Does this function "telemetry.subscribe_position" call the position one time? or more than one?
I thought setting the rate as this "const auto set_rate_result = telemetry.set_rate_position(1)" will call the position several time


Add my first file

I have created a new branch, created a file and made a commit!

Am I doing something wrong?


Provide ids for checks, turn duplicate-middleware into warning?

there should be ids for the checks, so that they can be ignored easily

I have not checked if I am affected by #986, but I have a valid use case for having the middleware in there twice: I want it as early as possible, but then also after AuthenticationMiddleware (for custom request.user checks).  I would argue that this should be a warning instead, so that it does not fail hard to start runserver for example.

Code: https://github.com/jazzband/django-debug-toolbar/blob/27f8a70953aebbd4949dc34ef552ce441d463a40/debug_toolbar/apps.py#L49

Fair enough, but does it even work if it's included twice?
And sure: Adding IDs to the errors sounds good!

It used to work at least.. ;)
I am using this:
def show_debug_toolbar(request):
    from debug_toolbar.middleware import DebugToolbarMiddleware
    import threading

    if (DebugToolbarMiddleware.debug_toolbars.get(
            threading.current_thread().ident)):
        return False
    if not int(request.GET.get('my_enable_ddt', not request.path.startswith(
            reverse('some-path')))):
        return False
    if hasattr(request, 'user'):
        return (request.user and
                request.user.groups.filter(name='Debug').exists())
    from debug_toolbar.middleware import show_toolbar as ddt_show_toolbar
    return ddt_show_toolbar(request)

So it checks if it has been enabled already:
    if (DebugToolbarMiddleware.debug_toolbars.get(
            threading.current_thread().ident)):
        return False


How to import translations ?

How to import translations from resources/lang/en/all.php ?
Lang.addMessages({         "en": {             "partnership": "Create report",         }     });     Lang.setLocale("en");     console.log(Lang.get('partnership'));

I wan't to import my translation, not build new one...

I am not sure if I get what your question is... You posted some JS snippet that is supposed to work.
What do you mean be "import, not build new one"?

How to import translations from resources/lang/en/all.php to .js file ?

See https://github.com/andywer/laravel-js-localization#configuration.
So if you use it like Lang::get('all.xy') in PHP, then configure it like:
    ...
    'messages' => [ 'all' ],
    ...

Hi @andywer ,
I have have a Italian language folder in resources/lang.
I added the Italian to config file :
'locales' => ['en', 'it']
And added my translations into messages :
'messages' => [         'test'     ],
When I viewing the page in English everything works fine, when I switch to Italian I am getting error :
localization.js:1 Uncaught Error: No messages defined for locale: "it". Did you forget to enable it in the configuration?
I checked the message.js file and it doesn't contain the IT translations in.
Any ideas why ?

Hey! No, it's hard to say. Did you do a php artisan js-localization:refresh?

@andywer, thanks for your reply.
I disabled the cache in config : 'disable_config_cache' => true.
In case, I ran `php artisan js-localization:refresh too but no success.

Sorry, @hpolonkoev. Can't tell from here...
I let you know if I have another idea.
Please let me/us know if you solve it :)

@andywer thanks anyway!

Hi, i use this config:
 ['ru'],

    /*
    |--------------------------------------------------------------------------
    | Define the messages to export
    |--------------------------------------------------------------------------
    |
    | An array containing the keys of the messages you wish to make accessible
    | for the Javascript code.
    | Remember that the number of messages sent to the browser influences the
    | time the website needs to load. So you are encouraged to limit these
    | messages to the minimum you really need.
    |
    | Supports nesting:
    |   [ 'mynamespace' => ['test1', 'test2'] ]
    | for instance will be internally resolved to:
    |   ['mynamespace.test1', 'mynamespace.test2']
    |
    */

    'messages' => [
        'status'
    ],

    /*
    |--------------------------------------------------------------------------
    | Set the keys of config properties you want to use in javascript.
    | Caution: Do not expose any configuration values that should be kept privately!
    |--------------------------------------------------------------------------
    */
    'config' => [],

    /*
    |--------------------------------------------------------------------------
    | Disables the config cache if set to true, so you don't have to
    | run `php artisan js-localization:refresh` each time you change configuration files.
    | Attention: Should not be used in production mode due to decreased performance.
    |--------------------------------------------------------------------------
    */
    'disable_config_cache' => true,

];
but in section @yield('js-localization.head') in layout i have code:

i fink, need  ?

sorry ,my english) i hope u will understand me

Hey @kocapb.
I guess you are lacking a App::setLocale('ru') in your PHP code. Does App::getLocale() return the expected result?

thanks a lot, bug resolved


Chart height option is ignored when series is an empty array

If the series prop passed to the VueApexCharts component is an empty array ([]), the chart ignores the given chart.height option.
example:
  <div style="width: 300px; height: 100px; border: 2px solid red;">
    <VueApexCharts
      type="line"
      :options="{
        chart: {
          height: 100,
        },
        noData: {
          text: 'No data as expected'
        },
      }"
      :series="[]"
    />
  </div>

renders as

I've made a reproduction here, it shows the bug with Series as [] and as [{data:[]}]


fix: search wallet balance

Summary
https://app.clickup.com/t/2pm0xja
requires https://github.com/ArdentHQ/arkscan/pull/55

Checklist

[ ] I checked my UI changes against the design and there are no notable differences
[ ] I checked my UI changes for any responsiveness issues
[ ] I checked my UI changes in light AND dark mode
[ ] I checked my (code) changes for obvious issues, debug statements and commented code
[ ] I opened a corresponding card on Clickup for any remaining TODOs in my code
[ ] I added a short description on how to test this PR (if necessary)
[ ] Documentation (if necessary)
[ ] Tests (if necessary)
[ ] Ready to be merged

can be updated @alexbarnsley


XP: Fix GetProcAddress() usage of CreateThreadpoolWork()

It was properly delay-loading kernel32.dll to find these entrypoints, but then using the non-GetProcAddress() calls.

Oops, thanks for fixing that for me.


General Grievance: Watch backlink

General Grievance requests the watch of the watch_keyword backlink. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Jeff Schaller in Charcoal HQ

backlink has been seen in 413 true positives, 102 false positives, and 17 NAAs.


Fix indentation

The different indentation of items causes toc cannot be correctly parsed, thus, the ops build/publish has failed.

Docs Build status updates of commit b9d8a3d:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

windows-iotcore/TOC.yml
:x:Error

Details

windows-iotcore/TOC.yml

[Error] W:\ni2g-s\windows-iotcore\TOC.yml is not a valid TOC File: W:\ni2g-s\windows-iotcore\TOC.yml is not a valid TOC file, detail: (Line: 193, Col: 20, Idx: 6400) - (Line: 193, Col: 20, Idx: 6400): Mapping values are not allowed in this context..
[Error] Unable to load file 'TOC.yml' via processor 'TocDocumentProcessor': W:\ni2g-s\windows-iotcore\TOC.yml is not a valid TOC File: W:\ni2g-s\windows-iotcore\TOC.yml is not a valid TOC file, detail: (Line: 193, Col: 20, Idx: 6400) - (Line: 193, Col: 20, Idx: 6400): Mapping values are not allowed in this context..
[Error] W:\ni2g-s\windows-iotcore\TOC.yml is not a valid TOC File: W:\ni2g-s\windows-iotcore\TOC.yml is not a valid TOC file, detail: (Line: 193, Col: 20, Idx: 6400) - (Line: 193, Col: 20, Idx: 6400): Mapping values are not allowed in this context..
[Error] Unable to load file 'TOC.yml' via processor 'ExperimentalTocDocumentProcessor': W:\ni2g-s\windows-iotcore\TOC.yml is not a valid TOC File: W:\ni2g-s\windows-iotcore\TOC.yml is not a valid TOC file, detail: (Line: 193, Col: 20, Idx: 6400) - (Line: 193, Col: 20, Idx: 6400): Mapping values are not allowed in this context..

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

Docs Build status updates of commit b9d8a3d:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

windows-iotcore/TOC.yml
:x:Error

Details

windows-iotcore/TOC.yml

[Error] W:\kyir-s\windows-iotcore\TOC.yml is not a valid TOC File: W:\kyir-s\windows-iotcore\TOC.yml is not a valid TOC file, detail: (Line: 193, Col: 20, Idx: 6400) - (Line: 193, Col: 20, Idx: 6400): Mapping values are not allowed in this context..
[Error] Unable to load file 'TOC.yml' via processor 'TocDocumentProcessor': W:\kyir-s\windows-iotcore\TOC.yml is not a valid TOC File: W:\kyir-s\windows-iotcore\TOC.yml is not a valid TOC file, detail: (Line: 193, Col: 20, Idx: 6400) - (Line: 193, Col: 20, Idx: 6400): Mapping values are not allowed in this context..
[Error] W:\kyir-s\windows-iotcore\TOC.yml is not a valid TOC File: W:\kyir-s\windows-iotcore\TOC.yml is not a valid TOC file, detail: (Line: 193, Col: 20, Idx: 6400) - (Line: 193, Col: 20, Idx: 6400): Mapping values are not allowed in this context..
[Error] Unable to load file 'TOC.yml' via processor 'ExperimentalTocDocumentProcessor': W:\kyir-s\windows-iotcore\TOC.yml is not a valid TOC File: W:\kyir-s\windows-iotcore\TOC.yml is not a valid TOC file, detail: (Line: 193, Col: 20, Idx: 6400) - (Line: 193, Col: 20, Idx: 6400): Mapping values are not allowed in this context..

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