Discussion of format of headers

What should we have in headers? Should they be somehow uniform?

Since there are headers everywhwe I though we have to have headers :).  I don't know.
Ondra version (e.g. in OrdVector I think), with something as Mata team instead of a name of the guy who created the file?

Or we want individual names there?

Short licence with Mata team seems fine to me. As I said, individual names can be inferred from the git blame

As @tfiedor says. Definitely the name of the team rather than an individual name. The individual name does not say anything useful. It is just the name of the first contributor to the file at hand.

Maybe, you would like the following: Lukas Holik's Righteous Army of Vengeance.

According to our latest agreement, we should make all headers in Mata library as well as in all bindings uniform. Furthermore, I think we have all agreed on using the name of the team instead of individual contributor names.

Can we agree on the following header? I.e., simply removing Ondra from the header?
/* nfa.hh -- nondeterministic finite automaton (over finite words)
 *
 * Copyright (c) 2018-2023
 *
 * This file is a part of libmata.
 *
 * This program is free software; you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation; either version 3 of the License, or
 * (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 */

This format looks good to me.


Update build and release instructions

Fixes #71

Thanks, @mernst, I am going to merge for now and open a follow-up task for addressing the other feedback


Bundle checking in package file filter should throw error instead of logging warn

currently file filter only reports about unnecessarily put JAR file under /apps/...install/ path, but better if it will be reporting error . verbosity of this feature should be changeable to reflect previous behavior

configurable via enum or property package.fileFilter.bundleChecking=none|warn|fail

to be released in 11.0.0


V1.2b import

Closes #6
Note: I'm changing the import format to import -f <file_path> [-g <number_of_group_columns>] [-a <number_of_assessment_columns>] [-t <number_of_tag_columns>] to support tags

Codecov Report

Merging #113 (ad80cab) into master (b5ff121) will increase coverage by 1.01%.
The diff coverage is 90.21%.

@@             Coverage Diff              @@
##             master     #113      +/-   ##
============================================
+ Coverage     73.65%   74.67%   +1.01%
- Complexity      499      531      +32
============================================
  Files            79       81       +2
  Lines          1507     1595      +88
  Branches        181      191      +10
============================================
+ Hits           1110     1191      +81
- Misses          333      337       +4
- Partials         64       67       +3

Impacted Files
Coverage Δ

.../seedu/address/logic/parser/AddressBookParser.java
80.00% <0.00%> (-4.62%)
:arrow_down:

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)

...va/seedu/address/logic/commands/ImportCommand.java
97.10% <97.10%> (ø)

.../java/seedu/address/logic/commands/AddCommand.java
100.00% <100.00%> (ø)

...ain/java/seedu/address/logic/parser/CliSyntax.java
87.50% <100.00%> (+1.78%)
:arrow_up:

...eedu/address/logic/parser/ImportCommandParser.java
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Conflicting behaviour with output region

Hi!
There seems to be difference in how the output_region is computed in dt.Optics and in dt.SampleToMasks.
dt.Optics seems to be taking output_region as [x, y, x + width, y + height] (last two values in the array are also being considered as coordinates), whereas dt.SampleToMasks takes it as [x, y, width, height].
Any one of this would work fine (probably the second style is better), but this needs to made consistent across both.

Thanks! We will correct this such that both are of the style [x, y, x + width, y + height] to keep it backward compatible.

Closed by #127


No configuration options and errors in logs

Description
No Settings page to configure and receiving error in logs
Environment
HACC 0.6.0
Odroid XU4
Ubuntu 18.04.5 LTS armv71
Steps to Reproduce
Install HACC using the "Easy" method on your site

Looking at other screenshots online, this Settings page should show the long-lived tokens section whereas it's not showing for me.

Any help would be appreciated

Using the "beta" option looks to have worked

Same problem here. Stable is not as stable as name says. It doesn't work under latest HA. Beta is working.


Fix broken links

I noticed that these links are pointing to the invalid URL. This commit should fix it.

Those links are working fine now and this pull request can be closed.

Thanks y’all!


Print start and end times to logs

Or maybe get a calculation of CPU hours per second of operation directly. Note whether it's running on 2 cores.

Elapsed time is actually already printed.


jaeger ingester panic with deadlockInterval > 0

I observe the ingested keeps restarting due to a panic when I set deadlockInterval: 5s (as shown in the doc).
"level":"panic","ts":1586465229.2084777,"caller":"consumer/deadlock_detector.go:71","msg":"No messages processed in the last check interval","partition":-1,"stack":"goroutine 56 [running]:\ngithub.com/jaegertracing/jaeger/cmd/ingester/app/consumer.newDeadlockDetector.func1(0xc0ffffffff)\n\tgithub.com/jaegertracing/jaeger/cmd/ingester/app/consumer/deadlock_detector.go:73 +0x24d\ngithub.com/jaegertracing/jaeger/cmd/ingester/app/consumer.(*deadlockDetector).start.func1(0xc000328090, 0xc0002901a0)\n\tgithub.com/jaegertracing/jaeger/cmd/ingester/app/consumer/deadlock_detector.go:163 +0x148\ncreated by github.com/jaegertracing/jaeger/cmd/ingester/app/consumer.(*deadlockDetector).start\n\tgithub.com/jaegertracing/jaeger/cmd/ingester/app/consumer/deadlock_detector.go:152 +0x185\n\ngoroutine 1 [select]:\ngithub.com/jaegertracing/jaeger/cmd/flags.(*Service).RunAndThen(0xc0001b0200, 0xc0001298a8)\n\tgithub.com/jaegertracing/jaeger/cmd/flags/service.go:137 +0xf3\nmain.main.func1(0xc0001f6900, 0xc0001b1a40, 0x0, 0x4, 0x0, 0x0)\n\tcommand-line-arguments/main.go:78 +0x84f\ngithub.com/jaegertracing/jaeger/vendor/github.com/spf13/cobra.(*Command).execute(0xc0001f6900, 0xc0000bc010, 0x4, 0x4, 0xc0001f6900, 0xc0000bc010)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/spf13/cobra/command.go:698 +0x42c\ngithub.com/jaegertracing/jaeger/vendor/github.com/spf13/cobra.(*Command).ExecuteC(0xc0001f6900, 0xc0001f6900, 0xc000129ee8, 0x3)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/spf13/cobra/command.go:783 +0x2c9\ngithub.com/jaegertracing/jaeger/vendor/github.com/spf13/cobra.(*Command).Execute(...)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/spf13/cobra/command.go:736\nmain.main()\n\tcommand-line-arguments/main.go:105 +0x568\n\ngoroutine 19 [syscall]:\nos/signal.signal_recv(0x0)\n\truntime/sigqueue.go:147 +0x9c\nos/signal.loop()\n\tos/signal/signal_unix.go:23 +0x22\ncreated by os/signal.init.0\n\tos/signal/signal_unix.go:29 +0x41\n\ngoroutine 37 [IO wait]:\ninternal/poll.runtime_pollWait(0x7f4769d2b028, 0x72, 0x0)\n\truntime/netpoll.go:184 +0x55\ninternal/poll.(*pollDesc).wait(0xc0001bc498, 0x72, 0x0, 0x0, 0x12642d0)\n\tinternal/poll/fd_poll_runtime.go:87 +0x45\ninternal/poll.(*pollDesc).waitRead(...)\n\tinternal/poll/fd_poll_runtime.go:92\ninternal/poll.(*FD).Accept(0xc0001bc480, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0)\n\tinternal/poll/fd_unix.go:384 +0x1f8\nnet.(*netFD).accept(0xc0001bc480, 0xc000092cc8, 0xc000059180, 0x7f476bf93008)\n\tnet/fd_unix.go:238 +0x42\nnet.(*TCPListener).accept(0xc000290020, 0xc000092cf8, 0x40ca28, 0x30)\n\tnet/tcpsock_posix.go:139 +0x32\nnet.(*TCPListener).Accept(0xc000290020, 0x117e360, 0xc0001a6ba0, 0x10a1420, 0x1d57c20)\n\tnet/tcpsock.go:261 +0x47\nnet/http.(*Server).Serve(0xc0002a2000, 0x1427980, 0xc000290020, 0x0, 0x0)\n\tnet/http/server.go:2896 +0x280\ngithub.com/jaegertracing/jaeger/cmd/flags.(*AdminServer).serveWithListener.func1(0xc0001ad260, 0x1427980, 0xc000290020)\n\tgithub.com/jaegertracing/jaeger/cmd/flags/admin.go:123 +0x61\ncreated by github.com/jaegertracing/jaeger/cmd/flags.(*AdminServer).serveWithListener\n\tgithub.com/jaegertracing/jaeger/cmd/flags/admin.go:122 +0x3ff\n\ngoroutine 44 [select]:\ngithub.com/jaegertracing/jaeger/vendor/github.com/olivere/elastic.(*Client).healthchecker(0xc0001f4b60)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/olivere/elastic/client.go:1054 +0x179\ncreated by github.com/jaegertracing/jaeger/vendor/github.com/olivere/elastic.DialContext\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/olivere/elastic/client.go:404 +0x5ae\n\ngoroutine 7 [IO wait]:\ninternal/poll.runtime_pollWait(0x7f4769d2ae88, 0x72, 0xffffffffffffffff)\n\truntime/netpoll.go:184 +0x55\ninternal/poll.(*pollDesc).wait(0xc000444198, 0x72, 0x1000, 0x1000, 0xffffffffffffffff)\n\tinternal/poll/fd_poll_runtime.go:87 +0x45\ninternal/poll.(*pollDesc).waitRead(...)\n\tinternal/poll/fd_poll_runtime.go:92\ninternal/poll.(*FD).Read(0xc000444180, 0xc00044e000, 0x1000, 0x1000, 0x0, 0x0, 0x0)\n\tinternal/poll/fd_unix.go:169 +0x1cf\nnet.(*netFD).Read(0xc000444180, 0xc00044e000, 0x1000, 0x1000, 0x43133c, 0xc000093b20, 0x45bbd0)\n\tnet/fd_unix.go:202 +0x4f\nnet.(*conn).Read(0xc00000e0b8, 0xc00044e000, 0x1000, 0x1000, 0x0, 0x0, 0x0)\n\tnet/net.go:184 +0x68\nnet/http.(*persistConn).Read(0xc0001c5320, 0xc00044e000, 0x1000, 0x1000, 0xc00024a180, 0xc000093c20, 0x404d15)\n\tnet/http/transport.go:1758 +0x75\nbufio.(*Reader).fill(0xc000076960)\n\tbufio/bufio.go:100 +0x103\nbufio.(*Reader).Peek(0xc000076960, 0x1, 0x0, 0x0, 0x1, 0xc00045c700, 0x0)\n\tbufio/bufio.go:138 +0x4f\nnet/http.(*persistConn).readLoop(0xc0001c5320)\n\tnet/http/transport.go:1911 +0x1d6\ncreated by net/http.(*Transport).dialConn\n\tnet/http/transport.go:1580 +0xb0d\n\ngoroutine 66 [select]:\ngithub.com/jaegertracing/jaeger/vendor/github.com/olivere/elastic.(*bulkWorker).work(0xc0001b1f80, 0x142d6c0, 0xc0000b2010)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/olivere/elastic/bulk_processor.go:480 +0x125\ncreated by github.com/jaegertracing/jaeger/vendor/github.com/olivere/elastic.(*BulkProcessor).Start\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/olivere/elastic/bulk_processor.go:336 +0x1e4\n\ngoroutine 57 [chan receive]:\ngithub.com/jaegertracing/jaeger/cmd/ingester/app/consumer.(*Consumer).Start.func1(0xc000328000)\n\tgithub.com/jaegertracing/jaeger/cmd/ingester/app/consumer/consumer.go:79 +0x187\ncreated by github.com/jaegertracing/jaeger/cmd/ingester/app/consumer.(*Consumer).Start\n\tgithub.com/jaegertracing/jaeger/cmd/ingester/app/consumer/consumer.go:77 +0x56\n\ngoroutine 8 [select]:\nnet/http.(*persistConn).writeLoop(0xc0001c5320)\n\tnet/http/transport.go:2210 +0x123\ncreated by net/http.(*Transport).dialConn\n\tnet/http/transport.go:1581 +0xb32\n\ngoroutine 67 [select]:\ngithub.com/jaegertracing/jaeger/vendor/github.com/olivere/elastic.(*BulkProcessor).flusher(0xc0001bfba0, 0xbebc200)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/olivere/elastic/bulk_processor.go:429 +0x10d\ncreated by github.com/jaegertracing/jaeger/vendor/github.com/olivere/elastic.(*BulkProcessor).Start\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/olivere/elastic/bulk_processor.go:342 +0x445\n\ngoroutine 28 [select]:\ngithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama.(*client).backgroundMetadataUpdater(0xc000452120)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama/client.go:713 +0x15d\ngithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama.withRecover(0xc0002f6230)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama/utils.go:45 +0x4d\ncreated by github.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama.NewClient\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama/client.go:172 +0x546\n\ngoroutine 26 [chan receive]:\ngithub.com/jaegertracing/jaeger/vendor/github.com/rcrowley/go-metrics.(*meterArbiter).tick(0x1d71260)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/rcrowley/go-metrics/meter.go:239 +0x31\ncreated by github.com/jaegertracing/jaeger/vendor/github.com/rcrowley/go-metrics.NewMeter\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/rcrowley/go-metrics/meter.go:46 +0x111\n\ngoroutine 27 [chan receive]:\ngithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama.(*Broker).responseReceiver(0xc00009c000)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama/broker.go:790 +0xc7\ngithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama.withRecover(0xc0002f61f0)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama/utils.go:45 +0x4d\ncreated by github.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama.(*Broker).Open.func1\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama/broker.go:215 +0x99e\n\ngoroutine 54 [select]:\ngithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama.(*Broker).sendAndReceive(0xc00015e000, 0x1430580, 0xc000076360, 0x140cf80, 0xc0000bc5a0, 0x0, 0xc00036a200)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama/broker.go:717 +0x145\ngithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama.(*Broker).JoinGroup(0xc00015e000, 0xc000076360, 0xf, 0xc00015e000, 0x0)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama/broker.go:396 +0x70\ngithub.com/jaegertracing/jaeger/vendor/github.com/bsm/sarama-cluster.(*Consumer).joinGroup(0xc00045e0f0, 0xc000290220, 0x3, 0xc000290220)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/bsm/sarama-cluster/consumer.go:669 +0x308\ngithub.com/jaegertracing/jaeger/vendor/github.com/bsm/sarama-cluster.(*Consumer).rebalance(0xc00045e0f0, 0x0, 0x0, 0xc0000bbc40)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/bsm/sarama-cluster/consumer.go:577 +0x179\ngithub.com/jaegertracing/jaeger/vendor/github.com/bsm/sarama-cluster.(*Consumer).nextTick(0xc00045e0f0)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/bsm/sarama-cluster/consumer.go:363 +0xa1\ngithub.com/jaegertracing/jaeger/vendor/github.com/bsm/sarama-cluster.(*Consumer).mainLoop(0xc00045e0f0)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/bsm/sarama-cluster/consumer.go:334 +0xc8\ncreated by github.com/jaegertracing/jaeger/vendor/github.com/bsm/sarama-cluster.NewConsumerFromClient\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/bsm/sarama-cluster/consumer.go:96 +0x3da\n\ngoroutine 29 [IO wait]:\ninternal/poll.runtime_pollWait(0x7f4769d2af58, 0x72, 0xffffffffffffffff)\n\truntime/netpoll.go:184 +0x55\ninternal/poll.(*pollDesc).wait(0xc000116418, 0x72, 0x1000, 0x1000, 0xffffffffffffffff)\n\tinternal/poll/fd_poll_runtime.go:87 +0x45\ninternal/poll.(*pollDesc).waitRead(...)\n\tinternal/poll/fd_poll_runtime.go:92\ninternal/poll.(*FD).Read(0xc000116400, 0xc000486000, 0x1000, 0x1000, 0x0, 0x0, 0x0)\n\tinternal/poll/fd_unix.go:169 +0x1cf\nnet.(*netFD).Read(0xc000116400, 0xc000486000, 0x1000, 0x1000, 0xc000116400, 0xc000305d00, 0xc000305cf0)\n\tnet/fd_unix.go:202 +0x4f\nnet.(*conn).Read(0xc0000b4058, 0xc000486000, 0x1000, 0x1000, 0x0, 0x0, 0x0)\n\tnet/net.go:184 +0x68\nbufio.(*Reader).Read(0xc000351ce0, 0xc0003ba098, 0x8, 0x8, 0x40904b, 0xc00001e000, 0x10c94a0)\n\tbufio/bufio.go:226 +0x26a\ngithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama.(*bufConn).Read(0xc0001b4560, 0xc0003ba098, 0x8, 0x8, 0xc000305df8, 0x40a195, 0x10c94a0)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama/utils.go:109 +0x4d\nio.ReadAtLeast(0x7f4769d2f488, 0xc0001b4560, 0xc0003ba098, 0x8, 0x8, 0x8, 0x1d71860, 0xc000305ec0, 0x40523a)\n\tio/io.go:310 +0x87\nio.ReadFull(...)\n\tio/io.go:329\ngithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama.(*Broker).responseReceiver(0xc00015e000)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama/broker.go:803 +0x290\ngithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama.withRecover(0xc0002f63d0)\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama/utils.go:45 +0x4d\ncreated by github.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama.(*Broker).Open.func1\n\tgithub.com/jaegertracing/jaeger/vendor/github.com/Shopify/sarama/broker.go:215 +0x99e\n","stacktrace":"github.com/jaegertracing/jaeger/cmd/ingester/app/consumer.newDeadlockDetector.func1\n\tgithub.com/jaegertracing/jaeger/cmd/ingester/app/consumer/deadlock_detector.go:71\ngithub.com/jaegertracing/jaeger/cmd/ingester/app/consumer.(*deadlockDetector).start.func1\n\tgithub.com/jaegertracing/jaeger/cmd/ingester/app/consumer/deadlock_detector.go:163"}
panic: No messages processed in the last check interval

goroutine 56 [running]:
github.com/jaegertracing/jaeger/vendor/go.uber.org/zap/zapcore.(*CheckedEntry).Write(0xc0002342c0, 0xc0001bc080, 0x2, 0x2)
	github.com/jaegertracing/jaeger/vendor/go.uber.org/zap/zapcore/entry.go:230 +0x546
github.com/jaegertracing/jaeger/vendor/go.uber.org/zap.(*Logger).Panic(0xc0001aa660, 0x1294f0f, 0x30, 0xc0001bc080, 0x2, 0x2)
	github.com/jaegertracing/jaeger/vendor/go.uber.org/zap/logger.go:225 +0x7f
github.com/jaegertracing/jaeger/cmd/ingester/app/consumer.newDeadlockDetector.func1(0xc0ffffffff)
	github.com/jaegertracing/jaeger/cmd/ingester/app/consumer/deadlock_detector.go:71 +0x40d
github.com/jaegertracing/jaeger/cmd/ingester/app/consumer.(*deadlockDetector).start.func1(0xc000328090, 0xc0002901a0)
	github.com/jaegertracing/jaeger/cmd/ingester/app/consumer/deadlock_detector.go:163 +0x148
created by github.com/jaegertracing/jaeger/cmd/ingester/app/consumer.(*deadlockDetector).start
	github.com/jaegertracing/jaeger/cmd/ingester/app/consumer/deadlock_detector.go:152 +0x185

However, if I disabled the deadlockInterval or set to 0s, the problem disappears. Does anyone else have the same issue or is this a bug? Thanks.

This is expected behaviour - if no messages are received within the defined time period the ingester is terminated in case it is due to a deadlock, enabling the environment (e.g. kubernetes) to restart it. As described in the CLI flag description: "Interval to check for deadlocks. If no messages gets processed in given time, ingester app will exit. Value of 0 disables deadlock check."

@objectiser , thanks for your answer, so I will ignore the restart in my deployment.
However, I think panic is different than exit, say with code 1. This seems a bug to me.

@yurishkuro Any idea why it used panic vs just warning with an exit?

 I don't think any more thoughts went into it than that.

 I don't think any more thoughts went into it than that.

Thanks.


array_sort unit test fix

Changelog category (leave one):

Bug Fix (user-visible misbehavior in official stable or prestable release)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
...

Array_sort unit test fix
dynamic unit test fix

Information about CI checks: https://clickhouse.com/docs/en/development/continuous-integration/

Merging as @kashwy advised.


[feature] saving conversations

Do you plan to add option to save conversations?

Looks like your wish will come true soon check out this pull request will be available soon:
https://github.com/UnknownEnergy/chatgpt-api/pull/11

@ipodobin the feature is now in production will close the request

Great! Thank you very much.
On Mon, 5 Jun 2023, 18:57 Daniel Schweiger, @.***>
wrote:

Closed #9 https://github.com/UnknownEnergy/chatgpt-api/issues/9 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/UnknownEnergy/chatgpt-api/issues/9#event-9436075189,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB3K6JCLOMUYC7UZOSCOLGLXJYFWNANCNFSM6AAAAAAW2QELJA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Members refinements

no refs.

updated loading placeholder for member plan details

Coverage remained the same at 70.255% when pulling 047cc161e6822dfe6e4c64c87b9a4c638715a844 on peterzimon:members-refinements into 14060e267cb437d9fefeb5fbb37e167e53e0071a on TryGhost:master.


[WIP] Switch orderbook to new metrics system

Note: some metric names have changed to follow prometheus naming conventions. Because of this, I want to deploy it to staging and create new grafana dashboards before releasing to prod.
Test Plan
CI + manual testing

Will split this into smaller PRs.


test: unittest pnpm extension version resolution

This is in preparation for #1907: As dependency resolution gets more complex, we should test it.
Note: No updates to to bzl_library targets, because it seems extensions.bzl isn't in a bzl_library.

Changes are visible to end-users: no
Test plan

Covered by existing test cases (e2e extension execution)
New test cases added

@gzm0 I think this was all correctly merged in the other PR?

Yep, looks like it (seems it got squashed, but the changes are there).


Question about hide_key argument

I was looking in the code on Git Hub and saw that, maybe, we could hide the key when writing to a file. The arg is in the Save function, but an error is logged when I try to invoke it.
TypeError: WordSearch.save() got an unexpected keyword argument 'hide_key'
I could have PIP installed an older version or the GitHub version is not in the pip module yet. Can you confirm that this should work?
I am newer to Python so I am a little shaky with the syntax and how modules are built.

 The latest version (v4.0.0) hasn't been released on Pypi yet. To access the new features you can pip install the most recent code directly from GitHub using the command below.
pip install git+https://github.com/joshbduncan/word-search-generator

Let me know if you have any other questions. I'm here to help!


feat(es-dev-server): improve node-resolve performance

The current implementation of node-resolve uses babel. This is wasteful, as we're creating a full AST of each js file just for resolving the imports.
In this implementation we use the lexer from es-module-shims. This just extracts information about where imports and dynamic imports are, so that we can resolve them and do a simple string replacement.
There is discussion on separating the lexer https://github.com/guybedford/es-module-shims/issues/35 but that may take a while, so I included it here until we resolve that.
Depending on the project, this can significantly speed up performance. Running the dev server and importing all of lodash-es takes 5200ms to load on master, and it takes 1800ms in this branch. Running large bundled libraries like sinon, chai, axe etc. through babel take much longer so that's a huge boost there.
On my machine, running the open-wc tests on master takes 5560ms while on this branch it takes 3670ms. This is without coverage turned on, because that still boots up babel. I also updated karma-esm to only run babel on files that need coverage. That brings it down to 4400ms with coverage enabled.

Would be good if we can test this one some different projects before merging

We can remove our local copy when https://github.com/guybedford/es-module-lexer/pull/1 is merged


Raise exception when getting timeout for running cmd to detect OS

For some images, like sciencelogicinc1622565452194, middleware publisher's images, running some basic commands will be time out. This is due to the image not allow to run commands or in bad state.
In _get_detect_string, there are more than 10 commands to detect the OS. If every command gets time out (10 min), the case might last longer than 1 hour.
The case should be go to the end immediately if the system can't run any commands.
In this PR, add timeout=30 for every command in _get_detect_string, and raise an exception when gets timeout to avoid more timeout commands.

@LiliDeng LGTM


Add enum validation to .spec.* fields in config/v1 node CRD

Added validation to workerLatencyProfile, cgroupMode fields in config.openshift.io/v1/node CRD.
Signed-off-by: Swarup Ghosh<EMAIL_ADDRESS>

/label px-approved
/label qa-approved
/label docs-approved


Excel report generation

Description
Implemented excel report generation for data comparison between source and target with primary keys

@coderabbitai review


Cannot type "~"

Lapce Version
0.2.5
System information
Ubuntu 20.04
Describe the bug
I cannot type ~. I usually type ~ followed by a space to make the ~ character appear on the screen when using my keyboard.
Additional information
I am using Ubuntu inside a remote desktop application. Other editors and software work as usual. Cannot reproduce on Windows.

Related to #1798 ?

Yes, I think so. I am using a Norwegian keyboard.
I will close this issue.

I think you should keep this open. It's a legitimate issue for a text editor to not work with some locales?

I think you should keep this open. It's a legitimate issue for a text editor to not work with some locales? BTW do you know of the specific unicode value that doesn't work here?

We already have issue opened for dead keys

BTW do you know of the specific unicode value that doesn't work here?

nb_NO

nb_NO

That's keyboard layout.

That's keyboard layout.

Go to keyboard settings in Lapce and try to map the key that doesn't work, if it shows as DEAD it's a dead keys issue #1094

It is a dead key indeed.


Added support for multiple model types to shared.js

Allows shared.js to store cache based on type of data, so it can store multiple objects with the same $pk, as long as they are of different types.  Most of this code came from @jasonayre in issue #208.
Lines  39-41 were added by me to fix a bug where model scopes would get nested after being cached.  This way the scope is refreshed every time it is read from the cache.

@ldeboer This PR is quite old and out of date. Do you still have any interest in getting it included?


Use d2l-course-management-behavior in d2l-my-courses

Not really a big/exciting change, but the same function was defined in two places. Might as well use the already-existing one!
Found this while doing #370, but twas unrelated so gets its own PR. Tested it and all seems to work - pinning/unpinning courses moves the tiles as expected. If this didn't work, that wouldn't work, so seems like adequate testing to me 😛

Good point! I quickly looked at a couple, but I'll look deeper and see if there are more duplicates. I think a lot of this stuff got pulled out into behaviors weeks or months after it was first written, so very likely some things got missed!

Fun fact, turns out the function I had initially de-duplicated is like... the only one that actually makes sense to have in the behavior. Everything else is either only used in one of d2l-my-courses/d2l-all-courses, or it's defined in the behavior and overridden in one of both of the above, so I moved things around so that things are exactly as specific as they need to be!


License

Could you please include a license file for the project, so people can fork it without violating your copyright?

I am very sorry that my response has taken two years.
I added the MIT license file.


P2P CR - 1 april 2022

good job on your code retky 👍  it works fina and everything 💯
only suggestion I would make to you is to take a look at jsdom for DOM mocking, instead of this:
https://github.com/Retky/Mv-ToDoList/blob/de8bd03b06233f189dabecd0da82f14919349f69/src/modules/__test__/editDesc.test.js#L7-L20

https://media0.giphy.com/media/XbTygeYSO8bQ5FB02p/200.gif?cid=5a38a5a2ac97g2lx7pbioos9zibwhb9b9sufmgsv1sm38nm4&rid=200.gif&ct=g

Nice code and well functioning :+1:

finally found out how to put gifs in the comments 😆


openlava-web

Web interface for OpenLava:
https://www.clusterfsck.io/static/openlava-web/index.html

I think the best place for this will be as a custom cookbook, which can be called as part of the Chef version of CfnCluster.


clean-up humidity parsing

Humidity can be None if not supported, issue #147 erroneously added numeric validation

this issue was closed already but checked in as issue #148 by mistake


MTLS auth

tracking issue longhorn/longhorn#3839
Currently running core tests, my confidence in the manager <-> im communication isn't as high as it should be.
I would prefer if we clean up the monitoring + im communication in general before we add additional complexity to it.
The previous code/architecture wasn't built with peristent connections in mind and instead is completly built as one shot commands, which is highly inefficient.
Since for each grpc call:

a single connection is setup (tcp handshake + co)
then grpc call is triggered
connection teardown

Some of the code smells surrounding the manager <-> im communication:

im monitor
instance-handler usage, same one shot idea.
im notifier / updater (I removed these wrappers and replaced them with a direct call to the im client)

Generally In the future I would like our code base to become more testable, for example we can mock on the im monitor/client level then the full im controller should be testable same with engine/replica controller.
I left most of these refactorings for longhorn/longhorn#2441

@joshimoo Please help rebase from the master again because the dep issue has been resolved.

Can be reviewed, I will need to update the im ref once https://github.com/longhorn/longhorn-instance-manager/pull/109
is merged, currently 2 bottom commits are dev commits that I will strip before merging.

Agree on the requirement for documentation, I will write some up.

@innobead updated the PR, stripped the dev commits and updated to the new im image + dependency


 I hope to refactor them later in October.
Fix bug in find_extrema_with_reduce(). Variable representing index had wrong type.

Coverage increased (+3.07%) to 83.191% when pulling 06124180dc052afb45d4c5a0d8bab8b2f6c8d3d1 on haahh:pr_find_extrema_cpu into d303097000552ae4e965b380d1c2954385e18205 on boostorg:develop.

Looks great! Merged!


Add 'tag' and 'ref' to GitbuilderProject's init, warn if more than one set, and tests

rebased to master again


Having trouble generating typescript from RESTful Url's

My service is RESTFul so my base url looks like '/api/profile'. A GET to that request should fetch all profiles and a POST should create a new one. Using the Angular Typescript template, two methods are created with the same name which is invalid.
The service should look at the HTTP Verb and prepend the method name with a helper like getProfiles and saveProfile.

Can you post a sample swagger spec?

Here are the relevant bits. If it could take the operationId and remove the tag and underscore and use the remainder of that camel cased, that would be exactly what i need. In this case the two methods generated would be getProfileForEdit and saveProfile.
{
   "swagger":"2.0",
   "info":{
      "version":"v1",
      "title":"Test API"
   },
   "host":"localhost:1994",
   "schemes":[
      "http"
   ],
   "paths":{
      "/api/Profile":{
         "get":{
            "tags":[
               "Profile"
            ],
            "operationId":"Profile_GetProfileForEdit",
            "consumes":[

            ],
            "produces":[
               "application/json",
               "text/json",
               "application/xml",
               "text/xml"
            ],
            "responses":{
               "200":{
                  "description":"OK",
                  "schema":{
                     "$ref":"#/definitions/EditProfileModel"
                  }
               }
            },
            "deprecated":false
         },
         "post":{
            "tags":[
               "Profile"
            ],
            "operationId":"Profile_SaveProfile",
            "consumes":[
               "application/json",
               "text/json",
               "application/xml",
               "text/xml",
               "application/x-www-form-urlencoded"
            ],
            "produces":[
               "application/json",
               "text/json",
               "application/xml",
               "text/xml"
            ],
            "parameters":[
               {
                  "name":"model",
                  "in":"body",
                  "required":true,
                  "schema":{
                     "$ref":"#/definitions/EditProfileModel"
                  }
               }
            ],
            "responses":{
               "200":{
                  "description":"OK",
                  "schema":{
                     "$ref":"#/definitions/Object"
                  }
               }
            },
            "deprecated":false
         }
      }
   },
   "definitions":{

      "EditProfileModel":{
         "type":"object",
         "properties":{
            "email":{
               "type":"string"
            },
            "firstName":{
               "type":"string"
            },
            "lastName":{
               "type":"string"
            },
            "phoneNumber":{
               "type":"string"
            }
         }
      }
   },
   "securityDefinitions":{
      "oauth2":{
         "type":"oauth2",
         "description":"OAuth2 Implicit Grant",
         "flow":"implicit",
         "tokenUrl":"/api/token",
         "scopes":{
         }
      }
   }
}

In ClientGeneratorBase, i think the following change would fix this
OperationNameLower = ConvertToLowerStartIdentifier(BaseSettings.OperationGenerationMode == OperationGenerationMode.MultipleClientsFromPathSegments ? operation.OperationId.Replace(operation.Tags[0],"").Remove(0,1) : operation.OperationId),
OperationNameUpper = ConvertToUpperStartIdentifier(BaseSettings.OperationGenerationMode == OperationGenerationMode.MultipleClientsFromPathSegments ? operation.OperationId.Replace(operation.Tags[0], "").Remove(0, 1) : operation.OperationId),

Web API generator generates operation ID in the form {controller}_{action}. The operation generation mode MultipleClientsFromOperationId (default) converts this ID to multiple clients with the correct action names:
https://github.com/NSwag/NSwag/commit/8feeee1051613a4578667853911e72e7c984871c

Thanks! When will you be able to roll out a new Nuget package?
On Mon, Mar 21, 2016 at 2:44 AM, Rico Suter<EMAIL_ADDRESS>wrote:

Web API generator generates operation ID in the form {controller}_{action}.
The operation generation mode MultipleClientsFromOperationId (default)
converts this ID to multiple clients with the correct action names:
8feeee1
https://github.com/NSwag/NSwag/commit/8feeee1051613a4578667853911e72e7c984871c
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/NSwag/NSwag/issues/49#issuecomment-199194505

Its released

v1.44

Works great! Thanks!
On Mon, Mar 21, 2016 at 1:25 PM, Rico Suter<EMAIL_ADDRESS>wrote:

v1.44
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/NSwag/NSwag/issues/49#issuecomment-199462282


Why HybridAuth.SocialProfiles hasMany instead of belongsTo or hasOne ..?

Hi,
Because in the example you make a hasMany link instead of belongsTo or hasOne ..?
    // Plugin Social HybridAuth.SocialProfiles

//        $this->hasMany('ADmad/HybridAuth.SocialProfiles');
    $this->belongsTo('ADmad/HybridAuth.SocialProfiles', [
        'foreignKey' => 'social_profiles_id',
        'joinType' => 'LEFT'
    ]);

OR
$this->hasOne('ADmad/HybridAuth.SocialProfiles');

Each user can have multiple social profiles. For e.g. you can have Google and Facebook account and can use either to login to the site but both profiles should be associated with same user account.

Ahhh ok
But in my case the user only has a social profile, User 1 <=> 1 SocialProfiles.
No problem using $this->hasOne('ADmad / HybridAuth.SocialProfiles');
Because from what I saw, it works for me. And if it works well this is good in the documentation also to facilitate the task to other users.


Update Link Container

Title
Update Link Container
Description
Added event listeners to inputs in the link class, added a prototype isValidURL function that uses a regex to check
if a string is a valid URL. Added function setIcon that fetches the corresponding icon or the default icon.
Added default.png for the icon class if there is no link in socialLinks or if ${key.png} exists.
Implemented previewLink(button), matching inputValue with a key in socialLinks if possible and handling the username.

Issue Ticket Number
Fixes #42

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] I have followed the contributing guidelines of this project as mentioned in CONTRIBUTING
[x] I have checked to ensure there aren't other open Pull Requests for the same update/change?
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes needed to the documentation

Hey @kimchiiboiii ,
Thanks for your contribution, the code is working but there is still some bug :sweat_smile: , but no worries, I will handle it.
As you've done so much effort, really hats off :saluting_face:
I am currently refactoring the code, once it's done, I will merge your PR.


Create Proxy AI server access guide and link to main page

Create a guide for accessing the Proxy Azure AI service that is set up for this Hack together event.
Includes how to steps on accessing personalised keys for the event, as well as details on the endpoint, and models available.
Provides examples and links to the docs for Python and .NET programs.

/lgtm


update clientgen to use helper script

This updates the clientgen script to use the upstream helper functions from code-generator. This approach is the updated guidance from the upstream repo.
Several APIs were missing the appropriate code generation comments, which are added.

/retest


Add customer flow for on boarding

to onboard users of the sdk we need them to 1. download 3bot connect and 2. download sdk
First step
we need to link all (get started buttons) to 3bot connect login/ sign-up

Second Step
once the login/sign up is done we lead them to wiki
for now leta link to wiki.threefold.io - will provide exact link once wiki is created later today.
This needs to be done before Friday 10th 12:00 CET for review

this is cancelled changed plans


[Rendering] Input.Time value is not received in the bot

In some occasions the time selected inside an Input.Time control is not sent back to the bot. This happens only on the Teams desktop app and I noticed this behavior when the column for seconds was introduced. In the Teams web app this column does not exist(yet) and the Input.Time works as it should.
Steps to reproduce:

Send the adaptive card with the json below
On the Teams Desktop app click on "Ok"
The payload receive will look like this { "startTime": "15:30" }
Change the date in the adaptive card
The payload will look like this {}

JSON:
{
    "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
    "type": "AdaptiveCard",
    "version": "1.2",
    "body": [
        {
            "type": "TextBlock",
            "text": "What time do you want to meet?"
        },
        {
            "type": "Input.Time",
            "id": "startTime",
            "min": "09:00",
            "max": "17:00",
            "value": "15:30"
        }
    ],
    "actions": [
        {
            "type": "Action.Submit",
            "title": "OK"
        }
    ]
}

Also, is there a way to hide the seconds?

@bogdanbujdea - We are looking into this issue.We will update you.

@bogdanbujdea - I have tried the adaptive card on  Microsoft Teams Version <IP_ADDRESS>066 on desktop.

Seconds are not showing.

It seems this is an issue only on my machine because I changed the short time format to be "hh:mm:ss" instead of the default "hh:mm tt". The desktop app picks this app and shows the seconds in the input time, while also having issues sending the time back to the card on certain occasions. Once I changed back to the default time format and restarted Teams, the Input.Time format got back to normal and now it works every time.
I should also point out that I changed my time format because on Windows 11 the clock doesn't have the seconds anymore so there may be other Windows 11 users trying this and then having issues with the Input.Time control.
Sorry for the false alarm, I think we can close this.

@bogdanbujdea -We are closing this issue now.


fix random_steering pool weights sent to the API on update from existing value

Hiya :wave:
This PR looks to fix a rather nasty bug of when using load balancer pool weights.
On creation of the weights, everything is as expected, however on subsequent updates/applies, even though the terraform plan and state looks correct, a default empty "random_steering": {} is sent to the API, removing any weights entirely.
This results in scenarios when changing pool weights for 2 pools from (0.98, 0.02) -> (0.95, 0.05) unexpectedly going to 50/50 on apply.
The d.GetOk("random_steering") that calls expandRandomSteering returns 1 entry (the desired values) in the set when no existing steering weights exist, and 2 when it does. The first is a default empty value in this scenario and causes the bug.This means that a second apply after the issue does "resolve" and sync to state until the next change.
This change to always get the last value in the set fixes the issue. I have added a covering update scenario test case that fails without this change.

The first is a default empty value in this scenario and causes the bug

Yikes! Thank you so much @jracollins for putting up this fix! 😄

thanks for the PR here @jracollins. i took your acceptance test and reverted the fix to ensure we are actually addressing the issue here.
TF_ACC=1 go test ./internal/sdkv2provider -v -run "^TestAccCloudflareLoadBalancer_RandomSteering" -count 1 -timeout 120m -parallel 1
=== RUN   TestAccCloudflareLoadBalancer_RandomSteering
=== PAUSE TestAccCloudflareLoadBalancer_RandomSteering
=== CONT  TestAccCloudflareLoadBalancer_RandomSteering
    resource_cloudflare_load_balancer_test.go:233: Step 2/2 error: After applying this test step and performing a `terraform refresh`, the plan was not empty.
        stdout

        Terraform used the selected providers to generate the following execution
        plan. Resource actions are indicated with the following symbols:
          ~ update in-place

        Terraform will perform the following actions:

          # cloudflare_load_balancer.imkmnzgyhq will be updated in-place
          ~ resource "cloudflare_load_balancer" "imkmnzgyhq" {
                id               = "5726fd6fbe4baa553425f0d75a86cb35"
                name             = "tf-testacc-lb-random-steering-imkmnzgyhq.terraform.cfapi.net"
                # (10 unchanged attributes hidden)

              + random_steering {
                  + default_weight = 0.8
                  + pool_weights   = {
                      + "85ee387e4a17bb17909f3636068f28ad" = 0.4
                    }
                }
              - random_steering {
                  - default_weight = 1 -> null
                  - pool_weights   = {} -> null
                }
            }

        Plan: 0 to add, 1 to change, 0 to destroy.


MudSelect not rendering properly when using ChildContent

Bug type
Component
Component name
MudSelect
What happened?
If I use the ChildComponent attribute and build a RenderFragment in code all of the Contents are ignored.
See here: https://try.mudblazor.com/snippet/cOmSOdcMIfiHDjVa
So no matter what's in builder.AddContent(0, "A thing") the item.ToString() seems to be called. You can see in the example the content of all of the select items should be "Go Fish" and the first item should be "None", but that is not the case.
Expected behavior
RenderFragment Contents should be what's used as the select item's text.
Reproduction link
https://try.mudblazor.com/snippet/cOmSOdcMIfiHDjVa
Reproduction steps

Create a MudSelect
Link the Child Contents to a method returning a RenderFragment
Have the RenderFragment return MudSelectItems with contents vis AddContents
...

Relevant log output
No response
Version (bug)
6.19.1
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@OFark your usage of the manual RenderTreeBuilder is incorrect. I would suggest to avoid that. See the fixed example for
https://try.mudblazor.com/snippet/ckmeYxwZHLyohZGJ
a) correct usage
b) much easier avoiding manual render tree creation
Please close this issue!
<MudSelect T="string" Label="Coffee" AnchorOrigin="Origin.BottomCenter" ChildContent="@GetContent()"/>

<MudSelect T="string" Label="Better" AnchorOrigin="Origin.BottomCenter">
    <MudSelectItem T="string" Value="@((string)null)">None</MudSelectItem>
    @foreach (var item in items) {
        <MudSelectItem T="string" Value="@(item)">Go Fish</MudSelectItem>
    }
</MudSelect>

@code {
    List<string> items = new() {"Cardinale", "Diavolo", "Margarita", "Spinaci"};

        public RenderFragment GetContent() => builder =>
        {
            builder.OpenComponent<MudSelectItem<string>>(0);
            builder.AddAttribute(1, "Value", (object)null);
            builder.AddAttribute(2, "ChildContent", new RenderFragment(b=>b.AddContent(3,"None")));

            builder.CloseComponent();

            foreach (var item in items)
            {
                builder.OpenComponent<MudSelectItem<string>>(4);
                builder.AddAttribute(5, "Value", (object)item);
                builder.AddAttribute(6, "ChildContent", new RenderFragment(b=>b.AddContent(7,"Go Fish")));
                builder.CloseComponent();
            }
        };
}

Hi.
@belucha answer is correct.
Just as side note that I would especially avoid:
private void GetContent(RenderTreeBuilder ...)
{
}

syntax, because if this https://github.com/dotnet/aspnetcore/issues/37907

I have so many MudSelect's and I'm trying to follow good coding practices by not repeating code. Why would the facility to do RenderFragments be there if it's not supposed to be used? If the functionality is there shouldn't it work? If my use is incorrect what is the correct way?

If my use is incorrect what is the correct way?

But @belucha posted the correct usage:
<MudSelect T="string" Label="Coffee" AnchorOrigin="Origin.BottomCenter" ChildContent="@GetContent()"/>

@code {
    List<string> items = new() {"Cardinale", "Diavolo", "Margarita", "Spinaci"};

    protected RenderFragment GetContent() => builder =>
    {
        builder.OpenComponent<MudSelectItem<string>>(0);
        builder.AddAttribute(21, "Value", (string?)null);
        builder.AddAttribute(6, "ChildContent", new RenderFragment(builder2 => builder2.AddContent(3, "None")));
        builder.CloseComponent();
        foreach (string str in items)
        {
            builder.OpenComponent<MudSelectItem<string>>(4);
            builder.AddAttribute(5, "Value", str);
            builder.AddAttribute(6, "ChildContent", new RenderFragment(builder2 => builder2.AddContent(7, "Go Fish")));
            builder.CloseComponent();
        }
    };
}

You problem is that you are using:
builder.AddContent(2, "None");

instead of
builder.AddAttribute(6, "ChildContent", new RenderFragment(builder2 => builder2.AddContent(3, "None")));

that's totally two different things, and that's how the razor would have generated  <MudSelectItem T="string" Value="@((string)null)">None</MudSelectItem> himself.

Why would the facility to do RenderFragments be there if it's not supposed to be used?

Nobody said that it's not suppose to be used, sure if you want to optimize code then use it, but the chance to make a user code mistake is high, and that's what happened here. The problem is not related to MudBlazor, but an invalid usage of RenderTreeBuilder API.

Ok, and that works, but AddContent is supposed to add text child content, is it not?

Looking at this again, in the fresh light of the day, it looks like OpenElement() AddContent() works, but OpenComponent() needs ChildContent setting.
As a general observation about Blazor, not MudBlazor, that isn't very well documented. My apologies for laying this at your doorstep, all the examples I was seeing were suggesting it should have worked the way I expected it to.

Ok, and that works, but AddContent is supposed to add text child content, is it not?

I think you can ask it in https://github.com/dotnet/aspnetcore or https://github.com/dotnet/razor.
My understanding is that AddContent works as child-content for the HTML elements (aka OpenElement).
For components we use AddAttribute(... "ChildContent", ...)
And lastly AddContent is used to display raw strings.


PAT Token with bypass policy

Hi @caarlos0,
Currently i had to deactivate the branch protection rules for the main branch, as I need to push the next version of the plugin to the main branch. I use the gh-action-bump-version plugin to update the package.json
Logs from the action:
✖  fatal     remote: error: GH006: Protected branch update failed for refs/heads/main.
remote: error:  Changes must be made through a pull request.
To https://github.com/goreleaser/goreleaser-azure-devops-extension.git
 ! [remote rejected] main -> main (protected branch hook declined)
error: failed to push some refs to 'https://github.com/***/***-azure-devops-extension.git'

git exited with code 1
✖  fatal     Failed to bump version

Question:
Can you create a new PAT with bypass policy and save it to the secret with the name: ACTION_GITHUB_TOKEN. I will then change the pipeline steps to use this PAT token.
Thanks!

@caarlos0 I tried but I could not create a token for this Repo. I saw only my own repos.


About view changes

Description

Make About view nicer
Put preferences location back in preferences pane (General)
Added an function to open the about view
Added AE Oniguruma package.

Related Issue

N/A

Checklist

[x] My changes generate no new warnings
[x] My code builds and runs on my machine
[x] I documented my code

Screenshots

Preferences Pane
About View

More grouping of preferences

I removed it out of preferences for a reason

I removed it out of preferences for a reason

I think this one line in settings looks way better then what is currently in the about window.

I removed it out of preferences for a reason

I think this one line in settings looks way better then what is currently in the about window.

Then you might just as well add location panel back because it will get removed once i redo settings

Also don't just group things together, i had it set like that because then the list doesn't look so big... because now with the grouping you made it feels overwhelming and not very clean

Also don't just group things together, i had it set like that because then the list doesn't look so big... because now with the grouping you made it feel overwhelming and not very clean

Build it, it looks way better then separate group boxes with one item in it.


How to label rotated bounding box?

Is there any convenient way to crop bounding box with a angle?
I tired polygon, but it is too complicated.
Or, is there any document on how to write plugin or extend CVAT object type?
thanks

There is no convenient way, yet. See https://github.com/openvinotoolkit/cvat/issues/1128

@bejoechen , it looks like we already have such issue. I will close this one.


Paper Prototypes

Paper Prototypes
You should have several ideas from your brainstorming session.

[ ] Group any similar ideas together.
[ ] Take one idea (or group of ideas) per person and spend 5 minutes building a paper prototype.

How to build a paper prototype
It's simple! All you need is a piece of paper and something to write with to draw out your solution.
No one expects it to be pretty, don't worry! Add as much detail as you can around how your solution might look or work.

Wireframe created
https://xd.adobe.com/view/59629185-54fb-457d-bb21-3bcc1985b314/


ability to use a custom CSS stylesheet

Adding to #56, this allows using a custom CSS stylesheet by passing a url in theme.customCSS:
var _bftn_options = {
  theme: {
    customCSS: 'https://example.com/my-custom-style.css',
    className: 'money',
    logos: ['images/money.png', 'images/stop.png'],
    headline: 'Your headline here.',
    body: 'Your body here.'
  }
};

Tested locally Chrome-only by manually loading the widget with and without customCSS.

This will be really helpful for us, @alanhamlett.
How was it tested?
Can you update the README?

I'm a bit nervous to merge something that allows injecting arbitrary 3rd party content (and yes, I'm aware of the hypocrisy here 😆). At minimum, this should probably set crossorigin="anonymous" on the link tag, I'd name the option just css instead of customCSS, and should add documentation to README.md.
I'm uncertain if there's an alternative workaround that doesn't involve forking the widget because of cross-origin restrictions on modifying the content of an iframe...

Since this might not be useful to other widget users, we can just fork and self-host the widget. That way all content comes from the same domain, so no chance for cross-origin issues.
I'll close this PR :)


Issue in sampled_count function

According to https://github.com/statsd/statsd/blob/master/docs/metric_types.md
sampling should be reported by adding to |@{rate} suffix to message.

Thank you 👍


Referencing desktop clr only projects in both frameworks in test project

In the master branch the EntityFramework.Relational.FunctionalTests project references EntityFramework.Core.Tests and EntityFramework.Core.FunctionalTests in both frameworks while they are built only for desktop clr. Due to this we cannot open EF solution in VS 2015 RC and build the it. Anyway this can be fixed in the master branch ?

Did you run:
build initialize

This is fixed in current code base so will soon be in master as part of the Beta 5 release


Adding GVRObjectPicker, fixed bug with hit position

This patch fixes a bug with the calculation of the hit location for
GVRMeshCollider
It add GVRObjectPicker which picks against the bounding box of the scene
object the picker is attached to.
Updated documentation on picking.
GearVRF DCO signed off by: Nola Donato<EMAIL_ADDRESS>

Please consider squashing the commits.

Replaced by PR #829 with commits squashed

Fyi, you don't have to close a pull request to update it after squashing or any other git operation.
You can be on any local branch (any name) and then do something like:
git push origin HEAD:<branch-name-you-use-for-the-pull-request> --force
This will overwrite the remote branch <branch-name-you-use-for-the-pull-request>.


java.lang.NullPointerException in GenotypeGVCFs

Hi,
I'm running into a java.lang.NullPointerException for some bcbio runs during the GenotypeGVCFs phase.
This is a known issue with GATK versions >= <IP_ADDRESS>, as outlined here: https://gatk.broadinstitute.org/hc/en-us/community/posts/360060283092-GenotypeGVCFs-java-lang-NullPointerException and here: https://github.com/broadinstitute/gatk/issues/6548.
According to the first link above, a solution is to run GenotypeGVCFs with the setting --genomicsdb-use-vcf-codec: https://gatk.broadinstitute.org/hc/en-us/articles/360040509751-GenotypeGVCFs#--genomicsdb-use-vcf-codec
Is there any way of passing this setting to GenotypeGVCFs from within Bcbio?

Thanks so much for figuring out the source of the problem and how to work around it-- you can pass it via the options config for gatk in your YAML file. I also just pushed a fix so we always run with it enabled while they fix the bug.

Alright, thanks a lot for looking into this.
I didn't realize that one could pass tool-specific settings via the gatk options, as well. Thanks for pointing that out.

No worries, not all of the tools support it, so it is hit or miss if it works or not. Thanks a lot for figuring out what was happening, hopefully the GATK team will fix it soon.

@roryk
Check this out for an update on this issue: https://github.com/broadinstitute/gatk/pull/6675.
Starting from release <IP_ADDRESS> (which came out 2 days ago and is already in bioconda), the VCF codec was made the default and the argument --genomicsdb-use-vcf-codec has been replaced by --genomicsdb-use-bcf-codec.


Scan all files in a folder

Make the code scan all files in a folder. Including subfolders.

Any idea of the timeline for this? @sreliasanderson


Updating Monolog to include security fix

See https://www.pivotaltracker.com/story/show/90607106 and https://github.com/Seldaek/monolog/pull/448#issuecomment-68208704

@rjkip

:+1:


Adding package reference to DACPAC project does not result in .dacpac file being copied to bin directory

In my solution (simplified) I have two projects:

Some.Namespace.Database - this is the MSBuild.Sdk.SqlProj SDK project for producing the DACPAC
Some.Namespace.DataAccess - this is a vanilla Microsoft.NET.Sdk (netstandard2.0) project containing C# data access code

I've added a project reference from Some.Namespace.DataAccess to  Some.Namespace.Database as follows:
  <ItemGroup>
    <ProjectReference Include="..\Some.Namespace.Database\Some.Namespace.Database.csproj" />
  </ItemGroup>

When I build Some.Namespace.Database it produces the .dacpac file in the output directory for that project, however when I build Some.Namespace.DataAccess I was hoping/expecting the .dacpac file to be copied to the output directory for that project, since it's a dependency.
Is there any easy way to achieve what I want? NB use-case is around programatically loading the DACPAC file via DacServices / DacPackage.Load in Microsoft.SqlServer.Dac (NuGet package Microsoft.SqlServer.DacFx.x64)
NB: I tried changing Some.Namespace.DataAccess to a concrete framework (netcoreapp3.1) however that didn't help 😢

@flakey-bit This isn't currently build in unfortunately. In fact I'm even surprised that it builds at all because I would expect your project Some.Namespace.DataAccess to not compile since it would try to include a reference to the .dacpac file which obviously doesn't work.
That being said I do think this could be a reasonable scenario to support. According to the docs for the project reference protocol we might be able to implement a GetCopyToOutputDirectoryItems target that would return the .dacpac which should then copy that file to the output directory.
If you're adventurous we would appreciate a PR for this. Not sure when we'll have time to implement this.

@flakey-bit This isn't currently build in unfortunately. In fact I'm even surprised that it builds at all because I would expect your project Some.Namespace.DataAccess to not compile since it would try to include a reference to the .dacpac file which obviously doesn't work.
That being said I do think this could be a reasonable scenario to support. According to the docs for the project reference protocol we might be able to implement a GetCopyToOutputDirectoryItems target that would return the .dacpac which should then copy that file to the output directory.
If you're adventurous we would appreciate a PR for this. Not sure when we'll have time to implement this.

A post build step could maybe also work?

A post build step could maybe also work?

This one was interesting... @jmezach is correct with his assumption that the project does not compile. However this can be achieved by adding <ReferenceOutputAssembly>False</ReferenceOutputAssembly>:
  <ItemGroup>
      <ProjectReference Include="..\Some.Namespace.Database\Some.Namespace.Database.csproj">
        <ReferenceOutputAssembly>False</ReferenceOutputAssembly>
      </ProjectReference>
  </ItemGroup>


[Bug] SQLite error parsing issue

Seeing TypeError: parameters.reduce is not a function when a SQLite error occurs.
Coming from this code
          let parameters = (args[0] ?? []) as string[];
          parameters =
            parameters.length <= 25
              ? parameters
              : parameters.slice(0, 26).concat(["..."]);
          const params = parameters.reduce<Record<number, any>>(
            (acc, parameter, idx) => {
              acc[idx + 1] = parameter;
              return acc;
            },
            {},
          );

located at https://github.com/ponder-sh/ponder/blob/main/packages/core/src/utils/sqlite.ts#L40
This is happening because parameters is a string and represents the first arg that was passed to the query as this code is currently written.
I'm not sure what exactly the intention is with this code, but perhaps it should be something more along the lines of this
          const params = (args ?? []).reduce<Record<number, any>>(
            (acc, parameter, idx) => {
              acc[idx + 1] = parameter.length <= 25
              ? parameter
              : parameter.slice(0, 26).concat(["..."]);
              return acc;
            },
            {},
          );

As it is right now, when a SQLite error occurs it's impossible to see the actual error that did occur because of this TypeError happening. So perhaps also worth adding a try/catch around this error parsing as well.
In case it's relevant, the error I was receiving was a integer overflow error from using sum on a bigint column with negative values and the EVM_MIN_INT encoding that is done by this repo causing that sum operation to overflow

Yea, this error parsing code is a bit cursed - thanks for flagging. Will work on a reproduction + fix.


Hammer.js not defined

Now that I'm up to date on 1.1, I'm still seeing this error.
I have a view with a form submission. Once the form submission is complete, I use the notification directive (through javascript) to display a message. Here's my JS code:
FoundationApi.publish('prayer-notifications',
{
    title: 'Prayer Added',
    content: 'Prayer "' + $scope.prayer.name + '" has been added successfully.',
    color: 'success'
});

While my message displays fine, I always wind up seeing the following error message in the web console
Hammer is not defined
I've checked my gulp scripts and it is bundling up the HammerJS library, so I'm wondering if I'm missing something in my controller or something, do I need to add that as a dependency to my controller dependency injection?

Can you post a stack trace for that error, so we can see where in the code it's being generated?

Interestingly enough, a single click triggered the error twice:

Error: Hammer is not defined
link@http://prayerjournal.dev/assets/js/foundation.js:34:1388
cloneAndAnnotateFn/<@http://prayerjournal.dev/assets/js/foundation.js:9:6028
invokeLinkFn@http://prayerjournal.dev/assets/js/foundation.js:9:6149
nodeLinkFn@http://prayerjournal.dev/assets/js/foundation.js:8:24405
compileTemplateUrl/<@http://prayerjournal.dev/assets/js/foundation.js:9:1995
compositeLinkFn@http://prayerjournal.dev/assets/js/foundation.js:8:14121
compile/<@http://prayerjournal.dev/assets/js/foundation.js:8:12792
createBoundTranscludeFn/boundTranscludeFn@http://prayerjournal.dev/assets/js/foundation.js:8:15476
controllersBoundTransclude@http://prayerjournal.dev/assets/js/foundation.js:8:20509
ngRepeatDirective</<.compile/</<@http://prayerjournal.dev/assets/js/foundation.js:13:29493
$watchCollectionAction@http://prayerjournal.dev/assets/js/foundation.js:10:25791
$RootScopeProvider/this.$get</Scope.prototype.$digest@http://prayerjournal.dev/assets/js/foundation.js:10:27348
$RootScopeProvider/this.$get</Scope.prototype.$apply@http://prayerjournal.dev/assets/js/foundation.js:10:29370
done@http://prayerjournal.dev/assets/js/foundation.js:9:17667
completeRequest@http://prayerjournal.dev/assets/js/foundation.js:9:21382
createHttpBackend/</xhr.onload@http://prayerjournal.dev/assets/js/foundation.js:9:22304

 foundation.js:10:5014

Error: Hammer is not defined
postLink@http://prayerjournal.dev/assets/js/foundation.js:38:2082
cloneAndAnnotateFn/<@http://prayerjournal.dev/assets/js/foundation.js:9:6028
invokeLinkFn@http://prayerjournal.dev/assets/js/foundation.js:9:6149
nodeLinkFn@http://prayerjournal.dev/assets/js/foundation.js:8:24405
compileTemplateUrl/<@http://prayerjournal.dev/assets/js/foundation.js:9:1995
compositeLinkFn@http://prayerjournal.dev/assets/js/foundation.js:8:14121
compile/<@http://prayerjournal.dev/assets/js/foundation.js:8:12792
createBoundTranscludeFn/boundTranscludeFn@http://prayerjournal.dev/assets/js/foundation.js:8:15476
controllersBoundTransclude@http://prayerjournal.dev/assets/js/foundation.js:8:20509
ngRepeatDirective</<.compile/</<@http://prayerjournal.dev/assets/js/foundation.js:13:29493
$watchCollectionAction@http://prayerjournal.dev/assets/js/foundation.js:10:25791
$RootScopeProvider/this.$get</Scope.prototype.$digest@http://prayerjournal.dev/assets/js/foundation.js:10:27348
$RootScopeProvider/this.$get</Scope.prototype.$apply@http://prayerjournal.dev/assets/js/foundation.js:10:29370
done@http://prayerjournal.dev/assets/js/foundation.js:9:17667
completeRequest@http://prayerjournal.dev/assets/js/foundation.js:9:21382
createHttpBackend/</xhr.onload@http://prayerjournal.dev/assets/js/foundation.js:9:22304

 foundation.js:10:5014
Here's a screenshot if it helps any better:

Hey guys,
Sorry about this. Turns out this was a non-issue. I had a custom build script (for my .Net project) and so my updated copy of foundation wasn't going to the properly location. I was still loading old foundation code. Once I fixed this and ran it, things were good.
Sorry for the hassle.


[Question]: What is the difference between PDFReader and PyMuPDFReader

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
What is the difference between PDFReaderand PyMuPDFReader

How do they differ in processing?
Is there any performance comparison?

Pdfreader uses pypdf
The other uses pymupdf
That's it :) I don't have a comparison, you can try them out yourself and compare.
LlamaParse also released recently, if you want to check that out
https://github.com/run-llama/llama_parse


Content Security Policy directive after build to production

Version
2.6.11
Reproduction link
https://realsportindo.com/event/kejuaraan-kata-virtual-forki-jabar-2020/medals
Steps to reproduce

npm run build with websocket
deploy to server

What is expected?
web socket should work properly
What is actually happening?
Refused to connect to 'wss://example.com/socket//websocket?vsn=1.0.0' because it violates the following Content Security Policy directive: "default-src 'self' http: https: data: blob: 'unsafe-inline'". Note that 'connect-src' was not explicitly set, so 'default-src' is used as a fallback.

nb:
already check on development work fine
already add meta on head

Hi, thanks for your interest but Github issues are for bug reports and feature requests about Vue only. You can ask questions on the forum, the Discord server or StackOverflow.

Hi, thanks for your interest but Github issues are for bug reports and feature requests about Vue only. You can ask questions on the forum, the Discord server or StackOverflow.


"Run VS Code outside the Developer Command Prompt" instructions fail if opened file's basename and/or dirname contains space characters

This is actually a reproduction of issue #135994 which was closed due to insufficient information to consistently recreate it. Hopefully, this new report will shed more light on the problem.
Steps to Reproduce

If necessary, configure VS Code for Microsoft C++ (see Prerequisites section in this tutorial).
Follow this guide to create a tasks.json configuration for running VS Code outside the Developer Command Prompt for Visual Studio.
Create a C/C++ source file containing at least one space character in its basename or dirname.
Attempt to build the source file using the task created in step 2.

Expected Result
The integrated shell opens and the build is successfully completed, e.g. building c:\Test\test.c results in this terminal output:
*  Executing task: cl.exe /Zi /EHsc /Fe: c:\Test\test.exe c:\Test\test.c

**********************************************************************
** Visual Studio 2022 Developer Command Prompt v17.3.0
** Copyright (c) 2022 Microsoft Corporation
**********************************************************************
Microsoft (R) C/C++ Optimizing Compiler Version 19.33.31629 for x86
Copyright (C) Microsoft Corporation.  All rights reserved.

test.c
Microsoft (R) Incremental Linker Version 14.33.31629.0
Copyright (C) Microsoft Corporation.  All rights reserved.

/debug
/out:c:\Test\test.exe
test.obj
 *  Terminal will be reused by tasks, press any key to close it.

Actual Result
The terminal process terminates with exit code 1, apparently because the double quotes enclosing the second shell argument string in tasks.json are being internally dropped, thus attempting to build c:\Test\test file.c yields:
 *  Executing task: cl.exe /Zi /EHsc /Fe: "c:\Test\test file.exe" "c:\Test\test file.c"

'C:/Program' is not recognized as an internal or external command,
operable program or batch file.

 *  The terminal process "cmd.exe /C "C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/Tools/VsDevCmd.bat" && cl.exe /Zi /EHsc /Fe: "c:\Test\test file.exe" "c:\Test\test file.c"" terminated with exit code: 1.
 *  Terminal will be reused by tasks, press any key to close it.

Additional Info
For reference, here is what my tasks.json file looks like:
{
	"version": "2.0.0",
	"windows": {
	  "options": {
		"shell": {
		  "executable": "cmd.exe",
		  "args": [
			"/C",
			// The path to VsDevCmd.bat depends on the version of Visual Studio you have installed.
			"\"C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/Tools/VsDevCmd.bat\"",
			"&&"
		  ]
		}
	  }
	},
	"tasks": [
        {
            "type": "shell",
            "label": "cl.exe build active file",
            "command": "cl.exe",
            "args": [
                "/Zi",
                "/EHsc",
                "/Fe:",
                "${fileDirname}\\${fileBasenameNoExtension}.exe",
                "${file}"
            ],
            "problemMatcher": [
                "$msCompile"
            ],
            "group": {
                "kind": "build",
                "isDefault": true
            }
        }
    ]
  }

Current Workarounds

Remove any space characters present in the opened file's basename and/or dirname.

If spaces must be retained, delete the escaped double quotes in line 10 of tasks.json and then use the caret (^) to escape all the space characters along with any parentheses, if present. Example:
Before:
          "\"C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/Common7/Tools/VsDevCmd.bat\"",

After:
          "C:/Program^ Files^ ^(x86^)/Microsoft^ Visual^ Studio/2019/Community/Common7/Tools/VsDevCmd.bat",

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.70.2 (also reproducible on 1.71.0-insider)
OS Version: Windows 10.0.19044.1889

@alexr00 has this ever been supported?

@meganrogge We try to handle escaping/quoting well, but it's been a losing battle to do it correctly on every platfom+shell combination. the quoting property (a sub-property of command in the json schema) also tries to help with this.
@casadan does it work if you try one of the other shell quoting options? Ex:
{
...
  "command": {
    "value": "cl.exe",
    "quoting": "escape"
}

@alexr00
I tried building c:\Test\test file.c with this tasks.json:
{
	"version": "2.0.0",
	"windows": {
	  "options": {
		"shell": {
		  "executable": "cmd.exe",
		  "args": [
			"/C",
			// The path to VsDevCmd.bat depends on the version of Visual Studio you have installed.
			"\"C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/Tools/VsDevCmd.bat\"",
			"&&"
		  ]
		}
	  }
	},
	"tasks": [
        {
            "type": "shell",
            "label": "cl.exe build active file",
            "command": {
				"value": "cl.exe /Zi /EHsc /Fe: ${fileDirname}\\${fileBasenameNoExtension}.exe ${file}",
				"quoting": "escape"
			},

            "problemMatcher": [
                "$msCompile"
            ],
            "group": {
                "kind": "build",
                "isDefault": true
            }
        }
    ]
  }

This time the Developer Command Prompt for Visual Studio is successfully run but then the compiler fails due to the unescaped space characters in cl.exe's /Fe argument:
 *  Executing task: cl.exe /Zi /EHsc /Fe: c:\Test\test file.exe c:\Test\test file.c

**********************************************************************
** Visual Studio 2022 Developer Command Prompt v17.3.3
** Copyright (c) 2022 Microsoft Corporation
**********************************************************************
Microsoft (R) C/C++ Optimizing Compiler Version 19.33.31629 for x86
Copyright (C) Microsoft Corporation.  All rights reserved.

cl : Command line warning D9024 : unrecognized source file type 'file.exe', object file assumed
cl : Command line warning D9024 : unrecognized source file type 'c:\Test\test', object file assumed
file.c
c1: fatal error C1083: Cannot open source file: 'file.c': No such file or directory

 *  The terminal process "cmd.exe /C "C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/Tools/VsDevCmd.bat" && cl.exe /Zi /EHsc /Fe: c:\Test\test file.exe c:\Test\test file.c" terminated with exit code: 2.
 *  Terminal will be reused by tasks, press any key to close it.

Again, removing the space character from the file name (e.g. c:\Test\test.c) works as expected:
 *  Executing task: cl.exe /Zi /EHsc /Fe: c:\Test\test.exe c:\Test\test.c

**********************************************************************
** Visual Studio 2022 Developer Command Prompt v17.3.3
** Copyright (c) 2022 Microsoft Corporation
**********************************************************************
Microsoft (R) C/C++ Optimizing Compiler Version 19.33.31629 for x86
Copyright (C) Microsoft Corporation.  All rights reserved.

test.c
Microsoft (R) Incremental Linker Version 14.33.31629.0
Copyright (C) Microsoft Corporation.  All rights reserved.

/debug
/out:c:\Test\test.exe
test.obj
 *  Terminal will be reused by tasks, press any key to close it.

Switching the quoting style from escape to strong breaks the task regardless of the file name:
 *  Executing task: "cl.exe /Zi /EHsc /Fe: c:\Test\test.exe c:\Test\test.c"

'C:/Program' is not recognized as an internal or external command,
operable program or batch file.

 *  The terminal process "cmd.exe /C "C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/Tools/VsDevCmd.bat" && "cl.exe /Zi /EHsc /Fe: c:\Test\test.exe c:\Test\test.c"" terminated with exit code: 1.
 *  Terminal will be reused by tasks, press any key to close it.

Adding escaped double quotes to the /Fe argument does not work either regardless of the quoting style or the file name, e.g.:
            "command": {
				"value": "cl.exe /Zi /EHsc /Fe: \"${fileDirname}\\${fileBasenameNoExtension}.exe\" \"${file}\"",
				"quoting": "escape"
			},

Results in:
 *  Executing task: cl.exe /Zi /EHsc /Fe: "c:\Test\test.exe" "c:\Test\test.c"

'C:/Program' is not recognized as an internal or external command,
operable program or batch file.

 *  The terminal process "cmd.exe /C "C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/Tools/VsDevCmd.bat" && cl.exe /Zi /EHsc /Fe: "c:\Test\test.exe" "c:\Test\test.c"" terminated with exit code: 1.
 *  Terminal will be reused by tasks, press any key to close it.

I solved this problem by using the pwsh.exe terminal, which may be because the cmd.exe terminal is too old
There are several key points that may need attention:

The type of the task is shell, not cppbuild

It is not necessary to use the shell of the Windows option on the outermost layer of tasks.json, but the shell of the current task's option can also be used

The Bat script file needs to be replaced with a ps1 script file (if you feel that the path is too long, you can create a soft link to a folder you often use, of course, this folder is in the environment variable Path)

Another benefit of using pwsh.exe is that you don't have to worry about garbled code

Below is my tasks.json
{
    "tasks": [
        {
            "type": "cppbuild",
            "label": "C/C++: gcc.exe 生成活动文件",
            "command": "G:\\msys64\\ucrt64\\bin\\gcc.exe",
            "args": [
                "-fdiagnostics-color=always",
                "-g",
                "${file}",
                "-o",
                "${fileDirname}\\${fileBasenameNoExtension}.exe"
            ],
            "options": {
                "cwd": "G:\\msys64\\ucrt64\\bin"
            },
            "problemMatcher": [
                "$gcc"
            ],
            "group": {
                "kind": "build",
                "isDefault": false
            },
            "detail": "调试器生成的任务。"
        },
        {
            "type": "shell",
            "label": "msvc cl.exe 生成活动文件",
            "command": "cl.exe",
            "args": [
                "/source-charset:utf-8",
                "/execution-charset:gbk",
                "/Zi",
                "/EHsc",
                "/nologo",
                "/Fe${fileDirname}\\${fileBasenameNoExtension}.exe",
                "${file}",
                "User32.Lib"
            ],
            "options": {
                "cwd": "${fileDirname}",
                "shell": {
                    "executable": "pwsh.exe",
                    "args": [
                        "-c",
                        "msvc",/*msvc is soft link*/
                        "&&"
                    ],
                }
            },
            "problemMatcher": [
                "$msCompile"
            ],
            "group": {
                "kind": "build",
                "isDefault": true
            },
            "presentation": {
                "echo": true,
                "reveal": "always",
                "focus": false,
                "panel": "shared",
                "showReuseMessage": false,
                "clear": false
            }
        }
    ],
    "version": "2.0.0"
}


Assembler error with sequence .if .db .endif

If there is a .db within an .if to .endif block then the byte data is placed in the executable code even when the .if condition is false.
Test Code
    .assume adl=1
    .org $000000

    ld      hl, 0x112233

.if 0

    ld      hl, 0x445566

    .db     "\r\nHL = ", 0

    ld      hl, 0x778899

.endif

    ld      hl, 0xAABBCC

Listing File
This is the listing file that is created with the -l compile option. It can be seen byte data is added at address range 0x000004 to 0x00000B.
PC     Output      Line

040000             0001      .assume adl=1
040000             0002      .org $000000
000000             0003
000000 21 33 22 11 0004      ld      hl, 0x112233
000004             0005
000004             0006  .if 0
000004             0007
000004             0008      ld      hl, 0x445566
000004             0009
000004 0D 0A 48 4C 0010      .db     "\r\nHL = ", 0
       20 3D 20 00
00000C             0011
00000C             0012      ld      hl, 0x778899
00000C             0013
00000C             0014  .endif
00000C             0015
00000C 21 CC BB AA 0016      ld      hl, 0xAABBCC
000010             0017
000010             0018

Fixed for all types of data/blk data statements in commit 549c238. I'll need to revisit which additional directives may benefit from also respecting if/else/endif.

Thank you, that was quick!
Confirmed that the issue with the code I was compiling is resolved.

Thanks for bringing this up; this reminds me to also add in per file stacked if/else/endif, so something like
.if 1
.include "1.s"
.else
.include "2.s"
.endif
will become possible, while any if/else/endif statements within the included files will also still work and the caller source file will retain the if state upon returning from the include.
I'd then be able to bring this functionality over to incbin and include that in v1.10

Fixed in release v1.10


Update smtpd.go: Adding case-insensitive flags to regex'es

The syntax is taken from https://stackoverflow.com/a/15326471/1668200
Do you have tests for case-insensitivity?

Nice. I don't believe there are any existing tests for that, however you can easily add one or two (no need to duplicate all tests). Please see the existing smtpd_test.go.
As you already saw too, I changed the paths of those modules to the internal folder 👌

I changed the next best MAIL FROM to lowercase ;-)


[Question]: Upgrade Krypton Toolkit in Visual Studio Project

I hope that asking this question here is appropriate!
I have several existing .net projects that use the 4.4 release of the Krypton Toolkit and am keen to upgrade this to the latest version on GitHub to take advantage of the bug fixes etc.
I am unclear about how to do this - I have tried changing the toolkit references in the VS project but then VS complains that all the components used are 'not defined'.
I would appreciate any advice on how to complete this upgrade.
Thanks
Andy

Hi @borismatthews
Are you using the original 4.4.0 release? If so, have you tried downloading the newer NuGet packages?

Hi
Thanks for responding so quickly.
Yes, I am using the original 4.4.0 from Component Factory.
I downloaded and installed version 5.470.872 from GitHub (did not use NuGet, just downloaded the exe file).
It appears that the name of the reference is different and so maybe that is why Visual Studio cannot match the references of the controls to the dll’s?
The 4.4.0 references appear as follows:
<EMAIL_ADDRESS>Then when I install the updated version
<EMAIL_ADDRESS>Regards
Andy
From: Peter Wagner<EMAIL_ADDRESS>Sent: 16 July 2019 15:45
To: Wagnerp/Krypton-NET-5.470<EMAIL_ADDRESS>Cc: Andrew Matthews<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Wagnerp/Krypton-NET-5.470] Upgrade Krypton Toolkit in Visual Studio Project (#210)
Hi @borismatthewshttps://github.com/borismatthews
Are you using the original 4.4.0 release? If so, have you tried downloading the newer NuGet packages?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Wagnerp/Krypton-NET-5.470/issues/210?email_source=notifications&email_token=AMFO36NUNCOGLAVPZHPQFTLP7XNIPA5CNFSM4IEAPFXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2BC4GI#issuecomment-511847961, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMFO36MRW36JM2ABZZSSBR3P7XNIPANCNFSM4IEAPFXA.

Hi @borismatthews
What version of the .NET framework are you using?

Peter
Sorry, I was targeting 4.0, have changed to 4.7.2 and all OK now.
Also, I see that you have the “Krypton-Toolkit-Suite-Extended-NET-5.470” project, is this intended to be used in addition to the standard toolkit or in place of?
Thanks
Andy
From: Peter Wagner<EMAIL_ADDRESS>Sent: 16 July 2019 17:01
To: Wagnerp/Krypton-NET-5.470<EMAIL_ADDRESS>Cc: Andrew Matthews<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Wagnerp/Krypton-NET-5.470] Upgrade Krypton Toolkit in Visual Studio Project (#210)
Hi @borismatthewshttps://github.com/borismatthews
What version of the .NET framework are you using?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Wagnerp/Krypton-NET-5.470/issues/210?email_source=notifications&email_token=AMFO36OGQ462GMXZ4MQHYFLP7XWFTA5CNFSM4IEAPFXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2BKUBA#issuecomment-511879684, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMFO36LRPBI4OOFCJFM6Y6TP7XWFTANCNFSM4IEAPFXA.

Hi @borismatthews
The Krypton-Toolkit-Suite-Extended-NET-5.470 is an additional set of controls, dialogs etc that can be used in conjunction with the standard toolkit. Glad you managed to find out what the issue was.

Hi Peter
I was hoping that the updated controls would fix an issue I have in the Visual Studio designer but unfortunately it has not.
Using VS2017 on Windows 7 machine the control spacing at design time is correct but when I upgraded to a Windows 10 machine and VS2017 the spacing around the Kypton ComboBox at design time is incorrect.
If you look at this screenshot you can see that at design time the combo boxes have additional space but when you run this it appears correct – clearly not easy to layout the controls to allow for this.
Is this an issue that you are aware of and any ideas if it can be rectified?
<EMAIL_ADDRESS>Thanks
Andy
From: Peter Wagner<EMAIL_ADDRESS>Sent: 17 July 2019 08:57
To: Wagnerp/Krypton-NET-5.470<EMAIL_ADDRESS>Cc: Andrew Matthews<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Wagnerp/Krypton-NET-5.470] [Question]: Upgrade Krypton Toolkit in Visual Studio Project (#210)
Hi @borismatthewshttps://github.com/borismatthews
The Krypton-Toolkit-Suite-Extended-NET-5.470 is an additional set of controls, dialogs etc that can be used in conjunction with the standard toolkit. Glad you managed to find out what the issue was.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Wagnerp/Krypton-NET-5.470/issues/210?email_source=notifications&email_token=AMFO36JBASQVIW22J5YMOYLP73GGBA5CNFSM4IEAPFXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2DLX2I#issuecomment-512146409, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMFO36L7OJ62WKXVEN4KFITP73GGBANCNFSM4IEAPFXA.

Hi @borismatthews
Unfortunately, the screenshots don't appear. I would suggest opening a bug report detailing the problem that you have with the padding.

Hi @borismatthews
Can this now be closed?

Hi
Yes, that’s fine.
Thanks
From: Peter Wagner<EMAIL_ADDRESS>Sent: 18 December 2019 19:43
To: Wagnerp/Krypton-NET-5.470<EMAIL_ADDRESS>Cc: Andrew Matthews<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Wagnerp/Krypton-NET-5.470] [Question]: Upgrade Krypton Toolkit in Visual Studio Project (#210)
Hi @borismatthewshttps://github.com/borismatthews
Can this now be closed?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Wagnerp/Krypton-NET-5.470/issues/210?email_source=notifications&email_token=AMFO36PA2XCGKG6ZOBEUCUTQZJ4NNA5CNFSM4IEAPFXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHHIGWQ#issuecomment-567182170, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMFO36JVJ36ZYFRY4HKEVWDQZJ4NNANCNFSM4IEAPFXA.

Hi @borismatthews
Thanks, if you have anymore questions in the future, please post them to this repo.


Authorization URL (State)

I could't figure out what state= in the auth url. It is saying "state-of-my-choice" but what do I need to put there

@furkanayilmaz It's a standard that makes your API calls secure to CSRF attacks, you can learn more about it here https://auth0.com/docs/secure/attack-protection/state-parameters


Fix syntax error in sample code

This would have otherwise caused a non-keyword arg after keyword arg

Can one of the admins verify this patch?

There are actually a bunch more issues in this sample code. I can fix some of these (or the author can take a look) later

@MyHumbleSelf definitely feel free to fix anything that you think is wrong.
Jenkins, test this please.


Jetty 12 detailed response metrics bug: IndexOutOfBoundsException

Jetty12 instrumentation with detailed response metrics are broken, because coarse metrics are not collected but the instrumentation still registers the gauges with hardcoded array indexes which don't exist. This breaks the complete Graphite metrics publishing in our case.
java.lang.IndexOutOfBoundsException: Index: 3
    at java.util.Collections$EmptyList.get(Collections.java:4807)
    at io.dropwizard.metrics.jetty12.AbstractInstrumentedHandler$3.getRatio(AbstractInstrumentedHandler.java:201)

Consider this code: https://github.com/dropwizard/metrics/blob/release/4.2.x/metrics-jetty12/src/main/java/io/dropwizard/metrics/jetty12/AbstractInstrumentedHandler.java#L162-L169
        this.responses = COARSE_METER_LEVELS.contains(responseMeteredLevel) ?
                Collections.unmodifiableList(Arrays.asList(
                        metricRegistry.meter(name(prefix, NAME_1XX_RESPONSES)), // 1xx
                        metricRegistry.meter(name(prefix, NAME_2XX_RESPONSES)), // 2xx
                        metricRegistry.meter(name(prefix, NAME_3XX_RESPONSES)), // 3xx
                        metricRegistry.meter(name(prefix, NAME_4XX_RESPONSES)), // 4xx
                        metricRegistry.meter(name(prefix, NAME_5XX_RESPONSES))  // 5xx
                )) : Collections.emptyList();

The responses array is only filled if coarse meter levels are enabled. In case of detailed meter levels the responses field is an emptyList.
Now consider this code later: https://github.com/dropwizard/metrics/blob/release/4.2.x/metrics-jetty12/src/main/java/io/dropwizard/metrics/jetty12/AbstractInstrumentedHandler.java#L198-L204
        metricRegistry.register(name(prefix, NAME_PERCENT_4XX_15M), new RatioGauge() {
            @Override
            protected Ratio getRatio() {
                return Ratio.of(responses.get(3).getFifteenMinuteRate(),
                        requests.getFifteenMinuteRate());
            }
        });

This registers a gauge which calls responses.get(3) on an emptyList which will fail with a runtime exception as the index is out of the array bounds. When using the GraphiteReporter, this bug breaks the metric publishing completely.

The coarse gauges should only be registered if the coarse meter levels are enabled
It would be nice to have a test case for this class which also covers detailed metrics without coarse

@mihalyr Thanks for reporting this! The fix for this bug will be included in the next version of Dropwizard Metrics.


⚠ Bofrost.nl has degraded performance

In e9ea77e, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1071 ms

Resolved: Bofrost.nl performance has improved in fff79d7.


Feature: add values deprecation management to helm

In our use of helm, we have created a chart that use values that can be modified and/or deleted during chart development. This happens because we can be led to modify the functioning of our chart or because values are no longer necessary for example.
We would therefore like to have the possibility to warn the users of our chart that some of the values used are deprecated and will be deleted in the near future.
For the moment we have put this information in the NOTES.txt for the user to have direct feedback. This has also required the addition of value checking to display the message only if deprecated values are used. In fact, we are not sure it's the best place to put these informations and users could miss them.
Would it be possible to add a native deprecated value management mechanism to helm allowing:

to indicate for each dict value if its deprecated
to inform the user by stdout ([WARN]) during helm install/upgrade if some deprecated values were used
to customize displayed message to invite the user to update its values

Thanks

I think a combination of notes and a JSON Schema to validate values would go a long way here. You can use the description fields in the JSON Schema to also note which fields plan to be deprecated in future iterations.
https://helm.sh/docs/faq/#validating-chart-values-with-jsonschema
http://json-schema.org/learn/getting-started-step-by-step.html

Thanks, we also already use JSON schema validator, we will take a look at description field

It would be useful to be able to detect "deprecated" annotations that have appeared since draft 2019-09.
http://json-schema.org/draft/2019-09/json-schema-validation.html#rfc.section.9.3
Perhaps one of the barriers is that xeipuuv/gojsonschema does not support draft 2019-09.
https://github.com/xeipuuv/gojsonschema/issues/289

I opened #10732 for this.


CamelCaseToDashFilter & numbers

Not sure if this is expected behavior but seems like a bug to me, but if you pass a string with numbers into CamelCaseToDashFilter the dash is not applied before a number, please see below for a failing unit test.
    public function testFilterSeparatesCamelCasedWordsContainingNumbersWithDashes()
    {
        $string   = 'ItIs2016';
        $filter   = new CamelCaseToDashFilter();
        $filtered = $filter($string);
        $this->assertNotEquals($string, $filtered);
        $this->assertEquals('It-Is-2016', $filtered);
    }

This repository has been closed and moved to laminas/laminas-filter; a new issue has been opened at https://github.com/laminas/laminas-filter/issues/10.


Android O Notification updates

W/Notification: See the documentation of setSound() for what to use instead with android.media.AudioAttributes to qualify your playback use case
W/Notification: Use of stream types is deprecated for operations other than volume control

Writes down the logs and drastically drops the performance.
Needs fix asap.
But who uses Android O this early?

The error is still on

According to this stackoverflow answer, this is a warning. Not bug to fix.


Officially document Heroku install

Jared's done a great job maintaining it here: https://github.com/szechyjs/kegbot-heroku
Give it a spin, address any issues, and point people that way.
Would love to make this the preferred DIY server route (compared to constantly dealing with the litany of Linux environments and variety of Linux experience levels).
cc @szechyjs

It's in the knowledge base now, pimp elsewhere as necessary.


PowerShell Example

Could you please provide an example using PowerShell, this is the scripting language that is used by systems administrators for all of the other Microsoft products.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: dcf430fa-621b-d643-ea58-e1752a57c0bf
Version Independent ID: 851a9803-18ad-da89-ffb5-7e0702e94d39
Content: Manage IP address ranges using the API
Content Source: CloudAppSecurityDocs/api-data-enrichment-manage-script.md
Service: cloud-app-security
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

Thanks for your question. We'll investigate and get back to you.

@DeanGross the Powershell API for Defender for Cloud Apps (MCAS) can be found here: https://github.com/Microsoft/MCAS note that this is an unofficial implementation but the examples found therein should satisfy your requirement

@DeanGross - As @ms-smithch mentioned, there is a PowerShell module that is not currently supported and is not official. It can be used as a sample, effectively that's what it is.
When/if in the future an official PowerShell module is developed, we will certainly let everyone know in the documentation.


Vulnerability roundup 113: gogs-0.12.6: 1 advisory [8.8]

search, files

[ ] CVE-2019-10348 CVSSv3=8.8 (nixos-unstable)

CVE details
CVE-2019-10348
Jenkins Gogs Plugin stored credentials unencrypted in job config.xml files on the Jenkins master where they can be viewed by users with Extended Read permission, or access to the master file system.

Scanned versions: nixos-unstable: ff9efb0724d.
Cc @kalbasit
Cc @schneefux

Unrelated to the packaged gogs


Make v2x support implicit flipflops

As I mentioned in #639;

The verilog always shown below (which is called a "process")
always @(posedge clk) begin
   ....
end

should cause Yosys to generate $dff objects once the proc pass is run (the proc_dff pass is the exact one).

So the issue might be these lines;
https://github.com/SymbiFlow/symbiflow-arch-defs/blob/2a41dcc11eee49c3315d811e7060409c4a9fac56/utils/vlog/yosys/json.py#L229-L236
The code seems to be striping out cells which start with $ like the $dff cells generated by the proc pass...
We should fix this and add tests.

This is part about https://github.com/SymbiFlow/symbiflow-arch-defs/issues/599


jalr issue

Hi
When i compile these, i got error:
.section .text
.global _start
_start:
    lui     a1, %hi(msg)
    addi    a1,a1,%lo(msg)
    jalr    ra, puts
2:  j 2b

.section .rodata
msg:
    .string "Hello World\n"

/root/workspace/riscv>riscv-none-embed-as a.s
a.s: Assembler messages:
a.s:6: Error: illegal operands `jalr ra,puts'

/root/workspace/riscv>riscv-none-embed-as -v
GNU assembler version 2.32 (riscv-none-embed) using BFD version (xPack GNU RISC-V Embedded GCC, 64-bit) 2.32

thanks
Peter

Perhaps you meant to use jal?  jal takes a 20-bit offset.  jalr takes a reg plus 12-bit offset.  We don't allow symbols with jalr as it isn't useful, this would be x0 as the base register plus a 12-bit offset which can only work if the target is in the low 4KB of memory which is rather unlikely.  jal with a symbol is allowed and gives a 1MB range which is enough to be useful.
If you really want jalr, you can use
jalr    ra, %lo(puts)(x0)
The standard way to do a call is to use the call macro
call puts
This expands to
c:	00000097          	auipc	ra,0x0
c: R_RISCV_CALL	puts
c: R_RISCV_RELAX	ABS
10:	000080e7          	jalr	ra # c <main+0xc>
where the R_RISCV_CALL reloc puts the high part of the address offset in the auipc and the low part in the jalr, and the linker will relax this to a jal if in range.  We unfortunately don't have assembler syntax to generate this directly, so you have to use the call macro.  There is also a tail macro for tail calls.

But note that
jalr ra, %lo(puts)(x0)
will silently fail if puts is not in the low 4KB of memory.

thanks @jim-wilson


Swift Compiler warning

Xcode version 11.5
Swift compile warning:
File: WSCompression.swift -

line: 150 ( strm.next_out = UnsafeMutablePointer(&buffer))
Initialization of 'UnsafeMutablePointer' results in a dangling pointer

line 212 (strm.next_out = UnsafeMutablePointer(&buffer))
warning:
Initialization of 'UnsafeMutablePointer' results in a dangling pointer

Facing same warnings.