Incorrect node name for nested components

The node names changed after upgrading from react 16 with official enzyme-adapter-react-16.
Dependencies:
        "react": "^17.0.1",
        "@wojtekmaj/enzyme-adapter-react-17": "^0.4.1",
        "enzyme": "^3.11.0",
        "enzyme-to-json": "^3.6.1",

The following test can reproduce the problem (using jest):
    test.only('Nested component snapshot', () => {
        // Arrange
        const DummyComponent = (_, ref) => <div ref={ref} />;
        const MemoForwardComponent = React.memo(React.forwardRef<HTMLDivElement>(DummyComponent));
        const Component: React.FC = () => <MemoForwardComponent />;

        // Act
        const wrapper = shallow(<Component />);

        // Assert
        expect(wrapper).toMatchInlineSnapshot(`<Memo(ForwardRef(DummyComponent)) />`);
    });

This React 16 is succeeding, but with React 17 with @wojtekmaj/enzyme-adapter-react-17 v0.4.1 is failing with the following message:
  ● Nested component snapshot

    expect(received).toMatchInlineSnapshot(snapshot)

    Snapshot name: `Nested component snapshot 1`

    - Snapshot  - 1
    + Received  + 1

    - <Memo(ForwardRef(DummyComponent)) />
    + <Memo([object Object]) />

      19 |
      20 |     // Assert
    > 21 |     expect(wrapper).toMatchInlineSnapshot(`<Memo(ForwardRef(DummyComponent)) />`);
         |                     ^
      22 | });
      23 |
      24 |

Seems the
This is related to the following issue in the official enzyme react 16 adapter: https://github.com/enzymejs/enzyme/issues/2481

Gotta love "self-fixing" bugs :D I appreciate your input mate!


Implement RPC endpoint to submit proven transactions

### Tasks
- [ ] Add the proto message for the proved transaciton ([ref](https://github.com/0xPolygonMiden/miden-base/blob/main/objects/src/transaction/proven_tx.rs))
- [ ] Add endpoint `submit_transaction` to rpc.proto and the request/response messages  (additional details https://github.com/0xPolygonMiden/miden-base/discussions/121)
- [ ] Add endpoint `submit_transaction` to store.proto and the request/response messages
- [ ] Add code to validate the submitted transaction

Another assumption that I added above, is that the transaction is saved to the store. Having it on the mempool only would make it harder for us to operate the node (unexpected restarts would lose data, and graceful restarts would need synchronize the RPC/mempool/provers)

An open question about the proof validator. Is it a code component embedded on the RPC server, or is it another server?

I'm not sure about this yet. In its final form, this component should be simple enough to be a part of the RPC server. However, initially, we'd want to have "training wheels" and as such, the node would re-execute every proven transaction (but not prove it) to make sure they get the same result as is attested by the proof. So, it may make sense to have both, transaction executor and transaction validator be in the same component.
Btw, re-execution is something that we'll enable after v0.1.

is this done?

is this done?

Nope, it is not merged yet #85
https://github.com/0xPolygonMiden/miden-node/pull/85/commits/956b8f64f0513cb01ef2a929eea6b291659260de


Add indexes for Neo4j

Closes #28.
The aim of this PR is to see if adding indexes makes Neo4j queries run any faster w.r.t Kùzu. I've added the following indexes:
CREATE INDEX country IF NOT EXISTS FOR (c:Country) ON (c.country)
CREATE INDEX city IF NOT EXISTS FOR (ci:City) ON (ci.city)
CREATE INDEX age IF NOT EXISTS FOR (p:Person) ON (p.age)

Closing as this isn't really helping achieve the overall goal without giving special treatment to Neo4j.


how i do a secret song

What is your question?
How do I make a song that to play it you have to press a specific arrow
for example if I press a notetype that its function is to take me to that song, it takes me to that song.
(I use the google translator sorry if I wrote bad)

also i use .lua

use loadSong(name, difficultynumber)


Better logic for DiagnosticsReport.passed method

Related to https://github.com/NebraLtd/Hotspot-Production-Tool/issues/55
Moved logic of did_miner_pass_diagnostics from Miner class in HPT to
DiagnosticsReport.passed method. Added relevant test cases.
Issue
Avoid code duplication and use better logic for determining if diagnostics have passed.

Link:
Summary:
Previously only the passed key was being checked to see if diagnostics report had passed. Now this checks more keys to make sure all keys required for passing the report at manufacturing have valid values.

How
Implementation logic from https://github.com/NebraLtd/Hotspot-Production-Tool/blob/kashifpk/refactoring/app/src/hpt/miner.py#L165 was taken and moved to DiagnosticReport.passed method with appropriate adjustments.

[ ] Documentation added
[X] Thought about variable and method names

@kashifpk not sure if you realised but you have to create a release using the instructions here also for it to get pushed to PyPI...
https://github.com/NebraLtd/hm-pyhelper#releasing

@shawaj - Yes was aware that will need to create a release but didn't know the documentation link. Thanks for that. Actually I'll make a few small changes before releasing the new version otherwise it might break hm-diag as we're trying to use friendly key names in hm_pyhelper while hm-diag uses old style 2 character key names. So updating code for that and will make a release after that.

hm-pyhelper v0.12.5 released.


[PE-D][Tester A] Ability to edit address does not reformat the integers before storing.

editstu 2 a/000000000000000000000000000000000000000000000000
Consider this valid edit. I am expecting the result is changing the attendance to 0/10.
Instead, the chunks of 0s are directly treated as a string instead of truncating and simplifying the 0s which is the expected behavior if a tutor is using the app.

Labels: severity.Medium type.FunctionalityBug
original: Eclipse-Dominator/ped#9

updated ug


Incorrect changes when using ReplaceAroundStep

When wrapping some existing change with for example a blockquote, the original change is adjusted incorrectly.
Reproduction

                               []
[insert ab]                           + [1, 1, 1, 3]

ab
                             = [1, 1, 1, 3]
[wrap with blockquote]                + [[0, 0, 0, 1], [4, 4, 5, 6]]

ab

    = [[0, 0, 0, 1], [3, 3, 5, 6], [1, 1, 4, 6]]

So the merge of [[1, 1, 1, 3]] with  [[0, 0, 0, 1], [4, 4, 5, 6]] is incorrect. Expected final change would be: [[0, 0, 0, 1], [1, 1, 2, 4], [2, 2, 5, 6]]
To investigate this we tried to simulate a wrap by inserting empty paragraphs around a change. The difference between a ReplaceAroundStep and two wrapping inserts is the resulting StepMap.
ReplaceAroundStep gives a single StepMap: [0 0 1 4 0 1] whereas two inserts in a single transaction give two StepMaps: [0 0 1] + [5 0 1].
This results in different stepChanges in the addSteps method:

ReplaceAroundStep StepMap forEach gives: [[0, 0, 0, 1], [4, 4, 5, 6]]
Two inserts StepMaps forEach gives: [[0, 0, 0, 1], [5, 5, 5, 6]]

Because Change.merge handles these two inputs differently, the resulting merged Changes is incorrect for the ReplaceAroundStep. It is correct for two inserts transaction.
Tests
The tests to reproduce this issue are available here: https://github.com/floorish/prosemirror-changeset/commit/2ad1e8f8f9aa6dab29ad3c2b2685e09b563a0339
Tests which describe the non-overlapping merges are available here:
https://github.com/floorish/prosemirror-changeset/commit/ff0837e2748eb03cc7a6f798c6cf872c5fbecfb5
Possible solutions
A
The forEach function on StepMap needs adjustment, such that an input of [0 0 1 4 0 1] returns [[0, 0, 0, 1], [5, 5, 5, 6]]
Implementation of this solution is available here:
https://github.com/floorish/prosemirror-transform/commit/d666b124c955a0c9a77c282e80ec369666b34ae9
B
StepMap needs adjustment.

Insert + Insert gives: [0 0 1] + [5 0 1]. When remapping pos 4 we'll get: mapping->map(4) = 5
ReplaceAround gives:   [0 0 1 4 0 1]. When remapping pos 4 we'll get: mapping->map(4) = 6

This seems like inconsistent behaviour, since start-doc and end-doc are in both cases the same.
C
The stepChanges creation in addSteps method needs adjustment, such that an input of [0 0 1 4 0 1] returns [[0, 0, 0, 1], [5, 5, 5, 6]]
Implementation of this solution is available here: https://github.com/floorish/prosemirror-changeset/commit/717aaf6d6cae10142327f8ed48d87d0940727483
D
The Change.merge method currently adjusts the fromA and toA values, so merging [0, 0, 0, 2] + [6, 6, 6, 8] gives [[0, 0, 0, 2], [4, 4, 6, 8]]. Maybe something can be changed here such that it keeps the original fromA and toA values?
These results all differ, which is expected?
merge [[1, 1, 1, 3]] + [[0, 0, 0, 2], [4, 4, 6, 8]]
= [[0,0,0,2],[1,1,3,5],[2,2,6,8]]

merge [[1, 1, 1, 3]] + [[0, 0, 0, 2], [6, 6, 6, 8]]
= [[0,0,0,2],[1,1,3,5],[4,4,6,8]]

@marijnh Perhaps any comments on this issue?
Currently were using fix C: https://github.com/floorish/prosemirror-changeset/commit/717aaf6d6cae10142327f8ed48d87d0940727483
If you want I can create a pull request for that as well, but perhaps a solution for B is better.

The forEach function on StepMap needs adjustment, such that an input of [0 0 1 4 0 1] returns [[0, 0, 0, 1], [5, 5, 5, 6]]

That method is defined as producing position referring to the pre- and post-step documents, which would be violated by this, so that doesn't seem like a good direction.
This does indeed look like a bug in StepMap, and your patch https://github.com/floorish/prosemirror-changeset/commit/717aaf6d6cae10142327f8ed48d87d0940727483 looks reasonable—blindly putting changes at the returned positions, as the code was doing before, isn't safe.

I've merged your patch as 54e7214a82abed5 and added the test in 1b665fa5ee5eafc7

Cheers, thanks!


initial work with event streams

Not working on at the moment, was a cool idea but meh seems we are going a different route.


No task lists

So this seems to be an issue with the graph API itself.
Long story short I originally was getting only the "Flagged Emails" list back from the extension even though i have over 20 lists in my app. I then used the Graph API explorer and confirmed the same - only the flagged emails list in the API response.
Now when I turned "Flagged Emails" on then off in the Windows App it stopped returning any lists at all. See below:

I then found this issue which told me to try the delta endpoint instead: https://learn.microsoft.com/en-us/answers/questions/594742/microsoft-graph-todo-api-doesnt-return-all-task-li

So clearly theres some issue with the actual API itself (maybe I have too many lists), but I thought it was worth creating this issue in case it would be better to use the delta endpoint instead?

Hmm, not an easy one since all the calls are made in an external library 'python-o365'.  I probably won't get to look at it until the week after next.

Thank you for doing some digging on this.
A quick look at this after I have got home... This would require a change in the O365 library that the integration uses, that would take some time (it just always does).
I have someone else reporting no todo lists, which may or may not be related since they haven't responded to my queries. If it is the same problem, then it will likely warrant the work on the O365 library.

Thanks for digging. I figured that would be the case. The workaround for now was to just manually take my list id's from the delta endpoint and manually update the yaml file.


Remove additional ../.. from dist on npm start dev

With the current path it keep giving me a 404 as the demo index.html is trying to reach dist/component.js but we're exposing ../../dist/componet.js instead.

Works, thanks!


Serving of simple text not updating in Docker, cached text displayed instead

Using webify in docker, I'm seeing cached values for simple text:
script.sh:
#!/bin/sh

echo foo

docker run -d --rm -p 8080:80 -v  $(realpath script.sh):/script beefsack/webify

If I change foo to bar, curl http://localhost:8080 still displays the old value.
This only occurs in Docker, if I run webify on baremetal, the updated value bar is displayed.

Hi @mihaigalos!
Can you output the content of the file running in the container, and check if the file inside the container is updated?
You can do this using something like:
docker exec -it 3d6e55c660a0 cat /script

Sometimes file mounts fail to update in docker containers because some editors use a separate temporary file while editing, then copy it into place once you complete, and Docker won't update the mounted file in the container in cases like this. This is a known issue with Docker and won't be fixed.
One workaround if you want to use a host mount is to mount a directory instead of the file directly. You could do that using something like:
docker run -d --rm -p 8080:80 -v /path/to/script/dir:/mnt -e SCRIPT=/mnt/script.sh beefsack/webify

Oh I see.
I use vim which effectively changes the inode because of writing to a separate file and replacing the original, which doesn't reflect in Docker.
Your solution with mounting the whole folder+env var worked - thank you!


alt+0 (to do "@") it changes the theme

Operating system:
Messenger for Desktop version:

when i alt+0 (to do "@") it changes my theme, so I have to change it again by doing alt+3 ('#')
I know it's not much but it's really annoying since i often write "@" for emails and stuff

👍 Very annoying. (Windows 10)

Fixed in 2.0.7 (will be released soon)


No JS supplied

it no work

prs are welcomed

@TodePond can this be a transpiler or do we do it the old fashioned way around here

@TodePond can this be a transpiler or do we do it the old fashioned way around here

no transpilation, all run-time :)

@TodePond can this be a transpiler or do we do it the old fashioned way around here

no transpilation, all run-time :)

aargh but like a good chunk of this is almost possible in pure js but not fully, like the with block thing

it is all possible with withs and Proxy. the main issue is that I don't think with gets passed down into functions blocks inside it, so I'm thinking about removing the with requirement, and just stick with Proxy madness. You could go ahead with some of the proxy-only features if you want! (Like .d for example)

@TodePond Actually, it seems with was the right choice. Without it, the variable definitions would be very hard to implement

without with you could do something like:
var.name = "Luke";;

@TodePond any assistance on this effort to make airspace a thing would be appreciated

@ezShroom Just added a PR - #3 - you might enjoy with all it's...interesting design decisions😉. Turns out almost everything is possible as is, and with not much code to boot. The only thing that doesn't work are using the debug/execute literals on JS literals (Except strings. Works on strings for unknown JavaScript-related reasons.)
[3, 2].x(add)

doesn't work, but
someArray = [3, 2]
someArray.x(add)

does. Go figure. I'll be investigating if there's any way to support this in the future, but given my IDE gets upset at the . and not the x or d, this might be a hard JS limitation. "Some string".d is fine though.

Awesome, will check it out!
I know for a fact that .d is possible though because I've done it in https://github.com/TodePond/Habitat
If your IDE is misbehaving it can probably be resolved by doing some jsdocs, or in the worst case scenario, an eslint plugin (or equivalent).

Oh hell, I just uncommented the code and now it works fine. Must have had a different bug that was breaking it, and my IDE just...knew? I'll check in a fix to the examples to use it as specced in the README.

For numbers you have to do (1234).d

I had a print statement on the prior line without colons or brackets.
print.x = "Array debug below:"
[1, 2, 3, 4].d

prints y
print.x = "Array debug below:";;
[1, 2, 3, 4].d

prints 1,2,3,4. I've since discovered that "abcdef"[1,2,3,4] prints e and only e, so that's another JavaScript-ism, and in my example the lack of colons or half-colons defaulted to indexing the prior line. Looks like colon auto-insertion isn't infallable in Airspace.

@danm36 How does modifying the main primitive prototypes for .d and such compare to implementing it on a getter (like in the very unfinished https://github.com/Shroom2020/airspace)?

Here it is in Habitat: https://github.com/TodePond/Habitat/blob/main/source/console.js

@ezShroom To be honest, I was just creating the examples in README.md one by one and when I got there, I figured "Hey, polluting the prototype is the easiest method!" Your method is definitely a lot cleaner when it comes to leaving an Airspace block. If I get time, I might ~steal~borrow your idea for my version and get rid of the global namespace pollution.


micronaut-test-resources is not supported

I use the kotest plugin a lot in my daily business and it helps me a lot!
Unfortunately the plugin does not support micronaut-test-resources as this framework needs tests to be started by gradle, see https://github.com/micronaut-projects/micronaut-test-resources/issues/120
A testproject can be found in https://github.com/mf-guse/micronaut-test-resources-test.
If you run the test using the kotest plugin the required Docker Testcontainer is not started. Run the test using Gradle, it is started.
Do you see any chance to let the plugin support micronaut-test-resources?

I have just run into this same issue. It would be resolved if we could delegate test running in IntelliJ to Gradle like what is possible with Junit. Any chance this can be looked at?

That's the plan for 6.0.0 @mmwinther


refactor: layout optimizations in tutorial

Some minor improvements in tutorial:

No need to set height="100%" on most places as elements are usually stretched in their layout container.
Removed the stack-layout from the first tab. One grid is enough

Awesome, thanks for this!


Add 'unmatch' feature

Add a way to unmatch characters, where 'unmatch' means to match any character that is not in a specified set.
This is useful for grammar rules that describe a blacklist.
ABNF (or any other BNF) does not support blacklists, so blacklists are usually described in the comments.
quoted-string = DQUOTE *(unicode-char) DQUOTE
                    ; The following characters MUST NOT appear in a quoted-string:
                    ; line feed (0xA)
                    ; carriage return (0xD)
                    ; double quote (0x22)

DQUOTE        =  %x22
                    ; " (Double Quote)

unicode-char  =  %x20-FF
                    ; Unicode Basic Latin + Latin-1 Supplement

Note that it's sometimes (always?) possible to rewrite rules in a way that splits the whitelisted and blacklisted characters into two rules, so that the original rule becomes the complement of the two new rules.
     quoted-string = DQUOTE *(quoted-string-char) DQUOTE
                   ; The double quote (0x22) character MUST NOT appear in a quoted-string

            DQUOTE =  %x22
                   ; " (Double Quote)

quoted-string-char = %x20-21 / %x23-FF
                   ; Unicode Basic Latin except DQUOTE + Latin-1 Supplement

      unicode-char = quoted-string-char / DQUOTE
                   ; Unicode Basic Latin + Latin-1 Supplement


using client.RunJobAndPollAsync

Hi
I have downloaded and used this wonderful library for months and its great. I want now to use the Bulk Upload methods but I do not see the RunJobAndPollAsync on my ForceClient object. I am using 1.3.2
Any help appreciated
Thanks
Steve

I have the same issue. How to use the RunJobAndPollAsync?

I didn't find it either... had to download the 1.4-beta version to get it... they never said a word and I'm currently dealing with issues of data size... thus far, 3k records is out the question, down to 100.

Sorry, bulk will be in the next public release.


Add pre-response to actions

It would be nice to have a pre response when an action was recognized. This is useful i.e. if you have long waits, so you can notify the user that you are performing an action and you will be back soon.

Closing in favor of #276 which Is how I hope to accomplish this.


tensorflow.linalg.norm document missing

Thank you for submitting a TensorFlow documentation issue. Per our GitHub
policy, we only address code/doc bugs, performance issues, feature requests, and
build/installation issues on GitHub.
The TensorFlow docs are open source! To get involved, read the documentation
contributor guide: https://www.tensorflow.org/community/contribute/docs
URL(s) with the issue:
https://github.com/tensorflow/tensorflow/blob/1cf0898dd4331baf93fe77205550f2c2e6c90ee5/tensorflow/python/ops/linalg_ops.py#L426
Description of issue (what needs changing):
I can't find tensorflow.linalg.norm on official v2.0 API document

Because tf.linalg.norm is same  as tf.norm, the documentation is shared with tf.norm https://www.tensorflow.org/api_docs/python/tf/norm

Because tf.linalg.norm is same as tf.norm, the documentation is shared with tf.norm https://www.tensorflow.org/api_docs/python/tf/norm

Thanks


False positives: MP3 streams

There are still false MP3 positives, e.g in this TeamViewer Debian package:
(0.00%) Possible bZip2-Stream found at position 2084, compression level = 9
Can be decompressed to 192122880 bytes
Identical recompressed bytes: 53084090
Identical decompressed bytes: 192122880 of 192122880
Real identical bytes: 53084090
Best match: 53084090 bytes, decompressed to 192122880 bytes
Recursion start - new recursion depth 1
(52.25%) Possible MP3 found at position 100378755, length 2400
packMP3 error: region size out of bounds (frame #0 at 0x0)
No matches
[...]
Recompressed streams: 1/140
MP3 streams: 0/139
bZip2 streams: 1/1
Doing the region size check before passing the data to PackJPG would solve this.

Related: Issue #35

The link above is dead, but this one seems to work.


Dashed lines

At the same time as #9, it probably makes sense to look into support for dashed lines drawn with U+2506 and similar.  There is already support for drawing them, just not recognizing them.

First of all Martin, this tool is already so incredibly slick. I have not been this excited about something at IETF in a very long time.
I am very interested in dashed and dotted lines. I'm guessing that there is a problem figuring out the semantics of period and colon characters, otherwise you could use those for horizontal and vertical dotted lines?

Yeah, the colons are easy.  Replicating the logic for recognizing vertical lines should work out reasonably well, with some tweaks (+ might not be something we want to allow at end points, I don't know).  Horizontal lines are tricky because period (.) is at the bottom of the box and there is a whole bit of logic that uses underscores for lines at the edge of the box.  We might need to use middle dot (·) or the box art options (┄ or ┅).
Mostly this just needs time, something I don't have much of, even though I enjoy this sort of low-pressure coding.


Cant get dom automatically when put the ref of directive in v-else branch

Version
3.0.5
Reproduction link
https://codesandbox.io/s/suspicious-wright-n3pi5?file=/src/App.vue
Steps to reproduce
<template>
  <a v-if="isShow" />
  <button v-else ref="el">el</button>
</template>

<script>
import { ref, watchEffect } from "vue";
export default {
  name: "App",
  setup() {
    const el = ref();
    const isShow = ref(true);

    setTimeout(() => {
      isShow.value = false;
    }, 3000);

    watchEffect(() => {
      console.log("change", el.value); // `undifined`
    });

    return {
      el,
      isShow,
    };
  },
};
</script>

What is expected?
should be get button
What is actually happening?
get undifined and just called once

watchEffect is flushed in 'pre' mode by default, which means its run before the DOM is updated. When working with template refs, which are updated after the DOM has been updated, you need to use flush: 'post'
    watchEffect(() => {
      console.log("change", el.value);
    }, { flush: 'post' });

watchEffect is flushed in 'pre' mode by default, which means its run before the DOM is updated. When working with template refs, which are updated after the DOM has been updated, you need to use flush: 'post'
    watchEffect(() => {
      console.log("change", el.value);
    }, { flush: 'post' });


Funds didn’t reach the destination

Hi,
I sent from https://getblockcard.com/ to Bitmart, I did enter xlm address and memo, It shows transfer completed Status:Successful, but the coins did not arrived to Bitmart.
Transaction dec08bb05a297d387b7e197767707edb09e08a5cbafedc7e1a49bb9feb91f3c7
Please help me with that
Thanks,
Vlad

Hi,
I sent XLM from lobstr to Bitmart, I entered the address and memo, It shows transfer completed Status:Successful, but the coins never arrived to my Bitmart.
Please help me with that
Thanks

Any update would be appreciated

Never received anything.
Sent from my iPhone
On Sep 6, 2021, at 7:06 AM, dulama @.***> wrote:

Any update would be appreciated
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/stellar-expert/stellar-expert-explorer/issues/416#issuecomment-913560455, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATZ773CTCJT4TPSANFFMVADUASOD3ANCNFSM43FKHVNA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Use certificate error page when using a weak SSL key. r=vporof

Coverage decreased (-1.06%) to 41.411% when pulling 883084558b23dcac2b5fce6bb78195fee3fa7b95 on jsantell:weak-key-errors into 5abd76f95b2caf8218a00179017ef4e25c6e3a27 on mozilla:master.

@jsantell there's some failures on CI, planning to look into them, need some help?


Meet lots of Deprecated warning with higher version of Pytorch. Do you have any idea?

Hi, thank you for your repo!  It's an awesome implement and helps me a lot. However when running it with higher version of pytorch, it raises lots of deprecated warning as this:
IndexingUtils.h:20: UserWarning: indexing with dtype torch.uint8 is now deprecated, please use a dtype torch.bool instead.

I searched around the project but find uint8 nowhere. Do you have any idea how to fix this? Or do I have to run it at version of 0.4. (I use 1.2 currently since the API didn't change a lot since 0.4)

I would run it with version 0.4 if at all possible. Unfortunately I wrote this code a while ago and haven't had time to update it to keep up with changes in PyTorch


Improvements to the systemd unit file

This makes it more generic so that it can be installed by default.
This PR also gives a more descriptive name to the vm binary.

/approve
LGTM

LGTM

/hold cancel


test require statements now work on case-sensitive platforms

Hey there! Thanks for the awesome library - its been a godsend in a project I'm working on. I ran into a small problem, though - we're using immutablediff both client-side and server-side, and while OSX isn't case-sensitive, other OSs (notably Ubuntu in this case) are. Most of the tests had require('Immutable') instead of require('immutable') in them, and that was causing some problems in deployment. Hope you don't mind the quick fix PR :)

:+1: Thanks for fixing this :smile:

np. Thanks for the great library!


[ZEPPELIN-3321] isRevisionSupported not working for importing notebooks.

What is this PR for?
when importing note of from one zeppelin instance that don't support revision to another zeppelin instance where revision is supported, the imported note still don't support revision.
What type of PR is it?
[Bug Fix]
What is the Jira issue?

ZEPPELIN-3321

Questions:

Does the licenses files need update? N/A
Is there breaking changes for older versions? N/A
Does this needs documentation? N/A

@zjffdu, @r-kamath  can you help review this.

@prabhjyotsingh please add a test

LGTM

LGTM

Thanks for the review will merge this if no more discussion.


Validate change address

Ensure the change address is an address that belongs to the user. We need to do this because we don't show the change address details to the user.

Codecov Report
Patch and project coverage have no change.

Comparison is base (d750e87) 91.41% compared to head (3b629a8) 91.41%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #32   +/-   ##
=======================================
  Coverage   91.41%   91.41%
=======================================
  Files          26       26
  Lines        1328     1328
=======================================
  Hits         1214     1214
  Misses        114      114

Flag
Coverage Δ

unittests
91.41% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Emulated_hue: default type to Google [Breaking change]

Description:
With the introduction of https://github.com/home-assistant/home-assistant/pull/5435 we are able to persist the numbers that emulated_hue gives to each entity id. This fixes grouping in Google Home and also makes it work for Alexa. This makes the google_home option now the correct option for all. The same ids for devices, including after reboots.
The alexa option is flawed as it uses entity ids as light ids while Hue is limited to numbers. Alexa supports strings so it worked.
Now that Google Home option is the correct implementation, it is time to switch over the default implementation.
If you are using Alexa and do not want to reconfigure it, update your configuration to be like this:
emulated_hue:
  type: alexa

A warning has been added that this option will be removed in the future.

This is going to need a massive breaking change warning in the blog post. /cc @fabaff

I am guessing we can still only use either Google Home or Alexa and not both. Is that correct?

So sad!


Missing feature in Readme

Describe the bug
Something is missing in the Project Board probably an Image
To Reproduce
Readme.md file
Expected behavior
Screenshots

I will work on this 🤓

@all-contributors please add @eddiejaoude  for design

@all-contributors please add @Prathamesh-Shanbhag  for bug


camerad: zero memory before returning from MemoryManager::alloc

reopen https://github.com/commaai/openpilot/pull/25728

Does this impact camerad startup time at all?

Theoretically it won't.  Also we have memset some of the memory in camera_qcom2.cc:
https://github.com/commaai/openpilot/blob/846be323d52ea07c7b303759beb3c4c45f526c45/system/camerad/cameras/camera_qcom2.cc#L141-L142
I'll reopen this pr after I've done the test

dont think it will impact the startup time. The average time of memset is 0.0005ms (500ns) ~0.0008ms only. It will hardly have any effect.
I think we dont need to save time here, We can optimize the camera's startup time in other places.


105 get all user's friends

https://github.com/OlhenShu/GreenCityProjectStage/issues/105

@DenysLiubchenko thank you for the useful tip with checkstyle


Connection issue when connecting azure speech to text service from an angular application -1006 status code

Hi All,
For one of my projects, I am using azure's speech to text service in angular app.  Both Speech service and Angular app are in same vnet but in different subnets. Angular app is VNet Integrated.
Speech service is working as expected when it is publicly exposed but failing when using private endpoint. On troubleshooting, I could see 1006 status code.
I have tested private end point connectivity by spinning VM from same VNET which is working. Please advice on how to troubleshoot further.
Thanks

Please find console logs -
CANCELED: Reason=0
CANCELED: ErrorCode=4
CANCELED: ErrorDetails=Unable to contact server. StatusCode: 1006, wss://abc-sphsvc.cognitiveservices.azure.com/stt/speech/recognition/conversation/cognitiveservices/v1
Reason:  undefined

Hi, please attach a full Speech SDK log: https://learn.microsoft.com/azure/ai-services/speech-service/how-to-use-logging

Closed since no logs provided, please open a new issue if more support is needed.


Cocoapods support?

Yes, that would be nice to have. Wasn't a priority for my initial work since I'm using Carthage these days. I'll get to it eventually, or, I'll be happy to take a pull request!

Closing: no longer maintaining this project. Thanks for the interest 🕳


Vector VXL python wrapper project addition proposal

Hi,
I just found today complete Vector VXL python wrapper, windows only, but should be interesting, since it wraps not only CANbus, but also other supported protocols like MOST, Ethernet channel/network and more
PythonVectorXL

Damian, thanks for you contribution.
I just added this item, please check: https://github.com/iDoka/awesome-canbus/commit/d15eb7b1de59cf9590b75e87ddd6479a781a6c53
Any feedback are welcome. If description is okay you could close this issue.

Looks okay, thanks a lot!


Write a description of what Free Code Camp Columbus specifically is.

This section

 from the screen designs

not exact, but it looks good :)


Speed optimalization

A bit cleaner and faster code. Speed improvement is somewhere between 20 to 40%. I say between because it depend on the format of messages. People that have busy TCP server which process for example JSON messages (as in my case) will notice some difference.

@Mattijah thanks!
Landed on v0.2.0.

@pgte I've tried to update the modul however npm downloads this release https://github.com/pgte/carrier/commit/0c32b096c2f284999dc20c220d9efe51188f88e1 (also v0.2.0)

Weird.
Anyway, released v0.2.1 from master, please check that one out.

@pgte It works correctly now. Thanks


Fixed DefinitionIsChildOfConstraintTest to use newer Symfony code

Thank you

looks like we also need to update the data provider to not include the data with the DefinitionDecorator class on Symfony 4

@stloyd I hope you do not mind that I made some changes to your branch

@Nyholm Not a problem ;) I didn't had time to finish this fast, feel free to squash & merge ;)

Thank you


Editor - text editor too big

The text editore extends below the bottom of the frm leading to scrolling: it needs to stay within the wndow limits where possible
By Julian

This seems ok to me now. It depends on the heights in XWD but I've tried all the page types and didn't notice any that were noticeably too big


Support for libraries?

Any plans to add library support? For example, to include the 'drawing' library, the script tag would be:
<script type="text/javascript" src="https://maps.googleapis.com/maps/api/js?libraries=drawing"></script>

I made the necessary change. If you give me access, I can create a PR.

Created a PR for this to save you time.


Subclass the PyTorchYolo and PyTorchDetectionTransformer estimators off the PyTorchObjectDetector estimator class

Is your feature request related to a problem? Please describe.
There is a lot of code overlap between the PyTorchYolo, PyTorchDetectionTransformer, and PyTorchObjectDetector estimators. This causes bugs to have to be fixed in both locations when reported. Missed changes in one estimator will still remain a bug in the other. In additional, the big overlap in code is redundant and makes it tedious from a maintenance perspective.
Describe the solution you'd like
The PyTorchYolo estimator should be subclassed off PyTorchObjectDetector. This will reuse a lot of the code and avoid redundant bugs.
Describe alternatives you've considered
N/A
Additional context
This is purely an codebase housekeeping change. Functionality should not be changed in any way.

Hi @f4str Thank you for opening this feature proposal. I think this is great idea. I have combined the two issues to see both changes in the same PR.


Add helper to make Sub from fs2.Stream

I've done a basic test of this and it does indeed work. I haven't tested removing a running sub yet though... that's my next trick.
  def stream: Stream[IO, Msg] =
    Stream.awakeEvery[IO](5.seconds)
      .map(t => Msg.Log("Log after " + t.toString))

  def sub: Sub[IO, Msg] =
    Sub.make("pulse", stream)

Produces:

Also tested removing and re-adding, all seems to work perfectly. Nice one @armanbilge. 😄


BUG: Literary realms not responsive

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The cards in Literary realms are not responsive. In smalls , full width is not appearing.

Please assign me this issue. I want to contribute to this project under GSSOC'24.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Assigned @nagalakshmi08


[feature] Add --logFile for appending logs to a file.

Goals

[x] Pipe logs to a file specified by the user
[x] Ensure the log file doesn't have the color codes as it makes sifting through the logs impossible

Notes

coloring of logs happens all over the repo, seemed like the most sane way to remove the ansi codes was to just strip them out rather than updating the callsites.
I didn't implement the "first log gets the full date format" stuff since it seemed unnecessary and introduced ugly complexity (if you want lemme know)
It seems like the --log option wasn't working before. I fixed this by adding the .level(level) on the dispatcher.

Can you also updates the options list here: https://github.com/moonrepo/moon/blob/master/website/docs/commands/overview.mdx

@tbezman Looks good. Can you create an integration test that verifies it works. Here's an example: https://github.com/moonrepo/moon/blob/master/crates/cli/tests/run_test.rs#L130

@tbezman Thanks for this. I'll merge it and fix the failing test in the develop branch.


fix(report): recognize doctypes in stylesheets

This should resolve the issue that doctypes are not recognized (at least for the way I use them in my stylesheets).

Could you provide test files for at least two cases? With and without ENTITY.
The test files are supposed to be in test/end-to-end/cases/ and the .xspec files should have <?xspec-test enable-coverage?>. So, they would look like:

test/end-to-end/cases/issue-1411_entity.xsl
Contains ENTITY in DOCTYPE.

test/end-to-end/cases/issue-1411_entity.xspec
<?xml version="1.0" encoding="UTF-8"?>
<?xspec-test enable-coverage?>
<x:description stylesheet="issue-1411_entity.xsl" xmlns:x="http://www.jenitennison.com/xslt/xspec">
   ...
</x:description>

test/end-to-end/cases/issue-1411_no-entity.xsl
Contains no ENTITY in DOCTYPE.

test/end-to-end/cases/issue-1411_no-entity.xspec
<?xml version="1.0" encoding="UTF-8"?>
<?xspec-test enable-coverage?>
<x:description stylesheet="issue-1411_no-entity.xsl" xmlns:x="http://www.jenitennison.com/xslt/xspec">
   ...
</x:description>

After putting the test files into test/end-to-end/cases/, open a terminal (or a command prompt) and set up a test environment:
$ sudo apt install curl
$ source test/ci/install-deps.sh

or
C:\xspec>test\ci\install-deps.cmd

which installs Ant, Saxon, Apach XML Resolver, XML Calabash, BaseX and sets up environment variables. All the files are placed in /tmp (or %TEMP%) and the environment variables are volatile, so rest assured that they won't affect your environment perpetually.
Now within the same terminal (or command prompt), run:
$ test/end-to-end/generate-expected.sh

or
C:\xspec>test/end-to-end/generate-expected.cmd

which executes the test/end-to-end/cases/*.xspec files.
Their coverage report files will be generated as test/end-to-end/cases/expected/stylesheet/issue-1411_*-coverage.html.
Verify that the coverage reports are fine with the bug fixed. If they are fine, commit them (test/end-to-end/cases/issue-1411_*.(xsl|xspec) and test/end-to-end/cases/expected/stylesheet/issue-1411_*.(html|xml)).

Thanks for the extended instructions on creating test cases. Worked as a charm.

Thanks for the extended instructions on creating test cases. Worked as a charm.

They look very good. Thanks for creating them swiftly.

Let me merge the latest changes from master into this branch, as it's required by the branch protection rule.

You're welcome, it was fun, and resolved an issue I bumped in.


index.GatherIndexIssueStats() assumes Chunk is non-nil

When attempting to downsample a block compacted by Prometheus I get panics in GatherIndexIssueStats() as it is trying to calculate the checksum of chunks in the following snippet. I think some handing of the case where Chunk is nil is needed.
			if c.MinTime == c0.MinTime && c.MaxTime == c0.MaxTime {
				ca := crc32.Checksum(c0.Chunk.Bytes(), castagnoli)
				cb := crc32.Checksum(c.Chunk.Bytes(), castagnoli)
				if ca == cb {
					// Duplicate.
					stats.DuplicatedChunks++
				}
				ooo++
			}

c and c0 may look like this (Printf %v format):
c0: {51623238719 <nil><PHONE_NUMBER>000<PHONE_NUMBER>000}
c: {51623238909 <nil><PHONE_NUMBER>000<PHONE_NUMBER>000}

There are comments in chunks.Meta definition saying Chunk may not be set if the Ref is set:
// Meta holds information about a chunk of data.
type Meta struct {
	// Ref and Chunk hold either a reference that can be used to retrieve
	// chunk data or the data itself.
	// Generally, only one of them is set.
	Ref   uint64
	Chunk chunkenc.Chunk

	MinTime, MaxTime int64 // time range the data covers
}

We can fix it, but would love to know why Prometheus compacted blocks are different... It is same compaction.

I don't know. They may not be, I just added that info in case it was significant.
Could it be related to some issue happening during scraping. (possibly same endpoint scraped for two different jobs so some race condition may be involved as well. I know we used to have some misconfiguration like that.)
It happened 37699 times in 2 blocks (162h lvl 5 compaction), but only happened for metrics collected from two of the servers scraped by prometheus. And it was almost all from the node exporter on one of the servers. Only a handful from another server and job.
When it happened, it happened across a number of metrics, there were only 88 different timespans. (minTime, maxTime)
The vast majority of the timespans were ~ 30 minutes, some were ~60 min and a few ~15min.

Interesting, thanks for details. Maybe worth to put issue on Prometheus as well?
Basically the issue here is that Chunk chunkenc.Chunk is nil, and you said (and you are right) that:

There are comments in chunks.Meta definition saying Chunk may not be set if the Ref is set:

However I am pretty sure that when chunk is nil, then Ref is not set either in your case. Chunk is not checked to be nil, because this method of reading the chunks assures that chunk is always not-nil (: I will make this code flow handle that, but still a nil chunk is within blocks is something wrong and need to be reported upstream (Prometheus).

Ref should be first values in the structs, I think:
c0: {51623238719 <nil><PHONE_NUMBER>000<PHONE_NUMBER>000}
c: {51623238909 <nil><PHONE_NUMBER>000<PHONE_NUMBER>000}

I think I'm hitting this also:
level=debug ts=2018-07-19T21:09:24.695545944Z caller=compact.go:166 msg="download meta" block=01CJT5F97BK7ZEY9VJT4DWD1MW
level=debug ts=2018-07-19T21:09:24.72254922Z caller=compact.go:184 msg="block is too fresh for now" block=01CJT5F97BK7ZEY9VJT4DWD1MW
level=info ts=2018-07-19T21:09:24.7226022Z caller=compact.go:132 msg="start of GC"
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x7bf70c]

goroutine 70 [running]:
github.com/improbable-eng/thanos/pkg/block.GatherIndexIssueStats(0xf14100, 0xc4203592c0, 0xc4204ab100, 0x75, 0x16459938f00, 0x1645d705400, 0xbc05, 0x0, 0x0, 0x0, ...)
        /go/src/github.com/improbable-eng/thanos/pkg/block/index.go:317 +0x52c
github.com/improbable-eng/thanos/pkg/compact.(*Group).compact(0xc4206eec80, 0xf20880, 0xc420195000, 0xc4200ab3e0, 0x54, 0xf1e880, 0xc4201a5270, 0x0, 0x0, 0x0, ...)
        /go/src/github.com/improbable-eng/thanos/pkg/compact/compact.go:695 +0x7ea
github.com/improbable-eng/thanos/pkg/compact.(*Group).Compact(0xc4206eec80, 0xf20880, 0xc420195000, 0xc4200bd9a0, 0xf, 0xf1e880, 0xc4201a5270, 0x0, 0x0, 0x36de30, ...)
        /go/src/github.com/improbable-eng/thanos/pkg/compact/compact.go:464 +0x185
main.runCompact.func2(0x48, 0xe9b000)
        /go/src/github.com/improbable-eng/thanos/cmd/thanos/compact.go:144 +0x21a
main.runCompact.func3(0x0, 0x0)
        /go/src/github.com/improbable-eng/thanos/cmd/thanos/compact.go:191 +0x277
github.com/improbable-eng/thanos/vendor/github.com/oklog/run.(*Group).Run.func1(0xc4201a3260, 0xc42033cd20, 0xc420193740)
        /go/src/github.com/improbable-eng/thanos/vendor/github.com/oklog/run/group.go:38 +0x27
created by github.com/improbable-eng/thanos/vendor/github.com/oklog/run.(*Group).Run
        /go/src/github.com/improbable-eng/thanos/vendor/github.com/oklog/run/group.go:37 +0xa8

I'm not an expert on prometheus internals, but if there's anything I can do to help debug, please let me know.

HI all - @Bplotka you mentioned you'd make the code flow handle this. Any PR or news on this. We hit this occasionally also.

I also meet the problem, and I print out the value of instance of chunks c and c0.
fn: /var/thanos-compact/compact/0@{monitor="prometheus",replica="prom-thanos-s3-0"}/01CRFAS69HNGJKKRMQ7QKKA02C/index
c: chunks.Meta{Ref:0x22827b6, Chunk:chunkenc.Chunk(nil), MinTime:1538078400000, MaxTime:1538078400000}
c0: chunks.Meta{Ref:0x228279f, Chunk:chunkenc.Chunk(nil), MinTime:1538078400000, MaxTime:1538078400000}

As @asbjxrn said, the Chunk chunkenc.Chunk is nil, but it has a stranger appearance, the MinTime and MaxTime are different with the meta.json in this block and as same as each other.
Here is the content of meta.json
{
	"version": 1,
	"ulid": "01CRFAS69HNGJKKRMQ7QKKA02C",
	"minTime":<PHONE_NUMBER>000,
	"maxTime":<PHONE_NUMBER>000,
	"stats": {
		"numSamples": 849990243,
		"numSeries": 1443007,
		"numChunks": 7630195
	},
	"compaction": {
		"level": 2,
		"sources": [
			"01CRE6DVYK9YDPEG5KT7FD75VP",
			"01CRED9K6KN835FMC79S0RBHXC",
			"01CREM5AEMM36RT49APRZ5Y60M",
			"01CREV11PMCJTJQ6FZTPCV7QJA"
		]
	},
	"thanos": {
		"labels": {
			"monitor": "prometheus",
			"replica": "prom-thanos-s3-0"
		},
		"downsample": {
			"resolution": 0
		},
		"source": "compactor"
	}
}

So is it something wrong for the compact processing? @bwplotka

So I added PR for fix to handle problem gracefully - was wrong - obviously ref was only fileld not chunks encoding, so I was wrong.
But the issue itself should not happen. And compactor will halt now anyway (just with proper message).
The @jojohappy info is insideful.. It seems we have just ghost chunks, that has just one sample or maybe even nothing. And also this is already compacted block (level 2). Worth to track this down.

This was fixed.


Nagware failure when processing responses associated to deleted forms

The current scenario is: A user can delete a form without confirming their responses. Because there are unconfirmed responses the nagware still fires but is unable to associate an owner to the response.
I think we have two options:

Stop a user from deleting a form if they have unconfirmed responses
Warn a user when they delete a form that they will be unable to retrieve responses. Then, confirm and delete the response for them.

Next steps: Discipline input and review cc: @srtalbot @connorscarolyns @craigzour @timarney @sarahhobson

Stop a user from deleting a form if they have unconfirmed responses

@craigzour --- outside of stopping them at a UI level should we also handle this at the API level and handle the error inside the current UI?
--
@samsadasivan I think we should look at My Form as the entry point for deletions.  It's currently the only way to delete.
The app doesn't match Figma at the moment --- in a previous iteration delete from my forms redirected to the settings page.  The code was later updated to handle deletes directly from my forms.  I haven't got back around to re-introducing deleting from the settings page.
The above does make me wonder if we need the forms management page under settings --- it's all duplicated actions from My Form .   My understanding was My Form was for forms management.

Stop a user from deleting a form if they have unconfirmed responses

@craigzour --- outside of stopping them at a UI level should we also handle this at the API level?
--
@samsadasivan I think we should look at My Form as the entry point for deletions. It's currently the only way to delete.
The app doesn't match Figma at the moment --- in a previous iteration delete from my forms redirected to the settings page. The code was later updated to handle deletes directly from my forms. I haven't got back around to re-introducing deleting from the settings page.
The above does make me wonder if we need the forms management page under settings --- it's all duplicated actions from My Form . My understanding was My Form was for forms management.

@timarney Yes I think we should double check at the REST API or lib/templates.ts API because we may want to allow users to delete forms using the API at some point in the future. Also if we ever introduce a bug at the UI level we will have a second barrier.
I can start working on that API check while you tackle the UI part.

@timarney that makes sense. If we need to add another spot to find a way to delete the form from user insights, we can revisit this, but I like the idea that the builder interface is for editing and downloading. Then All Forms is for Form Management. Separating the workflows will be helpful for future features.
Ok, in this case, the modal blocking deleting is the best route for now. I know clicking to eventually get blocked isn't ideal, but I don't want to have a disabled line item either haha.

I can also find a way to show how many responses are available to be download within the card. Carolyn had mentioned this in the past

Ok, in this case, the modal blocking deleting is the best route for now. I know clicking to eventually get blocked isn't ideal, but I don't want to have a disabled line item either haha.

I think this is ok for now. Make sure the user can get where they need to go from the error modal so that they can recover quickly and easily.

From a content perspective, since there are there multiple things people need to know/do as they're deleting, perhaps we could take them through each one step-by-step to reduce cognitive load?

Save (or send?) a copy of the form.
Download any form responses received.
Confirm the receipt of downloaded responses.
Re-confirm the irreversible deletion of the form.

UPDATE: @anikbrazeau is working on the copy and translation for the modal

UPDATED BILINGUAL CONTENT
Alert
EN:
**Download and confirm all responses before deleting form.**
[Go to Responses](/responses/)
FR:
**Télécharger et confirmer toutes les réponses avant de supprimer le formulaire.**
[Accéder aux réponses](/responses/)

Modal
EN:
`Get responses before deleting form
This form has been published and is receiving responses. You have new responses to download and confirm.
Before deleting this form:

Download all responses.
Confirm the receipt of downloaded responses.

[Go to Responses]`
FR:
`Retirer les réponses avant de supprimer le formulaire
Ce formulaire est publié et reçoit des réponses. Vous avez des nouvelles réponses à télécharger et à confirmer.
Avant de supprimer ce formulaire :

Télécharger toutes les réponses.
Confirmer la réception des réponses téléchargées.

[Accéder aux réponses]`

Just let Anik know that we aren't doing the /Settings route, so refer to the modal for now :)

Next steps can we confirm we're moving forward with the Modal screens above

Via My Form
User presses delete
We check for responses that haven't been downloaded
If we find  responses that haven't been downloaded -- show the screen above else try the delete...

We'll still want a nice error message if the delete fails ... unlikely but I think it would be possible for a new response to come in after the form response check but before I press delete.

I think we're good to move forward with the modal -- open to feedback if anything should change. 😃
As for the trigger -- is it checking for responses that haven't been downloaded and haven't been confirmed?
As for an error message if delete fails, how about something like:
EN:  The form could not be deleted. Ensure there are no new responses and try again.
FR: Le formulaire n'a pas pu être supprimé. Veuillez vous assurer qu'il n'y a pas de nouvelles réponses et réessayez.
cc: @samsadasivan

Yes - thats correct. I think in this case, the "toast" feature could work for the error.

Ahh that is a good use case to look out for - @anikbrazeau I think the message works in this case

We can also use this error component, to appear at the top:
For awareness, we have four types of error components:

Page errors (404,500)
Toast
Red alert
In-line form error messages.

In this case - we can use the red alert or the toast, but I agree - the toast can't disappear. For simplicity - lets move forward with the red alert component. Ill pop the FR and EN mocks here

If we do it this way. Would that mean that we can let the user press the delete button no matter what? Then depending on the API response we let the user know whether it succeeded or failed because unprocessed responses have to be dealt with.
cc @bryan-robitaille

:D No worries! It's good to clarify haha

Update
Modal (merged)
https://github.com/cds-snc/platform-forms-client/pull/1999
Toast message (updated current to not auto close)
https://github.com/cds-snc/platform-forms-client/pull/2009
Noting we'll need to look at updating error styling for toast messages in a future card as the current doesn't match Figma.  We can also look at using the above error inline but again we should look to do break that work into a new card as it's more of a global styling thing vs specific to this issue.

Closing the loop on this issue:

Toast functionality will appear at the top of the screen to let users know if they have are able to delete a form or not.
The option to delete will only exist in the "Your Forms" page/view and not in each in the /Settings pages of individual forms
Next steps: Apply icon+header+message alert styling to toasts in a different card, issue #2029

Closing this :)

FOR REFERENCE: For users who deleted their forms that had responses they didn't download, they will now get a 404 page. Context and decision referenced in issue #2023


Fix for Resource Owner Details URL and parsing

Check https://github.com/stevenmaguire/oauth2-microsoft/pull/10

All the @deprecated are here in order to grant backward compatibility and to warn the user that, in next major release, those API(s) will be gone.


Extra space in JSX self-closing components

Describe the bug
dprint-plugin-typescript version: 0.76.0
It seems like the new jsx.bracketPosition setting is adding an extra space to self-closing elements.
Input Code
// dprint file
{
    "indentWidth": 4,
    "typescript": {
        "quoteStyle": "alwaysSingle",
        "jsx.quoteStyle": "preferDouble",
        "trailingCommas": "never",
        "jsx.bracketPosition": "sameLine"
    },
    "includes": [
        "**/*.{ts,tsx,js,jsx,cjs,mjs}"
    ],
    "excludes": [
        "**/node_modules",
        "**/*-lock.json"
    ],
    "plugins": [
        "https://plugins.dprint.dev/typescript-0.76.0.wasm"
    ]
}

<MyComponent someProp={value} />

Expected Output
<MyComponent someProp={value} />

Actual Output
<MyComponent someProp={value}  />

As a follow-up, would it be too much to ask to have separate settings for regular and self-closing JSX elements? I feel awful asking this after the initial setting was so quickly implemented 🥲
For example, I would like this:
<MyComponent
  name="name"
  value="value"> // bracket in same line
</MyComponent>

<MyComponent
  name="name"
  value="value"
/> // bracket in separate line

Oops. Will take a look soon.
Yeah, I could split those up and have this as a high level property.

@dsherret Thanks a lot! Please let me know if you need anything else from me (regarding repro steps or anything really).

Will be released in 0.77 once the ci finishes running.

You're amazing, thanks @dsherret!

And I think you meant "jsxOpeningElement.bracketPosition" right? ;)

Yes, sorry. I updated the PR title. At least it was correct in the code 😅


[For review only] Minimize rereading of batches in [FSTLocalDocumentsView documentsMatchingCollectionQuery]

Gil, could you please take a quick look to see if this is equivalent to what you had in mind? This is not ready for the full review (I didn't copy some plumbing, it's basically just to verify the idea). This change brings significant performance improvements when querying for a lot of documents, so I'd like to double-check if there is actually a reason to be happy. Integration tests pass.

Closing in favor of #1533.


Hugo paper mod

https://github.com/adityatelange/hugo-PaperMod

[ ] Installation
[ ] Add Search: https://github.com/kaushalmodi/hugo-search-fuse-js
[ ] Mathjax instead of KaTeX
[ ] Chroma instead of highlight.js

Not worth the trouble


dita-ot: 4.2.3 -> 4.2.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/dita-ot/versions.
meta.description for dita-ot is: The open-source publishing engine for content authored in the Darwin Information Typing Architecture
meta.homepage for dita-ot is: https://dita-ot.org
meta.changelog for dita-ot is: https://www.dita-ot.org/dev/release-notes/#v4.2.4
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.2.4 with grep in /nix/store/pvjkqs9pzp55bd9jp7y8hynhprgjgp3b-dita-ot-4.2.4
found 4.2.4 in filename of file in /nix/store/pvjkqs9pzp55bd9jp7y8hynhprgjgp3b-dita-ot-4.2.4

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

dita-ot

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/e069168756a57500d2aeb5fb431bbcbc894ec709#dita-ot

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/pvjkqs9pzp55bd9jp7y8hynhprgjgp3b-dita-ot-4.2.4
ls -la /nix/store/pvjkqs9pzp55bd9jp7y8hynhprgjgp3b-dita-ot-4.2.4/bin

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

x86_64-linux

:white_check_mark: 1 package built:

dita-ot

Maintainer pings
cc @rrcomtech for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


add nodePosToString to the public API

TypeScript Version:  2.1.6 / nightly (2.2.0-dev.201xxxxx)
I'm using the typescript library to parse ts source code and I would like to have access to the nodePosToString function to format diagnostics. This function is internal for now.

just saw tslib (I was mislead by the @internal comment). I'll close the issue if I can find it in tslib.

No, I was mislead again by the export const externalHelpersModuleNameText = "tslib"; line below the @internal line. I'm keeping the issue open.

Not sure if anything has changed since v2.1.6, but in v2.3.2 it's at least accessible, though it's still not publicly exposed.
import * as ts from 'typescript';

let node = getNodeFromSomewhere();
let position = (ts as any).nodePosToString(node); // <-- It's on the `ts` object, but just not part of the definition.

Haven't gotten enough feedback on this one to warrant exposing it


CLOUD-410 fix odc

@biharitomi

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/792/


HTTP FLV playback failure with single client

注意：提问前，请先看FAQ(Please read FAQ before file an issue) https://github.com/ossrs/srs/issues/2716

描述(Description)

HTTP FLV playback doesn't happen until at least two clients try to watch stream. FLV.js console says [FlvPlayer] > Playback seems stuck at 0, seek to 0.033 and remains at player event Seeking forever

SRS版本(Version): SRS 4.0.198

SRS的日志如下(Log):

No log available

SRS的配置如下(Config):

# the config for srs to remux rtmp to flv live stream.
# @see https://github.com/ossrs/srs/wiki/v2_CN_DeliveryHttpStream
# @see full.conf for detail config.

listen              19351;
max_connections     60000;
pid                 objs/srs.flv.19351.pid;
daemon              off;
srs_log_tank        console;
#srs_log_level       error;
http_server {
    enabled         on;
    listen          8081;
    dir             ./objs/nginx/html;
}
vhost __defaultVhost__ {
    cluster {
        mode remote;
        origin localhost:3297;
    }
    http_remux {
        enabled     on;
        mount       [vhost]/[app]/[stream].flv;
    }
}

重现(Replay)

重现Bug的步骤(How to replay bug?)

Watch stream nobody else is watching via HTTP FLV
Then duplicate tab and watch the stream begin to load normally
Notice the stream won't load until second client loads same stream

期望行为(Expect)

HTTP FLV stream should load normally when accessing stream as only client

Can't reply.

Deleting in future.

I think you misunderstand. It works as expected with two or more viewing. But it does not work as expected if you are the only one viewing.

@invalidtask
Please ask questions on Stack Overflow using the #simple-realtime-server tag.
This issue will be eliminated, see $2716


build: compatible for future go version

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 51.68%. Comparing base (9e5bbc5) to head (e55bd38).
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #701      +/-   ##
==========================================
- Coverage   51.85%   51.68%   -0.17%
==========================================
  Files         130      126       -4
  Lines       11020    10919     -101
==========================================
- Hits         5714     5644      -70
+ Misses       4971     4949      -22
+ Partials      335      326       -9

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Re-add templating field

See https://github.com/kyverno/kyverno/issues/2188
This field was deleted in the latest update.

@yashvardhan-kukreja


Ego vehicles not moving

Hello, I've followed the tutorial shown in: https://autoware.org/running-multiple-autoware-powered-vehicles-in-carla-using-zenoh/. I have successfully installed the system and both Rviz visualizers opened up. However, when I set the Goal Pose, the vehicles couldn't move. Is there any configuration steps I missed? Is it possible for the problem to be related to internet connection? I appreciate any insight you may have.
Thank you so much for your answers.

You're using Humble version right?
Do you mean the vehicles still don't move after setting the goal and clicking auto?
Maybe you can try only one vehicle first and see whether the problem exists or not.
I'm not really sure if it's a performance issue or not.
If there is any video that can be provided, it might be helpful.

Thank you for your answer.
I am using the humble version indeed.
I tried to set up only one vehicle as suggested, and the vehicle is able to move now. However, the trajectory becomes unstable and the vehicle ends up crashing out of the road. Seems like Autoware has trouble following the vehicle's movement in Carla.
Also, I don't know if this is an issue, but the Auto button is normally greyed out, only activating every now and then for brief amounts of time before deactivating again. By clicking Local and then Stop, sometimes Autonomous mode just activates by itself.
I have recorded a video that shows the ego vehicle falling out of the road. This, as for now, has happened always, the vehicle has never properly arrived to the goal.
https://drive.google.com/file/d/1oLYfMF9eWgneoosvLbdFm1GDhT35TyUo/view?usp=sharing

Thank you for your answer. I am using the humble version indeed. I tried to set up only one vehicle as suggested, and the vehicle is able to move now. However, the trajectory becomes unstable and the vehicle ends up crashing out of the road. Seems like Autoware has trouble following the vehicle's movement in Carla.
Also, I don't know if this is an issue, but the Auto button is normally greyed out, only activating every now and then for brief amounts of time before deactivating again. By clicking Local and then Stop, sometimes Autonomous mode just activates by itself.
I have recorded a video that shows the ego vehicle falling out of the road. This, as for now, has happened always, the vehicle has never properly arrived to the goal.
https://drive.google.com/file/d/1oLYfMF9eWgneoosvLbdFm1GDhT35TyUo/view?usp=sharing
As you can see in the video, ROS Time seems a bit inconsistent. I've seen in Carla documentation (https://carla.readthedocs.io/en/latest/adv_synchrony_timestep/) that using synchronous mode without a fixed time-step may cause problems with the physics. Could this be related to the ego vehicle falling out of the road?

You're right. This is related to the ROS Time. I've found this issue before, but then disappear and can't be reproduced after some tries. I haven't found a way to reproduce this issue. I guess this is because the publisher of ROS time has an unstable frequency. I'll take a look at that.

@MarioAlmelaUMH  Can you give the branch a try? I've done some refactors, but I can't reproduce your issue, so not sure whether the issue is gone or not.
https://github.com/evshary/autoware_carla_launch/tree/update_bridge

I have installed the branch and made the ego vehicle run through maps Town01 and Town02, with and without Carla controlled vehicles. So far, I haven't encountered the issue anymore. The ego vehicle follows the road correctly. Thank you so much for your help.
I'm going to close this issue for now, and I'll reopen it if the problem appears again.


Refactor build.cmd/sh to not include blocks

Do this after the dotnet migration

More details in this PR: https://github.com/aspnet/HtmlAbstractions/pull/10#discussion_r49028200

Poaching this because I was looking at unexpected differences between repos. This problem has been fixed in a few but not the majority.

edd1f7a and loads of commits in individual repos to match


NoSuchElementException error

DevTools listening on ws://<IP_ADDRESS>:51248/devtools/browser/119322a7-0111-4a05-a64c-5a90a11c3894
[LOGIN] Logging-in...
[LOGIN] Writing email...
[LOGIN] Writing password...
[LOGIN] Passing security checks...
[LOGIN] Logged-in !
[LOGIN] Ensuring login on Bing...
[LOGIN] Logged-in successfully !
[POINTS] You have 19583 points on your account !
[DAILY SET] Trying to complete the Daily Set...
[DAILY SET] Completing quiz of card 2
Traceback (most recent call last):
File "Z:\Users\rapha\Desktop\ms_rewards_farmer.py", line 604, in
completeDailySet(browser)
File "Z:\Users\rapha\Desktop\ms_rewards_farmer.py", line 398, in completeDailySet
completeDailySetQuiz(browser, cardNumber, 3)
File "Z:\Users\rapha\Desktop\ms_rewards_farmer.py", line 276, in completeDailySetQuiz
browser.find_element_by_xpath('//[@id="rqStartQuiz"]').click()
File "C:\Python38\lib\site-packages\selenium\webdriver\remote\webdriver.py", line 394, in find_element_by_xpath
return self.find_element(by=By.XPATH, value=xpath)
File "C:\Python38\lib\site-packages\selenium\webdriver\remote\webdriver.py", line 976, in find_element
return self.execute(Command.FIND_ELEMENT, {
File "C:\Python38\lib\site-packages\selenium\webdriver\remote\webdriver.py", line 321, in execute
self.error_handler.check_response(response)
File "C:\Python38\lib\site-packages\selenium\webdriver\remote\errorhandler.py", line 242, in check_response
raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.NoSuchElementException: Message: no such element: Unable to locate element: {"method":"xpath","selector":"//[@id="rqStartQuiz"]"}
(Session info: headless chrome=87.0.4280.88)

Could you give me a screenshot of the quiz page and also give me the source code of the Microsoft Rewards Dashboard Page with gist.

sorry for not replying earlier.
I tried again today and got a different error. Failing on this or that

[DAILY SET] Trying to complete the Daily Set...
[DAILY SET] Completing search of card 1
[DAILY SET] Completing quiz of card 2
[DAILY SET] Completing poll of card 3
[DAILY SET] Completed the Daily Set successfully !
[PUNCH CARDS] Trying to complete the Punch Cards...
[PUNCH CARDS] Completed the Punch Cards successfully !
[MORE PROMO] Trying to complete More Promotions...
Traceback (most recent call last):
File "Z:\Users\rapha\Desktop\ms_rewards_farmer.py", line 610, in
completeMorePromotions(browser)
File "Z:\Users\rapha\Desktop\ms_rewards_farmer.py", line 560, in completeMorePromotions
completeMorePromotionThisOrThat(browser, i)
File "Z:\Users\rapha\Desktop\ms_rewards_farmer.py", line 532, in completeMorePromotionThisOrThat
answer1.click()
File "C:\Python38\lib\site-packages\selenium\webdriver\remote\webelement.py", line 80, in click
self._execute(Command.CLICK_ELEMENT)
File "C:\Python38\lib\site-packages\selenium\webdriver\remote\webelement.py", line 633, in _execute
return self._parent.execute(command, params)
File "C:\Python38\lib\site-packages\selenium\webdriver\remote\webdriver.py", line 321, in execute
self.error_handler.check_response(response)
File "C:\Python38\lib\site-packages\selenium\webdriver\remote\errorhandler.py", line 242, in check_response
raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.ElementNotInteractableException: Message: element not interactable
(Session info: headless chrome=87.0.4280.88)
https://gist.github.com/raphalation/d0fae7738220bb8b78695edf44ab49c6

As I can see on your screenshot, all is not loaded on the page. You may have a slow internet connection I think. So when the bot try to complete, it run very fast, so if an element is not loaded, it crashes with an error. I would recommand you to try with a faster computer or internet. I will also make an update to handle this type of problem.

I made an update to handle the error


add support for --key ARG

is needed? because for optional value arguments this is not supported

in order to keep consistent with keys which have optional value this feature is dropped.


Trading with NPC and autoaccepts.

Thanks for this amazing module.
I noticed the "Lack of support for trading with unclaimed player characters.". Is this anything that is being worked on? I think it would be a great addition to be able to send a trade request with an NPC or an "merchant" actor when you buy stuffs.  If you implement this in the future, maybe consider an setting to enable autoaccepts on an actors as well.

The issue is rather complicated. A non-totally inclusive list of problems that need to be solved:

DM control over which NPC sheets can be traded with and which can not.
If they can be traded with, but not on every scene.
Select NPCs from a list of extensive characters.
NPC sheets do not show all item types. Making "hidden" items that can never be seen.
NPC sheets do not show currency
For "merchants," item prices will need to be set.
NPCs understanding value over raw currency. If something is 10gp, it needs to accept 1pp or 100sp or 1000cp

Perhaps what needs to be done first, is to create a proper trade window, where both characters can make an offer at once, and only once both characters accept the pending offer the trade is complete. This would make merchant-style trading instead of freeform "give item" much easier.


Pre-requisites: Cloudshell in the AWS console.

1 major things that shows the error is Clone successful, but checkout failed, when I want to git add, It's not working as expected
in consequence when I run ./initial-workshop-setup.sh <aws_bucket_name_to_create> _it appears command not found
Full Picture implementation in Cloudsheel:
run git clone https://github.com/aws-deepracer-community/deepracer-genai-workshop.git
run cd deepracer-genai-workshop
run ./initial-workshop-setup.sh <aws_bucket_name_to_create>

Running this code in Cloudshell: run git clone https://github.com/aws-deepracer-community/deepracer-genai-workshop.git
it pops out Clone successful, but checkout failed, below pic

I wrote git status to see the checkout failed, when I git add them this was it appers to me.

List directories:

When I run this code run ./initial-workshop-setup.sh <aws_bucket_name_to_create> this appears to me, I even tried without bucket name just to see the result.

try pulling down the cloud-shell-init branch only (syntax can be seen here https://www.theserverside.com/blog/Coffee-Talk-Java-News-Stories-and-Opinions/How-to-git-clone-a-specific-branch-only), the issue is the repo is large and cloudshell has limited storage
Alternative is to use your own IDE

Readme updated to reflect for to initiate with cloudshell.


contradictory instructions on how to include DOI links in references

The help file compositionality-template.pdf says

As Compositionality is a member of Crossref, all references to works that have a DOI must be hyperlinked according to the DOI. Those links must start with https://doi.org/. Direct links to the website of the publisher are not sufficient. This can be achieved in several ways, depending on how you format your bibliography. Suppose the DOI of an article [1] that you want to cite is 10.22331/idonotexist. If you are formating your bibliography manually, you can cite this work using the following in your thebibliography environment:

\bibitem{examplecitation}
Name Surname,
\href{https://doi.org/10.22331/
idonotexist}{Compositionality
\textbf{123}, 123456 (1916).}

This does not make the DOI visible in the paper: it's obtained by clicking on the journal title.   But in the email Fosco is sending to authors he gives different instructions:

Crossref DOIs should always be displayed as a full URL link in the form https://doi.org/10.xxxx/xxxxx and should not be preceded by doi: or DOI:.

and recent papers on Compositionality show visible DOI links, like this:

Kenta Cho and Bart Jacobs. Disintegration and Bayesian inversion via string diagrams.
Mathematical Structures in Computer Science, 29(7):938-971, 2019. https://doi.org/10.
1017/S0960129518000488.

I'll go with the latter approach in the paper I'm submitting now, but I hope the contradiction in these instructions gets fixed.

Dear  John,
Thank you for pointing this out. The help file now (b4e6e5a) advises to use the following code instead, which makes the DOI visible in the paper.
 \bibitem{examplecitation}
   Name Surname,
   Compositionality \textbf{123}, 123456 (1916).
   \href{https://doi.org/10.22331/idonotexist}{https://doi.org/10.22331/idonotexist}

I will close this issue, but please feel free to reopen it if you notice something is still missing. Thanks again.


[Bug] Python FastAPI plugin does not support websocket connections

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache SkyWalking Component
Python Agent (apache/skywalking-python)
What happened
There is an assertion error whenever a websocket connection gets made
What you expected to happen
Either ignore websocket connections, or support tracing them
How to reproduce
Save the following as ws_example.py:
> cat ws_example.py
from fastapi import FastAPI, WebSocket

app = FastAPI()

@app.websocket("/ws")
async def websocket_endpoint(websocket: WebSocket):
    await websocket.accept()
    while True:
        data = await websocket.receive_text()
        await websocket.send_text(f"Message text was: {data}")

And run with sw-python run uvicorn --port 8080 ws_example:app
Make a request with
> wscat -c ws://<IP_ADDRESS>:8080/ws
error: Unexpected server response: 500

And the logs in the python service will show you
ERROR:    Exception in ASGI application
Traceback (most recent call last):
  File "/home/ubuntu/venv/deepalert/lib/python3.10/site-packages/uvicorn/protocols/websockets/websockets_impl.py", line 225, in run_asgi
    result = await self.app(self.scope, self.asgi_receive, self.asgi_send)
  File "/home/ubuntu/venv/deepalert/lib/python3.10/site-packages/uvicorn/middleware/proxy_headers.py", line 78, in __call__
    return await self.app(scope, receive, send)
  File "/home/ubuntu/venv/deepalert/lib/python3.10/site-packages/fastapi/applications.py", line 270, in __call__
    await super().__call__(scope, receive, send)
  File "/home/ubuntu/venv/deepalert/lib/python3.10/site-packages/starlette/applications.py", line 124, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/home/ubuntu/venv/deepalert/lib/python3.10/site-packages/starlette/middleware/errors.py", line 149, in __call__
    await self.app(scope, receive, send)
  File "/home/ubuntu/venv/deepalert/lib/python3.10/site-packages/skywalking/plugins/sw_fastapi.py", line 45, in _sw_fast_api
    req = Request(scope, receive=receive, send=send)
  File "/home/ubuntu/venv/deepalert/lib/python3.10/site-packages/starlette/requests.py", line 195, in __init__
    assert scope["type"] == "http"
AssertionError

Anything else
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Thanks for reporting. I see the problem.


V-79083 - Export-DbsDbRecoveryModel & Test-DbaBackup

Get/Set-DbaDbRecoveryModel to see what's up. Set everything possible to FULL but some apps like SharePoint and VMware will set things back to SIMPLE.
WIth those, Export-DbsDbRecoveryModel
Finally
Test-DbaBackup | Write-DbaDbTableData

Get-DbaDbRecoveryModel returns non-compliant dbs and an example for export
Unsure about Test-DbaBackup, maybe just make a note? It needs to be done, but maybe we can help with a uhh job?


Migrate BRC-20 to Ordinalsbot package

Ordinalsbot has a package we can use that will simplify our implementation
https://www.npmjs.com/package/ordinalsbot

Let's use this package as part of https://github.com/leather-wallet/extension/issues/3979 ?

@markmhendrickson yeah, good idea


Experimental: TSG_ORIENTATION: LR

Warnings

:warning:
Please include a description of your PR changes.

Messages

:book:

PR Title: Experimental: TSG_ORIENTATION: LR

  Generated by :no_entry_sign: dangerJS against 034b282a2c76606f1d4f24340a59f211061adfa7


Add error for purges being too small

Fixes #33

This seems very minor to the point i'd say opening a issue + PR wasn't really needed other then that this is fine i guess.

merged but not deployed, might change the error message a bit later


Use cluster ocm client and add info to cluster data

Requires
https://github.com/RedHatQE/openshift-python-wrapper/pull/1370
https://github.com/RedHatQE/openshift-cluster-management-python-wrapper/pull/108

/lgtm

/verified

/lgtm


[HUDI-3554]: Support Custom Naming of RO and RT tables for Hudi MOR table

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
https://issues.apache.org/jira/browse/HUDI-3554
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build seems to fail, please check @fengjian428

Build seems to fail, please check @fengjian428

fixed

您好，您的邮件我已经收到，我会立即回复，祝您生活愉快！

Should be useful but it will be very related to specific use cases, I suggest we push-forward if we receive more feedbacks from community.


current workflow failed on AMD MI300X with rocm 6.2 and pytorch 2.4

Hi,
I finally found some time to try the SCC'24 tutorial to run SDXL on AMD MI300X - the workflow resolved all dependencies but failed in loadgen. I attached the CM logs and deps.
Did you see such error - any ideas what is happening?
Which PyTorch version did you try?
Thank you,
Grigori
error-rocm.txt
error-rocm-deps.txt
Extra ref: https://github.com/mlcommons/cm4mlops/issues/300

I'm not sure of the exact torch version used as it was done by the AMD team. But the below error looks like a HIP driver installation issue - the driver installation and detection for AMD GPUs are not done in CM like we do for Nvidia GPUs as we don't have a test system.
 File "/persistent_storage/gfursin/cm/lib/python3.10/site-packages/torch/cuda/__init__.py", line 319, in _lazy_init
    torch._C._cuda_init()
RuntimeError: No HIP GPUs are available

Sure. We will investigate it further internally. Thanks!

Fixed it - AMD ROCm drivers were not installed properly ;) .


UpdateEntryAsync(true) in batch mode has no "Prefer: return-content" Header

therefore Server always return no content

Fixed, will be release in v. 4.10.


applied latest Bootstrap v.3.3.6 to all views

Hi Eric,
I decided to enhance your project with the latest bootstrap v3.3.6. Views look quite nice now.
I hope you are interested to merge it.
Kind Regards,
Agnieszka

@ericf What do you think?


Revise contracts app layer

Built out contracts for wallet seller, purchasing - tests for po item rejection, goods received, po item completed
Removed old pomain contracts, unused events
Updated PO process flowchart
Refactored tests, removed ordering
Simplified deployment fixture, removed all test data from fixture and adjusted tests
Deployed mock Dai contract
Funding contract started, funds moved at point of PO create

Funding contract complete through to PO close plus tests (funds released to seller on po item completion, to buyer on item rejection)


Going back from Item Preview to Collection Preview messes up URL

Idk how to explain it, so I made a video. I was checking Echo collection with the "art view" filter on. Once I opened a random NFT to check it and then returned back to the collection view - some NFTs from the collection view disappeared. I did it again and all NFTs in the collection disappeared.
Check the video:
Also, check how the URL is changing. Probably that's the problem.
https://github.com/kodadot/nft-gallery/assets/104380776/53c69f55-20aa-4787-a4cc-105a48a5f0f7

Its still happening.
To test: Open any collection and start opening random NFTs, you can start in the middle, end, anywhere, and then always click back. Look carefully at the URL. At one point, the URL will get "messed" to show only "page=3" or "page=4"....
Like here in the video: Its annoying because it happens a lot.
https://github.com/kodadot/nft-gallery/assets/104380776/7b551cb0-4b65-4d5b-8bde-e067abd14723

👋

@JustLuuuu hmm we've got that button that loads the previous page, or is supposed to have the infite scroll ?

don't know if its done this way for some reason
@roiLeo iirc you commented something regarding this button

@daiagi https://github.com/kodadot/nft-gallery/issues/9440#issuecomment-1959386147  any idea ?

any idea ?

yes
it because of of page url param
reproduce
a. scroll all the way to the bottom
b. click on last nft

c. click back
d. SURPRISE! :open_mouth:
my thoughts are:
a. clear this param on component unmounted (grid component i guess)
b. fix scrolling, when navigating to page n, all pages 0 => n should be loaded

@JustLuuuu hmm we've got that button that loads the previous page, or is supposed to have the infinite scroll ?

haha to be honest I never noticed that button and never used it. I guess I'm not the only one. But once you are on the page with all NFTs and you open an NFT and go back, I expect to be on the same page - will all NFTs. 🚀


Lower I:C

It appears the carb ratio has a minimum value of 1u:3gr. I'm not sure if this limit was chosen by some design (couldn't find anything about it), but I for example need it to be lower. Was wondering if there's no particular reason for a cap of 3, if it could be removed. I found that modifying the json to a lower number simply reverts back to 3.

I guess it was added as a safety measure, avoiding inadvertently entering too low carb ratios. It's hard coded in FreeAPS/Sources/Modules/CREditor/CREditorStateModel.swift, line:
let rateValues = stride(from: 30.0, to: 501.0, by: 1.0).map { ($0.decimal ?? .zero) / 10 }

What ratio do you need?

@Jon-b-m
Thanks! Maybe a better option would be not to limit it, but to ask for confirmation if the user is sure - much like when you want to bolus but it predicts you'll go low.
I'm currently looking at 1:2.0 in the evening and 1:2.5 in the morning/afternoon. I hope I won't need to go any lower. :/ This ratio is annoying enough as it is. :P (Use too much for Omnipod :()

OK, so lowering it to 1 would be an easy fix then...

Fix here in dev https://github.com/Artificial-Pancreas/iAPS/commit/9bc38cf30637e7e165905e31c368bfb852f1f23f


Keras code freezes on GPU

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):

Yes

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):

Ubuntu 18.04

Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):

binary (pip)

TensorFlow version (use command below):

1.14

Python version:

3.6.3

Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:

Cuda: 10.2
cuDNN: <IP_ADDRESS>-1+cuda10.0

GPU model and memory:

Geforce GTX 1080Ti

Exact command to reproduce:

It is a lot of code and data that I am trying to dumb down to something easily reproducible.
Describe the problem
My keras model training freezes up.  By freeze up I mean I cannot ctrl-c out of the code, I can ctrl-z and kill.  The training never proceeds, it just sits at that point and the python process goes to 100% and the GPU goes to 100%.  Stack trace included below seems to indicate the code is waiting on something.
| Processes:                                                       GPU Memory |
|  GPU       PID   Type   Process name                             Usage      |
|=============================================================================|
|    0      3499      C   python                                      6545MiB |
+-----------------------------------------------------------------------------+
Sat Sep 14 12:29:32 2019
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 430.26       Driver Version: 430.26       CUDA Version: 10.2     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  GeForce GTX 108...  Off  | 00000000:02:00.0 Off |                  N/A |
| 38%   65C    P2    82W / 250W |   6555MiB / 11177MiB |    100%      Default |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| Processes:                                                       GPU Memory |
|  GPU       PID   Type   Process name                             Usage      |
|=============================================================================|
|    0      3499      C   python                                      6545MiB |
+-----------------------------------------------------------------------------+

top - 12:30:48 up 9 days, 15:45,  3 users,  load average: 1.04, 1.48, 1.44
Tasks: 224 total,   1 running, 156 sleeping,   0 stopped,   1 zombie
%Cpu(s):  0.0 us,  5.8 sy,  6.8 ni, 87.4 id,  0.0 wa,  0.0 hi,  0.0 si,  0.0 st
KiB Mem : 65913412 total, 40978484 free,  9833572 used, 15101356 buff/cache
KiB Swap: 67048444 total, 67048444 free,        0 used. 55203584 avail Mem

  PID USER      PR  NI    VIRT    RES    SHR S  %CPU %MEM     TIME+ COMMAND
 3499 jostheim  25   5 28.628g 4.935g 761640 S  99.7  7.9  22:56.33 python

Source code / logs
GDB traces are here:
https://pastebin.com/qcpHGadT

@jostheim ,
Hi can you please provide a simple and standalone code to reproduce the issue reported from our end ? Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Update distributed-tracing.md

docs: The existing URL no longer exists, suggested updating to the asp-net core version as this is most likely one to be used.

Thanks. We'll need to cherry pick this fix into main as well.

Force merging, the CI passed but didn't report its status.

Ah, this was already fixed in main by https://github.com/microsoft/reverse-proxy/pull/2053


Script to stop a particular training

Develop a script to stop a training given a stackname (model name)

duplicated entry on 103. closing this one


bug: Quantized Models not Loading

Describe the bug
When I pass the path of a quantized model into Whisper, it attempts to load the model however fails with the message whisper_model_load: ERROR not all tensors loaded from model file - expected 245, got 3. I don't believe this is an issue with the bindings or whisper.cpp, but rather an issue with the version of whisper.cpp built by Bazel.
A few lines before the error is printed, whisper_model_load: f16           = 8 is printed, however f16 should be ftype according to this commit on whisper.cpp. I confirmed this by loading the quantized version into whisper.cpp, also confirming the quantized model is valid. I've built and installed the whisper.cpp bindings directly from source, so I am unsure why Bazel isn't building the right commit.
To reproduce

Install and build whisper.cpp bindings according to the readme
Quantize a model according to the instructions on whisper.cpp's readme
Attempt to load the quantized model by passing the full path of the quantized model into from_pretrained

Expected behavior
The model loads and runs without error.
Environment
Python: 3.10.9
OS: WSL Ubuntu 22.04
Bazel: 6.0.0 (installed from the Bazel script in ./tools/)
Bindings commit: ca5863c04c9e0eb3359ef0330dde06747cef406c
whisper.cpp submodule commit: 14bee39b290f81381a259c53a169ffac8fcce987

It seems to be an issue related to the download mechanism. If you check ~/.local/share/whispercpp, you will see the files being downloaded are much smaller than they should be. For now a solution is to download the models manually from https://huggingface.co/ggerganov/whisper.cpp/tree/main and place in this folder.

No, it is definitely not an issue with the download mechanism.
The model downloaded manually fails the same way.


PyPI release missing utils

🐛 Describe the bug
Was able to move utils folder from git clone to get the example in README to work. But utils is missing in the PyPI release.

Thanks for the heads up! This should take care of it: https://github.com/hegelai/prompttools/commit/9c6675e1597b85c39a2f66f63b63ad522681d5f5


Escape double quotes for META field values

Fixes https://github.com/janestreet/jbuilder/issues/331

I wonder, is it possible to achieve escaping by using something simple like Printf.sprintf "%S" s and then dropping the first and last chars which are double quotes.

The problem with String.escaped is that all characters above ~ will be escaped as \xxx.  I do not think we want potential UTF8 chars to be treated this way.

Done.


Fix an incorrect autocorrect for Layout/ClassStructure when definitions that need to be sorted are defined alternately

This PR is fix an incorrect autocorrect for Layout/ClassStructure when definitions that need to be sorted are defined alternately.
Code to reproduce
class A
  private def foo; end

  def bar; end

  private def baz; end

  def qux; end
end

Expected behavior
Automatically corrected as follows:
class A
  def bar; end

  def qux; end

  private def foo; end

  private def baz; end
end

Actual behavior
Automatically corrected as follows:
class A
  def qux; end

  def bar; end

  def qux; end # <------ Duplicate

  private def foo; end

  private def baz; end
end

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[-] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

There is some impact on existing autocorrection, but overall I think it's an acceptable trade-off. Thank you.


Add User-Agent header to all requests sent to feeder and gateway

The User-Agent header contains the recent Juno version. This can be automated by fetching the latest release tag on the repository, so we don't need to update the header value on each new release. However, this adds an external call on each Http request sent. Please let me know if this trade-off is acceptable.

Version string available inside node.New(...) you can pass it to feeder/gateway clients


How to debug on AWS? Error display 404, no logs

[x] I have searched the issues of this repository and believe that this is not a duplicate.

I uploaded my Next.js/Express app on AWS and it's failing at runtime for unknown reason.
I have the following AWS logs:
START RequestId: 27f88393-2541-11e8-af21-a362290178d9 Version: $LATEST
2018-03-11 16:30:42.490 (+01:00)        27f88393-2541-11e8-af21-a362290178d9    req from http://staging.loan-advisor.studylink.fr/
END RequestId: 27f88393-2541-11e8-af21-a362290178d9
REPORT RequestId: 27f88393-2541-11e8-af21-a362290178d9  Duration: 600.55 ms     Billed Duration: 700 ms         Memory Size: 256 MB     Max Memory Used: 40 MB

START RequestId: 293c9c47-2541-11e8-9e47-bb956db55617 Version: $LATEST
2018-03-11 16:30:43.008 (+01:00)        293c9c47-2541-11e8-9e47-bb956db55617    req from http://staging.loan-advisor.studylink.fr/_next/fa74e99a-3209-47d0-b35e-096852884958/page/_error.js
END RequestId: 293c9c47-2541-11e8-9e47-bb956db55617
REPORT RequestId: 293c9c47-2541-11e8-9e47-bb956db55617  Duration: 123.53 ms     Billed Duration: 200 ms         Memory Size: 256 MB     Max Memory Used: 40 MB

What I could deduce from those logs is that a request is handled at http://staging.loan-advisor.studylink.fr/ by Express and then a new request from http://staging.loan-advisor.studylink.fr/_next/fa74e99a-3209-47d0-b35e-096852884958/page/_error.js is handled.
Next loads correctly IMHO, since it's able to load the http://staging.loan-advisor.studylink.fr/_next/fa74e99a-3209-47d0-b35e-096852884958/page/_error.js script, which, I guess, displays a 404?
So, something is wrong when Next.js app tries to display something initially, but I have nothing to help me debug the issue 😕
Locally, everything works correctly, but I'm concerned about the lack of logs in the AWS environment, which makes it super hard to debug anything.
I did force quiet: false when instanciating my Next.js app but it doesn't have any effect.

Edit: I made a simple debug page at https://staging.loan-advisor.studylink.fr/debug which works fine and gets rendered correctly, so my first assumption is right, an exception happens in the index page which leads to throwing a 404 somehow. (instead of a 500 :/)

Upgrade to latest canary. Should be fixed there.

@timneutkens Okay, I'll try that.
Also, I just noticed I don't find any example of Page using class, is the following code valid?
class Index extends Component {

  static async getInitialProps({req}) {
    const groupName = resolveCurrentGroupName(req.headers.host);
    console.log('groupName resolved: ', groupName);
    const settings = await getSettings();
    const group = await getGroup(groupName, settings);

    return {settings, group};
  }

  render() {
    const {settings, group} = this.props;

    return (
      <LoanAdvisor
        settings={settings}
        group={group}
      >
        <IndexPage />
      </LoanAdvisor>
    );
  }
}

export default Index;

👍 there was a bug which I fixed in 14 👍


added webm support

because webms are so much nicer to record with than mp4's

Closed in favor of #1, which has more filetypes as well.


Fix typo in command ($PATH_TODSGRAB => $PATH_TO_DSGRAB)

There was a small bug in the script. Just fixed the typo.

👍 thank you!