6x Last Buy Price

Hello,
I don't know why, but after pulling the latest version of the bot with Docker compose and git pull, I just received 6 notifications about last buy price updated, while there was only one trade. I double checked in the GUI and the sum is correct, the amount of trades is correct, but the last buy price is adjusted as if it had traded 6x the amount. The slack message shows as if newTotalAmount also increased.

I checked it manually and now last buy price was 89.13 when it's supposed to be 89.86.

There's seemingly no Wiki entry about reverting an upgrade. Now I also got two random of these messages:
Unable to fully release the lock on resource "redlock:trailing-trade-symbols:FTMBTC-latest-candle".
Unable to fully release the lock on resource "redlock:trailing-trade-symbols:BCHBTC-latest-candle".
Even tho I am currently not trading with BTC as base.

Maybe most strange is seeing the commits that's been made, they don't seem like changes which could break the bot.
It is now trying to sell with a quantity as if I had done 6 more buy trades even if I adjusted the last buy price. What's strange then is the quantity in the GUI does not match what I suppose is stored in Mongo db.
I looked into ways of reverting both git pull and dockers image, but I'm not sure, is it possible to change the server yaml to an earlier version than latest? I have backed up the DB before updating, but also made some more trades after. Not sure if revert without DB restore would work.
Also, I haven't ran pip install -r requirements in a while, maybe that's the culprit if there was library updates?

This makes currently trailing trades stop updating too as soon as I get the "Unable to fully release the lock on resource "redlock:trailing-trade-symbols:UTKBTC-latest-candle"."

Can you try to reboot your server to reset your memory resources?
On Wed, Mar 29, 2023 at 10:37 AM dasbts @.***> wrote:

This makes currently trailing trades stop updating too as soon as I get
the "Unable to fully release the lock on resource
"redlock:trailing-trade-symbols:UTKBTC-latest-candle"."
—
Reply to this email directly, view it on GitHub
https://github.com/chrisleekr/binance-trading-bot/issues/618#issuecomment-1488169664,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALMG667HM6XQOJD3QBZNNLW6PYGJANCNFSM6AAAAAAWK7PN2I
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Rebooted the server.

Same issue on 0.0.97 docker image. 0.0.96 won't even start and I had to --remove-orphan when using docker-compose pull too..

Is there no way to downgrade to 0.0.96? I'm having sell orders currently trailing and every 20 minutes or so it just freezes leading to having to restart. :(

I haven't been developing docker images myself, so not sure if I have to build before changes will be made, but tried changing this PR parts to 1500 instead of 1000, I suppose that's the lock time that they talked about in the thread.
https://github.com/chrisleekr/binance-trading-bot/pull/522/files

Maybe i can download the 0.0.96 version? The thing is, I have both entered and closed some positions since my last backup of the database...
Currently the bot is completely broken since 0.0.97. Let me know if I can help, working as a consultant in finance and IT daily doing quite some programming.
Seems like this also freezes my server as I'm not able to connect to it through TeamViewer all night and these errors ha en been occurring every 2-5 seconds for mainly tokens I am not even trading/monitoring.

Hi @dasbts
I am sorry for the late reply. I am swamped, so I couldn't have a chance to look it up.
I think, for some reason, the data is messed up. Without a detailed log, I won't be able to troubleshoot the issue.
So I agree with you about downgrading. So good that you had a backup as well.
To downgrade, please follow the steps:

Shutdown docker containers with volume deletion.$ docker-compose down -v

Update docker-compose.server.yml

image: chrisleekr/binance-trading-bot:latest to image: chrisleekr/binance-trading-bot:0.0.96
image: chrisleekr/binance-trading-bot:tradingview to image: chrisleekr/binance-trading-bot:tradingview-0.0.96

Start docker containers$ docker-compose -f docker-compose.server.yml up -d

This should launch the bot with a clean database. Then you can use your backup to restore.
However, it means you are tied to this 0.0.96 version and won't be able to upgrade.
If you are happy with it, that is fine. But if you want to upgrade in the future, I suggest that you launch the bot with the latest version and a clean database. Then set the last buy price manually.
For the redlock error, can you tell me what server specification you run? Especially, disk speed.

Ok, so it seems like the ENOBUFS error is related to memory, so I did some performance upgrades I seem to have missed.
First off, I'm running Debian 11 with not so good laptop Acer Aspire 5750 which only has 6 GB RAM and a 640 GB HDD. The HDD seem to have 2500 MB/s speed when running hdparm test after setting APM to 254 to get more performance, but I don't trust that result 100%. I previously only had a swap file, but have now read up on and enabled zram-tools (zram-config on newer Debian) which allowed compressing the RAM. I have about 25% free RAM at all time when running the bot together with some other homebrew applications.
This seem to have worked partially, but then I realized, I should print out the logs for redis since redlock is connected to this.
1:M 31 Mar 2023 15:15:07.655 # WARNING: The TCP backlog setting of 65536 cannot be enforced because /proc/sys/net/core/somaxconn is set to the lower value of 1024.
1:M 31 Mar 2023 15:15:07.655 # Server initialized
1:M 31 Mar 2023 15:15:07.655 # WARNING overcommit_memory is set to 0! Background save may fail under low memory condition. To fix this issue add 'vm.overcommit_memory = 1' to /etc/sysctl.conf and then reboot or run the command 'sysctl vm.overcommit_memory=1' for this to take effect.
So, I have added the following to my /etc/sysctl.conf:
fs.inotify.max_user_watches=254800
vm.vfs_cache_pressure=300
vm.swappiness=100
vm.dirty_background_ratio=1
vm.dirty_ratio=50
vm.overcommit_memory = 1
I also added to the cache.js file:
await redlock.lock(redlock:${key}, 1500);
Instead of the 1000 added in the following commit:
https://github.com/chrisleekr/binance-trading-bot/pull/522/commits/98be882bc1d2995e4e3754fe201f3f336075d122
In docker-compose.server.yml I also changed the following which was suggested to fix the TCP backlog setting warning:
sysctls:
net.core.somaxconn: 65535
It was previously 1024 on the binance-redis image. I am a bit confused tho, shouldn't this be at least 65536 according to the warning? Also looking in /proc/sys/net/core/somaxconn it is 4096.
Then, since it is ioredis we are working with in the cache.js, I figured, why not look into the config more in depth since people were posting about params in the redlock issue that is about this same thing about not fully releasing lock, I found I could add the following params. Not sure if any/all is needed, or if I chose a too high value such as for maxRetriesPerRequest, but doing it to test:
keepalive: true,
retryStrategy(times) {
const delay = Math.min(times * 50, 2000);
return delay;
},
maxRetriesPerRequest: 50,
enableAutoPipelining: true,
showFriendlyErrorStack: true
const redlock I also changed to:
retryCount: 20,
retryDelay: 300
Will update on any progress. So far it seem to be way more stable, but cannot say for certain yet. At least I am able to keep the GUI open for longer times.

Gonna keep digging ASAP, got a bit much to do outside this too, but I believe it would be good to somehow make it keep retrying, as whenever the redlock fails, it will just freeze the element which is locked seemingly forever. Either that, or it just keeps awaiting the lock.release as it seem the code is made to do so, and if it's never released due to something that only partially unlocks the element, or if it's just never getting that await to complete, I guess that's why it freezes.

Ok so after trying a lot I might have figured it out.
First off, it was seemingly not correct to add the now commented code to the redis const:
const redis = new Redis({   host: config.get('redis.host'),   port: config.get('redis.port'),   password: config.get('redis.password'),   db: config.get('redis.db')   // keepalive: true,   // retryStrategy(times) {   //   const delay = Math.min(times * 50, 2000);   //   return delay;   // },   // maxRetriesPerRequest: 50,   // enableAutoPipelining: true,   // showFriendlyErrorStack: true });
I think this is not introduced until later releases than the binance-bot uses. So, instead, I tried updating redlock with NPM to the nevest beta. I added a 5 times retry of 1s each try for each "await lock.release()", changed all "redlock.lock" to "redlock.acquire", and all "lock.unlock" to "lock.release".
I also had to change "const Redlock = require('redlock');" to "const { default: Redlock } = require("redlock");", something which seem to become deprecated in the future.
So far this seem to be ok, will do some testing for a day or two first. Thinking of trying instead of X retries, just extend the lock with 1 second or such and retry until it works to release.
HOWEVER, this seemingly introduced a new bug:
https://github.com/mike-marcacci/node-redlock/issues/168
I "solved" this by temporarily adding "if (err.message.includes('quorum'))" to the handleError in error-handler.js.
"I believe this is a bug in the v5 implementation. When you attempt to set a lock using acquire() and the resource is locked, Redlock will throw an ExecutionError, but it should throw a ResourceLockedError (as it did in the previous version)."
There's also an alternative called "redlock.using" which allows adding:
// The minimum remaining time on a lock before an extension is automatically
// attempted with the using API.
automaticExtensionThreshold: 500, // time in ms
IF this works, the only thing I'm worried about is if something is locked, and it keeps triggering for example a buy order, does this mean there is a queue building up with multiple buy scenarios happening? Since that was the reason this issue was made. I think either that is still a possibility, or since I used to have poor server performance prior to tuning, maybe the RAM messed up the buy order?
By the way. I retract the ENOBUFS (which I even closed the issue for before) statement that it seem to be RAM (which is usually the case). I have optimized so have currently about 30% free all the time. However, it seems like tuning the CPU to network-performance profile instead and that seems more stable. This after I observed the error message again when CPU usage was high.
https://github.com/chrisleekr/binance-trading-bot/issues/617

Ok, so I received no errors after the previous attempt with awaiting the lock.release. However, this made the ticker stop completely with the GUI still working and it saying there was no update for 5 minutes (with the clock being stopped hours ago). Still an improvement since before the whole GUI would not be accessible.
I'm going to look through the commits again since previously it wasn't the tickers that stopped working, but rather for example the filter of profits between D, M, Y, all. If that can be fixed I believe there would not be any need for restarting the bot anymore.
The strange thing is I also then noticed that when losing internet, every open websocket seemingly breaks. Miight need a quick websocket check if there haven't been any activity for some minutes.

@dasbts
First of all, I really appreciate it if you follow the bug template if you submit the bug.
Is this continuously happening or it was just once off?
And if it is happening, try to get the log and export it.

That will tell what exactly happened.
I will close the issue for now. If you find it is happening again, please re-open and upload the log.

It happened once thankfully as otherwise I would have to switch out this bot, but since then I have also updated redlock, migrated the redlock by changing all "redlock.lock" to "redlock.acquire", and all "lock.unlock" to "lock.release". I also added a 30 times retry for each "await lock.release()" with 0.5 second sleep between each try. This have solved any issues with the GUI not responding or opening for me as well.
I also added the optimization to the Debian server as mentioned above and in general have much better performance now so hopefully it won't happen again.
About the logs, I believe I checked those when this happened, but not 100% sure, anyway it is very strange that it did only update the lastbuyprice but not the amount and number of grids/trades used in the GUI.

@chrisleekr I had a similar thing happen today. Not 6x Last Buy Price, but 0. Luckily it closed at a profit, but it now shows wrong in the GUI of the bot trade results, as it thinks the buy price was 0.
Here is the Slack log:
Jeff Bezos
APP  00:11
(22:11:52.776) TOMOUSDT Sell Action Grid Trade #1: STOP_LOSS_LIMIT

Order Params:
{
"symbol": "TOMOUSDT",
"side": "sell",
"type": "STOP_LOSS_LIMIT",
"quantity": 200.8,
"stopPrice": 2.1805,
"price": 2.1695,
"timeInForce": "GTC"
}
Current API Usage: 41

00:11
(22:11:53.152) Execution failed:
Job: Trailing Trade
Code: -2010
Message:
Stop price would trigger immediately.
 Delete last buy price.
{
"lastBuyPrice": 2.09155804,
"currentPrice": 2.2775,
"minNotional": "10.00000000",
"openOrders": []
}
Current API Usage: 133
00:15
(22:15:20.635) TOMOUSDT Profit:
Profit: 459.16936
ProfitPercentage: 0
Total Buy Amount: 0
Total Sell Amount: 459.16936

The log in the GUI was autodeleted due to the default config, will have to increase this for future events.


Check if URL is from packtpub

If someone (maybe in a fork) creates a page with the same DOM as packtpub and supplies a newsletter URL that points there, he could make the script download malicious or spammy PDFs without the users knowledge. So it's better to check if the provided URL actually points to packtpub.

Presumably if a user creates such a malicious fork, they would also remove this safety check?

Sure, but you would see the change in the source. Right now it's just a hidden security hole that every user has to find themselves. Also the check acceps any URL with http as a valid packtpub URL, which is just plain wrong.

Ah. It's the newsletter. Yeah, that makes sense then.

New tag: v2.2.4


Side panel tabs overflow without option to display them

When the side panels are horizontally too small to display their tabs completely, they overlap on the outer border of Firebug, though there is no option to bring them into display.

When switching away from the Firebug theme and back to it and resize the side panels you'll see an overflow menu button:

Though that button disappears when you shrink the side panels again, i.e. it's not positioned correctly.
So one solutions to this is to correct the display of this side panel overflow button.
A second solution would be to have scroll arrows at the left and right of the panel tabs in this case like for the main panel tabs.
Sebastian

Thanks for the report, should be fixed now, let me know if not.
Honza

Works. Thanks for the quick fix!
Sebastian


Need api to dynamic configure plugins, such as add or remove consumers of key-auth plugin.

Why do you need it?
Our platform opened some apis to users, and we decide to use api key to protect our apis.  Every user needs to apply an api key in our platform first, and call apis with this api key, so we need to dynamic configure consumers of key-auth plugin, but I not found any documents about this.
How could it be?
Configure plugin through api or using Kubernetes CRD.

The API design is not finalized yet. But for now, you can use http://higress-console-domain/swagger-ui/index.html to check out API document generated by Swagger.

OK. Is there any plan or when to finalize the api design?

OK. Is there any plan or time deadline to finalize the api design? BTW. the demo console is denied to access right now. As there have conflicts between Higrress and Rancher agent, so we cannot install Higress to our cluster to experience Higress.

The deadline hasn't been settled yet. The demo console will be fixed soon. I will let you know once it comes alive.

@jaggerwangThe demo console has been restored.

There is almost no documentation about each api, can you guys give an example to configure a route with key-auth plugin.

@jaggerwang 可以进钉钉群，目前这块文档还未最终定稿，但有内部API开发文档，在钉钉群文档里

我们不会提供Console的Admin API，这块的API后面可能会跟随版本发生变化，但底层的Ingress/Gateway API模型是不变的。可以基于Console的Java SDK进行集成，SDK封装了对Ingresss/Gateway API的操作，这块可以补充下文档 @CH3CHO
https://github.com/higress-group/higress-console/tree/main/backend/sdk

我们不会提供Console的Admin API，这块的API后面可能会跟随版本发生变化，但底层的Ingress/Gateway API模型是不变的。可以基于Console的Java SDK进行集成，SDK封装了对Ingresss/Gateway API的操作，这块可以补充下文档 @CH3CHO https://github.com/higress-group/higress-console/tree/main/backend/sdk

后续在完善 SDK 的配置机制后会提供相关文档。


Increase min saw amount before affiliate fee.

error
:
"affiliate swap failed: output RUNE (1910807) is not enough to pay transaction fee"

Min amount before the affiliate fee increased to $1001.


Truncate and map filenames that are too long in test archiver

Issue: CAP-1203
What Changed
Sometimes, filenames are too long to be written to the file system. Typically, this happens if a file name is longer than 256 bytes. When this happens, we want to truncate the file name and map references to this file to the new, shortened filename.
How to test
There are tests for this in write-archive/index.test.ts, but you can follow these steps to reproduce:

Link this branch so that you can use test-archiver locally or install version<EMAIL_ADDRESS>of the package using: yarn add -D<EMAIL_ADDRESS>Clone https://github.com/tevanoff/chromatic-e2e-issues
Run yarn playwright test toolong.test.ts
Verify that no error occurs.

Change Type

[x] patch

📦 Published PR as canary version: 0.0.27--canary.24.6561537.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

Perhaps we should have a zoom meeting to discuss this?

Agree we should get alignment on this. I'm not necessarily opposed to this approach -- one advantage of rrweb's JSON format is we can do transformations like this to it fairly easily. I wonder if we could switch back to using a vanilla version of it and use a similar post-processing step to achieve the changes I made to it (I'm thinking probably not but I figured I'd ask).

Perhaps we should have a zoom meeting to discuss this?

Agree we should get alignment on this. I'm not necessarily opposed to this approach -- one advantage of rrweb's JSON format is we can do transformations like this to it fairly easily. I wonder if we could switch back to using a vanilla version of it and use a similar post-processing step to achieve the changes I made to it (I'm thinking probably not but I figured I'd ask).

After speaking with @thafryer, his opinion is we should move forward with this solution, since it unblocks CAP-1204 and CAP-1124. If we decide that we're going to remove rrweb, or, even move this code into rrweb, then we can do that in a followup commit.

:rocket: PR was released in v0.0.27 :rocket:


No prompt to save scripts on closing

Description
Previously, when closing a locally saved sql script or when closing the app I was prompted if I wanted save save it or not  - this behavior has now changed in the latest release (or one of the recent ones) so that it just closes without the prompt, this means that whenever I close things its auto-saving so my scripts are never a known state.
There is no auto-save option I can see in the preferences
Using version <IP_ADDRESS>408041450 (Mac)
DBeaver Version
<IP_ADDRESS>408041450
Operating System
Mac 14.6.1 (23G93)
Database and driver
No response
Steps to reproduce

Open DBeaver
Open pre-saved sql script/s
Amend and use script/s as needed
Close script or application (no save prompt)
Re-open DBeaver
Re-open script from before - any changes previously made are there

Additional context
No response

I also get the exact same problem. I have a project full of scripts that I use regularly, when I make a change and close the script. The change is persisted without asking if i wish to keep it.
Steps to re-produce taken

Open Dbeaver
Open a script from the project
Amend the script
Close script
Reopen and the change persists

Previously i would be asked if i wanted to save the change. This no longer seems to happen, i can't be sure how long this has been happening as i've only just noticed. I've looked through the settings for any form of autosave and cannot seem to find any settings relating to it?
Dbeaver version
Version <IP_ADDRESS>408041450
OS
MacOS 14.5

Since DBeaver 24.1.1 the setting Auto-save editor on close  is turned on by default.
Check this thread for more information:
https://github.com/orgs/dbeaver/discussions/35158

Since DBeaver 24.1.1 the setting Auto-save editor on close  is turned on by default. Check this thread for more information: https://github.com/orgs/dbeaver/discussions/35158

Many thanks for providing a solution.
Very curious to know why this was changed, seems a bit backwards to me! (especially as its been that way for a very long time).

Very curious to know why this was changed, seems a bit backwards to me! (especially as its been that way for a very long time).

As I am just an ordinary user, I can only guess.
I think it can be benefitial in cases when a user has a lot of opened and unsaved tabs with scripts, and he wants to close the application (for example, to install the DBeaver's updates or reboot the system quickly) and doesn't want to accept confirmation about saving changes in 30 opened scripts 30 times.
The problem is that some accidential or unwanted changes can be automatically saved in scripts.
Lets wait for official answer from DBeaver's team.

Very curious to know why this was changed, seems a bit backwards to me! (especially as its been that way for a very long time).

As I am just an ordinary user, I can only guess. I think it can be benefitial in cases when a user has a lot of opened and unsaved tabs with scripts, and he wants to close the application (for example, to install the DBeaver's updates or reboot the system quickly) and doesn't want to accept confirmation about saving changes in 30 opened scripts 30 times. The problem is that some accidential or unwanted changes can be automatically saved in scripts and user may not notice it or notice it too late.
Lets wait for official answer from DBeaver's team.

Yeah I can get that viewpoint and approach for those that use it very heavily like that, I only use it with a max of about 4 or 5 scripts open at once.

Since DBeaver 24.1.1 the setting Auto-save editor on close  is turned on by default. Check this thread for more information: https://github.com/orgs/dbeaver/discussions/35158

Ace, Thank you for that. I guess there is a rationale behind turning it on by default, I can definitely see some use cases. I didn't think to check the top level of that menu to be honest. I assumed it would be a sub entry.
Most scripts i use are pulled from Devops / github and are template based so having the template overwritten can be awkward at times. Problem solved now though :)

Lots of popular editors have saves file on close automatically. We received a request from the community (#29925) and decided to turn it on.
If you want it to be turned off by default again, you can vote for the issue #34514  with 👍 under the issue description.
I'm closing this issue as a question.


[Event Request] Report 12152 "Create Subcontr.Transf. Order"

Describe the request
In function CheckPurchLine of report 12152 I need an event OnAfterProdOrderComponentSetFilters(var ProdOrderComponent: Record Prod. Order Component; PurchLine: Record Purchase Line)

Additional context
I need this event to add additional filters based on custom fields on Production Order Component
Internal work item: AB#548964

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 24337.


docs: add organizations to generated docs

Adds the organizations docs to the generated docs commands.

@brmzkw I don't have authorization to merge, so please feel free to merge this when you are ready.


StretchyFactor for animation purpose

Dear brother,
We would like to enhance and add nice thing to be used in the header something called StretchyFactor which is help for animation effects for example: change alpha / size / scale / position for any object inside the headerView.
Lets suppose we have UIButton in the headerView and while scrolling down/up we need to change alpha of that UIButton. If the user scrolling to segmentTopSpace so the UIButton alpha will be 0 then if the user scrolling up the alpha will be increase based on StretchyFactor or position.
Hope you got my idea. Am trying to use a sample code in other repository but it doesn't work.
CGFloat ownHeight = CGRectGetHeight(self.bounds);
CGFloat contentHeightDif = (self.maximumContentHeight - self.minimumContentHeight);
CGFloat maxContentViewHeight = ownHeight - self.verticalInset;
CGFloat newStretchFactor = (maxContentViewHeight - self.minimumContentHeight) / contentHeightDif;

There has a block "scrollViewDidScrollBlock" in HHHorizontalPagingView and it will call back when your scrollView scrolling up/down. you can use the variable "offset" to control the alpha of your button.
when your scrollView is in the initial state, the alpha of button should be 1.0f, and the offset of scrollView is - (headerHeight + segmentHeight).
if your scrollView scrolling to segmentTopSpace, the alpha of button should be 0.0f, and the offset of scrollView is -(segmentTopSpace + segmentHeight).
Then you can use the "two point form" (x - x1) / (x2 - x1) = (y - y1) / (y2 - y1) to calculate the alpha value.
(x + (headerHeight + segmentHeight)) / (-segmentTopSpace - segmentHeight) + (headerHeight + segmentHeight) = (y - 0) / (1 - 0)
The result is y = (x + (headerHeight +segmentHeight )) / (headerHeight - segmentTopSpace)
Make a test:

The alpha will be changed from 0.f to 1.f when you scrolling to segmentTopSpace.

@Huanhoo Thanks alot, your effort is highly appreciated I will make a smoke test and reply soon.

Hi @Huanhoo
Well done, its work fantastic. What do you think if you want to resize an UIImageView or change (y,x) based on scrolling.
Any hint?
Thanks alot

if the aspect ratio of the image is consistent, you can use the same way to implement it. "two point form", y2 and y1 is the max or min size of your image

Hi,
I tried but unfortunately I didn't get the correct solution and it will be appreciated to explain it with an example.

Ok, check this file.
HHHorizontalPagingView.zip

Am really don't know how to say Thank you, Really appreciated.

:)


HTTP 404 after connecting to the interface to view emails

Hello,
First of all, thank you for all the work you have done !
I installed on a server the Zimbra Ubuntu 20.04 release that I downloaded from your site and I have an issue after connecting to the interface to view emails.
The URL on my browser is https://xxxx.yyyy.com/modern

In the inspector of my browser, there is a 404 error for this URL.
Any idea ?
Best regards,

Hi,
This is a question you should be asking on the Zimbra forums - https://forums.zimbra.org - Zimbra configured it to use modern interface by default, so it means you need to edit the COS to not use the Modern web interface.  This has been covered already on the Zimbra forums.
Support here is for the build process only.

Most likely something in Zimbra's code defaults it to this.

The reason why innotel, is because they do this in their instructions:
#Disable Modern UI

nano /opt/zimbra/jetty_base/webapps/zimbra/WEB-INF/classes/messages/ZmMsg.properties

#Edit lines (673, 679, 680)to reflect

clientAdvanced = Default
clientPreferred =
clientModern =

the preferred way is as mentioned in my previous post by logging into Zimbra WebAdmin and edit default COS - which most likely does a similar thing, eg: update the text file, or perhaps other stuff in the background.

Just as an additional info:
Most values within Zimbra admin cannot be configured as I get several SOAP errors.
What worked was following CLI command with zimbra user:
zmprov mcf zimbraModernWebClientDisabled TRUE & zmmailboxdctl restart afterwards.


(Nested) Nested Content: DocType modal position

This came up on the forum, logging here for reference.
When you have a nested NC, the DocType selection modal position appears to be anchored much lower down.
https://our.umbraco.org/projects/backoffice-extensions/nested-content/nested-content-feedback/78105-suggestion-on-popup-to-add-content

Unsure whether this is an issue for the Umbraco Core version of NC?

Closing this one off, as I doubt we'll change this in the standalone version of the package, but it could be raised as a discussion for the native Umbraco version.


Add ignore_run_exports for python

This PR adds a ignore_run_exports declaration for python itself, to fix the latest build issue. The build number has not been bumped because the last master build failed.
Checklist

[x] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The build section contained an unexpected subsection name. error_overdepending is not a valid subsection name.
The build section contained an unexpected subsection name. error_overlinking is not a valid subsection name.


Delete Schema

I would like to work on this one once the PR for Delete Catalog is done! 😄

Hi @robertgv I need to get the delete schema functionality in, do you mind if I take this issue?


Footnote fact

Changes

Fixes two more instances of old "sources" markup. :grimacing:

For review: @DoSomething/front-end

:+1:


Add Font pipeline to webpack

We need the roboto font to be inlined to look best with the Material-UI stuff.

Closed via #36


[ISSUE-359] Monitoring mixins

Resolves: https://github.com/prometheus-community/elasticsearch_exporter/issues/359
Alerts and Rules are something I've created for our needs. Dashboard is something I borrowed from grafana net and modified... Dashboard also has cert expiry which could be a bit too specific for our setup..
Haven't had time/willingness to properly define the dashboard, but this could be better than nothing?

My main issue with the grafana net dashboard was that it had hardcoded datasource and I think it didn't work well if there was more than 1 exporter gathering data for all the nodes in the cluster.
In our setup we extend on this even more with ElasticsearchIndexingStopped and ElasticsearchClusterDegraded alerts, but these are too specific for our setup and needs, and have thus been kept separate from the exporter code.
Sorry for all the excess PRs maybe they can be squashed or I can raise a new PR with git diff from upstram..

Hmm.. Following the DCO check instructions created much more change than I wanted :/


uptime robot-able search health check

Feature Request
Desired behaviour
Need a health-check for elastic search (unhealthy both  when the index is not populated and/or down).
┆Issue is synchronized with this Jira Story
┆Fix Versions: Dockstore 1.6
┆Issue Number: DOCK-284

Seems working on staging, hooked up to uptime robot with success


Seems like page-visibility option on BrowserWindow is broken

In Electron 0.34.0, it doesn't seem like my page-visibility setting is being honored. I'm using it here to ensure test windows continue running tests even if they're in the background. It was working, but now it doesn't seem to be.

It seems that from Chrome 45 setting page-visibility doesn't prevent performance throttling any more, can you try using the --disable-renderer-backgrounding command line switch?
app.commandLine.appendSwitch("disable-renderer-backgrounding");

Refs #2822.

Is this something that can be done for just a test window or is it application-wide?

This flag is application-wide, you can append it before creating the test window so normal users won't be affected, but the windows created after that will still be affected though.
From the source code the only way to prevent only one window being put into background is to keep playing audio in the page.

One more question... is it possible to only append a flag for a single window's render process?

Yeah it is possible for renderer process, but --disable-renderer-backgrounding is read in the main process.
Playing audio is not a so bad option though, it can be done by creating an empty audio on the fly.

:zap: Tried the audio hack. Seems to work. Thanks a ton for that idea!

I'm still getting timeout failures on CI where animation frames aren't firing. Am I doing the sound thing wrong? I thought it was preventing throttling locally but maybe I was mistaken: https://github.com/atom/atom/pull/9485/files

It seems that Chrome had a bug that it doesn't update the background information immediately when audio starts/stops, and it got fixed very recently, I think we are hitting this bug. The --disable-renderer-backgrounding switch seems to be our only option.

I'm actually seeing that it's working locally. Running audio as I did in the above PR causes tests to complete in the background locally. I am seeing 20 second animation frames on CI regardless of the throtlling settings, either with the flag or the audio hack. For now I just increased the timeout, but it would be nice to know why these frames are taking so long.

@zcbenz Can you reference me to the relevant location in the Chromium source so I can open an issue on the Chromium bug tracker. After talking to some developers at the Chrome Dev Summit it seems like this is something they care about correcting.

The code that determines whether to lower the priority of renderer process is located at: https://code.google.com/p/chromium/codesearch#chromium/src/content/browser/renderer_host/render_process_host_impl.cc&sq=package:chromium&type=cs&l=2369
And the page-visibility option is implemented at:
https://github.com/atom/electron/blob/master/atom/renderer/atom_renderer_client.cc#L207-L218
I'm not sure from when it stops working in Chromium.

Thanks for sharing your solution @zcbenz!


gitleaks: 8.20.0 -> 8.20.1

Diff: https://github.com/zricethezav/gitleaks/compare/refs/tags/v8.20.0...v8.20.1
Changelog: https://github.com/zricethezav/gitleaks/releases/tag/v8.20.1

Things done

Built on platform(s)

[ ] 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
[ ] 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

[ ] Fits CONTRIBUTING.md.

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

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

1 package built:

gitleaks


Vendor<EMAIL_ADDRESS>

What this PR does / why we need it:
Vendor<EMAIL_ADDRESS>Which issue(s) this PR fixes:
Fixes #1349
Special notes for your reviewer:
Release note:

`sigs.k8s.io/controller-runtime` is updated to `v0.2.0-beta.5`.

Let's not merge it before a new release is cut.

Let's not merge it before a new release is cut.

Yep, sure, let's consider it after 0.31.0.


Adding Transparent Hover Effect code

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@izwanrusli Well done, excellent addition!


Add aggregates into config.md doc

Add aggregates: into the Aggregating functions configurationchapter sample

Merged, https://github.com/oroinc/platform/commit/45da866f7c5e12c357f2872d6b1f35abaa7ba3f2.
Thank you for your contribution, @sylvainraye!


there is no moveaxis method in np

there is no moveaxis method in np, or other method can replace this method?

Fixed for next release.
np.moveaxis

thanks!


Addition of mas uninstall support

Addition of functionality to allow the removal of Mac App Store deployed applications

Is there anything else you require to approve?


ENV_DIR requires an absolute path

I ran the script with ENV_DIR=env1 and was greeted with:
~/gcp-pcf-quickstart/src/omg-tf ~/gcp-pcf-quickstart
loading environment config: open env1/config.json: no such file or directory

I am certain that the env1/config.json file exists at this relative path.
If I rerun with ENV_DIR=/Users/me/gcp-pcf-quickstart/env1 everything works.

Looks like this happens because the deploy script changes into the ENV_DIR's terraform directory and then runs omg-cli commands.
Might be best just to add a note to the help message and/or README, as checking for relative paths in bash in a portable manner is somewhat nasty.


Iframe not working how to embed video iframe

Expected behaviour
Actual behaviour
Steps to reproduce
Environment

Browser:
Version:
Operating System:
Version:

Console errors (if any)
Link to where the bug is happening

Please complete the template.


Task bar settings

Description of the new feature / enhancement
Can we add taskbar autohide delay and mouse hover delay to show it back?
i.e., When I hover on the taskbar when it's auto hidden, show after X seconds. When I move out of the taskbar, hide after X seconds.
https://learn.microsoft.com/en-us/answers/questions/832901/add-delay-time-to-the-taskbar-auto-hiding-when-set.html
Also, hot corners like in ubuntu would be nice.
Scenario when this would be used?
OLED displays to prevent burn in, I autohide taskbar. AFAIK, There is no option (other than regedit) in windows to autohide after X seconds or show again after X seconds
Supporting information
No response

/feedbackhub


Filter set does not apply to suggestions

Description
When filtering by tags the costs and savings estimates are filtered, but the suggestions are not.
Steps to Reproduce
See screen shots
Expected Behaviour
When filtering by tags, the list of suggestions should also be filtered.
Relevant Logs / Console output

NOTE: results are filtered based on tags, but suggestions still include all 55.

Currently the functionality was intentionally not built for suggestions as we were uncertain on what direction to take it.
Will aim to address and add the functionality in our UI enhancement work.


squid:S1213 - The members of an interface declaration or class should…

This pull request is focused on resolving occurrences of Sonar rule squid:S1213 - The members of an interface declaration or class should appear in a pre-defined order
You can find more information about the issue here: https://dev.eclipse.org/sonar/coding_rules#q=squid:S1213
Please let me know if you have any questions.
M-Ezzat

LGTM!

LGTM

@marcosnils
Since it just rearranges fields/methods order, I'm not sure we want to backport this to all branches.

@HeartSaVioR I like this one. Let's apply from 2.8 onwards, WDYT?

Good. Please feel free to merge & backport. :)

Merged into master, 2.8, 2.9 respectively.


MetroLog stops logging when a few exceptions occurred

I have a universal app that runs on a Raspberry Pi with Windows IoT which uses MetroLog as loggings framework.
For some completely unknown reasons to me, sometimes MetroLog decides to stop logging to file completely. My app still runs, and it responds and does what it should do, except that there are no new log entries (and I know for certain that code is executes with log statements).
The usage of MetroLog in my app is as follows:
Creation/initialization in my App.xaml.cs
    private static readonly ILogger Log;

    static App()
    {
        var streamingFileTarget = new StreamingFileTarget(new SingleLineLayout());
        streamingFileTarget.FileNamingParameters.CreationMode = FileCreationMode.AppendIfExisting;
        streamingFileTarget.KeepLogFilesOpenForWrite = false;
        streamingFileTarget.RetainDays = 90;

        LogManagerFactory.DefaultConfiguration.AddTarget(LogLevel.Trace, LogLevel.Fatal, streamingFileTarget);

        Log = LogManagerFactory.DefaultLogManager.GetLogger<App>();
    }

For the rest, I have a lot of classes that have a static Logger instance, like this:
    private static ILogger Log = LogManagerFactory.DefaultLogManager.GetLogger(typeof(FlowSensor));

or:
    private static readonly ILogger Log = LogManagerFactory.DefaultLogManager.GetLogger<CloudClient>();

and usage of these instances is straight forward like this:
            try
            {
               // do stuff
            }
            catch (Exception e)
            {
                Log.Error($"'{nameof(GetEntityAsync)}' ran into an exception (retry counter = {i}): {e}");

                if (i == MaxRetries)
                {
                    throw;
                }

                await Task.Delay(30);
            }

The example above causes the logger to stop completely, it retries a couple of times, and I can see the log messages in the logging, and then nothing else gets logged.
Any idea how to troubleshoot this? I really have no clue whatsoever what is the cause of this weird behaviour.
Again. the App continues to run, and 100% sure code gets executes that contains log statements, so it seems that the logger died / stops flushing to disk?

@basprins its hard to tell what might be the problem, but can you try these options. Sometime the file is locked and not released and other process cannot access it .
streamingFileTarget.IncludeLogger = true; this should create a different file for each logger and you will find which logger is not working...
Also, what i notice is if you have any Dependency injection or different projects where you are using the logger, then make sure you initialize the logger in each project.
Also worth trouble shooting if the configuration code is executed when the logger tries to log..?
Try those let us know if you go any further

@MMalikKhan that's very helpful commenting! Thanks a lot for that.
I left our machine in production for a day, and I can at least confirm that loggings is continued when a day is passed (and thus a new log file is created). So your suggestion of a file lock issue makes sense.
I will start with creating a file per logger, and when I learn more I will share it here.
Thanks again !

@MMalikKhan
I added the "includelogger" in order to see which logs do continue to run. I made a copy of the directory listing, which indeed shows that a few loggers have stopped logging:
36bb0\LocalState\MetroLogs> ls | sort LastWriteTime -Descending
Directory: \\minwinpc\c$\Data\Users\DefaultAccount\AppData\Local\Packages\Durocan_ns7r26fq36bb0\LocalState\MetroLog
s

Mode                LastWriteTime         Length Name

-a----        10-3-2017     10:55          64130 Log - PulseDetector - 20170310.log
-a----        10-3-2017     07:58            640 Log - Heater3(pin=21 - 20170310.log
-a----        10-3-2017     07:58            640 Log - Heater2(pin=20 - 20170310.log
-a----        10-3-2017     07:58            640 Log - Heater1(pin=16 - 20170310.log
-a----        10-3-2017     00:44           1065 Log - Heater3(pin=21 - 20170309.log
-a----        10-3-2017     00:44           1065 Log - Heater2(pin=20 - 20170309.log
-a----        10-3-2017     00:44           1065 Log - Heater1(pin=16 - 20170309.log
-a----        10-3-2017     00:44         144423 Log - PulseDetector - 20170309.log
-a----         8-3-2017     22:03         135316 Log - PulseDetector - 20170308.log
-a----         8-3-2017     21:29            111 Log - StateMachine - 20170308.log
-a----         8-3-2017     21:29        1268990 Log - ExecutingScheme - 20170308.log
-a----         8-3-2017     21:29           9661 Log - 20170308.log
-a----         8-3-2017     21:29           8970 Log - CloudClient - 20170308.log
-a----         8-3-2017     17:26           1366 Log - Heater3(pin=21 - 20170308.log
-a----         8-3-2017     17:26           1366 Log - Heater2(pin=20 - 20170308.log
-a----         8-3-2017     17:26           1366 Log - Heater1(pin=16 - 20170308.log
-a----         8-3-2017     00:59          17761 Log - PulseDetector - 20170307.log
-a----         8-3-2017     00:59         175469 Log - ExecutingScheme - 20170307.log
-a----         8-3-2017     00:17            104 Log - Heater3(pin=21 - 20170307.log
-a----         8-3-2017     00:17            104 Log - Heater2(pin=20 - 20170307.log
-a----         8-3-2017     00:17            104 Log - Heater1(pin=16 - 20170307.log
-a----         7-3-2017     22:05            276 Log - 20170307.log
-a----         7-3-2017     22:05           1206 Log - StateMachine - 20170307.log
-a----         7-3-2017     22:05            644 Log - I2CChannel - 20170307.log
-a----         7-3-2017     22:05            168 Log - Arduino - 20170307.log
-a----         7-3-2017     22:05           2608 Log - LoadConfiguration - 20170307.log
-a----         7-3-2017     22:05            194 Log - App - 20170307.log
As you can see these three stop logging around 21:29
-a----         8-3-2017     21:29            111 Log - StateMachine - 20170308.log
-a----         8-3-2017     21:29        1268990 Log - ExecutingScheme - 20170308.log
-a----         8-3-2017     21:29           9661 Log - 20170308.log
-a----         8-3-2017     21:29           8970 Log - CloudClient - 20170308.log
An (user handled) exception occurred at that time (the server connection failed, which recovered later). No sure why that would cause any trouble. The catch is nothing special either, just log and handle the exception....
You mentioned two other points:
Also, what i notice is if you have any Dependency injection or different projects where you are using the logger, then make sure you initialize the logger in each project.
Also worth trouble shooting if the configuration code is executed when the logger tries to log..?
Can you elaborate on both? How do I initialize the logger in another project (other than LogManagerFactory.GetLogger)? And how exactly do I see if configuration code is executed when the logger tries to log?
Hope you can clarify a few parts for me, or maybe you have other ideas that might help me further.
Many thanks again.


Compute metrics individually per each class

Originally posted by @naga-karthik in https://github.com/ivadomed/MetricsReloaded/pull/17#issuecomment-2529934860

might be unrelated but because we're on the topic, i am documenting it
I think we should compute the metrics individually per each class. That is, say, if we have region-based seg/lesion label, then the current script automatically computes metrics for both of them sequentially in a for loop. But, for the bavaria-quebec project, I have had issues where even if the prediciton has only 1 class, I see two classes in the output csv file. When metrics are aggregated across all subjects then this results in incorrect scores. All I'm tyring to say here is that the for loop and iterating across unique labels is not robust. In the end, I had to separate SC and lesion labels and then compute the metrics for SC and lesions independently (to be sure)
how about we:

create temporary masks for each available class in the predictino mask
run the metrics on these temporary (single-class) masks
delete the temporary masks?

I have had issues where even if the prediciton has only 1 class, I see two classes in the output csv file.

This is interesting and definitely not intended. If the prediction has only 1 class, the output CSV file should also contain only a single class. Would you have a sample subject for debugging?


gruntfile.js found, Gruntfile.js not found

If I name the file gruntfile.js, everything's happy.
But naming it Gruntfile.js (capital G), results in the following in the grunt-runner log:
Error loading gruntfile: Gruntfile not found.
Atom 1.0.0
grunt-runner 0.10.0
Arch Linux
I can rename the file or symlink to it as a workaround, but I believe the capital G is canonical.

Hmm, this is interesting. Because for me Gruntfile.js works so would assume it's something with your OS.  Can you test the same file on another linux box and/or a win/osx box?

I'll try to get it set up on a Mac to test, but I think it will work fine there.
Changing "/gruntfile" to "/Gruntfile" grunt-runner-view.coffee causes it to work on my Linux box. I very much suspect that this is an issue related to the fact that filenames tend to be case-sensitive on Unix boxes, and case-insensitive on OSX/Windows.
If that's the case, maybe it makes more sense to refer to /Gruntfile? I'm out of my element on Mac/Windows, so I'll defer to someone else if that's a good or safe idea.

I have a speculative fix for this.
Can you test it out for me?
Replace lines 80 through 90 in ```grunt-runner-view.coffee``
        else
            Task.once require.resolve('./parse-config-task'), @path+'/gruntfile', ({error, tasks})->

                # log error or add panel to workspace
                if error
                    view.addLine "Error loading gruntfile: #{error}", "error"
                    view.toggleLog()
                else
                    view.addLine "Grunt file parsed, found #{tasks.length} tasks"
                    view.tasks = tasks
                    view.togglePanel()

with this
        else
            Task.once require.resolve('./parse-config-task'), @path+'/Gruntfile', ({error, tasks})->

                if error
                    # failed to load Gruntfile.js, try gruntfile.js
                    Task.once require.resolve('./parse-config-task'), @path+'/gruntfile', ({error, tasks})->

                        # log error or add panel to workspace
                        if error
                            view.addLine "Error loading gruntfile: #{error}", "error"
                            view.toggleLog()
                        else
                            view.addLine "Grunt file parsed, found #{tasks.length} tasks"
                            view.tasks = tasks
                            view.togglePanel()
                else
                    view.addLine "Grunt file parsed, found #{tasks.length} tasks"
                    view.tasks = tasks
                    view.togglePanel()

If it fails for /Gruntfile it just tries with /gruntfile :)

Tested, works for me! :)

Great! I'll add that then :)


feat: update adminconsole version

Automated changes by the update-addons GitHub action

[sc-115269]


Error in training Detection model using pytorch

Bug description
I am trying to train the detection model using pytorch on my own dataset and I am getting this error.
I am using macOS.
Can't pickle local object 'DetectionDataset.__init__.<locals>.<lambda>'
Code snippet to reproduce the bug
python references/detection/train_pytorch.py /Users/dishantpadalia/IITB/doctr_detection/train /Users/dishantpadalia/IITB/doctr_detection/val db_resnet50 --epochs 5

Error traceback
Traceback (most recent call last):
  File "/Users/dishantpadalia/IITB/doctr/references/detection/train_pytorch.py", line 414, in <module>
    main(args)
  File "/Users/dishantpadalia/IITB/doctr/references/detection/train_pytorch.py", line 345, in main
    fit_one_epoch(model, train_loader, batch_transforms, optimizer, scheduler, mb, amp=args.amp)
  File "/Users/dishantpadalia/IITB/doctr/references/detection/train_pytorch.py", line 110, in fit_one_epoch
    for images, targets in progress_bar(train_loader, parent=mb):
  File "/Users/dishantpadalia/opt/miniconda3/envs/iitb_2/lib/python3.10/site-packages/fastprogress/fastprogress.py", line 50, in __iter__
    raise e
  File "/Users/dishantpadalia/opt/miniconda3/envs/iitb_2/lib/python3.10/site-packages/fastprogress/fastprogress.py", line 41, in __iter__
    for i,o in enumerate(self.gen):
  File "/Users/dishantpadalia/opt/miniconda3/envs/iitb_2/lib/python3.10/site-packages/torch/utils/data/dataloader.py", line 444, in __iter__
    return self._get_iterator()
  File "/Users/dishantpadalia/opt/miniconda3/envs/iitb_2/lib/python3.10/site-packages/torch/utils/data/dataloader.py", line 390, in _get_iterator
    return _MultiProcessingDataLoaderIter(self)
  File "/Users/dishantpadalia/opt/miniconda3/envs/iitb_2/lib/python3.10/site-packages/torch/utils/data/dataloader.py", line 1075, in __init__
    w.start()
  File "/Users/dishantpadalia/opt/miniconda3/envs/iitb_2/lib/python3.10/multiprocessing/process.py", line 121, in start
    self._popen = self._Popen(self)
  File "/Users/dishantpadalia/opt/miniconda3/envs/iitb_2/lib/python3.10/multiprocessing/context.py", line 224, in _Popen
    return _default_context.get_context().Process._Popen(process_obj)
  File "/Users/dishantpadalia/opt/miniconda3/envs/iitb_2/lib/python3.10/multiprocessing/context.py", line 284, in _Popen
    return Popen(process_obj)
  File "/Users/dishantpadalia/opt/miniconda3/envs/iitb_2/lib/python3.10/multiprocessing/popen_spawn_posix.py", line 32, in __init__
    super().__init__(process_obj)
  File "/Users/dishantpadalia/opt/miniconda3/envs/iitb_2/lib/python3.10/multiprocessing/popen_fork.py", line 19, in __init__
    self._launch(process_obj)
  File "/Users/dishantpadalia/opt/miniconda3/envs/iitb_2/lib/python3.10/multiprocessing/popen_spawn_posix.py", line 47, in _launch
    reduction.dump(process_obj, fp)
  File "/Users/dishantpadalia/opt/miniconda3/envs/iitb_2/lib/python3.10/multiprocessing/reduction.py", line 60, in dump
    ForkingPickler(file, protocol).dump(obj)
AttributeError: Can't pickle local object 'DetectionDataset.__init__.<locals>.<lambda>'

Environment
DocTR version: 0.6.0a0
TensorFlow version: N/A
PyTorch version: 1.13.0.dev20220822 (torchvision 0.14.0.dev20220822)
OpenCV version: 4.6.0
OS: Mac OSX 12.5.1
Python version: 3.10.4
Is CUDA available (TensorFlow): N/A
Is CUDA available (PyTorch): No
CUDA runtime version: Could not collect
GPU models and configuration: Could not collect
Nvidia driver version: Could not collect
cuDNN version: Could not collect

Deep Learning backend
>>> from doctr.file_utils import is_tf_available, is_torch_available
>>> print(f"is_tf_available: {is_tf_available()}")
is_tf_available: False
>>> print(f"is_torch_available: {is_torch_available()}")
is_torch_available: True

Hi @dishant26 looks like a problem with fastprogress... Would it be possible to test with python 3.8 (Unfortunately i cannot much help on MacOS)

Hey. I tested with python 3.8. Still the same error.

Hey. I solved the issue. It is some problem with lambda functions.
If anyone encounters the same issue.  Just set num_workers=0  in train_loader.

Hi @dishant26 i think i got it python <3.8 should work seems to be an known issue with multiprocessing and macosx another solution (you posted) or modifying the script to use tqdm instead of fastprogress

Thanks for sharing your solution 👍


Task specific trainable parameters

Hi, I have a question regarding the layer weights and the gamma parameter that the paper states as fundamental to achieve good results in downstream tasks.
If I understand correctly there are 2 macro steps to use ELMo correctly.

Task agnostic: We train the biLM with lots of unlabelled data in a language modelling manner.
Task specific:
a) Finetune: Once we have our biLM model fully trained, we can finetune our model to the specific task at hand (say text classification for medical domain) an achieve much better results with a single pass of the data.
b) Actually use ELMo, and my actual question: In order to finally use ELMo embeddings we need to train the weights to average the token representation and the layers, along with a gamma parameter.

The elmo.py file seems to output the embeddings already weighted, but how can this be since the scripts have not necessarily seen the task specific data yet? The same goes for the gamma value.
Thanks

In issue #48 this is partially answered, but I still don't understand how the weights are learned even though the model has never seen the task specific data...

The function weight_layers uses tf.get_variable to introduce trainable parameters:
https://github.com/allenai/bilm-tf/blob/master/bilm/elmo.py#L68
Then when you use the returned weighted_op in your model, those parameters are optimized along with the rest of your model parameters.

Perfect. Thanks a lot!

Thank you very much.


Blank Base Map

Blank base map is often required. it would be good and handy if there could be some option to easily add a blank base map. It is more of new feature rather than issue but i did not know where to post new idea or requested so i posted it in issues . sorry and thanks for open layer 3. I like it. Hope someday i can contribute to it.

We have multiple applications with a blank background image. And we just use an ol.layer.Tile layer with no source. See https://github.com/Geoportail-Luxembourg/geoportailv3/blob/46e8190da8d2889b9338c7d614792072fbf83d0c/geoportailv3/static/js/backgroundlayer/blanklayerservice.js for example. I am going to close this issue as there's no plan to introduce more specific support for that in the library.


removes torch dependencies in utils.voxel

Description
This PR removes dependencies on torch_geometric, torch_cluster, and torch_scatter in stretch.utils.voxel.
Checklist

[ ] I have performed a self-review of my code
[ ] If it is a core feature, I have added thorough tests
[ ] I have added documentation for the changes
[ ] I have updated the README file if necessary
[ ] I have run on hardware if necessary

Screenshots (if applicable)
Add any relevant screenshots or screen recordings to help reviewers understand the changes.
Additional context
Add any other context or information about the pull request here.

need to add license declarations for 3rd parties to these files


Theorem Suggestion: Hyperconnected + paracompact => compact

Theorem Suggestion
If a space is:

P39
P30

then it is P16.
Rationale
This theorem would demonstrate that no spaces satisfy the following search:
https://topology.pi-base.org/spaces?q=hyperconnected+%2B+paracompact+%2B+not+compact
This also would show that S145 is not paracompact
Proof/References
If $\mathcal{U}$ is locally finite open cover of $X$ and $x\in X$ then there exists open $V\subseteq X$ such that $V$ intersects only finitely many members of $\mathcal{U}$. But since $X$ is hyperconnected, $V$ must intersect all of members of $\mathcal{U}$. So any locally finite open cover of $X$ is finite.
Its now enough for show that if every open cover of $X$ has a finite refinement, then every open cover has a finite subcover. Indeed, suppose that $\{V_1, ..., V_n\}$ is a refinement of open cover $\mathcal{U}$. For each $k$ pick $V_k\subseteq U_k$. Then $\{U_1, ..., U_n\}$ is a finite subcover of $\mathcal{U}$.

Nice result.

Wouldn't a similar argument also prove: [Hyperconnected + countably paracompact => compact] ?


Error with binary response

Description
I think this issue may have already been posted here, but the error I get is not exactly described in other issues. I have an endpoint that sends a binary blob (type string format binary), and when I use the endpoint I get this result:

Basically, seems like the client is trying to parse the binary string as JSON. The type of reponse is Blob | Binary
I am generating the client with fetch client.
Reproducible example or configuration
https://stackblitz.com/edit/hey-api-client-fetch-example-f8ipw7?file=src%2Fclient%2Ftypes.gen.ts
OpenAPI specification (optional)
openapi: 3.0.3
info:
  title: Myproj

paths:
  /report:
    post:
      responses:
        "200":
          description: Successful Response
          content:
            image/png:
              schema:
                type: string
                format: binary

Hey @tadejsv !
I reported a similar issue earlier today!
If you want a quick fix, try setting parseAs to "auto" in createClient!
https://github.com/hey-api/openapi-ts/issues/840

Hey @tadejsv, can you try to manually set parseAs either to auto or the expected value and see if that works? If yes, we've found our culprit https://github.com/hey-api/openapi-ts/issues/840

@tadejsv can you please check if the next release fixes this issue for you?

Will check as soon as possible:)

Thanks, will publish it in a few minutes

@mrlubos unfortunately I still get the promise rejected after applying the fix

Sorry, checked again, it works :)


Match common editor highlighters more closely.

These rules more closely match (as examples) themes like VSCode Dark+, Atom One Dark Pro, etc.

@dpeek Is this project active? Will these changes ever be merged in?

Apologies, I completely missed this notification for this PR. LGTM.


Upgrade to latest tc version

i.e. 1.1.0-snapshot

done in commit c86b02781de10f5f00d90cf1e17b47a73c262ce5


Different programs support

This code contains logic that displays the component IdEntry based on the program using the auth layer.

The URL will now contain program and authType.
Based on this program name, getProgramData API is called to retrieve all details regarding the program.
Basic validation login is based on the validation type (basicValidationMapping)
For advanced validation, the database and the column name is also a variable now, based on the program.

A few points --

I saw that there was no issue created and linked to this PR. We've written all the main, important steps in our development lifecycle doc here. Please make an issue and link here! It helps a lot in the long run

Also, for this feature, PRs had to be made in two repos, auth-layer and auth-layer-backend. The best practice in these cases is to keep the names of the PRs identical. Right now, this PR is called Different programs support and the one in the auth-layer-backend repo is called Program support. Ideally both names should be the same. Again, this helps a lot in the long term. For us (when we forget about the code) and for others who will work on the project.

Also, when two PRs are interlinked and depend on each other, one needs to mention that in the description.

These last two points were not written in the doc so I've added them. I'll keep adding things from my side regarding the process. Please make sure you go through the doc again


only want a shortcut to work on desktop

Description of the new feature / enhancement
what is the method for making a shortcut work only on the desktop? I'm used to macs, and command-D on a mac is duplicate, on a pc it's delete - which as you can imagine has caused no end of problems. However, in InDesign, command D with an image box selected means "go get an image to fill this box" and I use that a lot - so I need to change command-d on the desktop to something non-destructive like command-c.
thanks!
(Sorry if this is the wrong place to post this, I couldn't find a more suitable area..)
Scenario when this would be used?
any time I want to duplicate a file in a folder or on the desktop itself but don't want to (via muscle-memory) delete it by mistake (especially if the file is in a folder on our server where it would be deleted without the ability to visit the bin and retrieve it)
Supporting information
No response

I think this could be possible with Keyboard Manager, and targeting a single app. But the desktop is not an app, so I'm not sure what the target should be...

i would do this.  I verified this works too.
Alt-D mapped to Delete and limit to explorer.exe

/helped


Change Language Mode doesn't search by language ID

VSCode Version: Code - Insiders 1.19.0-insider (bc206bbacc13e48d4422b8c73334aa9dd6f782df, 2017-12-08T05:13:43.085Z)
OS Version: Windows_NT x64 10.0.16299
Extensions: none

Steps to Reproduce:

Invoke Change Language Mode action
Type csharp

Expected result: the picker should finds C#
Actual result: no languages are found, but when I type c#, it's there: C# (csharp)

Thanks!


fix: update package lock

#214  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


The config for glm-10b

I need to use the glm-10b with scripts/generate.sh and set MODEL_TYPE='glm-10b' in the config file in configs.
However, there are still errors that [Errno 2] No such file or directory: 'XXX/glm-10b-en/126000/mp_rank_01_model_states.pt' and [Errno 2] No such file or directory: 'XXX/glm-10b-en/126000/mp_rank_02_model_states.pt' maybe because glm_130b is used.
How can be the config file modified in configs to use glm-10b instead of glm-130b?
Looking forward to reply. Thanks.

According to the log, maybe tokenizer-type need to be modified. What is the glm-10b-en tokenizer type?

@xxupiano Hi,
Which repository of code are you using? The code in this repo (GLM-130B) is for the 130B version only. If you want to use monolingual GLM-10B, please use the framework in GLM's original repo.

Thanks for your reply. I'll try that repo.


Create array manipulation.py

Related Issue
Solved issue number: #473  that will be closed through this PR
Describe the changes you've made
Adding solution of array manipulation in python
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
A clear and concise description of it.
Language used for coding
Python
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have documented the code.
[x] New and existing unit tests pass locally with my changes.

This pull is for issue no #473

@Ayush7614 Please add some comments to your code so that beginners will find it easy to understand :)

@Nivedita967 plss now review it


Parse error on multiplying two switch expressions

switch 1 {} * switch 2 {}
            ^ Unexpected '*' (line 1, position 13)

It works when I parenthesize the first expression:
(switch 1 {}) * switch 2 {}

It is parsing it as a statement.  This works...
let x = switch 1 {} * switch 2 {};

That's what I thought. Why is there a non-expression switch statement anyway? 1; 2; 3; is a valid Rhai program, so clearly expressions can be statements too.

That's what I thought. Why is there a non-expression switch statement anyway? 1; 2; 3; is a valid Rhai program, so clearly expressions can be statements too.

That's because most normal uses would use it as a statement (if also).  Parsing it as an expression will incur an additional indirection every time it is encountered.

It parses as a statement even inside an if _expression:
let x = if true { switch 1 {} * switch 2 {} }
                              ^ Unexpected '*'

Yes, that's because things inside a block are statements...
Why would you like to multiply two switch statements without assigning the result to anything?

Why would you like to multiply two switch statements without assigning the result to anything?

I'm not saying you shouldn't or can't do it... but usually there is no great need.

I'm using https://rhai.rs/book/engine/expressions.html

So you want to evaluate expressions which may consist of switch?  That would be a bit involved, as the action blocks in a switch may be statements, so you'll end up running statements.
Unless it is also restricted to expressions there.

I'm running some Rhai code to compute some value from some inputs. It's intended to be a pure function. I'm exposing no functions to the engine that have side effects, and the scope is truncated afterwards back to its initial size. Whether Rhai thinks my code is an "expression" or a "statement" or a "block of statements" or a "box of chocolates" does not matter to me :)
It's strange to me that if and switch expressions would be disabled while evaluating expressions. We can emulate the same behaviour by exposing a Rust function to the engine.
A ternary condition ? value_if_true : value_if_false, would that be considered a "real expression without logic flow"? How about (condition as f64) * (value_if_true) + (1.0 - (condition as f64)) * (value_if_false)?

It's strange to me that if and switch expressions would be disabled while evaluating expressions. We can emulate the same behaviour by exposing a Rust function to the engine.

That's only because, when evaluating expressions, it is more likely to be an error when statements are encountered.

A ternary condition ? value_if_true : value_if_false, would that be considered a "real expression without logic flow"? How about (condition as f64) * (value_if_true) + (1.0 - (condition as f64)) * (value_if_false)?

In Rhai, a form of the tenary operator can be easily implemented as a custom syntax expressions... so yes it is an expression...  I know that's not the most ideal...
What I can do is to add an advanced version of compile_expression such that it takes a parameter allowing statements.  It is a very small change in the code (changing from false to true), but right now there is no out-of-the-box method to parse an expression with if/switch-expressions.

That's okay. Since a statement block also evaluates to the last statement, I can just use that. It also helps with more complex expressions, e.g. if the user wants to introduce helper variables or eve functions. On second thought, I don't really understand the use case of eval_expression, but that might just be me :)

I don't really understand the use case of eval_expression, but that might just be me :)

Different use cases.  Many people don't need a full-blown scripting language.  All they need is to evaluate expressions for customized config.
For these cases, eval_expression is ideal for them as they don't have to worry about users throwing in loops and variable declarations etc.

I can see how loops would open you up to DoS attacks if you are running this as e.g. a web service. But if and switch expressions can't be used for that. Nor can variable declarations.

That's true, except that they both have blocks which can run anything inside...

Maybe I can restrict the blocks to have only expressions...  then it is OK to have if and switch in expressions...

OK.  I don't think it would be difficult to add.  I'll add that support, then you can use Engine::eval_expression and both if and switch will work as long as their blocks contain only expressions.

PR https://github.com/rhaiscript/rhai/pull/630 does this - it allows if and switch expressions.


Consider using scribe backend

Hey @ChristopherDavenport, this is a really nice project! Looks great!
I was wondering whether you'd consider a PR to use outr/scribe rather than log4s or making the logging backend configurable? scribe has support for scala-native / scala.js and great performance, and it can still be used as an slf4j backend.

I would definitely consider it. After thinking about constraints I'm wondering whether we could abstract the fundamental algebra or something similar and then have backends which implement in different modules so it operates to each teams individual taste.
How do you feel about the structure of the algebra, as it seems fairly consistent across the logging libraries?

Yeah, it looks good to me - certainly covers everything that I'd use personally :+1:


Bucket2.0

This is a duplicate of PR 51. We cannot find the bug in that code and found it easier to start from scratch.
[X] Add in mask to conservation calculator since the land model does not use a mask in the RHS
[X] Define push/pull methods and implement
[X] Use reinit and start at t = dt
[X] Add back in artifacts for data access
[X] Remove q_sfc calculation and rho_sfc calculation from calc fluxes, move to atmos_pull!

This PR adds in a test case showing a coupled atmos, land, ocean, and sea ice model. The bucket model in place of the slab land model, which is how it differs from https://github.com/CliMA/ClimaCoupler.jl/pull/50.
We've also set it up so that the specific humidity at the surface uses the correct land value, and set the RRTMGP fields for surface temp and albedo correctly based on the surface values.
A future PR will update the version of ClimaAtmos we are using so that we can read the precipitation from the atmos cache, and pass it to the land model, and so that we are using topography in the atmos model.
A future PR will also implement a coupler interface as this interface is currently hard coded to some extent to the models being used here.
co-authors:
Ben Mackay @jb-mackay
Lenka Novak @LenkaNovak
Zhao-yi Shen @szy21

As of "conserves energy" push, commit # 57c8b49

As of added bucket capability minus albedo
Bucket model conserves energy

As of added q_sfc and artifacts

30 day run:

Videos:
\rho q t/\rho at first level:
https://user-images.githubusercontent.com/65979205/173682567-38f1c211-2996-4d00-897e-6b9f437cce0d.mp4
\rho e at first level
https://user-images.githubusercontent.com/65979205/173682574-75e3e26b-ff17-48b1-9b8b-7668898ddae1.mp4
wind u at first level
https://user-images.githubusercontent.com/65979205/173682581-04868dff-b1b7-4e07-aa94-e67cf9c70cfb.mp4
Earth T
https://user-images.githubusercontent.com/65979205/173682587-b4a85988-79a4-49b3-976d-4595f4eaa2af.mp4
Bucket Moisture - 0 plotted for ocean and ice.
https://user-images.githubusercontent.com/65979205/173682593-e9f219f6-4bd5-45bd-8c56-998f9ac0123a.mp4

Looks good! In the next PR with the updated ClimaAtmos interface (with precipitation + topography), we should
- add conductive sea-ice flux
- improve the docs
- add a better interface for the mask (similar to what was in Bucket1.0)
- double check the end - half issue in the conservation check that Zhaoyi mentioned (btw, end + half gives an error)
- ensure we remove (only) the unnecessary tmp files

Make sure prescribed SST still runs, and we should be able to generate videos for it

As of cleaned up push pull a bit
still conserving energy for slab and bucket versions of land, eg:

Looks good! In the next PR with the updated ClimaAtmos interface (with precipitation + topography), we should - add conductive sea-ice flux - improve the docs - add a better interface for the mask (similar to what was in Bucket1.0) - double check the end - half issue in the conservation check that Zhaoyi mentioned (btw, end + half gives an error) - ensure we remove (only) the unnecessary tmp files

I think the nz -half is fine - the calculation gives nz = 11 (for 10 layers), so 11-half = 10.5 = what we want for the upper face?

I think the nz -half is fine - the calculation gives nz = 11 (for 10 layers), so 11-half = 10.5 = what we want for the upper face?

Oh I didn’t know nz is 11. I think that’s fine then.

I think the nz -half is fine - the calculation gives nz = 11 (for 10 layers), so 11-half = 10.5 = what we want for the upper face?

Oh I didn’t know nz is 11. I think that’s fine then.

we can rename it!

I recommend reseting to f85f27fa868789f096558760083059bf69f67c6c and merging, the remaining changes could be applied in a separate PR. Upgrading to the latest versions of packages requires changing how we handle clima parameters, which is not trivial. These packages in include

RRTMGP v0.4.0
TurbulecenceConvection v0.33
Thermodynamics 0.9
CloudMicrophysics v0.6

and more. The list is actually pretty long, and the required changes are complicated, as you can see in the PR where we upgraded to latest packages in ClimaAtmos

Results after hooking up with latest clima atmos, precip fixed, coupler setting atmos BC:
qt lowest level:
https://user-images.githubusercontent.com/65979205/175792073-9c1c3d85-5fda-412a-ab77-85c45fe92da4.mp4
u lowest level:
https://user-images.githubusercontent.com/65979205/175792077-16f40323-da1a-4ee2-b633-23e0ddc5e0bc.mp4
earth T:
https://user-images.githubusercontent.com/65979205/175792084-b2879081-c95d-4c2b-bad4-ec917a0ca07b.mp4
bucket W:
https://user-images.githubusercontent.com/65979205/175792087-1e858d33-bf4d-43a3-93b5-2623da36f1e6.mp4
Energy error, fractional, of entire system:

@szy21 @charleskawczynski @LenkaNovak @jb-mackay
IMO we can merge this now. We're using the hot fix commit from ClimaAtmos, so the coupler is now setting the BC for clima atmos, and precipitation is nonzero by the end of the simulation, and land bucket water content doesnt seem unreasonable. To check the accuracy of this further, we probably need to have diagnostics set up. Land needs to make a callback function, we can implement/use the callback clima atmos already makes in the driver file.
Perhaps one small fix before merging is to figure out where the plots in buildkite get put/how to access those.

Agreed, thanks, Kat! After the issues on reverting and buildkite are sorted, let's merge and sort out diagnostics as part of the next interface-focused iteration. WRT buildkite plots, are you thinking of setting up something like what we had a while back for the GCM results where buildkite would send a selection of plots on Slack?

EDIT: I just saw @charleskawczynski made a comment about reverting to a previous commit, since different packages are in different stages of working with clima params. Charlie, the code runs right now, though....using the changes you made to climaLSM to use the new parameters, and I believe using a commit of clima atmos that is using the new params as well e88fbea293bf6fb48a927f0459556d1de4dca0e9. so why revert?

Because we haven't gone through and actually applied the clima parameter changes. If you want more information I can look through and point out all of the things that are inconsistent. Basically, anything that is using CLIMAParameters is inconsistent with how their being used downstream.

bors r+


anisotropy doesn't render right on custom meshes

Godot version: 3.1.1

OS/device including version: Ubuntu 18.10

Issue description:
Made a sphere in blender, imported and added a spatial material and set anisotropy to 1. Doesn't render the way it does for a default godot sphere (shown on right)

I've attached the mesh used:
sphere.zip

Does your mesh have UV maps? Anisotropy probably requires them.

Does your mesh have UV maps? Anisotropy probably requires them.

Does your mesh have UV maps? Anisotropy probably requires them.


"Cheat Sheet" hypelink is not valid

The "Cheat Sheet" reference does not have a valid URL.
[Enter feedback here]

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

ID: 55c7ea05-69b5-1c5e-eb55-07066a1b2d17
Version Independent ID: b75e9d43-6590-60bd-ca34-83c40abd9ae6
Content: Set up new environments, Azure DevOps, and branches for projects - Commerce | Dynamics 365
Content Source: articles/commerce/dev-itpro/new-environments-visual-studio-teams-branch-retail-projects.md
Service: dynamics-365-commerce
Product: ****
Technology: ****
GitHub Login: @andreashofmann1
Microsoft Alias: andreash

Created bug 594529 to track this issue.

I removed the link, looks like that website is gone.


[MLS Spec change]  Exempt default features from RequiredCapabilities (#560)

Description of the changes
Proposals and Extensions from the spec are required by default (i.e. MTI).
Link to the exact changes
https://github.com/mlswg/mls-protocol/pull/560

This depends on moving the required capabilities to the leaf node in #819.

I think this is implemented, but should be double checked before closing.

See, e.g., https://github.com/openmls/openmls/blob/6bb082b01195b29417483cae5abb34c0d7f58303/openmls/src/treesync/node/leaf_node.rs#L795


[dvsim] Enhance how our test build / run / etc progression looks like

ATM our build / run progression is logged to stdout every 10seconds like this:
INFO: [Deploy] [00:23:40]: [build]: [Q: 00, D: 08, P: 01, F: 17, K: 00, T: 26]
INFO: [Deploy] [00:23:50]: [build]: [Q: 00, D: 08, P: 01, F: 17, K: 00, T: 26]
INFO: [Deploy] [00:24:00]: [build]: [Q: 00, D: 08, P: 01, F: 17, K: 00, T: 26]
INFO: [Deploy] [00:24:10]: [build]: [Q: 00, D: 08, P: 01, F: 17, K: 00, T: 26]
INFO: [Deploy] [00:24:20]: [build]: [Q: 00, D: 08, P: 01, F: 17, K: 00, T: 26]
INFO: [Deploy] [00:24:30]: [build]: [Q: 00, D: 08, P: 01, F: 17, K: 00, T: 26]
INFO: [Deploy] [00:24:40]: [build]: [Q: 00, D: 08, P: 01, F: 17, K: 00, T: 26]
INFO: [Deploy] [00:24:50]: [build]: [Q: 00, D: 08, P: 01, F: 17, K: 00, T: 26]

For long running tests, this ends up cluttering the console. Third party tools already exist that provide a far better ways to visualize the progression. Example:
https://python-enlighten.readthedocs.io/en/stable/index.html
Consider switching to something like that.

This has been implemented.


polling database for correlationId - SLIM-528

 You should sign our Contributor License Agreement in order to get your pull request merged.

This pull request makes the build fail.

This pull request makes the build fail.


Project create: add easier way to specify a funding target

Many projects will want to simply set a funding target.
How might we make that clearer in UI?
User feedback:
https://discord.com/channels/775859454780244028/775859454780244031/986623033173213204

Working on this. Will drop a prog update soon


unused-method-argument doesn't pick up scoped arguments.

The rule for unused-method-arguments is a good one, however, it doesn't pick up if you've used the argument by scoping it, e.g. it will say 'foo' is unused even though 'arguments.foo' is used in the function.

Can you provide a code sample.

@ryaneberly sure:
`

<cfset testFunction(arguments.foo)>

`
Creates an 'unused-method-argument' warning.

@ryaneberly Is this the same as https://github.com/cflint/CFLint/issues/152?

yes, and it is fixed. 0.7.2 I think.

It seems to still be an issue in 0.7.3, at least in linter-cflint for whatever reason, but I need to confirm. Stay tuned!

Seems to be resolved in linter-cflint 0.13.0


polkadot: 0.9.29 -> 0.9.30

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/paritytech/polkadot/releases.
meta.description for polkadot is: "Polkadot Node Implementation"
meta.homepage for polkadot is: "https://polkadot.network"
meta.changelog for polkadot is: ""
Updates performed

Rust version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.9.30 with grep in /nix/store/078xzk8rgzk89r5hw3nga2hdhq69la64-polkadot-0.9.30
found 0.9.30 in filename of file in /nix/store/078xzk8rgzk89r5hw3nga2hdhq69la64-polkadot-0.9.30

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
polkadot

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/078xzk8rgzk89r5hw3nga2hdhq69la64-polkadot-0.9.30 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A polkadot https://github.com/r-ryantm/nixpkgs/archive/8da5fd6e9659fb7da3577598e179fcf99de4db3c.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/078xzk8rgzk89r5hw3nga2hdhq69la64-polkadot-0.9.30
ls -la /nix/store/078xzk8rgzk89r5hw3nga2hdhq69la64-polkadot-0.9.30/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

polkadot

Maintainer pings
cc @akru @andresilva @asymmetric @FlorianFranzen @RaghavSood for testing.

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

1 package built:

polkadot


FileCollection dependencies added as inputs to compile.

While we currently add project dependencies (their artifacts) as inputs
to the compile task, we were missing file collection dependencies.
This change rectifies this, which means any changes to file collection
dependencies will now cause the project to be recompiled.
Fixes issue #210.

Hi @SlevinBE, this fix is critical for our team. If you could review it at your earliest convenience, I'd greatly appreciate it. Thank you!

Merged. Thanks for the PR!

BTW, when can I expect the updated snapshot jar on oss.sonatype.org?

it's already on there :)
snapshots are automatically build and uploaded when we merge to the develop branch via our Travis build.

Ah, I must have had to clear my caches, for after removing ~/.gradle/caches, I downloaded the correct version. Thanks!


Figure k: 'Graphical representation of the base model results presented'

There are two green and black lines in Figure j, and two green, black, and blue lines in Figure k. I will look at this

This issue was due to the changes made in issue #335. The additional three columns in the 'risk' data frame caused additional lines to be shown. Fixed it in commit b3573ce086fd74a4de762bb9ecc4e49d970a8f69.


Add mpd-mpris.desktop

As also discussed in https://github.com/natsukagami/mpd-mpris/pull/37 .

Thanks!


Gifs upload by Remote control => Colors/Effects not moving (are static)

[x] I confirm that this is an issue rather than a question.

How to upload gif to make it moving?

Bug report
Steps to reproduce

Login to your hyperion panel
Go to Remote control => Colors/Effects
Upload gif
Check your leds

What is expected?
Gif is moving, leds are changing their states
What is actually happening?
Gif is not moving i can see only first frame.
System

yperion Server:
- Build:             (HEAD detached at 2.0.14) (LordGrey-c9a1c88/b109703-1669566414)
- Build time:        Nov 27 2022 18:59:24
- Git Remote:        https://github.com/hyperion-project/hyperion.ng
- Version:           2.0.14
- UI Lang:           en (BrowserLang: en-US)
- UI Access:         default
- Avail Screen Cap.: dispmanx,framebuffer,qt
- Avail Video  Cap.: v4l2
- Avail Services:    boblight,cec,effectengine,forwarder,flatbuffer,protobuffer,mDNS,SSDP,borderdetection
- Config path:       /root/.hyperion
- Database:          read/write

Hyperion Server OS:
- Distribution:      Raspbian GNU/Linux 11 (bullseye)
- Architecture:      arm
- CPU Model:         ARMv6-compatible processor rev 7 (v6l)
- CPU Type:          Raspberry Pi Zero W Rev 1.1
- CPU Revision:      9000c1
- CPU Hardware:      BCM2835
- Kernel:            linux (5.15.32+ (WS: 32))
- Root/Admin:        true
- Qt Version:        5.15.2
- Python Version:    3.9.2
- Browser:           Mozilla/5.0 (Windows NT 10.0; Win64; x64)

Would you mind to share the Gif which is not working, please?

@Lord-Grey sure thing, i uploaded it to my gdrive
https://drive.google.com/drive/folders/1DKtQuzgK9PtVKhyWE7jaEcsSHvz-2FsM?usp=sharing

"Picture" on the remote control will only show one picture not executing an animation.
If you would like to have the animated version of the image, create an "GIF's" Effect via the Effects Configurator.
Then you can select your custom effect and you have the animated GIF running via the LEDs.


Investigate potential improvements to Key.equals()

It appears that improvements could be made to Key.equals(). From the original jira ticket regarding this idea:

In the Key.equals(Key, PartialKey) overload, the current method compares starting at the beginning of the key, and works its way toward the end. This functions correctly, of course, but one of the typical uses of this method is to compare adjacent rows to break them into larger chunks. For example, accumulo.core.iterators.Combiner repeatedly calls this method with subsequent pairs of keys.
I have a patch which reverses the comparison order. That is, if the method is called with ROW_COLFAM_COLQUAL_COLVIS, it will compare visibility, cq, cf, and finally row. This (marginally) improves the speed of comparisons in the relatively common case where only the last part is changing, with less complex code.

I'm not sure why the changes proposed in the jira ticket were never merged, but this idea resurfaced when Key.equals() was identified in #1099. At first, I considered including the improvements of Key.equals() as part of the PR for #1099, but I don't think that this sort of improvement will make much, if any, difference towards the goal of #1099 so creating a separate ticket/PR seemed like the best option.

I've run an updated version of the benchmark from the original jira ticket and the results I received seem significant
Benchmark                    Mode  Cnt   Score   Error  Units
MyBenchmark.customDom       thrpt   25  56.241 ± 0.458  ops/s
MyBenchmark.customVanilla   thrpt   25  53.248 ± 0.475  ops/s
MyBenchmark.customWill      thrpt   25  92.400 ± 0.935  ops/s
MyBenchmark.standardEquals  thrpt   25  51.610 ± 0.470  ops/s

The MyBenchmark.customWill is the relevant equals() method with its comparison reversed. This looks like this:
  public boolean willEquals(WillKey other, PartialKey part) {
    switch (part) {
      case ROW_COLFAM_COLQUAL_COLVIS_TIME_DEL:
        if (deleted != other.deleted)
          return false;
      case ROW_COLFAM_COLQUAL_COLVIS_TIME:
        if (timestamp != other.timestamp)
          return false;
      case ROW_COLFAM_COLQUAL_COLVIS:
        if (!isEqual(colVisibility, other.colVisibility))
          return false;
      case ROW_COLFAM_COLQUAL:
        if (!isEqual(colQualifier, other.colQualifier))
          return false;
      case ROW_COLFAM:
        if (!isEqual(colFamily, other.colFamily))
          return false;
      case ROW:
        return isEqual(row, other.row);
      default:
        throw new IllegalArgumentException("Unrecognized partial key specification " + part);
    }
  }

Compared to the current code:
https://github.com/apache/accumulo/blob/36dd78ce6bbc2aae8db72014b8703f94b9b92ded/core/src/main/java/org/apache/accumulo/core/data/Key.java#L977-L1002
I did add another equals method to the benchmark for testing in relation to #2811 represented by MyBenchmark.customDom in the benchmark printout. For those interested that method can be found here.

Maybe I am reading you code wrong, but I don't see where your code is functionally equivalent to the current code that you posted here.
The current code seems to be performing partial key comparisons - so for example, if the case is ROW only the row data is compared.  If it was ROW_COLFAM_COLQUAL the fields ROW, COL_FAM and COL_QUAL are all checked - it looks like you code only checks the last value as specified by the depth.
With your code, if the timestamps are equal, would it consider the rows equal?
What I thought you were going for would use boolean operation short-circuiting so that comparing "long" rows would be
deferred until later.
original:
...
case ROW_COLFAM_COLQUAL_COLVIS_TIME_DEL:
 return isEqual(row, other.row) && isEqual(colFamily, other.colFamily)
           && isEqual(colQualifier, other.colQualifier)
           && isEqual(colVisibility, other.colVisibility) && timestamp == other.timestamp
           && deleted == other.deleted;

possible change:
...
case ROW_COLFAM_COLQUAL_COLVIS_TIME_DEL:
 return deleted == other.deleted &&
           && timestamp == other.timestamp
           && isEqual(colVisibility, other.colVisibility)
           && isEqual(colQualifier, other.colQualifier)
           && isEqual(colFamily, other.colFamily)
           && isEqual(row, other.row)

However, even this may end up with additional comparisons and would depend on the distribution of values within the table.
When comparing full depth, checking isDeleted and timestamp upfront seem to be a good optimization.
When comparing a partial key, things are not so clear cut.

Visibilities may or may not change frequently.
If locality groups are used, then column family / column qualifiers will be grouped together and then less likely to change from row to row.
for changes between row(s), the last bytes are more likely to change the fastest. if the rows are AAAA and AAAB starting from the end (when lengths are equal) requires one comparison. Starting from the head it takes four comparisons (assuming checking one character at a time, but holds for longer values when greater than a long or an integer if grabbing multiple bytes at a time)

Sorry if I misunderstand what you are showing.

@EdColeman, I think you are understanding things correctly. The code I posted above was from the original ticket and I had overlooked the points you have made. I'll work on some changes to try to address these things.

Looking at the method in 'Key' the method that performs the equality check on a byte array private static boolean isEqual(byte[] a1, byte[] a2) is already optimized for Accumulo row comparisons, checking the last few bytes first and does not seem to need additional optimization.

Looking at the method in 'Key' the method that performs the equality check on a byte array private static boolean isEqual(byte[] a1, byte[] a2) is already optimized for Accumulo row comparisons, checking the last few bytes first and does not seem to need additional optimization.

So then maybe the only thing that might be worth looking into further is your suggestion about the full depth check:

When comparing full depth, checking isDeleted and timestamp upfront seem to be a good optimization.

I am confused with the in the context of #2811 are any changes needed?
If not, changing the full depth equals to deleted, timestamp, row, col fam, col qual and then col visibility might be a minor improvement. deleted and timestamp should be easy checks, keeping the rest of the order (row, col fam,...) minimizes behavior changes - in some cases having vis, col fam, col qual checked before large row values would help, but in other cases it would be worse - so without evidence of a general improvement without negative impacts it seems safest to leave that part of the current order in place.

I am confused with the in the context of #2811 are any changes needed?

No changes are needed in the sense that they don't fix any bugs or change any behavior, but they reduce the size of some of the methods which makes them no longer "hot".

I ended up making a github repository for benchmarking various versions of Key.equals(). The repo can be found at https://github.com/DomGarguilo/key-equals-benchmark
Some of the results I got from running the benchmark:
With random PartialKey

Benchmark                      Mode  Cnt   Score   Error  Units
MyBenchmark.customVanilla     thrpt   25  27.139 ± 0.286  ops/s
MyBenchmark.customVanilla2    thrpt   25  28.130 ± 0.182  ops/s
MyBenchmark.refactoredEquals  thrpt   25  27.282 ± 0.096  ops/s

With deepest PartialKey only (ROW_COLFAM_COLQUAL_COLVIS_TIME_DEL)

Benchmark                      Mode  Cnt   Score   Error  Units
MyBenchmark.customVanilla     thrpt   25  47.726 ± 0.422  ops/s
MyBenchmark.customVanilla2    thrpt   25  74.062 ± 0.438  ops/s
MyBenchmark.refactoredEquals  thrpt   25  53.445 ± 0.557  ops/s

The run in the benchmark can be viewed here.
customVanilla - copied over the code for Key.equals()
customVanilla2 - same as current equals function just the delete and timestamp are evaluated first for the PartialKey.ROW_COLFAM_COLQUAL_COLVIS_TIME_DEL case (as suggested by @EdColeman in a comment above)
refactoredEquals - refactored to reuse code and make the method smaller
It is probably important to take note of how the data is being produced and other factors before drawing any conclusions here.
From the results above it seems like both of the other versions of this method are (potentially negligibly) more performant than the current code. The refactoredEquals method also has the advantage of not being "hot" (elaborated here).
I think I need some input on these benchmarks and their findings to determine if anything significant can be drawn from them. Specifically, how the data in the benchmark is being produced and the structure of the benchmark in general.


Week 4 Maps: Hello World

Completes first Week 4 PR-displays basic Google Map on webpage

Whoops! I did not mean for my Phase 1 of styling commit to be includes in this PR! Sorry for confusion it may have caused-it is not nearly ready for review at the moment :)


[SUGGESTION] Gas Giant Visuals

it would be cool for the gas giant to have phases depending on the time of day. During the night, the gas giant would be dark as there would be no sunlight hitting that side, and throughout the course of the day become brighter and darker similar to moon phases. It would also be cool to see atmospheric activities like storms and jetstreams.

I already thought about giant phases, but it will require large amount of work, that's why it was not implemented yet


Admin Boolean in User Table

Example: admin BOOLEAN default 'False'
const users = [
{
firstname: 'Josh',
lastname: 'Mace',
email<EMAIL_ADDRESS>password: 'josh123',
address: 'Some Place',
city: 'Dallas',
state: 'Texas',
zipcode: '75287',
admin: 'TRUE',
},

Looks good. 👍
On Fri, Mar 22, 2024, 10:06 AM Michael Jaroszynski @.***>
wrote:

Example: admin BOOLEAN default 'False'
const users = [
{
firstname: 'Josh',
lastname: 'Mace',
email: @.***',
password: 'josh123',
address: 'Some Place',
city: 'Dallas',
state: 'Texas',
zipcode: '75287',
admin: 'TRUE',
},
—
Reply to this email directly, view it on GitHub
https://github.com/2401-Group-A/Capstone-EComSite/issues/80, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BFEL2HTDKEXBDG6PEEYTLELYZRCITAVCNFSM6AAAAABFDPZVR2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGIYDENZRG42DGOA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>