Use OPTIMIZE_UTILIZATION cluster autoscaler profile

GKE supports setting a cluster autoscaler profile that
optimizes utilization1. This changes the behavior of
the autoscaler and the scheduler, packing pods in tighter
into nodes rather than 'spreading' them around. This is
what the z2jh user-scheduler primarily does. We can actually
do away with the user-scheduler if we use the OPTIMIZE_UTILIZATION
autoscaler profile - Berkeley has been running under this setup
for a few months now. This reduces how much resources our core
nodes need, and the GKE provided scheduler is also faster.
This will also prevent issues like
https://github.com/2i2c-org/infrastructure/issues/1004.

Is there any downside to doing this? It seems pretty reasonable to me!

It would be nice to see a terraform plan for a cluster so we know if this change will be destructive or not. I am +1 on merging it if we don't need down-time to apply the changes to our clusters.

And I think that the Toronto folks will be happy to learn that we've improved the pod packing logic!

@choldgraf, see this comment from Yuvi in #1004:

I don't think such a thing is possible in AKS yet.

We can actually do away with the user-scheduler if we use the OPTIMIZE_UTILIZATION autoscaler profile

Do you want to open a follow-up issue to track that removal in the future?

@sgibson91 for some reason, pilot hubs and cloudbank already have the autoscaler set to this mode?!
Here's the plan for meom-ige:
Terraform used the selected providers to generate the following execution plan. Resource actions are indicated with the following
symbols:
  ~ update in-place

Terraform will perform the following actions:

  # google_container_node_pool.dask_worker["huge"] will be updated in-place
  ~ resource "google_container_node_pool" "dask_worker" {
        id                          = "projects/meom-ige-cnrs/locations/us-central1-b/clusters/meom-ige-cluster/nodePools/dask-huge"
        name                        = "dask-huge"
        # (9 unchanged attributes hidden)

      ~ node_config {
            tags              = []
            # (12 unchanged attributes hidden)

          ~ workload_metadata_config {
              + mode = "MODE_UNSPECIFIED"
            }
            # (1 unchanged block hidden)
        }

        # (3 unchanged blocks hidden)
    }

  # google_container_node_pool.dask_worker["large"] will be updated in-place
  ~ resource "google_container_node_pool" "dask_worker" {
        id                          = "projects/meom-ige-cnrs/locations/us-central1-b/clusters/meom-ige-cluster/nodePools/dask-large"
        name                        = "dask-large"
        # (9 unchanged attributes hidden)

      ~ node_config {
            tags              = []
            # (12 unchanged attributes hidden)

          ~ workload_metadata_config {
              + mode = "MODE_UNSPECIFIED"
            }
            # (1 unchanged block hidden)
        }

        # (3 unchanged blocks hidden)
    }

  # google_container_node_pool.dask_worker["medium"] will be updated in-place
  ~ resource "google_container_node_pool" "dask_worker" {
        id                          = "projects/meom-ige-cnrs/locations/us-central1-b/clusters/meom-ige-cluster/nodePools/dask-medium"
        name                        = "dask-medium"
        # (9 unchanged attributes hidden)

      ~ node_config {
            tags              = []
            # (12 unchanged attributes hidden)

          ~ workload_metadata_config {
              + mode = "MODE_UNSPECIFIED"
            }
            # (1 unchanged block hidden)
        }

        # (3 unchanged blocks hidden)
    }

  # google_container_node_pool.dask_worker["small"] will be updated in-place
  ~ resource "google_container_node_pool" "dask_worker" {
        id                          = "projects/meom-ige-cnrs/locations/us-central1-b/clusters/meom-ige-cluster/nodePools/dask-small"
        name                        = "dask-small"
        # (9 unchanged attributes hidden)

      ~ node_config {
            tags              = []
            # (12 unchanged attributes hidden)

          ~ workload_metadata_config {
              + mode = "MODE_UNSPECIFIED"
            }
            # (1 unchanged block hidden)
        }

        # (3 unchanged blocks hidden)
    }

  # google_container_node_pool.dask_worker["very-large"] will be updated in-place
  ~ resource "google_container_node_pool" "dask_worker" {
        id                          = "projects/meom-ige-cnrs/locations/us-central1-b/clusters/meom-ige-cluster/nodePools/dask-very-large"
        name                        = "dask-very-large"
        # (9 unchanged attributes hidden)

      ~ node_config {
            tags              = []
            # (12 unchanged attributes hidden)

          ~ workload_metadata_config {
              + mode = "MODE_UNSPECIFIED"
            }
            # (1 unchanged block hidden)
        }

        # (3 unchanged blocks hidden)
    }

  # google_container_node_pool.notebook["huge"] will be updated in-place
  ~ resource "google_container_node_pool" "notebook" {
        id                          = "projects/meom-ige-cnrs/locations/us-central1-b/clusters/meom-ige-cluster/nodePools/nb-huge"
        name                        = "nb-huge"
        # (9 unchanged attributes hidden)

      ~ node_config {
            tags              = []
            # (12 unchanged attributes hidden)

          ~ workload_metadata_config {
              + mode = "MODE_UNSPECIFIED"
            }
            # (1 unchanged block hidden)
        }

        # (3 unchanged blocks hidden)
    }

  # google_container_node_pool.notebook["large"] will be updated in-place
  ~ resource "google_container_node_pool" "notebook" {
        id                          = "projects/meom-ige-cnrs/locations/us-central1-b/clusters/meom-ige-cluster/nodePools/nb-large"
        name                        = "nb-large"
        # (9 unchanged attributes hidden)

      ~ node_config {
            tags              = []
            # (12 unchanged attributes hidden)

          ~ workload_metadata_config {
              + mode = "MODE_UNSPECIFIED"
            }
            # (1 unchanged block hidden)
        }

        # (3 unchanged blocks hidden)
    }

  # google_container_node_pool.notebook["medium"] will be updated in-place
  ~ resource "google_container_node_pool" "notebook" {
        id                          = "projects/meom-ige-cnrs/locations/us-central1-b/clusters/meom-ige-cluster/nodePools/nb-medium"
        name                        = "nb-medium"
        # (9 unchanged attributes hidden)

      ~ node_config {
            tags              = []
            # (12 unchanged attributes hidden)

          ~ workload_metadata_config {
              + mode = "MODE_UNSPECIFIED"
            }
            # (1 unchanged block hidden)
        }

        # (3 unchanged blocks hidden)
    }

  # google_container_node_pool.notebook["small"] will be updated in-place
  ~ resource "google_container_node_pool" "notebook" {
        id                          = "projects/meom-ige-cnrs/locations/us-central1-b/clusters/meom-ige-cluster/nodePools/nb-small"
        name                        = "nb-small"
        # (9 unchanged attributes hidden)

      ~ node_config {
            tags              = []
            # (12 unchanged attributes hidden)

          ~ workload_metadata_config {
              + mode = "MODE_UNSPECIFIED"
            }
            # (1 unchanged block hidden)
        }

        # (3 unchanged blocks hidden)
    }

  # google_container_node_pool.notebook["very-large"] will be updated in-place
  ~ resource "google_container_node_pool" "notebook" {
        id                          = "projects/meom-ige-cnrs/locations/us-central1-b/clusters/meom-ige-cluster/nodePools/nb-very-large"
        name                        = "nb-very-large"
        # (9 unchanged attributes hidden)

      ~ node_config {
            tags              = []
            # (12 unchanged attributes hidden)

          ~ workload_metadata_config {
              + mode = "MODE_UNSPECIFIED"
            }
            # (1 unchanged block hidden)
        }

        # (3 unchanged blocks hidden)
    }

Plan: 0 to add, 10 to change, 0 to destroy.
╷
│ Warning: Argument is deprecated
│
│   with google_filestore_instance.homedirs,
│   on storage.tf line 4, in resource "google_filestore_instance" "homedirs":
│    4:   zone = var.zone
│
│ Deprecated in favor of location.

Most of the changes are unrelated, and I think meom-ige also has this set by default...

This terraform plan was also after applying https://github.com/2i2c-org/infrastructure/pull/1035, as I couldn't get plan to work without bumping versions.

I also opened https://github.com/2i2c-org/infrastructure/issues/1034 to reduce confusion about our workspaces.

I just had a thought - could we run regression tests on terraform plan? e.g., could we:

Run terraform plan with our config
Write the output to a file
Check that file into our github repository, one for each cluster
In CI/CD, do a quick diff comparison between the two and do something to highlight the difference

I do this with pytest-regressions a lot and have found it a useful workflow.

The terraform plan is already a diff so I don't think diffing 2 plans will be useful, especially if they touch different parts of the infrastructure and have no overlap.
What you'd actually want to run a diff on is the state file (and that is exactly what terraform plan does), but we can't check that into version control as secrets are not encrypted in the state.

ah I see, I think I am exposing my ignorance of what terraform plan really does 😅

Hey all - can we merge this or is it waiting on something?

I'm going to have to deploy this on all our GCP workspaces, which would probably also mean cleaning up https://github.com/2i2c-org/infrastructure/issues/1034 along the way.

Done for everything except the pangeo hubs.

Done for everything now

We can actually do away with the user-scheduler if we use the OPTIMIZE_UTILIZATION autoscaler profile

Do you want to open a follow-up issue to track that removal in the future?

@damianavila i opened https://github.com/2i2c-org/infrastructure/pull/1126

Thanks!


Why the last part of any phone number is joined with a - ?

Hi,
On this line
https://github.com/taiga-family/maskito/blame/74654ef3e3807e462be2f0db1469d0ea9a4079ca/projects/phone/src/lib/masks/phone/utils/get-phone-template.ts#L10
A join with '-' is made for the last part of any phone number. This is wrong and breaks any validation done by libphonenumber-js library when using parsePhoneNumberFromString.
I assume it is maybe a standard format number in russia or other countries but many EU countries do not have any characters in their national phone number

Original libphonenumber by Google does use dashes though. Since our own standards suggest using dash and removal of it was a decision of libphonenumber-js author we reintroduced them. I suppose we can make this configurable.

I see, thank you for clarification how ever as the phone addon is based on libphonenumber-js I think you should align the format.


retained messages / subscription filter

Added filter to ensure clients only receive retained messages for matching subscriptions.

Thanks!


Sponsored Forts shouldn't be count in the total POI number of a cell

Hi there !
Sponsored Forts shouldn't count in the total POI number of a cell, since they don't have any impact on it.
I checked my database and looks like I sponsored forts are ones with partner_ID not NULL.
It also seems that they don't have an impact on fort apparition (20m rule is useless & new fort can spawn even if there is one sponsored gym on the S17 cell).
=> Exclude sponsored fort in L14 cell total count, and also from the "Points of Interest" toggle.
Pokestops that are used for pokemon showcase can't be turned into a gym, even if there isn't a sowcase on it.
It would now be interesting to have a visual information of either the pokestop is used for showcase or not.
=> Golbat Would it be possible keep in the database that a stop is used for pokemon showcases,
=> ReactMap aware the user on fort pop-up ?

I talked with some of my heavy Wayfarer users and they concluded that sponsored stops do actually count towards the number of POIs, however, this only syncs every 1-3 months so it may not be immediately apparent that it does count. However, in an upcoming PR I will allow users to turn off sponsored/partner stops to set it how they want to view it.

Resolved


MBPro 17" Mid 2010 won't install

It appears the tool installation is not compatible with this configuration.  Am I correct or just doing something wrong?

Do you have a BT 4.0 Dongle or upgraded BT4.0 card?

Bluetooth 4.0 dongle

On Jul 4, 2015, at 11:25 AM, Jeroen van Keulen<EMAIL_ADDRESS>wrote:
Do you have a BT 4.0 Dongle or upgraded BT4.0 card?
—
Reply to this email directly or view it on GitHub https://github.com/dokterdok/Continuity-Activation-Tool/issues/191#issuecomment-118540124.

Is it a Broadcom BCM20702?

Bluetooth explorer shows it to be a Broadcom but I am not sure if that is the model number.  System report shows it to have a 2070B0 chipset.

On Jul 4, 2015, at 11:26 AM, Jeroen van Keulen<EMAIL_ADDRESS>wrote:
Is it a Broadcom BCM20702?
—
Reply to this email directly or view it on GitHub https://github.com/dokterdok/Continuity-Activation-Tool/issues/191#issuecomment-118540175.

Most likely that is the issue..

I will return it and order another through Amazon.  Back to you after I get the replacement.
Thanks.

On Jul 4, 2015, at 11:38 AM, Jeroen van Keulen<EMAIL_ADDRESS>wrote:
Most likely that is the issue..
—
Reply to this email directly or view it on GitHub https://github.com/dokterdok/Continuity-Activation-Tool/issues/191#issuecomment-118540583.

Actually, I just went to Amazon and it shows that the dongle I ordered, made by GMYLE with “Broadcom Inside” uses chipset BCM20702.

On Jul 4, 2015, at 11:38 AM, Jeroen van Keulen<EMAIL_ADDRESS>wrote:
Most likely that is the issue..
—
Reply to this email directly or view it on GitHub https://github.com/dokterdok/Continuity-Activation-Tool/issues/191#issuecomment-118540583.

Should I assume that it won’t work and stop wasting your time?

On Jul 4, 2015, at 11:26 AM, Jeroen van Keulen<EMAIL_ADDRESS>wrote:
Is it a Broadcom BCM20702?
—
Reply to this email directly or view it on GitHub https://github.com/dokterdok/Continuity-Activation-Tool/issues/191#issuecomment-118540175.


List files from all columns

I am usually coding in two-columns mode. Is there any way to get the list of files from both columns?

@maarsson added to the latest version. You just need to disable only_show_items_in_focused_group in the settings


gh-690 - tidied up poms

Codecov Report

Merging #691 into develop will not impact coverage.

@@            Coverage Diff             @@
##             develop     #691   +/-   ##
==========================================
  Coverage      60.84%   60.84%
  Complexity      2600     2600
==========================================
  Files            379      379
  Lines          11597    11597
  Branches        1331     1331
==========================================
  Hits            7056     7056
  Misses          3982     3982
  Partials         559      559

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 f33cb04...35c89d3. Read the comment docs.


Add "disabled" property to mute errors when calling VIs manually

If I call my build VIs manually, either directly from the development environment or via my build GUI, the "Start CLI Interface.vi" throws an error (scan from string fails because list of parameters is empty). What I would like to be able to do is tell the CLI interface that it should just ignore all following calls and not throw any errors, so I don't have to disable all of the occurances of the "Write String.vi"s all over my code.

Definitely, have a better handling of the connect case to give a specific error and perhaps a Boolean flag you can work from.
I am slightly concerned about having the API disable itself in the case of a failed connection. Whilst I appreciate it would be more convenient if it did, this feels like something that would be better handled at the application layer.

Happy to get the opinion of others though, is this something that makes sense to have in the library (point 2)?

I didn't mean it should disable itself automatically, my thought was to have just the property inside the class so I can set it whenever I want.
I can see two alternatives for me: Wrap all "Write String.vi"s into a subVI with a case structure inside (but I still would have to wire the CLI reference and the enable/disable info), or create a subclass that implements the disable feature.

As per our discussion yesterday, I will implement a wrapper class for the functionality I'd like to see. It will enable me to either output to the command line if called via CLI or to a GUI if called from my LabVIEW build tool.


feat: Redirect GroupedExercise to ExerciseGroup

Preparation for https://github.com/serlo/db-migrations/issues/252

BTW: @elbotho After merging you can test it in staging (in the test I used the GroupedExercise of https://de.serlo.org/___content) and then deploy it yourself by merging branch staging into production (similar for deployment in frontend repo)


Remove user prompts from host setup

PR#367 ported to Android 12 SMR0/Branch
Tracked-On: OAM-105716
Signed-off-by: vdanix<EMAIL_ADDRESS>

updated the pr


Accessibility: Button component is adding label as aria-label

Describe the bug
When an aria label has not been provided Button component is adding label as aria-label, which conflicts with the visible label.
Expected behavior
export const Button = ({
  label,
  ...
  ariaLabel = `${label}`, // this is not required

<button type="submit" aria-label="THIS IS NOT REQUIRED IF NOT PROVIDED" class="dcx-button" id="some-id">
Visible LabelHidden Text
</button>

** Required Action **

[ ] remove the default addition of label to aria-label

closed by #664


Fixed issue where Oracle database results were not ordered when filtering with multiple identity claims

Purpose
To fix the issue where, when filtering users by 2+ identity claims (such as accountLock, accountDisabled, accountState) from an Oracle database, the results received were not in order of UM_USER_NAME.
Resolves issue: https://github.com/wso2/product-is/issues/13883
Approach
The issue was with the closing of brackets when generating the SQL statement. The fix identifies the number of Identity claims that the request filters by and adds closing brackets appropriately, such that the order by is done at the correct level.
Related PRs
https://github.com/wso2-extensions/identity-governance/pull/592 should be merged to make this change functional.
Test environment

Open JDK 11
WSO2IS 5.12.0
Ubuntu Linux
Oracle 19.3.0-ee

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2627455711

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2627455711
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2634977080

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2634977080
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2637488162

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2637488162
Status: failure

The above test failure is known one.
[ERROR] org.wso2.identity.integration.test.rest.api.server.application.management.v1.ApplicationMetadataPositiveTest.testGetSAMLMetadata(org.wso2.identity.integration.test.rest.api.server.application.management.v1.ApplicationMetadataPositiveTest)
[ERROR]   Run 1: ApplicationMetadataPositiveTest.testGetSAMLMetadata:185 SAML Metadata returned from the API doesn't match. expected:<class SAMLMetaData {
    defaultNameIdFormat: urn:oasis:names:tc:SAML:1.1:nameid-format:unspecified
    certificateAlias: class MetadataProperty {
    options: [digicertassuredidrootca, comodorsaca, thawtepremiumserverca, swisssignplatinumg2ca, swisssignsilverg2ca, identrustdstx3, securetrustca, digicertassuredidg3, digicertassuredidg2, teliasonerarootcav1, identrustpublicca, affirmtrustnetworkingca, entrustevca, globalsigneccrootcar5, globalsigneccrootcar4, certumca, addtrustclass1ca, wso2carbon, entrustrootcag2, quovadisrootca3, quovadisrootca2, digicertglobalrootg3, digicertglobalrootg2, digicerthighassuranceevrootca, geotrustuniversalca, verisignclass3ca, thawteprimaryrootcag3, thawteprimaryrootcag2, buypassclass3ca, geotrustprimaryca, dtrustclass3ca2, buypassclass2ca, baltimorecybertrustca, starfieldclass2ca, actalisauthenticationrootca, camerfirmachamberscommerceca, globalsignrootcar6, ttelesecglobalrootclass3ca, digicerttrustedrootg4, verisignclass3g5ca, ttelesecglobalrootclass2ca, verisignclass3g4ca, verisignclass3g3ca, xrampglobalca, entrustrootcaec1, certumtrustednetworkca, verisignclass3g2ca, globalsignr3ca, secomscrootca2, secomscrootca1, affirmtrustcommercialca, verisignuniversalrootca, globalsignr2ca, letsencryptisrgx1, digicertglobalrootca, globalsignca, thawteprimaryrootca, starfieldrootg2ca, geotrustglobalca, soneraclass2ca, verisigntsaca, quovadisrootca, affirmtrustpremiumeccca, starfieldservicesrootg2ca, comodoaaaca, addtrustqualifiedca, keynectisrootca, addtrustexternalca, luxtrustglobalrootca, verisignclass2g2ca, quovadisrootca3g3, geotrustprimarycag3, geotrustprimarycag2, swisssigngoldg2ca, entrust2048ca, quovadisrootca2g3, chunghwaepkirootca, camerfirmachambersignca, quovadisrootca1g3, camerfirmachambersca, usertrusteccca, godaddyclass2ca, affirmtrustpremiumca, identrustcommercial, usertrustrsaca, dtrustclass3ca2ev, comodoeccca, godaddyrootg2ca]
    defaultValue: wso2carbon

should be fixed with https://github.com/wso2/product-is/pull/14159

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2640220406

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2640220406
Status: success


Fix Regression #751 due to IBM z/OS changes

Recently updated issue #751 describes the case where path can be null, thus causing an exception to be thrown.
So, rearranging the OR clause and adding a null check to fix this.

Great fix, thanks!

Released.
https://www.nuget.org/packages/FluentFTP/35.2.1


Permite enviar parâmetros na busca por transações de assinaturas

Description
Resolve o problema da issue https://github.com/pagarme/pagarme-js/issues/269
Com essa alteração será possível enviar os parâmetros count e page na rota subscriptions/:id/transactions
Dúvida: Preciso commitar os arquivos alterados com yarn build?
Your checklist for this pull request
:rotating_light: Please review this items for a good pull request. :four_leaf_clover:

I've read the project's Contributing Guidelines
My commits are well written and follow pagarme/git-style-guide
My changes are well covered by tests and logs
I've updated the project docs (if needed)
I fell safe about this implementation

In a good pull request, everything above is true :relaxed:

There are some issues with your commit and our naming convention

Commit: 1f15a9aa7a3314f6a7063e816504d544b847f26a
Commits must be in the following format: %{context}: %{description}

The complete commit style guide is available at https://github.com/pagarme/git-style-guide

This message was auto-generated by https://gitcop.com

There are some issues with your commit and our naming convention

Commit: 1f15a9aa7a3314f6a7063e816504d544b847f26a
Commits must be in the following format: %{context}: %{description}

The complete commit style guide is available at https://github.com/pagarme/git-style-guide

This message was auto-generated by https://gitcop.com

There are some issues with your commit and our naming convention

Commit: 1f15a9aa7a3314f6a7063e816504d544b847f26a
Commits must be in the following format: %{context}: %{description}

The complete commit style guide is available at https://github.com/pagarme/git-style-guide

This message was auto-generated by https://gitcop.com

There are some issues with your commit and our naming convention

Commit: 1f15a9aa7a3314f6a7063e816504d544b847f26a
Commits must be in the following format: %{context}: %{description}

The complete commit style guide is available at https://github.com/pagarme/git-style-guide

This message was auto-generated by https://gitcop.com

There are some issues with your commit and our naming convention

Commit: 1f15a9aa7a3314f6a7063e816504d544b847f26a
Commits must be in the following format: %{context}: %{description}

The complete commit style guide is available at https://github.com/pagarme/git-style-guide

This message was auto-generated by https://gitcop.com

There are some issues with your commit and our naming convention

Commit: 1f15a9aa7a3314f6a7063e816504d544b847f26a
Commits must be in the following format: %{context}: %{description}

The complete commit style guide is available at https://github.com/pagarme/git-style-guide

This message was auto-generated by https://gitcop.com

There are some issues with your commit and our naming convention

Commit: 262178e546eea454f0e840c9245db95ac2beb81d
Commits must be in the following format: %{context}: %{description}

The complete commit style guide is available at https://github.com/pagarme/git-style-guide

This message was auto-generated by https://gitcop.com

There are some issues with your commit and our naming convention

Commit: 262178e546eea454f0e840c9245db95ac2beb81d
Commits must be in the following format: %{context}: %{description}

The complete commit style guide is available at https://github.com/pagarme/git-style-guide

This message was auto-generated by https://gitcop.com

There are some issues with your commit and our naming convention

Commit: 262178e546eea454f0e840c9245db95ac2beb81d
Commits must be in the following format: %{context}: %{description}

The complete commit style guide is available at https://github.com/pagarme/git-style-guide

This message was auto-generated by https://gitcop.com

There are some issues with your commit and our naming convention

Commit: 262178e546eea454f0e840c9245db95ac2beb81d
Commits must be in the following format: %{context}: %{description}

The complete commit style guide is available at https://github.com/pagarme/git-style-guide

This message was auto-generated by https://gitcop.com


Add support for implicit member expression

Swift has support for implicit member expression (also called leading dot notation).
In python terms, the following codes would be valid
class Color(Enum):
	RED = 1

mycolor: Color = .RED

Static members can be looked up by type inference. Since mycolor expects a Color, .RED resolves to Color.RED.
class Color(Enum):
	RED = 1

	@staticmethod
	def random_color() -> "Color": ...

mycolor2: Color = .random_color()

Static methods can also be looked up.
def print_color(color: Color): ...

print_color(.RED)
print_color(.random_color())

This works for function arguments too.
This feature cannot be added to Python itself because if relies on type information.
But, it would be nice if it could be added to Pylance.
If pyright knows the types allowed, and the user presses the dot key in an empty field,

Pylance can suggest valid members for all valid types
If the user presses enter or continues lookup by pressing dot, Pylance fills in the complete type

This is not valid Python syntax. If you attempt to run any of the above code samples in the Python interpreter, you'll see that they all result in a syntax error.
If you want to propose changes to the Python language, the [python "ideas"](https://discuss.python.org/c/ideas/6 discussion) would be a more appropriate forum.

Yes, that code isn't valid. I'm asking for the dot key to enable more completions within Pylance.
Current
Currently, given
from enum import Enum

class Color(Enum):
	RED = 1

mycolor: Color = Color.

the moment I press the ., I get a suggestion for RED

Starting at mycolor: Color = , the keystrokes I use to type a valid value are ['Shift+c', 'Enter', '.', 'Enter'].
Request
It would be nice if in addition to that, given
from enum import Enum

class Color(Enum):
	RED = 1

mycolor: Color =

I can press ., and I'll be suggested Color.RED in its entirety.
The keystrokes I would use would be ['.', 'Enter'].
Because pyright knows the type is Color, it can fill in that information for me.
I hope this explains my request better. I understand that in this basic example, its two keystrokes saved. But, the end result is that I no longer have to thing about how a developer organizes / lays out the types in their library. The type in question could be multiple modules / nested classes deep, but Pylance would fill it in for me.
If a library had a controllerlib.HIDS.Controller.Hand enum with types Left and Right, and had a toplevel controllerlib.vibrate(hand: "the above type"), then typing a single . and having the correct type filled in would be way better than going hunting for the full type path required.

Moving this issue to discussion as an enhancement request for comments and upvotes.


genomics-platform/analyzing-data: Unhide cis-X workflow guide

This reenables the entry for cis-X in the table of contents.

Are Yu Liu, Chunliang Li, and Shuhong Shen still the correct authors?

The manuscript lists them as equally contributing primary authors. Jinghui Zhang is listed as the corresponding author. There are 22 authors total, if all of them should be listed here.

I think we're good to go with the 3 people you have listed.

From: Michael Macias<EMAIL_ADDRESS>Sent: Wednesday, May 20, 2020 1:55 PM
To: stjudecloud/docs<EMAIL_ADDRESS>Cc: Birch, Kirby<EMAIL_ADDRESS>Review requested<EMAIL_ADDRESS>Subject: Re: [stjudecloud/docs] genomics-platform/analyzing-data: Unhide cis-X workflow guide (#126)
Caution: External Sender
Are Yu Liu, Chunliang Li, and Shuhong Shen still the correct authors?
The manuscript lists them as equally contributing primary authors. Jinghui Zhang is listed as the corresponding author. There are 22 authors total, if all of them should be listed here.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fstjudecloud%2Fdocs%2Fpull%2F126%23issuecomment-631661144&data=01|01|Kirby.birch%40stjude.org|354fd840483642af521408d7fcef6298|22340fa892264871b677d3b3e377af72|0&sdata=o3OSS%2FgGaM4WjNFr0FCPzOZUxK1xKtfBUrDO746NsyU%3D&reserved=0, or unsubscribehttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIMFRMISNHJZHCO3OOFY4HTRSQRSTANCNFSM4NGGG2ZA&data=01|01|Kirby.birch%40stjude.org|354fd840483642af521408d7fcef6298|22340fa892264871b677d3b3e377af72|0&sdata=iX7VDVsxe4YM2Lwh7%2B8s%2BjF%2FVwB7p4KlF%2BAyx7CdvVQ%3D&reserved=0.

Email Disclaimer: www.stjude.org/emaildisclaimer
Consultation Disclaimer: www.stjude.org/consultationdisclaimer


Use kit.config.paths.base in local development

Describe the problem
Consider the use case of deploying to GitHub Pages, where a base URL must be specified (e.g., the name of the repo).
However, when developing a SvelteKit app locally, the base URL cannot be specified. A "not found" error is shown when attempting to navigate to http://localhost/<base-url>/.
As a result, base must be conditionally applied only when building the app for production.
/** @type {import('@sveltejs/kit').Config} */
const config = {
  kit: {
    adapter: adapter(),
    paths: {
       base: process.env.NODE_ENV === "production" ? "/repo-name" : "",
    },
  },
};

Prior Art
vite uses the base URL when developing locally:
// vite.config.js
import { defineConfig } from "vite";

export default defineConfig({
  base: "/repo-name/",
});

Run vite dev:
vite v2.7.13 dev server running at:

> Local: http://localhost:3000/repo-name/
> Network: use `--host` to expose

ready in 342ms.

Describe the proposed solution
The desired UX would be to avoid conditionally applying a base path:
/** @type {import('@sveltejs/kit').Config} */
const config = {
  kit: {
    adapter: adapter(),
    paths: {
       base: "/repo-name"
    },
  },
};

This would also make it easier to preview the app locally (i.e. svelte-kit preview).
Alternatives considered
No response
Importance
nice to have
Additional Information
No response

Sounds like a duplicate of #2958.

Oof. Yeah, this is a dupe.


Make BitmapDocIdSetOperator a top level operator

Make  BitmapDocIdSetOperator a top level operator so that it can be shared

Codecov Report

Merging #7068 (9fc179c) into master (b826f2f) will decrease coverage by 0.11%.
The diff coverage is 73.17%.

@@             Coverage Diff              @@
##             master    #7068      +/-   ##
============================================
- Coverage     42.04%   41.93%   -0.12%
  Complexity        7        7
============================================
  Files          1479     1480       +1
  Lines         72814    72814
  Branches      10471    10469       -2
============================================
- Hits          30615    30533      -82
- Misses        39651    39732      +81
- Partials       2548     2549       +1

Flag
Coverage Δ

integration
41.93% <73.17%> (-0.12%)
:arrow_down:

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

Impacted Files
Coverage Δ

...or/dociditerators/ExpressionScanDocIdIterator.java
62.85% <70.00%> (-0.91%)
:arrow_down:

...he/pinot/core/operator/BitmapDocIdSetOperator.java
75.00% <75.00%> (ø)

.../core/operator/query/SelectionOrderByOperator.java
94.73% <100.00%> (+0.25%)
:arrow_up:

...egment/local/io/writer/impl/MmapMemoryManager.java
0.00% <0.00%> (-72.31%)
:arrow_down:

...a/manager/realtime/RealtimeSegmentDataManager.java
50.00% <0.00%> (-50.00%)
:arrow_down:

...ache/pinot/core/operator/docidsets/OrDocIdSet.java
56.81% <0.00%> (-43.19%)
:arrow_down:

.../filter/predicate/InPredicateEvaluatorFactory.java
50.50% <0.00%> (-10.11%)
:arrow_down:

.../controller/helix/core/SegmentDeletionManager.java
53.27% <0.00%> (-9.84%)
:arrow_down:

...tion/groupby/DictionaryBasedGroupKeyGenerator.java
42.65% <0.00%> (-7.96%)
:arrow_down:

...impl/dictionary/DoubleOnHeapMutableDictionary.java
46.98% <0.00%> (-6.03%)
:arrow_down:

... and 39 more

Continue to review full report at Codecov.

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


Creating new amplify environment stuck on building

Before opening, please confirm:

[X] I have searched for duplicate or closed issues and discussions.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

JavaScript Framework
React
Amplify APIs
Not applicable
Amplify Version
v5
Amplify Categories
Not applicable
Backend
Amplify CLI
Environment information

@aws-amplify/cli: 12.2.3

Describe the bug
When I try to create a new hosting environment on the aws website I run into a problem.
The app get's stuck on build phase. The cli tries to open a website link to authenticate my user account. This is obviously not working as this is not my local cli, but running on AWS. The link it is try to open is:
Opening link: https://eu-west-3.admin.amplifyapp.com/admin/{**}/dev/verify/?loginVersion=1
Expected behavior
I expect the build to authenticate my amplify account automatically
Reproduction steps

create amplify project.
configure hosting
publish
Try to create a new hosting environment on the aws website

Code Snippet
This is not related to my code
Log output

Checking for associated backend environment...
Backend environment association found
2024-03-18T13:34:13.458Z [INFO]: Note: It is recommended to run this command from the root of your app directory
2024-03-18T13:34:13.728Z [INFO]: Opening link: https://eu-west-3.admin.amplifyapp.com/admin/**/dev/verify/?loginVersion=1

aws-exports.js
No response
Manual configuration
No response
Additional configuration
No response
Mobile Device
No response
Mobile Operating System
No response
Mobile Browser
No response
Mobile Browser Version
No response
Additional information and screenshots
No response

@tomruijgrok it appears the CLI is trying to authenticate using Studio authentication. You can disable Amplify Studio on the AWS console and retry the build. Refer to this issue https://github.com/aws-amplify/amplify-cli/issues/13661 providing additional information on this.
if you require any additional assistance do reach out AWS technical support for further assistance.


Illegal argument value: Positional arguments were provided ',barcodes.csv}' but no positional argument is defined for this tool.

Hi, I am trying to perform the step 4 Option (a) of the Sicelore pipeline. For this first test I used a bam with only one chromosome.
java -jar -Xmx40g Jar/Sicelore-2.1.jar IsoformMatrix -I ${umidir}chr19_only.primary_only.passed.Parsed.bam -GENETAG GE -UMITAG U8 -CELLTAG BC -REFFLAT $gencoderefflat -CSV ${siceloredir}barcodes.csv -DELTA 2 -MAXCLIP 150 -METHOD STRICT -AMBIGUOUS_ASSIGN false -OUTDIR $siceloredir -PREFIX sicelore
However, I am receiving following error:
Illegal argument value: Positional arguments were provided ',barcodes.csv}' but no positional argument is defined for this tool.
Please let me know if I need to provide you with further information.
Thanks,
Tijs

Hi,
never seen this kind of error message, please double check your command line:
https://gatk.broadinstitute.org/hc/en-us/community/posts/360076948391--No-positional-argument-is-defined-Error-in-HaplotypeCaller
https://gatk.broadinstitute.org/hc/en-us/community/posts/360077380972-ApplyBQSR-no-positional-argument-is-defined-for-this-tool
or try to provide the full path to the files
best,
kevin

Hi Kevin, providing full paths solved the issue.
Thanks for the quick solution!
Tijs


UML 2869 / UML-2858 images not ready or errored

Purpose
UML-2869 / UML-2858 show temporary text for image not yet ready or error text if errored out
Approach
Explain how your code addresses the purpose of the change
Learning
Any tips and tricks, blog posts or tools which helped you. Plus anything notable you've discovered about the Use a Lasting Power of Attorney service
Checklist

[ ] I have performed a self-review of my own code
[ ] I have added relevant logging with appropriate levels to my code
[ ] New event_codes have been documented on the wiki page
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] I have added welsh translation tags and updated translation files
[ ] I have run an accessibility tool on any pages I have made changes to and fixed any issues found
[ ] I have notified the Interaction Designer of any content changes so that appropriate screenshots/flow diagram changes can be made
[ ] The product team have tested these changes

This work is now merged into UML-2766_display_i_and_p_to_viewers branch so this individual PR no longer applies


Is there any way to inject a generic type?

I know the documentation says that this is not supported, but I'm wondering if there is any workaround to simulate this. I cannot find any DI solution for Android that supports injecting a generic type like this:
public class MyActivity<P extends BasePresenter> extends Activity {
    @Inject
    P presenter;
}

Thank you!

No, this is not supported by Toothpick. But it would work with a subclass
of MyActivity. We need a "real"/resolved class to perform injection.
2016-07-12 19:29 GMT-07:00 Martin Heras<EMAIL_ADDRESS>
I know the documentation says that this is not supported, but I'm
wondering if there is any workaround to simulate this. I cannot find any DI
solution for Android that supports injecting a generic type like this:
public class MyActivity extends Activity {
@Inject
P presenter;
}
Thank you!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stephanenicolas/toothpick/issues/122, or mute the
thread
https://github.com/notifications/unsubscribe/ABv33bTEXUH3KgUIxKGqQdnlx39pSlsbks5qVE2ggaJpZM4JLAbA
.

@stephanenicolas could you please extend your explanation. I assume you can @inject the filed as soon its type is contrait, however how could I work with the injected field at abstract level ?
public class TheActivity extends MyActivity<ThePresenter> {
    @Inject
    ThePresenter presenter;
}

What to do here to handle stuff with the presenter inject in the concrete activity
public class MyActivity<P extends BasePresenter> extends Activity {
    @Inject
    P presenter;

   @Override
   onResume(){
    presenter.start();
  }
}


[B03381] Define user roles, implement CURATOR

Coverage decreased (-0.2%) to 29.707% when pulling 412cf10a2d93b8d20483e5e8f925ab6d57bec01e on sprint2-b03381-user-roles into 4a5b149b370f9abca9641f74bb42822672e00a92 on sprint2-staging.


zh-trans: add includes/default-storage-class-prereqs.md

zh-trans: add includes/default-storage-class-prereqs.md

/assign @haibinxie

@zacharysarah I see that @haibinxie has no activies for more that one year. Maybe we want to remove him from the related OWNERS file at the moment :)

2：1，ok

/lgtm
/approve

可以合并了，原文翻译 解决了


[FEATURE] Another option to close the modal

Description
When a user clicks outside the modal, it should close it
Screenshots
No response

Thank you for creating this issue. Wait until you are allowed to work on this before you start, and do not forget to include this issue in your pull request. While you wait, don't forget to join the community - [Automated].

:tada: This issue has been resolved in version 1.9.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

Thank you for creating this issue. Wait until you are allowed to work on this before you start, and do not forget to include this issue in your pull request. While you wait, don't forget to join the community - [Automated].


Straighten the index actor signature

The internal query handler of the index used to return void but
actually sent an unsafe response with the uuid, candidates and
scheduled partitions triple to the requester. The historical
reason for this was to prevent a race condition which happened when
results started flowing to the requester before it received the
initial response. This problem happens when the OS scheduler
suspends the thread between initiating the worker and returning
the triple.
This commit replaces the old workaround with a cleaner approach:
We now use a response promise and deliver it before initiating the
worker. This is enough to guarantee that the messages will arrive
in the expected order at the requester.
Thanks to that the query handler signature now correctly indicates
that the requester should expect a <uuid, uint32_t, uint32_t> as
response.
:memo: Checklist

[x] The PR description contains instructions for the reviewer, if necessary.

:dart: Review Instructions
Verify that the message flow between index, requester and query_supervisor is unchanged.
Safe the unit test changes for last, the mock index implementations had to be refactored because the query handler is more strongly typed now.

The style check failed anyways, so please excuse my last minute spelling fix request.


Updated extraKeys (bold and italic) and added new shortcut (for span tags)

So, this pull request is to address #1148. I also added a shortcut to wrap a selection in span tags (I chose Ctrl+F1 because it don't think it messes with any browser/system shortcuts afaik, but I'm open to suggestions). With the new renderer, it may be interesting to add a shortcut for the new divs ({{ }}) as well.
I can't test if the Cmd+B/+I/+F1 shortcuts work (I'm on Windows), but there's no reason for them not to work.
I left strikethrough key maps out, but I can add them too.

I would caution away from mapping Cmd-[x], as those are the keys the browser uses for commands. Right now, Chrome doesn't have any key bindings for cmd-B or cmd-I, but it does have a mapping for cmd-S.
So, while we could be cautious and judicious with selecting specific keys for homebrewery (to avoid clashes), we will end up having to choose some odd-looking alternatives (e.g. we can't use cmd-S for strikethru so we to choose something else which is less obvious). [Actually, we've already hijacked cmd-S from the browser, invoking the "save edits of brew" function.]
Although there are no characters for ctrl-{ (like there are for ctrl-B), JS doesn't detect characters, it detects keys and modifiers. So we could go ahead with ctrl-{ for span (or for {{..}} blocks/spans).

I would caution away from mapping Cmd-[x], as those are the keys the browser uses for commands. Right now, Chrome doesn't have any key bindings for cmd-B or cmd-I, but it does have a mapping for cmd-S.
So, while we could be cautious and judicious with selecting specific keys for homebrewery (to avoid clashes), we will end up having to choose some odd-looking alternatives (e.g. we can't use cmd-S for strikethru so we to choose something else which is less obvious). [Actually, we've already hijacked cmd-S from the browser, invoking the "save edits of brew" function.]
Although there are no characters for ctrl-{ (like there are for ctrl-B), JS doesn't detect characters, it detects keys and modifiers. So we could go ahead with ctrl-{ for span (or for {{..}} blocks/spans).

This seems fine to me. Other websites (like Github) also have keybindings for Ctrl-I, Ctrl-B for bold, italicize. It's not unusual.  CodeMirror even explicitly gives examples of doing this to override existing browser key-combinations. And we do already use CTRL-S for saving; it's a hotkey for most other text editors, so it makes sense to change the default key-combo behavior in that case. The Homebrewery is a web-based App, not a static page in the browser, and many other "text-editor" webapps do the same thing. This is very common practice.
I do like the idea of changing the "span" key combo to use the { button though. @RKuerten Would you make that change and then I can merge this?


Update rooms-pro-management.md

Updating statement around supportability for MTR Android and Surface Hub

@altsou
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@altsou
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

Closing. Stale


RxSwift 6.0

Hello mates!, could You please update your Rx dependencies to 6.0 version

Hi! Thank you for the reminder, I'll do that in the coming days.

Hi! Thank you for the reminder, I'll do that in the coming days.

Release 1.7.0 uses RxSwift 6.0 now

Release 1.7.0 uses RxSwift 6.0 now

@pixeldock thanks a lot)

@pixeldock thanks a lot)


[SR-16004] [SwiftSyntax] New TSan error with 0.50600.0

Previous ID
SR-16004

Radar
None

Original Reporter
@jpsim

Type
Bug

Attachment: Download

Environment
Swift 5.6 / Xcode 13.3
Thread Sanitizer enabled
SwiftSyntax 0.50600.0
macOS 12.2.1, M1 Max

Additional Detail from JIRA

Votes
0

Component/s
SwiftSyntax

Labels
Bug

Assignee
None

Priority
Medium

md5: 71c102b19d875fbcb3290a083ca252e0

Issue Description:
Running the attached sample project with --sanitize=thread works fine with SwiftSyntax at the release/5.5 branch with Swift 5.5 but crashes with a stack overflow with SwiftSyntax at the 0.50600.0 branch with Swift 5.6.
Steps to reproduce the crash with Xcode 13.3:
$ export DEVELOPER_DIR=/Applications/Xcode-13.3.app/Contents/Developer
$ rm -rf .build
$ swift run --sanitize=thread swift-syntax-tsan-error --crash
Fetching https://github.com/apple/swift-syntax.git from cache
Fetched https://github.com/apple/swift-syntax.git (1.00s)
Creating working copy for https://github.com/apple/swift-syntax.git
Working copy of https://github.com/apple/swift-syntax.git resolved at 0.50600.0
[20096/20096] Downloading binary artifacts
Building for debugging...
[48/48] Linking swift-syntax-tsan-error
Build complete! (12.76s)
swift-syntax-tsan-error(36117,0x106f34580) malloc: nano zone abandoned due to inability to preallocate reserved vm space.
[1]    36117 illegal hardware instruction  swift run --sanitize=thread swift-syntax-tsan-error --crash

And to demonstrate that the same usage does not crash with Xcode 13.2.1:
$ export DEVELOPER_DIR=/Applications/Xcode-13.2.1.app/Contents/Developer
$ rm -rf .build
$ swift run --sanitize=thread swift-syntax-tsan-error --crash
Fetching https://github.com/apple/swift-syntax.git from cache
Fetched https://github.com/apple/swift-syntax.git (1.28s)
Creating working copy for https://github.com/apple/swift-syntax.git
Working copy of https://github.com/apple/swift-syntax.git resolved at release/5.5
[46/46] Build complete!
swift-syntax-tsan-error(36430,0x102a04580) malloc: nano zone abandoned due to inability to preallocate reserved vm space.

Sorry that I only notice this issue now.
I wasn’t able to reproduce the issue using Xcode 13.3 with the attached project. Does this still reproduce for you? And if so, does it also reproduce with the Xcode 14 beta? If yes, please re-open the issue.

Does this still reproduce for you?

Yes, the attached sample project still crashes for me with Xcode 13.3.1.

And if so, does it also reproduce with the Xcode 14 beta?

No crash with the Xcode 14 beta (14A5228q), but the malloc log is still printed, I don't know what it means.
swift run --sanitize=thread swift-syntax-tsan-error --crash
Building for debugging...
Build complete! (0.11s)
swift-syntax-tsan-error(3667,0x1068fc580) malloc: nano zone abandoned due to inability to preallocate reserved vm space.

I'm ok to keep this issue closed unless the malloc log is a problem.

If it passes on Xcode 14 beta, let’s keep this closed.
Regarding the malloc: nano zone abandoned due to inability to preallocate reserved vm space. message: I did some investigation and it appears to be benign (I’m not a TSan expert though). Apparently, it can be avoided by setting MallocNanoZone=0 in your environment.


restore the displays panel in the GUI

This might be a deceptively complex issue, as it might rely on other things getting fixed.

It may still have bugs, but it was reenabled in #123.


After upgrade to 0.0.339 Pulumi leaks error

Hello to all, couple of minutes ago I upgraded to 0.0.39 and can't start development enviroment:
ERROR:

The Pulumi runtime detected that 17 promises were still active
at the time that the process exited. There are a few ways that this can occur:

Not using await or .then on a Promise returned from a Pulumi API
Introducing a cyclic dependency between two Pulumi Resources
A bug in the Pulumi Runtime
Leaving promises active is probably not what you want. If you are unsure about
why you are seeing this message, re-run your program with the PULUMI_DEBUG_PROMISE_LEAKS
environment variable. The Pulumi runtime will then print out additional
debug information about the leaked promises.
|  Error

×  Failed

Just saw this too, taking a look.

Reverted it here, still looking at it https://github.com/sst/ion/releases/tag/v0.0.340


Revert S3 200 errors implementation

This PR reverts the S3 200 Errors implementation (#3276) to address bugs introduced introduced in payload operations.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 93.09%. Comparing base (0e229ae) to head (974dc1f).
Report is 131 commits behind head on develop.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3287      +/-   ##
===========================================
- Coverage    93.17%   93.09%   -0.09%
===========================================
  Files           66       66
  Lines        14339    14345       +6
===========================================
- Hits         13361    13354       -7
- Misses         978      991      +13

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


fine-tune this model in tensorflow  with 'nan' problem

I've converted this model to tensorflow framework,  and lauch the graph with .npy data file, the data file upload without any problem since I printed all value in tf.all_variables() for debug . But I found it cann't backprop properly,the trainable variables became nan at the first bp process. I'm not very skillful at CNN tricks since my physics background.  Do I need adjust some special layers of this net? and what's the proper
method of optimization?
Thanks for any insightful and helpful analysis and advice!

@lisztfrancis I was thinking the same thing, at the end I preferred Caffe. The performances on the api using Tornando Web Server + Docker is super with this ResNet-50 pre-trained model.
Which advantage did you find converting to TF?


WIP: Use a single queue per remote instead of 4 queues.

Checklist

[ ] Affected Issues have been mentioned in the Closing issues section
[ ] Documentation has been written/updated
[ ] PR title is ready for inclusion in changelog

The remote-controller has a stability improvement PR (https://github.com/uselagoon/remote-controller/pull/140) that enables a single queue per remote-controller.
NOTE: This has not been released yet in the remote-controller, so this PR is a WIP and will fail until remote-controller is released with the single-queue support update.
To support this, Lagoon needs to be updated to send all the messages to the new single queue.

@shreddedbacon to resuscitate this and move forwards

closing in favoor of NATS discovery (for now :thinking:


[REQUEST] let the output folder  be editable.

Request:
User could input/edit   the output folder.  if the folder dosen't exists , create it
Thanks

+1 for the output folder being editable

The output folder is editable via the destination field. The only thing it doesn't do is create a directory that doesn't exist yet.


Bug fix coredump on no null string type rf

Proposed changes
fix coredump on no null string type rf
** SIGSEGV address not mapped to object (@0x1) received by PID 9781 (TID 13516 OR 0x7f961d90c700) from PID 1; stack trace: ***
0# doris::signal::(anonymous namespace)::FailureSignalHandler(int, siginfo_t*, void*) at /home/zcp/repo_center/doris_master/doris/be/src/common/signal_handler.h:421
1# PosixSignals::chained_handler(int, siginfo*, void*) [clone .part.0] in /usr/lib/jvm/java-17-openjdk-amd64/lib/server/libjvm.so
2# JVM_handle_linux_signal in /usr/lib/jvm/java-17-openjdk-amd64/lib/server/libjvm.so
3# 0x00007FA0B1B5C090 in /lib/x86_64-linux-gnu/libc.so.6
4# void doris::StringSet, std::allocator > > >::_insert_fixed_len_string >(doris::vectorized::ColumnStr const&, unsigned char const*, unsigned long, unsigned long) at /home/zcp/repo_center/doris_master/doris/be/src/exprs/hybrid_set.h:424
5# doris::StringSet, std::allocator > > >::insert_fixed_len(COW::immutable_ptr const&, unsigned long) in /mnt/hdd01/ci/master-deploy/be/lib/doris_be
6# doris::RuntimePredicateWrapper::insert_fixed_len(COW::immutable_ptr const&, unsigned long) at /home/zcp/repo_center/doris_master/doris/be/src/exprs/runtime_filter.cpp:409
7# doris::RuntimePredicateWrapper::insert_batch(COW::immutable_ptr const&, unsigned long) in /mnt/hdd01/ci/master-deploy/be/lib/doris_be
8# doris::IRuntimeFilter::insert_batch(COW::immutable_ptr, unsigned long) at /home/zcp/repo_center/doris_master/doris/be/src/exprs/runtime_filter.cpp:971
9# doris::VRuntimeFilterSlots::insert(doris::vectorized::Block const*) at /home/zcp/repo_center/doris_master/doris/be/src/exprs/runtime_filter_slots.h:124
10# doris::Status doris::vectorized::process_runtime_filter_build(doris::RuntimeState*, doris::vectorized::Block*, doris::vectorized::HashJoinNode*, bool) at /home/zcp/repo_center/doris_master/doris/be/src/vec/exec/join/vhash_join_node.h:90

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


display of non- directional GIs

It might be better to switch the order so that the gene of the gene page always comes first:

i.e rad2 exo5 here

I'll do that.
I don't know if it's related, but looks like there is a bug displaying that interaction.  There is no relation between the "rad2" and the "exo5".  Probably missing configuration.

Probably missing configuration.

I think that's the problem.  I've added the config locally and it's displays correctly, with exo5 in the first column.

There is no relation between the "rad2" and the "exo5".

Can you suggest what the text should be?  In Canto it's "displays Synthetic Phenotype with"

Just "synthetically interacts with"

Thanks.  That's fixed now.


Should Fact Pages Rank Above Campaigns in Solr?

Perhaps more of a discussion, @mikefantini?

I guess we will never know.

Oh god. Sorry. When we tried to boost campaigns previously we were getting very funky results, so we took boosts off for search.


SQL connection timeout

System information:

Windows 10
DBeaver Community 6.2.1 and EE 6.2.0

Connection specification:

SQL Server 14
Microsoft JDBC Driver for SQL Server

Describe the problem you're observing:
When connecting to the server the recurrent connection from other sources raises a timeout error.
Steps to reproduce, if exist:

The SQL Server is a production server. The backend app is using the server.
After some time (not immediately) the backend app raises time out error.
When disconnecting DBeaver, automatically the backend works.

Thank you

Same issue with Mac dbeaver 6.2.3
Error: Error connecting to datasource Reason:
Connection attempt timed out.
System information:
MacOS Mojave 10.14.6
DBeaver Community 6.2.1 and EE 6.2.0
Connection specification:
PostgreSQL on AWS RDS
Describe the problem you're observing:
When connecting to the DB showing the error raises a timeout error. "Error connecting to datasource Reason:
Connection attempt timed out."
Start time: 11pm EST 10/29/2019, working fine before that
But when I use another SQL runner to run/open the DBs, it works fine, so only on DBeaver is working weird
Steps to reproduce, if exist:
The AWS RDS server is a production server.
Cannot reconnect after 10+ times trail
I have 3 different data connections on AWS RDS, none of them can connect

Please, change Keep-Alive value in connection properties to non-null.

@uslss Thank you

Thank you!!

Thank you!!


BZ1172988-Fix ill content in jboss-all.xml

Please also cherry-pick to master branch if appliable.

Already fixed on master


Stacked IndexTrackingViewModel not switching view

My bad, issue was was related to singleton creation, solved it, sorry for inconvience.

No an inconvenience. Ask whenever you're stuck. We'll respond as soon as we can.


Logging Level Adjustments

With the updated definition on logging levels (see: #378), there are a few events that are potentially issuing logs at too high of a level, alerting the DevOps team for things that aren't actionable, or at too low of a level.
Since a lot of things are changing with #329, will defer this until that work is completed.

I was wondering whether this should be included in #329. But I agree that it's better deferred until that's done, and approached as a separate issue. All new code I'm writing is (hopefully) using sensible log levels.

There aren't that many, so there isn't much issue doing it in #329, I just didn't want to do it in the middle of all the other changes

Since #329 is closed now, I presume this can be worked on?
If so, I  don't mind working through all the logging calls and making sure they match up to the definitions :)

Since #329 is closed now, I presume this can be worked on?
If so, I  don't mind working through all the logging calls and making sure they match up to the definitions :)

@ChrisLovering looks like this can be worked on. Would you like to go ahead?

I don't think this is relevant any more, we change the log levels as needed if we a sentry report come in at a wrong level.

Alright then. Thanks for the information.


cmake: Port PR25972 from the master branch

This PR ports https://github.com/bitcoin/bitcoin/pull/25972 after the recent sync/rebase PR.

-DCMAKE_CXX_FLAGS=... would have to be adjusted after APPEND_CXXFLAGS is done. Maybe postpone this PR until then or drop the last commit from this PR.

The last commit has been dropped.

I don't see how the last commit can be dropped, unless the CI here isn't equivalent to master? Otherwise it should now be failing with build errors?

I guess I don't understand why compilers are being changed to avoid warnings then? Why not just turn off Werror if the point isn't to match master? Rather than swap to a compiler we don't use?

I guess I don't understand why compilers are being changed to avoid warnings then? Why not just turn off Werror if the point isn't to match master?

Yes. This can be done.

Rather than swap to a compiler we don't use?

The number of developers, who uses Ubuntu 24.04, is expected to grow.
Anyway, all CI stuff has to be resolved in #142.


plugin breaks path prefixes

I think the url helper hack prevents the helpers from getting proper context information from the controller. Any ideas on how to solve this? Using thin with --prefix '/foo' lets you easily mount the app with a path prefix for testing.

Bump. I've just encountered this too. Only became a problem when I plugged an engine into my app (views rendered from the engine are unable to construct URLs because of this hack).

This fixes my problem. @adamaig, how about you?
https://github.com/lassebunk/gretel/pull/5


Issuer plugins Should have some order

For a freshly launched instance of Lemur, it may make sense to make 'Cryptography' the default issuer plugin to avoid confusion.
Surfaced by: #837

Any chance of an option in settings to set a default ?

Not today, but would be happy to accept a PR setting a default issuer.


Description about authentication is not clear.

It is not described how to configure the principal that has access to SharePoint. That means it is also not clear how to create ClientID etc.
The solution might be described here, but the current document does not clearly reference it.

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

ID: 8f11f0ed-eec4-dfbc-4498-87690a569091
Version Independent ID: 7c3ba7b6-5874-2cbb-0aef-aef24489d0b0
Content: Configure user authentication - Microsoft Copilot Studio
Content Source: power-virtual-agents/configuration-end-user-authentication.md
Service: copilot-studio
GitHub Login: @iaanw
Microsoft Alias: iawilt

Thanks for your comment, @ddobric. This issue has been assigned to the doc author for review.


fix(ui): Skip hideScanResults integration tests for openshift

Description

Compliance hideScanResults should hide HIPAA standard: should hide HIPAA standard
Message

Timed out retrying after 4000ms: Expected to find element: .widget:has('[data-testid="widget-header"]:contains("Passing standards across clusters")') a:contains("HIPAA"), but never found it.

Checklist

[ ] Investigated and inspected CI test results
[x] Edited integration tests

Testing Performed

/test ocp-4-12-ui-e2e-tests


Perl file containing :- misidentified as Prolog

Preliminary Steps
Please confirm you have...

[x] reviewed How Linguist Works,
[x] reviewed the Troubleshooting docs,
[x] considered implementing an override,
[x] verified an issue has not already been logged for your issue (linguist issues).

Problem Description

The file gdown.pl is being misidentified as Prolog when it is clearly Perl with a perl shebang.
Shebang:
#!/usr/bin/env perl

https://github.com/circulosmeos/gdown.pl/blob/178fa1061c858da375c4d4c2452b9ecb84789139/gdown.pl#L1
The line that I suspect causes it to be identified as Prolog (note the :- in the :-():
    die "Couldn't download the file :-(\n" if ($check==0);

https://github.com/circulosmeos/gdown.pl/blob/178fa1061c858da375c4d4c2452b9ecb84789139/gdown.pl#L57
We'll add a manual file type override for now, but this seems like an issue in the detector.
URL of the affected repository:
https://github.com/circulosmeos/gdown.pl
Last modified on:

2018-04-10
Expected language:

Perl
Detected language:

Prolog

Thanks for the neat report!
I have a pull request in the works which should solve this issue and #4090. Basically the shebang ends up not being used because it matches both Perl and Pod (both have the perl shebang registered). We could also improve our heuristic for .pl, but I'd prefer to avoid making it too complex. Unless we go down the path of explicitly excluding :- when followed by a parenthesis...

#4099 was just merged, so this issue should be fixed with the next release of Linguist. There's usually a new release every couple weeks.

Seems like a reasonable approach. Thanks!


Ability to specify crop.startSize

Currently the cropping step defaults the cropped section to 50% of the image size, but we'd like to default to 100%. This is because given our use case, most users are not going to need to crop the image. It'd be great if we could specify startSize in the configuration like we can for aspectRadio, maxSize, and minSize.
https://github.com/elninotech/uppload/blob/1fdc2379eb203de122e78fa4830c9e44bd7413d8/modules/services/crop.js#L68

In v0.3.4, you can now use the setting crop.startSize just like crop.aspectRadio:
const squareProfilePicture = new Uppload({
    value: "https://randomuser.me/api/portraits/men/17.jpg",
    crop: {
        aspectRatio: 1,
        maxSize: [1024, 1024, "px"],
        minSize: [64, 64, "px"],
        startSize: [100, 100, "%"]
    }
});

Fantastic, thanks!


"Error parsing text-format caffe.NetParameter" in Image-level-HAKE-Action

When I execute python scripts/train.py --pasta-mode linear in docker container, language_model.log is generated. And I checked this file，there are some problems:
I0916 11:37:38.907941   553 caffe.cpp:217] Using GPUs 0
I0916 11:37:38.949647   553 caffe.cpp:222] GPU 0: GeForce GTX 1080 Ti
I0916 11:37:39.380765   553 solver.cpp:51] Initializing solver from parameters:
test_iter: 2000
test_interval: 1000000
base_lr: 1e-05
display: 20
max_iter: 600000
lr_policy: "cosine"
gamma: 0.1
momentum: 0.9
weight_decay: 0.0005
stepsize: 30000
snapshot: 10000
snapshot_prefix: "snaps/language-model"
device_id: 0
net: "models/Language-Models/train_language-model.prototxt"
train_state {
  level: 0
  stage: ""
}
test_initialization: false
average_loss: 100
iter_size: 16
snapshot_format: HDF5
train_size: 38116
te: 2
multfactor: 2
tt: 2.05884722928
tenext: 2
I0916 11:37:39.381000   553 solver.cpp:94] Creating training net from net file: models/Language-Models/train_language-model.prototxt
[libprotobuf ERROR google/protobuf/text_format.cc:274] Error parsing text-format caffe.NetParameter: 3124:7: Message type "caffe.LayerParameter" has no field named "layer".
 So I want to ask what is the version of caffe for this project.
My caffe version is:
>>> caffe.__version__
'1.0.0-rc3'

Our caffe version is the same as yours.

My environment is built using docker.
The specific command is sudo nvidia-docker run -ti liangxuy/hoi-caffe

The bug is a missing right parenthesis on Line3122 of models/Language-Models/train_language-model.prototxt. We have fixed the bug and update the docker. Thank you for pointing out the problem.

Thanks for your reply! After using the latest file, I solved the problem!
But there are still some problems here:
IOError: [Errno 2] No such file or directory: './data/hico/hp_list_wts.txt' in language_model.log
IOError: [Errno 2] No such file or directory: './data/hico/hp_list_wts_hico_pvp55.txt' in 10v_attention.log
IOError: [Errno 2] No such file or directory: './data/hico/hp_list_wts.txt' in pasta_linear.log
Could you please tell me how to get these files?

hp_list_wts.txt and hp_list_wts_hico_pvp55.txt are provided under folder ./data/hico in this repo. Please update to the latest version. Thanks.

I updated to the latest version and it worked!
Thanks for your prompt reply!


Update dependency on action-utils

TL;DR
Thanks to @sethvargo 's great work in https://github.com/google-github-actions/actions-utils/pull/30, a number of issues in this repo that all boil down to "nested flags are hard" have been fixed. Thanks @sethvargo! It would be nice to release an update to this repo so that those changes can now be used.
Expected behavior
No response
Observed behavior
No response
Action YAML
N/A

Log output
No response
Additional information
Mostly this bug is a tracking bug. Thank you!

@sethvargo friendliest of pings on this 😄


style: ran yarn format

:tada: This PR is included in version 1.3.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Set up Archivematica upload for archivists

To help support born-digital cataloguing, we want to let the archivists begin to upload packages to Archivematica.
We'll need to set up appropriate access keys for use with the S3 client app they have access to.
And then work with @aray-wellcome to get them set up.

I've given the archivists some instructions and AWS access keys. I may need to do some troubleshooting, but I consider this functionally "done" once the Terraform is merged.


How to read the contents of other files?

I'm considering writing a language server for another language, but after going through the docs and example, I'm left with a question:
How should the language server read the contents of associated source files in the project that aren't opened in the editor? Should it just read them from disk itself, or should it be requesting the client for the file contents somehow; and if so, how?
#1 Seems to be related

This one is left up to language server implementors for now.  Though in the future I would imagine that any language-server-as-a-service running in the cloud would need this type of request.

Ok thanks for the info!
Well I guess my question's answered so I'll close this now.

@stkb Just for completeness: the contract between VSCode and the language server is as follows:

if a file is open the truth of a file is inside VSCode. Therefore the language server should never go to disk to read the content of the file. It should always rely on the initial content passed in the open notification and the updates in the update notification

if a file is closed the truth of the file resides (from a VSCode perspective) on disk. So the language server can read the file from disk. If the language server wants to react properly to file changes to closed files done by another tool the language server can either listen himself to file events or reuse the file events provided by the language server protocol.

@dbaeumer Thanks for the extra info. That solidifies it for me.


UWP app missing X with a circle or check mark with a circle on TakePhotoAsync method

Please fill out either the bug or feature request section and remove whatever section you are not using.
Bug
Version Number of Plugin: 2.3.0
Xamarin.Forms v<IP_ADDRESS>
PCL targeting profile 259
Device Tested On: ACER Liquid M330 running Windows 10
Simulator Tested On:
Expected Behavior
When I take a picture using the CrossMedia.Current.TakePhotoAsync method, I should be presented with a option to either cancel the photo or accept the photo. In a Windows 10 device, you get either an X with a circle around it or a check mark with a circle around it. The CrossMedia.Current.PickPhotoAsync method works just fine.
Actual Behavior
I don't see the X with a circle or the check mark with a circle. Please see attached photo.
Steps to reproduce the Behavior
var file = await CrossMedia.Current.TakePhotoAsync(new Plugin.Media.Abstractions.StoreCameraMediaOptions
                   {
                       DefaultCamera = Plugin.Media.Abstractions.CameraDevice.Front,
                       //PhotoSize = Plugin.Media.Abstractions.PhotoSize.Small,
                       //CompressionQuality = Plugin.Media.Abstractions.,
                       SaveToAlbum = true,
                       Directory = "somedirectoryname",
                       Name = string.Format("somedirectoryname_{0}", DateTime.Now.ToString("yyMMddhhmmss")),

                   });

Feature Request:
Please fill in what you would like
Screen shot showing that the options are missing,

I am calling the core APIs: https://msdn.microsoft.com/en-us/windows/uwp/audio-video-camera/capture-photos-and-video-with-cameracaptureui
https://github.com/jamesmontemagno/MediaPlugin/blob/master/src/Media.Plugin.WindowsStore/MediaImplementation.cs#L105
I can only assume a firmware bug.

@jamesmontemagno Thanks for the quick response! My OS build is 10.0.14393.67. The firmware is 1078.37.2000.0023. Do you know how I can file a bug with Microsoft? Thanks in advance!

I would test out their sample apps to see what they are doing with these apis and report a bug there if it exists. They usually have a repo on github of all the samples.

@jamesmontemagno It looks like this could be the issue. It's the same line of code that you indicated in the GitHub link you posted. I assume this reference needs to be added, so the method will resolve.
You must add a reference to assembly Windows.Foundation.FoundationContract, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null, ContentType=WindowsRuntime

I've only been developing for about a year and working with Xamarin for even less, so this is a little over my head. Any idea on how I can fix this? Any help is much appreciated. Thanks!

@jamesmontemagno The issue has been fixed. I am 99.9% sure that adding a width to get my ActivityIndicator to work in Windows 10 did the trick. Does this make sense? The spot where the ActivitiyIndicator goes contains the indicator and a status message. Now, the status message doesn't show in Windows 10 (works in iOS and Android), but the accept and reject photo selection displays. Here is the code.
<ActivityIndicator x:Name="activityIndicatorCamera" Style="{StaticResource ActivityIndicatorStyle}"
            		   IsVisible="{Binding IsBusy}"
                       IsRunning="{Binding IsBusy}"
                       HorizontalOptions="EndAndExpand" >
            <ActivityIndicator.WidthRequest>
              <OnPlatform x:TypeArguments="x:Double" iOS="50" Android="50" WinPhone="400" />
            </ActivityIndicator.WidthRequest>
</ActivityIndicator>


Get some padding in focused labels (in focus rings)

On some labels, the first and last letter is “cut’ by the focus ring (i.e. blue line around the focused item). This is especially the case for “skip to context” (before it is moved, which is another open issue). It is NOT the case for labels already in “boxes” (e.g. All Books”).
Outline-offset seems to be the proper CSS trick to use when needed.
https://css-tricks.com/almanac/properties/o/outline-offset/

Fixed by #637

@llemeurfr in some (many) cases, outline-offset introduces clipping due to the visual constraints of the containing CSS box. Thus the +2 / -2 outline pixels:
https://github.com/readium/readium-desktop/blob/cf8be49c29afea718fb3db4c78a4f4d485368065/src/renderer/assets/styles/focus-mixin.css#L4-L5
@Scarsniik 's fix introduces padding in affected GUI components (e.g. buttons), which provides additional room for the aforementioned outline technique.


filter browser logging by level of LOG

Additional fix to the next bug:
filter browser logging by level  (35965d9), closes #2228
otherwise console.log ignore doesn't work with
browserConsoleLogOptions: {
        level: 'debug',
        terminal: true
    }

I signed it!
From:  googlebot<EMAIL_ADDRESS>Reply-To:  karma-runner/karma<EMAIL_ADDRESS>Date:  Tuesday, January 17, 2017 at 3:14 PM
To:  karma-runner/karma<EMAIL_ADDRESS>Cc:  Oleg Gomozov<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject:  Re: [karma-runner/karma] filter browser logging by level of LOG (#2540)
Thanks for your pull request.  It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify.  Thanks.
If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address.  Check your existing CLA data and verify that your email is set on your git commits.
If you signed the CLA as a corporation, please let us know the company's name.
—You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/karma-runner/karma","title":"karma-runner/karma","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/karma-runner/karma"}},"updates":{"snippets":[{"icon":"PERSON","message":"@googlebot in #2540: Thanks for your pull request.  It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).\n\n:memo: Please visit \u003chttps://cla.developers.google.com/\u003e to sign.\n\nOnce you've signed, please reply here (e.g. I signed it!) and we'll verify.  Thanks.\n\n---\n\n- If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address.  Check your existing CLA data and verify that your email is set on your git commits.\n- If you signed the CLA as a corporation, please let us know the company's name.\n\n\u003c!-- need_sender_cla --\u003e"}],"action":{"name":"View Pull Request","url":"https://github.com/karma-runner/karma/pull/2540#issuecomment-273331778"}}}

sorry, I am trying to add functionality to fix integration tests, however "npm run test:integration" fails on my machine, cause non of the browsers is seen by karma while it tries to launch them. So I can't debug and understand why it is failing

@dignifiedquire ok, all fixed, except codeclimate. How to fix it? no debug information. Grunt lint doesn't show anything

Thank you, this looks good to me. @wesleycho could take a look please as well you worked on the other PR for the logging fix.

@wesleycho can you review?

@wesleycho please, review. My work is pending on this change

Thank you :octocat:


New Sample: InfoLabel

Hi Team,
I’ve submitted PR https://github.com/abap2UI5/abap2UI5-samples/pull/241 with a new demo showcasing the InfoLabel control. This example is inspired by the UI5 Demo Kit and demonstrates its usage effectively.
You can find the new sample here:

Here’s a preview:

I’d love to hear your thoughts or suggestions for any enhancements!
Best regards,
Francisco Milan

💪🏻💪🏻

Hi @francisco-milan
thanks again for the sample. Yes i think these samples are exactly what we need, making it easy for everyone to learn how to use abap2UI5 and providing abap snippets which can be easily copy&pasted. Maybe at a certain point we have the same samples as in the original UI5 sample repository and we can arrange them in a similar way or add a search function. But we'll see.

Best regards.


Add factory method to Try for Supplier interface

Hi,
I often use the Supplier interface so that I can decorate a method call and then I want to invoke the Supplier interface. It would be nice if Try would have a factory method for the Supplier interface. For example:
   Supplier<String> supplier = () -> businessService.doSomething("input");
   Supplier<String> decoratedSupplier = CircuitBreaker.decorateSupplier(circuitBreaker, supplier)
   Try.of(decoratedSupplier).get();

Instead of
   Try.of(decoratedSupplier::get).get()

The same also applies to the Callable interface.
What do you think?

I'm afraid, this is not possible - at least when using the same method name of, because it will raise ambiguities.
From the viewpoint of the compiler

a lambda () -> retVal matches both signatures Supplier, CheckedSupplier and Callable
a lambda () -> {} matches the signatures Runnable, CheckedRunnable

So only one is possible when having the same method name of.
I think it is not worth to introduce additional API, e.g.

Try.ofCallable(Callable)
Try.ofSupplier(Supplier)
...

Just calling

Try.of(supplier::get)

should be sufficient.

If you agree we should close the ticket.

Damn, that's true. Haven't thought of that, if you use a lamba directly instead of an interface.
You have to decide it. Other libraries like RxJava provide factory methods for this use case, for example:
Observable. fromCallable()

I thought a moment about removing Try.CheckedSupplier (in 3.0.0) and using Callable instead. But a CheckedSupplier throws Throwable and Callable throws Exception. It does not completely fit our needs.
We use the static methods fromXxx() to convert a value from one to another. It is the dual operation to toXxx(). Example:

Validation.fromEither()
Validation.toEither()

To to disambiguate of(CheckedSupplier) and of(Callable), the right name would be Try.ofCallable(Callable) (fromCallable does not fit because no conversion takes place).
To to disambiguate of(CheckedRunnable) and run(Runnable), the right name would be Try.runRunnable(Runnable) (which looks a bit weird).
To be congruent with Future, we need there also Future.ofCallable(Callable), Future.ofCallable(ExecutorService, Callable), Future.runRunnable(Runnable), Future.runRunnable(ExecutorService, Runnable)
If it helps I'm fine with adding the methods.

Yes, I also thought about replacing CheckedSupplier with Callable, but came to the same conclusion that Javaslang still needs Try.CheckedSupplier (or use CheckedFunction0 instead?).
What about?
Try.ofCallable(Callable callable)
Try.ofSupplier(Supplier supplier)
Try.runRunnable(Runnable runnable)

Future.ofCallable(Callable callable)
Future.ofSupplier(Supplier supplier)
Future.runRunnable(Runnable runnable)

(or use CheckedFunction0 instead?)

The Try.Checked* functions are a workaround. With Project Valhalla (primitive generics) we will remove the Try.Checked* methods and have

CheckedFunction0<R> extends CheckedFunction1<void, R>
CheckedFunction0<void> instead of Try.CheckedRunnable
CheckedFunction0<R> instead of Try.CheckedSupplier<R>
CheckedFunction1<T, boolean> instead of Try.CheckedPredicate<T>
...

For now we leave it as is.

What about?

Looks good. Additionally we need the ExecutorService param for Future:
Try.ofCallable(Callable callable)
Try.ofSupplier(Supplier supplier)
Try.runRunnable(Runnable runnable)

Future.ofCallable(Callable callable)
Future.ofSupplier(Supplier supplier)
Future.runRunnable(Runnable runnable)

Future.ofCallable(ExecutorService executorService, Callable callable)
Future.ofSupplier(ExecutorService executorService, Supplier supplier)
Future.runRunnable(ExecutorService executorService, Runnable runnable)

Shall I create a PR for Try?

Yes, would be cool!

Done


HKT as a spec and independent distribution

One problem wit the HKT pattern is that current it looks like it relies on including this library even for a small library--and its kind of obscure that way.
For example, rather than reference this project
https://github.com/devex-web-frontend/remote-data-ts/blob/261680d125176ebf021e6d209171a3e7c490897f/src/remote-data.ts#L24
This project should reference some standard endpoint. Some standardize way of naming a ADT by an author, ie 'babakness/functor'.
Maybe it could just be a standalone hkt library
so instead of this
declare module 'fp-ts/lib/HKT' {
	interface URI2HKT2<L, A> {
		RemoteData: RemoteData<L, A>;
	}
}

its
declare module 'hkt' {
	interface URI2HKT2<L, A> {
		'devex-web-frontend/RemoteData': RemoteData<L, A>;
	}
}

I'm currently working on a library and it doesn't make sense to include this entire library for just one ADT. I feel it should be small dedicated and independent library. Maybe this is something for Fantasyland or perhaps @gcanti can make that repo?
BTW, regarding a closed issue, I'm adding a fold method to my ADT for compatibility here and a reduce for compatibility with Fantasyland, but honestly it would be great if it was just a single standard.
Kudos on some great pioneering work with this library.

Yeah, I was also thinking about that... It reminds me https://github.com/benlesh/symbol-observable as a single point of interop between libs.
As a downside there would be too many repos to maintain (there already are - fp-ts, typelevel-ts, monocle-ts, io-ts etc). However there's a great solution which I'd really like to adopt here - turn this repo to monorepo (using lerna or yarn namespaces) . Also all libs could be moved to an npm namespace: @fp-ts/hkt, @fp-ts/typeclasses, @fp-ts/adt and so on. There still could be a full-blown package including everything like @fp-ts/platform.
Would really like to hear @gcanti opinion on that.

Certainly fp-ts could be split into several packages, what's the benefit though? You can import only what you need (HKT, Semigroup, Monad...) and since they are all just types your bundle doesn't bloat.

@gcanti I believe a valid reason is that types are compiled away, if I wanted to include Monad I also get a flatten function in the library.
I agree with @raveclassic, this is a great idea. My only reservation is that some of the conventions just don't match up with other standards--so maybe it is an independent standard. Yet this is a discussion separate from the simplest one--hkt as simply a common reference point.
For illustration, I have a universal filter function that takes some Filterable type; but Array does not implement partitionMap. So my function can't work on Arrays as they aren't Filterable given this definition:
https://github.com/gcanti/fp-ts/blob/c3ef83e00c068a502109680da2e1041441118dc5/src/Filterable.ts#L23
export interface Filterable<F> extends Functor<F>, Compactable<F> {
  /**
   * Partition a data structure based on an either predicate.
   */
  readonly partitionMap: <RL, RR, A>(fa: HKT<F, A>, f: (a: A) => Either<RL, RR>) => Separated<HKT<F, RL>, HKT<F, RR>>
  /**
   * Partition a data structure based on a boolean predicate.
   */
  readonly partition: <A>(fa: HKT<F, A>, p: Predicate<A>) => Separated<HKT<F, A>, HKT<F, A>>
  /**
   * Map over a data structure and filter based on an option predicate.
   */
  readonly filterMap: <A, B>(fa: HKT<F, A>, f: (a: A) => Option<B>) => HKT<F, B>
  /**
   * Filter a data structure based on a boolean predicate.
   */
  readonly filter: <A>(fa: HKT<F, A>, p: Predicate<A>) => HKT<F, A>
}

So I may define a Filterable as simply a Functor having a filter interface. I will still want to do this
declare module 'hkt' {
  export interface URI2HKT<A> {
    Filterable: Filterable<A>
  }
}

I also get a flatten function in the library

@babakness No you don't, that's the point of my previous comment: when compiling TypeScript is able to understand that you are just importing the type Monad and won't add any runtime dependency to your code

@babakness Well, I can't think of any other compelling reason to split but to keep things clean and separated. In fact everything is already "split" to separate almost independent modules and @gcanti is right, TS does not include runtime stuff if you import only types. Also I think it's a job for a bundler like webpack or rollup to do dead-code elimination.
Also as a sidenote, Array actually fully implements Filterable: https://github.com/gcanti/fp-ts/blob/c3ef83e00c068a502109680da2e1041441118dc5/src/Array.ts#L857

FYI, just wanted to link to #547, which may be relevant to this issue 🙂


chore: add 4.1.0-rc3 migration

Codecov Report

Merging #1846 (ac75e84) into main (a094ee0) will decrease coverage by 1.26%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main    #1846      +/-   ##
==========================================
- Coverage   58.13%   56.88%   -1.26%
==========================================
  Files          80       93      +13
  Lines        8213     8783     +570
==========================================
+ Hits         4775     4996     +221
- Misses       3097     3422     +325
- Partials      341      365      +24

Impacted Files
Coverage Δ

x/uibc/quota/keeper/grpc_query.go
71.42% <0.00%> (ø)

x/uibc/quota/keeper/genesis.go
100.00% <0.00%> (ø)

x/uibc/quota/keeper/quota.go
38.65% <0.00%> (ø)

x/uibc/quota/keeper/params.go
79.31% <0.00%> (ø)

x/uibc/genesis.go
55.55% <0.00%> (ø)

x/uibc/msg.go
22.22% <0.00%> (ø)

x/uibc/quota/keeper/keeper.go
100.00% <0.00%> (ø)

x/uibc/keys.go
0.00% <0.00%> (ø)

x/uibc/codec.go
46.66% <0.00%> (ø)

x/uibc/quota/keeper/relay.go
0.00% <0.00%> (ø)

... and 4 more

This should have been merged only to release/v4.1.x branch. Not to main.
ref: https://github.com/umee-network/umee/pull/1850


Sightly branding still used within tool

Sightly logo is still used within the REPL tool; Should move to official HTL logo

The old logo was removed in https://github.com/Adobe-Marketing-Cloud/aem-htl-repl/commit/57edcdd09a30bf2eeb574f06cf52412181112198.


Design collaboration guidelines

The goal of this PR is write a "Design Collaboration Guidelines" that cover the following topics:

Style guide - Foundation

Color Palette
Typography - Fonts

Layout

Sizing and Spacing
Size Classes - Responsive UI

Components - Pattern library - Atoms

Animations

Interaction Model - Patterns

Navigation
Screen transitions

Empty States

First use
User Cleared
Error Cases
Placeholder

Loading cases
Assets
User Generated Content

@pedrovgs @Serchinastico I address your comments, except "Usability: contentDescription/accessibility", that we can dedicate more time to it in the future.
I also rewrote part of it giving it a better structure and feeling to read.


Jadeite System Browser - hierarchy view should show all classes

Currently, only classes within the package are shown. Need to have the option to show all the classes in a hierarchy.

I think this was fixed some time back; classes in other packages are now shown in purple.


use aws sdk to validate regions

closes #2619
Signed-off-by: David Wu<EMAIL_ADDRESS>

LGTM

LGTM

Fixes #2619


txn: implement some async commit related logic

Try to implement these async commit protocol related features:

Update kvproto.
Return minCommitTS in Prewrite response, fetching new ts from PD.
CheckTxnStatus related change.
Return minCommitTS in lock error information.

Tests:

Unit tests.
Simple demo with TiDB changes using async commit protocol.

@coocood PTAL


Add prettier in devDependencies + scripts > format inside package.json

Since v8.4.7 the code should be formatted with prettier.
But to be able to format the code without having VScode + prettier extension, it should use the NPM dependency instead.
So it should be added in devDependencies and add the script to run it (inside package.json).

Done. Please check the PR.

https://github.com/ajax333221/isepic-chess/pull/40 fixes the issue, thanks (-: @arnab4477


Revert "[bot-automerge] satmad v0.0.6"

Reverts conda-forge/satmad-feedstock#11

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

noarch: python recipes are recommended to have a lower bound on the python version. This recommendation will become a requirement in the future.


Did-Exchange : Send Invitation and persist the state

This issue needs more grooming, how we can send invitation (which source of communication, our framework will support , email/text/qr code ) and after sending state needs to be persisted
@troyronda @llorllale

@talwinder50 Looks like this is implemented in currently. Is this issue valid now ?


Move to simd optimized omp code

I have tweaked the core logic of the M3 code to utilize openmp simd reductions inside of the openmp loops.  Ad-hoc testing shows that this gives a good speed up for column renormalization, computing matrix elements and matvec as a whole.  Moving from Cython to this c-code seems like an easy win.  The downside is that openmp must be v4 or higher, and I am not sure how much windows support there is for that.
Part of this will include going from double -> float data for the calibrations as the former has too much precision here.

This is done for the most critical pieces.


FontBlaster - Not maintained

FontBlaster framework is not maintained and it is not updated to support latest swift release. Probably it is better to move to a different framework and own implementation to easier maintenance.

Disclaimer: I am closing GH Issues on this project, I don't work full time on it and I haven't been able to update that recently if you would like to help to maintain that and to push the development of this project forward, just reach me out or open a PR and we can discuss it there.


Azure Functions on Kubernetes with KEDA feedback

Hi guys,
I have just been working on implementing this and have noticed some minor frustrations with getting it working.

From the documentation there appears to be no option on the 'func kubernetes deploy' command to specify an operating system or specifically a node pool. In my case, it was attempting to deploy a linux container onto a windows node pool in the same cluster...
I also had to deploy the container image to ACR first and use --image-name instead of --registry as docker would complain with the “Docker build Requires 1 Argument” error.
This might be something I'm missing on the Kubernetes side of things,  but is it / could it be possible to pass in a Key Vault secret reference in the local.settings.json and have it store the value in the Kubernetes Secrets?

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

ID: 2cbf4053-0a3d-a803-3e0d-8cf28db358e8
Version Independent ID: 782efd9c-beea-249c-f8e8-4ee4389bfb07
Content: Azure Functions on Kubernetes with KEDA
Content Source: articles/azure-functions/functions-kubernetes-keda.md
Service: azure-functions
GitHub Login: @eamonoreilly
Microsoft Alias: eamono

@thecmdradama
Thanks for your feedback! We will investigate and update as appropriate.


LCI parcelport: add yield to potentially inifinite retry loop

In the LCI parcelport, there are several while (resource is available, retry) loops, i.e. the nonblocking send loop and the medium buffer allocation loop. In extreme cases, if all worker threads got into those retry loops and no threads are working on releasing the resource, HPX can get into deadlocks.
This PR lets those threads yield themselves after the max retry count (32).
Preliminary experiments show that the max retry count does not have visible performance effects on actually applications' performance (tried 1-256), so we just set it to 32 arbitrarily for now.

Performance test reportHPX PerformanceComparisonBENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTORFor Each(=)??(=)InfoPropertyBeforeAfterHPX Datetime2023-05-10T12:07:53+00:002024-03-03T01:49:29+00:00HPX Commitdcb541576898d370113946ba15fb58c20c8325b20b4c93e1238de484573fd9fd36c25250960411c4Datetime2023-05-10T14:50:18.616050-05:002024-03-02T19:54:37.612255-06:00Hostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduCompiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1EnvfileClusternamerostamrostamComparisonBENCHMARKNO-EXECUTORFuture Overhead - Create Thread Hierarchical - Latch+++InfoPropertyBeforeAfterHPX Datetime2023-05-10T12:07:53+00:002024-03-03T01:49:29+00:00HPX Commitdcb541576898d370113946ba15fb58c20c8325b20b4c93e1238de484573fd9fd36c25250960411c4Datetime2023-05-10T14:52:35.047119-05:002024-03-02T19:56:25.286097-06:00Hostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduCompiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1EnvfileClusternamerostamrostamComparisonBENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATORStream Benchmark - Add(=)(=)(=)Stream Benchmark - Scale+(=)(=)Stream Benchmark - Triad(=)-(=)Stream Benchmark - Copy(=)(=)-InfoPropertyBeforeAfterHPX Datetime2023-05-10T12:07:53+00:002024-03-03T01:49:29+00:00HPX Commitdcb541576898d370113946ba15fb58c20c8325b20b4c93e1238de484573fd9fd36c25250960411c4Datetime2023-05-10T14:52:52.237641-05:002024-03-02T19:56:41.876331-06:00Hostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduCompiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1EnvfileClusternamerostamrostamExplanation of SymbolsSymbolMEANING=No performance change (confidence interval within ±1%)(=)Probably no performance change (confidence interval within ±2%)(+)/(-)Very small performance improvement/degradation (≤1%)+/-Small performance improvement/degradation (≤5%)++/--Large performance improvement/degradation (≤10%)+++/---Very large performance improvement/degradation (>10%)?Probably no change, but quite large uncertainty (confidence interval with ±5%)??Unclear result, very large uncertainty (±10%)???Something unexpected…