Program re-architecture

With the sudden interest in collaborating in this project - it's time to shift this project from being a script intended only for my use, and something that people can actually work together on.
If I could set this ticket to "blocking everything" I would - it's the highest priority.
Some changes I plan on making as part of this update:

Shift the important data structures away from complex stacked dictionaries and into structs (this includes the config variable, as well as any stock variables, as well as anything else I deem as something we might need in the foreseeable future). This will allow the easy addition of new kwargs and attributes without having the restructure all existing code.
Actually have more than one file for the entire program.
Increase the number of functions to make the code significantly more modular
Write some half-assed documentation.
A series of other small tasks, basically anything that comes up while making the translation.

!! If you are looking to collaborate on this project, do not write any code until this ticket has been marked as "done", any code you write beforehand will not be accepted. !!

That took longer than I wanted, but it's done. Much happier with where the project is at now - additional code should be much easier to add in the future.


Cascaded select - Parent with predefined value

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[x] support request

Current behavior
To explain the situation, I will take as an example the demo Cascaded select.
In the demo, the dropdown select sport has no predefined value. So, on change, form.get('sport') in team onInit from lifecycle is correctly bound.
But imagine that the first value of sport dropdown is already set while opening the form. In that case, form.get('sport') isn't bound, and no options while be available in team dropdown.
Expected behavior
If sport dropdown has a predefined value while opening the form, team dropdown values must be loaded.
Minimal reproduction of the problem with instructions
In the model, set sport with a value.

Angular version: 5.2.X

Language: [TypeScript 2.5.3 | ES6/7]

I see, using startWith operator would solve the issue,
form.get('team').valueChanges.pipe(
  ...
  startWith(form.get('team').value)
)

can you make a PR ?

Thanks for the answer, I tried it and it works perfectly !
In the same vein, imagine that team value is also predefined (the value exist in the model), how do I make this value selected in team dropdown ?
I tried with field.formControl.setValue(), but it doesn't seems to work :
// Team field
form.get('sport').valueChanges.pipe(
    startWith(form.get('famille').value),
    takeUntil(this.onDestroy$),
    tap(sport_id => {
        ...
        field.formControl.setValue(form.value.team);
        field.templateOptions.options = sousFamilleFormValues;
    })
)
field.formControl.setValue(form.value.team)

For the PR, do you mean adding this case in the Cascaded select demo page ?

I think it should work without explicitly calling setValue,

For the PR, do you mean adding this case in the Cascaded select demo page ?

yep

I think it should work without explicitly calling setValue

Still doesn't work.

For the PR, do you mean adding this case in the Cascaded select demo page ?

yep

I'm taking care of it.

can you try with:
          form.get('sport').valueChanges.pipe(
            takeUntil(this.onDestroy$),
            startWith(form.get('sport').value),
            tap(sportId => {
              field.templateOptions.options = teams.filter(team => team.sportId === sportId);
              if (field.formControl.value && !field.templateOptions.options.find(o => o.id === field.formControl.value)) {
                // issue with angular select field: we should add an extra empty option to avoid select the first option
                // TODO: report to angular
                field.templateOptions.options = [
                  {},
                  ...field.templateOptions.options,
                ];
                field.formControl.setValue(null);
              }
            }),
          ).subscribe();

The same should be applied to player field.


Updated dependencies

Added spoonx/aurelia-api
Removed aurelia-http-client (solely uses Fetch client now)

PR for #7

Oh yeah, we talked about this. Nice! Thanks man :)


Make the deployment in CI optional

To allow the installation in CI via the bundle, introduce SKIP_MAKE_DEPLOY which skips make deploy when true.
/cc @enriquebelarte

/lgtm


Opening an already-running app via a custom URL scheme link freezes the app

Closed apps can be opened fine via a custom URL scheme link, but if the app is running, it will freeze.
See also https://github.com/AppGyver/steroids/issues/865 for a related issue.
Previous ticket was #141 which I'll leave as closed because it was about a different issue.

A fix is live with version 4.0.5-edge8, available from the build service. Please test!

I'll give a try, is there a similar fix for iOS?

I wasn't aware of a similar problem with iOS. Are you experiencing this behavior on iOS too?

We still seem to have some issues here. We have 2 listeners hooked up for the phase and resume events and after edge8(& onwards) we only get notifications on one of the listeners. the second listeners does not get notified. Is there any change int eh pub sub model that would affect that behavior ?

Hey Harri…
The fix does not completely work I have updated the issues with my latest observations on iOS and Android.
https://github.com/AppGyver/steroids/issues/861 https://github.com/AppGyver/steroids/issues/861
https://github.com/AppGyver/steroids/issues/862 https://github.com/AppGyver/steroids/issues/862
Can you guys please take a look.
Thanks
Manuj

On Mar 12, 2015, at 2:32 AM, Harri Sarsa<EMAIL_ADDRESS>wrote:
I wasn't aware of a similar problem with iOS. Are you experiencing this behavior on iOS too?
—
Reply to this email directly or view it on GitHub https://github.com/AppGyver/steroids/issues/861#issuecomment-78447849.

What do you mean with "phase" event? What type of navigation stack structure you have when you're getting the error?

Sorry I meant pause event.. That maybe an issue on our side (we may not have structured it right).
However on Android I still see that the launch url is empty when the resume event is called.
Our code is structured in the following way ..
ionic.Platform.ready(function() {
// THIS IS FOR COLD START OF THE APP
$scope.checkAndHandleLaunchURL();
$ionicPlatform.on('pause', function () {
}, false);

$ionicPlatform.on('resume', function () {
  console.log('got resume');
  // THIS IS FOR WHEN THE APP WAS IN BACKGROUND
  $scope.checkAndHandleLaunchURL();
}, false);

});
$scope.checkAndHandleLaunchURL = function () {
var launchURL = steroids.app.getLaunchURL();
console.log('launchURL = ' + launchURL);
};
Here are the console outputs from cold start and background resume scenarios
using the myawesomeapp://main/invite  link from an email.
COLD START
got resume
got resume -  on to checkAndHandleLaunchURL
launchURL = myawesomeapp://main/invite
APP IN BACKGROUND
got resume
got resume -  on to checkAndHandleLaunchURL
launchURL =
Can you confirm if there is an issue configuring the launchurl for android for resume scenario.
The same scenario works fine on iOS.

Sorry I meant pause event.. That maybe an issue on our side (we may not have structured it right).
However on Android I still see that the launch url is empty when the resume event is called.
Our code is structured in the following way ..
ionic.Platform.ready(function() {
// THIS IS FOR COLD START OF THE APP
$scope.checkAndHandleLaunchURL();
$ionicPlatform.on('pause', function () {
}, false);

$ionicPlatform.on('resume', function () {
	console.log('got resume');
	// THIS IS FOR WHEN THE APP WAS IN BACKGROUND
	$scope.checkAndHandleLaunchURL();
}, false);

});
$scope.checkAndHandleLaunchURL = function () {
var launchURL = steroids.app.getLaunchURL();
console.log('launchURL = ' + launchURL);
};
Here are the console outputs from cold start and background resume scenarios
using the myawesomeapp://main/invite link from an email.
COLD START
got resume
got resume - on to checkAndHandleLaunchURL
launchURL = myawesomeapp://main/invite
APP IN BACKGROUND
got resume
got resume - on to checkAndHandleLaunchURL
launchURL =
Can you confirm if there is an issue configuring the launchurl for android for resume scenario.
The same scenario works fine on iOS.

On Mar 17, 2015, at 5:02 AM, Harri Sarsa<EMAIL_ADDRESS>wrote:
What do you mean with "phase" event? What type of navigation stack structure you have when you're getting the error?
—
Reply to this email directly or view it on GitHub https://github.com/AppGyver/steroids/issues/861#issuecomment-82307192.

Yes, that's currently still a bug, see https://github.com/AppGyver/steroids/issues/865 – let's follow up in the other issue about this!

Fixed in Android 4.0.5!

Hi,
I've been searching how to ask the user to open a link using my app that is made with AppGyver and I don't know how can I do it. Can I find some documentation about how to make my application default with my links?
Thank you!

@watzenare http://docs.appgyver.com/supersonic/guides/navigation/navigating-between-apps/communicating-with-external-apps/

Thank you @Leeds-eBooks !


goto defination not work

Would you like to elaborate?

I am using VScode to write R package. Pressing F12 give an error no defination found for all the functions.

Would you like to provide a minimal reproducible example?
Go-to-definition is actually provided by languageserver via vscode-r-lsp. Have you installed both correctly?

Taking this package (https://github.com/kongdd/CMIP5tools/blob/master/R/CMIP5Files.R#L185) as example, I just can't jump to the defination of function filter_duration.
Maybe I should pull an issue at vscode-r-lsp.

Would you like to try the latest development version of languageserver with
devtools::install_github("REditorSupport/languageserver")

I tried with your example of filter_duration and it works perfectly.

Sorry. I have tried. But still not worknig.

I  have reinstalled languageserver by devtools::install_github("REditorSupport/languageserver").

I have clear all the files of VScode in ~/.config/Code, ~/.vscode and reinstall VScode after sudo apt purge code.
After that, reinstall R LSP.

But still not work! Still give me the error that no defination found
Is OS system maters? I am using Ubuntu 20.04.

I'll try on Ubuntu 20.04 and see if there's something wrong.

Thank you! In my tries, win10 2004 and WSL ubuntu1804 have the same problem.

I tested on macOS 10.15.5 and Ubuntu 20.04 and everything works as expected.
Would you please follow the steps below:

Turn on r.lsp.debug in VSCode
Restart/reload vscode
Open the R project and select Output panel and view R Language Server logging.
Go to the file
Put the cursor on filter_duration and go to definition.
Copy all logs as code here and let's see if anything is not working properly.

Also, please post your R session info. Thanks!

Thank you! The log is in the attachment.
VScode V1.47.0, Ubuntu 20.04
> library(languageserver)
> sessionInfo()

R version 4.0.2 (2020-06-22)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 20.04 LTS

Matrix products: default
BLAS:   /usr/lib/x86_64-linux-gnu/openblas-pthread/libblas.so.3
LAPACK: /usr/lib/x86_64-linux-gnu/openblas-pthread/liblapack.so.3

locale:
 [1] LC_CTYPE=en_AU.UTF-8       LC_NUMERIC=C
 [3] LC_TIME=en_AU.UTF-8        LC_COLLATE=en_AU.UTF-8
 [5] LC_MONETARY=en_AU.UTF-8    LC_MESSAGES=en_AU.UTF-8
 [7] LC_PAPER=en_AU.UTF-8       LC_NAME=C
 [9] LC_ADDRESS=C               LC_TELEPHONE=C
[11] LC_MEASUREMENT=en_AU.UTF-8 LC_IDENTIFICATION=C

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] languageserver_0.3.6

loaded via a namespace (and not attached):
[1] compiler_4.0.2 R6_2.4.1       xml2_1.3.2

LSP.log

Thanks!
The expected logging should look like

[2020-07-13 01:02:10.204] SYMBOL filter_duration

[2020-07-13 01:02:10.219] guess namespace: _workspace_

[2020-07-13 01:02:10.219] guess namespace: _workspace_

[2020-07-13 01:02:10.220] deliver:  ["Response", "Message", "R6"]

[2020-07-13 01:02:12.238] received:  Content-Length: 191

[2020-07-13 01:02:12.249] received payload.

[2020-07-13 01:02:12.249] handling request:  textDocument/definition

[2020-07-13 01:02:12.261] token: {
  "full_token": "filter_duration",
  "right_token": "r_duration",
  "package": null,
  "accessor": "",
  "token": "filter_duration"
}

[2020-07-13 01:02:12.264] definition:  SYMBOL filter_duration

[2020-07-13 01:02:12.266] definition:  {
  "uri": "file:///home/renkun/Workspaces/CMIP5tools/R/CMIP5Files.R",
  "range": {
    "start": {
      "line": 126,
      "character": 0
    },
    "end": {
      "line": 149,
      "character": 1
    }
  }
}

But in your log, it seems it does not find the definition of a wrong token OP-COMMA:
[2020-07-13 00:55:54.768] handling request:  textDocument/definition

[2020-07-13 00:55:54.769] token: {
  "full_token": "filter_duration",
  "right_token": "",
  "package": null,
  "accessor": "",
  "token": "filter_duration"
}

[2020-07-13 00:55:54.777] definition:  OP-COMMA ,

[2020-07-13 00:55:54.778] deliver:  ["Response", "Message", "R6"]

I'll take a closer look at how this could happen.

Thanks for your help! Waiting for your news.

I am using the UTF-8.

Could you tell me how to debug function in languageserver. I am willing to involve in.

The token was detected correctly. It may be the issue of xdoc_find_token?

@kongdd
The easier way is to insert logger$info in the source code and examine the log.

Yes, document$detect_token(point) returns the correct document token, but xdoc_find_token(xdoc, row, col) returns the wrong XML token.
@kongdd I'm wondering if any definition ever works for you? Or it just stops working occasionally like this? I guess we could isolate some minimal code to reproduce this and see why the XML document and XPath have inconsistent results between your machine and mine.

@kongdd
You may try the following code in your R session in the workspace and see if it could reproduce the wrong XML token:
library(xml2)
library(glue)

e <- parse("./R/CMIP5Files.R")
xml <- xmlparsedata::xml_parse_data(e)
xml_file <- tempfile(fileext = ".xml")
writeLines(xml, xml_file)
xdoc <- read_xml(xml_file)

xdoc_find_token <- function(x, line, col) {
  xpath <- glue("//*[not(*)][(@line1 < {line} or (@line1 = {line} and @col1 <= {col})) and (@line2 > {line} or (@line2 = {line} and @col2 >= {col}))]",
    line = line, col = col)
  xml_find_first(x, xpath)
}

node <- xdoc_find_token(xdoc, 185, 62)
node
xml_text(node)

The expected result is
> node
{xml_node}
<SYMBOL line1="185" col1="57" line2="185" col2="71" start="20592" end="20606">

> xml_text(node)
[1] "filter_duration"

but somehow you get OP-COMMA and ,, respectively.

The only way I could reproduce your "no definition" is to put the cursor in the end of filter_duration and before ,, which we have the document token but cannot find the XML token at line 185 col 72 since the XML doc has
<expr line1="185" col1="57" line2="185" col2="71" start="20592" end="20606">
<SYMBOL line1="185" col1="57" line2="185" col2="71" start="20592" end="20606">filter_duration</SYMBOL>
</expr>
<OP-COMMA line1="185" col1="72" line2="185" col2="72" start="20607" end="20607">,</OP-COMMA>

You need to put your cursor on the word, not in the end of the word, or otherwise it only finds the OP-COMMA node which encloses your cursor position.

Thank you @renkun-ken  and @randy3k .
It turn out the problem of cursor position. When I double click the function name, the curcor jump to the end of the function name and "no defination found" was returned. But when I put the cursor in the middle of the function name, it works smoothly.
Thanks for your powerful package!

Just notice that selecting the function name and go-to-definition does not work as the cursor is in the end of the token, which I think is obviously not ideal.
I'm taking a closer look and see if we can do anything about it.

It seems to me it is an alignment issue. Maybe just need to +1 or -1.


Add Victoria University of Wellington (NZ) myvuw.ac.nz domain

@zoltan-nz Pull request merged. Thank you!


Windows Virtual Machine Naming - wrong length

Hi,
Why the windows VM resource name is limited to 15 chars, actually we can create more than 15 characters via the portal and API.
Also we can still set a different hostname than the azure resource name (for netbios blablabla)
So I think it should be 1 to 64 characters
Best regards

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

ID: 374ec8eb-64d4-3766-5c5e-39137a262ffd
Version Independent ID: 3abeeae9-6854-651a-483c-a1673eea9f9f
Content: Resource naming restrictions - Azure Resource Manager
Content Source: articles/azure-resource-manager/management/resource-name-rules.md
Service: azure-resource-manager
Sub-service: management
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

Just saw the note on the bottom of the table, so I open an issue on the CAF provider


Prisma uses "wrong" encoding / should be able to set charset for connection

Bug description
Prisma doesn't always use the right/same character encoding:
Today we migrated our production database, and started getting errors for queries containing emoji:
Conversion from collation utf8_general_ci into utf8mb4_unicode_ci impossible for parameter", state: "HY000"
While the database and table use the same character set and encoding, it seems prisma chooses a different charset for the connection to our "new" server.
It would be helpful to be able to specify the charset for the connection in the prisma settings. While I'm sure there is some strange setting in our db server, this would be super helpfull

Probably related: https://github.com/prisma/prisma/issues/11842

I agree they are related, and the other issue has more info. I have done quite some digging in the mysql server settings, and can't find the utf8mb3 anywhere, so I'm wondering if it comes from prisma as as (strange) default.

Posted a couple of not super great workarounds over on the other issue: https://github.com/prisma/prisma/issues/11842#issuecomment-2249043507


Resource "genesyscloud_tf_export" asking for permissions that do not exist

I've granted Terraform OAuth client credentials with a role that has every single permission applied to it. I've received errors upon trying to use the genesyscloud_tf_export resource without specifying any resource filters as we wish to export everything, hence why we gave it a role with all permissions applied.
Is this an issue with the provider? Or is there a way to workaround this? I've tried the log_permission_errors = true option under the genesyscloud_tf_export resource as one error stated the apply would continue if it found errors and that option was set, but it still stops and fails due to permissions errors that I cannot resolve.
Here is my main.tf, pretty straight-forward.
resource "genesyscloud_tf_export" "export" {
  directory                 = "./export"
  split_files_by_resource   = true
  export_as_hcl             = true
  log_permission_errors     = true
}

Here is the provider I have set. I did try the latest version 1.43.1, however it did not make a difference.
terraform {
  required_providers {
    genesyscloud = {
      source = "MyPureCloud/genesyscloud"
      version = "1.43.0"
    }
  }
}

Here are the permissions that I've seen so far in errors when trying to run the apply, but don't exist in my org.
employeePerformance:externalMetricDefinition:view
routing:transcriptionSettings:view
workitems:worktype:view

Hi @boston51
At the moment I'm unable to recreate the behaviour you're seeing, but I have created a ticket so we can get a dev working on this as soon as possible. If you don't mind, could you copy and paste the error message appearing in your logs? That will help us to debug.
Thanks for bringing this to our attention.
(Tracking with DEVTOOLING-760)

Hey @charliecon , here is an example. This time it happened to be the workitems:worktype:view permission that it's asking for, but again, does not exist as all possible permissions, in all divisions, have been granted to the OAuth credentials being used.
genesyscloud_tf_export.test: Creating...
╷
│ Error: Failed to get task management worktype error: failed to get worktypes: API Error: 403 - Unable to perform the requested action. You must have permission workitems:worktype:view in at least one division. (df95a5ef-dc75-4a55-ba1c-e3085b3dcad1)
│ To continue exporting other resources in spite of this error, set the 'log_permission_errors' attribute to 'true'
│
│   with genesyscloud_tf_export.test,
│   on main.tf line 10, in resource "genesyscloud_tf_export" "test":
│   10: resource "genesyscloud_tf_export" "test" {
│
│ {"resourceName":"genesyscloud_task_management_worktype","method":"POST","path":"/api/v2/taskmanagement/worktypes/query","statusCode":403,"errorMessage":"API
│ Error: 403 - Unable to perform the requested action. You must have permission workitems:worktype:view in at least one division.
│ (df95a5ef-dc75-4a55-ba1c-e3085b3dcad1)","correlationId":"df95a5ef-dc75-4a55-ba1c-e3085b3dcad1"}

Fixed.  Please pull the latest version.


[WIP] OLS-1108: Add attach YAML for Ingress, NetworkPolicy, Route and Service

Add the ability to attach the full YAML or just the YAML status section for pages under the Networking menu. Specifically, from the Ingress, NetworkPolicy, Route and Service details pages.

Override failing test not related to this change:
/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-console"


[FEAT]: Feature Request: Support for Microservices Architecture with Separate Containers in Docker

What would you like to see?
I would like to propose a feature request: it would be beneficial if, when running the application in Docker or on a desktop, the databases and backend services could run in separate containers. This architecture would enhance scalability and flexibility, especially for large deployments.
For instance, the web application could run in one or more containers, while backend services such as databases and Node.js could each be allocated to their own containers, depending on user preferences and resource availability. This separation allows for independent scaling of each component, optimizing resource allocation and management.
Additionally, it would be advantageous to add support for various types of databases, allowing users to select alternatives to Prisma. This flexibility empowers developers to choose the database solutions that best fit their specific use cases.
Moreover, having cron jobs run in dedicated containers would further improve the system’s flexibility, particularly in Docker Swarm or cluster environments. This setup could facilitate efficient task management and scheduling, making the overall system more robust and adaptable to changing demands.
While I’m not certain if this is feasible, I believe this approach could significantly enhance the application’s performance and maintainability, making it an idea worth considering.

I dont disagree but adding this into our core app is not going to happen - the main reason is portability as having a singlular image just works much easier and is way less complex for someone to deploy.
However, you can easily fork and make a docker folder that can make you images for server/collector and then simply connect the two using their URLS as some kind of ENV config.
In a server config, you would need to replace this with an ENV - now they can be separate, for example.
https://github.com/Mintplex-Labs/anything-llm/blob/a0c5d898f00c1f1af8a851e05606680946a91084/server/utils/collectorApi/index.js#L11
The main thing is you will want to make sure all containers are using the same storage volume mount since they share some file objects like the key that signs payloads to the collector, among a lot of other things.


constrainDuringPan and visibilityRatio do not work on 'canvas-release' event

Hi iangilman,
I have canvas images and I need to draw line on canvas image with mouse button press.
I have zoom in and moved to the corner part of the canvas image and set the properties with
viewer.constrainDuringPan = true;
viewer.viewport.visibilityRatio = 500;
After drawing the line on canvas image .On mouse 'canvas-release' event the canvas image comes to original position Even
viewer.constrainDuringPan = true;
viewer.viewport.visibilityRatio = 500; is set .
I Need canvas image to remain in the same position and zoom level after drawing the line
Can you please give any other solution to resolve this ??
Thanks
Vinay

visibilityRatio should be a number between 1 and 0, so that's your first problem.
I'm not sure what you're trying to accomplish... you just don't want the canvas to move at all while the user is drawing lines on it?

Hi iangilman,
Thanks for the response.
Above properties for  visibilityRatio should be a number between 1 and 0,
will get failed when you zoom in image and drag it (image moves to the horizontal and vertical)
Here is the code pin for the same
https://jsfiddle.net/ptqmLf2u/
Step 1:Able to draw the line without zoom in
Step 2:Zoom the image and draw the line .Images shift the potions according to the movement

Sounds like maybe you just want to stop panning? One way to do that would be:
viewer.panHorizontal = false;
viewer.panVertical = false;

Hi,
Its working with these setting and one problem i am facing is, its work perfectly when is image is fit to the viewer of the deep zoom [Image name :viewer1].when i zoom and move the image to the half of the viewer[image name:viewer2] and draw the line,then image will be fit(move) back the deep zoom complete viewer size(but before i draw the line i have moved the image to half of the viewer) how to overcome there problem??
These is the code i am using now
I am calling deepZoomImageMovement function on
viewer.addHandler('canvas-press', function (event) {
///lineAnnotationEnabled  value is set on button click
if (lineAnnotationEnabled == "true" ) {
deepZoomImageMovement(true);
} else {
deepZoomImageMovement(false);
}
}
function deepZoomImageMovement(result) {
if (result) {
viewer.constrainDuringPan = true;
viewer.viewport.visibilityRatio = 1;
viewer.panHorizontal = false;
viewer.panVertical = false;
} else {
viewer.constrainDuringPan = false;
viewer.viewport.visibilityRatio = 0;
viewer.panHorizontal = true;
viewer.panVertical = true;
}
}
}
Thanks in advance

Try removing the lines that change viewer.constrainDuringPan and viewer.viewport.visibilityRatio.

Its work perfectly.
Thank you so much


Avoid switching from Custom to OpenAI when seeing chat.completion

I believe this fixes https://github.com/carlrobertoh/CodeGPT/issues/391
The code previously switched current service to OpenAI whenever it was passed a conversation of type chat.completion. These code changes removes this behavior if we're using Custom OpenAI, but will keep the old behavior otherwise.

I am closing this PR since these code changes are not solving the problem (and I cannot prove that a problem actually exists). I was a bit trigger happy, and I probably messed up something in my reproduction when I thought I both found and fixed a problem.


Improve mobile user experience

Currently Keter is not accessible for mobile users who are not using the keyboard. We need to do something about it.
Are there any samples of mobile roguelike games?

Yes, there is a nice (and free) mobile roguelike named Dungeon of Slyn.


Prepare for release

API change check for Azure.Data.SchemaRegistry
API changes are not detected in this pull request for Azure.Data.SchemaRegistry


Testing, please ignore. (futmro)

Bug bounty test - please ignore.... Please DO NOT APPROVE THIS! (futmro)

 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.alveranuno seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Disruptive syntax errors from the built-in CSS language server

VSCode Version: 1.7.1
OS Version: macOS 10.12.1

Steps to Reproduce:

Save the following as test.css:

body {
  --my-theme: {
    color: red;
    background: white;
  };
}

body div {
  @apply --my-theme;
}

Open it in VSCode, observe syntax errors.

More context
The new css features are really nice, a significant improvement in most development workflows. However, there is some legal CSS syntax (like css custom property values) that is flagged as a syntax error, and there are some emerging CSS features (like @apply) that extend the standard CSS grammar.
I'm on the Polymer team at Google, and we've recently shipped early versions of a language server and a plugin for VSCode to provide a high quality developer experience to our users. I've filed https://github.com/Microsoft/vscode-css-languageservice/issues/7 and https://github.com/Microsoft/vscode-css-languageservice/issues/8 but in the meantime is there anything our users can do – or that we can do in our plugin – to reduce the disruptiveness of these syntax errors to their development experience?

Copying over a comment from #15032, it looks like a workaround for now is to set this preference:
"css.validate": false

I'm not familiar with Polymer's CSS. Are you using PostCSS for transpiling?
If so maybe you can write some linter rules and disable CSS validation as @mrmlnc suggested. StyleLint seems to be popular and support experimental CSS features.
This is what we suggest for users who want to use proposed syntaxes like object spread:
https://code.visualstudio.com/Docs/languages/javascript#_common-questions
Q: How do I disable Syntax Validation when using non ES6 constructs?
A: Some users want to use syntax constructs like the proposed Object Rest/Spread Properties. However, these are currently not supported by VS Code's JavaScript language service and are flagged as errors. For users who still want to use these future features, we provide the javascript.validate.enable setting.
With javascript.validate.enable: false, you disable all built-in syntax checking. If you do this, we recommend that you use a linter like ESLint to validate your source code. Since VS Code's JavaScript support doesn't understand ES7 constructs, features like IntelliSense might not be fully accurate.

Yeah, definitely, we're working on our own linter, including CSS.
Is it possible for a plugin to modify an existing setting's default value? This would be useful in a couple of places. When you install the polymer plugin we could disable css validation, but we could also contribute some json schemas to "json.schemas" as well (e.g. for polymer.json and elements.json, and it would be great to get intellisense and validation and the rest as you get out of the box with package.json and tsconfig.json)

s it possible for a plugin to modify an existing setting's default value?

Can you open a new issue for it?

Sure thing! Filed as https://github.com/Microsoft/vscode/issues/15066
(Oh, and answering my own question above, it looks like there's contributes.jsonValidation for plugins to contribute json schemas)

I sent out PRs to the vscode-css-languageserver and they have now been merged:

https://github.com/Microsoft/vscode-css-languageservice/pull/11
https://github.com/Microsoft/vscode-css-languageservice/pull/10

So I think this bug is safe to close, as it should be fixed in the next release!


Equals should not directly point to GetHashCode

Package+Version:
All current versions
Framework version:
All
Describe the bug
Currently, we've implemented the Equality comparison very lazily by just pointing to GetHashCode.
This is prone to collisions and needs to be updated.
See: https://twitter.com/MrMatthewLayton/status/1587428468731650049
To Reproduce
This is a technical bug.
Expected behavior
-
Screenshots
-
Additional context
-

For the converters, maybe using the equals to override existing serializers isn't smart.
Maybe just trust the stack functionality and make them no longer equatable?

The ./**/DataNodes/Nodes/*Equating.cs has been temporarily excluded in #206 and should be re-included in this story.


This should not need a password

The original sync saber did not need your password, just your username, this is a huge security risk.

I completely agree with @MagnumDopus. The in-game version simply scraped the publicly available bookmark pages. There is ZERO reason for this version to require any sort of login.

The original SyncSaber hasn't actually gotten any songs from your Bookmarks or Followings feed for quite awhile (it doesn't error, it just thinks there weren't any songs there). BeastSaber changed at some point and made your Bookmarks and Followings private, so it does require a login. The only other option would be to have a way for people to log in through their web browser and copy the cookies it makes for the site, which probably isn't worth the trouble. For security concerns, make sure your password for BeastSaber isn't used anywhere else. Unless you're a mapper, I'm pretty sure there isn't anything critical in BeastSaber accounts.

BeastSaber no longer requires a login cookie to get Bookmarks and Follows. Next version won't need a password.


input [type="range"]

Any reason why this selector isn't included in forms.less?

Looks like now it's supported almost everywhere. Can we have some nice styling for this type of input?

We don't restyle native form widgets (like checkboxes, radio buttons, range sliders, etc.) in Bootstrap v3.
We are considering adding optional styling for these widgets in Bootstrap v4.


Emailing companies title & logo

As a user, I want to see who's emailing me so that I can understand my email activity better.
Acceptance criteria: User can see who is emailing them by reading the headline and seeing the company's recognizable logo.
Technical specs: Include the name of the company as a title and include their logo (pull the company's logo from a public database)

In the case where there's no logo or company name available -> see Figma

The backend API is complete. Assigning to @ricardobrg to complete the UI.


Attempt to make timer test more resilient for MacOS CI testing

Signed-off-by: Fred Dushin<EMAIL_ADDRESS>These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).
SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

Didn’t fix the problem.  Retracting this PR.


form not rendered

Hi,
I have django cms 3.1.2 with django 1.8.3 installed.
I followed your installation instructions all work fine and I haven't any errors.
I can assign form app to a page, hook form on placeholder, edit the form and save it but the form is not rendered. When I inspect the DOM with chrome I only view the DIV container for the plugin and nothing inside this.
Any idea ?
thanks

It's a CMS bug.
Same as this https://github.com/stefanfoulis/cmsplugin-filer/issues/190
And according to @yakky
https://github.com/stefanfoulis/cmsplugin-filer/issues/190#issuecomment-123864943.
Fix is coming for 3.1.3

This has been fixed in django-cms 3.1.3.
Tested with djang==1.8.4 and django-cms==3.1.3


feat: [CN-16] - Add support of custom structs with encoding.TextMarshaler interface

Add support of custom structs with [encoding.TextMarshaler] interface. If the structure implements the encoding.TextMarshaler interface, it will be censored as a string (encoding.TextMarshaler result).

@Kalachevskyi LGTM
However, we have to update the docs/index.md as well.
I would add this as a new bullet to our features list and add a new example to Usage section (or update already existing) to show the example of TextMarshaller handling.


Ability to use redstone to control inputs.

We can use redstone to control extractions but not insertions. As it is it can be very trick (or just impossible) to make certain systems without being able to control whether a conduit is inputting or not.
Example case:
Currently:
You have a large storage drawers system and you are always extracting from the controller to fill the inventory of some machines dotted around. You then decide you only want to insert items into one of the machines when you don't have enough of a product. You can use a inventory panel sensor to detect when to output a redstone signal but how do you stop items going in without stopping any items from coming out the drawer controller?
With the change:
You hook up a item level checker of some sort and run the redstone over to the machine where it stops the machine and stops items being inserted.
The only workaround I can think of is to fill a hopper which goes into the machine. But that is both ugly and needlessly large.
Even if this functionality was added with an item conduit upgrade I would be happy.

Thank you for your suggestion. It has been added to our list or possible enhancement.
This ticket will be closed because we want the list of open tickets to reflect known issues.


Matting looks strange . . . compared to older version of picframe

Hi,
after finally setting up a new Pasi3 with Bookworm I see some issues with the matting.
The texture is visble only in the upper left corner with some colors based on the picture . . the rest of the mat remains without matting.

any idea ?
cheers
Torsten

@jgodfrey Hi Jeff, I just did a fresh install using my default method (https://www.thedigitalpictureframe.com/how-to-build-the-best-raspberry-pi-digital-picture-frame-with-bookworm-and-pi3d-picframe-2024-edition/) and indeed your wonderful matting is broken.
I guess due to a recent Bookworm OS update. It works perfectly on my earlier Bookworm installation, but no longer on a fresh install.
Could you try a fresh install for yourself? Thanks much!

Hi Wolfgang,
I've got a spare pi4 here. I can try to do a fresh install using your mentioned article to see if I can reproduce the issue. I'll try to do that sometime "soon'ish".

Update... I've just completed a fresh rpi4 installation with Bookworm using the article linked by Wolfgang above. That worked flawlessly (kudos!). With that, I am able to reproduce the matting problem reported here, though I haven't started to debug it yet. I'll post back when I know more.

OK - I think I found the problem, and it's not OS-related. It seems to be related to a numpy change. The problem is here:
https://github.com/helgeerbe/picframe/blob/6e9fecec6241880df6d462c4f76a9ffebed12b7a/src/picframe/mat_image.py#L308
Changing that line to...
return tuple(colors[0].tolist())
...fixes the problem for me locally. Can one of you verify that fixes the issue on your local system?
Some quick research indicates that in versions of numpy prior to 1.20, when a numpy array is converted to a tuple, the individual elements are converted to native python types (as expected by the consuming code). However, from v1.20 on, the converted data retains the underlying types (np.uint8 in this case) which apparently breaks the colorize() function of PIL (the consumer of this data) which is used to modify the colors in the background mat.
It seems that a new picframe install includes numpy v2.0.1, though I'm not sure when that changed exactly...
@helgeerbe / @paddywwoof - any thoughts?

just tried it . . . it worked right away . . .
thanks
Torsten

I just tried this change on my older picframe install (with a pre 1.20 version of numpy) and it doesn't seem to hurt anything. With that in mind, I'll try to submit a small PR with the update soon.

PR here...

Great, thanks so much, @jgodfrey .
So now @helgeerbe needs to put it in the latest release, and then we are back to all systems working again?

@sapnho - Correct. @helgeerbe will need to verify / merge the PR and release another build.

@sapnho - In the meantime, you can edit the source file to fix your local instance.

again thanks for the nice work and one more question . . .
the texture file . . . does it have to be only one file with that specific name or is a kind of random pick possible whenever there are .jpg files in the respective folder ?

Hi @TorstenSt,
Currently, the texture file is just a single image, identified by the name mat_texture.jpg. While it certainly could be expanded into something more like you describe, it's not supported currently.

Hi,
at least it would be a nice feature ....
cheers
Torsten
Jeff Godfrey @.***> schrieb am Fr., 2. Aug. 2024, 15:34:

Hi @TorstenSt https://github.com/TorstenSt,
Currently, the texture file is just a single image, identified by the name
mat_texture.jpg. While it certainly could be expanded into something more
like you describe, it's not supported currently.
—
Reply to this email directly, view it on GitHub
https://github.com/helgeerbe/picframe/issues/394#issuecomment-2265418903,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGIQVW6WFOQRB67NEK677OTZPODHVAVCNFSM6AAAAABLG3N2WSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRVGQYTQOJQGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

@jgodfrey In both picframe and pi3d pyproject.yaml files we've left the dependency as just numpy though some of the others (ninepatch, Pillow) have >= version. It would be possible to bake in version number but I feel it would be better to get an error and fix it. That's because generally speaking later versions fix problems or tighten up on things that were underspecified. I can maybe see instances where it would be annoying for tuple() to automatically change the types of elements in the result.
In later numpy list(x) and x.tolist() behave differently where the dimension are greater than one. i.e. tolist() converts the whole thing whereas list() only converts the first axis to a python list and leaves the elements inside as they were.
Looking at the Pillow code for ImageOps.colorize I would expect it to behave fine with an array of int even though they specify tuple (or str) in the function definition... Though it's not clear why it can't cope with a tuple of np.uint8 values. Maybe that fails the typing.cast() against a typing.Sequence[int] that gets done at the start of the function.
Paddy

Thanks for the additional detail @paddywwoof. I'd agree that we probably don't want/need to lock the version of numpy. Regarding the current (numpy-based) K-means clustering algorithm... The intent there is to pick the "most dominant" color. IIRC, the 3 returned colors are ordered such that element 0 is (an approximation of) the most dominant color - which is why it's being explicitly picked. I'd think that randomly picking from the returned collection could give undesired results in some situations...

Just tried the manual correction and it works fine. Thanks so much for the quick fix, @jgodfrey.
What's the verdict on numpy now? It would be nice to have a fully functioning version up again. :-)

As the changes from @jgodfrey works fine with older numpy packages. I won't add any version dependencies to numpy. I leave it as it is.
I'll prepare a new release today.


Create benchmark for tracking indexing performance

This issue is for tracking performance related to initial indexing of the blockchain.
@cberner What would be a useful benchmark for tracking index speed? I improved the integration test block and transaction generation code, so I could pretty easily create any amount of blocks and transactions, with arbitrary relationships, spread among any number of blockfiles.
The only thing relevant to indexing that's changed since we last worked on the code together is that I now store the full blocks in the redb database, using a new hash-to-block table. I think that there are no real guarantees about the contents of bitcoind's blocks directory, so I want to move away from using it as much as possible, except for perhaps initial indexing.

I think the easiest would be to track bytes of blockfiles ingested per second. That seems like a good metric, unless there are going to be blocks that are small but really expensive to process

I think the easiest would be to track bytes of blockfiles ingested per second.

Sounds good.

That seems like a good metric, unless there are going to be blocks that are small but really expensive to process

Transactions with many inputs and outputs will probably be more expensive to process on a per-byte basis, but maybe not enough so to be worth thinking hard about.
I'll probably just generate a few gigs of "average" blocks, whatever that means.
By the way, I'm using APFS, which is a copy-on-write filesystem. Will that have any weird performance consequences for redb?

Hmm, do you know what granularity it's copy-on-write at? I'd guess no, if things like postgres or sqlite work on APFS. If the granularity is really course, then you'd want to maximize the size of your write transactions.

Would it be the same as the block size? If so, my APFS filesystem which was initialized with default values is 4096 bytes/block.

It sounds like yes? If so, then that should be totally fine. 4096 is the same as the page size, and redb is copy-on-write at the page level (I removed those complicated delta writes)


Problem accessing /traccar/dataService

I start traccar service in Windows 7 , and try to visit localhost:8082
here is error I faced!
please help me.
HTTP ERROR 503
Problem accessing /traccar/dataService. Reason:
Service Unavailable

Caused by:
org.eclipse.jetty.servlet.ServletHolder$1: javax.persistence.PersistenceException: [PersistenceUnit: release] Unable to build EntityManagerFactory
at org.eclipse.jetty.servlet.ServletHolder.makeUnavailable(ServletHolder.java:498)
at org.eclipse.jetty.servlet.ServletHolder.initServlet(ServletHolder.java:552)
at org.eclipse.jetty.servlet.ServletHolder.getServlet(ServletHolder.java:417)

Hi
I've changed the "localhost" by IP because database is located at another server, but, after this change I cant connect to web anymore.The login window doesnt appear!
tks

@effumachi, are there any error messages in the log (wrapper.log)? What do you mean by not working? It doesn't show up or you cannot login?


08/01/2018 21:50 : Create Pull Request for branch master -> prTesting1_2018080121

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.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.


"r is null" error while recreating container

Bug description
When trying to recreate a container and pull latest image, the error message "r is null" appears.
Expected behavior
Container is recreated
Steps to reproduce the issue:

Go to a container's page
Click on recreate
Check "Pull latest image"
Click recreate
See error

Technical details:

Portainer version: 1.21.0
Docker version (managed by Portainer): 1.13.1, build 7f2769b/1.13.1
Platform: linux (centos 7)
Command used to start Portainer: docker-compose with portainer-compose
Browser: firefox quantum 69.0.1 (64-bit)

Additional context
Image:

Public image or image from private repo?
Anything special with the containers config?
Rgds,
Neil Cresswell
On 21/09/2019, at 7:16 PM, Dak Smyth<EMAIL_ADDRESS>wrote:

Bug description
When trying to recreate a container and pull latest image, the error message "r is null" appears.
Expected behavior
Container is recreated
Steps to reproduce the issue:

Go to a container's page
Click on recreate
Check "Pull latest image"
Click recreate
See error

Technical details:

Portainer version: 1.21.0
Docker version (managed by Portainer): 1.13.1, build 7f2769b/1.13.1
Platform: linux (centos 7)
Command used to start Portainer: docker-compose with portainer-composehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_portainer_portainer-2Dcompose&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=66uUMtByjaqEu_0NZ6_9Vvv-w9Wdy9cKkmlR6YO-VJQ&s=tlbz_5fduHTicllhRvMS3TlBuPYODMIyijM3zze4FZs&e=
Browser: firefox quantum 69.0.1 (64-bit)

Additional context
Image:
[image]https://urldefense.proofpoint.com/v2/url?u=https-3A__camo.githubusercontent.com_d7df4383106311c6ac5656c32d2489908d5d2e13_68747470733a2f2f692e696d6775722e636f6d2f3050486f7342542e706e67&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=66uUMtByjaqEu_0NZ6_9Vvv-w9Wdy9cKkmlR6YO-VJQ&s=Zcnx9THbCnMp8WpSfz04KTg3WSUyRrfIMtF8s1N4Uu4&e=
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_portainer_portainer_issues_3186-3Femail-5Fsource-3Dnotifications-26email-5Ftoken-3DAFQ2XFLQS3IYILQ64GF45FTQK3IRFA5CNFSM4IZAZRFKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HM3M2UA&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=66uUMtByjaqEu_0NZ6_9Vvv-w9Wdy9cKkmlR6YO-VJQ&s=DklVOhW6vqrcXuk-wT22dy4jG7NRiZ7gunSh0nMv4jU&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AFQ2XFO3PUCRRWBIBGJC4TDQK3IRFANCNFSM4IZAZRFA&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=66uUMtByjaqEu_0NZ6_9Vvv-w9Wdy9cKkmlR6YO-VJQ&s=MH2oYpZD9xpz3d73qYZH9mYGJ5CfJdugoD4LTLPNAqw&e=.

It's a public image from DockerHub. I tried recreating it manually again with the same settings, but that did not do anything to fix it.

Oddly I noticed an error while editing the container:
"The Docker registry for the docker.io/nextcloud:latest image is not registered inside Portainer, you will not be able to create a container. Please register that registry first."

Why are you using docker.io?
If its dockerhub, you just use nextcloud, not docker.io/nextcloud

I do not manually do docker.io. I just pull "nextcloud:latest" and it automatically goes to docker.io.

Is ee that you are running quite an old version of docker, are you able to upgrade and test this again?

I have upgraded to docker-ce on Centos 7 (version 19.03.4) and it now works.


404 on EPIServer.Insight.UI Link

When clicking on the EPIServer.Insight.UI link in the top right menu bar

You get a 404.
Should it be going to something like http://webhelp.episerver.com/latest/en/personalization/insight/about-insight.htm?
If someone can give me the correct link I'll fix it and create a pull request.
Matt

Episerver Add-on team will fix the issue.


Add Conductor Node for rythm part of game

More info at https://www.youtube.com/watch?v=_FRiPPbJsFQ

Branch equivocada hehe


fixes the format of the dates for project analysis searching.

They have to be in the format of yyyy-MM-dd without any time component.

Thanks @StingyJack !


Some questions about the code

@Daniil-Osokin  First of all thanks a lot for this nice work, I have some questions about the code in the translate poses section. I want to convert the output 3d relative coordinates to the world coordinate system, so I've looked at the following code, but didn't quite get the exact idea of it. So I would like to ask the following questions:
First, is this procedure specific to this project, or can it be applied to other 3d relative coordinates to world 3d coordinates?
Second, if so, can you tell me the specific operation idea?
`    # translate poses
for pose_id in range(len(poses_3d)):
pose_3d = poses_3d[pose_id].reshape((-1, 4)).transpose()
pose_2d = poses_2d[pose_id][:-1].reshape((-1, 3)).transpose()
num_valid = np.count_nonzero(pose_2d[2] != -1)
pose_3d_valid = np.zeros((3, num_valid), dtype=np.float32)
pose_2d_valid = np.zeros((2, num_valid), dtype=np.float32)
valid_id = 0
for kpt_id in range(pose_3d.shape[1]):
if pose_2d[2, kpt_id] == -1:
continue
pose_3d_valid[:, valid_id] = pose_3d[0:3, kpt_id]
pose_2d_valid[:, valid_id] = pose_2d[0:2, kpt_id]
valid_id += 1
    pose_2d_valid[0] = pose_2d_valid[0] - features_shape[2]/2
    pose_2d_valid[1] = pose_2d_valid[1] - features_shape[1]/2
    mean_3d = np.expand_dims(pose_3d_valid.mean(axis=1), axis=1)
    mean_2d = np.expand_dims(pose_2d_valid.mean(axis=1), axis=1)
    numerator = np.trace(np.dot((pose_3d_valid[:2, :] - mean_3d[:2, :]).transpose(),
                                pose_3d_valid[:2, :] - mean_3d[:2, :])).sum()
    numerator = np.sqrt(numerator)
    denominator = np.sqrt(np.trace(np.dot((pose_2d_valid[:2, :] - mean_2d[:2, :]).transpose(),
                                          pose_2d_valid[:2, :] - mean_2d[:2, :])).sum())
    mean_2d = np.array([mean_2d[0, 0], mean_2d[1, 0], fx * input_scale / stride])
    mean_3d = np.array([mean_3d[0, 0], mean_3d[1, 0], 0])
    translation = numerator / denominator * mean_2d - mean_3d

    if is_video:
        translation = current_poses_2d[pose_id].filter(translation)
    for kpt_id in range(19):
        pose_3d[0, kpt_id] = pose_3d[0, kpt_id] + translation[0]
        pose_3d[1, kpt_id] = pose_3d[1, kpt_id] + translation[1]
        pose_3d[2, kpt_id] = pose_3d[2, kpt_id] + translation[2]
    translated_poses_3d.append(pose_3d.transpose().reshape(-1))

`

Hi! These 3D coordinates are relative to the camera coordinate system. If you have camera pose wrt world coordinate system, you can apply it to the coordinates and get them wrt world coordinate system.

Thank you for your reply. I mostly didn't understand this part of the operation:
    numerator = np.trace(np.dot((pose_3d_valid[:2, :] - mean_3d[:2, :]).transpose(),                                 pose_3d_valid[:2, :] - mean_3d[:2, :])).sum()     numerator = np.sqrt(numerator)     denominator = np.sqrt(np.trace(np.dot((pose_2d_valid[:2, :] - mean_2d[:2, :]).transpose(),                                           pose_2d_valid[:2, :] - mean_2d[:2, :])).sum())     mean_2d = np.array([mean_2d[0, 0], mean_2d[1, 0], fx * input_scale / stride])     mean_3d = np.array([mean_3d[0, 0], mean_3d[1, 0], 0])     translation = numerator / denominator * mean_2d - mean_3d
My understanding is that the average of the 3D and 2D pose data is computed, then the sum of the Euclidean distances of the 2D and 3D pose coordinates to the corresponding averages is computed, and then a translation vector translation is computed for translating the 3D pose data. But I don't understand the purpose of this.

I mainly want to use this method for multiplayer 3d pose display, I am currently able to get the pixel coordinates of a 2d pose, and also the coordinates of a 3d pose relative to its own origin, and the internal and external parameters of the camera. I would like to use this part of the code to transfer the 3d pose to the world coordinate system, how would I apply this method please?

But I don't understand the rationale of this.

It is the solution for minimum projection error, you can check the 'Monocular 3D Human Pose Estimation In The Wild
Using Improved CNN Supervision' paper for the details. 3D keypoints coordinates are predicted relative to root keypoint coordinate system. If the root keypoint is translated in a proper place in 3D space, then all pose is in correct 3D coordinates (in camera coordinate system). So we find such translation vector, which minimizes 3D->2D keypoints projection error given camera intrinsics. Given camera pose in world coordinate system, just apply it to keypoint coordinates to move them from camera coordinate system to world coordinate system.

Thank you, I understand. I would like to ask one last question, regarding the following code:
mean_2d = np.array([mean_2d[0, 0], mean_2d[1, 0], fx * input_scale / stride])
If the 'mean_2d' I have calculated already has coordinates in the original image dimensions, does 'fx' still need to be scaled, or can I directly use 'fx'?

You do not need to scale focal length in case of mean_2d in the original image space.

Thank you so much for the explanation. I'll close the issue.

Great, that it helped!


Run XVR_Flink PostCommit


[4.0][Migrator] Add a stub for authored API change list to handle API changes that are not detected automatically. (#9477)

Train: Tioga swift-4.0-branch
Explanation: We need this change to teach migrator to handle API changes that cannot be automatically detected by swift-api-digester. Typically, these changes are from SDK overlay.
Scope of Issue: Migrator
Origination: Driven by known SDK overlay changes.
Risk: Low
Reviewed By: Argyrios Kyrtzidis
Testing: Regression test added

@swift-ci please test


Reid performance improvements

This changes affect:

Added identification and reidentification blobs compiled for 4 shaves to better distribute resources
Infer only on most recent frames while still keep in sync with source video / camera
Decreased latency dramatically

Note: use main_threaded.py for testing this PR. I haven't worked on original main.py script for current changes.

Thanks!  Trying now.

@Luxonis-Brandon have to update requirements.txt - I've used some recent features.
One minute

Got it thanks.  Actually I was just about to ask as I saw:
Traceback (most recent call last):
  File "main_threaded.py", line 346, in <module>
    app.run()
  File "main_threaded.py", line 272, in run
    pipeline = create_pipeline()
  File "main_threaded.py", line 65, in create_pipeline
    detection_nn.setNumInferenceThreads(2)
AttributeError: 'depthai.MobileNetDetectionNetwork' object has no attribute 'setNumInferenceThreads'

And was wondering if I hadn't properly updated yet.

Updated depthai dependency in requirements.txt. PR is ready now

Thanks trying it now.

Just tried real quick.  Could be a PEBKAC error:
git pull
remote: Enumerating objects: 4, done.
remote: Counting objects: 100% (4/4), done.
remote: Total 4 (delta 3), reused 4 (delta 3), pack-reused 0
Unpacking objects: 100% (4/4), done.
From https://github.com/luxonis/depthai-experiments
   30dbbf2..78e211e  reid_performance_improvements -> origin/reid_performance_improvements
Updating 30dbbf2..78e211e
Fast-forward
 gen2-pedestrian-reidentification/requirements.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
brandongilles@Leeroys-MBP-2 gen2-pedestrian-reidentification % python3 -m pip install -r requirements.txt
Defaulting to user installation because normal site-packages is not writeable
Looking in indexes: https://pypi.org/simple, https://artifacts.luxonis.com/artifactory/luxonis-python-snapshot-local/
Requirement already satisfied: opencv-python==<IP_ADDRESS> in /Users/brandongilles/Library/Python/3.8/lib/python/site-packages (from -r requirements.txt (line 1)) (<IP_ADDRESS>)
Requirement already satisfied: imutils==0.5.4 in /Users/brandongilles/Library/Python/3.8/lib/python/site-packages (from -r requirements.txt (line 2)) (0.5.4)
Collecting depthai==<IP_ADDRESS>+5cd4297f35cd44744886b6cf52c7413bbb4bc8c3
  Downloading https://artifacts.luxonis.com/artifactory/luxonis-python-snapshot-local/depthai/depthai-<IP_ADDRESS>%2B5cd4297f35cd44744886b6cf52c7413bbb4bc8c3-cp38-cp38-macosx_10_9_x86_64.whl (4.5 MB)
     |████████████████████████████████| 4.5 MB 1.9 MB/s
Requirement already satisfied: numpy>=1.17.3 in /Users/brandongilles/Library/Python/3.8/lib/python/site-packages (from opencv-python==<IP_ADDRESS>->-r requirements.txt (line 1)) (1.19.5)
Installing collected packages: depthai
  Attempting uninstall: depthai
    Found existing installation: depthai <IP_ADDRESS>+e1e2f25e12fb77c7e9d4392ecc0c338e1a4117c7
    Uninstalling depthai-<IP_ADDRESS>+e1e2f25e12fb77c7e9d4392ecc0c338e1a4117c7:
      Successfully uninstalled depthai-<IP_ADDRESS>+e1e2f25e12fb77c7e9d4392ecc0c338e1a4117c7
Successfully installed depthai-<IP_ADDRESS>+5cd4297f35cd44744886b6cf52c7413bbb4bc8c3
WARNING: You are using pip version 20.3.1; however, version 21.0.1 is available.
You should consider upgrading via the '/Library/Developer/CommandLineTools/usr/bin/python3 -m pip install --upgrade pip' command.
brandongilles@Leeroys-MBP-2 gen2-pedestrian-reidentification % python3 main_threaded.py -vid ./input.mp4
depthai module:  /Users/brandongilles/Library/Python/3.8/lib/python/site-packages/depthai.cpython-38-darwin.so
framerate:  29.97002997002997
Creating pipeline...
Creating Person Detection Neural Network...
Creating Person Reidentification Neural Network...
Pipeline created.
Starting pipeline...
[14442C107138BFD200] [4756.993] [NeuralNetwork(5)] [warning] Number of inference threads assigned for network is 1, assigning 2 will likely yield in better performance
[14442C107138BFD200] [4756.993] [NeuralNetwork(5)] [warning] Network compiled for 4 shaves, maximum available 16, compiling for 8 shaves likely will yield in better performance
[14442C107138BFD200] [4756.994] [DetectionNetwork(0)] [warning] Network compiled for 4 shaves, maximum available 16, compiling for 8 shaves likely will yield in better performance
[14442C107138BFD200] [4757.003] [NeuralNetwork(5)] [warning] The issued warnings are orientative, based on optimal settings for a single network, if multiple networks are running in parallel the optimal settings may vary
[14442C107138BFD200] [4757.003] [DetectionNetwork(0)] [warning] The issued warnings are orientative, based on optimal settings for a single network, if multiple networks are running in parallel the optimal settings may vary
2021-02-19 09:59:48.762 Python[50607:4668993] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'NSWindow drag regions should only be invalidated on the Main Thread!'
*** First throw call stack:
(
	0   CoreFoundation                      0x00007fff204806af __exceptionPreprocess + 242
	1   libobjc.A.dylib                     0x00007fff201b83c9 objc_exception_throw + 48
	2   CoreFoundation                      0x00007fff204a8a9a -[NSException raise] + 9
	3   AppKit                              0x00007fff22c4897d -[NSWindow(NSWindow_Theme) _postWindowNeedsToResetDragMarginsUnlessPostingDisabled] + 321
	4   AppKit                              0x00007fff22c336d5 -[NSWindow _initContent:styleMask:backing:defer:contentView:] + 1375
	5   AppKit                              0x00007fff22c3316f -[NSWindow initWithContentRect:styleMask:backing:defer:] + 42
	6   AppKit                              0x00007fff22f3d2c8 -[NSWindow initWithContentRect:styleMask:backing:defer:screen:] + 52
	7   cv2.cpython-38-darwin.so            0x000000010916bed5 cvNamedWindow + 677
	8   cv2.cpython-38-darwin.so            0x000000010916b79c cvShowImage + 188
	9   cv2.cpython-38-darwin.so            0x0000000109169c66 _ZN2cv6imshowERKNSt3__112basic_stringIcNS0_11char_traitsIcEENS0_9allocatorIcEEEERKNS_11_InputArrayE + 230
	10  cv2.cpython-38-darwin.so            0x00000001083cb39e _ZL18pyopencv_cv_imshowP7_objectS0_S0_ + 302
	11  Python3                             0x0000000107bf538a cfunction_call_varargs + 122
	12  Python3                             0x0000000107bf4dd6 _PyObject_MakeTpCall + 374
	13  Python3                             0x0000000107cd513c call_function + 652
	14  Python3                             0x0000000107cd16d6 _PyEval_EvalFrameDefault + 29782
	15  Python3                             0x0000000107bf578d function_code_fastcall + 237
	16  Python3                             0x0000000107bf81e2 method_vectorcall + 322
	17  Python3                             0x0000000107bf5124 PyVectorcall_Call + 100
	18  Python3                             0x0000000107cd1a34 _PyEval_EvalFrameDefault + 30644
	19  Python3                             0x0000000107bf578d function_code_fastcall + 237
	20  Python3                             0x0000000107cd5012 call_function + 354
	21  Python3                             0x0000000107cd16b9 _PyEval_EvalFrameDefault + 29753
	22  Python3                             0x0000000107bf578d function_code_fastcall + 237
	23  Python3                             0x0000000107cd5012 call_function + 354
	24  Python3                             0x0000000107cd16b9 _PyEval_EvalFrameDefault + 29753
	25  Python3                             0x0000000107bf578d function_code_fastcall + 237
	26  Python3                             0x0000000107bf81e2 method_vectorcall + 322
	27  Python3                             0x0000000107bf5124 PyVectorcall_Call + 100
	28  Python3                             0x0000000107d7700a t_bootstrap + 74
	29  Python3                             0x0000000107d28829 pythread_wrapper + 25
	30  libsystem_pthread.dylib             0x00007fff2030e950 _pthread_start + 224
	31  libsystem_pthread.dylib             0x00007fff2030a47b thread_start + 15
)
libc++abi.dylib: terminating with uncaught exception of type NSException
zsh: abort      python3 main_threaded.py -vid ./input.mp4

Tried it, and runs fine. Compiling for 6 shaves for video case improves FPS only by 1 fps, so disregard.
Remove old blobs though.

@Luxonis-Brandon retesting on mac - didn't experience on Ubuntu.

@szabi-luxonis there is also the -cam option which would require another set of blobs so I just went with "common" denominator (not really but close)
Will remove the other main.py and blobs

@Luxonis-Brandon retesting on mac - didn't experience on Ubuntu.

Yes, it seems like it is a Mac-specific thing based on the error message.  It actually seems to be something with how Python is interacting with windowing in macOS and how the threading is being handled.

Latest changes completely remove previous code and blobs in favor of this implementation.

Modified the example to use source (high resolution) frames by default, can be disabled using -lq
Added -w parameter to set output visualization width
Fixed macOS crash
Updated README.md


[caguero] Add pinger plugin to wamv_pinger.xacro

After #526 , the pinger plugin wasn't working. This pull request adds the plugin to the xacro macro.
For testing, follow the instructions detailed in #531 .

I'm seeing output now, but the range is around 42 rather than 29. Is that OK?


Should misaligned loads/stores panic?

Currently, load and store instructions can read/write from any address. My understanding is that this may be inefficient on x86, but it won't cause a crash. However, it can cause a fault on ARM AFAIK.
There's a question as to what UVM should do under the hood. If we want to allow unaligned load/stores, we may have to test for alignment, and use different instructions depending if the load/store is aligned or not.
If we disallow unaligned loads/stores, then we still have to test if the address if aligned or not, and fault if it's not. A JIT compiler may be able to optimize away some or most of these tests.
I'm leaning towards probably disallowing misaligned loads/stores because although this is more strict, it has the advantage that it will force people to write more performant code. Also, if loads and stores are expected to be aligned, then even if we have to test for alignment, we can assume that the address will in fact be aligned (branch not taken), whereas if we allow misaligned loads/stores, we can make no such assumption.

I implemented the enforcement of address alignment in: 38002acd1d1f15a3f2017bb1c435688172aead96
It's probably better to be strict because we can easily remove constraints like these later on, but we can't easily add constraints once people have written code assuming that alignment doesn't matter.


Update npm dependencies

Codecov Report

Merging #385 (8f4c512) into master (4c6a7a0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #385   +/-   ##
=======================================
  Coverage   62.77%   62.77%
=======================================
  Files          14       14
  Lines         771      771
=======================================
  Hits          484      484
  Misses        277      277
  Partials       10       10

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Update documentation to clarify ban of multi-line checklist items (if they are tasks)

Issue tracker is ONLY used for reporting bugs. New feature ideas should be discussed in the ideas section. Please use the Q&A section for supporting issues. Please use the search function.
If you encountered the issue after you installed, updated, or reloaded the tasks plugin, please try restarting obsidian before reporting the bug.
If you want to report a bug, please follow the guide lines below to help me resolve it.
Expected Behavior

Current Behavior

See this thread, with images. https://forum.obsidian.md/t/checklist-with-line-breaks-markdown-rendering-broken/29519
Steps to Reproduce

See thread. When Tasks plugin is disabled checklist items render correctly.
Context (Environment)

Obsidian version: 0.13.14
Tasks version: 1.4.1
[ ] I have tried it with all other plugins disabled and the error still occurs

Possible Solution

As per the documentation at https://schemar.github.io/obsidian-tasks/getting-started/

Tasks only supports single-line checklist items. You cannot have checklist items that span across multiple lines.
This works:

- [ ] This is a task
    - This is a sub-item
    - Another sub-item
    - [ ] And a sub task
        - Even more details

The following does not work:

- [ ] This task starts on this line
      and then its description continues on the next line

I understand the caveat but thought it only applied in a Tasks view. When Tasks is enabled and not in a Task view then rendering of a checklist item with a newline no longer works. Is that limitation what is expected by the warning message?

Ah, sorry. I misunderstood. Normal checklist items should of course not be affected. However, it depends on your settings of the Tasks plugin. If you don't use a "Global task filter", then every checklist item is a "task". In that case, you can no longer have multi-line checklist items, as you cannot have multi-line tasks. How did you configure Tasks?

I understand now. My misunderstanding then. I did not expect a document that gets scanned for tasks in the document to alter the behaviour of how those items display. I'm not sure if I'm clear on the language here... Generated lists I understand fall under the no multi-line rule. Non-generated lists (i.e.: checklist items I type in AND those items get scanned into a task list) also fall under the no multi-line rule – which I did not expect.
Maybe a line in the document of the feature would help?

Definitely. Thanks for the feedback! Do you think you have the time to update the documentation? :blush:
The current warning lives here: https://github.com/schemar/obsidian-tasks/blob/main/docs/getting-started/index.md?plain=1#L51

Is there not a potential way to make this work? For example, expecting an indentation line up to match, or even some sentinal value like
- [ ] TASK>>>
Multi
Line
Task
<<<TASK

I can see why it can't work that way by default because the last task in a task list would naturally include the rest of the document. But it would be nice if there was a (potentially toggleable) way to achieve multiline task bodies, even if it required manually doing so :)

@gwww I never saw that you made changes. I now created a PR form your repo to mine to merge your changes.

@zachdaniel technically, everything is possible. It's just too time consuming on implementation and maintenance for me to do at the moment.

Makes sense! I wonder if there was a way to offload that to the user? Like make a configurable "task regex" or something? And then if that's wrong then it's on the user to deal with.

I'm hesitant to give the user a foot gun. Though, who am I to withhold it. I am only worried about lots of "bug" reports due to malformed regexes :monocle_face: I'll think about it.

I'm hesitant to give the user a foot gun. Though, who am I to withhold it. I am only worried about lots of "bug" reports due to malformed regexes 🧐 I'll think about it.

As a user who found this issue when trying to figure out why my multi-line task wasn't getting rendered, I would endorse providing said "foot gun" behind an appropriately worded toggle/setting marked experimental and/or has a warning saying "I know this allows me to shoot myself in the foot, but I'm a rebel!" ... or some such.

I’ve taken over stewardship of Tasks from @schemar.
I do agree that Tasks swallowing text when rendering the markdown in Reading mode is a serious limitation which needs to be fixed. It is also highly non-trivial to fix it in the current code base.
If datacore turns out to be as effective as I hope, there may be a small chance that it could be adopted and enable a fix of the rendering.
This issue is about documenting the issue, however, and the limitation is already documented, so I will later close this issue.
But having said that, the time required every day to do user support for Tasks now is already a problem, as its use has grown.
So for the avoidance of doubt, I would not support any changes that would in any way be described as a foot gun.

The documentation for this limitation is at:
https://obsidian-tasks-group.github.io/obsidian-tasks/getting-started/#limitations-and-warnings


[Task]: Deduplicate filepicker component in CE & EE

Closing it coz of change in release plan. Tracking it in epic https://github.com/appsmithorg/appsmith/issues/20460


Nested Routes + useHead not working correctly

Environment

Operating System: Darwin
Node Version:     v20.7.0
Nuxt Version:     3.7.4
CLI Version:      3.9.0
Nitro Version:    2.6.3
Package Manager<EMAIL_ADDRESS>Builder:          -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/nuxt3-nested-routes-head?file=app.vue
Describe the bug
the body attributes are merged on nested routes.
<body class="scroll no-scroll">
useSafeHead({
bodyAttrs: {
class: () => (scroll.value ? 'scroll' : 'no-scroll'),
}
})
Additional context
No response
Logs
No response

cc: @harlan-zw

Any progress on this already? I'm running into the same issue. Also found @harlan-zw 's comment in another topic about this issue, with a workaround for as long as the bug still exists. Perhaps it is a good idea to post it here. This works for now:
onMounted(() => {
Object.values(injectHead()._dom?.sideEffects || {}).forEach((fn) => fn());
});

Duplicate of https://github.com/nuxt/nuxt/issues/25938 which will be fixed in the next Unhead release :slightly_smiling_face:


numba v0.61.0rc1 installation fails because missing MS Visual C++

I tried to install numba v0.61.0rc1 via pip on my windows 10 system. It fails as following:
      UPDATING build\lib.win-amd64-cpython-313\numba/_version.py
      set build\lib.win-amd64-cpython-313\numba/_version.py to '0.61.0rc1'
      running build_ext
      building 'numba._dynfunc' extension
      error: Microsoft Visual C++ 14.0 or greater is required. Get it with "Microsoft C++ Build Tools": https://visualstudio.microsoft.com/visual-cpp-build-tools/
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for numba
Failed to build numba
ERROR: ERROR: Failed to build installable wheels for some pyproject.toml based projects (numba)

Of course I have a MS Visual C++ runtime installed (system wide). Tried also with Cpython 3.12.7 - same problem.
It that known and exist a workaround?

@Alpengreis thank you for reporting this, can you list the command you used to attempt the install? I don't think I have seen anyone report this error before.

@esc
Sure, it was:
.venv\Scripts\python.exe -m pip install numba==0.61.0rc1
That means it was within a virtual env. Just for info: I can install the v0.60 with the same command without any problem.

I have the same problem

Any news here?

Any news here?

We are currently working on producing binary artifacts for the RC, we hope that this will eliminate this issue by no longer needing any compilers during installation.

Any news here?

We are currently working on producing binary artifacts for the RC, we hope that this will eliminate this issue by no longer needing any compilers during installation.

@Alpengreis the windows wheels for Numba 0.60.0rc1 have just arrived on PyPI, you should be able to pip install them now.

@esc
Was successful now, thanks very much!

@esc
Was successful now, thanks very much!

Wonderful, thank you for reporting back. Will close issue now, if there are still problems for anyone, please comment below and we will re-open this issue. Thank you all for helping to improve Numba.


Set default WSL path to distro $HOME

Set default WSL path to distro $HOME
The default path for WSL distros is currently the Windows user path (%USERPROFILE%) but it's may be better to set it to the distro $HOME.
Proposed technical implementation details (optional)
I don't know if it's possible, but ideally in the WslDistroGenerator.cpp the command line should be conditionally set to "wsl.exe -d " + distName if startingDirectory is set in profile.json, "wsl.exe ~ -d " + distName otherwise.
Another possibility is to add a boolean setting UseWSLHome on the wsl profile.json
Thank you.

Two ways:

Add this command in .bashrc file cd ~
Or
Run this command line wsl -d Ubuntu sh -c "cd ~;exec $SHELL"

I already set "commandline" : "wsl.exe ~ -d Ubuntu" in my profile profile.jsonand it works, but it's a workaround and a standard method should be better.

As said in this issue, you can set the startingDirectory to //wsl$/distro/home/username

Ok it's another good workaround
Thank you @leoniDEV

Anyway I agree with you that the default startingDirectory for WSL should be the $HOME of the WSL user and not the %USERPROFILE% of the Windows user

Anyway I agree with you that the default startingDirectory for WSL should be the $HOME of the WSL user and not the %USERPROFILE% of the Windows user

Agreed, at least going by the principle of DWIM. if we're launching WSL, it should default to normal *nices behaviour of logging into users home directory within wsl, not some other directory

Is this the default directory for WSL distros when opening with ConHost.

Is this the default directory for WSL distros when opening with ConHost?

Don't understand.  If I login into wsl using putty, mintty or any other terminal software, the shell prompt comes at /home/<userid> that you created at first install of wsl.
In MS terminal, the default seems to be /mnt/c/users/profile or something.  This is generally not the default way

Is this the default directory for WSL distros when opening with ConHost?

Don't understand. If I login into wsl using putty, mintty or any other terminal software, the shell prompt comes at /home/<userid> that you created at first install of wsl.
In MS terminal, the default seems to be /mnt/c/users/profile or something. This is generally not the default way

When we launch wsl.exe without ~  argument it's run WSL at the working directory and when we add ~ in the argument list of WSL it's launched at the distro $HOME (/home/<userid>) directory.
If we launch <distro>.exe of a distro downloaded from the store or when we open the distro app from the start menu the directory is also set to $HOME.
I hope I answered to your question @carlos-zamora

It looks like you've got a workaround here. For the rest, we'll track them in /dup #878 and #592

From Terminal Settings (ctrl + ,):

Starting directory should be: \wsl$\YOUR_DISTRIB\home\YOUR_USERNAME
You can access the path in Explorer.

Well, I am facing the same issue, and I launch my VM directly via the wsl command rather than opening a Terminal Profile, there is the workaround of typing wsl --cd ~ but maybe this startDirectory option could be added to wsl.config to not pollute my .bashrc.


Multiple Time picker not working on IE

Hi,
I need to use multiple dynamic instances of the time picker in my mvc razor page. It works great on firefox and chrome but on IE 11, clicking on the first time picker shifts focus and opens pop up on the last time picker in the page.  Thank you for any help and Thanks for the helpful plugin!

Please provide a jsfiddle of your issue. You can use this fiddle as a base.

Hi, I've updated the jsfiddle. on IE, the issue is on single click, but on other browsers it can be re produced by double clicking on the text box

Can you provide me with a link to your fork?

sorry about that, please find it here : http://jsfiddle.net/0Ltv25o8/1562/

your fiddle does the same thing for me in both IE11 and Chrome. Your init function must be assigning something incorrectly

Hi,
the issue was that i was doing a common init for all elements whose id's end with jobtime, once i changed it to initialise each one separately it's working fine. Thank you for the help! I have the updated working version here : http://jsfiddle.net/0Ltv25o8/1564/.


Update gitleaks.toml

We tested this repo in gitlab and it fails to see hardcoded keys with name API-XXXX and api-XXXX.These keys words need to be detected so adding this to fix the issue
Checklist:

[ ] Does your PR pass tests?
[ ] Have you written new tests for your changes?
[ ] Have you lint your code locally prior to submission?

Closing this PR since it's duplicated.


[components] Fix Grid export

Current Behavior

Expected Behavior

Screenshots/Logs

Environment

Host OS: Mac Linux Windows
Platform: Docker or Kubernetes
Meshery Server Version: stable-v
Meshery Client Version: stable-v

Contributor Guides and Resources

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Layer5 site in Figma (open invite)
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

@nebula-aac i would like to work on this.


internal: document review requesting etiquette

don't feel obliged to quickly review every PR assigned to you
so that other folks can notify you about interesting PRs without
thinking to much about creating additional work for you

WDYT?
The problem I want to solve is that we have a sizable queue of PRs, and it's unclear who is default reviewer for them. Ie, when I have my slot for reviews, it's unclear to me which PRs already have someone intending to r+, and which PRs are blocked on no one feeling confident enough to review them.
I wish I could just sort by PRs assigned to me, and then triage the PRs without updates.

This might be a good place to mention not pinging people in PR messages (e.g. with "r? @flodiebold") since that leads to spam whenever someone pulls the PR merge commit into their fork? :grimacing:

bors r+


Masked GTrXL

Write a masker version of GTrXL.
Base on Ray's implementation: https://github.com/ray-project/ray/blob/master/rllib/models/torch/attention_net.py#L261

Putting this on pause to work other issues. This is a reminder post for when I pick back up.
In issues/iss104 there are a few attempts at creating the GTrXL network that I started after punchclock/nets/gtrxl.py got too complicated. The latest-developments are in hearts_example.py, which modifies models from a public GitHub repo. See the file doc string for details.
As of today, the script runs and appears to learn when using a test environment (MaskRepeatAfterMe).
Architecture: An action mask wrapper (class TorchMaskedActionsWrapper(TorchModelV2)) wraps any generic model. The .forward method strips out the action mask from the observation, then passes the real observation to the wrapped model. However, the observation space of the action mask wrapper itself includes the action mask. To add in the attention net, another wrapper is used that is a child class of TorchMaskedActionsWrapper, called TorchMaskedActionsAttentionWrapper. The attention wrapper creates a wrapped environment that is in-turn wrapped with Ray's automatic GTrXLNet by using the "attn_cls" arg.


Enabling Change Tracking for the User

Currently, we expect that a user has already enabled change tracking on the database and on the table that the function is being triggered on. In the future, we'd like to support an option that the user can set which indicates they would like us to enable change tracking for them.

Closing trigger binding related issues until we revisit support for that


Query Error In Azure WAF Log4j CVE-2021-44228 hunting

Describe the bug
A clear and concise description of what the bug is.
https://github.com/Azure/Azure-Sentinel/blob/master/Hunting Queries/AzureDiagnostics/WAF_log4j_vulnerability.yaml
To Reproduce
Steps to reproduce the behavior:

Go to 'Sentinel->Hunting->Azure WAF Log4j CVE-2021-44228 hunting'
Run 'Azure WAF Log4j CVE-2021-44228 hunting'
See error:

'where' operator: Failed to resolve table or column or scalar expression named 'originalRequestUriWithArgs_s'.

Expected behavior
The query result without error,
Possible solution:
Maybe add something like:
extend originalRequestUriWithArgs_s = column_ifexists("originalRequestUriWithArgs_s", "")

The query has been updated now to possibly fix the issue. Please check.


Reload certificates from disk automatically + bump to v2.7.2

Branch comparison between the upstream v2.7.2 tag and this PR:
https://github.com/prometheus/prometheus/compare/v2.7.2...simonpasquier:bump-prometheus-common
This includes https://github.com/prometheus/common/pull/173 which makes sure that Prometheus reloads certificate files from disk automatically whenever they are rotated.

cc @brancz


remove .Net 5 requirement

Dotnet 5 is not needed, and should not be included in the pipeline.
This PR makes the learn docs mirror what's stated at the GitHub repo as well as the Marketplace documentation

@elazark
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@bjompen I am reaching out to the relevant stakeholder sot see if they are ok with removing this.

@bjompen I spoke with the appropriate stakeholders and they informed me that .net 5 is needed and that it should not be removed. Thank you for taking the time to make this suggestion but at this time I can't remove it form the yaml file.

#please-close


Feature: Screenshot URLS from urls.txt file without port knocking

Awesome work mate, as always! Legend!
Would be alright to somehow dont make aquatone to scan for new urls once we provide the full URL with the following port ? ie. https://secret.google.com:9001
Reason why I'm saying that, doing a raw xml from masscan/nmap can sometimes miss things due server_name just stuff, however, would be nice to use aquatone and its amazing screenshot and reporting js to just shoot a screenshot on a giving.
Hope it makes sense.
Cheers

nevermind ... it actually does just as that... i got confused because the "ports" menu banner still showing up when run as targets.txt
Cheers

Yeah, I can see why showing the port list would be confusing to see when it shouldn't be scanning any ports.
I will look into only showing the port list if it got any targets that need to undergo portscanning.


start up issue

File "/home/andy/Documents/python_algorithm/tdameritrade4.py", line 30, in
td_client=tdameritrade.get_login()
File "/home/andy/Documents/python_algorithm/tdameritrade.py", line 39, in get_login
td_client = TdAmeritradeClient(credentials=td_credentials)
File "/home/andy/anaconda3/lib/python3.8/site-packages/td/client.py", line 36, in init
self.td_session = TdAmeritradeSession(td_client=self)
File "/home/andy/anaconda3/lib/python3.8/site-packages/td/session.py", line 43, in init
logging.basicConfig(
File "/home/andy/anaconda3/lib/python3.8/logging/init.py", line 2005, in basicConfig
raise ValueError('Unrecognised argument(s): %s' % keys)
ValueError: Unrecognised argument(s): encoding
The next will execute fine. On start up, I found this error both times I tried running the code.  I am pretty sure I have all the dependencies.

I believe he just merged a change into master that someone did a PR for that fixes this.


Move off of webpacker

Seems like the two best options are either Shakapacker or jsbundling-rails.
Shakapacker

Based off of webpacker v6
Most compatible with Webpacker apps (easier transition)
Uses webpack
Well maintained with activity from the Rails team
Better for larger, more complex projects
Less popular than jsbundling-rails
Not Rails default
More likely to eventually become unmaintained since it is not being maintained by the Rails team

Jsbundling-rails

Rails default
Well maintained by the Rails team
Uses webpack
Probably better for smaller projects
Not built off of webpacker so likely harder transition
Simpler than Shakapacker
May be easy to transition off of webpacker if there aren't too many custom webpacker configurations
Less likely to become unmaintained (is being maintained by Rails team)

closed by #675


Overlapping dependency names

What Git revision are you using?
v1.0.23-alpha
What operating system are you using, and which version?
N/A
Describe what the problem is?
Currently build system "normalizes" dependency name to <owner>-<repo> which can create collisons. Suppose following aiken.toml file:
[[dependencies]]
name = "aiken-collision-foo-bar/baz"
version = "cca485f3c4d192893ea3118b6e49afab2bdd4d17"
source = "github"

[[dependencies]]
name = "aiken-collision-foo/bar-baz"
version = "97cd61345bcc8925c521b30d0f354859eb0148cd"
source = "github"

clearly packages are different but during aiken build they will create the same ./build/packages/aiken-collision-foo-bar-baz directory and override one another.
    Compiling aiken-collision-foo-bar/baz cca485f3c4d192893ea3118b6e49afab2bdd4d17 (<path>/build/packages/aiken-collision-foo-bar-baz)
    Compiling aiken-collision-foo/bar-baz f4d549762a4d82644b7587e8dc659df8aea89ba7 (<path>/build/packages/aiken-collision-foo-bar-baz)
        Error aiken::module::duplicate

what's more, there's no actual duplicate between these two packages.
What should be the expected behavior?
name encoding both owner and repo is a bad idea imo, but this can still be avoided by using different separator, one that's illegal in both organization and repository name on github. aiken build should create two different directories for both packages.

I'm going to convert this to a discussion. We are aware of this and what we decide to do for package management could also affect this.


Data service security not working without adding rampart module to axis2.xml

Description:
https://ei.docs.wso2.com/en/7.1.0/micro-integrator/develop/creating-artifacts/data-services/securing-data-services/ doc need to mention that users need to add Rampart as a module as mentioned in the following doc https://docs.wso2.com/display/EI611/Engaging+Modules

Created an issue to validate the documentation in MI. https://github.com/wso2/micro-integrator/issues/2031

closing the doc issue as we have created an issue in the product.

closing the doc issue as we have created an issue in the product.


Cytoscape installation fails without warning

What steps will reproduce the problem(s)?
1. Download AltAnalyze for a machine running Windows and install a species
database. Cytoscape will be downloaded and extracted automatically. However,
Cytoscape will prompt will either simply fail to open or prompt an
incompatibility warning (32bit or 64bit incompatibility).
2. Download AltAnalyze for a machine running OSX and install a species
database. Cytoscape will be downloaded and extracted automatically. However,
Cytoscape will fail to open when the Cytoscape.app is selected without warning.
Only opening it in a terminal window with the command "./Cytoscape.app" or
opening the .jar file will initiate Cytoscape.

Cause of the error:
(Windows) After the cytoscape.zip file is downloaded, zip extraction creates
faulty binary files due to an apparent problem with Python's builtin zipfile
library.
(Mac) Unlike Windows, zipfile propperly extracts the cytoscape.zip download,
however, permissions on the directories cause restricted access to files,
unless permissions are changed (chmod -R 777 directory).

Solution in next release:
(all operating systems) The Cytoscape folder will be distributed as a
cytoscape.gz.tar file which is extracted properly by the AltAnalyze supported
extraction methods. Cytoscape was shown to run properly when extracted from
this file.
(Mac) Permissions on the Cytoscape directory have been changed.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 26 Mar 2011 at 6:49

Note: Cytoscape integration no longer supported in AltAnalyze version 2+