Multi language

Hi Daniel!
How to use another language in this keyboard?
Thanks!

Hi! For now, KeyboardKit only uses the standard api:s from Apple, which means that you have to do it as if you'd create a custom keyboard from scratch. If you'd like to support multiple languages, you'll have to create multiple keyboards manually.
However, I think it'd be a great idea to have standard keyboards built into the library as well, so that you'd get the correct action layout and could just map it to your own button design.

@thetay55 @danielsaidi I did implement a custom keyboard. (I had created a new project, add a custom keyboard extension and imported KeyboardKit.framework)
The keyboard is actually for Jawi (Malay - Arabic) and it had worked fine for me. All you need is to modify AlphabeticKeyboard.swift and you're good.
Thank you @danielsaidi for all your hard work to create this awesome flexible and maintainable library.
Also, I would like to add you into the license (still using MIT) but i didn't attribute to you yet because i thought you might not like that idea.

Wow, that's amazing! Thanks for sharing :)

@muhammadn Wow, that's amazing! Thanks for sharing :) Feel free to attribute me or link to this repository if you want to 👍

@danielsaidi Thanks, i will attribute you and the link to this repository on the main app's view on launch which shows the MIT license.
I was waiting for something like this to appear because other implementations aren't actively maintained. (eg. from swift 3 to swift 4 to 5 breaks a lot of things and it made it harder without a clean set of APIs)

Wonderful, I'm so happy to hear that you find it useful. Let me know if you are missing any features that you'd like to see in the library.

@muhammadn I have added a new set of "input sets" to KeyboardKit 3.0. This means that you could specify input sets for "alphabetic", "numeric" and "symbolic" for the keyboard above and use it to build multi-language keyboards A LOT easier.
Would you be interested in creating such a set for arabic? I have created one for English and will add Swedish before publishing 3.0.

@danielsaidi Thank you for your effort. Do you mean for arabic is my for current keyboard project or should i create a new project for me to try it?
My main worry is the support for iOS 11 on my current project to support older devices. But i think i will try to work on a new project based on KeyboardKit 3.0 if iOS 11 is no longer supported. 👍

@muhammadn First of all....KeyboardKit 3.0 will still support iOS 11, so you can relax :) Even though a bunch of SwiftUI features will require iOS 13, I hope that the new major will bring a bunch of improvements even if you target iOS 11 and 12.
Second, KK will now come with new input sets, which in the library are defined like this:
public extension KeyboardInputSet {

    static var english: AlphabeticKeyboardInputSet {
        AlphabeticKeyboardInputSet(inputRows: [
            ["q", "w", "e", "r", "t", "y", "u", "i", "o", "p"],
            ["a", "s", "d", "f", "g", "h", "j", "k", "l"],
            ["z", "x", "c", "v", "b", "n", "m"]
        ])
    }

    static var englishNumeric: NumericKeyboardInputSet {
        NumericKeyboardInputSet(inputRows: [
            ["1", "2", "3", "4", "5", "6", "7", "8", "9", "0"],
            ["-", "/", ":", ";", "(", ")", "$", "&", "@", "\""],
            [".", ",", "?", "!", "´"]
        ])
    }

    static var englishSymbolic: SymbolicKeyboardInputSet {
        SymbolicKeyboardInputSet(inputRows: [
            ["[", "]", "{", "}", "#", "%", "^", "*", "+", "="],
            ["_", "\\", "|", "~", "<", ">", "€", "£", "¥", "•"],
            [".", ",", "?", "!", "´"]
        ])
    }
}

This means that it will be very easy to create a new "alphabetic keyboard" for a new language...even though I still have to find a good way to also define the system keys as well, since they vary from language to language.

@danielsaidi ah ok! I will try it out 3.0-beta branch now to see how it goes for my end. 🚀

Multi-language keyboards are much easier to implement in KeyboardKitSwiftUI, so I will close this ticket for now.

Please provide me solution in swift for change the language of keyboard on the button above added in the keyboard .Please also provide me some solution

@danielsaidi I try to use vietnamese in your project but it not active.How can I set vietnamese


LookupPath, Path, and context

I'm not sure what my question here is, so I'll start with a motivating example. Maybe this is more of a feature request by the end.
I have the following function. One would expect c and p to be equal, without greater "context" (note that I'm not referring to cue.Context directly, but in the more general sense)
func foo(val cue.Value) {
	c := "c"
	v := val.LookupPath(cue.ParsePath(cp))
	p := v.Path()
	fmt.Println(p)
}

However, if one calls this function with something other than a root value, the result is not what one would expect
package main

import (
	"fmt"

	"cuelang.org/go/cue"
	"cuelang.org/go/cue/cuecontext"
)

const val = `
a: b: c: "c"
`

func main() {
	ctx := cuecontext.New()
	v := ctx.CompileString(val)
	b := v.LookupPath(cue.ParsePath("a.b"))
	foo(b)
}

This prints a.b.c.
How might one "reset the context" so that the output is always c?
Or another way, when I use a subvalue, how can I make it behave like a root value without rebuilding the value? Perhaps something like a copy-on-write without the write and rather a call to do so?

I found myself also in this situation and my workaround is also to recompile the value to be a root value. So I'm interested to know if there is another way.
Anyway I think the current behavior makes sense as you lookup some value I would expect it to keep the info from where it comes from originally.

Are there use cases where you don't already know the subpath when you need it? In the example, the function foo() has the information already, so doesn't really need to call v.Path(). It could just print the subpath. Anyway, it would always be possible to calculate the necessary subpath by using the lenght of the paths, and then slice operations.
I don't know if there is a best way to rid a value of its path (losing information), but you could FillPath it into a new struct, or perhaps unify with top:
top := ctx.BuildExpr(ast.NewIdent("_"))
val = top.Unify(val) // now at root

Here is a slightly modified and extended version of you example that explores both approaches:
package main

import (
	"fmt"

	"cuelang.org/go/cue"
	"cuelang.org/go/cue/ast"
	"cuelang.org/go/cue/cuecontext"
)

// foo expects that v is at root
func foo(v cue.Value, p cue.Path) {
	x := v.LookupPath(p)
	fmt.Println(p, x, x.Path())
}

// bar doesn't require v to be at root,
// but instead does calculations on the
// paths. Then again, it already knows
// the subpath.
func bar(v cue.Value, p cue.Path) {
	x := v.LookupPath(p)
	xps := x.Path().Selectors()
	vps := v.Path().Selectors()
	// Here it is, but then again it is the
	// same as p.
	subpath := cue.MakePath(xps[len(vps):]...)
	fmt.Println(p, x, subpath)
}

func main() {
	ctx := cuecontext.New()
	v := ctx.CompileString(`
		a: b: c: d: e: "f"
		`)
	c := v.LookupPath(cue.ParsePath("a.b.c"))
	p := cue.ParsePath("d.e")
	bar(c, p) // d.e "f" d.e
	top := ctx.BuildExpr(ast.NewIdent("_"))
	foo(c, p)        // d.e "f" a.b.c.d.e
	c = top.Unify(c) // c is now at root
	foo(c, p)        // d.e "f" d.e
}

There are cases, such as walking and looking for values with attributes.
Again, this is just a demonstration. The point is that one cannot get a "local context" for a value passed to a function without completely rebuilding or having to be aware that it could be from anywhere. I'm bringing it up not to find a workable way forward using the code base as it is today, but rather start a discussion and feature request to see if there is a more ergonomic way this could be done. Maybe something like a Clone() function could be a naive solution.
More generally, I think this could play into Structural Sharing, and how one might be able to achieve independent context without copying the entire value by working with the arcs and other internal structures that implement the lattice. Something like a copy-on-write or copy-on-request like capability. Ideally only the current node would need separation and that child nodes could remain untouched, unless they are modified through the "copied" value, which would then require a deeper copying, transparent to users of the API.
@mpvl ?


Implement ta import and ta import-tdb2

Fixes #94.
Two caveats here:

UDAs are not imported correctly, since we lack type information.  My preferred fix is

#328

Several kinds of data are not yet supported in TaskChampion.  I've added TODO notes for those.

I implemented #328 on this branch, too.
I don't think it hurts to keep both import commands -- this is largely for testing anyway, right?

Good points all, and implemented.


[BUG] Error in grader

Node number
Codercise I.14.2
Expected behavior
IMPLEMENT THE MULTIPLEXER
IF STATE OF FIRST TWO QUBITS IS 01, APPLY X TO THIRD QUBIT
    qml.MultiControlledX(control_wires=[0,1], wires=2, control_values='01')
The above code should result in the desired behaviour, that is, give the correct output.
Actual behavior

Incorrect: your circuit does not have the correct action on |01>.

Is however, the error being raised!
Additional information
No response
Source code
No response
Tracebacks
No response

[X] I have searched existing GitHub issues to make sure the issue does not already exist.

Hi @p-j-r , thanks for bringing this up! Would you be able to post your full solution code for this problem so I can investigate?

Hi @p-j-r, the first MultiControlledX has the correct effect on the |01> state, but if you check further down, qml.CRZ(np.pi, wires=[1, 2]) also affects the case where the first two qubits are in |01> since it is a control from wire 1 in state |1> to wire 2; this is why it's saying the results for the |01> case are not correct. If you have any conceptual questions about this, please feel free to ask them on the PennyLane forum and we can discuss in more detail! :slightly_smiling_face:

Alright, got it. Will work around that. Thank you!

No problem, I will close this issue then, feel free to open another if needed!


fix enable supervisor failed on Debian Jessie

Because Ubuntu uses Upstart, but Debian Jessie use Sysvinit. So I create the new init script of supervisor for Debian.

Thank you for the PR!


If there is a column named Index then just use it, instead of adding a new one

from the autotranslation tut:

not 100% sure about this

good here too

davide pointed out that a column named "Index" may not be in order, so this isn't safe.
but maybe we can check for that.
anyway, it's too complicated for a last minute change.


[ITensors] [ENHANCEMENT] Allow Matrix representations for operators in expect and correlation_matrix

Description
Allow Matrix representations for operators in expect and correlation_matrix. This can make it a bit easier to measure an operator that is not defined already without having to define the operator by overloading op.
For inspiration see the discussion here: https://itensor.discourse.group/t/measuring-correlations/51/6
Minimal demonstration of new behavior
julia> using ITensors

julia> s = siteinds("S=1/2", 4);

julia> ψ = randomMPS(s);

julia> expect(ψ, [1/2 0; 0 -1/2])
4-element Vector{Float64}:
  0.1968928836524881
 -0.4253469177197064
  0.4028392528567332
 -0.4026126134149185

julia> expect(ψ, "Sz")
4-element Vector{Float64}:
  0.1968928836524881
 -0.4253469177197064
  0.4028392528567332
 -0.4026126134149185

julia> correlation_matrix(ψ, [1/2 0; 0 -1/2], [1/2 0; 0 -1/2])
4×4 Matrix{Float64}:
  0.25       -0.0837478   0.0793162  -0.0792716
 -0.0837478   0.25       -0.171346    0.17125
  0.0793162  -0.171346    0.25       -0.162188
 -0.0792716   0.17125    -0.162188    0.25

julia> correlation_matrix(ψ, "Sz", "Sz")
4×4 Matrix{Float64}:
  0.25       -0.0837478   0.0793162  -0.0792716
 -0.0837478   0.25       -0.171346    0.17125
  0.0793162  -0.171346    0.25       -0.162188
 -0.0792716   0.17125    -0.162188    0.25

How Has This Been Tested?
Added tests of the behavior shown above.
Checklist:

[x] My code follows the style guidelines of this project. Please run using JuliaFormatter; format(".") in the base directory of the repository (~/.julia/dev/ITensors) to format your code according to our style guidelines.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have added tests that verify the behavior of the changes I made.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] Any dependent changes have been merged and published in downstream modules.

I like the idea so let's merge this. The only possible failure modes are these:

for fermions, making the operator from a matrix will lose the information about whether it has J-W string in the current system, so would lead to wrong results if someone tried to make "C" or "Cdag" from a matrix
if the sites are of mixed type or dimension, then the operator from a matrix may only be meaningful for some of the sites

I'm ok with #1 because it's just another one of the pitfalls of the current fermion system, and would be fixed by the new one. For #2 it's a little more of a possible silent failure mode if all the sites have the same dimension but a different physical meaning. But I think we're ok there because the user is still being explicit about what they are doing i.e. "sandwich this matrix between every site" so they have to know what to expect.
In summary again, I think this is good to merge.

Agreed that those are the main downsides, but I also agree with your assessment that 1. should be fixed by the new Fermion system and 2. is acceptable for the reasons you give.


occasional bug with can't open syntax.vim and syntax highlighting is lost

Not sure what's triggering this quite yet, but occasionally I lose syntax highlighting and get an error about xhtml.vim not being available.

i'm not seeing this much anymore. going to close out until i see it again


sidebar - refresh optimization last fix

i can´t get all code optimizatied (please it´s important to merge this, because of big trouble with scrolling) - no more fixes after this needed. sorry

@opnsenseuser no problem, things happen, maybe we should leave this on master for some time for further testing.

@AdSchellevis no. it really works now. Please release it in the next version. thx for your support 👍

@fichtner is it possible to go live with this update with the next patch day? regards rené

@opnsenseuser I’d like to kindly ask you to stop asking this particular question for each of your submission. Thank you.


Added support for client side

Thanks very much for working on this and contributing your changes back. It looks good to me! 😄
I will merge this and release a new version post haste.
The project you're working on https://github.com/Seluxit/hap_client to make a Ruby client for HomeKit looks very interesting! I will be following its progress 😸

Thanks.
We are playing around with creating our own homekit devices.
So it would be fun to try to control them using Ruby.


make sendmany API compatible with bitcoin-core

Describe the issue
Can you reliably reproduce the issue? Kind of yes. This is feature request, not an issue
If so, please list the steps to reproduce below:

run dogecoin daemon
create new address with dogecoin-cli getnewaddress myaccountname (you will get address that represents myaccountname account
receive payment to your address
try to use sendmany API
set 1st arg to "" to represent default account (so that dogecoin-cli sendmany "" "{\"DH9fPpKHLiP5eaAD3pXxxUZmPktGNGTFp6\":0.01,\"1353tsE8YMTA4EuV7dgUXGjNFf9KpVvKHz\":0.02}"
payment will fail since default account does not have that much money

Expected behavior
Tell us what should happen
Payment should be successful (like in bitcoin).
The reason why it works in bitcoin, is because there we have label instead of accounts and accounts are deprecated and removed since bitcoin-core V0.17.
What I'm suggesting is make sendmany API compatible with bitcoins sendmany API
This will allow to send DOGEs not only from one address, but from many
ALSO it will allow us to just send coins without hassling with addresses and accounts
Actual behaviour
Tell us what happens instead
It's listed in steps to repoduce
What version of Dogecoin Core are you using?
1.14.3

That will be in the next major release, for sure. In the meantime, you can use move to "internally transfer" all coins to "default account" from all other accounts.

@shibe2 thanks, I see that move is deprecated. Can you please DO NOT delete it until the version with new sendmany API?
So that there is no BC in minor release.
I mean, delete the move function only after new sendmany is available. So that there is a version that supports both functions at same time.

API will stay the same in minor releases. The next major release will not have accounts, and sendmany will work right away.

Thanks so much guys.   When you're ready, I can help w video
tutorials/graphics and other media. Let me know if there is someone you
want me to contact, so you can stay focused on code.
On Wed, Apr 7, 2021 at 10:03 AM shibe2 @.***> wrote:

API will stay the same in minor releases. The next major release will not
have accounts, and sendmany will work right away.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dogecoin/dogecoin/issues/1814#issuecomment-814940616,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS7OED6YIKNLFNQO3QD6WFLTHRQZRANCNFSM42NRZRLQ
.

--
Cat Mayville
President, Spot Digital Media
404-357-4503

sendmany "" {"address":amount} ( minconf "comment" ["address",...] replaceable conf_target "estimate_mode" fee_rate verbose )
{
"address": amount,    (numeric or string, required) The bitcoin address is the key, the numeric amount (can be string) in BTC is the value
}

@patricklodder @rnicoll Where are we on this?


Add Activities tab populated with active ServiceRequests

Pull ServiceRequests
Sort by descending order by recency

Data to show with ServiceRequests? Check if there's an option to go to Medline Plus.
Have limitations for procedure codes to make patient-friendly language.

@swmuir to look at the queries for interventions tab in provider app. If they use Category, it's not going to work.

Activities tab is completed and ready for QA.  Populated with all ServiceRequest with 'active' status, sorted in descending order by the 'start' date of occurrenceTiming (this is what appears to be populated in sample data from epic.fhir.com.
Use any of the patients in CarePlanning sandbox for testing.
Cannot find any ServiceRequest resources to test in Epic sandbox.

This looks good. Did you look into the option to link to Medline Plus for the procedure codes?


Remove version: taers232c.GAMADV-XTD3 version 6.71.17

This version is older than what has been set in max-versions-to-keep by the publisher.
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/taers232c/GAMADV-XTD3/releases/download/v6.71.17/gamadv-xtd3-6.71.17-windows-x86_64.msi
OK
(Automated message - build 910)


Dialog Grid elements change CSS

Hi,
I have data coming into a form from a database and this displays correctly and I can format the gird and colour a row if it does not match a certain criteria.
My problem comes, that when I click the add button and the Dialog box opens I cannot do the same.  I have the data in my grid, which the add button is pulling the data in for me, but I cannot for the life of me see where to be able to manipulate the rows or columns in the grid that is displayed in the dialog box.
Any help would be great thanks.
//Add button code from my DskLdnMorningCheckGrid.ts page
protected addButtonClick() {
        this.editItem(<Serene.Deskside.IntachangeStarterRow>{
            IntachangeList: Serene.Deskside.IntachangeStarterRow.getLookup().items.map(x => {
                return <Serene.Deskside.IntachangeStarterRow>{
                    FullName: x.FullName,
                    Class: x.Class,
                    Company: x.Company,
                    StartDate: x.StartDate,
                    UserAccountSetup: x.UserAccountSetup,
                    DeskSetup: x.DeskSetup,
                    PcSetup: x.PcSetup
                }
            })
        });

//my dialog code
@Serenity.Decorators.registerClass()
export class DskLdnMorningCheckDialog extends Serenity.EntityDialog<DskLdnMorningCheckRow, any> {

    protected getFormKey() { return DskLdnMorningCheckForm.formKey; }

    //protected form = new DskLdnMorningCheckForm(this.idPrefix);
    protected form: DskLdnMorningCheckForm;

    constructor() {
        super();
        this.form = new DskLdnMorningCheckForm(this.idPrefix);
    }

//I have tried to get details from
protected getPropertyItems()
{
//i can get elements but I cannot set any grid properties
}

if I understand you correctly, or is it about masterdetail?
loadEntity(entity: DogovoraRow) {
super.loadEntity(entity);
        if (this.entity.DogovorStateId == 2) this.form.DogovorStateId.element.addClass("red");

    }

I also guess it's MD. So then manipulate DetailList collection (or whatever is the property name where is the collection of details)

Sorry for the late reply back.  Its not MD, as the detail is not changing, the information is there for informative purposes.  So imagine a daily check, we tick off some items, but then we pull data into a table about items not done.  This is oneway readonly list.
I tried the loadEntity method, but when I try to get information from the entity list, so this.entity.IntachangeList[i].FullName for example (in a for loop) it just hangs and gets no information back.
As I say, this works on a grid.ts but I cannot seem to pull that into the dialog for some reason.
I hope this helps explain more what I am trying to do.

@ngaggiano  sorry, man, not clearer at all. any screenshots?
are you trying to read data from in memory grid or writing data to it, or something completely different?
I;d love to help you but I have no idea anymore what's going on.

@ngaggiano use foreach
protected updateInterface() {
            super.updateInterface();
            if (this.entity.IntachangeList) this.entity.IntachangeList.forEach(x => {
                if (x.StateId == 2) {
                    ....
                }
            })
        }

Thanks, I will get some screen shots

@ngaggiano ,
where are the screenshots? :-)
John

Sorry, been away.  The main editor screen is displayed in the first one, and when I click the "New Core Morning Check" button brings data from another table and displays it (IntachangeUsers.png, but I would like where it says 'User Setup', 'Desk Setup' etc to be highlighted in Red.
I pull the main screen from one database table A and the Interchange users from table B

So you just want to change color of columns?
I think you can CSS it...
there are also some samples of coloring the rows, so u can try it for columns as well.

I do, but its based on criteria, if DeskSetup == "No" then Display Red Row, and thats the problem I having on the form.

The problem is, u just want single cell colored, rather than row?
If that is the case here is sample:
https://serenity.is/demo/BasicSamples/ConditionalFormatting

Sorry, I would like the entire row highlighted,  but that highlight is based on 3 cells criteria.  If the 3 cells are No then display Red.
Thanks
Sent from Email+ secured by MobileIron
-------- Original Message --------
From: Juraj Benadik <notifications<EMAIL_ADDRESS>Date: Tue, 2 Apr 2019, 03:01
To: volkanceylan/Serenity <Serenity<EMAIL_ADDRESS>CC: "Gaggiano, Nathaneal" <Nathaneal.Gaggiano@coremont.commailto:Nathaneal.Gaggiano@coremont.com>,Mention <mention<EMAIL_ADDRESS>Subject: Re: [volkanceylan/Serenity] Dialog Grid elements change CSS (#4322)
The problem is, u just want single cell colored, rather than row?
If that is the case here is sample:
https://serenity.is/demo/BasicSamples/ConditionalFormatting
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/volkanceylan/Serenity/issues/4322#issuecomment-478813977, or mute the threadhttps://github.com/notifications/unsubscribe-auth/At-SNJVx8Vp8KDTSWy2UrnojWIh3WZm0ks5vcroNgaJpZM4boOh1.
This email, the information contained therein and any attached materials (collectively the "Email") are intended only for the addressee(s) and may contain confidential, proprietary, copyrighted and/or privileged material. If you have received this Email in error please delete it and notify the sender immediately. This Email remains the property of Coremont, which reserves the right to require its return (together with any copies or extracts thereof) at any time upon request. Any unauthorised review, retransmission, dissemination, forwarding, printing, copying or other use of this Email is prohibited. Coremont may be legally required to review and retain outgoing and incoming email and produce it to regulatory authorities and others with legal rights to the information. Internet communications cannot be guaranteed to be secure or error free as information could be intercepted, changed corrupted, lost, arrive late or contain viruses.
Any information in this Email should be regarded strictly as guidance and for informational purposes only. Coremont does not provide investment, legal, accounting or tax advice. Coremont and the sender make no warranties regarding the accuracy or completeness of the information in this Email and it should not be relied upon for any purpose and is subject to change without notice. Information contained in this Email is unaudited and Coremont accepts no liability for any errors or omissions in this Email. Coremont and its representatives, members, officers and employees accept no responsibility for any losses suffered as a result of reliance on the information in this Email or the reliability, accuracy, or completeness thereof. Coremont assumes no obligation to update any information contained in this Email. Any comments or statements made in this Email do not necessarily reflect the views of Coremont.
You agree and acknowledge that you will handle any personal data which Coremont provides to you in accordance with the requirements of applicable data protection legislation, including EU data protection legislation.
In this Email, "Coremont" means Coremont LLP and its affiliates. Coremont LLP is a limited liability partnership authorised and regulated by the Financial Conduct Authority of the United Kingdom and registered in England & Wales (reg. no. OC420985).

I don't understand. In formatting callback u have whole row at your disposal. Just use logical AND and add as many conditions as u like

I have done that, hence why the dialog is showing the 3 rows with all No cells.  The bit I cannot do is to format these rows to be highlighted in red in the dialog box.

Perhaps in the form.cs , or on the xyzrow a custom formatter ?

omg, are you talking about in-memory grid of master/detail ?
good luck with that. It should be possible though, I guess it's still slickGrid.
there were some attempts (AFAIKsuccessful) to do sorting and filtering on it, so I guess it may not be SG...
anyway, I would check those and maybe u can get some inspiration from there?

I believe so yes.
I came to that conclusion as well when trying to do it.  Hence I wondered if anyone has managed it.
Cool will take a look at slickGrid and see if anything.
Thanks everyone.


Make the Navbar Responsive

@swapnilsparsh I would like to work on this issue. Can you please assign me this issue?

Ok @sakshikhachane

@swapnilsparsh I have some suggestions.
Can we make the whole contents of the navbar except home shift to right? Also if we give the same color to it then it will look better. Should I try designing using my idea or should I modify your design only?

@sakshikhachane You can design according to your comfort.

@swapnilsparsh Okay. Can you assign me this issue? So that I can proceed to do work.

Hey @swapnilsparsh can I also work on this issue?

Hey @swapnilsparsh can I also work on this issue?

Ok

Hey, can I please take the same issue?

Hey, can I please take the same issue?

Hey, can I please take the same issue?

Thank you for showing you interest in the project.
@Sloth-Panda  you have to wait for 3-5 days as 2 people are already assigned on this issue.
Kindly give some time to the assignees to resolve the issue then you can work on this issue till then look after some other issues.
We appreciate your patience

Hey, can I please take the same issue?

Thank you for showing you interest in the project.
@Sloth-Panda  you have to wait for 3-5 days as 2 people are already assigned on this issue.
Kindly give some time to the assignees to resolve the issue then you can work on this issue till then look after some other issues.
We appreciate your patience

@swapnilsparsh where should I show navbar that I prepared?

@swapnilsparsh where should I show navbar that I prepared?

@swapnilsparsh where should I show navbar that I prepared?

Share the Screenshot here

@swapnilsparsh where should I show navbar that I prepared?

Share the Screenshot here

It's  has animation effect

It's  has animation effect

(https://drive.google.com/file/d/1GEL_38Wm2BUa2CAXCDRW85B1NFfS7H5p/view?usp=drivesdk)
Please see its screenshot here.

(https://drive.google.com/file/d/1GEL_38Wm2BUa2CAXCDRW85B1NFfS7H5p/view?usp=drivesdk)
Please see its screenshot here.

(https://drive.google.com/file/d/1GEL_38Wm2BUa2CAXCDRW85B1NFfS7H5p/view?usp=drivesdk)
Please see its screenshot here.

Attach the photo here only as I am unable to see on your drive as it is asking for permission.

(https://drive.google.com/file/d/1GEL_38Wm2BUa2CAXCDRW85B1NFfS7H5p/view?usp=drivesdk)
Please see its screenshot here.

Attach the photo here only as I am unable to see on your drive as it is asking for permission.

I am unable attach  photo here. Please once that Link now you can able to see

I am unable attach  photo here. Please once that Link now you can able to see

I am unable attach photo here. Please once that Link now you can able to see
I had seen your navbar its quite good but I want you to make the navbar as shown in the photo

I am unable attach photo here. Please once that Link now you can able to see
I had seen your navbar its quite good but I want you to make the navbar as shown in the photo

I am unable attach photo here. Please once that Link now you can able to see

I had seen your navbar its quite good but I want you to make the navbar as shown in the photo

I am unable attach photo here. Please once that Link now you can able to see

I had seen your navbar its quite good but I want you to make the navbar as shown in the photo

Sure

Sure

https://drive.google.com/file/d/1GXX5oV35TqOA7vRzbpLxBD0RtEoaqpRy/view?usp=drivesdk
@swapnilsparsh  i have done with this please check out once

https://drive.google.com/file/d/1GXX5oV35TqOA7vRzbpLxBD0RtEoaqpRy/view?usp=drivesdk
@swapnilsparsh  i have done with this please check out once

@Swapnil-Gaur Make a PR so that I can review your code.

@Swapnil-Gaur Make a PR so that I can review your code.

@swapnilsparsh Is this better? Should I make a PR? Sorry for the delay.

@swapnilsparsh Is this better? Should I make a PR? Sorry for the delay.

@swapnilsparsh Is this better? Should I make a PR? Sorry for the delay.

Yes it's looking great. 🤩

@swapnilsparsh Is this better? Should I make a PR? Sorry for the delay.

Yes it's looking great. 🤩

@sakshikhachane Navbar is looking great 🤩

@sakshikhachane Navbar is looking great 🤩

@swapnilsparsh please review my PR.

@swapnilsparsh please review my PR.

@swapnilsparsh I have made a PR. Have a look at it :)

@swapnilsparsh I have made a PR. Have a look at it :)

@swapnilsparsh please review my PR.

You didn't made any PR.

@swapnilsparsh please review my PR.

You didn't made any PR.

Issue resolved by @sakshikhachane

Issue resolved by @sakshikhachane


0.20.0

Make it work with current Nim 0.20.0.
Clean out code.
Tiny Improvements without API change.

:slightly_smiling_face:

Great work so far @juancarlospaco

@juancarlospaco are we soon ready to update to Nim 0.20.0?

Few days I hope, worse case scenario before next month.
Sorry I been sending improvements to Nim itself.
:slightly_smiling_face:

Few days I hope, worse case scenario before next month.
Sorry I been sending improvements to Nim itself.
🙂

No problem! I'm happy that you keep working on the PR 😉 Please let me know, if there's something, where you need my help! Otherwise I'll wait until review time due to limited time.

@ThomasTJdev Let me know any comments. :slightly_smiling_face:

Where is the file nimwcpkg/resources/web/google_recaptcha.nim ?

Where is the file nimwcpkg/resources/web/google_recaptcha.nim ?

Ahh it's the nim-html-tools. I'm really in doubt about that package. It removes some fundamental functions from NimWC, such as recaptcha, which the plugins also could use and that causes a break. The nim-html-tools package is furthermore specified for NimWC (such as statusIntToText), but I don't see the benefit of splitting NimWC across multiple packages, when we are talking about less than 100 lines of code, which is required by NimWC core - it's not optional code.
I will re-insert the following code into NimWC since it is NimWC-specific + reCaptcha:

checkboxToInt
checkboxToChecked
statusIntToText
statusIntToCheckbox

Furthermore, I think we shall roll back on inputEmailHtml, inputNumberHtml, etc. and keep them in clean HTML inside NimWC.

Hi @juancarlospaco
I'm done testing. I have spinned it up on a production server and everything works flawless.
I don't have any further comments - ready for merging.

Error on clean install with no plugins and accessing "Installed plugins"

Error on clean install with no plugins and accessing "Installed plugins"

Fixed c024cdf

Merging... :slightly_smiling_face: :+1:


LogReader unable to read some rlog.bz2 files.

Describe the bug
rlog.zip
#%%
import os
import pandas as pd
# from tools.lib.logreader import LogReader
from tools.lib.robust_logreader import RobustLogReader
from datetime import datetime
import glob
import pytz
from pathlib import Path
import tools.lib.logreader
files=[]

files.append('rlog.bz2')

for file in files:
    print(file)
    lr=RobustLogReader(file)
    logs=list(lr)
    print([set(l.which() for l in logs)])

Unable to read attached rlog file using the script above. I was able to read other rlog files, and qlog files from this trip using the same script, but reading rlog files from this trip gives the following error. Tried both LogReader and RobustLogReader. PlotJuggler opens the same files with no problem.
Traceback (most recent call last):
  File "capnp/lib/capnp.pyx", line 1165, in capnp.lib.capnp._DynamicStructReader._which
RuntimeError: Member was null.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/ubuntu/PycharmProjects/IMU/micgihan_converter.py", line 32, in <module>
    print([set(l.which() for l in logs)])
  File "/home/ubuntu/PycharmProjects/IMU/micgihan_converter.py", line 32, in <genexpr>
    print([set(l.which() for l in logs)])
  File "capnp/lib/capnp.pyx", line 1182, in capnp.lib.capnp._DynamicStructReader.which.__get__
  File "capnp/lib/capnp.pyx", line 1168, in capnp.lib.capnp._DynamicStructReader._which
capnp.lib.capnp.KjException: Attempted to call which on a non-union type

OS Version
Ubuntu 20.04
openpilot version or commit
No response
Additional info
No response

Is your cereal repo (specifically log.capnp), on the same version as was used to generate the logs?

That's true, it was my mistake. The problem was using the older cereal repo in a virtual environment.
Problem solved with using the most recent cereal version.
Thank you @pd0wm !


ross-config is broken

When installed, I get empty strings for every variable (including CC and friends). This is breaking a lot of things - see the recent codes-ross-users emails.

are you setting CC before configuring cmake? CMake does use cache files, so you have to make sure you CC before configuring with cmake the first time.
Otherwise, I'm not sure what the issue is. Could you clarify the steps the user is taking? We don't usually "install" ROSS.

Yes. Here are the steps to reproduce, starting in the ROSS directory:
mkdir build
cd build
CC=mpicc CXX=mpicxx cmake -DCMAKE_INSTALL_PREFIX=/path/to/install <other options> ../
make && make install
If you check out /path/to/install/bin/ross-config (also at build/ross-config), you'll see that "ROSS_CC" and related variables are empty.
Things were ok as of commit c04babe (pre-SR), so I imagine something happened during the SR merge.

ROSS@4e2269020004db6d508867f369306fd51a18925f

Thanks for the quick resolution!


WIP Create TableOfContent component

This PR adds TableOfContent component to manage anchors on a page.

[x] automatic toggle of active classes
[x] automatic scroll to
[x] automatic generation of the links
[x] add doc

This PR will close #50.


seeding DynamoDB with data

What's the best way to seed DB with data from a json file?

Another lambda/worker function?
How to best make sure the function is run one-off & only if DB is not already seeded?

Great question... one-off admin tasks is not obvious with Lambda in general. Particularly if you are used to interactive admin like “heroku run bash”.
The default strategy is to do everything on you computer with AWS keys and the AWS CLI.
But I suspect there is a more “app” centric pattern to develop here

Ah, I see. Wish there was a better way. I'd would be great to package everything together in a single cloudformation template to get things up and running.
Would a bash script that runs conditionally in the Makefile make sense?

Could you use something like data pipeline to dump something in s3 > dynamo?

@mwarkentin that's a good idea. I haven't used that before. I see there is support for it in Cloudformation templates https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-datapipeline-pipeline.html


Extension issue

Issue Type: Bug
Extension Name: python
Extension Version: 2021.1.502429796
OS Version: Windows_NT x64 10.0.18363
VSCode version: 1.52.1

:warning: We have written the needed data into your clipboard. Please paste! :warning:

Thanks for the bug report, but unfortunately we need some more information from you in order to triage this issue appropriately. Please created a new bug by providing a filled-in copy of our issue template.


Enhancement

when will download queuing added to the Bot. Eagerly waiting for that feature

It'll need a complete restructuring of the code. I guess I'll eventually get around to it.

Download queuing and concurrent downloads are now supported since cfc67461.


Fix for default_action

tc template code change for default miss action
NoAction case generation in c code for tc backend
default case deletion in c code for tc backend
p4tc testcases output updated
https://github.com/p4tc-dev/p4tc-e2e/issues/40

Please check comments. Is frontend emitting error when parameter with direction is used as default action? If not, we should handle it in backend and add some tests for the same.

default_action  with parameter is allowed by the backend and no template code change is needed for now. Once design is confirmed for those, we need to generate code accordingly

Second comment @Sosutha (tagging: @usha1830 @komaljai @vbnogueira @tammela)
For the earlier cited annotated example:
action a(bit<32> x) { /* body omitted */ }
action b(bit<32> y) { /* body omitted */ }
table t {
   actions = {
      a;
      b;
   }
   default_action = b(123) @tc_may_override;
}

At the template we would like to add a flag as such (disclaimer, we may change the specification a little, but consider that there is small syntax change here):
$TC p4template update table/... default_miss_action action b param y flags runtime

This tells us something along the lines "the default action value is determined by the datapath runtime and is NOT retrieved from the P4TC domain". IOW, the eBPF code determines the values.
Another place where this template flag would be required is in cases where the action attribute value is computed by the datapath at runtime, example:
action a(bit<32> x) { /* body omitted */ }
action b(bit<32> y) { /* body omitted */ }
table t {
   actions = {
      a;
      b;
   }
   default_action = b(z);
}
...
if (hdr.x = 5)
   z = 4;
else
  z = 3;

t.apply();

We are also considering perhaps instead of making these action attributes global within the eBPF code to rather put them in our existing filter structure you are looking at on the other issue:
struct p4tc_filter_fields {
        __u32 pipeid;
        __u32 handle;
        __u32 classid;
        __u32 chain;
        __u32 blockid;
        __be16 proto;
        __u16 prio;
};

As an example, this structure could be dynamically generated to include these variables we determine to be global, example for the earlier use case of:
action a(bit<32> x) { /* body omitted */ }
action b(bit<32> y) { /* body omitted */ }
table t {
   actions = {
      a;
      b;
   }
   default_action = b(123) @tc_may_override;
}

The compiler would generate:
struct p4tc_filter_fields {
        __u32 pipeid;
        __u32 handle;
        __u32 classid;
        __u32 chain;
        __u32 blockid;
        __be16 proto;
        __u16 prio;
        __u32 y;
};

This would simplify our control plane code.
The important part here is we want to discuss this with you - and if it can be part of the same fix, that would be great.
If you think its too much work then we can make it a different PR. Sorry for not being very clear earlier on.

@jhsmt We can have these changes as separate PR.
Few points and queries on the examples given.
bit<32> y = 123;
action a(bit<32> x) { /* body omitted */ }
action b(bit<32> y) { /* body omitted */ }
table t {
   actions = {
      a;
      b;
   }
   default_action = b(y);
}

action a(bit<32> x) { /* body omitted */ }
action b(bit<32> y) { /* body omitted */ }
table t {
   actions = {
      a;
      b;
   }
   default_action = b(z);
}
...
if (hdr.x = 5)
   z = 4;
else
  z = 3;

t.apply();

These examples are not allowed by P4 syntax. default_action should have compile time constant, variables are not allowed.
Compiler frontend will not allow variables, hence these cases can be ignored.
Now there are two queries.
action a(bit<32> x) { /* body omitted */ }
action b(bit<32> y) { /* body omitted */ }
table t {
   actions = {
      a;
      b;
   }
   default_action = b(123);
}

If only the constant is specified in the P4 Program without the annotation, should we emit the value in the template as implemented now?
$TC p4template update table/... default_miss_action action b param y 123

Now the second case, where annotation is specified
action a(bit<32> x) { /* body omitted */ }
action b(bit<32> y) { /* body omitted */ }
table t {
   actions = {
      a;
      b;
   }
   default_action = b(123); @tc_may_overrride
}

Should the template look like this? It has the constant from the P4 and also the flag to mention that annotation is given in P4 that this value might change in runtime. Ignoring the constant fully and having only the flag does not make since value is given in P4. Please confirm
$TC p4template update table/... default_miss_action action b param y 123 flags runtime

For this case with annotation, as you mentioned, compiler can generate the placeholder variable inside the p4tc_filter_fields
structure.

@jhsmt We can have these changes as separate PR.

NP.

Few points and queries on the examples given.
bit<32> y = 123;
action a(bit<32> x) { /* body omitted */ }
action b(bit<32> y) { /* body omitted */ }
table t {
   actions = {
      a;
      b;
   }
   default_action = b(y);
}

action a(bit<32> x) { /* body omitted */ }
action b(bit<32> y) { /* body omitted */ }
table t {
   actions = {
      a;
      b;
   }
   default_action = b(z);
}
...
if (hdr.x = 5)
   z = 4;
else
  z = 3;

t.apply();

These examples are not allowed by P4 syntax. default_action should have compile time constant, variables are not allowed. Compiler frontend will not allow variables, hence these cases can be ignored.

Thanks for catching these before we code away. The first above though looks like:
https://github.com/Sosutha/p4c/pull/1/files#diff-49f9c1d028a162b115a9e6be50f34fbda8d4c15c311be8d359b10e5fcbf04722R91
Did i miss something?
Q: Can the default action value "4" in:
https://github.com/Sosutha/p4c/pull/1/files#diff-49f9c1d028a162b115a9e6be50f34fbda8d4c15c311be8d359b10e5fcbf04722R91
be overriden later by the control plane?

Now there are three queries.
action a(bit<32> x) { /* body omitted */ }
action b(bit<32> y) { /* body omitted */ }
table t {
   actions = {
      a;
      b;
   }
   default_action = b(123);
}

If only the constant is specified in the P4 Program without the annotation, should we emit the value in the template as implemented now?

$TC p4template update table/... default_miss_action action b param y 123

Yes.

Now the second case, where annotation is specified

action a(bit<32> x) { /* body omitted */ }
action b(bit<32> y) { /* body omitted */ }
table t {
   actions = {
      a;
      b;
   }
   default_action = b(123); @tc_may_overrride
}

Should the template look like this? It has the constant from the P4 and also the flag to mention that annotation is given in P4 that this value might change in runtime. Ignoring the constant fully and having only the flag does not make since value is given in P4. Please confirm
$TC p4template update table/... default_miss_action action b param y 123 flags runtime

We dont need to specify "123"

For this case with annotation, as you mentioned, compiler can generate the placeholder variable inside the p4tc_filter_fields structure.
struct p4tc_filter_fields {
        __u32 pipeid;
        __u32 handle;
        __u32 classid;
        __u32 chain;
        __u32 blockid;
        __be16 proto;
        __u16 prio;
        __u32 y;
};

Excellent.

Third query is a different scenario.
Consider there are two different tables with two different default actions a and b, both with same param name as y but different types. In that case, how do we generate placeholder variable name in the structure?  To make it clear, Can we generate action_param name as variable name? In this case (a_y, b_y), Will this be clear?

Yes, makes sense.

Should we use the name in template as well, so ebpf code can communicate with TC code?

It is not needed for the template side.

The first above though looks like: https://github.com/Sosutha/p4c/pull/1/files#diff-49f9c1d028a162b115a9e6be50f34fbda8d4c15c311be8d359b10e5fcbf04722R91
Did i miss something?

Here variable is used in the action list. P4 allows variable to be used in action list and default_action when the parameter od the action is with direction. In this case, z is a inout parameter. In the examples you mentioned, we are only considering directionless parameter.


Fix: Envoy breaking ZDM during lifecycle operations

Summary
Once Envoy is enabled for ensuring instance identity validation from Go Router to the containers, we notice 502 HTTP Errors start to appear when containers are stopped (restart, evacuation, scale down)
The root cause seems to be that Envoy is accepting connections, and once accepted Gorouter is not able to retry it on another instance.
What seems to happen is following:

BBS sends request to stop one node
the chosen REP sends SIGTERM to the container
in the same time TPS, routeemitter and gorouter coordinate to remove the route
when the container receives the sigterm, it has the webserver running from the app and the envoy proxy
when the web-server gets the SIGTERM it stops accepting new requests and waits some time (for tomcat it was 30 sec by default to stop inflight requests (but gets only 10s for this)
when envoy proxy gets SIGTERM it does not stop accepting new requests, because it does not seem to have a consistent graceful shutdown concept, where ingress listeners are removed and egress ones are kept (here and here)

So we have a race condition, where gorouter has not yet removed the route, the web-server is failing new connections and envoy is accepting them. And what happens is that gorouter forwards a request to the container, envoy accepts the connection and is not able to forward it. But gorouter then can not retry the openend connection so customer gets 502
If i haven't missed some point, this is rather a design flaw in CF that is caused by the introduction of Envoy
In reality - well written HTTP applications should be aware that HTTP communication is best-effort, and ensure to do retries, idempotent calls or transactions. But i fear most customers assume HTTP is exactly-once QoS and write their apps like this
The same issue can happen during any stop of an instance (restart, evacuation) but this happens rarely maybe
Steps to Reproduce

Deploy an application.
Send a POST request to the application endpoint with some script.
Scale down / restart the application.
Few 502s will be returned, which should not happen

Diego repo
Environment Details
CF-Deployment v26.4.0

This behaviour was introduced with this story: https://www.pivotaltracker.com/n/projects/1003146/stories/156440269

After checking again the links provided by Max, i believe my initial assumption for this weird race condition between gorouter, envoy and the app is most likely wrong. I didn't realize that if the app fails for some reason - OOM, Graceful Shutdown timeout, it would respond with 502, due to working behind a proxy.
So what we see as issues during scaledown, restart and evacuation is most likely just prematurely cancelled requests in-flight. What we see under load, is application OOM-ed during processing a request, at least in the ticket discussed above


Don't know what this is, Just an FYI I guess

This happened when I restarted my server. I don't think it affects the server in any way, but I thought I'd let you guys know.
17.07 10:21:45 [Server] WARN java.io.IOException: Server returned HTTP response code: 500 for URL: http://neo.mcmonkey.org/plugins/public_statslugin=Denizen&version=1.0.2-SNAPSHOT+(build+1638)
17.07 10:21:45 [Server] WARN at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1876)
17.07 10:21:45 [Server] WARN at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLCon

Thanks for the report, it's a minor problem with the statistics server crashing for little apparent reason.
I've set it to autorestart daily so that should be less of a problem now!


Use of StringIO make module incompatible with Python 3

Hi @vgm64 ,
The introduction of StringIO makes the module not compatible with Python 3.
Raises: ModuleNotFoundError: No module named 'StringIO', see more here.
I haven't encountered any other incompatibilities with python 3 apart from this one, so I wonder whether the feature implemented in this recent commit is worth the generating an incompatibility.
Perhaps there is an alternative way of implementing it?

It isn't worth the incompatibility, you're right. I'll revert it and look
for solutions later.
On Mon, Mar 19, 2018, 3:44 AM paololucchino<EMAIL_ADDRESS>wrote:

Hi @vgm64 https://github.com/vgm64 ,
The introduction of StringIO
https://github.com/vgm64/gmplot/blob/master/gmplot/gmplot.py#L6 also
make the module not compatible with Python 3.
Raises: ModuleNotFoundError: No module named 'StringIO', see more here
https://stackoverflow.com/questions/11914472/stringio-in-python3.
I haven't encountered any other incompatibilities with python 3 apart from
this one, so I wonder whether the feature implemented in this recent
commit
https://github.com/vgm64/gmplot/commit/ad6df42e96f9c3accefe691212af06376706f96e
is worth the generating an incompatibility.
Perhaps there is an alternative way of implementing it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vgm64/gmplot/issues/66, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABml08HBPZvSLl6Ozy4_M1tgmfue7NIDks5tf4wHgaJpZM4Sv7wu
.

Thanks @vgm64 .
Issue addressed in this commit. Closing issue.


consider defining and throwing custom error classes

JavaScript tradition seems to be throw Error, but perhaps JavaScript/TypeScript provides some capabilities for defining custom error classes.  Let's evaluate whether we should throw custom error classes for various error conditions in the API.

What value does it add?

A few thoughts:

a well-named error class provides a little more information to the receiver
in cases where a method's implementation might have multiple error conditions, throwing a different error class for different types of errors allows the caller to handle the errors differently (by defining a catch block per error)
per the reviewer's comment in #291, catching Error doesn't allow the test code to assert whether the method is throwing for the expected reason, so we're not fully verifying the contract of the methods

And while it's possible to achieve similar results by inspecting an error's message, that tends to be a brittle approach (e.g., if we change an error message's spelling/punctuation/whatever, calling code may break).

In typescript theres little difference between a checking a error's "class" and checking an errors string message. instance of is just checking a string in an object's prototype vs checking the string msg.
Not sure i agree with the first point.

per @almann in #307:

We really should consider having some first-class error type like IonError and have it prototype extend Error. Since we're targeting ES6, this is pretty straightforward, e.g.:

class IonError extends Error {
  constructor(...args) {
    super(...args);
  }
}

These errors should be explicit, and we should support forms that allow for properties like lineNumber and columnNumber or the like.


adr: expose jaeger ui

adr: expose jaeger ui in wc

Any progress on this ?

Hello @lucianvlad :D

@cristiklein is the wording correct now?
@raviranjanelastisys since you have more knowledge on Oath2-Proxy, can you please draw the diagrams, preferably using the diagram as code ?

wledge on Oath2-Proxy, can you please draw the diagrams, preferably usin

Sure @lucianvlad i would love to do that :) . thanks

@lucianvlad @cristiklein  As discussed , I have added the Architecture Diagram for all the considered options.

[option 1] - Expose  Jaeger UI via ingress in wc cluster and use Oauth2-proxy for request authentication.

[option 2] - Do not expose Jaeger UI

[option 3] - Expose Jaeger UI, but completely behind oauth2-proxy. Use config domain, groups, IP allowlisting and request logging for protecting it.

[option 4] - Expose Jaeger UI and audit access via request logging in Oath2Proxy.

Didn't we say we would use Python Diagrams? My experience with Graphviz is that it's a PITA when you care about how to lay out the elements. And in arch diagrams, I found it necessary to carefully lay out elements for the sake of being able to mentally follow the diagram.
Sorry if this wasn't clear.

Sure, I agree python code would be nicer :) , seems like i got confused as https://diagrams.mingrammer.com also uses the Graphviz to render the diagram https://diagrams.mingrammer.com/docs/getting-started/installation.
I will do the required changes :)

Thanks @lucianvlad @cristiklein for your guidance, I have updated the architecture diagram accordingly . I liked this tool and looking forward to use this tool for future diagram too :)

@raviranjanelastisys Anything else missing to merge this one?

@raviranjanelastisys Anything else missing to merge this one?

Nope , @lucianvlad Will merge as he owns this PR :)


[FIX] Improve residuals-based reconciliation stability and faster ma.cov

Adds faster ma.cov method to mint_cov
Improves stability of residuals-based methods.

I was so confused why you were transposing a square zero matrix but it makes complete sense now that you wanted to change the order! 😅
What are you using to benchmark there? It looks amazing and much nicer than timeit. Yes, if you want the output W to have 0 instead of np.nan, your approach is definitely the best. I am surprised that the strict typing had any impact on the performance in this case, even when being really pedantic and specifying a row-major W return with f8[::1, :].
Anyway, the last thing I can think of would be to specify the Fortran order when allocating and initialising the array, but transpose is so fast as it only needs to swap the strides of the array, so it would probably only shave off a few nanoseconds or microseconds depending on the number of time series.
    W = np.zeros((n_timeseries, n_timeseries), dtype=np.float64, order="F")

LGTM! 🚀

Thanks - I agree. The reason the .T is necessary is that Numba doesn't support the order-argument of Numpy functions (that's why I wrote it like that, it's a bit of a low-cost hack to get F-order in Numba).


AggregateException after upgrading to 1.0.1: An attempt was made to load a program with an incorrect format

I just updated my ASP.NET Core project to 1.0.1 via Nuget and now I'm unable to run the app.
Exception
 Exception thrown: 'System.AggregateException' in Microsoft.AspNetCore.Server.Kestrel.dll

{"An attempt was made to load a program with an incorrect format. (Exception from HRESULT: 0x8007000B)"}

StackTrace
   at Microsoft.AspNetCore.Server.Kestrel.Internal.Networking.Libuv.NativeMethods.uv_loop_size()
   at Microsoft.AspNetCore.Server.Kestrel.Internal.Networking.UvLoopHandle.Init(Libuv uv)
   at Microsoft.AspNetCore.Server.Kestrel.Internal.KestrelThread.ThreadStart(Object parameter)

project.json
{
  "dependencies": {
    "Autofac": "4.0.0-rc3-286",
    "Autofac.Extensions.DependencyInjection": "4.0.0-rc3-280",
    "BrockAllen.MembershipReboot": "8.4.0",
    "BrockAllen.MembershipReboot.Ef": "8.0.0",
    "log4net": "2.0.5",
    "Microsoft.AspNet.Authentication.JwtBearer": "1.0.0-rc1-final",
    "Microsoft.AspNetCore.Authentication.JwtBearer": "1.0.0",
    "Microsoft.AspNetCore.Diagnostics": "1.0.0",
    "Microsoft.AspNetCore.Hosting": "1.0.0",
    "Microsoft.AspNetCore.Identity": "1.0.0",
    "Microsoft.AspNetCore.Identity.EntityFrameworkCore": "1.0.0",
    "Microsoft.AspNetCore.Mvc.Core": "1.0.1",
    "Microsoft.AspNetCore.Owin": "1.0.0",
    "Microsoft.AspNetCore.Server.IISIntegration": "1.0.0",
    "Microsoft.Extensions.Configuration.EnvironmentVariables": "1.0.0",
    "Microsoft.Extensions.Configuration.FileExtensions": "1.0.0",
    "Microsoft.Extensions.Configuration.Json": "1.0.0",
    "Microsoft.Extensions.Logging": "1.0.0",
    "Microsoft.Extensions.Logging.Console": "1.0.0",
    "Microsoft.Extensions.Logging.Debug": "1.0.0",
    "Microsoft.Owin.Host.SystemWeb": "3.0.1",
    "Microsoft.Owin.Security": "3.0.1",
    "Microsoft.Owin.Security.Jwt": "3.0.1",
    "Microsoft.IdentityModel.Tokens": "5.0.0",
    "System.IdentityModel.Tokens.Jwt": "5.0.0",
    "StackifyMiddleware": "1.4.1",
    "EntityFramework.BulkInsert-ef6": "<IP_ADDRESS>",
    "AutoMapper": "5.0.2",
    "IdentityServer4": "1.0.0-beta4-update2",
    "IdentityServer4.AccessTokenValidation": "1.0.1-beta1",
    "Microsoft.AspNetCore.Mvc": "1.0.1",
    "Microsoft.AspNetCore.Mvc.ViewFeatures": "1.0.1",
    "Microsoft.AspNetCore.Mvc.WebApiCompatShim": "1.0.1",
    "Microsoft.AspNetCore.Server.Kestrel": "1.0.1"
  },

  "commands": {
    "web": "Microsoft.AspNet.Server.Kestrel"
  },

  "tools": {
  },

  "frameworks": {
    "dnx46": {
      "frameworkAssemblies": {
        "System.IdentityModel": "<IP_ADDRESS>",
        "System.Runtime.Caching": "<IP_ADDRESS>"
      }
    }
  },

  "runtimes": {
    "win7-x86": {},
    "win7-x64": {}
  },

  "buildOptions": {
    "platform": "anycpu",
    "emitEntryPoint": true,
    "preserveCompilationContext": true,
    "copyToOutput": [ "cert.pfx", "log4net.config" ]
  },

  "runtimeOptions": {
    "gcServer": true
  },

  "publishOptions": {
    "include": [
      "wwwroot",
      "Views",
      "appsettings.json",
      "web.config",
      "log4net.config"
    ]
  },

  "scripts": {
    "postpublish": [ "dotnet publish-iis --publish-folder %publish:OutputPath% --framework %publish:FullTargetFramework%" ]
  },
  "userSecretsId": "aspnet-Api-20160702072021"
}

Environment
Visual Studio 2015 Update 3 + KB3165756
ASP.NET Core 1.0.1

It seems #3103 is also related to this one.
Setting platform to x86 works but neither anycpu nor x64 works and cause exceptions
 "buildOptions": {  "platform": "x86" }.

Microsoft.Owin.Host.SystemWeb for ASP.NET Core? How's that working out for you? :stuck_out_tongue:

@khellang I don't use them. They were probably mistakenly added while trying to add references to some classes. The strange point is that they even appeared in the bin directory. :-)

Some observations:
Change runtimeOptions
  "runtimeOptions": {
    "gcServer": true
  },

should be
  "runtimeOptions": {
    "configProperties": {
      "System.GC.Server": true
    }
  },

Change frameworks
  "frameworks": {
    "dnx46": {
      "frameworkAssemblies": {
        "System.IdentityModel": "<IP_ADDRESS>",
        "System.Runtime.Caching": "<IP_ADDRESS>"
      }
    }
  },

should be
  "frameworks": {
    "net461": {
      "frameworkAssemblies": {
        "System.IdentityModel": "build",
        "System.Runtime.Caching": "build"
      }
    }
  },

Delete
  "commands": {
    "web": "Microsoft.AspNet.Server.Kestrel"
  },

Delete this ( AspNet not AspNetCore )
"Microsoft.AspNet.Authentication.JwtBearer": "1.0.0-rc1-final",

Delete this
"Microsoft.Owin.Host.SystemWeb": "3.0.1",

Delete your bin directories. Try again.

@benaadams just curious - I'm using "System.IdentityModel": "" (without any version or build) for assemblies in my frameworkAssemblies section - do you know if there are any important differences?!

@mafshin Did @benaadams suggestions fix your issue?

@halter73 I did some of suggestions and my new project.json is as follows:
{
  "dependencies": {
    "Autofac": "4.0.0-rc3-286",
    "Autofac.Extensions.DependencyInjection": "4.0.0-rc3-280",
    "BrockAllen.MembershipReboot": "8.4.0",
    "BrockAllen.MembershipReboot.Ef": "8.0.0",
    "log4net": "2.0.5",
    "Microsoft.AspNet.Authentication.JwtBearer": "1.0.0-rc1-final",
    "Microsoft.AspNetCore.Authentication.JwtBearer": "1.0.0",
    "Microsoft.AspNetCore.Diagnostics": "1.0.0",
    "Microsoft.AspNetCore.Hosting": "1.0.0",
    "Microsoft.AspNetCore.Identity": "1.0.0",
    "Microsoft.AspNetCore.Identity.EntityFrameworkCore": "1.0.0",
    "Microsoft.AspNetCore.Mvc.Core": "1.0.1",
    "Microsoft.AspNetCore.Owin": "1.0.0",
    "Microsoft.AspNetCore.Server.IISIntegration": "1.0.0",
    "Microsoft.Extensions.Configuration.EnvironmentVariables": "1.0.0",
    "Microsoft.Extensions.Configuration.FileExtensions": "1.0.0",
    "Microsoft.Extensions.Configuration.Json": "1.0.0",
    "Microsoft.Extensions.Logging": "1.0.0",
    "Microsoft.Extensions.Logging.Console": "1.0.0",
    "Microsoft.Extensions.Logging.Debug": "1.0.0",
    //"Microsoft.Owin.Host.SystemWeb": "3.0.1",
    //"Microsoft.Owin.Security": "3.0.1",
    //"Microsoft.Owin.Security.Jwt": "3.0.1",
    "Microsoft.IdentityModel.Tokens": "5.0.0",
    //"System.IdentityModel.Tokens.Jwt": "5.0.0",
    "StackifyMiddleware": "1.4.1",
    "EntityFramework.BulkInsert-ef6": "<IP_ADDRESS>",
    "AutoMapper": "5.0.2",
    "IdentityServer4": "1.0.0-beta4-update2",
    "IdentityServer4.AccessTokenValidation": "1.0.1-beta1",
    "Microsoft.AspNetCore.Mvc": "1.0.1",
    "Microsoft.AspNetCore.Mvc.ViewFeatures": "1.0.1",
    "Microsoft.AspNetCore.Mvc.WebApiCompatShim": "1.0.1",
    "Microsoft.AspNetCore.Server.Kestrel": "1.0.1"
  },

  "commands": {
    "web": "Microsoft.AspNet.Server.Kestrel"
  },

  "tools": {
  },

  "frameworks": {
    "dnx46": {
      "frameworkAssemblies": {
        "System.IdentityModel": "<IP_ADDRESS>",
        "System.Runtime.Caching": "<IP_ADDRESS>"
      }
    }
  },

  "runtimes": {
    //"win7-x86": {},
    //"win7-x64": {}
  },

  "buildOptions": {
    "platform": "x86",
    "emitEntryPoint": true,
    "preserveCompilationContext": true,
    "copyToOutput": [ "cert.pfx", "log4net.config" ]
  },

  "runtimeOptions": {
    "gcServer": true
  },

  "publishOptions": {
    "include": [
      "wwwroot",
      "Views",
      "appsettings.json",
      "web.config",
      "log4net.config"
    ]
  },

  "scripts": {
    "postpublish": [ "dotnet publish-iis --publish-folder %publish:OutputPath% --framework %publish:FullTargetFramework%" ]
  },
  "userSecretsId": "aspnet-App-20160702072021"
}

Although, now I can only get x86 binary.

You have to remove
"Microsoft.AspNet.Authentication.JwtBearer": "1.0.0-rc1-final",

Its not AspNetCore

@benaadams , Thanks I did it.
Upon changing frameworkAssemblies I get this exception:
'build' is not a valid version string.

Yeah, that should be { "type": "build" } :smile:

Everything is fine and OK now, except that I'm locked to x86 which is tracked by other issues (https://github.com/dotnet/cli/issues/3091 & https://github.com/dotnet/cli/issues/2668)

@mafshin check your path make sure
C:\Program Files\dotnet
occurs before
C:\Program Files (x86)\dotnet

@benaadams yes that was the case, the x86 version was before x64 version.
Now, after I swapped them and restarted the machine, the output is locked to x64. It seems the first version (x86 or `x64') in the PATH would only work.

We are closing this issue because no further action is planned for this issue. If you still have any issues or questions, please log a new issue with any additional details that you have.


Add syntax highlighting to READMEs

IMHO this improves the readability of the READMEs substantially

Hi
Thank you for showing interest in the project.
During the pull request, two errors were discovered, which I corrected. Please update your branches in the fork.

@kebasyaty Done

@kebasyaty It looks like someone introduced formatting errors in the Rust code, which is unrelated to these changes.

This formatting is fine for VSCode, but not for GitHub.
When I figure it out, I'll let you know.

This has created a merge conflict and I don't really know the codebase well enough to sort that out, so I'll close this. I would, however, recommend providing syntax highlighting.

I checked out the entire project with cargo fmt --all -- --check and updated the master branch. The formatting problem was only in the migration.rs file. I think you need to update this file in your fork and resubmit the pull request.


Fix typo in ExportJob.java

Signed-off-by: Ramon Medeiros<EMAIL_ADDRESS>

I signed it!

I signed it!


[Ask] How about GSHP?

Hi, andrew
I'm trying to simulate the PyLESA!
But, HeatPump section GSHP doesn't work!!!
Can i get Answer

Unfortunately the development of a ground source heat pump model hasn't been done yet for PyLESA. Currently only air-source or water-source heat pumps can be modelled.
There is complexity with tracking the heat flow and temperature levels between the heat exchanger and the ground. This has been done in other models and it would be a great addition to PyLESA if I ever have the time, or if someone else would like to contribute.
All the best,
Andrew


Setting CookieAuthenticationOptions.ReturnUrlParameter to null or empty does not produce expected results

When setting CookieAuthenticationOptions.ReturnUrlParameter to null or whitespace, unexpected behavior is observed.
When null:
Throws an exception when calling QueryString.Create() from within CookieAuthenticationHandler.cs
When empty:
Produces a mostly un-usable query string parameter (e.g. ?=%2F)
Suggested Fix:
If the CookieAuthenticationOptions.ReturnUrlParameter is set to null or empty, rather than throwing an exception (in the null case) or redirecting with an empty query string key, it should instead redirect to the  appropriate path WITHOUT the query string appened.
For Example:
If the CookieAuthenticationOptions.ReturnUrlParameter is set to null or empty and the root url is accessed in an unauthenticated manner, the redirect path should simply be the CookieAuthenticationOptions.AccessDeniedPath without any query string.

As a rule we don't support nulling out various cookie auth options to disable various features. In most cases you can override the events to customize the behavior.

Looking at the code, it does not appear that there currently is a way to
disable the ReturnUrlParameter getting appended to the Url.
Can this be added as a feature in the form of another property on
CookieAuthenticationOptions such as "AppendReturnUrlParameter" that
defaults to true ?
I'm developing many asp.net core apps that don't rely on the Url parameter
and I'd prefer not to pollute the URL with un-necessary information that
can lead to user confusion.
On Mon, Mar 5, 2018 at 2:12 PM, Chris Ross<EMAIL_ADDRESS>wrote:

As a rule we don't support nulling out various cookie auth options to
disable various features. In most cases you can override the events to
customize the behavior.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aspnet/Security/issues/1682#issuecomment-370585950,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAU-57KkQcGO0CTUgnXt024aC0FtSPsDks5tbbhogaJpZM4SdqSW
.

What about the URL do you believe would confuse your users?

Users may become confused or concerned when strange characters that have no
obvious meaning getting added to a URL where they are entering sensitive
information.
This is not such a big problem when the URL is already long and largely
illegible, but when it's something short like "/sign-in", the extra
characters are much more obvious.
When left as the default, ReturnUrl, it has a fairly obvious meaning.  If
I set the parameter to empty, the URL now has ?=%2F added to it, which
has no meaning and may lead to user confusion.
I'm not looking to get into a philosophical debate about this, however, I
do think that if I can set it to empty, I should be able to disable it
altogether.
On Tue, Mar 6, 2018 at 7:30 AM, Barry Dorrans<EMAIL_ADDRESS>wrote:

What about the URL do you believe would confuse your users?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aspnet/Security/issues/1682#issuecomment-370818875,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAU-59RPg23XQ3r812pPO-KEIpYxeGLHks5tbqsngaJpZM4SdqSW
.

So as you're the only person to ask for this right now it's unlikely to happen. We're confused how blanking the return URL would work in practice too, and adding more options to what is essentially a security API widens the scope that people may misconfigure things and end up in a state where things are going to go wrong.

Fair enough on the demand.
Here's my use case, if you are interested.
I'm building an ASP.NET Core web application that is composed of multiple
front end single page applications that are powered by Vue.js.  I'm using
Identity with Cookies for authentication on the server.  Authentication
happens via a front end component that utilizes a WebApi to communicate
with Identity.  The WebApi sets the auth cookie upon successful
authentication and responds to the client. The client then handles which
screen is shown next, which has nothing to do with http redirection.
No need to beat this to death, but if I understand it correctly, the
ReturnUrl is never automatically redirected to.  The developer has to
explicitly grab the value and redirect from within their controller.  It
seems reasonable to make it configurable since it is never automatically
redirected to by the framework.
Thanks for taking the time to respond.  Cheers.
On Tue, Mar 6, 2018 at 9:51 AM, Barry Dorrans<EMAIL_ADDRESS>wrote:

So as you're the only person to ask for this right now it's unlikely to
happen. We're confused how blanking the return URL would work in practice
too, and adding more options to what is essentially a security API widens
the scope that people may misconfigure things and end up in a state where
things are going to go wrong.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aspnet/Security/issues/1682#issuecomment-370868415,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAU-55ucMSjVYkJe2xvQvc4fseIwDVOGks5tbsyOgaJpZM4SdqSW
.

Ah that's rather out of scope for our templates :) The parameter is used for "normal" usage in the templates, and others use is for forbidden handling as well.

Mind you, I support having the ability to clean up the ReturnUrl query string or even completely disabling it.
It might interfere with caching systems both local and remote; some might be configured to treat the query string part of the URL as a key in their internal store. For example:

/Login?ReturnUrl=%2FHome%2FIndex
/Login?ReturnUrl=%2Resources%2Hello%20World
/Login?ReturnUrl=%2Resources%2Foo
/Login?Example=%2FHome%2FIndex

... might be treated as different resources to be cached while effectively being a single resource to be cached. Using dummy query strings is actually used as a way to 'trick' caches (both local and remote) into pulling a fresh resource from the URL.
Also; it honestly just looks plain ugly to pass around dummy URLs in the query string.

Are you open to a pull request on this if someone else does this?  It looks
pretty trivial.
On Wed, Mar 7, 2018 at 6:29 PM, Barry Dorrans<EMAIL_ADDRESS>wrote:

Ah but having the query string enables a cache server to not suddenly
cache the wrong redirect ... assuming it's not disobeying headers.
You could always just inherit off our cookie pieces and strip it out, but
I really don't think the demand is there in the box, and honestly ... I
really want your users. Lord knows mine don't look at URLs at all.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aspnet/Security/issues/1682#issuecomment-371357484,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAU-5yNyLuLNb2Hw3xucJeMF_YRGPybqks5tcJebgaJpZM4SdqSW
.

Even if the users don't look at your URLs at all, the OWASP black box testers usually do look at your inputs.
Though the software underneath doesn't redirect at all, the testers still have a valid point that a URL parameter can be used for potentially insecure redirects. And while it's completely valid and possible to simply not use the 'returnUrl' parameter from the QS, this is hard (impossible?) to verify that this parameter is unused in a proper black box external audit.
In combination with the potential caching issues, I feel we might have a good reason to simply disable the QS parameter altogether.
Also see: https://www.owasp.org/index.php/Unvalidated_Redirects_and_Forwards_Cheat_Sheet

If your testers are flagging things based on potential rather than reality then my opinion is your testers aren't very good.

I've wondered in the past why the default isn't for the return url to be omitted and for devs to have to opt in if their application actually uses it.  I would turn it off if I could.

Such is the reality we have to work with; black box security testers with a healthy-perhaps-unhealthy dose of paranoia, I suppose.
What are the chances that we might one day expect the ability to completely remove the return URL from the query string?

It's not something we're considering adding. A PR might be considered in our next release.

Thanks for the feedback.  When is the next release scheduled?
On Fri, Mar 9, 2018 at 2:41 PM, Barry Dorrans<EMAIL_ADDRESS>wrote:

It's not something we're considering adding. A PR might be considered in
our next release.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aspnet/Security/issues/1682#issuecomment-371966184,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAU-5xfKWv4tHVZhc3CFZfR_u5grfjLQks5tcwUhgaJpZM4SdqSW
.

Unknown. It won't be in 2.1 though, it'd be 2.2 at the earliest.

After further discussion, we have concluded that this suggestion does not align with what we want to do for cookie auth, so we're closing this issue.


Export webpage files

Hey @gabotechs! I'm curious your thoughts on implementing an export function in the cli, which would simply copy the output files to the specified dir rather than a temp folder. I really want to embed 3D graphs of our code bases inside our Docusaurus pages, which should work perfectly if all the generated files can easily be exported to a dir, that way we can add a dep-tree build command into our documentation CI/CD. If you're keen to implement this, I'd be super grateful, happy to help out in any way including updating readme etc, probably a simple quick feature for you to implement knowing the codebase though!

Hey! I think that should be possible now:
dep-tree path/to/entrypoint --no-browser-open --render-path foo.html

This will dump the html output in foo.html without opening it in a browser. Does that work for you?

Damn that works a treat! Thank you so much! 🚀🚀 Didn't see this in the docs, I'll have a crack editing the readme to add this in if that's okay, if I search the codebase by cmd.Flags() does that have all the extra options or is there other places I should look too?

Not all the CLI options are in the docs, they are discoverable by appending the --help flag in the cli, that will output the available flags with the descriptions. This one is discoverable by:
dep-tree entropy --help

My suggestion is: instead of documenting all the flags one by one in the README.md, let the users know that all the available flags can be displayed by appending the command with a --help flag

Ahh gotcha, I ran -h and thought that was everything, I didn't realise -h and --help give different outputs. Maybe we could merge these and have it in the docs somewhere, then it should be clear enough 👍👍


add configmap support

this PR add support for watching on configmap events

@jjo can you take a quick review on this one if you have the chance?

@jbianquetti-nami LGTM


NvEnc CreateEncoder failed

I get this error message every time I start ALVR.
If I click ok it lets me click connect and launches both Oculus and SteamVR.
Then server goes down as soon as Oculus and SteamVR start.
Then it throws this Unhandled Exception error.

Suggestions?

I think your drivers are outdated or not nvidia...


Readme still says cpp only writes version 0.11?

The readme says that c++ version only writes version 0.11 but from https://github.com/apache/orc/blob/cf720d7b2333618c652445299486cb9600bdbe4f/c%2B%2B/src/Writer.cc#L47
it seems that the default has already changed to 0.12

Thank you for the reporting. Ya, there might be mismatch between the documentation and code.
Could you confirm this, @wgtmac and @stiga-huang ?

Thank you for the reporting. Ya, there might be mismatch between the documentation and code.
Could you confirm this, @wgtmac and @stiga-huang ?

I can confirm that the README is outdated about this. Thanks @XinyuZeng for reporting.

I created a PR #1054 regarding this.


Problem running examples in nix-shell

I'd like to add nix support but I'm having trouble running the examples from a nix-shell:
Listening...
Opening browser...
Fontconfig warning: "/etc/fonts/fonts.conf", line 86: unknown element "blank"

(brave:30564): Gtk-WARNING **: 12:20:35.899: Theme parsing error: gtk.css:68:35: The style property GtkButton:child-displacement-x is deprecated and shouldn't be used anymore. It will be removed in a future version

(brave:30564): Gtk-WARNING **: 12:20:35.899: Theme parsing error: gtk.css:69:35: The style property GtkButton:child-displacement-y is deprecated and shouldn't be used anymore. It will be removed in a future version

(brave:30564): Gtk-WARNING **: 12:20:35.899: Theme parsing error: gtk.css:73:46: The style property GtkScrolledWindow:scrollbars-within-bevel is deprecated and shouldn't be used anymore. It will be removed in a future version
Browser opened.
[30596:30596:0100/000000.000265:ERROR:sandbox_linux.cc(374)] InitializeSandbox() called with multiple threads in process gpu-process.
[30596:30596:0100/000000.000669:ERROR:broker_posix.cc(43)] Invalid node channel message
New connection received.
Reloading code...
[30603:30603:0825/122036.043326:ERROR:broker_posix.cc(43)] Invalid node channel message

Not sure what I might be missing here!
I'm also getting Failed to find own source code. fairly often, which I assume has something to do with stack being called to compile the example?
I tried to change one of the examples to use cabal like so:
#!/usr/bin/env cabal
{- cabal:
build-depends: base
            , reanimate
            , reanimate-svg
            , JuicyPixels
-}

Which allows me to run the code at least. Is there anything else I could share to aid with debugging? I'm also happy to dog in myself with some pointers :-)

Does it work if you specify the path to GHC:
runhaskell examples/doc_drawCircle.hs --ghc-opt `which ghc`

Oh, and the error messages you pasted are coming from brave, not reanimate. The messages from reanimate look to be alright. The code is running, it received a connection from the browser and it's replying with the SVG frames. Are you seeing any error messages in the browser window?

The browser window just says compilation failed. I'm rebuilding my shell to add texlive at the moment (which may take some time). I'll try with a simpler example after that and report back. Thanks for the swift response :-)

Update:
I tried the runhaskell command and it still doesn't work. Complains about not being able to find reanimate.
I've got the rendering working by changing the #! and comment as described.
I still can't get the browser view to work at all. I've tried brave (as opened by the command line) and firefox pointing to the same file. I've got some nixish looking error that I need to look into:
MESA-LOADER: failed to retrieve device information
MESA-LOADER: failed to open i915 (search paths /run/opengl-driver/lib/dri)
failed to load driver: i915
MESA-LOADER: failed to open kms_swrast (search paths /run/opengl-driver/lib/dri)
failed to load driver: kms_swrast
MESA-LOADER: failed to open swrast (search paths /run/opengl-driver/lib/dri)
failed to load swrast driver

But not sure if that could be causing the problem. All I see in the browser is "Compiling..." followed by "Compilation failed" after a short wait.

Are you using stack or cabal to build reanimate?
If runhaskell doesn't work, you can use GHC directly:
ghc examples/doc_drawCircle.hs
./examples/doc_drawCircle --ghc-opt ghc

If everything fails, you can render animations to video files and view them with mpv or another player. The browser viewer is there for convenience rather than necessity.
If you want to chat, feel free to join our discord server. https://discord.gg/Qs28Dv6 I'm quite keen on making reanimate easier to use with nix.

Latest upate, after updating the fork of master:
[nix-shell:~/docs/1.Projects/VisualiseValue/reanimate]$ ./examples/doc_drawCircle.hs
Warning: The package list for 'hackage.haskell.org' is 122 days old.
Run 'cabal update' to get the latest list of available packages.
Resolving dependencies...
Build profile: -w ghc-8.8.3 -O1
In order, the following will be built (use -v for more details):
 - fake-package-0 (exe:script) (configuration changed)
Configuring executable 'script' for fake-package-0..
Preprocessing executable 'script' for fake-package-0..
Building executable 'script' for fake-package-0..
[1 of 1] Compiling Main             ( Main.hs, /home/ben/docs/1.Projects/VisualiseValue/reanimate/dist-newstyle/build/x86_64-linux/ghc-8.8.3/fake-package-0/x/script/build/script/script-tmp/Main.o )
Linking /home/ben/docs/1.Projects/VisualiseValue/reanimate/dist-newstyle/build/x86_64-linux/ghc-8.8.3/fake-package-0/x/script/build/script/script ...
Rendering in browser window is only available when interpreting.
To render a video file, use the 'render' command or run again with --help
to see all available options.

Can I close this?


No workspaces experience isn't accessible

I don't think this is accessible, as the user can't interact with that combo box.

Are we sure that the accessibility "cursor" can't get there? Maybe we can ask Dante?

They'll find this in the Accessibility testing. Closing.


Add more unit tests for secondarynetwork feature

Fix one of the tests (TestPodControllerAddPod/missing_Status.PodIPs)
and add a few more unit tests.
For #4142
Signed-off-by: Antonin Bas<EMAIL_ADDRESS>

/skip-all

/skip-all


Add support for Sourcelink

You can now specify that Coverlet uses SourceLink URIs in place of document paths in coverage results.
MSBuild:
dotnet test /p:CollectCoverage=true /p:UseSourceLink=true

Global Tool:
coverlet test-assembly.dll --target "dotnet" --targetargs "test test-project --no-build" --use-source-link

cc @ViktorHofer
Fixes #257

You know that question was coming. When can we consume that change?

@ViktorHofer most likely tomorrow. I need to add a few more things before making a new release

@ViktorHofer sorry it took much longer than expected, here's the new release with source link support https://www.nuget.org/packages/coverlet.msbuild/2.5.0


add package-lock.json

just a quick lil PR to add a package-lock.json

thank you! this repo is using pnpm, so it has a pnpm-lock.yaml rather than a package-lock.json — closing this in favour of #13 which gitignores other lockfiles and adds an explanatory note to the README


Fixed typos for 'search' & 'deprecated'.

thanks for the corrections, @phoenixfangor . The build seems to be failing for an unrelated cause. Will look at it and merge post that.


Update version: GiulioSorrentino.numeronesoeminformations version 1.1

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

hello, the package works and is il it's simply blocked by smartscreen

hello, the package works and is il it's simply blocked by smartscreen

Automatic Validation ended with:

Installation failed with exit code -1978335187
2023-09-15 02:03:45.452 [FAIL] Installer failed security check. Url: https://github.com/numerunix/numoeminformations/releases/download/1.1/numoeminformations-1.1.exe Result: 0x80004005

Unfortunately, this package didn't pass the security check inside WinGet. This might be a false positive and we can retry tomorrow.

Ok, then i'll not update pull request, but wait.


Add KCSNA2021 summit information

This stubs out a good chunk of the initial summit information that we have so far.

/lgtm
/hold
If others want to review it.

/hold cancel

/lgtm


Cant not support Chinese file name.

subprocess.getoutput() cant not execute when the post contains Chinese character.

Show me more detials about the issue, please:

What's your system locale encode type ?
Would you paste the Python error logs here ?

Fixed at 71328bd055d155b010f6ef378cd9f924be0045df.


implemented usage of numpy memmap for spilling to disk

Please do not auto merge as, auto merge will probably remove cache_blocking

Can we please reconcile this with master before review?

I have done a manual merge with the master, and also simplified the implementation so the only changes should be interfaces.py and tti_examples.py. Please take a look, thank you.

Ok, following the flurry of recent merges this needs to be reconciled with master again. Sorry about that.
Also, docstrings are missing throughout, and it would be great if we all become more conscientious about in-code documentation.

Hi, regarding the docstring, I have not written these class, just add some functions into the classes, I was just following the coding style. I will just add docstring for these classes then. Thank you.

Yes, please provide docstrings for the functions you add, we'll take care of improving the class documentation. Admittedly, we have up to now been pretty naughty regarding in-place documentation, but as our developer group and feature range grows we all need to improve on this, so as to make everyone's life easier going forward.

Hi, I have made changes according to previous discussion, please take a look, thank you.

Ok, will do. Can you please reconcile with master to allow auto-merge?

Hi, I have already merged master into this branch, should be fine with auto merge, thanks.

Yet github tells me you are 40 commits behind master.

I think that its probably because I used meld to merge manually, I did fetch the latest master, I will just do git merge on memmap and push a new commit

Hi, I have merged with master.

Hey, you merged into master again, without checks passing! Your changes should be reviewed again and a full travis run needs to pass before you can merge! Do I really need to lock the master?

Hi, sorry, I will remove the commit

Ok, note that you will need to open a new PR for this branch now, since github won't let you re-open already merged ones.

Hi, I think that I did something wrong and removed too much, I should I revert it? Thanks.

Its fine now, I figured out how to revert it. Sorry for that.


Wrong banner color?

I'm not sure if this is part of the ui theme, but the banner should have a green color.

Hmmm, that's interesting @danqing. Can you provide steps to recreate that behavior?

You need to have some packages with the red bug icon in the status bar. Then if you click on it and choose "rebuild package", you will see this banner.

What was the package in question @danqing ?

The one I downloaded is go-debug :)

Thanks again for finding this @danqing. I've updated the code so these alerts appear as they should.


Sanitycheck fixes

tests: syscalls: ignore faults, they are intentional
We are blowing up the kernel here intentionally, so ignore the faults on some
platforms.

sanitycheck: do not abort logging on faults
We have been dropping lines after finding a fault which resulted in missing
information in the log. Make sure we continue and only report failure at the end
of the execution.

sanitycheck: capture delayed faults
Do not close console after PASS is reported, wait a bit for any remaining
messages from the tests, sometimes we have faults that need to be parsed. This
now works for Qemu handler, support for other handlers to follow.

Fixes #9646

Codecov Report

Merging #9716 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #9716   +/-   ##
=======================================
  Coverage   52.31%   52.31%
=======================================
  Files         212      212
  Lines       25939    25939
  Branches     5590     5590
=======================================
  Hits        13569    13569
  Misses      10123    10123
  Partials     2247     2247

Continue to review full report at Codecov.

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

blocked by #9737

recheck


Feature: Collaborative editing

Is your feature request related to a problem? Please describe.
It is often that we are working synchronously together when looking at a kanban board. This means the displays can get out of sync, if one user edits something, and the other doesn't reload afterwards. This can lead to inconsistencies.
Describe the solution you'd like
An implementation of a real-time communication layer (through long polling, Websockets or WebRTC) together with a conflict resolution mechanism could build the foundation for collaborative editing of boards - not cards.
Pad like cards with real-time editing of their descriptions could then be some kind of Etherkanban, where changes are picked up immediately and distributed to other contributors. The Taskcafe Markdown editor is a nice approach at composing Markdown texts, almost like Ghost's and WordPress' card based editors nowadays.
Describe alternatives you've considered
An alternative could be to lock a board for writing through everyone, and only display changes made by the "presenter" to the "audience".
Additional context
Like in Trello/Wekan/Kanboard/Taiga ...

Hey thanks for the feature request, however this is already planned and in the roadmap.

I've enabled simple update polling so that at the very least if another user adds a task or something, you don't have to manually refresh the page to see the new changes!
I'd like to eventually move this over to a websockets system using GraphQL subscriptions, but I think this will be fine for the time being.
(the polling interval is 3 seconds, though that might change as I play with it more).


WIP: Prepare snapshot for Node 10

Please submit this PR against https://github.com/kartotherian/kartotherian


authorized_keys inconsistency when using "admin regenerate keys"

Gitea version (or commit ref): 1.10.0+dev-271-g7eacdcf39
Git version: 2.22.0
Operating system: Alpine Linux 3.10.2
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

Description
Adding an SSH key to your account adds an entry in authorized_keys with command="/app/gitea/gitea serv key-X ..." .... Using the gitea admin regenerate keys command, those entries will be turned into command="/usr/local/bin/gitea serv key-X ..." ....

Are you passing the same configuration to the CLI command as you have gitea web (the -c flag is helpful in this case)?
Are you running gitea inside Docker?

Hey @techknowlogick,
I am using Gitea from a Docker container. I do not pass a -c flag to gitea web.

The inconsistency is because when docker runs Gitea it runs Gitea from the /app/gitea/gitea path directly whereas when you run it from the command line you're running it from the symlink /usr/local/bin/gitea
Gitea just puts whatever is in $0 when it's run.
It won't have any effects as far as I can tell.
We could change our docker to just put /app/gitea on the path and drop the symlink? Or replace the symlink with a script.

I think gitea has been the PATH in the docker.

Docker runs /app/gitea/gitea directly from the /app/gitea directory:
https://github.com/go-gitea/gitea/blob/273a24f22676b73a648fd2a5467e385ec41e84e2/docker/root/etc/s6/gitea/run#L4-L6
There is no PATH change in Dockerfile.
The final step in Dockerfile is to create a symlink:
https://github.com/go-gitea/gitea/blob/273a24f22676b73a648fd2a5467e385ec41e84e2/Dockerfile#L64
This has two effects:

When running Gitea from the command line in docker the $0 will not be the same as the docker run Gitea. Hence the /usr vs /app
The AppWorkDir and hence custom will not be the same when running from docker command line compared with running from the docker. entry point

There are three possible solutions to the first part:

Run as /usr in the s6 run
Add /app to the path and drop the symlink
Make /usr a script shim which runs /app

The second has the following options:

Change the docker build to use the LDFLAGS trick as I previously suggested. (Needed for the first two cases)
Set these in the script.


Package source folder including sub-directories (packages)

This is a fix, inspired by the detailed report from #51 , that also fixes #55.
Tests:
automated tests not applicable, as it was a bug relating only to the setup.py file, which had to be changed now that we have a subdirectory (sub-package?) in the appdaemontestframework directory.
I did of course test with the test case I had constructed in #55, by installing the wheel I created using this new setup.py file and the test then succeeded and failed as expected when changing the "automation".
So I confirmed this works.

Thanks a lot! I'll merge this 😃👍
Out of curiosity, what does find_packages do?

Thanks for merging!
From what I read before, it allows you to specify folders to include, but also (or only) exclude, and has support for stuff like wildcards. Basic functionality for if you want to package more than just one or two flat folders.
PerhapsI should have left out the star, would have been more in line with this repo's structure.

Oh, so if I understand correctly the version without find_packages was failing because it failed to fetch the appdaemon_mock sub-package. While find_packages will recursively select all packages? Is that correct?
On a side note, the build broke on python 3.6. But considering python 3.6 is going to be deprecated by the end of the year I don't mind too much. I removed 3.6 from the tox setup. Also, since the wildcard wasn't necessary I removed it. Let me know if that causes problems 🙂

Yes, exactly! That is why I added find_packages instead of adding the sub-package to the list explicitly: this will also work if we add more nested packages. inside appdaemon-test-framework.
And yes, AFAIK it should work without the wildcard, let's see :)


[ISSUE #336]add DelayTimeLevel enum

Please do not create a Pull Request without creating an issue first.
What is the purpose of the change
Add DelayTimeLevel enum;
add setDelayTimeLevel(DelayTimeLevel level) method;
Brief changelog
Because I used ROCKETMQ for the first time, I was very puzzled about the setdelaytimelevel function when I used the delay function, it took a lot of time to find the solution, so I wanted to add an enum to more intuitively Setdelaytimelevel
Verifying this change
run unit test  testGenerate_Delay_Enum()
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Nice catch!

Not a good idea. The mapping between DelayTimeLevel integers and the actual delay time is configurable on the broker side.

I will close the pr, if you happened to the same question, please let me know.