no module named kivy.garden.graph

Hey i've install this module using pip and it seems to install correctly in my .kivy folder but when i try running the example it says no module named kivy.garden.graph. I use the send to method with kivy.

We need a full traceback and example code to be able to resolve that. Please reopen with that information.


Add landing screen

When starting the application for the first time it's just blank (except the sidebar). We should add some kind of simple landing screen that just welcomes the user.

My initial thought was to ceate a settings page with different tabs. The first tab would be the landing screen with some kind of introduction (e.g. info, changelog, proton blog updates for).

I think the optimal solution would be to have something user friendly. Like "Hi and welcome to ProtonMail Desktop" and then a short intro and an arrow pointing to the "Add account" button. Just to make it crystal clear for the user how he should start using the application.
But it's a question of time. I think your approach works for now.

Mmm, that is an interesting idea. What do you think about a simple page with the logo, the add account button, Intro | Settings | About links (small at the bottom). Plain background, minimalist style.

I think it would be enough with the logo and add account button. Regarding the links you proposed I don't think Intro is necessary, and the other two can the user reach via the application menus. What do you think about that?
I'm not a designer but I can see if I can find someone at Fiverr to create a good looking (and minimalistic) landing page.

The simplest design as you suggest is the way to go.
I am not a designer either but I have some basic knowledge and I will be able to do it without problems.

#132 implemented this feature.


Typing a message without sending, then choosing an emoji will discard the text

In many cases you would want an emoji as part of a text. However, when trying to do so, any message you've already typed instead disappears, and only the emoji is sent immediately.
I'd like to propose the following:
Instead of directly sending the emoji after clicking it, instead insert it into the text area. Then the user can do what they want; send it together with a text, or send only the emoji if they want.

I see now that there's already a PR for this, https://github.com/mattmezza/vue-beautiful-chat/pull/202


Can i use vim-ai chat with local ollama models ?

       Hi, Can i use vim-ai chat with local ollama models ?  if can , how to config?

Hi, check my config file here.


[Fix][Connector-V2] Fix Hive fails to load *-site.xml file in hive.hadoop.conf-path

Using all site.xml from hadoop hive config folder
Created HiveConf from Hadoop conf

Purpose of this pull request

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist
Add ci label in label-scope-conf
Add e2e testcase in seatunnel-e2e
Update connector plugin_config

[ ] Update the release-note.

Kerberos name No rule issue is fixed with patch but there are som other issues in overall flow Hive connector flow. I will fix and update the PR.


After upgrading to NuGet package 1.3.18. from 1.2.155, DataTablesResult disappeared

The C# backend code doesn't compile any more.
Old packages.config:
  <package id="Mvc.JQuery.Datatables.Templates" version="1.2.155" targetFramework="net45" />```
New:
```  <package id="Mvc.JQuery.Datatables" version="1.3.18" targetFramework="net45" />
  <package id="Mvc.JQuery.Datatables.Core" version="1.3.14" targetFramework="net45" />
  <package id="Mvc.JQuery.Datatables.Templates" version="1.3.18" targetFramework="net45" />```

Example code which doesn't compile:
return DataTablesResult.Create(calculationChecks.AsQueryable(), dataTableParam);

Can you try again with 1.3.20 - there were some duff builds which got partially pushed up. The example project works now, so that code should work too. Reopen if you have further difficulties.

I upgraded to 1.3.22 but the DataTablesResult class is still gone. Maybe I'm following a wrong patter?

Oh, now it's in Mvc.JQuery.DataTables instead of Mvc.JQuery.Datatables;

Yep, sorry, the mismatch in capitalization caused all sorts of mayhem,
On 31 August 2015 at 17:30, Csaba Toth<EMAIL_ADDRESS>wrote:

Oh, now it's in Mvc.JQuery.DataTables instead of Mvc.JQuery.Datatables;
—
Reply to this email directly or view it on GitHub
https://github.com/mcintyre321/mvc.jquery.datatables/issues/129#issuecomment-136421754
.


Take local water supply into account for allocation

Currently allocation only uses source capacities to fulfill the demands of users. However, (a part of) the demand of users could be supplied by local sources (e.g. precipitation, water readily available in the basin a user is subtracting from).
An approach for the 'vertical fluxes' on a basin could be to explicitly add these as a source/sink in the max flow problem.

Closed in favor of https://github.com/Deltares/Ribasim/issues/564.


Correct link for tailwind css

Corrects the fragment for the tailwind css docs, from #tailwind to #tailwind-css

Hey @TomVance and thank you for this fix 🙌🏼
Can you add yourself to the contributors list in this PR? Make sure to use TomVance instead of tomvance as this is case sensitive.
Let me know if you need assistance with that

Hey @TomVance and thank you for this fix 🙌🏼
Can you add yourself to the contributors list in this PR? Make sure to use TomVance instead of tomvance as this is case sensitive.
Let me know if you need assistance with that

Makes sense! I for sure should be drinking coffee before opening upstream changes 😂


Remove image derivative method?

Do we need the image_derivatives method in GeoConcerns::FileSet::Derivatives? Seems redundant and could cause conflicts with overrides in host applications.
https://github.com/projecthydra-labs/geo_concerns/blob/master/app/models/concerns/geo_concerns/file_set/derivatives.rb#L21-L28

@drh-stanford Can you think of any reason why we would need to process scanned map / image derivatives in a special way?

is that code overriding regular hydra-derivatives? i don't think we need that if so...

@drh-stanford Yeah, we're just overriding hydra-derivatives. PR to fix this: https://github.com/projecthydra-labs/geo_concerns/pull/266


Select JDK when creating the project

Give us an option to create every time a project with different JDK versions in the "File->New project" menu in the

screen with the name "JDK version" and then you can select the JDK which you want to use from the installed JDKs.
That would be really helpful in creating a new project.
Thank you.

First this repo is the repo of the website of NetBeans, not of the IDE. Second you created a pull request instead of an issue. Issues will be handled in JIRA for Apache, not on GitHub unfortunately.

@PhilippBaumi the Apache NetBeans IDE Code repo is here: https://github.com/apache/incubator-netbeans and the JIRA url is this: https://issues.apache.org/jira/projects/NETBEANS


[SR-4190] Text position and span tests

Previous ID
SR-4190

Radar
None

Original Reporter
@bitjammer

Type
Bug

Additional Detail from JIRA

Votes
0

Component/s
Compiler

Labels
Bug, Syntax, Test

Assignee
@bitjammer

Priority
Medium

md5: f36e531d397ed4947aa333f609d8d625

Issue Description:
Source file -> parse -> lib/AST -> transform -> lib/Syntax -> Syntax/ASTNode map
For each entry in the map, check that { source file, offset, line, column } are the same for each pair.

libSyntax was removed in #62145 and succeeded by swift-syntax.


No message shown on empty text preview click

Describe the bug
There is no toast message when there is empty text and click on preview button.
To Reproduce
Click on preview button when no text written in edittext.
Expected behavior
There should be a toast message like "Write text for preview"
Smartphone Info:

Device
Redmi 5A

Android Version
7.1.2

Would you like to work on the issue?

Yes.

Duplicate of #65. I am already working on it.

@yashk2000 go ahead bro :)


Use proper format for WP-CLI's add_command

See https://github.com/wp-cli/wp-cli/blob/d7df11b43e9ece6990ac42371aadac2cbd9d6554/php/class-wp-cli.php#L450
and https://github.com/szepeviktor/phpstan-wordpress/blob/6195a6a19830d5ad187df0a3fb350e77cd571a5f/example/phpstan-full.neon#L38-L39
From https://github.com/leonstafford/static-html-output/commit/ba9fa5b27a31d9a67c1aeb320b2070c59c9fc7c7

@leonstafford Hm??


Internal Server Error in Teams App Service after creating new Client Secret

Steps to reproduce
We use Teams Toolkit for VSCode for developing/provision/deployment of our Teams Bot App.
The client secret under the Teams App Registration expired recently. We created a new client secret under the App Registration; However, an internal server error still happens in the App Service, and the Teams App (Bot App) is not working.
We've also tried the following:

Updated the client secret (SECRET_BOT_PASSWORD under .env.production.user) and publishing new version of the app, but that did not work.
Added new OAuth Connection under Bot Services > [Teams Bot App Name] > Configuration blade (Azure Active Directory as Service Provider). This did not work either.

Expected behavior
The bot should work as expected - responding back to messages with text or adaptive cards, just like before the client secret expired.
Actual behavior
After creating new client secret and publishing a new version of the teams app with the latest client secret, the app service still runs into internal server error whenever sending a message from the teams app, thus resulting in no response from the bot.
Error details
ServerError: invalid_client:

7000215 - [2024-09-16 08:49:39Z]: AADSTS
7000215: Invalid client secret provided. Ensure the secret being sent in the request is the client secret value, not the client secret ID, for a secret added to app '97xxxxb0'.

Trace ID: 40xxxx00
Correlation ID: 12xxxx9a

Timestamp: 2024-09-16 08:49:39Z
- Correlation ID: 12xxxx9a
- Trace ID: 40xxxx00

  at ServerError.AuthError [as constructor] (C:\home\site\wwwroot\node_modules\@azure\msal-node\node_modules\@azure\msal-common\dist\index.cjs.js:499:24)
  at new ServerError (C:\home\site\wwwroot\node_modules\@azure\msal-node\node_modules\@azure\msal-common\dist\index.cjs.js:3407:28)
  at ResponseHandler.validateTokenResponse (C:\home\site\wwwroot\node_modules\@azure\msal-node\node_modules\@azure\msal-common\dist\index.cjs.js:5442:19)
  at ClientCredentialClient.<anonymous> (C:\home\site\wwwroot\node_modules\@azure\msal-node\node_modules\@azure\msal-common\dist\index.cjs.js:6803:41)
  at step (C:\home\site\wwwroot\node_modules\@azure\msal-node\node_modules\@azure\msal-common\dist\index.cjs.js:79:23)
  at Object.next (C:\home\site\wwwroot\node_modules\@azure\msal-node\node_modules\@azure\msal-common\dist\index.cjs.js:60:53)
  at fulfilled (C:\home\site\wwwroot\node_modules\@azure\msal-node\node_modules\@azure\msal-common\dist\index.cjs.js:50:58)
  at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
{errorCode: 'invalid_client',errorMessage: "7000215 - [2024-09-16 08:49:39Z]: AADSTS7000215: Invalid client secret provided. Ensure the secret being sent in the request is the client secret value, not the client secret ID, for a secret added to app '97xxxxb0'. Trace ID: 40xxxx00 Correlation ID: 12xxxx9a Timestamp: 2024-09-16 08:49:39Z - Correlation ID: 12xxxx9a - Trace ID: 40xxxx00",subError: ''}

Thank you for reporting this, we will check this and get back to you.

Hello @roshankumar-UP - Looks like you are passing Secret ID but not the value.

Please check your configuration and update it once.
Please let us know if you still facing this issue.

Hello @ChetanSharma-msft,
We used the secret value. After the old one expired, we created a new client secret. We then copied the secret value since it's only visible for once after creation, and used it for SECRET_BOT_PASSWORD value in .env.production.user file in our teams app repository. Even then, it gave the same error.
For now, we provisioned new resources and changed the BOT_ID, BOT_DOMAIN and BOT_AZURE_APP_SERVICE_RESOURCE_ID, while keeping the old TEAMS_APP_ID and TEAMS_APP_PUBLISHED_APP_ID so that we can use the existing published app with the new provisioned resources.
But because of this, the teams app has been down for four working days and we're still waiting on validation; and if I am correct, this approach also means user's chat history will be lost, and we'll eventually run into this issue once the client secret expires again.
Can you let us know if this is an issue on MSFT Teams end, and if/when it will be resolved? or did our approach of changing the client secret had issues?

Hello @roshankumar-UP - As discussed over call, this issue is fixed after updating bot secret in App service configuration.
Please let us know if you need further assistance.
Please share your valuable feedback here.

Hello @ChetanSharma-msft - very thankful to you for spending your time debugging this issue to the end, and providing an answer as to how to tackle this if it happens again. For anyone searching for this issue, the solution was to go to App Service >  [App Service Name] > Environment Variables > update the BOT_PASSWORD with the new client secret.
One recommendation that I can provide, is that although the issue in itself is a single environment variable change, we failed to identify it on our own because we failed to find correlation between the Client Secret used by the Scaffolded Teams app, and the Bot Password of the App Service involved...
Maybe this was pointed out in documentation and we missed it; but it also means someone else will miss it in the future as well. We hope the documentation for scaffolded teams bot app, and teams apps in general, gets improvements and include highlighted points for issues like this, which have a potential to cause production apps to go down.


Publish generated HTML in the client

Fixes https://github.com/dotnet/aspnetcore/issues/14286

Refactored GeneratedCodeContainer and friends to include two output versions instead of one (CSharp and HTML)
Added an updateHtmlBuffer command to update the buffer on the client
Updated RazorDocumentManager and HtmlDocument on the client side to follow the same behavior of CSharp for updating the virtual background document
Fixed a bug in UnsyncronizedDocumentProcessedListener where we were updating the background C# document with all razor content
Updated tests

🆙 📅


Coinbase transactions are dead

A coinbase transaction is displayed as dead in some cases, even with multiple confirmations. Check faucet database for more information.

Fixed by #314


Create community page for VDA5050-related resources

It would be great if there were a community wiki or something like it, where resources relevant to this standard could be shared. For example, so far during my work with the specification I've got several things that might come in handy to others:

a stricter JSON schema
a VDA 5050 payload visualizer, for easier debugging and replicating scenarios

We will add a wiki with community ressources and link your projects there. Thank you for your input!


Add codeowners file and action for PR merging

TODO: Review code in https://github.com/javalin/codeowner-selfmerge-action before merging


[NTR] bronchial-associated lymphoid tissue lymphocyte

Preferred term label:
bronchial-associated lymphoid tissue lymphocyte
Synonyms
lymphocyte of bronchial-associated lymphoid tissue
BALT lymphocyte
Definition (free text, please give PubMed ID)
Any lymphocyte that is part of bronchial-associated lymphoid tissue
Parent term
mucosa-associated lymphoid tissue lymphocyte (#671 )

closing for now as it's not clear whether these are different from other lymphocytes


The main window can be resized enough to hide the bottom controls, with no scrolling available

Title says it all. Here's a screenshot in which the window is as small as possible:

The bottom two sliders, "play", and "loop" are not visible

Indeed, the window should define a minimum size.


compile failed for nixos

I did completely same as the readme.
with flake and homemanager,
but it builded failed,
here is the info:
[8/12] Compiling C++ object src/libhyprgrass.so.p/main.cpp.o FAILED: src/libhyprgrass.so.p/main.cpp.o  g++ -Isrc/libhyprgrass.so.p -Isrc -I../src -I/nix/store/6aay4yykygav8vmsfaqbchbhi6796b94-wf-touch-git-dev/include -I/nix/store/gprgf8rc2ngn6ighv1z61pvgac0k2k63-glm-<IP_ADDRESS>/include -I/nix/store/ldz76qcgv1ic8s8jlvhcyw51s7pzl2bx-pixman-0.43.4/include/pixman-1 -I/nix/store/bnanzp44z8gxxqcm2b9sw4wpymnlfqpp-libdrm-2.4.120> ../src/main.cpp: In function 'PLUGIN_DESCRIPTION_INFO pluginInit(void*)': ../src/main.cpp:73:96: error: no match for 'operator=' (operand types are 'std::shared_ptr<std::function<void(void*, SCallbackInfo&, std::any)> >' and 'CSharedPointer<std::function<void(void*, SCallbackInfo&, std::any)> >')    73 |     g_pTouchDownHook = HyprlandAPI::registerCallbackDynamic(PHANDLE, "touchDown", hkOnTouchDown);       |                                                                                ^ In file included from /nix/store/nw6sp0xjb83svy8a5d6i46islcv3l6sm-gcc-13.2.0/include/c++/13.2.0/memory:80,                  from ../src/./gestures/Gestures.hpp:7,                  from ../src/GestureManager.hpp:2,                  from ../src/main.cpp:1: /nix/store/nw6sp0xjb83svy8a5d6i46islcv3l6sm-gcc-13.2.0/include/c++/13.2.0/bits/shared_ptr.h:418:9: note: candidate: 'template<class _Yp> std::shared_ptr<_Tp>::_Assignable<const std::shared_ptr<_Yp>&> std::shared_ptr<_Tp>::operator=(const std::shared_ptr<_Yp>&) [with _Tp = std::function<void(void*, SCallbackInfo&, st>   418 |         operator=(const shared_ptr<_Yp>& __r) noexcept       |         ^~~~~~~~ /nix/store/nw6sp0xjb83svy8a5d6i46islcv3l6sm-gcc-13.2.0/include/c++/13.2.0/bits/shared_ptr.h:418:9: note:   template argument deduction/substitution failed: ../src/main.cpp:73:96: note:   'CSharedPointer<std::function<void(void*, SCallbackInfo&, std::any)> >' is not derived from 'const std::shared_ptr<_Tp>'    73 |     g_pTouchDownHook = HyprlandAPI::registerCallbackDynamic(PHANDLE, "touchDown", hkOnTouchDown);       |                                                                                ^ /nix/store/nw6sp0xjb83svy8a5d6i46islcv3l6sm-gcc-13.2.0/include/c++/13.2.0/bits/shared_ptr.h:429:9: note: candidate: 'template<class _Yp> std::shared_ptr<_Tp>::_Assignable<std::auto_ptr<_Up> > std::shared_ptr<_Tp>::operator=(std::auto_ptr<_Up>&&) [with _Tp = std::function<void(void*, SCallbackInfo&, std::any)>]'   429 |         operator=(auto_ptr<_Yp>&& __r)       |         ^~~~~~~~ /nix/store/nw6sp0xjb83svy8a5d6i46islcv3l6sm-gcc-13.2.0/include/c++/13.2.0/bits/shared_ptr.h:429:9: note:   template argument deduction/substitution failed: ../src/main.cpp:73:96: note:   'CSharedPointer<std::function<void(void*, SCallbackInfo&, std::any)> >' is not derived from 'std::auto_ptr<_Up>'    73 |     g_pTouchDownHook = HyprlandAPI::registerCallbackDynamic(PHANDLE, "touchDown", hkOnTouchDown);       |                                                                                ^ /nix/store/nw6sp0xjb83svy8a5d6i46islcv3l6sm-gcc-13.2.0/include/c++/13.2.0/bits/shared_ptr.h:446:9: note: candidate: 'template<class _Yp> std::shared_ptr<_Tp>::_Assignable<std::shared_ptr<_Yp> > std::shared_ptr<_Tp>::operator=(std::shared_ptr<_Yp>&&) [with _Tp = std::function<void(void*, SCallbackInfo&, std::any)>]'   446 |         operator=(shared_ptr<_Yp>&& __r) noexcept       |         ^~~~~~~~ /nix/store/nw6sp0xjb83svy8a5d6i46islcv3l6sm-gcc-13.2.0/include/c++/13.2.0/bits/shared_ptr.h:446:9: note:   template argument deduction/substitution failed: ../src/main.cpp:73:96: note:   'CSharedPointer<std::function<void(void*, SCallbackInfo&, std::any)> >' is not derived from 'std::shared_ptr<_Tp>'    73 |     g_pTouchDownHook = HyprlandAPI::registerCallbackDynamic(PHANDLE, "touchDown", hkOnTouchDown);

does #118 work?

does #118 work?

sorry, still same problem.
../src/main.cpp:73:96: error: no match for 'operator=' (operand types are 'std::shared_ptr<std::function<void(void*, SCallbackInfo&, std::any)> >' and 'CSharedPointer<std::function<void(void*, SCallbackInfo&, std::any)> >') 73 |     g_pTouchDownHook = HyprlandAPI::registerCallbackDynamic(PHANDLE, "touchDown", hkOnTouchDown); |
maybe bacause i am using hyprland 0.39.1? that is newst for nixos

This doesn't seem like 0.39.1, CSharedPointer wasn't even in the release. Not sure what commit nixpkgs is on, but I don't want to dig it out, especially since 0.40.0 is already out.
If someone wants to fix it, write a patch off 413f186d and PR, I'll figure out how to merge it

This doesn't seem like 0.39.1, CSharedPointer wasn't even in the release. Not sure what commit nixpkgs is on, but I don't want to dig it out, especially since 0.40.0 is already out.
If someone wants to fix it, write a patch off 413f186 and PR, I'll figure out how to merge it

I changed to 0.40, but it still got the same error, I really don't know why.

exactly on 0.40.0, cba1ade (I typed that commit from memory so it might be wrong).
Hyprland master branch has more breaking changes and broke plugins so it won't work


Ubuntu 16.04 template

As requested in #1101.
I haven't yet attempted to build this.

Building a definition from this template fails because it cannot get past the typing sequence on the boot screen. This sequence has to be revised and probably the preseed configuration too.

I managed to do some finagling and was able to get it to boot by manually typing into at the boot menu...
Type:
<Esc><F6><Esc>

brings up an editable partial boot string then prepend to the front of that prompt
DEBIAN_FRONTEND=noninteractive preseed/url=http://<IP_ADDRESS>:7122/preseed.cfg vga=788 fb=false

Final tested results:

Full unattended not yet working
Manual intervention at the boot screen began and finished installing unattended
The preseed seems adequate
the vbox.sh installer references aptitude which is not present and will need to be replaced with apt-get
chef.sh failed
puppet.sh didn't run, I commented it out
box is useable


[FLINK-10797] "IntelliJ Setup" link is broken in README.md

Use the correct link of docs/flinkDev/ide_setup.md#intellij-idea
Change-Id: I293b3ed51e5fa54046d1a97614f02e31c93fc899

@shaoxuan-wang regardless the duplication of this pr(#7025 ) and #7024 I'd like to point out two concerns on this thread.

would it be better to fix the issue by replacing site link with relative path?(thus keep link issue isolated among versions)
what version(s) should this fix back port to?
x. there is another incorrect path of "IntelliJ Setup" in pom.xml as modify within #7024 .

@shaoxuan-wang regardless the duplication of this pr(#7025 ) and #7024 I'd like to point out two concerns on this thread.

would it be better to fix the issue by replacing site link with relative path?(thus keep link issue isolated among versions)
what version(s) should this fix back port to? (I guess all versions with README.md link to the site url are broken)

x. there is another incorrect path of "IntelliJ Setup" in pom.xml as modify within #7024 .

Thanks @TisonKun .  I did not notice there is an existing PR.    @xndai can you please close this PR as  #7024 already addressed problem reported in FLINK-10797

Didn't know there's an existing PR either. Will close this one.


Kube Web/Resource View - Read cluster name

See https://github.com/flanksource/canary-checker/blob/master/pkg/kube.go#L46

If using a kubeconfig file for connectivity the name is taken from that.
For in-cluster service-accounts:

kube-resource-report allows specifying the name via the DEFAULT_CLUSTER_NAME environment variable
kube-web-view has it hardcoded as local

Simplest approach for both seems to just to always generate the kubeconfig file.
Looking at recent deployments the SA approach is giving access issues anyway.


win: Clang compatibility & solving SystemFunction036 undefined reference

Clang throws errors for void * to x* cast
ex in fs.c, line 297 :
target = uv__malloc(target_len + 1);
to
target = (char *)uv__malloc(target_len + 1);
On Windows, SystemFunction036 is not resolved, works with :
#define SystemFunction036 NTAPI SystemFunction036
#include <ntsecapi.h>
#undef SystemFunction036

Hi, thanks for your contribution!
This PR reverts https://github.com/libuv/libuv/commit/796744869669842bd5405a71de8ba60b1556fc24, is it intentional?
The SystemFunction036 thingy is also a fix for an issue: https://github.com/libuv/libuv/commit/335e8a6d128646e5a19d39dfc677f5a5a555f7cc.
Finally: is there maybe a Clang switch for the casts? I did a quick search, looks like -std=c99 might be the solution but I did not investigate that.

Hi,
No, 7967448 is not intentional, i will check if it's ok with Clang
With -std=c++11, Clang throws fatal error without proper cast

This should be split into (at least) two commits, one for the casts, one for SystemFunction036, but apropos this:

Clang throws errors for void * to x* cast

That sounds like you're compiling libuv as C++ rather than C. Casting from void* to any other pointer type is always legal in C (but not C++.)

Yes, I'm integrating it with C++,
G++ does not complain about casting, only Clang++. This change would help me (and others who would like to compile with Clang++) to directly integrate libuv with c++ code base without forcing cast for each updates
I can split the SystemFunction036 modification in another commit if needed

I can't speak for the other maintainers but personally I'm not really open to trying to support C++. Build it the way it's intended to be built, as C. That likely fixes the SystemFunction036 issue too.

I'm kinda -0.1 on the C++thing because we won't maintain it. We test libuv with C so it is bound to break C++ compatibility at some point in the future.
But I'm -1 on the SystemFunction036 because we made that change to fix an issue faced by some users.

I'd also prefer not supporting C++.

I'll take the liberty of closing this out. Thanks anyway for taking the time to put together a pull request, Adrien.


Separate the output content from CMD with the form feed character

This can separate the output of "vcvarsall.bat" and set , and make subsequent processing a bit easier.

Oh well, this is a cute hack 👍


Cropped Image has vertical row of Missing Pixels on Right / Image Shift when Zoomed

Expected Behavior
User should be able to zoom in and out on a picture without it moving.
Actual Behavior
Moving the zoom slider even 1 notch causes the image to jump several pixels left, which results in a blank strip being output in the final file on the right side of the viewport. In the case of my example (which does not actually write the file), a 400x400 pixel image is output, but a gap of appx 68 pixels occurs on the right side of the image.
Steps to Reproduce the Problem

https://jsfiddle.net/xpvt214o/770382/
Upload a picture (preferably with something that you can reference the right and left sides on)
Move the zoom slider one notch and notice how the image jumps.
In a version of the page that I use where the image is then written as a file, that shift represents the pixels that are missing in the final image. I have verified if I write the file without touching the zoom, that the image writes correctly.

Example Link

Link: https://jsfiddle.net/xpvt214o/770382/

Specifications

Browser: Attempted on both Safari and Chrome

Left image is what was saved into after the crop, right image is what it looked like when submitted. If the image was at it's right-most boundary, the pixels would be missing and cut off.

Hi I have the same issue. Have someone found a workaround this?


Update UOP section to mention Zen4 DIV/IDIV and NOP fusion.

Since Zen4, besides the usual branch fusion, DIV/IDIV and NOP fusion are also supported. This is relatively new and not widely known, so mention it.

Thank you @cf-natali, I didn't know about this. I will add the AMD document to the references, instead of a footnote link. I will update the PR.

Ready to merge.

LGTM.


EnsembleClassifier not receiving raw parameter from attack

Describe the bug
EnsembleClassifier expects a parameter raw for predict, loss_gradient, and class_gradient, which is not passed by ProjectedGradientDescent or HopSkipJump generate functions.
To Reproduce
Steps to reproduce the behavior:
import torch
from torchvision import models

from art.classifiers import PyTorchClassifier, EnsembleClassifier
from art.attacks import ProjectedGradientDescent, HopSkipJump

# load and preprocess imagenet images
images = load_preprocess_images(...)

resnet18 = models.resnet18(pretrained=True)
alexnet = models.alexnet(pretrained=True)

wrapped_models = []
for model in [resnet18, alexnet]:
    loss = torch.nn.CrossEntropyLoss()
    optimizer = torch.optim.SGD(model.parameters(), .1, momentum=0.9, weight_decay=1e-4)
    wrapped_model = PyTorchClassifier(model=model, loss=loss, optimizer=optimizer, input_shape=(3,224,224), nb_classes=1000)
    wrapped_models.append(wrapped_model)

ensemble = EnsembleClassifier(wrapped_models)
attack = ProjectedGradientDescent(ensemble)
adv_images = attack.generate(x = images)

"""
yields
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-117-766010f0b586> in <module>
      1 attack = ProjectedGradientDescent(ensemble, eps=10000)
----> 2 adv_image = attack.generate(x = images)

~/anaconda3/envs/brainscore/lib/python3.6/site-packages/art/attacks/projected_gradient_descent.py in generate(self, x, y, **kwargs)
    108
    109             # Use model predictions as correct outputs
--> 110             targets = get_labels_np_array(self.classifier.predict(x, batch_size=self.batch_size))
    111         else:
    112             targets = y

~/anaconda3/envs/brainscore/lib/python3.6/site-packages/art/classifiers/ensemble.py in predict(self, x, batch_size, **kwargs)
    114             raw = kwargs['raw']
    115         else:
--> 116             raise ValueError('Missing argument `raw`.')
    117
    118         preds = np.array([self._classifier_weights[i] * self._classifiers[i].predict(x)

ValueError: Missing argument `raw`.

"""

Expected behavior
To generate adversarial images based on the ensemble of classifiers.
System information (please complete the following information):

Fedora Linux
Python 3.6.7
ART version: adversarial-robustness-toolbox-1.0.1
PyTorch 1.0.1.post2

Hi @dapello Thank you very much for using ART and raising this issue! We will take a look at it as soon as possible.

I think we need to add a default value of False for the keyword argument raw in predict, loss_gradient, and class_gradient in ensemble classifier instead of raising ValueError.

Looking at the source, that seems right -- happy to PR if you like.

That would be great! I'll assign this issue to you. Please start from branch dev_1.1.0 and let me know if you have any questions.

This is working perfectly now, thanks for taking my PR!


fix(expansion-panel): handle long text inside panel header title and description

Handles long text inside the mat-panel-title and mat-panel-description. Previously the text would continue wrapping and overflowing the header.
Fixes #10744.

Just a thought; material-ui expands to fit the text (example: https://codesandbox.io/s/917pm83o5r). Would it be better to follow that practice, rather than ellipsis?

nice, thanks a lot!
hope this will be merged at some point


Add support for Python3

Do we need to update the Django version also, as you mentioned the current is the LTS version and it also works fine.
PR for issue: #24
Signed-off-by: Tushar Mittal<EMAIL_ADDRESS>

@rtgdk please review :slightly_smiling_face:

I was busy for the past few days. I think updating the Django version to 2.x would keep us ahead since Python2.7 support will end by 2020. Also, as you reported, is jpype working correctly on Python3?

Also I think it would be better to work in a separate branch till we move the codebase to Python3. So I created a new branch python3 for this. Pull to this branch for any code updation.

about jpype, the pip install on Python 3.5 and Python 3.6.1 works fine but when I install using Python 3.6.3 then it gives an error, it might be problem with my computer but jpype should work fine with Python3. For django i will update it to 2.x version in the next commit.

@rtgdk everything updated to the latest version, everything is working fine on Python3.5, please check on some other python3 version.


Moved a couple of words to align with the landing page.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/do-doc-build


feat!: cleanup custom logging

This changes the logger option provided to reviewPullRequest() and createPullRequest() to be a custom Logger interface that code-suggester declares:
interface LogFn {
  /* eslint-disable @typescript-eslint/no-explicit-any */
  <T extends object>(obj: T, msg?: string, ...args: any[]): void;
  (msg: string, ...args: any[]): void;
  /* eslint-enable @typescript-eslint/no-explicit-any */
}

export interface Logger {
  error: LogFn;
  warn: LogFn;
  info: LogFn;
  debug: LogFn;
  trace: LogFn;
}

This logger interface can be fulfilled by a default Pino logger as well as plain old console.
We now default the logger to a null implementation to keep output to a minimum while allowing for a custom logger if you want output.
We can drop the pino dependency as we now provide a null default logger implementation.

Fixes #178
Fixes #183

going to release a candidate version and batch some other potentially breaking cleanup changes.

Released this as 2.0.0-candidate.1 and it integrates fine with release-please.


Table rows cannot have empty strings

I ran into this the other day and it took a really long time to figure out, I'm not really sure if this is a bug or not but figured I'd bring this to your attention.

Failed to parse Dialogflow response into AppResponse

In my case I was generating a table with some data from an API endpoint like so:
const timetableCells = timetable.Trains.map((item) => {
  return {
    cells: [item.Line, item.Destination, item.Car, item.Min],
  }
})

conv.ask(new Table({
  title: `Rail Timetable for ${station}`,
  subtitle: 'Timetable as of x date',
  image: new Image({
    url: 'http://google.com/image.png',
    alt: 'Logo'
  }),
  columns: [
    {
      header: 'Line',
      align: 'LEADING'
    },
    {
      header: 'Destination',
      align: 'LEADING'
    },
    {
      header: 'Car',
      align: 'LEADING'
    },
    {
      header: 'Arrival',
      align: 'LEADING'
    },
  ],
  rows: timetableCells,
  buttons: new Button({
    title: 'Button Title',
    url: 'https://github.com/actions-on-google'
  })
}))

The returned data from timetable looks like this and the array I'm generating from map followed the correct schema for rows, as far as I could tell there should have been no issues as the generated JSON was valid.
{"Trains":[{"Car":"8","Destination":"Glenmont","DestinationCode":"B11","DestinationName":"Glenmont","Group":"1","Line":"RD","LocationCode":"B09","LocationName":"Forest Glen","Min":"9"},{"Car":"8","Destination":"Glenmont","DestinationCode":"B11","DestinationName":"Glenmont","Group":"1","Line":"RD","LocationCode":"B09","LocationName":"Forest Glen","Min":"18"},{"Car":"8","Destination":"Glenmont","DestinationCode":"B11","DestinationName":"Glenmont","Group":"1","Line":"RD","LocationCode":"B09","LocationName":"Forest Glen","Min":"36"},{"Car":"8","Destination":"Shady Gr","DestinationCode":"A15","DestinationName":"Shady Grove","Group":"2","Line":"RD","LocationCode":"B09","LocationName":"Forest Glen","Min":""}]}

timetableCells:

DialogFlow kept throwing a JSON parsing error and I couldn't seem to figure out what the issue was, it was obvious to me that the data length needed to match the headers, but what wasn't obvious was that data in row cells cannot be empty strings. Sometimes the API endpoint returns empty strings for some data that isn't available yet. I resolved the issue with the following:
const timetableCells = timetable.Trains.map((item) => {
  return {
    cells: [item.Line || "N/A", item.Destination || "N/A", item.Car || "TBD", item.Min || "TDB"],
  }
})

I spent quite a long time debugging this, and I think this would be a good addition to the documentation if it's indeed intended and not a bug.

I tried to address this within this repo but I guess the error isn't being thrown here after looking into the code. I'll gladly post this in the right repository if pointed in the right direction.

The error probably happens on the Google Assistant side after it processes your webhook response.

The server-side parts of the Google Assistant are not open source. Any fixes would need to be done on the side of the developer to validate the input before it is returned (plus adding a note in the documentation).

Where can I file the bug report?

I can no longer reproduce this. A cell with an empty string will render without throwing an exception.


tootltips in filter page updates

What's wrong

this screenshot sucks but there are

 tags in the tooltips. each one will need to be inspected
What done looks like

the tooltips should work on hover of the question mark and not just on click
html tags should be removed
there should be a newline in place of breaks

Where the changes need to be made
filter component

i don't think this is done yet. it doesn't work on hover and there's no newline


Fix style and UI

Fixed: https://github.com/homenoc/dsbd-web/issues/37

Fix: https://github.com/homenoc/dsbd-web/issues/164

Support: https://github.com/homenoc/dsbd-feedback/issues/3 in https://github.com/homenoc/dsbd-web/pull/180/commits/f495e1abd93460a102ae29563b14befbe0dcf83d , https://github.com/homenoc/dsbd-web-admin/pull/153/commits/a15d3f5a7c796b22c4e7491dbcd57fac58f8cca8

@symysak
Thank you !!


[SPARK-40592][PS] Implement min_count in GroupBy.max

What changes were proposed in this pull request?
Implement min_count in GroupBy.max
Why are the changes needed?
for API coverage
Does this PR introduce any user-facing change?
yes, new parameter
        >>> df = ps.DataFrame({"A": [1, 2, 1, 2], "B": [True, False, False, True],
        ...                    "C": [3, 4, 3, 4], "D": ["a", "a", "b", "a"]})

        >>> df.groupby("D").max().sort_index()
           A      B  C
        D
        a  2   True  4
        b  1  False  3

        >>> df.groupby("D").max(min_count=3).sort_index()
             A     B    C
        D
        a  2.0  True  4.0
        b  NaN  None  NaN

How was this patch tested?
added UT

Merged into master, thanks @HyukjinKwon for reviews


Bye

https://github.com/WICG/document-policy/issues/29

I know not who you are, or why your posting this.


TimeFrame property name

The enum param “timeFrame” name is confusing and should be changed to “timeInterval”.
OMH does have a “time_interval” property to define the time interval, although their definition is different from ours:

http://www.openmhealth.org/documentation/#/schema-docs/schema-library/schemas/omh_time-frame
http://www.openmhealth.org/documentation/#/schema-docs/schema-library/schemas/omh_time-interval

"timeFrame": "TEN_SECOND",
should be
"timeInterval": "TEN_SECOND",

TODO

[ ] rename response timeFrame to timeInterval
[ ] Fix inconsistency between the property name in the GET Request and the Response

Note that in Kafka streams, where the data comes from, the TEN_SECOND symbol is called a time window. That is different from the time frame, since the timeFrame describes a dataset limited to a certain time frame. This latter name is consistent to the name data frame used in analytics packages like pandas or R. Could that be a resolution? Call TEN_SECOND a timeWindow and keep timeFrame for the dataset time frame?

timeWindow works well, the confusing name was timeFrame.
Regarding the timeFrame and effectiveTimeframe, I added a comment to clarify it here: https://github.com/RADAR-CNS/RADAR-RestApi/issues/37#issuecomment-331550793

Fixed with RADAR-CNS/RADAR-Schemas#55.


Firestore switch to Caches Directory for tvOS

[REQUIRED] Step 2: Describe your environment

Xcode version: 10.1
tvOS version: 12.1.2
Firebase SDK version: Newest commit https://github.com/firebase/firebase-ios-sdk/commit/45909c3ceb2318cd537adbd6930ba004d809a729
Firebase Component: Firestore

[REQUIRED] Step 3: Describe the problem
Steps to reproduce:
I tried to get Firestore running on an actual tvOS device. The Firestore startup fails at the moment since it uses the Documents directory, like iOS.
Relevant Code:
https://github.com/firebase/firebase-ios-sdk/blob/45909c3ceb2318cd537adbd6930ba004d809a729/Firestore/Source/Local/FSTLevelDB.mm#L397-L413
Switching to the NSCachesDirectory allows it to start & I could read from my Cloud Firestore.
However, I'm not sure how long the db would survive in the Caches directory...

Yeah, it looks like TARGET_OS_TV implies TARGET_OS_IPHONE (hence why it's not hitting that #error case.)
I don't know anything about tvOS, but it looks like writing to the caches directory is probably the right thing. It might be periodically deleted (while the app is not running), but it sounds like this is the norm on tvOS. Firestore should be able to recover from this, though it would imply having to fetch everything from the server after a cache purge, instead of being able to use the local disk.
We don't yet support tvOS, but as it happens, @wilhuff is actively working on it. I'll assign this issue to him; he should be able to give you a more intelligent response.

See #2496 for the comparable change in Firebase/Messaging

Thanks for trying this out. Unfortunately the simulator doesn't catch this :-(.
We don't really have much of a choice about where to put this file. My best advice is that on tvOS, keep the cache size small to avoid eviction (see cacheSizeBytes for how to do this).


PSR7 various functions accessibility issue

I tried to use two plugins on the WordPress platform with PHP 8.0.0 which uses Google services, it created a critical error when one of them gave the error https://d.pr/i/jj9FSB
After some digging, I found that the PSR7 has changed some functions which are being used by Guzzle and ending up in the services which are being used in this repo when the composer is run with 2.10.
I have fixed the issue for now for the Google Login and can attach the PR as well.

Please don't use screenshots to share code/errors. (and if you do, choose a faster website; it takes a couple of seconds loading from here)

@dwsupplee and @stephanvierkant Any updates on this one?


TA 97090

https://cockroachlabs.atlassian.net/browse/DOC-7020

links need to be updated once fix is released


🐛 useExhaustiveDependencies too specific in 1.5.3

Environment information
CLI:
  Version:                      1.5.3
  Color support:                true

Platform:
  CPU Architecture:             x86_64
  OS:                           windows

Environment:
  BIOME_LOG_DIR:                unset
  NO_COLOR:                     unset
  TERM:                         unset
  JS_RUNTIME_VERSION:           "v18.19.0"
  JS_RUNTIME_NAME:              "node"
  NODE_PACKAGE_MANAGER:         "npm/10.0.0"

Biome Configuration:
  Status:                       Loaded successfully
  Formatter disabled:           false
  Linter disabled:              false
  Organize imports disabled:    false
  VCS disabled:                 true

Workspace:
  Open Documents:               0

What happened?
When using a useMemo/useEffect hook (A hook that has a dependency array), the useExhaustiveDependencies sometimes requires a dependency that is not possible in the given context.
I for example have a state called nutrients which is an array. I am looping over the array using the following snippet:
for (let i = 0; i < nutrients.length; i++) {
   if (i + 1 === nutrients.length) {
      // Do something with the last item
   } else {
       const nextNutrient = nutrients[i + 1]
   }
}

In the above snippet useExhaustiveDependencies has, since 1.5.3, started requiring nutrients[i + 1] as a dependency. This is not possible since i is defined within the useMemo factory.
In other cases it will also try to imply that nutrients.filter should be set as a dependency even though nutrients is defined in the dependency array.
Expected result
It should not show the useExhaustiveDependencies
Code of Conduct

[X] I agree to follow Biome's Code of Conduct

I guess it's a bug from https://github.com/biomejs/biome/pull/1327, I'll look into it shortly.

#1636  same?


Improper spelling of "frozen" in terminology.

What happened:
In the terminology used and in the API, the key "freezed" is used. This is the incorrect spelling in modern usage.
What you expected to happen:
The correct spelling is "frozen".

Mercury version or commit hash (git log -1): v0.2.0 (stable)

First of all thank you for pointing out this spelling error. We used a verb form.
If we modify directly based on the stable version of v0.2.0, it will cause breaking changes, and the SDK needs to be modified accordingly. Any suggestions for this situation?

If I understand correctly, Mercury deployments are low, but it is already being used in production environments for some exchanges. My suggestion would be to leave it as is until the next major stable version that contains other breaking changes.

I agree with your suggestion very much, thank you. @jordanmack

https://github.com/nervosnetwork/mercury/pull/391


Google CPI documentation does not mention service_account parameter

In the Google CPI documentation relating to vm-types the service_account parameter is not mentioned at all. This option is necessary for bosh to be able to launch an instance with the permissions of a service account. This option is described in the bosh resource pool options section of the Google CPI README.

Closing - thanks for reporting! PR #401 fixes this and the bosh.io/docs site will be updated shortly.


Unsure how to import small molecule

Describe the bug
I'm trying to follow the steps at this link to use OpenFF/OpenMM to run MD on small molecules, but I haven't been able to load in any structures. I have the 3D coordinates of my molecule (testing on propylene but hoping to extend to other cases where SMILES/name import might not work) but I can't find a way to import it! I've tried the below .sdf solution as well as .mol and .mol2 files, but can't seem to load the information in an openforcefield-compatible way. It's possible that GaussView is creating files that are missing some information?
I would appreciate guidance on how to go from an .xyz file or equivalent to an openforcefield Molecule instance.
To Reproduce
Here's my code:
from openforcefield.topology import Molecule, Topology
from openforcefield.typing.engines.smirnoff import ForceField

from simtk import openmm, unit

pdbfile = openmm.app.PDBFile('propylene.pdb')
mol = Molecule.from_file('propylene.sdf')

propylene.pdb:
REMARK   1 File created by GaussView 6.0.16
HETATM    1  C           0      -1.459  -1.439   0.000                       C
HETATM    2  H           0      -1.101  -2.447  -0.001                       H
HETATM    3  H           0      -1.104  -0.933  -0.874                       H
HETATM    4  H           0      -2.529  -1.441   0.001                       H
HETATM    5  C           0      -0.946  -0.713   1.257                       C
HETATM    6  C           0      -1.234   0.554   1.522                       C
HETATM    7  H           0      -0.321  -1.330   1.918                       H
HETATM    8  H           0      -0.868   1.072   2.418                       H
HETATM    9  H           0      -1.859   1.170   0.861                       H
END
CONECT    1    2    3    4    5
CONECT    2    1
CONECT    3    1
CONECT    4    1
CONECT    5    6    7    1
CONECT    6    5    8    9
CONECT    7    5
CONECT    8    6
CONECT    9    6

propylene.sdf:
Molecule Name

Created by GaussView 6.0.16
  9  8  0  0  0  0  0  0  0  0  0    0
   -1.4590   -1.4390    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   -1.1010   -2.4470   -0.0010 H   0  0  0  0  0  0  0  0  0  0  0  0
   -1.1040   -0.9330   -0.8740 H   0  0  0  0  0  0  0  0  0  0  0  0
   -2.5290   -1.4410    0.0010 H   0  0  0  0  0  0  0  0  0  0  0  0
   -0.9460   -0.7130    1.2570 C   0  0  0  0  0  0  0  0  0  0  0  0
   -1.2340    0.5540    1.5220 C   0  0  0  0  0  0  0  0  0  0  0  0
   -0.3210   -1.3300    1.9180 H   0  0  0  0  0  0  0  0  0  0  0  0
   -0.8680    1.0720    2.4180 H   0  0  0  0  0  0  0  0  0  0  0  0
   -1.8590    1.1700    0.8610 H   0  0  0  0  0  0  0  0  0  0  0  0
  1  2  1  0  0  0  0
  1  3  1  0  0  0  0
  1  4  1  0  0  0  0
  1  5  1  0  0  0  0
  5  6  2  0  0  0  0
  5  7  1  0  0  0  0
  6  8  1  0  0  0  0
  6  9  1  0  0  0  0
M  END

Output
LICENSE: Could not open license file "oe_license.txt" in local directory
LICENSE: N.B. OE_LICENSE environment variable is not set
LICENSE: N.B. OE_DIR environment variable is not set
LICENSE: No product keys!
LICENSE: No product keys!
LICENSE: No product keys!
LICENSE: No product keys!
Warning: Unable to load toolkit 'OpenEye Toolkit'. The Open Force Field Toolkit does not require the OpenEye Toolkits, and can use RDKit/AmberTools instead. However, if you have a valid license for the OpenEye Toolkits, consider installing them for faster performance and additional file format support: https://docs.eyesopen.com/toolkits/python/quickstart-python/linuxosx.html OpenEye offers free Toolkit licenses for academics: https://www.eyesopen.com/academic-licensing
/n/home03/cwagen/.conda/envs/openmm/lib/python3.7/site-packages/simtk/openmm/app/internal/pdbstructure.py:537: UserWarning: WARNING: duplicate atom (HETATM    3  H           0      -1.104  -0.933  -0.874  1.00  0.00           H  , HETATM    2  H           0      -1.101  -2.447  -0.001  1.00  0.00           H  )
  warnings.warn("WARNING: duplicate atom (%s, %s)" % (atom, old_atom._pdb_string(old_atom.serial_number, atom.alternate_location_indicator)))
/n/home03/cwagen/.conda/envs/openmm/lib/python3.7/site-packages/simtk/openmm/app/internal/pdbstructure.py:537: UserWarning: WARNING: duplicate atom (HETATM    4  H           0      -2.529  -1.441   0.001  1.00  0.00           H  , HETATM    3  H           0      -1.104  -0.933  -0.874  1.00  0.00           H  )
  warnings.warn("WARNING: duplicate atom (%s, %s)" % (atom, old_atom._pdb_string(old_atom.serial_number, atom.alternate_location_indicator)))
/n/home03/cwagen/.conda/envs/openmm/lib/python3.7/site-packages/simtk/openmm/app/internal/pdbstructure.py:537: UserWarning: WARNING: duplicate atom (HETATM    5  C           0      -0.946  -0.713   1.257  1.00  0.00           C  , HETATM    1  C           0      -1.459  -1.439   0.000  1.00  0.00           C  )
  warnings.warn("WARNING: duplicate atom (%s, %s)" % (atom, old_atom._pdb_string(old_atom.serial_number, atom.alternate_location_indicator)))
/n/home03/cwagen/.conda/envs/openmm/lib/python3.7/site-packages/simtk/openmm/app/internal/pdbstructure.py:537: UserWarning: WARNING: duplicate atom (HETATM    6  C           0      -1.234   0.554   1.522  1.00  0.00           C  , HETATM    5  C           0      -0.946  -0.713   1.257  1.00  0.00           C  )
  warnings.warn("WARNING: duplicate atom (%s, %s)" % (atom, old_atom._pdb_string(old_atom.serial_number, atom.alternate_location_indicator)))
/n/home03/cwagen/.conda/envs/openmm/lib/python3.7/site-packages/simtk/openmm/app/internal/pdbstructure.py:537: UserWarning: WARNING: duplicate atom (HETATM    7  H           0      -0.321  -1.330   1.918  1.00  0.00           H  , HETATM    4  H           0      -2.529  -1.441   0.001  1.00  0.00           H  )
  warnings.warn("WARNING: duplicate atom (%s, %s)" % (atom, old_atom._pdb_string(old_atom.serial_number, atom.alternate_location_indicator)))
/n/home03/cwagen/.conda/envs/openmm/lib/python3.7/site-packages/simtk/openmm/app/internal/pdbstructure.py:537: UserWarning: WARNING: duplicate atom (HETATM    8  H           0      -0.868   1.072   2.418  1.00  0.00           H  , HETATM    7  H           0      -0.321  -1.330   1.918  1.00  0.00           H  )
  warnings.warn("WARNING: duplicate atom (%s, %s)" % (atom, old_atom._pdb_string(old_atom.serial_number, atom.alternate_location_indicator)))
/n/home03/cwagen/.conda/envs/openmm/lib/python3.7/site-packages/simtk/openmm/app/internal/pdbstructure.py:537: UserWarning: WARNING: duplicate atom (HETATM    9  H           0      -1.859   1.170   0.861  1.00  0.00           H  , HETATM    8  H           0      -0.868   1.072   2.418  1.00  0.00           H  )
  warnings.warn("WARNING: duplicate atom (%s, %s)" % (atom, old_atom._pdb_string(old_atom.serial_number, atom.alternate_location_indicator)))
[13:24:41] ERROR: CTAB version string invalid at line 4
[13:24:41] ERROR: moving to the beginning of the next molecule
Traceback (most recent call last):
  File "test_openff.py", line 9, in <module>
    mol = Molecule.from_file('propylene.sdf')
  File "/n/home03/cwagen/.conda/envs/openmm/lib/python3.7/site-packages/openforcefield/topology/molecule.py", line 3001, in from_file
    'Unable to read molecule from file: {}'.format(file_path))
Exception: Unable to read molecule from file: propylene.sdf

Computing environment (please complete the following information):
Python 3.7.6, AmberTools 19, openforcefield 0.6.0, openmm 7.4.2, rdkit 2020.03.2, Anaconda 5.0.1
Additional context
Thanks for the help!

It looks to me like RDKit's SDF reader is choking on your input sdf file (you're using the RDKit backend since you're not using OpenEye). I'd guess, given the error message about "CTAB version string invalid at line 4", it's having a problem with the 4th line of your file, which says "Created by GaussView 6.0.16". Maybe it is unhappy with spaces, which I THINK may not be typical for these.
I generated a quick propylene sdf file myself using the OpenEye toolkits and it looks something like this. Perhaps you'd care to try this in your setup and see if it'd run. It may just be something silly like needing to remove the spaces or decimals or both from that GaussView line noted above.

  -OEChem-06012008573D

  9  8  0     0  0  0  0  0  0999 V2000
    0.8212   -0.6789   -0.2025 C   0  0  0  0  0  0  0  0  0  0  0  0
    1.3491   -1.4450    0.7582 C   0  0  0  0  0  0  0  0  0  0  0  0
    0.8797   -1.4501    2.1744 C   0  0  0  0  0  0  0  0  0  0  0  0
    1.2086   -0.7222   -1.2145 H   0  0  0  0  0  0  0  0  0  0  0  0
    0.0000    0.0000    0.0000 H   0  0  0  0  0  0  0  0  0  0  0  0
    2.1710   -2.1092    0.5059 H   0  0  0  0  0  0  0  0  0  0  0  0
    1.4342   -0.7057    2.7538 H   0  0  0  0  0  0  0  0  0  0  0  0
    1.0159   -2.4253    2.6524 H   0  0  0  0  0  0  0  0  0  0  0  0
   -0.1838   -1.1958    2.2155 H   0  0  0  0  0  0  0  0  0  0  0  0
  1  2  2  0  0  0  0
  2  3  1  0  0  0  0
  1  4  1  0  0  0  0
  1  5  1  0  0  0  0
  2  6  1  0  0  0  0
  3  7  1  0  0  0  0
  3  8  1  0  0  0  0
  3  9  1  0  0  0  0
M  END
$$$$

(I'm not currently running the RDKit backend or I'd try it myself; I can dig in if this doesn't get you anywehre, but it would take longer because I'd have to make a new conda env, etc.)

I'm not used to seeing so many columns in the bonds section. I know that there's a difference between V2000 and V3000 mol/sdf files, but none of the documentation for those covers the extra columns. Is there a spec for the subtype of SDF files that Gaussian will write out? It would be good to add logic to handle that if there's another format we should check for.
I can get it to work if I replace the last three 0s on the counts line with 0999 V2000. This still doesn't help with the root cause, though.
Molecule Name

Created by GaussView 6.0.16
  9  8  0  0  0  0  0  0  0  0999 V2000
   -1.4590   -1.4390    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   -1.1010   -2.4470   -0.0010 H   0  0  0  0  0  0  0  0  0  0  0  0
   -1.1040   -0.9330   -0.8740 H   0  0  0  0  0  0  0  0  0  0  0  0
   -2.5290   -1.4410    0.0010 H   0  0  0  0  0  0  0  0  0  0  0  0
   -0.9460   -0.7130    1.2570 C   0  0  0  0  0  0  0  0  0  0  0  0
   -1.2340    0.5540    1.5220 C   0  0  0  0  0  0  0  0  0  0  0  0
   -0.3210   -1.3300    1.9180 H   0  0  0  0  0  0  0  0  0  0  0  0
   -0.8680    1.0720    2.4180 H   0  0  0  0  0  0  0  0  0  0  0  0
   -1.8590    1.1700    0.8610 H   0  0  0  0  0  0  0  0  0  0  0  0
  1  2  1  0  0  0  0
  1  3  1  0  0  0  0
  1  4  1  0  0  0  0
  1  5  1  0  0  0  0
  5  6  2  0  0  0  0
  5  7  1  0  0  0  0
  6  8  1  0  0  0  0
  6  9  1  0  0  0  0
M  END


Deceptively named negu for MIPS

The negu instruction for MIPS, or "Negate unsigned," is deceptively named.
negu d, s is not d = ~s, as I originally thought, but d = 0 - s. It's a pseudo instruction for subu.
The following script in python shows the correct semantics:
from unicorn import *
from unicorn.mips_const import *

CODE = '\x00\x05\x20\x23'

ADDRESS = 0x10000

mu = Uc(UC_ARCH_MIPS, UC_MODE_32 + UC_MODE_BIG_ENDIAN)
mu.mem_map(ADDRESS, 4096)
mu.mem_write(ADDRESS, CODE)
mu.reg_write(UC_MIPS_REG_A1, 0xff00ff00)
mu.emu_start(ADDRESS, ADDRESS + 4)
print hex(mu.reg_read(UC_MIPS_REG_A0))
print hex(mu.reg_read(UC_MIPS_REG_PC))

One would think negu $a0, $a1 would be $a0 = ~$a1, but...
hex(~0xff00ff00 & 0xffffffff) # is 0xff00ff
hex((0 - 0xff00ff00) & 0xffffffff) # is 0xff0100

We can see by running the following script through unicorn engine that the semantics of MIPS, according to qemu and spec, is arg0 = 0 - arg1.
This instruction is currently translated to arg0 = neg.d(arg1) in LLIL.

As more people weigh-in in slack, this may be an erroneous bug report, and just show my lack of understanding in BINJA IL. If that is so, please close :).

The neg LLIL instruction does what you want.

My misunderstanding.


5-95% balance in classes trouble with predicting unseen data

Question: how do you train for classifying dataset which has got hugely misbalanced classes ?
I have 230000 rows of data in which about 4% is class=1, for the rest class=0. When I train with whole dataset I get poor prediction accuracy about 10%. I tried taking all samples of class 1(8000 rows) and then 8000 random of class 0. With that dataset of 16000 I get awesome accuracy of 90% which is more than enough for me. BUT when I use that model to predict based on all the other left samples from big set which all of them are 0 I get very bad accuracy. What do I do wrong? I tried weights ( auto and manual) with no luck.
When I try to predict that other set of zeros I use polynomials, scaler, PCA and one hot encoders which are all fitted during training cycle.

for imbalance classes try to use other metric like weighted avg macro precision and recall etc.

@haker88 Please try with the above comment and let us know if this is still an issue.Thanks!

Closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


on_pre_enter and on_enter are not called when a Screen instance is declared in kv

The python equivalent works as expected. The bug only appears for the first added screen, and only at add time, subsequent screen changes fire the events.
from kivy.app import App
from kivy.lang import Builder
from kivy.factory import Factory

kv = """
<Scr@Screen>:
    on_pre_enter: print('on_pre_enter', self.name)
    on_enter: print('on_enter', self.name)

<Test>:
    ScreenManager:
        Scr:
            name: 'from kv'
"""

Builder.load_string(kv)

class Test(Factory.BoxLayout):
    def __init__(self, *args, **kwargs):
        super(Test, self).__init__(*args, **kwargs)
        sm = Factory.ScreenManager()
        s1 = Factory.Scr(name='from python')
        sm.add_widget(s1)
        self.add_widget(sm)

class TestApp(App):
    def build(self):
        return Test()

if __name__ == '__main__':
    TestApp().run()

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Currently, the on_enter does get triggered from KV. But when first Screen's on_enter is triggered, the Screen does not yet have a name - self.name is still ''.
The current code in ScreenManager changes the self.current from None to ''
How about postponing dispatching all events until the root is entirely built?

Was a solution ever implemented? Could on_enter be delayed until KV assigns a name?

@victorbutler I'm not aware of any implemented solution, however I like the A option as @tito mentioned it as it would provide quite a good anchor to work with and not just attempt to delay anything. Specifying the exact place and the whole code flow for KV interpreting with some variables seems more reasonable to me as it could be used for KV compiler, unlike the option B which I bet complicates stuff even more (at least in my eyes if I'd like to write a unittest for it).
With A implemented we could add the KV lang flow to the docs i.e. how it's parsed, what happens in the rules, etc, which seems to me like a common missing knowledge for beginners that is learned via trial & error.

Confirmed still a problem in Kivy 2.2.1, Windows 11.


[AMBARI-23121] Fix few instance manager issues found during ZK deploment (dsen)

What changes were proposed in this pull request?
Fixed few bugs I found during ZK deployment
How was this patch tested?
Tesed manually.
Planning to add UTs in separate commit

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/904/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/929/
Test FAILed.
Test FAILured.


change scope of dockerfile

Description
textmate grammar isn't maintained in seven years and fails to highlight new syntax https://github.com/asbjornenge/Docker.tmbundle/issues/39. changing scope to bash per @princemaple comment.
Checklist:

[ ] I am adding a new extension to a language.

[ ] The new extension is used in hundreds of repositories on GitHub.com

Search results for each extension:

https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3AFOOBAR+KEYWORDS+NOT+nothack

[ ] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

[URL to each sample source, if applicable]

Sample license(s):

[ ] I have included a change to the heuristics to distinguish my language from others using the same extension.

[ ] I am adding a new language.

[ ] The extension of the new language is used in hundreds of repositories on GitHub.com.

Search results for each extension:

https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3AFOOBAR+KEYWORDS+NOT+nothack

[ ] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

[URL to each sample source, if applicable]

Sample license(s):

[ ] I have included a syntax highlighting grammar: [URL to grammar repo]

[ ] I have added a color

Hex value: #RRGGBB
Rationale:

[ ] I have updated the heuristics to distinguish my language from others using the same extension.

[ ] I am fixing a misclassified language

[ ] I have included a new sample for the misclassified language:

Sample source(s):

[URL to each sample source, if applicable]

Sample license(s):

[ ] I have included a change to the heuristics to distinguish my language from others using the same extension.

[ ] I am changing the source of a syntax highlighting grammar

Old: [URL to grammar repo]
New: [URL to grammar repo]

[ ] I am updating a grammar submodule

[ ] I am adding new or changing current functionality

[ ] I have added or updated the tests for the new or changed functionality.

[ ] I am changing the color associated with a language

[ ] I have obtained agreement from the wider language community on this color change.

[URL to public discussion]
[Optional: URL to official branding guidelines for the language]

Sorry I didn't make it clear. I thought the issue was filed against the sublime syntax, so I suggested using the Dockerfile (with Bash) sublime syntax instead. The GitHub syntax has nothing to do with sublime. If it was using the TM language file from the referenced repo (which has both TM language and sublime syntax), someone has to update the TM language file. I don't think there is any change that needed to be done here.

this is switching tm to sublime. linguist understands both syntaxes. so if sublime is newer and maintained, then we should switch to it.

You can't randomly change the grammars.yml file. It is automatically maintained based on the content of the grammar repos. This change also won't make any difference as the tm_scope defined in languages.yml is what will be used. If the grammar is no longer maintained and has bugs that won't be fixed, you should submit a PR to change the grammar (see CONTRIBUTING.md).
Closing this PR as it's the wrong approach to take.

this is switching tm to sublime. linguist understands both syntaxes. so if sublime is newer and maintained, then we should switch to it.

Oh yes, Linguist only supports TextMate compatible Sublime grammars. Sublime 3 and later grammars are not TextMate compatible.


build: tweak LLDB flags for testing

Adjust LLDB flags for new Foundation layout.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please smoke test


Has anyone tried rendering chord diagrams?

Has anyone tried rendering chord diagrams with `rich-pixels?

Going to close this due to inactivity.


Leader Line Position causes lag when number of leader lines is high

I am using leader lines to show Dependency lines between different divs. When the number of lines are in hundreds, and I scroll the main div, I call the position method, but it appears very slow while repositioning. Takes 4 to 5 seconds to get redrawn to its new position after horizontal scroll. Surprisingly it doesn't lag at all on vertical scroll. Please let me known the solution to this problem

Hi @TinuTennyson, thank you for the comment.
Check whether the position method is called too frequently or not. The scroll event especially is fired too many even if you don't have to call the position method.
You can use e.g. AnimEvent library to control calling event listeners.

Or, you can move the SVG elements to the scrollable element instead of calling the position method if you really want it.
For example:
https://jsfiddle.net/6pw9edaj/
Note that it is not supported, and it is deprecated. You should not use this if you can't understand the logic.

No reply came, then this abandoned issue is closed.

Thanks.  Issue can be marked as resolved


npm install fails on Linux for Node 4.0/4.1 and iojs v3.3.1

Hi,
Seems like latest versions of nodejs and iojs are not supported even though there is a closed issue mentioning the support for Node 4.0.
I've tried deleting the node-gyp folder mentioned in the error message. Still nothing works. There's no local node_modules folder in the project that could cause this issue.
rm -rf /home/user/.node-gyp

The full error output can be seen here (https://gist.github.com/6D65/d3efadef2b9acec20a28)
Am I missing something?
Thank you.

FWIW, master does compile for me.
"ffi": "git+https://github.com/node-ffi/node-ffi.git"

@TooTallNate, anything blocking a new release?

See https://github.com/node-ffi/node-ffi/issues/262


Fix formating issue

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@cazawideh PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

Hello @cazawideh. Can we please merge this one? Thanks


How to detect bus activity by other master

Hi Sergey,
in the datasheet is written that the IICMB supports multi master. How i can detect that another master has occupied the bus?
In my opinion it's the Bus Busy or Bus captured bit:

But on another place is written:
Bus Bysy bit (BB) and Bus Captured (BC) bit (as well as Bus ID field) should be used mostly for diagnostic purposes. For example, there is no need to refer to BB bit to determine bus busyness before giving a Start command – this is done automatically by the byte-level FSM.
Do you have a suggestions for me?
Thanks for your support.
BR,
Andreas

Hi Andreas!
Yes, you are right, you can detect that another master has occupied the bus using the Bus Busy bit (BB).
Each pair of SCL / SDA signals connected to IICMB has a personal bus state analyzer attached (bus_state.vhd) that constantly monitors the bus state to detect start and stop conditions. Bus Busy bit = 1 means that a start condition has been observed on the bus, which means someone has captured it. This "someone" could be you or another master on the bus. To determine who has captured the bus you can refer to the Bus Captured (BC) bit. A value of 1 for this bit means the IICMB occupied the bus, a value of 0 means that the bus is occupied by someone else.
So, to summarize:
a. BB = 0; -- the bus is free
b. BB = 1, BC = 0; -- the bus has been captured by other master
c. BB = 1, BC = 1; -- IICMB occupies the bus
Best regards,
Sergey

Hi Sergey,
thanks for the clarification. I'll prepare an PR to add this funcs to iicmb.c.
BR,
Andreas

done in PR https://github.com/sshuv/iicmb/pull/9


UDP LAN Discovery fails on OSX

EDIT: It works in Unity 2018. Seems to be related to the mono version of Unity 2017
Version Number and Operating System(s):
Current Master and DEV. OSX
Edit: Unity version is 2017 LTS
API compatibility is Mono 2.0 Subset
Expected behavior:
Tick "Get Local Network Connections" in MultiplayerMenu.unity
Build and host server.
Run game in unity editor.
See IP address of discovered host printed into console
Actual behavior:
No IP address is printed
Steps to reproduce:
New unity project with forge, tick "Get Local Network Connections"
[Optional] Discord Username:
AttackingHobo

Interesting as I don't have that issue. I am working on upgrading the multiplayer menu to automatically list local games and potentially master server games in a list.
Are you running both the client and host on OS X?

Here is a fix with platform dependent compilation. Don't know if these changes will break under specific conditions. So someone might want to take a look at this, and see if this is good enough, or if there is a better solution.
https://hastebin.com/kitogigata.cs

Note: @AttackingHobo confirmed on discord that Unity 2018 does not have the same issue.
It seems to be a problem with the version of mono shipped with earlier versions of Unity.

Issue introduced by PR #227

LAN discovery also seems to fail on Android, on Unity 2018. I've not been able to test with the commit prior to PR #227 being merged to see if it was working before that.

LAN discovery also seems to fail on Android, on Unity 2018. I've not been able to test with the commit prior to PR #227 being merged to see if it was working before that.

I tested it on some android devices, with Unity 2019.2.10f1. The build is failing on some devices (Oculus Quest) and is working on others (Galaxy Note 8 for example). Problem is the NetworkInterface.GetAllNetworkInterfaces() throw an exception: "Unable to find libc".

For Android issues please see PR #304

2017 LTS is soon end of life.


feat(backup): add granular control with toBackup and toLoad options

This PR introduces significant improvements to the backup process by adding the toBackup and toLoad options. These changes provide more granular control over channel inclusion and exclusion during backup and restoration.

Added toBackup and toLoad options to specify channels for inclusion in the backup and restoration process.
Updated logic to enforce mutual exclusivity between toBackup and doNotBackup.
Updated documentation to reflect the new options and their usage.

These updates improve flexibility, maintainability, and clarity in the backup process.

Not done. Will need to remove some DEBUG console.log and refine changes.

I have tested all new params more than 4 times in total with different Discord Channels and Categories. No issues. Whenever someone has time please do test this Github PR as well. All changes have been documented, thus please read the README.md file for the info.
Note:

I have noticed that threads continue to be populated with new messages while the channel is advised to have been restored.
Threads also restore themselves at the end of a channels latest backed up message instead of restoring during the time and location in the channel they first were created and populated. (I am going to look into this).

Once this PR is ready to be merged, I can run through the code and test it before merging. Regarding the what you said about threads, do you mean when you mark a channel to be restored, the connected threads are also being restored?

Understood.
I will elaborate:
Threads continue receiving new messages even after the channel is restored:
After restoring a channel from a backup, the threads associated with that channel remain active and continue to populate with new messages. This suggests that the restoration process does not pause the threads or return them to a "restored" state, which could lead to inconsistencies in how they appear post-backup.
Threads restore at the end of the channel’s latest backed-up message:
Instead of being restored at their original point in the conversation, threads appear at the end of the latest backed-up message. This disrupts the chronological order of messages and threads, breaking the flow of the conversation.
Tree Example:
Let’s say we have a channel, Channel 1, with the following structure before backup:

Channel 1:

Message 1: "Xue Xian ni Lang."
Message 2: "Je suis Charlotte."

Thread 1: Roman (Started in response to Message 2)

Thread message: "Let's discuss Roman history."

Message 3: "Comment ça va."

After restoring from the backup, the expected structure would be:

Channel 1:

Message 1: "Xue Xian ni Lang."
Message 2: "Je suis Charlotte."

Thread 1: Roman (Restored next to Message 2)

Message 3: "Comment ça va."

However, what currently happens is that the thread Roman restores at the end of the channel:

Channel 1:

Message 1: "Xue Xian ni Lang."
Message 2: "Je suis Charlotte."
Message 3: "Comment ça va."

Thread 1: Roman (Restored here, after Message 3, instead of next to Message 2)

This breaks the conversation flow, making it unclear which message the thread was originally responding to.

Oh I see what you mean, this is likely a bug carried over from the original discord-backup package. Are you implementing a fix for this or should a new issue be created for somebody to fix?

I am going to look it over and implement a fix for this. I will open an Issue for it today or tomorrow.

This PR is complete as well.

@NorkzYT, regarding the concurrency feature, I saw it defaults to running 45 requests at a time, does this cause any race conditions? for example does it open up the possibility of a channel being restored before the category it belongs to is restored? Also has it had any meaningful impact to the performance?

I tested for that possibility and no race conditions have appeared. Speeds are a bit better than before, although what I need to do is have the fetched messages of a channel function work in parallel to fetch the max of 49 messages per second for channels to restore much quicker since I have seen that sometimes there are 5 or 7 requests to the Discord API in 1 second and sometimes 37 in 1 second. It is very inconsistent.
My thoughts is to curate a GitHub PR that will have a working functionality of concurrency, parallelism, and possibly batching.

In that case can we take the concurrency feature out of this PR and move it to its own PR that way it can be delayed until its more stable/consistent? (it looks like it was merged into master from the other PR, so im unsure why its saying the change is also new for this one)

This PR's branch was not created from master, although from the concurrency branch, I fixed and added more features on top of it. Which is most likely why it is advising the change is new.
I do believe the concurrency implementation is stable enough.

Great, I will merge this branch into master then. before doing so, would you advise everything ready to make a release or should it be delayed until your next PR?

Understood. Delay the release for my next PR. I will work on it sometime this week or next week as it requires a lot of testing.


Support for Twitter API v2

Now that the new Twitter API has been launched, support for it needs to be added in Tweepy.
This is planned to be part of v4.0.
Initial support for Twitter API v2 has been added with Tweepy v4.0.
Support for additional endpoints and features have been added with Tweepy v4.1, v4.2, v4.3, v4.4, and v4.5.
Support for new endpoints and streaming will be added with future v4.x releases.
v3.10 will be released first, as the last version to support Python 2.7 (and probably Python 3.5).
My goal is to get v3.10 out before the end of the year and then work on dropping support for Python 2.
v3.10 has been released now, and support for Python 2.7 and 3.5 has been dropped.
Afterwards, This will be the next priority, along with support for asyncio.
 I've (I think understandably) fallen behind a bit this year in my plans for Tweepy and in keeping up with issues and Discord messages, but I'm working on catching back up.

I've created an initial implementation in an api-v2 branch and PR #1535.

hey @Harmon758 thanks for this, can we get some idea about when is it going to be released? Thanks

Most of the endpoints should now be part of the api-v2 branch / PR #1535.
Once that PR is ready, I'll take it out of its draft state and leave it open for a few weeks for people to test and review.
Afterwards, it'll be merged into the master branch.
As for v4.0 itself, there's other features (https://github.com/tweepy/tweepy/milestone/11) that I want to include in it before it's released.

The api-v2 branch / pull request #1535 should be complete now.
Any feedback or review would be appreciated.

Hey,
I am not a programming professional and now I have added the new api-v2-branch from tweepy. Somehow I don't understand how I can send queries with search_full_archive now? I have adapted my script:
try:
    auth = tweepy.AppAuthHandler(Credentials.consumer_key2, Credentials.consumer_secret2)
    api = tweepy.API(auth,wait_on_rate_limit=True)
except Exception as e: # catch connection errors
        print("Authentication connection error: {}".format(e.reason))

try:
    tweets = tweepy.Cursor(api.search_full_archive,
                label = ''
                query='\"{}\"'.format(searchWords),
                lang="en",
                fromDate= data_since_pro
                toDate='202005030000'
                ).items(numTweets)
except Exception as e: # catch connection error
    print("API connection error: {}".format(e.reason))

# store these tweets in a Python list
tweet_list = list(tweets)

# retrieve information from tweets iterable object
for tweet in tweet_list:
    print(tweet.text)

What am I doing wrong?
How can I ask for concrete objects afterwards? Like count_likes, or retweets?
Translated with www.DeepL.com/Translator (free version)

The API v2 interface is the Client class and the pagination interface for it is Paginator. See #1535.
API.search_full_archive uses the premium search endpoints, part of API v1.1, not v2.
This is unrelated to this issue.
You need to specify your environment label. lang is not a valid parameter for the method/endpoint, and you should not assume that any exception raised is a connection error.

The api-v2 branch / pull request #1535 has been merged into the master branch.

Client.like and Client.unlike have been added with https://github.com/tweepy/tweepy/commit/5beaa24e4c39b9c6777c87dd7e0eeed9a9ee8355.

Client.block and Client.unblock have been added with https://github.com/tweepy/tweepy/commit/18e0b1a37c36e270ccc9c50cb1343f03637f70e0.

hi @Harmon758 how to pick up one of these tasks and start contributing? I am new to contributing to open source projects. Could you point me to link or doc that can help me to get started?

Pull requests are welcome. You can find many resources for how to make pull requests and the best practices for that with a simple web search.
Otherwise, for Tweepy, there's not much additional information you need to know besides how to run tests, which is covered in the documentation.
As for Twitter API v2 support specifically, I plan on adding most of the new endpoints myself this week.
You should start a new discussion if you have further questions, as this isn't specifically relevant to this issue.

Client.get_liking_users, Client.get_liked_tweets, and Client.get_blocked have been added with https://github.com/tweepy/tweepy/commit/a32e6cea08d170e618d46eabc82d673074555c3d.

Client.get_all_tweets_count and Client.get_recent_tweets_count have been added with https://github.com/tweepy/tweepy/commit/49156cb9f1d74d709480b8d746b01e755f5c312b.

Client.unretweet, Client.get_retweeters, and Client.retweet have been added with https://github.com/tweepy/tweepy/commit/b4985e04acb87b791e4d0d3f45bd57e5410e7d48.

The alt_text field for Media has been added with https://github.com/tweepy/tweepy/commit/379e99a672537776ac0e160999967b5efce29305.

Newer endpoints and streaming will be added as part of later v4.x releases.
I want to get v4.0 out with initial support for Twitter API v2 and the tons of other changes it includes.

v4.0.0 (https://github.com/tweepy/tweepy/discussions/1666) has now been released with support for Twitter API v2 🎉

Support for Spaces, with a Space model, Client.search_spaces, Client.get_spaces, and Client.get_space, has been added with https://github.com/tweepy/tweepy/commit/5c688922cf0f8e89d401ea5108b06aaa8c12d71b.

v4.1.0 (#1684) has now been released with support for Spaces, batch compliance, and mutes lookup (as well as Python 3.10) 🎉

v4.2.0 (#1694) has now been released with support for managing lists and the state parameter change for Spaces search 🎉

Client.delete_tweet and Client.create_tweet have been added with https://github.com/tweepy/tweepy/commit/7884e3a7253d9a821ff46160ec0d3811f299327f.

v4.3.0 (#1701) has been released with support for the Twitter API v2 manage Tweets endpoints launched today 🎉

@Harmon758 I am interested in contributing. Though this would be my first time contributing to opensource. Any suggestions where to start?

Pull requests are welcome. You can find many resources for how to make pull requests and the best practices for that with a simple web search.
Otherwise, for Tweepy, there's not much additional information you need to know besides how to run tests, which is covered in the documentation.
As for Twitter API v2 support specifically, I'm working on adding support for the the last streaming endpoints.
You should start a new discussion if you have further questions, as this isn't specifically relevant to this issue.

A List model, Client.get_list_tweets, Client.get_list_followers, Client.get_followed_lists, Client.get_list, Client.get_owned_lists, Client.get_list_members, Client.get_list_memberships, and Client.get_pinned_lists have been added with https://github.com/tweepy/tweepy/commit/0aa2366a875464756507abf42709a3db676f4cee.

Client.get_space_buyers has been added with https://github.com/tweepy/tweepy/commit/8bf58ca5754f72292d1d86367a02778af7a44f20.

Space.ended_at and Space.topic_ids have been added with https://github.com/tweepy/tweepy/commit/c89a233a96142ce01ecd5e1372c3fdce45c6601f.

v4.4.0 (#1705) has been released with support for the new List lookup and Space endpoints, as well as the new Space object fields.

Oauth2Handler has been added with https://github.com/tweepy/tweepy/commit/2b835073cb193ca6f2849c8cb6ef4322e5b16f24, and Client.get_me has been added with https://github.com/tweepy/tweepy/commit/c49cbdfcbda48295591d731446cd03b2eb2332ae and https://github.com/tweepy/tweepy/commit/62b5b586e75a850427eabdf31448d73a9e564f66.

v4.5.0 (#1786) has been released with support for OAuth 2.0 Authorization Code Flow with PKCE and the new authenticated user lookup endpoint 🎉

Support for the new sort_order parameter for Client.search_all_tweets and Client.search_recent_tweets has been added with https://github.com/tweepy/tweepy/commit/bd202e5be19670116ebe98329d388b4821d5f3eb and https://github.com/tweepy/tweepy/commit/8b47170e7c20127add3e67190890d2a4cef92266, respectively.

Client.get_space_tweets has been added with https://github.com/tweepy/tweepy/commit/c8d5d9ad2c2b3c6b58b987286726cad467ce888f.

Support for streaming, with StreamingClient, StreamResponse, and StreamRule, has been added with https://github.com/tweepy/tweepy/commit/86244c1a82a1852d04f3695b03201363f5d5eafd.

Space.subscriber_count has been added with https://github.com/tweepy/tweepy/commit/1ffc8cda6d2b9670b86610859f21ac05d726eaf9.

v4.6.0 (#1824) has been released with support for streaming, the new Retweets lookup and Likes lookup endpoint parameters, the new sort_order parameter for search endpoints, and the new Tweets shared in a Space endpoint and RSVP count field for Spaces 🎉
With this release, Tweepy now fully supports all of Twitter API v2, almost exactly a year after the initial implementation for Twitter API v2 support was pushed. New individual issues will be created for any future new Twitter API v2 endpoints and features.
(The COVID-19 stream is the only remaining active portion of Labs and support for it is being tracked in #1790.)


Installs but doesn't work on GoLand 2018.3.3

Taken from review

@x1unix I saw your review comment about this not working in GoLad 2018.3.3.  Could you provide more details about the issue?  My guess is you aren't seeing the theme show up since the readme is setup for version 2 which changed how to install the theme due to changes to the IntelliJ that changed how themes are setup in order to support custom UI themes.
I'll add some information to the installation docs for how to install the legacy version (v1) of this plugin.  Hopefully that will fix your issue.

I have updated the Wiki to include legacy installation steps.

@markypython theme does not appears in Settings | Appearance section on Windows.
On Linux (Ubuntu 18.10) everything is fine

Here is settings screenshot. One Dark theme plugin is installed
Environment:
GoLand 2018.3.3
Build #GO-183.5153.54, built on January 16, 2019
JRE: 1.8.0_152-release-1343-b26 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 10 10.0

@x1unix After seeing your review, I updated the installation docs to include information about the legacy version of this plugin.  The important difference is you must set the color theme in Preferences | Editor | Color Scheme rather than the appearance section.
Can you look in the color scheme section to see if the theme options are there?

@markypython color scheme appears but it changes only editor color scheme, not a whole UI

That is expected.  The UI customizations are only available on IDE versions 2019.1 and above.  See the original blog post by Jetbrains for more details.

If you would like the UI theme you will need to upgrade your IDE to the latest version (2019.1.1).

@markypython understood. Closing the ticket

Thanks for posting your feedback as it gave me the opportunity to update the installation docs.  Hope you find this theme helpful and feel free to open any other issues for questions, bugs, or feature requests!

@markypython thank you for the theme! I'm using it almost on any IDE i have


How can we disable soft delete for a container using C#/ .Net?

How can we disable soft delete for a container using C#/ .Net?
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 84ac5495-fab7-02a9-37d9-dde1e463cb4a
Version Independent ID: 48fc73ee-bb7a-6175-e962-31731b795d31
Content: Use .NET to manage properties and metadata for a blob container - Azure Storage
Content Source: articles/storage/blobs/storage-blob-container-properties-metadata.md
Service: storage
GitHub Login: @pauljewellmsft
Microsoft Alias: pauljewell

@babarkhalid
Thanks for your feedback! We will investigate and update as appropriate.

Hi @babarkhalid
To disable soft delete for a container using C#/.NET, you can use the SetMetadataAsync method of the BlobContainerClient class.You can set the deleted metadata property to true to disable soft delete. Here's an example code snippet:
BlobContainerClient containerClient = new BlobContainerClient(connectionString, containerName);
containerClient.SetMetadataAsync(new Dictionary<string, string>
{
{ "deleted", "true" }
});
Please note that disabling soft delete will permanently delete any blobs that were previously soft-deleted and have exceeded their retention period.
For more information;
https://learn.microsoft.com/en-us/azure/storage/blobs/soft-delete-container-overview

Hi @babarkhalid
To disable soft delete for a container using C#/.NET, you can use the SetMetadataAsync method of the BlobContainerClient class.You can set the deleted metadata property to true to disable soft delete. Here's an example code snippet:
BlobContainerClient containerClient = new BlobContainerClient(connectionString, containerName); containerClient.SetMetadataAsync(new Dictionary<string, string> { { "deleted", "true" } });
Please note that disabling soft delete will permanently delete any blobs that were previously soft-deleted and have exceeded their retention period.
For more information; https://learn.microsoft.com/en-us/azure/storage/blobs/soft-delete-container-overview

This is not working, please see the screenshot,

Here, I want to disable it,

@babarkhalid
As mentioned earlier, system containers, such as the $logs container, cannot have their soft delete settings modified. So you will not be able to disable soft delete for the $logs container using the code snippet I provided earlier.
To disable soft delete for a custom container using C#/.NET, you can modify the code snippet as follows:
BlobContainerClient containerClient = new BlobContainerClient(connectionString, containerName);
containerClient.SetMetadataAsync(new Dictionary<string, string> { { "deleted", "true" } }).Wait();
Replace connectionString and containerName with the connection string and name of your custom container, respectively.
Please note that disabling soft delete will permanently delete any blobs that were previously soft-deleted and have exceeded their retention period.

I managed to Disable Soft delete for Blobs and Container. Here's the code:
TokenCredential credential = new AzureDefaultCredential();
armClient = new ArmClient(credential);
ResourceIdentifier storageAccountResourceId = StorageAccountResource.CreateResourceIdentifier(subscriptionId,
resourceGroupName,
storageAccountName);
var storageAccount =  armClient.GetStorageAccountResource(storageAccountResourceId);
var blobService = storageAccount.GetBlobService();
blobService = blobService.Get();
var blobServiceData = blobService.Data;
//Disable Container Soft delete
blobServiceData.ContainerDeleteRetentionPolicy.IsEnabled = false;
blobServiceData.ContainerDeleteRetentionPolicy.Days = null;
blobServiceData.ContainerDeleteRetentionPolicy.AllowPermanentDelete = true;
//Disable Blob Soft Delete
blobServiceData.DeleteRetentionPolicy.IsEnabled = false;
blobServiceData.DeleteRetentionPolicy.Days = null;
blobServiceData.DeleteRetentionPolicy.AllowPermanentDelete = true;
        var armBlobOperation = blobService.CreateOrUpdate(WaitUntil.Completed, blobServiceData);


get security group policies fails to pass group ID

result = client_session.read('secGroupPolicies', uri_parameters={'ID': group_id})
Returns
send: 'GET /api/2.0/services/policy/securitygroup/%7BID%7D/securitypolicies HTTP/1.1\r\nHost: nsx.server.com\r\nConnection: keep-alive\r\nAccept-Encoding: gzip, deflate\r\nAccept: */*\r\nUser-Agent: python-requests/2.18.4\r\nAuthorization: Basic XXXXX=\r\n\r\n' reply: 'HTTP/1.1 404 Not Found\r\n' header: Date: Mon, 23 Jul 2018 16:14:57 GMT header: Server: header: X-Frame-Options: SAMEORIGIN header: Cache-Control: private header: Expires: Thu, 01 Jan 1970 00:00:00 GMT header: Cache-Control: no-cache header: Strict-Transport-Security: max-age=31536000; includeSubDomains header: X-Frame-Options: SAMEORIGIN header: Content-Type: application/xml;charset=UTF-8 header: Keep-Alive: timeout=15 header: Connection: Keep-Alive header: Transfer-Encoding: chunked
Specific error
{'error': {'errorCode': '202', 'moduleName': 'core-services', 'details': 'The requested object : {ID} could not be found. Object identifiers are case sensitive.'}}

The variable here /%7BID%7D/ is not being interpolated


Add a 'back to top' button in the README.md file

I want to add a "back to top" button at the bottom of readme file which will scroll to top on clicking.

interested


wit-bindgen: 0.17.0 -> 0.19.1

Description of changes

wit-bindgen: migrate to by-name
wit-bindgen: 0.17.0 -> 0.19.1
Update wit-bindgen to the latest release.
https://github.com/bytecodealliance/wit-bindgen/releases/tag/v0.19.1

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[x] Fits CONTRIBUTING.md.

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

Hi @marsam, please help me review this PR, thank you so much.


Remove items from payload dictionary if their value is None

Create class method from following code-block and apply to all functions so that they are removed from the URL query when sent to iRacing.
for key in payload.copy():
            if payload.get(key) is None:
                del(payload[key])
            else:
                pass

Functions need to be tested for proper functionality afterwards. It is known that a value of -1 is equal to the key being set to nothing key= but it is unknown if queries will still work if the key is removed entirely.

Confirmed for event_results() that removing raceweek= did not affect results. (This was an endpoint that had raceweek= in the URL query when using iRacing's "My Results" page)


com.google.cloud.Identity does not support the new IAM deleted user syntax

The new IAM deleted user syntax is not supported by com.google.cloud.Identity:
https://cloud.google.com/iam/docs/policies#handle-deleted-members
Environment details

Affects the Core library and all dependant libraries
OS type and version: any
Java version: any
google-cloud-core version(s): any

Steps to reproduce

Try to parse a deleted user string with Identity.valueOf
This results either to a stacktrace (1.74.0 and lower) or a DELETED identity type (1.75.0 and higher)

Code example
Identity identity = Identity.valueOf("deleted:user:donald@example.com?uid=234567890123456789012");
System.out.println("type = " + identity.getType());
System.out.println("value = " + identity.getValue());

Stack trace
1.74.0 and lower
java.lang.IllegalArgumentException: Illegal identity string: "deleted:user:donald@example.com?uid=234567890123456789012"

1.75.0 and higher
type = DELETED
value = user:donald@example.com?uid=234567890123456789012

The "fix" is caused by an unrelated change: https://github.com/googleapis/java-core/commit/24667e5c27ab71bd217d6667c1cb0affefe09b13
External references such as API reference guides
https://cloud.google.com/iam/docs/policies#handle-deleted-members

@clementdenis
The fixed was already made here, so i would recommend you to used 1.75.0 and higher.
FYI, The output of the split can not be longer than 2, It will throw an IllegalArgumentException if there's more parts than we expect.

i am closing this issue, as it's not able to reproduce in latest version.


Bug/addguard-integration

Thank you for contributing to Homarr! So that your Pull Request can be handled effectively, please populate the following fields (delete sections that are not applicable)
Category
Bugfix
Overview

Fixes dnsHole toggle not working
Ads Button change status on single app
Implements logic so the request it not send for an app if it is already on that status
Adds a blue feedback green/red status dot. It will turn red when the app is disable, green when enable, and blue when there is a fetch that wil update the that app (it wont change to blue if the fetch is not related to the app),

Issue Number (if applicable)

Related issue: #00

New Vars (if applicable)

If you've added any new build scripts, environmental variables, config file options, dependency please outline here.

Screenshot (if applicable)

If you've introduced any significant UI changes, please include a screenshot.

@taos15 @Tagaishi i the issue now resolved? Can we merge?

Yes
On Tue, Aug 29, 2023 at 13:23 Manuel @.***> wrote:

@taos15 https://github.com/taos15 @Tagaishi
https://github.com/Tagaishi i the issue now resolved? Can we merge?
—
Reply to this email directly, view it on GitHub
https://github.com/ajnart/homarr/pull/1306#issuecomment-1697853175, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A5DZCKHDOY4PNFZVX3KI25DXXYQPZANCNFSM6AAAAAA3ZICZNA
.
You are receiving this because you were mentioned.Message ID:
@.***>

We've been asked on discord already and we said yes

Sorry that this took long. Thank you for your contribution anyway! We appreciate it a lot 🎉🥂