TabularDataNode supports replace for AiGen

TabularDataNode now has the AiGen button available and can generate a table from a prompt.

There was also a typing issue that I fixed with Models in fromModelId but this can be dropped.

Added changes for extend in the most recent commit, as well as removing the extraneous logs and imports.

Add column has been Added along with changes to the popover UI


Generator: update SDK /services

Generated from GitHub run<PHONE_NUMBER> (https://github.com/stackitcloud/stackit-api-specifications/actions/runs/7421077856)

Closing in favor of #238


Updates three.js syntax, fixes #84

Nice! Thanks @CaioAlonso
note: circleci fails because there are note tests


WIP: go live with modularization

TODO:

[ ] Get module registry/decoders into DB transactions
[ ] Make tests run

Codecov Report
Base: 64.70% // Head: 66.99% // Increases project coverage by +2.28% :tada:

Coverage data is based on head (0e36903) compared to base (8043cee).
Patch coverage: 83.87% of modified lines in pull request are covered.

:exclamation: Current head 0e36903 differs from pull request most recent head 2e96437. Consider uploading reports for the commit 2e96437 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #965      +/-   ##
==========================================
+ Coverage   64.70%   66.99%   +2.28%
==========================================
  Files         114      113       -1
  Lines       16595    16705     +110
==========================================
+ Hits        10738    11191     +453
+ Misses       5857     5514     -343

Impacted Files
Coverage Δ

client/client-lib/src/transaction.rs
85.30% <0.00%> (-1.24%)
:arrow_down:

client/client-lib/src/utils.rs
11.11% <0.00%> (ø)

fedimint-api/src/encoding/btc.rs
100.00% <ø> (ø)

fedimint-api/src/module/mod.rs
94.73% <ø> (ø)

fedimint-dbdump/src/main.rs
0.42% <0.00%> (-0.01%)
:arrow_down:

fedimintd/src/bin/main.rs
1.12% <0.00%> (-0.10%)
:arrow_down:

modules/fedimint-wallet/src/txoproof.rs
70.96% <0.00%> (-3.69%)
:arrow_down:

fedimint-api/src/core/client.rs
44.11% <28.57%> (+34.44%)
:arrow_up:

client/client-lib/src/wallet/mod.rs
74.08% <65.00%> (+0.47%)
:arrow_up:

fedimint-core/src/transaction.rs
84.15% <67.46%> (-13.91%)
:arrow_down:

... and 53 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


How to get unix timestamp format

I'm completely new to JS, even angular, but I really need to get unix-like timestamp in UTC, but stil show default view in .
If it only could be something:
ng-model='end_date'
{{ end_date.timestamp }}

Can't figure out how to do this.
Can you please show more examples of using angular-bootstrap-datetimepicker?
Thanks.

See https://github.com/dalelotts/angular-bootstrap-datetimepicker#modeltype and also the demo directory.
You might consider using egghead.io or pluralsight to learn javascript and angular.


Feature/stp 843

Description
This feature is about making designer error highlighting consistent with gov uk design standard

Removed onBlur validation in most of the react components
Removed html 5 validations like - required and reportValidity
Added onSubmit validation for components

Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?

[x] Manual testing to see if the error messages are displayed correctly.
[x] New test cases add in existing Test classes to check for validation and error messages

Checklist:

[x] My changes do not introduce any new linting errors
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation and versioning
[ ] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] I have rebased onto master and there are no code conflicts

Thanks Sree! I noticed some linting errors but no need to action now, things like let instead of const, and switch statements with var assignments aren't blocked properly, It's possible eslint isn't flagging errors to you in your IDE. I'm going to open a bug to fix linting on CI.


Automatically split | characters to a new line

Summary of the new feature/enhancement
I should state that I am using VS Code. I was told that for this feature to be possible in VS Code, it has to first be implemented here. The setting I mention down below is a VS Code Powershell setting.
I'd like to the ability to take the following line:
$Ports | ForEach-Object -Process { NETSTAT.EXE -ano | findstr.exe :8080 | ForEach-Object -Process { ($_ -split '\s+')[-1] } | Sort-Object | Get-Unique | ForEach-Object -Process { taskkill.exe /F /PID $_ } }

and convert it to:
$Ports |
    ForEach-Object -Process {
        NETSTAT.EXE -ano |
            findstr.exe :8080 |
            ForEach-Object -Process {
                ($_ -split '\s+')[-1]
            } | Sort-Object |
            Get-Unique |
            ForEach-Object -Process {
                taskkill.exe /F /PID $_
            }
    }

I think this would go a long way in approving readability. At the very least, with the
"powershell.codeFormatting.pipelineIndentationStyle": "IncreaseIndentationForFirstPipeline"

setting enabled, when I manually add the new line after each |, { , and }, it will format the indentation correctly which is nice.

For some preliminary triage info, please refer to the original issue #2194
Thinking about this again, how do you think the rule should treat simple one line expressions like foo | bar? It would be annoying if those were to be split across 2 lines. Something like requiring the number of pipes to be greater than 2? We have a new AvoidLongLines rule by @thomasrayner , maybe we could incorporate that into this rule to split a pipeline expression into more than 1 line to make it fit into the line length limit as an alternative to always splitting every expression across 1 line?

For some preliminary triage info, please refer to the original issue #2194
Thinking about this again, how do you think the rule should treat simple one line expressions like foo | bar? It would be annoying if those were to be split across 2 lines. Something like requiring the number of pipes to be greater than 2? We have a new AvoidLongLines rule by @thomasrayner , maybe we could incorporate that into this rule to split a pipeline expression into more than 1 line to make it fit into the line length limit as an alternative to always splitting every expression across 1 line?

I think merging this request with a AvoidLongLines rule would work perfectly. For shorter lines, There's no need to break them up. The above formatting should only take be applied when necessary. eslint handles this fantastically.


Do not execute manual test cases

Resolves: #755

Test failure on Fedora 34 Couldn't resolve host name... is a known and unrelated issue, fyi @thrix. Merging.


naive attempt at appending to /etc/rancher/k3s/k3s.env

fixes overwriting of k3s.env for Alpine images by using utils.WriteEnv

Thanks for having a look at this! This seems to go in the good direction, as ideally I'd like to merge the k3s and the k3s-agent block, just adding some inline fixes to make test run

Codecov Report

Merging #41 (a96534e) into master (5272892) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #41   +/-   ##
=======================================
  Coverage   21.17%   21.17%
=======================================
  Files          20       20
  Lines        1431     1431
=======================================
  Hits          303      303
  Misses       1079     1079
  Partials       49       49

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 5272892...a96534e. Read the comment docs.

Looking good as basic k8s test setup seems to pass, merging !


Transpile error in play().then of HTML5 video tag

Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[X] 3.x
[ ] 4.x
I'm submitting a ...  (check one with "x")
[X] bug report
[ ] feature request
Current behavior:
I have a <video> tag controlled by code for play/stop user actions. When user asks to play, the following code is executed (as per Google Chrome hint:)
this.myVideo.play().then(
  (value) => {
    // doing stuffs after video started (i.e. enable pause/stop button)
    ...
   },
   (error) => {
     // handling errors
     ...
   }
);

When I run the code using ionic serve everything goes fine, but when I compile for platform using ionic cordova build iOS --prod the following error occurs:
[10:10:41]  typescript: src/components/my-viewer/my-viewer.ts, line: 117
            Property 'then' does not exist on type 'void'.

     L117:  this.myVideo.play().then(
     L118:      (value) => {

Error: Failed to transpile TypeScript
    at errorCheckProgram (/Users/Sergio/Projects/myvideo-test/node_modules/@ionic/app-scripts/dist/aot/aot-compiler.js:119:39)
    at Object.<anonymous> (/Users/Sergio/Projects/myvideo-test/node_modules/@ionic/app-scripts/dist/aot/aot-compiler.js:89:21)
    at step (/Users/Sergio/Projects/sot-actionmode/myvideo-test/@ionic/app-scripts/dist/aot/aot-compiler.js:32:23)
    at Object.next (/Users/Sergio/Projects/myvideo-test/node_modules/@ionic/app-scripts/dist/aot/aot-compiler.js:13:53)
    at fulfilled (/Users/Sergio/Projects/myvideo-test/node_modules/@ionic/app-scripts/dist/aot/aot-compiler.js:4:58)
    at <anonymous>
[

Expected behavior:
I would expect that the transcript compiling process to complete successfully independently of what type of platform I'm building for.
Steps to reproduce:
Just create a simple page with a <video> tag and a button, then use the code above to start play when the user click on the button. Try to compile and see what happens. I don't provide any stackblitz/plunker because the issue is related to the transcript compiling process.
Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
cli packages: (/usr/local/lib/node_modules)

    @ionic/cli-utils  : 1.18.0
    ionic (Ionic CLI) : 3.18.0

global packages:

    cordova (Cordova CLI) : 7.1.0

local packages:

    @ionic/app-scripts : 3.1.2
    Cordova Platforms  : android 6.2.3 ios 4.5.3
    Ionic Framework    : ionic-angular 3.9.2

System:

    Android SDK Tools : 26.1.1
    ios-deploy        : 1.9.2
    ios-sim           : 5.0.13
    Node              : v8.9.1
    npm               : 5.5.1
    OS                : macOS Sierra
    Xcode             : Xcode 9.0.1 Build version 9A1004

Environment Variables:

    ANDROID_HOME : /Library/Android/sdk

Misc:

    backend : legacy

This looks like a typescript thing rather than Ionic issue. You can set the type of this.myVideo to be a HTMLVideoElement to get the proper info.

class MyClass{

  @ViewChild('video') myVideo: HTMLVideoElement;

Thanks @mhartington, actually this.myVideo is already an HTMLVideoElement got via @ViewChild() but it doesn't work.

Hey, sorry, mistake on my end. VideoChild returns an ElementRef, not a direct DOM reference. You need to then grab the nativeElement.
    this.video.nativeElement
      .play()
      .then(() => this.video.nativeElement.pause());

Actuallythis.video.nativeElement it works, but consider the following:
  @ViewChild('myVideo') private myVideoRef: ElementRef;
  private myVideo: HTMLVideoElement;

  this.myVideo = this.myVideoRef.nativeElement;
  ....

then  this.myVideoRef.nativeElement.play().then() works but this.myVideo.play().then() returns the compile error Property 'then' does not exist on type 'void'. Why?

Hello @mhartington, any news on this?

That example does not happen for me.
import { Component, ElementRef, ViewChild } from '@angular/core';
import { NavController } from 'ionic-angular';

@Component({
  selector: 'page-home',
  templateUrl: 'home.html'
})
export class HomePage {
  @ViewChild('video') video: ElementRef
 private myVideo: HTMLVideoElement;
  constructor(public navCtrl: NavController) {
  }

  ionViewDidLoad(){
    this.myVideo = this.video.nativeElement;
  }

  togglePlay(){
     this.myVideo
      .play()
      .then(() => this.myVideo.pause());
  }

}

As an example, this compiles fine in both dev builds and production builds

Can you please share the output of your ionic info?

$ ionic info

cli packages: (/Users/mhartington/n/lib/node_modules)

    @ionic/cli-utils  : 1.19.0
    ionic (Ionic CLI) : 3.19.0

global packages:

    cordova (Cordova CLI) : 7.1.0

local packages:

    @ionic/app-scripts : 3.1.6
    Cordova Platforms  : ios 4.5.4
    Ionic Framework    : ionic-angular 3.9.2

System:

    Android SDK Tools : 25.2.5
    ios-deploy        : 2.0.0
    Node              : v8.9.1
    npm               : 5.5.1
    OS                : macOS High Sierra
    Xcode             : Xcode 9.2 Build version 9C40b

Environment Variables:

    ANDROID_HOME : /Users/mhartington/Library/Android/sdk

Misc:

    backend : pro

This was a newly created project from the CLI. Let's continue the conversation over on the ionic forum instead of here.

Thanks, I'll give it a try using a new project tomorrow morning (I'm from Italy) and let you know what happens


v27 Bugs on Sonim XP3plus

Downloaded v27 today and two bugs I'm noticing right off the bat are:

The 4 key is stuck on defaulting to "H", whereas v26 immediately learned to default to "I", which is obviously typed much more frequently.
When I opened a text message up in v26, the first key I pressed would type something. In v27 the first keypress "activates" TT9, and the second key press then types something.

There seems to still be an issue with missed keypresses but I need to spend more time with it to verify. If there are new settings that are causing these issues that I don't know about my bad, but please let me know if there are.
Edit: I'm also noticing when I add a new word to the dictionary, before the popup would happen in whatever text window I was in, now the popup takes me to the app settings, and then after pressing ok I have to navigate all the way back to the text window I was in.

I see. There is a problem with uppercase words in general, not just "I". I'll fix it in the next version.
This is intentional. By default, Android does not activate the keyboard until you touch a text or number field. This is how many apps are designed, not taking into account there are non-touchscreen phones.

Here is the longer explanation of 2, if you are curious why is that. In the beginning, TT9 used to always automatically activate itself, but there were many reports it wouldn't deactivate when one finished typing. This is because Android doesn't know how to handle the case when the keyboard activates itself, instead of the current application activating it. The last two examples are #364 and #367, but the same happens in other applications and on other phones, too. And now, the control is entirely in your hands. If the app decides not to activate TT9 on startup, you can do so using OK, or by touching the screen, as per the Android standard.
I realize this may cause inconvenience, but it is the lesser of two evils.
Also, after discussing the same in #429, I realized all this is a bit unclear. I'll add the explanation in the manual to avoid the confusion to some extent.

The lagging issue does seem better. The "pressing once to activate TT9" can be easily gotten used to, and sounds like capitalization will be fixed. Just bought you 5 beers man, thanks for the great work.

I have fixed small bugs, somewhat related to what you reported. The only thing I could not reproduce was the Add Word window navigating to some other screen. However, I have changed the way it works completely, so the problem might have just went away. Let's wait for v28.0 a couple more days and see how it works. If you are still experiencing any of the above, please report back.

Just bought you 5 beers man, thanks for the great work.

Much appreciated!

@theancientways, v28.0 is out. Please, install and let me know if any of the issues still persists. I think I was able to fix everything, besides the Add Words thing, which I was not able to reproduce.

@sspanak I don't know what you did in v28.0 but you fixed the lag issues on the Sonim XP3plus. I also see you fixed the "add word" and upper case word issues. Kudos!! Donating again to thank you.

Excellent! And thank you for the donation again.
Since there is nothing left to do, I am closing this.


Move module-level comment out of _RNNCell class

The _RNNCell class docstring contained a description of the module
itself. I moved it back to the module's docstring.
I also fixed a comment that wasn't correctly indented.

Can one of the admins verify this patch?

Perhaps the comment belongs in tensorflow/contrib/rnn/python/ops/core_rnn_cell_impl.py instead?

@jhseu what's the rationale behind the existence of both tensorflow/python/ops/rnn_cell_impl.py and tensorflow/contrib/rnn/python/ops/core_rnn_cell_impl.py? why not merge them in a single file?

contrib specifies APIs that aren't stable yet. Some parts of the RNN API have stabilized, and others haven't.

Jenkins, test this please.

Great! Maybe close this?

It is merged already.


Unreviewed commit on master!

Unreviewed commit(s) COMPARE was merged to master by cbdnyu.
Comment with appropriate number of :+1: to approve.

Reviewed requirements satisfied? No
Ancestors reviewed? No

Minimum Reviews
0/2

More info

+1

Minimum Reviews
1/2

mikelodder7
1

More info

+1

Review Error for aalireza @ 2022-06-02 23:05:39 UTC
User failed mfa authentication, public email is not set on your github profile. see go/mfa-help

+1

Minimum Reviews
2/2

mikelodder7
1

aalireza
1

Pull request is reviewed and can be merged to master!
More info


TF modules and Templates

Suggest to move the reusable modules under "modules" folder (i.e. "templates" with "elz-") and leave different landing zone templates in the "templates" folder (i.e. enterprise-landing-zone, freetrial-landing-zone). It would be more align with the terraform best practices and oracle's terraform module template as well. The landing zone templates can serve as an example how those elz modules can be used for constructing a custom landing zone. So that additional landing zone templates using elz modules can be contributed by the community.

Hey @chanstev, this structure is by design. We opted for this structure to keep the hierarchy flatter. We are opened to revisit this structure in the future.


Creation of personal portfolio project

Building a personal mobile portfolio from Figma.
Adding HTML and CSS
made some changes on styling of the css by removing spaces
added more information on README that were missing
improvement of README file to look professional

Hi @AsciencioAlex
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Reques
Kindly fix all linters errors
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

@Kamba56 I have removed the errors.


protoc-gen-ng: Optional/Required fields when using proto2

Hi, I'm aiming to avoid all the typescript errors regarding "everything is optional" in proto3 by using proto2 (where you can specify optional and required).
It seems like the plugin (protoc-gen-ng) isn't generating the typings properly for those cases, and still emits string | undefined signature.
Any ideas how to fix this?

Seems like the || undefined bit is hardcoded in
https://github.com/ngx-grpc/ngx-grpc/blob/master/packages/protoc-gen-ng/src/output/types/fields/string-message-field.ts#L79

When using proto3 optional fields experimental feature (--experimental_allow_proto3_optional) this is the error message I get:
XXX.proto: is a proto3 file that contains optional fields, but code generator protoc-gen-ng hasn't been updated to support optional fields in proto3. Please ask the owner of this code generator to support proto3 optional

Hi @isc30

proto2 is not intendend to be supported; so everything is considered optional from proto3 prospective, as you correctly note. And I must admit it should not anyhow be reflected in TypeScript; even if the field is optional and is scalar it is still not-null / not-undefined
I cannot name what was the original idea behind having everything optional. It might be the reason is already gone, since there were a lot of changes since then. However, what should always be optional is message fields or the members of oneof, that's just by Protobuf definition
experimental_allow_proto3_optional is not supported ATM and I don't really know what's required on plugin side to support it. There is a library underneath protoc-gen-ng called protoc-plugin, it might be this lib needs to be adapted in order to support it.

It might make sense to remove the optional markers on the scalar fields. However, one still can assign them to undefined, can't one?

When defining optional fields, it marks the rest as required (proto2 style) and that's where TS should not contain the | undefined in this case. The gain isn't massive but it allows to get compiler errors instead of sending invalid data to the server

same issue here
https://github.com/protobufjs/protobuf.js/issues/1468

hi @isc30
the intention is clear; could you share your thoughts on how exactly you see this?
What should be changed? What is desired output for relevant parts of the protobuf file?

Hi, since proto2 seems to be working atm, i would get extra metadata on the fields to see if they were decorated as required or optional. Then, for required fields the hardcoded place can omit the | undefined bit.
I tried building the repo but not sure how to debug the change.

Hi, since proto3 added support for optional per default recently protobuf v3.15.0, it would be great to reflect it inside the generated classes instead of declaring every field as optional. I think printAsObjectMapping and printAsJSONMapping in the fields generators should only print the "?" if the field is explicitly declared as optional in the proto3 definition.

Closing in favour of #54
Proto 2 is not going to receive a lot of support anyway


Moving from Microsoft.SqlServer.DacFx.x64 to the Microsoft.SqlServer.DacFx

SqlPackage or DacFx Version: 160.6296.0
.NET Framework (Windows-only) or .NET Core: .Net 6.0
Environment (local platform and source/target platforms): Windows

Steps to Reproduce:

Install the newest official package (not prerelease)
Try to call SqlHierarchyId.Parse()

After moving from the legacy package which is Microsoft.SqlServer.DacFx.x64 I'm not able to call some methods from SqlHierarchyId class. However, when I select the prerelease version of the package, all of that methods are available, because there is a lack of this dll and it is taken from Microsoft.SqlServer.Types installed package. As far as I investigated in the official packages Microsoft.SqlServer.Types.dll library is really small for net6.0 and netstandard2.1 frameworks, which indicates that those methods are not prepared, while net462 contains all the needed methods. Moreover, in the newest prerelease version, which is 161.6304.0-preview, this dll is not present at all for all frameworks. Is it on purpose that this dll is reduced in the official version and there is none in the prerelease version? Am I able to manually force using the Microsoft.SqlServer.Types.dll file by another package like Microsoft.SqlServer.DacFx, while this dll is used in this package? Or at least am I able to force an application to use the net462 framework package in .net 6.0 application?
PS.
There is also a strange behavior on SqlHierarchyId, because in one of our classes we are using the following structure for specific field.
    public SqlHierarchyId NodeId => SqlHierarchyId.Parse(Path);

While this structure works fine during the application run, it breaks during the unit tests, even on the legacy package version. I hope that solving the problem with lacking Microsoft.SqlServer.Types.dll also fix problem with unit tests.
(DacFx/SqlPackage/SSMS/Azure Data Studio)

Hi @jaskiniowiec135, As of now, could you kindly use the pre release version and the GA version with all the needed methods would be released soon. Thank you for your patience. Also, could you please share the details of the error message when the unit tests failed and the details of the unit test.

Hi @SeenaAugusty, because of company policy we are not able to use the pre-release version, so for now we have to stick with the legacy package. Using the pre-release version work just fine, but using the newest package (160.6296.0) results in the compilation error says there is no Parse method for SqlHierarchyId class. I tried to use the dotmorten.Microsoft.SqlServer.Types package as a workaround to get rid of the legacy package, because to be honest I only need Microsoft.SqlServer.Types namespace from the whole Microsoft.SqlServer.DacFx package in this particular project, but there is still an error with the unit tests. The error during running the unit tests is raised while getting the NodeId field which I showed above, with the message saying that there is no Parse method, just like the compilation error while using the released version of Microsoft.SqlServer.DacFx package.

Hey @jaskiniowiec135
Can you try in Microsoft.SqlServer.DacFx Version 161.6374.0 and see if the issue still exists. As it is resolved there, we could close this issue. Thank you very much.

Hi @SeenaAugusty,
I have tried the mentioned version and after removing other packages I'm able to proceed with the unit tests. It is all green, so we can close the bug. Thank you!

Thank you @jaskiniowiec135 for the update.


Add documentation page

The documentation page is missing, please complete the documentation.

@opozo Thank you for your valuable feedback!
I have made the following updates:

Added a link to the template documentation in the metadata.yaml.
Created comprehensive documentation that covers the step-by-step process of importing, configuring, and deploying the Webmetic Tag Manager Template.


[test] Make spi-available-inline test target dynamic

Same as #39676, test fails locally running on M1 did not find a prebuilt standard library for target 'x86_64-apple-macos' compatible with this Swift compiler;. Since it seems not target specific and is already constrained to VENDOR=apple should be fine to just make dynamic target.

@swift-ci Please smoke test

@LucianoPAlmeida sorry but x86_64-apple-macosx11.9 is intentional here. This is because target CPU can be i386, which leads to i386-apple-macosx11.9 which doesn't exist.

@LucianoPAlmeida you can try %target-cpu again but this time add // UNSUPPORTED: CPU=i386 and run full suite.

Ah I see @nkcsgexi, but it still will fail locally on arm64 Macs, could we limit to run only on REQUIRES: OS=macosx, so that will ensure to run only on x86_64, arm64 and possibly arm64e, would that be viable?

@LucianoPAlmeida Looks like there was a race here, either yours or mine is going to work I think, maybe it's actually better to do a require OS= in this case

@LucianoPAlmeida that would work. Could you open another PR to implement it?

@LucianoPAlmeida Looks like there was a race here, either yours or mine is going to work I think, maybe it's actually better to do a require OS= in this case

Yeah, seems like a data race in the comments 😆

@LucianoPAlmeida that would work. Could you open another PR to implement it?

Sure :)

Opened @nkcsgexi @xedin #42075


add spaces WebDAV URL as a third variant for WebDAV path

Part of #2808
currently we use old and the new WebDAV path in the tests
old: remote.php/webdav
new: remote.php/dav/files/<username>
now we also need to run the tests with the new spaces concept.
The "private" space should react the same as the other WebDAV urls /dav/spaces/<uuid>/
The Example tables of the webdav related tests need to get an extra line. e.g.
  Scenario Outline: upload a file and check download content
    Given using <dav_version> DAV path
    When user "Alice" uploads file with content "uploaded content" to <file_name> using the WebDAV API
    Then the content of file <file_name> for user "Alice" should be "uploaded content"
    Examples:
      | dav_version | file_name      |
      | old         | "C++ file.cpp" |
      | old         | "file #2.txt"  |
      | new         | "C++ file.cpp" |
      | new         | "file #2.txt"  |

    @skipOnOcV10
    Examples:
      | spaces      | "C++ file.cpp" |
      | spaces      | "file #2.txt"  |

@amrita-shrestha is looking into this issue

TODO:
Note: Please create a separate issue when you're taking one of the suites from above todo, link that issue with the correct suite and properly (check/uncheck) boxes after that. Also, mark your in-progress suites, so that others will not start working on that.

[ ] apiAuth
[ ] apiAuthOcs
[ ] apiAuthWebDav
[ ] apiCapabilities
[ ] apiComments
[ ] apiFavorites
[ ] apiFederationToRoot1
[ ] apiFederationToRoot2
[ ] apiFederationToShares1
[ ] apiFederationToShares2
[ ] apiMain
[ ] apiProvisioningGroups-v1
[ ] apiProvisioningGroups-v2
[ ] apiProvisioning-v1
[ ] apiProvisioning-v2
[ ] apiShareCreateSpecialToRoot1
[ ] apiShareCreateSpecialToRoot2
[ ] apiShareCreateSpecialToShares1
[ ] apiShareCreateSpecialToShares2
[ ] apiSharees
[ ] apiShareManagementBasicToRoot
[ ] apiShareManagementBasicToShares
[ ] apiShareManagementToRoot
[ ] apiShareManagementToShares
[ ] apiShareOperationsToRoot1
[ ] apiShareOperationsToRoot2
[ ] apiShareOperationsToShares1
[ ] apiShareOperationsToShares2
[ ] apiSharePublicLink1
[ ] apiSharePublicLink2
[ ] apiShareReshareToRoot1
[ ] apiShareReshareToRoot2
[ ] apiShareReshareToRoot3
[ ] apiShareReshareToShares1
[ ] apiShareReshareToShares2
[ ] apiShareReshareToShares3
[ ] apiShareUpdateToRoot
[ ] apiShareUpdateToShares
[ ] apiSharingNotificationsToRoot
[ ] apiSharingNotificationsToShares
[ ] apiTags
[ ] apiTranslation
[ ] apiTrashbin
[ ] apiTrashbinRestore
[ ] apiVersions
[ ] apiWebdavEtagPropagation1
[ ] apiWebdavEtagPropagation2
[ ] apiWebdavLocks
[ ] apiWebdavLocks2
[ ] apiWebdavLocks3
[ ] apiWebdavLocksUnlock
[ ] apiWebdavMove1
[ ] apiWebdavMove2
[ ] apiWebdavOperations
[ ] apiWebdavPreviews
[ ] apiWebdavProperties1
[ ] apiWebdavProperties2
[ ] apiWebdavUpload1
[ ] apiWebdavUpload2
[ ] apiWebdavUploadTUS

the mentioned todo is finished :partying_face: :fireworks:


Feature request: Add selected token name to the roll dialog window.

As a simple feature request I would like to have the name of the selected token listed on the roll dialog window. Either in the title or above the modifiers.
A more sophisticated (and complex?) solution would be a dropdown to the controlled tokens on the canvas? Not even sure whether this is possible.

Duplicate of #1384


add output from analyze_lesion and shape_analysis in batch_processing

Description
For the purpose of testing PRs and pre-release on functions that are not well covered by integrity testing, it would be nice to add a few functions in the batch_processing and append results at the end.
Notably:

sct_analyze_lesion
sct_process_segmentation -p shape (already run but not appended to final results)

Closing as I've grouped all of the ideas into one issue here: https://github.com/neuropoly/spinalcordtoolbox/issues/3250


Redirect back after discarding a job

Discarding a job can currently be done from "Failed jobs" and "Scheduled jobs" pages, but, as a result, the user is always redirected to "Failed jobs" page.
This change will make the user be redirected back instead.

Thank you @thiagobc, and sorry for the delay! I'be been busy with other stuff and haven't had time to look at this gem, but I'm catching up now.

Ahh, this breaks the case of discarding a job from its individual page. It'd try to redirect to the job page, but it'll get a 404 error because the job is gone. I'll fix it.

Oh, no worries at all, not your fault! I'm not sure why the tests didn't run automatically as part of your PR... they should have! You shouldn't need to remember that. Thanks a lot for your fix and your kind words!


Trigger urls_need_reloading signal on slug change

Fixes #3689

@stefanfoulis please verify this fixes your issue.

@ojii thanks for taking a stab at this! Travis is failing because of the pyflakes linkter. We simply need to remove the unused import here: https://github.com/ojii/django-cms/blob/fix-3689/cms/tests/signals.py#L9
I've run tests locally and they all pass. I'm also chasing @stefanfoulis for a concise test that this fixes the original issue.


Fix maps issues in clj.lfe

This PR will close #279. Thanks for spotting the issue, @rvirding!
I've added a couple more tests, as a minimal sanity check and also took this opportunity to do some more dogfooding by preferringclj macros in guards when applicable.
https://www.youtube.com/watch?v=GlxKxMfpCb4

Note that prior to these changes, clj:map?/1 was illegal in guards. 🙊


Update README.md

Coverage remained the same at 100.0% when pulling b37a2db9ad923e67f1e25816b7da65f0fac6de78 on jeantimex-patch-1 into b1fc21a721e3a00556339822228f4956d64ff488 on master.


WordConfetti: add simple hashing function

With word confetti, we were still getting an issue of the word color and tilt changing on rerender (the useMemo originally implemented wasn't working).
This PR adds a simple hashing function so the same word will always have the same color and tilt
https://github.com/user-attachments/assets/2fd15dd7-3670-421c-8837-cd593a1330f4

@jliotta created a follow up at https://github.com/Cambly/syntax/pull/559


scalafmt issue in build.sbt

relates to https://github.com/apache/incubator-pekko-connectors-kafka/actions/runs/3474817556/jobs/5808394670

Is this necessary right now? I actually have local changes which updates scalafmt and formats everything (just like the other PR's), its only blocked due to https://github.com/apache/incubator-pekko-connectors-kafka/issues/2 but I can ping the scalafmt guys to make a new release (they already fixed the bugs in master).

@mdedetrich the 2 commits here fix issues that are causing Github Actions CI to fail at the moment. We can still do #2 but this helps until that is done.


Research Machine selection when scaling down a MachineSet

When a MachineSet is scaled down, we may have additional criteria to help chose which Machine is the most appropriate one to Delete.  We should determine what the current logic is, and if we should introduce some additional functionality where we can influence which Machine is chosen.

This PR merged, which adds the ability to annotate a machine to signify that it's the one you want to delete.
https://github.com/kubernetes-sigs/cluster-api/pull/726

We still need to update our cluster-api fork to include that change

merged cluster-api-update PR

note the key bit about selecting a Machine for deletion when scaling down a MachineSet:
Simple delete policy has been modified to select a Machine for scale down with the annotation machineset.clusters.k8s.io/delete-me=yes.


Contribution: evaluate adding PR description templates

PRs should have a minimum description stating WHAT has been changed and perhaps even WHY the change was necessary. This will help reviewers greatly. Also, if there is a related issue, the PR should link to that issue.
This issue deals with investigating whether those templates can be provided automatically, which seems to be the case according to the links below.
Once a suitable template has been created, the contributing.md file should document it as well.

https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/about-issue-and-pull-request-templates
https://embeddedartistry.com/blog/2017/08/04/a-github-pull-request-template-for-your-projects/

Having a template would help reviews a lot. With the aim of keeping them simple and quick, I think the template should include and agree it should include:

The type of change, e.g. bug fix, new feature, cosmetic (no code changes), etc.
Specifically what was changed. This should be clear and detailed to avoid reviewers having to comb through the PR before they understand its scope.
If not a bug, why the change was necessary (Motivation)
Link to an issue that explains the change in more detail. If the issue is detailed, step 2 can be shortened.

Another good example could be the k8s PR template: https://github.com/kubernetes/kubernetes/blob/master/.github/PULL_REQUEST_TEMPLATE.md?plain=1
If you take a look at the source code, you will see that they have added some markdown/html comments with tips and so on to guide the author.


1.16.0.pre.1 and .pre.2 sometimes do not generate binstubs on bundle install

Can be reproduced with: bundler_bug.zip. One would need to change the path to the ruby executable and the ruby version at the top of the file (ruby version is the version used by bundler in the gem path, e.g. 2.3.0 for ruby/2.3.0/).
If you run the shell script the first time, it will setup a self-contained gem home (in env/gems) and install bundler in it, then run bundle install. The first time, the binstubs are generated as expected in env/bundle_dir/bin.
However, if one rm -rf env/bundle_dir and re-run the script, the install does not re-generate the stubs. 1.15.4 does generate them. Neither 1.15.0.pre.1 or .pre.2 do.

Neither 1.15.0.pre.1 or .pre.2

Do you mean 1.16.pre.1 and 1.16.pre.2?

Ah ... yes .. edited to fix.

Does running bundle pristine fix the issue for you?

No. The binstubs folder remains empty

Sorry, that's inexact. There is one binstub in there: bundle.

Waitaminute.
Not true, running pristine with 1.16.pre.2 does re-add the binstubs.
I don't know if it helps finding the source, but running pristine using 1.15.4, on the install made with 1.16.pre.2 does not reestablish the binstubs.

In summary:

Install with 1.15.4 binstubs are present. Delete bundle_dir folder, install again with 1.15.4: binstubs are present
Install with 1.16.pre.2, binstubs are present. Delete bundle_dir folder, install again with 1.16.pre.2: binstubs are missing, only bundle is present.

running bundle pristine using 1.16.pre.2 re-adds the binstubs.
running bundle pristine using 1.15.4 does not re-add the binstubs

Ping

I’m tempted to say this isn’t an issue, and to use pristine if you’ve manually mucked with gem installations (no different than if you’d deleted any other file in the gem), but I suppose I could be convinced otherwise

I can understand the temptation ...
However, the whole things looks mushy to me. First, bundler does recreate the bundle binstub (the only one it does recreate). Moreover, I don't touch the gems, only the binstub dir (and nothing else, everything else is left as-is).
My use-case here is to make sure that obsolete binstubs are removed. Having to do a full pristine just for that sounds like a whole lot of work.
Finally, this breaks backward compatibility, and so far nobody seem to have an idea why ...

Oh I know exactly why it happened, I made the PR that made the change because it lets “no-op” bundle installs be a lot faster

Could you give me a link to that PR ? At least I know what I'm up against ... :P

I thought it was https://github.com/bundler/bundler/pull/5844 but I'm no longer sure... I'll look into this when I get a cache

@doudou do you know if this issue still occurs on the latest version of Bundler?


StorageError-"Server encountered an internal error. Please try again after some" while uploading blobs to local storage on Azure Storage Emulator

Storage Explorer Version: 1.5.0
Platform/OS Version: Windows 10
Steps to Reproduce:

Downloaded blobs from cloud to local disc.
Uploading to blob container on Storage Emulator using Microsoft Azure Storage Explorer.

Error is also reproducible while uploading using AzCopy.
Exception Details:
{
  "name": "StorageError",
  "message": "Server encountered an internal error. Please try again after some time.\nRequestId:15694ebf-d08e-4daa-955f-029800ca11d0\nTime:2018-10-29T11:06:57.1328600Z",
  "stack": "StorageError: Server encountered an internal error. Please try again after some time.\nRequestId:15694ebf-d08e-4daa-955f-029800ca11d0\nTime:2018-10-29T11:06:57.1328600Z\n    at Function.StorageServiceClient._normalizeError (C:\\Program Files (x86)\\Microsoft Azure Storage Explorer\\resources\\app\\node_modules\\azure-storage\\lib\\common\\services\\storageserviceclient.js:1198:23)\n    at Object.StorageServiceClient._processResponse (C:\\Program Files (x86)\\Microsoft Azure Storage Explorer\\resources\\app\\node_modules\\azure-storage\\lib\\common\\services\\storageserviceclient.js:744:50)\n    at Request.processResponseCallback [as _callback] (C:\\Program Files (x86)\\Microsoft Azure Storage Explorer\\resources\\app\\node_modules\\azure-storage\\lib\\common\\services\\storageserviceclient.js:317:37)\n    at Request.self.callback (C:\\Program Files (x86)\\Microsoft Azure Storage Explorer\\resources\\app\\node_modules\\request\\request.js:185:22)\n    at emitTwo (events.js:126:13)\n    at Request.emit (events.js:214:7)\n    at Request.<anonymous> (C:\\Program Files (x86)\\Microsoft Azure Storage Explorer\\resources\\app\\node_modules\\request\\request.js:1157:10)\n    at emitOne (events.js:116:13)\n    at Request.emit (events.js:211:7)\n    at IncomingMessage.<anonymous> (C:\\Program Files (x86)\\Microsoft Azure Storage Explorer\\resources\\app\\node_modules\\request\\request.js:1079:12)",
  "code": "InternalError",
  "statusCode": 500,
  "requestId": "15694ebf-d08e-4daa-955f-029800ca11d0"
}

Did you have your emulator running?  Storage Explore doesn't automatically start the emulator.

Hi,
Thank you for looking into it.
Yes, I am running emulator before trying the copy.
One more clue is, upload to local storage is failing only for few of the
blobs.
On Mon, 29 Oct 2018 at 23:53, jinglouMSFT<EMAIL_ADDRESS>wrote:

Did you have your emulator running? Storage Explore doesn't automatically
start the emulator.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/AzureStorageExplorer/issues/817#issuecomment-434022947,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AT6npDLLJRRj2M_Rb0iIEfQ6pgQihf4pks5up0eZgaJpZM4X-80Q
.

What kind of blobs are those that are failing?  Append blobs?  Page blobs? Block blobs?

It sounds like this is an issue with the emulator. Can we ping someone on the emulator team to help investigate this?

Yes, we also suspect that, because we are getting this exception while
copying using AzCopy command also.
Please point me to right community.
On Wed, 31 Oct 2018 at 04:38, Matthew Rayermann<EMAIL_ADDRESS>wrote:

It sounds like this is an issue with the emulator. Can we ping someone on
the emulator team to help investigate this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/AzureStorageExplorer/issues/817#issuecomment-434502908,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AT6npFfLF9mv5QGcppPSmNafGPo0yNIMks5uqNvtgaJpZM4X-80Q
.

I've already contacted the emulator team. Someone there will comment on the issue here.

It turns out the emulator is unable to keep up with the amount of requests AzCopy is sending all at once. It is a known issue. We'll have to tweak it so that AzCopy is sending requests to the emulator at a slower rate.

      It turns out the emulator is unable to keep up with the amount of requests AzCopy is sending all at once. It is a known issue. We'll have to tweak it so that AzCopy is sending requests to the emulator at a slower rate.

Could you let me know if there any ETA for this?
Also one more clue is that the entire thing works when the copy command is executed from Ukraine but not working from Hyderabad/INDIA.

@mvmnrao could you please clarify what is not working from Hyderabad? Are you uploading files to the public cloud?

Hi @mvmnrao, thanks for getting back to us. Could you please clarify what "local
storage created in Azure Storage Emulator" means?
@MRayermannMSFT Any insights?

I created local storage for development and debugging purpose. Please refer below link for the details, those are the steps I followed to setup local storage.
https://docs.microsoft.com/en-us/azure/storage/common/storage-use-emulator

Hi @mvmnrao, thank you so much for getting back to me!
I see, so you are transferring blobs from the public cloud to the emulator directly. @MRayermannMSFT, what is the AzCopy command that Stg Exp uses in this scenario? Does it do a download from public cloud to a local cache first, and then upload to emulator from local?

Hi,
Below is the command we are using:
/source:<> /dest:devstoreaccount1 /sourcekey:<> /destkey:Eby8vdM02xNOcqFlqUwJPLlmEtlCDXJ1OUzFT50uSRZ6IFsuFq2UVErCz4I6tq/K1SZFPTOtr/KBHBeksoGMGw== /pattern:<> /snapshot /y /s /synccopy
And yes, I believe AzCopy downloads blobs to local cache first and then it pushes to local storage and it is behavior of the AzCopy. I don't think there is other way AzCopy works?

I see. So you are using AzCopy V8. @EmmaZhu could you please help with this issue?

Sure @EmmaZhu , I will try this option and get back, thank you for inputs.

Doesn't appear to be a Storage Explorer issue, so I will close.
@mvmnrao, @EmmaZhu, @zezha-msft, since this appears to be an emulator issue, please continue the thread at an appropriate location.

@craxal would you mind sharing locations for AzCopy and Storage Emulator? I will post it there because this is still an issue for us and needs to be fixed.

This error started happening to me when I was using a different storage account connection string for the runtime (AzureWebJobsStorage) and for the actual BlobTriggerAttribute(Connection = "%ConfigSection:ConnectionString%")

@fperreaultnv Are you referring to the .NET client library? We use the JavaScript client library.

@craxal Indeed I am.

@fperreaultnv Are you suggesting then the problem is with the emulator? Since we don't yse the .NET client library, I'm not sure how that fits in here.

I tried to put (upload) multiple files on a container with SAS token, but get the same Response 500:
PUT http://<IP_ADDRESS>:10000/devstoreaccount1/inbox/9jsV5uKbAEM.jpg?sv=2018-03-28&sr=c&sig=ax9Flsc5F9wtjwqjAt76P9akEE3s3Geiy%2Bzmlpry0WY%3D&st=2020-09-18T18%3A53%3A10Z&se=2020-09-18T19%3A53%3A10Z&sp=w 500 (Server encountered an internal error. Please try again after some time.)
But it says "201 created" at the end...

@philipp-bananaware What emulator are you using? Connecting to an emulator via SAS is not supported scenario, as it depends on whether the emulator supports authenticating requests using a SAS. If you continue to encounter issues, and you believe this to be a bug in Storage Explorer, please open a new issue.

It happened to me as well and it seems it's caused by a combination of an old (deprecated) emulator (now Azurite is the one you should use) and file name being too long (over 128 chars including container name).


fix: fix QSP-4 according to quandstamp recomendation

Description

Checklist:

[x] Pull Request references Jira issue
[x] Pull Request applies to a single purpose
[x] I've added comments to my code where needed
[x] I've updated any relevant docs
[x] I've added tests to show that my changes achieve the desired results
[x] I've reviewed my code
[x] I've followed established naming conventions and formatting
[x] I've generated a coverage report and included a screenshot
[x] All tests are passing locally

merging fix to add the tests on the base branch


Refactor llm and add http compute option

This PR refactors the llm host component to make it easier to add alternate implementations. This PR also introduces an alternate implementation where the LLM compute workload is moved to a remote server through HTTP.  This allows for a better local development experience by enabling access to AI-grade GPUs.
The runtime-config looks like the following
[llm_compute]
type = "remote_http"
url = "<server URL>"
auth_token = "Auth token"

The remote server needs to implement 2 endpoints - /infer and /embed.  Both the endpoints consume the auth_token through the Authorization: Bearer <token> header.
Both the end points take are of POST method. The response and request bodies contain JSON of the following types.
Infer
// Request
{
interface InferenceParams {
  model: string;
  prompt: string;
  options: {
    maxToken: u32,
    repeatPenalty: f32,
    repeatPenalty_last_n_token_count: u32,
    temperature: f32,
    topK: u32,
    topP: f32,
  };
}
}

// Response
{
  text: string,
  usage: {
    PromptTokenCount: u32,
    GeneratedTokenCount: u32
}

embed
// Request
{
  model: string,
  input: Vec<string>
}

// Response
{
  embeddings: Vec<Vec<f32>,
   usage: {
    PromptTokenCount: u32
  }
}

There is something a bit structurally odd about implementing a feature in the Spin core to support a plugin.  I think it's unavoidable for now but it would be a good one to revisit once we have WASI2 host components.

Going to merge this. We haven't had any objections, and the implementation looks good!


Migrated Application cannot update: throw an general error

Description:

Migrated from APIM 2.6.0 to APIM 3.2.0, then tried to update an application that migrated from the old version. It is throwing a general error in UI and following the error log in the log file.
ERROR - GlobalThrowableMapper org.wso2.carbon.apimgt.api.APIManagementException: Mapped applications are not allowed to update.

Steps to reproduce:

Migrated APIM 260 to 320
In APIM 320, go to the Application list of the Devportal UI and select a migrtaed application
Go to production Key tab and click "Update" button

Affected Product Version:

APIM 3.2.0
Environment details (with versions):

OS:
Client:
Env (Docker/K8s):

Optional Fields
Related Issues:

Suggested Labels:

Urgent
Suggested Assignees:

Hi, the issue is resolved?


Added a dark mode switch/style

This is really cool, thanks! Just redistributed a little bit the code and I merge it!


feat: tx simulator

Ref: https://github.com/celestiaorg/celestia-app/issues/1256
This PR introduces a new package txsim for contolled fuzz testing at a transaction level. It's purpose is to simulate a wide range of possible user interactions while also being able to apply a considerable load to the network.
How it works
The package is designed in a way to make it easy to compose new fuzzing scenarios, removing the complexity around account nonces and balances, signing, encoding and broadcasting. The basic element is a Sequence. This is an interface that represents a pattern of recursive messages (one could be I send 5 dollars to you and you send 5 dollars back and so forth). Each sequence is provisioned a set of accounts that are responsible for formulating the messages. The accounts are isolated, allowing for a single sequence to be replicated hundreds of times. Sequences are also provided access to the current state as well as a deterministic source of randomness for constructing their output messages.
There are three current sequences:

Blob: a single account create random permutations of PayForBlob messages
Send: two or more accounts transferring funds to one another
Stake: a single account continuously staking, claiming and redelegating

Remaining work:

Create a command line version of the tool
Explore the need for persistence and garbage collection of accounts
Look to brainstorm useful sequences such as goverance proposals and authz.

Looks like there's some data race occurring but I'm not sure if it's actually triggered from the new code I wrote

Codecov Report

Merging #1613 (60e53ca) into main (e905143) will increase coverage by 1.50%.
The diff coverage is 61.06%.

@@            Coverage Diff             @@
##             main    #1613      +/-   ##
==========================================
+ Coverage   49.17%   50.67%   +1.50%
==========================================
  Files          84       91       +7
  Lines        4948     5691     +743
==========================================
+ Hits         2433     2884     +451
- Misses       2277     2508     +231
- Partials      238      299      +61

Impacted Files
Coverage Δ

testutil/testnode/read.go
0.00% <0.00%> (ø)

testing/txsim/run.go
61.81% <61.81%> (ø)

testing/txsim/stake.go
65.21% <65.21%> (ø)

testing/txsim/account.go
65.63% <65.63%> (ø)

testing/txsim/blob.go
77.35% <77.35%> (ø)

testing/txsim/client.go
77.48% <77.48%> (ø)

testing/txsim/send.go
100.00% <100.00%> (ø)

testutil/testnode/full_node.go
82.87% <100.00%> (ø)

testutil/testnode/node_init.go
64.17% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Added SPRemoteFarmTrust resource

This PR adds a SPRemoteFarmTrust resource - this is used when trusting a remote farm for things such as federating search (as described at https://technet.microsoft.com/en-us/library/dn133749.aspx)
I've not added this to the main examples as its more of a niche thing, and when the revamped examples are done they will be added in for a resource specific example at that point.

[X] Change details added to Unreleased section of changelog.md?
[X] Added/updated documentation and descriptions in .schema.mof files where appropriate?
[n/a] Examples updated for both the single server and small farm templates in the examples folder?
[X] New/changed code adheres to [Style Guidelines]?(https://github.com/PowerShell/DscResources/blob/master/StyleGuidelines.md)?
[X] Unit and Integration tests created/updated where possible?

This change is

Since the referenced article specifically references Search: Does this also work for consuming other services than search??

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


System.ArgumentOutOfRangeException: Count cannot be less than zero. Parameter name: count

My app hosted in azure had lot of failure on secondary instance. When investigating the app logs we had x number of requests returning 500 and no corresponding exception in application insights. The ones that did made to server returned this error tagged to operation_Id.

System.ArgumentOutOfRangeException: Count cannot be less than zero.
Parameter name: count
at System.String.CopyTo(Int32 sourceIndex, Char[] destination, Int32 destinationIndex, Int32 count)
at System.IO.StreamWriter.Write(String value)
at Microsoft.AspNetCore.Hosting.Views.ErrorPage.d__6.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Extensions.RazorViews.BaseView.d__29.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http.Frame`1.d__2.MoveNext()

I am not sure if this is an issue, could anyone tell me how to solve this from happening. This happens intermittently and i believe this brings the app down.
Extra information
{
   "parsedStack":[
      {
         "method":"System.String.CopyTo",
         "assembly":"System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e",
         "level":0,
         "line":0
      },
      {
         "method":"System.IO.StreamWriter.Write",
         "assembly":"System.Runtime.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a",
         "level":1,
         "line":0
      },
      {
         "method":"Microsoft.AspNetCore.Hosting.Views.ErrorPage+<ExecuteAsync>d__6.MoveNext",
         "assembly":"Microsoft.AspNetCore.Hosting, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60",
         "level":2,
         "line":0
      },
      {
         "method":"System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw",
         "assembly":"System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e",
         "level":3,
         "line":0
      },
      {
         "method":"System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess",
         "assembly":"System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e",
         "level":4,
         "line":0
      },
      {
         "method":"System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification",
         "assembly":"System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e",
         "level":5,
         "line":0
      },
      {
         "method":"Microsoft.Extensions.RazorViews.BaseView+<ExecuteAsync>d__29.MoveNext",
         "assembly":"Microsoft.AspNetCore.Hosting, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60",
         "level":6,
         "line":0
      },
      {
         "method":"System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw",
         "assembly":"System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e",
         "level":7,
         "line":0
      },
      {
         "method":"System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess",
         "assembly":"System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e",
         "level":8,
         "line":0
      },
      {
         "method":"Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http.Frame`1+<ProcessRequestsAsync>d__2.MoveNext",
         "assembly":"Microsoft.AspNetCore.Server.Kestrel.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60",
         "level":9,
         "line":0
      }
   ],
   "outerId":"0",
   "message":"Connection id \"0HLFGA416MUIK\", Request id \"0HLFGA416MUIK:00000313\": An unhandled exception was thrown by the application.",
   "type":"System.ArgumentOutOfRangeException",
   "id":"28060465",
   "severityLevel":"Error"
}

Please let me know if you want more details, i have raised this(https://github.com/aspnet/KestrelHttpServer/issues/2753)  in the Kestrel community as well as i am not sure if this is an issue with Hosting or Kestrel. Any pointers would be helpful.
Many Thanks

Closing as a dup of https://github.com/aspnet/KestrelHttpServer/issues/2753 because it's the same issue.


Error when attempting to install dependencies from conda environment.yml

Describe the bug:

Directly adding a environment.yml package to my code directory does
not automatically install the correct dependencies.

Expected behaviour:

I install various dependencies including pip dependent packages to
conda environments as part of my machine learning workflows. Based
on my uncertain understanding of “Installing Dependencies”
instructions, I expect that if I add a conda environment.yml to the
directory from which I run the docker container the docker container
will use the environment.yml file to initialize the base conda
environment.

Steps to reproduce the issue:

Use conda to generate a environment.yml file from an existing conda
environment with additional pip installed packages.
Add this file to a specific home directory in which you want to
work: /user/home/folder_to_use
cd into this directory and run the following start command:
sudo docker run -d -p 8080:8080 --gpus all --name "ml-workspace" -v "${PWD}:/workspace" --env AUTHENTICATE_VIA_JUPYTER="mytoken" --shm-size 512m --restart=no mltooling/ml-workspace-gpu:latest
Navigate to the virtual desktop and start VSCode. The python
environment does not appear in the options menu.

Technical details:

Workspace version: 0.9.1
Docker version: 19.03.05
Host Machine OS (Windows/Linux/Mac): Ubuntu 18.04
Command used to start the workspace:
sudo docker run -d -p 8080:8080 --gpus all --name "ml-workspace" -v "${PWD}:/workspace" --env AUTHENTICATE_VIA_JUPYTER="mytoken" --shm-size 512m --restart=no mltooling/ml-workspace-gpu:latest
Browser (Chrome/Firefox/Safari): Firefox

Possible Fix:

Rebuild the docker file with the necessary packages and channels
hardcoded in.

Additional context:

I fully acknowledge that I am most likely missing something. What is
the proper way to install various dependencies to the workspace
besides editing the dockerfile?

@tbrassel The Install Dependencies section is related to running code via the workspace container without starting any interactive tools. In the normal usage, the environment.yml is currently not installed automatically (we might consider that in the future). When you start the workspace, it is somewhat like a virtual machine running on you computer with its own OS, installed libraries and tools.
The beste way to install additional dependencies is by just using the terminal within the workspace (find out more in the extensibility section). So, you can just create a new conda environment from your environment.yml as explained here. VS-code should automatically discover this new environment. If you like to use Jupyter, just use ipykernel to install a kernel in the given environemnt as explained here.
However, since the base conda environment of the workspace already comes preinstalled with many of the most commonly used libraries, you might not even have to install your environment.yml (depending on the complexity of your dependencies). I mostly manage my project dependencies within a requirements.txt which I install via pip within the base conda environment of the workspace.


[C++] Require CMake 3.16 or later

Describe the enhancement requested
We need to support Ubuntu 20.04 (CMake 3.16) or later, CentOS 7 (CMake 3.17) or later, and Debian GNU/Linux buster (CMake 3.13 but backports repository has CMake 3.18) or later.
So we can require CMake 3.16 or later.
Component(s)
C++

I agree.  Is there a change needed here or can this be closed now that https://github.com/apache/arrow/pull/34969 has merged?

We can't close this yet because #34969 is just for an example project.
This issue is for cpp/CMakeLists.txt not an example project.

Debian GNU/Linux buster reached EOL on 2022-09-10:
https://wiki.debian.org/DebianReleases

Version
Code name
Release date
End of life date
EOL LTS
EOL ELTS

10
Buster
2019-07-06
2022-09-10
2024-06-30

Oh... Amazon Linux 2 uses CMake 3.13.1...


Simple notifications

This is a bit of a complicated change :(
Because the plan for notifications radically changed during development, there is work here that's been undone. It seemed like a good idea to not drop the history of the older version of notifications for reference as there's a chance that some of it will come back in future iterations.
Everything before Merge branch 'notifications-master' into simple-notifications had already been reviewed and was awaiting merging, so it's only the subsequent changes that are of interest to this PR.

Depends on api#521

Main changes since the previous iteration of notifications:

removal of time-based security profile totals
removal of most notification rules (only monitored prisoners and payment sources remain)
display of notifications is now grouped per-date rather than last xx


Update: MKVToolNix.MKVToolNix version 48.0.0

Removing old versions to reduce clutter
Microsoft Reviewers: Open in CodeFlow

/azp run


Latest Version of Pelikan on Crates.io

Is your feature request related to a problem? Please describe.
I am evaluating using Pelikan for one of my project. The build system I am using(company specific) is only allowed to fetch the rust libraries from crates.io. While I see that Pelikan is available on crates.io, I saw that only 0.0.0 version is available which has dummy code in it.
https://crates.io/crates/pelikan
Describe the solution you'd like
I'd like Pelikan team to release the latest version of the code on crates.io so that I can start writing code using Pelikan.
Describe alternatives you've considered
Tried forking the code, but we are not allowed to do that in our organization.
Additional context
None!

Hi, any updates?


1

First check

[ ] I added a very descriptive title to this issue.
[ ] I used the GitHub search to find a similar issue and didn't find it.
[ ] I searched the FastAPI documentation, with the integrated search.
[ ] I already searched in Google "How to X in FastAPI" and didn't find any information.
[ ] I already read and followed all the tutorial in the docs and didn't find an answer.
[ ] I already checked if it is not related to FastAPI but to Pydantic.
[ ] I already checked if it is not related to FastAPI but to Swagger UI.
[ ] I already checked if it is not related to FastAPI but to ReDoc.
[ ] After submitting this, I commit to:

Read open issues with questions until I find 2 issues where I can help someone and add a comment to help there.
Or, I already hit the "watch" button in this repository to receive notifications and I commit to help at least 2 people that ask questions in the future.
Implement a Pull Request for a confirmed bug.

Example
Here's a self-contained minimal, reproducible, example with my use case:

from fastapi import FastAPI

app = FastAPI()

@app.get("/")
def read_root():
    return {"Hello": "World"}

Description

Open the browser and call the endpoint /.
It returns a JSON with {"Hello": "World"}.
I would like it to have an extra parameter to teleport me to the moon and back.

The solution you would like

I would like it to have a teleport_to_moon parameter that defaults to False, and can be set to True to teleport me:
from fastapi import FastAPI

app = FastAPI()

@app.get("/", teleport_to_moon=True)
def read_root():
    return {"Hello": "World"}

Describe alternatives you've considered

To wait for Space X moon travel plans to drop down long after they release them. But I would rather teleport.
Environment

OS: [e.g. Linux / Windows / macOS]:
FastAPI Version [e.g. 0.3.0]:

To know the FastAPI version use:
python -c "import fastapi; print(fastapi.__version__)"

Python version:

To know the Python version use:
python --version

Additional context

Currently, we don't have support for teleport. But since 62.0 we support top-level shared dependencies. You can power up your application using SpaceX's newest engine.
from fastapi import FastAPI, Depends
from spacex import raptor

app = FastAPI(dependencies=[Depends(raptor)])

This will add raptor engines to all your application, however, if you want to use this for a single endpoint, you can use spacex's power_endpoint function which will wrap your endpoint and power it.
from fastapi import FastAPI, Depends
from spacex import raptor

app = FastAPI()

@raptor.power_endpoint()
@app.get("/some_endpoint", include_in_schema=False)      # Keep this secret by using include_in_schema=False
async def some_function():
    return something

:joy: :joy: :joy: that was priceless @ycd ! :rocket: :cake:
@id18522096554 I imagine you accidentally submitted the issue before editing it. If you still have problems, feel free to create a new issue following the issue template step by step.


Remove some "translatable=false"

Can you remove some "translatable=false" in strings.xml, e.g. from error_server_unknown or about_libraries (so I can translate the "by")

done, see https://github.com/HugoGresse/Anecdote/commit/9dce89cdf912aa7a470a28e73d246a07f80fe137


As a user, I want to select from available RPC methods in the RPC tester app, so that I don't have to look up documentation

Closed with https://github.com/ryanio/grid-rpc-app/pull/1


HPCC-17879 Suppress warnings on unbounded index reads

Signed-off-by: Gavin Halliday<EMAIL_ADDRESS>

@richardkchapman please review


How to configure ClientCertificateCredential to avoid MsalServiceException: AADSTS700023 exception

Query/Question
I am trying to use First party app (FPA) details to access KV in C1 tenant subscription but getting below exception -

Caused by: com.microsoft.aad.msal4j.MsalServiceException: AADSTS700023: Client assertion audience claim does not match Realm issuer. Review the documentation at https://docs.microsoft.com/azure/active-directory/develop/active-directory-certificate-credentials .

I tried to refer to the link in the exception but unable to fix a way to configure ClientCertificateCredential correctly.
Code sample -
ClientCertificateCredential clientCertificateCredential = new ClientCertificateCredentialBuilder()
    .tenantId("<FPA_TENANT_ID>")
    .clientId("<FPA_CLIENT_ID>")
    .pemCertificate("<FPA_CERT>")
    .sendCertificateChain(true)
    .additionallyAllowedTenants("*")
    .build();

// create SecretClient using FPA credentials and C1 KV URL
SecretClient client = new SecretClientBuilder()
    .vaultUrl("<C1_KV_URL>")
    .credential(clientCertificateCredential)
    .buildClient();

// Throws `MsalServiceException: AADSTS700023: Client assertion audience claim does not match Realm issuer.`
KeyVaultSecret keyVaultSecret = client.getSecret("<SECRET_NAME>");

Why is this not a Bug or a feature Request?
To me, it looks like more of a configuration that I am not able to figure out than a bug.
Setup (please complete the following information if applicable):

OS: Windows 11
IDE: IntelliJ
Library/Libraries: com.azure:azure-identity:1.8.2

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Query Added
[x] Setup information Added

@y0geshdev
Thank you for reporting this issue.
Here are the steps you can consider to troubleshoot this issue:

Since you're using a First Party App (FPA) in one tenant to access resources (Key Vault) in another tenant (C1), make sure that you have configured cross-tenant access correctly. Azure AD B2B (Business to Business) might need to be set up to allow your FPA to be recognized and authorized in the C1 tenant. This involves adding the external application or service principal as a guest in the C1 tenant.

Ensure the application registration (for the FPA) has the necessary permissions to access Key Vault in the C1 tenant. If the FPA is properly registered and permissioned in its home tenant but hasn't been given access or doesn't have the right permissions in the C1 tenant, you'll encounter authorization issues.

Double-check the tenantId and clientId values you're using in your ClientCertificateCredentialBuilder setup. The tenantId should refer to the tenant where the Key Vault resides (C1 tenant), assuming you're trying to access C1's resources. If you're intending to authenticate against the FPA's tenant, ensure that all configurations align with cross-tenant access requirements.

The error might also stem from issues with the certificate itself. Ensure that the certificate used (<FPA_CERT>) is correctly uploaded in both tenants (if necessary) and that it matches the one associated with the application registration in Azure AD.

Closing this, let us know if issue still persists.


Move yasnippets to a fork

Versioning them here is a bad idea. I could easily keep up with upstream if I simply forked the yasnippet snippets repo and included it with quelpa.

Or straight.el :)

Lol thanks for your input, @raxod502.


Issue with my protractor tests using version 2.5.1 of protractor: Async callback was not invoked within timeout specified by jasmine.DEFAULT_TIMEOUT_INTERVAL

Hello,
I am running into the following issue: Async callback was not invoked within timeout specified by jasmine.DEFAULT_TIMEOUT_INTERVAL with version 2.5.1 of protractor.
I have opened a post on SO here: http://stackoverflow.com/questions/33858854
It seems to be a bug.
Can someone please suggest a workaround?

Looks like the issue was that the email was never sent


build fix

fix build failure on Archlinux and possible any other Linux distros.

i swear @CaptainChloride merged this?

Nope


Update the docs to make migration and installation bit clearer

Updates the notes to reflect the fact that Rails v6 ships with rails/webpacker by default and will need a manual installation/upgrade if users wish to use shakapacker

💯


error: flake '/home/bbigras/nixflk' does not provide attribute 'nixosConfigurations.new_host.config.system.build.toplevel'

Describe the bug
I got this error message. Note that I didn't edit the config file yet. Did I need to do it first. In the Readme it seems I can rebuild before.
To Reproduce
Steps to reproduce the behavior:

git clone https://github.com/nrdxp/nixflk.git
cd nixflk/
nix-shell
git checkout -b moi template
nixos-generate-config --show-hardware-config > ./hosts/new_host.nix
sudo ln -s $PWD /etc/nixos
rebuild new_host switch

[nix-shell:~/nixflk]$ rebuild new_host switch
evaluating file '/nix/store/2vdffwd9qpamvk9z3qhaig9fs0dam0cw-nix-2.4pre20191022_9cac895/share/nix/corepkgs/derivation.nix'
using revision 1 of repo '/home/bbigras/nixflk'
evaluating file '/nix/store/89jnlh3dlszwbnsj291s8pq1bvn9yz6a-source/flake.nix'
evaluating file '/nix/store/89jnlh3dlszwbnsj291s8pq1bvn9yz6a-source/hosts/default.nix'
evaluating file '/nix/store/89jnlh3dlszwbnsj291s8pq1bvn9yz6a-source/lib/utils.nix'
downloading 'https://api.github.com/repos/nrdxp/nixpkgs/tarball/1e232452ed4c57e9968f4342d9ba044d36403928'...
unpacking 'https://api.github.com/repos/nrdxp/nixpkgs/tarball/1e232452ed4c57e9968f4342d9ba044d36403928'...
evaluating file '/nix/store/lwp8hnp8z5i11rpjkjcp67h7zvl28kzd-source/flake.nix'
evaluating file '/nix/store/lwp8hnp8z5i11rpjkjcp67h7zvl28kzd-source/lib/default.nix'
evaluating file '/nix/store/lwp8hnp8z5i11rpjkjcp67h7zvl28kzd-source/lib/fixed-points.nix'
evaluating file '/nix/store/lwp8hnp8z5i11rpjkjcp67h7zvl28kzd-source/lib/attrsets.nix'
evaluating file '/nix/store/lwp8hnp8z5i11rpjkjcp67h7zvl28kzd-source/lib/lists.nix'
evaluating file '/nix/store/lwp8hnp8z5i11rpjkjcp67h7zvl28kzd-source/lib/strings.nix'
evaluating file '/nix/store/lwp8hnp8z5i11rpjkjcp67h7zvl28kzd-source/nixos/lib/eval-config.nix'
[17.3 MiB DL]
error: flake '/home/bbigras/nixflk' does not provide attribute 'nixosConfigurations.new_host.config.system.build.toplevel'

Expected behavior
Additional context

Yes my mistake, I must have removed a segment from the README.md that was there before, in recent edits. I'm fixing it right now, but the problem is that the flake is vcs based, it only bundles files into a flake that git is aware of. Adding the new file to the git staging area is enough to solve the problem.

Now that you mention it I remember that part from your forum post. Thanks.


Question

Thank you for this interesting work,
I want to use this method to estimate camera calibration parameters. Can it work on fixed camera videos in scenes with a lot of movement?

Hi @KOuldamer
I haven't tried training in the scenario you mentioned, but I guess it is challenging according to the design.
Assume the model could accurately learn the camera pose is 0's and the relative motions of objects. Since background is static and does not inject any information to update the networks via image reconstruction loss, it is likely to degrade the whole learning process (including camera parameters) and break the initial assumption made above.
Best,
Bolian

Hi, can this just use a separate RGB image? Or does it require additional pose input?

Thank you for this interesting work,
I want to use this method to estimate camera calibration parameters. Can it work on fixed camera videos in scenes with a lot of movement?
Hi, can this just use a separate RGB image? Or does it require additional pose input?

@zhangzhenfengjy
You can input a separate RGB image to obtain its depth map without pose input. Please check the Run Inference section.  https://github.com/bolianchen/pytorch_depth_from_videos_in_the_wild#run-inference.

您好，我想问一下您的数据集是怎么下载的呢？我的使用您给的命令是连接不上的。
发自我的iPhone
------------------ Original ------------------
From: BO-LIAN CHEN @.>
Date: Thu,Jul 14,2022 6:47 PM
To: bolianchen/pytorch_depth_from_videos_in_the_wild @.>
Cc: zhangzhenfengjy @.>, Mention @.>
Subject: Re: [bolianchen/pytorch_depth_from_videos_in_the_wild] Question  (Issue #11)
@zhangzhenfengjy
You can input a separate RGB image to obtain its depth map without pose input. Please check the Run Inference section.  https://github.com/bolianchen/pytorch_depth_from_videos_in_the_wild#run-inference.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@zhangzhenfengjy
[在專案資料夾下執行以下指令，我剛試過都還是可以下載的
./datasets/data_prep/kitti_raw_downloader.sh

您好，这个预测的深度信息是怎么进行约束的呢？深度的损失是什么呀？请您为我解惑，谢谢🙏
发自我的iPhone
------------------ Original ------------------
From: BO-LIAN CHEN @.>
Date: Mon,Jul 18,2022 6:23 PM
To: bolianchen/pytorch_depth_from_videos_in_the_wild @.>
Cc: zhangzhenfengjy @.>, Mention @.>
Subject: Re: [bolianchen/pytorch_depth_from_videos_in_the_wild] Question  (Issue #11)
@zhangzhenfengjy
[在專案資料夾下執行以下指令，我剛試過都還是可以下載的
./datasets/data_prep/kitti_raw_downloader.sh
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@zhangzhenfengjy 抱歉晚回覆您了！
建議您看SfMLearner論文，對於loss會有比較清楚的說明。
簡單來說，就是根據Structure from Motion原理，用depth network + pose network + camera intrinsics，得出前後禎畫面像素點的關聯性，把預測深度的問題轉化為前後禎畫面的重建問題。而Depth from Videos in the Wild則是緩解SfM方法要求畫面中不能有運動物體的限制，針對所有可能運動物體，額外預測3個軸向的平移，另外也提供在訓練過程中習得camera intrinsics的選擇。

感谢您的建议，数据集我还是下载不了，您有什么好办法可以分享一下吗？我不确定是否是我的网络原因，感谢🙏
发自我的iPhone
------------------ Original ------------------
From: BO-LIAN CHEN @.>
Date: Mon,Jul 25,2022 10:56 AM
To: bolianchen/pytorch_depth_from_videos_in_the_wild @.>
Cc: zhangzhenfengjy @.>, Mention @.>
Subject: Re: [bolianchen/pytorch_depth_from_videos_in_the_wild] Question  (Issue #11)
@zhangzhenfengjy 抱歉晚回覆您了！
建議您看SfMLearner論文，對於loss會有比較清楚的說明。
簡單來說，就是根據Structure from Motion原理，用depth network + pose network + camera intrinsics，得出前後禎畫面像素點的關聯性，把預測深度的問題轉化為前後禎畫面的重建問題。而Depth from Videos in the Wild則是緩解SfM方法要求畫面中不能有運動物體的限制，針對所有可能運動物體，額外預測3個軸向的平移，另外也提供在訓練過程中習得camera intrinsics的選擇。
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@zhangzhenfengjy
可以到官方網站註冊後，就能下載資料http://www.cvlibs.net/datasets/kitti/raw_data.php

谢谢你的建议，我还是下载不了，你有什么好的办法可以分享一下吗？我认为数据是我的网络原因吗？ 发自己的iPhone
…
------------------ Original ------------------ From: BO-LIAN CHEN @.> Date: Mon,Jul 25,2022 10:56 AM To: bolianchen/pytorch_depth_from_videos_in_the_wild @.> Cc: zhangzhenfengjy @.>, Mention @.> Subject: Re: [bolianchen/pytorch_depth_from_videos_in_the_wild] Question (Issue #11) @zhangzhenfengjy 抱歉晚回覆您了！ 建議您看SfMLearner論文，對於loss會有比較清楚的說明。 簡單來說，就是根據Structure from Motion原理，用depth network + pose network + camera intrinsics，得出前後禎畫面像素點的關聯性，把預測深度的問題轉化為前後禎畫面的重建問題。而Depth from Videos in the Wild則是緩解SfM方法要求畫面中不能有運動物體的限制，針對所有可能運動物體，額外預測3個軸向的平移，另外也提供在訓練過程中習得camera intrinsics的選擇。 — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

如果你是在大陆，不翻墙的话估计是下不了，很多事都干不了，ubuntu建议下个Pigchar买个无限流量就可以了，win系统好用的梯子就比较多了


MSID: 94738 Version: 1 DOI: 10.1101/2<IP_ADDRESS>6032

MSID: 94738
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>6032
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=94738
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>6032

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/94738v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>6032
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/94738/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

warn: Content of <title> element is entirely in upper case: Is that correct? 'DNA FISH'


hardMuteVideo not work

Hi, try to use hardMuteVideo method, and it tell me that no ice-ufrag, but I have this in sdp
http://i.imgur.com/eRII5ky.png

why do you want to use hardMute? the "soft" one which sends black video frames is usually enough.

Which method should I use for it? I can see only hardMuteVideo and sendMute which send iq. I need this for turning off microphone on call.


with in-publish, could npm run release be changed to npm publish?

http://ericdouglas.github.io/2015/09/27/checklist-for-your-new-open-source-javascript-project/ might have some good ideas.


First step for generating documentation

Apologies for opening the PR; I pressed the wrong button!
I closed it now again.


Use jQuery as a module

This requires some design changes to address issues I pointed out in #17439 and #17512.
Cc @showell

We’ll be able to reduce the extra typing with the directory rearrangement I described earlier. We’ll also be able to replace zrequire with normal require.

Yeah, this would be a really nice change.  Maybe once this block of things land, we should finalize the naming for that directory structure move so that we can start planning it?

I believe the two conflicts here are just with_field colliding with mock_* renames and nothing deeper.
FFTM after fixing conflicts.

I chatted with @showell, and you should feel free to merge after resolving merge conflicts.  I think ultimately the "how confusing are these names" question will need to be tested with actual contributors, and it's easy to rename things.  Might be worth posting to #frontend after you merge with the update on syntax/styling/etc.


Added another file to install that Thrive now requires

required for fixing thrive building on linux

I just found this in my pull request history... This is repo is no longer needed by thrive but for closure's sake I'm going to merge this


[Docs] Fix doxygen

Fix some doxygen errors
Fix author email typo /cc @zhoonit

Signed-off-by: Juyeong Lee<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1138. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnstreamer.mooo.com/.

:octocat: cibot: @JuYeong0413, test/unittest/unittest_nntrainer_lazy_tensor.cpp does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

:octocat: cibot: @JuYeong0413, test/unittest/unittest_nntrainer_lazy_tensor.cpp does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

:octocat: cibot: @JuYeong0413, test/unittest/unittest_nntrainer_lazy_tensor.cpp does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

:octocat: cibot: @JuYeong0413, test/unittest/unittest_nntrainer_lazy_tensor.cpp does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

Sorry for the frequent force-pushing.
The remaining doxygen errors are about missing @brief tags.
Other things like syntax errors or typos are fixed.


Doesn't work in IE11

The histograms render ok in Chrome, but not in IE11. Other Grafana charts work ok in IE11.

Sorry, I use Mac, replicate issue is hard for me.
If you provide the following information, I can check it little...

error message
histogram panel version

Screen capture of console tab in developer tools is very helpful.
https://msdn.microsoft.com/library/bg182326(v=vs.85).aspx

Hi, thanks for the response. Screenshots are attached. If you have any thoughts on where the issue is, I'd be happy to test it. I haven't gotten deep enough into Grafana to recognize where the error is coming from.
On a side note I love the panel though... the only thing I wish I could do was to normalize the histograms so that they're displayed as if they had the same total count. That would help visualize probability distribution differences between the two. This could probably also be accomplished by being able to put a second histogram on a different Y axis.

Thanks for sharing screenshot!
Not sure, but using let might be not good.
So, I rewrite it by _.each(), and make a PR.
https://github.com/mtanda/grafana-histogram-panel/pull/43
Please try this version. (copy files in dist dir to your local panel plugin dir)

On a side note I love the panel though... the only thing I wish I could do was to normalize the histograms so that they're displayed as if they had the same total count. That would help visualize probability distribution differences between the two. This could probably also be accomplished by being able to put a second histogram on a different Y axis.

I may understand your request. I'll try to implement it.

I updated the code... getting a slightly different error now :P
I only updated histogram_ctrl.js, were there any others that needed updating?

hmm, it's very strange.
When you access by Chrome, the panel plugin still work correctly?

By the way, if you're interested there is a way to test on IE from a Mac... at some point in the near future I need to learn how to write these plugins myself though so that I'd be able to help with the debugging :)
https://code.tutsplus.com/tutorials/a-fast-accurate-way-to-test-internet-explorer-on-ios-mac-os-x-and-android--cms-24192
It'd also be nice if Grafana had a way to render these on the server side. Thought it did but I couldn't find one. The whole reason we're having a problem is because we use Xibo for digital signage and Xibo uses IE for the browser.

Yes, it still works in Chrome... wondering if I did something wrong when copying the javascript

Oh, I didn't know Xibo, may be this one?
http://xibo.org.uk/
Sorry, I don't have enough time to debug this issue today. It is time to sleep :-)
If you have experience to use JS debugger, try to debug by step execution.
set breakpoint to following line, you may find something.
https://github.com/mtanda/grafana-histogram-panel/blob/c986ff8785d41dfdaff3e3a3182db254f38eb954/src/histogram_ctrl.js#L175
https://github.com/mtanda/grafana-histogram-panel/blob/c986ff8785d41dfdaff3e3a3182db254f38eb954/src/histogram.js#L52

IE doesn't support Symbol, it might cause the issue.
http://stackoverflow.com/questions/33828840/symbol-is-undefined-in-ie-after-using-babel

Is there any workaround for this, till the time the feature is available?


print_r(  $_SERVER["SERVER_NAME"]); not working

print_r(  $_SERVER["SERVER_NAME"]);
give an error
Notice: Undefined index: SERVER_NAME in C:....\xxx.php on line XX

This is not an error for the language server


Redis Kafka Connect data not in order?

Hi,
I tried to use simple 1 worker redis sink - but found at times the order of the data in kafka queue is not maintained. Is that meant to be in order?

I think you opened this on the wrong project. This project is transformations. Are you thinking of kafka-connect-redis? Kafka will process the data in the order of each partition. If your data is not keyed that could be the cause. There is no global order, only order by a partition.


Introduction - Josh Dembling (Intel)

Hello,
My name is Josh Dembling and I run the Intel Product Security Incident Response Team (PSIRT) and Bug Bounty Program. I and my team manage and coordinate the disclosure of security vulnerabilities with our customers, partners and external researchers.  I am also one of the chairs of the industry driven PSIRT Special Interest Group under FIRST.org.  In addition, I am a rapporteur for the recently approved ISO Technical Report on Multi-Party Coordinated Vulnerability Disclosure.
I look forward to working together to shape a strategy that will help the broader community develop a more stable approach to coordinated vulnerability disclosure.
Thank You
Josh

@jdemblin Welcome to the WG! Can you create a PR similar to #20 to add yourself to the list of WG members?


Switch to entry_points in setup.py

This is a topic we should discuss first. The main issue is compatibility with Jython and IronPython. Do you know are entry points supported by them? Should a separate issue be opened about that discussion so that this PR can be left for discussion about the actual implementation?

No idea about Jython and IronPython.
IMHO, the actual implementation is pretty simple, so we can continue here with everything.

We actually need an issue anyway if we merge this PR. We have a script for creating release note templates based on issues with a certain milestone. Can obviously add an issue before merging with just a reference to this PR.

Situation with Jython, IronPython and also PyPy needs to be tested.

Could you clarify what actual benefits entry points have compared to the current system?

3:

Simplify setup.py
On windows:
native executables (.exe) instead of .bat files
when interrupting an execution, I won't be asked if I want to terminate the current script (side effect of using .bat files)

I have only tested this with Python 2.7.11 and 3.5.1 on Windows.
Is it possible to setup Travis CI for this repo?

Created an issue for this, which includes a link to valid reasons for this switch: #2415


Specific class on robot response

Hi guys,
first of all great work! I used Conversational Forms on a couple of sites and I just managed to get it working on Modx as well :-).
I would like to know if there is any way to change the color of the Robot Responses (using addRobotChatResponse). I have tried for a specific class or something to pass to the function but so far I had no success.
Can you help me?

Hi @MatteoPieroni. Thank you for your interest in this project. No, you can only pass the message to addRobotChatResponse. But maybe you could do something like this? https://codepen.io/jenssog/pen/aVqVyY

Hi @jenssogaard unfortunately that cannot solve my problem, since I would have to change css every time I add or remove a question. Thanks a lot anyway, for now I solved by going for a green text! It could be an interesting feature to think of implementing though, for UI purposes. :-) Again thanks for the great work!

@MatteoPieroni I see. So your idea would be to be able to pass a class name addRobotChatResponse(), correct? Eg. addRobotChatResponse("Thank you.", "green")

That could be great for compliance with any used frameworks like bootstrap or else.
For me any preset class would be more than enough to target via css. Eg.

Hi @MatteoPieroni. I have discussed this with the team and we do not believe that there currently is a strong enough case for this discussed feature. Thank you for suggesting and you are always welcome to do a PR if you feel the need for a feature. That can sometimes increase the chances of it being adopted in the project :)

Hi @jenssogaard, thank you very much for taking the time to discuss this! I hope it will be addressed in the future if the case for its use will grow stronger. In the meantime keep up the great work!
If you would like to include cases of real use on the web in the docs I have some... :-)


with  inside returns an empty string

The following HTML code:

gets converted to an empty string.
I wrote a simple failing test:
it('should convert an anchor with img tag inside to markdown', function () {
  isEqual.inline('', '[![Alt text](/image.jpg)](/)');
});

I'm experiencing this issue as well, particularly with any images that are wrapped as a link as in the example above. How can i fix this?
Thanks!
Ray

I have the same issue, but I found out in my case it was a  inside a
 inside a
, something like:

I tested and removing only the
 fixes it.


Avoid SIGFPE when bytespersample is zero

0002-Avoid-SIGFPE-when-bytespersample-is-zero.patch from Debian,
see https://bugs.debian.org/635906 .

[Ralph Giles]

I think it would be better to return OV_EINVAL here. Neither word
nor channels should be zero here.

  I only know what has been used in Debian, and do not
really have any opinion on how to properly fix it, but can of course
modify the patch based on input. :)
--
Happy hacking
Petter Reinholdtsen

I'm happy to have you write a better patch. If you can't figure it out submit what you have and I'll help.

[Ralph Giles]

I'm happy to have you write a better patch. If you can't figure it out
submit what you have and I'll help.

OK.  Updated patch pushed, this time only checking word and channels for
zero, and returning OV_EINVAL if this is the case.  I have no idea if
this solve the original problem, though. :)
--
Happy hacking
Petter Reinholdtsen

[Ralph Giles]

Thanks for writing a patch! I made a few suggestions to make the
argument checking more defensive. With those addressed it should be
good to merge.

OK, pushed updated patch making sure word is a positive number and
channel is 1-255.
--
Happy hacking
Petter Reinholdtsen

Merged as 0a4beb1d04f802c48016b11fb939690e24173168. Thanks!


Update FUNDING.yml with Ethereum address

Description

Adding an Ethereum address per https://github.com/carbon-app/carbon/issues/853#issuecomment-526777956

Yes, I could link it to Etherscan if you'd like, that would give a generic overview of the address

@lastmjs sure. Whatever is the most user-friendly URL.

Etherscan I believe is the most used block explorer for Ethereum, so for crypto users it should be pretty nice. The best thing to do would probably be to create a very user-friendly landing page at something like sustainus.io/address/whatever-ethereum-address, that explains that donations are accepted to this address and gives the user options for which crypto tools to use to donate...I do not have that created yet


Update Coffea version to 2024.11.0

A new Coffea version has been detected.
Updated Dockerfiles to use 2024.11.0.

Thanks @lgray, I was too fast ;-(
I am rebuilding image / tag with pinned dask version https://github.com/CoffeaTeam/af-images/pull/44