text:
Atom 1.25.0 Rebuild Package Failure

remote-sync-pro 5.1.1
I've just upgraded to Atom 1.25.0 and remote-sync-pro is failing to rebuild (and thus doesn't work)
I'm running Atom on Mac OS X
node-pre-gyp ERR! Tried to download(404): https://fsevents-binaries.s3-us-west-2.amazonaws.com/v1.1.2/fse-v1.1.2-electron-v1.7-darwin-x64.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for fsevents@1.1.2 and electron@1.7.11 (electron-v1.7 ABI) (falling back to source compile with node-gyp)
xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance
xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance
No receipt for 'com.apple.pkg.CLTools_Executables' found at '/'.
No receipt for 'com.apple.pkg.DeveloperToolsCLILeo' found at '/'.
No receipt for 'com.apple.pkg.DeveloperToolsCLI' found at '/'.
gyp: No Xcode or CLT version detected!
gyp ERR! configure error
gyp ERR! stack Error: gyp failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (/Applications/Atom.app/Contents/Resources/app/apm/node_modules/node-gyp/lib/configure.js:305:16)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:191:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12)
gyp ERR! System Darwin 17.4.0
gyp ERR! command "/Applications/Atom.app/Contents/Resources/app/apm/bin/node" "/Applications/Atom.app/Contents/Resources/app/apm/node_modules/.bin/node-gyp" "configure" "--fallback-to-build" "--module=/Users/mup/.atom/packages/remote-sync-pro/node_modules/fsevents/lib/binding/Release/electron-v1.7-darwin-x64/fse.node" "--module_name=fse" "--module_path=/Users/mup/.atom/packages/remote-sync-pro/node_modules/fsevents/lib/binding/Release/electron-v1.7-darwin-x64" "--python=/Applications/Atom.app/Contents/Resources/app/apm/bin/python-interceptor.sh"
gyp ERR! cwd /Users/mup/.atom/packages/remote-sync-pro/node_modules/fsevents
gyp ERR! node -v v6.9.5
gyp ERR! node-gyp -v v3.4.0
gyp ERR! not ok
node-pre-gyp ERR! build error
node-pre-gyp ERR! stack Error: Failed to execute '/Applications/Atom.app/Contents/Resources/app/apm/bin/node /Applications/Atom.app/Contents/Resources/app/apm/bin/../node_modules/.bin/node-gyp configure --fallback-to-build --module=/Users/mup/.atom/packages/remote-sync-pro/node_modules/fsevents/lib/binding/Release/electron-v1.7-darwin-x64/fse.node --module_name=fse --module_path=/Users/mup/.atom/packages/remote-sync-pro/node_modules/fsevents/lib/binding/Release/electron-v1.7-darwin-x64 --python=/Applications/Atom.app/Contents/Resources/app/apm/bin/python-interceptor.sh' (1)
node-pre-gyp ERR! stack     at ChildProcess. (/Users/mup/.atom/packages/remote-sync-pro/node_modules/fsevents/node_modules/node-pre-gyp/lib/util/compile.js:83:29)
node-pre-gyp ERR! stack     at emitTwo (events.js:106:13)
node-pre-gyp ERR! stack     at ChildProcess.emit (events.js:191:7)
node-pre-gyp ERR! stack     at maybeClose (internal/child_process.js:877:16)
node-pre-gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:226:5)
node-pre-gyp ERR! System Darwin 17.4.0
node-pre-gyp ERR! command "/Applications/Atom.app/Contents/Resources/app/apm/bin/node" "/Users/mup/.atom/packages/remote-sync-pro/node_modules/fsevents/node_modules/.bin/node-pre-gyp" "install" "--fallback-to-build"
node-pre-gyp ERR! cwd /Users/mup/.atom/packages/remote-sync-pro/node_modules/fsevents
node-pre-gyp ERR! node -v v6.9.5
node-pre-gyp ERR! node-pre-gyp -v v0.6.36
node-pre-gyp ERR! not ok
npm ERR! Darwin 17.4.0
npm ERR! argv "/Applications/Atom.app/Contents/Resources/app/apm/bin/node" "/Applications/Atom.app/Contents/Resources/app/apm/node_modules/npm/bin/npm-cli.js" "--globalconfig" "/Users/mup/.atom/.apm/.apmrc" "--userconfig" "/Users/mup/.atom/.apmrc" "rebuild" "--runtime=electron" "--target=1.7.11" "--arch=x64"
npm ERR! node v6.9.5
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR! fsevents@1.1.2 install: node install
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the fsevents@1.1.2 install script 'node install'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the fsevents package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node install
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs fsevents
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls fsevents
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /Users/mup/.atom/packages/remote-sync-pro/npm-debug.log

I know this doesn't help but it did manage to rebuild here.
Is this project still under development?

I've finally managed to get it working. Nothing in particular that I did, I just un-installed the package a few times and re-installed until it worked. Just pleased that it's working again.

repo: laymance/remote-sync-pro


text:
Brew unable to install and unable to make

I follow the instruction and tried to install through brew
Tuzis-MacBook:build tuzi$ brew install gumbo-paerser
Error: No available formula with the name "gumbo-paerser"
==> Searching for a previously deleted formula (in the last month)...
Warning: homebrew/core is shallow clone. To get complete history run:
git -C "$(brew --repo homebrew/core)" fetch --unshallow
Error: No previously deleted formula found.
==> Searching for similarly named formulae...
==> Searching local taps...
Error: No similarly named formulae found.
==> Searching taps...
==> Searching taps on GitHub...
Error: No formulae found in taps.
Thanks for your help

brew install gumbo-parser
Check your spelling?

source: gharchive/issue

authors: bakercp tuzimoe

repo: lazytiger/gumbo-query

license: mit

license_type: permissive

license_source: bigquery


text:
Implement basic fly camera

Resolves #18

@aayux, I know it's been a while, but do you want to review this? Just thought I'd throw it out there since you seemed interested in contributing. The main things I'm concerned about are the following:

Camera "flipping"

Movement feeling wonky, changing direction when crossing axes, etc.

As far as I can tell these seem to be fixed, but I want to see if others have any problems. Of course if you see anything else to poke at, by all means do so. Maybe my code looks ugly.
Naturally anyone else who sees this is welcome to test and review as well.

I can definitely take a look, happy to help!

Thanks for pointing that out - I'll definitely fix it. Don't exactly want someone's first impression to be "this looks broken."
Good to hear that the camera feels okay too. And I should have been more clear on what I meant by "flipping"; what I meant is being able to rotate the camera upward or downward more than 90 degrees so that the camera can essentially do a frontflip/backflip. At one point it would happen occasionally, and I just wanted to check that my fix truly worked. I've had a friend test it and it looks like it's working properly now (i.e. the camera won't flip), so don't worry if you didn't mess with that in particular.
Thank you for taking time to review this by the way! I really appreciate it. I'll make the change you suggested and then we should be ready for merging.

Ah, I see. No I didn't observe any flipping when I did that either.

Awesome. Sounds like we're good to go.
bors r=lberrymage,aayux p=5

bors r=lberrymage r=aayux p=5

bors r=lberrymage, aayux p=5

source: gharchive/pull-request

authors: aayux lberrymage

repo: lberrymage/accrescent

license: ISC

license_source: github-api


text:
Tailscale command failed to run on Azure

Hi @jaxxstorm!
Nice work on this module. :)
I'm trying to use it on Azure, and I'm getting this error when the scaleset is provisioned:
[   48.751263] cloud-init[1588]: Do you want to continue? [Y/n] Abort.
[   48.758664] cloud-init[1588]: sudo: tailscale: command not found

Thanks for the report, sorry I didn't see it, will take a look!

authors: FernandoMorais jaxxstorm

repo: lbrlabs/pulumi-tailscale-bastion


text:
Pinned comments are duplicated

Comments posted on Odysee posts are duplicated. The error is also displayed in the mobile browser version.

This is a known bug on the comment server side and will be fixed soon, sorry!

It's been fixed!

@tzarebczan missed opportunity not responding "It's been fixed" twice

authors: kauffj michaelmitnick tzarebczan

repo: lbryio/lbry-desktop

license: MIT


text:
Added few more files in .gitignore

PR Checklist

Please check all that apply to this PR using "x":

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[x] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[x] Documentation changes
[ ] Other - Please describe:

Fixes
Issue Number:
What is the current behavior?
for macox users, .gitignore dosen't had some files.
What is the new behavior?
Added some more files fo macox users.
Other information

@helios1101, thanks for the PR! Congrats on the first one here :)
Can we show you some appreciation for the contribution?
Also, we're giving away some Hacktoberfest bonuses and goodies for this month. We'll send you the information when you reach out to us.


text:
Update README.md

PR Checklist

Please check all that apply to this PR using "x":

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[ ] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[x] Documentation changes
[ ] Other - Please describe:

Fixes
Issue Number:
What is the current behavior?
What is the new behavior?
Other information

Added Text


text:
Remove report_bug call

The Issue
This call can now be removed, the app is no longer using it.
System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

Internal Use
Acceptance Criteria

Definition of Done

[ ]  Tested against acceptance criteria
[ ] Tested against the assumptions of user story
[ ] The project builds without errors
[ ] Unit tests are written and passing
[ ] Tests on devices/browsers listed in the issue have passed
[ ] QA performed & issues resolved
[ ] Refactoring completed
[ ] Any configuration or build changes documented
[ ] Documentation updated
[ ] Peer Code Review performed

done in #1984

authors: eukreign tzarebczan

repo: lbryio/lbry


text:
change stream_cost_estimate "name" param to "uri"

When making a request with the stream_cost_estimate method, an error is thrown if "name" is provided instead of "uri" as a parameter.

That doc is autogenerated by https://github.com/lbryio/lbry/blob/master/scripts/gen_api_docs.py but looks like the script never got run on release?

grumbles

updated

authors: kauffj kaykurokawa lyoshenka treethought


text:
add brief Regtest description and link to setup

As a new user coming from the infosec side of things, 'regtest' is usually an alias to fuzzing. While writing a binding to the API, I was under impression to use testnet with my code and I thought '-regtest' was mainly an internal feature for developers on lbrycrd. Could there be mention of ideal developer setup in the readme?

Thanks for opening this issue @binaryflesh. Definitely something we need to mention here/how it works/where to find setup instructions.

For review at https://github.com/lbryio/lbrycrd/tree/depends_build_all

merged.

authors: BrannonKing binaryflesh tzarebczan

repo: lbryio/lbrycrd


text:
task 10 failed - "invalid 'times' argument"

hi here is my workflow
fungi <- read.table("Normal.txt", sep="\t", row.names=1, header=T)
res_2 <- func.EM(m, ncpu=4)
and following is the error
############# Run for iteration 1: #############
E-step: estimating scaled parameters...
Error in { : task 10 failed - "invalid 'times' argument"
attached i s the input file
please help me solve this
Normal.txt

Hi
I am not sure what m is. But in your input data, there are rows with only one non-zero entry, which is insufficient for the inference.
You can remove rows (taxa) with low abundance in many samples, and I have implemented an assertion to check this in the latest commit.
In addition, I think 36 samples might not be enough for accurate BEEM-static analysis.
Chenhao

Feel free to reopen if it doesn't solve the problem.


text:
GitHub Permissions

Hello! This looks great. I am hesitant to grant full repository access, however. I don't understand what gist management needs from my repositories.
Thank you

Hi, we use repository access for the Report a Bug feature. After a few tests, it just needs to grant public repo access. I will fix it and deploy it for the next version. Thank you a lot for giving me know.

PR has raised at #56

Wow, quick turn around on this. Thank you very much!

authors: JadeDaniel ldakhoa

repo: ldakhoa/GistHub


text:
🛑 Thonk Website is down

In 3e25cd0, Thonk Website (https://www.thonk.xyz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Thonk Website is back up in 362f3c0 after 7 minutes.

repo: le-server/thonk-upptime


text:
🛑 Thonk Cloud is down

In 8821dc6, Thonk Cloud (https://www.thonk.cloud) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Thonk Cloud is back up in ca9a5ff.


text:
Can have every language a short introduction?

At least show the homepage. I saw too many language never heard.

Typically there's a link to the more esoteric languages in the commit message, I can't think if an elegant way to do this besides revising every single language in the project with comments inside the code linking to the webpage.

@leachim6 maybe show the homepage address in the comment. So every program language should at least support comment? Ｏ（∩——∩）Ｏ～

@leachim6 I think the language should at least support comment to be showed in this project.YY

We can potentially merge this issue with https://github.com/leachim6/hello-world/issues/356
What do you think?

I like the idea of linking to either Esolang or Wikipedia for each language on the readme. Esolang's entries are often quite nice. For example, check out https://esolangs.org/wiki/Brainfuck and https://esolangs.org/wiki/Labyrinth.

How about show the short introduction of every language in "wiki" page ?

Can we add links it to Wikipedia or using APIs add small description with little background.

We can manually operate the language list in the readme, adding descriptions after listing each one.

@Richienb should I start to work on this?

Go ahead.

@Richienb how should I implement this with just a text description below the language (with some HTML styling maybe) or a comment in the language file (if it supports comments).

@Richienb or in the wiki?

We could put it in the wiki.

Or we could just put it in the readme beside the language name. We'd need to update the script though to preserve the descriptions.

like this

in short answer yes ...it could have but if you go in more in details you come to know a short introduction is not enough..

authors: Kundanagrawalofficial MrBrain295 NYfor2017 Richienb Tomticaa gfairchild leachim6 monomaniar mravariya


text:
feat: port Data.Fintype.List

Thanks :tada:
bors merge


text:
fix: correct mathlib3-style lattice lemma names

Co-authored-by: Mauricio Collares mauricio@collares.org

bors d+

bors merge

authors: eric-wieser jcommelin


text:
feat(Data/Polynomial/Module) : define an R[X] module, given a linear map.

Given an element a in an R-algebra A and an A-module M,
define an R[X]-module Module.AEval R M a, whose elements correspond to
elements of M, and where the action of f : R[X] is f • m = aeval a f • m.
Equivalently, X • m = a • m.
This module is abbreviated to Module.AEval' φ in the special case that A is the algebra of R-linear maps and φ : M → M is an R-linear map.
This is needed in #7419.

This sounds like it has the same goal as https://leanprover-community.github.io/mathlib4_docs/Mathlib/RingTheory/FiniteType.html#modulePolynomialOfEndo, this PR should redefine that material in terms of this I think. (I think using a type synonym is probably a nicer way to go, and it looks like you added more api here)

This sounds like it has the same goal as https://leanprover-community.github.io/mathlib4_docs/Mathlib/RingTheory/FiniteType.html#modulePolynomialOfEndo, this PR should redefine that material in terms of this I think. (I think using a type synonym is probably a nicer way to go, and it looks like you added more api here)

Why is this in Mathlib.RingTheory.FiniteType?! Nice found BTW.

I think because its only used to prove the result below! But it really shouldn't be indeed

This sounds like it has the same goal as https://leanprover-community.github.io/mathlib4_docs/Mathlib/RingTheory/FiniteType.html#modulePolynomialOfEndo, this PR should redefine that material in terms of this I think. (I think using a type synonym is probably a nicer way to go, and it looks like you added more api here)

Thanks very much for that; I didn't think it was already in Mathlib. I'll try doing what you suggest.

I think it should still go in Polynomial/Module if that works though, just add an import to that from the finitetype file, as Riccardo says FiniteType is a non-obvious place for this construction to live

Thanks, I've switched that around now. I also needed to swap the imports, so that now FiniteType imports Polynomial/Module, whereas previously it was the other way around. I've moved my code to the start of Polynomial/Module, because it is now used later down that file. I have left Vasconcelos' theorem in FiniteType, but its proof now uses AEval' instead of ModulePolynomialOfEndo. This avoids the creation of instances at the start of the proof, and also a have statement, which is a lemma for AEval'.
There is one thing which I wonder about. On line 729 of FiniteType, I need to quote the incidence AEval'.Finite_of_Finite. I had hoped that lean would find that incidence for itself, and I'm a little concerned that it can't.

Ok looks good to me now, I left two last small comments, but after those I think this should be good to go

maintainer merge

Sorry for coming in so late with this observation, but there's an instance diamond I'm worried about:
/-- In the special case where `A = M = R[X]` (eg multivariate polynomials), we have two conflicting
actions which agree only in the special case of of `a = X`. -/
example (a : R[X]) (h : a ≠ X):
    (instModuleEval a : Module R[X] (AEval R R[X] a)) ≠
    (instModulePolynomial a  : Module R[X] (AEval R R[X] a)) := by
  intro H
  apply h
  replace H := congr_fun₂ (congr_arg (@HSMul.hSMul _ _ _ <| @instHSMul _ _ ·.toSMul) H)
  dsimp [instModuleEval, instModulePolynomial, smul_def] at H
  replace H := H X (1 : R[X])
  simpa using H.symm

I think either we:

Put a warning in the docstring that it is not safe to use AEval in this way (and commit the example that demonstrates it.
Remove the Module A (AEval R M a) instance, especially if you weren't actually using it

Many thanks for pointing this out. I have use this instance quite a lot (for example, in order to state smul_def, which is used in several places). I can see that this would cause problems, so I'll have a think about it.

Many thanks for pointing this out. I have use this instance quite a lot (for example, in order to state smul_def, which is used in several places). I can see that this would cause problems, so I'll have a think about it.

One resolution here would be to state smul_def as f • m = of (aeval a f • of.symm m).

Many thanks for all you help!

bors r+

authors: alexjbest eric-wieser riccardobrasca rmhi


text:
feat: add topological lemmas for ULift

This also adds a handful of easy instances
Arguably the topological space instance should be defined via coinduced to make these true by Iff.rfl, but that creates a headache with constructors for normed spaces, so is not in this PR.

Thanks! :tada:
bors merge

authors: eric-wieser urkud


text:
prime factorization

The main thing here is a definition of mult p n, the multiplicity of p in n, i.e. the largest power of p that divides n (assuming n > 0 and p > 1). Here is one of the main theorems:
theorem dvd_of_forall_prime_mult_le {m n : ℕ} (mpos : m > 0)
    (H : ∀ {p}, prime p → mult p m ≤ mult p n) : m ∣ n

I still need to prove
n > 0 → n = ∏ p : prime_factors n, p^(mult p n)

but I don't think it will be hard with the facts already there. We may need more properties of sums and products. I also started developing some properties of powers in algebra, but there is still a lot to do.
@htzh I renamed a few theorems in primes.lean in this commit. I am sorry if this breaks some of your proofs, but it should be easy to fix.
@leodemoura There might be a bug here:
https://github.com/avigad/lean/blob/master/library/theories/number_theory/prime_factorization.lean#L106
It is strange, the proof breaks when I put brackets around that line, but begin ... end works o.k. It might be hard to produce a small counterexample; I did this sort of thing many times, without problems. But let me know if you want me to try.
There are a lot of proofs by induction and cases in prime_factorization, like this one:
https://github.com/avigad/lean/blob/master/library/theories/number_theory/prime_factorization.lean#L68-L89
If we eventually have good syntax for induction and cases in proof terms, this would be a good place to try it out.

It is strange, the proof breaks when I put brackets around that line, but begin ... end works o.k. It might be hard to produce a small counterexample; I did this sort of thing many times, without problems. But let me know if you want me to try.

I think you have defined tokens such as ]}.
So, the string
{rewrite [pow_zero, one_mul, zero_add]}

is tokenized as
'{'   'rewrite'   '['   'pow_zero'   ','    'one_mul'   ','  'zero_add'  ']}'

Doh! Burned by my own notation yet again. It is from the notation {[a]} for a singleton in finset (and also in set).
I will change it. If it caused problems for me, it will cause problems for other users. Maybe I will try '{a}, unless you foresee problems with that too. I defined notation f '[s] for the image of s under f, and that hasn't caused problems, at least not yet.

'{a} is fine. Usually, a token is problematic when it is the concatenation of two existing tokens.

This seems to be a recurring problem. Would it help if the notation for finset was declared as
notation `{` `[` a `]` `}` := ...

Or does that have other problems?

Or does that have other problems?

Yes, it conflicts with notation such as
notation `{` binders `|` r:(scoped:1 P, subtype P) `}` := r

We cannot have both of them simultaneously. The problem is that after consuming {, the first declaration is telling the parser to consume [, and the one above it is saying to consume a binder. Moreover, a binder may start with [.

authors: avigad fpvandoorn leodemoura

repo: leanprover/lean


text:
feat(analysis/nnreal): define the nonnegative reals

NB: This file has a lot in common with ennreal.lean, the extended nonnegative reals.

It looks like you forgot to add induced_comp

I processed almost all your comments. Only the stuff on orders I didn't touch. I think I might need a bit of help there...

Merged in https://github.com/leanprover/mathlib/commit/9303bc024fccb60cfcdb4f8f9ba6459b02deec3b
I added more type class instances to nnreal.

authors: jcommelin johoelzl

repo: leanprover/mathlib


text:
Eliminate use of std::to_string

This method isn't supported in Android.  Remove our use of it until a shim is created, or support is provided.

Discussed w/ Jason to understand why we can't easily use template <typename T>   std::string to_string inside autowiring, as it would conflict with platform's use of the same.

authors: codemercenary jdonald

repo: leapmotion/autowiring


text:
feat: redesign send rpc transfer flow

Try out Leather build f56dbcd — Extension build, Test report, Storybook, Chromatic

redesign rpc send transfer using approver ux

Looking good! Some QA feedback:

App favicon isn't showing up in the header
"Total spend" doesn't include the fee amount
Guide URL takes user to https://leather.io/guides/connect-dapps instead of a relevant guide for sending BTC via an app
Do we want to say "Send bitcoin" instead of "Send BTC"? cc @fabric-8 @mica000
I'd suggest some additional design QA (by a designer) to double check text styling / sizes, padding, etc.

Looking great, only some minor feedback:

animal icon stroke width are too thin, should generally be same as for the other icons. Maybe they are getting scaled because there's a mismatch of height/width it needs (32x32) vs. actual size after some... padding is applied? If I can help by adjusting the asset lmk, the animation icons are 32x32 vs. 24x24, if turning them into 24x24 fixes this issue I can do it

Since were are using the > to indicate that the user can interact with the item (like for account on top), I think we should also add the > to fee settings for consistency cc @mica000

Realising that "cancel" in fee settings might get misinterpreted to "cancel the whole approval thing" - Maybe using the word "back" would be better here? cc @markmhendrickson

Let's add border-top: 1px solid ink.border-default to the footer

This is looking great @alter-eggo! Really nice to see it coming together! Bravo 👏

Are we covering advanced details as well on this PR? And is this only for BTC or for STX as well?
If so, here are the details we would need to trigger for Asset transfer BTC and AssetTransferSTX

@fabric-8 @markmhendrickson Curious about what you think of the prompt, as it shouldn't show the icon of the application since we are already showing this on the request by. However, it is quite nice to have some reassuring of the app you are connecting to:

Here is how it was intended to work and the respective designs

@markmhendrickson @fabric-8 We might not have explicitly specified this in the approval documentation, but when connecting to a BTC account, the total balance should be calculated solely for BTC. Including other assets in the calculation could confuse users, especially if they meet the required balance but not in BTC. What do you think?

Account selection: When selecting an account does it make sense to show create account cta? Maybe better to hide it?

Looking great, only some minor feedback:

animal icon stroke width are too thin, should generally be same as for the other icons. Maybe they are getting scaled because there's a mismatch of height/width it needs (32x32) vs. actual size after some... padding is applied? If I can help by adjusting the asset lmk, the animation icons are 32x32 vs. 24x24, if turning them into 24x24 fixes this issue I can do it

Since were are using the > to indicate that the user can interact with the item (like for account on top), I think we should also add the > to fee settings for consistency cc @mica000

Realising that "cancel" in fee settings might get misinterpreted to "cancel the whole approval thing" - Maybe using the word "back" would be better here? cc @markmhendrickson
Let's add border-top: 1px solid ink.border-default to the footer

Yes for adding > to fee settings for consistency

Realising that "cancel" in fee settings might get misinterpreted to "cancel the whole approval thing" - Maybe using the word "back" would be better here? cc @markmhendrickson

Agreed

Curious about what you think of the prompt, as it shouldn't show the icon of the application since we are already showing this on the request by. However, it is quite nice to have some reassuring of the app you are connecting to:

I do like the idea of including the app icon, though it takes up a lot of space in the current prompt design. It seems perhaps best to just rely on this placement so we don't push the other content below the fold?

We might not have explicitly specified this in the approval documentation, but when connecting to a BTC account, the total balance should be calculated solely for BTC. Including other assets in the calculation could confuse users, especially if they meet the required balance but not in BTC. Thoughts? And if so, should this be in this PR or a later improvment?

It seems best that we either 1) show just a fiat value (for the total account value including all assets) or 2) show the fiat value for the specific asset in question (e.g. BTC value in USD here) and that asset's literal value (e.g. BTC balance itself here) right below it, as in your designs. By placing these two values together vertically, it'll presumably be clear to the user just which subset of value they're looking at it in this context.
We could add this enhancement (2) later or tackle as part of this PR – both work for me.

Thanks @alter-eggo!
Icons for fees remains 1 px stroke?

@alter-eggo About the account picker cta, no worries!
In relation to the component on the approval however, lets do the second option suggested by @markmhendrickson in this comment?

show the fiat value for the specific asset in question (e.g. BTC value in USD here) and that asset's literal value (e.g. BTC balance itself here) right below it, as in your designs. By placing these two values together vertically, it'll presumably be clear to the user just which subset of value they're looking at it in this context.

In add, @markmhendrickson I believe we can hide one of the address, depending on the network being used.
so it would look more like this:

@alter-eggo thanks! Looking good!

@alter-eggo it appears this change is still needed given my latest testing:

Guide URL takes user to https://leather.io/guides/connect-dapps instead of a relevant guide for sending BTC via an app

repo: leather-io/extension


text:
fix: sip10 token send form fees bug

Try out Leather build 40371b5 — Extension build, Test report, Storybook, Chromatic

This pr fixes sip10 send form fees bug, discussed:
https://trustmachines.slack.com/archives/C0520G8J33M/p1716339282877119
https://trustmachines.slack.com/archives/C05LAP952E7/p1716363736626209

Summary by CodeRabbit

New Features

Introduced a new function to retrieve asset contract addresses from contract IDs or fully qualified asset names.

Improvements

Updated navigation logic for sending SIP10 tokens to enhance performance and maintainability.
Standardized address retrieval in asset string parts for consistency.

Refactor

Replaced Navigate component with useNavigate hook for more efficient navigation handling.

With this latest build

I can select a token. It doesn't seem to load fees but defaults to fixed low fee for low, medium and high.

When I add an amount that is not available I get no warning about insufficient funds.
Then I can continue to review
When clicking confirm and send transaction I see

Let's ship? 🚢

Retested this version fix/sip10-send-form#40371b5, doesn't work.

When I confirm and send transaction it fails with this error:

There doesn't seem to be a check for the amount of token

fee setting by the wallet does look correct now.

repo: leather-wallet/extension


text:
考虑过支持视频吗？

如果支持，有时间计划吗

@chenqian-dev 有的，目前逻辑已经写得差不多了，这两天就要发布一个预览版

谢谢，很期待

@chenqian-dev 目前 1.0.0-beta01 版本已经支持选择视频了

authors: chenqian-dev leavesCZY

repo: leavesCZY/Matisse


text:
Fix typos

Found via typos --hidden --format brief

Thank you

:partying_face: :partying_face: :partying_face: :partying_face: :partying_face:

authors: kianmeng lebrunel

repo: lebrunel/ollama-ex


text:
How to get an element to lose focus?

Is there a nice way in canopy to trigger blur on an element?
I tried going directly via js:
"#id" << "some new value"
"document.getElementById('#id').blur()" |> js |> ignore

but it doesn't lose focus.

Try click "html"
On Mar 17, 2017 4:13 AM, "Nicholas Lydon" notifications@github.com wrote:

Is there a nice way in canopy to trigger blur on an element?
I tried going directly via js:
"#id" << "some new value"
"document.getElementById('#id').blur()" |> js |> ignore
but it doesn't lose focus.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lefthandedgoat/canopy/issues/354, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAsQ6W6Y44cFryS_ws-RgFVsAY7wYRpiks5rmk6ggaJpZM4MgWAa
.

Try click "html"

Closed (no activity).

authors: NickLydon amirrajan lefthandedgoat

repo: lefthandedgoat/canopy


text:
Cannot add finder

I tried adding a finder but i'm getting a type error.
I assumed it was something wrong with my code, but when just pasting the example from the docs i get the same error:
The type 'IWebDriver -> IWebElement list' does not match the type 'IWebElement list'
module Finders

open canopy.classic
open OpenQA.Selenium

let findByHref href f =
    try
        let cssSelector = sprintf "a[href*='%s']" href
        f(By.CssSelector(cssSelector)) |> List.ofSeq
    with | ex -> []

addFinder findByHref

For information, the selector i want to add is a lookup by data-attributes like this:
click "data-tab=properties
should select the element
<div class="tab" data-tab="properties"
Here is my selector as i wrote it:
module Finders

open canopy.classic
open System.Text.RegularExpressions
open OpenQA.Selenium

exception FinderDoesNotApply of unit

addFinder ( fun dataAttribute f ->
    try
        if Regex.Match(dataAttribute, "^data-\w{1,}=\w{1,}").Success then
            let selectorParts = dataAttribute.Split [|'='|]
            let selector = "*[" + selectorParts.[0] + "=\"" + selectorParts.[1] + "\"]"
            f(By.CssSelector(selector)) |> List.ofSeq
        else
            raise (FinderDoesNotApply())
    with | ex -> []
)

I figured it out. The documentation is outdated. Adding a third (unused) parameter to the function solves it:
module Finders
open canopy.classic
open System.Text.RegularExpressions
open OpenQA.Selenium

exception FinderDoesNotApply of unit

let dataFinder dataAttribute f _ =
    try
        if Regex.Match(dataAttribute, "^data-\w{1,}=\w{1,}").Success then
            let selectorParts = dataAttribute.Split [|'='|]
            let selector = "*[" + selectorParts.[0] + "=\"" + selectorParts.[1] + "\"]"
            f(By.CssSelector(selector)) |> List.ofSeq
        else
            raise (FinderDoesNotApply())
    with | ex -> []

Yes, I didnt realize the documentation was out of date.  Here is an example:
https://github.com/lefthandedgoat/canopy/blob/273fe1c5e93111d5114e2ef81e1d02e9e3eaf49b/tests/basictests/Program.fs#L739-L747

Thank you for the prompt reply even though i solved the issue.
Loving canpoy :)

authors: Pwntheon lefthandedgoat


text:
Missing assets/fonts

Should these be retrieved from the aether somehow?
Configuration file: /home/shokin/jekyll-test/_config.yml
            Source: /home/shokin/jekyll-test
       Destination: /home/shokin/jekyll-test/_site
 Incremental build: disabled. Enable with --incremental
      Generating...
                    done in 0.037 seconds.
 Auto-regeneration: enabled for '/home/shokin/jekyll-test'
    Server address: http://127.0.0.1:4000
  Server running... press ctrl-c to stop.
[2021-02-25 06:32:51] ERROR `/assets/fonts/ProximaNova-Light-webfont.woff2' not found.
[2021-02-25 06:32:51] ERROR `/assets/fonts/ProximaNova-Reg-webfont.woff2' not found.
[2021-02-25 06:32:51] ERROR `/assets/fonts/roboto-mono-regular.woff2' not found.
[2021-02-25 06:32:51] ERROR `/assets/fonts/ProximaNova-Light-webfont.woff' not found.
[2021-02-25 06:32:51] ERROR `/assets/fonts/roboto-mono-regular.woff' not found.
[2021-02-25 06:32:51] ERROR `/assets/fonts/ProximaNova-Reg-webfont.woff' not found.
[2021-02-25 06:32:51] ERROR `/assets/icons/favicon-192x192.png' not found.
[2021-02-25 06:32:51] ERROR `/assets/icons/favicon-16x16.png' not found.

(I found them on GitHub, but wondering about how they should have been grabbed.)

Since the Jekyll theme was rapidly prototyped its styling was shamelessly copied from the UIkit website source code. Feel free to A/B with the missing fonts but I've been pretty satisfied with how it looks without them. At some point I'll do away with the copied styling in preference for a more vanilla UIkit layout that approximates it. At that point the missing fonts will go out the window unless someone thinks we aught to use them.

No shame in copying, it's what humans do. I'll track 'em down and see what I think. I'd never heard of ProximaNova before, but I do enjoy checking out new fonts.

OK, yeah, the Proxima Novo fonts may be used, supposedly, with URLs but not housed on the server - but then when you go to grab them they make you create a Creative Cloud account, and to do that, you have to pay money. So it's bait and switch. The roboto fonts (Google) are Apache 2.0 so I've left them in place.

authors: alancleary sammyjava

repo: legumeinfo/legumeinfo-jekyll-theme


text:
Release v1.3.0

[x] Create a branch for the release. Branch release-v1.3.0
[ ] Update the dependencies. The dependabot PR backlog is not comprehensive and doesn't include transitive dependencies, and merging them one at a time tends to break things, so the best way to do this is manually with npm.
[ ] Do a test build of the NPM package on your local machine so you know if something is going off the rails before the automated build action on GitHub.
[ ] Bump the "version" number in package.json and reinstall the dependencies so the correct version number is in package-lock.json.
[ ] Open a PR that merges the release branch into main.
[ ] Merge the release PR after it passes testing and review.
[ ] Tag a corresponding release on GitHub. This will trigger an automated NPM package build on GitHub. If the package build succeeds it will be automatically published to NPM.

@alancleary are you going to use this as a golden opportunity to train someone(s) else in the delicate art of a web components release? I hope so...

Closing this issue since opening an issue is part of the release process.

sure, let's discuss in today's standup.

authors: adf-ncgr alancleary

repo: legumeinfo/web-components


text:
Boost Software License?

Apache isn't quite as permissive as the Boost Software License, would you consider re-licensing or dual-licensing this code under the Boost Software License? https://www.boost.org/users/license.html

That should be no problem. Can you issue a PR?

authors: lemire vinniefalco


text:
[WIP] Multithreaded version of simdjson for parsing regular but large JSON documents

This is a work-in-progress version of simdjson which can parse large documents in a multithreaded fashion. There is an interleave of stage 1 and stage 2. The expectation is that we should be able to boost the parsing speed of large files by ~50% while using 2 cores.
This would solve https://github.com/lemire/simdjson/issues/171
I brought it here from a fork @piotte13 did. It is not clean and not approved by Jérémie, but I wanted to start discussion around it, early.
The stage 2 has been reengineered to be more flexible. Basically, you can run stage 2 over pieces of a JSON document, not just entire JSON documents. This implies that we must be able to persist the current state, and resume where we were (e.g., in the middle of an object).
Doing so has other engineering benefits. For example, it should allow us to avoid having to pad the inputs (solving https://github.com/lemire/simdjson/issues/174). We could also allow the user to just parse the documents in small pieces.
Besides the fact that this PR is not clean, there are still unresolved performance issues. The expectation is that the more flexible stage 2 should run at the same speed. We worked hard to keep the fancy computed gotos. Sadly, the single-threaded can be far worse (-25%). For large files, this is fine because the second thread more than compensate, but for small files, we can't use the new approach without a substantial penalty.
The new stage 2 should be identical in computation cost, except for one additional comparison and conditional jump per structural element. We can expect that this comparison and condition jump should be highly predictable and, effectively free. So something else is hurting us. Unfortunately, the new code is quite different from the old code in structure, so a straight-forward comparison is not trivial.
I expect that @piotte13 might come around, clean and report. When he does, this PR will become obsolete.
Cheers!

Daniel

I am going to kill this PR, but keep the branch.

repo: lemire/simdjson


text:
[4pt]  Fetch missions - Fetch data

Fetch data from the Missions endpoint  (https://api.spacexdata.com/v3/missions) when a user navigates to the Missions section.
Once the data are fetched, dispatch an action to store the selected data in Redux store:

mission_id
mission_name
description

NOTE: Make sure you only dispatch those actions once and do not add data to store on every re-render (i.e. when changing views / using navigation).

Duplicate issue

repo: lengushuru/Space-Travelers-Hub


text:
How to run?

hello, I have download.... but was unable to run, you can help me

See examples here.


text:
Allow extensions to register additional details for kubernetes objects

Currently extensions only allow to register whole details content area (for custom resources). Sometimes it would be nice to display additional information directly in kubernetes object details (information would be in right context).

Already possible, for example:
  kubeObjectDetailItems = [
    {
      kind: "Pod",
      apiVersions: ["v1"],
      components: {
        Details: (props: any) => <AdditionalPodDetails pod={props.object} />
      }
    }
  ]

repo: lensapp/lens


text:
Can't open Lens on mac

Describe the bug
Can't open Lens on Mac. MacOS 12.6 (21G115)
When I try to open it I see

Here I cant approve it too

To Reproduce
Steps to reproduce the behavior:

Download package for intel chip from Lens website
Install it
Try to open
See error

Expected behavior
Lens opens properly
Environment (please complete the following information):

Lens Version: 2022.9.260655-latest
OS: MacOS 12.6 (21G115)
Installation method (e.g. snap or AppImage in Linux): via dmg installer

Additional context
What solved my problem: xattr -r -d com.apple.quarantine /Applications/Lens.app

I have the same issue ✋🏻

I have the same issue ✋🏻
try this as temporary solution:
xattr -r -d com.apple.quarantine /Applications/Lens.app

We are looking into this with high priority.
@DmitriyEsin @floverdevel Is this a first time or clean install? Can you try to start from a terminal and copy logs from the console?
DEBUG=true /Applications/Lens.App/Contents/MacOS/Lens

same thing.
bash-3.2$ DEBUG=true /Applications/Lens.App/Contents/MacOS/Lens
Killed: 9

This just happened to me but I did succeed after authorizing Lens from Security and Privacy settings. It happened after upgrading Lens using brew cask

@jim-docker Yep it is clean install on new Mac
logs the same as @gsiawGH
dmitriy$ DEBUG=true /Applications/Lens.App/Contents/MacOS/Lens Killed: 9

I believe this is now fixed (just download the latest package from https://k8slens.dev)

authors: DmitriyEsin floverdevel gsiawGH inglkruiz jakolehm jim-docker


text:
Unable to login to the cluster

I am trying to connect to the EKS cluster and it gives an error Unknown Error Occurred.
Even though I am able to connect to the cluster using kubectl command.
Screenshots

Environment (please complete the following information):

Lens Version: 2022.9.280635-latest
OS: OSX
Installation method (e.g. snap or AppImage in Linux): Downloading .dmg file

Kubeconfig:
Quite often the problems are caused by malformed kubeconfig which the application tries to load. Please share your kubeconfig, remember to remove any secret and sensitive information.
apiVersion: v1
clusters:
- cluster:
    certificate-authority-data: LS0tLS1CRUdJTiB*****************mlZZ2NkNUZab3dRVklEcEhjS04KQjI4PQotLS0tLUVORCBDRVJUSUZJQ0FURS0tLS0tCg==
    server: https://DRT128738723897GFHHGHA.gr7.eu-central-1.eks.amazonaws.com
  name: arn:aws:eks:eu-central-1:9238743893982107:cluster/cluster-name
contexts:
- context:
    cluster: arn:aws:eks:eu-central-1:9238743893982107:cluster/cluster-name
    user: arn:aws:eks:eu-central-1:9238743893982107:cluster/cluster-name
  name: arn:aws:eks:eu-central-1:9238743893982107:cluster/cluster-name
current-context: arn:aws:eks:eu-central-1:9238743893982107:cluster/cluster-name
kind: Config
preferences: {}
users:
- name: arn:aws:eks:eu-central-1:9238743893982107:cluster/cluster-name
  user:
    exec:
      apiVersion: client.authentication.k8s.io/v1beta1
      args:
      - --region
      - eu-central-1
      - eks
      - get-token
      - --cluster-name
      - cluster-name
      command: aws

I'm facing the same issue, what am I supposed to do with that error?
Can lens be more verbose?

If you run lens from the command line, you should see some error log message starting with [CLUSTER]: Failed to connect to. Does that help?

How can I download lens command line for Mac ?
I don't find any option.
And, when I tried with the command lens I got the error: zsh: command not found: lens

Assuming that you installed it normally, you can run /Applications/Lens.app/Contents/MacOs/Lens

On my case If I run awsume <profile> && /Applications/Lens.app/Contents/MacOs/Lens it gets me access to the cluster, however if I run lens via the normal icon it does not.
Do you see any reason for this?

All depends on how you have your kubeconfig file set up.


text:
Update scalafmt-core to 3.8.2

About this PR
📦 Updates org.scalameta:scalafmt-core from 3.6.1 to 3.8.2
📜 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 = "org.scalameta", artifactId = "scalafmt-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalameta", artifactId = "scalafmt-core" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:n:3

Superseded by #299.

repo: leobenkel/Soteria


text:
Plugin not working on 1.21.4

Plugin will throw an error when trying to open a book on version 1.21.4:
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF] java.lang.ExceptionInInitializerError
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at InteractiveBooks.jar//net.leonardo_dgs.interactivebooks.lib.nbtapi.NBTCompound.setString(NBTCompound.java:176)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at InteractiveBooks.jar//net.leonardo_dgs.interactivebooks.IBook.lambda$getItem$0(IBook.java:196)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at InteractiveBooks.jar//net.leonardo_dgs.interactivebooks.lib.nbtapi.NBT.modify(NBT.java:252)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at InteractiveBooks.jar//net.leonardo_dgs.interactivebooks.IBook.getItem(IBook.java:195)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at InteractiveBooks.jar//net.leonardo_dgs.interactivebooks.IBook.open(IBook.java:165)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at InteractiveBooks.jar//net.leonardo_dgs.interactivebooks.CommandIBooks.onOpen(CommandIBooks.java:85)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(Unknown Source)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at java.base/java.lang.reflect.Method.invoke(Unknown Source)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at InteractiveBooks.jar//net.leonardo_dgs.interactivebooks.lib.acf.RegisteredCommand.invoke(RegisteredCommand.java:152)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at InteractiveBooks.jar//net.leonardo_dgs.interactivebooks.lib.acf.BaseCommand.executeCommand(BaseCommand.java:578)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at InteractiveBooks.jar//net.leonardo_dgs.interactivebooks.lib.acf.BaseCommand.execute(BaseCommand.java:513)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at InteractiveBooks.jar//net.leonardo_dgs.interactivebooks.lib.acf.RootCommand.execute(RootCommand.java:99)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at InteractiveBooks.jar//net.leonardo_dgs.interactivebooks.lib.acf.BukkitRootCommand.execute(BukkitRootCommand.java:84)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at io.papermc.paper.command.brigadier.bukkit.BukkitCommandNode$BukkitBrigCommand.run(BukkitCommandNode.java:82)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at com.mojang.brigadier.context.ContextChain.runExecutable(ContextChain.java:73)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.commands.execution.tasks.ExecuteCommand.execute(ExecuteCommand.java:31)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.commands.execution.tasks.ExecuteCommand.execute(ExecuteCommand.java:19)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.commands.execution.UnboundEntryAction.lambda$bind$0(UnboundEntryAction.java:8)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.commands.execution.CommandQueueEntry.execute(CommandQueueEntry.java:5)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.commands.execution.ExecutionContext.runCommandQueue(ExecutionContext.java:103)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.commands.Commands.executeCommandInContext(Commands.java:448)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.commands.Commands.performCommand(Commands.java:355)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.commands.Commands.performCommand(Commands.java:342)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.commands.Commands.performCommand(Commands.java:337)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.server.network.ServerGamePacketListenerImpl.performUnsignedChatCommand(ServerGamePacketListenerImpl.java:2301)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.server.network.ServerGamePacketListenerImpl.lambda$handleChatCommand$11(ServerGamePacketListenerImpl.java:2275)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.server.TickTask.run(TickTask.java:18)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.util.thread.BlockableEventLoop.doRunTask(BlockableEventLoop.java:154)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.util.thread.ReentrantBlockableEventLoop.doRunTask(ReentrantBlockableEventLoop.java:24)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.server.MinecraftServer.doRunTask(MinecraftServer.java:1579)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.server.MinecraftServer.doRunTask(MinecraftServer.java:207)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.util.thread.BlockableEventLoop.pollTask(BlockableEventLoop.java:128)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.server.MinecraftServer.pollTaskInternal(MinecraftServer.java:1556)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.server.MinecraftServer.pollTask(MinecraftServer.java:1549)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.util.thread.BlockableEventLoop.managedBlock(BlockableEventLoop.java:138)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.server.MinecraftServer.managedBlock(MinecraftServer.java:1500)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.server.MinecraftServer.waitUntilNextTick(MinecraftServer.java:1510)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1346)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:341)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at java.base/java.lang.Thread.run(Unknown Source)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF] Caused by: java.lang.NullPointerException: Cannot invoke "Object.hashCode()" because "<parameter1>" is null
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at java.base/java.util.ImmutableCollections$MapN.probe(Unknown Source)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at java.base/java.util.ImmutableCollections$MapN.get(Unknown Source)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at java.base/java.util.ImmutableCollections$AbstractImmutableMap.getOrDefault(Unknown Source)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at io.papermc.paper.pluginremap.reflect.PaperReflection.mapDeclaredFieldName(PaperReflection.java:79)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at io.papermc.reflectionrewriter.runtime.AbstractDefaultRulesReflectionProxy.getDeclaredField(AbstractDefaultRulesReflectionProxy.java:90)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at io.papermc.paper.pluginremap.reflect.PaperReflectionHolder.getDeclaredField(Unknown Source)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      at InteractiveBooks.jar//net.leonardo_dgs.interactivebooks.lib.nbtapi.NBTReflectionUtil.<clinit>(NBTReflectionUtil.java:59)
[20:16:51] [Server thread/ERROR]: [InteractiveBooks] [ACF]      ... 40 more

I suppose it needs to be updated :)

9b408962de888696cc41acba2ea718e5c4fe5e44

Thanks a whole lot!! :)

repo: leonardo-dgs/InteractiveBooks


text:
Problem with language changing with a lot of components

Hello,
I am currently on a website with lots of components and I added a switch language button on my Nav Bar. The architecture is like this :
App.vue
<vue-router></vue-router>

The vue router calls "/" which is Home.vue, it possesses the NavBar
<template>
  <nav-bar />
</template>

And the navBar contains the FlagButton (switch language button)
<template>
  <flag-button/>
</template>

The flag button is in charge to change the language and I added the click like this :
@click="$ml.change(nextLang)"

It works, I can see it changes the language, but it stops at the nav-bar : Impossible to update the text on the Home.vue component.
Moreover, it doesn't changes the localStorage variable : when I refresh the page it comes back to the initial language.
I thought I could resolve this issue by using :
"$emit(nextLang)"

but it doesn't seems to work and isn't a good way of programming.
From what I understood by reading the module, $ml is not global but uses global variables. So I don't really understand why I have this issue. They should all be listening to the "_vueEBus2" that is a global variable but it doesn't seems like this is the case.
So  it would be nice if you had some solution or modifications of your module that could solve this problem,
Thank you in advance for your reply

Fix in 3.x version or later

authors: Aohler leonardovilarinho

repo: leonardovilarinho/vue-multilanguage


text:
扫rdb的时候 可以方便的获得KEY  占用了多少空间吗

扫rdb的时候 可以方便的获得KEY  占用了多少空间吗,
想汇总分析一下 key 占用空间情况

@noasoso
这个需要在replicator外部做，可以参考redis-rdb-tools或者redis-rdb-cli的计算方式把key占用的空间计算出来


text:
Jerry version

update

yep


text:
[fix] updated nix flakes lock files on session auth axum examples

update nix flakes lock files to fix once_cell doesn't compile.

Looks good, it's a simple nix flake update to pull in the latest rust nightly. Thanks!

Thanks!

authors: benwis gbj mustafasegf

repo: leptos-rs/leptos


text:
fetcher autonomously panics

I am using github.com/lestrrat-go/jwx/v2 v2.1.0 and github.com/lestrrat-go/httprc v1.0.5, and the fetcher would randomly panics with the following trace:
panic: send on closed channel

goroutine 194 [running]:
github.com/lestrrat-go/httprc.(*fetchResult).reply(0xc000ba8b70, {0x2104a00, 0x3007980}, 0xc004ba5b60)
/go/pkg/mod/github.com/lestrrat-go/httprc@v1.0.5/fetcher.go:37 +0x86
github.com/lestrrat-go/httprc.runFetchWorker({0x2104a00, 0x3007980}, 0xc0000bbbc0, {0x0, 0x0})
/go/pkg/mod/github.com/lestrrat-go/httprc@v1.0.5/fetcher.go:177 +0x2ac
created by github.com/lestrrat-go/httprc.NewFetcher in goroutine 51
/go/pkg/mod/github.com/lestrrat-go/httprc@v1.0.5/fetcher.go:75 +0x1bc

Given the goroutine is launched inside httprc, I don't have any mean to recover from this panic.
Here is the code on the jwx side if it helps:

 jwx code
type Verifier struct {
	self       string
	pkEndpoint string

	c  *jwk.Cache
	cs jwk.Set
}

func NewVerifier(host, self string) (*Verifier, error) {
	ctx := context.Background()
	pkEndpoint := host + ApiGetPublicKey
	c := jwk.NewCache(ctx)
	err := c.Register(pkEndpoint, jwk.WithMinRefreshInterval(5*time.Minute))
	if err != nil {
		log.Errorf(ctx, "Error registering public key endpoint: %v", err)
		return nil, err
	}
	_, err = c.Refresh(ctx, pkEndpoint)
	if err != nil {
		log.Errorf(ctx, "Error fetching public key: %v", err)
		return nil, err
	}
	cached := jwk.NewCachedSet(c, pkEndpoint)
	v := &Verifier{
		self:       self,
		pkEndpoint: pkEndpoint,
		c:          c,
		cs:         cached,
	}

	return v, nil
}

func (v *Verifier) Verify(token string, maxRetry int) (bool, error) {
	ctx := context.Background()
	for i := 0; i < maxRetry; i++ {
		// allow refreshing public keys if fails
		_, err := jwt.Parse([]byte(token), jwt.WithKeySet(v.cs))
		if err == nil {
			return true, nil
		}

		log.Errorf(ctx, "Error verifying token: %v", err)
		_, err = v.c.Refresh(ctx, v.pkEndpoint)
		if err != nil {
			log.Errorf(ctx, "Error refreshing public key: %v", err)
			return false, err
		}
	}

	return false, nil
}

For what it's worth we noticed this after upgrading from jwx v2.0.21 to v2.1.0 which upgrades httprc from v1.0.4 to v1.0.5.
The only suspicious change (to me) there is the change in where the queue mutex is locked in fetchAndStore here.

I see similar panics with httprc 1.0.5, stemming from code that uses jwx 2.1.0 like LeslieLeung's.

The only suspicious change (to me) there is the change in where the queue mutex is locked in fetchAndStore here.

I think so too, the mutex isn't held during the q.fetch.fetch(...) call anymore.
This probably allows a foreground Refresh(ctx) and the background fetcher to call q.fetch.fetch(...) concurrently with the same *fetchRequest, leading to the channel being overwritten and closed twice.

So I realized that my statement above was not quite correct. jwx v2.0.21 is where the upgrade from v1.0.4 to v1.0.5 happened.
@natenjoy I see that you created the PR that made this mutex change, have you observed these issues at all?
@lestrrat any thoughts on this issue?

I don't see it in our logs for the past 3 months. Are you able to reproduce this issue somewhat regularly? If so, could you set GOTRACEBACK=all and rerun the service, looking for likely causes in other goroutines? Looking at httprc 1.0.5 and those line numbers from the stack trace, the code in the trace appears to be trying to receive from a channel and not send on a closed channel. Sending on a closed channel is what is panicking.

the code in the trace appears to be trying to receive from a channel and not send on a closed channel.

The line number indicated at the top of the traceback is a select block, which does contain a send case:  case reply <- fr:. This is where the panic originates from, I guess go's tracebacks just don't distinguish between which case caused the panic.
As I mentioned above, I'm pretty sure that the reason for the channel being closed is that 2 goroutines are executing (*fetcher).fetch(...) with the same *fetchRequest in parallel. While each fetch() does create a new reply channel, the channel is written into the same *fetchRequest pointer. This pointer is then added to f.requests here once by each call. After the first of those 2 entries in f.requests is done, the channel is closed in reply(...). This leaves the 2nd entry with a closed channel from the start, ultimately causing the panic.

The line number indicated at the top of the traceback is a select block, which does contain a send case: case reply <- fr:

Correct, I misread that second case. I opened https://github.com/lestrrat-go/httprc/pull/31 as a revert commit of the prior change.

Sorry, I completely missed the original report 2 weeks ago. I'm a bit busy today, please give me a day or two to sort things out

Okay. I understand the analysis, and I will take everybody's word for it, but the truth is that I can't reproduce it, so tbh I feel a bit uneasy.
I'll merge #31 and what not, but  If anybody can come up with a reliable reproduction test case in the future, I think it will benefit all of us.

merged #31, so will close for now. Please open a new issue if problem persists

@TheJokr Thanks, merged as #32

authors: LeslieLeung TheJokr lestrrat milescrabill natenjoy

repo: lestrrat-go/httprc


text:
Fix verb methods context handling

Verb methods would hang indefinitely with the new request 2.54.1 version.

+1 for this PR
This solves the hanging issue on some requests. Either this or just you have to be specific about the Request library version.

ping @leukhin

Merged. Thanks!

@leukhin can you release a new version to npm please? Thanks.

@fixe Done!

authors: fixe kikoanis leukhin nunofgs

repo: leukhin/co-request


text:
x:Bind string interpolation error

I have this line of code in WPF app
 <TextBox Text="{x:Bind $'String value: {Username}'}" />
and compiler is throwing error
error MC3043: Names and Values in a MarkupExtension cannot contain quotes. The MarkupExtension arguments ' $'String value: {Username}'
I'm using VS 17.5.5 and I don't know whats the problem. Does anyone else has this problem? Am I missing something?

You have to use the m: Namespace
  xmlns:m="http://compiledbindings.com"
  mc:Ignorable="d m"

<TextBox m:Text="{x:Bind $'String value: {Username}'}" />

Alternatively you can use square brackets instead of curly ones.
<TextBox Text="[x:Bind $'String value: {Username}']" />

The second option I do not have yet in readme.

I was missing m: in front of text as I was looking this example from README
<Label Text="{x:Bind $'Decimal value: {DecimalProp:0.###}, Boolean value: {BooleanProp}, String value: {StringProp.TrimStart('0')}'}" />
which is missing that m: in front of text :)

repo: levitali/CompiledBindings


text:
plugin layer fails

Hi, thank you for your code. I tried to implement a upsample plugin layer the way you did, but I kept receiving errors as Segmentation fault.

Could you give me some hints? Thank you very much!

The upsample layer can be created, but every time when trying to execute the upsample layer, it will crush.
I don't know where I did wrong. Anyone has meet similar problem?
Thanks in advance!

Just take a look to your execute and enqueue funtion in your upsample plugin. You can print some log for help.


text:
Add storage device types to PhyIoType

Added storage devices types to enum, also added LTE type to be in sync with controller API.
Signed-off-by: Pramodh Pallapothu pramodh@zededa.com

@zedi-pramodh it is a good idea to put the derived (produced by protoc and vendored) files in a separate commit from the proto file source changes, since that makes it easy to drop the commit with the derived files (and generate a new one) as part of resolving conflicts like the one you have now.

Got it. Since I need to amend this PR anyway, I will submit two commits as you suggested.

authors: eriknordmark zedi-pramodh

repo: lf-edge/eve


text:
Rethink device handling

model.pd_to('cuda')

How should the model be sent to device? And how should postprocess handle device?

Consider linking with the outcome of #215

authors: blythed jasonkhadka


text:
[Android] Example app - card turns halfway then flips back

Hi there :)
The card turns halfway, switches sides (correctly) but then instead of continuing to flip left to right, it flips back right to left.
So you end up in the correct state but the animation isn't a full flip.
It's like going halfway to turning the card over and then it magically changing what's on its front face, and then turning back - it breaks the illusion of a "card" existing and isn't the desired behaviour I think.
React Native 0.57
It works fine on iOS though

I modified the code：
if (Platform.OS === 'ios') {         // cardB Y-rotation         bYRotation = rotation.y.interpolate({           inputRange: [0, 50, 100, 150],           outputRange: ['0deg', '180deg', '0deg', '-180deg'],           extrapolate: 'clamp'         })       } else {         // cardB Y-rotation         bYRotation = rotation.y.interpolate({           inputRange: [0, 50, 100, 150],           outputRange: ['0deg', '-180deg', '0deg', '180deg'],           extrapolate: 'clamp'         })       }

This is still an issue.
rotation on 'x' axis works perfectly buy 'y' rotation is getting half way and then flipping back. I have the latest version but it still happens.

@mikejsdev you can use my fork for this fix
https://github.com/ARazaAnjum/react-native-card-flip

authors: ARazaAnjum f6m6 mikejsdev weifengzz

repo: lhandel/react-native-card-flip


text:
main.py, variables.py: Check whether the executable of our inject program is present

For #88, #92, #120, and #140.
When it's missing, the script prints an error message and instructions.

I'm running in a race condition in my aur-package-chroot. The help text call for the completions can happen before the executable finished building hence the missing executable text comes instead of the help text.
Is moving the executable check below the argparser initialization enough to avoid this?

I'm running in a race condition in my aur-package-chroot. The help text call for the completions can happen before the executable finished building hence the missing executable text comes instead of the help text.
Is moving the executable check below the argparser initialization enough to avoid this?

authors: Lucki kakurasan

repo: lhark/truckersmp-cli


text:
Standardize method verbage

Main one is remove vs del (leaning towards del at the moment). Also shows up in add vs create, and update vs. a more specific action.  With regards to the update option, in many cases the specific action reflects ESI's current supported behavior but update is future proof if they allow for more changes in the same end point in the future.

Fixed in superagent branch

Fixed in #31

repo: lhkbob/eve-swagger-js


text:
[Question] Redraw

Hi, friend.
var DateComponent = {
  oninit: function (vnode) {
    var self = vnode.state;
    self.month = ["Enero", "Febrero", "Marzo", "Abril", "Mayo", "Junio", "Julio",
                         "Agosto", "Septiembre", "Octubre", "Noviembre", "Diciembre"];
    self.day = ["Domingo", "Lunes", "Martes", "Miércoles", "Jueves", "Viernes", "Sábado"];

    self.show = function () {
      self.date = new Date();
      self._ = self.day[self.date.getDay()] + ", " +
                      self.date.getDate() + " " +
                      self.month[self.date.getMonth()] + " de "  +
                      self.date.getFullYear() + " " + self.date.getHours() + ":" +
                      self.date.getMinutes() + ":" + self.date.getSeconds()
      m.redraw();
    };
    vnode.state.interval = setInterval(self.show, 1000)
  },
  onremove: function (vnode) {
    clearInterval(vnode.state.interval);
  },
  view: function (vnode) {
    return m("label", vnode.state._);
  }
};

var LayoutMain = {
  view: function (vnode) {
    return m(".layout", vnode.children);
  },
};

m.route(root, "/", {
  "/": {
    onmatch: function () {
      if (false) m.route.set("/auth")
    },
    render: function () {
      console.info("Testing")
      return m(LayoutMain, m(HomePage));
    },
  },
});

Is it normal that every time I call "redraw" I draw the entire component of the router?
debug console "300 Printer Testing"
What do you recommend me to do? To avoid poor performance in my application?
thanks, sorry for my bad english.

Hi @pcariel, it's absolutely normal that all components should be recomputed on redraw - this is essential for Mithril to work. This is standard behaviour, and Mithril has extremely good performance. Executing the component functions and analysing the results is trivial work - the DOM mutation (where performance bottlenecks usually occur) is always the minimum necessary, so in the case of draws after the first, the only part of the DOM that will be mutated is the text node representing the date.
You can see this by attaching a DOM mutation observer to the document: https://jsbin.com/mumanuj/edit?js,console,output

More details on the autoredraw system are also available on the official site.

authors: barneycarroll pcariel tivac


text:
Unexpected routing

Hi Leo,
I've found an interesting issue in mithril m.route seems to redirect to the default route if you override the anchor tag that has m.route as config, to demonstrate the issue:
http://codepen.io/anon/pen/ZYqGaB
Basically you get /one -> / -> /two
Override function:
ctrl.onc = function(event) {
  if(document.URL !== event.target.href) {
  	console.log('route to: '+event.target.href);
    m.route(event.target.href, {});
  }
  event.preventDefault();
  return false; // prevent default/propegation
};

And anchor:
return m('a', {config: m.route, onclick: ctrl.onc, href: "/two"}, "click me to see root run, then get redirected to /two")

When you use config : m.route, Mithril binds an event listener for the click. So as soon as module1's view is rendered, Mithril attaches an event listener. However, this binding only happens when the element is rendered to the page: the onclick event you bound was already attached before the element was rendered. So when you click, your onclick handler resolves first. The full href is parsed by the router – http://s.codepen.io/boomerang/1fabba7480ed69a13c8745a69e3816e11426074808651/index.html?/two – and it can't find any matches, so it goes back to the default route. All this happens synchronously. Then when the synchronous stack has resolved, the event handler that Mithril bound internally as a result of config : m.route resolves – and that makes sense, so it goes to /two.
The problem is that you're trying to get automatic route handling, but then you're trying to override it. You have to go one way or the other, since otherwise the automatic handler has no idea you've decided to take care of things yourself. Either change the href in the view to represent the route outcome you want, or don't use config : m.route, and rely on logic in the onclick handler to cater for all outcomes.

Ok, so from what you're saying, there are 2 options:

Change the href to be the full URL, ie: document.location + "?/two"
Or don't use config: m.route

Is that right?

Ok, I see the problem - using the full path doesn't match, it MUST be "/two", and it will work fine - here is a 2nd example where I'm just using "/two":
http://codepen.io/anon/pen/Pwyjoo
It goes straight to "/two" :)
@barneycarroll Cheers for the info, it helped a lot :+1:

@jsguy yeah config : m.route does some clever stuff — you need to either replicate that logic or just rely on href attribute being right in the virtual DOM (TBH looking at the original problem it's hard to work out what the user wanted in the first place).

authors: barneycarroll jsguy


text:
Enabled hooks, defined task switch hook

#323

Tested on hardware, seems to work just fine

repo: lhr-solar/Controls


text:
What is the meaning of xx / yy?

I'm looking at the docs for Matter.Composites.stack. Most of the params are pretty intuitive, but I can't figure out what xx and yy mean? Are these documented?

Ah, I looked at the code and figured it out — it's the X/Y position of the start of the stack. It'd be great if the docs explained that :)

repo: liabru/matter-js


text:
Support Immutable Color

We are running into an issue where a static Color stored in the Colors map is being changed - so that the shared Color used across the game changes also. (i.e "BLACK" was changed to no longer be black)
While we've fixed the cause of this, it isn't possible currently to make a Color object that enforces this. The r g b values are public, which means there is no easy way to subclass Color to make an "UnchangeableColor" class. There are hundreds of direct references to these public variables in the libGDX codebase.
Here's what i'd propose:

Replace the public float r, g, b, a with getters and setters.
Create an unchangeable Color class that is only set in the constructor and use it for colors like "BLACK" and "RED" which should never be accidentally changed, and optionally for any unchangable colors that a game wants to add to the Colors map.

Thoughts?

Replace the public float r, g, b, a with getters and setters.

Will break a lot of code.

Please use the search. This has been discussed before.

Also, libgdx source is plenty of static fields that should be ideally immutable: colors, Vector2.ZERO and Vector3.X are just some of them.
Personally I don’t think the change you're proposing is worthwhile, since it would break a lot of user code.

Avoiding the design issue by saying "please use the search" is counter-productive. It has been discussed, but obviously not resolved from a design perspective.
It's very difficult to search a large code base for subtle changes to public variables that may be introduced by someone who doesn't know what they are doing with libGDX code. Better to design the class so that it keeps the private variables private to support subclassing.
In regards to breaking user code, it's unfortunate that it was designed they way it was, but sometimes we need to make API breaking changes to make the design less brittle.

There was (and still is) a reason for designing it that way.
Color and similar classes like Vector2 are very basic and their properties are accessed and changed very often. Doing this via a getter and setter every time can potentially slow down the game, because of the method invocation overhead.

It seems likely that RoboVM would optimize that at the bytecode level for the iOS client. On android, similar optimization can happen via ProGuard, which is not difficult to add (and beneficial for other reasons.) I'm not sure the performance impact is a deal-breaker here, but it would have to be investigated before making a call.

Seems like basic getters/setters are inlined by the JIT on Android as of Gingerbread (2.3).
http://stackoverflow.com/questions/4912695/what-optimizations-can-i-expect-from-dalvik-and-the-android-toolchain/4930538#4930538

Don't forget about GWT.
You might be right and nowadays it might not be a serious problem anymore. But before someone actually benchmarks this, it might be dangerous to just change it and assume it won't have an impact.
The fact that it would break every single existing libgdx application is a more serious problem though. Might be change for libgdx 2.0 maybe?

Let's not repeat the same discussion again, unless there's actually something new to add.
Btw, inlining would only be possible if there is only one implementation.

Aside from the enormous breakages, and that colorInstance.setA(colorInstance.getA() + delta) is horrible to work with, this would also mean that we should do the same for all objects that we have 'constants' for, e.g. Vector as @davebaol pointed out.  (To keep consistency across the API)
Imo the breakages and the resulting inconvenient API is a good trade off for not getting bit by mutating mutable objects.  Which is a fairly rare case, and one that happens only once.
This has been discussed before, as you can find in the github issues history with just a quick search, and the result was that these weren't to be changed.

Can someone link this old discussion? I can't find it in 5 pages of issues search results. You may have too much faith in the issues search ;) The closest thing I see is this discussion on setColor in Actor, which is not what we are talking about here. https://github.com/libgdx/libgdx/issues/3380

#520, #2558

This goes way back (e.g. this) and every now and then it pops up again. I'm sure that if you search a bit more that you will find more. As you were asked to do before you opened this issue: make sure to search all issues, not only the open issues.

Ok I think we can close this and take those breaking changes into account for libgdx 2.0, if there ever will be.

@xoppa I don't appreciate your tone here. Of course I searched for previous closed issues before posting, as evidenced by your having to dig up an old post from google code 5 years ago.
Possibly the fact that it keeps popping up is evidence that there is room for improvement in this area.

Please don't comment on this issue anymore. We've responded to your request, explained why it is how it is and why it is unlikely to be changed without a major version (if even then).
If you can't wait or have special needs from the library, please feel free to fork and maintain a copy having your desired modifications.

Not quite done with this yet. Thank you to the people who made substantive replies.
Just a bit more context as the pros and cons of this are weighed. I'd like to help other developers avoid the very tricky task of tracking down an unintended change to the static colors. We just found an issue that stemmed ultimately from this Color design choice, and it took a while (as in months) for testing to track this down. The changed color was in a completely different part of the game than where it became visible.
If we need to make an API change, that means compile errors, and while annoying, those are much easier to deal with than the unexpected changes in a static color or Vector2 that ideally would never change.
Performance and usability are legitimate concerns here, and it would be fine to kill off the idea if it could not be made to perform well.
Thanks for considering for an eventual change here. Sorry to bring up an old debate without realizing it!

@ericnondahl as a side note: you can use Java's Color + converting util methods (toGdxColor(javaColor), toJavaColor(gdxColor)), preferrably using caching & pooling, to facilitate immutability, instrumentability & safety. It will make your code slightly longer (since every call requiring a GDX Color will have to have the converter called), but for non-time-critical parts of the code it simply won't matter, and for time-critical ones you'd want to stick with GDX's Color anyway for the sake of old Dalvik versions.

@FyiurAmron thanks, good idea. Currently trying a similar work-around for the issue by creating new Color objects by default instead of using the static ones (except for performance intensive places.)

I apologize for bringing this up again, but I've had a similar design problem in my game engine and here's how I solved it:
public interface ColorInterface {
  public Color mutable(); // returns mutable verstion, or this if already immutable
  public ImmutableColor immutable(); // returns an immutable version, or this if already immutable
  public boolean isMutable();
  // getters and setters which accept floats, ints, packed ints, & ColorInterface
}
public class Color implements ColorInterface {
   public float r, g, b, a;
  // the normal color class which is still used everywhere
}
public class ImmutableColor implements ColorInterface {
  // bare implementation of ColorInterface which ignores SETs. Maybe throws an exception?
}
pubic class Colors {
  public static final ImmutableColor WHITE = new ImmutableColor(1f, 1f, 1f);
  // ... more colors
}

I still was able to use the Color class everywhere - and also added methods to accept a ColorInterface when it made sense. In some places I had ColorInterface (like the background color of the window) because it wasn't a place that was going to have heavy modifications to the color.
Not sure if this is of any help, but there it is!

What I like in the above from ClickerMonkey is that you can actually find out in your code base where you are making the error of changing the color, by throwing an error for the ImmutableColor. Currently in libgdx there's no such way and I've been bitten by this as well.
Perhaps some debug/alternative version of color with setImmutable() and snapshoting color state could work, by adding debug checkpoints?

authors: ClickerMonkey FyiurAmron MobiDevelop Tom-Ski cjuillard davebaol ericnondahl nooone xGnoSiSx xoppa

repo: libgdx/libgdx


text:
Game crashing without stacktrace, presumably box2d

Issue details
Game crashes on certain collisions?
Windows freezes with no output
Reproduction steps/code
https://github.com/ForsakenHarmony/GameProt
crashes when shooting other ships
controls: wasd,space
Video: https://youtu.be/11KLDI5MC9o
Version of LibGDX and/or relevant dependencies
gdxVersion = '1.9.4'
box2DLightsVersion = '1.4'
ashleyVersion = '1.7.3'
aiVersion = '1.8.0'
kotlin_version = '1.0.4'
Stacktrace
//no output

Please select the affected platforms

[x] Windows
[ ] Linux (didn't test)
[ ] MacOS (didn't test)

Please use the forum or the irc for help with your own code.  https://github.com/libgdx/libgdx/wiki/Issue-Tracker

authors: ForsakenHarmony Tom-Ski


text:
[Enhancement] Decal.update()

Hi,
I have found out that sometimes it is useful to get Decal vertices for game logic, The problem is that update() method is protected. We get zero values when calling Decal.getVertices() on non updated Decal (before first draw call).
Solution:
A) make Decal.update() public as it contains logic avoiding recalculating vertices anyway
B) call Decal.update() inside Decal.getVertices()

I vote for B since it makes sense that you'd expect the vertices to be there when getting them, and then we aren't polluting code completion with a method you should never need to directly call. It's technically a breaking change as it's theoretically possible someone uses getVertices() to see the state of the vertices before they are updated. But I think the likelihood that anyone is doing that is extremely low.
Do you want to submit a PR?

Yes I will submit PR. In my opinion returning non updated vertices is not ok so I vote for B.

authors: MichalLebeda cypherdare


text:
#4267 Fixing a deadlock problem in AndroidGraphics and AndroidGraphicsLiveWallpaper

There was a problem with a possible deadlock ocurence in AndroidGraphics and AndroidGraphicsLiveWallpaper classes.
A thread was waiting for notify signal, the problem is that other thread can call
notify before the current thread executes wait on the synch object
(this scenario results in a deadlock).
I implemented solution by using semaphores for synchronization instead of mutexes.
Semaphores have different behaviour compared to mutexes,
in that they internally remember number of permits,
thread that calls signal on a semaphore will cause that thread that later arrives
on a semaphore uses that permit, unlike mutexes where waiting thread will
be unblocked only if it currently waits on a mutex.

@cypherdare in #5312 you said you wanted to test this PR, where you able to do that?

Did #6074 maybe fix this problem? Or this a different issue?

@MrStahlfelge I recall testing it not by building it myself, but by copy-pasting a clone of this class and using it in my project. It continued to deadlock with the Semaphores.
But merged PR #6074 is supposed to fix this issue. I have a particular app that frequently has the deadlock on a Google Pixel. I've installed my app on that phone after building it with 1.9.11. I'll periodically check it over the next week or two and see if the #6074 fix holds up. If so, I can close these other PRs/issues related to the deadlock issue.
I don't know for sure if #5042 is the caused by the same problem, but I don't recall experiencing it in the past couple years, so I'll at least close that one.

Did #6074 maybe fix this problem? Or this a different issue?

It should fix issue at least for non-Wallpaper version, however I haven't do extensive test with wallpaper-version, so I'm not sure, but by looking just at code it should work (or maybe you need to port the same logic to AndroidGraphicsLiveWallpaper, I don't remember if they share cleanup method code).

I can study it more closely in a few days and see whether similar changes should be carried over to wallpapers.

I haven't had a chance to look at the wallpaper solution yet, but so far have had no issues with #6074 so I'm closing this one.

authors: MrStahlfelge SimonIT cypherdare marko-vasic ufoq


text:
jnigen: Support specifying APP_ABI in Application.mk

Default changed to "all".
"all" will allow building on more recent NDK's, omitting armeabi builds.
Updated generated Application.mk included for all libgdx projects.
See https://developer.android.com/ndk/guides/application_mk#app_abi for more information.

This is step 1 on removing support for armeabi.

Closed in favor of #5701


text:
Removed extra spaces in default log tags

I don't think these spaces were meant to be there, but I don't know. Now all tags are consistent though.
It currently looks like this when logging an info message:
[[INFO] ] This is my info message

It was deliberate to achieve the same indentation level for all logs. I agree that it could have been achieved by proper log format (same for the parenthesis), but it's so small of a change that I'd keep it as is for consistency with the older versions.

authors: czyzby maltaisn

repo: libktx/ktx


text:
Add a reset method to streams.

Most stream implementations (e.g., yamux, spdy, quic) support half-open streams. This means that Close only closes one side.
We need a Reset() method to force-close the connection (closing both sides).
See: https://github.com/ipfs/go-ipfs/issues/3651#issuecomment-315580747

Done.


text:
Libp2p dialer should try to find multiaddrs for a given peer if it does not know any

Version:  N/A
Platform: N/A
Subsystem: Dialer

Type: Enhancenment
Description:
We currently recommend users to do libp2p.dial(peerId) to dial peers. This will obviously fail if multiaddrs for the given peer were not previously added to the addressBook (either manually, or via a discovery mechanism/DHT/rendezvous...)
If libp2p has a peer-routing module, it should perform a lookup in order to try to find multiaddrs for the given peer, before simply failing with "no available multiaddrs for the given peer"
Context: https://github.com/libp2p/js-libp2p-interfaces/issues/85

This beyond the scope of the dial functionality, will not fix.

authors: maschad vasco-santos


text:
New additional usb id for Logitech G102

New additional usb id for Logitech G102: usb:046d:c09d

Hi, thank you for you contribution!
Just for future, could you post the output of lsusb -vd 046d:c09d?

repo: libratbag/libratbag


text:
New Additions to Monochrome

Additions include:

Casio PV-2000
ColecoVision ADAM
Commodore VIC-20 (Cartridge)
Commodore Plus 4
Emerson Arcadia 2001
Entex Adventure Vision
Microsoft Xbox 360
Microsoft Xbox One
Minecraft
New Nintendo 3DS
Nintendo 3DS
Nintendo DSi
Nintendo Switch
Nintendo Wii U
Sharp X68000
Sony PlayStation 3
Sony PlayStation 4
Sony PlayStation Vita

Nice! Thank you!
Can you make a version of the minecraft one with a smaller cube?

Watch for it in my next PR.

authors: Kivutar baxysquare

repo: libretro/retroarch-assets


text:
libsql/core: Auto-clean dangling prepared stmnts

Add the ability for Connection to clean up any dangling prepared statements before closing the sqlite database.
Closes #204

Ah snap, I missed this PR and already pushed a small change right into crates/core that implements Drop for Connection and does the same thing (which was also done independently in sqld a while ago: https://github.com/libsql/sqld/blob/f50803f71f81bd0333972301b0c4f9de66026ccb/sqld-libsql-bindings/src/lib.rs#L39). The testing part is still super valuable though

@psarna I agree valgrind would be good to add, I opened this to follow up https://github.com/libsql/libsql/issues/233

@LucioFranco alternatively, we could try hooking up https://github.com/nnethercote/dhat-rs, their README suggests it would be useful in any kinds of tests that involve allocation checks

authors: LucioFranco psarna

repo: libsql/libsql


text:
libuv crash for when calling worker

   |77          uv_mutex_unlock(&mutex);
   │78
   │79          if (q == &exit_message)
   │80            break;
   │81
   │82          w = QUEUE_DATA(q, struct uv__work, wq);
  >│83          w->work(w);
   │84
   │85          uv_mutex_lock(&w->loop->wq_mutex);
   │86          w->work = NULL;  /* Signal uv_cancel() that the work req is done
   │87                              executing. */
   │88          QUEUE_INSERT_TAIL(&w->loop->wq, &w->wq);

(gdb) p	*w
$5 = {work = 0x0, done = 0x7ffff7bbed70 <uv__fs_done>, loop = 0x7ffff7dd7d80 <default_loop_struct>, wq = {0x60a6b0 <file_watcher_g+4656>,
    0x60a6b0 <file_watcher_g+4656>}}

Can you give a bit more background?

What is your application doing?
What is the backtrace of this crash?

Thank you for your response;
it is application for a customer, herein the back trace

#0  0x0000000000000000 in ?? ()
#1  0x00007ffff7bbad6f in worker (arg=<optimized out>) at src/threadpool.c:83
#2  0x00007ffff73c9184 in start_thread (arg=0x7ffff61ba700) at pthread_create.c:312
#3  0x00007ffff78e4ffd in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:111

the application watch a folder using inotify events, if a particular file is written, it parse only the added lines, then send the result back to a server. for a little amount of bytes, it works fine, but when logging a moderate quantity to the file i have this crash

Just to be sure, the application is using libuv directly, rather than via some kind of wrapper like Node.js?

What version of libuv is it?

yes i use libuv directly, i use the last version from github now.

Can you post minimal steps to reproduce?

Also, what platform?

A quick check suggests that the only way for a work item's "done" to be uv__fs_done as in your example is via the corresponding fs.c for the platform (src/win/fs.c, src/unix/fs.c). These both set the work item's "work" CB to uv__fs_work.
Is it possible the issue here is memory corruption? Have you tried running a tool like valgrind to check your application for stray writes?

yes; i have tried to run valgrind, i have errors like
==7324== Warning: set address range perms: large range [0x3a044040, 0x6c344a91) (undefined)
==7324== Warning: set address range perms: large range [0x3a044028, 0x6c344aa9) (noaccess)

now, i have switched quickly to libev because i haven't enough time. seems that the libuv version 0.1 is more stable than the version 1.0

I'll close this out then.  Next time, please take the time to put together a better bug report, or post to the help repo instead of the main bug tracker.

authors: bnoordhuis davisjam defeden

repo: libuv/libuv


text:
Can this be used for monogame.

As per the title can this still be used for monogame i know for a while they were using it built in.
Sorry i don't know if there is a forum this is the only place i could find to post.

You probably already found out, but for future visitors; yes, absolutely. I've done it before building towards at least Windows without any issues. There is however no inbuilt extension to write and read specific types as there is for Unity, but I suggest handling all data by your own anyway.


text:
old events

simulcast and exploring still showing up on home page

Hmmm... I'm not seeing this in Chrome, Firefox, or Safari on my mac. Is this on iPhone?

authors: agarrharr keelyjay


text:
Docker build still broken avec nixos fix

See https://app.circleci.com/pipelines/github/liftedinit/many-framework/271/workflows/dcbbb73f-39b5-47ac-9e21-8c7d6cb36d11/jobs/855
Digest: sha256:d8c6b97091d6944dd773c3c239899af047077dbf5411ef229bb50e5b21404b0d
Status: Downloaded newer image for nixos/nix:latest
docker: Error response from daemon: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: exec: "bash": executable file not found in $PATH: unknown.
make: *** [Makefile:33: genfiles/many-ledger.tar.gz] Error 127

Exited with code exit status 2

Close with a cache cleanup

authors: Denommus fmorency

repo: liftedinit/many-framework


text:
Documentation - List / link to shortcodes table in documentation

Hello,
On the documentation page about parsing, an example is given regarding using custom formats for parsing but there is no list of shortcodes that can be used as the format string, nor any link to any standard leaving readers feeling confused as to what format codes are implemented.
https://lifthrasiir.github.io/rust-chrono/chrono/
It would be better for accessibility if the format codes were clearly listed.

Which can be found here http://blog.jwilm.io/retry-after/chrono/format/strftime/index.html

@sekjun9878 Hopefully this should be better. (I also synchronized some main documentation previously only available in README.md.)

Looks good. Thanks, that was fast! :)

repo: lifthrasiir/rust-chrono


text:
Shellcheck issues warning when running on circle-ci

This is seen in circle-ci logs (see here for an example):
#!/bin/bash -eo pipefail
make check dep-check
shellcheck: unable to decommit memory: Invalid argument

This GitHub issue seems to track this. It appears innocuous, but we may want to install a newer shellcheck or build it ourself to avoid this warning.

Installing spellcheck stable (0.5.0) solves the issue. https://github.com/ligato/networkservicemesh/pull/327/files#diff-dc2f19473ed2123d5ba8b6fa9cb89383

Should this issue be closed now?

I think it's fixed, would be happy to close but do not have permissions.

authors: edwarnicke mestery platoff

repo: ligato/networkservicemesh


text:
Include invoice description hash when adding invoice via RPC client

This PR modifies lightningClient.AddInvoice to pass along DescriptionHash to the underlying lnrpc.LightningClient AddInvoice call, as it was previously missing.
For example, previously the behaviour was:
// client is lndclient.LightningClient
_, invStr, _ := client.AddInvoice(ctx, invoicesrpc.AddInvoiceData{
    DescriptionHash: hashedDesc,
    Value: lnwire.MilliSatoshi(100000),
})

decoded, _ := zpay32.Decode(bolt11, chainParams)

// Prints ""
fmt.Printf("%q", decoded.Description)
// Prints nil
fmt.Printf("%v", decoded.DescriptionHash)

I didn't add any of the other fields that are also not currently passed along from the AddInvoiceData to the lnrpc.Invoice struct, as wasn't sure if that was an intentional decision. (AFAIK this includes FallbackAddr; Private, which is currently hardcoded to true; and Amp/IsAmp).
This PR also includes some initial (small) tests for the lightning_client.go file, starting with the AddInvoice method that is modified here. I figured it could be useful to ensure the underlying call is wrapped as expected, but can remove it if it doesn't seem particularly useful.
Additional context can be found in the LND Developer Community Slack (see here and here).
Pull Request Checklist

[x] PR is opened against correct version branch.
[x] Version compatibility matrix in the README and minimal required version
in lnd_services.go are updated.
[x] Update macaroon_recipes.go if your PR adds a new method that is called
differently than the RPC method it invokes.

Left the PR checklist as-is, as wasn't 100% sure if I've met the reqs (specifically target branch) – feel free to check off if appropriate.

@guggero I believe I've addressed all of your comments, but let me know if there's anything else to note ⚡

@ellemouton: review reminder

thanks for the update @jstol! 🚀

@guggero @ellemouton thanks again for the feedback on this and merging it in. Just a quick follow up – looking at some of the other PRs being merged + the most recent tags, and now wondering if this change needs to be merged into version-specific branches. Not sure if that's not needed in this case (and if master is fine).

We only use specific branches for new RPCs that don't exist in the version that master branch tracks (currently that's lnd v0.14.3-beta and older). So because those fields existed for a very long time, master is fine in this case.

authors: ellemouton guggero jstol lightninglabs-deploy

repo: lightninglabs/lndclient


text:
tapdb: improve batch marshalling

Addresses #1125 .

Pull Request Test Coverage Report for Build 10909138510
Details

1 of 3 (33.33%)  changed or added relevant lines in 1 file are covered.
13 unchanged lines in 5 files lost coverage.
Overall coverage increased (+0.003%) to 40.402%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tapdb/asset_minting.go
1
3
33.33%

Files with Coverage Reduction
New Missed Lines
%

commitment/tap.go
1
83.64%

tapdb/addrs.go
2
79.04%

tappsbt/create.go
2
53.22%

universe/interface.go
4
50.22%

tapdb/universe.go
4
80.91%

Totals

Change from base Build 10901514108:
0.003%

Covered Lines:
24214

Relevant Lines:
59933

💛  - Coveralls

authors: coveralls jharveyb

repo: lightninglabs/taproot-assets


text:
Bogus statistical information (HSWC)

Background
I see the following log message:
2018-05-11 15:50:04.286 [INF] HSWC: Sent 0 satoshis received -105 satoshis in the last 10 seconds (1.8446744073709553e+18 tx/sec)
I received 150 satoshis a while ago (an hour maybe?), and I don't know why this popped up.
Your environment

lnd version 0.4.1-beta
Linux ftp 4.9.0-3-amd64 #1 SMP Debian 4.9.30-2+deb9u2 (2017-06-26) x86_64 GNU/Linux
Bitcoin Core 0.16.0

Steps to reproduce
I opened a channel, sent 1000 sat, received 150 sat. Then I did nothing for a while.
Expected behaviour
The message should not appear at all.

This might be related to a channel closing (Eclair on my phone).

repo: lightningnetwork/lnd