Why is the line height not being considered before hbox's?

In my quest for typesetting books directly from Markdown I'm I'm trying to come up with a basic function for typesetting series pages using meta yaml data. For some series this data has authors, for some it doesn't so there are two forms.  The Pandoc template is feeding SILE code that looks like this:
\seriespage:series{Müjde Koalisyonu}
\seriespage:title[author=Philip Graham Ryken]{Aklanma}
\seriespage:title[author=Richard D. Phillips]{Gerçeği Bilebilir Miyiz?}
\seriespage:title[author=Reddit Andrews III]{Günah ve Düşüş}
\seriespage:title[author=Sam Storms]{Her Şeyin Yenilenmesi}
\seriespage:title[author=Tim Savage]{Kilise: Tanrı’nın Yeni Halkı}
\seriespage:title[author=Kevin L. DeYoung]{Kutsal Ruh}
\seriespage:title[author=Sandy Willson]{Mesih'in Kefareti}
\seriespage:title[author=D. A. Carson ve Timothy Keller]{Müjde Merkezli Hizmet}
\seriespage:title[author=Bryan Chapell]{Müjde Nedir?}
\seriespage:title[author=Mike Bullmore]{Müjde ve Kutsal Yazılar: Kutsal Kitap'ı Nasıl Okumalıyız?}
\seriespage:title[author=Colin S. Smith]{Plan}
\seriespage:title[author=Stephen T. Um]{Tanrı'nın Krallığı}
\seriespage:title[author=Thabiti Anyabwile ve J. Ligon Duncan]{Vaftiz ve Rab'bin Sofrası}
\seriespage:title[author=Andrew M. Davis]{Yaratılış}

Some of these will fit nicely on one line, some will not. If necessary I wanted it to wrap before the author rather than anywhere else in the line so I wrapped the author in an hbox.
SILE.registerCommand("seriespage:series", function(options, content)
  SILE.call("center", {}, function()
    SILE.call("book:chapterfont", {}, function()
      SILE.process(content)
      SILE.typesetter:typeset(" Serisi’ndeki Kitaplar")
      SILE.call("book:chapter:post")
    end)
  end)
end)

SILE.registerCommand("seriespage:title", function(options, content)
  SILE.call("raggedright", {}, function()
    SILE.settings.set("current.parindent", SILE.nodefactory.newGlue("-2em"))
    SILE.settings.set("document.lskip", SILE.nodefactory.newGlue("2em"))
    if not options.author then
      SILE.call("font", { style="Italic", language="und" }, content)
      SILE.call("medskip")
    else
      SILE.call("font", { style="Bold", language="und" }, content)
      SILE.typesetter:typeset(" ")
      SILE.call("hbox", {}, function()
        SILE.typesetter:typeset("— ")
        SILE.call("font", { style="Italic", language="und" }, function()
          SILE.typesetter:typeset(options.author)
        end)
      end)
      SILE.call("smallskip")
    end
  end)
end)

This almost worked. The breaks are happening at the right place. First the author drops to the next line and only if necessary does the title also get split. However the leading is all wrong:

What value controls the leading in a situation like this? Or am I perhaps approaching this all wrong? Or is SILE tripping up on something?

Would this be a reasonable use of small negative penalties?
SILE.call("penalty", { penalty=-1 })

i.e. Please break here if you break anywhere?

Yes, I would use penalties here to control line breaking. You could try using \goodbreak, which is defined as being a penalty of -500.
As far as the \hbox, it's not necessarily a bad way to do it, although it means if you have a very long author name that won't get split at all when perhaps you might want it to. I don't know why that doesn't work. \hbox should set its height to the height of the highest contained box; in fact, reading the code, it does. But I wonder if you are using the linespacing package, and something in there is not reading the heights correctly. What linespacing mode are you using?

For some reason \goodbreak didn't work for me when I first started tinkering, which is what led me to \hbox and I forgot about penalties for a while. On reviewing today I tried the penalty manually and found that -1 does what I was hoping for. Anything more (e.g. -100) and it forces a break even if the line doesn't need one at all.
Yes I'm using line-spacing with a value of fit-font.

Yep, so fit-font scans the line for any hint of how big the font is, but all it sees on the line is one big ol' opaque hbox. There's no actual glyphs on the line, so the line has a font height of zero, hence what you are seeing.
Broken as designed, unfortunately.


Serverless Microservices and Service meshes

Does a microservices using serverless need a Service meshes?

Document details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f259ccea-882e-9496-27cf-af8d13edaae4
Version Independent ID: a7bab5c8-7227-e836-a721-0c54f94a2d8c
Content: Interservice communication in microservices
Content Source: docs/microservices/design/interservice-communication.md
Service: architecture-center
Sub-service: reference-architecture
GitHub Login: @MikeWasson
Microsoft Alias: pnp

Service mess is more of a container orchestrator concept. This article doesn't really address serverless architectures -- we're hoping to provide more guidance on serverless microservices architectures in the future. In the meantime, I'll try to clarify this point in the article. Thanks!


Web interface to write your own chart

Re-write existing javascript page to allow self generated bumps charts

select series or create your own
if series has abbreviations, show them
enter results, updated alongside results
prefer to use python rendering rather than javascript to avoid differences
option to download/show current chart full screen

Script added with ca649532eb73eac63d5600ecb82150b2c24c1a8e
To make this work

I changed the d results code to use () rather than <>: 9af122e2451bc22c610df572e25693f618964c36
I make sure it doesn't crash if no crews are listed: e9b3243c47fd9c0ee5b98a6683389d450e35afae

Need to add option to download the chart

Can now view only the chart in a8320145c7cd70a913f28d32070f27fa57cfd2f6, is good enough solution.


Receiving extended scans on an Adafruit nRF 52840

Describe the bug
A clear and concise description of what the bug is.
The "bt_le_scan_start" method fails with error -5, when using extended advertisements (CONFIG_BT_EXT_ADV).
After investigating I've found the error " bt_hci_core: opcode 0x2041 status 0x01". This comes from hci_core.c, line 341, It returns an IO error. Not too sure what could be wrong with IO?
What have you tried to diagnose or workaround this issue?
I've ran the sample sync_adv with the same error.
CONFIG_BT_EXT_ADV disabling solves the issue but won't scan extended advertisements obviously.
Tried with different Adafruit feather, same result.
To Reproduce
Steps to reproduce the behavior:
Since the error happens on the sample as well, run the sample bluetooth > periodic_sync.
Issue isn't related with any periodic advertising. Only starting the extended scanner.
Expected behavior
A clear and concise description of what you expected to happen.
Same behaviour as starting it without extended advertisements: Callbacks being invoked on each scan.
Impact
What impact does this issue have on your progress (e.g., annoyance, showstopper)
Blocker. Have a deadline to work with new Bluetooth features (ext advertisements)
Logs and console output

Environment (please complete the following information):

OS: Win 10
Toolchain Zephyr SDK
newest version 2.4, 1e61d3eb9b

did you turn on the experimental CONFIG_BT_CTLR_ADV_EXT=y ?
See, https://github.com/zephyrproject-rtos/zephyr/blob/master/samples/bluetooth/periodic_sync/boards/nrf52840dk_nrf52840.conf

did you turn on the experimental CONFIG_BT_CTLR_ADV_EXT=y ?
See, https://github.com/zephyrproject-rtos/zephyr/blob/master/samples/bluetooth/periodic_sync/boards/nrf52840dk_nrf52840.conf

I did not. This fixed it for me!
I didn't knew there weren't any board specific config's...
You just saved me a from a lot of frustration. :)
Tyvm

I did not. This fixed it for me!
I didn't knew there weren't any board specific config's...
You just saved me a from a lot of frustration. :)
Tyvm


dht.toJSON().values never gets populated

I've been running a DHT node with this package for several hours now, but the dht node never seems to store any values. Here's my code:
...

console.log = function(data) {
   if(arguments.length) {
      var timestamp = '[' + new Date().toUTCString() + ']';
      this.logCopy(timestamp, arguments);
   }
};

...

dht.on('announce', function(peer, infoHash) {
  console.log("announce "+node_index+": "+peer.host + ':' + peer.port+' : '+infoHash.toString('hex'));
  console.log("values stored by node #" + node_index)
  console.log(dht.toJSON().values);
});

...

And here's my output:

[Tue, 26 Jul 2016 13:32:28 GMT] { '0': 'announce 0: 128.28.XXX.XXX:XXXXX : f2d7dd2727d807050edc56fb1c28XXXXXXXXXXXX' }
[Tue, 26 Jul 2016 13:32:28 GMT] { '0': 'values stored by node #0' }
[Tue, 26 Jul 2016 13:32:28 GMT] { '0': {} }

Why is dht.toJSON().values always empty?

If you're behind a firewall or a NAT, then no peers will be able to connect to you and store values. That's my bet for what's going on here.

I don't think that's the problem. My firewall is disabled and the PC I'm running this on is DMZed in my router's settings.
Before I DMZed my PC, bittorrent-dht never received any announces. Now, I'm receiving announces, but no values are being stored.
How is it possible to receive announces while not receiving any values?

When you receive announces, those are peer IP addresses, but that data is not important to persist so toJSON doesn't return it. The values key is for storing extra additional, much larger data in the DHT, via an extension called BEP44. In order to enable that, you need to follow the instructions in the readme and initialize the DHT like this:
var ed = require('ed25519-supercop')
var dht = new DHT({ verify: ed.verify })


Client Insights missing TX/RX stats - Prometheus

It appears the Client Insights Dashboard for the Wired/Wireless devices does not contain total TX/RX stats anymore. Is this a limitation with Prometheus?

What versions of UniFi Controller, Grafana, Poller and Prometheus are you running? Can you show me a screenshot of the broken dashboard? Does poller output any errors?

Unifi Controller = 5.12.35; Grafana = 6.5.2; Poller= 1.6.3 Prometheus = latest
No errors on the Poller. These stats were available in the influxDB version of the Dashboards for what it's worth.

I think I just removed it from there since traffic is shown below. You can probably add that metric back real easy.

I just looked at this a bit more. In Prometheus, you can only really get one metric value into a graph. I chose rate in this case. The other values are just labels. There is probably a way to join two metrics, but I'm not real good with Prometheus and I'm not sure how. @SuperQ may be able to figure it out. :D
For an easy workaround, you can do this. Change the query from rate (what it is now) to:
unifipoller_client_receive_bytes_total+unifipoller_client_receive_bytes_total{site_name=~"$Site", name=~"$Client", ap_name=~"$AP"}

That should give you what you want. You'll have to rename the column in the Visualize settings.

Keep in mind this table does not respect the time range, and you will see accumulated totals since the unifi controller started up. I do not know how to fix this, and that's where @SuperQ's help may be useful. It may not really be something that can be fixed. Prometheus and tables don't play that well together.

Understood. I'll take a closer look at it.

Grafana can render multiple queries by matching label values for Prometheus.
ap_rate_panel.json.gz

Grafana can render multiple queries by matching label values for Prometheus.
ap_rate_panel.json.gz

I duplicated the original panel so I could experiment. When I enter the new query A you have listed, it shows all of my devices vs just wireless devices. How would I filter it out so it only shows wireless devices still?

@truncsphere Add wired="false" to the label filter list in the query to only show wifi in that table.

If that doesn't work, for some reason, I prefer wired != "true".
My question is: Can we make this show data transfer totals for the selected time range without using Instant ?

@davidnewhall No, instant query is the right thing to do. Grafana provides the $__range template variable to give you the selected time range. If you want the transfer totals, not the average transfer rate, switch the rate() for increase().
For example:
sum by (name, ap_name, mac, oui, ip, site_name) (
  increase(unifipoller_client_receive_bytes_total{wired!="true", site_name=~"$Site", name=~"$Client", ap_name=~"$AP"}[$__range])
)

These are the examples I need more of :D


Linux

Is there anyway to use data curator on a Linux distro (apart from using a VM, wine etc.)?

Hi @joelgombin,
Developing a Linux distribution is not in our plans. All I can suggest is exploring the Electron docs for clues on how you could do the build on your computer. If you work it out, a PR is very welcome  :-)

In principle electron apps are cross-platform. If I check out the repo on linux the application will run in dev mode:
git clone https://github.com/ODIQueensland/data-curator.git
cd data-curator/
yarn install
yarn run dev

Given the application launches under linux, there shouldn't be much of a barrier to producing a linux binary of some description, especially given data curator removes the dependency on travelling ruby/csvlint (which was one of the things that required some fiddling to get the builds to work cross-platform in comma chameleon). Attempting to runyarn build on linux fails with Error: Icon set directory /bla/bla/bla/build/icons doesn't contain icons. It may be that adding an icon  in the correct format is the only thing you need to do in order to build a linux binary.

Great! Thanks for investigating @chris48s.

Yep - I've just checked out the latest version. yarn build now runs correctly. :+1: By default it produces an AppImage, but it seems like its fairly easy to configure electron-builder to produce other package formats: https://www.electron.build/configuration/linux
I'm not sure on your process for packaging a release, but given you've got a build process which can build a linux binary it might be nice to include one in future releases.

see also #830


As a customer/visitor, I want to be able to add a product to my cart so I can collect my desired products in one place

Frontend Tasks:

Create a skeleton Cart react component
Create an "Add to cart" button component that has an onClick handler function to add a product to a users cart
Update redux store w/ action type, creator, thunk, and reducer to fetch products currently in users cart + update store when user adds new items to cart

Backend Tasks:

Create a GET route to retrieve all of the products in a users cart
Create a PUT route to update the users cart with the item they selected

Potential implementation strategy: we utilize sessionStorage.setItem('key', 'value') to save any item users add to their cart regardless of signed in status (this can happen in the onClick in side of the single product view when adding to cart). Additionally in the onClick, we do a check if the user is signed in or not (w/ the auth) and if yes also make a PUT request to update cartLiquor w/ the product info user is adding to cart. This will ensure this info is saved both in sessionStorage (for if users are not signed in/guest) as well as in the cartLiquor db table (for signed in users to have cart persistance).
The next step will be on the actual cart page, again using auth to see if the user is signed in or not. If yes, then we fetch the items from cartLiquor using the users cart id. If no, then we use sessionStorage.getItem('key') to load items in the cart. Then, once user completes checkout, we clear sessionStorage/update the db once more.

Need to logic on line 20. (logic is automatically adding webpack login garbage).


Add ctx in bindings

Signed-off-by: pigletfly<EMAIL_ADDRESS>Description
add context.Context in bindings

Issue reference
https://github.com/dapr/dapr/issues/2716

Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[x] Unit tests passing
[x] End-to-end tests passing
[x] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[x] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[x] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Once https://github.com/dapr/components-contrib/pull/1681 is merged please update this PR with a commit to pull in the latest contrib -- go get github.com/dapr/components-contrib@master

Once dapr/components-contrib#1681 is merged please update this PR with a commit to pull in the latest contrib -- go get github.com/dapr/components-contrib@master

ok

I think the following lines need to be updated:
func (b *mockBinding) Read(handler func(*bindings.ReadResponse) ([]byte, error)) error {
should be
func (b *mockBinding) Read(handler func(context.Context, *bindings.ReadResponse) ([]byte, error)) error {
https://github.com/dapr/dapr/blob/aa6a9092e67bf50ca921a4f443e00e9afa468b04/pkg/runtime/runtime_test.go#L2977
and
func (m *MockBinding) Read(handler func(*bindings.ReadResponse) ([]byte, error)) error {
should be
func (m *MockBinding) Read(handler func(context.Context *bindings.ReadResponse) ([]byte, error)) error {
https://github.com/dapr/dapr/blob/4b920e3a69cc92b26b998e6ea267f73ac8faf64d/pkg/testing/bindings_mock.go#L20

Same thing for invoke actually:
https://github.com/dapr/dapr/blob/4b920e3a69cc92b26b998e6ea267f73ac8faf64d/pkg/testing/bindings_mock.go#L26
https://github.com/dapr/dapr/blob/aa6a9092e67bf50ca921a4f443e00e9afa468b04/pkg/runtime/runtime_test.go#L2996

@artursouza the error looks like it's in runtime.go but that's actually because the runtime_test.go is calling the real runtime with the wrong signature.

I think the following lines need to be updated:
func (b *mockBinding) Read(handler func(*bindings.ReadResponse) ([]byte, error)) error {
should be
func (b *mockBinding) Read(handler func(context.Context, *bindings.ReadResponse) ([]byte, error)) error {
https://github.com/dapr/dapr/blob/aa6a9092e67bf50ca921a4f443e00e9afa468b04/pkg/runtime/runtime_test.go#L2977
and
func (m *MockBinding) Read(handler func(*bindings.ReadResponse) ([]byte, error)) error {
should be
func (m *MockBinding) Read(handler func(context.Context, *bindings.ReadResponse) ([]byte, error)) error {
https://github.com/dapr/dapr/blob/4b920e3a69cc92b26b998e6ea267f73ac8faf64d/pkg/testing/bindings_mock.go#L20

gotcha

after https://github.com/dapr/components-contrib/pull/1688 is merged, I will update go.mod again

@pigletfly I asked @artursouza to merge the other PR for me (to overwrite failing tests again) and then to update this PR here.
So it is up to date now. Hopefully everything passes.

@pigletfly why did you update this again? It was already done as I said!  Please run that and commit the change.
Then ask @mukundansundar or @daixiang0 to run this test again.

@pigletfly why did you update this again? It was already done as I said. Now we unfortunately have to run these very slow tests again... and I cannot trigger the tests because I'm not a maintainer in dapr/dapr. -- The PR was already ready to be merged because Artur updated it. Maintainers can update PRs.
Please ask @daixiang0 to trigger this test again so we can merge this very quickly.
rebased from upstream/master

@mukundansundar @daixiang0 we are ready to go

/ok-to-test

I have re-run failed jobs by hand :)

/ok-to-test

@pigletfly thanks for your contribution!


Calls slack api to check if user is active

Excuse my lack of Python knowledge I'm sure you can find a better way to not use the filter method.

Great addition! This definitely works fine for the time being


Try use StaticArrays.jl again for Julia

Following the latest discussion in:

https://github.com/jinyus/related_post_gen/issues/194#issuecomment-1777172218

@algunion @LilithHafner

Main:
Julia | 30.38 ms | 481.33 ms | 4.23 s
This PR:
Julia:

        Benchmark 1: julia --project=Related -e "using Related; main()"
        Processing time (w/o IO): 30 milliseconds
        Processing time (w/o IO): 30 milliseconds
        Processing time (w/o IO): 30 milliseconds
        Processing time (w/o IO): 30 milliseconds
        Processing time (w/o IO): 30 milliseconds
        Processing time (w/o IO): 30 milliseconds
        Processing time (w/o IO): 30 milliseconds
        Processing time (w/o IO): 29 milliseconds
        Processing time (w/o IO): 30 milliseconds
        Processing time (w/o IO): 30 milliseconds
        Processing time (w/o IO): 30 milliseconds
        Processing time (w/o IO): 30 milliseconds
        Processing time (w/o IO): 30 milliseconds
          Time (mean ± σ):     990.7 ms ±  12.9 ms    [User: 883.0 ms, System: 211.2 ms]
          Range (min … max):   973.7 ms … 1012.2 ms    10 runs

Julia:

        Benchmark 1: julia --project=Related -e "using Related; main()"
        Processing time (w/o IO): 367 milliseconds
        Processing time (w/o IO): 389 milliseconds
        Processing time (w/o IO): 366 milliseconds
          Time (mean ± σ):      1.487 s ±  0.008 s    [User: 1.334 s, System: 0.255 s]
          Range (min … max):    1.481 s …  1.493 s    2 runs

Julia:

        Benchmark 1: julia --project=Related -e "using Related; main()"
        Processing time (w/o IO): 3157 milliseconds
        Processing time (w/o IO): 3158 milliseconds
        Processing time (w/o IO): 3156 milliseconds
          Time (mean ± σ):      4.585 s ±  0.013 s    [User: 4.353 s, System: 0.327 s]
          Range (min … max):    4.576 s …  4.594 s    2 runs

> math '30+389+3158'
3577

would rought put Julia somewhere around 5th? not bad I guess

> math '30+389+3158'
3577

would roughly put Julia somewhere around 5th? not bad I guess

@Moelf , I am not sure this is optimal.
The original StaticArrays implementation constantly achieved around 23ms for 5K posts (which is actually the value of first place right now - which is Zig).
37.15 ms is quite departing from that result (but I cannot say why - I just took a peek at the code).
I don't have too much time now - but if you have and are willing, you might try to reconcile the past top result from here with the current precompilation setup.

I don't want to add the dynamic dispatch to different type length, because other implementations don't do that I believe

Strangely it seems to me that the Julia version didn't even improve after merging this, the sum of the times is still nearly 5 seconds


Test misalign-jal-01.S generates compressed instruction for platforms that support only RV32I

Test is compiled using riscv32-unknown-elf-gcc -march=rv32i_zicsr -mabi=ilp32 ... and the generated instructions are:
...
0000                	.short	0x0000
00354513          	xor	a0,a0,3
00000013          	nop
0001                	.insn	2, 0x0001
00000117          	auipc	sp,0x0
02610113          	add	sp,sp,38
...

Toolchain is compiled without compressed instruction support.
This seems to be related with the alignment in LA macro:
/**** fixed length LA macro; alignment and rvc/norvc unknown before execution ****/
#define LA(reg,val)     ;\

This will fail this tests because a second exception will happen that is illegal instruction for RV32I platforms!

hmm, I'm not sure I understand where the .short, and the .insn  came from.
The code turns on compressed ops (using .push, ;option rvc) in case the
model implements C extension and is currently at odd alignment),
then aligns the code to a 2^(2..4)B aligned address (which should insert
noops),
then forces norvc, implements the la reg, val pseudo op, re-aligns to a
2^2..4) boundary, and restores the original rvc/norvc status.
This kind of looks like the toolchain is inserting zeroes instead of noops
(which is correct if it is data... but not necessarily for *inline" code.
I'm not sure how to cause .align to force noops.
On Mon, Mar 11, 2024 at 1:59 PM ArbnorSh @.***> wrote:

Test is compiled using riscv32-unknown-elf-gcc -march=rv32i_zicsr
-mabi=ilp32 ... and the generated instructions are:
...
0000                	.short	0x0000
00354513          	xor	a0,a0,3
00000013          	nop
0001                	.insn	2, 0x0001
00000117          	auipc	sp,0x0
02610113          	add	sp,sp,38
...
Toolchain is compiled without compressed instruction support.
This seems to be related with the alignment in LA macro:
/**** fixed length LA macro; alignment and rvc/norvc unknown before execution ****/
#define LA(reg,val)     ;\
This will fail this tests because a second exception will happen that is
illegal instruction for RV32I platforms!
—
Reply to this email directly, view it on GitHub
https://github.com/riscv-non-isa/riscv-arch-test/issues/442, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHPXVJULQJ6S3D3R6RZG2Z3YXYLMRAVCNFSM6AAAAABERBGLECVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE4DAMRSGE4DGNQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

OK, re-reading this: this test is deliberately causing a misaligned trap,
and that branches to the trap handler.
The trap handler is compiled to be all aligned, so that shouldn't happen in
the handler.
This must then be happening in the (only) test inside the TEST_JAL_OP.
The TEST_JAL_OP macro (which is complicated with tons of conditionals and
no comments)
so it will take quite a bit of work to figure out what it is doing and
where it is going wrong.
Can you post (or send me ) a larger snippet of the compiled code, starting
at label inst_0: through rvtest_code_end::
On Mon, Mar 11, 2024 at 5:50 PM Allen Baum @.***>
wrote:

hmm, I'm not sure I understand where the .short, and the .insn  came from.
The code turns on compressed ops (using .push, ;option rvc) in case the
model implements C extension and is currently at odd alignment),
then aligns the code to a 2^(2..4)B aligned address (which should insert
noops),
then forces norvc, implements the la reg, val pseudo op, re-aligns to a
2^2..4) boundary, and restores the original rvc/norvc status.
This kind of looks like the toolchain is inserting zeroes instead of noops
(which is correct if it is data... but not necessarily for *inline" code.
I'm not sure how to cause .align to force noops.
On Mon, Mar 11, 2024 at 1:59 PM ArbnorSh @.***> wrote:

Test is compiled using riscv32-unknown-elf-gcc -march=rv32i_zicsr
-mabi=ilp32 ... and the generated instructions are:
...
0000                	.short	0x0000
00354513          	xor	a0,a0,3
00000013          	nop
0001                	.insn	2, 0x0001
00000117          	auipc	sp,0x0
02610113          	add	sp,sp,38
...
Toolchain is compiled without compressed instruction support.
This seems to be related with the alignment in LA macro:
/**** fixed length LA macro; alignment and rvc/norvc unknown before execution ****/
#define LA(reg,val)     ;\
This will fail this tests because a second exception will happen that is
illegal instruction for RV32I platforms!
—
Reply to this email directly, view it on GitHub
https://github.com/riscv-non-isa/riscv-arch-test/issues/442, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHPXVJULQJ6S3D3R6RZG2Z3YXYLMRAVCNFSM6AAAAABERBGLECVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE4DAMRSGE4DGNQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Attaching the requested part as a file!
inst_0.disass.txt

@ArbnorSh This test is deliberately causing a misaligned-fetch trap by jumping to a misaligned-address. The disass is showing you misaligned address where pc will jump in order to check that feature. Here is the complete disassembly and Sail_log file through which you can understand the behavior of the test. Let me breakdown the behavior of the test for your better understanding.
address 0x8000026c instr jal	a0,80000476 is causing pc to jump to a misaligned address.
If misaligned access is enabled and DUT/Sail has compressed instruction support, then DUT/Sail will execute c.nop instruction and no exception will be thrown,
but in your case, when misaligned-access is disabled and/or compressed instruction support does not exist, then it will cause misaligned-fetch exception and will call the trap handler. You will see the following prompt in Sail log file in your case,
mem[X,0x8000026C] -> 0x056F
mem[X,0x8000026E] -> 0x20A0
[107] [M]: 0x8000026C (0x20A0056F) jal a0, 522
trapping from M to M to handle misaligned-fetch
handling exc#0x00 at priv M with tval 0x80000476
CSR mstatus <- 0x00001800
For reference => Line 645-650 (in the given log file)

Here mtval is getting 0x80000476 value showing you the misaligned address. This address is same as where you have seen .short 0x0000 or .2byte 0x0 (both are the same things).
Upon returning from the trap (or no trap in case-1), the program will keep executing instructions line by line.

Now, the address 0x80000476 is also in code region, so DUT/Sail will eventually reach to this address to execute this instruction. Over here, it will fetch a compressed instruction, if compressed instruction is supported by DUT/Sail, it will execute that c.nop instruction, otherwise (when compressed support is not there => Your case) it will cause illegal-instruction exception and once again calls the trap handler. For your case, you will see the following prompt in Sail log.
mem[X,0x80000474] -> 0x0000
[453] [M]: 0x80000474 (0x0000) c.illegal 0
trapping from M to M to handle illegal-instruction
handling exc#0x02 at priv M with tval 0x00000000
CSR mstatus <- 0x00001800
For reference => Line 2636-2641 in the given log file

In either case (trap or no trap), the program will complete its execution without any trouble, so you should be getting an exit from this program and this test will be passing on both DUT and Sail in all possible cases (with/without misaligned, with/without compressed support), so dont worry if you are seeing any compressed instruction in your disass, it is there for deliberate reason.

Yes, this test is deliberately inserting a compressed noop ahead of a 32bit
instruction to force it to be misaligned, so it can see that the core
handles the misaligned up correctly.
What still confuses me is that you say there is a "second exception", and I
don't know which exception that would be.
Normally, traps return to the instruction recorded in EPC, but the trap
handler will always return to the next aligned 8byte boundary that follows
the instruction at the EPC address.
This guarantees that it won't get into an infinite trap loop, and also
gives code the opportunity to branch to special handling code if it didn't
trap
(because that will be skipped over if there is a trap). Normally, tests
will add 2 noops after test to ensure the return won't skip over the any
non-noops upon return.
This test has several different misaligned branches and jumps, both forward
and backward. This is expected.
So, could you be a little more clear about what error you're seeing?

Is it refusing to compile?
Are you not expecting multiple traps (there shouldn't be two
back-to-back though)
Is it not matching what Sail is doing?
Or did it actually branch to an unaligned address and try to execute the
compressed noop or illegal zero opcode (which it can't because it doesn't
have C-extension support).
That might be a bug in the DUT rather than in the test.

On Thu, Mar 14, 2024 at 10:23 PM Umer Shahid @.***>
wrote:

@ArbnorSh https://github.com/ArbnorSh This test is deliberately causing
a misaligned-fetch trap by jumping to a misaligned-address. The disass is
showing you misaligned address where pc will jump in order to check that
feature. Here is the complete disassembly
https://github.com/riscv-non-isa/riscv-arch-test/files/14611013/ref_disass.txt
and Sail_log file
https://github.com/riscv-non-isa/riscv-arch-test/files/14611002/misalign-jal-01.log
through which you can understand the behavior of the test. Let me breakdown
the behavior of the test for your better understanding.
address 0x8000026c instr jal a0,80000476 is causing pc to jump to a
misaligned address.
If misaligned access is enabled and DUT/Sail has compressed instruction
support, then DUT/Sail will execute c.nop instruction and no exception
will be thrown,
but in your case, when misaligned-access is disabled and/or compressed
instruction support does not exist, then it will cause misaligned-fetch
exception and will call the trap handler. You will see the following prompt
in Sail log file in your case,
mem[X,0x8000026C] -> 0x056F
mem[X,0x8000026E] -> 0x20A0
[107] [M]: 0x8000026C (0x20A0056F) jal a0, 522
trapping from M to M to handle misaligned-fetch
handling exc#0x00 at priv M with tval 0x80000476
CSR mstatus <- 0x00001800
For reference => Line 645-650 (in the given log file)
Here mtval is getting 0x80000476 value showing you the misaligned
address. This address is same as where you have seen .short 0x0000 or .2byte
0x0 (both are the same things).
Upon returning from the trap (or no trap in case-1), the program will keep
executing instructions line by line.
Now, the address 0x80000476 is also in code region, so DUT/Sail will
eventually reach to this address to execute this instruction. Over here, it
will fetch a compressed instruction, if compressed instruction is supported
by DUT/Sail, it will execute that c.nop instruction, otherwise (when
compressed support is not there => Your case) it will cause
illegal-instruction exception and once again calls the trap handler. For
your case, you will see the following prompt in Sail log.
mem[X,0x80000474] -> 0x0000
[453] [M]: 0x80000474 (0x0000) c.illegal 0
trapping from M to M to handle illegal-instruction
handling exc#0x02 at priv M with tval 0x00000000
CSR mstatus <- 0x00001800
For reference => Line 2636-2641 in the given log file
In either case (trap or no trap), the program will complete its execution
without any trouble, so you should be getting an exit from this program and
this test will be passing on both DUT and Sail in all possible cases
(with/without misaligned, with/without compressed support), so dont worry
if you are seeing any compressed instruction in your disass, it is there
for deliberate reason.
—
Reply to this email directly, view it on GitHub
https://github.com/riscv-non-isa/riscv-arch-test/issues/442#issuecomment-1998958532,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHPXVJQYPOUPVHKSUSK3DNTYYKAWNAVCNFSM6AAAAABERBGLECVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJYHE2TQNJTGI
.
You are receiving this because you commented.Message ID:
@.***>

@UmerShahidengr
My understanding was that the testcase has one objective to test the misaligned fetch implementation on RV32I cores, now I understand that a core that implements misaligned fetch exception but it does not implement illegal instruction exception should fail this test. Thank you for the detailed explanation! Now it is clear on the generated compressed instructions.

@allenjbaum
Signatures were different from DUT and Sail. Will close this as resolved! Thank you for your help!


fix: ExpressionVisitor.visit(AllTableColumns) method isn't being called.

This fixes a regression introduced in change #4b4ae04f44ff18b669d0f30d637e5b7c64b085e4: feat: BigQuery Except(..) Replace(..) syntax
https://github.com/JSQLParser/JSqlParser/commit/4b4ae04f44ff18b669d0f30d637e5b7c64b085e4

This fixes a regression introduced in change #4b4ae04f44ff18b669d0f30d637e5b7c64b085e4: feat: BigQuery Except(..) Replace(..) syntax

Upps, that is on me!
Thanks for bailing me out and for contributing!


Automatically call stop() on ssb.close()

I'm running ssb.close() to close my peer, but when this plugin is loaded the process stays open. My current workaround is ssb.lan.stop(); ssb.close() but I think it might be nice to hook ssb.close() similar to other plugins (e.g. SSB-Serve-Blobs (https://github.com/ssbc/ssb-serve-blobs/commit/ff1392871ad4a1d7c46915dd35f545b408ff659a)).

Agreed!


fix: Support referencing complex elements in conditional expressions.

Fixes: #1604

@igarashitm - No real React UI changes.  Please review all files.

Is it populating named inputFieldGroup with having children like this? runtime depends on those field structure to process expression.
https://github.com/atlasmap/atlasmap/blob/master/lib/itests/core/src/test/resources/mappings/atlasmapping-expression-filter-select-java.json#L19-L33

Is it populating named inputFieldGroup with having children like this? runtime depends on those field structure to process expression.
https://github.com/atlasmap/atlasmap/blob/master/lib/itests/core/src/test/resources/mappings/atlasmapping-expression-filter-select-java.json#L19-L33

Yep - you're right - inputField need to be inputFieldGroup.  I'll work on that and resubmit - pls take a quick look at the rest.

will resubmit - working on inputField -> inputFieldGroup.


Storybook > Components>User Basics >Placeholder is not proper and Add phone no validation

Describe the bug
Components>User Basics >Placeholder is not proper and Add phone no validation.
To Reproduce
Steps to reproduce the behavior:
1.Go to 'Storybook'
2.Click on 'Components'
3.Click on 'User Basics'
4.Click on 'Default'
Screenshots

Desktop (please complete the following information):
-OS: [Windows 10 Enterprise]
-Browser [ edge]
-Version [ 125.0.2535.92]
Smartphone (please complete the following information):NA

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Completed -

Version 1.1.7


Request about ndarray take() function

In my code, a function which I commonly call does the following operation. Assume that we have a tensor A of shape (d1,d2,...,dn) and tensor B of shape (D1,d2,...,dn) where D1 > d1. So, the axis0 indices of the tensor A is mapped to a subset of axis0 indices of B. Assume that this mapping is stored into a dict, f. What I want to do this is to pick A[i] and add to B[f[i]], for each axis0 index of A, basically:
for i in (0, A.shape[0]):      B[f[i]] += A[i]
This operation is inefficient and constitutes a bottleneck. Theoretically, I can solve this with the take operation, where I can get the subset of B, as determined by the mapping f and add A, as in the example:
B = mx.ndarray.zeros(shape=(10, 4), ctx=mx.gpu(0))
A = mx.ndarray.ones(shape=(3, 4), ctx=mx.gpu(0))
# f = { 0:3, 1:4, 2:6}
indices = mx.ndarray.array(source_array=[3, 4, 6], ctx=mx.gpu(0))
b_subset = mx.ndarray.take(B, indices)
b_subset[:] += A[:]
The problem here is, b_subset is a new buffer, it does not refer to the original B and when I add A on it, this new buffer is modified, not the original one.
Is it possible to add at least a flag to ndarray's take, so that it enables the modification of the original array to handle the cases like above?

The difficulty is creating a view of the take result from an ndarray in which case the memory is not contiguous. I don't think mxnet currently supports that since most of the operators relies on contiguous memory allocation for tensors. Have you timed your first solution in your application? Is it really a bottleneck?
@piiswrong

Actually, yes. Basically I use a heuristic in which I group the samples to be added from A into contiguous locations at B, and for addition then I use slicing for example like
            self.sourceArgument.gradientArray[source_tuple[0]:source_tuple[1] + 1] += \                 self.destinationArgument.gradientArray[destination_tuple[0]:destination_tuple[1] + 1]
and this clearly improves the performace compared to one by one adding.

This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this is still an active issue. Thanks!


Updated routes.py file for descrip. & create promo

Updated the routes.py file to include our name and description. Also, updated the file within the Get Index function to inlcude read promotion.

reviewed!


Introduce terraform state bucket creation in furyctl create cluster

feat: autocreate the terraform state bucket before starting the apply action

What happens if the bucket already exists?

@nutellinoit nothing, it only creates it if it's not there


ddnslab.tech domain is parked, so not working

I've discovered this project via https://workers.cloudflare.com/built-with/projects/ddnslab
However, the domain looks parked, as can be seen in the following screenshoot:

I'm just asking out of curiosity, has it been discontinued?

Try https://jwaladiamonds.github.io/ddnslab.tech/


add travis-ci

Add travis-ci so we can watch our PRs pass or fail across multiple Python environments.

Started troubleshooting arangodb as a dockerized service:
https://github.com/brennv/pyArango/tree/add-travis-ci

travis is working but it looks like we'll need to merge @dotthebart's https://github.com/tariqdaouda/pyArango/pull/22 before even the python2 tests will pass

Is it working now? The current version on python3 should run on python 3.4 and above. It's not compatible yet with python 2.7.

Its working :) https://travis-ci.org/tariqdaouda/pyArango/pull_requests


Apply normalised line height to form element text

line-height: 1; feels right as its set to the height of the text but we need to think about text in <label>s wrapping.

https://github.com/campaignmonitor/shell/pull/58


Update README.md

Changes:

Fix typo GaiasX
Change Except to Besides, GaiaX did provide an SDK.
Some grammar refinement

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


cmd/go/internal/web: test script failed when have ipv6 address but no ipv4 address

What version of Go are you using (go version)?

$ go version
go version go1.16.5 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
TESTGONETWORK="panic"
(when run test in cmd\go\script_test.go)

What did you do?

Hello, I was running 'all.bash' on our pipeline and there were some errors occurred. The error message is:
--- FAIL: TestScript/mod_get_ambiguous_pkg (0.16s)
    script_test.go:213:
        # Both example.net/ambiguous v0.1.0 and example.net/ambiguous/pkg v0.1.0 exist.
        # 'go mod tidy' would arbitrarily choose the one with the longer path,
        # but 'go mod tidy' also arbitrarily chooses the latest version. (0.000s)
        # From a clean slate, 'go get' currently does the same thing as 'go mod tidy':
        # it resolves the package from the module with the longest matching prefix. (0.163s)
        > go get -d<EMAIL_ADDRESS>        [stderr]
        panic: use of network<EMAIL_ADDRESS>
I found that the virtual machine doesn't have ipv4 address but have ipv6 address. It seems that this situation has not been considered.
In file cmd\go\internal\web\http.go:
if os.Getenv("TESTGONETWORK") == "panic" && !strings.HasPrefix(url.Host, "<IP_ADDRESS>") && !strings.HasPrefix(url.Host, "<IP_ADDRESS>") {
	panic("use of network: " + url.String())
}

May I change it to:
if os.Getenv("TESTGONETWORK") == "panic" && !strings.HasPrefix(url.Host, "<IP_ADDRESS>") && !strings.HasPrefix(url.Host, "<IP_ADDRESS>") &&
	!strings.HasPrefix(url.Host, "[::1]") && !strings.HasPrefix(url.Host, "[::]") {
	panic("use of network: " + url.String())
}

What did you expect to see?
expect all test passed
What did you see instead?
--- FAIL: TestScript/mod_get_ambiguous_pkg (0.16s)
    script_test.go:213:
        # Both example.net/ambiguous v0.1.0 and example.net/ambiguous/pkg v0.1.0 exist.
        # 'go mod tidy' would arbitrarily choose the one with the longer path,
        # but 'go mod tidy' also arbitrarily chooses the latest version. (0.000s)
        # From a clean slate, 'go get' currently does the same thing as 'go mod tidy':
        # it resolves the package from the module with the longest matching prefix. (0.163s)
        > go get -d<EMAIL_ADDRESS>        [stderr]
        panic: use of network<EMAIL_ADDRESS>

Rather than using hardcoded addresses, let's use net.IP.IsLoopback.


fix: Tizen video error fixed by checking the extended MIME type

We have reproduced an issue in some tizen tv's, and we think it's the cause of #4634 and possibly #4503
It happens with some streams that have 3 variants, 2 of them are supported on these TVs, but the one with higher resolution isn't. This variant uses this video: video/mp4;codecs="avc1.64002a";framerate="25";bitrate="6000000";width="1980";height="1080" and in this TV the higher resolution that is supported is 1920x1080
At filterManifestByMediaCapabilities function from stream_utils.js we were checking Capabilities.isTypeSupported(fullType) , where this "fullType" only includes the mime type and the codecs (video/mp4; codecs="avc1.64002a" for example). This is currently returning true, so the player believes that this variant is supported, but then it fails when it tries to play it.
However, there was an older implementation that used shaka.media.MediaSourceEngine.isStreamSupported() function which internally checks Capabilities.isTypeSupported(extendedMimeType), where "extendedMimeType" includes also the width, height, bitrate and framerate (for example, video/mp4;codecs="avc1.64002a";framerate="25";bitrate="6000000";width="1980";height="1080"), which in this specific case returns false because in this case the 1980 width is not supported (it should be 1920), the variant is filtered out, and the playback is started with the other 2 variants.
In this proposed change I'm basically changing isTypeSupported(fullType) by isTypeSupported(extendedMimeType), keeping in mind that the codecs and mimeType could have been modified in the call to getFullOrConvertedType and we must keep using those modified values.
In this change I haven't modified the mutiplexed streams case (the case from the if (video.codecs.includes(',')) at filterManifestByMediaCapabilities), because the getExtendedType function gets the whole variant as parameter, I don't have any good way of testing the case, and also I think that the audio tracks are less likely to be affected by this issue, so it's probably not that important.

@FernandoGarciaDiez Please, sign the CLA. Thanks!

I did already, but I think that as it is a corporate CLA from a new corporation, it can take some days until it's processed (https://opensource.google/documentation/reference/cla#cla-accepted)

I can see the logic of this, however, resolution constraints are meant to be checked through MediaCapabilities.
Is MediaCapabilities implemented on this model of Tizen, or is our polyfill being used instead?
If it's the polyfill, let's make this change at that level (building the extended MIME type there).
If it's a broken native implementation, let's still make the change at the level of the polyfill, but let's force its use on Tizen.

I see what you mean, yes, the polyfill is installed and I see how that's where the fix should be, as it is not taking into account the resolution. Another possible issue is that I am not 100% sure that even if the polyfill is installed it is working as expected, it looks as if it was never called, but as I say, I still need to investigate it a bit more

As I'm working on this, I have realized that this issue is pretty much the same as #4726 which was done exclusively for the Cast platform. @JulianDomingo since you worked on that issue some months ago, why was is so tailored for the video component just in the cast platform? Do you think there would be any problem in checking the extended mime type parameters in all platforms and both for video and audio? Or would it be better to add some similar code only for tizen video?

why was is so tailored for the video component just in the cast platform?

In general, these "extended MIME types" are non-standard, and the parameters are ignored by most platforms.  Cast is an exception, though I hope to see the platform embrace the web-standard way through the MediaCapabilities API.

Only for Tizen, because there are some caches with https://github.com/shaka-project/shaka-player/blob/main/lib/polyfill/media_capabilities.js#L124, and the extended mime type produces many new possibilities

Also, this.  :grin:

I'll run the GitHub tests and the lab.


library paths for Lambda packaging is hardcoded

Due to hard coded library paths, Lambda cannot be executed.
No module named yaml: ImportError Traceback (most recent call last): File "/var/task/custodian_policy.py", line 4, in run return handler.dispatch_event(event, context) File
...
File "/var/task/skew/config.py", line 18, in <module>
import yaml
ImportError: No module named yaml

via CloudWatch Logs.
When creating Lambda packages, c7n/mu.py only packages libraries under BASEPATH/lib/python2.7/site-packages/ .
https://github.com/capitalone/cloud-custodian/blob/master/c7n/mu.py#L114
But packages can be installed under different paths.
For example, on Amazon Linux, after creating virtualenv and using that pip, run $ pip install XXX will  install packages under VENVPATH/lib64/python2.7/site-packages/(Note lib and lib64 difference.). That's why yaml libraries are not packaged.
There should be more clever way to search for Python lib paths.

applicable is sys.path os os.environ['PYTHONPATH'].split(":") in a loop around the same line.

fwiw this has worked on both rhel7.2 and ubuntu14.04 and 16.04, curious what the delta is on this environment besides amz linux, needs some a bit more investigation, there's some .pth files here that we may have to parse.


Implement urlincontent functionality

This adds the functionality to replace "href" and "src" attributes in
"e-content" with the absolute urls.

The problem is that I don't know how to adjust the code, so it adds the 4 spaces at the end of the htmlBody to make the test pass. Removing the space in frontend the last  will make the test pass.

This also appears for the self-closing tag of the img element, where the test expects a space before />. The go html renderer doesn't add this space.

actually, the more I look at this, the weirder that whitespace issue looks. I'll dig into that a little more first.

I moved the "/>" normalization into testsuite_test.go, rebased onto master, squashed the commits, and merged it all in.  Thanks for this!

I see that the change doesn't follow the spec. It only passes the test suite it seems.
The point of the expansion would be that the use case for the parsed html is to use it in other websites, as a reply, or in Microsub readers. It will be difficult (impossible?) to resolve the references as a user of the library.
In other steps hrefs are expanded and it makes sense for e-* properties as well. In a way the meaning of the relative urls in a webpage are the absolute urls. If you move the content to another context, the links should point to the same thing.
Perhaps the spec should be changed?

Perhaps the spec should be changed?

Yep, that was exactly the conclusion we came to in that chat.  I'll file a bug proposing that shortly.


playlist urls are not really working correctly.

The url:
https://music.youtube.com/playlist/?list=PLhlYlDBoaODZh7oYlcARKFvFiocV5DOeZ
does NOT Work
but
https://music.youtube.com/playlist?list=PLhlYlDBoaODZh7oYlcARKFvFiocV5DOeZ
DOES
what is different?
the / after playlist -> it's still valid, as if you visit the page it's valid.
maybe there are more cases like that
this should be working due to url formatting
like if you enter it with playlist/?list= the browser formats it to playlist?list=
should be easy to fix.

Does YouTube gives you this wrong URL?


Suggest wiping your environment to folks with bad build outcomes.

This is a basic start to trying to be more helpful to users.

I like the idea of giving some tips/advice on the "why this build has failed?". I would like to start testing something simple like this, how it goes, and maybe later try to make it "more intelligent" but I wouldn't spend more time on the intelligence at the moment but maybe more tips.

Aye. I think we could build some nicer styles, and have easier ways for users to contribute them (eg. a YAML file of possible errors). But this is a nice start.


How chek path exist by not throw error

How to check nill of specific paht which throw to show error?
such I want to check
N.Path('o.k2')
or
N['o']['k2']
= nill ?
but now if path not exist McJSON show error automatically, but I don't want that.
I just want to be able to check for myself whether it's there or not. Even if there is, I will proceed with one thing. But if you don't have it, don't do another thing, for example.
if N['o']['k2']  = nill then
 // do something
else
 // do something else ;

or
if N['o']['k2'].ItemType  = jitNone then
 // do something
else
 // do something else ;

Note : for JsonDataObjects can do this
if N['o']['k2'].Typ = jdtNone then
 // do something
else
 // do something else ;

Please guide me for this
Thank you

Hi,
I need to improve McJSON to meet this demand. Thanks for getting in touch, I'll work on it.

Hi, just committed a new version in order to manage nonexistent paths.
Could you try it and report here?

Hi,
I'am not sure that should which method to check the existence of the path.
So I test by
      if N.Path('o.k2') = nil then
       // do something
       ShowMessage('nil')
      else
       // do something else ;
       ShowMessage('ok');

but It still show error message that : Object reference is nil: has key "k2"
It seems that Method : HasKey that is called within Method : Path is displaying this error.
or
Did I do something wrong? Please advise.
Thank you

Hi,
Today's commit has a new Test20 that tests this and it is working here with Lazarus/FPC. Could you check with Delphi? I'm not in my office today:
N.AsJSON := '{"o": {"k1":"v1", "k2":"v2"}}';
// get second object using path of keys.
// ...
M := N.Path('o.k3');
Result := Result and (M = nil);

Thank you for your quick reply.
But if it's convenient for you or it's during your working hours, please help answer my questions here.
      N := TMcJsonItem.Create();
      N.AsJSON := '{"o": {"k1":"v1", "k2":"v2"}}';
      M := N.Path('o.k3'); //-- One
//      M := N.Path('p');  //-- Two
//      M := N.Path('ppp'); //-- Three
//      M := N.Path('ppp.k3'); //-- Four
      if M = nil then
       // do something
       ShowMessage('nil')
      else
       // do something else ;
       ShowMessage('ok');

Result for above statement :

One : show message 'nil' ==> Yes
Two : show message 'ok' ==> No, It should show message 'nil'
Three : show message 'nil' ==> Yes
Four : show error message : Object reference is nil: has key "k3" ==> No, It should show message 'nil'

I think case "Two" and "Four" incorrect
Thank

Hi,
You're right.
I'll work on it.

Please, test last commit and report here.

@hydrobyte Thank very much, For now I think it's great to be able to check the existence of a path via method Path. 👍
And if you have another method that gives the same result, please recommend me.

@baxing, thanks for testing and reporting here.
I think Path() returning nil is good enough to test invalid object paths.


19 delta log should use relative paths instead of absolute

This PR makes the BlockWriter to use relative names for the data files.
In addition it also changes QbeastSnapshotTest, because it was failing on my machine due to rounding errors while merging the weight estimations of the partitions.

I just used mechnically scalafmtAll in sbt console. And I like this style, i.e code first then format with the build tool. We also used the same approach in qbeast-keeper.


Add statistics panel

Closes #47
Data is processed using a Google Apps Script on every form submission.
Here is the closed panel:

Here is the open panel (black background):

Here is the open panel (gray background):

Thoughts?

This is dope! Some thoughts:

Is it actually too resource intensive to compute these stats in JS rather than in a Google Apps script? I would prefer tracking code within the GH repository to the maximum extent possible
The arrow on the right doesn't draw the eye. Perhaps we could move the trigger to be around the options panel in the top left, and maybe have a grey background around the button so it's easier to see against complex backgrounds?

Is it actually too resource intensive to compute these stats in JS rather than in a Google Apps script? I would prefer tracking code within the GH repository to the maximum extent possible

It's not the resource intensity that is the problem, but the computation to find the number of states/countries involves a geocode that needs to be run upon every submission to the forms. The Google Apps Script has an onFormSubmit trigger that serves this purpose nicely. I can add the code for it in this repo, but it wouldn't really do much and it'd be a bit of a duplication. I tried to put it in the Google Drive but for some reason Google Apps Scripts don't appear...

The arrow on the right doesn't draw the eye. Perhaps we could move the trigger to be around the options panel in the top left, and maybe have a grey background around the button so it's easier to see against complex backgrounds?

I'll give it a try on both the left and the right, but it certainly does need a gray background.

I would prefer that we at least copy the apps script code into this repo in case we were to lose it somehow.

I think Erik already covered the actual coding stuff which I'm far less qualified to give advice on anyway but as this page gets more crowded, I had a couple of design thoughts:
To start, I'd think the order of importance for top layer items goes (and if you guys disagree the same sorta thoughts apply regardless):
year selection > destination submission > stats tab > full screen > zoom
When you look at a page, most western readers typically go top left to top right, to the bottom left, to bottom right

(as you can see I am excellent at design)
The top left and bottom right normally has the most important stuff, and you'll see this everywhere --like a GitHub comment box!

I think the current layout with all those items could definitely be improved, rearranged to put the more important stuff in places that catch the eye more immediately, which would also help to see that arrow. Low quality paint job below -- if this doesn't make any sense, please disregard haha.

I like the layout change, but I think leaving Google's buttons where they are is important for user recognition. So, I left the stats panel where it is on desktop because I changed the css to look like a Google button.
However, on mobile I think we need more space for people to see the map, so I did choose to move the panel below the options panel as you showed.

Ok turns out it looks better on desktop too. Final positioning is below options panel on all devices.

I like the layout change, but I think leaving Google's buttons where they are is important for user recognition. So, I left the stats panel where it is on desktop since I already changed the css to make it look like a Google button.

Yeah if the Google menu items take deference that's totally fair - idk how to put my green checkmark back but pretend it's below

I don't think it's very feasible to move Google's buttons, but otherwise I like that take.


Maths CPP

Implementation of the following in C++:

[x] Find volumes of various shapes (cube, cuboid, cone, right circular cone, prism, pyramid, sphere, circular cylinder)

[x]  Zellers Congruence Algorithm

[x] Ugly numbers

[x] Sum of n terms in a Geometric Progression

[x] Sum of Digits of a number

[x] Sum of n terms in an Arithmetic progression

[x] Simpson Rule

[x] Segmented Sieve

[x] Roots for any Quadratic Equation

[x] QR-decomposition of the matrix A using Householder reflection

[x] Pythagoras Implementation

[x] Prime Numbers up to N numbers

[x] Area under the curve using the trapezoidal rule

[x] Newton-Raphson method

[x] Lucas Sequence Using Recursion

[x] Least common multiple of two numbers

[x] Peterson Number

[x] Jaccard similarity

[x] Greatest Common Divisor

[x] Extended Euclidean Algorithm

[x] Bailey-Borwein-Plouffe (BBP) Implementation

Needs to be merged in "cpp/maths"

Can i work on this?

I would like to work in this. Please assign some of the above problems.

Hey @LakhanMardi and @mustang519, please be specific. Which problems would you like to work on?

kindly assign Sum of n terms in an Arithmetic progression to me

Assigning um of n terms in an Arithmetic progression to @pranav-khandka.

Please kindly assign Sum of digits of a number , volumes of various shapes , LCM of 2 numbers to me.

Hi, can I work on Ugly numbers, Sum of n terms of a gemetric progression, Segmented sieve, Roots for any quadratic equation, Lucas Sequence using recursion and Peterson number

Hey!
Please assign me Greatest common divisor and Extended Euclidean algorithm.

I would like to work on Peterson number and Pythagoras implementation. Could you please assign it to me?

I have solved some of these, please checkout my Pull Request

Assigning Sum of digits of a number, volumes of various shapes, and LCM of 2 numbers to @mustang519.
Assigning  Ugly numbers, Sum of n terms of a geometric progression, Segmented sieve, Roots for any quadratic equation, Lucas Sequence using recursion to @itish-agarwal.
Assigning Greatest common divisor and Extended Euclidean algorithm to @Aarushi11H.
Assigning Peterson number and Pythagoras implementation to @anushkasinghal11.
Hey @preyam2002, you were not assigned to any of these issues. Please ask for the assignment first. Once assigned, then move to solve it. Thankyou.

Can I work on Zellers Congruence Algorithm?

Assigning Zellers Congruence Algorithm to @v-malwade.

I have added a pull request for the Zellers Congruence Algorithm; please check it.

Can you assign the prime numbers one and the newton-raphson method to me?

@geekquad please assign the problem prime numbers upto N numbers to me.

Hey @v-malwade, PRs made will be reviewed. You don't have to comment for the same. Thanks.
Assigning Prime Numbers upto N and Newton-Raphson method to @preyam2002.
Sorry @130sayan, issues are first come first serve. Someone asked it before you. But we gave plenty more. Feel free to ask for them. :)

Hey!
I would like to work on Simpson Rule.
Which Simpson rule I have to implement Simpson 1/3 rule or Simpson 3/8 rule?

Hey!
I would like to work on the Zeller's Congruence Algo.

hey @geekquad ...I would like to work on  Area under the curve using the trapezoidal rule

Hey @Shagufta08, let's do both 😉. Assigning Simpson Rule to you.
Assigning Area under the curve using the trapezoidal rule to @BRJbibhu10.
Hey @ThePhilosopher37, assigning Bailey-Borwein-Plouffe (BBP) Implementation, Jaccard similarity, and QR-decomposition of the matrix A using Householder reflection to you. Do let me know if you want to opt-out of any.

Hey guys, @BRJbibhu10, @preyam2002, @itish-agarwal, @Shagufta08, @v-malwade, @Aarushi11H, @anushkasinghal11, @ThePhilosopher37, @mustang519, @pranav-khandka, there is some issue. You guys are getting unassigned. Please continue to work on the issue. It will be fixed soon. Apologies!

hi there, can I contribute to this issue?

hi could i work on the following issue ?


DOC: Updated type stubs for next release

Coverage remained the same at 88.195% when pulling c7060aa7adae35849c693434b95b50ac053a2070 on update-type-stubs into b46a80174b29588f49efd4e51fa2c273c2548265 on master.


[wip]Vibrate the device on game lost

https://twitter.com/hashseed/status/1127875233531289600

I'm going to add vibrate on-off preference in settings menu as well.

this partially addresses #453


Earn page

Earn page:

Background broken? Why is it grey now.
"Provide Liquiditys" typo
APY missing next to fees earned (see lucidchart)
Deposit:
Provide liquidity button is green in lucidchart
BTC Price Limit - should show current spot price as default (see lucidchart)
BTC address validation: see other PR. regex is not fully correct.
Provide liquidity button should be disabled until all input fields are valid

Withdraw:

Show amount in withdraw popup
Clear withdraw input from field after withdraw.
Withdraw button should be disabled until valid input is provided
Table and input fields shifted/look weird when in "half-screen" view. Please fix.
Deposit and Withdraw tab headings should not be blue (please use XOpts color palette)

Minor things:

Topbar menu not unselected when switching to the account page


Fix #75: Mac OS X doesn't recognize -o for uname

In tools/zim_info, conditionally call uname with different parameters depending on the operational system type.
Also fix uname calls with -o parameter in trace-zim and zim_issue files. A oneliner was preferred at trace-zim to avoid making the code more complex.

Or maybe uname -msr can be used for all OSs? Solaris also apparently does not support -o.

I've already made some of these changes in my local branch but have yet to push. I may cherry-pick some of the other changes you've made elsewhere, as I only fixed the issues in zim_info. The uname command I'm looking for on Mac is uname -mvsr, not just uname -msr. Regardless, I should have this issue fixed fairly soon. Thanks :+1:

Cool. You could consider using uname -mvsr for all OSs to avoid having additional logic for this.
There's a last place I didn't touch, in modules/debug/functions/trace-zim:88 that may fail in Solaris:
  print "your system, ${OSTYPE}, is an unknown system.
Please create an issue at:
  https://github.com/Eriner/zim/issues
Include the following output in your report:
  ${OSTYPE}  -  $(uname -mosr)"
  return 1

I've held off on merging this. I think using uname -a is the best catch-all that removes all of the additional logic and will always give us what we need. I originally only grabbed the info we needed to debug, but just printing everything reduces code complexity.
If you'd like to update this PR with that change and the change elsewhere, I'll merge it.

Cool. Done. https://github.com/Eriner/zim/pull/84 already got rid of one of the uname calls. The 3 remaining ones are being changed in the updated commit.

@ericbn Great, thanks. I'll merge this promptly. :+1:

Merged. Thanks again @ericbn!


feat request: store uploaded files in another column

Package
filament/filament
Package Version
v2
Laravel Version
v9
Livewire Version
v2
PHP Version
8
Bug description
to introduce a method to FileUpload component to store file's original name in another column
possible syntax
FileUpload::make('files')
->storeFileNameIn(columnName: 'original_filenames');

Steps to reproduce
No response
Relevant log output
No response

You want the original file name on it's own, without the path prefix etc?

yeah, probably the clientOriginalName even if the file is renamed by Livewire randomly.

I'm open for any as long as i can refer it back
eg:
# files
[
  "hhasuqwe123...zip",
   "image/kkoqwe....jpg
]

# original_filenames
[
   "logs.zip", # or
   "hhasuqwe123...zip" => "logs.zip",
     // ...
]

Hey @wychoong, could you please test #3407 and let me know if it works well for you. Please test single & multiple uploads, and check the file names get saved correctly in the database.

Sure! Will get back to you.

Hey @wychoong, could you please test #3407 and let me know if it works well for you. Please test single & multiple uploads, and check the file names get saved correctly in the database.

hey Dan, i tested this. I got a few issues with this

No property found for validation: [original_filename]
https://flareapp.io/share/pPvWWx17

dd in CanBeValidated
dd($this->getValidationRules());

^ array:2 [▼
  "data.compressed_file" => array:3 [▶]
  "original_filename" => array:1 [▶]
]

if I set ->storeFileNamesIn('data.original_filename'), above issue is gone, but my column still empty

this is my schema
public static function form(Form $form): Form
    {
        return $form
            ->schema([
                            FileUpload::make('compressed_file')
                                ->label('Compressed file')
                                ->disk('bulk_upload')
                                ->multiple()
                                ->required()
                                ->acceptedFileTypes(['application/zip', 'application/x-rar'])
                                ->hint('Max file size 250MB')
                                ->storeFileNamesIn('original_filename')
                                ->maxSize(1024 * 250),
            ]);
    }

model is setup correctly
var $fillable = [
       'files', 'original_filename',
    ];

    var $casts = [
        'files' => 'array',
        'original_filename' => 'array',
    ];

I dug further and did following changes
//BaseFileUpload.php
public function getFileNamesStatePath(bool $isAbsolute = true): ?string
    {
        # replicate from getStatePath
        $pathComponents = [];

        if ($isAbsolute && $containerStatePath = $this->getContainer()?->getStatePath()) {
            $pathComponents[] = $containerStatePath;
        }

        if (($statePath = $this->evaluate($this->fileNamesStatePath)) !== null) {
            $pathComponents[] = $statePath;
        }

        return implode('.', $pathComponents);
    }

    protected array | null $fileNames = null;

    public function storeFileName(string $file, string $fileName): void
    {
        $statePath = $this->getFileNamesStatePath();

        if (blank($statePath)) {
            return;
        }

        $this->evaluate(function (BaseFileUpload $component, Closure $get, Closure $set) use ($file, $fileName, $statePath) {
            if (!$component->isMultiple()) {
                $set($statePath, $fileName);
                $this->fileNames = $fileName; # added this

                return;
            }

            $fileNames = $get($statePath) ?? [];
            $fileNames[$file] = $fileName;

            $set($statePath, $fileNames);
            $this->fileNames = $fileNames; # added this
        });
    }

    # getter
    public function dehydrateFileNames(): array | null
    {
        return $this->fileNames;
    }

// HasState
public function dehydrateState(array &$state = []): array
    {
        foreach ($this->getComponents() as $component) {
            if ($component->isHidden()) {
                continue;
            }

            $componentStatePath = $component->getStatePath();

            if ($component->isDehydrated()) {
                if ($component->getStatePath(isAbsolute: false)) {
                    data_set($state, $componentStatePath, $component->dehydrateState());

                    # when `dehydrateState` called by `$this->form->state` it doesn't get the value from $set
                    if (
                        $component instanceof BaseFileUpload &&
                        ($componentFileNamesStatePath = $component->getFileNamesStatePath())
                    ) {
                        data_set($state, $componentFileNamesStatePath, $component->dehydrateFileNames());
                    }
                }

                foreach ($component->getChildComponentContainers() as $container) {
                    if ($container->isHidden()) {
                        continue;
                    }

                    $container->dehydrateState($state);
                }
            } else {
                Arr::forget($state, $componentStatePath);
            }
        }

        return $state;
    }

do you think it's necessary to test Edit as well? my current resource don't have that, if required I can setup another to test

Hey @wychoong, updated the PR. Can you check again please?

Hey @danharrin , i tried the updated PR, it's working now. I taken a look on your latest changes, as usual you make it looks so simple 😂


Email address field it not styled

Please provide us with the following information:

This issue is for a: (mark with an x)
- [X ] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Follow the steps on the readme to upload the template to blob storage. Update your custom policy to use the api.signuporsignin contentdefinition

Any log messages given by the failure

NA

Expected/desired behavior

The email address field should be styled the same as the password field.

OS and Version?

Windows 10

Versions

Mention any other details that might be useful
Screenshot of the issue below. Also you can see my login page at:
Link

Thanks! We'll be in touch soon.

I found the issue, there was a chunk of css missing to style the signInName field. I have put in a PR to add the code
Pull Request #8

I think issue is still there, it looks like PR from @andrewwiebe was not actually pulled, anyway the fix is just line 109 in file unified.css


migrate from consul to etcd

Hi,
Since consul got deprecated I need to migrate our consul kv to etcd.
I can easily read out all data from consul like this consul kv get -http-addr=$KV:8500 -recurse ceph-config
And I can easily write single k/v to etcd like this etcdctl --endpoints http://labb01:2379 mk ceph-config/ceph/osd/osd_mkfs_options_xfs '"-f -i size=2048"'
But how can I write multi line values like the ceph-config/ceph/monmap with the following content?
begin 644 -
M`P/8`0``Q5V4G`;31EZ\1G[P0)H[50,````#````!````'9M,#$`````````
M```"&H4*`SP9````````````````````````````````````````````````
M````````````````````````````````````````````````````````````
M````````````````````````````````````````````````````!````'9M
M,#(````````````"&H4*`SP7````````````````````````````````````
M````````````````````````````````````````````````````````````
M````````````````````````````````````````````````````````````
M````!````'9M,#,````````````"&H4*`SP5````````````````````````
M````````````````````````````````````````````````````````````
M````````````````````````````````````````````````````````````
<````````````````V2#P5CUKC@7'(/!6N/C,%```
`
end

https://github.com/ceph/ceph-docker/blob/master/ceph-releases/jewel/ubuntu/14.04/daemon/config.kv.etcd.sh#L40

Ok, cool
Didn't know that I could pipe in data via etcdctl
So now I did this
./consul kv get -http-addr=vm01:8500 -recurse ceph-config | grep : | cut -d: -f1 | while read line ; do ./consul kv get -http-addr=vm01:8500 $line | ./etcdctl --endpoints http://labb01:2379 set $line ; done

But all the values inserted have an extra empty line at the end.
I need to figure out how to get that removed.

@Raboo perhaps the extra line comes from consul kv get, if so you can trim this using a bash substitution.

./consul kv get -http-addr=vm01:8500 -recurse ceph-config | grep : | cut -d: -f1 | while IFS= read -r line ; do ./consul kv get -http-addr=vm01:8500 $line |head -c-1 | ./etcdctl --endpoints http://labb01:2379 set $line ; done


Cocoapods install needs full git path

target 'testPicker'
pod 'YPImagePicker',:git => 'https://github.com/Yummypets/YPImagePicker'
use_frameworks!

Thank you very much :)

Just needed some time to be deployed to Cocoapods

now works as expected


list index out of range

I got this error when running main_video.py
Traceback (most recent call last):
File "\License-Plate-Recognition-YOLOv7-and-CNN-main\main_video.py", line 37, in
pred, LP_detected_img = detect(model_LP, source_img, device, imgsz=640)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "\License-Plate-Recognition-YOLOv7-and-CNN-main\detect.py", line 78, in detect
return final_pred[0].to(device='cpu').detach().numpy(), imgsz
~~~~~~~~~~^^^
IndexError: list index out of range

List out of range is bc your list is empty but u want to make a loop or read sth inside. The reason why your list is empty is mainly because your model cannot catch any license plate or you didnt input the picture so please check the picture or check the output of the model if they return any recognized object


ci: Bump pytorch 1.10.1 -> 1.10.2

For the 1.10.2 release
Signed-off-by: Eli Uriegas<EMAIL_ADDRESS>

py3.6 unittests are expected to fail, merging


Different Luminence values for Saturation vs Hue

Hi,
I was just wondering if there is a specific reason why you are using different luminence values for Saturation and for Hue.
As far as I know the values for Hue should be correct if you use sRGB which uses the BT.709 standard underneath. The relative luminence values for it are described here and match with your Hue adjustment but not with the Saturation one.
Thanks a lot!
Leon

it's been a long time, but iirc it is a direct copy of how the Flash (now Animate) authoring environment applies color effects.

The only source I could find for this is this old documentation from 1993.
He quoted that these values are better for the linear RGB space but I think the current best practise has changed in the last 30 years, time flies :D, and I think your Hue values are actually providing better results.
But no need to change, I was just curious about it!

I think Grant has a little "in" at Adobe on occasion!  I am closing this, but anyone is welcome to continue the conversation!


Fixes bad log order. completionHandler can make a new request before log results from previous request

The current behavior leads to fail if we have many requests. Let's say we do this:
request A
request B (using some data retrieved from request A)
The process works as expected, but not the log.
Expected
log request A
log data received from request A
log request B
Observed:
log request A
log request B
log data received from request A
...
This pull request moves the postNotification right just before the completionHandler and inside the dispatch_group_async. Tested with the same environment and works as expected

See #2571.


KeepKey initialize and recovers flows goBacks behavior is wrong

Overview
I found a strange behavior/bug, if you enter the wallet name on both flows (recovery and initialize), you'll end to the pin view, then if you go back, you are asked to enter the wallet name again but the view is blocked after submitting it because both wallet.reset and wallet.recover throws an error ActionCancelled without any error code
As these methods throws, we don't receive the PINMATRIXREQUEST message, we end up being blocked on the Label view with the keepkey on the Enter new pin state...
References and additional details
See the use of wallet.recover and waller.reset inside Label.tsx or the upcoming hook using it
They shouldn't throw at this point but probably accept the new label
Acceptance Criteria
When you go back right after inserting your label, you should be able to either send the label again or the error should be handled
Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
No response

@cjthompson @pastaghost FYI


Additional tests for code coverage

Coverage increased (+1.2%) to 95.738% when pulling 8c6be97c54feccca655c340151a6fc592ffb6ec6 on wadearnold:additional-tests-for-code-coverage- into 3684831a5df8a315244bebfd5d7052b1134fc57e on moov-io:master.

Coverage increased (+1.2%) to 95.738% when pulling 142330f63c4bf98b807a7715bf4d181de394e4a1 on wadearnold:additional-tests-for-code-coverage- into 3684831a5df8a315244bebfd5d7052b1134fc57e on moov-io:master.

Coverage increased (+1.2%) to 95.738% when pulling 2ee194cfcde207af82c4688d13663b744ec4fecd on wadearnold:additional-tests-for-code-coverage- into 3684831a5df8a315244bebfd5d7052b1134fc57e on moov-io:master.

Coverage increased (+1.3%) to 95.855% when pulling f4692f878341cbc751d8b5d8944a282301bef8d4 on wadearnold:additional-tests-for-code-coverage- into 3684831a5df8a315244bebfd5d7052b1134fc57e on moov-io:master.

Coverage increased (+1.5%) to 95.972% when pulling d7e2b4de74715c159eeae59110353e140b5d6186 on wadearnold:additional-tests-for-code-coverage- into 3684831a5df8a315244bebfd5d7052b1134fc57e on moov-io:master.

Coverage increased (+1.3%) to 95.826% when pulling 83680a1885d0942f56ed7dc86c06db410a82cd45 on wadearnold:additional-tests-for-code-coverage- into 3684831a5df8a315244bebfd5d7052b1134fc57e on moov-io:master.


Open project from Jump-List as Administrator

I have my VS set up to always run as Administrator.
However, when I open a project/solution from the taskbar jump-list it doesn't launch as administrator.
Is this something that could be tweaked?

I don't think it can be done from an extension, but have you looked at this: https://ppolyzos.com/2017/08/08/always-run-visual-studio-as-administrator/

I don't think it can be done from an extension, but have you looked at this: https://ppolyzos.com/2017/08/08/always-run-visual-studio-as-administrator/

That's how I run as an administrator.
I didn't think there was a way of doing this as an extension but you're good at doing things I didn't know were possible. 😉
I'll open a request on the developer community site.


feat: install cmake config module in arch-indep location

toml11 is an header-only library, and the exported cmake target for it only points to the headers; because of this, it seems a good idea to install the cmake config module in an architecture-independent location for cmake, rather than under a library (architecture-dependent) location.
Doing so is easy:

tweak TOML11_INSTALL_CMAKE_DIR to point to a cmake subdirectory of the shared dataroot location (typically $prefix/share)
pass ARCH_INDEPENDENT argument to write_basic_package_version_file(): this is because the cmake config module generated by default contains the architecture bits check (32bit vs 64bit vs ...); since it does not matter in this case, make the config module arch-independent

Thank you for the suggestion!
But, actually toml11 optionally supports precompiled library (libtoml11.a) that includes some deinline-ized functions and explicit specialization of frequently used functions (e.g. toml::parse<toml::type_config>). When -DTOML11_PRECOMPILE=ON, cmake will build binary and install it.
Since CMake says ARCH_INDEPENDENT is intended for packages with no binaries, it seems to be better not to use it in that case.
We can check TOML11_PRECOMPILE variable in cmake, but it makes script a bit complicated and harder to maintain.
I looked some of the libraries that supports both header-only and compiled modes, and many of them just uses LIBDIR in both modes (though I don't know they actually intended to do that). I will also take the same approach here.
So, I am sorry but I am closing this. I appreciate your cooperation. If you find anything else, please feel free to send it.


Partially fixing the scripts and documenting them

@VishnuDuttSharma @routeaccess @prudhvid
I have added documentation to the top of both scripts. (visible in the diff view).
Please see and verify the same.
I have built the package with rosmake and the build does pass.

This gets us closer to solving #49 and #50.

@routeaccess @prudhvid
Completed testThrusters.py. Please review and merge.

@icyflame The SIGINT was a nice touch. I've been meaning to add it in when this file is refactored, you almost read my mind.
I'm taking your word for this PR - I trust this has been tested on the bot.

@routeaccess
Thank you for the merge!
I have been looking into the signal library that Python has, and I am amazed by the functionality it brings to any basic application!
We should start using it in some of the other files, where it may be needed too.


Alert details for VM_DataEgressArtifacts (App Services alert) is outdated

Alert details for VM_DataEgressArtifacts (App Services alert) is outdated (the alert title was incorrect and was corrected in the product).
Should be changed from:
Possible loss of data detected
(VM_DataEgressArtifacts)
to
Possible data exfiltration
(VM_DataEgressArtifacts)

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a2eb1ae9-1cfc-0dd0-a055-79ae32b62a6d
Version Independent ID: 66ce0f69-16aa-c1f8-5163-7cc5392a3399
Content: Reference table for all security alerts in Microsoft Defender for Cloud
Content Source: articles/defender-for-cloud/alerts-reference.md
Service: defender-for-cloud
GitHub Login: @memildin
Microsoft Alias: memildin

@tspivakms Thanks for the feedback! I have assigned the issue to the content author to evaluate and update the doc as appropriate.

Hi @tspivakms, thanks for the update.
I've changed the doc and the revised page will go live within 24hrs.
#please-close


stress-ng: 0.15.10 -> 0.16.04

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/ColinIanKing/stress-ng/releases.
meta.description for stress-ng is: Stress test a computer system
meta.homepage for stress-ng is: https://github.com/ColinIanKing/stress-ng
meta.changelog for stress-ng is: https://github.com/ColinIanKing/stress-ng/raw/V0.16.03/debian/changelog
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.16.03 with grep in /nix/store/3jgyw00g1l0qjqndqcsr7laqn5z3l0ir-stress-ng-0.16.03
found 0.16.03 in filename of file in /nix/store/3jgyw00g1l0qjqndqcsr7laqn5z3l0ir-stress-ng-0.16.03

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
stress-ng

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/3jgyw00g1l0qjqndqcsr7laqn5z3l0ir-stress-ng-0.16.03 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A stress-ng https://github.com/r-ryantm/nixpkgs/archive/79917447213025324fe868a194dc3533da65dc30.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/3jgyw00g1l0qjqndqcsr7laqn5z3l0ir-stress-ng-0.16.03
ls -la /nix/store/3jgyw00g1l0qjqndqcsr7laqn5z3l0ir-stress-ng-0.16.03/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

stress-ng

Maintainer pings
cc @c0bw3b for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/stress-ng/versions.
meta.description for stress-ng is: Stress test a computer system
meta.homepage for stress-ng is: https://github.com/ColinIanKing/stress-ng
meta.changelog for stress-ng is: https://github.com/ColinIanKing/stress-ng/raw/V0.16.04/debian/changelog
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.16.04 with grep in /nix/store/fa53xmcx34iy14diakf687cx2xy9qp41-stress-ng-0.16.04
found 0.16.04 in filename of file in /nix/store/fa53xmcx34iy14diakf687cx2xy9qp41-stress-ng-0.16.04

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
stress-ng

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fa53xmcx34iy14diakf687cx2xy9qp41-stress-ng-0.16.04 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A stress-ng https://github.com/r-ryantm/nixpkgs/archive/43475f181c7318e3c085bd41653ce896513dd25f.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fa53xmcx34iy14diakf687cx2xy9qp41-stress-ng-0.16.04
ls -la /nix/store/fa53xmcx34iy14diakf687cx2xy9qp41-stress-ng-0.16.04/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

stress-ng

Maintainer pings
cc @c0bw3b for testing.


Hide items in right-click menu

When you make an item hidden in ground items, it shows up in right-click menu, maybe have an option that hidden items don't show in the menu like RuneLoader.

Duplicate of #1615


Picture title

Is there a way to show a simple text or title under each picture? The text could be provided by the alt attribute, for example.

@edoplaza Please stop using this package... It's not maintained anymore and has so many comical design decisions that it's not even funny anymore.
This is a good replacement: https://github.com/leandrowd/react-responsive-carousel

Thank you!

@edoplaza Please stop using this package... It's not maintained anymore and has so many comical design decisions that it's not even funny.
There are better, maintained, and simpler packages available instead.
This is a good replacement: https://github.com/leandrowd/react-responsive-carousel

Hi davwheat,
Thank you for maintaining react-images all that time,
But I disagree on your assertion that react-responsive-carousel is a good substitute: react-images and react-responsive-carousel are totally different products : for instance I'm using the modal Gateway of react-images, and there's abolutely no equivalent in react-responsive-carousel

Is this still an issue?


BackHandler not working TabNavigator to StackNavigator

BackHandler not working in StackNavigator nested with TabNavigator
Current Behavior

In my app I use first StackNavigator then nested with TabNavigator. When I want to back from Tab to stack route page, with the help of hardware back in android, there BackHandler not working. Can't get any event of BackHandler.
Not use also  backBehavior: 'none',    option in tabNavigator configuration.

Expected Behavior

I want to get any event or any solution so that I can back to previous page or any other action by backHandler.
Suggest any solution.

Your Environment

software
version

react-navigation
^1.0.0-beta.15

react-native
0.44

node
v6.11.5

npm or yarn
npm

you are using an old version of RN, of this lib, and node. Moreover, which version of npm are you using?
it's probably related to https://github.com/react-community/react-navigation/issues/2523
without the code of the two navigators it's really hard to understand how you set them up

@kelset  Thanks for your response.

my npm version is 3.10.10. Till now we can't upgrade RN version.
my route navigation structure is below:

AppNavigate = StackNavigator({
  Home: {
    screen: Home,
    navigationOptions: {
      header: null
    },
  },
  About: {
    screen: About,
    navigationOptions: {
      header: null,
    },
  },
  Chat: {
    screen: TabNavigator({
      Tab1: { screen: Tab1 },
      Tab2: { screen: Tab2 },
      Tab3: {
        screen: TabNavigator({
          TabInner1: { screen: TabInner1 },
          TabInner2: { screen: TabInner2 },
          TabInner3: { screen: TabInner3 },
        },{
          headerMode:'float',
          tabBarPosition: 'top',
          tabBarOptions:{
            showLabel: true,
            showIcon: false,
            style: {
              backgroundColor: '#ffffff',
            },
          }
        }),
      },
      Tab4: { screen: Tab4 },
      Tab5: { screen: Tab5 },
    },{
      tabBarPosition: 'bottom',
      tabBarOptions: {
        activeTintColor: 'red',
        showLabel: false,
        showIcon: true,
        iconStyle:{
          height:40,
          width:44
        },
        labelStyle: {
          fontSize: 12,
        },
        style: {
          backgroundColor: '#ffffff',
        },
      }
    }),
  },
});

And I setup BackHandler in Tab1 page like below:
componentDidMount() {
    BackHandler.addEventListener('hardwareBackPress', this._handleBackHandler);
  }

  componentWillUnmount() {
    BackHandler.removeEventListener('hardwareBackPress', this._handleBackHandler);
  }

 _handleBackHandler = () => {
   navigation.goBack(null);
    return true;
  }

Now tell me any suggestion to solve the problem.

This worked for me you can try,
import { NavigationActions } from 'react-navigation'
...
...
this.props.navigation.dispatch(NavigationActions.back())

@ibrahimcanbaz   Thanks for your suggestion.
In my project BackHandler event can't fire not a singlt time, if it fire then
navigation.goBack(null);    this is working fine. I need solution how to
BackHandler.removeEventListener('hardwareBackPress', this._handleBackHandler);    this code is working.

Can anyone suggest any solution to the problem?? We are stuck still now with this issue.

this should be resolved in latest version. if not, please create a new issue with a runnable example on https://snack.expo.io -- thanks!

@MSSPL-PiyaleeMaiti  Did you find any solution till now? If yes, please do share here.

@theutkarshsoni create a new issue with more info.


Add Continuous Integration details for project Emome.

Cool. I'll merge. Generally you can keep links to your project's resources, such as CI, in the project's README. It's not very important to have it here.

@dblock thanks for your comment. we'll re-organize everything soon to make sure that the project page won't be too messy :p


[BUG] HttpContext.User is not used in Serilog EcsTextFormatter

ECS integration/library project(s): Elastic.CommonSchema.Serilog 8.6.1, Serilog 3.1.1
ECS .NET assembly version (e.g. 1.4.2): 8.6.1
.NET framework / OS: .NET 6 / Windows 11
Description of the problem, including expected versus actual behavior:
Steps to reproduce:

Use ASP.NET Core with authentication and ClaimsIdentity (e.g. a user is logged in)
The user claims are not included in the ecs output

Cause:
The logEvent.AddPropertyIfAbsent(propertyFactory.CreateProperty(PropertyName, r)); in the HttpContextEnricher doesn't store the raw object, instead the default propertyFactory does a .ToString() of the r-value, which makes the value of the property: Elastic.CommonSchema.Serilog.HttpContextEnricher,Elastic.CommonSchema.Serilog instead or the original object. In the LogEventConverter, the object cannot be retrieved in (for example) the GetUser method
if (e.TryGetScalarPropertyValue(SpecialKeys.HttpContext, out var httpContext)
	 && httpContext.Value is HttpContextEnricher.HttpContextEnrichments enriched) // httpContext.Value is a string!!
	return enriched.User; // <-- never returned

This causes that the reported user is always the environment user instead of the user doing the request.
A workaround is to override the enricher:
using Elastic.CommonSchema.Serilog;
using Serilog.Core;
using Serilog.Events;

public class PatchedHttpContextEnricher : HttpContextEnricher, ILogEventEnricher
{
    private readonly RawPropertyFactory _rawPropertyFactory;

    public PatchedHttpContextEnricher(IHttpContextAccessor httpContextAccessor) : base(httpContextAccessor)
    {
        _rawPropertyFactory = new RawPropertyFactory();
    }

    void ILogEventEnricher.Enrich(LogEvent logEvent, ILogEventPropertyFactory propertyFactory)
    {
        base.Enrich(logEvent, _rawPropertyFactory);
    }

    private class RawPropertyFactory : ILogEventPropertyFactory
    {
        public LogEventProperty CreateProperty(
            string name,
            object? value,
            bool destructureObjects = false)
        {
            return new LogEventProperty(name, new ScalarValue(value));
        }
    }
}

builder.Host.UseSerilog((ctx, config) =>
{
    // Ensure HttpContextAccessor is accessible
    var httpAccessor = ctx.Configuration.Get<HttpContextAccessor>();

    config
        .ReadFrom.Configuration(ctx.Configuration)
        //.Enrich.WithEcsHttpContext(httpAccessor) <-- this is the unpatched code
        .Enrich.With(new PatchedHttpContextEnricher(httpAccessor));
});

Thanks I also ran into this issue and this helped me get httpContext information
One minor change was that for me the following line in the RawPropertyFactory  has to be
 return new LogEventProperty(name, new ScalarValue(value));

Thanks I also ran into this issue and this helped me get httpContext information
One minor change was that for me the following line in the RawPropertyFactory has to be
 return new LogEventProperty(name, new ScalarValue(value));

You're right, I've updated the original post with your comment.

Any plan to fix this upstream?

Opened: https://github.com/elastic/ecs-dotnet/pull/443 to address this. Thanks for reporting this everyone 👍

Is this supposed to be working now?
Looks like this has been moved into a new package called Elastic.Serilog.Enrichers.Web since v8.12.0 but for me there is still to HttpContext enrichment happening in the ECP output?

Hey @NKnusperer
It was suppose to be working but UAParser was throwing a constructor exception because the embedded resource list of regexes name was off.
Opened: https://github.com/elastic/ecs-dotnet/pull/452 to address this.
Will get this out in an update ASAP, thank you for catching this so fast and apologies for shipping this in the first place.

Now works as expected using v8.12.1.