Unexpected type signature when ascribing value with intersection type

Issue
The following program:
from Standard.Base import all

main =
    (42 : Text & Integer)

results in a compilation error:
~/Documents/enso-projects/Issue_11481/src/Main.enso:4:6: error: Unexpected type signature.
    4 |     (42 : Text & Integer)
      |      ^~~~~~~~~~~~~~~~~~~
Aborting due to 1 errors and 0 warnings.

Expected
I would expect the program to compile successfully

sounds similar to #7841
there was a change in SignatureTest - let's add one more


AUR packages sometimes shown as missing

I know that the packages flagged as "missing" are packaged here. Yet, yay sometimes shows this, sometimes it doesn't. Any idea why?
yay -Syyuu
:: Synchronizing package databases...
 core                   160.3 KiB  1263 KiB/s 00:00 [###########################] 100%
 extra                 1829.0 KiB  11.7 MiB/s 00:00 [###########################] 100%
 community                7.5 MiB  11.9 MiB/s 00:01 [###########################] 100%
 multilib               166.4 KiB  6.25 MiB/s 00:00 [###########################] 100%
 manjaro-sway            45.0   B  1125   B/s 00:00 [###########################] 100%
:: Starting full system upgrade...
 there is nothing to do
:: Searching databases for updates...
:: Searching AUR for updates...
 -> cliphist: local (1:0.3.1-2) is newer than AUR (1:0.3.1-1)
 -> greetd: local (0.8.0-2) is newer than AUR (0.8.0-1)
 -> poweralertd: local (0.2.0-4) is newer than AUR (0.2.0-2)
 -> sworkstyle: local (1.3.0-3) is newer than AUR (1.3.0-1)
 -> Missing AUR Packages:  gtk3-nocsd  manjaro-sway-settings-git  manjaro-sway-wallpapers  oh-my-zsh  rofi-wayland  wlay  zeit

I think that meant that our package db was offline (45bytes is not really possible).

Our package repo (https://packages.manjaro-sway.download) is now served via cloudflare pages. Feedback regarding speed and availability is highly welcome!


added a more friendlier choice system

Thanks a lot ! Good work ^^.
I've to update the script to includes ddosCheck and other scripts in the repo ..
Do you use mep-git ?

yes i do

Hey mate please check map-git v2 ;)


unitary tests AVA

I have different problems to make unit tests with AVA. The project uses React 17: impossible to use Enzyme. Moreover, I can't find a way to call tsx components in AVA test functions.
I think the best thing would be to do at least the first tests in peer programming, and maybe to consider changing the initial test lib (we tried to do it a few months ago with Mireille, and to use Jest, but we had dependencies problems with Babel).
So I'm going to leave the tests aside for this project, I could do it when I'm back.

Yes I wouldn't use ava in the front end
Jest is cool, I might be able to help integrating it. There's also Cypress Component testing which I use a lot at the moment, easy to use and runs in the browser

I think we can use booth (jest + cypress) : Jest for unit tests and Cyrpress for end to end.


Alternate version of clean-rich-text-editor

Some items in text editor toolbar can be toggled on mobile:

I'm thinking about bringing this to desktop instead of hiding them:

(This screenshot is with Refined GitHub turned off)
This is not fully equivalent to current clean-rich-text-editor (only 3 hidden items overlap), but it does hide the same amount of noise (hide 5 items -> replace 6 items with 1 toggle).
    @media (min-width: 768px) {
        markdown-toolbar > .d-none.d-md-inline-block {
            display: none !important;
        }

        markdown-toolbar > .d-block.d-md-none {
            display: inline-block !important;
        }
    }

    @media (min-width: 1012px) {
        markdown-toolbar.Details--on > .Details-content--hidden {
            position: absolute;
            top: 48px;
            width: auto !important;
            z-index: 10;
            background-color: var(--color-canvas-default);
            border: 1px solid var(--color-border-default);
            border-radius: 6px;
        }

        markdown-toolbar [aria-label="Toggle text tools"] {
            padding: 4px !important;
        }
    }

😅 Works on new reviews but not review replies, the native style is already pretty broken

If this can be done with just CSS, sure, otherwise I'd rather not spend effort on this. We hide the buttons because they can be easily achieved via keyboard shortcuts

Related bug that should be fixed regardless: the feature breaks the dropdown on mobile

If this can be done with just CSS, sure

Right, the CSS I posted is a working version.

We hide the buttons because they can be easily achieved via keyboard shortcuts

I thought another reason is to make room for our other features?
The only button that doesn't have a keyboard shortcut (not counting the ones added by us) is "Add saved reply", yet we are just hiding an seemingly arbitrary selection of the buttons.
I think following GitHub here is not a bad choice.

Related bug that should be fixed regardless: the feature breaks the dropdown on mobile

I'm aware of this, it's a GitHub bug unrelated to RGH

arbitrary selection of the buttons.

The logic is "hide buttons that have obvious shortcuts" like bold italics and then other buttons that don't do anything at all (like the mention button, that just enters "@")

This is no longer actionable as the new menu is responsive. Can't simply make it appear


Wrong BiquadFilter type

The default BiquadFitler type is highshelf.
The documentation says "Frequencies higher than the frequency get a boost or an attenuation; frequencies lower than it are unchanged.", but I think a lowpass would be better, considering its description "Standard second-order resonant lowpass filter with 12dB/octave rolloff. Frequencies below the cutoff pass through; frequencies above it are attenuated.".
I might not understand these well enough, but lowpass better.

During the development phase we tried lots of different filter combinations, and indeed low pass was our first choice.
However, it didn't work as expected and we got better performance using the highshelf. However, if you decide to test it and you get better results let me know, and we can always switch..


Ga.ReciprocalFrame is nonsensical

This behavior doesn't make much sense to me:
>>> from galgebra.ga import Ga
>>> g, *_ = Ga.build('e*0|1|2')
>>> g.ReciprocalFrame(g.basis)
(1, -1, (e_1.e_2)/(e_2.e_2) + e_1^e_2/(e_2.e_2))
>>> g.ReciprocalFrame(g.basis, mode='no-norm-please')  # any string will do
((e_2.e_2)*1**2,
 -(e_2.e_2)*1**2,
 (e_1.e_2)*1**2 + 1**2*e_1^e_2,
 (e_2.e_2)*1**2)

Things which are weird here:

[ ] I was expecting to get some vector elements in this output, not a mixture of scalars and bivectors.
[x] 1 is actually Symbol('1'), not the value 1, in the second output! (fixed in #219)

Labelling as docs too, since there is no docstring

The second output certainly should be corrected. What's you exact expected output in the first one?

This issue was caused by using basis and not mv_basis

Is this completely addressed by #219 ? @eric-wieser

Nope, added a checkbox for the bullet points above. g.ReciprocalFrame(g.basis) ought to either error, which is caused by g.mv_basis[0] ^ g.basis[1] behaving poorly

Removed the docs tag, since this now has a docstring and type annotations.


Operator.compose should allow indices

resolves #1088
resolves #1143
This PR is created to verify the solution for expanding operator in #1088 (permute part of issue is not addressed yet)
The method expand for expanding the operator with identity is defined in OperatorBase,
because it enhances compose (defined for OperatorBase),
and uses tensor (defined for OperatorBase), and new method identity (abstract in OperatorBase).
identity is implemented for each subclass of 'OperatorBase',
but I am unsure of implementation for DictStateFn and VectorStateFn (instead of identity, zeros are used).
The new features are demonstrated intest_expand_on_state_fn and test_compose_op_of_different_dim
Summary
Progress for #1088 and solution for #1143
Details and comments

Thanks for the commitment @molar-volume! I think we need to clarify what #1088 should do and how we could do it. The idea was that you could pass concrete indices (not only ops of different sizes) and e.g. do
op = (X ^ 5)
op.compose( Z ^ Y, qubits=[0, 3], inplace=True)
print(op)  # ComposedOp( [X ^ X ^ X ^ X ^ X], [Z ^ I ^ I ^ Y ^ I] )

So it is closer to the same method of the quantum circuit, see QuantumCircuit.compose. (You could also add the front feature if you want! 🙂 ).
The approach in here does only allow for consecutive qubit indices (e.g. [1,2,3]) not for others (such as [0, 3]) if I understand it correctly. I have two concerns about the methods you added:

expand already exists in the quantum_info operators but has a different signature. There you can call Operator.expand(other_operator_of_same_dim), I think we should keep the same syntax if possible and therefore not call it expand here. If this turns out to be necessary we could possibly call it expand_to_dim or pad_with_identities or so.
Adding an identity method is maybe not the best approach since it is not clear to me what you would expect upon calling CircuitOp.identity? On a circuit this adds an identity gate, here it does something different. You could hide this inside pad_with_identities.

A possible approach would be to add a pad_with_identities (or whatever you want to call it) method looking like
def pad_with_identities(self, new_dim, qargs=None):
    """Extend this operator to ``new_dim`` where this operator acts on ``qargs``.

    Args:
        new_dim: The new dimension, must be larger equal than ``self.num_qubits``.
        qargs: The qubit indices this operator acts on, must be a list of length ``self.num_qubits``.
    """

And then in compose you can use the above to expand other to the correct size and compose it. What do you think?

Hi @Cryoris thank you for the feedback !
Yes, let’s rename expand to expand_to_dim.
Now identity (or we should rather name it identity_operator) is OperatorBase with primitive corresponding to identity matrix, so for CircuitOp, it equals to (I ^ num_qubits).to_circuit_op() .
I think it has some advantages, for example you can define expand_to_dim for all subclasses in OperatorBase as
def expand_to_dim(self, num_qubits: int) -> OperatorBase:
	return self.tensor(self.identity_operator(num_qubits))

I understand that we need a more general method with indices, just like you defined
def pad_with_identities(self, new_dim, qargs=None)

For the CircuitOp, that functionality can be already provided using permute (as suggested by @dongreenberg )
circ_op = (X ^ Y).to_circuit_op().permute([0,2]) @ (X ^ Y ^ Z ^ X).to_circuit_op()

print(circ_op.to_circuit())
     ┌───┐
q_0: ┤ X ├─────
     ├───┤┌───┐
q_1: ┤ Z ├┤ Y ├
     ├───┤└───┘
q_2: ┤ Y ├─────
     ├───┤┌───┐
q_3: ┤ X ├┤ X ├
     └───┘└───┘

I think I know how to implement permute for PauliOp, but do you have any suggestion how to do it for MatrixOp?

The implementation of permute for ListOps might go in line with MatrixOp.permute

Is there are good reason for the rename of identity to identity_operator? Since this is a breaking change, if we needed to do this, then we ought to be keep identity still (which would call over to the new function) but issue a deprecation warning with an appropriate message to use the method by its new name.

@woodsp-ibm identity in OperatorBase was defined by me as part of this PR, I changed it to identity_operator, because @Cryoris pointed out that it is unclear what to expect from calling CircuitOp.identity

after reading #1086 I note that expand_to_dim will be reimplemented for SummedOp and ComposedOp, because e.g SummedOp.num_qubits is supposed to return the List[int]

Ah ok, identity is a new method that was added by this PR. so you just renamed your new method. From my perspective I am not sure that the name change does so much for me since they are all operators according to OperatorBase. Maybe the method anyway should be some class or even module level function to return a specific operator of said type i.e. an identity in this case. I do not know why it would be an instance method since it has no real need of self (or am I mistaken again) - unless it were an identity spanning the same number of qubits as the operator in self in which case I could somehow see it.

@woodsp-ibm self.identity_operator is called in method expand_to_dim, which is defined in OperatorBase.
So based on the self the specific implementation of identity_operator.

Yes, I see some implementations of identity_operator do use self (when they need to copy across a property like is_measurement) while some do not. Do you see this as function that would be called in general by a user of the API - I guess you can if you want an instance that is an identity across that number of qubits - but of course with it being tied to self one needs to do that off an existing object which just seems weird. Otherwise its just a really some helper method to facilitate the one expand_to_dim implementation in the base class rather than have it implemented in each of the classes.

I agree with @woodsp-ibm, identity_operator is more a helper for expand_to_dim than a method, it doesn't implement enough logic IMO. Could you just put the code inside expand_to_dim, since its only a few lines anyways?

Right, originally I defined it as ```python
@staticmethod
def identity_operator(num_qubits: int)
but then I needed self anyway for StateFns, so I redefined it as class method. As static method, you could use it as e.g. PauliOp.identity_operator

Yeah, but I don't think its a good idea to add an identity_operator method. This seems like a very specific functionality and it doesn't add so much logic. I think it is better to don't overpopulate the Operators with too many methods 🙂 If later we find, that this is actually a much needed functionality we can still add it.

@Cryoris OK I will, I can do some optimisations then, instead of using tensor all the time.

@Cryoris @woodsp-ibm
I think this PR is ready, so if you have some comments or hints for improvement, I will do my best 👍
What was achieved:

expand_with_identity implemented for each subclass of OperatorBase
permute implemented for each subclass of OperatorBase
compose signature changed to accept indices for permutation of self and other
composing of operators with different dimension enabled (automatic expansion of the smaller operator)
minor refactoring to simplify the new features
unit tests for the new features

@molar-volume Thanks - will take a look. Sorry we have been rather busy with the release this week.

Changes based on @Cryoris 's code review:

To avoid using sympy, custom method implemented in arithmetic module.
Signature of compose changed - accepts a single list of indices for permutation of other operator.
Front feature for compose - if parameter front==True, return other.compose(self), but only after expansion and permutation is performed (consistent with Operator in quantum_info)
Reimplemented VectorStateFn.permute for better performance
expand_with_identity made private and renamed to _expand_dim (former name was misleading, because for DictStateFn and VectorStateFn, zeros are used for expansion)
_check_for_zero_composition_and_expand renamed to _expand_shorter_operator_and_permute, which is more self-explanatory (_check_for_zero_composition feature only expands the Zero operator)
added comment to explain CircuitStateFn._expand_dim
modified unit tests to test new features

Hi @Cryoris 🙂  thank you for your review and for all the comments.

I should add thanks for all your hard work and persistence to see this through. It has been a while, with a lot of discussion, so thanks again for your patience and perseverance in making this happen!

My pleasure @woodsp-ibm ! I know the discussion was necessary and helped me a lot, so thanks for all your comments. Thanks @Cryoris !


Import fails in tbl_availability_groups and tbl_availability_replicas (with latest Diagmanager version)

Here is a sample:
The affected file is Snapshot_startup.out and the affected table is tbl_availability_groups.
SQLNexus Information: 0 : RowsetImportEngine Error: An unexpected error has occurred:
System.ArgumentException: Column 'runtime' does not belong to table tbl_availability_groups.
at System.Data.DataRow.GetDataColumn(String columnName)
at RowsetImportEngine.TextRowsetImporter.InsertRow()
at RowsetImportEngine.TextRowsetImporter.ProcessFile()

The issue stems from the fact that two different scripts collect the same data  - in this case tbl_availability_group - and one collects it with runtime column (snapshot file) and the other without (miscpssdiaginfo file). Then in SQL Nexus it depends which file gets imported first – it this case it seems that miscpssdiaginfo.out got imported first and created the table without the runtime column. Then when we try to import the data with runtime column from snapshot file , it throws this.
This issue has already been filed for Diagmanager - https://github.com/microsoft/DiagManager/issues/162
But in the mean time we can take a workaround approach and create two tables in SQL Nexus to accommodate both outputs and let the import proceed.


plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled on terraform apply when updating old grants

Terraform CLI and Provider Versions
Terraform v1.5.7
on darwin_amd64

provider registry.terraform.io/hashicorp/aws v2.70.4
provider registry.terraform.io/hashicorp/random v3.5.1
provider registry.terraform.io/snowflake-labs/snowflake v0.79.1

Terraform Configuration
terraform {
  required_version = "= 1.5.7"

  required_providers {
    snowflake = {
      source  = "Snowflake-Labs/snowflake"
      version = ">= 0.79.1"
      configuration_aliases = [
        snowflake,
        snowflake.loader
      ]
    }
    aws = {
      source  = "hashicorp/aws"
      version = "~> 2.57"
    }
    random = {
      source  = "hashicorp/random"
      version = "3.5.1"
    }
  }
}

provider "aws" {
  region = "eu-west-1"
}

data "aws_iam_role" "snowflake" {
  name = "snowflake"
}

Expected Behavior
Apply to work successfully after a plan.
Actual Behavior
After updating provider versions I have attempted to replace all old object grants snowflake_table_grant for example, with snowflake_grant_privileges_to_role resources.
Initially, this was 160 additions but even after lowering it to just 50 changes I get an error during apply at a seemingly random point with the following output:
 Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ApplyResourceChange request was cancelled.
╵
╷
│ Error: Plugin did not respond
│
│ The plugin encountered an error, and failed to respond to the plugin6.(*GRPCProvider).ApplyResourceChange call. The plugin logs may contain more details.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Plugin did not respond
│
│ The plugin encountered an error, and failed to respond to the plugin6.(*GRPCProvider).ApplyResourceChange call. The plugin logs may contain more details.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Plugin did not respond
│
│ The plugin encountered an error, and failed to respond to the plugin6.(*GRPCProvider).ApplyResourceChange call. The plugin logs may contain more details.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ApplyResourceChange request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Plugin did not respond
│
│ The plugin encountered an error, and failed to respond to the plugin6.(*GRPCProvider).ApplyResourceChange call. The plugin logs may contain more details.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ApplyResourceChange request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Plugin did not respond
│
│ The plugin encountered an error, and failed to respond to the plugin6.(*GRPCProvider).ApplyResourceChange call. The plugin logs may contain more details.
╵
╷
│ Error: Plugin did not respond
│
│ The plugin encountered an error, and failed to respond to the plugin6.(*GRPCProvider).ApplyResourceChange call. The plugin logs may contain more details.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵
╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).ValidateResourceConfig request was cancelled.
╵

Stack trace from the terraform-provider-snowflake_v0.79.1 plugin:

panic: runtime error: index out of range [2] with length 2

goroutine 308 [running]:
github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/sdk.NewSchemaObjectIdentifierFromFullyQualifiedName({0xc000aa6980?, 0x29cb3dc?})
        github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/sdk/identifier_helpers.go:209 +0x45d
github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/resources.configureAccountRoleGrantPrivilegeOptions(0xc000c13840?, {0xc000faccf0, 0x1, 0x1}, 0x0?, 0xc000fd6cf0)
        github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/resources/grant_privileges_to_role.go:790 +0xf46
github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/resources.CreateGrantPrivilegesToRole(0x0?, {0x29119a0?, 0xc000c67040?})
        github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/resources/grant_privileges_to_role.go:464 +0x226
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*Resource).create(0x2c7f8e0?, {0x2c7f8e0?, 0xc001111590?}, 0xd?, {0x29119a0?, 0xc000c67040?})
        github.com/hashicorp/terraform-plugin-sdk/v2@v2.29.0/helper/schema/resource.go:766 +0x178
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*Resource).Apply(0xc0006b9ce0, {0x2c7f8e0, 0xc001111590}, 0xc000f820d0, 0xc000a9d200, {0x29119a0, 0xc000c67040})
        github.com/hashicorp/terraform-plugin-sdk/v2@v2.29.0/helper/schema/resource.go:909 +0xa7e
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*GRPCProviderServer).ApplyResourceChange(0xc000d36360, {0x2c7f8e0?, 0xc0011113b0?}, 0xc0012144b0)
        github.com/hashicorp/terraform-plugin-sdk/v2@v2.29.0/helper/schema/grpc_provider.go:1060 +0xe8d
github.com/hashicorp/terraform-plugin-mux/tf5to6server.v5tov6Server.ApplyResourceChange({{0x2c9eda0?, 0xc000d36360?}}, {0x2c7f8e0, 0xc0011113b0}, 0x0?)
        github.com/hashicorp/terraform-plugin-mux@v0.12.0/tf5to6server/tf5to6server.go:37 +0x5a
github.com/hashicorp/terraform-plugin-mux/tf6muxserver.(*muxServer).ApplyResourceChange(0x2c7f838?, {0x2c7f8e0?, 0xc001111080?}, 0xc001214460)
        github.com/hashicorp/terraform-plugin-mux@v0.12.0/tf6muxserver/mux_server_ApplyResourceChange.go:36 +0x1b5
github.com/hashicorp/terraform-plugin-go/tfprotov6/tf6server.(*server).ApplyResourceChange(0xc000c788c0, {0x2c7f8e0?, 0xc001110870?}, 0xc00063e0e0)
        github.com/hashicorp/terraform-plugin-go@v0.19.0/tfprotov6/tf6server/server.go:857 +0x574
github.com/hashicorp/terraform-plugin-go/tfprotov6/internal/tfplugin6._Provider_ApplyResourceChange_Handler({0x28b9f20?, 0xc000c788c0}, {0x2c7f8e0, 0xc001110870}, 0xc00063e070, 0x0)
        github.com/hashicorp/terraform-plugin-go@v0.19.0/tfprotov6/internal/tfplugin6/tfplugin6_grpc.pb.go:467 +0x170
google.golang.org/grpc.(*Server).processUnaryRPC(0xc0000a52c0, {0x2c8f600, 0xc000603860}, 0xc0005925a0, 0xc000dc9ad0, 0x39b56d8, 0x0)
        google.golang.org/grpc@v1.58.2/server.go:1376 +0xdd2
google.golang.org/grpc.(*Server).handleStream(0xc0000a52c0, {0x2c8f600, 0xc000603860}, 0xc0005925a0, 0x0)
        google.golang.org/grpc@v1.58.2/server.go:1753 +0xa36
google.golang.org/grpc.(*Server).serveStreams.func1.1()
        google.golang.org/grpc@v1.58.2/server.go:998 +0x98
created by google.golang.org/grpc.(*Server).serveStreams.func1
        google.golang.org/grpc@v1.58.2/server.go:996 +0x18c

Error: The terraform-provider-snowflake_v0.79.1 plugin crashed!

This is always indicative of a bug within the plugin. It would be immensely
helpful if you could report the crash with the plugin's maintainers so that it
can be fixed. The output above should help diagnose the issue

Steps to Reproduce

terraform apply

Will happen on every single apply
How much impact is this issue causing?
Medium
Logs
https://gist.github.com/amf-32/e887f8d74f04f208b10754f20763db0e
Additional Information
No response

Hey @amf-32. Thanks for reporting the issue.
Error github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/sdk.NewSchemaObjectIdentifierFromFullyQualifiedName suggests that there is a problem with the identifier with one of the resources. In most cases, this results from improper usage (consult e.g. #2253).
Can you share the snowflake_grant_privileges_to_role you have a problem with?

Hey @amf-32. Thanks for reporting the issue.
Error github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/sdk.NewSchemaObjectIdentifierFromFullyQualifiedName suggests that there is a problem with the identifier with one of the resources. In most cases, this results from improper usage (consult e.g. #2253).
Can you share the snowflake_grant_privileges_to_role you have a problem with?

yes this was the problem, one improperly formatted schema name was the cause. Thanks for the pointer.


Incorrect number of GTs and Preds used in eval.py

I was debugging the evaluation part of the code and found that total_dt_num and total_gt_num are interchanged while fetching the returns from calculate_iou_partly()
calculate_iou_partly() returns the following :
https://github.com/fnozarian/OpenPCDet/blob/c3bcae9622bc48f7f3196f947fc75346b820cbc6/pcdet/datasets/kitti/kitti_object_eval_python/eval.py#L414
However, while using these in L474, we are interchanging the positions of total_dt_num and total_gt_num
https://github.com/fnozarian/OpenPCDet/blob/c3bcae9622bc48f7f3196f947fc75346b820cbc6/pcdet/datasets/kitti/kitti_object_eval_python/eval.py#L473-L474

Oh, thanks for the confirmation !


Fix test_show_collection_number [#173205465]

This is the 1st commit in a branch intended to get CollectionNumbersControllerTest to pass.

Coverage increased (+0.03%) to 84.254% when pulling 7b6507e7bfabb6fd0b191528335a1c92409a8010 on JoeCohen:collection_nos_ctlr_test-#173205465 into 800f86278aeebfce11e28e427dcc9914ac1f8735 on MushroomObserver:bootstrap-4.

I'd like to get this merged although there are still 2 failures; I may not have serious time to code until ~Jun 16 (and I will be off the grid during much of that time).
It's an improvement, and merging it sooner will reduce the odds of conflicts.

The 2 failures are quoted below. I'm stumped by both of them.
vagrant@vagrant /vagrant/mushroom-observer (collection_nos_ctlr_test-#173205465)
$ rails test test/controllers/collection_numbers_controller_test.rb
Started with run options --seed 59822

 FAIL["test_create_redirect", #<Minitest::Reporters::Suite:0x0000562e2ca8ad08 @name="CollectionNumbersControllerTest">, 25.48611131600046]
 test_create_redirect#CollectionNumbersControllerTest (25.49s)
        Expected at least 1 element matching "form[action*='observations/432936842?q=1V']", found 0..
        Expected 0 to be >= 1.
        test/controllers/collection_numbers_controller_test.rb:213:in `test_create_redirect'

 FAIL["test_edit_redirect", #<Minitest::Reporters::Suite:0x0000562e2debeec8 @name="CollectionNumbersControllerTest">, 25.605426284000714]
 test_edit_redirect#CollectionNumbersControllerTest (25.61s)
        Expected at least 1 element matching "form[action *= 'collection_number/461709293?back=foo&q=1W']", found 0..
        Expected 0 to be >= 1.
        test/controllers/collection_numbers_controller_test.rb:342:in `test_edit_redirect'

  22/22: [====================================================================================================] 100% Time: 00:00:28, Time: 00:00:28

Finished in 28.32966s
22 tests, 217 assertions, 2 failures, 0 errors, 0 skips
Coverage report generated for Unit Tests to /vagrant/mushroom-observer/coverage. 4787 / 11317 LOC (42.3%) covered.

I'd recommend dumping the @response.body right before the assertion to see
what Rails is actually outputting.  For example, the new forms may be
putting the q=IV (query_params) into form data instead of the action URL.
Who knows?  So much has changed so subtly!
On Sun, Jun 7, 2020 at 8:39 PM Joseph D. Cohen<EMAIL_ADDRESS>wrote:

The 2 failures are quoted below. I'm stumped by both of them.
vagrant@vagrant /vagrant/mushroom-observer (collection_nos_ctlr_test-#173205465)
$ rails test test/controllers/collection_numbers_controller_test.rb
Started with run options --seed 59822
FAIL["test_create_redirect", #<Minitest::Reporters::Suite:0x0000562e2ca8ad08 @name="CollectionNumbersControllerTest">, 25.48611131600046]
test_create_redirect#CollectionNumbersControllerTest (25.49s)
Expected at least 1 element matching "form[action*='observations/432936842?q=1V']", found 0..
Expected 0 to be >= 1.
test/controllers/collection_numbers_controller_test.rb:213:in `test_create_redirect'
FAIL["test_edit_redirect", #<Minitest::Reporters::Suite:0x0000562e2debeec8 @name="CollectionNumbersControllerTest">, 25.605426284000714]
test_edit_redirect#CollectionNumbersControllerTest (25.61s)
Expected at least 1 element matching "form[action *= 'collection_number/461709293?back=foo&q=1W']", found 0..
Expected 0 to be >= 1.
test/controllers/collection_numbers_controller_test.rb:342:in `test_edit_redirect'
22/22: [====================================================================================================] 100% Time: 00:00:28, Time: 00:00:28
Finished in 28.32966s
22 tests, 217 assertions, 2 failures, 0 errors, 0 skips
Coverage report generated for Unit Tests to /vagrant/mushroom-observer/coverage. 4787 / 11317 LOC (42.3%) covered.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/570#issuecomment-640305481,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNIV6MRR7DQ4QD6CXWDRVQXLVANCNFSM4NVLFBUA
.

In test_edit_redirect, the query params are actually getting to the right place. The problem is @back_object, whichs point to an observation, instead of a collection number. I'm actually befuddled how it worked in master.
In test_create_redirect, the test assertion is not finding a form.
I'll try your suggestion in about a week -- no time until then. Right now I have to see if I can quickly fix all the conflcts.

I'd recommend dumping the @response.body right before the assertion to see what Rails is actually outputting. For example, the new forms may be putting the q=IV (query_params) into form data instead of the action URL. Who knows? So much has changed so subtly!
…
On Sun, Jun 7, 2020 at 8:39 PM Joseph D. Cohen @.> wrote: The 2 failures are quoted below. I'm stumped by both of them. @. /vagrant/mushroom-observer (collection_nos_ctlr_test-#173205465) $ rails test test/controllers/collection_numbers_controller_test.rb Started with run options --seed 59822 FAIL["test_create_redirect", #<Minitest::Reporters::Suite:0x0000562e2ca8ad08 @name="CollectionNumbersControllerTest">, 25.48611131600046] test_create_redirect#CollectionNumbersControllerTest (25.49s) Expected at least 1 element matching "form[action*='observations/432936842?q=1V']", found 0.. Expected 0 to be >= 1. test/controllers/collection_numbers_controller_test.rb:213:in test_create_redirect' FAIL["test_edit_redirect", #<Minitest::Reporters::Suite:0x0000562e2debeec8 @name="CollectionNumbersControllerTest">, 25.605426284000714] test_edit_redirect#CollectionNumbersControllerTest (25.61s) Expected at least 1 element matching "form[action *= 'collection_number/461709293?back=foo&q=1W']", found 0.. Expected 0 to be >= 1. test/controllers/collection_numbers_controller_test.rb:342:in test_edit_redirect' 22/22: [====================================================================================================] 100% Time: 00:00:28, Time: 00:00:28 Finished in 28.32966s 22 tests, 217 assertions, 2 failures, 0 errors, 0 skips Coverage report generated for Unit Tests to /vagrant/mushroom-observer/coverage. 4787 / 11317 LOC (42.3%) covered. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#570 (comment)>, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNIV6MRR7DQ4QD6CXWDRVQXLVANCNFSM4NVLFBUA .

Please ignore this PR for now; I'm converting it back to a draft.
There are too many conflicts for me to deal with tonite. (And maybe resolving them will help with the tests.)

I am closing this PR. It is replaced by Draft PR #571

I finally figured out that the thing that was hangin me up was:
<% render "form" ... which should have been
<%= render "form" ...
Fixing that (in #571) revealed the other problems (also now fixed in #571).

I'd recommend dumping the @response.body right before the assertion to see what Rails is actually outputting. For example, the new forms may be putting the q=IV (query_params) into form data instead of the action URL. Who knows? So much has changed so subtly!
…
On Sun, Jun 7, 2020 at 8:39 PM Joseph D. Cohen @.> wrote: The 2 failures are quoted below. I'm stumped by both of them. @. /vagrant/mushroom-observer (collection_nos_ctlr_test-#173205465) $ rails test test/controllers/collection_numbers_controller_test.rb Started with run options --seed 59822 FAIL["test_create_redirect", #<Minitest::Reporters::Suite:0x0000562e2ca8ad08 @name="CollectionNumbersControllerTest">, 25.48611131600046] test_create_redirect#CollectionNumbersControllerTest (25.49s) Expected at least 1 element matching "form[action*='observations/432936842?q=1V']", found 0.. Expected 0 to be >= 1. test/controllers/collection_numbers_controller_test.rb:213:in test_create_redirect' FAIL["test_edit_redirect", #<Minitest::Reporters::Suite:0x0000562e2debeec8 @name="CollectionNumbersControllerTest">, 25.605426284000714] test_edit_redirect#CollectionNumbersControllerTest (25.61s) Expected at least 1 element matching "form[action *= 'collection_number/461709293?back=foo&q=1W']", found 0.. Expected 0 to be >= 1. test/controllers/collection_numbers_controller_test.rb:342:in test_edit_redirect' 22/22: [====================================================================================================] 100% Time: 00:00:28, Time: 00:00:28 Finished in 28.32966s 22 tests, 217 assertions, 2 failures, 0 errors, 0 skips Coverage report generated for Unit Tests to /vagrant/mushroom-observer/coverage. 4787 / 11317 LOC (42.3%) covered. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#570 (comment)>, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNIV6MRR7DQ4QD6CXWDRVQXLVANCNFSM4NVLFBUA .


Add support for Typescript (tsx)

Description
Add Support for TypescriptReact (.tsx)
Greetings :-)

duplicated, see #601

Merged https://github.com/Glavin001/atom-beautify/pull/2021
Coming in next release.

Published to v0.32.0


[SPARK-22569] [SQL] Clean usage of addMutableState and splitExpressions

What changes were proposed in this pull request?
This PR is to clean the usage of addMutableState and splitExpressions
How was this patch tested?
The existing test cases

cc @cloud-fan @kiszk

Test build #84045 has started for PR 19790 at commit a0272d9.

Test build #84045 has finished for PR 19790 at commit a0272d9.

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

Merged build finished. Test FAILed.

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

LGTM, can you improve the PR description to describe the major change? e.g.
1. replace hardcoded type string to ctx.JAVA_BOOLEAN etc.
2. create a default value of the initCode for ctx.addMutableStats

Test build #84057 has started for PR 19790 at commit a5ee80b.

Test build #84057 has finished for PR 19790 at commit a5ee80b.

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

Merged build finished. Test PASSed.

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

thanks, merging to master!


Rename reviewRequests to requestsForMyReview

Getting ready for https://github.com/artie-shevchenko/pr-review-your-turn/issues/26

Migration in the middle of a random function is bad.
I really don't like that we have to migrate on each variable rename. It could turn into migration hell very soon. Only solution I could see for now is to create DTO classes for storage only and don't change it as much as possible. I wanted to do this as a second part of #10 .
If we had to rename something to dto, write down the dto version in the storage. Then use the migrate function that will know how to transform from current version to the next. Then just transform from current version to the next one till it reaches current "main" version.
Perhaps this migration algorithm is already implemented in some library?


[outreachy] Registry: add filters for search

Outreachy task for @taylorhelene
Based on your user experience report, please provide the following suggested improvement via a PR:
Improve/Add the filtering capabilities of the search bar. Make suggestions how an end user can add filters to their search, including "deprecated", "native", "first party", "license". Also include the existing filters in your considerations.
Note: This is an assigned issue for a participant that has completed the user experience report, if you are an outreachy participant as well, find instructions here: #5290

I have added a flags filter. Here is my pr: https://github.com/open-telemetry/opentelemetry.io/pull/5328. Please review it and add any more details and tests I may have to remove or add.


[LibSwiftScan] Add entry-points for querying protocol conformances from the static mirror

This PR adds new entry-points to libSwiftScan that operate on the new BinaryScanningTool, which reads out Swift type information from object files, starting with a query of all protocol conformances.

@swift-ci please test

@swift-ci please test


Request no-named-as-default rule options which allows Re-Export default syntax

Hello! 👋
Since import/no-named-as-default rule supports post-ES2015 import syntax, it would be nice if it has some option that allows Re-Export default as default syntax.
For instance:
Component/Component.js:
export {
  Component as default,
};

Component/index.js:
export default from './Component'; // not considered problem

How about it?

What is the problem with it, currently? Can you provide a failing example? (Could be same code, but with the reported message?)

@benmosher Yes. currently, above example is reported:

Using exported name 'default' as identifier for default export.

That's wild. Definitely sounds like a bug. Will take a look.


Update getting started doc to describe basic inner loop workflow

This doesn't necessarily need to do much more than cleanup the:
https://github.com/OpenLiberty/application-stack/blob/master/README.md
which is a bit out-of-date (e.g. the odo url create is no longer required on Open Shift).
Bonus points to have a generic k8s (vs. OCP) section.. or else just make clear the doc is OCP only.
If we add the Maven volume from: https://github.com/OpenLiberty/application-stack/issues/47 that'd probably be worth explaining.
The "debug" probably is worth a quick mention (maybe a separate https://github.com/OpenLiberty/application-stack/issues/59 isn't even needed).
SAMPLE VS STARTER?
We might be able to use the starter as-is or may need to extend via a "sample"..not sure yet.

We probably need a reference somewhere to more basic odo and/or devfile doc as well.

We can call this completed in: https://github.com/OpenLiberty/application-stack/pull/66


Performance very slow on Windows 7

Expected Behaviour (for problems)
No lag in data entry.
Reasonable speed in opening data or guessing column properties
Current Behaviour
Significantly slower than 0.36
Your Environment

Data Curator version 0.4.0
Operating System Windows 7

Not sure there is too much we can do about this specifically for Windows 7. Any chance this is a 32bit machine (as electron with latest chromium known not to play nice with anything other than 64 bit)?


Improve performance for classical shadows method

Classical shadows algorithm performance can be greatly improved by doing appropriate parallelization of the loop for shadow computation and in other places within the code (see comments throughout the code).
UPDATE 06/11/23: let's use this issue for testing also the performance of the bitstring corruption routines which might be quite inefficient. Up to how many qubits can we go?

Part of Ethan internship ?

@mlahariya This will be revisited in protocols. I also have #608 open that fixes an issue with the current shadow implementation and improves a bit the runtime.


Add analysis checker within analyses data entry point

This should be the way to check if an analysis should be visible or not. And we have move all the parameters related with analysis to the same point, the 'data/analyses' file. There we will have all the info we need to start working with any analysis in any place, form, add-analysis modal, onboarding, animations,...
Also, due to the fact that we have several forms where we use more than one analysis, we have added some checkers in:
Area-of-influence: time option shouldn't be enabled if there is no routing provider.
Geocoding: only lat-lon option should be available when there is no geocoder provider.

CR: @javierarce (when tests turns into green).
Also, I'll need help in order to test at length those changes.
@javierarce and @piensaenpixel can I count with you? 🙏  (maybe?).
Fixes https://github.com/CartoDB/cartodb/issues/10771.
cc @rporres @javisantana @santisaez @saleiva

So adding @CartoDB/dataservices team in order to check if we are validating "available" analyses properly. Also, those analyses we are checking are:

data-observatory-measure: by custom install flag.
georeference-city: by custom install flag.
georeference-ip-address: by custom install flag.
georeference-country: by custom install flag.
georeference-postal-code: by custom install flag.
georeference-street-address: by geocoder_provider.
georeference-admin-region: by custom install flag.
routing-sequential: by routing_provider.
routing-to-layer-all-to-all: by routing_provider.
routing-to-single-point: by routing_provider.
trade-area: by isolines_provider.

These cases that rely on service providers:

georeference-street-address: by geocoder_provider.
trade-area: by isolines_provider.
routing-sequential: by routing_provider.
routing-to-layer-all-to-all: by routing_provider.
routing-to-single-point: by routing_provider.

The problem is that now we can have default providers so. Using the quota has a similar problem because internal services can be used without a quota.
We should probably remove those defaults, at least for quota have non zero quotas for all usable services, but right now we better use another way of disabling services. A (hardcoded) configuration flag could be added for on-premises/saas/...
@ethervoid @rafatower will have a more informed opinion, please comment on this

This config should be enough, what do you think @xavijam ?

That's ❤

Deployed. Merge master and enjoy :)

I'm deploying changes in ded02. So we should check that:

Available analyses appear, those not available should not appear.
Analyses animations appear in the proper analysis and work correctly in the add-analysis-modal.
Onboarding messages appear correctly after an analysis is applied.

cc @javierarce @piensaenpixel

Ooooooook, deployed again in ded02 after some small problems :).

Merging when green.

Thanks @javierarce for your help.


Steam Deck OLED Frame Cap Broken Below 90 hz

Describe the bug
When I set the frame cap in Gaming Mode to anything below 90 hz, the game seemed to be stuck at 40 - 45 hz no matter what I did (it did stay at 90 hz when I set the frame cap to 90 hz).
What did you expect to happen?
I expected the frame cap to apply appropriately (for example, setting it to 60 hz should cap the frame rate to 60 hz).
Output of rpm-ostree status
State: idle
Deployments:
  ostree-image-signed:docker://ghcr.io/ublue-os/bazzite-deck:stable
                   Digest: sha256:f9070c349af242e5d47ed9a7a21418c42694d71c5fa0e5567b3c92d5036d9ca2
                  Version: 40.20240728.0 (2024-07-28T16:39:55Z)
            SecAdvisories: 1 moderate
                     Diff: 247 upgraded, 1 downgraded, 23 removed, 23 added
          LayeredPackages: sunshine

● ostree-image-signed:docker://ghcr.io/ublue-os/bazzite-deck:stable
                   Digest: sha256:56e6286b623e843164af43e1bdc951bb365e8a9bb9c444e0f8130c346bef2f5e
                  Version: 40.20240722.0 (2024-07-22T14:32:51Z)
          LayeredPackages: sunshine

  ostree-image-signed:docker://ghcr.io/ublue-os/bazzite-deck:stable
                   Digest: sha256:82624ab8dc34f83515f74cdaa160683f342cc62b8dd2d485fa2d465385ec5fab
                  Version: 40.20240717.0 (2024-07-17T17:10:50Z)
          LayeredPackages: sunshine

Hardware
Steam Deck OLED
Extra information or context
Edit: After updating Bazzite earlier today (after posting this bug report), this seems to be occurring much less often, but it still seems to be happening occasionally.
Edit 2: It seems like this issue is very similar to mine, although this person has it happening with an ROG Ally X.

Hi. I have just tried to play a game today at 40, 45, and 50 hz. None of these caused the issue, therefore, I think I'm going to close this issue for now as solved.


Add cycle plot series

Not sure how appropriate it is in the series package but it was the closest match I could find.
Demonstrates a workaround for https://github.com/ScottLogic/d3-financial-components/issues/400.
For reference, this is what a cycle plot looks like and here's one @godds did in d3.

I've been playing around with this and really like it. However, it took a while to get an example up and running, quite a few scales and value accessors to get right!
This series will benefit from a clear and simple example on the website.
I re-implemented Graham's example using the cycle series, which went pretty well:
d3.json('data.json', function(error, data) {
    var parseDate = d3.time.format('%Y%m%d').parse;

    // parse the data / coerce types
    data.forEach(function(d) {
      d.date = parseDate(d.date);
      d.itemsSold = +d.items_sold;
    });

    var dayScale = d3.scale.linear()
        .domain([-0.5, 6.5])
        .range([0, width]);

    var salesScale = d3.scale.linear()
        .domain(fc.utilities.extent(data, 'itemsSold'))
        .range([height, 0])
        .nice();

    var subScale = d3.scale.linear()
        .domain(fc.utilities.extent(data, 'date'));

    var lineSeries = fc.series.line()
        .xValue(function(d) { return d.date; })
        .yValue(function(d) { return d.itemsSold; });

    var cycle = fc.series.cycle()
        .barWidth(fc.utilities.fractionalBarWidth(0.9))
        .yScale(salesScale)
        .xScale(dayScale)
        .xValue(function(d) { return d.date.getDay(); })
        .subScale(subScale)
        .subSeries(lineSeries);

    container.append('g')
        .datum(data)
        .call(cycle);
});

Although as soon as I added the 'baseline', the average price for each day of the week, it became very ugly! It is a real pain having to repeat the same nesting that the cycle series performs.
This functionality could be baked in to the cycle series, or perhaps added as a form of axis?

Yea I agree, doing the algorithmic work outside it starts to feel a bit messy like the stacked bar. I toyed with the idea of using an axis too, but it felt wrong to force it into the multi. Thinking about it again though, cycle could explicitly accept an axis e.g. subAxis?

I'm trying it it the other way around, an axis that accepts a cycle series. I'll see how it turns out.

I've opted to create an adapter for the axis which allows it to be used as a series. This is then added to the multi series used as the cycle plot's subSeries.
This approach seems to offer the most potential for re-use whilst cleaning up the visual test.
What process are we following for adding stuff to the website? It would be good to have a new build containing the functionality before attempting the example...

Much better :+1:


Rotation not allowed

Hi,
This library  is good. I am using it in one of my projects.
But , the Calculator Activity doesnt allow rotation. Can you  fix it?
Also, allow the calculator to open in a fragment with a listener so that it can send back the result via the listener instead of setResult. It is useful when developer doesn twant extra activities to be added in project
thanks.

we'll work on rotation and fragment support.
thanks for you feedback.

fragment support added.

'com.github.gperez88:calculatorInputView:1.0.3'

    @Override
    protected void onViewCreated(View view, Bundle savedInstanceState) {
        super.onViewCreated(view, savedInstanceState);

        editText = (EditText) findViewById(R.id.editText);
        editText.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                new CalculatorBuilder()
                    .withTitle(TITLE)
                    .withValue(editText.getText().toString())
                    .start(MainFragment.this);
            }
        });
    }

    @Override
    protected void onActivityResult(int requestCode, int resultCode, Intent data) {
        super.onActivityResult(requestCode, resultCode, data);

        if (resultCode == CalculatorActivity.REQUEST_RESULT_SUCCESSFUL) {
            String result = data.getStringExtra(CalculatorActivity.RESULT);
            editText.setText(result);
        }
    }


cmd/go: improve support for using modules in offline mode

I was on a plane earlier this week, and not being able to use go get was unfortunate. Any queries like go get some-dependency@latest or @master always reach for the internet, failing immediately. Something akin to go get some-dependency@latest-cached would have been neat, because that dependency was many months out of date, and my cache had multiple newer tagged versions.
I ended up looking directly into my module download cache, finding the latest tagged version there, then using go get<EMAIL_ADDRESS>Which worked, but it's also somewhat tedious, and I don't like having to peek at the local cache in this way to be able to use it.
I believe that we've talked about better offline support before, but I could not find a tracking issue for it.
cc @bcmills @matloob

Aha! I found the issue I thought I remembered for this.

Duplicate of #43646

Ah, thanks!


Why use Cognito User Pool at all? Shouldn't we use Federated Identity Pool?

I've been having trouble with Authorization (API Gateway) because of the password field. Diving a little deeper I've looked into the COGNITO_IDENTITY_POOL_ID variables that were added.
Since this boilerplate doesn't include User+Pass auth, is there any reason why we don't use Cognito Federated Identity Pools (as they seem to be more fitting for oauth providers)?

@kidsil I had developer authenticated cognito provider example before I refactored it to save user info to cognito user pool, https://github.com/laardee/serverless-authentication-boilerplate/commit/d9ace9918c6c0cb3a43e65925f6e206d59949cc2#diff-32d3eb1dd8b96b3bdae61b4a635a09e3L30. So, then this needs to be reverted and integrated with user pool.

I see, may I ask what was the reason of switching to User Pool? It seems that if we're doing Social Oauth Identity Pool is the way to go... I must be missing something..?

(As I remember) the initial plan was to save the user info to user pool and then integrate it to federated identities, something like this http://docs.aws.amazon.com/cognito/latest/developerguide/amazon-cognito-integrating-user-pools-with-identity-pools.html

My first thought was that this is in fact a replacement for cognito. Is this authentication method only an alternative to Cognito or it has another advantage?

@rcfrias, yes, this boilerplate is an alternative for Cognito in some sense. The idea behind this boilerplate was that it would be a serverless passportjs equivalent - API based server-side authentication that requires no client-side js libraries.
Maybe the main advantages are that no extra libraries in the client are needed and you don't need to use Cognito User Pool for user storage if you don't want or already have an existing user database. And hopefully, in the future, I can support Microsoft Azure and other cloud vendors too.
What comes to this issue, @kidsil and I decided to drop the Federated Identity and try if Cognito User Pool can keep the "session" in server-side when it is used as user storage.


Generics support in sealed

What would this look like? Is there anything we need to do? I'm not actually sure if PolymorphicJsonAdapter supports them either
sealed class Message<T> {
  data class Success<T>(val data: T)
  data class Failure(val reason: String)
}

This isn't really possible

@ZacSweers I facing exactly this issue right now.
There is no way to support this?

There is not. PRs welcome but I could never figure out a way that made sense


ci: codecov

This PR

enables codecov action block

Related Issues
Fixes #18
Notes
Still needs a secret token set up for Codecov
Follow-up Tasks
How to test
Run the CI, Codecov upload should be successful if configured correctly

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@9afef9e). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #52   +/-   ##
=======================================
  Coverage        ?   84.17%
  Complexity      ?      221
=======================================
  Files           ?       39
  Lines           ?      575
  Branches        ?        0
=======================================
  Hits            ?      484
  Misses          ?       91
  Partials        ?        0

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


"shell project" POC

As a: design system team
I want: Know if service teams would get significant value out of having a "shell project" from which to start from.
so that: We can understand if/when the team should be putting time and effort into this endeavour.
Acceptance criteria

[ ] Create a POC for a shell project (in code or Figma, dev and designer to work together to make this decision)
[ ] Interview 3-5 developers to understand if a shell project would be useful for them, try to understand, why or why not.
[ ] Summarize findings and deliver the findings back to the team for further discussion and decision making.

Resources:
Mockups: [If applicable a URL to or the name of the mockup(s)]
Testing URL: [If applicable a URL to the testing branch]

Review the nx templated generators as an approach for the shell project: https://github.com/GovAlta/nx-tools


Apache?

Is it possible run Laradock with Apache2 instead Nginx? I'm asking because here on Github page is mention there there are 3 web servers including Apache. But on http://laradock.github.io/laradock/ which is Laradoc information page there are no word about Apache2 at all.

Oh thanks for the notice we do actually support Apache2 but that pages wasn't updated. I will update it right now. Thanks again


[Example] Pin nextjs example to react 15

NextJS latest does not support React 16 so pin to React 15.

While the beta of NextJS supports React 16 having both material-ui beta and nextjs beta in the example seems like a good recipe for the example to break regularly so better to pin to NextJS@latest and supported versions of React rather than NextJS@beta and React 16.

@eyn Sounds good 👍 .

@eyn Thanks, I can't wait for being able to upgrade the internal dependencies to react@16!


Fix: dtype of agentid wrapper.

What?
Fixes #1032 :)
Why?
Dtype was not enforced to be consistent, caused errors.
How?
Enforced dtype in concatenation operation.

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


fix(deps): allow protobuf 3.19.5

There is a known issue with samples in this repo: https://github.com/googleapis/python-documentai/issues/387


Unable to work out how to change options

Can you please put an example in the README? I have tried to Google to no avail, Thanks!!

I can see it, but then how do I change it? Apologies if I'm missing something obvious

Open the chat and click on the setting with your mouse, then it should put a command in your chat area.
There you can add the number you want to set it to and then you just sent the command.

OHHH, thanks!!! Thats the bit I didn't know! Thanks!


Fix URL in link to sass-lint

The link in the readme to sass-lint was pointing to a similar but different tool, gulp-scss-lint. This commit fixes the link so that it points to the tool we are actually using according to the package maniftests.

You can preview this change here:
Built with commit b106c320049ab95d8ae149033c99adf5e3b75762
https://deploy-preview-1662--govuk-design-system-preview.netlify.app


Python: Azure AI Inference tracing SDK

Motivation and Context

Description

Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

Python Unit Test Overview

Tests
Skipped
Failures
Errors
Time

2867
4 :zzz:
0 :x:
0 :fire:
1m 8s :stopwatch:


setuptools or numpy.distutils for the setup.py file?

We should check the differences between using setuptools and numpy.distutils to configure the setup.py file. Which one should we use?

The Python Packaging User Guide (https://packaging.python.org/guides/tool-recommendations/) recommends now:
Use setuptools to define projects and create Source Distributions.
And explains:
"Although you can use pure distutils for many projects, it does not support defining dependencies on other projects and is missing several convenience utilities for automatically populating package metadata correctly that are provided by setuptools. Being outside the standard library, setuptools also offers a more consistent feature set across different versions of Python, and (unlike distutils), setuptools will be updated to produce the upcoming “Metadata 2.0” standard formats on all supported versions.
Even for projects that do choose to use distutils, when pip installs such projects directly from source (rather than installing from a prebuilt wheel file), it will actually build your project using setuptools instead."
But many packages still use distutils and those that support setuptools often mix setuptools with distutils.

Hmm, we should then work with both. Let's keep it this way as long as the setup.py file is functional.

This issue is solved if the MOLSSI cookie-cutter is used. See uibcdf/OpenPharmacophore#4

Duplicate of uibcdf/OpenPharmacophore#4


Gradle SBOM Issue

cdxgen doesn't seems to work for Gradle projects.
Tried all these for gradle projects:
cdxgen -t java -r --resolve-class
cdxgen -t java -r
cdxgen -t java

execution log:
Executing gradlew dependencies in /
null null
Unable to produce BOM for .

Environment:
OS: MacOS
cdxgen version: 3.1.0 (also tried in 3.0.9 resulted the same)
java version: openjdk 11.0.12 2021-07-20 LTS
npm version: 7.13.0
node version: 16.5.0

@navulirs Can you execute the below gradle command directly and let me know if it works correctly?
gradlew dependencies

Hello @Prabhu, I had same issue for Gradle project. Tried gradlew deppendencies

Welcome to Gradle 7.0.2!                                                                                Here are the highlights of this release:

File system watching enabled by default
Support for running with and building Java 16 projects
Native support for Apple Silicon processors
Dependency catalog feature preview

For more details see https://docs.gradle.org/7.0.2/release-notes.html                                                                                Starting a Gradle Daemon (subsequent builds will be faster)                                                                                FAILURE: Build failed with an exception.                                                                                * What went wrong:
Unable to start the daemon process.
This problem might be caused by incorrect configuration of the daemon.
For example, an unrecognized jvm option is used.
Please refer to the User Manual chapter on the daemon at https://docs.gradle.org/7.0.2/userguide/gradle_daemon.html     Process command line: C:\Program Files (x86)\Java\jre1.8.0_291\bin\java.exe -Xmx4096m -Dfile.encoding=windows-1252 -Duser.country=IN -Duser.language=en -Duser.variant -cp C:\Users\msrik.gradle\wrapper\dists\gradle-7.0.2-all\7era6s5ay7zsbhuvl0oc9g94s\gradle-7.0.2\lib\gradle-launcher-7.0.2.jar org.gradle.launcher.daemon.bootstrap.GradleDaemon 7.0.2
Please read the following process output to find out more:
Invalid maximum heap size: -Xmx4096m
The specified size exceeds the maximum representable size.
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit.                                                                                * Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.                                                                                * Get more help at https://help.gradle.org

Can you please let me know, whats the best approach to generate BOM for similar Java/Gradle projects

@Masireddy65 Noticed a typo in gradlew dependencies There were 2 P.
This particular error sounds like related JVM version. Not sure if gradle 7 supports java 8.
Could you use the scan docker image if you aren't in a position to change the jvm version in your machine?
docker run --rm -e "WORKSPACE=${PWD}" -v "$PWD:/app" shiftleft/scan scan -t depscan

This would produce a bom as well as perform dependency scan.

Thanks for quick response @prabhu .

On Windows, the command changes slightly depending on the terminal.
cmd
docker run --rm -e "WORKSPACE=%cd%" -e "GITHUB_TOKEN=%GITHUB_TOKEN%" -v "%cd%:/app:cached" shiftleft/scan scan

I only found above command for windows cmd with github. However,I need to run a test on local directory of windows machine or may be on bitbucket pipeline. can you please help me tailer the command to my need here?

@Masireddy65 You can find bitbucket pipeline instruction here - https://slscan.io/en/latest/integrations/bitbucket/

Thanks @prabhu. Now, I have both bitbucket pipeline instruction and windows cmd docker run. Trying both only does SAST SCAN, Secrets audit etc. No dependency scan even with -depscan argument and obviously no BOM file saved at /reports.
Also,In terms of bitbucket i tried Docker version and received error in final step to find the file.
Windows cmd  Attempts:
docker run --rm -v "%cd%:/app:cached" shiftleft/scan scan --type depscan
docker run --rm -v "%cd%:/app:cached" shiftleft/scan scan --type depscan,bom
Please find attached screenshots for all attempts and guide me accordingly. Appreciate your prompt response so far.

@Masireddy65 Could you kindly email me prabhu @ appthreat dot com so that we could do a separate troubleshooting session for you? It may or may not be related to this particular ticket. You could also rerun by setting the environment variable
SCAN_DEBUG_MODE=debug to see if there are any additional information.

@ruckc elasticsearch repo appears to be requiring java 16. With java 16, gradlew dependencies is not returning the full tree for me so not sure how to make this work
 /mnt/work/sandbox/elasticsearch  master  ./gradlew dependencies                                                                                1 err  25m 37s
=======================================
Elasticsearch Build Hamster says Hello!
  Gradle Version        : 7.2
  OS Info               : Linux 5.13.13-051313-generic (amd64)
  JDK Version           : 16.0.2 (GraalVM Community)
  JAVA_HOME             : /home/prabhu/.sdkman/candidates/java/21.2.0.r16-grl
  Random Testing Seed   : 956BF8F9EB026B6A
  In FIPS 140 mode      : false
=======================================

> Task :dependencies

------------------------------------------------------------
Root project 'elasticsearch'
------------------------------------------------------------

es_distro_extracted_local
No dependencies

es_distro_file_local
No dependencies

jdk_provisioned_runtime
\--- adoptium_17:linux:17

A web-based, searchable dependency report is available by adding the --scan option.

BUILD SUCCESSFUL in 4s
20 actionable tasks: 1 executed, 19 up-to-date

Any ideas?

@navulirs Can you execute the below gradle command directly and let me know if it works correctly?
gradlew dependencies

Hi, Sorry for delayed response. It is working now. it has to do with the gradle version. Thanks

Why did I report this error when analyzing the Grale project

@wujunhuge this must be due to the presence of space and brackets in the directory name. We have an existing issue like this for windows. Could you rename that project directory and try?

Thank you for your reply. I'll give it a try


Google fit dependencies- fixed

[x]  run npm install react-native-google-fit --save
[x] run react-native link react-native-google-fit

make sure the following files have the googleFit dependencies

MainApplication.java
android/settings.gradle
package.json
and
AndroidManifest.xml contains the ACTIVITY_RECOGNITION permission

Assigning back to @crugwiro to fix identified issues - stuck on loading after logging in.

Assigning back to @crugwiro to fix identified issues - stuck on loading after logging in.

No longer needed


RFC: VM 2.0

Replaces four abstractions of the existing Neon API (Scope, Arguments, FunctionCall, and the vm module) with one Vm trait, representing a (context-sensitive) view of the JavaScript virtual machine.
Rendered
WIP Implementation

📢 This RFC is going into final comment period! We will plan to merge it within a week if there aren't significant new issues that arise in the next week. 📢


How to prevent Comment Reformatting (Alt-Q) from flattening code samples?

If I have a comment block like so:
/**
 * Use this function like so:
 *
 * <code>
 *     foo({ baz: "bat" }).then(function(){});
 * </code>
 *
 * @param {object=} options - Options.
 */
function foo( options ){}

Then reformat using alt-Q, my example code is flattened:
/**
 * Use this function like so:
 *
 * <code> foo({ baz: "bat" }).then(function(){}); </code>
 * @param {object=} options - Options.
 */

I haven't figured out how to format my example code so that the reformatter will skip it. Is that possible?

Not as far as i'm aware, sorry.


Error in readMe file

TaskCommand(...) mentioned here https://github.com/Codegyre/Robo/blame/master/README.md#L76
do not exists

We should probably mention taskCodecept() instead, but not sure

Actually it was about running Symfony Commands. Sorry that was not obvious

Fixed.


Undefined behavior when dividing -0x80000000/-1

See https://github.com/iftechfoundation/ifarchive-if-specs/issues/14 . Same fix as https://github.com/DavidKinder/Git/issues/24 .

Fixed https://github.com/erkyrath/glulxe/commit/633c4d6d104afbb2a3e477d9c5ebc184f9ba95fc .


It is not possible to create account if Identity attributes contain special characters

To reproduce:

Create identity with name that contains one of Danish characters æ ø å.
Try to create account with that identity.

Result:
Runtime error.
Expected:
It should be possible to create account.
Workaround:
There is no workaround in BW, one could recover in CryptoX and then create account.
Note:
This issue seems to be introduced with the latest release 1.6.0 as I was not able to reproduce it in 1.5.2.
Same goes for characters in German like Ü.

This is caused by a bug in @concordium/web-sdkPR with fix can be found hereWhen this fix is released in @concordium/web-sdk, a patch will be released for the browser wallet.


Split scan execute

This PR let's us better model the stages of a scan, so we can be used by an engine that doesn't just want the kernel to do all the reading in whatever thread called Scan::execute.
We introduce a method in Scan that can return EngineData with a known schema and a selection vector. That data will have all the relevant info for performing the scan. The engine can serialize/use that data however it likes to distribute work.
Currently also includes a GlobalScanState which is serializable. This is a bit of a shortcut for now, and we may want to generalize it more to not require an engine to use our serialization at all, but in the interest of time I'd like to keep this unless there are serious objections.
DuckDB notes:

They have a capable parquet reader, so we want to ensure they can get things like partition column info before they read the file, because their reader can insert that info at read time.
Similar for DVs

The approach being set up looks reasonable, but I feel like I'm missing something to fully "close the loop" for an external engine? I guess it needs the corresponding engine code to actually implement an engine_visitor function and pull the information out of the engine data?

Did you see the code in the example here

This was all included in #175


Behaviour of selection to next find match using command

Command CTRL+D is not giving as output as expected.

I can't reproduce the behavior you show. Pls provide a concrete sample with instructions. Thanks!

/needsMoreInfo