Angular 2 / Express isomorphic

Just found an official repo for create isomorphic App with Angular 2 and Express : https://github.com/angular/universal
After a quick look doesn't look too difficult to integrate this with trailpack-express. Maybe I'll try to make a repo example with it to see what it can look like at the end with Trails.
That make me ask myself about trailpack support. What trailpack do you want to support for a V1 ?

Just see there an hapi example too :)

I should have linked you to this, cause that was my inspiration!
Let's support Angular for V1.  Just so we can get a work flow, and then we can pull some one on who is more experienced in React.

Ha ha no problem ;) this discussion can be interesting here to https://github.com/trailsjs/trails/issues/208#issuecomment-222172265 maybe you will have some ideas ^^

Ok I have a start here https://github.com/jaumard/trails-angular2-isomorphic Trails part is working. All Trails file are in TS and compile into ES6 by gulp.
I try to integrate the universal starter example https://github.com/angular/universal-starter but as it use webpack :/ I try to only include Angular2 part, it was also compile (into es5 this time) by gulp. But when I start the server it's not working I have :

/Users/jaumard/Downloads/trails-angular2-isomorphic-master/node_modules/@angular/core/src/util/decorators.js:165
        throw 'reflect-metadata shim is required when using class decorators';
        ^
reflect-metadata shim is required when using class decorators

npm ERR! Darwin 15.5.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "start"
npm ERR! node v4.4.4
npm ERR! npm  v3.9.0
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: `gulp && node dist/server.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'gulp && node dist/server.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the trails-angular-2-isomorphic package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     gulp && node dist/server.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs trails-angular-2-isomorphic
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls trails-angular-2-isomorphic
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/jaumard/Downloads/trails-angular2-isomorphic-master/npm-debug.log

I see a lot of things about this on internet but nothing fix the issue for me... Maybe you will be more lucky with a fresh look :D
My idea here is to first create an example of isomorphic with Trails/Angular2 and when we have something working start working on treefrog, it will be more simple to see the architecture, file structure....

I keep you updated, I fix all server generation problem and the Trails server start correctly now. But the front part doesn't works yet :( in the universal starter ket they use webpack witch bundle everything so they just load bundle.js to the front ^^. As I use gulp nothing was done for front yet. So all the index.js need to be written (load all library files and app) in order to work. No luck until now for me to make it work. But I think we're close to the end to have a working example :)

Awesome! I will most definitely take a look and see what i can do to help.  I like the idea of getting an example app together and then using it to prototype treefrog.

Cool ! Let me know if you found something ^^ make me crazy for now :)
There multiple way to do it :

make the actual code (base on universal starter repo) working by making the systemJS config
remove the actual code and try instead to use universal example on main repo https://github.com/angular/universal/tree/master/examples/src witch not use webpack by default (so maybe more luck with this)

Yeah, we will definitely need something that allows people to use whatever task manager they want (webpack, gulp, whatever new comes out).

@scott-wyatt did you have time to look a little ? I really could use a helping hand on this. :/

I simplify the all things... Trails app doesn't need to be in TS so I restore the JS version who load the compiled todo angular app. Same error but more clean architecture.

Will look soon! Sorry it's taking me so long.

Hey @scott-wyatt ! Finally it's working ! Not optimal at all I have to work more on it to have something more clean with the architecture but that's a start who's working !
Those PRs will be helpful to make things more clean :
https://github.com/trailsjs/trailpack-router/pull/56
https://github.com/trailsjs/trailpack-express/pull/30

You are a rock star!

@scott-wyatt I just update the repo https://github.com/jaumard/trails-angular2-isomorphic I think the file structure and architecture is clean :)
Let me know what do you think of it !

I'll take a look!

This is incredible! Wow, well done! I'm going through all the src now.

Okay, @jaumard and I have an idea how we will accomplish what was discussed in this conversation. Closing this discussion to open new issues.


✨[feature Request]: improve the ui

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
improve ui of contact page
Use Case
for better user experience
Benefits
No response
Add ScreenShots

improve the ui of this
Priority
Medium
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24EXTD contributor
[X] I have starred the repository

@RiyaJain8114 it is already assigned.


Incorrect hostname lookup when relying on $_SERVER['HTTP_HOST']

Affected Version
Seen on 5.1.0 but probably affects 4.x as well
Description
When using the \SilverStripe\Control\Director::hostName() method to get the hostname of the current request you under certain circumstances get a null when you expect a properly formatted hostname. This happens when \SilverStripe\Control\Director::host() returns the $_SERVER['HTTP_HOST'] header which lacks the protocol of the current request, as defined in the rfc (https://www.rfc-editor.org/rfc/rfc7230#section-5.4) although I've noticed that on certain localhost setups it incorrectly does include the protocol. When calling parse_url with the protocol less value of the HTTP_HOST header (as long there is no port number embedded!) parse_url will assume that it is a path and not the hostname, causing a null to return.
Since the configuration override default_base_url which is used in other cases expects a protocol to be set to be used in this instance you'll end up with inconsistent behavior across environments based upon whether or not the HTTP_HOST or the default_base_url is used to resolve the hostname.
Steps to Reproduce
Not really reproducable steps as it depends on the setup. See: https://3v4l.org/blk8f for the observed behavior.

Seeing this issue on 4.13 as well. Was a real mystery as it only occurred in our Silverstripe Cloud environment, but not in local or other environments.


fix: sending images to the vscode extension fails, if the tensor is not local to the cpu

Related-to: https://github.com/Safe-DS/DSL/pull/954 (the PR requires this fix to work)
Fixes the following error:
RuntimeError: Attempted to send CUDA tensor received from another process; this is not currently supported. Consider cloning before sending.
This error only occurs on systems that support graphics acceleration of tensor operations.
On other systems, the tensor is local to the CPU by default and does not need to be converted.
The conversion process is a noop, if the tensor is already local to the CPU.

@SmiteDeluxe The PR for the fix we talked about

The code looks good. I cannot currently verify, though, whether this fixes the issue. Still, feel free to merge if you've checked this.

As I'm not affected by this, @SmiteDeluxe would be able to verify and merge this

:tada: This PR is included in version 0.8.0 :tada:
The release is available on:

v0.8.0
GitHub release

Your semantic-release bot :package::rocket:


[FEATURE] AddOn abstract class that allows for between mod's communication

Feature Information
Feature description
An abstract class that the mod developer can inherit from to create a public facing API to be used by other modders to communicate with his mod. He would redestribute that on his github (or wherever he likes) so that other modders could download that, add to their codes and use the methods without having to reference the original code.
Solution proposal
Something like what RemoteTech2 has with its abstract AddOn class
Additional context
This came from this chat in discord where the modder couldn't properly build an integration with ManeuverNodeController properly. He had to use reflection to achieve the desired behaviour.

For reference, here is such an abstract class in KSP 1:
https://github.com/RemoteTechnologiesGroup/RemoteTech/blob/develop/src/RemoteTech/SimpleTypes/AddOn.cs
And here is an example implementation in KSP 1: https://github.com/RemoteTechnologiesGroup/RemoteTech/blob/develop/src/RemoteTech/AddOns/KerbalAlarmClockAPI.cs
Both examples taken from the RemoteTech KSP 1 mod.

#252 adds a message bus system, which when used with a parameter that can store state does much the same thing


Parallelize adding/removing issues and add tests

Introduces tests using gock since cli/cli and cli/go-gh are deprecating their httpmocks in favor of gock.

@pallavit @jsquire @mayurid this should significantly speed up adding issues to projects in bulk.


npm downloads for new packages

Instead of showing invalid it should show 0 or something.

:+1:

/cc @espadrine :bell:

@espadrine Thanks for fixing! :sparkles:

Thank you for reporting! Sorry it took time.


Fix broken self-link

Good catch, thanks!
@bors r+

:pushpin: Commit 67ec0a5ce371e997a4bd18b026e7e6a17f9770c8 has been approved by RalfJung

:hourglass: Testing commit 67ec0a5ce371e997a4bd18b026e7e6a17f9770c8 with merge cf9d77de470098cb02ed4a0ba7fa6c66025562e4...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing cf9d77de470098cb02ed4a0ba7fa6c66025562e4 to master...


what are your results?

Hi!
Did you manage to reproduce the results on the paper?
I find the repo potentially very useful, thanks for sharing.

Hi, thanks for your interest, we weren't exactly able to replicate the paper's result but we got close.
For ADE20K dataset, without TTA, these are our resulting metrics
aAcc: 82.1200,  mIoU: 48.9400,  mAcc: 61.2500
The reported mIoU was 51.3

thanks! will let you know if I manage to get that 2.4% back :)

from the paper, the results are using TTA. This should be fine

+ms: a boosted version of the linear setup. We concatenate the patch tokens of the 4 last layers, use a larger image resolution of 640, and use multiscale test-time augmentations to improve the predictions.


feat: expose canary reconcile and connection check interval for config

No change in existing behaviour, the canary's existing defaults are propagated upwards.
The motivation for this change is to help investigate MGDSTRM-8698 by having the ability to turn off the connection check.

I think this change would be a prime candidate to use additionalProperties in OperandOverride. For example, the canary could look for a ~Map<String, EnvVar>~ List<EnvVar> entry under additionalProperties.get("env") that could contain any user-provided environment entries and pass them through to the canary container. That leaves a door open to configure individual canary instances rather than having the config apply to the entire fleet.

I like that idea. I think in the cm it would look like:
canary:
  image: quay..
  env:
   - name: FOO
      value: bar

but I don't immediately see how we'd persuade Jackson to manufacture EnvVar objects rather (than Map).   What am I missing?

but I don't immediately see how we'd persuade Jackson to manufacture EnvVar objects rather (than Map). What am I missing?

Use something like Serialization.jsonMapper().convertValue(map, EnvVar[].class) with the map from the env field.

We could just spin off a separate List<EnvVar> env property on the OperandOverride I suppose rather than use the catch-all.

+1

@shawkins @MikeEdgar how does this look?
I needed to make a decision about whether the override feature should apply to any environment variable, even those that probably shouldn't be overridden.  I thought about separating a container's env var into a set of those which are reasonable to override and those which are not (KAFKA_BOOTSTRAP_SERVERS for instance).    In the end I decided it were simpler to assume that the feature would be used wisely.


Aqara E1 thermostat external temperature sensor

Proposed change
Support for external temperature sensor.
Additional information
This is a copy of the not merged part from #2109. Discussion starts under #2687

Checklist

[x] The changes are tested and work correctly
[x] pre-commit checks pass / the code has been formatted using Black
[ ] Tests have been added to verify that the new code works

What is the status here? I could help implement or test, as I would also like to use this feature in the future :-)

Hi @fzob, feel free. The new cluster appears in the ZHA device tab and I can send command to it. But if I read them they are not changed. My guess is, that the message is not correct, but I didn‘t find the time to debug it.

Indents are wrong on line 403-567. So custom attributes weren't written. After this fix setting sensor and sensor_temp works!

@fzob I fixed it. Coverage could gave a hint on that 🤦 . Anyway, thanks for taking a look. Now, it also works for me.

@mb1248 I have added this quirk to my ZHA, now I am asking myself a very stupid question.
Is it correct that I am defining the external sensor by going to Manage Zigbee Device -> Cluster (In) -> Sensor, and write the IEEE of my temp sensor via the value?
Also, does it automatically use the external sensor temperature entity or do I still need an automation to push the temp entity of an external sensor into the SENSOR_TEMP entity?

Hi @Patrick-DE, you need to add an automation. The sensor value is either internal (Default/0) or external, meaning that the internal temperature sensor is either used or not. If an external sensor is selected, the temperature value has to be set manually. This works similarly to how the Aqara hub operates. Note that, due to compatibility with Z2M, the temperature value is in centi-degrees (c°C), so you have to multiply it by 100. (This seems awkward, and I'm considering changing it.) I also read that the sensor type is reset; however, I didn't observe this in my test (around 2 hours). Therefore,  the automation for Z2M, first, set the sensor type and then the temperature value.

Hi, I'm trying to add an automation based on this quirk. Unfortunately I'm stuck when trying to pass the value of the external sensor...
That's what I'm trying to do and it ends up with 0° in the TRV current temperature :
service: zha.set_zigbee_cluster_attribute
data:
  cluster_type: in
  ieee: 54:ef:44:10:00:73:cb:f6
  endpoint_id: 1
  cluster_id: 64704
  attribute: 5010
  value: "{{ states('sensor.cl_chp_temperature')|float|round(1) * 100 }}"

Would you have any idea?

you have to pass an integer, after the multiplication add |int and it should work

you have to pass an integer, after the multiplication add |int and it should work

service: zha.set_zigbee_cluster_attribute
data_template:
  cluster_type: in
  ieee: 54:ef:44:10:00:73:cb:f6
  endpoint_id: 1
  cluster_id: 64704
  attribute: 5010
  value: "{{ states('sensor.cl_chp_temperature')|float|round(1) * 100|int }}"

I someone wants to try your quirk that's how I made it work as a whole for one TRV, once I migrate the other 9, I'll just add some triggers and template up the ieee (or duplicate the actions if I can't generate the ieee in a template) :
alias: TH Mise à jour température vannes
description: Spécifique aux vannes Aqara SRTS-A01
trigger:
  - platform: state
    entity_id:
      - sensor.cl_chp_temperature
    id: parents
condition: []
action:
  - choose:
      - conditions:
          - condition: trigger
            id:
              - parents
            enabled: false
          - condition: template
            value_template: >-
              {{ (state_attr('climate.th_chp_thermostat',
              'current_temperature')|float -
              states('sensor.cl_chp_temperature')|float)|abs >= 0.1 }}
        sequence:
          - variables:
              sensor_temperature: >-
                {{ (states('sensor.cl_chp_temperature')|float|round(1) *
                100)|int }}
          - alias: Définir le capteur externe
            service: zha.set_zigbee_cluster_attribute
            data:
              cluster_type: in
              ieee: 54:ef:44:10:00:73:cb:f6
              endpoint_id: 1
              cluster_id: 64704
              attribute: 638
              value: "1"
            enabled: true
          - delay:
              hours: 0
              minutes: 0
              seconds: 3
              milliseconds: 0
            enabled: true
          - service: zha.set_zigbee_cluster_attribute
            data:
              cluster_type: in
              ieee: 54:ef:44:10:00:73:cb:f6
              endpoint_id: 1
              cluster_id: 64704
              attribute: 5010
              value: "{{ sensor_temperature }}"
            alias: Passer la température du capteur externe
mode: queued
max: 10

Unfortunately this does not work for me. Turning on the external sensor command does get completed by homeassistant, but the entity state keeps saying 'off'. Additionally, i get an error when the temperature needs to be sent.

Can you maybe make a blueprint for this?

Hi. Is there any chance this gets merged soon? Is this blocked only by testing? If that’s the case, I can maybe help.

@Antoninj I had no time to work on the test. The quirk is working. So only the testing is missing. Would be nice if you could help there.

Thanks, it works for me too!

Thanks for doing this. It works for me.

It seem to work for a few people here. When can the PR get resolved? Is there still stuff to do? Would love to use the feature. :)

Yes It would be  nice👍

Any update on this? It would be a nice feature to add 👍


Ability to Pickle RootModels

Initial Checks

[X] I confirm that I'm using Pydantic V2 installed directly from the main branch, or equivalent

Description
PyIceberg is used in some systems where Pydantic objects are being pickled. We don't always control the serialization framework, therefore we need to support this.
Example Code
Python 3.9.6 (default, May  7 2023, 23:32:44)
[Clang 14.0.3 (clang-14<IP_ADDRESS>.1)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from pydantic import RootModel, ConfigDict
>>> from typing import Generic, TypeVar
>>>
>>> T = TypeVar('T')
>>>
>>> class FrozenRootModel(RootModel[T], Generic[T]):
...     model_config = ConfigDict(frozen=True)
...
...
>>> FrozenRootModel('yes') == FrozenRootModel('yes')
True
>>> import pickle
>>> FrozenRootModel('yes') == pickle.loads(pickle.dumps(FrozenRootModel('yes')))
False

### Python, Pydantic & OS Version

```Text
>>> import pydantic.version; print(pydantic.version.version_info())
             pydantic version: 2.0.1
        pydantic-core version: 2.0.2 release build profile
                 install path: /Users/fokkodriesprong/Library/Python/3.9/lib/python/site-packages/pydantic
               python version: 3.9.6 (default, May  7 2023, 23:32:44)  [Clang 14.0.3 (clang-14<IP_ADDRESS>.1)]
                     platform: macOS-13.4.1-arm64-arm-64bit
     optional deps. installed: ['typing-extensions']

Selected Assignee: @Kludex

It seems to me that the False arises because after the pickle round-trip, the __dict__ picks up some extra members. (Which leads to False being returned from BaseModel.__eq__.)
>>> a = FrozenRootModel('yes')
>>> b = pickle.loads(pickle.dumps(a))
>>> a.__dict__
{'root': 'yes'}
>>> b.__dict__
{'root': 'yes', '__pydantic_extra__': None, '__pydantic_private__': None}


[RUM-4594] Increase upload timeout to 120s to match EvP limits

What does this PR do?
This PR increases the upload timeout limit from 60s to 120s in all DCs. This change was made in order to support customers with slower upload speeds.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 80.43%. Comparing base (9834378) to head (3ca03e6).
Report is 1 commits behind head on develop.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop     #261   +/-   ##
==========================================
  Coverage      80.43%   80.43%
  Complexity       209      209
==========================================
  Files             26       26
  Lines            828      828
  Branches         110      110
==========================================
  Hits             666      666
  Misses           116      116
  Partials          46       46

Files
Coverage Δ

...m/datadog/gradle/plugin/internal/OkHttpUploader.kt
78.20% <100.00%> (ø)

... and 2 files with indirect coverage changes


Websockets support

Hello !
Thank you for this project, it is really useful :)
I saw there is some websockets support on the branch - I was wondering what would it take to get it to master ?
thank you !

the websockets version in the branch is not under development, it is ready to use. I did not want to mix all the versions in the master so I separated using branches.


RDKTV-12823: Option to set individual 'memorypressure' configs for Sa…

…D, RA, HtmlApp and LightningApp
Reason for change: Option to set individual 'memorypressure'
configs for SaD, RA, HtmlApp and LightningApp
Test Procedure: Refer ticket.
Risk:Low
Signed-off-by: Adrian Muzyka<EMAIL_ADDRESS>

Could you please set defaults in the CMakeLists.txt. default to ${PLUGIN_WEBKITBROWSER_MEMORYPRESSURE}


add sort botton to the feed page

Sort feeds by a given order.
Solved #632
This is intentional done on the client side. No need to send requests to the server.

I think the web ui needs some work to improve the user experience:

The sort criteria appears only when clicking on "Sort By". The first time I used it, I thought it was broken, but I realized that I have to click on the link.
When you click on the sort criteria, the end-user doesn't know the direction: ascending or descending.

I can see that you have implemented the sorting in Javascript in the web ui, I would have preferred having this on the server side...

What is your reason to do the sorting on the server side?
I am doing it on the client side because I believe it would be more responsive, especially when you need to sort feeds many times using different criterias.
My intentional use case is to help user (i.e. me) find feeds that

Inactive (considering unsubscribe)
Update a lot, but not read many (considering unsubscribe or moving to the unimportant folder)
Not update often, but read many (considering moving to the favorite folder)
May be more, these are what I am thinking now.

These require more than 1 sorts.
So I want the sort function

Responsive
Minimize transaction between server and client
Easy to reach and select criteria (on the same feeds page)

Doing the sorting on the client saves the computing power on the server and the bandwidth between the server and the client. We also do not need to  worry about the latency between the server and the client and the page refreshing delay on the client.
All data required for sorting are already available on the client side. And I think most clients today have enough computing power for sorting.
In issue #699, it is mentioned that it takes 2 seconds to load the feeds page. If we do sorting on the client side, we save the 2 seconds on every sort.
Maybe "Sort by" is a bad wording? I want to say: clicking this to take the sorting action. It is not a setting for user.
Last but not least, sorting on the client does not conflict with issue #697.
It is not done in this commit. But we could allow user to set a default order of feeds (on the setting page, not on the feeds page). Then call the corresponding JavaScript sorting function on page loading.
On the server side we choose the order which is fastest for query. (Or not to choose any order? The query could be even faster?)

It is inspired by "Organize Sources" in Feedly.

The scenario I can think when sorting on server is useful is that people want to get the sorted feed list from an API. And the client cannot sort the list. The sorting criteria should also be specified by the API.

@shizunge Miniflux is created to be minimalist, simple and modern. Also it aims to use JS as little as possible to work at any devices, even super slowest.

@L11R I do not think minimalist means to stop adding new features. I did not change the any default view of Miniflux. So I do not think I broke anything here.
We should definitely consider slow clients. So the javascript I added only run on demand. On the other hand, should we consider slow server and slow network as well? Unless we benchmark them, it is hard to say which is slower: Send requests to the server and load the entire page, or running a javascript on the client. Whether we should implement a feature on client side or on the server side, it depends on the use case. In my previous reply https://github.com/miniflux/miniflux/pull/711#issuecomment-650499046, I have explained my use case. It seems better to do it on the client side to me. And it does not conflict with further improvement (i.e. setting the default order of feeds) on the server side.

I agree with @fguillot that this change would be most benefical if done server-side.
For the matter, my instance with 21 feeds, costs me 15kb on a fresh load and 3kb on a cached reload
If taking about from a slow server perspective, I live in the south of Brasil and my server is a Linode VPS at
Atlanta, US.
Shoving down that on the user, when the perspective of the project is to have a light,
minimalistic approach to things, sounds strange.

"think most clients today have enough computing power for sorting"

If the user have 200 feeds on it's list and an old mobile phone, he doesn't have as well. So why not
deliver that job to the server, as It sits there all day long doing nothing but checking and serving assets?

I'm using the paid version and right now I don't see a way to sort anything. Then again, I'm also using a screen reader. I too would rather have my feeds sorted by unread count permanently. Mine's already slow so I'm not too worried.


Keybinding not consistent. Works one way through the command palette, different through the keybinding

I'm trying to invoke a command from the atom-turbo-javascript package:

End New Line CTRL-ENTER
Terminates the current line with a colon or semicolon, followed with a new line. A comma is inserted when the cursor is inside an object literal, otherwise a semicolon is inserted.

I want the keybinding to be ⌘+⇧+⏎ and so in my keymap.cson I have:
'atom-text-editor':
    'cmd-shift-;': 'turbo-javascript:end-line'
    'cmd-shift-,': 'turbo-javascript:end-line-comma'
    'cmd-shift-enter': 'turbo-javascript:end-new-line'

Now, if I use cmd-shift-enter I get (i guess) the core functionality: a newline above the caret line. But if I search the same command in the command-palette, I see that it correctly has cmd-shift-enter and executing through the palette, it works!

So using the same shortcut the palette acknowledges directly, it gives me a new line above, not the turbo-javascript functionality. (in shift-cmd-enter order also doesn't work).

Have you tried using the Keybinding Resolver (Cmd+. on OS X and Ctrl+. on other platforms)? It will show you what key Atom saw you press and the command that was triggered because of it.
To answer your implicit question about the Command Palette, the Command Palette lists the mapped keys. But since it doesn't have information about all the other mapped keys for wherever focus is at the moment you press the key, it can't predict all possible situations where the key may or may not apply. That's what the Keybinding Resolver is for, to help debug these things.

I didn't know that existed ! Very useful, very worth while to mention it in the Keybinding settings or even in the keymap.cson
This is what I get:

I thought your keymap.cson always have precedence over. At least in these docs it says:

By default, ~/.atom/keymap.cson is loaded when Atom is started. It will always be loaded last, giving you the chance to override bindings that are defined by Atom’s core keymaps or third-party packages.

So... why is it not overriding that command and what's the best course to fix this?

The Keybinding Resolver is mentioned in the Debugging section of the docs.
Keybindings use a CSS-like selector mechanism that is based on specificity rules. The reason why editor:newline-above is winning over your mapping (even though your mapping is loaded last) is because its selector is more specific. If you change your mapping to:
'atom-workspace atom-text-editor:not([mini])':
    'cmd-shift-;': 'turbo-javascript:end-line'
    'cmd-shift-,': 'turbo-javascript:end-line-comma'
    'cmd-shift-enter': 'turbo-javascript:end-new-line'

then your version of Cmd+Shift+Enter will win. This is also covered in the documentation.

@lee-dohm I know (i searched for it when you mentioned it) it is in the debugging section, but it's too useful not to be mentioned at least in the keymap.cson where there's a huge block of text not even mentioning keybinding debugging.
Don't take it the wrong way, just my two cents. I really think it should be there giving how easy it is to not have the keybindings work the way they say they should work.
I'll fix my specificity, thanks.

Fair enough :grinning: Added a PR.


Fix use of undeclared identifier __cpuidex error on MinGW

This change requires the target to be msvc in order to reference the method __cpuidex.  The reasoning is due to a breaking change that appears to have come into effect recently across all major c / c++ compilers.  Details are as follows:
LLVM, Zig (which uses LLVM under the hood), and GCC all recently updated their compilers changing what was a warning to an error.  The compiler will now error if attempting to reference __cpuidex and not targetting msvc.  It will no longer work for MINGW targets.
Here are relevant docs / source code for reference
Discovered by @Rexicon226, here is the LLVM code and how it handles it:
https://github.com/llvm/llvm-project/blob/main/llvm/lib/Support/BLAKE3/blake3_dispatch.c#L46-L60
Another helpful zig dev who's discord username is "random internet person" found: https://github.com/ziglang/zig/blob/master/lib/zig.h#L3902-L3913
Which appears similar to LLVM's new implementation.
And GNU posted a more formal announcement notifying of the warn to error change that appears to have propagated across all major compilers recently: https://gcc.gnu.org/gcc-14/porting_to.html#warnings-as-errors
Shoutout to @paperdave for all the help and guidance along the way! :)

For further clarification.  The warning-to-error is not specific to __cpuidex.  That is one of the effected methods.  Through conversation in the zig discord, I believe it has to do with no longer allowing implicit function definitions at compile time.  Fortunately this is the only affected spot I've found so far.

Thanks for the research and the fix. Some comments:

This should be reported / PR'ed upstream to https://github.com/RenderKit/embree. We can have a downstream patch temporarily but eventually this needs to be fixed in the original library.
I haven't looked into details yet, but would it work to use the #else (non _WIN32) branch, which I supposed is meant for GCC/Clang? It might work with MinGW GCC/Clang. So the fix would then be:

diff --git a/thirdparty/embree/common/sys/sysinfo.cpp b/thirdparty/embree/common/sys/sysinfo.cpp
index d01eab3c9d..4c5495c2ce 100644
--- a/thirdparty/embree/common/sys/sysinfo.cpp
+++ b/thirdparty/embree/common/sys/sysinfo.cpp
@@ -293,11 +293,8 @@ namespace embree
     int cpuid_leaf_7[4] = { 0,0,0,0 };
     int cpuid_leaf_e1[4] = { 0,0,0,0 };
     if (nIds >= 1) __cpuid (cpuid_leaf_1,0x00000001);
-#if _WIN32
-#if _MSC_VER && (_MSC_FULL_VER < 160040219)
-#else
+#if _MSC_VER && (_MSC_FULL_VER >= 160040219)
     if (nIds >= 7) __cpuidex(cpuid_leaf_7,0x00000007,0);
-#endif
 #else
     if (nIds >= 7) __cpuid_count(cpuid_leaf_7,0x00000007,0);
 #endif

Thanks for the research and the fix. Some comments:
* This should be reported / PR'ed upstream to https://github.com/RenderKit/embree. We can have a downstream patch temporarily but eventually this needs to be fixed in the original library.

* I haven't looked into details yet, but would it work to use the `#else` (non `_WIN32`) branch, which I supposed is meant for GCC/Clang? It might work with MinGW GCC/Clang. So the fix would then be:

diff --git a/thirdparty/embree/common/sys/sysinfo.cpp b/thirdparty/embree/common/sys/sysinfo.cpp
index d01eab3c9d..4c5495c2ce 100644
--- a/thirdparty/embree/common/sys/sysinfo.cpp
+++ b/thirdparty/embree/common/sys/sysinfo.cpp
@@ -293,11 +293,8 @@ namespace embree
     int cpuid_leaf_7[4] = { 0,0,0,0 };
     int cpuid_leaf_e1[4] = { 0,0,0,0 };
     if (nIds >= 1) __cpuid (cpuid_leaf_1,0x00000001);
-#if _WIN32
-#if _MSC_VER && (_MSC_FULL_VER < 160040219)
-#else
+#if _MSC_VER && (_MSC_FULL_VER >= 160040219)
     if (nIds >= 7) __cpuidex(cpuid_leaf_7,0x00000007,0);
-#endif
 #else
     if (nIds >= 7) __cpuid_count(cpuid_leaf_7,0x00000007,0);
 #endif

I will try and see if that works

I can confirm that it does not like that.  It errors with:
[ 29%] thirdparty/embree/common/sys/sysinfo.cpp:299:59: error: too few arguments provided to function-like macro invocation
  299 |     if (nIds >= 7) __cpuid_count(cpuid_leaf_7,0x00000007,0);
      |                                                           ^
/home/tcroc/dev/BlockyBallOT/submodules/godot-src/gitignore/zig/0.13.0-dev.267+793f820b3/zig-linux-x86_64-0.13.0-dev.267+793f820b3/lib/include/cpuid.h:265:9: note: macro '__cpuid_count' defined here
  265 | #define __cpuid_count(__leaf, __count, __eax, __ebx, __ecx, __edx) \
      |         ^
thirdparty/embree/common/sys/sysinfo.cpp:299:20: error: use of undeclared identifier '__cpuid_count'
  299 |     if (nIds >= 7) __cpuid_count(cpuid_leaf_7,0x00000007,0);
      |                    ^
[ 30%] 2 errors generated.

Anything else you would like me to try or should we leave it as is?  Should I put a PR upstream to see if they have any feedback?

Should I put a PR upstream to see if they have any feedback?

Yes, please do. If they're reactive, they should point to the proper / best solution for their codebase.

Should I put a PR upstream to see if they have any feedback?

Yes, please do. If they're reactive, they should point to the proper / best solution for their codebase.

Perfect!  Then we can replace our patch with their fix assuming its appropriate for our codebase as well :)

Put in an upstream PR over here: https://github.com/RenderKit/embree/pull/487

Could you also include the patch we made to thirdparty code as a patch file in the same commit?
I.e. thirdparty/embree/patches/mingw-no-cpuidex.patch:
diff --git a/thirdparty/embree/common/sys/sysinfo.cpp b/thirdparty/embree/common/sys/sysinfo.cpp
index d01eab3c9d..4ecab05265 100644
--- a/thirdparty/embree/common/sys/sysinfo.cpp
+++ b/thirdparty/embree/common/sys/sysinfo.cpp
@@ -295,7 +295,7 @@ namespace embree
     if (nIds >= 1) __cpuid (cpuid_leaf_1,0x00000001);
 #if _WIN32
 #if _MSC_VER && (_MSC_FULL_VER < 160040219)
-#else
+#elif defined(_MSC_VER)
     if (nIds >= 7) __cpuidex(cpuid_leaf_7,0x00000007,0);
 #endif
 #else

This is how we document (and reapply on update) the downstream changes we made.

@akien-mga Got distracted by some things on our game.  I'll look into that this week.

@akien-mga I added the patch file :)

Thanks!


Document how to setup drupalvm to be used for core tests

I was just thinking about how awesome this project is for core development. It would be great if there was a default test.config.yml or some documentation that can be used for the sole purpose of a local core drupal testbot for drupal contribution.

@frob - I generally do core dev within Drupal VM (it's especially handy to bring up a fresh new instance, hack on a patch, run tests, then destroy the instance as needed), and I fully support the practice.
However, there's already a fairly well-documented (though dauntingly complex) way to build the exact testbot infrastructure locally: https://www.drupal.org/automated-testing
I'd rather not try to divert people who want to do that away, but rather leave people to do their own configuration and figure out their own optimal workflow for tests, whether it be a container to run the tests in, using drupal.org's issue queues, or doing it all inside Drupal VM.

See also:

https://www.drupal.org/project/testbot
https://www.drupal.org/project/drupalci
https://www.drupal.org/node/2463169


Additional joins are ignored inside an aggregate block with all condition

I'm creating a dataset which needs to consider all available data with multiple join conditions.
The additional joins work fine when the join condition is set to before or after but is completely ignored in the all condition.

Given the syntax here,

aggregate all updated_working_hours (
    min(working_date) as next_initial_eval
    filter {{dbt_aql.json_extract('{feature_json}', 'working_date')}}::date > current_timestamp::date
    filter {{dbt_aql.json_extract('{feature_json}', 'available_intake_appointments')}}::float > 0.0
    join {{ dbt_aql.json_extract('{joined}.{feature_json}', 'clinician_id') }} = {primary}.clinician_id
    join {{ dbt_aql.json_extract('{joined}.{feature_json}', 'territory') }} = {primary}.territory
)

The generated code looks like this,

joined__aggregate__all__updated_working_hours__4 as (
    -- special join case for aggregate all to improve performance
    select
        j.clinician_id,
sum(cast(nullif(json_extract_path_text(j.attributes, 'current_availability'), '') as FLOAT)) as availability_7_days,
sum(cast(nullif(json_extract_path_text(j.attributes, 'hours_booked'), '') as FLOAT)) as hours_booked_7_days
    from "postgres"."dbt_rukshan"."clinician_stream" j
    where j.activity = 'updated working hours'
        and json_extract_path_text(attributes, 'working_date')::date > current_timestamp::date
        and json_extract_path_text(attributes, 'working_date')::date <= (current_timestamp::date + INTERVAL '7 day')::date
    group by 1
)

However, when the join condition is after the generated code is,

joined__aggregate__after__updated_working_hours__4 as (
    select
        p.req__activity_id,
sum(cast(nullif(json_extract_path_text(j.attributes, 'current_availability'), '') as FLOAT)) as availability_7_days
    from primary__all__updated_working_hours__None__1 as p
    left join "postgres"."dbt_rukshan"."clinician_stream" j
        -- filter joined activity first to improve query performance
        on j.activity = 'updated working hours'
        and p.req__clinician_id = j.clinician_id
        and p.req__activity_at < j.activity_at
        and json_extract_path_text(j.attributes, 'clinician_id') = p.clinician_id
        and json_extract_path_text(j.attributes, 'territory') = p.territory
        and json_extract_path_text(attributes, 'working_date')::date > current_timestamp::date
        and json_extract_path_text(attributes, 'working_date')::date <= (current_timestamp::date + INTERVAL '7 day')::date
    group by 1
)

Hey @r-u-k-i-y-a - thanks for posting about this. This behavior is actually by design, as aggregate_all relationships can bog down query performance for activities that occur with high frequency. There's documentation about it here.
Can you provide a little more context on what business question you're trying to answer? I see that you're building a dataset that contains 1 row for each updated_working_hours activity occurrence, but knowing what business question(s) that dataset will answer will be useful context to figure out an alternate resolution.

Thanks for the quick response @bcodell and apologies for taking so long to reply.
My requirement was to aggregate a certain column (for example lets say daily state sales) to create a weekly state sales column and append it to each row of the dataset. Since I needed all data from the main activity with additional joins on state, I expected the join feature to work when all occurrences were selected as well.
For a solution, I was able to cover this up with some raw SQL on the dataset so all is good!

Thanks for the update! Apologies for the delayed reply. This is exactly why I built this project with dbt - when a use case arises that is outside the scope of what this project does, the developer can always fall back on raw SQL. I'm going to close this issue for now - feel free to re-open if there's anything else you want addressed.


Workaround finding correct local address

This fixes spring-cloud/spring-cloud-dataflow-admin-yarn#23 and closes spring-cloud/spring-cloud-dataflow-admin-yarn#24
Until we get SHDP-546 fixed, override yarnAmTrackservice bean
for appmasters by re-implementing EmbeddedAppmasterTrackService
from yarn boot to be able to use custom function favour of
NetworkUtils.getDefaultAddress(). This bean is responsible for
registering embedded boot endpoint address which is then used
to communicated with appmaster.

btw, EmbeddedAppmasterTrackService is copied twice to both appmasters. Tried to add it to common package but then got deps hell from from hadoop/boot and things went bad. This commit would anyway be reverted when bits are fixed in SHDP.

I tried few streams with this fix and I don't see the timeout errors described in spring-cloud/spring-cloud-dataflow-admin-yarn#24. The streams successfully deploy when connected to VPN.
Just to double check, let's also have @trisberg review the code changes of this temporary workaround. O'wise, LGTM.

Merged via commit 64e018544bf4bf2aa32fa0c943ee5d04bf4be458


photo

requested icon
photo

category suggestion
item
search synonyms
photograph, picture, shot, snap, image, camera-roll

released


Enable Python 3.12 builds

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

I will go ahead with merging since the change it trivial and all builds just pass 😄


Fix rendering for elasticsearch visibility storage

What was changed
To specify Elasticsearch as only visibility storage, the value server.config.persistence.visibility.driver should be set to elasticsearch.
This omits configuring Cassandra as visibility storage coming from default values.
Why?
When Cassandra was disabled and external Postgres was used as default storage and external Elasticsearch for visibility storage, helm failed to install because created inconsistent resources and configuration.
Checklist

Closes

How was this tested:

Any docs updates needed?

Fixed by #522


[Storage] Name props on clients blob - Provide better error messages and some tests

Looks good.

Hi, @HarshaNalluru
FYI. There could be emulator case where URL could be in format: http://<IP_ADDRESS>:10000/devstoreaccount1/sascontainer

Please double check if emulator support is essential for the optimized functions.

FYI. There could be emulator case where URL could be in format: http://<IP_ADDRESS>:10000/devstoreaccount1/sascontainer
Please double-check if emulator support is essential for the optimized functions.

Good point. Creating an issue for this. https://github.com/Azure/azure-sdk-for-js/issues/5446
Will either do tomorrow or post preview 4.


Rely on remote image to fix breaking bug

Before this PR
The compose file relied on a local image
After this PR
The compose file relies on the remote dockerhub version

Going to merge without review, because it's critical.


How to import functions.sass to create custom colors?

This is about Bulvar.

Overview of the problem

I am trying to add custom colours and can't find a way to import functions.sass in order to access findColorInvert function.
Description

When trying this code:
@charset "utf-8";

@use<EMAIL_ADDRESS>as fn;

$antidote-blue: #00435C;
$antidote-blue-light: fn.findColorInvert($antidote-blue);

// @use<EMAIL_ADDRESS>@use<EMAIL_ADDRESS>with (
    $primary: #E56F56,
    $success: #66C87D,
    $custom-colors: ("antidote-blue": ($antidote-blue, $antidote-blue-light))
);

I get this error:
File<EMAIL_ADDRESS>  Error: This module was already loaded, so it can't be configured using "with".
     ┌──><EMAIL_ADDRESS>  3  │   @forward "initial-variables"
     │   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ new load
     ╵
     ┌──><EMAIL_ADDRESS>  8  │   @use 'initial-variables' as vars
     │   ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ original load
     ╵
     ┌──> src/index.scss
  9  │ ┌ @use<EMAIL_ADDRESS>with (
  10 │ │     $primary: #E56F56,
  11 │ │     $success: #66C87D,
  12 │ │     $custom-colors: ("antidote-blue": ($antidote-blue, $antidote-blue-light))
  13 │ │ );
     │ └─^ configuration
     ╵
<EMAIL_ADDRESS>3:1  @forward
<EMAIL_ADDRESS>6:1                             @use
    src/index.scss 9:1                                                       root stylesheet

I tried to use @forward but got some errors too.
How would one import the file functions.scss in order to use the defined function in its code?

@MangelMaxime you don't have to look for invert colors (or any other) manually, mergeColorMaps function does it automatically.
So instead use just pass $custom-colors: ("antidote-blue": ($antidote-blue)

But I'll remove derived variables reference from the functions anyway. Thank you, good catch

Oh okay, I didn't go into this function implementation indeed.

I just tried in my application to pass $custom-colors: ("antidote-blue": $antidote-blue) however this doesn't generate/set the inverted color variable.
$antidote-blue-primary: #05435C;

@use<EMAIL_ADDRESS>with (
    $custom-colors: (
        "antidote-blue-primary": $antidote-blue-primary
    )
);

@use<EMAIL_ADDRESS>

@MangelMaxime  that should be fixed in new major release

@daniil4udo I confirm that version 0.11.1 fixed this bug thank you

Thank you @MangelMaxime


Support for np.fft.fft, np.fft.ifft etc.

Feature request
It would be amazing if numba would support the FFT pack of numpy. Personally, I would be interested in
   np.fft.fft
   np.fft.ifft
   np.fft.rfft
   np.fft.irfft

Obviously, there are many applications in signal processing that could benefit from this.
Are there any plans for that?

Thanks for the request. This would be great to have, the reason it hasn't been done is that it would rely upon:

Some package, e.g. SciPy providing a consistent means of access. Numba uses scipy.linalg.cython_blas and scipy.linalg.cython_lapack as a means of obtaining the addresses of BLAS and LAPACK functions so as to implement numpy.linalg.*, similar would be needed for FFT functions.
There would need to be a "consistent" interface (or subset of an interface) for accessing and using the functions that do the FFT work (e.g. FFTW vs MKL FFT vs ... something else). This would likely be more of a challenge to overcome than exposing a raw binding. From memory, the interfaces are quite involved and to get good performance require things like the caching of plans.

Hi @stuartarchibald ,  Is there any progress or workaround? want to release GIL while applying STFT to lots of audios, thanks.

@vzxxbacq There's no update I'm afraid. May be worth checking if the Python wrappers around FFT libraries you want to use are holding the GIL and raise issues with them?

Is there a workaround or any update on this? Just want to chime in and say I'd find fft support very valuable.

I stumbled upon this thread having the same problem everyone else has: I want to use fft's routines in my numba functions.
What I found is the following (maybe useful for others). If the fft is only a small part of the function, object mode can be used as a good workaround.
A sketch of code would look like:
@njit
def myfunc(input)
     .....
     .....
     with objmode(out='complex128[:]'):
            out = np.fft.fft(in)
            # Maybe also out = np.fft.fftshift(out)
     .....
     # heavy computations
     return ...

In my specific case, this is still significantly faster than a pure implementation. Of course, it would be great to have first-class support for these functions.

Why not build and implementation of fft/ifft within numba itself so that the issues with outside dependencies are gone?

Why not build and implementation of fft/ifft within numba itself so that the issues with outside dependencies are gone?

It'd be very hard to write something that's competitive with highly tuned and optimised FFT libraries, e.g. MKL FFT or FFTW.

@stuartarchibald  That makes sense.  But, batch fft is a common operation.  What is the recommended way to use ffts with numba?

@stuartarchibald That makes sense. But, batch fft is a common operation. What is the recommended way to use ffts with numba?

Probably like this: https://github.com/numba/numba/issues/5864#issuecomment-690838747 ? The chances are your FFT operations are expensive and run multithreaded, the cost of jumping back into the interpreter for doing this is probably small in comparison to the amount of work.

@stuartarchibald I am doing a lot of small FFTs on overlapped time windows.  I have implemented the entire code in CUDA but am trying to create a "teaching" code in python.  I suppose there is another way to write the code to batch this out but I'd prefer to keep the code as clean and readable as possible.

Re-posting from gitter https://gitter.im/numba/numba?at=6347db962a06f4566b341e00
It's worth looking at how JAX provide fft.

It'd be very hard to write something that's competitive with highly tuned and optimised FFT libraries, e.g. MKL FFT or FFTW.

I don't think that is true anymore. FFTPACK, which had very poor performance, is gone. pypocketfft )written in C++) has performance that's competitive. It's used in scipy.fft; numpy.fft uses an older and slightly slower C version of the same package. Now that NumPy has C++ code in it, numpy.fft could be upgraded to the same version as SciPy uses.
So I guess what remains is that you need a cython_fft?

It'd be very hard to write something that's competitive with highly tuned and optimised FFT libraries, e.g. MKL FFT or FFTW.

I don't think that is true anymore. FFTPACK, which had very poor performance, is gone. pypocketfft )written in C++) has performance that's competitive. It's used in scipy.fft; numpy.fft uses an older and slightly slower C version of the same package. Now that NumPy has C++ code in it, numpy.fft could be upgraded to the same version as SciPy uses.

Thanks for the update @rgommers, seems like parts of the original issue may not be a problem any more.

So I guess what remains is that you need a cython_fft?

Yes, I think cython or ctypes bindings to the same C/C++ functions that NumPy calls would likely be most/all of what is needed to implement this.

I'd probably want to treat that the same way as linalg: have scipy.fft provide a superset of functionality compared to numpy.fft, using the same pypocketfft C++ code under the hood. And then have a scipy.fft.cython_fft rather than do that in NumPy

I'd probably want to treat that the same way as linalg: have scipy.fft provide a superset of functionality compared to numpy.fft, using the same pypocketfft C++ code under the hood. And then have a scipy.fft.cython_fft rather than do that in NumPy

This seems like a good approach in terms of what would be needed with respect to the different functionality in NumPy vs. SciPy and is consistent with the existing cython exports (BLAS/LAPACK and scipy.special). The only immediate issue I can foresee with adding this to SciPy would be that it increases Numba's reliance on SciPy. Numba currently uses the SciPy BLAS/LAPACK cython bindings to provide the routines needed for numpy.linalg, however, recent experiments have indicated that this could be replaced by finding the same functions in process from NumPy libraries. IIRC most of the reason for exploring reducing Numba's reliance on SciPy is because there are (somewhat rare) occasions when SciPy is not available, i.e. packages not built yet, or not available for some arch/OS etc. I'll raise this at the next Numba public meeting.

I implemented numpy.fft and scipy.fft with Numba using the C++ PocketFFT library some time ago (https://github.com/styfenschaer/rocket-fft). It is now also on PyPi, but the installation does not always work properly. Any help with proper distribution is highly appreciated. Unfortunately, I'm not experienced enough at it. It could also serve as inspiration if the Numba development team plans to seriously support the two modules.

I am trying to make this threat active again.
I am currently writing an implementation for Particle Image Velocimetry that uses the wrappers of @styfenschaer. They work very well and for larger problems in my application result in a 5x speedup.
It would make sense to integrate these solutions in the main numba code for maintenance reasons. Is this possible?


Error in building pytorch-fairseq container

When running create_container.sh to build pytorch-fairseq container in fairseq_sagemaker_translate_de2en notebook, I got the following error.
It seems to be a pip related error that pip can't uninstall distutil PyYAML package.
Installing collected packages: jmespath, python-dateutil, docutils, botocore, s3transfer, boto3, protobuf, protobuf3-to-dict, texttable, docopt, idna, requests, websocket-client, docker-pycreds, pyasn1, pynacl, bcrypt, paramiko, docker, dockerpty, jsonschema, PyYAML, cached-property, docker-compose, sagemaker, Werkzeug, MarkupSafe, Jinja2, click, itsdangerous, flask, greenlet, gevent, gunicorn
  Found existing installation: idna 2.8
    Uninstalling idna-2.8:
      Successfully uninstalled idna-2.8
  Found existing installation: requests 2.21.0
    Uninstalling requests-2.21.0:
      Successfully uninstalled requests-2.21.0
  Found existing installation: PyYAML 5.1
Cannot uninstall 'PyYAML'. It is a distutils installed project and thus we cannot accurately determine which files belong to it which would lead to only a partial uninstall.
The push refers to repository [666254511816.dkr.ecr.us-east-1.amazonaws.com/pytorch-fairseq]
b4531a324bf6: Preparing
9f880b0076a6: Preparing
6eb97e6149d0: Preparing
bf3aec866297: Preparing
6e5274f7f7e9: Preparing
8846a0005308: Preparing
cadf36f9989b: Preparing
03da9992e0d0: Preparing
0b3aca7c75ea: Preparing
297fd071ca2f: Preparing
2f0d1e8214b2: Preparing
7dd604ffa87f: Preparing
aa54c2bc1229: Preparing
cadf36f9989b: Waiting
03da9992e0d0: Waiting
0b3aca7c75ea: Waiting
297fd071ca2f: Waiting
8846a0005308: Waiting
2f0d1e8214b2: Waiting
7dd604ffa87f: Waiting
aa54c2bc1229: Waiting
WARNING! Using --password via the CLI is insecure. Use --password-stdin.
WARNING! Your password will be stored unencrypted in /home/ec2-user/.docker/config.json.
Configure a credential helper to remove this warning. See
https://docs.docker.com/engine/reference/commandline/login/#credentials-store

WARNING! Using --password via the CLI is insecure. Use --password-stdin.
WARNING! Your password will be stored unencrypted in /home/ec2-user/.docker/config.json.
Configure a credential helper to remove this warning. See
https://docs.docker.com/engine/reference/commandline/login/#credentials-store

The command '/bin/sh -c pip install sagemaker flask gevent gunicorn' returned a non-zero code: 1

To fix this, I think  --ignore-installed PyYAML should be added to Dockerfile.
https://github.com/awslabs/amazon-sagemaker-examples/blob/master/advanced_functionality/fairseq_translation/Dockerfile#L45

@smrmkt created a PR here: https://github.com/awslabs/amazon-sagemaker-examples/pull/728

Thanks for submitting this issue and the fix! The fix has been merged, so I will close this issue.


Add a link to talk demo

Tooting my own horn a little... thought this might be useful to include in the official docs? Let me know if not, of course 😄

Thanks @ehashman 🎸


Lock down docker port access to only the runner security group

Description
Alternative to https://github.com/npalm/terraform-aws-gitlab-runner/pull/139 as I dont think we need to expose the docker port to the world. Just to the gitlab runner instance and maybe the other docker machines?
Migrations required
NO

@npalm Does it make sense to make a similar change to the docker machine ssh security group rule? The docker machine instances only receive SSH connections from the runner right?

@bishtawi yes the ssh connection is used by docker-machine on the runner agent instance. Feel free to make the same change for this security rule


Add task list page template

This adds a Task list page to the common templates plugin, to make it easier for people to prototype task lists using the GOV.UK Prototype Kit.
See Decide what to do with the Prototype Kit Task List plugin.

@colinrotherham @stevenjmesser this can be merged and released now.
It bumps the major version of the plugin, as it now depends on v5 of govuk-frontend.

@frankieroberto Not sure if your thumbs up is for the way to install the previous version or adding extra info to the name, please let me know which. I've quickly mocked up some extra info next to the name, if that's helpful:

"for GOV.UK Frontend >=5.0.0"

or, slightly more compact, "for GOV.UK Frontend ≥5.0.0"

@frankieroberto Not sure if your thumbs up is for the way to install the previous version or adding extra info to the name, please let me know which. I've quickly mocked up some extra info next to the name, if that's helpful:

"for GOV.UK Frontend >=5.0.0"

Done. It's a big ugly, but clarity beats aesthetics here, right?! 😄


Testing: Add fixtures that don't use consistent snapshots

I believe this was fixed in #277.

Agreed!


IAT segment files

Codecov Report

Merging #591 into master will increase coverage by 0.05%.
The diff coverage is 91.52%.

@@            Coverage Diff            @@
##           master    #591      +/-   ##
=========================================
+ Coverage   91.64%   91.7%   +0.05%
=========================================
  Files          64      64
  Lines        5329    5390      +61
=========================================
+ Hits         4884    4943      +59
- Misses        300     302       +2
  Partials      145     145

This looks great @bkmoovio! I'm glad to see it was straightforward to support IAT.


NXLog DNS Logs data connector: Initial Commit

Initial submission of the NXLog DNS Logs data connector

Connector UX: NXLogDnsLogs.json
Sample Data:  NXLogDNS_Logs_CL.json
Logo:         NXLog.svg

@techwriter-dev - Please have a look at the comment on fixing the document link validation issue.  @nazang - can you have a look at the specific data connector pieces?

@v-jayakal, please review the connector.

@techwriter-dev - Please have a look at the comment on fixing the document link validation issue. @nazang - can you have a look at the specific data connector pieces?

@shainw The data connector NXLogDnsLogs.json was updated in commit ded24573. Thanks!

@techwriter-dev - Please have a look at the comment on fixing the document link validation issue. @nazang - can you have a look at the specific data connector pieces?

@nazang I took a look at the DocumentsLinkValidation failures for lines 22 and 23. I can only assume it is related to the original filename I chose
Sample Data/Custom/NXLogDNS_Logs_CL.json which I changed to
Sample Data/Custom/DNS_Logs_CL.json in commit 5ef155728adc67c9249c2bfb38b1bcd448faecf5.

Closing this pull request to avoid duplication as https://github.com/Azure/Azure-Sentinel/pull/1280 has the files for NxLog DNS Logs data connector.

@v-jayakal Created Pull Request 1394 on forked branch nxlog-dns-logs-rev1 in order to move forward with this data connector.

@v-jayakal Please disregard my last comment. I see that the DNS Logs connector was merged into master.
I closed PR 1394.
Thanks!


Start signing pom files in Gradle Plugin portal

Start signing pom files in Gradle Plugin portal.
Today, none of the pom files in Gradle Plugin portal have .asc files associated with them even if the rest of the artifact is signed (e.g. jars). Examples of these include gradle-enterprise-gradle-plugin (as of 3.9), common-custom-user-data-gradle-plugin (as of 1.6.5)

We are soon releasing version 1.0.0 of the Plugin Publish Plugin, which will properly do this signing (all you will need to do is apply the signing plugin, the rest will happen automatically). We already have 1.0.0-rc-1 out, but it's still being tested: https://plugins.gradle.org/plugin/com.gradle.plugin-publish/1.0.0-rc-1. Feel free to try it, some preliminary documentation is here: https://plugins.grdev.net/docs/publish-plugin

Thanks
------------------ 原始邮件 ------------------
发件人:                                                                                "gradle/gradle"                                                                                @.>;
发送时间: 2022年4月6日(星期三) 下午4:27
@.>;
@.***>;
主题: Re: [gradle/gradle] Start signing pom files in Gradle Plugin portal (Issue #20349)
We are soon releasing version 1.0.0 of the Plugin Publish Plugin, which will properly do this signing (all you will need to do is apply the signing plugin, the rest will happen automatically). We already have 1.0.0-rc-1 out, but it's still being tested: https://plugins.gradle.org/plugin/com.gradle.plugin-publish/1.0.0-rc-1. Feel free to try it, some preliminary documentation is here: https://plugins.grdev.net/docs/publish-plugin
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


Documentation on using Redis Brain

Is there some more detailed documentation on using the Hubot redis brain effectively, or good practices for it?
For example, the scripting docs (https://github.com/github/hubot/blob/f672bc03cf4bf415ce45ed60c38334d988fae8f4/docs/scripting.md) simply talk about storing and retrieving key-value strings.
However, if you look at scripts like this:
https://github.com/hubot-scripts/hubot-team/blob/4a811e2f1881e85848c5801f5ff8b665f09390fa/src/models/team.coffee
They use more slightly more complex structures (arrays, dicts etc.) - I assume these just get stringified, then stored in redis:
    teams = []
    for key, teamData of @store()
      continue if key is @defaultName()
      teams.push new Team(teamData.name, teamData.members)
    teams

They also seem to namespace things as follows (which I'd hadn't realised you could do until I saw this):
    @robot.brain.data.teams or= {}

It'd be nice if there were more examples of the redis-brain in action, or even just pointers to good example scripts where we can see good practices for these things?

OK, I take that back, I don't get how the namespacing thing works above...

I'm also a little unclear of the significance of the data in robot.brain.data - e.g.:
  isNotified = (keyword, username) ->
    robot.brain.data[keyword] ?= []
    robot.brain.data[keyword].some (x) -> username == x
    ...
    unless isNotified("invite", username)
      replyTo(username, "Are you looking for an atom invite?  Go to https://atom.io/ and put in your email, or join the #product-invitations room.")
      robot.brain.data.invite.push username

http://theprogrammingbutler.com/blog/archives/2014/02/28/robots-can-do-support-too/

In terms of "best practices", they are pretty lacking beyond what is in the scripting docs. Namespace your stuff so that you don't conflict with other scripts and use the key-value methods (get/set). Everything is persisted via JSON.stringify. I know @technicalpickles has talked about rewriting the brain, but I don't know what the direction was.

Hmm, but if you use get/set, how do you store more complex data structures?
For example, say you have an array:
{ 'tickets': [ { 'id': 'C-41', 'title': 'foo is broken', 'due_date': Date.now() }, { 'id': 'G-41', 'title': 'bar is working', 'due_date': Date.now() } ] }

How should you push/pop individual items to this, assuming it's been set in the redis brain?
 Or is there a better way, using get/set?

Given the way that persistence works, you can actually just maintain a pointer to an object in the brain, manipulate it all you want, and it will be persisted every saveInterval. So, in code:
myList = []
robot.brain.set('my-list', myList)
myList.append(1)
# robot.brain.get('my-list') == [1]


Fix typos in virtctl comments

What this PR does / why we need it:
Signed-off-by: Orel Misan<EMAIL_ADDRESS>Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/cc @EdDev

/ok-to-test


Merging from Dev-Repo

Commit: refer to  https://github.com/AppLozic/Applozic-iOS-SDK/commit/75b06c324a300d07646efb131e69e07e12ad7747

BUG: Conversation list show null for unread count field when login with same user in two devices
FIX : refer to f301f02208d932b637dde45e3b478314030303e3


Update version on matrix-reloaded

It would be nice if the version on matrix-reloaded could be updated (likely once all the PE-fixit PRs have gone through) so that those of us at UCSB can make use of all the nice new features. Note that this may require an instruction guide (not in this repo) on how to deploy this on matrix-reloaded. Thanks!

@maffoo You briefly mentioned you wanted to do better in terms of deployment process for Scalabrad-web.
As the software stabilizes, a rapid deployment process is probably unnecessary. It would probably be good to decentralize the update process though, and perhaps have an update script (or similar) that will pull the latest files from master, or preferably, from a provided tag. It would then be possible in the future to use Ansible to call this updater to allow for automated deployment whenever a new release is made.


Alignment issue with form-select/floating label

I have a couple of issues showing up when using a form-select with floating labels inside a modal. Here's a screenshot of the issue.

The left side of the Product select control on the last row is slightly off compared to the left side of of all the other controls above it even though they have the same offset. IN addition the Product select is not as wide as the other inputs even though they have the same col specs.
If I move the Product select to appear on the row after the Location name input, both issues go away. I've attached a file of the complete html for the page.
BS5 issue.txt

This needs to be a reduced test case with a live demo, not a comprehensive text file that doesn't isolate the issue within our code base. I'm going to close this out for now, but please feel free to comment or open a new issue once you're able to provide that. Please read through our issue template for reporting bugs as well—it mentions all of this. Thanks!


feat(ssi): remove old ssi endpoints

Description

remove all CompanySsi related endpoints
remove all CompanySsi related database tables

Why
Since the new sis credential issuer was introduced the endpoints in the portal are obsolete.
Issue
Refs: #538
Checklist

[x] I have followed the contributing guidelines
[x] I have performed IP checks for added or updated 3rd party libraries
[x] I have created and linked IP issues or requested their creation by a committer
[x] I have performed a self-review of my own code
[x] I have successfully tested my changes locally
[x] I have added tests that prove my changes work
[x] I have checked that new and existing tests pass locally with my changes
[x] I have commented my code, particularly in hard-to-understand areas

@ntruchsess we should clarify with @jjeroch when this pr can get merged, since we do remove the data from the database as well


Cleanup for passing the linter

This PR does some linting  related cleanups:

Fetch Golangci-Lint using binary instead of compiling from source (golangci-lint docs advise against building from source)
Move some package level vars and consts from _test.go files to non _test.go files in test package.
Remove linting on some _conversion.go functions as they are infinitely recursive and cause false triggers.

/retest


Expose Pty resize and CloseStdin RPCs

Closes #691
This exposes the existing rpcs for Pty resize as well as adds a CloseStdin rpc for closing a containers STDIN.
I implemented pty resize in ctr for both run and exec so with -t it will properly resize to your terminal.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #702   +/-   ##
=======================================
  Coverage   62.73%   62.73%
=======================================
  Files           5        5
  Lines         687      687
=======================================
  Hits          431      431
  Misses        160      160
  Partials       96       96

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 e5c8c56...734bc1b. Read the comment docs.

ping @Random-Liu
I went with the approach of separate APIs for each function

@crosbymichael Cool, LGTM.

This doesn't build under windows now :stuck_out_tongue_winking_eye:

@mlaventure what do you mean?  The CI built and passed?

it was ran before #700 was merged

@mlaventure no, I rebased because of conflicts after that merge and the CI ran again, take a look

weird, it shouldn't build, windows doesn't have the 2 new methods, it fails on my machine:
root@f7eb519b3e27:/go/src/github.com/containerd/containerd# git log -1
commit a7da08b7ba1b2e10923d10622fbf20ebfbf5b300
Author: Michael Crosby<EMAIL_ADDRESS>Date:   Thu Apr 6 12:19:00 2017 -0700

    Add Pty and CloseStdin RPCs

    Signed-off-by: Michael Crosby<EMAIL_ADDRESS>root@f7eb519b3e27:/go/src/github.com/containerd/containerd# OS=Windows_NT GOOS=windows GOARCH=amd64 make
+ bin/ctr.exe
# github.com/containerd/containerd/windows
windows/runtime.go:136: cannot use ctr (type *container) as type containerd.Container in return argument:
	*container does not implement containerd.Container (missing CloseStdin method)
windows/runtime.go:140: impossible type assertion:
	*container does not implement containerd.Container (missing CloseStdin method)
windows/runtime.go:165: cannot use c (type *container) as type containerd.Container in append:
	*container does not implement containerd.Container (missing CloseStdin method)
Makefile:131: recipe for target 'bin/ctr' failed
make: *** [bin/ctr] Error 2
root@f7eb519b3e27:/go/src/github.com/containerd/containerd#

I can fix those later (my brain is refusing to properly concentrate atm)

OK, it looks like we will have to fix the windows CI if it is not catching this stuff

The commit it built is not the correct one.: 2b2e3f9 instead of a7da08b

ping @containerd/containerd-maintainers

LGTM


1.0.0.beta.2: type '_R.nib._LaunchScreen' does not conform to protocol 'NibResource'

R.generated.swift:159:12: error: type '_R.nib._LaunchScreen' does not conform to protocol 'NibResource'
    struct _LaunchScreen: NibResource {
           ^
Rswift.NibResource:3:16: note: protocol requires property 'instance' with type 'UINib'
    public var instance: UINib { get }

Close it. Because it seems I was using beta.2 binary with beta.1 library.

Yeah, sounds like the libraries were our out of date. Glad you solved it yourself! :)


prettier/plugin-pug

I'm trying to use coc-prettier with pug but it's not working.
my package.json has theses dependencies

"@prettier/plugin-pug": "^1.4.4",
"prettier": "^2.0.5",

I changed coc-settings.json to have:

"prettier.onlyUseLocalVersion": true

and ran the command inside a pug file

:CocCommand prettier.formatFile

and returned the error

pug not supported by prettier

It's expected since this plugin doesn't have support for set prettier plugins yet.

I won't take time to fix this for now, it would be appreciated if somebody want to help.

prettier/plugin-pug#151 should resolve this issue once it is published.

Make sure install prettier plugin with the prettier that coc-prettier is using.


A guide for creating swappable modules

Given a module A which has a dependency on module B, show how you can build module C which can be swapped with B.
Teaches:

How to use traits properly
Required Functions and Implemented Functions
Making a module have no direct dependency on another module
Building two modules which satisfy the conditions
Some live/practical example of swapping the two

I'll take this. Here's my plan in case anyone has feedback

Marketplace module that is basically like craigslist except that it depends on a notion of "reputation system".
Reputation system is a Trait
SimpleFeedback module implements Reputation Trait
BetaFeedback module implements Reputation Trait
Marketplace can work with either notion of feedback.

Two reputation systems I mentioned are taken from https://www-users.cs.umn.edu/~weiss039/papers/Trust-Grid2005.pdf

Sounds a little too complicated to me?
Why not keep it really bare bones. I would think something like a number modifier module, where one module adds to the number, the other subtracts from the number or something.
Something like a beginner and advance reputation system could work here, but making a "marketplace" module should not be an involved part of this process.

I feel like this could work well with module instancing. I don't fully understand instancing yet, but it seems that srml/balances and srml/collective allow you to create multiple instances of the same module with different configurations. So, for example, if you wanted multiple councils with different voting members and different weighting algorithms. You choose council invocation among the multiple councils based on the nature of voting topics.

Here's a start I've been toying with. Learned a lot along the way. https://github.com/joshorndorff/marketplace

Do you want / Is it ready for for a code review? How best to give you feedback?

@shawntabrizi Feedback welcome at any time. You can comment on my commits or open PRs. It isn't "ready" in the done sense, but there's plenty there. I used your simpler idea as an exercise in substrate collaborative learning today and that code lives https://github.com/JoshOrndorff/state-changer

I think the simpler one makes a strong foundation for a 1/2 page tutorial imo.
haven't reviewed either yet. will try to find time


Initial (WIP) implementation of --dns-propagation-time.

#474: this is an initial (WIP) implementation of allowing users to override DNS propagation time with a --dns-propagation-time command-line argument.

@mholt / @xenolf Is there any comment on this PR? I believe it to be clean, but I'm happy to push updates to it if there's a problem that we need to fix. We require this functionality in order to use lego, and I'd prefer if we didn't have to maintain our patch in perpetuity.

@mholt Your requested changes have been made and the PR has been updated.

Sounds really useful, for no apparent reason letsencrypt staging server fails to acknowledge TXT record in time.

I think this can be merged?

Closed in favor of  #634


Fix loading configuration from YAML

Motivation
Fixes #76. Unfortunately, upgrading the controller runtime which required manual config parsing, resulted in not reading the YAML config correctly. This caused the port for the health endpoints to not be set, thus, causing all health probes to fail.
I don't know why the tests didn't catch this, I think they need some overhaul...
Checklist

[x] I added relevant tests
[x] I have updated the documentation if necessary

I looks to me that tests are skipped every time, because they are executed only if there are changes to the chart files
https://github.com/borchero/switchboard/blob/main/.github/workflows/application-build-test.yml#L93

@vandot good pointing, thanks for pointing it out!


Rename Record -> DisplayWork

What is this PR trying to achieve?
Give our internal objects more sensible names to make them easier to reason about.
Who is this change for?
Developers working on this project.
Have the following been considered/are they needed?

[X] Tests?
[X] Docs?
[ ] Spoken to the right people?

🎆 IntelliJ refactor🎆


fix(rollup-relayer): avoid compressing "small tip" block

Purpose or design rationale of this PR
Applying related fix: https://github.com/scroll-tech/da-codec/pull/16

[ ] Updating the dependency in go.mod after merging the above pr.

PR title
Your PR title must follow conventional commits (as we are doing squash merge for each PR), so it must start with one of the following types:

[x] fix: A bug fix

Deployment tag versioning
Has tag in common/version.go been updated or have you added bump-version label to this PR?

[x] Yes

Breaking change label
Does this PR have the breaking-change label?

[x] No, this PR is not a breaking change

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 53.50%. Comparing base (86aed8e) to head (5aae583).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1414      +/-   ##
===========================================
- Coverage    53.53%   53.50%   -0.03%
===========================================
  Files          155      155
  Lines        11655    11655
===========================================
- Hits          6239     6236       -3
- Misses        4930     4932       +2
- Partials       486      487       +1

Flag
Coverage Δ

rollup
57.54% <ø> (-0.09%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

LGTM, let's update deps once the other PR is merged.

updated.

some issues in unit tests, debugging.


⚠ Bofrost.be has degraded performance

In 9097829, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1585 ms

Resolved: Bofrost.be performance has improved in 6ab2311 after 26 minutes.


Allow list-qt to print a list of archives for use with the --archives option

Let's say you want to install Qt 5.15.0 for linux desktop, but only install the 7z archives for {qtbase, qtdeclarative, icu}. You would use this command:
aqt install-qt linux desktop 5.15.0 --archives qtbase qtdeclarative icu

This feature is documented  at https://aqtinstall.readthedocs.io/en/latest/cli.html#cmdoption-list-tool-archives, and there's an example under https://aqtinstall.readthedocs.io/en/latest/cli.html#command-examples; it's the third example in the list.
I think it would be useful to add an option that lists archives to list-qt, but I cannot do it without an idea of what the interface would look like. I would like to use this issue to figure out what that interface would look like.
Originally posted by @ddalcino in https://github.com/miurahr/aqtinstall/issues/382#issuecomment-911653563

Part of the problem here is that in order to print a list of the archives you can choose from, you need to know the version of Qt, the architecture (see #371; I think it really makes a difference here), and which modules the user wants. Most modules only include a single archive, but many of them include several. For instance, the debug_info module contains dozens of modules, and it's nearly 1GB if you try to install them all.
Here are some potential interfaces:

Use --archives flag to pass in a Qt version number and a list of modules, and then print all of the archives for those modules. This would print all of the archives as one long list, so you cannot easily tell which module each archive belongs to.aqt list-qt windows desktop --archives 5.15.2 win32_mingw81 qtcharts debug_info  # archives for base and modules
aqt list-qt windows desktop --archives 5.15.2 win32_mingw81                      # archives just for base Qt

Add separate flags for the base and modules, and only print archives pertinent to one module at a time.aqt list-qt windows desktop --base-archives 5.15.2 win32_mingw81               # archives just for base Qt
aqt list-qt windows desktop --module-archives 5.15.2 win32_mingw81 qtcharts    # archives just for qtcharts
aqt list-qt windows desktop --module-archives 5.15.2 win32_mingw81 debug_info  # archives just for debug_info

Add a --long-archives flag that prints a table, just like the --long flag we used for list-tool.$ aqt list-qt windows desktop --long-archives 5.15.2 win32_mingw81 qtcharts debug_info  # print a table
Module       | Archives
===========================
Base 5.15.2  | qtbase
             | qtdeclarative
             | icu
             | ...
qtcharts     | qtcharts
debug_info   | ... all the archives for debug_info, on separate lines ...

Or this?
aqt list-qt windows desktop --archives 5.15.2 win32_mingw81 qtcharts debug_info  # archives for modules only
aqt list-qt windows desktop --archives 5.15.2 win32_mingw81                      # archives for base Qt only

Or this?
aqt list-qt windows desktop --archives 5.15.2 win32_mingw81 qtcharts debug_info  # archives for modules only
aqt list-qt windows desktop --archives 5.15.2 win32_mingw81                      # archives for base Qt only

I think this proposal makes more sense to me than the three possibilities that I came up with, so unless anyone has any objections, I will plan on implementing this. Thank you @pzhlkj6612


[enhancement]: Add some basic image manipulating tools to the canvas

Is there an existing issue for this?

[X] I have searched the existing issues

Contact Details
No response
What should this feature add?
As part of my workflow, I find I often need to save out an image, bring it in to photoshop or GIMP to do some very basic things, like cropping or selection/move/resize of some visual elements. Invoke is already a powerful tool with some basic functionality for painting and masking, inpainting and outpainting. But cropping and selection/manipulation is missing. It doesn't need to be deep or full featured. Just some really simple select/cut/copy/paste/resize/rotate.
Alternatives
The current workflow involves saving the canvas out, opening locally in your photo editor of choice, doing the manipulations, uploading back into invoke, and then adding as a raster layer. Even with this proposed added functionality it still may be required to do this process for more complex edits, but for simple things I think it would be valuable.
Additional Content
No response

You can freely move, resize, rotate and edit layers in Canvas. I think everything you have described is currently doable, though we don't support a selection tool for technical reasons.

Layers, yes, but not all features will be on their own layers, and the lack
of a selection tool prevents putting things on their own layer.
On Mon, Nov 4, 2024, 5:15 PM psychedelicious @.***>
wrote:

You can freely move, resize, rotate and edit layers in Canvas. I think
everything you have described is currently doable, though we don't support
a selection tool for technical reasons.
—
Reply to this email directly, view it on GitHub
https://github.com/invoke-ai/InvokeAI/issues/7257#issuecomment-2455811256,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADG7J2LWKFZ26V7GW7MZ3H3Z67WZ7AVCNFSM6AAAAABREGHVSSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJVHAYTCMRVGY
.
 You can use the bbox tool to crop a layer to a rect. I understand this isn't exactly the same as selection as a process, but you can get the same outcomes.

Thanks, will try that next time.
On Mon, Nov 4, 2024, 6:52 PM psychedelicious @.***>
wrote:

Duplicate a layer and erase the parts of it you don't want. You can use
the bbox tool to crop a layer to a rect. I understand this isn't exactly
the same as selection as a process, but you can get the same outcomes.
—
Reply to this email directly, view it on GitHub
https://github.com/invoke-ai/InvokeAI/issues/7257#issuecomment-2455938394,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADG7J2LCJ4IJK2KW55D6M43Z7ACCTAVCNFSM6AAAAABREGHVSSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJVHEZTQMZZGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Use Standard JS standard

Trivia: I wanted to include this file as a fixture for testing semicolon-style comments.
Wisely decided not to.


Cookbook compilation error

Hi, I'm getting a error using this cookbook. My setup is;

metadata.rb
depends 'runit'```

Berksfile
cookbook 'runit', '~> 1.7.2'```

recipes/logstash.rb

 include_recipe 'simple-logstash'```

If I dont include 'runit', I get this error,

```ERROR: Cookbook runit not found. If you're loading runit from another cookbook, make sure you configure the dependency in your metadata```

After I included it, kitchen converge fails saying,

Recipe Compile Error in /tmp/kitchen/cookbooks/simple-logstash/libraries/provider_logstash_service.rb
NameError
---------
uninitialized constant Chef::Provider::Service::Runit
   Cookbook Trace:
   ---------------
     /tmp/kitchen/cookbooks/simple-logstash/libraries/provider_logstash_service.rb:4:in `<class:Service>'
     /tmp/kitchen/cookbooks/simple-logstash/libraries/provider_logstash_service.rb:3:in `<class:Provider>'
     /tmp/kitchen/cookbooks/simple-logstash/libraries/provider_logstash_service.rb:2:in `<class:Chef>'
     /tmp/kitchen/cookbooks/simple-logstash/libraries/provider_logstash_service.rb:1:in `<top (required)>'

I'm new to Chef so this may not be a problem with this cookbook but with my configuration. Any ideas on how to resolve this ?

Do you use a wrapper cookbook to call this one?
Which version of Chef you used?

Yeah, the wrapper is supposed to install logstash and configure it with some templates.
Chef Development Kit Version: 0.6.2

My cookbook depends on runit, but in minor version (1.7.2) they break some functionality: renamed namespace for provider RunitService.
Try to use runit cookbook version 1.5.18 till I close this issue

Sure, thanks :smiley:


Update v3-0-reference.md

United States, Europe, and Asia Pacific are geographical (not regional) endpoint.

@krishna-doss-mohan : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @krishna-doss-mohan - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. We can keep this PR open for review and merge, but would you make future content updates in the private repo? If you need information about working in the private repo, please let me know and I'll send it via email. Thank you!

Hi Kristine,
Thanks for letting me know this.
I would need your help here.
Thanks.
M Krishna Doss | Microsoft Translator
From: Kristine Toliver<EMAIL_ADDRESS>Sent: Friday, December 4, 2020 1:55 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Krishna Doss Mohan<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Update v3-0-reference.md (#67176)
Hi @krishna-doss-mohanhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fkrishna-doss-mohan&data=04|01|krishna.doss%40microsoft.com|cc02d8e2ce4c44c25db208d8989f3ffc|72f988bf86f141af91ab2d7cd011db47|1|1|637427157025305915|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=P2r3lazvf0VmOIfsPWWJ77tKh1D1laQp3%2BUWma6%2BGJc%3D&reserved=0 - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. We can keep this PR open for review and merge, but would you make future content updates in the private repo? If you need information about working in the private repo, please let me know and I'll send it via email. Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F67176%23issuecomment-739041944&data=04|01|krishna.doss%40microsoft.com|cc02d8e2ce4c44c25db208d8989f3ffc|72f988bf86f141af91ab2d7cd011db47|1|1|637427157025310912|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=SZXPRXz1LLKRblxAljhIAOBovlNV2vDpP2a3cFCF5%2Fw%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAN6A7672BPVTTF3KQBN4FQDSTFLDLANCNFSM4UN6BYPA&data=04|01|krishna.doss%40microsoft.com|cc02d8e2ce4c44c25db208d8989f3ffc|72f988bf86f141af91ab2d7cd011db47|1|1|637427157025315910|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=uwLrJ0LrLzgAxiy5PI92N3kPIqOCC1Xr76bZipfMvpM%3D&reserved=0.

@krishna-doss-mohan Email sent. Thanks.

Since this is such a small edit, can we merge this content. I've reviewed it and #sign-off. I'll work with Krishna to ensure all future PRs go through the private repo.

Since this is such a small edit, can we merge this content. I've reviewed it and #sign-off. I'll work with Krishna to ensure all future PRs go through the private repo.

@erhopf: I'm sorry - only the author of this article, @swmachan, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@erhopf: I'm sorry - only the author of this article, @swmachan, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off

#sign-off

@erhopf Agreed! Merging.

@erhopf Agreed! Merging.


Make it use DCAwareRoundRobinPolicy unless rack is provided

Currently c-s uses RackAwareRoundRobinPolicy in any case which leads it targeting particular rack only, even if rack is not provided.
This PR makes it use DCAwareRoundRobinPolicy unless rack is provided.
Closes #20

There is still a question, if we need this fix "backported" into scylla-tools repo, and to releases

Also meanwhile I think we might have persuaded those who reported this to switch to use latte and not c-s...

There is still a question, if we need this fix "backported" into scylla-tools repo, and to releases

100% needs to be ported.


HA not working, right?

@luxas High Availability on ARM is not yet working, right? I saw this on your Roadmap.
I am having hard times starting a service in a high available manner. I tried replication controllers,  kubectl run ... expose ... etc. - without success.
Of course, I can expose a single service from a single node. But high availability requires any service being available on any node, even if I http-request a node that does not run the container/pod I am requesting. Docker Swarm Mode 1.12.1 has built in Mesh routing via Ingress and can do exactly this.
Do you have any ideas how to achieve this in K8s?

High Available for the Kubernetes components themselves aren't the same thing as high availability for the apps running on top of Kubernetes, right?
In the roadmap, I meant that right now the master is the SPOF, and multiple masters should be supported in future releases.
High availability for apps should be as easy as creating a RC with replicas>1, and exposing the RC as a service. Then the service will load-balance the requests to all endpoints.
Does this answer your question? Otherwise it's clearly a kubernetes/kubernetes issue I think

We're working on HA in sig-cluster-lifecycle right now with kubeadm, but it's a highly complex task with a lot of corner cases and stuff, so we don't have a ETA yet.
Follow us on Slack: https://kubernetes.slack.com/messages/sig-cluster-lifecycle/ and/or attend our meetings if you're interested.

All right, that's all info I was looking for to add to my research project ("HA performance of K8s on ARM"). Thank you!

Just for my curiosity... can you send the research paper to me when it's in beta version or done?
It would be nice to read what you've written about it...
(We can take this conversation by email as well)

I send you an email :)

@luxas Did you get my email from 13.10. ? I could send you the beta version of the paper, if you confirm the email.

I'm sorry for not responding, first I waited quite a long time for the email to arrive... just to now find out that MS Outlook SmartScreen or something like that had marked it as spam :(.
And this week I've been on a journey without internet, so responding now.
Yes, please send the paper.
I'll be more careful now :)


"AuthenticateFailException: Login prepare unsuccessful" When logging in

I'm getting the following error when trying to log in: AuthenticateFailException: Login prepare unsuccessful. The credentials are correct, i've checked them, so I'm not sure what is going wrong. Have you experienced this error before?

I ran the login command manually, and it turns out to be related to my proxy :). Sorry for bothering haha.


add deploy env variable to config.yaml and adjust charm code

Why do we need a new configuration option? Why this can't be inferred by the system, e.g. by looking at the the backend url?

I wouldn't say it can't because it can, the question is: is the best approach? It's just too simple to add a new variable since the agent already expects it (https://github.com/omnivector-solutions/license-manager/blob/main/agent/lm_agent/config.py#L53).

Why do we need a new configuration option? Why this can't be inferred by the system, e.g. by looking at the the backend url?

I wouldn't say it can't because it can, the question is: is the best approach?

That's precisely my point. We can have the environment with an ever growing number of variables, but I don't feel that is a good approach. This increases the complexity of maintenance and deploys, making it more expensive and error-prone.
So, can't be inferred by the system?

I disagree it increases the complexity of maintaining. The purpose of the task is indeed to make it easier to understand the Sentry environments and the way the apps use them.
Answering the question indeed: it would work only for jobbergate cli and license manager agent I think, but I would need to add a parser in each application to infer it, and that indeed would increase the complexity.
To decrease the complexity in order to have less files created (each config is a file) we can plan another approach. As soon as we grow the apps it's expected to have more variables.

Why do we need a new configuration option? Why this can't be inferred by the system, e.g. by looking at the the backend url?

I wouldn't say it can't because it can, the question is: is the best approach?

That's precisely my point. We can have the environment with an ever growing number of variables, but I don't feel that is a good approach. This increases the complexity of maintenance and deploys, making it more expensive and error-prone.
So, can't be inferred by the system?

I disagree it increases the complexity of maintaining. The purpose of the task is indeed to make it easier to understand the Sentry environments and the way the apps use them.
Answering the question indeed: it would work only for jobbergate cli and license manager agent I think, but I would need to add a parser in each application to infer it, and that indeed would increase the complexity.
To decrease the complexity in order to have less files created (each config is a file) we can plan another approach. As soon as we grow the apps it's expected to have more variables.

Also, let's say it can be inferred from the backend url (I can't any other way to identify), it wouldn't be possible to set for local purposes because there's no backend url like https://armada-k8s.local.omnivector.solutions