[DEV] Not evolving any Pokemon

Expected Behavior
Bot should evolve Pokémon on startup.
Actual Behavior
He basically skips the evolution(no console log)
[10:04:56] Pokemon Captured: 1309 | Pokestops Visited: 547
[10:04:57] Pokemon Bag: 227/250
[10:04:57] Items: 321/350
[10:04:57] Stardust: 146604 | Pokecoins: 0
[10:04:57] PokeBalls: 48 | GreatBalls: 0 | UltraBalls: 32
[10:04:57] Razz Berries: 74
[10:04:57]
[10:04:57]
[10:04:59] Starting PokemonGo Bot....
[10:04:59] Scanning area for objects....
[10:04:59] Walking from (49.303157, 8.642888) to [49.303157, 8.642888] 0.00km
[========================================] 100%
[10:05:05] Cleaning up Pokemon Bag using the release config
[10:05:05] Pokemon Bag has been cleaned up!
[10:05:05] Using lucky egg ... you have 2
[10:05:06] Found fort 142f22cda073459ab55132cdc45190f0.16 at distance 0.02km
[10:05:06] Need to move closer to Pokestop
Steps to Reproduce
config:
"evolve_all": "all", not working
"evolve_all": "Pidgey,Zubat,Weedle" , does not work either
Other Information
OS:  Ubuntu 16.10
Git Commit: 97658283f37db827af940f5d74146346ccfcdf06

Same problem here

The same problem there is a solution?

There seems to be a bug in the _should_run() function. You can comment out the following two lines as a workaround
https://github.com/PokemonGoF/PokemonGo-Bot/blob/dev/pokemongo_bot/cell_workers/evolve_all_worker.py#L16#L17

This issue was caused by PR #1159 by the following addition to evolve_all_worker: 'skip_evolves = False
    if self.config.evolve_all:
        return skip_evolves'

this logic will return false every time the use_lucky_egg config is true. Causing the evolve to be skipped when a lucky egg is used.

@codybaldwin @sn0opy If I want to have a temporary work around which edit do I use?

as @sn0opy  said:  in pokemongo_bot\cell_workers\evolve_all_worker.py comment out line 16 and 17 by placing a # in front of them like so
'
def work(self):
#if not self._should_run():
#return
    self.api.get_inventory()
    response_dict = self.api.call()
    cache = {}

'

@codybaldwin @sn0opy Workaround is working for me it appears in latest DEV branch. Thank you for the help. Let me know if this ever gets fixed upstream.

will the workaround work for master?

Newest git pull   on the DEV branch and still experiencing the above bug.   Does the work around still apply?
Other Information
OS: Arch Linux 64bit
Git Commit: 15a034427d805e0bacb79fc0a1f3c96a6c93f0e3

Workaround mentioned above still works.
Other Information
OS: Arch Linux 64bit
Git Commit: 15a034427d805e0bacb79fc0a1f3c96a6c93f0e3

Submitted a PR here to fix this bug.
https://github.com/PokemonGoF/PokemonGo-Bot/pull/1256

PR has been opened to fix this issue. When you see this PR has been merged #1256 , update to latest dev branch. Closing this for now.
/closed


Files with an HTM or HTML extension are having the wrong mime type identified

When using the Lookup method, file names with an extension of HTM or HTML are being identified as application/xhtml+xml rather than text/html

After forking the current source, I see this problem has been fixed.  However, there is not currently an updated Nuget package with the latest source that has this problem resolved.


Not able to sign in with microsoft account after sign in with github

Type: Bug
After accidently signin in with a github account I'm no longer able to sign in with my microsoft account where are all my settings are synced.
Reproduction steps:

Fresh install
Click Profile icon in bottom left corner
Sign in to sync settings
Select Github
Complete sign in
Click profile icon
Select Github account
Select Sign Out
Click profile button
Tap Sign in to sync settings (1)
See that Browser opens and automatically signs you in with github. No option to select a different microsoft account

Expected result:
After signing out of GitHub account I should be able to select sign in with Microsoft again.
VS Code version: Code 1.79.2 (Universal) (695af097c7bd098fbf017ce3ac85e09bbc5dda06, 2023-06-14T08:58:52.392Z)
OS version: Darwin arm64 22.5.0
Modes:
Extensions: none
A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vsdfh931cf:30280410
vshan820:30294714
vstes263cf:30335440
vscoreces:30445986
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
py29gd2263:30776702
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
282f8724:30602487
pyind779:30671433
89544117:30613380
pythonsymbol12:30671437
showlangstatbar:30737416
vsctsb:30748421
pythonms35:30701012
03d35959:30757346
ecj1e332:30736112
pythonfmttext:30731395
pythoncmvfstrcf:30756944
fixshowwlkth:30771522
showindicator:30766890
pythongtdpath:30769146
e440d664:30776459
pythonidxptcf:30772540
pythondjangotscf:30772537
pythonnoceb:30776495

@sandy081 this seems more related to which account Settings Sync uses so I think this is you.

@sandy081 this seems more related to which account Settings Sync uses so I think this is you.

This is a VS Code bug. It should offer the 2 types of login, microsoft account or github after you log out but it remembers what the user choose and doesn't let it change after a logging out. I managed to replicate it as well.

This is expected and designed. If you want to use different account for settings sync, the recommended way is to turn off and turn on settings sync again - https://code.visualstudio.com/docs/editor/settings-sync#_switching-accounts

This is expected and designed. If you want to use different account for settings sync, the recommended way is to turn off and turn on settings sync again - https://code.visualstudio.com/docs/editor/settings-sync#_switching-accounts

I see. Thats super unintuitive imo. As a user I expect to be able to login with a different account after manually logging out.
If it cannot be changed. May I suggest adding a dialog prompt on logout asking if you wish to disable sync as well?

Thanks for the feedback, we are currently exploring this user experience.

/duplicate
https://github.com/microsoft/vscode/issues/104934


pasteboard plugins require windows developer mode opening

first, thanks your great work,
when i use pasteboard plugins, that flutter raise exception like this:
Exception: Building with plugins requires symlink support.

Please enable Developer Mode in your system settings. Run
  start ms-settings:developers
to open settings.

I want to know why this plugin require symlink support? thanks, because of some security issues, I must figure out it,

I want to know why this plugin require symlink support?

Because flutter_tool required it.
Flutter build app for windows with plugins required symlink support.
Not only this plugin, but other plugins will also require this option on. such as window_manager
if you want figure out it, here is the flutter tool source code, maybe you can find more things in this codes. https://github.com/flutter/flutter/blob/803ef6a456c3127a4c5509eb726f9319569edb3d/packages/flutter_tools/lib/src/flutter_plugins.dart#L1031

ok. thanks!


[mtl] rewrite handling of dynamic offsets and immutable samplers

Fixes dynamic offsets (once again).
I accidentally noticed that piece of Vulkan spec:

Values are taken from pDynamicOffsets in an order such that all entries for set N come before set N+1; within a set, entries are ordered by the binding numbers in the descriptor set layouts; and within a binding array, elements are in order. dynamicOffsetCount must equal the total number of dynamic descriptors in the sets being bound.

My previous code was written within an assumption that the user-given order of things define how both immutable samplers and dynamic buffers are handled. Apparently, the latter are always handled in the sorted order, which we already do (yay!), so with this (and a bit of extra reordering of immutable samplers) we are able to get rid of the associated data index (of descriptor layouts) and the corresponding indirection. This should make things simpler and even a tiny bit faster ;)
Note that the previous code also had another bug of not tracking the count of dynamic offsets per descriptor set, so we used the same set of offsets for all the descriptors.
PR checklist:

[ ] make succeeds (on *nix)
[x] make reftests succeeds
[x] tested examples with the following backends: metal
[ ] rustfmt run on changed code

I think the concerns are addressed. Follow-up is possible but not required, let's proceed.
bors r=grovesNL

I think your confusion is about what offset is being modified. It's not the one in descriptor pool, it's a local copy of it. So if no stages end up using it - no problem.

On Aug 3, 2018, at 01:20, Josh Groves<EMAIL_ADDRESS>wrote:
@grovesNL commented on this pull request.
In src/backend/metal/src/command.rs:

@@ -3088,6 +3084,8 @@ impl com::RawCommandBuffer for CommandBuffer {
}
target_offset.buffers += 1;
}

                   } else if layout.content.contains(native::DescriptorContent::DYNAMIC_BUFFER) {

                       dynamic_offset_iter.next();

Right, but for the bind_graphics_descriptor_sets it seems we don't check whether it's being used with VERTEX or FRAGMENT stages (like we do for bind_compute_descriptor_sets/COMPUTE) – we increase offset before checking these flags.
Or do we assume at least one of VERTEX or FRAGMENT flags must be set?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Yeah, that was exactly it – not sure how I missed it. This makes sense now, thanks!


Status codes as consts everywhere.

Pull Request Test Coverage Report for Build 99

5 of 6 (83.33%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 95.054%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/UI/Http/Rest/EventSubscriber/JsonBodyParserSubscriber.php
0
1
0.0%

Totals

Change from base Build 97:
-0.01%

Covered Lines:
442

Relevant Lines:
465

💛  - Coveralls

Nice one. Thanks


bugfix: convert to utf8mb4 if column is json type

[ ] I have registered the PR changes.

Ⅰ. Describe what this PR did

MySQL handles strings used in JSON context using the utf8mb4 character set and utf8mb4_bin collation. Strings in other character sets are converted to utf8mb4 as necessary. (For strings in the ascii or utf8mb3 character sets, no conversion is needed because ascii and utf8mb3 are subsets of utf8mb4.)

from https://dev.mysql.com/doc/refman/8.0/en/json.html
Ⅱ. Does this pull request fix one issue?

fixes https://github.com/apache/incubator-seata/issues/5107
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Please add changelog


Additional LDAP test cases:

Around 6 Test cases targeting this sprint.

[ ] LDAP Server Config with Invalid Base DN and Invalid Bind DN

[ ] LDAP Server Config with Invalid Base DN

[ ] LDAP Server Config with Invalid Bind DN

[ ]  LDAP Server Config with Invalid Bind DN Password

[ ] login with a user where user is not defined in the LDAP server and verify that login is not allowed

[ ] Create Privilege where it is already exist:

[ ] https://gerrit.openbmc-project.xyz/#/c/openbmc/openbmc-test-automation/+/25040/

IN review


LM model not found error when training

I downloaded the pre-processed data set and unzipped into the data directory.
I ran: !python -u src/run_kbc_subgraph.py --dataset conceptnet --evaluate-every 10 --n-layers 2 --graph-batch-size 60000 --sim_relations --bert_concat
And got this result:
2020-08-27 15:20:31.912536: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcudart.so.10.1
DGL backend not selected or invalid.  Assuming PyTorch for now.
Setting the default backend to "pytorch". You can change it in the ~/.dgl/config.json file or export the DGLBACKEND environment variable.  Valid options are: pytorch, mxnet, tensorflow (all lowercase)
Using backend: pytorch
Namespace(bert_concat=True, bert_mlp=False, bert_sum=False, cpu_decoding=False, dataset='conceptnet', debug=False, decoder='ConvTransE', decoder_batch_size=128, dropout=0.2, embedding_dim=200, eval_batch_size=500, eval_only=False, evaluate_every=10, feature_map_dropout=0.2, gcn_type='WGCNAttentionLayer', gpu=-1, grad_norm=1.0, graph_batch_size=60000, init_embedding_dim=200, input_dropout=0.2, input_layer='lookup', label_smoothing_epsilon=0.1, layer_norm=False, load_model=None, lr=0.0001, n_bases=100, n_epochs=200, n_hidden=200, n_layers=2, negative_sample=0, no_cuda=False, output_dir='saved_models', regularization=0.1, seed=42, sim_relations=True, sim_sim=False, tying=False, use_bias=False, write_results=False)
Number of edges: 99999
Graph Summary
Nodes: 78088
Edges: 100000
Relations: 34
Density: 0.000016
******************* Sample Edges *******************
ReceivesAction: hockey --> play on ice
IsA: hockey --> team sport
IsA: hockey --> violent sport
IsA: hockey --> game
IsA: hockey --> great sport
HasProperty: hockey --> violent
HasProperty: hockey --> cold
IsA: hockey --> type of game
IsA: hockey --> sport of skill and precision
IsA: hockey --> sport game

Average Degree:  1.254213195369327
Adding sim edges..
bert_model_embeddings/nodes-lm-conceptnet/conceptnet_bert_embeddings.pt
Downloading: 100% 232k/232k [00:00<00:00, 423kB/s]
Loading model from bert_model_embeddings/nodes-lm-conceptnet/
Traceback (most recent call last):
File "src/run_kbc_subgraph.py", line 532, in
main(args)
File "src/run_kbc_subgraph.py", line 115, in main
args.sim_relations)
File "src/run_kbc_subgraph.py", line 52, in load_data
train_network.add_sim_edges_bert()
File "/content/drive/My Drive/common_sense/commonsense-kg-completion/src/reader.py", line 82, in add_sim_edges_bert
bert_model = BertLayer(self.dataset)
File "/content/drive/My Drive/common_sense/commonsense-kg-completion/src/bert_feature_extractor.py", line 175, in init
self.bert_model = torch.load(output_model_file, map_location='cpu')
File "/usr/local/lib/python3.6/dist-packages/torch/serialization.py", line 571, in load
with _open_file_like(f, 'rb') as opened_file:
File "/usr/local/lib/python3.6/dist-packages/torch/serialization.py", line 229, in _open_file_like
return _open_file(name_or_buffer, mode)
File "/usr/local/lib/python3.6/dist-packages/torch/serialization.py", line 210, in init
super(_open_file, self).init(open(name, mode))
FileNotFoundError: [Errno 2] No such file or directory: 'bert_model_embeddings/nodes-lm-conceptnet/lm_pytorch_model.bin'

Please download the embeddings from here and unzip in the root directory of the repo.

Thank you


Add id custom type and migration generator

This PR adds ability to change translatable_ids type on Ecto migrations and schema
Notes:

There is no API changed whatsoever so a patch bump will be ok in case of merging.
The README is updated with new instructions on migration generator.
The README is updated with instructions about changing translatable_ids type.

Cheers

Coverage decreased (-29.8%) to 52.0% when pulling 67ffa7f1678acb93ba33964a288ed0d862d4d799 on alisinabh:custom_type_sq into c6eb1c787a0fdadacb9af8f013050d0ea52d1f9b on smeevil:master.

Thanks a lot for these changes :)
If you have time, could you see if you can up the coverage again ?
For the meantime I'll merge this in and bump the hex.
Gerard.

@smeevil Thank you very much for creation of this library.
And yes i believe i can look at that in this week.
My pleasure
:heart: :blue_heart: :purple_heart: :yellow_heart:


ARDUINO_ARCH_ESP32 forcing ESP32 internal CAN controller - Need MC2515

How do I use this library on an ESP32 (wroom-E32) with an MCP2515 external CAN controller.  I am doing this project in PlatformIO with the Arduino framework. When I include CAN.h it checks ARDUINO_ARCH_ESP32 and selects the internal controller as ESP32SJA1000.h .  If ARDUINO_ARCH_ESP32 is not defined (somehow) then MCP2515.h is included.  This is the code in CAN.h:
#ifdef ARDUINO_ARCH_ESP32
#include "ESP32SJA1000.h"
#else
#include "MCP2515.h"
#endif

How do I get this to properly include MCP2515.h?  I don't want to modify the CAN.h or any other part of the library unless this is the only way.

you could try this
https://github.com/Unicorn-Biotechnologies/arduino-CAN


cmd/go: "go get github.com\alexbrainman\gowingui" should fail

What version of Go are you using (go version)?
go version go1.10 windows/amd64
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\Alex\AppData\Local\go-build
set GOEXE=.exe
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=c:\users\alex\dev
set GORACE=
set GOROOT=c:\users\alex\dev\go
set GOTMPDIR=
set GOTOOLDIR=c:\users\alex\dev\go\pkg\tool\windows_amd64
set GCCGO=gccgo
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0 -fdebug-prefix-map=C:\Users\Alex\AppData\Local\Temp\go-build838310042=/tmp/go-build -gno-record-gcc-switches

What did you do?
I run
go get github.com\alexbrainman\gowingui

command.
What did you expect to see?
I expected command to fail, because I used \ instead of / in package import path.
What did you see instead?
I see no error message, and I also see gowingui.exe in my %GOPATH%\bin directory.
Comments
I discovered this while investigating #24232
And if I run
go install -ldflags "-H windowsgui" github.com\alexbrainman\gowingui

the %GOPATH%\bin\gowingui.exe file does not have IMAGE_SUBSYSTEM_WINDOWS_GUI flag set properly.
So if this issue is rejected as working as intended, than we should fix cmd/go flag processing to avoid cases like #24232
Alex

on darwin:
thorn@t2 ~/go % go get github.com\alexbrainman\gowingui
package github.comalexbrainmangowingui: unrecognized import path "github.comalexbrainmangowingui" (https fetch: Get https://github.comalexbrainmangowingui?go-get=1: dial tcp: lookup github.comalexbrainmangowingui: no such host)

Windows:
C:\Users\thorn>go get -v github.com\alexbrainman\gowingui
github.com/alexbrainman/gowingui (download)

Mac:
thorn@t2 ...go/src/playground % go get -v 'gitfhub.com\alexbrainman\gowingui'
https fetch failed: parse https://gitfhub.com\alexbrainman\gowingui: invalid character "\\" in host name

This happens in ImportPathsNoDotExpansion(args []string) []string function
https://github.com/golang/go/blob/master/src/cmd/go/internal/load/search.go#L360-L362
which is called by downloadPaths: https://github.com/golang/go/blob/master/src/cmd/go/internal/get/get.go#L172
On windows go get silently fixes \ into / which is the right thing, but as a side effect, it applies the same fix to URLs. Which again could be a good thing because Chrome browser does exactly the same thing.
There is just inconsistency in behaviour between OSes with fileseparator = \ and other Oses.

I'm pretty sure we should fix this, but marking it as needs decision just in case. @ianlancetaylor, @rsc?

Well, yes, what I meant that inconsistency should be fixed for sure as it is confusing. But I was not sure that we should fix it that Windows invariant would produce an error.

I agree.

The fact that the backslashes work on Windows is intentional:
https://github.com/golang/go/blob/5a7cfbc0117bce314c3f079ece459173b9efc854/src/cmd/go/internal/search/search.go#L352-L357
We should figure out why they don't work on macOS.


Create nl_NL.json

Added dutch localisation

seems really early to be working on localization, but it does mean i have more testing material for configs


False negative on Struts 2.5.12

Just installed Struts 2.5.12 on Tomcat 9 as a test target running locally
http://<IP_ADDRESS>:8080/struts2-showcase/index.action shows the 'Welcome!' screen
$ ./struts-pwn.py -u 'http://<IP_ADDRESS>:8080/struts2-showcase/index.action'

[*] URL: http://<IP_ADDRESS>:8080/struts2-showcase/index.action
[*] Status: Not Affected.

Same result with any URL (even non-existent ones)

Hi @hubert3,
Apache Struts v2.5.12 is not vulnerable to CVE-2017-5638. This is why you have received the "Not Affected" status message when testing your local machine.
Best,
Mazin


Can we drop support for legacy Python

Can we limit support to https://devguide.python.org/#status-of-python-branches
and drop support for https://devguide.python.org/devcycle/#end-of-life-branches ?

Not that I have any clout around here, but I definitely agree FWIW.

Given that Python 2 died 1103 days ago on 1/1/2020 and there are almost no Py2 downloads, let's drop legacy Python.


feat: Add additional context info when throw exception

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 18 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 86.586%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
4193

Relevant Lines:
4691

💛  - Coveralls


Not compatible with webview_flutter 1.0.0

Trying to use both html_editor and webview_flutter fails in my project with the following error:
Running "flutter pub get" in enough_mail_app...
Because every version of html_editor depends on webview_flutter ^0.3.21 and enough_mail_app depends on webview_flutter ^1.0.0, html_editor is forbidden.

So, because enough_mail_app depends on html_editor ^1.0.1, version solving failed.

As the 1.0.0 release of webview_flutter is currently the only stable release, it would be great if that is supported by html_editor, too.

@robert-virkus facing the same issue

Same issue here,
and _keyEditor.currentState.getText() dont work in ios, only in Android works.


No test for fs.chmod when the path is unvalued

I was reading the code of fs.chomd, and there is no test for the case when people passed the unvalued path.
I would like to work on this, if that ok.

Go for it @yuzhouChen


Changing a button text based on a state variable

I need to change text of a button base on the state of a variable in my keyboard. The way I am thinking to do this is to override the buttonText function in Appearance class, like what the demo keyboard does. Where should I  put the state variable? The KeyboardAppearance class does not have access to the main KeyboardViewController instance.  The keyboard appearance has the type KeyboardAppearance. Any variable I define in the subclass won't be accessible in the overridden function.
The only way I can think of is to add a reference to an KeyboardViewController instance in the KeyboardAppearance class by modifying the library.
I am thinking about this correctly? Is there a way to do that without modifying the library?
Thanks!

Hi @ctawong
Is the variable you want to set is a boolean value? if yes, you can do so by setting a variable in UserDefaults

Thanks @mrhaxic. That's certainly one way to do it.  I just want to make sure I am not missing any existing mechanism.
Turned out I added reference to the KeyboardViewController instance through init. That seems overkill for reading one variable...
Looks like appearance is supposed to only rely on context. Wish we would be able to define a custom context.

You can subclass StandardKeyboardAppearance and inject your custom appearance as is done in the demo.
You can also create your own observable context and inject it into the view hierarchy if you want the views to automatically adapt depending on the value of the text.
Just because there is one single keyboard context doesn't mean you can't create your own contexts. You can leverage the full power of SwiftUI alongside KeyboardKit.

I like the idea of custom observable context. Thanks!


2.0.0 pod namespace env

Addresses #249
Resets the injected KH_POD_NAMESPACE env var.

In the future, we may want to consider a refactor of this in such a way that a single func can be called to both purge and set the proper environment variables.
Then, we wouldn't have to maintain a list of vars that we purge - we just loop over all the ones we want to add and the purge is automatic based on the var being enforced upon the pod spec.


add import bulk tuples

PR Template:
Describe your changes

Add Bulk Import tuples

grpcurl -plaintext -d @ localhost:9000 kessel.relations.v1beta1.KesselTupleService.ImportBulkTuples <<EOM
  {
  "tuples": [
     {
    "resource": {
      "type": {
        "namespace": "rbac",
        "name": "group"
      },
      "id": "bobclub"
    },
    "relation": "member",
    "subject": {
      "subject": {
        "type": {
          "namespace": "rbac",
          "name": "user"
        },
        "id": "bob"
      }
    }
  }
  ]
}
EOM

response
{
  "numImported": "1"
}

...

Ticket reference (if applicable)
Fixes #
Checklist

[ ] Are the agreed upon acceptance criteria fulfilled?

[ ] Was the 4-eye-principle applied? (async PR review, pairing, ensembling)

[ ] Do your changes have passing automated tests and sufficient observability?

[ ] Are the work steps you introduced repeatable by others, either through automation or documentation?

[ ] If automation is possible but not done due to other constraints, a ticket to the tech debt sprint is added
[ ] An SOP (Standard Operating Procedure) was created

[ ] The Changes were automatically built, tested, and  - if needed, behind a feature flag - deployed to our production environment. (Please check this when the new deployment is done and you could verify it.)

[ ] Are the agreed upon coding/architectural practices applied?

[ ] Are security needs fullfilled? (e.g. no internal URL)

[ ] Is the corresponding Ticket in the right state? (should be on "review" now, put to done when this change made it to production)

[ ] For changes to the public API / code dependencies: Was the whole team (or a sufficient amount of ppl) able to review?

LGTM

/lgtm - @akoserwal dont mind the failing github action of IQE - feel free to merge it


New version: NLPModelsKnitro v0.4.0

Registering package: NLPModelsKnitro
Repository: https://github.com/JuliaSmoothOptimizers/NLPModelsKnitro.jl
Created by: @dpo
Version: v0.4.0
Commit: 6cee855226713f9af1edbde9bb943d5abfcc7354
Reviewed by: @dpo
Reference: https://github.com/JuliaSmoothOptimizers/NLPModelsKnitro.jl/commit/6cee855226713f9af1edbde9bb943d5abfcc7354#commitcomment-40468038

[noblock] NB: This package depends on a commercial library.


infra: onboard to use ruff

Issue #, if available:
Description of changes:
An example of how to onboard to ruff with most rules enabled.
For use when upgrading to Python 3.10
Testing done:
Merge Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.
General

[ ] I have read the CONTRIBUTING doc
[ ] I used the PR title format described in CONTRIBUTING
[ ] I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

[ ] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ ] I have checked that my tests are not configured for a specific region or account (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Can we keep the the 3.10-style typing that we have in files with from __future__ import annotations? This change seems to undo them all.


If website doesn't allow previewing image, it won't show up in embed

Describe the bug
When linking to a website, if the url given by the website initiates a download instead of previewing the image (as happens with most S3 providers), Bluesky will not preview the image in the embed
e.g.
https://untonemusic.com/img/release/my-style/cover.jpg

https://bsky.app/profile/tanza.me/post/3l26d37525j2t
To Reproduce
Steps to reproduce the behavior:

Post a link to a website with the above behaviour (e.g. https://untonemusic.com/release/my-style)

Details
Happens on android and web, haven't tested iOS but im assuming it's serverside

I can confirm this on iOS too.
Someone from the Bluesky team will be able to give a proper answer, but in the meantime I noticed two things.
Firstly, if you use a website like this it shows that the Facebook and LinkedIn cards showing up fine but not the Twitter one. It complains: Oh no – there’s no Share Image defined! and also The ratio of your twitter:image isn't optimal. I don't know if these errors have any relevance here but I thought they might be worth mentioning.
Also, the output from cardyb (which AIUI generates the link cards on Bluesky, seems to find the image URL OK:
{"error":"","likely_type":"html","url":"https://untonemusic.com/release/my-style","title":"UNTONE Music / Release / My Style","description":"Released Friday, 16th Aug 2024","image":"https://cardyb.bsky.app/v1/image?url=https%3A%2F%2Funtonemusic.com%2Fimg%2Frelease%2Fmy-style%2Fcover.jpg"}

but then is unable to serve the image:
{"error":"Unable to serve image from URL","likely_type":"","url":"","title":"","description":"","image":""}


Implement connector to Debugger @daveleroy

This is the connector for  daveleroy/sublime_debugger#106

Should the command be also added to the Command Palette?

Should the command be also added to the Command Palette?

No. This command is called by Debugger (the ST Plugin) to enable communication between the 3.8 and 3.3 plugin host.


Add default reviewers in CODEOWNERS

Description

Next in the series of improving Luigi's review process is auto-assign reviewers based on "code owners". (As mentioned in #2463).
To begin with, I've added 3 maintainers to be auto-assigned to all and a specific example to assign all redshift reviews to me. More can be added as time goes on.
Motivation and Context

See #2463
Have you tested this? If so, how?

Supplementary file - tests not needed

@Tarrasch @ulzha Do we want to start out with any other default codeowners? Or perhaps adding other Spotify employees to the * list?

If no one comments to request modifications, I will merge this on 2018-07-25


[Distro Request] Add Void Linux as supported distro.

Hello.
I have created build scripts for xbps-src (Void Linux Build Tool) and wanted to share them, because as of right now I don't see any official packages for Void.
template (main build script)
# Template file for 'swaylock-effects'
pkgname=swaylock-effects
version=1.6.11
revision=1
archs="x86_64"
build_style=meson
conf_files="/etc/pam.d/swaylock"
hostmakedepends="pkg-config wayland-devel scdoc git"
makedepends="wayland-protocols cairo-devel gdk-pixbuf-devel pam-devel libxkbcommon-devel wayland-devel libgomp-devel"
checkdepends="wayland libxkbcommon cairo"
short_desc="Swaylock, with fancy effects"
replaces=swaylock>=0
maintainer="<>"
license="MIT"
homepage="https://github.com/jirutka/swaylock-effects"
distfiles="${homepage}/archive/refs/tags/v${version}.tar.gz"
checksum=44804d1d9dff189d1b113e44b75f68eb9fc6cf76eb4638658496ad50903cb7b9
wrksrc=${pkgname}-${version}-${revision}

post_install() {
	vlicense LICENSE
}

NOTE: For now I have only added x86_64 architecture, because it's only architecture I can build and test on (my vm doesn't work properly with wayland so I am testing on spare laptop).
If there's anything more to do let me know. I would really like to bring your fork to Void Linux, or at least make it easier for people to install it.

Please open a pull request in void-linux/void-packages, they will (hopefully) provide you a code review.

Sorry for necroposting, but @music-cat-bread see this and linked issues as you're not the first to attempt packaging this and its upsteam. Also see the contributing guidelines and the manual if you do want to try contributing.


JSONDecodeError("Expecting value", s, err.value) from None

Hey guys, first time posting here.
Well, I've running gdown for a couple of weeks now, but from this afternoon I'm getting this error which, I'm guessing, It is a fault in gdown because the code I'm running now, is exactly the same as of a couple of hours ago, nothing has changed, even the sample code from Kentaro trows this error, does anyone knows the solution for it?
Thanks all,
Python code:
import gdown
url = 'https://drive.google.com/uc?id=0B9P1L--7Wd2vNm9zMTJWOGxobkU'
output = '20150428_collected_images.tgz'
gdown.download(url, output, quiet=False)
ERROR: >>>>
Traceback (most recent call last):
File "C:/Temp/xxx.py", line 5, in
gdown.download(url, output, quiet=False)
File "C:\Users\Marcio\AppData\Local\Programs\Python\Python37\lib\site-packages\gdown\download.py", line 97, in download
cookies = json.load(f)
File "C:\Users\Marcio\AppData\Local\Programs\Python\Python37\lib\json_init_.py", line 296, in load
parse_constant=parse_constant, object_pairs_hook=object_pairs_hook, **kw)
File "C:\Users\Marcio\AppData\Local\Programs\Python\Python37\lib\json_init_.py", line 348, in loads
return _default_decoder.decode(s)
File "C:\Users\Marcio\AppData\Local\Programs\Python\Python37\lib\json\decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "C:\Users\Marcio\AppData\Local\Programs\Python\Python37\lib\json\decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

That's probably because of the cache, so please try below.
rm -f ~/.cache/gdown/cookies.json

Hi Kentaro!
Spot on, problem solved!! Thanks so much for taking your time to reply!
So being a bit bold, would you add a function to get the downloading File size?  ;-)
And congrats for the Gdown!
All the best mate!

Great to hear.
I'm not sure if that's possible to get file size with GET request. Do you know that's possible?


lodestar-beacon-state-transition comment updates

part of the effort documented in #1775
most of the comments that were added were pulled from either the eth2spec, eth2fastspec (https://github.com/protolambda/eth2fastspec), or Vitalik's annotated spec (https://github.com/ethereum/annotated-spec)

If anyone knows of any documents that further clarify the eth2fastspec design, might be helpful to put that as a link in the index.ts of the fast folder or something like that.


Avs 0.11.0 support

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 86.52%. Comparing base (b7de05e) to head (528ce59).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #17      +/-   ##
==========================================
+ Coverage   86.24%   86.52%   +0.27%
==========================================
  Files          41       41
  Lines        2777     2834      +57
==========================================
+ Hits         2395     2452      +57
  Misses        289      289
  Partials       93       93

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Provide Dutch translation of button and dialog texts

(Potentially this could involve a lot of work, so for now this has a low priority).

Check out the gettext module:
https://inventwithpython.com/blog/2014/12/20/translate-your-python-3-program-with-the-gettext-module/


Fix bug CTAS with multi partition desc

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #5500
Problem Summary(Required) ：

This because interval is expr and can not cast to Long
then catch exception use old got an strange error
we should log some error for new parser

[FE PR Coverage check]
:disappointed: fail : 4 / 7 (57.14%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/analysis/MultiRangePartitionDesc.java
0
1
00.00%

:large_blue_circle:
com/starrocks/qe/ConnectProcessor.java
0
1
00.00%

:large_blue_circle:
com/starrocks/sql/parser/AstBuilder.java
4
5
80.00%

[FE PR Coverage check]
:disappointed: fail : 4 / 6 (66.67%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/analysis/MultiRangePartitionDesc.java
0
1
00.00%

:large_blue_circle:
com/starrocks/sql/parser/AstBuilder.java
4
5
80.00%

[FE PR Coverage check]
:disappointed: fail : 4 / 6 (66.67%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/analysis/MultiRangePartitionDesc.java
0
1
00.00%

:large_blue_circle:
com/starrocks/sql/parser/AstBuilder.java
4
5
80.00%


[RFC] Update chapter 3

Branched from #46.
Remarks on the problem definition section:

Are there any more questions we should/could add?

Remarks on the problem analysis section:

In the section about the execution model, we (sh|c)ould mention just trying different start symbols.
Maybe we should remove the paragraph about detecting unfinished expressions, as it has never been a problem: the console input editor works well as-is, and in Eclipse a multiline input editor is always available.
Are there more problems to discuss, perhaps?

Remarks on the requirements analysis section:

Please see the FIXMEs and TODOs in the tex file.
The requirements refer to literate programming, which is explained in chapter 7. That's possibly kind of confusing -- what to do? Literate programming has been moved to chapter 2 in #48.
Does the minimal viable product require an explanation as to why only the must haves and, more importantly perhaps, why only the first two design goals?
This section is the only section that still mentions "all languages", which Hendrik had comments on: In final report, be more clear about 'work with all languages', since this can be interpreted as 'work without extra effort ...'. I believe I removed that assumption, but please verify.

Perhaps the problem analysis section should discuss the questions defined in the problem definition section one by one, instead of what is there currently.
If there currently is a section in the problem analysis that does not map to such a question, then we either missed a question or it hasn't really been a problem for us.

This section looks good to me now.

I just noticed that the first two design goals have changed. Both are very similar now, and the requirement that the REPL should work for any language (which is, admittedly, obvious) is now nowhere to be found. Related to this is the second must have requirement.

LGTM.