HomeBase3 Not Arming/Disarming

Hello, I have one HomeBase2 and one HomeBase3, and a lot of automations that are arming_home/ arming_away when I reach/leave home.
The problem is that the automations are working pretty well with HomeBase2 but with HomeBase3 they are not arming home/away. The HomeBase3 is added to the automation at the same way as the HomeBase2. When the automation run nothing happen on HomeBase3. Why it is happening? Is the HomeBase3 still not compatible?
thanks a lot

Having the same problem, in autoamtions, und, when you try to arm it over a manually over HA:

Looks like Bropat updated the driver / lib to 2.2.0 (2022-11-01). Are his Lib's in this package? If so, could you update them? Kindly :)

I am on bumping the version of integration, please wait for it.

@fuatakgun are you adding support for the homebase 3 and eufycam 3?

Yes would do

Nice! Any time estimate on when? :)

Probably latest this weekend. It is taking time because there are some fundamental changes i need to introduce to go with latest schema version

Hye @fuatakgun, how are you getting on with the above? Let me know if you need a Homebase 3 or Eufycam3 shared to test with.

please test with alpha release v5.x.x


Fresh install errors with sh: patch-package: command not found

Describe the bug
Starting a new project with latest<EMAIL_ADDRESS>is resulting in all sorts of errors, using Node version 20.11.1
npx ignite-cli@latest new PizzaApp --yes

                                               .;;,.    .clll.
                                              .clll,    ,llll.           ...
                                               .,;,.    ;lllc'.,.   .';cllllc,
     .',,.                           ...      .'',,. .:lllllllll. .;lllc:;clll,
    .llll,                    .,...;cllll:.   ,llll, .llllllc,,'  clll:'.,cllc.
    ,llll'       ......;::.  'llcclllllllll'  ,llll,   .llll.    ,lllllllll:'
    ,llll.    .;clllllclll.  'llllc,..,llll,  :llll.   'llll.    ,lllc,''..
    ;llll.   :llllc:cllllc.  ,llll.   .llll, .lllll.   ,lllc...  ,lll:...';:.
   .llllc.  ;lll;.   :lll,   ;lll;    'llll, .llllc.   ,llllcll' .:lllllllll,
   .llll,  'lll:    .clll,   :lll,    ,llll. 'llll,    'lllllcc'   ';::::;'.
   .llll,  ,lll:    .llll'  .llll,    ;llll. ,llll,     .....             .',..
   ,llll.  'llll:,,:cllll.  .lllc.    .''..  ... ....',;         ..',,:::;'..
   ,llll.   'clllllllllll.     .....        ..,::cllllc.   .',:clllc;,'.
   :llll.     ',;,'.;lllc...,;:clll;  ..',:cllllllllll:,;clllllc;,'.
   llll:        ...'cllllllllllllll::cllllllc;,',cllllllllc;'..
   llll,   ..,clllllllll::,,.,lllllllc:,'...    :lllllc,..
   c:;,.  .cllllc:;:llll.    ,ll:;,..          ;ll:,..
          :lll,.  .clll,     ..               .,..
          :lll:,,cllll,
          .cllllllllc'
           .'cllll:.

    █ Creating PizzaApp using Ignite 9.6.3
    █ Powered by  ∞ Infinite Red  (https://infinite.red)
    █ Package Manager: npm
    █ Bundle identifier: com.pizzaapp
    █ Path: /Users/user/Developer/samplereactnative/PizzaApp
    ────────────────────────────────────────────────────

    🖨  3D-printing a new React Native app
    🎨 Getting those last few details perfect
    ❌ Installing npm dependencies (wow these are heavy)

The following error occurred:

    npm WARN deprecated<EMAIL_ADDRESS>This functionality has been moved to @npmcli/fs
npm WARN deprecated<EMAIL_ADDRESS>This proposal has been merged to the ECMAScript standard and thus this plugin is no longer maintained. Please use @babel/plugin-transform-optional-catch-binding instead.
npm WARN deprecated<EMAIL_ADDRESS>This proposal has been merged to the ECMAScript standard and thus this plugin is no longer maintained. Please use @babel/plugin-transform-numeric-separator instead.
npm WARN deprecated<EMAIL_ADDRESS>This proposal has been merged to the ECMAScript standard and thus this plugin is no longer maintained. Please use @babel/plugin-transform-class-properties instead.
npm WARN deprecated<EMAIL_ADDRESS>This proposal has been merged to the ECMAScript standard and thus this plugin is no longer maintained. Please use @babel/plugin-transform-nullish-coalescing-operator instead.
npm WARN deprecated<EMAIL_ADDRESS>This proposal has been merged to the ECMAScript standard and thus this plugin is no longer maintained. Please use @babel/plugin-transform-logical-assignment-operators instead.
npm WARN deprecated<EMAIL_ADDRESS>This proposal has been merged to the ECMAScript standard and thus this plugin is no longer maintained. Please use @babel/plugin-transform-async-generator-functions instead.
npm WARN deprecated<EMAIL_ADDRESS>This proposal has been merged to the ECMAScript standard and thus this plugin is no longer maintained. Please use @babel/plugin-transform-optional-chaining instead.
npm WARN deprecated<EMAIL_ADDRESS>This proposal has been merged to the ECMAScript standard and thus this plugin is no longer maintained. Please use @babel/plugin-transform-object-rest-spread instead.
sh: patch-package: command not found
npm ERR! code 127
npm ERR! path /Users/user/Developer/samplereactnative/PizzaApp
npm ERR! command failed
npm ERR! command sh -c patch-package

npm ERR! A complete log of this run can be found in: /Users/user/.npm/_logs/2024-05-07T01_29_04_295Z-debug-0.log

    Consider opening an issue with the following information at:
    https://github.com/infinitered/ignite/issues/new?template=bug_report.yml&labels=bug

    ⠸  Gathering system and project details   Unable to gather system and project details.

Running npm run android in the new project returns this error:
npm run android

><EMAIL_ADDRESS>android
> npx expo start --android

Starting project at /Users/user/Developer/samplereactnative/PizzaApp
Error: Error reading Expo config at /Users/user/Developer/samplereactnative/PizzaApp/app.config.ts:

Cannot find module 'ts-node/register'
Require stack:
- /Users/user/Developer/samplereactnative/PizzaApp/app.config.ts
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>- /Users/user/Developer/samplereactnative/PizzaApp/node_modules/@expo/cli/build/bin/cli
- /Users/user/Developer/samplereactnative/PizzaApp/node_modules/expo/bin/cli
Error: Error reading Expo config at /Users/user/Developer/samplereactnative/PizzaApp/app.config.ts:

Cannot find module 'ts-node/register'
Require stack:
- /Users/user/Developer/samplereactnative/PizzaApp/app.config.ts
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>- /Users/user/Developer/samplereactnative/PizzaApp/node_modules/@expo/cli/build/bin/cli
- /Users/user/Developer/samplereactnative/PizzaApp/node_modules/expo/bin/cli
    at Module._resolveFilename (node:internal/modules/cjs/loader:1144:15)
    at Module._load (node:internal/modules/cjs/loader:985:27)
    at Module.require (node:internal/modules/cjs/loader:1235:19)
    at require (node:internal/modules/helpers:176:18)
    at Object.<anonymous> (/Users/user/Developer/samplereactnative/PizzaApp/app.config.ts:7:1)
    at Module._compile (node:internal/modules/cjs/loader:1376:14)
    at requireFromString (/Users/user/Developer/samplereactnative/PizzaApp/node_modules/require-from-string/index.js:28:4)
    at evalConfig (/Users/user/Developer/samplereactnative/PizzaApp/node_modules/@expo/config/build/evalConfig.js:68:47)
    at readConfigFile (/Users/user/Developer/samplereactnative/PizzaApp/node_modules/@expo/config/build/getConfig.js:45:41)
    at getDynamicConfig (/Users/user/Developer/samplereactnative/PizzaApp/node_modules/@expo/config/build/getConfig.js:54:18)
    ```

### Ignite version

9.6.3

### Additional info

System
  platform           darwin
  arch               arm64
  cpu                8 cores             Apple M2
  directory          samplereactnative   /Users//Developer//samplereactnative

JavaScript (and globally-installed packages)
  node               20.11.1                                   /Users//.nvm/versions/node/v20.11.1/bin/node
  npm                10.2.4                                    /Users//.nvm/versions/node/v20.11.1/bin/npm
    corepack         0.23.0
    npm              10.2.4
  yarn               -                                         not installed
  pnpm               9.1.0                                     /opt/homebrew/bin/pnpm
                     <no pnpm global package info available>
  bun                -                                         not installed
  expo               -                                         not installed

Ignite
  ignite-cli         9.6.3        /Users//.npm/_npx/e31027f3785124a8/node_modules/.bin/ignite
  ignite src         build        /Users//.npm/_npx/e31027f3785124a8/node_modules/ignite-cli/build

Android
  java               17.0.11      /usr/bin/java
  android home       -            /Users//Library/Android/sdk
/Users/user/.npm/_npx/e31027f3785124a8/node_modules/gluegun/build/index.js:15
    throw up;
    ^

Error: Command failed: xcodebuild -version
xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance

    at ChildProcess.exithandler (node:child_process:422:12)
    at ChildProcess.emit (node:events:518:28)
    at maybeClose (node:internal/child_process:1105:16)
    at ChildProcess._handle.onexit (node:internal/child_process:305:5) {
  code: 1,
  killed: false,
  signal: null,
  cmd: 'xcodebuild -version',
  stdout: '',
  stderr: "xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance\n"
}

Node.js v20.11.1

(Since I'm not using iOS/Xcode for this RN project, I assume the xcodebuild error can be ignored)

@mrcelo Hello, I have also encountered the same problem.


Prod and dev files are out of date

After the merge of #78 the source file is updated with freezing of dates but no updated versions of the prod and dev builds have been generated and committed. So those that doesn't use the source file are missing the functionality.
@rtfeldman should I include the generated files in future PRs? It's very easy to forget about that step before doing a release.

Oh weird. Thought I did that. Probably a good idea, yeah, but ultimately something I ought to do.

Published as 5.0.1.

Thanks @crudh!

@rtfeldman :+1: nice, 4 minutes from report to release :smile:


Authentication tokens lifecycle

Hello,
I've hit the problem of DRCA not refreshing tokens for repositories already interacted with. I find this as major problem, since from how DRCA is structured it's clear that DRCA is best instantiated once for the lifecycle of the application (taking care of connection pooling and credentials management) but with no token refreshing this is pretty much impractical now.
I have the use case where I'm downloading images for various forms of scanning that my backend is doing. To improve performance and scalability I'm checking if an image has already been scanned, so I can skip scanning it again. However, if the user supplies me with image:tag I have to resolve that to image@digest (by making a request via DRCA to the registry) and if I have indeed scanned that image this would mean I already acquired a token for that repository (scope). So, when I try to resolve a tag to digest a second time and this happens several minutes later, the token will be expired and the request would fail with 401 Unauthorized.
The same happens even with anonymous access to DockerHub because DockerHub mandates token-based access for anonymous users.
For the time being, I've made brutal hack to workaround the problem: monkey-patched DRCA with _get_token=_get_auth_token which forces getting new token on every request. Apparently, this is very bad for performance (and increases the chances of throttling on the registry side), so it'd be great to have token refresh logic.

Hey @crashvb, I'm yet to implement OAuth2 auth for the application I'm doing, but I intend to use https://docs.authlib.org
However, it dropped its support for aiohttp (in favor of httpx) that DRCA is using. :/


DBZ-736 DBZ-733 Remove MySQL Legacy Parser

https://issues.jboss.org/browse/DBZ-736 - Remove Legacy Parser
https://issues.jboss.org/browse/DBZ-733 - Consolidate Parser Tests
I've bundled these two issues together in a single PR since both have a bit of overlap.  Feel free to let me know if there is anything else I missed or need to change.

@Naros Hi, could you please remove it from Travis CI config file too?

@Naros Thanks a lot, generally I think it is doe, just a small proposal ddl/mysql-test-statements-fixed.ddl should probably be renamed to ddl/mysql-test-statements.ddl.


Mechanically reformat the Misk codebase.

This uses IntelliJ's formatter and Square's kotlin style rules.
The biggest difference with what is currently checked in is that
method parameters and constructor parameters are indented +2 instead
of their current +4.

See also: https://github.com/square/java-code-styles/pull/54


Only enabled on few filetypes

Problem
I know there is ibl.config.exclude.filetypes. But I want to disable all filetypes and enable on few filetypes like python.
To achieve this, I have to provide a lot of filetypes in ibl.config.exclude.filetypes. I don't think this makes sense.
Expected behavior
Provide the setting ibl.config.include.filetypes. When setup this, all filetypes will be disabled. Only the list in this setting will be enabled.

You can use :help ibl.hooks.cb.active() for this. Read up on hooks with :help ibl.hooks.

For future searchers: this is a way to only enable it for a specified filetype.
local hooks = require("ibl.hooks")
hooks.register(hooks.type.ACTIVE, function(bufnr)
    return vim.opt_local.filetype:get() == "yaml" -- for example
end)

A drawback is that this is inflexible, for instance :IBLEnable will fail on non-specified filetypes because this hook will return false.

If you want the setting less strict and be able to use IBLEnable in buffers that are excluded by default, I'd do it differently.
You can setup IBL disabled globally, and then use :help ibl.setup_buffer() from :help ftplugin to turn it on for specific filestypes.

Yep, good point. Can do the same with an autocommand.
require("ibl").setup {
    enabled = false,
    -- etc.
}

vim.api.nvim_create_autocmd("FileType", {
    desc = "Enable indent guides",
    pattern = "yaml", -- for example
    group = vim.api.nvim_create_augroup("ibl", {clear = true}),
    callback = function()
        require("ibl").setup_buffer(0, {enabled = true})
    end,
})

In my case, I want to enable it when the indent width is small, so
vim.api.nvim_create_autocmd("BufEnter", {
    desc = "Enable indent guides",
    group = vim.api.nvim_create_augroup("ibl", {clear = true}),
    callback = function()
        if vim.lsp.util.get_effective_tabstop(0) < 4 then
            require("ibl").setup_buffer(0, {enabled = true})
        end
    end,
})

works well. Personally I'm happy with the inflexible solution, although this might be more performant. (Might move setup inside the autocommand too.)


[Feature] enable change of x-axis of the slider in images

Is your feature request related to a problem? Please describe.
When using WandB with pytorch lightning, all logged metrics have an option to change the x axis to step, epoch ... etc. However, The logged images has by default step as the x-axis of the slider in the UI. This gives unnatural control over the slider of the images in the UI, in case images are not logged at every step. When specifying a particular invalid step in the slider ( a step where no image was logged), the shown image does not change and stays at the last step that was shown in the UI.
Toy Example:
In the following code, I am only logging images at every epoch, where each epoch consists of 2 steps. That means I am logging at even steps.

The logged image on the last step (18)

When trying to see the logged image at step 5, where there is no logged image. the shown image does not change and the UI keeps showing the image at step 18
Describe the solution you'd like
Solution 1 (preferred): gives the ability to change the x-axis of the slider in the image, so that images can be aligned with epochs, in case the value of epoch is given.
Solution 2: When the user puts an invalid step size (i.e a step where no image is logged) in the slider, the slider should go to the nearest valid step (i.e nearest step where there is a logged image) rather than staying at the current step.

Hey there, thanks for writing in! We already have a ticket filed for your request. I'll bump up its priority and let you know once  the option is available

Solution 1 (preferred): gives the ability to change the x-axis of the slider in the image, so that images can be aligned with epochs, in case the value of epoch is given.

Hey hey! I have a suggestion here. You can log the epoch (or any increasing value) as a metric. Since it will be monotonically increasing, in your plot settings on wandb dashboard you will be able to select this metric as the x-axis. This should remap the slider on your images and plots. Plus, Lightning logs the epoch by default, so you should be able to go the plot settings and select that directly.
cheers and happy logging

Thanks for the suggestion @awaelchli, I've tried change the x-axis to global_step which is logged by pytorch lightning, but it doesn't change the slider of the images. It only affects scalar metrics.. :(
The slider still depends on that step variable which is not aligned with the other metrics (which are correctly aligned with self.global_step from pytorch lightning).
(I'm using wandb 0.13.10 and pytorch lightning 1.8.6)

I'm also very interested in this feature. Not having a consistent step across runs (e.g., because a newer run logs more visualizations, making step misaligned) makes it hard to A/B test things by looking at logged images.

Hello, not to add noise, but I think this is a very useful feature as well. It makes image comparisons across runs significantly easier. At the moment, step sizes are not necessarily consistent between runs, leading to a very confusing time for image panels. Changing the global x axis does not affect the slider.


SETTINGS: add SESSION_EXPIRE_AT_BROWSER_CLOSE

from https://github.com/MetLifeLegalPlans/portunus/issues/107

superseded by https://github.com/MetLifeLegalPlans/zygoat/pull/116


Selecting additional search checkboxes causes entire window to reload

On the search tab, if you select a checkbox, and then select a second checkbox for the same widget, the entire window will reload instead of just updating the search.

I can't get this to happen anymore. Will close until it's reproduced.

This is happening regularly for me again. Start with a fresh OPUS. Select Instrument Name. Click on Cassini ISS. Wait. Click on Voyager ISS. The window will reload.


Updated Facebook typings and added example

I had to create a Facebook app for this example to work. If you want to send me either your fbid or username I will add you as an owner to Facebook application.

The first contribution! \o/ How do you celebrate something like that? 🎉

My Facebook profile can be accessed at https://www.facebook.com/janaagaard, so I guess that makes my username janaagaard.

Done 👍

I took the liberty to make a couple of minor syntax changes to the code. Is that... Okay? I believe in the value of using the same syntax throughout the code, but I would alsolove to build a community to help completing and keeping these type definitions up to date, and I guess these kind of changes could turn some developers aways. What do you think?

Yeah go ahead with any changes you want. Personally I find the duplicate import syntax a bit harder to read, but it works. It might be worth using prettier for formatting so that there's no need for discussions on styling with future contributors.

The main reason I use one import per line is that it is my experience that the conventional syntax is very prone to cause merge conflicts. With a single import per line and everybody sorting the imports alphabetically, these conflicts should be much less frequent.
I haven't thought about the readability of the imports, though. And now that I think about it, I think it's very rare that I look at the imports for a file. It would mostly be checking if there are a lot of them, because that would be a sign that the class has too many responsibilities.

That's true on both merge-conflicts and not really looking at the imports. The big issue with enforcement will be the lack of an existing TSLint rule for multi-import in that style. I suppose another minor issue would not being able to use automatic imports, but I don't think too many people use that feature yet.


running migrate --pretend executes query if affecting non-default DB

This seems to be a regression (per: https://github.com/laravel/framework/pull/4362).
I have a migration that like this:
function up() {
       Schema::create('permission_user', function (Blueprint $table) {
            $table->increments('id');
            $table->integer('user_id');
            $table->integer('permission_id');
            $table->timestamps();
        });

       Schema::create('permissions', function (Blueprint $table) {
            $table->increments('id');
            $table->string('name');
            $table->timestamps();
        });
}

I'm trying to create these tables in a different database (not the default connection database) called users.
I've tried to run php artisan migrate --pretend, and in each case I'm getting undesirable behavior. I've tried mixing up the migration a number of ways (see below):

Method
Tables Created
Pretend Output?
Migration Logged

inline: Schema::create('users.permissions')
Yes, in correct database (not supposed to create!)
No
No

method: Schema::connection('users')->create(...
Yes, in correct database (not supposed to create!)
No
No

update protected $connections = 'users'
Yes but in default DB, $connection appears to be ignored
Yes
No

don't specify database at all
No
Yes
No

As you can see, only the last case performs as expected. The problem with that is that the live migration would create the tables in the wrong database.
How can I make this migration "pretend" correctly, while still achieving the desired behavior on a live migration? If someone wants to point me in the right direction, I'm happy to look into it and potentially submit a PR to fix.
Thanks.

AFAIK The only way to make this work is to set the protected $connection = 'connection' and also specify the connection on the Schema instance Schema::connection('connection').
If the connection of the migration doesn't match the connection in your schema instance you'll experience this problem you mentioned.
This is because the pretend method is called on the migration connection, while in your Schema instance you're running the query on a different connection.

That worked! Thanks. Still seems kind of buggy. Only one of those should be required, not both at the same time, yeah?

Closing since the issue seems to be a rare edge case that can be solved with a workaround.


Consume the API Proxy in SAP Web IDE

Tutorial: https://developers.sap.com/tutorials/hcp-apim-webide-int.html
Write here what you need help with ...
This tutorial is not working in the CF runtime nor with BAS.
In the step where we create a destination to the dev portal (APIM) it says we need to use AppToAppSSO - however this option does not existing Cloud foundry - how do we proceed?
In the step where you run a template/generator in WebIDE I an trying with Business Application Studio. In BAS we cannot see the destinations as available systems to choose from. The only way I can see the destination in BAS is to add "WebIDEUsage: odata_abap" - but then I get Authentication issues.
Is it not possible to use APIM from BAS as of yet? If it is can you please point me to the documentation on how it is done?

This tutorial will be decommissioned and the team is working on provisioning the new ones.
I hope this helps!

This tutorial will be decommissioned and the team is working on provisioning the new ones.
I hope this helps!

Thankyou for the update. I would say it's best to pull this tutorial immediately if it doesn't work. Same goes for all other tutorials that don't work as described.

Thankyou for the update. I would say it's best to pull this tutorial immediately if it doesn't work. Same goes for all other tutorials that don't work as described.

Dear Raghu,
Can you please take care.
Regards
Shilpa
Sent from my iPhone
On 12-Jan-2021, at 7:48 PM, Jason Scott<EMAIL_ADDRESS>wrote:

Closed #34https://github.com/SAPDocuments/Issues/issues/34.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://github.com/SAPDocuments/Issues/issues/34#event-4196099521, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKQ3P5ZGESQYDXY2XQHUYCLSZRKXXANCNFSM4VW44VEA.

Dear Raghu,
Can you please take care.
Regards
Shilpa
Sent from my iPhone
On 12-Jan-2021, at 7:48 PM, Jason Scott<EMAIL_ADDRESS>wrote:

Closed #34https://github.com/SAPDocuments/Issues/issues/34.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://github.com/SAPDocuments/Issues/issues/34#event-4196099521, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKQ3P5ZGESQYDXY2XQHUYCLSZRKXXANCNFSM4VW44VEA.


Fix bug in error result text

This fixes a bug in how erroring checks present their errors.
Signed-off-by: Jose R. Gonzalez<EMAIL_ADDRESS>

Coverage remained the same at 60.038% when pulling 0546c262810dde7135fd0e3d81c78315d9fe49a9 on komish:err-log-line-fix into 18ced47666a8ba33dd3c2fecd85dac3bf5ec7844 on redhat-openshift-ecosystem:main.


Bilinear upsampling mapping missing

Pretty sure that there is a bug in the upsample conversion, where the "bilinear" node type doesn't map to "BILINEAR". I am not sure if the "linear": "BILINEAR" mapping is needed elsewhere, but this map should include "bilinear": "BILINEAR" for correct conversion.

@sakishinoda Upsample is deprecated in ONNX operator version 10
as per version-9, https://github.com/onnx/onnx/blob/master/docs/Changelog.md#Upsample-9
I have seen ONNX version 9 models with 'linear' mode.
Do you have any ONNX model example for the same?

Okay, this may be because I'm using pytorch version 0.4.1 to export to ONNX in the first place. Thank you


Extend S3 bucket policy to ensure encryption-in-transit

Whilst the current option policy ensures server-side encryption,
encryption of the transport mechanism isn't enforced.
This change extends the S3 bucket policy to enforce encryption in
transit, which is necessary to satisfy the s3-bucket-ssl-requests-only
AWS Config Rule[1]
Given the option to prevent unencrypted uploads is already present,
and this change fits in the spirit of that, I've not introduced a
separate flag for this behaviour.
[1] https://docs.aws.amazon.com/config/latest/developerguide/s3-bucket-ssl-requests-only.html

/codefresh run test

/rebuild-readme

/rebuild-readme

@maximmi looks like the existing tests are failing because the README hasn't changed.
There's no configurable change and I didn't think it was noteworthy enough to update the README, yet the tests enforce it. Do you have any suggestions (or fixes for the tests)?

/rebuild-readme

/rebuild-readme

/rebuild-readme

/rebuild-readme

/rebuild-readme

/rebuild-readme

/rebuild-readme

/rebuild-readme

/rebuild-readme

/rebuild-readme

/rebuild-readme

/rebuild-readme

/rebuild-readme

/codefresh run test

/codefresh run test

@maximmi I strongly recommend pulling Actions and/or README related changes to get the build passing out into another PR so master works, then I can open a new PR with just my changes rebased against the updated master.
Right now this PR is changing several things at once.

@bazbremner here is new release: https://github.com/cloudposse/terraform-aws-tfstate-backend/releases/tag/0.16.0

@maximmi Thanks!


Create Docker images for the latest software

This enables us to test against the latest compiler features and local
dependencies. I'm giving the asan build a little bit of double duty
here in that it will also catch issues specific to later compiler
versions. The potential confusion or extra iteration time from this
combination seems worth it for the compute/latency savings. The asan
build seems a good candidate for using the latest compiler because
sanitizers presumably catch more issues with later versions. We don't
necessarily need to make guarantees that our build works with our
oldest supported compiler version and sanitizers. It seems reasonable
that people running sanitizer builds could update to the latest
versions.
This change also restructures the way we use Docker contexts. We have
already accrued some duplication because of the difficulty factoring
Docker files. Instead, this moves the docker context to include
potentially any file in the repository, which allows reuse via scripts.
We can continue to reduce duplication in future changes.

I saw the logs of asan and tsan still show clang version = 9.0, is that expected?


Support Named types in pflags

TL;DR
Adding support for NamedTypes in pflags
Type

[ ] Bug Fix
[X] Feature
[ ] Plugin

Are all requirements met?

[X] Code completed
[X] Smoke tested
[X] Unit tests added
[X] Code documentation added
[X] Any pending items have an associated Issue

Tracking Issue
fixes https://github.com/flyteorg/flyte/issues/1839

@EngHabu successfully generated the flags but getting error in compile time
# github.com/flyteorg/flytectl/cmd/config/subcommand/sandbox
cmd/config/subcommand/sandbox/config_flags.go:56:18: cannot use &DefaultConfig.ImagePullPolicy (type *ImagePullPolicy) as type *int in argument to cmdFlags.IntVar
cmd/config/subcommand/sandbox/config_flags.go:56:111: cannot use DefaultConfig.ImagePullPolicy (type ImagePullPolicy) as type int in argument to cmdFlags.IntVar

@evalsocket mind trying the updated PR?

@EngHabu one more
go build -o bin/flytectl -ldflags="-s -w -X github.com/flyteorg/flytestdlib/version.Version=v0.3.20-2-gae1eb07 -X github.com/flyteorg/flytestdlib/version.Build=ae1eb07 -X github.com/flyteorg/flytestdlib/version.BuildTime=2021-11-19" main.go
# github.com/flyteorg/flytectl/cmd/config/subcommand/sandbox
cmd/config/subcommand/sandbox/config_flags.go:56:18: cannot take the address of int(DefaultConfig.ImagePullPolicy)

Currently it is generating this
cmdFlags.IntVar(&int(DefaultConfig.ImagePullPolicy), fmt.Sprintf("%v%v", prefix, "imagePullPolicy"), int(DefaultConfig.ImagePullPolicy), "Optional. Defines the image pull behavior.")

but it should be this
cmdFlags.IntVar((*int)(&DefaultConfig.ImagePullPolicy), fmt.Sprintf("%v%v", prefix, "imagePullPolicy"), int(DefaultConfig.ImagePullPolicy), "Optional. Defines the image pull behavior.")


feat: update golang-github-mattn-go-ieproxy to 0.0.9

Go package to detect the proxy settings on Windows platform (library)
Issue: https://github.com/deepin-community/sig-deepin-sysdev-team/issues/552
Log: update repo

/lgtm
/approve


Changed directory to preview

The readme.md pointed to the wrong directory.  Changed to point to preview directory.
PR information

[ ] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-node

Automation for azure-libraries-for-java

@mcardosos Ping!

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go


Idea: make conditional content line up with label content

Hey, I've had an idea based on playing around with conditional content on radio buttons.
With conditional content drop downs, the text doesn't align with the content in the label above, which looks a bit weird and makes it harder to read.
Before:

What we've been playing around with:

Looks nice, what part is conditional in this example? Just the continue button?

Ignore the button for now... that's also another new thing we've invented apparently!
But this particular idea is about bringing the text in the drop down inline with the text above, and then shifting the grey line across to sit aligned to the middle of the radio.

I like the idea of content lining up with the labels - it feels and looks neater.
I'm less sure about moving the border, but can see that if you don't, you get a big bit of white space.
In most cases conditional content is the child of a radio or checkbox - but we also use inset for child content of the details element. Would we also inset these?

Thanks for raising this @kateiw!
I'm trying to close some of the govuk-elements issues, what's the verdict on this one - would we like to move the left grey border when we reveal extra content - using radio buttons or checkboxes?
I'm happy to make that change if there's consensus.

It would be good to see this in other use cases.

Following the launch of the GOV.UK Design System, GOV.UK Elements will now only get major bug fixes and security patches, so I'm going to close this.
This has however been implemented in the Design System for conditional reveals in both the checkboxes and radios components:

Thanks, @kateiw! 🎉


Clobbers custom domain

After publishing, the Github Pages "Custom Domain" repository setting is blanked out.

Same issue for me

I've found the issue. It's simply because of the missing CNAME file in the branch. You just have to create one after the build. It would be nice to have a parameter to allow it.

Thanks, that fixes it.

Where's that custom domain setting? Had you asked I would've said to just add the CNAME file into your doc build somehow (ie, just check it into Git in the docs directory or what have you). But I should add a note to the README if there's a setting on GitHub now that gets out of whack with a push.

In Github website, in the repository "Settings" tab, there is a "Github Pages" section, which has a "Custom Domain" setting. I think this is somewhat new, and didn't realize it was just a front-end for the old CNAME file method.

There's now a command line option to write a CNAME file in 0.5.4.


Test for numeric timestamp parser with string timestamp

@samarthjain - I added a test case for #9877 - I think this is the intended behavior. Can you confirm?


NotImplementedError: The following operators are not implemented: ['aten::values']

I'm using file /python/tvm/relay/frontend/pytorch.py to convert pytorch model, but raise error like this:
Traceback (most recent call last):
  File "apply_opt_torch.py", line 193, in <module>
    run_direct()
  File "apply_opt_torch.py", line 166, in run_direct
    mod, params, shape_dict = get_network()
  File "apply_opt_torch.py", line 109, in get_network
    mod, params = relay.frontend.from_pytorch(model, shape_dict)
  File "/data00/maxiandi/opensource/incubator-tvm/python/tvm/relay/frontend/pytorch.py", line 2626, in from_pytorch
    _report_missing_conversion(op_names, convert_map)
  File "/data00/maxiandi/opensource/incubator-tvm/python/tvm/relay/frontend/pytorch.py", line 2112, in _report_missing_conversion
    raise NotImplementedError(msg)
NotImplementedError: The following operators are not implemented: ['aten::values']

pytorch == 1.3.1 and tvm is latest

cc @MarisaKirisame

I don't think aten::values should appear if you use torch.jit.trace(). If this is a real issue, please add a reproducible script.


Feature Request - Modify Container Resource limits on the fly

I would like to be able to modify the resource limits of a container while it is running. I guess the bigger question is: can container resource limits be changed while the container is running and have the new limits work? If so is it possible for you to implement this?

Completed by PR #14


[E2E Test] Many actor failures in the same run

On Linux runs, sometimes the test shows multiple failures on actor tests.

No failures for this particular test on linux in >45 runs. It is exceptionally rare, and therefor pretty low on the priority list compared to some other more frequent issues.

Closing this since it does not seem to be reproducible anymore.


Decorate an event handler

Hi guys,
I'm trying to decorate a class that implements IEventHandlerAsync<T>.
I'm using AutoFac as my DI framework.
As far as I understand, Kledex registers all the classes that implements this interface IEventHandlerAsync even the decorator, so it ends up creating a circular reference...
Any idea ?
Thanks !

HI @quentin-villevieille, yes all implementations are registered automatically if you pass the types when registering the library:
services.AddKledex(typeof(Product));

where Product is a type in the library that you want to auto register all the implementations from.
You can try to register Kledex without passing any types:
services.AddKledex();

and then register your implementations manually:
services.AddKledex();
services.AddTransient(typeof(IEventHandlerAsync<CreateProduct>), typeof(CreateProductHandler));

cool, I'll try that, thanks!

Also, remember that for your commands you can pass your own method without the need of implementing a ICommandHandlerAsync interface. More info here.


[ENH] reset back to previous JS api, keeping other improvements

Upon further discussion - we decided we preferred the object-oriented JS API given our design constraints.
This PR reverts the api to the previous design while keeping the other improvements introduced in https://github.com/chroma-core/chroma/pull/2542

@codetheweb my todos (not blocking your review)

[ ] ensure docs generation from the code works and looks good
[ ] figure out why that linter didnt pass

#2756
#2714  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jeffchuber and the rest of your teammates on  Graphite


[$1000] "Mark as done" button is grayed out and appears pressed when clicking on "To" to assign a task

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open a chat
Navigate to the "Assign Task" section of the chat
Enter the task title and click the "Next" button
Confirm the task by clicking "Confirm task."
Click on "To" to assign the task

Expected Result:
The "Mark as done" button should not be grayed out when clicking on "To" to assign a task. It should not appear pressed either
Actual Result:
When clicking on "To" to assign a task, the "Mark as done" button is grayed out and appears pressed, giving the impression that it is also pressed
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: yes
Reproducible in production?: yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/15fa91f6-3e26-48cf-8380-b6e2df5101c2
https://github.com/Expensify/App/assets/93399543/d369f876-3a7e-44e9-9e99-52c856c7e1e9
Expensify/Expensify Issue URL:
Issue reported by: @ayazhussain79
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686222783852909
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01d97c2de3b309cd18
Upwork Job ID:<PHONE_NUMBER>862142976
Last Price Increase: 2023-06-16

Proposal
Please re-state the problem that we are trying to solve in this issue.
"Mark as done" button is grayed out and appears pressed when clicking on "To" to assign a task
What is the root cause of that problem?
Button Mark As Done is inside is in the clickable PressableWithFeedback that open Assignee modal, and the PressableWithFeedback has pressDimmingValue={0.2} which make the whole PressableWithFeedback has opacity when clicked.
https://github.com/Expensify/App/blob/4343d287ce1fff5bd1c4112c379fe645ba8650fc/src/components/TaskHeader.js#L59
What changes do you think we should make in order to solve the problem?
Change the view structure to make button Mark As Done outside the clickable PressableWithFeedback.
the new structure will be
<View style={[styles.flexRow]}>
    <PressableWithFeedback
        onPress={() => Navigation.navigate(ROUTES.getTaskReportAssigneeRoute(props.report.reportID))}
        wrapperStyle={[styles.flex1]}
        ....
    >
        <AssigneeAvatarAndNameIfFound/>
        // <we can also add placeholder text, to tell user to click here to assign>
   </PressableWithFeedback>

   <MarkAsREadButton>
</View>

when press in

full code for test
return (
    <View style={styles.borderBottom}>
        <View style={[{backgroundColor: themeColors.highlightBG}, styles.pl0]}>
            <View style={[styles.ph5, styles.pb5]}>
                <Text style={[styles.textLabelSupporting, styles.lh16]}>{props.translate('common.to')}</Text>
                <View style={[styles.flexRow]}>
                    <PressableWithFeedback
                        onPress={() => Navigation.navigate(ROUTES.getTaskReportAssigneeRoute(props.report.reportID))}
                        disabled={!isOpen}
                        accessibilityRole="button"
                        accessibilityLabel={props.translate('newTaskPage.assignee')}
                        hoverDimmingValue={1}
                        pressDimmingValue={0.2}
                        wrapperStyle={[styles.flex1]}
                    >
                        <View style={[styles.flexRow, styles.alignItemsCenter, styles.pv3]}>
                            {!_.isEmpty(props.report.managerEmail) && (
                                <>
                                    <Avatar
                                        source={assigneeAvatar}
                                        type={CONST.ICON_TYPE_AVATAR}
                                        name={assigneeName}
                                        size={CONST.AVATAR_SIZE.HEADER}
                                    />
                                    <View style={[styles.flexColumn, styles.ml3]}>
                                        <Text
                                            style={[styles.headerText, styles.pre]}
                                            numberOfLines={1}
                                        >
                                            {assigneeName}
                                        </Text>
                                    </View>
                                </>
                            )}
                        </View>

                    </PressableWithFeedback>

                    <View style={[styles.flexRow, styles.pv3]}>
                        {isCompleted ? (
                            <>
                                <Text>{props.translate('task.completed')}</Text>
                                <View style={styles.moneyRequestHeaderCheckmark}>
                                    <Icon
                                        src={Expensicons.Checkmark}
                                        fill={themeColors.iconSuccessFill}
                                    />
                                </View>
                            </>
                        ) : (
                            <Button
                                success
                                isDisabled={TaskUtils.isTaskCanceled(props.report)}
                                medium
                                text={props.translate('newTaskPage.markAsDone')}
                                onPress={() => TaskUtils.completeTask(props.report.reportID, title)}
                            />
                        )}
                    </View>

                </View>
            </View>
        </View>
        <MenuItemWithTopDescription
            shouldShowHeaderTitle
            title={props.report.reportName}
            description={props.translate('newTaskPage.task')}
            onPress={() => Navigation.navigate(ROUTES.getTaskReportTitleRoute(props.report.reportID))}
            disabled={!isOpen}
            interactive={isOpen}
        />
        <MenuItemWithTopDescription
            title={lodashGet(props.report, 'description', '')}
            description={props.translate('newTaskPage.description')}
            onPress={() => Navigation.navigate(ROUTES.getTaskReportDescriptionRoute(props.report.reportID))}
            disabled={!isOpen}
            interactive={isOpen}
        />
    </View>
);

What alternative solutions did you explore? (Optional)
set pressDimmingValue={1}

We're redesigning the task view in https://github.com/Expensify/App/issues/19330, so let's not do anything there

Yeah I mean this issue - I'll just put it on hold so we don't get any more bug reports for this and then we can close it once the other PR is done!

Issue is on HOLD, i'll demote it to weekly so that melvin does not ping us about it being overdue often.

https://github.com/Expensify/App/issues/19330 has been completed. Taking our issue off Hold
@thienlnam shall we close this issue?

Yup! I think if we can't reproduce this anymore we can close!

Awesome, let's close this.

Issue not reproducible during KI retests. (First week)


python311Packages.dm-control: 1.0.19 -> 1.0.20

Description of changes
Changelog: https://github.com/google-deepmind/dm_control/releases/tag/1.0.20

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.

Result of nixpkgs-review pr 317952 run on aarch64-darwin 1

4 packages marked as broken and skipped:

python311Packages.dm-control
python311Packages.dm-control.dist
python311Packages.shimmy
python311Packages.shimmy.dist

Result of nixpkgs-review pr 317952 run on x86_64-darwin 1

4 packages marked as broken and skipped:

python311Packages.dm-control
python311Packages.dm-control.dist
python311Packages.shimmy
python311Packages.shimmy.dist

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

4 packages built:

python311Packages.dm-control
python311Packages.dm-control.dist
python311Packages.shimmy
python311Packages.shimmy.dist

Result of nixpkgs-review pr 317952 run on aarch64-linux 1

4 packages built:

python311Packages.dm-control
python311Packages.dm-control.dist
python311Packages.shimmy
python311Packages.shimmy.dist


implemented remaining methods

remove npm-debug.log
fix typo in npm build script
update appInsights node modules to latest version
implemented remaining methods
removed notImplemented

LGTM! Thank you @viperguynaz!
How is it working so far for you?
Any issues/problems, or is it working correctly?
I'll push a new version to NPM, great job! :1st_place_medal:

@MarkPieszak - I'm going to be working on putting it to work this week.  Couple questions:

Why the "isBrowser" checks?  I saw some discussion about use with Universal - if that's the case, why not use the AppInsights SDK for Node?
If this is intended for client-side use, why the boilerplate pass-throughs in the service? We could simply extend AppInsights with the config and the init method for a more maintainable solution.

I'll get a PR up later to show you what I'm thinking.  I'm in need of both server (Universal/node) and client-side integration for AppInsights.  Maybe we can bring in both packages - or it might be simpler to keep them separate?

I just published the new version to npm btw.
So originally that was in case someone was using it with Angular Universal (I just wanted those requests to be ignored on the server) since they would of all gotten re-ran on the Client. Also the current SDK we were using blew up on the server in general (so to avoid it, just removed it completely).
I didn't do enough research, but can the other one be used for any platform?
I'm open to any ideas honestly! This was just an initial prototype/concept, if there's a way so that it could work Universal'y that'd be great :) @viperguynaz


Error when trying to use Money

Trying to add Money to an (older) project, and I'm bumping up against this, any ideas?
 ERROR in<EMAIL_ADDRESS>7:33-50   Module not found: Error: Package path . is not exported from package /Users/xxxx/src/my-tiller-tools/node_modules/big.js (see exports field in /Users/xxxx/src/my-tiller-tools/node_modules/big.js/package.json)
The package versions installed are:
`yarn list --pattern money
yarn list v1.22.19
└─<EMAIL_ADDRESS>yarn list --pattern big.js
yarn list v1.22.19
├─<EMAIL_ADDRESS>

I was not able to figure out what exactly was causing this, but after deleting node_modules, and cleaning all generated bits, it is all working again.

After deleting node_modules, and cleaning all generated artifacts from my project, things are working again. I was never able to figure out why this was happening.

After deleting node_modules, and cleaning all generated artifacts from my project, things are working again. I was never able to figure out why this was happening.


A few notes for the FAQ

Wanted to add notes and corrections, felt like the easiest solution was to fork and do some edits. Not really mergeable because some of my notes are just thoughts I've dumped in there.
I didn't hit the entire FAQ but at least 66%

Hm, I am not sure I understand the followin

Key rotation/replacement is one of the motivations for introducing DNS to Hyper's addressing scheme as it would enable key rotation via the indirection of name lookup.

I discussed key rotation for individual Hypercores with @mafintosh and multi-key module came up as the solution. Not sure how DNS could help and why would we use DNS which is quite centralized and hard to update. Didn't we git rid of it with DHT?

can you elaborate on the manifest

I discussed key rotation for individual Hypercores with @mafintosh and multi-key module came up as the solution. Not sure how DNS could help and why would we use DNS which is quite centralized and hard to update. Didn't we git rid of it with DHT?

Well, people still want shortnames. We've discussed some alternatives to the ICANN DNS but I think any alternatives would have to be additions, not replacements.
I'm not sure if DHT == DNS is quite the right way to describe it. I often think of it more as a replacement for IP addresses.
I wouldn't propose DNS as the only solution to key rotation, but it's obviously able to accomplish that task.

can you elaborate on the manifest

Nothing thought-out, just suggesting a userland application, eg a package manager, could have manifests which record the strong-link hashes.

on Key rotation, I see that we are discussing 2 separate things. I think you are talking about human readable URLs, as in the discovery key to the Hyperdrive in the swarm. And I agree, people will need short names that can be remembered and searched well. For me DNS has 2 dimensions:

mapping a location independent string to a location (IP address). AWS URLs are not short as they have a very long host name part in them. They provide this independence of the actual IP address which decouples network infrastructure from services. DHT does that in a decentralized manner (except the bootstrap IPs)
reserving a user-friendly recognizable name. This we do today with the registrar. This part is very to decentralize https://en.wikipedia.org/wiki/Zooko's_triangle. DHT does not help here. Namecoin was first to solve Zooko triangle puzzle, and Ethereum ENS smart contract is now well on the way to be adopted as as decentralized solution for this. I spent quite some time discussing this subject in 2014 with various people who were trying to solve it, e.g. with Greg who created okTurtles and with the Blockstack guys.

When I talked about key rotation though, I meant another problem:
There is a need for changing private key of ecisitng Hypercore. It is needed when you restore on a new device, for moving to a new key when old key is lost or suspected to be compromised, and for security best practices, as keys should change (get rotated) over time. We need to solve this for mass deployment. We all are familiar with Gmail mechanisms for recovery of lost credentials. They are plentiful and are being extended with more options every year. As a first step, I was thinking how to use multi-key to continue with the same Hypercore but new credentials.

I seem to have merged all suggestions above and added a more content to FAQ, e.g. USP 'time travel', 'real-time UI updates', more use cases for hypebee as streaming DB.


generated_plugin_registrant.dart does not respect prefer_double_quotes: true

Every time run debug for the flutter app, the generated_plugin_registrant.dart file got generated with content like this:
//
// Generated file. Do not edit.
//

// ignore_for_file: directives_ordering
// ignore_for_file: lines_longer_than_80_chars

import 'package:shared_preferences_web/shared_preferences_web.dart';

import 'package:flutter_web_plugins/flutter_web_plugins.dart';

// ignore: public_member_api_docs
void registerPlugins(Registrar registrar) {
  SharedPreferencesPlugin.registerWith(registrar);
  registrar.registerMessageHandler();
}

but I have set prefer_double_quotes: true in analysis_options.yaml so the linter keeps complain about Prefer double quotes where they won't require escape sequences.

Please file this issue against the flutter/flutter repo. They can take the steps mentioned above.

Thank you @srawlins @Levi-Lesches!


InterpolationMode "bicubic" produces values outside of [0, 255] with float tensors.

🐛 Describe the bug
So yea as the title says "bicubic" doesn't work as it should if the passed tensor values are floats.
This might happen for example if the user converts their numpy.ndarray image to torch.Tensor with the functional transform to_tensor.
Here's how to reproduce:
import torch
import torchvision.transforms as T

# Create a half white half black image.
image = torch.tensor([
    [[255,   0,   0]],
    [[255,   0,   0]],
    [[255,   0,   0]]
],  dtype=torch.uint8)

# Define resize ops.
resize = T.Resize((1,2), T.InterpolationMode("bicubic"))

# First as `uint8`.
resize(image)

>> tensor([[[197,   0]],
           [[197,   0]],
           [[197,   0]]], dtype=torch.uint8)

# Then as floats
resize(image.to(torch.float32))

>> tensor([[[197.2266,  -8.9648]],
           [[197.2266,  -8.9648]],
           [[197.2266,  -8.9648]]], dtype=torch.float64)

How it should work.
Either an error should be raised if the type isn't torch.uint8 or the interpolation should be calculated correctly (preferable).  The same error doesn't happen with cv2.resize(image, size, cv2.INTER_CUBIC).
Versions
Collecting environment information...
PyTorch version: 1.10.0+cu113
Is debug build: False
CUDA used to build PyTorch: 11.3
ROCM used to build PyTorch: N/A

OS: Ubuntu 20.04.3 LTS (x86_64)
GCC version: (Ubuntu 9.3.0-17ubuntu1~20.04) 9.3.0
Clang version: Could not collect
CMake version: Could not collect
Libc version: glibc-2.33

Python version: 3.8.12 (default, Oct 12 2021, 13:49:34)  [GCC 7.5.0] (64-bit runtime)
Python platform: Linux-5.11.0-38-generic-x86_64-with-glibc2.17
Is CUDA available: True
CUDA runtime version: Could not collect
GPU models and configuration: GPU 0: Tesla V100-PCIE-16GB
Nvidia driver version: 470.63.01
cuDNN version: Could not collect
HIP runtime version: N/A
MIOpen runtime version: N/A

Versions of relevant libraries:
[pip3] numpy==1.21.3
[pip3] pytorch-lightning==1.4.9
[pip3] torch==1.10.0+cu113
[pip3] torchmetrics==0.6.0
[pip3] torchvision==0.11.1+cu113

cc @vfdev-5 @datumbox

@jopo666 it is expected result for float dtype. For example, opencv gives the same result:
import torch
import torchvision.transforms as T

image = torch.tensor([[
    [255,   0,   0],
    [255,   0,   0],
    [255,   0,   0]
]],  dtype=torch.uint8)

print(image.shape)  # 1, 3, 3

# Define resize ops.
m = T.InterpolationMode("bicubic")
print("mode:", m)
tv_resize = T.Resize((2, 2), m)

tv_resize(image.to(torch.float32))
> tensor([[[197.2266,  -8.9648],
           [197.2266,  -8.9648]]])

import cv2
print(cv2.__version__)  # '4.5.3'

cv2.resize(np_image[0, ...], dsize=(2, 2), interpolation=cv2.INTER_CUBIC)
> array([[197.22656 ,  -8.964844],
         [197.22656 ,  -8.964844]], dtype=float32)

Am I missing something ?

Ahh I had a typo in my cv2.resize function... Maybe there should be a warning if the passed image(s) aren't uint8?

@jopo666 I think it is still valid to pass float images to resize with bicubic mode.


Remake cargo orders to contains flatpacks

About the PR

All cargo orders that contains engineering structures now goes in flatpacks
Completely fixes #24823
I left some crates that arent orderable contains structures, should be fixed in the future so ill create an issue.
Why / Balance

emo and sloth said its good
Technical details

Media

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

Breaking changes

Changelog

:cl:

tweak: Engineering structures orders in cargo now supplied via flatpacks (e.g. tesla & singulo generators, other stuff for them).
tweak: Thrusters and gyroscopes now supplied in crates via flatpacks.

This makes a lot more sense lol


why @numba.jit(nopython=True)  cause memory leak, thanks

Reporting a bug

I have tried using the latest released version of Numba v0.56.4

thanks!!!

Thanks for reporting the issue.
I don't recognize the tool that you are using. Can you tell us more about how you are detecting memory leak? That will help us diagnosis the problem.

@sklam Thanks for your reply! I used 'objgraph' to check for the leaks. I put the relevant code in the 'for loops'. In the end , i found that the problem occurred here. My python version is 3.9, and my numpy version is 1.21.5, If I remove the @numba.jit() before the code, the leaks will not occur.

This issue still needs to be addressed,thanks!

@Creazy-MR are you able to share a fully reproducible example? It would be useful to see what the input data looks like & how you are invoking that function. It's helpful to put code into a python code block, which makes it easier to reproduce and debug
eg.
import numba

@numba.jit(nopython=True)
def update_H(*args, **kwargs):
    ....

@jack-mcivor I am sorry for not replying for a long time. Due to project confidentiality, I am unable to provide a complete program. However, in the function mentioned above, the input U and H are numpy arrays.

@jack-mcivor I am sorry for not replying for a long time. Due to project confidentiality, I am unable to provide a complete program. However, in the function mentioned above, the input U，sum_tpo, sum_down and H are numpy arrays,The others are float32 or integer.thanks


Convert Tab's context flyout into a XAML control

Tabs currently create their context flyout menu items in a big create function. It would be nice to convert that into a custom XAML control so that the more menu items we decide to add to the flyout, the easier it will be to manage. We should also design it in a way where different types of tabs (i.e. SettingsUI tab vs TerminalTab) can choose what menu items they would like to use.

Funny enough, I tried to do this (just having the context menu in a .xaml file) and it was a total pain to hook up the event handlers without introducing another class just to implement the events that could come out of the context menu :smile:

Damn, it's not as easy as exposing the MenuFlyoutItems in the IDL and just being like menuflyoutitem.Click{function}?

Oh nvm I see, you tried it out by only having a XAML file?

Yep. Just a XAML file w/ a flyout, then one with a usercontrol w/ a flyout, then then then

Have you looked into using Xaml UserControls or CustomControls?

@mdtauk most of our project is UserControls or CustomControls.

@mdtauk most of our project is UserControls or CustomControls.

Fair enough then lol

What if we just passed a ShortcutActionDispatch into the (theoretical) TabContextMenuControl? Then we could have the items in the flyout dispatch actions to that, routing them up to the Page as usual. We'll probably want that anyways, esp if we want to let users customize the content of that flyout.


gogs keep hanging in intranet environment.

Gogs version (or commit ref): <IP_ADDRESS>1
Git version: 2.6.4
Operating system: linux
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[x] SQLite

Can you reproduce the bug at https://try.gogs.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

Description
gogs keep hanging in intranet environment.
It finds secure.gravatar.com for a long time.
Although I choose custom avatar.

You can turn on offline mode: https://github.com/gogits/gogs/blob/master/conf/app.ini#L122

Oh, OK. Thank you for letting me know!


make arangodump/restore fail with invalid collection names

Scope & Purpose
arangodump and arangorestore will now fail when using the --collection option and none of the specified collections actually exist in the database (on dump) or in the dump to restore (on restore). In case some of the specified collections exist, arangodump/restore will issue warnings about the invalid collections, but will continue to work for the valid collections.
Documentation PR: https://github.com/arangodb/docs/pull/452

[x] Strictly new functionality (i.e. a new feature / new option, no need for porting)
[x] The behavior in this PR can be (and was) manually tested (support / qa / customers can test it)
[x] The behavior change can be verified via automatic tests

Testing & Verification
This PR adds tests that were used to verify all changes:

[x] Added new integration tests (shell_client, for arangodump)

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/10217/

@KVS85 : I disagree with the requested changes. If we do a graceful exit, the exit code will be 0, so for automated processes there is no way to detect any error.
This is a fatal error IMHO, as we will stop and not continue with dump/restore. Same thing as if command-line options are wrong and startup fails.

http://<IP_ADDRESS>:8080/job/arangodb-matrix-pr-linux/11975/


Fix Issue 22125 - std.process.Config was changed to a struct but miss…

… operator overloads

Oops, stable


High contrast mode isn't persisted on skillmap

Describe the bug
If I have high contrast on and start a skillmap the setting isn't on. It is activated once you select a level though but the landing page isn't accessible
To Reproduce
Steps to reproduce the behavior:

Set high contrast mode on
Open any skillmap

Expected behavior
High Contrast is persisted
Screenshots

The level itself is on high contrast

Desktop (please complete the following information):

OS: Windows 11
Browser Edge

This seems to be fixed .


Reply in thread

Is there a way to have the bot reply in a thread? Right now, our bot successfully listens to messages for threads in a channel, but it replies to the channel.
I'm aware that threads are a relatively new feature, but is there a way to reply within the thread? If not, will this be possible in the future?

I have been playing with this feature this morning and it seems fairly easy to implement by including the thread_id in the message send to the API.
The question remains how Slackbot should be made aware that the message should be send to a thread. I would suggest something along the lines of the code below. Notice the extra in_thread argument passed to reply method.
@listen_to("Howdy")
def greet(message):
    message.reply("Howdy to as well", in_thread=True)

Internally Slackbot would determine the thread's ID by looking at the message.body property. For example the following property could be added to the message class:
@property
def thread_ts(self):
    try:
        thread = self.body['thread_ts']
    except KeyError:
        # This would mean there's no thread yet and one will be started
        thread = self.body['ts']

    return thread

If people are interested, I think I could implement this one of these days.

This would be highly appreciated :)

Just wondering if this enhancement is planned anytime soon :)

I must be honest, I kind of slipped my mind. I will see if I can make a decent start on it this weekend.

#153

Thanks a lot, @NiekKeijzer 😄

I was just trying the in_thread parameter using the code you provided for testing and i keep getting this error. Any idea why?
Traceback (most recent call last):   File "/usr/local/lib/python2.7/dist-packages/slackbot/dispatcher.py", line 55, in _dispatch_msg_handler     func(Message(self._client, msg), *args)   File "/home/dev/Desktop/Github/securitybot/securitybot2/securitybot2/plugins.py", line 27, in help     message.reply("Here's a threaded reply", in_thread=True)   File "/usr/local/lib/python2.7/dist-packages/slackbot/dispatcher.py", line 175, in wrapped     return func(self, text, *a, **kw) TypeError: reply() got an unexpected keyword argument 'in_thread'

Hi, @jaysig112 - are you using the latest code on the develop branch? You may want to rerun setup or pip install requirements.txt - the threaded reply option requires a relatively new version of one of our library dependencies (slacker>=0.9.50).

I just installed everything today and I checked slacker it is version 0.9.50

What version of slackbot do you have? The version with in_thread is not yet on pypi. You'll need to install slackbot from the develop branch of git.

Yep that fixed it! Thanks! While I have you here if I wanted to have it keep replying in the thread how would I go about that. Like I start a conversation  with the bot it replies in the thread with a question then I answer in the thread and it keeps going until I answer all the bots questions.

How do I use this feature?


Bug in inference of control of transparent proxies

If an OAuth-style authorization service controlling a service accessed via a transparent proxy is also accessed via the same proxy, it should be inferred to have control of the proxy (being another back-end service of the proxy). A construction sequence error means this inference is made after it would be used. See #47.

Now fixed in branch 47.


colorscheme failing to load if set via vimscript

First of all thank you again for yet another great and solid work!
I am getting an exception while loading colorschemes if the instructions for it come from vimscript runtime paths (allegedly not if setting the option in init.lua). I have created a minimally reproducible neovim configuration as follows:
├── lua
│  └── plugins.lua
├── plugin <-- this is standard vim runtime path
│  └── settings.vim
└── init.lua

where init.lua contains, as per README
local lazypath = vim.fn.stdpath("data") .. "/lazy/lazy.nvim"
 if not vim.loop.fs_stat(lazypath) then
   vim.fn.system({
     "git",
     "clone",
     "--filter=blob:none",
     "--single-branch",
     "https://github.com/folke/lazy.nvim.git",
     lazypath,
   })
 end
 vim.opt.runtimepath:prepend(lazypath)

require("lazy").setup("plugins")

with the corresponding lua/plugins.lua file containing the list of plugins to load, as per your example
return {
    'folke/tokyonight.nvim'
 }

Now if I activate the colorscheme adding, in plugin/settings.vim
colorscheme tokyonight

I get an exception from this line and the colorscheme does not load (even if I manually set it within neovim with :colorscheme tokyonight). Viceversa, if I instead append vim.cmd[[colorscheme tokyonight]] to init.lua (after loading lazy.nvim) then all is good.
TL;DR colorschemes are only applied if instructed so in init.lua, not so if the instruction is given through standard vim runtime path plugin/<something>.vim
P. S. You may be asking why I still give the instruction via vimscript files: it is because I am currently using VimPlug and with it most settings are more easily set still via vimscript.
Which version of Neovim are you using?
nvim --version
NVIM v0.9.0-dev-531+gf04087d8b-dirty
Build type: Release
LuaJIT 2.1.0-beta3

Is it possible that anything contained in the runtime path plugin/*.vim is loaded before init.lua? I can obtain the correct behaviour (with colorschemes loading properly) doing
require("lazy").setup(
	{
		'folke/tokyonight.nvim',
		config = function()
			vim.cmd[[colorscheme tokyonight]]
		end
	}
)

would you say this is the preferred method to load colorschemes?

it is (he also said this in other issues)

it is (he also said this in other issues)

Great! Could you point me to the corresponding issue, so that I can read up more about it?

https://github.com/folke/lazy.nvim/issues/40

Thank you, I browsed the previous issues but somehow missed that one: it solves the problem (sorry folke for having created a redundant one)!

I just fixed this, since your setup should have worked too.
I guess the correct NEovim way would have been for you to put those files in /after/plugin, but it will work now in /plugin too.
I've updated the readme about this change: https://github.com/folke/lazy.nvim#-startup-sequence


REPO-1126 - Task Master Integration

@prdanelli please let's remember to squash the commits before merging... the history on this branch is getting bigger :)

@prdanelli very very minor, but the name of the service is taskmaster rather than Task Master or task_master

@prdanelli very very minor, but the name of the service is taskmaster rather than Task Master or task_master

@fradeve so it would be Taskmaster if we are naming classes?


TS-4088: Add support for BoringSSL

Looks good to me, ship it!

Looks good to me as well.

+1 after fixing review comments. We should aim to not have to use OPENSSL_IS_BORINGSSL.

James, the problem is that if you do what you suggest, it likely will fail. It will still find that OpenSSL include file (because it's likely to be on the system), and it conflicts with BoringSSL. This is what I recall from testing this before.

Oh that's a bit nasty, we should add a comment to that effect. I think it would be best to consolidate the openssl includes in one header so we can capture this kind of weirdness in just one place.

Yeah, that's a good idea (putting all SSL library weirdness in one file).


Gh 675 create spark import to accumulo from pair rdd operation

Codecov Report

Merging #703 into develop will increase coverage by 0.12%.
The diff coverage is 68.58%.

@@              Coverage Diff              @@
##             develop     #703      +/-   ##
=============================================
+ Coverage       60.6%   60.73%   +0.12%
- Complexity      2693     2735      +42
=============================================
  Files            393      404      +11
  Lines          12106    12296     +190
  Branches        1383     1397      +14
=============================================
+ Hits            7337     7468     +131
- Misses          4193     4240      +47
- Partials         576      588      +12

Impacted Files
Coverage Δ
Complexity Δ

.../spark/operation/scalardd/ImportRDDOfElements.java
0% <ø> (ø)
0 <ø> (?)

...ark/operation/javardd/ImportJavaRDDOfElements.java
0% <ø> (ø)
0 <ø> (?)

...fs/handler/ImportAccumuloKeyValueFilesHandler.java
9.09% <ø> (ø)
1 <ø> (ø)
:x:

...operation/utils/java/ElementConverterFunction.java
100% <100%> (ø)
3 <3> (?)

...n/javardd/ImportKeyValueJavaPairRDDToAccumulo.java
100% <100%> (ø)
6 <6> (?)

...tion/scalardd/ImportKeyValuePairRDDToAccumulo.java
100% <100%> (ø)
6 <6> (?)

...o/operation/utils/AccumuloKeyRangePartitioner.java
62.96% <62.96%> (ø)
7 <7> (?)

...alardd/ImportKeyValuePairRDDToAccumuloHandler.java
64.28% <64.28%> (ø)
5 <5> (?)

...dd/ImportKeyValueJavaPairRDDToAccumuloHandler.java
68.42% <68.42%> (ø)
4 <4> (?)

...andler/javardd/ImportJavaRDDOfElementsHandler.java
73.33% <73.33%> (ø)
3 <3> (?)

... and 13 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7357660...a0747a1. Read the comment docs.

I'll implement the requested changes.

All the comments have been addressed. @gaffer01 please could you check over the changes - thanks.


Add Topics to Challenge Object

[ ] add inputDataTypes required in the challenge search page
[ ] add topics required in the challenge profile page
[ ] optional, add websiteUrl which might be used in challenge profile as well - by clicking, redirect to challenge page

@rrchai inputDataTypes and topics deserve their own issue. Implementing inputDataTypes should be relatively straightforward but tag may be a bit more complicated depending on whether we plan for user to define new tags. Thanks!

The inputDataTypes property is now tracked in #1179

Adding this ticket to Sprint 23.02

We could include on the home page a section with popular tags/topics like here.


Unable to perform reduce operation on list

I am using the map operation  in the pipeline.yaml to generate the following o/p
{
"content": "abcde",
"classification": ["CLASS1"]
},
{
"content": "qwerty",
"classification": ["CLASS1","CLASS2"]
}
I used the reduce operation to aggregate the content based on the classification key (using reduce_key as "classification") in pipeline.yaml . However getting error
"TypeError: unhashable type: 'list'"

Thanks for flagging this! I just added support for list-type reduce_keys, which will sort the values in the list, convert to a tuple, and use this tuple for the reduce_key. LMK if this is not what you intended; feel free to re-open the issue.

The aggregation is working, but if the same classification is present in multiple lists, then it is getting aggregated separately.
example
If this is the input
{
"content": "Document about AI and ML",
"classifications": ["AI", "ML"]
},
{
"content": "Document about AI only",
"classifications": ["AI"]
}
then the expected aggregation  should be
AI -> Document about AI and ML. Document about AI only
ML -> Document about AI and ML
The actual aggregation is
AI -> Document about AI only
AI,ML -> Document about AI and ML


Weird struct alignment

With this as .rustfmt.toml:
format_strings = true
merge_imports = true
normalize_comments = true
reorder_impl_items = true
struct_field_align_threshold = 5
use_field_init_shorthand = true
use_try_shorthand = true
wrap_comments = true

rustfmt 0.4.2-nightly (dd807e2 2018-04-18) puts forward the following diff:
 #[derive(Fail, Debug, Clone)]
 pub enum BuildError {
-    LineTooLong { length: usize, limit: usize },
-    DisallowedByte { b: u8, pos: usize },
+    LineTooLong { length: usize, limit:  usize },
+    DisallowedByte { b:   u8, pos: usize },
     ContainsNewLine { pos: usize },
 }

I guess the struct-like elements inside the enum is putting rustfmt off the track?
Hope that helps, and thanks for your work!

I see something similar due to combining indent_style = "Visual" with struct_field_align_threshold
    Ok(ServerInformation { name:         unwrap_message_string(items.get(0)),
        vendor:       unwrap_message_string(items.get(1)),
        version:      unwrap_message_string(items.get(2)),
        spec_version: unwrap_message_string(items.get(3)) })

Encountered the same problem as @hoodie.

Encountered this myself, boiled it down to a simple repro:
enum Foo {
    A { a:     usize, bbbbb: () },
    B { a:     (), bbbbb: () },
}

rustfmt.toml:
struct_field_align_threshold = 5

Seems like something is trying to line up all 4 things, but not realizing they aren't on separate lines.

@hoodie As per the documentation on struct formatting, I guess the following formatting for https://github.com/rust-lang-nursery/rustfmt/issues/2633#issuecomment-404679937 would be the expected output, isn't it ?
    Ok(ServerInformation { name:         unwrap_message_string(items.get(0)),
                           vendor:       unwrap_message_string(items.get(1)),
                           version:      unwrap_message_string(items.get(2)),
                           spec_version: unwrap_message_string(items.get(3)) })

Below is the output when using only the parameter indent_style = "Visual":
     Ok(ServerInformation { name: unwrap_message_string(items.get(0)),
                            vendor: unwrap_message_string(items.get(1)),
                            version: unwrap_message_string(items.get(2)),
                            spec_version: unwrap_message_string(items.get(3)) });

Once the parameter struct_field_align_threshold = 50 is added, then it gets formatted to what you shared.


Copying certs doesn't work again

Task:
name: Gen_certs | Copy certs on masters
Error:
...FAA== '|' base64 -d '|' tar xz -C /etc/ssl/etcd/ssl", "failed": true, "msg": "[Errno 7] Argument list too long", "rc": 7}

Number of nodes: 70
cc @bogdando @mattymo @pigmej

@pskrzyns Can you calculate how many characters your 70 hosts ansible hostnames take up?

Perhaps smaller bacthes of nodes in plays could w/a that

@mattymo something about 710 characters

@pskrzyns I'll work on this tomorrow. I was out sick today. It has to do with the base64-encoded tar data


Implement shutdown_anysegwit

This is something that should be done early, so that when we start leveraging Taproot wallets already have support for accepting P2TR addresses.

Done in #313


[Activate] test SMT repositories before image activation

It would be better to check the presence of the required repositories on the SMT server provided by the user.
That would prevent failures when activating the image.

Closing, starting from version 0.4.0 it will no longer be possible to build the SLE Docker images from the templates.


sagemath derivation fails

src/lp_cdd.cpp:4:10: fatal error: setoper.h: No such file or directory

Hello, @jfaure!
I have no idea what environment you use, but you might get a working SageMath by saving the following code as shell.nix:
{ pkgs ? import (fetchTarball
  "https://github.com/NixOS/nixpkgs/archive/9480bae337095fd24f61380bce3174fdfe926a00.tar.gz") {}
}:
pkgs.mkShell {
  buildInputs = [ pkgs.sage ];
}

and running nix-shell.
The build failure you described was fixed in https://github.com/NixOS/nixpkgs/pull/101046 but building sage itself is not yet recovered. Currently, some devs including myself have been trying to update SageMath to a newer version.

Sage has been working for a while now and will be in 21.05. Please reopen if you are still having problems.


Improve protocol configs in application section

Describe the issue:
Following improvements are needed for the application protocol section

Rearrange grant types in OIDC protocol
Improve SAML protocol attribute section
Improve custom application UI
Improve custom application description in the template

This issue is fixed


[Release] v0.0.18

Changed

Return params/headers None when request doesn't have params/headers #87

Fixed

Report-example image not loading on PyPi #86

Codecov Report

Merging #95 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #95   +/-   ##
=======================================
  Coverage   90.18%   90.18%
=======================================
  Files          14       14
  Lines         377      377
=======================================
  Hits          340      340
  Misses         37       37

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8fc1332...86e89e6. Read the comment docs.


Post

Complete tutorial for post, which adds comment form to Contact-Me section and stores them as json objects. Currently, this implementation brings user to a blank page with a json list of comments. This will be improved in the next pull request.

Please merge the json branch to master first
then you'll probably need to rebase post to master, then push post to remote
then, you can edit the base of this PR to be master and merge when ready