cip-19!: simplify Bitswap compostion

While the status of the CIP is still Review, we want to make a protocol-breaking amendment to Shwap prompted by the ongoing production-ready implementation. This change simplifies the protocol, particularly the Bitswap composition, by making a single protobuf definition of a Block instead of N definitions per every current and future Container type.
This change is motivated by a desire to make it easier for Shwap to extend horizontally in the abstract specification and its implementations. Previously, the new container would have to be added to both the core Shwap container specification and the Bitswap composition. Now, only a single place needs to be updated(and Bitswap codec registry).

(Noting that we should not rush moving Shwap to Final, as we figured there is one more feature needs to be added -- ability to request only Container proofs without its data)


Set crd:trivialVersions=false for easier API upgrade

We were going to set crd:trivialVersions=false for per-version
validation in 0.6. However, we cannot patch CRD if the one in v0.5 is
still using top-level schema. We would get "Forbidden: top-level and
per-version schemas are mutually exclusive" error when upgrading HNC,
specifically CRDs.
Tested by API upgrading from this to the v0.6 change in a GKE cluster.
It passed after this change and failed with the above error before. Also
tested by make test to ensure it doesn't break v0.5.

/assign @adrianludwin
/assign @rjbez17

One thing I don't understand is that the CRD yamls are not changed. I would expect there's any schema change there.
However, it did make a difference that the Forbidden: top-level and per-version schemas are mutually exclusive error is gone.

/hold
Will do more tests on this.

False alarm: the exclusive schema error only happens when I mistakenly use create instead of apply: kubectl create -f {hnc 0.5 yaml} and then kubectl apply -f manifest/hnc-manager.yaml. I should use kubectl apply -f {hnc 0.5 yaml} instead (the error is gone).
I searched online and found there's a difference but not sure how it affects this: kubectl create uses Imperative Management; kubectl apply uses Declarative approach.

Thanks for clearing that up!
On Thu, Sep 24, 2020 at 4:01 PM Yiqi Gao<EMAIL_ADDRESS>wrote:

False alarm: the exclusive schema error only happens when I mistakenly use
create instead of apply: kubectl create -f {hnc 0.5 yaml} and then kubectl
apply -f manifest/hnc-manager.yaml. I should use kubectl apply -f {hnc
0.5 yaml} instead (the error is gone).
I searched online and found there's a difference but not sure how it
affects this: kubectl create uses Imperative Management; kubectl apply
uses Declarative approach.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-sigs/multi-tenancy/pull/1145#issuecomment-698559076,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE43PZCMBBH6OJL2MHD73ULSHOQSDANCNFSM4RYU6F7Q
.


a hack to for onnxruntime to use cuda

onnxruntime may not able to use CUDAExecutionProvider even if cuda installed
a simple hack is to import torch before import onnxruntime
u may think im joking but it's explained here: https://github.com/microsoft/onnxruntime/issues/11092#issuecomment-1484756347
u can search for "Failed to create CUDAExecutionProvider" on onnxruntime github to verify, people report simply import torch helps
this PR may help closing #36 #37 #38 #45 #49

oh sh** i didn't realize torch wasn't in description
@xaviviro should i submit a pr to revert this ?

oh sh** i didn't realize torch wasn't in requirements
@xaviviro should i submit a pr to revert this ?


ZMTP not in Decode As menu on Wireshark 4.0.1

Similar to issue #16.
Per the issue and per https://stackoverflow.com/questions/59269429/unable-to-see-protocol-in-preferences-or-enabled-protocols-using-lua-in-wireshar  I tried reverting to hash 0702cc44, but that didn't fix it.
Per the issue, I'll try using wireshark 3.2.1 to see if that will work, but just a heads up to people that this dissector is no longer working with recent Wireshark versions.

Well...  wireshark 3.2.1 with latest master and with 0702cc4 also doesn't work.  Like issue #16, ZMTP shows up in the list of Enabled Protocols, so I know the plugin was picked up at wireshark startup.  I also quit wireshark each time I checked out a different hash.
Sad face as the plugin is great.  :)

I take that back.  Operator error on my part -- didn't realize that to decode as ZMTP, there is a second dropdown box on the right hand side within the Decode As window where ZMTP is selected.  I got bit by non-obvious user interface quirk.


The code is non-runnable

At the very least have the courtesy to run this code in isolation to see if it runs.....there are no import statements anywhere. I tried running it in anaconda....there are so many import statements missing I just gave up.

I just realized....it needs sage to run. Even then, I don't know that it would run out of the box. Can we get a strictly python version....please.


Problems using Tabs with Navbar

Hello,
I am trying to build a webpage with a navbar and different tabs in it.
I build the following codesnippet: http://jsbin.com/nikakararo/edit?html,output
The navbar is good working, but I have one problem: When you click first to the Button "Alle", you will the the tab for this. In this is a clickable text and this should change the tab to something other. In this case it is the word "Zangen", if you click on it, you will see the tab content of "Werkzeuge -> Zangen" but the Button "Alle" is still selected. Now it is the problem, that I can't click again to "Alle" to see this tab content again. The pages thinks that "Alle" is still clicked :/
What would be the correct way to change to another tab when clicking on text or pictures or something else? Is it possible to "manipulate" the active tab in the navbar or is it possible to "simulate" a click on it? I think it would be an easy work for you, but I can't find any solution
Regards

Hi @JumpY2k3!
You appear to have posted a live example (http://jsbin.com/cotuyelive/edit), which is always a good first step. However, according to the HTML5 validator, your example has some validation errors, which might potentially be causing your issue:

line 14, column 7 thru column 25: Element li not allowed as child of element div in this context. (Suppressing further errors from this subtree.)
line 52, column 5 thru column 40: Attribute href not allowed on element p at this point.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

@JumpY2k3 Please fix the above errors in your example.

@cvrebert The example is fixed now.

Reposting to trigger revalidation: http://jsbin.com/nikakararo/edit?html,output

Hi @cvrebert!
You appear to have posted a live example (http://jsbin.com/nikakararo/edit), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

W002:  is missing X-UA-Compatible <meta> tag that disables old IE compatibility modes
W003:  is missing viewport <meta> tag that enables responsiveness

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

Okay, those 2 warnings should be inconsequential in this case.

@JumpY2k3 Protip: Don't hotlink to http://getbootstrap.com/dist/...*; use BootstrapCDN instead.
Tweaked example: http://jsbin.com/yafefo/1/edit?html,output

Hi @cvrebert!
You appear to have posted a live example (http://jsbin.com/yafefo/1/edit), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

W002:  is missing X-UA-Compatible <meta> tag that disables old IE compatibility modes
W003:  is missing viewport <meta> tag that enables responsiveness

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

It's not valid to have  links outside of the nav
 (such as your Zangen within the .tab-pane) since this violates DOM structure assumptions that the Tab plugin makes.
I will reinterpret this as a feature request for Bootstrap v4 to remove this limitation.

Okay, thank you. Is there any other way to link to a nav like simulate a click to it? I want to use it in every tab-content for fast switch to correct content.

Yeah, you can either fire a synthetic click at it or use the Tab plugin's JS API to show it.
Example: http://jsbin.com/bodiwu/1/edit?html,output

Hi @cvrebert!
You appear to have posted a live example (http://jsbin.com/bodiwu/1/edit), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

W002:  is missing X-UA-Compatible <meta> tag that disables old IE compatibility modes
W003:  is missing viewport <meta> tag that enables responsiveness
line 52, column 5: W007: Found one or more <button>s missing a type attribute.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

Thanks for your fast response, this seems to be a perfect solution for my problem. I will check it later on my whole page & hope that it will work fine. In case of more problems with it, I will ask here again.
PS: I don't know how to set labels here, could you set the issue on closed? Thank you :+1:

Leaving this open as a feature request.


[QUERY] Delay after setting the response/mock in ApiManagement

Library name and version
Microsoft.Azure.Management.ApiManagement
Query/Question
We use the Api Management to mock endpoints. We set mock endpoints during tests and depending on which test is running, we change the mock.
With help of PolicyContract we set our mocked content:
<policies>
    <inbound>
        <base />
        <return-response>
            <set-status code="200" />
            <set-header name="Content-Type" exists-action="override">
                <value>text/xml</value>
            </set-header>
            <set-body template="liquid">
               <mockedcontent>content</mockedcontent>
            </set-body>
        </return-response>
    </inbound>
    <backend>
        <base />
    </backend>
    <outbound>
        <base />
    </outbound>
    <on-error>
        <base />
    </on-error>
</policies>

In code we use the nuget package Microsoft.Azure.Management.ApiManagement to set the PolicyContract. When we call the IApiOperationPolicyOperations.CreateOrUpdateWithHttpMessagesAsync the operation is successful and afterwards IApiOperationPolicyOperations.GetWithHttpMessagesAsync returns new updated PolicyContract. But here comes the problem. The mocked endpoint itself doesn't return the set mocked content directly. There is a delay before the endpoint is being updated. Currently we have code to check if the endpoint is updated. But this is creating other problems in our architecture.
Is this behavior attended? Should the method IApiOperationPolicyOperations.CreateOrUpdateWithHttpMessagesAsync not return successful first when the endpoint is returning the updated value?
Is there another way how to detect if the mock endpoint has really updated?
Environment
No response

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Storage:0.16514263,Service Bus:0.13891442,Cognitive Services:0.07000429'
Label prediction was below confidence level 0.6 for Model:CategoryLabels: 'Client:0.5805451,Mgmt:0.40426606,Service:0.0136808455'

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

@Yao725  please help to check if this is happened on sdk side or a service delay

I checked this behavior on https://portal.azure.com and could also reproduce this issue. If I set the response in the Design tab and afterwards quickly go to the Test tab it do not always show directly the new response.

In this case I have to execute test again to get the correct value.

Hi @schw3de, per your description, I think it is a service-related issue because our SDK worked as expected but there is a delay between setup and status update. I tagged this issue with service attention and hope we can get some help from the service team.

@Yao725 Thanks for the update!

Hi @schw3de, sorry I haven't updated this issue for a long time, it seems the service team will not respond here. Do you resolve this issue? If not, maybe you can open an ICM to get more assistances.

Hi @Yao725,
Thanks for your reply. No, the issue could not be fixed. Sorry to hear that the service team will not respond.
We will not continue to investigate the issue further with an ICM issue, because of resources.
Too bad the issue can not be resolved, but I guess there are just not enough people interested in this topic.
I wish you all the best!

Thanks for your reply. For your issue, it's hard for us to do something in SDK side to fix it, really sorry for that.

Will close this issue, feel free to open new issues if you have problems with SDK, thanks.


pkg/endpoint: Pass endpoint alive context to regeneration tasks

A per endpoint controller is started when an endpoint is being created, or restored. The controller triggers regeneration process for the endpoint that's tasked with attaching BPF programs to the endpoint device. When an endpoint is deleted, the regeneration task is expected to fail to attach BPF programs as the endpoint devices are cleaned up. In order to differentiate such expected cases from the failure ones, the code checks if the endpoint is alive via a context that's cancelled when the endpoint is deleted. However, the right context wasn't passed down to the regeneration
routines via the controller. As a result, the regeneration task would flag an error in the expected scenarios.
Fix it by calling the right function that populates the per endpoint alive context. The other places in pkg/endpoint already pass the correct context.
Fixes: https://github.com/cilium/cilium/issues/18073
Signed-off-by: Aditi Ghag<EMAIL_ADDRESS>Signed-off-by: Joe Stringer<EMAIL_ADDRESS>Release note
Fix a bug where agent would log warnings such as "JoinEP: Failed to load program" in legitimate cases where endpoints are getting deleted.

/test

We may want to evaluate backporting this even further given its effect on CI.

While inspecting the code, I found that the right context was being passed already - https://github.com/cilium/cilium/blob/master/pkg/endpoint/policy.go#L683-L683. Need to attach breakpoints to see the tangled mess of contexts. Let's hold off backports for now.


Peer to peer Code Review

Hi @JuanDiegoGuerra!
You have done an excellent job and your code is looking good! 😃

Your code is clean and organized!
The JavaScript file looks really nice, It's very compressed and easy to understand, congrats! 🥂

Keep it up and Happy coding!! 🎉🍾

Apologies for the delay, but I discovered an area in your HTML document that could be improved :

Kindly suggest enhancing accessibility by adding the aria-label attribute to each line of your input field

https://github.com/JuanDiegoGuerra/Leaderboard/blob/07b08ce464f46a1f72ec86b84e941851870632ec/src/index.html#L26
https://github.com/JuanDiegoGuerra/Leaderboard/blob/07b08ce464f46a1f72ec86b84e941851870632ec/src/index.html#L28
I admire your coding style and the way you implement the functionality. Keep up the great work and enjoy coding! :partying_face: :robot:


[ARC] Support for arguments in ARC search command

Extension
https://www.raycast.com/the-browser-company/arc
Description
Would love to have a feature, where we can add arguments in the Search Command for Arc extension.
Argument Reference - https://developers.raycast.com/information/lifecycle/arguments
This would allow for us to be able to search directly using deeplink, something like this

Who will benefit from this feature?
Anyone using deeplinks, or this extension. This also might solve an issue where Search Arc is a fallback command
Example -

After we click on search with arc, this does not actually search

Have to press space or some character for it to search.

Anything else?
No response

Cool idea, I added support for it in #5168. Thanks for the suggestion!


Web and other support

Type question here.
Wow amazing package and it works amazing right out of the gate. Just wondering if there will be web support in the future? Thank you

Hi there, the package supports Web officially. No extra configuration required, it should work immediately. Let me know if you have any issues with the Web support. Thanks!


To add a quality test for checking min and max between dates.

Checking min and max between dates is required to evaluate business use cases.

Please share the status

With new implementation of data quality the above issue should be resolved, please try and share your feedback!

Hi Shailesh,
I believe it's not resolved. Can you please look into it? Can only see
below three options for test type for a date column.
[image: image.png]
On Mon, 9 Jan 2023 at 13:56, Shailesh Parmar @.***>
wrote:

With new implementation of data quality the above issue should be
resolved, please try and share your feedback!
—
Reply to this email directly, view it on GitHub
https://github.com/open-metadata/OpenMetadata/issues/6704#issuecomment-1375356313,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2L3ZRZTNBW7MHLHSUB4DHLWRPOETANCNFSM56KTB7HQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
[image: Email header]
[image: Company logo]
Laila Patel
Data Governance Business Analyst
www.moove.io https://s1.sendassets.io/s2/onr5697j
*[image: facebook-f] * https://s1.sendassets.io/s2/ma1p4i37 *[image:
twitter] * https://s1.sendassets.io/s2/3lnrjzed *[image: instagram] *
https://s1.sendassets.io/s2/6b77t5x0 *[image: linkedin-in] *
https://s1.sendassets.io/s2/l68rh2pf
[image: Banner]

Hi, @TeddyCr can you please help here?

@lailapatel could you provide more details?

Sure.
The requirement is to have test types through which we can validate
minimum date/start date and maximum date /end date against a date column.
example : Column : National ID issuance date
test case 1 : To check National ID Issuance has max date (end date) as 31st
Dec 2002, in order to check national ids which were issued before the
defined cut off date.
test case 2:  To check National ID Issuance has min date (start date) as
1st Jan 2003, in order to check national ids which are issued after the
defined cut off date.
On Mon, 9 Jan 2023 at 14:12, Teddy @.***> wrote:

@lailapatel https://github.com/lailapatel could you provide more
details?
—
Reply to this email directly, view it on GitHub
https://github.com/open-metadata/OpenMetadata/issues/6704#issuecomment-1375377925,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2L3ZR5EA7ABHO5TINW5GJ3WRPP77ANCNFSM56KTB7HQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
[image: Email header]
[image: Company logo]
Laila Patel
Data Governance Business Analyst
www.moove.io https://s1.sendassets.io/s2/onr5697j
*[image: facebook-f] * https://s1.sendassets.io/s2/ma1p4i37 *[image:
twitter] * https://s1.sendassets.io/s2/3lnrjzed *[image: instagram] *
https://s1.sendassets.io/s2/6b77t5x0 *[image: linkedin-in] *
https://s1.sendassets.io/s2/l68rh2pf
[image: Banner]

Thanks @lailapatel we'll be looking into it.


integerType generating numbers larger than integer. #209 fix in version 3.2.3 can't be run in project using Java 1.7

In my project we are still running Java 1.7. It seems like the jars for version 3.2.3 are compiled using java 1.8, so when trying to run with this version I get this error:
An exception or error caused a run to abort: au/com/dius/pact/consumer/TestRun : Unsupported major.minor version 52.0
java.lang.UnsupportedClassVersionError: au/com/dius/pact/consumer/TestRun : Unsupported major.minor version 52.0
Which is the latest version that was still compiled with Java 1.7 and could it get this fix (#209) included?

Use the 2.x branch, it is compiled with Java 1.6. The current version is 2.4.8, which includes this fix (it was fixed in version 2.4.5)

Awesome! thanks.


Revert "Fixes build with ngc 2.3.x"

Reverts angular-redux/ng2-redux#283

reverting for now, dropping forRoot is a breaking change - will review again later and do a major bump if needed


Disorder When Pulling  Message from WorkQueue

It's weird that when I kept sending && receiving msgs ,some msgs would be received behind those sent after it .
Here is my stream info and consumer info :
Configuration:

             Subjects: test.>
     Acknowledgements: true
            Retention: File - WorkQueue
             Replicas: 1
       Discard Policy: New
     Duplicate Window: 2s
     Maximum Messages: 500
        Maximum Bytes: unlimited
          Maximum Age: 0.00s
 Maximum Message Size: unlimited
    Maximum Consumers: unlimited

State:

             Messages: 0
                Bytes: 0 B
             FirstSeq: 111,019
              LastSeq: 111,018 @ 2021-03-03T00:42:08 UTC
     Active Consumers: 1

Information for Consumer TEST > TEST created 2021-03-02T18:12:23+08:00

Configuration:

        Durable Name: TEST
           Pull Mode: true
      Filter Subject: test.1
         Deliver All: true
          Ack Policy: Explicit
            Ack Wait: 30s
       Replay Policy: Instant

State:

   Last Delivered Message: Consumer sequence: 111022 Stream sequence: 111018
     Acknowledgment floor: Consumer sequence: 111022 Stream sequence: 111018
         Outstanding Acks: 0
     Redelivered Messages: 0
    Waiting Pull Requests: 11
     Unprocessed Messages: 0

I'm using go client to send && receive msg,and would store the md5 of each msg to a file after the operation complete:
Send:
	rand.Seed(time.Now().UnixNano())

	for {
		length := (rand.Intn(9) + 41) * 104857
		data := utility.GetRandomBytes(length)

		for {
			err = Send(js, data)
			if err == nil {
				break
			}
			errlogger.Println(err)
			time.Sleep(5 * time.Second)
		}

		utility.Md5Write(recordFile, data)
		data = nil
	}

}

Receive :
	for {

		defer func() {
			if nerr := recover(); nerr != nil {
				errlogger.Println(err)
			}
			time.Sleep(time.Second)
		}()

		data, err := receive(con)
		if err != nil {
			errlogger.Println(err)
			time.Sleep(time.Second)
			continue
		}

		utility.Md5Write(storeFile, data)
		data = nil
	}
}

the amount of msg in send && receive matches ,but there are 4 msgs received in wrong order (equals to Acknowledgment floor: Consumer sequence: 111022 Stream sequence: 111018).am I using it wrong?
line : 23759 does not match Receive : 37d5b94def7f32c47609c895f50b9826  Send : c8a03cd20f215139153668a19ee5844c
line : 23823 does not match Receive : c8a03cd20f215139153668a19ee5844c  Send : 31c3e92c9897ef0ea8747fb1e92cb846

line : 31969 does not match Receive : 3e595f35081389480f17889f118b1b65  Send : c425192653b0f9c5ab634191805bfd6c
line : 32032 does not match Receive : c425192653b0f9c5ab634191805bfd6c  Send : 5dba23efcfb028104d15eeb61bced45c

line : 50163 does not match Receive : 776f6b21d6cd3382a57a9f8d3f0fae0a  Send : dc8e105e33c3e77207b137ff1dde892a
line : 50227 does not match Receive : dc8e105e33c3e77207b137ff1dde892a  Send : 77d9b824b92c972ab9ff87752c2bde72

line : 90089 does not match Receive : aee9ffc806ca6abdf8c0bb1cca6c1027  Send : e4160f778ef9c70c942c187e45a7fa26
line : 90150 does not match Receive : e4160f778ef9c70c942c187e45a7fa26  Send : 14848ba5e984f1f1cc5d2ac7923382d9

The only way to ensure strict ordering is to set MaxAckOutstanding to 1, soon as there are parallel pulls (which seems to be the case here due to waiting pull requests) you can't control order anymore.


[HotFix] [Connector-V2] Fix doris StateSerializer miss field

Purpose of this pull request

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

@zy-kkk PTAL

Please fix CI

This issue has already been fixed in https://github.com/apache/seatunnel/pull/4853.


kernel: mm: guard demand_paging.h include with kconfig

This puts a kconfig to include demand_paging.h only when demand paging is enabled.
Fixes #65506

Is there a reason for the header to be transitive in the first place? Generally with an optional API, you make the users (who have to turn on the kconfig) include it explicitly, as by definition nothing depends on it.
(I guess one answer is that headers are APIs and we're stuck with it, but IIRC demand paging isn't a stable API yet, so we're safe?)

That actually is a better idea. I've made the changes to include demand_paging.h only if needed. Hopefully CI will pass.


Update assetlist.json

request to add WXRPS to frontier assist list so we can start our own liquidity pool
Thank you

gone stale


[ORC-1062]Rename orc.schema.evolution.case.sensitive to orc.schema.evolution.case.aware

What changes were proposed in this pull request?
Currently, OrcConf have a enum configuration to determine if the comparision of field names in schema evolution is case sensitive, However, the code block using this configuration thinks that this configuration should be orc.schema.evolution.case.aware
Why are the changes needed?
code simplification
How was this patch tested?
Pass GitHub Action and jenkins

cc @dongjoon-hyun Thanks for your review.

@dongjoon-hyun  Also, the configuration default value is true, There are some compatibility problems
We have a case, In spark 2.4, the table can be queried normally even if the schema of the table and file is different in case
But in spark 3.1, we need to explicitly configure orc.schema.evolution.case.sensitive as false to query normally.(https://github.com/apache/orc/blob/aee81324507de1c5eb8e86bd7aa93a0f6b02bfd9/java/core/src/java/org/apache/orc/impl/SchemaEvolution.java#L514) this cause fileIncluded  not to include this column and cant read data
Wether we should set default value as false to maintain compatibility.

@yikf, Thank you for making this pr, should we also modify this test
https://github.com/apache/orc/blob/aee81324507de1c5eb8e86bd7aa93a0f6b02bfd9/java/core/src/test/org/apache/orc/impl/TestRecordReaderImpl.java#L119-L126

modify

Done

@yikf, Thank you for making this pr, should we also modify this test
https://github.com/apache/orc/blob/aee81324507de1c5eb8e86bd7aa93a0f6b02bfd9/java/core/src/test/org/apache/orc/impl/TestRecordReaderImpl.java#L119-L126

Done

Sorry, but I'm -1 for this breaking change, @yikf .

Shall we modify the semantics of the orc.schema.evolution.case.sensitive？The default value is true, This indicates that it is not sensitive, But in fact, it is regarded as sensitive when used

The default value is true, This indicates that it is not sensitive

@yikf, It looks like you have some misunderstanding of the source code, the default value is true which means it is sensitive.


GENERAL_ERROR: Failed to launch the browser process!

homeserver@homeserver:/app$ sudo docker compose up firefly-importer
[+] Running 4/0
 ✔ Container firefly-db        Running                                                                 0.0s
 ✔ Container traefik           Running                                                                 0.0s
 ✔ Container firefly           Running                                                                 0.0s
 ✔ Container firefly-importer  Created                                                                 0.0s
Attaching to firefly-importer
firefly-importer  |
firefly-importer  | ><EMAIL_ADDRESS>start
firefly-importer  | > node src/index.js
firefly-importer  |
firefly-importer  | [2024-03-24 20:20:49.793] DEBUG (19): Config file './config.yaml' loaded.
firefly-importer  | [2024-03-24 20:20:49.794] INFO (19): Starting Israeli Bank Firefly iii Importer
firefly-importer  |     version: "1.4.1"
firefly-importer  | [2024-03-24 20:20:49.794] INFO (19): Getting state from firefly...
firefly-importer  | [2024-03-24 20:20:49.818] INFO (19): Getting scrap data...
firefly-importer  | [2024-03-24 20:20:49.819] DEBUG (19): Scrapping...
firefly-importer  |     options: {
firefly-importer  |       "companyId": "hapoalim",
firefly-importer  |       "startDate": "2019-03-24T20:20:49.819Z",
firefly-importer  |       "args": [],
firefly-importer  |       "combineInstallments": false,
firefly-importer  |       "showBrowser": false,
firefly-importer  |       "outputData": {
firefly-importer  |         "enableTransactionsFilterByDate": false
firefly-importer  |       },
firefly-importer  |       "additionalTransactionInformation": true
firefly-importer  |     }
firefly-importer  | [2024-03-24 20:20:49.912] ERROR (19): Unexpected error while scrapping
firefly-importer  |     options: {
firefly-importer  |       "companyId": "hapoalim",
firefly-importer  |       "startDate": "2019-03-24T20:20:49.819Z",
firefly-importer  |       "args": [],
firefly-importer  |       "combineInstallments": false,
firefly-importer  |       "showBrowser": false,
firefly-importer  |       "outputData": {
firefly-importer  |         "enableTransactionsFilterByDate": false
firefly-importer  |       },
firefly-importer  |       "additionalTransactionInformation": true
firefly-importer  |     }
firefly-importer  |     error: {}
firefly-importer  | [2024-03-24 20:20:49.912] ERROR (19): hapoalim  failed with type GENERAL_ERROR: Failed to launch the browser process!
firefly-importer  | Failed to move to new namespace: PID namespaces supported, Network namespace supported, but failed: errno = Operation not permitted
firefly-importer  | [0324/202049.845670:FATAL:zygote_host_impl_linux.cc(202)] Check failed: . : Operation not permitted (1)
firefly-importer  |
firefly-importer  |
firefly-importer  | TROUBLESHOOTING: https://github.com/puppeteer/puppeteer/blob/main/docs/troubleshooting.md

Any help please?

You are using docker compose that is not part of this project.
In order to run a container, please follow the following instructions here.
If you are trying to do something else following the troubleshooting guidelines link in the error log

You are using docker compose that is not part of this project [...]
AFAIK, there's no difference between spinning a container using the docker CLI and compose. The instructions include a Docker section.

For anyone reading this, that's my compose:
  firefly-importer:
    image: itair86/israel-bank-firefly-importer:1.2.3
    container_name: ${SLUG}-importer
    restart: always
    volumes:
      - ./.config:/home/pptruser/app/config.yaml
    depends_on:
      - firefly

Thank you for the information, it might benefit for others
This project wasn't tested against Hapoalim, so your feedback will be beneficial
Regarding your errors, those are from this project https://github.com/eshaham/israeli-bank-scrapers
I recommend to check if those issues are known and report if needed

Due to response I'm closing this ticket, please open a new ticket if needed


Updating user guide

Include instructions for sort and paid command in the User Guide
Closes #115

LGTM.


I have a bug with the engine, but it might just be me?

What is your question?
PlayState.hx (Line 7644)
PlayState.hx (Line 6463)
PlayState.hx (Line 5994)
openfl/events/EventDispatcher.hx (Line 426)
openfl/display/DisplayObject.hx (Line 1398)
Uncaught Error: Null Object Reference
Please report this error to the GitHub page: https://github.com/JordanSantiagoYT/FNF-PsychEngine-NoBotplayLag

Crash Handler written by: sqirra-rng
The engine has saved a crash log inside the crash folder, if you're making a GitHub issue you might want to send that!

That's the crash stuff above.
I don't know if I deleted or replaced something that is important but, what this bug is, is that whenever I hit the first note in my songs, I always crash. But if I miss the first note and hit the second and the rest of the notes, its fine?
Please help me.🥹🙏

You added custom rating quotes, but you forgot to put one for 100%

You added custom rating quotes, but you forgot to put one for 100%

I am sorry, but I don't understand what you mean?

I am sorry but i dont know what you mean

basically in 1.26.0 i added the ability for you to customize your rating quotes, which added a ratingQuotes folder in assets/data. if that's not there, you basically deleted it.

i did, bro i feel so stupid 😭
are you able to send me that file? or am i going to have to re-download it??

ratingQuotes.zip

maybe I should make it more clear not to delete that file, or add a failsafe if you do


Update circle-ci config

Would be great to add a few standard enhancements to the circle-ci setup:

Cacheing: It would be good to cache certain steps. For example, currently we re-install pytorch from scratch (which is actually a fairly large download).
Build Docs: Building the docs with circle-CI would be helpful because the 3gb memory of readthedocs is limiting for one of the example notebooks
Release to GH

Feedback welcome on anything else that is standard / best practice.

@jwdink what's the priority level on this?

This is fairly low priority


Dhcp static leases - IPv4 Address field too small

While editing/browsing static leases on the Network→DHCP and DNS page all the IPv4 address are obscured by the pull-down menu button. While editing one of the address entries the column becomes wider and all entries are readable. The lease Time Field is wider than required.

A possible work around would be too increase the size of the table header by adding extra text.
replace
/usr/lib/lua/luci/model/cbi/admin_network/dhcp.lua:308
ip = s:option(Value, "ip", translate("IPv4-Address"))
with
ip = s:option(Value, "ip", translate("IPv4-Address xxx.xxx.xxx.xxx"))

I don't see any problem here. We're using dynamic, mostly responsive templates in LuCI - without any hardcoded table structure/measurements. During editing you'll get/see always the full content (same applies to other fields e.g. like MAC address), after that you'll see the full content or a part of it - depends on (mobile) device, screen resolution, browser window/tab size etc. ... in my opinion it works as intended.
Your proposed "fix" doesn't change that at all.

It doesn't work as intended, the last octet is always hidden by the button of the combo-box.
Inspecting the element in the browser shows the width is set to 100%
.table.cbi-section-table textarea, .table.cbi-section-table select {
    width: 100%;
}

Setting it to auto will resolve the issue however I am not sure where to make this change.
.table.cbi-section-table textarea, .table.cbi-section-table select {
    width: auto;
}


can't set border color for trigger-style?

Can't seem to set a border color for the trigger swatch.
So when white is selected on a white background you can't see the swatch.
Tried in the props objects & CSS.
These are my current trigger styles:
      triggerStyles: {
        width: '32px',
        height: '32px',
        borderRadius: '4px',
      }

Is this possible at all? And any hover styles possible?
thanks, D.

I'm not seeing any borderWidth or borderColor in your snippet. If you still have this problem, can you add a codesandbox?


WIP: Counter Refine

WIP. Changes:

[ ] WIP: Rearrange counter into multiple files to keep it clean.
[x] Introduce LocalCounterPrimitive, which is some kind of a LocalCounter but not attached to any existing Counter. TiKV self-implemented similar modules, which is a duplication. I think this library can provide such primitives so that we don't need to implement it again in TiKV.
[x] Introduce ICounter trait (interface), to make Counter, LocalCounter, LocalCounterPrimitive compatible with each other (i.e. can be merged into another).
[x] inc_by -> try_inc_by (#151)
[ ] WIP: LocalCounterVecPrimitive.
[ ] WIP: Eliminate locks in LocalCouter (question: do we really need remove_label_values in LocalCounter?)
[ ] More to come.

Similar refinements will be applied to Histogram as well, later.

@siddontang ideas or suggestions?

IMO, I think ICounter is needless and introduces complexity. Why the Counter needs merge or flush_to?  Is it fine to let a Counter flush to a local counter?
Btw, why do we need a CounterPrimitive? why not only providing a LocalCounter directly?

@siddontang It's mainly for:

CounterPrimitive flush to CounterPrimitive (use case)
CounterPrimitive flush to Counter (use case)
CounterPrimitive flush to LocalCounter (use case)
LocalCounter flush to Counter

So I make them ICounter in order to allowing flushing to each other.

Btw, why do we need a CounterPrimitive? why not only providing a LocalCounter directly?

Because LocalCounter is binded to a specific Counter, but sometimes we just need a simple counter (i.e. LocalCounterPrimitive) that can be flushed to somewhere else (maybe another LocalCounterPrimitive) and finally be flushed into a real Counter (i.e. LocalCounter). For example, this ScanCounter is a self-implementing a LocalCounterPrimitive. Once we have a LocalCounterPrimitive, these code can be simplified.
Also, please consider some more complex use cases, for example, it is not a counter, but a CounterVec, or a HistogramVec. Self-implementing such modules is some kind of duplication. If we could just provide very primitive structures for handling these tasks, it will be nicer.

This is a Prometheus library, please notice it. So why we need to provide a primitive type for other use?
Don't over design.

@siddontang Simply speaking, is to provide a metrics collecting facility which is not binded to a specific metrics at the time of constructing. This brings better flexibility.
For example, for a single function, by using this primitive type, it can "tell others what kind of metrics I have" and the caller is free to collect some of them into a specific global metric, instead of letting the caller to inform the function (or hard coded a target metrics in the function) "what metrics you need to collect to".
This pattern is widely used in TiKV. e.g. in coprocessor executors, the executor itself is responsible for collecting seek counts, point lookup counts, etc. However the executor does not know where these counts should be collected into (because of lacking some other knowledge). The caller deep outside is responsible for aggregating all useful informations (labels) and its metrics into a global or local metrics.

Split into several PRs later. Closing.


Add link to troubleshooting process in all our install docs

Prominently and at the top
https://www.mattermost.org/troubleshoot/

I've added a note to the troubleshooting guide to the docs I thought needed it.  Could whoever reviews the PRs let me know whether they think any other doc needs a reference to the troubleshooting guide?  Thanks!


Handle open_result message that arrives after the delete

Opening a database is asynchronous, therefore it's possible for the
database to have been deleted before the open is complete. The
open_result code assumes this doesn't happen, so when it does,
couch_server crashes, taking the couch_dbs ETS table with it.
Explicitly handle the case and keep on keeping on.
COUCHDB-3241

+1


Prevent stack overflow in completion

Adds a visited set that keeps track of already seen grammar features. The stack overflow can be experienced in some parts of the langium-lox language example.

Closing in favor of a more intricate refactoring, see https://github.com/langium/langium/pull/623.


[FLINK-25654][network] Remove the redundant locks in SortMergeResultPartition

What is the purpose of the change
After FLINK-2372, the task canceler will never call the close method of ResultPartition, this can reduce some race conditions and simplify the code. This PR aims to remove some redundant locks in SortMergeResultPartition.
Brief change log

Remove the lock of SortMergeResultPartition#writeSegments
Remove the lock of SortMergeResultPartition#hasNotifiedEndOfUserRecords
Remove the lock in PartitionSortedBuffer

Verifying this change
Please make sure both new and modified tests in this PR follows the conventions defined in our code quality guide: https://flink.apache.org/contributing/code-style-and-quality-common.html#testing
This change is already covered by existing tests, such as SortMergeResultPartitionTest and PartitionSortedBufferTest.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@TanYuxin-tyx Thanks for the update. Changes LGTM. Will merge after tests pass.


Move to g++ and 11 clang++ 12 in Github Actions

If you change docker, can you please update how to build doc


[Request] Possibility for plugins to provide custom metadata for querying

I'll start off by saying that I have no idea whether this is even feasible, I haven't the slightest idea if the code can easily be extended to support this feature, or if it requires an entire rewrite. Feel free to close this issue if you feel that it would not be possible.
Description
Problem statement: plugins may extract their own metadata from notes, which are not necessarily contained within the frontmatter
Example 1 (arbitrarily chosen simple plugin)
For every note in the vault, the plugin keeps track of whether a note was changed during the current session (since app start-up).
In-memory, this plugin contains a mapping of changed: Record<TFile, boolean>, with a 'true' result indicating that the file has indeed been changed.
The user of the plugin would like to be able to use this metadata to get an overview of all the files that they changed located within a certain folder, using the datacore plugin they already have installed.

Example 2 (my plugin)
The plugin processes custom syntax and renders it within notes. All the occurences of the syntax are also being stored within a database (something akin to cache: Record <TFile, Syntax[]>), where the syntax itself can contain a list of custom metadata.
Similarily to the previous example, the user wants to list all the occurences of the syntax within a certain folder, but also filter it on the custom metadata within the syntax itself.

Idea
Each of the plugins described above is able to provide custom metadata given a file, which could in turn be used when querying data.
The existence of these datatypes could be registered as such:
export class MyPlugin extends Plugin {
	database: Record<TFile, boolean>;
	async onload() {
		this.app.workspace.onLayoutReady(() => {
			if (datacore)
				this.registerAPI(
					datacore.registerMetadataProvider('myplugin', (file: TFile) => database[file])
				);
		});
	}

	onDatabaseUpdate() {
		datacore?.triggerQueryUpdate();
	}

	// And some code for unregistering metadata provider on plugin unload
}
(Example code)

Once registered, users would now be able to also use the metadata provided by the plugins within their query requests:
```datacore
LIST
FROM #unfinished
USING myplugin
WHERE myplugin = true
```
(Example based on current Dataview syntax, names/keywords used are just for demonstrative purposes)

i think it won't be possible like this but you could just write an simple api that gives access to the data so that you can load your data inside the js.
for example currently you can already do something like this in dataview by loading the api with app.plugins.plugins.myplugin.api or something like this (this is how you can load the dvjs api inside other plugins.)

There are two aspects to this:

Having plugins add new objects to the datacore storage (which can then be queried / loaded / cached) is fairly easy - all you need to do is use the datastore store APIs and provide some types to store.
Having plugins add new fields to existing objects is a bit harder - essentially, you would have to hook into some post-processing whenever datacore reindexes a page so you can add new fields to them. You would also need to be able to ask datacore to re-index an object because some of your local metadata has changed.

I think (1) is very easy to support and is essentially already supported. For (2) I would need to provide hooks for allowing objects to be decorated with new fields, and to re-index objects outside of the regular page reload. Maybe some basic APIs for that aren't so bad.


Mismatch between some icons on the lucide.dev/icons site and the Reflex code

I had to go into the Lucide Icons Reflex source code file to figure out what name I had to use
For instance https://lucide.dev/icons/chart-pie -> pie_chart and https://lucide.dev/icons/chart-area -> area_chart
whereas https://lucide.dev/icons/book-open -> book-open
This is somewhat misleading

This is caused by us using a specific version of Lucide Icons (to avoid apps breaking all the time due to changing name, we're fixing the version at some point)
However, the lucide.dev sometimes changes names, and they don't offer a view of "names" for old versions.
We'll make our own icons docs page, with all the actual names, to solve that problem.

hey there, I would like to pick this up and work on it as my first ticket. Can I get some more details on it?

This is caused by us using a specific version of Lucide Icons (to avoid apps breaking all the time due to changing name, we're fixing the version at some point) However, the lucide.dev sometimes changes names, and they don't offer a view of "names" for old versions.
We'll make our own icons docs page, with all the actual names, to solve that problem.

@Lendemor
https://icon-sets.iconify.design/
https://iconify.design/docs/icon-components/ (Guide)
Consider wrapping this in a single API to integrate various icon libraries. This approach supports lucide and React, but it would be better to deprecate the classical rx.icon in favor of something like rx.iconx. This library is for development (--save-dev) and requires an additional decision on whether to install it as a development library or not. It dynamically loads icons when the webpage loads, preventing bloat and ensuring fast loading times.

@mohammed-anwar404 is this still on your to do list? If not, I may want to take a stab at it. LMK


az login --use-device-code error

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az login
Errors:
local variable 'code' referenced before assignment
Traceback (most recent call last):
Local\Temp\pip-unpacked-wheel-s90ugolk\knack\cli.py, ln 215, in invoke
azure\cli\core\commands\__init__.py, ln 654, in execute
azure\cli\core\commands\__init__.py, ln 717, in _run_jobs_serially
azure\cli\core\commands\__init__.py, ln 710, in _run_job
AppData\Local\Temp\pip-unpacked-wheel-7f66x3i5\six.py, ln 703, in reraise
azure\cli\core\commands\__init__.py, ln 688, in _run_job
azure\cli\core\commands\__init__.py, ln 325, in __call__
pip-unpacked-wheel-_1bqfhco\azure\cli\core\__init__.py, ln 784, in default_command_handler
azure\cli\command_modules\profile\custom.py, ln 158, in login
pip-unpacked-wheel-_1bqfhco\azure\cli\core\_profile.py, ln 202, in find_subscriptions_on_login
pip-unpacked-wheel-_1bqfhco\azure\cli\core\_profile.py, ln 889, in find_through_interactive_flow
UnboundLocalError: local variable 'code' referenced before assignment

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az login --use-device-code

Expected Behavior
Environment Summary
Windows-10-10.0.18362-SP0
Python 3.6.8
Installer: MSI

azure-cli 2.15.0

Additional Context

az login

Duplicate of #15961


Fix typo in sparse_pauli_op.py

Summary
fixed typo.
Details and comments
indicies -> indices

@Mergifyio backport stable/1.0 stable/0.46


unable to load '@google-cloud/datastore'

Hi, when I am trying to load '@google-cloud/datastore' in electron app I am getting error as
A JavaScript error occurred in the main process
Uncaught Exception:
Error: /home/heresy/electron_apps/app_2/node_modules/grpc/src/node/extension_binary/grpc_node.node: undefined symbol: _ZN2v86Object14SetHiddenValueENS_5LocalINS_6StringEEENS1_INS_5ValueEEE
    at Error (native)
    at process.module.(anonymous function) [as dlopen] (ELECTRON_ASAR.js:173:20)
    at Object.Module._extensions..node (module.js:583:18)
    at Object.module.(anonymous function) [as .node] (ELECTRON_ASAR.js:173:20)
    at Module.load (module.js:473:32)
    at tryModuleLoad (module.js:432:12)
    at Function.Module._load (module.js:424:3)
    at Module.require (module.js:483:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/home/heresy/electron_apps/app_2/node_modules/grpc/src/node/src/grpc_extension.js:38:15)

We don't yet support electron. Work on that is ongoing in pull request #8601.

The npm module<EMAIL_ADDRESS>should now work on electron. Make sure that you follow the instructions in the electron docs for installing native modules. You should be able to omit the line export npm_config_build_from_source=true to install the precompiled binary file for electron.


Reasons to use EEL for Data Ingestion at REST over Spark

For documentation???
Spark is great at parallel processing data already in a distributed store like HDFS but it's not really designed for ingesting data at REST from a non-distributed store like a Local File System though there is support for it, i.e. local mode.
The disadvantage of ingesting data at REST from a local file system:

There's no advantage in using YARN on a local file system as its not a distributed store - you would need to distribute the data beforehand which defeats the purpose
Though it's tempting to use local mode with all the supported file formats that come out-of-box with Spark you are still faced with the same issues mentioned previously. Moreover in local mode all partitioned data is distributed across N threads in the same process as the client which can become a memory bottleneck.
Memory is also a bottleneck If you decide to collect the data in Spark...this pulls all the data into memory before you process - collect is not stream based like a Java InputStream or a JDBC ResultSet iterator
Spark does support JDBC datasets but you still need to provide a partitioning strategy so that Spark can split your query into multiple select statements for each partition - therefore it's possible to get more throughput but if you are saving to HDFS you can end up with lots of small files - not good for Hadoop. With EEL you have more control because you can specify N ioThreads for your sources and sinks, i.e. use more threads to read in parallel for your Source and fewer on your Sink resulting in sensible file sizes if you are writing to HDFS

I think the collect issue is the same in eel anyway - if you collect into memory, doesn't matter if its stream based input or not.
Local mode with Spark does work, so that isn't an issue really. Although it never seems proper. Any docs that state they don't want you really using it ?


💡[FEATURE]:  adding the financial analysis on twitter data

Is your feature request related to a problem? Please describe.
This repository contains code and datasets for conducting financial analysis on Twitter data. Explore Part 1 for EDA and Part 2 for sentiment analysis. See README for details
Additional context
Add any other context or screenshots about the feature request here.

@Akshat111111 can you please assign me this task

use web scrapping


Update README.md

fix gitter link

Sorry I missed this, but thank you for showing me how to do the link properly :). Incorporated this into the README.md at https://github.com/spinsphotonics/fdtdz/commit/59850e3d88be6df1259690cc3a3e0de5b60a5348


url_redirect issue on net-glb module

I found an issue when i try to create route_rules with neg-glb module.
Description:
url_redirect fails with route_rules.value is object with 6 attributes  This object does not have an attribute named "default_url_redirect".
So i see that there is no default_url_redirect attribute in route_rules in variables-urlmap.tf but url_redirect dynamic block [link]tries to use this attribute and hence it fails with above mentioned error.
Solution:
We can fix this If we can replace the attribute route_rules.value.default_url_redirect with route_rules.value.url_redirect in url_redirect dynamic for_each block
FYI:

Thanks for this!


Add Termination Conditions to Broyden

Needs to wait on https://github.com/SciML/DiffEqBase.jl/pull/878

Make sure to follow up with some docs in NonlinearSolve, and TerminationCondition should get a doc page of its own.


Implement stop_listening

Close all listening connection to stop accepting new clients.
Part of graceful shutdown procedure.

@redixin thanks


R.swift warning (false positive)

Hi. I got a warning in my project using R.swift:
warning: [R.swift] Destination view controller with id XcG-7G-HZh for segue showTroubleshoot in HomeMenuViewController not found in storyboard HomeMenu. Is this storyboard corrupt?
I already tried to change the segue to another name, delete and create again the storyboard reference, among other things, but nothing worked.
What's the solution?
Thanks

Could be that the segue is in the XML, but not showing up in the UI since the destination view controller is not there. You could check that by opening the storyboard file as a source code file.
If it really is a false positive it would be nice to have a example storyboard with this problem so we can investigate what case we're missing while parsing storyboards.

Closed due to inactivity.


--name-only option for ACD

ncw,
Can we get a --name-only option for ACD ignoring the mod time, md5 and size?

Could you elaborate on the use case for that feature?

It will round out the options available for ACD. I.e. we have the md5 mod date, and size-only, also known to us is the file name on ACD.
Another use case is when encrypting files on the fly before they are uploaded to ACD. I can use gpg to encrypt a file and the size and md5 will be different every time, so it will essentially update this file on ACD.  Adding the --name-only option should mitigate this.

So in effect it is a "skip-existing-files" behaviour?

That's what I'm picturing. If the option --name-only is set and you come across a file of the same name on ACD, it should skip this file.

Back to the rsync man page I see these useful looking options...
            --existing              skip creating new files on receiver
            --ignore-existing       skip updating files that exist on receiver

I think the second one of these would be equivalent to --name-only but better named and with a precedent from rsync.
How does that sound?

I think that is along the lines of what I was mentioning, thanks for the find.

Added PR #276.


make creatures have multiple hitboxes

this was going to be inevitable, ngl
will be somewhat hard to pull off though

lmao


Expense management with GraphQL

approve/reject mutations
added add funds using the createOrder mutation
PayPal connection

Coverage decreased (-0.2%) to 76.831% when pulling 5f4d740f5cc07b4af50ec9b1656dbfc32e0c98b9 on expenses into 0d329a8cb62ec87810a7296da38b039a8f134bd5 on master.

Coverage decreased (-0.2%) to 76.489% when pulling c8a7f85b6013b025e856028c568113d705e68eb1 on expenses into 572ac6e5b802e2c0937eb105b499c7e4c0e8d4ea on master.

Coverage decreased (-0.2%) to 76.485% when pulling cb51ec5dd3a47889e599a0286e216127d9e3a93e on expenses into 63768a36538c57c6f23c28065977954040aa3c7e on master.

Coverage decreased (-0.2%) to 76.485% when pulling cb51ec5dd3a47889e599a0286e216127d9e3a93e on expenses into 63768a36538c57c6f23c28065977954040aa3c7e on master.

Coverage decreased (-0.2%) to 76.485% when pulling b5e9136476d9feccc572f5cd36c6e48ecf6f1307 on expenses into 63768a36538c57c6f23c28065977954040aa3c7e on master.

Coverage decreased (-0.2%) to 76.505% when pulling cde1708539ae3c77426297e8db70994a1a9695a0 on expenses into 63768a36538c57c6f23c28065977954040aa3c7e on master.

Coverage decreased (-0.4%) to 76.313% when pulling e0b16aed6e7b4dda57d27c136c6e8715df2ff7b0 on expenses into 63768a36538c57c6f23c28065977954040aa3c7e on master.


[New Dashboard] Fix drag and drop behaviour

This PR fixes this issue that @ramiroaznar posted on the Feedback Form:

After importing a .carto into the home, the list of Your maps is not updated. Same is happening when importing a dataset. But the import also generates a map which I think it is not the expected behavior. Should we allow users to import stuff into the home? Should CARTO understand what is a dataset and what is a .carto?

The new behaviour is:

Drag and drop from Home:

When importing a dataset, it doesn't create a new visualization.
When importing a .carto or .zip, it creates a new visualization.

Drag and drop from Maps Page:

When importing a dataset, a .carto, or a .zip, it creates a new visualization.

Drag and drop from Dataset Page:

When importing a dataset, it doesn't create a new visualization.
When importing a .carto or .zip, it creates a new visualization.

Maybe this is a long shot but... could it be possible to avoid generating a new dataset (like table_1) if the table already exists when importing a .carto?
cc @javitonino

That may be a reaaaally long shot :P, because I suppose that we'd have to check that the table is actually identical, not only by name or something naïve.

Yeah, what @jesusbotella said. There is a parameter to the import API that can be used for this (collision_strategy=skip) but it matches only by name. So that's why it's not the default.

retest this please


Cannot login to private server, throws Error: Not Authorized

When I start the private server via the Steam UI, call setPassword("BBCorn", "1234") and then try to start multimeter using multimeter -s private, I get Error: Not Authorized.
My .screeps.yaml:
servers:
  main:
    host: screeps.com
    secure: true
    token: '00000000-0a0a-0a00-000a-a0000a0000a0'
  ptr:
    host: screeps.com
    secure: true
    path: '/ptr'
    token: '00000000-0a0a-0a00-000a-a0000a0000a0'
  season:
    host: screeps.com
    secure: true
    path: '/season'
    token: '00000000-0a0a-0a00-000a-a0000a0000a0'
  private:
    host: <IP_ADDRESS>
    port: 21025
    secure: false
    username: BBCorn
    password: 1234
  configs:
  multimeter:
    plugins: []
    aliases: {}
    logFilename: 'multimeter.log'

It does work if I make the same request with a REST Client:

Am I doing something wrong?

Try quoting the password in your config file, since its all numeric, its likely being read as a number rather than a string, which might cause it to fail auth


DHCPv4 client

Hi
This is an attempt at creating a DHCP client for IPv4. Please give feedback.
Cc: @phil-opp, this PR is uses the work merged in #75.
I've taken a completely different approach compared to #63. UDP sockets don't allow us to send and receive on unspecified/broadcast IPv4 addresses. Therefore IPv4/UDP is parsed/serialized on top of a RawSocket.
There are still two problems with RawSocketBuffer:

How do I let the user provide storage? I have tried but failed to accept that as parameters in Client::new(). I've found the proper lifetime relation by now.
The RawSocket is getting stuck with Error::Exhausted when contig_window is too small for the DHCP egress packet. Somehow it appears to behave like sent packets don't get dequeued from the tx_buffer. Is that known? What is my code doing wrong regarding RawSocket? See #187

:umbrella: The latest upstream changes (presumably 6c363762845e69dd316617f7ae0a083b4e534d5b) made this pull request unmergeable. Please resolve the merge conflicts.

I'm using this branch (rebased on master, which was mostly painless), and it works fine for me. The only thing I changed was to increase the discover timeout since my DHCP server was too slow in sending replies, and the sequence number of the reply never matched the last request.

I'm sorry for the delay in review.
Can you please extract the changes to wire/dhcp.rs first so we can merge that (I don't see anything wrong on the first glance) and then discuss the rest? I think your overall approach is fine but some API consistency changes would be nice

Would this PR be a suitable starting point for implementing PXE for no_std environments?  If so, do we have a estimate of when it may land?

:umbrella: The latest upstream changes (presumably 1fbcfb09e1b4165a2778d6ab55bbbc462444dfa1) made this pull request unmergeable. Please resolve the merge conflicts.

:umbrella: The latest upstream changes (presumably 58a54730dba398aaa5ea88e6c6f89e0234f89f42) made this pull request unmergeable. Please resolve the merge conflicts.

Rebased this one.

@whitequark Is there anything we (I) can do to move this forward?

@whitequark what's the status of this PR?

I wonder if we should temporarily fork smoltcp…

I'll take a look at this soon.

Rebased and added logging.

There seems to be an issue with the renew code. It throws a Truncated error for me:

Fixed this is in bc498fa7e9d883609962f76e860501eb3fe31c05 and subsequently corrected some fields in the Renew packets.

Thank you! Can you please squash the commits so that the history would be more sensible?

Thank you! Can you please squash the commits so that the history would be more sensible?

Into just one commit? Is that really easier on the eyes?

Into just one commit? Is that really easier on the eyes?

Ideally, into logical chunks. I group my commits in such a way that they introduce functionality part by part, with every intermediate commit building and passing tests.

Here we go!

[…] Can we use UDP sockets here?

I've tried but the dst_addr checks in process_ipv4() prevent UDP sockets from receiving DHCP responses to the address yet to be bound. What do you suggest?

I've tried but the dst_addr checks in process_ipv4() prevent UDP sockets from receiving DHCP responses to the address yet to be bound. What do you suggest?

Let's relax that check.

@whitequark Addressed the last review and squashed.

@m-labs-homu r=whitequark

@astro: :key: Insufficient privileges: Not in reviewers

As the recently introduced use of a UdpSocket has removed our ability to control the packet's sender address, I have added more opportunities for the user to assign the right address. Please review 8333045.

@astro I tried the updates and I now get a DHCP NAKs with message "lease not found" from the server on every DHCP request. The code in d834b20d7df07dca7f883973ae8b9d2da90ee000 still works fine.

@astro I tried the updates and I now get a DHCP NAKs with message "lease not found" from the server on every DHCP request.

Could you please paste the output of tcpdump -ni $interface -vv udp port 67 or udp port 68?

Could you please paste the output of tcpdump -ni $interface -vv udp port 67 or udp port 68?

Output
16:24:37.689198 IP (tos 0x0, ttl 64, id 0, offset 0, flags [DF], proto UDP (17), length 281)
    <IP_ADDRESS>.bootpc > <IP_ADDRESS>.bootps: [udp sum ok] BOOTP/DHCP, Request from 00:08:dc:ab:cd:ef (oui Unknown), length 253, xid 0x3, Flags [none] (0x0000)
	  Client-Ethernet-Address 00:08:dc:ab:cd:ef (oui Unknown)
	  Vendor-rfc1048 Extensions
	    Magic Cookie 0x63825363
	    DHCP-Message Option 53, length 1: Discover
	    Client-ID Option 61, length 7: ether 00:08:dc:ab:cd:ef
16:24:37.689266 IP (tos 0x0, ttl 64, id 0, offset 0, flags [DF], proto UDP (17), length 292)
    <IP_ADDRESS>.bootpc > <IP_ADDRESS>.bootps: [udp sum ok] BOOTP/DHCP, Request from 00:08:dc:ab:cd:ef (oui Unknown), length 264, xid 0x4, Flags [none] (0x0000)
	  Client-Ethernet-Address 00:08:dc:ab:cd:ef (oui Unknown)
	  Vendor-rfc1048 Extensions
	    Magic Cookie 0x63825363
	    DHCP-Message Option 53, length 1: Request
	    Client-ID Option 61, length 7: ether 00:08:dc:ab:cd:ef
	    Server-ID Option 54, length 4: _gateway
	    Parameter-Request Option 55, length 3:
	      Subnet-Mask, Default-Gateway, Domain-Name-Server
16:24:37.791654 IP (tos 0x0, ttl 64, id 0, offset 0, flags [none], proto UDP (17), length 576)
    _gateway.bootps > <IP_ADDRESS>.bootpc: [udp sum ok] BOOTP/DHCP, Reply, length 548, xid 0x4, Flags [none] (0x0000)
	  Client-Ethernet-Address 00:08:dc:ab:cd:ef (oui Unknown)
	  Vendor-rfc1048 Extensions
	    Magic Cookie 0x63825363
	    DHCP-Message Option 53, length 1: NACK
	    Server-ID Option 54, length 4: _gateway
	    MSG Option 56, length 15: "lease not found"
16:24:38.713594 IP (tos 0x0, ttl 64, id 0, offset 0, flags [DF], proto UDP (17), length 292)
    <IP_ADDRESS>.bootpc > <IP_ADDRESS>.bootps: [udp sum ok] BOOTP/DHCP, Request from 00:08:dc:ab:cd:ef (oui Unknown), length 264, xid 0x5, Flags [none] (0x0000)
	  Client-Ethernet-Address 00:08:dc:ab:cd:ef (oui Unknown)
	  Vendor-rfc1048 Extensions
	    Magic Cookie 0x63825363
	    DHCP-Message Option 53, length 1: Request
	    Client-ID Option 61, length 7: ether 00:08:dc:ab:cd:ef
	    Server-ID Option 54, length 4: _gateway
	    Parameter-Request Option 55, length 3:
	      Subnet-Mask, Default-Gateway, Domain-Name-Server
16:24:38.713646 IP (tos 0x0, ttl 64, id 0, offset 0, flags [none], proto UDP (17), length 576)
    _gateway.bootps > <IP_ADDRESS>.bootpc: [udp sum ok] BOOTP/DHCP, Reply, length 548, xid 0x5, Flags [none] (0x0000)
	  Client-Ethernet-Address 00:08:dc:ab:cd:ef (oui Unknown)
	  Vendor-rfc1048 Extensions
	    Magic Cookie 0x63825363
	    DHCP-Message Option 53, length 1: NACK
	    Server-ID Option 54, length 4: _gateway
	    MSG Option 56, length 15: "lease not found"

@phil-opp Thanks.
Why is there a discover and a request without an offer in between?
Also, did you change your code to assign the new address configuration yourself? This DHCP client will no longer do that for you.
While this is still not merged: do we want to represent NACKs in the API?

@astro Thanks for the help!

Why is there a discover and a request without an offer in between?

I don't know, maybe we're using the API in a wrong way.

Also, did you change your code to assign the new address configuration yourself? This DHCP client will no longer do that for you.

Ah, this might be the problem. cc @oli-obk

:umbrella: The latest upstream changes (presumably 6793792b02d7da436d08b6ab6eecf25c171eb3b5) made this pull request unmergeable. Please resolve the merge conflicts.

Ping?

As far as I understand there are outstanding issues, so I'm not merging this yet.

@whitequark It's unclear whether there's a problem or not, it sounds as if @phil-opp might be using it wrong.
Merging "some" version would allow more people to use it and test. I'd never assume a new implementation of a messy protocol like DHCP is going to be perfect from day one but if it is not landed at some point it'll never have a chance to become perfect.

OK, so at least this'll need to be rebased.

I've been experimenting with this and everything but assigning the new IP address seems to work. Even not using the DHCPv4 client I am unable to change the IP address after building an EthernetInterface. We might indeed be using the API wrong, so please don't block this on us if it works for you, I'll figure it out aftewards.

OK, so at least this'll need to be rebased.

Rebased!
Note that #276 added more dst_addr checks to process_ipv4() which were removed in this PR.

:umbrella: The latest upstream changes (presumably 5334b93158c954b61f5daf0d6d5d9d0c91588a26) made this pull request unmergeable. Please resolve the merge conflicts.

Rebased again.

Note that #276 added more dst_addr checks to process_ipv4() which were removed in this PR.

To explain this again: We cannot filter on bound IP addresses as the DHCP responses are targeted at IP addresses that are yet to be bound. Using DHCP would depend on always running in any_ip mode. This is a consequence of moving from RawSocket to UdpSocket, as was desired in code review. Removal of the filtering code also means we would depend on running with a unique mac addr on a switched ethernet link.
I suggest that we revert back to using a RawSocket for the DhcpClient, like it is being done in other implementations on other stacks. That way we can keep dst address filtering. What do you think?

I suggest that we revert back to using a RawSocket for the DhcpClient, like it is being done in other implementations on other stacks. That way we can keep dst address filtering. What do you think?

I think this is unfortunate but probably a reasonable way forward. Sorry for making you do the work twice.

Switched back to raw sockets, rebased, and squashed.
Please review my changes to src/iface/ethernet.rs. I had to prioritize raw sockets so that UDP won't respond with ICMP port-unreachable even when a packet has been handled by a raw socket.

:umbrella: The latest upstream changes (presumably e7e267f218bb30488eb5b593d6da02dfa774d659) made this pull request unmergeable. Please resolve the merge conflicts.

After #292, #293/#295, rebased once again.

I think this is basically okay, and there is no reason to have it drag on any longer even if it's imperfect in minor ways. I apologize that this took so long in the first place.
@m-labs-homu r+

:pushpin: Commit 14695c1 has been approved by whitequark

:hourglass: Testing commit 14695c14637ba2f33b091a6443a2c8fe47c75b16 with merge 657c9aa63a56af08b8e4330544ec302fc5f56dfd...

:sunny: Test successful - status-travis
Approved by: whitequark
Pushing 657c9aa63a56af08b8e4330544ec302fc5f56dfd to master...

Wow, after 13 months!

it's imperfect in minor ways

I agree. It's meant to be a basic implementation to get DHCP capabilities started.
I don't believe in code ownership, so anyone is invited to overhaul this code. Mention @astro in the issues/PRs so that I know where I can help.


Enabling AdvRocketry lets my lights and shadows disappear...

Minecraft Version:
-1.12.2
Forge Version:
-newest
Advanced Rocketry Version:
-1.2.6-49
LibVulpes Version:
-0.2.7-25-universal
Optifine + Sildur Shader (also tested on Chocapic13)

Is this still a issue? Please comment if it is and i will reopen if if you are unable? We are closing old issues to see what is still relevant and needs to be fixed.


flespi protocol remastered to receive data from flespi stream

Hi Anton,
This is flespi protocol remastered to receive data from flespi stream.
Also more parameters are converted into internal representation.
Could you please review the commit to merge it into the Traccar project.

Code refactored to use switch instead of ifs: 445b1e1

Return value of saveParam method is changed to boolean: 4d188ce

Merged, thanks.


API v2 import-scan endpoint should be able to receive empty files

Bug description
Some security tools generate empty report file when there is no security findings.
The API v2 is not able to receive it.
Steps to reproduce
Steps to reproduce the behavior:

Generate a empty report (with Nuclei for ex)
Upload the report with the API v2 /import-scan endpoint
See error

Expected behavior
Empty report accepted and Test uploaded
Deployment method (select with an X)

Docker Compose

Environment information

Operating System: Arch
DefectDojo version 2.6.2

Logs
<Response [400]>
400 Client Error: Bad Request for url: https://demo.defectdojo.org/api/v2/import-scan/
(.venv2) [damien@damien dd2]$ python3 test_empty_report.py
b'{"file":["The submitted file is empty."],"message":"{\'file\': [ErrorDetail(string=\'The submitted file is empty.\', code=\'empty\')]}"}'
<Response [400]>
400 Client Error: Bad Request for url: https://demo.defectdojo.org/api/v2/import-scan/

Sample scan files
Any empty file with scan_type = 'Nuclei Scan'
Additional context (optional)
It is a road-blocker for secureCodeBox for some scanners. See https://github.com/secureCodeBox/secureCodeBox/issues/877

Fixed in 2.7.0


Redis Prefix not prepending

Laravel Version: 7.2.2
PHP Version: 7.4.4
Database Driver & Version: Redis 4.0.9, Predis 1.1.1

Description:
We have encountered an issue that randomly made our redis prefix disappear. We have no Idea why this occurs. One day the prefix, which at that point was not of any use for us disappeared. So I just changed the channel name our frontend listens to back to the name without a prefix. Now, however, we wanted to add a second project on the same redis database, but with a different prefix. The prefix does not prepend properly although it is cached and accessable from config('database.redis.options.prefix') We have tested that. Looking at our laravel-echo-server terminal, we observed that no prefix was attached unless we added it manually to the channel name like below image shows.

The date I observed that the name was not attached was January 31, 2019. That was on v6.12.0 Please let me know if you need additional information.

Please fill out the issue template with proper versions.

I have updated the versions accordingly I believe.

Updating to Redis version to 5.x will fix the error!


Pygmentize ansi escape sequences are stripped from Windows pipes

Pygmentize wraps stdout with colorama on Windows. This ensures correct color output on the console but when output is redirected to a pipe it strips the ansi sequences by default. This prevents me from piping the output to less or other tools. On the modern windows terminal ansi escapes are handled directly by the terminal anyway so colorama is not necessary, and even on older windows tools like Cygwin's less/cat handle ansi sequences internally so being able to pipe them is still useful.
The fix I think is either setting strip=False in colorama.initialise.wrap_stream() so it won't touch non-console output, or (for the most recent colorama version) calling colorama.just_fix_windows_console instead (see https://github.com/tartley/colorama/pull/139#issuecomment-1278346238)

What about adding an option to pygments/cmdline.py that disables all of the special handling for Windows Terminal?  This is what I think was suggested as the preferred option at the end of the discussion for issue #2492.  Adding an option "--color" that causes the special handling for Windows to be disabled is an easy fix.  I didn't try to do anything fancy -- just added an option and a test that the option was not set to the checks that the script is running on Windows, using a terminal formatter, and not sending output to a named file.  I encountered a few issues in using pytest to test the changes, which work when the package is installed, but I can work those out and submit a pull request if anyone is interested?
This was mostly done to address my own irritation when using pygmentize under Windows, but I can't see and major issues with the solution.  It only affects calls to main in which the previously unused "--color" option is provided, and does nothing more than disable some special handling under Windows.  The semantics could be extended to handle similar cases with other systems, but I don't think there are any right now.

The pytest issue was that I forgot to set the formatter to "terminal" in the command line test; I fixed it, and the test now passes.  The tox tests too, though that was expected.  A total of 18 lines added and 1 slightly changed, including the unit test, which I added to tests/test_cmdline.py.

The resolution in that PR was not to disable all Windows handling, but just the escape stripping when redirecting to a file/pipe, and to have the stripping opt-in and available on all platforms.

(I believe that the ansi-to-win32 conversion behavior will be automatically disabled by colorama when it detects the windows terminal's built in ansi support anyway. But it still enables the escape stripping when isatty is false, with the current arguments.)

Thanks for the clarification.  I appreciate you getting back to me so quickly.
I skimmed the discussion, and guess I got out of it what I wanted to read :(
I've used the tool on other systems, and finding things not working under
Windows was irritating.  The easy fix was a bit too appealing, I guess.
Despite this, the idea of being able to override settings that are normally
inferred from the environment does still appeal to me.
What would you like me to do with this?  I'm not hugely invested in the fix;
it was just the product of fixing something that was annoying me.
Commenting on it has taken longer than making the changes.
On the other hand, if you would like me to put together something
more in line with what was suggested, it doesn't sound like a
bad idea.  It would be nice if there was a way of resolving problems
with piping.

So my fix in PR #2492 solved the issue just fine but I think the project owners want to be able to re-enable that behavior by flag, and also have the flag apply on Unix as well. Experimentally it seems that colorama can wrap a stream and strip escapes even on Linux so that shouldn't be a hard change, but Anteru implied they were going to make the change themself so I didn't bother following up on it.

Fair enough; your change seems to give what I originally wanted as the default behaviour.  It has been more than a year since you proposed the change, though, and the anticipated change does not seem to have made it into the main branch of the code.  I have altered my code so that it extends what I believe (on re-reading) was suggested in the comments on PR #2492.  What I understood of the officially proposed solution is that it would introduce an option "--strip-escapes" that can have values "on", "off", and "auto", which will produce behaviours of stripping escapes on all platforms, leaving escapes in place (while converting them to work in the detected environment), and applying the current heuristics to determine what should be done.   I would still like to support the option of disabling the post-processing, just in case something else goes wrong, and have done this by "raw" as a possible value for "--strip-escapes".
Currently the tests for "--strip-escapes=off" fail, while those for "--strip-escapes=raw" succeed.  This is odd, since running the script with the two options produces identical output but for the conversion of '\n' to '\r\n' in the output with "--strip-escapes=off".  I checked by running the two under Windows, with output redirected to file, used sed under wsl to translate the line endings for the "raw" file and put both files through od -a before diffing them (for readability).  No differences.  Under pytest, the "off" and "semi-raw" cases cut off at around 32808 characters.  I suspect a bug in the test, but it isn't obvious, and I don't currently have time to check.
I'm going to push the changes to my repo anyway.  If you feel like looking, you are welcome; if not, this is mostly for my own satisfaction at this point.  Changes are up to about 70 lines, including tests and making '--color' an alias for '--strip-escapes=raw'.

I would just send a PR.

Done.  PR #2802.

I've looked into the test failure that I mentioned earlier, and it looks like it has something to do with a file not being flushed in pygments.terminal.format_unencoded.  Tracing through with a debugger revealed that the underlying BytesIO object is not updated by the time the function exits, and calling flush from the debugger at that point causes the test to pass.  One is called as a file-like object is closed shortly thereafter, but that does not appear to fix things.  Maybe a different object?  I skipped over a few steps in tracing.
I suspect that I should raise this as a separate issue and add the flush in format_unencoded.  Determining exactly where the data gets lost is still something that needs to be done, but flushing the output at the end of this function makes sense.

After writing a nice test case designed to demonstrate the flushing problem independantly from the fix to this issue, I realised that unit-testing using the terminal would trigger the original bug,  This would cause the output to be stripped of escape sequences, which means that the bug does not trigger, at least with the input file that I was using.  I knew this at the start, because the issue with the test case for PR #2802 only surfaces with "--strip-escapes=off" or "--strip-escapes=semi-raw", which are the two cases where the colorama wrapper is used but the output is not stripped.
I am adding the fix to my PR until I can reproduce the bug without my fix to this issue, at which point I will open another issue.

Amend the "with the input file I was using" to "with any file in the test directory".  The difference in behaviour does not seem dependant on the number of bytes written to outfile.  I'll track down the exact source at a later date; for now, I have a fix to apply.

I wonder if it's not easiest to use colorama.just_fix_windows_console(). I overlooked that solution initially.

Yes, I also think that is the simplest solution which I mentioned in the original post, assuming that we don't care about tying to a newer version of colorama. But from the discussion in the closed PR they were reluctant to make that change without adding an option to enable stripping in a cross-platform way.
Personally I think that could be left to a separate issue, because I think the number of people relying on the current stripping feature on pipes/redirection on windows are likely very small, and it was already agreed that changing the default behavior is acceptable here.

Though looking at this again, the just_fix_windows_console approach is a bit nuanced bc it only wraps the text streams (on old windows where the wrapper is still needed) but not the underlying binary streams, and currently pygments will re-wrap the windows binary stdout stream in a new text stream and the colorama wrapper. So to more closely preserve that behavior I think calling wrap_stream manually still may be needed.
Also just_fix_windows_console appears to have a subtle bug which may not matter much here but could break some usages of stdout/stderr, which I just filed a PR to fix: https://github.com/tartley/colorama/pull/397

Ok, even simpler - since Python 3.9 is the oldest non-EOL version, and it doesn't run on Windows 7: do the nuanced behavior and the subtle bug still matter? :) AIUI, there is no wrapping at all in Windows 10+, only the console "ioctl" equivalent to switch on processing.

Windows 8?

Probably not. And Windows 8 must be a myth anyway 🤣


Using new parameter 'interval' (v. 0.2.0) causes exception

When the widget is removed from the widget tree it still tries to animate, causing the following exception:
AnimationController methods should not be used after calling dispose.
#0      _AssertionError._doThrowNew (dart:core-patch/errors_patch.dart:46:39)
#1      _AssertionError._throwNew (dart:core-patch/errors_patch.dart:36:5)
#2      AnimationController.forward (package:flutter/src/animation/animation_controller.dart:456:7)
#3      _ShimmerAnimatorState.initState.<anonymous closure>.<anonymous closure> (package:shimmer_animation/src/shimmer_animator.dart:40:60)
 #4      new Future.delayed.<anonymous closure> (dart:async/future.dart:326:39)

In brief the internally used AnimationController has already been disposed (correctly) when the internal Future (used to delay the shimmer) tries to animate forward.
To reproduce the problem just create a basic app with a button to perform a Navigator.push to push a route, with a Page that use the Shimmer with the "interval" param, and then just press "back" or programmatically add a button to perform the Navigator.pop in order to remove the Shimmer widget from the widget-tree. Wait the 'interval' delay to see the exception.

Perhaps at line 39 of shimmer_animator.dart you just need to check the mounted property of the State. Something like:
[...] // line 37
..addListener(() async {
        if (controller.isCompleted && mounted)
          Future.delayed(widget.interval, () => controller.forward(from: 0));
        setState(() {});
      });
[...]

... in theory after the widget is removed from the tree (disposed) the state is unmonted so it will never try to animate after that.

Thank you for raising this issue. I would integrate this ASAP or would you like to open a Pull request, since I see that you have already tried and tested this?

Since it's just an "if" statement it would be simpler for you integrate
this, so I don't have to clone the project just for this... you know, I
just changed that line "on the fly" on the flutter downloaded directory and
tested "on the field" (not automated test or anything), not really the
correct workflow.. just to see if it was "just that"...
Thanks.
Il lun 12 ott 2020, 16:47 Madhav Baveja<EMAIL_ADDRESS>ha
scritto:

Thank you for raising this issue. I would integrate this ASAP or would you
like to open a Pull request, since I see that you have already tried and
tested this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/maddyb99/shimmer_animation/issues/11#issuecomment-707165666,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALAIBCYGBXI7HIYG2H3Z7LSKMJH3ANCNFSM4SMPARIQ
.

I totally agree, will do it, its just that I won't be able to push it out now due to some reasons, would try to do it in a couple of days!
Thanks again for sharing this 🙌


TEP 0097: Add breakpoints, breakpoints.onFailure to debug and extend support to PipelineRun

Feature request
Add breakpoints, breakpoints.onFailure to debug spec in TaskRuns which uses existing logic from the onFailure option in the debug.breakpoint spec in the TaskRun spec. Extend this logic to support PipelineRuns as well.
This will be the start development on TEP 0097 using the debug.breakpoints spec insteam of the debug.breakpoint spec to not break existing behavior. eventually deprecate debug.breakpoint in the TaskRun Spec.
Use case
Debug PipelineRuns on Failure

/assign

/remove-lifecycle frozen
/reopen

@waveywaves Is it in any future roadmaps?

/reopen
/lifecycle frozen


Invalid host

When sending a request to Repeater, there is an exception happening. This issue is present in the jar in commit 935eb02. Similar exceptions are encountered when:

sending to Active scanner and Intruder
with and without the http:// prefix
with and without the /somepath postfix

Here's a stacktrace of the error:
java.lang.IllegalArgumentException: Invalid host: http://some.random.host:8080/somepath
	at burp.bd.sendToRepeater(Unknown Source)
	at burp.ContextMenu$3.actionPerformed(ContextMenu.java:69)
	at javax.swing.AbstractButton.fireActionPerformed(Unknown Source)
	at javax.swing.AbstractButton$Handler.actionPerformed(Unknown Source)
	at javax.swing.DefaultButtonModel.fireActionPerformed(Unknown Source)
	at javax.swing.DefaultButtonModel.setPressed(Unknown Source)
	at javax.swing.AbstractButton.doClick(Unknown Source)
	at javax.swing.plaf.basic.BasicMenuItemUI.doClick(Unknown Source)
	at javax.swing.plaf.basic.BasicMenuItemUI$Handler.mouseReleased(Unknown Source)
	at java.awt.Component.processMouseEvent(Unknown Source)
	at javax.swing.JComponent.processMouseEvent(Unknown Source)
	at java.awt.Component.processEvent(Unknown Source)
	at java.awt.Container.processEvent(Unknown Source)
	at java.awt.Component.dispatchEventImpl(Unknown Source)
	at java.awt.Container.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
	at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
	at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
	at java.awt.Container.dispatchEventImpl(Unknown Source)
	at java.awt.Window.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
	at java.awt.EventQueue.access$500(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue$4.run(Unknown Source)
	at java.awt.EventQueue$4.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)

Thank you for your feedback @juhakivekas.
I am very busy at thye moment, I will have to time to take a look at this issue only next week.
I saw your merge request, which I approuved.
Did it fix this issue?

I can't verify at the moment because the code doesn't compile on my debian with up to date default-jre from apt repositories. I'll try to get the code to compile and I'll let you know as soon as I do. At the moment I'm still getting six errors, but I'll be working on this code about an hour every morning this week.

In Burps extender the host and port for sendToRepeater are passed in separate variables. The host some.random.host:8080 is a valid host according to the swagger specification, but not as a host in the Burp extender. This is the reason for the exception, and it seems this is just a use-case that hasn't been taken into account.
Swurg already guesses the port number based on the schemes  field in the swagger file. In order to enable connecting to APIs not on the schemes default ports, the host parameter should first be checked for a port number, and if one is found, then this should be used instead of the value returned by protocolToPort(protocol).

This solution is now implemented, an update will soon released.


Need gradeint view on calendar selection

I am in a deep trouble with calendar view and I really need your help for this. Please check the attached image. The thing I need a gradient view for a calendar. By selecting one date and then another will make the select as a gradient.
I followed your code but I am unable to get this done with 100% result. If I use an image it will only work on one date and to cover the whole selection.
I would really appreciate if you can share the logic or a code behind this as this is something on high priority on my project.

This should be easy to do.
You will need to create some function that creates a gradient color to a view.
So lets say you have 16 dates selected like you have in the image above, this function will create 16 gradient color slots. The color should change slightly with each increasing index.
Once you have the gradient colors calculated, then you should be able to apply them in the willDisplayCell function.
Sounds possible?

No response for 2 days.
Closing issue.
If you still have questions on this then reopen is this, and i'll continue.


Switching value of switches

How can I switch boolean switches on and off

@pprevalon
row.value = true as AnyObject?


Version 2.2 fails to find webdriver jar

With previos version 2.1 it all worked but when i update to version 2.2 i get the following error:
/home/blacksonic/workspace/angular2-webpack-es6-starter/node_modules/gulp-protractor/node_modules/protractor/node_modules/q/q.js:155
                throw e;
                ^

Error: Could not find chromedriver at /home/blacksonic/workspace/angular2-webpack-es6-starter/node_modules/gulp-protractor/node_modules/protractor/selenium/chromedriver
    at LocalDriverProvider.addDefaultBinaryLocs_ (/home/blacksonic/workspace/angular2-webpack-es6-starter/node_modules/gulp-protractor/node_modules/protractor/lib/driverProviders/local.js:54:15)
    at LocalDriverProvider.setupEnv (/home/blacksonic/workspace/angular2-webpack-es6-starter/node_modules/gulp-protractor/node_modules/protractor/lib/driverProviders/local.js:71:8)
    at Runner.run (/home/blacksonic/workspace/angular2-webpack-es6-starter/node_modules/gulp-protractor/node_modules/protractor/lib/runner.js:278:31)
    at TaskRunner.run (/home/blacksonic/workspace/angular2-webpack-es6-starter/node_modules/gulp-protractor/node_modules/protractor/lib/taskRunner.js:123:19)
    at createNextTaskRunner (/home/blacksonic/workspace/angular2-webpack-es6-starter/node_modules/gulp-protractor/node_modules/protractor/lib/launcher.js:216:20)
    at /home/blacksonic/workspace/angular2-webpack-es6-starter/node_modules/gulp-protractor/node_modules/protractor/lib/launcher.js:239:7
    at _fulfilled (/home/blacksonic/workspace/angular2-webpack-es6-starter/node_modules/gulp-protractor/node_modules/protractor/node_modules/q/q.js:834:54)
    at self.promiseDispatch.done (/home/blacksonic/workspace/angular2-webpack-es6-starter/node_modules/gulp-protractor/node_modules/protractor/node_modules/q/q.js:863:30)
    at Promise.promise.promiseDispatch (/home/blacksonic/workspace/angular2-webpack-es6-starter/node_modules/gulp-protractor/node_modules/protractor/node_modules/q/q.js:796:13)
    at /home/blacksonic/workspace/angular2-webpack-es6-starter/node_modules/gulp-protractor/node_modules/protractor/node_modules/q/q.js:604:44

It seems that my config parameter may not be passed down although i pass in seleniumServerJar:
exports.config = {
  baseUrl: 'http://localhost:9000/',
  framework: 'jasmine',

  seleniumServerJar: findSeleniumJar(),

  useAllAngular2AppRoots: true
}

seems to be failing when the version of my protractor doesnt match the one required by gulp-protractor

Here is the repository https://github.com/blacksonic/angular2-es6-starter

Hmmm, nothing changed with webdriver in the last patch... This error is not indicating the Selenium JAR is missing, it is complaining because it can't find Chromedriver. There are a couple things I can suggest.
You can run the included webdriver update task that will download both Chromedriver and Selenium to the default locations. Then you just have to make that a dependency for your test-e2e task:
var webdriver_update = require("gulp-protractor").webdriver_update;
gulp.task('webdriver_update', webdriver_update);

Output should look like:
[14:28:06] Using gulpfile ~/src/oss/angular2-es6-starter/gulpfile.js
[14:28:06] Starting 'webdriver_update'...
Updating selenium standalone
downloading https://selenium-release.storage.googleapis.com/2.51/selenium-server-standalone-2.51.0.jar...
Updating chromedriver
downloading https://chromedriver.storage.googleapis.com/2.21/chromedriver_mac32.zip...
chromedriver_2.21.zip downloaded to ~/src/oss/angular2-es6-starter/node_modules/protractor/selenium/chromedriver_2.21.zip
selenium-server-standalone-2.51.0.jar downloaded to ~/src/oss/angular2-es6-starter/node_modules/protractor/selenium/selenium-server-standalone-2.51.0.jar
[14:28:25] Finished 'webdriver_update' after 18 s

Protractor successfully runs for me on a clone of your repo after adding that task. Can you give that a shot and let me know how it works for you?

Yes now it runs, but fails if i upgrade protractor to the latest

The difference is that with current version gulp-protractor's protractor version and my installed one from package.json is the same. If i upgrade protractor it will complain about the version missmatch and the error comes it cant find chromedriver

Yes I agree with @blacksonic  I use Protractor 3.2.2 latest and the gulp-protractor says it cannot find chrome driver in /node_modules/gulp-protractor/

Currently gulp-protractor installs Protractor as a dependency instead of a peerDependency (see https://github.com/mllrsohn/gulp-protractor/issues/101). Until that change is made, adding a different version of Protractor as an explicit dependency may have unexpected side effects when gulp-protractor searches for binaries.
I'll update the version tonight to v3.2.2, but this problem is caused by installing multiple mismatched versions and is not supported. Let me know if you have any additional information and I'll reopen the issue.