chore: update evm

Purpose
EVM branch https://github.com/Taraxa-project/taraxa-evm/pull/97
How does the solution address the problem

Changes made

prnet cleaned-up successfully


Add IDictionary<string, string> Properties on MongoClaim #54

Description
System.Security.Claims.Claim contains a property named Properties of type IDictionary<string,string> which doesn't exist in MongoClaim. This property can be used to store additional metadata which can be consumed at runtime which is currently not possible. This change should enable persisting Properties from Claim to MongoClaim and retrieve them later.

@alexandre-spieser  Request you to take a look.

This is not supported by MSFT's Identity package either.
https://github.com/dotnet/aspnetcore/blob/05b2afb73b55b11d8cb3691dedfebba1f0285332/src/Identity/Extensions.Stores/src/IdentityUserClaim.cs
IdentityUserClaim's do not have such properties.
You'll have to find another way.

I was easily able to customise for MSFT's Identity package for SQL database. However, I am not finding a good way to do the same for Mongo as I am not sure how do I hook a derived implementation of MongoClaim.  Please let know if it would  be ok with you if I include this project as a dependency and customise as required for my needs instead of using packaged nuget from this repository. Alternatively, I can raise a new jira and follow up with a PR to do some refactoring that might allow custom implementations of MongoClaim to be hooked for extensibility. Please advice.

you could inject a custom implementation of:
public class MongoRoleStore<TRole, TContext, TKey, TUserRole, TRoleClaim> :
IQueryableRoleStore,
IRoleClaimStore
where TRole : MongoIdentityRole
where TKey : IEquatable
where TContext : IMongoDbContext
where TUserRole : IdentityUserRole, new()
where TRoleClaim : IdentityRoleClaim, new()
TRole has a list of Claims, but you could extend this class to have a list of ClaimProperties , and override the necessary methods to achieve the behaviour needed?

Using:
public class ApplicationRole :  : MongoIdentityRole
{
/// custom prop storing claim metada
public List ClaimProperties { get;set; }
}
Then customer rolestore with an override on :
///
/// Get the claims associated with the specified  as an asynchronous operation.
///
/// The role whose claims should be retrieved.
/// The  used to propagate notifications that the operation should be canceled.
/// A  that contains the claims granted to a role.
public virtual async Task<IList> GetClaimsAsync(TRole role, CancellationToken cancellationToken = default(CancellationToken))
{
ThrowIfDisposed();
if (role == null)
{
throw new ArgumentNullException(nameof(role));
}
    return await RoleClaims.Where(rc => rc.RoleId.Equals(role.Id)).Select(c => new Claim(c.ClaimType!, c.ClaimValue!)).ToListAsync(cancellationToken);
}

Src:
https://github.com/dotnet/aspnetcore/blob/05b2afb73b55b11d8cb3691dedfebba1f0285332/src/Identity/EntityFrameworkCore/src/RoleStore.cs#L361

To ensure you stuff gets injected, look at:
https://github.com/alexandre-spieser/AspNetCore.Identity.MongoDbCore/blob/master/src/Extensions/MongoDbIdentityBuilderExtensions.cs
https://github.com/alexandre-spieser/AspNetCore.Identity.MongoDbCore/blob/b9d2d1a5ac3f5ce5577d9ce715dd0a68ae6b33f1/src/Extensions/MongoDbIdentityBuilderExtensions.cs#L82
and call:
            builder.Services.TryAddScoped<IRoleStore<TRole>>(provider =>
            {
                return new MyCustomMongoRoleStore<TRole, IMongoDbContext, TKey>(provider.GetService<IMongoDbContext>());
            });

Apologies I could not reply earlier . My project supports using multiple databases ( postgres, microsoft sql, mongodb, etc)  through plugins .  Maintaining ClaimProperties  on ApplicationRole  is forcing me to handle authentication logic differently for mongo plugin and breaking some abstractions. Highly appreciate your effort to suggest this workaround . However, this is not working well for my scenario. I will try to refactor  and see If I can make it simpler to customise Claim in a similar way I could do with AspNetCore.Identity.EntityFrameworkCore package.


Remove optimizer unless code

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

run starrocks_fe_unittest


Updatin DPC++ compiler version

Description
Please include a summary of the change. For large or complex changes please include enough information to introduce your change and explain motivation for it.
Changes proposed in this pull request:

@PetrovKP  @michael-smirnov  - there are still failures for examples. However i think it worth merging update with build on new compiler


fix: remove accountid property from TunnelListParams

A trivial PR to remove the AccountID property from the TunnelListParams struct in tunnel.go.
Types of changes
What sort of change does your code introduce/modify?

[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 change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] This change is using publicly documented (api.cloudflare.com or developers.cloudflare.com) and stable APIs.

Codecov Report

Merging #993 (29dc450) into master (f91b3b5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #993   +/-   ##
=======================================
  Coverage   48.94%   48.94%
=======================================
  Files         108      108
  Lines       10384    10384
=======================================
  Hits         5082     5082
  Misses       4194     4194
  Partials     1108     1108

Impacted Files
Coverage Δ

tunnel.go
37.44% <ø> (ø)

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 f91b3b5...29dc450. Read the comment docs.


Use new InternalError().initCause(e) for < java8 compat

This should restore buildability with java7.
Highly recommend you look this related PR to help avoid brining in incompatible JDK usage in the future:
https://github.com/orientechnologies/orientdb/pull/4661

@jdillon Thanks again.


[2.0] Fixing a couple of issues found during functional testing

The temporary credentials method needs to be POST and I added a ksort to the OAuth Authorization header key-value pairs.

@bencorlett any thoughts on this or the other 2.0 PRs?

This looks great.
I'm back in the office and getting back into OS now :)


Can install since the signing certificate has expired

App will not install due to being signed with an certificate which expired last year.

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

ID: b86dc038-4654-aeed-09db-8e702fba499c
Version Independent ID: 50ec40a3-0fcb-1d68-782d-d1efbaf69312
Content: AppInstaller File Builder tool - MSIX
Content Source: msix-src/toolkit/msix-toolkit-appinstallerfilebuilder.md
Product: msix
GitHub Login: @RDMacLachlan
Microsoft Alias: romaclac

You NEED to timestamp so apps can install after the cert expires.  I use sectigo eith Device Guard.  https://sectigo.com/resource-library/time-stamping-server.   Just take the timestamp url (http://timestamp.sectigo.com) and add it when you're signing.

AppInstallerFileBuilder_1.2019.1001.0.msix installs now.  Looks like it was re-signed back in June. Closing.


Hello World

Generated a new rails project with webpack and postgres
Installed react dependences
Generated Message model
Created messages_controller api endpoint
Created redux store for messages
Created messagesSlice

Hello @Omar-Muhamad i didn't notice that my readme file was incomplete, really sorry. Thank you


[ko] Unsupervised Learning

i`m progressing =]

Hello
I finish translation of Unsupervised Learning =]
but i have a question
No.51 The Machine Learning cheatsheets are now available in Japanese.
this cheatsheets for Korean so i think Japanese should change Korean.

Thanks for your work @kwanghyeokahn!
You are right -- the template should indicate Koran instead.

Thx for answer it  =]
Have a good day ~

Great job @kwanghyeokahn and thank you for your prompt response!
Now, we can get started with the review process (cc: @WooilJeong).

It's perfect!

good translation!


get with missing slash and cookies

I was using capybara to test my application and it was calling Rack::Test#get with http://www.doorkeeper.jp. As this did not include a trailing slash, cookies were not being set (as uri.path was nil, which doesn't match the regex), although the request otherwise responded normally. I don't want anyone else to go through the debugging headache, so it would be nice to fix this. Is RackTest the appropriate place to fix this? If so, what is your recommended approach (warning, exception, add the slash, etc)?

Hi Paul -- I'm unsure of the correct place to fix the issue you saw. Can you try and boil the behavior down to an example that uses Rack-Test only (no Capybara)? That will make it easier to identify if Rack-Test is performing the correct behavior. If you do this, I'm sure we can help.
Thanks,
-Bryan

Closing because of lack of feedback.


Fix for the Self-XSS

Changes

Added HTML sanitisation.

Issues

Fix for the self XSS #GHSA-694f-jjpc-58g9

Thanks for the pr!


update(query): update App Service Not Using Latest TLS Encryption Version query to the latest version

Latest TLS Encryption Version is 1.3
Closes #
Reason for Proposed Changes
Proposed Changes

I submit this contribution under the Apache-2.0 license.

Hey @anterosilva1985,
Nice contribution and great update to the query! 🎉
To make the test cases more comprehensive and complete, I would suggest the following changes:
1 - Add a positive expected result (positive2.tf) to signal an invalid configuration;
2 - Update positive_expected_result.json with the new expected result added;
3 - Update negative1.tf modifying the file with the new valid encryption level (1.3);

 Negative1.tf update suggestion
resource "azurerm_app_service" "negative1" {
  name                = "example-app-service"
  location            = azurerm_resource_group.example.location
  resource_group_name = azurerm_resource_group.example.name
  app_service_plan_id = azurerm_app_service_plan.example.id

  site_config {
    dotnet_framework_version = "v4.0"
    scm_type                 = "LocalGit"
    min_tls_version = 1.3
  }
}

 Additional Positive2.tf result && Updated Positive_expected_result.json suggestion
We could modify the current negative1 file to reflect the new invalid encryption level, such as using a version lower than 1.3 (e.g., encryption: 1.1 and 1.2 with the new query update).
This ensures the query accurately flags cases with unsupported encryption levels and effectively warns new users with the new valid encription level, which needs to be equal or higher than 1.3.
resource "azurerm_app_service" "positive2" {
  name                = "example-app-service"
  location            = azurerm_resource_group.example.location
  resource_group_name = azurerm_resource_group.example.name
  app_service_plan_id = azurerm_app_service_plan.example.id

  site_config {
    dotnet_framework_version = "v4.0"
    scm_type                 = "LocalGit"
    min_tls_version = 1.2
  }
}

Regarding the positive_expected_result file, we could just add a new entry, updating the file as demonstrated below:
[
  {
    "queryName": "App Service Not Using Latest TLS Encryption Version",
    "severity": "MEDIUM",
    "line": 10,
    "fileName": "positive1.tf"
  },
  {
    "queryName": "App Service Not Using Latest TLS Encryption Version",
    "severity": "MEDIUM",
    "line": 10,
    "fileName": "positive2.tf"
  }
]

These are just community suggestions, but they would fix the failing unit tests and, I believe, complete your query update while enhancing the test cases a little bit. 😄
Thanks!


Convert .NET Interactive notebooks' kernel spec to a VS Code compatible kernel when saving.

We have to do some conversion for .NET Interactive notebooks so that their kernel spec matches our internal representation. This is because our Jupyter notebook kernels line up with the contributed kernels from our notebook extension, but .NET Interactive ipynb notebooks have kernels that don't match the kernel contributed from the Interactive extension. Our code is built on top of Jupyter kernels originally, so a kernelspec is often our source of truth for a lot of notebook details. Thus we need to change up the Interactive notebook's kernelspec in order to get VS Code API compatibility to work. We were already doing this conversion when opening an existing Interactive notebook, but not when changing to the .NET Interactive kernel on the fly in an opened notebook. This change adds that converted metadata.
An example of what this is doing:

This is an internal kernel representation we use to get compatibility working:

{
  name: 'jupyter-notebook', // Matches the notebook provider name
  language: 'dotnet-interactive.csharp', // Matches the contributed languages from Interactive extension
  display_name: '.NET Interactive' // The kernel name we need to show in our dropdown to match VS Code's notebooks
}

This is the kernel spec that notebook would need to be saved with to work in VS Code:

{
  name: '.net-csharp',
  language: 'C#',
  display_name: '.NET (C#)'
}

I also cleaned up some constants that get used in this conversion step in vscodeExecuteProvider.ts and notebookInput.ts.
This PR fixes #19162

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 22 (9.09%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 42.183%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/services/notebook/browser/models/notebookModel.ts
0
1
0.0%

src/sql/workbench/contrib/notebook/browser/models/notebookInput.ts
0
3
0.0%

src/sql/workbench/api/common/notebooks/vscodeExecuteProvider.ts
0
16
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
27752

Relevant Lines:
61544

💛  - Coveralls

Interactive already has to do similar language wonkiness when creating their own notebooks in vscode. Here's an example from their extension: https://github.com/dotnet/interactive/blob/main/src/dotnet-interactive-vscode-common/src/commands.ts#L202
In that section, if you were to create a new C# notebook, then dotnet_interactive.language in the cell's metadata would be "csharp", the kernelspec's name would be ".net-csharp", and the language ID used for the vscode.NotebookCellData object would be "dotnet-interactive.csharp".


[WIP] serve.js

I have written most components required for offline signing server
And CA part is removed since client should talk with CA server directly, unless you need a proxy helper during development.
Signed-off-by: david<EMAIL_ADDRESS>

Sorry for the late reply david.
I have run the version of this pr with my frontend app, and it seems quite incompatible with it.
First, the host on my machine is undefined:
http server started at { host: undefined, port: 8080 }

and when I request the function on my frontend app, it has no response (not sure whether it's because of the host problem then...). And other functions on the fabric-client-flutter repo will also need to be changed (please refer to the fabric-client-flutter repo).
I'm sequencing all the original commits to one signed commit and prepare to submit it to hyperledger-labs.
At this time I'll temporarily close this pr and migrate this repo to the hyperledger-labs repo.
Welcome to re-commit your pr.
Thanks.


Unify tls-auth and tls-crypt codepaths more

We now have instead of hmac_and_out and encrypt_and_out a function wrap_and_out that decides by inspecting control_crypto how to encrypt and authenticate.
Similarly, we now have wrap_control for both tls-auth and tls-crypt. This allows us to merge handle_client_{tls_auth,tls_crypt} into handle_client.
I don't like that it introduces assert false in Static _ cases.
We should also be able to merge the non-decoding parts of incoming in a similar way. This may require refactoring code into function(s).

to me this looks great


ERROR: duplicate key value violates unique constraint "schema_migrations_pkey"

Bug
Bug was encountered while onboarding a new employee. He cloned our code repository which contained our supabase folder.
The supabase folder has the following structure:

supabase

migrations -> where all of our migrations are defined in .sql files
config.toml
seed.sql

Then he ran the commands in the following order after cloning the repository and clean installing supabase:

supabase start to start the local dev stack
supabase db reset to reset the database, apply migrations, and seed the database

Note that we don't use supabase db link for local development, we manage all our migrations to the remote db with separate actions triggered by PRs.
He encountered the following error when supabase db reset tried to make the first migration:
ERROR: duplicate key value violates unique constraint "schema_migrations_pkey" (SQLSTATE 23505)
At statement 16: INSERT INTO supabase_migrations.schema_migrations(version, name, statements) VALUES($1, $2, $3)

Note: statement 16 is the last statement in our first migration file, so it appears that the issue comes not from the actual migration but when the migration is being recorded into the migrations table
We removed the migrations and seed data from his supabase folder, and have him run supabase db reset again and noticed that no schema_migrations schema was created (should appear below schema_functions):

Expected behavior
For the rest of our team, this process runs without a hitch, properly applying migrations and seed data. Leading me to believe it may have something to do with the install, but we've tried a clean install to no avail.
Reason to suspect issue
Have seen that there are other issues related to insertions into the schema_migrations table, although these seem to occur after linking and pulling. The issue we are experiencing however, appears to just occur during local development.

Which version of cli are you running? Could you try npx supabase@beta start just to be sure?

turned out something was wrong with the employee's postgres install, closing

For those who are coming to this issue in the future, I had this issue on my local development because my migration timestamps were the same. To fix it, just recreate one of the migrations with a later timestamp


unmet peer dependency

<EMAIL_ADDRESS>/Users/olivier/projects/cpuk/vuetober/themes/cpuk
├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

Closed by 525bd34b77e9cf088a5f7bc7030f147257f1a371


Add Kitaoka

(1) I added 2111091   - Tetsuya Kitoaka<br/> to Contoributors.md.
(2) I added 2111091, Tetsuya Kitaoka to SE-class-2021.

Congratulations on your Pull Request! See you in the next class.


improve sizing of footer

Fixes Issue
Closes #206
Changes made-

Improved the sizing of footer elements for the mobile viewport.
Increased the size of social logos in the footer.
Increased the size of the copyright text in the footer.
Increased the size of the mail link in the footer.

Screenshots
Earlier -

Now-

please send me the ss of desktop view too

please send me the ss of desktop view too

The desktop view is still the same. I haven't made any changes to it.
Do you have any changes for the desktop view too?

I just made another change-

I improved the spacing between the mail icon and the email in the footer.
There was no margin in between these elements so I've added margin in between them.

can you increase the gmail icon a bit in the mobile view

can you increase the gmail icon a bit in the mobile view

Should I increase it more? or is it fine?

LGTM
thank you for the contribution

LGTM
thank you for the contribution

Thanks for the merge!!


EF Core 2.0.1 Generating Nullable Timestamp Column

Hi All,
I'm looking at a freshly created database after having upgraded to EF Core 2.0.1. I have one table that has a timestamp column. In my existing production and QA databases, the column is NOT NULL, and in a freshly created DB using 2.0.1 it is nullable.
Existing Prod & QA (created w/1.1.2):
create table ...
[Timestamp] [timestamp] NOT NULL,
....

New DB Freshly Created (w/2.0.1):
create table ...
[Timestamp] [timestamp] NULL,
....

No biggie, but I do like my SQL Compares to come out clean. So I tried changing the nullability using an alter column:
alter table [dbo].[MeetUp] alter column [Timestamp] [timestamp] NULL

And that leads to an error:
Msg 4927, Level 16, State 1, Line 13
Cannot alter column 'Timestamp' to be data type timestamp.

I guess what I'm really looking for is help with migrating this change... I'm not really an expert enough to know why y'all changed the nullability.
Thanks!
-Brian Eriksen
Further technical details
EF Core version: 1.1.2 & 2.0.1
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Windows 10
IDE: Visual Studio 2017 update 4.4

PS - The line of code that defines the column in my model class is:
public byte[] Timestamp { get; set; }

and the line of code that configures / describes the Timestamp column is:
builder.Entity<MeetUp>().Property(p => p.Timestamp).ValueGeneratedOnAddOrUpdate().IsConcurrencyToken();

Note for triage: In 1.1.2 I see this generated:
CREATE TABLE [MeetUps] (
    [Id] int NOT NULL IDENTITY,
    [Timestamp] rowversion,
    CONSTRAINT [PK_MeetUps] PRIMARY KEY ([Id])
);

I 2.0.1 I see this:
CREATE TABLE [MeetUps] (
    [Id] int NOT NULL IDENTITY,
    [Timestamp] rowversion NULL,
    CONSTRAINT [PK_MeetUps] PRIMARY KEY ([Id])
);

I'm guessing, but I haven't confirmed, that this is another consequence of #8146 in that the default is non null but now we are explicitly specifying it as null.

We discussed this in triage and decided:

The new behavior is "correct" in that the property is nullable/optional in the model and so making in nullable/optional in the database reflect that.
However, it shouldn't really matter in most cases because SQL Server is internally handling the value, so behavior is unlikely to change
For normal use of Migrations, no new migration will be generated because there is no change in the nullability of the property in the model
If an existing (old) database is reverse engineered, then the property will become required even if it wasn't originally created from a required property, but again this should not impact behavior.

So as of right now, this has become accidentally correct without any major consequences. If major consequences are discovered, then we can reconsider.
Closing as by-(accidental)-design.

Thought I would contribute to this conversation as I was struggling to get this to work correctly, primarily due to a bug in NumberToBytesConverter.
My attempts to use ulong as my class RowVersion value, and the Timestamp type in SQL Server failed in multiple ways. Initially I was getting the column showing up as varbinary in SQL Server, and really what I wanted was the Timestamp type. I also don't want to use a byte[] in my class as it makes comparisons less straightforward. So in specifying the Timestamp type on the column, a ulong in my class, and having tried the suggestions above leads to EF throwing an error when fetching the data because for some reason it's being passed an empty or null byte array (I checked, all database rows have a populated timestamp value). Also, when specifying a Timestamp column type the auto-generated migration tries to apply a defaultValue which is invalid. Here's how I got it to work:
Table model definition:
public class MyTable {
  ...
  [Timestamp]
  public ulong RowVersion { get; set; }
}

DbContext:
builder.Entity<MyTable>()
                .Property(p => p.RowVersion)
                .HasConversion<byte[]>(v => BitConverter.GetBytes(v), v => BitConverter.ToUInt64(v != null && v.Length > 0 ? v : new byte[] { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }))
                .HasColumnType("Timestamp")
                .IsRowVersion();

Migration:
migrationBuilder.AddColumn<byte[]>(
  name: "RowVersion",
  table: "MyTable",
  type: "Timestamp",
  nullable: false);
  // defaultValue: new byte[] { }); -- remove this, invalid on a Timestamp type column


CAPOperator CRD change

Moved dnsTarget out of subscriptionServer.
Added oneOf required to CRD for dnsTarget and ingressGatewayLabels
Adopted transformer and the tests

Test Image- ghcr.io/anirudhprasad-sap/cap-operator-lifecycle/cap-operator-manager:test


feat: add sync from awf-latest -> awf-latest-xx1

Signed-off-by: Takayuki Murooka<EMAIL_ADDRESS>PR Type

New Feature

Related Links

Description
sync of autoware_launch:awf-latest -> autoware_launch:awf-latest-xx1
FYI: autoware_launch:awf-latest-xx1 is the branch which is applied xx1's parameters (vehicle_model, planning, ...) to autoware_launch:awf-latest

Review Procedure

Remarks

Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[x] Code follows coding guidelines
[x] Assign PR to reviewer

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code follows coding guidelines
[ ] (Optional) Unit tests have been written for new behavior
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets
[ ] Write release notes

CI Checks

Build and test for PR: Required to pass before the merge.
Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

@tkimura4 Could you review this.


Add warning of Jupyter kernel shutdown upon closing vscode window

Currently, if I close a VS code window running Jupyter, underlying kernels (and their state) will be lost.
I suggest that VS code should warn the user upon closing that this will happen, and get a confirmation if they want to go ahead anyway.

@davidefiocco . Thanks for the suggestion on this. Currently we shutdown the kernels that we start locally ourselves but we don't shutdown severs that we connect to via a remote URI (Via "Python->Specify local or remote Jupyter server for connections" setting). If you wanted to keep a local server running you could start it directly from the command line, and then connect to as if it was a remote server, which should keep us from shutting it down on close. Not the cleanest perhaps, but it does work as a workaround.
We'll discuss adding a warning or possibly a way to keep local servers running at our triage meeting.

Thanks for the feedback! However, we don't have plans on adding this functionality at this time.

Any chance this issue could stay open for some future re-evaluation? @greazer Thanks!


Remove deprecated functions for 1.10

From CHANGES:

Removed InfoBoxViewModel.defaultSanitizer, InfoBoxViewModel.sanitizer, and Cesium.sanitize, which was deprecated in 1.7.
Removed InfoBoxViewModel.descriptionRawHtml, which was deprecated in 1.7.  Use InfoBoxViewModel.description instead.

Removed GeoJsonDataSource.fromUrl, which was deprecated in 1.7. Use GeoJsonDataSource.load instead. Unlike fromUrl, load can take either a url or parsed JSON object and returns a promise to a new instance, rather than a new instance.

Removed GeoJsonDataSource.prototype.loadUrl, which was deprecated in 1.7.  Instead, pass a url as the first parameter to GeoJsonDataSource.prototype.load.
Removed CzmlDataSource.prototype.loadUrl, which was deprecated in 1.7.  Instead, pass a url as the first parameter to CzmlDataSource.prototype.load.
Removed CzmlDataSource.prototype.processUrl, which was deprecated in 1.7.  Instead, pass a url as the first parameter to CzmlDataSource.prototype.process.
Removed the sourceUri parameter to all CzmlDataSource load and process functions, which was deprecated in 1.7.  Instead pass an options object with sourceUri property.

CC @mramato

Also removed PolygonGraphics.positions which was deprecated in 1.6. Instead, use PolygonGraphics.hierarchy.

I'll take a look at this tonight or early tomorrow.  The PolygonGraphics.positions removal might have some gotchas (but I don't remember exactly what).

Thanks for taking care of this @bagnell, it all looks good to me.  My concern over PolygonGraphics.positions was unfounded, I was simply remembering that CZML still didn't support polygons with holes and was worried it was related, but it isn't.  I did make one minor tweak to the InfoBoxViewModel to get rid of a no longer necessary property.


open or off video while making calls

Hi all,
i want open or off video while making calls. But session no method, please help me.
thanks all and good luck today.

Hi @quanvq,
The mute function, with the correct arguments, will do this for you.
Please direct quesitons like this to our mailing list in the future. We try to keep this space for bugs or feature requests.
James

please demo for me. thanks you
ex: i want on,off video media while making calls
thanks you,

we'll get right on that.


Bugfix for Keyboard Controls

Variable got the wrong name, so the if condition never got to true and you couldn't split/feed

This pull request is now managed with CodeReviewHub. Refresh.


Can't recognize "between now and date" daterange #20712

Can't recognize "between now and date" daterange:
Query: between now and November 15th
wrong answer: two text: "now" "November 15th"
right answer: a daterange text: "between now and November 15th"
Supported in .NET, java
also fix"between today and date"
wrong answer: number of days with decimal point
right answer: Integer days
Supported in .NET, java

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


METRICS-4792 : OpenTelemetry Traces Protobuf InputFormat

Description

Add an OpenTelemetry Traces InputFormat so that Druid can ingest records that are in the form of an OpenTelemetry Traces Protobuf.

Note: This does not cover some fields in Otel Trace proto such as Events and Links. The extension as a whole is meant to be contributed upstream for further improvements and maintenance.

Key changed/added classes in this PR

Refactor common functionality between Metrics and Traces into abstract classes OpenTelemetryInputFormat / OpenTelemetryProtobufReader
OpenTelemetryTracesProtobufReader
OpenTelemetryTracesProtobufInputFormat

Testing

Unit tests
Add Benchmark test for traces

Benchmark                                  (instrumentationScopeCount)  (resourceSpanCount)  (spansCount)   Mode  Cnt       Score       Error  Units
OpenTelemetryTracesBenchmark.measureSerde                            1                    1             1  thrpt    5  290712.694 ± 16258.114  ops/s
OpenTelemetryTracesBenchmark.measureSerde                            1                    1             2  thrpt    5  179720.468 ±  3255.420  ops/s
OpenTelemetryTracesBenchmark.measureSerde                            1                    1             4  thrpt    5   97546.502 ±  2833.230  ops/s
OpenTelemetryTracesBenchmark.measureSerde                            1                    1             8  thrpt    5   54750.250 ±  1943.527  ops/s
OpenTelemetryTracesBenchmark.measureSerde                            1                    2             1  thrpt    5  145419.049 ±  3366.026  ops/s
OpenTelemetryTracesBenchmark.measureSerde                            1                    2             2  thrpt    5   90352.972 ±  3579.583  ops/s
OpenTelemetryTracesBenchmark.measureSerde                            1                    2             4  thrpt    5   48941.613 ±  5208.820  ops/s
OpenTelemetryTracesBenchmark.measureSerde                            1                    2             8  thrpt    5   27515.141 ±  3776.942  ops/s
OpenTelemetryTracesBenchmark.measureSerde                            1                    4             1  thrpt    5   72962.145 ±  2408.336  ops/s
OpenTelemetryTracesBenchmark.measureSerde                            1                    4             2  thrpt    5   45559.232 ±  1580.323  ops/s
OpenTelemetryTracesBenchmark.measureSerde                            1                    4             4  thrpt    5   26834.922 ±  1049.008  ops/s
OpenTelemetryTracesBenchmark.measureSerde                            1                    4             8  thrpt    5   13932.725 ±   739.088  ops/s
OpenTelemetryTracesBenchmark.measureSerde                            1                    8             1  thrpt    5   38448.021 ±  1102.249  ops/s
OpenTelemetryTracesBenchmark.measureSerde                            1                    8             2  thrpt    5   23787.015 ±   775.349  ops/s
OpenTelemetryTracesBenchmark.measureSerde                            1                    8             4  thrpt    5   13311.163 ±   299.108  ops/s
OpenTelemetryTracesBenchmark.measureSerde                            1                    8             8  thrpt    5    7073.504 ±   336.945  ops/s

Local testing

Deployed druid locally.
Ingested Kafka stream with Otel trace data.
Verified the data in the data source.

This PR has:

[x] been self-reviewed.
[x] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[x] been tested in a test Druid cluster.

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


AprilTagPoseEstimator yaw repositioning

ATPE is not working correctly when yaw angles to the tag are supplied, heading information is also noisy

[x] Rot. matrix
[x] Kalman filter offset issue (the 360s bug)

Partial fix in place, for future reference in v6.0.0 this component should be done nicer by working with a new Localizer that uses deltas directly


Fixed the width style of 'img' in 'Markdown'

first PR

@thomasJang thanks for submitting PR. I think a 100% width there will cause other issues... were you able to verify it on mobile and desktop? assigning to @xdamman who was talking to you about it.

100% width no problem. because 'img' must be child of '.Collective-markdown'


Test PR to determine results of rebase and merge for virtualizer migration

This is an experiment to see the results of the "rebase and merge" option when trying to merge the lit-virtualizer-migration branch into main.  The lit-virtualizer-migration-rebase-destination is a branch just cut from main and so the results from this operation should tell us what we need to know to determine whether to proceed with this strategy to close out PR #2962

Choosing "Rebase and merge" presents a disabled button with the message "This branch cannot be rebased due to conflicts"


[vcpkg_acquire_msys] add findutils as a dependency to libtool

libtool sometimes requires find to be available.
Without this dependency libtool is partly unuseable.

/azp run

/azp run


remove annoyting static status bar

What steps will reproduce the problem?
1. install nostalgy

What is the expected output? What do you see instead?
A clean status bar.

What version of the product are you using? On what operating system?
<IP_ADDRESS> on TB24 on Ubuntu 12.04

Please provide any additional information below.

See also https://groups.google.com/forum/#!searchin/nostalgy-extension/status

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 4 Oct 2013 at 2:12

I'm not exactly sure what the reporter means:

They would like to hide the save (S) copy (C) go (G) element from the status bar (e.g. with a preference).
They would like to hide the status bar, which can already be done in TB via the menu: View / Toolbars / Status Bar*

Since this is not clear, I'm closing this issue for now. Feel free to open a new issue with a more detailed description of the expected behavior.


How to create an rviz MarkerArray display in code?

Hi,
I'm writing an rviz plugin and am attempting to write some code which automatically creates a MarkerArray display when a button is clicked. I'm already doing the same with a Marker display using the VisualizationManager::createDisplay() method, but can't find any docs on how I would do the same with a MarkerArray. I did see writing which suggested that Marker can also display MarkerArray messages. This allowed me to add the Display, but nothing rendered when the topic was set correctly.
Any help would be much appreciated!
Tom

Just use createDisplay("rviz/MarkerArray", "display name", true)?


There's no VS2010-15.Integration.MD.props or VS2010-15.Integration.MT.props file

I installed the SR300 SDK, and trying to set up a C++ develop environment, but there's no VS2010-15.Integration.MD.props or VS2010-15.Integration.MT.props file on my entire computer

In my installation of the 2016 R2 SDK, the props files can be found at this SDK folder location:
C: > Program Files (x86) > Intel > RSSDK > props

@MartyG-RealSense I found my RSSDK path: "C:\Windows\System32\config\systemprofile\AppData\Local\Intel\RSSDK", but nothing exists.
Which one did you install?I am installing "Intel.RealSense.SDK.exe"on github

There are two RSSDK paths installed.  The correct one to find the props is the one I quoted.  The one you linked to stores data related to the SDK, I believe.
The props files have no relation to the current 'RealSense SDK 2.0' on the GitHub.  They are meant for use with the older '2016 R2' or '2016 R3' SDKs.
I have attached to this message a zip of the props folder so you can get the props files.
props.zip
RealSense SDK 2.0 uses a different method for setting up a development environment, based around a program called CMake.

@MartyG-RealSense I want to realize the function of face recognition. The old version of SDK document has related introduction, but the new version of SDK document does not. It's worrying, so try the old version of SDK.

3DiVi (makers of the popular Nuitrack skeletal tracking software) have a face recognition SDK for $19.99 a month.   It has a free trial version so you can test whether it will work with your SR300.
https://facesdk.3divi.com/
If you would prefer to develop your own face recognition solution then the older '2016 R2' SDK will likely be the best way (face recognition was less stable on the 2016 R3 SDK).
Assuming that you have the SR300's drivers installed and working correctly, the next step will be to download and install the 2016 R2 SDK using the link below, which launches a 1.8 gb download of the installer file in your browser.
http://registrationcenter-download.intel.com/akdlm/irc_nas/vcp/9078/intel_rs_sdk_offline_package_<IP_ADDRESS>6.exe

@MartyG-RealSense I'm not very good at development, so I tried the older SDK first.Thank you.

I posted a link above for the 2016 R2 SDK if you do not have it already.

@MartyG-RealSense  I just downloaded it and am installing it.Thank you.


analysis service:  pub upgrade failed. ERR: The current Dart SDK version is 2.0.0.

ERR: The current Dart SDK version is 2.0.0.

 Because split_panel depends on logging_handlers >=0.5.0+2 which requires SDK version >=0.5.0+1.r21823 <2.0.0, version solving failed.

This is not an error from a pub point of view, but the correct outcome of analyzing a package that does not have Dart 2 ready dependencies. We should demote this particular case (which will need to tell apart from other cases) to a note. I'm not even sure if any pub version solving problems should be logged as errors or warnings, only spectacular pub failures (unable to query packages) should be logged as errors.
These errors are currently obscuring all other errors on the analysis service.

Likewise, there are also errors for packages that fail to parse during dartfmt.

pana/three/0.2.5+1: Failed dartfmt

Error:
    Exception: dartfmt on lib/ failed with exit code 65
    Could not format because the source could not be parsed:

    line 66, column 24 of lib/extras/controls/firstpersoncontrols.dart: Unexpected text '?'.
        this.domElement = (?domElement) ? domElement : document.body;
                           ^
    Could not format because the source could not be parsed:

    line 62, column 25 of lib/extras/controls/trackball.dart: Unexpected text '?'.
        this.domElement = ( ?domElement ) ? domElement : document;
                            ^
    Could not format because the source could not be parsed:

    line 42, column 8 of lib/extras/tween.dart: Unexpected text '?'.
      if (!?time) {
           ^
    line 75, column 10 of lib/extras/tween.dart: Unexpected text '?'.
        if ( ?duration ) {
             ^
    line 110, column 19 of lib/extras/tween.dart: Unexpected text '?'.
        _startTime = (?time)? time : new DateTime.now().millisecondsSinceEpoch;

While this is a package issue, this error is a correct outcome of the analysis, and should not be logged as an error on the pub site itself.

I am seeing this in the dartdoc service as well:

pub.dartdoc.runner: Error while running pub upgrade for susi 0.1.1.
exitCode: 1
stdout: MSG : Resolving dependencies...
stderr: IO  : Get versions from https://pub.dartlang.org/api/packages/uuid.
IO  : HTTP GET https://pub.dartlang.org/api/packages/uuid
    | Accept: application/vnd.pub.v2+json
    | X-Pub-OS: linux
    | X-Pub-Command: upgrade
    | X-Pub-Session-ID: 6D575222-E168-4017-A04C-860C6C4F052C
    | X-Pub-Environment: bot.pub_dartlang_org.docker:bot.pkg_pana
    | X-Pub-Reason: direct
    | user-agent: Dart pub 2.0.0
IO  : HTTP response 200 OK for GET https://pub.dartlang.org/api/packages/uuid
    | took 0:00:00.112112
    | transfer-encoding: chunked
    | date: Tue, 04 Sep 2018 08:09:53 GMT
    | content-encoding: gzip
    | vary: Accept-Encoding
    | via: 1.1 google
    | content-type: application/json
    | x-frame-options: SAMEORIGIN
    | x-xss-protection: 1; mode=block
    | x-content-type-options: nosniff
    | server: dart:io with Shelf
ERR : The current Dart SDK version is 2.0.0.
    |
    | Because susi depends on uuid >=0.0.7 <1.0.3 which requires SDK version <2.0.0, version solving failed.

That is coming from here:
https://github.com/dart-lang/pub-dartlang-dart/blob/master/app/lib/dartdoc/dartdoc_runner.dart#L175
I wouldn't remove it from there, but similar checks are needed to those we did in pana.


Deprecate traditional route generation in favor of service-consolidated routes

https://github.com/Kong/kubernetes-ingress-controller/pull/2507 introduced a feature flag that changes the way the controller generates routes. The current default generates a route per path rule in an Ingress to easily deal with multiple rules in an Ingress potentially having different Services and thus requiring separate Kong routes. The new flag mode consolidates all rules within an Ingress that share a Service and creates a single Kong route for them. This can result in a significant reduction in configuration size and DB-less configuration load time.
The new method does not functionally impact routing: requests that went to a given Service using the original method still go to the same Service in the new method. It is, however, a soft breaking change, as it affects some non-functional metadata that some environments may scrape for unknown purposes: the number of routes changes and the new routes use a completely different naming scheme, so any third-party tooling expecting consistent route naming will be interrupted by the new system. We expect that the number of affected environments is likely low, however, and would like to roll out the new system as the new default to provide its performance benefits to a wider audience. We wish to initially flip the default while leave the old method available, but deprecated, initially, and later remove it entirely if there are no significant issues or requests to keep the old method around.
The new method currently only covers Ingress v1 translation, not v1beta1 or HTTPRoute translation. It will need to be expanded to the others. We may optionally omit support for v1beta1 because it will soon no longer be supported by any supported Kubernetes version (I'm not sure exactly what our 3.0 plans are as far as version support or removal of v1beta1, but I suspect it's up for consideration at least).

[ ] Combined routes functionality is available for both HTTPRoute and Ingress v1. It is optionally available for Ingress v1beta1.
[ ] The combinedRoutesFeature gate is enabled by default, and the previously gated functionality becomes the default functionality.
[ ] There is a new follow-up issue to remove support for the original method entirely in the 3.0 milestone.
[ ] Documentation announces that the original method is deprecated and links to the removal issue for any negatively affected users to offer comment.
[ ] The changelog lists this as a breaking change and describes the effects (no functional change but new route names and IDs with multiple paths versus a single path per route).

We have this in v2.7.0, but the reason this became a gated feature previously had to do with not wanting any surprises on upgrade (e.g. routes all change right during the upgrade). Should this be in v3.0.0, or what's the thinking behind trying now to do it in v2.7.0?

I'd like to remove the old code path entirely (I don't think there's any compelling reason to keep it around), and 3.0 would be a good time to do so, assuming we're using it as the actual breaking change release. There's going to be some upgrade where all the routes change if it becomes the default or only option, so I'm inclined to have at least one version where that happens but you can flip back to the old version and come appeal to us to not remove it entirely.

@rainest WOuld you mind if I take a look ot this feature, starting with  Combined routes functionality for HTTPRoute"

Sure, go ahead.

I am not yet sure how the service based consolidation for the HTTPRoute  should look.
For the Ingress, the IngressRules are aggregated under the same service if they share the key defined as a tuple:
(
  ingress.Namespace,
  ingress.Name,
  ingressRule[i].HTTP.Paths[j].Backend.Service.Name,
  ingressRule[i].HTTP.Paths[j].Backend.Service.Port.Number
)

In case of HTTPRoute it more complex, and a rule like above is not enough to aggregate. Without the service based consolidation the parser creates separate service for the key:
(
  httpRoute.Namespace,
  httpRoute.Name,
  httpRouteRuleNumber
)

In order to enable the service based consolidation for HTTPRoute there is a need to determine a way to identify the services that can aggregate the routes. Given the fact that HTTPRouteRule may reference multiple weighted backends, it seems reasonable to aggregate the routes only if they point to the same set of weighted backeds.

It is indeed more complicated. With the HTTPRoute translator as-is, I think the answer here is that multi-service backends just simply aren't eligible for such consolidation. We could potentially change that, but there's not a clear answer as to how. We may want to check with the other Gateway API implementations to see if they have good prior art, though I think a lot of them have proxies that just natively support multiple services per route and it's a non-issue as such.
The change to support multi-Service backends uses the rule number-named service mentioned above via generateKongServiceFromBackendRef(), invoked from the HTTPRoute translator. There's further processing to generate the Kong service, but as the name is already decided, so that doesn't matter as much.
The host+service+port route name scheme we use for Ingress doesn't transfer well to multi-service backends, since those can have (theoretically) infinite service+port combinations, and would be unwieldy to name.
I can't think of anything that works well, save our old bad standby option, "hash some unique identifying inputs and stuff the hash in the name". Maybe a bit less unwieldy is appending each distinct combination of N service+port+weight combos to a list and referring to them by index, so for
apiVersion: gateway.networking.k8s.io/v1beta1
kind: HTTPRoute
metadata:
  name: simple-split
spec:
  rules:
  - backendRefs:
    - name: foo-v1
      port: 8090
      weight: 90
    - name: foo-v2
      port: 8080
      weight: 10
  - backendRefs:
    - name: foo-v1
      port: 8090
      weight: 90
    - name: foo-v2
      port: 8080
      weight: 10
  - backendRefs:
    - name: foo-v1
      port: 8080
      weight: 90
    - name: foo-v3
      port: 8080
      weight: 10

You get two routes, namespace.name.multi-0.xx and namespace.name.multi-1.xx, where the former are the two rules that use v1/v2 and the latter is the rule that uses v1/v3. That's at least no less unintelligible than hashes, and does provide a means (though it is indirect--you have to count the combos yourself) of determining what they're referencing.
I don't think we can get a good indication of the underlying services with gateway labels (FTI-4368), which still could be large (hopefully not in practice), but at least isn't stuffed into the route name with all other identifying info.

Yeah, creating a key from a backend refs works.
Given the above example, the metadata would be
("<G>.<K>.<NS>.foo-v1.8090.90;<G>.<K>.<NS>.foo-v2.8080.10;", ["rule1", "rule2"], ["foo-v1", "foo-v2"])
("<G>.<K>.<NS>.foo-v1.8090.90;<G>.<K>.<NS>.foo-v3.8080.10;" ["rule3"], ["foo-v1", "foo-v2"])

The result would be
for backend refs key "<G>.<K>.<NS>.foo-v1.8090.90;<G>.<K>.<NS>.foo-v2.8080.10;"
create service with ["rule1", "rule2"] and backends ["foo-v1", "foo-v2"],

for backend refs key "<G>.<K>.<NS>.foo-v1.8090.90;<G>.<K>.<NS>.foo-v3.8080.10;"
create service with ["rule3"] and backends ["foo-v1", "foo-v2"]

I can't think of anything better right now.

In https://github.com/Kong/kubernetes-ingress-controller/pull/3008#pullrequestreview-1141603475 and https://github.com/Kong/kubernetes-ingress-controller/pull/3008#issuecomment-1278638285 we've found that HTTPRoute rules are much more difficult to consolidate than Ingress rules, as they allow for more complex matching (headers, methods) and transformations (filters, which will generally apply a standard plugin configuration to the route) per rule. Ingress only supported these via Ingress-wide annotations.
As such, #3008 consolidates Service generation for HTTPRoutes (the original implementation created them per rule to handle multi-Service backends easily), but we'll need to handle route consolidation in a new PR.


bigquery: use Pool to create a pool of clients

Hi @shannonlal, this PR updates your branch google-big-query.
The main commit you want to check is https://github.com/shannonlal/falcon-sql-client/commit/fbd5fe8aa49d471099d73cd7fb802740de1907b6 . All the others are the result of bringing the branch in sync with Falcon v3.0.3.
In https://github.com/shannonlal/falcon-sql-client/commit/fbd5fe8aa49d471099d73cd7fb802740de1907b6 you'll find:

I've introduced the use of Pool to avoid creating a new client for every request.
I've also re-factored some of the code to answer your questions (or to get rid of unnecessary promises).

You'll find I've left 2 TODOs:

in connect, we need to validate the connection (to do this, we want a request as cheap as possible; retrieving all the tables may be too slow for large numbers).
in schemas, we have a similar problem: making a query per table may not scale well; specially if the all the queries are concurrent. I'd investigate the use SQL to do this.

I haven't tested this PR; you may find mistakes!

@n-riesco Thanks for the information.  I will start to look at Pool.  For the connect I should be able to find a light-weight connect call.  For the tables/schema I will have to dig into the APIs for Google Big Query to see what they have


Annotate CpuFuture with #[must_use].

When dropped CpuFuture will try to cancel the computation,
thus not using it is most likely an error.

Thanks!


spin deploy error missing base

From spin @ main after adding default for base from #1803 :
➜  http-rust git:(main) ✗ spin deploy
Warning: You're using a pre-release version of Spin (1.6.0-pre0). This plugin might not be compatible (supported: >=1.3). Continuing anyway.
Error: Cannot read spin.toml manifest from "spin.toml"

Caused by:
    could not load application trigger parameters: missing field `base` at line 14 column 1

Learn more at https://developer.fermyon.com/cloud/faq

cc/ @lann

cloud-plugin will need to upgrade its spin dependencies to pick up the field being optional.


Move UCP, DTR to /datacenter//

Introduce a way for us to start supporting docs for multiple product versions

You'll want to change aliases: to redirect_from: :(

Deploy preview processing.
Built with commit e78f9aaa895b946d2d859120adf1366be1e8b685
https://app.netlify.com/sites/docker-docs/deploys/581ea7f7c4d9cc4def51bc0b


What human reference were the files aligned to?

Hi what human reference was used for the alignment of the files?

I just saw it is b37 hence hg19


DerbyDatabase.getDefaultDriver() returns liquibase.repackaged.org.apache.derby.jdbc.EmbeddedDriver

Environment
openjdk version "11.0.13" 2021-10-19 LTS
OpenJDK Runtime Environment Zulu11.52+13-CA (build 11.0.13+8-LTS)
OpenJDK 64-Bit Server VM Zulu11.52+13-CA (build 11.0.13+8-LTS, mixed mode)
Apache Maven 3.6.3 (cecedd343002696d0abb50b32b541b8a6ba2883f)
Liquibase Version:  4.6.x
Liquibase Integration & Version: maven
Liquibase Extension(s) & Version:
Database Vendor & Version:  Derby <IP_ADDRESS>
Operating System Type & Version:  MacOS Monterey
Description
DerbyDatabase.close() is throwing a java.lang.ClassNotFoundException: liquibase.repackaged.org.apache.derby.jdbc.EmbeddedDriver.
This is because DerbyDatabase.getDefaultDriver() returns the liquibase.repackaged... driver class name rather than the actual one.
It seems that the source code and the binary do not correspond to each other for 4.6.x.
Steps To Reproduce

git clone https://bitbucket.org/alexlitovsky1/liquibase-derby-bug.git
mvn clean test

Actual Behavior
Test fails:  expected: <org.apache.derby.jdbc.EmbeddedDriver> but was: <liquibase.repackaged.org.apache.derby.jdbc.EmbeddedDriver>
Expected/Desired Behavior
The test should pass.

After decompiling DerbyDatabase from 4.6.2, this is the code:
    public String getDefaultDriver(final String url) {
        if (url.startsWith("jdbc:derby://")) {
            return "liquibase.repackaged.org.apache.derby.jdbc.ClientDriver";
        }
        if (url.startsWith("jdbc:derby") || url.startsWith("java:derby")) {
            return "liquibase.repackaged.org.apache.derby.jdbc.EmbeddedDriver";
        }
        return null;
    }

It's different than the source code from that tag:
https://github.com/liquibase/liquibase/blob/v4.6.2/liquibase-core/src/main/java/liquibase/database/core/DerbyDatabase.java#L42

This bug just sat there for over a month.  I closed and opened a new one.  Somehow it fell through the cracks.


[Excel] (Data types) Add property metadata to Entity card article

Internal work item.

Docs Build status updates of commit 8237055:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/excel-data-types-entity-card.md
:white_check_mark:Succeeded
View

docs/images/excel-data-types-entity-card-property-metadata.png
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 273aa7d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/excel-data-types-entity-card.md
:white_check_mark:Succeeded
View

docs/images/excel-data-types-entity-card-property-metadata.png
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 5c56baa:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/excel-data-types-entity-card.md
:white_check_mark:Succeeded
View

docs/images/excel-data-types-entity-card-property-metadata.png
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit a0bbbef:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/excel-data-types-entity-card.md
:white_check_mark:Succeeded
View

docs/images/excel-data-types-entity-card-property-metadata.png
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 7d9511f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/excel-data-types-entity-card.md
:white_check_mark:Succeeded
View

docs/images/excel-data-types-entity-card-property-metadata.png
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add prettier and run prettier command via lint-staged

Type of Change

Build Scripts: Add prettier with husky for code formatting
Tool Source: JS and Vue
Something else: Added prettier which runs with lint-staged.

What issue does this relate to?
resolves #447
What should this PR do?

Add new package prettier
Run the prettier using lint-staged when creating new git commit

What are the acceptance criteria?

prettier should run in only the files that are changed when new git commit is added.

here are the changes that are made by prettier. https://github.com/maskeynihal/nginxconfig.io/pull/1
should I also push the changes? @MattIPv4?
the test failing should fixed by the changes.

I think you'll probably want to add a dedicated test step for prettier -- eslint is definitely not flagging all the issues.
And yeah, a commit with all the changes prettier wants would be good as well.

@MattIPv4 All the checks are passing.

I think you'll probably want to add a dedicated test step for prettier

I didn't get what you meant by this.

You'll want to add a new GitHub Actions test check for prettier -- eslint did fail, but it definitely did not fail with everything that prettier wanted to change.

@MattIPv4  I have added github action to check code by prettier.
Thanks for taking the time to look into the PR.

@MattIPv4 Would you mind adding the "hacktoberfest" tag to this PR if you can? It would be fantastic for keeping track of contributions during Hacktoberfest, but don't worry if you can't. I'm thrilled to contribute either way!

@MattIPv4 Gotcha. Thanks. Previously it was showing Not-Accepted in my profile but now it's working fine.
Thanks once again.


Specify path to Python in settings

Hi,
On osx 10.10, I'm trying to use python3.4 with atom (macports install in /opt/local/bin/python3.4) by default, but i cant find a proper way than :

cmd+maj+I : command = /opt/local/bin/python3.4, but i need to do that at each launch of atom
OR
set shebang to "#!/usr/bin/env python3.4", modify "python" path to > python3.4 in zsh and edit launchctl to pass vars.
(https://github.com/rgbkrk/atom-script/issues/61#issuecomment-37337827)
OR
set shebang to absolute path of my python3.4 install which make my script less portable.

Can you set an option to specify path to exec interpreter saved once for all ?
Have a nice day.

Is python3.4 in your PATH? Have tried creating a symlink to it?

Same problem here: several version of python installed; script chooses the wrong one :(

Same here, would be great to be able to save the configuration once and for all

i tried that also didnt work

Did you try to use new "profile" feature? You can see how it works here, in top post https://github.com/rgbkrk/atom-script/pull/750

Did you try to use new "profile" feature? You can see how it works here, in top post #750

I was wondering whether it is possible to find some examples how the profile feature should be configured. For example, when working with two Python versions available in:
C:\Python27
C:\Python35-32

How the profile window below should be configured to provide a convenient switching mechanisms for running scripts in each version.

Did you manage to do it ?

@ABUISSON No, I actually came back to vim.

In my Ubuntu I manage it by insert in "command" the path to my executable python file, e.g.  /home/user/anaconda3/envs/anaconda2/bin/python2.7. Then, I have saved it as profile named 'python2'. Now when I press shift-ctrl-alt-b and hit the 'run' button I run my code in the active table using python2.7. Using ctrl-alt-b I run my code using my default python version, i.e 3.6.


Make display-balance

This display should replace current balance, and it should contain the formatting logic

The possible values are in redux - just need to pick the one you want.

these redux values are automagically updated when either the balance changes - or the FX rate changes.

thx @sponnet !


[v2] There is no top level schema object defined

If I understood correctly, the plugin is looking for schema directive which is not generated by my server as defaults are used. The docs says the schema directive is optional so I think that the plugin should try to find Query, Mutation and Subscription types by default if not returned by introspection.

If you don't have a schema directive and have a Query type, this error indicates that the file your has not discovered the Query type definition.
If you're using the single schema option in GraphQL settings, then this is a bug, and I'd like a minimal    project that reproduces.
If you using scopes or graphql-config, please review your settings and ensure this file is associatd with the schema file that contains the query type. On that note, if this is the case, you should see a yellow warning bar on top of the editor. Can you confirm that you see one?

I'm using the graphqlconfig option
This is the content:
{
  "extensions": {
    "endpoints": {
      "Default GraphQL Endpoint": {
        "url": "http://localhost/graphql/",
        "headers": {
          "user-agent": "JS GraphQL"
        }
      }
    }
  }
}

The endpoint http://localhost/graphql/ is working well, I thought that the plugin would do introspection throught this endpoint
When I press the arrow

it says

GraphQL Configuration Error: No schemaPath found for url http://localhost/graphql/

The workflow is somewhat different in v2 given that it relies on graphql-config.
I'll see if I can find some time to write up some minimal docs in the near future.
So the workflow with graphql-config is that you configure a schemaPath in your .graphqlconfig. When you run the introspection query by clicking on the arrow, the result is written to the file defined by the schemaPath, e.g.:
{
  "shemaPath": "./schema.graphql",
  "extensions": {
    "endpoints": {
      "Default GraphQL Endpoint": {
        "url": "http://localhost/graphql/",
        "headers": {
          "user-agent": "JS GraphQL"
        }
      }
    }
  }
}

With that it works, thanks

Ok. I'll keep this issue open, as the plugin could provide better error messages, and help with the creation of a schemaPath in case it's missing.

Looks like my Rails graphql engine is receiving these parameters when the plugin is trying to query the schema:
{"query"=>"http://localhost:3000/graphql/private", "graphql"=>{"query"=>"http://localhost:3000/graphql/private"}}

Looks like a valid query is not being passed.

Ok, nevermind.  I had some bogus configuration in the GraphQL settings panel.

Closing this as the plugin now has a "Schemas and Project Structure" UI that provides a better schema discovery status. I've also defaulted new .graphqlconfig files to have a default schemaPath.

I've run into an issue where this error pops up if the schema you're referred to is inside a folder that's been excluded from the IDE. For whatever reason the plugin can't find it or parse it properly. Would be nice if it could explain that's what's going on


Change ADMIN_USERS env value in contributionsBB release

Description
Change ADMIN_USERS env value to "sandeep-ps,minump,bingzhang" for the next release of contributionsBB.
Acceptance Criteria
contributionsBB env variable ADMIN_USERS has value "sandeep-ps,minump,bingzhang" in TEST and PROD.

Env variable changed in 1.18.0 release.

Issue fixed in 1.18.0 release. Release master branch merged to develop.
Closing issue


Search via Nominatim

Fixes #25
Needs CSS fixes.
Mobile:

Desktop:

Hi,
Why this PR is not merged yet? This enhancement is still missing on openaedmap.

It's unfinished. Styling (CSS) tweaking is required. I just have never returned to it. I will work on it next year, hopefully soon 😄


Checks if it is alphanum

I don't see anyone typing in not-alphanumeric, even if they do it would probably be like 1 in a thousand times if not zero. This also has one more problem that it does not check for underscores in names. I feel like this is not very needed and would slow down the process than it already is. @mew @PyICoder

I don't see anyone typing in not-alphanumeric, even if they do it would probably be like 1 in a thousand times if not zero. This also has one more problem that it does not check for underscores in names. I feel like this is not very needed and would slow down the process than it already is. @mew @PyICoder

I don't really think its a huge issue. I imagine well over 90% of the time getting a player's stats is waiting for a response from the Hypixel API. The amount of time it takes to check if the name is alphanumeric is less than a tenth of a second. There is no way the average user notices this time loss.
If we really cared about performance, we wouldnt be using Gson.

I don't really think its a huge issue. I imagine well over 90% of the time getting a player's stats is waiting for a response from the Hypixel API. The amount of time it takes to check if the name is alphanumeric is less than a tenth of a second. There is no way the average user notices this time loss.
If we really cared about performance, we wouldnt be using Gson.

Yeah i guess so, but still realistically speaking i dont think anyone would type in non-alphanumeric names anyway but go ahead if you think its useful enough(also add a support for underscores in names). 😅
P.S. I added Gson because I thought it'd be more future proof if we decide to add more configs in the future, also it just takes the key once out during startup so i dont think it affects performance much anyway.

I understand thanks.


FastLED

Was there something special to do to get the FastLED library working?
I am trying just a simple blink app and while it compiles and loads, no pin seems to be outputting the data signal. Was there a special version or something you used?

User Error! SRY.


Set VERSION to 1.14.0-dev

1.14.0-dev is more accurate version than 1.13.90 for the development branch for the next Cilium minor release, which is 1.14.0.

i have a lingering suspicion this will break some release scripts somewhere. requesting review from andre and joe.

In principle I'm OK with this, I just don't know what breaks or how to detect that (I guess one way is just try and see..). Anything using a proper semver-v2 library will properly match this as <1.14.0, the long tail is random bash scripts with more naive version checks.
For release scripts specifically it's probably not too much of a problem, because we change the version for all releases--both snapshots or regular releases-- so I wouldn't expect this version number to show up in a release at all.

Helm chart failure is caused by the PR, helm charts include this version string in them.

/test

/test


Exporting to .PST gives guzzle errors

I'm trying to restore files from a restore Exchange session, however when I hit the ".PST" option I get the following error:
PHP Warning:  fseek(): supplied resource is not a valid stream resource in C:\inetpub\wwwroot\vendor\guzzlehttp\psr7\src\Stream.php on line 206
PHP Warning:  stream_get_contents(): supplied resource is not a valid stream resource in C:\inetpub\wwwroot\vendor\guzzlehttp\psr7\src\Stream.php on line 95
PHP Notice:  Trying to access array offset on value of type null in C:\inetpub\wwwroot\veeam.class.php on line 1501

Have you run into this issue before? The portal works fine with restoring singular .MSG files.

Can u verify outlook is installed on the VBO server (it is required for exporting to PST)?

That's an quick answer, I can verify that Outlook is installed on the VBO server itself.
Is it running Outlook in the background as a back-end call?
Then I might need to check if the regular restore procedure works, if not I'll go ahead and fix that first.

We figured out there is an issue ongoing with the latest Oktober updates and Veeam Explorer.
Currently we raised a ticket at Veeam as we have the latest updates, which breaks the Explorer.
Workaround at the moment is:

Reinstall office, and prevent any updates. Which is for most business a no-go, including us.

Thanks for reporting this. I was not aware of this and will do the test as well.


Added portuguese vocabulary

This fork contains added vocabulary for the portuguese language and corresponding new english expressions.

Please, see https://dateparser.readthedocs.io/en/latest/contributing.html#guidelines-for-editing-translation-data


Delete wsms.txt

Fictitious page.

@aaapl Yes, the website is a doorway.
Thank you for the update! Pull request merged.


Contact page added #198

Related Issue or bug
This pull request is associated to issue number #198
Fixes: #198
Describe the changes you've made
Added a complete contact page where users can send any message through e-mail and the admin would receive the message from the user.
( On running the following command, the message sent could be viewed:
python -m smtpd -n -c DebuggingServer localhost:587 )
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Code style update (formatting, local variables)

Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
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.

Screenshots

@swapnilsparsh  @sakshikhachane Please review.

@swapnilsparsh I have made the suggested changes. Please review the PR. Thank you!

@swapnilsparsh Please review the PR.

@swapnilsparsh What type of hover should I apply then?

@swapnilsparsh What type of hover should I apply then?

On hover green colour should be there as it is there on login page.

Hey @somya51p,
Any updates on the PR?

@somya51p Resolve the conflicts

Hey @somya51p,
Can you please give us updates? We can't keep PR pending for more than 15 days unless we receive the updates or changes are being done.

@sakshikhachane @swapnilsparsh I will be resolving the conflicts in the next two days. I was having quizzes in colledge due to which I was unable to give my contribution in a particular span.

@sakshikhachane Changed the hover to green and resolved conflicts. Kindly review

@swapnilsparsh Resolved. Have a look

Hey @somya51p,
Sorry for the delayed response. We were trying to review this PR for a long time. But we are getting errors while fetching the PR. Can you please look into this?

@sakshikhachane You need to run this command in a parallel terminal as I have given an smtpd response to it.
python -m smtpd -n -c DebuggingServer localhost:587
I have initially attached the screenshots it will work in the same manner.

@swapnilsparsh Can you have a look into this?

@swapnilsparsh Done


Add a Dashboard Link to CloudWatch

It is very useful to add a CloudWatch Dashboard link to the task. User may need to view the details, such as bandwidth, lambda invocation, task running time and etc.

resolved


large data set

Hello,
Thanks a lot for your work with kriging.js
Hear is the problem.
I have a data set with
118632 register

whe I create a model
var model = "spherical";
var sigma2 = 0, alpha = 100;
var variogram = kriging.train(array_z, array_x, array_y, model, sigma2, alpha);
I get error "RangeError: invalid array length @ file:///E:/kringingjs/javascript/kriging.js-master/kriging.js:233"

What can I do to make kriging with this dataset?
maybe select few data. convert 118632 samples in 500 samples?

I started exploring this module recently, I am curious how did you solve it?
Instead of random 500 samples you can take every nth in x direction and nth in y direction depending on data density you want.

I have the same issue. I have found the limit to be 501 data points.
But I have a different error message in the console:
kriging.js:15
Uncaught RangeError: Maximum call stack size exceeded
at Array.rep (kriging.js:15)
at kriging_matrix_diag (kriging.js:34)
at Object.kriging.train (kriging.js:333)

This error is due to recursive function call. Check here at stackoverflow with nice explanation.
But this in not solution, further @oeo4b can throw some light on this.


Make all Razor Nexus DDRIT tests be non-experimental

Continuation of https://github.com/dotnet/aspnetcore/issues/23964. Now that we have SOME tests Required we should drive our remaining tests to a reliability level where they can be required as well.

[ ] RazorDiagnosticsTest
[ ]  RazorCodeActionTests
[ ] RazorHTMLIntellisenseTest
[ ] RazorCSharpIntellisenseTest

@ryanbrandenburg @NTaylorMullen Before being transferred this was marked as "Working", are we able to close out this issue?

I definitely think we can retire and/or transfer the Nexus DDRIT variant of our tests to be all local


Release Kalix JavaScript SDK 1.0.7

Prepare

[x] Make sure all important PRs have been merged
[x] Check that the latest build successfully finished
[x] Make sure a version of the proxy that supports the protocol version the SDK expects has been deployed to production

You can see the proxy version on prod on grafana or using various other methods.
Cutting the release

[x] Edit the draft release notes and publish with the appropriate tag.

CircleCI will automatically publish the @kalix-io/kalix-javascript-sdk package to the npm registry based on the tag
CI will update the docs/kalix-current branch

Check availability

[ ] Check that docs/kalix-current has been updated
[x] Check the release in the npm registry

Fix and publish docs

[ ] If relevant, update the supported version in the main docs (affects Supported Versions)
[ ] Add an item to the Release Notes in the documentation
[ ] Release the Kalix documentation to get the SDK docs updates published. After running the commands in the previous link the release is automated.

Update to the latest version

[ ] Run samples/bin/update.sh to update the samples to the latest released versions.

Check docs update

[ ] After quite a while TechHub shows updated docs (see the timestamp on the bottom of the page)

Announcements

[ ] Announce in the forum

tag with Kalix, releases and announcement

[ ] Inform Lightbend marketing and engineering if it's a noteworthy release
[ ] Close this issue

Docs failing silently to get published: https://github.com/lightbend/kalix-javascript-sdk/issues/473

Superseded by #494


Add a Scala example

e.g. https://github.com/mattmoor/bazel-glibc/blob/master/java/examples/BUILD#L25

I think this is fixed now, right?


Add CPP pragmas for Plutus version

There may be nicer ways to achieve this (maybe by conditionally re-exporting modules?), but something to this effect will support more dependency configurations using the plutus-scaffold base.

I was going to add this as soon as Plutarch gets vasil support actually. But I should probably make a staging branch right now with plutarch staging and update to the newer api.
Will get to it this week.

In that case I'll mention that we're already using Ply with Plutarch staging and I've made a few tweaks to enable that:
https://github.com/danielfarrelly/ply/tree/df/plutarch-staging

Updated in the staging branch. Note that it also uses a much newer Plutarch version (with a huge amount of breaking changes).

We found that just replacing the old namespace with the new was insufficient since our offchain dependencies still relied on the old one. In our case we were able to remove plutus-apps from plutus-simple-model, so it was solvable in the end, but for supporting existing dependencies of plutip/bot-plutus-interface you might want to allow both namespaces somehow. Either that or ensure that master and staging branches of ply-core remain cross-compatible so that a project can include both without issue.

yeah the plan is to keep both master and staging intact until plutus-apps gets their crap together and updates.


CASMNET-1389 test case/remove folder

Summary and Scope
Description: add mellanox dns test case. remove unused folder
PR checklist (you may replace this section):

[ ] I have run nox locally and all tests, linting, and code coverage pass.
[ ] I have added new tests to cover the new code
[ ] My code follows the style guidelines of this project
[ ] If adding a new file, I have updated pyinstaller.py
[ ] I have updated the appropriate Changelog entries in readme.md
[ ] I have incremented the version in the readme.md
[ ] I have incremented the version in canu/.version

Issues and Related PRs

Resolves CASMNET-1389
Relates to Issue
Requires Issue

Testing
Tested on: nox/redbull

List of virtual system tested.
List of physical systems tested.

@kosonenj Please remove the docs/network\ install\ and\ upgrade folder (the one with spaces) in this branch and make sure it's in the directory w/ underscores.
https://github.com/Cray-HPE/canu/tree/CASMNET-1389


Transcripts are all one person

E.g. This one shows only Scott: https://syntax.fm/show/727/how-to-code-opinionated-typescript-stack-tooling-choices-explained/transcript
This one shows only Wes: https://syntax.fm/show/726/is-htmx-a-joke/transcript
In case it’s just me, I’m on iPadOS 17.2 (11-inch 3rd gen)

Thanks - duplicate of https://github.com/syntaxfm/website/issues/1193
something is going  on with the regex - will dig in..


Fixed unexpected behaviour of returning an index less than the length…

Due to slice ordering, when using sort.SearchStrings the return result is based on the ordering of the existing list.
So the index returned could be less than len(list), depending on existing entries.
Replaced the check with a simple nested for loop to check for exact value instead of using sort.SearchStrings

@mklnz please remove the change the module name and I'll merge the PR

Fixed sorry

@mklnz Thanks for the bug fix :grin:


[SMALLFIX] Supply the variable name to Preconditions.checkNotNull in …

…MasterWorkerInfo.java

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/16914/
Test PASSed.


Commands for Components

This page should also contain the commands of C++, C# and other optional installation features of the Visual Studio 2017.

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

ID: a6f80d9f-168b-e3cb-6499-e03a44206826
Version Independent ID: 4b235b0d-1436-0a36-ef55-f76ba8d95bd1
Content: Visual Studio 2017 workload and component IDs - Visual Studio
Content Source: docs/install/workload-and-component-ids.md
Product: visual-studio-dev15
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@its4zahoor Thanks for your feedback. Might I inquire, what is it exactly that you seek? Is it workload and component information for C++, such as is listed from the links for each release (i.e., were you to click Visual Studio Community 2017, you'd see components for C++) -- or is it the command-line commands you'd use to install Visual Studio, such as is presented on the Use command-line parameters to install Visual Studio 2017 page?

@its4zahoor Feel free to comment again if you have further information for us, but we'll close this for now.


adding SwarmCI

Very cool submission @ghostsquad I look forward to testing this out when you get to Beta.


requested feature (switcher mode) show  tiling for all open windows

can cortile has feature that show all open windows(active and not active) as switcher app.
i know he do tiling for the raise window, what if i want to have(tiling show) which show the minimize windows too as switcher mode like skippy-xd and xfcedashboard

can cortile has feature that show all open windows(active and not active) as switcher app.

I do not fully understand the question, most switchers can be configured to also show inactive (minimized) windows.

...i want to have(tiling show) which show the minimize windows too as switcher mode like skippy-xd and xfcedashboard

Does anything speak against using the build in window switcher, or do you mean in cases where no window switcher is integrated in the system?

there is no conflict bw cortile and the switchers apps, it's good.
i mean i want not use them during using cortile, so i disabled them when cortile running. and i want to use cortile to show all opened apps: so i used that command to show all opened windows during cortile:
wmctrl -l | awk '{print $1}' | xargs -I {} wmctrl -ia {}
it's good, but  i wish if there option in cortile that automatically detects the maximum slaves and masters to show all opened windows.

it's good, but cortile doesn't show all opend windows,because i think the default of it is 3 slaves and one master, so i can change that default to show all opened windows?

Since v.2.4.0 the last number of maximum windows is preserved, so you could theoretically set (e.g.),
# Maximum number of allowed master windows (0 - 5).
window_masters_max = 5

# Maximum number of allowed slave windows (1 - 5).
window_slaves_max = 5

then use socket communication to increase masters/slaves before calling wmctrl -l | awk '{print $1}' | xargs -I {} wmctrl -ia {}. Nevertheless, there is a limit to the number of windows displayed, and as the number get larger the arrangement will become quite lumpy.
In general, minimized windows are not known (tracked) by cortile at all. All other open windows are tracked, but may be invisible, because they are stacked behind other slaves. Raising them to front can be done by using:

build in taskbars/panels that show opened windows
build in or external window switcher applications
cortile next/previous window shortcuts:

Keys
Description

Ctrl+Shift+KP_2
Focus next window

Ctrl+Shift+KP_8
Focus previous window

use socket communication to increase masters/slaves

sorry, what is the command(script) in cortile  to do that?

# Maximum number of allowed master windows (0 - 5).
window_masters_max = 5

# Maximum number of allowed slave windows (1 - 5).
window_slaves_max = 5

can set that with script?

Those values are the maximum values (upper bound)
# Maximum number of allowed master windows (0 - 5).
window_masters_max = 5

# Maximum number of allowed slave windows (1 - 5).
window_slaves_max = 5

in case you press

Keys
Description

Ctrl+Shift+KP_Add
Increase number of master windows

Ctrl+Shift+Plus
Increase number of maximum slave windows

several times.
You can set that to any value you want (in this case maximal 10 windows will be visible, no matter what). The configuration is applied on app startup and you can`t change it afterwards.
When cortile is running you can still change the current max window size using the shortcuts above. By using socket communication this would be,
echo '{"Action":"master_increase"}' | nc -U /tmp/cortile.sock.in
echo '{"Action":"slave_increase"}' | nc -U /tmp/cortile.sock.in

but I'm still not sure if this is what you need.

The socket communication will not work any more, please review cortile-addons and #65.


Error downloading template for submission 6352aee215ccca0001f5f44f

This publication (36066633) was manually reopened for a sumstats-only submission. Now when I try to download the submission template, I get an error:
There is a fault on our end. Please contact<EMAIL_ADDRESS>for help.
Submission page: https://www.ebi.ac.uk/gwas/deposition/submission/6352aee215ccca0001f5f44f

Hi @Elliot for the submission the sumstats related data was not populated for the Pmid '36066633' in Deposition DB , this usually happens by Depo Sync process , not sure why it was not triggered for this Pmid , Have manually added the details from Backend . You will need to create a fresh sumstats only submission which will create the prefilled template for sumstats

@sajo-ebi thanks for doing that, but the publication is currently closed (because it already has some sumstats) - can you please reopen it again, so I can try a new submission?

@eks-ebi the publication is open for sumstats submission now, as discussed the Depo Sync resets it every night when it runs so we need to create the submission today only


Setup Development and Production Environment for Workflow

Currently the workflow has the URL for the Github Domain fixed to the deployment domain as follows:
export const GITHUB_APP_DOMAIN = "https://open-search-changeset-bot.vercel.app";

For development, each contributor has to change that URL with a tunnel service (for instance, ngrok) to be able to contribute in the GitHub App.
A solution to this problem is to setup this constant as an environmental variable in the .env file for development and in the workflow file for the working branch of OpenSearch-Dashboard for the changelog project pass the deployment URL as a parameter.
The setup for the GITHUB_APP_DOMAIN should be finally as follows:
export const GITHUB_APP_DOMAIN = process.env.ENV
  ? process.env.INPUT_GITHUB_APP_DOMAIN
  : process.env.GITHUB_APP_DOMAIN;

With the workflow file parse_changelog_workflow.yml passing an input as follows:
# Other code...
with:
     token: ${{secrets.GITHUB_TOKEN}}
     github_app_domain: 'https://open-search-changeset-bot.vercel.app'
     changeset_path: changelogs/fragments

And in the .env variable for the Reusable Workflow for OpenSearch_Parse_Changelog_Action, adding the tunnel domain for working in dev mode with the Github App
GITHUB_APP_DOMAIN='<your-tunnel-domain>'

This finally does not work as expected. Closing issue


Do we need to fetch the complete git history?

Describe the bug
Checking out existing branch fails in my CI workflow.
To Reproduce
My workflow file: https://github.com/chances/descartes-d/actions/runs/477666176/workflow
Failing CI build: https://github.com/chances/descartes-d/pull/2/checks?check_run_id=1681534009
Expected behavior
Your license year task is skipped because the license/2021 branch already exists with license updates.
Screenshots
N/A
Additional context
Failing CI build: https://github.com/chances/descartes-d/pull/2/checks?check_run_id=1681534009

Hi @chances!
I think replacing
- uses: actions/checkout@v2

with
- uses: actions/checkout@v2
  with:
    fetch-depth: 0

should fix your problem. Can you try it out and report back?

Hi @chances!
I think replacing
- uses: actions/checkout@v2

with
- uses: actions/checkout@v2
  with:
    fetch-depth: 0

should fix your problem. Can you try it out and report back?

@FantasticFiasco Hm, that seems to resolve it.
Why is fetch-depth: 0 required?

@FantasticFiasco Hm, that seems to resolve it.
Why is fetch-depth: 0 required?

It seems that actions/checkout@v2, due to an optimization, only gets the lates commit by default. That operation will not get the tags, nor the branches of the repository. I can understand their decision, because most often in CI you only get the files to run the tests or what not, but we need more of the repo, and its history, thus the necessity to set fetch-depth: 0.
You can read more about their reasoning here.
Please consider giving this repo a star if you find it beneficial. Currently stars are the only way to indicate usage for GitHub Actions. Thanks!

It seems that actions/checkout@v2, due to an optimization, only gets the lates commit by default. That operation will not get the tags, nor the branches of the repository. I can understand their decision, because most often in CI you only get the files to run the tests or what not, but we need more of the repo, and its history, thus the necessity to set fetch-depth: 0.
You can read more about their reasoning here.
Please consider giving this repo a star if you find it beneficial. Currently stars are the only way to indicate usage for GitHub Actions. Thanks!

Other GitHub actions that play with Git don't require this flag on the checkout action, e.g. crazy-max/ghaction-github-pages@v2.
Can your action not workaround it like these other actions?

Other GitHub actions that play with Git don't require this flag on the checkout action, e.g. crazy-max/ghaction-github-pages@v2.
Can your action not workaround it like these other actions?

crazy-max/ghaction-github-pages@v2 and this action doesn't share the same behaviour, thus you cannot compare them.
Where we in this action run git operations on the same repository checked out by the preceding actions/checkout@v2, crazy-max/ghaction-github-pages@v2 doesn't. They clone the specific branch of the target repository into a temp directory, and doesn't do any changes at all in the source repository that was checked out by their preceding actions/checkout@v2.
I'm quite sure that you could have found that out by comparing the source code in this repo with the source code in their repo. So let me turn the table. If removing fetch-depth: 0 is important to you, I challenge you to find a solution where we don't have to specify it using this action, and still remain with the same functionality.

crazy-max/ghaction-github-pages@v2 and this action doesn't share the same behaviour, thus you cannot compare them.
Where we in this action run git operations on the same repository checked out by the preceding actions/checkout@v2, crazy-max/ghaction-github-pages@v2 doesn't. They clone the specific branch of the target repository into a temp directory, and doesn't do any changes at all in the source repository that was checked out by their preceding actions/checkout@v2.
I'm quite sure that you could have found that out by comparing the source code in this repo with the source code in their repo. So let me turn the table. If removing fetch-depth: 0 is important to you, I challenge you to find a solution where we don't have to specify it using this action, and still remain with the same functionality.

I'm well aware of the actions' differing behavior. My intention in bringing up the crazy-max/ghaction-github-pages action was with regard to its implicit recognition of the fetch depth optimization.
The fetch-depth requirement your action imposes wastes CI resources. The optimization is important, especially for large repos with heavy branch churn.
Your action could, instead of expecting the entire history of the target repo, git fetch the branch refs of interest.

I'm well aware of the actions' differing behavior. My intention in bringing up the crazy-max/ghaction-github-pages action was with regard to its implicit recognition of the fetch depth optimization.
The fetch-depth requirement your action imposes wastes CI resources. The optimization is important, especially for large repos with heavy branch churn.
Your action could, instead of expecting the entire history of the target repo, git fetch the branch refs of interest.

You are correct, the change would be for a better one. One could play around with fetch depth, one could also remove the dependency towards actions/checkout@v2 altogether and clone the repo ourselves.
I'll leave the issue open. Let's see if it gets some traction.

You are correct, the change would be for a better one. One could play around with fetch depth, one could also remove the dependency towards actions/checkout@v2 altogether and clone the repo ourselves.
I'll leave the issue open. Let's see if it gets some traction.

I've come to the conclusion that depending on composability, where we depend on actions/checkout@v3, isn't a bad thing.
Closing issue.


Support limit pushdown when filter and sort can apply pushdown entirely

Description
To support limit pushdown when filter and sort can apply pushdown entirely:

Loose limit pushdown conditions in handler.
Add a boolean return value to functions prepareWhereClause and prepareOrderByClause to reuse current logic to check if some filters or sort can not be applied pushdown. Only if prepareWhereClause and prepareOrderByClause does not find any parameters that cannot be pushed down, will apply limit pushdown.
Update some integration tests and add comments.

Closes #3644.
Readiness checklist

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

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: simmonwu:x: cvbhjklsimmonwu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you for your review. Please see my recent commit and comments. And integration/compatibility tests, with one or both of flags disable-filter-pushdown and enable-sort-pushdown set, pass now.

Thank you for your careful review. Fix issue with $eq and $ne and add more test cases. Please see comments.

@cvbhjkl Sorry, we have to put it on hold for now – we need to deeply refactor our existing code for pushdown before we can proceed, and there will be a lot of merge conflicts.


fix: store selected language in cookies

Changes:

...

Type of change

[ ] Bug fix
[ ] New feature
[ ] Update feature
[ ] Refactor code
[ ] Translation to code
[ ] Translation to crowdin
[ ] Script configuration
[ ] Improve performance
[ ] Style only
[ ] Dependency update
[ ] Documentation update
[ ] Release

Closing this PR to avoid conflicts


Parser drops second **kwargs

https://play.ruff.rs/8503f0e1-7965-4515-8043-b711159a466b
The second kwargs must be preserved in the AST

Related: https://github.com/astral-sh/ruff/issues/10653


ENYO-5419: Sync page up/down navigation spec between VirtualList and Scroller

Issue Resolved / Feature Added
Scroller scrolls differently from VirtualList via page up or down key if it has disabled items.
Resolution

Find spottable item in viewport
Find a spottable item in the next page
Find a spottable item in the whole content to the key direction

Additional Considerations
VirtualList and Scroller support page up and down key navigation when they scrolls vertically. So I removed the code to scroll horizontally via page up/down keys in the VirtualList.
Links
ENYO-5419
Comments
Enact-DCO-1.0-Signed-off-by: YB Sung<EMAIL_ADDRESS>

#1822 PR should be merged first. Then I'll update CHANGELOG.md. Finally we could merge this PR into develop branch.


renal beta-intercalated cell should be part of collecting duct of renal tubule

Poss supporting refs (from ASCT+B table version 1.1)
PMID: 31249312. PMCID: PMC6597610	DOI: 10.1038/s41467-019-10861-2		PMID: 31689386	DOI: 10.1016/j.devcel.2019.10.005

@dosumis, would you want to also include 'part of' connecting tubule? I think type B = beta.
PMID: 29285423:
"Type B and non-A, non-B intercalated cells are found within the connecting tubule and the cortical collecting duct."
Not sure if the following text  only refers to beta-intercalated, but to consider:
PMID: 25632105:
"[Intercalated] cells appear in the late distal convoluted tubule or in the connecting segment, depending on the species. They are most abundant in the collecting duct... "
Also found typos in the definition for CL:0002201 renal beta-intercalated cell:
"A renal intercalated cell that secretes base and reabsorbs acid in the coritcal collecting duct to mantain acid/base balance."
--> cortical, maintain; respectively


[WISH]list required or recommended permissions for the bot in the FAQ

Thank you for your wish.
Can now be found in the current Read.me.


Missing/Invalid claim "kid" in auth header

Getting this error when trying to save a todo item... Any ideas what it could be refering to? All my google searches lead me to using something called JWT (http://fabricemontfort.com/firebase-and-as3-ep-2-login-with-oauth-2-0/).
I appreciate the help!

This means you are sending the idToken as the auth parameter.
You must send the access_token from a Refresh Token request as the auth parameter.
Later this week, I will update this app to reflect this changes.

Got it working... Thanks!