[SPARK-45959][SQL] : Deduplication of relations causes ambiguous exception to be raised, though logically it is Unambiguous

…ing joins once the plan is de-duplicated. The fix involves using Dataset ID associated with the plans & attributes to attempt correct resolution
There seems to be a regression from Spark 2.4 as the following code no longer succeed:
val df = Seq((1, 2)).toDF("a", "b")
val df2 = df.select(df("a").as("aa"), df("b").as("bb"))
val df3 = df2.join(df, df2("bb") === df("b")).select(df2("aa"), df("a"))
df3.show()
=== Applying Rule org.apache.spark.sql.catalyst.analysis.DeduplicateRelations ===
!'Join Inner, '=(bb#12, b#8)              'Join Inner, '=(bb#12, b#18)
:- Project [a#7 AS aa#11, b#8 AS bb#12]    :- Project [a#7 AS aa#11, b#8 AS bb#12]
:  +- Project [_1#2 AS a#7, _2#3 AS b#8]   :  +- Project [_1#2 AS a#7, _2#3 AS b#8]
:     +- LocalRelation [_1#2, _2#3]        :     +- LocalRelation [_1#2, _2#3]
!+- Project [_1#2 AS a#7, _2#3 AS b#8]      +- Project [_1#15 AS a#17, _2#16 AS b#18]
!   +- LocalRelation [_1#2, _2#3]              +- LocalRelation [_1#15, _2#16]
and so Spark 3 thinks df("a") is ambigous:
Column a#7 are ambiguous. It's probably because you joined several Datasets together, and some of these Datasets are the same. This column points to one of the Datasets but Spark is unable to figure out which one. Please alias the Datasets with different names via Dataset.as before joining them, and specify the column using qualified name, e.g. df.as("a").join(df.as("b"), $"a.id" > $"b.id"). You can also set spark.sql.analyzer.failAmbiguousSelfJoin to false to disable this check.
If we disable spark.sql.analyzer.failAmbiguousSelfJoin then the real issue reveals: Due to the deduplication the last .select(df2("aa"), df("a")) doesn't work any more.
[MISSING_ATTRIBUTES.RESOLVED_ATTRIBUTE_APPEAR_IN_OPERATION] Resolved attribute(s) "a" missing from "aa", "bb", "a", "b" in operator !Project [aa#11, a#7]. Attribute(s) with the same name appear in the operation: "a".
Please check if the right attribute(s) are used. SQLSTATE: XX000;
!Project [aa#11, a#7]
+- Join Inner, (bb#12 = b#18)
:- Project [a#7 AS aa#11, b#8 AS bb#12]
:  +- Project [_1#2 AS a#7, _2#3 AS b#8]
:     +- LocalRelation [_1#2, _2#3]
+- Project [_1#15 AS a#17, _2#16 AS b#18]
+- LocalRelation [_1#15, _2#16]
Similar issues are seen in nested joins and AsOfJoins where same reference Logical Plans are duplicated in the query .
What changes were proposed in this pull request?
The PR attemps to fix the issue in following way

If the projection fields contain AttributeReference which are not found in the incoming AttributeSet,  and the AttributeRef metadata contains the DatasetId info, then the AttributeRef is converted into a new UnresolvedAttributeWithTag and the original attributeRef is passed as paramter .

In the ColumnResolutionHelper,  to resolve the UnresolvedAttributeRefWithTag, a new resolution logic is used:
The dataSetId from the original attribute ref's metadata is extracted.

The first  BinaryNode  contained in the LogicalPlan containing this unresolved attribute,  is found.
Then its right leg & left lag's unary nodes are checked for the presennce of DatasetID of attribute ref, using TreeNodeTag("__datasetid").
If both the legs contain datasetId or neither contains, then resolution exception is thrown
Else the leg which contains datasetId is used to resolve.

Why are the changes needed?
To fix the bug as exposed by the unit tests in the PR
Does this PR introduce any user-facing change?
No
How was this patch tested?
Precheckin run.
Was this patch authored or co-authored using generative AI tooling?
No

While fixing the issue, found that tests in DataFrameSelfJoinSuite, where the two legs have distinct datasetId , the tests are expecting AnalysisException due to ambiguity in the join condition, which IMO is not correct as it is possible to dis-ambiguate using resolution via Dataset ID.
This PR fixes the above and have test modifications accordingly.
The same applies to some tests in DataFrameAsofJoinSuite.
I believe there is still need to do some enhancement  in the existing  source code related to AsofJoin, as the condition part is still not trying to disambiguate properly ( unlike the changes done for Join).


added Number of K-Length Sublists with Average at Least Target.py

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
I have updated the Number of K-Length Sublists with Average at Least Target [Python] #2545 file
Checklist

[✔] I've read the contribution guidelines.
[✔] I've checked the issue list before deciding what to submit.
[✔] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#2545

sir @bhav09 i have added the file please verify and merge

sir @HarshCasper please add Gssoc'21 label

@bhav09 sir please review my PR ....

yes @bhav09 sir sure

@bhav09 sir please review i have made the required changes

@bhav09 sir changes are done

Tq @bhav09 and  @HarshCasper sir for starting my Gssoc'21 Journey by merging my 1st PR, I wish to contribute more and more to this Repository......

Sir @bhav09 @HarshCasper please add level to my PR...

sir @bhav09 Level-3 label got removed i guess........ that is y in profile section my score is showing 0.So please can you look  into it

@all-contributors bot please add @kushal13112001 for code

Tq @HarshCasper now got the score in profile


initial config api

Order as
export class ClientConfig { //
export class NetworkConfig { //

Causes use before declaration error in compiled JS code at jshint stage although I think it is not an issue in TypeScript. Anyway I re-ordered the classes so that it passes without an error now.

:+1:


Error handled when error thrown while connecting to ldap server

Error:
events.js:136
throw er; // Unhandled 'error' event
^
Error: connect ECONNREFUSED <IP_ADDRESS>:10389
at Object._errnoException (util.js:1031:13)
at _exceptionWithHostPort (util.js:1052:20)
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1195:14)

Thanks!

Published as<EMAIL_ADDRESS>


Tax on delivery fee bug

Expected behavior:
(Subtotal	                     €10,25
Delivery	                     €2,00
TAX [9% included].     €1,28
Order total	            €12,25)
Actual behavior:
(Subtotal	                     €10,25
Delivery	                     €2,00
TAX [9% included].     €1,28
Order total	            €14,25)
Reproduce steps:
(When you turn on Tax over delivery fee)
Version:
(v3.0.4-beta.26)
Additional Information:
(When you turn on Tax over delivery fee it will calculate dit delivery fee twice)

Any chance this is related to: https://github.com/tastyigniter/TastyIgniter/issues/687

Any chance this is related to: https://github.com/tastyigniter/TastyIgniter/issues/687

Opened a PR for this: https://github.com/tastyigniter/ti-ext-cart/pull/47

Opened a PR for this: https://github.com/tastyigniter/ti-ext-cart/pull/47

I’ll add some screenshots

I’ll add some screenshots

@diverds1992 Did you look at the code I added in the pull request referenced earlier?

@diverds1992 Did you look at the code I added in the pull request referenced earlier?


Can't cross-compile for Raspberry Pi (probably general cross-compile breakage)

cargo build --target=armv7-unknown-linux-gnueabihf
...
  = note: /usr/lib/libpython3.6m.so: file not recognized: File format not recognized
          collect2: error: ld returned 1 exit status

I'm pretty sure this is can be fixed with a cross compiled python


[feature]: Enhance lnd with custom channels on-chain functionality

Deliverables

[x] Co-op Channel closure

[x] MVP: Add another output to co-op close into which tap proofs are committed

[x] Force Channel closure

[x] More complex, dependent on recent lnd sweeper changes.  Current functionality: a force closure will be initiated but the tapd logic is lacking

[ ] Asses HTLC Sweeper logic augmentation

Feature Branches:
https://github.com/lightninglabs/taproot-assets/tree/custom-channels-poc-coop-close
https://github.com/lightningnetwork/lnd/tree/custom-channels-poc-coop-close

With #970 merged, the necessary support from tapd was added to transfer -- and arbitrate HTLC settlement -- of taproot assets over the lightning network.
See breach code here: https://github.com/lightninglabs/lightning-terminal/pull/782


Add partition mode to treemap

Also combine the simple-treemap and circle packing examples to show all of the treemap modes.


cvat requires docker image to start

look at examples from https://github.com/PyTorchLightning/lightning/blob/0f7ca444e6a6f2278fdd45a22e41bce1fcdb4cc4/tests/examples/custom_work_dependencies/app.py
class WorkWithCustomBaseImage(LightningWork):
    def __init__(self, cloud_compute: CloudCompute = CloudCompute("cpu", 1), **kwargs):
        # this image has been created from ghcr.io/gridai/base-images:v1.8-cpu
        # by just adding an empty file at /content/.e2e_test
        custom_image = "ghcr.io/gridai/image-for-testing-custom-images-in-e2e"
        build_config = BuildConfig(image=custom_image)
        super().__init__(parallel=True, **kwargs, cloud_compute=cloud_compute, cloud_build_config=build_config)

CVAT availability has been not reliable.  switching to label studio

Interested in how your experiments work out with Label Studio. Please also see this.
https://github.com/Lightning-AI/lightning/issues/13520


Error: socket hang up in Node 4.2.1 (but not 0.12)

Hello!
I'm trying to debug this issue: https://github.com/blakmatrix/node-zendesk/issues/110
Having looked into it a bit, it looks like it is a bug in request—the only alternative could be that the API I'm making requests to is rejecting the useragent with node/4.2.1 in and not the one with node/0.12 in.
request is being passed the same data in both versions of Node, and is being called multiple times. It never fails on the first time it is called, it usually fails between the third and sixth times—it's being passed only slightly different data (page numbers), but sporadically fails.
This is some sample data that sometimes succeeds and sometimes fails:
{ stores:
   { defaults:
      Literal {
        type: 'literal',
        store: [Object],
        mtimes: {},
        readOnly: true,
        loadFrom: null,
        logicalSeparator: ':' } },
  sources: [],
  headers:
   { 'Content-Type': 'application/json',
     Accept: 'application/json',
     'User-Agent': 'node-zendesk/1.1.5 (node/4.2.1)',
     Authorization: 'Basic [redacted]' },
  uri: 'https://[redacted].zendesk.com/api/v2/search.json?page=3&query=checkout-failure',
  method: 'GET' }

And request is being given the following defaults:
{ jar: RequestJar { _jar: CookieJar { store: { idx: {} } } },
  encoding: null,
  timeout: 240000,
  proxy: null,
  secureOptions: 134217728 }

The following error is (eventually) thrown:
Error: socket hang up
    at createHangUpError (_http_client.js:203:15)
    at TLSSocket.socketOnEnd (_http_client.js:288:23)
    at emitNone (events.js:72:20)
    at TLSSocket.emit (events.js:166:7)
    at endReadableNT (_stream_readable.js:903:12)
    at doNTCallback2 (node.js:439:9)
    at process._tickCallback (node.js:353:17)

What could be causing this? Is this a bug in request, or is there something wrong with how we're calling it?
I'm too far deep into a library I am unfamiliar with to make a stripped down test case. If necessary, I can make one—wanted to check it wasn't something obvious before investing the time, first, though.
Here is where request is initiated: https://github.com/blakmatrix/node-zendesk/blob/master/lib/client/client.js#L35-L41
Here is where it is called: https://github.com/blakmatrix/node-zendesk/blob/master/lib/client/client.js#L100-L102
Help would be much appreciated, thanks!

@callumacrae did you try running request in debug mode? You could then compare the data when using node 0.12 and node 4 to see if there is any discrepancy.

Hi guys. I have same problem here.
I tried to compare data between request on node 0.12.9 and 4.2.3, they are all the same.
It might be because of the http module itself.

Same error after exact 2 minutes of waiting response.

I don't have much to add other than I can confirm that I'm getting the same error after making requests in node 4.

@simov hate to bug you, but do you have any insight into what could be happening here?

We've also run into the same issue, but we've reproduced with just the raw http library. So I don't really think this is anything to do with request.
Its a totally bonkers bug and the only thing I can think of is that zendesk's back end has an issue somewhere.

@fiznool okay, hmm. Do you think it's worth connecting with someone at Zendesk?

I can't see it being an issue with Zendesk if it's specific to a version of node. Maybe worth opening an issue on the node repo?

@erikzrekz we already got in touch with them, and they directed us to open an issue on their node repo. That's the issue mentioned above.

@callumacrae agreed, this probably isn't the best place to be discussing this since I don;t think it has anything to do with request.

Please refer to https://github.com/blakmatrix/node-zendesk/issues/110#issuecomment-171275768, the solution is to add forever: true to the request options.

Did you use a keepAlive socket? If the server requires a keep-alive connection , and your headers doesn't include the keep-alive option and it will return an socket hang up error.
and the most important is , In node 0.12, setting 'Connection': 'Keep-Alive' property in http option will work. But in node 4.0 or above, if you would like to use keep-alive connection, you have to create an agent and set its keepAlive property to true. So,in this situation, the same code will work on 0.12 but won't work anymore on 4.0.
I hope this will help you .

I have same problem while creating new object in s3.


GEODE-4908 - Rename GFCPP

Also some formatting and readability changes

Will wait for @echobravopapa approve before merging.


Split bill - IOU preview in 1:1 DM is blank with undefined username when bill is split offline

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to staging.new.expensify.com and log in with a new Expensifail account
Go offline
Create a split bill with users that have no chat history
Return to LHN
Click on the search icon
Click on the newly created 1:1 DM

Expected Result:
The IOU preview shows the requested details
Actual Result:
The IOU preview is blank. The usernames above IOU preview shows undefined
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.70-2
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/4547b924-e4d6-4c54-8f53-c3fc932d0c84
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Can reproduce on staging and not production, trying to identify the issue now.

I wasn't able to reproduce on staging and on dev

This might have to do with betas as it's not reproducible on accounts that are on public domains. It also doesn't prevent from creating the IOU when the user comes back online, so I'll remove the blocker label for now and we can keep investigating later.

I just tried this again in dev environment and can no longer reproduce the issue, any chance you can confirm @lanitochka17 please?

Issue is not reproducible on Build 1.3.74-0
https://github.com/Expensify/App/assets/78819774/8ec0a0b9-d436-4785-95b0-270f65491813

Still can't reproduce on 1.3.77-2, closing!

@francoisl Issue is still  reproduce this by using Expensify account to create split bill with gmail accounts in offline mode.
https://github.com/Expensify/App/assets/78819774/37a30dc5-a4f5-4cfc-bc2a-37c34f4a3eb0

Still can't reproduce on v1.3.92-0 😕 I'll demote to weekly for now and try again later

Can't reproduce, closing.


After update .77 -> .78 No UI at all

After common update routines (git pull, docker-compose pull, docker-compose up -d) from version .77 to .78 UI says "connected to the bot", but no bot interface is shown further.
What do I do/how do I diagnose problems?

The same issue here. Bot is running in docker container under Linux at Hetzner cloud VPS with 2 VCPU, 2GB RAM and 20GB of disk space.
The problem appeared after upgrading to version  0.0.78: Web dashboard is not showing in browser.
The bot itself is running and performing trades. However, it shows empty page in the browser right after logging into web dashboard. it also shows the notifications bun nothing excepting that.
I tried to roll back to previous working version (0.0.77) but bot was not even running on it. Then I pulled back the latest version (0.0.78), so bot is running but dashboard is still not accessible.
I have tried different browsers from desktop and mobile, I tried to hard-refresh the browser, nothing helped.
Please see the screenshots from the browser console below.
There are two errors before entering the password:

And the rest errors after logging in:

Same thing in Chrome console.

Can you check the following things?

In Chrome, open the Developer console and go to Network -> WS -> click first thing in the Name. And then you will see Messages. Refer screenshot. Copy the JSON and paste it here as a snippet.

Access to MongoDB and get configuration and paste it here as a snippet.
$ docker exec -it binance-mongo /bin/bash
$ mongo
$ use binance-bot
$ db['trailing-trade-common'].find();

Access to Redis and get configuration and paste it here as a snippet.
$ docker exec -it binance-redis /bin/bash
$ redis-cli
$ AUTH <redis password>
$ HGET trailing-trade-configurations global

@chrisleekr , where is redis password is set and what is default one? I see no related var in .env.dist.

It should be right in the docker-compose.yml file

@chrisleekr, here you go.
1.
{"command":"latest","authToken":"AUTHTOKENHERE","data":{}}
{"result":true,"type":"latest","isAuthenticated":true,"botOptions":{"authentication":{"lockList":true,"lockAfter":600},"autoTriggerBuy":{"enabled":false,"triggerAfter":20}},"configuration":{"_id":"60b2c729981080ba1b777e45","key":"configuration","enabled":true,"cronTime":"* * * * * *","symbols":["ETHUSDT","XRPUSDT","EOSUSDT","DOTUSDT","LINKUSDT","ETCUSDT","LTCUSDT","ZECUSDT","BAKEUSDT","UNIUSDT","THETAUSDT","BCHUSDT","FILUSDT","XLMUSDT","SOLUSDT","XMRUSDT","AAVEUSDT","DASHUSDT","SUSHIUSDT","ATOMUSDT","RUNEUSDT","TRXUSDT","VETUSDT","ADAUSDT","ONTUSDT","NEOUSDT","BNBUSDT","ALGOUSDT","TROYUSDT","WRXUSDT","FIROUSDT","PUNDIXUSDT","WAVESUSDT","LUNAUSDT","KSMUSDT","REEFUSDT","PNTUSDT","BLZUSDT","LITUSDT","CFXUSDT","BELUSDT","UNFIUSDT","EPSUSDT","RENUSDT","BADGERUSDT","KAVAUSDT","PERPUSDT","KNCUSDT","GXSUSDT","TKOUSDT","NANOUSDT","1INCHUSDT","XTZUSDT","ZILUSDT","CAKEUSDT","TFUELUSDT","CTSIUSDT","FORTHUSDT","CHZUSDT","SANDUSDT","COMPUSDT","ENJUSDT","YFIUSDT","MATICUSDT","BTTUSDT","MINAUSDT","AUDIOUSDT","SUPERUSDT","AVAXUSDT","DOGEUSDT"],"candles":{"interval":"4h","limit":50},"buy":{"enabled":true,"lastBuyPriceRemoveThreshold":-1,"lastBuyPriceRemoveThresholds":{"USDT":10},"athRestriction":{"enabled":true,"candles":{"interval":"1d","limit":50},"restrictionPercentage":0.8},"gridTrade":[{"triggerPercentage":1,"stopPercentage":1.01,"limitPercentage":1.02,"maxPurchaseAmount":-1,"maxPurchaseAmounts":{"USDT":100,"BTC":0.001}},{"triggerPercentage":0.9,"stopPercentage":1.01,"limitPercentage":1.02,"maxPurchaseAmount":-1,"maxPurchaseAmounts":{"USDT":120,"BTC":0.001}}],"maxPurchaseAmount":-1},"sell":{"enabled":true,"stopLoss":{"enabled":true,"maxLossPercentage":0.7495,"disableBuyMinutes":120,"orderType":"market"},"gridTrade":[{"triggerPercentage":1.1,"stopPercentage":0.99,"limitPercentage":0.98,"quantityPercentage":-1,"quantityPercentages":{"USDT":1}}]},"system":{"temporaryDisableActionAfterConfirmingOrder":20,"checkManualBuyOrderPeriod":5,"refreshAccountInfoPeriod":1,"placeManualOrderInterval":5,"checkOrderExecutePeriod":10,"checkManualOrderPeriod":5},"botOptions":{"authentication":{"lockList":true,"lockAfter":600},"autoTriggerBuy":{"enabled":false,"triggerAfter":20}}},"common":{"version":"0.0.78","gitHash":"60a464a","accountInfo":{"makerCommission":10,"takerCommission":10,"buyerCommission":0,"sellerCommission":0,"canTrade":true,"canWithdraw":true,"canDeposit":true,"updateTime":1630820383119,"accountType":"SPOT","balances":[{"asset":"BNB","free":"0.11216231","locked":"0.00000000"},{"asset":"USDT","free":"299.15971203","locked":"0.00000000"},{"asset":"VET","free":"0.19213743","locked":"0.00000000"},{"asset":"VTHO","free":"94.44137546","locked":"0.00000000"},{"asset":"LDADA","free":"823.12397771","locked":"0.00000000"},]}}
<..>

db['trailing-trade-common'].find();
{ "_id" : ObjectId("60b2c729981080ba1b777e45"), "key" : "configuration", "enabled" : true, "cronTime" : "* * * * * *", "symbols" : [ "ETHUSDT", "XRPUSDT", "EOSUSDT", "DOTUSDT", "LINKUSDT", "ETCUSDT", "LTCUSDT", "ZECUSDT", "BAKEUSDT", "UNIUSDT", "THETAUSDT", "BCHUSDT", "FILUSDT", "XLMUSDT", "SOLUSDT", "XMRUSDT", "AAVEUSDT", "DASHUSDT", "SUSHIUSDT", "ATOMUSDT", "RUNEUSDT", "TRXUSDT", "VETUSDT", "ADAUSDT", "ONTUSDT", "NEOUSDT", "BNBUSDT", "ALGOUSDT", "TROYUSDT", "WRXUSDT", "FIROUSDT", "PUNDIXUSDT", "WAVESUSDT", "LUNAUSDT", "KSMUSDT", "REEFUSDT", "PNTUSDT", "BLZUSDT", "LITUSDT", "CFXUSDT", "BELUSDT", "UNFIUSDT", "EPSUSDT", "RENUSDT", "BADGERUSDT", "KAVAUSDT", "PERPUSDT", "KNCUSDT", "GXSUSDT", "TKOUSDT", "NANOUSDT", "1INCHUSDT", "XTZUSDT", "ZILUSDT", "CAKEUSDT", "TFUELUSDT", "CTSIUSDT", "FORTHUSDT", "CHZUSDT", "SANDUSDT", "COMPUSDT", "ENJUSDT", "YFIUSDT", "MATICUSDT", "BTTUSDT", "MINAUSDT", "AUDIOUSDT", "SUPERUSDT", "AVAXUSDT", "DOGEUSDT" ], "candles" : { "interval" : "4h", "limit" : 50 }, "buy" : { "enabled" : true, "lastBuyPriceRemoveThreshold" : -1, "lastBuyPriceRemoveThresholds" : { "USDT" : 10 }, "athRestriction" : { "enabled" : true, "candles" : { "interval" : "1d", "limit" : 50 }, "restrictionPercentage" : 0.8 }, "gridTrade" : [ { "triggerPercentage" : 1, "stopPercentage" : 1.01, "limitPercentage" : 1.02, "maxPurchaseAmount" : -1, "maxPurchaseAmounts" : { "USDT" : 100, "BTC" : 0.001 } }, { "triggerPercentage" : 0.9, "stopPercentage" : 1.01, "limitPercentage" : 1.02, "maxPurchaseAmount" : -1, "maxPurchaseAmounts" : { "USDT" : 120, "BTC" : 0.001 } } ], "maxPurchaseAmount" : -1 }, "sell" : { "enabled" : true, "stopLoss" : { "enabled" : true, "maxLossPercentage" : 0.7495, "disableBuyMinutes" : 120, "orderType" : "market" }, "gridTrade" : [ { "triggerPercentage" : 1.1, "stopPercentage" : 0.99, "limitPercentage" : 0.98, "quantityPercentage" : -1, "quantityPercentages" : { "USDT" : 1 } } ] }, "system" : { "temporaryDisableActionAfterConfirmingOrder" : 20, "checkManualBuyOrderPeriod" : 5, "refreshAccountInfoPeriod" : 1, "placeManualOrderInterval" : 5, "checkOrderExecutePeriod" : 10, "checkManualOrderPeriod" : 5 }, "botOptions" : { "authentication" : { "lockList" : true, "lockAfter" : 600 }, "autoTriggerBuy" : { "enabled" : false, "triggerAfter" : 20 } } }

<IP_ADDRESS>:6379> HGET trailing-trade-configurations global
"{"_id":"60b2c729981080ba1b777e45","key":"configuration","enabled":true,"cronTime":"* * * * * *","symbols":["ETHUSDT","XRPUSDT","EOSUSDT","DOTUSDT","LINKUSDT","ETCUSDT","LTCUSDT","ZECUSDT","BAKEUSDT","UNIUSDT","THETAUSDT","BCHUSDT","FILUSDT","XLMUSDT","SOLUSDT","XMRUSDT","AAVEUSDT","DASHUSDT","SUSHIUSDT","ATOMUSDT","RUNEUSDT","TRXUSDT","VETUSDT","ADAUSDT","ONTUSDT","NEOUSDT","BNBUSDT","ALGOUSDT","TROYUSDT","WRXUSDT","FIROUSDT","PUNDIXUSDT","WAVESUSDT","LUNAUSDT","KSMUSDT","REEFUSDT","PNTUSDT","BLZUSDT","LITUSDT","CFXUSDT","BELUSDT","UNFIUSDT","EPSUSDT","RENUSDT","BADGERUSDT","KAVAUSDT","PERPUSDT","KNCUSDT","GXSUSDT","TKOUSDT","NANOUSDT","1INCHUSDT","XTZUSDT","ZILUSDT","CAKEUSDT","TFUELUSDT","CTSIUSDT","FORTHUSDT","CHZUSDT","SANDUSDT","COMPUSDT","ENJUSDT","YFIUSDT","MATICUSDT","BTTUSDT","MINAUSDT","AUDIOUSDT","SUPERUSDT","AVAXUSDT","DOGEUSDT"],"candles":{"interval":"4h","limit":50},"buy":{"enabled":true,"lastBuyPriceRemoveThreshold":-1,"lastBuyPriceRemoveThresholds":{"USDT":10},"athRestriction":{"enabled":true,"candles":{"interval":"1d","limit":50},"restrictionPercentage":0.8},"gridTrade":[{"triggerPercentage":1,"stopPercentage":1.01,"limitPercentage":1.02,"maxPurchaseAmount":-1,"maxPurchaseAmounts":{"USDT":100,"BTC":0.001}},{"triggerPercentage":0.9,"stopPercentage":1.01,"limitPercentage":1.02,"maxPurchaseAmount":-1,"maxPurchaseAmounts":{"USDT":120,"BTC":0.001}}],"maxPurchaseAmount":-1},"sell":{"enabled":true,"stopLoss":{"enabled":true,"maxLossPercentage":0.7495,"disableBuyMinutes":120,"orderType":"market"},"gridTrade":[{"triggerPercentage":1.1,"stopPercentage":0.99,"limitPercentage":0.98,"quantityPercentage":-1,"quantityPercentages":{"USDT":1}}]},"system":{"temporaryDisableActionAfterConfirmingOrder":20,"checkManualBuyOrderPeriod":5,"refreshAccountInfoPeriod":1,"placeManualOrderInterval":5,"checkOrderExecutePeriod":10,"checkManualOrderPeriod":5},"botOptions":{"authentication":{"lockList":true,"lockAfter":600},"autoTriggerBuy":{"enabled":false,"triggerAfter":20}}}"

@VSofter Global configuration looks ok.
Can I see symbol information in the WebSocket?
Your first snippet does not include stats -> symbols.

@VSofter Global configuration looks ok.
Can I see symbol information in the WebSocket?
Your first snippet does not include stats -> symbols. Please paste the whole JSON.

Whole JSON is too large and it doesn't fit into github limitations :-)
First symbol (of 50+):
0: {symbol: "REEFUSDT", isLocked: false,…}
accountInfo: {makerCommission: 10, takerCommission: 10, buyerCommission: 0, sellerCommission: 0, canTrade: true,…}
action: "not-determined"
baseAssetBalance: {asset: "REEF", free: 0, locked: 0, total: 0, estimatedValue: 0,…}
buy: {currentPrice: 0.0284, limitPrice: 0.028968, highestPrice: 0.0295, lowestPrice: 0.02262,…}
featureToggle: {notifyOrderConfirm: true, notifyDebug: false, notifyOrderExecute: true}
indicators: {highestPrice: 0.0295, lowestPrice: 0.02262, athPrice: 0.0295}
isActionDisabled: {isDisabled: false, ttl: -2}
isLocked: false
lastCandle: {eventType: "kline", eventTime:<PHONE_NUMBER>771, symbol: "REEFUSDT", startTime:<PHONE_NUMBER>000,…}
openOrders: []
order: {}
quoteAssetBalance: {asset: "USDT", free: "299.15971203", locked: "0.00000000"}
saveToCache: true
sell: {currentPrice: 0.0284, limitPrice: null, lastBuyPrice: null, triggerPrice: null, difference: null,…}
symbol: "REEFUSDT"
symbolConfiguration: {_id: "60b2c729981080ba1b777e45", key: "configuration", enabled: true, cronTime: "* * * * * *",…}
symbolInfo: {symbol: "REEFUSDT", status: "TRADING", baseAsset: "REEF", baseAssetPrecision: 8, quoteAsset: "USDT",…}
Is that enough?

@VSofter No, I will need to see the whole JSON. Try to use https://gist.github.com/ you can remove it later.

@chrisleekr , alright, here you go: https://gist.github.com/VSofter/3613fe04274c88542fec9fcd07b2cf98

@VSofter
Ok, the issue is SHIBUSDT did not migrate properly. And it is not included in the global configuration symbol list.
"symbolConfiguration": {
         ...
          "buy": {
            "enabled": true,
            "maxPurchaseAmount": 40,
            "triggerPercentage": 1.05,
            "stopPercentage": 1.02,
            "limitPercentage": 1.025,
            "lastBuyPriceRemoveThreshold": 10,
            "athRestriction": {
              "enabled": true,
              "candles": { "interval": "1d", "limit": 50 },
              "restrictionPercentage": 0.75
            }
          },
          "sell": {
            "enabled": true,
            "triggerPercentage": 1.1,
            "stopPercentage": 0.985,
            "limitPercentage": 0.975,
            "stopLoss": { "enabled": false, "maxLossPercentage": 0.7, "disableBuyMinutes": 120, "orderType": "market" }
          },
          "system": {
            "temporaryDisableActionAfterConfirmingOrder": 20,
            "checkManualBuyOrderPeriod": 5,
            "refreshAccountInfoPeriod": 1,
            "placeManualOrderInterval": 5
          }
        },

Can you try to flush the Redis cache?
 $ docker exec -it binance-redis /bin/bash
# redis-cli
<IP_ADDRESS>:6379> AUTH <redis password>
OK
<IP_ADDRESS>:6379> FLUSHALL
OK
<IP_ADDRESS>:6379>

@VSofter
Ok, in your case, the issue is SHIBUSDT did not return the data properly. I assume this is happening because it is not included in the global configuration symbol list.
The below is the symbol configuration of SHIBUSDT from your snippet, which has the wrong format.
"symbolConfiguration": {
         ...
          "buy": {
            "enabled": true,
            "maxPurchaseAmount": 40,
            "triggerPercentage": 1.05,
            "stopPercentage": 1.02,
            "limitPercentage": 1.025,
            "lastBuyPriceRemoveThreshold": 10,
            "athRestriction": {
              "enabled": true,
              "candles": { "interval": "1d", "limit": 50 },
              "restrictionPercentage": 0.75
            }
          },
          "sell": {
            "enabled": true,
            "triggerPercentage": 1.1,
            "stopPercentage": 0.985,
            "limitPercentage": 0.975,
            "stopLoss": { "enabled": false, "maxLossPercentage": 0.7, "disableBuyMinutes": 120, "orderType": "market" }
          },
          "system": {
            "temporaryDisableActionAfterConfirmingOrder": 20,
            "checkManualBuyOrderPeriod": 5,
            "refreshAccountInfoPeriod": 1,
            "placeManualOrderInterval": 5
          }
        },

Can you try to flush the Redis cache?
 $ docker exec -it binance-redis /bin/bash
# redis-cli
<IP_ADDRESS>:6379> AUTH <redis password>
OK
<IP_ADDRESS>:6379> FLUSHALL
OK
<IP_ADDRESS>:6379>

@chrisleekr , tried to flush cache and then restarting docker-compose container. No go. This exact configuration worked flawlessly with .77. How can I include SHIBUSDT to global configuration manually (or any other workaround)?

With great help of user clearin at Discord, I have managed to solve this issue.
My mistake was that I always upgraded just by using docker-compose -f docker-compose.server.yml pull to upgrade but I should use git pull firstly, so my docker-composer.server.yml (and other .yml files as well) file became too old.
I am not sure that it was the only real reason but when I did git pull firstly, then docker pull, the problem was eliminated.

I also got stuck and couldn't access the dashboard but after resetting rpi4 everything is solved.

Nothing helped me yet: neither reboot, nor git pull.
My running configuration is Ubuntu 20.04 LTS/8 cores/16 GB. There is enough free disk space available.

@chrisleekr, just to inform you, I've added SHIBUSDT to 'trailing-trade-common' collection manually and got back my dashboard afterwards.
Still not sure though, how had it happened to my installation after update (it never had before). Probably, really a bug between .77 --> .78.

@VSofter
Technically, it shouldn't work previously with your case because the data was not in the correct format. The code you having an error was changed a month ago.
And if you flush Redis completely, it should've worked, because SHIBUSDT-data shouldn't be returning to the frontend.
Not sure what happened in there as well.
Anyhow, good to hear your issue is resolved. :)
Please close the issue.


Licensing Requirements and availabilty in sovereign clouds

[Enter feedback here]
Would like to see the licensing required to apply compliance boundaries and also the roadmap for availability in GCC, GCCH and DoD (if not there yet)

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

ID: 7ffb84a4-0c98-3fc2-8963-4c809efaa149
Version Independent ID: 305dd7b2-382e-a0da-40b5-1457e6b625bd
Content: Set up compliance boundaries for eDiscovery investigations - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/set-up-compliance-boundaries.md
Service: o365-seccomp
GitHub Login: @markjjo
Microsoft Alias: markjjo

@SeCossio Thanks for the inquiry. As per our offline thread, users (whose data might be searched by eDiscovery managers in a compliance boundary scenario) have to be assigned an EXO license and and an SPO license (for OneDrive provisioning). Also, there user's mailbox must be 10 MB or larger, otherwise the user is not processed when syncing the compliance attribute to their OneDrive account. I'll add these prerequisites to the topic.
And as per our conversation, compliance boundaries are available/supported in all the GOV clouds.


eo-math: 0.1.1 release

@Graur Could you please clarify what's wrong?

@maxonfjvipon Looks like somewhere in this code we use nan without +alias. Maybe in number-tests?

@Graur you were absolutely right! Many thanks! But there is some c.eo object and it can not be found

@maxonfjvipon check number.eo

@Graur Oops, my bad, sorry

@maxonfjvipon I don't quite understand the problem. Did you make these changes with the pull.sh script or just by hand?

@Graur by hand :D I did not noticed pull.sh... Will try it next time, thanks!)

@maxonfjvipon You are right, we miss it. Thanks!

@maxonfjvipon I think you need to check the eo-runtime math package again, as the random object has changed significantly

@Graur ok, I'll do it

@Graur I think that's it

@maxonfjvipon Looks good, thanks!
@yegor256 please, have a look

@maxonfjvipon there are some conflicts, please resolve them and we'll merge

@maxonfjvipon Maybe it would be better to create a new pull request with help of the pull.sh script?

@yegor256 It's done, but something is still wrong here with list

@maxonfjvipon you should use org.eolang.collections.list

@yegor256 It's done, thanks!

@maxonfjvipon thanks!

@yegor256 Should I make a PR to eo-runtime and remove math objects from there?

@maxonfjvipon the puzzle #48 is still not solved.

@maxonfjvipon the only puzzle #48 is solved here.


Add use case for Service Account Name

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

From comment on CI-5758. I know this page needs more updates but I wanted to capture this scenario quickly.

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://hdh.pr.harness.io/pr-988


CustomRP: Updated description, added descriptions in other languages

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@Trenly Can't find BCP 47 code for Catalan, Ukrainian, Vietnamese. Any suggestions?

ca-ES, uk-UA, vi-VN?

Exactly correct; The two letter language code, and the two letter country code from ISO-639 and ISO-3166 would make those the correct BCP-47 codes


Fix: mobile modals

Pull request template
Checklist

[ ] I have read the CONTRIBUTING guide
[ ] branch rebased on develop
[ ] styleguide check npx eslint shared/**/*.js
[ ] good naming
[ ] keep simple
[ ] video or screenshot attached
[ ] testing instruction attached
[ ] check your PR once again

Description

Video or screenshot

What and how to test

баг

исправлено


J.howard

https://github.com/jkarlin/topics/blob/main/taxonomy_v1.md

@Dreamypops33 Could you please stop opening not related issues in this repository. This is not a playground.


Add 1-hour timeout to each Cloud TPU CI job.

Sometimes they hang, and the default timeout is 6 hours, which is way too long.

Test run: https://github.com/google/jax/actions/runs/4558148006

RTD failure has been fixed on main


Pass original .NET callback exception as TrapException's inner exception

Hi, this PR implements the functionality to pass a .NET exception thrown at a wasm-to-host transition (.NET callback) (which means the exception is converted to a trap) as inner exception of the created TrapException when the trap bubbles up to the next host-to-wasm transition, by using a ThreadStatic (thread-local) field to temporarily store the exception until the stack is unwound.
That way, the original .NET exception that caused a trap can be retrieved from TrapException.InnerException, e.g. for debugging purposes.
If the exception thrown at the .NET callback is already a TrapException, we will not store that exception itself, but its InnerException, so that when the trap bubbles through multiple levels of wasm-to-host transitions, the InnerException at the root host-to-wasm transition will still be the original .NET exception.
Fixes #63
Note: The mechanism requires that on every location where a trap can occur due to calling wasm code, TrapException.FromOwnedTrap() is called when a trap actually occured, so that the thread-local field is cleared afterwards.
TODO:

If #163 is merged before this PR, the exception-handling code in the generated code for Function.FromCallback and Linker.DefineCallback needs to be adjusted to call Function.HandleCallbackException().

What do you think?
Thanks!

The tests are currently failing due to #171.

We should be able to rebase this PR now.


Can't restore 2FA wallet in 2.9.4 on Mac

Hello,
I have a 2FA wallet that I'm trying to restore but there is no such option in the UI...

I was able to do it using v2.9.2


Foreman API v1 support

Hello,
how easy/difficult would it be to support the old Foreman API v1 too?
It's feasible at all?
If yes, I'll do a fork and give it a try....
Cheers
Roland

Hello,
Hard to say, it will depend on the degree of details you need, v1 is less detailed than v2, but the structure seems to be close. In a first step you can try commenting all unexisting ressources in v1 and focus on one item for the debug.
BR

Thx for your response. I found another lib: https://github.com/david-caro/python-foreman
This one is more low-level, but responds to different API versions.
Cheers
Roland


cant add img

self.cAlg.add_img(img)
AttributeError: 'NoneType' object has no attribute 'add_img'

please ensure that your path do not contain chinese char. only support english path for image file


rough schedule for implementations track

Thanks for this! I directly edited 2 characters (fixed a typo, nitpicked a capitalization). Merging now.


Fixes for includeChildMeshes:true when constructing physics shapes

Allows MESH and CONVEX_HULL shapes to be constructed from TransformNode or InstancedMesh instances and  correctly account for transform of children when includeChildMeshes is set to true.
Discussed this with @carolhmj - there were a couple of failure cases or unexpected behaviour when building physics shapes.
Here's an example of one of the failure cases (check the console logs): https://playground.babylonjs.com/#5BJIYA#3 - this loads a glTF file which contains a mesh with multiple materials. The loader will create a single TransformNode with two Mesh children - one for each material.  If a user tries to build a physics shape from the "Cube" object (which is a glTF mesh, but a Babylon TransformNode) they'll hit the failure cases in the console.log()s. Similarly, if the mesh was instanced multiple times, the old code previously tried to use an InstancedMesh instance as if it were a Mesh, which would raise an exception accessing an undefined property (Mesh.getVerticesData)
This change allows a user to create a single physics shape from this scene construct. I've also tested this with some additional playground scenes, which @carolhmj linked me to:
https://playground.babylonjs.com/#5BJIYA#4
https://playground.babylonjs.com/#K7TJIG#363
https://playground.babylonjs.com/#9T64J6
https://playground.babylonjs.com/#R9DTZ0#16
And made a new variant of one, which tests this includeChildMeshes functionality in the complicated "Buggy" asset:
https://playground.babylonjs.com/#K7TJIG#371
Since I can imagine that this case of constructing a single physics shape from a whole branch of the scene hierarchy is useful, it would probably also be nice to expose the includeChildMeshes property to the PhysicsShapeMesh and PhysicsShapeConvexHull constructors, without having to require users use the PhysicsShape base class.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14030/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14030/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14030/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14030/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14030/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14030/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14030/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14030/merge#BCU1XR#0

Visualization tests for webgl2 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14030/merge/testResults/webgl2/index.html
If tests were successful afterwards, this report might not be available anymore.

@eoineoineoin - is this the full fix? or is there more to it?


Documentation: How to sync the main repo and own repo in GitHub.

To sync the main repo and own repo,

pointing to the main repo and pull from it, for that create a new remote in local for pointing the main repo .
git remote add upstream https://github.com/twindle-co/twindle.git
To download all the changes and commits from main Repo.
git fetch upstream
The current branch will be updated to reflect the merge.
git merge upstream/main
push the changes to own repository.
git push origin main

Great, @praveen2896 thanks for documenting this! @Sheetal777 or anyone else could verify?

You have two options

On your Git in VS Code, click on the three dot option and click push/pull
OR
In your terminal do this 'git push' or 'git pull'

Pull will get the latest changes from GitHub
And push will send your latest changes to GitHub

@thunder-coding Could you collaborate on making sure this is a step by step explanation so that a newbie can understand? The discussion is about how to keep the forked repo in sync before creating PRs

https://github.com/twindle-co/twindle/pull/8#issuecomment-708329032 may also be helpful

An important step is missed here.
Before you can sync your fork with an upstream repository, you must configure a remote that points to the upstream repository in Git.
Here are the steps: Configuring a remote for a fork

AzureAD+SwatiAnandRao@LAPTOP-MJRBVINQ MINGW64 ~/Desktop/GIT/twindle (main)
$ git push origin main
remote: Permission to twindle-co/twindle.git denied to SwatiRaoDataEnthusiast.
fatal: unable to access 'https://github.com/twindle-co/twindle/': The requested
URL returned error: 403
I am getting above error when I am trying to push into origin?

 You have to first commit to your forked repo then you can make a PR from Github.
I have attached some snaps below.
Here you can see my origin is my forked repo link. You can check your's by running git remote -v.

Then after making changes you can run git status commands to check what changes you have made.

Here you can see I have made some changes to 'pranav.md' file.
So to push these changes to my forked repo I can run these commands.

git add .
git commit -m "Updated pranav.md"
git push -u origin main

Now they will be reflected to my forked repo.

Now you can make a PR.

@johnjacobkenny Please pin this issue, as it is hard to find it now.

Could someone follow up on getting this into FAQ on a priority

@SwatiRaoDataEnthusiast
You are making a commit directly to main repo. You have to first commit to your forked repo then you can make a PR from Github to the main repo.
I have attached some snaps below.
Here you can see my origin is my forked repo link. You can check your's by running git remote -v.

Then after making changes you can run git status commands to check what changes you have made.

Here you can see I have made some changes in 'pranav.md' file.
So to push these changes to my forked repo I can run these commands.

git add .
git commit -m "Updated pranav.md"
git push -u origin main

Now they will be reflected to my forked repo.

Now you can make a PR.

Just to add onto what @pranavgoel29  has said, you can change the path for origin using
git remote set-url origin _your forked repo url here_

@MumbiGachomba04 next time you can use the "Quote reply" option in the menu for any comment

It should be closed?

#450 if anyone picks up can close both issues

@johnjacobkenny documentation related thing can be moved to appropriate file under docs & this issue can be closed.

I'll close this since there is another issue to add it to docs #450


Add more links like linkedIn, Instagram in the LinkFree

We should add support for additional social platforms such as:

LinkedIn
Instagram
Discord

Each social should have support for the platform's icon.

Dev.to forem link can be added can I add it??

Anybody working on this?

yep I m working to add devto forem

i would like to contribute

can i work on this ?

Discord link PR is ready and also ready to merge I guess.
And Instagram link is added .
Any other social media link in your mind and if you are Confused how to make first open source contribution I m there to help.
Tag me ( @StarTrooper08 )

Can i add discord link

LinkedIn link is remaining I guess check for the closed PR.
If no one added linkedin you can add it😃

Hey @Vyvy-vi , I see that instagram and Linkedin links haven't been added yet. Can you assign me this issue?

Hey @Vyvy-vi , I see that instagram and Linkedin links haven't been added yet. Can you assign me this issue?

#675 doesn't this add the LinkedIn icon

Hey @Vyvy-vi , I see that instagram and Linkedin links haven't been added yet. Can you assign me this issue?

#675 doesn't this add the LinkedIn icon

No one has added Linkedin icon yet.

Hey @Vyvy-vi , I see that instagram and Linkedin links haven't been added yet. Can you assign me this issue?

#675 doesn't this add the LinkedIn icon

No one has added Linkedin icon yet.

But there is a open PR for this 😅 Let's wait for the maintainers

Hey @Vyvy-vi , I see that instagram and Linkedin links haven't been added yet. Can you assign me this issue?

#675 doesn't this add the LinkedIn icon

No one has added Linkedin icon yet.

But there is a open PR for this 😅 Let's wait for the maintainers

yea, sure, 😄

If Nobody's working on this I am willing to contribute

WIP: https://github.com/EddieHubCommunity/LinkFree/pull/823

React icons was added with PR #823 I'm going to close this issue as it encompasses a lot of different icons but if any icons are still missing please open individual issues for each one :)


fix error in method signature

Fix error reported in https://travis-ci.org/phpstan/phpstan-webmozart-assert/jobs/467220813#L629

Hi, thank you, but I've chosen a different fix (consistent with other repos):
https://github.com/phpstan/phpstan-webmozart-assert/commit/77282b4dec1f2a0d6b44ddf5d2f0e18d38a1a95b


HDDS-9266. Acceptance test for snapshot data validation after OM bootstrapping

What changes were proposed in this pull request?
This PR, is adding a new test setup to run under ozonesecure-ha docker env for validating snapshot data installed with a Ratis snapshot under a bootstrapped OM.
The test follows the below steps

Starts with om3 inactive and uninitialized
Calls a robot test for creating data, the test:

Creates 100 metadata keys
Creates snapshot 1
Creates 2 actual keys and set the key contents the same as the key name
Creates snapshot 2

Checks that om3 isn't running
Checks that om3 has no data
Bootstraps om3
Checks that om3 is now running
Checks that om3 has data
Calls a robot test for validating the data, the test:

Waits until checkpoints have been created on om3
Checks that  om3 isn't the leader
Transfer leadership to om3, to perform regular leader reads
Checks that the snapshots exist on om3
Runs a snapshot diff and validates results
Checks key contents

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-9266
How was this patch tested?
This patch is adding new acceptance tests. Two green CI workflows on my fork.
https://github.com/xBis7/ozone/actions/runs/6264845503
https://github.com/xBis7/ozone/actions/runs/6264807295

Thanks for working on this @xBis7

Thanks @hemantk-12 for the review and merge.


Bug

I'm not sure if this is intended or not, but I think there's a terrible bug where if you set the interval to say 10 requests per 10 minutes, and the user requests come in a 2 per minute rate. That user will always be blocked, where 10 out of those 20 requests should get in and 10 blocked.
Each time it'll reset update window whether or not the request has been fulfilled.

It was intended and is documented in the README:
Note: if an action is blocked, it is still added to the set. This means that if a user is continually attempting actions more quickly than the allowed rate, all of their actions will be blocked until they pause or slow their requests.

This limitation is the result of the fact that the solution here doesn't require a separate process to maintain the rate limit. It's a tradeoff that might not be acceptable for your use case.

This is correct.  Thanks, @neverfox!


MSFT: 8554547  This change adds passphrase conversion support on the …

…IotOnboarding producer.
The AllJoyn Onboarding protocol expects that the passphrase or network key is converted to a
HEXized Ascii String (e.g. “Apple123” converts to "4170706C65313233").  On the AllJoyn Producer
side however, the HEXized string must be converted back to a passphrase to make the WinRT call
with the correct passphrase.  This code converts the string back before onboarding the device
to the desired network.

Hi @jslobodzian, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


correcting links to blog posts

Links to blog posts are not working on a localhost because of "//"
Signed-off-by: Martin Fopp<EMAIL_ADDRESS>

Thank you!


ROCm 1.8.2 broken on Ubuntu 16.04 LTS and 18.04 LTS with kernel 4.15.0-33-generic

ERROR: Cannot create report: [Errno 17] File exists: '/var/crash/rock-dkms.0.crash'
Error! Bad return status for module build on kernel: 4.15.0-33-generic (x86_64)
Consult /var/lib/dkms/amdgpu/1.8-192/build/make.log for more information.

Distributor ID: Ubuntu
Description:    Ubuntu 18.04.1 LTS
Release:        18.04
Codename:       bionic

Installed only stable updates (not pre-release updates)
sudo apt update && sudo apt dist-upgrade
$ gcc -v
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-linux-gnu/7/lto-wrapper
OFFLOAD_TARGET_NAMES=nvptx-none
OFFLOAD_TARGET_DEFAULT=1
Target: x86_64-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Ubuntu 7.3.0-16ubuntu3' --with-bugurl=file:///usr/share/doc/gcc-7/README.Bugs --enable-languages=c,ada,c++,go,brig,d,fortran,objc,obj-c++ --prefix=/usr --with-gcc-major-version-only --with-as=/usr/bin/x86_64-linux-gnu-as --with-ld=/usr/bin/x86_64-linux-gnu-ld --program-suffix=-7 --program-prefix=x86_64-linux-gnu- --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --libdir=/usr/lib --enable-nls --with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --with-default-libstdcxx-abi=new --enable-gnu-unique-object --disable-vtable-verify --enable-libmpx --enable-plugin --enable-default-pie --with-system-zlib --with-target-system-zlib --enable-objc-gc=auto --enable-multiarch --disable-werror --with-arch-32=i686 --with-abi=m64 --with-multilib-list=m32,m64,mx32 --enable-multilib --with-tune=generic --enable-offload-targets=nvptx-none --without-cuda-driver --enable-checking=release --build=x86_64-linux-gnu --host=x86_64-linux-gnu --target=x86_64-linux-gnu
Thread model: posix
gcc version 7.3.0 (Ubuntu 7.3.0-16ubuntu3)

Thank you, I can confirm that I can recreate the problem. I can also confirm that it happens on Ubuntu 16.04 with 4.15.0-33. I believe we've found the problem, I'll try to keep you updated as we make progress on this.

Same with Ubuntu 18.04 kernel 4.15.0-33

It seemed to compile for me on 4.15.0-30 I believe, but I am having issues with Radeon Pro Render: https://github.com/RadeonOpenCompute/ROCm/issues/503
Cheers

The issue is caused by a patch backported into the -33 kernel. We have identified the problem and are working through process of releasing a hotfix.

The last version of rocm I installed was 1.7.1 beta3 posted by @gstoner, it worked fine on my Vega64 GPU and Ubuntu 16.04 with a 4.13 kernel.
I just upgraded rocm to 1.8.192, and the kernel is also upgraded to 4.15.0-33, but rocm stops working. dmesg | grep kfd returns
[    1.108303] kfd kfd: Initialized module
[    3.048527] amdgpu 0000:03:00.0: kfd not supported on this ASIC

clinfo returns 0 platform and rocminfo returns
hsa api call failure at line 900, file: /rocmdata/jedwards/git/compute/rocrinfo/rocminfo.cc. Call returned 4104
I do not see the error dump as reported by other users (/var/crash/rock-dkms.0.crash), but something isn't working.
let me know if there is a workaround.

ROCm 1.8.2 does not work with Ubuntu running Linux kernel 4.15.0-33. The workaround (until we can finish validating our patch) is to downgrade to downgrade the Linux kernel to 4.15.0-32 or below. For instance, sudo apt install linux-image-4.15.0-32-generic and choosing to boot into that from GRUB.

I just tried several older kernels (4.15.0-32 and 4.13.0-38), but strangely, I was getting the same empty clinfo output and hsa api call failure error for rocminfo as I got for the -33 kernel. The "kfd not supported on this ASIC" disappeared for the older kernels (perhaps because I removed the amdgpu drivers?).
I did remove/purge the amdgpu kernel and saw rock-dkms rebuilt the kernel  - it only gave the crash message when compiling the -33 kernel, but other older kernels were compiled smoothly. but once I reboot, and verified the kernel version wasn't the -33 kernel, I still could not get rocm 1.8 to run.
any other things I can look into or provide? thanks

by the way, is there an older version rocm (say, 1.8.1) that I can try? deb packages are preferred. thanks

Older versions are also likely to fail for 4.15.0-33; the problem is a compilation error for amdgpu that will be sensitized by the -33 release; older ROCk driver releases will also have this.
If you purge (and maybe removeall) rock-dkms, then boot into your e.g. -32 kernel, and install rock-dkms -- it should properly build the DKMS modules. It might not properly build your initramfs, so after you reboot into -32, try sudo update-initramfs -u -k all and reboot again.

We have released ROCm 1.8.3 which should fix this compatibility issue with 4.15.0-33.

confirm the new 1.8.199 release fixed the issue on 4.15.0-33. thanks


Correct License?

Hi guys,
Could somebody please just double check to see if the licenses are correct here.
https://github.com/eclipse-ee4j/angus-activation/blob/master/LICENSE.md
is BSD 3-Clause.
The Readme says:
https://github.com/eclipse-ee4j/angus-activation/blob/master/README.md
Eclipse Distribution License (EDL) v1.0
The headers in the files say:

Copyright (c) 2021 Oracle and/or its affiliates. All rights reserved.

This program and the accompanying materials are made available under the
terms of the Eclipse Distribution License v. 1.0, which is available at
http://www.eclipse.org/org/documents/edl-v10.php.

SPDX-License-Identifier: BSD-3-Clause

This isn't a trivial nitpick either, since I'd like to understand what my legal rights are and what my legal responsibilities will be if I use your code (and you might want to double check ALL projects under the eclipse-ee4j repo).
Thank you,

see https://www.eclipse.org/org/documents/edl-v10.php :

The Eclipse Distribution License is an OSI Approved Open Source License by means of the New BSD License.


Prompted for proxy auth by bootstrap-vcpkg.bat

Overview
Running vcpkg on Windows 10 from master.
Have git installed outside of vcpkg and when bootstrap-vcpkg.bat is run for the first time, a proxy auth dialog pops up. It is around the text Downloading aria2... is printed to the console.
I am not behind a proxy.
Having the proxy window pop up by default is going to break remote CI builds. Or am I missing something?
If you just click cancel, bootstrap-vcpkg.bat still completes.
There is no documentation I could find documenting preventing this behaviour.
How to reproduce:
Using Windows 10.
Ensure that you have the latest version of git installed.
Open a command prompt and navigate to a folder where you will test.
Do the following:
MKDIR vcpkg
CD vcpkg
git init
git pull https://github.com/Microsoft/vcpkg
.\bootstrap-vcpkg.bat

The popup will appear.
Is there a way to suppress the proxy pop up? I don't need to use a proxy.
Thanks.

My mistake, a proxy was set for http and https but it does not require authentication.
vcpkg should be trying anon anon first before requesting credentials using a popup IMO. If anonymous access fails, then popup an authentication prompt.


schema.json expects retryStrategy.limit to be string only

Summary
What happened/what you expected to happen?
Latest schema.json complains that the retryStrategy.limit field can only be type string:

I expect it to accept integer.
Diagnostics
What version of Argo Workflows are you running?
v3.1.0-rc11
Did this work in a previous version? I.e. is it a regression?
Yes, it works with v2.12 versioned schema. It happened since v3.0
Workflow spec to reproduce:
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: hello-world-
  labels:
    workflows.argoproj.io/archive-strategy: "false"
spec:
  entrypoint: whalesay
  templates:
    - name: whalesay
      retryStrategy:
        limit: 10

      container:
        image: docker/whalesay:latest
        command: [cowsay]
        args: ["hello world"]

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

The type of this is IntOrString. I.e it can be int or string. What is the impact please?

Unfortunately, only strings are supported until Argo upgrades to Swagger >= 3.1, or a custom format is implemented.
Swagger >= 3.1: https://github.com/argoproj/argo-workflows/issues/4817#issuecomment-778855785
Custom format: https://github.com/argoproj/argo-workflows/issues/4817#issuecomment-754819948


Ukrainian translation is of extremely low quality, probably should be removed

http://brettterpstra.com/nvalt-ukranian-translation/
It is a very low-quality machine translation, most likely provided to promote a commercial website by Max Harchenko (referenced from http://brettterpstra.com/projects/nvalt/).
It might be best to remove it altogether until a quality one is available, if ever.

I was wondering about that, thanks for the tip.
On 17 Jul 2015, at 7:25, Roman Boiko wrote:

http://brettterpstra.com/nvalt-ukranian-translation/
It is a very low-quality machine translation, most likely provided to
promote translator's commercial website by Max Harchenko (referenced
from http://brettterpstra.com/projects/nvalt/).
It might be best to remove it altogether until a quality one is
available, if ever.

Reply to this email directly or view it on GitHub:
https://github.com/ttscoff/nv/issues/343

You're welcome. It might be worth checking with some native speakers the Serbian version, too. The link provided with it seems to have the same goal, but I cannot say anything about the quality of that translation.


nautilus Could not connect to unix socket /mnt/wslg/runtime-dir/Nextcloud/socket. [Errno 2] No such file or directory

Version
Microsoft Windows [Version 10.0.22598.200]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 22.04/21.10
Other Software
No response
Repro Steps
Commands:

wsl.exe -d UbuntuTst --cd ~
nautilus (in window that opens)

Expected Behavior
No socket error
Actual Behavior
eigil@UbuntuTst:~$ nautilus&
[1] 184
eigil@UbuntuTst:~$ Initializing nautilus-image-converter extension
nautilus-wipe-Message: 11:59:28.857: Initializing
Initializing nautilus-dropbox 2019.02.14
Traceback (most recent call last):
File "/usr/share/nautilus-python/extensions/nautilus-hide.py", line 20, in
from gettext import ngettext, locale, bindtextdomain, textdomain
ImportError: cannot import name 'locale' from 'gettext' (/usr/lib/python3.10/gettext.py)
Initializing Nextcloud-client-nautilus extension
Using python version sys.version_info(major=3, minor=10, micro=4, releaselevel='final', serial=0)
Could not connect to unix socket /mnt/wslg/runtime-dir/Nextcloud/socket. [Errno 2] No such file or directory
kdeconnect-nautilus: [Errno 2] No translation file found for domain: 'kdeconnect-nautilus-extension'
Initializing nautilus-ideviceinfo extension
sys:1: Warning: Two different plugins tried to register 'NautilusDiscBurn'.
sys:1: Warning: g_type_add_interface_dynamic: assertion 'G_TYPE_IS_INSTANTIATABLE (instance_type)' failed
Nautilus-Share-Message: 11:59:29.568: Called "net usershare info" but it failed: Failed to execute child process “net” (No such file or directory)
Could not connect to unix socket /mnt/wslg/runtime-dir/Nextcloud/socket. [Errno 2] No such file or directory
Could not connect to unix socket /mnt/wslg/runtime-dir/Nextcloud/socket. [Errno 2] No such file or directory

Something like the last two lines are repeatedly written on the terminal, e.i.
Could not connect to unix socket /mnt/wslg/<something>/Nextcloud/socket. [Errno 2] No such file or directory
Diagnostic Logs
No response

It seems purging and installing nautilus solves the problem.


Add "time to pass through intersection" metric in ULTRA?

On a successful episode (when the agent reaches its goal), record the number of time steps it took the ego vehicle to pass through the intersection.
Questions:

What is recorded on unsuccessful episodes (when the ego vehicle does not reach the intersection, or does not pass through the intersection)?
In ULTRA, will we ever run into a case where the scenario does not have an intersection?

A standard set of metrics is now included in SMARTS.


Detect errors caused by exceeing max number of columns from sequel_pg

Here we are detecting the OGR2OGR warning for more than 1600 columns, but sequel_pg is throwing the error when the number of columns is 256 due to the default restriction SPG_MAX_FIELDS = 256.
Therefore, the files with more than 255 columns but less than 1600 are reported with error code = 9999 (unknown) and the user doesn't get any useful feedback.
Besides recognizing the error itself, this also involves treating:

[ ] https://github.com/CartoDB/cartodb/issues/6251
[ ] https://github.com/CartoDB/cartodb/issues/3975

At first sight, this error seems to come from geocoding guessing.

Okay, so: Let's generate a new exception from this issue that will be matched with the current TooManyColumns error code. We will update this message to say that max is 250 columns.


Version problem

guys the version now is 4.1.2-0.22, but the start project is again 3.5.0. I have to change manually the version inside config.xml after I start new project?

Thanks for the heads up @meth984!
I've updated the config.xml to default to the latest version of PGB: b27359124ec2e0556fa94ead67db89036e93ecb0 If you want to manually specify a different version, then you can uncomment the phonegap-version preference.


Button width evaluation is incorrect

According mjml, width calculation is using getShorthandAttrValue parser, that destruct composed inner-padding attribute:
https://github.com/mjmlio/mjml/blob/master/packages/mjml-button/src/index.js#L117
.Net impl uses padding-left and padding-right:
https://github.com/SebastianStehle/mjml-net/blob/main/Mjml.Net/Components/Body/ButtonComponent.cs#L191
I suggest to add attributes inner-padding-left,right,top,bottom and use attribute expansion mechanism instead of parsing inner-padding in rendering.

Pby inner-padding should be add to candidates for expansion:
https://github.com/SebastianStehle/mjml-net/blob/main/Mjml.Net.Generator/BindGenerator.cs#L210

It does that, because it uses Contains and EndsWith checks, I have tested that.


Add validations for envoy filter

Please provide a description of this PR:
EnvoyFilter allows specifying workloadSelector without labels - which could be unintentional. The present behavior is to apply envoy filters to all proxies in the namespace. This PR does not change that behavior, but adds a warning if a workload selector is specified without the labels.

Hi @varks. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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.

/ok-to-test

@varks: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
a40d5667888d1ea60a09e822a003cb93f6d79e12
link
true
/test unit-tests-arm64

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.

@varks: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
a40d5667888d1ea60a09e822a003cb93f6d79e12
link
true
/test unit-tests-arm64

lint_istio
a40d5667888d1ea60a09e822a003cb93f6d79e12
link
true
/test lint

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.

@varks: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
a40d5667888d1ea60a09e822a003cb93f6d79e12
link
true
/test unit-tests-arm64

lint_istio
a40d5667888d1ea60a09e822a003cb93f6d79e12
link
true
/test lint

unit-tests_istio
a40d5667888d1ea60a09e822a003cb93f6d79e12
link
true
/test unit-tests

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.

@varks: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
a40d5667888d1ea60a09e822a003cb93f6d79e12
link
true
/test lint

unit-tests_istio
a40d5667888d1ea60a09e822a003cb93f6d79e12
link
true
/test unit-tests

unit-tests-arm64_istio
3fd20bf435a9486cbcd03475a503bcddd292c010
link
true
/test unit-tests-arm64

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.

@varks: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
3fd20bf435a9486cbcd03475a503bcddd292c010
link
true
/test unit-tests-arm64

unit-tests_istio
3fd20bf435a9486cbcd03475a503bcddd292c010
link
true
/test unit-tests

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.

/cherry-pick release-1.15

@GregHanson: new pull request created: #40875

In response to this:

/cherry-pick release-1.15

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.

since this adds a warning validation message versus error message, should be non-breaking and safe to back port


API to get all Column references in an Expr without cloning Columns

Is your feature request related to a problem or challenge?
This API is used in several optimizer passes to recursively find all column references in an expression
pub fn Expr::to_columns(&self) -> Result<HashSet<Column>, DataFusionError>
However, it requires clone() ing all Columns (which requires allocating and copying a string) even if the caller only needs to check if there is a reference. This is inefficient and leads to a slower optimizer
Describe the solution you'd like
I would like an API that does not require copying
Describe alternatives you've considered
One version might be this (returned hasset has &Expr not `Expr)
impl Expr {
  /// Return all referenced  `Column`s  anywhere in this `Expr`
  ///
  /// For example
  /// * `a+b` would return `{a,b}`
  /// * `a + 1 + a` would return `{a}`
  fn column_refs(&self) -> HashSet<&Expr>  {
  ...
  }

Additional context
No response

Here is a first PR: https://github.com/apache/datafusion/pull/10948


fix: Nutrition edition: no listener was set on unit/"data per"

I think I've found why the "nutrition_data_per" was always wrong: no listener was set on the radio group.
This should resolve #4470
(But please test it, as I'm not 100% sure)

I'm going to merge and deploy it to internal so that we can thoroughly test.

This seems to work. In the process, I uncovered #4582

I have noticed this behavior yesterday, but wasn't sure if it was a bug or not.
If the switch is set to ON, we should hide the rest of the UI? And the same goes for the data sent to the server, isn't it?

We should hide it, but reshow it when we untick the toggle, and ensure the boolean is sent each time.

Ok I will fix that


support multiple paypal accounts from one application

I have a need to collect money for multiple clients using a single web site but as far as I can tell, the account information must be in your settings files.
So I plan to convert the settings into a tuple of tuples like settings.databases and have a selector function or parameter.
Any recommendations/suggestions on how to proceed? I am an experienced programmer (C++ and perl mostly) and I have been using paypal API's for years but I am somewhat new to python.

What you highlight is an example of a bigger problem in Django-world - that many applications/libraries are effectively singletons, due to their use of global settings.
Fixing this properly is not easy. However, rather than the route you propose, I'd suggest tackling like it this:
For any function that uses settings.PAYPAL_*, provide a way to supply the config options directly (e.g. by keyword arguments).
In fact, this has been done already in some cases. If you are using IPN, you'll see you can already use different accounts:
https://django-paypal.readthedocs.org/en/stable/standard/ipn.html
In the example code, you pass in settings.PAYPAL_RECEIVER_EMAIL, but you could pass in whatever you want. If you look at the source code for PayPalPaymentsForm, you'll see the business field has an initial value that defaults to the value in settings, but you can override at runtime due to the way Django forms work.
I don't know if the same is true for similar functions in the 'pro' app. If not, I'd be open to similar functionality.
So the basic approach is to move away from needing settings.py at all - it should be possible to pass all configuration data directly to the code that needs it. settings.py will only be used for default values.

Looking at it more - the PayPalWPP class also already has this functionality - you can pass in your own 'params' instead of the default BASE_PARAMS.
These things would ideally be documented!

Thanks Luke...I'll dig into it a little and write up what I learn. I guess
I should clone the project and push an update the the doc? Or can I just
send my notes to this list?
JLM
On Mon, Feb 2, 2015 at 10:45 AM, Luke Plant<EMAIL_ADDRESS>wrote:

Looking at it more - the PayPalWPP class also already has this
functionality - you can pass in your own 'params' instead of the default
BASE_PARAMS.
These things would ideally be documented!
--
Reply to this email directly or view it on GitHub
https://github.com/spookylukey/django-paypal/issues/77#issuecomment-72478824
.

--
Jeremy Mordkoff
Our lives begin to end the day we become silent about things that matter.
--  Dr.Martin Luther King Jr.

The easiest for me is if you:

fork the repo on GitHub
clone your fork
create a branch for your work
push your branch to GitHub
send me a pull request.

This article seems pretty good if you are new to GitHub: https://gun.io/blog/how-to-github-fork-branch-and-pull-request/
If you are just doing smallish doc updates, you can use the 'edit' button (pencil) that appears in GitHub e.g. here: https://github.com/spookylukey/django-paypal/blob/master/docs/install.rst
This streamlines the above process. However, for longer doc updates, it doesn't always work well, because you really need to check that the docs build correctly with Sphinx.
Thanks!

Closing for reasons described in the project README


Recovery error with Trezor seed

I have a 24-word recovery seed originally generated by a Trezor 1. I attempted to import it into Zap for Windows (0.7.2-beta) to take advantage of the Lightning Network features, but I receive this error:
An error has occurred
The following error has occurred when recovering a wallet:
2 UNKNOWN: wrong seed version

Is this expected? Is there a workaround?

No, it's  different type of seed. lnd uses aezeed, Trezor uses BIP39.
https://github.com/lightningnetwork/lnd/tree/master/aezeed

Thanks for the link @mrfelton . I understand now.


Source path base

It can be useful to define a base path for each individual entry in the sources to avoid paths that lead with .., especially since such paths cause Firefox to ignore the embedded source and request the actual file, which can cause an error if it is not there.
For example, this option can be used to change this.
{"version":3,"sources":["../other.css","test.less"],....}

To this.
{"version":3,"sources":["other.css","less/test.less"],....}

Without the hassle of the base option to gulp.src, which adds the full path to the destination path.

Coverage decreased (-0.77%) to 99.23% when pulling 4a8bedac21feee7439c25a61ba8d652a7315744e on AlexanderOMara:source-path-base into 8349c0c9c13305aa46e59d842400380a2ca27356 on floridoo:master.

Thanks @AlexanderOMara!
I am afraid the name sourcePathBase could lead to confusion, because the option adds a path prefix, while e.g. the base option on gulp.src() changes what the paths are relative to (effectively removing a prefix).
How about calling it sourcePathPrefix instead? What do you think?

I'm impartial towards the name, I can make the change if you like.
One thought I had was that path.join will normalize the path, possibly removing the prefix, which isn't behavior I would normally expect from something called a "prefix".
Let me know what your preference is. I'll use it whatever name it has.

:+1:

this is also useful when src/dest are in different location and the files in src might reference other files in src that would not get copied to dest
for more information: hitting this issue now while searching for a replacement for: https://github.com/gruntjs/grunt-contrib-less#sourcemaprootpath

+1 for this feature

sourcePathBase still needs to be renamed to  sourcePathPrefix . Lastly the PR needs to be rebased against 1.X . Closing until there is feedback.


Add missing semicolons

Thanks for these! I need to get the JS linter working again in my editor, hrmm.


How to override the Ctrl+S command in Markdown mode?

The "Ctrl+S" command in toast editor is used to add "strike" and how to override it for "Save" usage?
And  how to add any new command in right click context menu?

set useCommandShortcut={false}
This will remove all keyboard shortcuts and then you can override them using onKeydown event.

hello, the solution will stop the whole shortcut, i want just to prevent the Ctrl+S an keep the rest, there is a configuration for that?


Raspberry pi specific: docker.service.d/overlay.conf overrides exec command.

Server: Jessie-lite on raspberry pi 2
Client: docker-machine 0.8.1, build 41b3b25
Client modifications to be detected as debian:
$cat /etc/os-release
PRETTY_NAME="Raspbian GNU/Linux 8 (jessie)"
NAME="Raspbian GNU/Linux"
VERSION_ID="8"
VERSION="8 (jessie)"
ID=debian   <---------------------
ID_LIKE=debian
HOME_URL="http://www.raspbian.org/"
SUPPORT_URL="http://www.raspbian.org/RaspbianForums"
BUG_REPORT_URL="http://www.raspbian.org/RaspbianBugs"

Command:
docker-machine create \
  --driver generic \
  --engine-storage-driver=overlay \
  --generic-ip-address rpi-ds1-000 \
  --generic-ssh-user=pi \
  rpi-ds1-000
Running pre-create checks...
Creating machine...
(rpi-ds1-000) No SSH key specified. Assuming an existing key at the default location.
Waiting for machine to be running, this may take a few minutes...
Detecting operating system of created instance...
Waiting for SSH to be available...
Detecting the provisioner...
Provisioning with debian...
Copying certs to the local machine directory...
Copying certs to the remote machine...
Setting Docker configuration on the remote daemon...
Error creating machine: Error running provisioning: Unable to verify the Docker daemon is listening: Maximum number of retries (10) exceeded

It seems that there is an additional folder with a file that overrides docker daemon exec command.
/etc/systemd/system/docker.service.d/overlay.conf
[Service]
ExecStart=
ExecStart=/usr/bin/dockerd --storage-driver overlay -H fd://

Solution (?)
I'm no expert in systemd, but renaming /etc/systemd/system/docker.service.d to /etc/systemd/system/docker.service.d.disabled worked.
sudo mv /etc/systemd/system/docker.service.d /etc/systemd/system/docker.service.d.disabled
sudo systemctl daemon-reload
sudo systemctl restart docker

now on the client:
docker-machine ls
NAME          ACTIVE   DRIVER    STATE     URL                      SWARM   DOCKER    ERRORS
rpi-ds1-000   -        generic   Running   tcp://rpi-ds1-000:2376           v1.12.1

This has been fixed in https://github.com/docker/docker/pull/26199, I don't know when it will be reflected on the downloadable install script though. In any case you found a workaround until then 👍

Perfect. I'll be closing this then.


(PDB-5645) Add query monitor

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


RemoveDuplicates and nodeAffinity

What version of descheduler are you using?
descheduler version: 0.26.1
Does this issue reproduce with the latest release?
Not sure, not compatible with my k8s version
Which descheduler CLI options are you using?
I am using the helm chart with no changes: https://artifacthub.io/packages/helm/descheduler/descheduler
Please provide a copy of your descheduler policy config file
I am using the helm chart with no changes: https://artifacthub.io/packages/helm/descheduler/descheduler
What k8s version are you using (kubectl version)?
kubectl version Output
Client Version: version.Info{Major:"1", Minor:"26", GitVersion:"v1.26.14", GitCommit:"6db79806d788bfb9cfc996deb7e2e178402e8b50", GitTreeState:"clean", BuildDate:"2024-02-14T10:42:41Z", GoVersion:"go1.21.7", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v4.5.7
Server Version: version.Info{Major:"1", Minor:"26+", GitVersion:"v1.26.14-eks-b9c9ed7", GitCommit:"7c3f2be51edd9fa5727b6ecc2c3fc3c578aa02ca", GitTreeState:"clean", BuildDate:"2024-03-02T03:46:35Z", GoVersion:"go1.21.7", Compiler:"gc", Platform:"linux/amd64"}

What did you do?
I have a ReplicaSet that should create 2 replicas. On that ReplicaSet (and thus the pods) I have the following:
nodeAffinity:
  preferredDuringSchedulingIgnoredDuringExecution:
    - weight: 1
      preference:
        matchExpressions:
          - key: eks.amazonaws.com/capacityType
            operator: NotIn
            values:
              - SPOT

So: do not run on any AWS spot instances.
Currently I only have a SINGLE node that matches this selector. (And 3 that spot instances)
What did you expect to see?
I expect the descheduler to NOT evict any pods from this deployment on that single node. It should not show other pods as valid options.
What did you see instead?
The pods keep getting evicted.

Why does it show as if 4 nodes would be ok while only 1 matches to the nodeAffinity rules?

/remove-lifecycle stale