[multistage] performance and stability tracker

We've received many feedback from multi-stage engine beta test.
This issue tracks all the reported and fixed performance and stability issues

[ ] large data block cause grpc failure: https://github.com/apache/pinot/issues/9272
[ ] cancel stuck queries: https://github.com/apache/pinot/issues/9241
[ ] allow pruning on broker: https://github.com/apache/pinot/issues/9165
[ ] limit causes compilation error: https://github.com/apache/pinot/issues/9209

CC @siddharthteotia ^

Should we separately track resource / thread pool / execution scheduling issue or add it here ?
Without that, this is unlikely to work for a mixed workload deployment where few new (join, subquery) queries go down the new engine (via option) with 2 or more stages whereas existing prod queries continue to run in the current engine with exactly 2 stages.
For users who are purely interested in using the new engine, this is not that big of a problem.
But since there is still a long way to go before fully adopting multi-stage engine for all queries, we need to solve this otherwise it will hinder usage / adoption of new engine's functionality for existing production users.

i think we should track them together here. I already have some in mind


Setting timezone?

Hi,
First time using GitHub, hope this is the right place for this.
I can't find a way of setting a timezone on the datetime picker, can anyone point me in the right direction?
Thanks!

Hi,
I saw a closed issue about, this #31

Thanks! Sorry, I didn't think to view the closed issues - as I said, I'm new to GitHub.


Feature/david/serp header experiment hide it

Task/Issue URL: https://app.asana.com/0/1157893581871903/1175100132980403
Description:
This PR implements Option C of this Experiment variant:

Hide the SERP using the ko parameter
Ensure the Vertical is kept after requery
Fire Requery pixel indicating if the query has changed or not

Screenshots:

Clear Data
Clearn on

Steps to test this PR:
*** SERP Parameter ***

Open app
Enter any query in the Omnibar
Check that the SERP header disappears in every Vertical (Images, Videos, News)
Check that the SERP is visible in the Maps Vertical

*** Ensure Vertical is Kept ***

Open app
Enter any query (apples) in the Omnibar and tap on Images
Enter another query (pears)
Check that the Images vertical is still visible and filled with photos of pears

*** Fire Pixel ***

Open app
Enter any query in the Omnibar
Check that the "rq_1" Pixel is fired
Without changing the text in the Omnibar, force a requery
Check that the "rq_0" Pixel is fired

Internal references:
Software Engineering Expectations
Technical Design Template

Thanks for submitting this. Is this up-to-date with develop? If not, could you bring it up-to-date? You probably know this already but to avoid pain in case you don't - be careful to bring you main feature branch up-to-date with develop and then merge your main feature branch into this one. Feel free to ping if you're unsure.

@subsymbolic this PR is only about the specific experiment variant to hide the SERP. I was hoping to tackle any merge issues against develop once the Product Review is completed and a feature PR is open. Does that make sense to you?

Is this your last PR before the final one to merge the feature into develop?

This is the last PR with the agreed scope, but depending on what the Product Review concludes something from this or the other variant might have to change. Nonetheless, I'd like to track those changes in a separate task and create a PR with those changes should it be necessary.
For the purpose of this PR, I'd like to have this experiment variant reviewed and other changes can be reviewed in a separate PR.

After discussing with @subsymbolic we've decided to merge this PR and do a full review against develop once Product Review is completed.


Badges

How i can set badges more than 2 or 3 badges ?!
Please help me, I saw many pepople can set more than 3 badges
For example: I saw a client, he had 10 badges!!!

More then 3 badges only show for clients that have 3.1.7 or below


feat: add Drift auth connector

Closes #314
Checklist

[x] Ran Linter
[x] Catalog tests passing
[x] Created PR from non-main branch

Notes
Testing
GET
URL: localhost:4444/contacts

POST
URL: localhost:4444/contacts

PATCH
URL: localhost:4444/contacts/id

DELETE
URL: localhost:4444/contacts/id

Pagination
Please add screenshots that show successful pagination using the connector.

Can you please add a screenshot of the raw token (as you've included a TokenMetadataField) and also add in screenshots for pagination?


Implement a feature for managing staff and employee profiles.

we use accounts.User model instead of staff model


Fix [GithubManifest] and [GithubPackageJson] when static auth is not configured

This started as investigation work for #2719. I noticed that none of our Manifest or Package.json badges seemed to be working in production, not just @AdityaTD's specific example:

https://img.shields.io/github/manifest-json/v/RedSparr0w/IndieGala-Helper.svg
https://img.shields.io/github/package-json/v/badges/shields.svg

When we retrieve the file contents via the API, we get a JSON payload as described in the documentation. The content is embedded in one of the JSON fields, encoded in Base64. Our Joi schema expects to find the content and encoding fields in the API response.
The issue is that we try to apply similar schema validation and parsing when static auth is not configured, i.e. when we retrieve the file's contents via the "raw" endpoint. In this case, I suggest we just parse the JSON as plaintext.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @PyvesB!

  Generated by :no_entry_sign: dangerJS against 144d4d1844d8a1c722fbd552d7c075cf04dddb6f

The same two badges as above on the review app:


Exceptional not properly analysing functions

For some reason resharper is not correctly inspecting my functions. A lot of items are listed as throwing exceptions however the analysis says it cannot be thrown. Why is this?

My options are set up a follows:

edit: Another example of how it seems to fail to inspect system classes.

Anyone?

Hi @rolandh
I've made a change for a seemingly unrelated issue and while I was at it, tested your case.
It seems to work with my current change so, can I ask you to test an experimental package?

Hi,
This is old, but still applies.
Using 2022.2.0 version.


CSS Zoom put the mess with the hover

Expected Behavior
When I'm using the CSS property zoom: 2; the hover elements of my canvas should be sync with the canvas element.
Current Behavior
If I'm applying zoom:2; the hover is desynchronized from the canvas element
Possible Solution
Steps to Reproduce (for bugs)

Generate your Chart graphic
Add to the container the css property: "zoom: 2;"
Hover the different parts of the canvas to constat it's no sync anymore

Context
The Zoom property is mostly used for having a nice behavior on all the screens (responsive).
In my case, I was implementing Chart.js to the Reveal.js framework (a presentation framework).
Environment

Chart.js version: 2.9.3
Browser name and version: All
Link to your project: https://revealjs.com/

I ran into the same problem and can hence confirm it. It would cool to find a solution!

The problem is that the mouse position has to be scaled by the inverse CSS zoom. You can fix it by injecting this piece of code into line 107 of helpers.dom.js, i.e., after determining mouseX and before considering padding:
        // Take into account CSS zoom on some parent element.
        // Zoom is used, e.g., by Reveal.js.
        var zoom = 1;
        for (var elem=canvas; elem; elem=elem.parentElement) {
            zoom *= elem.style.zoom || 1
        }
        if (zoom != 1) {
            mouseX = mouseX / zoom;
            mouseY = mouseY / zoom;
        }

Maybe a maintainer can turn this into a pull request, since they know better how those should look like?

Can confirm this is still broken in v3.0.0-beta.9 however the CSS zoom feature is non-standard and not on the standards track according to https://developer.mozilla.org/en-US/docs/Web/CSS/zoom so I don't think we should fix this.
@kurkle thoughts?

Can confirm this is still broken in v3.0.0-beta.9 however the CSS zoom feature is non-standard and not on the standards track according to https://developer.mozilla.org/en-US/docs/Web/CSS/zoom so I don't think we should fix this.
@kurkle thoughts?

I don't think we should add such code in core. You can do it in a plugin: https://jsfiddle.net/85f41yra/

I don't think we should add such code in core. You can do it in a plugin: https://jsfiddle.net/85f41yra/

The "zoom" property is now part of baseline for all the major browsers since May 2024 (https://developer.mozilla.org/en-US/docs/Web/CSS/zoom).  Can we reopen this issue?

The zoom property is part of the Baseline 2024.
Since May 2024, this feature works across the latest devices and browser versions.
Please address this issue.


use ctrlc for windows

unix::signals obviously depends on being on a unix machine, here I add the option to use control c if you are in windows making the tool cross-platform

I still have to normalise the paths if you are using windows in case that people are using CSV, do not merge

Thanks for this @JosepBove let me know it all still works on unix il take a look at this PR in the morning - not had loads of bandwidth to test windows so thanks for doing these fixes

I had to find a gap on my schedule to test it!


Ryan M: Watch tips4hacking.com

Ryan M requests the watch of the watch_keyword tips4hacking\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

tips4hacking\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

!!/approve

Approved by Mithical in Charcoal HQ


TypeError: Cannot read properties of null (reading 'insertAdjacentHTML')

I cloned your starter and modified it to run the editor at localhost:3000/demo. While I was able to launch the editor, I can't drag and drop sections from the sidebar, which results in an error.

I have a couple of questions. Can I create multiple pages? According to your README file, it seems possible, but that guidance doesn't seem to work. I can't get localhost:3000/demo to render as expected, and it throws various errors.
Sometimes I encounter "module not found: 'fs'" at build/server, and other times I get a {"error": "forbidden"} message.
This is a cool tool, but I think it needs more refactoring. I’m happy to assist with documentation, but I really need help resolving these issues.
mycode: https://github.com/mihir-kanzariya/destack-by-mihir

It runs smoothly at the root endpoint (localhost:3000).

@mihir-kanzariya I tried git cloning the repo you send me earlier and it works on my machine. I see you are using Windows so it might be a windows specific problem. I also check the changes you made and they seem reasonable so being issue with Windows is very likely.
Another observation from the screenshots is that every API request returns a 404 which should not happen.

@LiveDuo
It works well on localhost:3000/ but not on localhost:3000/demo, where it encounters errors on multiple pages. You can test the localhost:3000/demo endpoint in my code.
Please check out the Loom video I recorded to reproduce the issue.https://www.loom.com/share/2258b8eb600d415d87b8def9f5b880ee?sid=d0adc25b-a675-48a2-9770-d26983d5655f

@mihir-kanzariya Thanks for the video, I thought you are using Windows before but you are clearing using OSX.
I manage to reproduce the issue now and will take a closer look later today or tomorrow

Thank you @LiveDuo
let me know if I can help somewhere

Yes, I am also facing the same issue when I try to use it with NextJS app router.

I am getting this error when trying to add below code in any of the component (i.e. root layout/page or other page) inside app directory
code:
export { getStaticProps } from "destack/build/server";
export { ContentProvider as default } from "destack";

Is there any typescript support?
As i am getting this error
also npm i --save-dev @types/destack is not working

@mihir-kanzariya Just pushed a fix https://github.com/LiveDuo/destack/commit/511578388e55389180df3c8826707b74269835c5.
To get it update your destack version with npm i destack@latest.

Yes, I am also facing the same issue when I try to use it with NextJS app router. I am working on Ubuntu - Linux machine

NextJs app router is not supported right now. There was some effort before https://github.com/LiveDuo/destack/issues/79 which solved it for earlier versions of Nextjs.

Is there any typescript support? As i am getting this error also npm i --save-dev @types/destack is not working

While the library is written in Typescript, there's no @types/destack. At this point, the build system (rollup, react, typescript) creates enough problems that I'm not sure it's the way to go. Work started on this with https://github.com/LiveDuo/destack/issues/104.

I am getting this another error while working with pages router:

@meet-bhimani Can you create a repo to reproduce the error? It helps to have the exact dependencies that created the problem.

I am getting this error when trying to add below code in any of the component (i.e. root layout/page or other page) inside app directory
code:
export { getStaticProps } from "destack/build/server";
export { ContentProvider as default } from "destack";

### This will be solved by following code:
instead of directly exporting getStaticProps and ContentProvider, write it like this:
"use client"
import { getStaticProps } from "destack/build/server";
import { ContentProvider } from "destack";

async function page() {
  const props = await getStaticProps().then((d) => d.props);

  return <ContentProvider data={props?.data}></ContentProvider>;
}

export default page;

To get it update your destack version with npm i destack@latest.

Thanks @LiveDuo , resolved first error, Now I am getting
Module not found: Can't resolve 'fs' at ./node_modules/destack/build/server/index.js:6:1

To get it update your destack version with npm i destack@latest.

Thanks @LiveDuo , resolved first error, Now I am getting
Module not found: Can't resolve 'fs' at ./node_modules/destack/build/server/index.js:6:1

@mihir-kanzariya Either you have a different setup or trying to do something different in destack that I haven't tried yet. Have to figure it out to investigate further.

ps: quite strange that the Nextjs API route can't find fs

@LiveDuo
I'm trying to make it work with the Next.js app router, but I'm having trouble getting it to function correctly.
Do you plan to support the next app router in Next.js in near future?

@LiveDuo I'm trying to make it work with the Next.js app router, but I'm having trouble getting it to function correctly.
Do you plan to support the next app router in Next.js in near future?

@mihir-kanzariya Yes but there isn't a timeline for that. Next.js has an upgrade guide and there are relevant sections about API routes and getStaticProps that Destack uses https://nextjs.org/docs/app/building-your-application/upgrading/app-router-migration.
If anyone made a PR or suggest a solution for these issues I'd try to review them asap.

@LiveDuo Good news!
I have integrated support for the Nextjs app router in your repo! I have created PR here
take a look and let me know!

@meet-bhimani  thanks for the PR, let's continue the review there


Lazy loading for Gtk

PR Checklist
If you are contributing to Javis.jl, please make sure you are able to check off each item on this list:

[x] Did I update CHANGELOG.md with whatever changes/features I added with this PR?
[x] Did I make sure to only change the part of the file where I introduced a new change/feature?
[x] Did I cover all corner cases to be close to 100% test coverage (if applicable)?
[x] Did I properly add Javis dependencies to the Project.toml + set an upper bound of the dependency (if applicable)?
[x] Did I properly add test dependencies to the test directory (if applicable)?
[ ] Did I check relevant tutorials that may be affected by changes in this PR?
[x] Did I clearly articulate why this PR was made the way it was and how it was made?

Link to relevant issue(s)
Closes #193
How did you address these issues with this PR? What methods did you use?
Continuation of https://github.com/Wikunia/Javis.jl/pull/225

Hey @Sov-trotter - thanks for continuing to look at this. I am still getting the same error that I got last time being this along with some warnings about overwriting:
(@v1.5) pkg> activate .
 Activating environment at `/home/src/Projects/javis/Project.toml`

julia> include("view.jl")
[ Info: Precompiling Javis [78b212ba-a7f9-42d4-b726-60726080707e]
WARNING: Method definition __init__() in module Javis at /home/src/Projects/javis/src/Javis.jl:101 overwritten at /home/src/Projects/javis/src/Javis.jl:441.
  ** incremental compilation may be fatally broken for this module **

ERROR: LoadError: UndefVarError: _javis_viewer not defined
Stacktrace:
 [1] javis(::Video, ::Array{Action,1}; framerate::Int64, pathname::String, liveview::Bool, tempdirectory::String) at /home/src/Projects/javis/src/Javis.jl:227
 [2] top-level scope at /home/src/Projects/javis/view.jl:94
 [3] include(::String) at ./client.jl:457
 [4] top-level scope at REPL[2]:1
in expression starting at /home/src/Projects/javis/view.jl:94

Here is the code I was using:
using Javis
using Random

function ground(args...)
    background("white")
    sethue("black")
end

function draw_line(p1 = O, p2 = O, color = "black", action = :stroke, edge = "solid")
    sethue(color)
    setdash(edge)
    line(p1, p2, action)
end

function circ(p = O, color = "black", action = :fill, radius = 25, edge = "solid")
    sethue(color)
    setdash(edge)
    circle(p, radius, action)
end

function info_box(video, action, frame)
    fontsize(12)
    box(140, -210, 170, 40, :stroke)
    text("10-20 EEG Array Readings", 140, -220, valign = :middle, halign = :center)
    text("t = $(frame)s", 140, -200, valign = :middle, halign = :center)
end

function electrode(
    p = O,
    fill_color = "white",
    outline_color = "black",
    action = :fill,
    radius = 25,
    circ_text = "",
)
    sethue(fill_color)
    circle(p, radius, :fill)
    sethue(outline_color)
    circle(p, radius, :stroke)
    text(circ_text, p, valign = :middle, halign = :center)
end

electrode_locations = [
    O,
    Point(-70, 0),
    Point(70, 0),
    Point(-140, 0),
    Point(140, 0),
    Point(0, 70),
    Point(-50, 70),
    Point(50, 70),
    Point(0, -70),
    Point(-50, -70),
    Point(50, -70),
    Point(115, -80),
    Point(-115, -80),
    Point(115, 80),
    Point(-115, 80),
    Point(40, -135),
    Point(-40, -135),
    Point(-190, -10),
    Point(190, -10),
    Point(-40, 135),
    Point(40, 135),
]

electrode_names = [
    "Cz",
    "C3",
    "C4",
    "T3",
    "T4",
    "Pz",
    "P3",
    "P4",
    "Fz",
    "F3",
    "F4",
    "F8",
    "F7",
    "T6",
    "T5",
    "Fp2",
    "Fp1",
    "A1",
    "A2",
    "O1",
    "O2",
]

radius = 15
indicators = ["tomato", "darkolivegreen1", "gold1", "white"]
demo = Video(500, 500)
javis(
    demo,
    [
        BackgroundAction(1:10, ground),
        Action(
            :inside_circle,
            (args...) -> circ(O, "black", :stroke, 140, "longdashed"),
        ),
        Action(:head, (args...) -> circ(O, "black", :stroke, 170)),
        Action(
            :vert_line,
            (args...) ->
                draw_line(Point(0, -170), Point(0, 170), "black", :stroke, "longdashed"),
        ),
        Action(
            :horiz_line,
            (args...) ->
                draw_line(Point(-170, 0), Point(170, 0), "black", :stroke, "longdashed"),
        ),
        Action(
            :electrodes,
            (args...) ->
                electrode.(
                    electrode_locations,
                    rand(indicators, length(electrode_locations)),
                    "black",
                    :fill,
                    radius,
                    electrode_names,
                ),
        ),
        Action(:info, info_box),
    ],
    pathname = "eeg.gif",
    framerate = 1,
    liveview = true
)

It seems like you are getting closer based on the grid worlds issue you found here: https://github.com/JuliaReinforcementLearning/GridWorlds.jl/commit/f46e78c3ff224039adb42a3362edee5c8a053428 Maybe ask @findmyway for some help? He used Javis to make the logo for JuliaReinforcementLearning. :smile:

Yeah. I am digging in a bit more since we are now clear about what we want and also that we have a use case. I'll let you know once I make sure that things work well on my end. :)))

Thanks for @TheCedarPrince 's invitation.
Hi @Sov-trotter ,
I notice that you've commented out some tests. I guess some important parts are missed.
To keep tests working as usual, you need to add the GTK and GtkReactive in the Extras and Targets section of the root Project.tom.
Just like this:
https://github.com/JuliaReinforcementLearning/ReinforcementLearningEnvironments.jl/blob/0932006ceafae3073ac569021bf925200d91e1cf/Project.toml#L22-L32
Another way is to create a new environment inside of tests folder. (By creating a Project.toml file in the tests) . But I forget when is this feature enabled. (I guess since<EMAIL_ADDRESS>Another tip.
Given that GtkReactive relies on GTK, you can simply require GtkRequires and then
using .GtkReactive
using .GtkReactive.GTK

So that one can simply enable those extra features by using GtkReactive instead of using two packages.
(I haven't test it. But it should work ;)

Thank you so much @findmyway for the help. I was able to get things working with the examples @TheCedarPrince quoted but the tests were failing for some reason. For now I fixed it by using GtkReactive.Gtk qualification for Gtk methods.
Also there are two warnings viz.
WARNING: using GtkReactive.label in module Javis conflicts with an existing identifier.
WARNING: using GtkReactive.textbox in module Javis conflicts with an existing identifier.

I am not really sure what can be done on this side but I'd love to work on any suggestions. :)

Thanks @Sov-trotter and @findmyway
The warnings appear because both GtkReactive and Javis export label and textbox. This can be fixed by using import instead of using.
I think you can remove the test extras from the Project.toml as you have them in test/Project.toml
Additionally I am wondering whether you deleted the reference images.

Oh. Sorry for that, since the tests were successful for the first time on my machine, I though those were supposed to be ignored.
Replacing import for using does remove the warnings but it creates issues with Requires,
 Warning: Error requiring GtkReactive from Javis:
│ LoadError: LoadError: LoadError: UndefVarError: @guarded not defined

You probably want to use something like:
import Luxor
import Luxor: Point, @layer

in this case import Gtk: @guarded and probably some others. (maybe GtkReactive :smile:  It's not my part of the code base so my knowledge is limited)
Refs are the reference images which stay constant such that the newly computed images are checked against them.

Great work. Now is it somehow possible that the user doesn't have to write using GtkReactive ?
I would like that it's automatically loaded when the user sets the keyword liveview=true

Yeah was thinking of the same. That side would be kinda different from the requires route I guess. One way I am thinking of is where the true flag is checked for? Not sure though

Maybe you can double check that this works:
 if liveview == true
        @eval import .GtkReactive
        @eval import .GtkReactive: slider, signal
        @eval using .GtkReactive.Gtk
        _javis_viewer(video, length(frames), actions)
        return "Live preview started."
    end

(inside src/Javis.jl)
and then remove the import from javis_viewer.jl

Oh cool. What I wanted to try was
if liveview == true
        include("javis_viewer.jl")
        _javis_viewer(video, length(frames), actions)
        return "Live preview started."
    end

while the file would remain the same, when I call the function with liveviewer= true, I get a ERROR: could not open file /home/ezio/.julia/dev/Javis/javis_viewer.jl

Yours might fail because it tries to load it in the current working directory instead of the javis src directory?
Unfortunately mine doesn't work.
The problem with loading GtkReactive as a user might be that we introduce the warnings again so the user would need to write import?
Feels a bit weird to write that as the user 😕
Hope we can figure it out

The problem with loading GtkReactive as a user might be that we introduce the warnings again so the user would need to write import?

Sorry you mean with Requires or the if liveview=true condition method?

Sorry I mean if the user has to write:
using Javis
using GtkReactive

like it is currently the case with this PR.

Ah yeah. But I don't think that will bring up the warnings since we have it fixed inhere https://github.com/Wikunia/Javis.jl/blob/f5c299aabf8ab6dcf23b0698c9199344ef35bf12/src/javis_viewer.jl#L1-L3. The main role of using GtkReactive as a user would be to invoke the Requires API?

I think there is a problem if the user writes label in his/her code, right?

Sorry didn't get you?

Sorry my explanation is probably kinda lacking :smile:
If the user has
using Javis
using GtkReactive

function drawing(args...)
  label() # should be Javis but is ambiguous as GtkReactive also exports it, right?
end

I hope this makes clear what I'm worried about. Anyway best would be if the user doesn't need to write using GtkReactive

Oh. I think the warning can be avoided by using a qualification ? But yeah this seems to take us down to more such future warnings. :/

It can be avoided by using import I think but yeah it's not a nice user experience. Based on our discussion that mostly @TheCedarPrince and myself are experiencing the slow down because of precompilation it's not really ideal. I hope we can finde a way around this.

Hey @Sov-trotter - I posted about this on the Julia Discourse and it seems like we may have a few potential answers (see here: https://discourse.julialang.org/t/dynamic-code-loading-to-use-packages-on-demand/48655). Hopefully, one of these can address this problem.

Hey @Sov-trotter do you still want to work on this or should I take it over? Thanks!

Yeah. I'd love to. Sorry I am a bit slow due to some other commitments. I'll get it done soon. :)

No hurry @Sov-trotter We probably need at least 10 more days to bring v0.3 and I think this issue can't be added before that anyway as it needs a new dependency.

So following up on the discussion over discourse, I am still getting the world age error. Not sure what's incorrect though. :/

Hey @Wikunia and @Sov-trotter - I checked this out again and I am really stuck on what to do with this issue. I am debating about closing it altogether sadly. :disappointed:

I feel like it's not strictly necessary but appreciate the effort @Sov-trotter .
Would have be great but I think we have more important issues to tackle.
Feel free to close this @Sov-trotter if you don't want to further work on it.

Okay. Sorry though I am not able to commit time rn(caught up in academic stuff currently), I think till mid-december. I think you can close it for now. We can however revisit this later as the package grows.


Make uv__get_osfhandle() public

MSVCR is statically linked into node, which means that users can't pass file descriptors from nodejs add-ons to the JavaScript code, given that the file descriptor to handle map in the add-on is separate to the one statically linked into the node executable.
This was initially reported in https://github.com/nodejs/node/issues/6369, where the suggestion was to make uv__get_osfhandle() public, so add-ons can make use of it, and resolve the file descriptor correctly.
Would such a change be accepted?
The contributing guide says:

The stable branch is effectively frozen; patches that change the libuv API/ABI or affect the run-time behavior of applications get rejected.

However I wonder what are your thoughts here, given that the public API will remain backwards compatible.
If this is not something you'd accept, would you mind suggesting an alternative to mitigate this issue?
Thanks in advance!

I think this related with #1121, #1119 and #1166

@jviotti Yeah, I think it's an acceptable change, specially since we are adding it in #1166. If you write a PR, take the code from there so we don't diverge master from v1.x.


Text Decorations Not Working for Untagged Text

The underline decoration does not work if the text is not wrapped in a tag, even when textDecoration is specified in the default style.
This will not show the underline:
const underlineText = `some random text`;
const underlineStyle = {
  default: {
    textDecoration: "underline",
  },
  random: {}
};

This code will show the underline:
const underlineText = `<random>some random text</random>`;
const underlineStyle = {
  default: {
    textDecoration: "underline",
  },
  random: {}
};

fixed in 3.15.2


Auto fitting to the height of pane1 histograms

hi , I am testing out v5-candidate with multpane , I added volume series to new pane via
   const volumeSeries = chart.addHistogramSeries({
            color: '#26a69a', // Default color (used for volume bars)
            priceFormat: {
                type: 'volume',
            },
        }, 1); // Add to second pane (pane index 1)

now if i want to scale this volumeseries , it is also scaling the pane 0 price series.
volumeSeries.priceScale().applyOptions({
    autoScale: false, // disables auto scaling based on visible content
    scaleMargins: {
        top: 0.9,
        bottom: 0,
    },
});

Its expected to fit / scale only pane 1 as the series is in pane1

use an id for price scale :
    const volumeSeries = chart.addHistogramSeries({
    color: '#26a69a',
    priceFormat: {
        type: 'volume',
    },
    priceScaleId: 'rightVolume',
}, 1);
chart.priceScale('rightVolume').applyOptions({
    autoScale: false,
    scaleMargins: {
        top: 0.9,
        bottom: 0,
    },
});

If i set  priceScaleId: 'rightVolume', , I am not able to scale by dragging its pricescale.. also when i hover the volume its not displaying price label on right side.

If i set priceScaleId: 'rightVolume', , I am not able to scale by dragging its pricescale.. also when i hover the volume its not displaying price label on right side.

As I understand it, if you want to have full control over the volume, the best approach is to build a plugin, or you can use a secondary overlay canvas or a second price scale at the left

sorry , I am not a professional to build plugin..Also i want both price and volume scale on the right, thats why using the pane.
so can you tell me about secondary overlay canvas ?

sorry , I am not a professional to build plugin..Also i want both price and volume scale on the right, thats why using the pane.
so can you tell me about secondary overlay canvas ?

as i am a fan of pure js i created a plugin for lightweight chart. you can see the concept and use it :
https://github.com/safaritrader/lightweight-chart-plugin

wow , thanks i was thinking to create volume profile for my project , but did u extend version 4 or 5 ..i need multi pane support.

wow , thanks i was thinking to create volume profile for my project , but did u extend version 4 or 5 ..i need multi pane support.

your welcome. i am expanding that plugin frequently. it's based on v 4.2.0

ok , then I want to build plugin for my v5 , can you guide me with any basic tutorials ? there is no documentation in TV how to create a plugin from scratch.
I am coding vanilla js for past 8 years.. still there was no use for constructor , canvas I did not learn , but I am willing to learn now ..  can you  tell me what should I learn ? because I am new to constructor , canvas etc ..
is there a skeleton or template that i must follow to create plugin from scratch and how it works.

ok , then I want to build plugin for my v5 , can you guide me with any basic tutorials ? there is no documentation in TV how to create a plugin from scratch.
I am coding vanilla js for past 8 years.. since there was no use for constructor , canvas I did not learn , but I am willing to learn now .. can you tell me what should I learn ? because I am new to constructor , canvas etc ..
is there a skeleton or template that i must follow to create plugin from scratch and how it works.

As i know the last version for lightweight charts is 4.2

ok there is another branch named 5 , you can use that to checkout multi pane feature - https://github.com/tradingview/lightweight-charts/tree/v5-candidate
btw please tell me what should I learn , or any tutorials from scratch to create one simple plugin

ok there is another branch named 5 , you can use that to checkout multi pane feature - https://github.com/tradingview/lightweight-charts/tree/v5-candidate
btw please tell me what should I learn , or any tutorials from scratch to create one simple plugin

Official Library suggest to do that :
https://github.com/tradingview/lightweight-charts/tree/master/packages/create-lwc-plugin
which i am working with pure js and its not fitting my requirements and im using overlay canvas which i described in my Lightwight charts plugin. its very simple when you read you will learn!. the main approach of mine is to use another canvas over the lwc canvas and do every thing you want

Ok I am reading your code now. along with chatgpt help
On Wed, 16 Oct, 2024, 4:17 pm Hassan Safari, @.***>
wrote:

ok there is another branch named 5 , you can use that to checkout multi
pane feature -
https://github.com/tradingview/lightweight-charts/tree/v5-candidate
btw please tell me what should I learn , or any tutorials from scratch to
create one simple plugin
Official Library suggest to do that :
https://github.com/tradingview/lightweight-charts/tree/master/packages/create-lwc-plugin
which i am working with pure js and its not fitting my requirements and im
using overlay canvas which i described in my Lightwight charts plugin. its
very simple when you read you will learn!. the main approach of mine is to
use another canvas over the lwc canvas and do every thing you want
—
Reply to this email directly, view it on GitHub
https://github.com/tradingview/lightweight-charts/issues/1709#issuecomment-2416441546,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQ5HCSFGTVALQBKU4NOK3TZ3Y75TAVCNFSM6AAAAABP4OUWBWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJWGQ2DCNJUGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hey @safaritrader I saw plugin use fancy-canvas etc , how did u overcome all those things using vanilla js.

Hey @safaritrader I saw plugin use fancy-canvas etc , how did u overcome all those things using vanilla js.
Hi Dear @vishnuc
Sometimes a simple solution is faster for achieving the desired outcome, plus I tested that plugin on 3 million data points for volume profile, and it worked perfectly. I try not to get too caught up in complexities and aim to reach my goal with the simplest approach. However, there are still some small bugs that I'll fix in the next update. Since I work with large datasets, I need to make sure the code can handle at least 10 million data points. One of the good features of this plugin is that it doesn’t require extensive expertise to modify or rewrite.

hi , but I compiled and run your example.. its not fluid when i scale price axis .. native plugins are way smoother...also when time axis is scaled I think you redraw after certain interval.

hi , but I compiled and run your example.. its not fluid when i scale price axis .. native plugins are way smoother...also when time axis is scaled I think you redraw after certain interval.

its depend on your data scale. for now its loop through all data but im replacing a simple algorithm to not loop through all data in next version and its depend on your browser also it take 5 sec to build a volume profile from 3m data (tested in firefox). other plugins are using lightweight chart built-in feature's which is great but not fit my requirements

hi , i succeeded creating primitive with pure js , its smooth . now working on series , will update you.
    <script >

        class AnchoredTextRenderer {
            constructor(options) {
                this._data = options;
            }

            draw(target) {
                target.useMediaCoordinateSpace((scope) => {
                    const ctx = scope.context;
                    ctx.font = this._data.font;
                    const textWidth = ctx.measureText(this._data.text).width;
                    const horzMargin = 20;
                    let x = horzMargin;
                    const width = scope.mediaSize.width;
                    const height = scope.mediaSize.height;

                    switch (this._data.horzAlign) {
                        case 'right':
                            x = width - horzMargin - textWidth;
                            break;
                        case 'middle':
                            x = width / 2 - textWidth / 2;
                            break;
                    }

                    const vertMargin = 10;
                    const lineHeight = this._data.lineHeight;
                    let y = vertMargin + lineHeight;

                    switch (this._data.vertAlign) {
                        case 'middle':
                            y = height / 2 + lineHeight / 2;
                            break;
                        case 'bottom':
                            y = height - vertMargin;
                            break;
                    }

                    ctx.fillStyle = this._data.color;
                    ctx.fillText(this._data.text, x, y);
                });
            }
        }

        class AnchoredTextPaneView {
            constructor(source) {
                this._source = source;
            }

            update() {}

            renderer() {
                return new AnchoredTextRenderer(this._source._data);
            }
        }

        class AnchoredText {
            constructor(options) {
                this._data = options;
                this._paneViews = [new AnchoredTextPaneView(this)];
            }

            updateAllViews() {
                this._paneViews.forEach(pw => pw.update());
            }

            paneViews() {
                return this._paneViews;
            }

            attached({ requestUpdate }) {
                this.requestUpdate = requestUpdate;
            }

            detached() {
                this.requestUpdate = undefined;
            }

            applyOptions(options) {
                this._data = { ...this._data, ...options };
                if (this.requestUpdate) this.requestUpdate();
            }
        }

        // Create the chart
        const chart = LightweightCharts.createChart(document.getElementById('chart'), {
            width: window.innerWidth,
            height: 500,
        });

          // Add a line series
          const lineSeries1 = chart.addLineSeries({},0);
        const data1 = [
            { time: '2024-01-01', value: 120 },
            { time: '2024-01-02', value: 130 },
            { time: '2024-01-03', value: 125 },
            { time: '2024-01-04', value: 135 },
            { time: '2024-01-05', value: 140 },
        ];
        lineSeries1.setData(data1);

        // Add a line series
        const lineSeries = chart.addLineSeries({},1);
        const data = [
            { time: '2024-01-01', value: 120 },
            { time: '2024-01-02', value: 130 },
            { time: '2024-01-03', value: 125 },
            { time: '2024-01-04', value: 135 },
            { time: '2024-01-05', value: 140 },
        ];
        lineSeries.setData(data);

        // Add Anchored Text
        const anchoredText = new AnchoredText({
            vertAlign: 'middle',
            horzAlign: 'middle',
            text: 'Anchored Text',
            lineHeight: 54,
            font: 'italic bold 54px Arial',
            color: 'green',
        });
        lineSeries.attachPrimitive(anchoredText);

        // Change the text after 2 seconds
        setTimeout(() => {
            anchoredText.applyOptions({
                text: 'New Text',
            });
        }, 2000);

        // Update chart size on window resize
        window.addEventListener('resize', () => {
            chart.applyOptions({ width: window.innerWidth });
        });
    </script>

ok series too works fine with pure js  , why dont u use like this in ur plugin,.. so even when u scale it will look good.

ok series too works fine with pure js , why dont u use like this in ur plugin,.. so even when u scale it will look good.

   <script type="module">
        import { createChart } from '/tv1.mjs';

        // Renderer for the Lollipop series
        class LollipopSeriesRenderer {
            constructor() {
                this._data = null;
                this._options = null;
            }

            draw(target, priceConverter) {
                target.useBitmapCoordinateSpace(scope => this._drawImpl(scope, priceConverter));
            }

            update(data, options) {
                this._data = data;
                this._options = options;
            }

            _drawImpl(scope, priceToCoordinate) {
                if (!this._data || !this._options || this._data.bars.length === 0 || !this._data.visibleRange) {
                    return;
                }

                const bars = this._data.bars.map(bar => ({
                    x: bar.x,
                    y: priceToCoordinate(bar.originalData.value) ?? 0,
                }));

                const lineWidth = Math.min(this._options.lineWidth, this._data.barSpacing)*3;
                const radius = Math.min(Math.floor(this._data.barSpacing / 2), 5);  // Max radius for the circles
                const zeroY = priceToCoordinate(0);

                for (let i = this._data.visibleRange.from; i < this._data.visibleRange.to; i++) {
                    const bar = bars[i];
                    const xPosition = bar.x * scope.horizontalPixelRatio;
                    const yPosition = bar.y * scope.verticalPixelRatio;

                    scope.context.beginPath();
                    scope.context.fillStyle = this._options.color;

                    // Draw the line from zero to the value
                    scope.context.fillRect(xPosition - lineWidth / 2, zeroY * scope.verticalPixelRatio, lineWidth, yPosition - zeroY * scope.verticalPixelRatio);

                    // Draw the circle on top
                    scope.context.arc(xPosition, yPosition, radius * scope.horizontalPixelRatio, 0, Math.PI * 2);
                    scope.context.fill();
                }
            }
        }

        // Lollipop Series class
        class LollipopSeries {
            constructor() {
                this._renderer = new LollipopSeriesRenderer();
            }

            priceValueBuilder(plotRow) {
                return [0, plotRow.value];
            }

            isWhitespace(data) {
                return data.value === undefined;
            }

            renderer() {
                return this._renderer;
            }

            update(data, options) {
                this._renderer.update(data, options);
            }

            defaultOptions() {
                return {
                    lineWidth: 2,
                    color: 'rgb(0, 100, 255)',
                };
            }
        }

        // Create the chart
        const chart = createChart(document.getElementById('chart'), {
            width: window.innerWidth,
            height: 500,
        });

        // Add Lollipop series to chart
        const customSeriesView = new LollipopSeries();
        const myCustomSeries = chart.addCustomSeries(customSeriesView, {
            lineWidth: 1,   // Make lines thinner
            color: 'rgb(0, 200, 255)',  // Lollipop color
        });

        // Generate random data for the Lollipop series
        function generateLollipopData() {
            const data = [];
            let baseTime = new Date('2021-01-01').getTime() / 1000; // Unix timestamp in seconds

            for (let i = 0; i < 50; i++) {
                const value = Math.random() * 300;  // Use a wider value range
                data.push({
                    time: baseTime + i * 60 * 60 * 24, // Increment by 1 day
                    value: value,
                });
            }
            return data;
        }

        // Set Lollipop data
        const lollipopData = generateLollipopData();
        myCustomSeries.setData(lollipopData);

        // Update chart size on window resize
        window.addEventListener('resize', () => {
            chart.applyOptions({ width: window.innerWidth });
        });
    </script>

that using for loop for redraw too which cant make a different for my current plugin. its executing redraw on resize and changing scales too it just using less code to attach custom shapes or series which is good

No yours is not giving native feel , I am checking in my macbook retina..Its easy I think u can create lots of plugin with native way with vanilla js.
also if i scale my pricescale up and down urs is hiding the volume profile etc


FGDC metadata, step 2 (v1.6.9); UnicodeDecodeError

Hello! I have run into the following issue with the FGDC metadata, step 2 tool (v1.6.9).  The tool is failing when trying to the XML file for the DescriptionOfMapUnits table and is reporting a UnicodeDecodeError (see error message below). I have tried to remove any non-ASCII characters from the table but the issue persists. Any assistance with this issue will be greatly appreciated!
Pruning unused srcinfo branches
Replacing title and suppplinf

G:\CGS\GM_Work\Arcwork\Oakdale_100k_GeMs_Oakdale_100k\Oakdale_100k__NAD27_UTM10\Oakdale_100k\workspace\EastHalfOakdale_100k.gdb
fileName=EastHalfOakdale_100k_DescriptionOfMapUnits-metadata.xml
Writing XML to EastHalfOakdale_100k_DescriptionOfMapUnits-metadata.xml
Failed script MetadataFGDC-2...
Traceback (most recent call last):
File "G:\CGS\GM_Work\Arcwork\ArcGIS_Files\GeMS\test\cg_testing\gems-tools-arcmap-master-v1.6.9\Scripts\GeMS_FGDC2_Arc10.py", line 503, in
fixObjXML(objName,objType,objLoc,domMR)
File "G:\CGS\GM_Work\Arcwork\ArcGIS_Files\GeMS\test\cg_testing\gems-tools-arcmap-master-v1.6.9\Scripts\GeMS_FGDC2_Arc10.py", line 446, in fixObjXML
writeDomToFile(wksp,dom,domName)
File "G:\CGS\GM_Work\Arcwork\ArcGIS_Files\GeMS\test\cg_testing\gems-tools-arcmap-master-v1.6.9\Scripts\GeMS_FGDC2_Arc10.py", line 460, in writeDomToFile
dom.writexml(out, encoding="utf-8")
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 1749, in writexml
node.writexml(writer, indent, addindent, newl)
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 814, in writexml
node.writexml(writer, indent+addindent, addindent, newl)
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 814, in writexml
node.writexml(writer, indent+addindent, addindent, newl)
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 814, in writexml
node.writexml(writer, indent+addindent, addindent, newl)
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 814, in writexml
node.writexml(writer, indent+addindent, addindent, newl)
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 814, in writexml
node.writexml(writer, indent+addindent, addindent, newl)
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 814, in writexml
node.writexml(writer, indent+addindent, addindent, newl)
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 810, in writexml
self.childNodes[0].writexml(writer, '', '', '')
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 1038, in writexml
_write_data(writer, "%s%s%s" % (indent, self.data, newl))
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 298, in _write_data
writer.write(data)
File "C:\Python27\ArcGIS10.7\Lib\codecs.py", line 708, in write
return self.writer.write(data)
File "C:\Python27\ArcGIS10.7\Lib\codecs.py", line 369, in write
data, consumed = self.encode(object, self.errors)
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 1: ordinal not in range(128)
Failed to execute (MetadataFGDC-2).
Failed at Wed Sep 29 11:53:03 2021 (Elapsed Time: 1 minutes 6 seconds)

Try the toolbox in the branch I just pushed to the repo:
https://github.com/usgs/gems-tools-arcmap/tree/issue-80
and let me know.

Thanks for the quick reply. I just tried the updated toolbox and it failed immediately on a syntax error in line 446:
(wksp,dom,domName)writeDomToFile
I corrected the syntax error but the tool still fails, this time with a UnicodeEncodeError:
...
DescriptionOfMapUnits
Updating E-A stuff
Field: OBJECTID
Field: MapUnit
Failed script MetadataFGDC-2...
Traceback (most recent call last):
File "G:\CGS\GM_Work\Arcwork\ArcGIS_Files\GeMS\test\cg_testing\gems-tools-arcmap-issue-80\Scripts\GeMS_FGDC2_Arc10.py", line 503, in
fixObjXML(objName,objType,objLoc,domMR)
File "G:\CGS\GM_Work\Arcwork\ArcGIS_Files\GeMS\test\cg_testing\gems-tools-arcmap-issue-80\Scripts\GeMS_FGDC2_Arc10.py", line 433, in fixObjXML
dom, dataSourceValues = updateTableDom(dom,objLoc,logFile)
File "G:\CGS\GM_Work\Arcwork\ArcGIS_Files\GeMS\test\cg_testing\gems-tools-arcmap-issue-80\Scripts\GeMS_FGDC2_Arc10.py", line 383, in updateTableDom
dom, dataSourceValues = __updateEntityAttributes(fc, fldNameList, dom, logFile)
File "G:\CGS\GM_Work\Arcwork\ArcGIS_Files\GeMS\test\cg_testing\gems-tools-arcmap-issue-80\Scripts\GeMS_FGDC2_Arc10.py", line 200, in __updateEntityAttributes
defs[t].append(row.FullName.decode('utf-8'))
File "C:\Python27\ArcGIS10.7\Lib\encodings\utf_8.py", line 16, in decode
return codecs.utf_8_decode(input, errors, True)
UnicodeEncodeError: 'ascii' codec can't encode character u'\xe9' in position 21: ordinal not in range(128)
Failed to execute (MetadataFGDC-2).
Failed at Thu Sep 30 16:41:39 2021 (Elapsed Time: 25.35 seconds)

Heck. Sorry about all this. I still do not quite understand encoding/decoding (and I didn't write this tool).
I just pushed another version to that branch. If this doesn't work, I will dig deeper - or work to get the Python 3 version finished.
Finally, I don't like the idea of excluding non-ASCII text from the unit names or descriptions, but it appears that the unicode character is a lower case accented e (é)  (https://www.compart.com/en/unicode/U+00E9). If it's acceptable, you could change that and see if it works.

No need to apologize, Evan. I really appreciate all the work put into these tools and the assistance provided when they don't quite work as they should. I tried the latest version of the tool but it still complained about that character so, just to see if it would work, I changed the character to a standard e, and now the tool fails and complains about another character--0xe2--which apparently is an a with a caret (â).  This character does not exist in my DMU table so not sure what that is all about. Here's the full error message:
DescriptionOfMapUnits
Updating E-A stuff
Field: OBJECTID
Field: MapUnit
Field: Name
Field: FullName
Field: Age
Field: Description
Field: HierarchyKey
Field: ParagraphStyle
Field: Label
Field: Symbol
Field: AreaFillRGB
Field: AreaFillPatternDescription
Field: DescriptionSourceID
Field: GeoMaterial
Field: GeoMaterialConfidence
Field: DescriptionOfMapUnits_ID
Pruning unused srcinfo branches
Replacing title and suppplinf
Writing XML to SanLuisObispo100kWest_DescriptionOfMapUnits-metadata.xml
Failed script MetadataFGDC-2...
Traceback (most recent call last):
File "G:\CGS\GM_Work\Arcwork\ArcGIS_Files\GeMS\test\cg_testing\gems-tools-arcmap-issue-80\Scripts\GeMS_FGDC2_Arc10.py", line 503, in
fixObjXML(objName,objType,objLoc,domMR)
File "G:\CGS\GM_Work\Arcwork\ArcGIS_Files\GeMS\test\cg_testing\gems-tools-arcmap-issue-80\Scripts\GeMS_FGDC2_Arc10.py", line 446, in fixObjXML
writeDomToFile(wksp,dom,domName)
File "G:\CGS\GM_Work\Arcwork\ArcGIS_Files\GeMS\test\cg_testing\gems-tools-arcmap-issue-80\Scripts\GeMS_FGDC2_Arc10.py", line 460, in writeDomToFile
dom.writexml(out, encoding="utf-8")
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 1749, in writexml
node.writexml(writer, indent, addindent, newl)
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 814, in writexml
node.writexml(writer, indent+addindent, addindent, newl)
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 814, in writexml
node.writexml(writer, indent+addindent, addindent, newl)
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 814, in writexml
node.writexml(writer, indent+addindent, addindent, newl)
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 814, in writexml
node.writexml(writer, indent+addindent, addindent, newl)
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 814, in writexml
node.writexml(writer, indent+addindent, addindent, newl)
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 814, in writexml
node.writexml(writer, indent+addindent, addindent, newl)
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 810, in writexml
self.childNodes[0].writexml(writer, '', '', '')
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 1038, in writexml
_write_data(writer, "%s%s%s" % (indent, self.data, newl))
File "C:\Python27\ArcGIS10.7\Lib\xml\dom\minidom.py", line 298, in _write_data
writer.write(data)
File "C:\Python27\ArcGIS10.7\Lib\codecs.py", line 708, in write
return self.writer.write(data)
File "C:\Python27\ArcGIS10.7\Lib\codecs.py", line 369, in write
data, consumed = self.encode(object, self.errors)
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 148: ordinal not in range(128)
Failed to execute (MetadataFGDC-2).
Failed at Thu Sep 30 20:49:48 2021 (Elapsed Time: 45.40 seconds)


Consider use cases for difference accounting methods in NPSP

Issue by greenstork from Monday Nov 12, 2012 at 17:49 GMT
Originally opened as https://github.com/SalesforceFoundation/Recurring_Donations/issues/22

Behavior should vary based on whether or not the organization is using Cash basis vs. Accrual basis accounting.  This would be a handy setting to allow our code (e.g. for creating recurring donations) to behave according to the best practices of the accounting method being used.

Payment based rollups for cash based accounting. Good candidate for an external add-on package, which would be preferable to replacing existing rollup functionality.

**lurch: attach W-007266

Tracking W-007266

Included in beta release 3.128 (Beta 7)


Bug: Cant replace default data form_helpers

Direction
form_button('', 'submit', ['type' => 'submit']);
// expected: <button type="submit">submit</button>
// returned: <button type="button" type="submit">submit</button>

Describe the bug
Cant replace default type of form when using extra data.
Also in all of Form Helpers.
CodeIgniter 4 version
4.1.1

type is supplied via the $data param, not the $extra

type is supplied via the $data param, not the $extra

I know,
So, we don't have the option to change the default data via $extra?

In that case, this is not a bug an enhancement. Please send a PR if you can.

Is there a specific use case for this? Why not pass the type via $data?

with $data it may be a habit, without the need to add other attributes such as the second, third and maybe fourth parameters.
but the case is
if $data is only used as 'form name', and the second parameter is 'value'
then in the third parameter ($extra) because you can input an array and maybe its function is the same as $data, then it's not wrong right?

I don't see a PR incoming for this.
Closing for now, as this is a feature request.


Ninja cant handle long filepaths

Building a project with ninja at the moment that i added a submodule my build was broken, because ninja cant handle long paths while msbuild projects can.
In a first attempt i had the project under C:/dev/repo/ and it didnt worked
later i did on C:/ and it workded, showing that the limitation is from ninja.
Both buildsystems were generated by cmake
here are some logs
msbuild cmake generator logs
1> CMake generation started for configuration: 'x64-Release_msvc'.
1> Found and using vcpkg toolchain file (C:/dev/repos/vcpkg/scripts/buildsystems/vcpkg.cmake).
1> Command line: "cmd.exe" /c ""C:\PROGRAM FILES (X86)\MICROSOFT VISUAL STUDIO\2019\COMMUNITY\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\CMake\bin\cmake.exe"  -G "Visual Studio 16 2019" -A x64  -DCMAKE_CONFIGURATION_TYPES:STRING="Release" -DCMAKE_INSTALL_PREFIX:PATH="C:\dev\repos\COLLADA2GLTF\out\install\x64-Release_msvc"  -DCMAKE_TOOLCHAIN_FILE="C:/dev/repos/vcpkg/scripts/buildsystems/vcpkg.cmake" "C:\dev\repos\COLLADA2GLTF" 2>&1"
1> Working directory: C:\dev\repos\COLLADA2GLTF\out\build\x64-Release_msvc
1> [CMake] -- Selecting Windows SDK version 10.0.18362.0 to target Windows 10.0.19041.
1> [CMake] -- Configuring done
1> [CMake] -- Generating done
1> [CMake] -- Build files have been written to: C:/dev/repos/COLLADA2GLTF/out/build/x64-Release_msvc
1> Extracted CMake variables.
1> Extracted source files and headers.
1> Extracted code model.
1> Extracted includes paths.
1> CMake generation finished.

ninja cmake generator logs:
1> CMake generation started for configuration: 'x64-Release-ninja'.
1> Found and using vcpkg toolchain file (C:/dev/repos/vcpkg/scripts/buildsystems/vcpkg.cmake).
1> Command line: "cmd.exe" /c ""C:\PROGRAM FILES (X86)\MICROSOFT VISUAL STUDIO\2019\COMMUNITY\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\CMake\bin\cmake.exe"  -G "Ninja"  -DCMAKE_BUILD_TYPE:STRING="RelWithDebInfo" -DCMAKE_INSTALL_PREFIX:PATH="C:\dev\repos\COLLADA2GLTF\out\install\x64-Release-ninja" -DCMAKE_C_COMPILER:FILEPATH="C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.27.29110/bin/Hostx64/x64/cl.exe" -DCMAKE_CXX_COMPILER:FILEPATH="C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.27.29110/bin/Hostx64/x64/cl.exe"  -DCMAKE_MAKE_PROGRAM="C:\PROGRAM FILES (X86)\MICROSOFT VISUAL STUDIO\2019\COMMUNITY\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\Ninja\ninja.exe" -DCMAKE_TOOLCHAIN_FILE="C:/dev/repos/vcpkg/scripts/buildsystems/vcpkg.cmake" "C:\dev\repos\COLLADA2GLTF" 2>&1"
1> Working directory: C:\dev\repos\COLLADA2GLTF\out\build\x64-Release-ninja
1> [CMake] -- Configuring done
1> [CMake] CMake Warning in dependencies/OpenCOLLADA/modules/COLLADASaxFrameworkLoader/CMakeLists.txt:
1> [CMake]   The object file directory
1> [CMake]
1> [CMake]     C:/dev/repos/COLLADA2GLTF/out/build/x64-Release-ninja/dependencies/OpenCOLLADA/modules/COLLADASaxFrameworkLoader/CMakeFiles/COLLADASaxFrameworkLoader.dir/./
1> [CMake]
1> [CMake]   has 156 characters.  The maximum full path to an object file is 250
1> [CMake]   characters (see CMAKE_OBJECT_PATH_MAX).  Object file
1> [CMake]
1> [CMake]     __/__/OpenCOLLADA/COLLADASaxFrameworkLoader/src/generated14/COLLADASaxFWLColladaParserAutoGen14PrivateFunctionMapFactory.cpp.obj
1> [CMake]
1> [CMake]   cannot be safely placed under this directory.  The build may not work
1> [CMake]   correctly.
1> [CMake]
1> [CMake]
1> [CMake] CMake Warning in dependencies/OpenCOLLADA/modules/COLLADASaxFrameworkLoader/GeneratedSaxParser/CMakeLists.txt:
1> [CMake]   The object file directory
1> [CMake]
1> [CMake]     C:/dev/repos/COLLADA2GLTF/out/build/x64-Release-ninja/dependencies/OpenCOLLADA/modules/COLLADASaxFrameworkLoader/GeneratedSaxParser/CMakeFiles/GeneratedSaxParser.dir/./
1> [CMake]
1> [CMake]   has 168 characters.  The maximum full path to an object file is 250
1> [CMake]   characters (see CMAKE_OBJECT_PATH_MAX).  Object file
1> [CMake]
1> [CMake]     __/__/__/OpenCOLLADA/GeneratedSaxParser/src/GeneratedSaxParserRawUnknownElementHandler.cpp.obj
1> [CMake]
1> [CMake]   cannot be safely placed under this directory.  The build may not work
1> [CMake]   correctly.
1> [CMake]
1> [CMake]
1> [CMake] -- Generating done
1> [CMake] -- Build files have been written to: C:/dev/repos/COLLADA2GLTF/out/build/x64-Release-ninja
1> Extracted CMake variables.
1> Extracted source files and headers.
1> Extracted code model.
1> Extracted includes paths.
1> CMake generation finished.

when building the project with cmake(ninja) effectively it shows some weird errors

Duplicate of #1514.


Enable torch.float8_e5m2 for Ada lovelace

https://github.com/huggingface/text-generation-inference/blob/d0225b10156320f294647ac676c130d03626473d/server/text_generation_server/layers/fp8.py#L4
@Narsil what do you think about enabling torch.float8_e5m2 too for ada lovelace architecture ?
Is there any specific reason why you didn't add both fp8 types ?

i think float8_e5m2 is more adapt for backward, tgi need to only forward pass, so i think e4m3fn is enough to tgi,

Typically forward activations and weights require more precision, so E4M3 datatype is best used during forward pass. In the backward pass, however, gradients flowing through the network typically are less susceptible to the loss of precision, but require higher dynamic range. Therefore they are best stored using E5M2 data format. https://docs.nvidia.com/deeplearning/transformer-engine/user-guide/examples/fp8_primer.html


fix(designer): Allow drag and drop of nodes in/out of scopes

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes/features)
[x] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug Fix

What is the current behavior? (You can also link to an open issue here)
Fixes https://github.com/Azure/LogicAppsUX/issues/2455

What is the new behavior (if this is a feature change)?
Able to drag nodes in/out of scope as long as it doesn't move a node that requires a dependent variable below its original output

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Please Include Screenshots or Videos of the intended change:

Just adding to this, this only adds protection against dragging nodes above their token dependencies, this does not have any logic over dragging token sources below their dependents

Just adding to this, this only adds protection against dragging nodes above their token dependencies, this does not have any logic over dragging token sources below their dependents

yep, I've added a TODO, but we can create a separate issue for tracking as well


Copy layout params from oldEditText

This allows for implementing an EditTextPreference as in Android 9.0's Wi-Fi AP configuration.
(also I find default value 5 to be nonsensical)
Fixes #181.

Instead of putting those attributes into the XML, they should be contained in the class, like this (maybe at the end of the constructor?):
getEditText().setTypeface(Typeface.MONOSPACE);
getEditText().setImeOptions(EditorInfoCompat.IME_FLAG_FORCE_ASCII);

I prefer to do it this way so that user can have control over its style.

@Gericop Please review again. :)

This PR has been updated according to my idea as per #181.

I'll release 1.0.0 now and add this later because it messes with the AutoSummaryEditTextPreference (it crashes now since getEditText() returns null).


Add documentation to add 'openid' scope for OIDC client

Keycloak versions 20 and up require the 'openid' scope for OIDC clients for user search requests.

add a PR to mention this in the docuementation -> https://github.com/camunda-community-hub/camunda-platform-7-keycloak/pull/246

Issue can be closed


Component can't run

after npm run build
there is an error：
Uncaught (in promise) TypeError: Super expression must either be null or a function, not undefined

Having same issue

You don't have to run npm run build to use the library.
Try upgrading to 0.7.* and see if it works for you.


Syncing Change Feed Processor

This change adds GetEstimatedRemainingWork to the ChangeFeedEventHost that was introduced in Nuget 1.1.0.

@ealsur,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@rnagpal Can this be merged? I added the same changes here that were published on the official release.


WFLY-18966: bump licenses-plugin to 2.4.1.Final, delete unneeded entries

Issue: WFLY-18966

Thanks @michpetrov


IdentityPopover Last bought

It show "Last bought 4month ago" when I bought one 14 days ago

Originally posted by @roiLeo in https://github.com/kodadot/nft-gallery/pull/3153#pullrequestreview-1008663687

👋


Do not create new window from provided configuration

to avoid mismatch of source between js and native

When you press facebook button, you can see in xcode log a line:
-canOpenURL: failed for URL: "about:blank" - error: "This app is not allowed to query for scheme about"


Write documentation for user facing classes

https://github.com/bmateusz/zio-memcached/commit/459aaf6cb87cb7c4bf595c61ff20ebc47d589412


Research swap pallet on asset hub

Research assetConversion pallet

[ ] Read source code of assetConversion pallet
[ ] Estimate workload and timeline
[ ] Implement into swap service

Version test: http://<IP_ADDRESS>:8900/s/f37q93wKfz6jmWf
Please check issues:

[ ] Add warning in case low liquidity

[ ] Add logo for Provider

[ ] Improve validate show swap quote

[ ] Show incorrect swap quote rate when the input amount is a small number

[ ] Confirm the default slippage

[ ] Handle the ED when swap

Actual: Swap all then transaction failed: https://assethub-polkadot.subscan.io/extrinsic/0xb70a57d4865f0920cf488d3831e3904c48da235895496f7c943cb5645a86a55e
Expect: Validate ED

Version test: https://5af8730e.subwallet-webapp.pages.dev/

[ ] Show incorrect message when change swap pair

Reproduce: Change from DOT --> USDC to USDC --> DOT with small amount
Actual: Show incorrect message
Expect: If characters > decimals then convert to 0

[ ] Validate incorrect with other provider

Version test: https://c20004c7.subwallet-webapp.pages.dev/
Validation passed in cases:

Swap DOT (Asset Hub) --> USDC: https://assethub-polkadot.subscan.io/extrinsic/0x4ef1e42e466ba543d501f2dd97159790df5d1823a6807ab15314999ef6f4f1df
Swap DOT (Asset Hub) --> USDt: https://assethub-polkadot.subscan.io/extrinsic/0xd55969e4894ae88f9e13d47a1867da0e791f6f034c45494a5467b5b1a39da771
Swap USDC (Asset Hub) --> DOT: https://assethub-polkadot.subscan.io/extrinsic/0x9c7cc47c28713c75742d28697c236a59f6fb433d5c061868456c5ff2c0e1b66a
Swap USDt (Asset Hub) --> DOT: https://assethub-polkadot.subscan.io/extrinsic/0xca56f9cb95a8acdc760202619e12e01ef54d29f8808c0f9b79fc046600e19037
Swap DOT (Asset Hub) with XCM
Re-check all validate

Amount = 0

Insufficient balance

ED of the local token

Change swap pair

Re-check other provider


Add a contributing guide

Assign me please.

Ok so you can either make the readme and docs or code I would tell you to code the filters and you are technically spamming rn

I would like to contribute on README and Docs. There are some duplicate issues that why I requesting both on it.

Assign me please.

done :)


Reset timer on show advance/step_back

Resets the show timer when advancing or stepping back.
I am setting up an attract mode that advances or steps back when the flipper buttons are pressed. Each slide changes after 4 seconds. When rapidly pressing the flipper button, say 5 times, the slide that it stops on doesn't change for 20 seconds or so as if the times are being accumulated. The expected behavior is to change after 4 seconds.
This change resets the next step time to the current time when advancing or stepping back.
Let me know if you have any questions.

Thanks!


CC-25311: Bump postgresql driver version to 42.6.1

https://confluentinc.atlassian.net/browse/CC-25311

The build has been failing for these two tests across different runs:
io.debezium.connector.postgresql.RecordsSnapshotParallelProducerIT.shouldGenerateSnapshotAndContinueStreaming
io.debezium.connector.sqlserver.IncrementalSnapshotWithRecompileIT.stopCurrentIncrementalSnapshotWithoutCollectionsAndTakeNewNewIncrementalSnapshotAfterRestart
Both seems flaky as they are passing when ran locally. But somehow failing in the build.


No matching distribution found for torch-scatter==latest+cu102

📚 Installation
Not able to install the binaries in centOS 8.
Environment

OS: CentOS 8
Python version: 3.6
PyTorch version: 1.5.0
CUDA/cuDNN version: 10.2/ 7.6.5
GCC version: 8.3.1
How did you try to install PyTorch Geometric and its extensions (wheel, source): Binaries
Any other relevant information:

Checklist

[Yes ] I followed the installation guide.
[ Yes] I cannot find my error message in the FAQ.
[ Yes] I set up CUDA correctly and can compile CUDA code via nvcc.
[ No] I do have multiple CUDA versions on my machine.

Additional context
The error message is as follows  - Could not find a version that satisfies the requirement torch-scatter==latest+cu102 (from versions: 0.3.0, 1.0.2, 1.0.3, 1.0.4, 1.1.0, 1.1.1, 1.1.2, 1.2.0, 1.3.0, 1.3.1, 1.3.2, 1.4.0, 2.0.2, 2.0.3, 2.0.4, 2.0.4+cpu, 2.0.4+cu101, 2.0.4+cu102, 2.0.4+cu92, 2.0.5, 2.0.5+cpu, 2.0.5+cu101, 2.0.5+cu102, 2.0.5+cu92)
No matching distribution found for torch-scatter==latest+cu102

Ok, that‘s interesting. Does installing version 2.0.5+cu102 work?

Nope, I still get the same error...

Does it work to install from this link directly?
pip install https://pytorch-geometric.com/whl/torch-1.5.0/torch_scatter-latest%2Bcu102-cp36-cp36m-win_amd64.whl

Yep, this works, although I ran the linux_x86 and with pip I got the message saying  file does not exist.

Yeah, that was a mistake. I wanted to link the linux wheel, sorry! So that‘s working now?

Just realized the pip3 version shipped in centOS 8 in version 9.. upgraded it to 20 and installed the wheels after downloading them.
I suppose installing  the binaries would work fine too


fix: fb feed redirect error

@ArnaudLigny: please, take a look (once again Facebook ruined F-Notifier by making HTTP redirect)

Hello @logeen , sorry for the delay and thanks a lot for this fix.

Damned, the auto-publish script for Chrome Webstore is broken... (https://github.com/ArnaudLigny/F-Notifier/actions/runs/10058357691/job/27801274252#step:6:20)


removed debug print

removed debug print that is added on #6

Coverage remained the same at 100.0% when pulling 201a932a1bbc1de753ff59856dee3fc636f3ca05 on valueshimoda:master into 5617a81cf628ac7b76fc8311b4f43b52b297482f on GoogleCloudPlatform:master.

I signed it!

Thanks!


Modifications to allow Serialbox to be used in CMake superbuild projects

This PR allows Serialbox to be used in CMake superbuild projects,
so that one could have code like
add_subdirectory( serialbox2 )  # --> will set "Serialbox_DIR" in cache
add_subdirectory( project_using_serialbox )

and within project_using_serialbox we have
find_package( Serialbox )  # --> will use Serialbox_DIR as default hint

Most issues were related to use of ${CMAKE_BINARY_DIR} and ${CMAKE_SOURCE_DIR}which reflect the top-level CMake project rather than${PROJECT_BINARY_DIR}and${PROJECT_SOURCE_DIR}` which reflect the most recent project.
As a result it could be that certain products are no longer in a previously expected directory now, as I am not able to test everything

Moreover in this PR I have created alias targets (if possible) Serialbox::Serialbox_C and Serialbox::Serialbox_Fortran that prefer the shared versions Serialbox::SerialboxCShared and Serialbox::SerialboxFortranShared over the static versions Serialbox::SerialboxCStatic and Serialbox::SerialboxFortranStatic
In the client software you can then just use the alias target instead.

Hi there, this is jenkins continuous integration...
Do you want me to verify this patch?

launch jenkins

launch jenkins

launch jenkins

launch jenkins


"Luna" is listed in the drop-down menu

Description:
When user writes "Luna" the location appears in the drop-down menu next to the comune and the county which is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Luna"
Expected results
The location appears in the drop-down menu as "Comuna Luna, CLUJ"


How to get view

Hi,
Is there a way to get active view after change the view? In full calendar docs there is a callback method called viewRender. But I could not find a way to fire it.

Hey, Mehmet.
Thanks for your interest in using my npm package. The package uses the
latest version. 4.0.0 of full calendar and you need to make sure you are
viewing the v4 docs for it.
https://fullcalendar.io/docs/v4#toc
As you can see the callback viewRender has been replaced with
viewSkeletonRender:
https://fullcalendar.io/docs/v4/viewSkeletonRender
ie.

    viewSkeletonRender={info => { this.callbackFunction(info) } }

On Mon, Dec 24, 2018 at 11:43 AM Mehmet Yılmaz<EMAIL_ADDRESS>wrote:

Hi,
Is there a way to get active view after change the view? In full calendar
docs there is a callback method called viewRender. But I could not find a
way to fire it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/SardiusMedia/sardius-fullcalendar-wrapper/issues/1,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AldDoXY6dzh9vcaLvoRzWCzMOM2UfnANks5u8RIzgaJpZM4ZgmSR
.


Missing join train speaker encoder

Hi, thank you for great paper and repo
I got a confusion when debug your repo following your paper in Interspeech 2023.
I saw you said in the paper that you join train speaker encoder alongside VITS model but when I inspect the code, I can not find where you have implemented that join training. As I understand, your code use speaker encoder as an layer in VITS model.
Pls can you make it clear for me?
Thank you !!!

Thank you for visiting our work.
In the paper, we used the expression 'jointly trained' to represent the opposite meaning of 'pre-trained'.
This meant that the speaker encoder would be trained from scratch with the entire model.
And as you mentioned, the speaker encoder is part of our entire model.
Hope this clarifies your question, and feel free to reopen if you have further questions regarding the issue.

Oh thank you for clarifying that, I just thought "jointly trained" mean along side training TTS model, you also training speaker encoder network and join their losses.
I am training your model in Vietnamese language. Can I keep the issue open while the model is not convergence yet ? So I can have your advise for that process?
Thank you!!!


Missing man page for nixos-container

There is no useful online help for nixos-container. nixos-container --help only lists syntax but no semantics. A man page which describes the workings of this tool would be helpful here.

:+1:

nixos-container seems to be deserted in a lot of ways. Probably caused by docker becoming the de-facto tool and machinectl mostly replacing the need for nixos-container. But I agree that better documentation (or at least a deprecation notice) should be added to the tool.

still important to me

@matthiasbeyer nixos-container is implemented directly in nixpkgs, do you want to contribute an initial manpage?
https://github.com/profpatsch/nixpkgs/blob/master/pkgs/tools/virtualization/nixos-container/nixos-container.pl#L1


wcg.rac file is corrupted

There's bad data in the wcg.rac
There's repeated string values 1000 in the file which may be causing parsing errors.
Based on my observation, I think when user.xml is being processed, it lands in our biblepay san/wcg.rac as a corrupt file.
For example:
oncoapop3 showed a RAC of 171000446.350588 yesterday.
It should have been 17446.
and the wcg.rac file was encoded partly as having line endings with CR only (macOS) as line ending.
He worked around the issue by restarting the server. I think quitting biblepay, erasing wcg.rac in san folder, and restarting biblepay will also work
<user>

 <id>1072773</id>

 <name>oncoapop</name>

 <country>CANADA</country>

 <create_time>1533274493</create_time>

 <total_credit>4958657.778454</total_credit>

 <expavg_credit>17

1000

446.350588</expavg_credit>

 <expavg_time>1578311975.659187</expavg_time>

 <cpid>168c62b82fb40fa3db0d34605439daa5</cpid>

 <teamid>35006</teamid>

</user>

The XML from the WCG stats download site doesn't have this issue:
https://download.worldcommunitygrid.org/boinc/stats/

What else?
There are some XML records that are not parsed correctly.
226381 - shows all xml attributes
283635
397932
888260 - randrews :)

Comparing from today noon and today from 7pm. You can see bbpminer4's cpid was split apart by the 1000 injected string. Are you using curl by chance and getting the HTTP headers and output?

What else...
There are some XML records that are not parsed correctly.
226381 - shows all xml attributes
283635
397932
888260 - randrews :)

Do you know this is in the wcg.rac file?
The 1000 string appears every 4096 bytes.
Are you using CURL to retrieve the file? Perhaps turning off http headers and getting the binary file will help.
People are also reporting their CPID are sometimes missing (bbpminer4, barton26) from the leaderboard.

HTTP/1.1 200 OK
Date: Fri, 10 Jan 2020 14:02:45 GMT
Server: Kestrel
Transfer-Encoding: chunked
Expires: Fri, 10 Jan 2020 14:32:45 GMT

Actually, whats interesting is when Oncoapop e-mailed me about this, I took a look at this issue from a different perspective and made the assumption that we had a one time 1000 injected for a different reason; I was not aware the 1000 marker was injected every 4096 bytes.  I just pulled the data today out of my SAN directory and saw the repeated injection.  Thanks for bringing this to my attention.
So the problem is actually, and fortunately not related to biblepays core wallet, or related to the WCG source data.
The problem is that I parse the applicable XML data in by team and eliminate those with non-bbp rac using .netcore, which a cross platform version of .net, because this is the middleware that BMS (DSQL) is written in.  Well it turns out that aspnetcore by default does not have the ability to chunk http responses (it chunks them only one way, with an incompatible buffer).
So what I had to do is add a class of chunking middleware to the dsql server so its responses will be like other web servers, and that did fix the problem.
Please restart the core wallet and now I see looking at wcg.rac the file no longer contains the 1000 markers!

1000 every 4096 bytes are gone
did you change the teams extracted by chance? pollux and barton26 on gridcoin team are missing from the list. They were there yesterday.

Items in the bad news list are incorrect.

Items in the bad news list are incorrect.

#2 wcg.rac is a moving target but I assure you 4 days ago this was the case.
You can review the zip file yourself here:
https://whitewalr.us/wp-content/uploads/2020/01/wcg.zip
#3 is still present. If the wallet parses the XML correctly, its a non-issue. Just takes up more bandwidth and slightly more processing time reading the disk I suppose.
#4 I'll edit 4 out


Question: How to scroll to selection in CodeArea?

Currently I am aware that we need to use requestFollowCaret() to scroll to caret.
However, I am not finding a solution for displaying the selected text (entirely if fits in viewport, otherwise partially).
For most cases it is working fine. However, sometimes, the selected text is just to the left of the viewport and scrolling a bit to the left reveals the highlighted text.
Is there some way I can make sure that the selected text gets displayed inside the viewport?
Here is a sample code of what I am doing right now:
codeArea.selectRange(start, end);
codeArea.requestFollowCaret();

I tried doing the following, but to no avail:
codeArea.moveTo(start);
codeArea.requestFollowCaret();

codeArea.selectRange(start, end);

The use of these selections, for me, is for a find (and replace) option as found in most text editors.

Thanks, I'll submit a PR to address this.
BTW your codeArea.moveTo(start); .... attempt can work but it requires a scene graph pulse to occur between it and codeArea.selectRange(start, end); .... So like a 20ms delay needs to happen between the two, which is a pain.
An alternative which I haven't tried is to change the anchor position of the selection when needed. So if the next selection is to the left of the last selection [use getParagraphSelection( parIdx ) to get the selection start & end relative to the paragraph] then select the word/phrase backwards so that the caret is at the start of the selection.
The above are just suggestions if you can't use the PR or wait until the next release.

I found a simple solution to my problem.
// select the text
codeArea.selectRange(start, end);

// scroll to selection
codeArea.scrollXToPixel(0);
codeArea.requestFollowCaret();

This works perfectly, by first scrolling to the very left, and then scrolling to the caret, making it always display the selection inside the viewport.
Thanks!


Get scroll thumb drag events

Is there an obvious way to hook into the drag events of the scroll thumb? One of the reasons that I am looking to a package that replaces native scrollbars is for the ability to handle the drag events directly. However the core package here blocks a bunch of event propagation so I'm having trouble finding a way to handle drag events. Thanks.
Edit: I should point out that I'm using the React version of this library which may complicate things a bit for cleanly getting drag events.

It took me a while but I finally realized that the class simplebar-dragging is applied to the outer container so that solves my problem. I can detect if that is set during a scroll event to determine if scrolling is triggered by dragging.

It took me a while but I finally realized that the class simplebar-dragging is applied to the outer container so that solves my problem. I can detect if that is set during a scroll event to determine if scrolling is triggered by dragging.

It took me a while but I finally realized that the class simplebar-dragging is applied to the outer container so that solves my problem. I can detect if that is set during a scroll event to determine if scrolling is triggered by dragging.

but if there are two scrollbars on the page, you don't know which one is currently being dragged


HRtP should separate by route, and not by shot

HRtP only has one playable shot, but multiple routes. Therefore, the frontend should display route names and not shot names as filter options.

Fixed by #331


Slow Performance

We have written the needed data into your clipboard because it was too
Issue Type: Performance Issue
I've installed VSC in windows 11 currently. I didn't use any extensions but It is extremely slow. It's not very responsive. I've used it in windows 10.Even there it works fine
VS Code version: Code 1.63.2 (899d46d82c4c95423fb7e10e68eba52050e30ba3, 2021-12-15T09:40:02.816Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i3-1115G4 @ 3.00GHz (4 x 2995)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.74GB (2.01GB free)

Process Argv
--crash-reporter-id a8395a2c-94b6-4765-8f5b-267771525808

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
    1	   102	 18688	code main
    0	   185	  6356	   window (Settings - app - Visual Studio Code)
    0	    23	 10016	   crashpad-handler
    0	    85	 18384	   shared-process
    0	    60	  5472	     watcherServiceParcelSharedProcess
    0	    63	 10020	     ptyHost
    0	    48	  2148	       C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe
    0	     6	 16200	       console-window-host (Windows internal process)
    0	    80	 20644	   extensionHost
    0	    83	  2364	     "C:\Users\yogar\AppData\Local\Programs\Microsoft VS Code\Code.exe" --ms-enable-electron-run-as-node c:\Users\yogar\.vscode\extensions\ms-python.vscode-pylance-2022.1.5\dist\server.bundle.js --cancellationReceive=file:a369630ff5329304eee26737cf46b765dd905cfdc0 --node-ipc --clientProcessId=20644
    0	    64	  6480	     "C:\Users\yogar\AppData\Local\Programs\Microsoft VS Code\Code.exe" --ms-enable-electron-run-as-node "c:\Users\yogar\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\html-language-features\server\dist\node\htmlServerMain" --node-ipc --clientProcessId=20644
    0	    68	 17972	     "C:\Users\yogar\AppData\Local\Programs\Microsoft VS Code\Code.exe" --ms-enable-electron-run-as-node "c:\Users\yogar\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\json-language-features\server\dist\node\jsonServerMain" --node-ipc --clientProcessId=20644
    0	   183	 21056	   gpu-process
    0	    36	 21952	   utility-network-service
    0	    84	 23032	   issue-reporter

large to send. Please paste.

We've also noticed significant degradation in performance after January update. The "Electron" process often takes a pretty good chunk of CPU and stalls basic operations in the editor.
Version: 1.64.1
Commit: d6ee99e4c045a6716e5c653d7da8e9ae6f5a8b03
Date: 2022-02-07T17:26:08.977Z (1 day ago)
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Darwin x64 21.2.0

Can you check if the issue is present with latest insiders https://code.visualstudio.com/insiders/

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Allow more tools in the bottom navigation

Not sure if there's a hard limitation by material, but if not allow the user to add more tools.

Unfortunately this is a limitation of the bottom navigation. If I wanted to support more than 5, then I would need to create my own bottom navigation component which I'm not sure I want to do right now.
java.lang.IllegalArgumentException: Maximum number of items supported by BottomNavigationView is 5. Limit can be checked with BottomNavigationView#getMaxItemCount()


Ignore blank repl input

I noticed that when I entered an empty string in the REPL input box at the bottom I got an error and a debug box on the right.  I found a place where I could fix this but I'm not certain this is the correct place to fix it. I can't imagine this breaking any legitimate usage.

I realized that what I really wanted when I hit <enter> was to add a blank line to the output to space out future results. This may be a bridge too far and I can remove it.


Enable the ability to create a WebGL rendering context with options

The current method for creating a WebGL session createWebGLRenderingContext() simply tries to default a WebGL EGL/GL context and session with the best options. It would be nice to allow for specific versions and potentially extensions to be enabled/disabled at creation.

Sane defaults are preferred for beginners, so would love to keep this convenience (many of us are interested in using this for DL tasks with TF.js, and have no clue about WebGL specifics) -- I agree that advanced users should be able to define.
I would love to see TF.js users being the primary target audience with the general GL audience being served, too, but not at the expense of TF.js audience. May be hard to address two audiences but splitting along the lines of beginner and advanced is reasonable, IMO. My $0.02

Sure - this package will be an dependency to a TensorFlow.js backend integration. The focus on this library is to provide headless GL ES/Compute Shader functionality to Node.js.

Adding width & height to the options would be great

@AlexVestin just published 0.0.14 of node-gles w/ your changes


After upgrading google_sign_in it doesn't work as promised anymore

Steps to reproduce
I am currently using  google_sign_in: ^6.2.1. Everything was working fine before. Also have read the migration guide at https://pub.dev/packages/google_sign_in_web It says signIn() will still work but will only give me the accessToken.
Probably I would just need a new guide for Firebase google sign in implemenation but there is only few documenation on this.
I also tried to use the old version but there I get the error popup_closed_by_user
Expected results

A user is clicking on the google sign in button
A auth pop up is opening
The user is giving access to its account

Actual results

A user is clicking on the google sign in button
A auth pop up is opening
The browser console says: popup_closed with no further explanation. This error is also not explained at https://developers.google.com/identity/sign-in/web/reference?hl=de#error_codes_2 only popup_close_by_user

Code sample
Code sample
@override
  Future<Either<String, auth.User>> signInWithGoogle() async {
    try {
      final GoogleSignInAccount? googleUser = await GoogleSignIn().signIn();

      final GoogleSignInAuthentication? googleAuth =
          await googleUser?.authentication;

      final credential = GoogleAuthProvider.credential(
        accessToken: googleAuth?.accessToken,
        idToken: googleAuth?.idToken,
      );

      final result =
          await FirebaseAuth.instance.signInWithCredential(credential);

      final newUser = result.user;

      if (newUser == null) {
        throw Error();
      } else {
        return right(newUser);
      }
    } catch (e) {
      print("Error");
      print(e);
      return left(WebserviceErrorHandler.handleError(e));
    }
  }

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.24.0, on macOS 14.6.1 23G93 darwin-arm64, locale de-DE)
[!] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/to/macos-android-setup for more details.
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2024.1)
[✓] VS Code (version 1.92.2)

Issue related to firebase_auth

Issue related to firebase_auth

Issue related to firebase_auth


submodules: add rule and update them

I forgot to add the new rule to help. I'm doing it now.

:+1:

merged, thanks


Make a button, don't just attach to click handler

While we can continue to expose a function to just pop the modal via JavaScript as show() does now, the default UX in my opinion should be that we create the button on behalf of the client, to ensure that it works and feels right.

@osdiab That's sound great! How do you think we should expose the button? I mean, maybe a  FAB fixed to the host page? Or expose a button that people can use adding some code in their pages?

I'm thinking a couple ways:

provide a selector to a div you wanna put the button in, and then render it there

perhaps a more foolproof option as well is to provide a copy-pastable chunk of code for people to paste wherever they want to put the button

provide a react/preact component (more relevant once we have #48 )

Also relevant is #1 , but doesn't have to happen in a PR that implements this


New style sheets for plots

I've set astroplan's default style sheet to be the astropy.visualization default style sheet (update: I've undone that, see later notes), and created my own version of the astropy style sheet with a dark background for the "night mode" that we've long sought after. This should close #74. For example, here's what the airmass plot looks like now in "light" mode:

and in "dark" mode:

I've added a note in the plotting tutorial on how to switch between them.
cc @jberlanga @ejeschke @adrn

This looks good to me. It would be nice to be able to see some more examples of plots in dark mode, to make sure that it works.
@bmorris3 – Maybe you can set dark mode in docs/conf.py and produce a version of the docs with all plots in dark mode and put it online? Then we have plenty of example plots to see if dark mode works or if there are any kinks.

@cdeil I've built the docs with all dark themes here, take a look!
I managed to change the color cycle for lines, which is why the lines that use default colors in the dark theme are readily visible on the dark background. One of the problems I've been having is that it seems that there's no way to change the default color cycle for plt.scatter markers. Do you know if there are any style sheet tricks for that?

The dark plots look good, pretty cool!
The visibility of blue on black is not great, but OK.
I don't know any tricks for color cycles in general, and for plt.scatter in particular. :-)

@bmorris3 – Is this ready for final review?
@jberlanga @eteq @adrn @bsipocz Anyone else want to review this before it goes in?

@cdeil It is, and I'd appreciate the review! Let's see if anyone responds in the next day, I guess?

I just thought about this again: probably we should follow the recommended MPL pattern of adding styles (http://matplotlib.org/users/style_sheets.html) instead of adding a style_sheet option to all plot functions?

@bmorris3 re scatter colors: you can define a color cycle with either a list of strings or a number sequence, and pass it to the c parameter
c : color or sequence of color, optional, default : 'b'
    `c` can be a single color format string, or a sequence of color
    specifications of length `N`, or a sequence of `N` numbers to be
    mapped to colors using the `cmap` and `norm` specified via kwargs
    (see below). Note that `c` should not be a single numeric RGB or
    RGBA sequence because that is indistinguishable from an array of
    values to be colormapped.  `c` can be a 2-D array in which the
    rows are RGB or RGBA, however.

btw the default is stored in the u'axes.color_cycle' and is  [u'b', u'g', u'r', u'c', u'm', u'y', u'k'] on my setup of mpl 1.4.3.

@bmorris3 : I do like how the night/dark plots look--supah nice!
Apologies for a late or no response on several PR's now--I don't have time to look at code in PR's at the moment (I have to get my thesis to my committee in 12 days, and I defend in a few weeks), but I'll have lots more time at the end of November.

@bsipocz - The axes.color_cycle parameter changes the colors of lines, but not plotted points, it seems.
@jberlanga Thanks, and no worries! I'm trying to get things tidy before the conference starting Nov 2, so there will be a flurry of activity. Any cursory reads and thumbs-up that you can manage are appreciated but I won't nag.
@cdeil I agree -- in the most recent commit, the astroplan.plots

@bsipocz - The axes.color_cycle parameter changes the colors of lines, but not plotted points, it seems.
@jberlanga Thanks, and no worries! I'm trying to get things tidy before the conference starting Nov 2, so there will be a flurry of activity. Any cursory reads and thumbs-up that you can manage are appreciated but I won't nag.
@cdeil I agree -- in the most recent commit (d1efcce), the astroplan.plots methods don't do anything to matplotlib.rcParams unless the style_sheet keyword argument is set to something other than None. No style sheet is applied unless one is supplied.
Aside, for @cdeil -- I think the best way to manage the styles would be to use the matplotlib.style.context manager within the astroplan.plots functions. In mpl1.4.3 the context manager doesn't allow custom user-defined styles to be applied via the context manager, but in the source version on GitHub the context manager does allow custom style sheets to be used. Once that's been released, I'll probably stick all of the contents of each astroplan.plots function within one of those context managers to apply the styles to only those individual plots. That way interactive sessions won't get polluted by our style sheets, as they're likely to at present.
Once the tests pass, I'll merge.

@bmorris3 - OK, now I went back and check the ancient script that dealt with a bit non-advanced coloring of scatter plots, and indeed it didn't used a modified axes.color_cycle, but assigned a color to each and every point in a for iteration.


when i using the databox in Andriod,Authorization required is occured

Hello,
I accessed the databox in Andriod app,I enabled the mobile sensor data and I installed the driver sensingkit for light graph app,
After installation Authorization required error is shown.

Android OS version would be helpful. @ktg any advice on how to fix this?

hello,
I tested Databox in Nexus ,Android version is 6.0 .
When I accessed the Databox,there was needed the security password for device .