Generating backup checksum

I'm wondering if it would be hard to implement generating checksum for each archive. If you don't have time to implement it, maybe you have some advice where to put this? Maybe even separate module?

@szymonpk sounds like a neat idea. I currently have zero knowledge of how to generate a checksum on files, I assume there are unix utilities that can take care of this? It would be great to be able to check whether the local and the remote files are the same, and raise an exception if not.
What's important is that this utility is both available on Linux and OSX. Is it a default utility, or one that you must install via a package manager? Or are there built-in Ruby functions that can do this? As I said, I have zero experience with this so if you could provide some information on what tools there should be used, maybe some simple command snippets so I can try it locally, etc, that would be great!
Also, something like this would likely want it's own class/module depending on how it's being implemented. A new object would have to be created inside the storage objects (FTP/SFTP/SCP/etc). Also, I'm not sure whether this would work with services like Amazon S3, Cloud Files, Ninefold, Dropbox, etc. I would assume you need the checksum utility on both the local as well as the remote machine? In which case it would only work with the FTP/SFTP/SCP storage methods.
But, if you could provide me with some more insight in what utilities to use, how it should be used, maybe some code snippets to get a good checksum going so I can test it locally, then I can better tell what/how/where and if it could/should be implemented.
Cheers!

Cool, thanks for the detailed explanation!
This, and #241 are what I want to get incorporated if possible in one of the future releases of backup. I'll have to look at the possibilities.
A question though as I don't know how to get it working, i'm probably doing something wrong.
I have a file called foo.rb and I attach a sha1 string (prefixed) to it with:
mv foo.rb "$(shasum foo.rb)"

So then I have:
57e585af50fec18c1373e26aa679de8386cb0680  foo.rb

Now when I run this:
shasum -c "57e585af50fec18c1373e26aa679de8386cb0680  foo.rb"

It doesn't return anything and stays silent, but when I run it with the -w flag (flags: show warnings)
shasum -wc "57e585af50fec18c1373e26aa679de8386cb0680  foo.rb"
shasum: 57e585af50fec18c1373e26aa679de8386cb0680  foo.rb: 1: improperly formatted SHA checksum line
shasum: 57e585af50fec18c1373e26aa679de8386cb0680  foo.rb: 2: improperly formatted SHA checksum line
shasum: 57e585af50fec18c1373e26aa679de8386cb0680  foo.rb: 3: improperly formatted SHA checksum line
shasum: 57e585af50fec18c1373e26aa679de8386cb0680  foo.rb: 4: improperly formatted SHA checksum line
shasum: 57e585af50fec18c1373e26aa679de8386cb0680  foo.rb: 5: improperly formatted SHA checksum line
shasum: 57e585af50fec18c1373e26aa679de8386cb0680  foo.rb: 6: improperly formatted SHA checksum line
shasum: 57e585af50fec18c1373e26aa679de8386cb0680  foo.rb: 7: improperly formatted SHA checksum line
shasum: 57e585af50fec18c1373e26aa679de8386cb0680  foo.rb: 8: improperly formatted SHA checksum line

Any idea what I'm doing wrong?
→ shasum -h
Usage: shasum [OPTION] [FILE]...
   or: shasum [OPTION] --check [FILE]
Print or check SHA checksums.
With no FILE, or when FILE is -, read standard input.

  -a, --algorithm    1 (default), 224, 256, 384, 512
  -b, --binary       read files in binary mode (default on DOS/Windows)
  -c, --check        check SHA sums against given list
  -p, --portable     read files in portable mode
                         produces same digest on Windows/Unix/Mac
  -t, --text         read files in text mode (default)

The following two options are useful only when verifying checksums:
  -s, --status       don't output anything, status code shows success
  -w, --warn         warn about improperly formatted SHA checksum lines

  -h, --help         display this help and exit
  -v, --version      output version information and exit

The sums are computed as described in FIPS PUB 180-2.  When checking, the
input should be a former output of this program.  The default mode is to
print a line with checksum, a character indicating type (`*' for binary,
`?' for portable, ` ' for text), and name for each FILE.

I think you misunderstanded me a little. You don't create file '57e585af50fec18c1373e26aa679de8386cb0680 foo.rb' but file foo.sum WITH content:
57e585af50fec18c1373e26aa679de8386cb0680  foo.rb

Full example:
# compute sum and write it to file
$ shasum foo.rb >> foo.sum
# check sum
$ shasum -c foo.sum
foo.rb: OK

Ah, now it works. Yeah I misunderstood that part. So you basically append multiple sums to that single file in the case if backing up a backup in multiple chunks?

Yes, exactly as you've described. It's easy to check if data is consistent.

Have you tried to implement this? If not, maybe do you have any ideas where it should be implemented.

I think @burns might be on to something already. It's something that we'd like to have implemented, but preferably only if it works across all storages (for consistency). We just finally pushed out 3.0.20 so we're now looking in to issue #241 and this.


http: O(1) custom header map

Implement custom O(1) header registration for header maps.

Remove virtual inheritence from header maps.
Implement variable inline storage for concrete header maps.
Various TODO/logic cleanups from previous header refactors.
Demonstrate what custom header registration looks like for the
CORS filter. More cleanup of this type can be done.

Fixes https://github.com/envoyproxy/envoy/issues/4815
Risk Level: High
Testing: TBD
Docs Changes: N/A
Release Notes: N/A

Here is a perf delta between the old header map and the new one on the benchmark:
ubuntu@ip-10-0-0-215:~$ Source/benchmark/tools/compare.py benchmarks ./old_header.json ./new_header.json
Comparing ./old_header.json to ./new_header.json
Benchmark                                          Time             CPU      Time Old      Time New       CPU Old       CPU New
-------------------------------------------------------------------------------------------------------------------------------
HeaderMapImplCreate                             +0.4041         +0.4041            13            19            13            19
HeaderMapImplSetReference/0                     +0.0491         +0.0491            46            48            46            48
HeaderMapImplSetReference/1                     +0.0372         +0.0372            53            55            53            55
HeaderMapImplSetReference/10                    +0.0150         +0.0150           141           143           141           143
HeaderMapImplSetReference/50                    +0.0192         +0.0192           338           344           338           344
HeaderMapImplGet/0                              -0.0026         -0.0026            10            10            10            10
HeaderMapImplGet/1                              +0.0407         +0.0407            19            19            19            19
HeaderMapImplGet/10                             +0.0223         +0.0223           106           108           106           108
HeaderMapImplGet/50                             +0.0204         +0.0204           300           306           300           306
HeaderMapImplGetInline/0                        +1.4427         +1.4427             2             6             2             6
HeaderMapImplGetInline/1                        +1.4472         +1.4472             2             6             2             6
HeaderMapImplGetInline/10                       +1.4388         +1.4387             2             6             2             6
HeaderMapImplGetInline/50                       +1.4444         +1.4443             2             6             2             6
HeaderMapImplSetInlineMacro/0                   +0.2190         +0.2191             9            11             9            11
HeaderMapImplSetInlineMacro/1                   +0.2246         +0.2246             9            11             9            11
HeaderMapImplSetInlineMacro/10                  +0.1733         +0.1732            10            11            10            11
HeaderMapImplSetInlineMacro/50                  +0.2539         +0.2539             9            12             9            12
HeaderMapImplSetInlineInteger/0                 -0.0202         -0.0202            28            27            28            27
HeaderMapImplSetInlineInteger/1                 +0.0159         +0.0159            28            28            28            28
HeaderMapImplSetInlineInteger/10                -0.0111         -0.0111            28            27            28            27
HeaderMapImplSetInlineInteger/50                -0.0142         -0.0143            28            27            28            27
HeaderMapImplGetByteSize/0                      -0.1622         -0.1622             1             1             1             1
HeaderMapImplGetByteSize/1                      -0.1627         -0.1627             1             1             1             1
HeaderMapImplGetByteSize/10                     -0.1495         -0.1494             1             1             1             1
HeaderMapImplGetByteSize/50                     -0.1522         -0.1522             1             1             1             1
HeaderMapImplIterate/0                          +0.0611         +0.0611             2             2             2             2
HeaderMapImplIterate/1                          +0.2846         +0.2846             3             4             3             4
HeaderMapImplIterate/10                         +0.0197         +0.0197            14            15            14            15
HeaderMapImplIterate/50                         +0.0061         +0.0061            74            74            74            74
HeaderMapImplLookup/0                           +0.5095         +0.5095            13            20            13            20
HeaderMapImplLookup/1                           +0.5076         +0.5076            13            20            13            20
HeaderMapImplLookup/10                          +0.5062         +0.5062            13            20            13            20
HeaderMapImplLookup/50                          +0.5088         +0.5088            13            20            13            20
HeaderMapImplRemove/0                           +0.0540         +0.0540            53            56            53            56
HeaderMapImplRemove/1                           +0.0334         +0.0334            60            62            60            62
HeaderMapImplRemove/10                          +0.0245         +0.0244           148           151           148           151
HeaderMapImplRemove/50                          +0.0221         +0.0222           345           352           345           352
HeaderMapImplRemoveInline/0                     +0.2864         +0.2863            67            86            67            86
HeaderMapImplRemoveInline/1                     +0.2814         +0.2813            67            86            67            86
HeaderMapImplRemoveInline/10                    +0.3723         +0.3723            67            92            67            92
HeaderMapImplRemoveInline/50                    +0.3782         +0.3782            67            92            67            92
HeaderMapImplPopulate                           +0.2822         +0.2822           412           528           412           528

There is definitely a cost for the new implementation as it involves some more cycles of indirection to look up various pieces of data, and a few more virtual calls. I can likely run some of this through callgrind and optimize it some more. Overall though I think the cost is likely worth it given the cleanup and the opportunity to have more custom headers, and compile out custom headers that the user doesn't care about.
Note that there is further cleanup work I want to do including removing more default headers and moving them into extensions as well as making the route header add/remove functions pre-lookup O(1) headers and use those if they are defined instead of the O(N) versions.
I also want to add some printing of registered headers and sizes at server startup. I will do that on this PR before we finalize it. @jmarantz I wanted to get some initial feedback of this refreshed code though. I have fixed all the previous comments and the impl is even simpler now.

Thanks for fleshing this out!
Should we also try a pure flat_hash_map with int-ordered entries just to see what perf looks like there as well, a well as try to compare in one table the performance of this vs the lazy-map variants described in https://docs.google.com/document/d/1yxyRODqaLwMyzBGRkzF_6DJGCGfp4nfjzkUIhtQ8Tc0/edit and prototyped in https://github.com/envoyproxy/envoy/pull/9261 ?

Should we also try a pure flat_hash_map with int-ordered entries just to see what perf looks like there as well, a well as try to compare in one table the performance of this vs the lazy-map variants described in https://docs.google.com/document/d/1yxyRODqaLwMyzBGRkzF_6DJGCGfp4nfjzkUIhtQ8Tc0/edit and prototyped in #9261 ?

The issue with ^ is that IMO to do it justice we need to flesh out the benchmarks we have quite a bit, to account for the different perf profiles that we expect to see with the different implementations (e.g. re-ordering on encoding, possibly maintaining a linked hash map, building the lookaside map, etc.). My preference would be to land this as we understand the perf profile, it will improve perf in certain cases, and the perf degradation above over the existing impl is on the order of single digit nanoseconds per operation. After that I can start poking at some other stuff in my spare time. WDYT? If we feel that want to compare this with the other potential implementations that will take a lot more time.

I agree that having a comprehensive suite that evaluates the best strategy across realistic use-cases is out-of-scope.
I was just thinking about getting a matrix of results for the existing speed tests.
@htuch would be interested in your take here too.

Looks cleaner to me, and more readable, thanks!
The registration of custom headers seems straight-forward.
SGTM!

Yeah, I think landing this is totally fine as an immediate step. I'd love to see the absl::flat_hash_map comparison at some point, because I think there is a lot of opportunity to simplify if we can find a way to make that work (and even if we don't, we would have convinced ourselves that the current level of complexity is necessary).

I'm willing to work on additional implementations and benchmarks after I land this, but if we agree that we can land this now that would avoid it going out of date as we do that. Since it sounds like we agree, I will finish the PR and get it into its final state, but PTAL now and lmk if you have any major comments.

@jmarantz this should be ready for final review.

Would you consider adding a header_map.md file to describe the architecture, data structure diagram, threading model, life-of-a-custom-header, etc?

Yes will do.

@jmarantz updated PTAL and lmk if you want to see anything else in the markdown file.


Conditional types cause failure of substitution property of definitions

By 'substitution property' I mean in this case that if I define
type Foo<T> = Bar<T, T>;

then I expect the expressions that have the type Foo<T> to be the same as those that have type Bar<T, T>, for any T. If this property is a non-goal --- if its violation is an expected consequence of distributive conditional types --- then feel free to close this issue.
TypeScript Version:  2.8.0-dev commit 0c6ef348ce1
Search Terms: conditional types, substitution
Code
type MyTypes = {
  'text': string;
  'bool': boolean;
};

type Datum<K, V> = { t: K, v: V };
type FromKeys<T, U> = T extends keyof U ? Datum<T, U[T]> : never;
type UnionMaker2<T, U> = FromKeys<keyof T, U>;
type UnionMaker<T> = UnionMaker2<T, T>;

const x1: UnionMaker2<MyTypes, MyTypes> = {t: 'text', v: 'string'};
const x2: UnionMaker2<MyTypes, MyTypes> = {t: 'bool', v: true};
// fails:
// const x3: UnionMaker2<MyTypes, MyTypes> = {t: 'text', v: true};

// however this succeeds?
const x4: UnionMaker<MyTypes> = {t: 'text', v: true};

Expected behavior:
Both x3 and x4 should fail to typecheck. I expect both UnionMaker<MyTypes> and UnionMaker2<MyTypes, MyTypes> to be
{t: 'text', v: string} | {t: 'bool', v : boolean}

And most of all I expect UnionMaker<MyTypes> in any event to be identical in behavior to UnionMaker2<MyTypes, MyTypes>, because of the definition of UnionMaker.
Actual behavior:
x3 if uncommented yields
../a.ts(17,7): error TS2322: Type '{ t: "text"; v: boolean; }' is not assignable to type 'Datum<"float", number> | Datum<"text", string> | Datum<"bool", boolean> | Datum<"price", number> ...'.
  Type '{ t: "text"; v: boolean; }' is not assignable to type 'Datum<"text", string>'.
    Types of property 'v' are incompatible.
      Type 'boolean' is not assignable to type 'string'.
5:54:33 PM - Compilation complete. Watching for file changes.

but x4 typechecks without error.
Related:
Conditional types PR https://github.com/Microsoft/TypeScript/pull/21316
Stackoverflow question which prompted this observation: https://stackoverflow.com/questions/43197523/can-an-indexed-type-map-be-expanded-into-a-discriminated-union/49266392#49266392

I just found https://github.com/Microsoft/TypeScript/issues/22302 after searching recent git commit history for 'conditional', seeing if that's related...

After looking at it, I'm not convinced it is relevant, but perhaps someone who knows the implementation of conditional types better can correct me if it is.

With typescript@next I see the expected failure on x3 and x4 ?

Apologies, I can repro the expected failure now. I didn't realize that TypeScript/bin/tsc from a fresh clone of the repo was merely LKG and not the same as the result from building or npm-installing @next. Should have read the docs more carefully! :)

No worries - thanks for trying it out!

Reciprocal thanks to the whole ts team for all their work. I am super excited about conditional types; they seem like a very expressive feature in combination with the other type-level operations already present.


Are there plans to implement segmentation in Pytorch?

As written using TensorFlow here: https://github.com/DylanWusee/pointconv

@isabellahuang I plan on writing that and would like to cooperate with you. Can I have you email or something to contact?

Feel free to contact me at @.***
On Sun, Jun 20, 2021 at 10:21 PM tranganhthuan @.***>
wrote:

@isabellahuang https://github.com/isabellahuang I plan on writing that
and would like to cooperate with you. Can I have you email or something to
contact?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DylanWusee/pointconv_pytorch/issues/13#issuecomment-864737050,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJ2RHSYNU5MSGXOGH2ZVZLTT3D47ANCNFSM42RXAY6Q
.

I can not see your email, just type it with out "@" and ".". I will contact you.

Isabella[dot]huang[at]berkeley[dot]edu
On Mon, Jun 21, 2021, 6:48 PM tranganhthuan @.***>
wrote:

I can not see your email, just type it with out "@" and ".". I will
contact you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DylanWusee/pointconv_pytorch/issues/13#issuecomment-865458628,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJ2RHSZFYPVR4R3FKBDCOLTT7TXDANCNFSM42RXAY6Q
.

@tranganhthuan Did you get a chance to finish your work for segmentation?

This repo will no longer be updated.
For Semantic Segmentation on ScanNet, Please refer to the code in https://github.com/Xingyi-li-Vince/VI_PointConv, which includes the code to process the dataset and a similar deep model for ScanNet dataset.


[get-start]update build first Message extension

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/get-started/first-message-extension.md
88
20
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 68f318d:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/get-started/first-message-extension.md
:warning:Warning
View
Details

msteams-platform/get-started/first-message-extension.md

Line 123, Column 4: [Warning-file-not-found] Invalid file link: '../assets/images/teams-toolkit-v2/search-message-extension.png'.
Line 123, Column 4: [Warning-duplicate-alt-text] Alt text 'Your Search-based messaging extension in action' is duplicated. Within an article, alt text must be unique.
Line 126, Column 5: [Warning-file-not-found] Invalid file link: '../assets/images/teams-toolkit-v2/msgext-teams-search-bar.png'.
Line 126, Column 5: [Warning-duplicate-alt-text] Alt text 'Your Search-based messaging extension in action' is duplicated. Within an article, alt text must be unique.
Line 129, Column 5: [Warning-file-not-found] Invalid file link: '../assets/images/teams-toolkit-v2/msgext-adptive-card.png'.
Line 129, Column 5: [Warning-duplicate-alt-text] Alt text 'Your Search-based messaging extension in action' is duplicated. Within an article, alt text must be unique.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/get-started/first-message-extension.md
88
20
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit a317eef:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/get-started/first-message-extension.md
:warning:Warning
View
Details

msteams-platform/assets/images/teams-toolkit-v2/msgextn-file-layout.png
:white_check_mark:Succeeded
View

msteams-platform/get-started/first-message-extension.md

Line 123, Column 4: [Warning-file-not-found] Invalid file link: '../assets/images/teams-toolkit-v2/search-message-extension.png'.
Line 123, Column 4: [Warning-duplicate-alt-text] Alt text 'Your Search-based messaging extension in action' is duplicated. Within an article, alt text must be unique.
Line 126, Column 5: [Warning-file-not-found] Invalid file link: '../assets/images/teams-toolkit-v2/msgext-teams-search-bar.png'.
Line 126, Column 5: [Warning-duplicate-alt-text] Alt text 'Your Search-based messaging extension in action' is duplicated. Within an article, alt text must be unique.
Line 129, Column 5: [Warning-file-not-found] Invalid file link: '../assets/images/teams-toolkit-v2/msgext-adptive-card.png'.
Line 129, Column 5: [Warning-duplicate-alt-text] Alt text 'Your Search-based messaging extension in action' is duplicated. Within an article, alt text must be unique.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/get-started/first-message-extension.md
88
20
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/get-started/first-message-extension.md
88
20
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit baa771e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/get-started/first-message-extension.md
:warning:Warning
View
Details

msteams-platform/assets/images/teams-toolkit-v2/msgextn-create-project-capabilities.png
:white_check_mark:Succeeded

msteams-platform/assets/images/teams-toolkit-v2/msgextn-file-layout.png
:white_check_mark:Succeeded
View

msteams-platform/get-started/first-message-extension.md

Line 48, Column 4: [Warning-file-not-found] Invalid file link: '../assets/images/teams-toolkit-v2/msgextn-create-project-capabilities.png'.
Line 123, Column 4: [Warning-file-not-found] Invalid file link: '../assets/images/teams-toolkit-v2/search-message-extension.png'.
Line 123, Column 4: [Warning-duplicate-alt-text] Alt text 'Your Search-based messaging extension in action' is duplicated. Within an article, alt text must be unique.
Line 126, Column 5: [Warning-file-not-found] Invalid file link: '../assets/images/teams-toolkit-v2/msgext-teams-search-bar.png'.
Line 126, Column 5: [Warning-duplicate-alt-text] Alt text 'Your Search-based messaging extension in action' is duplicated. Within an article, alt text must be unique.
Line 129, Column 5: [Warning-file-not-found] Invalid file link: '../assets/images/teams-toolkit-v2/msgext-adptive-card.png'.
Line 129, Column 5: [Warning-duplicate-alt-text] Alt text 'Your Search-based messaging extension in action' is duplicated. Within an article, alt text must be unique.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit ab9ecfe:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/get-started/first-message-extension.md
:warning:Warning
View
Details

msteams-platform/assets/images/teams-toolkit-v2/msgext-adptive-card.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/msgext-teams-search-bar.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/msgextn-create-project-capabilities.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/msgextn-file-layout.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/search-message-extension.png
:white_check_mark:Succeeded
View

msteams-platform/get-started/first-message-extension.md

Line 123, Column 4: [Warning-duplicate-alt-text] Alt text 'Your Search-based messaging extension in action' is duplicated. Within an article, alt text must be unique.
Line 126, Column 5: [Warning-duplicate-alt-text] Alt text 'Your Search-based messaging extension in action' is duplicated. Within an article, alt text must be unique.
Line 129, Column 5: [Warning-duplicate-alt-text] Alt text 'Your Search-based messaging extension in action' is duplicated. Within an article, alt text must be unique.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


huawei sync instance offset bugfix

这个 PR 实现什么功能/修复什么问题:

huawei sync instance offset bugfix

是否需要 backport 到之前的 release 分支:

release/2.13
release/2.12
release/2.11
release/2.10.0

/area region
/cc @swordqiu @yousong @ioito

Codecov Report

Merging #4141 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #4141   +/-   ##
======================================
  Coverage    6.87%   6.87%
======================================
  Files         519     519
  Lines       87546   87546
======================================
  Hits         6021    6021
  Misses      81027   81027
  Partials      498     498

Flag
Coverage Δ

#aFlag
6.87% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

/lgtm

/lgtm

/approve


Remove heading method

This method can use existing _toggleHeading private method to allow removing heading markup (if present).
This method can be used to format heading to regular paragraph (with was my use case)

Thanks for contributing, however a method on its own won't be too helpful. You'll need to add a built in button to interact with it as well. Try adding the following element to the toolbarBuiltInButtons array variable after the heading-3 element:
"heading-remove": {
	name: "heading-remove",
	action: removeHeading,
	className: "fa fa-header fa-header-x fa-header-remove",
	title: "Remove Heading"
},

And then add the following CSS snippet to the CSS file:
.editor-toolbar a.fa-header-remove:after {
	content: "X";
}

Then, you'll also need to document the code in the README in the same manner as the other built-in icons are.
Finally, please set up a JSFiddle demonstrating the new button so that it can be tested before it's merged.

@WesCossick
In my mind this was rather api function for developers then toolbar functionality for end user.
I created another PR with more usefully functionality that can be also used for cleaning other types of block formatting.


Warn about unused comments

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.59%. Comparing base (df1c22b) to head (0b4e31e).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1027      +/-   ##
==========================================
+ Coverage   94.55%   94.59%   +0.04%
==========================================
  Files          96       96
  Lines        9718     9792      +74
==========================================
+ Hits         9189     9263      +74
  Misses        374      374
  Partials      155      155

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


perf(coverage): improve HitMap merging and internal repr

use u32 and HashMaps instead of usize/u64 and BTreeMap
simplify HitMaps::merge

Lots of time spent in merge for inserting in the right order, but the order was never checked (consistent_bytecode is unused and HitMap::merge returned Result, presumably this was removed at a certain point)
forge coverage on solmate from 29s to 16s

29s to 16s

wow, just hitmaps

Yeah turns out that optimizing a function that's called 2 billion times might have a lot of impact :+1:

asked superform to report back new coverage time, would be interesting to see, starting ref 1h15m
https://github.com/superform-xyz/superform-core/actions/runs/12116942111/job/33780028573?pr=614


Can not serialise all vehicle information

https://strusoft.freshdesk.com/a/tickets/31188

https://strusoft.fogbugz.com/f/cases/125323/Deserialise-object-with-CLR-classes


ADD TOKEN ZIA

ADD TOKEN ZIA

[ ] I will not ping the Discord about this pull request.

Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR
At minimum each entry should have

Token Address: 78kKxYt1c8exXFRhc8gZ2uUySeEBM3DyMhxcf61UmkvZ
Token Name: ZIA
Token Symbol: ZIA_ZILLIQA
Logo https://user-images.githubusercontent.com/7285831/122025305-7b0b0980-cdf3-11eb-8832-9f967d30f9a8.png
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Error


REFER ua event?

When baresip ua receives REFER request from the other party of the call, I don't see a ua event being generated so that the ua receiving the REFER could decide whether to accept of reject it.  Instead the ua  automatically accepts the REFER with 202 response and generates INVITE based on it.
This seems dangerous to me.  Is there any means to allow the ua receiving REFER to ask the user what to do with the REFER?

Before asking of confirmation is supported, it should be possible to remove REFER from uag_allowed_methods() and use null as refer handler in sipsess_connect() call.

Alfred E. Heggestad writes:

you are right, it is currently implemented like that. But there is no
specific reason for it, we can easily change the behaviour.
the code in question is here in ua.c:
	case CALL_EVENT_TRANSFER:

...

one solution can be to move this block out of libbaresip.a, to menu.so
or app.

Yes, app (menu.so is app) would be the right place so that app can act on
the event.
And we would need to have an acc param to tell if REFER is advertised at
all, since if the app decides not to support call transfer, it does
not make sense to let the other party think that is supported because
REFER appears in Allow header.

Currently API does not have this call:
		err = ua_call_alloc(&call2, ua, VIDMODE_ON, NULL, call,
				    call_localuri(call), true);

Perhaps should be added to API?  Or would it be better to leave the call
in 'case CALL_EVENT_TRANSFER' and then give call2 as arg to the ua_event?
This one does not exist in API either:
(void)call_notify_sipfrag(call, 500, "Call Error");

thanks @juha-h
just a note about timing; this work will be merged to master after 0.5.11 is released.

I think we first need to decide if the call_transfer=yes|no config option
should be global, or per SIP Account.
perhaps "Call Transfer" is a better and more descriptive term than "refer".
Refer is very SIP specific.

Alfred E. Heggestad writes:

I think we first need to decide if the call_transfer=yes|no config option
should be global, or per SIP Account.

I my opinion, per SIP account.  Sometimes I need transfer and sometimes
I want to be protected from it.

perhaps "Call Transfer" is a better and more descriptive term than "refer".
Refer is very SIP specific.

So, the account param would be called ;call_tansfer with "yes" and "no"
as possible values?

I renamed account param to ;call_transfer and added account_call_transfer() and account_set_call_transfer() functions plus test case.
Now it should be possible to turn off call transfer. Next step would be to add the ua event, but would need guidance on it.

can you create a PR from your branch, so I can review it and comment on it ?

I'll close this one and open a new one on handling of CALL_EVENT_TRANSFER in application (menu) module.


Missing 2.1.1 binaries, linked from Scala website

reported on Scala Discord just now:

hello, I'm trying to install Scala, but the link on the official website appears to be dead? (for Windows)
this link:
https://github.com/coursier/coursier/releases/download/v2.1.1/cs-x86_64-pc-win32.zip
found on this page:
https://docs.scala-lang.org/getting-started/index.html

That link works for me, not sure why it's a problem for that Discord user.

it works now — didn't work at the time I opened the ticket

The link on the website is actually to latest, which at the time redirected to v2.1.1.
The discord user happened to try using the latest link between when the v2.1.1 tag was created and when the v2.1.1 github actions finished the "Upload launchers to GitHub release assets" step. Since the action has finished by now, the link does work. However, I imagine the latest link would have the same hourish of downtime immediately after any future coursier releases.
If there were a way to mark a release as latest after the actions have finished instead of at the start the actions, that should prevent the latest binary links from having any downtime, but I don't know if there is a way to do that.

If there were a way to mark a release as latest after the actions have finished instead of at the start the actions, that should prevent the latest binary links from having any downtime, but I don't know if there is a way to do that.

That’s what we should do, yes…


tensorflow Module object has no attribute keras.

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): Pip install...
TensorFlow version: latest
Python version: python 2.12.15
Installed using virtualen? pip? conda?:pip/virtualenv
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: Cuda 9.2
GPU model and memory: NVIDIA MX150

Describe the problem
I have tried to install tensorflow with gpu support both with pip and virualenv several times, I have got following erros. anyone has idea?

`python
Python 2.7.15rc1 (default, Nov 12 2018, 14:31:15)
[GCC 7.3.0] on linux2
Type "help", "copyright", "credits" or "license" for more information.

import tensorflow
Traceback (most recent call last):
File "", line 1, in
File "tensorflow.py", line 2, in
mnist = tf.keras.datasets.mnist
AttributeError: 'module' object has no attribute 'keras'`

Provide the exact sequence of commands / steps that you executed before running into the problem
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

I suspect its a due to lower tf version.  What's your tf version? Can you try updating tf
pip install -U tensorflow

Hi, I have 1.12.0
Is there any requirement for CUDA version?
Name: tensorflow Version: 1.12.0 Summary: TensorFlow is an open source machine learning framework for everyone. Home-page: https://www.tensorflow.org/ Author: Google Inc. Author-email<EMAIL_ADDRESS>License: Apache 2.0 Location: /.local/lib/python3.6/site-packages Requires: tensorboard, protobuf, keras-applications, wheel, astor, absl-py, termcolor, grpcio, numpy, six, gast, keras-preprocessing Required-by:

I think there was some CUDA toolkit problem, it worked after following
https://medium.com/@taylordenouden/installing-tensorflow-gpu-on-ubuntu-18-04-89a142325138

Yes, if you are using TF 1.12 gpu version then you have to install cuda 9.0. I will close this issue since you have found the solution. Thanks!


Update workflows + typo fix

awesome, thanks for getting the build working I wasn't sure how to fix it


chore(config): add go mod configuration

Adds go mod configuration.

Just got back from vacation.  I recreated your go.mod change since you closed this and pushed it.  Thanks for prodding this!


Check to see if zenodo account is linked

Opening as draft. Currently you will be redirected back to the workflow but not the versions tab itself after linking your zenodo account. I think it's probably ok for now.

FYI, changing base branch @david4096

Thanks @denis-yuen this is now open for review!

Added pipe first and Andrew's comments to the other ticket for this. dockstore/dockstore#2846


Fixes bug 127938 - Update FullCalendar to latest version

Coverage remained the same at 90.36% when pulling 0dad47fd17ab54ad31c03f49b8bf1187e56f348d on lcamacho:bug-127938 into c758fb86695e16891e45dc24a2cf1406c54f2686 on mozilla:master.

So, we need to change the back end. When I run this I get a 400 Bad Request on the URL:
http://airmozilla.dev/calendar/data.json?start=2015-02-02T00%3A00%3A00-08%3A00&end=2015-03-14T00%3A00%3A00-07%3A00&_=1422921207335
I think it's because before it used to send UTC timestamps. E.g. from prod
https://air.mozilla.org/calendar/data.json?start=1422864000&end=1426316400&_=1422921225056
So I think you need to change CalendarDataForm to instead of taking in to timestamps, it should take in two datetimes.
Something like this might actually work:
# UNTESTED
class CalendarDataForm(BaseForm):
    start = forms.DateTimeField()
    end = forms.DateTimeField()
    # same clean() as before

There should be tests also that you need to change.

Also, do you think you can look into making the buttons appear better?

I think they look strangely spaced together. It looks much better on prod where the buttons work as "pill boxes" (I think it's called).
Also, when I hover over the buttons there's a strange gradient of blue. Can we remove that? Not only is it odd to have gradients like that, there's no other button that has that anywhere on the site, it's also new blue we don't use anywhere else I don't think.
I'm sorry if you're no expert at FullCalendar's CSS but I know you're clever so maybe you can find something.
We can't edit the fullcalendar.min.css and we shouldn't but that's why we have the calendar.css file. Perhaps that file needs a complete refresher since it has hacks that were made based on the old FullCalendar.

Coverage increased (+0.01%) to 90.4% when pulling 413616a2ad8311f8d332b4f3c222682a48e405d9 on lcamacho:bug-127938 into e45b7f934abaefabd017a368a378242f98ebb45a on mozilla:master.

What has changed in this last commit?
By the way, I noticed lots of files that are now 755 instead of 644.

Coverage increased (+0.01%) to 90.4% when pulling 16f75753c64b9158da664c0dd6a10695ce9ad213 on lcamacho:bug-127938 into e45b7f934abaefabd017a368a378242f98ebb45a on mozilla:master.

List of changes:

Fix event font color in Day and Week views
Removed a couple of clean functions from form.py
'_ts' suffix changed to something more appropiated in test_view.py
Fix event time, now it shows the event in local time
Fix permissions of FullCalendar assets

Just one thing now.

We lost the formatting of the times. At least for me and my view.
I see "9:30 MoFo Staff Call November". What's lacking is the am/pm notation.
Now, I can guess that because it's not "09:30 MoFo Staff Call November" I can guess it means that it's not a 24h clock. However, it's not clear.

I'm using a 24 hours clock format like on production.

I'm using a 24 hours clock format like on production.

 It's bad on prod too. Can we do something about that?

I can modify to show am/pm on month view and 12 hours clock without am/pm on week and day view, are you ok with this formats or it should show am/pm in every view?

I think it would be best with am/pm everywhere.
Ideally, in some far future, we can use that detect.js library and make it "1:30 pm" for Americans and "13:30" for others.

Coverage increased (+0.01%) to 90.4% when pulling 26ed1e5696e15c3f43e59863e1b374dec5b29f15 on lcamacho:bug-127938 into 240c437920fa812e4b0a829c7b2731a1983487e6 on mozilla:master.

r+
Well done!
That's much easier to read. I'll merge it once travis goes green.

Coverage increased (+0.01%) to 90.4% when pulling 701d58626aba7e2a29d6ddfbde652a9f31a12546 on lcamacho:bug-127938 into 240c437920fa812e4b0a829c7b2731a1983487e6 on mozilla:master.


Why the dewarping_loss uses two channel?

@ZZZHANG-jx
Hi, Jiaxin! Thanks for your great work.
I have a question why the dewarping_loss uses two channel?
dewarping_loss = l1(pred_im[:,:2,:,:], gt_im[:,:2,:,:])
Thank you!

Thank you for your question. Unlike the other four tasks that output RGB images, the direct output of the dewarping task is a 2-channel flow field. These two channels indicate the displacement in the x and y directions, respectively. Based on this flow field, the dewarped RGB image can be sampled, which is the usual practice in document dewarping tasks. Therefore, we only need to use two of the three output channels and can ignore the remaining channel.


Snyk

Hi, thanks for the PR. Please explain what it does and what problem is solved that is not covered by the current version.


Piccap autstart does not work

Hi, first of all: thanks everyone who contributed to this project. :-)
I have an issues with autostart on my TV:
    "board_type": "O18_DVB",
    "hardware_id": "HE_DTV_W18O_AFABABAA",
    "product_id": "OLED65C8LLA",
    "core_os_release": "4.4.0-9",
    "webos_manufacturing_version": "05.40.09",

using piccap version 2.3
When I turn on the TV, background lit up, but stays on white color. I need to go into Piccap and stop & start service to have connection to the hyperion server. then it works fine. Any Idea why it does not work automatically?

I have exactly the same problem

I assume both of you are using libdile_vt?
Have you used some older piccap version? If so, then please try running rm /var/lib/webosbrew/init.d/piccapautostart, then go to piccap settings, disable autostart, save, reenable autostart, save... I think we still have a minor bug in autostart handling which fails when some old piccap version (before 0.2.x) was installed before and left some files on the filesystem.

I assume both of you are using libdile_vt?
Have you used some older piccap version? If so, then please try running rm /var/lib/webosbrew/init.d/piccapautostart, then go to piccap settings, disable autostart, save, reenable autostart, save... I think we still have a minor bug in autostart handling which fails when some old piccap version (before 0.2.x) was installed before and left some files on the filesystem.

yes i had 2 older versions of picap on my lg. I will test this and come backk

did tthat, but for me it seems not to fix the issue.
Autostart seems to working, but not connecting to the hyperion server. via manual start it works.
how can i deinstall completely and install again?

told them multiple times on the discord that this happens also on cx devices.
you disable the Transmit GUI option -> you get Video out but no gui on auto boot
you enable Transmit GUI option -> you get No video but the GUI on auto boot
manually stopping and starting the service in the app straightens things out

Do you guys have some more information for us?
Which settings are you using?

I investigated a workaround to autostart the capture service without rebooting the tv or disabling the quick start. I used home assistant to turn off and on the capture service once the tv turned on. If someone is interested in it .. Don't know if this is the right place for this.

I investigated a workaround to autostart the capture service without rebooting the tv or disabling the quick start. I used home assistant to turn off and on the capture service once the tv turned on. If someone is interested in it .. Don't know if this is the right place for this.

yes i am interested in can you post it ?

I investigated a workaround to autostart the capture service without rebooting the tv or disabling the quick start. I used home assistant to turn off and on the capture service once the tv turned on. If someone is interested in it .. Don't know if this is the right place for this.

yes i am interested in can you post it ?

Enable SSH access on your tv.
Place your public key on your tv for connecting via ssh without password
Create the following script in /home/root on your tv:

root@LGwebOSTV:~# cat my-piccap-service.sh
#!/bin/bash

set -e
luna-send -d -n 1 "luna://org.webosbrew.piccap.service/$1" "{ \"method\": \"$1\"}" &

Create a shell command in home assistant (configuration.yaml). You have to change the path to your private key (im using docker and had to mount my key in the home assistant container):

shell_command:
  piccap_capture_service: 'ssh -i "/config/.ssh/id_rsa" -o "StrictHostKeyChecking=no" root@<your-tv-ip> "bash my-piccap-service.sh {{ command }}"'

Create an automation which triggers when your tv turns on:

- service: shell_command.piccap_capture_service
  data:
    command: stop
- delay:
    hours: 0
    minutes: 0
    seconds: 5
    milliseconds: 0
- service: shell_command.piccap_capture_service
  data:
    command: start

I don't know if its necessary to stop the service first. But it works. And I dont know why the luna command has to be launched as a background service (&). But without the command gets not executed.

Permissions 0744 for '/ssl/lg_homeassistant_private' are too open.
That's a good summary.
You first need to run:
chmod 0600 /ssl/lg_homeassistant_private

Hello,
i have also the problem that the autostart doesn't work. If i open piccap manuel, than it works.
Setting is new 2020+ tv / video capture / graphic capture.
I think it worked until there was a update from the tv. but root is still ok says webosbrew.
Can I just root the TV again to be sure that this is not the problem or will this cause other problems?
Thank you in advance!

Hi
Is there any other solution to this issue? Because the app doesn't start for me either. It only works if you send the TV to the reboot from the app itself.
Larsen

I investigated a workaround to autostart the capture service without rebooting the tv or disabling the quick start. I used home assistant to turn off and on the capture service once the tv turned on. If someone is interested in it .. Don't know if this is the right place for this.

yes i am interested in can you post it ?

Enable SSH access on your tv.
Place your public key on your tv for connecting via ssh without password
Create the following script in /home/root on your tv:

root@LGwebOSTV:~# cat my-piccap-service.sh
#!/bin/bash

set -e
luna-send -d -n 1 "luna://org.webosbrew.piccap.service/$1" "{ \"method\": \"$1\"}" &

Create a shell command in home assistant (configuration.yaml). You have to change the path to your private key (im using docker and had to mount my key in the home assistant container):

shell_command:
  piccap_capture_service: 'ssh -i "/config/.ssh/id_rsa" -o "StrictHostKeyChecking=no" root@<your-tv-ip> "bash my-piccap-service.sh {{ command }}"'

Create an automation which triggers when your tv turns on:

- service: shell_command.piccap_capture_service
  data:
    command: stop
- delay:
    hours: 0
    minutes: 0
    seconds: 5
    milliseconds: 0
- service: shell_command.piccap_capture_service
  data:
    command: start

I don't know if its necessary to stop the service first. But it works. And I dont know why the luna command has to be launched as a background service (&). But without the command gets not executed.

Welcome.  This looks like exactly what I need.  but lack of knowledge to setup ssh connection(

Autostart is already fixed in newer development versions. If you aren't able to connect to your TV using SSH, just wait a bit more until release.

@TBSniller :

how to we start & stop the service.
As the autorun does not work, therefore I need to STOP and START manually, sometimes it  happens that the service does not start anymore, then I need to reboot completely, any chance via SSH? What the exact command to start / stop the service
Are there any precompiled developer versions avaiable? I dont have any enviroment setup yet, therefore I am asking. When is the new release planned approx?

2. sometimes it  happens that the service does not start anymore, then I need to reboot completely

try killall -9 hyperion-webos via ssh and start should work

sometimes it  happens that the service does not start anymore, then I need to reboot completely

try killall -9 hyperion-webos via ssh and start should work

thank you, and restarting?

luna-send -n 1 'luna://org.webosbrew.piccap.service/start' '{}'

in my case, I created a script on the TV, and call it via SSH twice if the autostart does not work
pc_status=`luna-send -n 1 'luna://org.webosbrew.piccap.service/start' '{}'`
if echo $pc_status | grep -q '{"returnValue":false}'; then
	killall -9 hyperion-webos
	luna-send -n 1 'luna://org.webosbrew.piccap.service/stop' '{}'
fi

thanks  @mksy, I will give it a try. Looks good.

With latest release we reworked almost anything. Please check it out, it should work now for webOS 3.4 and above.

Thanks for the new version. But its now even worse. Need ro reboot to get it working. Before manuö restarting worked at least with 0.23

@denimjeans Can you please create an issue here https://github.com/webosbrew/hyperion-webos/issues ?
We would also need some logs. You can get them in Log Tab in PicCap -> First set it up using buttons in log tab -> Then produce your problem -> then get last log -> Put the picture in your issue.
You can also get logs directly from CLI by manually running it: https://github.com/webosbrew/hyperion-webos#running
Make sure to kill proccess kill -9 $(pidof hyperion-webos) and PicCap before

@TBSniller thank you. issues opened here ->
#https://github.com/webosbrew/hyperion-webos/issues/95
for everyone who has the same issue. I know that there are at least 2 ppl with the same issue

@TBSniller one thing more: could you please let me know on how to downgrade from 0.41 to 0.23. I have managed to copy the 0.23 ipk file to /tmp. But not sure on how to install now? ipkg and opkg not working here. thanks

@TBSniller one thing more: could you please let me know on how to downgrade from 0.41 to 0.23. I have managed to copy the 0.23 ipk file to /tmp. But not sure on how to install now? ipkg and opkg not working here. thanks

@denimjeans sure, you can use Install command mentioned here: https://github.com/TBSniller/piccap#manual-way
Just replace it with your filename. When its finished you can exit it with CTRL+C

luna://org.webosbrew.piccap.service/$1

hey this still works for you?
I would like to turn on and off piccap from ha but i am missing something copying your way. I put a public key in my tv, and placed the private one in a file in ha but the shell command always gives de error 255.

command: stop

I'm getting an error when i call service of the shell command
Identity file /config/.ssh/id_rsa not accessible: No such file or
  directory.


Heroku No Route Matches [GET]

I know I'm missing something dumb, but could use any help. I have a Rails 3.0 app, and am migrating from ember-rails to ember-cli-rails. It works great in development, but in production, I get four 404 errors for /assets/emberappname/vendor.js, /assets/emberappname/emberappname.js, and the same with s/js/css.
I verified that I can in fact deploy https://github.com/seanpdoyle/ember-cli-rails-heroku-example, but trying to copy those parameters over doesn't seem to be working
Running heroku run rake assets:precompile produces a lot of non-error output from bower, etc, and ends with a line "Building.Building..Building...Built project successfully. Stored in "/app/tmp/ember-cli-b7e6135c-35ae-42f0-bd1b-ff1603bd867d/apps/emberappname".
As far as I can tell, it's being built correctly, but the link is just wrong or not fingerprinted or some such. Any help is much, much appreciated - will be happy to submit a PR for the docs if it is something that can be improved.

Does it actually write files on assets prcompile step? Are you getting something like this:
remote:        I, [2015-03-27T13:33:59.722509 #1378]  INFO -- : Writing /tmp/build_69e5b278cdae72bf475a99dacf411fe3/public/assets/application-319bcc5a7d959d859c5a58603adffb59.css
remote:        I, [2015-03-27T13:34:04.388805 #1378]  INFO -- : Writing /tmp/build_69e5b278cdae72bf475a99dacf411fe3/public/assets/frontend/frontend-d901a7d8003ea1f344824d2b917772c8.css
remote:        I, [2015-03-27T13:34:09.093172 #1378]  INFO -- : Writing /tmp/build_69e5b278cdae72bf475a99dacf411fe3/public/assets/frontend/frontend-a2dd36295bdc6bd043bd0c04fbabc60b.js
remote:        I, [2015-03-27T13:34:09.786161 #1378]  INFO -- : Writing /tmp/build_69e5b278cdae72bf475a99dacf411fe3/public/assets/frontend/vendor-c06dd6a542ea15147e6794e03643f87c.css
remote:        I, [2015-03-27T13:37:36.560430 #1378]  INFO -- : Writing /tmp/build_69e5b278cdae72bf475a99dacf411fe3/public/assets/frontend/vendor-3c0e2aeacd2848bb13089e31eb72304f.js

No, the closest to that output that I am getting is this line:
Building.Building..Building...BuildingBuilt project successfully. Stored in "/app/tmp/ember-cli-f0320e4f-1eb2-4bad-af5c-d25f881c52b0/apps/emberappname".

There's a lot of output from npm and bower, so it is doing something - just not the right thing, it appears.
Rails Root package.json:
{
  "dependencies": {
    "bower": "*"
  }
}

{
  "name": "emberappname",
  "version": "0.0.0",
  "private": true,
  "directories": {
    "doc": "doc",
    "test": "tests"
  },
  "scripts": {
    "start": "ember server",
    "build": "ember build",
    "test": "ember test",
    "postinstall": "node_modules/.bin/bower install"
  },
  "repository": "https://github.com/stefanpenner/ember-cli",
  "engines": {
    "node": ">= 0.10.0"
  },
  "author": "",
  "license": "MIT",
  "devDependencies": {
    "bower": "*",
    "rimraf": "2.3.0",
    "body-parser": "^1.2.0",
    "broccoli-asset-rev": "0.3.1",
    "broccoli-ember-hbs-template-compiler": "^1.6.1",
    "ember-cli": "0.1.5",
    "ember-cli-content-security-policy": "0.3.0",
    "ember-cli-ic-ajax": "0.1.1",
    "ember-cli-inject-live-reload": "^1.3.0",
    "ember-cli-qunit": "0.1.0",
    "ember-cli-rails-addon": "0.0.11",
    "ember-data": "1.0.0-beta.10",
    "ember-export-application-global": "^1.0.0",
    "express": "^4.8.5",
    "glob": "^4.0.5"
  }
}

Ember app bower.json:
{
  "name": "emberappname",
  "dependencies": {
    "handlebars": "~1.3.0",
    "jquery": "^1.11.1",
    "ember": "1.10.0",
    "ember-data": "1.0.0-beta.16",
    "ember-resolver": "~0.1.7",
    "loader.js": "stefanpenner/loader.js#1.0.1",
    "ember-cli-shims": "stefanpenner/ember-cli-shims#0.0.3",
    "ember-cli-test-loader": "rwjblue/ember-cli-test-loader#0.0.4",
    "ember-load-initializers": "stefanpenner/ember-load-initializers#0.0.2",
    "ember-qunit": "0.1.8",
    "ember-qunit-notifications": "0.0.4",
    "qunit": "~1.15.0"
  }
}

/config/initializers/ember.rb:
EmberCLI.configure do |c|
  c.app :emberappname, path: "emberappname"
end

I've tried a lot of permutations of options, but unfortunately have not been very scientific.

Just realized, this is Rails 3.2.17; is the gem Rails 4 only?

@jamescgibson no, it supposed to support 3.1+

any way you could submit a test repo recreating the issue?

Will do my best. Feel free to close if you prefer close-reopen with test repo. Thanks for your help!

Client decided to stick with ember-rails for the time being; will re-open much later if possible. Thanks @rwz for all the help!


Feat: Suggest Java 1.8 users upgrade JVM via logging

Is your feature request related to a problem? Please describe.
It is difficult for the project to get information on usage in different runtimes. It would be useful to start logging an INFO suggesting users upgrade JVM if they are on a very old one and link to some documentation on the powertools site. This way we 1/ can encourage best practices - that is, running a modern Java stack and 2/ start to collect feedback about why users might not want to do that.
Describe the solution you'd like
Log an INFO/WARN suggesting upgrading and linking to our documentation
Describe alternatives you've considered

Additional context

Isn't it too intrusive? abusive?
Isn't WARN too much?
We should log this only during cold start.
How do you see "start to collect feedback about why users might not want to do that." ? By counting the complaints on Twitter and issues raised on GitHub to remove this ?

@jeromevdl
I think in the INFO message we could link to the GitHub issue where we say we'd like to deprecate Java 8, and ask users to leave a comment there. WDYT? 🤔

Is there a way we could print such message at build time, rather than runtime?
My concern with the runtime is the additional logging and the cost/noise that will come as a result.


login page text is not customized

anyone can me tell me how i can customize login page text in [OpenSearch]
@opensearch-ci-bot

@kavilla can you give me the path of security dashboards plugin.

Thanks for opening this, we will have one look at it.

I will take a look at this and get back to you @amanthakur.

Hi @amanthakur,
I was able to gather some resources that may be helpful:
https://opensearch.org/docs/latest/dashboards/branding/ -- Dashboards branding tutorial
https://github.com/opensearch-project/OpenSearch-Dashboards -- Dashboards github repo
Right now, I do not believe it is possible to directly modify the login page style setting with the opensearch_dashboards.yml file; however it is still an option to modify the HTML/CSS options in order to place your company's logo above the login.
If you would like further support for modifying the login page, please fill out a feature request by going to Issues>Feature Request and filling out the form. Likewise any contributions to this point would be welcome.

Hi @amanthakur,
I think we do support the customization of basic auth's login button style and branding image for current version. You can modifying these by adding the feature flags of:
opensearch_security.ui.basicauth.login.buttonstyle and opensearch_security.ui.basicauth.login.brandimage
In addition, there is a feature PR https://github.com/opensearch-project/security-dashboards-plugin/pull/1110. We are currently working on this right now and it includes the support of those style customization for external authentication methods, such as OIDC and SAML.

@RyanL1997 @scrawfor99  I have changed it.  you can find the text running this command  ( grep -rnw / -e 'OpenSearch Dashboards' )  and there is file that name is index.js where you can change it .  :)


List own comments

Test
KP » Settings » Account » Comments
Issue
Closes #5146 Show all own comments
Known issues
The total number of comments that Commentron reports vs. what it returns, can be different.  Checking with Mark/Tom.

cd2f3d2 to bac26bc
Commentron now requires authorization for this action.

Don't love the placement of where this is avaialble, but probably the best bet for now.
Checking with Beamer on a sorting issue, otherwise it looks good!

bac26bc to 05f2aed

Add i18n strings
Rebased

There's a new sorting method (5 I think - check DMs) to remove pins.
Can we make the comment clickable or linkable?

05f2aed to 48247cb

Mobile: removed indent; reduced claim thumbnail size.
Remove automatic pinning when fetching.
Clicking on the claim (left side) now links to the comment directly.  The comment (right side) remains unclickable due to:

I find hard to copy/paste parts of the comment if the entire section is clickable.
But I did try and couldn't get NavLink to work with Comment -- it causes lots of renders and blinking.  Maybe re-visit this next time, but for now use the left-side to click?

Also considered adding a "View comment" button at the lower-right of each row, but that made the page busier than it is already.


Remove not using input parameter at connection.py

Pull Request check-list
Please make sure to review and check all of these items:

[ ] Does $ tox pass with this change (including linting)?
[ ] Does travis tests pass with this change (enable it first in your forked repo and wait for the travis build to finish)?
[ ] Is the new or changed code fully tested?
[ ] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

NOTE: these things are not required to open a PR and can be done
afterwards / while the PR is open.
Description of change

remove 'command_name' at get_connection@ConnectionPool
-> get_connection(self, command_name, *keys, **kwargs) => get_connection(self, *keys, **kwargs)
remove 'command_name' at get_connection@BlockingConnectionPool
-> get_connection(self, command_name, *keys, **kwargs) => get_connection(self, *keys, **kwargs)

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1251   +/-   ##
=======================================
  Coverage   92.56%   92.56%
=======================================
  Files          20       20
  Lines        6143     6143
=======================================
  Hits         5686     5686
  Misses        457      457

Impacted Files
Coverage Δ

redis/connection.py
82.4% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

This is a hook so that user subclasses can know what command the connection will execute. We're going to leave it in for the time being.


Question about discrite sampling in RT-DETRv2

Hello,
I love your work. I have a question regarding the discrete sampling.
You state on the paper:
we propose an optional discrete_sample operator to replace the grid_sample, thus removing the deployment constraints of RT-DETR. Specifically, we perform a rounding operation on the predicted sampling offsets, omitting the time-consuming bilinear interpolation.

Could you please explain how much faster the model is when you use discrete sampling?
Also, what is the difference between your proposed grid_sampling and torch.grid_sample with interpolation = nearest?

Also, why is there a deployment constraint when torch.grid_sample is used?

When you deploy with TensorRT, grid_sample is only supported in versions above 8.5. Therefore, it is necessary for some old devices and scenarios where software cannot be upgraded.
And other proprietary inference engines for some npu may not support grid_sample as well.
So RTDETRv2 is necessary for the community.

For specific speed, this is related to specific equipment and software. We only give the difference of theoretical calculation.

Thank you so much for your well-explained answer.
If, for deployment, you used discrete sampling, why is the model not trained using torch.grid_sample(..., mode='nearest')


[api-minor] Only support browsers/environments that have basic support for Promise natively

Based on https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise#Browser_compatibility and https://caniuse.com/#feat=promises, all even remotely modern browsers already support basic Promise functionality natively.
The only reason for keeping the Promise polyfill (at all) is to be able to support recent additions to the specification, such as e.g. finally and allSettled.
Note that this patch will, on its own, remove support for IE 11/Edge (non-Chromium based) in both the general PDF.js library and the default viewer.

/botio unittest

Looks good; thanks!


dollar-variable-colon-space-after: At least one space

Would it be complex to add the possibility of having at least one space after the colon, for the purpose of aligning values ?
$base-color:     #ff0000;
$primary-color:  #00ff00;
$alt-color:      #0000ff;

Ref: SCSS Lint at_least_one_space

🎉

Hmm, I'm getting
Invalid Option: Invalid option value "at-least-one-space" for rule "scss/dollar-variable-colon-space-after"
after adding the new option to my .stylelintrc.js but I can't figure out why...

@stormwarning make sure that you have the latest version installed.
Works for me with:
"scss/dollar-variable-colon-space-after": "at-least-one-space"

Yeah, looks like the latest is in node_modules 🤷♂ Must be something with this specific project I guess. Thanks for confirming it's working!

Implemented! Thanks a lot 👍
https://github.com/absolunet/stylelint-config-nwayo/blob/master/.stylelintrc.yaml#L501

Nice to hear @jblandry. :) Let us know if you find any bugs related to that rule or any other rules.


drata-agent popping up EPIPE errors on Ubuntu 22.04

I have been asked by my employer to run drata-agent.
I downloaded drata-agent from your website, and was served a .deb file.
$ ls -l ~/Downloads/Drata-Agent-linux.deb
-rw-rw-r-- 1 hanwen hanwen 96036216 Sep 23 10:35 /home/hanwen/Downloads/Drata-Agent-linux.deb

I installed it, and am regularly greeted by an error-popup about EPIPE. (see image)

The drata-agent can't tell me which version it is,
$ drata-agent --version
menubar:  Error: ERR_FAILED (-2) loading 'file:///opt/Drata%20Agent/resources/app.asar/dist/index.html'
    at rejectAndCleanup (node:electron/js2c/browser_init:2:84559)
    at WebContents.stopLoadingListener (node:electron/js2c/browser_init:2:84946)
    at WebContents.emit (node:events:514:28) {
  errno: -2,
  code: 'ERR_FAILED',
  url: 'file:///opt/Drata%20Agent/resources/app.asar/dist/index.html'
}

This is due to the executing context not having permissions to the folders, files, or endpoints required to run the Drata Agent.  The website link is the latest (version 3.6.1), AppImage is also available and may be downloaded directly at https://github.com/drata/agent-releases/releases/latest.
Typically, the user will need access to the following at default installs:
(local storage) RW: ~/.config/drata-agent/app-data.json
(local logging) RW: ~/.config/drata-agent/logs/main.log
(library) RWX: /opt/Drata Agent/resources/lib/linux/bin/osqueryi
(auto updates) RW: /opt/Drata Agent
Endpoints
https://agent.drata.com/ or https://agent.eu.drata.com/
https://*.datadoghq.com/
You may also contact our Support Team to get assistance with installation where we assist more directly with your specific setup and private machine/network details. We do have a ticket for the next release to make this error message more friendly and explicit.
Note: There is not a --version parameter nor CLI for the drata-agent application.

(library) RWX: /opt/Drata Agent/resources/lib/linux/bin/osqueryi

hanwen@hanwen-flow:~/vc/engflow$ ls -l  /opt/Drata\ Agent/resources/lib/linux/bin/osqueryi
-rwxr-xr-x 1 root root 86097376 Mär 26 17:56 '/opt/Drata Agent/resources/lib/linux/bin/osqueryi'

so should I make this file mode 0777 then?

(auto updates) RW: /opt/Drata Agent

this is owned by root as well; please advise how to change the install.

Note: There is not a --version parameter nor CLI for the drata-agent application.

I gathered as much. I was trying to suggest you might want to add it.

@hanwen-flow I would get in touch with our support team to resolve detailed installation issues. Ownership of certain files will vary based on the install used, for deb on Ubuntu you have to elevate for installation.
Please follow the permissions outlined in my previous comment or use the AppImage version linked there if preferred https://github.com/drata/drata-agent/issues/31#issuecomment-2368643139.


new feature 3  dev

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


re-integrate DataClay with CIMI, through mvn

related with https://github.com/mF2C/cimi/issues/42

Done.


fix: unroll Oneofs before other fields

When marshaling messages with oneofs, the marshal feature was generating bytes in a different order than proto.Marshal. This is due to proto.Marshal dealing with oneofs first, before any of the other fields in the message.
This PR changes the marshal feature to unroll all Oneofs before the rest of the message fields

Thanks!


Analyzer should issue a "mixed mode unsoundness" warning (or hint) in certain scenarios

Due to https://github.com/dart-lang/language/issues/1143, we are changing the front end so that inserts a synthetic throw if certain "impossible" situations arise due to the inherent unsoundness of mixed mode.  For most of these scenarios, there should be a corresponding analyzer warning (or hint) to alert users to the situation and point them to documentation of what to do about it.
(Note that some or all of these scenarios may already be covered by analyzer hints, but I believe that in a lot of cases the hint is too general, e.g. a "dead code" hint.  We want to have one or more hints that are specific to the following scenarios so that we can link to the appropriate documentation.)
The scenarios the analyzer should warn (or hint) about are as follows:

Flow analysis believes it is impossible for an is test to produce a false result (or an is! test to produce a true result).  This can be identified by FlowAnalysis.isExpression_end returning false.

Note that the user can eliminate the warning by removing the check, or suppress it by casting the type of the operand of is to dynamic.

Flow analysis believes it is impossible for an == test to produce a true result (or a != test to produce a false result).  This can be identified by FlowAnaylsis.equalityOp_end returning false.  This will only occur when one side of the comparison has type Null, so in practice this probably means one side of the comparison is the literal null.

Note that the user can eliminate the warning by removing the check, or suppress it by casting the type of the non-null operand to dynamic.

Flow analysis believes that it is impossible for the right hand side of a ?? or ??= operation to be reached.  This can be identified by FlowAnalysis.ifNullExpression_rightBegin returning false.

Note that for a ?? expression, that the user can eliminate the warning by replacing the expression with its left hand side, or suppress it by casting the left hand side to a nullable type.
For a ??= expression, the user can eliminate the warning by removing the assignment, or suppress it by replacing it with its desugared equivalent and using a cast to dynamic (e.g. replacing x ??= y; with if ((x as dynamic) == null) x = y;).

Flow analysis believes that it is impossible for the target of a ?. or ?.. to be null.  This can be identified by FlowAnalysis.nullAwareAccess_rightBegin returning false.

Note that the user can eliminate the warning by removing the ?, or suppress it by casting the target to a nullable type.

There is one other scenario in which the front end inserts a synthetic throw: after evaluation of an expression having type Never.  However, I don't think this should have a special analyzer warning beyond the usual "dead code" hint, because it crops up too frequently in legitimate code (e.g. calling fail in a test).  Also I believe it's unlikely to be a problem in practice.
I plan to follow up with a language test that expects an analyzer warning in each of these cases.  If we decide that we want a hint instead, we'll have to work with @munificent to adapt the language test infrastructure to be able to test for hints.

@leafpetersen Please triage

Move this work out of Beta EPIC

Mixed mode is no longer supported.


We just had to simply whitelist accordion since upstream semantic doesn't support any settable properties on it (and there isn't a good way to detect it.)


Merge 'GO:0015557 arginine targeting transmembrane transporter activity' into 'GO:0009977 proton motive force dependent protein transmembrane transporter activity'

'GO:0015557 arginine targeting transmembrane transporter activity' represents a transport pathway,  GO:0043953 protein transport by the Tat complex.
Proteins involved in GO:0043953 protein transport by the Tat complex are annotated to 'GO:0009977 proton motive force dependent protein transmembrane transporter activity', so I will (1) merge GO:0015557 into that term.
and (2) add synonym 'Twin-arginine translocation pathway' to GO:0043953 protein transport by the Tat complex.
FYI there are annotations by
TAIR @tberardini
EcoCyc @keseler
UniProt @sylvainpoux
and PseudoCAP.
Thanks, Pascale

Do the annotations need reviewing?

Seems an odd merge if it is describing an arginine transporter?

It's not an arginine transporter - it recognizes twin arginines as signals (note that term label was GO:0015557 arginine targeting transmembrane transporter activity'
@tberardini There were no annotations to GO:0015557 (sorry about the confusion; my query was rather 'does this make sense since you have annotated the same function differently - but I should have formulated it !) .
Thanks, Pascale


'View'ing the results of a search cannot be modelled well in the schema

In situations where a user is viewing the results of a previous search, e.g. when the search results have been saved, the options available to model this with a View schema action are not ideal.  Currently under View you have 1..many of the MultiObjectComplexType objects, one of which is SearchResult.  SearchResult only really has Relevancy and Data, with no means of recording details of the items in that result set.
There is also ambiguity over whether SearchResult means a single items in a collection of results from a search, or the whole collection of items found in a single search.

Can't one just use View of a  Criteria and make use of the CriteraComplexType to provide details of the original search and result set details?

See #11

Fixed in >v3.3.1


[VISUAL BUG] Connect buttons is hidden being just accessible using tab

The connect buttons is hidden, I only found it using the tab.
Seems a visual issue only :)

Oh wow, thanks for finding this! I'll work on a fix. In the meantime, just stretch the window vertically and it will show you the save button.

This is fixed and will be in the next release!


🌟Feature request: Prevent Popover close based on condition (e.g. tainted input fields)

Is your feature request related to a problem? Please describe.
I'm working on a builder-like application, and some of my profile features are edited using the popovers with some inputs inside. These popovers either need to be saved or canceled.
Describe the solution you'd like
Ideally, I would like to prevent close based on some condition, like unsaved changes or tainted input fields.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
I've tried to reopen the popop using the $open store and a click outside handler, but as you can expect that does not lead to the desired outcome.
Additional context
n.a.

Related: #70

It is also somewhat related to #80 whereby I would like to be able to keep the parent popover open while a child popover is open..


fix: remove common protos

Common protos do not belong here, they are a part of google-gax.

ok woah this was a pre-gax thing


STT with Teams Unmixed Audio Buffer

So I'm trying to create a STT solution that's part of a Teams Bot. The Bot takes part of the meeting and listens to participants' audio. The audio is set to Unmixed = true so that each speaker gets their own channel. The solution is in C#.
I receive the separate audio buffers in real-time, and I send them to a Cognitive Services class to recognize the speech. Every participant gets their own Cognitive Services recognizer class instance.
My problem is that the messages get to the recognizer, and you can see the recognizing event fire consistently. However, the final recognized event only fires sporadically, even with long periods of silence. I'm not sure where the issue is. Any help is greatly appreciated.
Here's where the audio is sent to langServices:
 if (audioFrame.UnmixedAudioBuffers != null)
 {
     var tasks = new List<Task>();
     foreach (var buffer in audioFrame.UnmixedAudioBuffers)
     {
         var length = buffer.Length;
         var data = new byte[length];
         Marshal.Copy(buffer.Data, data, 0, (int)length);

         var participant = CallHandler.GetParticipantFromMSI(this.callHandler.Call, buffer.ActiveSpeakerId);
         var identity = CallHandler.TryGetParticipantIdentity(participant);
         if (identity != null)
         {

             buffers.Add(identity.Id, (new AudioBuffer(data, audioFormat), audioFrameTimestamp));

             //send to Cognitive Services to transcript
             if (!langServices.ContainsKey(identity))
             {
                 langServices.Add(identity, new CognitiveServicesService(identity, this.callHandler.botConfiguration, logger, this.callHandler.Call.Id));
             }

             tasks.Add(langServices[identity].AppendAudioBuffer(data));

             //try a new instance every time!
             //var c = new CognitiveServicesService(identity, this.callHandler.botConfiguration, logger, this.callHandler.Call.Id);
             //tasks.Add(c.AppendAudioBuffer(data));
         }
         else
         {
             this.logger.Warn($"Couldn't find participant for ActiveSpeakerId: {buffer.ActiveSpeakerId}");
         }
     }
     await Task.WhenAll(tasks);
 }

Here are the relevant snippets from CognitiveServices class:
public CognitiveServicesService(Identity identity, BotConfiguration settings, IGraphLogger logger, string callId)
{
    _logger = logger;
    _callId = callId;
    _identity = identity;

    _speechConfig = SpeechConfig.FromSubscription(settings.SpeechConfigKey, settings.SpeechConfigRegion);
    _speechConfig.SpeechSynthesisLanguage = settings.BotLanguage;
    _speechConfig.SpeechRecognitionLanguage = settings.BotLanguage;

    //_speechConfig.SetProperty(PropertyId.Speech_SegmentationSilenceTimeoutMs, "1000");
    //_speechConfig.SetProperty(PropertyId.SpeechServiceConnection_InitialSilenceTimeoutMs, "1000");

    var audioConfig = AudioConfig.FromStreamOutput(_audioOutputStream);
}

public async Task AppendAudioBuffer(byte[] audioBuffer)
{
    //RealtimeTranscriptionHelper.TranscribeAsync(audioBuffer, _speechConfig, _logger);
    if (!_isRunning)
    {
        Start();
        await ProcessSpeech();
    }

    try
    {
        _audioInputStream.Write(audioBuffer);
    }
    catch (Exception e)
    {
        _logger.Log(System.Diagnostics.TraceLevel.Info, e, "Exception happend writing to input stream");
    }
}

 private async Task ProcessSpeech()
 {
     try
     {

         var stopRecognition = new TaskCompletionSource<int>();

         using (var audioInput = AudioConfig.FromStreamInput(_audioInputStream))
         {
             if (_recognizer == null)
             {
                 _logger.Log(System.Diagnostics.TraceLevel.Info, "init recognizer");
                 _recognizer = new SpeechRecognizer(_speechConfig, audioInput);

             }
         }

         _recognizer.SpeechStartDetected += async (s, e) =>
         {
             Console.WriteLine($"Speech Start Detected. Offest: {e.Offset}");
         };

         _recognizer.SpeechEndDetected += async (s, e) =>
         {
             Console.WriteLine($"Speech End Detected. Offest: {e.Offset}");
         };

         _recognizer.Recognizing += (s, e) =>
         {
             string msg = $"RECOGNIZING: Text={e.Result.Text}";
             _logger.Log(System.Diagnostics.TraceLevel.Info, msg);
             Console.WriteLine(msg);
         };

         _recognizer.Recognized += async (s, e) =>
         {
             if (e.Result.Reason == ResultReason.RecognizedSpeech)
             {
                 if (string.IsNullOrEmpty(e.Result.Text))
                     return;

                 // We recognized the speech
                 var msg = $"'timestamp': '{DateTime.Now}', 'speaker': '{_identity.DisplayName}', 'text': '{ e.Result.Text}'";

                 Console.WriteLine(msg);
                 _logger.Log(System.Diagnostics.TraceLevel.Info, $"***Recognized***: {msg}");

             }
             else if (e.Result.Reason == ResultReason.NoMatch)
             {
                 _logger.Log(System.Diagnostics.TraceLevel.Info, $"NOMATCH: Speech could not be recognized.");
             }
         };

         _recognizer.Canceled += (s, e) =>
         {
             _logger.Log(System.Diagnostics.TraceLevel.Info, $"CANCELED: Reason={e.Reason}");

             if (e.Reason == CancellationReason.Error)
             {
                 _logger.Log(System.Diagnostics.TraceLevel.Info, $"CANCELED: ErrorCode={e.ErrorCode}");
                 _logger.Log(System.Diagnostics.TraceLevel.Info, $"CANCELED: ErrorDetails={e.ErrorDetails}");
                 _logger.Log(System.Diagnostics.TraceLevel.Info, $"CANCELED: Did you update the subscription info?");
             }

             stopRecognition.TrySetResult(0);
         };

         _recognizer.SessionStarted += async (s, e) =>
         {
             _logger.Log(System.Diagnostics.TraceLevel.Info, "\nSession started event.");
         };

         _recognizer.SessionStopped += (s, e) =>
         {
             _logger.Log(System.Diagnostics.TraceLevel.Info, "\nSession stopped event.");
             _logger.Log(System.Diagnostics.TraceLevel.Info, "\nStop recognition.");
             stopRecognition.TrySetResult(0);
         };

         // Starts continuous recognition. Uses StopContinuousRecognitionAsync() to stop recognition.
         await _recognizer.StartContinuousRecognitionAsync().ConfigureAwait(false);

         // Waits for completion.
         // Use Task.WaitAny to keep the task rooted.
         Task.WaitAny(new[] { stopRecognition.Task });

         // Stops recognition.
         await _recognizer.StopContinuousRecognitionAsync().ConfigureAwait(false);
     }
     catch (ObjectDisposedException ex)
     {
         _logger.Log(System.Diagnostics.TraceLevel.Error, ex, "The queue processing task object has been disposed.");
     }
     catch (Exception ex)
     {
         // Catch all other exceptions and log
         _logger.Log(System.Diagnostics.TraceLevel.Error, ex, "Caught Exception");
     }
 }

Thanks for using the Cognitive Speech services SDK. In order to be able to debug this further could you please:

Capture the SDK logs for a run that exhibits the problematic missing recognition events. You can find instructions on how to enable this here: https://learn.microsoft.com/azure/ai-services/speech-service/how-to-use-logging
If possible, include a sample of the audio that you are sending the speech recognizer? This will help in diagnosing and reproducing the issue on our end
Capture the session ID for the problematic run so we can correlate with the speech recognition service logs. This can be found in the SDK logs, or as part of SessionStarted, and SessionStopped event arguments

On a side note, I noticed in your code sample above that you are using a using block with the AudioConfig which is causing it to be disposed early. Generally speaking, you the lifetime of the AudioConfig should match that of the SpeechRecognizer.

Closing, requested logs from customer, no response.


Migrate to Rust 2021

Our MSRV (1.59) is higher than 1.56 that Rust 2021 was stabilized.

CI failure is https://github.com/smol-rs/event-listener/issues/84.

CI was fixed in #87


Question about jsx-child-element-spacing logic and
 tags

A couple examples:
  return (

      one line

      another line

  );

  return (

      one line

      another line

  );

Both of these are flagged for ambiguous spacing by jsx-child-element-spacing. But it seems in both cases the intent is clear and both would render as expected in the browser. I wonder if there could be some exception made for
 tags?

I would definitely expect the first example to be acceptable.

@ljharb I've opened PR https://github.com/yannickcr/eslint-plugin-react/pull/2989 which simply removes BR from the set of inline elements, because I don't think there's any case where whitespace around a br tag would result in a different result rendered in the browser. Can you have a look?


Fix an issue that zero balance may not show on chart.

When one of the commodities an account holds goes to zero, it will not
show in balance field in the corresponding entry. A previous attempt
to workaround the issue didn't consider when an account holds more than
two commodities. This change fixed the workaround. Test case has been
updated to cover it.

Current coverage is 56.70% (diff: 100%)

Merging #356 into master will not change coverage

@@             master       #356   diff @@
==========================================
  Files            12         12
  Lines          1037       1037
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            588        588
  Misses          449        449
  Partials          0          0

Powered by Codecov. Last update 145bd5a...4957bc1

Thanks!


Text-adjacent HTML comments cause markup generation to fail

Identified in tc39/ecma262#1589.
 I'll investigate.


Feature/ddw 133 Paper wallets certificate generation: Acceptance tests implementation

This PR introduces acceptance tests for paper wallet certificate generation and wallet restoration from a paper wallet certificate

This PR will be once again reviewed in the scope of https://github.com/input-output-hk/daedalus/pull/779


assertj-guava does not support assertj-core 3.0.0

Which version do you used.
I have tested assert-core 3.0 + assertj-guava 1.3.1 and it worked fine

Ok I found the issue it's a little bit tricky. I was trying to use version ranges for both assertj-core and assertj-guava:
    <dependency>
        <groupId>org.assertj</groupId>
        <artifactId>assertj-core</artifactId>
        <version>[3.0.0,)</version>
        <scope>test</scope>
    </dependency>
    <dependency>
        <groupId>org.assertj</groupId>
        <artifactId>assertj-guava</artifactId>
        <version>[1.3.1,)</version>
        <scope>test</scope>
    </dependency>

Doing this seems to trigger extra checks on version dependencies leading to the following error:
[ERROR] Failed to execute goal on project adhefi-revamp: Could not resolve dependencies for project com.evollis.cacf.utils:adhefi-revamp:jar:0.1.0-SNAPSHOT: Failed to collect dependencies for com.evollis.cacf.utils:adhefi-revamp:jar:0.1.0-SNAPSHOT: Could not resolve version conflict among [org.assertj:assertj-core:jar:[3.0.0,), org.assertj:assertj-guava:jar:[1.3.0,) -> org.assertj:assertj-core:jar:[1.7.1,1.8.0)] -> [Help 1]

Using a fixed version for assertj-core solves the issue

What is weird is that it's showing org.assertj:assertj-guava:jar:[1.3.0,) ... I'm closing the issue as there is a fix.


Point to conversion branch of cql_qdm_patientapi and update tests

Pull requests into cqm-converter require the following. Submitter and reviewer should :white_check_mark: when done. For items that are not-applicable, note it's not-applicable ("N/A") and :white_check_mark:.
Point to the correct branch in cql_qdm_patient_api and update tests.
Submitter:

[x] This pull request describes why these changes were made.
[x] Code diff has been done and been reviewed
[x] Tests have been run locally and pass
[x] ~If applicable, the library version number in cqm-converter.gemspec has been updated~
[x] ~Update cqm-execution test fixtures using update_cqm_execution_fixtures.sh script from server-scripts repo~

Bonnie Reviewer:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Codecov Report

Merging #48 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #48   +/-   ##
=======================================
  Coverage   59.77%   59.77%
=======================================
  Files           7        7
  Lines         532      532
=======================================
  Hits          318      318
  Misses        214      214

Continue to review full report at Codecov.

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


feat: SASL client support

Is your feature request related to a problem? Please describe.
Currently, there is no client support for SASL authentication when connecting to the proxy
Describe the solution you'd like
Add SASL authentication as a configurable strategy. The default authentication mechanism will remain md5.
This will likely involve refactoring the authentication logic, possibly into a separate module.

I'm happy to work on this!

Is your feature request related to a problem? Please describe.
Currently, there is no client support for SASL authentication when connecting to the proxy
Describe the solution you'd like
Add SASL authentication as a configurable strategy. The default authentication mechanism will remain md5.
This will likely involve refactoring the authentication logic, possibly into a separate module.

I've been doing this for the last two weeks. I don't have a lot of time, but I've already made a lot of progress.
I plan to rewrite almost entirely SCRAM authentication as it is now, and add client-server SCRAM authentication.


How to get camera.pkl for SnapshotPeople

Hello! I want to launch rendering for SnapshotPeople, but I get this error:
FileNotFoundError: [Errno 2] No such file or directory: '../../data/peoplesnapshot_arah-format/people_snapshot_public/female-3-casual/camera.pkl'
As I understand it, camera.pkl can be generated with ARAH code. But they don't have a script for SnapshotPeople. Could you please share your preprocessing script?

Hi @david-svitov @mikeqzy
I also do not know how to get these camera.pkl from ARAH code(this is not released).  Could you please share your preprocessing script?

@zhao-yiqun  They meant the usual camera.pkl from the SnapshotPeople dataset. Download the original dataset, it is there.


IstioOperator values for prometheus ignored

Bug description
Setting values
values:
  prometheus:
    enabled: false

Disables prometheus server in 1.5.4 but in 1.6.0 switching between enabled: false/true makes no difference in 1.6.0
[ x ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Expected behavior
Prometheus server should be deployed for enabled: true and not deployed for enabled: false
but this is not the case
Steps to reproduce the bug
Install istio version 1.5.4 and toggle between enabled: true/false
values:
  prometheus:
    enabled: true / false

Upgrade to 1.6.0 do the same and it has no effect.
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
client version: 1.6.0
control plane version: 1.6.0
How was Istio installed?
Standalone operator
Environment where bug was observed (cloud vendor, OS, etc)

Use addonComponents.prometheus.enabled=false from istioctl or
addonComponents:
  prometheus:
    enabled: false

in the operator.
@therealmitchconnors @ostromart I feel we might want to add an "analyzer" so istioctl analyze detects this and the IOP's .Status field is set. This will require making IstioOperator be the same kind of collection as the other .protos.

Seems to work, but the addonComponents.prometheus.enabled=true/false does not seem to work on 1.5.x

Analyzing addons or install commands sounds like a great idea to me, and it fits well with what the ToC has requested for warning about breaking changes on upgrade.  Architecturally, there may be a good deal of work to do in order to get this working, but conceptually I think it's a good plan.

Seems to work, but the addonComponents.prometheus.enabled=true/false does not seem to work on 1.5.x
Hi @ericjohansson89,

I am having similar issue on addonComponents.istiocoredns.enabled: false, for some reason setting it to false does not remove istiocredns.
Do you mind sharing your istioctl command(s)? Appreciate your help on this.


Custom dialogs

Can i use custom modal dialogs for insert, update and delete?
Thanks

For insert and update checkout the following issue https://github.com/tabalinas/jsgrid/issues/350.
For delete, please, see this issue https://github.com/tabalinas/jsgrid/pull/221