Unnavailable to use Xdebug

Hey!
I'm trying to use Xdebug for debug Magento2 modules, the configuration seems to be good but when try to enable debug mode it just isn' working. I connected to container to see logs of ngix and I'm getting this message
[error] 46#46: *328 FastCGI sent in stderr: "PHP message: Xdebug: [Step Debug] Could not connect to debugging client. Tried: <IP_ADDRESS>:9003 (from HTTP_X_DEBUG_HOST HTTP header), localhost:9003

Technical Specifications

Docker desktop with WSL2
Ubuntu 22.04
Magento version magento/magento-enterprise-metapackage[2.4.5]
Reward version

PHPStorm configuration
Server

CLI Interpreter
Here I added some changes to test, but nothing works

Debug settings

Remote debug settings

Just let me know pls if I'm missing something in configurations, Thanks!

Hi @leoner-omnipro
You can try to configure this in the .env file:
XDEBUG_REMOTE_HOST=host.docker.internal

Sadly it's quite complicated to setup Xdebug on WSL2 and it's out of the scope of Reward.
See these articles:
https://intellij-support.jetbrains.com/hc/en-us/community/posts/4406842719378-Setting-up-debugging-with-XDebug-3-and-WSL2?page=1#comments
https://intellij-support.jetbrains.com/hc/en-us/community/posts/360010615740-Xdebug3-WSL2

Hi @leoner-omnipro
You can try to configure this in the .env file:
XDEBUG_REMOTE_HOST=host.docker.internal

Sadly it's quite complicated to setup Xdebug on WSL2 and it's out of the scope of Reward.
See these articles: https://intellij-support.jetbrains.com/hc/en-us/community/posts/4406842719378-Setting-up-debugging-with-XDebug-3-and-WSL2?page=1#comments https://intellij-support.jetbrains.com/hc/en-us/community/posts/360010615740-Xdebug3-WSL2

I already tried this config in the additional options of CLI configuration, works on different way? Or maybe I'm choosing the wrong container

Hi @janosmiko @leoner-omnipro
The same issue on clear Ubuntu 22.04.
Temp solution is:
reward debug
echo "xdebug.client_host=host.docker.internal" >> /etc/php/8.1/fpm/conf.d/xdebug.ini
echo "xdebug.client_host=host.docker.internal" >> /etc/php/8.1/cli/conf.d/xdebug.ini
exit
reward env stop php-debug ; reward env up php-debug

I updated the xdebug containers to contain those settings. I also created a new release (v0.4.3) with various bug fixes.
Could you try if it's working now?
$ reward self-update
$ reward env pull
$ reward env up -- php-debug
$ reward debug
www-data@m2-php-debug:/var/www/html$ cat /etc/php/8.1/fpm/conf.d/php-fpm.ini
[php]
memory_limit=-1

[xdebug]
xdebug.mode=debug,trace,profile
xdebug.start_with_request=trigger
xdebug.client_host=host.docker.internal

www-data@m2-php-debug:/var/www/html$ cat /etc/php/8.1/cli/conf.d/php-cli.ini
[php]
memory_limit=-1

[xdebug]
xdebug.mode=debug
xdebug.start_with_request=yes
xdebug.client_host=host.docker.internal

After adding a custom value to my .env and I restart the debug container, it's updated as well:
$ echo XDEBUG_REMOTE_HOST=<IP_ADDRESS> >> .env
$ reward env up -- php-debug
$ reward debug
www-data@m2-php-debug:/var/www/html$ cat /etc/php/8.1/fpm/conf.d/php-fpm.ini
[php]
memory_limit=-1

[xdebug]
xdebug.mode=debug,trace,profile
xdebug.start_with_request=trigger
xdebug.client_host=<IP_ADDRESS>

Thank you @janosmiko!
I'll test it shortly

Hey @janosmiko
Thanks, xdebug works as expected on clear Ubuntu 22.04

I can confirm that it is working as expected, tysm! 🚀


Python Dataframe APIs need to support pagination

The underlying implementation has pagination support, but we need to plumb it through the API surface.

Dup of: https://github.com/rerun-io/rerun/issues/7804


Issue 34 - Fix some README issues and add the final application listing.

Issue 34 notes some problems in the text for this guide.  Additionally, the guide lacks a listing of the final version of the application class.  I believe my changes address these issues.
My changes do not address a different issue, specifically that the guide refers to some code that does not exist.  Namely, a method in the repository named findByTeammatesName and code in main that exercises it (Finally, check out that other query where you look backwards, answering the question of "Who works with whom?").  I will submit a separate patch request with such code.

Thank you. I look forward to your next PR.


Can Keras use processed output tensor rather than just output tensort to compute the loss tensor?

In Tensorflow, I can do something like this
x = tf.placeholder(tf.float32, shape=[None, input_dim], name='x')
y_true = tf.placeholder(tf.int64, shape=[None], name='y_true')

y_true_one_hot = tf.one_hot(y_true, output_dim, on_value=1.0, off_value=0.0)

h1 = fully_connected(x, input_dim, activation_fn=tf.nn.sigmoid)
h2 = fully_connected(h1, input_dim, activation_fn=tf.nn.sigmoid)
y_pred_logits = fully_connected(h2, output_dim, activation_fn=tf.nn.softmax)

y_pred = tf.argmax(y_pred_logits, axis=1)

loss = tf.reduce_mean(tf.nn.softmax_cross_entropy_with_logits(y_pred_logits, y_true_one_hot))
optimizer = tf.train.AdamOptimizer(0.01).minimize(loss)

Please notice the loss is computed by y_pred_logits, instead of just using y_pred, while I still use y_pred, as the output of the model, to predicate.
Can Keras do the same thing? I mean, seems the Keras Model.compile's parameter optimizer automatically just use the output of Model...

This isn't really a bug, so I would post things like this in the google group in the future.
Yes, this is possible. The way I do it is to have the keras model output the loss, and have the target be an appropriately-sized tensor of zeros. The loss function can then be something like mean absolute error, or you can make your own lambda expression.


Goodbye packrat, hello renv

In the part "Computational reproducibility", packrat is evocated as a solution to manage package dependencies with R. I suggest replacing it in the text by renv, its successor, because packrat has been soft deprecated: https://github.com/rstudio/packrat

Thanks! I was reading about groundhog and I think I will actually replace pack rat with groundhog - this is a good extra push.

From: Pierre-Yves de Müllenheim @.>
Sent: Friday, February 16, 2024 9:33:04 PM
To: Lakens/statistical_inferences @.>
Cc: Subscribed @.***>
Subject: [Lakens/statistical_inferences] Goodbye packrat, hello renv (Issue #105)
In the part "Computational reproducibility", packrat is evocated as a solution to manage package dependencies with R. I suggest replacing it in the text by renv, its successor, because packrat has been soft deprecated: https://github.com/rstudio/packrat
—
Reply to this email directly, view it on GitHubhttps://github.com/Lakens/statistical_inferences/issues/105, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABRQPYPABROISGXTVI6NLBTYT67ABAVCNFSM6AAAAABDMTT3WWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGEZTSNBRHE2TGMY.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

I did not know groundhog! I personnally use renv. After some minutes looking for some comparisons, I am not sure groundhog would be as good as renv. I am wondering how groundhog would deal with two versions released on the same date (for github packages). Renv tracks the state of a package at each commit. But I do not know enough groundhog to ensure it is not the best, so why not :).


format info object appears to be polluted if the first parameter to logger is an object

Please tell us about your environment:

winston version?

[ ] winston@2
[x]<EMAIL_ADDRESS>

_node -v outputs: 8.11.1
Operating System? macOS
Language? ES5

What is the problem?
If the first parameter to one of the logger functions (e.g. logger.info()) is an object, it appears to pollute the info object during formatting.  The following code replicates the issue for me:
const winston = require('winston')
const { format, transports } = winston
const logger = winston.createLogger()
const util = require('util')

logger.add(new transports.Console({
    format: winston.format.combine(
        winston.format( (info) => {
            console.log(util.inspect(info, 4))
            return info
        })(),
        winston.format.splat(),
        winston.format.simple()
    )
}))

logger.info('with string %s %j', 'first string', { with: 'object' })
logger.info('without string %j', { with: 'object' })

I get the following output
{ level: 'info',
  message: 'with string %s %j',
  [Symbol(level)]: 'info',
  [Symbol(splat)]: [ 'first string', { with: 'object' } ],
  [Symbol(message)]: '{"level":"info","message":"with string %s %j"}' }
info: with string first string {"with":"object"}
{ with: 'object',
  level: 'info',
  message: 'without string %j',
  [Symbol(level)]: 'info',
  [Symbol(splat)]: [ { with: 'object' } ],
  [Symbol(message)]: '{"with":"object","level":"info","message":"without string %j"}' }
info: without string {"with":"object"} {"with":"object"}

What do you expect to happen instead?
I would not expect the with element to be present in the info object during the second call to logger.info()

I have a workaround, which seems a little brutal but does the job for me.  I simply added this formatter to top of the list:
format( (info) => {
    const keep = [ 'level', 'message', MESSAGE, LEVEL, SPLAT ]
    Object.keys(info).forEach( k => {
        if(!keep.includes(k)) delete info[k]
    })
    return info
})()

Did a little digging and think I found where this is happening:
https://github.com/winstonjs/winston/blob/919ce410af1e5e5ba82fd4c2f9af4a5a28218d5c/lib/winston/logger.js#L172-L180
So meta is getting set to the first argument and if it's an object, it gets Object.assign()d.  I'm guessing this is for some kind of backward compatibility?

The part of code above is also responsible for other issue:
When there is an object that should be stringified using toJSON() method, i.e.
        // sequelize object
        const user = await UserEntity.create({
            login: data.login,
            passwordHash,
        });
        log.error(`FINAL USER`, user);

It's context is replaced by this.write(Object.assign({}, meta, (...) and further it causes error:
[TypeError: exception] Cannot read property 'include' of undefined {
    at Model.get (C:\Users\rkroh\Desktop 3\shard\server\node_modules\sequelize\lib\model.js:3251:66)
    at Model.toJSON (C:\Users\rkroh\Desktop 3\shard\server\node_modules\sequelize\lib\model.js:4106:12)
    at JSON.stringify (<anonymous>)
    at stringify (C:\Users\rkroh\Desktop 3\shard\server\node_modules\logform\node_modules\fast-safe-stringify\index.js:11:18)
    at Format.module.exports.format [as transform] (C:\Users\rkroh\Desktop 3\shard\server\node_modules\logform\json.js:24:19)
    at DerivedLogger._transform (C:\Users\rkroh\Desktop 3\shard\server\node_modules\winston\lib\winston\logger.js:250:29)
    at DerivedLogger.Transform._read (C:\Users\rkroh\Desktop 3\shard\server\node_modules\winston\node_modules\readable-stream\lib\_stream_transform.js:184:10)
    at DerivedLogger.Transform._write (C:\Users\rkroh\Desktop 3\shard\server\node_modules\winston\node_modules\readable-stream\lib\_stream_transform.js:172:83)
    at doWrite (C:\Users\rkroh\Desktop 3\shard\server\node_modules\winston\node_modules\readable-stream\lib\_stream_writable.js:428:64)
    at writeOrBuffer (C:\Users\rkroh\Desktop 3\shard\server\node_modules\winston\node_modules\readable-stream\lib\_stream_writable.js:417:5)
    at DerivedLogger.Writable.write (C:\Users\rkroh\Desktop 3\shard\server\node_modules\winston\node_modules\readable-stream\lib\_stream_writable.js:334:11)
    at DerivedLogger.log (C:\Users\rkroh\Desktop 3\shard\server\node_modules\winston\lib\winston\logger.js:189:12)
    at DerivedLogger.(anonymous function).args [as error] (C:\Users\rkroh\Desktop 3\shard\server\node_modules\winston\lib\winston\create-logger.js:62:21)
    at UserService.<anonymous> (C:\Users\rkroh\Desktop 3\shard\server\src\services\user.service.ts:36:13)
    at Generator.next (<anonymous>)
    at fulfilled (C:\Users\rkroh\Desktop 3\shard\server\src\services\user.service.ts:10:58)

That might actually be a slight bug in sequelize too.  During the get() it references this._options.include, but never checks first if this._options exists:
https://github.com/sequelize/sequelize/blob/122bd1743f273e1f0e577759bc5ce86e0d874b65/lib/model.js#L3320-L3324
Are you able to call user.toJSON() without an error?

Yes, calling user.toJSON() directly wasn't causing error.
If you check the this object inside sequelize toJSON() call you will see that the context changes, so I think it's ok that sequelize expects include option to be guaranteed.

Fixed in #1552 and<EMAIL_ADDRESS>


Fix querying for an embedded field in array

Description
Closes #1724.
Closes #1828.
See also my comment in #1724, it has some examples of this logic.
This is the first PR for this task, without pushdown.  Pushdown will be covered in a separate PR.

Readiness checklist

[ ] I added/updated unit tests.
[x] I added/updated integration/compatibility tests.
[x] I added/updated comments and checked rendering.
[x] I made spot refactorings.
[ ] I updated user documentation.
[ ] I ran task all, and it passed.
[x] I ensured that PR title is good enough for the changelog.
[x] (for maintainers only) I set Reviewers (@FerretDB/core), Assignee, Labels, Project and project's Sprint fields.
[x] I marked all done items in this checklist.

Discussed implementation details on the call. Looks like there could be 2 possible options:

combine getAllByPath and getExactByPath adding a parameter for the kind of result we are looking for and returning a slice of values;
refactor those functions and turn getExactByPath into something like filterByPath which would be used for queries and filtering.

Also as I understand there is an undocumented feature of dot notation which we probably want to write down somewhere. It is so that when you are filtering with dot notation, the path v.foo.bar is acting more like regex v.*foo.*bar resulting in matches with the path v.0.foo.0.bar. Although when you are trying to update a document with the dot notation path v.foo.bar it should be matched exactly in order to perform the correct update operation.

@rumyantseva @w84thesunmMind blown with the dot notation :see_no_evil:. Particularly regrex part. I think having specific function for filtering is way to go here. Because getting path and getting filters are sufficiently different. But let's ask others what they think.
Yes, documentation would be great help here!
Finally, I have a question. If I have a query v.food.bar it seems it matches newly added document at v.0.foo.0.bar since the test returns nonEmptyResult. I suppose, regex is applied there.
What about if I query v.fo.bar, does it match v.0.foo.0.bar?

@noisersup @w84thesun @chilagrow thank you, everyone, for your comments! I think I now have some ideas about what I should refactor (both tests and code need some changes). I think I should also leave some comments in tests to explain how they work, or maybe at least I can include the exact _id to the filter, so it's easier to understand which documents are matched in which tests.

I close this one for now as it doesn't make much sense, but I'd like to reuse some intermediate commits. I'll open a new PR with more actual approach.


Artifactory down - impossible to install

Description :
After trying to npm install the project, it seems that the registry used isn't reachable.
Version information:

Tableau  extension version: 1.8.0 (but that seems independant from the version
OS: MacOS (don't think it's relevant anyway)

Steps to reproduce :

clean local npm packages so it gets loaded again
clone this repository
run npm install
See error

npm ERR! code ETIMEDOUT
npm ERR! syscall connect
npm ERR! errno ETIMEDOUT
npm ERR! network request to<EMAIL_ADDRESS>failed, reason: connect ETIMEDOUT <IP_ADDRESS>:443
npm ERR! network This is a problem related to network connectivity.
npm ERR! network In most cases you are behind a proxy or have bad network settings.

@ihrimech thanks for taking the time to submit this issue.  Are you still seeing this?  I just now tried running npm install against the main branch and I wasn't able to reproduce this.

Hello, and thanks for the quick response !
Unfortunatly, I'm still having the issue. And i'm pretty sure I'm not behind a proxy.

Hey @ihrimech !
Sorry about this. A workaround for this is to delete the package-lock.json file and run npm install again.
Let us know how that goes!

Works perfectly well thanks !


newton's dx() function is basically never used

Other than in tests and within a couple of isDone implementations, we never seem to actually use dx(). We should probably drop it.
cc @chinmaygarde for sanity check.

The end of some simulations cannot be reliably determined from just x(). So I am not sure you will be able to get rid of it completely. And if its not used in most case, it should get tree shaken away right?

If it exists, it's part of the API, and there's a huge developer cognitive cost to having things in the API.
In the cases where simulations want to know their velocity, they also implement their velocity, right? Can't those just have a private velocity calculation they use for isDone?

Turns out the dx has a very important call site: simulation groups that switch to a spring need to know the instantaneous velocity when they create the spring.


Identical expressions should not be used on both sides of a binary operator

This pull request is focused on resolving occurrences of Sonar rule squid:S1764 - Identical expressions should not be used on both sides of a binary operator
You can find more information about the issue here:
https://dev.eclipse.org/sonar/coding_rules#q=squid:S1764
Please let me know if you have any questions.
Kirill Vlasov

BTW: not a bug as the test is done again in the size check in the next line. So the entire line should be removed instead.

Thanks again - this was indeed bug. In case of PointList.EMPTY we need this explicit check to avoid calling methods making no sense for an empty PointList like is3D (could be true or yes)


Cancelling Google Sign In on Android & iOS throws exception instead of returning null

The code below throws a platform exception - even if I add a on PlatformException catch(e) when pressing cancel on the sign in that pops up
Future<bool> signInWithGoogle() async {   GoogleSignInAccount googleSignInAccount;   try {     await googleSignIn.signIn();     return true;   } catch (error, s) {     print(error);     print(s);     return false;   } }

`[✓] Flutter (Channel stable, v1.17.3, on Mac OS X 10.15.2 19C57, locale en-US)
• Flutter version 1.17.3 at /Users/oscardelaheragomez/Documents/Flutter/flutter
• Framework revision b041144f83 (11 days ago), 2020-06-04 09:26:11 -0700
• Engine revision ee76268252
• Dart version 2.8.4
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
• Android SDK at /Users/oscardelaheragomez/Library/Android/sdk
• Platform android-29, build-tools 29.0.3
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.5, Build version 11E608c
• CocoaPods version 1.9.2
[✓] Android Studio (version 3.6)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 45.1.1
• Dart plugin version 192.7761
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
[✓] VS Code (version 1.45.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.11.0
[✓] Connected device (1 available)
• iPhone XS • ios • iOS 13.5.1`

PlatformException (PlatformException(sign_in_canceled, com.google.GIDSignIn, The user canceled the sign-in flow.))

Please note this only occurs when debugging, if I run flutter run it works really well

duplicate of https://github.com/flutter/flutter/issues/44431


Documentation regarding new releases/latest packages?

Hi,
i have a Postal installation that i maintain. It is stable and i was wondering which improvements would have if i upgrade it. So i have two questions:

Is there any documentation to know the improvements of each release/lastest package?
How can i know which version i'm using and which one is the latest? (i installed it following the Installation guide which downloads the latest package _https://postal.atech.media/packages/stable/latest.tgz_)

thanks in advance

Thanks for the info @willpower232
Do you know how can i check which version i have installed in order to compare it with the last package?

As you are using the packages, you can run postal version to see some version information.
The version information is not recorded in github so anyone using a direct git clone would get less useful information.


ts_config requires deps even when it would be empty

ts_config(
  name = "tsconfig_build",
  src = "tsconfig-build.json",
)

Gives an error:
ERROR: /usr/local/google/home/jelbourn/material2/src/cdk/BUILD.bazel:16:1: //src/cdk:tsconfig_build: missing value for mandatory attribute 'deps' in 'ts_config' rule

right, if there is a single file then ts_config doesn't do anything - it's basically just a filegroup with no actions.


[tsp-client] tspconfig.yaml import-not-found error

Add additionalDirectories to the current tspconfig.yaml
parameters:
  dependencies:
    additionalDirectories:
      - specification/azurestackhci/Operations.Management
    default: ""

error:

TempTypeSpecFiles:

Hi @Alancere did you add the new directory to your tsp-location.yaml?

Hi @Alancere did you add the new directory to your tsp-location.yaml?

my tsp-location.yaml
directory: specification/azurestackhci/AzureStackHCI.StackHCIVM.Management
commit: 9a88bba565544d5907aaee7b10170fd770572904
repo: Azure/azure-rest-api-specs
additionalDirectories:
- specification/azurestackhci/Operations.Management

@Alancere this issue is due to the emitter-package.json not having all of the dependencies you need to be able to run the code. Here is an example of a properly configured emitter-package.json: https://github.com/Azure/azure-sdk-for-python/blob/main/eng/emitter-package.json
What does your emitter-package.json look like? You will probably need to specify additional libraries in it. tsp-client uses emitter-package.json to install necessary libraries to compile a typespec project.

@Alancere this issue is due to the emitter-package.json not having all of the dependencies you need to be able to run the code. Here is an example of a properly configured emitter-package.json: https://github.com/Azure/azure-sdk-for-python/blob/main/eng/emitter-package.json
What does your emitter-package.json look like? You will probably need to specify additional libraries in it. tsp-client uses emitter-package.json to install necessary libraries to compile a typespec project.

my emitter-package.json, please check if I am missing any dependencies.
{
  "main": "dist/src/index.js",
  "dependencies": {
    "@azure-tools/typespec-go": "0.1.0"
  },
  "devDependencies": {
    "@azure-tools/typespec-autorest": "0.44.0",
    "@azure-tools/typespec-azure-core": "0.44.0",
    "@azure-tools/typespec-azure-resource-manager": "0.44.0",
    "@azure-tools/typespec-azure-rulesets": "0.44.0",
    "@azure-tools/typespec-client-generator-core": "0.44.0",
    "@typespec/compiler": "0.58.0",
    "@typespec/http": "0.58.0",
    "@typespec/openapi": "0.58.0",
    "@typespec/rest": "0.58.0",
    "@typespec/versioning": "0.58.0"
  }
}

Should directory and additionalDirectories share a package.json instead of only existing in directory?

Looks like you have the packages that are being reported as missing. Are you sure you're running the code from a branch that has the emitter-package.json configured? Also are your emitter-package.json and emitter-package-lock.json in sync? The tool will give preference to the lock file for installing packages. I'll try to generate this package and report back here with what I find.

Yes, sharing the package.json is expected. We expect azure specs in the specs repository to use the package.json that's at the root of the repo, so both your directory and additional directories should work with the same package.json if the tsp compiled with the additional directories.

You shouldnt have a custom package.json btw in the specs repo, you should only rely on the one at the root. https://github.com/Azure/azure-rest-api-specs/blob/main/package.json

You shouldnt have a custom package.json btw in the specs repo, you should only rely on the one at the root. https://github.com/Azure/azure-rest-api-specs/blob/main/package.json

Sorry for my wording. What I meant was that the TempTypeSpecFiles directory should share a package.json.
Fix it like this: https://github.com/Alancere/azure-sdk-tools/commit/c77de0dd28b1576a5fc19c097cd912354b4274c1?

So this issue is happening here because of the configuration of the Operations.Management tsp and how it directly imports the libraries instead of relying on the imports of the main package under AzureStackHCI.StackHCIVM.Management/ which seems to be the main difference between this tsp project and others that have specified additional directories. I'm moving the package.json + package-lock.json up to the TempTypeSpecFiles/ directory as you suggested. I have a PR open to address this issue, once it's merged I'll release an updated version of tsp-client that you'll be able to use for this project.
By the way, I think you dont need the main.tsp in your Operations.Management/ directory since it doesnt seem to be defining a standalone service.


Dependency conflic Laravel 8.29

I am trying to install and this is the output:
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - marquine/php-etl v2.3.0 requires illuminate/container ^5.0|^6.0 -> found illuminate/container[v5.0.0, ..., 5.8.x-dev, v6.0.0, ..., 6.x-dev] but these were not loaded, likely because it conflicts with another require.
    - marquine/php-etl v2.3.1 requires illuminate/container ^5.0|^6.0|^7.0 -> found illuminate/container[v5.0.0, ..., 5.8.x-dev, v6.0.0, ..., 6.x-dev, v7.0.0, ..., 7.x-dev] but these were not loaded, likely because it conflicts with another require.
    - Root composer.json requires marquine/php-etl ^2.3 -> satisfiable by marquine/php-etl[v2.3.0, v2.3.1].

Installation failed, reverting ./composer.json and ./composer.lock to their original content.

Using -W switch:
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - marquine/php-etl v2.3.0 requires illuminate/container ^5.0|^6.0 -> found illuminate/container[v5.0.0, ..., 5.8.x-dev, v6.0.0, ..., 6.x-dev] but these were not loaded, likely because it conflicts with another require.
    - marquine/php-etl v2.3.1 requires illuminate/container ^5.0|^6.0|^7.0 -> found illuminate/container[v5.0.0, ..., 5.8.x-dev, v6.0.0, ..., 6.x-dev, v7.0.0, ..., 7.x-dev] but these were not loaded, likely because it conflicts with another require.
    - Root composer.json requires marquine/php-etl ^2.3 -> satisfiable by marquine/php-etl[v2.3.0, v2.3.1].

Installation failed, reverting ./composer.json and ./composer.lock to their original content.

Hi @lucianobargmann
You can use our fork :)  https://github.com/wizaplace/php-etl
Changelog here


Add an example on how to ingest data via STAC catalog

@mcovalt I believe I heard that we are very close. Can we put together an example on how it can be done. We need this asap for our University customers.

This is pretty much done in #461. It's rather simply really:
simply load each raster/STAC Item as a dict in Python and pass that to the RGD Python client:
stac_item = ...  # load JSON as dict

client.create_raster_stac(stac_item)

This requires that each asset in the STAC item have the appropriate data tag and the files be pre-uploaded to S3.


In workload details, checking a pod pops up buttons that are in wrong position

Steps:

Create a deployment
Go to the deployment details page and scroll to the Pods table.  (issue 1, checkboxes but no buttons?)
Click on a checkbox

Results:  The download yaml & delete show up but they are partially on top of the Pod title

[ ] Issue 1 - not sure if this is on purpose or not.

[ ] issue 2 - bug

Duplicate: https://github.com/rancher/dashboard/issues/1287


Implement HTTP module

HttpFileSystem -> sits on top of HttpClient
HttpClient
HttpServer (only a simple server for serving debug pages and remote-controlling the app)
HttpRequest, HttpResponse
support for JSON, XML, JSON-RPC, XML-RPC stuff
prefer platform-specific HTTP APIs or use CURL as fallback

closing old tickets


Refactor workers manager

It will always react to DOWN messages so there's no way a worker will be "lost"

Changes Unknown when pulling 1950da916deca100086e08f47ee5c0bb909bf067 on refactor-workers-manager into ** on master**.

Changes Unknown when pulling 7482711cc89d24d87e6f50f8feccfd93eb5cb43b on refactor-workers-manager into ** on master**.


fix(ops): support returning non-anyhow Result from fast sync op

// compiles
#[op2]
pub fn op_slow_anyhow_result(_: &mut v8::HandleScope) -> anyhow::Result<()> {
    Ok(())
}

// compiles
#[op2(fast)]
pub fn op_fast_anyhow_result() -> anyhow::Result<()> {
    Ok(())
}

// compiles
#[op2]
pub fn op_slow_std_io_result(_: &mut v8::HandleScope) -> std::io::Result<()> {
    Ok(())
}

// does not compile (fixed by this PR)
#[op2(fast)]
pub fn op_fast_std_io_result() -> std::io::Result<()> {
    Ok(())
}

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

Looks good to me. Could you add a corresponding section to the ops/op2 README with a note that return values need to be into anyhow::Error?


Problem: need Mina PCB logging PR in our repo

Related:

https://github.com/MinaProtocol/mina/pull/13236
https://github.com/Granola-Team/mina-indexer/issues/211

This task was for the pre-berkeley network

Adding another for post-berkeley network


Try addressing remaining closed connection issues

Now only the peerdata one.
/cc @sneako

This has not affected the rate of connection closed errors, and I am also seeing the port leak that we fixed with NimblePool.precheckin return, so I have opened a PR to put back that function call https://github.com/keathley/finch/pull/73

Hmm, I am actually still seeing the port leak with the precheckin in place... We may need to revert #73 and #71


Various args and test fix

Some test doesn't pass locally. Consequently, I propose the following fix/improvement:
1. Remove unused peft_test marker
tests/test_ppo_trainer.py:910
  /fsx/qgallouedec/trl/tests/test_ppo_trainer.py:910: PytestUnknownMarkWarning: Unknown pytest.mark.peft_test - is this a typo?  You can register custom marks to avoid this warning - for details, see https://docs.pytest.org/en/stable/how-to/mark.html
    @mark.peft_test

2. Don't report to wandb
wandb does not cope well with multiple executions in the same process, and generates errors such as
wandb.sdk.lib.config_util.ConfigError: Attempted to change value of key "model/num_parameters" from 1658681 to 1663801

the easiest is not to report during test. Already fixed for reward tester in #1895
3. Don't share model across tests
Also partially addressed in #1895. When sharing the model, it creates conflicts between tests, due to model re-training mostly.
FAILED tests/test_dpo_trainer.py::DPOTrainerTester::test_dpo_trainer_w_dataset_num_proc - _pickle.PicklingError: Cannot pickle a prepared model with automatic mixed precision, please unwrap the model with `Accelerator.unwrap_model(model)` before pickling it.

The solution is to replace setUpClass by setUp (so that the setup is not shared)

Finally 😮💨


fix: move interfaces/ dir under the src/, documentation update, rearrange npm scripts

fix: move interfaces/ dir under the src/
docs: documentation update
chore: rearrange npm scripts
revert: changeset for notios 0.3.0

Types of changes

[ ] Bug fixes

resolves #

[ ] Features

resolves #

[ ] Maintenance
[ ] Documentation

Changes

...

Additional context
Community note

Please upvote with reacting as :+1: to express your agreement.

CI does not running...


Add Perks Tutorial Part 2

Not sure if we are sticking with the name "Perk Systems", for now I have appended "Part 2" until we think of something better.

@PopThosePringles Wanna add the forums link now? Then I'll include it in today's deploy.


Update CI to use tools from vendor folder

Currently we fetch the binaries during CI runs, but it's better to put it in the vendor folder and execute directly.
This PR puts 3 tools in the vendor folder:

golangci-lint
ginkgo
envtest

/lgtm
/approve
We can probably override the E2E here as this is a repo tooling only change

/override ci/prow/e2e-aws
/override ci/prow/e2e-aws-capi-techpreview
/override ci/prow/e2e-aws-serial
/override ci/prow/e2e-aws-techpreview
The changes in this PR do not affect the E2E jobs listed above, so no point re-running them


CrankShaft Logo

Every big project needs a banner to fly.  If you have great design skills or atleast want to be apart of the process of picking a Logo. Chime in here.
If someone has replied that they want to contribute their design skills, that does not mean you cannot. If we get multiple submissions that makes things easier and we can choose from the best.
Things to keep in mind - This is not an official HubSpot Repo, we do not own any of their copyrights or trademarks. That does not mean we cannot keep a visual theme.
If you want to design a logo for CrankShaft you can do so either by submitting a pull request with your logo files OR by posting the logo to this issue.

I'll take a stab at it.

Andreas Wessoly sent me these concepts.

What I like

it's a semi-literal crankshaft interpretation
It's got a similar-to HubSpot aesthetic without being overt about it.
I can see this thing being animated

What maybe could be improved upon

I'm not a mechanic - when I look at it I do see Pistons which is cool but I think a literal crankshaft looks more like this:

Pistons are mounted to the crankshaft

oOOooohh. thank you for the explanation, my mechanic father would not be proud of me lol.
I like the logo direction you went with. Pretty dang spiffy.

Thanks, If you have any thoughts about it let me know and I'll put some final touches and then do a pull request with several files size and types / icon buttons.

feels like the icon may need some minor clean up - like the lines may not line up (I get that that it's intentional that there's like 1-2px gap along the sides similar to the photos. I'm not talking about that) just looks like the lines are not actually parallel
I could be wrong, just looks like it to me.
https://spin.d.pr/o6246U
also really minor the circles could be centered better on the ends of the lines.
Otherwise I personally love it.
Another super minor think, but maybe reduce the letter-spacing very slightly, might just be me, it's just slightly hard to read. (keep the effect just saying a little less might make it easier on the eyes)
Again those notes are super minor.
I think we once you're done tweaking maybe we'll post it to the slack and do a /poll and see if people like it, if majority like it by end of day today, then we'll go with it

Okay Great, I probably won't get back around to it until later tonight and Ill DM it to you on slack so you can do whatever you want as far as the Poll Goes.

Discussed in slack, just making sure it's logged here.
if anyone wants to contribute their logo ideas, you have until next Friday
If by Friday no one has submitted anything new we’ll go based on the poll we took above(including new votes that come in)
And polish whichever logo the majority wanted.
This is to avoid the creators of the existing logos going through a lot of trouble to polish them up or make changes that might get scrapped if the logo isn't chosen. This also buys time for those who are interested in designing a logo that didn't yet get a chance.
The due date of Friday, gives a weekend and a full work week to come up with something.

Hello everyone, here is another Logo Idea that i came up with:

another logo contender from @joneichler

We're going to hold a vote early I think due to popular demand in slack

Jon Eichler's logo won the vote.
If someone wants to contest this the policy is as follows:
For those that wanted  logo. If you post an issue and can get 5 more “votes” than Ikes got(meaning Ikes total from the poll + 5 people posting and confirming they will vote 1 single other logo) we can do a revote. Otherwise we’re moving on to the code and docs for the project and I hope everyone can come along and help with that stuff too.


Error when trying the create jaspr project command.

`C:\Users\saifk\AppData\Local\Pub\Cache\bin>jaspr create jasp_web_app
Creating jasp_web_app using template web-simple...
.gitignore
analysis_options.yaml
CHANGELOG.md
pubspec.yaml
README.md
web\index.html
web\main.dart
web\styles.css
lib/app.dart
lib/app.dart
Resolving dependencies...
Null check operator used on a null value
package:pub/src/entrypoint.dart 157:60                           Entrypoint.lockFilePath
package:pub/src/entrypoint.dart 100:21                           Entrypoint._loadLockFile
package:pub/src/entrypoint.dart 97:42                            Entrypoint.lockFile
package:pub/src/entrypoint.dart 286:21                           Entrypoint.acquireDependencies.
package:pub/src/entrypoint.dart 280:68                           Entrypoint.acquireDependencies.
package:pub/src/log.dart 428:18                                  progress
package:pub/src/entrypoint.dart 280:26                           Entrypoint.acquireDependencies
package:pub/src/command/get.dart 52:22                           GetCommand.runProtected
package:pub/src/command.dart 183:45                              PubCommand.run.
package:pub/src/command.dart 183:33                              PubCommand.run.
dart:async                                                       new Future.sync
package:pub/src/utils.dart 109:12                                captureErrors.wrappedCallback
dart:async                                                       runZonedGuarded
package:pub/src/utils.dart 126:5                                 captureErrors
package:pub/src/command.dart 183:13                              PubCommand.run
package:args/command_runner.dart 209:27                          CommandRunner.runCommand
package:dartdev/dartdev.dart 231:30                              DartdevRunner.runCommand
package:args/command_runner.dart 119:25                          CommandRunner.run.
dart:async                                                       new Future.sync
package:args/command_runner.dart 119:14                          CommandRunner.run
package:dartdev/dartdev.dart 66:29                               runDartdev
C:\b\s\w\ir\cache\builder\sdk\pkg\dartdev\bin\dartdev.dart 11:9  main
This is an unexpected error. The full log and other details are collected in:
C:\Users\saifk\AppData\Local\Pub\Cache\log\pub_log.txt

Consider creating an issue on https://github.com/dart-lang/pub/issues/new
and attaching the relevant parts of that log file.
Null check operator used on a null value
#0      Entrypoint.lockFilePath (package:pub/src/entrypoint.dart:157:60)
#1      dumpTranscriptToFile (package:pub/src/log.dart:375:50)
#2      PubCommand.run (package:pub/src/command.dart:225:13)

#3      CommandRunner.runCommand (package:args/command_runner.dart:209:13)

#4      DartdevRunner.runCommand (package:dartdev/dartdev.dart:231:18)

#5      runDartdev (package:dartdev/dartdev.dart:66:16)

#6      main (file:///C:/b/s/w/ir/cache/builder/sdk/pkg/dartdev/bin/dartdev.dart:11:3)
`

This seems to be an issue with running dart pub get. What happens if you run it manually? What version of dart and jaspr are you using?

The jaspr project was still generated right?

Can you try with jaspr serve -vand show the output.

Should be fixed in<EMAIL_ADDRESS>


Role: list_roles() fails with  Input should be a valid string [type=string_type, input_value=[], input_type=list]

call to api_instance.list_roles() is failing:
import json
import sailpoint
import sailpoint.v3

from typing import List
from sailpoint.configuration import Configuration

configuration = Configuration()
with sailpoint.v3.ApiClient(configuration) as api_client:
    i: int = 0
    role: sailpoint.v3.Role = None
    roles: List[sailpoint.v3.Role] = []
    str_role: str = ""
    json_role: dict = {}
    api_instance = sailpoint.v3.RolesApi(api_client)

    try:
        roles = api_instance.list_roles()
        print("==== Records found: %s" % len(roles))
        for role in roles:
            i += 1
            str_role = role.model_dump_json(by_alias=True)
            json_role = json.loads(str_role)
            print(f"Item: {i}: Json: {json.dumps(json_role, indent=2)}")
    except Exception as e:
        print(f"Exception: {e}")

I'm getting this :
python3 test-roles.py
Exception: 1 validation error for Role
dimensionRefs
  Input should be a valid string [type=string_type, input_value=[], input_type=list]
    For further information visit https://errors.pydantic.dev/2.9/v/string_type

This is fixed with the following commit https://github.com/sailpoint-oss/python-sdk/commit/fd837e5b5ed6930539c27abd0c572ac87548738d and will be in the next release!


precip viz function

I'll take this, but if anyone else wants it, they can snag it from me. I have it because I am working on #10 and I need to create fetch_precip_bins and make some decisions on colors - so anyone working on this will probably want to discuss.

fixed in #60


How to change "kvStoreName" for an index

Have a bucket with 3 million items and started indexing against it couple of days back, so far only 18116 documents have been indexed. At this rate, I'm not sure how long will it take to complete indexing.
JFYI, box has loads of system firepower(24 cores, 150+GB RAM, spinning disks which probably can handle 5K - 10K IOPS) - right now box is sitting idle for most of the time.
Is there is way to change "kvStoreName" from default boltdb to something else? I'm willing to try out leveldb/rocksdb and see if LSM based storage engines would speed up indexing or not.

Found it hiding inside advanced settings :)


Updates copy in Shows tab on Favorites

Links to LD-346

Fails

:no_entry_sign:

danger-swift` failed.

Log
Cloning and building inline dependencies: import DangerSwiftLint // package: https://github.com/ashfurrow/danger-swiftlint.git, this might take some time.
ERROR: 💥  Failed to compile script

  Generated by :no_entry_sign: dangerJS against 7fa821704495eacf513f1d23e6dab341ad637f8d


May need to import render

I've tried a tutorial "Build Python Django apps with Microsoft Graph".
Then I found that I needed an additional code at last section in step "Create a Python Django web app".
I added a below line in top of ./tutorial/views.py .
from django.shortcuts import render

If needed please confirm this.
Thank you for your great tutorial.

Yes, that is required. Thanks for reporting, I'll get this fixed.


feat(rust): added bind check before using a user supplied port

Added
Bind check before using the user supplied port in node create, tcp-listener create and tcp-inlet create
Helper function is placed under utils
Ouput

ran the following commands

Checks

[x] All commits in this Pull Request are signed and Verified by Github.
[x] All commits in this Pull Request follow the Ockam commit message convention.
[x] I accept the Ockam Community Code of Conduct.
[x] I have accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam-contributors repository.

Thank you for another awesome contribution! 🥳 🎈


Epub fails to download

Before you open an issue
Please make sure you've upgraded to the latest version of kobodl on pypi or docker.  Also read through the troubleshooting guide.  Try running with --debug enabled and see what comes up in the debug log.
Do not erase this template, it's here to help you.
Describe the issue
An ebook I was trying to download failed.
To Reproduce
Steps to reproduce the behavior:

Run kobodl-windows.exe book get UUID
Download attempts to start, but fails.
Stack trace:

Traceback (most recent call last):
  File "kobodl\__main__.py", line 5, in <module>
  File "click\core.py", line 829, in __call__
  File "click\core.py", line 782, in main
  File "click\core.py", line 1259, in invoke
  File "click\core.py", line 1259, in invoke
  File "click\core.py", line 1066, in invoke
  File "click\core.py", line 610, in invoke
  File "click\decorators.py", line 33, in new_func
  File "kobodl\commands\book.py", line 83, in get
  File "kobodl\actions.py", line 231, in GetBookOrBooks
  File "kobodl\kobo.py", line 348, in Download
  File "kobodl\kobo.py", line 268, in __DownloadToFile
  File "requests\models.py", line 1021, in raise_for_status
requests.exceptions.HTTPError: 403 Client Error: Forbidden for url: https://getbook.kobo.com/koboid-prod-private/...
[24800] Failed to execute script '__main__' due to unhandled exception!

Debug log will be emailed.
Expected behavior
Book should download
Screenshots
N/A
Desktop (please complete the following information):

OS: Windows 10 Pro
Install method and python version: I can't recall but I'm assuming it's the version of python in my program files (3.12.1)? I would have expected it to be referenced in my path environment variable but it's not (I don't typically use python, so I'm not as familiar with the required environment).
Kobodl Version: should be the latest, as my date modified on the exe is 22/01/2023. However, when I run kobodl-windows.exe --version I also get an error:

Traceback (most recent call last):
  File "kobodl\__main__.py", line 5, in <module>
  File "click\core.py", line 829, in __call__
  File "click\core.py", line 781, in main
  File "click\core.py", line 700, in make_context
  File "click\core.py", line 1212, in parse_args
  File "click\core.py", line 1048, in parse_args
  File "click\core.py", line 1630, in handle_parse_result
  File "click\core.py", line 123, in invoke_param_callback
  File "click\decorators.py", line 295, in callback
RuntimeError: Could not determine version
[7204] Failed to execute script '__main__' due to unhandled exception!

Additional context
N/A

This is caused by the same problem that causes audiobooks to not download. https://github.com/subdavis/kobo-book-downloader/pull/124 should fix it.

I've since found another half dozen or so failures with this same error. I'm guessing it's related to the audiobooks problem (as mentioned above) and possibly the add user problem seen on #121 ?

Going to add another plea for this and other 403 bugs to be fixed, as I found another dozen or so books today that won't download. I realise it's the holiday season, so I'm not expecting something immediate, but these bugs have been open for at least 2 months now with no real update on whether they're even being progressed. I hope we might see at least a progress report sometime soon.


[BUG]: Mapping of Calibrate operator is stuck in "Please wait..." indefinitely

Describe the bug
Molly from MITRE reported this bug and I was able to reproduce it in their project on Prod.
https://app.terarium.ai/projects/15e6144e-d8ac-4438-b9e4-a6704ef5a41c/workflow/9f4dec9a-088b-4443-bf4c-daa4bbf799e7
The "Mapping" section of the Calibrate operator is either incredibly slow or stuck indefinitely in "Please wait...".
This happens with Dataset 1 but not Dataset 2. It's likely because of the size of Dataset 1 (~10 MB or >10,000 rows).
Dataset 1: https://drive.google.com/file/d/1L8uGgNzo2KEqk_IvXgwXtShEICVD_Ww8/view?usp=drive_link
Dataset 2: https://drive.google.com/file/d/1HghnPA0_ET8qeCiM_DxylMPj9P4VouC5/view?usp=drive_link
To Reproduce
Steps to reproduce the behavior:

Upload any model and select some default configuration
Upload Datasets 1 and 2
Connect each to a Calibrate model operator
See that "Mapping" of the operator connected to Dataset 1 is stuck

Expected behaviour
The operator might be trying to load the entire dataset when it only needs to read the first row to get the column headers.
It might also be doing this before loading the model and reading its state variables and observables.
In any case, the usage of a large dataset shouldn't be bogged down already at the calibration settings step.
Screenshots

Additional context
Add any other context about the problem here.

There are two issues here that will be fixed

With a large datafile it can take a few seconds to finish initializing. Testing that first datafile of 10mb it will take approximately 6 seconds locally
when plugging in a simulation result we the columns field in the dataset asset is not filled in and therefore we do not have columns to select


Convert VAD to Ekman

Hello,
This model provides VAD values in 3D space.
However, the Ekman model is more intuitive to share the results with users.
I have found papers with 3D representations hinting at how to perform this conversion.
Are you aware of a straightforward approach to perform the conversion between both models?
Ideally in Python, but any hint on the algorithm would also do.
Best,
Ed

The VAD model is only fine-tuned on the MSP-Podcast dataset, which has several shortcomings for a full blown VAD model:

Podcast recordings most likely do not contain all possible emotions, e.g. fear
The dominance and arousal annotations show a high correlation, that is mimicked by the model, which means we most likely do not cover the 3D space of VAD in a meaningful way

Having this in mind I would propose to be very carefully when trying to map the VAD values to emotional categories.
Another way might be to further fine-tune the model on a given database containing the desired emotional categories, or using the embeddings of the model to train a simple classifier on such a database like we do in the notebook under the "https://github.com/audeering/w2v2-how-to/blob/main/notebook.ipynb" section.

@hagenw
Thanks a million for the clarifications.
In general, the conversion from VAD to Ekman seems to provide useful results:
https://github.com/mirix/approaches-to-diarisation/tree/main/emotions
However, it is true that fear is never detected.
I will see what other models are available and pay more attention to which datasets were used.

Hi @hagenw
I have forked MOSEI for SER:
https://huggingface.co/datasets/mirix/messaih
https://github.com/mirix/messaih
Now I will try to train a model and test it in a real-life scenario.


adding module to add missing tags to aws autoscaling group

@akasprzok I'm coming for you ;)

Legibility really isn't TF's strong suite, eh?

@akasprzok lol... TF doesn't do deeply nested object array object well.


VM filter list does not allow to filter VM's based on VMI status

We will need to improve tectonic in order to do so

/remove-lifecycle rotten
/reopen
Still valid - okd-core status filtering needs to be provided for additional resources.


Archive this?

Hey pals,
We archived the amen-js repo last year.  What say we archive this overall?

Suits me!


Align the OSS ConPTY API with Windows 11 24H2

This removes ClosePseudoConsoleTimeout as waiting for conhost
to exit is prone to deadlocks due to incorrect API usage.
Additionally, this publicly exposes ReleasePseudoConsole.

Tests are broken ;)


Fix panel active not remove between views and GoBack failure between different view.

1)Fix issues #850 #866 #873;
2)Fix explorer GoBack failed between different view;
3)Fix back button appended by code is invalid when 'click' on some device;
4)Add parameter 'back' to indicate whether transition is back in event 'panelunload' and 'panelload';
5)Code review for run 'grunt'
Major fixed issue is about panel active status between different views. Test case can use appframework DEMO. When loginview is modal, activated panel in 'mainview' is still active. After patching my code, the activated panel is remove.
For 'back' in event 'panelunload' and 'panelload', we can know whether transition is back.

Thanks for the commit.  I will review this today.

We now have an issue with the slide up transition.  Since you remove active class on it, the content is white as the panel slides up.  To test this, open up the kitchen sink app and click touch events.  The content view content is hidden, but the header is still available.

https://github.com/01org/appframework/tree/ArcRain-master
I pushed a fix up there for the slide up transition. Can you please check it out?

OK. I'll check it out and test again.

I have checked it your fixed version out. Before I replaced changed files, I noticed that the content is white while doing slide up transition. With your fixed version, the issue has gone.
And I also checked my web app project with your fixed version. It works perfectly as I expect.
Thanks for your time and fixing up.


PWM & observation frequency

Hi @SvenGronauer
Thank you a lot for contributing to this repo, it is really useful!
I have a question about how the frequency of PWM and observation (100hz) is determined. Is it a built-in speed of Pybullet, or it would get affected by the computer hardware speed?  I also saw for the env step loop, there is something like time.sleep(0.05), time.sleep(1/60). why do we need time.sleep()? will it change the 100hz frequency?
Look forward to your reply!
Best,
Xubo

Dear @xubo92,
Glad to hear. First of all, I set the simulation time step of PyBullet to 5ms (i.e. 200Hz):
https://github.com/SvenGronauer/phoenix-drone-simulation/blob/b43e8881f8a253b1984f97924d0fcd59311573c2/phoenix_drone_simulation/envs/base.py#L44
The observation time step is a multiple of the physics time step: I used 100Hz for PWM control. The numerical results of the physics simulator are not affected by your hardware but only the computation time.
The time sleep calls (e.g. time.sleep(1/60)) are only used for rendering purposes and has nothing to do with the internal physics computations.
Cheers,
Sven


storage/reads: table should return empty as true when the table is empty

During a past refactor for storage/reads, the Empty() method was set to always return false for simplicity. We should see if it is easy enough to have Empty() return a meaningful value to improve query efficiency and reduce confusion about the return value.

It seems really unintuitive to have a method called Empty() that will return false when the table is empty.  I understand why we may not know if the table is empty until we actually try to iterate over it, but the potential for a bug resulting from this behavior seems high.
Other possible ways to address this confusion:

Change the name of the method to EmptyHint() or something
Make Empty() part of an optional interface that not all implementors of flux.Table have, if the underlying data model doesn't support it.

From my comment here https://github.com/influxdata/flux/pull/1414#discussion_r296946105
Empty is not a hint and all implementations need to implement it correctly. This need stems from the need to be able to encode and decode empty tables in CSV. I don't think this need is unique to the CSV format and I would expect other formats to need to understand how to encode empty tables explicitly. This is because while tables are empty (they have no rows) the table still has a group key and its values that need to be encoded.
I believe this issues is a non-issue though because the storage RPC calls never return empty tables. Meaning at the storage layer if there was no block to decode with values then no table is produced and returned over the RPC interface. As such the current return false implementation of the table implementation in storage/reads is correct.

@nathanielc I don't think that's true, but we'll have to talk to @stuartcarnie because the previous implementation of storage/reads did return an actual value for it and the test seems to pass on CI if I stop relying on Empty().
I'm not sure what you mean since I think it would be possible to encode and decode empty tables in CSV without Empty() (you just delay writing the header until you're in the call to Do). It also makes the Do implementations more complicated for each of the tables. But, I've added a bug label to this so it can be fixed.
We can talk about it more tomorrow. If we're deciding that the problem is in influxdb, then influxdata/flux#1414 can likely be closed instead of merged.


📖 Add some internal documentation pkg/placement/doc.go

Summary
This PR adds some comments in pkg/placement, mainly adding an outline the newly created doc.go file.
Related issue(s)
Fixes #

/cc @waltforme
/cc @yana1205

/lgtm
/approve


Question about evaluation using new data set

Hi, I want to use TMO for new data set. My data set is such that in some videos it has single object, but it also has multiple objects in some videos. In this case can I use the code of " if mode == 'val'" in trainer.py file to calculate F and J for my dataset?

Hi,
It depends on how you defined the target objects.
If multiple objects are identically annotated with 1, you can directly use the validation code.

Hi again,
Sorry for asking so much questions, I am new in this area, would you please give more clue?

You mean to use validation code directly the pixel values of a frame should be 0 and 1, such that 0 must represent all the non objects area(Background) and 1 must represent all the objects(Multiple objects in my case)?
My data set is been normalized by dividing each pixel by 255.0, is this format ok for using validation code directly?
My last question is about the TMO's decoder, according to current structure of TMO, its encoder is only extracting appearance information and short term temporal information, after feature extraction sum of these 2 are given to decoder to generate object mask. I want to know if we change the decoder to consider long term temporal information also, in that case do we need to change decoder also or the decoder has that capability to consider these new features with old features?

I would appreciate your patience and response.


row_limit doesn't seem to work, but I could have done it wrong?

Description
In #125 row_limit was added per my request, but I'm having a difficult time using it.
code sample
I've added the following card to my dashboard:
type: custom:formulaone-card
card_type: constructor_standings
title: Constructor Standings
standings:
  show_teamlogo: true
  row_limit: 3
I also tried this variant:
type: custom:formulaone-card
card_type: constructor_standings
title: Constructor Standings
row_limit: 3
standings:
  show_teamlogo: true
Expected behavior
I expect the card only to show the top 3 results of the constructors standing, however the card still shows the full constructor standings list. Same for the drivers standing card
Desktop (please complete the following information):

OS: MacOS
Browser: Vivaldi
Version: 5.6.2867.50

I am probably doing something wrong...

Huh, it now does seem to work for the driver standings card, but not yet for the constructors.

 I forget the code in the constructors card. Let me fix that

should work in 0.8.2


Various fixes surrounding OutputTimeFn

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

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

These are some bugs left behind by changes to OutputTimeFn.

R: @dhalperi


Eldritch Step Distance

Game 1.5.94
UB: <IP_ADDRESS>
It appears as though the eldritch invocation Eldritch Step allows for a 20-25 foot teleport. 25 (5 cells) if travelling in on the x/y axis, 20 if deviating therefrom.
Eldritch Step teleports 20.zip

There isn’t much we can do on how game calculates distance. Won’t fix.

To be clear, I'm not quibbling about the fluctuation in distance, I get that is just a bit funky. The description for the ability says 30 ft, so I figured it should be 30 ft in a straight line instead of 25. I looked back at my comment and it never mentioned the description - my bad.


Prevent duplicate ontology entry display in the project summary

What happened?
When I create multiple experiments but with the same species / specimen / analyte, the summary shows duplicate entries:

What browsers are you seeing the problem on?
Firefox
Relevant log output
No response
Code of Conduct

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

reported by @oquendoM

no triage needed, obvious bug


[EPIC] Plugin SDK

Context
Instead of implementing the raw generated Guest trait from witbindgen, skyforge should offer its own plugin SDK.
Description
The plugin SDK should offer users the ability to communicate through a function hook system and interface with the rust CLI, seamlessly.
It should make plugins as easy to write as possible so that users can focus on the implementation details of the cloud provider and not have to worry about things like:

Interpreting starlark structures and functions (at all if possible)
Worrying about having to implement the raw wit_bindgen generated Guest trait. These details can be hidden under the hood.

Acceptance Criteria
This is an EPIC, and as such is hard to specifically point to an acceptance criteria since the initial interface still needs to be defined. However, we can outline a few key points.

Initially, add a structure to the SDK with a procedural macro that can generate implementations that hide the Guest trait implementation under the hood.
Define the interface of the plugin and its methods both for interpreting starlark code and for parsing and creating resources.
Users should not have to worry about starlark interpretation of their commands, these things should be generated by a proc macro

/triage
/assing
/label Feature

/label epic

/retitle Plugin SDK

/label epic

/label epic


Android build fail  - Unable to make progress running work. There are items queued for execution but none of them can be started

Description
We recently upgraded my project from react-native:068.7 to 0.73.2.
We get the below errors when we run the Android build in react-native:0.73.2.
we are using the gradle version 8.3 in my project.

Unable to make progress running work. There are items queued for execution but none of them can be started
2)Unable to make progress running work. There are items queued for execution but none of them can be started

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.
Get more help at https://help.gradle.org.
Deprecated Gradle features were used in this build, making it incompatible with Gradle 9.0.
You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.
For more on this, please refer to https://docs.gradle.org/8.3/userguide/command_line_interface.html#sec:command_line_warnings in the Gradle documentation.

Steps to reproduce
build the app in android studio
React Native Version
0.73.2
Affected Platforms
Runtime - Android
Output of npx react-native info
System:
  OS: macOS 14.2.1
  CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
  Memory: 34.03 MB / 16.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 21.7.3
    path: /usr/local/bin/node
  Yarn: Not Found
  npm:
    version: 10.5.0
    path: /usr/local/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /usr/local/bin/watchman
Managers:
  CocoaPods:
    version: 1.15.2
    path: /Users/gopi/.rvm/gems/ruby-2.7.6/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.2
      - iOS 17.2
      - macOS 14.2
      - tvOS 17.2
      - watchOS 10.2
  Android SDK:
    API Levels:
      - "23"
      - "24"
      - "25"
      - "26"
      - "27"
      - "28"
      - "29"
      - "30"
      - "31"
      - "32"
      - "33"
      - "34"
    Build Tools:
      - 29.0.2
      - 30.0.3
      - 33.0.1
      - 34.0.0
    System Images:
      - android-29 | Google APIs Intel x86 Atom
      - android-29 | Google Play Intel x86 Atom
      - android-30 | Google APIs Intel x86 Atom
      - android-32 | Google APIs Intel x86 Atom_64
      - android-32 | Google Play Intel x86 Atom_64
      - android-33 | Google APIs Intel x86 Atom_64
      - android-33 | Google Play Intel x86 Atom_64
    Android NDK: Not Found
IDEs:
  Android Studio: 2021.2 AI-212.5712.43.2112.8609683
  Xcode:
    version: 15.1/15C65
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.10
    path: /Library/Java/JavaVirtualMachines/jdk-17.jdk/Contents/Home/bin/javac
  Ruby:
    version: 2.7.6
    path: /Users/gopi/.rvm/rubies/ruby-2.7.6/bin/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.73.2
    wanted: 0.73.2
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: Not found
  newArchEnabled: Not found
iOS:
  hermesEnabled: Not found
  newArchEnabled: false

info React Native v0.74.1 is now available (your project is running on v0.73.2).
info Changelog: https://github.com/facebook/react-native/releases/tag/v0.74.1
info Diff: https://react-native-community.github.io/upgrade-helper/?from=0.74.1
info For more info, check out "https://reactnative.dev/docs/upgrading?os=macos".

Stacktrace or Logs
Unable to make progress running work. There are items queued for execution but none of them can be started

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
> Get more help at https://help.gradle.org.
Deprecated Gradle features were used in this build, making it incompatible with Gradle 9.0.
You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.
For more on this, please refer to https://docs.gradle.org/8.3/userguide/command_line_interface.html#sec:command_line_warnings in the Gradle documentation.

Reproducer
No Git url - My company project
Screenshots and Videos
No response

I replicated this issue by creating the react-native project.
Please find the below repo.
https://github.com/gopikrishnacse55/UIExample.git

Do we have solution for this ?

Same issue, it's happened on my side.
I tried on both 0.74.1 and 0.73.8, but the same issue happened.
Here is my env:
System:
  OS: Windows 11 10.0.22000
Binaries:
  Node:
    version: 18.3.0
    path: C:\Program Files\nodejs\node.EXE
  Yarn:
    version: 1.22.21
    path: C:\Program Files\nodejs\yarn.CMD
  npm:
    version: 8.11.0
    path: C:\Program Files\nodejs\npm.CMD
  Watchman: Not Found
SDKs:
  Android SDK: Not Found
  Windows SDK: Not Found
IDEs:
  Android Studio: AI-233.14808.21.2331.11709847
  Visual Studio: Not Found
Languages:
  Java: 11.0.23
  Ruby: Not Found
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.73.8
    wanted: 0.73.8
  react-native-windows: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: Not found
  newArchEnabled: Not found

Related to https://github.com/gradle/gradle/issues/21325
Are you building from Android Studio or from the CLI?
Can you try both and report from which one can you build?

Also experience this. sometimes it show, sometimes its not. I am not sure why this occurs. I clean and rebuild and invalidate cache still the same.

I am facing this issue When I trying to rebuild the project from Android Studio.
I am getting the below error when trying to run the app on the device: react-native run-android.
build failed in gradle port:8081.

Hello! 👋
I would also like to report that this issue happened to me today too.
Today I ran this command: npx react-native init abc
I did not install any other dependencies, just the pure React Native project.
I am using a windows machine.
I use Android Studio to build the build the project.
Here are the steps to reproduce the problem (on a windows machine):

Run this command: npx react-native init abc
Open the project on Android Studio
Rebuid the project with Android Studio:
Menu => Build => Clean Project
Menu => Build => Rebuild Project

And this issue appeared to me.
Thanks a lot! 🙏

npx react-native init abc
npx react-native init abcd --version="0.73"
npx react-native init abcde --version="0.72"

All these commands will create the same bug when follow the same 3 steps in the my previous comment.

Added this to my root build.gradle file and it fixed my issue
gradle.startParameter.excludedTaskNames.addAll(
        gradle.startParameter.taskNames.findAll { it.contains("testClasses") }
)

Added this to my root build.gradle file and it fixed my issue
gradle.startParameter.excludedTaskNames.addAll(
        gradle.startParameter.taskNames.findAll { it.contains("testClasses") }
)

It worked perfectly. Thanks a lot @markneh!
I've tested it across several React Native versions.

Just to clarify. Where Should this code be. It should be in root build gradle file like this:
gradle.startParameter.excludedTaskNames.addAll(
        gradle.startParameter.taskNames.findAll { it.contains("testClasses") }
)

buildscript {
    // code
}

I added the below code in build.gradle and rebuilt the project in Android Studio. The issue is fixed.
gradle.startParameter.excludedTaskNames.addAll(
gradle.startParameter.taskNames.findAll { it.contains("testClasses") }
)
Trying to run the command: react-native run-android on Android device.Getting the below error.
info Installing the app...
info 💡 Tip: Make sure that you have set up your development environment correctly, by running npx react-native doctor. To read more about doctor command visit: https://github.com/react-native-community/cli/blob/main/packages/cli-doctor/README.md#doctor
error Failed to install the app. Command failed with Unknown system error -8: ./gradlew app:installDebug -PreactNativeDevServerPort=8081
spawn Unknown system error -8.
info Run CLI with --verbose flag for more details.

@gkasireddy202
Try the following:
In android studio,

Open the project folder
Menu => File => Open => (Select the "/android" folder in your React native project)
(Assuming that you are using a React Native CLI build and not an Expo build)

Clean and rebuild
Menu => Build => Clean Project
Menu => Build =>Rebuild Project

Install app
Now, press that green run button to install the application

Run application
npm start
open the emulator
Open the application

Summary: on windows device, do not run npx react-native run-android directly, it may not work sometimes.

@OmarThinks - Thanks for your reply.
I followed your steps and app is installed on my device.
Getting the below error.
Questions Tags Users Companies LABS Jobs NEW Discussions COLLECTIVES Communities for your favorite technologies. Explore all Collectives TEAMS Ask questions, find answers and collaborate at work with Stack Overflow for Teams. Unable to load script.Make sure you are either running a Metro server or that your bundle 'index.android.bundle' is packaged correctly for release

@gkasireddy202
https://stackoverflow.com/questions/55441230/unable-to-load-script-make-sure-you-are-either-running-a-metro-server-or-that-yo
Maybe you are not running npm start in the background.
But the application has been installed successfully, that's progress. 😁🎉💪💯

@OmarThinks - App is running on my device.

npm start
I used to run button in Android Studio.
The app is successfully running on the device along with the metro bundler server.

Congrats, bro! 🎉🎂
You made progress. 🫡 💪
Happy, coding! 💻😁

@OmarThinks  - Thanks for your help.

@gkasireddy202
You're welcome. 😁
I need to clarify.
The npm start runs the metro in the background, so that you can run the React Native app.
The green arrow does the following:

Install the app on the emulator if it wasn't installed.
After it makes sure that the app has been installed, it opens the application in the emulator (but not metro)

So, you need the app to be open in the emulator, while npm start is running in background.

Hello, @cortinico ! 👋
I would like to say that the issue with React Native still hasn't been solved.
Issue description:
On a windows machine, for a new pure isolated (the template from npx react-natine init abc) latest version project of React Native.
On a windows machine, using Android Studio, when we run:
Menu => Build => Clean Project
Menu => Build =>Rebuild Project
This issue appears:
Unable to make progress running work. There are items queued for execution but none of them can be started

The issue will not be solved, unless we add this code to the root build gradle file:
gradle.startParameter.excludedTaskNames.addAll(
        gradle.startParameter.taskNames.findAll { it.contains("testClasses") }
)

buildscript {
    // code
}

Thanks a lot! 🙏

I would like to say that we would really appreciate it if someone from the React Native teams solved this issue, we would be so grateful.

Thanks for flagging it. Unfortunately, I don't have capacity at the moment to work on this.
I'll be happy to receive and review a PR for a fix for this.
Needless to say that  gradle.startParameter.excludedTaskNames.addAll( is a workaround and we should fix the root cause

이렇게 추가하고 성공했어요!
buildscript {
...
gradle.startParameter.excludedTaskNames.addAll(
            gradle.startParameter.taskNames.findAll { it.contains("testClasses") }
    )
}

Added this to my root build.gradle file and it fixed my issue
gradle.startParameter.excludedTaskNames.addAll(
        gradle.startParameter.taskNames.findAll { it.contains("testClasses") }
)

This Saved me

Hello, @cortinico ! 👋
I would like to say that we would really appreciate it if someone from the React Native teams solved this issue, we would be so grateful.
Issue description: On a windows machine, for a new pure isolated (the template from npx react-natine init abc, without installing any other dependencies) latest version project of React Native.
On a windows machine, using Android Studio, when we run: Menu => Build => Clean Project Menu => Build =>Rebuild Project
This issue appears:
Unable to make progress running work. There are items queued for execution but none of them can be started

The issue will not be solved, unless we add this code to the root build gradle file:
gradle.startParameter.excludedTaskNames.addAll(
        gradle.startParameter.taskNames.findAll { it.contains("testClasses") }
)

buildscript {
    // code
}

Thanks a lot for you, and for the React Native team for your hard work and consistent effort! 🙏

This save me

@rafiiAP
You're welcome, bro! 😁


make(Binary)Wrapper: record created wrappers

Description of changes
Write a machine-readable list of wrappers (each line is a $wrapper:$original pair) to ${!outputBin}/nix-support/wrappers.
A wrapper -> original mapping makes it easier to see through wrappers, especially binary wrappers, to the original executable.
One case where this helps is when you need to perform analysis on the actual executables and associate that analysis with exec wrappers.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Before I started trying to figure out how to support multiple outputs here, the first draft of this wrote these into ${!outputBin}.
For a bit, I had myself convinced that automatic and manual moveToOutput behavior would break some of the linkages here (leaving the record in the wrong output and pointing at a stale path) unless we could either figure out how to write the record into the correct output from the start, or perhaps relocate them during fixup.
Now I'm starting to think it may be okay not to sweat this much, since wrappers should also get broken by moveToOutput (probably leading packages that need both to create wrappers in postFixup or later). To that end, I've force-pushed to update this to try to just discover the store path prefix that the wrapper is being written to, and record the wrapper within that output.
(I think the risk of multi-output handling breaking this linkage is technically still here, but any package with this problem is likely already broken and I suspect that fixing the wrapper will generally fix that problem?)


python310Packages.pyinsteon: 1.1.1 -> 1.1.2

Description of changes
https://github.com/pyinsteon/pyinsteon/releases/tag/1.1.2

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 180587 run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.insteon
python310Packages.pyinsteon
python39Packages.pyinsteon


[YahtzeeDevDiary]: New Bridge

More sites that make a series of videos and dump them on a single YouTube account, without a playlist.
This is a 1 year project, it's about half way there already, so it's actually an ephemeral feed. Worth to share if anyone cares.

Looks like everything is in order. Merging...


⚠ RPC server - Figment Solana has degraded performance

In df38f21, RPC server - Figment Solana ($FIGMENT_SOLANA_RPC_URL) experienced degraded performance:

HTTP code: 429
Response time: 323 ms

Resolved: RPC server - Figment Solana performance has improved in ec0a657.


Feature Request

i think you must put cmd.exe by deafault at the start.

What does this have to do with the new Terminal? You're talking about the old command prompt.

@tisukk u can customize your app to set whichever terminal u want to be the default. It is already there.

You can change your default profile to cmd by changing the "defaultProfile" field in your profiles.json to the GUID assigned to your cmd profile


v2.5.0

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_55dk3aubvh/clone/recipe/meta.yaml:15: pip_install_args: pip install should be run with --no-deps and --no-build-isolation.
Errors were found


Compile python requirements

Automated changes by create-pull-request GitHub action

Breaking change from numpy 2.0.0!
https://github.com/NYCPlanning/data-engineering/actions/runs/9627004556/job/26553697935#step:6:870
It is based on string representation of an error message that we generate. Seems that using a dict within a format string now specifies type

@fvankrieken We should probably just cast to the python int in the failing case. Happy to make that change, if you're not already on it

would love to upgrade as well

@fvankrieken We should probably just cast to the python int in the failing case. Happy to make that change, if you're not already on it.
All for upgrading, otherwise.

All yours 🙂

Fix here (created a new branch/PR so it wouldn't be overwritten)


Update wording and formatting of Binary Artifacts issue

Proposal is to look like:
(No change to title)
Security Policy violation Binary Artifacts
This issue was automatically created by Allstar
Security policy violation
Security policy Binary Artifacts failed: binaries present in source code
Rule Description
Binary Artifacts are an increased security risk in your repository. Binary artifacts cannot be reviewed, allowing the introduction of possibly obsolete or maliciously subverted executables. For more information see Scorecards Documentation.
Remediation Steps
To remediate, remove the generated executable artifacts from the repository.
Artifacts found

dummy.dll
dummy.exe
dummy.jar
dummy.so

Additional Information
This policy is drawn from Security Scorecards, which is a tool that scores security best practices for a project. You may wish to run a Scorecards scan directly on this repository for more details.

Staging deploy of Allstar for testing, see https://github.com/ossf-tests/.allstar for config. (This is the custom footer)
This issue will auto resolve when the policy is in compliance. (This is current Allstar footer)
Issue created by Allstar. See https://github.com/ossf/allstar/ for more information. For questions specific to the repository, please contact the owner or maintainer.

Currently looks like: https://github.com/ossf-tests/scorecard-check-binary-artifacts-e2e-4-binaries/issues/4

@olivekl Can you take a look and share your thoughts? Thanks!

All minor stuff:
Security Policy Violation (capitalization for consistency with other headings)
For more information see the [Security Scorecards documentation] for Binary Artifacts. (full Scorecards name and the policy name)
'Artifacts Found(caps again)Binary Artifacts security policy has failed: binaries present in source code(change order for clarity)This issue was automatically created by Allstar.(period at end)which is a tool that scores a project's adherence to security best practices.` (reorder)

Updated:
This issue was automatically created by Allstar.
Security Policy Violation
Binary Artifacts security policy has failed: binaries present in source code
Rule Description
Binary Artifacts are an increased security risk in your repository. Binary artifacts cannot be reviewed, allowing the introduction of possibly obsolete or maliciously subverted executables. For more information see the Security Scorecards Documentation for Binary Artifacts.
Remediation Steps
To remediate, remove the generated executable artifacts from the repository.
Artifacts Found

dummy.dll
dummy.exe
dummy.jar
dummy.so

Additional Information
This policy is drawn from Security Scorecards, which is a tool that scores a project's adherence to security best practices. You may wish to run a Scorecards scan directly on this repository for more details.

Staging deploy of Allstar for testing, see https://github.com/ossf-tests/.allstar for config. (This is the custom footer)
This issue will auto resolve when the policy is in compliance. (This is current Allstar footer)
Issue created by Allstar. See https://github.com/ossf/allstar/ for more information. For questions specific to the repository, please contact the owner or maintainer.

Thanks, Jeff. I have one more thought now that I'm looking again with fresh eyes. What do you think of:
Project is out of compliance with Binary Artifacts policy: binaries present in source code (since a policy doesn't fail, and we changed "check" to "policy")

Sounds good, so now:
This issue was automatically created by Allstar.
Security Policy Violation
Project is out of compliance with Binary Artifacts policy: binaries present in source code
Rule Description
Binary Artifacts are an increased security risk in your repository. Binary artifacts cannot be reviewed, allowing the introduction of possibly obsolete or maliciously subverted executables. For more information see the Security Scorecards Documentation for Binary Artifacts.
Remediation Steps
To remediate, remove the generated executable artifacts from the repository.
Artifacts Found

dummy.dll
dummy.exe
dummy.jar
dummy.so

Additional Information
This policy is drawn from Security Scorecards, which is a tool that scores a project's adherence to security best practices. You may wish to run a Scorecards scan directly on this repository for more details.

Staging deploy of Allstar for testing, see https://github.com/ossf-tests/.allstar for config. (This is the custom footer)
This issue will auto resolve when the policy is in compliance. (This is current Allstar footer)
Issue created by Allstar. See https://github.com/ossf/allstar/ for more information. For questions specific to the repository, please contact the owner or maintainer.

LGTM!


No repositories found after extension update

Extension version: 0.58.0
VSCode Version: 1.75.0
OS: Linux

Updated the extension and lost all my repos (Github enterprise)

Something I'm missing ? Was working fine before I updated the extension

I solved it by creating a new workspace and adding all my repos


Add ICMP ping exporter

Summary
Adds an ICMP ping exporter that opens an unpriviledged ICMP socket for each configured target. ICMP session ids are handled by the kernel.
Service discovery is handled by periodically reloading DNS. By setting a headless service as a target, this allows kubernetes agnostic service discovery.
The ping codebase is split into two parts:

Pingers that manage a single connection to a target and consists of a sending go routine and a receiving go routine that communicate through a sync map.

A manager that resolves DNS entries and starts/stops pingers periodically.

Checklist

[x] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog
[x] Update tests.

The code is not as well tested as I'd like, especially for error cases, but I'd like to get this running for now and might be able to come back to it later to improve test coverage


Add Philips Zhirui Downlight support

cp. https://github.com/syssi/philipslight/issues/18
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io/pull/7498

LGTM :+1:


Feature/adding csharp support to textsplitter

Description: Adding C# language support for RecursiveCharacterTextSplitter
Issue:   N/A
Dependencies: N/A

awesome, thanks @Ather23!

if you have time for a follow up pr, would be great to mention this in docs/snippets/modules/data_connection/document_transformers/text_splitters/code_splitter.mdx!


Issue with distance metric computation

Dimtra said that she often gets assertion errors when computing the distance metric due to nonpositive values.

I looked into this, and one way this might come about is if there is an underflow in converting between log and linear scales in the hyperparameters. That makes the diagonal entries on the distance metric NaNs, which tests False for positivity. I will fix this to throw a floating point error if NaNs are detected in the distance metric so that those iterations are correctly skipped. Hopefully this is the source of this issue!


ImportError: cannot import name 'ProvisioningSession' from 'azure.iot.device'

3.0.0b2 seems to be missing classes.
Context

OS and version used: Windows 11
Python version: 3.10.11
pip version: 23.2.1
list of installed packages: azure-iot-device=3.0.0b2
cloned repo:  ( If you are using a cloned sdk repository, in a command prompt: git describe )

Description of the issue
Looks like ProvisioningSession is missing from the pip module.
See dir listing below
Directory of C:\Python10\Lib\site-packages\azure\iot\device
25-08-2023  04.32 PM              .
25-08-2023  04.32 PM              ..
25-08-2023  04.32 PM             6,937 config.py
25-08-2023  04.32 PM             3,904 connection_string.py
25-08-2023  04.32 PM               944 constant.py
25-08-2023  04.32 PM             1,458 custom_typing.py
25-08-2023  04.32 PM             6,659 edge_hsm.py
25-08-2023  04.32 PM             1,948 http_path_iothub.py
25-08-2023  04.32 PM            12,573 iothub_http_client.py
25-08-2023  04.32 PM            31,367 iothub_mqtt_client.py
25-08-2023  04.32 PM            20,351 iothub_session.py
25-08-2023  04.32 PM               818 iot_exceptions.py
25-08-2023  04.32 PM             8,949 models.py
25-08-2023  04.32 PM            41,408 mqtt_client.py
25-08-2023  04.32 PM            10,800 mqtt_topic_iothub.py
25-08-2023  04.32 PM             4,560 mqtt_topic_provisioning.py
25-08-2023  04.32 PM             2,141 request_response.py
25-08-2023  04.32 PM            11,271 sastoken.py
25-08-2023  04.32 PM             2,746 signing_mechanism.py
25-08-2023  04.32 PM             1,403 user_agent.py
25-08-2023  04.32 PM               557 init.py
25-08-2023  04.32 PM     pycache
19 File(s)        170,794 bytes
3 Dir(s)  287,644,717,056 bytes free
C:\Python10\Lib\site-packages\azure\iot\device>
Code sample exhibiting the issue
from azure.iot.device import (
ProvisioningSession,
MQTTConnectionDroppedError,
MQTTConnectionFailedError
)
Console log of the issue
File "G:\Other computers\E8\Venture\Halo IT\Customers\MiiCare\Development\code\provisioning_symmetric_key.py", line 3, in
from azure.iot.device import (
ImportError: cannot import name 'ProvisioningSession' from 'azure.iot.device' (C:\Python10\lib\site-packages\azure\iot\device_init_.py)

We apologize for the confusion regarding communication around version 3.0.0b2. The status of this release is in question, and we have since reverted the main repository back to version 2.x.
My apologies for the inconvenience.


Remove the testing functionality from the status bar

It's a bit redundant with the Test Explorer. It also adds clutter to an already crowded status bar. And it's simply more code to support for things that can be done via the Test Explorer or the Command Palette.

Will impact https://github.com/microsoft/vscode-python/issues/664.

This is fixed in our new, upcoming test provider.


Please update to league/oauth2-client version 2

Version 2 is available now for https://github.com/thephpleague/oauth2-client/releases/tag/2.0.0

+infinity

Done. :)


Extensible Certificate Issuing Controller

/cc @munnerz
This controller is used to implement the issuing controller, as described in the extensible certificates controller proposal here #2753. This controller is responsible for writing the resulting certificate, ca, and private key to the target certificate resource, in the correct fromat.
Adds new extensible issuing certificate

@munnerz Should be good to have another look :)
/assign @munnerz

I'm going to be following up with additional PRs around this (+ additional controllers) 😄
/lgtm


(PDB-1179) Remove PE references from PDB packaging

This commit removes the PE references from the pdb ezbake packaging
configs as we are going to build the pe package of puppetdb out of the
pe-puppet-extensions/puppet-sync repo.

@kbarber @rbrw will these paths need updating for AOI agent /cc @haus ?

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/863/

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/864/

@pljenkinsro retest this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/865/

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/870/

@pljenkinsro retest this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/871/

+1 from me and @haus, will merge when we're green again


Implement initial Phoenix Channels module

Current coverage is 84.30% (diff: 82.02%)

Merging #15 into master will decrease coverage by 2.64%

@@             master        #15   diff @@
==========================================
  Files             5         21    +16
  Lines            69        274   +205
  Methods          65        266   +201
  Messages          0          0
  Branches          3          8     +5
==========================================
+ Hits             60        231   +171
- Misses            9         43    +34
  Partials          0          0

Powered by Codecov. Last update 776a467...7319bcb


Add icons to the database

Embed the icons are base64 in the returned JSON.
https://issues.jboss.org/browse/IPAAS-295

Embedding the base64 version image in the JSON just bloats the response IMO and affects stuff like caching, which we should be looking to utilise for static assets as much as possible. I'd prefer to serve images from a separate endpoint and the JSON instead contain the link to the image.
Thoughts @gashcrumb, @kahboom?

I am happy for this in two phases though, embedded in JSON for now and introduce a proper image service later, if doing the image service will take too long for now.

I think it'd be just as easy to host the image files in the console and just use CSS classes from the json response.  Definitely inlining images is error-prone, plus there's a size limit that you have to worry about, think it's 4k but it might be different cross-browser.

We just discussed on IRC with @kahboom, I'm happy to do whatever. Whatever is easier..

So, there are several things we can do:

Return PNG as JSON (must be encoded in Base64). Probably the most RESTful way of doing things, but as @jimmidyson pointed out, could get a bit bloaty. In our case, it's a small set of images that are maybe 100x100 pixels each and re-used frequently, but if we support uploading larger SVGs or PNGs that then need to be scaled, stored, etc. we should absolutely look into a dedicated service for this alone or it will become bloated. At that point, the API would just return the absolute path to the image within the connection or integration object and we'd reference that in the UI (see 4 below).
Send PNG as multipart/form data. Not as easy to work with, but follows the HTTP spec and mimetypes the way they were intended. This is probably the best if we will definitely eventually support uploading SVGs and PNGs.
Keep in the UI per @gashcrumb 's suggestion, and return a class name from the API (or use a unique identifier to reference them, like the name??). I'd normally gag and say 'no way', but again, these are a small set of icons and we're thinking short-term, specific solutions, so it's not too bad and probably the easiest to implement short term.
Similar to the first option, except without a service, we can just have a public endpoint for images that whitelists the UI hostname, and the API would return the integration or connection as normal with an icon property as it does now, except this would be an absolute path to the public endpoint that loads the image directly. This requires more work in the API and overworks it for larger files (again, we'd need a dedicated service), but works just as well for now.

cc @chirino

For now lets do the fastest thing possible.. so I'd got with @gashcrumb's suggestion and take this on as technical debt.

Simples.

UPDATE: We decided to go with option 3, with the exception of the class names. Was just discussing with @gashcrumb, we can just use the name or whatever property from the connector, it should map 1:1 for all common services. For now, while we still don't have users inputting the name of the images, etc.
My point is, we won't need to use the label property at all.. we can reference the connection's connector or integration's connection connector (if it's accessible). whatever property defines the name of the service (e.g. twitter). it might need sanitization in the UI but would be even faster than labels in the HTML we'd do something like /icons/{{ connection.connector.name }}.connection.png and similar for integrations the .connection.png would be to distinguish the image types, since there is apparently a different one for the Integration List and one for the Connection List, etc.
In other words, @KurtStam you don't have to do anything, for now, I think.. as we'll ignore the label property for now until we begin to accept user input where the name may vary.
Please keep this issue open for now until works out okay. Hoping to get this in today; need to re-scale the images to size so that the API and UI don't have to do this on the fly. Considering providing black&white versions for disabled states (need to consult UXD first) as well.

cc @dongniwang @sjcox-rh  - Please read above and consult about black&white icons for disabled states, or if we should do this on demand in the UI (preferably not). Also, I'll first be testing with 40x40 for the integration list, smaller for connection list. Let me know what you think.


Consider adding WaitForCompletion convenience methods back

See  https://github.com/Azure/azure-sdk-for-net/pull/8193

Not sure if this is even actionable


Prevent updates of primary key properties

Closes #353

We should still allow primary key changes during migration - is there some other mechanism that Kotlin is employing to support that?

We should still allow primary key changes during migration - is there some other mechanism that Kotlin is employing to support that?

I am not sure that we support that in realm-java, so will have to discuss with the team what we actually want.

I suspect that with automatic migrations where you create a new primary key column, all objects have the default value as their primary key...but that is not user-initiated behavior. Not sure if there is another use case? 🤔

Updated it to allow updates on primary key for DynamicRealm. I guess this is not exactly what we want, but as we currently only expose DynamicRealm as part of migrations, it works out. There are different schemes for enforcing this and we should consider if it should be enforced in core directly before changing the compiler plugin, etc.