Why not create newaccount?

I have error:
Traceback (most recent call last):
  File "/usr/local/bin/steempy", line 11, in <module>
    sys.exit(legacy())
  File "/usr/local/lib/python3.5/dist-packages/steem/cli.py", line 1282, in legacy
    delegation_fee_steem=args.fee,
  File "/usr/local/lib/python3.5/dist-packages/steem/commit.py", line 484, in create_account
    self.wallet.addPrivateKey(active_privkey)
  File "/usr/local/lib/python3.5/dist-packages/steem/wallet.py", line 212, in addPrivateKey
    self.keyStorage.add(self.encrypt_wif(wif), pub)
  File "/usr/local/lib/python3.5/dist-packages/steem/wallet.py", line 151, in encrypt_wif
    self.unlock()
  File "/usr/local/lib/python3.5/dist-packages/steem/wallet.py", line 100, in unlock
    masterpwd = self.MasterPassword(pwd)
  File "/usr/local/lib/python3.5/dist-packages/steembase/storage.py", line 401, in __init__
    self.decryptEncryptedMaster()
  File "/usr/local/lib/python3.5/dist-packages/steembase/storage.py", line 413, in decryptEncryptedMaster
    raise WrongMasterPasswordException
steembase.storage.WrongMasterPasswordException

I tested command:

steempy newaccount divali8 --account furion --fee "9 STEEM"

steempy newaccount divali888 --account steemy

steempy newaccount divali888
I used pass example
P5KQK4iogQypqYAzXbHWmhbwxddVFz8Xhg69czjKLDVhKwEQT2KY - use online generate
mp903804124
My OS
Distributor ID: Ubuntu
Description:    Ubuntu 16.04.2 LTS
Release:        16.04
Codename:       xenial
Python
Python 3.6.1 :: Anaconda 4.4.0 (64-bit)
Please help me
))

I am not sure what are you trying to do here.
steembase.storage.WrongMasterPasswordException means that your offline storage wallet password (this is not your steem key/password) is incorrect. You are asked to pick a password the first time you import an account into steempy. This is the password that is also required every time you want to use the local wallet.

I did not create passwords, accounts in steempy.
I want to create a new account for steemit.com.
My first command was for this:
steempy newaccount merlo8901 --account steempy
http://joxi.ru/DmBgabzhXYad2P?d=1
But the account was not created.
Tell me how to create an account for steemit.com using steempy.
Thanks

You need an account with some liquid STEEM to create an account.

For example, like this?
steempy newaccount divali1234 --account  dinoo --fee "1 STEEM"
dinoo  have STEEM
http://joxi.ru/82QjgPOHPNXBAd?d=1
Error again
http://joxi.ru/krDg3Yjh19e5Ap?d=1

Here is a step by step requirement.
1.) Import your existing account (dinoo) into steempy
2.) Set local wallet password (this should happen automatically in step 1), you can set env variable UNLOCK to avoid being prompted for pw every time.
3.) Run steempy newaccount

I took all the steps.

Imported account
Run creation new account.

everything is ok!
I have a question:

Minimum account_creation_fee in steempy 1 STEEM?
Can I create account without fee or with official account_creation_fee: 0.200 STEEM? How to do it?

I tried to do without fee and got an exception:
Traceback (most recent call last):
  File "/usr/local/bin/steempy", line 11, in <module>
    sys.exit(legacy())
  File "/usr/local/lib/python3.5/dist-packages/steem/cli.py", line 1282, in legacy
    delegation_fee_steem=args.fee,
  File "/usr/local/lib/python3.5/dist-packages/steem/commit.py", line 563, in create_account
    return self.finalizeOp(op, creator, "active")
  File "/usr/local/lib/python3.5/dist-packages/steem/commit.py", line 128, in finalizeOp
    return tx.broadcast()
  File "/usr/local/lib/python3.5/dist-packages/steem/transactionbuilder.py", line 131, in broadcast
    raise e
  File "/usr/local/lib/python3.5/dist-packages/steem/transactionbuilder.py", line 129, in broadcast
    self.steemd.broadcast_transaction(self.json())
  File "/usr/local/lib/python3.5/dist-packages/steem/steemd.py", line 860, in broadcast_transaction
    return self.exec('broadcast_transaction', signed_transaction, api='network_broadcast_api')
  File "/usr/local/lib/python3.5/dist-packages/steembase/http_client.py", line 186, in exec
    return_with_args=return_with_args)
  File "/usr/local/lib/python3.5/dist-packages/steembase/http_client.py", line 206, in _return
    raise RPCError(error_message)
steembase.exceptions.RPCError: 10 assert_exception: Assert Exception
creator.balance >= o.fee: Insufficient balance to create account.
    {"creator.balance":"0.640 STEEM","required":"6.000 STEEM"}


Add Emoji Reactions Based Off Keywords

https://github.com/hackclub/scrappy/pull/58#issuecomment-656736494
https://github.com/hackclub/scrappy/pull/58#issuecomment-656940842
https://hackclub.slack.com/archives/CNMU9L92Q/p1594444638361400?thread_ts=1594442577.357200&cid=CNMU9L92Q

Thanks so much 🙌 made some optimizations. Completely untested, so time to test!! Lol

AND WE'VE BROKEN SCRAPPY :PARTYPARROT:
On Tue, Jul 14, 2020 at 11:58 PM Matthew Stanciu<EMAIL_ADDRESS>wrote:

Merged #71 https://github.com/hackclub/scrappy/pull/71 into main.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hackclub/scrappy/pull/71#event-3544829582, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AJP3VRA4GQHJTLMAPNQ33JLR3R6A7ANCNFSM4OZD6UDQ
.

--
This email and any attachments are confidential and may also be privileged.
If you have received this message in error, please (a) notify the sender
immediately, (b) destroy this email and any attachments, and (c) do not
use, copy, store and/or disclose to any person this email and any
attachments. The sender gives no representation or guarantee with respect
to the integrity of any emails or attached files and the recipient should
check the integrity of and scan this email and any attached files for
viruses prior to opening.


Implemented GenUI for NiGui

This is an implementation of the GenUI macro for NiGui. It includes an example and an explanation of how to use it and what it is.

Wow, very good work!

But I think you need to make "genui" available when installing from nimble.
Maybe move it under nigui directory?

Sorry, my bad. It was made with the old folder structure, and I forgot to check when I merged in your changes to my fork. Should be fixed now.

Is there a way to export variable from @result?
genui:
  Window[width = 800, height = 600, show]("Nickel"):
    LayoutContainer(Layout_Vertical):
      {var guiLog* = @result} TextArea()

this doesn't work

Currently I need to use this as I need to export two variables:
genui:
  {var tempWin = @result} Window[width = 800, height = 600, show]("Nickel"):
    LayoutContainer(Layout_Vertical):
      {var temp = @result} TextArea()

var window* = tempWin
var guiLog* = temp

Ah, no, this sadly doesn't work

Hmm, it doesn't work for me this way..
Maybe you can somehow add possibility to export variables from genui macro?

Also I did a quick port of NiCalc gui to genui:
genui:
  {var window = @result} Window[width = 600, height = 450]("NiCalc"):
    LayoutContainer[padding = 6](Layout_Vertical):
      LayoutContainer(Layout_Horizontal):
        Label("Input:") [minWidth = LabelWidth, heightMode = HeightMode_Fill]
        {var inputTextBox = @result} TextBox() [fontSize = EditFontSize, fontFamily = EditFontFamily]
        {var clearButton = @result} = Button("Clear") [minWidth = ButtonWidth, heightMode = HeightMode_Fill]
      LayoutContainer(Layout_Horizontal):
        Label("Result:") [minWidth = LabelWidth, heightMode = HeightMode_Fill]
        {var resultTextBox = @result} TextBox() [fontSize = EditFontSize, fontFamily = EditFontFamily]
      {var historyContainer = @result} LayoutContainer(Layout_Vertical)[frame = newFrame("History")]:
        {var historyTextArea = @result} TextArea() [fontSize = HistoryFontSize, fontFamily = EditFontFamily]

Ah yeah. The {} code thing is still a work in progress. Previous versions of genui used identifier % Button to assign a new button to the identifier "identifier". This was limited to only that though and couldn't be used for thing like array assignment and such. The plan with {} was to allow any code to be inserted but it proved a bit hard because of how Nim parses things before the macro gets a look at it. I'll try to work out something a bit more robust though. The @result was also a bit of a necesitty of the same reason, Nim refuses to parse only @. I was also thinking that maybe different @ values could be used to get different things, like using @children in a loop for a repeating section for example. But I agree that it's a bit long and clumsy.

@PMunch anyway genui is a great addition to NiGui

Oh thanks, I think so to. By the way, another workaround for your specific problem which might be better is to simply define the variable before assigning to it in the curly brackets like so:
var myButton*: Button
genui:
  Window[width=800, height=600]:
    {myButton = @result} Button("Hello world")

Okay, I've now added a small check to the handling of pure code. If the only node in the code is a string literal it parses it as code instead of inserting it directly as code. This means that you can now do:
genui:
  Window[width=800, height=600]:
    {"var myButton* = @result"} Button("Hello world")

And that would be parsed into code and work as you would expect. It's not perfect but it should work for now.

@PMunch thanks!

Hmm, just talked to some guys over in the IRC channel and it seems like the error is with how curly brackets are parsed. I checked and if you simply add parenthesis like {(code)} then it also works fine (plus you get syntax highlighting).

@PMunch thanks for this workaround!
works perfectly

How was the porting experience by the way? One thing I feel is important with genui is how it is a fairly simple conversion to actual code so you should be able to easily port examples over. I hate when working with Gtk and the GUI builder just to Google something and only finding code snippets which I can't easily convert, or vice versa.

@PMunch porting NiCalc gui to genui was fairly simple.
genui is the best for projects where GUI code is larger than one screen in length.
For example NiCalc has 47 lines of gui creating code (including some spaces between different sections)
And also for now nigui doesn't contain many widgets, but I think this will improve in the future.
Maybe this is the simple GUI framework for Nim we've waited for :)

I'm definitely hoping so, which is why I wanted to port genui to it as quickly as possible. I added a shorthand @r now by the way that can be used instead of @result, is that a fair compromise?

@PMunch yeah, thanks again!

So, let's wait for @trustable-code  :)

@trustable-code so what is the status of this PR?

@Yardanico I don't like additional complexity and don't see an advantage of using such macros as of now.

I wrote a bit about the benefits of this system in my post here(primarily under the "A simple GUI DSL" heading). The macro in play here is actually fairly simple as long as the general interface is kept fairly uniform. It maps 1:1 from the DSL input to generated code, and doesn't really do a whole lot more than structure the code. The entire implementation is only about 150 lines, and could probably be shortened even more, plus it would probably not have to be changed at all for the foreseeable future unless you add some widgets which acts completely different than the ones you've got.

Well technically yes. The wxNim version of genui was made a part of wxNim simply because I had to make changes to the code in order to make it work for all widgets. For NiGui on the other hand it could easily be implemented as an entirely different package.

I tried to add "genui:" for nigui's example_02_controls.nim but hit:
Error: undeclared identifier: 'genui'

@andrewgohlk, on my fork, or the main repository? This PR is pretty outdated by now, no idea if it still works or not.


Cleaned up delay test initialization

I didn't go with the attribute route because I think this makes things a bit cleaner for the test method and init method declarations. Happy to change it though if you think the attributes are better for this or other reasons.

PTAL

Actually, nothing else about registering delay tests is thread safe, so I'm just going to remove the attribute.

(rebased)


Dataset opening on last page

I have a dataset that is empty on mount. The data is loaded in the parent's mounted() method and arrives whenever the AJAX call there has finished. The dataset does populate correctly, then, but it always opens on the last page. Is there a way to fix this?

Here's a demo of that case, the dataset opens on the first page
https://stackblitz.com/edit/vue-rbm9pl?file=src%2FApp.vue
Feel free to re-open this issue when you have a codesandbox or stackblitz replicating the behavior.


Data repetition in get_multi_joined and issue in join_on

data = crud_user.get_multi(
    db,
    nest_joins=True,
    joins_config=[
        JoinConfig(
            model=Portions,
            join_on=User.portion_id == Portion.id,
            join_prefix='portions',
            join_type="left",
           relationship_type='one-to-many'
        ),
        JoinConfig(
            model=Category,
            join_on=User.created_id == Category.id,
        ),
    ],
)

the above code is what I used to run. When it is complete, it give data like this but the problem is that in portions, the data is repeating like the below one;
{
    "id": 1,
    "created_user_id": 1,
    "name": "Test",
    "email"<EMAIL_ADDRESS>    "portions": [
        {"id": 1, "user_id": 1, "quantity": "Half"},
        {"id": 2, "user_id": 1, "quantity": "Full"},
        {"id": 1, "user_id": 1, "quantity": "Half"},
        {"id": 2, "user_id": 1, "quantity": "Full"},
    ],
    "category": [{"id": 1, "category": "Food"}, {"id": 2, "category": "Real-Estate"}],
}

when i debug the code, the problem is found in here;
def _nest_multi_join_data(
    base_primary_key: str,
    data: list[Union[dict, BaseModel]],
    joins_config: Sequence[JoinConfig],
    return_as_model: bool = False,
    schema_to_select: Optional[type[BaseModel]] = None,
    nested_schema_to_select: Optional[dict[str, type[BaseModel]]] = None,
) -> Sequence[Union[dict, BaseModel]]:

    pre_nested_data = {}

    for join_config in joins_config:
        join_primary_key = _get_primary_key(join_config.model)

        for row in data:
            if isinstance(row, BaseModel):
                new_row = {key: (value[:] if isinstance(value, list) else value) for key, value in row.model_dump().items()}
            else:
                new_row = {key: (value[:] if isinstance(value, list) else value) for key, value in row.items()}

            primary_key_value = new_row[base_primary_key]

            if primary_key_value not in pre_nested_data:
                for key, value in new_row.items():
                    if isinstance(value, list) and any(item[join_primary_key] is None for item in value):  # pragma: no cover
                        new_row[key] = []

                pre_nested_data[primary_key_value] = new_row
            else:
                existing_row = pre_nested_data[primary_key_value]
                for key, value in new_row.items():
                    if isinstance(value, list):
                        if any(item[join_primary_key] is None for item in value):  # pragma: no cover
                            existing_row[key] = []
                        else:
                            existing_row[key].extend(value)

    nested_data: list = list(pre_nested_data.values())

here the
for join_config in joins_config:
        join_primary_key = _get_primary_key(join_config.model)

        for row in data:

When the first loop is complete, it gives the correct data, but the problem is that there are two join configs so the outside loop will execute again and do the same thing. I think that's why the data is repeating

And also, there is another issue, like if I add join_on, it will show this error message: Could not automatically determine join condition. Please provide join_on.  The reason is because:
 join_on=join_on or _auto_detect_join_condition(self.model, join_model),
To solve it, I added it like this:
if join_on is None and joins_config is None:
            join_on = _auto_detect_join_condition(self.model, join_model)

Hey, @mithun2003, thanks for the issue! Do you want to fix it as well?

Hey @igorbenav, Well I have been busy for a while now. But I will check on this issue when I got some time.

Closed by #186


Porting some of the missing tests

Abhishek,
I added back some of the missing tests. Can you please review?
Rahul

Abhishek,
Just realized that this pr has other commits as well. They should be reviewed as well
Rahul

I"m assuming you got a clean run.

Verified the new interpreted partition pruning tests being added.
Verified the correction made to download baselines to the right directory.

Looks good. +1

Yes I did get a clean run!


Add a timing function for experiment

Linked with issue tracked with https://github.sec.samsung.net/jhoon-it-lee/nntrainer-experiments/issues/7

:octocat: cibot: Thank you for posting issue #787. The person in charge will reply soon.


Could not deploy lambda the first time

While deploying a Lambda function for the first time the stage may fail when executing the task - Lambda Event Configuration Task.
Error:
Exception ( Lambda Event Configuration Task )

Could not find lambda to update event config for

This task is executed regardless of if an event source was set or not.
Reproducing

Create a Lambda Create stage
Save Pipeline
Execute Pipeline

This issue is intermittent, and no root cause was identified in the AWS environment.
The second run of the pipeline will pass successfully.
Reference JSON used to create/execute the stage:
{
    "account": "${parameters[\"account\"]}",
    "batchsize": 10,
    "cloudProvider": "aws",
    "deadLetterConfig": {
      "targetArn": ""
    },
    "detailName": "",
    "enableLambdaAtEdge": false,
    "envVariables": {},
    "functionName": "${execution.application}-${parameters[\"lambda_function_name\"]}",
    "functionUid": "${parameters[\"lambda_function_name\"]}",
    "handler": "${parameters[\"lambda_handler\"]}",
    "memorySize": 128,
    "name": "Deploy Spinnaker Events Lambda",
    "publish": true,
    "region": "${parameters[\"aws_region\"]}",
    "reservedConcurrentExecutions": 10,
    "role": "${parameters[\"role_arn\"]}",
    "runtime": "${parameters[\"lambda_runtime\"]}",
    "s3bucket": "${parameters[\"s3_bucket\"]}",
    "s3key": "${parameters[\"s3_key\"]}",
    "securityGroupIds": [],
    "stackName": "${parameters[\"environment\"]}",
    "subnetIds": [],
    "tags": {
      "purpose": "Lambda Deployment"
    },
    "timeout": 90,
    "tracingConfig": {
      "mode": "PassThrough"
    },
    "triggerArns": [],
    "type": "Aws.LambdaDeploymentStage"
  }

I'm running into this. Found this related issue: https://github.com/spinnaker-plugins/aws-lambda-deployment-plugin-spinnaker/issues/51 - not sure if the root cause is related or the error message is the commonality. It could be related to timing or a race condition of caching or saving the data in clouddriver vs creation/querying the upstream resource.

@shaiguitar and I have been able to consistently reproduce this issue using version 1.0.6 of this plugin while deploying functions to us-east-1. However, unlike mentioned above, a second execution of the pipeline will continue to fail with Function already exist: <function name> (Service: AWSLambda; Status Code: 409.
Likely Cause
It is taking much longer to create a function in us-east-1 and the Lambda Event Configuration Task is timing out. The error is thrown at L70 of LambdaUpdateEventConfigurationTask.java:
    public TaskResult execute(@NotNull StageExecution stage) {
       ...
        LambdaDefinition lf = utils.findLambda(stage, justCreated);
        if (lf == null) {
            return formErrorTaskResult(stage, String.format("Could not find lambda to update event config for"));
        }
       ...
    }

This means that utils.findLambda cannot find the Lambda Function after creating it, even though it attempts to find the function 5 times (link to code here):
    public LambdaDefinition findLambda(StageExecution stage, boolean shouldRetry) {
        LambdaGetInput lgi = this.getInput(stage, LambdaGetInput.class);
        lgi.setAppName(stage.getExecution().getApplication());
        //LambdaGetOutput lf = (LambdaGetOutput)stage.getContext().get(LambdaStageConstants.lambdaObjectKey);
        LambdaDefinition lf = this.retrieveLambda(lgi);
        int count = 0;
        while (lf == null && count < 5 && shouldRetry == true) {
            count++;
            lf = this.retrieveLambda((lgi));
            this.await();
        }
        return lf;
    }

NOTE: We do not see this issue while deploying to us-west-2. However, we do see it occurring while deploying to us-east-1.
Short Term Solution Proposals

Increase the number of attempts to retrieve the lambda information
Add a longer wait time in between each attempt

Reproducing
To reproduce follow the steps below:
Create a Lambda Create stage with JSON mentioned below (notice region set to us-east-1)
Save Pipeline
Execute Pipeline

Reference JSON for Spinnaker stage (notice region set to us-east-1)
{
  "account": "${parameters[\"account\"]}",
  "batchsize": 10,
  "cloudProvider": "aws",
  "deadLetterConfig": {
    "targetArn": ""
  },
  "detailName": "",
  "enableLambdaAtEdge": false,
  "envVariables": {},
  "functionName": "${execution.application}-${parameters[\"lambda_function_name\"]}",
  "functionUid": "${parameters[\"lambda_function_name\"]}",
  "handler": "${parameters[\"lambda_handler\"]}",
  "memorySize": 128,
  "name": "Deploy Spinnaker Events Lambda",
  "publish": true,
  "region": "us-east-1",
  "reservedConcurrentExecutions": 10,
  "role": "${parameters[\"lambda_role_arn\"]}",
  "runtime": "${parameters[\"lambda_runtime\"]}",
  "s3bucket": "${parameters[\"s3_bucket_name\"]}",
  "s3key": "${parameters[\"s3_key\"]}",
  "securityGroupIds": [],
  "stackName": "${parameters[\"environment\"]}",
  "subnetIds": [],
  "tags": {
    "purpose": "Lambda Deployment",
  },
  "timeout": 5,
  "tracingConfig": {
    "mode": "PassThrough"
  },
  "triggerArns": [],
  "type": "Aws.LambdaDeploymentStage"
}

Great details, thanks @coleduclos ... I have been using us-west-2 primarily. possibly why this appeared to be a non-issue. will try with us-east as well. should be an easy fix.


[SPARK-49086][CONNECT] Move ML function registration to SparkSessionExtensions

What changes were proposed in this pull request?
This PR moves ML function registration from the SparkConnectPlanner to the internal function registry. This registration is done using the SparkSessionExtensions mechanism.
Why are the changes needed?
Unification of Connect and Classic Column API. This PR decouples the ML functions from Catalyst expressions.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing tests.
Was this patch authored or co-authored using generative AI tooling?
No.

@zhengruifeng this is failing org.apache.spark.ml.FunctionsLoadingSuite. Can I remove this? I can't make it error out anymore after this.

@hvanhovell I think we can remove test org.apache.spark.ml.FunctionsLoadingSuite introduced in https://github.com/apache/spark/pull/43739, because we don't need objects vectorToArrayUdf and vectorToArrayFloatUdf any more
also cc @WeichenXu123 and @zsxwing who are the reviewers of https://github.com/apache/spark/pull/43739

@hvanhovell wanted to merge this but there's a conflict ..

Merging to master!


Problem running on a stack or folder of multiple images (tuple index out of range)

I get a 'tuple index out of range' error when trying to load multiple images, the segmentation works great on a single image.
Any ideas?

For a folder of images, check out:
https://github.com/guiwitz/napari-serialcellpose

thanks @tlambert03 and @psobolewskiPhD


Update gradlew.bat

I added an extra empty space line, at the end. :+1: (From Code-In Task)

This change is

@ThunderMan2300 Thanks for checking out Blockly. Be aware, this file is periodically overwritten by Android Studio when the version of gradle updates.
Was there an issue that prompted this change?


workflow: add publish action

Adding workflow to publish the package with provenance.
Close #14

BTW, I didn't have right access to put env key. so you I'll need to put your token


Powershell doesn't have an arm64 package for Ubuntu

Summary of the new feature / enhancement
Add the arm64 package for ubuntu alongside the existing amd64 one
Proposed technical implementation details (optional)
No response

The standard process is to raise a DSR
Eg https://github.com/PowerShell/PowerShell/issues/16617 and https://github.com/PowerShell/PowerShell/issues/24076


How to get IP addresses of connected clients?

I would have thought that GetClientSessionsStatusAsync()'s returned element's Endpoint member would give me the IP address but it doesn't.  It is null.
Am I missing something?  Running version 2.8.2.

Nevermind.  Something happened and they are not null now.

How Did you done this?

There used to be something called MqttServer.GetClientSessionStatusAsync() where you could query by client Id and get the endpoint.  Not sure if this is there in later versions.

I use the last version of branch 2. I'm able to use GetClientSessionStatusAsync. A question: Why I don't see if client is off? Do it depend by timeout?


nixos/systemd: add an option to suppress system units

What? Why?
Originally #59742, but I also comment-out a bunch of upstream units (e.g. rescue and time-sync things) on my NixOS+systemd systems anyway, so this feature is warranted even without that PR, IMHO.
git log

nixos/systemd: add an option to suppress system units

nix-instantiate environment

Host OS: Linux 4.9, SLNOS 19.09
Nix: nix-env (Nix) 2.2
Multi-user: yes
Sandbox: yes
NIXPKGS_CONFIG:

{
  checkMeta = true;
  doCheckByDefault = true;
}

nix-env -qaP diffs

On x86_64-linux:

Updated (1):

tests.nixos-functions.nixos-test

On aarch64-linux: ditto
On x86_64-darwin: noop

ping?

Thanks!


EdTechJoker UX Lab problem 1

Setup:

Create a blank site in HAXcms on https://hax.psu.edu

User 1:

Talk through the process of editing the page of content you are presented in HAXcms
Edit this page to add a video from youtube, without leaving the editor
Add an image from NASA to the page and meme it, again without leaving the editor

What follows are the user stories documented for this task in the comments from User 2

User 1 feedback:

Can't just click on text to be able to edit it (header). Can't figure out how to change header "Welcome to a new HAXcms site!".

Took about 15 seconds to find the "add section". Didn't know someone could find a Youtube video right from this editor, so we left page to get a link.

Did not realize you could use the "Find" - didn't associate this menu with a web search.

Can't figure out easily how to "meme" it.

Feedback: Should put the "find" under add" .

Gazing at tabs to see what his options are
Clicked to see what “import” did
Went to his site – looking at side menu
Decides to add a page.
Confused on what “location” means when creating a page
Easily started editing page.
Not sure if adding url in add content will let him imbed video
Reading through options when embedding – not confused on how to add video once he read all the buttons
Adding a new page – confused on location again (can he move it somewhere else(aka not on left sidebar))
Hovered over “image” when adding content for awhile – then saw “meme” and quickly picked that one
Once he did the video one, the image one went a lot faster

The user first hit add page and named it youtube.
Next he hit edit, "Im sure we can add a video this way"
User went to add content and could not find a way to locate a video without leaving the site.
User retrieved youtube URL and copy pasted it into the page then saved it.
User made new page titled nasa pciture.
User hit add URL and configure item to find an image. Confusion ensued so user went to NASA.com and copy pasted an image in.

User: Henry Pearce
Observer: Stephen Dalo

Created site cause of the blue button
Click on edit page button
Went on youtube to find a video. To add link, click the button to add the link and pasted the url.
To exit the editor click exit on the left hand side
Went to google to find nasa memes
Clicked "add" to drop a file into it. File was image. Hit update and the meme was on the page

Go to the Edit page, click the add button.
Go to the YouTube, and copy video link.
Paste the link on URL and upload it.
Go to the NASA website, copy the image link.
Paste the link on URL and upload it.
And save page.

First hit blue button to create page
Entering site name (test site)
No domain no description
Create site
Clicked edit
Clicking make
Clicking video player
Clicking advanced
Configure
CANCEL ------
Click find
Click youtube
Search doug demuro
Click video
Click video player
Click update  - DONE
Click find, click nasa, click image, click image, update, hit settings, transform to, meme,  type random text , update - DONE

User: Reed McGarvey
Observer: Nick Witmer

Got to Reclaim Hosting, installing necessary HAX components, went to /login to get to editor screen.
Created a new site via the "Create" button, Site Name: Great Site For Haxing the Web. Domain: labproblem1
Changed to theme tab, minimalist site option chosen.
Pick an icon under themes tab "I dont know if this is useful, but ill pick the full moon"
Added a janky PSU emblem as the "Image"
"Create My Site!"
Clicked on the Site Outline button, clicks on the "Add" button
Created a homepage title page, indent and outdent seem to relate to a hierarchy for setting up a file structure
Under Home, indented and created a new component called "youtube video". What is being added is unclear since their is no indication other than "site outline"
Saved

To add youtube video
11. Clicked on the edit page button from left vertical pane
12. Clicks on heading button from horizontal top pane
13. "Sort of looks like the Gutenberg site creator from wordpress"
14. Clicks "Add" button from top pane, add existing link and copy/paste youtube link into field, configure item, video player, update
Add NASA image
15. Similar to adding a youtube video, clicked on the edit page button from vertical pane and then "add" button from horizontal pane, added text and uploaded image from source, then clicked "upodate button" to refresh page.


GSK-2887 Upload tests with project when uploading a suite

Always send the project_key when uploading/downloading tests

LGTM Just some comments to be removed or polished.
Not related to this PR: We will neither have global resources anymore, nor "internal worker". So we can also cleanup the Optional[GiskardClient], can we create a card for this?

I've updated th PR and remove the optional client


Ensures fold gutter is always added after line numbers

This addresses #11577 - where code folding gutter is not displayed nicely when line numbers are disabled. Also addresses #10864 where toggling the display of line numbers while code-folding is enabled causes fold gutters to be rendered before line numbers.

I think that this PR is pretty crucial and absolutely worth revisiting asap
/cc @abose @swmitra

@thehogfather is this not necessary anymore?

@ficristo actually it is! - apologies. I was cleaning up my git branches and inadvertently deleted the branch. This is my bad. Will create a new PR and it can be reviewed.


endpoint: Fix endpoint restore functionality

Commit 7948462f5 "pkg/endpoint: rename Endpoint SecLabels field"
unintentionally broke endpoint restore across upgrades crossing
the commit.
Signed-off-by: Thomas Graf<EMAIL_ADDRESS>

test-me-please


Date days are added with additional digit

Environment

Tech
Version

material-ui-pickers
rc14

material-ui
3.0.1

React
16.4.2

Browser
Chrome

Peer library
date-fns alpha.16

it also happens on the official docs example:
https://material-ui-pickers.firebaseapp.com/guides/formats
Expected behavior
show the day properly
Actual behavior
days are added with 3rd digit.

I am facing the same issue too.

Upgrade from rc13 to rc14
Upgrade date-fns from 2.0.0-alpha.7 to 2.0.0-alpha.16

@gairom ,
I have the same issue. I found out the issue is with the format.
The correct format is case sensitive
format="dd/MM/yyyy"
make sure the dd is not DD.

Yep, D -> d fixed it, thanks!


Add default values for enums

Damn, this doesn't work because the generated class from json_serializable doesn't add the enums. prefix.

@Vovanella95 as I mentioned this doesn't actually work, because the files generated by json_serializable can't reference the as enums namespace. Trying to find a solution now.

@via-guy  ok, waiting for fix. Also, I have issue with generation of some files (Format exception). Will fix this one.

I'm struggling like crazy to solve this issue in json_serializable. In enum_utils.dart there's the function enumValueMapFromType, which creates the enum maps. The problem is that the Element value doesn't contain the import prefix, and I don't know how to get it from the Element, since it's the parent class that uses the prefix, not the Element itself. Could you have a look to see if you can find out how to get it?

@via-guy  no, enums prefix has been added to avoid models and enums names conflicts

@via-guy I will also take a look into this soon and try to fix it

@via-guy no, enums prefix has been added to avoid models and enums names conflicts

Is that really a problem? Are API designers creating enums and models with the same name? That sounds like a serious design issue, no? Oh well... 😞

@via-guy I will also take a look into this soon and try to fix it

Thanks. I spent a very long time trying to see if I could create a fix for json_serializable where I could get the import prefix, but it seems like that information is lost along the way in the build/analyzer flow :(

We can pass fromJson and toJson methods to @JsonKey for enums. It's already should happening for all enum properties. In this case json_serializable knows nothing about enums.
@via-guy

@Vovanella95 the problem is that when you set a default value the generated code needs to use that, so you get something like this:
validationStatus: json['validation_status'] == null
    ? AddExistingTicketValidationStatus.inProgress
    : addExistingTicketValidationStatusFromJson(
        json['validation_status'])

so even though it's using our fromJson function, it needs to provide the default value, which means needing the enums. prefix.
Another solution would be, instead of using the actual prefix, to pass (in this example)
validationStatus: json['validation_status'] == null
    ? addExistingTicketValidationStatusFromJson('in_progress')
    : addExistingTicketValidationStatusFromJson(
        json['validation_status'])

Let me work on this. It's not an ideal solution, but it should work.

@via-guy let's maybe revert your changes, raise and issue, when you describe problem, you tried to solve and I will try to fix it. Share swagger code snippet and expected result.

Look like issue is not trivial, it will be better to implement it by me

My alternate solution doesn't work, because it generates:
@JsonKey(
    name: 'validation_status',
    toJson: addExistingTicketValidationStatusToJson,
    fromJson: addExistingTicketValidationStatusFromJson,
    defaultValue: addExistingTicketValidationStatusFromJson('in_progress'))

which fails to compile because addExistingTicketValidationStatusFromJson('in_progress') isn't a constant expression.

@Vovanella95 I think you're right, we need to revert my changes.

@via-guy  let's raise issue. I will try to take a look into it asap


[apache/pulsar-client-go] the function Send(ctx,msg) performance is not good

Expected behavior
I want to send a quantity of messages using pulsar-client-go (not cgo), and I hope it can be done in less than 1 second
Actual behavior
I call Send(ctx,msg) about 2000 times in a loop, it costs me 6 seconds to finish totally.
One message's capacity is 60 Bytes or so.
Steps to reproduce
       for i:=0; i<2000; i++ {
                        ..........
		msg, err := proto.Marshal(&xxx)
		if err != nil {
			fmt.Println(err)
			return
		}

		if err := producer.Send(context.Background(), &pulsar.ProducerMessage{Payload: msg}); err != nil {
			fmt.Println(err)
		} else {
			//fmt.Printf("[%d]success\n", i)
		}
	}

System configuration
OS: macOS 10.14
docker in localhost: apachepulsar/pulsar-standalone:2.3.1 bin/pulsar standalone

Now, the go client depends on the cpp client. When we send the message every time, we will call the send of the cpp client to implement the real send function. In this process, a system call from go to c will occur, so there will be a certain performance issues.
Currently we use native go to rewrite pulsar's go client, the interface is consistent with the cgo client, the project address: https://github.com/apache/pulsar-client-go/, you can try to use this client to see if it can meet your performance requirements.


Multiple teeproxies on same host

Hello,
On a production server we have several web servers running on the same host, under distinct ports (8080, 8081, 8082). We want to have teeproxy forwarding to each of these ports, but we have no luck on doing this.
If running a single teeproxy process (eg.: using 8080 as A), everything works as expected.
Although, if launching a second or more teeproxy processes, nothing else is forwarded to the A server.
For example:
teeproxy -l :9000 -a localhost:8080 -b another:7777
teeproxy -l :9001 -a localhost:8081 -b another:7777
If kept the first line, A and B servers receive the requests. When launching the second process, A:8080 stops receiving requests, but B:7777 keeps receiving them (when posting to :9000).
Is this the expected behavior?

no. I do not even think its possible. Could you please do some debugging? the code should be easy to read and debug.

Due to some deadline constraints I ended up using an nginx to have this behavior implemented (with a single teeproxy). It worked flawlessly. Thanks!


psreadline and mintty errors

Oops, something went wrong.  Please report this bug with the details below.
Report on GitHub: https://github.com/lzybkr/PSReadLine/issues/new
-----------------------------------------------------------------------
Last 0 Keys:

Exception:
System.IO.IOException: The handle is invalid.

   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
   at System.Console.GetBufferInfo(Boolean throwOnNoConsole, Boolean& succeeded)
   at Microsoft.PowerShell.Internal.ConhostConsole.get_CursorLeft()
   at Microsoft.PowerShell.PSConsoleReadLine.Initialize(Runspace runspace, EngineIntrinsics engineIntrinsics)
   at Microsoft.PowerShell.PSConsoleReadLine.ReadLine(Runspace runspace, EngineIntrinsics engineIntrinsics)
-----------------------------------------------------------------------

FYI  Mintty works when running powershell -version 2

Most likely a dupe of #158, reopen with additional information if you disagree.


Fixed the bug in mkl_input_conversion_op when reorder is not needed

In mkl_input_conversion, if one tensor is mkl tensor an another one tf tensor, we check if broadcast is needed. If broadcast is not needed, tf tensor is converted to mkl tensor. However, when tf tensor and mkl tensor has the shape, mkldnn reorder is not run, the output buffer is not updated, so the output buffer is random data.
The fix is to copy the memory if reorder is not needed.

@shengfuintel Thanks for the fix. Please confirm that the corresponding unit tests passes on your end.

I do not think my change will break so many XLA tests. Can you check? thanks.

Looks like there are something wrong in your llvm environment. in //tensorflow/compiler/tests:adagrad_test_cpu, the compile error is:
tensorflow/compiler/xla/service/gpu/llvm_gpu_backend/gpu_backend_lib.cc:37:41: fatal error: llvm/CodeGen/CommandFlags.inc: No such file or directory
1705
#include "llvm/CodeGen/CommandFlags.inc"

Is it ready to merge? thanks.


Column Break followed by Vertical Spacing doesn't work

Title. The column break + spacing combo I use for formatting banner art and pictures isn't working any longer. The Vertical spacing doesn't work, anyway. Leaving the columns uneven or interfering with the art.
It breaks the brew in Editor and Share, and I haven't finagled a fix yet.
Additional Details
Share Link :
or
Brew code to reproduce : Click to expand
PASTE BREW CODE HERE

I don't think the Homebrewery has had any recent code changes that would affect it (I noticed this as well on my brews); it might be a change in the way Chrome is handling CSS now. For me, it looks like top and bottom margins are now being ignored for empty divs when placed at the beginning of any column other than the first. Mid-column is fine, but top of column doesn't work.
One option that worked for me is to simply change the divs I use for spacing to use padding rather than margins. It might work for you, but since you didn't post the "share" link to your brew its kind of hard to tell what exactly you are trying to do.

I figured out the problem (by trying to toy with the CSS).
The way I did it was to always space out every line of wording (so it's just easier for me to read/pinpoint stuff) and the vertical spacing divs don't accept space between it and what it's spacing. (I have no clue how to say it correctly, but basically, example below)
The way I used to do it:

Blah blah apples

And that wasn't working anymore, so all my brews were messed up (because the margin wasn't working) So it works like this now

Blah blah apples

Literally, one line of spacing was the problem. XD. But, I fixed it. Went through all my past brews and fixed it too. So, thanks for the help! This is a case closed, now.


Missing tests

It's seems like the tests/ are missing in the repository.

@raphaelstolt, yes, they will be added later :)


Prayer times website is empty

The example website "https://habous-prayer-times-api.onrender.com/api/v1/prayer-times?cityId=109" does not return the prayer times of any city.
It shows: {"message":"Request could not be fulfilled"}

Sorry for not responding to this earlier. The website seems to be working again as expected.
If you plan to use a prayer API that is accurate, you can use the Menara.ma prayer API. It follows the format:
https://backend.menara.ma/api/prayers/{cityId}/12.
I still haven’t figured out the list of city IDs (e.g., 69 is the city ID for Rabat) or the purpose of the number 12, but the API is working and accurate.


Feature request: custom languages

Would be very useful to be able to pass in custom language definitions which would be passed down to highlight.js
Cheers

FYI: This would be a 3 line patch... you'd merely need to wrap addLanguage the way that listLanguages already is.  Though I'm not sure this is maintained anymore?
Disclaimer: Current Highlight.js maintainer here.


Add support for Zenless Zone Zero

Hi, could you add support for Zenless Zone Zero since it's there for Honkai impact 3rd? Thanks.
Also the bot doesn't redeem the rewards for Honkai Impact 3rd, I've tried logging back in but it didn't work.

Yes, ZZZ is on my todo list. Probably will get it done this weekend

also the bot supports genshin and HSR, not HI3

Weird because for some months it worked fine for HI3

Oh I see why it worked. The default config didn't have honkai 3rd but you could still add it. I didn't try so I didn't think it would work


cvxpy + ecos2.0.8 no longer compatible with numpy<1.20

Hi all,
I'm experiencing the same bug as I reported a year ago: https://github.com/cvxpy/cvxpy/issues/1229. This time around it is actually the ecos package (v2.0.8) that is built on numpy 1.20 and therefore is incompatible with environments that installed numpy<1.20. This bug makes cvxpy unusable.
ecos also needs a pyproject.toml, just like cvxpy has.
I submitted a PR here: https://github.com/embotech/ecos-python/pull/37
But I need someone to kick-off the testing pipeline (@SteveDiamond ? 😄  )
ps: OSQP is also missing this pyproject.toml. Would be good to have it there as well!
Thanks! Rob

@roberthuisman could you also check diffcp and scs to see if they are compatible with numpy<1.20?

@roberthuisman could you try ecos again?

@SteveDiamond
For ecos 2.0.9 i ran two test cases

Databricks runtime 7.3 LTS (https://docs.databricks.com/release-notes/runtime/7.3ml.html)
TLDR; python 3.7, numpy 1.18.1, pandas 1.0.1, cvxpy 1.1.18, cvxopt 1.2.7
Databricks runtime 9.1 LTS
python 3.8, numpy 1.19.2, pandas 1.2.4, cvxpy1.1.18, cvxopt 1.2.7

In both test cases cvxpy cannot find the ECOS installation? I now get the error message;
def _find_candidate_solvers(self,
solver=None,
gp: bool = False):
"""
Find candidate solvers for the current problem. If solver
is not None, it checks if the specified solver is compatible
with the problem passed.
    ---------
    solver : Union[string, Solver, None]
        The name of the solver with which to solve the problem or an
        instance of a custom solver. If no solver is supplied
        (i.e., if solver is None), then the targeted solver may be any
        of those that are installed. If the problem is variable-free,
        then this parameter is ignored.
    gp : bool
        If True, the problem is parsed as a Disciplined Geometric Program
        instead of as a Disciplined Convex Program.

    Returns
    -------
    dict
        A dictionary of compatible solvers divided in `qp_solvers`
        and `conic_solvers`.

    Raises
    ------
    cvxpy.error.SolverError
        Raised if the problem is not DCP and `gp` is False.
    cvxpy.error.DGPError
        Raised if the problem is not DGP and `gp` is True.
    """
    candidates = {'qp_solvers': [],
                  'conic_solvers': []}
    if isinstance(solver, Solver):
        return self._add_custom_solver_candidates(solver)

    if solver is not None:
        if solver not in slv_def.INSTALLED_SOLVERS:

          raise error.SolverError("The solver %s is not installed." % solver)

E               cvxpy.error.SolverError: The solver ECOS is not installed.
scs is fine; see https://github.com/bodono/scs-python/pull/39/files
osqp i just requested a merge https://github.com/osqp/osqp-python/pull/67/files
diffcp i just request a merge https://github.com/cvxgrp/diffcp/pull/52

@roberthuisman can you try again? ecos should be fixed. There was a bug going back to ecos 2.0.8 actually...

@SteveDiamond my issues are resolved with ecos2.0.10. Many thanks for the swift fixes :)


selected rows in list should merge style with inline style

List SelectedRowStyle should be merged with inline styles. This change does two things:

Use ParseStyles for SelectedRowStyle.
Merge modifiers in readStyle instead of inline styles clearing any default modifiers.

I'm not sure if it was intended for SelectedRowStyle to completely override the inline styles. If it was, please feel free to disregard and reject this pull request 😃

Thanks for bringing this up. So I think I intentionally made the colors override instead of merge, but now I'm trying to figure out which is the better solution. I guess it would be good to make that an option for the user? But then we would probably have to add an argument to ParseStyles which might be a breaking change unless we do it with interface{}s. And then we have to figure out whether inline colors should be merged vs overridden like with the paragraph widget etc. So I'm not totally sure what the best method here is :D Probably best to get more opinions.


Changing from v1.3.1 to v2.0.0 breaks builds with AWS S3 storage backend

After the switch to version 2.0.0 of the bazel-remote image our builds started failing.
Example error from bazel:
(03:06:37) ERROR: An error occurred during the fetch of repository 'io_bazel_rules_go':
   Traceback (most recent call last):
	File "/var/lib/buildkite-agent/.cache/bazel/_bazel_buildkite-agent/2c5758fdd18438639bdb6a4b4f0b5e90/external/bazel_tools/tools/build_defs/repo/http.bzl", line 111, column 45, in _http_archive_impl
		download_info = ctx.download_and_extract(
Error in download_and_extract: java.io.IOException: io.grpc.StatusRuntimeException: INVALID_ARGUMENT: Invalid size (must be non-negative): -1

We use --experimental_remote_asset_api.
From the logs of the bazel-remote I don't get any more insights: (the log is
Using S3 backend.
2021/04/05 03:06:16 GRPC GETCAPABILITIES
2021/04/05 03:06:17 S3 CONTAINS buildkite-bazel-cache-anapaya cas.v2/ec/ece669d52998c7a0df2c2380f37edbf4ed8ebb1a03587ed1781dfbececef9b3d OK
2021/04/05 03:06:17 S3 DOWNLOAD buildkite-bazel-cache-anapaya cas.v2/ec/ece669d52998c7a0df2c2380f37edbf4ed8ebb1a03587ed1781dfbececef9b3d OK
2021/04/05 03:06:17 GRPC BYTESTREAM READ: Invalid size (must be non-negative): -1
2021/04/05 03:06:37 GRPC GETCAPABILITIES
2021/04/05 03:06:37 S3 DOWNLOAD buildkite-bazel-cache-anapaya cas.v2/ec/ece669d52998c7a0df2c2380f37edbf4ed8ebb1a03587ed1781dfbececef9b3d OK
2021/04/05 03:06:37 GRPC BYTESTREAM READ COMPLETED blobs/ece669d52998c7a0df2c2380f37edbf4ed8ebb1a03587ed1781dfbececef9b3d/9356
2021/04/05 03:06:37 S3 CONTAINS buildkite-bazel-cache-anapaya cas.v2/6f/6f111c57fd50baf5b8ee9d63024874dd2a014b069426156c55adbf6d3d22cb7b OK
2021/04/05 03:06:37 S3 DOWNLOAD buildkite-bazel-cache-anapaya cas.v2/6f/6f111c57fd50baf5b8ee9d63024874dd2a014b069426156c55adbf6d3d22cb7b OK
2021/04/05 03:06:37 GRPC BYTESTREAM READ: Invalid size (must be non-negative): -1

Any pointers in what I could look into, to figure out what the problem is?
Let me know if I can help it anyway to further debug this.
For now we just switched back to v1.3.1

Hi, I suspect this is a bug when parsing the resource name from the bytestream read request, but the error logs don't show the resource name. I wonder if you could try using a build in the branch from https://github.com/buchgr/bazel-remote/pull/421 and paste the new error message here?
And could you also try to figure out which resource corresponds to the sha256 hash included in the read request- is it an "asset" hash that appears in your WORKSPACE file, or is it something else like the hash of one of the source files in your codebase?

Added a potential fix to the PR: https://github.com/buchgr/bazel-remote/pull/421/commits/1cccbcff552441b6dff8b1bb13f1cb10fbca8241

Thanks a lot for this quick reaction. I'll try it out first thing tomorrow.

Reopening until the fix is verified.

Okay I built the new image and tried to run it. It still seems to fail out:
Logs from image:
Using S3 backend.
2021/04/07 05:35:43 GRPC GETCAPABILITIES
2021/04/07 05:35:44 S3 CONTAINS buildkite-bazel-cache-anapaya cas.v2/ec/ece669d52998c7a0df2c2380f37edbf4ed8ebb1a03587ed1781dfbececef9b3d OK
2021/04/07 05:35:44 S3 DOWNLOAD buildkite-bazel-cache-anapaya cas.v2/ec/ece669d52998c7a0df2c2380f37edbf4ed8ebb1a03587ed1781dfbececef9b3d OK
2021/04/07 05:35:44 GRPC BYTESTREAM READ: Invalid size (must be non-negative): -1 from "blobs/ece669d52998c7a0df2c2380f37edbf4ed8ebb1a03587ed1781dfbececef9b3d/-1"

snippet from bazel output:
(05:35:44) Loading: 0 packages loaded
    Fetching @apple_rules_lint; fetching
(05:35:44) INFO: Repository apple_rules_lint instantiated at:
  no stack (--record_rule_instantiation_callstack not enabled)
Repository rule http_archive defined at:
  /var/lib/buildkite-agent/.cache/bazel/_bazel_buildkite-agent/70271074a3502a131360185dc98c961a/external/bazel_tools/tools/build_defs/repo/http.bzl:336:31: in <toplevel>
(05:35:44) Loading: 0 packages loaded
    Fetching @apple_rules_lint; fetching
(05:35:44) ERROR: An error occurred during the fetch of repository 'apple_rules_lint':
   Traceback (most recent call last):
	File "/var/lib/buildkite-agent/.cache/bazel/_bazel_buildkite-agent/70271074a3502a131360185dc98c961a/external/bazel_tools/tools/build_defs/repo/http.bzl", line 111, column 45, in _http_archive_impl
		download_info = ctx.download_and_extract(
Error in download_and_extract: java.io.IOException: io.grpc.StatusRuntimeException: INVALID_ARGUMENT: Invalid size (must be non-negative): -1 from "blobs/ece669d52998c7a0df2c2380f37edbf4ed8ebb1a03587ed1781dfbececef9b3d/-1"
(05:35:44) Loading: 0 packages loaded
(05:35:44) ERROR: no such package '@apple_rules_lint//lint': java.io.IOException: io.grpc.StatusRuntimeException: INVALID_ARGUMENT: Invalid size (must be non-negative): -1 from "blobs/ece669d52998c7a0df2c2380f37edbf4ed8ebb1a03587ed1781dfbececef9b3d/-1"

This is an asset from an http_archive:
# linter rules
http_archive(
    name = "apple_rules_lint",
    sha256 = "ece669d52998c7a0df2c2380f37edbf4ed8ebb1a03587ed1781dfbececef9b3d",
    urls = [
        "https://github.com/apple/apple_rules_lint/releases/download/0.1.0/apple_rules_lint-0.1.0.tar.gz",
    ],
)

Note that we run with --experimental_remote_asset_api.
Anything more I can do to debug?

One special thing about our setup is that we start a bazel-remote instance for each buildkite agent. So they will initially start with an empty disk cache and will proxy to the S3 backend.

Just to confirm- which commit did you build + test with?

blobs/ece669d52998c7a0df2c2380f37edbf4ed8ebb1a03587ed1781dfbececef9b3d/-1

This suggests that the client has provided an incorrect size. Since this is an "asset", I think the flow must have been:

Client calls FetchBlob
Server (incorrectly) responds with a FetchBlobResponse with a Digest with SizeBytes -1
Client makes a bytestream Read request with the invalid size

We'll need to focus on step 1. In FetchBlob there are two paths where we return a Digest, the first is when we get a cache hit and this checks that the size is non-negative. The second path attempts to download the item, add it to the cache and return its Digest (which includes the size) - I suspect the error is here. One way to debug this would be to add a panic in this second path if the size is negative, then trigger the bug and look through the stack trace to find where the value came from.

Just to confirm- which commit did you build + test with?
I used latest master, i.e. 1cccbcff552441b6dff8b1bb13f1cb10fbca8241

I'll try to reproduce the error on my host and add the panic, so that I get more insight.

I found the issue and have a fix in https://github.com/buchgr/bazel-remote/pull/422
What is not clear to me however, is why the S3 backend doesn't report the correct size here. We only set the size under certain circumstances:
https://github.com/buchgr/bazel-remote/blob/1cccbcff552441b6dff8b1bb13f1cb10fbca8241/cache/s3proxy/s3proxy.go#L254-L256
What is the reasoning behind this ^ ?

Great- I will check the PR later this afternoon.
The reason that the size is not always known is because v2.0.0 introduced compressed CAS blob storage, which is enabled by default. When compressed storage is enabled, blobs are also stored compresed on proxy backends, and then we can't use the s3 object's reported size (since we need the size once uncompressed).


Artisan mod

Why not 2 submodules :)
Changes in scenarios/coffee-roaster-MRV-ecosystem

Looks good for me. Guess we need to rethink the way of how we handle submodules for the future.


Missing warning section in airline

environment

vim: 8.0
vim-airline: 0.9, c7fb175d3565159699885653767214a6aa583ea4
OS: Lubuntu 17.10
Have you reproduced with a minimal vimrc: No, sorry
What is your airline configuration:

  " AIRLINE
   let g:airline_powerline_fonts = 1
   set ttimeoutlen=50 " statusline switches back to normal mode instantly

   let g:mybufstatus = ''
   function! GetBufGitStatus()
       if g:mybufstatus == ''
           return ''
       endif
       return "\ua0[" . g:mybufstatus . "]"
   endfunction
   call airline#parts#define_function('gitflag', 'GetBufGitStatus')
   function! MyAirlineInit()
       let g:airline_section_b = airline#section#create(['hunks','branch','gitflag'])
   endfunction
   autocmd VimEnter * call MyAirlineInit()
   au BufEnter,BufWritePost * if expand('%:p') != "" | let g:mybufstatus = strpart(system('git status --porcelain ' . expand('%:p') ), 0, 2) | else | let g:mybufstatus = '' | endif

if you are using terminal:

terminal: bash
$TERM variable: xterm
color configuration (:set t_Co?): 256

actual behavior
The right hand warning section is missing, so I don't see warnings anymore (for example the whitespace warnings). I'm not sure when the problem first manifested.

expected behavior
The following screenshot is from my other Lubuntu machine (same version, same configuration). The only difference is that my other machine has an older airline version:
0.8, 72e5f04f7c422e21cb6f6856c4e94cef25ea2288

Any ideas on what I can try to restore it?

Actually, it seems that it also doesn't show the git status, nor in code shows in which function I'm currently in. I apologize for missing this, I only noticed now.

it looks like you have the skip_empty_section variable set.
As to why this doesn't show up on section_b, I don't know. Have you verified, that the variable  g:mybufstatus is actually being set?

Sorry, it was my bad, I had mistyped something in the configuration. My apologies.


Failed to pull OCI bundle from private registry

Expected Behavior
When using OCI bundles feature, I expect a Task Run with an OCI bundle Task reference will use the Docker Config Kubernetes secret (referenced in the Service Account) to access a private registry.
Actual Behavior
I receive a 401 error.
status:
  completionTime: "2023-03-15T17:44:43Z"
  conditions:
  - lastTransitionTime: "2023-03-15T17:44:43Z"
    message: 'failed to get task: error requesting remote resource: error getting
      "bundleresolver" "test-pipeline/bundles-d672fb3f2ec09ff198790535e8d424d8": cannot
      retrieve the oci image: GET https://api.digitalocean.com/v2/registry/auth?scope=repository%!A(MISSING)my-organisation%!F(MISSING)my-repository%!A(MISSING)pull&service=registry.digitalocean.com:
      unexpected status code 401 Unauthorized: {"id": "Unauthorized", "message": "Unable
      to authenticate you" }'
    reason: TaskRunResolutionFailed
    status: "False"
    type: Succeeded

Steps to Reproduce the Problem
Apply the following:
kind: Namespace
apiVersion: v1
metadata:
  name: test-pipeline
---
apiVersion: v1
data:
    .dockerconfigjson: yoursecrethereinbase64encoding
kind: Secret
metadata:
    name: docker-config-json
    namespace: test-pipeline
type: kubernetes.io/dockerconfigjson
---
kind: ServiceAccount
apiVersion: v1
metadata:
  name: test-pipeline
  namespace: test-pipeline
secrets: []
imagePullSecrets:
  - name: docker-config-json

kubectl create a TaskRun (file adapted from example):
apiVersion: tekton.dev/v1
kind: TaskRun
metadata:
  generateName: test-taskrun-
  namespace: test-pipeline
spec:
  serviceAccountName: test-pipeline
  taskRef:
    kind: Task
    params:
    - name: bundle
      value: private.registry.example.org/repository:tag
    - name: name
      value: yourtaskname
    - name: kind
      value: task
    - name: serviceAccountName
      value: test-pipeline
    resolver: bundles

The above works when the bundle is in a public registry, but not when the bundle is in a private registry.
The secret works as I am able to docker login and docker pull the bundle using the secret.
I am also able to run TektonTasks with an inline taskSpec. The Task Step referenced an image from the same private registry. Tekton is able to mount the same secret (which has entire registry read scope) and have Kubernetes pull the image to run in the Task Step.  This issue therefore differs slightly from previously reported issues https://github.com/tektoncd/pipeline/issues/3604 and https://github.com/tektoncd/pipeline/issues/6276.
A difficulty I have had in attempting to debug this is that the error message does not indicate which service account, secret name, or namespace, it has attempted to mount secrets from.
As part of this debugging, I tried the above TaskRun with non-existent/invalid service account names. This also results in a 401 error. (unrelated to the core issue, but I would have expected a different error related to the non-existent service account.)
Additional Info

Kubernetes version:
Output of kubectl version:

Client Version: v1.26.2
Server Version: v1.25.4

Tekton Pipeline version:
Output of tkn version or kubectl get pods -n tekton-pipelines -l app=tekton-pipelines-controller -o=jsonpath='{.items[0].metadata.labels.version}'

v0.45.0

Feature flags ConfigMap

apiVersion: v1
data:
  await-sidecar-readiness: "true"
  custom-task-version: v1beta1
  disable-affinity-assistant: "false"
  disable-creds-init: "false"
  enable-api-fields: alpha
  enable-provenance-in-status: "false"
  enable-tekton-oci-bundles: "true"
  enforce-nonfalsifiablity: none
  require-git-ssh-secret-known-hosts: "false"
  resource-verification-mode: skip
  running-in-environment-with-injected-sidecars: "true"
  send-cloudevents-for-runs: "false"
kind: ConfigMap
metadata:
  name: feature-flags
  namespace: tekton-pipelines

Bundle Resolver ConfigMap

apiVersion: v1
data:
  default-kind: task
  default-service-account: test-pipeline
kind: ConfigMap
  name: bundleresolver-config
  namespace: tekton-pipelines-resolvers

Resolver Feature Flags ConfigMap

apiVersion: v1
data:
  enable-bundles-resolver: "true"
  enable-cluster-resolver: "true"
  enable-git-resolver: "true"
  enable-hub-resolver: "true"
kind: ConfigMap
metadata:
  name: resolvers-feature-flags
  namespace: tekton-pipelines-resolvers

Failed to pull OCI bundle from private registry
Same problem.

I follow the doc and tekton taskrun is not working for me .
https://tekton.dev/docs/pipelines/bundle-resolver/#parameters
My  yamls:
apiVersion: v1
kind: ServiceAccount
metadata:
  name: tekton
imagePullSecrets:
  - name: tekton
---
apiVersion: tekton.dev/v1beta1
kind: TaskRun
metadata:
  name: hello-task-run-private-bundle
spec:
  serviceAccountName: tekton
  taskRef:
    resolver: bundles
    params:
      - name: bundle
        value: ghcr.io/liangyuanpeng/tekton:task-hello
      - name: name
        value: hello
      - name: kind
        value: task
      - name: serviceAccount
        value: tekton

This is the error log:
           * failed to get task: resource request in progress
  Warning  Failed         14s  TaskRun  failed to get task: error requesting remote resource: error getting "bundleresolver" "default/bundles-95376cec82c97b72a22333a02a4b76de": cannot retrieve the oci image: GET https://ghcr.io/token?scope=repository%!A(MISSING)liangyuanpeng%!F(MISSING)tekton%!A(MISSING)pull&service=ghcr.io: UNAUTHORIZED: authentication required
  Warning  InternalError  14s  TaskRun  1 error occurred:
           * failed to get task: error requesting remote resource: error getting "bundleresolver" "default/bundles-95376cec82c97b72a22333a02a4b76de": cannot retrieve the oci image: GET https://ghcr.io/token?scope=repository%!A(MISSING)liangyuanpeng%!F(MISSING)tekton%!A(MISSING)pull&service=ghcr.io: UNAUTHORIZED: authentication required

Tekton Pipeline version:
0.44.0

So looking at https://tekton.dev/docs/pipelines/auth/ a bit, Tekton usually looks at attached secrets from the ServiceAccount, and not necessarily the imagePullSecret.
Does it work with the following ?
secrets:
- name: tekton
imagePullSecrets:
- name: tekton


DAT-17566

fix: .github/workflows/run-tests.yml: OpenJDK 8 is not supported on macos-14+ (M1).
slack chat : https://liquibase.slack.com/archives/CK9JJEENB/p1714052752548829?thread_ts=1714052222.916469&cid=CK9JJEENB
Impact

[x] Bug fix (non-breaking change which fixes expected existing functionality)
[ ] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Things to be aware of

Things to worry about

Additional Context

Did you do a find and replace of macos-latest with macos-13? Just making sure there aren't more places that we missed here.

anything that had macos-latest and java-8 have been replaced. Others which are using just macos-latest are the same.


chore(Deps) dependency next to v14.2.1

This PR contains the following updates:

Package
Type
Update
Change

next (source)
dependencies
patch
14.2.0 -> 14.2.1

Release Notes

vercel/next.js (next)
v14.2.1
Compare Source

[!NOTE]
This release is backporting bug fixes. It does not include all pending features/changes on canary.

Core Changes

use pathToFileUrl to make esm import()s work with absolute windows paths (#64386) @sokra

Credits
Huge thanks to @sokra for helping!

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

🦙 MegaLinter status: ✅ SUCCESS

Descriptor
Linter
Files
Fixed
Errors
Elapsed time

✅ COPYPASTE
jscpd
yes

no
1.12s

✅ CSS
stylelint
1
0
0
1.34s

✅ JAVASCRIPT
standard
1
0
0
1.54s

✅ JSON
eslint-plugin-jsonc
3
0
0
3.44s

✅ JSON
npm-package-json-lint
yes

no
0.45s

✅ JSON
prettier
3
0
0
0.34s

✅ JSON
v8r
3

0
8.89s

✅ MARKDOWN
markdownlint
8
0
0
0.6s

✅ MARKDOWN
markdown-link-check
8

0
2.16s

✅ MARKDOWN
markdown-table-formatter
8
0
0
0.36s

✅ REPOSITORY
gitleaks
yes

no
0.26s

✅ REPOSITORY
git_diff
yes

no
0.01s

✅ REPOSITORY
grype
yes

no
9.97s

✅ REPOSITORY
secretlint
yes

no
1.24s

✅ REPOSITORY
trivy
yes

no
6.18s

✅ REPOSITORY
trivy-sbom
yes

no
1.1s

✅ YAML
prettier
2
0
0
0.58s

✅ YAML
v8r
2

0
4.95s

See detailed report in MegaLinter reports
MegaLinter is graciously provided by


添加新的Set Map以支持unhashable type

Can you pull updates from main? The ci thing is not running

let me know if you are done with this, ill tackle the rest

let me know if you are done with this, ill tackle the rest

yes, I'm done.

If I want to add some tests about new Set and Map, I need expose them. What should I do? Should I add them to init.py?

Codecov Report

Merging #7 (c01f55c) into main (43cd939) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main        #7   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        11    +1
  Lines          984      1077   +93
  Branches       220       233   +13
=========================================
+ Hits           984      1077   +93

Impacted Files
Coverage Δ

types_linq/enumerable.py
100.00% <100.00%> (ø)

types_linq/lookup.py
100.00% <100.00%> (ø)

types_linq/util.py
100.00% <100.00%> (ø)

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 43cd939...c01f55c. Read the comment docs.


[front] fix: Reset input value once file is added

fixes https://github.com/dust-tt/dust/issues/5937
Description
just reset input value so that reuploading the same file again trigger the handleChange again
Risk

Deploy Plan
deploy front

This changes the current behaviour: we can now upload the same file multiple times. I guess we want to keep the old behaviour and only be able to upload the same file once?

yes, good catch, I thought I tested that but apparently not ..


Database manager for driver mariadb is not registered

Bug description
creating a record inside database doesn't work for mariadb because database manager driver for mariadb is not registered.
Stancl\Tenancy\Exceptions\DatabaseManagerNotRegisteredException
Steps to reproduce

start fresh laravel project:
set db connection to mariadb in .env:
DB_CONNECTION=mariadb
install stancl/tenancy normally
create custom tenant model
`namespace App\Models;

use Stancl\Tenancy\Database\Models\Tenant as BaseTenant;
use Stancl\Tenancy\Contracts\TenantWithDatabase;
use Stancl\Tenancy\Database\Concerns\HasDatabase;
use Stancl\Tenancy\Database\Concerns\HasDomains;
use Stancl\Tenancy\DatabaseConfig;
use Illuminate\Database\Eloquent\Model;
use Truck;
use Driver;
use Staff;
use Warehouse;
class Company extends BaseTenant implements TenantWithDatabase
{
//
public function database() : DatabaseConfig
{
    return new DatabaseConfig($this);
}
public function drivers() {
    return $this->hasMany(Driver::class);
}

public function staff() {
    return $this->hasMany(Staff::class);
}

public function warehouses() {
    return $this->hasMany(Warehouse::class);
}

public function trucks(){
    return $this->hasMany(Truck::class);
}

}`

use tinker tool to instantiate an object of the custom tenant model
$company1 = App\Models\Company::create(['id' => 0, 'name' => 'my_company', 'special_code' => 'foode']);

Expected behavior
An Object of The Custom Tenant model is successfully instantiated and inserted into database.
Laravel version
11.28.0
stancl/tenancy version
3.8

Try using the MySQL DB manager for 'mariadb' in the tenancy config.


OCM-5406 | fix: set att to computed so that it may reset to null

Related issue: OCM-5406

@gdbranco we had the same issue with the attribute labels in mp.
We decided not to set it to computed as a solution but to deal with that in the provider itself
@sagidayan can you help here?

Can't we just set nil in case of empty string?

@tsorya then we will need to set the attrs to be computed. And I am not convinced that they need to.

The problem is that we can't sent nil to ocm as it does nothing so we have nil till those fields were not set but in order to empty them we need to send empty string. That will always create case that we will not have match between ocm and tf like deleting those fields from main.tf that will actually always have empty string value in ocm.
I agree with Sagi, we should just block setting empty string

@gdbranco Can you please add the proper tests to the subsystem (i.e passing an empty string should result in an error)

Oh, and please change the PR title and commit message... 😉  They are not reflecting the change now.

/lgtm
/approve

/cherry-pick v1.5.0-branch


Text Analytics not working as expected if text contains 'case ('

Text Analytics not working as expected if text contains 'case ('.
For example using the form in https://azure.microsoft.com/en-us/services/cognitive-services/text-analytics/#features:
The text "Azure is global. case ( Azure is local." returns nothing.
The text "Azure is global. Azure is local." returns "Azure"

moved to
https://cognitive.uservoice.com/forums/555922-text-analytics/suggestions/41042683-text-analytics-not-working-as-expected-if-the-text


OpenSSL misinterprets BER constructed BIT STRINGs

(ASN.1 snippets use der-ascii syntax, from https://github.com/google/der-ascii.)
Summary
OpenSSL's decoding of constructed BIT STRINGs does not interoperate with the specification or other implementations. It injects extra bits into the resulting value due to flipping the order of padding and concatenation.
Details
BER, unlike DER, has a number of overcomplicated encodings for streaming, notably constructed strings. String types are normally encoded as primitive elements, such as OCTET_STRING { "hello, world" }. BER additionally allows a constructed string where the outer element is constructed and inner elements are concatenated:
[OCTET_STRING CONSTRUCTED] {
  OCTET_STRING { "hello, " }
  OCTET_STRING { "world" }
}

(This, when combined with indefinite-length encoding, lets the encoder stream a large string without knowing the length ahead of time, though none of the uses in OpenSSL actually rely on streaming.) The rules for most strings are described in 8.7.3 of X.690 (08/2015). OpenSSL implements this in asn1_collect.
BIT STRINGs are different. A BIT STRING is a sequence of bits, possibly not a multiple of 8 of them. They are encoded by appending 0 <= N <= 7 bits up to multiple of 8. N is then prepended as a byte. Thus BIT_STRING { `00abcd` } is a 16-bit BIT STRING (N = 0), and BIT_STRING { `04abc0` } is a 12-bit BIT STRING (N = 4).
BIT STRINGs can also be constructed, and OpenSSL runs the same asn1_collect logic for them.
https://github.com/openssl/openssl/blob/master/crypto/asn1/tasn_dec.c#L731-L756
But constructed BIT STRINGs work differently. See 8.6.3 and 8.6.4 of X.690 (08/2015). Each segment has its own leading byte, so a streaming encoder does not need to commit to the length mod 8. (Although only the final segment can have padding bits. This means those bytes are redundant, but a streaming receiver doesn't know the segment is the last one.) OpenSSL instead flips the order: it concatenates first, then resolves padding. That is, the 12-bit string BIT_STRING { `04abc0` } can be encoded as:
[BIT_STRING CONSTRUCTED] {
  BIT_STRING { `00ab` }
  BIT_STRING { `04c0` }
}

But OpenSSL misinterprets it as the 24-bit string BIT_STRING { `00ab04c0` }. Instead, OpenSSL accepts the following as the 12-bit input:
[BIT_STRING CONSTRUCTED] {
  BIT_STRING { `04ab` }
  BIT_STRING { `c0` }
}

This is not a valid BER BIT STRING encoding at all because c0 is greater than 7. See also the test program below. It compares OpenSSL's behavior with that of NSS, which seems to handle it correctly.
Recommendation
Since this BIT STRING handling doesn't match spec or other implementations, and will misinterpret valid inputs (quite risky in a security-sensitive library), OpenSSL should either switch to the correct handling, or reject constructed BIT STRINGs altogether. I would suggest the latter (i.e. don't support constructed BIT STRINGs). BER is unnecessarily complicated, and this bug means constructed BIT STRINGs aren't usable in anything which needs to work with existing OpenSSLs anyway.
As far as I can tell, OpenSSL will never produce constructed BIT STRINGs, so I don't think the current behavior is particularly indicative compatibility-wise. Indeed most uses of cryptography like X.509 require DER, so using a BER parser at all is too loose. PKCS#7 and PKCS#12 blobs sadly do often use BER, but BIT STRINGs only carry public keys and signature values, neither of which are large enough to bother streaming. (I would argue the entirety of PKCS#7 and PKCS#12 aren't worth streaming in the first place. Indeed OpenSSL doesn't stream them.) This is probably why no one's noticed this before.
Test program

/*
Input:   030300abcd
OpenSSL: num_bits=16 abcd
NSS:     num_bits=16 abcd

Input:<PHONE_NUMBER>ab030200cd
OpenSSL: num_bits=24 ab00cd
NSS:     num_bits=16 abcd

Input:<PHONE_NUMBER>ab0301cd
OpenSSL: num_bits=16 abcd
NSS:     invalid input

Input:   030304abc0
OpenSSL: num_bits=12 abc0
NSS:     num_bits=12 abc0

Input:<PHONE_NUMBER>ab030204c0
OpenSSL: num_bits=24 ab04c0
NSS:     num_bits=12 abc0

Input:<PHONE_NUMBER>ab0301cd
OpenSSL: num_bits=12 abc0
NSS:     invalid input

Input:<PHONE_NUMBER>
OpenSSL: num_bits=16 0000
NSS:     num_bits=16 0000

Input:<PHONE_NUMBER>0003020000
OpenSSL: num_bits=24 000000
NSS:     num_bits=16 0000

Input:<PHONE_NUMBER>00030100
OpenSSL: num_bits=16 0000
NSS:     num_bits=8 00
*/

#include <assert.h>
#include <stddef.h>
#include <stdio.h>

#include <openssl/asn1.h>
#include <secasn1.h>

static void print_bytes(const unsigned char *in, size_t len) {
  for (size_t i = 0; i < len; i++) {
    printf("%02x", in[i]);
  }
}

static void test_openssl(const unsigned char *in, size_t len) {
  ASN1_BIT_STRING *str = d2i_ASN1_BIT_STRING(NULL, &in, len);
  if (!str) {
    printf("OpenSSL: invalid input\n");
    return;
  }
  assert(str->flags & ASN1_STRING_FLAG_BITS_LEFT);
  int num_bits = 8 * ASN1_STRING_length(str) - (str->flags & 0x7);
  printf("OpenSSL: num_bits=%d ", num_bits);
  print_bytes(ASN1_STRING_get0_data(str), ASN1_STRING_length(str));
  printf("\n");
  ASN1_STRING_free(str);
}

static void test_nss(const unsigned char *in, size_t len) {
  SECItem input;
  input.type = siBuffer;
  input.data = (unsigned char *)in;
  input.len = len;

  SECItem output;
  output.type = siBuffer;
  output.data = NULL;
  output.len = 0;

  if (SEC_ASN1DecodeItem(NULL, &output, SEC_ASN1_GET(SEC_BitStringTemplate),
                         &input) != SECSuccess) {
    printf("NSS:     invalid input\n");
    return;
  }

  /* NSS returns BIT STRINGs as SECItems whose length is measured in bits
   * rather than bytes. */
  printf("NSS:     num_bits=%u ", output.len);
  print_bytes(output.data, (output.len + 7) / 8);
  printf("\n");
  PORT_Free(output.data);
}

static void test(const unsigned char *in, size_t len) {
  printf("Input:   ");
  print_bytes(in, len);
  printf("\n");
  test_openssl(in, len);
  test_nss(in, len);
  printf("\n");
}

int main() {
  // BIT_STRING { `00abcd` }
  static const unsigned char kDER0[] = {0x03, 0x03, 0x00, 0xab, 0xcd};
  test(kDER0, sizeof(kDER0));

  // Valid BER constructed version of the above:
  // [BIT_STRING CONSTRUCTED] {
  //   BIT_STRING { `00ab` }
  //   BIT_STRING { `00cd` }
  // }
  static const unsigned char kBER0[] = {0x23, 0x08, 0x03, 0x02, 0x00,
                                        0xab, 0x03, 0x02, 0x00, 0xcd};
  test(kBER0, sizeof(kBER0));

  // Invalid BER constructed version of the above:
  // [BIT_STRING CONSTRUCTED] {
  //   BIT_STRING { `00ab` }
  //   BIT_STRING { `cd` }
  // }
  static const unsigned char kInvalidBER0[] = {0x23, 0x07, 0x03, 0x02, 0x00,
                                               0xab, 0x03, 0x01, 0xcd};
  test(kInvalidBER0, sizeof(kInvalidBER0));

  // BIT_STRING { `04abc0` }
  static const unsigned char kDER1[] = {0x03, 0x03, 0x04, 0xab, 0xc0};
  test(kDER1, sizeof(kDER1));

  // Valid BER constructed version of the above:
  // [BIT_STRING CONSTRUCTED] {
  //   BIT_STRING { `00ab` }
  //   BIT_STRING { `04c0` }
  // }
  static const unsigned char kBER1[] = {0x23, 0x08, 0x03, 0x02, 0x00,
                                        0xab, 0x03, 0x02, 0x04, 0xc0};
  test(kBER1, sizeof(kBER1));

  // Invalid BER constructed version of the above:
  // [BIT_STRING CONSTRUCTED] {
  //   BIT_STRING { `04ab` }
  //   BIT_STRING { `cd` }
  // }
  static const unsigned char kInvalidBER1[] = {0x23, 0x07, 0x03, 0x02, 0x04,
                                               0xab, 0x03, 0x01, 0xcd};
  test(kInvalidBER1, sizeof(kInvalidBER1));

  // BIT_STRING { `000000` }
  static const unsigned char kDER2[] = {0x03, 0x03, 0x00, 0x00, 0x00};
  test(kDER2, sizeof(kDER2));

  // Valid BER constructed version of the above:
  // [BIT_STRING CONSTRUCTED] {
  //   BIT_STRING { `0000` }
  //   BIT_STRING { `0000` }
  // }
  static const unsigned char kBER2[] = {0x23, 0x08, 0x03, 0x02, 0x00,
                                        0x00, 0x03, 0x02, 0x00, 0x00};
  test(kBER2, sizeof(kBER2));

  // Invalid BER constructed version of the above:
  // [BIT_STRING CONSTRUCTED] {
  //   BIT_STRING { `0000` }
  //   BIT_STRING { `00` }
  // }
  static const unsigned char kInvalidBER2[] = {0x23, 0x07, 0x03, 0x02, 0x00,
                                               0x00, 0x03, 0x01, 0x00};
  test(kInvalidBER2, sizeof(kInvalidBER2));
}

(I would argue the entirety of PKCS#7 and PKCS#12 aren't worth streaming in the first place. Indeed OpenSSL doesn't stream them.)

The history is that we implemented streaming exactly because there were reports of very large PKCS#7 messages

Ah, I see, there's some S/MIME-specific streaming filtering logic in some separate BIO and only does streaming encode of particular fields via callbacks. I don't see a streaming decode, though perhaps I've missed it.
Regardless, this bug report is about the core ASN.1 parser decoder, which isn't resumable, and the S/MIME-specific logic does not appear to select any of the BIT STRING fields, just a few OCTET STRINGs, as one would expect.

closing for inactivity.  If there is any desire to pick this back up, please feel free to reopen

This bug is still present, and I see you all previously judged it "severity: important". Is there more information you need on it? The information I wrote in September 2020 was pretty detailed, and included a suggested remediation. The fix should be quite simple, as it's just a matter of making the parser reject constructed BIT STRINGs. A fuller fix would be to parse them correctly, but seeing as OpenSSL has never parsed them correctly (i.e. it would misinterpret them with the wrong value), rejecting it is the more secure option.

(Note that reopening bugs is not something that external folks, myself included, can do.)

I'm happy to re-open the issue for you.  I'm honestly not sure if there is any additional information needed.  The only thing I'm sure of is that, even though the issue might be present, and marked as important, no one has considered it important enough to address in the last 4 years, which makes me question the label.  But if you're interested in continuing it, I'll gladly add it to our review queue to consider

I mean, I filed the bug because I noticed it while working on our parser, and fixed BoringSSL accordingly. I don't have any immediate plans to fix it in OpenSSL, but I figured it was worth taking the time to write it up for you all. Completely misinterpreting some input seemed worth passing along.
If you all now think this is working as intended and not worth fixing, I mean, I obviously think that's not a good stance for a security-critical library, but ah well. 😛 I replied because it sounds like this was closed not because you all took a stance here, and more out of closing bugs that you all forgot about.


Make it possible to decode streams of messages

Exported the 'Decoder' class.
Added 'bytesRemaining' property to decoder class.
Added a wrapper method for 'Decoder.parse' in order to keep the
'bytesRemaining' from updating on parse failure.

Looks neat, do you still want this merged? Not sure how I missed this PR.

Feel free to merge it, but the msgpack spec has evolved and now has a way to distinguish between byte and text strings, so I wouldn't use this library. https://github.com/mcollina/msgpack5 implements the latest version.
In any case, there's a way to design a msgpack parser that is fully reentrant without backtracking(rolling back to previous parse position in case of missing data). This is one of the reasons I have created https://github.com/tarruda/libmpack, but didn't get the time to put together a node.js binding yet.


feat: create kb article for github raw proxies in idp

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.

Creating a new folder for sharing proxy configurations
Added an article to guide users in creating a backend proxy for raw github files

Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66d72eed5cc75269074a268b--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66d73142a7edd4650970d804--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66d73a466e9fc37642225703--harness-developer.netlify.app

Thank you Riley! First non-IDP team contributor to our docs :')

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66d76084983990abdd470aea--harness-developer.netlify.app


How to find the serial number

Hello,
I want to know what is serial number on my canon printer.

Um... Let me refer you to Google mate... http://lmgtfy.com/?q=What+is+Canon+printer+support+customer+care+number


[GDScript 2.0] "Compiler bug: unresolved assign" in setters

Godot version
master d6f972fad45d28222b1437a047241dd74f0d5b90
System information
Windows 10
Issue description
A compiler error is reported when an assignment is made in a property setter using a method from a Resource.
modules\gdscript\gdscript_byte_codegen.cpp:826 - Compiler bug: unresolved assign.

The same error is reported by the debugger
E 0:00:01:0183   GDScriptByteCodeGenerator::write_assign_with_conversion: Compiler bug: unresolved assign.
  <C++ Source>   modules\gdscript\gdscript_byte_codegen.cpp:826 @ GDScriptByteCodeGenerator::write_assign_with_conversion()

Identical code in a function that isn't a setter doesn't cause the error.
As long as the Resource is correct (i.e. is of the correct class) the code runs fine.
Steps to reproduce
I attached a small sample with two scripts.
Playing the scene will show the error in the Debugger.
Editing and saving the MyTestScene.gd script will also show the error in the Output tab as well as in a notification:

Minimal reproduction project
godot4test.zip

Hi, I would like to begin contributing to Godot, tried to find the problem of this issue and I likely found it.
Here's a pull request where a possible solution is described.
I'm a newbie in Open Source and Godot source code, so would like to see any recommendations from experienced devs.


Roll engine 3c51a7bfff77..db4d6836acc4 (1 commits)

https://github.com/flutter/engine/compare/3c51a7bfff77...db4d6836acc4
git log 3c51a7bfff776e9e3011384f4ff09c13a697c6b6..db4d6836acc4f1df66a538c79e55f4952c923e2a --no-merges --oneline
db4d6836a Roll src/third_party/skia 857c9f955edb..8063f6cca025 (3 commits) (flutter/engine#9679)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (jsimmons@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/5217672806006784

Commit queue failed; closing this roll.


Parrot Explosion!

Party Parrot had been getting lots on new parrots recently.   I've set things up so when a new parrot is added it updates a YAML here:  http://cultofthepartyparrot.com/parrotparty.yaml
What's the best way to get that synced into this repo?
Thanks!

@jmhobbs The easiest way is to periodically submit updated YAML files.

@jmhobbs You are a gentleman and a scholar.

Slack says:

Custom emoji names can only contain lower case letters, numbers, dashes and underscores.

@jmhobbs Could you update it so the names in http://cultofthepartyparrot.com/parrotparty.yaml meet that requirement?
If you wanted to keep the existing names, perhaps put them in fullname: and have the Slack-compatible name in name:.

Absolutely! They are now based on the gif filename.

Merged the latest I think - glad to keep seeing PRs! Party on, parrrot!


DOC: Instructions on running tests locally

It would be nice if there were some instructions on running tests locally for ophyd. One can follow along with .travis.yml file but this is a little crude. This also might want to take place after the proposed epics_on_travis change to the ophyd CI

@mrakitin worked on something for this -- I think it was this: https://nsls-ii.github.io/ophyd/docker.html
We should iterate on that.


Docs request: how to place files in dist/

I have a publish GitHub Action that looks like this, using https://github.com/hynek/build-and-inspect-python-package/tree/v2.9.0:
    steps:
      - uses: actions/checkout@v4
      - uses: hynek/build-and-inspect-python-package@v2
      - uses: pypa/gh-action-pypi-publish@release/v1

However, this does not work, the gh-action-pypi-publish fails due to not finding the dist/ (CI run):
Traceback (most recent call last):
  File "/app/print-pkg-names.py", line 30, in <module>
    pkg_name for file_path in packages_dir.iterdir() if
                              ^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/pathlib.py", line 1056, in iterdir
    for name in os.listdir(self):
                ^^^^^^^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory: '/github/workspace/dist'
Warning: Trusted Publishers allows publishing packages to PyPI from automated environments like GitHub Actions without needing to use username/password combinations or API tokens to authenticate with PyPI. Read more: https://docs.pypi.org/trusted-publishers

Warning:  It looks like there are no Python distribution packages to publish in the directory 'dist/'. Please verify that they are in place should you face this problem.
ERROR    InvalidDistribution: Cannot find file (or expand pattern): 'dist/*'

Can you add docs on how to pipe the output of hynek/build-and-inspect-python-package into a publish step such as pypa/gh-action-pypi-publish or uv publish?
Alternately, can you add an input to let one specify where to place the built wheel and sdist?

Doing something like this does not work:
      - id: build
        uses: hynek/build-and-inspect-python-package@v2
      - uses: pypa/gh-action-pypi-publish@release/v1
        with:
          packages-dir: ${{ steps.build.outputs.dist }}
          password: ${{ secrets.PYPI_API_TOKEN }}

The gh-action-pypi-publish fails (CI run):
Traceback (most recent call last):
  File "/app/print-pkg-names.py", line 30, in <module>
    pkg_name for file_path in packages_dir.iterdir() if
                              ^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/pathlib.py", line 1056, in iterdir
    for name in os.listdir(self):
                ^^^^^^^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory: '/tmp/baipp/dist'
Warning: Trusted Publishers allows publishing packages to PyPI from automated environments like GitHub Actions without needing to use username/password combinations or API tokens to authenticate with PyPI. Read more: https://docs.pypi.org/trusted-publishers

Warning:  It looks like there are no Python distribution packages to publish in the directory '/tmp/baipp/dist/'. Please verify that they are in place should you face this problem.
ERROR    InvalidDistribution: Cannot find file (or expand pattern):
         '/tmp/baipp/dist/*'

This was what needed to be done btw:
      - id: build
        uses: hynek/build-and-inspect-python-package@v2
      - name: Download built artifact to dist/
        uses: actions/download-artifact@v4
        with:
          name: ${{ steps.build.outputs.artifact-name }}
          path: dist
      - uses: pypa/gh-action-pypi-publish@release/v1

It would be nice to document this if possible

Hmmm, this might be GHA tightening the protection between steps and not allowing actions to access each others /tmp.
It's a bit extra, but try adding:
      - name: Download packages built by build-and-inspect-python-package
        uses: actions/download-artifact@v4
        with:
          name: Packages
          path: dist

before the publish step.

Ah you figured it out yourself. Another option would've been setting the dist output:
      - id: build
        uses: hynek/build-and-inspect-python-package@v2
        with:
          dist: dist

That might be simpler for your usecase.

Hello @hynek thanks for getting back. I like your suggestion of with: dist: dist, but is that possible?
From https://github.com/hynek/build-and-inspect-python-package/blob/v2.9.0/action.yml#L9-L25, I don't see dist being an input argument, unless I am missing something

Yeah sorry, you're right, I misread. I wrote that before 5am my time. ;)
Maybe something like that should be added. Not to make excuses, but I'm confident that the reason for packages-dir: ${{ steps.build.outputs.dist }} not working is that GHA changed the rules and I pray they don't change it any further.
This will take some testing, but it really should work, I think.

Hi @hynek yes I agree with everything you're saying. Thanks again for making this, we are going to roll this GHA universally for all packages within FutureHouse 🚀
I made a brief docs PR documenting this workflow, feel free to not accept it if you want to do something different


Fix maxSession directive in nodeConfig example

There seems to be a typo. The directive is actually maxSession: https://github.com/SeleniumHQ/selenium/blob/4c39e9d0894068dcdb29094217510b2fed892143/java/server/src/org/openqa/grid/common/defaults/DefaultNodeWebDriver.json#L30 etc.
Using maxSessions does not affect the actual node configuration at all.

Good catch @OndraM, thanks for your contribution!


Add support for dashed line

Hi @roipeker,
Would there be an easy way to add support to draw dashed lines with Graphx ?
I remember having a similar ask when I was working with Graphic and the PR addressing it might give some pointers on how to implement it relatively easily.

https://github.com/entronad/graphic/commit/33385a917c676f66c1521f5f317213ac145dabeb

The gist of it being:

Hi @tlvenn, I forgot to reply earlier today, sorry.
You can totally use that library.
Graphics class has getPaths() and drawPath() as input/output for raw Paths.
source:

Output:

Good luck!

Feel free to reopen the issue if the solution doesn't fit your needs.

Hi @roipeker ,
Thanks a lot for the example, that is super useful !
The implementation for this within Graphics is simply a wrapper leveraging path_drawing lib. Could we do something similar so that we could draw dashed paths without importing an entire graph lib ? I might be able to contribute this given the example you provided.
Do you have a preference on how we should provide this within Graphx ?

Probably as a lineStyle parameter?, although flash had no support for it i believe (api wise).
Anyway, depending on the level of complexity that u need, for straight dashed lines is much more performant to use shaders i think (gradients or image) than compute path segmentation.

Ya computing path segmentations seems pretty painful and expensive.. And yes while looking at it, I also discovered that Flash had no native support for this and people came with some pretty crazy stuff to do path segmentations on their own.
Right now my use cases is all around very simple straight lines so probably the path segmentation is relatively simple but would be curious to see how you would approach it using shaders if you can spare a little time.
Thanks in advance.

Btw, by shaders i meant the classic Gradient or Image Shaders from flutter. Not the experimental GLSL (that will be probably removed with Impeller).
Give me a few mins and i will prepare a sample.


feat: add delegate endpoints

Overview
This adds the add, edit, delete and delete (all?) endpoints to the SDK as well as their types. There are still a few TODOs:

It seems as though the DELETE method is not allowed by the CORS policy. As such, the delete functions aren't working and I can't confirm the return types (although the typed ones are taken from the docs).
The get/add functions have a union return type for the '/v1/chains/{chainId}/delegates' endpoint that needs to be fixed.

It seems as though the DELETE method is not allowed by the CORS policy. As such, the delete functions aren't working and I can't confirm the return types (although the typed ones are taken from the docs).

Does the backend enforce it on purpose, or did they forget to enable it?

Does the backend enforce it on purpose, or did they forget to enable it?

I haven't consulted them yet. This is a draft that we're going to come back to.

note: last commit was to try the HTTP DELETE method for https://github.com/safe-global/safe-client-gateway/blob/08655807c78b81f79cb4be91a1bc95f95d09ca37/src/routes/delegates/routes.rs#L36-L49 and https://github.com/safe-global/safe-client-gateway/blob/08655807c78b81f79cb4be91a1bc95f95d09ca37/src/routes/delegates/routes.rs#L51-L71

I'm going to close this. We can use this as reference if we revisit it later.


Fix link markdown markup

Markdown link to ansible settings is not formatted correctly in large deployments documentation file.

CLA signed
/meow


Add more regex for Go http package

Added extra regex as this pattern also hitting our test servers.
Link: https://golang.org/pkg/net/http/

Coverage remained the same at 100.0% when pulling 98007d77de0829443324ee5533463abe66fe789f on ayumi-cloud:Go-http-package into c65ad41ee66cc126a7740d929aab706dd7304c1c on WhichBrowser:master.


Book Shelf  Icon

Icon Request

Icon name:  Book Shelf
Tags of this icon:  Will be used when designing a books library
Use case: Indicate a books library, categories of books
Screenshots of similar icons: don't have anything.

https://remixicon.com/icon/book-shelf-line

The icon you recommended has been launched. Don't forget to star our repo if you like our project.


Charset of response is not UTF-8 on Windows

In standalone mode, when i post utf-8 mock from postman, my browser show response with windows charset.
I don't find solutions even with transformers to force utf-8 data in response.
My data is based on UTF-8
Java file.encoding : cp1252
it's not normal, my reponse http://localhost:8080/a must be in UTF-8 charset because all of web resources are often in UTF-8.

Do you have a solution ?

For future ref, please can you post questions on the mailing list as per the README.
Meanwhile, have you tried setting the charset explicitly in the java command?

Yes, i tried to force "file.encoding" to UTF-8 and it's solves my issue.
But, i can not use this because my JVM uses other soft.
In fact, i use wiremock in maven plugin mode in a Jenkins environment. I can not afford to globify the Maven configuration for a Wiremock bug.
The best practice is not to rely on system encoding.
I will post on the mailing list, thank you,


refactor for Generate method

Pull Request Test Coverage Report for Build 164

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 162:
0.0%

Covered Lines:
316

Relevant Lines:
316

💛  - Coveralls