Sending payment not working

My node is online:
https://explorer.acinq.co/#/n/0366e4191c0271562700edfcfadca560c1a999b0b76731cf97eed1c5695bc418d9
Got 3 peers and 6 Channels.
Balance is as following:
"total_balance": "61109819",
"confirmed_balance": "61109819",
"unconfirmed_balance": "0"
When making a transaction I input following:
``$ lncli sendpayment --pay_req=lntb19u1pdt07h5pp5ra6qa7hmpd3qatr.........07g46mu97u5s2q2jsyjvg`
The issue is that nothing happens after this step. The "cursor" just stays one line below without the prefix
admin@AURORA:~ $
I can press Ctrl+C and then enter $ lncli listpayments
Result is:     "payments": [
]
Tried 5 times with different payment requests.
What may the reason be that I can't perform payments?

Does this happen with all payments you are trying to send? I had a similar issue with some certain nodes, the Blockstream node in particular (in the LND Slack was suggested that it is because they are running an older version of c-lightning, which causes the issue). Perhaps try to get/send a tip here: http://tip.bight.nl - the maintainer has a LND node.
Also, perhaps try lncli payinvoice <paymentrequest> instead?

It happens with Y'alls and Starblocks.
Tried  http://tip.bight.nl without adding node infos or anything besides the amount of satoshis and got this message:
 lncli sendpayment --pay_req=lnbc1u1pdtsqlcpp5sg7xe76q...............qp7jv8u9
[lncli] rpc error: code = Unknown desc = unknown network
Same Error when adding my own node's ID in the node description.
Tried lncli payinvoice <paymentrequest> and got the following:
{
"payment_error": "unable to route payment to destination: UnknownNextPeer",
"payment_preimage": "",
"payment_route": null
}

Oh, are you on testnet or mainnet? Starblocks is definitely testnet and Yalls has both versions (depending on their URL). "Unknown network" might suggest you are on testnet trying to pay a mainnet invoice?

I'll drop a part of the log in here in case it helps. Can always delete it if it doesn't help.
Mar 25 22:11:20 AURORA lnd[20662]: 2018-03-25 22:11:20.756 [INF] PEER: unable to read message from <IP_ADDRESS>:46062: read tcp <IP_ADDRESS>:9735-><IP_ADDRESS>:46062: use of closed network connection
Mar 25 22:11:20 AURORA lnd[20662]: 2018-03-25 22:11:20.756 [INF] HSWC: Removing channel link with ChannelID(eb16e993ccf7cc679c01198a2b5def9f322100b293abbc07cae566225046a4c6)
Mar 25 22:11:20 AURORA lnd[20662]: 2018-03-25 22:11:20.759 [INF] HSWC: ChannelLink(c6a446502266e5ca07bcab93b20021329fef5d2b8a19019c67ccf7cc93e916eb:0) is stopping
Mar 25 22:11:20 AURORA lnd[20662]: 2018-03-25 22:11:20.761 [INF] NTFN: Cancelling epoch notification, epoch_id=195
Mar 25 22:11:20 AURORA lnd[20662]: 2018-03-25 22:11:20.762 [INF] HSWC: ChannelLink(c6a446502266e5ca07bcab93b20021329fef5d2b8a19019c67ccf7cc93e916eb:0) has exited
Mar 25 22:11:21 AURORA lnd[20662]: 2018-03-25 22:11:21.126 [ERR] HSWC: unable to find links by destination unable to locate channel link bydestination hop id 022bc10a5df9d410edf883916e08e2ed35f5ea948d3e3e16c3e15fba6cbdd5287d
Mar 25 22:11:21 AURORA lnd[20662]: 2018-03-25 22:11:21.127 [ERR] CRTR: Attempt to send payment 1f75291ee123b8db41d46d1ab52195a90e7b0962b03bebae289d6d7de7599228 failed: UnknownNextPeer
Mar 25 22:11:22 AURORA lnd[20662]: 2018-03-25 22:11:22.209 [INF] HSWC: Sent 0 satoshis received -304 satoshis in the last 10 seconds (1.8446744073709553e+18 tx/sec)
Mar 25 22:11:22 AURORA lnd[20662]: 2018-03-25 22:11:22.447 [ERR] CRTR: Attempt to send payment 1f75291ee123b8db41d46d1ab52195a90e7b0962b03bebae289d6d7de7599228 failed: TemporaryChannelFailure
Mar 25 22:11:22 AURORA lnd[20662]: 2018-03-25 22:11:22.580 [INF] DISC: Broadcasting batch of 3 new announcements
Mar 25 22:11:23 AURORA lnd[20662]: 2018-03-25 22:11:23.806 [ERR] CRTR: Attempt to send payment 1f75291ee123b8db41d46d1ab52195a90e7b0962b03bebae289d6d7de7599228 failed: UnknownNextPeer
Mar 25 22:11:25 AURORA lnd[20662]: 2018-03-25 22:11:25.789 [ERR] CRTR: Attempt to send payment 1f75291ee123b8db41d46d1ab52195a90e7b0962b03bebae289d6d7de7599228 failed: UnknownNextPeer
Mar 25 22:11:27 AURORA lnd[20662]: 2018-03-25 22:11:27.825 [ERR] CRTR: Attempt to send payment 1f75291ee123b8db41d46d1ab52195a90e7b0962b03bebae289d6d7de7599228 failed: TemporaryChannelFailure
Mar 25 22:11:29 AURORA lnd[20662]: 2018-03-25 22:11:29.907 [ERR] CRTR: Attempt to send payment 1f75291ee123b8db41d46d1ab52195a90e7b0962b03bebae289d6d7de7599228 failed: UnknownNextPeer
Mar 25 22:11:31 AURORA lnd[20662]: 2018-03-25 22:11:31.976 [ERR] CRTR: Attempt to send payment 1f75291ee123b8db41d46d1ab52195a90e7b0962b03bebae289d6d7de7599228 failed: UnknownNextPeer
Mar 25 22:11:32 AURORA lnd[20662]: 2018-03-25 22:11:32.209 [INF] HSWC: Sent 0 satoshis received 0 satoshis in the last 10 seconds (1.2 tx/sec)
Mar 25 22:11:33 AURORA lnd[20662]: 2018-03-25 22:11:33.260 [ERR] CRTR: Attempt to send payment 1f75291ee123b8db41d46d1ab52195a90e7b0962b03bebae289d6d7de7599228 failed: UnknownNextPeer
Mar 25 22:11:34 AURORA lnd[20662]: 2018-03-25 22:11:34.571 [ERR] CRTR: Attempt to send payment 1f75291ee123b8db41d46d1ab52195a90e7b0962b03bebae289d6d7de7599228 failed: TemporaryChannelFailure
Mar 25 22:11:35 AURORA lnd[20662]: 2018-03-25 22:11:35.795 [ERR] CRTR: Attempt to send payment 1f75291ee123b8db41d46d1ab52195a90e7b0962b03bebae289d6d7de7599228 failed: TemporaryChannelFailure

Ooooh yes I'm on mainnet. I'll try a Y'alls payment with your method.

 You linked to your node on the Acinq explorer in the OP, and that explorer is definitely for testnet.

I'm so sorry my brain had a fatal error when typing. I'm on testnet!
I get:
"identity_pubkey": "0366e4191c0271562700edfcfadca560c1a999b0b76731cf97eed1c5695bc418d9",
"alias": "AURORA",
"num_pending_channels": 2,
"num_active_channels": 3,
"num_peers": 6,
"block_height": 1289172,
"block_hash": "00000000de8de71b9e0d97b56a641d53ef59d4eab20dab2b74604d54731dde87",
"synced_to_chain": true,
"testnet": true,
"chains": [
    "bitcoin"
],
"uris": [
    "0366e4191c0271562700edfcfadca560c1a999b0b76731cf97eed1c5695bc418d9@<IP_ADDRESS>:9735"
],
"best_header_timestamp": "1522009807"

Y'alls payment with lncli payinvoice <paymentrequest> and got no reaction like in the first post.

I'm afraid my knowledge is limited here. Hopefully someone with more understanding will chime in. The only thing that comes to mind to try is a good old client restart (sudo systemctl start lnd) and/or to open up a second ssh window, log in to your Pi and start a realtime log file: sudo journalctl -f -u lnd, while trying the sending command in the first window, and try to make sense of what is going on.
There is a lot of chatter in the log file though, so probably this won't make much sense... So I am hoping someone else will have a better idea.

As you are up and running and LND is synced to the chain, this is somewhat outside the scope this guide. I would suggest you

ask in the LND slack channels (invite link on the lnd resources page)
open an issue in the LND github repo, but make sure to document it thoroughly

Great to hear. I wouldn't give too much thought regarding the Explorer visualization. It's not very accurate and also contains stale information. It's nice, nontheless. :-)


RandomGenerator not found

Summary
Hi friends, just got started on level 5 of the DES and I'm not sure how to use the provided class.
Description
RandomGenerator.class in cs2030.simulator package with the rest of the classes, except the main classes.
The following line is in Simulator::simulate
RandomGenerator rg = new RandomGenerator(0, 0, 0, 0);
It's the standard symbol not found error, occurring at both RandomGenerators.
Do I have to include an import statement or something in my Simulator class? I tried it and couldn't get that to work either (I don't think it's necessary anyway, since they're in the same package and RandomGenerator is package private?)
Thanks in advance!

It seems right to me but I'd like to reiterate the overall directory structure so that other viewers may know as well.
|
|_ cs2030
|   |_ simulator
|           |_ RandomGenerator.class
|           |_ Simulator.java
|           |_ Event.java
|           |_ ...
|_ Main.java

It seems right to me but I'd like to reiterate the overall directory structure so that other viewers may know as well.
.
|
|_ cs2030
|   |_ simulator
|           |_ RandomGenerator.class
|           |_ Simulator.java
|           |_ Event.java
|           |_ ...
|
|_ Main.java

or perhaps (didn't try if this works)
.
|
|_ cs2030.simulator
|       |_ RandomGenerator.class
|       |_ Simulator.java
|       |_ Event.java
|       |_ ...
|
|_ Main.java

Hi Russell, thanks for your reply. Yes, this is how my project is currently set up. Is there anything else you had to do when you first downloaded the class?

No, I simply put it inside the cs2030/simulator directory.

I am guessing you're probably using some IDE like IntelliJ or Eclipse, and trying to click run. These IDEs will throw error because they can't find RandomGenerator.java
You have 2 options:

Run using command line. Don't click the run button on ur IDE.
Copy paste the decompiled RandomGenerator.java file (I think someone posted this on some GitHub issue forgot which issue) into ur project, so that your IDE recognizes it. Rmb not to submit the file for ur submission

I am guessing you're probably using some IDE like IntelliJ or Eclipse, and trying to click run. These IDEs will throw error because they can't find RandomGenerator.java
You have 2 options:

Run using command line. Don't click the run button on ur IDE.
Copy paste the decompiled RandomGenerator.java file (I think someone posted this on some GitHub issue forgot which issue) into ur project, so that your IDE recognizes it. Rmb not to submit the file for ur submission

#127

I am guessing you're probably using some IDE like IntelliJ or Eclipse, and trying to click run. These IDEs will throw error because they can't find RandomGenerator.java
You have 2 options:

Run using command line. Don't click the run button on ur IDE.
Copy paste the decompiled RandomGenerator.java file (I think someone posted this on some GitHub issue forgot which issue) into ur project, so that your IDE recognizes it. Rmb not to submit the file for ur submission

Nope, I am using vim + javac. Screenshots as attac

hed!

I am guessing you're probably using some IDE like IntelliJ or Eclipse, and trying to click run. These IDEs will throw error because they can't find RandomGenerator.java
You have 2 options:

Run using command line. Don't click the run button on ur IDE.
Copy paste the decompiled RandomGenerator.java file (I think someone posted this on some GitHub issue forgot which issue) into ur project, so that your IDE recognizes it. Rmb not to submit the file for ur submission

#127

Ah the second one worked for me. Thanks Bing Sen and Russell!

cs2030/simulator directory

How do I save it in/find this file?
Because I didn't make this directory though? Is it auto created when you package cs2030.simulator?

Hi Devvyn, you should already have a cs2030/simulator directory for the rest of your .java classes, just save the RandomGenerator together with the rest of them. Hope this helps.


PCSX2 - system requirements tab

Similar to GitHub:

This would seems like something best suited for the guide, especially if we rename it (See #89 ) and move it into the menu. This way the first thing users will see is what it requires to run it, doesn't makes much sense on Downloads page or anywhere else to me.


Keybase App shows only blank panes

It's me again - the guy from #4472. Same problem - white panes in the app, rebooting and reloading the launchctl process don't restore it. Currently trying a complete reinstall of keybase. The app was working about a week ago, but it appears there was a brew update on 2017-05-15 which appears to have broken it for me.
my log id: f6c23ce3e165de7710bc151c

Looks like your keybase service isn't running...

keybase status says:
KBFS:
    status:    running
    version:   1.0.2-20170515141757+8c10902
    log:       /Users/joeysmith/Library/Logs/keybase.kbfs.log

Service:
    status:    running
    version:   1.0.22-20170515141716+b608f0e
    log:       /Users/joeysmith/Library/Logs/keybase.service.log

I have also just confirmed that I am able to have chats with people using the "keybase chat" tool.

Hmm, then weirdly the electron UI is having issues connecting.  It is a
permissions thing, did you run some install components as root?
Cc: @gabriel
On Thu, May 18, 2017 at 8:47 PM, Joey Smith<EMAIL_ADDRESS>wrote:

I have also just confirmed that I am able to have chats with people using
the "keybase chat" tool.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/7049#issuecomment-302579013,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA05__KfXUeDqMqqZjcn1L50_pTfJpi_ks5r7OakgaJpZM4Nf8sk
.

Not that I am aware of. However, a complete uninstall and reinstall of the app seems to have cleared the issue up, so feel free from my end to mark this as closed - mostly, I just wanted to make sure you guys got the logs in case it's a deeper issue.
If there's any commands I can run or info I can provide that might help you uncover the root cause, let me know.

This might be related to #6965 ?


Websocket Closed

Followed all steps, what do I need to open the websocket. I see disconnected but have confirmed all API info is correct.

Hey! Sorry I didn't see this sooner.
Are you seeing anything in log/tako.log?

Sadly I couldn’t get it to work so I’ve removed it from my system.


Crash after too many connections

Got the following error after about 10,000 connections:
buffer.js:1028
    throw new TypeError('"value" argument is out of bounds');
    ^

TypeError: "value" argument is out of bounds
    at checkInt (buffer.js:1028:11)
    at Buffer.writeUInt16LE (buffer.js:1086:5)
    at Mgmt.write (/…/node_modules/bleno/lib/hci-socket/mgmt.js:80:7)
    at Mgmt.loadLongTermKeys (/…/node_modules/bleno/lib/hci-socket/mgmt.js:66:8)
    at Mgmt.addLongTermKey (/…/node_modules/bleno/lib/hci-socket/mgmt.js:47:8)
    at Smp.handlePairingRandom (/…/node_modules/bleno/lib/hci-socket/smp.js:138:10)
    at Smp.onAclStreamData (/…/node_modules/bleno/lib/hci-socket/smp.js:58:10)
    at emitTwo (events.js:111:20)
    at emit (events.js:191:7)
    at AclStream.push (/…/node_modules/bleno/lib/hci-socket/acl-stream.js:26:10)
    at BlenoBindings.onAclDataPkt (/…/node_modules/bleno/lib/hci-socket/bindings.js:198:21)

Node: 6.3.0
Bleno: 0.4.1

@guimier this one is going to be tricky for me to reproduce ...
A few questions for you:

What are you using as the central?
Can you please provide an HCI dump capture?
Do you have bluetoothd running - if so which BlueZ version?
Do you have any secure characteristics in your bleno app?

The central is Bluez/gatttool on another Pi/Raspbian. I do not have a dumps and probably won’t have the time now to reproduce. I think bluetoothd was running, version 5.23. I did not configure any secure characteristic. It was running with BLUETOOTH_HCI_SOCKET_FORCE_USB=1 because I did not see characteristics otherwise.
It was running with DEBUG set, too, and it was logging really big buffers (unfortunately I don’t have these any more) just before the crash. Seeing the stack above, I think the buffer became too big and the length variable could not be stored as uint16 in Mgmt.prototype.write.

@guimier how many Bluetooth adapters do you have on the Pi?

probably won’t have the time now to reproduce

If it's not easy to reproduce, please close the issue - it doesn't seem like something that is high priority ....

Closing for now due to lack of activity.


Confusion self-hit doesn't trigger Sitrus + super sitrus Berries

Mentioned in https://github.com/Zarel/Pokemon-Showdown/issues/2367#issuecomment-327332860
Not sure if this is the best way to implement this, but I can't think of anything else.
@Marty or @DaWoblefet can either of you confirm if Oran Berry/Berry Juice/Stat-boosting berries also does this or if it's only the healing berries. Also is this new in Gen 7, or was this also the case before?

I can confirm this has been the case since Gen 5 (not sure about prior). I'll check the others today.

The thing about Update is it triggers almost all the time. Even if you prevent these Berries from activating on the self-hit from confusion, they'll just activate immediately after you next use a move or confusion ends. The Berries shouldn't activate until your HP is next updated by an attack or something.
Also re: the test, the only reason it passes is because Swagger can miss, Deoxys might not hit itself in confusion, and even if it does, it knocks itself out in one hit at +2 so the Sitrus Berry is never used.

Does this work better then? I can't think of anything else beside doing something with damage events.

Also re: the test, the only reason it passes is because Swagger can miss, Deoxys might not hit itself in confusion, and even if it does, it knocks itself out in one hit at +2 so the Sitrus Berry is never used.

Will this be enough? Test PRNG will hit + self-hit in confusion on the first turn, so it shouldn't be able to pass due to luck.
0 Atk Deoxys-Attack Struggle vs. 0 HP / 0 Def Deoxys-Attack: 150-177 (62.2 - 73.4%) -- guaranteed 2HKO

The confused check for Figy and friends seems to be the opposite of what you want but otherwise this looks like a great solution! 👍

rip did
Still wondering about Oran Berry/Berry Juice/etc. though.

So what's wrong with listening to damage events?

If you mean onDamage:
 1) Emergency Exit should not request switch-out if first healed by berry:

     AssertionError [ERR_ASSERTION]: 'switch' === 'move'
     + expected - actual

     -switch
     +move

 2) Pickup should pick up a consumed item:
    AssertionError [ERR_ASSERTION]: Pick Up should retrieve consumed Sitrus Berry

 3) Sitrus Berry should heal 25% hp when consumed:
    AssertionError [ERR_ASSERTION]: Expected 'holdsItem' assertion to fail.

 4) Heal Block [Gen 4] should allow HP recovery items to activate:

     AssertionError [ERR_ASSERTION]: 'sitrusberry' === ''
     + expected - actual

     -sitrusberry

And if you meant onAfterDamage, this makes direct damage like Substitute (and various other things that uses directDamage) not trigger the berries at all. I don't know of any other damage event that could be reasonable, but neither of those work without breaking something different.

We could fix onAfterDamage...

Would it not be better to make directDamage trigger onAfterDamage (unless this was what you meant)? Or is there a reason why it doesn't at the moment.

Yes, that's what I was proposing.

@Marty or @DaWoblefet can either of you confirm if Oran Berry/Berry Juice/Stat-boosting berries also does this or if it's only the healing berries

Here's my testing (if you need the video for confirmation / how the interaction works, let me know).

Oran Berry - does not activate after confusion

Berry Juice - does not activate after confusion

Stat pinch Berries - does not activate after confusion

Berserk - does not activate after confusion

Emergency Exit / Wimp Out - does not activate after confusion. Subsequent attacks that would damage the Emergency Exit / Wimp Out Pokemon below 50% do not activate the ability.

Zen Mode - activates normally

Schooling - activates normally

Shields Down - activates normally

Soul Heart - activates normally

Defeatist - halves the user's stats normally

Doing that, the remaining problem from what I can see is that non-damage events (like Harvest harvesting a berry) won't then continue to trigger the Berry, which I think is the intended mechanic. There may be a way to work around that with some other event maybe?

Attach it to onStart too?

Uh this doesn't work as it did before. There's some problems with the order that item effects happen (Recycle eat the berry before notifying that that it got one). I'll try to see if I can fix this somehow.

Okay, this might work then. I'm not entirely sure on the activation order after Recycle Air Balloon, but otherwise this should be good.

There's at least one regression with this approach. If you're facing something with Unnerve and it knocks you into range for your Berry to activate, then it switches out, your Berry wouldn't activate immediately anymore. (In fact, it should activate even more immediately than it does currently since switching to another Unnerve Pokemon stops the Berry from activating, which is itself a bug.)

That's getting to a quite a lot of different events to consider... How many other likely regressions will come from this (Does Magic Room running out make a Pokemon eat a Berry straight away if they are in range for example?).
I guess an option would be to activate the onStart from the end events of the respective source that blocks items like this, but from what I see these events run before the switch actually happen, which would lead to berries being eaten too early instead...
onEnd: function (pokemon) {
	for (let i = 0; i < pokemon.side.foe.active.length; i++) {
		let active = pokemon.side.foe.active[i];
		let item = active.getItem();
		this.singleEvent('Start', item, active.itemData, active, pokemon, item);
	}
},

I wonder what happens if a Pokemon knocks itself below half through confusion, keeping its berry, and then something with Unnerve switches in and out, or if the confusion hit happens while the berry is being negated through Unnerve, and then the Unnerve mon switches.

At this point, I think the approach of "Update event, don't activate if previous event was confusion" might be the best approach.
But I'm starting to wonder how exactly Game Freak managed this bug. What happens if you hurt yourself in confusion, and then an opposing Unnerve user switches out?

I'm honestly not sure what the best way around this is, and there's some interactions that I can't find information on (Magic Room expire after you hit yourself in confusion, do you eat the berry?, or Zarel's Unnerve situation).

I wonder what happens if a Pokemon knocks itself below half through confusion, keeping its berry, and then something with Unnerve switches in and out before the Pokemon with the uneaten berry takes damage, or if the confusion hit happens while the berry is being negated through Unnerve, and then the Unnerve mon switches.

All of these scenarios trigger the Berry.

This should not cause any regressions (finally), but it doesn't fix the Unnerve into Unnerve problem Marty brought up earlier.

@Marty-D After some experimentation regarding

In fact, it should activate even more immediately than it does currently since switching to another Unnerve Pokemon stops the Berry from activating, which is itself a bug.

it seems as the below would "fix" it, though I don't know how cart activates the berry (in-between switches when nothing is on the field/before the pokeball or message for the new pokemon is shown/before the unnerve activation message) so I cannot claim this is an actual fix to the issue.
		onPreStart: function (pokemon) {
			this.add('-ability', pokemon, 'Unnerve', pokemon.side.foe);
		},
-		onFoeTryEatItem: false,
+		onFoeTryEatItem: function () {
+			return !this.effectData.target.isStarted;
+		},
		id: "unnerve",
		name: "Unnerve",
		rating: 1.5,

It sounds like the best solution might be a StartUpdate event that triggers on switch-in and also when Magic Room or some other effect suppressing items ends.

I don't think I know how the event system PS uses well enough to even know where to start in implementing something like that.

in-between switches when nothing is on the field/before the pokeball or message for the new pokemon is shown

It is this, yeah. No need to have it fixed right now since I think it's beyond the scope of this PR, I just mentioned it to clarify what I meant by "immediately" in that case.

So, what happens now?

I'm not sure I like the idea of adding a new member .lastEffect to the Pokémon object, purely to work around a lack of familiarity with the event system.

What if berries were refactored to only activate onAfterDamage and onStart, and effects that negate berries and other items trigger the onStart effect of all affected items when they end, like QuiteQuiet suggested earlier? That would probably end up causing a regression with Air Balloon (do something with onRestart?) but it sounds to me like that would be closer to how Game Freak implements berries in-game.

Realistically, this is the best I can manage. There's nothing that can keep the onUpdate events intact without adding a new tracker since Update events have no idea what caused the last damage.
It's basically what was before, plus the Unnerve/Magic Room stuff working properly...

Here are some tests I did in Gen 6 along with UPC's "after an attack" ordering, which I've amended using my results. Make of this what you will.
Flare Blitz -> burn -> recoil -> target's Sitrus
Knock Off -> Rocky Helmet damage -> target faints -> Knock Off effect
Knock Off -> Rocky Helmet damage -> user's Sitrus -> Knock Off effect
Knock Off damage brings target in range for Sitrus activation -> Knock Off effect
Knock Off -> target faints -> Destiny Bond knocks out user
Poison Touch Thunder Punch -> paralysis -> target's Lum Berry -> poison
Flare Blitz -> Rough Skin -> recoil -> target thaws -> target's Kee
Dragon Tail -> target is forced out before Kee Berry activates
Dragon Tail -> target is forced out before Sitrus activates
Rapid Spin -> user is freed from stuff -> target's Sitrus
Flame Burst -> bursting flame -> target thaws -> target's Sitrus
Poison Touch contact recoil move on a Pokemon using Rage -> rage is building -> poison -> target's Lum -> recoil
Flare Blitz -> recoil -> user's Sitrus -> target's Kee
Flare Blitz -> Rocky Helmet -> user's Sitrus -> recoil
Flare Blitz -> recoil -> user's Sitrus -> target thaws
Lum contact move VS Air Balloon Flame Body -> burn -> user's Lum -> Air Balloon pops
Scald -> burn -> Synchronize -> target's Lum
Mold Breaker Nuzzle VS Limber Lum -> paralysis -> target's Lum
Mold Breaker Nuzzle VS Synchronize Lum -> paralysis -> Synchronize -> target's Lum
Sitrus contact move VS Rocky Helmet Rough Skin -> Rough Skin -> user's Sitrus -> Rocky Helmet

Most "attack is successful" effects
Additional effects
Berries can trigger here (target's status curing Berries)
Grudge/Rage
Most Ability effects
Berries can trigger here (user's Berries that check HP, user's status curing Berries)
Item effects that trigger when the target is hit by the attack (including Air Balloon and Sticky Barb)
Berries can trigger here (user's Berries that check HP)
The target faints if it has 0 HP
Destiny Bond if the target fainted
Enigma Berry
Recoil damage/Struggle
Berries can trigger here (user's Berries that check HP)
The target thaws out if attack is a Fire-type attack, Scald, Steam Eruption
Knock Off, Rapid Spin, Bug Bite/Pluck, Thief/Covet effects (only if the user has not fainted, and even if the target fainted), Dragon Tail/Circle Throw, SmellingSalt/Wake-Up Slap effects (only if the user has not fainted)
Berries can trigger here (target's Berries that check HP)

After all hits of a successful attack have resolved (Ability and item effects mentioned below don't occur if the attacker has Sheer Force and the attack has an additional effect):
Color Change
Kee and Maranga Berries
Red Card (only if attacker and item's holder haven't fainted)
Eject Button message (only if item's holder hasn't fainted; if more than one target's Eject Button could trigger, only the first such target in turn order triggers).
Shell Bell/Life Orb (only if attacker hasn't fainted)
Eject Button effect (only if item's holder hasn't fainted). If effect happens, the U-turn message/effect doesn't happen.
U-turn message (only if attacker hasn't fainted)
Pickpocket (only if the Ability's bearer hasn't fainted)
U-turn effect (only if attacker hasn't fainted)

Your previous code didn't use .lastEffect. Can you move back to that for now?

I already did in https://github.com/Zarel/Pokemon-Showdown/pull/4045/commits/9701abef276cfcc85b1066042bee09a7d9bc2097

I think the code as it currently stands is an improvement upon the previous situation. @Marty-D, LGTM?

Magic Room isn't the only thing that stops item usage, see also https://github.com/Zarel/Pokemon-Showdown/blob/master/sim/pokemon.js#L375-L377.

Does a Pokemon under the effects of Embargo that switches out eat its berry? I think I've covered the other cases now, but I'm not sure about that one.

When it comes back in later, yeah. Not during the switch or anything.

This should cover everything then.

I think Klutz needs an onEnd also?

Oh, right. Did.

onAfterEnd -> onEnd for Klutz; onAfterEnd won't work for Abilities.
The last thing would be to include Berry Juice in all your checks since it's not actually a Berry.

During the onEnd event Klutz is still active so it will always end with pokemon.ignoreItem() being true. The onEnd as a whole in that regard doesn't work for anything here, which is why the onAfterEnd exist at all.

I just tested it; using your code, Super Fang to Gastro Acid didn't activate the Berry, but when I change it to onEnd it works.

I was testing with Super Fang to Simple Beam, which doesn't work with onEnd, but onAfterEnd does. I guess I could be doing both (or find out what's happening exactly), but just onEnd isn't enough.

Oh, weird. I'll have to look into that later. I assume ignoringAbility() is checked in hasAbility(), which lets Gastro Acid work.

From what I can tell, the Gastro Acid volatile is applied and then the ability onEnd runs. onEnd checks ignoringItem(), which does this.hasAbility('klutz'), and hasAbility checks for Gastro Acid.
On the other hand, when an ability ends in setAbility the onEnd event runs before the ability is replaced, so in this case Klutz is still active during the onEnd event.
Moving https://github.com/Zarel/Pokemon-Showdown/blob/master/sim/pokemon.js#L1032 to the line above the singleEvent('End') call seems like a viable alternative alongside only the onEnd for Klutz, but I'm not sure if that would cause any unintended side-effects (the event is already called based on the old ability and ability data)

Hmm, I sprinkled exceptions to AfterEnd all over singleEvent() and nothing helped. I'm missing something here; any ideas, @Zarel?

I'm confused. Can you describe the problem precisely?

Using onEnd for Klutz to make Berries activate immediately only works for the case where Gastro Acid stops Klutz from working, and onAfterEnd works for cases where Klutz gets replaced by some other Ability. How to make one of said events work for all cases?

I would add a new item event, StartUpdate and trigger it before Start for items, as well as after Klutz. I would probably trigger it from onAfterEnd for Klutz, and figure out why it's not being triggered by Gastro Acid, and fix that (it's probably just missing a manual trigger).
I'd also consider this sort of change outside the scope of a single pull request, and split it into two.

I believe I sorted out the Gastro Acid not properly triggering the onAfterEnd event for Klutz now, while still working for every other case too.

Oh, that's what I was missing, I never looked at Gastro Acid itself. Nice work, thanks! 👍


typo: agressive -> aggressive

@nschonni : Thanks for your contribution! The author, @, has been notified to review your proposed change.

Merging typo fix in placeholder content


Get started page not styled

The Github 'get started' page is one I actually never open up, and don't really care about. I just noticed this page was unstyled and thought you may had forgotten about it. Actually it's odd that the styles don't work there (on my computer), but I'm quickly guessing it has different color definitions, or maybe Stylus doesn't work on the specific link for some reason. If you can't reproduce, I can of course investigate it.
Thanks for all the hard work; I'm amazed with the theme's granularity!

Hi, I am now taking over this project. Is this still an issue?

It seems to be a different page than what I remembered, but the background is alright. I see now that I should have included a screenshot of my quick note after all, as we aren't able to check easily where it occurs. I'll close this issue. Good luck with styling github!


Coinratecap - Week - 2

📝 Weekly Report For TNB Projects
Basic Details
🧪 Name of the Project: [Coinratecap]
📅 Report Date: [02-26-2021]
📜 Team Work Report
✅ Accomplishments this week

[Coin Listing Methodology]
[Admin design]

🚩 Goals for next week

[Coin listing]
[Admin testing]

🔭 Long term goals

[Coin listing]
[Database structure to work alongside coin listing]

❗❓ Concerns
[ANSWER]
👨💻👩💻 Team Members Contribution Details
Coinratecap Project Weekly Task Report. Date: 2021-02-26
Kindly Submit the Following in this order
Name: Hussu
GitHub url: dhakal0kushal
Account number: 22d0f0047b572a6acb6615f7aae646b0b96ddc58bfd54ed2775f885baeba3d6a
Task:
Researched on the api endpoints to get the price and volume weighted price of a particular coin which was used in https://github.com/coinratecap/Backend/pull/52.
Made a list of API documentation that we will need in future.
Proof of work: https://github.com/coinratecap/Backend/issues/53#issuecomment-783788586, https://github.com/coinratecap/Backend/issues/53#issuecomment-786542496
Name: Seeker
Account Number: 58e74fbbe63a07fc001bda1025e7bb1d37ff567d7fcd735f2b41ae3222049795
GitHub url: nateshmbhat
Task :

added dummy data for exchange and exchangeApiStructure schema
added a service in coin listing that iterates through each entry in exchangeApiStructure collection and sends the request to the corresponding exchange.
added sample code to save the price and volume that is extracted into another separate collection for later processing (aggregation)
updated exchange schema to include api end point
Proof of work:
https://github.com/coinratecap/Backend/pull/52

Name: Paul Taiwo
Account Number: c6bc0d956d668ed2464684b352226f4df1f86ed213783ba82a97a2fe38252761
GitHub url: https://github.com/Paul-Taiwo
Task :  Integrate admin login endpoint
Proof of work: https://github.com/coinratecap/Admin/pull/4
Name: Sam Ayorinde
Account Number: 4bcd5759b5574bf76c37728d6794937b6880a37922e4aab859b4c76612d3a5b1
GitHub url: Sam-Devs
Task :  Created Educational videos on blockchain
Proof of work: Submitted to Francisca
Name: Noor Ahmad
Account Number:<PHONE_NUMBER>369c2dbdd10e65437d25605b05dfba10d10e2d65a6c24b88f307ef
GitHub url: https://github.com/NoorAhmd
Task :  a test demo for getting the price of a coin from provided URL
Proof of work: https://github.com/coinratecap/Backend/pull/55
Name: Ijeoma Okoh
Account Number:7db921e630fc3b2ee68896e45d0345760f32b1990496def33e45aee8de4c340e
GitHub url: https://github.com/aiijay
Task :  Managed the team, organized meetings and made sure tasks was done
Proof of work: Reported to Francisca
Name: Wisdom Ekpot
GitHub url: https://github.com/wisdom132
Account Number: 3c380835ed222ea92dcd727b0260276d9b898b60fadd80ad068e925f92e9e8cb
Task :  Created database seeder for demo coins listing, created issues for coin listing<bitcoin and Ethereum.
Proof of work: https://github.com/coinratecap/Backend/issues/53
Name: Florence
Account Number:  1db434fe29f300de9376a2d5be2f885cd099a95258f7622baad8b77cdfbf2f63
GitHub url: https://github.com/florae
Task :  Data entry
Proof of work: https://coinratecap.com
Name: Emibrown
Account Number: 1c3f2bf408fd1ba8fa57d946a9a9aa510968bb28b6402c32b08f3ca1762591a3
GitHub url: https://github.com/Emibrown
Task :  Created seed data on exchangeAPIStructure for Ethereum using the ftx and kraken API.
Proof of work: https://github.com/coinratecap/Backend/pull/55
🖹 Any Additional Notes (Optional)
[Your Notes]

I will add the account numbers, have just requested it  now

Added account number

approved


Add blackbox classifier functionality

Signed-off-by: robo-sq<EMAIL_ADDRESS>Implementation of blackbox classifier for issue Create new ART classifier for remote models #123
Demo in notebooks/classifier_blackbox.ipynb

@robo-sq Thank you very much for this exciting contribution! This enables ART to be used to test remote and deployed models.


"You need to install gyp in build/gyp first. See the README." on Linux

As I read in README.md this is only for Windows. What is the problem, how to fix?

OK. gyp is installed in my system. I don't need local copy. But let's do.
$ git clone https://chromium.googlesource.com/external/gyp.git build/gyp
$ ./gyp_uv.py -f ninja
Traceback (most recent call last):
  File "./gyp_uv.py", line 21, in <module>
    import gyp
  File "./build/gyp/pylib/gyp/__init__.py", line 37
    print '%s:%s:%d:%s %s' % (mode.upper(), os.path.basename(ctx[0]),
                         ^
SyntaxError: invalid syntax

Very bad.

gyp needs python 2, it doesn't work with python 3.  You can use autotools or Makefile.mingw if gyp doesn't work for you.

Thanks for support. 👎

You're the guy from #1145, aren't you?  If you keep up the negative attitude, just stay away, will you?

I believe can configure system to use python2. Thank you for idea.
You marked issue as invalid but it is not. Problem touches libuv. And I'll provide good answer if 'll find.
Your 1st answer sounds like second "keep away". Is this your usual manner to provide support and help people? Very angry.
I met a problem and need help. If you don't want solve the problem or don't know how to make it work please let others to answer, don't close quickly, don't hurry to mark it as rubbish.

You marked issue as invalid but it is not.

I tagged it invalid because it's not an issue with libuv itself but with gyp and your local setup.

Your 1st answer sounds like second "keep away".

I told you what the issue was and how to fix it.  I even gave you a workaround for when python 2 is not an option.  You seem to expect more but we're not a free support hotline.

I met a problem and need help [...] don't close quickly

I've told you before that the mailing list should be your first go-to resource.  We try to keep a high S/R ratio on this bug tracker and that means closing out reports that aren't actual bugs quickly.

Understand. I did not know about ratio. Sorry.

No problem.  Glad it's cleared up.

May be this can help somebody. Here described a way how to temporary change default python version.  This way should be the same for all *nixes.
I tried, but did not use this solution, because I switched to CMake.

i am also facing the same problem.
Any solution?
You need to install gyp in build/gyp first. See the README.

@mianaliasjad we dropped support for gyp back in early 2020.

Hi @bnoordhuis i was rebuilding a project from 2020 that's why i faced this other because back then gyp was used.
Can you guide me then how do i build the libuv for android arm64 now?


[NEW ALGORITHM] Add Rank-Balanced AVL Trees to AVL Trees Folder

Name:
Rank-Balanced AVL Trees
About:
Implement an advanced variant of AVL trees known as Rank-Balanced AVL Trees in the AVL trees folder. Rank-Balanced AVL Trees use a rank-based balancing criterion, where each node has an associated rank, and rotations are performed based on rank differences between nodes. This variant is useful for maintaining balance with potentially fewer rotations compared to traditional AVL trees.
Labels:
new algorithm, gssoc-ext, hacktoberfest, level1
Assignees:

[x] Contributor in GSSoC-ext
[x] Want to work on it

assigned

no update


fix: couple aspects of the jump attack

What was fixed?

Jump attack does not push back the minibosses or the final boss (they are suposed to be heavier and sturdier).

Since Mark couldn't be this weekend, I took the branch:

I have fixed a bug caused by using different areas instead of Bix's enemy detection. I leave all code because Alura also uses part of this alternative trigger and I don't have time to see if she suffers from the same bug. I'm gonna check it when I finish all the particle Systems of bix and the merging of lvl1.
I have divided the jump between falling and landing to damage enemies when we touch the ground and not in the air inmediatly when we have the input.
I have added visual effects for both moments.

GJ, the new effects for the jump look really good! The only issue I found is that, when the effect for breaking the ground when doing the jump attack is displayed, it rotates along with the player, isn't there any way to make it static or to disable its rotation?
BixJumping.mp4

Yes I see that problem too, I speak with graphics and the only solution is to create a game object apart from bix, I can do it in code and delete it when all the particles die. I'm on it


pb_decode_varint32 doesn't detect overflow properly?

It checks bitpos >= 32 but that value would have already overflowed.
At the beginning of the loop, bitpos = 7 indicates we're going to try to read the 2nd byte.  bitpos=14 indicates we're reading the 3rd.  bitpos = 21 indicates we're reading the 4th.  bitpos = 28 would indicate an overflow, not 32.

Hmm yeah, it is not an exact check currently. It would need to have extra check whether all the upper bits are zero in the last byte with bitpos = 28. I guess the speed penalty should be pretty small.

Fix released in nanopb-0.3.9.


Implement crypto_secretstream as IncrementalXChaCha20Poly1305

I am not very comfortable with references and pointers in C#, but I did my best to copy your patterns.
I'm going to leave some comments in the PR in some areas I'm unsure of.

This was a pleasant surprise! Thanks Jack! I will try to take a look today.

Thanks.  I'm looking into alternatives to libsodium-core and I think Geralt is the closest to what I'm looking for.  crypto_secretstream support may be the only thing missing for my use-case.

I'm looking into alternatives to libsodium-core and I think Geralt is the closest to what I'm looking for.

That's good to hear. Please feel free to share any feedback on the API and documentation. I need to rewrite all the tests to try and reduce repetition at some point, and I'd like to change the BLAKE2b stream implementation so it uses spans.

crypto_secretstream support may be the only thing missing for my use-case.

Yes, I think it makes sense now there's IncrementalBLAKE2b and Poly1305.

@samuel-lucas6 I implemented most of your feedback and resolved some compiler messages I hadn't seen before.  I'll leave it up to you to resolve any open conversations.

Appreciate it. Last thing is could you merge the no associated data and associated data methods for Pull like you did for Push? Not a fan of the GitHub editor. Then I'll merge.

The overload for Pull is gone.


Pools Tabs shouldn't stay in 0.

To replicate the error make:

External Incentives pools
Search "huahua". This will let an empty list of pools.
Click on any of the side arrows.

Delete the searched string ( in this case "huahua")
Now you're in the tab 0/4

This can be solved by keeping updated the tab number to 0 or 1 depending on if at least 1 pool exists.

Hoping for this to be deleted entirely with completion of #1175


[Bug]: Can not list the result if the input has -

Bug Description
Hello, I found that no result return if my input has symbol (e.g. -), even though the bibtex entry exists. See the screenshot (I change ct to zt for this workflow).
Is it possible to fix it? Thank you.
Best regards,
Viming
Relevant Screenshot

To Reproduce
No response
Debugging Log
No response

Sample BibTeX Entries
@incollection{Okamoto_1998_new,
  title = {A new public-key cryptosystem as secure as factoring},
  booktitle = {Advances in Cryptology \textemdash{} EUROCRYPT'98},
  author = {Okamoto, Tatsuaki and Uchiyama, Shigenori},
  year = {1998},
  volume = {1403},
  pages = {308--318},
  doi = {10.1007/BFb0054135},
  language = {en}
}

Buffer Files
No response
Workflow Configuration
No response
Checklist

[X] I have updated to the latest version of this workflow.
[X] I have tried manually reloading the buffer (Alfred keyword scp, select Force Buffer Reload).

thx, fixed in the latest release 10.0.14


Fix/lobby popup UI

Cherry pick to fix the loosing Popup related commits caused by
Merge branch 'feature/profile' into feature/lobby-ui 790511e

Befor Fix

## After Fix

How to pass the SonarCloud quality gate? The testing coverage keeps failing since we don't test our frontend?


DELETE FROM command wipes index records [moved]

This is Issue 1130 moved from a Google Code project.
Added by 2012-10-22T12:11:48.000Z by<EMAIL_ADDRESS>Please review that bug for more context and additional comments, but update this bug.
Original labels: Type-Defect, Priority-Medium, Release
Original description
OrientDB release:
orientdb-graphed-1.3.0-SNAPSHOT-r7048

System / Platform details:
Ubuntu 12.04 64bit
java version "1.7.0_07"
Java(TM) SE Runtime Environment (build 1.7.0_07-b10)
Java HotSpot(TM) 64-Bit Server VM (build 23.3-b01, mixed mode)

What steps will reproduce the problem:

1) prepare OrientDB without changing any default configuration:

mkdir ~/servers/orient
cd ~/servers/orient
svn checkout http://orient.googlecode.com/svn/trunk/ orient-read-only

# use your revision ID which you can see after svn checkout
ORIENT_REV=7048

cd orient-read-only

# regular version
ant clean
ant install

# graphed version
ant clean
ant installg

cd ../releases/

mv orientdb-1.3.0-SNAPSHOT orientdb-1.3.0-SNAPSHOT-r$ORIENT_REV
mv orientdb-graphed-1.3.0-SNAPSHOT orientdb-graphed-1.3.0-SNAPSHOT-r$ORIENT_REV

cd orientdb-graphed-1.3.0-SNAPSHOT-r$ORIENT_REV

chmod +x bin/*.sh

touch log/orient-server.log

cd ./bin/

# start the server
./server.sh

# open new console and go the same "bin" directory

# start the console
./console.sh

2) using the running console perform these operations:

# create database
# replace sample path with your path
create database local:/some/local/path/databases/bugs/development admin admin local

# change database's type
ALTER DATABASE TYPE graph

# disconnect from the database
disconnect

# establish connection to the database again
# replace sample path with your path
connect local:/some/local/path/databases/bugs/development admin admin

3) copy the content of the attached file "wiped_indexes_issue_queries.sql" and paste it into the console

What is the expected output / result:

After each run of the "info" command we should see more and more index records being created inside different indexes

What do you see instead:

After each run of the "info" command (except last one) we see that the previously created index records are being removed, except those which are being created after the closest "delete from" command (in relation to our "info" command) by using "insert" commands

After the last "info" command we see that there are no index records at all.

For a full log of console commands and responses , please see the attached file "wiped_indexes_issue_console_output.txt"

Old issue, this kind of problems was fixed long time ago
Closing
Thanks
Luigi


Refactors RootPeerDNS.hs and DNS providers to accomodate IOSim

First iteration complete, still needs some tweaks and test suite

This PR tackles #3102
https://github.com/input-output-hk/ouroboros-network/pull/3125/commits/b6633cf884ff199a6dcff6c5aae8f59967862bc7#diff-522b3db99d6cdfd34b04e2b1cdd7c1ccb832dfc5c454aaf82e53de61122884adR128
This tackles #3104


Error When Trying to Delete Other People's Messages

Just started using it now, whenever I would try to put in the User ID of the person's DM's I'm trying to delete, it goes through, did everything I need to do, then this error pops up whenever I press Start.
Error deleting message, API responded with status 403!	{"message":"Cannot execute action on a DM channel","code":50003}
No clue what I'm doing wrong, please help.

Hey hi @victornpb, can MEE6 or Dyno delete messages posted by others on group DMs?

Hey hi victornpb, can MEE6 or Dyno delete messages posted by others on group DMs?

I don't think that'd be possible.

You can't invite bots to group DMs as far as I know.
In DMs one can only delete their own messages.

Hey hi victornpb, can MEE6 or Dyno delete messages posted by others on group DMs?

I don't think that'd be possible.

You can't invite bots to group DMs as far as I know.
In DMs one can only delete their own messages.

Would you be willing to add a field that deletes a specific user's (by ID) messages? Bots are unable to delete messages older than 14 days and sometimes bots like Dyno stop responding with mass messages. For wanting to delete all of a bot's messages but not delete the channel as a result (as tasking as it sounds) would be amazing. This tool would be useful for such moderation purposes since the bots can't do the job for me.

Would you be willing to add a field that deletes a specific user's (by ID) messages? Bots are unable to delete messages older than 14 days and sometimes bots like Dyno stop responding with mass messages. For wanting to delete all of a bot's messages but not delete the channel as a result (as tasking as it sounds) would be amazing. This tool would be useful for such moderation purposes since the bots can't do the job for me.

Sorry if I don't understand, but doesn't this already support that?


feat: Wire useSearchParams to url rewriting

This is a follow up on feat: add URL transformation hook
My implementation of URL rewriting was working great until I tried using useSearchParams setter which would update the browser's URL to the route:
From: /foo.html?bar=baz updating bar to biz
Result: /product/123?bar=biz
Expected: /foo.html?bar=biz
So I needed  to wire it, but the logic is reversed, compared to the previous PR.

transformUrl transforms incoming requests pathname to router's route.
for useSearchParams setter we the need the opposite, from a route displaying the rewritten url in the browser.

So to make this reversed behavior clearer, I renamed transformUrl (with a deprecation notice, even though I'm pretty sure I'm the only one using rewrites :-) ) to rewritePathToRoute and introduced rewriteRouteToPath.
I plan to write some doc at some point, but I might be still missing some rewrite wiring here and there.
And you implement it like this:
const pathnameToRoute = (url: string) => {
    const u = new URL(url, "http://localhost/");
    return rewriteMap[u.pathname] ? rewriteMap[u.pathname] + u.search : url;
  };

  const routeToPathname = (url: string) => {
    const u = new URL(url, "http://localhost/");
    if (Object.values(rewriteMap).includes(url)) {
      const swapped = swapKeyValue(rewriteMap);
      return swapped[u.pathname] + u.search;
    }
    return url;
  };

  return (
    <Router
      explicitLinks={true}
      preload={false}
      rewritePathToName={pathnameToRoute}
      rewriteRouteToPath={routeToPathname}
      root={(props) => (
        <Suspense>
          <RootContexts>{props.children}</RootContexts>
        </Suspense>
      )}
      rootPreload={() => {
        getMenuMain();
        getMenuFooter();
        getSettings();
      }}
    >
      <FileRoutes />
    </Router>
  );

Let me know what you think!
Thank you!
Jérémy

Hmm.. I wonder if this is necessary. I was hoping we'd only need the transform in one direction. I see it is because we call navigate and we pass in the path there but if there was a way to communicate the params change without including the path I think it would be unnecessary. Of course if there were the whole setter side of useSearchParams might be a questionable API. I wasn't the originator of this API and in my previous routers you'd just pass an object to navigate if you wanted to update the search params.
I feel like this is an area that could be streamlined. Given the get is just a projection of location and the set could be handled by navigate. There might be reasoning I'm missing but my gut is we should look at alternatives.

ok, I agree with you.
I just had a look, I'll create a new PR during the day with way more simple.


Support truthy values in boolean unit options

The Global option is lower-cased then checked directly against true [0], while systemd units accept "truthy" values (yes, on, 1, t) [1]. fleet should follow suit and accept the same truthy values when parsing boolean options.
[0] https://github.com/coreos/fleet/blob/d76e5ade2f285ff9ec3d0fbe57fc17f5af94d61a/job/job.go#L116
[1] http://www.freedesktop.org/software/systemd/man/systemd.unit.html#Description

Closed via https://github.com/coreos/fleet/pull/1574/.


update changelog for Key Vault Keys beta3

Wait to merge https://github.com/Azure/azure-sdk-for-cpp/pull/2430 before this one :)

@vhvb1989 and @danieljurek what is this warning/failure?
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=936386&view=logs&j=71bf1c24-ec86-528d-72b2-902c92392b38&t=6c49089d-3e50-514c-b9ae-3064e1a8d6de&l=20
[debug]Extracting entries from [D:\a\_work\1/packages/azure-security-keyvault-common/CHANGELOG.md].
Found the following change log entry for version '4.0.0-beta.3' in [D:\a\_work\1/packages/azure-security-keyvault-common/CHANGELOG.md].
-----
## 4.0.0-beta.3 (Unreleased)

-----
##[error]Entry has no release date set. Please ensure to set a release date with format 'yyyy-MM-dd'.
##[error]PowerShell exited with code '1'.

https://github.com/Azure/azure-sdk-for-cpp/commits/master


Directory based kubectl apply when there are more than 1k resources

Deploys with thousands of resources can fail with Argument list too long - kubectl (Errno::E2BIG). This happens because we render each resource to its own file and then pass all of the files to kubectl apply via the -f` flag. This pr detects when there are greater than 1k resources copies them to a temp directory and passes only the directory to kubectl apply.
cc: @dwradcliffe

Changes are in. Would like feedback on the ensure block vs wrapping almost the entire apply_all method in a block.

Would like feedback on the ensure block vs wrapping almost the entire apply_all method in a block.

I don't have a strong preference since the nesting doesn't get too crazy.


future of luxcore?

Hi,
Seeing a decrease of traffic on github and the forum seems locked, is the project about to be abandoned?
Or is it just an impression?

The answer to my question lies here:


Implement Table::changePrimaryKey and Table::changeComment

Addresses requests for being able to modify the primary key, as well as the table comment and other table options. Implemented and tested for all adapters.
I decided to implement change() instead of changePrimaryKey() and changeComment() because I thought it would be useful to have the same interface as when creating the table, with the same available options. Check migrations.rst for an explanation of how it works. Of course you may disagree, and maybe it should be reworked -- all feedback is welcome.
Affected issues:
https://github.com/cakephp/phinx/issues/335
https://github.com/cakephp/phinx/issues/802

I appreciate the feedback!
And I look forward to contributing more, since I already have my eye on several other issues from the pile, that I'll get around to whenever I get enough free time :)

@lorenzo Ping!


Off-Centre radio buttons when zooming

As you zoom in and out, our custom-styled radio button become off-centred.
The CSS for these is a touch over-engineered, and can be simplified with just a ::before pseudo element.
Here is a very nice guide:
https://moderncss.dev/pure-css-custom-styled-radio-buttons/

@chriswilty Ta-da

Doro tried it, didn't work 😢


[Question][DevOps]  Deploy devlake with managed GCP Cloud database service (with SSL enabled)

Question
Hi currently we are trying to deploy devlake in our dev env. We are trying to integrate devlake using helm chart with managed GCP Cloud database service.
Our DB is SSL enabled and we don't see how to pass the SSL certs in the DB conn string part of helm chart.
Could you please let us know what additional params we need to use for using SSL in additional to below params we pass to helm command.
- `mysql.useExternal`: this should be `true`
- `mysql.externalServer`: use your RDS instance's IP address or domain name.
- `mysql.externalPort`: use your RDS instance's database port.
- `mysql.username`: use your `username` for access RDS instance's DB
- `mysql.password`: use your `password` for access RDS instance's DB
- `mysql.database`: use your RDS instance's DB name, you may need to create a database first with `CREATE DATABASE <DB name>;`

Screenshots
If applicable, add screenshots to help explain.
Additional context
Add any other context here.

Per above suggestion when deployed woth above config it says unknown value tls=custom for tls option.
Tried with tls=skip-verify which gave access denied error.
  --from-literal=DB_URL='mysql://<your-username>:<your-password>@<dbserver>:<port>/<database>?charset=utf8mb4&parseTime=True&tls=custom' \

Now we are seeing below error with certs installed
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x1394498]

panic: runtime error: invalid memory address or nil pointer dereference

@dosu
Hi We are seeing below error when deploying with SSL
"Wraps: (3) x509: cannot validate certificate for X.X.X.X(ip) because it doesn't contain any IP SANs"

Hi Yes we are already mounting the required certs in to pod in our helm chart config
volumeMounts:
            - name: newcerts
              mountPath: /etc/ssl/certs

@dosu
We are using clouds sql from GCP and in the generated ssl certs we have CN=Google Cloud SQL Server CA but missing SAN name in it.
But lake is trying to verify SAN in the certificates.
Any workarounds.

Hi @dosu
I have executed above suggested steps but still I am getting same error
Also generated a rootCA.crt using the DB dns and placed below certs in/etc/ssl/certs folder.
devlake-client-key.pem
devlake-client-cert.pem
devlake-server-ca.pem
rootCA.crt

Also places rootCA.crt under /usr/local/share/ca-certificates in lake container
Wraps: (3) x509: cannot validate certificate for <IP_ADDRESS> because it doesn't contain any IP SANs

Also in the init container I am seeing below error.
ERROR 2024-08-23T06:13:29.444710841Z [resource.labels.containerName: update-ca-certificates] sh: update-ca-certificates: not found

@dosu
Per your if we do below step, do we still need to create the devlake-mysql-auth with DB url and SSL certs as you have mentioned here https://github.com/apache/incubator-devlake/issues/7928#issuecomment-2300787106
`lake:
initContainers:
- name: update-ca-certificates
image: debian:bullseye
command: ["sh", "-c", "apt-get update && apt-get install -y ca-certificates && cp /mnt/rootCA.crt /usr/local/share/ca-certificates/rootCA.crt && update-ca-certificates"]
volumeMounts:
- name: root-ca-cert
mountPath: /mnt
volumes:
- name: root-ca-cert
secret:
secretName: root-ca-cert
`

In that case its still not working with above suggestion I was able to get the both below init containers up and running.

update-ca-certificates
database-wait

But after that the lake container is still failing with
Wraps: (3) x509: cannot validate certificate for <IP_ADDRESS> because it doesn't contain any IP SANs

It appears that MySQL SSL connections are currently not supported. For now, I recommend trying a connection without an SSL certificate.

Thank you @klesh yes mysql SSL were not supported, but we were able to resolve the issue by connecting using cloudsql-auth-proxy.

Hi @dosu
How to add new plugins to devlake using helm chart.

hi @dosu
Can we integrate devlake with OKTA SSO config....?

@dosu
How do I get value of OAUTH2_PROXY_COOKIE_SECRET to configure oauth2-proxy....?

@dosu
Per suggestion I have added the oauth2-proxy, but still I get basic auth username/password pop up coming up.
Do we need to delete the pod resources and recreate from scratch...?

@dosu
Did configuration per above message. I can see ingress got added but oauth2-proxy pod is not coming up.
I can only see 2 pods dev-lake and UI.
NAME                            READY   STATUS    RESTARTS        AGE
devlake-lake-   2/2     Running   1 (4m15s ago)   4m16s
devlake-ui    1/1     Running   0               4m16s

Hi @dosu
We are have forked https://github.com/apache/incubator-devlake repo and developed a new internal custom plugin.
Now how can we generate a new devlake image to include new plugin and deploy it in our own cloud using helm chart.
Thanks in advance.

Thank you @dosu
Can you also send me details on how to generate a image just for plugin by itself and have the plugin image in helm values file.

My helm chart looks like below
lake:
initContainers:
- name: update-ca-certificates
image: debian:bullseye
command: ["sh", "-c", "apt-get update && apt-get install -y ca-certificates && cp /mnt/root-ca-cert /usr/local/share/ca-certificates/root-ca-cert && update-ca-certificates"]
volumeMounts:
- name: root-ca-cert
mountPath: /mnt
volumes:
- name: root-ca-cert
secret:
secretName: ca-cert
envs:
API_TIMEOUT: "120s"
API_RETRY: "3"
API_REQUESTS_PER_HOUR: "10000"
PIPELINE_MAX_PARALLEL: "1"
IN_SECURE_SKIP_VERIFY: "true"
here ca-cert is the SOPS secret for ca-cert.crt data. I am trying to use SSL integration for Azure MySQL.
Error:
Defaulted container "lake" out of: lake, waiting-database-ready (init)
time="2024-10-07T08:12:36Z" level=info msg="no [.env] file, devlake will read configuration from environment, please make sure you have set correct environment variable."
Version:  v1.0.1@e061ef2
panic: open /usr/local/share/ca-certificates/root-ca-cert: no such file or directory
Wraps: (2) open /usr/local/share/ca-certificates/root-ca-cert
Wraps: (3) no such file or directory
Error types: (1) *hintdetail.withDetail (2) *fs.PathError (3) syscall.Errno


[C# Phi-2] System.DllNotFoundException: Unable to load shared library 'onnxruntime-genai' or one of its dependencies.

When trying out a samples that uses Phi in .NET, I get the following error.
Unhandled exception. System.DllNotFoundException: Unable to load shared library 'onnxruntime-genai' or one of its dependencies. In order to help diagnose loading problems, consider setting the LD_DEBUG environment variable: libonnxruntime-genai: cannot open shared object file: No such file or directory
Full Stack Trace:
Hello From Phi-2
Unhandled exception. System.DllNotFoundException: Unable to load shared library 'onnxruntime-genai' or one of its dependencies. In order to help diagnose loading problems, consider setting the LD_DEBUG environment variable: libonnxruntime-genai: cannot open shared object file: No such file or directory
   at Microsoft.ML.OnnxRuntimeGenAI.NativeMethods.OgaCreateModel(Byte[] configPath, IntPtr& model)
   at Microsoft.ML.OnnxRuntimeGenAI.Model..ctor(String modelPath)
   at Program.<Main>$(String[] args) in /workspaces/ort-genai-phi2/ORTGenAIPhi/Program.cs:line 8

Repo: https://github.com/luisquintanilla/ort-genai-phi2

@baijumeswani I am on Windows. However, I'm using devcontainers.
Here is my configuration:
https://github.com/luisquintanilla/ort-genai-phi2/blob/main/.devcontainer.json
System Info:

Debian
Python 3.10
.NET 6
Intel CPU

Let me know if you need anything else.

We have not published the nuget binaries for linux yet. This will be supported in future releases but is not supported right now.

I got the same error on windows, with nuget packages.
 var a = new Microsoft.ML.OnnxRuntimeGenAI.Model(@"C:\dev\hugginfacecli\multilingual-e5-large\onnx");

Unhandled exception. System.DllNotFoundException: Unable to load DLL 'onnxruntime-genai' or one of its dependencies: The specified module could not be found. (0x8007007E)
   at Microsoft.ML.OnnxRuntimeGenAI.NativeMethods.OgaCreateModel(Byte[] configPath, IntPtr& model)
   at Microsoft.ML.OnnxRuntimeGenAI.Model..ctor(String modelPath)
   at TestLocalEmbed.Program.Main(String[] args) in C:\dev\TestLocalEmbed\TestLocalEmbed\Program.cs:line 149
   at TestLocalEmbed.Program.<Main>(String[] args)

I also encountered the same error when loading phi-3, but it occurred during cuda and was normal on CPU

I also encountered the same error when loading phi-3, but it occurred during cuda and was normal on CPU

For CUDA, make sure your CUDA binaries are discoverable by adding them to your PATH.
For example, for me the cuda binaries are located in: C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v11.8\bin and this path should be added to your PATH env variable.

at Microsoft.ML.OnnxRuntimeGenAI.NativeMethods.OgaCreateModel(Byte[] configPath, IntPtr& model)
at Microsoft.ML.OnnxRuntimeGenAI.Model..ctor(String modelPath)
at OnnxGenAI.Program.Main(String[] args) in C:\Users\rhenr\source\repos\OnnxGenAI\OnnxGenAI\Program.cs:line 42
i don't have nvidia, just want to run on CPU

I got the same error experimenting on a Surface Studio Laptop 2 on Windows, but discovered I could get past this error by ensuring I had both references:
<PackageReference Include="Microsoft.ML.OnnxRuntimeGenAI" Version="0.2.0-rc4" />
<PackageReference Include="Microsoft.ML.OnnxRuntimeGenAI.Cuda" Version="0.2.0-rc4" />

Still running into issues on Windows with version 0.3.0-rc2.
Here's the error I'm getting.
C:\a\_work\1\s\onnxruntime\core\session\provider_bridge_ort.cc:1426 onnxruntime::ProviderLibrary::Get [ONNXRuntimeError] : 1 : FAIL : LoadLibrary failed with error 126 "" when trying to load "C:\Dev\ort-genai-phi2\ORTGenAIPhi\bin\Debug\net8.0\runtimes\win-x64\native\onnxruntime_providers_cuda.dll"

I'm using the Phi-3 model. https://huggingface.co/microsoft/Phi-3-mini-4k-instruct
With FP16 CUDA

We have not published the nuget binaries for linux yet. This will be supported in future releases but is not supported right now.

That would be perfect, especially for the Cuda version.

Just to make sure - there's currently no way of running apps using onnxruntime-genai on Linux purely with Nuget packages?
I've been trying to run the example app from https://github.com/microsoft/Phi-3CookBook/tree/main/md/07.Labs/Csharp/src since yesterday, I thought I had some issues on a system level.
If I read the support matrix correctly, it suggests Linux is supported.

0.4.0-rc1 NuGet package is out with Linux-x64 support. Feel free to check it out: https://www.nuget.org/packages/Microsoft.ML.OnnxRuntimeGenAI/0.4.0-rc1
I'll close this for now.

0.4.0-rc1 NuGet package is out with Linux-x64 support. Feel free to check it out: https://www.nuget.org/packages/Microsoft.ML.OnnxRuntimeGenAI/0.4.0-rc1
I'll close this for now.

I'm seeing dependencies on packages I can't resolve via nuget.org

I'm seeing dependencies on packages I can't resolve via nuget.org

Please add the Ort-Nightly index as a source. You can find more instructions here: https://onnxruntime.ai/docs/genai/howto/install.html#pre-requisites


[Build] Add code coverage service and eliminate dead code

Use the code coverage script from https://gitlab.com/jamesqo/PolynomialReducer as a starting point.

Of course, this will first entail setting up unit tests for the app, which I don't know how to do yet.


Add a contributors.md to nx-guides

Mirror it with what we do in networkx/networkx

Should we also add an explicit "Authors field" in the notebooks ? Can this be done via the sphinx theme?

Should we also add an explicit "Authors field" in the notebooks ? Can this be done via the sphinx theme?

This can be done, but may introduce an extra layer of unnecessary complexity... I think the number and degree of contribution is more than adequately reflected in the git history, and it'd be very easy for an "author" field to fall out of sync with the actual contribution history for any given tutorial.
In all, I'm +1 for a nx-style contributors.md doc, and -1 on a per-file "authors" tracker.

Hey,  I would like to do this. Should I create a PR?

I know I said I was +1 in my previous comment, but honestly even that feels like overkill. GitHub already has a really nice interface for this info, so it seems silly to duplicate it. Maybe we can just link to the GitHub contribution page for nx-guides from the main page? Doing what NetworkX does (i.e. maintaining a contributor's list manually) seems not worth the effort here.

Ah wait, I think I meant CONTRIBUTING.md like https://github.com/networkx/networkx/blob/main/CONTRIBUTING.rst 😅
Currently there is no information about how to submit the myst notebooks to the this repo.

Got it. Thanks @MridulS

@MridulS I am thinking of creating another CONTRIBUTING.md for this repository. But I have some queries -

Is the project only meant to be run on Ubuntu & MacOS platforms? Symlink is being used which doesn't work on Windows. Do we need to make some changes there for this project to work on Windows?
Earlier, after creating a pull request, I faced some issues in the CI workflows. Instead of repeatedly making changes in the code and waiting for the workflows to pass, should I include a guide on the workflows so the users can easily check the build process on their local systems?

Apart from that I will include the steps to change from jupyter notebook to markdown using jupytext, maintaining the correct file structure, etc

@inomag you're right that we definitely need a contributor guide... it probably makes sense for one of us to take the first crack at it since we should (at least in principle) remember how we had structured the project :). I'll try to prioritize this and will definitely ping you to test drive the guide and ask you for feedback!


Formatter is not concurrency-safe

Due to this, output may end up completely garbled. Either synchronisation or a complete rewrite is needed (probably both).

Fixed in 75a011d5


chore: 更新 CONTRIBUTOR.md

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add syntax highlighting to code examples in docs

Syntax highlighting to code examples (especially in tutorials) would help with readability / helping beginners see that it's not just line noise.

We are already doing syntax highlighting.
Feel free to re-open if you have a specific example that isn't getting syntax highlighting.

It must have broken everywhere then.
You can see my UA info here: http://www.whatsmyua.com/#Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/55.0.2883.87 Safari/537.36
And you can see a screenshot below.

I double this issue. I thought highlighting was turned off for some reason. Latest firefox.
I do see highlighting classes in html, but they are still not colored. It used to work for me some time ago.

Yeah, I checked earlier, the highlighting transformation appears to be happening in most places but it isn't loading the stylesheet I think.

samcv noted that one of the nodepm packages we depend on changed all their class names without warning. b1d51f1 was supposed to help, but I don't see any changes on the site yet.

see also 0193f3e4dc and 9525968e78

docs.perl6.org is fixed now as of https://github.com/perl6/doc/commit/9525968e78ac239b69a975d8ff56fa2813aabe68
Closing!

It looks like some are fixed depending on what css is:

class="editor editor-colors" works, but class="pod-block-code" does not.

That is not the same problem. That is related to what is in the pod6 files the docs are created from. Anything tagged as pod-block-code usually means that's what the block was inside tho doc file
'=begin code :allowthat is on the page you showed the screenshot of. If it has:allow` then it renders as Pod not as perl 6 code

@samcv Should it be left open as a "someone can feel free to change this so it's consistent" type of task?

I think there's another ticket for that, not sure which

But it's not the original issue, which is why I closed it. It's a separate problem

If you can't find one, please open an issue for it.
Make sure to paste this into the issue:
The pod6 file has :allow in the code designation:
=begin code :allow<B L> that is on the page you showed the screenshot of. If it has :allow then it renders as Pod not as perl 6 code
Thanks

@WildYorkies I have opened an issue here for this https://github.com/perl6/doc/issues/1164
I also wrote my proposed solution for this, and a more detailed explanation of why this currently happens.


Read folderfiles

This PR resolves #(put issue number here).
PR Checklist
Please ensure that you have done the following:

[ ] PR in from a fork off your branch. Do not PR from <your_username>:master, but rather from <your_username>:<branch_name>.
[ ] If you're not on the contributors list, add yourself to AUTHORS.rst.

Quick Check
To do a very quick check that everything is correct, follow these steps below:

[ ] Run the command make check from pyjanitor's top-level directory. This will automatically run:

black formatting
pycodestyle checkin
running the test suite
docs build

Code Changes
If you are adding code changes, please ensure the following:

[ ] Ensure that you have added tests.
[ ] Run all tests ($ pytest .) locally on your machine.

[ ] Check to ensure that test coverage covers the lines of code that you have added.
[ ] Ensure that all tests pass.

Documentation Changes
If you are adding documentation changes, please ensure the following:

[ ] Build the docs locally.
[ ] View the docs to check that it renders correctly.

PR Description
Please describe the changes proposed in the pull request:

Relevant Reviewers
Please tag maintainers to review.

@ericmjl

@dave-frazzetto  @ericmjl I was looking through the functions and saw read_csvs() and had a question about its usage. Is its intended usage is to supply a path to a file: "path/to/file.csv" or a path to a folder containing .csvs with or without wildcards: "path/to/folder/", "path/to/file*.csv".
Is that right? Would there also be usage where a list of paths is passed, and creates a combined dataframe?

@HectorM14 yes, you are correct - we pass a directory path into read_csvs, and then it automagically reads all CSVs inside that directory as a single DataFrame.

Would there also be usage where a list of paths is passed, and creates a combined dataframe?

This I think isn't implemented here, but would definitely be within scope for a PR!

Okay, got it. I think that's a PR I could do in the near future (unless anyone else does it first).
In the meantime, does it make sense to move its tests to an io folder? It would just be the single file at the moment, but it would follow the structure of the other tests.

does it make sense to move its tests to an io folder? It would just be the single file at the moment, but it would follow the structure of the other tests.

It can work both ways! I would also definitely accept a PR to move it into an io/ directory if you did it, but no pressure here :smile".


Use the exact same "weights" as cmake

The weights need to be of the exact same version, extracted by the exact same way.
It seems that this is currently not the case.
Workaround
If applicable, use a copy the extracted weights files and fetch them from the bazel build.

https://github.com/abebeos/weights

The weights are identical confirmed by me and @parvit


At debug pausing, "this" in A.constructor() shows undefined when target=ES6 and A extends B and A.constructor() has lambda function.

VSCode Version: 1.40.1
OS Version: Windows 10

Steps to Reproduce:

tsconfig.json
{
"compilerOptions": {
"target": "es6",
"module": "es6",
"sourceMap": true,
"strict": true,
"esModuleInterop": true
}
}

main.ts
class SUPER{}

class main extends SUPER
{
constructor()
{
super();
debugger;
let aa = this;
//**
let bb = ()=>{
console.log(this);
};
//*/
}
}
new main();

Does this issue occur when all extensions are disabled?: Yes

What node version? Works for me.

It shows 'undefined' in the variables window, but I see the same behavior in chrome devtools.
Note that there are lots of oddities around how 'this' work in a debugger, due to optimizations that the runtime makes, and there's probably nothing I can improve here. But let me know if you see a difference between vscode and chrome devtools.

C:\Users\ys>node -v
v10.15.3
(1) A extends B
(2) ES6
(3) lambda have ref to this, likes
let bb = ()=>{
console.log(this);   //<----- this appear here
};
I need your full test code.

Here's full nodejs proj
test_this_undefined.zip

class SUPER {
}
class main extends SUPER {
constructor() {
super();
debugger;
let aa = this;
/**
let bb = () => {
console.log(this);  //ref "this" here, bug appear
};
//
let bb = ()=>{
console.log("no this here");   //bug gone
};
///
}
}
new main();

With that version, I see the same, and the same in chrome devtools, so I can't do anything to improve this. But it looks like it gets better in more recent versions of node.


HAPI FHIR JAX-RS server endpoints always returning prettified JSON

Description
The endpoints in our server annotated with the HAPI FHIR JAX-RS annotations documented here always return prettified JSON (added whitespace + newlines) despite the HAPI FHIR documentation stating that pretty print of JSON should be disabled by default.
We are using DSTU3.
Package: package ca.uhn.fhir.rest.annotation;
To Reproduce
Steps to reproduce the behavior:

Implement a REST endpoint and annotate it with @READ from this library (it also occurs for @UPDATE and @CREATE):
Package: package ca.uhn.fhir.rest.annotation;

Call the endpoint and return a FHIR Domain resource. I used Observation DSTU3.

Expected behavior
A response with a body in JSON format in one long line non-prettified, optimized for machine-to-machine communication.
Observed behavior
Response body is prettified containing newlines and whitespace.
Screenshots
Below screenshot shows an example of the issue:

The same endpoint implemented without HAPI FHIR like below screenshot, correctly produces a non-prettified response:

Environment (please complete the following information):

HAPI FHIR Version: 6.2.5
Spring Boot version: 2.7.15
OS: MAC

Additional context
The spring.jackson.serialization.indent_output is not defined in our project, nor are any other jackson objectmapper properties. As far as i can tell, then the default HAPI FHIR parser, prettifies the output despite documentation stating otherwise. And i cannot figure out how to disable it.

So for us the solution was to override the isDefaultPrettyPrint method in our class that extends the AbstractJaxRsResourceProvider, making it return false instead of true.
The thing is that AbstractJaxRsResourceProvider extends AbstractJaxRsProvider which by default returns true when isDefaultPrettyPrint is called.
Given that the HAPI FHIR documentation here states that it is false by default and Spring also has this disabled by default, then this should probably be changed to return false instead of true.


[1.20.1] EssentialGUI crashes when hovering over End Remastered ender eyes.

Mod Name
EssentialGUI
Mod Homepage
https://modrinth.com/mod/essentialgui
Minecraft version
1.20.1
Describe the bug
When hovering over any ender eye from End Remastered, the game crashes.
Steps to reproduce

Open the game.
Load a world.
Obtain any of the unique ender eyes.
Crash.

Logs
https://mclo.gs/mD3qOfQ
Additional context
No response

This is an unfortunate interactions between the two mods. End Remastered marks its ender eye items as damageable, but doesn't set their max damage. As a result, essential gui will try to divide by zero thinking the item's max damage is always non-zero if isDamageable returns true.
I'd recommend forwarding this issue to essentialgui so that they can check the max damage before dividing and avoid the crash.


TempFileMunge cannot work correctly on windows with ARMCC(a toolchain of MDK)

This issue was originally created at: 2013-03-02 22:00:47.
This issue was reported by: prife.
prife said at 2013-03-02 22:00:47

windows7  32bit
scons 2.2.0
HI, all
I use scons to build sources, and the C compiler is the armcc (which is from
MDK).
In most cases, scons works very well, but if when there are too many objects in
the final command line, then I will get an error.
like this：

armlink -o rtthread-stm32.axf --device DARMSTM --info sizes --info totals --
info unused --info veneers --list rtthread-stm32.map --scatter stm32_rom.sct --
libpath D:/Keil/ARM/RV31/LIB --keep _fsym* --keep _vsym* build\application.o
build\startup.o build\board.o build\stm32f10x_it.o build\rtc.o
........................(many objs)
The Command line is too long.
scons: *** [rtthread-stm32.axf] Error 1

because the armcc has a limited command line length.  It provides an option
--via=file
This option reads an additional list of input filenames and linker options from
file.
so, I want to get  the final command line, the write some  to an temp files, and
use this --via option, then I can solve this issue.
so I add the following code in my SConsruct

if rtconfig.PLATFORM == 'armcc':
env["TEMPFILE"] = SCons.Platform.TempFileMunge
env["LINKCOM"] = "$LINK -o $TARGET $LINKFLAGS --via
${TEMPFILE('$SOURCES')}"
env["TEMPFILEPREFIX"] = '  '      # arm tool chain

Then I recompile with scons -j4  in CMD
armlink -o rtthread-stm32.axf --device DARMSTM --info sizes --info totals --info
unused --info veneers --list rtthread-stm32.map --scatter stm32_rom.sct --libpa
th D:/Keil/ARM/RV31/LIB --keep _fsym* --keep vsym* --via build\application.
o @c:\users\admini~1\appdata\local\temp\tmp5r9pd.lnk
Using tempfile c:\users\admini~1\appdata\local\temp\tmpojzbmz.lnk for command li
ne:
build\application.o build\startup.o build\board.o build\stm32f10x_it.o build\rtc
.o build\usart.o build\serial.o build\led.o build\enc28j60.o build\dm9000a.o bui
ld\touch.o build\calibration.o build\ssd1289.o build\src\clock.o
build\src\device
......
seems there are two mistakes.

there should not be an @ at the head of the tempfile's path, while
env["TEMPFILEPREFIX"] = ' ' doesn't help.
there is an 'build\application.o' behind '--via'  in the command line, but
it is already in the tempfile, and should not be there

the Right command should be：
armlink -o rtthread-stm32.axf  $LINKFLAGS --via
c:\users\admini~1\appdata\local\temp\tmp5r9pd_.lnk
Then I found a hack-way to solve this problem.
first modify SConstruct

if rtconfig.PLATFORM == 'armcc':
env["TEMPFILE"] = SCons.Platform.TempFileMunge
env["LINKCOM"] = "$LINK -o $TARGET $LINKFLAGS ${TEMPFILE('--via
$SOURCES')}"
env["TEMPFILEPREFIX"] = ' '      # arm tool chain

but a patch for C:\Python27\Lib\site-packages\scons-
2.2.0\SCons\Platform_init_.py is needed
--- init.old.py	Mon Nov 19 02:59:32 2012
+++ init.py	Mon Nov 19 02:59:19 2012
@@ -202,7 +202,7 @@
    prefix = env.subst('$TEMPFILEPREFIX')
    if not prefix:

       prefix = '@'

       prefix = ''

  args = list(map(SCons.Subst.quote_spaces, cmd[1:]))
  os.write(fd, " ".join(args) + "\n")

then, run scons -j4 in CMD, I can build correctly as following
armlink -o rtthread-mini2440.axf --device DARMSS9 --strict --info sizes --info t
otals --info unused --info veneers --list rtthread-mini2440.map --ro-base 0x3000
0000 --entry Entry_Point --first Entry_Point --libpath "D:/Keil/ARM/RV31/LIB" --
keep _rtmsym* --keep _fsym* --keep _vsym* --via c:\users\admini~1\appdata
local\temp\tmp05gi1v.lnk
Using tempfile c:\users\admini~1\appdata\local\temp\tmpvte9xd.lnk for command li
ne:
--via build\application.o build\startup.o build\board.o build\console.o build\le
d.o build\sdcard.o build\dm9000.o build\touch.o build\key.o build\calibration.o
build\lcd_t35.o build\src\clock.o build\src\device.o build\src\idle.o build\src
......................
ox.o build\examples\gui\demo_view_edit.o build\examples\gui\demo_view_bmp.o buil
d\examples\gui\demo_plot.o build\examples\gui\mywidget.o
fromelf --bin rtthread-mini2440.axf --output rtthread.bin
fromelf -z rtthread-mini2440.axf
========================================================================
** Object/Image Component Sizes
 Code (inc. data)   RO Data    RW Data    ZI Data      Debug   Object Name

629776      33616     705644       3784     170348    2727209   rtthread-min
i2440.axf (uncompressed)
629776      33616     705644       1544     170348    2727209   rtthread-min
i2440.axf (compressed)
629776      33616     705644       1544          0          0   ROM Totals f
or rtthread-mini2440.axf
scons: done building targets.
PS. the patch is really ugly, Is there any way that I can omit the '@' in the
head of the path fo tempfile?
env["TEMPFILEPREFIX"]='  ' (space here) doesn't help, spaces in 'TEMPFILEPREFIX'
will be deleted.

Setting
env['TEMPFILEPREFIX'] = '-via '
Should resolve this per entry for TEMPFILEPREFIX in manpage.
Closing.
Please reopen if still an issue


complete pipe chains

This adds completion for pipe chains, without requiring compilation, when the return type can be inferred for the current pipe chain.
Example:
let arr = [1, 2, 3]

// Everything below is without saving (without compilation)
let _ = arr->
//           ^ this already completed to Js.Array2 before

let _ = arr->Js.Array2.map(v => v)->
//                                  ^ but this didn't. It does now, as we complete from the last expression before the final pipe (Js.Array2.map in this case)

It handles when the return type is known without having to infer type parameters. Next step would be to handle inferring via type parameters too, but that's a vastly larger thing.
This however should make a lot of workflows more convenient, even in its current state.

Side note (and not something to solve in this PR per se) - it would be interesting to figure out why pipe completion doesn't work after reduce in the example below, even after compiling:
let extractDeps = rawDeps =>
  rawDeps
  ->String.split(" ")
  ->Array.map(s => (s->String.trim->Path.parse).name)
  ->Array.reduce([], (acc, curr) => {
    if acc->Array.includes(curr) {
      acc
    } else {
      let _ = acc->Array.push(curr)
      acc
    }
  })

I think a concise description of what this does is:

Complete e->id-> as if it were id(e).
Complete e->foo(a,b)-> as if it were foo(e,a,b).

so this is teaching completion the semantics of pipe.
Correct?

Correct.

Side note (and not something to solve in this PR per se) - it would be interesting to figure out why pipe completion doesn't work after reduce in the example below, even after compiling:
let extractDeps = rawDeps =>
  rawDeps
  ->String.split(" ")
  ->Array.map(s => (s->String.trim->Path.parse).name)
  ->Array.reduce([], (acc, curr) => {
    if acc->Array.includes(curr) {
      acc
    } else {
      let _ = acc->Array.push(curr)
      acc
    }
  })

For the same reason that this does not work:
let id = x=>x

let e = id([])->

but this does:
let idArray = (x:array<_>)=> x

let e = idArray([])->

Namely, completion for application takes the result type. And without typing the not-yet-compiled code, one cannot deduce the return type of the function. And gets the declared type instead ('a).

This is with compiling though. That should make a difference, right?

This is with compiling though. That should make a difference, right?

You can't compile incomplete code.
Even if you successfully compile this:
let e = id([])

it does not mean you know the type of id([]) in general. It only means you know the type of e is an array.
So when you then do this:
let e = id([])->

it really does not help to know that something called e was an array last time you compiled.


Fix invalid Maven group ID in docs

Hello :) The group ID in documentation was invalid.

Good catch. Thanks for taking the time to open a PR.

@all-contributors please add @sarxos for doc


Remove .gitattributes

I don't believe this file has any use anymore. The vendored folder was out of date and no longer exists. The automatic line ending normalization was commented out years ago.

Acceptable to me.


[Bug] Blank lines in code when I convert Markdown to HTML

Summary
I want correct convert  of Markdown to HTML. Now I need manually remove or add blank lines each time.
Expected behavior 1 — one-line code

If you Litecoin client not supported Litecoin schema URI, my Litecoin address:

LLVvhNKGMLGHa8QmeRrBsjZUBjSpQMjUkP

Actual behavior 1 — one-line code

If you Litecoin client not supported Litecoin schema URI, my Litecoin address:

LLVvhNKGMLGHa8QmeRrBsjZUBjSpQMjUkP

Expected behavior 2 — multi-line code

Alt+N → Settings - User → add in open file 2 lines:

"color_scheme": "Packages/SashaSublime/SashaSublime.tmTheme",
"theme": "SashaSublime.sublime-theme",

Actual behavior 2 — multi-line code

Alt+N → Settings - User → add in open file 2 lines:

"color_scheme": "Packages/SashaSublime/SashaSublime.tmTheme",
"theme": "SashaSublime.sublime-theme",

Screenshot
For highlight, I use rainbow library.
My code: https://github.com/Kristinita/Kristinita.github.io/blob/master/technical_texts/markdown_preview_rainbow.html

Steps to reproduce
My User\MarkdownPreview.sublime-settings file:
{
    "html_simple": true,
}

I have Markdown file:
If you Litecoin client not supported Litecoin schema URI, my Litecoin address:

    LLVvhNKGMLGHa8QmeRrBsjZUBjSpQMjUkP

---

---

Alt+N → `Settings - User` → add in open file 2 lines:

    "color_scheme": "Packages/SashaSublime/SashaSublime.tmTheme",
    "theme": "SashaSublime.sublime-theme",

Be attentive, [correctly place](http://ru.stackoverflow.com/a/238934/199934) commas, quotes, brackets and colons.

Ctrl+Shift+P → Markdown Preview: Export HTML In Sublime Text → markdown → I get results for actual behavior, not for expected behavior.
Environment
Operating system and version:
Windows 10.0.14393
Sublime Text:
Build 3126
Thanks.

I'm not sure this is a problem.  Is this introducing something undesirable?  The visual rendering is the same whether there is a new line at the end or not.  And to be honest, this is probably due to the the Python Markdown parser underneath, not directly Markdown Preview.

Let me explain further.
I entered this; notice I am doing one code block the supposed "bug" way, and the other forcing the "right" way:
If you Litecoin client not supported Litecoin schema URI, my Litecoin address:

    LLVvhNKGMLGHa8QmeRrBsjZUBjSpQMjUkP

LLVvhNKGMLGHa8QmeRrBsjZUBjSpQMjUkP

---

---

Alt+N → `Settings - User` → add in open file 2 lines:

    "color_scheme": "Packages/SashaSublime/SashaSublime.tmTheme",
    "theme": "SashaSublime.sublime-theme",

Be attentive, [correctly place](http://ru.stackoverflow.com/a/238934/199934) commas, quotes, brackets and colons.

Here is my visual results.  Notice there are no differences:

I imagine your visual issues are due to your CSS.

@facelessuser , I remove all CSS into https://github.com/Kristinita/Kristinita.github.io/blob/master/technical_texts/markdown_preview_rainbow.html files. Results for actual and extended behavior still are different:

Thanks.

All right, I disable html_simple. Now my MarkdownPreview.sublime-settings file is:
{
    "html_simple": false,
    "embed_css_for_sublime_output": false,
}

But I get same conversion results as actual behavior. How I can get expected behavior?
Thanks.

This won't help: "embed_css_for_sublime_output": false,.
Visually, I have no issues when html_simple is disabled.  The actual output is this

LLVvhNKGMLGHa8QmeRrBsjZUBjSpQMjUkP

This is fine and is expected for Python Markdown and should not not give you any visual issues.
What I thought I was seeing at the time was thathtml_simple caused extra white space (notice the indentation):

LLVvhNKGMLGHa8QmeRrBsjZUBjSpQMjUkP

But now that I am trying to reproduce it, I cannot.  Maybe it was an issue on my part.  The new line is not a bug.
Look here.  It shows the output of many Markdown parsers, and a lot do the same thing, only a few do not: http://johnmacfarlane.net/babelmark2/?text=++++LLVvhNKGMLGHa8QmeRrBsjZUBjSpQMjUkP.

All right, its no bug. What I need to make to obtain expected behavior, not actual behavior? The reasons I wrote in previous comments.

Thanks.

I think the issue has nothing to do with Markdown preview.  I think you are doing something to the HTML after it is processed.  I looked at your HTML at this link https://github.com/Kristinita/Kristinita.github.io/blob/master/technical_texts/markdown_preview_rainbow.html.  Markdown Preview does not ident elements like that.  I feel like you are prettifying your HTML, and because you are, it is introducing  an extra blank line:
This:

LLVvhNKGMLGHa8QmeRrBsjZUBjSpQMjUkP

Is very different from this:

LLVvhNKGMLGHa8QmeRrBsjZUBjSpQMjUkP

Notice the second example introduces white space (not just a new line, but indentation) at the end of code.  Markdown Preview does not ident pre and code tags like this.  Markdown Preview may not pretty output the HTML, but it is valid, but if things are prettified incorrectly, it can introduce a difference in output.
So my question is, what else are you doing to the HTML after it outputs it?

All right, I'm not prettify my code. I only make DOCTYPE — I print ! and press Tab — and insert conversion result between body tags.
<!DOCTYPE html>

    <meta charset="UTF-8">
    <title>Document</title>

If you Litecoin client not supported Litecoin schema URI, my Litecoin address:

LLVvhNKGMLGHa8QmeRrBsjZUBjSpQMjUkP

Alt+N → Settings - User → add in open file 2 lines:

"color_scheme": "Packages/SashaSublime/SashaSublime.tmTheme",
    "theme": "SashaSublime.sublime-theme",

Be attentive, correctly place commas, quotes, brackets and colons.

I no get expected behavior:

Question
How I can get expected behavior?
Thanks.

You are still doing something to the output afterwards.   This is the output with "html_simple": true:

If you Litecoin client not supported Litecoin schema URI, my Litecoin address:

LLVvhNKGMLGHa8QmeRrBsjZUBjSpQMjUkP

Alt+N â†’ Settings - User â†’ add in open file 2 lines:

"color_scheme": "Packages/SashaSublime/SashaSublime.tmTheme",
"theme": "SashaSublime.sublime-theme",

Be attentive, correctly place commas, quotes, brackets and colons.

Notice that the content is not indented.  When you insert the content into your HTML, you need to insert it like this:
<!DOCTYPE html>

    <meta charset="UTF-8">
    <title>Document</title>

If you Litecoin client not supported Litecoin schema URI, my Litecoin address:

LLVvhNKGMLGHa8QmeRrBsjZUBjSpQMjUkP

Alt+N → Settings - User → add in open file 2 lines:

"color_scheme": "Packages/SashaSublime/SashaSublime.tmTheme",
"theme": "SashaSublime.sublime-theme",

Be attentive, correctly place commas, quotes, brackets and colons.

You can't insert indentation like that as it affects the content of some of the HTML tags.

@facelessuser , thanks, I understand.
But I should not use my beautifier? I need beautifier.

But I should not use my beautifier? I need beautifier.

It's not an easier answer.  I know this, the way Python Markdown outputs pre, code blocks is sensitive to indentation.  If you wish to generate HTML with Python Markdown, but use a beautifier that does not recognize when indentation can mess up the content, then I would suggest to not use that beautifier.  If you have a beautifier that can handle situations like this proper, then it would be okay.
Ideally, you would have a system that generates your HTML content from markdown and you never have to look at the HTML source making the need for pretty HTML obsolete. When I work with Markdown, I am using tools where I rarely have to look at my HTML, so I don't care how pretty it looks.  I rarely pretty print because I don't care how it looks, just that it is formatted proper.
If you generally trust the output tool, then you rarely have to look at the actual output.  On rare occasions when debugging an output you might want to run the content through a reliable pretty print tool to debug an issue, but other than that, you shouldn't really need to pretty print your content.
I'm not sure what beautifier you are using, but I would argue that maybe it is the tool with a bug as it does physically alter the content in your pre, code blocks and adds undesired whitespace.

I'm not sure what beautifier you are using, but I would argue that maybe it is the tool with a bug as it does physically alter the content in your pre, code blocks and adds undesired whitespace.

I use HTML-CSS-JS Prettify, its recommended on Stack Overflow. HTML-CSS-JS Prettify prettify good code blocks, but I use paste and indent. I understood, I don't need paste_and_indent for code blocks.

When I work with Markdown, I am using tools where I rarely have to look at my HTML, so I don't care how pretty it looks.

I wrote text in Markdown, but I designed text in HTML. I can not have live preview my design changes in Markdown. If I further make editions, I edit HTML, not Markdown. Therefore, I need beautifier. Thanks.

I wrote text in Markdown, but I designed text in HTML. I can not have live preview my design changes in Markdown. If I further make editions, I edit HTML, not Markdown. Therefore, I need beautifier. Thanks.

There are probably ways to make this work, but it would probably require work upfront. More importantly, you have to find a workflow that works best for you, so I understand.


Unable to override assets on existing release

Same issue as #2
        - name: Upload assets to release
          uses<EMAIL_ADDRESS>          with:
            token: ${{ secrets.GITHUB_TOKEN }}
            tag: latest
            name: latest
            allow_override: true
            gzip: false
            files: >
                filename1.ext
                filename2.ext

Create Release
Upload Assets
  Files to be uploaded to Github:
  filename1.ext
  filename2.ext
  {
    "message": "Not Found",
    "documentation_url": "https://docs.github.com/rest/reference/repos#delete-a-release-asset"
  }
  Error: failed to upload asset: filename1.ext (see log for details)

  	ERR: Failed asset upload: filename1.ext
  {
    "message": "Validation Failed",
    "request_id": "07C0:4CCE:385984:52E5BB:6120476B",
    "documentation_url": "https://docs.github.com/rest",
    "errors": [
      {
        "resource": "ReleaseAsset",
        "code": "already_exists",
        "field": "name"
      }
    ]
  }

I also tried with uses<EMAIL_ADDRESS>however that fails to find the version.
Error: Unable to resolve action<EMAIL_ADDRESS>unable to find version `2.0.3`

This has just come up for me too. It was working fine 5 days ago. Was there a change in the github API?

This has just come up for me too. It was working fine 5 days ago. Was there a change in the github API?

Not sure if there was a change in the API or not, but I guess that's probably the cause. It appears this action has some logic in it which is supposed to delete an asset if it already exists (https://github.com/meeDamian/github-release/blob/7ae19492500104f636b3fee4d8103af0fed36c8e/entrypoint.sh#L217). I have tried several other release actions, and they all seem to suffer from the same issue when the asset already exists. One of the repos seems to work when there's a single file uploaded, but not when several are.
There is an update asset section in the github api (https://docs.github.com/en/rest/reference/repos#update-a-release-asset) but not in github CLI (https://docs.github.com/en/repositories/releasing-projects-on-github/managing-releases-in-a-repository#editing-a-release) unless you use the gh api command (https://cli.github.com/manual/gh_api).
Probably going to make an attempt at creating a release with a python script (not good enough with bash to be honest)