To Do List

To Do List! ES6 WebPack
✨ Requested Implementations ✨
✔ Project set up with WebPack
✔ To Do List dynamically generated with Javascript, using an empty ul
✔ Created a default to-do task array with description, boolean & index number
✔ Created showList function to iterate through  the list array and render the HTML elements
✔ Created a style.css file that is imported through WebPack and it's a little clone of the video
✔ index.html, index.js & style.css located inside /src and WebPack generate all the used code on the /dist folder

💡 Thank you for your review! 💡

Hi @VelzckC0D3,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that is invalid because the project lacks a professional README file. Kindly check this template for assistance
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Fastball: Getting last kill on player finishing hacking control panel crashes majority of respawning players

Getting the kill on the last player in the enemy team just as they finish hacking the control panel crashes the majority of the players respawning from the succesfully hacked control panel.
Video of the bug in action:
(look at the killfeed at the beginning and scoreboard at the end of the clip)
https://user-images.githubusercontent.com/40122905/148071443-7a9c6175-701b-419e-b878-1d7afa1a8e07.mp4
Sadly no logs for the time being, will add them once I have some.

Simillar thing happened to me where the player hacking got killed right after respawning his team; round ended even tho my entire team got respawned which could be considered a bug; but mainly noone crashed unlike the linked video

From personal experience it seems to mainly happen when the killer is using a weapon that has splash damage.

On that note, I was on the receiving end of the bug recently
Clip:
https://user-images.githubusercontent.com/40122905/149836119-fca74066-e72e-4a3d-b3ba-8869db015322.mp4
Trimmed log: (trimmed) nslog2022-01-15 21-50-09.txt
(CTRL+F for #96 to find the part in the log where the crash happened.)

i think this is caused by the use of GetLocalViewPlayer rather than GetLocalClientPlayer in fastball client scripts, i can try and get a potential fix for this comitted later

should be fixed as of this commit, leaving the issue open until someone can verify this all works

is this good to close now?

Haven't experienced the issue again since, so I guess yes ^^


getboundingclientrect issues

There is a lot of issues bound to getboundingclientrect. Cross-browser issues and Chrome related issues. You find 26 pages with Chrome issues here.
https://bugs.chromium.org/p/chromium/issues/list?can=2&q=getboundingclientrect&colspec=ID+Pri+M+Stars+ReleaseBlock+Component+Status+Owner+Summary+OS+Modified&x=m&y=releaseblock&cells=ids
For some of this issues Inferno would need to find workarounds.

@Kflash lazy will have lots of doc when it's official in API, currently it's not, so I'm closing this issue as it's not to be used in production currently.

Sure thing. Just mention this in the docs so people knows there are tons of issues related to this.

@Kflash it's not in docs, because lazy is currently WIP as per release notes and thus not stable or supported by Inferno for production.


Update @asciidoctor/core dep version

@asciidoctor/core is now version 3.0.2
It would be great to have a version of this plugin compatible with the new version on @asciidoctor/core

Fixed in https://github.com/ggrossetie/asciidoctor-kroki/commit/523de35a57e3bda79d276769b28e3175c14b1a92


⚠ TROJAN 🇮🇩 Indonesia IDJ 1 has degraded performance

In 5869d2a, TROJAN 🇮🇩 Indonesia IDJ 1 (https://idt-1.opensvr.net/) experienced degraded performance:

HTTP code: 200
Response time: 11197 ms

Resolved: TROJAN 🇮🇩 Indonesia IDJ 1 performance has improved in c6e7ec1.


Implement WAMP peer identification

For now this has no use for us, but may be revisited/reworked as part of the WAMP standard.
Abandon for now


bug: Scroll bar cannot be hidden in ionic 4 with chrome browser.

Bug Report
Ionic version:

[x] 4.x
Current behavior:

Expected behavior:

Steps to reproduce:

Related code:

insert short code snippets here

Other information:

Ionic info:

insert the output from ionic info here

Ionic:
Ionic CLI                     : 5.4.16 (C:\Users\Dreamuser\AppData\Roaming\npm\node_modules\ionic)
Ionic Framework               : @ionic/angular 4.11.5
@angular-devkit/build-angular : 0.801.3
@angular-devkit/schematics    : 8.1.3
@angular/cli                  : 8.1.3
@ionic/angular-toolkit        : 2.1.1
Cordova:
Cordova CLI       : 8.0.0
Cordova Platforms : android 7.0.0, browser 5.0.4
Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 4.1.3, (and 10 other plugins)
Utility:
cordova-res : not installed
native-run  : 0.2.7
System:
Android SDK Tools : 26.1.1 (C:\Users\Dreamuser\AppData\Local\Android\Sdk)
NodeJS            : v10.16.2 (C:\Program Files\nodejs\node.exe)
npm               : 6.5.0
OS                : Windows 10

Just let me know how to hide the scroll bar in the browser platform

@Manojdreams Check this out, you can hide it with CSS http://jsfiddle.net/5GCsJ/954/ :)

Hi @NikolaPeevski  But in ionic 4 we cannot access the scrollbar using css. I have already tried method but it doesn't work.

@Manojdreams Can you provide a repo where you're trying to hide the scrollbar ?

Sorry @NikolaPeevski  I don't have access to share the repo. I am trying to hide the scroll bar in the web browsers using ionic 4 but it is not accessible. Can you tell any solution for this it will be very helpful for me.

@Manojdreams  Do you want to hide all scrollbars or just  the top one level one?

I need to hide all the scrollbars. I can hide it using
ion-content { --overflow: hidden; }
but when I give like this I cannot scroll the page. I need to hide only the scrollbars in all pages.
@NikolaPeevski

Thanks for the issue. I am going to close this as a duplicate of https://github.com/ionic-team/ionic/issues/17685.


"400 - ExpectedFileSystemMethodsToGetTags"

Hi. I'm running the latest version of BoS on a raspibolt node, debian x86_64 with kernel 6+ ,  systemd.    When I run "bos lnurl pay" with the "--avoid" option it raises an error message "400 - ExpectedFileSystemMethodsToGetTags"

Should be fixed in the latest update, can you try again?

Okay! 100% !  thank you very much and a merry Christmas.


Recover / login with Ledger should not be adding access keys

Currently if user recovers accounts with Ledger or logins into an app - the new key is added to the account.
There is no reason for this. Wallet UI can maintain in the local storage the information about the key location.
Apps login also should not require adding key, if the app is not requesting to add an access key for it's usage. Also user should be able to select even if app is requesting, to still only share the account without adding access keys.

This is a duplicate of https://github.com/near/near-wallet/issues/772#issuecomment-671761847 and https://github.com/near/near-wallet/issues/1536#issuecomment-808784422
It's not completely clear what's the needed effort, @vgrichina do you have any updates? It seems to me that we may need to review the Ledger app to make this happen.

Adding a p1 as I don't like the idea of "tainting" a ledger account with on-chain keys generated by the wallet


Use release/targetComp setting of compile task for jvm version attribute

Before the 'release' property was ignored completely. The' targetCompatiblity' was always taken from the extension/convention and not from the task. Now we have the following setup which corresponds to how the compile task determines settings for the Java compiler:

Use the release property of the task
If that is not set, its conventions is the release property of the extension
If that is not set, we take the targetCompatiblity of the task
If that is not set, its convention is the targetCompatiblity of the extension/convention
If that is not set, its convention is the current Java version

Follow up to: #12648

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test RFM

Sorry the build you've already triggered is still running, I will not trigger a new one.

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.


Add DraftLog to the list of CLI utilities

This might look like the existing log-update, but it's more simple and accepts multiple updates concurrently.
The Gif says what it can do, and the docs even more: https://github.com/ivanseidel/node-draftlog

By submitting this pull request, I promise I have read the contribution guidelines twice and ensured my submission follows it. I realize not doing so wastes the maintainers' time that they could have spent making the world better. 🖖
⬆⬆⬆⬆⬆⬆⬆⬆⬆⬆

By submitting this pull request, I promise I have read the contribution guidelines twice and ensured my submission follows it. I realize not doing so wastes the maintainers' time that they could have spent making the world better. 🖖

? If you just created something, wait at least 30 days before submitting. This is to give it some time to mature and ensure it's not just a publish-and-forget type of project.
✔ Keep in mind that the list is very mature by now, so the bar of getting something accepted is high. Only submit something unique and generally useful. The world (and this list) doesn't need yet another ORM or framework. For CLI tools, the bar is especially high, and unless it's something very awesome, I would suggest submitting to awesome-cli-apps instead.
✔ If you submit a project that is similar to an existing project in the list, argument how it's better.
✔ Search previous suggestions before making a new one, as yours may be a duplicate.
✔ Suggested packages should be tested and documented.
✔ Make an individual pull request for each suggestion.
✔ Use the following format: package - Description.
✔ Additions should be added to the bottom of the relevant category.
✔ Link to the GitHub repo, not npmjs.com.
✔ Keep descriptions short and simple, but descriptive.
✔ Don't mention Node.js in the description as it's implied.
✔ Start the description with a capital and end with a full stop/period.
✔ Don't start the description with A or An.
✔ Check your spelling and grammar.
✔ Make sure your text editor is set to remove trailing whitespace.
✔ The pull request should have a useful title and include a link to the package and why it should be included.
N/A New categories or improvements to the existing categorization are welcome, but should be done in a separate pull request
By logic, I say that the only reason is Time. Not sure how a maturity can be measured by a threshold, but surely is something to make it easy to manage Awesome Lists.
If that's the only reason, shall we just let the time pass?

If that's the only reason, shall we just let the time pass?

Yes, that's what it says.

One suggestion: Cleanup the list.
There are many items with replicate idea or really low utility in the list. For me, less than 1/4 is considered "Awesome" in the CLI tools. Many don't even have at least 50 lines of code.
Worth consider cleaning up since "the bar is especially high", but with so many not awesome at all tools. Gives space for cool projects
anyway, just a suggestion. ;)
Will notify in 26 days 21 hours and 14 minutes


Define an extensible data processing pipeline

Define a basic pipeline where raw sensor data can be pooled, processed via a number of specialized processing nodes, and then output as actionable data.
Exact architecture TBD. Filing this issue to track the task as it gets further defined.
Example A

Collect n synchronized samples (~same sample time) from an accelerometer, magnetometer and gyroscope
Run data through cleanup and normalization processing nodes
Run sanitized and normalized data through a specialized sensor fusion algorithm to output quaternions
Convert quaternions to Euler angles for human display

Example B

Configure an analog to digital converter
Normalize incoming sample values
Buffer n samples at the start of the pipeline
Run samples through a simple moving average filter processing node
Analyze normalized and averaged data with domain-specific processing node (analog data to rotation speed, saturation level, etc.)
Output final results to update a state machine, etc.

Closing as out of scope, and will be addressed in a separate project.


refactor(derive): drop tokio dependency

The references to 'tokio' in async-dropper-derive are in quoted code, that crate is a proc macro that doesn't need tokio to run (ie. to manipulate token streams).
Leaving it in as a dependency just means tokio would show up twice in the dep graph, built once for async-dropper-derive (as a proc macro dependency and unused), and once as a regular dependency.
Having it as a dependency of async-dropper-derive also doesn't force users of the derive feature to have a dependency on tokio: they can still have compile errors due to the tokio symbol not being found if they don't also depend on tokio in their crate, just like how https://docs.rs/tracing-test/latest/tracing_test/ cannot force your crate to depend on tracing (even though its macro expansion needs it).

Hey @fasterthanlime thanks for the recommendation and PR -- greatly enjoy your reading/watching your adventures.
Change looks good to me, would you mind changing your PR comment to match a conventional commit (as we believe in them). Something like:
refactor(derive): drop tokio dependency

The release tooling does pick this up and actually use it :)

[EDIT] Oh BTW if you'd like me to make this change, please click the little "Allow edits by maintainers" checkbox and I will!

I haven’t seen this checkbox in a while and… here’s why:
https://github.com/orgs/community/discussions/5634
I’ll stop making contributions from orgs I guess!

https://github.com/orgs/community/discussions/5634

TIL
Hey @fasterthanlime actually I should have been clearer -- the commit is actually what matters :)
Something like:

the commit is actually what matters :)

Yup yup, am aware, I was just away from a computer x) amending/force-pushing is more than I'm willing to do from a phone. Should be good now! Interestingly the repo settings are so conservative that they need your approval again to run a second time.


Update for unsafe-any and rust changes

Other deps (rustc-serialize, regex) are still broken, so don't merge yet.

Will need a rebase since I merged in @pyfisch's PR.

regex and rustc-serialize are still broken despite open PRs, so this doesn't really fix anything yet.

regex and rustc-serialize are updated now and compiles on rustc 0.13.0-nightly (c89417130 2015-01-02 21:56:13 +0000)

I just bumped Travis, and serialize errores again...

Oh, seems something is explicitly depending on an older version.

@reem just checked again, Travis errors in hyper now.

Trying to fix... So many errors...

Trying to fix but getting bizarre errors when trying to implement Deref and DerefMut, looks like an associated types error.

@reem Anything I can do to help with this fix?

@Gankro do you have push rights/crates.io rights for the other crates in rust-lang (regex, log, etc.)?

Sadly not. :(

@seanmonstar r?


Update openapi-circe-yaml, openapi-model to 0.6.0

About this PR
📦 Updates

com.softwaremill.sttp.apispec:openapi-circe-yaml
com.softwaremill.sttp.apispec:openapi-model

from 0.4.0 to 0.6.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.sttp.apispec" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.softwaremill.sttp.apispec" }
}]

labels: library-update, early-semver-major, semver-spec-minor, version-scheme:early-semver, commit-count:1

Superseded by #372.


Merge delegate and accessible into core

I would propose merging those two as they both have no dependencies. This would also improve the user experience as only one coordinate needs to be specified.
I would keep accessable in its own module, but maybe name it test-macros or similiar
@ioleo what do you think?

:+1:


std.random.RandomSample and RandomCover are poorly designed

dlang-bugzilla (@CyberShadow) reported this on 2011-12-05T04:00:22Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=7067
CC List

alex (@alexrp)
andrei (@andralex)
b.helyer
bearophile_hugs
code (@MartinNowak)
dmitry.olsh (@DmitryOlshansky)
gassa
issues.dlang (@jmdavis)
jens.k.mueller
joseph.wakeling
monarchdodra

Description
The following tests will always fail:

    int[] a = [ 0, 1, 2, 3, 4, 5, 6, 7, 8 ];
    assert(!equal(randomCover(a, rndGen()), randomCover(a, rndGen())));
    assert(!equal(randomSample(a, 5, rndGen()), randomSample(a, 5, rndGen())));

The reason why these tests will fail is that both functions take the RNG by value. Not only is this unintuitive, this is also a security liability - someone depending on these functions to generate random identifiers can be surprised that two successive calls generate the same ID.

I strongly suggest that RNG types are disallowed from being implicitly copied. Even though pseudo-random number generators shouldn't be used for security purposes, they're still likely to be used in such contexts - especially considering lack of better sources of random data in Phobos.

bearophile_hugs commented on 2011-12-05T04:09:15Z
See also issue 6593

andrei (@andralex) commented on 2011-12-05T21:28:32Z
(In reply to comment #0)
> The following tests will always fail:
>
>     int[] a = [ 0, 1, 2, 3, 4, 5, 6, 7, 8 ];
>     assert(!equal(randomCover(a, rndGen()), randomCover(a, rndGen())));
>     assert(!equal(randomSample(a, 5, rndGen()), randomSample(a, 5, rndGen())));

Good point.

> The reason why these tests will fail is that both functions take the RNG by
> value.
>
> Not only is this unintuitive, this is also a security liability -
> someone depending on these functions to generate random identifiers can be
> surprised that two successive calls generate the same ID.

I think we can safely eliminate this argument from the discussion.

> I strongly suggest that RNG types are disallowed from being implicitly copied.
> Even though pseudo-random number generators shouldn't be used for security
> purposes, they're still likely to be used in such contexts - especially
> considering lack of better sources of random data in Phobos.

The problem with taking a random generator by reference is that it then needs to be escaped. So people would be quite surprised to see that:

auto gen = rndGen;
return randomSample(a, 5, gen);

has undefined behavior.

One way or another we need to solve this, e.g. by creating a wrapper with reference semantics over generators. Ideas are welcome.

bearophile_hugs commented on 2011-12-05T23:53:48Z
(In reply to comment #2)

> The problem with taking a random generator by reference is that it then needs
> to be escaped. So people would be quite surprised to see that:
>
> auto gen = rndGen;
> return randomSample(a, 5, gen);
>
> has undefined behavior.
>
> One way or another we need to solve this, e.g. by creating a wrapper with
> reference semantics over generators. Ideas are welcome.

Turn random generators into final classes?

andrei (@andralex) commented on 2011-12-06T07:53:28Z
> Turn random generators into final classes?

We have backward compatibility to worry about.

dlang-bugzilla (@CyberShadow) commented on 2011-12-06T08:04:49Z
The disadvantages of breaking backwards compatibility need to be considered on a case-by-case basis. I think that turning RNGs into reference types has the potential to be a relatively low-impact change, while also having a good chance of revealing broken code.

The typical usage of std.random does not involve using the RNG types directly, and rather using the implicit thread-local RNG.

An example of affected code:

Random r;
// ... use r

I think that generally allowing such code is a mistake, because it's not clear that the RNG is not seeded.

auto r = Random(42);

We can implement this for backwards compatibility using static opCall (which shall be scheduled for deprecation).

The biggest problem is intentional usage of value semantics (it would transparently turn into reference semantics). Perhaps there's something we could do with the help of opAssign?

joseph.wakeling commented on 2015-02-28T21:51:31Z
(In reply to Martin Nowak from comment #19)
> We had a talk about this during the 2nd D meetup in Berlin.
> One remaining question was how to deal with memory management when using a
> reference type RNG.
>
> I came up with a different solution and less invasive solution.
>
> We simply disable implicit copying of RNGs (@disable this(this)) and ask
> people to use explicitly use .save when copying is indeed intended.
>
> We can even deprecate postblit like so.
>
>     deprecated("Please use .save to explicitly copy RNGs.")
>     this(this) {}
>
> Internally we'd need to change all copying to use std.algorithm.move.

Interesting thought.  We'd have to take care that phobos-side things don't get immediately "fixed" by people just using .save inside RandomCover and RandomSample constructors.

> If reference semantic is wanted people can use either
>     new Random(unpredictableSeed)
> or
>     refCounted(rnd)
>     refCounted(Random(unpredictableSeed))
> or (unsafe)
>     refRange(&rnd)
> .
>
> This should also work for combined random generators like.
>
> new Exponential(Random(unpredictableSeed))
>
> refCounted(Exponential(Random(unpredictableSeed)))
>
> auto rng = Exponential(Random(unpredictableSeed));
> refRange(&rng);

I guess what I don't like about this solution is that it requires the user to take responsibility for generating the RNG (or wrapper-of-RNG) as a reference type, when that ought to be the default behaviour.  But OTOH we could take advantage of the fact that RNG structs are templated, and rework their convenience aliases, e.g.:

    alias Mt19937 = RefCounted!(MersenneTwisterEngine!(uint, 32LU, 624LU, 397LU, 31LU,<PHONE_NUMBER>u, 11LU, 7LU,<PHONE_NUMBER>u, 15LU,<PHONE_NUMBER>u, 18LU))

... which would still leave the underlying struct available for use by someone who really wants to use it directly in that form.

joseph.wakeling commented on 2013-06-20T16:27:56Z
(In reply to comment #15)
> Why do you/we care so much for breaking backwards compatibility with something
> that is so broken? If you let it take the generator by ref I think you will
> remove from bugs from user code.

You can pass the generator to the constructor by ref, but you can't safely store it as a reference type, and so far as I can see you DO need to store it internally -- RandomCover is a range object, not a function where you can just use it and return.

_If_ you could safely store a reference to the RNG, then of course what you say would be right.  But you can't, not while they are value types.

My proposal won't break backwards compatibility (which is not in itself a good thing -- this back deserves to be broken:-) and won't fix the bigger picture, but it will improve the status quo without having to wait for the large-scale redesign that random number generation needs.

joseph.wakeling commented on 2015-03-12T21:48:43Z
@Martin @Jens: sorry for radio silence on this.  It's a busy period, and I recently moved to a new apartment where I still don't have home internet.

> I don't think that sharing rng state among multiple consumers is a good idea.
> Of course it looks like you want random numbers and shouldn't care about the
> order, but often fixed seeds are used to get a reproducible pseudo-random
> range.
>
> When you share such a RNG as in
>    auto rng = Random(fixedSeed);
>    assert(!equal(randomCover(a, rng), randomCover(a, rng)));
> the sequences depends on the implementation of equal and randomCover.

Yes, a case like this obviously creates complications.  But I don't think subtleties like this should prevent a user from instantiating one RNG instance and using it with multiple consumers.  The "default" case (where no RNG instance is passed) already employs a common RNG instance between different consumers; I'd rather have consistency of behaviour whether an RNG is explicitly provided or not.

Of course, we can advise associating different RNG instances with different consumers (or making sure that a consumer does its work before associating an RNG with a new consumer), but that's different from compelling the user to follow this pattern.

Note also that one-RNG-per-consumer really doesn't scale to e.g. a simulation where you are generating thousands or millions of random samples.  The risk of the different RNG seeds generating correlated sequences seems greater than the risks associated with using one RNG underlying all sample instances.

> Also we'd set a bad precedent by making one of std's ranges a ref type. Ref ranges have a lot of subtleties and aren't that well tested with std.algorithm.

I agree there are a lot of subtleties, but my feeling is that we need to embrace and explore those in order to identify what the best way forward is for RNGs.

code (@MartinNowak) commented on 2015-03-02T22:40:58Z
(In reply to Joseph Rushton Wakeling from comment #20)
> I guess what I don't like about this solution is that it requires the user
> to take responsibility for generating the RNG (or wrapper-of-RNG) as a
> reference type, when that ought to be the default behaviour.  But OTOH we
> could take advantage of the fact that RNG structs are templated, and rework
> their convenience aliases, e.g.:
>
>     alias Mt19937 = RefCounted!(MersenneTwisterEngine!(uint, 32LU, 624LU,
> 397LU, 31LU,<PHONE_NUMBER>u, 11LU, 7LU,<PHONE_NUMBER>u, 15LU,<PHONE_NUMBER>u, 18LU))
>
> ... which would still leave the underlying struct available for use by
> someone who really wants to use it directly in that form.

I don't think that sharing rng state among multiple consumers is a good idea.
Of course it looks like you want random numbers and shouldn't care about the order, but often fixed seeds are used to get a reproducible pseudo-random range.
When you share such a RNG as in
    auto rng = Random(fixedSeed);
    assert(!equal(randomCover(a, rng), randomCover(a, rng)));
the sequences depends on the implementation of equal and randomCover.
Would be cleaner to use 2 RNGs with independent state here, so the fact that a  shared state requires to be explicit is a good thing. Also we'd set a bad precedent by making one of std's ranges a ref type. Ref ranges have a lot of subtleties and aren't that well tested with std.algorithm.

code (@MartinNowak) commented on 2015-02-28T20:32:46Z
We had a talk about this during the 2nd D meetup in Berlin.
One remaining question was how to deal with memory management when using a reference type RNG.

I came up with a different solution and less invasive solution.

We simply disable implicit copying of RNGs (@disable this(this)) and ask people to use explicitly use .save when copying is indeed intended.

We can even deprecate postblit like so.

    deprecated("Please use .save to explicitly copy RNGs.")
    this(this) {}

Internally we'd need to change all copying to use std.algorithm.move.

If reference semantic is wanted people can use either
    new Random(unpredictableSeed)
or
    refCounted(rnd)
    refCounted(Random(unpredictableSeed))
or (unsafe)
    refRange(&rnd)
.

This should also work for combined random generators like.

new Exponential(Random(unpredictableSeed))

refCounted(Exponential(Random(unpredictableSeed)))

auto rng = Exponential(Random(unpredictableSeed));
refRange(&rng);

bearophile_hugs commented on 2013-06-20T16:13:04Z
(In reply to comment #14)

> It's putting a sticking plaster on a gaping wound,

Why do you/we care so much for breaking backwards compatibility with something that is so broken? If you let it take the generator by ref I think you will remove from bugs from user code.

joseph.wakeling commented on 2013-06-20T16:02:07Z
I just want to put on record some remarks about short- and long-term solutions to this problem.

Long-term as I think we all agree we need RNGs to be reference types.  In this case the general design of ranges like RandomCover and RandomSample will be not entirely different from what RandomCover has now: the constructor will take a RNG as one of its arguments, and (as it's a reference type) this can be copied (by reference!) to an internal variable:

    struct SomeRandomRange(Rng)
    {
        private Rng _gen;
        ...
        this(/*stuff*/, Rng gen)
        {
            _gen = gen;
            ...
        }
        ...
    }

... and then you'd have some helper functions that would handle the case where the user doesn't specify an RNG by passing rndGen to the constructor.

However, while we're stuck with the situation we have, the design of RandomCover is statistically completely unsafe, as you'll inevitably get unwanted correlations due to storing the RNG by value.  As RandomCover's constructor insists on receiving an RNG, the only way to escape this is to pass it a new RNG seeded with unpredictableSeed.

What we can do, though, is to salvage things minimally by patching RandomCover to use the same technique as RandomSample -- that is, if the constructor gets passed an RNG, copy it; but if not -- call rndGen directly.  I'm going to prepare some patches to that effect, which will also try and do something about the .save methods of Random{Cover,Sample} -- both of which are currently broken.

It's putting a sticking plaster on a gaping wound, but at least it should mean that the simplest case available to the user -- doing something like this:

    auto c = randomCover(input);

... will be statistically reliable.

The caveat here is that we have to remember to tweak the constructors back to insisting on getting an RNG once we can guarantee that RNGs will behave as reference types.  (Is there any kind of template constraint that could be used to guarantee reference copying semantics?)

Anyway, I'll get on with writing the patches.

jens.k.mueller commented on 2015-03-03T15:24:32Z
We should try out Martin's idea.
I'll do it but I'd like to write some tests first. Joseph you mentioned several "suprizes" with the current design. I'd like to create some tests for those. Can you give some code for the flaws in the design?

I'm also looking at http://en.cppreference.com/w/cpp/numeric/random
But adding more generators (including abstractions for non-deterministic) and distributions can be done after this issue has been addressed.

joseph.wakeling commented on 2013-08-21T07:38:13Z
(In reply to comment #14)
> What we can do, though, is to salvage things minimally by patching RandomCover
> to use the same technique as RandomSample -- that is, if the constructor gets
> passed an RNG, copy it; but if not -- call rndGen directly.  I'm going to
> prepare some patches to that effect, which will also try and do something about
> the .save methods of Random{Cover,Sample} -- both of which are currently
> broken.

Just to note, I've submitted a pull request with updates along these lines:
https://github.com/D-Programming-Language/phobos/pull/1499

github-bugzilla commented on 2013-08-29T00:34:46Z
Commit pushed to master at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/8d9233cf8b9e4d27bd70dd0fcd171d2f6dc2f2c0
Partial fix for Issues 7067 and 10434 - std.random.RandomCover

The existing RandomCover design is fatally flawed because it
requires a RNG as input, which it then copies internally by
value.  So, unless the user is smart enough to pass something
like e.g. SomeRNG(unpredictableSeed), there will be unintended
correlations in random behaviour.

This partial fix follows the design of RandomSample in allowing
RandomCover to use the thread-global default RNG rndGen.  It
also improves the choice of template parameter and variable
names in line with Issue 10434.


copr: support Bit column push downBit column

TiKV coprocessor: support Bit column push down
Issue Number: close #11893
TiDB old version not support Bit column push down;
One reason is TiKV not support,  another reason is TiDB set it unsupport.
This PR have Some files and functions update:

components/tidb_query_aggr/src/util.rs
rewrite_exp_for_sum_avg()
components/tidb_query_datatype/src/def/eval_type.rs
TryFrom()
components/tidb_query_datatype/src/codec/chunk/column.rs
from_raw_datums()
from_vector_value()
+append_bit_datum()
4.components/tidb_query_expr/src/impl_cast.rs
cast_bit_as_string()

TiDB's corresponding PR: pingcap/tidb#31884
Tests: test reposrt

Unit test
Integration test
Manual test

Side effects

No side effects.
Be backward compatibility
Need TiDB upgrade together to use the feature.

Release note
none

/run-all-tests


Redirect all Mobile OS's to HTTPS Browser

Closes #521. A more extreme version of #502, simply redirects all mobile (and tablet) operating systems to browser.blockstack.org. We have no native apps yet that can handle blockstack://, and when we do, they can just intercept browser.blockstack.org.
As an aside, we'll probably want to eventually redirect to browser.blockstack.org always, so this is a good step in that direction.
Tested that the following...
Now redirect to browser.blockstack.org

Android 8 Chrome - Pixel 2
Android 8 Firefox - Pixel 2
iOS 11 Safari - iPhone X Simulator

Still redirect to blockstack:// when detected

macOS 10.13.4 Safari
macOS 10.13.4 Chrome
macOS 10.13.4 Firefox

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: wbobeirne:x: larrysalibraYou have signed the CLA already but the status is still pending? Let us recheck it.

Discussion from Slack:
JustinHunter [Yesterday at 5:58 AM]
In this pull request (https://github.com/blockstack/blockstack.js/pull/522)

macOS 10.13.4 Safari
macOS 10.13.4 Chrome
macOS 10.13.4 Firefox```

What does “when detected” mean? Does that mean when the local browser is detected as being installed on the machine?

6 replies
aaronb [1 day ago]
Yeah -- if it detects an application that can handle the `blockstack://` URI protocol, it will still redirect with that method

Also sent to the channel
JustinHunter [1 day ago]
Doesn’t seem to be working properly then. I do not have the local instance of the browser downloaded, but in Firefox and Safari, it tries to open it. In chrome and brave, the handler takes me to browser.blockstack.org as expected.

Also sent to the channel
Hank [1 day ago]
yeah. if you look at the code, we only redirect to the hosted version in the `failCallback`, which i think only works in chromium browsers. we still have the `unsupportedBrowserCallback`, which just tries to force a redirect to the native app. i was discussing this with @larry in https://github.com/blockstack/blockstack.js/pull/524 , and if we made that code change, it would break for users who use safari but have the native app installed, because there isn't a good way to do a `protocolCheck` for that browser

JustinHunter [1 day ago]
Ah I could have swore this had already been solved for. Guess not. Thanks Hank!

Hank [1 day ago]
i'm not sure, but think we can add a timeout or something in `unsupportedBrowserCallback` to see if the location actually changed, and if not (aka if the timeout runs), then we redirect to the hosted version (edited)

Also sent to the channel
larry [1 day ago]
the way it "detects" the protocol handler in chromium is also rather questionable - i believe it tries to detect a UI element appearing


Feature Request - Add "replace text on current" options when I click on the "translate" button

Since a journal often has links to other journals with specific uuid, I need to keep the journal I translate with the same uuid.
Is it possible to add when I click on the "translate" button make a dialog appear with the option to do text replace on the current journal ?
I think a confirmation dialog is more than enough to avoid miss clicks from the user.

Implemented in the v2.1 release as a module setting to enable/disable.


[windows] skip window creation on auth exec

Signed-off-by: goenning<EMAIL_ADDRESS>Motivation
When using kube-rs on a Windows GUI application, the auth exec is showing a terminal window during execution. It often takes less than 1 second, but it's noticeable enough to become annoying :)
https://user-images.githubusercontent.com/94755/205284967-3466c250-8a09-4cd1-b7df-f7e35dd6d05f.mp4
Solution
use the CREATE_NO_WINDOW flag to omit that.

Codecov Report

Merging #1095 (53cb6ce) into main (593fa89) will increase coverage by 0.23%.
The diff coverage is n/a.

:exclamation: Current head 53cb6ce differs from pull request most recent head b9368a6. Consider uploading reports for the commit b9368a6 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1095      +/-   ##
==========================================
+ Coverage   72.39%   72.62%   +0.23%
==========================================
  Files          65       65
  Lines        4774     4819      +45
==========================================
+ Hits         3456     3500      +44
- Misses       1318     1319       +1

Impacted Files
Coverage Δ

kube-client/src/client/auth/mod.rs
56.61% <ø> (+0.23%)
:arrow_up:

kube-runtime/src/wait.rs
75.47% <0.00%> (-1.89%)
:arrow_down:

kube-runtime/src/reflector/mod.rs
100.00% <0.00%> (ø)

kube-runtime/src/events.rs
97.29% <0.00%> (+0.07%)
:arrow_up:

kube-runtime/src/scheduler.rs
96.95% <0.00%> (+0.13%)
:arrow_up:

kube-runtime/src/controller/runner.rs
95.45% <0.00%> (+0.14%)
:arrow_up:

kube-client/src/api/util/mod.rs
93.50% <0.00%> (+0.17%)
:arrow_up:

kube-client/src/client/mod.rs
70.48% <0.00%> (+0.17%)
:arrow_up:

kube-runtime/src/utils/event_flatten.rs
92.30% <0.00%> (+0.20%)
:arrow_up:

kube-client/src/lib.rs
94.00% <0.00%> (+0.20%)
:arrow_up:

... and 9 more


Add text share support

Would be awesome to be able to share text from my Android (currently getting a format not supported error), and get a notification on my Mac with the option to Copy to clipboard.
There was a comment suggesting exactly this, but it didn't have any response:
https://github.com/grishka/NearDrop/issues/16#issuecomment-1736403786
Cheers

I recently wanted to do this, ran into the error and think it would be super useful! +1

Currently implemented by #147 and #112

+1


[UI] Adopt fundamental styles - view History

Target

[ ] Adopt the fundamental styles in History view

Use https://github.com/dirigiblelabs/ide-next as reference

Issue cloned to https://github.com/eclipse/dirigible/issues/1498


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.13 to before v5.15.11",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=69c4a42d72eb9b41e1c6e4bc9ab7f3650bf35f62",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=d1bac0d97bc9b18b7c077a66375cc4282d01c367"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "69c4a42d72eb9b41e1c6e4bc9ab7f3650bf35f62",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "d1bac0d97bc9b18b7c077a66375cc4282d01c367",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "selinux: fix sleeping function called from invalid context\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.15.11 by commit d1bac0d97bc9b18b7c077a66375cc4282d01c367, it was introduced in version v5.13 by commit 69c4a42d72eb9b41e1c6e4bc9ab7f3650bf35f62. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002756


Which of the two integrated packages GPT SoVITS beta0706 and GPT SoVITS beta0306fix2 works better

0706=0217=better than 0306 bs>1
speed:
0706 double speed compared with 0217
0306 at least double speed compared with 0217, but faster when bs>1
0217 == 0306_bs=1？ This needs to be verified


Problem with setConstraintUrl and custom validation

If json with validation is loaded from server you can not setup custom validation for same valdr-type.
Custom validation is deleted with validation from server.
Problem is with this:
var addConstraints = function (newConstraints) {
      angular.extend(constraints, newConstraints);
 };

Need to be user angular.merge();

Why do you have tighter validation on the client than on the server (I understand that's what you're doing)? It should be the other way around.

Not exactly, there is and validation on server but it is not been validation, it is custom validation (because of step when data are validated)


Fix publish script and test --version command

In .travis.yml, cargo run -- --version is used to extract the current package version.
But #106 introduced a small issue, that caused the --version command to be executed twice.
This PR should fix the issue, and adds a test to ensure the version is correctly returned.

oups sorry, thanks for repairing my mistakes!

no worries, that's something we should have caught during the review.


Reset on formly and form makes dirty

In resetting a form validation is run, fields are $dirty and marked as invalid if they're required.
http://output.jsbin.com/pifericawe

Good point. The reset feature should not only reset the model value, but
the NgModelController as well. I'll get this fixed as soon as I am able.
Thanks.

Kent C. Dodds

(Sent from my mobile device, please forgive typos or brevity)
On May 8, 2015 12:30 PM, "Chris Boden"<EMAIL_ADDRESS>wrote:

In resetting a form validation is run, fields are $dirty and marked as
invalid if they're required.
http://output.jsbin.com/pifericawe
—
Reply to this email directly or view it on GitHub
https://github.com/formly-js/angular-formly/issues/278.

Note, if you need a workaround immediately, all fields have the controller
attached to them called formControl. You could reset the dirty and touched
state yourself until I can get this fixed (or submit a PR. Should be fairly
simple...

Kent C. Dodds

(Sent from my mobile device, please forgive typos or brevity)
On May 8, 2015 1:18 PM, wrote:

Good point. The reset feature should not only reset the model value, but
the NgModelController as well. I'll get this fixed as soon as I am able.
Thanks.

Kent C. Dodds

(Sent from my mobile device, please forgive typos or brevity)
On May 8, 2015 12:30 PM, "Chris Boden"<EMAIL_ADDRESS>wrote:

In resetting a form validation is run, fields are $dirty and marked as
invalid if they're required.
http://output.jsbin.com/pifericawe
—
Reply to this email directly or view it on GitHub
https://github.com/formly-js/angular-formly/issues/278.

Ah, thanks @kentcdodds! I was only calling $setPristine(). I didn't think of calling $setUntouched().

Awesome, thanks @kentcdodds!


[CELEBORN-1187] Unify the size and file count of active shuffle metrics for master and worker

What changes were proposed in this pull request?
Unify the size and file count of active shuffle metrics for MasterSource and WorkerSource.
Why are the changes needed?
MasterSource uses PartitionWritten and PartitionFileCount metrics as the size and file count of active shuffle for all workers. Meanwhile, WorkerSource uses ActiveShuffleSize and ActiveShuffleFileCount metrics as the size and file count of active shuffle for a worker including master replica and slave replica. It's recommended to unify the size and file count of active shuffle metrics between MasterSource and WorkerSource.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Internal tests.

changing existing metrics' name should also update the migration guide

@pan3793, thanks for the migration reminder. I have updated the migration guide to record that master metrics PartitionWritten and PartitionFileCountare renamed toActiveShuffleSizeandActiveShuffleFileCount`. PTAL.

@onebox-li, thanks for detailed review of dashboard. I have addressed above comments. PTAL.


how to get doucment id after listDocuments?

how to get doucment id after listDocuments?

I got it by using getAbsoluteName()


Separate asmlib package requirements

Suggesting this patch so that udev persistence does not require asmlib packages.
This is for role: orahost
Split the ASMlib RPM packages out of the main package list and put them in their own.
If user selects device_persistence == 'asmlib' then this RPM group is installed.

Good idea!
You should also add the kmod-oracleasm package to the oracle_asm_packages list

Ah! Can't believe I missed that. How's that?

Merged


Create an error log

We're not saving any information on what has gone wrong. Create an error log for this.

Basic logging in for now. Will come back to this later as necessary.


dep(go): Bump golang version to 1.20.6

Automatic bump of golang-1-release from go version 1.20.5 to version 1.20.6.
Link to golang blobstore package: None
After merge, consider making a new release.

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

v1.1.0
GitHub release

Your semantic-release bot :package::rocket:


Add Stan static analysis tool

Stan is a Haskell static analysis tool. It was created recently, but it already gained some attraction in the Haskell community.

Looks good. Just changed the homepage link before merging to point to the website given in the Github repo.
Thanks @chshersh. 😊


Sidebar default links and library cards widget paths should follow Windows defaults

**Is your feature request related to a problem? No
Describe the solution you'd like
I have multiple drives for different purposes: C: solid state with only Windows and programs for fast loading.  I've set up my documents, music, photos, videos to be saved on my user files HDD (E:) through the windows storage setting.  In Files, I'd like the library cards widget to open the correct libraries given this set up.  Given it's a standard setting in Windows to be able to change the default location for different libraries, I'd expect Files to take me to the correct location when I click the library cards widget links or the default links in the sidebar.

I've removed and added the correct libraries in the sidebar, just can't do something similar in the widget space.

Describe alternatives you've considered
None that I know of.
Additional context
v<IP_ADDRESS>

@meow81 could you check if deleting this file: "%localappdata%\Packages\49306atecsolution.FilesUWP_et10x9a9vyk8t\LocalCache\PinnedItems.json" make the sidebar pick up the new paths?
Note: you'll loose manually pinned folders

@meow81 could you check if deleting this file: "%localappdata%\Packages\49306atecsolution.FilesUWP_et10x9a9vyk8t\LocalCache\PinnedItems.json" make the sidebar pick up the new paths?
Note: you'll loose manually pinned folders

@gave92 just deleted it, restarted the app and it did reset the sidebar, but still points to C: (windows drive) default paths.

@gave92 just deleted it, restarted the app and it did reset the sidebar, but still points to C: (windows drive) default paths.

Merging with #2709.

Merging with #2709.


Windows fixes

This PR contains various fixes for Windows:

disable some 32 bit specific stuff on 64 bit Windows which does not work there any more
shorten path length to enable coq-serapi builds.

The MacOS CI failures are network issues - the Windows CI did run through fine.


Issue no. 213... Font family and bg image

Related Issue

Issue number goes here:  #213

Proposed Changes

Change 1: Updated font family
Change 2: Small change in linear gradient in bg image ( Screenshots are attached below )

Checklist

[x] Documentation of the code
[x] Add your name to the CONTRIBUTORS file
[x] Testing your code in the local machine

Screenshots

@joyjeetcoding change the font family for the navbar rest is ok. Also add a margin between our past events and our programs section.

Okay sure👍🏻


An ability to easily import another file or function from another file!

Often you need to add stuff from files that are not exactly siblings to each other. It'd be really useful If I could do this:

Right click in a source file.
Click "Select for import".
Go to target file and Right click in there
Click "Import selected file"
import statement gets added at the top but under all other imports with something like import  ___  from './d/a/c/d/File;

Simple and non-disruptive.
Then you could go one step ahead, and simply select a function that is being exported and right click the export statement and do the same thing.
Thanks.

Not a duplicate. Closest one I found was drag and drop. I believe not only this approach is simpler but more useful.

This is fundamentally useful feature. Not something I'd like to have. The impact is deep and meaningful.

Sounds like a good fit for an extension! Take look at our extension authoring docs to get started

please reconsider this is needed on daily basis on large code base. autocomplete is useless if you multiple files or functions with same names as is the case in big projects.


About this plugins refreshed question

when i refresh , the  always add one

Hi @Octopus912 , Sorry for the delayed response. Can you provide a minimal demo to reproduce the issue?

Hi @Octopus912 , Sorry for the delayed response. Can you provide a minimal demo to reproduce the issue?

@Runjuu  I think I can elaborate on that issue.
I'm using the latest versions of webpack, html-webpack-plugin, mini-css-extract-plugin and html-inline-css-webpack-plugin itself.
While working with webpack dev server, each time CSS re-compiles, this plugin increments output and breaks output CSS content. This can be tracked by logging style while using styleTagFactory config option.
Let's say I have:
styleTagFactory({ style }) {
  console.log(' ============== styleTagFactory ', style);
  return `<style type="text/css">
/* begin */
${style}
/* end */
</style>`;
},

during the initial build it will output just as expected:
<style type="text/css">
/* begin */
.my-class {
  opacity: 1; }
/* end */
</style>

then if I change CSS (set opacity to 0.5), the second time output will be:
<style type="text/css"><style type="text/css">
/* begin */
.my-class {
  opacity: 0.5; }
/* end */
</style>
/* begin */
.my-class {
  opacity: 1; }
/* end */
</style>

and in the logs there will be 2 entries (old and new):
============== styleTagFactory: .my-class {
  opacity: 1; }

 ============== styleTagFactory: .my-class {
  opacity: 0.5; }

so it looks like the plugin apply twice. And then, if I try to make another change, I will see 3 passes and so on.
Hope there is some workaround or possible quick fix!

@idudinov Huge thanks for providing reproduction steps! I think I’ve found the cause of the problem and release an alpha version 1.11.0-alpha.1 for testing. Let me know if there are any problems.

@Runjuu yep, that worked!! thanks a lot!


this.base gets recycled while component is mounted?

Hey,
this.base gets recycled and reused somewhere else while the a component is mounted, this breaks things for me as I bind events to this.base
this only happens in beta11 not beta10 probably due to 8790b2de4fbd38eb3277cbfc7e609f5332ebbc4d
I could bind my events in some other way but don't see a good way to do it except this.base.addEventLister/this.base.removeEventListener
thanks for any help

Interesting. I'm not sure if that's an unmount bug, or just the intended behavior being corrected.
For your situation, are you rendering a component as the child?  If so I would suggest passing your event handlers down to it as props. Preact (and as far as I can tell React) treats High Order components as simple shells around a concrete component, which means the event handlers for a given component are owned by the innermost HOC.
I'll see if I can find a case where unmount isn't supposed to happen, the commit you linked does appear to be related.

The unmount on the component in question actually does not happen (at least the function is not called) but the element that is this.base during componentDidMount ends up not being the root of the component but one of its children

Ah I saw this on Saturday and wondered. Sounds like the same issue! It's insanely difficult to pin down in tests but I do have a repro that will let me set breakpoints and figure out what is triggering recycling.

Great to hear that! If you need additional testing I can run it on my code as well, have one scenario it repros every time.
Though it is very weird as it is a component I use on pretty much on every page and all other seem to be just fine
The only thing I could think of is I am using 2 copies on this page nested with a bunch of components in between

🔨 Found the proper fix! My repro was a <Router /> wrapping multiple instances of <Preview />, and re-rendering the <Preview /> with different props was incorrectly unmounting Router:

tested with beta12 and it works :+1:

Awesome!  I had to tweak the version a bit, apparently I've been using invalid semver (beyond the obvious piling beta releases into 5.0.0, which I regret...) - the most recent version is now properly formatted but required bumping patch: 5.0.1-beta.12.
I'll close this for now, but this is another reason 5.x needs to get rolled out as stable ASAP!


State not being passed with Link

Version
5.2.0
Test Case
https://codesandbox.io/s/react-router-passing-state-ytk96?file=/src/App.js
Steps to reproduce
Go back and forth between using the link.
Expected Behavior
I'd expect /world route would receive {hello: 'world'} state. It works using <Redirect />.
Actual Behavior
location.state is undefined.

HashRouter doesn't support location.state: https://github.com/ReactTraining/react-router/blob/master/packages/react-router-dom/docs/api/HashRouter.md


Add support for custom error payload

It would be nice to allow to define custom payload when raising exceptions within HTTP handler. Also added documentation around expected_exceptions.

Hi,
thanks for the contribution!
I might be missing something, but what does this add over just
@http('GET', '/expected_custom_exception')
def expected_custom_exception(self, request):
    return 400, json.dumps({
        'error': 'INVALID_REQUEST',
        'description': 'This is invalid request.',
        'code': 400
    })

like your example, the content type is text/plain but should probably be application/json, which can be achieved by returning a 3-tuple (status_code, headers dict, response body) instead of just (status code, response body)

Primary reason for raising vs just returning would be ability to capture and alert on these outside of main code with something like nameko-sentry
I agree application/json should be returned in this instance.

ok, i see what you mean (though one might wonder why your service wants to notify sentry about caller errors. do your webapps log 404s or invalid forms to sentry?)
going along with this though, dumping json into exceptions feels to me like mixing responsibilities. i think a better solution would be to open up for customising the way exceptions are turned into http responses (currently https://github.com/kooba/nameko/blob/custom-error-payload/nameko/web/handlers.py#L94). in fact, an early version of the web handler had a much more generic implementation (see e.g. https://github.com/kooba/nameko/blob/1a416524d52107fc93831bc42076bf5c14588da1/nameko/web/handlers.py#L69) we scaled it back until we could get a better feel for what we might want out of this type of feature.
we might not need something quite this generic (protocol, vs say just a method in charge of this that you can subclass HttpRequestHandler and override to customise)

Sentry is a specific example. Any DependencyProvider can implement worker_result and it seems reasonable that an http entrypoint returning a 4xx/5xx status should follow the "error" path:
def worker_result(worker_ctx, result=None, exc_info=None):
    if exc_info is not None:
        # process worker error

I like the suggestion of an overridable method that transforms an exception into a response.

Superseded by #291


Favicons for feeds don't work without an S3 Bucket

Favicons for feeds are stored in an S3 bucket at the moment. Worth looking into possible ways to decouple from S3, maybe with a container that can act as S3 if that exists, or by storing the favicons for feeds in some other way?

Import opml also fails without an S3

See https://github.com/rach-sharp/freedbin/issues/13#issuecomment-585224049


velocityjs.org not serving https

Just a heads up.
Interesting to authors because direct links to site are failing. (such as those from wikipedia article)

Your system information

VelocityJS version: n/a
Browser: firefox
Operating System: ubuntu

Checklist

Is this an issue with code?: no
Is this an issue with documentation?: yes
Have you reproduced this in other browsers?: yes
Have you checked for updates?: n/a
Have you checked for similar open issues?: yes

Please describe your issue in as much detail as possible:
Connecting to velocityjs.org over https results in error page in firefox and curl and fails openssl test. Looks like apache is just serving a puppet test page.
Steps for reproducing this issue (code):

Navigate to https://velocityjs.org

curl -vv http://velocityjs.org:443
curl -vv https://velocityjs.org:443
openssl s_client -connect velocityjs.org:443

Whatever that was seems to have been temporary, didn't see it when I got this notification (admittedly a while after you posted), and just checked again - no issues on any browsers, so either the server, or some transparent proxy...

@Rycochet -- still seeing it on all fronts here in Boston
https://www.whatsmydns.net/#A/velocityjs.org
https://www.ssllabs.com/ssltest/analyze.html?d=velocityjs.org
Do I have the wrong hostname? Very odd if you are seeing something different

To be clear, http works just fine. https does not.

Meh, I thought it was working, not now apparently - not sure if it wa before but going to say I was tired and misread it. Not something I have any control over at all - @kvirani can you have a look into this?

(Just a note, the website is serving documentation to do with Velocity V1 - so it's not a huge issue that it's not working right at this moment in time, but needs sorting out relatively soon).

No worries, thanks for takin' a second look :)


Class[Nftables::Bridges]['bridgenames'] contains a Regexp value. It will be converted to the String '/^br.+/'

Affected Puppet, Ruby, OS and module versions/distributions

Puppet: 6.20.0

Noticed in our puppetserver logs:
2021-03-23T07:28:07.219+01:00 WARN  [qtp1376093148-42] [puppetserver] Puppet Class[Nftables::Bridges]['bridgenames'] contains a Regexp value. It will be converted to the String '/^br.+/'

I don't understand the issue. It should be a regexp and it also is: https://github.com/voxpupuli/puppet-nftables/blob/18b211e72a046db71acda7bf2fad8144b9e72e8b/manifests/bridges.pp#L5
Why is it complaining?

Agree , i can't see what might be wrong if anything.

https://tickets.puppetlabs.com/browse/PUP-7735 gives some insight.

Can't reproduce it with the example:
# puppet apply foo.pp
Notice: Scope(Class[main]): Regexp[/^.$/]
Notice: Scope(Class[Foo]): Regexp[/^.$/]
Notice: Compiled catalog for xyz in environment production in 0.02 seconds
Notice: Applied catalog in 0.30 seconds
# puppet --version
7.5.0

And I also don't see it in our puppetserver logs.

@duritong that's with a Puppet 7 you do not see message ? If so suggest we ignore.

Ignoring as apparently not present in newer puppets.


Recommend toolerific.ai

Toolerific
https://toolerific.ai/
Find the best AI tools for your tasks with Toolerific,Unlock 10,000+ AI tools for innovation and productivity.Simply browse our categorized listings or use our intelligent search feature to find tools that align with your project's needs and objectives.

please send a PR


Multiple Sticky components per-Container are contraindicated.

In the event that two Sticky nodes are added to the same Container, both will independently stick when scrolling (as intended).  However, if they share horizontal space, they will overlap one another rather than "stack".
This is not an immediate problem (and may sometimes be desirable), but likely warrants some documentation and possibly a warning.

is there a solution to this? if not, why it is close?

This is working as intended, sticky containers tell a sticky element when it's at the top of the screen. There is no support for multiple sticky elements within a single container.


Polygon flow: Add loading indicator, prevent user from exiting until uploaded / failed, remove ERC20 mint fee

Add loading indicator
Prevent user from exiting until uploaded / failed
Remove ERC20 mint fee
Goal is a nice user experience for minting to polygon, as fast as possible.

@supernftier this has changed a little
We want the mint to be asynchronous. So all same, show loading indicator, but don't block user screen. User can close window while in progress and continuing walking around in 3D.
When minted, show toast indicating mint was successful (like other toasts) and indicate that item was added to user's inventory


Add inferred_type field to Expr struct

Codecov Report

Merging #285 (f0c55f4) into main (b1ae7de) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #285      +/-   ##
==========================================
- Coverage   90.12%   90.10%   -0.02%
==========================================
  Files          32       32
  Lines        4829     4833       +4
==========================================
+ Hits         4352     4355       +3
- Misses        477      478       +1

Impacted Files
Coverage Δ

crates/crochet_ast/src/expr.rs
0.00% <ø> (ø)

crates/crochet_infer/src/infer_expr.rs
94.08% <ø> (ø)

crates/crochet_parser/src/lib.rs
96.63% <100.00%> (+0.01%)
:arrow_up:

crates/crochet_types/src/type.rs
80.55% <0.00%> (-5.56%)
:arrow_down:

crates/crochet_infer/src/util.rs
96.03% <0.00%> (+0.49%)
:arrow_up:

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


Upload of pre-defined races

Hello ryan,
i want to create a race with: http://www.pacecreator.com/index.php
It would be fine to upload the created race to the watch

I'll have a look into this when I get some history data on my watch and see what information is actually stored on the watch. My watch died so I just got a new one (fortunately under warranty) and I haven't had a chance to get some data on it yet.

If you need some data from the watch let me know.

Ok, I went for a run this afternoon and looked at the data the watch generates for the "race myself" function. Essentially, this should be fairly easy to implement, although to make it simpler and easier, I think I'll implement the race generation completely, rather than loading a file generated by that website. Something like:
ttwatch --create-continuous-race r,"3km@4:00"
ttwatch --create-continuous-race t,"3km@240s"
ttwatch --create-continuous-race c,"12km/1:00:00"
ttwatch --create-continuous-race f,"2km@3:00,10km@4:00"

The first letter of each option value would be the activity type. The first two commands would create the same race (but for different activities), creating a race of 3km at a pace of 4min/km. The third would create a cycling race of 12km in 1hr. The fourth would create a freestyle race of 2km at a pace of 3min/km followed by 10km at 4min/km. How does that sound?
I can't think of a better option name at the moment, but if you have any ideas...

I have pushed an update that implements this. Let me know if it works for you. It's basically exactly as I specified above.

Wow! That sounds great. Is there a way  to send you an paypal donation, for a beer for example, ryan? :-)

Thanks for the thought, but no, that's ok. I don't drink anyway ;-)

Only one question Ryan. Is there any limitation in this function for <distance/time> / @
or is it possible to create a race with 21 or even 42 values?

No, there's no limitation. It will keep on extending the race for as long as there are values left in the string.

Thanks. So the only limitation is the watch itself and i guess only TomTom knows how many values are possible.


Fix Unknown DOM property tabindex and accesskey

Fix 2 warning

Warning: Unknown DOM property accesskey. Did you mean accessKey?
Warning: Unknown DOM property tabindex. Did you mean tabIndex?

Thanks!

Hey, can you please release a patch update to NPM with this change? I'm still getting this error in 0.15.0. Thanks!

Okay, done as 0.15.1.


15 finish tests for root sweet command

Added test cases for root sweet cmd
Fix single test case
Tidy up code
Move single test case into array of test cases
Add more test cases
Added test todos
Add building and testing pro tip to readme
Remove unwanted print statements
Test root cmd with stdin
Added language selection flag and tests
Finish testing branches for root fromArgs function

Fixes #15, #11, and #14

This is a comment but I don't konw what this does.


"JavaScript heap out of memory" when continuously running too many scenarios

I encountered the problem when migrating to 2.0.
After tracing code in Serenity/console-reporter,
I found that ActivityRelatedArtifacts recorded too many Artifact data when continuously running those scenarios, and those data would not be delete after each scenario finished, which probably occupy too much heap and cause allocation fail.

Yeah, it looks like that bit of code might be using too much memory. That's a nice catch, thanks @wallybb000! Could you please tell me a bit more about what you mean by continuously running scenarios? How exactly do you run them?


(SOLVED)

(ISSUE SOLVED)

refering to the manual, you can pass a set of parameters while creating the instance of Instaloader.
parameters you may like are save_metadata=false, filename_pattern(...).

refering to the manual, you can pass a set of parameters while creating the instance of Instaloader.
parameters you may like are save_metadata=false, filename_pattern(...).

@saad-sifar deleting title and issue text is a bad idea. Other people who want to look after their problem can't use the information anymore!

@saad-sifar deleting title and issue text is a bad idea. Other people who want to look after their problem can't use the information anymore!


fix: add '~' to sentry source path #trivial

The type of this PR is: Bugfix
This PR resolves CX-956
Description
Follow-up to https://github.com/artsy/eigen/pull/4368 which correctly uploaded the artifacts to the right release but still weren't getting deobfuscated js code in errors.
This

adds a tilde '~' to the source file upload which is a special character in sentry to ignore protocol when matching sources to errors
calls setRelease explicitly to workaround an issue in sentry lib that was not respecting release param in init call

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a CHANGELOG.yml entry or my changes do not require one.

Self-merging this one since it is such a trivial change so I can test in beta, but let me know if anything looks off!


fix various problems with iptables.Exists

This modifies iptables.Exists so that it must be called with an explicit
table and chain.  This allows us (a) to generate an appropriate command
line for iptables -C, which was not previously possible, and (b) it
allows us to limit our strings.Contains() search to just the table and
chain in question, preventing erroneous matches against unrelated rules.
Resolves #10781
Signed-off-by: Lars Kellogg-Stedman<EMAIL_ADDRESS>

ping @icecrime
I think you were looking at this Exist method also

@crosbymichael  @icecrime ping anybody? I'm trying to keep this up-to-date with master, but if someone else is working on a better solution I'll stop paying attention.

This looks fine to me, but I am still confused on the problem you were seeing, can we maybe have an integration test so we can be sure there are no regessions

Yes, I had an issue with Exists() that this PR does seem to solve: in the case where iptables -C isn't enough in determining if a rule exists, we fallback to parsing the output of iptables-save. However, this parsing logic is flawed: it searched for a matching rule in all chains and tables.
So definitely +1 on the design, but I only very superficially reviewed the code for the moment.

LGTM

This looks good, but is there any reason not to use the const's defined for table names from pkg/iptables in the bridge driver?  Seems like that would keep a future change of table name from having to spread itself across the code.

+1 to @estep 's question

I've removed literal table names from daemon/networkdriver/bridge/driver.go.

Thanks! LGTM

I am working on https://github.com/gdm85/go-libiptc, although I plan to use it myself for small projects (until nftables becomes mainstream) please let me know if it could be an option for integration in Docker (pkg/iptables?). It could ease maintaining iptables-related codebase and in general more strict/consistent error checking of iptables results.
All feedback welcome :)


Exclude process or process binary - which is more secure?

Which is better from a security standpoint:
Excluding a process or the process binary?

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

ID: 7d6bcf39-9a8c-7fa3-803a-508e7b74cc9b
Version Independent ID: 9fd6d652-df16-7f4b-d253-6aee2362f836
Content: Configure and validate exclusions for Microsoft Defender for Endpoint on Linux
Content Source: microsoft-365/security/defender-endpoint/linux-exclusions.md
Product: m365-security
Technology: mde
GitHub Login: @Dansimp
Microsoft Alias: dansimp

@foss4ever Thank you for your feedback. Exclusion of process is better as it excludes the process and all files opened by it.
Hope this helps!
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @foss4ever Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Add puml files for find command

Codecov Report

Merging #223 (b709ece) into master (6d61858) will increase coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #223      +/-   ##
============================================
+ Coverage     73.87%   73.89%   +0.02%
- Complexity      772      775       +3
============================================
  Files           109      109
  Lines          2519     2521       +2
  Branches        274      273       -1
============================================
+ Hits           1861     1863       +2
  Misses          584      584
  Partials         74       74

Impacted Files
Coverage Δ

src/main/java/seedu/fast/model/person/Email.java
90.00% <0.00%> (ø)

...ava/seedu/fast/logic/parser/FindCommandParser.java
79.48% <0.00%> (+1.10%)
:arrow_up:

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 6d61858...b709ece. Read the comment docs.


Empty document after drawing

Hi,
lately, boxdraw would empty my whole files after I used any of the boxdraw commands.
When using +O I can enter a label, but vim show the following message:
Shell returned 1
x lines filtered

My setup:

vim 8.1
Windows 10
Python 3.6.5

Since I've run out of ideas why this happens, maybe you can give me a hint?
Best regards and thanks in advance
Thomas

Hi Thomas,
The plugin works by sending the buffer contents to a Python script via stdin, and replacing it with the script output. The script has no external dependencies, only Python. So I think something basic is missing, and the script cannot start.
Try executing the script from a shell:
cd ~/.vim/bundle/vim-boxdraw/python
./boxdraw.py +O 0 0 2 10 hello

Enter a few empty lines, then press Ctrl-D (maybe it's Ctrl-Z in Windows, the point is to stop stdin).
It should return something like this:
+---------+
|  hello  |
+---------+

I haven't tested the plugin in Windows, maybe it should execute Python differently there (e.g. run python.exe boxdraw.py instead of just boxdraw.py). This is the part that calls the script, maybe it should be modified a bit: https://github.com/gyim/vim-boxdraw/blob/dc2800086356be09c41876e8c67b4de8fca71b45/plugin/boxdraw.vim#L25-L35

Hi Akos,
thanks for your reply!
When I execute the script as you show above, it immediately returns to the
shell (cmd and powershell) without any output.
I don't even have the occasion to enter CTRL-D/CTRL-Z.
Update to python 3.7.1 also didn't bring any success.
Thomas
On Thu, Dec 13, 2018 at 8:41 AM Akos Gyimesi<EMAIL_ADDRESS>wrote:

Hi Thomas,
The plugin works by sending the buffer contents to a Python script via
stdin, and replacing it with the script output. The script has no external
dependencies, only Python. So I think something basic is missing, and the
script cannot start.
Try executing the script from a shell:
cd ~/.vim/bundle/vim-boxdraw/python
./boxdraw.py +O 0 0 2 10 hello
Enter a few empty lines, then press Ctrl-D (maybe it's Ctrl-Z in Windows,
the point is to stop stdin).
It should return something like this:
+---------+
|  hello  |
+---------+
I haven't tested the plugin in Windows, maybe it should execute Python
differently there (e.g. run python.exe boxdraw.py instead of just
boxdraw.py). This is the part that calls the script, maybe it should be
modified a bit:
https://github.com/gyim/vim-boxdraw/blob/dc2800086356be09c41876e8c67b4de8fca71b45/plugin/boxdraw.vim#L25-L35
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gyim/vim-boxdraw/issues/3#issuecomment-446871777, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADXepYsZSfbm0b3u5W5IXaDLHnUxysyGks5u4gSwgaJpZM4ZQDYU
.

--
threema://add?id=TD2J9FS2
XMPP/Jabber<EMAIL_ADDRESS>PGP fingerprint:
7EF9 C540 E713 E081 90E5 3913 0F01 8742 F245 D553
Find me on https://keybase.io/treuss

Hi Akos,
sorry for disturbing you again.
I found the root cause while experimenting with the Python script. It
cannot be run using the SheBang-Line, like in Linux.
Windows does not execute the python script the right way. I don't know why,
but if I modify your plugin code, boxdraw works again:
let s:drawscript = *"python " . *expand(':p:h:h') .
"/python/boxdraw.py"
Best regards
Thomas
On Thu, Dec 13, 2018 at 8:41 AM Akos Gyimesi<EMAIL_ADDRESS>wrote:

Hi Thomas,
The plugin works by sending the buffer contents to a Python script via
stdin, and replacing it with the script output. The script has no external
dependencies, only Python. So I think something basic is missing, and the
script cannot start.
Try executing the script from a shell:
cd ~/.vim/bundle/vim-boxdraw/python
./boxdraw.py +O 0 0 2 10 hello
Enter a few empty lines, then press Ctrl-D (maybe it's Ctrl-Z in Windows,
the point is to stop stdin).
It should return something like this:
+---------+
|  hello  |
+---------+
I haven't tested the plugin in Windows, maybe it should execute Python
differently there (e.g. run python.exe boxdraw.py instead of just
boxdraw.py). This is the part that calls the script, maybe it should be
modified a bit:
https://github.com/gyim/vim-boxdraw/blob/dc2800086356be09c41876e8c67b4de8fca71b45/plugin/boxdraw.vim#L25-L35
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gyim/vim-boxdraw/issues/3#issuecomment-446871777, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADXepYsZSfbm0b3u5W5IXaDLHnUxysyGks5u4gSwgaJpZM4ZQDYU
.

--
threema://add?id=TD2J9FS2
XMPP/Jabber<EMAIL_ADDRESS>PGP fingerprint:
7EF9 C540 E713 E081 90E5 3913 0F01 8742 F245 D553
Find me on https://keybase.io/treuss

Hi @treuss,
Sorry, I forgot about this ticket. Can you test it with PR #9? It will hopefully fix the problem.

Hi @gyim ,
looks like the issue is gone 👍
I did a :PluginUpdate in Vundle.
Thanks and best regards
Thomas

Thank you for the test! I'll close this ticket then.


Manual example of python expressions with math.sqrt does not work

The example for python formatting in the manual:
python-expressions
does not work because an import math seems to be missing. For the simpler expression:
/py math.sqrt(1.0)

I get the error name 'math' not defined.
Is there some place where I can import modules used in debug expressions? If so, the manual should mention this.
OS: macOS Catalina
VSCode version:  1.48.2
Extension version: 1.6.0

You need "initCommands": ["script import math"]

Sorry for not following upon this - indeed adding "initCommands": ["script import math"] to the settings does work.


Added Lithuanian keyboard pack.

This is based on the original Lithuanian pack still available on Google Play, but vastly expanded.
In addition to the original three-row QWERTY layout, this pack now also ships:

four-row QWERTY-based layout (similar to the default layout on all PCs)
three-row ĄŽERTY-based layout (similar to the official Lithuanian standard keyboard)
16-key vertical layout (which switches to the the aforementioned four-row layout in landscape mode)

I have also cleaned up some files which are generated at compile time and added them to .gitignore.
On top of all that, I am adding a crowd-sourced autocorrection dictionary. That one needs some reviewing, but I think it should be a good start.
The screenshots under StoreStuff were taken on my Samsung Galaxy S2 phone. I couldn't find the theme that was used for the original screenshots, so the ones I made are quite dark and might need to be re-taken.

Similarly to #11, I think you will want to create a new branch before merging. If that is not possible, I can just create a new PR when a branch is initiated.

Great work! Thank you.
I'll create a new branch for this pack.
On Tue, Jun 7, 2016, 4:46 AM Rimas Kudelis<EMAIL_ADDRESS>wrote:

Similarly to #11 https://github.com/AnySoftKeyboard/LanguagePack/pull/11,
I think you will want to create a new branch before merging. If that is not
possible, I can just create a new PR when a branch is initiated.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/AnySoftKeyboard/LanguagePack/pull/16#issuecomment-224217898,
or mute the thread
https://github.com/notifications/unsubscribe/AAdX-OYoeRgnQTaU_VN0rRscq1PqF2Xcks5qJS_MgaJpZM4IvsOB
.

Created https://github.com/AnySoftKeyboard/LanguagePack/tree/Lithuanian
Please recreate this PR and target the new branch.


Failing to Implement my own ICONS

Hi again,
I tried to implement my own icon but could not figure out.
tried to use this code t
engl_flag_ic = Icon(name="/assets/uk_round_flag.ico")
icon_lang = icon_button_creator(ic_color=None,ic_size=text_size*1.5,ic_tip="language",  ic=engl_flag_ic, ref=icon_lang_ref)
Error message:
Icon button does not have an icon neither content specified.

I also tried to use it with Image but could not manage it as well.
Also I looked into the icons. py tried to implement my icon address from there. However I could  not manage it  again.

Can you please add a guildeline for it?
I wanted to the ask these questions from stackoverflow but my popularity score was below the required level.

I don't think .ico format is supported by Flutter at the moment and there is an open issue to add that support: https://github.com/flutter/flutter/issues/105848
Convert .ico to PNG, so you can use it with Flet.

I am trying to use it on a button. But I will try. Thank you very much feodor. I had another question. I hope you may find the time to look into it. Thank you very much for your wonderful work and effort. When I finish my app I will mention your name in credits.


Build 1st iteration of rest server authentication and authorization

[x] Vagrant
[ ] Auth and Auth
[x] Create new repo for Yakchat Auth Server

Repo:

Closed .. moved to yakchat server dashaboard


Regarding Embedding size

While going through the paper, the dimension or size of the embedding (mentioned as d in the paper) was equal to the size of the memory vectors.
In your implementation of the code, can you clarify how you are using the default size of embedding == 20? Was this used to pad extra null to the sentences or sequences?

In section 4.4 it says they used d=20 for independent training.
We report a variety of design choices: (i) BoW vs Position Encoding (PE) sentence representation;
(ii) training on all 20 tasks independently vs jointly training (joint training used an embedding
dimension of d = 50, while independent training used d = 20); (iii) two phase training: linear start
(LS) where softmaxes are removed initially vs training with softmaxes from the start; (iv) varying
memory hops from 1 to 3.

Oops I misread your question. They're used in the dimensions of the embedding matrices A, B, C, and doesn't have anything to do with padding.

Okay. Thanks for your help. I did understand the concept of embedding matrices and their dimensions but got confused with how it was used to embed each word.
As mentioned in 4.1, they have used BoW representation for embedding words but BoW represents a word via an integer and not a vector. Hence, what is the significance of these dimensions?

Or are these dimensions completely unrelated while embedding words? I think you have simply changed words to indices for representing text in the memory network.

The term Bag of Words is used in the paper to indicate that the order of the words is not handled in the embedding of the sentence, and is not the canonical BoW where you use counts. Each word in the sentence is mapped to an index (which is the word embedding in the embedding matrix), and all the word embeddings are summed to produce a single vector representing the sentence (see section 4.1, Sentence Representation). In the Positional Encoding (PE), l_j was added to the summation in order to capture the order of the words.

Okay. Thanks for the help!


not enough arguments in call to l.listener.Close

Hi , when i try to install this package it gives me the following error:
github.com/go-ocf/go-coap/net
../../go/src/github.com/go-ocf/go-coap/net/dtlslistener.go:126:25: not enough arguments in call to l.listener.Close
have ()
want (time.Duration)
I have already installed this package many times and today this is happening.
Help me please it's urgent !
Go version 1.12.9
Thank you.

Hi. The issue is that pion/dtls changes API. go-coap is fixed to version 1.5.0 https://github.com/go-ocf/go-coap/blob/92481866bd1713e2c98a02933f472c0f8a28a91c/Gopkg.lock#L22
I locked to the new version 1.5.1 but it still use Close function without timeout argument.
To fix your issue you can use dep utility  (https://github.com/golang/dep) in you project for now.

To fix your issue you can use dep utility (https://github.com/golang/dep) in you project for now.

I'm new to golang, care to explain what I should/could do?

$ dep ensure did the trick.

Fixed and updated to 1.5.2.


Make a manually-run test to generate a saved document with full feature coverage

Right now, the check_if_graphite_file_version_upgrade_is_needed test opens the graphite-test-document.graphite file and checks if the deserialization fails due to an outdated format. If it fails, the developer should bump the document version number and manually save a new file to replace it.
But that file doesn't use all the features (such as layer types) that are serialized out of the set of all possibilities. The developer should try to make this use lots of features. But this should really be a manually-run test (which is disabled in the test suite, but can be run by the developer) which has the side effect of replacing the file with one that uses the editor and makes some layers of different types in order to maximize coverage of the features used and serialized into the file.
There should be instructions on how to run the test, and a reminder to bump the GRAPHITE_DOCUMENT_VERSION.

This is probably not as necessary anymore now that things are generally all built around the centralized node graph, so it's more a matter of testing individual nodes not various forms of content features like in the early days.


feat(Archive/Imo): formalize IMO 1982q3

Formalized problem 3 of 1982 IMO.
Co-authored by: Violeta Hernández Palacios<EMAIL_ADDRESS>

@AlexBrodbelt, I have the impression that you have quite a few tricks yet to learn about writing analysis arguments in Lean.  As an example, I have pushed an efficient proof of your first lemma.
The "Archive" is for high quality proofs that, while "leaves", show off the automation and libraries developed in Mathlib.  At the moment this proof is quite far from being this kind of showcase.  If you want to work on getting this proof to that level, it will take a lot of back-and-forth, and this may proceed rather slowly, depending on reviewers' availability to point out inefficiencies.
If you just want to make this proof available in a public repository for the community to use, you might consider instead contributing it to https://github.com/dwrensha/compfiles.  Are you aware of that project?

FYI I added this solution to Compfiles: https://github.com/dwrensha/compfiles/commit/e7f00a521a0e1b5c0a80b4e759ff26e2c61ffc4d

@AlexBrodbelt I did a complete rewrite of the proof on my own branch. It ends up being only about 100 lines long, and I think it better showcases Mathlib's automation. If you think what I did is sensible, you can merge the changes in that branch with this branch. Note that my proof depends on PR #19311.

Since there's been no activity from OP since September, I've decided to take over this PR. The new proof is an almost complete rewrite from the original one, and makes use of the version of Sedrayan I had previously PR'd onto Mathlib.

Thanks!
bors merge


clock: add to track

A port of the xjava implementation.
Canonical data

Thanks for the work on this.


Fix throwException formatting in generated Typescript clients

Fixes the formatting of generated Typescript clients (adds a space in the if-statement).
function throwException(message: string, status: number, response: string, headers: { [key: string]: any; }, result?: any): any {
    if(result !== null && result !== undefined)
        throw result;
    else
        throw new SwaggerException(message, status, response, headers, null);
}

This makes formatting consistent with other if-statements within the generated clients. It's something that autoformatters will often change otherwise.

Thanks for the pr

Thanks for merging, that was fast! 😄


[Feature request] Text object for comments

Basically the same as https://github.com/tomtom/tcomment_vim/issues/237

I think this should be three regions. I agree with the examples you made.

https://github.com/RRethy/nvim-treesitter-textsubjects has single and multiline comment support in the textsubjects-smart query (although there is a bug in tree-sitter queries which makes it limited), another query could be added specifically for comments.

@RRethy If that is possible then it would be great ;)

I just discovered that https://github.com/nvim-treesitter/nvim-treesitter-textobjects also have @comment.outer.

I just discovered that nvim-treesitter/nvim-treesitter-textobjects also have @comment.outer.
(Also mentioned at #77 (comment))

Unfortunately, it doesn't work with line comments very well:

@comment.outer only matches a single line when there are multiple line comments in a row.
@comment.outer matches exactly the comment instead of the whole line, so Comment.nvim will turn a line comment as the content of a block comment. e.g. it turns // comment to /* // comment */.

@ouuan Yes, it doesn't work with line comments. Although it works very well with block comments.
Regarding the feature itself, I am still confused as to whether should I implement this myself or let the other plugins like https://github.com/nvim-treesitter/nvim-treesitter-textobjects or https://github.com/RRethy/nvim-treesitter-textsubjects provide the text objects. If any of the plugins can achieve this https://github.com/numToStr/Comment.nvim/issues/11#issuecomment-944922532 behavior then I would recommend using that plugin.

I agree that Comment.nvim doesn't need to implement this feature. It could be a part of another plugin or a new plugin.

There's also a problem with multiline block comments.
Comment.nvim adds // at the beginning of each line:
/**
 * multiline comment
 */

I think you are using the gc bindings. Try with gb.

@ouuan Also make sure that the plugin contains queries for comments. The last time I checked https://github.com/nvim-treesitter/nvim-treesitter-textobjects was missing comments queries for some languages.

Yes, and help is welcome :)

There's a bug in tree-sitter queries which is blocking on this, you can see it with this query:
(((_) @head . (comment) @_start . (comment)+ @_end (_) @tail)
    (#not-has-type? @tail "comment")
    (#not-has-type? @head "comment")
    (#make-range! "range" @_start @_end))

It will find a match in this file:
def foo
  # foo
  # foo
  # foo
  puts 'hello'
end

But it will fail to find a match in this file with more comments, even though there should be a match.
def foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  # foo
  puts 'hello'
end

I haven't filed anything yet, just been busy.

mini.comment already has this, perhaps inspirations can be drawn there? I'm not aware of the architecture but this would be really helpful, since as of now I have to choose between block comments or comment text objects, or make some Chimera of the two, none of which is ideal.


Customization of task view

such as today vs upcomming, today vs tomorrow, today vs next 10, today vs etc.

Duplicate issue.


highlight infos

highlight docs, types, definitions etc. use infos from here and here

highlight in output channel directly

pending on upstream issues
https://github.com/Microsoft/vscode/issues/5274
https://github.com/Microsoft/vscode/issues/571

create a html view

pending on upstream issue
https://github.com/Microsoft/vscode/issues/272

a good approach


Tab and " with french keyboard not working

Hi
for example I start with that:
1
2
3
I record a macro that transform the 1st line in that:
<hard tab>"1"
and the result with macro execution on next line is :
323
333
version used 0.8.0
Thanks

I fixed about hard tab, maybe.
But I don't know about french keyboard, so don't fix about ".
Do you know key code of " in french keyboard?

Thanks for your update the hard tab and the " work.
this is weird because you say that you haven't do anything with the ".


Upgrade Project Reactor 2023.0.6 -> 2023.0.7

This PR contains the following updates:

Package
Type
Update
Change

Project Reactor (source)
import
patch
2023.0.6 -> 2023.0.7

Release Notes

reactor/reactor (Project Reactor)
v2023.0.7
Compare Source
2023.0.7 release train is made of:
reactor-core 3.6.7
reactor-netty 1.1.20
reactor-pool 1.0.6
These artifacts didn't have any changes:
reactor-kafka 1.3.23
reactor-addons 3.5.1
reactor-kotlin-extensions 1.2.2

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade Project Reactor 2023.0.6 -> 2023.0.7

See:
- https://github.com/reactor/reactor/releases/tag/2023.0.7
- https://github.com/reactor/reactor/compare/2023.0.6...2023.0.7