method error with Distributions.BinomialGeomSampler when implementing new particle filter

Hi!
I'm trying to modify the particle filter example for a simple SIR model in discrete time (a Markov chain). When using ForwardDiff.gradient, I get the following MethodError:
ERROR: MethodError: no method matching Distributions.BinomialGeomSampler(::Int64, ::ForwardDiff.Dual{ForwardDiff.Tag{var"#106#107", Float64}, Float64, 2})

It arises from the line where the dynamics are applied to each particle to update their trajectory X = map(x -> dyn(x,p,Δt), X). I've posted my working example to a gist here: https://gist.github.com/slwu89/a165175c39cdf6ff9744df892238ac03
If you have any time to take a look or suggest changes I'd really appreciate it. Thanks.

Our example had continuous dynamics and only the resampling step was discrete. In your example the resampling step is also discrete. The smoothing backend which will take care of this is not yet merged, but we are working on that. In the mean time, maybe it helps converting the Dual into a stochastic triple, sampling the discrete step and smoothing the result back on a dual works. We can look into this together.

Thanks @mschauer, yes both the dynamics and resampling are discrete in my case. Ok, thanks for the advice! To be specific, do you mean doing the conversion right before the particle filter calls the dyn function to run the dynamics of the model? And to smooth the result, you recommend to use the stop-gradient method?

@mschauer interestingly my SIR example (with the SIR discrete resampling dynamics unchanged) works when taking the derivative of a simple function which returns the total epidemic size (size of the R compartment after the transient dynamics end). Using StochasticAD.derivative_estimate returns a reasonable answer (derivative of final epidemic size wrt contact term is positive and recovery term is negative).
using StochasticAD
using Distributions
using DistributionsAD

@inline function rate_to_proportion(r, t)
    1-exp(-r*t)
end

function dyn(x, p, Δt)
    S,I,R = x
    β,γ = p
    N = S+I+R
    ifrac = rate_to_proportion(β*I/N,Δt)
    rfrac = rate_to_proportion(γ,Δt)
    infection = rand(Binomial(S,ifrac))
    recovery = rand(Binomial(I,rfrac))
    return [S - infection, I + infection - recovery, R + recovery]
end

function total_epidemic_size(nsteps, x0, p, Δt)
    x = copy(x0)
    for n in 2:nsteps
        x = dyn(x, p, Δt)
    end
    return x[3]
end

const p = [0.5, 0.25]
const Δt = 0.1
const x0 = [990, 10, 0]

nsteps = 400
tmax = nsteps*Δt

StochasticAD.derivative_estimate((x) -> total_epidemic_size(nsteps, x0, x, Δt), p)

Your implementation above, with concurrent discrete random steps for S I and R, should be unproblematic for stochastic triples 🙂

Hi @gaurav-arya! The particle filter in the github gist or the small thing above just summing up the total epidemic size? I now understand the issue was because both the model's dynamics were discrete and the resampling step in the particle filter was discrete too. Anyway, I'll rebuild the latest version of StochasticAD and see if it works now!

Sorry, I was looking at just your small snippet above, not the full particle filter you originally sent. In #70 I've added new_weight support for stochastic triples, so that the unbiased approach can be used with them too.
However, you're indeed correct that discrete randomness in both the model dynamics and the resampling step simultaneously is something that we have not explored in depth with stochastic triples. I did try to get smoothing via ForwardDiff to go through for your example, however smoothing the (integer) number of trials parameter in the Binomial proved tricky to do with duals; something to revisit in the future!

@gaurav-arya is this something I could potentially help with? It's been a few months since I read your paper describing the methods so I'll have to reread that, but in the meantime are there any places in the code to check out?

Hey! Sorry for the late reply here. If you're interested, I think one interesting direction to go in would be to push through the smoothing approach to work on the particle filter. The tricky part would be getting something like this to work:
function f(p)
           y = rand(Binomial(10, p))
           z = rand(Binomial(y, p))
end

ForwardDiff.derivative(f, 0.5) # throws an InexactError

A couple places to look in the code are the ForwardDiff rules and this hack to get stochastic triples to work with the n of a binomial.
I also do hope to get a stochastic triple-based smoothing approach implemented soon, which should be more flexible in terms of propagating through discrete constructs -- once I've done that I could also let you know if you'd like to play around with it for the particle filter:)

Thanks @gaurav-arya! Okay, I'll look around in those places you suggested. Also, I'll keep on the lookout for your update, which may happen before I get time to check out the code in detail again.

@gaurav-arya I'm somewhat coming back to this, it looks to me as if that "hack" is already implemented for perturbing the n of a binomial? I'm looking at this: https://github.com/gaurav-arya/StochasticAD.jl/blob/main/src/discrete_randomness.jl#L127 Could you describe in a little more detail exactly what needs to happen to get that working?
Also, I somehow accidentally deleted the gist with the original SIR particle filter, I recoded a new one here: https://gist.github.com/slwu89/f6d0c595cea9cc1a86acda470709b140


9/4 PM Publish

Docs Build status updates of commit fe3bbfc:
:clock10: Full build: the average full build time is 22 min(s) 22 sec(s), this is based on the last 197 build(s) for this repository.

Docs Build status updates of commit fe3bbfc:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-conceptual/azps-1.8.0/release-notes-azureps.md
:warning:Warning
View (azps-1.8.0)
Details

docs-conceptual/azps-2.6.0/release-notes-azureps.md
:warning:Warning
View (azps-2.6.0)
Details

azuresmps-4.0.0/include/sb-deprecation.md
:warning:Warning
View (azuresmps-4.0.0)
Details

azuresmps-4.0.0/include/aa-deprecation.md
:warning:Warning
View (azuresmps-4.0.0)
Details

azuresmps-4.0.0/Azure/Publish-AzureWebsiteProject.md
:white_check_mark:Succeeded
View (azuresmps-4.0.0)

docs-conceptual/azps-1.8.0/release-notes-azureps.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.
[Warning] Missing attribute: title. Add a title string to show in search engine results.

docs-conceptual/azps-2.6.0/release-notes-azureps.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.
[Warning] Missing attribute: title. Add a title string to show in search engine results.

azuresmps-4.0.0/include/sb-deprecation.md

[Warning] Missing attribute: title. Add a title string to show in search engine results.

azuresmps-4.0.0/include/aa-deprecation.md

[Warning] Missing attribute: title. Add a title string to show in search engine results.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Use ISO 8601 datetime for logger output

previously:
03/22/2019 09:06:59
now:
2019-03-22T09:11:34

I'd like the T to be removed, makes this hard to visually parse.


Release 4.16

This is a tracking issue for release 4.16.
Please keep feature requests in their own issues
If you want to make a comment on a particular change, please make the comment in the "Files changed" tab so comments are not lost during a rebase.
List of changes for release:

[ ] Add 'json escape' setting #3268 #3269
[x] Improve error message when autoloading invalid view engine #3251
[x] Skip Buffer encoding when not generating ETag for small response
[x] Support Node.js 8.x
[ ] Upgrade content-type module to 1.0.4 to bring in performance updates
[ ] Upgrade etag module to 1.8.1 to bring in a performance update
[ ] Upgrade finalhandler module to 1.1.0 to bring in a performance update and no _header usage
[ ] Upgrade parseurl module to 1.3.2 to bring in a performance update
[ ] Upgrade proxy-addr module to 2.0.2 to bring in a performance update
[ ] Upgrade qs module to 6.5.1 to bring in a performance update
[ ] Upgrade send module
[ ] Upgrade serve-static module
[x] Upgrade setprototypeof to 1.1.0
[ ] Upgrade utils-merge to 1.0.1
[ ] Upgrade vary to 1.1.2 to bring in performance update

Testing this release
If you want to try out this release, you can install it with the following commands:
$ npm cache clean express
$ npm install expressjs/express#4.16

Owners/collaborators: please do not merge this PR :)

Due to #3432 I'm likely going to cut another 4.15 release, which means I'll need to rebase this again, though I still haven't finished the rebase from the first security injection :)
I also looked harder at the mime dependency on if there is any possible way to get it into a new 4.15 release, but that does not seem feasible without violating semver, since it would be a lot of work (almost to re-implementing mime) to undo the semver changes that was included in the release.
Getting a new mime 1.3.x is also not workable, because we are on 1.3.4 and one of the patch versions above that includes a change that is semver-minor to us, so there isn't a version number available for a surgical 1.3.x release of mime for us anyway.

Due to the influx of issues related to mime and the logistics of getting forwarded to no longer include < 0.1.2 in the semver range, I think this release should be accelerated, which will likely mean not including changes that were originally slated for 4.16 in order to get it out the door. This should allow a quality release of 4.16 shipped Thursday Sep 28 instead of Monday Oct 2.

Just read thru the entire changeset and it looks stellar ✨ just in case you wanted another pair of 👀 on it 🙂


fix #75: cross-browser support

Safari & Firefox have a different result for the constructor.toString() method than Chrome/v8.  Specifically it has a few extra spaces/newlines.  This change makes the comparison support both types.  Seems to be working fine in Safari/FF after this modification.

May want to re-run the minifier on sift.min.js


feat(popover): support template url for partial

As requested in #202 by @joshdmiller here's my request for external partial support for popover which would make popover much more versatile and lower the need of duplicate code in the apps. Already working implementation can be found in one of the "competing" Bootstrap implementations at http://mgcrea.github.com/angular-strap/#/popover.

I got fed up waiting for this feature, so I made the ngPopoverButton directive. Posting it here in case some of you guys would find it useful.

can't believe this thread is still open!

What is preventing the PR to be merged (apart from conflicts) ?
I'd be glad to help fixing the last issues!
It needs to be merged asap because this is becoming hilarious (1 year issue, with 200+ comments, on a 7k stars project, just to allow html in a popover...)

@QuentinFchx https://github.com/angular-ui/bootstrap#impt---2015-plans-and-angular-13-support

The 0.12.1 milestone only contains the https://github.com/angular-ui/bootstrap/pull/2996 PR tagged as needs:merging. How can I help to make this happen ?
This will allow us to move forward toward 0.13.

We'll be merging that issue at the next maintainers meeting.  @chrisirhc will be teaching the rest of us new maintainers to cherry pick the fix and perform a full release.  Then we get onto the hard work of triaging the ~120 or so PR's out there :-).

@karianna :+1: This is great news for this library.

That is such an enormous relief. I was under the impression that the project had seriously lost momentum but it looks like I'll soon be able to start recommending it again.

Well I'm not going to promise miracles as there are 187 outstanding Pull Requests to get through for starters, but yes you should start seeing some movement over the coming weeks.

Marvelous !

Any progress? I understand guys are working purely part-time and unpaid and I appreciate that.
But I think this issue is blocking many of us.

Meeting tonight to begin the first steps.

Hey guys? Did you got a chance to talk about this?

No, our priority is to get Angular 1.3 compatibility - next step is to triage 200 open PRs, step after that is to triage 300 open issues.  Then we look at the rest :-)

I do not comprehend. Been using 1.3 since beta without a problem. The amount of response here is ridiculous.

Wait, so this wasn't discussed at all? I appreciate that you're trying to get this project back on track after it completely fell of the rails for whatever reason, but this issue is coming up on it's 2 year anniversary (and it wasn't even the first to request this functionality), all the while people are forced to use other libraries like angular-strap, fork this repo and make the fix themselves, or do crazy duck-punches in their projects, all for what is a relatively simple change that has a PR ready to go. If this doesn't come through in 0.13 it's going to be hard to justify recommending this library to anyone anymore.

Guys, use angular strap (http://mgcrea.github.io/angular-strap/)  until this issue will be fixed. The same number of features + popover has HTML

@forza020 it's not a clean switch. You can't put templates / html in your dropdown menus cleanly, for example. You just have to consider what features you need and pick the best one based on that; it's not as simple as "they still don't have feature x, just go use y."

I know. That is why the only way is to combine these 2 modules until
bootstrap ui will not support HTML in popover.
2015-02-25 15:05 GMT+02:00 Kyle J. Kemp<EMAIL_ADDRESS>
@forza020 https://github.com/forza020 it's not a clean switch. You
can't put templates / html in your dropdown menus cleanly, for example. You
just have to consider what features you need and pick the best one based on
that; it's not as simple as "they still don't have feature x, just go use
y."
—
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/bootstrap/issues/220#issuecomment-75956127
.

This was in 2013, today 2015 still without any update about it. This is the reason why people great forks and boom a new javascript framework because even when they take the time for create the PR for contribute.
@forza020  I will take a look to angular-strap, hopefully is a good route to go.

These comments aren't adding anything to the issue, I recommend locking this issue to contributors, and updating it when the feature is added.

@danielcompton you suggestion is shut down the opinion of the people. This add value to the issue, you know why, because you can see the effort from them. People like me trust them and the fact some guys even create the PR + Unit Testing + Doc and they don't do anything about. This was 2 years ago sir, this wasn't 1 month and look the how many +1 are here ..... Close the comments, "good way" of fix problems

@yordis I think we're all well aware of the need for this feature, and I too am frustrated that it's not available yet. However GitHub issues aren't the place for complaining about a bug or missing feature, they're for fixing it. The negative comments aren't adding anything of value, and they're bringing down the contributors. If you want something to change, your options are:

Become a contributor to angular-ui, and land the patch in a way that fits with the contribution guidelines
Do nothing and wait for the feature to arrive sometime in the future
Use a different UI library with this feature

If you just want to complain, then I suggest you save it for your bartender :wink:.

This issue is a tough one, largely because it's been buzzing around for so long. I've been debating the merits of closing this for I don't know how long, largely due to a couple of significant fixes for this issue as open PR's. And since I'm not actively contributing to the project right now, it's debatable whether or not I should.
That being said, this project lost momentum as many of the maintainers had to real-life obligations and other projects take over. As a result the current/new maintainers have a lot of work ahead of them with some 200 PR's and over 500 open issues.
As a result, I'm going to close this for now (with the caveat that a current maintainer is more than welcome to re-open for any reason). Not because I have any desire to close down discussion, but because this horse has been dead a long time and it's less a matter of needing a fix and more a matter of needing to fix other things in order for this to get merged in.

@hallister surely in the time it took you to write that comment you could've just merged in one of the multiple open PRs for this (#1848,#2479, etc.)

@DaveWM Both of those have merge conflicts that need to be sorted out and I'm not actively contributing to the project right now, so it's highly unlikely that I could have merged it quickly.
Furthermore, because there are so many outstanding PR's, merging into master right now is going to do nothing more than cause more problems. With 200 open PR's, a great deal of care has to be taken on what gets merged.
I realize everyone is frustrated with the pace here, but this is still an open source project developed entirely by volunteers. I realize this issue has been around for a long time, but it's going to take a little more patients before this is merged in.

@DaveWM @hallister  My thoughts are this should not be part of the library.  We should be focusing on making directives that have a single responsibility that people can use as components to build more complex directives.  We shouldn't build directives that transcludes 2 different content areas, handles dismissing, and handles popup placement all in one.  Rather these should be discreet directives (dismiss, placement, etc...) that are independent and can be re-used.  That being said, here is a plunk that demonstrates that approach for the popover.  You will notice I am using the native TWBS markup and classes to build the popover and only have a single directive that handles the dismiss.  No need to transclude content, or trigger a scope.$apply to reposition the element after content changes, or the numerous other rabbit holes we go down to support a 'god' directive.  While it is convenient not to have to write all that html, that should be a convenience reserved for simple cases.
I am also working on an approach for more placement options by using classes (can be bound to ng-class for dynamic placement), which can be seen in this plunk.  This does not use pixels to set top, left, right, or bottom so as content grows, the container will resize without losing the anchor point.
If you are set on template support in the existing popup directive, I have a pull request here with all conflicts resolved against the current master branch,

@RobJacobs I strongly disagree that this shouldn't be part of the library, and clearly so do many others. If the goal of this library is to recreate TWBS functionality with Angular-style markup, anything like this should definitely support templates at the bare minimum.

Completely agree with @seiyria. @RobJacobs, you seem to be saying that the entire popover directive should be re-written, and regardless of whether that's a good idea or not, it's not really what's up for discussion here. Also, I hardly think adding something as simple as dynamic content would make popover a "god directive".

Interesting thing, but it looks like popover template worked before. I found working plunkr [http://plnkr.co/edit/5oX2lA?p=preview] with angular 1.2.9. However 1.3.x doesn't work.
As an alternative, i found nsPopover https://github.com/nohros/nsPopover. Going to give it a try

Since my previous comment on 28 May 2014, I've been away. Surprised to see that this is still an issue, I have just made a new version, mergeable to current HEAD.

See it in action
Get the ready-to-use code here or here

There is a difference now. While I was not expecting merge for my previous versions, because I knew they dissonate from the project's "roadmap", so to say... this time the patch does not affect absolutely nothing on existing code and embraces the design decisions taken. (While I previously made it NOT destroy the tooltip when it was hidden, now I kept the tooltip-removal code untouched.)
This is why, unlike many of my previous attempts, at this time I am opening a new PR. Just as I knew chances of merging were poor in the past - this time I sincerely believe there is no reason for not being embraced. The code is absolutely simple, unobstrusive and harmless. Please, take a look and consider. Thank you.

I want to cry. Thanks everybody.

FYI - Actually resolved in 0.13.0 but due to GitHub DDOS I can't change the milestone

Will it be part of 0.12.2 release?
What is the expected release date (or month) of 0.12.2 or 0.13.0?

There won't be a 012.2, 0.13.0 is the next release, no concrete ETA yet.

fantastic !. any ETA yet ?

just wait the release version, the issue last for too much time

Why is this not released yet? The popover is mostly useless without the ability to use templates. Might as well just use tooltips.

+1

+1

Locking this issue as the feature has been released. Please open a new issue if there are further comments.


bug(#120): try to update to 0.49

Ref: #120

@maxonfjvipon Thanks for the contribution! You've earned +4 points for this: +4 as a basis; -8 for too many hits-of-code (269 >= 200); -16 for the lack of code review; -16 for absolutely no comments posted by reviewers; +40 to give you at least something. Please, keep them coming. Your running balance is +8.


multi-module use cases are completely broken: could not open compilation database

multi-module use cases appear to be completely broken. if i have a module named A with example code:
// A.swift
public
enum A
{
    struct Foo
    {
    }
}
extension A.Foo
{
    func test(x _:Int)
    {
    }
}

i get normal highlighting and LSP features, though it emits an error to the console:
2022-11-13 18:22:26.497 sourcekit-lsp[393:402dc700] could not open compilation database for /example/Sources/A/A.swift

if i go to another module that imports A, sourcekit-lsp doesn’t seem to work at all.
// B.swift
import A

extension A
{
    struct Bar
    {
    }
}
extension A.Bar
{
    func test(x _:Int)
    {
    }
}

it also emits a similar error to the console:
2022-11-13 18:27:00.923 sourcekit-lsp[393:402dc700] could not open compilation database for /example/Sources/B/B.swift

but because it also emits the same error in A.swift, i am not sure if the error is related to this issue.
here is a full example project, including a .devcontainer directory for reproduction:
vscode-example.zip
the specific toolchain i am using currently is swift-DEVELOPMENT-SNAPSHOT-2022-11-11-a, but the problem has been going on for most of november.

more forum context: https://forums.swift.org/t/vscode-swift-getting-really-flaky-again/61403/2

rdar://102321349

When trying to open the project in the devcontainer locally, I’m getting the following error. Do you know what I might be doing wrong here?
ERROR: The Compose file is invalid because:
Service swift has neither an image nor a build context specified. At least one must be provided.

@adam-fowler Is the devcontainer setup something that the Swift VSCode extension supports (and that is actively being used)? Could it be that the VSCode extension is building the package on the host (with a toolchain from the host) but writing the build products to the containers build directory?

@ahoppen Swift extension works fine with devcontainers, well at least up to swift 5.7. The error message implies there is something wrong with the devcontainer setup in the example. Maybe @kelvin13 can help

i don’t get that error when i open the example project locally, but according to this SO question it could have to do with the service name swift (which is a common service name, i imagine) conflicting with a docker-compose.override.yml file somewhere on your system.
so maybe try renaming the service name swift (in docker-compose.yml and devcontainer.json) to something more unique?

AFAICT there’s nothing actionable for sourcekit-lsp here, so I’m closing the issue. Feel free to re-open it if you believe there’s a SourceKit-LSP issue that needs to be fixed.


WCT ignores CLI config for polyserve (proxy)

I'm using polyserve with additional CLI params:
polyserve --proxy-target https://myredirect:8282/foo --proxy-path api/v1

(as suggested in polyserve --help)
So I'm able to test it manually in the browser. All tests are running and passing as expected.
Unfortunately, WCT disregards those and does not forward it to polyserve
wct --proxy-target https://myredirect:8282/foo --proxy-path api/v1

Also, I see no way to specify it in wct.conf.json https://github.com/Polymer/web-component-tester/blob/9e75caa/runner/config.ts#L35
I'm running wct version 6.1.5.

@usergenic Could we get it merged to master? I could write the test for this parameter, but I need some support, see https://github.com/Polymer/web-component-tester/pull/608#issuecomment-338385500


Update ml_concepts.md

d2 : delete header : the idea is that each video is independent. If the mooc has to evolve this video can become the 2nd or 3rd without modifying neither your slides nor your text
d9 : I propose to list the different information to follow an eventual drawing, or just give time to moocers to discover the board before focusing on the rest of the speech
d15 & 19 : maybe add a little conclusion
d16 : for neophytes like us, could you describe or give an example for "Then, give a new individual, we predict the income of its closest match in our database."
d21 : sorry, but for neophytes likes us : we did not all understand the same thing: is Generalizing to be associated with the test phase and Memorizing with the training phase? or just as Generalizing and Memorizing are different, as are the train and test data.
d25 & 26 : could you suggest an example without x and y : s
I try to propose examples with irises but I'm not at all sure of my proposal...
d27 : What do you think about adding 2 graphics to better visualize?
Sur certaines slides : nous avons proposé des petites modifications pour essayer d'accentuer un peu plus l'enchainement de tes parties. Faire des points ou des petites conclusions. Mon anglais est très approximatif et mes connaissances en machine learning inexistantes, donc ces phrases sont sans nul doutes à reprendre, mais j'espère qu'elles te donneront des idées et te faciliteront le travail.

I made a couple of tiny comments and would be happy to know what you think. Aside from this, your changes look great and I will merge them as soon as you tell me your thoughts on header and video/lesson.

Merging. I will address the comments


Fixed small typos in README.md

small disclaimer: I don't have a C# development environment set up so I could not compile or runtime test the examples.

Hi @mblumenstingl-n11
Thank you for PR.


Sadness Chemical

About the PR
 Adds Sadness to the game (pending the inclusion of a means to obtain it)
Why / Balance
 For a new plant and new mutation by a friend
Technical details
 I think the guidebook is automatic?
Media

Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

 You are missing the loc strings for the reagent name and description at the moment, they should be added to the corresponding ftl file.

If you want to add the reagent to the game without waiting on a plant mutation you could add a chemical reaction for it or add it to the liquid anomaly.

The guidebook entry is automatically generated for each reagent if the prototype has no errors, so you should be able to see it in-game. You are missing the loc strings for the reagent name and description at the moment, they should be added to the corresponding ftl file.

Whoops, I'll do that right quick

If you want to add the reagent to the game without waiting on a plant mutation you could add a chemical reaction for it or add it to the liquid anomaly.

Fair, I could make it Happiness + Ethanol or some shit

If you want to add the reagent to the game without waiting on a plant mutation you could add a chemical reaction for it or add it to the liquid anomaly.

Fair, I could make it Happiness + Ethanol or some shit

I cannot find the Liquid Anomaly Chemical Table

I believe this works, time to ask the checks

I don't understand but I'll read up on it and come back later

Did you make any webedits or did you use an IDE and git? You seem to have used the wrong new line settings, which should not happen if your git client is configured correctly.

Did you make any webedits or did you use an IDE and git? You seem to have used the wrong new line settings, which should not happen if your git client is configured correctly.

I did webedits exclusively, I thought it would be easy and quick.

Did you make any webedits or did you use an IDE and git? You seem to have used the wrong new line settings, which should not happen if your git client is configured correctly.

I did webedits exclusively, I thought it would be easy and quick.

You didn't test this in game at all?

I did webedits exclusively, I thought it would be easy and quick.

Yeah, please don't do that. Like mentioned in the PR guidelines they are prone to errors and all changes should be tested in-game.
Have a look at our guides for setting up your IDE and using git to learn how to do this properly. If you need help with any steps, feel free to ask on our discord server.

I'll just redo all of this in my dev environment, I don't like it as much but if I gotta do it that way then so be it.

Alrighty, closing this for another PR.


AdminLTE conflicts with vue that comes with laravel

I am wondering why I can't use vue that comes with laravel , I think there is issue with adminlte

The issue is that webpack requires a specific way to set the path of assets in LESS. So it's failing to figure out where the boxed-bg image is. If you or anyone finds a way to make this work cross platform, I will implement the solution in AdminLTE. So far, I am not aware of a way that allows this to work for WebPack and not fail everywhere else.
Thanks

The fast (and not the best) solution is to use this setting:
mix.options({
  processCssUrls: true,
  imgLoaderOptions: {
    enabled: false,
  }
});

I got it to work.
In my case (with Laravel) a binary file compile is failed on npm install and caused this "I don't find that  boxed-bg image" issue.
After running
npm rebuild mozjpeg
turned out that nasm is missing from the server. After install the compile works as a charm.


write to a separate source map file

right now, you can only inline source maps. any way to allow separate source map files?

https://github.com/cssnext/cssnext#map-default-depends-on-sourcemap
Is this enough for you ?

sorry. has to be more specific. i'm currently trying via CLI w/ the watcher. trying to avoid a custom node.js script :)

Hum so I think I've done nothing yet. Maybe we should handle #73 to allow options from json ?

Maybe you can check if --config can help you (you can provide a .js config)...
Hum it seems -map option doesn't exist yet. Should not be that hard to add around here https://github.com/cssnext/cssnext/blob/62aeafb32a8d8f50a655152486bad0bac65e25b4/bin/cssnext.js#L72
PRs are welcome :)

yeah was looking into the config. i can add  .map option to the config, but the source map is still not saved.
how would you want the option to look like? --map <filename>?

--sourcemap is already inlining normally, so I guess if you use --map you must provide a filename. But yeah we can do that :) That should enable as well map entry in config file from wat I see in the code (I did not create the --config option).

Not relevant anymore. cssnext is now postcss-cssnext. cssnext is deprecated. See postcss-cssnext migration guide http://cssnext.io/postcss/


Comandline program does not work

Hi,
I came acrosse this package while exploring linux utilities implemeted in Go.
I am having  difficulties to execute this program with the examples given in the README.md. I am a beginner in world of Go.
Please give me some pointers on how to get started with this?
Thanks

Hi,
Sorry to hear things didn't work as expected.
Are you able to share specific details on what you tried a d what the result or error messages were?
Thanks!
Jay

Hi Jay,
Thanks for speedy reply.
I did not try much. I was just reading through readme and found out that the program did not work when I am gving command like below:
$ go-find ~/binaries/ -name 'goplay'
2023/04/29 04:22:47 lstat -name: no such file or directory
It did not work even if I give the absolute path.
$ go-find /home/kenopasa/binaries/ -name /home/kenopasa/binaries/goplay
2023/04/29 04:24:33 lstat -name: no such file or directory
I find out with my little knowledge about Go. the "name" flag is not parsed since the parsing is stopped at first non-flag argument given in the command. I have find a workaround but I am trying to make it flexble like Unix style.
Regards,
Sohail

Hi Jay,
I wanted to let you know I have added below feature and fixed some bugs. Please let me know if I should raise a PR.
Feature added:

Added support for multiple search path.
Added support to parse the flags correctly.

Please do open a PR!
Can't imagine why you want a golang GNU find, lol.

I am not sure about the changes, you would lke it or not. let me raise it anyway by EOD. Thanks!

Ah, I see what's going on here!
So one slightly annoying thing (in this case only) is the way Go Cobra handles command-line args. It wants [flags] followed by [args] (e.g. [paths]).  I'd like to find a way to support the same argument ordering as GNU flavors of find, but didn't find a way to convince or even coerce Cobra into tolerating this.
Try go-find -name /home/kenopasa/binaries/goplay /home/kenopasa/binaries, although if goplay is sitting directly in the "binaries" directory, you probably want to use -wholename (FWIW, I believe this is the same behavior and GNU find).
Also see go-find -h for further usage details.
Usage:

        go-find [flags..] [paths...]
        Available predicate tests:

        -empty          File is empty and is either a regular file or a directory.

        -maxdepth       Descend at most levels (a non-negative integer) levels of directories below the starting-points.  -maxdepth 0 means only apply the tests and actions to the starting-points themselves.
                                                                                -mindepth       Do not apply any tests or actions at levels less than levels (a non-negative integer).  -mindepth 1 means process all files except the starting-points.
        -name           Base of file name (the path with the leading directories removed) matches shell pattern pattern.  Because the leading directories are removed, the file names considered for a match with -name will never include a slash, so `-name a/b' will never match anything (you probably need to use -path instead).

        -print0         Print the full file name on the standard output, followed by a null character (instead of the newline character).  This allows file names that contain newlines or other types of white space to be correctly interpreted by programs that process the find output.  This option corresponds to the -0 option of xargs.
                                                                                -regex          File name matches regular expression pattern.  This is a match on the whole path, not a search.
                                                                                -type           File is of type c:
                        c      character (unbuffered) special                                                           d      directory
                        p      named pipe (FIFO)                                                                        f      regular file
                        l      symbolic link; this is never true if the -L option or the -follow option is in effect, unless the symbolic link is broken.  If you want to search for symbolic links when -L is in effect, use -xtype.                                                                                s      socket
                To search for more than one type at once, you can supply the combined list of type letters separated by a comma `,' (GNU extension).
        -wholename      File name matches shell glob pattern.

p.s. I hope you do submit a PR, I'll look forward to reviewing any submission you wish to make!!!

Hi Jay,
here you go.. I have submitted a PR #2 to support the GNU-like find comand style.
Please have a look at it.

Note: Fixed by PR #2


convertToHtml : not covered bbcode

Hi,
I discover your parser.
I try convertToHtml() function.
Some bbcode are not covered by parser :

quote bbcode with parameters :
[quote="username"]text[/quote]

('quote' bbcode with username variable ;  quote bbcode may exist with several parameters, i.e. phpbb3  :
[quote="username" post_id=8850 time=152549378 user_id=2]
)
2 : attachments bbcode
[attachment=id]filename[/attachment]
I don't know if it has a general interest to support them... or a specific to some bb software...
Thanks for your work.
Best regards.

Will investigate.

the documentation is incorrect, convertToHtml converts from BBcode to HTML


Ionic Platform - iPad Pro

Ran into a bug with Platform on an iPad Pro.
When returning this.platform.platforms() on an iPad Pro I get the following: ["cordova","mobile","ios","iphone"]
An iPad will return the following: ["cordova","mobile","ios","tablet","ipad"]
My current workaround is to get the Platform width and height to determine if it's tablet size.
Would be nice if we could get the iPad Pro to return the same as the iPad.

At the same time, an iPad Mini should be tested as well.

Hello all! This has been fixed and is in the latest release (in fact, it was fixed a few releases ago). Thanks for using Ionic!


Fix studio heading in firefox

Description

Fix the studio heading in Firefox using z-index

Thank you for submitting your Pull Request, the following links will become available for preview shortly:

StudioCMS Landing Page

StudioCMS Documentation

StudioCMS Demo (node-playground) (Also deployed with package changes)

StudioCMS Ui Demo (ui-playground)


Replace old java files

Replace old java files related to generating syntetic data.

Replaced with new data classes.


[HOTFIX] Update on README.md and gitignore

This pull request includes the following changes:

Remove eclipse project files
Update README.md

https://issues.apache.org/jira/browse/HIVEMALL-22

merged in https://github.com/apache/incubator-hivemall/commit/a2f78df058b3ee6ad55b6d980366a4495f468fa2


fix : unwrap on Err inside node_id

The application crashes when we try to connect with wrong node id. This commit will fix it.
fix #166

open again in here https://github.com/vincenzopalazzo/lampo.rs/pull/175


I pressed the shorts video with one address, but the video with another address opens.

Hello doma-itachi!
https://addons.mozilla.org/ru/firefox/addon/youtube-shorts-block/
I use yours extension in Firefox (windows, 64x, Nightly build) and thank you for it!
Today I noticed the error in work your extension -
Wrong video opens, that is, I pressed the shorts video with one address, and the video with another address opens.
(I bring the cursor on the shorts video and choose to open it in a new tab)
This is repeated often, it was already today 5 times.
Even if I copy the correct shorts video address, insert it into the address bar in a new tab, then the address changes(!!) and another video opens! generally random (but with a large number of views).
If you do this several times, the right video will open. At first I thought that this extension has slipped another video to me for the purpose of advertising or mockery, and then I think - well, no, probably a mistake.
Fix it, please.
I can’t give an example - after the correct video opens - the error does not appear once again.
And yet - I don’t know if it is possible to fix it:
Even with normal work of your extension, before the shorts video appears, the youtube shows the identifier error and offers to watch the video later(in the place of video, in this black rectangle). You just have time to read it - the video opens.
please fix it too.

For Firefox version 1.31 cannot be installed :(

I confirmed the phenomenon with firefox 104.0.2 (x64)

Event (if extension is enabled)
If you enter a shorts link in the address bar in the shorts page, you will be skipped to the shorts video page you were watching before.

There is no extension compatibility between Chrome (Manifest v3) and Firefox (Manifest v2), so after writing for Chrome (v3), I rewrote it for Firefox (v2). Therefore the update is delayed. m(_ _)m

As for the issue of being redirected to a different video when opened in a new tab, I have tried about 100 times and it is not happening.
It is more likely a bug in the extension, but there is no code in it that would send you to a different video (that you haven't seen before), so it could be a problem on Youtube's end.
Possible reasons here are.

Bug in extension (very likely)
Problems with Firefox, Youtube
Language or region specific problem (unlikely)

In any case, I will read the code for firefox once again.

I get this bug with (using Firefox) even without having installed this addon. Most likely it is an issue with Firefox.

Indeed, the error was not in the extension.
Thanks for answers, issue may be closed.
I want to share the supervision - one of the extensions installed in my Firefox has been updated - "Enhancer for youtube" https://addons.mozilla.org/ru/firefox/addon/enhancer-for-youtube/
In extension "Enhancer for youtube" option appeared - "Convert Shorts".
It works better than your expansion, Shorts open like ordinary videos faster - you can not even understand that this is not ordinary videos. Look, if interesting, maybe the authors found some secret way to do what you do better.


Assertion failed when disabling ref counts and returning a string literal

I'm getting "assertion failed: !DUK_TVAL_IS_HEAP_ALLOCATED(tv)" in v2.0.2 (though I also tried in v2.0.0 with the same result) when disabling DUK_USE_REFERENCE_COUNTING and returning a string literal from a function. It doesn't do so with any other value type, or if you assign the string to a var.
The assert occurs this line on duk_executor.c.
Enabling ref counting fixes it.
C repro:
int main( int argc, const char* argv[] )
{
    const char js[] = "function() { return '' }";
    duk_context* cx = duk_create_heap_default();

    duk_push_string( cx, js );
    duk_push_string( cx, "main.js" );
    duk_pcompile( cx, DUK_COMPILE_FUNCTION );
    duk_pcall( cx, 0 );

    return 0;
}

With config:
-DDUK_USE_ASSERTIONS                \
-UDUK_USE_REFERENCE_COUNTING

Call Stack:
 	DukBug.exe!duk_default_fatal_handler(void * udata, const char * msg) Line 126	C
>	DukBug.exe!duk__js_execute_bytecode_inner(duk_hthread * entry_thread, unsigned int entry_callstack_top) Line 3917	C
 	DukBug.exe!duk_js_execute_bytecode(duk_hthread * exec_thr) Line 2362	C
 	DukBug.exe!duk__handle_call_inner(duk_hthread * thr, int num_stack_args, unsigned int call_flags, int idx_func) Line 1570	C
 	DukBug.exe!duk_handle_call_protected(duk_hthread * thr, int num_stack_args, unsigned int call_flags) Line 1051	C
 	DukBug.exe!duk_pcall(duk_hthread * ctx, int nargs) Line 134	C
 	DukBug.exe!main(int argc, const char * * argv) Line 17	C

That assert is incorrect when reference counting is disabled: because no refcount update is done, RETCONSTN is used for all return values. The actual behavior is correct though, so the assert doesn't indicate a bug. I fixed the assert in #1427 (to be cleaned up and merged).

Hmm, but I think I'd better cherry pick the fix for 2.0.3.

The actual behavior is correct though, so the assert doesn't indicate a bug.

I figured it wouldn't, but thought I'd let you know anyway :wink:

Appreciate that, thanks :) Assert problems are bugs to be fixed for sure.

Ok, the fix from #1433 is merged to v2.0-maintenance too - in case you want to check that it got fixed for you too: https://github.com/svaarala/duktape/tree/v2.0-maintenance.

Yup, working great now over here too.
Thanks for the quick fix. :)


fix(esl-image-utils): de-scope (img-cover, img-contain) classes

:tada: This issue has been resolved in version 5.0.0-beta.32 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


RedisTrackerStore could allow the setting of Redis record timeout

FEATURE REQUEST
I use the RedisTrackerStore and wanted to expire the Redis records after a period of time, as I am using Redis with persistence and didn't want to accumulate these records for eternity in Redis, as at startup Redis reads the AOF file used for persistance and as gets longer, longer the bootup takes and more memory is used by Redis. Also I write out the records to a relational database when the conversation ended anyhow. In short, I wanted to set a default expiration on the Redis records.
Currently the RedisTrackerStore only allows setting the timeout in the save method (see https://github.com/RasaHQ/rasa_core/blob/master/rasa_core/tracker_store.py#L129), so I had to subclass RedisTrackerStore to set the timeout by doing:
class MyRedisTrackerStore(RedisTrackerStore):
    def save(self, tracker, timeout=RASA_TRACKER_STORE_EXPIRE):
        return super(MyRedisTrackerStore, self).save(tracker, timeout)

This works, but I think it would be nicer if the RedisTrackerStore would have a record_exp initialization variable, which would set the ex of the red.set() if the timeout of the save method is None.
I wanted to make a PR on this, but I saw in the contribution section of the docs, that I am supposed to create a feature request issue first, so here I am, creating an issue first, PR follows.

See PR https://github.com/RasaHQ/rasa_core/pull/923


How does interpreting and linearity of languages interact?

Especially important in the context of #42 .

If you have a term then interpret e.g. a linear variable to a free variable,
then you have a new term morally that is equivalent modulo that one point.
It shouldn't be feasible for any part of the term to depend on this being a linear variable.
Imagine you have a node that stores a Term (LinVar a : ls) tag and preserves the LinVar in the languages,
then interpreting this would result in a situation where this node is "opened up", then
the interpreters are applied to the subnodes, and we use the interpreters to transform the ls to
something we can understand, without interpreting LinVar into something else.

What other implications does this have?
Imagine you have a similar situation, but fix another language that isn't LinVar,
which you transform into a LinVar then use within a linear lambda.
This is well-typed, does it break soundness? I will think about it more.


fix bug according to #1947 - when working with PatternMatchingJGitEnv…

…ironmentRepository. The GitCredentialsProviderFactory instance in the PatternMatchingJGitEnvironmentRepository remains the original GitCredentialsProviderFactory and not the one set by the build method in MultipleJGitEnvironmentRepositoryFactory.

@ryanjbaxter - done


Replace all usages of mem::uninitialized with mem::MaybeUninit

As said in the title, this PR simply replaces any uses of mem::uninitialized in the repo with mem::MaybeUninit. I've run the test-runner to make sure that everything checked there is still working, and encountered no errors.

Thanks for the PR!


Create base for new UI library

Research library candidates for React and ReactNative
Test candidates with POC design set
Set up new monorepo for shared code across products
Determine how to distribute tokens via package(s) from new monorepo
Determine whether to use libraries directly in product repos with theme configs OR wrap into own monorepo for importing
Choose one library for both React and ReactNative OR two separate ones
Set up initial theme configurations with current visual elements
Determine how to replace stacks-ui integration

Thanks for sharing Michelly. I've only skim watched this video, but Figma tokens looks great. Totally up for using this.
I don't know if I'd consider the Framer site as an alternative to Storybook. It might make a great brand colophon kinda site, but wouldn't work well as the home for our UI components, which is where we'd probably colocate Storybook. Open for trying the React component export, but very likely has limitations.
This approach should work well with any themeable ui library.
Current thinking:
flowchart TB
  A(FigmaTokens) --> B("Figma brand site/wallet website")
  A --Auto-updated--> C("@wallet/tokens")
  C --> D(Web-based UI library)
  C --> E(React Native-based UI library)

Thanks for sharing the video Michelly and for the diagram and ideas above Kyran. I watched the video today and it does look really interesting. I spent some time today researching this and library candidates and some of my notes/ feedback is
Figma tokens
I think that Figma tokens will be really useful and we should use them for a lot of things: colours, spacing, typography etc.
Framer
I think that Framer is great for marketing/websites also but I believe for us it would be better to use Storybook for our app components as then we can more easily add tests and maintain a coding playground. Storybook isn't perfect and there maybe a better alternative but I think it ticks most boxes and is pretty standard
**Chakra / Panda etc. **

Chakra seems to be making a lot of improvements - changing to split things up and be multi platform (Panda, Zag)
They still don't offer native support though but they said they will do it  'pretty soon' in 2021
A lot of people have been pushing for React Native support

There is an interesting article here about the future of chakra. If we were just creating a web only platform or choose to have different libraries for web and native then I'd be all for using Panda + Zag.js.
Library for React + React Native
As we are changing the UI library now and plan to create a mobile app soon, I think it would be prudent to consider choosing a library that will support both. From my initial research there seems to be a few options out there but the most promising seem to be:

Nativebase

Native base supposedly has performance issues
https://dev.to/redbar0n/how-to-use-chakraui-on-native-mobile-53hg
https://github.com/GeekyAnts/NativeBase/issues/4302
It has a lot of app development starter kits though

Tamagui

Looks promising and seems to get better feedback from people pushing for Chakra on native
They offer a paid TakeOut starter also which is intriguing

Web Extension support Q
I'm adding on a Q about web extension support here too. I think we support chromium and webkit browsers but not Edge and Safari if I'm not mistaken? Is there a reason we can't support this?
Asking as if we are changing somethings now, perhaps its a good time to also consider trying to support more browsers. Maybe using Next.js would help us to achieve that?

We are working on this as part of the mono-repo and will add storybook for web and react-native there
https://github.com/leather-wallet/extension/issues/1230

It seems we can close this particular issue since we're rolled out the new library. @pete-watters agreed?

Yes lets close this and continue work here https://github.com/leather-wallet/extension/issues/1230. We haven't actually implemented this but we will in #1230


New Network Request: Arbitrum Sepolia

Network information

Name: Arbitrum Sepolia
ChainId: 421614
RPC http/s address: https://sepolia-rollup.arbitrum.io/rpc
Link to https://chainlist.org/: https://chainlist.org/chain/421614
(Optional) Blockexplorer: https://sepolia-explorer.arbitrum.io/
[x] I have run RPC='http://my-rpc-address' yarn estimate
[x] I have sent requiredFunds to 0xE1CB04A0fA36DdD16a06ea828007E35e1a3cBC37

Contract was already deployed, please check before opening a PR: 0x914d7Fec6aaC8cd542e72Bca78B30650d45643d7

I opened the issue because I didn’t find the artifacts/421614/deployment.json file.
Am I missing something?

You are right, I didn't push it to this PR: https://github.com/safe-global/safe-singleton-factory/pull/226/files


Order trigger doesn't work

Hello, I have met an issue with ordering.
When I click on column name it just does nothing, even order icon doesn't change.
Here is my code:
<md-card-content>
           <md-data-table-container>

                       {{person.firstName + " " + person.lastName}}

                           Currently working

                       {{person.jobs[person.jobs.length - 1].position + ", " + person.jobs[person.jobs.length -
                           1].organization.name}}

                       {{person.industry}}
                       {{person.locality}}

                           Linkedin

           </md-data-table-container>

           <md-data-table-pagination md-limit="home.query.limit" md-page="home.query.page"
                                     md-total="{{home.persons.length}}"
                                     md-trigger="home.search"></md-data-table-pagination>

       </md-card-content>

And controller:
vm.query = {
            organizationName: '',
            order: 'lastName',
            limit: 10,
            page: 1
        };

vm.search = search;

function search() {
                vm.persons = [];

                PersonsSearchFactory.query(vm.query,
                    function (data) {
                        vm.persons = data;
                    }, function (e) {
                        console.error(e);
                    });
}

HI @ozasadnyy ,
Could you try to replicate this on the codepen?
Here is the link http://codepen.io/anon/pen/zGXWWp?editors=101
Julian

I have edited your codepen, and it's working. I have no idea why it doesn't work for me.. Can you suggest where to start looking for issue?

is your search function on the scope?
vm.search = function () {
  var self = this;
  self.persons = [];

  PersonsSearchFactory.query(vm.query,
    function (data) {
      self.persons = data;
    }, function (e) {
      console.error(e);
  });
};

@ozasadnyy are you still having an issue with this? I didn't see in your controller you were assigning vm.search = search. I don't think it should matter that you are assigning it before you define it. Can you provide the link to your edited codepen?

@daniel-nagy it's definitely defined, because it's triggers on pagination.
<md-data-table-pagination md-limit="home.query.limit" md-page="home.query.page"
                                      md-total="{{home.persons.length}}"
                                      md-trigger="home.search"></md-data-table-pagination>

@ozasadnyy I don't see anything wrong with your code. If you can use the un-minified version of the library, put a break point on line 31 in the setOrder function. This is the function that should executed when a column header is clicked. Let me know if

This function is never called
headCtrl and scope.order are undefined
headCtrl.pullTrigger is undefined

@daniel-nagy headCtrl.pullTrigger is undefined. Everything else is fine.

Okay this may be a scoping issue unique to your application. Can you now place a break point on line 330? it should say
if(angular.isFunction(scope.trigger)) {

and tell me if scope.trigger is defined. I suspect it is not which would mean that your vm.search function is not visible within scope of the
 element.

'function postLink(scope, element, attrs, tableCtrl) ' doesn't even invoking.

Okay wow, if you open the developer tools and look at the thead element does it have the md-table-head attribute?

Yes, it has.

Okay I have tried again it calls function postLink(scope, element, attrs, tableCtrl) once on initialization.

okay, for whatever reason home.search is not defined in the scope of the thead element. I'm guessing you are using the controller as syntax, are there any new or isolated scopes between where this controller is defined and the data table?
I don't know if I will be able to debug this unless you can reproduce it in the codepen, it is important that you try to replicate the layout of your application as much as possible.

Yeah, I am using new router, which uses controller as syntax by default. No another controller is defined in that file. I am curious why home.search invokes in pagination...

The pagination directive is outside the scope of the table and the table creates an isolated scope. The table doesn't actually need to create an isolated scope because I am using the bindToController syntax. I may need to remove its isolated scope. Can you try to reproduce this in the codepen? You don't need to pull in new router you can just create a controller that uses the controller as syntax.

After bower_components updating it has started to work. Thanks for you help.

I am having a very similar problem, where my
`md-trigger='onOrderChange'`

function never even gets hit when I click on the headers. The Icon on the header is there and recognizes that it's clicked but my function in my directive never runs. I have the table inside of a directive and my function is inside of link in my directive . I think it might be a scope issue.
When I put a breakpoint just after
    function postLink(scope, element, attrs, tableCtrl) {
attrs show that mdTrigger has been assigned onOrderChange, but nothing happens.


Api to delete route of spring actuator gateway always return 404

I have built an gateway service and I want to delete some routes in that gateway.
I use the API /actuator/gateway/routes/{route_id} with DELETE request as in the document of Spring, but it responses 404, but when I use the API /actuator/gateway/routes/{route_id} with GET request, I got the exact route. I'm sure the route_id in 2 request are the same
Is there any wrong with the API of Spring?

@tungmg
How do you define routes? Spring Cloud Gateway Actuator API can only delete routes which defined not use configure file and fluent API.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.


Feedback on user submit (Now Playing preview DM)

It would be nice if Busty could DM the user a preview of what their song will look like on the "Now Playing" message upon submit, since some people might be unsure they're doing it right especially for more exotic (but space efficient!) formats like ogg or opus. It would also serve the dual purpose of a "confirmation", though ofc even if busty is not running when a song is submitted she will still play it during bust time.
This is the first feature we'd implement which would require Busty to be "always on" instead of just on during the sync. Because of this, there should be some sort of flag or environment config allowing you to turn off all "always on" features.

Because of this, there should be some sort of flag or environment config allowing you to turn off all "always on" features.

What would be the use case for disabling this? If you're not going to have the bot running outside of busts, then this feature won't work anyways?

What would be the use case for disabling this? If you're not going to have the bot running outside of busts, then this feature won't work anyways?

That's true if you time it perfectly, but it also makes it so that if you want consistent behavior you CAN'T run Busty outside of the bust. For ex, you might want to turn busty on 15 minutes before the bust or so, but then only people submitting during that time will get DMs. Same with not turning off the bot quickly enough if people submit directly after the bust (which did happen last time!) or turning it on to test !list or something. Imo it's potentially confusing to only sometimes get confirmation, it makes it feel like the times with no confirmation were not "counted properly".
Since adding the ability to turn it off is probably no more than a single line for reading the config variable and a single line for the if statement to check it, I think it's worth adding

Ah, I see the reasoning now. I didn't twig that one may become confused if they didn't receive a DM from the bot after submitting.
I'm still not sure how this is different from taking the bot down though. If you do that, busty also won't respond to submission.
I think having to disable/enable options at specified times is a bit of a bad smell. What if I'm away from internet for a week?
Perhaps instead, we could make a design tweak to prevent the confusing situation from happening. With the current solution, you may run into issues where:

You find out and need to correct mistakes by publicly deleting/re-uploading submissions, which is a bit awkward.
If the bot does happen to be unexpectedly down, you'd have to delete/re-submit your track in the submission channel in order to get a preview.
We have to create an environment variable / command to specify the new submission channel each time.

What do you think about instead just allowing people to DM the bot a track, and the bot would respond with a Now Playing preview?

I realized that because of the whole having to specify a submission channel thing adding an extra config option to disable this feature is actually somewhat pointless in the first place, since you could presumably just not specify a submit channel for no previews. If we were to do it the submit-channel way we could just have an env variable config for channel name regex which defaults to something like bustys-mailbox. That way there's no need to reboot the bot after every bust.
The DM option is still cool though, I like it since it's less public. I think the way I'd prefer is:

Add env variable BUSTY_MONITOR_CHANNELS_REGEX (default bustys-mailbox)
A channel is "monitored" if it matches the regex, or if it is a DM channel
Whenever a media attached message is sent in a monitored channel, busty DMs the user a preview of the "Now Playing" message

Part of why I like the message-on-submit is that we can also use that message to remind ppl of features busty has like cover art or tag reading if they didnt read the info channel carefully.

Ah! Right, that's a very good point. I'm starting to see how this could work now...
On submission, it could DM the user with a series of messages like:

Thanks for submitting to busty's. Here's what your submission will look like during the upcoming bust:

Now playing: ...

Don't forget, you can:

Edit your audio file's tag to include an artist, song name or cover art and it'll be shown to people when your song plays. You can use a program like https://kid3.kde.org to add tags to your file.
Include a message with your submission, and it'll appear under a "More info" banner when your song comes on.
...

If you would like to try uploading a new file, just DM it here and a new preview will be generated.

Upon submitting an audio file, the bot would reply with the same thing, just without the "thanks for submitting to busty's" intro.


Changed order argument for unknown order test as SciPy causes DeprecationWarning

SciPy causes DeprecationWarning when order argument has more than two characters before it raises an actual error. To prevent it, I changed the test argument.

LGTM!


fix cloud_volume_backups size

fixes cloud_volume_backups size column to match that of cloud_volumes

@miq-bot add_label euwe/yes

At this point the last PR in euwe is timestamp<PHONE_NUMBER>1248, it may make sense at this point to try to keep that timestamp as the barrier between euwe migrations and F-release migrations.
This means that all migrations that need to be in euwe should be timestamped between<PHONE_NUMBER>0347 and<PHONE_NUMBER>1248
Sound good @Fryguy ?

@carbonin I've made the requested change, let me know if I did it wrong!

You may want to fix up the rubocop issues, but I'll approve the migration rename. Thanks @tzumainn

done, thanks!

Euwe Backport details:
$ git log
commit 5bd4cfe87cb9f779f74d82be279b68d1a72aea53
Author: Jason Frey<EMAIL_ADDRESS>Date:   Fri Sep 30 21:44:03 2016 -0400

    Merge pull request #11599 from tzumainn/fix-cloud-volume-backup-size

    fix cloud_volume_backups size
    (cherry picked from commit 6abc37bc054bd32bed3e309d353357255abeec12)


[ci skip] adding user @thewchan

Hi! This is the friendly automated conda-forge-webservice.
I've added user @thewchan as instructed in #31.
Merge this PR to add the user. Please do not rerender this PR or change it in any way. It has [ci skip] in the commit message to avoid pushing a new build and so the build configuration in the feedstock should not be changed.
Please contact conda-forge/core to have this PR merged, if the maintainer is unresponsive.
Fixes #31

@hmaarrfk here

I know this request is not consistent between core members, but can you take this opportunity to refresh the recipe and get it working?
You can probably get pretty far without waiting.

I guess, next time you can probably debug the recipe, get it working, and add yourself at the end.
Skipping the whole waiting game cycle.


Accuracy Weirdness

What feature do you want to get added on the base engine?
Right now, it seems like if you get the same score, but a better accuracy, your personal best accuracy displayed in the Freeplay menu does not change to reflect the new personal best. This can happen when you get an FC, but don't get perfect accuracy. FCing at any point, regardless of your actual accuracy, appears to always give you the same final score. This means that it won't update the accuracy pb with a new accuracy if you did better than your current pb on accuracy.
I suggest 1 of 2 things, either/or is fine, but both would be cool too:

if your score is equal to your current pb but your accuracy is better, then set the currently saved pb to the new score+accuracy.
make accuracy affect your score. IE if you get a shit or a bad it should be worth say, 50% of the points you would get normally, a good would be worth maybe 70%, and a sick is worth 100% of the points. (at least, it doesnt look like it already does something like this, from in-game and from glancing over the code). or you could make it really wild and scale it by how far away from a perfect hit you were, like a difference of say 1 millisecond would be 100 points * (1 - (ms_diff / 1000)), rounding however seems most appropriate, probably always rounding down

the second one is possibly too drastic of a change, but i think the first one should be an easy and reasonable one to make.

In my honest opinion number 2 isn't too drastic, this is how literally almost every rhythm game does it anyways :P
Like osu!mania uses 320 points for a sick hit within a 17ms window if I'm not wrong and then 300 for any other sick hits. Then 200 for Good and 100 for bad.

The accuracy saved is the accuracy from your best score, not your best accuracy.

Yes. That's exactly. The reason. Why I opened. This issue. That should change.


Add get_current_user helper func to fetch data about authenticated user without providing username or user_id

Like the Idea but would not have an Idea what to use it for?

I use it to retrieve the broadcaster ID and username for users I don't know in advance. This allows users to authenticate dynamically with their accounts and play my game without manually entering their username.


Crash in parsing in prometheus metrics

Describe the bug
Malformed DNS data causes a crash in the Prometheus sections of code.  I am using the pipeline branch, but I suspect that is not relevant.
To Reproduce
Difficult to say other what the exact query was that caused the crash - I wasn't recording this data anywhere else, other than what you see on the output in the crash.
Expected behavior
Hoping for no crash.  :-)
Additional context
My prometheus configuration
pipelines:
  - name: prom
    prometheus:
      listen-ip: <IP_ADDRESS>
      listen-port: 8081
      basic-auth-enable: false
      prometheus-prefix: "dnscollector"
      top-n: 100
      histogram-metrics-enabled: true
      prometheus-labels: ["stream_id"]

The crash:
INFO: 2023/12/14 02:45:03.397202 [dnsdist-from-outside] processor=dnstap#1 - waiting dns message to process...
INFO: 2023/12/14 02:45:03.399139 [dnsdist-from-outside] collector=dnstap#1 - receiver framestream initialized
panic: label value "dr._dns-sd._udp. \xbf\x9f\x01" is not valid UTF-8

goroutine 78 [running]:
github.com/prometheus/client_golang/prometheus.MustNewConstMetric(...)
	/root/go/pkg/mod/github.com/prometheus/client_golang@v1.17.0/prometheus/value.go:129
github.com/dmachard/go-dnscollector/loggers.(*PrometheusCountersSet).Collect(0xc0001343c0, 0xc000475380?)
	/home/jtodd/go-dnscollector/go-dnscollector-pipeline-branch/go-dnscollector/loggers/prometheus.go:528 +0x1aed
github.com/prometheus/client_golang/prometheus.(*wrappingCollector).Collect.func1()
	/root/go/pkg/mod/github.com/prometheus/client_golang@v1.17.0/prometheus/wrap.go:129 +0x28
created by github.com/prometheus/client_golang/prometheus.(*wrappingCollector).Collect in goroutine 77
	/root/go/pkg/mod/github.com/prometheus/client_golang@v1.17.0/prometheus/wrap.go:128 +0x7d
root@ub20template:/home/jtodd/go-dnscollector/go-dnscollector-pipeline-branch/go-dnscollector#

I don't success to reproduce with invalid name :(
Is it possible to provide the complete query name or share a pcap file?

Sorry, I don't have the packet in question - I wasn't capturing at that point, and haven't been able to reproduce yet.

It just happened again, and I now figured out what the trigger is: when a pull is made against my Prometheus endpoint, the crash occurs. It is not happening when the packet reaches go-dnscollector - it is only when I do "curl http://<IP_ADDRESS>:8081/metrics" that the crash happens.
Same general failure, but the line is:
panic: label value "lb._dns-sd._udp.\xd0\xdfP\x01" is not valid UTF-8

I suspected that, but for my part, the crash did not occur when I do the curl. Instead, the metrics are truncated with is not normal too.
Before fixing I would like to reproduce and write a test unit.
Thanks to report that.

Panic error reproduced !
This issue is related to the prometheus logger only.

fix pushed in main and pipeline branch

Fixed in v0.39.0.


Segmentation fault when writing to netcdf with dask-enabled xarray dataset

I have a 4 GB netcdf file and am running on a machine with 32 GB of memory.  The following works just fine without error on this large memory machine:
ds = xarray.open_dataset('input.nc')
ds.to_netcdf('output.nc')

This dask + pynio approach also works correctly:
ds = xarray.open_dataset('input.nc', chunks={'a': 25, 'b': 25}, engine='pynio')
ds.to_netcdf('output.nc')

But the following dask + default engine (netcdf4 probably?) approach slowly sucks up all the system memory, writes out a file twice as large as it should be with variable values that are extremely large, and then fails with seg fault, bus error or other low-level system errors we'd rather not be seeing in python!
ds = xarray.open_dataset('input.nc', chunks={'a': 25, 'b': 25})
ds.to_netcdf('output.nc')

Adding lock=True to the open_dataset call does not help.
I have two workable solutions to my problem (run without dask because I have a lot of memory available, or use engine='pynio') but this error was hard to track down so thought you would want to know.  Would be glad to hear also if I missed something in the docs and the all-too-common user error is to blame =)

An important addition -- the following also causes low-level system errors (bus error or seg fault, can't remember) so the problem does not originate in the to_netcdf per se, but rather in the chunking / loading of the dataset.
ds = xarray.open_dataset('input.nc', chunks={'a': 25, 'b': 25})
ds.load()

@hdail thanks for the report! Which verison of xarray are you using?
We fixed something that sounds pretty similar (https://github.com/pydata/xarray/issues/936) in v0.8.2.

I'm using 0.8.2.  Thanks for the issue link; I had read through that, but since I am not using open_mfdataset and lock=True did not fix my issue, I figured my problem was subtly different.  Some incompatibility / race condition when using netcdf4 and dask together?   This might be a tricky problem to track down as my code did complete without seg faulting when my dimensions were subtly different (about 10% small in space and 20% smaller in time), even on a server with half as much memory.  Bleck.

Yes, this is different.
I think this is bug in how we write netCDF files. Currently, we always use a new thread lock in ArrayWriter.sync(). To avoid possible concurrency issues with the HDF5 API, we really should be reusing the same _default_lock that we use for reading netCDF files.

Thanks for the info!  Given this potential bug, is the driver='pynio' solution acceptable, or is it just working for me for now and may fail in some subtly different configuration / data size?

It is possible that pynio is linking to an independent HDF5 installation, which should eliminate the need for a shared lock. But if that's not the case, then you probably just got lucky.


[Question] Plugin to load server by context

Many thanks for this great plugin.
I was wondering if this plugin is possible, it should also be also possible to load lsp servers per treessiter context.
For example, in tsx, if we are in the markup we should be able to load the HTML and emmet servers, likewise inside
some CSS, or even if we are in an attribute of the tag, like the string in the class attribute, load the tailwind server.
I know this is not the scope of this plugin but because you are kind of doing something similar I was wondering if this would be at all possible.

Hey @ofrades!
I have also thought about this subject a little bit. I think that it depends on the LSP since LSPs don't generally understand embedded languages. Vue language server has been built specifically to handle injected languages in .vue files.
One (extremely hacky 😄) possibility would be to copy the injected language part of the file into a new temporary file, run LSP on that file, and pass the LSP diagnostics / completion results to the current buffer. I think that that's how Vue LSP handles multiple LSPs in a single file internally. I don't know if something like that should be in a language server or maybe it could be implemented as a Neovim plugin with the help of treesitter and customized built-in LSP client.


RootModel.model_construct() typing

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
The signature of the model_construct method of RootModel does not match the signature in the supertype BaseModel, causing mypy to disfunction when type checking.
Using my_root_model.model_construct('smth') results in 2 errors:
error: Missing named argument "root" for "model_construct" of "MyRootModel"  [call-arg]
error: Argument 1 to "model_construct" of "MyRootModel" has incompatible type "str"; expected "Optional[Set[str]]"  [arg-type]

because mypy is trying to match the call with the signature of the supertype method.
I see no easy fix, because there is no way of forcing mypy to type check the call against the signature of the subtype method. And, after all, mypy is right? The signature of a subtype should match that of the supertype. Perhaps this is an indication that RootModel should not inherit from BaseModel? While I see why you implemented it that way, I think there is a case to be made against it. What are your thoughts on this?
In any case, the current type checking is very annoying (especially with a lot of tests) because one has to manually ignore those 2 errors each time Root.model_construct is called.
Example Code
No response
Python, Pydantic & OS Version
pydantic version: 2.7.1
        pydantic-core version: 2.18.2
          pydantic-core build: profile=release pgo=true
                 install path: -
               python version: 3.11.2 (tags/v3.11.2:878ead1, Feb  7 2023, 16:38:35) [MSC v.1934 64 bit (AMD64)]
                     platform: Windows-10-10.0.19045-SP0
             related packages: mypy-1.2.0+dev.456dcbdd6a2f9b36249e0e7d31e521f5678a8235 typing_extensions-4.11.0
                       commit: unknown

I'm not able to reproduce, could you please throw a reveal_type(my_root_model) in your snippet to see the actual type?

import pydantic

class MyRootModel(pydantic.RootModel[str]):
    pass

if __name__ == '__main__':
    reveal_type(MyRootModel)
    reveal_type(MyRootModel.model_construct('yo'))

results in
note: Revealed type is "def (root: builtins.str) -> scratch_11.MyRootModel"
error: Missing named argument "root" for "model_construct" of "MyRootModel"  [call-arg]
note: Revealed type is "scratch_11.MyRootModel"
error: Argument 1 to "model_construct" of "MyRootModel" has incompatible type "str"; expected "set[str] | None"  [arg-type]

mypy 1.2.0 is pretty old, do you get the same error on latest (currently 1.10.0)?

yes i actually tried this on 1.10 directly, after realizing i was on 1.2

did you perhaps try this by specifying the keyworded parameter: MyRootModel.construct(root='yo')? when you do it that way mypy is happy.
i'd argue that specifying the root parameter by name is counter-intuitive, and perhaps is another indication that RootModel should not inherit from BaseModel.

Hum really strange, it typechecks correctly on my end even when specifying root as a positional argument. Sorry for asking again, could you now add a reveal_type(MyRootModel.model_construct)? Hopefully this will add more info.

ok, the difference is that i'm using pydantic.mypy as a mypy plugin. sorry for not noticing this earlier.
reveal_type(MyRootModel.model_construct) outputs note: Revealed type is "def (_fields_set: Union[builtins.set[builtins.str], None] =, *, root: builtins.str, **kwargs: Any) -> scratch_11.MyRootModel".

Ok great, at least we were able to understand where it was coming from.

thank you for the fix!


fix(amplify-dynamodb-simulator):callback revoked within time-limit

Description of changes
Revoked async function within time-limit by calling the removeSync() function only for test cases requiring the same.
Since, one of the test case did not require sync to be broken right after as another test case was to run on previous test cases result,thus removeSync() was not to be called.
Issue #, if available
#7842
Description of how you validated changes

cd packages/amplify-dynamodb-simulator
yarn test

Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added
[x] Relevant documentation is changed or added (and PR referenced)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

It's not clear to me how this fixes the failures in #7842.

For the test that times out, does increasing the timeout fix the problem?
For the test failing with EBUSY - does this test fail 100% of the time? It seems like something else has the file open and is preventing it from being deleted.

It's not clear to me how this fixes the failures in #7842.

For the test that times out, does increasing the timeout fix the problem?
For the test failing with EBUSY - does this test fail 100% of the time? It seems like something else has the file open and is preventing it from being deleted.

No, increasing the time out doesn't fix the problem, it tests on different database samples and always fails on some database testing even when increased.

Yes, it fails every time, did try restarting servers and operating system, but had no effect, don't know what it means by EBUSY here
afterEach(ensureNoDbPath)  throws an error only with the test case  'should preserve state between restarts with dbPath'

afterEach(ensureNoDbPath) throws an error only with the test case 'should preserve state between restarts with dbPath'

In that test, does changing emuTwo.terminate(); to await emuTwo.terminate(); make any difference?

No, adding await doesn't have any effect, did try this out

The afterEach() that performs the cleanup doesn't run for that test though, right (because the test is not in the same describe() block)?
I think the underlying issue is that the emulators aren't being cleaned up properly. On Windows, a file can't be deleted if another process has the file opened.
Another thing to try - can you combine the two afterEach() calls into a single afterEach() and make sure that the emulators are terminated before trying to remove the file. It's possible that Jest is currently running the afterEach() blocks in the order that they are defined in the code, which would mean trying to delete the file before terminating the emulator.

Yes, afterEach() was not being called for that test (in previous commit made)
Tried out a new way by combining both beforeEach() and afterEach() into one


Don't fail pint ci if a directory is renamed

Codecov Report

Merging #532 (5b30c75) into main (0fcd986) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #532      +/-   ##
==========================================
+ Coverage   93.70%   93.81%   +0.10%
==========================================
  Files          81       81
  Lines        7593     7612      +19
==========================================
+ Hits         7115     7141      +26
+ Misses        368      363       -5
+ Partials      110      108       -2

Impacted Files
Coverage Δ

internal/discovery/git_blame.go
97.10% <100.00%> (+0.20%)
:arrow_up:

internal/git/changes.go
92.30% <100.00%> (+0.44%)
:arrow_up:

internal/reporter/github.go
75.36% <0.00%> (+1.44%)
:arrow_up:

cmd/pint/ci.go
86.41% <0.00%> (+2.17%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Allow merge.ThreeWay() to operate across DBs

Much like Diff, we want to be able to merge values from different
databases. Unlike Diff, Merge needs to be able to resolve Refs --
which requires a ValueReadWriter that has access to the referenced
Value. So, merge.ThreeWay() now takes a ValueReadWriter for each Value
being considered for the merge -- the parent and both descendants.
Towards #148

PTAL

lgtm with opt nit

turns out this won't work in all cases :-) Going to close this out and keep the branch locally in case I circle back


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

In 5bb1c9f, sandbox-api.piste.gouv.fr healthcheck (https://sandbox-api.piste.gouv.fr/cassation/judilibre/v1.0/healthcheck) experienced degraded performance:

HTTP code: 200
Response time: 2679 ms

Resolved: sandbox-api.piste.gouv.fr healthcheck performance has improved in 9132ff1.


Unhelpful message when unable to iroh start

I managed to solve this one myself, but I didn't see an existing Issue for this so figured I'd file it in case someone else runs into something similar in the future.
I think the root cause is that I had some old Iroh files on my system. Here's the relevant parts of my console history (omitting various troubleshooting steps).
$ nix shell nixpkgs/nixos-unstable#iroh

$ iroh --version
iroh-cli 0.14.0

$ iroh start
Error: Table 'records-by-key-1' does not exist

$ RUST_LOG=info iroh start
2024-04-25T23:51:23.598411Z  INFO iroh_metrics::service: Starting metrics server on <IP_ADDRESS>:9090
⠙ Iroh booting...                                                                                2024-04-25T23:51:23.602662Z  INFO iroh_sync::store::fs::migrations: Executed migration migration_004_populate_by_key_index (0 rows affected)
2024-04-25T23:51:23.614849Z  INFO iroh_sync::store::fs::migrate_v1_v2: migrate /Users/expede/Library/Application Support/iroh/docs.redb to /Users/expede/Library/Application Support/iroh/docs.db.migrateSqkBCS
2024-04-25T23:51:23.636360Z  INFO iroh_sync::store::fs::migrate_v1_v2: migrate authors-1 (0 rows)..
2024-04-25T23:51:23.636382Z  INFO iroh_sync::store::fs::migrate_v1_v2: migrate authors-1 done
2024-04-25T23:51:23.636410Z  INFO iroh_sync::store::fs::migrate_v1_v2: migrate namespaces-2 (1 rows)..
2024-04-25T23:51:23.636424Z  INFO iroh_sync::store::fs::migrate_v1_v2: migrate namespaces-2 done
2024-04-25T23:51:23.636438Z  INFO iroh_sync::store::fs::migrate_v1_v2: migrate records-1 (0 rows)..
2024-04-25T23:51:23.636442Z  INFO iroh_sync::store::fs::migrate_v1_v2: migrate records-1 done
2024-04-25T23:51:23.636454Z  INFO iroh_sync::store::fs::migrate_v1_v2: migrate latest-by-author-1 (0 rows)..
2024-04-25T23:51:23.636458Z  INFO iroh_sync::store::fs::migrate_v1_v2: migrate latest-by-author-1 done
Error: Table 'records-by-key-1' does not exist

$ rm -r /Users/expede/Library/Application\ Support/iroh/

$ iroh start
Iroh is running
Node ID: <HIDDEN>

This is absolutely an edge case so likely not high priority. It might be nice to either add a better error message (e.g. "you have an existing install and we failed to migrate") or a CLI command to do a completely fresh install (e.g. iroh clean). I knew that this is a  Rust project and thus was able to use RUST_LOG, but a --verbose flag may be more familiar to non-Rustacians.

thanks, yes we should improve on these error messages

Closing, as we have removed iroh-cli


Batch PUTs?

The Kinesis API supports batching PutRecord requests, does/will pynesis?

Yes, we'd definitely like to support this. I'll try to find some time this weekend to give it a try. Would you be able to help by trying it out (beta test) this before we merge it in?


Fix expanded order details

Expand order data in the response, instead of accessing directly via a URL (comma separated list). Valid values are holds, items , returns and trackings, splitOrders, or simply all. See resources Holds, Items, Returns, Trackings and splitOrders for information on the data model returned by this parameter.

https://www.shipwire.com/w/developers/order/#panel-shipwire0

FIxed and created a new version v0.2.4


Lede logo on the new 18.06-rc2 firmware

Hey guys!
Just made a sysupgrade from lede-17.01 to openwrt-18.06. Noticed that LuCI still shows "Lede" instead of "OpenWRT". Is it due to sysupgrade and may be I should make a clean installation without "keep conf files" option? Or is it by design and will be fixed in future?
http://jpegshare.net/79/ff/79ff0fe760b235155ce57f1b94776c69.png.html
Thanks!

That is not logo, but the name of the router that you have set in /etc/config/system
That config file has been kept in sysupgrade, so it shows the previous default.


Attempt to fix broken redirects wiht api

api redirects are not working when deploying. Going to try to  reverse order to see if that is the issue. If that does not work, I will add leading slashes.

This worked but I merged locally so will close this PR just to keep things cleaner. Will push up to master directly.


Fix out of range birth date triggering an error from CRM

Context
https://trello.com/c/w4spryA5/206-fix-validation-exception-from-get-teacher-endpoint
Changes proposed in this pull request
Add extra validation to ensure the birth date is within the acceptable bounds for CRM.


run benchmark tests with the race detector on Travis

I already tried this a couple of months ago, but for some reason this failed on Travis (with an ackhandler error that I don't remember). Now it seems to work without any further changes, so we can merge this.
Running a benchmark test with race detector enabled takes about 4s (compared to about 0.8s without).

Codecov Report

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

@@           Coverage Diff           @@
##           master     #740   +/-   ##
=======================================
  Coverage   91.34%   91.34%
=======================================
  Files          79       79
  Lines        5669     5669
=======================================
  Hits         5178     5178
  Misses        307      307
  Partials      184      184

Continue to review full report at Codecov.

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


 Is this prohibited ? Is this allowed ? Is this illegal or legal ?
Please, provide more information on docs, provide more information for this error.
And, im comparing items with hashcode, im totally assure thats there is same item in my list, as i am searching to, i am tried comparing it with hashcode, id, title, not myself override compare - no difference

Would you provide minimal case, and your flutter/dart versions used (using flutter doctor -v).
With IntelliJ, clicking on the indexOf opens the declaration in list.dart, and one can see the documentation there, I guess VisualStudioCode does the same thing.
The below case prints 1:
void main() {
  var list = <int>[5, 3];
  var i = list.indexOf(3);
  print(i);
}

Dart version 2.17.0 (build 2.17.0-265.0.dev)

there is no fucking reference in documentation for it https://api.dart.dev/stable/2.16.2/dart-core/List/indexOf.html

@thq919
Please read github code of conduct and kindly try not to use such language while reporting issues or commenting.
Also this seems to be related to dart language and not Flutter, so please open this issue in dedicated repo: https://github.com/dart-lang/language/issues
Closing from here.

Would you provide a minimal case, and your flutter/dart versions used (using flutter doctor -v).
With IntelliJ, clicking on the indexOf opens the declaration in list.dart, and one can see the documentation there, I guess VisualStudioCode does the same thing.
The below case prints 1:
void main() {
  var list = <int>[5, 3];
  var i = list.indexOf(3);
  print(i);
}

`Dart version 2.17.0 (build 2.17.0-265.0.dev)

import 'dart:ui';
class SinglePhoneHomeStore {
final int id;
final bool? isNew;
final String title;
final String? subtitle;
final String picture;
final bool? isBuy;
final bool? isFavorites;
final int? priceWithoutDiscount;
final int? discountPrice;
SinglePhoneHomeStore({
this.isFavorites,
this.priceWithoutDiscount,
this.discountPrice,
this.isNew,
this.subtitle,
this.isBuy,
required this.id,
required this.title,
required this.picture,
});
@override
bool operator ==(dynamic other) {
// ignore: unnecessary_this
if (this.runtimeType == other.runtimeType && hashCode == other.hashCode) {
return true;
} else {
return false;
}
}
@override
int get hashCode => hashValues(id, title);
}
place where i use index of
indexOfElementToShow = phoneList!.indexOf(phone);
(phone type SinglePhoneHomeStore)
there is variables in runtime
List (4 items) phonelist
SinglePhoneHomeStore
SinglePhoneHomeStore
SinglePhoneHomeStore
SinglePhoneHomeStore
item that im searching index of has hash 391735408
in list that hash has first element 391735408 and has index 0
why do i get -1 ?
[✓] Flutter (Channel stable, 2.10.3, on Ubuntu 20.04.4 LTS 5.13.0-39-generic, locale en_US.UTF-8)
• Flutter version 2.10.3 at /home/thq/snap/flutter/common/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 7e9793dee1 (5 weeks ago), 2022-03-02 11:23:12 -0600
• Engine revision bd539267b4
• Dart version 2.16.1
• DevTools version 2.9.2
[!] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at /usr/lib/android-sdk
• Platform android-31, build-tools 29.0.2
• ANDROID_HOME = /usr/lib/android-sdk
• Java binary at: /home/thq/android-studio/jre/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)
✗ Android license status unknown.
Run flutter doctor --android-licenses to accept the SDK licenses.
See https://flutter.dev/docs/get-started/install/linux#android-setup for more details.
[✗] Chrome - develop for the web (Cannot find Chrome executable at google-chrome)
! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.
[✓] Android Studio (version 2021.1)
• Android Studio at /home/thq/android-studio
• Flutter plugin version 63.2.2
• Dart plugin version 211.7798
• Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)
[✓] VS Code (version 1.66.0)
• VS Code at /usr/share/code
• Flutter extension version 3.38.0
[✓] Connected device (1 available)
• Android SDK built for x86 (mobile) • emulator-5554 • android-x86 • Android 8.1.0 (API 27) (emulator)
[✓] HTTP Host Availability
• All required HTTP hosts are available
! Doctor found issues in 2 categories.

Would you provide a minimal case, and your flutter/dart versions used (using flutter doctor -v).
With IntelliJ, clicking on the indexOf opens the declaration in list.dart, and one can see the documentation there, I guess VisualStudioCode does the same thing.
The below case prints 1:
void main() {
  var list = <int>[5, 3];
  var i = list.indexOf(3);
  print(i);
}

Dart version 2.17.0 (build 2.17.0-265.0.dev)

int index = -1;
int indexCount = 0;
phoneList?.forEach((element) {
if (element == phone) {
index = indexCount;
return;
}
indexCount++;
});
indexOfElementToShow = index;
that works properly


can-dom-binding

I know it's boring, but to be consistent with names ... should we call this can-bind-dom (and rename can-stache-bindings to can-bind-stache) or can-dom-bindings?

I vote for can-dom-binding

+1 for can-dom-bindings


Change avro dependency to version 1.11.1.

The previous version of avro was 1.8.2, but parquet-avro:1.12.0, which is a dependency of beam-sdks-java-io-parquet, requires avro:1.10.1: https://mvnrepository.com/artifact/org.apache.parquet/parquet-avro/1.12.0.
Alternatively, we may update the version to avro:1.10.1 (required by parquet-avro:1.12.0), but avro:1.11.1 has fewer vulnerabilities:
https://mvnrepository.com/artifact/org.apache.avro/avro/1.10.1
https://mvnrepository.com/artifact/org.apache.avro/avro/1.11.1
Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

There is on-going work on the dependency issue: https://github.com/apache/beam/issues/24292. Closing this PR.


Copy and Paste in tmux

I have tmux installed and I'm trying to use the mouse to copy and paste between the tmux session and the system clipboard.
From what I'm reading online it suggests that I need to install xclip but that's not available in the repos.
Any suggestions on how I can do this in a PASE session?

We do not have xclip (or xsel as I've seen suggested also). I don't think there's any way to do that in PASE currently.


Give core project a meaningful name in Android Studio

It's about to change this name

When every repo is called android , you don't know which one it is

I made the samo pull request in my fork https://github.com/hannesa2/virocore/pull/1
But  there the CI works properly, here nothing happens

@dthian Who is the maintainer here.
Please think about to have more of them


Add Payload to reducer

As it is right now the only way to use the action is by
class MyReducers {
    @Reducer() add(state): { return { count: state.count + 1; } }
}

An idea that i have on how to make this better is doing something like:
class MyReducers {
    @Reducer() add(state, action): { console.log(action.payload) }
}

What do you think?

Currently the only way to access the action is through the root reducer.  This would be a nice  improvement to action reducers.  Nice suggestion :+1:

Okay I've looked into this a little and I'm not sure this is required.  Let's look at an example, let's say we dispatch this action:
dispatch('addPost', 'My title', 'My body text');

The action is automatically created for you, and follows a predetermined pattern.  This means when the action reducer is called it's passed the state and any other data attached to the action.  For example the above action passes the title and body values to the action reducer:
@Reducer() addPost(state, title, body): {
  // we are passed the title and body as arguments
}

So although you can't access the action, there's really no need to because you can't attach anything to the action.
Does this resolve your issue?

I'm assuming this is resolved.