Deactivated collider and/or collider shape doesn't deactivate.

Never mind. I think I'm just a git and programed myself into  a corner or something... I'm going to have to just start the game over I guess because I can't seem to get it right. So, I think it's fine I just didn't make a unique collider... but that's not what is wrong in my main game so I dunno... probably just me being shit even though I've done this for a couple of decades.

Nevermind. FML.


Mouse cursor not moving to external display

Happening on kudu1 with displays at multiple locations. Happens both with VGA, HDMI. External display shows the content. It's just mouse cursor.

Happening on kudu1 with displays at multiple locations. Happens both with VGA, HDMI. External display shows the content. It's just mouse cursor.

Rebooting OS while keeping the dual display config solved this.


Charm installs a snap version that is not working

Bug Description
Charm installs by default latest/stable snap version even on charm channel latest/edge
This snap version (latest/stable 0.29.0) is not starting its services
To Reproduce
I deployed Charm latest/edge version:
juju deploy ubuntu maas-one --series jammy --to 0
juju deploy grafana-agent --series jammy --channel latest/edge
juju add-relation maas-one grafana-agent

Environment
Charm version: grafana-agent: latest/edge (rev 11)
Grafana-agent snap version

Installed by default: latest/stable 0.29.0 broken 2023-03-23
Upraded to make it work: grafana-agent: latest/edge 0.34.3 2023-07-13

Relevant log output
# It installed a version of grafana-agent snap (latest/stable) that seems broken:
$ juju ssh maas-one/0

$ snap info grafana-agent
name:      grafana-agent
summary:   Telemetry Agent
publisher: Simon Aronsson (0x12b)
store-url: https://snapcraft.io/grafana-agent
license:   Apache-2.0
description: |
  Grafana Agent is a telemetry collector for sending metrics, logs, and trace data to the
  opinionated Grafana observability stack
services:
  grafana-agent: simple, enabled, inactive
snap-id:      T1u01xKcraA4IpWHpIGNU2nAOom9qG7i
tracking:     latest/stable
refresh-date: yesterday at 20:48 UTC
channels:
  latest/stable:    0.29.0 2023-03-23 (11) 230MB -
  latest/candidate: 0.29.0 2023-03-23 (11) 230MB -
  latest/beta:      0.29.0 2023-03-23 (11) 230MB -
  latest/edge:      0.34.3 2023-07-13 (13) 264MB -
installed:          0.29.0            (11) 230MB -

$ systemctl status snap.grafana-agent.grafana-agent.service
× snap.grafana-agent.grafana-agent.service - Service for snap application grafana-agent.grafana-agent
     Loaded: loaded (/etc/systemd/system/snap.grafana-agent.grafana-agent.service; enabled; vendor preset: enabled)
     Active: failed (Result: exit-code) since Fri 2023-07-14 16:56:09 UTC; 3h 3min ago
    Process: 6510 ExecStart=/usr/bin/snap run grafana-agent (code=exited, status=1/FAILURE)
   Main PID: 6510 (code=exited, status=1/FAILURE)

# Tried restarting service. Still broken.

# Logs:
$ grep grafana-agent /var/log/syslog
# [..]
Jul 14 20:00:11 maas-1 grafana-agent.grafana-agent[24573]: 2023/07/14 20:00:11 error loading config file /etc/grafana-agent.yaml: failed to unmarshal integration "node_exporter": yaml: unmarshal errors:
Jul 14 20:00:11 maas-1 grafana-agent.grafana-agent[24573]:   line 31: field sysctl_include not found
Jul 14 20:00:11 maas-1 systemd[1]: snap.grafana-agent.grafana-agent.service: Main process exited, code=exited, status=1/FAILURE
Jul 14 20:00:11 maas-1 systemd[1]: snap.grafana-agent.grafana-agent.service: Failed with result 'exit-code'.
Jul 14 20:00:12 maas-1 systemd[1]: snap.grafana-agent.grafana-agent.service: Scheduled restart job, restart counter is at 5.
Jul 14 20:00:12 maas-1 systemd[1]: Stopped Service for snap application grafana-agent.grafana-agent.
Jul 14 20:00:12 maas-1 systemd[1]: snap.grafana-agent.grafana-agent.service: Start request repeated too quickly.
Jul 14 20:00:12 maas-1 systemd[1]: snap.grafana-agent.grafana-agent.service: Failed with result 'exit-code'.
Jul 14 20:00:12 maas-1 systemd[1]: Failed to start Service for snap application grafana-agent.grafana-agent.

Additional context
Upgrading the snap version to latest/edge makes it work
I believe there are two issues:

Grafana-agent snap latest/stable 0.29.0 broken
I would expect charm channel version to match snap channel version

Fixed in https://github.com/simskij/grafana-agent/pull/21.
$ snap info grafana-agent
--snip--
channels:
  latest/stable:    0.34.3 2023-07-19 (13) 264MB -
  latest/candidate: 0.34.3 2023-07-19 (13) 264MB -
  latest/beta:      0.34.3 2023-07-19 (13) 264MB -
  latest/edge:      0.34.3 2023-07-13 (13) 264MB -


Typo in docs: value::is_uint64()

https://github.com/CPPAlliance/json/blob/2d0bef633d47418461710639fda58f6db68f6def/include/boost/json/value.hpp#L1651
Should say unsigned instead of signed.

Should be fixed now, thanks!


JSPrintNumbers file added issue [#2719]

Why:

Linked to issue  [#2719]

Closes #2719
What's being changed:
Added Printnumber.js file to program folder

Awesome! Thank you so much for this opportunity.

@PaxtonDevs
The issue is assigned to someone else. Please look for other issues to contribute. Make sure you type !assign to grab the issue
Thank you!


[Coverity CID :203415]Memory - illegal accesses in /subsys/shell/shell_telnet.c

Static code scan issues seen in File: /subsys/shell/shell_telnet.c
Category: Memory - illegal accesses
Function: write
Component: Other
CID: 203415
Please fix or provide comments to square it off in coverity in the link: https://scan9.coverity.com/reports.htm#v32951/p12996

@pizi-nordic or @pabigot could you fix this one, since @nordic-krch is off-work?

I don't have access to Coverity (yet?  did request it) so can do nothing without details of what the problem is.

I don't have access to Coverity (yet? did request it) so can do nothing without details of what the problem is.

I thought you already managed to get it, to solve the coverity fixes you pushed yesterday, sorry.

367static int write(const struct shell_transport *transport,
368                 const void *data, size_t length, size_t *cnt)
369{
370        struct shell_telnet_line_buf *lb;
371        size_t copy_len;
372        int err;
373        u32_t timeout;
374
   	1. Condition sh_telnet == NULL, taking false branch.
375        if (sh_telnet == NULL) {
376                *cnt = 0;
377                return -ENODEV;
378        }
379
   	2. Condition sh_telnet->client_ctx == NULL, taking false branch.
   	3. Condition sh_telnet->output_lock, taking false branch.
380        if (sh_telnet->client_ctx == NULL || sh_telnet->output_lock) {
381                *cnt = length;
382                return 0;
383        }
384
385        *cnt = 0;
386        lb = &sh_telnet->line_out;
387
388        /* Stop the transmission timer, so it does not interrupt the operation.
389         */
390        timeout = k_timer_remaining_get(&sh_telnet->send_timer);
391        k_timer_stop(&sh_telnet->send_timer);
392
393        do {
   	4. Condition lb->len + length - *cnt > 80, taking true branch.
   	9. Condition lb->len + length - *cnt > 80, taking true branch.
   	14. Condition lb->len + length - *cnt > 80, taking true branch.
   	21. Condition lb->len + length - *cnt > 80, taking false branch.
394                if (lb->len + length - *cnt > TELNET_LINE_SIZE) {
395                        copy_len = TELNET_LINE_SIZE - lb->len;
   	5. Falling through to end of if statement.
   	10. Falling through to end of if statement.
   	15. Falling through to end of if statement.
396                } else {
397                        copy_len = length - *cnt;
398                }
399

CID 203415 (#1 of 1): Out-of-bounds read (OVERRUN)
22. overrun-local: Overrunning array of 80 bytes at byte offset 80 by dereferencing pointer lb->buf + lb->len. [Note: The source code implementation of the function has been overridden by a builtin model.]
400                memcpy(lb->buf + lb->len, (u8_t *)data + *cnt, copy_len);
401                lb->len += copy_len;
402
403                /* Send the data immediately if the buffer is full or line feed
404                 * is recognized.
405                 */
   	6. Condition lb->buf[lb->len - 1] == 10, taking true branch.
   	11. Condition lb->buf[lb->len - 1] == 10, taking true branch.
   	16. Condition lb->buf[lb->len - 1] == 10, taking false branch.
   	17. Condition lb->len == 80, taking true branch.
   	18. cond_const: Checking lb->len == 80 implies that lb->len is 80 on the true branch.
406                if (lb->buf[lb->len - 1] == '\n' ||
407                    lb->len == TELNET_LINE_SIZE) {
408                        err = telnet_send();
   	7. Condition err != 0, taking false branch.
   	12. Condition err != 0, taking false branch.
   	19. Condition err != 0, taking false branch.
409                        if (err != 0) {
410                                *cnt = length;
411                                return err;
412                        }
413                }
414
415                *cnt += copy_len;
   	8. Condition *cnt < length, taking true branch.
   	13. Condition *cnt < length, taking true branch.
   	20. Condition *cnt < length, taking true branch.
416        } while (*cnt < length);
417
418        if (lb->len > 0) {
419                /* Check if the timer was already running, initialize otherwise.
420                 */
421                timeout = (timeout == 0) ? TELNET_TIMEOUT : timeout;
422
423                k_timer_start(&sh_telnet->send_timer, timeout, 0);
424        }
425
426        sh_telnet->shell_handler(SHELL_TRANSPORT_EVT_TX_RDY,
427                                 sh_telnet->shell_context);
428
429        return 0;
430}

@pabigot ^^

Ugh.
I think Coverity is wrong here, and that it's not detecting that a successful call to telnet_send() will have reset lb->len to zero, so it's no longer true that lb->len = 80.
On the other hand, the loop is grossly overcomplicated.  There's a trivial and obviously correct transformation that makes it a bit more clear to a human.  Under other circumstances I'd want to replace it with something less obscure, but that would introduce assumptions that other threads aren't accessing the buffer.
diff --git a/subsys/shell/shell_telnet.c b/subsys/shell/shell_telnet.c
index 5d57ac086e..fc84dc4984 100644
--- a/subsys/shell/shell_telnet.c
+++ b/subsys/shell/shell_telnet.c
@@ -391,10 +390,10 @@ static int write(const struct shell_transport *transport,
        k_timer_stop(&sh_telnet->send_timer);

        do {
-               if (lb->len + length - *cnt > TELNET_LINE_SIZE) {
+               copy_len = length - *cnt;
+               if ((lb->len + copy_len) > TELNET_LINE_SIZE) {
                        copy_len = TELNET_LINE_SIZE - lb->len;
-               } else {
-                       copy_len = length - *cnt;
                }

                memcpy(lb->buf + lb->len, (u8_t *)data + *cnt, copy_len);

Not sure what to do.

I think Coverity is wrong here, [..] Under other circumstances I'd want to replace it with something less obscure, [..]

If the understanding is that this is a false positive, then I'd either lower the priority and/or move the milestone to "future" and wait for @nordic-krch to be back to discuss with him. No need for doing things in a hurry.
I'll do so now, if you disagree please undo.

Closed as false positive with this comment:
All invocations of telnet_send() that succeed clear the buffer len field to zero.  In the problematic code lb->len aliases to this field.
Step 18 is reached by a path where lb->len == 80 but the success of telnet_send() checked at step 19 ensures that lb->len was zeroed, so the detection at step 22 of an access via lb->len at offset 80 appears to be impossible.


Changes to introduce NcIdentBeacon and gainValue

Merged NcIdentificationActuator and NcIdentificationSensor into a single worker called NcIdentBeacon
change NcGain property name to gainValue

Reviewed on the call on the 24th of January 2023


Hai / Carbon account question follow

Follow up on email from Hai:

We have an existing Carbon account that we'd like to change to go directly to our OpenCollective account.

done and answered via email


Images with space between the name is not loading

When I insert a image in a post with spaces in between the name, it is not loading.
I use forestry to add the post.

nThis looks like a global issue, Not specific to this theme.
@regisphilibert Do you know of a workaround that could make the theme more bulletproof or shall we just mark this as won't fix?

@maharjanmilan This is a Goldmark issue (default since 0.60). If you're not specific about your markdown renderer, you can switch to Blackfriday and all will be good.
# config.yaml
markup:
  defaultMarkdownHandler: blackfriday

See: https://gohugo.io/getting-started/configuration-markup/


ASIC support tracking issue

 Issue by whitequark
Thursday Aug 22, 2019 at 21:49 GMT
Originally opened as https://github.com/m-labs/nmigen/issues/184

A number of people have expressed a desire to use nMigen for ASIC design. This issue tracks missing aspects of that workflow.

[x] On FPGAs, asynchronous resets inhibit inference of BRAMs, DSPs, etc. On ASICs, asynchronous resets are required to make sure the chip is reset properly even with no clock.
[ ] On FPGAs, resets are generally positive polarity, and using the wrong polarity can have severe consequences for timing. On ASICs, resets are generally negative polarity, such that the chip is held in reset during power rail ramp-up.
[ ] On ASICs and in some cases on FPGAs, memories may not be initialized.
[ ] On ASICs, reset_less signals are not initialized at power-on.

(There are likely more.)

@Fatsie you're interested in active low resets for ASIC work, right? I was wondering if you have any opinion on handling ResetSignal in that case; unlike ClockSignal, a lot of generic code depends on ResetSignal polarity being logic level (i.e. active high), so if we add clock domains with negative polarity resets that have inverted levels of ResetSignal, all such logic will break. There are a few solutions I can think of, but none are ideal.

@whitequark I don't think it is a problem to have nmigen only have active high reset support in it's clock domain.
For ASICs you typically want the external reset pin active low so you can keep the reset low during power ramp up by a simple RC circuit. But you can handle that by PinsN definition for the platform. If the standard cell library would have flip-flops with active low set and reset signals yosys should optimize away the two inverters in the logic path of the set/reset signal.

I've seen lots of inverters get added to a design since tools will often synthesize things to use active-low resets (sometimes due to what's available in technology libraries). That ends up leading to a bunch of extra inverters in the design, especially at black-box boundaries where you've synthesized something to be used as a hard macro elsewhere (back to back inverters in timing reports confused the heck out of me for a while).
I can't say to what degree it actually affected the quality of the design, I don't remember the numbers at the moment.

One possible solution here would be to add a late-bound ResetActive() AST node (so that, if you are branching on a reset in a generic module, you'd need to use m.If(ResetActive()) rather than m.If(ResetSignal())), and then emit a diagnostic on use of m.If(ResetSignal()).

I've seen lots of inverters get added to a design since tools will often synthesize things to use active-low resets (sometimes due to what's available in technology libraries). That ends up leading to a bunch of extra inverters in the design, especially at black-box boundaries where you've synthesized something to be used as a hard macro elsewhere (back to back inverters in timing reports confused the heck out of me for a while).

For ASIC one typically does tech mapping on the full flattened design, this allows yosys optimization pass to removed all double inverters.

For ASIC one typically does tech mapping on the full flattened design, this allows yosys optimization pass to remove all double inverters.

We were using Design Compiler and Innovus, not yosys (same problem, different source language than nmigen). The issue in our case stems from the fact that we have a repeated architecture with the same elements stamped out multiple times. To speed up synthesis, we synthesize each of the repeated elements and then stamp them out as hard macros at the top level. The double-inverters are seen at the boundaries of the hard macros and won't be optimized out.


AKS-Integrate ACR (Azure Container Registry)

Is there an existing issue for this?

[x] I have searched the existing issues

Description
Integrate ACR (Azure Container Registry)
New or Affected Resource(s)/Data Source(s)
azurerm
Potential Terraform Configuration
No response
References
No response

PR created: Integrate aks with azure rbac #35


Istio-telemetry generating a lot of traces

This is an issue just to track what I have discussed on slack with @douglas-reid and @mandarjog
Istio-telemetry is generating a lot of traces in Jaeger, see:

Under high load, this is effectively DDOSing our Jaeger instance with Junk.
I believe this is being addressed in 1.0.5 and 1.1.x (would be nice if someone could link an issue or PR?)
@mandarjog suggested removing the --trace_zipkin_url setting from the istio-telemetry deployment to stop some spans, however @douglas-reid said "that will stop mixer-proper generated traces. unfortunately, it won't stop the envoy in front of mixer from generating... and that is hard-coded (not controlled by Pilot)"

resolved by #10281

Dupe of https://github.com/istio/istio/issues/4227 ?  Should we close this now?  Or wait until verified with 1.0.5 ?

Happy to close in favour of 4227!

Happy to close in favour of 4227!

Hello, I am facing the similar problem you meet, I delete the --trace_zipkin_url arg in istio-demo.yaml and the mixer stop sending spans. However, I receive many istio-telemetry span appearing single in zipkin. Did you meet this problem?

@JackGzj which version of Istio?  This is what https://github.com/istio/istio/pull/10281 is meant to address.


Unprocessed items with BatchWrite

I've had almost 1 unprocessed item for every batch write, and retrying for unprocessed items significantly slows down the upload.
I wonder if you experience this too?

Yes, some throttling is inevitable.

But overall, the effect is a high throughput.

I'm not sure about the "unprocessed item" though, that doesn't sound correct.

Hi, just to follow up, there's a great article from Yan Cui on this. It describes a strategy for setting up DynamoDB tables to get the maximum capacity straight away: https://theburningmonk.com/2019/03/understanding-the-scaling-behaviour-of-dynamodb-ondemand-tables/

Very insightful. Thanks for that!


Designing application & parallex scrolling effect

Hi @aishwaryas15 can you please provide few more details or a demo that I can see and assign it to you.

Shall I attach a screenshot? So u can get it?

Yes please @aishwaryas15.

https://user-images.githubusercontent.com/62896706/125332007-56f50680-e366-11eb-92ad-a14a02f4df2d.mp4
this is just a example i made

Looks great @aishwaryas15. I am assigning you the issue. Start working on designing the application & adding parallex scrolling effect. We also have one UI design ready with us. Please have a look and try designing the application according to that if possible. OR if you have another design then you can show that as well.

@aishwaryas15 Updates?

Yeah sorry.By tonight I'll submit it


Feature suggestion: dropdown's could add an "onCreateNew" option?

when a dropdown has option "allowAdditions" set to "true": I need to tell if a newly created selection is added or an existing selection, and more importantly, I need a way to ** control the actually added value/text ** (for example, I need to create a new student, and the addedValue is the student's id, and the student's name should be capitalized).
so instead of:
$('.ui.dropdown').dropdown({
    allowAddtions: true,
    onCreate: function (addedValue, addedText, $chioce) {
        if ($chioce.hasClass('addition')) {
          console.log('newly created selection');
        } else {
          console.log('existing selection');
          $.ajax(...).done(function (rsp) {
              // HOW COULD I CHANGE THE ADDED VALUE?
          });
        }
    }
});

I think this way is better:
$('.ui.dropdown').dropdown({
    allowAddtions: true,
    onCreate: function (addedValue, addedText, $chioce) {
        console.log('existing selection');
    },
    onCreateNew: function (addedValue, addedText, $chioce) {
        console.log('newly create selection');
        var d = $.Deferred();
        // creat a new object
        $.ajax(...).done(function (rsp) {
            d.resolve([rsp.id, rsp.name]);
        });
        return d;
        // or just return a new [id, value] if you just need synchronous operations
    },
});

Thank you very much!

Where did you get that onCreate option? The documentation says onAdd and even onAdd doesn't seem to work for me.

This is over complicating, it's easy enough tO control the added text and values with metadata data-text, etc.
You can also use a custom action setting to trick out something more complex, for example a drop down choice that adds multiple values at once.


[dv, fcov] PMP related coverage holes

@ctopal assigned to all of these
The following PMP coverpoints are not yet being hit:

[ ] A higher priority entry allowing an access a lower priority entry denies - cp_pmp_iside_region_override/cp_pmp_iside2_region_override/cp_pmp_dside_region_override
[ ] NA4 and NAPOT matching modes and all privileges other than RWX for region 0 - cp_region_mode/cp_region_priv_bits
[ ] All locked privilege settings for all regions - cp_region_priv_bits
[ ] Attempting to edit a locked region - cp_edit_locked_pmpcfg/cp_edit_locked_pmpaddr

I've ignored ePMP related functionality for now as it's known that's not being stimulated and initial work to rectify this is underway.
Original estimate 8

estimate 16
remaining 2022-07-28 8

I haven't looked at the first one yet but here are my general findings:
For the second one, I am not exactly sure what region 90 means but I can also see R/RW/X/XR privileges and every matching modes in every region except region 0 when I run these tests: riscv_pmp_basic_test, riscv_pmp_disable_all_regions_test, riscv_pmp_out_of_bounds_test with iteration count of 5.
Locked privilege settings are indeed not getting set, which is probably something to do with some riscv-dv knob and require further investigation. That explains item 3 and 4.

After having a more detailed look at the reports and testing out Marno's PR for ePMP this one looks like it's going to be a bigger task than I first thought.
1 - Region override: This one is only possible when we are in NA4 mode in PMP (which is not our default mode and it's currently not possible to only randomize mode field of pmpcfg's.) so we need to either define a directed test to hit this or refactor how we handle PMPCFG randomization in riscv-dv.
2 - Similar as above, if we are not explicitly setting NA4 mode we are highly unlikely to see it due to not having a field specific randomization in PMPCFG registers. Also as far as I understand, we are not even supposed to see NAPOT with our current Ibex configuration of opentitan. The reason is the parameter of PMPGranularity is 0 in opentitan config (which only allows for TOR and NA4 according to the spec.
3 - This will only be 100% when we have non-failing riscv_pmp_full_random_test with a lot of iterations
4 - This one is easy and gets done with Marno's PR (#1695).

Region override: This one is only possible when we are in NA4 mode in PMP

Why is this the case? A smaller region inside a larger region should result in an override for instance. Overlapping regions using TOR could also exhibit overrides.

Also as far as I understand, we are not even supposed to see NAPOT with our current Ibex configuration of opentitan. The reason is the parameter of PMPGranularity is 0 in opentitan config (which only allows for TOR and NA4 according to the spec.)

PMPGranularity of 0 gives you the maximum possible options, allowing NA4 regions along with NAPOT regions of sizes 8 bytes and up.
I think we definitely need to improve the PMP randomization here to hit the above two cases.

3 - This will only be 100% when we have non-failing riscv_pmp_full_random_test with a lot of iterations

From what I saw of the coverage we didn't get any locked regions at all. Have you observed locked regions anywhere in RISC-V DV?

Region override: This one is only possible when we are in NA4 mode in PMP

Why is this the case? A smaller region inside a larger region should result in an override for instance. Overlapping regions using TOR could also exhibit overrides.

As I explained in the next part, I mistakenly looked at the code and spec together to decide we can't do it but it's probably a code bug. In these constraints: https://github.com/lowRISC/ibex/blob/6518cb6db63ef56f428331cf534794336a5c7af9/vendor/google_riscv-dv/src/riscv_pmp_cfg.sv#L108-L124
we are effectively disabling a possibility of override in TOR mode

Also as far as I understand, we are not even supposed to see NAPOT with our current Ibex configuration of opentitan. The reason is the parameter of PMPGranularity is 0 in opentitan config (which only allows for TOR and NA4 according to the spec.)

PMPGranularity of 0 gives you the maximum possible options, allowing NA4 regions along with NAPOT regions of sizes 8 bytes and up.
I think we definitely need to improve the PMP randomization here to hit the above two cases.

Okay I think I made the mistake of looking at riscv-dv source code and spec together, thinking it was bug-free :) In riscv-dv we have this constraint: https://github.com/lowRISC/ibex/blob/6518cb6db63ef56f428331cf534794336a5c7af9/vendor/google_riscv-dv/src/riscv_pmp_cfg.sv#L89-L94
that guarantees if the granularity is set to 0, we wouldn't get NAPOT mode.
I definitely agree we need to improve PMP randomization.

3 - This will only be 100% when we have non-failing riscv_pmp_full_random_test with a lot of iterations

From what I saw of the coverage we didn't get any locked regions at all. Have you observed locked regions anywhere in RISC-V DV?

I tried to run a regression on top of Marno's changes in PR that touches PMP tests in #1695 . In that we were setting locked field (it's not getting set because in most of our current PMP tests locked field of pmpcfgN is not set -leaves it at default because of how riscv-dv handles things.)

First item is getting hit with the latest work around pmp randomization.
Second item will need further investigation.
Third item will be complete after ePMP integration is done. Already hitting every option with #1734  (- epmp related ones) in every region except region 0.
Fourth item will be complete after ePMP integration is done. It was getting hit with Marno's PR.

Second item can be solved by adding a "if we are enabling pmp_randomize, do not owerride pmp_cfg0" clause to the place where we do PMP related instruction generation in riscv-dv. However, when I do that -in almost half of the seeds- I encounter issue https://github.com/lowRISC/ibex/issues/1720.

Second item can be solved by adding a "if we are enabling pmp_randomize, do not owerride pmp_cfg0" clause to the place where we do PMP related instruction generation in riscv-dv. However, when I do that -in almost half of the seeds- I encounter issue #1720.

As discussed yesterday, the problem is we do not want full random PMP when MML and/or MMWP are enabled. This is because it will never reach useful instructions unless the instructions before and after main are set as executable. As a solution, we discussed that in the case that PMP randomization is enabled and MML/MMWP are set, we do not hardcode region 0 (instruction before main) and region 1 (instructions after main). Instead, we select a random value i between 1 and n (where n is the highest PMP region), we then set region i to executable, TOR and with the address <main> + 0x2666 to. Finally we set region i-1 address to 0x80000000.

I created a PR for the second point: https://github.com/google/riscv-dv/pull/885

spent 4

I adjusted https://github.com/google/riscv-dv/pull/885 to initialize the PMP config entries as disabled in the case that MML or MMWP are enabled.

spent 2

I spent quite some time debugging https://github.com/google/riscv-dv/pull/885 and solved a number of problems:

Full random test now generates random addresses instead of offsets from main.
Stop CSR write tests being generated when full random is enabled.
When MML mode is enabled fix the region 0 CSR permission for the PMP config write test.
Generate a PMP region to be read/write for the kernel stack.
Open up an issue on an observed divergence: https://github.com/lowRISC/ibex/issues/1774

Since this task seems to be growing larger I created another issue for this: https://github.com/lowRISC/ibex/issues/1777

spent 4

I've run riscv_pmp_full_random_test 12 times with coverage enabled, looks like we are hitting the last point of this task which was NA4 and NAPOT matching modes and all privileges other than RWX for region 0. Thanks @marnovandermaas for your work regarding PMP in general!


Remove extra line break in tab name

Fixes #9918

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/b1cbb375bc8da160cb7f7337a77d9e4068301565/
Edited pages:

cockroachcloud/connect-to-your-cluster.md

@mikeCRL, @gemma-shay, the tabs now look a bit odd on hover:

Can we remove style="width: 15%" from each button? I think that would resolve this.

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/890cf5263c719b4fbf1ecfb3a6300e8a5d64087f/
Edited pages:

cockroachcloud/connect-to-your-cluster.md

@jseldess That did resolve the issue here. Thanks. Merging with that change.


architectures=avr in library.properties seems to cause some issues.

I was working on my Telemetrix4Esp8266 server today and noticed a few things you may be interested in.

I installed Telemetrix4Arduino using the library manager in a clean Arduino environment. DHTstable is in the depends section of my library.properties file. DHTstable is loaded properly, and I see it where I expect it to be when I click on File/Examples in the Arduino IDE.  I then install Telemetrix4Esp8266, which also specified DHTstable in its depends section, and now DHTstable is moved to INCOMPATIBLE. This happens whether or not I have an avr board or esp8266 board selected in the IDE.

Also, as a result of the architectures=avr (I think), when I compile for the ESP8266, I get the following warning:

WARNING: library DHTStable claims to run on avr architecture(s) and may be incompatible with your current board, which runs on esp8266 architecture(s).

Despite the warning, when I run my code against a DHT11 and DHT22 connected to an ESP8266, the library works perfectly.

BTW, I tried running DHTNew with my library and the built-in Arduino examples, and both the humidity and temperature report incorrect values as opposed to DHTstable.

Great find, i am going to change this architecture flag.
Arduino lint did not complain about this.
It might be also occuring in other libraries, need to check them if they are really only avr.

fixed in 1.0.1


Allow configuring ssh options

SSH defaults to reading things from ~/.ssh, which is problematic when the service runs without a home directory. Allow specifying a config file to be used. In this pull request we still have a split IdentityFile and UserKnownHostsFile, but I later put a frozen /etc/ssh/ssh_known_hosts file in the filesystem image which removes the need for the known hosts file, and we pass a path to the config file (using ssh -F), rather than passing individual options with -o.
A continuation of this is currently running in production.
Depends on #73 and #71.

LGTM f34ebff

LGTM 1dd9b86


Feature Request - implementing the Playlist Importer functionality

Playlist Importer is a mod that is no longer in development and not supporting 0.8.x playlist metadata additions.
Playlist Importer is available here https://github.com/JacobMcAuley/playlist_import
What this mod does is, after setting a core folder on your drive or S3 storage, adds a button that imports all new sounds using "folders become playlists" file structure, set them defaults (only volume is supported, as this mod is no longer supported for 0.8). Existing sounds that match file names are not imported.
Since this mod is adding Bulk Edit, it can use this Bulk Edit defaults to import playlists from a file structure and set their default properties.

Playlist importer was replaced with core functionality in 0.8.x if I'm not mistaken which is why it's no longer in active development, so I'd likely not recode that function but rather see what I can add to it for bulk editing of imported sounds.

Not completely. Playlist Imported allowed to have a directory structure that you can reimport multiple times, adding only the newly added sounds. There was a button to wipe the memory of previously imported sounds if you wanted to make a full import.
0.8.x Bulk Import would reimport as new files duplicate sounds. It also doesn't set defaults to them.

Ah I see - I'll look into it!

Just to set the scope for this feature, which is enhancements to the core "bulk import" funtionality to allow:

Dialog option to replace/skip/duplicate sounds when adding to an existing playlist
Open Playlist Enhancer's "Bulk Edit Sounds" when using the bulk import to edit the playlist.

I would personally go with Settings rather than dialog. Default behavior should be to skip sounds that already exist in a playlist.
Instead of opening Bulk Edit Sounds, apply the default (as set in Bulk Edit Sounds or in Settings) to all added sounds.

The only problem with the first being solely settings is that the requirement can change depending on what tracks you're importing and if it were me I'd only realise that after running the import - settings would be used to choose a default option so a simple "yes" would be all that you'd need to press to continue the default action to reduce overheads a little.
Second point I feel this is a large change to a large amount of sounds without a confirmation from the user so I'd at least have a yes/no confirmation on apply bulk-edit defaults to the sounds.

I don't mind that much, my main issue is showing too many dialogs to the user when he has a default behavior. Usually the default behavior is "add only sounds that are missing from this playlist, and remove those that I have removed from the folder" and "apply default settings to all new added songs so they match the rest of the playlist".
But I have to agree that importing sounds will be a relatively infrequent occasion, not multiple times per session, so it is okay to show a dialog.

I did have a look into this but haven't had much time recently to work on big features like this - It will be one I'll keep in the backlog for future releases, though!


async support for tungstenite and ws clients

Currently only the jsonrpsee client has actual support for async, the others simply block no matter what.
If possible we should implement async for the tungstenite and ws clients as well. If not possible we should document that async is not supported.

ws does not have an async API

https://github.com/housleyjk/ws-rs/issues/288

For tungstenite there is a async wrapper crate: tokio-tungstenite


Ctrl+C does not kill process

After running trunk watch the code is built, and trunk sits waiting for file changes.
At this point, hitting "Ctrl+C" does nothing, I have to explicitly kill the process via task manager.
OS: Windows 10

Update: it appears this only occurs when the process is running under "windows terminal". When run from the legacy "cmd.exe" then Ctrl+C works normally.

This magically fixed itself, so I'm going to put it down to a bug in windows terminal...


Bluetooth: has: Check writable preset support on preset registration

This fixes issue where API use is able to register writable preset even if the CONFIG_BT_HAS_PRESET_NAME_DYNAMIC Kconfig option was disabled.

Updated commit message


钉钉交流群群满

Mark项目三个月了，忘记加群，今天看的时候发现群全满了，求博主建新群

钉钉群：30301472

Close the issue during QR code has been updated.


set up travis/appveyor CI

also update readme (update the CI urls and uncomment the CI status) when that happens

#2 merged and 4bfe2df573d6756d113f4f2eacec420baea31eac updated badges.
As of now, only I have access to Travis and Appveyor, so I recommend at some point we create accounts on those services so we can have a group of people able to be administrator for the CI.


Create RESTful API and call model prediction from another VM

So basically, we'll host the model and RESTful API on a workstation and make calls to the API from the ETL machine (which will tie in to the platform extensibility point).
In this task, I will use two Azure VMs within a VNet to test this functionality.
Note that the API call (from the SQL Server VM) will not pass values, but will instead make the call to the workstation VM to run the entire script (including pulling data in from the SQL Server VM).

I wonder if wrapping a trained model in flask would be a fast way to accomplish this.


Existing text files with German characters get corrupted after opening in VS Code

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

VS Code Version: 1.78.2 and before.
OS Version: Windows_NT x64 10.0.19044

Steps to Reproduce:

Select encoding as UTF-8.
Open exiting file with German characters.
Special German characters shows as � in the file.
Make changes to the file. Don't even need to change German characters. Any changes are fine.
Push code to git. The characters show as 'ï¿½' in git diff.
Below are some examples of German chars getting corrupted and when they are pushed from VS Code:

ä -> ï¿½
ß -> ï¿½
ö -> ï¿½

/gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Clicking the Filesystem Folder under 3 dot menu results in a 401 Unauthorized web page and a Failed Login Notification

What version of Music Assistant has the issue?
2022.8 beta 0
The problem
I click on the File Icon labeled Filesystem under the 3 dot menu and a new tab opens in the browser (Edge and Chrome)
and it says 401 Unauthorized , and I get a Notification that states failed login from the IP address I was using Music Assistant
How to reproduce
Choose Artist /Album/Track and Click the Icon
Relevant log output
Logger: homeassistant.components.http.ban
Source: components/http/ban.py:84
Integration: HTTP ([documentation](https://rc.home-assistant.io/integrations/http), [issues](https://github.com/home-assistant/home-assistant/issues?q=is%3Aissue+is%3Aopen+label%3A%22integration%3A+http%22))
First occurred: 1:48:58 PM (2 occurrences)
Last logged: 1:53:46 PM

Login attempt or request with invalid authentication from <IP_ADDRESS> (<IP_ADDRESS>). Requested URL: '/media/MUSIC/Styx'. (Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/103.0.5060.134 Safari/537.36 Edg/103.0.1264.77)
Login attempt or request with invalid authentication from <IP_ADDRESS> (<IP_ADDRESS>). Requested URL: '/media/MUSIC/Various%20Artists/Greatest%20Hits'. (Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36)

Additional information
No response
What version of Home Assistant Core are your running
2022.8.0b2
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Raspberry Pi

That link should be disabled for the filesystem provider... Looks like I missed a spot.
 I'll fix it. Maybe make it a tiny bit smarter and link to the "browse" section instead...
For streaming providers that link takes you to the details page of the item in their catalog but for the file provider it tries opening a file location which it can't

Well  Found it in 2022.8 beta 0... but I just checked my Production system and its in 2022.7.4 as well... just a FYI

Fixed in MA V2.0.0b12


Can this PR be closed?

Yes, sure! Thanks for suggesting me how to workaround it.


Cache Headers aren't avaible on headers

Hi,
I have an API that uses HTTP Caching, I cannot access using "headers" I look for a configuration misread or something but I can't find nothing about that.
For example my test:
example "http_cache" do
      75.times { create(:event)}
      do_request
      expect(status).to eq(200)
      expect(headers).to include("Cache-Control")
      expect(headers).to include("Last-Modified")
end

Looking in the documentation there are a config for the headers that I modified but the headers doesn't appear.
My Rspec Api Documentation configuration:
RspecApiDocumentation.configure do |config|
  config.api_name = "API Documentation"
  config.response_headers_to_include = ["Content-Type", "Cache-Control", "Etag"]
end

Using a normal Rspec test I cant access to response headers and the "Cache-Control" is avaible, but using DSL I can't access to the Caching headers.
It's is the normal behavior or I didn't configure the test.

@jflobos what happens if you comment out response_headers_to_include?

Hi,
Finally I found the problem, It was me, I was looking for the "request", when I must look for "response_headers". The response_headers are working fine.
This is the code without the problem:
example "http_cache" do
      75.times { create(:event)}
      do_request
      p response_headers
      expect(status).to eq(200)
      expect(response_headers).to include("Cache-Control")
      expect(response_headers).to include("Etag")
end

Thanks!


Body for RedirectResponse

Good day.
I found some conflict (as I see it) between RedirectResponse class and openapi schema.
class RedirectResponse(Response):
    def __init__(
        self,
        url: typing.Union[str, URL],
        status_code: int = 307,
        headers: dict = None,
        background: BackgroundTask = None,
    ) -> None:
        super().__init__(
            content=b"", status_code=status_code, headers=headers, background=background
        )
        self.headers["location"] = quote_plus(str(url), safe=":/%#?&=@[]!$&'()*+,;")

We can't define a content. But only 304 status code (of the redirect codes) really can't have a body.
This fact and STATUS_CODES_WITH_NO_BODY = {..., 304} makes a trouble with openapi schema validation.
May be, we should allow set a content with check for the 304 status_code (and nullify content if it is)?

I believe 204 is fine since it is named  No Content, also RFC states it clearly by saying it must not include a message-body.

The 204 response MUST NOT include a message-body, and thus is always terminated by the first empty line after the header fields.

I believe 204 is fine since it is named  No Content, also RFC states it clearly by saying it must not include a message-body.

The 204 response MUST NOT include a message-body, and thus is always terminated by the first empty line after the header fields.

It's issue only about 3XX responses. About 307 especially.

Hmm, I'm not sure I understand what you need or what you are suggesting 🤔
Could you please create a minimal self-contained example that shows your use case? ☕

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


LA-1864 - Allow overriding operations to support both query & infinite query using the same custom queryOptions mutator

Summary
This PR introduces support for passing the infinite property when a custom options mutator is specified in the query options. This was added to allow overriding operations to support both query & infinite query using the same custom queryOptions mutator with the additional infinite context for conditional behaviour). This section uses this infinite property to add getNextPageParam and getPreviousPageParam for infinite queries.
Changes

Updated the logic to include the infinite property when queryOptionsMutator has a second argument.

Dependencies

#2

Links

LA-1864

Can you update the PR with the purpose of the change and usage? Looking at the code we can see the additional infinite prop is passed along to the function, but why? ie: allow overriding operations to support both query & infinite query using the same custom queryOptions mutator with the additional infinite context for conditional behaviour)

Added


Start boxes unable to resize

Can't adjust the start boxes(move, resize).

Complete with https://github.com/beyond-all-reason/bar-lobby/pull/189

Done with https://github.com/beyond-all-reason/bar-lobby/pull/189
Precise resizing will come later


add generator expressions to matcher for python argumentOrParameter

Closes #323

Oh yeah and sorry bout CI; will be fixed once I merge #638

pre-commit.ci autofix

I just pushed the latest round of changes.


Delete component issue

Run VSCDE ext.
Use any sample mobile app e.g. M_Sample_TAU_1.1.5.zip
D&D any component e.g. Image or Graph
Click on the index.html source code editor tab
Click on the design Editor Tab
Try to delete component

Observe:
Not able to delete component - both using  button after element click nor  key on Structure view
Expected:
Component is deleted

2.

3.

This is a difficult issue. While we come back to editor mode - we are going out of electron context (It works similar to browser cards) and lose focus. When we go back to the DE, then we get focus on whole webview, not a particular element. This is difficult because debugging in VSCode webview is not working. I try to downgrade version of VSCode, but API has changed and it doesn't work as well.

I get to know what kind of issue is this one. I think this is VSCode issue. When the Design Editor is restored, not every event listener is rebinded. I will try to code the solution that can restore binding of Delete event listener. I will end it on monday

I don't know what is the issue purpose, I am switching to another one. Rebinding event listeners don't. work


Add 2.4 api

implement vsync2.4 callback, setactiveconfigwithconstraints
and getdisplayvsyncperiod.
Tracked-On: OAM-92711
Signed-Off-By: Liu, Yuanzhe<EMAIL_ADDRESS>

It looks not good to insert the version number into the name of method and property.

this patch only implement registering callback, and when the callback will be invoked?

LGTM


Clarify use-case and docs structure

This is a quick PR to clarify how documentation structure maps onto the structure of this theme, and to provide guidance for folks that want a single-sidebar theme instead. cc @zaneselvans who I think was confused by our docs - would this help?

Hey thanks!  Yes, this would have made it much clearer. My first intuition was that I had messed up the formatting or configuration somehow, or done something dumb that just wasn't being exercised on the RTD theme. Knowing that this is the expected behavior and that users might want to re-jigger the toctree into a small number of high-level sections with a number of sub-pages is definitely helpful.


MINOR: Remove redundant apostrophe in doc

Found a redundant apostrophe appeared in doc. Also, if we wrap commands in <code  tag, we should not need the apostrophe.
Before:

Before:

After:

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@chia7712 , thanks for reviewing the PR. Could you also check the PR to fix in kafka-site repo: https://github.com/apache/kafka-site/pull/325. Thanks.

@chia7712 , thanks for reviewing the PR. Could you also check the PR to fix in kafka-site repo: https://github.com/apache/kafka-site/pull/325. Thanks.


fix(github): OAuth code missing sometimes

When GitHub redirects OAuth flows back to us on success, sometimes the code parameter is missing.
Instead of exchanging it for an OAuth token in order to get the app ID (and other details), store the app ID as an environment variable, just just the app name. We don't need OAuth tokens anyway, we generate JWTs.
On the way, we also report an accurate error message if the user tries to initialize a GitHub connection with an app in a GitHub org, but isn't an org admin.
Lastly, we also save more useful debug details in the connection vars.
Refs: INT-7

Instead of exchanging it for an OAuth token in order to get the app ID (and other details), store the app ID as an environment variable, just just the app name. We don't need OAuth tokens anyway, we generate JWTs.

You mean connection variable, right?

No, OS env var for the server's default app - see https://github.com/autokitteh/docs.autokitteh.com/pull/133/files - but the app ID is also saved as a connection var, which will support custom OAuth

will this work with custom oauth?

will this work with custom oauth?

Yes!
This env var will be used only once when creating a connection with the default OAuth (to save it as a connection var).
In all other times the connection var will determine the app ID - and that does align with custom OAuth (custom OAuth connections will specify the app ID when initializing the connection instead of reading the server's env var).
So in effect, connections won't care if they're using default or custom OAuth.


Steamroller program issue.. not accepting the solution even it is passing all the test cases

Challenge Steamroller has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.3; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.103 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

  var resArr=[];
function steamrollArray(arr) {
  for(var i=0;i<arr.length;i++)
    {
      if(Array.isArray(arr[i]))
        steamrollArray(arr[i]);
      else
        resArr.push(arr[i]);
    }

  return resArr;
}

steamrollArray([[["a"]], [["b"]]]);

Global variables are not supported, refer #9199
Please follow along on that thread.
Other than that if you need any coding assistance please visit the JavaScript Chat Room.


-Source not documented very well

Is it possible to specify more than one source? Should it be comma separated? An array?...

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

ID: fd704fe8-fd6e-ba7f-6021-15f280657f4d
Version Independent ID: dc047102-284f-ad70-99ee-dac5be511392
Content: NuGet CLI sources command
Content Source: docs/tools/cli-ref-sources.md
Product: nuget
GitHub Login: @karann-msft
Microsoft Alias: karann

Was just looking for this very thing. Documentation states a list is valid but doesn't show what to use.

Same! This feature can come in handy when restoring nuget in Teamcity.

Also there is no documentation about how to remove a source, how is this done?

I was able to manually remove a source by editing Nuget.config directly but would be nice to have a command to do so.

currently, you can specify only one source when working with the nuget source command.
@Lexcess - can you point me to the statement that led you to believe that a list is valid?

@karann-msft This was nearly a year ago, but as I recall you can specify multiple Source parameters to get the desired effect. Re-reading now I suspect I was mis-reading the 'manages the list of...'.
That said if you can still supply multiple source parameters to get the effect then the documentation is incomplete.

@Lexcess - if you pass more than one source, only the last one is used. So you cannot get a result where you can act on more than one source with a single command.

@karann-msft I don't have access to the source code any more so I'll take your word for it!

@Lexcess - nuget is open source including the nuget client :) - https://github.com/NuGet/NuGet.Client


Use QuantFormat.QOperator by default to avoid fake quantization

Since onnxruntime==1.11.0, quantize_static.quant_format is set to QuantFormat.QDQ by default, which produces fake quantizated models. Explicitly setting it to QuantFormat.QOperator solves this problem.

LGTM!


Install instructions: jupyter_nbextensions_configurator missing

I got this error on OSX when first trying to run jupyterlab: [W 07:47:37.110 LabApp] Error loading server extension jupyter_nbextensions_configurator. Easy workaround: mamba install -c conda-forge jupyter_nbextensions_configurator. Would be great if you could add this to the README.

Hi Tobias, sorry for the late answer.
Can you give me more information? I can't reproduce the error.
Anyway, this extension is not registering a notebook extension, soo I'm not sure where this error comes from.


Enhancement: Debugging fairly difficult - no log output from containers

Right now, as far as I can tell, logs aren't being redirected from things like redfishread.go. In my case, I was having problems with the queue not being created in ActiveMQ. I had to set up my target Linux box as a remote debug target and then debug redfishread.go but the problem is it doesn't have the same DNS namespace so I'm having to resolve that before moving on.

I'm going to fix this with some additional documentation


winget

Brief description of your issue
awful winget does not have texlive install.
provide tex live quality build for free voluntarily.
a true open community would quickly provide great latex install tex live which is used by sharelatex.
but windows is user hostile, and awful with open source.
Steps to reproduce
winget install texlive
Expected behavior
install texlive build that works great out of the bux supports everything no bugs no awful windows microsoft bad engineering behavior
Actual behavior
hate winget, it makes windows feel even more like windows - broken, unpredictable, scary, opaque (as opposed to transparent), full of broken things and leftovers
Environment
idk

everyone I know that gets exposed to winget will quicky get even more frustrated and even more hateful and spiteful of microsoft and msft software in general. waste of time, always, extremely user hostile and breaks stuff

Duplicate of https://github.com/microsoft/winget-pkgs/issues/34998

hadn't seen that issue, why is msft winget package team so hostile to users?


Take Online Notes

Describe the project you want to add with tech stack
Take Online Notes project that is easy to use and to understand. I created this website while learning Frontend Development. I used all the basic concepts of React JS (components, props and hooks) in this project, and my project will help learners to understand the basic concepts of React JS and implementation of the concepts. Records of notes are maintained and saved through Local Storage. I hosted this website using the Azure Static Web Apps service.
Expected behavior
Help beginner in understanding the concepts the React.js by implementing it.
Screenshots (optional)

Additional context (optional )

@prateekj7777 Kindly assign this issue to me under hacktoberfest label.


DataArray creation prone to errors when data shares dimension shapes

It seems like there would be unexpected behavior whenever someone creates a DataArray using data with some subset of dimensions that have the same shape, and when supplying coords as a dictionary of coord_name: coords pairs. Since the dictionary isn't ordered, won't it be unclear which dimension it is referring to?
Maybe that's not such a good description, here's an example:
data = np.random.randn(20, 5, 5)
da = xarray.DataArray(data, {'1': np.arange(20),
                             '2': np.arange(15, 20),
                             '3': np.arange(5, 10)})
print(da.coords)

da = xarray.DataArray(data, {'10': np.arange(20),
                             '11': np.arange(15, 20),
                             '12': np.arange(5, 10)})
print(da.coords)

For the first dimension it's no problem, since it's the only one with length 20. For the 2nd and 3rd dimensions since they are the same shape, the coordinates will be assigned depending on which order python takes the keys, which means they could sometimes be flipped depending on something arbitrary like what you named a dimension.
It seems like this could be addressed by doing a quick check (when supplying coordinates in this fashion) to see if any of the dimensions of the coordinates are the same, and throwing a warning that behavior might be unstable. Or maybe it throws an error and says "if data coordinates are same length, please supply coordinate values in a tuple or list instead"?

I agree, this can be ambiguous. It makes sense for ordered mappings, but not for normal dicts. And unfortunately there is no Pythonic way to tell whether a mapping is ordered or not.
I originally included it because I wanted to make sure operations like xr.DataArray(2 * x.data, x.coords) preserve all original dimensions in order. But, given that we define DataArray.coords.dims and use that to fill in an empty dims argument, we don't really need this anymore. If someone really likes supplying coordinates as an OrderedDict, it's simple enough for them to write coords.items() instead.
So I would support deprecating supplying dict-like coordinates without explicitly provided dims. It's more magic than we really need. This line is in the relevant code path.
I don't want to simply error when coordinates have ambiguous lengths compared to the size of the data, because adding errors in special cases makes things harder to predict.

So I would support deprecating supplying dict-like coordinates without explicitly provided dims. It's more magic than we really need. This line is in the relevant code path.

I've also run into this and I'm :+1: on this solution.

That sounds like a fair plan to me - throwing errors etc definitely
complicates things more than just deprecating the option :)
On Tue, Jan 26, 2016 at 8:58 PM, Joe Hamman<EMAIL_ADDRESS>wrote:

So I would support deprecating supplying dict-like coordinates without
explicitly provided dims. It's more magic than we really need. This line is
in the relevant code path.
I've also run into this and I'm [image: :+1:] on this solution.
—
Reply to this email directly or view it on GitHub
https://github.com/pydata/xarray/issues/727#issuecomment-175399404.

Any interest in putting together a PR? :)
On Tue, Jan 26, 2016 at 9:46 PM, Chris Holdgraf<EMAIL_ADDRESS>wrote:

That sounds like a fair plan to me - throwing errors etc definitely
complicates things more than just deprecating the option :)
On Tue, Jan 26, 2016 at 8:58 PM, Joe Hamman<EMAIL_ADDRESS>wrote:

So I would support deprecating supplying dict-like coordinates without
explicitly provided dims. It's more magic than we really need. This line
is
in the relevant code path.
I've also run into this and I'm [image: :+1:] on this solution.
—
Reply to this email directly or view it on GitHub
https://github.com/pydata/xarray/issues/727#issuecomment-175399404.

—
Reply to this email directly or view it on GitHub
https://github.com/pydata/xarray/issues/727#issuecomment-175414737.

Lemme take a look at the code tomorrow. I'm pretty new to xarray but I've
been meaning to learn it a bit more :-)
On Jan 26, 2016 9:51 PM, "Stephan Hoyer"<EMAIL_ADDRESS>wrote:

Any interest in putting together a PR? :)
On Tue, Jan 26, 2016 at 9:46 PM, Chris Holdgraf<EMAIL_ADDRESS>wrote:

That sounds like a fair plan to me - throwing errors etc definitely
complicates things more than just deprecating the option :)
On Tue, Jan 26, 2016 at 8:58 PM, Joe Hamman<EMAIL_ADDRESS>wrote:

So I would support deprecating supplying dict-like coordinates without
explicitly provided dims. It's more magic than we really need. This
line
is
in the relevant code path.
I've also run into this and I'm [image: :+1:] on this solution.
—
Reply to this email directly or view it on GitHub
https://github.com/pydata/xarray/issues/727#issuecomment-175399404.

—
Reply to this email directly or view it on GitHub
https://github.com/pydata/xarray/issues/727#issuecomment-175414737.

—
Reply to this email directly or view it on GitHub
https://github.com/pydata/xarray/issues/727#issuecomment-175415408.

So it looks like the relevant code is somewhere around here, no? Is "auto-coords" done in many other places in the codebase?
And as far as future behavior, do you imagine just changing the utils.is_dict_like bits so that instead of doing magical stuff under the hood, they just throw an error from now on?

@choldgraf Yes, that looks right to me -- this code path in particular should trigger a DeprecationWarning:
            # try to infer dimensions from coords
            if utils.is_dict_like(coords):
                dims = list(coords.keys())

Cool - I'll make a PR in the next week or two...I know this isn't a big change but have a few deadlines in the coming days.

whoops - that's what I get for not putting this on my to-do list...totally forgot about this issue. My bad, but thanks for fixing!

@choldgraf no worries!


Command to schedule meeting is not implemented

HustleBook users needs to manage meeting with their clients through
the command box.
Let's create a new command "meet" that takes in the parameters of
date and time.
Should close #85.
Note:

Test for meet commands not yet tested.
Need to discuss on how to handle default value of no scheduled
meeting in Json format.
Edit and deleting of scheduled meeting is not implemented yet.

After latest commit, users can edit the scheduled meeting by using the same command to schedule a meeting. It will replace the current scheduled meeting.
This should close #86.

Codecov Report

Merging #112 (b12954a) into master (11a263d) will decrease coverage by 0.63%.
The diff coverage is 34.65%.

@@             Coverage Diff              @@
##             master     #112      +/-   ##
============================================
- Coverage     66.75%   66.11%   -0.64%
- Complexity      442      459      +17
============================================
  Files            81       83       +2
  Lines          1540     1635      +95
  Branches        163      174      +11
============================================
+ Hits           1028     1081      +53
- Misses          457      486      +29
- Partials         55       68      +13

Impacted Files
Coverage Δ

...java/seedu/address/logic/commands/MeetCommand.java
0.00% <0.00%> (ø)

...a/seedu/address/logic/parser/HustleBookParser.java
80.00% <0.00%> (-4.22%)
:arrow_down:

.../seedu/address/logic/parser/MeetCommandParser.java
0.00% <0.00%> (ø)

...in/java/seedu/address/logic/parser/ParserUtil.java
73.43% <0.00%> (-13.60%)
:arrow_down:

src/main/java/seedu/address/model/HustleBook.java
76.19% <0.00%> (-5.87%)
:arrow_down:

src/main/java/seedu/address/model/Model.java
100.00% <ø> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
88.67% <0.00%> (-3.48%)
:arrow_down:

...a/seedu/address/model/person/UniquePersonList.java
77.55% <0.00%> (-5.06%)
:arrow_down:

.../java/seedu/address/model/util/SampleDataUtil.java
20.00% <ø> (ø)

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

... and 9 more

Continue to review full report at Codecov.

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


UI doesn't show up in colab when using the faces dataset

Hi! Thanks for making this code available. I saw your lecture in Ali Jahanian's class on yt and wanted to try this out with faces.
However, when changing the colab to use faces instead of church, the UI doesn't show up when the cell completes. There's no error or anything as far as I can tell, execution just ends without anything happening. Things work correctly when I leave the modelname as church though.
Thanks again for the help!

seems like a colab issue. Im able to run it locally in jupyter but not on Colab. See this issue: https://github.com/davidbau/rewriting/issues/6


Labels are missing in the Product View

Labels and their associated logos, such as Bio, AB, AOC… are missing

The Place to look at is summary.
https://github.com/openfoodfacts/OpenFoodFacts-androidApp/blob/master/app/src/main/java/openfoodfacts/github/scrachx/openfood/fragments/SummaryProductFragment.java

@teolemon I don't find any information about the link of label images.
I see example like:

http://static.openfoodfacts.org/images/lang/fr/labels/bio-europeen.135x90.png
http://static.openfoodfacts.org/images/lang/fr/labels/ab-agriculture-biologique.74x90.svg

But i don't know if there is a standard way to fetch labels images.
I guess we have also to fetch all the labels names (with http://world.openfoodfacts.org/labels.json).

The label API doesn't contains links between labels, like in the website http://fr.openfoodfacts.org/label/bio-europeen

filed: https://github.com/openfoodfacts/openfoodfacts-server/issues/559

filed at https://github.com/openfoodfacts/openfoodfacts-server/issues/560

Good news, the labels images are now available :
https://world.openfoodfacts.org/labels.json


chore: optimized matrix ci

Motivation
https://github.com/apache/pekko/discussions/1354
Verification
Looks like it works!
the main branch's latest commit id: https://github.com/apache/pekko/commit/92b38e72d6a2701da06f483e8cd27953e403240e

main branch ci

1.0.x branch's  latest commit  id: https://github.com/apache/pekko/commit/4f77c8108aaf548a65531d2c8807da13dbba8146

1.0.x branch ci

@pjfanning Could I inquire about a few queries I have regarding this PR?

what's the difference between .../pekko/1.1.0-M1/docs and .../pekko/1.1/docs ??
could we replace them with branch names, like main and 1.0.x?

https://github.com/apache/pekko/blob/67211737daa9b3f6d24ed63563efc78fc9945fef/.github/workflows/publish-1.1-docs.yml#L64-L74

@pjfanning Could I inquire about a few queries I have regarding this PR?

what's the difference between .../pekko/1.1.0-M1/docs and .../pekko/1.1/docs ??
could we replace them with branch names, like main and 1.0.x?

https://github.com/apache/pekko/blob/67211737daa9b3f6d24ed63563efc78fc9945fef/.github/workflows/publish-1.1-docs.yml#L64-L74

We have 10+ git repos running jobs like this and redirect rules set up in apache/pekko-site based on this path format. I would prefer not to have to change everything.
We publish with the full version (1.1.0-M1) and the minor version (1.1).
Have a look at full version docs:

https://pekko.apache.org/docs/pekko/1.1.0-M1/
https://pekko.apache.org/docs/pekko/1.0.3-M1/
https://pekko.apache.org/docs/pekko/1.0.2/
https://pekko.apache.org/docs/pekko/1.0.1/
https://pekko.apache.org/docs/pekko/1.0.0/

Minor version:

https://pekko.apache.org/docs/pekko/1.1/
https://pekko.apache.org/docs/pekko/1.0/

The minor version docs give you the latest for each release line. But with full version, you can get docs for a specific release.


Deploy Test Fails

Jenkins Version: Jenkins ver. 1.588
Code Deploy Plugin Version. 1.8 and 1.9 tested
Process to replicate:
1 - Install plugin manually via the .hpi file.
2- Restarted Jenkins
3 - Create job and add post build action for Code Deploy
4 - Entered required fields
5 - Run Jobs.
6 - Immediate error.
Troubleshooting.
No dependencies are listed, however I checked the plugins/libs folder to confirm the right joda-time version jar was in there. Other than that, not much else to test.
Error Logs:
ERROR: Publisher com.amazonaws.codedeploy.AWSCodeDeployPublisher aborted due to exception
java.lang.NoClassDefFoundError: Could not initialize class com.amazonaws.auth.AWS4Signer
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:57)
at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
at java.lang.reflect.Constructor.newInstance(Constructor.java:525)
at java.lang.Class.newInstance0(Class.java:374)
at java.lang.Class.newInstance(Class.java:327)
at com.amazonaws.auth.SignerFactory.createSigner(SignerFactory.java:121)
at com.amazonaws.auth.SignerFactory.lookupAndCreateSigner(SignerFactory.java:107)
at com.amazonaws.auth.SignerFactory.getSigner(SignerFactory.java:80)
at com.amazonaws.AmazonWebServiceClient.computeSignerByServiceRegion(AmazonWebServiceClient.java:311)
at com.amazonaws.AmazonWebServiceClient.computeSignerByURI(AmazonWebServiceClient.java:284)
at com.amazonaws.AmazonWebServiceClient.setEndpoint(AmazonWebServiceClient.java:160)
at com.amazonaws.services.codedeploy.AmazonCodeDeployClient.setEndpoint(AmazonCodeDeployClient.java:1881)
at com.amazonaws.services.codedeploy.AmazonCodeDeployClient.init(AmazonCodeDeployClient.java:347)
at com.amazonaws.services.codedeploy.AmazonCodeDeployClient.(AmazonCodeDeployClient.java:231)
at com.amazonaws.codedeploy.AWSClients.(AWSClients.java:73)
at com.amazonaws.codedeploy.AWSClients.fromBasicCredentials(AWSClients.java:86)
at com.amazonaws.codedeploy.AWSCodeDeployPublisher.perform(AWSCodeDeployPublisher.java:196)
at hudson.tasks.BuildStepMonitor$2.perform(BuildStepMonitor.java:32)
at hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:770)
at hudson.model.AbstractBuild$AbstractBuildExecution.performAllBuildSteps(AbstractBuild.java:734)
at hudson.model.Build$BuildExecution.post2(Build.java:183)
at hudson.model.AbstractBuild$AbstractBuildExecution.post(AbstractBuild.java:683)
at hudson.model.Run.execute(Run.java:1784)
at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
at hudson.model.ResourceController.execute(ResourceController.java:89)
at hudson.model.Executor.run(Executor.java:240)
Finished: FAILURE

Looks like you are missing the AWS SDK. That's listed as a dependency (which itself would bring in joda time)
https://github.com/awslabs/aws-codedeploy-plugin/blob/master/pom.xml#L53

Thanks for the suggestion. The strange thing is that if I do a fresh jenkins and install just this plugin, it works fine. I guess it is either an issue with Maven getting the depencies or conflicting joda-time.jars, as there are about 20 on this server from other plugins. Anything I can do to test the AWS SDK is installed?

You can look for the JAR in your maven libraries. you can use the dependencies:get plugin to fetch a dependency to make sure maven is pulling it correctly.
It seems like you are having a conflict with your other plugins. I don't think there's much we can do from our end.


Alternative for Dax Stuido DMV $SYSTEM.DISCOVER_CALC_DEPENDENCY in Tabular Editor

What are you trying to achieve?
In Daxstudio one can use select * from $SYSTEM.DISCOVER_CALC_DEPENDENCY to find all objects in the models used in measures.  This DMV does not work in daxstudio, if the pbix file has parameters. To find alternative, i tired this advanced script .  It shows the Measure Name ,  Dependent Table Name. I wonder if this can also show the Column Name used in the measures.

Yes - absolutely. Here's an updated script that shows all direct dependencies for all model measures (tables, columns and measures).
string tsv = "Measure\tObjectName\tObjectParentName\tType"; // TSV file header row

// Loop through all measures:
foreach(var m in Model.AllMeasures) {

    // Get a list of objects referenced by this measure (only direct references):
    var allReferences = m.DependsOn.Select(r => r.Key);

    // Output TSV rows - one for each table reference:
    foreach(var r in allReferences)
        tsv += string.Format("\r\n{0}\t{1}\t{2}\t{3}",
            m.Name,
            r.Name,
            r is ITabularTableObject ? (r as ITabularTableObject).Table.Name : "Model",
            r.GetType().Name
        );
}

tsv.Output();
// SaveFile("c:\\MyProjects\\MeasureDependencies.tsv", tsv); // Uncomment this line to save output to a file

Note that there are some corner cases where this technique produces an incorrect result, since it's based on simple DAX tokenization, and not a full semantic analysis (like the CALC_DEPENDENCIES DMV). For example, if your DAX declares a temporary column with the same name as a measure:
[Sales Amount less Tax] :=
SUMX(
    ADDCOLUMNS(
        'FactSales',
        "Sales Amount", 'FactSales'[SalesAmount] * 0.9
    ),
    [Sales Amount]
)

In this case, Tabular Editor would think that [Sales Amount Less Tax] depends on a measure named [Sales Amount] (if such a measure exists), even though the iteration refers to the [Sales Amount] temporary column. For this reason, a best practice is to explicitly name temporary columns in a way to differentiate them from measures.

Awesome. Thanks a bunch. I works great and also thanks for flagging the limitation.


ARTEMIS-2781 return only local queue names for AddressControl.getQueu…

…eNames()

@jbertram this will need to be reverted.
Not only the semantic of getQueues is changing, there are methods inside the broker needing this.
For instance destroyTopic tests are failing because only local queues are now destroyed
I suggest a new method getLocalQueues is created and the current returns the combined version.

followed up with https://github.com/apache/activemq-artemis/pull/3151

and then followed up with #3152


feat: add openid in keycloak scope

Bug fix related to issue: #1219

hi @yos1p, thanks for your contribution! can you please fix the failing test and ensure that your PR follows the contributing guidelines please?

@kangmingtay Should be good now with the test. Thank you!

Hi @yos1p, what if the application is using older versions of keycloak? would specifying additional scopes like the openid scope result in an error returned? this would break backward compatibility and i would much rather just highlight this in the docs - we allow one to pass in the scopes through the signInWithOAuth method (https://supabase.com/docs/reference/javascript/auth-signinwithoauth)

Closing as this is backward incompatible.


Add field name to ValueError raised by field adapter

The use of "f" string formatting is failing in both Python 2.7 and 3.5. You should swap this with "".format(), but otherwise this looks good! Thanks for raising a PR.

Thank you! I'll add a test for this error case to stop Coveralls from complaining.

I wonder if this is worth adding to KeyError and IndexError too since it would make uncaught exceptions in i2cdevice a little more useful to end-users. 🤔


Bug with pillow==10.0.0 and detectron2==0.6

Describe the bug
Algorithms from detectron2 do not load and leads to : module 'PIL.Image' has no attribute 'LINEAR'
To Reproduce
Installing ikomia==0.9.2 which has a requirement pillow>=8.1 installs pillow 10.0.0 which is incompatible with detectron2 0.6
Expected behavior
A clear and concise description of what you expected to happen.
Logs/Screenshots
If applicable, add logs/screenshots to help explain your problem. Ikomia logs are save in $HOME/Ikomia/log.txt.
Environment:

OS: [e.g. Windows11, Ubuntu 22.04]
Python version [e.g. 3.7, 3.8, 3.9, 3.10]
Ikomia API version 0.9.0

Additional context
Add any other context about the problem here.

Maybe we could add upper limit for Pillow (< 10.0.0) in Ikomia-dev/detectron2?
Ikomia API should be compatible with recent versions of Pillow.


Adding ANERcorp-CAMeLLab dataset

Adding ANERcorp dataset
Adding a Dataset

Name: ANERcorp-CAMeLLab
Description: *Since its creation in 2008, the ANERcorp dataset (Benajiba & Rosso, 2008) has been a standard reference used by Arabic named entity recognition researchers around the world. However, over time, this dataset was copied over from user to user, modified slightly here and there, and split in many different configurations that made it hard to compare fairly across papers and systems.

In 2020, a group of researchers from CAMeL Lab (Habash, Alhafni and Oudah), and Mind Lab (Antoun and Baly) met with the creator of the corpus, Yassine Benajiba, to consult with him and collectively agree on an exact split, and accepted minor corrections from the original dataset. Bashar Alhafni from CAMeL Lab working with Nizar Habash implemented the decisions provided in this release.*

Paper: *(a) Benajiba, Yassine, Paolo Rosso, and José Miguel Benedí Ruiz. "Anersys: An Arabic named entity recognition system based on maximum entropy." In International Conference on Intelligent Text Processing and Computational Linguistics, pp. 143-153. Springer, Berlin, Heidelberg, 2007.

(b)Ossama Obeid, Nasser Zalmout, Salam Khalifa, Dima Taji, Mai Oudah, Bashar Alhafni, Go Inoue, Fadhl Eryani, Alexander Erdmann, and Nizar Habash. "CAMeL Tools: An Open Source Python Toolkit, for Arabic Natural Language Processing." In Proceedings of the Conference on Language Resources and Evaluation (LREC 2020), Marseille, 2020.*

Data: https://camel.abudhabi.nyu.edu/anercorp/
Motivation: This is the standard dataset for evaluating NER performance in Arabic*

Instructions to add a new dataset can be found here.


📝 PLAT-897 Allow separate signing and wrapper keys

Proposed Changes
PLAT-897

Add a second 'signerPublicKey' field to an EO
This is an ephemeral public key a client may use to sign rewrap and other requests associated with the EO.
This is required as some algorithms and key types are more suited for encryption and others for signatures. Notably, we must support this for the smaller keys and restricted set of algorithms that NanoTDF will likely impose
Implementation:

Client nanotdf for javascript
Client c++
Service OpenStack EAS and KAS (python)

Checklist

[ ] A clear description of the change has been included in this PR.
[ ] The changelog has been updated.
[ ] All schema validation tests have been updated appropriately and are passing.
[ ] Version numbers have been updated as per the Versioning Guidelines.
[ ] Major/minor version changes only: A writeup has been included discussing the motivation and impact of this change.
[ ] Major/minor version changes only: The minimum wait time has elapsed.
[ ] Project version changes only: I have updated both the VERSION file and the git tag.
[ ] This change otherwise adheres to the project Contribution Guidelines.
[ ] Every item in this checklist has been completed.

Is this more clear?


Add schedule

Test cases for ScheduleCommand and ScheduleCommandParser will be added at a later date

Lgtm, proceed to merge with add-schedule-command branch


[WIP] Fix SSLKsFactory issue

@justinchuch I've merged this to a branch on nosqlbench for now. I'll work on fixing it up once I get the next mainline push ready, perhaps tonight.


istio upgrade e2e tests using either helm or istioctl

https://github.com/istio/istio/issues/17649
https://github.com/istio/istio/issues/17670
This PR is relying on this new istioctl x upgrade command: https://github.com/istio/operator/pull/348

please add some output sample here.

/LGTM

@carolynhu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-istioio-k8s-tests_istio
8020623ee8ff1ba9736e9816964040979feba11b
link
/test integ-istioio-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@carolynhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-istioio-k8s-tests_istio
8020623ee8ff1ba9736e9816964040979feba11b
link
/test integ-istioio-k8s-tests_istio

integ-new-install-k8s-tests_istio
8020623ee8ff1ba9736e9816964040979feba11b
link
/test integ-new-install-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@carolynhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-istioio-k8s-tests_istio
8020623ee8ff1ba9736e9816964040979feba11b
link
/test integ-istioio-k8s-tests_istio

integ-new-install-k8s-tests_istio
8020623ee8ff1ba9736e9816964040979feba11b
link
/test integ-new-install-k8s-tests_istio

integ-security-k8s-tests_istio
c73e6bf25c9ae4c09217e9717977545af0f5f39d
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@carolynhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-istioio-k8s-tests_istio
8020623ee8ff1ba9736e9816964040979feba11b
link
/test integ-istioio-k8s-tests_istio

integ-new-install-k8s-tests_istio
8020623ee8ff1ba9736e9816964040979feba11b
link
/test integ-new-install-k8s-tests_istio

integ-security-k8s-tests_istio
c73e6bf25c9ae4c09217e9717977545af0f5f39d
link
/test integ-security-k8s-tests_istio

integ-mixer-k8s-tests_istio
c73e6bf25c9ae4c09217e9717977545af0f5f39d
link
/test integ-mixer-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@carolynhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-istioio-k8s-tests_istio
8020623ee8ff1ba9736e9816964040979feba11b
link
/test integ-istioio-k8s-tests_istio

integ-new-install-k8s-tests_istio
8020623ee8ff1ba9736e9816964040979feba11b
link
/test integ-new-install-k8s-tests_istio

integ-security-k8s-tests_istio
c73e6bf25c9ae4c09217e9717977545af0f5f39d
link
/test integ-security-k8s-tests_istio

integ-mixer-k8s-tests_istio
c73e6bf25c9ae4c09217e9717977545af0f5f39d
link
/test integ-mixer-k8s-tests_istio

unit-tests_istio
c73e6bf25c9ae4c09217e9717977545af0f5f39d
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest
Known flake https://github.com/istio/istio/issues/18385


truncates breadcrumb navigation trail

Coverage remained the same at 27.509% when pulling b697384325016bb5818f2e61ce1d266b910d4489 on theseyi:truncated-breadcrumbs-trail into 2509b1ce776313dc520bd4192f1bb9a0369dea8f on linkedin:master.

Codecov Report

Merging #1333 into master will increase coverage by 0.35%.
The diff coverage is 76.19%.

@@            Coverage Diff            @@
##           master   #1333      +/-   ##
=========================================
+ Coverage   54.25%   54.6%   +0.35%
=========================================
  Files         181     182       +1
  Lines        3747    3765      +18
  Branches      830     833       +3
=========================================
+ Hits         2033    2056      +23
+ Misses       1714    1709       -5

Impacted Files
Coverage Δ

wherehows-web/app/helpers/split-text.ts
100% <100%> (ø)
:arrow_up:

...b/app/components/datasets/urn-breadcrumbs/crumb.ts
100% <100%> (ø)

...ows-web/app/components/datasets/urn-breadcrumbs.ts
80% <66.66%> (-20%)
:arrow_down:

wherehows-web/app/routes/browse/entity.ts
36.36% <0%> (+27.27%)
:arrow_up:

wherehows-web/app/controllers/browse/entity.ts
100% <0%> (+100%)
:arrow_up:

Continue to review full report at Codecov.

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


Add documentation on how to join TestNet

This will be used by people in our company and ultimately, externally.

Depends on https://github.com/bosagora/agora/pull/3015
Need the IBD screenshot.

Should be G2G.

Updated CI script

First commit message relates to a “previous” commit

s/The/A/


vscode addon for formatting and highlighting

it would be nice if you made a vscode addon for Nilesoft Shell Script (.shl).
it would add simple text highlighting and support for:

collapsing {...} and multi-line items
maybe IntelliSense

(also, it would be nice if you had a discord server👍)

i'm looking into it and i think i might beable to do it myself.

This would be a really great contribution

@moudey could you create a discord server so we could chat about this on there?


docs: remove duplicate manpage for bootc-fetch-apply-updates.service

Hm, looks like the labeler didn't add documentation here.  It's running the full suite and actually skipped the docs job :(

Hm, looks like the labeler didn't add documentation here. It's running the full suite and actually skipped the docs job :(

I think #986 fixes this


Please consider adding the licensing level needed for this feature.

[Enter feedback here]
Please consider adding the licensing level needed.  I have Microsoft 365 Business Premium. It has some of the Purview features, but not all.  I know this feature is new.  In some screens in the M365 portal, it shows me has having an E3 license.  I cannot access the "devices" feature listed in this article with my M365 Business Premium license - it does not show.
thx

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

ID: 76da4eda-4ccf-ff34-6a48-d9c5553c2ed6
Version Independent ID: 7e887aff-1b85-1651-2bff-63425c54ae63
Content: Using Endpoint DLP - Microsoft Purview (compliance)
Content Source: microsoft-365/compliance/endpoint-dlp-using.md
Service: o365-seccomp
GitHub Login: @chrfox
Microsoft Alias: chrfox

@thomasjsweet Thank you for your feedback. I have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/9135 and would like to have the author review and approve before the changes go live.
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @thomasjsweet Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


terminal1 combined with terminal2 doesn't work as feature set

Similar to issue #53. When terminal 1 is combined with terminal2 a similar error happens. It seems to be specific to just this combination, I tried a few other combinations and they worked. I assume there's a similar problem with the logic, but I couldn't make sense of it when I looked at the code so i can't provide a code solution.
Example code below:
# Setup
from glycowork.motif.analysis import get_heatmap
import pandas as pd
data = {
    'Glycan': ['Gal(b1-3)GalNAc', 'GalOS(b1-3)GalNAc', 'Gal(b1-3)[Fuc(a1-?)]GalNAc', 'GlcNAc(b1-2)Man(a1-3)Man', 'Man(a1-6)[Man(a1-3)]Man(b1-4)GlcNAc(b1-4)GlcNAc', 'Neu5Ac(a2-6)Gal(b1-4)GlcNAc(b1-2)Man', 'Neu5Ac(a2-3)Gal(b1-3)GalNAc'],
    'Sample1': [1.1, 0.2, 0.3, 0.5, 0.7, 1.0, 0.6],
    'Sample2': [1.2, 0.1, 0.2, 0.4, 0.8, 0.9, 0.5],
    'Sample3': [0.1, 1.8, 1.9, 0.3, 0.6, 0.8, 1.2],
    'Sample4': [0.2, 1.1, 1.2, 0.2, 0.5, 0.7, 1.1],
    'Sample5': [1.3, 0.3, 0.4, 0.6, 0.9, 1.1, 0.7],
    'Sample6': [1.4, 0.4, 0.5, 0.7, 1.0, 1.2, 0.8],
    'Sample7': [0.3, 1.9, 2.0, 0.4, 0.7, 0.9, 1.3],
    'Sample8': [0.4, 1.2, 1.3, 0.3, 0.6, 0.8, 1.2]
}
data = pd.DataFrame(data)

# This works
get_heatmap(data,
           motifs = True,
           feature_set=[
               'terminal1',
               #'terminal2',
               #'terminal3'
                       ])

# This works
get_heatmap(data,
           motifs = True,
           feature_set=[
               #'terminal1',
               'terminal2',
               #'terminal3'
                       ])

# This fails
get_heatmap(data,
           motifs = True,
           feature_set=[
               'terminal1',
               'terminal2',
               #'terminal3'
                       ])

# This works
get_heatmap(data,
           motifs = True,
           feature_set=[
               'terminal1',
               #'terminal2',
               'terminal3'
                       ])

# This works
get_heatmap(data,
           motifs = True,
           feature_set=[
               #'terminal1',
               'terminal2',
               'terminal3'
                       ])

# This fails
get_heatmap(data,
           motifs = True,
           feature_set=[
               'terminal1',
               'terminal2',
               'terminal3'
                       ])

Done with commit #db6c19d12088e44a68df6c6335b10e6972ac2c91

Thanks! So the logic worked fine actually; the issue was with the grouping of features, where nested lists were not handled properly with terminal1+terminal2 combination
Fixed with f6ff830 (I've tried all combinations with that example and everything worked)


[FEATURE REQUEST] Make armor values of alfsteel armor modifiable via config

As title says, for balancing purposes I would like to change these values

Also wouldn't mind an option where it requires mana to use the ranged attack since that would make sense.

It does require mana. However that mana is taken from its durability, just like it's done with the terra blade.

Want it in a pack I'm making but my only option at the moment is to remove the mod.

What do you want to achieve with that? Well it sounds like your goal is to "force" me to add the config option. In that case: Do you really think, modders make mods for people to put them in a modpack? Do you think, my primary goal is to have my mods in as many modpacks as possible, and thus, I'll do anything for a modpack developer to get my mod in?
I have do disillusion you here. I make mods, because I enjoy making mods. And modpack makers should make modpacks because they enjoy making modpacks. Of course, if there is a feature that would be useful for a modpack, don't bother asking about it. Asking, however is fundamentally different from threatening with removal of the mod from your pack.
So don't expect anyone (including me) to care about that. I do care about feature requests that would make my mod better. And if they help a modpack maker as well, that's even better. But threatening with mod removal is definitely the wrong way.


Scroll to top on page change

[x] Code builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I didn't break any original functionality

Optional

[x] I've tested it on mobile and everything works

PR type

[x] Bugfix

What's new? (maybe part of changelog)

closes #567  - Added scroll top on page change

WFM


Find all should actually find all

find_all originally only returned 1 element, the first element it found. Because it was using the wrong callback, which terminated iteration after the first match was found.
Tested on local project, works great!

Yeah, you are right.


No video stream running on Raspberry

I use MotionEye on my raspberry for my camera so I got the stream URL to <IP_ADDRESS>:8081
I connected homebridge on my iPhone and the camera works with snapshots but when I want to see the live stream I got this error :
[10/30/2019, 12:08:58 AM] [Camera-ffmpeg] Snapshot from Camera Surveillance at 480x270
[10/30/2019, 12:09:09 AM] [Camera-ffmpeg] Start streaming video from Camera Surveillance with 1280x720@299kBit
[10/30/2019, 12:09:09 AM] [Camera-ffmpeg] ERROR: FFmpeg exited with code 1
[10/30/2019, 12:09:24 AM] [Camera-ffmpeg] Start streaming video from Camera Surveillance with 1280x720@299kBit
[10/30/2019, 12:09:24 AM] [Camera-ffmpeg] ERROR: FFmpeg exited with code 1

Here's my config.json
{
	"bridge": {
		"name": "Homebridge",
		"username": "CC:22:3D:E3:CE:30",
		"port": 51826,
		"pin": "031-45-154"
	},

	"description": "This is an example configuration file with one fake accessory and one fake platform. You can use this as a template for creating your own configuration file containing devices you actually own.",
	"ports": {
		"start": 52100,
		"end": 52150,
		"comment": "This section is used to control the range of ports that separate accessory (like camera or television) should be bind to."
	},
	"platforms": [{
		"platform": "Camera-ffmpeg",
		"cameras": [{
			"name": "Camera Surveillance",
			"videoConfig": {
				"source": "-re -i http://user:password@<IP_ADDRESS>:8081",
				"vcodec": "h264_omx",
				"maxStreams": 1,
				"maxWidth": 480,
				"maxHeight": 270,
				"maxFPS": 15
			}
		}]
	}]
}

I fixed it by refering the correct ffmpeg by adding "videoProcessor": "/usr/bin/ffmpeg",
see my config.json now :
{
	"bridge": {
		"name": "Homebridge",
		"username": "CC:22:3D:E3:CE:30",
		"port": 51826,
		"pin": "031-45-154"
	},

	"description": "This is an example configuration file with one fake accessory and one fake platform. You can use this as a template for creating your own configuration file containing devices you actually own.",
	"ports": {
		"start": 52100,
		"end": 52150,
		"comment": "This section is used to control the range of ports that separate accessory (like camera or television) should be bind to."
	},
	"platforms": [{
		"platform": "Camera-ffmpeg",
		"videoProcessor": "/usr/bin/ffmpeg",
		"cameras": [{
			"name": "Camera Surveillance",
			"model": "MotionEye",
			"videoConfig": {
				"source": "-re -i http://user:password@<IP_ADDRESS>:8081",
				"vcodec": "h264_omx",
				"maxStreams": 2,
				"maxWidth": 1280,
				"maxHeight": 720,
				"maxFPS": 15
			}
		}]
	}]
}

This is my config.json, but I still see no images on my cam, only a red !
    {
	"platform": "Camera-ffmpeg",
	"videoProcessor": "/usr/bin/ffmpeg",
	"cameras": [{
		"name": "Spycam",
		"manufacturur": "Raspberry Pi",
		"model": "Motion Eye",
		"videoConfig": {
			"source": "-re -i mjpeg http://admin:*******@<IP_ADDRESS>:8081",
			"stillImageSource": "-f mjpeg -i http://http://admin:*******@<IP_ADDRESS>:8765/picture/1/current/",
			"vcodec": "h264_omx",
			"maxStreams": 2,
			"maxWidth": 1280,
			"maxHeight": 720,
			"maxFPS": 30
		}

But I don't see ffmpeg in /usr/bin/

This is my config.json, but I still see no images on my cam, only a red !
    {
	"platform": "Camera-ffmpeg",
	"videoProcessor": "/usr/bin/ffmpeg",
	"cameras": [{
		"name": "Spycam",
		"manufacturur": "Raspberry Pi",
		"model": "Motion Eye",
		"videoConfig": {
			"source": "-re -i mjpeg http://admin:*******@<IP_ADDRESS>:8081",
			"stillImageSource": "-f mjpeg -i http://http://admin:*******@<IP_ADDRESS>:8765/picture/1/current/",
			"vcodec": "h264_omx",
			"maxStreams": 2,
			"maxWidth": 1280,
			"maxHeight": 720,
			"maxFPS": 30
		}

And/but I don't see ffmpeg in /usr/bin/
And did you something to the config of Motion Eye?

Remove the mjpeg in source
Run whereis ffmpeg to locate your ffmpeg
And run ffmpeg -codecs to se if h264_omx is  supported

I think we've got the problem...
pi@raspberrypi:~ $ whereis ffmpeg
ffmpeg:
pi@raspberrypi:~ $
pi@raspberrypi:~ $ ffmpeg -codecs
bash: ffmpeg: command not found
pi@raspberrypi:~ $
I need to figure out how I can install ffmpeg! :D
I did change some things in MotionEye... shouldn't I have? How can I reset them to default?


Empty MultiLineString has "GeometryCollection" as type attribute

When building an empty MultiLineString, the type attribute returns "GeometryCollection" instead of "MultiLineString".
Steps to reproduce the problem.
import shapely
from shapely.geometry import Point, LineString, MultiLineString

print(shapely.__version__) # returns 1.8.3

point1 = Point(0, 0)
point2 = Point(1, 1)
point3 = Point(2, 2)

line1 = LineString([point1, point2])
line2 = LineString([point2, point3])

multi_line_string1 = MultiLineString([line1, line2])
multi_line_string2 = MultiLineString()

print(type(multi_line_string1)) # returns <class 'shapely.geometry.multilinestring.MultiLineString'>
print(type(multi_line_string2)) # returns <class 'shapely.geometry.multilinestring.MultiLineString'>

print(multi_line_string1.type) # returns "MultiLineString"
print(multi_line_string2.type) # returns "GeometryCollection"

I have also noticed that the empty method  changes the type attribute:
multi_line_string1 = MultiLineString([line1, line2])

print(multi_line_string1.type) # returns "MultiLineString"

multi_line_string1.empty()

print(multi_line_string1.type) # returns "GeometryCollection"

Which makes me think that this is wanted behavior rather than an issue.
Operating system
Ubuntu 20.04
Shapely version and provenance
1.8.3 installed from PyPI using pip

This inconsistency is discussed in #742, and other geometry type behave similarly, e.g. Polygon() also returns GeometryCollection. If you expect "MULTILINESTRING EMPTY" then you would need to load it from WKT.
With Shapely 2.0, this behavior is fixed, and MultiLineString() will keep its intended geometry type.

Thank you for you answer and sorry for not referencing this issue!
I was a bit puzzled by some inconsistencies like this one and glad to here that they will be fixed !


Modified test target name (to test-all)

As this target depends on external tools, and packagers run "make test",
I think the name should be less generic.

Packers do run make test. The external tools are build dependencies. Tests that aren't suitable in this context are not included in make test (i.e. make vgtest and make vloghtb).


chore: update README.md to recent changes

Has been superseded by our move to a separate repository.


Improvements

I noticed that these hasTime, hasDate, hasSeconds functions are used in file DateRangePicker, but never were declared. So when I pass null ->format(null) an exception is thrown.

Changes:

Removed calls to functions that do not exist hasTime, hasDate, hasSeconds.
Added a label for the selected range. (does not affect the state, only visual). ->useRangeLabels()

https://github.com/malzariey/filament-daterangepicker-filter/assets/142592979/8e6fc85b-35fb-4082-8383-5e50876f78a0

added separator option. ->separator(' - ')
added an option to disable custom range. ->disableCustomRange()
formatted the daterangepicker.js file a little.
added the hiddenLabel option to filter
added the placeholder option to filter

Thanks for your great addition.


[Create New Issue]:JAVA TEMPLEATE TO RUN THE CODE IN 1 SEC

Description
Description:JAVA TEMPLEATE TO RUN THE CODE IN 1 SEC
Domain
Competitive Programming
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign