"Brăești", "Brătilești", "Goidești" and "Ivănețu" are listed in the drop-down menu

Description:
"Brăești", "Brătilești", "Goidești" and "Ivănețu" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Brăești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Brăești, Buzău(Brăești)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Brătilești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Brătilești, Buzău(Brăești)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Goidești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Goidești, Buzău(Brăești)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Type "Ivănețu" in the search bar.

Expected results
The location is listed in the drop-down menu as "Ivănețu, Buzău(Brăești)".


Bug on live docs site

Environment
Just reporting an issue on the nuxt docs site.
Version
https://ui.nuxt.com/components/button
Reproduction
Go on to the nuxt website, this section in particular:
https://ui.nuxt.com/components/button#icon
and click 'trailing'. this should reproduce the issue.
Description
When on the live docs site, on the button page, clicking 'trailing' docs this happens:

The page is:
https://ui.nuxt.com/components/button#icon
Additional context
No response
Logs
No response

Fixed by setting the @nuxt/content resolution to 2.13.2 for now.


No  baseline_15.0.516.30.zip found [Issue]

no zip for this baseline number which results in the following error:
LoadFromGitHub : C:\users\nsoadmin\desktop\Hashes\baseline_15.0.516.30.zip not found... please open issue on
https://github.com/microsoft/CSS-Exchange/issues, we will work on it
At C:\users\nsoadmin\desktop\Hashes\CompareExchangeHashes.ps1:613 char:13

        LoadFromGitHub -url $zip_file_url -filename $zip_file -installed_ver ...

  + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
  + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,LoadFromGitHub

Tracked by: https://github.com/microsoft/CSS-Exchange/issues/222


Adds testing and CI workflow for pull requests.

Adds sample unit and widget tests.
Adds CI workflow to run tests, upload coverage report to Codecov.
Codecov badge in README.md

We are currently reaching 100% code coverage excluding locator, navigation and their generated files. @ankank30 should we look into excluding these files from the coverage report since there seem to be no plans of writing tests for these files anytime in the long run.

We are currently reaching 100% code coverage excluding locator, navigation and their generated files. @ankank30 should we look into excluding these files from the coverage report since there seem to be no plans of writing tests for these files anytime in the long run.

Those are the self-generated codes, so we can definitely look for skipping those file.


installation on centos

HI
can some one tell me how to install it on centos. i have vps server
regards
wasiq

please tell me if you still have problems installing treeio, I will close the ticket if you don't

yes still i cant install it

I made a install script on ubuntu, you can take a look at it and adapt to centos.
take a look at the install/ubuntu.sh file.
let me know if you have trouble understanding the file

ok let me try to check that

please document your efforts, write a blog post or contribute the install_centos.sh you create to the repository


Remove duplicate Data Management test.sln

Remove duplicate Data Management test.sln. The Autodesk.DataManagement.Test project is already included in the datamangement.sln & this matches the format of the other API solutions/projects.

Hi @tylerwarner33, Thanks for this PR. We will be working on updating the DM SDK shortly. Will merge this PR directly onto the DM feature branch.

Fixed this as part of the release. Thanks!


add custom attributes names for error messages

Added custom niceNames on models.
I see that it's an old issue, and I had to change a little this class for that can use nicenames.

There is a change like this one at #213.
Any change was merged ? I think that there are a lot of people waiting for it.

Fixed through #213. Thanks anyway @jarbitlira!


Cord and GM segmentation should only be performed on the first scan

the 2nd scan is only used to compute SNR using the diff method, i.e.:

register scan 2 to scan 1
segment cord on scan 1
compute SNR using diff method between 1 and 2_reg

Correct. The segmentations for the second scan are not necessary and should not be included.

what is the status of this issue?

in the future, when you fix an issue, close it by referring to the commit that fixes the issue-- thx

Yes, of course. Sorry about that.
Stephanie

On May 9, 2018, at 6:57 PM, Julien Cohen-Adad<EMAIL_ADDRESS>wrote:
in the future, when you fix an issue, close it by referring to the commit that fixes the issue-- thx
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

reopening this issue because https://github.com/neuropoly/gm_challenge/pull/5 is not merged.
order should be:

open issue
fix issue
open PR referencing issue
close PR
close issue

Ah, ok, I see. Thank you for the clarification.

On May 17, 2018, at 8:44 PM, Julien Cohen-Adad<EMAIL_ADDRESS>wrote:
Reopened #4.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

This was resolved with PR #5.


Live

@v-sambati : Thanks for your contribution! The author(s) have been notified to review your proposed change. @sambati

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

File
Status
Preview URL
Details

microsoft-community-training/whats-new-in-microsoft-community-training/2022/whats-new-in-february-2022.md
:white_check_mark:Succeeded
View

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


Heavy-handed hack to stop CentOS PaaS SIG repos from breaking jobs

Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>/assign @sdodson

/lgtm

here goes nothing...


fix typo to more precise.

change
discoverCancel: []context.CancelFunc{},

to
discoverCancels: []context.CancelFunc{},

We have looked at it in our bug scrub but it is not a typo.
Thanks.


Adds FormRequest::getValidated() to take jus a specific validated key

This PR allows to ask a FormRequest instance to get a single specific key from the validated pool.  This can be useful to avoid creating an additional variable to store the validated values in order to extract the one you need:
With the current codebase I should do:
public function store(MyFormRequest $request)
{
    /* other stuff */

    $data = $request->validated();

    $foo = MyModel::create($data);

    $foo->handleMoreData(data_get($data, 'bar', 'default'));

    /* more other stuff */
}

With this PR it would be fine to write:
public function store(MyFormRequest $request)
{
    /* other stuff */

    $foo = MyModel::create($request->validated());

    $foo->handleMoreData($request->getValidated('bar', 'default'));

    /* more other stuff */
}

EDIT: moved to a dedicated getValidated() method to avoid breaking changes.

EDIT: moved to a dedicated getValidated() method to avoid breaking changes.

This is a really nice change! But it would be nice to not have a second method just for fetching a specific key, would also be more similar to current methods in Laravel.
Would it be ok to target 9.x to allow for the breaking change and avoid adding a new method?
It would also be awesome if it could accept an array of keys to retrieve (no default value support in that use case ofcourse)
This would allow for these 3 use cases:
$validated = $request->validated(); // how it works now

$validated = $request->validated('key', 'default'); // your pr

$validated = $request->validated([
   'foo',
   'bar',
]); // returns foo and bar (if they exist, would work exactly like Arr::only, with non existing keys)

Just a suggestion (and wish) :)

@johanrosenson thinking it could be better moving to 9.x in validated() method
Just waiting to see what are thinking others

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.

@taylorotwell
above all: this is the first time we get in touch, so let me thank you for your hard and awesome work on Laravel, all my career is tied to you, so there are no words to fully explain how I appreciate it
I fully understand your position, the maintainability of a framework is a delicate ecosystem, so if my reply would take too much time away from your work, do not bother to answer, I would accept it ;)
just wondering if a new PR of this feature on the master branch (for the incoming 9.x) would be worth of a look by the maintainers and I'd like to know what do you think about it.
my pros:

As the new parameters are optional, FormRequest::validated() would still work as it does

FormRequest::validated('key', 'default') would be consistent with the behaviour of other laravel methods

the PR would just require to change two lines of code:

//from
public function validated()
{
    return $this->validator->validated();
}

//to
public function validated(string $key = null, $default = null)
{
    return data_get($this->validator->validated(), $key, $default);
}

All existing test keep working without any change

I added specific tests for this feature

Anyway, thanks again to you and all your team!


flutter format doesn't work with --dry-run or --machine

Steps to Reproduce
running flutter format lib -n errors with: Cannot use --dry-run and --overwrite at the same time.
running flutter format lib -m errors with: Cannot use --overwrite and --machine at the same time.
in both cases I don't have explicitly set --overwrite (-w).
[✓] Flutter (on Mac OS X 10.12.6 16G29, locale en-DE, channel unknown)
    • Framework revision e8aa40eddd (13 days ago), 2017-10-17 15:42:40 -0700
    • Engine revision 7c4142808c
    • Tools Dart version 1.25.0-dev.11.0

Bug
the flutter format command always appends -w which causes this problem
Workaround
call dartfmt instead of flutter format
Possible solutions

don't expose all dartfmt arguments
append -w only if -n or -m are not set

Scenario
I intend to use flutter format on CI to enforce a common code style. -m would allow better output in json format.

+1 for this, I'm hitting the very same issue.

This looks done:
https://github.com/flutter/flutter/blob/c725f107a178edf04a8ecf857597dc0c131c8933/packages/flutter_tools/lib/src/commands/format.dart#L69

+1 for this, I'm hitting the very same issue. For now I can "just" pass --set-exit-if-changed instead, but that means source files will be changed (as -w is always added) which is not something I'd want in a CI.

Use  flutter format lib -n  to solve

The trick is to but the options before the format command:
Works
flutter format -n lib
Fails
flutter format lib -n
Cannot use --dry-run and --overwrite at the same time.


Assertion error in globalEventExecutor

I am using Vertx 3.1 with Netty 4.0.31. Every once in a while, I got an assert on this line:
 protected final Runnable pollScheduledTask(long nanoTime) {
    assert this.inEventLoop();

    Queue scheduledTaskQueue = this.scheduledTaskQueue;
    ScheduledFutureTask scheduledTask = scheduledTaskQueue == null?null:(ScheduledFutureTask)scheduledTaskQueue.peek();
    if(scheduledTask == null) {
        return null;
    } else if(scheduledTask.deadlineNanos() <= nanoTime) {
        scheduledTaskQueue.remove();
        return scheduledTask;
    } else {
        return null;
    }
}

with the following stack trace:
"globalEventExecutor-1-1"@4,148 in group "main": RUNNING
pollScheduledTask():83, AbstractScheduledEventExecutor {io.netty.util.concurrent}
fetchFromScheduledTaskQueue():114, GlobalEventExecutor {io.netty.util.concurrent}
takeTask():99, GlobalEventExecutor {io.netty.util.concurrent}
run():230, GlobalEventExecutor$TaskRunner {io.netty.util.concurrent}
run():137, DefaultThreadFactory$DefaultRunnableDecorator {io.netty.util.concurrent}
run():745, Thread {java.lang}
The thread that starts GlobalEventExecutor is
"main"@1 in group "main": RUNNING
():37, GlobalEventExecutor {io.netty.util.concurrent}
():35, MultithreadEventExecutorGroup {io.netty.util.concurrent}
():49, MultithreadEventLoopGroup {io.netty.channel}
():61, NioEventLoopGroup {io.netty.channel.nio}
():52, NioEventLoopGroup {io.netty.channel.nio}
():125, VertxImpl {io.vertx.core.impl}
():114, VertxImpl {io.vertx.core.impl}
():110, VertxImpl {io.vertx.core.impl}
vertx():34, VertxFactoryImpl {io.vertx.core.impl}
vertx():78, Vertx {io.vertx.core}

Will check

@luengnat ok I think i know where the problem is... working on a fix now :+1:

Fixed

Oddly I started encountering this issue while upgrading from jdk 8u60 to 8u65.  Not really sure why that would explain this.  I was just curious @luengnat do you happen to remember what JDK you were on when you were encountering this?    We've been on 4.0.26/4.0.27 for about 6 months or and I hadn't seen this issue once until we recently upgraded JDKs in our CI environment and it seems to happen occasionally.


Pod failed with error: Pod was active on the node longer than the specified deadline -remain in status Running

Pre-requisites

[X] I have double-checked my configuration
[X] I can confirm the issues exists when I tested with :latest
[ ] I'd like to contribute the fix myself (see contributing guide)

What happened/what you expected to happen?
I set a timeout of 10 seconds in a template.
activeDeadlineSeconds: 10
After 10 seconds the pod received an error: the pod was active on the node for more than the specified deadline
After a few minutes the pod is deleted in kubernetes, but its remain in status Pending or Running.
The flow is blocked and no continue to the next template.
I expected that the pod get failed status
In previous version the same workflow work ok.
Version
V3.4.2
Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: hello-world-  # Name of this Workflow
spec:
  entrypoint: engine
  podGC:
    strategy: OnPodSuccess
  templates:
  - name: whalesay            # Defining the "whalesay" template
    activeDeadlineSeconds: 10
    container:
      image: docker/whalesay
      command: ['sh','-c']
      args: ["cowsay hello world && sleep 600"]   # This template runs "cowsay" in the "whalesay" image with arguments "hello world"
      resources:
        requests:
          memory: "3Gi"
          cpu: "2000m"
        limits:
          memory: "3Gi"
          cpu: "2000m"
  - name: engine
    parallelism: 7000
    steps:
      - - name: whalesay
          template: whalesay
          withSequence:
            count:  1

Logs from the workflow controller
kubectl logs -n argo deploy/workflow-controller | grep ${workflow}
workflow.log
Logs from in your workflow's wait container
kubectl logs -n argo -c wait -l workflows.argoproj.io/workflow=${workflow},workflow.argoproj.io/phase!=Succeeded
wait.log

This is a fatal error. It prevents us from upgrading argo version to latest.
Can someone please help here?

@shiraOvadia it looks like the k8s API is taking time to update pods when activedeadlineseconds reached. you can try the timeout instead of activedeadlineseconds.
// Timeout allows to set the total node execution timeout duration counting from the node's start time.
	// This duration also includes time in which the node spends in Pending state. This duration may not be applied to Step or DAG templates.

https://github.com/argoproj/argo-workflows/blob/ed351ff084c4524ff4b2a45b53e539f91f5d423a/sdks/python/client/docs/IoArgoprojWorkflowV1alpha1Template.md

I also tried "timeout" , and the behavior was the same as "activedeadlineseconds" .
The pods still in status Running and the never changes to Failed/Error.

I'm also experiencing this issue.

Example Workflow
apiVersion: argoproj.io/v1alpha1
kind: WorkflowTemplate
metadata:
  name: active-deadline-test
spec:
  entrypoint: active-deadline-test
  templates:
  - name: active-deadline-test
    parallelism: 10
    steps:
    - - name: active-deadline-test-timeout
        inline:
          activeDeadlineSeconds: '5'
          script:
              image: devops.cheyne.local:8081/docker/cheyne/cheyne-ubuntu:{{.Chart.AppVersion}}
              command: [bin/bash]
              source: |
                sleep 100s

My suspicion is that the deadlineExceeded node isn't having it's phase updated correctly here: https://github.com/argoproj/argo-workflows/blob/master/workflow/controller/steps.go#L249-L258
I think ErrDeadlineExceeded should have the same if not similar logic to ErrTimeout
Using timeout instead of activeDeadlineSeconds did however work

Using timeout instead
apiVersion: argoproj.io/v1alpha1
kind: WorkflowTemplate
metadata:
  name: active-deadline-test
spec:
  entrypoint: active-deadline-test
  templates:
  - name: active-deadline-test
    parallelism: 10
    dag:
      tasks:
        - name: test-timeout-set
          template: test-timeout
          arguments:
            parameters:
              - name: timeout
                value: '5s'
        - name: test-timeout-unset
          template: test-timeout
        - name: test-timeout-set-empty
          template: test-timeout
          arguments:
            parameters:
              - name: timeout
                value: ''
        - name: test-timeout-set-zero
          template: test-timeout
          arguments:
            parameters:
              - name: timeout
                value: '0s'

  - name: test-timeout
    inputs:
      parameters:
        - name: timeout
          default: ''
    timeout: '{{`{{inputs.parameters.timeout}}`}}'
    script:
        image: devops.cheyne.local:8081/docker/cheyne/cheyne-ubuntu:{{.Chart.AppVersion}}
        command: [bin/bash]
        source: |
          sleep 100s

@sarabala1979 Hi, I got the same issue with timeout.
The pod status is DeadlineExceeded but the workflow step phase is still Running.

we met the same issue.

@sarabala1979 We're still experiencing this issue.

We are also facing the same issue with either 3.4.6 or 3.4.5 and even with v3.4.7. Is there any ETA to fix this issue?

Any updates on this?

This is also affecting us. Can anybody give an update as to whether there's work happening on this issue?

Effectively superseded by #12329. We should probably backport #12761 to release-3.4 then


Rosetta-cli checks

Added rosetta-cli configs and some fixes to get them to pass.

Merged to main in #8. Thanks!


Create chatwithimagesending

onImageUpload() :handles the upload of image file. <input type='"file">:selection of image file for upload. handleImageUpload:is called when the user selects an image file. onChange:gets triggered when the user selects the image file.

Is this about https://github.com/Avaiga/taipy/issues/1314 ?
What is the goal of the code, and how does it interact with the chat control?

So this PR looks like you started by copying frontend/taipy-gui/src/components/Taipy/Chat.tsx then added the onImageUpload property to the Chat component. This would be invoked when the end user requests the upload of an image.
Now we don't quite understand how this articulates with the 'chat' control on the backend, which is what the application really sees, and certainly... how we would integrate that.
Is this PR like showing a concept that we might want to get into the product somehow?


fix: losing connection after the refresh page

Closes: #133

Hey @LuizAsFight! Can you take a look at this?

closing it as was already fixed in https://github.com/FuelLabs/fuel-connectors/commit/e98f5623d46404dfc193e7a0c2319dba699b2cd8

reopening as adding to localstorage is a good feature anyway


impl IntoIterator for &'a DocumentSymbol

Hi,
I'd like to implement IntoIterator for &'a DocumentSymbol because that would simplify displaying the results of textDocument/documentSymbol in my LSP client of choice.
Would that be a welcome addition?

Not sure about this -- DocumentSymbol is a tree, and there isn't one obvious way to iterate the tree. Iteration is also not necessary the best interface to consume a tree, as recursion is also a reasonable choice.
Without knowing the details, I'd say it's perhaps better for the client to define their own way of iteration:
fn iter_doc_symbol(symbol: &DocumentSymbol) -> impl Iterator<Item=&DocumentSymbol> + '_

Oh wow, I've switched over to the builtin client in the meantime. Thanks for the insight, though, much appreciated :)


OpenAPI - Distinguish between stored procedure parameters and result set columns

Why make this change?

Closes #1509

What is this change?

The OpenApiDocumentor service now correctly distinguishes between input parameters and result set columns when documenting a stored procedures REST endpoint. Previously, only the result set columns were resolved. As a result, the OpenApi document that is generated now shows input parameters as fields to be used in the request bodies of POST, PUT, and PATCH.  (GET and DELETE have no request bodies).

How was this tested?

[x] Integration Tests - Added tests with additional stored procedures in our sql deployment script which demonstrates how parameters and output result set columns (and their JSON data types) are resolved and included in the open api doc.

OpenAPI document example
An example OpenAPI document which helps visualize how the parsing works. Irrelevant fields were removed.
{
    "openapi": "3.0.1",
    "paths": {
        "/sp1": {
            "get": {"..."},
            "post": {
                "tags": [
                    "sp1"
                ],
                "description": "Executes a stored procedure.",
                "requestBody": {
                    "content": {
                        "application/json": {
                            "schema": {
                                "$ref": "#/components/schemas/sp1_sp_request"
                            }
                        }
                    },
                    "required": true
                },
                "responses": {
                    "201": {
                        "description": "Created",
                        "content": {
                            "application/json": {
                                "schema": {
                                    "properties": {
                                        "value": {
                                            "type": "array",
                                            "items": {
                                                "$ref": "#/components/schemas/sp1_sp_response"
                                            }
                                        }
                                    }
                                }
                            }
                        }
                    },
                    "400": {
                        "description": "BadRequest"
                    }
                }
            }
		}
    },
    "components": {
        "schemas": {
            "sp1_sp_request": {
                "type": "object",
                "properties": {
                    "inputName": {
                        "type": "string",
                        "format": ""
                    }
                }
            },
            "sp1_sp_response": {
                "type": "object",
                "properties": {
                    "outputName": {
                        "type": "string",
                        "format": ""
                    }
                }
            }
        }
    }
}

you mentioned in PR description that GET  and DELETE won't have request bodies, but they can have input params, right? So, why not request bodies?

you mentioned in PR description that GET and DELETE won't have request bodies, but they can have input params, right? So, why not request bodies?

Request bodies in those two request types are not convention per HTTP spec:

GET https://www.rfc-editor.org/rfc/rfc9110.html#section-9.3.1-6
DELETE  https://www.rfc-editor.org/rfc/rfc9110.html#section-9.3.5-6

Relevant text with my annotation in square brackets:

A client SHOULD NOT generate content in a [GET/DELETE] request unless it is made directly to an origin server that has previously indicated, in or out of band, that such a request has a purpose and will be adequately supported.

Nit: The comment here: https://github.com/Azure/data-api-builder/blob/73716e70134ed6ab09fb482746d135019528c202/src/Core/Services/OpenAPI/OpenApiDocumentor.cs#L169 is repeated at line 178.

It might not be relevant for this PR but I think we default stored procedures to atleast support POST operation if no method is defined. This doesn't seem to be the case here:
https://github.com/Azure/data-api-builder/blob/73716e70134ed6ab09fb482746d135019528c202/src/Core/Services/OpenAPI/OpenApiDocumentor.cs#L456

Good catch. The sp default methods are added (POST) when RuntimeEntities is created and ProcessRestDefaults() is executed. This also looks to be artifacts of old config system where the collection may be null. The nullable annotation and null check are not necessary.


fix tests in unique_with_mow_p0/partial_update are flaky

Proposed changes
Issue Number: close #xxx
tests in unique_with_mow_p0/partial_update are flaky
add sync and order by
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


YouTube Red needs to go

Could you add an option to remove any and all traces of YouTube Red from YouTube? I don't even want to see the videos appearing on the channel pages.

Ironically when I load the page you linked to everything loads properly including the gear options for YTC.  Thanks?


Migrate data with node-pg-migrate

Please, how can I migrate data using node-pb-migrate?
For example, I have column A with different string values, and I want to change each value in each columnA cell in a following way: if value is "hello", replace it with "hi", if value is "goodbye", replace it with "bye", else stay as it was.
Example:
| Column A |   -> | Column A |
 -----------       ------------
| hello    |      | hi       |
| goodbye  |      | bye      |
| mum      |      | mum      |

Please, delete this issue if this is already implemented, I didn't found another way to ask about it.

There are only helpers for DDL. You need to write your own SQL for this.
Something like:
pgm.sql(`UPDATE T SET A = 'hi' WHERE A = 'hello'`)


Processor: Merge two data streams by timestamp

A user can use this processor to merge two data streams by their timestamp.
The processor should be implemented in Java to also run on edge devices.
Imported from Jira STREAMPIPES-59. Original Jira may contain additional context.
Reported by: zehnder.

Closing the issue as it is inadvertently in open status due to migration from JIRA.


Xiaomi vacuum V2 fan speed is not mapped correctly

Home Assistant release with the issue:
0.80.3

Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):
Hass on raspberry pi 3b+

Component/platform:
Xiaomi Mi Robot Vacuum
https://www.home-assistant.io/components/vacuum.xiaomi_miio/
Description of problem:
Fan speed are not mapped correctly for the Xiaomi vacuum V2 Correct values are:
Quiet: 101
Balanced: 102
Turbo: 103
Max: 104
Mop: 105
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
vacuum:
  - platform: xiaomi_miio
    host: !secret xiaomi_vacuum_host
    token: !secret xiaomi_vacuum_token

Traceback (if applicable):

Additional information:

See my comment in #17090 - this needs to be fixed in the upstream library (PRs welcome!).

It is not a problem - only that Mop is missing.
Xiaomi App started using the 101+ values, but you can still use 0-100 that sets the speed percentually.
90 = 90% speed.

But it is not an issue for the underlying library because it doesn't know about the speeds and it doesn't enumerate them, it just set them by integer value. HA is enumerating the speeds and using library to set them to certain integer value. That would be brand new feature for underlying library to know which type of vacuum it is communicating with and the PR would also need to change the API because now the underyling library would be doing the enumeration of speeds. And then we would also need to change HA.

mapping is still wrong.

Just for the record, @bugficks created a PR to fix this issue in the backend library (https://github.com/rytilahti/python-miio/pull/523), but closed it afterwards out of no interest.

@rytilahti Maybe do a shot at moving it forward nevertheless?

@bugficks A week? I'm sorry sir, but that is not uncommon in the open source world.

@frenck if you say so :) I'm used to some sort of reply within a few days. My fork is still available for now and if someone had asked to reopen PR I would have done so.

Sorry for not being responsive enough, I've seen the PR and it looks good from the surface, but I haven't unfortunately have had enough time to dig into the specifics nor do I have currently access to test it myself.
As I cannot test it myself, I'm really hesitant on simply merging it and pushing out a new release (in case it then breaks down on someone's setup). So if anyone is willing to do some testing (verifying that everything works just fine also with the v1 vacuum), I think we can go ahead, merge that PR and I can prepare a new release so we can finally get that fixed.
If there are no volunteers I will be doing that by myself when I'll get access to a device later this year (some point in October). As I hope it doesn't have to wait that long, I hope that some other people will do testing on it and report on that PR :-)

If there are no volunteers I will be doing that by myself when I'll get access to a device later this year (some point in October). As I hope it doesn't have to wait that long, I hope that some other people will do testing on it and report on that PR :-)

Hi! I just start the jurney with HA and I really want help with the MOP functions. I create special area to clean with mop and now I need to change normal mode to mop :D
How can I help You with that?


dude, what's with all the whitespace?

Dissallow trailing whitespace.

@minrk did you hit the wrong button?

Nope.

OK. :(


Verification Code?

Supposedly I've been sent an email with a verification code, but I haven't received any email at all?

+1, using lastest version, windows 10

+2 version from http://igdm.me/, windows 10


Change the filtered value key.

Is there a possible way to change the key used by the component to filter the data ?
tree_data = [     {         "value": 1,         "title": """Test    parent HTML test""",         "children": [             {                 "value": 2,                 "title": "parent 1-0",                 "children": [                     {                         "value":3,                         "title": "leaf1",                     },                     {                         "value": 4,                         "title": "leaf2",                     },                 ],             },             {                 "value": 5,                 "title": "parent 1-1",                 "children": [                     {                         "value":7,                         "title": """ leaf3 """,                     },                 ],             },         ],     }, ]
for example, the value there is the number but the filter should work on the title.

Hey,
I am working around that by putting everything in the "value" I want to be able to search for.
Example:
    class Question():
        def __init__(self, id, label, question_text):
            self.id = id
            self.label = label
            self.question_text = question_text
    question = Question(5, "Label", "Question Text")
    node = {
        "value": f"""{{"id": {question.id}, "label": "{question.label}", "text": "{question.question_text}"}}""", #value is a dictionary as a string
        "title": f"{question.label if show_question_label else question.question_text}",
    }

Then you can read out the return value like this:
import ast
return_value = ast.literal_eval(return_object_by_component)


restore: support file level routing

What problem does this PR solve?
Implement table level routing, see Lighting RFC 5 for more detals
What is changed and how it works?
With now configurations mydumper.files, user can privide regexp pattern to match source files and conf the source relate schema/table/type either by constant string or template value expand from the regexp pattern.
NOTE: this pr doesn't implement following features:

file sorting key. relate to the mydumper.files.key conf. User can still set this conf, but is currently useless.
file sorting key transform. relate to the mydumper.files.key-transform. This conf is not provided.
compression. relate to the mydumper.files.compression. As we don't support read compression files yet.  User can still set this conf, but is currently useless.

We should deprecate the feature table router and remote the lightning.router config fileds in the future.
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects

Increased code complexity
Increased config complexity

Related changes

Need to cherry-pick to the release branch
Need to update the documentation
Need to be included in the release note

[2020-08-07T10:28:24.499Z] Error: restore table `error_summary`.`a` failed: checksum mismatched remote vs local => (checksum:<PHONE_NUMBER>9643620760 vs<PHONE_NUMBER>657377814) (total_kvs: 3 vs 2) (total_bytes:87 vs 58)

remote

This is expected. Because target table contains 1 row before restore.

@kennytm  PTAL again.
BTW, shall we add the new config field path to [files] to allow user set a file path instead of regexp pattern?

BTW, shall we add the new config field path to [files] to allow user set a file path instead of regexp pattern?

do it if it's easy

i still think there's no point having a constExpander.

you are also making extractSubPatterns way too complicated by using strings.IndexByte. try https://play.golang.org/p/pxPaTbLPuWH.
var expandVariablePattern = regexp.MustCompile(`\$(?:\$|[\pL\p{Nd}_]+|\{[\pL\p{Nd}_]+\})`)

func VerifyExpansion(re *regexp.Regexp, template string) error {
	for _, variable := range expandVariablePattern.FindAllString(template, -1) {
		var name string
		switch {
		case variable == "$$":
			continue
		case strings.HasPrefix(variable, "${"):
			name = variable[2 : len(variable)-1]
		default:
			name = variable[1:]
		}
		if number, err := strconv.Atoi(name); err == nil {
			if number > re.NumSubexp() {
				return fmt.Errorf("sub pattern capture '%s' out of range", variable)
			}
		} else if re.SubexpIndex(name) < 0 {
			return fmt.Errorf("invalid named capture '%s'", variable)
		}
	}
	return nil
}

(btw a trailing $ and malformed ${... is fine by go standard)

@kennytm  I updated the pattern check logic and add support for path config. PTAL, again 😂
BTW:
after #361  , the file path traversed by WalkDir will be relative path, but currently, they are all abs path. So I decide to support both abs path and relative path for files.path in that pr, because there are a lot of tests to modify and I don't want to do again in this pr

PTAL @3pointer


Corrections on Czech language

Bug

[ ] Related issues linked using closes: #number

Feature

[ ] Related issues linked using closes: #number
[ ] There is only 1 db migration with no breaking changes.

Translations

[ ] .json files are formatted: yarn format
[ ] Translations are correct
[ ] New translation? It's been added to next.config.js

Id love to get some permission to edit this section, without you, is that possible? :D

Id love to get some permission to edit this section, without you, is that possible? :D

I'd like to keep stuff organized via Pull Requests. So that's a no right now; If you want to make changes to a language, make a PR :)


workload create/udpate/apply supports --export -- to optimize programmatic  workflows

Description of problem
adding support for --export to  tanzu apps workload apply will make the tilt file command simpler/more efficient otherwise devs will have to specify tanzu apps workload apply, then tanzu apps workload get --export.  Seems a little unwieldy.
If adding this support to apply - we should add it to workload create and workload update for consistency (so far all the commands have the same flags and we're setting an expectation).
Proposed solution
When --export is passed in with create/update/apply, Expect it to return the workload just created/updated/applied.
--- will need to flesh this out further WRT:

is --yes assumed?
help output - explanation
error cases

Given I have a compliant workload.yaml for my-app
When I create or update my-app workload and include the --export flag in my incantation
Then the output from the command contains only the content that would be returned if I were to have run "tanzu apps workload get my-app --export"

Example
Current UX: (--export isn't supported)
03/17/22-15:28:59 tanzu % tanzu apps workload create -f app-assets/valid-yaml.yaml
Create workload:
      1 + |---
      2 + |apiVersion: carto.run/v1alpha1
      3 + |kind: Workload
      4 + |metadata:
      5 + |  labels:
      6 + |    app.kubernetes.io/part-of: tanzu-java-web-app
      7 + |    apps.tanzu.vmware.com/has-tests: "true"
      8 + |    apps.tanzu.vmware.com/workload-type: web
      9 + |  name: tanzu-java-web-app-2-6
     10 + |  namespace: default
     11 + |spec:
     12 + |  source:
     13 + |    git:
     14 + |      ref:
     15 + |        branch: spring-boot-2.6
     16 + |      url: https://github.com/odinnordico/tanzu-java-web-app

? Do you want to create this workload? Yes
Created workload "tanzu-java-web-app-2-6"

Proposed UX (--export assumes --yes)
03/17/22-15:28:59 tanzu % tanzu apps workload create -f app-assets/valid-yaml.yaml --export
---
apiVersion: carto.run/v1alpha1
kind: Workload
metadata:
  labels:
    app.kubernetes.io/part-of: tanzu-java-web-app
    apps.tanzu.vmware.com/has-tests: "true"
    apps.tanzu.vmware.com/workload-type: web
  name: tanzu-java-web-app-2-6
  namespace: default
spec:
  source:
    git:
      ref:
        branch: spring-boot-2.6
      url: https://github.com/odinnordico/tanzu-java-web-app
03/17/22-15:29:13 tanzu %

Describe alternatives you've considered
requiring two commands as is current
Additional context
--- will need to flesh this out further WRT:

is --yes assumed?
should we support -o yaml/json when --export is provided?
help output - explanation
error cases

comments originally submitted by @paulcwarren
Just FYI, for tilt we require the --export (in general) to be able to return the workload in the cluster. Tilt requires the uid field which is what it then bases its tracking off of.

We would also require the ability to --quiet any other output that isn't the eventual workload yaml.

@paulcwarren - do you see any issue with --export implying --yes and --quiet?

Just FYI, for tilt we require the --export (in general) to be able to return the workload in the cluster. Tilt requires the uid field which is what it then bases its tracking off of.

The idea behind --export is that is makes take a resource in the cluster and makes it appropriate to write into a gitops store and share. It intentionally strips off the uid and other cluster specific fields. You want something more like -o yaml for create/update/apply

do you see any issue with --export implying --yes and --quiet?

I do, --yes is an explicit opt-in to not be prompted before potentially destructive actions. Making it implicit weakens its purpose. In the case here of using a script to drive the command, the script can specify the extra flags it cares about. One flag changing the implicit value of an unrelated flag is a good way to surprise users.

@scothis - I agree with the sentiment in your previous comment and have updated the issue accordingly.
--yes is not implied if a user includes --export (users must provide --yes to skip prompts).

It's worth double checking that the prompts are on stderr so that stdout can be piped to another tool with the exported content.

After further clarifying discussions with @paulcwarren and my team, I've decided against implementing this and we'll stick with the 2 command approach which works today.
apps workload apply... followed by apps workload get -o yaml/json to get the uid.


Added Blogs on Python by Programiz PRO

This commit introduces a blogs by Programiz PRO, which contains a series of blogs on Python. It covers a wide range of topics like:

Everything you need to know about Python
Best Python Certification in 2024
Best Way to Learn Python
Python Cheat Sheet
so on
Due to its wide coverage on Python, it is suitable for readers of all levels.

@mikeyny
It's been a while.
Could you review this PR?


feat(Midi): Adds support for multiple midi inputs

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Adds the ability to have multiple midi devices (pedals) connected to the prompter at the same time.

What is the current behavior? (You can also link to an open issue here)
Only allows for a single device at the time.

What is the new behavior (if this is a feature change)?
Listens to all connected inputs. Accepts CC notes on channel 8 from all active inputs.

Other information:

Status

[ ] Code documentation for the relevant parts in the code have been added/updated by the PR author
[ ] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

@jstarpl updated, ready for review

@jstarpl updated, ready for review

Codecov Report

Merging #432 (c55e100) into release30 (4e207c4) will increase coverage by 0.02%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           release30     #432      +/-   ##
=============================================
+ Coverage      68.60%   68.62%   +0.02%
=============================================
  Files            213      213
  Lines          14258    14258
  Branches        3213     3213
=============================================
+ Hits            9781     9785       +4
+ Misses          4297     4293       -4
  Partials         180      180

Impacted Files
Coverage Δ

meteor/server/api/systemTime/systemTime.ts
60.86% <0.00%> (+4.34%)
:arrow_up:

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 4e207c4...c55e100. Read the comment docs.

Codecov Report

Merging #432 (c55e100) into release30 (4e207c4) will increase coverage by 0.02%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           release30     #432      +/-   ##
=============================================
+ Coverage      68.60%   68.62%   +0.02%
=============================================
  Files            213      213
  Lines          14258    14258
  Branches        3213     3213
=============================================
+ Hits            9781     9785       +4
+ Misses          4297     4293       -4
  Partials         180      180

Impacted Files
Coverage Δ

meteor/server/api/systemTime/systemTime.ts
60.86% <0.00%> (+4.34%)
:arrow_up:

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 4e207c4...c55e100. Read the comment docs.


[Feature request] Efficient bulk inserts (single bulk insert statement)

https://github.com/aws/amazon-redshift-python-driver/blob/e0192784f5c949da50469a566599a1cc519670f1/redshift_connector/cursor.py#L235
In the above i see executemany() function runs the query in loop. In redshift say if i am loading thousands of records into redshift via a lambda, i would like to have a method which can run a single bulk insert statement
Eg
INSERT INTO
    links (url, name)
VALUES
    ('https://www.google.com','Google'),
    ('https://www.yahoo.com','Yahoo'),
    ('https://www.bing.com','Bing');

@kishaningithub can I work on this ?

@Yash621 Sure 👍

The execute_values method in psycopg2 library can probably used as an inspiration for this implementation :-)

https://www.psycopg.org/docs/extras.html#psycopg2.extras.execute_values
https://billyfung.com/posts/2017-06-30-psycopg2-multiple-insert/

Have you tried COPY commands? Many developers are on vacation this week. We can give you feedback next week. But at the surface, it looks good to use multi tuple insert statement.

Thanks for the response @iggarish Yeah it is end of the year, i am fine with delayed responses :-)
From what i understand COPY is great for loading mountains of data into redshift but the use case we have is a realtime streaming one. To give more context, we are using redshift to power a realtime dashboard i.e. the data flow is like
AWS kinesis stream -> AWS lambda -> AWS Redshift -> Dashboard
In the lambda, We receive batch of records in lambda which we want to insert using the bulk insert syntax(described above in the thread) that way we send a single SQL statement to redshift with a lot of records
Also when you say "multi tuple insert statement" which is the method in this library you are referring to? Does it send single bulk insert sql statement to redshift?

Hi @kishaningithub,
Thank you for reaching out about your interest in this feature. I agree this would provide benefit in the mentioned use case.
@Yash621, thank you for being proactive in addressing this issue. I will take a look at your PR and provide some feedback shortly :).

@Brooke-white  thank you so much :)

@kishaningithub, please take a look at #81 so we can ensure this PR satisfies your use case :)

Big thank you to @Yash621 for taking on this feature request and @david-dest01 for supporting this effort. PR, #81 has been merged and will be included in our next release scheduled for February.

@Yash621 nice work!

@david-dest01  thank you for your support and guidance :)

@Brooke-white could n't be more happy to hear that :)

@Brooke-white @kishaningithub @david-dest01 it was great experience to collaborate with you all.


ja: Improve translation of docs/concepts/services-networking/ingress.md

This is a Feature Request
This is a follow-up issue of #21608. The current ingress page has some work to do as it does not sound quite natural as Japanese.
What would you like to be added
Update translation of docs/concepts/services-networking/ingress.md
Why is this needed
Comments

/assign

/close


brace_style doesn’t respect PreferSameLine

I have the behavior of SameLineWhere while I have this in my rustfmt.toml:
brace_style = "PreferSameLine"

File and project here

brace_style as currently intended is only applicable in item contexts. We'd started going down the path of extending it to apply in certain expression contexts, but after some reflection think we need to take a different path in the config option surface in order to provide sufficient control and flexibility.
Closing this accordingly, but feel free to subscribe to #3376 for updates on when/how we'll expose brace style control in some of these expression contexts


Travis yml and variance cutoff

Added travis.yml file for CI integration (Sergio) and added variance cutoff as command line parameter.

A bit late to comment but it looks good to me.


copyDirectory using regular expressions

Laravel Mix Version: 1.1.1
Node Version (node -v): 6.11.0
NPM Version (npm -v): 5.1.0
OS: Ubuntu 17.04

Description:
Hi, when I try to use copyDirectory with regular expressions (angular/app/**/*.html) it makes a copy of html files without nested directories.
Steps To Reproduce:
I have next files in angular folder (from):
angular/app/layouts/default.html
angular/app/pages/lessons/lessons.html

I have a views folder (to):
public/views (without any file)

webpack.mix.js
mix
  .copyDirectory('angular/app/**/*.html', 'public/views/app')

[Uncorrect]After compile it looks like:
public/views/app/default.html
public/views/app/lessons.html

[Correct] But I think it should looks like:
public/views/app/layouts/default.html
public/views/app/pages/lessons/lessons.html

I think laravel-mix should create nested folders, because without it  using regular expressions (example: **) is not much powerful as it should be.
Tooltip
When I was using laravel-mix v.0.12.0, I overwrote 'FileCollection.js' to fix my issue. (I'm not sure that it will help you):
let DefaultFileCollection = require('../node_modules/laravel-mix/src/FileCollection');

class FileCollection extends DefaultFileCollection {

	/**
	 * Create a new FileCollection instance.
	 *
	 * @param {string|array} files
	 */
	constructor(files) {
		super(files);

		if (typeof files === 'string' && files.includes('*')) {  // edited
			this.rmDestSymbolsCount = files.indexOf('*'); // edited
		} // edited
	}

	outputPath(src) {
		let output = new File(this.destination).parsePath();

		// If the src path is a file, but the output is a directory,
		// we have to append the src filename to the output path.
		if (src.isFile && output.isDir) {
			var sufix = Array.isArray(this.files) ? src.file : src.path.replace(this.files, ''); // edited

			if(this.rmDestSymbolsCount) { // edited
				sufix = sufix.substring(this.rmDestSymbolsCount, sufix.length); // edited
			} // edited
			output = path.join(output.path, sufix); // edited

			if (new File(output).parsePath().isDir) {
				output = path.join(output, src.file);
			}

			return output;
		}

		return output.path;
	}
}

module.exports = FileCollection;

P.S. Sorry for my bad english

This is expected behavior for using a regex of files.

@JeffreyWay Okay, no problem. What would you recommend when I have next files?
angular/lessons/index.html
angular/orders/index.html

Sometimes I have 50+- html files to copy. Do I need register all of them without regular expressions?


Destructive node collapse

It should be possible to right click a node and collapse its effect into its upstream nodes, whenever that's possible. For example, a Transform node should be able to update the nodes upstream of it to produce the same result without its continued presence. This should work for all nodes that could support something of the sort. For example, if a Round Corners node (which we don't have yet) is applied downstream of a Rectangle node, it could be collapsed by setting the original Rectangle node's corner rounding parameter to an equivalent value after factoring in any transformations that have altered its scale along the data flow. We may need some kind of system to encode and calculate equivalence relations. (This full system could be a GSoC project.)

A short-term hard-coded solution for Transform and Path nodes would be useful before a proper system like this is built.

I got this

@moOsama76 this is a reasonably complex task (even the short-term version) that would be a sizable step up in difficulty over your past contributions so I'd recommend considering tasks that are a bit simpler for now and continue working up towards this one. I can try and pick out some suggestions for you if you ask on Discord (but I've been short on time recently).


not an issue, just a comment

great job aetopia, i love ur projects so much. they are really useful for people and they work greatly

Thanks :3

my discord break ends in some days so don't be worried :p

:D


Fix for issue #924

As mentioned in https://github.com/coreos/prometheus-operator/issues/924#issuecomment-362859346 .

Thanks! I will merge this and push a new release of the image.


Fix path casing issues

As of v2.0, Pumlhorse does not run on OSes that have case-sensitive file paths. There are a handful of import statements that have incorrect casing.

Fixed in v2.2.0


how to detect change or udpate column in table

i want to fire event after change or update column in table
sqlite3_update_hook , is any solution please thanks

@bnadem Can you elaborate on your requirements? It sounds like you want a trigger, which could be implemented  via calling db.exec() on at CREATE TRIGGER sqlite statement.
An example CREATE TRIGGER statement might look like this:
CREATE TRIGGER add_change_to_foo_audit_log
   AFTER UPDATE ON list_of_foos
   WHEN old.foo_col <> new.foo_col

BEGIN
    INSERT INTO foo_audit_log (
        old_id,
        new_id,
        old_foo_col,
        new_foo_col,
        created_at
    )
VALUES
    (
        old.id,
        new.id,
        old.foo_col,
        new.foo_col,
        DATETIME('NOW')
    ) ;
END;

Then you would call db.exec() to execute it on the database.


Disable icons on cell toolbar other than hiding to avoid flickering

https://user-images.githubusercontent.com/876920/114232039-20a38780-9930-11eb-95a9-dc9286cfcf8c.mov

I updated the behavior of Split Cell Action to support splitting at the begin and end of the model then it is always visible on the toolbar.

I don't think there's anything remaining to do then?


Enhancement ideas

These could potentially be different flavors of the DDS with additional functionality:

Provide a clear op to clear the list/reset. In case full history is not important.
Currently ledger grows unbounded. We could potentially have configurable N and M such that when ledger reaches N items, we delete (shift) the first M items and fire an event for clients to handle. We lose the full history but we gain infinite append.
Context-dependent append: client might chose to append an item based on its known state of the world (local ledger). If another client interleaves an append, so when our append gets sequenced, ledger is different, we can fire a different event (rejected) and let client decide whether to resubmit. Other clients would ignore the op. We should be able to easily implement this by including ledger length when we submit the op and checking against actual length when we get the op back (note previous 2 ideas might shorten ledger, in which case we can keep a separate counter for the length).

clear API is now available.


Ability to move custom views across containers

Extensions contributing view containers and views are providing a feature allowing users to choose the view container into a contributed view can be shown. Under the hood extensions register multiple same views with different ids for each location and use the when context to define the visibility. This is cumbersome from an implementation perspective also causing issue like https://github.com/Microsoft/vscode/issues/57422
Cleaner way is to VS Code OOB provides support to move contributed views across view containers

@alexr00 Should this be closed now that views can be moved by the user?

True, this can be closed.


Updated README.md

Changed comments indentation so metrics and tls blocks stay under api block.
Corrected typos.

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

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 63.056%

Files with Coverage Reduction
New Missed Lines
%

pkg/config/env.go
4
79.65%

Totals

Change from base Build<PHONE_NUMBER>4:
-0.2%

Covered Lines:
1234

Relevant Lines:
1957

💛  - Coveralls

@diaznet thank you very much for the corrections


Logout not working

logout does not remove the "key" from the user in login_info.json as intended

d39f30e732229cac75dc4c22ba579883dfed463a


URL in description and README broken

The URL in the repository description and README are pointing to teradyne-dev-workshops's copy of the game instead of yours.
Please fix both, so they point to your copy of the game at https://teradyne-dev-workshops.github.io/github-games-ter-wangdavi.

Readme.md URL link changed to https://github.com/teradyne-dev-workshops/github-games-ter-wangdavi


Pumpkin and Melon Stems not reverting

Spark link
https://spark.lucko.me/EQARhZjhZN
Expected behavior
Pumpkin and Melon stems should revert when the attached Pumpkin/Melon is broken.
Observed/Actual behavior
One of my player's pumpkin farm has a design where once the stem has changed, an observer will detect it and crush the pumpkin. However sometimes the pumpkin stem doesn't revert back to the upright position.

Steps/models to reproduce

Here are some pictures of the farm.
Purpur version
Current: git-Purpur-1715 (MC: 1.19)*
Previous: git-Purpur-1632 (MC: 1.18.2)

You are 17 version(s) behind
Download the new version at: https://purpurmc.org/downloads

Agreements

[X] I am running the latest version of Purpur available from https://purpurmc.org/downloads.
[X] I have searched for and ensured there isn't already an open issue regarding this.
[X] My version of Minecraft is supported by Purpur.

Other
I know I'm outdated and not running the latest, but I don't think that really should affect it as its been a very long and reacurring issue on my server. Just let me know if you want me to update to latest and test.

Your version of Minecraft is not supported by Purpur.

I keep misreading! sorry
Are you sure this isn't client-sided?

Yes, this was the issue with a vanilla Java account with no mods or anything, along with a bedrock user. I will be testing the latest Purpur version tonight.

I keep misreading! sorry (read the previous version instead of current)
Though you are not running the latest version of Purpur available from https://purpurmc.org/downloads, which is still important because this could've been fixed recently.
Are you sure this isn't client-sided? Tried on the vanilla client, no mods?

When I had this issue it was only for the other people on my server (I usually have under 5 ping, all stems update fine for me), as the farm seemed to be working as normal. (when testing, we found that relogging updates everything, making me believe this is a client-side issue.)

Odd alright, let me have them relog.

Here's what the players said: "I relog everyday and even after relog this issue still persists
Also if its a client side issue only i would see this technically. But everyone who come on my island can see the stems not reverting"

Alright, let me have them record a video.

Went in on my server running build #1721 (I am aware this is outdated, this issue is too.)
Quick video of a melon & pumpkin farm working properly. (I have under 10 ping, running on fabric with sodium, some other peformance mods.) No protocol hacks, no lag-reducing plugins, or anything of that sort.
https://cdn.driver.fyi/r/2022-07-28_19-04-48.mp4 (video is too big to send via github or discord)

Yep, can confirm with my players after some testing that it is player sided


Use same IP address for both boots and nginx

Signed-off-by: Michael Richard<EMAIL_ADDRESS>Description
This configures NGINX to listen on port 8080 and lets go the need to configure a second IP address on the host dedicated to NGINX.
Why is this needed
Setting up a second IP address to host NGINX on the same host is not always easy, especially when running tinkerbell on network devices like switches. The second IP address adds a useless level of complexity. In the future, all the code required to identify the host operating system and configure the IP address could even be removed and left as a prerequisite, since the host is likely to have an IP address already configured.
How Has This Been Tested?
The untouched vagrant_test.go test ran sucessfully.
How are existing users impacted? What migration steps/scripts do we need?
Simply re-applying the docker-compose.yml should be sufficient (untested).
Additional firewall rules to allow traffic on port 8080 could be required depending on user's network configuration.
Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

Hello!
Thank you for this work, can you rebase, please?
Thanks

Hello!
Thank you for this work, can you rebase, please?
Thanks

Done

Done

Ops, I am sorry
This PR should fix the CI issue https://github.com/tinkerbell/sandbox/pull/44

Ops, I am sorry
This PR should fix the CI issue https://github.com/tinkerbell/sandbox/pull/44

I am not sure how this will impact users that are running Nginx and I am not sure if we have to mark it as breaking change or not. I will have a look tomorrow 👍

I am not sure how this will impact users that are running Nginx and I am not sure if we have to mark it as breaking change or not. I will have a look tomorrow 👍

Now it's fixed!

Now it's fixed!

It's probably fair to consider it as a breaking change, even though it won't break anything in a majority of cases. Upgrading will fail if:

Some service is already running on port 8080
Firewall's blocking traffic on port 8080

It's probably fair to consider it as a breaking change, even though it won't break anything in a majority of cases. Upgrading will fail if:

Some service is already running on port 8080
Firewall's blocking traffic on port 8080


Allow choice of Azure App Gateway, over vanilla load balancer & nginx https

The current template uses the vanilla Azure Load Balancer with nginx https proxy to terminate https connections. It'd be good to allow choice of Azure App Gateway, over the current default setting. The SSL cert deployment on the Azure App Gateway might remain a small issue?

Incorrectly closed from #96 (which is just a pre-requirement for the actual fix). Reopening as such.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-25-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3151

Questions ?
Please refer to the Backport tool documentation


missing implementations to serialize models

I'm trying to serialize a simple LinearRegression<f64, DenseMatrix<f64>> model, which fails in current development state,
the trait `serde::ser::Serialize` is not implemented for `smartcore::linear::linear_regression::LinearRegression<f64, smartcore::linalg::naive::dense_matrix::DenseMatrix<f64>>

This also applies to f32 and occurs for serializing and deserializing. It works on public version 0.2.0.
let model_binary = bincode::serialize(&model).expect("Can not serialize the model");
...

...
bincode::deserialize(&buf).expect("Can not deserialize the model");
...

info
[dependencies]
wasm-bindgen = "=0.2.63"
smartcore = {git = "https://github.com/smartcorelib/smartcore", branch="development"}
serde = "1.0.125"
bincode = "1.3.3"
ssvm-wasi-helper = "=0.1.0"

Adding features=["serde"] did the fix. I didn't know serde has been made optional.


Error in dumping weights from Pytorch model

I used pytorch dumpers and faced few problems, i am listing them down with the solution .

I am using ../scripts/pytorch_checkpoint_dumper.py to dump my weights from pytorch model. But i
am getting below error
AttributeError: 'collections.OrderedDict' object has no attribute 'iteritems' this error is coming from the
function build_and_dump_vars
for var_name, var_weights in self.state_dictionary.iteritems(): in file
pytorch_checkpoint_dumper.py .
To resolve this i had to replace self.state_dictionary.iteritems() by self.state_dictionary.items()

In file ../scripts/pytorch_checkpoint_dumper.py  in function build_and_dump_vars i was
getting  serializable error with map so i had to replace line
var_shape = map(int, list(var_weights.size()))  by
var_shape = list(map(int, list(var_weights.size())))

In file ../scripts/pytorch_checkpoint_dumper.py  in function build_and_dump_vars
i had to convert cuda tensors to cpu tensors by replacing the line
tensor = var_weights.numpy() by  tensor = var_weights.cpu().numpy()

My system configuration is :

Python : 3.6.2

Cuda: V 8.0.61

DeepLearnjs : 0.3.11

Pytorch : 0.2.0_4

Let me know you want to know more how to produce these errors.

@mishra-avinash Hi! Thanks for opening this issue. These scripts were written in Python 2 and some of these features are not available in Python 3. Perhaps we should make it compatible with both Python 2 and Python 3. I'll send a PR soon for the same!


[RFR]: Pass through action in generic Fetch actions

I need to monitor each individual fetch request to the initiator. Currently there's no way of telling which FETCH_* action belongs to which action. I like to tie them down to an action by simply passing the action in the generic fetch actions (FETCH_START,FETCH_END,FETCH_ERROR and FETCH_CANCEL). This is not in anyway helpful for ra at the moment, but would help me.

Sorry, I don't see a use case for react-admin.


The class which has no source can't show the substructure

OS: Windows
Configurations: Vscode 1.29.0&&JDK1.8.0_191
Sample project:
simple.zip
When I open the class in the jar,I can't see its substructure, the behavior has below:

Dup as https://github.com/Microsoft/vscode-java-dependency/pull/105


Animation Player no longer calls a method twice when playing the anim…

Bug
When an animation is played backwards, and there is a Call Method Track in it with a method, that method is called twice. This doesn't happen when the animation is played forwards.
Replicate
Create a Node3D, add an AnimationPlayer as a child. Attach a script to the Node3D, implement the _ready() method and create an other method called foo(). The script should look like this:
extends Node3D

@onready var animation_player: AnimationPlayer = $AnimationPlayer

func _ready() -> void:
    animation_player.play("bar")

func foo() -> void:
    print("foo")

Create a new animation called "bar", add a new Call Method Track, and in it call the foo() method.
If you run the scene, observe that the foo() method is called once.
Now change the _ready() method to this:
func _ready() -> void:
    animation_player.play("bar", -1, -1.0, true)

Run the scene again, and observe that foo() is being called twice.
Reasons for the bug
There are 3 steps to understand the bug:

In Animation::track_find_key, in "case TYPE_METHOD", the _find method is being called, however the backwards flag is not being passed to it.
We need to call this function with this flag, so we need to know whether we are playing backwards or not.
Currently, there is no way to pass that info to Animation::track_find_key.
Even if we could pass that info, AnimationMixer::_blend_process, the "bool backward" variable does not have the correct value at the very beginning. (And Animation::track_find_key is called from that method when the bug happens.)

Solution

and 2) Animation::track_find_key now has an other parameter called p_backwards (called like this in _find), which defaults to false. In AnimationMixer::_blend_process, in "case Animation::TYPE_METHOD" the Animation::track_find_key is now called with this parameter.
The backward variable is incorrect because ai.playback_info.delta is 0.0, and not -0.0. To fix this, in AnimationPlayer::_process_playback_data, there are added checks now to make sure if it's correct. Also, in that function when p_started is true, a constant 0.0 was passed as delta, regardless of playing direction.

Possible extensions
In Animation::track_find_key, there are other cases where _find is called without the backwards flag. Might be interesting to check whether they need it as well. This pull request is intended to solve the method call problem only.

Please open a bug report for this, or link one that exists, to track the issue separately from the PR

Done! https://github.com/godotengine/godot/issues/86648

Thank you for your contribution and welcome to contributing! I've left some details on this as these changes add some things that needs fixing
Someone on the appropriate team will come and look at this when they have the time, I suspect it might take a few days because of the holiday season

Thanks, but this should be mostly fixed by the existing PR as https://github.com/godotengine/godot/pull/86227. If you have tested https://github.com/godotengine/godot/pull/86227 and have problems with playback in the backward, it may be possible to use a different PR as a follow up.


Latest Release issues

Noticed that older versions of mediapipe still work  but  after each save  and  file  rename TD will fail to load all the media pipe assets   and  whole networks  are  gone inside the mideapipe window  , including operator IN and OUT nodes.
could this be due to the new  release update ?
it was saving a and reloading  fine 2 weeks ago last release update was 7 days ago?
images below of the standard network vs  the missing one ,
im teaching a class MAY 11TH  using touchdesinger and mediapipe was a big part of the  workshop.
please get back to me

Hey @lifeonmod if you're saving to a new folder/location, you also need to include the toxes folder and all the tox files in it.

Thank you for the speedy reply
The suggestion worked perfectly.
I will be looking to  reload the external toxes during each boot up.
Thank you
-LoM


Bump version to 0.2.0

4 files to change version? it looks suspicious...

@mmalohlava yes, config/version.mk, README, Makefile and one kmeans class, which doesnt even need it - will remove it.
I made an issue for Makefile - it should read the version from version.mk.
But version.mk and README will have to be changed every time. Unless we know a way to read the version from version.mk in README?

@mdymczyk hmm, it could be tricky - you can invoke JS which will access the file via HTTP and read it. Not worth it. But you can look if there is some kind of way to do that via image service:

@mmalohlava will have a look. For now I fixed the Makefile to read it from make/version.mk - from now on we'll only have to change make/version.mk and README.md.


Update azure-stack-marketplace-azure-items.md

Fixed a wrong line break and parenthesis.

@kongou-ae : Thanks for your contribution! The author, @sethmanheim, has been notified to review your proposed change.

Thanks. Verified and merged.


Change default headers, or at least cookies

I have found many times when I've been creating a wrapper for a rather unstandardized API that I need to change either the default headers or a specific cookie's value to be sent with all requests. Currently my only option for that is to build a brand new client, losing all of my cookie store. This seems like a big oversight in my opinion as many other libraries from other languages allow the changing of the cookie and/or header storage on the fly.
To be clear it should not be the library's goal to babysit an individual's requests, so if one's code manages to make an incorrect request due to manipulating the headers/cookies just before being sent, it is the individual's responsibility to fix that scenario.

I think providing a custom cookie store will allow you determine how to synchronize the changes. The best proposal that I'm looking at getting merged is in https://github.com/seanmonstar/reqwest/pull/1105

This PR looks perfect for what I was imagining. Looking forward to it being merged, thank you!
Closing because referenced PR #1105 is the expected solution.


Negative Structure Ordinal produces positive value field

Describe the bug
Accessing a structure with a negative value will produce the same results as if the value was positive.
To Reproduce
Steps to reproduce the behavior:

Compile below example code
Load and analyze in ghidra
Change getValue parameter from int* to A*

int getValue(int* a_ptr) {
    return a_ptr[-1];
}

typedef struct A {
    int a_data;
};

typedef struct B {
    int b_data; /* buffer for protection */
    struct A a;
};

int main() {
    struct B b;
    b.b_data = 5;
    printf("%d\n", getValue(&b.a));
    return 0;
}

Expected behavior
Unsure.
Screenshots

Environment (please complete the following information):

OS: Microsoft Windows [Version 10.0.18362.356]
Java Version: 12.0.2
Ghidra Version: 9.1_DEV

This is how pointers/memory work though, right?  You are giving it an array of A, and its getting the -1 index and returning a_data.  You might want to expand your example and what you're expecting.  struct B doesn't provide any value in your example, did you mean to cast getValue(int *) to getValue(B *)?

This is how pointers/memory work though, right? You are giving it an array of A, and its getting the -1 index and returning a_data. You might want to expand your example and what you're expecting. struct B doesn't provide any value in your example, did you mean to cast getValue(int *) to getValue(B *)?

Oh yes, I misinterpreted what was going on.


Remove deprecated WinGet package for October 2023 GDK Update 2

Remove deprecated WinGet package for October 2023 GDK Update 2
Microsoft Reviewers: Open in CodeFlow

@microsoft/xbox-developer-experience-toolbox for review


A different camera entity id and hls buffersize can now be specified

A separate camera entity id can now be specified for fullscreen cameras, for example to show a high resolution rtsp stream on the popup, while the camera thumbnail tile uses a low resolution and low bandwidth still image sequence. If the id is not specified in the fullscreen object, then the tile item id is used by default, providing backwards compatibility.
The HLS buffer default length was reduced from 30 to 5 seconds to reduce latency, and can optionally be co figured from config.js. If bufferLength is not supplied, 5 seconds are used by default.
Usage:
fullscreen: {    type: TYPES.CAMERA_STREAM,    ObjFit: 'contain',    id: 'camera.highres_camera',    bufferLength: 3 }

The style (indentation, spaces within braces) is quite messy.
Also, we should add an example for CAMERA_STREAM in TILE_EXAMPLES.md as we shouldn't expect people to read source code to figure out what can be passed.

The 'messy' white space between braces in conditional expressions is my code formatting style :) I also noticed that something was wrong with indentation after committing. I didn't think too much of it, as all that can be reformatted in seconds when merging. I guess I can correct it, if needed.
Agreed on the examples.

Fixed whitespace stuff and added example.

Thanks!


undefined is not an object (evaluating this._config.generator_power.is_used)

Hi there,
Great card thank you, i use it with Guy’s LuxPower inverter integration.
I ran a few updates today, core HA and in HACS and since then I have been getting this error.
I cannot find the variable in the card’s settings so I am a bit lost. Thanks
undefined is not an object (evaluating this._config.generator_power.is_used)

Hi. After updates when I change the config, the card always acts up a little. Please check issue #40 and see if that fixes it, otherwise I'll see about the fix
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Stewbo1 @.>
Sent: Saturday, August 3, 2024 9:59:02 PM
To: DanteWinters/lux-power-distribution-card @.>
Cc: Subscribed @.***>
Subject: [DanteWinters/lux-power-distribution-card] undefined is not an object (evaluating this._config.generator_power.is_used) (Issue #57)
Hi there,
Great card thank you, i use it with Guy’s LuxPower inverter integration.
I ran a few updates today, core HA and in HACS and since then I have been getting this error.
I cannot find the variable in the card’s settings so I am a bit lost. Thanks
undefined is not an object (evaluating this._config.generator_power.is_used)
—
Reply to this email directly, view it on GitHubhttps://github.com/DanteWinters/lux-power-distribution-card/issues/57, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APGA3BYIKAAUXXGCUVANHO3ZPUZANAVCNFSM6AAAAABL6FJU36VHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ2DMNJWHA4DGNI.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thank you. I followed those steps to clear cache (well on IoS so logged out/back in) and I still have it. Either the error or a partial status image which doesn’t show the output power to home/grid. I’ll move over now to #40 and provide a screenshot when I get a mo.

Closing.

please check on another device or website if you still have the issue. If the issue is still visible on web, then it's a config or card issue. If not, then it's a cache issue

Yes on Chrome on my PC I get "Cannot read properties of undefined (reading 'is_used')

Fixed by doing the forced cache clearance on all devices one by one.


Is there a way to show the error on status bar?

The eslint plugin on vim have a nice feature that if your cursor is on the line of a eslint error it shows the error on status bar, this plugin can do this too?

This should be a general concept in VS Code IMO and not implemented by the extensions. I opened https://github.com/Microsoft/vscode/issues/20183

My bad, I thought it was a plugin feature because the vim mode plugin shows which "mode" I am using (Normal, Insert, Visual)


chore(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-k8s-27-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3123

Questions ?
Please refer to the Backport tool documentation


Edit inline functionality on DataGrid

Is it possible to edit textfields in the datagrid inline? When you click the text field, it should change into an input field, and a save button should appear behind the row. When saved it should call the same action as the edit form (create/edit view) that is currently on a separate page.

I'll create my own custom EditableTextField to accomplish this behaviour.

@pixelscripter Hi! can you show me that component? I wan't to do that but the NumberFiled

Hi @jozekcore unfortunately I never finished this component, because we switched to building a custom admin.


Suggest to use upload_large_folder when using upload_folder with folders containing many files

Is your feature request related to a problem? Please describe.
I had an upload that took 40 min to fail (some windows driver messed up, not in scope for this issue); then, while reading the docs, saw the magical function upload_large_folder, with restartability, and even multiple workers! I thought "surely this just takes a bit faster" and swapped the original upload_folder function with it. In less than 30 minutes the entire 35GB folder had been uploaded onto HF, with no critical errors. This function is black magic (the good kind).
Describe the solution you'd like
If a folder is being uploaded that has over 100 files, or is onto a dataset, put a message/warning reading something alike WARNING: upload_folder cannot be restarted without losing progress and has a single worker by default. Only use this for small amounts of data., actual message can be changed.
Describe alternatives you've considered
Modify the docs for upload_folder to heavily suggest the alternative function.
Additional context
N/A

This function is black magic (the good kind).

😁

Glad you liked this black magic! Hope we don't use it too much :grin:
Regarding your suggestion, this is exactly what we did in https://github.com/huggingface/huggingface_hub/pull/2547. In theory it should be triggered on any upload_folder larger than 30 new files. The message is logged at the INFO level as we don't want to "warn" the user (there is nothing wrong in uploading 50 files with upload_folder). When you use huggingface-cli upload, the default log level is INFO so the users see the suggestion to use huggingface-cli upload-large-folder. However if you're using upload_folder in a script you'll most probably miss the suggestion. Was this your case? I think it's fine to keep it as it is now but feel free to let us know if you have another suggestion.

[...] However if you're using upload_folder in a script you'll most probably miss the suggestion. Was this your case? I think it's fine to keep it as it is now but feel free to let us know if you have another suggestion.

Yes, the original usage was in a notebook, and as for suggestion, maybe increase it from "info" to "warn" after 300 files, since as you pointed out, 50 files seems reasonable, but I feel like 300 should be enough to inform the user.


goldendict-ng: init at 23.07.23

Description of changes
Add new package https://github.com/xiaoyifang/goldendict-ng
close https://github.com/NixOS/nixpkgs/issues/239921

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

I cannot reproduce this https://github.com/NixOS/nixpkgs/issues/216396 The problem no longer exists in Qt6?
It works fine :)

Thanks for the review. The things you mentioned are implemented.
:)

Thanks for the review. The things you mentioned are implemented. Much to learn! @drupol
:)


Will express support generator or async function as request handler so there will be no callback ?

Like koa

Yes. Please see open issues :)


nodejs reference hashes are not unique

What happened?
When you navigate to https://www.nixhub.io/packages/nodejs search for the hash a3ed7406349a9335cb4c2a71369b697cecd9d351 using your browser.  You'll see the hash is in use for versions 18.19.1, 20.11.1, 21.7.1 just as an example

@ben-redvanworkshop IIUC that means that these versions of nodejs are from the "nix derivations" that are defined in the same nixpkgs commit-hash i.e. https://github.com/nixOS/nixpkgs/commit/a3ed7406349a9335cb4c2a71369b697cecd9d351
cc @gcurtis in case I'm misrepresenting this.


Documentation update - BTSubTree

Added a note about updating subtree at runtime.

The content is good, but the files under doc/source/classes/ are auto-generated using godot's make-rst.py script. You need to change related XML-file in doc_classes/  instead.

Right. So, it turns out that per the class xsd, links don't seem to be allowed, and I couldn't hardcode the rst in there either, so here it is plain :vomiting_face:
Feel free to suggest a better alternative.


Syntax highlighting active outside of module boundaries

Text up here shouldn't cause "shouldn" to be highlighted (the syntax grammar thinks "shouldn" is a primed variable).
----- MODULE M -----
...
=====

This is very similar to #117. Semantic highlighting is stable since VSCode v1.48, so it should be doable.

@lemmy the toolbox highlights keywords and comments outside of module boundaries; is this desired behavior?

I don't know if this is intentional, but I like its behavior better than highlighting don't, doesn't, .... Perhaps, the pretty-printer is the authority?

Marking as closed thanks to @edaena!


merge main to 50

better and more minimal update to latest ibc
golangci-lint run ./... --fix

note to self: root command needs to be refactored

please note that this will require another bump of the sdk, and that we're having some issues with that version of the SDK here:

https://github.com/cosmos/ibc-go/pull/4393

I'll be doing a bit more debugging there and then I should be able to bring those changes here.

https://github.com/CosmWasm/wasmd/pull/1547

Changes based on @sontrinh16 's work on ibc-go here:

https://github.com/cosmos/ibc-go/pull/4392

Every tests except the simulation test should pass now

make test-system passed in my local but it failed in CI ( out of gas)

mine also passed locally but fail in CI, I'll investigate more.

same here

Note: the test is expected to fail, but CI decide to let it fail in a weird way that it not recognize in the CI that is true to fail.

We need to figure out how to trigger ErrInvalidQuery: query wasm contract failed  @anhductn2001

Let's try merging #1567 here.  It was just approved.

I think gas will keep increase every time we bump it.

Oh, because of the thing you mentioned, right?

We need to figure out how to trigger ErrInvalidQuery: query wasm contract failed @anhductn2001

NOTE: as I researched ErrInvalidQuery should be trigger by recursion calls before OutOfGas happen. But we get OutOfGas in the CI first, which is really weird. And also the tests pass on all kind of machine but not on CI

So, the fact that this is happening in CI, but not elsewhere, is pretty interesting.  I am not certain how the system tests work, but you might want to try linting with thelper (likely just need to descend into that path) -- thelper will help you put t.Helper() in every test helper function and that will make finding the root of the thing easier.
I like your explaination about gas increaasing each time by the way and I have just one counterpoint -- lemme try a million :P.
My reasoning is that each sdk version does tweak how gas is determined somewhat.

note: if the yoloing a million gas commit works, then there's still probably a bug of some sort, somewhere.
Have a look in the .circleci folder, it is possible that the system tests are run differently in ci.

@hoangdv2429 -- one more thing -- if you want to use a linted version, this PR:

#1565

is ready to go with the relevant lints.  I kept it separate for ease of merge when @alpe gets back from vacation, but please feel free to mnerge this into that (but don't merge that into this).  I don't think it has linted system tests though.

Yes sir. We'll definitely will fix the simulation fail and test-system with @alpe 's help.

eh let's give him a nice getting back from vacation present, if we can, in the form of a completed upgrade.
Not so many vacations in this business and I'm so grateful for the support of so many while I was out having a child.  Hope to return the favor. <3

@hoangdv2429 it looks like it is different in ci.

I think the @chipshort 's commit seems to have solved the problem, try looking into it @faddat @hoangdv2429

So there is a test call TestRecursiveSmartQuery, as we can see in the test. There is one case that trigger ErrQueryInvalid and the outOfGas is expected to be false. So I guess we can working on that direction, comparing the gas consumption between running it on CI and on local, investigate more on that.

nice work @hoangdv2429 !

Thanks again for all the work. 💪
Closing this in favour of https://github.com/CosmWasm/wasmd/pull/1611


Update express to 4.17 to set sameSite: 'none' cookie

https://github.com/expressjs/express/issues/3958

This is really important. I have a production application breaking now when clients update their Chrome to a version that rejects any cookies without this set. It should be an easy fix, no? Express has already been updated to support this.

Any progress? I also have a production app that users are no longer able to log in to.

Bump. Having the same issue. Can we please support sameSite: 'none'?

Yes, need this supported please!

You should be able to get it working by resolving Express to a newer version such as 4.17.0. You can add this to your package.json file as an extra field.
"resolutions": {
  "express": "4.17.0"
}

If you are using yarn then support for this functionality is included out of the box, if you are using npm then you need to use a package such as npm-force-resolutions. This should allow you to then set the sameSite property. 🙂
https://github.com/rogeriochaves/npm-force-resolutions#how-to-use

As an update to this issue, you can fix it by deleting your package-lock.json or yarn.lock file and then reinstalling the npm packages. When you install the npm packages it should then resolve to a version of Express and you will get a updated package-lock.json file and sameSite will work 🙂

Hey @adithya321 @baileymckay @Asjas @gregegan @greggall @lejager we @the-guild-org are the new maintainers for this project. We are actively developing v2. The new yoga server is framework agnostic, so you can bring framework of your choice! Checkout our express server integration https://www.graphql-yoga.com/docs/integrations/integration-with-express would love to get your feedback on this!


Nonce it, nonce it good

Added some content for a csp/misconfiguration "attack". I tried adding a test, but it doesn't look like phantomjs was honoring it. Tests pass, just no new test.
I didn't add the language around an actual tutorial tho :ear:
I pointed livereload at a branch I made a loooong time ago. Fixes the livereload issues w/ inline script.

Just cleaning out my old pulls. Feel free to reopen :+1:

Sorry about this, we never got the rack live-reload issue sorted out IIRC and the project needs some TLC to upgrade to 4.x. Having said that, I am curious to see if the upgrade fixes any issues

On Jun 1, 2015, at 6:02 PM, Neil Matatall<EMAIL_ADDRESS>wrote:
Just cleaning out my old pulls. Feel free to reopen
—
Reply to this email directly or view it on GitHub.


  }
]

I wonder if there is a way to get more info about the masters so that masters and slaves look more similar? It's too bad this doesn't return info about the masters:
>>> state = client.get_state_summary()
>>> state.keys()
dict_keys(['hostname', 'cluster', 'slaves', 'frameworks'])
>>> state['hostname']
'<IP_ADDRESS>'
>>> state['cluster']
'my-cluster'

Can one of the admins verify this patch?

Let's keep the table in the same order, and move type to the end of the list.
You can find the mesos id of the leading master with mesos/master/state.json, but I'm not aware of a way to get the mesos id of the other masters :/

ok to test

OK, I moved the TYPE column to the end. And I also added some code that uses a special type for the leading master -- e.g.:
$ dcos node
   HOSTNAME         IP                          ID                    TYPE
 <IP_ADDRESS>  <IP_ADDRESS>  74df965e-a46e-46a9-8d2a-639db0af0e25-S0  slave
 <IP_ADDRESS>  <IP_ADDRESS>  74df965e-a46e-46a9-8d2a-639db0af0e25-S4  slave
 <IP_ADDRESS>   <IP_ADDRESS>   74df965e-a46e-46a9-8d2a-639db0af0e25-S2  slave
 <IP_ADDRESS>   <IP_ADDRESS>   74df965e-a46e-46a9-8d2a-639db0af0e25-S3  slave
master.mesos.  <IP_ADDRESS>                    N/A                    master
master.mesos.  <IP_ADDRESS>                     N/A                    master (leader)
master.mesos.  <IP_ADDRESS>                     N/A                    master

Yikes. I'm 0 for 4 on CI on this one.

test_marathon_debug.test_debug_details is failing because it uses dcos node --json which is now different.

It looks to me like that test failed because it only picked up 2 nodes instead of 3. Maybe one was down?

but it's hard for me to debug this because I haven't yet figure out how to get the integration tests to work locally for me.

Ah, the test is failing because dcos node now has masters + agents, while before it only had agents. So the test needs to be updated.

Interesting.
Weird that these didn't show up before. We're things set up so that a failing test would abort the test run before it got to other tests?
I'll work on fixing these failures later when I'm back at a keyboard. I do hope that these are all the test failures, so I won't have to iterate a bunch of times.

retest

Ah the reason why I'm having to fix the test failures one a time is that the tox.ini passed the -x option to py.test, which makes it abort on the first test failure.
I submitted https://github.com/dcos/dcos-cli/pull/964 to remove that option, as it will be helpful to find out about all the test failures at once.

I cherry-picked 17679a4 from #964 so I can see what all the failures are.

Oh my goodness. Most of the CI tasks passed!

Here's the OS X Binary failure:
args = ['--mesos-id=0bd80034-1eac-4d9b-bbf2-d2a6c559c42d-S1', '--master-proxy', '/opt/mesosphere/bin/detect_ip']
expected_returncode = 0, expected_stdout = '<IP_ADDRESS>'

    def _node_ssh(args, expected_returncode=None, expected_stdout=None):
        if os.environ.get('CLI_TEST_MASTER_PROXY') and \
                '--master-proxy' not in args:
            args.append('--master-proxy')

        stdout, stderr, returncode = _node_ssh_output(args)
>       assert returncode is expected_returncode, \
            'returncode = %r; stdout: = %s; stderr = %s' % (
                returncode, stdout, stderr)
E       AssertionError: returncode = None; stdout: = b'<IP_ADDRESS>\r\nConnection to <IP_ADDRESS> closed.\r\r\n'; stderr = b'Running `ssh -A -t -o StrictHostKeyChecking=no<EMAIL_ADDRESS>ssh -A -t -o StrictHostKeyChecking=no<EMAIL_ADDRESS>/opt/mesosphere/bin/detect_ip`\nKilled by signal 15.\r\n'
E       assert None is 0

tests/integrations/test_node.py:266: AssertionError

wat

Grrr. Now Linux Binary failed.

And now Linux Binary passed and windows-binary failed. I feel like we're playing CI Roulette here.

Yay, all CI tasks passed!  🎉

🚀


cloud: Adding admin user to a new master fails

Happened twice today -- not sure of the exact repro steps. Maybe: browser has not logged in to Kontena cloud -> add admin -> this.

Thanks for reporting this. I think I have identified the issue and fixed that. Please reopen if you still face this issue.


Not working

I opened a book, right click -> preferences -> styles -> then pasted the file.
I am not able to observe any change

In Preferences > Userstyle paste contents of CSS file, click OK
In Preferences > Theming click Save

Will close, but feel free to re-open, or open another if the above isnt' fixed.


Chore: remove manualscaler

Description of your changes

Fixes #2262
I have:

[x] Read and followed KubeVela's contribution process.
[x] Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
[x] Run make reviewable to ensure this PR is ready for review.
[x] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Special notes for your reviewer

Modification method
The modification is complicated.
The main problem may be the test code.Many test codes are constructed with manualscaler as the core component. The deletion of these codes resulted in a certain degree of test coverage reduction.
If there are any changes that are wrong, please help to point out, thanks.


Paul audio capture

Add audio real-time visualization & record for s2i model using NS.

@carloshmorales I update audio capture using s2i as an example. Please follow readme or below
EVB and GUI recording:
From the nnsp/evb/ directory:

make clean
make GUI_ENABLE=1
make deploy Prepare two USB cables. Ensure your board is connected via both the JLINK USB port and the audio USB port. Then turn on the power on EVB.
Plug a mic into the 3.5mm port, and push BTN0 to initiate voice recording
make view will provide SWO output as the device is running, showing
predicted slots/intents etc.
On your cmd, type$ python ../python/tools/audioview.py --tty=/dev/tty.usbmodem1234561

You should see a GUI popping out.
You might need to change the option --tty depending on your OS.
On your GUI, prress record to start recording and stop to stop recording. The recorded file audio.wav is under the folder nnsp/evb/audio_result/.


Error at Step #6

Method invocation failed because
[Microsoft.IdentityModel.Clients.ActiveDirectory.AuthenticationContext] does
not contain a method named 'AcquireToken'.
At C:\Balaji\ACS
Migration\spsaml11-master\scripts\MSGraphTokenLifetimePolicy.psm1:53 char:3

$Global:authenticationResult = $authContext.AcquireToken($resourceA ...

+ CategoryInfo          : InvalidOperation: (:) [], RuntimeException
+ FullyQualifiedErrorId : MethodNotFound

You cannot call a method on a null-valued expression.

@balajimadgundi The easiest way to do this is to install Azure PowerShell. Once installed, make sure to run the script exactly as demonstrated in the article:
Import-Module <file path of Initialize.ps1>  $objectid = "<Application Object ID from Table 1>" $saml2policyid = Get-PoliciesAssignedToServicePrincipal -servicePrincipalId $objectid | ?{$_.displayName -EQ "TokenIssuancePolicy"} | select objectId Remove-PolicyFromServicePrincipal -policyId $saml2policyid -servicePrincipalId $objectid $policy = Add-TokenIssuancePolicy -DisplayName SPSAML11 -SigningAlgorithm "http://www.w3.org/2001/04/xmldsig-more#rsa-sha256" -TokenResponseSigningPolicy TokenOnly -SamlTokenVersion "1.1" Set-PolicyToServicePrincipal -policyId $policy.objectId -servicePrincipalId $objectid

Thanks Kirk for response. Now I am able to execute the "$saml2policyid = Get- ..." step successfully, issue was I had old version of PowerShellGet.
But I have received below error for the next line "Remove-PolicyFromServicePrincipal.."
"(unsupported query ..)"
When I have replaced the variable $saml2policyid with actual value then it works (as pointed out in by the others on similar issue #60 https://github.com/MicrosoftDocs/OfficeDocs-Enterprise/issues/60).
Is there anything I am missing?

Updated sample code and verified in lab. PR 78


⚠ api.piste.gouv.fr healthcheck has degraded performance

In 15d9ec4, api.piste.gouv.fr healthcheck (https://api.piste.gouv.fr/cassation/judilibre/v1.0/healthcheck) experienced degraded performance:

HTTP code: 200
Response time: 3119 ms

Resolved: api.piste.gouv.fr healthcheck performance has improved in 8274dfe.