Add healthcheck so DB is up before Fineract starts

Description
Describe the changes made and why they were made. Ignore if these details are present on the associated Jira ticket
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[ ] Commit message starts with the issue number from https://issues.apache.org/jira/projects/FINERACT/. Ex: FINERACT-646 Pockets API.

[x] Coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions have been followed.

[ ] API documentation at https://github.com/apache/fineract/blob/develop/api-docs/apiLive.htm has been updated with details of any API changes.

[ ] Integration tests have been created/updated for verifying the changes made.

[ ] All Integrations tests are passing with the new commits.

[x] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the list.)

Our guidelines for code reviews is at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide

@vorburger - can you check this. Just making sure that the db container is running before Fineract starts

@conradsp I was initially going to suggest that we could finish #644 first, and then have this tested by running it through that, but instead I just locally pulled this, ran podman-compose down && podman-compose build && podman-compose up -d && sleep 5s  &&  http --verify no --timeout 240 --check-status get https://localhost:8443/fineract-provider/health and saw that it still worked, so it's OK for me and I'm merging it.


The Server Issue

We have ran into an issue with the server field. I think this might actually be endemic of a much larger problem which will likely be phased out soon. So right now, the field is set to be Any. Which of course is not ideal, as a WebServer will always contain the same type of Sockets server. That being said, I think the final portion of the 0.2.2 patch will address this problem a bit more broadly. I think the server will actually be created on the calling of the constructor. Thus, we can add port listening and those things after.. With this version's release, we will also be seeing a new Session release, a ToolipsAuth update, Uploader release, and a Defaults release. All of these are on the horizon.

we move this, from _start for the servers...
    server = Sockets.listen(Sockets.InetAddr(parse(IPAddr, ip), port))

... to the WebServer Constructor. I can also make this type be no longer Any and be a Server, as now ServerTemplate and WebServer concepts are refined

we move this, from _start for the servers...
    server = Sockets.listen(Sockets.InetAddr(parse(IPAddr, ip), port))

... to the WebServer Constructor. I can also make this type be no longer Any and be a Server, as now ServerTemplate and WebServer concepts are refined

this worked


Message needs 24, not 16 bytes

Looks good to me. Any objections @christian-marie @tranma?

this binary packing makes me sad


Internal model design changes

Changed internal model design around the pathfinder to be able to expand.
Specifically: Dissolved the regular PathfinderImpl into an AbstractPathfinder with the specific implementation AStartPathfinder.
Also the PathingTask together with the ProgressMonitor got removed and the return type of the findPath method adjusted.
NOTE: This also affects the public api in some way (f.e. the return type of the #findPath method in Pathfinder.java)

In near future the AbstractPathfinder will probably find its way into the api module, but for now its hard-wired into the model module.


ci: split up lint job

This helps with identifying what failed right from the title. Also steps after a failing one are now always executed.
Also shortens the steps a bit by removing obvious names.

If we are going to step names then we should do it in all places imo.

If we are going to step names then we should do it in all places imo, so other workflows as well.
They do not hurt :)

Some are helpful as easier to read in my opinion, but Run cargo make clippy-all for run: cargo make clippy is both outdated and exactly the same on the actual run ;)
Can also remove them on other files when they feel duplicated / not at helpful. Mainly didn't want to scroll as much :innocent:

We can remove the other labels in another PR - I'd like to see how this looks without them, so merging as-is.


Feature request: PR description

Hello again ;-)
Is it possible to add description to the pull request?
Currently I'm using this when repo A updates (merge), then it commits the new tag to repo B (helm charts) and create a PR, but in the created PR there's no reference to the merge commit in repo A (or could be a release as well).
Is it possible to add description to the PR creation?

Hey, thanks for you request. I will check the APIs if this is possible.

You can use "description" as new optional input to describe your PR. Its available in version v0.5.0 or latest.

Thanks! This is great! 🥇


bug: link_destinations with spaces do not get parsed

Did you check existing issues?

[x] I have read all the tree-sitter docs if it relates to using the parser
[x] I have searched the existing issues

Tree-Sitter CLI Version, if relevant (output of tree-sitter --version)
No response
Describe the bug
Image paths with spaces do not get recognized as link_destinations
Steps To Reproduce/Bad Parse Tree
Markdown file:
![](./image1.png)

![](./image 2.png)

Parses as
section [0, 0] - [3, 0]
  paragraph [0, 0] - [1, 0]
    inline [0, 0] - [0, 17]
      image [0, 0] - [0, 17]
        link_destination [0, 4] - [0, 16]
  paragraph [2, 0] - [3, 0]
    inline [2, 0] - [2, 18]

Expected Behavior/Parse Tree
I expect the second link with a space to still parse. Is this not valid markdown?
Repro
No response

Hey, spaces in links like this are indeed not valid markdown, you would need to enclose the file name (or url or whatever your link destination is) in <...>. To quote the spec:

A link destination consists of either

a sequence of zero or more characters between an opening < and a closing > that contains no line endings or unescaped < or > characters, or
a nonempty sequence of characters that does not start with <, does not include ASCII control characters or space character, and includes parentheses only if (a) they are backslash-escaped or (b) they are part of a balanced pair of unescaped parentheses. (Implementations may impose limits on parentheses nesting to avoid performance issues, but at least three levels of nesting should be supported.)

While links with spaces in <> indeed get parsed, the parsed url includes the angle braces so the nvim key combos gf and gx still have issues with that. See #174 .

Thanks so much for the explanation! I will close this as WONTFIX since it seems like it was my error in understanding the Markdown spec, and issue 174 is already dealing with the functionality question for gf/gx


proposal: Add access log specific FactoryContext interface

Title: Add access log specific FactoryContext interface
Description:

Presently access log factory uses CommonFactoryContext interface that is also shared with the upstream implementation. Access loggers need access to listener specific methods of the FactoryContext such as listenerMetadata() which are not applicable to upstreams.

[optional Relevant Links:]

PR #15461 with the initial refactor to allow access loggers for admin interface.
Subsequent PR #15829 for fixing broken proprietary access loggers.

Fixed in #22666


ENOENT Warnings

i've installed the package with npm but i get these ENOENT warnings. Do you know a way to solve this?
Using Version 2.0.7

`WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such
file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no
such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such
file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file
or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such
file or directory, open<EMAIL_ADDRESS>WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/react-scripts/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>

Seeing this as well with the latest version

@charlesfayal Did you find any solution?

Which platform do you use (Next.js etc.)? I don't have this problem using Next.js 14.2.14

Using React 18.2.0

I'm on React v18.3.1 and seeing the same warning messages on a debian v12 box with react-qr-scanner v2.0.8. The software seems to be functioning fine for my purposes though.

I am also seeing this with a create-react-app solution.
WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>
There is no src folder in C:\Users\...\GitHub\Zapp.ie\tabs\node_modules\@yudiel\react-qr-scanner.

I am also getting the same warnings using CRA.
WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/source-map-loader/dist/cjs.js): Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>

I am also getting the same warnings  with React v17.

Error this exists, should not be marked as completed...

Error still exists, should not be marked as completed...

I am also still running into the same error.


[Windows] Clean up and simplify Windows templates

Description
This PR clean up templates by hardcoding some values.
It is easier to write $env:TEMP "toolset.json" in two places instead of having huge expression in every provisioner:
"environment_vars": [
    "TOOLSET_JSON_PATH={{user `toolset_json_path`}}"
],

It is easy to forget passing values to some provisioner group for example.
In this case, hardcoding values simplify template code and reduce templates (this approach is already used in macOS).
Also, union some scripts to single provisioner group

/azp run windows2016, windows2019

/azp run windows2016, windows2019

/azp run windows2016, windows2019


TypeError: Cannot read property 'callsite' of undefined (follow up ticket #3441)

What is your Test Scenario?

I created a new issue ticket thread to follow up on my previous issue ticket 3441 as it was marked as duplicate of 1920 and I see both 3441 & 1920 closed.  This issue still happens intermittently (when testing against internal staging env test sites) and its really hard to reproduce and point out which specific test is causing this to happen. (see screenshot)
Anyone from TestCafe team know if this is a known issue and any tickets in upcoming sprint/milestone to fix this issue? Few numbers of users have reported this issue and I'm still seeing this happen in v1.1.2. This happens on both MacOS & Windows 10 I'm willing to collect & share any debug logs if someone can provide me the steps/instructions to get the logs.
What is the Current behavior?

TestCafe quits execution with the following error:
C:\jenkins\slave\workspace\...\node_modules\brotli\build\encode.js:3
1<process.argv.length?process.argv[1].replace(/\\/g,"/"):"unknown-program");b.arguments=process.argv.slice(2);"undefined"!==typeof module&&(module.exports=b);process.on("uncaughtException",function(a){if(!(a instanceof y))throw a;});b.inspect=function(){return"[Emscripten Module object]"}}else if(x)b.print||(b.print=print),"undefined"!=typeof printErr&&(b.printErr=printErr),b.read="undefined"!=typeof read?read:function(){throw"no read() available (jsc?)";},b.readBinary=function(a){if("function"===
                                                                                ^

TypeError: Cannot read property 'callsite' of undefined
    at TestRun.callsite [as _rejectCurrentDriverTask] (C:\jenkins\slave\workspace\...\node_modules\testcafe\src\test-run\index.js:397:75)
    at TestRun._rejectCurrentDriverTask [as _disconnect] (C:\jenkins\slave\workspace\...\node_modules\testcafe\src\test-run\index.js:688:14)
    at BrowserConnection._disconnect (C:\jenkins\slave\workspace\...\node_modules\testcafe\src\test-run\index.js:281:44)
    at Object.onceWrapper (events.js:315:30)
    at emitOne (events.js:116:13)
    at BrowserConnection.emit (events.js:211:7)
    at Timeout.emit [as _onTimeout] (C:\jenkins\slave\workspace\...\node_modules\testcafe\src\browser\connection\index.js:127:18)
    at ontimeout (timers.js:482:11)
    at tryOnTimeout (timers.js:317:5)
    at Timer.listOnTimeout (timers.js:277:5)

What is the Expected behavior?

TestCafe should run without any errors and should not quit execution
What is your web application and your TestCafe test code?

Execute this command...
See the error...

Your Environment details:

testcafe version:                    happens from v1.0.0 to latest v1.1.2
node.js version:                     8.11.1
command-line arguments:
browser name and version:
platform and version:           macOS 10.14 & Windows 10
other:

I see the line in code which can lead to the issue, however, I still cannot get the cause of the issue. We'll fix the code, however, we would really appreciate it if you share some additional information about how you can reproduce it.
The problematic code is here:
https://github.com/DevExpress/testcafe/blob/ad8e7f2c446f3793dceaec00b6ac0869d775f595/src/test-run/index.js#L398


Setting the speed

It would be really nice if it was possible to setting speed of a video or playlist.
Some videos of conferences or tutorials have too much pauses or to little content to be watch in regular speed.
This might be helpful

Done. Refresh cache and give it a try: https://yt.aergia.eu/#v=h73bd9b5pPA&speed=1.5


Satellite Fetching Progress

Updated Animation Generation for RDWATCH database to upscale when rescale is on for images that are lower than 500x500 pixels.  I changed it to use a 2X multiplier until both the width and height are above 500.  This makes it easier to draw the fonts and other items that are needed.  It resizes using the PIL.Image.Resample.NEAREST method.
Updated the satellite-fetching/running endpoint to return back the celery information that is logged during the image downloading progress.
This image is now a hover card that shows up when hovering over the downloading status indicator.
Show more information about downloading when hovering over the downloading icon
Show errors from downloading images in the same interface when an error/exception has occurred.

Haven't tested it, but overall LGTM


Interactive App

Using tools like Panel it may be possible to create an interactive app to create a visualise the results of at the filter before applying it to all levels

Interactive tools like this could be sued to draw ontop of the image what we want to filter

Starting on a minimum version of the dream below:


[Bug]: can't create Application missing API permission

Is there an existing issue for this?

[X] I have searched the existing issues

Affected Resource(s)
apiVersion: applications.azuread.upbound.io/v1beta1
kind: Application
Resource MRs required to reproduce the bug
apiVersion: applications.azuread.upbound.io/v1beta1
kind: Application
metadata:
  name: test-12a3234
spec:
  forProvider:
    displayName: toto-eso
    owners:
      - xxxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxxx6e

Steps to Reproduce
create cred with following command like the readme explains

az ad sp create-for-rbac --sdk-auth --role Owner --scopes /subscriptions/$SUB_ID > auth.json
k apply -f mr-app.yaml

What happened?
async create failed: failed to create the resource: [{0 Could not create
      application ApplicationsClient.BaseClient.Post(): unexpected status 403 with
      OData error: Authorization_RequestDenied: Insufficient privileges to complete
      the operation. []}]

i have to manually add these permission to allow App to be created

https://learn.microsoft.com/fr-fr/graph/permissions-reference#applicationreadwriteownedby

but i do this with the same TF Provider Version
i got not issue (i use the same login on both side)

terraform {
 required_providers {
   azurerm = {
     source  = "hashicorp/azurerm"
     version = "3.52.0"
   }
   azuread = {
     source  = "hashicorp/azuread"
     version = "2.47.0"
   }
 }
}

provider "azurerm" {
 features {}
}

provider "azuread" {

}

data "azurerm_client_config" "current" {}
resource "azuread_application" "eso-app" {
 display_name = "External Sectret Demo App"
 owners       = [data.azurerm_client_config.current.object_id]
 #logo_image       = filebase64("/path/to/logo.png")
}
output "objectid" {
 value = data.azurerm_client_config.current.object_id
}

slack-discussion
Relevant Error Output Snippet
No response
Crossplane Version
1.16
Provider Version
1.3.0
Kubernetes Version
1.28
Kubernetes Distribution
OVH
Additional Info
No response

encountered the same issue for applications and groups. Adding the permissions Application.ReadWrite.All, User.Read.All for the service principal resolved it. Provider Version 1.4.0 does not provide an error message for this bug.


Multiecho: accept SDC warpings in T2* calculation

Same way it accepts the HMC xforms, enable unwarping of EPIs if it was calculated

Moving the conversation from #1021 here -- cc @oesteban @prantikk:
I think the tension I see in this is using the T2* map for coregistration vs. using it for subsequent Kappa/Rho calculation. Right now the T2* map isn't even connected to a data sink, but especially if it's going to become the default with multi-echo data (as @oesteban suggested), I think it makes sense to generate it by default, pipe it to the data sink, and then only pass it along for SDC if it's used in coregistration.
That being said, I think two of the steps @oesteban proposed (removing negative values from the echos and calculating T2* in a mask) are relatively non-controversial (not sure if you agree, @prantikk). The idea of clipping the histogram could be a bit more problematic down the line -- what if we added that as a function node immediately before the SDC correction ?
The one concern this brings up is dealing with normalization differences between the echos and the T2* map. Could we just apply those parameters before passing to the data sink ? WDYT ?

Currently, STC, HMC, and SDC are applied before T2* calculation, after which coregistration to T1 and warping to standard are done. T2*-based coregistration was removed in #2109.
Is the goal here to apply SDC after T2* calculation instead of before? I know that's a concern for some folks, although I doubt that SDC would have a major impact on T2* maps and optimally combined time series.

I think the original goal was to apply after, but our understanding has evolved since then and I agree that it should be fine as-is.
I'm fine with closing this !


Fix movefile-method.md formatting

Fix movefile-method.md formatting

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

VBA/Language-Reference-VBA/articles/movefile-method.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


travis-ci.com

@geek1011 GitHub is deprecating the old services integration and telling users to switch to "GitHub Apps".
I "installed" Travis' GitHub App (travis-ci.com vs travis-ci.org) into our org and whitelisted some repositories.
Now I need to ask Travis' support to migrate these repos from travis-ci.org to travis-ci.com (this whole migration thing is a bit confusing, I may be getting something wrong).
Migration appears to be faster if we don't retain build history (source).
Is it OK if I include jiup-go in the list of repos to migrate to their new infrastructure?

Yep. I've been meaning to do this for a while, but never got around to it. I think I'll need to set GITHUB_TOKEN again after the migration.

It should be done. I see GITHUB_TOKEN is there, the migration tool seems to have ported it over.
Did it have a cron job? The migration process doesn't transfer that.

Yep. I've put the cron job back. Thanks!


for dataset

can you give me dataset for Adobe Deep Image Matting Dataset<EMAIL_ADDRESS>thanks

I want to get the dataset for Adobe Deep Image Matting Dataset too<EMAIL_ADDRESS>

I am not authorized to redistribute it, please contact the authors.


fix backupmeta Invalid data length complained by thrift and a little refactoring

this pr includes:

fix Invalid data length complained by fbthrift.
extract put/get backupmeta to a common function.
add an option for allowing standalone meta service in restore cmd for testing scenes.

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


refactor tutorials a little to make them more consistent with one another

Also shorten a little where possible

@arikalon1 please review. We still need more work on the "Alert Enrichment" tutorial which currently isn't really a tutorial but is reference docs instead.
@Sheeproid tagging you too as I modified the tutorial you wrote a little. Feel free to comment.

nice.
Left one small comment


New release that doesn't require git

Hi @CharlesMangwa
You've fixed package.json so that simple-markdown is installed using https://... instead of git://..., but it hasn't been released yet, which requires my CI to have git installed to use your package. I'm using it as a transitive dependency through Storybooks ondevice-notes.
Could you please release a new version with this fix?
Best regards
Jens

by the way, if you are using yarn it seems adding something like the following to your package.json does the trick:
"resolutions": {
    "@storybook/addon-ondevice-notes/react-native-simple-markdown": "github:CharlesMangwa/react-native-simple-markdown#pull/112/head"
  }

@CharlesMangwa pls merge pr: https://github.com/CharlesMangwa/react-native-simple-markdown/pull/112


Motion Plus Slow / Fast Mode

Hello !
I'm trying to use xwiimote to get orientation angles from gyro of RVL-CNT-01-TR wiimotes.
Now that I'm a little more documented on wiimote hardwares and protocols, i've seen MotionPlus got  two "speed" modes (slow or fast). If I understand it correctly, the mode is auto-switched by hardware according to velocity of user movements and it modifies the scale of the rate output.
I've seen some old libraries reading the speed mode info on the fly and providing normalisation to ease values integration, but I didn't find references to these modes in API or core code of xwiimote, so I don't know if and how it's handled.
Do you have some knowledge on it ?

Your kernel normalizes the data already. See here:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/hid/hid-wiimote-modules.c#n2034

Thank You David, so I understand I don't have to care about.

Yes, if the code is correct, the data should already be returned in normalized form.


Added ability to include relations allowed fields

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


How to use datetime picker in aimeos extension

Hello everyone!
I am first in aimeos and started learning.
Now I have created new extension to CRUD customer for only test and customers have fields such as name, sex and birthday.
But I don't know how to create datetime picker for birthday field of my customer
I hope to make it like this

Current code of item.php of mycustomer ----------------------------------------------------

current creat view------------------------------------------------------------------------------

In my think, I need to add some js code but I don't know how to do.
Please help me

Please ask for help in the Aimeos forum (https://aimeos.org/help) as the issue tracker is only for bugs and feature requests.


Use channelName if title empty on support/tipping modal

PR Checklist

Please check all that apply to this PR using "x":

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[x] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Fixes
Issue Number: #3466
What is the current behavior?
What is the new behavior?
Other information

Fixed with #4091


[BEAM-1803] Fixing bug in Metrics filters. Adding unittests.

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

r: @aviemzur ¿Can you take a look? It should be fairly simple.
Thanks.

Coverage increased (+0.08%) to 70.186% when pulling a53a0f38206f6a93f5e9620773d98c25b9a630c0 on pabloem:fix-filtering into 804440826e9a5dabfaa772dca900feaf3dd6e3b7 on apache:master.

LGTM


Masking Models

Compute the marginal of a MoMo on a set of points.

Computes marginal on MoMoBasic and MoMoExpress.
Can be used to mask a model.

How to create a masked model:

Define a mask on the reference mesh.
Create the masked model.
Snippet:

    val model: MoMo = ???
    val fix = Point(-71.2, 12.8, 39.9)
    val op: MeshCompactifier = model.referenceMesh.operations.maskSpatially(pt => (pt-fix).norm > 100.0)
    val maskedMesh = op.transformedMesh
    val oldIds = maskedMesh.triangulation.pointIds.map(op.pointBackMap)
    val maskedMoMo = model.marginal(oldIds)

I think the functionality is useful for some applications. But the interpretability of the model is somehow changed. An orthogonal basis will most probably no longer be orthogonal if we just "mask" the model. Could this be problematic for some of the model operations? If yes, can we find a way to communicate this to our software users? Having not thought through all implications of this model transformation I find it dangerous to provide this in our core library. But maybe my concerns can be disproven.

Yes orthogonality is lost:
    def orthognality[A](gp: PancakeDLRGP[_3D, UnstructuredPointsDomain[_3D], A]) = {
      def getCol(c: Int) = gp.basisMatrix(::, c)
      getCol(0).dot(getCol(1))
    }
    println(orthognality(model.neutralModel.shape))
    println(orthognality(maskedMoMo.neutralModel.shape))

Output:

original model: 7.235728834674438E-11
masked model: 0.021161337168098197

Do you think that orthogonality is a necessary requirement for the MoMo or is it only required for the DiscreteLowRankGaussianProcess?

We should check the interface of momo if any function assumes orthogonality. For example, is
def project(sample: VertexColorMesh3D)
still well defined?

@Ghazi-Bouabene  thanks! This makes the whole thing much simpler. I updated the PR accordingly.
Will the orthogonality issue be solved by using the marginal?

Integrated marginal into MoMo.


Add tracking for compilation times of important circuits in CI

We currently have no good way to quantitatively track whether a PR will cause a regression in compilation time for important user circuits. A recent example of this coming back to bite us is in the bloblib circuits where compilation times jumped from 10 to 90mins.
Similar to run-external-checks, we should add a workflow to CI which will compile a number of protocol circuits (the rollup circuits in particular) and measure the time taken to do so. This can then:

Print a comment to the PR showing the time and highlight any regressions (would require a modification of https://github.com/noir-lang/noir-bench-report to support this new metric)
Define a hard limit on compilation times which will automatically block the PR if exceeded.

As a first cut we can just use time nargo compile --force to extract the compilation time so this is just a devops issue and shouldn't need modifying the compiler.

If we're compiling many protocol circuits as part of CI we may want to split this over multiple runners and then merge the reports together after the fact. This can be done later though to avoid premature optimization.


activate CI for the repo

I tried to fix the issue but, to test this, we need tests and a released version.
Also the dependency is needed.
        <dependency>
            <groupId>org.apache.maven.plugin-testing</groupId>
            <artifactId>maven-plugin-testing-harness</artifactId>
            <version>3.3.0</version>
            <scope>test</scope>
        </dependency>

See this guide(https://maven.apache.org/plugin-testing/maven-plugin-testing-harness/getting-started/index.html). Also, for some reason this project uses junit 3.8. To my best knowledge IntelliJ simply fails to run junit 3 tests. Before starting with mojo testing, I would recommend adding renovate or update dependencies first.

Ci works and is active


use a fast and workaround way to relieve the problem of Dropping Table/Index, etc, temporarily

Now when the user drops table/index, etc, the internal DeleteRange command will block the apply worker and cause a high latency for the cluster.
Although we have #2960 to solve the problem, this PR may still be tested for a long time. Now we have been OnCall for this problem many times and I think we need to find an easy but not perfect way to work around it.
We know that the DeleteRange contains 1. DeleteFilesInRanges + 2. Scan region + Delete left data, the step 2 will block the apply worker, so we can remove it, but this may cause the problem that we will leave some stale data in the cluster, so we need to clear this stale data later. some ways:

Scan whole TiKV regions and find if there is a region contains the stale data, clean up. We can decode the region range, get the table ID and find the table still exists or not, or some other optimizations.
Record all DDL operations in history, find them directly and do the cleanup work.

Of course, these ways are not perfect, but here we need to relieve the problem ASAP.
/cc @zhangjinpeng1987 @BusyJay

Previously i implemented a branch to use a individual thread to handle DeleteRange(Cleanup stale ranges) asynchronously.

Apply worker send a cleanup task to a persistent task queue after step 1. DeleteFilesInRanges.
Cleanup worker consume cleanup stale task, handle step 2 asynchronously.
I think this is more clear and simple, i prefer this solution.

Cool, we can use this as a workaround way now.

After discussing with @BusyJay , we came to a better solution:

TiKV only do DeleteFilesInRanges that mentioned above.
Instead of delete the key range from system table mysql.gc_delete_range after DeleteRange requests have been done, we mark these key ranges as done status.
After TiKV has fix the issue of applying cleanup a range will block apply worker(apply pool), resume the original logic of step 2.


Export wizards: check for extant files and display overwrite warning rather than silently overwriting

Fixed in new ngscopeclient export filters..


Migration + Eloquent Model naming convention

Hi,
By default php artisan make:model Xxx -m for Foo and Bar will create both the migration and the model files:

xxxx_xx_xx_xxxxxx_create_foos_table.php
xxxx_xx_xx_xxxxxx_create_bars_table.php
App\Foo.php
App\Bar.php

Apply the same logic for a pivot table, php artisan make:model FooBar -m will create both the migration and the model files. We can delete the model file. (The purpose of using the php artisan command was just to see how Laravel handles the naming)

xxxx_xx_xx_xxxxxx_create_foo_bars_table.php

Now, into both Foo & Bar models if we define a belongsToMany relationship it will assume the pivot table name is singular foo_bar and not foo_bars as stated by default into the migration file created by the artisan command.
Of course, into the relationship definition, one can define the table name.
Why such a behaviour? Is it wanted?
If by default the relation expect a singular table name why the artisan command creates a plural one? Vice-versa.

Because the command has no clue that you are creating a pivot table. The command you are running is asking it to make a model and a migration, and therefore is doing what it is supposed to be doing.

@lagbox you didn't read properly my comment. I know that. I'm talking about default behaviour:

If by default the relation expect a singular table name why the artisan command creates a plural one? Vice-versa.

Again ... because it doesn't know you are creating a pivot table. Therefore its following the normal naming convention for models and tables. It doesn't know about the relationship you intend on creating or that you are intending on creating a pivot table.

I'm not talking to know or to not know about a pivot table. Artisan command makes the table name foo_bars plural and the relation method expect the table name to be singular foo_bar by default. It's more a question about naming convention and why, out of the box, it won't work (event if as you mention there are plenty of ways to make it works...)

I have already explained why twice.

So let me modify my question:
Don't you think the relation method default behavior should expect a plural name? If no, can you please explain as I'm interested to understand the why of the choice.
Thank for explaining "why" twice.

On May 16, 2016, at 11:46 AM, lagbox<EMAIL_ADDRESS>wrote:
I have already explained why twice.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub


not able to create a project if npm is not installed

Version
3.3.0
Environment info
Environment Info:

  System:
    OS: macOS High Sierra 10.13.6
    CPU: (4) x64 Intel(R) Core(TM) i7-2620M CPU @ 2.70GHz
  Binaries:
    Node: 8.15.0 - /opt/local/bin/node
    Yarn: 1.12.3 - /opt/local/bin/yarn
    npm: Not Found
  Browsers:
    Chrome: 72.0.3626.81
    Firefox: 64.0.2
    Safari: 12.0.3
  npmGlobalPackages:
    @vue/cli: Not Found

(@vue/cli is installed globally via yarn.
Steps to reproduce
vue create myproject
What is expected?
it to work toward creating the project
What is actually happening?
 ERROR  Error: spawn npm ENOENT
Error: spawn npm ENOENT
    at Process.ChildProcess._handle.onexit (internal/child_process.js:190:19)
    at onErrorNT (internal/child_process.js:362:16)
    at _combinedTickCallback (internal/process/next_tick.js:139:11)
    at process._tickCallback (internal/process/next_tick.js:181:9)
    at Function.Module.runMain (module.js:696:11)
    at startup (bootstrap_node.js:204:16)
    at bootstrap_node.js:625:3

@urkle do you have the code for me? Im having the same issue here

@yooouuri this is fixed in recent versions of Vue cli (at least for create)

@urkle
Sadly it is not for vue add ... and vue upgrade ....
I am using a monorepo with (for now) one Vue project.
When I run for example vue add vuex in the root of my project, it tries to add it to the root. Ofc I don't want that. When I run vue add vuex inside the Vue project package (packages/vue-project), it forces me to use npm...

I've opened https://github.com/vuejs/vue-cli/pull/5150 to fix this issue.

@sodatea
Thanks, would be nice to see this issue to be fixed.
Removed npm, added the following script (as npm) to my path.
#/bin/sh

args=("$@")

echo $args

if [ "${args[0]}" == "install" ]; then
    echo "lets fake the npm install"

    # where args[3] is for example @vue/cli-plugin-eslint and args4 is -D
    if [ "${args[3]:0:4}" == "@vue" ]; then
        yarn add ${args[3]} ${args[4]}
    # if we are not installing any package, install all depencencies
    else
        yarn install
    fi
fi


Feature was removed on the current commit.

@HackedByChinese, any reason why the code from this pull
github.com/HackedByChinese/ng-idle/pull/93 was removed on the last commit?

@HackedByChinese, You moved this to the TitleProvider. Sorry for this issue.


⚠ Bofrost.fr has degraded performance

In 14fcbe4, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1121 ms

Resolved: Bofrost.fr performance has improved in 9138f54.


force = true?

Hi, just wondering why you're setting force to true for all tasks.  Is this necessary?  It seems like it has the potential to be dangerous and I wonder if there was a reason to do that rather than parameterize it.

Just got bitten bad by this horrible choice today. I regret ever deciding to go the plugin route instead of chaining via && in bash.


In Android N, from FCM OnMessageReceived, OkHttp network call timeout when app is in background or killed. Need help or some guidance to fix this.

In Android N (7.0), from FCM OnMessageReceived, OkHttp network call connection timeout when app is in background or killed. Need help or some guidance to fix this.
This work in Android 6.0 and Android O (8.0).
Regards,
Aditya

Looks like my issue is fixed. Starting Android 6 (API Level 23), the GCM/FCM messages with normal priority (default) will not have access to network in Idle or Doze mode. When I set the message priority to "high" in GCM/FCM messages payload, my OnMessageReceived handler was able to access the network properly. But I'm puzzled on how Android 6.0 and Android O (8.0) were able to access the network while in Idle/Doze mode. Anyhow hope this will help some one out there with similar issue.


Don't break if A extensions are not enabled

Addon breaks in Ember 3.4 if Array extensions are disabled

Thanks for the patch! I need to fix CI but wants it’s back this looks good to merge.

This is really a quick patch. Optimally, the addon shouldn't use or rely on those array prototype extensions.
sortBy('field') is just sort(x => x.field) in native javascript.


[HOTFIX] Only update stat and clear contents when old stat is newer than current stat.

:fire: Potentially fixes #11826 :fire:
There is a long running issue with Brackets where the async stat implementation for fetching the timestamp (stat._mtime) can cause rare :racehorse: race conditions :racehorse:. See this issue: https://github.com/adobe/brackets/issues/295
This is very rare and usually doesn't matter much. But it does matter when FileSystem.js does this:
      // Update stat and clear contents, but only if out of date
      if (!(stat && oldStat && stat.mtime.getTime() === oldStat.mtime.getTime())) {
         entry._clearCachedData();
         entry._stat = stat;
         this._fireChangeEvent(entry);
      }

Due to the race condition stat.mtime is sometimes older than oldStat.mtime, causing a change event to happen and cache get cleared, nullifying the history ->  causing #11826.
Q & A
Q: Brackets has been like that for a long time. Why did this start occurring in 1.5.?
A: I only got speculation, but as it is a race condition, I guess increased code base plays an unfortunate part here: some additional file system operations or anything else made the race condition much more reoccurring.
Q: Does this really fix #11826
A: I... don't know for sure. There's no steps to reproduce #11826 with even 1% success rate
Q: Why do you think it fixes #11826
A: It fixes the issue with my corrupt-brackets-history extension: I tried to save 10000 times in a row and history was kept intact. Without the fix the issue usually occurs before the 50th save.
Q: Does this have other side-effects
A: Not sure. Most likely not, but I am a bit worried that it might have unwanted effects on some external changes. Obviously it should be better to fix the  race condition itself, but this is just a hotfix.
Q: Why is the history getting reseted
A: The change event getting erroneously called eventually leads to this chain:
FileSyncManager#reloadChangedDocs
-> FileSyncManager#reloadDoc
-> Document#refreshText
-> Editor#resetText

Editor#resetText which calls:
 // Make sure we can't undo back to the empty state before setValue(), and mark
// the document clean.
this._codeMirror.clearHistory();
this._codeMirror.markClean();

@petetnt I have been racking my brain and I can't think of a reason why we would want to even consider a scenario where the old modified date is newer than the new modified date.
The comment before the code: // Update stat and clear contents, but only if out of date seems to indicate that we should only be testing for a single scenario: new modified date is newer than old modified date.
Based on these two arguments, your fix seems solid. But I have the same reservations as you have about changing code at such a low level.
Maybe someone else who knows the file system code better than I do has some insight as well.

:fire_engine:
@swmitra @nethip

the fix looks safe[though a bit wierd] to me.
@petetnt Could you target the change to the release branch too.
We are going to refresh silently the release builds of 1.6 with this change.

@abose See #12195 which is targeted against release.

Merging to master as the release discussion is still going on for 1.6.
This will give enough testing in the brackets-dev groups.
Merging.

@petetnt Can't thank you enough for this fix :smiley:

@abose :joy_cat: Thanks!
:birthday:


Save directory location

Figure out how to save to external storage with Android's new security features

see #216


Medical scanner will not delete bodies anymore

About the PR

Fixed bug that deletes bodies from medical scanner.
Technical details

I just removed UpdateAppearance(uid, scannerComponent); in medical scanner EjectBody method. Im not sure that its best way to fix it, but looks like better than keep it as it is.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

fix: Medical scanner will not delete bodies anymore.

https://github.com/space-wizards/space-station-14/pull/21071 this one working too.

Fixed with https://github.com/space-wizards/space-station-14/pull/21071


Fix osx CI build

Part of https://github.com/gtk-rs/gdk/pull/326

I wonder why the diff isn't happy... To be fixed in another PR I guess.
cc @EPashkin @sdroege

Its same as with gdk: gir submodule updated in #48 without regen


Error generating some QRCodes (Uppercase issue?)

sometimes creating qr codes will throw this error (running through ghostscript)
Unrecoverable error: typecheck in moveto
Operand stack:
--nostringval--<PHONE_NUMBER>1
Seems to be when there is a number of uppercase characters, I've not been able to track down the exact cause. however...
this works ok
350 -80 moveto (testing) (eclevel=Q) /qrcode /uk.co.terryburton.bwipp findresource exec
as does this
350 -80 moveto (TEsting) (eclevel=Q) /qrcode /uk.co.terryburton.bwipp findresource exec
while this brings up the error
350 -80 moveto (TESting) (eclevel=Q) /qrcode /uk.co.terryburton.bwipp findresource exec
And this also errors
350 -80 moveto (TESTING) (eclevel=Q) /qrcode /uk.co.terryburton.bwipp findresource exec
I'm using the latest version of the qrcode.ps from the repository.

These all seem to work fine using the code from before the update in July 2014.

Very strange, I'm unable to reproduce. Can you please send me a complete PS file that errors and let me know the version of GhostScript.
Also, are you able to get the online generator to error? It uses the latest BWIPP https://www.terryburton.co.uk/barcodewriter/generator/
It looks like something is getting screwed up in your environment. The only moveto operators are the initial one that you supply when invoking the resource, then two instances in renmatrix: the first of which is initialised using constants ("0 0 moveto") and the second takes "x" as the first ("--nostringval--") input which is defined from an earlier calculation "dup pixx mod /x exch def".
Does your preceding code modify the mod function in some way?

Not as far as I know but there is a fair amount of prolog functions so it's possible (and since you can't reproduce I guess probable). I've emailed you the postscript. If I change the qrcode encoder over to the pre July 2014 version then both work ok.

TL;DR
Change:
350 -180 moveto (TESTING) () /qrcode /uk.co.terryburton.bwipp findresource exec

To this:
countdictstack array dictstack dup length 3 sub 3 exch getinterval cleardictstack
350 -180 moveto (TESTING) () /qrcode /uk.co.terryburton.bwipp findresource exec
{begin} forall

Long Version
The problem is that your prolog is redefining "m" to moveto which is getting immediately bound into the encA (upper-case alphanumeric encoding) procedure which is used only when it is most efficient to do so:
/encA {
    /in exch def
    fnc1first {  % FNC1 as %
        /in [ in { dup -1 eq {pop 37} if } forall ] def
    } if
    /out in length 11 mul 2 idiv 1 add string def
    /k 0 def /m 0 def {
        k in length eq {exit} if
        k in length 1 sub lt {
            charvals in k get get 45 mul charvals in k 1 add get get add 11 tobin
            /k k 2 add def
        } {  % Final character
            charvals in k get get 6 tobin
            /k k 1 add def
        } ifelse
        dup out exch m exch putinterval
        length m add /m exch def
    } loop
    out 0 m getinterval    <=== This m is being replaced with "moveto" at bind time
} bind def

You could avoid this problem at the cost of a bit of performance by removing the bind at the end of the definition. But a better way is to control the context in which the code is executed by using a wrapper around the invocation as demonstrated above.
I'll update the documentation with this tip.

Ahh that explains it, perfect thanks. I take it that wrapper would work around calls to any of your barcode routines and would be a good precaution when mixing in with other prolog bits? I'll test this solution now.

It ought to work fine however it isn't well field tested and the cleardictstack operator is (strictly speaking) forbidden in EPS files - although this use leaves the state intact so many not matter in practise.

Field testing commencing! Thanks again.


Adding performance prep script to customize sysctl and i/o disk scheduler

…

Bash scripts reside under remote-scripts/ica folder, so please move this under that path.
Also, the Apache license must be added.


Moving discriminant parameters to component base

By moving the discriminant to the base type we ensure that the component model always matches the implementation. When the discriminant is reflected in the implementation, it was up to the user to keep the model in sync with the code, which is far from ideal. This commit fixes that issue.

Abandoning. This does not quite work with the way Ada handles inherited discriminants, ie.
error: inherited discriminant not allowed here (RM 3.8 (12), 3.8.1 (6))

Abandoning. This does not quite work with the way Ada handles inherited discriminants, ie.
error: inherited discriminant not allowed here (RM 3.8 (12), 3.8.1 (6))


IntelliSense of HTML is not displayed in Korean.

VSCode Version: Code 1.11.2 (6eaebe3b9c70406d67c97779468c324a7a95db0e, 2017-04-13T08:03:11.395Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions: none

Steps to Reproduce:
Almost stuffs are displayed in Korean becuase dispaly language is already configured to "ko(Korean)". But intelliSense of HTML is not.

Mouse over HTML tag.
The information popped up under the cursor is not displayed in Korean.

The HTML language server is missing translations.

@dbaeumer What do have to do to add the vscode-html-languageservice module to the list of parts to be translated?

@michelkaporin can you please help @aeschli

@aeschli for the onboarding to Transifex you should introduce three gulp tasks for pushing, pulling and importing from Transifex with transifexExtensionName variable to be set to vscode-html-languageservice. See gulp file in vscode-node-debug as an example (https://github.com/Microsoft/vscode-node-debug/blob/master/gulpfile.js#L145.
You need: transifex-push, transifex-pull, i18n-import tasks with all dependency variables to be set. vscode-nls-dev npm package is needed to use the methods defined in these tasks.
To verify whether you are set up, you can call transifex-push initially yourself, checking if vscode-html-languageservice resource appeared in Transifex under vscode-extensions project. Going further, translation team (or build agent) will be doing that for you.

IntelliSense of HTML is still not displayed in Korean.  I want to introduce the vscode to Korean newbie but sorry to English-displayed intellisense window.

Sorry, there are currently no plans to add translations to the HTML tag descriptions. Please add your vote (:thumbsup:) to the description.

Hello,
From my brief investigation on this issue, ec9571d7953d52e4eb8880cd1a725b4261fc6c59 had languages folder on i18n/kor/src/vs/ but 3af180691c618c2238a7b8081b7e3f6e56d5b824 removed translations on the languages folder since languages folder was not synced with Transifex.
It seems that there has been no drastic source changes on vscode for i18n support. The simple way to support translations for intelliSense of HTML would be to restore languages folder I think. Thoughts @aeschli  and @michelkaporin ? Or adding language translations to Transifex would be an alternative way to support translations for intelliSense of HTML? In this case, talking to Microsoft language team would be a faster way?
With many thanks,

Korean Microsoft MVP (Cloud & Datacenter Management).

Restoring the i18n files is not an option since these files are machine generated and their source should live in a translation database (e.g. Transifex). However If we decide to translate the HTML language server these files could be used as a seed for the translation database.

It is good idea.
And i want other Language(CSS,Javascript) also applied to.

@aeschli Current 45 thumbs and 4 hearts might not be sufficient? :) Hope that this issue would have higher priorities in vscode development team soon. (By the way, I really appreciate all kind replies and votes!)


UI Multilanguage

In the frontend it should be possible to select multiple languages

Implemented with version 0.0.2


BUG: optimize: revert changes to bfgs in gh-7165

This reverts commit 56e42a5eef7778ff33345d732a0421b81fda7dbc, reversing
changes made to c324145ac07ccfde41b209496f10abdb0cb08a59.
Further testing showed some issues (gh-7959):

The "if k == 0" hessian scaling on first step does not necessarily
play along well with the initial step scaling done using the line
search. Whether it really is incorrect is unclear, but this would
require further investigation looking at the test cases in
statsmodels.

If this condition is removed, it appears the new formula may lose
precision when yk_sk is large, indicated by failures in
TestOptimizeSimple.test_initial_step_scaling.

Addressing the latter point would require more work, so it's simpler as
a first step to revert the change.
Closes gh-7959

Some test failures but they are due to gh-7961 and new mpmath release today.

Going to merge this myself soon, so the backport makes rc2...


Example

Please add PowerAutomate tab example.

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

ID: 523b3456-9ec1-f405-a9f6-cf1e15b5a0b7
Version Independent ID: a534e0cf-9e6d-5d85-0915-8c817569a989
Content: Delete a user - Microsoft Graph API - Microsoft Graph v1.0
Content Source: api-reference/v1.0/api/user-delete.md
Product: users
Technology: microsoft-graph
GitHub Login: @yyuank
Microsoft Alias: MSGraphDocsVteam

Thank you @GlennQLS for this request. Power Automate isn't an SDK so we might now be able to add the tab as requested.  You can explore the Microsoft 365 & Power Platform Community to find samples for Power Automate.


[JAX] Add back the xla deterministic flag

Description
There was a segfault issue on XLA, so we temporary disabled the deterministic flag in https://github.com/NVIDIA/TransformerEngine/pull/1262. XLA recently fixed the issue in https://github.com/openxla/xla/pull/18766, so it is good to add back that flag now.
Fixes # (issue)
Type of change

[ ] Documentation change (change only to the documentation, either a fix or a new content)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Infra/Build change
[ ] Code refractor

Changes
Add --xla_gpu_deterministic_ops to have run-to-run deterministic to have the stable CI results
Checklist:

[x] I have read and followed the contributing guidelines
[x] The functionality is complete
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] 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

/te-ci jax

/te-ci jax

/te-ci jax L1

Pipeline #20118954, all tests passed.


[Bug] Additional Notifications when an app is deleted

Expected
When an application is deleted from the homepage, there should be only 1 notification which informs the user that the application has been successfully deleted.
Observation
Instead we see 2 additional notifications which do not make sense.
Screenshot

Hey @prapullac
Can I take a look at this ? This is when we see error notifications after deleting an application. Right ?

@jyash97 Yes you are right. I've added more details to the issue description. Have assigned you the issue as well.

@mohanarpit
I have found the root cause for the issue and have 2 approaches that may require inputs from the team. I am adding all the details below:
Current Issue
The above issue happens because we send an update API request just after deleting; Triggered due to the onBlur event of EditableText present here.
Update API is also called every time we close the context menu. ( Even if we don't change the app name ). Check the below video for more details:
https://www.loom.com/share/a225b18ba0984563bf575de5c6f6416c
Solution
We need to prevent update API on blur event which is called after a successful deletion of the application.

Trigger update API in onTextChanged which will be on enter press. The current behavior is as we type on the input and close the context menu the application name gets updated. [ This changes current behavior ]

Issue in EditableText onTextChanged prop
onTextChanged prop in EditableText is not called as we type the input. It is triggered only when we press enter as the callback is linked in onConfirm method here.
Solution
Call onTextChanged as the input changes. We can add debouncing so that if an API is linked with onTextChanged we don't end up in calling the API as user types. [This doesn't change the current behavior ]

According to me the latter option is a good approach, as it makes the component ( EditableText ) more flexible.
Let me know what you guys think on this.

@jyash97 Thanks a lot for the detailed analysis and suggestions on the code change.
@satbir121 Can you please comment on the above suggestions?

@satbir121 Bumping this up for you. Can you please check if the above implementation is valid?

Hey @mohanarpit 👋   was this already fixed in the PR linked?

Verified the issue


[tesseract] Fix non-Windows build

Use official patch to fix the non-Windows build error:
In file included from /tmp/vcpkg/buildtrees/tesseract/src/4.1.1-1402ed03b0.clean/src/api/capi.h:27,
                 from /tmp/vcpkg/buildtrees/tesseract/src/4.1.1-1402ed03b0.clean/src/api/capi.cpp:21:
/tmp/vcpkg/buildtrees/tesseract/src/4.1.1-1402ed03b0.clean/src/ccutil/ocrclass.h:117:18: error: field 'end_time' has incomplete type 'timeval'
  117 |   struct timeval end_time;
      |                  ^~~~~~~~
/tmp/vcpkg/buildtrees/tesseract/src/4.1.1-1402ed03b0.clean/src/ccutil/ocrclass.h:117:10: note: forward declaration of 'struct timeval'
  117 |   struct timeval end_time;
      |          ^~~~~~~
/tmp/vcpkg/buildtrees/tesseract/src/4.1.1-1402ed03b0.clean/src/ccutil/ocrclass.h: In member function 'bool ETEXT_DESC::deadline_exceeded() const':
/tmp/vcpkg/buildtrees/tesseract/src/4.1.1-1402ed03b0.clean/src/ccutil/ocrclass.h:142:20: error: aggregate 'timeval now' has incomplete type and cannot be defined
  142 |     struct timeval now;
      |                    ^~~
...

Fixes #13622.

Regression will be fixed by https://github.com/microsoft/vcpkg/pull/20439.

Thanks for the fix!


Tighten up the CSS a little

Here's the CSS I'm using ala myspace (great feature!) to create a more mastondon-like UI. The maintainer(s) may choose to adopt some of it into the core app, or perhaps it will just sit here for users to snaffle for their own use.
Attached are some screenshots of how it looks.
body {
    font-size: 16px;
    font-family: "Atkinson HyperLegible", system-ui, -apple-system, BlinkMacSystemFont, Segoe UI, Oxygen, Ubuntu, Cantarell, Fira Sans, Droid Sans, Helvetica Neue, sans-serif;
    line-height: 1.35;
}

#main-content {
    padding: 0;
}

.post {
    box-shadow: none;
    margin-bottom: 0;
    border-bottom: solid 1px var(--color-text-dull);
}

.post .content {
    max-width: 665px;
}

.post .content a {
    word-break: inherit;
}

.boost-banner {
    background-color: var(--color-bg-box);
    padding-top: 10px;
    padding-left: 14px;
    padding-right: 12px;
    padding-bottom: 0;
}

form .buttons {
    padding-right: 15px;
}

form label {
    font-size: 115%;
    text-transform: inherit;
}

fieldset legend {
    left: 0;
}

On mobile:

Desktop:

Thanks for leaving this here, but I'm not interested in reducing the spacing much further at this point - at some point, I want to revisit the whole layout and see if we can get it tighter without losing readability, but right now it's at a sweet spot for me personally (and unfortunately, that is the bar by which I currently judge the layout!)

Totally fair enough.
Maybe one day there will be a takahe css theme garden for sharing snippets like this...


Urgent need for another download link

Could you provide another download link? The current link always fails after downloading for serval hours.

I have already changed three servers to download this data, and it always fails.

I started hosting the data on Kaggle now and updated the data link in readme. This should allow you to download individual files. Hope this helps.


API document not available

Hi Team,
Defined API document not available. Could you please check
URL - https://docs.microsoft.com/en-us/java/api/overview/azure/servicebus/clientlibrary
Regards,
Abhishek Kumar

Hi @abhikt48, @yvgopal will follow up, but in the mean time you can find service bus quick start guides in all languages, including java, at https://docs.microsoft.com/en-us/azure/service-bus-messaging/

Hey @abhikt48
Can you please share where you found this link?
I do see the "client library" linked, but as a section under parent page. See https://docs.microsoft.com/en-us/java/api/overview/azure/servicebus#client-library

@spelluru Do you know where https://docs.microsoft.com/en-us/java/api/overview/azure/servicebus/clientlibrary might be linked from?

@ramya-rao-a  - It's not linked from any of the articles in the folder: https://github.com/MicrosoftDocs/azure-docs/tree/master/articles/service-bus-messaging.
@abhikt48 - could you let us know if you are using a direct link to this page or tried to navigate to this page from another page? Thanks

@ramya-rao-a  i am sorry not remember exact location now.  i think I found this on Git ServiceBus location.


Add link in grafana.com plugins to point back to github repo

First off, congratulations on a great idea and interesting implementation. I look forwarding to testing it.
Please add link in grafana.com plugins to point back to the github repo.

Thanks for the feedback. There is a link at the bottom of https://grafana.com/grafana/plugins/andrewbmchugh-flow-panel/ under 'Want to Make Changes', but the name of 'Grafana Getting Started' is a bit confusing. I'll add a more obvious link back in the next release and close out the issue when the version is pushed.

I usually look for the link back to the github repo on the sidebar: Ressources and Projet Site. Just under Developer.

I usually look for the link back to the github repo on the sidebar: Ressources and Projet Site. Just under Developer.

thanks, I'll get that added

Links now added as 1.6.0. https://grafana.com/grafana/plugins/andrewbmchugh-flow-panel/
Thanks for the tip.


feat: support MikroORM v6

BREAKING CHANGE:
MikroORM v6 and Nest.js v10 are now required. Also Node.js 18.12 is required.

Note for myself so I don't forget, we also need to rename the injection tokens for repositories (https://github.com/mikro-orm/nestjs/discussions/106) for v6.

something that needs to be considered for this PR is that MikroORM via the re-export changes could no longer viable as a DI token due to it potentially being another type e.g. MikroORM export from @mikro-orm/postgresql is actually PostgreSqlMikroORM.
import { MikroORM } from '@mikro-orm/postgresql'

@Injectable()
export class TestService {
  constructor(private readonly orm: MikroORM) {}
}

It would fail with the error Please make sure that the argument PostgreSqlMikroORM at index [0] is available in the RootTestModule context.
You can work around this via one of the following:

Keep MikroORM type import from @mikro-orm/core
Give MikroORM a context name and use @InjectMikroORM() decorator for injection


Add deployment debug setting

@hovsepm @markcowl @stankovski Can one of you please review and merge this PR? Thanks.


#12919: Cleanup multigammaln_bw, lgamma_bw

This is a sub branch PR that is created to simplify the work on branch virdhatchani/set_2_clean as

There is a lot of code line changes involved
CI cannot be re-run if I push a new commit into main branch

https://github.com/tenstorrent/tt-metal/actions/runs/11049296542 - passed, Hence merging


[5.8]Remove select columns to the query

Remove select columns can remove previous select columns.

Can you provide an example of why you would use this?

I am closing this pull request because it lacks sufficient explanation, tests, or both. It is difficult for us to merge pull requests without these things because the change may introduce breaking changes to the framework.
Feel free to re-submit your change with a thorough explanation of the feature and tests - integration tests are preferred over unit tests. Please include it's benefit to end users; the reasons it does not break any existing features; how it makes building web applications easier, etc.
Thanks!


Control the exposed public API

I want to add some control over the public API we expose.
One particular option is Microsoft.DotNet.ApiCompat.

Okay, ApiCompat is not that great.
It requires a mandatory DLL that it will compare the current code with.
It has no ability to export the baseline API and somehow compare it in a "gold master testing" way.
I'll seek for a better solution.


line_length and line_count not honored on RTF

Appears that line_length and line count are being ignored on RTF, at least.

It's when output_type is on write_report().  When output_type is on create_report(), everything is fine.


Remove Automator code from Patrol plugin

Can't verify if works because Jitpack is down.

We decided to stay with the Flutter plugin approach and introduce a less invasive solution.


Maven dependency isn't working

The maeven link to the repository dependency isn't working. Amazing library!

Hey, thanks for checking out the repo. The library isn't finished yet so it isn't actually published, although the readme makes it seem otherwise. I haven't touched it in a few months now, but I do intend to get back to it soon.

Thanks. Nice to know. As soon as it get finished i will use it. Regards

Hi. It seems it's pretty complete to me.
Would be really cool and convenient if you could publish your library.
We can always pull request patches if something changes 👍

You're right, sometime soon I'll have a look through the project to see if there's anything that's blocking me from publishing it, otherwise I'll publish it.

Sorry for the delay, I had some trouble with the deployment. It's done now though and should be accessible via maven central. I've updated the readme with installation instructions. Let me know if you run into any problems! :tada:

The maven link to the repository dependency still isn't working.

Thanks for mentioning that, I just checked and apparently I only pushed it as a snapshot and staged a release. Not sure what happened there, but it's now available as a proper release.


Is this deprecated?

I'm wondering since TSLint mentions their repo is deprecated, should tslint-config-prettier also be considered deprecated?

Sort of. tslint-config-prettier is still maintained, but tslint-config-prettier does not need to be updated as long as there is no update coming from supported rulesets.


Implement  Detailed HTTP Response for Destination Test

Describe your changes:
Added TestDestinationStatus to Capture Detailed HTTP Response for Destination Test

statusCode, statusInfo for response metadata
headers, entity, mediaType, and location for body and headers
timestamp for response timing

Type of change:

[ ] Bug fix
[x] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[x] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

Moved here: #18448


[Information]systemjs.config

Hi,
It will be good to add in the documentation that we need to modify systemjs.config.
Code:
'ng4-geoautocomplete': 'npm:ng4-geoautocomplete/bundles/ng4-geoautocomplete.umd.js'
Have a great day!
Thank you for the lib!

Hi. Thanks for the suggestion. I have updated the document with the above-suggested way,


Enhancements

Run-time Chat implementation, UI changes(history page added and home-page changed), amount of text transfer increased from 5 to 30 letters, logo added, Name extracted from google account. Chat stored and parsed from local JSON file.

PR updated!!!!


Add build support for arm64 architecture

Check the main Makefile to suport the ARM64 architecture, considering the
default build architecture x86_64.
Building on arm64 on a different architecture requires a toolchain
to be installed by the user. The default toolchain chosen is
the one offered by Linaro. Also, a specific toolchain can be set by
defining the CROSS_COMPILE environment varriable.
An example of building on arm64 architecture is:
make ARCH=arm64
Future commits will move the build inside a Docker container where the
default toolchain will be automatically installed.
Signed-off-by: George-Aurelian Popescu<EMAIL_ADDRESS>Observation
I don't know why git displays the Makefile formatting like that, but if you open the Makefile from the repository you'll see that
are the variables are aligned.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Observation
I don't know why git displays the Makefile formatting like that, but if you open the Makefile from the repository you'll see that
are the variables are aligned.

It's likely because tabs and spaces are used on the same line. I would suggest using just spaces for in-line aligning.


builders and github push not working for new CLs

http://build.golang.org does not show builder runs for the last few CLs that were submitted (~24 hours)
and the github repo also does not seem to have catched up on the last few commits that were submitted.

Last time this happened there was a stack trace in the maintner logs (which I think is powering a lot of the data sources here) but I don't have access to the production logs for that service. @adams-sarah ?

+andy
Probably pubsubhelper:
2017/09/12 18:26:17 Client error: read error: EOF
2017/09/12 18:26:18 Client: "AUTH LOGIN\r\n", verhb: "AUTH"
2017/09/12 18:26:18 Client error: read error: EOF
2017/09/12 18:26:19 Client: "AUTH LOGIN\r\n", verhb: "AUTH"
2017/09/12 18:26:19 Client error: read error: EOF
2017/09/12 18:26:19 Client: "AUTH LOGIN\r\n", verhb: "AUTH"
2017/09/12 18:26:19 Client error: read error: EOF
2017/09/12 18:26:20 Client: "AUTH LOGIN\r\n", verhb: "AUTH"
2017/09/12 18:26:20 Client error: read error: EOF
2017/09/12 18:28:15 Client: "AUTH LOGIN\r\n", verhb: "AUTH"

Hm. https://pubsubhelper.golang.org/recent is indicating that new GitHub events are being processed, but not new Gerrit events...

Is the "AUTH LOGIN" coming from Google's MTA or just random spammers from the Internet finding a port 25 open? If the latter, it's a red herring.
But if Google has changed its delivery to now attempt AUTH, that's weird.
FWIW, our SMTP greeting is:
$ telnet pubsubhelper.golang.org 25
Trying <IP_ADDRESS>...
Connected to pubsubhelper.golang.org.
Escape character is '^]'.
220  ESMTP gosmtpd
^]q

Maybe it needs to say something more or less to make clients not try to authenticate?
Or maybe we should just reply to any AUTH request with a success message.
But first I'd like to see if that's actually Google authenticating.

subrepo commits are appearing in datastore cache, but not go main repo commits since 2d69e9e259ec0f5d5fbeb3498fbd9fed135fe869 yesterday

Found the issue. The bot that is set to receive the emails had all of its watches deleted from its Gerrit account. Re-added a watch on All-Projects and will follow up with the Gerrit team. Let’s see if this fixes the underlying problem.

:tada: :tada: thanks andy

Pubsubhelper can't be the only problem. It's only a wake-up for maintner. Maintner polls Gerrit regularly as a fallback without any pubsubhelper.
So what's the deeper problem?

maintner is seeing the new commits over and over...
eg.
2017/09/12 18:21:19 mutation: gerrit:<project:"go.googlesource.com/go" commits:<sha1:"b86fae041baff7a7077c67060bc4282d78a2118b" raw:"tree 422b34aa8cbf67be5c17ff045cbbdb5bd5c12340\nparent 80b2ae5878c7b08b605128cc885305bf86bc5475\nauthor tbunyk<EMAIL_ADDRESS>1504627276 +0300\ncommitter Ian Lance Taylor<EMAIL_ADDRESS>1505239944 +0000\n\nencoding/json: update documentation for MarshalIndent\n\nMake arguments semantics clear without the need to look for\njson.Indent documentation.\n\nChange-Id: If9adfe9f477a30d426ae83790b0f2578c0a809b7\nReviewed-on: https://go-review.googlesource.com/61670\nRun-TryBot: Ian Lance Taylor<EMAIL_ADDRESS>Gobot Gobot<EMAIL_ADDRESS>Ian Lance Taylor<EMAIL_ADDRESS>diff_tree:<file:<file:"src/encoding/json/encode.go" added:2 > file:<file:"src/encoding/json/example_test.go" added:19 > > > >
2017/09/12 18:21:19 wrote record; using existing flush timer.
could it be that something is off with processGerritMutation() (maintner/gerrit.go:483)?
have to run to a mtg but will keep looking at this after.

ok they're back up.
i still have no idea what happened. but this seemed related to https://github.com/golang/go/issues/21555 so tried restarting gitmirror b/c i have to leave work now.
that worked...
... will look at this more tomorrow.

Copy-pasting my update from #21555 :
[The root cause of this issue is that] the builders are set up such that if a commit is pushed to the dashboard, and the dashboard does not recognize the new commit's parent commit SHA, it will reject the new commit. This puts us in a place where, if we lose one commit, we can never recover (until we restart gitmirror).


ROX-14979: Trim sso:provider-id: prefix from user ID

Description
We have 2 points of user identity extractions:

when a user logs in for the first time, the central will add the user to the tenant group;
when the user calls central, the central may issue an event, with the user ID.

There's apparently a problem in that those two cases produce different user IDs:

uses the actual user ID, like admin or<EMAIL_ADDRESS>uses the token-based ID, like<EMAIL_ADDRESS>
As the two IDs do not match, the actual events get no assigned tenant group. This explains why we miss the 'Secured Cluster Registered' event in the funnel.
The change
This PR make the hash function to trim the prefix generated by the tokenbased id extractor, so that
user IDs extracted from tokens of the requests to central would match the original auth provider id of the user.
Previously, the admin user was added to the tenant group in both forms: admin and sso:...:admin. This is now removed.
Removed also the FormatUserID method extraction, now unused.
Checklist

[ ] Investigated and inspected CI test results
[x] Unit test and regression tests added
[ ] ~Evaluated and added CHANGELOG entry if required~
[ ] ~Determined and documented upgrade steps~
[ ] ~Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

Testing Performed
"Secured Cluster Registered" event initiated by the admin user via UI shows the user ID in the original form of "admin" without the sso: prefix. Compare the user hash here:
$ print -n "8da267ec-9cc6-443b-ae70-69ecff1b8654:4df1b98c-24ed-4073-a9ad-356aec6bb62d:admin" | sha256sum | xxd -r -p | base64
obWqrhs09AnMESltTmgXWF5VY8BNpcLFFndV1kMwLqY=

And there:
{
  "context": {
    "device": {
      "id": "8da267ec-9cc6-443b-ae70-69ecff1b8654",
      "type": "Central"
    },
    "library": {
      "name": "analytics-go",
      "version": "3.0.0"
    }
  },
  "event": "Secured Cluster Registered",
  "integrations": {},
  "messageId": "dfe0b8db-2cf9-4809-ab2b-1a210b6ab424",
  "originalTimestamp": "2023-02-10T10:40:35.590761234Z",
  "properties": {
    "Cluster ID": "6bbb49b6-19d6-4585-b111-2567832299ab",
    "Cluster Type": "KUBERNETES_CLUSTER",
    "Managed By": "MANAGER_TYPE_UNKNOWN"
  },
  "receivedAt": "2023-02-10T10:40:43.778Z",
  "sentAt": "2023-02-10T10:40:43.118Z",
  "timestamp": "2023-02-10T10:40:36.250Z",
  "type": "track",
  "userId": "obWqrhs09AnMESltTmgXWF5VY8BNpcLFFndV1kMwLqY=",
  "writeKey": "..."
}

/retest

/retest


Create an EventBus class

Create an EventBus class
Create class or method which would create an event
Find a way to pass EventBus to Model nodes (DI?)

updated
  index: `${DocumentId}:${BlockId}:data@${DataKey}|tune@${TuneName}:${StartIndex}`
  event: ‘added’ | ‘removed’ | ‘modified’,
  data:
    text (added, removed) -> new text value
    text (modified) -> end index, new formatting name + value, isInverted
    value (modified) -> new value
    tune (modified) -> new tune value
    block -> BlockSerialized
    document (modified) -> prop name , prop value
}

Closed by #43


correct the build command

Build command was not correct as there is no --live-reload flag with build option

How long does an individual PR preview stick around?

On Fri, 3 Apr, 2020, 08:37 Lee Calcote<EMAIL_ADDRESS>wrote:

Merged #154 https://github.com/layer5io/meshery.io/pull/154 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/layer5io/meshery.io/pull/154#event-3194663367, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKQMTLUNTQXVVRNNUE6QZ2DRKVHHFANCNFSM4L3C37GA
.


[ASTVerifier] Check the type of a TupleExpr, at least at a basic level.

Nothing too interesting; a swift-ci sanity check.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci Please test.

Linux failure is unrelated --> merging.


Handled the case where an IAM User was deleted since the last evaluation.

Handled the case where an IAM User was deleted since the last evaluation. The next time the config rule is run, the configuration_item["configuration"] will be 'None' causing a runtime error.

No response. Closing


Add Playbook field to OpenStackDataPlaneServiceSpec

The services under config/services will be migrated to use playbooks
from edpm-ansible. This commit adds the Playbook field to the service
CRD so the services can be switched over to use playbooks. Once all
services are migrated, the Role field will be removed in a future
commit.
Signed-off-by: James Slagle<EMAIL_ADDRESS>

recheck

recheck

recheck

recheck

dataplane-operator-crc-podified-edpm-baremetal is consistently failing, even though I don't think it is related to this PR

recheck

recheck

/override rdoproject.org/github-check


Offset dragging point in PathFollow(2D) with value lesser than 1 overlap with Inspector

Godot version:
3.1 8dd00ed
OS/device including version:
Windows 10
Issue description:
When Path node with Curve and PathFollow is created, then changing offset to values lesser than 1, cause change position of dragging point to Inspector area

A circle for clarity:


Unable To Dispose RSA Key before JWT creation.

Hi guys!
Today I updated my Microsoft.AspNetCore.Authentication.JwtBearer to v2.1.0 and I ran into some issues which I think should be fixed.
Basically I have an API that generates JWT tokens signed with RSA. That was my code:
using (RSA publicRsa = RSA.Create())
{
            publicRsa.EdoFromXmlString(_options.PublicKey);
            _issuerSigningKey = new RsaSecurityKey(publicRsa);
}
using (RSA privateRsa = RSA.Create())
{
            _privateRsa.EdoFromXmlString(_options.PrivateKey);
            RsaSecurityKey privateKey = new RsaSecurityKey(_privateRsa);
            _signingCredentials = new SigningCredentials(privateKey, SecurityAlgorithms.RsaSha256);
}
_jwtHeader = new JwtHeader(_signingCredentials);

//payload generation

JwtSecurityTokenHandler _jwtSecurityTokenHandler = new JwtSecurityTokenHandler();
_jwtSecurityTokenHandler.WriteToken(new JwtSecurityToken(_jwtHeader, payload));

Basically, created my RSAs in usings statements, used them and they were auto disposed. Now after upgrading this code was broken and was throwing exceptions on both sides, when creating the JWT and when trying to read it.
I use the _jwtHeader generated above and the payload is not that important.
That is the exeption:
That is the exeption:
System.ObjectDisposedException: Safe handle has been closed
   at System.Runtime.InteropServices.SafeHandle.DangerousAddRef(Boolean& success)
   at System.StubHelpers.StubHelpers.SafeHandleAddRef(SafeHandle pHandle, Boolean& success)
   at Interop.Crypto.RsaSize(SafeRsaHandle rsa)
   at System.Security.Cryptography.RSAImplementation.RSAOpenSsl.TrySignHash(ReadOnlySpan`1 hash, Span`1 destination, HashAlgorithmName hashAlgorithm, RSASignaturePadding padding, Boolean allocateSignature, Int32& bytesWritten, Byte[]& signature)
   at System.Security.Cryptography.RSAImplementation.RSAOpenSsl.SignHash(Byte[] hash, HashAlgorithmName hashAlgorithm, RSASignaturePadding padding)
   at System.Security.Cryptography.RSA.SignData(Byte[] data, Int32 offset, Int32 count, HashAlgorithmName hashAlgorithm, RSASignaturePadding padding)
   at System.Security.Cryptography.RSA.SignData(Byte[] data, HashAlgorithmName hashAlgorithm, RSASignaturePadding padding)
   at Microsoft.IdentityModel.Tokens.AsymmetricSignatureProvider.Sign(Byte[] input)
   at System.IdentityModel.Tokens.Jwt.JwtSecurityTokenHandler.CreateEncodedSignature(String input, SigningCredentials signingCredentials)
   at System.IdentityModel.Tokens.Jwt.JwtSecurityTokenHandler.WriteToken(SecurityToken token)
   at APIGateway.Auth.JwtHandler.CreateUserToken(User user, List`1 roles) in /home/kaloyanmanev/Projects/edotailor/APIGateway/Auth/JwtHandler.cs:line 64
   at APIGateway.Services.UserService.AuthenticateUserAsync(String username, String password) in /home/kaloyanmanev/Projects/edotailor/APIGateway/Services/UserService.cs:line 27
   at APIGateway.Controllers.UserController.AuthenticateUser(String Username, String Password) in /home/kaloyanmanev/Projects/edotailor/APIGateway/Controllers/UserController.cs:line 46
   at Microsoft.AspNetCore.Mvc.Internal.ActionMethodExecutor.TaskOfIActionResultExecutor.Execute(IActionResultTypeMapper mapper, ObjectMethodExecutor executor, Object controller, Object[] arguments)
   at System.Threading.Tasks.ValueTask`1.get_Result()
   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.InvokeActionMethodAsync()
   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.InvokeNextActionFilterAsync()
   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Rethrow(ActionExecutedContext context)
   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)
   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.InvokeInnerFilterAsync()
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.InvokeNextResourceFilter()
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.Rethrow(ResourceExecutedContext context)
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.InvokeFilterPipelineAsync()
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.InvokeAsync()
   at Microsoft.AspNetCore.Builder.RouterMiddleware.Invoke(HttpContext httpContext)
   at Microsoft.AspNetCore.Builder.RouterMiddleware.Invoke(HttpContext httpContext)
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http.HttpProtocol.ProcessRequests[TContext](IHttpApplication`1 application)

After I saw the exeption I changed my code to:
_publicRsa = RSA.Create();
_publicRsa.EdoFromXmlString(_options.PublicKey);
_issuerSigningKey = new RsaSecurityKey(_publicRsa);

_privateRsa = RSA.Create();
_privateRsa.EdoFromXmlString(_options.PrivateKey);
RsaSecurityKey privateKey = new RsaSecurityKey(_privateRsa);
_signingCredentials = new SigningCredentials(privateKey, SecurityAlgorithms.RsaSha256);

_jwtHeader = new JwtHeader(_signingCredentials);

//payload generation

string result = _jwtSecurityTokenHandler.WriteToken(new JwtSecurityToken(_jwtHeader, payload));
_publicRsa.Dispose();
_privateRsa.Dispose();

As you can see I removed the using statements and I dispose the RSAs myself and it now works. (Had to do this on both sides, when creating the token and when validating it).
I am not sure if this is an issue or feature but to me it seems that my old should still be working.
This only appears when I run the app from the CLI, if I am debugging the application no such problems occur and the token is generated without problems. Really strange...
Firstly, I thought that the issue has to do something with the .NET Core 2.1 as I migrated to it and I posted an issue here: https://github.com/dotnet/core/issues/1688#event-1677675645. The guys there told me to post it here, hope we can sort it out :)

@kmanev073 I am not sure why this worked before, what version would were you using?
When the RsaSecurityKey is passed a RSA object, it will use it when the JWT is signed. You have disposed the RSA before it is being used, so I would expect it to fail.

let me know if you need any additional information.

Hi,
Sorry for the delay. This is the line from my csproj file:

So I guess I was using version 2.0.3

We encountered the same issue exactly as @kmanev073 described.
Previously we used version 2.0.4 of Microsoft.AspNetCore.Authentication.JwtBearer, with our RSA being created in a using block.
After updating to 2.1.0, things continued to work with the debugger attached, but without debugger we get the ObjectDisposedException exception.
What makes things tricky is that the exception does not bubble up and therefore did not get caught by our global exception handler. Instead, all we got was a signature invalid response ( even though the signature is valid ).


Codeium throws errors when treesitter is disabled

Hi, I am playing around with this plugin and found an issue: I have treesitter configured to disable text highlight in big files (in those cases it is too slow and makes the editor hang for a couple of seconds after each key stroke) and it seems that doing this makes your plugin break with the following error
Error detected while processing TextChangedI Autocommands for "*":
Error executing lua callback: .../.local/share/nvim/lazy/plenary.nvim/lua/plenary/job.lua:405: Failed to spawn process: {
  _additional_on_exit_callbacks = {},
  _shutdown_check = <userdata 1>,
  _stderr_results = {},
  _stdout_results = {},
  _user_on_exit = <function 1>,
  args = { "-sSL", "-D", "/tmp/plenary_curl_2fe047dc.headers", "--compressed", "-X", "POST", "-H", "Content-Type: application/json", "--data-raw", "{\"editor_options\": {\"insert_spaces\": true, \"tab_size\": 2}, \"metadata\": {\"extension_name\": \"vim\", \"ide_name\": \"neovim\", \"request_id\": 3, \"api_key\": \"API_KEY_HERE\", \"extension_version\": \"1.1.32\", \"ide_version\": \"0.8.3\"}, \"document\": {\"editor_language\": \"sql\", \"language\": 41, \"line_ending\": \"\\n\", \"cursor_offset\": 1043599, \"text\": \"ENTIRE_FILE_HERE\"}

(api key and text have been cropped out just to be safe)
If I enable treesitter again the error disappear but then the editor became unusable, it would be great to fix this issue.
Let me know if you need something else to help debug this issue :).

I'm having trouble reproing this issue. How are you disabling TS? TSBufDisable?

I am disabling it directly in the configuration like this (basically what treesitter suggests in their docs https://github.com/nvim-treesitter/nvim-treesitter/blob/master/doc/nvim-treesitter.txt#L117 )
      highlight = {
        enable = true,
        disable = function(_, buf)
          local max_filesize = 100 * 1024 -- 100 KB
          local ok, stats = pcall(vim.loop.fs_stat, vim.api.nvim_buf_get_name(buf))
          if ok and stats and stats.size > max_filesize then
            return true
          end
        end,
      }

Now that I reread the issue again it was slightly misleading, what I am actually doing is disabling the highlight module and fallback to the regex, not disabling the entire treesitter, sorry my bad.

Not sure if this is related, but also getting an error from the command window (q:), when editing text and completion is about to appear
Error detected while processing TextChangedI Autocommands for "*":
stack traceback:
	[C]: in function 'json_encode'
	...g/.local/share/nvim/lazy/codeium.nvim/lua/codeium/io.lua:407: in function 'post'
	.../.local/share/nvim/lazy/codeium.nvim/lua/codeium/api.lua:146: in function 'request'
	.../.local/share/nvim/lazy/codeium.nvim/lua/codeium/api.lua:287: in function 'request_completion'
	...ocal/share/nvim/lazy/codeium.nvim/lua/codeium/source.lua:103: in function 'complete'
	...jemag/.local/share/nvim/lazy/nvim-cmp/lua/cmp/source.lua:325: in function 'complete'
	...e/jemag/.local/share/nvim/lazy/nvim-cmp/lua/cmp/core.lua:289: in function 'complete'
	...e/jemag/.local/share/nvim/lazy/nvim-cmp/lua/cmp/core.lua:169: in function 'callback'
	...e/jemag/.local/share/nvim/lazy/nvim-cmp/lua/cmp/core.lua:219: in function 'autoindent'
	...e/jemag/.local/share/nvim/lazy/nvim-cmp/lua/cmp/core.lua:161: in function 'on_change'
	...e/jemag/.local/share/nvim/lazy/nvim-cmp/lua/cmp/init.lua:313: in function 'callback'
	...local/share/nvim/lazy/nvim-cmp/lua/cmp/utils/autocmd.lua:49: in function 'emit'
	...local/share/nvim/lazy/nvim-cmp/lua/cmp/utils/autocmd.lua:23: in function <...local/share/nvim/lazy/nvim-cmp/lua/cmp/utils/autocmd.lua:22>

I got the same error, but in my case the problem was not in the treesitter highlight, but in the big files themselves.
I was having this problem on a file with around 3500 lines and when I deleted around 1000 lines the problem was gone. On the other hand, I took a plain text file and copy/pasted until I got to about 5000 lines and the problem started.
I was also able to reproduce this in .zshrc and Lua files.

I'm getting this same error.
I think my issue is related to @brunolpsousa where I'm getting this on large C# files.
Files that don't work:

34KB with ~ 700 lines
31KB with ~ 600 lines
28KB with ~ 680 lines

Working files

29KB with ~ 550 lines

It looks like the issue is tied more to the number of lines than anything in my case. Once the file gets to be around 600 lines it starts to fail in C#.
I'm running on Windows. I can provide any logs as needed

The fix is we probably have to switch the curl to accept data from stdin instead of passing it as an argument.

Still happening in Apr 2024 :(

Still happening in Apr 2024 :(

Same thing for me. Getting this error at big yml files, than any completion call on any file raise this error

I was experiencing this issue too. I ended up configuring nvim-cmp per buffer, and excluding codeium on files over 100KB. From above comments, that number should probably be lower, but this works for me at the moment. You can see the change in my config here.
I referred to the nvim-cmp documentation on this page: https://github.com/hrsh7th/nvim-cmp/wiki/Advanced-techniques#disabling-completion-in-certain-contexts-such-as-comments
This obviously doesn't fix the root issue, and it would be nice if codeium worked in large files, but it will keep your neovim usable.

I made a PR to fix this. Can anyone test it and provide feedback?

Yup, I can confirm that it works. I just changed Exafunction to brunolpsousa in my config and reinstalled the plugin. Awesome stuff, hope it gets merged soon!

I'm getting this problem too. It doesn't seem to be for big files necessarily, but seems to come when I have many tabs open/leave neovim on for long periods of time. I switched to brunolpsousa and ran Lazy sync but still get this problem, am I doing something wrong?

I'm getting this problem too. It doesn't seem to be for big files necessarily, but seems to come when I have many tabs open/leave neovim on for long periods of time. I switched to brunolpsousa and ran Lazy sync but still get this problem, am I doing something wrong?

You don't need to use my branch because it has already been merged. Make sure your Lazy config is not locked to a specific commit or any other version control. If it still doesn't work, I think it would be better to open a new issue and provide logs.


[fix](broker load) sync the control flow of BrokerScanner to other Scanner to avoid oom

Proposed changes
Issue Number: close #xxx
Problem Summary:
Describe the overview of changes.
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

Hi @SleepyBear96 Please also create an issue to describe the problem you solved.

Hi @SleepyBear96 Please also create an issue to describe the problem you solved.

Done.

If both of all child scanner have same method， you better impl the funciton in BaseScanner

Done.


Untilting a panorama

I'm using pannellum to view panoramas, some of which are tilted. Along with the panoramas, I'm given the orientation of the central camera when the panorama was taken (i.e. the center of the panorama) expressed as a quaternion (which I know can then be converted to a rotation matrix). I also know pannellum has tools to untilt a panorama, namely setHorizonRoll and setHorizonPitch. However, I'm struggling to go from the rotation matrix to angles I can feed into pannellum. I can use scipy's Rotation.as_euler to decompose a rotation matrix into euler angles, which might be promising. However, I can't quite figure out the necessary order of the rotations, and whether they're intrinsic or extrinsic. Does anyone have any pointers on this or any method which has worked for them? Thank you!

The horizon pitch and roll parameters were added to support Google Photo Sphere XMP metadata, so they follow the convention defined there:
https://developers.google.com/streetview/spherical-metadata#euler_overview
You can also go straight from quaternions to Euler angles. Pannellum uses quaternions internally to deal with the Device Orientation API, so it needs to convert back to Euler angles:
https://github.com/mpetroff/pannellum/blob/83084e892cc3dc0bedc53307fabe6db6acc93937/src/js/pannellum.js#L1697-L1709
These are applied as pitch = phi, roll = -theta, yaw = -psi. You might (probably?) have to flip the signs, since you're trying to undo a rotation instead of apply one.

Thank you! After some work I was able to apply the inverse operations to untilt the panoramas.