Mat-Table text vertical alignment wrong in IE 11 browser

Bug, feature request, or proposal: vertical text alignment is always top in IE 11 browser. unable to control the alignment.
What is the expected behavior? Need vertical text alignment to be middle in Mat-table in IE 11 browser
What is the current behavior? vertical text alignment is always top in IE 11 browser
What are the steps to reproduce?
Check the official mat-table link below.
https://material.angular.io/components/table/overview
What is the use-case or motivation for changing an existing behavior?
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
IE 11 browser. Angular version 5
Is there anything else we should know?

flexbox and min-height not working together in IE, you have 2 options:

Set a fixed height for rows:

.mat-header-row, .mat-row {
    min-height: 0;
    height: 48px;
}

Add a flex parent, so it will use the min-height set on children:

.mat-table {
    display: flex;
    flex-direction: column;
}

Will be fixed by https://github.com/angular/material2/pull/6820.

Thank you.

I solved this problem with this style for IE. Work texts with badges.
@media all and (-ms-high-contrast: none), (-ms-high-contrast: active) {
  /* IE10+ CSS styles go here */
  .mat-header-row, .mat-row {
    -ms-flex-align: stretch;
  }
}


Include LICENSE in the release packages

Feature Request
The MIT license is a custom license as it includes a copyright notice.
It should therefore be included in the release packages.
Solution
This patch should fix the .github/workflows/release.yml to include LICENSE in the linux package.
70c70
<           tar -czvf zork-linux.tar.gz ./dist/zork++
---
>           tar -czvf zork-linux.tar.gz ./dist/zork++ ./LICENSE

A Similar change should be made to release-config/windows-installer-zork.iss in the [Files] section

Hello @foip
Want you to open a PR with the changes?


Maps 6.8.0 bump

Part of the 6.8.0 release of the Maps SDK

I'm going to go ahead and merge this in preparation for the Localization Plugin release later today.


[$1000] splitting bill with phone account shows @expensify.sms

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

Action Performed:

login with an sms account
split bill with a few members
Check the who paid field's subtitle text

Expected Result: @expensify.sms should not be shown
Actual Result:
@expensify.sms should is shown
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

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

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

Version Number: 1.3.41-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/050219d3-c177-41b7-b2a2-92168126c7dc

Expensify/Expensify Issue URL:
Issue reported by: @chiragxarora
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688810314872429
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0128597348b75205c2
Upwork Job ID:<PHONE_NUMBER>742786048
Last Price Increase: 2023-07-22

PROPOSAL
Please re-state the problem that we are trying to solve in this issue.
splitting bill with phone account shows @expensify.sms
What is the root cause of that problem?
Root cause of the issue is that in split bill flow, when we are fetching the payee's personal details by the method getIOUConfirmationOptionsFromPayeePersonalDetail, we have not formatted the text and alternate text which we end up showing in <MoneyRequestConfirmationList/> > <OptionsSelector/>
https://github.com/Expensify/App/blob/60f48175c9e3342ef192d878218820e5ed5a76cd/src/libs/OptionsListUtils.js#L878-L894
What changes do you think we should make in order to solve the problem?
We need to format the alternate text and display name fields using formatPhoneNumber() method in LocalePhoneNumber util
What alternative solutions did you explore? (Optional)
We can find other inconsistencies (if any) and fix them all at once

Commented in Slack

Confirmed @expensify.sms is not expected to be shown anywhere in app

Proposal
Please re-state the problem that we are trying to solve in this issue.
If the payee is logged in using phone number, then @expensify.sms should not be displayed while splitting bill after login id.
What is the root cause of that problem?
The login information is displayed plainly from props without formatting and as it has the format <phone<EMAIL_ADDRESS>What changes do you think we should make in order to solve the problem?
In the function getIOUConfirmationOptionsFromPayeePersonalDetail we need to provide check for if there is phone number(sms) based login, then we need to format that phone number by removing @expensify.sms in another variable and then put it in alternateText.
https://github.com/Expensify/App/blob/cd851d0fbd901c70a32220472104ffc887639f2a/src/libs/OptionsListUtils.js#L905-L921
What alternative solutions did you explore? (Optional)
NA

One note to the internal engineer eventually assigned - Lets double check if we're actively removing @expensify.sms from the front end or the back end across the board just to be 100% sure

Friendly reminder to review the existing proposals @0xmiroslav 👍

Waiting on @0xmiroslav's review of the existing proposals, Melvin

Friendly bump @0xmiroslav

Sent a friendly bump to @0xmiroslav in Slack DM for their follow up here 👍

Coming from @0xmiroslav's request, I will help review as C+

@stitesExpensify can you please assign me too?

Bump on the BZ steps @situchan

Payment summary:
Reporting: @chiragxarora $250 (Upwork)
C: @chiragxarora $1500 (Upwork)
C+: @situchan $1500 (Upwork)

Paid!

Regarding BZ checklist:
This is minor inconsistent display issue specific to phone users and does not affect any app logic including split bill.
No need regression test.

Great, thanks for confirming @situchan - All done here!


Fix signed integer overflow issues

It seemed that the consensus in #313 was that integer arithmetic should be two's complement and there should be no warning for overflows. I have implemented that by converting values to unsigned types when appropriate. These changes will mostly preserve existing behavior, except for a few cases that were being handled incorrectly before.
The case of dividing INT_MIN by -1 previously resulted in a CPU exception and program termination. Now, that case is detected and results in a warning and a value of INT_MIN.
Similarly, INT_MIN % -1 would have resulted in a CPU exception. Since this is a mathematically valid operation with a result of 0, it now simply gives that result without a warning.
I noticed that in rpn.c, there were attempts in certain operation handlers to validate the nVal members of the source expressions even when the expressions may have been relocatable expressions with meaningless numbers for the nVal member. This could have caused spurious errors/warnings, so I made those handlers confirm that isReloc is false before validating nVal.
Also, integer constants that are too large now result in a warning. The post-conversion values have not been changed, in order to preserve backward compatibility.
Fixes #313.

As I mentioned in another PR, the information of the PR message should be in the commit.

Updated commit message.


pyinfra: 2.9.2 -> 3.0

Description of changes
Unbreak and update pyinfra.
pyinfra is currently broken, as building the current version fails with the following error for all Python versions >= 3.12:
ModuleNotFoundError: No module named 'distutils'

The deprecated usage of distutils was already dealt with in pyinfra with https://github.com/pyinfra-dev/pyinfra/pull/1096.
However, since no 3.x release including these changes is ready, I propose updating the package from 2.9.2 to 3.0.beta2 as this beta release seems already very stable.
pyinfra 3.0 just got released so I updated the PR. Find the changelog here.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

@mweinelt Applied all your suggestions and squash commits. Thanks!

Looks like the bloop commit does not belong in here, does it?

You're right. I accidentally authored it during rebase.

Superseded by #333625.


[v6.rc1] field array validation problem

For the field array example in the examples doc, validation seems doesn't work very well.

displaying errors when the input is not empty

not able to display error: No more than five hobbies allowed

when clicking submit with only club name filled in, no able to submit(which is correct), but also not having error: 'At least one member must be entered'. This can be quite confusing

Can confirm that validation seems to have been broken in rc1. I have a reduxForm with a validate function. The validate function returns an object that looks like:
{ email: [ 'This is not a valid email address'] }

but when I call fields.map in the component I'm passing to FieldArray, the error prop that gets passed along with the emails[0] member is undefined instead of my error string.

I can also confirm the issue as per @stevengoldberg's description. For me it's happening in rc2.
The fields.error prop is always set to 'undefined'.
Within the example the line {fields.error &&
{fields.error}
} never renders.

Should be fixed in 6.0.0-rc.3.


Int64

I have another one to report (hope still a quick one). Int64 fails but if we do Int().Int64() than it's OK:
func TestDecInt64(t *testing.T) {
	x := New(10240000000000, 0)
	x.Mul(x, New(976563, 9))
	if x.Int64() !=<PHONE_NUMBER>0 {
		t.Error("error int64: ", x.Int64(), x.Int().Int64())
	}
}

decimal_test.go:187: error int64:  -844673895<PHONE_NUMBER>0

this is expected behavior. Int64 is undefined if x cannot fit into an int64. I suppose I could (internally) call Int().Int64() if x is larger than an int64...

fixed in 1.0


Added back data pack.

Checking to see if it causes an issue with CI.

Now builds correctly in CI and through running Visual Studio.  Should be good to do.

Have you checked that the files are valid? IIRC currently the infrastructure uses the recreated dat packer, which I think still had some diffs with the original one.

I will take a little more them then and double check that the dat files made by the CI work the same.

I checked that the ones in the artifact loaded.  It turns out I have been using the packer from dependencies for a while now and haven't had any issues.  Is there anything in particular you think I should test?

I just vaguely remember doing a bitwise comparison between our datpacker and the one that relies on tio.dll and seeing some differences. It could be that:

It got fixed in the meantime (by @shartte), so there are no more differences.
Errors are still present, but are subtle.
Differences exist, but they are meaningless.

So I guess the first step would be to check if there's still any diffs :)

Locally you mean?
Just to state the obvious, does it have tio.dll and the zip dll in the same folder?

Ah, one thing I can think of - which version of tio.dll are you using? It needs the one dated 2004-10-12 (CRC2 = 2E1CB46B, available in GoG, not the patch2 version)

The dll is indeed dated 2005-10-12.

Alright i finally got around to comparing the results myself. Looks ok (as far as i can tell the diffs are just due to uninitialized data in fixed size null terminated strings, which should be of no consequence).

Thanks a ton for doing that!  This should make things easier going forward.


Add Socratic 29

Topics and links can be pushed directly to this branch, or added as comments/review.

cool to explore new things, links on bitvm and runes the new ordinals thing on my side: https://github.com/ariard/londonbitdevs-website/commit/befb45669fdc5e8a81b200b8e67bce1c677b17b6

Thanks @ariard, cherry-picked your commit. FYI I've given you write access on this repo so could add it as a remote.

I was doing redundant work :facepalm: just a few additions

Cleaned up, RFM @glozow

I added links to the bips repo and reorganized the commits. Merging now.


fix: footer of pomodoro page

Closes: #1199

Great! Thanks

did u change the colour of pomodoro decscription text? please correct it
I am reverting this pr back
nothing is visible due to the grey colour

did u change the colour of pomodoro decscription text? please correct it I am reverting this pr back nothing is visible due to the grey colour

@akshitagupta15june this was a level 1 issue which was about matching tha footer with other page.
Now what else are you talking about?
I guess you are matching someone else issue with me.

If you want I can create a new issue & pr for it but why would you revert this pr it was just for footer? @akshitagupta15june ?
Is there still any confusion?

Yes but as you change the colour of text in footer it Changed the pomodoro description and content colour

When a solving a problem it create another problem

ok sure

will fix it


Installation issue

Installation should be from:
yarn add @meedwire/react-native-video-manager

Thanks for the observation, I'm working on improvements, migrating all modules to c++


All unit tests for VisualRecognition and AssistantV1

I got most of the major paths covered in VisualRecognition+UIImage and VisualRecognition+CoreML, but there were a couple of things I was unable to mock.
I created a new VisualRecognitionTestUtilities.swift file so that we don't have to repeat resource loading between the unit tests and integration tests.

@mkistler I added AssistantV1 tests to this PR just in case so I don't lose my 1800 lines of tests over the weekend. They are quite consistent, so looking at just a few of them will give you a good idea of how they work.
I'm really glad you took the initiative to change all models to be Codable, because it made testing WAY easier!


Ignoring properties in an owned type using fluent API

How should one ignore properties in an owned type using fluent call? I have figured out how to do this with attributes by adding [NotMapped] to the property, but not how the same is done using Fluent API.
In EF 6 I was able to ignore properties on a complex type like this:
modelBuilder.ComplexType<StreetAddress>().Ignore(a => a.City);
In EF Core 2.2, it is possible to define the type but not continue with Ignore:
modelBuilder.Owned<StreetAddress>();
The by-design restrictions tell me I cannot do this:
modelBuilder.Entity<StreetAddress>().Ignore(a => a.City);
I think it would be helpful with a section in the documentation of Owned Entity Types about ignoring properties in an owned type using both attributes and fluent API.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d02e6748-42e0-000a-4945-42cd05a0b875
Version Independent ID: e3effbd0-07f2-423d-4032-1722e75adb37
Content: Owned Entity Types - EF Core
Content Source: entity-framework/core/modeling/owned-entities.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @AndriySvyryd
Microsoft Alias: ansvyryd

Currently it's only possible to do for a specific reference.
modelBuilder.Entity<Order>().OwnsOne(
    o => o.ShippingAddress,
    sa =>
    {
        sa.Ignore(p => p.City);
    });


Easiest way to move the detail description of a class to the top?

Hi @jothepro ! 😸
When documenting a class, I would like the @details section to show just after the @brief section (instead of having it at the very end of the page). I know I can mess up the doxygen layout by generating a template file controlling the layout of the generated documentation with doxygen -l layoutFileName.xml (see details here) but I am worried it could mess doxygen-awesome in an unforeseen way?

I cannot give you advice about that, I never tried it. Can you let me know in case it comes with unwanted side effects?
I don't think there is another way to move the section without editing the layout file, but I am not an expert on that.

Closing this issue for now.
Please re-open if there are any learnings that you would like to share or if you had problems with the CSS when changing the layout.


Report Channel "Elasticsearch"

Describe what you would like to achieve
It would be great, if all the Report Data can sent directly to Elasticsearch. Of course, it is possible to fetch the logfile, and send all the data via local logstash instance. But direct "Link" to Elasticsearch would be great.
The BGPalerter Data in Elasticsearch would be a great place for all the SIEM aggregations! Together with Network Host Logs, BMP Data, would be a fancy solution.
Describe why the current solution (if any) is not satisfactory
Connect a local logstash instance to report.log, and send it to Elasticsearch
Provide an example
  - file: reportES
    channels:
      - hijack
      - ...
    params:
      host: _ES_HOST_
      port: _PORT_
    templates:
       ...i think not needed...

Your information
Adrian - AS15576
If Elasticsearch is not on your roadmap, i can try to contribute!
Please let me know!

Hi @cadirol,
I think this is a great idea! BGPalerter could send both alerts and BGP data (that triggered alerts) to Elasticsearch.
A good example to start with could be reportKafka.
It is currently not in the roadmap, but I bet it would be relatively easy to implement with something like this. Please, contribute, I will help you.

Hi
I just did the Elasticsearch export today with a lokal Logstash instace running.
Connected the rReportSyslog to a local Logstash port, and shipped the data with some Grok magic to Elasticsearch.
... It is working, but yes, you need logstash running.
I will have a look on nmp. ... nmp is quite new for me - i'm more in Python. But it would be a great adventure! :-)
I will let you know.
Cheers
On Fri. 3. December 2021 16:24 CET, Massimo Candela @.***> wrote:

Hi @cadirol,
I think this is a great idea! BGPalerter could send both alerts and BGP data (that triggered alerts) to Elasticsearch.
A good example to start with could be reportKafka.
It is currently not in the roadmap, but I bet it would be relatively easy to implement with something like this. Please, contribute, I will help you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.


[OTHER]: Transfer all list of arrays to another folder

What would you like to share?
The current pages are cluttered with a lot of lists of arrays. I would like to request that all of these arrays be transferred to a separate folder. This will help to keep the main pages clean and organized.
We can create a utils/queries folder to store all the arrays.
Code example
We can create a separate folder to store all of those types of arrays

Additional information
If all sounds good, I would like to work on it.

Thank you for raising but this is not something that we are looking to change at the moment given our roadmap, so I will close this for now.


Packageservice error

I'm getting this error when trying to create a migration:
ERROR (5.4.2+00453)

variable [PACKAGESERVICE] doesn't exist

F:\CommandBox\cfml\modules\commandbox-migrations\models\BaseMigrationCommand.cfc
: line 71
69:
70:         // Check and see if box.json exists
71:         if( !packageService.isPackage( directory ) ) {
72:             return error( "File [#packageService.getDescriptorPath( director
y )#] does not exist." );
73:         }
called from F:\CommandBox\cfml\modules\commandbox-migrations\models\BaseMigratio
nCommand.cfc: line 11
called from F:\CommandBox\cfml\modules\commandbox-migrations\commands\migrate\cr
eate.cfc: line 16
called from F:\CommandBox\cfml\system\services\CommandService.cfc: line 345
called from F:\CommandBox\cfml\system\services\CommandService.cfc: line 139
called from F:\CommandBox\cfml\system\Shell.cfc: line 787
called from F:\CommandBox\cfml\system\Shell.cfc: line 607
called from F:\CommandBox\cfml\system\Bootstrap.cfm: line 160

Hi, @Tropicalista. Is this still an issue?  I haven't seen or heard of this from any other users.

You can close. Thanks


Inclusion of FaceNet TFLite model to TF-Hub

FaceNet has been one of the most efficient models to produce face embeddings from a picture. Inclusion of this model in TensorFlow Hub would enable developers, especially working on face detection/classification systems, to leverage the ability of a FaceNet model to generate unique embeddings ( N-dimensional vectors ).  Different faces could be identified by comparing these embeddings using a suitable metric.
Also, the FaceNet can be quantized, so as to achieve a reduced file size and low latencies on edge devices. There could be numerous use-cases wherein the model could be used in Android/iOS apps. I have demonstrated one such use-case effectively in an Android app in this project.
It will be super helpful for both TF and Android/iOS developers to have a FaceNet model featured at TensorFlow Hub in the TFLIte format.
I would request the TF-Hub team to consider this inclusion.

Do I need to sign the Contributor License Agreement now? Please let me know, this is my first contribution to an open-source project.

@shubham0204,
Yes, signing CLA is mandatory for contributions. Please refer this link for details on CLA. Thanks!

@rmothukuru Thanks for the confirmation. I have signed the CLA. Please tell more on how I can publish the FaceNet model on TF-Hub.

Hi @shubham0204, if you want to contribute a model to www.tfhub.dev you can have a look at the publishing guide at https://www.tensorflow.org/hub/publish.
tfhub.dev content was recently migrated to the https://github.com/tensorflow/tfhub.dev which is now the correct repository to contribute PRs related to tfhub.dev. Let me know if you encounter any problems.


New regex implementation.

Out with the old, in with the new!  This resolves #16.

Eh. We're just going to merge it.  I'm sure the tests will pass :)


Improved watering can functionality

If permanent farmland hydration is disabled, watering cans should still be able to normally hydrate farmland (this would be useful for using the mod along with moisturization). Additionally, you should only be able to place it when sneaking and using it on crops should hydrate the farmland beneath them.

Good point, should be pretty simple to implement! I'll comment here again once I do that :grin:


Handle multiple profiles

For example, if we want to switch from one connection to another, we shouldn't have to do
SetUser
SetPassword
SetHost

Instead we should define a dictionary in this kind of format
let connections = {
    'dev': {
        'user': joe,
        'password': testing
        'host': localhost
    },
    'live': {
        'user': joe,
        'password': fre5t4543"&
        'host': <IP_ADDRESS>
    }
}

Then it should simple be a call like
SQHSwitchConnection live
SQHSwitchConnection dev

https://github.com/joereynolds/SQHell.vim/commit/33483350d6256f53f8074388aeefa3d8908f3c77


cmd/compile: the loong64  intrinsic for CompareAndSwapUint32 function needs to sign extend its "old" argument.

What version of Go are you using (go version)?

$ go version
go version devel go1.20-b16e94d13d Mon Dec 12 23:10:52 2022 +0000 linux/loong64

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?

package main

import (
	"fmt"
	. "sync/atomic"
)

const (
	_magic = 0xfbcdefaf
)

func main() {
	var x struct {
		before uint32
		i      uint32
		after  uint32

		o uint32
		n uint32
	}

	x.before = _magic
	x.after = _magic

	for t := uint32(0x7FFFFFF0); t < 0x80000003; t += 1 {
		x.i = t + 0
		x.o = t + 0
		x.n = t + 1

		if !CompareAndSwapUint32(&x.i, x.o, x.n) {
			panic(fmt.Sprintf("should have swapped %#x %#x", x.o, x.n))
		}

		if x.i != x.n {
			panic(fmt.Sprintf("wrong x.i after swap: x.i=%#x x.n=%#x", x.i, x.n))
		}

		if x.before != _magic || x.after != _magic {
			panic(fmt.Sprintf("wrong magic: %#x _ %#x != %#x _ %#x", x.before, x.after, _magic, _magic))
		}
	}
}

What did you expect to see?

(i.e. no output)
What did you see instead?
panic: should have swapped 0x80000000 0x80000001

goroutine 1 [running]:
main.main()
	/home/chenguoqi/atomic.go:31 +0x31c

@golang/loong64

I reproduced this issue on linux/riscv64.

Checked that it does NOT repro on ppc64le (seemed unlikely, however trust-but-verify).

@gopherbot please open backport issues.


Add newline to end of docs

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/Alluxio-Pull-Request-Builder/4291/
Test PASSed.

alluxio-bot, merge this please.

alluxio-bot, cherry-pick this to branch-2.0 please.


Compatibility with Token Action Hud Core 2.04

I expect you are already working on it but, the current SWADE version is not compatible with the latest (2.04) version of the Token Action Hud Core.

there will be a release by the end of the week (saturday).
There are no changes in the system HUD, only the enhanced v2 added by the core.

Compatibility to HUD core 2 has been released


Support for updating name and metadata of LocalParticipant

https://github.com/livekit/client-sdk-js/pull/599
Should we have a way to fail if CanUpdateOwnMetadata permission doesn't exist ?

I think that makes sense, @davidliu proposed to wait for a local participant metadata update event to come in before returning.
using the permissions locally has the disadvantage that the permissions might have just been updated and only the server is aware of them right now.
using the metadata update as an indicator that it resolved properly has the downside that this update might also be coming from a server sdk and might be unrelated (we could get around that by checking the contents of the update)

That sounds good 👍

@lukasIO Can I merge this ?

yeah, let's iterate on how to acknowledge it later on. As long as the function is async we can add additional logic for that later!


crypto.h: honor no-filenames config in missed cases

Fixes https://github.com/openssl/openssl/issues/704 (except for a single instance in 'engines' code)

+1 will merge once second-reviewed.

+1

pushed at eaa7e48  thanks!


Add a boolean setting to show the 'arrange' button in the UI.

I am trying to restore auto-arrange. Do you remember why it was removed?
https://github.com/Vahera/godot-orchestrator/assets/32321/68c54a55-b2e0-4042-9144-f0d063e86d99
I noticed that auto-arrange undo and redo is bugged - Will fill a bug report.
Also frames are not interacting.
I might be able to get these changed patched upstream.

I propose that this is kept in the settings menu off by default so I can debug it.

Hi @fire, thanks for the contribution.
If I recall, there was an issue with auto-arrange where it places the nodes in really obscure ways stacked in really tall vertical way rather than horizontal like you would normally expect, so we disabled it.  I certainly don't mind using a toggle for this.
As for the comment nodes, those aren't GraphFrame types.  The GraphFrame class wasn't added to Godot 4.2, so we had to roll our own implementation based on GraphNode.  There is a GH issue to port comments to GraphFrame for Orchestrator 2.1, but this won't be possible for 2.0 since that must remain based on Godot 4.2 -- unless upstream backports GraphFrame to 4.2 ofc.

Here's a great example, this is the event graph for camera.torch:

Now, when using the auto-arrange feature:

I haven't looked at the logic in GraphEdit in the upstream, but that really doesn't make it look any better imo.

Applied to main (=2.1) and branch 2.0.  Congrats on your first contribution to the repository @fire 🎉


Add missing frontmatter to some docs (release-v0.21.x patch)

:information_source: This PR targets a release branch, not main.
Changes
Patch a previous release branch with the changes introduced in #3958, i.e:
Some documentation files had no frontmatter block, making them invisible
in the sidebar navigation on the website (since they were missing the
linkTitle property).
Update weights to adjust the order of links:

Installation to the top
Deprecations at the bottom
Migration guides just before Deprecations

See tektoncd/website#271 for more background.
/kind documentation
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[:no_good_man:] Docs included if any changes are user facing
[:no_good_man:] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in or deleted (only if no user facing changes)

Release Notes
NONE

/ok-to-test

/test pull-tekton-pipeline-unit-tests
/test pull-tekton-pipeline-build-tests

/lgtm


Fix simple pre commit without dot

Fixes #15

Adds an ability to set either simple-pre-commit.json or .simple-pre-commit.json as configuration.

Self accept


[QPPA-1368] Ignore certain error types in New Relic

Currently, New Relic logs as errors failed validation issues (as 422s) and invalid request types (as 405s), on the QRDA3 converter. These aren't errors that should be in New Relic, or trigger Pagerduty alerts.
My editor automatically strips whitespace; to ignore whitespace changes, use this link: https://github.com/CMSgov/qpp-conversion-tool/compare/master...kalvin-error-types?w=1
Jira: https://jira.cms.gov/browse/QPPA-1368
Reviewer: TBD

@halprin added the additional status codes, do you want to give your approval on this PR? thanks!

@halprin @sfradkin thanks! do you know roughly when you'll be deploying to prod next? just got paged this weekend for this, so I have a vested interest :)

@kalvinwang: Probably not until the 19th or 20th with M26.


arm: systick: default CORTEX_M_SYSTICK based on dts

Use setting from devicetree to drive the default setting for
CORTEX_M_SYSTICK.  We update the dts files to default systick to be
enabled since the major of cortex-m platforms utilize it by default
(except on Nordic SoCs in which we default to disabled).
Fixes #25299
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

Thanks @galak .
If nrf5_common.dtsi sets the SysTick status to disabled, there is probably no reason that the same setting appears in nrf5340_cpuxxx.dtsi, is that correct?

nrf5340_cpuxxx doesn't include nrf5_common.dtsi from what I can tell.

There is also the SAM0_RTC_TIMER that, I understand, is preferable against the SysTick in SAM0 platforms, is that correct? Then you probably need to disable SysTick in these ones, as well.

The SAM0 family defaults to SYSTICK right now, so left the behavior the same.

let's have the "disabled" status in the nrf5340 specific dts headers.

Are you asking for a change here?

let's have the "disabled" status in the nrf5340 specific dts headers.

Are you asking for a change here?

Nope, I wanted to check why the common.dtsi is not included in nrf5340. Change is good.

nrf5340_cpuxxx doesn't include nrf5_common.dtsi from what I can tell.

This is probably a bug (@anangl to confirm). If not then sure, let's have the "disabled" status in the nrf5340 specific dts headers.

Yes, the name of this file is misleading (and this should be corrected in some way) as it can be included only for nRF51 and nRF52 Series SoCs (the driver for the "nordic,nrf-sw-pwm" compatible node requires PPI channels that are not present in nRF53).


DataPointsFilters, OutlierFilters, Transformations, and TransformationCheckers structs inherit from a class without virtual destructor

The structs mentioned in the description inherit directly from std::vector, an STL container. For example:
https://github.com/ethz-asl/libpointmatcher/blob/master/pointmatcher/PointMatcher.h#L422
The above can lead to undefined behavior, as the STL containers don't have virtual destructors. Therefore, any collection of polymorphic objects from those types could have issues getting destroyed.
References:

https://quuxplusone.github.io/blog/2018/12/11/dont-inherit-from-std-types/
https://stackoverflow.com/questions/6806173/subclass-inherit-standard-containers/7110262#7110262
https://www.researchgate.net/post/I-really-want-to-understand-if-there-is-any-reason-why-one-should-not-inherit-from-the-stl-vector-class-in-c/5304df29d3df3edb158b45f3/citation/download

An example of how this could impact usage of the library is when people inherit from one of the containers cited above, as DataPointsFilters/Gestalt. This filter is stateless (saves no state, only parameters):
https://github.com/ethz-asl/libpointmatcher/blob/master/pointmatcher/DataPointsFilters/Gestalt.h#L94
When the filter is created, used and tore down, it is undefined how the parameters are destructed. If we were to implement a filter that uses history, this could result in a memory leak in the worst case.

Interesting reading this morning. What would be your solution here?


Provided support for userpool backup in csv format

[x] Add take file format as parameter for back function and use corresponding writer to write backup file.
[x] Add output file option in cli.
[x] Add cli prompt for output file format with supported file formats.
[ ] Document supported file format in csv as separate PR.
[x] Provide support for backfrom csv file.

@rahulpsd18 Awaiting feedback before moving further.

Also, could you please try running the new code on a demo cognito pool and report if backup/restore for json is working perfectly (just to make sure). You probably have a test pool configured and I can't get myself to create a test pool right now 😅.

@rahulpsd18 is it good to merge?


Can't turn off hover

Expected Behavior
Using options for tooltips and hover to turn off tooltips and hover should work
Current Behavior
I am unable to turn off tooltips and hover
Steps to Reproduce (for bugs)
This is what I have.
`this.renderChart({
datasets: [
            {
                backgroundColor: [this.gradient, '#ece8e5'],
                data: [200, 160],
                borderWidth: 0,
                weight: 10
            },
            {
                backgroundColor: ['transparent', 'transparent'],
                data: [360],
                borderWidth: 0,
                weight: 1
            },
            {
                backgroundColor: ['#0b3f54', 'transparent'],
                data: [230, 130],
                weight: 2
            }
        ],
        options: {
            responsive: true,
            events: [],
            cutoutPercentage: 60,
            tooltips: {
              enabled: false
            },
            hover: {
              mode: null
            }
        }
})`

This does not work. If I add Chart.defaults.global.tooltips.enabled = false the tooltips are removed, but Chart.defaults.global.hover.mode = null does not stop hover.
Context
I will be using this in a widget, so don't want hover effects.
Environment

Chart.js version: 2.8.0
Browser name and version: Chrome

events: [] efficiently disables all events.
The fact that tooltips are disabled when you define that setting globally, hints me that something is wrong elsewhere. What happens in you renderChart?

The graph displays as it is supposed to, but the tooltips still show. I just put a div over the graph so the hover isn't detected on the graph. The code I have above is pretty much all there is. The only other thing I can think of is that it is brought in as a component.

Your options are probably at wrong level of config (under data, should be same level with data)
Pen with no hover


Support to not use the default button

How can we use just the modal and not using any children?
I just wanna the modal and set the visibility passing a state through the "visible" prop.
EDIT: And also, needs a "onCancel" prop...

You probably don't want to use this component for that.  You can just build your own using React Bootstrap.
Here's their modal component that you may find useful. https://react-bootstrap.github.io/components.html#modals-live


ENH: MRIQC for DWI

CC/ @mckenziephagen @arokem @teresamg

Interesting addition of an estimator of the number of shells (nipreps/dmriprep#64):
https://github.com/nipreps/mriqc/pull/1113/commits/95497ca40054c02854e349550e3dc0bb54d9b0b6#diff-0afea89f34b27abdc8fb70e57182420cdfb40a7779a3ca808feebc2978e12e56R145-R158
CC/ @edickie @arokem @mckenziephagen
I have tested with my data and it seems to work pretty well. The idea is that the estimated number of shells is very very high (e.g., 8 or 9), this is probably a DSI dataset.
To test it:
import numpy as np
from mriqc.interfaces.diffusion import NumberOfShells

bvals =  np.loadtxt("path/to/dwi.bval")

print(NumberOfShells(in_data=bvals.tolist()).run().outputs.n_shells)

let me know what you think


Ensures that proper sort directions are extracted on explicit sort mode

Issue is https://github.com/Sology/smart_listing/issues/109

Can this be merged in please?

What if someone uses some complex joined columns sort attributes? Like i.e. user's association to profile and sorting by profile name: profiles.name. I don't think in this case making it all symbols is good.
Instead, you should be consistent and just use either symbols or strings.


Test todo-list features with Jest

Test todo-list features with Jest
In this branch we:

Set up Jest for unit testing.
Test add_task function and delete_task function.
Test if they save to localStorage and delete from it too.
Ignore unnecessary files in .gitignore file.🙃
Demo:

Hi @brytebee ,
Thanks for the feedback. But if I'm not mistaken the required changes are not a core project requirement for this block thus no need to block. The test is passing as required for this milestone in the Project requirements.🤔


Using nbsphinx with autosummary

Hi,
I'm trying to run nbsphinx as per the docs. I am also using the autosummary extension. I get the following error:
[autosummary] generating autosummary for: api.rst, changelog.rst, examples.rst, examples/README.rst, examples/SMHM.ipynb.rst ...
...
IOError: [Errno 2] No such file or directory: u'/home/steven/Documents/Projects/MRP/mrpy/docs/examples/SMHM.ipynb.rst'

As you can see, it's trying to add the rst to the end of SMHM.ipynb and not finding it. Any idea what's going wrong?

I don't know how to reproduce this error.
Can you please give a minimal example and the exact commands that you were calling?

Same issue here.
I do not have a simple reproducible example (doc that includes nb and API docs are never simple!) but I think it is related to the function  _add_notebook_parser(app) as it presents ipynb files like an rst file to sphinx.
Then, autosummary is looking for the any_notebook.ipynb.rst file in the documentation, and does not find it !
I have temporarily modified autosummary extension to be able to create the doc. I do not have (yet) a solution with a simple patch on nbsphinx...

I am trying to produce a minimal example with no success (as in my minimal example works...). I don't have much time at the moment, and I have made a workaround by creating my own extension in which my ipynb files are hidden by putting them under the "exclude_patterns" config list, and I automatically cover them to rst and place them in a different automatically created folder upon the builder-inited signal. Not a very flexible solution but works in my case. I'd rather use nbsphinx if possible. Sorry I can't help more at this time.

I still can't reproduce the error, but perhaps I don't have to ...
I had a look at the autosummary source code and I think I found the culprit:
https://github.com/sphinx-doc/sphinx/blob/04a8c26eabba3e728a5b74afd15f328f9235b2d0/sphinx/ext/autosummary/__init__.py#L559
This line doesn't work if there are any source suffixes other than the first one in source_suffix (typically '.rst'). It should be changed to check for all values of source_suffix, not only the first one.
Or, alternatively, it should filter out the files which have a suffix other than the first one.
AFAICT, there is nothing I can do in nbsphinx to solve this. You should create an issue for that at https://github.com/sphinx-doc/sphinx/issues.
There might be a work-around, though:
I assume you are not using autosummary_generate in your conf.py, right?
http://www.sphinx-doc.org/en/stable/ext/autosummary.html#confval-autosummary_generate
If you assign a list of files which should be searched for autosummary stuff to this variable, the problem should (hopefully!) go away:
autosummary_generate = 'myfile1.rst', 'myfile2.rst', 'and_another_file.rst'

You can drop the file extensions if you like, but all files in this list must have the same extension which has to be the first in source_suffix (which by default is '.rst').
Alternatively, it should (hopefully!) also work if you disable auto-generation of "stub" files altogether:
autosummary_generate = False

Does this work?

Setting autosummary_generate to be a list of files where I specify my "autosummary" commands does work (and iteratively as well it would seem). Thank you!
Although, I've realised (having done this the more manual way I mentioned in a previous comment) that with nbsphinx, there is no syntax highlighting? Also, is there a way to turn off the cell numbering? That doesn't appear when I do it manually.

I'm glad that the work-around works, the real solution has to be done in the autosummary extension.

... with nbsphinx, there is no syntax highlighting?

There should definitely be syntax highlighting!
If not, that's a bug and you should create a new issue (probably attaching an example notebook).
As you can see in the examples, there is syntax highlighting: http://nbsphinx.readthedocs.org/en/0.2.2/example.html#Code-Cells

... is there a way to turn off the cell numbering?

I went a long way to add the prompts, why would you want to remove them?
Or do you want to have the prompts and only remove the numbers themselves (like In [ ]:)?
If you want nbsphinx without the prompts, you can use the very first version: http://nbsphinx.readthedocs.org/en/0.1.0/example.html#code-cells
... or you can try one of the alternatives mentioned there: http://nbsphinx.readthedocs.org/en/latest/links.html

i just had ran into this problem. This is a great work around, perhaps you should mention this in your docs until sphinx fixes their autosummary

@arsenovic Can you please make a PR with the suggested changes to the docs?
I don't know if the Sphinx people are aware of this bug. If not, someone should create an issue there!

added update to the docs here, https://github.com/spatialaudio/nbsphinx/pull/63
i think maybe its best if you add the bug report to sphinx,  because if they require more info or collaboration i will be of no use.

Turns out that this is a known bug: https://github.com/sphinx-doc/sphinx/issues/2485.
It will be fixed in Sphinx 1.5.
Thanks for #63, I've merged it and added a link to the bug report.

Sphinx 1.5 was just released, is the problem fixed now?

Sorry it may be a while before I can check this properly.


Changing method names to match FTP commands

For instance: make_dir => mkdir, current_dir => pwd, change_dir => cwd (or cd?), change_dir_to_parent => cdup, retr => get, stor => put, etc.
For listing the contents of a folder (pull request #20 ), use list or ls?

I was thinking of doing a pull request for this, after which I think we could make a new release of this crate so that users can benefit from the improvements.


False positive warning for Android plugins where version is not needed

Current Behavior
Some plugins specified in version catalog seems no longer detected as being used but removing it will also cause build/syncing error in the project.

Expected Behavior
Not sure when this happens, it seems after updating to Gradle 8.2 cause it.
Context (optional)
False positive
Steps to Reproduce
Using plugins for Android project that is using classpath or do not need version when specifying un plugin block, thus using id() with get().pluginId instead of alias() for Kotlin DSL.
plugins {
    alias(libs.plugins.android.application)
    alias(libs.plugins.android.kotlin) // This should be put before any Kotlin dependent plugin
    alias(libs.plugins.ksp)
    id(libs.plugins.parcelize.get().pluginId)
}

[versions]
kotlin = "1.9.0"

[plugins]

parcelize = { id = "org.jetbrains.kotlin.plugin.parcelize", version.ref = "kotlin" }  // Mark as unused despite being used and removing this will cause build error

Gradle version
8.2
Build scan URL (optional)
No response
Your Environment (optional)
Android

Sorry that you're having trouble with Gradle!
This is an issue with IntelliJ IDEA’s Gradle integration, which is developed by JetBrains and cannot be addressed in the Gradle Build Tool project.
Please open such IntelliJ-related issues on the JetBrains Issue Tracker at https://youtrack.jetbrains.com/newIssue?project=IDEA.


setup.py install puts binary module to arch-independent directory

setup.py install installs to sitelib directtories, not sitearch.
Specifically python* setup.py install installs on Linux everything to /usr/lib not to /usr/lib64. Specifically, _nvx_utf8validator.so should be in arch-dependent directory.
Also, see https://src.fedoraproject.org/rpms/python-autobahn/blob/master/f/python-autobahn.spec where files also end in %{python3_sitelib}, so it is not just openSUSE problem, but apparently my colleagues at RedHat missed that. @hroncok, what do you think?
Ah, correction, our colleagues at RedHat don't see it, because they don't set AUTOBAHN_USE_NVX. How much does it speed things up?

@mcapl I'm not responsible to all Python packages in Fedora and I don't maintain autobahn nor i know anything about it.

NVX is "work in progress" - don't set AUTOBAHN_USE_NVX is the best for now. we introduced this env var now, because we recently touched the serializer code parts
the speed ups can be huge, but it is not yet production ready, and currently amd64 only (SSE2 and SSE4.1). we will finish and ship it though at one point, since XOR masking and UTF8 validation is the main (measured) CPU bottleneck of Autobahn (at the WebSocket level)
sorry if that sounds harsh, but: I don't care about SuSE or RedHat packaging policies - distro packaging of python packages is "wrong" IMO
If you claim that we incorrectly package Autobahn from a Python perspective (might be the case indeed), then we will of course fix that. we do want to be good citizens in the Python universe
fwiw, we have started to package our stuff as snaps, because that follows a vendor-packaging approach which cuts out the distro between us and our users - snaps are awesome

@mcapl I'm not responsible for all Python packages in Fedora and I don't maintain autobahn nor I know anything about it.

You are just the last one who committed to its git? Anyway, sorry for bothering then.

sorry if that sounds harsh, but: I don't care about SuSE or RedHat packaging policies - distro packaging of python packages is "wrong" IMO

OK, I hear you. I think you are completely wrong, but I understand we are on our own here. Duly noted.

If you claim that we incorrectly package Autobahn from a Python perspective (might be the case indeed), then we will of course fix that. we do want to be good citizens in the Python universe.

Yes, that's what I claim, but perhaps if NYX (and it is the only binary module) is still WIP that it is understandable. I will get rid of that variable and just let it be.


question about what guarnatees the correctness of the c11_atomics atomic_flag test

This is the kernel being executed in the first atomic_flag test, but I'm actually wondering what in the OpenCL spec guarnatees that this executes as expected by the CTS.
More specifically, what prevents a thread to race with a thread from a different subgroup on destMemory[cnt]. One thread could enter the iteration for e.g. cnt == 5, while another thread just reached atomic_flag_clear_explicit(&destMemory[cnt], for the same cnt, which means that two threads will enter the critical section, just the second one won't do anything.
I'm seeing this behavior with rusticl on zink on radv and at the moment it's not clear to me if that's my or the CTS' bug. Removing the atomic_flag_clear_explicit(&destMemory[cnt],  makes only one thread execute the "criticial section" for each value of cnt, but that fails the test later.
So what's actually guaranteeing the correctness of this test here? Or would this test need to be rewritten? I'm also mildly wondering what clvk did in order to pass this test or if that was never a problem in the first place. I can probably come up with a fix to guarantee that behavior, I'm just wondering if there is some undefined behavior at play here.
Program source:
__kernel void test_atomic_kernel(uint threadCount, uint numDestItems, __global int *finalDest, __global int *oldValues, volatile __local atomic_flag *destMemory)
{
  uint  tid = get_global_id(0);

              // initialize atomics not reachable from host (first thread
              // is doing this, other threads are waiting on barrier)
              if(get_local_id(0) == 0)
                for(uint dstItemIdx = 0; dstItemIdx < numDestItems; dstItemIdx++)
                {
                  if(finalDest[dstItemIdx])
                    atomic_flag_test_and_set_explicit(destMemory+dstItemIdx,
                                                      memory_order_relaxed,
                                                      memory_scope_work_group);
                  else
                    atomic_flag_clear_explicit(destMemory+dstItemIdx,
                                               memory_order_relaxed,
                                               memory_scope_work_group);    }
  barrier(CLK_LOCAL_MEM_FENCE);

  uint cnt, stop = 0;
  for(cnt = 0; !stop && cnt < threadCount; cnt++) // each thread must find critical section where it is the first visitor
  {
    bool set = atomic_flag_test_and_set_explicit(&destMemory[cnt], memory_order_relaxed, memory_scope_work_group);
    atomic_work_item_fence(CLK_LOCAL_MEM_FENCE | CLK_GLOBAL_MEM_FENCE, memory_order_acquire,memory_scope_work_group);
    if (!set)
    {
      uint csIndex = get_enqueued_local_size(0)*get_group_id(0)+cnt;
      // verify that thread is the first visitor
      if(oldValues[csIndex] ==<PHONE_NUMBER>)
      {
        oldValues[csIndex] = tid; // set the winner id for this critical section
        stop = 1;
      }
      atomic_work_item_fence(CLK_LOCAL_MEM_FENCE | CLK_GLOBAL_MEM_FENCE, memory_order_release,memory_scope_work_group);
      atomic_flag_clear_explicit(&destMemory[cnt], memory_order_relaxed, memory_scope_work_group);
    }
  }

  // Copy final values to host reachable buffer
  barrier(CLK_LOCAL_MEM_FENCE);
  if(get_local_id(0) == 0) // first thread in workgroup
    for(uint dstItemIdx = 0; dstItemIdx < numDestItems; dstItemIdx++)

                finalDest[dstItemIdx] =
                    atomic_flag_test_and_set_explicit(destMemory+dstItemIdx,
                                                      memory_order_relaxed,
                                                      memory_scope_work_group);}

mhh, no, that would mean that something with atomic_flag_test_and_set_explicit is going wrong, but that's kinda of weird.. maybe there is a coherency issue somewhere, anyway, it's probably not a CTS bug then and I initially thought something else is going on.


Spread array of values generated by an expression to form a record

When we use an expression which returns an array in the Transform or Add column based on this column dialogs, the resulting cells are currently left blank because we cannot store an array in a single cell.
This is of course not ideal and quite confusing because there is no explanation or warning about this (#1475, #1088).
Proposed solution
This should behave like the Add column from reconciled values operation: spread multiple values in consecutive cells to form a record.
For instance, if I start with a table such as:

ID
JSON

1
{"letters":["a","b"]}

2
{"letters":["c","d","e"]}

If I add a column letters generated by the expression value.parseJson().letters, I want to get the following result:

ID
JSON
letters

1
{"letters":["a","b"]}
a

b

2
{"letters":["c","d","e"]}
c

d

e

Alternatives considered
Just keep the logic as it is but improve error reporting
We could simply make it clearer at the preview stage that those values will not be stored. @thadguidry proposed the following UI:

@ostephens has noted that it would be better not to render those as errors (because the expression does not return an error itself) but rather to signal this in a different way (for instance by improving the way we distinguish between data types of returned values in this preview area).
Spread the values as columns
In issue #36, it is proposed that values are spread on columns instead. With the example above, you would get:

ID
JSON
letters 1
letters 2
letters 3

1
{"values":["a","b"]}
a
b

2
{"values":["c","d","e"]}
c
d
e

I find that less helpful because storing an array like this makes it hard to reuse the results afterwards.
The lists can have variable lengths, so the columns created by the operation depend on the data.
And this behaviour can already be obtained by successively adding columns using the expression value.parseJson().letters[0], value.parseJson().letters[1] and so on (or rather in the reverse order to get the exact result above).
Store arrays as strings in cells
@thadguidry also proposed to convert these values to strings before storing them, for instance by serializing them to JSON with jsonize().
That is indeed a sensible workaround that users can resort to at the moment, but it does not feel so convenient to reuse afterwards (although it is not as bad as when values are spread as columns in my opinion).
Make it possible for cells to store arrays natively
In PR #1586 @ostephens proposed to make it possible to store arrays in cells natively.
That is definitely an interesting approach, which could potentially even remove the need for the records mode entirely: just store lists of values in the cells instead of spreading them on consecutive rows. Hence it seems to be a much bigger project to me. What I find daunting about it is that:

if we start allowing arrays in cells, we basically need to rethink most operations and decide how they should handle arrays. What happens when you reconcile a cell which contains an array? What happens when one of the transpose operations encounters that? And so on.
unfortunately, as explained in a comment on #1586, we currently allow an Java Object to be stored as cell value, which means that we cannot rely on the compiler to make sure we are handling this new possibility in all places of the code where we should add this handling. This is also a problem for extension developers, who will not become aware of this unless they read our release notes.

That being said I would be happy to consider this change in 4.0, where we already change to a very different data model - such a change could be introduced there. That does not make the design effort less daunting of course (it is still basically the same problem as redesigning the records mode).
Give the user a choice between various of those options
The dialogs where those expressions are previewed could be adapted to give the user a choice in how they want their arrays to be stored. Those expression preview dialogs are already quite full, but if we think there can be multiple sensible things to do there, it is natural to give the user a choice. But it is not really clear to me that it is worth it, because:

spreading the values in columns could be done using the Split columns operation instead (by adapting it to accept an expression returning an array), as suggested by @dfhuynh and others
storing the arrays as strings can already be done by adding .jsonize() at the end of the expression. Arguably this is not as discoverable as an explicit option in the UI to do it for you, but I would expect people who want this to be aware of parseJson() at least (otherwise how do they intend to reuse the results?) so it is also quite likely that they know of jsonize()
storing arrays in cells is not something we currently support, so we cannot give the option to do that yet

Additional context
Related issues I am aware of: #36, #1475, #1088.
This is important in the context of the Structured Data on Commons integration project because the CommonsExtension offers GREL functions which return arrays.

@wetneb I am in agreement with the proposal.  Could you update the beginning of the proposal (or milestone) to signify that this would be a pre-4.0 feature enhancement?  I understand from your notes it will not be targeting 4.0, where in 4.0 we might do further data model changes and affected code to support storing arrays directly in cells.

I can see all three options being useful here:

Spread results across rows
Spread results across columns
jsonize array to a string

I wonder if a control like we have for errors would be useful here - in the transformation dialogue the user is offered options as to what to do with errors (keep original, store error, store blank) - could we offer a set of options for arrays in a similar way so that the user chooses how to deal with them?
My biggest frustration with arrays (and the reason I originally looked at storing them in cells) is that currently it is too easy (IMO) for the user to overlook the need to convert the array to a string (or other storable representation) and end up storing nothing in the cell instead - any solution that avoids this has my support!
One argument I can see in favour of preferring the jsonize approach is that an array can contain arrays (which can contain arrays, etc.) which means that splitting across rows or columns would have to be recursive and could result in a complex set of rows/columns being added.

Indeed, I did not think about the recursive case, that is going to be pretty fiddly to implement.
Adding an option to choose between various behaviours seems doable to me too.

For me getting an option for jsonize option into the process would be a good and relatively straightforward start. I agree with your comment above @wetneb


ticdc: fix unstable test TestRemoveExpiredFilesWithoutPartition

What problem does this PR solve?

Issue Number: close #10815
What is changed and how it works?
Currently, the date part in storage file path is formatted using local timezone.
And the date part generated in TestRemoveExpiredFilesWithoutPartition when removing expired files is different in different timezone which make the test unstable.
This pr add another parameter loc to RemoveExpiredFiles to specify the timezone info.
Note the current code still have some problem if the TZ is different in cdc servers. We may need a better solution to fix it.
Check List
Tests

Unit test

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

/test cdc-integration-kafka-test

/test dm-integration-test

/test dm-integration-test

/test dm-integration-test

/test verify

/test dm-integration-test

/test cdc-integration-kafka-test


Removed Big Switch Fabric

The links for this entry are broken.

removing this content (cloud provider specific) seems fine to me.
/lgtm

Confirmed links for BigSwitch no longer works, BigSwitch was acquired by Arista networks
/lgtm

/approve


feat!: initial description-list support; render_header cleanup

This PR implements description lists, based on the way py-shiny and plotnine extended the renderer in their docs.
While in the Renderer, I'll work on...

migrating to use the quartodoc.pandoc submodule.
consolidate more header generation into the .render_header() method.
planning out nice HTML classes, so it's easier to style everything

Backwards incompatible changes:

render(self, dc.Object | dc.Alias) now calls render(self, dc.Docstring), rather than iterating over the docstring itself.

This makes the printout of dc.Docstring renders in snapshot tests look a bit nicer.

HTML overview
Here's a rough sketch of the HTML structure rendered by quarto, using PUG:
// Doc example for quartodoc.MdRenderer
//
// Sections are automatically added around headers by quarto
// and we can add ids or classes to them
section.level1.quartodoc-MdRenderer.quartodoc(id='quartodoc.MdRenderer')

  // object title ----
  h1.quartodoc-MdRenderer.quartodoc MdRenderer

  // object signature ----
  p
    code
      | MdRenderer(self, header_level=1, show_signature=True ...)
  // doc section: short description ---
  p Render docstrings to markdown.

  // doc section: parameters ---
  section#parameters.level2.parameters
    h2.parameters.anchored(data-anchor-id='parameters')
      | Parameters
      a.anchorjs-link(...)

    // parameters definition list ---
    dl
      dt
        span.parameter-name
          code header_level
        span.parameter-annotation-sep :
        span.parameter-annotation
          a(href='https://docs.python.org/3/library/functions.html#int') int
        span.parameter-default-sep  =
        span.span-parameter-default
          code 1
      dd
        p The level of the header (e.g. 1 is the biggest).

  // doc section: example ---
  section#examples.level2.examples
    h2.examples.anchored(data-anchor-id='examples')
      | Examples
      a.anchorjs-link(...)
    #cb1.sourceCode
      pre.sourceCode.python.code-with-copy.

TODO:

Adding CSS styles to quartodoc sites (with description lists, you will need to add styles for them to look reasonable).
Diagram

rebased against main

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 92.85714% with 5 lines in your changes missing coverage. Please review.

Project coverage is 88.46%. Comparing base (81cccdf) to head (5e36bcc).
Report is 2 commits behind head on main.

Files
Patch %
Lines

quartodoc/renderers/md_renderer.py
91.07%
5 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #325      +/-   ##
==========================================
- Coverage   88.49%   88.46%   -0.04%
==========================================
  Files          37       37
  Lines        2843     2895      +52
==========================================
+ Hits         2516     2561      +45
- Misses        327      334       +7

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

This is mostly complete, but we need to figure out how users will include the necessary css (e.g. could educate or orchestrate).

It seems like for css, we could provide an extension that includes the necessary css via quarto's lua add_html_dependencies().
If we do this, we should make obvious people how to escape hatch by copying the css file into their own project.


Out of memory error on 3.2 with RocksDB

my environment running ArangoDB

[x] 3.2

Mode:

[x] Single-Server

Storage-Engine:

[x] rocksdb

On this operating system:

[x] Linux

[x] Ubuntu .deb

I have a high number of out of memory errors in my Foxx scripts. It seems that all of them concern the same operation (a replace within a transaction). I should clarify that there is plenty of available memory on the server. Full stack trace:
at callback (/usr/share/arangodb3/js/server/modules/@arangodb/foxx/service.js:347:35)
at Tree.dispatch (/usr/share/arangodb3/js/server/modules/@arangodb/foxx/router/tree.js:135:7)
at dispatch (/usr/share/arangodb3/js/server/modules/@arangodb/foxx/router/tree.js:426:3)
at next (/usr/share/arangodb3/js/server/modules/@arangodb/foxx/router/tree.js:410:7)
at next (/usr/share/arangodb3/js/server/modules/@arangodb/foxx/router/tree.js:410:7)
at next (/usr/share/arangodb3/js/server/modules/@arangodb/foxx/router/tree.js:412:15)
at Route._handler (/var/lib/arangodb3-apps/_db/keakr/commands/APP/routes/common2.js:23:8)
at ArangoDatabase._executeTransaction (/usr/share/arangodb3/js/server/modules/@arangodb/arango-database.js:130:10)
at action (/var/lib/arangodb3-apps/_db/keakr/commands/APP/routes/common2.js:49:49)
via ArangoError: out of memory
Service "/commands" encountered error 500 while handling POST http://xxx.westeurope.cloudapp.azure.com:8529/_db/keakr/commands/v1/common/createLike?userEdge=isAuthorOf
encountered out-of-memory error

For reference:
common2.js:23:8 is the start of the transaction
common2.js:49:49 is the replace operation
I couldn't find any reference to that particular error so I'm wondering what could cause it?
Thanks in advance

Potentially has the same root cause as
https://github.com/arangodb/arangodb/issues/2810
Is there a fulltext index present for the collection in question?

Indeed, there is a full-text index on that collection!

Then this bug is hopefully fixed in 3.2.1. The out-of-memory error is wrong here. This error occurs when a document is inserted into the fulltext index that does not have the indexed attribute or for which no words will be inserted into the index.
The error is just wrong. Instead the operation should just continue, even without inserting anything into the index.

Great news, thanks Jan. You probably guess my next question: any ETA for the 3.2.1?

3.2.1 is already built, and is currently being tested internally. When that testing is done, it will be released.
I expect this to happen this week.

3.2.1 has been released and is available for download.


v4.0 DRAFT says affects is mandatory at the root-level, but often is not

I noticed that in the v4.0 DRAFT affects is Mandatory in the root-level. However, many CVE files in cvelist omit "affects".

It is only mandatory for PUBLIC CVE records.
I can only find one such CVE (likely an artifact of some syncing problem)
https://github.com/CVEProject/cvelist/search?q=NOT+affects+NOT+reserved+NOT+REJECT
https://github.com/CVEProject/cvelist/blob/master/2019/6xxx/CVE-2019-6149.json


Adapting the IconColor and IconBackgroundColor to ThemeResources Color Based on Different Types of Controls

Adapting the IconColor and IconBackgroundColor to ThemeResources Color Based on Different Types of Controls to Make the Icon Consistent with the ThemeResources Instead of Setting Manually

Fixed with #62


Setup Openshift 4.x cluster

Required openshift cluster to test openebs installation through director.

The openshift cluster is now up and running.


chordx:0.1.0

I am submitting

[x] a new package
[ ] an update for a package

I have read and followed the submission guidelines and, in particular, I

[x] selected a name in conformance with the guidelines
[x] added a typst.toml file with all required keys
[x] added a README.md with documentation for my package
[x] have chosen a license and added a LICENSE file or linked one in my README.md
[x] ensured that my submission does not infringe upon the rights of a third party
[x] tested my package locally on my system and it worked
[x] named this PR as name:version of the submitted package
[x] agree that my package will not be removed without good reason

Description: A library to write song lyrics with chord diagrams (for example for guitar or ukulele instruments). Also, you can add chord names over a specific word or character to create song lyrics.

Thanks, this is really cool!


_includes/extentions-v1beta1-definitions.html has a weird char

http://kubernetes.io/v1.1/docs/devel/api-conventions.html#types-kinds"
HTML is getting corrupted due to a strange character in the heading ID that is not being formatted well in the code that generates HREFs.

As it's not actually corrupting the html, just creating spew that gets ignore, I'm closing as harmless


renew Kerberos tickets

Can anyone explain how spring cloud stream Kafka binder renew Kerberos tickets?

@raiusa Kafka binder in spring cloud stream does not do anything for renewing Kerberos tickets. I don't think its a binder concern at all. If the broker is secured, then the binder provides mechanisms to communicate with that broker, but how the broker is secured is outside of the binder's responsibilities.

This is outside of the scope of the kafka binder. Closing it as invalid.


update model resolution to use node16

Description
Add your description here!
Checklist
To ensure a quick review and merge, please ensure:

[ ] The PR has a understandable title and description explaining the why and what.
[ ] The PR is opened in draft if not ready for review yet.

If opened in draft, please allocate sufficient time (24 hours) after moving out of draft for review

[ ] The branch is recent enough to not have merge conflicts upon creation.

Ready to Land?

[ ] Build is completely green

Submissions with test failures require tracking issue and approval of a CODEOWNER

[ ] At least one +1 review by a CODEOWNER
[ ] All -1 reviews are confirmed resolved by the reviewer

Override/Marking reviews stale must be discussed with CODEOWNERS first

merged in https://github.com/Azure/autorest.csharp/pull/3809


Transfer Runtime tests to GitHub actions

Description
The current CI setup runs Runtime and Privileged Tests on Jenkins. However, we can achieve the same functionality using GitHub Actions, providing faster feedback and a better user experience. Jenkins, which can sometimes take an hour to allocate nodes just to run the tests, can be replaced with GitHub Actions to obtain quicker results.
Testing
The jobs are taking less than 15 minutes to complete. The GitHub Actions also offers operations such as re-run a specific focus group which is helpful in case of flakes since it's not necessary to re-run the entire test suite.

GitHub Action run: https://github.com/cilium/cilium/actions/runs/5003625978

Notes

@nbusseneau can we remove jenkinsfiles/ginkgo-runtime-kernel.Jenkinsfile or we need to leave it here because of stable branches? If we leave it can we stop triggering it once this PR is merged?

In the  GitHub Action example run you will see a summary for each Job (for example here). This summary is not part of the changes on this PR but they will be added in a follow up PR.

There are some optimizations made specifically for the LVH as part of the GH worfklow. Those optimizations will be removed from this workflow in a follow up PR once they are added as part of the LVH action itself.

@joestringer you brought up a very good point in terms of the github-sec. On the safe side I ended up switching to run the workflow on pull request events instead of issue_comment. If we see that it's being unmanageable then we can switch it back to issue_comment.

/test

/test-1.26-net-next

/test-runtime

/test-runtime

/test-runtime

/test


Access to wifi state?

Could it be possible to have access to wifi state of the car and to have the option to put wifi on/off?

Is this possible via the app? We're basically mimicking that.

Sorry, can't really do anything. You can try looking at the API or enabling your teslajsonpy debug logs to see what you can access.


Swiping multiple Dismissables leads to inconsistent behavior

In a ListView with Dismissable widgets, swiping 2-3 tiles at the same time leads to inconsistent behavior. Sometimes all items are dismissed, sometimes a single item is dismissed and others pop back into the list.
This is observable within the Gallery app.

This seems fixed on the latest master 1.22.0-10.0.pre.356

flutter doctor -v
[√] Flutter (Channel master, 1.22.0-10.0.pre.356, on Microsoft Windows [Version 10.0.19041.508], locale en-US)
    • Flutter version 1.22.0-10.0.pre.356 at C:\Development\flutter_master
    • Framework revision c412fd9967 (2 hours ago), 2020-09-24 22:47:03 -0700
    • Engine revision 435db234e4
    • Dart version 2.11.0 (build 2.11.0-161.0.dev)

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at C:\Users\marku\AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.1
    • Java binary at: C:\Users\marku\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\193.6626763\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.6.5)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.6.30320.27
    • Windows 10 SDK version 10.0.18362.0

[√] Android Studio (version 4.0)
    • Android Studio at C:\Users\marku\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\193.6626763
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] Connected device (6 available)
    • SM G950F (mobile)           • ce12171c51cc001c03 • android-arm64  • Android 9 (API 28)
    • sdk gphone x86 arm (mobile) • emulator-5554      • android-x86    • Android 11 (API 30) (emulator)
    • Windows (desktop)           • windows            • windows-x64    • Microsoft Windows [Version 10.0.19041.508]
    • Web Server (web)            • web-server         • web-javascript • Flutter Tools
    • Chrome (web)                • chrome             • web-javascript • Google Chrome 85.0.4183.121
    • Edge (web)                  • edge               • web-javascript • Microsoft Edge 85.0.564.51

• No issues found!

Minimal code sample
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Sample',
      home: Scaffold(
          appBar: AppBar(
            title: Text('Sample'),
          ),
          body: ListView.builder(
            itemBuilder: (BuildContext context, int index) {
              return Dismissible(
                key: Key('dismissible-$index'),
                child: Card(
                  child: Container(
                    child: Text('Dismissible $index'),
                    height: 50,
                  ),
                ),
              );
            },
            itemCount: 15,
          )),
    );
  }
}

Should we close this @mehmetf?

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


Fix doxygen page for Hello World example.

Make sure any reference to functions or classes are actual links to the
documentation. Also cross-reference other functions or classes that the
user might be interested in, for example, if we show one type of
mutation, link the other types too.
The output can be reviewed at:
https://coryan.github.io/google-cloud-cpp/0.2637.0/bigtable/bigtable-hello-world.html

This change is

/cc: @billyjacobson in case you are interested.


[Hexagon] Add HexagonThreadManager

This PR was made in collaboration with @JosephTheOctonaut.
Add HexagonThreadManager class and unit testing.  HexagonThreadManager can be used to spawn a given number of QURT threads on which the user can Dispatch work including basic Signal and Wait functionality to create synchronization points between threads.  This PR adds the low-level HexagonThreadManager class only.  Related Hexagon Device API support and codegen to come in future PRs.
This PR is dependent on #11635 to pass CI.
cc @mehrdadh

On another note---all the thread manager files spell "threadmanager" without the underscore, while "buffer manager" has one.  Could you add the underscore for consistency?

Could you move the buffer manager changes to a separate PR? They seem independent from the thread manager.

I am basically moving this functionality from within the Hexagon Device API where it lived previously to a separate header file.  Both the Hexagon Device API and the Hexagon Thread Manager need a Hexagon Buffer Manager.  The alternative would be two duplicate code in two places.

Thanks Adam!

The (TVMStreamHandle)i is still there. It's not that much of a big deal, but it would be nice to replace it with static_cast eventually.

Missed this.  Will fix.

Just realized that I named my commit "static cast for TVMStreamHandle` before I realized that reinterpret_cast was required.  I can rename that commit or ...
Just thinking out loud here ... the other thing we can do RE TVMStreamHandle is just remove it from this PR.  It's there because we are looking ahead to working with actual stream handles at the Device API level.  But, what we really need is just an unsigned thread ID.  What's there is a bit of a misuse of a pointer type to stash an integer and probably over-engineered at least for the time being.

I wouldn't worry about renaming a commit in the PR.  The code looks fine.  For some reason I thought that the stream handle was actually an integer...
When the CI passes, I can change the entry in the commit message to read "reinterpret" instead of "static" right before committing.

Just a reminder that CI is failing because it requires this PR https://github.com/apache/tvm/pull/11635


INSTALL_FAILED_CONFLICTING_PROVIDER

After adding the scanlibrary in my personal project it gave me no error until
implementation project(':scanlibrary')
but when I hit the run it gave me this error and I am unable to resolve it.
Installation did not succeed.
The application could not be installed: INSTALL_FAILED_CONFLICTING_PROVIDER

List of apks:
[0] 'D:\Projects\Android\DocumentScannerandOCR\app\build\outputs\apk\debug\app-debug.apk'
Installation failed due to: 'null'

Looking forward for help.
Thanks

Change file authorities in scanlibrary manifest file with your application id.


Moving towards yeoman core features

A couple days ago we started talking about consolidating the yeoman core build system in a reusable module https://github.com/yeoman/yeoman/issues/1440 - This module obviously is generator-node.
All the features contained in https://github.com/yeoman/generator will be added or updated to match our internal style.
To make a short list:

Code coverage
Coveralls integration
Unit test (mocha)
Travis.CI integrtion
jshint (we also have eslint on yeoman-generator, but we'll make a cut and probably keep only one)
node-jscs (code style checker)
jsdoc
Benchmarks (matcha)
Security linting (nsp)

Default build will use gulp, but we intend to provide the choice between:

Gulp
Grunt
Simpler raw npm build config (only jshint and mocha)

Any requests or comments is welcome! (Help marking this happens is also welcome!)

Started working on this on the node.next branch.
The goal is to have a series of self contained sub-generators that could be composed with by any other generators.
Basic sub-generators would be:

travis
package (ask questions and fill package.json)
readme (will take a bunch of command line options for badges and stuff)
editor config
jshint
jscs
git
build (will ask questions about the tasks we want to run) - initially we can make it just Gulp, but then we'd want three options (gulp, grunt and raw)

Sounds awesome.

@arthurvr feel free to send PRs, the more the merrier!

@SBoudrias The first one is already opened :lollipop:

Just making a TODO list out of what @SBoudrias suggested.

[x]  travis

[ ]  package (ask questions and fill package.json)

[ ]  readme (will take a bunch of command line options for badges and stuff)

[x]  editor config

[x]  jshint

[ ] jscs

[ ]  git

[ ] build (will ask questions about the tasks we want to run) - initially we can make it just Gulp, but then we'd want three options (gulp, grunt and raw)

[ ] license (I think there's one out there already, we could compose with it)

I'm on node:git ;)

There we go. PR opened for node:git.

On the jscs one but the PR will be for tomorrow.

You know what? PR opened :tada:

That was quick! yippee!

Started to work on the readme sub-generator

With my last PR, the last major roadblock for a first release will be the license sub generator.
There's generator-license we could update to be composable, but ATM it is not.
We also want to create boilerplate module + boilerplate test generator. With an option to skip them.
Then we need to update our documentation and ideally have a section about how to compose other generators on top of this one.
After, the next step will be:

[ ] Raw build system option (we could base ourselves on yeoman-environment who's not using Gulp. (see https://github.com/yeoman/environment/commit/d89bdc3a7888d303b485233b5cb0ee597cd624e9#commitcomment-9690814)
[ ] Ensuring reproducibility (we want to be able to easily re-run this generator on top of older projects and have it updated to the latest version with as few friction as possible)
[ ] Compose node based official generators on top of this one (generator-generator and generator-gruntplugin, and even generator-node! Let's dog food it). This process should help us improve our composition story, I'm pretty sure we'll need more option and make the code more clever around extending pre-existing files.

Couple things we'll need to fix with what we have now:

[ ] Should include coveralls badge in Readme

Made some update to the branch and finished the core things we needed to do before our first release.
Please try it out on your machine and report feedback so we have a first new solid release.

I just noticed one thingie when running gulp:
/Users/arthurvr/code/testing-node-gen/gulpfile.js:49
gulp.task('default', ["static", "test"]);
                      ^
SyntaxError: Unexpected token &

sounds like there's just some bad escaping going on

Oh, must be an ejs change. We'll need to output an unescape string there.

Is there anything missing on the list?

Nope, I'm in the process of releasing 1.0 in the next few minutes.

:tada:

:+1:

:horse:


Enable CD for catlight-plugin

Description
Configuring continuous delivery for CatLight plugin according to instructions at https://www.jenkins.io/doc/developer/publishing/releasing-cd/
Plugin repo: https://github.com/jenkinsci/catlight-plugin

Submitter checklist for adding or changing permissions

Always

[x] Add link to plugin/component Git repository in description above

When enabling automated releases (cd: true)

[ ] Add a link to the pull request, which enables continous delivery for your plugin or component.
Follow the documentation to ensure, your pull request is set up properly.

When adding new uploaders (this includes newly created permissions files)

[ ] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[ ] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[ ] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[ ] All newly added users have logged in to Artifactory and Jira at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @Wadeck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

Hey,
Please include a link to your pull request, where you enable CD in your plugin.
Thanks!

Link to the pull request: https://github.com/jenkinsci/catlight-plugin/pull/1


/nfs

Description

Steps to reproduce the issue:

Any other info e.g. Why do you consider this to be a bug? What did you expect to happen instead?
Technical details:

Portainer version:
Target Docker version (the host/cluster you manage):
Platform (windows/linux):
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer):
Target Swarm version (if applicable):
Browser:

Please open an issue with a proper title and description.


Support constrained and multi-objective functions?

Hi there,
I'm new to this library, and would like to know if emukit supports to constrained and multi-objective functions. Hope anyone could help to answer my problem. Any comments are highly appreciated.

Hi, thanks for the interest.
In terms of constrained functions, Emukit supports both constraints of the input space (see https://github.com/amzn/emukit/blob/master/emukit/core/constraints.py) and acquisition constraints (see https://github.com/amzn/emukit/blob/master/emukit/bayesian_optimization/acquisitions/probability_of_feasibility.py)
As far as multi-objective goes, we haven't implemented anything yet. But it should be very straightforward to extend emukit for this use case. And we always welcome PRs!

@jmren168 were you able to get the multi-objective going? Or did you find any other library which is cable of multi-objective optimization? Any hints would you really useful. Thank you.

BoTorch has a flexible support multiobjective optimization https://github.com/pytorch/botorch.


Release Notes XPLAT - Doesn't produce any release notes for new Rm environment

VSTS Extension you are using
Release Notes XPLAT
Where are you running it?
VSTS
Version of Extension/Task
2.x
Expected behaviour and actual behaviour
I just wanted to log this issue for discussion.
If you create a new environment in RM and use the release notes task, currently no release notes will be generated.
I'm not 100% sure what I would expect it to do, but the options seem to be to create releasenotes for the entire repo from the beginning of time, or to keep current behaviour and produce nothing.
Steps to reproduce the problem

Create a new environment in RM
Add the task

I think this was reported last week in #319 - I produced a fix and shipped it. It was just as case of on the first run making sure the compare release to be the same current release. So really minor.
Is this the same issue?

I think it's subtly different.  This one is about what releasenotes should be included if it's never been deployed before.  Should it be either all the commits since the beginning of time, or none (current behaviour).

I would go with all commits, but I think this is a personal choice ?
Maybe there should be an advanced checkbox option that defaults to the current behaviour, which I think is show all?

I think it would be a waste of effort to build in a choice.  It only matters on the first run.
I agree with you that it should be all commits.

Isn't that the default now?

I'm afraid not.  It shows nothing.  That's why I created the issue.

Taken forever but shipped in 2.81


Support SDK Version 19

In order to support SDK Version 19 (forcing it to use SDK 19 in manifest and successfully compiled), use another constructor that still support SDK version 19. But I don't know if this change will break other feature like Jetpack Compose or not.

I'm closing this issue since there is already a solution for decreasing minSdk to 19 without manifest overrides. #57


[RedditBridge] Add option to choose for New, Hot and Top submissions

Also, i'm adding a line break for each image in the gallery to avoid broken url issue when using MonitoRSS bot.

gj!


ajudst ogr time type to json format

[ ] I added an entry to CHANGELOG.md if knowledge of this change could be valuable to users.

Here is a brief summary of what I did:

 UI für OGR Datasets

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
8 unchanged lines in 7 files lost coverage.
Overall coverage increased (+0.004%) to 91.023%

Files with Coverage Reduction
New Missed Lines
%

services/src/pro/api/handlers/users.rs
1
95.75%

services/src/api/handlers/layers.rs
1
88.81%

services/src/pro/quota/mod.rs
1
94.12%

services/src/datasets/external/aruna/mock_grpc_server.rs
1
82.93%

services/src/api/handlers/wcs.rs
1
89.16%

services/src/datasets/external/pangaea/mod.rs
1
86.66%

services/src/pro/contexts/postgres.rs
2
96.78%

Totals

Change from base Build<PHONE_NUMBER>2:
0.004%

Covered Lines:
133704

Relevant Lines:
146890

💛  - Coveralls


Bump mock to 2.0.0

Hmm, struggling with the travis error
Potentially related: https://github.com/pypa/setuptools/issues/945
https://github.com/pypa/setuptools/issues/942

@idavidmcdonald Updated commit comment and rebased

Thanks Ben 👍


How to use from Java code?

We're using react-native-i18n for our project localization.  I now need to fetch a localization string from Java code for Toast.makeText.  How can this be done?  I couldn't find any instructions anywhere.

Form React-native documentation:
ToastAndroid.show(t('translation_key'), ToastAndroid.SHORT);

https://facebook.github.io/react-native/docs/toastandroid.html

I'm not trying to generate a toast from react code, I'm trying to generate it from native Android Java code.  How can I get a localization string from within Java code?

You can't do JS -> Java, only expose native module to JS.