When building, setting unsplash keys?

@basshelal
Hi,
I could only build Waqti when I uncommented unsplash imports and environment varibles.
I did register and get my own keys from unsplash.com, but to what file I should add those keys so that they work correctly, so I don't need to uncomment unsplash?
I have built Waqti on Android Studio on Ubuntu 20.04 64bit.
Waqti debug mode does seem to work on emulator.
On OnePlus 3 Android 9 phone when I run Waqti directly via adb on phone, and there click board, Waqti crashes. Maybe this could be related to that I don't have unsplash settings correct yet.

It seems that these are related to background image #4

Hi there @xet7
Thanks for showing interest in my little project here.
See if this quick fix works for you:
I'm gitignoring a file called Keys.kt which is in a package called keys.

Add this file with with the same variable names representing your access key and private key and uncomment what you've commented out and see if it works for you.
Please note, Waqti is extremely unstable, I developed it as a way of learning and playing around with Android and it has grown far beyond that (I'm so bad with feature creep 😩) so I've ended up deciding to modularize it. But unfortunately I haven't been able to work on it as much as I'd like, University steals a lot of my time from open source 🙁.
If you're interested in the BoardView then I recommend you check out my new repo BoardView. I'm nearly done with it and I'd say it's 80% stable and usable, just missing some features. It aims to make a BoardView which will in the future help remove a lot of complexity from Waqti.
Let me know if I can help you in any other way, I'd be more than happy to 😊.
I assume you're looking at this for Wekan?

@basshelal
I'm just testing at first can I get this working. I don't know yet am I able to make any significant changes because I have not programmed with Kotlin.


[WIP] Adding Kubernetes instructions

Signed-off-by: metonymic-smokey<EMAIL_ADDRESS>Description
Adding instructions on using mesheryctl to install Meshery on Kubernetes.
Screenshot

This PR fixes #409
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@leecalcote draft PR opened, awaiting your feedback!
Should I highlight/change text colour for config.yaml and kubernetes?

Thank you. Will you confirm that this command does not create the meshery namespace?

On Mar 25, 2021, at 7:27 AM, Aditi Ahuja @.***> wrote:

@leecalcote draft PR opened, awaiting your feedback!
Should I highlight/change text colour for config.yaml and kubernetes?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

You're right @leecalcote, my bad.
Fixed that.

@leecalcote anything more I need to add to this PR?
Thanks!

Thanks for this @metonymic-smokey 👍


Test: IPC refactoring

[ ] Windows @alexandrudima
[ ] Mac @waderyan
[ ] Linux @gregvanl

The IPC infrastructure had a bit of a refactoring. Thus a good smoke test over the following features would be much appreciated:

Git
Search, Quick Open
Extension management

Touched all these areas today through self-hosting


Add addBook function

In this pull request we have made the following changes

Add linter config files
Add index and script file
Add Books to a book list
Delete Books from the book list
Save data to local storage

Hi @kazim110 & @Yordinia,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that doesn't have clear commits from both partners as requested by the previous Code Reviewer.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Added Appium+Kobiton Integration Tests

Probably, but the clean up does nothing of value. If we're changing it, we
may as well remove it.
On Mon, Jun 10, 2019 at 4:15 PM Nick Graeff<EMAIL_ADDRESS>wrote:

@NickGraeff commented on this pull request.
In
integration/src/main/java/com/iovation/launchkey/sdk/integration/mobile/driver/android/BaseAndroidDriver.java
https://github.com/iovation/launchkey-java/pull/63#discussion_r292226392
:

   navigate().back();

}

public void pressMenu() {

   findElementByAndroidUIAutomator("new UiSelector().description(\"Navigate up\")").click();

}

public void pressElementWithName(String name) {

   findElementByAndroidUIAutomator("new UiSelector().textContains(\""+name+"\")").click();

}

public void resetTheApp() {

   resetApp();

   appWasReset();

   try {

       Thread.sleep(10000);

Getting rid of the sleep seems to have worked just fine.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/iovation/launchkey-java/pull/63?email_source=notifications&email_token=AACQHSFBESNZXZM7IPRZDBTPZ3OBBA5CNFSM4HVNOU6KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB3DHYDQ#discussion_r292226392,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACQHSDS5KVFVDJOE4K7NYDPZ3OBBANCNFSM4HVNOU6A
.

--
[image: line]
Adam Englander
Manager of Engineering
O +1<PHONE_NUMBER> | M +1<PHONE_NUMBER>
[image: Iovation] https://www.iovation.com/


Global admin clarification

Global admin also gives access to Azure RBAC, see https://twitter.com/PyroTek3/status/1265718452041519107

@AlexFilipin : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


KMS_KEY_ID recreates database everytime

How do we solve the problem of kms_key_id field?
When creating the rds for the first time kms_key_id = arn.

Then running terraform again, kms_key_id = is not the id and hence it force replaces. since arn is being passed.
Is there a way we could add a ignore change lifecyle for the kms_key_id
I can create a PR if you'd like.
Thank you.

I guess it would be possible to use a lifecycle hook to ignore changes, but to be honest the issue here is that you're passing a ID not an ARN, so the setting was never valid and in that case ignoring the change seems misguided at best, and pointless at worst.
The preferred solution would be to distroy the instance, and recreate it with a correctly-specified ARN.  Or if your DB is in-use and cannot be replaced then don't update the ARN leaving it without encryption at rest.


codedeploy-scripts not found during deployment

On deployment  the codedeploy-scripts module is not found.
Do I need to add this to my package.json ?

Of course, same as any node module. You'll also need to make sure that npm install has run before creating the codedeploy package; running npm on the server won't work with the default setup, though in theory you could wrap deployment.js in another script that calls npm install first.

Solved the note on npm didn't have the --save. SO now it is in the
project

thanks again


Notifiacition every 5 minutes

This issue is:

[x]  Bug report
[ ] Feature request
[ ]  Improvement

Actual behaviour
Telegams notifications are sent everytime the cronjob is executed.
Expected behaviour
Notifications are sent only once or in configurable intervals (e.g. at most once every day).
Steps to reproduce
Configure the cronjob to run every 5min.
Create a task and let it run beyond the due date
Receive an email every 5min

Configuration
Versión de la aplicación: 1.2.16
Versión de PHP: 7.3.16
PHP SAPI: fpm-fcgi
Cliente HTTP: cURL
Versión del sistema operativo: Linux 3.10.105
Driver de la base de datos mysql
Versión de base de datos: 10.3.21-MariaDB
Navegador: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:82.0) Gecko/20100101 Firefox/82.0

Generation of notifications are handled by Kanboard directly. The Telegram plugin only delivers the notification once they are generated. Please raise this issue upstream in the kanboard project.


Update Mache's example on README.md

I found a typo where mykey was showed instead of mytest and quickly fixed it. I also added an extra example line to clarify what mache does.

I'll make sure the README.md change is reflected in npm in the next version (sometime this week-ish).


Implement POST /preview

Coverage: 89.308% (+0.2%) from 89.1% when pulling e43acbf8bbde6e7b735bac70a882a3fcba050c66 on add-transaction-preview into 9a1937152015b41a460365054de711767d97c691 on main.


Migrate @metamask/eth-json-rpc-provider into core monorepo

Explanation

References

Closes #1682
Closes #1683
Closes #1551
See #1079

Changelog

Checklist

[ ] I've updated the test suite for new or updated code as appropriate
[ ] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
[ ] I've highlighted breaking changes using the "BREAKING" category above as appropriate

@MajorLift perhaps would be better to break down this PR into multiple ones to make it easier to see and review the changes, and creating the first two PRs on the original repository:

#1682
#1683

Thanks for the feedback! I'll be closing this PR and reopening it.
@legobeat Thanks for pointing this out. It looks like using the latest version of @metamask/utils is necessary to clear the errors currently showing up in CI. I'll bump it in the new PR.
@mikesposito I just opened the first two PRs on the original repo. It's looking like most of the commits will happen on the last PR after the merging step.


(BKR-307) Add Beaker::Platform support for debian jessie (aka 8)

@anodelman @melissa @camlow325
This should fix the debian8 failure on the puppet-server pipeline. I believe the Beaker team will be releasing a new dot release this afternoon, hopefully this patch will be included which would minimize the effort necessary to get the debian8 cells green on the puppet-server master branch pipeline.

ok to test

So, while we're here, should we also add some of the other new platforms?
ubuntu 14.10 -> utopic
ubuntu 15.04 -> vivid

@melissa that sounds good, what about ubuntu 15.10 -> wily? https://wiki.ubuntu.com/DevelopmentCodeNames

That's not out until October, but sure, why not!

@melissa thanks for the feedback, I have updated the PR

:+1: thanks for taking care of this @waynr !

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/899/
Test FAILed.

@anodelman so as far as I can tell, tests passed?

Yup, looks good.

Oh wait, it is actually completing a test run right now.  Give it 20 more minutes.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/901/
Test PASSed.

Aw yiss :+1:


added effect on selection and added animated cursor

Added  Animated Cursor on the website
please accept for hacktoberfest 2023

Please accept this pr for Hacktoberfest

@samratghosh291 issue no is #37


fix pdf file

Changes Unknown when pulling 18be21ab8f97634b56a2366f1fbf9d282fafad3a on indriHutabalian:uat into ** on danliris:uat**.


Fix typo

Thanks for PR'ing this typo!
The README.rst is generated so the fix should be done within the fixer itself as well.
https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/master/src/Fixer/Comment/NoEmptyCommentFixer.php#L70
Let me know if you've the time to do this, otherwise I can pick it up later.

Yeah, I can get that spot tomorrow (Japan time).

Updated.

:+1: thanks :)

Thank you @fritz-c.


Fix tk attribute

Was able to launch on windows with this change.

Thank you so much! I just stumbled on this fix myself. Its commit out in 3.16

Thanks, closing as it's fixed in main

Please feel free to continue to submit any PRs. I'll do my best to stay on top of them.


amethyst shard drop on punch

As depicted, the normal function of shift+punching the frame works, but it spits out particle effects and an amethyst shard along with the chime sound effect. I can't really imagine why this is happening, but if no one else is experiencing this, it must be just a weird mod conflict as I have many installed. I opened this issue incase this was of any interest to you, not so much that it was a problem strictly on your part.

conflict, it can probably be easily resolved by just giving invisframes a higher priority. i would like to know whats causing it though just to make sure

so i at least need a mod list / modpack name

sure thing, I apologise though as there are 86 other mods installed, though
I'm sure more than a few can be ignored for this matter.
Tell me if there's anything else i can share
(version 1.17)
https://drive.google.com/drive/folders/1KhOuImUfxC79KLsG63inVX_FY33a29HX?usp=sharing
On Tue, Jul 6, 2021 at 2:57 AM SFort @.***> wrote:

so i at least need a mod list / modpack name
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SFort/MC-invisframes/issues/4#issuecomment-874237249,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUXSSJUFIB4B6UZ4OFPPNDTTWHP5ZANCNFSM473BBJ2Q
.

ye that weas easy to find conflict with charm

there's no way this can be fixed by me it is compleatly caused by charm.
it assumes it's the only thing that can set entities invisable which is a really dumb
as several mods and even vanilla can do that,
so this is flat out just a item generation bug caused by charm
(made easier by invis frames)
sorry theres nothing i can do you should open a issue on their bugtracker

thank you very much for getting back to me on this, I'll definitely report
the issue to them.
On Tue, Jul 6, 2021 at 2:38 PM SFort @.***> wrote:

there's no way this can be fixed by me it is compleatly caused by charm.
it assumes it's the only thing that can set entities invisable which is a
really dumb
https://github.com/svenhjol/Charm/blob/5f454874683710a504317423b50016f46c991130/src/main/java/svenhjol/charm/module/clear_item_frames/ClearItemFrames.java#L73
as several mods and even vanilla can do that,
so this is flat out just a item generation bug caused by charm
(made easier by invis frames)
sorry theres nothing i can do you should open a issue on their bugtracker
https://github.com/svenhjol/Charm/issues
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SFort/MC-invisframes/issues/4#issuecomment-874452999,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUXSSJXJ4QG5H7OTFWQL333TWKCC3ANCNFSM473BBJ2Q
.


ip command ( replacement for ifconfig) not working

Build 14931
This might be a duplicate issue as ifconfig does not work as well. Just logging it here.
the command ip fail: (example)

ip
SO_SNDBUF: Invalid argument

From a networking point of view it would be nice for this to work as "ifconfig" as I understand it is being replaced with "ip"
Should note that all commands to ip seem to fail the same way.

Thanks @dragon512 for the feedback. Yes, ip tool does not work today, just as ifconfig, for different reasons though. ip has dependency on NETLINK sockets and ifconfig relies on SIOCG* IOCTLS. Both of these are currently not implemented in WSL and we are looking into enabling these tools. This is also covered in this User Voice Post. Feel free to voice your opinion and vote there. Similar posts are #908, #767. Do bear with us as we make progress and hopefully you are having fun with WSL!

Closing as dupe of  #908, #767.


Code generation and more flexible quantities

Yes in hindsight, it was a dump idea to apply style changes in this PR as well ...
I am sorry


Add support for loyalty point events

Twitch Loyalty system has events for redeption, probably this can be added?

Channel point redemptions are handled through the Twitch PubSub API, which is distinct from the Twitch IRC system that most of Twitch chat is built on.

I managed to interact with the loyalty system by toggling the option "Ask for user input" on my rewards and then using the unique custom_reward_id every rewarded message has.
When you ask for user input, Twitch ships the custom_reward_id along the mesage through the IRC channel.
And twitch-bot parses that value without ny further modification.


[DO NOT MANUAL MERGE] Release Qualification

Trigger release qualification jobs
none

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

Test name
Commit
Details
Rerun command

prow/daily-e2e-cluster_wide-auth.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-cluster_wide-auth

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

@silverdev: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-e2e-cluster_wide-auth.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-cluster_wide-auth

prow/daily-e2e-cluster_wide-auth-default.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-cluster_wide-auth-default

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

@silverdev: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-e2e-cluster_wide-auth.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-cluster_wide-auth

prow/daily-e2e-cluster_wide-auth-default.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-cluster_wide-auth-default

prow/daily-e2e-rbac-no_auth.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-rbac-no_auth

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

@silverdev: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-e2e-cluster_wide-auth.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-cluster_wide-auth

prow/daily-e2e-cluster_wide-auth-default.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-cluster_wide-auth-default

prow/daily-e2e-rbac-no_auth.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-rbac-no_auth

prow/daily-e2e-rbac-no_auth-default.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-rbac-no_auth-default

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

@silverdev: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-e2e-cluster_wide-auth.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-cluster_wide-auth

prow/daily-e2e-cluster_wide-auth-default.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-cluster_wide-auth-default

prow/daily-e2e-rbac-no_auth.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-rbac-no_auth

prow/daily-e2e-rbac-no_auth-default.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-rbac-no_auth-default

prow/daily-e2e-rbac-auth-default.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-rbac-auth-default

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

@silverdev: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-e2e-cluster_wide-auth.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-cluster_wide-auth

prow/daily-e2e-cluster_wide-auth-default.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-cluster_wide-auth-default

prow/daily-e2e-rbac-no_auth.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-rbac-no_auth

prow/daily-e2e-rbac-no_auth-default.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-rbac-no_auth-default

prow/daily-e2e-rbac-auth-default.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-rbac-auth-default

prow/daily-e2e-rbac-auth.sh
1afc463bcbdee7d07a120f79afe7a02efe391e9f
link
/test daily-e2e-rbac-auth

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


#19 Add Dockerfile.

Include ca-certificates.crt to allow secure connections to the outside
world.
Future backends may require this.

@stephenwithav Although you have permissions, do not merge until a reviewer(me in this case) approves. I will do the same. I was still reviewing this as I want to make sure that we have the right build process for images.

Apologies, @vimukthi-git--I will do so in the future.
Aside from a force push, is there any way to unmerge this PR?

https://help.github.com/articles/reverting-a-pull-request/

@stephenwithav no need to do so for this one, we can follow review process from now on. Having a docker image is still good.


ISCN 3.5: Jorgensen_ARCN

[x] Evaluate dataset
[x]  Complete template checklist
[ ]  Write up summary report narrative
[ ]  Good commenting
[ ]  External review by someone else

9d920126d86253cbec4ca6263edb7586cc63d7a3 Dataset complete! There is a cryptic note in vegclass_local and textureclass.

@ktoddbrown Ready for Review? Not sure if I should merge before or after the review?


is this work complete so can train or test?

hello ,
very good work,   but i want to know if this work is finished so that we can train or test ?  and any guide to run the project, thank you very much, hope your reply.

Hi, I would like to know as well.

Hey any updates on the completion ?

Also very interested to see the project fully developed for its application in full human pose inference


Labels info

Add labels_info/0 to CIFAR100, CIFAR10, and FashionMNIST to show descriptions of dataset labels.

Oh, something I want to add is that we eventually want to add this kind of pretty printing to Nx directly as part of FeatureSet. So perhaps it makes sense to hold on merging this until that work is done? 🤔

@josevalim sure thing, let's wait on this until we can integrate with FeatureSet.


Validation videos availablility

Hi,
Thanks a lot for sharing the code.
When I downloaded the validation videos, I found that a number of them are 'unavailable'. Could you please tell how many were available for you?
Thanks,

Hi,
It was a project from long time ago, so I don't really remember the number, but I guess it was probably less than 20? However, since the videos are directly from some links on Youtube as far as I remember, so I guess over the years, more and more videos might be invalid already. Hope this could help a little bit.
Thanks!

Thanks a lot for the quick reply and the info.


Honestly, I have no idea why it is not working. Maybe the issue lies in my audio related stuff? I'm not good with code and whatnot so I'm clueless here

You can remove cable output from your hardware input. You should also reduce open mic record threshold setting in apex to 0

I originally didn’t have a hardware input but even then it was not working. Maybe my issue lies outside of the audio related stuff? Regardless, thank you for your response.


Insufficient memory leads to package install failure, but the build still succeeds

Issue
I am using micromamba with a large set of both conda and pip dependencies. When running docker build, the micromamba install step fails to install some dependencies, but the docker build continues without error, leading to an image missing a number of dependencies.
I'm using Docker Desktop on Mac, and I noticed that increasing the memory resources for Docker from the default of 2.0GB up to 2.5GB results in successful build, so it seems like insufficient memory is causing the micromamba install failure.
Desired behavior
The main issue is that the docker build completes successfully even though micromamba failed to install all the packages, resulting in a broken image. Ideally, micromamba would cause the build to fail.
Details
My dockerfile:
# syntax=docker/dockerfile:1
FROM --platform=linux/amd64 mambaorg/micromamba:0.13.1
COPY environment-lock.yml /environment-lock.yml
RUN --mount=type=cache,target=/opt/conda/pkgs --mount=type=cache,target=/root/.cache \
    micromamba install -y --file /environment-lock.yml

The environment-lock.yml is very large, and includes both conda and pip dependencies. I tried paring it down, but it was hard to reproduce the failure with a more minimal list. Let me know if you want me to share the full list.
Error
With --progress plain, I see the following output from docker build while micromamba is in the middle of installing pip dependencies specified in environment-lock.yml (this is after the conda deps were installed):
[...]
#9 441.2 Collecting tensorboard-plugin-wit==1.8.0
#9 441.2   Downloading tensorboard_plugin_wit-1.8.0-py3-none-any.whl (781 kB)
#9 442.7 Collecting tensorflow==2.5.0
#9 442.7   Downloading tensorflow-2.5.0-cp38-cp38-manylinux2010_x86_64.whl (454.4 MB)
#9 466.3 /tmp/mambafVPPFiskWCz: line 4:  2363 Killed                  /opt/conda/bin/pip install absl-py==0.11.0 apache-beam==2.30.0 [...a bunch of other pip packages...] xarray==0.16.2 --no-input
#9 DONE 466.7s

With some random poking, I found an example /tmp/mamba* file that looked like this:
lib/python3.8/site-packages/ansiwrap/__init__.py
lib/python3.8/site-packages/ansiwrap/ansistate.py
lib/python3.8/site-packages/ansiwrap/core.py
eval "$('/bin/micromamba' 'shell' 'hook' '-s' 'bash' '-p' '/opt/conda')"

Solution(?)
It seems like the "Line 4:  ... Killed" is referring to the final line in the script mentioned above. I think the problem is that the system is terminating the command on line 4 due to running out of memory, but this failure isn't propagated to the RUN micromamba install ..., so the docker build succeeds.
I think including set -e or using exec in this script would cause the error to propagate to micromamba.
Misc
Some instances of people saying they needed to increase the memory limits for docker build:

https://github.com/commercialhaskell/stack/issues/3963
https://stackoverflow.com/questions/26913110/allow-more-memory-when-docker-build-a-dockerfile

Hi @jli  - thanks for the detailed report.
I am not able to come up with any way to mitigate this at the docker-level other than giving more memory to docker, which doesn't solve the root issue.  I think this issue should be moved to https://github.com/mamba-org/mamba, which is responsible for the micromamba codebase.
I did do some work reproducing this. I wanted to avoid the complexities of the docker build ... environment but I needed a way to limit the memory and ulimit wasn't working. I ended up running micromamba in a container so I could use docker's resource limiters.
$ cat pip.yaml
name: base
channels:
  - conda-forge
dependencies:
  - pip
  - pip:
    - tensorflow
$ docker run -it --rm --memory=300m --memory-swap=300m --mount type=bind,source=$(pwd),target=/mnt mambaorg/micromamba:latest micromamba install -y -vvvv -f /mnt/pip.yaml

##### MUCH OUTPUT REMOVED ######

TRACE   hard-linked '/opt/conda/pkgs/wheel-0.36.2-pyhd3deb0d_0/site-packages/wheel/wheelfile.py'
         --> '/opt/conda/lib/python3.9/site-packages/wheel/wheelfile.py'
DEBUG   23 files linked
INFO    Compiling "lib/python3.9/site-packages/wheel/__pycache__/__init__.cpython-39.pyc"
INFO    Compiling "lib/python3.9/site-packages/wheel/__pycache__/__main__.cpython-39.pyc"
INFO    Compiling "lib/python3.9/site-packages/wheel/__pycache__/bdist_wheel.cpython-39.pyc"
INFO    Compiling "lib/python3.9/site-packages/wheel/cli/__pycache__/__init__.cpython-39.pyc"
INFO    Compiling "lib/python3.9/site-packages/wheel/cli/__pycache__/convert.cpython-39.pyc"
INFO    Compiling "lib/python3.9/site-packages/wheel/cli/__pycache__/pack.cpython-39.pyc"
INFO    Compiling "lib/python3.9/site-packages/wheel/cli/__pycache__/unpack.cpython-39.pyc"
INFO    Compiling "lib/python3.9/site-packages/wheel/__pycache__/macosx_libfile.cpython-39.pyc"
INFO    Compiling "lib/python3.9/site-packages/wheel/__pycache__/metadata.cpython-39.pyc"
INFO    Compiling "lib/python3.9/site-packages/wheel/__pycache__/pkginfo.cpython-39.pyc"
INFO    Compiling "lib/python3.9/site-packages/wheel/__pycache__/util.cpython-39.pyc"
INFO    Compiling "lib/python3.9/site-packages/wheel/vendored/__pycache__/__init__.cpython-39.pyc"
INFO    Compiling "lib/python3.9/site-packages/wheel/vendored/packaging/__pycache__/__init__.cpython-39.pyc"
INFO    Compiling "lib/python3.9/site-packages/wheel/vendored/packaging/__pycache__/_typing.cpython-39.pyc"
INFO    Compiling "lib/python3.9/site-packages/wheel/vendored/packaging/__pycache__/tags.cpython-39.pyc"
INFO    Compiling "lib/python3.9/site-packages/wheel/__pycache__/wheelfile.cpython-39.pyc"
INFO    Running wrapped python compilation command /opt/conda/bin/python3.9 -Wi -m compileall -q -l -i /tmp/mambafTl74sI4Jss -j0
INFO    noarch pyc compilation failed (cross-compiling?). Success
INFO    /tmp/mambafJxKSiatqAG: line 43: : command not found

TRACE   entry point path: "bin/wheel"

DEBUG   post-link script for 'wheel' does not exist ('/opt/conda/bin/.wheel-post-link.sh')
DEBUG   Finalizing linking
TRACE   Adding package to prefix metadata at '/opt/conda/conda-meta/wheel-0.36.2-pyhd3deb0d_0.json'
Transaction finished
INFO    Opening history file: "/opt/conda/conda-meta/history"

Installing pip packages: tensorflow
INFO    Calling: /opt/conda/bin/pip install -r /tmp/mambafvJT15WettK --no-input
/tmp/mambaf8guFlIGAYT: line 43: : command not found
Collecting tensorflow
  Downloading tensorflow-2.5.0-cp39-cp39-manylinux2010_x86_64.whl (454.4 MB)
     |██████████▍                     | 147.2 MB 220 bytes/s eta 16 days, 2:11:32/tmp/mambaf8guFlIGAYT: line 4:    57 Killed                  /opt/conda/bin/pip install -r /tmp/mambafvJT15WettK --no-input
INFO    Freeing transaction.
INFO    Freeing solver.
INFO    Freeing pool.
$ echo $?
0
$

I believe your issue would be solved if that test case returned a non-0 exit code. And it also would seem to me that the expected behavior is that micromamba install ... commands that don't complete their install should return a non-0 exit code.

Ah, I was looking for a "micromamba" repo and failed to find it, so defaulted to reporting here. I'll report over on the mamba repo, thanks! Also nice finding a simple test case :)

I filed the issue w/ mamba-org/mamba: https://github.com/mamba-org/mamba/issues/1100
Thanks!


Need help for setting custom baseline file for Ktlint plugin

Hello @nbadal @paul-dingemans!
Background
I have been using Ktlint & your plugin since couple of years.
In all my Kotlin projects, as a personal preference I stuck to the linting & formatting with the plugin version 0.13.0.
Problem
For all the later versions, my code was resulting in a major code reformatting because of the Ktlint enhancements.
So I stuck to 0.13.0 for since then.
But the recent IDE versions 2023.3.1+ are not supporting 0.13.0 version, so had to upgrade to the latest plugin version 0.22.0.
Which is again resulting into massive code reformatting.
Solution
To avoid this I need to set the older version of Ktlint baseline to suit my preference.
I tried, but I am not able to set it.

Can anyone please help me?
Details
My Android Studio config:
Android Studio Jellyfish | 2023.3.1 Patch 1
Build #AI-233.14808.21.2331.11842104, built on May 15, 2024
Runtime version: 17.0.10+0-17.0.10b1087.21-11572160 aarch64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
macOS 14.4.1
GC: G1 Young Generation, G1 Old Generation
Memory: 4096M
Cores: 10
Registry:
  ide.instant.shutdown=false
  debugger.new.tool.window.layout=true
  ide.experimental.ui=true
  idea.ignore.disabled.plugins=true
  ide.images.show.chessboard=true
Non-Bundled Plugins:
  com.github.patou.gitmoji (1.14.0)
  io.github.biezhi.plugins.gitmoji (1.0.2)
  com.zhw.tgnet.android.AndroidProGuard (1.4)
  wu.seal.tool.jsontokotlin (3.7.4)
  com.github.airsaid.androidlocalize (3.0.0)
  com.nbadal.ktlint (0.22.0)
  com.programmersbox.Full-Multiplatform-Compose (1.0.26)
  com.jetbrains.kmm (0.8.2(233)-8)
  com.suusan2go.kotlin-fill-class (1.0.23)
  org.sonarlint.idea (<IP_ADDRESS>339)
  com.kaygisiz.dependencygraph (2.1)
  cn.yiiguxing.plugin.translate (3.6.1)

Any help is appreciated.
Cheers. 👍🏼 🚀

It is a risky strategy to pin your ktlint version to an ancient version, although I can understand your wish to prevent massive code formatting changes.
In this case you are forced to upgrade to 0.22 version as the newer versions of the Intellij IDEA deprecated/removed functionality used in the older plugin version. We try to keep the plugin working with the oldest possible version of the Intellij IDEA platform, but sometimes we are forced to raise the minimum required version.
In the upcoming version of the plugin, it will be possible to select a specific version of the ktlint ruleset. This will also support the 0.50.0 version of ktlint. It is still a newer version than the one that was integrated in plugin version 0.13.0 but it might be accpetible to you. But do note, that there no guarrantee that this version will be kept supported in upcoming releases. You can already try out the beta version 0.23.0-beta-1.2024-05-19_16-14-58 (https://plugins.jetbrains.com/plugin/15057-ktlint/versions/beta).

Closed due to lack of response.


This PR adds support to m740dasm for the M50734 processor

This PR includes two changes to the main repo to add support to m740dasm for the M50734 processor:

Adds the interrupt vectors table, the special function registers symbols, and the vectors for this particular processor. These are added to devices.py.
Adds a M50734.py file to the docs folder, which also contain a list of the special function registers.

The source of the key information in this PR comes from an extremely large PDF called Mitsubishi Semiconductors 1989 Single-Chip 8-bit microcomputers databook. It is available via archive.org: Source here See pages 667-715.
I've tried to mimic the existing code, and touch as little as possible. The devices.py changes are simply appended to the bottom of the existing main copy. The docs file is just added to the folder. I tried to use a similar naming convention to other processors when possible, but YMMV.
Testing

I ran pylint on both files, before and after, and the scores have not dropped.
I rebuilt via a setup.py install and no errors were thrown.
I tested the newly created m740dasm against one of the only ROM images I have. I don't have a great sample size here, but it appears to be functioning fine.
I ran the end-to-end tests and they pass --- a diff between original and reassembled return nothing. I'm not changing anything, however, that I would expect to have any effect on this.

Looks good!  Thanks for all the effort you made to test the changes.


v2 Switch List

Create a switch to v2 (moves slack bot, UI to v2) by workspace.
We'll use it to switch us, then design partners and then remove the switch and switch everyone

https://dust4ai.slack.com/archives/C050SM8NSPK/p1695384624626379

https://github.com/dust-tt/dust/pull/1678


Refactor API abstractor

@na3d mind providing a short description of the pain points?

The abstractor was built pretty early on, and specs have changed since then.  We need to be able to handle all API endpoints in reasonable ways.  We have a couple endpoints, such as linode/ips and account/profile, that need to be handled appropriately.

note: because of how we build objects based on endpoint name and response format, we have a case where we build an undefined key. This needs to be resolved as part of this issue.
see this.props.profile.profile.undefined;

[ ] src/profile/layouts/AuthenticationPage.js
[ ] src/profile/layouts/DisplayPage.js
[ ] src/profile/layouts/IndexPage.js
[ ] src/profile/layouts/ReferralsPage.js
[ ] src/settings/layouts/IndexPage.js
[ ] src/users/user/layouts/PermissionsPage.js

This has been fixed with the addition of the singular vs. plural configs.


Unable to use project name containing hyphens

Summary
The flit docs say this about the project.name attribute in pyproject.toml:

name
The name your package will have on PyPI. This field is required. For Flit, this name, with any hyphens replaced by underscores, is also the default value of the import name (see Module section if that needs to be different).

However if I try to use a name that contains a hyphen, I get an error.
This means I cannot use a distribution name containing a hyphen, even though the upstream spec allows it.
Steps to reproduce

mkdir testcase && cd $_
mkdir my_package && touch my_package/__init__.py
Create the pyproject.toml using:cat > pyproject.toml<< 'EOF'
[build-system]
requires = ["flit_core >=3.2,<4"]
build-backend = "flit_core.buildapi"

[project]
name = "my-package"
version = "0.0.0"
description = ""
EOF

pip install -e .

Expected
Pip install succeeds.
Actual
(venv) testcase $ pip install -e .
Obtaining file:///Users/emorley/src/testcase
  Installing build dependencies ... done
  Checking if build backend supports build_editable ... done
  Getting requirements to build editable ... error
  error: subprocess-exited-with-error

  × Getting requirements to build editable did not run successfully.
  │ exit code: 1
  ╰─> [12 lines of output]
      Traceback (most recent call last):
        File "/Users/emorley/src/testcase/venv/lib/python3.10/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 363, in <module>
          main()
        File "/Users/emorley/src/testcase/venv/lib/python3.10/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 345, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
        File "/Users/emorley/src/testcase/venv/lib/python3.10/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 144, in get_requires_for_build_editable
          return hook(config_settings)
        File "/private/var/folders/kp/1j9qfrqn79b867c9xzshx9kr0000gp/T/pip-build-env-ov8_3n9i/overlay/lib/python3.10/site-packages/flit_core/buildapi.py", line 31, in get_requires_for_build_wheel
          module = Module(info.module, Path.cwd())
        File "/private/var/folders/kp/1j9qfrqn79b867c9xzshx9kr0000gp/T/pip-build-env-ov8_3n9i/overlay/lib/python3.10/site-packages/flit_core/common.py", line 59, in __init__
          raise ValueError("No file/folder found for module {}".format(name))
      ValueError: No file/folder found for module my-package
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build editable did not run successfully.

Notes

Using flit 3.7.1, pip 22.2.2, Python 3.10 (from homebrew), macOS
Flit seems to not be normalising the module name, contrary to what the docs say.

You're looking at /latest/ documentation, which reflects the state of main and not the current release of flit.
https://flit.pypa.io/en/stable/pyproject_toml.html#new-style-metadata is the page, with the documentation for the current release of flit.

Ah, I followed the link from the GitHub repo's description, which is https://flit.pypa.io/ and redirects to /latest/.
Perhaps the default for the redirect should be /stable/?
Also, checking https://pypi.org/project/flit/ I see the "documentation" link actually directly links to /latest/, so should probably be changed too :-)

I've changed that in the default -- it'll take a while to purge from the cache. :)

Wanna file a PR to drop the en/latest/ from https://github.com/pypa/flit/blob/f86127764bddb05cb6b5d9a1b6ae4346f786c94a/pyproject.toml#L40? :)

I've changed that in the default -- it'll take a while to purge from the cache. :)

Thank you!

Wanna file a PR to drop the en/latest/ from

Sure - opened #589 :-)

regarding the actual bug (that I also suffered and then found this issue), it seems it's already fixed by this line, but it's not yet released.
https://github.com/pypa/flit/commit/ae453f538e24072cdcf7fea703d0d622eb5be18b#diff-0bda35abe5b807229c9e97babeb5c873c57c0921c4f0f0834f02db5dfa093666R430


Update LAB_AK_03_Lab1_Ex1_retention_policies.md

Correction for typo noted in https://github.com/MicrosoftLearning/SC-400T00A-Microsoft-Information-Protection-Administrator/issues/5

Good catch.  Merge approved.


⚠ Bofrost.lu has degraded performance

In 9dd6ccc, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2542 ms

Resolved: Bofrost.lu performance has improved in dd9bfd2 after 6 minutes.


ECONNRESET thrown after connection closed

less v4 uses needle for https requests.  It fails with ECONNRESET on macOS specifically, which is reproducible in GitHub Actions as well as on our private mac hardware.  The failure is caused by an ECONNRESET error after the connection is closed.
Details can be found here:

https://github.com/less/less.js/issues/3693
https://github.com/nodejs/node/issues/27916

The node discussion includes a workaround, which is that you can call abort() explicitly after closing the connection.  I will send a PR with that fix shortly.

abort is deprecated since node.js v13.14.0 as per the documentation here. destroy should be used instead.


Multiple SMS for a share

If a share price has been increased more than the defined limit, each time the code runs SMS will be sent. This should be avoided.

Possible suggestions:

Send SMS uniquely for each time the share price crosses on the multiples of the defining_limit.

Plan to solve:

Set a flag and changed percentage if the share has been notified for the first time in a day.
Reset all the flags and percentages EOD.
Check the share's flag --> If set check if the price changed more than the set second_defining_limit.


offline handle

In offline,
$ deepl :zh <<<"hello"
Traceback (most recent call last):
  File "/home/eggplants/.pyenv/versions/3.8.0/bin/deepl", line 33, in <module>
    sys.exit(load_entry_point('deepl-cli', 'console_scripts', 'deepl')())
  File "/home/eggplants/prog/deepl-cli/deepl/main.py", line 6, in main
    print(t.translate())
  File "/home/eggplants/prog/deepl-cli/deepl/deepl.py", line 104, in translate
    input_area = d.find_element_by_xpath(
  File "/home/eggplants/.pyenv/versions/3.8.0/lib/python3.8/site-packages/selenium/webdriver/remote/webdriver.py", line 394, in find_element_by_xpath
    return self.find_element(by=By.XPATH, value=xpath)
  File "/home/eggplants/.pyenv/versions/3.8.0/lib/python3.8/site-packages/selenium/webdriver/remote/webdriver.py", line 976, in find_element
    return self.execute(Command.FIND_ELEMENT, {
  File "/home/eggplants/.pyenv/versions/3.8.0/lib/python3.8/site-packages/selenium/webdriver/remote/webdriver.py", line 321, in execute
    self.error_handler.check_response(response)
  File "/home/eggplants/.pyenv/versions/3.8.0/lib/python3.8/site-packages/selenium/webdriver/remote/errorhandler.py", line 242, in check_response
    raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.NoSuchElementException: Message: no such element: Unable to locate element: {"method":"xpath","selector":"//textarea[@dl-test="translator-source-input"]"}
  (Session info: headless chrome=83.0.4103.116)

https://github.com/eggplants/deepl-cli/commit/ec90a1231ffb43a1f86228796183826de30faced


Load services by name and external nodes

Hi,
For my purpose, i have to add some features to Fabio:

I've added a new boolean option called registry.consul.register.byServiceName. If it is set to true, then fabio will ignore urlprefix- tag existence and add service to routing table by its name. E.g. if service is called test-service then path to this service will looks like  /test-service. I've added this because we have lots of legacy services in consul and it's really difficult to change all deployment roles to add urlprefix- in consul. Such option provide us a possibility to migrate to fabio without pain.
Also i've added a flag option called registry.consul.service.externalNodes.  This option accepts a list of external nodes in consul (https://www.consul.io/docs/guides/external.html) separated by ,. If this option exists, then fabio will register all services from this nodes to it's routing table. Purpose of this option is to have external services in our routing table and standard fabio way is not acceptable cause external services doesn't have healthchecks. Comma-separated flag is also a work-around cause there is no possibility in Consul (0.7.0) API to tag node as external.
And another option, called proxy.stripPath. There were a lot of issues about this problem (e.g. my last one #164). If this option is enabled, then path to service will be stripped (E.g. GET /test-service/api → GET /api

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

@Salmondx Thanks for the contribution. A couple of comments:

That sounds like an interesting feature but I would prefer a different implementation. Right now the code that gets the services from consul and the code that builds the routing table from this service list is one. What would make sense is to separate fetching services and generating a routing table from a list of services into separate pieces to make both of them configurable independently. So in essence build an internal plugin like the Picker or Matcher that builds a routing table from a list of services based on some config. Then your approach for your legacy services would just be another filter. Or we could have multiple as long as they are exclusive.

I think you can solve the external nodes problem by adding static routes to consul. Just add the routes you need. This is kind of a limitation of supporting only one service registry at a time. If you could have both consul and file then you can just copy a file somewhere.

I'm already working on the path stripping.

Taking these things into account I won't merge the change as is. Also, because it combines several features into one larger change. If the patch is working for you right now feel free to use it and I can refactor the code as described so that you can build a filter. Or you can provide some code if you want. Separate PR please.

@magiconair Thanks for review!

Ok, i'll fix this and make a separate PR.
Hm, static route is not a solution for me. For example, in my company we register legacy soap services as services on external node. So, node test has lots of services and its not possible to describe them in external config file, especially when fabio doesn't support dynamic config reload.
May be there are another solution for my problem?
Ok, it'll be really cool feature!

As for 2. why don't you register them in consul on deployment or write a small tool which manages the registration? That seems like the Right Thing (TM) to do instead of patching the load balancer. This would also solve your first problem.

So please wait with another PR until we've settled on an approach.

@magiconair they are registered in consul on external node.
E.g. soap-services node has several registered services. But there is no possibility for us to register healthcheck cause they doesn't provide such endpoint.

Can't you just register a TCP check or a simple script that performs a function that concludes that the service is up? You don't need to have a specialized /health endpoint to perform the health check.

Sorry, missed the fact that there are several services on a single node. So the underlying problem is that fabio is looking for passing services in consul but your services do not have a health check. fabio has an option to specify the consul service status but that may be too broad. registry.consul.service.status. You could set this to passing,unknown

I cant, cause consul api doesn't return services without registered health checks through /v1/health/state endpoint that used in fabio.

How do you know your service is up then?

It's a third party services and we have no control over them and no fallback is possible. Like in consul doc's, external node points on google service :)

Hmm, this registers an external service on my local consul including tags and fabio integration. What am I missing?
# register the service
$ curl -i -XPOST -H 'Content-Type: application/json' -d '{"ID": "marktplaats", "Name": "marktplaats", "Tags": ["urlprefix-www.marktplaats.nl/"], "Address": "<IP_ADDRESS>", "Port": 80, "EnableTagOverride": false, "Check": {"HTTP": "http://<IP_ADDRESS>/", "Interval": "15s"} }' 'http://localhost:8500/v1/agent/service/register'
HTTP/1.1 200 OK

# check that health check is passing
$ curl -i 'http://localhost:8500/v1/health/state/passing?pretty'
HTTP/1.1 200 OK
Content-Type: application/json
X-Consul-Index: 63
X-Consul-Knownleader: true
X-Consul-Lastcontact: 0
Date: Fri, 30 Sep 2016 11:26:26 GMT
Transfer-Encoding: chunked

[
    {
        "Node": "LM-AMS-00963965",
        "CheckID": "serfHealth",
        "Name": "Serf Health Status",
        "Status": "passing",
        "Notes": "",
        "Output": "Agent alive and reachable",
        "ServiceID": "",
        "ServiceName": "",
        "CreateIndex": 4,
        "ModifyIndex": 4
    },
    {
        "Node": "LM-AMS-00963965",
        "CheckID": "service:marktplaats",
        "Name": "Service 'marktplaats' check",
        "Status": "passing",
        "Notes": "",
        "Output": "HTTP GET http://<IP_ADDRESS>/: 200 OK Output:   ..."
    }
]

# check that fabio picks it up
$ fabio

...
2016/09/30 13:27:31 [INFO] Updated config to
route add marktplaats www.marktplaats.nl/ http://<IP_ADDRESS>:80/ tags "urlprefix-www.marktplaats.nl/"
...

# test that it works (separate terminal)
curl -i -H 'Host: www.marktplaats.nl' 'localhost:9999/'

# deregister service
curl -i -XPOST http://localhost:8500/v1/agent/service/deregister/marktplaats

With service you also register a healthcheck, and in my case its not an option:
"Check": {"HTTP": "http://<IP_ADDRESS>/", "Interval": "15s"}

If Fabio is supposed to route HTTP traffic to this service how can this not be an option? You need to have some monitoring around this which verifies that your external service is up. If you don't do this you can register a health check that never fails like shell with /bin/true.
I'm not saying you don't have a case but right now I fail to see it. Please keep explaining what I'm missing.

Because a fake healthcheck is a work around, and external services shouldn't be monitored cause you can't change anything if they fail. For example, if i have a google as external service, i should also use an http health check and add unnecessary load on my consul cluster? May be i'm wrong.

The fake health check would indeed be just a workaround.
The hole in your argument is that you claim that you can reach the service via HTTP since fabio should forward traffic to it but you cannot use exactly the same mechanism to establish the consul health check. That doesn't make sense to me.

Hmm, I should have read your full response instead of just the first sentence. :/
OK, so in essence you could monitor the external service but you don't want to since you don't think this is the right thing to do. You also do not want to provide a always-true health check to ensure that fabio works in your environment since you consider that a kludge. You would rather change the load balancer instead? Is this your argument?
I get your point but I'm not sure I agree with it. You are basically saying that health checks should be actionable which is fine. But it falls short if your service depends on the availability of the external service. Then you want to know that your service is either failing or degraded because of an external factor even if you cannot change it. At least you can stop digging and maybe escalate the problem. This then is the action of your failing health check.
From that perspective I'd absolutely would check external services if my own service depends on them.
And by doing that it solves your first two problems at once without any code change which has to be maintained. Only the path stripping is missing.

@Salmondx Are you still using this patch? Path stripping has been added some time ago.

Closing because of age. Pls comment to re-open.


[BugFix] concurrent export failed

Steps to reproduce the behavior (Required)
http://<IP_ADDRESS>:8090/view/version_publish/job/branch_release_2.3/113/testReport/junit/test_export_aggtbl_cols_to_hdfs/TestExportAggTblColsToHDFS/test_aggtbl_cols_export_task/
nosetests-3.4 -s --with-xunit --xunit-file=./result/test_export_aggtbl_cols_to_hdfs_py_TestExportAggTblColsToHDFS_test_aggtbl_cols_export_task.xml --verbose case/system_case/test_ddl/test_export/test_export_aggtbl_cols_to_hdfs.py:TestExportAggTblColsToHDFS.test_aggtbl_cols_export_task --nologcapture

modify case

Executed successfully
StarRocks version (Required)
2.3/2.4

Problem:
This problem is cause by the following reason: Coordinator.exec may execute through different Coordinator instance. When they get a ScopedTimer, the same timer may be return to different thread in in concurrent scenarios. This will cause the same ScopedTimer instance to be started and closed multiple times, and an exception will be thrown.
Solution:
New a new PlannerProfile to ensure that different Coordinator.exec get different timers when executing export(export don't set ConnectContext.threadLocalInfo, so we can use ConnectContext.get() to determine whether export is being executed).


Ember 3.12 Support?

The README mentions that this addon is compatible with Ember 3.12, but the tests don't run against that version of Ember and, as far as I can tell from usage, it does not actually work. Could you clarify whether it's intended that this addon works with Ember 3.12?

Hey @alexlafroscia thanks for flagging this. I think that's just the default boilerplate from the version of Ember CLI with which @pzuraq generated this, and given it uses autotracking extensively I don't see how it could work with 3.12! I'll open a PR momentarily which resolves this by correcting that!


⚠ ClamAV has degraded performance

In 5301f49, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 709 ms

Resolved: ClamAV performance has improved in 3f1b1cb.


docs, test, fix : updated circular queue using array

The current code is more efficient and more structured than the previous one, both in terms of readability and performance. I removed some unnecessary checks and added more documentation
Description of Change

Checklist

[x] Added description of change
[x] Added file name matches File name guidelines
[x] Added tests and example, test must pass
[x] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[x] Relevant documentation/comments is changed or added
[x] PR title follows semantic commit guidelines
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] I acknowledge that all my contributions will be made under the project's license.

Notes:

Please check the checkboxes in your pr.

ok working on it

@realstealthninja please check again and let me know if anything needs to be addressed


hdiutil fails when using debug=1 on Monterey (running installomator.sh from ~/Downloads)

I noticed that Installomator fails on Monterey with Debug=1, running the script from ~/Downloads.
It seems Monterey has more checks for mounting dmg's, it fails from ~/Downloads, but not from /var/folders...
Running Installomator with sudo.
the error is (sorry, some Dutch here)
2021-07-08 10:01:07 googlechrome Mounting Downloads/Google Chrome.dmg
hdiutil: attach failed - Bestand of directory bestaat niet
2021-07-08 10:01:07 googlechrome Error mounting Downloads/Google Chrome.dmg

With Debug=0 a tmpdir is created, and the mounting works as expected:
2021-07-08 10:05:08 googlechrome Mounting /var/folders/zz/zyxvpxvq6csfxvn_n0000000000000/T/tmp.VoGKMY3Q/Google Chrome.dmg
2021-07-08 10:05:18 googlechrome Mounted: /Volumes/Google Chrome
2021-07-08 10:05:18 googlechrome Verifying: /Volumes/Google Chrome/Google Chrome.app
2021-07-08 10:05:33 googlechrome Team ID matching: EQHXZ8M8AV (expected: EQHXZ8M8AV )

Tested with Google Chrome and Slack.
Monterey beta 2, on M1 MacBook Pro, master branch.
I am new to this team, so not sure how to proceed. Can provide more logs if needed.

Can you try to give /bin/zsh access to Downloads folder. I would think that should make it work. Otherwise test it from another folder.

Good catch, @Theile, these are things one can forget on Beta testing devices.
I have checked, 'Terminal' has access to ~/Downloads.
I have added "Terminal' and /bin/zsh to 'Full Disk Access' and same error keeps repeating.
Luckily, it can mount the .dmg when I run the script from /Users/Shared/ both for debug=0 and debug=1
Note: with debug=1 there is nothing copied to /Applications/, but that is expected behavior
Consider this issue closed


Lizard plushie color and marking variation

About the PR

Greyscales the lizard plushie and allows them to have up to sixteen different colors for the body and four markings included.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

add: The Weh™ plushies now come with four markings and up to sixteen different colors! Collect around 73.000 of them!

@DogZeroX You coming back to this?

Requires another PR (tbd) so yes

re-open once the other pr is done

Is the pr done yet :godo:


IDB getAll() sample

R: @addyosmani @gauntface @beaufortfrancois
CC: @inexorabletash
This implements a basic sample to go along with the new M48 IndexedDB "batch get" methods (https://www.chromestatus.com/feature/6537756637396992).
I'm using a light Promise wrapper around getting the IndexedDB instance, but I don't think that's too distracting.
You can view a deployed version of this sample at https://jeffy.info/samples/idb-getall/

To be honest, I was confused about the Promise wrapper. I think something simpler as below would help developers to focus more on the getAll method.
-var DB_NAME = 'timestamps';
-var DB_VERSION = 1;
 var STORE_NAME = 'store';
-var dbInstance;
-
-// Simple Promise-based wrapper around getting an IndexedDB instance.
-function getDB() {
-  if (dbInstance) {
-    return Promise.resolve(dbInstance);
-  }
-
-  return new Promise(function(resolve) {
-    var request = indexedDB.open(DB_NAME, DB_VERSION);
-
-    request.onupgradeneeded = function() {
-      request.result.createObjectStore(STORE_NAME, {autoIncrement: true});
-    };
-
-    request.onsuccess = function() {
-      dbInstance = request.result;
-      resolve(dbInstance);
-    };
-  });
-}
+var db;

+var request = indexedDB.open('timestamps', 1);
+request.onsuccess = function() {
+  db = request.result;
+};
+request.onupgradeneeded = function() {
+  db.createObjectStore(STORE_NAME, { autoIncrement: true});
+};

 document.querySelector('#add').addEventListener('click', function(event) {
-  getDB().then(function(db) {
   var transaction = db.transaction(STORE_NAME, 'readwrite');
   var objectStore = transaction.objectStore(STORE_NAME);
   objectStore.put(Date.now());
-  }).catch(function(error) {
-    ChromeSamples.setStatus(error);
-  });
 });

 document.querySelector('#display').addEventListener('click', function(event) {
-  getDB().then(function(db) {
   var transaction = db.transaction(STORE_NAME, 'readonly');
   var objectStore = transaction.objectStore(STORE_NAME);
   // IDBObjectStore.getAll() will return the full set of items in our store.
   objectStore.getAll().onsuccess = function(event) {
     var timestamps = event.target.result;
-      ChromeSamples.setStatus('There are ' + timestamps.length + ' timestamps saved in IndexedDB: '
-        + timestamps.join(', '));
+    ChromeSamples.setStatus('There are ' + timestamps.length +
+        ' timestamps saved in IndexedDB: ' + timestamps.join(', '));
   };
-  }).catch(function(error) {
-    ChromeSamples.setStatus(error);
-  });
 });

It looks like there's a race condition in your proposed modifications, in that the click event listeners might be triggered before db is assigned to a fully functional IndexedDB instance.
The promise wrapper could be overkill, sure. What I could do is start out with the buttons initially disabled, and only enable them inside of request.onupgradeneeded, which should be safe enough for a sample.

Yeah you're right.
Enabling buttons should be enough then.

PTAL.

LGTM except nit.

Worth adding a check when the sample is run on a browser that doesn't support this API?

@gauntface, there's a global error handler in place to avoid having to add in feature checks in all the samples. Here's what it should look like on unsupported browsers:

(The earlier version of this samples didn't trigger the handler, since a failure involved a promise rejection rather than an exception, but the modified version without promises will trigger it.)

My main thinking is that there may be developers that want to feature detect.

Given that there's a clear fallback behavior to use, sure, I'll update the sample to use that if getAll() isn't supported.

@gauntface PTAL. I've modified the sample to check for getAll() support, and fallback to cursors if it's not supported.

LGTM


Reorder contents on the workshop page

We should put more important information closer to the top of the page.
How about something like this?
Introduction
Important dates
Topics of interest
"Hack Together" programming event
Submission guidelines
Invited speakers
Awards
Diversity and inclusion statement
Workshop format
Registration
Venue
Program committee
Organizing committee
Contact information
More information

Updated, thanks! https://github.com/sigturk/sigturk.github.io/pull/28

Great job @akoksal , feel free to close issues at your discretion when you feel it is addressed sufficiently


Fixed missing ingredient if called 'I have X and would like tocook'

Thanks @mariushegele  now the tests pass


Operator creating a registry which connects to a secured Kafka

As far as I can tell the Operator only supports creating registry instances which connect to a Kafka broker which has no security enabled. In production most Kafka brokers will have some kind of encryption, authentication and authorisation. For instance

TLS encryption
SASL authentication
SCRAM-SHA-512 authentication

For example setups see: https://github.com/strimzi/strimzi-kafka-operator/tree/master/examples/security

Resolved, a follow up issue is #32


How to transition old pads using ep_headings to ep_headings2?

I have ep_headings and ep_headings2 installed.
If I uninstall ep_headings and go to a pad that has headings created with it, I get an error message referring to some template not found in ep_headings folder.
How can I solve this?

look at the source code and figure it out?

I could not reproduce this in a fresh install. Installed ep_headings, created a pad and set text to heading. When I uninstalled it, the heading was shown *like this.
I tried to solve it in the problematic install by resetting the heading style to normal in the affected pad I was testing with, but no luck.
Here is the error message:
Error: ejs:67
   65|                 <% e.begin_block("editbarMenuLeft"); %>
   66|                 <%- toolbar.menu(settings.toolbar.left) %>
>> 67|                 <% e.end_block(); %>
   68|             </ul>
   69|             <ul class="menu_right" role="toolbar">
   70|                 <% e.begin_block("editbarMenuRight"); %>

Cannot find module 'ep_headings/templates/editbarButtons.ejs' from '/home/etherpad/etherpad-lite/node_modules/ep_etherpad-lite/templates'
   at Function.module.exports [as sync] (/home/etherpad/etherpad-lite/src/node_modules/resolve/lib/sync.js:33:11)
   at Object.exports.require (/home/etherpad/etherpad-lite/src/node/eejs/index.js:106:25)
   at Object.exports.eejsBlock_editbarMenuLeft [as hook_fn] (/home/etherpad/etherpad-lite/node_modules/ep_headings/index.js:4:38)
   at normalizedhook (/home/etherpad/etherpad-lite/src/static/js/pluginfw/hooks.js:15:27)
   at hookCallWrapper (/home/etherpad/etherpad-lite/src/static/js/pluginfw/hooks.js:21:14)
   at /home/etherpad/etherpad-lite/src/static/js/pluginfw/hooks.js:76:14
   at Function._.map._.collect (/home/etherpad/etherpad-lite/src/node_modules/underscore/underscore.js:172:24)
   at Object.exports.callAll (/home/etherpad/etherpad-lite/src/static/js/pluginfw/hooks.js:75:24)
   at Object.exports.end_block (/home/etherpad/etherpad-lite/src/node/eejs/index.js:82:9)
   at eval (eval at <anonymous> (/home/etherpad/etherpad-lite/src/node_modules/ejs/lib/ejs.js:455:12), <anonymous>:51:10) ``

I'm available as a gun for hire if you want me to fix this for you.  I'm not going to fix it for free though because it's not a problem with my code :)

I'm a volunteer gun in the LibreOffice project so I'll probably badger our infra team about it. I'll update and close, when the cause is found.


The crawl dashboard is not coming up

I get the following when I try to bring the crawl dashboard up:

I get the same screen when I go to http://<IP_ADDRESS>:8983/solr/#/ I get that same error.  Ditto with http://<IP_ADDRESS>:8983/banana/.
However, when I check the docker images, they look fine (though there are extras given the testing I was doing a while ago):

and the containers seem to be fine as well:

but the logs do show that there is a problem - the sce.log repeats the following error every second or two:
2019-03-06 05:36:47 WARN  NativeCodeLoader:62 [main] - Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
2019-03-06 05:36:49 INFO  Crawler$:147 [main] - Committing crawldb..
Exception in thread "main" java.lang.reflect.InvocationTargetException
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at edu.usc.irds.sparkler.Main$.main(Main.scala:49)
at edu.usc.irds.sparkler.Main.main(Main.scala)
Caused by: org.apache.solr.client.solrj.SolrServerException: Server refused connection at: http://localhost:8983/solr/crawldb
at org.apache.solr.client.solrj.impl.HttpSolrClient.executeMethod(HttpSolrClient.java:617)
at org.apache.solr.client.solrj.impl.HttpSolrClient.request(HttpSolrClient.java:279)
at org.apache.solr.client.solrj.impl.HttpSolrClient.request(HttpSolrClient.java:268)
at org.apache.solr.client.solrj.SolrRequest.process(SolrRequest.java:149)
at org.apache.solr.client.solrj.SolrClient.commit(SolrClient.java:484)
at org.apache.solr.client.solrj.SolrClient.commit(SolrClient.java:463)
at edu.usc.irds.sparkler.service.SolrProxy.commitCrawlDb(SolrProxy.scala:62)
at edu.usc.irds.sparkler.pipeline.Crawler.run(Crawler.scala:148)
at edu.usc.irds.sparkler.base.CliTool$class.run(CliTool.scala:34)
at edu.usc.irds.sparkler.pipeline.Crawler.run(Crawler.scala:48)
at edu.usc.irds.sparkler.pipeline.Crawler$.main(Crawler.scala:310)
at edu.usc.irds.sparkler.pipeline.Crawler.main(Crawler.scala)
... 6 more
Caused by: java.net.ConnectException: Connection refused (Connection refused)
at java.net.PlainSocketImpl.socketConnect(Native Method)
at java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:350)
at java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:206)
at java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:188)
at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
at java.net.Socket.connect(Socket.java:589)
at shaded.org.apache.http.conn.scheme.PlainSocketFactory.connectSocket(PlainSocketFactory.java:117)
at shaded.org.apache.http.impl.conn.DefaultClientConnectionOperator.openConnection(DefaultClientConnectionOperator.java:177)
at shaded.org.apache.http.impl.conn.ManagedClientConnectionImpl.open(ManagedClientConnectionImpl.java:304)
at shaded.org.apache.http.impl.client.DefaultRequestDirector.tryConnect(DefaultRequestDirector.java:611)
at shaded.org.apache.http.impl.client.DefaultRequestDirector.execute(DefaultRequestDirector.java:446)
at shaded.org.apache.http.impl.client.AbstractHttpClient.doExecute(AbstractHttpClient.java:882)
at shaded.org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:82)
at shaded.org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:55)
at org.apache.solr.client.solrj.impl.HttpSolrClient.executeMethod(HttpSolrClient.java:515)
... 17 more
It looks like Solr is not up and running or at least neither I nor the SCE can connect to it.
Suggestions?

Hey @rduerr ,
I just cloned the repo and installed it locally on my Mac for the first time. It came up alright, although Solr doesn't have an index, that aside it did all start so I'm not getting the error you see.
If you do still see it, can you do:
docker logs compose_sparkler_1
and see if there's anything solr related in it.

I need to turn the system off - as the logs are still emitting errors every second or two.  However I saved a minute or so from the beginning of the log and grep -i for solr which had lots of perfectly fine looking lines - see attached file...
headsolrquery.txt
but the end of the file is just continual "Server refused connection" errors (see attached file)
tailsolrquery.txt

Did a pull and now the crawl appears to be working...


Error using transform-define

An example of using transform-define for universal configuration is provided, but I have a weird error, which I narrowed down below (run it with babel-node):
// package.json
{
  "dependencies": {
    "babel-plugin-transform-define": "^1.2.0"
  },
  "babel": {
    "plugins": [
      [
        "transform-define",
        "./config.js"
      ]
    ]
  }
}

// config.js
module.exports = { KEY: 'value' };

// index.js
const config = require('./config');
console.log(config.KEY);

Somehow next manages to load my app based on the above principle just fine, but the console.log throws when I run the server myself (I'm trying to add integration tests):
TypeError: /Users/sdubois/Documents/relate-debug/index.js: Property property of MemberExpression expected node to be of a type ["Identifier"] but instead got "StringLiteral"
    at validate (/usr/local/lib/node_modules/babel-cli/node_modules/babel-types/lib/definitions/index.js:109:13)
    at Object.validate (/usr/local/lib/node_modules/babel-cli/node_modules/babel-types/lib/definitions/core.js:412:50)
    at Object.validate (/usr/local/lib/node_modules/babel-cli/node_modules/babel-types/lib/index.js:505:9)
    at NodePath._replaceWith (/usr/local/lib/node_modules/babel-cli/node_modules/babel-traverse/lib/path/replacement.js:176:7)
    at NodePath.replaceWith (/usr/local/lib/node_modules/babel-cli/node_modules/babel-traverse/lib/path/replacement.js:160:8)
    at replaceAndEvaluateNode (/Users/sdubois/Documents/relate-debug/node_modules/babel-plugin-transform-define/lib/index.js:111:12)
    at processNode (/Users/sdubois/Documents/relate-debug/node_modules/babel-plugin-transform-define/lib/index.js:148:5)
    at PluginPass.Identifier (/Users/sdubois/Documents/relate-debug/node_modules/babel-plugin-transform-define/lib/index.js:23:9)
    at newFn (/usr/local/lib/node_modules/babel-cli/node_modules/babel-traverse/lib/visitors.js:276:21)
    at NodePath._call (/usr/local/lib/node_modules/babel-cli/node_modules/babel-traverse/lib/path/context.js:76:18)

Strangely enough, it works when I change to console.log(config['KEY'])).

// index.js = your custom server?
If you run it with babel-node it will transpile all occurences of KEY which might cause what you're seeing?

OK it's because my custom server is normally simply started with node server, which doesn't do any babel stuff, but in my jest tests, the require('../../server') I'm using there is transformed by babel.
I'll try to adapt the babel config not to load that plugin in test env.
It's not trivial to get all the universal config right...

@tim how can I set up my babel config, so that it doesn't transpile a certain folder (server/)?


CAMEL-18904:  Add functions to simple language to create empty map/li…

Description
Ticket:  https://issues.apache.org/jira/browse/CAMEL-18904
This is one of my first OpenSource-Commits, so I hope I understood the task correctly.
I wasn't quite sure, of which type the empty json should be or which other types would be useful, so this is still an open TODO.

Thanks, we can skip json for now. And then make equalsIgnoreCase for all of them.

Thank you, I removed the open json-TODO for now and used equalsIgnoreCase for String, List and Map


Kubernetes deployments

Description
We are not only deploying to servers (VM and bare metal), but to Kubernetes Clusters too.  Can we enhance Waltz to allow for a further hardware option for being deployed to K8s?
It could be hosted on our own K8s cluster on prem, or to a K8s cluster hosted by a Cloud provider (GCP, AWS etc)
The cost reporting system must aggregate the infrastructure cost categories (CPU, memory, GPU, storage, network and load balancer) in different dimensions that include the following:
Container
Pod
Cluster
Namespace
Controller
Service
Label
This level of aggregation is essential when allocating costs across business concepts such as applications, projects, products (SaaS products hosted in a cluster), teams, environments (production versus testing), departments or cost centers. Any business unit to which costs are being allocated should map to a Kubernetes component (such as a namespace)
Resourcing
We would like to add this request to the Waltz team's feature backlog

Model
Would you view this as a new set of entities in Waltz, e.g.:
Cluster :: Node :: Pod :: Contanier  :: ....
... the lowest level (service / container / namespace ?) would be associated to apps.
Or would you view this as denormalized entitiy with the cluster,pod etc as attributes ?
Costs
Costs can be associated to non-application entities, but currently we record cost information against applications.

We will try and get some views from ourside on how this could be structured as we have little expertise within the team.


[submodule-sync] bot-submodule-sync-branch-23.10 to branch-23.10 [skip ci] [bot]

submodule-sync to create a PR keeping thirdparty/cudf up-to-date.
 If failed, it will remain open until test pass or manually fix.


Vision fine-tuning

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [X] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Mention any other details that might be useful
When fine-tuning the vision-model I think it's possible for fine-tuning the vision model with non-lora and fine-tune the language model with lora.
I've made a code for this by borrowing code from llava. I hope this could be helpful for updating the training script for fine-tuning vision model.
https://github.com/2U1/Phi3-Vision-ft

@2U1 Hi would like make a pull request with you addition?

@2U1 please :raised_hands:

@leestott @franperezlopez Yes sure!

Sorry I was bit busy. I'm working on to make in a one script file. I'll use the DocVQA for the example.

I'd like to understand why the CLIP model can't be trained using LORA, as stated in this comment https://github.com/microsoft/Phi-3CookBook/blob/20d56d79cfd38eb175118ecc961a9b49e2341de2/code/04.Finetuning/vision_finetuning/finetune_hf_trainer_docvqa.py#L94
I made myself a lora_config based on this code, and so far, it worked
    linear_modules = [
        # CLIP modules
        'q_proj',  # attention
        'k_proj',
        'v_proj',
        'out_proj',
        'fc1',  # MLP
        'fc2',
        # 'img_projection.0',
        # 'img_projection.2',
        # FIXME: can't lora CLIP is a known issue of Phi-3-V
        # Phi language modules
        'qkv_proj',  # attention
        'o_proj',
        'down_proj',  # MLP
        'gate_up_proj',
        # 'lm_head',
    ]
    lora_config = LoraConfig(
        r=rank,
        lora_alpha=round(rank * alpha_to_rank_ratio),
        lora_dropout=dropout,
        target_modules=linear_modules,
        init_lora_weights='gaussian',
        task_type=TaskType.CAUSAL_LM,
        modules_to_save=["lm_head"],
    )

I was a bit busy doing some other works. I'm a bit struggling with making my codes in one script that I've changed some codes in the processor and image_embedding. I'll make it asap.

@ChenRocks
Interesting comment:

I'd like to understand why the CLIP model can't be trained using LORA, as stated in this comment
https://github.com/microsoft/Phi-3CookBook/blob/20d56d79cfd38eb175118ecc961a9b49e2341de2/code/04.Finetuning/vision_finetuning/finetune_hf_trainer_docvqa.py#L94
I made myself a lora_config based on this code, and so far, it worked
    linear_modules = [
        # CLIP modules
        'q_proj',  # attention
        'k_proj',
        'v_proj',
        'out_proj',
        'fc1',  # MLP
        'fc2',
        # 'img_projection.0',
        # 'img_projection.2',
        # FIXME: can't lora CLIP is a known issue of Phi-3-V
        # Phi language modules
        'qkv_proj',  # attention
        'o_proj',
        'down_proj',  # MLP
        'gate_up_proj',
        # 'lm_head',
    ]
    lora_config = LoraConfig(
        r=rank,
        lora_alpha=round(rank * alpha_to_rank_ratio),
        lora_dropout=dropout,
        target_modules=linear_modules,
        init_lora_weights='gaussian',
        task_type=TaskType.CAUSAL_LM,
        modules_to_save=["lm_head"],
    )

on latest main branch this is no longer a limitation

@ChenRocks Oh I was too late for this. I'll close the issue, it will be too much for changing the other things I've made.
Thanks for updating the code :) !


Use updated DEX logo as prescribed from brand guidelines

What would you like to be added:
Replace the previous DEX logo with a new one to align with newly prescribed brand guidelines
Why is this needed:
Brand consistency and visual refinements. Brand guidelines prescribed the use of the logo in this way
 can be seen here in this screen link
Here are the files for convenience (includes 3 sizes of PNG and SVG format):
Defichain DEX logo.zip

/triage accepted
/assign @siradji


Support execute if function and future execute if conditions.

Made the changes

Anything else I need to do?


Disable cors on vod api

What does this pull request do? Explain your changes. (required)

Specific updates (required)

-

How did you test each of these updates (required)

Does this pull request close any open issues?

Screenshots (optional):

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

LGTM! go ahead and merge.


Sidebar status icons should have accessibility attributes

Fixes AB#82763
The sidebar and card status icons should have accessibility attributes such that screen reader users can understand the information being conveyed by the icons displayed.

If it's possible to move the image inside the <Link> I think that would be helpful. That would mean when you tabbed to a link a screenreader would say "Link Asset Breakdown Section Not Complete" rather than "Link Asset Breakdown" and then wait for the user to discover that there's an image afterwards with further information.

This has been done in commit 77b58b9, I've tested using NVDA in Chrome, FIrefox & IE11 and in each of those browsers NVDA reads out the link name with the status as suggested.
In Edge, NVDA does not read out the status, but I'm not sure there's anything we could do about that.


Replacement for GeneratePseudoFromClause

What's the recommended way to replace GeneratePseudoFromClause now missing in 3.1? Do I really have to override VisitSelect and copy the whole logic only for this case handling?

Probably in 3.1 you need to override VisitSelect. We will be adding more extensibility point in next release.

That's sad. Especially given it was already there? Out of curiosity, is there any reason it was removed? Or plain obliviousness?

Old query pipeline had a lot of APIs which were design errors. So we decided to prune surface and make sure that we expose only things which are needed and makes sense. This got removed because it was not used. We are willing to add surface as it is needed and specifically if it is right way to go.

This got removed because it was not used.

Firebird provider used it. And the Oracle sample too (but I know that was dropped even before this).
What sucks in current design, with overriding whole VisitSelect, is that if something unrelated is fixed there, I have to first notice it and then obviously copy it. But it's what it is now.


Enable "Private Intransitive Dependencies" By Default

"Private Intransitive Dependencies" differ from the status quo by no
longer requiring the concept of a "cascading dependency edge". This is
because the request evaluator automatically tracks, records, and replays
the names looked up while a given file is being processed by the
frontend. To remove transitivity, each primary file is charged for all name
lookups that occur while it is being processed.  Further, because of
the replay step, lookups hidden behind cached requests are now entirely
visible to the dependency tracking code.
The net result is that all formerly implicit transitivity in the
dependency graph has been made completely explicit and direct. This
establishes a tighter overall dependency structure for each individual
file, and results in a remarkable decrease in the amount of
files that are rebuilt for any particular change.
This feature can be disabled via
-disable-experimental-private-intransitive-dependencies, which will
cause a return to the cascading status quo.

@swift-ci smoke test

Should we drop "experimental" from the name if we're enabling it by default?

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test

@swift-ci clean smoke test

@swift-ci clean smoke test

@swift-ci smoke test macOS platform

@swift-ci smoke test macOS platform

@swift-ci smoke test Linux platform

@swift-ci smoke test Linux platform

@swift-ci smoke test macOS


Issue-540: Includes fix from goldstone-compliance issue-540

Reviewers
Alex Jacobs
Description
Update goldstone-compliance fixtures for saved search 4b4d5447-f689-49c2-ac95-bb27f2f4897a. The fix included replacing the "should" clause with a "must" clause that contains a single "query_string" with the query criteria.  Also use a YAML block quote format to deal with embedded apostrophe.
closes #issue-540
How Has This Been Tested?
See comments in compliance PR https://github.com/Solinea/goldstone-compliance/pull/78
Types of changes

[x] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to change).

Checklist:

[ ] My change requires new model migrations.
[ ] I have executed makemigrations (if necessary).
[ ] My change requires new docker images for development.
[ ] I have created and pushed new docker images (if necessary).
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly (if necessary).
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed locally.
[ ] Test coverage percentage has improved.
[ ] PEP8 passes (for python code).
[ ] Pylint score has improved (for python code).

[Current coverage][cc-pull] is 69.01%

Merging [#562][cc-pull] into [develop][cc-base-branch] will not change coverage

@@            develop       #562   diff @@
==========================================
  Files            37         37
  Lines          1336       1336
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            922        922
  Misses          414        414
  Partials          0          0

[][cc-pull]

Powered by Codecov. Last updated by [28607fe...1fbf825][cc-compare]
[cc-base-branch]: https://codecov.io/gh/Solinea/goldstone-server/branch/develop?src=pr
[cc-compare]: https://codecov.io/gh/Solinea/goldstone-server/compare/28607fee7d68ed871ae2265257b0c93c42d18ff7...1fbf8257649c9a1f0f32b3a503674eff6e9ae300
[cc-pull]: https://codecov.io/gh/Solinea/goldstone-server/pull/562?src=pr


A Multi-phase, Scheduled Finetuning Callback

🚀 Feature
A callback that enables multi-phase, scheduled finetuning of foundational models.
    from pytorch_lightning import Trainer
    from pytorch_lightning.callbacks import FinetuningScheduler
    trainer = Trainer(callbacks=[FinetuningScheduler()])

Motivation
Gradual unfreezing/thawing can help maximize foundational model knowledge retention while allowing (typically upper layers of) the model to optimally adapt to new tasks during transfer learning [1, 2, 3].
When tuning pre-trained large language (aka foundational) models for downstream tasks over the last couple years, I've personally observed the benefits of this technique in multiple project contexts and have been using/refining code to expedite the application of this pattern. Given that this approach to finetuning continues to be widely used and that multi-phase finetuning has be requested by others in the PL community I thought it could be immensely useful to the community to provide a callback (extending BaseFinetuning) for this purpose. I've created said callback (named FinetuningScheduler) and have been using it from my PL fork to great effect that last few months and am hoping others may find it similarly useful.

Howard, J., & Ruder, S. (2018). Fine-tuned Language Models for Text Classification. ArXiv, abs/1801.06146.
Chronopoulou, A., Baziotis, C., & Potamianos, A. (2019). An embarrassingly simple approach for transfer learning from pretrained language models. arXiv preprint arXiv:1902.10547.
Peters, M. E., Ruder, S., & Smith, N. A. (2019). To tune or not to tune? adapting pretrained representations to diverse tasks. arXiv preprint arXiv:1903.05987.

Pitch
Though approaches to leveraging foundational models for downstream tasks are continually evolving (e.g. prompt/prefix-tuning etc.), finetuning w/ gradual unfreezing continues to be widely used and multi-phase finetuning has be requested by the PL community (https://github.com/PyTorchLightning/pytorch-lightning/issues/2006) to boot. I think multi-phase finetuning is a natural extension of the BaseFinetuning functionality that PL provides and comports nicely with its aspiration to decouple the science from the engineering.
The PR I'm submitting includes a fully-functional, tested and documented beta version of the FinetuningScheduler (fts) callback as well as a new example (in ./pl_examples/basic_examples/fts) demonstrating a few use cases as applied to a SuperGLUE benchmark task using the LightningCLI. Given the nature of this callback, I thought a LightningCLI-based example was better suited than a notebook-based one.
Rather than re-iterate the documentation here in detail, I think the best way to get a sense of the potential utility of this callback would be to review the documentation I've provided in the PR and execute the new example. At a high-level though, this callback essentially implements gradual unfreezing of foundational models via either explicit or implicit finetuning schedules. Explicit finetuning mode involves unfreezing/thawing layers based upon user-defined layer groupings. Schedule definition is facilitated via a method that dumps a default finetuning schedule which can be adjusted as desired by the user and subsequently passed to the callback. Implicit finetuning mode generates the default schedule and proceeds to finetune according to the generated schedule.
Alternatives
Since this pattern is so commonly used, I think it makes sense to have it available in the PL framework as a callback rather than have it implemented in each user LightningModule or as a community example. I'd note that I did consider modifying ModelCheckpoint to accommodate this callback, but ultimately decided given the extensive usage of that callback, it would be more prudent to extend ModelCheckpoint with FTSCheckpoint, at least while FinetuningScheduler is in beta.
Additional context
Feel free to look at the Tensorboard experiment demo I've linked to in the documentation. While I've made other minor contributions to PyTorch Lightning, this is my first feature contribution, so please bear with me if there are any shortcomings wrt my contribution. Thank you so much to everyone in the PL community for contributing to this awesome framework! I've found it immensely useful and plan to continue using it (and evangelizing about it) in the future.
cc @borda

Unless there's objection, I think we can close this now that the requested functionality is available via the Finetuning Scheduler extension (used in this lightning tutorial).
On a related note, any sense of when the _notebooks submodule will be updated? Looks like it was last updated in mid-Jan so wondering if there will be an update tied to the 1.7 release. Nice work on 1.7 so far btw! :zap: :rocket: :tada:


Drop Python 3.3 support, Add 3.6

Support for version 3.3 has been dropped by Pypa projects, and we will support 2.7.

Coverage remained the same at 83.639% when pulling 1137ffb2ceebeb2360f79de615ca8a906be07521 on development into f0a3e28138e781b6132894d77d4a18d8228dc9a0 on master.


Launch Streetlives website at streetlives.org and streetlives.nyc

We have the URL at gandi
The website is on webflow and we can upgrade the tier to enable the bespoke URL. let me know when you're ready to deploy to streetlives.org and I'll upgrade it @jbeard4
The webflow instructions for custom domains is here
https://university.webflow.com/lesson/manually-connect-a-custom-domain?topics=hosting-code-export#how-to-verify-and-publish-your-site
Also connect streetlives.nyc, which currently re-directs to yourpeer.nyc

@adambard1 this is done,
have you forgotten about YouthPeer?


chore(main): release taiko-client 0.30.0

:robot: I have created a release beep boop
0.30.0 (2024-07-20)
Features

bridge-ui: release  (#17071) (2fa3ae0)
taiko-client: add --epoch.minTip flag (#17726) (a331e9d)
taiko-client: add defaultRequestTimeout for SGXProofProducer (#17724) (e301451)
taiko-client: add proof status check before generating proof (#17711) (9a8e15e)
taiko-client: allow hooks to be empty if prover and proposer are the same address (#17520) (2db6d2c)
taiko-client: build using cross-compilation (#17564) (e66a0c8)
taiko-client: bump Go contract bindings (#17068) (f1462c2)
taiko-client: call CreateAccessList (#17691) (9bb4b9c)
taiko-client: change guardian provers  to directly submit proofs instead of contests (#17069) (d669263)
taiko-client: disable hook in client (#17642) (099ce22)
taiko-client: fix a prover flag (#17315) (5806fcc)
taiko-client: improve ProverProvenByGuardianGauge metric (#17703) (c88fcd1)
taiko-client: improve chain head check in NeedReSync (#17431) (4dbe0af)
taiko-client: improve progress tracker (#17281) (5d05226)
taiko-client: improve some comments in prover (#17668) (e7afcfe)
taiko-client: integrate ProverSet in client (#17253) (f086850)
taiko-client: integrate new Raiko APIs (openAPI) (#17039) (e36d5b7)
taiko-client: integrate new Raiko APIs (openAPI) (#17254) (fea1cbe)
taiko-client: integrate SocialScan blob storage service (#17297) (0aaa6ce)
taiko-client: introduce --epoch.allowZeroInterval flag (#17727) (e76890d)
taiko-client: introduce AccessList (#17676) (3c95477)
taiko-client: introduce TryDecompressHekla() (#17735) (67a7a37)
taiko-client: make request timeout to raiko configurable (#17728) (f8f796c)
taiko-client: only invalid status need to return error (#17266) (e7111e0)
taiko-client: private key both with and without the hex prefix (#17098) (f80504a)
taiko-client: remove raiko from guardian prover (#17289) (bed6414)
taiko-client: remove an unused file (#17166) (5a9b0ec)
taiko-client: remove prover server package (#17748) (b064ea0)
taiko-client: remove useless flags about raiko (#17302) (7945142)
taiko-client: remove useless function and correct erorr handling (#17463) (4e93a57)
taiko-client: revert access list changes (#17694) (fd15dab)
taiko-client: revert the change of handling forkchoiceupdate status (#17288) (140aa25)
taiko-client: switching from blobstorage server to blobscan-api (#17244) (fbdb7c0)
taiko-client: temp log (#17324) (c96c6ee)
taiko-client: update bindings && fix tests (#17680) (10b95e1)
taiko-client: update bindings based on protocol TierRouter update (#17475) (0e9d160)
taiko-client: update Go contract bindings (#17086) (14ebeff)
taiko-client: update Go contract bindings (#17568) (f190919)
taiko-client: update Go contract bindings (#17733) (9d18504)
taiko-client: use taiko-geth params (#17044) (9e85e2f)

Bug Fixes

taiko-client: config.LProposerPrivKey loads an incorrect flag value (#17107) (8f69963)
taiko-client: avoid seting nil value to GuardianProverHeartbeater (#17802) (4076324)
taiko-client: fix a JWT parsing issue (#17325) (6c30f93)
taiko-client: fix a prover balance check issue (#17318) (f0b26b9)
taiko-client: fix an address check in AssignProver (#17526) (d6001d3)
taiko-client: fix an issue in BlockProposed event handler && remove some unused code (#17065) (5c4a025)
taiko-client: fix an issue in checkMinEthAndToken (#17320) (e007150)
taiko-client: fix an issue in CreateAssignment (#17323) (5ddb4c5)
taiko-client: fix an url path issue in BeaconClient (#17417) (012d532)
taiko-client: remove Go build cache (#17661) (f6075f7)
taiko-client: remove useless config in prover (#17335) (3c37890)

Chores

eventindexer,relayer,taiko-client: prep swagger apis to migrate (#17148) (357ecf2)
main: release taiko-client 0.23.0 (#17134) (d62f193)
main: release taiko-client 0.23.1 (#17418) (f6f32d8)
main: release taiko-client 0.24.0 (#17434) (e8fbfd4)
main: release taiko-client 0.25.0 (#17476) (c2da0b8)
main: release taiko-client 0.26.0 (#17522) (d907359)
main: release taiko-client 0.27.0 (#17567) (9119fdd)
main: release taiko-client 0.27.1 (#17663) (55bcc3d)
main: release taiko-client 0.28.0 (#17669) (a114ce9)
main: release taiko-client 0.29.0 (#17677) (fd52af1)
taiko-client: revert the failed cross-compilation (#17670) (0a1de79)

Documentation

taiko-client: fix an issue in comment (#17063) (49fdd52)

Code Refactoring

taiko-client: rm tier fee related flag (#17750) (b41437b)
taiko-client: rm unused L1_NODE_HTTP_ENDPOINT (#17768) (73c7aee)
taiko-client: rm unused code and refactor (#17723) (8c9a032)
taiko-client: rm unused tier fee (#17740) (3e139fa)

Tests

taiko-client: enable a skipped unit test (#17165) (92b657c)
taiko-client: fix tests (#17523) (085a127)

Workflow

repo,taiko-client: fix client build (#17040) (dbf4413)
taiko-client,repo: add proverserverapi workflows (#17057) (b90b932)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

taiko-client-v0.30.0
:sunflower:


Suhas Gumma

Name
Suhas Gumma
Discord Username (if applicable)
No response
Twiiter username (if applicable)
suhas_gumma
Additional Context
No response

Please accept the invite

Welcome to our community, please add your journey to Journey Book


Downgrade error severity to INFO when failing to add a listener

The error message may make users think their code is broken, when running in an IPv6-only environment.

#10187

@pmarks-net
Done. Clang-format is in charge of the format, but it's also happy with the new one.
Thanks for the review!

Known issues:
Basic Tests Linux: #9831
Basic Tests Mac: #9804
Basic Tests Windows: #9954


x-forwarded-port not updated in req headers during redirect

Hi,
I have a reverse proxy (HAProxy) running on port 80.
This will proxy the requests to express server running on port 4040 after setting X-Forwarded-Port header to 80 and it is reflecting in the express server when I did console.log(req.get('X-Forwarded-Port'));
From here I redirected the response to port 3939
res.redirect(301, 'http://<IP_ADDRESS>:3939');
Now the redirect is not carrying forward the request header X-Forwarded-Port.
I'm not sure if this is by design and we need to explicitly set the header or if this is an issue?

res.redirect issues a 302 to the browser, initiating a whole new request cycle from the client.  So it would never propagate a header set in the original request.
If you want express to act as a second reverse proxy you can do that by piping the req/res through, but that seems to be something different than you want.

Thanks. Got it.
Also I got to know how to use it as a second proxy from here.
http://stackoverflow.com/questions/18728039/express-req-pipe-does-not-work
Closing the issue. Thanks a lot for clarification.


docs(snippets): add docs to write custom snippets

Add docs to write custom snippets

Thanks again!


.env_created_/_settings.py_configured

.env created using django-environ,
settings.py changed as intended,
requirements.txt updated
MEDIA_URL added
TIME_ZONE set in .env


array subclasses doesn't allow kwargs

Related test: test_array::BaseTest::test_subclass_with_kwargs
As I understand,
array.__new__  allows kwargs
array.__init__  prohibit kwargs
But we only implements Constructor for PyArray which corresponds to __new__.
Adding Initializer for __init__ will be a starting point to solve this issue.

I will work on this issue!

Is this resolved?

Yes, this is done! Thank you for reminding!


[Request] Pull music information from Windows music controls

When using different sources such as spotify, groove, pandora, they all have different apps but they all use the windows media controls. If you could pull the song information from there you wouldn't have to select one input source, but instead could use multiple apps.

Marco's stance on this is that there has to be a plugin for this kind of stuff.
However when I last checked it is impossible to retrieve that information as there is no windows function for that and it can not be overloaded to send that info to rainmeter.
Everything I can think of except groove support though either a plugin exists for or will be added with my WebNowPlaying plugin though.

Thank you for the quick reply and information