Bug: Incorrect Rendering of Dollar Amounts in Output

Bug Report
Description
Bug Summary:
The AI-generated output contains dollar amounts, but they are being incorrectly rendered with LaTeX-style formatting (e.g., $740 for $5 million is rendered as 740for5 million in LaTeX font). This makes the output difficult to read and understand.
Steps to Reproduce:

Provide the AI with input text containing dollar amounts and insurance coverage details, as shown in the example.
Generate the AI output based on the provided input.
Observe the rendered output image.

Expected Behavior:
The dollar amounts in the AI-generated output should be correctly rendered as regular text, without any LaTeX-style formatting. For example, "$740 for $5 million coverage" should appear as is, without any distortion.
Actual Behavior:
The dollar amounts in the AI-generated output are being rendered with LaTeX-style formatting, such as "740for5 million coverage".
Environment

Operating System: macOS, iOS
Browser (if applicable): Safari

Logs and Screenshots
Screenshots (if applicable):

Additional Information

The issue seems to be related to the rendering of dollar amounts specifically, as other parts of the output appear to be rendered correctly.
This bug may be caused by a conflict with LaTeX-style formatting or an issue with the rendering engine used in the project.

@tjbck it looks like a fix was introduced in #130 but then regressed in  #835 ([line]).(https://github.com/open-webui/open-webui/blame/5558514ff19c088fd0c8c87205d7a89a0d115a1c/src/lib/components/chat/Messages/ResponseMessage.svelte#L131)].

LaTeX has been a thorn in our side for a bit.. 😔

Should be fixed on our dev branch now, let us know if the issue persists!

Closing this as fixed due to 2c03021, will re-open if needed!


DPL-183 As Stef L. I would like the MVP features for a new Targeted NanoSeq  pipeline so that we can accurately track, process and report samples (C=?, V=4)

User story
As Stef L. I would like the MVP features for a new Targeted NanoSeq  pipeline so that we can accurately track, process and report samples.
Who are the primary contacts for this story
Stef L.
Acceptance criteria
To be considered successful the solution must allow:

[x]  For samples that don’t need to be cherrypicked the LTN stock plate can be converted to a LTN Cherrypick plate in sequencescape.
[x] New plate purposes - prefixed LTN instead of LDS #913
[x]  The supplied material is sheared by sonication. Upon scanning the LTN Cherrypick plate into Limber enable ‘Add an empty LTN shear plate’ (print labels).
[x] Bed verification should be available for the Bravo (cherrypick plate on position 7, shear plate onto position 9)
[x] ‘Add an empty LTN Post Shear plate’ (print label).
[x] Bed verification on Bravo (shear plate onto position 9 and post shear plate onto position 7).
[x]  Two new library types are required for the Re-ISC submission: Targeted NanoSeq Pulldown Twist and Targeted NanoSeq Pulldown Agilent.
[x] An integration suite test
[x]   The final sequencing submission is made by the operational team as is currently the case for ReISC.

Dependencies
References
This story has a non-blocking relationship with:
#913
Additional context
This came out of discussions around the changes we made to accommodate the Targeted Nanoseq pipeline within the existing Duplex Seq pipeline, and the conclusion that this was not really appropriate and that it should have it’s own pipeline.
Ops have a workaround for this BUT it involves the submission of plates with handwritten barcodes and shearing in SM without sample tracking.
This will not scale to volumes of samples expected in Jan 2022.

Targted Duplex-seq Process map.pdf

NB. On deploying we need to check the following request types via the rails console (as the configuration changes in the code do not update request types that already exist).
Check request type 'Limber ISC' does not have the 'Targeted Nanoseq Pulldown Twist' and '.. Agilent' library types.
Check request type 'Limber ReISC' does have the 'Targeted Nanoseq Pulldown Twist' and '.. Agilent' library types.
Check request type 'Limber Targeted Nanoseq' only has the 'Targeted Nanoseq' library type and not the Twist and Agilent ones.
e.g.
rt = RequestType.find_by(name: 'Limber Targeted Nanoseq')
rt.library_types
lt  = LibraryType.find_by(name: 'name of library type')

to add a library type to list:
rt.library_types << lt
to remove from list:
rt.library_types.delete(lt)


Formatting out commas before line breaks

Hi! I love this extension, but it has a serious problem at least for my use case. Since my prompts get pretty long, I often format them with line breaks to help me see each part of the prompt. With this extension, if my prompt has a comma at the end of one part where there is a paragraph, it takes the comma out
For example
beautiful blue sky, sandy beach, fluffy clouds,

1girl, one piece bathing suit, happy, volleyball

becomes
beautiful blue sky, sandy beach, fluffy clouds

1girl, one piece bathing suit, happy, volleyball

At first I thought maybe this was purely aesthetic, but it does change the output (in my opinion for the worse) when the comma before a line break is removed.
It would be incredible if there was a way to disable this behavior!

ngl, I also encountered this issue from time to time XD
Added an option (defaults to enabled) that appends a comma at the end of a line, when there are multiple lines.

Based dev

I tested this and there is a problem. Now, commas appear on all line breaks. For example,

1girl, one piece bathing suit, happy, volleyball```

Becomes

```beautiful blue sky, sandy beach, fluffy clouds,
,
1girl, one piece bathing suit, happy, volleyball```

Ah, good call

If we're looking at exceptions like BREAK, there is also ADDCOL, ADDCOM, ADDROW from regional prompter to consider lol
Do commas actually break this stuff, though? I'm not sure they do.


xSQLServer: Opt-in for the common test for markdown files

Details of the scenario you tried and the problem that is occurring:
We should opt-in for the common test for markdown files.
To opt-in, create a '.MetaTestOptIn.json' at the root of the repo in the following format:
[
     "Common Tests - Validate Markdown Files"
]

The DSC configuration that is using the resource (as detailed as possible):
n/a
Version of the Operating System, SQL Server and PowerShell the DSC Target Node is running:
n/a
What module (SqlServer or SQLPS) and which version of the module the DSC Target Node is running:
n/a
Version of the DSC module you're using, or 'dev' if you're using current dev branch:
Dev

I'm working on a PR to fix lint error MD013 and MD036. But to be able to opt-in we need to fix MD024, and for that we need to have the common test changed. See issue https://github.com/PowerShell/DscResource.Tests/issues/98.


docs(skeleton): added doc text and images

General summary
Added docs for skeleton component which will help build loading states (doc coming soon)
Screenshots

Info

Demo blocks are images and need replacement by real coded component need:dev-integration
Properties aren't the good ones yet, need to by replaced by skeleton's need: dev-frontend

Cherry-picked in skeleton PR


Support for tensorflow 2.x

The current package uses the TensorFlowSharp package, which has unfortunately been deprecated before adding support for TF 2.x. This is currently preventing this package from supporting models trained with the latest DLC releases.

hi - any progress on this? else a workaround? I am having the issue that we've trained in DLC with TF 2.3 and then I've been trying to export in various ways and with various TF exports. thanks! happy to help if there's a good way to help

for posterity, DLC twitter replied that we should just retrain on the same tensorflow version, which we did

Fixed in a808ad57ec1bb2d0a4be760a996fe6ed4bfac634


To do list structure

I have created an index.html file and wrote my HTML markup there. I have created an empty To Do List placeholder ( element). The index.html file is set as a template using the HTML Webpack Plugin.

I have created an index.js file and set an array of some simple to do tasks (array of objects). Each task object  contains three keys:
-description [string].
-completed [bool].
-index: [number].

I have a function to iterate over the tasks array and populate an HTML list item element for each task.

On page load I have rendered the dynamically created list of tasks in the dedicated placeholder. The list appears in order of the index values for each task.

I have created a style.css and set rules for the To Do List. CSS is also loaded by Webpack Style/CSS Loader.


Always generate .corrected files when used in dune

While toying around with mdx.1.4.0 I stumbled upon the bug fixed by https://github.com/realworldocaml/mdx/pull/136. This bug is now fixed but one thing I noticed is that it manifested in a really weird and disturbing way if you used ocaml-mdx rule to generate your mdx test rules.
Because the generated rule uses (diff? %{x} %{x}.corrected) actions, it will silently pass if no .corrected file has been generated.
I'm wondering if it wouldn't be more robust to have ocaml-mdx rule to produce a rule that generates all the .corrected no matter what, using the --force-output option of ocaml-mdx test and to produce strict diff rules thus preventing any such silent failures.
Let me know what you think!

See https://github.com/ocaml/dune/issues/2482

I'm not sure it's related as the issue here was that nothing happened because the second argument of diff? was missing (ie wasn't generated by MDX). I might be misunderstanding this but I don't think the new diff? will change that.

In other words diff? behaved as it was supposed to but that made it hard to even know there was a bug in the first place.

Isn't this a similar issue that @clecat had in https://github.com/ocaml/dune/issues/2396 ?
But if the real issue is that mdx doesn't generate a .corrected file when it should, it will be better to just fix that issue instead of trying to work-around possible bugs.

I'm closing this as:

it is not entirely true, Jenga still relies on this behaviour (i.e. running the diff ourself in mdx)
we now have a dune stanza which forces that behaviour by passing --force-output anyway


New install: julia hangs for few seconts and does nothing

Hello I tried to install juliaup from the Windows Store. After installation if I call Julia from the start menu the REPL windows opens, fully black, hangs for a few seconds and closes. If I try to call julia in the shell (both PowerShell and Command Prompt) the call hangs for some seconds and returns to the prompt without opening thee REPL.
I can confirm that juliaup was installed, as it seems to work. For example:
PS C:\Users\XXXXXXX> juliaup status
Installed Julia channels (default marked with *):
  * release

I also have the ~/.julia directory and I even can find the julia executable in there at ~/.julia\juliaup\julia-1.6.1+0~x64\bin. I can even make it work by moving to that dir and typing  ./julia at PowerShell.
I have also checked whether something that looks like a symlink to the julia executable appears in the ~\AppData\Local\Microsoft\WindowsApps and it looks like it is there (I am no expert in Windows):
PS C:\Users\XXXXXXXXXX\AppData\Local\Microsoft\WindowsApps> ls

    Diretório: C:\Users\XXXXXXXXXXX\AppData\Local\Microsoft\WindowsApps

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
d-----        14/07/2021     20:36                Backup
d-----        14/07/2021     20:39                JuliaComputingInc.Julia_b0ra4bp6jsp6c
d-----        27/11/2020     19:18                Microsoft.DesktopAppInstaller_8wekyb3d8bbwe
d-----        14/07/2021     11:50                Microsoft.MicrosoftEdge_8wekyb3d8bbwe
d-----        14/07/2021     20:19                Microsoft.SkypeApp_kzf8qxf38zg5c
d-----        14/07/2021     20:18                Microsoft.XboxGamingOverlay_8wekyb3d8bbwe
-a---l        14/07/2021     20:18              0 GameBarElevatedFT_Alias.exe
-a---l        14/07/2021     20:39              0 julia.exe
-a---l        14/07/2021     20:39              0 juliaup.exe
-a---l        20/04/2021     18:28              0 MicrosoftEdge.exe
-a---l        27/11/2020     19:18              0 python.exe
-a---l        27/11/2020     19:18              0 python3.exe
-a---l        14/07/2021     20:19              0 Skype.exe

I am not sure what I can try next.

Is there a file ~.\.julia\juliaup\juliaup.json? If it is, could you post the content of that here?
One other idea: are there any spaces or special Unicode characters in your profile directory name, i.e. the part that you XXXed out? Staring at the C++ code, I am actually starting to think that I might have an error in how I handle (or rather don't) Unicode characters in paths...
Oh, and just to confirm: you never called juliaup add or juliaup update in any form, right?
Thanks so much for the detailed error report!

The juliaup.json is there. Its contents are:
{
    "Default": "release",
    "InstalledChannels": {
        "release": {
            "Version": "1.6.1+0~x64"
        }
    },
    "InstalledVersions": {
        "1.6.1+0~x64": {
            "Path": ".\\julia-1.6.1+0~x64"
        }
    }
}

My user name is "Paulo J. S. Silva" with the spaces but no accents or other diacritics.
I have never called juliaup add or juliaup update.  If want me to try something else, please just let me know.

The file looks good, that can't be it... I'm building a version now that outputs a bit more diagnostics, maybe that will help. One thing that would be great is if I could put you on a different version in the store (or flight, as it is called in Windows Store terminology), that way I could push new builds out much quicker to you to try to get to the bottom of this. To do that I would need to know your Microsoft Account name. If that would be ok, could you maybe send that either to my email address (on my GitHub profile) or via Slack? Once we have resolve this I'd put you back on the normal version in the store. There is nothing else you would need to do, i.e. can configure that you get special builds on my end as long as I know your Microsoft Account.

I'm sorry if this is not the problem but I wonder if this is related. But did you have a previous Julia install with something in ~/.julia/startup.jl? I have had julia sessions die immediately for some thing that are in that file (something related to revise perhaps).

@pjssilva I've actually pushed the build with more diagnostic output to everyone. It will take a few hours for that to make it through the Windows Store pipeline, but then it should auto-install for you. Once you are on version <IP_ADDRESS> (you can see that in the Store, or via juliaup -v), can you then try to run julia again from PowerShell? Hopefully it should now show some diagnostic output.
@aramirezreyes ah, that is a good idea! But I'm actually fairly positive that in this case there is a real bug in my stuff: I can see that there is one machine in Brazil that properly crashes, the Windows Store shows me that much information (but nothing else). So my best guess is that that is actually @pjssilva :)

@aramirezreyes I don't remember using Julia in this machine at all. I actually don't user Windows, basically never. But I am going to give to courses at the University that involve Julia and I had decided to try juliaup to see if it was an easy way for the students to install Julia.
@davidanthoff I have sent you my Microsoft user by email. I have also tried the new version on power shell. It outputs
PS C:\Users\Paulo J. S. Silva\.julia\juliaup> julia
The latest version of Julia in the `release` channel is 1.6.2+0~x64. You currently have 1.6.1+0~x64 installed. Run:

  juliaup update

to install Julia 1.6.2+0~x64 and update the `release` channel to that version.
runtime_error: %1 nÒo Ú um aplicativo Win32 vßlido.

The first part is just warning me that Julia 1.6.2 was just released. The actual error is on the last line. The string is trying to say in Portuguese "não é um aplicativo Win32 válido". As you can see the accented characters are garbled. Probably a bad encoding is being used. In English the same string should probably be "It is not a valid Win32 application".

Ah, this is all super helpful!
I looked at the code again, and I'm actually simply not handling spaces in paths properly, so pretty sure that explains the error you are seeing. I'm pushing a new build right now that should fix that. And I also tried to fix the encoding for error messages, although I am a little less sure whether I got that one right.
And thanks for your email, I'm going to push this new build out to just you, and maybe you could let me know if that fixes things for you? If it does, I'm going to release it broadly. The version of the new build is <IP_ADDRESS>. It is going through the store submission right now, but that shouldn't take too long (they run fewer tests on it if it is pushed to a private target group like you).

@davidanthoff I am still at 1.56.0. Do I need to do anything on my end. Like uninstall juliaup and install again?

@pjssilva, in theory you should just automatically get the update, I think, as long as the email address you sent me is the one you use to logon to Windows itself. You could maybe check in the Windows Store app for updates?

I had to logout and login again to make it install thee new version. juliap -v now gives v1.0.57 (no .0 at the end). Most important: it works! Great! Thanks  @davidanthoff!

Fantastic, thanks for providing all that detailed info, that was a pretty important bug to fix :) I'll push this build out to everyone now and then I'll move you back to the normal released versions.


Fix wetday frequency correction

[x] closes #173
[x] tests added / passed
[x] docs reflect changes
[x] entry in CHANGELOG.md

essential changes :

in the core function, instead of using a unique replacement value, use an array of (likely) different values, in process='pre'
values strictly below 1 are modified, rather than below or equal to 1, in process='pre'.
make tests deterministic (with a seed), smaller and more precise.

secondary changes :

in the core function, directly operate on the data array object within the dataset -- which means I have to introduce a var parameter -- default value is 'pr', but in tests we have other variable names. Need that parameter in the service as well.

@dgergel I requested your review for the method check.
@brews, in the computational side. I tested this on both Jupyter and Argo, and this ran within 3 mins using around 15GB of memory. with the size of the numpy sample array, with a GCM data array shape. We might need to increase slightly the resources of that step so that the ERA-5 data fits in.
But it will fit in the node and run fast.

Thanks for cleaning this up, @emileten.
My one suggestion is to rename this new var argument to variable. This makes it consistent with the other functions and methods that grab a variable name. ...I see "var" and I think "variance" but that's an aside....

Thanks @brews, I changed this !


[feat] add ability to choose rpc nodes

Describe the problem
Currently the rpc node is hard coded in some places.
For example https://github.com/teia-community/teia-ui/blob/develop/src/context/HicetnuncContext.js#L46
Describe the solution you'd like
It would be nice to be able to configure the rpc node endpoint in the .env file.
Alternatives considered
No response
Additional context
No response

There's also this one: https://github.com/teia-community/teia-ui/blob/develop/src/context/HicetnuncContext.js#L909

It might also make sense to make the rpc node selection user configurable: https://docs.google.com/document/d/1vYkB_BTwCQIzMPxQuX0EHrBCvjjxPJ-eZEtztN15mGI/edit?mode=html#heading=h.8unfjcgegwi8
versum.xyz already allows users to change the rpc node.

I'll close this but reference #189


Undo is not working

Woodworking actions are not registered for undo which makes undoing mistakes very hard.

Yes, this is true. But the idea behind woodworking tools is to keep it so simple, to not need undo things. You can remove object and quickly create new one.
I am not sure if some operation can be "undo" especially those Cube -> Pad replace. Probably FreeCAd will not be able to re-create the Cube from Thickness profile. It is more quicker to remove the profile and create new one.
However, I will keep this issue open for some time, to look closer at this. You have any specific situation that the undo would be needed?

This becomes a problem when you have decades of muscle memory.
I have the habit of measuring and working without pencil notes, I do several things at the same time while taking care of my young son. So I go to the kitchen to observe the food that is on the fire, I answer the phone and when I come back my son is already using the keyboard or mouse. No matter what happens, I unconsciously use ctrl+z and lose everything I've done hahahaha.
But don't worry about me, the undo history is a chronic problem with FreeCad.

@MauricioLucas Someday you will come and see there is new FreeCAD version written ;-)
The woodworking workbench approach is little different than FreeCAD "parametric" and "undo" approach. The woodworking workbench approach is more like "building from boxes", should be quick and simple. For sure you can "undo" some PartDesign operations.

Don't worry, even with some mishap, I'm really enjoying using your Workbench

Seems i have to rework my work flow a bit. I am quite similar to @MauricioLucas  so Ctrl-Z is a reflex which unfortunately does a lot of damage in woodworking workbench. But ok, i need to approach the workbench like real woodworking where there's also no undo :-)
Besides that i really love your workbench.

i need to approach the workbench like real woodworking where there's also no undo :-)

yes, think twice, cut once ;-)

Besides that i really love your workbench.

Thanks. It is always very nice to hear that someone like the work you do. The "undo" is not priority for the workbench but I will look into that if I finish the support for irregular shapes. So, I leave this issue open until this time.
I usually use CTRL-Z at Kwrite so I know it might be useful. At FreeCAD I click CTRL-S very often and make backup of the file ;-)

Adding undo is a mater of adding two lines. Why wouldn't you add that?
You can look at my simple tools that uses the undo mechanism.. It is easy to implement .. just a copy-paste

Why wouldn't you add that?

Hello Mariwan here LOL ;-) because I don't use undo? for me it is not needed feature. If I make mistake I usually remove object and create new one. Backup is always essential. This is simple with PartDesign objects, here is sample how you can quickly remove object and create new one: 3D furniture designing - construction profiles of course not all features works like this...

Adding undo is a mater of adding two lines.

I am pretty sure this will be more complicated and FreeCAD will not be able to re-create objects...

You can look at my simple tools that uses the undo mechanism.. It is easy to implement .. just a copy-paste

but I will look at your code and at this issue closer, I will play with CTRL-Z with all features and will see what will happen ;-) kernel panic? ;-) first I need to find some kind of workaround for 0.20, and next I will look at it...

For all PartDesign features you can remove last operation. If you press CTRL-Z sometimes the Cube is restored. So I don't see issue here. Please be more specific.


You can use a Timer as well.

For the SetFocus work around that gets triggered by a button, you can also have a screen with a timer that has a duration of 1 to perform a Navigate to the next page where the SetFocus is. This makes it look like the app is just loading and avoids any extra legwork of pushing a button. I use this for an app on its startup and also to circle back to the its "Start screen" after form submission and it works for having the SetFocus automatically selecting a text input.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 073de527-a32b-8d1d-05d5-d3fd629008cd
Version Independent ID: 6137f8a6-7d45-67f2-5d59-02e3d12ff5ed
Content: SetFocus function - PowerApps
Content Source: powerapps-docs/maker/canvas-apps/functions/function-setfocus.md
Service: powerapps
GitHub Login: @gregli-msft
Microsoft Alias: gregli

Thank you for reaching out and sharing your findings. Please feel free to share such findings, ideas and solutions with rest of the community using Power Apps community forum.


Embed latex error messages in the document

Right now when latex fails, we still have to go to logs for debugging. If we can replace the frame with the error message, it would be much nicer.

Try compiling it for the first time. If can't work, parse the error message to see which frame did the errors come from. Replace that page's content with the error message when generating the parsed content and recompile.

#25


Security vulnerability: Web Server Directory Traversal Arbitrary File Access

It is possible to read arbitrary files on the remote host outside the web server's document directory using a specially crafted URL. An unauthenticated attacker may be able to exploit this issue to access sensitive information to aide in subsequent attacks.
For example, after deploying Stratos as all-in-one docker container, I can read the /etc/passwd file content with https://:4443/../../../etc/passwd URL using a tool like Fiddler.

Thanks @rokejung for reporting this.

Looks like this was an issue restricted to the all-in-one deployment type

all-in-one - contains the troublesome Echo middleware that serves files and allowed the bad path. The fix now ensures requests with these paths are rejected. If not building locally, please wait 24hrs for the AIO images to be updated.
cf application (cf push) - Not affected - also uses Echo, however CF routing seems to refuse these paths.
kubernetes & helm charts - Not affected - Serves files by nginx, which handles bad paths
docker compose - Not affected - same as k8, uses nginx


[Do not merge] Add randomness to the first CSR request

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: wattli
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

security/OWNERS [wattli]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@wattli: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
27cf033ef9683cf157e82bf6ea953ff790cc16a4
link
/test e2e-simpleTests-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
9fdf778e61497aa31f57fffe0eb52f8276f85fbe
link
/test e2e-simpleTests-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
9fdf778e61497aa31f57fffe0eb52f8276f85fbe
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
9fdf778e61497aa31f57fffe0eb52f8276f85fbe
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
9fdf778e61497aa31f57fffe0eb52f8276f85fbe
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
9fdf778e61497aa31f57fffe0eb52f8276f85fbe
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
9fdf778e61497aa31f57fffe0eb52f8276f85fbe
link
/test istio_auth_sds_e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: quanjielin, wattli
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

security/OWNERS [quanjielin,wattli]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
9fdf778e61497aa31f57fffe0eb52f8276f85fbe
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
9fdf778e61497aa31f57fffe0eb52f8276f85fbe
link
/test istio_auth_sds_e2e

prow/e2e-simpleTests-cni.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test e2e-simpleTests-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e_pilotv2_auth_sds.sh
9fdf778e61497aa31f57fffe0eb52f8276f85fbe
link
/test istio_auth_sds_e2e

prow/e2e-simpleTests-cni.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e_pilotv2_auth_sds.sh
9fdf778e61497aa31f57fffe0eb52f8276f85fbe
link
/test istio_auth_sds_e2e

prow/e2e-simpleTests-cni.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test istio-pilot-multicluster-e2e

prow/e2e_pilotv2_auth_sds.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test istio_auth_sds_e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test istio-pilot-multicluster-e2e

prow/e2e_pilotv2_auth_sds.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test istio_auth_sds_e2e

prow/e2e-simpleTests-cni.sh
fb4ed4cc03ee8397e27d07900b928fddcecf0d65
link
/test e2e-simpleTests-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test istio-pilot-multicluster-e2e

prow/e2e_pilotv2_auth_sds.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test istio_auth_sds_e2e

prow/e2e-simpleTests-cni.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test e2e-simpleTests-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test istio-pilot-multicluster-e2e

prow/e2e_pilotv2_auth_sds.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test istio_auth_sds_e2e

prow/e2e-simpleTests-cni.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
4ee3603245f6cb417f8c18a8c9e9214e08973c55
link
/test istio-pilot-multicluster-e2e

prow/e2e-simpleTests-cni.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio_auth_sds_e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wenchenglu please help to merge, this is for DD, thanks

/hold for more testing

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: quanjielin, wattli
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: rshriram
If they are not already assigned, you can assign the PR to them by writing /assign @rshriram in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@wattli: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio-pilot-multicluster-e2e

prow/istio-presubmit.sh
185b928e87568621a2d8c1441c620faa72326a3e
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio-pilot-multicluster-e2e

prow/istio-presubmit.sh
185b928e87568621a2d8c1441c620faa72326a3e
link
/test istio-presubmit

prow/istio-unit-tests.sh
185b928e87568621a2d8c1441c620faa72326a3e
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio-pilot-multicluster-e2e

prow/istio-presubmit.sh
185b928e87568621a2d8c1441c620faa72326a3e
link
/test istio-presubmit

prow/istio-unit-tests.sh
185b928e87568621a2d8c1441c620faa72326a3e
link
/test istio-unit-tests

prow/istio-integ-local-tests.sh
185b928e87568621a2d8c1441c620faa72326a3e
link
/test istio-integ-local-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test e2e-simpleTests-cni

prow/e2e_pilotv2_auth_sds.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio-pilot-multicluster-e2e

prow/istio-presubmit.sh
185b928e87568621a2d8c1441c620faa72326a3e
link
/test istio-presubmit

prow/istio-unit-tests.sh
185b928e87568621a2d8c1441c620faa72326a3e
link
/test istio-unit-tests

prow/istio-integ-local-tests.sh
185b928e87568621a2d8c1441c620faa72326a3e
link
/test istio-integ-local-tests

prow/istio-integ-k8s-tests.sh
185b928e87568621a2d8c1441c620faa72326a3e
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test e2e-simpleTests-cni

prow/e2e_pilotv2_auth_sds.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio-pilot-multicluster-e2e

prow/istio-presubmit.sh
185b928e87568621a2d8c1441c620faa72326a3e
link
/test istio-presubmit

prow/istio-unit-tests.sh
185b928e87568621a2d8c1441c620faa72326a3e
link
/test istio-unit-tests

prow/istio-integ-local-tests.sh
185b928e87568621a2d8c1441c620faa72326a3e
link
/test istio-integ-local-tests

prow/istio-integ-k8s-tests.sh
d846e2ae8266d2de9f03f3197212ace26a490e7a
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test e2e-simpleTests-cni

prow/e2e_pilotv2_auth_sds.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio-pilot-multicluster-e2e

prow/istio-presubmit.sh
185b928e87568621a2d8c1441c620faa72326a3e
link
/test istio-presubmit

prow/istio-integ-local-tests.sh
185b928e87568621a2d8c1441c620faa72326a3e
link
/test istio-integ-local-tests

prow/istio-integ-k8s-tests.sh
d846e2ae8266d2de9f03f3197212ace26a490e7a
link
/test istio-integ-k8s-tests

prow/istio-unit-tests.sh
d846e2ae8266d2de9f03f3197212ace26a490e7a
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test e2e-simpleTests-cni

prow/e2e_pilotv2_auth_sds.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio-pilot-multicluster-e2e

prow/istio-presubmit.sh
185b928e87568621a2d8c1441c620faa72326a3e
link
/test istio-presubmit

prow/istio-integ-k8s-tests.sh
d846e2ae8266d2de9f03f3197212ace26a490e7a
link
/test istio-integ-k8s-tests

prow/istio-unit-tests.sh
d846e2ae8266d2de9f03f3197212ace26a490e7a
link
/test istio-unit-tests

prow/istio-integ-local-tests.sh
d846e2ae8266d2de9f03f3197212ace26a490e7a
link
/test istio-integ-local-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test e2e-simpleTests-cni

prow/e2e_pilotv2_auth_sds.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
fbb82840499d04b9183e094e6d73d873b51cfb4f
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
d846e2ae8266d2de9f03f3197212ace26a490e7a
link
/test istio-integ-k8s-tests

prow/istio-unit-tests.sh
d846e2ae8266d2de9f03f3197212ace26a490e7a
link
/test istio-unit-tests

prow/istio-integ-local-tests.sh
d846e2ae8266d2de9f03f3197212ace26a490e7a
link
/test istio-integ-local-tests

prow/istio-presubmit.sh
d846e2ae8266d2de9f03f3197212ace26a490e7a
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wattli this missed 1.1.1 but whenever you're ready please let me know

@duderino When was 1.1.1 cut?

@duderino  thanks for your info. I will let you know when it's ready


Italic and bold doesn't work?

Very cool plugin. However, bold and italic bbcodes seem to not work. Example pictured. Version 1.0.2a, Godot 4.2

This seems to be an issue with the default Godot font. Even a simple RichTextLabel does not show bold/italic fonts.

Setting custom fonts seems to fix the issue:

I'll set a custom font in the editor DialogueBox by default.

Pushed an update in this commit. The demo dialogue box in the editor should show bold and italic fonts normally now. Used the Poppins font.


Get Elements instead of Get Element

Is your feature request related to a problem? Please describe.
Selectors can match multiple targets. Currently Get element just "randomly" returns the first match.
This might not be the correct thing.
Get Element is used so that further manipulations or validations can be made to that element. Selector might match 0, 1 or N elements.
Describe the solution you'd like
Replace Get Element with Get Elements that returns a list of matches. Also have assertion engine there for easy manipulation of that list such as Get Elements   selector   then   value[0]
Describe alternatives you've considered
Get Element might also raise an error when multiple matches.

Hi, in my opinion Get Elements should allow to return an empty list instead of raising "Could not find element with selector XXX within timeout." exception, just like SeleniumLibrary

This seems to already work. Although Bug defined here is a valid one. I'll edit the title.

@aaltat I think we need to handle the timeout.

I mean that it will with the exception waiting version wait until timeout

Yes, it will search elements until it finds at least it one or it will raise an exception in the node side. But I don't understand why waiting is bad?

I think get elements should be instant check

But better have this discussion on slack.

From dynamic behaviour point of view (such as crawling unknown site), it is a normal situation that elements are not found. So it also can’t take 10 seconds when ending to this kinds of pages..

Maybe I just need to set the timeout to 0.. 🤪

I agree if it should do check quickly,  users can set timeout on smaller value. What shall we do with the issue?

I think you fixed it, I just kind of forgot that timeout setting is an option.


double quotes misplaced

@rultor release, tag is 0.3.3

@rultor release, tag is 0.3.3

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.3.3

@yegor256 Done! FYI, the full log is here (took me 10min)


Binance.js unable to do market orders

Note: for support questions, please join our Discord server

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Action taken (what you did)

Changed Binance.js to use 'MARKET' buys instead of limit (I deleted the price and one other setting I forget but I checked the API -- sorry I'm mobile right now)
let reqData = {
symbol: this.pair,
side: tradeType.toUpperCase(),
type: 'MARKET',
timeInForce: 'GTC', // Good to cancel (I think, not really covered in docs, but is default)
quantity: amount,
//price: price,
//timestamp: new Date().getTime()
};

Expected result (what you hoped would happen)
it works

Actual result (unexpected outcome)
400 error and crash

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)
I would love to be able to market orders with binance! <3 halp

RECEIVED ERROR IN GEKKO<PHONE_NUMBER>993364

@mvanbosc did you ever find out how to make Market order instead of limit orders?

No sir but I did make a fast trading strategy that does something similar
On Tue, Jun 5, 2018 at 2:06 PM Jabbaxx<EMAIL_ADDRESS>wrote:

@mvanbosc https://github.com/mvanbosc did you ever find out how to make
Market order instead of limit orders?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/askmike/gekko/issues/1743#issuecomment-394825016, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFbCwnC7CbPT3N6saV0TmFTTShHesy9wks5t5tbQgaJpZM4RlrTx
.

OK. What that means ?

Have gekko replace orders every ms at the next order book position until
the order is filled. It also makes sure the price doesn’t have extreme
variance from when we said to buy.
On Tue, Jun 5, 2018 at 2:09 PM Jabbaxx<EMAIL_ADDRESS>wrote:

OK. What that means ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/askmike/gekko/issues/1743#issuecomment-394825823, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFbCwpnx_WqzQaxQvF-SZU1G15s1X2xJks5t5tdhgaJpZM4RlrTx
.

Sounds good ;) Is it somewhere to find public ?
Or do you want to share ?

No sorry not public
On Tue, Jun 5, 2018 at 2:13 PM Jabbaxx<EMAIL_ADDRESS>wrote:

Sounds good ;) Is it somewhere to find public ?
Or do you want to share ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/askmike/gekko/issues/1743#issuecomment-394826971, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFbCwrbE8-kZEm1_UYji95LwXD4ofhC1ks5t5thggaJpZM4RlrTx
.

Sad. Wanna share private ? ;)

$5k
On Tue, Jun 5, 2018 at 2:22 PM Jabbaxx<EMAIL_ADDRESS>wrote:

Sad. Wanna share private ? ;)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/askmike/gekko/issues/1743#issuecomment-394829568, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFbCwmt0nPw0eZIbpxDiu_j3CtdkqAtiks5t5tp7gaJpZM4RlrTx
.

It is always nice to have a dream ;)


Filters cause segfault

Any filter will cause toilet v0.3 to segfault. See screenshot. Same exact thing happens with current git.
 Github is potato-retarded. Here's the screenshot lol

Hi, it works fine on all my machines.
Can you provide more information like operating system and ideally a stacktrace?

Also, what is your version of libcaca?

OS: Gentoo
Kernel: 4.4.1-gentoo
libcaca version: 0.99_beta18-r2
I have tried rebuilding libcaca already. Didn't change a thing.
Here's the gdb stack trace:
┌──  ~  ──┐
└ $ gdb toilet
GNU gdb (Gentoo 7.10.1 vanilla) 7.10.1
Copyright (C) 2015 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later http://gnu.org/licenses/gpl.html
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "x86_64-pc-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
https://bugs.gentoo.org/.
Find the GDB manual and other documentation resources online at:
http://www.gnu.org/software/gdb/documentation/.
For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from toilet...(no debugging symbols found)...done.
(gdb) run --gay
Starting program: /usr/bin/toilet --gay
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib64/libthread_db.so.1".
Program received signal SIGSEGV, Segmentation fault.
0x00007ffff7b17b7d in cucul_free_canvas () from /usr/lib64/libcaca.so.0
(gdb) backtrace
#0  0x00007ffff7b17b7d in cucul_free_canvas () from /usr/lib64/libcaca.so.0
#1  0x00007ffff7b207fb in ?? () from /usr/lib64/libcaca.so.0
#2  0x00007ffff7b20b7a in cucul_canvas_set_figfont () from /usr/lib64/libcaca.so.0
#3  0x0000000000402642 in ?? ()
#4  0x0000000000401924 in ?? ()
#5  0x00007ffff7793790 in __libc_start_main () from /lib64/libc.so.6
#6  0x00000000004019d9 in ?? ()
(gdb)


Microsoft Learn Great ,But !

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
I started learning with the New Microsoft Learn and discovering different area, is really very nice interactive and let you learn more and more without get boring and especially let me take the challenge on Azure, but as a .Net /Xamarin developer I would like to find more content about other technologies than Azure such as.net core and if there is any system like challenges or a page where we find top learner depend on their collected points (like the old TechRewards program).
Thank you.

Thank you for your feedback! At Ignite 2018, we delivered role-based training and learning paths for Azure Developer, Azure Solution Architect, and Azure Administrator, as well as role-based training and learning paths for Power BI, PowerApps, Microsoft Flow, and Dynamics 365 roles. Post-Ignite, we will continue to build content beyond these areas and improve the overall experience with additional features than what you see today.

@h0s Thanks for your feedback. I added an internal work item to track it. Once we add it, will let you know.

I agree with @h0s that there are those of us looking forward to different content than Azure at some point.  I, too, am doing Xamarin and .NET Core development and would love how-tos & whys, etc. on building Xamarin apps as well as architecture choices. Also, understanding APIs, async/await use, best practices for ASP.NET Core Web API integration, etc.  C# in general.  Is this MSFT Learn initiative to replace MVA at some point?

@yunyunyunyun Assign this feedback to you since you are Learn PM.

So learn has been here for a while, MVA has been gone for some time now. There is another apprentice in our company now, and there's still most of the MVA content missing from MS Learn. I'm honestly giving people the youtube playlists that saved the MVA videos. Where's the content MSFT kept talking about?


Falcon unzip 0-phasing jobs failing at the last step

I run unzip and creates the reads folder then the 0-phasing folder. But only one of the jobs completed. It seems to progress to the very last step for each job in 0-phasing. for instance 000045F creates the uow-00 folder and in the proto folder I have regions.json, preads.fasta, preads.sam, ref.fa link.
Seems to be failing here:
python -m falcon_unzip.proto.main_augment_pb     --wd ./proto/     --ctg-id 000045F     --p-ctg ${falcon_asm_dir}/p_ctg.fa     --p-ctg-tiling-path ${falcon_asm_dir}/p_ctg_tiling_path     --a-ctg ${falcon_asm_dir}/a_ctg.fa     --a-ctg-tiling-path ${falcon_asm_dir}/a_ctg_tiling_path     --p-variant-fn get_phased_blocks/phased_variants     --preads-sam proto/preads.sam     --extracted-ctg-fasta ${unzip_dir}/reads/000045F/ref.fa     --rawread-bam ${ctg_aln_out}     --rid-phase-map rid_to_phase.true     --out-updated-rid-phase_map rid_to_phase.tmp
mv rid_to_phase.tmp rid_to_phase

The IntervalTree problem should be solved in the latest release. conda update --all.


Time Planning & First Release

Hey @bhoov and @VladimirFokow,
I think we're close to a first release of Manim Notebook 🙌 So maybe before Christmas this year?

Just for my own time planning: could you give a rough estimate by when you will be able to review my current pull requests? And a time slot (maybe during mid-December) where you could spare some time to review even more PRs that will happen close to the first release? If you don't have this time, would it be ok if I do my own reviews as best as possible?
Also asking because with more PRs that pile up, it's getting more difficult to avoid annoying merge conflicts, especially when I want to do some refactoring of the code...
But of course, don't want to put any pressure, this is still a fun hobby-project, so no need to rush anything. Just want to avoid what happens to many of my projects: that I lose track and abandon them on my hard drive, just to realize later they were maybe almost finished but didn't see the light of the world because of my perfectionism and 80/20 pareto...

With regards to the advertisement video in #37, I don't think I will be able to pull this off this month, but I don't think a first release has to depend on it. We could release v1.0.0 this month and maybe do the big social campaign end of January or February next year. This would have the advantage that we are able to fix the teething troubles before the extension is more well-known. And before the video, we can still showcase the features in the Readme.

With regards to testing (#47): I haven't yet looked into it in detail, but wrote a short comment here. So for the release v1.0.0, what do you think about writing some unit tests and getting the non-trivial integration tests up to work later on? At least we also have a "Record log file" mechanism such that we should be able to identify user bugs quickly.

Thank you for understanding!
I am totally ok with you pushing on your own, but of course I agree, reviews are also important,
(Currently don't have much time before 2025)..
Hopefully, bhoov doesn't mind either

So maybe before Christmas this year?

And with this I was way too optimistic. I myself don't even have the time to get the first version shipped before Christmas 😅 But hopefully end of January is more realistic.
This also has the advantage that maybe 3b1b will have published a new Manim release by then which includes my new focus() and reload() commands as well as the --autoreload command line flag. Then we can point to a released version and don't have to say: look at that commit from the Manim master branch ;)

Hi @Splines, I am fully in favor of you making executive choices and aggressively merging your PRs in name of speed, to push getting the extension ready. I believe we have built a foundation of collaboration and slower/more professional coding that we can reinstate to fix bugs after release.
This said, I'm hoping to be a bit more active on the PR reviews in the next week or so before I take holiday. Hopefully I can help quickly debug the features you introduce!
Regarding timeline: yes, let's push before Christmas. We can make a soft announcement, maybe a few hobbyists will play around over the holidays, but we can make a bigger announcement in Jan when the demo video is done (it's ok if it is simple!) and the initial release kinks are worked out. And if we aren't ready by christmas, that is also fine!

Re: testing. I'm a bit daunted by the implementation, having not built a testing framework for an extension before and being this far into the project, but if it is easy to write some example unit tests and maybe one integration test for a template to build on, that's probably sufficient until Jan/feb next year. The log file is likely the most helpful of the debugging
In some way, we will be handicapped by the pace of development on manimgl, where there are no real tests, documentation, and breaking changes to this extension can be made on the whim of 3b1b 😂. I think we have to embrace the spirit of this dependency, and be willing to develop and fix bugs as we go...


Minor style adjustment

Post merge QA

[x] 1. Fix client intro style
before:

after:

[x] 2. Add underline to navBar hover
e.g.,

[x] 3. Alphabetical order clients in navbar

Thanks @hwwhww! I had some concerns about the cards on smaller screen sizes, which is why I went with the wrapped design.
I was just playing around with this solution:

What do you think? Happy to go with whatever you prefer :)

Also, we may want to adjust the cards in a slightly different way as I think they will overlap the borders on a smaller screensize

@georgeweiler
your solution looks great! 👍
To be clear, my front-end skill is miserable. Looking forward to your follow-up PR. :)


Fix Bug class Locale not found

This error is triggered when you want to access the route /canvas after connection.

I don't want to simply remove the name, I'd like to parse out the actual translation display name and avoid a list of locale abbreviations.


Fix pry v0.14 compatibility

See https://github.com/deivid-rodriguez/pry-byebug/pull/386

Hmm, still getting:
  1) Pry::Shell has a version number
     Failure/Error: @pry = Pry::REPL.start_without_pry_byebug(frame._binding, Pry::Shell.active_shell_options)

     ArgumentError:
       wrong number of arguments (given 2, expected 1)
     # ./lib/pry/shell/patches/pry_byebug.rb:41:in `start_new_pry_session'
     # ./lib/pry/shell/patches/pry_byebug.rb:28:in `block in resume_pry'
     # ./lib/pry/shell/patches/pry_byebug.rb:27:in `resume_pry'
     # ./spec/pry/shell_spec.rb:6:in `block (2 levels) in <top (required)>'

Still trying to get this to work. @meinac Can you take a look?

Thanks for working on this @stanhu! I've merged #15, could you please check if that fixed the issue?

@meinac I'm not sure if I'm doing this right, but pry-shell doesn't seem to show any available sessions.

I'm not sure if I'm doing this right, but pry-shell doesn't seem to show any available sessions.

Are you setting the breakpoint with binding.pry_sheel?

Closing as fixed by #15. Thanks @stanhu!


Pylance can't recognize this type, but it's standard in Python 3

from django.test.client import Client

def a(client: 'django.test.client.Client'):
    client.get('/')
    return 1

def b(client: 'Client'):
    client.get('/')
    return 1

def c(client: Client):
    client.get('/')
    return 1

In the function a it must recognize the type of client, it's useful with Pytest
It's using:

Python 3.11.5
Linux LAPTOP-IKLDLJHK <IP_ADDRESS>-microsoft-standard-WSL2 #1 SMP Fri Jan 27 02:56:13 UTC 2023 x86_64 GNU/Linux (Archlinux)

Version: 1.83.0-insider (user setup)
Commit: 4268e464763087044d0c1b5bdd37ebbe683cadfa
Date: 2023-09-29T20:42:50.630Z
Electron: 25.8.4
ElectronBuildId: 24154031
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621

Pylance is correct in flagging this as an error. There is no symbol django in the global namespace of your source file. If you want to use the symbol django in your code, you need to add an import statement import django.test.client.
You have imported a value for the symbol Client, so you can use that in a type annotation (as you have done in function b and c).


[Blazor][Webassembly] CORS sample

This is an area of confusion for people, so it will be nice to have a sample we can point people to that showcases how to achieve this scenario.
This is a tracking issue for us to do the work here to provide such sample.

We sort of do have this. It's in the common sample app and there's a brief section. Perhaps, we'll need to expand the section. However, keep in mind that CORS is a very known thing with a set of standard behaviors that the cross-linked content in our ASP.NET Core CORS docs covers in detail.
https://docs.microsoft.com/en-us/aspnet/core/blazor/call-web-api?view=aspnetcore-3.1#cross-origin-resource-sharing-cors

There are a bunch fo Blazor specific things here, like configuring the HttpClient instance and is painful in general to setup CORS, the CORS doc describes several ways to do it and I think we should have one specific straightforward way to do it with Blazor.
I'll elaborate on this once I get time to look into the area.


Use React 0.14

Use React 0.14 as peer dependency, if no modification is required

Updated to react 0.14


option-getter is called many many times

The option-getter callback is not called once when the user choose the option, but many times both when opening modal and when selecting.
This issue is present in the codepen sample, too.

The option-getter is called to determine the value associated with an option, in order to add the css class to the sected option.
It's not to be considered a callback but a way to get the model value from the option object.


ARC3 - Non normative IPFS best practices

For images stored on IPFS the files may be unpinned at any time. If a user has not saved the media locally, they may lose access to it.
We should add a non-normative comment to suggest that platforms warn users about the possibility and recommend downloading the image immediately to prevent losing access.
There may be other recommendations that make sense as well

@barnjamin - I am wondering if this belongs more as a best practices/collector onboarding README/FAQ etc.
I have been putting a lot of thought into the archivability of NFT/the files in which they point to, and how this reflects the legitimacy of the movement within institutions/museums (and if this even really matters to most marketplaces/artist/collectors?). In addition  considering the argument that NFTs have “solved” a long-standing problem of uniqueness and provenance simply ignores the fact that they rely on forms of trust, negotiation and authentication blah blah blah as @ChrisAntaki so correctly pointed out.
I view the belief/concept that everything is temporal (because it is) in a few ways in the context of NFT best practices:

marketplaces store a central backup of the IPFS for the collectors which they offer as a sort of hot storage of the works
artist/auction houses have provided an encrypted drive or thumb drive with the files/source codes. Including a COA which is physical but points to the ASA hash, in addition to other conditions (such as how an artist would like to displayed, what to do if the file is corrupt or the IPFS goes down etc)

Might something like this be more of interest?

best practices/collector onboarding README/FAQ etc.

This makes sense, where do you think this lives though? Each marketplace links off to something or some extra thing here?
Good points to add to the document and should probably allow additions for other storage options like Arweave


Think of a better name...?

PHP-Compiler seems too blah. Perhaps rename to PHPPHP? Or something else more creative?

How about PPHP?
PHP PHP Hypertext Preprocessor

phpc? (I'm not very good at naming either.)

PHP²

PAC - PHP AOT Compiler

PHC - PHP HyperCompiler

ComPHiler?

PHPHP

PHPHP  +1

phpc +1

Or why not a name which is not directly related to the  scope of the project (and then avoid strange strings with a lot of P and H :yum:)...
I'm proposing Elephant with a little slogan like The PHP Compiler.
It sounds more elegant to me (like other names, Laravel or Composer to list a few).
Beside this proposition, +1 for ComPHiler by @gren236

@na2axl I agree that, even though phpc reminds me of the old days (e.g. gcc) and is therefore a nice one, it lacks however... Branding. Elephant is clean and winks away at the PHP logo. It's a good find!

I want to make an argument against "Elephant".
We want to make the compiled php as our target. This will save a lot of resources (CO2 and Energy) if we push "php compiled" as a brand.
The future of this project should be that the Wordpress project makes a Issue "Lets go php compiled"


Added checks in BMI calculator

Fixes Issue🛠

Closes #1368
Description👨💻

Added general limits and checks for age groups.
Type of Change📄

[x] Bug fix (non-breaking change which fixes a bug)

Checklist✅

[x] I am an Open Source contributor
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project
[x] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

The bug for which the issue is opened is not solved, go through the issue description properly @kauxp


eclipse-sumo: Add version 1.7.0

Hi, I would like to add a new softeare sumo to the extras.

There is another error which is "Can't shim 'bin\sumo-gui.exe': File doesn't exist."  I think probabaly it is because of the install directory is apps\eclipse-sumo\1.7.0\sumo-1.7.0 rather than apps\eclipse-sumo\1.7.0. I am not sure how to solve this.

There is another error which is "Can't shim 'bin\sumo-gui.exe': File doesn't exist." I think probabaly it is because of the install directory which is apps\eclipse-sumo\1.7.0\sumo-1.7.0 rather than apps\eclipse-sumo\1.7.0. I am not sure how to solve this.

Use extract_dir. Here's an example:
https://github.com/ScoopInstaller/Main/blob/0922e66d6e2a730b1b9dac6f75f1932b2f9f62dc/bucket/ccat.json#L6-L31

Why there is file bin\start-command-line.bat??
There is number of executables with command line support. Are there meant to be internally used or by end user?

Hi, yes, these command line tools in the folder "bin" need to be exported to users. That is why the document of sumo suggest adding that folder to the path. The file "bin\start-command-line.bat" is used to set path and environment variable.

/verify


Migrate Commands from Prefix to Slash

Slash commands make your bot easier to use and work as an automatic help menu for your bot. Adding Slash Commands, will gain your bot more reach, which may also result in more contributors. Additionally, verified Discord Bots also require Slash Commands.
If you want, I am free to work on this issue.
Regards,
Zemerik

ok? :DD


Question: Is input user customizable?

Before forking and hacking on this myself, I have a requirement for the input to contain a calendar icon along with the normal entry input, so basically a span that tacks on a small icon to the right of the input.
Any current available hook to accomplish this?

See the customInput prop...
We're using a button with a small calendar icon to the right of the input, rendered outside react-datepicker. Eg: https://wcca.wicourts.gov/advanced.html We're only using customInput for the sake of aria-describedby.

Huh... well that's an approach I hadn't considered. Thanks aij.

@aij How are you triggering the calendar to pop when you click on the added button?

@kkras3 There is a customInput={} property you can set with your custom input component. In there you can add multiple elements, so in my case I just focus on the input on the click event of a button that is within that component. That'll open the datepicker automatically.

Thanks, yep, found a solution by hunting around the issue items.


Initial files structure

Initial file structure.

@austinkelleher I don't know why, but I can't reply your comments :s

It probably would not a bad idea to allow https cert options to be passed as CLI args. If they're provided, use https otherwise use http

I like the idea.

I'm actually leaning towards this being a bad idea. If the port is in use, I feel that the user should know as it's probably not intended. What do you think?

I copy that from a previous project where we need to launch some servers at the same time (different type of tests and different browsers at the same time). Maybe in this project it doesn't make sense (at least for now). I'm going to remove it!

@austinkelleher done!

@molant Done!
Anyway, @molant @austinkelleher remember this PR is just for the file structure, I'm working on the job manager in another branch and some things are changing. What I mean is that right know, the code is not important, just the file structure.

@sarvaje Should this be closed in favor of #2?

@sarvaje Should this be closed in favor of #2?

That is an option, the other option is merge this one if it is ok hehehehe


A GPT companion for the Zod to Valibot conversion

I wanted to migrate to Valibot a while back but didn't have much time, especially to handle transforms or refines.
I now put quickly together a GPT companion which hopefully can get people going faster and help them bridge that gap.
Far from perfect, so happy to address feedback and fix this in that GPT.

That's great! Thank you very much. Is it only available with GPT Plus? Once the output is correct in most cases, we could mention in the repo or docs.

@fabian-hiller  those custom gpts  only work with GPT Plus

Thank you for the information!

I will close this issue for now.


Update SVViewController.m

Hi I merged #240 code to https://github.com/LawrenceHan/SVPullToRefresh
It will solve your problem.


Windows 10 IoT Core Can't Update Time Automatically After Rebooting

We found an issue after rebooting the Windows 10 IoT Core device, the date and time can't be updated automatically.

Please file this bug using the steps listed here so we can repro this issue. Thank you for your help!


tuf: failed to decode timestamp.json: expired at 2017-07-18 06:00:03 +0000 UTC

I am receiving the following error this morning when attempting to do a flynn install.  It seems that the timestamp.json has expired and needs to be updated.
===> 08:54:10.861 installing ZFS
===> 08:54:10.863 running "apt-get update"
Hit:1 https://repos.sonar.digitalocean.com/apt main InRelease
Hit:2 http://sfo1.mirrors.digitalocean.com/ubuntu xenial InRelease
Hit:3 http://sfo1.mirrors.digitalocean.com/ubuntu xenial-updates InRelease
Get:4 http://security.ubuntu.com/ubuntu xenial-security InRelease [102 kB]
Hit:5 http://sfo1.mirrors.digitalocean.com/ubuntu xenial-backports InRelease
Fetched 102 kB in 0s (132 kB/s)
Reading package lists... Done
===> 08:54:12.654 installing runtime dependencies
===> 08:54:12.655 running "apt-get install --yes iptables zfsutils-linux ntp"
Reading package lists... Done
Building dependency tree
Reading state information... Done
iptables is already the newest version (1.6.0-2ubuntu3).
ntp is already the newest version (1:4.2.8p4+dfsg-3ubuntu5.5).
zfsutils-linux is already the newest version (<IP_ADDRESS>-0ubuntu17).
0 upgraded, 0 newly installed, 0 to remove and 46 not upgraded.
===> 08:54:12.921 loading zfs kernel module
===> 08:54:12.922 running "modprobe zfs"
===> 08:54:12.924 downloading flynn-host binary to tmp dir
===> 08:54:13.204 verifying flynn-host binary checksum
===> 08:54:13.239 running "gunzip flynn-host.gz"
===> 08:54:13.400 running "chmod +x flynn-host"
===> 08:54:13.402 setting release update channel to "stable"
===> 08:54:13.404 downloading Flynn components
===> 08:54:13.406 running "./flynn-host download --repository https://dl.flynn.io/tuf --tuf-db /etc/flynn/tuf.db --config-dir /etc/flynn --bin-dir /usr/local/bin"
INFO[07-18|08:54:13] initializing ZFS volumes
DBUG[07-18|08:54:13] zfs get all -Hp flynn-default            package=zfs
DBUG[07-18|08:54:13] zfs get all -Hp flynn-default/data       package=zfs
DBUG[07-18|08:54:13] zfs get all -Hp flynn-default/squashfs   package=zfs
DBUG[07-18|08:54:13] zfs get all -Hp flynn-default/ext2       package=zfs
INFO[07-18|08:54:13] initializing TUF client
EROR[07-18|08:54:13] error updating TUF client                err="tuf: failed to decode timestamp.json: expired at 2017-07-18 06:00:03 +0000 UTC"
08:54:13.583270 host.go:166: tuf: failed to decode timestamp.json: expired at 2017-07-18 06:00:03 +0000 UTC
===> 08:54:13.586 ERROR: failed to run "./flynn-host download --repository https://dl.flynn.io/tuf --tuf-db /etc/flynn/tuf.db --config-dir /etc/flynn --bin-dir /usr/local/bin", exit status 1```

This has been fixed, sorry about that!

Something has expired again...
===> 16:15:27.088 running "./flynn-host download --repository https://dl.flynn.io/tuf --tuf-db /etc/flynn/tuf.db --config-dir /etc/flynn --bin-dir /usr/local/bin"
INFO[11-21|16:15:27] initializing ZFS volumes
DBUG[11-21|16:15:27] zfs get all -Hp flynn-default            package=zfs
DBUG[11-21|16:15:27] zfs get all -Hp flynn-default/data       package=zfs
DBUG[11-21|16:15:27] zfs get all -Hp flynn-default/squashfs   package=zfs
DBUG[11-21|16:15:27] zfs get all -Hp flynn-default/ext2       package=zfs
INFO[11-21|16:15:27] initializing TUF client
EROR[11-21|16:15:27] error updating TUF client                err="tuf: failed to decode timestamp.json: expired at 2017-11-21 12:00:03 +0000 UTC"
16:15:27.307879 host.go:166: tuf: failed to decode timestamp.json: expired at 2017-11-21 12:00:03 +0000 UTC
===> 16:15:27.312 ERROR: failed to run "./flynn-host download --repository https://dl.flynn.io/tuf --tuf-db /etc/flynn/tuf.db --config-dir /etc/flynn --bin-dir /usr/local/bin", exit status 1


Upgrade to eslint v4

http://eslint.org/blog/2017/06/eslint-v4.0.0-released
I don't know if it will take much work to upgrade.

I always wait until at least x.x.1 so I don't have to deal with regressions. I'll keep this open until we upgrade.

@sindresorhus
4.1.1 is out now as the first x.x.1 since 4.0.0 :)
http://eslint.org/blog/2017/06/eslint-v4.1.1-released

They have a bunch of regressions with the rewritten indent rule. I prefer to wait a little bit more.

Just to add my two-pence, the Snyk is reporting XO as having "High severity" vulnerabilities which were solved since v4 ESLint; the latest ESLint comes clean.

I know being a dev dependency it's not a big deal, but badges of all code quality measurement services and Snyk itself look nasty currently.

The latest eslint-config-xo now relies on eslint@4.
Trying to use xo with the latest eslint-config-xo, I get:
Error: /projectDir/node_modules/eslint-config-xo/index.js:
        Configuration for rule "no-else-return" is invalid:
        Value "[object Object]" has more items than allowed.

upgrading xo to eslint@4 fixed the problem.

The ESLint upgrade was done in https://github.com/sindresorhus/xo/commit/a11e288d7227aa55db6df130d42f3593f8474ded. I plan to do a new release soon.


fix autologout and add unit testing

Link to Issue

The previous code is incapable of detecting user behavior (mouse, keyboard) and resetting the timer to avoid auto logout.
The previous code had no unit test.

Description of changes

AutoLogout:

Reduces the idle time limit from 2 hours to 50 minutes and the time from warning to logout from 1 minute to 10 minutes.
Additionally, the time before the component is considered asleep has been reduced from 15 seconds to 5 seconds.

Unit testing

Simulate mouse and keyboard activity and check that the warning modal is displayed after 50 minutes of inactivity and the user is logged out after 10 minutes.
Check that user activity prevents the warning modal from being displayed.

Screenshot(s) or GIF(s) of changes

wrong version of PR


allow snippets to run regardless of language mode

or maybe just add markdown? I don’t know the syntax, do I duplicate the entry or is there an “OR” syntax?

Its just JSON if you wanna target multi language support you'll have to dig into the VSCode extension docs. Its doable and really should be pretty easy -- Time is just a little hard to come by for myself.
Here is the → API docs

@Haroenv if you wanna clone the branch feature/#7-markdown-support to you .vscode/extensions path and try it out. Snippets should work inside of markdown files.


[alertmanager-bot] bump version

Description of the change

Benefits
https://github.com/metalmatze/alertmanager-bot/releases/tag/0.4.3

Possible drawbacks

Applicable issues

fixes #

Additional information

Checklist

[ ] Chart version bumped in Chart.yaml according to semver.
[ ] Title of the PR starts with chart name (e.g. [home-assistant])
[ ] Variables are documented in the README.md (this can be done with using our helm-docs wrapper ./hack/gen-helm-docs.sh stable <chart>)

Thanks for the PR! Can you also update the changelog and README files?

@bjw-s I have't seen anything in the changelog or readme... aren't these files autogenerated anyway?

Currently no, https://github.com/k8s-at-home/charts/issues/665
It's explained in the PR issue template how to run helm docs manually.


Step 1.4 of new DAR form

DUOS-1753

Created a new Collaborator component with Row, Summary, Form, and Error components
CollaboratorList is the entry point, where an existing list of collaborators that are passed in the DAR form are mapped to the CollaboratorRow component.
The CollaboratorRow component has logic to show the Summary / Form components based on a boolean.
The CollaboratorForm component uses FormField components for the collaborator field entries.
CollaboratorErrors does some basic validation on the text fields and validation of email input using a regex check.

Would be nice to add:

Styling changes to gray out the Add / Save button when the values entered are not valid.
Hover color for buttons?
I feel like the styles for my new components are all over the place - some reference index.css, some have the styles explicitly stated - wondering what the best course of action would be to standardize those settings.

Have you read Terra's Contributing Guide lately? If not, do that first.

Label PR with a Jira ticket number and include a link to the ticket
Label PR with a security risk modifier [no, low, medium, high]
PR describes scope of changes
Get a minimum of one thumbs worth of review, preferably two if enough team members are available
Get PO sign-off for all non-trivial UI or workflow changes
Verify all tests go green
Test this change deployed correctly and works on dev environment after deployment

There is a bug where an existing download/approval status is not being rendered in the current form.
Old:

New:

Fixed the bug where an existing Download/Approval status was not being displayed. This is because the old form was using booleans to track that option whereas my FormField implementation uses "yes" and "no" radio groups.
https://github.com/DataBiosphere/duos-ui/pull/1839/files#diff-5773e5fffe2ec216dc0218a0730c867178c942863cf2e2a64e506deb82a91e5eR117-R119


Inconsistency "username" in request != "name" in JWT

The Quid API expects "username" within a request body JSON:
https://github.com/synw/quid/blob/master/quidlib/server/api/token.go#L223
But the Quid JWTs contain "name" for the same meaning:
https://github.com/synw/quid/blob/master/quidlib/tokens/claims.go
The database tables also use "name":
https://github.com/synw/quid/blob/master/quidlib/server/models.go

We probably should use `username" all over the place as it is more specific

Hi @synw
What do you think to use user instead of username?
The request for a new token/refresh using "user":
curl -X POST http://localhost:8082/token/refresh/10m          \
     -H 'Content-Type: application/json'                      \
     -d '{"namespace":"xxx", "user":"xxx", "password":"xxx"}'

The content of a token (JWT) with "user":
{
    "namespace": "xxx",
    "user": "xxx",
    "groups": ["xx1", "xx2"],
    "exp":<PHONE_NUMBER>
}

What do you think to use user instead of username?

I used username for all, so it is consistent now. I prefer username than user, it is more explicit


Next

Next Este WIP: https://github.com/este/este/tree/next

[ ] Dynamic routing
[ ] Don't show eslint errors as overlay. CRA does it somehow. Until fix, eslint-loader is disabled in webpack for dev.
[ ] Intl. Blocked by React Fiber and react-intl plain string support update.
[ ] React Native
[ ] Jest
[ ] Favicon generator. Do we still need gulp?]
[ ] Flow by default, but .flowconfig all=true doesn't work yet (infinite run).
[ ] Consider ramda-fantasy and ramda-fantasy-validation.
[ ] Port the rest of the Este...

will you consider adding react-navigation?

@steida Love your work and Thanks for considering next, it should considerably reduce the DOMContentLoaded time with its automatic code-splitting feature. Also please consider adding webpack-bundle-analyzer and analyzing from the beginning so that we know what modules make up the most of the bundle size and try to optimize it.

I will. Thank you for your tip.

@ravikiran438 If you want to help me, figure out how to add eslint for development, but without error overlay and with console output only. Just like CRA does it. For now, eslint is disabled for dev, and checked only during build. Thank you :-)

@steida .. Sure I will let you know once I have something to send a PR.

Almost done.


Inference Graph SDK

/kind feature
Describe the solution you'd like
Extend the existing kserve python SDK to support latest inference graph API.
Anything else you would like to add:
@yuzisun will followup with instructions on how to contribute

@vvnpan You can follow the sdk generation instruction to create the inference graph python API and then we can add an e2e test for the inference graph like here.

Thanks @yuzisun for the doc, I followed the doc and updated the api using the scripts, I think some apis didn't get generated. The InferenceGraph API references V1alpha1InferenceGraphSpec and V1alpha1InferenceGraphStatus but neither were generated
I'm testing the the new API, but I need to wrap the inferencegraph API to be created with the kubernetes client.api_instance.create_namespaced_custom_object(), should I update the kserve client logic to include inferencegraph?


Remove urban_runoff from Basin

This takes care of the removing urban_runoff part of #1071. Technically breaking if people had this unused column in their Basin tables.

Shouldn't e.g. the MetaSWAP runoff be received in a runoff column?


Update PRIMER.md

Without testing my changes, I am pretty sure there is a mistake in the documentation.

LGTM, thx.


cargo fmt no longer accepts --write-mode overwrite after --

cargo fmt -- --write-mode overwrite
Invalid argument: `overwrite`.
usage: cargo fmt [options]

Options:
    -h, --help          show this message
    -q, --quiet         no output printed to stdout
    -v, --verbose       use verbose output
    -p, --package <package>
                        specify package to format (only usable in workspaces)
        --version       print rustfmt version and exit
        --all           format all packages (only usable in workspaces)

This utility formats all bin and lib files of the current crate using rustfmt. Arguments after `--` are passed to rustfmt.

This is preventing rustfmt from working in the Playground.

Can reproduce outside of the playground:
$ cargo +nightly fmt -- --version
0.3.8-nightly (346238f4 2018-02-04)

$ cargo +nightly fmt -- --write-mode overwrite
Invalid argument: `overwrite`.
usage: cargo fmt [options]

$ rustup run nightly -- rustfmt --help | grep write
        --write-mode [replace|overwrite|display|plain|diff|coverage|checkstyle]

It appears that cargo +nightly fmt -- --write-mode=overwrite works, so perhaps something is trying to be overly clever about whitespace parsing?

This is https://github.com/rust-lang/cargo/issues/5208

So this should be fixed now? (Or at least when the Cargo change hits nightly). Please comment if not.


New feature

Changed the control of the game using visual feedback. The dino now jumps when you move your nose up and ducks when you move your nose down. To start the game, you still have to press , and  to exit.

Hi
Thanks a lot for your interest in this project. Apparently, I am no longer maintaining this code, hence won't be able to accept any new features or pull requests.
You are, however, free to fork this project maintain it as your own.


ABN improvements

Hi,
I have few sugestions to change ABN behaviour (if it would be possible):

After each mined block it creates ABN tx, what is good, but after each tx there remains cumulative ABN tx and <1BBP tx. I have tons of these small txs (https://prnt.sc/pbcrmg) which I think is not desirable. It would be nice if there will be only one cumulative tx.
Change ABN tx behaviour to same as GSC tx. To divide it to smaller bankrolls, eg. with changequantity or to exact eg. 125000 BBP bankrolls (or smaller).
Change ANB tx creation. I know that it still starts with coins with lowest coinage, but sometimes it's not "good". Eg.: you have 1M BBP bankroll with weight 60k, second bankroll 1M BBP with 60k weight and 125k BBP bankroll with 180k weight. Current algorhitm take all of your bankrolls and combine it to one ABN tx with 2,25M BBP and 300k weight even if it will be enough to take bankroll with 125k BBP.

Take it just a suggestions what could help someone (someone like me :)). Thanks.

3. Current algorhitm take all of your bankrolls and combine it to one ABN tx with
2,25M BBP and 300k weight even if it will be enough to take bankroll with 125k BBP.

I commented same thing two months ago, but dev made no comment about it. The behavior I noticed back then, like you said was because of same input amount. I haven't tested with latest version, but this may already be fixed. Is it still same with 1447o?
https://github.com/biblepay/biblepay-evolution/commit/6d76ad8222962d35c1b3af91911241615797cf4b#commitcomment-34325822

@sunk818 No, when you commented on the pasted commit, you inferred that we were using duplicate inputs (which is not correct because we dont duplicate inputs).  You also posted 6 other things (IE spam).  I recommend consolidating them into one issue and entering a github issue (not questions about things you are "speculating" in code comments).
Orbis is recommending that we sort the inputs by age first in # 3.  Which I believe we do - but maybe we dont in ABN's, Ill have to look at this issue when we get to this ticket.

Orbis is recommending that we sort the inputs by age first in # 3. Which I believe we do - but maybe we dont in ABN's, Ill have to look at this issue when we get to this ticket.

I'm sure that it doesn't work for me. It's enough to take a look through exec getabnweight a you'll see that it doesn't sorts it by age. It'll be best to take 1st package with 125k weight.

Is it still same with 1447o?

I'm still trying to have newest wallet version on all of my miners, but I'm using pre-compiled binaries and I'm not able to find out which version I have, because those "letters" version (1447a,b,c, ...) are not visible in getinfo.

I like how clear this tickets requirements are.

Thanks

The new version 1448 is almost out - but in the mean time, let me take a look at sorting for ABNs.

Hi Orbis,
So on # 1 and # 2, I believe we can do something there - modify ABN "change" handling to not create change < 1 bbp, and, provide multiple outputs for people who break big bills.
Moving on to # 3 however, I did a test yesterday with 'exec getabnweight 125000 1', and I see that we already do sort by coin-age before selecting the inputs.  We sort the entire wallet by coin-age ascending, then we select the least amount of coin age possible - in order.  Could you please do a re-test on # 3 and confirm that # 3 is not an issue?

Thanks.
Ok, now I have only one huge package, but I'll try to install 1148, make exec bankroll and test it. I'll post my results here.

I was thinking that I won't be able to reproduce it, because I've had a luck and hit two blocks :) , but here is my actual result of exec getabnweight 125000 1:
  "Command": "getabnweight",
  "version": 2.7,
  "weight":<PHONE_NUMBER>606482,
  "total_required": 2867711,
  "coin_age_data_pre_select": "999999.0200(0.03)=[32928.21] depth=11,  <ROW>1867712.3384(0.09)=[172849.83] depth=23,  <ROW><TOTALREQ>2867711.36</TOTALREQ><TOTALAGE>205778</TOTALAGE>",
  "weight 125000.00":<PHONE_NUMBER>606482,
  "total_required 125000.00": 2867711

You can see, that even if the second package would be enough ABN will use the first one "uselessly" instead of letting him mature.

@OrbisPictus Ok, you are referring to using the small input first and then the bigger input - because we told it to sort by coin-age ascending - which covers the biggest part of the use case (not wasting extremely large ages).  Take a situation where someone has ten 1 million coin age coins and one 200K coin-age coin.  It would sort these by coin-age ascending.  So it would use the 200K coin first, be satisfied and break out (and not waste the users large coin age).
In your case you are saying you dont like it using up the 32K first then the 172K because that spends 204K - and does not preserve the 32K.  But even if we used the 172K we would have wasted 50K anyway (by going over by 50K).
So it is designed like this for a reason - to use up the smallest coin age first before tapping into the bigger coin age.  Its not a good idea to make this so complicated that no one will be able to support the code.
Asking us to modify the "change" emitted to be more plentiful sounds like it will address this anyway.  If we give you change of an adjustable quanity (IE 25 tx's of change from an ABN, a broken bill will end up being more like 5K change * 25 quantity) meaning the code as is would sort by coin-age ascending and use up more of the small ones first, satisfying the requirement without tapping into your bigger ones.
So Ill keep looking into # 1 and # 2 very soon.

So, Im starting to look at implementing # 1  and #  2, and I have a question.  In my wallets case, I send ABNs occasionally into the pool, and because we sort coin-age ascending, my GSCs consume my .98cent (IE sub-1 bbp) transactions regularly, because the GSCs use up that small coin-age first.  So in my case I only have one .99 bbp in the whole wallet.  Question, what wallet version were you using that was piling up many sub 1 bbp tx?  Was this a very old wallet?  (As Im assuming this shouldnt be happening with the coin-age ascending rule).
Ill move on to # 2 in the mean time.

 I don't like it, but it's only my opinion :)
On #1: I'm using always newest versions of wallets. These sub 1BBP tx are creating in 1148 too.  After every hitted block. ABN tx is everytime divided to ABN weight consuming tx and sub 1 BBP.

I'm confused on # 3 though, you don't like it, but its there to protect your coin age (we use the smallest coin age first).  I explained how we went out of our way to sort the entire wallet by coin-age ascending before touching your coin age.  Are you saying you would rather we sort it, then scan it, then if we find one transaction we match it (out of order) then loop back and start over with ascending then loop again?
On # 1, could you please paste your 'exec getabnweight 125000 1' so I can see if you are including the sub 1 bbp tx in your getabnweight selector?  If you have all these in the wallet, they are supposed to be first in that resulting selection output?

On # 3: Yes, it's like you wrote.... My experience was that everytime I've made exec bankroll, my coins was  slit into one package in less than 6 hitted (by me) blocks. So I'm mining most of the time with one huge package. I was trying to made exec bankroll, then not to mine for one day (to reach coinage), but after 3 days I was on the same. SO, yes, my problem is that most of my coins was with coinage less than ABN requirement. But I hope, that it'll be better with 1149 and implemented changequantity (Thanks for that :) )
On # 1: I think it's not needed to paste exec getabnweight, because those sub 1 BBP txs are not included getabnselector (never). Now I have 35 of those txs. Time to time I sent them to my wallet.

On # 3, OK great, so 1149 should fix that as more quantity of change works its way through the system , OK good.
Now on # 1, these sub 1 transactions are included in ABNs or GSCs  (thats how they get consolidated and get used up).  The rule for GSCs/ABNs is :  Dont spend sanctuary funds, dont spend locked funds, dont spend more than 500 inputs, and Do sort by coin-age first, then pull them in.  So as long as you have less than 500 wallet transactions, you should be picking them up in the exec getabnweight 125000 1.  If you arent, just try -zapwallettxes=1 and restart (or rescan your wallet) and try again.  Or just spend them back to yourself once, and verify after that they do get included in ABNs.  They should be- so technically nothing to do on # 1, then we can close the ticket?

I've made -zapwallettxes=1 more than once before, but they've never been in my ABN tx. Nevermind. I'll send them to myself and they'll be gone :)
Thanks for all. I'm closing this ticket.

@biblepay those sub 1 BBP txs are still creating after hitted block in 1149: http://prnt.sc/pcyb04
but changequantity  is working 👍

Need clarification on changequantity. If we set changequantity=9 in biblepay.conf is this honored when change is created after an ABN transaction (a block is mined)?

On # 3, OK great, so 1149 should fix that as more quantity of change works its way through the system , OK good.

The default is 10? the changequantity entry in biblepay.conf does not affect how much change is generated after a mined block?

@OrbisPictus If change can be added to one of the outputs or allow <1 BBP be used as ABN input, then we won't have any small change.

@sunk818 I don't know where is problem, but all of my "small" balances are still included in ABN input except those sub 1 BBP. So I don't have any small change. Every new hited and aged block is included into ABN. So in my wallet are only those sub 1 BBP and bannkrolls from ABN. If those sub 1 BBP txs are expected I don't have problem to send them time to time to myself.
And yes, it works with changequantity=X in biblepay.conf

I don't think we have a rule, but, if its doing again for you, Ill check more thoroughly all the rules now.

Does it mean that this happens only to me? Has anyone else this problem?

I have a wallet that only POBH/ABN mines and I also get a lot of 0.99* amounts

I'm checking the rules to see how this can happen.

Is this irrelevant now that is ABN is turned off? I think we can all live with 0.99 change as output.

I agree. I think, it can be closed. Thanks.


Display a graph of components

It would be very useful to be able to display a graph of components as a dot file, showing dependencies and where singletons have been made, with various options to remove unnecessary information (display dependencies only for example).

dot file?

Yes, for example.

I'm not sure I know what a dot file is :|

Sorry :-). This is DOT.

@etorreborre thanks :D

Done.


overload for java void in scala workflow builder

Read then delete this section
- Make sure to use a concise title for the pull-request.
- Use #patch, #minor or #major in the pull-request title to bump the corresponding version. Otherwise, the patch version
will be bumped. More details
TL;DR
Please replace this text with a description of what this PR accomplishes.
Type

[ ] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[ ] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
How did you fix the bug, make the feature etc. Link to any design docs etc
Tracking Issue
Remove the 'fixes' keyword if there will be multiple PRs to fix the linked issue
fixes https://github.com/flyteorg/flyte/issues/
Follow-up issue
NA
OR
https://github.com/flyteorg/flyte/issues/

Closing since it's very out of date. Feel free to re-open if still relevant.