transaction rejected by network rules" 16 mandatory script verify flag failed - signature must be zero for failed check-(multi)sig operation

after imported an electrum btc wallet with 2fa into electron cash wallet
and sending bitcoincash coins to another electron cash address.
the bitcoin cash coins are correctly seen but im unable to send them

i have the exact same issue. see my screenshot below:
http://imgur.com/a/KR8cV
please fix this asap..please!

I am also experiencing this problem.
I noticed that when you import the seed it creates 3xx addresses, so maybe not obeying the enable/disable 2FA?
Maybe this is separate but would be nice if you could atleast send the coins you have in a 2FA...
Please fix this ASAP.

having the same problem transaction rejected by network rules" 16 mandatory script verify flag failed - signature must be zero for failed check-(multi)sig operation transaction rejected by network rules"my wallet is also 2fa

getting this problem too
PLEASE WARN PEOPLE ABOUT THIS! :eagle:

can it be solved or not?

this is not dieselgate - it's software, of course it can be fixed

from reddit: "I have to see if non hardware multisigs will be supported on Electrum Cash. Let's let Kyuupichan verify this." https://www.reddit.com/r/btc/comments/6pnv29/i_will_be_maintaining_the_electrum_cash_wallet_ama/
This suggest multisig has not been tested before release

before you confirm that you have the same issue, please make sure that you have disabled 2FA, and that you are connected to a BCC server, and not a BTC server.

OK, I guess I understand why this is broken.
The problem is with b3081759c250c0f17f758074c27485551f55f7b8
Electron Cash uses Electrum's segwit code for the serialization of transaction preimages.
The problem is that support for segwit+multisig has never been added to the original Electrum code.

what does that mean?we can withdraw our bcc from elctron cash wallet or not?

@dare2win2 no you cannot, it is broken.
this is what happens when people copy code without understanding what it does.
they have to fix it now..

hi @ecdsa thanks for your helpful input-- does this suggest the coins are only temporarily lost

@alsaine The coins are not lost. The software is just incapable of creating a transaction spending them, for now.

thanks @SomberNight
peace of mind for us worried noobs :+1:

[23:33]  If the code base were better.  I would like to help but I don't have much time.  And I'm away for 2 weeks now.  It will solve itself.

I might be out of context...
I just caught onto the fact that MUtibit HD is really outdated and doesn't have support anymore. I ran into a nice tutorial they made 5 days ago about transitioning to an Electrum wallet, Now, I am wondering how to transition safely to make sure I don't lose my Bitcoin cash (BCC). I just bought more before the fork. I have my seed words.
Here is what I was thinking trying:
Upload my seeds to my new electrum wallet.
Transfer the Bitcoins (BTC) to a different Electrum wallet ( on another computer)
Then use my MULtibit HD seeds ( same as my new electrum right?) to  upload my BCC on the Bitcoin cash wallet.
Thanks a million for any help, tips or inputs.
QUESTION: Should I even bother uploading my seeds from Multibit to Electrum or I could just transfer directly my BTC to a different Electrum wallet ?

Can one not construct the multi-sig transaction using Bitcoin ABC? If so, how does one extract the two necessary private keys from the 2-of-3 addresses that Electrum uses?

hi @adamkrellenstein
you might want to read this
https://bitcoin.stackexchange.com/questions/57441/how-to-transfer-bitcoin-cash-from-a-multi-sig-vault
however I am not technically proficient enough to instruct you further

It happened to me too. No 2FA. I tried to send a tiny amount - 0.9mBCC and got the same error about network rules 16. I didn't open a multisig wallet though. I guess I will have to wait for the coders to patch it.

OK, Looks we lost our BCC because we tried importing the private key before the Electron Cash Wallet was fully synced. Now when we try to sweep the private key we get an error similar to the one in this thread.

But I did it in the last few minutes. I can still see my balance of the princely sum of 1.23319BCC. I only transferred a small amount as a test. But I still get the same error, mandatory script verify flag failed (signature hash type missing or not understood)...
So if I transfer the rest in a day or two, are you suggesting @adsactly that it might work? Or do we need to create a new wallet from seed after that time?
Another thing. All the guides I have read say to create a new wallet in electrum, transfer BTC to that, then use the seed from the old, now empty, wallet. I tried this both within electrum and using the seed fell a Blockchain wallet. All it does is display a zero balance of BCC and the transactions that led to that (emptying).  But when Intried it from an electrum wallet with a tiny balance, the same balance appeared in electron, while the original BTC balance was unaffected. Except that I cannot transact in BCC as above.

https://blockchair.com/bitcoin-cash/transaction/cf057c3d642207b90ba56a513b86e307f59931480fcdfd611782e5e153e993be#o=1

Our Transaction is still unconfirmed. Is there any way anybody can manually verify a transaction?

[23:33] kyuupichan: If the code base were better. I would like to help but I don't have much time. And I'm away for 2 weeks now. It will solve itself.

So basically, he screwed us all out of our BCC by not understanding what he was doing with the project...

https://github.com/fyookball/electrum/commit/b3081759c250c0f17f758074c27485551f55f7b8#commitcomment-23438589
I found the fix, if anyone wants to Pull Request it or make some builds.

@ecdsa (You should prolly merge this fix into Electrum as well)
@fyookball

Line 730 of lib/transaction.py should be changed:
scriptCode = push_script(self.get_preimage_script(txin))

to
preimage_script = self.get_preimage_script(txin)
scriptCode = var_int(len(preimage_script)) + preimage_script

This will allow multisig redeemscripts over 75 bytes to be properly signed. (PUSHOPs and VARINTS are the same bytes up to lengths of 75, then PUSHOPS change to multibytes where VARINT can go up to 252 before going multibyte.)

@dabura667
This did not fix anything for me; error is still occurring, coins still unmovable.

As I understand this, if you were using 2FA on Electrum, you are not able to send BCH coins using the same wallet in Electron Dash. Is that correct?
The 2FA support in Electrum seems to be reliant on the services provided by trustedcoin.com. I have not seen any communication from trustedcoin.com that they are, or will, support Bitcoin Cash.

  Or at least a coin saver.  :-)  Can confirm this works fine; transaction sent and waiting on confirmations.

@SiriusStarr great news. thank you @dabura667

@adsactly your tx cf057c3d642207b90ba56a513b86e307f59931480fcdfd611782e5e153e993be has now been included in a block.

@staffnsnake please can you check you are on the right chain. See https://steemit.com/bitcoin/@electroncash/electron-cash-how-to-check-you-are-on-the-bitcoin-cash-chain

As I understand this, if you were using 2FA on Electrum, you are not able to send BCH coins using the same wallet in Electron Cash. Is that correct?
The 2FA support in Electrum seems to be reliant on the services provided by trustedcoin.com. I have not seen any communication from trustedcoin.com that they are, or will, support Bitcoin Cash.

My understanding is that if you "recover" your 2fa Electrum (Cash) wallet from the seed, then you no longer need to rely on trustedcoin.com to sign the tx---you have two keys locally.
So if I take an Electrum 2fa wallet and use its seed to recover it into Electrum Cash with the above patch, will I be able to send BCH?

@adamkrellenstein
Correct, that is what I did, and it has worked fine and been confirmed.

@dabura667 thanks!

@adamkrellenstein @SiriusStarr yes we've been receiving reports that this works correctly

I have merged this in, can someone please test it?
@ecdsa , you guys are the pros.  Lmk when you wanna take over the show here.

how do we access the fixed code on the program?

How do we test it?

would this explain why I can't sign a transaction or view seed(pin error) on the android apk version?

How do we get this fixed? I can't send anything!

@therealninza  i dont have any problem viewing the seed on android.  Signing a tx doesnt work for the moment, thats already a reported issue.
To test the fix you need to clone the repo again (or just git pull) and run again from source.

I have zero programming knowledge, so I have no idea what that means.

@roonsa then you can wait for the next release or try the workaround mentioned above.

It seems to be working now.
I was able to send payment.
Here is what I did:
I compiled the new merged source code in Ubuntu.
remember to select the correct server:  bcc.arihanc.com
don't use electrum server.
recover my 2fa wallet from seed.
disabled 2fa.
send payment. error is now gone.
Good job for the quick fix!!!

Am I able to do this through the github desktop? I have it open in github desktop but don't know what to do now :(

This affect me too
Please fix in the next release

I recieve a pin error when trying to decrypt. Even after changing the pin.
K
On Aug 2, 2017 5:20 PM, "fyookball"<EMAIL_ADDRESS>wrote:

@therealninza https://github.com/therealninza i dont have any problem
viewing the seed on android. Signing a tx doesnt work for the moment, thats
already a reported issue.
To test the fix you need to clone the repo again (or just git pull) and
run again from source.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fyookball/electrum/issues/6#issuecomment-319833427,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AP20UnI4QRmWWrjHVvuqZlxWSJ0sOyt3ks5sURI8gaJpZM4Oqo_d
.

@ronosa command line is not that hard...
git clone http://github.com/fyookball/electrum.git

@charleshsu168  great!  Sounds like its working and great work @dabura667 .   i am going to close this issue now.
@ronosa please open another ticket if you want help with git.
@therealninza  please open another ticket for the pin issue.

Note this fix only works if 2fa is disabled.
if 2fa is enabled, you've got an open issue https://github.com/fyookball/electrum/issues/41

someone please help!!!! I keep getting transaction rejected because of multisig how do I send Bitcoin Cash with a 2FA electron cash wallet?

I keep getting this error message can someone please help me? I am new and not very tech savvy

best to remove the photo @ajfeinman

removed.  just flipping out a little....any ideas?

I found it easier on Linux. Just go to a pawn store and pick up a cheap old netbook for less than $50, then install a lightweight linux distribution on it like Xubuntu. I used Mint. Then follow the directions further up in the discussion to install Fyookball's clone of electron-cash. I am a newbie and that is what I did.

And AFAIK you cannot do it with 2FA yet

@dabura667 could you please compile a new binary for osx with the fix??
Is there any estimated time for releasing 2.9.2?
Thanks

@dabura667

Line 730 of lib/transaction.py should be changed:
scriptCode = push_script(self.get_preimage_script(txin))
to
preimage_script = self.get_preimage_script(txin)
scriptCode = var_int(len(preimage_script)/2) + preimage_script
This will allow multisig redeemscripts over 75 bytes to be properly signed. (PUSHOPs and VARINTS are the same bytes up to lengths of 75, then PUSHOPS change to multibytes where VARINT can go up to 252 before going multibyte.)

This produce me
Traceback (most recent call last):
  File "electron-cash", line 95, in <module>
    imp.load_module('electrum', *imp.find_module('lib'))
  File "C:\Documents and Settings\Admin\Desktop\Electron Cash-2.9.0\lib\__init__
.py", line 3, in <module>
    from wallet import Synchronizer, Wallet, Imported_Wallet
  File "C:\Documents and Settings\Admin\Desktop\Electron Cash-2.9.0\lib\wallet.p
y", line 55, in <module>
    import transaction
  File "C:\Documents and Settings\Admin\Desktop\Electron Cash-2.9.0\lib\transact
ion.py", line 726
    preimage_script = self.get_preimage_script(txin)
    ^
IndentationError: unexpected indent

Seems that's a Text editor error -_-
Trying to edit with Python IDLE and success
Notepad++ seems to show different identation

@Maxradice I don't own a Mac.......... so I can't compile. Sorry.

Traceback (most recent call last):
File "setup.py", line 5, in
from setuptools import setup
ImportError: No module named setuptools
launching python setup.py install

I got the new version for mac. 2.9.2, downloaded installed and cant send my bcc Im getting this error
"error: The transaction was rejected by network rules.  (16: mandatory-script-verify-flag-failed (Signature must use SIGHASH_FORKID))"
My electrum wallet had 2fa, so im asked every time I do a transaction for google auth numbers, If i dont get the numbers, transaction is not correctly signed and it is missing 1 of the 2 steps..
If I set the Google Auth numbers, it gives me the error...
This build was intended to fix the signing issue right? I'm doing something wrong?
Thanks!

yes you are doing it wrong.
you have to disable 2FA in the BCC version (for that you must restore wallet from seed)

It worked...
Anyways, I was able to send Part of my BCC... I did a few test transactions and the I sent a large part of my balance.. what is left Im not able to send it, as it says I have insufficient funds.
As an example,
I sent 1 to test out of 10 Balance was 9 (minus fees)
Then I sent 1 more (balance was 8 (minus fees)
I sent 5 Balance was 3 (minus fees)
I tried to sent the balance left... I can only send 30% of what is left... something seems to be wrong with Balance and is not allowing me to send the complete balance.
It may be related to unconfirmed transactions?
Any idea? I can see the correct balance on the history, I cant send more than 30% of it.
Thanks

there's an option to allow electrum to send unconfirmed coins

Worked,
Thanks guys... regards!

I’ve downloaded 2.9.2 (OSX) and am still having trouble.
history: using 2.9.0
wallet showed my BCC coins, but unable to send with the same error as OP
using 2.9.2
Initial attempt: simply opened up my previous wallet, which shows my BCC coins, attempted to send but received this new error:
The transaction was rejected by network rules.  (Safe mode: Warning: We do not appear to fully agree with our peers! You may need to upgrade, or other nodes may need to upgrade.)
notes
The above wallet was still in 2FA mode, but when clicking on TrustedCoin icon I got this: This wallet is was restored from seed, and it contains two master private keys. Therefore, two-factor authentication is disabled.  (which is why I thought I’d try sending before restoring to a new wallet)
Also, my public addresses in this wallet all start with ‘3’
So, restoring my wallet into a new wallet without 2FA had problems.  Here’s what I’ve tried:

I created a new ‘standard’ wallet, using my seed.

1st attempt: when pasting my seed Electron said that it was a 2FA seed.  Assuming the new wallet being created will not be, as I’d previously selected ’standard’ wallet, I selected OK.  Electron window showed ‘please wait’…. eternally.  I waited for 30min and assumed something wrong.

2nd attempt: When pasting my seed in this time I clicked ‘options’ next to where it displayed that my seed was 2FA and then selected the BIP39 checkbox to make the 2FA notification go away.  I don’t know what ‘wallet derivation’ means, but the code in there was: m/44’/0’/0’  I left that alone.  This created a new wallet successfully.  However the addresses it imported were not like my original addresses starting with ‘3’.  These were starting with ‘1’ and displayed no coin balance, so obviously couldn’t send anything.

3rd attempt: into that new wallet I tried sweeping my BCC holding private keys.  This gave a ’no inputs found’ error.  I’m assuming because it’s only attempting to sweep from the addresses starting with ‘1’, and not the ‘3’ addresses where my BCC is displaying correctly.

ANY help would be greatly appreciated.  I don’t know any code, or even how to compile source into binaries, so you’re talking to a relative noob.  I’m just trying to use my BCC.  :)

ecdsa: "you have to disable 2FA in the BCC version (for that you must restore wallet from seed)"
How do you restore from the seed? Whenever I hit file>new/restore I am only given the option to create a new wallet.
Please help, thanks!

@devilbond  not sure why you are checking bip39.  unless its a bip39 seed, dont do that.  Electron Cash doesnt support 2fa but you should be following the instructions to restore from seed.  i havent done this myself as i dont use 2fa but it seems to have worked for others.

devilbond: had same issue as your 1st attempt. I type in the seed after previously selecting standard wallet, then click next and it just says please wait
Screenshot: https://gyazo.com/9c8483681674f0935c747b6ab963275c

@ftwonka Yup, that's the screen I get.
@fyookball I only checked that BIP39 box as an experiment on my 2nd attempt because Electron just hung on 'please wait' the previous attempt. It was just something I tried.
It is possible that restoring from seed may/may not work depending on what wallet the seed is originally from?  i.e. does it only work on wallets that were not multisig wallets, etc?
@fyookball for me and anyone else trying to do this, please can you (or anyone) confirm that the below are the correct instructions to restore from seed:

Select 'New/Restore Wallet' from File Menu
Select 'standard wallet' (not multisig, not 2fa)
Select Restore from Seed
Paste seed.
4.1. If seed is from a 2FA wallet, that fact is displayed.
4.2. If seed is not from a 2FA wallet, that fact will not be displayed.
Don't click any of the options, just OK

Following the above does not work for me, the new wallet doesn't open, it just hangs on 'please wait' so if I'm missing anything, or doing any thing wrong above, do let me know.
Questions I have:

If the above works for some people, could they please advise what kind of wallet they were restoring.  Was it a multisig wallet?, did it originally have 2FA? Did you original wallet's public addresses start with a '1' or a '3' and what number do they begin with now?, etc
if the above did NOT work for some people, could they please do the same and advise what kind of wallet they were attempting to restore.

Thanks all in advance,
db

im not that familiar with 2fa , sorry i cannot be more helpful here.

It might all be moot anyway. After all the fiddling to get access to my BCC via this program, my deposit on Bittrex is only up to 7/20 confirmations after 12 hours. By the time it reaches 20, I'll be lucky to get even one MicroBit for it.

@staffnsnake Same. Just going to hold I guess.

@staffnsnake I just did restore then did normal wallet, can't remember the name but the top option. Then asked if I wanted to disable 2FA later, and I said yeah. I did the same steps you did above and it worked.

ShapeShift

@ronosa I think you mean "if I wanted to enable 2FA later", but happy you got your BCC, even if it is as illiquid AF. I'm up to 10/20 confirmations - half-way!

Can anyone with it working confirm/deny they have multisig wallets/addresses please?   Yes if your public address starts with 3. No if it starts with 1.
Thanks.

On Aug 4, 2017, at 21:39, staffnsnake<EMAIL_ADDRESS>wrote:
@ronosa I think you mean "if I wanted to enable 2FA later", but happy you got your BCC, even if it is as illiquid AF. I'm up to 10/20 confirmations - half-way!
―
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

hi i have been holding my BCC in electron cash 2.9.3 and now i found out that prior to 2.9.4 electron cash is not supported by network rules, but when i try to open my 2fa disabled wallet, the 2.9.4 software just crashes, what can i do please i have been stuck on this all day?


contrib: Extract aur-vercmp-devel

I realized that I would actually benefit more from having a script that detects outdated VCS packages, and extracted it into aur-vercmp-devel. The aur-update-devel becomes a one-liner, I still find it useful, but I'm not sure if you are OK with keeping it in the contrib 🙂

I like aur-vercmp-devel since it has one strictly defined task. I'd remove aur-update-devel from contrib then and maybe add something to the README (see https://github.com/AladW/aurutils/commit/95bcd0bbd606a858fafdb5c35823202c1bcf5861)

Done 👍


Show database references in project tree

Now the name of the dacpac of database references shows in the project tree:

Now the name of the dacpac of database references shows in the project tree:

are you ensuring that ref and data sources always show at top before files and folders?

@kisantia missing your gifs :)

For my understanding - you are not adding any ref only parsing if already present right?
(asking since while adding new - we will have to make sure of SSDT compat as discussed)

Coverage increased (+3.3%) to 34.235% when pulling 283fe4a9bb44579324293ced6bfebc4ceb516065 on kisantia/showReferences into 187667f4f1f52ee5e5a4b74e9a183574d6f46273 on master.

@udeeshagautam the ability to add database references was added in #10684. The changes in this PR are to parse them and display the references in the project tree. I'll send out a separate PR so that newly added references will also work in SSDT.


implement sancus

I want to implement sancus, but I found that xula2-lx25 and StickIt! Board have stopped production, how can I get these two boards, thank you

Hey,
for an academic software project we want to implement Sancus on the Cyclone V FPGA.
We are working with the Altera DE10 Standard board instead of the older Altera DE1 that you've already provided in your repository.
At the OpenMSP430 github repository, we noticed that the DE1 board was moved to obsolete directory.
Now we're at the point to implement the OpenMSP430 architecture onto the Cyclone V FPGA and already worked on adjusting the local assignments.
Here comes the question: Are you also working on the implementation of the Sancus architecture on the Altera DE10 Standard board? If so, there is no need for doing redundant parallel work and we'd prefer to support you instead and maybe continue from your current status.
If not: Would you be interested in including our DE10 results in your repository, if we'd manage to get the architecture working?
We've also contacted the OpenMSP430 developers for including the Altera DE10 in their repository.
Apart from that, we would also like to know whether you have any older documentation which you used when you started creating the example project for the different boards, or if there might be anything that is not provided in the repository that we need.

Hi Anton,
Thanks for getting in touch! We are not working on an DE10 implementation ourselves right now and we would be happy to include your port in the sancus-core repo. Did you get it to work already? Let us know if you need help or advice.
We will probably start working on porting Sancus to some Igloo FPGA from MicroSemi, early next year.
Also, we are always interested in understanding what other groups are doing with our work and prototypes and we are happy to link to applications and follow-up projects on the Sancus website. Please be in touch if you think you have something that could be of value to the community.
Happy New Year,
JT


force delete bucket

@EarthmanT how was it implemented? Looking at the code, I see that we delete all the contents of the bucket before removing it. Which means that "force delete" isn't really an option — it's implied. Can you confirm?
Also, which milestone is it available in?

Exactly.


x/tools/go/types/typeutil: TestMapGenerics failure on linux-amd64 TryBot

From https://storage.googleapis.com/go-build-log/60d5200e/linux-amd64_5c4ddb20.log (a TryBot on CL 367848):
--- FAIL: TestMapGenerics (0.00s)
    map_test.go:353: At(F2) = <nil>, want new entry: false
    map_test.go:353: At(Fb1) = <nil>, want new entry: false
FAIL
FAIL	golang.org/x/tools/go/types/typeutil	0.052s

However, there have been no occurrences in the dashboard logs so far:
greplogs --dashboard -md -l -e 'FAIL: TestMapGenerics'
(No results.)
(CC @findleyr)

The CL needs to be rebased, as it is missing https://golang.org/cl/366178 (parent is from Nov 16).


README improvements

Changes

Improved visual style of documentation
Added table of contents
Added "Kaspa On Rust" logo
Simplified the steps to setup a node
Removed redundant description of utilizing nodejs WASM.

Why
tldr; I improved the readability of the readme.
Simplifying the documentation and making it more accessible is key for Kaspa adoption. When I first came to this repo a couple days ago, I was struck that such a cool network had such an unorganized repository. I believe that the proposed changes will simplify the process for new developers to get started.
These changes are obviously not perfect and I think there could be more information added regarding the WASM so that front-end developers that have little to no experience with complex languages like Rust to get engaged as well. :)

@aspect

Love what you did here. Readmes have been super undervalued lately and aren't given a lot of love due to other developments.

Love what you did here. Readmes have been super undervalued lately and aren't given a lot of love due to other developments.

Thanks, I totally agree and feel like README's are one of the most important parts in a repo since they are the first impression of the project.

Apologies for delayed feedback.
This looks good, but I would like to request the following changes.

for now, let's remove the image from the top of the document.  there is no consensus and the feedback for that specific image wasn't great.  so we will think about it, but for now this should go.

Please refactor linux instructions to be presented as multi-line code.  I think this can be done by adding back-slashes like so:

apt-get install clang-format clang-tidy \
clang-tools clang clangd libc++-dev \
libc++1 libc++abi-dev libc++abi1 \
...

The fact that this code now offers copy button is great, but the fact that you can not see the whole set of instructions on the screen and have to scroll for it is not good.  So we need to make this fit on the screen.

I wanted to do this for a while, perhaps you can restructure the document to separate each platform section into <details> and <summary> so that each section becomes an openable section.

For example:
<details>
<summary>Installation on MacOS</summary>
... macos instructions
</details>

Then the other sections could perhaps be grouped this way as well.  IMO this will significantly reduce the footprint of the README making it much cleaner.
For example, there can be a set of details sections for OS-specific installation + WASM + wRPC, then "Running the node" and "Mining" as well as "Simpa" can be in the open.

there need to be changes to include sudo in all appropriate linux-related commands as per https://github.com/kaspanet/rusty-kaspa/pull/341

@aspect
Good points! Refactored in accordance with your suggestions. Includes the edits made by @coderofstuff in #341, added as co-author. Therefore also closes #340

So now this looks the opposite side of the spectrum.  There needs to be a balance of useful material presented and technical material being hidden.  Here is the list of sections (in a specific order) that should be visible or hidden:

Remove TOC, there is no point of having TOC to a bunch of hidden sections.  Hidden sections are a form of TOC.  I think perhaps the TOC is what forced you to create headings to be able to link to this.  If the document is structured properly with hidden sections, there is no need for TOC.

Installation

Building on Linux (hidden)
Building on Windows (hidden)
Building on MacOS (hidden)
Building WASM framework (hidden)
Wallet CLI (hidden)

Running the node

Start mainnet (open)
Start testnet (open)
wRPC
Mining

Benchmarking & Testing

Simulation Framework (Simpa)
Heap Profiling (closed)
Tests (closed)

Run unit and most integration tests
Using nextest

Benchmarks (closed)
Logging

So that is roughly 3 top-level sections (3 H1 headings), where Installation and Benchmarking is closed (thus it looks like TOC), and the main focus is on Running the node, where instructions are open by default.

@aspect The documentation has been adjusted I have grouped everything and hidden everything except for how to run the node. I added an intro text that I wrote with the help of an LLM. Let me know your thoughts on it. I believe it can serve as a starting point.


"echoerr" test in Ch_communicate() can randomly fail

Describe the bug
On line number 122 in the file testdir/test_channel.vim, an "echoerr"
command is sent using ch_evalexpr() to a python server that has been
created by the test.  The server responds with two JSON arrays:
["ex","echoerr \"this is an error\""]
[1,"echoerr"]

If the Vim instance under test retrieves both with a single read:
["ex","echoerr \"this is an error\""][1,"echoerr"]

then the test will fail.  It will fail because Vim will process the
numbered array first, apparently causing the ch_evalexpr() call to return
and control will flow out of the try-catch block that contains it.  Later,
when the "echoerr" command is evaluated, control has left the catch-try
block and the exception is not caught.  Since the purpose of the test is
to catch the exception, the test fails.
To Reproduce
Run the test in testdir:  make test_channel.res
However, as the Title says, the test may or may not fail.  See "Additional Context" below.
Expected behavior
The Vim instance under test will read the channel once and retrieve only
the "echoerr" command which is then executed, generating an exception
which is then caught as intended.  A second read will retrieve the
numbered array.
Screenshots
N/A
Environment (please complete the following information):

Vim version: 8.2.3242 with normal features and no gui
OS:  Linux  4.19.0-17-686-pae #1 SMP Debian 4.19.194-3 (2021-07-18) i686 GNU/Linux
Terminal: XFCE terminal with TERM=xterm-256

Additional context
I have eliminated this error by adding a small time delay in the python
server between the two messages it sends.  Specifically, I inserted
time.sleep(0.01)
after line number 116 in file test_channel.py.  I chose that value because
it is the same one used by several other cases that send an unnumbered
response followed by a numbered one.
This "solution" is not a fix- it simply makes the error less likely to occur.

Michael Soyka wrote:

Describe the bug
On line number 122 in the file testdir/test_channel.vim, an "echoerr"
command is sent using ch_evalexpr() to a python server that has been
created by the test.  The server responds with two JSON arrays:
["ex","echoerr \"this is an error\""]
[1,"echoerr"]

If the Vim instance under test retrieves both with a single read:
["ex","echoerr \"this is an error\""][1,"echoerr"]

then the test will fail.  It will fail because Vim will process the
numbered array first, apparently causing the ch_evalexpr() call to return
and control will flow out of the try-catch block that contains it.  Later,
when the "echoerr" command is evaluated, control has left the catch-try
block and the exception is not caught.  Since the purpose of the test is
to catch the exception, the test fails.

Thanks for observing this.

To Reproduce
Run the test in testdir:  make test_channel.res
However, as the Title says, the test may or may not fail.  See "Additional Context" below.
Expected behavior
The Vim instance under test will read the channel once and retrieve only
the "echoerr" command which is then executed, generating an exception
which is then caught as intended.  A second read will retrieve the
numbered array.
Screenshots
N/A
Environment (please complete the following information):

Vim version: 8.2.3242 with normal features and no gui
OS:  Linux  4.19.0-17-686-pae #1 SMP Debian 4.19.194-3 (2021-07-18) i686 GNU/Linux
Terminal: XFCE terminal with TERM=xterm-256

Additional context
I have eliminated this error by adding a small time delay in the python
server between the two messages it sends.  Specifically, I inserted
time.sleep(0.01)
after line number 116 in file test_channel.py.  I chose that value because
it is the same one used by several other cases that send an unnumbered
response followed by a numbered one.
This "solution" is not a fix- it simply makes the error less likely to
occur.

Well, it's as good as a fix.  I had seen Ch_communicate() fail randomly,
especially with IPv6.  Let's see if this sleep fixes it.
--
Facepalm statement #4: "3000 year old graves?  That's not possible, it's only
2014!"
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

The reason I hesitated to call  it a fix is because of the  out-of-order handling of the JSON messages in function "channel_read_json_block".  It appears that when multiple messages are retrieved by a single read that numbered messages will be handled before the others (I've based this opinion on my investigation of this test, not on any other tests I've run).  I may have missed it but I don't see this behavior documented in the help topic "channel-use".
By the way, I also saw this test fail for IPV6 but neglected to mention it.  The proposed delay fixed that one as well.

#8672

Michael Soyka wrote:

The reason I hesitated to call  it a fix is because of the
out-of-order handling of the JSON messages in function
"channel_read_json_block".  It appears that when multiple messages are
retrieved by a single read that numbered messages will be handled
before the others (I've based this opinion on my investigation of this
test, not on any other tests I've run).  I may have missed it but I
don't see this behavior documented in the help topic "channel-use".

The order is not guaranteeded.  Communication is asynchronous by nature.
It's also possible that a message gets split into parts, and there can
be a delay in between, esp. on a busy system.  And the systems running
CI are usually busy.

By the way, I also saw this test fail for IPV6 but neglected to
mention it.  The proposed delay fixed that one as well.

It was flaky, thus we'll have to see if this fixes at (and no other
problem pops up).
--
Facepalm statement #6: "Estland is a fantasy place, just like Middle Earth and
Madagaskar"
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

fixed by 890ee4e2be1dca0c07a91f836e26baead952ae7c


Security issues

Hello,
within a student project we needed to find 0-day vulnerabilities in java applications.
We've tested yours and found several security issues, and we would like to let you know about them.
If you're interested, please give us a contact email address.
Best regards
CY

@Capfly : Hello Capfly, why don't you share it here? I just found this repo, and would like to invest time to try.
If there're anything which I can contribute, I would like to help.

In general I give developers the chance to fix their security issues before publishing them to the public. I will publish my report after 6 months anyway.


Fix another bug in aggregates

So far at least 70 Souffle tests pass, but I haven't finished trying all of them yet.

this failed because of a timeout


issue running unpack_repack_bin.sh

I get the following error. and do not know how to source env.sh, can you point me in the right direction?
error:
[unpack_repack_bin] This tool relies on env.sh which has not been sourced
Thanks
Darrell

Type source env.sh from the command line. The env.sh script sets a bunch of environment variables, so you want to use that same shell to run the tools, as they expect these variables to be set. If you use a new shell, you will have to source env.sh again in the new shell.

tried that, but I was running slax in a vm, will try and get ubuntu going tomorrow and see if I have better results. thanks for the reply!

Got it to run, extracted the rootfs image and turned it into a qcow2 file. Still cant get it to run in eve-ng...

If you show what the problem was with getting the scripts to work and how you fixed it we can document it somewhere in the repo to help others.
I've never heard of eve-ng, but it looks like it's also built on qemu. We only ever used GNS3. The official qcow2 firmware files provided by cisco appeared to have driver/kernel modifications that let them run in an emulated environment, whereas running a .bin firmware wrapping the raw contents inside a qcow2 we never had working fully in GNS3 either I think. There would always be at least some broken functionality like no network, etc.


feat(switchFirstMap): rename switchMapFirst to switchFirstMap

Renaming switchMapFirst to switchFirstMap to match the naming scheme of <mergingStrategy>Map
related #915

quickly :+1: by PR only contains name change. :)


Cannot autostart crouton chroot when bootup

Hi there,
I have been following this autostart crouton guide but failed to get it working.
https://github.com/dnschneid/crouton/wiki/Autostart-crouton-chroot-at-ChromeOS-startup
I did not put crouton.init in ~/Downloads directory, just modified /etc/init.d/crouton.conf file like this:
...
env DELAY=10
env CHROOT=trusty <- I installed trusty.
env START_DE=startxfce4 <- Installed target was xfce
env XMETHOD=xorg <- I'm not using xiwi
env RUN_STATE=y
...
I think the guide is quite simple but I do not know what I missed...
Thanks in advance.

Those changes in crouton.conf look alright to me, I don't think that's the problem.
Did the first step running rw-rootfs complete without error and you rebooted?
If you left DEBUGging on, then you may want look at the file in /tmp/crouton-$PID.log to see what might have failed and where.

Also, you don't have to put crouton.init in your ~/Downloads folder, that's okay.
You can also put it in /usr/local/ too, it's just a little easier to tweak the crouton.init file than editing /etc/init/crouton.conf each time - for me at least.

Hi DennisLfromGA,
Thanks for the reply.
I remember there was one error when running rw-root-fs, which was something about brackets with -gt, but simply I have my root filesystem writable. Could that error be a problem? I think I can reproduce the error by reinstalling chroot environment.
I will look into /tmp/crouton*.log files when I get back home.

No, that error won't affect anything, it's a bug in my script that I haven't fixed yet.
The DEBUG log file should show us where it's failing and we can go from there, hopefully.

Very strange, I checked that DEBUG was on in /etc/init.d/crouton.conf but no crouton log files under /tmp.
Can you guess what I am missing?

I got where I made a mistake...
I put crouton.conf file under /etc/init.d, not /etc/init. What a shame...
Problem solved. Thanks!

That is very good news!
Glad it worked out, I've made one or two mistakes myself... ;)
You might want to tryout the crouton.init script option, I think it's pretty handy.

@initguru - If you don't mind, please close this issue if all is well - Thanx.


Typing Issues related to amazon-cognito-auth-js

From: aws/aws-amplify#737

Do you want to request a feature or report a bug?
Report a bug
What is the current behavior?
After installing AWS Amplify & AWS Amplify React on a newly created create-react-app Typescript application and importing AWS Amplify in my App.tsx, I get the following error:
C:/Users/***/***/project/node_modules/aws-amplify/lib/Common/Facet.d.ts
(6,34): Could not find a declaration file for module 'amazon-cognito-auth-js'. 'C:/Users/***/***/project/node_modules/amazon-cognito-auth-js/lib/index.js' implicitly has an 'any' type.
  Try `npm install @types/amazon-cognito-auth-js` if it exists or add a new declaration (.d.ts) file containing `declare module 'amazon-cognito-auth-js';`

Which versions of Amplify, and which browser / OS are affected by this issue? Did this work in previous versions?
I'm using the AWS-Amplify ^0.3.3, AWS-Amplify-React ^0.1.43.
Also tried using<EMAIL_ADDRESS>Is this a bug with the library? Is there a way to bypass/fix this?

I assume type definitions need to be created, or at least polished, by hand.
I just submitted a PR to add amazon-cognito-auth-js definitions to DefinitelyTyped, see: DefinitelyTyped/DefinitelyTyped#25622
If interested, I can create a PR to add the same type definitions here for review and comment.

Type definitions have been published to DefinitelyTyped and are now available through the @types/amazon-cognito-auth-js npm package. The DefinitelyTyped package should be deprecated once types are added here.

hi all, I would like to ask your opinion about this fork I created where I made these changes:

javascript to typescript
Promises and userhandler functions
test with jest

There are small changes to make but your opinion is welcome so as to know if I can make a PR or publish it on npm.
Thanks


osu-lazer{,-bin}: 2024.816.0 -> 2024.817.0

Description of changes
Release: https://github.com/ppy/osu/releases/tag/2024.817.0

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 335262 run on x86_64-linux 1

2 packages built:

osu-lazer
osu-lazer-bin

Result of nixpkgs-review pr 335262 run on aarch64-darwin 1

1 package built:

osu-lazer-bin


Correct mispelling of cargo.

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

Thanks!
@bors r+

:pushpin: Commit 08ba47d3d173c7fe341d53f48af615cece93bbff has been approved by ehuss

:hourglass: Testing commit 08ba47d3d173c7fe341d53f48af615cece93bbff with merge 442de291ca040cf894a1845e6be62f4c081f7b29...

:sunny: Test successful - checks-azure
Approved by: ehuss
Pushing 442de291ca040cf894a1845e6be62f4c081f7b29 to master...


Demo portal at https://irischatgpt.streamlit.app/ no longer runs

When opening the demo app, the following message is displayed:

ImportError: This app has encountered an error. The original error message is redacted to prevent data leaks. Full error details have been recorded in the logs (if you're on Streamlit Cloud, click on 'Manage app' in the lower right of your app).
Traceback:
File "/home/adminuser/venv/lib/python3.9/site-packages/streamlit/runtime/scriptrunner/script_runner.py", line 535, in _run_script
exec(code, module.dict)
File "/mount/src/streamlitllm/streamlitAPP.py", line 3, in
import irisChatGPT
File "/mount/src/streamlitllm/irisChatGPT.py", line 11, in
from langchain import  SQLDatabase, SQLDatabaseChain

Online demo has been removed from the repo


Add supporting functions for user defined sql planners to FunctionRegistry

Is your feature request related to a problem or challenge?
Currently with the function registry trait you can register a user defined sql planner to the FunctionRegistry but there is no ability to get a list of registered user defined sql planners.
Describe the solution you'd like
fn user_defined_sql_planners(&self) -> Vec<Arc<dyn UserDefinedSQLPlanner>>
Describe alternatives you've considered
No response
Additional context
No response

take


Tabster should move focus to the selected radio item in a group

Expected Behavior
When tabster moves focus programatically to a radio group, it should move focus to the selected item.
In this case, a radio group is defined as a collection of <input type="radio"> that have the same name attribute. It's not necessarily inside a <div role="radiogroup">, or inside a <RadioGroup> component.
Observed Behavior
Tabster moves focus to the first radio item, even if another one is selected.
Repro
https://codesandbox.io/s/86p2ck?file=/example.tsx
Tip: use the keyboard to open the dialog so the focus outline is visible.
Related
This is similar to #144, but this issue is not specifically related to useFocusableGroup.
Fixing this issue in tabster will fix part 2 of https://github.com/microsoft/fluentui/issues/28438 affecting FluentUI's RadioGroup.

Should be done in tabster 8.0. @layershifter / @mshoho please verify and close


array: generalize S.takeLast and S.dropLast

The implementations are not as efficient as they could be, but I want to avoid spreading complexity currently confined to _takeDrop.

If I'm right about the applicative constraint not being required in #627, then we probably don't need it here as well.

You have raised a good point, @Bradcomp; I had not considered infinite structures. I would not expect these functions to work on infinite structures, so I do not find the behaviour surprising, but it is a shame that the type signatures do not accurately convey the constraints.
https://github.com/sanctuary-js/sanctuary/pull/627#issuecomment-487275940 applies here too, @masaeedu, so I will merge this pull request in its current form. If it turns out that I have misunderstood the constraints, we can relax the constraints on S.take, S.drop, S.takeLast, and S.dropLast in another pull request. :)


test/cases: move CIV options into a variable

In order to provide different options when running CIV from it's own CI, move them to the variable CIV_OPTIONS
This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

Extra commit to remove cloud cleaner


checkboxes in datatables are bugged

go into containers.
select one of the items. -> note that the checkbox is selected, but no message (1 selected)
deselect -> 1 item selected
reselect -> message disappears

@chiptus there is still an issue with checkboxes in the dev branch

Click select all checkbox
Click checkbox in middle expecting it to unselect
See that 4 items are reported as selected
Click remove and all 4 items will be removed

Fixed.
@deviantony @ncresswell
seems like shift+select is also broken. What is the right behavior?
Select the first one, then hold shift and select another -> selects everything in between.
What about de-select?
and what if I keep shift held and select another row? what if that row is already selected?

seems like shift+select is also broken. What is the right behavior?
Select the first one, then hold shift and select another -> selects everything in between.

That's the right behavior.
I'm not sure if we ever have a mass de-select feature, might be worth checking in previous version of Portainer. This was introduced in version 1.22.0 of Portainer via https://github.com/portainer/portainer/issues/344

Seems selectAll has been affected by #4116 .
Go on any table that has actions (remove is the best) > select all items with the "select all" checkbox > uncheck manually some items > perfom remove action > action will be applied on selected elements + 1 (seems to be the last unchecked).
Counter on the bottom of the table is still bugged, is the goal of #4122 to fix it ?
@deviantony

I'm not sure if we ever have a mass de-select feature, might be worth checking in previous version of Portainer. This was introduced in version 1.22.0 of Portainer via #344

We had a copy of github issue selection checkboxes, so we had mass unselect. At least it's what we have currently running on demo site.
First reported bug (checkboxes + bottom text not matching) is also not present on demo website

Counter on the bottom of the table is still bugged, is the goal of #4122 to fix it ?

@chiptus we should also fix this as part of this work.

@xAt0mZ did you test #4122 ? everything should be fixed there (I just tested)

@chiptus I only noticed it while doing another feature, didn't test #4122 so far


chore: Update MobileCapabilityType to support W3C protocol

Change list
-Add prefix  'appium:' to required Mobile Capability Types
Types of changes
What types of changes are you proposing/introducing to .NET client?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] New feature (non-breaking change which adds value to the project)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Documentation

[ ] Have you proposed a file change/ PR with appium to update documentation?

This can be done by navigating to the documentation section on http://appium.io selecting the appropriate command/endpoint and clicking the 'Edit this doc' link to update the C# example
Integration tests

[ ] Have you provided integration tests to pass against the beta version of appium? (for Bugfix or New feature)

Details
Update MobileCapabilityType to support W3C protocol.
**Will update documentation after 5.0 is released, for now there is no point.

Check #471

I don't see my changes there as well, so currently if a user will pass one of the mobile caps types it will fail, unless we want the user to pass a string as an option.

Maybe I have just skipped something. @akinsolb might have better overview on the project

@Dor-bl This change has been implicitly done in AppiumOptions class in the v5 branch. It takes any cap that doesn't have a prefix and adds the appium prefix. Ideally, these capability classes can be deleted, providing driver specific options add their own prefix. Given the comments on #471 I'm not sure if 'app' is an appium capability or a driver specific capability, especially in the windows instance.


Selection of sorting is not persistent

Every time I close safari the sorting reset to the default (best) and __remarkSort disappears from the local storage.

I'm using Safari Version 12.0.1 (146<IP_ADDRESS>.1) and cannot reproduce the issue. I close the tab, close browser and after that I can see __remarkSort option in local storage.

@deril the bug above reproducible if you have prevent cross-site tracking on or disabled third-party cookies in safari's preferences. Also remark's page should be loaded in iframe. Safari treats localstorage of an iframe as a sessionStorage and wipes it on exit.
This is safari's anti-tracking 'feature': https://bugs.webkit.org/show_bug.cgi?id=168631
I'm using 12.0.1 (136<IP_ADDRESS>.2), maybe they fixed it, which I highly doubt.

I moved sorting key from LS to cookies in e09678c. It must work fine now.

Works for me

yep, it works


Just a quick wonder: why resize to 1333, 800

most configuration file  have
img_scale=[(1333, 800)]
or
dict(type='Resize', img_scale=(1333,800), keep_ratio=True)
wonder how's that come from?

@hellock

In Fast RCNN, Faster RCNN (without FPN) paper: shorter side s=600
In FPN paper: shorter side s=800
Feature Pyramid Networks for Object Detection


[Bug fix] Fix mathematica settings

Mathematica settings had a typo; they would refer to Kotlin's settings instead of their own. I'm not sure how/why that happened, but this is a fix for it.

Thank you for this fix! The blame is on me, I added it and forgot to test it...


Suggestion: Solar generator power output varies with sun proximity/brightness

Basically, the brighter the sun is relative to the solar generator, the more power it produces. If you were in an orbit close to the sun, solar generators would produce a huge amount of power. If you were in the outer solar system, you would need a giant array of panels just to produce a few RF per tick.

So, basically the brighter the star and the closer the orbit of the planet, the more power output?

So, basically the brighter the star and the closer the orbit of the planet, the more power output?

Yes, exactly.

So, basically the brighter the star and the closer the orbit of the planet, the more power output?

Yes, exactly.

This is a great idea.

Possibly compatability with other solar panels like those from NuclearCraft could be added.


Runtime network not ready: NetworkReady=false reason:NetworkPluginNotReady message:docker: network plugin is not ready: cni config uninitialized

I deployed Rancher cluster v2.0.0. on a private network and none of the nodes are available due to the following error:
Runtime network not ready: NetworkReady=false reason:NetworkPluginNotReady message:docker: network plugin is not ready: cni config uninitialized
Tried all available networks (Canal, Calico, Flannel) without any success.
Could this be related to self signed certificates? And if yes what is the proper way to install them?
Instructions on setting up SSL from this link did not work:
https://medium.com/@superseb/ssl-tls-options-for-rancher-2-0-dca483a7070d

Useful
Info

Versions
Rancher v2.0.0 UI: v2.0.41

Access
local admin

Route
authenticated.cluster.index

Usually a network problem. How did you deployed your Rancher cluster? Instructions for setting it up are here: https://rancher.com/docs/rancher/v2.x/en/installation/single-node-install/#choose-how-you-want-to-use-ssl
Be sure to cleanup nodes when re-using them in a setup: https://gist.github.com/superseb/2cf186726807a012af59a027cb41270d
If you have done all of that, usually the pod status shows what's going on:
kubectl get pods --all-namespaces

Thank you it now works.
FYI for anyone using cleanup script. It wipes out all docker containers and volumes. Not safe for nodes in use for other purposes.

Removing the $KUBELET_NETWORK_ARGS in /etc/systemd/system/kubelet.service.d/10-kubeadm.conf works for me.

Hello,
This problem also occurs when deleting a node from a cluster then restarting it. Running the cleanup script, rebooting the host does not fix the problem. Does anyone have a similar problem?

same issue.
E1025 09:02:22.614850   11254 kubelet.go:2167] Container runtime network not ready: NetworkReady=false reason:NetworkPluginNotReady message:docker: network plugin is not ready: cni config uninitialized
W1025 09:02:27.617176   11254 cni.go:188] Unable to update cni config: No networks found in /etc/cni/net.d
E1025 09:02:27.617475   11254 kubelet.go:2167] Container runtime network not ready: NetworkReady=false reason:NetworkPluginNotReady message:docker: network plugin is not ready: cni config uninitialized
W1025 09:02:32.619714   11254 cni.go:188] Unable to update cni config: No networks found in /etc/cni/net.d

and this is my solution.
mkdir -p /etc/cni/net.d
cat > /etc/cni/net.d/10-flannel.conflist <<EOF
{
    "name": "cbr0",
    "plugins": [
        {
            "type": "flannel",
            "delegate": {
                "hairpinMode": true,
                "isDefaultGateway": true
            }
        },
        {
            "type": "portmap",
            "capabilities": {
                "portMappings": true
            }
        }
    ]
}

Wow, @laoshancun's solution fixed the issue instantly.  Is this something that should be PR'd into a release?

@seffyroff  thanks bro...

@laoshancun cool, thanks bro.

@laoshancun awsome ! Fixed my problem perfectly : )

@laoshancun Did cat > /etc/cni/net.d/10-flannel.conflist need to be added to every node.

@spstratis   You should make sure that /etc/cni/net.d/ is empty.

@laoshancun In my case it currently is empty, on the master node I was looking at. So based on your solution I would need to add 10-flannel.conflist to the net.d folder, right?

@spstratis I will say yes if you got this error cni config uninitialized.

@laoshancun thanks! and that would need to be on every node?

@spstratis Only on the nodes that have same trouble.

In directory I see that i have canal files. I am not using the fannel. What to do in that case.

For anyone running Rancher on RHEL 7 on AWS GovCloud, is appears that default Amazon RHEL AMI disables bridging.  After enabling bridging in /etc/sysctl.conf, one needs to go into /usr/lib/sysctl.d/ and either remove the default bridging rules or enable them from within the 00-system rule file otherwise the settings in /etc/sysctl.conf will be overriden.  Hope this helps someone in the future.

TO anyone out there finding this issue and the solution of creating the 10-flannel.conflist file doesn't work, I was able to fix it right away by installing calico CNI by just running
kubectl apply -f https://docs.projectcalico.org/v3.11/manifests/calico.yaml
After some seconds all nodes were in ready state.

To anyone out there running rancher 2.4.8 on CentOS 7 in combination with calico.
I had to create a /etc/resolv-kubernetes/resolv.conf with my nameservers in it.


fix position of > after anon type

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@a96925c). Click here to learn what that means.
The diff coverage is 100%.

@@           Coverage Diff           @@
##             master   #115   +/-   ##
=======================================
  Coverage          ?    73%
=======================================
  Files             ?     39
  Lines             ?   3241
  Branches          ?     52
=======================================
  Hits              ?   2366
  Misses            ?    840
  Partials          ?     35

Impacted Files
Coverage Δ

src/tokentree/walk/WalkBlock.hx
94.91% <100%> (ø)

Continue to review full report at Codecov.

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


Misc (non-code) spelling fixes

Like #10715, not sure if we care about non-user-visible-spelling pre V1, but these are non-code so lower risk.

LGTM, @quinton-hoole for another

I guess we don't need 2 lgtms for doc fixes

LGTM


look into sqlx

figuring out how to do simple stuff in diesel is giving me a lot of trouble. here's my first attempt at splitting out a portion of a WHERE clause so that it can be reused:
pub fn alive_tasks<'a>() -> Box<dyn BoxableExpression<task::table, Pg, SqlType = Bool> + 'a> {
    Box::new(task::mode.eq(any(vec![MODE_PENDING.0, MODE_ACTIVE.0])))
}

absolutely hideous.
also, afaict i can't do this:
    let query = task::table.find(task_id);
    if !include_deleted {
        query = query.filter(task::mode.ne(MODE_DELETED.0));
    }

because the second query value has a different type from the first.
hopefully i'm just doing stuff wrong and diesel is actually great! but comradeblue in irc uses https://github.com/launchbadge/sqlx , so i should do some research on that someday. low-priority.

got help from the diesel maintainer via gitter, he suggested using a typedef for the first one and .into_boxed() for the second. ended up with this:
pub fn alive_tasks<'a>() -> SqlExpr<'a, task::table, Bool> {
    Box::new(task::mode.eq(any(vec![MODE_PENDING.0, MODE_ACTIVE.0])))
}

and
    let mut query = task::table.find(task_id).into_boxed();
    if !include_deleted {
        query = query.filter(task::mode.ne(MODE_DELETED.0));
    }

which seems p reasonable! so this sqlx investigation ticket is super non-urgent / low-priority


RedissonRateLimiter tryAcquire and availablePermits throw RedisException: too many results to unpack

Expected behavior
method call should return without exception
Actual behavior
A RedisException is thrown:
org.redisson.client.RedisException: ERR Error running script (call to f_d66032e767a3ea7814ca959c0a81ae0baa165f75): @user_script:1: user_script:1: too many results to unpack . channel: [id: 0xf4ad5dc9, L:/<address> - R:<address>] command: (EVAL), params: [local rate = redis.call('hget', KEYS[1], 'rate');local interval = redis.call('hget', KEYS[1], 'inter..., 5, MyRateLimiter_107728585506707301, {MyRateLimiter_107728585506707301}:value, {MyRateLimiter_107728585506707301}:value:b3c4bb40-b4ac-46f8-a6f7-4f8161b4842..., {MyRateLimiter_107728585506707301}:permits, {MyRateLimiter_107728585506707301}:permits:b3c4bb40-b4ac-46f8-a6f7-4f8161b48..., 1,<PHONE_NUMBER>119, 606081525...
Steps to reproduce or test case

Add more than 8k entries to rate limiter.
Wait for all of them to expire.
call availablePermits/tryAcquire.

RRateLimiter rateLimiter = redissonClient.getRateLimiter("test"); rateLimiter.setRate(RateType.OVERALL, 10_000, 2, RateIntervalUnit.MINUTES); for (int i = 0; i < 8_000; i++) {     rateLimiter.tryAcquire(); } try {     Thread.sleep(2*60*1000); } catch (InterruptedException e) {     e.printStackTrace(); } long check = rateLimiter.availablePermits(); //throws the unwanted exception ... boolean check2 = rateLimiter.tryAcquire(); //also throws the unwanted exception ... System.out.println("result=" + check +", " + check2);
Redis version
6.2.1
Redisson version
3.14.1
Redisson configuration
singleServerConfig:
idleConnectionTimeout: 10000
connectTimeout: 10000
timeout: 3000
retryAttempts: 3
retryInterval: 1500
subscriptionsPerConnection: 5
sslEnableEndpointIdentification: true
sslProvider: "JDK"
pingConnectionInterval: 20000
keepAlive: true
tcpNoDelay: false
address: "redis://localhost:6379"
subscriptionConnectionMinimumIdleSize: 1
subscriptionConnectionPoolSize: 50
connectionMinimumIdleSize: 2
connectionPoolSize: 8
database: 4
dnsMonitoringInterval: 5000
threads: 16
nettyThreads: 32
codec: !<org.redisson.codec.JsonJacksonCodec> {}
referenceEnabled: true
transportMode: "NIO"
lockWatchdogTimeout: 30000
reliableTopicWatchdogTimeout: 600000
keepPubSubOrder: true
useScriptCache: false
minCleanUpDelay: 5
maxCleanUpDelay: 1800
cleanUpKeysAmount: 100
nettyHook: !<org.redisson.client.DefaultNettyHook> {}
useThreadClassLoader: true
addressResolverGroupFactory: !<org.redisson.connection.DnsAddressResolverGroupFactory> {}

Looks like similar issue was fixed here: https://github.com/redisson/redisson/issues/2014

No, it's a different one. Fixed

Thanks!


add tcc to dockerfile

Apparently, tcc is just available in the Ubuntu repos (in reference to mattgodbolt/gcc-explorer#82). That being said, it does not support C++, so that could be a visible limitation.

We can give it a go!

tcc doesn't support a "output assembly" mode which means it won't work with non-Binary GCC explorer. I'll have to have a think about how this could work: it's already the case that some compilers don't support binary-only mode. This would be the first to only support binary mode, which means the UI is even more complicated and unintuitive.


Rahul raghuwanshi/UI improvement

improve recyclerview item layout

Can you attach the screenshot or a screen recording of the new UI?

Here is a video
https://user-images.githubusercontent.com/52368582/194605629-fd904b30-e698-45b4-ae88-60dbff1db35a.mp4

Thanks and Kudos for making another contribution to MyNotes!
I'll be sharing yours and others contribution on Twitter and LinkedIn shortly. If you like do connect with me at @avidraghav on both Twitter and LinkedIn

Actually i am not on twitter but on linkedin i start following you

And also can you please label my pr's with hacktoberfest accepted


WIP: Adding a Grafana 8 case to addpanel selectors

What this PR does / why we need it:
Adding addPanel case that might work with >=8 <=8.5 Grafana versions
Using PR to get canary versions to test.

any idea why canary package is not being published? @jackw @sunker

Closing in favor https://github.com/grafana/grafana/pull/97210


Fix rbac issue

add namespace to work

@richardwxn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_operator
9a922af8465acdabe9334ca3e8066cbbbc0675d4
link
/test gencheck_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

The namespaces are not redundant.  Please add the istio-operator namespace to all objects in the repo.  See PR https://github.com/istio/istio.io/pull/5142 for why this is necessary.
Cheers
-steve

@richardwxn service.yaml also in need of a namespace.

subsumed by: https://github.com/istio/operator/pull/374

@richardwxn I can handle expanding RBAC for the operator here: https://github.com/istio/istio/issues/17841.  If you still suffer from a PDB issue, please paste the log message here or message to me on slack and I'll take a look.  If your satisfied with https://github.com/istio/operator/pull/374 - please close this issue and leave a comment.
Cheers
-steve

@sdake thanks for the clarification! I am closing this PR now as yours was merged already. I would send the error log to you on slack tmr.


Add License

@LucasGandel
It is great to see all the work you have done so far :100:
That said the project seems to be missing a license file. :scream:
No worry, I suggest you follow these few steps:

[x] :one: Choose a license. We suggest you use a permissive license that includes patent and contribution clauses. This will help protect developers and ensure the code remains freely available. We suggest you the Apache 2.0. Read here to learn more about licenses.

[x] :two: Add the the license file to your project. This guide may be helpful.

[x] :three: Add a section like the following at the end of your project's README.md file:
:warning: Make sure to update the abcdefghi and YYYY-MM-DD placeholders respectively with the short commit SHA and the commit date corresponding to step 2 above
## License

This software is licensed under the terms of the [Apache Licence Version 2.0](LICENSE).

The license file was added at revision [abcdefghi](https://github.com/Slicer/slicer_download/commit/abcdefghi) on YYYY-MM-DD, but you may consider that the license applies to all prior revisions as well.

Background
Without a license file, interpreting the licensing terms becomes complex. In a nutshell, code hosted on GitHub without a license is likely made available through section D. 5. License Grant to Other Users of the GitHub terms of service.
Reading issue github/choosealicense.com#196 (No license" does not allow commercial or private use) is also informative.
Links

Description
Link

Which open source license is appropriate for my project?
https://opensource.guide/legal/#which-open-source-license-is-appropriate-for-my-project

Adding a license to a repository
https://docs.github.com/en/communities/setting-up-your-project-for-healthy-contributions/adding-a-license-to-a-repository

All About Open Source Licenses
https://fossa.com/blog/what-do-open-source-licenses-even-mean/

Choose an open source license
https://choosealicense.com/

No License
https://choosealicense.com/no-permission/

Disclaimers
I am not a lawyer. As such, my suggestions can not serve as legal advice.

This comment was adapted from my GitHub saved replies, it is licensed under a Creative Commons Attribution 4.0 International License and you are welcome to reuse it.

Fixed in e184c198 and #2


update Dockerfile, docker-compose and three nodes config

 update Dockerfile, docker-compose and three nodes config light-controller master


Enable-WriteCore-feature-for-batch

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.Batch


Feature/holo 1480 fix get owner

Describe Changes
⚠ DON'T MERGE

This was just a temporary solution to help get tests passing, but needs to be handled more appropriately a different way


Adds a tileDisplay param to override default fade-in behaviour

Hi @tprebs - hope you're well!
Our requirements are for the heatmap layer to appear instantaneously rather than using the TileLayer's 100ms fadeIn delay, as we update heatmap data regularly.
This PR adds a new tileDisplay parameter, so that this default behaviour can be changed.
If a tileData value is not provided, it will default to TileDisplay.fadeIn() which is the current default behaviour, so anyone who doesn't provide this will see no change in functionality.

Now available in release 0.0.4+2


Cannot sync locally

Error ID
8bff26ef-340f-4b4b-b093-98cb3db20bb1
What happened?
I got 2 errors while trying to sync local files.

When I select merge files I got the error below:

"GGT_CLI_UNEXPECTED_ERROR: An unexpected error occurred
Error: EISDIR: illegal operation on a directory, read"
I do have read and write access. I tried running the command with sudo but the the browser did not open to login.

When I select reset local files, I got the error below:

[RangeError: path should be a path.relative()d string, but got ".."
This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason:
RangeError: path should be a path.relative()d string, but got ".." ]
What did you expect to happen?
I expected to sync files I guess.
Steps to reproduce
try to sync.
Anything else?
Running on Ubuntu 22.04

Hey @mstekinalp 👋 -- sorry for the bugs 😞
I just released v0.1.16 which includes https://github.com/gadget-inc/ggt/pull/414 that fixes the first issue.
You can install the latest version via:
npm install --global @gadgetinc/ggt

I'm not sure why the second issue is occurring. I'm hoping that the second issue is caused by the first and both are resolved.
If either issue persists after upgrading, please re-open this issue.

@scott-rc Thanks. I appreciate the quick help. It seems the issue is resolved.


Issue #261: Bring phone-number into compliance with canonical data

Added tests for cases we hadn't covered and changed example implementation to work in those cases. This should solve issue #261 .

This looks good to me! I notice that the final three tests are not included in the canonical data - are we ok with leaving these additional tests in the suite? (Note that they also increase the API surface of the implemented class).

Yes, I did notice and wonder about that. I don't know why they were originally introduced so I didn't want to remove them without checking. I don't mind them being there, they don't add too much difficulty to the exercise and they make it a bit more interesting. @jtigger, @matthewmorgan do you think they should be there?

(Sorry for the delay, I missed the @-mention.)
While they are interesting, I agree, they seem to fall outside the intended scope of the exercise.  The description even establishes why we're scrubbing the phone number: "... so that they can be sent SMS messages."  That one phrase makes the .pretty() and .getAreaCode() seem like they're just hanging out for no apparent reason.
I look at it from the practitioner's point of view and pretend I'm a designer (that last part takes a lot of imagination).  I ask, "how's does this exercise make the practitioner feel?"  Cohesion of problem (and therefore effort) makes me feel like I'm solving a whole problem; the lack thereof makes me feel like I'm being asked to do busywork.

Okay, that makes sense. I'll remove those tests then :)

Thanks for weighing in, @jtigger, and for the follow-up changes, @FridaTveit! LGTM :)


[BUG] Flowise freezes UI and takes 300+ seconds to answer

Describe the bug
I don't know why, but seems that, after the fourth or fifth message, flowise, both self hosted and cloud version, halt the execution and take this much long to finish the run.
Doesn't seem like a rate limiting from the OpenAI but something within flowise, because smaller sequential agents flows won't present such beahavior, not as often at least.
LangSmith Report
To Reproduce
Steps to reproduce the behavior:

Ask anything for the flow, until it reaches the fourth or fifth message, through UI or API
It freezes and hangs until it quits or finishes after 300 seconds or so.

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.

Flow
If applicable, add exported flow in order to help replicating the problem.
Time Protocolo da autoconfiança Agents (3).json
Setup

Installation Docker
Flowise Version 2.1.1
OS: Ubuntu 20.04
Browser opera

Additional context
Help

thats a nice graph! I think something about the retriever is not working correctly, since you can see alot of not making sense response, and this causes ChatGPT not able to respond correctly:


How can we use it for multiple faces ?

If there is multiple faces , it is not displaying output for all faces , How can we use it for multiple faces ?

change this line:
faces = face_detection.detectMultiScale(gray, scaleFactor=1.1, minNeighbors=5, minSize=(30, 30),
flags=cv2.HOUGH_MULTI_SCALE)  # CASCADE_SCALE_IMAGE

Details

bug


Make local pacman use flexo's cache

It would be really great to somehow make local pacman use flexo's cache as a cache directory for local system updates;
So for example:

Suppose I have a certain package in my flexo cache and I wish to pacman -S that package,
My suggestion is to not indulge pacman to download this certain package from localhost flexo service (thus creating a wasteful copy of the file),
But instead to directly install it from flexo's cache!

I've though of few approaches:

Encourage pacman to officially support storing and using its' system cache in flexo's format (aka the mirror-able tree structure).
Include a configurable feature in flexo that will create a symlink for every package;

In /var/cache/flexo/pkg/ create a new directory (lets call it ./pacman/ for demonstration);
Symlink all latest versions of all packages into /var/cache/flexo/pkg/pacman/*
Point into the new cache directory in your local /etc/pacman.conf

Edit:
I would gladly contribute but unfortunately i don't know Rust (yet!?).

So far, I've dealt with this problem by setting the CacheDir of pacman to /tmp/pacman_cache. This means files are still stored redundantly, but are removed after each reboot due to /tmp being mounted as tmpfs. Also, it's all stored in RAM, so you don't have any additional wear-and-tear on your SSD. But I understand that this is not ideal if you don't have enough RAM to waste.
It might be possible to solve this problem without making any changes to Flexo: according to man 5 pacman.conf, it's possible to set multiple directories as CacheDir:

Multiple cache directories can be specified, and they are tried in the order they are listed in the config file.

However, currently it does not work to just set something like:
CacheDir  = /tmp/pacman_cache /var/cache/flexo/pkg/community/os/x86_64 /var/cache/flexo/pkg/core/os/x86_64 /var/cache/flexo/pkg/extra/os/x86_64 /var/cache/flexo/pkg/multilib/os/x86_64

But this seems to be due to the fact that Flexo currently does not cache .sig files. So, if pacman doesn't find the .sig files (along with the actual package file) in the local file system, it fetches both the .sig file and the package file via HTTP.
I've created a separate issue for this (#65). As soon as Flexo caches .sig files, I'll have a closer look at this.

Thanks for the reply!
Firstly I learned something new today, didn't knew I could specify multiple cache directories in pacman.conf.
You wrote:

However, currently it does not work to just set something like:
CacheDir  = /tmp/pacman_cache /var/cache/flexo/pkg/community/os/x86_64 /var/cache/flexo/pkg/core/os/x86_64 /var/cache/flexo/pkg/extra/os/x86_64 /var/cache/flexo/pkg/multilib/os/x86_64

I found out that using it as:
CacheDir = /tmp/pacman_cache
CacheDir = /var/cache/flexo/pkg/core/os/x86_64
CacheDir = /var/cache/flexo/pkg/extra/os/x86_64
... etc ...

Is actually a valid solution for the local pacman.. !
I've tested it with a single package that I've downloaded from remote machine that used Flexo as its mirror, and then tried to install  that same package on the Flexo server, it worked thus it did not seem that .sig was required...
Although I've seen in /var/cache/pacman that only some packages are being fetched with .sig files alongside, which makes me wonder whether you are right about pacman falling-back to downloading when it will not find a .sig alongside a particular package.
you may close.

Sounds good! I haven't investigated if and why pacman was really falling back to downloading the entire file when the .sig file is missing, but either way, with the newest release, Flexo caches .sig files just like normal package files, so this should not be a problem anymore.
Just to reiterate: If you want pacman to directly fetch existing files from the file system instead of downloading them via HTTP and storing a redundant copy, modify your /etc/pacman.conf to use multiple CacheDir entries, for example:
CacheDir = /var/cache/pacman/pkg
CacheDir = /var/cache/flexo/pkg/community/os/x86_64
CacheDir = /var/cache/flexo/pkg/core/os/x86_64
CacheDir = /var/cache/flexo/pkg/extra/os/x86_64
CacheDir = /var/cache/flexo/pkg/multilib/os/x86_64

If you use any additional repositories, you need to add more CacheDir entries.
You might also want to replace the first CacheDir entry with:
CacheDir = /tmp/pacman_cache

to avoid redundant package files: /tmp is usually mounted as tmpfs, so the files disappear after reboot, which is fine since the files are still stored inside /var/cache/flexo.


Fb transpiler

-YAML elements file

Implemented some elements
Added more markdown stuff

Before we merge this, can I suggest the same thing I did on #242: could we put a single blank line between each element so it's easier to read?
- element: "λ"
  name: Lambda
  arity: "*"
  description: Open a lambda - λ...;

- element: "ƛ"
  name: Lambda Map
  arity: 1
  description: Open a mapping lambda - ƛ

- element: "¬"
  name: Logical Not
  arity: 1
  description: Return the inverse (negation) of the truthiness of an item.
  overloads:
    num: not a
    str: a != "" | len(a) > 0
    lst: a != [] | len(a) > 0
  vectorise: false
  tests:
    - "[1] : 0"
    - "[0] : 1"

# ...

I agree with @pxeger

No.
We have a bunch of digraphs left to document. I'll probably finish them within a day or two, maybe less if someone else helps, but until then, no.

Before we merge this, can I suggest the same thing I did on #242: could we put a single blank line between each element so it's easier to read?
- element: "λ"
  name: Lambda
  arity: "*"
  description: Open a lambda - λ...;

- element: "ƛ"
  name: Lambda Map
  arity: 1
  description: Open a mapping lambda - ƛ

- element: "¬"
  name: Logical Not
  arity: 1
  description: Return the inverse (negation) of the truthiness of an item.
  overloads:
    num: not a
    str: a != "" | len(a) > 0
    lst: a != [] | len(a) > 0
  vectorise: false
  tests:
    - "[1] : 0"
    - "[0] : 1"

# ...

done

@vyxal-cookie I don't like that you keep doing actions on GitHub shortly after I do. It makes me look very suspicious

:(


cxf extension creation

fix #764

hello @dufoli, I would be happy to try it out with our project, any updates so far?

@dufoli Do you mind if I take over this project?

I think this could be suppressed by https://github.com/apache/camel-quarkus/pull/3931


Support extension field numbers

Current dependencies on/for this PR:

main

PR #329  (this PR)

This comment was auto-generated by Graphite.


Breaking change: edit, edit-2, and edit-3 removed in 0.262.0

Package

[X] lucide
[ ] lucide-angular
[ ] lucide-flutter
[ ] lucide-preact
[ ] lucide-react
[ ] lucide-react-native
[ ] lucide-solid
[ ] lucide-svelte
[ ] lucide-vue
[ ] lucide-vue-next
[ ] Figma plugin
[ ] source/main
[ ] other/not relevant

Version
0.262.0
Browser

[ ] Chrome/Chromium
[ ] Firefox
[ ] Safari
[ ] Edge
[ ] iOS Safari
[ ] Opera
[X] Other/not relevant

Operating system

[ ] Windows
[ ] Linux
[ ] macOS
[X] Other/not relevant

Description
The edit, edit-2, and edit-3 icons were removed in version 0.262.0. Possibly as part of https://github.com/lucide-icons/lucide/pull/1286.
While this change may make sense, it is a breaking change and should have been versioned as a new major version and communicated as such.
Question 1: Can we get these icons back?
Question 2: Are there any guardrails or documentation that will prevent breaking changes from being introduced in minor versions?
Steps to reproduce

Search for  edit, edit-2, and edit-3 icons in a release > 0.261.0
Notice that the icons are missing

Checklist

[X] I have searched if someone has submitted a similar issue before and there weren't any. (Please make sure to also search closed issues, as this issue might already have been resolved.)

Question 1: Can we get these icons back?

They have only been moved https://github.com/search?q=repo%3Alucide-icons%2Flucide edit-3&type=code

Question 2: Are there any guardrails or documentation that will prevent breaking changes from being introduced in minor versions?

What we need to do:

Add alias support to that package.
https://github.com/lucide-icons/lucide/issues/1550

Question 1: Can we get these icons back?

They have only been moved https://github.com/search?q=repo%3Alucide-icons%2Flucide edit-3&type=code

Question 2: Are there any guardrails or documentation that will prevent breaking changes from being introduced in minor versions?

What we need to do:

Add alias support to that package.
Build release workflow for after 1.0.0 #1550

"They have only been moved", I'm not sure what you mean by this. They are not present:
`<!doctype html>

`
Yields an empty page. This is a breaking change to anyone who was using these icons.

This it because we didn't implemented aliases in the lucide package yet.
I will make some time to implement the icon name aliases, after that this will work.

@life4aiur Nope lucide-static is something different. But we should work on that one as well!

Also noticed this breaking change in a production application. Following for updates... for now I temporarily switched to a similar Fi icon.


feat: make hostNetwork configurable

disable it by default (fixes port conflict issue on the node if another port is trying to acquire the same port)

Why
We use hostNetwork: true by default which causes problems in the following scenario:

another-csi-driver pod is running on the same node A
another-csi-driver uses port 8080
warm-metal pod is scheduled on node A
warm-metal pod tries to use port 8080 as well (for metrics endpoint) but fails because the port is already in use

This happens because we use hostNetwork: true. Based on my tests and analysis (along with integration tests), we don't really need hostNetwork because we are not accessing anything on the node nor is the node accessing anything on the warm-metal pod directly.
More info
hostNetwork has been a part of the initial commit: https://github.com/warm-metal/csi-driver-image/commit/515cec1b144bda269a87ec78f46203f5686dfc43

The liveness probe is a sidecar container that exposes an HTTP /healthz endpoint, which serves as kubelet's livenessProbe hook to monitor health of a CSI driver.

The liveness probe uses Probe() call to check the CSI driver is healthy. See CSI spec for more information about Probe API call. Container Storage Interface (CSI)

https://github.com/kubernetes-csi/livenessprobe/tree/master?tab=readme-ov-file#liveness-probe
So kubelet basically executes livenessProbe which pings the healthz endpoint to check if the CSI driver is alive and livenessprobe server forwards that request to the CSI driver's Probe() endpoint to check if the driver is healthy.
https://github.com/kubernetes-csi/livenessprobe/blob/ebd49b57031ab90f6f376f1472884948f381558c/cmd/livenessprobe/main.go#L169 -> https://github.com/kubernetes-csi/livenessprobe/blob/ebd49b57031ab90f6f376f1472884948f381558c/cmd/livenessprobe/main.go#L73
I don't see a need for warm metal to use hostNetwork since it doesn't need to access anything on the node.
warm-metal can access all the unix sockets via volumeMounts:
...
    volumeMounts:
    - mountPath: /csi
      name: socket-dir
...
    - mountPath: /run/containerd/containerd.sock
      name: runtime-socket
...
  volumes:
  - hostPath:
      path: /var/lib/kubelet/plugins/csi-image.warm-metal.tech
      type: DirectoryOrCreate
    name: socket-dir
...
  - hostPath:
      path: /run/containerd/containerd.sock
      type: Socket
    name: runtime-socket
...
  - name: kube-api-access-xst7n
...

ref: https://github.com/warm-metal/csi-driver-image/blob/master/charts/warm-metal-csi-driver/templates/nodeplugin.yaml

It would be good if we could also update other references for hostNetwork and set it to false for maintaining consistency.

I have replaced the occurrences.

Will ping you here once the CI finishes successfully.

@mugdha-adhav can you review the PR again.

@mugdha-adhav can I ask for a review again 🙏


Patch upstream to keep our modified proto files

Summary
Upstream master switched to strictly using the tendermint spec as the single source of proto files #6976. We need to create a patch that will do the same, except keep our modifications to those files.
References
#6976
#343

Repo at: https://github.com/celestiaorg/spec

Is this issue needed anymore?

@evan-forbes Hello, can I take this on?

Certainly!


为什么在设计log时，加上了Index0？

I also have  question：maybe there is no difference between  idx in "Entries[idx]" and index in "Entries[idx].index" ?

Hi @tiruisi ,  logs may not start from 0, when log compression is enabled.


Botton back doesn't work in some cases.

Hi,
Exploring some sub-folder, reaching one empty sub-folder, the system cannot get back from it.
https://github.com/gusmanb/micropicodrive/assets/33696002/de9a0643-b0f7-4508-9eb6-2fc9ccfbf7ee

Will test it once I get a bit of spare time, right now I'm really busy until christmas.

Good afternoon!
Just in case my suspects are right... modifying my version of your firmware I got the candidate for the bug.
If you consider to modify yours, that is what I understood.
It join a infinite loop.

Thanks

Ok, i have corrected it. Thanks.


Handle the case that a parameter group does not have label

According to the spec the heat template (HOT) parameter group should have a label and description. Our service dialog generation tool uses the label for a tab label.
Here is the spec requirement:
parameter_groups:
- label: <human-readable label of parameter group>
  description: <description of the parameter group>
  parameters:
  - <param name>
  - <param name>

However our customer has a template without label and description which causes the dialog generation failure.
parameter_groups:
- parameters:
  - key_name
- parameters:
  - flavor
  - image

The fix is to automatic create a label for non-labeled parameter group.
In order to test the fix we purposely deleted the label and description from the sample template.
Links [Optional]

http://docs.openstack.org/developer/heat/template_guide/hot_spec.html#parameter-groups-section
https://bugzilla.redhat.com/show_bug.cgi?id=1378976

Steps for Testing/QA [Optional]
Use a heat template that has parameter group but does not have label and description. You manually delete them like the our test sample.

Euwe Backport details:
$ git log
commit 2d3355c06f9fe68dbf17047f6ae1c2ceb5a40de8
Author: Greg McCullough<EMAIL_ADDRESS>Date:   Fri Sep 30 10:22:09 2016 -0400

    Merge pull request #11589 from bzwei/heat_parameter_groups

    Handle the case that a parameter group does not have label
    (cherry picked from commit 14f5a684a2246fff4ebede06551b52408cccd5f8)

@simaishi need Darga BZ

Darga BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1382819

Darga Backport details:
$ git log
commit b4186e13e4dfb62935b1c81aa9cd3f573c157f1a
Author: Greg McCullough<EMAIL_ADDRESS>Date:   Fri Sep 30 10:22:09 2016 -0400

    Merge pull request #11589 from bzwei/heat_parameter_groups

    Handle the case that a parameter group does not have label
    (cherry picked from commit 14f5a684a2246fff4ebede06551b52408cccd5f8)

    https://bugzilla.redhat.com/show_bug.cgi?id=1382819


Add is-active class to div.nav-folder-title when folder note is active

Hey Aiden,
First of all, thank you for an amazing plugin!
I think it would make sense to style the folder as 'active' in a similar way to how Obsidian styles nav-files, such that a is-active class is added.
This would allow the user to more easily see that the folder note is open.

Was just thinking this morning that this would be a great feature! I ran into a weird edge case: if my folder note is a kanban board using obsidian-kanban, then is-active is not added. I can make a repo with a sample vault if that would be helpful.


test commit directly to main branch

Git push directly to main branch confirmed not working.
Testing of the pull request works. If successful. Issue https://github.com/cmtr/cmtr-static-public/issues/1 completed

Change the setting to remove rule requirement for 1 approver (as self approving will be the norm)

Also changed the settings to remove the requirement for all to be singed. Need to look more closely into what signed means


Add the ability to change suite name in runtime

Needed in case the same suite runs for different browsers, databases, etc.

@kermek such ability will be avaliable since Allure 2.0 I guess.


Using vm in directive without isolated scope

I want to use Your controllerAs method with directive that not has isolated scope.
How to get outer controller's vm object from inner controller?
Here is example http://jsfiddle.net/5ew48u8b/15/ - look for comment.
Edit: I've updated example.

Ok this is problem with transclude and scope (directive2 is not a child of directive1)


service mem_limit is not always a string

In https://github.com/kontena/kontena/blob/master/cli/lib/kontena/cli/services/services_helper.rb#L276 the mem_limit is assumed to be a string (@param [String] memory)
But if one follows the kontena docs and puts mem_limit:<PHONE_NUMBER> then this method will explode in kontena app deploy --no-build with
undefined method `end_with?' for<PHONE_NUMBER>:Fixnum

even if this case is support and handled at end end of the method (no suffix).
There are two options to fix this:
a) change the assumption so that the string is ensured with .to_s - might be confusing later on and is a code smell
b) change the YAML result parsing so that memory is always parsed as a string - don't know if something else will start failing because of this.
How should this be fixed?

c) Don't support mem_limits without a suffix (best?)


Question about GSA

Hello, thank you very much for your excellent work. I have some questions about GSA. According to my personal understanding, GSA in the paper takes one representation from each window, so the sr_ratio should be the same as the window size ([7, 7, 7, 7]) when calculating Key and Value, but it is [8, 4, 2, 1] in the code. Is there anything wrong with my understanding?
@BACKBONES.register_module()
class alt_gvt_large(ALTGVT):
    def __init__(self, **kwargs):
        super(alt_gvt_large, self).__init__(
            patch_size=4, embed_dims=[128, 256, 512, 1024], num_heads=[4, 8, 16, 32], mlp_ratios=[4, 4, 4, 4], qkv_bias=True,
            norm_layer=partial(nn.LayerNorm, eps=1e-6), depths=[2, 2, 18, 2], wss=[7, 7, 7, 7], sr_ratios=[8, 4, 2, 1],
            extra_norm=True, drop_path_rate=0.3,
        )

Thanks for your attention.
You are right and e will make it more clear in the next version of the paper.
If we use 77 in the last  stage, there will be only 1 key (the feature map is 77).  Therefore, in such case GSA is normal global self attention (see the code).
As for stage 3 (feature size 14*14), if we use sr=7, there will be only 4 keys, which will limit the representative power of the network.
In general, GSA is a mechanism of collecting global information efficiently. In fact, you can try different sr_ratios in your project.

Thanks for your attention.
You are right and e will make it more clear in the next version of the paper.
If we use 7_7 in the last stage, there will be only 1 key (the feature map is 7_7). Therefore, in such case GSA is normal global self attention (see the code).
As for stage 3 (feature size 14*14), if we use sr=7, there will be only 4 keys, which will limit the representative power of the network.
In general, GSA is a mechanism of collecting global information efficiently. In fact, you can try different sr_ratios in your project.

I see. Thanks for your reply.


[Feature Request] Add Ability to Set Timestamp in EventSub Commands

I think it would be useful to be able to test for when an EventSub notification is old/stale.
Not really sure exactly what this would look like.  Maybe a flag on the trigger command to set the timestamp some about of time back, or to set a specific time?

Just to be clear, are you trying to test this use case from https://dev.twitch.tv/docs/eventsub/handling-webhook-events#processing-an-event:

Replay attacks
If you need resiliency against replay attacks, consider the following:

Make sure the value of the Twitch-Eventsub-Message-Timestamp request header isn’t older than 10 minutes.
Make sure you haven’t seen the ID in the Twitch-Eventsub-Message-Id request header before.

If not, is there another use case I may have missed?

Yeah that's the key use case I'd like to test

Should be added next release

Awesome, thanks for getting to that!


Update GitLink tool to use GitLink 3.0.0

This Pull Request relates to https://github.com/cake-build/cake/issues/1734 and updates the GitLink tool to reflect the major restructure that was done in GitLink 3.0.0.
Instead of taking a solution directory, the new GitLink tool takes singular pdb files and patches them. I've added a helper alias to allow passing a FilePathCollection to allow globbing.

@Cheesebaron,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

This will be a breaking change for anyone not using GitLink 3.0, right? Any possibility to introduce a GitLink3-runner or similar?

@patriksvensson correct. This could be made as a different runner and alias. I have no issue with doing that instead.
@devlead will fix that.

Agree with @patriksvensson, could you keep old aliases and add new 3 editions

Should the AliasCategory still be GitLink or a separate GitLink3 one instead?

 do seperate like NUnit

Done. Old Gitlink stuff is as is. I've added version pinning in the XML docs for the v2 stuff.
I've renamed all the new GitLink stuff to GitLink3 and made a category called GitLink v3, similar to NUnit.

@devlead & @mholo65 anything else you guys want me to change? I think I have addressed both the issues you pointed out. I've rebased the PR so many times now, I can rebase it again when you think it is ready to merge.

@Cheesebaron your changes have been merged, thanks for your contribution 👍


Cannot wrap JSON views due to paging parent DIV ms-List-page

Category

[ ] Question
[x] Bug
[ ] Enhancement

Expected or Desired Behavior
When I use JSON list view formatting, the list loads a parent DIV in batches of 10 that you cannot control. So if you have 4 tiles across, you get 3 rows but the last row only has 2 items and 2 empty spots.
This is related to issue #108
I was able to hack some CSS to make the parent display in FLEX mode since float:left was breaking the paging and only allowing the first 10 items to come back. Flex allows all of them but I still have this parent div row issue every 10 items. I want my cards to stack like they use to in Display Templates for search, but I cannot use those in SPO.
Observed Behavior
If you are reporting a bug, please describe the behavior you expected to occur when performing the action. If you are making a suggestion, you can delete this section.
Steps to Reproduce
Create a JSON view with more than 10 items, format your JSON with a float:left attribute on the first div element. Only 10 items are returned and the list breaks on document ready (though during page load the scrollbar loads all 100 items on the page if you scroll down fast enough you can see them, then they disappear).
IF you apply Flex display via CSS to ms-List-page, you can wrap the items but every row of 10 does not look aesthetically pleasing if you don't have a number evenly divisible by 10 in the rows.

Manipulating the DOM is not recommended due to the fact we do not control the DOM and it can be changed at any time outside of our control.
Floating does create issues with the scrolling. You can look into the tileProps property to see if that can fulfill your need. Thanks!


[WIP] Eclipse console

As per 4a6f9062fcbe4662b25211a7a96b518513241f94:

I believe that Eclipse IConsole and friends offers us all the features we need,
whilst being vastly simpler (both in architecture and implementation) than
providing, say, a new view. Going this way also allows us to reuse the console
interface we already have, meaning we won't have two completely separate user
interfaces to maintain.

We might have to discuss some architectural items, for example I am not particularly excited about the implementation of ReplConsole. But hey, at least we have something now!
Comes after #18.

Closing in favor of #22.


v11.18.0 iOS autolinking

Bug description:

iOS autolinking doesn't work. I downgraded to 11.17.2 and it works
To Reproduce:

run:
yarn add react-native-webview && cd ios && pod install

Expected behavior:

Native dependency gets installed
Environment:

OS: macOS
OS version: 12.2.1
react-native version: 0.67.2
react-native-webview version: 11.18.0

Trying to manually link the package exposes the error:
➜  Frontend.MobileApp git:(master) ✗ npx react-native link react-native-webview
warn Package react-native-webview has been ignored because it contains invalid configuration. Reason: Cannot find module 'react-native-test-app/package.json'

Trying to manually link the package exposes the error:
warn Package react-native-webview has been ignored because it contains invalid configuration. Reason: Cannot find module 'react-native-test-app/package.json'

Got the same issue

This should be fixed in 11.18.1 via #2413