Modify MiqAction so that queue_name is set for ems operations

This PR modifies the private method invoke_or_queue so that it will set the queue_name whenever the role is an ems operation.
The name is set using the target's queue name for ems operations, or "generic" if that's not possible. In most cases the target and the rec are the same for ems operations. Only action_vm_retire actually has a different target and would have to default as far as I could tell.
I updated the specs a bit to include one of the other operations besides action_vm_retire as well.
Part of #19543
Cross-repo tests at https://github.com/ManageIQ/manageiq-cross_repo-tests/pull/41

@miq-bot add_reviewer @agrare


Add reshape and flatten for dask.array

Fixes #570
Currently only does order='C'. A future PR might add the option for order='K', which could be useful in some cases.
Example usage:
In [1]: import numpy as np

In [2]: import dask.array as da

In [3]: a = da.from_array(np.random.randint(10, size=(3, 4)), chunks=2)

In [4]: a.reshape(2, 6).compute()
Out[4]:
array([[7, 9, 6, 4, 2, 7],
       [7, 8, 3, 5, 4, 7]])

In [5]: a.reshape(-1).compute()
Out[5]: array([7, 9, 6, 4, 2, 7, 7, 8, 3, 5, 4, 7])

In [7]: a.ravel().compute()
Out[7]: array([7, 9, 6, 4, 2, 7, 7, 8, 3, 5, 4, 7])

Have you tried this on real-world problems?  I'm a bit concerned that this might result in very many tasks.

The compelling use case for this PR is reshaping a stack of images along the last axes. For this use case, the shortcut logic in this PR is actually per efficient.
Notice how compact these graphs are:
import dask.array as da
x = da.ones((50, 1000, 1000), chunks=(10, 1000, 1000))
x.ravel().visualize()

x.reshape(50, -1).visualize()

On the other hand, if we try to reshape along the wrong axis, things get much messier:
da.ones((5, 10, 10), chunks=5).reshape(5, -1).visualize()

Given how bad this looks, it might be better to simply disallow this sort of reshaping.
Note that the "stack of images" is probably the most compelling use case for dask.array right now -- I would put climate data use cases such as "one big file stores each month's data" under that same category. The efficient types of reshaping that are allowed here would be enough to allow for arbitrary raveling/unraveling of dimensions in xray in many cases (e.g., converting a 2D array with dimensions ('x', 'y') to a 1D array with a multi-index and vise-versa).

OK, sounds like we should merge this and wait to see if it gets misused.  If misuse ever occurs then maybe we look into raising warnings at the appropriate times.
+1

Hi,
I'd just like to add that I think I have tripped on the problem you were worrying about.
setup:
window_size = 1e6
number_of_windows = 7e5

data = da.from_array(d, chunks=(1e6, ))

# this takes forever and crashes
windowed = data[:window_size*number_of_windows].reshape((-1, window_size))

and the result:
Traceback (most recent call last):
  File "/Users/jeremy/Google Drive/phd/downsample_plot/downsample.py", line 81, in <module>
    real_window_size))
  File "/Users/jeremy/anaconda/lib/python3.5/site-packages/dask/array/core.py", line 890, in reshape
    return reshape(self, shape)
  File "/Users/jeremy/anaconda/lib/python3.5/site-packages/dask/array/core.py", line 2224, in reshape
    return unravel(ravel(array), shape)
  File "/Users/jeremy/anaconda/lib/python3.5/site-packages/dask/array/core.py", line 2209, in unravel
    for n in range(shape[0])])
  File "/Users/jeremy/anaconda/lib/python3.5/site-packages/dask/array/core.py", line 1494, in stack
    assert len(set(a.chunks for a in seq)) == 1  # same chunks
AssertionError

# this appears to work and is instant
windowed = data[:window_size*number_of_windows].reshape((number_of_windows, window_size))

Those lines of code should do exactly the same thing, so there might be a bug lurking here. Does it work if you cast the sizes to integers with int?
On Mon, Dec 7, 2015 at 2:58 AM, Jeremy Herbert<EMAIL_ADDRESS>wrote:

Hi,
I'd just like to add that I think I have tripped on the problem you were worrying about.
setup:
window_size = 1e6
number_of_windows = 7e5
data = da.from_array(d, chunks=(1e6, ))

# this takes forever and crashes
windowed = data[:window_size*number_of_windows].reshape((-1, window_size))

and the result:
Traceback (most recent call last):
  File "/Users/jeremy/Google Drive/phd/downsample_plot/downsample.py", line 81, in <module>
    real_window_size))
  File "/Users/jeremy/anaconda/lib/python3.5/site-packages/dask/array/core.py", line 890, in reshape
    return reshape(self, shape)
  File "/Users/jeremy/anaconda/lib/python3.5/site-packages/dask/array/core.py", line 2224, in reshape
    return unravel(ravel(array), shape)
  File "/Users/jeremy/anaconda/lib/python3.5/site-packages/dask/array/core.py", line 2209, in unravel
    for n in range(shape[0])])
  File "/Users/jeremy/anaconda/lib/python3.5/site-packages/dask/array/core.py", line 1494, in stack
    assert len(set(a.chunks for a in seq)) == 1  # same chunks
AssertionError

# this appears to work and is instant
windowed = data[:window_size*number_of_windows].reshape((number_of_windows, window_size))

Reply to this email directly or view it on GitHub:
https://github.com/blaze/dask/pull/758#issuecomment-162482683

Hi @shoyer,
I can't get my code to break anymore unfortunately, I'm guessing it was something I did wrong. But I did find a problem with float handling, and I have created a new issue for it.

Great work @shoyer

Possible fail case due to many-small-tasks: http://stackoverflow.com/questions/37912827/dask-array-reshape-very-slow
I don't think we should really do anything so resolve the problem (other than maybe raise an informative warning.)  Just pointing it out.


JSON field types are unhandled

When a table contains field type of JSON, plugin errors out.
Steps to reproduce
Query SELECT * FROM something where the something table contains a field of type JSON and observe the error.

This is a duplicate of https://github.com/grafana/google-bigquery-datasource/issues/145


Intermittent error on multi-tenancy: "Error: 16 UNAUTHENTICATED: Expected Identity to provide authorized tenants, see cause for details"

Expected Behavior
No error should pop up
Current Behavior
A cluster is created, multi-tenancy enabled. 2 tenants are created, blue and green.
A script is created:

deploying the same simple process on both tenants. This process contains a service task
Starting a worker, with a filter on each tenant (so two workers are started)
Every 5 seconds, a new process instance is created on the green tenant
every 20 seconds, a new process instance is created on the red tenant

Process instances are created, and workers execute jobs, but from time to time, an Error: 16 UNAUTHENTICATED: is visible in the log. Even so
, all service tasks are completed.
Possible Solution
No solution
Steps to Reproduce
Create a cluster, with two tenants red and green
create a client ID / Client Secret
Run the script
Context (Environment)
Multi-tenancy usage
Detailed Description
Log visible:
 ##### Started/Job RED: 0/0, GREEN: 1/2
------  Green process instance started: [2251799813686010]
>>>> Execute GreenJob PI=[2251799813686010] tenant=[green]
15:42:07.412 | zeebe |  [service-task] ERROR: Grpc Stream Error: 16 UNAUTHENTICATED: Expected Identity to provide authorized tenants, see cause for details

 ##### Started/Job RED: 0/0, GREEN: 2/3
------  Green process instance started: [4503599627371180]
>>>> Execute GreenJob PI=[4503599627371180] tenant=[green]
15:42:12.524 | zeebe |  [service-task] ERROR: Grpc Stream Error: 16 UNAUTHENTICATED: Expected Identity to provide authorized tenants, see cause for details

 ##### Started/Job RED: 0/0, GREEN: 3/4
(node:226) UnhandledPromiseRejectionWarning: Error: 16 UNAUTHENTICATED: Expected Identity to provide authorized tenants, see cause for details
    at callErrorFromStatus (/mnt/d/pym/CamundaDrive/Support & Consulting/S-19554 Avertra NodeJS/avertra-test/node_modules/@grpc/grpc-js/build/src/call.js:31:19)
    at Object.onReceiveStatus (/mnt/d/pym/CamundaDrive/Support & Consulting/S-19554 Avertra NodeJS/avertra-test/node_modules/@grpc/grpc-js/build/src/client.js:192:76)
    at listener.onReceiveStatus.processedStatus (/mnt/d/pym/CamundaDrive/Support & Consulting/S-19554 Avertra NodeJS/avertra-test/node_modules/@grpc/grpc-js/build/src/call-interface.js:78:35)
    at Object.onReceiveStatus (/mnt/d/pym/CamundaDrive/Support & Consulting/S-19554 Avertra NodeJS/avertra-test/node_modules/zeebe-node/dist/lib/GrpcClient.js:97:36)
    at InterceptingListenerImpl.onReceiveStatus (/mnt/d/pym/CamundaDrive/Support & Consulting/S-19554 Avertra NodeJS/avertra-test/node_modules/@grpc/grpc-js/build/src/call-interface.js:73:23)
    at Object.onReceiveStatus (/mnt/d/pym/CamundaDrive/Support & Consulting/S-19554 Avertra NodeJS/avertra-test/node_modules/@grpc/grpc-js/build/src/client-interceptors.js:360:141)
    at Object.onReceiveStatus (/mnt/d/pym/CamundaDrive/Support & Consulting/S-19554 Avertra NodeJS/avertra-test/node_modules/@grpc/grpc-js/build/src/client-interceptors.js:323:181)
    at process.nextTick (/mnt/d/pym/CamundaDrive/Support & Consulting/S-19554 Avertra NodeJS/avertra-test/node_modules/@grpc/grpc-js/build/src/resolving-call.js:99:78)
    at process._tickCallback (internal/process/next_tick.js:61:11)
for call at
    at ServiceClientImpl.makeUnaryRequest (/mnt/d/pym/CamundaDrive/Support & Consulting/S-19554 Avertra NodeJS/avertra-test/node_modules/@grpc/grpc-js/build/src/client.js:160:32)
    at ServiceClientImpl.<anonymous> (/mnt/d/pym/CamundaDrive/Support & Consulting/S-19554 Avertra NodeJS/avertra-test/node_modules/@grpc/grpc-js/build/src/make-client.js:105:19)
    at Promise (/mnt/d/pym/CamundaDrive/Support & Consulting/S-19554 Avertra NodeJS/avertra-test/node_modules/zeebe-node/dist/lib/GrpcClient.js:272:47)
    at process._tickCallback (internal/process/next_tick.js:68:7)
(node:226) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:226) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
--

Possible Implementation
red-green.txt
simple-bpmn.txt

This is likely caused by resource starvation in the Zeebe gateway. Allocating more resources to the gateway should mitigate this.
@pierre-yves-monnet what resources did you allocate for the cluster?
Do you have a cluster that I can test against?

See this issue in Zeebe-land: https://github.com/camunda/zeebe/issues/18697
That's an attempt to mitigate this on the gateway.
Package 8.5.4-alpha2 has a fix for #163 in it that may solve this.

Notes for myself:
See here: https://jira.camunda.com/browse/SUPPORT-21985?focusedCommentId=350685&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-350685
and here: https://jira.camunda.com/browse/SUPPORT-22321

"to workaround the problem, please increase the maximum allowed number of threads via zeebe.gateway.threads.grpcMaxThreads. The default value is 2*numberOfCores"


brid.gy does not find website in facebook profile anymore

didn’t manage to reopen #393 so i duplicate it here: facebook does not find my sites anamore, for my personal profile, as well as pages that i admin.
https://www.facebook.com/diplix/about
https://www.facebook.com/wirres/about
changing my website to https does not work either.

thanks for reporting, @diplix! looks like this is due to our in-progress migration to facebook API v2.6. should hopefully be an easy fix.

thanks again for the report!


Hide the signature message

Idea
In 1.19.3, NCR detects the signature requirement by

chat.disabled.missingProfileKey
Chat disabled due to missing profile public key. Please try reconnecting.

That message should be hidden as it doesn't really give much info to the user.
Reasoning

User gets confused by the wording of the message
User does not actually need to reconnect
User can already see the NCR chat status
The message becomes wrong as soon as the user actually signs the chat

Other Information
No response

I think I'll just replace it with something more informative instead of hiding it entirely.

That's a possibility, however it would still spam the chat and logs, regardless of whether the user actually decides to sign at that point...
And when the user sends the logs to a server admin for any reason, the admin may get mad or confused about this kind of mod's presence, even if the server does enforce signatures. So that would conflict #255, kind of.

So that would conflict https://github.com/Aizistral-Studios/No-Chat-Reports/issues/255, kind of.

#255 is about a feature user has no direct control over. It's user's ability and responsibility to supervise what they send to server admins however, especially when they are known for enforcing unfavorable policies.
The reason I would prefer to keep some kind of message is because some servers might opt to replace it with something custom, in which case NCR will fail to detect it. Establishing a connection between warning message and prompt that the mod opens will give users better chance to identify this as an incompatibility and report it.

The reason I would prefer to keep some kind of message is because some servers might opt to replace it with something custom, in which case NCR will fail to detect it. Establishing a connection between warning message and prompt that the mod opens will give users better chance to identify this as an incompatibility and report it.

Well, I would say that the user could also draw the connection between a lack of an error vs an error + NCR bug. Anyway, would you consider an option to simply make that message blank (if removing would take more effort)?

Anyway, would you consider an option to simply make that message blank (if removing would take more effort)?

Removing it wouldn't take much effort. Yeah, I can do an option.


Template support for k8s.

@fiji-flo r?

Coverage remained the same at 87.201% when pulling 323b0947bfec046b8f168490436d67a1c326abc7 on akatsoulas:dev-preview-templates into e959428a75080a26b1419535b6d407d7ceaf1685 on mozilla:dino-park.


Testing guide

We are looking at providing an overview of what "testing with Prisma" entails:

Using Prisma migrate to set a testing environment up in a shape for running integration tests (as elaborated on in this article)
Mocking a client to simplify such a setup when doing unit testing
Testing an actual migration (e.g. whenever involving custom changes or data transformations)

This article would talk to the various kinds of testing situations where Prisma is involved and offer pointers to existing articles first.
We'd like to write the one about testing an actual migration using the concept of data transformation: custom migration handling combining firstName and lastName into fullName could be something to create a test for, to add confidence in the process of running such a migration on a production environment.

Involves translating this draft into a guide: https://www.notion.so/prismaio/Testing-overview-article-f26a234c84f940bf85b03bad0fed6d8f


Flutter Web/Dwds can lose print messages during a hot restart

Whille debugging failures in https://github.com/flutter/flutter/pull/85325/checks?check_run_id=3025626159 , I've found that asserting for output can fail unless a message is repeated N times. This may indicate that print statements get dropped somewhere along the line of a hot restart

To repro, remove the while loop from the generated web hot reload template. This will cause the print statement to made only a single time, which has a high chance of getting lost.

Closing as I don't think this is really that important.


composebox_typeahead: Remove redundant options from the / typeahead

Fixes #18318
Removes the /day and /night options from the typeahead menu while still allowing the commands to be used. Typing /day and /night will now suggest /light and /dark, respectively. Also changes the display settings to use Light mode/Dark mode. Finally, changes the Dark mode and Light mode popups that appear after using the corresponding command.

Testing plan:
Manual testing, node tests, and backend tests.
GIFs or screenshots:
New slash typeahead without the /day and /night options

/day typeahead

/night typeahead

/da typeahead (which can either be /day or /dark)

New personal settings menu (display settings)

Popups after using /light or /day

Popups after using /dark or /night

@timabbott could you please take a look at this? I think it is ready to be merged. Thanks!

@alya should the default commands still be /day and /night? I saw some people suggesting in https://github.com/zulip/zulip-mobile/issues/4009 that we make /light and /dark the new defaults, but that issue was from over a year ago.

Ah, yeah, good question! Looking into it, as discussed in this thread, there's general consensus that "dark" and "light" are better terms. So let's make those be the ones we display.
@jonathanktran would you be up for also changing the personal settings menu (Display settings) to use Light mode/Dark mode?

@alya @timabbott
Thanks so much for reviewing my pull request! I've just pushed my latest changes. In addition to changing the personal settings menu to use Light mode/Dark mode, I've also updated the Dark mode and Light mode popups that appear after using the corresponding command. Please take a look when you can! :)

Hm, I'm still seeing all the slash commands on the list. Am I doing something wrong?

There might be something weird about my git state...

@alya That's strange I just checked again on the development server and it seems to be working. Could you try it again?

Yeah, I don't know what was wrong before -- I tried it again and it looks good!
@jonathanktran would you be up for also doing a pass on our documentation to replace mentions of "day mode"/"night mode" with "light mode"/"dark mode"? https://zulip.com/help/night-mode is the main user doc page, but there are probably other references to it as well.

Thanks, @jonathanktran !

Just a ping on updating this.

Actually, it's probably better to track the remaining work as a new issue: https://github.com/zulip/zulip/issues/20228.  Closing this as the main work done here has been merged.  Thanks @jonathanktran!


add IBAN to 'features' section in README.md

Add IBAN to 'features' section in README.md

Thanks a lot!!


Ensure bin/setup works

Closes #1224
It's unclear why we need to run migrations, since we were originally
calling db:prepare which does the following:

If the database exists but the tables have not been created, the
command will load the schema, run any pending migrations, dump the
updated schema, and finally load the seed data. See the Seeding Data
documentation for more details.

However, I found that calling db:migrate in the application template
worked.

This might need more work.
bin/rails aborted!
ActiveRecord::NoDatabaseError: We could not find your database: app_name_development. Available database configurations can be found in config/database.yml. (ActiveRecord::NoDatabaseError)

To resolve this error:

- Did you not create the database, or did you delete it? To create the database, run:

    bin/rails db:create

- Has the database name changed? Verify that config/database.yml contains the correct database name.

Caused by:
PG::ConnectionBad: connection to server on socket "/tmp/.s.PGSQL.5432" failed: FATAL:  database "app_name_development" does not exist (PG::ConnectionBad)

I might need to call both?


[Egg Request]: KSP - Luna Multiplayer

Does this expand an already existing service
Yes
Link to a game or software
https://lunamultiplayer.com/
Links for the download
https://github.com/LunaMultiplayer/LunaMultiplayer/releases
Links for the install docs
https://github.com/LunaMultiplayer/LunaMultiplayer/wiki/Create-a-server

https://github.com/pelican-eggs/games-standalone/tree/main/ksp
As i know, is KPS Luna abdoned

closed as it already exsist

https://github.com/pelican-eggs/games-standalone/tree/main/ksp
As i know, is KSP Luna abdoned

It doesn't seem like it's abandoned, last nightly release was in September

closed as it already exsist

were?
I could only find "Dark Multiplayer" for ksp

closed as it already exsist

were? I could only find "Dark Multiplayer" for ksp

I tried to make this in the pases. But the projects is a liite unmaintaind. I will try again but only there layest beta will be supporting aa everyhing is using dotnet 5


Optional<Entity> optLightning = loc.getExtent().createEntity(EntityTypes.LIGHTNING, loc.getPosition());
if (optLightning.isPresent()) {
    Lightning lightning = (Lightning) optLightning.get();
    lightning.setEffect(true);
    loc.getExtent().spawnEntity(lightning, Cause.of(this));
}

Per the java-docs when setEffect is set to true lightning should not do damage, and thus should not create fire.

@DarkArc: I'm having trouble reproducing this. Can you give me a test plugin that can easily reproduce it?

@Aaron1011 seems it's been fixed by something (either from updating Forge or Sponge, made the 1.8.9 jump) either way, no longer an issue.

This is back.

I'll look into this

@DarkArc I can't reproduce this on the latest SpongeForge. Is there still an issue?

@simon816 I'm still seeing fire blocks.

Oh I think I had the difficulty level set incorrectly. I've reproduced it now.


ssr - document is not defined

I'm submitting a
Feature/improvment (a behavior that used to work and stopped working in a new release)
Current behavior
Not sure if worth addressing but when using edit-on-place with Angular 17 I am getting the following error:
ERROR ReferenceError: document is not defined
at editMode (./node_modules/@ngneat/edit-in-place/fesm2020/ngneat-edit-in-place.mjs:83:55)
at editMode (./node_modules/@ngneat/edit-in-place/fesm2020/ngneat-edit-in-place.mjs:90:43)
at value (./node_modules/rxjs/dist/esm/internal/operators/switchMap.js:14:23)
at _next (./node_modules/rxjs/dist/esm/internal/operators/OperatorSubscriber.js:13:21)
at next (./node_modules/rxjs/dist/esm/internal/Subscriber.js:31:18)
at _subscribe (./node_modules/rxjs/dist/esm/internal/BehaviorSubject.js:12:44)
at _trySubscribe (./node_modules/rxjs/dist/esm/internal/Observable.js:37:25)
at _trySubscribe (./node_modules/rxjs/dist/esm/internal/Subject.js:74:22)
at cb (./node_modules/rxjs/dist/esm/internal/Observable.js:31:30)
at errorContext (./node_modules/rxjs/dist/esm/internal/util/errorContext.js:19:9)
Expected behavior
Should work out of the box with SSR
Minimal reproduction of the problem with instructions
NA will try and get around to given time
What is the motivation / use case for changing the behavior?
NA. For some reason started coming across this issue with Angular. I suspect because domino which mocks window and document does not work with Angular 17.
Environment
1. Mac OS
2. Angular 17

I have limited experience with Angular and SSR.
Based on the documentation, the fix would be to move the ngOnInit block of the EditableComponent to afterRendernot to rely on the document API on server side: https://angular.dev/guide/ssr#authoring-server-compatible-components.

ng neat is registered in the component html itself, via a module. To fix this as ngneat used in the HTML itself, would be to make this fix in the library itself

Indeed, a fix is in progress to support SSR by using afterNextRender.
It means the minimum Angular version required for this new version will be Angular v16.

It is causing my ngModules to break

Module was removed and changed to standalone component: https://github.com/ngneat/edit-in-place/pull/94/files#diff-3d83580a41cb8bd79a4248b846ad31e50f2c369ad86cabc5a996cb9e02a1dffa
I will look into importing directives individually but will require refactor of codebase

I need to update the documentation: the new APi is using the Standalone API.
There is no EditableModule anymore.

sure, i am trying that now

Successfully migrated over to the next syntax. Will keep you posted if i still experience document issue

No longer getting the error. I use EditableCOmponent and directives. Can mark this as complete until further notice. Thank you and good work

I am now getting
WARNING: sanitizing HTML stripped some content, see https://g.co/ng/security#xss
ERROR TypeError: Cannot read properties of undefined (reading 'tpl')
at EditableComponent_Template (./node_modules/@ngneat/edit-in-place/fesm2022/ngneat-edit-in-place.mjs:22:103)
at executeTemplate (./node_modules/@angular/core/fesm2022/core.mjs:12159:9)
at refreshView (./node_modules/@angular/core/fesm2022/core.mjs:13392:13)
at detectChangesInView (./node_modules/@angular/core/fesm2022/core.mjs:13617:9)
at detectChangesInViewIfAttached (./node_modules/@angular/core/fesm2022/core.mjs:13580:5)
at detectChangesInComponent

Can you share related html code?
Given the error I'd say either the editMode or viewMode directive is not applied with an editable component.

<editable>
    <ng-template viewMode>{{defaultValue}}</ng-template>
    <ng-template editMode>
        <input matInput [style.width]="defaultValue.length * 14 + 'px'" editableFocusable editableOnEnter [placeholder]="defaultValue" (keyup.enter)="saveEdit($event)"/>
    </ng-template>
</editable>

Using your template, i can only reproduce it if the directives are not imported: https://stackblitz.com/edit/angular-ugpjwc
Can you check your imports?

that is it, thank you

resolved.


Polish and fix translation of simple Chinese

[x] Have you followed the contributing guidelines?

Sure

[x] Have you explained what your changes do, and why they add value to the Guides?

Some wording are not proper and Django part was missing. So I help to fix them.
Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

bhe

hxd


Is this compatible with the ESP_RTOS_SDK?

Is this library compatible with ESP_RTOS_SDK?

No, it is not.


Adding two missing calls to the Connection class.

Added releaseConnection and end to the methods that are available on the Connection class.

Thanks!


Spike: evaluate databus interface and provide feedback

We had two discussions on the Plonk data bus and its interface.
Discussion 1: Internal presentation to bonobos team. We covered the following topics:

The idea of public inputs in the current Plonk setup
Limitations of current public inputs
Zac's new ideas of how to handle public inputs - the Plonk data bus
Improvements to Plonk data bus design
Plonk data bus in Aztec's kernel circuits

Our focus was mainly understanding the data bus design, trade-offs in data bus and high-level overview of the final protocol. We did not discuss any math or constraints.
Discussion 2: Data bus discussion with Luke and Mike. We covered the following topics:

Validating internal presentation's correctness
Q&A about data bus desing
Plonk data bus interface proposal by Zac
Timeline moving forward

Next steps:

Collect all our questions and discuss with Zac (next week)
Circuits team should start exploring with the interface design and start implementing the interface without any backend.

Resources:

Slides: https://aztec.slides.com/suyashbagad_aztec/plonk-data-bus
Interface proposal: https://hackmd.io/@aztec-network/H1wASaI23?type=view
Miden's communication bus: https://wiki.polygon.technology/docs/miden/design/multiset/#communication-buses


Adding pods/log as resource

Adding pods/log as resource fix for #13

Thanks for making this chart better !

Thanks for making this chart better !


Executors: create while executor

We need a while executor, that checks a condition function every loop and repeats a loopExecutor each time.
Target Code:
controller.addListener('whileEvent', [
  while(condition, loopExecutor)
]);

while is a reserved word, so we'll need another name for this syntax.  Maybe we could use loop instead of while?  I don't care much for wwhile.  Even stranger would be using ifdo

Let's use loop for now. This is as you pointed out basically an interval executor with a 0ms timeout.


Studying "I ADOPTED A KITTEN!!!"

https://www.youtube.com/watch?v=2-bFB6_iOCQ

Dictation
Hey guys.
I am in the car and I am on the way to get a kitten.
Someone I know from high school is giving away their kittens and I am going to go check them out if they are health and if they look like well-taken care. Then I will bring one home.
But we don't have anything for the cat yet. So first we're going to go Pet Smart and then buy the littele box.. the food and bed and everything like that.
So cute. Oh my gosh. Hi. So cute.
Ok. We just picked out our kitten. Oh my gosh.
It's a girl. We didnt' know if we wanted a boy or a girl.
I am a little bit allergic to cats and I already got hives on right hand. Yeap. That's a hive.

hives: 두드러기 (note: hive 는 벌집))

Meaw.
Oh my gosh. Just probably scared. It's a really bumpy road.

TBD: From https://youtu.be/2-bFB6_iOCQ?t=86

Dictation (From https://youtu.be/2-bFB6_iOCQ?t=86_)
OK, we just got home. We bought the can, a little bed and a scratcher toy and some food, it doesn't eat hard food yet but it will in a few.

a few
A short period of time
Hey, I'll be back in a few, so start making dinner now.

We got some canned food, let's see.
We have to buy, you know like, everying from scratch because we don't own any pets.

If you do something from scratch, you do it without making use of anything that has been done before.

This is for the food, this is for the water.
It matches. We got the matching set. This your litter box we got.

She just came out of her cage.
This is the first time in her new home.
She has stripes like a tiger. We should call her tigress.
Tiger lily, oh my gosh, _____________

TIGRESS is a female tiger

Shouly we call her tiger lily? She's so small.... she's like so small.

TBD : Dictation (from https://youtu.be/2-bFB6_iOCQ)

Dictation (from https://youtu.be/2-bFB6_iOCQ?t=253)

Dictation (from https://youtu.be/2-bFB6_iOCQ?t=289)
So the second question that was asked was why did I get the kitten in ~for~ the first place because everybody knows i am a dog person.
And it's because I couldn't ~cannot~ have a dog. My landlord would not let me get a dog because our apartment was newly renovated. She didn't ~doesn't~ want the wear and tear on her new appliances, doors, wall, anything like that.
So I can't have a dog. I really wanted to have Australian Shepherd ~Sheppard~ but it's just not in the cards.

in the cards: likely to happen:

Marriage isn't in the cards for me right now – I'm focused on my career.
Note: Used especially in negative sentences.

Now, kittens, I have no idea that they're just as high maintenance as a dog.

high-maintenance: adj.

needing frequent or major maintenance or repair:

Imported sports cars are high-maintenance.

Informal. (of a person) demanding a great deal of attention, money, or effort:

His high-maintenance girlfriend refuses to get engaged without a big diamond ring.

The reason I wanted a kitten in the first place was because I thouht that they are ~are~ aloof, I thouht that they were ~are~ independent, I thought that they play ~play~ by themselves, I thought that they just lay their ~eat~ food out and they just eat it whenever..

aloof

adj, distant either physically or emotionally

an aloof, unfriendly manner

adv, at a distance

trying to keep failure aloof

No No I had no idea that cats were ~need~ so high maintenance.
(TBD: https://youtu.be/2-bFB6_iOCQ?t=340)

Dictation (from https://youtu.be/2-bFB6_iOCQ?t=340)
Kittens are like new born babies. They always want your attention.
I can't even walk outside in my room without their mewing, looking for me. She always wants to be by me 24/7.
She usually sleeps right on my neck, on my boob, on my shoulder, on my head, on my side, like tucked under.

tuck: to put something in a particular place, especially in order to keep it safe or hidden.

So this is her hugging my face while falling asleep. She's so sweeet and affectionate. I am really happy that I got her as a furry ~free~ friend.

furry: covered with fur

The last question that was asked was that what is her breed.
Shel's a brown tabby ~hairy~ cat and her breed comes from a line of leopards.
And then what is her age. She is six weeks old.

tabby cat: a domestic cat with a striped and mottled coat
mottled: 얼룩덜루한

I will make sure to do an ~doing~ updte on her when she gets older. But that's all for now.
Thank you guys for watching, like, and subscribe. Bye.

Questions for discussion

Why does she get a kitten instead of a dog?
What does the cat look like?
What change happen to her life since she got the kitten?
What did she prepare to live with the kitten?


Support picture

Like #2, but could be problematic because <picture> overrides whatever is in <img src>. It might need an additional overriding <source> element to be added and removed while checking.
This could already be working with a <picture> polyfill because it supports changes to the src property (which a polyfill would have to change)
This is only needed where <picture> is supported, but object-fit/object-position isn't, which is Edge 13-14, Safari 9.1, iOS 9.3
The support here could be added by duplicating the entire <picture> element and use that to figure out the current value of currentSrc on resize and could also be a solution to #13

Thanks for this wonderful script.
I'm now in need for the picture element and noticed, that it doesn't work with picture.
Is there any news/progress on this one?
Thanks!

Is it not working when object position values are given in Percentage(%)?

@touch2hussain if it isn't, open a separate issue

As far as I understood the problem, we currently support the picturefill-functionality but the native picture-element is not supported and the image is not replaced on mobile?
I would like to get in touch with this problem since I need this functionality pretty often. Did you already started @bfred-it or have any concept for this?
Also are there any rules for contribution for this repository?

Sorry for the delay but I haven't had much time to contribute to this.
The idea I had was to have a copy of the original img (and its <picture>) and poll that element for the computed src, but I haven't gotten around to it. This solution would also replace the current logic for the srcset. I'm afraid it's not going to be particularly easy to implement, though, I think picture depends on the actual size on the page as well.
An alternative for now would be to use a different script, like constancecchen/object-fit-polyfill

I have a workaround idea on the native picture support (Edge 13-14, Safari 9.1, iOS 9.3)
After setting the background-image on img tag, we just remove the relevant srcset value on source element, the defined background style on img tag will work as expected.
el.parentNode.querySelectorAll('source').forEach((sourcenode) => {
    let srcset = sourcenode.getAttribute('srcset')
    if (srcset === el.currentSrc) {
        nativeSetAttribute.call(sourcenode, "data-ofi-srcset", srcset);
	sourcenode.setAttribute('srcset', '')
   }
})

When el.currentSrc changed, restore the source srcset value from data.

I took up the idea of having a copy of the picture tag. It seems to me a bit of a quick'n'dirty solution, but it work fine for my use case at least. So right after new Image(...)
if (window.HTMLPictureElement) {
	var sourceNodes = el.parentNode.querySelectorAll('source');
	if (sourceNodes.length > 0) {
		ofi.picture = document.createElement('picture');
		ofi.picture.style.display = 'none';
		for (var i = 0; i < sourceNodes.length; i++) {
			ofi.picture.appendChild(sourceNodes[i]);
		}
		ofi.picture.appendChild(ofi.img);
		el.parentNode.insertBefore(ofi.picture, el);
		ofi.img.onload = function(e) {
			fixOne(el);
		};
	}
}

Any news on this issue?

Edge supports object-fit on images, so these days I think you'd just need this and picture-fill and you should be good, until you want to use object-fit on a video or canvas element.?

From what I can tell this library works perfectly with picture element in IE11. It's my understanding that object-fit is meant to be applied to the img element instead of picture so that's how I have it configured.  Is there a specific use case or something else I'm missing?

Actually, I'm noticing that this seems to be working now in Edge 17 (released April 30, 2018). I believe a vast majority of people using Edge will likely automatically be on this newer version. Should we go ahead and close this issue?
Here's a little test page I've got with both object-fit (applied to the inner <img> tag) and <picture> with various sources depending on the current breakpoint:

Edge 16 (October 2017) supports object-fit natively, so this polyfill doesn't apply there at all.


Support bevy 0.15

hello :)

Required components look nice :)
I'm a bit swamped with work at the moment but I'll take a look whenever I have some free time.

hey ! After looking more at bevy 0.15, i think im gonna transition to the built-in bevy animation with the new events integration, which is better for my use as i only used simple 2d animation with events.
so this issue can be closed if you want, or just know that nothing is pressing, at least on my side !
Thanks a lot for your reactivity and helpfulness with my issue and PRs ! have a good one

@mirsella Thanks for your help and good luck with your projects :)

bevy_spritesheet_animation 2.0.0 now supports Bevy 0.15


Bug: unpkg field in package.json DOES NOT work

<EMAIL_ADDRESS>{
  "name": "issue-reporter-web",
  "version": "0.0.4",
  "main": "index.js",
  "unpkg": "dist/issue-reporter-web.umd.js",
}

Access<EMAIL_ADDRESS>would redirects to<EMAIL_ADDRESS>by main field rather than unpkg field.

duplicate of #156

There was a bad value in the cache that I cleared. It should be working now. Thanks for the report.


FR: Encode default backend in uv

I have previously floated the idea of uv defining/providing its own backend (see #3957) since the standard library does not provide this critical capability. I think that hatchling is a great build backend to use in the meantime and I am happy to see that it has been adopted as the default backend (#5527) when initializing projects.
I want to float the idea of base85 (or other more efficient) encoding a default backend in uv to bootstrap offline systems with a way to build when utilizing commands such as uv init or uv sync. Inspiration for this can be seen in the get-pip.py bootstrapping script.
Not only do I think that encoding/decoding a blessed backend will aid offline users, but I also think that it could improve performance by eliminating download for the transient build environment for users who elect to use the provided default.
If there are concerns with hatchling and its dependencies bloating the size of the uv binary, flit-core is a great alternative. It seems to be the most minimal modern backend and has no dependencies itself. It supports editable installations and a number of other common tasks.

I’d be happy to implement this myself as a build time job that dynamically downloads and encodes the remote build backend when there’s a new version found.
Feel free to close this as an absolutely terrible idea 😄

There’s a base116 crate that advertises a data size increase of only 1/6 (versus 1/4 for base85) by leveraging UTF-8 over pure ASCII encoding.

I think we'd implement our own build backend before embedding a Python one into uv itself.
Related:

https://github.com/astral-sh/uv/issues/6299

It's a cool idea though. The offline use-case for #3957 is interesting.


[1pt] PR: Add CRS handling to USGS gage processing

Updates USGS gage processing to use the correct projection (determined by whether the HUC is in Alaska or not).
Changes

src/run_by_branch.sh: Added huc_CRS as an input argument for usgs_gage_crosswalk.py
src/run_unit_wb.sh: Added huc_CRS as an input argument for usgs_gage_unit_setup.py and usgs_gage_crosswalk.py
src/usgs_gage_crosswalk.py: Added huc_CRS as an input argument for the run_crosswalk() function and added re-projection steps wherever new data is being read in so that the files are able to be properly merged.
src/usgs_gage_unit_setup.py: Added huc_CRS as an input argument for the Gage2Branch() crosswalking class.

Testing
Tested on 129040101 (CONUS) and 19020301 (Alaska). Before these changes, 12434 gages were found in 12040101 and 0 gages were found in 19020301. With these updates, the same amount of gages were found in 12040101 and 7 gages in 19020301 were found.
Deployment

Needs to be on the 4.5 Build

Issuer Checklist (For developer use)
You may update this checklist before and/or after creating the PR. If you're unsure about any of them, please ask, we're here to help! These items are what we are going to look for before merging your code.

[x] Informative and human-readable title, using the format: [_pt] PR: <description>
[x] Links are provided if this PR resolves an issue, or depends on another other PR
[x] If submitting a PR to the dev branch (the default branch), you have a descriptive Feature Branch name using the format: dev-<description-of-change> (e.g. dev-revise-levee-masking)
[x] Changes are limited to a single goal (no scope creep)
[x] The feature branch you're submitting as a PR is up to date (merged) with the latest dev branch
[x] pre-commit hooks were run locally
[x] Any change in functionality is tested
[x] Passes all unit tests locally (inside interactive Docker container, at /foss_fim/, run: pytest unit_tests/)
[x] New functions are documented (with a description, list of inputs, and expected output)
[x] Placeholder code is flagged / future todos are captured in comments
[ ] CHANGELOG updated with template version number, e.g. 4.x.x.x
[x] Reviewers requested
[x] Add yourself as an assignee in the PR  as well as the FIM Technical Lead

Merge Checklist (For Technical Lead use only)

[ ] Update CHANGELOG with latest version number and merge date
[ ] Update the Citation.cff file to reflect the latest version number in the CHANGELOG
[ ] If applicable, update README with major alterations

I tested the new changes on a Alaska HUC 19020401. The changes correctly creates 'usgs_elev_table.csv' with correct gages within the HUC.
Just to clarify, this PR's score is not to verify the correct application of USGS gages for calibration.


Enforce "ruby" platform when fetching version data

When fetching version data for individual Ruby gems, we don't want to rely on creation order of the gems on RubyGems.org. Instead, we use the standard ruby platform as a sensible default (as we do elsewhere).
Partially addresses https://github.com/oss-review-toolkit/ort/issues/8695.

@sschuberth @fviernau Thanks for the review comments. I had intentionally marked it as a Draft MR in order not to bother you to review yet, while I was sorting out the build etc. :) I guess my intention failed there.
Either way: It seems that there currently isn't a test that fails with the Gemfile changes aside from the obvious one which checks the resulting list.
My question here would be: If I revert all the changes to the Gemfile and Gemfile.lock and keep the change to the URL plus add a comment that references to the corresponding resolve_dependencies.rb change that it mimicks, would the change be acceptable for you even without an explicit test for it?
I just ran it in CI and here's what I'm getting:
Before the change:

After the change:

I guess my intention failed there.

No worries. Sometimes we're just curious and willing to spend time on reviewing draft PRs, well knowing that our comments might become null and void due to ongoing changes.

would the change be acceptable for you even without an explicit test for it?

For me, it would be acceptable, as it's obviously doing the right thing, even if you cannot prove that it makes a difference according to test assets.

After the change:

Interesting that this writes out a smaller file although it has the same number of projects / packages. Mind investigating where the size difference comes from?

The main difference that I can spot is that the issues are captured in the analyzer-result.json:

There are 32 such entries. If I extract a single one and put it into a file, it's roughly 350-400 bytes (depending on the length of the respective project name) => 400 x 32 = ~12 KB. This comes fairly close to the (presumably rounded) 20 KB difference that the output lists.

So I adapted everything, but now we seem to have what looks like random/unrelated test failures…? I'm a bit stuck here. :/

I'm a bit stuck here. :/

No worries. We'll realize this and eventually retrigger flaky tests etc.

So I adapted everything

The commit message still talks about "we". Please reword as described above.


Fixes #645: Timezone Feature & Fixes #649 Change Username Bug

Description
Just added a different way to get the timezone for the lower android API(<24),  instead of using the latest TimeZone class from Android, I use the Timezone class from java that gets the same results.
Fixes #(#645 )
Fixes #649
Type of change
Just put an x in the [] which are valid.

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes.

[ ] ./gradlew assembleDebug assembleRelease
[ ] ./gradlew checkstyle
[X] Used two smartphones to test, one with Android API 21 and the other with 23

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings

Great @malieop 👍


sc-consensus-beefy: graceful support for pallet-beefy reset

BEEFY consensus can be restarted by resetting "genesisBlock" in
pallet-beefy, but we don't want to also reset authority set IDs so
that they are uniquely identified across the entire chain history
regardless of how many times BEEFY consensus has been reset/restarted.
This is why the client now also accepts initial authority_set_id != 0.
BEEFY client now detects pallet-beefy reset/reinit and errors-out and
asks for a restart.
BEEFY client persisted state should be discarded on client restarts
following pallet-beefy reset/reinit.
End result is BEEFY client/voter can now completely reinitialize using
"new" on-chain info following pallet-beefy reset/reinit, discarding old state.
Fixes https://github.com/paritytech/substrate/issues/14203
Fixes https://github.com/paritytech/substrate/issues/14204

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2888889

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2888890


UploadType parameter is not set for uploads

<EMAIL_ADDRESS>on 3 May 2012 at 5:55:
When uploading, e.g.

func (c *FilesInsertCall) Do() (*File, error) {

in drive/v1/drive-gen.go

The uploadType parameter should be set to multipart, like:

uploadType=multipart

<EMAIL_ADDRESS>on 9 May 2012 at 5:17:
Patch to fix the generator, and generated new drive package.

<EMAIL_ADDRESS>on 29 Mar 2015 at 3:28:
uploadType is set to multipart.
Marking as obsolete and closing.

Status changed to Done.


FR,ENH: Separate defaultRole for authenticate and guest user

Introduce new properties

public $defaultAuthRoles = [];
public $defaultGuestRoles = [];

// DbManager::checkAccessFromCache()
if (isset($assignments[$itemName]) || in_array($itemName, $this->defaultRoles) ||
        ($user !== null && in_array($itemName, $this->defaultAuthRoles) || ($user === null && in_array($itemName, $this->defaultGuestRoles)) {
            return true;
        }

Please elaborate.

currently yii\rbac\Manager::$defaultRoles affected to both guest and authenticate user. But, there some page that only allowed for guest user like login, signup, etc. Another page only allowed for authenticate user. Without defaultRoles, mean i must attach role to each user. But using defaultRoles mean, guest also has authority to access the page.

So what's preventing doing a role check currently?

The goal is. I want all authenticate user has the default roles but not for guest user.
So, when i do this in action
if(!$user->can('authenticate')){
    throw ForbiddenHttpException();
}
// ...

then guest user will be rejected to access page. And do this
if(!$user->can('guest')){
    throw ForbiddenHttpException();
}
// ...

Then all loged user (authenticate user) will be rejected.

you want to split defaultRoles per role. Isnt that adding too much complexity?
For example. How should we have to use $defaultGuestRoles. This would imply that the Application knows which role a guest has, which he can't right, because he is a guest? Unless you mean that authenticated users should be able to fake to be a guest, which is still a bit complex and confusing to be adopted as core functionality?

@dynasource What your suggestion?

which role a guest has, which he can't right, because he is a guest?

Think more simple. Lock at login action. She is only can be accessed by guest.
But, the true problem is, current defaultRoles also affected to guest user.

@dynasource What your suggestion?
I dont have a suggestion for you yet, sorry. Within a few weeks I will, when I get to the RBAC parts of my development. Ill come back to you.

Apart from the solution. Why do we actually want to prevent users from looking at the login page? Is it not up to the user to make that decision? Ie. why can't he login into another account he is managing? Ive noticed this restriction in the Advanced App already.

Why do we actually want to prevent users from looking at the login page?

Ok, forget about defaultGuestRoles. How about defaultAuthRoles?

do you really want to add another dimension to the defaultRoles concept?

My issue is more simple. I want all authenticate user has default role (only authenticate, not guest). So, if you have better(more simple) solution, i will take it.

wouldnt a solution with the special role @ (http://www.yiiframework.com/doc-2.0/guide-security-authorization.html) be more appropiate?

I didn't get it. Why don't you use @ for authenticated users and ? for guests as shown in the guide @dynasource linked to?

@ only work for AccessControl. I am using $user->can() to calculate menu list.

So why it's a problem? Guest user doesn't have any roles so it will give false.

So why it's a problem? Guest user doesn't have any roles so it will give false.

I dont think so. This code seem not doing like that https://github.com/yiisoft/yii2/blob/master/framework/rbac/DbManager.php#L157
If role is mentioned in defaultRoles, checkAccess() will return true event the user are guest.

If role is mentioned in defaultRoles, checkAccess() will return true event the user are guest.

This is expected behavior. You can use business rule to exclude non authenticated users from default role.


UX: istioctl register seems to incorrectly imply labels per endpoint

Kubernetes endpoints objects do not support labels.
istioctl register svc ip port -l foo:bar conveys the impression that labels are per endpoint, which is not true. Users need to be made aware that labels are per service and that they need to make sure all invocations of istioctl for the same service need to have identical labels (or that the labels are merged/overwritten).
This requires

appropriately wording the usage printouts in the command
changing the option name to "service labels"
issuing appropriate warnings when labels are being overwritten
Docs explaining the caveats and limitations (no intelligent routing features)

This is a tracking issue for all of the tasks above.
cc @ldemailly @costinm

having docs is always useful
it does apply the labels and annotations to the endpoints (and the service) so no for 2. (the kubernetes behavior can be explained/pointed to)

Endpoint objects do not have labels. Please see https://kubernetes.io/docs/api-reference/v1.7/#endpointport-v1-core
Labels are for the service. Without labels per endpoint, there is no smart routing in pilot. Its just load balancing stuff.

https://github.com/istio/pilot/blob/master/platform/kube/register.go#L129 as proof.. labels and annotations applied to the service object.

https://github.com/istio/pilot/blob/master/platform/kube/register.go#L142

What are service labels used for? AFAIK, we haven't been using them for anything in pilot.

This is getting quite off topic but as we talked about yesterday (and I need to send an email summary) we need to not rely on pods exclusively and start looking for labels/annotations on services / endpoints

need to add unit test (though probably first the e2e one), for unit test , ptr to makeTempClient in https://github.com/istio/pilot/issues/1233#issuecomment-329999170

Issue moved to istio/istio #1448 via ZenHub


Move all the base jobs to base.yaml

Moving all the base jobs to a single file allow us to use anchors. Here irrelevant files always causes job freeze issue.
Keeping base-jobs at one place allow us to manage irrelevant files easily.
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running

/test all

recheck

/lgtm cancel
missing the edpm_multinode base job


Use #[serde(field_identifier)] instead of hand-made workarounds

This attribute doesn't seem to be documented btw...

See also: https://github.com/serde-rs/serde/issues/1701, https://github.com/serde-rs/serde/issues/1238.


Fix grammar in maximumCpuCount string

"your" is possessive
"you're" is an abbreviation for "you are"

technically it's a contraction, not an abbreviation :)

nevermind, i'm wrong again :) - according to wiki

In strict analysis, abbreviations should not be confused with contractions, acronyms, or initialisms, with which they share some semantic and phonetic functions, though all four are connoted by the term "abbreviation" in loose parlance. An abbreviation is a shortening by any method

i see another one in the same paragraph. i'll fix.

This is the generated file that is overwritten at build time. I merged a PR that fixes it in the source (task.json).

Thanks, @ericsciple for the fix, and the wiki-lesson. I learned something too.


Integrate Web Dev Server

Note to self: Started working on this already and am able to launch @web/dev-server from Bazel and have developed a private npm package to self contain the configuration settings (unsure if needed for now).
Next steps are to combine it with an actual demo folder to launch and think about how to generalize this so it becomes trivial to spin up a dev server at different levels i.e. Show me a single component, show me a series of components in a design system or show me the app with everything already built in


find_with_related implicitly adds ordering on left.id

Description
find_with_related implicitly adds an order by on the left table's id. This is extremely unintuitive; for instance:
LeftEntity::find()
  .find_with_related(RightEntity)
  .filter(...)
  .order_by(LeftCol1, Order::Desc)
  .order_by(LeftCol2, Order::Desc)

generates a query that is ordered by (LeftId, LeftCol1, LeftCol2) rather than (LeftCol1, LeftCol2), as might be expected.
Steps to Reproduce

Write a query using find_with_related that includes an order_by after the find_with_related
Either try to execute it, or .build(..).to_string() it
Observe that the order clause has order by left.id asc prepended to it

Expected Behavior
In the example case from the description, I would expect the ordering to be (LeftCol1, LeftCol2).
Actual Behavior
In the example case from the description, the actual ordering is (LeftId, LeftCol1, LeftCol2).
Reproduces How Often
Deterministically.
Workarounds
Placing the order_by before the find_with_related, like so:
LeftEntity::find()
  .order_by(LeftCol1, Order::Desc)
  .order_by(LeftCol2, Order::Desc)
  .find_with_related(RightEntity)
  .filter(...)

produces an ordering on (LeftCol1, LeftCol2, LeftId), which is acceptable. However, the fact that the query needs to be constructed in this manner is very non-obvious.
Reproducible Example
See description
Versions
├── sea-orm v0.12.15 (*)
├── sea-query v0.30.7 (*)
│   ├── sea-orm v0.12.15 (*)

in the past, the consolidator needs the query results to be ordered. this constrait was dropped in https://github.com/SeaQL/sea-orm/pull/1743 but the behaviour has not changed. removing the default order by is a break I want to avoid, but I agree users should be given the preference if they want to override the ordering

I think just calling attention in documentation to the fact that find_with_related introduces an ordering would be valuable for now, since the current behavior is very surprising.
The workaround I noted (placing the order_by calls before find_by_related) isn't terrible or anything—it's just not very intuitive—so maybe even a small note explaining it as an option would probably cover most ordering use cases for the time being.

I totally agree with you, would appreciate if you can make a PR on any of the points above


bug: Error: Could not find haxelib "hxvm-luajit", does it need to be installed?

Everytime I try to compile the newest kade I get
Error: Could not find haxelib "hxvm-luajit", does it need to be installed?
What's this supposed to be? And how do I add it or fix it so I can finally compile my game,

try haxelib git linc_luajit https://github.com/AndreiRudenko/linc_luajit.git
or haxelib git https://github.com/KadeDev/hxvm-luajit

try haxelib git linc_luajit https://github.com/AndreiRudenko/linc_luajit.git
or haxelib git https://github.com/KadeDev/hxvm-luajit

It still gives me the same error shit, even when I installed both of them

try:
haxelib git hxvm-luajit https://github.com/KadeDev/hxvm-luajit

try: haxelib git hxvm-luajit https://github.com/KadeDev/hxvm-luajit

thanks :>


(WIP) Type metadata access by mangled name

Experiment with accessing concrete types with complex instantiation sequences by mangling their names and interpreting with getTypeByMangledName instead of with open-coded accessors. In my experiments this saves about 128KB of code size in the dylibs.

@swift-ci Please benchmark

Cute.

Build comment file:
Build failed before running benchmark.

@swift-ci Please benchmark

Build comment file:
Build failed before running benchmark.

Can we reasonably combine this with the embedded type-descriptor references that we do in the remote-mirrors metadata?  It would avoid some expensive symbol-table lookups and fix any potential problems we might have with local types.

In particular, if we combined direct type-descriptor references with a more efficient type demangler that didn't construct an intermediate demangling tree, it would probably cut the runtime overhead of this to the point of possibly vanishing in the i-cache improvements (assuming we still have one level of caching in the runtime).

@rjmccall Yeah, this already uses the same format as the remote mirror type references, including support for embedded pointers in the mangling to refer to private or local types.

Closing this in favor of a new implementation coming shortly.


Fix proposals list UI

🗃 Github Issue Or Explanation for this PR. (What is it supposed to do and Why is needed)
No issue
✅ Checklist

[x] Github issue details are up to date for people to QA.
[x] I have tested all my changes.

🕵♂ Notes for Code Reviewer
Minor changes to UI
. Made campaign label more visible
. Added a missing line on the tabBar so it looks like figma
. Fix voting label
. Remove tabBar padding.
🙈 Screenshots

👯♀ Paired with
"nobody"

Great job.

Thank you (:


sidebar per_player handling "breaks" with <proc[]> tag in value:

- sidebar set title:test lines:1 "value:<proc[procedurecrap].context[<player.money>]>"<EMAIL_ADDRESS>per_player

Errors with:
[16:37:18 INFO]:  ERROR! Invalid or missing player for tag <player.money>!
[16:37:18 INFO]:  ERROR in script 'PRISONSCOREBOARD'! Tag <player.money> is
                   invalid!
[16:37:18 INFO]:  ERROR! 'player.money' is not a valid number.

Full debug rec for reference pls

http://mcmonkey.org/paste/20322

That's... just a long list of errors with debug disabled throughout.

http://mcmonkey.org/paste/20323

I realized that after I posted it

Now you enabled debug for the wrong script?

http://mcmonkey.org/paste/20324

I turned on the sidebar that wasn't erroring.. blah blah.. that has errors

and forgot the procedure ;-;

http://mcmonkey.org/paste/20330
The proc shouldn't matter, its the .context[] which is trying to process the tag before per_player kicks in

Script:
http://mcmonkey.org/paste/20656
Broken ones are the commented out ones.

@mcmonkey4eva why does this make no sense?
There most certainly is a player in the BukkitTagContext.

Because proc: is being processed before it is going into the sidebar?

This was fixed.


SILOptimizer: Don't optimize initializers for dynamically-sized globals

If the size of a global variable is not known at compile time, we emit
a fixed size buffer together with initialization code when the global
variable is initialized.
Make sure the SIL optimizer does not convert this into a static
initialization, even if the size of the type is known inside the
module where the global is declared, because we don't have a way to
statically initialize something that looks like a fixed-size buffer
to client code.
Fixes https://bugs.swift.org/browse/SR-11709.

@swift-ci Please smoke test

@swift-ci Please test source compatibility

@swift-ci Please smoke test Linux

@swift-ci Please smoke test

@swift-ci Please smoke test macOS


Train Sound issue

BkCommonLib version: Latest
TrainCarts version: Latest
Spigot version: Paper 1.15.2 build 90
Problem or bug: When a train is stopped by setting it's speedlimit to 0 the train still makes (a lot) of noise while standing still as if it's still going.
Expected behaviour:
No moving train sound while the train is not moving.
Steps to reproduce:
Launch a train with the launch sign at a high speed for example 100, further on the track make a speedlimit sign which is set to 0.

Interesting that this bug (still) exists, I thought I patched it.
A workaround is to create a property sign like so:
[+train]
property
sound
false

Marking issue as HW, requires crossed out from WIP list in order to be closed.

For me it works correctly.
BkCommonLib: 1.15.2-v3
TrainCarts: 1.15.2-v2-611
Paper: 1.15.2-165

I'll close this then


Long-term project maintenance

@driverdan Thank you for building and maintaining the project!
Since it appears you may have less time to give the project attention, would you consider adding additional Collaborators to the project so some of the open PRs and issues can be addressed?
Thanks!

Would love to help here...

I would love to see some of the pull-request accepted and published, too :-/

There's finally been a little bit of activity after 2 years of no changes:
https://github.com/driverdan/node-XMLHttpRequest/pull/153
@driverdan would you consider the above?

There's finally been a little bit of activity after 2 years of no changes:

Asking things please on Twitter makes miracles :-)

@driverdan @piranna
Hi there! Are you going to release #153, btw?

I don't have permissions.

@driverdan Would you consider adding additional collaborators to help manage some of the open requests?

I think it makes more sense today to use fetch() API, this project would only make sense to provide compatibility will old projects that still use XHR, and in fact they should be updated.


Add support for Meson build tool

Meson (https://mesonbuild.com/) is a popular building system similar to CMake, used by projects such as GLib (https://gitlab.gnome.org/GNOME/glib/).

I'm not entirely sure where I should do that, would something like this work in .github/workflows/ci.yml?
index e8d74d7..7a2e2e0 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -26,6 +26,10 @@ jobs:
         with:
           distribution: ${{ matrix.distribution }}
           java-version: ${{ matrix.java }}
+      - uses: actions/setup-python@v1
+        with:
+          python-version: '3.x'
+      - run: pip install meson ninja

       - name: Check formatting
         run: sbt scalafmtCheckAll

@mihnea-s yea, just try it out. I also think there is a solid chance that python is preinstalled, so it could be just pip install 🤷

Thanks! That works


Allow specific users registered in the metabase to add data

Use the OpenID Connect endpoint from the metabase to authenticate users and store additional information about those users in the database.

Resolved in https://github.com/building-envelope-data/database/pull/45/commits/2df895627bec795dba8369002cb0e56578868e85 which is a commit in #45

Note completely. In the mentioned commit we store users in the database that are authenticated through the metabase. But we do not use this information yet to authorize database mutations.


models_rank.json issue

Describe the bug
The model_path in models_rank.json is the input model path instead of the optimized model path
Expected behavior
model_path is the optimized model path
Add Olive configurations here.
Olive logs
models_rank.json:
[
    {
        "rank": 1,
        "model_config": {
            "type": "ONNXModel",
            "config": {
                "model_path": "D:\\fang\\cputest\\code\\oaas_models\\deeprank.onnx",
                "onnx_file_name": null,
                "inference_settings": {
                    "execution_provider": [
                        "OpenVINOExecutionProvider"
                    ],
                    "provider_options": [
                        {}
                    ],
                    "io_bind": false,
                    "tuning_op_result": null,
                    "session_options": {
                        "execution_mode": 0,
                        "graph_optimization_level": 99,
                        "intra_op_num_threads": 15
                    }
                },
                "use_ort_extensions": false,
                "model_attributes": null
            }
        },
        "metrics": {
            "latency-avg": {
                "value": 3.88114,
                "priority": 1,
                "higher_is_better": false
            }
        }
    },
    {
        "rank": 2,
        "model_config": {
            "type": "ONNXModel",
            "config": {
                "model_path": "D:\\fang\\cputest\\code\\oaas_models\\deeprank.onnx",
                "onnx_file_name": null,
                "inference_settings": {
                    "execution_provider": [
                        "CPUExecutionProvider"
                    ],
                    "provider_options": [
                        {}
                    ],
                    "io_bind": false,
                    "tuning_op_result": null,
                    "session_options": {
                        "execution_mode": 0,
                        "graph_optimization_level": 99,
                        "intra_op_num_threads": 16
                    }
                },
                "use_ort_extensions": false,
                "model_attributes": null
            }
        },
        "metrics": {
            "latency-avg": {
                "value": 4.08497,
                "priority": 1,
                "higher_is_better": false
            }
        }
    }
]

Other information

OS: Windows,&Linux
Olive version: 0.5.0
ONNXRuntime package and version: all

@xiaoyu-work, could you please help take a look?

That's weird. Mine has correct output model path:
[
    {
        "rank": 1,
        "model_config": {
            "type": "ONNXModel",
            "config": {
                "model_path": "/home/xyz/github/Olive/examples/bert/cache/models/2_OnnxQuantization-1-1431c563dcfda9c9c3bf26c5d61ef58e/output_model/model.onnx",
                "onnx_file_name": null,

But what I can improve here is updating this model_path to the model path in output zip file.

I think that's mean the actual path in the zip file. The model_path will not be valid after the zip file is downloaded in another machine.

The PR had been merged. Now the model_path in models_rank.json is the relative path to zip file. Closed.


[SPARK-29683][YARN] Job will fail due to executor failures all available nodes are blacklisted

What changes were proposed in this pull request?

Check the nodes in spark.yarn.exclude.nodes if exists in Yarn cluster.
Loosen condition of checking if all node blacklisted: give a waiting time before job failed.

Why are the changes needed?
My streaming job will fail due to executor failures all available nodes are blacklisted. This exception is thrown only when all node is blacklisted:
def isAllNodeBlacklisted: Boolean = currentBlacklistedYarnNodes.size >= numClusterNodes

val allBlacklistedNodes = excludeNodes ++ schedulerBlacklist ++ allocatorBlacklist.keySet

After diving into the code, I found some critical conditions not be handle properly:

unchecked excludeNodes: it comes from user config. If not set properly, it may lead to "currentBlacklistedYarnNodes.size >= numClusterNodes". For example, we may set some nodes not in Yarn cluster.

excludeNodes = (invalid1, invalid2, invalid3)
clusterNodes = (valid1, valid2)

numClusterNodes may equals 0: When HA Yarn failover, it will take some time for all NodeManagers to register ResourceManager again. In this case, numClusterNode may equals 0 or some other number, and Spark driver failed.
too strong condition check: Spark driver will fail as long as "currentBlacklistedYarnNodes.size >= numClusterNodes". This condition should not indicate a unrecovered fatal. For example, there are some NodeManagers restarting. So we can give some waiting time before job failed.

Does this PR introduce any user-facing change?
No
How was this patch tested?

Add new UT and update current UTs.
manual test

Test build #113015 has started for PR 26343 at commit dcb7dcd.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/17937/
Test PASSed.

Test build #113015 has finished for PR 26343 at commit dcb7dcd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

cc @srowen

I don't have a strong opinion on this. I think that adding more and more configs are just hacks though.  I feel like these things are proposed to address particular use cases and they do not generalize.

@srowen Yes, the new added config is used to loosen condition of checking if all node blacklisted in some particular use cases in my description. It makes the check more soft but not lead to job fail directly, but not generalize.

@dongjoon-hyun Could you please ping someone familiar with the code?

ping @dongjoon-hyun

Friendly ping for @dongjoon-hyun. I don't have a strong opinion on whether the PR is the right solution, but it'd be great to avoid the application failures during RM failover on YARN.

retest this please.

Test build #116280 has started for PR 26343 at commit dcb7dcd.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/21071/
Test PASSed.

Test build #116280 has finished for PR 26343 at commit dcb7dcd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/116280/
Test PASSed.

Kindly reminder to @dongjoon-hyun

@uncleGen I have checked this on a cluster and I would not use the spark.blacklist.waiting.millis for every case when there is no more nodes to allocate on as this would mix the following two cases:

there are cluster nodes but all the nodes are blacklisted by Spark (waiting is not needed we can stop right away)
there is no available nodes at all because of RM failover (we could wait)

So what about only using the timer when there is no reported available nodes?
Or not using the timer at all. Stopping at the YARN RM failover can be avoided by this few line change:
-  def isAllNodeBlacklisted: Boolean = currentBlacklistedYarnNodes.size >= numClusterNodes
+  def isAllNodeBlacklisted: Boolean =
+    numClusterNodes != 0 && currentBlacklistedYarnNodes.size >= numClusterNodes

I know in this case we would wait unconditionally to RM (as it was before SPARK-16630) but I think this is an operational issue at YARN and we could keep this old behavior.
The few line change was tested with stopping/starting the RM daemon manually for each RM nodes (as on this cluster the auto-failover was on):
$ yarn --daemon stop resourcemanager
$ yarn --daemon start resourcemanager

cc @squito

There's still a similar issue in the proposal to do def isAllNodeBlacklisted: Boolean = numClusterNodes != 0 && currentBlacklistedYarnNodes.size >= numClusterNodes. Since the NodeManagers register with the newly active RM one at a time after the failover, you can wind up with a situation where numClusterNodes is temporarily 1 during the failover (or some other small number). If we happen to have 1 blacklisted node at this time, and we happen to also trigger the isAllNodeBlacklisted check, then we still fail the application.
That said, I think the situation I'm describing should happen quite rarely in practice, so it's still a small change that makes things much better. I'm supportive of that idea. @uncleGen, does that seem reasonable to you?

Friendly ping for @uncleGen on the idea from @attilapiros

@sjrand @attilapiros Sorry for the delay in posting a reply. I will check the idea this week.

@attilapiros Adding double check numClusterNodes != 0 can not actually resolve the issue. Like I and @sjrand have pointed, when active RM switches after failover, it will take a while for NM to RM, then numClusterNodes will be less than 1. This will trigger the isAllNodeBlacklisted check.

Test build #121171 has started for PR 26343 at commit 425fa52.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/25858/
Test PASSed.

Test build #121171 has finished for PR 26343 at commit 425fa52.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/121171/
Test PASSed.

Adding double check numClusterNodes != 0 can not actually resolve the issue.

Adding this condition makes isAllNodeBlacklisted happen quite rarely in practice, although it does not resolve issue in some critical conditions. I makes things much better.

Test build #122500 has started for PR 26343 at commit 79edb32.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27161/
Test PASSed.

Test build #122502 has started for PR 26343 at commit 8981bd1.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27163/
Test PASSed.

Test build #122500 has finished for PR 26343 at commit 79edb32.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122500/
Test FAILed.

Test build #122502 has finished for PR 26343 at commit 8981bd1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122502/
Test PASSed.

cc @attilapiros and @sjrand Take a review please.

The Exception mentioned above also occurs when RM switches with versions 2.4.0 and above.

The Exception mentioned above also occurs when RM switches with versions 2.4.0 and above.

Have you backport this pr?

This PR is not committed but the problem is solved by SPARK-29683, see https://github.com/apache/spark/pull/28606#issuecomment-636900759
Which is missing from 2.4:
╭─attilazsoltpiros@apiros-MBP16 ~/git/attilapiros/spark ‹branch-2.4*›
╰─$ git log --oneline| grep SPARK-29683


chore: add sandbox scripts

#141  👈
main

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

Join @NathanFlurry and the rest of your teammates on  Graphite

Merge activity

Aug 20, 9:41 PM EDT: @NathanFlurry started a stack merge that includes this pull request via Graphite.


feat: Download Buttons only for supported browser

Acceptance Criteria:

eg. Chrome user clicks download for Firefox -> Show tooltip
Update Firefox download version from 2.2.0 to 2.3.0

Close #8


Canonical comparison bug whan rhs contains a multiplication

Bug is located on line 273 but it is not easy to read/debug for me. Please have a look Dimos :)
iv[1] gets removed in the transformation
iv = intvar(-8,8, shape=5)
cons = [iv[2] == (iv[0] * (iv[1])) + 1]
print(cons,canonical_comparison((cons)))

cons should be flattened first, before it is given to canonical_comparison.

ah i see, discovered it when testing choco so the fix there also fixes this


[FEATURE] Add support for AWS Sigv4 request signer

Add support for signed requests via AWS sigv4.

Aws sigv4 is already supported https://github.com/opensearch-project/opensearch-go/blob/main/signer/aws/aws.go.

Also, I've created a PR for using AWS Sigv4 with AWS SDK V2 in case you are searching for that.  Do we need something more than this?

@VijayanB Thanks for pointing it out! I created this as part of a meta issue for clients. Will make sure to mark this as done.

Also, I've created a PR for using AWS Sigv4 with AWS SDK V2 in case you are searching for that. Hopefully it gets approved, but until then you can just put the signer in your codebase.
#116

@matelang Thank you very much for adding this!

Support for AWS Sigv4 is added.


using micro-ROS libraries in Vitis on MicroBlaze platform

Hello everyone,
I am using freeRTOS on MicroBlaze platform and I want to run micro-ROS on top of the freeRTOS.
I saw documentations on integrating micro-ROS with different firmware, but I wanted to use Vitis to include those micro-ROS libraries.
I tried to create micro-ROS app on freeRTOS by creating MicroBlaze custom micro-ROS library, but I got stuck in the CMake and meta files because I wanted Vitis to handle it.
How can I use micro-ROS in freeRTOS based project on Vitis?
Thank you!

Hello, the latest documentation on how to port micro-ROS is: https://docs.vulcanexus.org/en/latest/rst/tutorials/micro/custom_platforms/custom_platforms.html
If you need professional support please contact<EMAIL_ADDRESS>

Hello @pablogs9, I already saw this tutorial, but I didn't understand how to integrate micro-ROS with Xilinx's Vitis.
I am a beginner in micro-ROS and Vitis so maybe I failed to link the tutorial. I would appreciate if you could focus me on the context between the micro-ROS and Vitis in the tutorial.

Sorry, but we do not support Vitis now. If you need us to check the possible integrations and evaluate this port please contact<EMAIL_ADDRESS>

Thank you @pablogs9 for the reply.
I saw several articles that suggest FPGA-ROS integration, in particular on Xilinx processors (that Vitis is the software development) [1].
How can I adapt the FPGA integration with ROS to micro-ROS?


Guard against popping item from empty vector (list_elements_stack)

Fixes https://github.com/newsboat/newsboat/issues/1974

Coverage increased (+0.004%) to 59.168% when pulling 9011df80ca9f5e22426a03c27db720b74d40fdc0 on dennisschagt:fix/issue-1974 into 3870c899a0cc6588259230264195644d5c4669f6 on newsboat:master.

Thanks for the fix! So the issue was caused by closing a li tag twice? Could you add a test for that please? :) (It's fine if the test just crashes; Catch2 doesn't let us do any better).

Thanks for the fix! So the issue was caused by closing a li tag twice?

It looks like it was caused by a double closed <ul> or <ol> tag.
That is also how I got this issue to reproduce in a regression test.

Could you add a test for that please? :) (It's fine if the test just crashes; Catch2 doesn't let us do any better).

Sure, I got a bit distracted, but the requested test is now available.


Wireless & Controller Models UI Tweaks

Add missing + button for Controller Managed Device Groups in the sidebar for consistency. Rename nav entry to just Device Groups.

Under Radio Profile make the help text for channels more explicit Comma-separated list of allowed channel numbers for this radio profile.

Add Regulatory Domain to the list view for Radio Profiles.

In the detailed view for Controllers the column heading for the "Controller managed device groups" should be summarized to just Device Group.

Wireless Networks tab in Controller detail view - table has some extra half row at the bottom and an extra header:

@lampwins Are we renaming "Access Point Groups" to "Device Groups" in the Nav under Wireless?


How to filter a query by a list argument?

Hello there!
I'm trying to use the bult-in filtering mechanism to filter a query, but my field is a list.
Initiative.cs
public class Initiative
{
    public long Id { get; set; }
    public IEnumerable<Tag> Tags { get; set; } // I want to filter Initiatives by Tag.Name
}

Tag.cs
public class Tag
{
    public long Id { get; set; }
    public string Name { get; set; }
}

What I want to achieve is something like this query:
 initiatives(where: { tags_contains: { name: "test" } })
or
 initiatives(where: { tags_name_contains: "test" })
So that I could filter out initiatives that have tags with Name that contains "test".
I followed the docs for Custom Filters but without success. That's what I came up with (it won't build)
public class InitiativeTagFilterType : FilterInputType<Initiative>
    {
        protected override void Configure( IFilterInputTypeDescriptor<Initiative> descriptor)
        {
            descriptor
                .BindFieldsExplicitly()
                .Filter(d => d.Tags) // What should I specify here?
                .AllowEquals().Name("equals").And()
                .AllowContains().Name("contains").And()
                .AllowIn().Name("contem");
        }
    }

How am I supposed to achieve this kind of filter?
Thank you!

Hi @felrugai
To use the filters you have to specify the attribute on the field.
There are two basic principals.
Explicit Binding
You can create your own Filters by extending FIlterInputType<T>. This way you can e.g. only add the Field you like.
Implicit Binding
You can let the filters infer your type. This way all fields are filterable.
Registering Filters
Code First
public class InitiativeType: ObjectType<Initiative>
{
    protected override void Configure(IObjectTypeDescriptor<Initiative> descriptor)
    {
       // Implicit
      descriptor.Field(t => t.Tags).UseFiltering();
       // In case you want to use your own filter type
      descriptor.Field(t => t.Tags).UseFiltering<CustomFilterType>();
    }
}

Or Pure Code First
public class Initiative
{
    public long Id { get; set; }
    [UseFiltering]
    public IEnumerable<Tag> Tags { get; set; }
}

Hi @PascalSenn, thanks for the reply!
By using the Pure Code First attribute aproach, I can only filter the tags inside of an Initiative. My goal is to filter all Initiatives by their tags. For example, I have 10 Initiatives, and 5 of them holds a list of Tags which have an Name of "GitHub".   I would like to query only those 5, like
initiatives ( where: { tag_name_contains: "GitHub" } ) {}

I think this image can show in details what I mean.
That said, how could I implement an CustomFilter which I believe that would use LINQ to get only the Initiatives that have at least one tag of name "GitHub"?

@felrugai on which version are you? 10.3.x?

@felrugai on which version are you? 10.3.x?

Yes! All my packages are at v10.3.2

OK, that is why. In V11 previews the filters are much more powerful and let you do all those things.

Nice! Im just updating my packages to the prerelease versions. How could I implement this specific need in the V11 preview?

With v.11.0.0-preview.74 I could accomplish what I wanted with:
initiatives(where: { tags_some: { name_contains: "GitHub" } })
Thanks!


[Publisher] Error message when a user with creator role edits the subscriptions page of a published API

When a user with create only permission tries to edit the 'Subscription availability' at the subscription page of a published API, an error message is shown as below:

Subscription availability UI is disabled for a creator when the api is in PUBLISHED state.


iOS input fields & keyboard popup behaviour unacceptable

Ionic version:  (check one with "x")
[ X ] 2.x
I'm submitting a ...  (check one with "x")
[ X ] bug report
Current behavior:
On iOS:
Erratic behavior of keyboard show/hide when navigating forms. The keyboard only stays up when tapping (or scrolling) on an actual input field, resulting in a completely unacceptable user experience. It's virtually impossible to properly fill out and navigate/scroll a longer form.
Expected behavior:
I expect the keyboard to either stay up all the time until dismissed by the user, eg via a 'Done' key.
OR
NOT to  hide the keyboard when scrolling (no matter what element may be panned on)  and NOT hiding it when tapping anywhere inside a form.
Steps to reproduce:
Any simple form will do which includes structures such as:
<ion-item>
    <ion-label fixed>Username</ion-label>
    <ion-input type="text" value=""></ion-input>
</ion-item>

Other information:
I've tried with or without these config settings:
ios: {
  scrollAssist: false,
  autoFocusAssist: false
}

but they make things even worse when active, randomly hiding content when the keyboard pops up.
Cordova CLI: 6.5.0
Ionic Framework Version: 2.1.0
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.1.4
ios-deploy version: 1.9.1
ios-sim version: 5.0.13
OS: macOS Sierra
Node Version: v6.10.0
Xcode version: Xcode 8.2.1 Build version 8C1002

Murphy's Law:
I just found the culprit, it's in the native-input.ts code, see below.
Can we change the code to accept
a) accept a custom class to prevent the unnecessary blur() (next to input-cover)
b) can we have the tapped.classList.contains() part be looking also at parent elements?
If I artificially bypass the below check and hence avoid the blur() the form / input experience increases 1000 fold.
Thank you for the consideration -
NativeInput.prototype._focus = function () {
        var self = this;
        self.focusChange.emit(true);
        if (self._blurring) {
            // automatically blur input if:
            // 1) this input has focus
            // 2) the newly tapped document element is not an input
            (void 0) /* console.debug */;
            var unregTouchEnd = this._plt.registerListener(this._plt.doc(), 'touchend', function (ev) {
                var tapped = ev.target;
                if (tapped && self.element()) {
                    if (tapped.tagName !== 'INPUT' && tapped.tagName !== 'TEXTAREA' && !tapped.classList.contains('input-cover')) {
                        self.element().blur();
                    }
                }
            }, {
                capture: true,
                zone: false
            });
            self._unrefBlur = function () {
                (void 0) /* console.debug */;
                unregTouchEnd();
            };
        }
    };

Until we can more flexibly control the blurring of input fields, this might help a lot of folks:
ios: {
   inputBlurring: false
}

Thanks for opening this issue, we will look into this.

@olivermuc I'm experiencing the same issue with my app. Where exactly did you put that config?

@pacMakaveli the inputBlurring one, as mentioned above. Add it to the ios section within the config object.

Hey guys is there any update on this issue?
Our application is un-usable on long forms as the page jumps up and down. Short of that is there a patch we can apply?

The keyboard still stays even if the focus is moved out of the input
<ion-item>
    <ion-label fixed>Chicken weight</ion-label>
    <ion-input type="text"></ion-input>
 </ion-item>
<ion-item>
    <ion-label fixed>Chicken Head</ion-label>
    <ion-input type="text"></ion-input>
</ion-item>
<ion-item>
    <ion-label fixed>Chicken Fingers</ion-label>
    <ion-input type="text"></ion-input>
 </ion-item>


How to configure this container to send e-mail using SMTP

One important feature of Wordpress CMS is the capability to send e-mail to users
How to I can configure this container to send e-mail using SMTP ?
Tanks in advance

?


Silent notifications on android

Hi,
I'm trying to push a silent notification which seems to work on ios (standalone expo 42 app) but the same doesnt seem to work on android (also standalone expo 42 app).
from exponent_server_sdk import (
    DeviceNotRegisteredError,
    PushClient,
    PushMessage,
    PushServerError,
    PushTicketError,
)

extra = {'content-available': 1}

// this doesnt raise notification on iphone, as expected
PushClient().publish(PushMessage(to=ios_token,data=extra))

// both raise empty notification on android 9
PushClient().publish(PushMessage(to=android_token,data=extra))
PushClient().publish(PushMessage(to=android_token))

What am I doing wrong here ?

is this still an issue? I'm not sure I've seen this before


Add ipmi firmware check to Pre-Reqs

After an issue with virtual media, I was guided to the section of the openshift docs that indicates the supported firmware revisions for ipmi related things. Per a request from @akrzos I am adding that link here.
https://docs.openshift.com/container-platform/4.14/installing/installing_bare_metal_ipi/ipi-install-prerequisites.html#ipi-install-firmware-requirements-for-installing-with-virtual-media_ipi-install-prerequisites
The short version is that metal3 has a minimum supported firmware rev for idrac and iLO.

#439 addresses this


Can I easily put text on objects

This is just a question. Sorry if creating an issue is not the appropriate thing to do. I want to create an animation of word tiles that behave like rigid bodies. So move around bouncing off each other. But I need to be able to easily populate the tiles with different strings. Would like to use matter.js to manage the movement. Is this possible?

Sure you can render the world how ever you like, please see the wiki on rendering.


feat: use BUILD_HASH if present; otherwise git rev-parse

allows the environment to supply the commit hash

Coverage remained the same at 100.0% when pulling bbe093ed86dc99b6311335083862a866d7887597 on one-way-or-another into 330fe5c0f1a0f1e58a605d9c3a4b18c4744172fa on master.


rows and columns are swapped in TerminalDimensions (passed to Pseudoterminal)

VS Code Version: 1.56.0-insider, Commit: 394a2e4779ff767e2761a59a51b15848dcd24173
OS Version: MacOS 11.2.3

Steps to Reproduce:
I have an extension that creates a task with a Pseudoterminal. The initialDimensions that are passed to the open call, as well as the dimensions that are passed to setDimensions shortly thereafter, both have their rows and columns fields swapped.
This has started to happen with yesterday's insiders build; it used to work fine before.
Possibly related to https://github.com/microsoft/vscode/issues/107239 (I suppose).

Does this issue occur when all extensions are disabled?: N/A (can't test without my extension)

I think this is caused by 8e183f0d2ed6, and I guess this patch might fix it:
diff --git a/src/vs/workbench/api/browser/mainThreadTerminalService.ts b/src/vs/workbench/api/browser/mainThreadTerminalService.ts
index 54166bace62..49ad9f65a50 100644
--- a/src/vs/workbench/api/browser/mainThreadTerminalService.ts
+++ b/src/vs/workbench/api/browser/mainThreadTerminalService.ts
@@ -132,7 +132,7 @@ export class MainThreadTerminalService implements MainThreadTerminalServiceShape
 			strictEnv: launchConfig.strictEnv,
 			hideFromUser: launchConfig.hideFromUser,
 			customPtyImplementation: launchConfig.isExtensionCustomPtyTerminal
-				? (id, rows, cols) => new TerminalProcessExtHostProxy(id, cols, rows, this._terminalService)
+				? (id, cols, rows) => new TerminalProcessExtHostProxy(id, cols, rows, this._terminalService)
 				: undefined,
 			extHostTerminalId: extHostTerminalId,
 			isFeatureTerminal: launchConfig.isFeatureTerminal,
diff --git a/src/vs/workbench/contrib/tasks/browser/terminalTaskSystem.ts b/src/vs/workbench/contrib/tasks/browser/terminalTaskSystem.ts
index ad4df1a6304..7699679e10f 100644
--- a/src/vs/workbench/contrib/tasks/browser/terminalTaskSystem.ts
+++ b/src/vs/workbench/contrib/tasks/browser/terminalTaskSystem.ts
@@ -1164,7 +1164,7 @@ export class TerminalTaskSystem implements ITaskSystem {

 		if (task.command.runtime === RuntimeType.CustomExecution) {
 			this.currentTask.shellLaunchConfig = launchConfigs = {
-				customPtyImplementation: (id, rows, cols) => new TerminalProcessExtHostProxy(id, cols, rows, this.terminalService),
+				customPtyImplementation: (id, cols, rows) => new TerminalProcessExtHostProxy(id, cols, rows, this.terminalService),
 				waitOnExit,
 				name: this.createTerminalName(task),
 				initialText: task.command.presentation && task.command.presentation.echo ? `\x1b[1m> Executing task: ${task._label} <\x1b[0m\n` : undefined,

But I only had limited time to investigate, so I'm not totally sure.
Cc: @connor4312

Oops, thanks for the issue!


Empty result from question 7e6d2522-795d-42ac-84a0-ab6d4842e262

Hi, The result of question is empty
The title is: big react projects
Generated SQL is:
SELECT
  `gr`.`repo_name`,
  `gr`.`stars`,
  `gr`.`forks`
FROM
  `github_repos` AS `gr`
WHERE
  `gr`.`primary_language` = 'React'
ORDER BY
  `gr`.`stars` DESC
LIMIT
  20

Fixed by prompt React projects: LOWER(description) LIKE LOWER('%react%')


Make c-ares resolver unit tests runnable under python3

This resolves internal issue b/159243208
In python2, <string object>.split(<bytes object>) worked, but in python3, this throws an exception complaining that the parameter to split "must be string or None".
The twisted DNS module makes split calls on objects with byte parameters (https://github.com/twisted/twisted/blob/121c98e006a31750661107d390ec2dc4ffe28e8a/src/twisted/names/dns.py#L245), thus when using the twisted DNS module in python3, we can no longer configure DNS records as strings. This PR makes sure to only configure DNS records as bytes objects.

Thanks for the quick review!

Do we still care about Python2? Do we plan to run them in Python3 in OSS as well?

I'm actually not too clear about the status of OSS tests and the Python 3 migration, so I kept this test also running under Python2. I do want to continue running this test in OSS -- I figure this PR allows for that independently of where we are in the migration.
However, internally, I plan to follow up this PR with a change to only run this under python3.

artifact build macos: https://github.com/grpc/grpc/issues/24986


feat(BILL-CPC-1055): add update bill for admin

JIRA: https://champlainsaintlambert.atlassian.net/browse/CPC-1055
Context:
Implemented update bill feature to allow admins to update a feature.
Does this PR change the .vscode folder in petclinic-frontend?:
No it does not
Changes

Implemented update method in BillController in the api-gateway
Implemented a new update form page
Implemented API request to update in the frontend
Implemented 2 tests for the controller method (one positive, one negative)

Before and After UI (Required for UI-impacting PRs)
Before

After:

Be sure to run 'npm run lint -- --fix' !


[#noissue] Add gRpc configuration

Codecov Report

Merging #5872 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5872      +/-   ##
==========================================
+ Coverage   40.77%   40.77%   +<.01%
==========================================
  Files        2950     2950
  Lines       88694    88694
  Branches    11668    11668
==========================================
+ Hits        36161    36168       +7
+ Misses      49396    49390       -6
+ Partials     3137     3136       -1

Impacted Files
Coverage Δ

...p/pinpoint/rpc/stream/StreamChannelRepository.java
75% <0%> (-10%)
:arrow_down:

...ofiler/sender/StandbySpanStreamDataSendWorker.java
80.59% <0%> (-2.99%)
:arrow_down:

...navercorp/pinpoint/rpc/common/SocketStateCode.java
82.92% <0%> (-2.44%)
:arrow_down:

...pinpoint/profiler/monitor/DeadlockMonitorTask.java
18.6% <0%> (-1.17%)
:arrow_down:

...point/rpc/client/DefaultPinpointClientHandler.java
72.4% <0%> (+0.32%)
:arrow_up:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
80.28% <0%> (+1.05%)
:arrow_up:

...java/com/navercorp/pinpoint/rpc/DefaultFuture.java
74.16% <0%> (+1.66%)
:arrow_up:

...om/navercorp/pinpoint/rpc/codec/PacketDecoder.java
59.57% <0%> (+2.12%)
:arrow_up:

.../profiler/sender/StandbySpanStreamDataStorage.java
76.27% <0%> (+3.38%)
:arrow_up:

...vercorp/pinpoint/rpc/packet/ClientClosePacket.java
66.66% <0%> (+11.11%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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