Cannot log in with Azure credentials anymore

Description
Hi!
This is my first issue I post here (and the first serious issue I ever have had with dbeaver after having used it in several years), please bear with me. After upgrading dbeaver to the most recent version, I cannot connect to my database anymore. It looks as if dbeaver uses some kind of internal browser for authentication. But when I enter my credentials I seem to be in a loop. The authentication starts again - and that never ends, I cannot connect to my azure database at all. Sometimes this browser showed a error text saying "Cannot connect to localhost" although my connections set up with Azure database host names which point to their respective IP addresses.
Is there a workaround for this? I tried in the preferences Web browser => use Chrome, but that did not work. Can I change anywhere to use Chrome again?
Can you please fix? dbeaver became unusable for me :(
DBeaver Version
23.2.2
Operating System
Windows 10
Database and driver
I use Azure SQL driver with an Azure database
Steps to reproduce
Try to connect to an Azure database with azure username and password.
Intern browser opens and asks to log in. Window opens to confirm idendity (2FA).
I get the message "Authentication complete. You can close the browser and return to the application."
I click on the database and it asks me again to log in.
Additional context
No response

@NicolWeis does it help?

@NicolWeis Could you please send me the log files? You can find instructions here: https://dbeaver.com/docs/wiki/Log-files/. Thank you.

It's been a while since no update here. If the issue is still actual please let me know and provide additional information.


[Feature]: Add support for Mistral AI API

Feature Request
Mistral is offering an API now (based on OpenAPIs API). Interesting thing is that mistral-small (Mixtral-8x7b) outperforms ChatGPT 3.5 and is cheaper (0,0018 €/1k Tokens instead of 0,0020 €/1k Tokens for gpt 3.5-turbo). So it makes only sense to support it as an alternative.
Note: Currently use is NOT possible with the existing chat-node:
The API fails if you add the fields "frequence_penalty" and "presence penalty" and there is no way to not send them via Rivet Chat node.
Example:
{
    "object": "error",
    "message": "{\"detail\":[{\"type\":\"extra_forbidden\",\"loc\":[\"body\",\"frequency_penalty\"],\"msg\":\"Extra inputs are not permitted\",\"input\":1,\"url\":\"https://errors.pydantic.dev/2.5/v/extra_forbidden\"}]}",
    "type": "internal_error_proxy",
    "param": null,
    "code": "1000"
}

Code of Conduct

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

Link to Mistral documentation:
https://docs.mistral.ai

Idea: Add all integrations that use the Open AI API format as templates for the chat-node
So you would first select the provider, e.g. "OpenAI" or "Mistral" and than depending on that provider you have slightly different options for the node. E.g. no presence_penalty + frequency_penalty parameters for Mistral

Overloading the chat node like that would make it ever more complicated than it already is unfortunately... I'll have to think of a good solution that doesn't overcomplicate the node...

In 70fa4cea9f25a6c82a2c9ec7fa1f7ef8d1624ec5 you can remove the presence penalty and frequency penalty params and they don't get sent to the API. By default they're unset, as well.


Make currently-executing test's file path available as jest.currentTestPath()

It is useful to have the current test filename available in the test reporter environment. For instance, we use the JUnitReporter with our CI server, and because each test file is run in its own process, this enables us to output a results file named for each test:
// This script enables JUnit XML output by the test runner to be parsed and reported by Bamboo
jasmine.VERBOSE = true;

require('jasmine-reporters');

// massage the test file path into an output filename
var relativeTestPath = 'test' + _jestTestFilePath
    .replace(process.cwd(), '')
    .replace(/\//g, '.')
    .replace('__tests__.', '')
    .replace(/\.jsx?$/, '');

var reporter = new jasmine.JUnitXmlReporter('test_results', true, true, relativeTestPath, true);
jasmine.getEnv().addReporter(reporter);

I looked a bit into how to best test this but it seemed … complicated :) I would appreciate any thoughts on a nice way to do this? One possibility would be to add it to the testEnvData but I feel like that would be a misuse. Otherwise it seems like the way would be to run an end-to-end test with a setupTestFrameworkScriptFile configuration parameter, and assert the test filename therein. I'd be happy to add one if that seems like the right approach.

I can see the need for this, however, I'm thinking whether the _jestTestFilePath "injected global" is the best approach here? We already expose Jest's runtime for setupTestFrameworkScriptFile, so alternative way would be jest.getTestFilePath(). OTOH, it may unnecessary bloat the runtime's API, and maybe this injected global is OK-ish.
@vjeux @amasad @zpao, what you guys think?

I would be very happy for the file path to available via a method — I think I did the simplest thing that could work but I'm not super-familiar with the jest gestalt :) I can definitely refactor this to expose on jest directly.

Yeah, let's just add the jest.getCurrentTestPath(). You may pass the filePath either as a parameter to the loader (or alternatively, just attach it to the environment object, and then you will be able to access it in the module loader constructor). Then add the method here: https://github.com/facebook/jest/blob/master/src/HasteModuleLoader/HasteModuleLoader.js#L956

Thanks!


Broken after libevent 2.1.11-1 update due to soname bump

chromium fails to launch with the following error after updating libevent from 2.1.10-1 to 2.1.11.-1 due to the soname being bumped from 6 to 7.
/usr/lib/chromium/chromium: error while loading shared libraries: libevent-2.1.so.6: cannot open shared object file: No such file or directory
Package built from ungoogled-chromium-archlinux on AUR, version 76.0.3809.100-1.

you have to recompile chromium for it to be linked to the new version of libevent.

Wait, is that it? I've always thought that soname updates required a package update... I'm embarrassed.
Sorry if I wasted anyone's time!


Add support for @throws

Description
Given a function has a @throws JSDoc tag, it would be helpful to apply some analysis and raise a compiler warning (or, if configured, an error) when the caller does not handle the error and it doesn't declare its own @throws tag.
Examples
/**
 * @throws {SomeException}
 */
function myFunction() {
  // some logic
  // then for some reason we throw an exception
  if (somethingWentWrong) {
    throw new SomeException("something happened");
  }
  // some more logic
}

function potentialMess() {
  myFunction(); // Compiler error: 'myFunction' may throw `SomeException`. ts(9876)
}

/**
 * @throws {SomeException}
 */
function letItBubble() {
  myFunction(); // No compiler errors
}

function aFunctionThatHandlesTheException() {
  try {
    myFunction(); // No compiler errors
  }
  catch (error) {
    // do something with the error
  }
}

Questions

Ideally, it would be even better if VSCode realizes that myFunction throws an error, and then @throws is not required for the analysis. Still, this approach would be useful when dealing with external code/interfaces.

It is my understanding that JSDoc's @throws is allowed once -- if this is correct, then VSCode could ignore this and accept multiple tags anyway? Otherwise, maybe we can use a different tag, e.g. @exception?

Duplicate of https://github.com/microsoft/TypeScript/issues/31329
Please don't open duplicate issues


Fix Android build error

Fixes #3 by removing the android directory, which is not required anymore in Flutter.
It was required for a short amount of time when web was new, however, Flutter will now simply ignore plugins without an android directory when building Android projects.

@florent37 Could you take a look at this / merge & publish by any chance?

k' I publish it, sorry for the delay


Allowing "apply false" to core plugin.

Expected Behavior
Allow the following code in build.gradle.kts:
plugins {
    signing apply false
}

Current Behavior (optional)
Error resolving plugin [id: 'org.gradle.signing', apply: false]
> Plugin 'org.gradle.signing' is a core Gradle plugin, which is already on the classpath. Requesting it with the 'apply false' option is a no-op.

Context
When we use Kotlin DSL, we need gradle generate stubs for script executing. In root project I have to use allprojects{} and subprojects{} block. If we don't add the plugin to plugins{} the stubs will not be generated. Otherwise the plugin will generate unused tasks. It's a problem, it mess up the task hierarchy.

gradle/kotlin-dsl-samples#916

It's hard to achieve this with a static compiled language.
The workaround is to use convention plugins or using dynamic accessors like project.extensions.getByName().

And, there's a post as a reference that explain I'm in a X-Y problem and my solution doesn't work for the feature request.


v1.3.1: test_complex and test_pod fail on ppc64le

From https://koji.fedoraproject.org/koji/taskinfo?taskID=82080105 on pcc64le:
95% tests passed, 2 tests failed out of 39
Total Test time (real) =   6.78 sec
The following tests FAILED:
          6 - test_complex (Failed)
         18 - test_pod (Failed)
Errors while running CTest

Build log here: build_ppc64le.log.txt

Only affecting the json/xml archives serializing long double. Very likely some goofy sizing on pcc64le that goes through the wrong serialization function.

您的邮件我已收到，谢谢合作！

您的邮件我已收到，谢谢合作！

It would be great if you could stick to English. ;-)

I should have time to address piled up issues in November.

From: Andreas Tille @.>
Sent: Friday, October 7, 2022 7:19:02 AM
To: USCiLab/cereal @.>
Cc: Shane Grant @.>; Comment @.>
Subject: Re: [USCiLab/cereal] v1.3.1: test_complex and test_pod fail on ppc64le (Issue #744)
您的邮件我已收到，谢谢合作！
It would be great if you could stick to English. ;-)
―
Reply to this email directly, view it on GitHubhttps://github.com/USCiLab/cereal/issues/744#issuecomment-1271659341, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAMPCTZUFHRE5WGI37GF4FLWCAWNNANCNFSM5NB5GADA.
You are receiving this because you commented.Message ID: @.***>


Docker/workspace testing

:tada: This PR is included in version 1.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Sodium incompatibility

Is it a mod compatibility issue?
Yes
If it's a mod compatibility issue, please provide the link(s) to the incompatible mod(s)
Sodium
Minecraft version
1.20.1
Describe the issue
Immersive Portals crashes when installed in the same instance as Sodium 0.5.8

I second this request.

Seconding this, I'm trying to make a decently sized modpack and this is the only incompatibility that I'm having an issue with right now.


Option for Access-Control-Allow-Credentials header

Required information

Distribution: Debian
Distribution version: Jessie
The output of "lxc info" or if that fails:

Kernel version: 4.7.0
LXC version: 2.0.0
LXD version: 2.0.0
Storage backend in use: dir

Issue description
Allow web applications to run CORS.
Firefox browser fails to make any request to lxd daemon api. It throws a Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at error.
Steps to reproduce

Install lxd on one machine
Install lxd-webgui on the other
Try to open it with Firefox Browser

Information to attach
Can we have an option for Access-Control-Allow-Credentials header?

We already have config options to allow for cross origin connections.
See doc/configuration.md and the core.https_allowed_origin	, core.https_allowed_methods and core.https_allowed_headers keys.

@stgraber then how I should set Access-Control-Allow-Credentials header properly? Can't find anything usable in doc/configuration.md. Firefox really doesn't work without that header. Chrome works ok.

Hmm, maybe I misunderstand the reason for Access-Control-Allow-Credentials.
The description I read says it's related to being able to forward cookies, but LXD doesn't use cookies at all so why is this relevant?

Let me describe it as it should be done.
In order, to use LXD API we should send credentials(certificate) with each request. So, to use it with JavaScript XHR's we should set "withCredentials=true" to XHR Request. When we do so, Firefox can not finish this request, because there's no Access-Control-Allow-Credentials: true header in response (Google Chrome works ok, BlackBerry browser throws security alert, but we can proceed, Safari doesn't work as FF).

Seems odd that they require this for x509 authentication but browsers do tend to be weird and inconsistent...

@stgraber thank you for your advice on the introduction of amendments to the code. Modestly hope that satisfied all requirements.


 Some of the changes here conflict with #456. Can you remove those and rebase?

@fragio done 👍

Thanks!


[bug] training xgboost dosen't work with dataframe, only numpy array

Hello and thanks you for that package.
I came across a problem while trying to use a xgboost model that was trained on dataframe.
So this is my code:
X_train, X_test, y_train, y_test = load_csv('X_train'), load_csv('X_test'), load_csv('y_train'), load_csv('y_test')
model = XGBClassifier(tree_method='hist')
X_train_val, y_train_vals = X_train.values, y_train.values.squeeze()
X_test_val, y_test = X_test.values, y_test.values.squeeze()
model.fit(X_train, y_train)

# fit influence estimator
explainer = BoostIn().fit(model, X_train, y_train)

Which produce this exception:
Traceback (most recent call last):
  File "/home/jupyter/owlytics-data-science/influence/influence.py", line 35, in <module>
    explainer = BoostIn().fit(model, X_train, y_train)
  File "/opt/conda/envs/py39/lib/python3.9/site-packages/tree_influence/explainers/boostin.py", line 44, in fit
    super().fit(model, X, y)
  File "/opt/conda/envs/py39/lib/python3.9/site-packages/tree_influence/explainers/base.py", line 31, in fit
    self.model_ = parse_model(model, X, y)
  File "/opt/conda/envs/py39/lib/python3.9/site-packages/tree_influence/explainers/parsers/__init__.py", line 33, in parse_model
    trees, params = parse_xgb_ensemble(model)
  File "/opt/conda/envs/py39/lib/python3.9/site-packages/tree_influence/explainers/parsers/parser_xgb.py", line 17, in parse_xgb_ensemble
    trees = np.array([_parse_xgb_tree(tree_str) for tree_str in string_data], dtype=np.dtype(object))
  File "/opt/conda/envs/py39/lib/python3.9/site-packages/tree_influence/explainers/parsers/parser_xgb.py", line 17, in <listcomp>
    trees = np.array([_parse_xgb_tree(tree_str) for tree_str in string_data], dtype=np.dtype(object))
  File "/opt/conda/envs/py39/lib/python3.9/site-packages/tree_influence/explainers/parsers/parser_xgb.py", line 88, in _parse_xgb_tree
    node_dict = _parse_line(line)
  File "/opt/conda/envs/py39/lib/python3.9/site-packages/tree_influence/explainers/parsers/parser_xgb.py", line 190, in _parse_line
    res['feature'], res['threshold'] = _parse_decision_node_line(line)
  File "/opt/conda/envs/py39/lib/python3.9/site-packages/tree_influence/explainers/parsers/parser_xgb.py", line 201, in _parse_decision_node_line
    feature_ndx = int(feature_str[1:])
ValueError: invalid literal for int() with base 10: 'ecent_beta_blockers_change'

However, When training X_train_val, y_train_val (which is a numpy array) works perfectly good.
It would be great if you could support training with DataFrame as well.
Thanks again!

Hi Yarden234! Thanks for bringing this up. I believe I've fixed this issue now in version 0.1.1. Please give it a try and feel free to open this issue back up if it's not working. Thanks again!

Hi there,
I encountered this error still. I was wondering if you might check on it again. Thanks!

Hi @aclarkse, can you provide a fully reproducible example, please?


[Enhancement] Make Distribution Downloader download from Snapshots instead of Releases and add min keywords

Describe the enhancement

The current distribution downloader is using the RELEASES channel for downloading artifacts
https://github.com/opensearch-project/OpenSearch/blob/0e9f74e35f1255cb9ec45be3d8960aad195a9f6e/buildSrc/src/main/java/org/opensearch/gradle/DistributionDownloadPlugin.java#L159

This was due to an earlier decision that distribution downloader requires a static link, and cannot consume artifacts with dynamic build number.
The new approach is to have a snapshot artifact labeled as latest to avoid promoting to RELEASE channel.

Also, due to the plan to support core artifacts without plugins, we added min keyword in the core tarball naming convention, which also needs to be addressed in DistributionDownloader code.
https://github.com/opensearch-project/opensearch-build/issues/31

Expected behavior
The new link should point to
/snapshots/core/opensearch/[revision]/[module]-min-[revision]-SNAPSHOT(-[classifier])-latest.[ext]

which results in something similar to:
opensearch-min-1.0.0-rc1-SNAPSHOT-linux-x64-latest.tar.gz

Additional context

Have a quick conversation with @saratvemulapalli and we have some thoughts on this.

Instead of hardcoding it to snapshots, we can start fixing the actual snapshots ivyrepo setup here: https://github.com/opensearch-project/OpenSearch/blob/0e9f74e35f1255cb9ec45be3d8960aad195a9f6e/buildSrc/src/main/java/org/opensearch/gradle/DistributionDownloadPlugin.java#L172
We will have these two urls fixed for snapshots vs releases:

Snapshots: https://artifacts.opensearch.org/snapshots/core/opensearch/[revision]/[module]-min-[revision]-SNAPSHOT(-[classifier])-latest.[ext]

Releases: Snapshots: https://artifacts.opensearch.org/snapshots/core/opensearch/[revision]/[module]-min-[revision](-[classifier]).[ext]

This is based on the fact that snapshots is a moving target and releases is not.
You can expect snapshots changing on a daily basis but releases might change every release as stable builds.

We also expect to have snapshots and releases built from fresh

Parent issue: https://github.com/opensearch-project/OpenSearch/issues/91

I put up a draft #904 of an idea on how to solve this issue.
This change will register the existing snapshot repo, so builds/integ tests executed with -Dopensearch.version with -SNAPSHOT will fetch from that repo.
It also adds a new release-candidate repo that exists as a fallback if a version is not in releases.  So for our current situation where we require a 1.0.0 bundle that is not a snapshot build yet has not yet been released, it can fetch from this staging repo and test against it.
@peterzhuamazon @camerski @saratvemulapalli thoughts?

Looks reasonable to me.

Looks good to me.


mini-svg-data-uri dependency

Forgive my drive by issue filing but I noticed that mini-svg-data-uri is a dependency (in plugin.js).
It seems that you're just using it for background images. Couldn't this be done at compile time?

Moved to dev dependencies.

Seems like the removal of this dependency now causes Tailwind to crash while trying to load the Flowbite plugin (v1.4.0):
Error: Cannot find module 'mini-svg-data-uri'
Require stack:
- D:\Source\TestProject\node_modules\flowbite\plugin.js
- D:\Source\TestProject\tailwind.config.js
- D:\Source\TestProject\node_modules\tailwindcss\lib\cli.js
    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:933:15)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.<anonymous> (D:\Source\TestProject\node_modules\flowbite\plugin.js:1:22)
    at Module._compile (node:internal/modules/cjs/loader:1103:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1155:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Module.require (node:internal/modules/cjs/loader:1005:19) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    'D:\\Source\\TestProject\\node_modules\\flowbite\\plugin.js',
    'D:\\Source\\TestProject\\tailwind.config.js',
    'D:\\Source\\TestProject\\node_modules\\tailwindcss\\lib\\cli.js'
  ]
}

Hey @ElPerenza,
You're right, fixed the issue in a v1.4.1 patch.
Thanks for the notice!


PDO exception on batch insert with specially formed strings on postgresql database

There is an issue when trying to batch insert on postgresql database using the following code:
$rows = [
    [':a ? \\'],
];
$db->createCommand($queryBuilder->createTable('test', ['text' => 'string']))->execute();
$db->createCommand($queryBuilder->batchInsert('test', ['text'], $rows))->execute();

Running this will provide the following exception:

Exception: SQLSTATE[HY093]: Invalid parameter number: mixed named and positional parameters
Failed to prepare SQL: INSERT INTO "test" ("text") VALUES (':a ? ')

I'm using PHP 5.5.27.
My guess is that sql, created by batchInsert should not contain escaped values, but placeholders.

related to #6526

@cebe I think its simpler than that. https://github.com/yiisoft/yii2/blob/master/framework/db/ColumnSchema.php#L132
When using an insert in batchInsert() this is the line that hadles the sanitization of the data which as you can see it doesn't sanitize special characters like quotes, slashes and parenthesis

that line is not for sanitizing. it is for PDO type casting.

I see what it is bug of PDO driver for PostgreSQL:
$pdo = new \PDO('pgsql:host=localhost;dbname=yii2basic', 'username', 'password');
$pdo->prepare("select ':a ? \\'");

PDO::prepare(): SQLSTATE[HY093]: Invalid parameter number: mixed named and positional parameters
And this code works properly:
$pdo = new \PDO('mysql:host=<IP_ADDRESS>;dbname=yii2basic', 'username', 'password');
$pdo->prepare("select ':a ? \\'");

Hi, i have a problem with batchInsert postgresql- error SQLSTATE[HY093]: Invalid parameter number: mixed named and positional parameters.
Only with those 3 strings
Vorfahrt Mercedes GLC Coupй:Vorfahrt Mercedes GLC Coupй
Backstage DTM 2016 (Teil 2):Backstage DTM 2016 (Teil 2)
planet e.:Glitschige Geschдfte-Die Aalmafia
,when i replace ':' with ': ', everything works Ok .
Other records with strings that content ':' working fine
Can anyone help? Thanks in advanse

Hi, i have a problem with batchInsert postgresql- error SQLSTATE[HY093]: Invalid parameter number: mixed named and positional parameters.
Only with those 3 strings
Vorfahrt Mercedes GLC Coupй:Vorfahrt Mercedes GLC Coupй
Backstage DTM 2016 (Teil 2):Backstage DTM 2016 (Teil 2)
planet e.:Glitschige Geschдfte-Die Aalmafia
,when i replace ':' with ': ', everything works Ok .
Other records with strings that content ':' working fine
Can anyone help? Thanks in advanse

Yii version2.0.36 (not sure about 2.0.40).
Original error: SQLSTATE[HY093]: Invalid parameter number: mixed named and positional parameters Failed to prepare SQL: INSERT INTO ...
Bug still present even for : , like @nazakar suggested. Escaping in \: and \? also didn't solved problem. Setting the
Yii::$app->db->emulatePrepare = false;

, as suggested many times for such case, didn't solved the issue. Also, this bug, in my case, is present in raw createCommand()->execute() flow. Only suitable solution for me was direct call of PDO->exec():
Yii::$app->db->pdo->exec('your raw sql command');

That's why this is not the PDO nor PostgreSQL bug, but the Yii2 itself

Yii version2.0.36 (not sure about 2.0.40).
Original error: SQLSTATE[HY093]: Invalid parameter number: mixed named and positional parameters Failed to prepare SQL: INSERT INTO ...
Bug still present even for : , like @nazakar suggested. Escaping in \: and \? also didn't solved problem. Setting the
Yii::$app->db->emulatePrepare = false;

, as suggested many times for such case, didn't solved the issue. Also, this bug, in my case, is present in raw createCommand()->execute() flow. Only suitable solution for me was direct call of PDO->exec():
Yii::$app->db->pdo->exec('your raw sql command');

That's why this is not the PDO nor PostgreSQL bug, but the Yii2 itself

Related to PHP BUG #79276.

It's sad to say, but it makes batchInsert() upsafe.


Update submit.py to be more flexible with return types of solve_it

Adding support for non-string return types and allowing unicode strings.

Current coverage is 89.63% (diff: 100%)

Merging #25 into master will increase coverage by 0.37%

@@             master        #25   diff @@
==========================================
  Files            17         17
  Lines          1061       1061
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            947        951     +4
+ Misses          114        110     -4
  Partials          0          0

Powered by Codecov. Last update 8e1e011...f679dfe

May address issues discussed in #22 and #24.


Line break removal for Paste as Plain Text and Text Extractor

Description of the new feature / enhancement
The text when copied from a PDF, for example, comes with line breaks and it is necessary to modify it to suit the standard format. It would be interesting if both tools, Text Extractor and Paste as Plain Text, could have some option to activate or deactivate this feature.
Scenario when this would be used?
By copying texts from a PDF file (Paste as Plain Text) or by pasting texts from the Text Extractor.
Supporting information
The website below works perfectly for this purpose.
https://www.textfixer.com/tools/remove-line-breaks.php

This feature is currently planned for Text Extractor under issue #21455, I am not sure about Paste As Plain Text though.

Another excellent use case for this in Paste as Plain Text is when you select whole lines (clicking on the line number to the left) in most text editors (eg VS Code or Notepad ++) the line breaks are copied. This can be inconvenient if you are trying to paste that line into of a line of code or function.
eg copying lines of text (strings) that will be used as function arguments.

Possible Avenue of Implementation: #24619

My suggestion in #25197 involves a context menu for a combined "Text Extractor + Snipping Tool" PowerToy.
The proposed customizaable PrintScreen Context Menu powertoy would activate via a main key (e.g. PrintScreen), which lets you select your rectangle and then the popup menu immediately appears:
 or
The menu is proposed to be customizable by advanced users.
The menu could potentially be customized to include various methods of text formatting:

"Copy as Text (No Line Breaks)"
"Copy as Text (Windows)"
"Copy as Text (UNIX)"
"Copy as CSV"
"Copy as RTF"

Via a customizable context menu, this keeps Text Extractor simple for everyday users, while allowing advanced users to use various different methods of text formatting.

actually, lets /dup https://github.com/microsoft/PowerToys/issues/21455


IMN-141 -  Agreement email sender

Closes IMN-141
This PR introduces the Email manager in the commons package and add the agreement-email-sender that listen for AgreementSubmitted or AgreementActivated events and send the activation email.
The Email manager uses nodemailer to send the email.
To test the email locally and during integration tests I introduced MailPit which simulates a SMPT server (with Rest API and UI).
I smoke-tested submitting an agreement mocking the call to selfcare with this:

Selfcare mock
diff --git a/packages/agreement-email-sender/src/services/agreementEmailSenderService.ts b/packages/agreement-email-sender/src/services/agreementEmailSenderService.ts
index 76ef903d..3cb5839c 100644
--- a/packages/agreement-email-sender/src/services/agreementEmailSenderService.ts
+++ b/packages/agreement-email-sender/src/services/agreementEmailSenderService.ts
@@ -15,7 +15,7 @@ const { emailManagerSender } = emailManagerConfig();
 export async function getActivationMailFromAgreement(
   agreement: AgreementV2,
   readModelService: ReadModelService,
-  selfcareV2Client: SelfcareV2Client
+  _selfcareV2Client: SelfcareV2Client
 ): Promise<{
   subject: string;
   body: string;
@@ -54,12 +54,14 @@ export async function getActivationMailFromAgreement(
     getEServiceById(agreement.eserviceId),
     getTenantById(agreement.producerId),
     getTenantById(agreement.consumerId),
-    selfcareV2Client.getInstitution({
-      params: { id: agreement.consumerId },
-    }),
-    selfcareV2Client.getInstitution({
-      params: { id: agreement.producerId },
-    }),
+    // selfcareV2Client.getInstitution({
+    //   params: { id: agreement.consumerId },
+    // }),
+    // selfcareV2Client.getInstitution({
+    //   params: { id: agreement.producerId },
+    // }),
+    { digitalAddress<EMAIL_ADDRESS>},
+    { digitalAddress<EMAIL_ADDRESS>},
   ]);

   if (!eservice) {
diff --git a/packages/agreement-process/src/services/agreementContractBuilder.ts b/packages/agreement-process/src/services/agreementContractBuilder.ts
index ba45bd1f..beba0929 100644
--- a/packages/agreement-process/src/services/agreementContractBuilder.ts
+++ b/packages/agreement-process/src/services/agreementContractBuilder.ts
@@ -29,13 +29,13 @@ import {
 } from "pagopa-interop-models";
 import {
   UserResponse,
-  selfcareV2Client,
+  // selfcareV2Client,
 } from "pagopa-interop-selfcare-v2-client";
 import { match } from "ts-pattern";
 import {
   agreementMissingUserInfo,
   agreementStampNotFound,
-  userNotFound,
+  // userNotFound,
 } from "../model/domain/errors.js";
 import { UpdateAgreementSeed } from "../model/domain/models.js";
 import { ApiAgreementDocumentSeed } from "../model/types.js";
@@ -46,20 +46,23 @@ const CONTENT_TYPE_PDF = "application/pdf";
 const AGREEMENT_CONTRACT_PRETTY_NAME = "Richiesta di fruizione";

 const retrieveUser = async (
-  selfcareId: SelfcareId,
-  id: UserId
-): Promise<UserResponse> => {
-  const user = await selfcareV2Client.getUserInfoUsingGET({
-    queries: { institutionId: selfcareId },
-    params: { id },
-  });
-
-  if (!user) {
-    throw userNotFound(selfcareId, id);
-  }
-  return user;
-};
+  _selfcareId: SelfcareId,
+  _id: UserId
+): Promise<UserResponse> =>
+  // const user = await selfcareV2Client.getUserInfoUsingGET({
+  //   queries: { institutionId: selfcareId },
+  //   params: { id },
+  // });

+  // if (!user) {
+  //   throw userNotFound(selfcareId, id);
+  // }
+  // return user;
+  ({
+    name: "Mario",
+    surname: "Rossi",
+    taxCode: "RSSMRA80A01H501A",
+  });
 const createAgreementDocumentName = (
   consumerId: TenantId,
   producerId: TenantId

@galales addressed all comments 👌


Privacy concerns with logging

From looking at the code I noticed it logs some info to appget.net at the end of every install (InstallerContext.SubmitResult), plus troubleshooting info sent through Sentry.
Now, I'm not implying anything nefarious is going on - I can see how this info is useful and I see no personal info collected.
However, without visibility into what appget.net is doing, I'm sure you can see how this can be a problem for privacy-minded people. So, my questions are:

Any plans to make appget.net more transparent, or at least to have a privacy policy?
Any plans to have the ability to only log locally?
Can we have more info about the reasons for logging at the end of installs?

Thanks!

Sorry about the delayed response, I didn't want to rush it, and it has been busy in the past few days at work,
First of all, I really appreciate you taking the time taking a look over at the source code,
As you mentioned, we try our best not to collect information unless it's something that can be used to drastically improve support and functionality of AppGet (Making money or profit is not considered).
Not to cover some of the data we collect:
Privacy Policy
Absolutely, I'll start working on something right away.
Disable logging
To be honest, the main thing that has been holding this back is we don't have a config infrastructure. I'll add this to our todo list with a high priority because I think it's important.
Sentry
We use an opensource tool called https://github.com/getsentry/sentry to collect crash data, anytime AppGet crashes or fails to do what a user has requested it to do we send the crash information as well as some context information, e.g. command that was being executed, version of Windows, if the user was an admin or not etc. This has helped us resolved MANY issues that we've only seen in the wild.
I should also note, We host our own instance of sentry as you can see (sentry.appget.net) which means all data is hosted, stored and processed by AppGet. Also, we purge all crash information after 14 days.
InstallerContext.SubmitResult
This data is useful in two ways,

It tells us if a specific package has an installation issue. Maybe we aren't passing the correct arguments, and all installs by appget are failing. If we see a large number of failures for a specific package, we look into it and see if it's something we could resolve.

One of the main features of appget is that we can manage app installs by applications that weren't even installed by AppGet, we do this by reading and parsing the Windows installer database.

However, some installers make things hard by not exposing the IDs they will be installed as, see #15. If you look at the data submitted by InstallerContext you'll notice we are sending the Windows Installer Records that are added/removed after an install or uninstall. This allows us to find a pattern and match pretty much all installed application regardless of install method. (eg, 100% of Firefox installs use ID FIRE_FOX).

Thanks for the detailed response and your commitment to improve on this. Your reasons for logging also make sense. Keep up the good work! :+1:


test: [#955] Add cypress test for api docs

Closes #955

Hi @tenthe,
for me it was pretty straightforward to set up and start the tests.
I have added the data-cy property now :)


new release?

Dear developers, although there are almost no updates, several important amendments have been made since the last release.
Is a new release expected in the near future?

sure

https://github.com/hyprwm/hyprpicker/releases/tag/v0.3.0

Thank you very much!


[NRCE] Input node for all devices?

Thanks for your work on this. I have lots of enocean devices and it seems that I need a node for each one.
For the future, could I assign multiple devices to one node? Say add all switches to one, temperature devices to another etc.... Or maybe all just to one node?
Thanks again

there is currently an output node for a button and a general output node for everything else.
the rational behind this was that this way it's easy to manage channels and eeps. but i don't see we couldn't have an output node where you can pass channel and eep.
Maybe you can tell me something about your use case and how you imagine it could be used, and i can build a prototype for this.

I have a number of devices and was wondering if there was one node that I could connect all of my logic to.
It's not important though..  i can pipe all the individual nodes to one set of function s

are we talking about listening to multiple external devices? or simulating multiple devices devices internally and then sending the data?

just published v0.4.3 based on your input.
moved the parser and transformer stream to the config node, so these are now shared between all nodes, and the single nodes just listen to the stream. that should prevent the sowing down and be much more scaleble.
before we had 2 transform streams per node, now we just to for the whole flow.

i thougth about the "one node for all inputs" idea for a bit, and it might allow for more powerful flows for the cost of higher complexity.
I'm still not sure how to handle the teach in and management of the EEPs in that case...
I#m open for ideas though... it will have to wait until v1.0.0

there is now an output node available for all devices... still thinking about the input...

What do you think about separating the input node from the teach in altogether?
we could have a general input node, that forwards all telegrams, and teachable "device nodes" that connect to that node?
this way you can implement your own teach in logic, group sensors and logic etc.
the most important part though is that this will enable virtual devices with bidirectional communication...

actively working on a rewrite of the input node as proposed with separated input node and device node which can hold one or more devices. it's a much more flexible architecture...

currently with the old version i used the nodes context to save the sensor info, that has the disadvantage of requiring the user to change a node-red setting in order for it to work.
I now use the nodes config to store the information. The disadvantage here is that now you can not teach in sensors without implementing the flow each time a new sensor is tought in...
i will probably go with this approach for now anyways until i have a native memory in enocean-js in place.

Back from travels.... It works really nicely as it is now. I will put in a request for passing the name of the input node in meta output as this could solve my use case!

with 0.5.0-alpha.1  this should be fixed now


dep: replace tree-sitter-json crate with git repo

Replacing crate with git repo because maintainers have no permission to publish new crate versions

Please put a comment in the Cargo.toml to explain this. Someone might want to revert this change otherwise :)


Writing node paths leading with a digit in their name results in an error.

Godot version:
Godot Engine v3.2.3.stable.official
OS/device including version:
Windows10
Issue description:
When writing the node path in the editor, for example

$Node/1/Button.disabled = true

if a node within the path, is leading with a digit, the following error occurs.

Expected string constant or identifier after "$" or "/".

Renaming the node culprit to lead with a letter instead, fixes the issue, but is hard to intuitively know that at a glance.
At a bare minimum, the error message should include that nodes leading with a digit should be renamed.
Minimal reproduction project:
errorPJ.zip

You can also quote the string, no need to rename your node:
$"Node/1/Button"

Literals starting with a digit are not valid identifiers (var 2pi = TAU won't work). Therefore, the current error message is correct.


Support self-signed certificates in CURLStreamFile

The current construction of the curl command in CURLStreamFile does not allow for self-signed https downloads from s3 buckets.
Why
Buckets created in local NAS (like QNAP/QuObjects) or local workflows built around minio containers take self-signed certificates for https endpoints. Using tensorizer natively in these environments is pretty useful for dev iterations and workflows before actually deploying.
Solution
A possible solution is that curl allows for bypassing the check all together with the -k flag (reference). Of course, it may also be possible to provide the cert to the curl call as an alternative. However, for these onprem workflows it may be ok to go with the insecure curl call.
Possible Implementation
Propagate a allow_insecure boolean flag through open_stream to CURLStreamFile which adds the -k flag to the curl command.
def open_stream(
    ...,
    allow_insecure: bool = False,
)

class CURLStreamFile:
    def __init__(
        allow_insecure: bool = False
        *,
    ):

    # Individual flag or self._additional_curl_flags list
    # which could be used in _reproduce_and_capture_error()
    kflag = "-k" if allow_insecure else ""

    cmd = [
            curl_path,
            ...
            kflag,
            uri,
        ]

I can put up a PR for this, if ok.

@spmkone :+1: We'll happily accept a PR for this.


Update historical-chronicals.html

Fixes:  #(issue no 4438)
Description
Removed Duplicate Font Awesome Links:
Removed redundant imports of Font Awesome, leaving only one  for efficiency.
Simplified Event Listeners:
Reorganized the event listener for the logout button to ensure only valid elements are targeted and added error handling for missing elements.
Reduced Inline CSS:
Moved some inline CSS styles to external CSS files (like .menu, .circle) to improve maintainability.
Removed Unused Scripts:
Removed scripts that were not referenced in the file, such as scrollToGenre, to reduce clutter and potential load times.
Favicon Paths:
Consolidated the favicon  tags in the  section.
These adjustments optimize the structure and loading efficiency of the HTML file.

Type of PR

[x] Bug fix

Checklist:

[x] I have made this change from my own.
[x] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

@anuragverma108 @abhi03ruchi do check


[hotfix] Temporarily disable RollbackTest

I am temporarily disabling the rollbacktest, pending a fix in https://issues.apache.org/jira/browse/FLINK-29771
I will try to fix the underlying non obvious problem in the next 1-2 days. Until then this should alleviate the large pressure this causes on the CI system.

Daren Wong found the probable cause, he will fix this properly


Add ACLED armed conflicts data COUNTRY=mozambique

This PR enables PRISM to fetch and process incidents from the ACLED API.
How to test the feature:

Register and get the ACLED api key. https://acleddata.com/acleddatanew/wp-content/uploads/dlm_uploads/2020/10/ACLED_AccessGuide_2020.pdf
Include the following environment variables:

Backend

ACLED_API_KEY
ACLED_API_EMAIL

Include the following parameter within the country prism.json file

acled_iso (number). The list of iso codes is found here: https://www.acleddata.com/download/3987/

NoteI updated this PR...the acled iso parameter is set within the layers.json

@JorgeMartinezG - I think what @ericboucher is suggesting is that you do the code review that was requested for https://github.com/WFP-VAM/prism-app/pull/559. Once that is merged, we can revisit how this PR was implemented


Added Server parameter to the example because it´s required (New-MailboxDatabase.md)

Docs Build status updates of commit f913ae8:
:clock10: Full build: the average full build time is 17 min(s) 55 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit f913ae8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-MailboxDatabase.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 3f97231:
:clock10: Pending: waiting for processors (22 builds ahead of you)
:warning: Docs Build is busy, currently there are 22 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 3f97231:
:clock10: Full build: the average full build time is 17 min(s) 54 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 3f97231:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsCallQueue.md
:x:Error

Details

skype/skype-ps/skype/Set-CsCallQueue.md
:x:Error

Details

filemap.json
:x:Error

Details

:x:Error

Details

skype/skype-ps/skype/New-CsCallQueue.md

[Error-PublishToDHSFailed] Publishing of document 'skype/skype-ps/skype/New-CsCallQueue.md' to document hosting service failed with asset id: module/skype/new-cscallqueue locale: en-us, depot name: MSDN.skype-ps, branch name: pr-en-us-5978. Error: A task was canceled.

skype/skype-ps/skype/Set-CsCallQueue.md

[Error-PublishToDHSFailed] Publishing of document 'skype/skype-ps/skype/Set-CsCallQueue.md' to document hosting service failed with asset id: module/skype/set-cscallqueue locale: en-us, depot name: MSDN.skype-ps, branch name: pr-en-us-5978. Error: A task was canceled.

filemap.json

[Error-PublishToDHSFailed] Publishing of document 'filemap.json' to document hosting service failed with asset id: filemap.json locale: en-us, depot name: MSDN.skype-ps, branch name: pr-en-us-5978. Error: A task was canceled.

[Error-BackendEngineExceptionError] An exception occurred during the build process. Please check the raw log and retry building your content again. If the issue persists, please open a ticket in https://SiteHelp and include the URL of publishing report as well as below information: One or more errors occurred..

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

Docs Build status updates of commit 3f97231:
:clock10: Full build: the average full build time is 17 min(s) 54 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 3f97231:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-MailboxDatabase.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


symbol not found in flat namespace '__ZN4mlir2TF6detail25ResourceAliasAnalysisInfo18kMaxResourceTypeIdE'

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

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):macOS monterey 12.0.1
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:No
TensorFlow installed from (source or binary):source
TensorFlow version: r2.7
Python version: python3.9
Installed using virtualenv? pip? conda?:pip
Bazel version (if compiling from source): bazel 3.7.2
GCC/Compiler version (if compiling from source):  Apple clang version 13.0.0 (clang-13<IP_ADDRESS>)
CUDA/cuDNN version: no enabled
GPU model and memory:: no enabled(Just CPU model)

Describe the problem
when running the "import tensorflow as tf" , I got the exception below:
"Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/tensorflow/python/pywrap_tensorflow.py", line 64, in
from tensorflow.python._pywrap_tensorflow_internal import *
ImportError: dlopen(/usr/local/lib/python3.9/site-packages/tensorflow/python/_pywrap_tensorflow_internal.so, 0x0006): symbol not found in flat namespace '__ZN4mlir2TF6detail25ResourceAliasAnalysisInfo18kMaxResourceTypeIdE'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/xxxxx/PycharmProjects/pythonProject1/tensorflow/testdebug.py", line 1, in
import tensorflow as tf
File "/usr/local/lib/python3.9/site-packages/tensorflow/init.py", line 41, in
from tensorflow.python.tools import module_util as _module_util
File "/usr/local/lib/python3.9/site-packages/tensorflow/python/init.py", line 40, in
from tensorflow.python import pywrap_tensorflow as _pywrap_tensorflow
File "/usr/local/lib/python3.9/site-packages/tensorflow/python/pywrap_tensorflow.py", line 79, in
raise ImportError(
ImportError: Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/tensorflow/python/pywrap_tensorflow.py", line 64, in
from tensorflow.python._pywrap_tensorflow_internal import *
ImportError: dlopen(/usr/local/lib/python3.9/site-packages/tensorflow/python/_pywrap_tensorflow_internal.so, 0x0006): symbol not found in flat namespace '__ZN4mlir2TF6detail25ResourceAliasAnalysisInfo18kMaxResourceTypeIdE'
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/errors for some common causes and solutions.
If you need help, create an issue at https://github.com/tensorflow/tensorflow/issues and include the entire stack trace above this error message.
Process finished with exit code 1
"
Provide the exact sequence of commands / steps that you executed before running into the problem
steps:

download source code and " git checkout r2.7"
./configure  (choose "N" for all of the questions)
bazel build --config=dbg --strip=never -c dbg --copt='-g' --cxxopt='-g'   //tensorflow/tools/pip_package:build_pip_package
./bazel-bin/tensorflow/tools/pip_package/build_pip_package pkt2/tensorflow_pkg
pip install /Users/xxxxx/ai/tf/tensorflow/pkt2/tensorflow_pkg/tensorflow-2.7.0-cp39-cp39-macosx_12_0_x86_64.whl --force-reinstall
try to run "import tensorflow as tf" and got the exception:"ImportError: dlopen(/usr/local/lib/python3.9/site-packages/tensorflow/python/_pywrap_tensorflow_internal.so, 0x0006): symbol not found in flat namespace '__ZN4mlir2TF6detail25ResourceAliasAnalysisInfo18kMaxResourceTypeIdE'
"

Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

BTW, got the same exception if using the build command "bazel build --config=dbg --per_file_copt=+tensorflow/core/kernels/identity_op.*@-g //tensorflow/tools/pip_package:build_pip_package"

Hi @sushreebarsa  , do you have any idea or can duplicate for this issue?

The issue also happens at TF2.8 (all of the other configs and commands are the same as  TF2.8)

@shenyufeng1234,
Can you please take a look at link1, link2 which discusses about the similar issue and let us know if it helps? Thanks!

link2

No,I already updated Xcode to the most recent.

BTW,how to know the  python version used to compile the TF source code?

force_flat_namespace option for Clang of macos (I don't know if it is helpful) ,however I cannot use it with commmand "bazel build", it is not supported:"ERROR: -force_flat_namespace :: Invalid options syntax: -force_flat_namespace".

I found a big diff between r2.6 and r2 .7, the variable  "__ZN4mlir2TF6detail25ResourceAliasAnalysisInfo18kMaxResourceTypeIdE"  (static constexpr int64_t kMaxResourceTypeId  )doesn't exist in  r2.6, it is new from r2 .7 to master, maybe the new  variable  kMaxResourceTypeId cuased the issue at the macos.

It seems TF needs to move the two variables  "static constexpr int64_t kMaxResourceTypeId = 9999;" and "static constexpr int64_t kUnknownResourceId = -1;" from resource_alias_analysis.h to resource_alias_analysis.cc ,otherwise it would cause the issue at mac os.

BTW ,if I build with "bazel build --config=opt  //tensorflow/tools/pip_package:build_pip_package  " everything is ok,  however if I build with  enable any debug info , it would be failed with exception above.

I was trying to build with a different command but encountered the same issue. Adding the following one line fix this issue for me.
diff --git a/tensorflow/compiler/mlir/tensorflow/analysis/resource_alias_analysis.cc b/tensorflow/compiler/mlir/tensorflow/analysis/resource_alias_analysis.cc
index 5c366896349..40ef08483d5 100644
--- a/tensorflow/compiler/mlir/tensorflow/analysis/resource_alias_analysis.cc
+++ b/tensorflow/compiler/mlir/tensorflow/analysis/resource_alias_analysis.cc
@@ -262,6 +262,7 @@ bool IsResourceAllocatingOp(Operation* op) {
 }  // namespace

 constexpr int64_t ResourceAliasAnalysisInfo::kUnknownResourceId;
+constexpr int64_t ResourceAliasAnalysisInfo::kMaxResourceTypeId;

 void IncrementResourceTypeId(int64_t& resource_type_id) {
   if (resource_type_id == ResourceAliasAnalysisInfo::kMaxResourceTypeId) {

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


Memory leak

Hello!
First of all, thank you for the awesome code piece! :-)
But I would like to pinpoint an issue with JPS code:
there is a memory leak because you dont call destructor for elements inside PodVec,
but you use it for complex type with PodVec Buckets; (PodVec of PodVec's)
A simple fix is to add destructor to NodeMap:
"
~NodeMap() {
dealloc();
}
"
Proper fix would include calling destructor for PodVec elements.
Best,
Pavel

The point of a PodVec (as the name suggests) was that elements are POD and thus don't require a dtor. And yes, using a PodVec of PodVecs is a bit of a hack but that's why i added explicit dealloc() for that one single case where it's used that way.
But you're right, i totally overlooked the automatic destruction, good catch. Thanks for pointing out.
I'll add the dtor but will otherwise leave the elements POD, without dtor.
Out of curiority, did you find this by looking at the code or using a leak checker? Because i was sure i had leak-checked the code at some point...

Glad to help!
I found the leak by using default CRT leak checker (_CRTDBG_LEAK_CHECK_DF). :-)


Object spread fails to type check

Given
/** @param {{x: number, y: string}} x */
function f(x) {}

/** @type {{y: string}} */
const o = {y: "a"};

f({x: 1, ...o})

https://closure-compiler.appspot.com/home#code%3D%252F%252F%2520%253D%253DClosureCompiler%253D%253D%250A%252F%252F%2520%2540compilation_level%2520ADVANCED_OPTIMIZATIONS%250A%252F%252F%2520%2540output_file_name%2520default.js%250A%252F%252F%2520%2540formatting%2520pretty_print%250A%252F%252F%2520%253D%253D%252FClosureCompiler%253D%253D%250A%250A%250A%252F**%2520%2540param%2520%257B%257Bx%253A%2520number%252C%2520y%253A%2520string%257D%257D%2520x%2520*%252F%250Afunction%2520f(x)%2520%257B%257D%250A%250A%252F**%2520%2540type%2520%257B%257By%253A%2520string%257D%257D%2520*%252F%250Aconst%2520o%2520%253D%2520%257By%253A%2520%2522a%2522%257D%253B%250A%250Af(%257Bx%253A%25201%252C%2520...o%257D)
I'm getting a type error. I assume this is just because object spread is not fully implemented yet?

It is true that the typecheck passes haven't yet been updated to understand object spread.
Instead object spread is transpiled away before type checking.
I wouldn't expect that to change until sometime next year.
Since o is a constant, one would expect the compiler to be able to figure out that the object literal
passed to f has a y property containing a string. The transpilation of object rest probably obscures
this. It is possible that this issue will be solved "for free" when the transpilation moves after type checking, but I'm not certain of that.
In any case I wouldn't expect us to work at fixing it until the transpilation pass gets moved.
In the meantime the workaround is to use a cast.

FYI: TypeScript supports it (sample code)

Internal issue created http://b/122266991

@teppeis thanks for pointing that out, but we were actually aware of this.
We disagree with TypeScript on this issue because we have different goals.
TypeScript doesn't care about optimizing output code size, but that's one of our primary focuses.
Some of our advanced optimizations rely on the compiler knowing about all places where a given property is read or written and can break code when some of them are hidden.
Type errors like this one help our users recognize that they are writing code that closure-compiler cannot analyze as well and could be problematic to compile.


first steps

Can someone tell me how to start?

Well I continue with first steps course

How to asign the issue to myself¿?

Done


Stableswap

PR adds the stableswap invariant developed by Curve Finance (WIP).

Closed in favor of https://github.com/BeanstalkFarms/Basin/pull/135


Pathfinding tuning: Includes testing out other pathfinding planners, including RRT*, RRTX, PRM*, Informed RRT

Something like
roslaunch local_pathfinding all.launch planner_type:=RRTStar
<!-- Sets planner type, accepted types include: "RRTStar", "RRTSharp","RRTXStatic" (full list in allocatePlanner()) -->

Can also set the random_seed parameter as well
Characterize qualitatively and quantiatively

How long it takes to generate paths
How good are the paths
How many times it fails to find a path
How many times it creates a very bad path

Look at https://github.com/UBCSailbot/local-pathfinding/blob/master/python/Path.py
def createPath(state, runtimeSeconds=1.0, numRuns=2, resetSpeedupDuringPlan=False, speedupBeforePlan=1.0,)
Play around with runtimeSeconsd and numRuns

rostopic pub /changeGlobalWind sailbot_msg/globalWind "directionDegrees: 45.0
speedKmph: 0.0"
to change the wind and push the boat harder

rostopic pub /boat_on_path local_pathfinding/addBoat "addType: 'onBoat'
ship: {ID: 0, lat: 0.0, lon: 0.0, headingDegrees: 0.0, speedKmph: 0.1}
waypointIndex: 0"
rostopic pub /boat_on_path local_pathfinding/addBoat "addType: 'trailing'
ship: {ID: 0, lat: 0.0, lon: 0.0, headingDegrees: 0.0, speedKmph: 1.0}
waypointIndex: 0"
To put a new boat on or behind our boat.

The code for adjusting how much obstacles are projected (moved forward) and extended (stretched forward) https://github.com/UBCSailbot/local-pathfinding/blob/master/python/obstacles.py#L11

Related: #86

AIS_BOAT_RADIUS_KM = 0.2
Probably should be increased to at least 0.5 to really avoid hitting boats.

Pathfinding planner used here:
python/updated_geometric_planner.py

Can test with "toy" integration tests by following instructions at:
https://github.com/UBCSailbot/local-pathfinding/tree/master/json

Testing with the launch parameters:
<arg name="num_ais_ships" default="15" />
<arg name="random_seed" default="123" />

and trying out different pathfinding algorithms

Also useful to change parameters in obstacles.py
AIS_BOAT_RADIUS_KM = 0.2
and in MOCK_wind_sensor.py
globalWindDirectionDegrees = 45
ends up creating a test case where the sailbot is trying to path around another boat which is travelling perpendicular to the path of the sailbot. It causes the sailbot to get into a cyclical loop of generating new bad paths which does not avoid the other boat, but continues to be in the path of the boat.

Testing with the launch parameters:
<arg name="num_ais_ships" default="15" />
<arg name="random_seed" default="123" />

and trying out different pathfinding algorithms

Also useful to change parameters in obstacles.py
AIS_BOAT_RADIUS_KM = 0.2
and in MOCK_wind_sensor.py
globalWindDirectionDegrees = 45
ends up creating a test case where the sailbot is trying to path around another boat which is travelling perpendicular to the path of the sailbot. It causes the sailbot to get into a cyclical loop of generating new bad paths which does not avoid the other boat, but continues to be in the path of the boat.

Great work and analysis done in #220


mq (ticdc): improve readability of some code

What problem does this PR solve?

Issue Number: ref #10348
What is changed and how it works?

Add some comments
Adjust variable names to make them more understandable.
Correct the incorrect usage of defer time.Since()

Check List
Tests

No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

/retest


Cannot load localforage-cordovasqlitedriver behind a firewall

The dependency localforage-cordovasqlitedriver cannot be loaded within enterprise setting due to closed ports and firewalls. A git clone can typically not access github directly.
Please use a npm dependency instead, as this is resolved via local enterprise caching repositories.

I have the same issue, please publish a new version of localForage-cordovaSQLiteDriver

Hey, going to be moving this over to the official plugin now that typings are available. Stand by

Fixed, use 1.1.6. Thanks

Thanks, LGTM 👍


Can I create a carousel that scrolls automatically?

I'd like to know if I can make my carousel automatically go through the photos.
Here is my carousel code:

I do not see the code pasted in that so I'm pasting again

A sliding carousel would require javascript which isn't supported in emails.


Readme instructions don't work

The first thing to do according to the instructions is dotnet install snowflaqe -g, which fails.
The correct one is dotnet tool install snowflaqe -g, found it here.
Any chance the readme could be corrected?

Hi @MargaretKrutikova, thanks a lot for pointing the silly mistake, really appreciate it 🙏 It is now fixed and the README shows the correct install command


Add Kube DNS alert and recovery documentation

connects to ministryofjustice/cloud-platform#253
WHAT

KubeDNS alert added to Prometheus. If the job "kube-dns" is not up for 5 minutes, then critically alert.
Documentation on what to do if the kube-dns alarm goes off.

WHY

"kube-dns" is a crucial component to a cluster functioning correctly. If failing, it will impact the DNS performance on the cluster

https://kubernetes.io/docs/concepts/services-networking/dns-pod-service/

Overall LGTM, check comment above.


Fix http server parsing post parameters

Motivation:
Provide fix for issue https://github.com/netty/netty/issues/6418.
Modification:
In case of empty buffer there was attempt to create already created file, this should not be the case here, because sufficient is to just skip writing empty bits to disk, file is already created.

@KowalczykBartek after reviewing the code and report more I think your change is not correct... Let me close this. The correct fix is here https://github.com/netty/netty/pull/7890/files


fix(RNTester): fix RNTester Android working on newer tooling (WIP)

Summary:

While trying to test the backports https://github.com/facebook/react-native/pull/37000 I hit some issues with RNTester on Android which I think are mostly due to that config not playing well with new settings (such as M1, newer Android studio versions etc etc).
So this PR is an attempt to address all the problems. (the current commit is inspired by https://github.com/facebook/react-native/commit/4befd2a29cb94b026d9c048a041aa9f1817295b5)
Currently, I've fixed a build error but it still instacrashes. More work will be required 😭
Changelog:

[INTERNAL] [FIXED] - fix RNTester Android working on newer tooling
Test Plan:

Use ./scripts/test-manual-e2e.sh to run RNTester on Android.

closing this one - talked with @cortinico and we agreed that basically it's very unlikely that there was ever a phase in which RNTester Android worked reliably on an M1 setup, so we'll keep it as-is for the remainer of the 0.69 lifecycle, which should end soon with 0.72 getting closer.


change hyperparameter of yolov3 for custom dataset

I have trained yolov3 for face detection with WIDER face dataset. I want to detect small faces. When I trained with pretrained weight from coco dataset the loss was too high, it was 24.14 and after 14 epoch the training loss became NaN. Then I trained again from scratch without pretrained weight. This time training loss was 14.34 and after 30 epoch loss became NaN. I didn't change anything in config.py , Somehow the loss function is not improving as expected. Could you please guide me how can tune these parameters for my dataset so that loss can be improved and will not become NaN? My training image size is around 1024x800. Do I need to resize my training set?

hello, @sabrinatuli I have exactly the same problem. did you solve it?
what i should change in my config file?

I am no more working in tensorflow, you can check pytorch or keras.
On Thu, Nov 28, 2019, 5:40 PM Kerolos ghobrial<EMAIL_ADDRESS>wrote:

hello, @sabrinatuli https://github.com/sabrinatuli I have exactly the
same problem. did you solve it?
what i should change in my config file?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/YunYang1994/tensorflow-yolov3/issues/351?email_source=notifications&email_token=ACZJSCQVFUOIP23I2WDXRS3QV6G2RA5CNFSM4JCT3MH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFMAZMY#issuecomment-559418547,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACZJSCVFOWKNDBIXINO72ILQV6G2RANCNFSM4JCT3MHQ
.

@kerolos  @sabrinatuli
Try this solution, it will resolve the Train NAN issue, however for me, test loss is still NAN from start till end. but the model works fine when I test after training.
在yolo3.py的bbox_giou上改两处：
iou = inter_area / tf.maximum(union_area, 1e-12)
giou = iou - 1.0 * (enclose_area - union_area) / tf.maximum(enclose_area, 1e-12)
在yolo3.py的bbox_iou上改一处：
iou = 1.0 * inter_area / tf.maximum(union_area, 1e-12)
亲测再也没有出现Nan的问题

thanks @MuhammadAsadJaved . Will try it.


RC2 Release 1 Progress List

[x] Main site structure
[x] Avatars
[x] Notepad
[x] ArtPad
[x] Todo List
[x] Apps
[x] Backgrounds
[x] Sidebar + Banner customisation
[x] Better "Inverted Text"
[x] Inverted Buttons
[x] Logo goes to home page
[x] "Reset Customisation" button
[x] ArtPad "Save" button
[x] ArtPad "Clear" button
[x] Notepad "Open" button
[x] Update Release name

Released


Document path globs support

Hello,
I'd like to ask you to implement support for specifying input files by globs. That would allow us to run remark from build (e.g. npm) scripts in a portable (cross-platform) way. There is actually a lot of linters that accept globs already. They use minimatch AFAIK. I mean some module based on it like globby, glob-all, globule or simple-glob.

Remark has support for globs, and uses minimatch, globby, internally. Could you show me a test case which doesn’t work?

Sure, let me change this issue to a request to document the support. I'll create another report for the issue.

Thanks for the issue. I’m not sure what kind of documentation you expect though. There are already some examples of . usage in https://github.com/wooorm/remark#cli and https://github.com/wooorm/remark/blob/master/doc/remark.1.md. What do you suggest?

AFAIK, . is not a glob. Or at least it's not a wildcard. Or in other words, the path . can be handled without any support of globs and thus it's hard to deduce whether globs are supported or not.
Anyway, ideally, you could mention the supported glob syntax (at least something like this: https://www.npmjs.com/package/globby#globbing-patterns). But if it's too much, adding an example with * could be enough.

Cool, thanks

@wooorm Does this module still use globby internally? I can't find usage of it in this repo or in unified-args. And I'm struggling to figure out how to pass globs with negation (!) — which I think will be a more performant way to ignore all Markdown within node_modules than an ignore file, allowing me to use a broad **/*.md glob (e.g. **/*.md, !**/node_modules/**)

@davidtheclark 👋
Did you find unified-engine? It’s what powers unified-args. The stuff you’re looking for is probably here:
https://github.com/unifiedjs/unified-engine/blob/b5bb6e2986efc0644d78c22ccf78855e630f5312/lib/finder.js#L37-L68
I only know of negation patterns due to the ignore part (for remark, .remarkignore). It used to be ported over from ESLint, but now it’s externalised and in node-ignore.
Note: what exactly are you doing? Globbing may be faster, but the file-in-directory finding itself shouldn’t be super slow either!

what exactly are you doing?

I have repos with more Markdown files to lint that just at the root level.  Sometimes there are subdirectories with README.mds of their own. Sometimes there's a monorepo with packages/ that each have their own possibilities. I'd love to recursively search for all .md files without entering any node_modules subdirectories.

👍
Why not just use 'remark .'? That should cover all cases!

Oh geez, I'm sorry! I think while iterating on something I confused myself into thinking that remark . was only ever touching the root directory. Turns out it works. Thanks for the help!

Oh no problem at all! Yeah the directory search stuff is pretty powerful. Let me know if you have suggestions for doc additions to make this more clear!

One thing that would be cool to have in the docs is a globby example for MDX files.
I am using remark-cli with remark-lint-no-dead-urls and I'm using the following in my package.json:
{
  "scripts": {
    "lint-md": "remark \"src/pages/**/*.mdx\""
  }
}


fix: TimeTrait::createFromTimestamp() method signature for PHP 8.4

Description
See #9116
Related  #9105

PHP Fatal error:  Declaration of CodeIgniter\I18n\TimeTrait::createFromTimestamp(int $timestamp, $timezone = null, ?string $locale = null) must be compatible with DateTimeImmutable::createFromTimestamp(int|float $timestamp): static in /home/runner/work/CodeIgniter4/CodeIgniter4/system/I18n/TimeTrait.php on line 268
https://github.com/codeigniter4/CodeIgniter4/actions/runs/10481860628/job/29032020981?pr=9133

Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

If we don't support PHP 8.4 in v4.5.x, we don't need this PR.

We do not support PHP 8.4 in v4.5.x.


how to swap the opponent note location and the players

What is your question?
so i have a song in my mod this song bf and the opponent are swapped sides like this:

i wana make it so that the players notes and opponent notes are swapped to make it fit better with how bf is on the left instead of right

function onCreatePost()

    local notePositions = {
		defaultPlayerStrumX0,
		defaultPlayerStrumX1,
		defaultPlayerStrumX2,
		defaultPlayerStrumX3,
		defaultOpponentStrumX0,
		defaultOpponentStrumX1,
		defaultOpponentStrumX2,
		defaultOpponentStrumX3,
	}

	for i = 0, 3, 1 do
		noteTweenX('noteTween'..i, i, notePositions[i], 1, 'linear')
	end

	for i = 4, 7, 1 do
		noteTweenX('noteTween'..i, i, notePositions[i], 1, 'linear')
	end

    end
end

This should work, but I don't know if it does because my computer doesn't want to execute the code for some reason.

local notePositions = {}

function onCreatePost()
    notePositions = {
        defaultPlayerStrumX0,
        defaultPlayerStrumX1,
        defaultPlayerStrumX2,
        defaultPlayerStrumX3,
        defaultOpponentStrumX0,
        defaultOpponentStrumX1,
        defaultOpponentStrumX2,
        defaultOpponentStrumX3
    }

    for i = 1, 8, 1 do
        noteTweenX('noteTween'..i, i-1, notePositions[i], 0.01, 'sine')
    end

end

Fixed the code, it should work now.

local notePositions = {}

function onCreatePost()
    notePositions = {
        defaultPlayerStrumX0,
        defaultPlayerStrumX1,
        defaultPlayerStrumX2,
        defaultPlayerStrumX3,
        defaultOpponentStrumX0,
        defaultOpponentStrumX1,
        defaultOpponentStrumX2,
        defaultOpponentStrumX3
    }

    for i = 1, 8, 1 do
        noteTweenX('noteTween'..i, i-1, notePositions[i], 0.01, 'linear')
    end

end

Fixed the code, it should work now.

kay thanks!

local notePositions = {}

function onCreatePost()
    notePositions = {
        defaultPlayerStrumX0,
        defaultPlayerStrumX1,
        defaultPlayerStrumX2,
        defaultPlayerStrumX3,
        defaultOpponentStrumX0,
        defaultOpponentStrumX1,
        defaultOpponentStrumX2,
        defaultOpponentStrumX3
    }

    for i = 1, 8, 1 do
        noteTweenX('noteTween'..i, i-1, notePositions[i], 0.01, 'linear')
    end

end

also is it like this?

Yeah it's correct.

Yeah it's correct.

how do i get it to work properly? it wont swap the notes let me double check tho

it works thank you so much grape boy


Fixed swagger refract and other source map related bug fixes

@pksunkara Ok, I've gone through all of this. I'm good to merge, but I just want to make sure you're OK that I really can't heavily review this. I went through each commit and looked at all of the changes, but I'm afraid that's all I can go.
Is that good with you?

Yes, I have reviewed all of this twice.


Uninstaller does not work

Attempting to run uninstaller.jar using javaw on Windows 7 does nothing. Running through the console produces no output.

Double click the installer (\Uninstaller\uninstaller.jar) and it'll launch a GUI that'll help you uninstall the app.

Double-clicking did not launch the uninstaller. I was eventually able to launch it using "java -jar uninstaller.jar" from the command line.

Do you know what version of Java you have installed?


Quality control and clustering analysis

This contains the quality control and clustering analysis right after demultiplexing analysis.

LGTM, thanks for taking care of this!


fix(HtmlSafeContext): do not set props dynamically

Dynamically setting properties has been deprecated in PHP version.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+10.0%) to 96.923%

Totals

Change from base Build<PHONE_NUMBER>:
10.0%

Covered Lines:
63

Relevant Lines:
65

💛  - Coveralls


Allow use of

Currently, if I use a template tag within my main <template native|web>, the regex pattern will stop at the first closing template and the compiler will throw an error. I have found a way around it by adding a positive lookahead for a starting <template native|web>. If none is found, it falls back to the current pattern. I don't know if that's the most elegant way to do it but at least it works :)

Can you give a short example of a what you are doing to get this error to show up?  I've done some basic testing and can't get the compiler to throw an issue.  I'd like to be able to reproduce before I merge this.

Hi!
If i recall, you only need to add a template tag within the "main" template tag, e.g.:
<template web>

        <template v-if="someCondition">

Something

            <button>My button</button>
        </template>
        <template v-else>

Something else

            <button>My other button</button>
        </template>

</template>

<template native>
    <StackLayout>
        <template v-if="someCondition">

            <Button text="My button"/>
        </template>
        <template v-else>

            <Button text="My other button"/>
        </template>
    </StackLayout>
</template>

I'm going to reject the PR as I could not reproduce this.  However, I have left your regex statement in v0.2.0 as commented out in case we need to revisit this.
Try 0.2.0 and see if this still happens.  It may very well happen for you, but in my test, I could not get any error to come up.  This may be because with with 0.2.0, we are updating everything to Nativescript 6.1 and more importantly, to NS-Vue 2.4.0.


World Quest Group Finder and World Quest tracker don't work anymore.

Description
When I installed the addon DiabolicUI, the addon World Quest Group Finder stopped to work. For some reason, I'm not able to find groups.
Another addon World Quest tracker has the same functionality and it isn't working as well. Both add-ons have the functionality to find the group manually, clicking on the right side of the quest and this functionality doesn't work. Actually, the icon isn't there.
Version Number
1.3.97-Release
Steps to Reproduce

Go to the World Quest area.

Expected Behavior

Go to the World Quest area.
Wait for the addon World Quest Group Finder or World Quest tracker ask to you if you want to find a group.
Find a Group.
Finish the World Quest.

Actual Behavior

Go to the World Quest area.
Wait for the addon World Quest Group Finder or World Quest tracker ask to you if you want to find a group.
Nothing happens.
You aren't able to find the button manually.

Only DiabolicUI Enabled
I took off all the others addons and let just DiabolicUI enabled and World Quest Group Finder and World Quest tracker enabled.

Try clicking on the middle mouse button on the quest title in the tracker :)

Since there already is built-in support for one of these addons (though poorly documented so far by me, something I will remedy in the new wiki pages here) I'm closing this issue.


Updates to the CLI

Yeah, so confusingly there is 2 different messages there. The "Witness server is not running" is actually fine - it will skip the witness tests in this case.
The error is actually coming from the interop test w/ @andrewwhitehead's DID. I'm not sure why that is happening as I can download that file right now.. I should actually remove the interop tests as this isn't the right place for them.. just was nice to have during development

And to run a "witness server" it's the same as running a resolver bun run server


fix the CSS part of Movie because we had some issues / a div deleted …

… by resolving conflict

also delete the isLoading state because the fetch also give us a result
we dont must keep it in our code for the moment


[Firefox addon] Feature detect originAttributes to make it Seamonkey 2.40 compatible

Fixes #7245.

/botio-windows preview

Looks good. Thank you for the patch.


Allow specification of matcher for patterns in TOML config file

Describe the bug
Some match/rewrite pairs only work well when specific matcher (language) is used.
Currently, the TOML config file does not allow setting matcher for pattern and it must be given on command line.
Reproducing
It makes re-using config files harder, as the required matcher parameter must be documented in the file, user must remember to check the file for correct parameters and use them.
Having to give -matcher flag on CLI forces the same matcher for all patterns in single TOML file, forcing user to split it if different matchers are needed for different patterns.
Expected behavior
Support for matcher field in TOML config file patterns
Additional context
There are other flags that may be required for pattern in TOML file to work, like the -omega flag.

Hi, thanks for this, I'd say this is more of a feature request, I'll see what I can do :-)


Update the onboarding checklist to account for new training schedule

We have recently discussed as a team that we will change our training schedule in order to remove the 10-day requirement for the contingency planning training.  In order to do this, we cannot grant full admin AWS access right away, so we are changing the onboarding checklist to account for this process.
This changeset also adjusts the order of some of the onboarding items and adds a bit more context for the different sections.
Changes proposed in this pull request:

Reorders the checklist to be more in line with the flow of work
Adjusts the training schedule
Adds more explicit references to Buddy and NewPerson
Changes AWS account setup to be read-only at the start
Explicitly calls out that Buddy will create new tickets to complete subsequent work (e.g., trainings, adjusting AWS account permissions) so as not to hold up the main onboarding ticket
Adds more descriptions for each section
Removes admin rights as an explicit task

Security considerations:

Updates our onboarding process to be a bit more streamlined and accounts for the controls we need to be mindful of when granting permissions to new team members vis-a-vis when they complete their trainings

@cloud-gov/platform-ops, this could use one more set of 👀 again to make sure my updates are good to go.  Thanks!


Update README

To reference distros supported.
See https://github.com/OE4T/meta-tegra/issues/336#issuecomment-703045517

@madisongh I'd like to backport this commit or a similar one into dunfell as well so I can reference it in the mender hub update.


Homebrew installation appears to broken

Describe the bug
The following error is given when installing any plugin using the homebrew installation, following error is seen:
Setting up snippet: OMZL::git
Downloading `OMZL::git` (with curl, wget, lftp)…
.zinit-download-file-stdout:34: no such file or directory: /usr/local/opt/zinit/share/single-line.zsh
.zinit-download-file-stdout:15: no such file or directory: /usr/local/opt/zinit/share/single-line.zsh
ERROR: Download failed.

To Reproduce
$ brew install zinit
$ cat > $HOME/.zshrc << EOF
setopt promptsubst
source /usr/local/opt/zinit/zinit.zsh
zinit wait lucid for OMZL::git
EOF
$ zsh

Expected behavior
The plugins should install.
Screenshots
Getting the following error twice per plugin:
Setting up snippet: OMZL::git
Downloading `OMZL::git` (with curl, wget, lftp)…
.zinit-download-file-stdout:34: no such file or directory: /usr/local/opt/zinit/share/single-line.zsh
.zinit-download-file-stdout:15: no such file or directory: /usr/local/opt/zinit/share/single-line.zsh
ERROR: Download failed.

Versions:

$ZSH_VERSION == 5.8
Operating system name and version (uname -a): Darwin  20.4.0 Darwin Kernel Version 20.4.0: Thu Apr 22 21:46:47 PDT 2021; root:xnu-7195.101.2~1/RELEASE_X86_64 x86_64 i386 MacBookPro16,1 Darwin

Additional context
it's looking for this file: /usr/local/opt/zinit/share/single-line.zsh
It does exist, but in this location: /usr/local/opt/zinit/share/share/single-line.zsh`

Closing since it looks like homebrew isn't an officially recommended installation methid


Exceptions when ads enabled

https://github.com/Novage/jwplayer-hlsjs/blob/91bfd11fc2916a975c326bd2a53196a5ea4f5894/src/jwplayer.hlsjs.js#L609
Error in "beforePlay" event handler: TypeError: "D.bufferController is undefined"

Looks like just removing the line helps but requires investigation...


Embassy depends on fixed which requires rust 1.79 which is newer than 1.75 set in rust-toolchain.toml

This confuses the rust-analyzer. Embassy should either bump the rust requirement or drop fixed requirement.

rust-toolchain.toml now points to rust 1.80, maybe you need to update the repo?
https://github.com/embassy-rs/embassy/blob/main/rust-toolchain.toml#L2


Update emojis list / description

Emojis list and their description is not in sync with Gitmoji anymore.

fixed


Support for logging & plotting multi-dimensional "Signal" types

Context
Many datasets use state vectors to capture robot state. For example lerobot uses a 7-dimensional vector of motor positions as their state.
Needing to break this up into a collection of separate columns/entities is somewhat awkward compared to just being able to log and plot using a ListArray instead.
New Archetype Proposal
Since Scalar vs Scalars as archetypes has the potential to be somewhat confusing, it seems worth introducing a new archetype here.
Current proposal is to name this archetype Signals which generally aligns with the use-cases we would expect to see.
Although we don't provide name in the Scalar archetype, forcing users to use SeriesLine / SeriesPoint to name  things, the need here is more evident as all these signals belong to the same entity path.
That said doing "name-based" joining of sparse signals adds a lot of complexity, and so we should still document this as being informational-only and simply produce a warning if users accidentally use mixed-name signals.
archetypes/signals.fbs
/// A collection of synchronous scalar signal values.
///
/// For example, a trace of the individual axes in the output of an IMU.
///
/// This archetype does not allow for null-valued or otherwise non-uniform-length signals.
table Signals (
  "attr.rust.derive": "PartialEq",
  "attr.docs.category": "Plotting",
  "attr.docs.view_types": "TimeSeriesView"
) {
  // --- Required ---

  /// Multiple channels worth of signal data.
  ///
  /// The logical identity of a signal is based on its index in this array and is not impacted by the
  /// value of labels component.
  ///
  /// The length of this array should be constant for an entire dataset. If multiple signal
  /// records have different lengths, any unset signal values shall be assumed to be 0.
  ///
  channels: [rerun.components.Scalar] ("attr.rerun.component_required", order: 1000);

  /// --- Optional ---

  /// Optional names for each of the signals in the signal array
  ///
  /// This component is exposed for convenience of data annotation at the log-site. Specifying
  /// inconsistent names over time is undefined behavior. However, the Rerun viewer will detect
  /// that this has happened and produce a warning.
  ///
  /// In situations where the performance and data overhead is relevant, you should only log this
  /// component once via blueprint or static logging APIs.
  names: [rerun.components.Name] ("attr.rerun.component_optional", nullable, order: 2000);
}

Visualizer Consequences
The SeriesLine and SeriesPoint visualizers should both be compatible with the new archetype.
This will require updating both archetypes to support batch, rather than mono- components for attributes such as color and width.
Known Limitations
Since visualizers act on entities (and not sub-entity selections), you will have to choose whether to plot all the signals as lines, or all the signals as points. There is not currently an effective way to mix and match.
Future work
It would be nice to somehow be able to treat these signal channels as "virtual" entities in a nested entity-path-like representation and just configure Lines/Points using a path-selection syntax. This concept needs a lot more work though and doesn't need to be a blocker for making this MVP work.

I'm not sold on the name. "Signals" is often used to mean "callbacks/delegates", especially in the context of UIs.

My original thought is the signals/slots terminology was pretty removed from our particular use-case, but I went and took a look at Vega which also happens to use signals in the way you're describing (https://vega.github.io/vega/docs/signals/) as a specification related to reactive UI input events.
@emilk any alternative suggestions?

Other names that have come up:

Vector (concerns that it's too overloaded with spatial, e.g. Vec3)
Tensor1D (not super obvious to users that haven't heard the tensor terminology before)
Array (probably too generic)
ScalarArray :shrug:

+1 on this feature, having a for loop to log state vectors on a timeline is really not ideal. Ideally the viewer can internally do the unpacking. My 2c is ScalarArray is not bad as a name. if the idea is that this is just a bundle of the original Scalar archetype.


[PL-39145] Update Harness Sass IP allowlist

Update Harness Sass IP allowlist
For reviewers, previews are available:
FirstGen
NextGen
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Where did we reference the list of IPs?

I updated them based on the new IPs in the ticket from the Slack thread:


TRANSACTION_CLOSED issue with Prisma

Someone reported that with the following Prisma example, they're always getting TRANSACTION_CLOSED error on the first write:
https://www.prisma.io/blog/prisma-turso-ea-support-rXGd_Tmy3UXX

This is the repo that I'm seeing the error on:
https://github.com/ruheni/fullstack-prisma-turso-embedded-db
My steps to reproduce are:

npm i
Add environment variables to .env:

TURSO_AUTH_TOKEN=
TURSO_DATABASE_URL=
SESSION_SECRET=llkasjdflkjasdflnasdlkjnalsdf
DATABASE_URL=file:./prisma/dev.db

npm run dev
Open localhost:3000 and try to signup

Related Discord thread: https://discord.com/channels/933071162680958986/1168952021064351886
And PR that will hopefully resolve this: https://github.com/tursodatabase/libsql/pull/559


Qdrant is missing in the docs

A landing page is missing for the Qdrant extension.
Some page like the pgvector one https://docs.quarkiverse.io/quarkus-langchain4j/dev/pgvector-store.html

Good catch!
Would you like to add one?

@zbendhiba ^

@geoand yes, can you assign this to me ?

Done!


Allow access to uwhackweeks grafana outside of 2i2c

Context
We recently received a request for Grafana access https://2i2c.freshdesk.com/a/tickets/158.
Currently, the way to do this is to enable GitHub authentication for that particular cluster Grafana and allow access to the community rep either by adding them to our 2i2c org and adding this org to the allowed list, or allowing another organization entirely, where the community rep has access.
Proposal
We should enable GitHub auth for the uwhackweeks Grafana and:

figure out if we should allow an entire org
or if we should allow only a specific GitHub team

Haven't tested last one, but there are docs about it here -> https://grafana.com/docs/grafana/latest/setup-grafana/configure-security/configure-authentication/github/#team_ids.
We would need to grab the id of that team and add that to the allow list instead of the team name. Docs about getting extra info about a team here https://docs.github.com/en/rest/teams/teams#get-a-team-by-name (thanks @sgibson91 ✨ )
Updates and actions
No response

Signaling the discussion in https://github.com/2i2c-org/infrastructure/pull/1830 which I believe is relevant for this one too

I think we should just grant people access as described in https://infrastructure.2i2c.org/en/latest/sre-guide/support/grafana-account.html?highlight=grafana

Closing as uwhackweeks cluster is terminated


[BEAM-4751] fix missing pylint3 check for io subpackage

This pull request finishes the preparation of the io subpackage for Python 3 support. The work for io was started in #5715, however the py27-lint3 test was missing in tox.ini, resulting in some open issues and missing checks for new PRs.
The PR is part of a series in which all subpackages will be updated using the same approach.
This approach has been documented here and the first pull request in the series (Futurize coders subpackage) demonstrating this approach can be found at #5053.

R: @tvalentyn @charlesccychen

The test failure looks strange, perhaps something changed on master and we should rebase this?

LGTM, thank you!


Put pytriton.client in the separate package/wheel.

Is your feature request related to a problem? Please describe.
We are building the serving solution for DL logic using Pytriton at work. We ourselves would like to separate the client stubs from the server logic as separate packages. The idea is that the users of our models use the client package, which does not have all the heavy dependencies our server code has.
Unfortunately, if we implement our client code using pytriton, the entire pytriton becomes a dependency, which includes Triton inference server itself and cuda-python package. Installing those dependencies would be a major inconvenience for the client package users and an entirely unnecessary one since neither of those heavy dependencies is actually used in the pytriton.client code implementation.
Describe the solution you'd like
It would be great and elegant if pytriton.client submodule was in a separate package, e.g. nvidia-pytriton-client, which nvidia-pytriton could depend upon. nvidia-pytriton-client itself would not require inclusion of Triton server in the wheel; the tritonclient dependency could also be reduced for it from tritonclient[all] to tritonclient[grpc, http] (removing cuda dependency group). This will allow our derived client package for our service to be very light in dependencies. I am quite sure that this will be very useful for other projects facing similar problems.
Describe alternatives you've considered
Alternatively, nvidia-pytriton package itself can by default go without Triton server and tritonclient[cuda] dependencies, which would only be included if the optional dependency group is provided (e.g. nvidia-pytriton[all]).
Additional context
If the core maintainers have no time to do so, I could prepare a pull request myself, since it seems to be a straightforward refactoring; however, such PR should be synchronized with your workflow with great care, since it alters the packaging structure and hence any CI you might have.

@piotrm-nvidia , are there any updates on the potential prioritization of this?

@martin-liu We are considering migrating PyTriton client to tritonclient repository. It will have new API much better aligned with Triton.
We have several proposals for new API revamp.
Synchronous interface
# Decoupled model with streaming

from tritonclient import Client

# Change url to 'http://localhost:8000/' for utilizing HTTP client
client = Client(url='grpc://loacalhost:8001')

input_tensor_as_numpy = np.array(...)

# Infer should be async similar to the exising Python APIs
responses = client.model('simple').infer(inputs={'input': input_tensor_as_numpy})

for response in responses:
	numpy_array = np.asarray(response.outputs['output'])

client.close()

# None-decoupled model

from tritonclient import Client

# Change url to 'http://localhost:8000/' for utilizing HTTP client
client = Client(url='grpc://loacalhost:8001')

input_tensor_as_numpy = np.array(...)

# Infer should be sync similar to the exising Python APIs
responses = client.model('simple').infer(inputs={'input': input_tensor_as_numpy})

numpy_array = np.asarray(list(responses)[0].outputs['output'])

client.close()

Active waiting
from tritonclient import Client
import time

input_tensor_as_numpy = np.array(..)

client = Client(url='grpc://localhost:8001')
client.wait_for_readiness()

model = client.model('simple', wait_for_ready=True, timeout=wait_time)

responses = model.infer(inputs={'input': input_tensor_as_numpy})

for response in responses:
	numpy_array = np.asarray(response.outputs['output'])
client.close()

Async client example
from tritonclient.aio import AsyncClient

# Change url to 'http://localhost:8000/' for utilizing HTTP client
# Opening client connection is asynchronous call
client = AsyncClient(url='grpc://loacalhost:8001')
await client.wait_for_readiness(wait_timeout=wait_timeout)
# Opening model connection is asynchronous call
model = client.model('simple')
await model.wait_for_readiness()
# Infer should be async similar to the exising Python APIs
responses = await model.infer(inputs={'input': np.array(..)}

async for response in responses:
	numpy_array = np.asarray(response.outputs['output'])

Context Manager Example
from tritonclient import Client
import numpy as np
# Context manager closes client
with Client(url='grpc://localhost:8001') as client:
      model = client.model('simple')

response = model.infer(inputs={"input": np.array(..)})

# Numpy tensor result is default output
      print(response['output'])

Using Client with GPT tokenizer
from tritonclient import Client
from transformers import GPT2Tokenizer, GPT2Model

tokenizer = GPT2Tokenizer.from_pretrained('gpt2')
return_tensors = "pt" if local_model else "np"
inputs = tokenizer("Hello, my dog is cute", return_tensors=return_tensors)

if local_model:
        model = GPT2Model.from_pretrained('gpt2')
        response = model(**inputs)
else:
        client = Client('grpc://localhost:8001').model('gpt2')
 	response = client.infer(inputs)
        client.close()
print(response)

What do you think about such solution? What do you love or hate about current client? What do you think about these examples of new client API?

@piotrm-nvidia, migrating to tritonclient sounds like a great move!
Regarding the code examples, they are generally well-crafted. However, I have a couple of questions:

The distinction between decoupled and non-decoupled modes is not immediately intuitive. Would it be beneficial to have a clearer, more explicit way to differentiate between them in the code?
The wait_for_readiness seems verbose. Would it be possible to handle this implicitly to streamline the code?

Also do you have a rough ETA of the migration?