BLE client fails to get serviceUUID

First of all, thanks for sharing the library and all the contribution you made at arduino-esp32.
I'm testing BLE_server and BLE_client from your ESP32_BLE_Arduino repo with two of my ESP32 devices, one as a client, another as a server.
I copied the service uuid and characteristic uuid written on the server example and replaced the ones on the client example with them, so that the client can find the server.
On my serial monitor, I could fine a line

BLE Advertised Device found: Name: MyESP32, Address: ...

which would apparently came from
https://github.com/nkolban/ESP32_BLE_Arduino/blob/cb1ab4ea76af17d347cdadcc003d1fe53af12aa7/examples/BLE_client/BLE_client.ino#L74-L75
but doesn't seem to have reached at
https://github.com/nkolban/ESP32_BLE_Arduino/blob/cb1ab4ea76af17d347cdadcc003d1fe53af12aa7/examples/BLE_client/BLE_client.ino#L81
I looks like the if statement right before that returned false.
So I added some lines like these

Serial.print("haveServiceUUID: "); Serial.println(advertisedDevice.haveServiceUUID());
Serial.print("device's UUID: "); Serial.println(advertisedDevice.getServiceUUID().toString().c_str());

to somewhere around
https://github.com/nkolban/ESP32_BLE_Arduino/blob/cb1ab4ea76af17d347cdadcc003d1fe53af12aa7/examples/BLE_client/BLE_client.ino#L76
and this is what I got (I have replaced the <MAC_ADDRESS> part, it was correctly shown on the serial monitor)

BLE Advertised Device found: Name: MyESP32, Address: <MAC_ADDRESS>, txPower: -21
haveServiceUUID: 0
device's UUID: <NULL>

I don't have much experience in BLE environment.  Maybe failing to get a service uuid is not a rare thing to happen in the first place.
Could you tell me if those two examples are still working at your environment and if so, is there anything I can try to get this working?

Hello my friend.  I'll be delighted to work with you to get these going.  There is lots we can try and do.  First of all there has been a lot of churn in this area recently so I want to be sure that we are all using the same code base.  The owners of the Arduino-ESP32 library now include the BLE-C++ code with their distribution.  This means that we should be exclusively using the distribution that comes with Arduino-ESP32 library.
I too have a box of ESP32 to play with.  Can I ask you to "pastebin" the two scripts (one that will be running on ESP32 number 1 and the other running on ESP32 number 2).  Add the links to the pastebin to this issue.  When I get them, I'll grab them and run up two ESP32s to try and recreate the same issue and then I'll be 100% on the same page as you.  I'll be around this evening and all weekend to work on it.  We'll use your two scripts as the common place against which we can talk.
If you could also pastebin the serial logs exactly as you see them from both ESP32s that would also be useful.

Im trying to run my old test BLEServer sample programs and they dont have any characteristics advertised. Something got changed lately in BLE stack?

@chegewara In ESP-IDF or in Arduino?

Plain esp-idf. I know we are working in this issue in Arduino, but if i can make BLEServer working with BLEClient under esp-idf then i can move to Arduino IDE to ckeck if it works. But now BLEServer dont advertise any characteristics under esp-idf.

Hi @weizenbock, with @nkolban coop ive been able to run SamleServer and SampleClient under esp-idf. Now, can you edit sdkconfig file and change those lines to match this:
CONFIG_OPTIMIZATION_LEVEL_DEBUG=
CONFIG_OPTIMIZATION_LEVEL_RELEASE=y

See also problem #109
I think I'm going to consider this issue  a duplicate of the primary (#109).  As such, tracking the resolution to #109 to conclusion.  Please post new findings on this issue in #109 as I think they will apply there too.  There is also a lot more detail on what we tried and where we are there.

Thanks for the help.
I wasn't sure if you also wanted me to move to #109.  My issue looks different than that of @chegewara.  So I'll just keep using this thread.  Let me know if you want otherwise.
I actually can see the service uuid on my Android app (called BLE Scanner).
So the server looks working ok.
Here are the codes I'm using.
BLE_server
https://pastebin.com/3Cak8jsv
BLE_client
https://pastebin.com/2Dxr56Xz
Serial log
https://pastebin.com/qeTjuGwy

Your SampleServer is not advertising any services hence client cant find one. You have to add
pAdvertising->addServiceUUID(pService->getUUID()); after line 31 and before line 32

@weizenbock Did you shorten server name? Check with android app if you can see any advertised service. And it has to be advertised before you connect to server, not service seen after connecting

I'm not sure what you meant by "shortening server name" but I haven't changed anything other than the line you gave me since I pasted it on the pastebin.
If you are talking about this line,

BLEDevice::init("MyESP32");

I can find "MyESP32" on the list of my Android app and its "connect" button gets enabled.
However I cannot find any service uuid on the list.  I can only see it after I hit "connect" button.

Try with BLEDevice::init("MyESP"); or BLEDevice::init("ESP32");

You have to see advertised service before conecting to esp32 like on picture

Yes that worked!!! @chegewara
So is there a limit for the length of server name??

Yes, there is limit. Because you are trying to find and connect to a device with custom UUID, which is 128 bit long and has to be advertised, and bluetooth limitation to send advertising packet is 31 bytes the name is limited to about 5 characters. That depends if you are advertising more info or not.

Good call mr Weizenbock ... it turns out that your problem wasn't the same as #109 so your choice to keep posting here was 110% correct.

Love and Kudos to mr @chegewara  ... you are awesome!

Its not my "fault", you have made very good logging, easy to debug.

Oh I see.  If I chose to use custom uuid then it will take so much space in the packet.  That makes sense.
Now I close the issue.  Thank you all!

Hello @nkolban,
I had faced the same issue as @weizenbock  when I was testing BLE_server and BLE_client in Arduino IDE with 2 ESP32 DevKitC devices (one as the server and another as the client). It worked only after I made changes to the BLE_server code as suggested by @chegewara i.e., to reduce the length of device name to 3 (5 characters device name didn't work for me) and adding "pAdvertising->addServiceUUID(pService->getUUID());" to the code.
I was wondering if you could please update the code for BLE_server in ESP32_BLE_Arduino repo so that in future no one else faces this same issue.
Thanks

You have created examples for the Arduino IDE. Your BLEClient instance does not connect to your BLEServer instance. Try with any esp32. Codes are not modified. So the one-to-one code in the samples.
First, an Esp32 is loaded with BLEServer.
Then another Esp32 was loaded with BLEClient.
The advertisedDevice.haveServiceUUID () function in the BLEClient instance returns false. However, a Service ID was assigned to the BLEServer instance. But BLEClient doesn't see it.
Have you reached a solution?

Hello @nkolban
You have created examples for the Arduino IDE. Your BLEClient instance does not connect to your BLEServer instance. Try with any esp32. Codes are not modified. So the one-to-one code in the samples.
First, an Esp32 is loaded with BLEServer.
Then another Esp32 was loaded with BLEClient.
The advertisedDevice.haveServiceUUID () function in the BLEClient instance returns false. However, a Service ID was assigned to the BLEServer instance. But BLEClient doesn't see it.
Have you reached a solution?

Hello @nkolban,
I'm just starting with ESP32 and BLE on Arduino-IDE, using your BLE-Client example to connect to an iSteady pro gimbal. Unfortuanatly the gimbals name is "Gimbal_14", and I run into the problems shown above. Is there any hope that there will be a solution?

Have same problem with lenght of server name. Im correct understand - there is no solution for this lib?


bibtex case sensitive tags

Bibtex/text doesn't parse uppercase tags correctly.
Example:
Cite.input('@article{test, Author = "test"}')
creates a bibtex citation without the author field, whereas
Cite.input('@article{test, author = "test"}')
works as expected.

Yes this is strange. I used the following to take care of it before passing the string
 stringVar.replace(/Abstract =/g, "abstract =")             .replace(/Author =/g, "author =")             .replace(/Title =/g, "title =")             .replace(/Year =/g, "year =")             .replace(/= {{/g, "= {")             .replace(/}},/g, "},");

This issue should be fixed in v0.5.0-alpha.10. If not, please re-open the issue.

This issue should be fixed in v0.5.0-alpha.10. If not, please re-open the issue.


Add other pages images and content for men, kid pages

Can you assign me this task to create men or kids page for your website under SSOC'24

@JagjeetChauhan  What are the updates?

@Kritika30032002 I am working on it but I need some time, as my semester exam have started

when you can submit pr for this? @JagjeetChauhan

by Friday, confirm

@JagjeetChauhan updates?

I will make the PR today.

@JagjeetChauhan


Chrome -> iOS OWR call setup fails with ICE failure

Trying to call iOS OWR (develop-branch Matrix iOS console) from Chrome (develop-branch Matrix webclient) results in ICE failure.  iOS->Chrome works however.

(this tested against the matrix.org homeserver & coturn deployment, with both clients sitting on the same subnet)

I can reproduce this. chrome://webrtc-internals is so horribly unusable it makes me :cry:

I've tested with the http://demo.openwebrtc.io test code (from the openwebrtc-examples repo - there's a simple node server in the web directory) and NativeDemo using the Matrix TURN servers. I called Chrome to NativeDemo and it works fine.
I suspect this is a problem with either the OWR iOS SDK integration in matrixConsole or something with how the WebRTC API usage in the web app works in Chrome in conjunction with the OWR iOS SDK. I'm almost inclined to suspect the latter. Perhaps you guys can compare your web app WebRTC code with our demo code to see if there's anything particularly egregious?


fix Width and height for svgs

https://taskman.eionet.europa.eu/issues/274485

moved https://github.com/eea/volto-embed-static-content/pull/8


Avoid redefinition of bit_scan{forward,reverse} macros

Fixes #247.

How's this version? I tried to run tests, but ran into
[ 10%] Building CXX object tests/CMakeFiles/logarithm_avx2.dir/logarithm.cpp.o
In file included from /Users/sftnight/amadio/Vc/tests/logarithm.cpp:28:
/Users/sftnight/amadio/Vc/tests/unittest.h:30:10: fatal error: 'virtest/vir/test.h' file not found
#include "virtest/vir/test.h"
         ^~~~~~~~~~~~~~~~~~~~
1 error generated.

which I don't know how to fix. What's providing this header? I guess it's an external dependency, but CMake configured without errors, so I'm confused.

Hi @mattkretz, I cannot see what the error was, so I don't know if it's related to my changes. Can you please let me know?

Ping, @mattkretz please! :-)

@amadio Google says it's https://github.com/mattkretz/virtest - could you try with that, please?

How's this version? I tried to run tests, but ran into
[ 10%] Building CXX object tests/CMakeFiles/logarithm_avx2.dir/logarithm.cpp.o
In file included from /Users/sftnight/amadio/Vc/tests/logarithm.cpp:28:
/Users/sftnight/amadio/Vc/tests/unittest.h:30:10: fatal error: 'virtest/vir/test.h' file not found
#include "virtest/vir/test.h"
         ^~~~~~~~~~~~~~~~~~~~
1 error generated.

which I don't know how to fix. What's providing this header? I guess it's an external dependency, but CMake configured without errors, so I'm confused.

The header comes from a submodule (of the repo that Axel posted), the README mentions git submodule update --init.
After doing this, I was able to run the tests and all pass with this change (GCC 8.3.1 and Clang 9.0.1 from CentOS 8) while making the build with Clang free of warnings. Can we merge this one? @mattkretz

Can we merge this now?

Looking at it now. Probably yes. :-)

Can we use C++20? :stuck_out_tongue_winking_eye:
The bsr inline asm for clang is scary. I'm sure we can use a __builtin nowadays. The PR seems to be a strict improvement, but I'd rather rewrite this header altogether. I'll merge it for now and might push a cleanup later.


Added unit test for long tokens

Closes #1634

Codecov Report

Merging #1635 (5afb1fe) into main (94313b6) will increase coverage by 0.02%.
The diff coverage is 100.00%.

:exclamation: Current head 5afb1fe differs from pull request most recent head c278f0f. Consider uploading reports for the commit c278f0f to get more accurate results

@@            Coverage Diff             @@
##             main    #1635      +/-   ##
==========================================
+ Coverage   93.95%   93.97%   +0.02%
==========================================
  Files         253      253
  Lines       47615    47649      +34
==========================================
+ Hits        44735    44777      +42
+ Misses       2880     2872       -8

Impacted Files
Coverage Δ

src/indexer/segment_writer.rs
97.52% <100.00%> (+0.46%)
:arrow_up:

src/reader/mod.rs
93.63% <0.00%> (-0.64%)
:arrow_down:

src/fastfield/multivalued/mod.rs
99.22% <0.00%> (+0.77%)
:arrow_up:

src/directory/error.rs
6.55% <0.00%> (+3.27%)
:arrow_up:

src/error.rs
30.15% <0.00%> (+4.76%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


jdk8: Remove default java.library.path

Motivation for this change
This patch fixes https://github.com/NixOS/nixpkgs/issues/103493 for JDK8 only.
I'm upstreaming only for JDK8 to get quorum on the approach and then
adding the patches to the remaining versions.
❯ ./result/bin/java -XshowSettings:properties
Property settings:
    awt.toolkit = sun.awt.X11.XToolkit
    file.encoding = UTF-8
    file.encoding.pkg = sun.io
    file.separator = /
    java.awt.graphicsenv = sun.awt.X11GraphicsEnvironment
    java.awt.printerjob = sun.print.PSPrinterJob
    java.class.path = .
    java.class.version = 52.0
    java.endorsed.dirs = /nix/store/nhlpy8zcxncw52ylmk8aacii3h00cmf0-openjdk-8u272-b10/lib/openjdk/jre
    java.ext.dirs = /nix/store/nhlpy8zcxncw52ylmk8aacii3h00cmf0-openjdk-8u272-b10/lib/openjdk/jre
    java.home = /nix/store/nhlpy8zcxncw52ylmk8aacii3h00cmf0-openjdk-8u272-b10/lib/openjdk/jre
    java.io.tmpdir = /tmp
    java.library.path = (null)

See  java.library.path = (null)
Previously it was:
java.library.path = /usr/java/packages/lib
        /usr/lib/x86_64-linux-gnu/jni
        /lib/x86_64-linux-gnu
        /usr/lib/x86_64-linux-gnu
        /usr/lib/jni
        /lib
        /usr/lib

Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

I was hoping this solves my JRuby bug which it does not. JRuby uses https://github.com/jnr/jnr-ffi which also pulls in some system libraries for now (chatting with @headius).
The patch is still meaningful and helps isolate JDK but there's more fish to fry here for the linked bug if we want to get away from having to manually specify LD_LIBRARY_PATH

@SuperSandro2000 thanks for looking at it.
I feel fairly confident this is the right approach to help guarantee hermeticness for the JDK platform.
I'm just a bit shy here on pulling the gun because the JDK is so pervasive.
I would appreciate another +1 from another JDK maintainer.

@edwtjo thoughts?

@edwtjo thoughts?

I like it and it should be adopted and applied for the other JVM versions as well, i.e., good find good fix

@edwtjo I'll work on JDK11 next.


chore: release 1.2.2

:robot: I have created a release *beep* *boop*
1.2.2 (2019-06-26)
Bug Fixes

docs: link to reference docs section on googleapis.dev (#229) (2ec1152)

This PR was generated with Release Please.

Release is at https://github.com/googleapis/nodejs-bigquery-data-transfer/releases/tag/v1.2.2

The release build has started, the log can be viewed here. :sunflower:

:egg: You hatched a release! The release build finished successfully! :purple_heart:


2262 add cisd date vars to create_formatted_datetime_string_columns

Pre-review checklist
I have ensured that:

[ ] Code runs successfully in the development environment.
[ ] New code is tested to prove that it works. If not, justify reasoning.
[ ] New code is documented using type hinting and following the numpydoc docstring format.
[ ] New code follows the project naming standards.

Overview of changes

Post-review checklist
As I merge these changes, I have:

[ ] Updated the related Jira ticket, including approximate time spent (including review).
[ ] Moved the related Jira ticket to Done.

Closing pull request to clean branches, this inadvertently incorporates commits from 2243.


Several warnings about the component radio (Cannot read properties of undefined (reading 'type'))

Steps to reproduce
Use the latest version and SSR.
What is expected?
No warnings / errors.
What is actually happening?
I get the following warning in the console every time I log in to the application.
1 rules skipped due to selector errors:
  span.ant-radio * -> Cannot read properties of undefined (reading 'type')

Environment
Info

ng-zorro-antd
13.1.1

Browser
Microsoft Edge Versión 100.0.1185.29

解决了吗

解决了吗

It happens with SSR.

@ljfass It happens with SSR.

@vthinkxie You can help? Sorry about the ping

看起来像是 https://github.com/angular/universal/issues/2627 引起，等修复

@cipchk A commit https://github.com/angular/universal/commit/35a477e115d52fdad13e993753ad136f0466af1e was already implemented and a new version was released and the problem persists.

@kaname-png Delon 之前也有出现这个错误，但是将 Angular 升级至 13.3.3 以后不再出现了。

@cipchk I have the latest version of angular and the problem persists.

您好，请问您后面是怎么样解决的？求助

@kaname-png

Did this get fixed? I'm seeing it on v14

Same for me, also having issue with icons

我也出现了

I'm also seeing this along with the warning..

any solution?

ping

+1
Angular: 15.2.0
ng-zorro-antd: 15.0.3

In server.ts file Try this:
app.engine('html',  ngExpressEngine({
      bootstrap: AppServerModule,
     /*
      *@DISABLE inlineCriticalCss
      */
      inlineCriticalCss: false,
    }))

+1
Angular 15
ng-zorro-antd: 15.1.0


[TASK] Prefix native function calls with a backslash

This slightly improves performance.

IMHO the ugliness on display here is not worth any performance gains.

It also helps clarify to the reader that function being called is in the global namespace (and thus likely to be a built-in PHP function), rather than in the current namespace or some other namespace being used (without having to look that up or have encyclopaedic knowledge).  (It's true we don't have any classless functions, or AFAIK use any from another library, but the reader may not know that.)
(Ugliness is in the eye of the beholder.  I find these a thing of beauty.  They satisfy my inner OCD ;))


test: ensuring the embedded UI works

I'm running PM-mode just to have access to DB.
And then running devmode with no auth and a different port and doing the basic requests

thanks!

Maybe make a test Context to make testing against a running instance in the future easier also?

no idea if that is easy but sounds like a great idea :+1:

pm-mode-only will be a bit easier than running 2 instances...(thinking on testContext)
the second trustd is running after (not in parallel with) pm-mode to avoid this error constraint "pg_type_typname_nsp_index" https://stackoverflow.com/questions/54351783/duplicate-key-value-violates-unique-constraint-postgres-error-when-trying-to-c


Can code reloading work without stack

It would be nice to be able to enjoy the code reloading without using stack. Perhaps just replacing "stack ghc ..." with "ghc ..." would work. An option for this would be super.

Hi Joe.
Added a flag for specifying the location of 'ghc'. When the flag is enabled, ghc will be used directly instead of going through stack. Let me know if it works for your setup.

I'll give it a try. Thanks!
On Fri, Jun 12, 2020, 6:13 PM David Himmelstrup<EMAIL_ADDRESS>wrote:

Hi Joe.
Added a flag for specifying the location of 'ghc'. When the flag is
enabled, ghc will be used directly instead of going through stack. Let me
know if it works for your setup.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Lemmih/reanimate/issues/91#issuecomment-643193118,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGRJXCTOLADWZ2NYKSGJFTRWH5RPANCNFSM4N4ACQAQ
.

Works very well!


Update software-setup.rst

Corrected the bash formatting and added sizing precisions to the container file creation

Thanks @tommydoucet!


feat: Support underscores in numbers

Completes https://github.com/PRQL/prql/issues/1451
I haven't updated all the various grammars, I think that doesn't need to block (there will be some false errors but no false successes)
The pest is actually kinda difficult; expressing "can't end with an underscore".

Very cool!
I was wondering if for your small example you might want to use 0.000_000_1 rather than 1.000_000_1? Of course that could also be expressed as 1e-7 so perhaps your example is better.

That's what I had — but unfortunately sqlparser-rs messes up the formatting... (Something we should work out separately)


[FEATURE] Jobs.create method should also accept a JobDefinition

Problem Statement
The jobs.create() method only accepts the individual settings as parameter. This is especially annoying when having job settings as json/yaml or dict, because they have to be mapped to the parameters
Proposed Solution
The jobs.create() method should accept an object of JobDefinition just as the jobs.reset method does.
job_settings = JobSettings.from_dict(job_settings_dict)
w.jobs.create(job_settings)

Additional Context
Likely other methods, not only for Jobs would benefit from an easy way to send json/yaml/dicts to the API without mapping it to method parameters.

Hi @mniehoff, thanks for reporting this. For context, the API of the SDK is meant to match the REST API nearly exactly, which you can see here: create a job, reset a job. However, to make this use case easier, our plan is to change the SDK to support plain objects/strings in place of dataclasses/enums. Then, you should be able to do the following:
w.jobs.create(job_settings_dict**)

using kwargs from your settings dictionary for each of the parameters of the create API.

That would be great and i guess also more pythonic than dataclasses :-)

Is there any update or alternative for this? Currently it stops us from adopting the sdk, as we have stored job and cluster config in yaml/json files and we would need map each attribute when using the create methods.

Here's a dirty workaround:
job_settings = JobSettings.from_dict(job_settings_dict)
w.jobs.create(**job_settings.__dict__)

Is there any update on this?


Failed to get /openapi/v2 and /swagger.json: Created Component, but require templated one.

Hello all:
v0.8.2 had some unfortunate side-effect on us. The fix #335 broke kubernetes-external-secrets
Error: Failed to get /openapi/v2 and /swagger.json: Created Component, but require templated one.

Only way to get it back to working was reverting to v0.7.0 and deleting the apiservice v1beta1.custom.metrics.k8s.io
Tickets:
external-secrets/kubernetes-external-secrets#563
silasbw/swagger-fluent#55
The fix should be done by silasbw/swagger-fluent#55 correct?

Hello @anden-dev,

The fix should be done by silasbw/swagger-fluent#55 correct?

After going through the various links you provided, I would say so.
So far, we've not seen any other issue with the content of the openapi spec that is served by prometheus-adapter and the error you are seeing seems to induce a bug with the openapi client you are using. My hunch mainly comes from a similar issue already existing there and the fact that the error message your are seeing contains:

This is a bug. Please report: https://github.com/silasbw/fluent-openapi/issues

Reverting back to v0.7.0 should indeed fix your problem as we only started to serve openapi spec in v0.8.0. However, note that you might stumble upon #254 in the process.

Thought as much.
There is a rewrite in go coming anyhow which will also mean a new openapi client external-secrets/external-secrets.
Thanks for your input.


Add an easy to digest description of how vaults earn yield

The product page for yVaults is a good start, but it's missing a description of how vaults actually earn yield. If there's a good description somewhere else in the documentation, it should be crosslinked to the product overview page.
This should address users who drop into discord and ask questions like

Hey, where do the rewards for stablecoins come from when you deposit in the iron bank or when you supply them in the vault?

so been trying to figure out without understanding the difference between yearn vault vs yearn iron bank? What risks are also associated with either one?

Documentation should address:

How vaults earn yield
The difference between depositing into a yearn vault and depositing into aave/iron bank
The difference between Yearn and Beefy

I tried to fix the first 2 points, I don't have the knowledge atm to do the last one, will go after it! https://github.com/yearn/yearn-devdocs/pull/155

Some resources on beefy https://medium.com/beefyfinance/whats-a-yield-optimizer-efed7f5aee6f
https://docs.beefy.finance/moo/ecosystem/products/vaults
https://docs.beefy.finance/moo/faq/infographics

No major differences found between Beefy, other than higher risks (multi-chain vulnerabilities, lack of source code history) https://www.defisafety.com/pqrs/38

The product page for yVaults is a good start, but it's missing a description of how vaults actually earn yield. If there's a good description somewhere else in the documentation, it should be crosslinked to the product overview page.

Descriptions are here: https://vaults.yearn.finance/

This should address users who drop into discord and ask questions like

Hey, where do the rewards for stablecoins come from when you deposit in the iron bank or when you supply them in the vault?

so been trying to figure out without understanding the difference between yearn vault vs yearn iron bank? What risks are also associated with either one?

IronBank has nothing to do with vaults. This should be in IronBank's docs or somewhere else.

Documentation should address:

How vaults earn yield
This is here: https://vaults.yearn.finance/

The difference between depositing into a yearn vault and depositing into aave/iron bank
The difference between Yearn and Beefy

We don't have to mention other protocols. We explain ours. I don't even know what beefy is.

@saltyfacu I think this one was reopen by accident?


bot.onText and messages offset

Clarifying that I'm new with telegram Bot APIs, I understood that when you start the bot, a polling is started. To match the text, I'm not using .getUpdates() but .onText() with the regex as parameter. Each time you make a request towards bot:token/getUpdates command, as implemented into the same-name method, you can pass an offset value as POST parameter to not get old messages. Is there a way to update this offset when I use .onText() every time I get an answer from the user or another way I can use to not receive old messages? Thank you.

Solved... with a big facepalm.


[tool_crash] StateError: Bad state: No context with the running Dart application.

Command
flutter --no-color run --machine --track-widget-creation --device-id=chrome --start-paused --no-sound-null-safety --dart-define=flutter.inspector.structuredErrors=true lib\main.dart

Steps to Reproduce

...
...
...

Logs
StateError: Bad state: No context with the running Dart application.
#0      RemoteDebuggerExecutionContext.id (package:dwds/src/debugging/execution_context.dart:55:7)
<asynchronous suspension>
#1      AppInspector._getExtensionRpcs (package:dwds/src/debugging/inspector.dart:538:20)
<asynchronous suspension>
#2      AppInspector._initialize (package:dwds/src/debugging/inspector.dart:97:34)
<asynchronous suspension>
#3      AppInspector.initialize (package:dwds/src/debugging/inspector.dart:160:5)
<asynchronous suspension>
#4      ChromeProxyService.createIsolate (package:dwds/src/services/chrome_proxy_service.dart:216:18)
<asynchronous suspension>

[✓] Flutter (Channel stable, 2.10.2, on Microsoft Windows [Version 10.0.19042.1526], locale en-US)
    • Flutter version 2.10.2 at C:\src\flutter_windows_2.10.2-stable\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 097d3313d8 (5 days ago), 2022-02-18 19:33:08 -0600
    • Engine revision a83ed0e5e3
    • Dart version 2.16.1
    • DevTools version 2.9.2

[!] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at C:/Users/khaldoun10/AppData/Local/Android/Sdk/jdk-17/bin
    • Platform android-32, build-tools 32.1.0-rc1
    • ANDROID_HOME = C:/Users/khaldoun10/AppData/Local/Android/Sdk/jdk-17/bin
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✗] Visual Studio - develop for Windows
    ✗ Visual Studio not installed; this is necessary for Windows development.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components

[✓] Android Studio (version 2020.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] VS Code (version 1.64.2)
    • VS Code at C:\Users\khaldoun10\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.32.0

[✓] Connected device (2 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 98.0.4758.102
    • Edge (web)   • edge   • web-javascript • Microsoft Edge 98.0.1108.56

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 2 categories.

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: master
Creation framework version: bc6fd9e17daa803742dc78ef8c3001fde99d37b3
Plugins
firebase_auth-3.3.7
firebase_auth_web-3.3.7
firebase_core-1.12.0
firebase_core_web-1.5.4

Duplicate of #95916


Allocate work array in LAPACKE ?TGSEN when ijob is zero (Reference-LAPACK PR 733)

this should have been "PR 773" rather than 733


Add function to convert whole dictionaries

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (76883ab) compared to base (6d79e9c).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #200   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines            9         9
=========================================
  Hits             9         9

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Make shift+enter create a new line wihout indentation.

Feature request.
Example now:
	This is an indented line (press shift-enter)
	(the cursor is new here)

What I'd like it to do:
	This is an indented line (press shift-enter)
(the cursor is new here)

I don't think this is possible because terminals don't make a difference between enter and shift-enter.


🐛Fix regex issue

Description
This pr:

Adds a .env.example file to make it easy for contributors to clone and use.
Fix regex issue(read more about it in the issue)

Fixes #102
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How Has This Been Tested?

[x] The shrink URL route was tested using postman.
[x] The redirection was also tested in the browser.

@shawshankkumar Is this code tested with the example URL in the issue #102?

Yes.


Torrents with subdirectories dont finish

Behaviour
Torrents with subdirectories, do not finish the download. They either stop or go to paused state.
Steps to reproduce this issue

Add torrent with subdirectories in it
Torrents finish every file execpt the subdirectory
e.g.

Expected behaviour

Finish the download to 100%

Actual behaviour

Stops at ~95 - 99% - dependend on the size of the torrent.

add. info what i've tested and found on the net

independent of the chunksize
double checked with torrents without subdirectories in it
checked if the user owner of the parent directory
umask is 022 so create subdirectories should work?!
This user seems to have the same problem Link
The user seems to have a simliar symptom (but that issue indicated to sonarr), but the last sentence is exactly my problem Link to Github issue

Configuration

Docker version (type docker --version) : 18.09.0-0506
Docker compose version if applicable (type docker-compose --version) :
Platform (Debian 9, Ubuntu 18.04, ...) :  Synology DSM
Include all necessary configuration files : docker-compose.yml, .env, ...

Docker info
sudo docker info
Password:
Containers: 1
 Running: 1
 Paused: 0
 Stopped: 0
Images: 5
Server Version: 18.09.6
Storage Driver: btrfs
Logging Driver: db
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
 Log: awslogs db fluentd gcplogs gelf journald json-file local logentries splunk                                                                                syslog
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 0cf16177dbb234350dc27dd2bbd1d7cebd098108
runc version: 6cc9d3f2cd512eeb3d548e2f6b75bcdebc779d4d
init version: e01de58 (expected: fec3683)
Security Options:
 apparmor
Kernel Version: 4.4.59+
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 11.55GiB
Name: monster
ID: VBXU:TKSU:VHCL:TNTK:MXSQ:OU7Z:MXDH:ESL7:3UTR:MNBG:QNDG:D4AL
Docker Root Dir: /volume1/@docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Labels:
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

WARNING: No kernel memory limit support
WARNING: No cpu cfs quota support
WARNING: No cpu cfs period support

Logs
1578046192 E F934DC88ED19C62A5B675FF922FBECFABF227089->file_list: Failed to prepare file '/Sample/.sample.mkv': Could not open file: No such file or directory
1578046192 E Could not resume download: Could not open file: No such file or directory

Hi @lordvalium, As you so aptly pointed out this problem is related to rTorrent or ruTorrent. Sorry but I don't bring support directly to them (see Closure policy section). You can find help on https://github.com/Novik/ruTorrent or https://github.com/rakshasa/rtorrent.

Hi @crazy-max  okay thank you i will do as proposed. Thx anyway. cheers


X files factor fixes

Rebased and cleaned version of #63
"It seems as if the method was left over after implementing a comparison using only two digits of the float values for xFilesFactor. This leads to migration of every single whisper file"

@esc: would you mind addressing @steve-dave's review comments and then merging this yourself? If not, I can merge it, but would prefer if you do address those.

I am totally out of this, sorry for dropping the ball on it. I'd prefer to not have to deal with it, TBH.

Are any objections for closing this?


[BUG]: bug in Stable-Diffusion, RuntimeError: expected scalar type Half but found Float

when i add a "callbacks", RuntimeError: expected scalar type Half but found Float
lightning:
trainer:
accelerator: 'gpu'
devices: 1
log_gpu_memory: all
max_epochs: 5000
precision: 16
auto_select_gpus: False
strategy:
target: strategies.ColossalAIStrategy
params:
use_chunk: True
enable_distributed_storage: True
placement_policy: auto
force_outputs_fp32: False
log_every_n_steps: 2
logger: True
default_root_dir: "/tmp/diff_log/"
profiler: pytorch

callbacks:
image_logger:
target: main.ImageLogger
params:
batch_frequency: 1
max_images: 4
increase_log_steps: False
log_first_step: False
#log_all_val: True
log_images_kwargs:
use_ema_scope: True
inpaint: False
plot_progressive_rows: False
plot_diffusion_rows: False
N: 4
unconditional_guidance_scale: 3.0
unconditional_guidance_label: [""]

can you post which layer you counter precise problem

there are so many layers, just like ddpm.py  line:1432

@Fazziekey The worst problem is that when i use your code to train a model, the txt2img.py outputs a picture with random noise. Could you tell me what should I do?

@yufengyao-lingoace hello, did you fix this error?

We have updated a lot. This issue was closed due to inactivity. Thanks.


Reuse compliancy

Make edgehog-esp32-device component compliant with
REUSE.

The email in this signature doesn’t match the committer email.


Usage of 3rd party API to fill in missing data

Data such as brawler mastery and ranked information is not available in the regular API by Supercell.
With a unofficial API such as hpdevfox-API the holes could likely be filled in.

Added to dev branch.


Ml Project Add

Issue Title
Add Most Important Project for Today's era #615
Describe the Add-ons or Changes You've Made
Give a clear description of what you have added or modified.
Describe your changes here.

[x] I have described my changes.

Type of Change
Select the type of change:

[x] New feature (non-breaking change which adds functionality)
[x] Code style update (formatting, local variables)

Checklist
Please confirm the following:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

@UTSAVS26   check now i make new branch !

@shaansuraj  can you please review PR!

@UTSAVS26  is there any conflicts?

@SimranShaikh20 create a detailed readme file for the project.

@UTSAVS26  can you pls change level  you had assigned as per work i had done !


[stable/rocketchat] Fixes compatibility with 1.16

Fixes compatibility with 1.16
To support 1.16. A quick overview of deprecated APIs can be found on their blog post: https://kubernetes.io/blog/2019/07/18/api-deprecations-in-1-16/.
Signed-off-by: Xiang Dai<EMAIL_ADDRESS>

/ok-to-test

/approve
/lgtm


ENG-4667 feat(portal): replace fetchers with react query hooks in follow modal

Affected Packages
Apps

[ ] data populator
[x] portal
[ ] template

Packages

[ ] 1ui
[ ] api
[ ] graphql
[ ] protocol
[ ] sdk

Tools

[ ] tools

Overview

Added useFollowMutation hook which handles the transaction and updating queries based on their queryKeys.
Removed fetchReval fetcher in favor of this new method.
Renamed GetStats query key to get-stats to follow standard.

Screen Captures
If applicable, add screenshots or screen captures of your changes.
Declaration

[x] I hereby declare that I have abided by the rules and regulations as outlined in the CONTRIBUTING.md

@Vitalsine85 -- I removed the do not merge label since this is onto the feature branch


Add ObjectStorage (and GCS implementation) as a registry provider

Add a way to get a file using credentials from a manager through registry to support private repos.
Add ability to create / update a registry through manager
Add support for Google Cloud credential type

@jackgr @technosophos
Can you guys take a look at these changes based on our discussions about the pending repo format. Ignore the test files for now please as I'm still working on them.

Nice PR. Really like the abstractions. Mostly minor comments but key question in my mind is alignment of the short url parsing with the code that @technosophos just submitted. Would prefer to see a common library used on both client and server to parse the various forms of chart reference.

Please rebase to remove conflicts.

Thanks :)
On Tue, Feb 2, 2016 at 6:33 PM, Jack Greenfield<EMAIL_ADDRESS>wrote:

Nice PR. Really like the abstractions. Mostly minor comments but key
question in my mind is alignment of the short url parsing with the code
that @technosophos https://github.com/technosophos just submitted.
Would prefer to see a common library used on both client and server to
parse the various forms of chart reference.
—
Reply to this email directly or view it on GitHub
https://github.com/kubernetes/deployment-manager/pull/214#issuecomment-178964309
.

This is looking good to me. I can see how we'd be able to easily hook in the client (in fact, I want to try that as soon as this PR gets merged). There are minor things that I think we'll be able to simplify once the big chart branch is merged, since that will give us a high-level abstraction for things like short/long names. But those shouldn't block this merge.
One open question for me (that influences the code I'm writing) is this: Should we be supporting the gs:// scheme directly? The spec just had http[s], file, and helm. But we could easily add gs, I think.
My view on this is that the sooner we get it merged, the better. Today I'm starting on a WIP to integrate the new chart format, and having this code checked in will streamline that for me.

@adamreese This ticket would open up a route to do h repo add, h repo list, and h repo rm.

Thanks! Yeah, my view on gs:// is simply that it and s3:// are in
established use today (gsutil, and boto?) Anyways, we can always change
this, And as you point out correctly, there are some questions out there on
how we merge some of the short/long names, and so on but from looking at
your CL I think it is going to slip in nicely. My view also is that getting
this in will allow us to start working those pieces independently.
On Wed, Feb 3, 2016 at 9:21 AM, Matt Butcher<EMAIL_ADDRESS>wrote:

This is looking good to me. I can see how we'd be able to easily hook in
the client (in fact, I want to try that as soon as this PR gets merged).
There are minor things that I think we'll be able to simplify once the big
chart branch is merged, since that will give us a high-level abstraction
for things like short/long names. But those shouldn't block this merge.
One open question for me (that influences the code I'm writing) is this:
Should we be supporting the gs:// scheme directly? The spec just had
http[s], file, and helm. But we could easily add gs, I think.
My view on this is that the sooner we get it merged, the better. Today I'm
starting on a WIP to integrate the new chart format, and having this code
checked in will streamline that for me.
—
Reply to this email directly or view it on GitHub
https://github.com/kubernetes/deployment-manager/pull/214#issuecomment-179358279
.

Thanks for the reviews, I think I addressed all the comments, PTAL at the questions. I'm working on rebasing / merging next and I'll send a note once that's done.

I believe I rebased this now. PTAL.

Ok, I think all the comments have been addressed. PTAL.

Merged.
Some clean up is possible around GithubRepositoryService, per the line comment there, but no reason to block @technosophos.
Ditto for Travis, which is having problems.

One thing I would like to see get a bit cleaner is the Doer. Per the line comment, it would be preferable to surface the chart/template abstractions.
Instead of returning download URLs that the caller has to GET via the registry, it would be cleaner to return the Chart TOC and make the caller ask for items by relative name. Then, the registry can turn that into a URL and go the GET behind the scenes.
I'll open an issue for this concern.


Does not change font after reload.

The font does not change after reload, i have tried multiple fonts, also does not show any warning about corrupted VS Code installation, which seems like it did nothing.
Using VS Code 1.84.2.
If any logs are required, do ask.

Can you provide details like what procedure you followed, what was font name and what actual input you gave , remeber it's case sensitive so you have to enter exact name.

Extention is updated, Do following steps:

Install VS code again even if it's already installed so that configuration will go to default.
Run Extension, enter exact name of font you want to have on you VS Code UI, make sure font is present in your system.


similar

https://github.com/lerenn/asyncapi-codegen looks pretty similar

Kafka and NATS
generates golang client and server

True, but this ones multilingual (kind of, since it's only Java). Not saying mines better though 😉 Maybe the other one suites my purposes as well, then maybe I'll drop this one.


Saving settings outgoing email on mailbox fails

Summary
Setting up a mailbox, configuring the outgoing mail. When clicking "Save Settings" I get an error
"Whoops, looks like something went wrong — check logs in /storage/logs"
Steps to reproduce
Create a new mailbox and define the outgoing mail settings.
Relevant logs and/or screenshots
No errors in the logs from the Docker containers.
Environment
Used the default docker-compose file

Image version / tag:  tiredofit/freescout:latest
Host OS: Ubuntu 20.04 (not that that would matter)

You should be able to find a log in /www/logs/laravel inside the container.
Also try setting DISPLAY_ERRORS=TRUE in environment variables and retry and we should get some detailed info.

Ah tnx
[2022-10-18 18:38:00] production.ERROR: preg_match(): Passing null to parameter #2 ($subject) of type string is deprecated {"userId":1,"email":"ron@example.org","exception":"[object] (ErrorException(code: 0): preg_match(): Passing null to parameter #2 ($subject) of type string is deprecated at /www/html/app/Http/Controllers/MailboxesController.php:337)
only replaced the domainname from the mail address

Hm. I'm not sure that's a docker image issue. Have you posted that at https://github.com/freescout-helpdesk/freescout ?
Inside the container is an environment variable PHP_BASE (echo $PHP_BASE) that when submitting to the source code repo what PHP version you are getting that error on.
If it does turn out to be an error in image, let me know I'll come back at this.

It shows version PHP 8.1
I am also not sure if it's an error of the docker image, I can save the settings in Mange->Settings->Mail settings
That works fine.
I will post the same issue at https://github.com/freescout-helpdesk/freescout

OK. If you wanted to do some sleuthing and try on a different PHP version I also build some undocumented images;
:1.15.68_php8.1
:1.15.68_php8.0
1.15.68_php7.4

Sure, I will try those


bootstrap: Update Font Awesome to 5.15.4

First part of #978.
Another thing is, Huraga theme is still using FA 3.x. We can update that to use 5.x too, if we make FA common and sharable across themes, we won't need to include the same version of FA twice in our source code. That will save us a total of ~1.6MBs. Now we need to discuss if we're going to do this.

Merging this right now as we don't need to wait for the discussion to update FA for a single theme.


Buttons are not showing correctly on Android

I am using block buttons one with colour and one with transparent background.
**on Android transparent button is showing some grey background.
Also the text is showing in caps on Android only.**

  <NativeButton
            testID={testID}
            small={isSmall}
            disabled={isDisabled}
            block={isFullLength}
            rounded
            style={[
                tailwind(`justify-center ${buttonStyle[isVariant](isPressed)} ${buttonClasses}`),
                {
                    ...buttonStyles,
                    ...shadowStyle,
                    ...floatingStyle,
                },
            ]}
            onPress={onPress}
            onPressIn={onPressIn}
            onPressOut={onPressOut}
            activeOpacity={1}>
            <NativeText
                style={[
                    tailwind(`${textStyle[isVariant](isPressed)} font-extrabold   ${textClasses}`),
                    {
                        ...textStyles,
                        fontFamily: 'Poppins-SemiBold',
                        textAlignVertical: 'center',
                        textAlign: 'center',
                        ...smallButtonStyle,
                    },
                ]}>
                {children}
            </NativeText>
            {icon}
        </NativeButton>
    </Content>

Any help would be really appreciated..

@MaganAnkur Can you let us know the version of NativeBase you are using?

@MaganAnkur Can you let us know the version of NativeBase you are using?

2.13.14


Update Fastly Documentation

Fastly is switching to its new user interface on November 17. The documentation on SendGrid's website should be updated to match the new interface. We've opened pull request #2304 with the recommended changes.
Thanks for looking into this!

Thanks for raising the issue!
I made a request on the PR, let's discuss there and then we can update this issue accordingly.
Thank you!!

The necessary updates have been made in https://github.com/sendgrid/docs/pull/2323.
Thanks for the collaboration @mattcone and @mbernier !


Option to change Layout of the Feed Contents

I'm a diehard NNW3 user.  There are just some thing about it that I can't give up if I move to this new NNW 5.  One of them is how the feed entries are displayed.  I feel that the new design take up a lot of space.
I have attached a screenshot of the old view that I have as well as the Layout submenu. That is placed next to the new view.  They are not to scale.
Could there be a layout option that would allow for a smaller, cleaner feed entry display?

Would a more information dense timeline solve your problem or is this the same request as #102?  Below is a mockup of a more information dense timeline.  Let me know if this is what you are asking for.

Not making the area thinner, but more in line as to how the listing looks in my image on the left.
I was thinking how the new version shows too few feed entries in a list. While the site icons are nice, they are taking up a lot of extra space from the title of the articles.  Also, the bylines are usually longer and get truncated, so I would think that kinda makes it pointless to show.
I don't show the date, subject, or creator from NNW3, just the source when looking at the folder view of a collection of feeds.  When viewing individual feeds, the source doesn't display, just the title of the entry.
I should note that the light image on the left is a listing of my "Apple" related feeds as a whole.  This is why you see the names of the sites listed on the right.  I don't list them all since I can tell who is who by the first few letters of their name.
I hope this makes sense.  This, and a few other things, are what is keeping me from moving to the new version.  But the fact that I'm starting to have issues with the bright light colors is starting to push me along.

Thanks for the detailed information on how you are using NNW 3.  We have a number of Timeline customization Github issues, but I think this one is a little unique.  I'm going to change the title of this issue so that it easier when we search for it.
I think some things on the Timeline will become more customizable, but honestly, they probably won't happen anytime soon and not to the degree that you want.  Hiding the feed name and date is a an unusual usage pattern.
I hope you give the new NNW a chance and are able to adjust to it.  I think eventually the new features will win you over, even it it feels strange to use at first.  If it doesn't work out for you, try again after we release NNW 5.1.  There are lots of cool things coming.


Modernization UI

Installed the Modernization successfully - however, I have a few issues.

Using Edge I get the following error (works in Chrome and IE) - Not all parameters are configured, can't execute. Please run the setup steps.
In IE the Keep/Discard options do not work.
is there a mapping for "Content Editor Web Part" to "Text" web part in modern? I can transform to "Embed" without issue - however, would like to map to "Text"

Greg

Just installed the dev Edge (Chromium open source) and the Modernization UI works.

Correct, IE11 does not work at this moment. With dev Edge becoming the standard MSFT browser we currently do not have plans to add support for IE11.

Hey @gregwilliams14 , wanted to cross check with you if using Edge chromium or Chrome is a valid workaround for you? If so then feel free to close this issue.

Not really.  We are in process for rolling out Win10 which has Edge (non-Chromium).  It would only work if we get user to install Edge chromium or Chrome.  At some point we will roll out Edge Chromium.  Will it not work in Edge?

Include IE support

There are no current plans to work on the page transformation UI. The preferred option for doing page transformation is using PnP PowerShell or .Net. Check out the guidance at https://aka.ms/sppnp-pagetransformation.
This issue will be closed but can be re-opened if we restart the work on the page transformation UI.


[BUG] Can't commit with ssh signing enabled

Describe the bug
I configured git to automatically sign commits with an ssh key and it works like a charm with git, but fails with cog.
To Reproduce
cog commit feat 'blah'

Note: it fails with or without the --sign flag because I configured git to automatically sign commits.
Expected behavior
Commits should work.
Additional context
This is the error I'm getting:
Couldn't load public key /var/folders/gz/cq29z4yj29lc12h2l7lb2zd80000gn/T/.tmpOGt5oW: No such file or directory
Error: IO Error

cause: No such file or directory (os error 2)

Can you provide the relevant gitconfig section to reproduce ?


:seedling: Add check for client in e2e

Depending on how e2e is run, we may not have a k8s client available.

Description

Reviewer Checklist

[ ] API Go Documentation
[ ] Tests: Unit Tests (and E2E Tests, if appropriate)
[ ] Comprehensive Commit Messages
[ ] Links to related GitHub Issue(s)

Ok, I'll make a corresponding change to catalogd.

See https://github.com/operator-framework/catalogd/pull/485 @camilamacedo86 for the catalogd version


Add my ISPRASOPEN-2019 talk

Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

Thanks!


Opening system project in GUI causes massive network traffic

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):
Set up Rancher HA on RKE Cluster behind Loadbalancer
Deploy cluster
Open System project
Result:
Opening the project on the GUI causes a massive network spike lasting a few minutes, coming from the rancher-cluster-agent pod, to the loadbalancer, and then to the Rancher HA cluster.
Other details that may be helpful:
This only happens in one of our clusters, for some reason. Besides triggering it manually it also occurs every 30 minutes,
I have tried looking around in the logfiles on the node that hosts the pod, as well as the pod itself. There is absolutely nothing that would point to it causing such high traffic.
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.2.8
Installation option (single install/HA): HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): vSphere
Machine type (cloud/VM/metal) and specifications (CPU/memory): VM
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.9", GitCommit:"3e4f6a92de5f259ef313ad876bb008897f6a98f0", GitTreeState:"clean", BuildDate:"2019-08-05T09:22:00Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.6", GitCommit:"96fac5cd13a5dc064f7d9f4f23030a6aeface6cc", GitTreeState:"clean", BuildDate:"2019-08-19T11:05:16Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

Client:
 Version:           18.06.3-ce
 API version:       1.38
 Go version:        go1.10.4
 Git commit:        d7080c1
 Built:             Wed Feb 20 02:24:22 2019
 OS/Arch:           linux/amd64
 Experimental:      false

Server:
 Engine:
  Version:          18.06.3-ce
  API version:      1.38 (minimum version 1.12)
  Go version:       go1.10.3
  Git commit:       d7080c1
  Built:            Wed Feb 20 02:25:33 2019
  OS/Arch:          linux/amd64
  Experimental:     false

Alright, we have figured out the solution to the issue, but now it's really more of an enhancement.
The issue was caused by a large number of configmaps in the kube-system namespace. We did not set a limit on our helm revisions, and one of our developers deployed quite a bit and quite a lot. Every deployment created a 400kb configmap - and we had over 700 of them at one point.
We have solved this issue by simply limiting the max-history with helm, but this really causes another question: when opening a project, is Rancher actually loading every single configmap/deployment/etc. to present to the user as opposed to when requesting it?
We only noticed this issue because Rancher is running behind an F5 Loadbalancer with a license maximum of 200mbit/s. It still caused a lot of traffic, and I don't think this is good/intended behavior.  Cluster resources in a project should really only be loaded if a user actually requests them by interacting with it.

I think it's the same problem as https://github.com/rancher/rancher/issues/21587.


Update docs rasa x

Proposed changes:

Fixes https://rasahq.atlassian.net/browse/ATO-125
Rename "Rasa X" to "Rasa Enteprise" in docs, and remove sections that do not make sense anymore
Cleanup mentions of "Rasa X" in the code base
rasa --version does not print Rasa X Version anymore

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

@alexweidauer could you review the documentation changes that I made in this PR for Rasa OSS?
@losterloh feel free to also take a look at the documentation changes if you want, but I'll need your review on anything that is not under the docs/ folder 😄
Thanks 🙏🏻


Add dumpdata + loaddata example to Wagtail docs

There have been several discussions around exporting and importing data from a Wagtail site, e.g. as part of a development process to allow developers to 'scaffold' fixture data. However, there doesn't seem to be a clear answer (or any answer at all on StackOverflow).
Please consider adding a section to the Wagtail documentation describing how to generate fixture data with the Django dumpdata and loaddata commands, as this as not as straightforward as it would seem.
Related

#4472
#742
https://stackoverflow.com/questions/50160514/migrating-wagtail-data-between-environments
https://stackoverflow.com/questions/47476226/wagtail-migrate-copy-data/47481861#47481861

@brylie
I have written a blog talking about this How to Export & Restore Wagtail Site

I will try to create a PR to add a section to Wagtail doc in a bit.
Thx.

@brylie
I have written a blog talking about this How to Export & Restore Wagtail Site
I will try to create a PR to add a section to Wagtail doc in a bit.
Thx.

@michael-yin
I followed your example but I got an error
django.contrib.contenttypes.models.DoesNotExist: Problem installing fixture '/home/oladhari/Reachat-v4/backend/reachat/fixtures/db.yaml': ContentType matching query does not exist.

I would like to cast a vote in favour of this issue after dealing with a problem in #6043 and getting marvellous advice about Wagtail Transfer (thank you @jacobtoppm & @gasman).
A description of these approaches to maintaining your content are sorely needed in the documentation - as well as potentially some tips about where to find information about doing similar procedures with relational DB tools. Anyone here up for drafting a page in the Advanced topics?

Shouldn't this be handled by wagtail natively ?
I guess the right incantation can be defined in a wagtail management command which could overwrite the one from Django.


Destructuring array into an object is assignable to an array type

TypeScript Version:  3.8.0-dev.20191026
Search Terms:

array destructuring object
array destructure assign

Code
let bar = [1, 2, 3]

let foo: any[] = { ...bar }

console.log(Array.isArray(foo)) // false

Expected behavior: Typescript errors about type mismatch.
Actual behavior: Typescript does neither warn nor error about this and foo is no longer an array.
Playground Link: https://www.typescriptlang.org/play/?ts=3.8.0-dev.20191026&ssl=1&ssc=18&pln=1&pc=8#code/DYUwLgBARghgThAvBA2gRgDQQExYMwC6AUEaJAGYD2lAXBDAHYCeKBSEA3hAHS+wIBfEgGNKDAM6VQ3YJQDmACgCCcODCbcAluJVqmCqpQCURiAHozEcjGDiQQA
Related Issues: #14768

See #9726 - TS would need a way to know that the array methods don't end up on foo.

@RyanCavanaugh is that really about enumerable properties? I believe in a very similar case for objects not arrays the type is correctly inferred:
class Customer {
  constructor(public name: string) {}
  greet() {
    return 'Hello ' + this.name;
  }
}
var customer = new Customer('David');
var clone = {...customer};
// console.log(clone.greet()); //Does not compile

Playground
Could we have similar feature for arrays?

yeah i found that too doing a reduce and accidentally returning {...acc} instead of [...acc]. i was puzzled by why TS would be happy about it 😄

@sebapawlak TS knows which members of a class are enumerable and which aren't, but we don't have any declaration form for annotated the value-declared type of Array. That's literally what I'm talking about re #9726


Crashes on empty line

Crashes on empty line. This has been fixed once, seems to have reappeared.

The problem was related with sorting of sentences (mini--batch, maxi-batch).


OM127 - 7.0 node view changes

initial checkin with all 7.0 changes
7.0 based theme to combine 6.0 and 7.0 changes while showing state, topk and time-series values

Reviewed this across a split cluster. Planning some more testing, but some initial notes:

Having clear version is really helpful for supporting upgrade

Alerting feedback

I'd expect this to be a sum of number of alerts on the node
I don't think we should include warnings in this view. Just Critical IMO

Best Practice Check

Should be yellow to match warnings color on alerts view

Ideally best practice checks would describe the specific best practice. Or we can add instructions to run the command asadm show best-practice

It may make sense to have an orange box for errors level alerts here also.

Finally I believe it is more correct to use Record Count instead of Object count.

One more comment.
I wonder about offering a namepsace variable on this dashboard. Strictly speaking it seems like we should only filter on node for this dashbaord (I think its fine to keep it just wanted to note it here).

changed the alerts panel as a pie chart - with this we can show all alerts in relevant colour
alerts panel is modded to show sum of all alerts
best-practice and node-up panels now represent same colour as warn & critical like alerts
added "asadm show best-practice" in the best-practice description
namespace filter not-changed as we felt as it would be helpful
"master objects" panel title is left as-is instead of records as metric name is master_objects


chore: Included githubactions in the dependabot config

This should help with keeping the GitHub actions updated on new releases. This will also help with keeping it secure.
Dependabot helps in keeping the supply chain secure https://docs.github.com/en/code-security/dependabot
GitHub actions up to date https://docs.github.com/en/code-security/dependabot/working-with-dependabot/keeping-your-actions-up-to-date-with-dependabot
https://github.com/ossf/scorecard/blob/main/docs/checks.md#dependency-update-tool
Signed-off-by: naveensrinivasan<EMAIL_ADDRESS>

Closing as it was created w/o permissions for maintainers and it contained errors. I will get this in with another Pr.


Remove duplicate javadoc in Intersector

Why not remove the main method as well while you're at it?

That main looked like a test, so it probably isn't wise to just remove it. It could be made into unit test later.

That's why I've just comment it out after merging you PR ;)

Why are there no JUnit tests? :(


New Component: Canvas

A canvas component, so it can be used to capture signatures, or any kind of drawing.

What exactly you expect? Native js API for canvas or drawing surface?
for native api we have https://github.com/BlazorExtensions/Canvas

What exactly you expect? Native js API for canvas or drawing surface?
for native api we have https://github.com/BlazorExtensions/Canvas

Yeah, I know about that one, I thought it would be better to have this component inside MatBlazor.

MatBlazor planned as AIO (ALL in One), but this is not point of interest of MatBlazor. Maybe I will make another package MatBlazor.Canvas.
But I see https://github.com/BlazorExtensions/Canvas and they was great. I dont think. that I can really improve something.

@SamProf Canvas is a critical part of any Javascript UI framework. There are so many variants. However, all come with ease of use in implementing event binding and mouse event handling.
I think this is currently missing in all Blazor attempts I have seen. I wish I am wrong.
Please others comment on this.
I think we need talents into these GAPs among all canvas, svg  blazor implementations.


tools: osbuild-dev quality of life

I've been using osbuild-dev quite a bit on manifests, however it seems to have broken? This fixes it and some other minor improvements.

Some documentation of osbuild in general is a great idea for a followup (I think I have a local branch somewhere!).
@thozza are you OK with including osbuild-dev in the RPM only on Fedora?

@thozza are you OK with including osbuild-dev in the RPM only on Fedora?

Ok with me, but I'd like Tomas' approval too.

@thozza are you OK with including osbuild-dev in the RPM only on Fedora?

Ok with me, but I'd like Tomas' approval too.

I don't think that we have any other choice, since its dependencies are not available on RHEL. Also, we do not ship the -tools sub-package on RHEL, so it does not matter much IMO.


Added New widget

77 . Repo Roster
Shout-out supporters in your GitHub README file.

📍 For example :
Stargazers

Forkers

77 . Repo Roster
Shout-out supporters in your GitHub README file.
📍 For example :
Stargazers

Forkers

Thanks for adding this tool 🙏❤


Cannot Goto declaration of static extension functions in Haxe

This is a known limitation of FD built in engine. But if you try a dev build this should work if the compiler is able to resolve it (compiler-based completion must be working in your project).

this is works, because #729


vim-mode switch/move panes using C-w {h,j,k,l}

Before you begin
Check the backlog of issues to reduce the chances of creating duplicates; if an issue already exists, place a +1 (👍) on it.
Is your feature request related to a problem? Please describe.
Switching and moving panes in vim mode using the keyboard only.
Describe the solution you'd like
So in vim, you can normally switch panes by pressing C-w ("Control + w") and then h, j, k or l to move to that pane. Another thing that is kind of tied to that is moving the panes them selves using C-w + H, J, K or L (so the uppercase versions of the previously described action, swaps out the current pane with the pane you'd move to).
Describe alternatives you've considered
Well, you could use the mouse, but if you're already using vim mode, why not use the existing vim motions.
Screenshots
Not applicable

@HanEmile The lower-case version of this is now implemented! We haven't built the ability to swap (yet) though.

The lower-case version of this is now implemented! We haven't built the ability to swap (yet) though.

@ConradIrwin Oh neat, I was just looking for this! ✨
Do you perhaps know if this is using a Zed command that can be used even when not in vim mode? I'd find it useful to use such a command even outside of vim mode, via a keyboard shortcut. Since there are already other commands related to panes (e.g. splitting a pane), it doesn't seem unreasonable to have a command for switching panes as well. 🙏 😄

There is not yet, but there should be!

Thanks for your patience here!
This is implemented in Zed 0.106 (currently in preview, and likely to hit stable next wednesday).
In vim mode, ctrl-w shift-{h,j,k,l,left,right,up,down}, and outside of vim mode as cmd-k shift-{left, right,up,down}.

I'm running 0.108.2, but when I do cmd-k shift-right nothing happens

@adri1wald that's strange, it should work if you have two (or more) vertical split panes (which you can create with :vsplit in vim mode).
We've occasionally seen problems with keyboard shortcuts on non-US keyboards. What layout are you using and I'll try and test it.
Otherwise, is it possible you have a conflicting keybinding configured?


memory usage too high

Environment
Platform: [e.g. Kubernetes, OpenShift, Rancher, Managed k8s (AKS, EKS, GKE, IKS, ...), ...]
Kubernetes/Platform Version(s):
1.20.4-aliyun.1
Describe the bug
A clear and concise description of what the bug is.  (If you are encountering a deployment error, check out the README or navigate to https://open-docs.neuvector.com for full deployment documentation.  Incorrect runtime is the most common reason for Controller and Enforcer CrashLoopBackOff events.)

memory usage too high
To Reproduce
Steps to reproduce the behavior:
2022-04-07T10:36:44.15 |INFO|AGT|main.(*Bench).doContainerCustomCheck: Running benchmark checks done
2022-04-07T10:37:07|DEBU|dp0|dp_data_thr: epoll error: /host/proc/3631255/ns/net-eth0
2022-04-07T10:37:20.452|ERRO|AGT|osutil.getCGroupSocketTable: open net/tcp,udp - error=open /proc/3632603/root/proc/1/net/tcp: no such file or directory
2022-04-07T10:37:20.452|ERRO|AGT|osutil.getCGroupSocketTable: open net/tcp,udp - error=open /proc/3632603/root/proc/1/net/tcp6: no such file or directory
2022-04-07T10:37:20.452|ERRO|AGT|osutil.getCGroupSocketTable: open net/tcp,udp - error=open /proc/3632603/root/proc/1/net/udp: no such file or directory
2022-04-07T10:37:20.453|ERRO|AGT|osutil.getCGroupSocketTable: open net/tcp,udp - error=open /proc/3632603/root/proc/1/net/udp6: no such file or directory
2022-04-07T10:37:47.541|ERRO|AGT|main.updateContainerNetworks: Error reading container network endpint - container=lb-none endpoint=none-endpoint error=Not found
2022-04-07T10:38:08.115|ERRO|AGT|osutil.getCGroupSocketTable: open net/tcp,udp - error=open /proc/3631255/root/proc/1/net/tcp: no such file or directory
2022-04-07T10:38:08.115|ERRO|AGT|osutil.getCGroupSocketTable: open net/tcp,udp - error=open /proc/3631255/root/proc/1/net/tcp6: no such file or directory
2022-04-07T10:38:08.115|ERRO|AGT|osutil.getCGroupSocketTable: open net/tcp,udp - error=open /proc/3631255/root/proc/1/net/udp: no such file or directory
2022-04-07T10:38:08.115|ERRO|AGT|osutil.getCGroupSocketTable: open net/tcp,udp - error=open /proc/3631255/root/proc/1/net/udp6: no such file or directory
2022-04-07T10:38:08.194|INFO|AGT|main.taskAddContainer: - id=ae98a232623f98f38c5df29b3a9f44a8382b70f3d7a3edda4367ee56046489e0 name=b9555b6dd-zjp78_871538d8-8e9b-449a-8255-6d7e530bcca3_0
2022-04-07T10:38:20.718|INFO|AGT|main.(*Bench).doDockerContainerBench: Running benchmark checks done
2022-04-07T10:38:20.725|INFO|AGT|main.(*Bench).doContainerCustomCheck: Running benchmark checks done
2022-04-07T10:38:22.663|INFO|AGT|main.(*TaskScanner).scanSecretLoop: SCRT: in progress ...
2022-04-07T10:38:35.739|INFO|AGT|system.(*SystemTools).CGroupMemoryStatReset: - threshold=2080374784 usage=4770603008
2022-04-07T10:38:35.739|ERRO|AGT|system.(*SystemTools).CGroupMemoryStatReset.func1: - err=write /sys/fs/cgroup/memory/memory.force_empty: device or resource busy
2022-04-07T10:38:36.663|INFO|AGT|main.(*TaskScanner).scanSecretLoop: SCRT: done - Finished=1 TimeUsed=13.999985626s
2022-04-07T10:39:44.852|INFO|AGT|main.taskStopContainer: - c.pid=3632603 container=3ffa5f5f4db8eb3550cb757b0df154b3c0dda18b2af5d44b0f7e335edd75e401 pid=0
2022-04-07T10:39:44.853|ERRO|AGT|main.taskStopContainer: Failed to read container. Use cached info. - error=No such container: 3ffa5f5f4db8eb3550cb757b0df154b3c0dda18b2af5d44b0f7e335edd75e401
 id=3ffa5f5f4db8eb3550cb757b0df154b3c0dda18b2af5d44b0f7e335edd75e401
2022-04-07T10:39:45.088|ERRO|AGT|osutil.getCGroupSocketTable: open net/tcp,udp - error=open /proc/3632603/root/proc/1/net/tcp: no such file or directory
2022-04-07T10:39:45.088|ERRO|AGT|osutil.getCGroupSocketTable: open net/tcp,udp - error=open /proc/3632603/root/proc/1/net/tcp6: no such file or directory
2022-04-07T10:39:45.088|ERRO|AGT|osutil.getCGroupSocketTable: open net/tcp,udp - error=open /proc/3632603/root/proc/1/net/udp: no such file or directory
2022-04-07T10:39:45.088|ERRO|AGT|osutil.getCGroupSocketTable: open net/tcp,udp - error=open /proc/3632603/root/proc/1/net/udp6: no such file or directory
2022-04-07T10:39:46.071|INFO|AGT|main.taskStopContainer: - c.pid=3631255 container=f59b29c5c55b7d011f102b050867f2a072357495dbadc9cc419855b47d2eed83 pid=0
2022-04-07T10:39:46.071|ERRO|AGT|main.taskStopContainer: Failed to read container. Use cached info. - error=No such container: f59b29c5c55b7d011f102b050867f2a072357495dbadc9cc419855b47d2eed83
 id=f59b29c5c55b7d011f102b050867f2a072357495dbadc9cc419855b47d2eed83
2022-04-07T10:39:56.127|INFO|AGT|main.(*Bench).doDockerContainerBench: Running benchmark checks done
2022-04-07T10:39:56.13 |INFO|AGT|main.(*Bench).doContainerCustomCheck: Running benchmark checks done```

**Expected behavior**
A clear and concise description of what you expected to happen.

**Screenshots**
If applicable, add screenshots to help explain your problem.

**UI Issue**
Desktop (please complete the following information):**
 - OS: [e.g. MacOs 12.1, Windows 10]
 - Browser [e.g. chrome, safari]
 - Version [e.g. 22]

**Additional context**
Add any other context about the problem here.

Please provide the following informations,
What NeuVector version do you use? The size of the host where the enforcer is running, CPU/memory, how many pods?

I can confirm that all known NV v5.0.x releases eats up a huge amount of memory.
In my lab environments with K3S and RKE2, the memory requirement for NeuVector increases permanently over time. I have noticed this recently.
For this reason, I have integrated the namespaces with the NeuVector deployments in the lab into our monitoring.
I will keep an eye on this and report back.
This is, what my memory consumption in my playground looks like over the time:

My playground:

SingleNode K3S Cluster with v1.24.7+k3s1
MultiNode RKe2 Cluster with v1.23.10+rke2r1
NeuVector 5.0.1-4

Hi!
I have the same problem. Do you have any solution?

I have the same problem. Do you have any solution?
1.22.5-tke.15
NeuVector 5.1.2


Update teams-live-event-usage-report.md

Updating documentation to reflect existing behavior.

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

Looks good to me! I have committed that suggestions: Thank you!

Also @JohanFreelancer9 - how do I fully commit the PR and merge it with the Main branch?

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

@scanum, @SerdarSoysal - can you please help me merge these requests? This PR has been open for a long time and we have a customer escalation around this.


Updated MAINTAINERS.md format.

Coming from https://github.com/opensearch-project/.github/issues/121, updated MAINTAINERS.md to match opensearch-project recommended format.

@seanneumann What do you want to with this repo? It seems like it was an attempt at something new and we left it at that?


Add a slurp method to Treiber stack

This is useful because it is wait-free instead of lock-free.

This looks quite useful! What about implementing a more general swap instead? The swap would just swap the stack with the one given in its argument and slurp would be a swap with a new empty stack passed into it.

@Vtec234 Good question. That seems reasonable. Annoyingly you can't go from an Atomic to an Owned but only a Shared so I'll have to think about how to implement it properly.

This is an old PR, but here's a quick suggestion for several methods we could add to TreiberStack:
fn clear(&self) -> TreiberStack<T>;
fn into_iter(self) -> IntoIter<T>;
fn push_batch<I: IntoIterator<Item = T>>(&self, iter: I);

Using these methods, one can move all elements from stack a to b with:
b.push_batch(a.clear())

(method names are bikesheddable)

This is a stale PR. Closing, but if you want to keep working on it, I'll reopen.


update deps

Decreasing the number of reported vulnerabilities by 2, more robust hot reloading in situations we haven't come across yet, a bit more flexibility for table columns, and newer clarity something? I don't think anything actually changed in the icon package, it seems like they just bump the number to keep it consistent with the rest of the clarity packages, but I figured why not just get everything current...


If there is no term mapped there is no term mapping

On downloading the mapping for schema.org Organization to the spine I see many JSON objects along the lines of
        {
            "@id": "http://desmsolutions.org/TermMapping/1332",
            "@type": "desm:TermMapping",
            "dcterms:isPartOf": {
                "@id": "http://desmsolutions.org/TermMapping/9"
            },
            "desm:comment": null,
            "desm:mappedterm": [],
            "desm:mappingPredicate": {
                "@id": null
            },
            "desm:spineTerm": {
                "@id": "https://tool.desmsolutions.org/resources/terms/Industry+Type"
            }
        },

Note that the mappedTerm and mappingPredicate are null. That is because there is no mapping for the spineTerm. In such cases there should be no TermMapping in the output.
related: #406 on how these null values cause problems in some tools.

fixed, see #406 for remaining problems


Installation issues (ModuleNotFoundError on import)

It seems I can't import the package (import kmerexpr gives ModuleNotFoundError) and the individual files (import simulate_reads) can only be imported when installed in editable mode. I tried installing through github, locally, and in editable mode. I'm using python 3.9 and pip 22.2.2.
Not sure how to fix it. Maybe setup.cfg:L18-28 needs to be tweaked to match this structure with a src/ folder between the base of the repository and the kmerexpr package?
Fixing this might mess existing import statements? For example in simple_example.py:L1-L7, imports such as import simulate_reads as sr should probably be import kmerexpr.simulate_reads as sr to work when the file is called from another directory?
Reproducing the errors
Attached are scripts to reproduce the issue (and their outputs): testing.zip.
The scripts assume they are in a folder containing the repo in a subfolder called kmers_repo
(after git clone https://github.com/bob-carpenter/kmers kmers_repo)
/test/ < console here
     /kmers_repo/
     /test_install_github.sh
     /test_install_local.sh
     /test_install_local_editable.sh

The scripts install the library and try to import kmerexpr and simulate_reads.
Note: the scripts need to first uninstall kmerexpr and might mess up existing installations.

test_install_github.sh = pip install git+https://github.com/bob-carpenter/kmers.
python -c "import kmerexpr" ❌
python -c "import simulate_reads" ❌
test_install_local.sh = pip install ./kmers_repo.
python -c "import kmerexpr" ❌
python -c "import simulate_reads" ❌
test_install_local_editable.sh = pip install -e ./kmers_repo.
python -c "import kmerexpr" ❌
python -c "import simulate_reads" ✔

The issue is with this:
https://github.com/bob-carpenter/kmers/blob/be5d806b928253cbc94d58e59fa2378d79c97d00/setup.cfg#L19-L20
The package_dir key does not indicate the folder the package lives in, it indicates the folder to look for packages. By specifying it we are actually saying kmerexpr is not a package, but everything in that folder is.
We should just delete those two lines (I've confirmed you can then import kmerexpr)

Not working for me.
If I delete those lines in setup.cfg and reinstall (pip uninstall kmerexpr, pip install . in the repo)

python -c "import kmerexpr" works when in the root of the repo
but moving to another directory fails with a ModuleNotFoundError.

I'm guessing the first works because it imports by path.
I can get it to work by putting a src folder in between the root and the kmerexpr package, to get this structure
repo_root/
  src/
    kmerexpr/
      __init__.py
     ...

and editing the end of setup.cfg to
[options]
package_dir =
    = src
packages = find:
python_requires = >=3.7
install_requires =
    numpy >= 1.18
    scipy >= 1.5

[options.packages.find]
where = src

I'm up to write a PR with this if you're ok with the change.

This might be a separate issue;
I tried writing a test for the imports, but the imports work when in the test folder.
They stop working (which is the behavior I expect) when I remove the __init__.py in the test folder.
According to the pytest docs, there shouldn't be an __init__.py here. Should it be removed to not hide the import issues?

Ah, you also need to remove
https://github.com/bob-carpenter/kmers/blob/be5d806b928253cbc94d58e59fa2378d79c97d00/setup.cfg#L27-L28
These lines have basically the same meaning as the others, telling Python to look in the wrong place. I generally do not like having the extra layer of a src directory in a pure python package.

This might be a separate issue;

For tests, I think it is optional. Pytest says in the "Note" on the page you list that it "should" work without the __init__.py, but there is no reason to delete it I believe (I can't recall it now, but I seem to remember some other tool, maybe mypy, really wanting it there)

After fixing setup.cfg there will also need to be a fair number of changes to the imports in test files, things like
from transcriptome_reader import valid_kmer, shred, kmer_to_id
import fasta

becoming
from .transcriptome_reader import valid_kmer, shred, kmer_to_id
from . import fasta


'No matches for' after blur and focus on input

Not useful

Could you please provide the required steps to reproduce the issue you're seeing?

Go to https://reactcommunity.org/react-autocomplete/custom-menu/
Enter or choose 'Alabama'.
Click outside field
Click on input.

Thanks! I was able to reproduce the issue and create a fix.


docs: use Docker as the execution environment

After this change lands, I'll file a bug to switch to the non-beta version of gcloud when the execution environment feature lands.

CC @subfuzion
Maybe we should just list this as an option? Not sure it's best to have most users go this route.

CC @subfuzion
Maybe we should just list this as an option? Not sure it's best to have most users go this route.

I think that the problem is that, if the user wants to connect to a database (likely) and uses the recommended approach (likely) then the gen1 execution environment won't work.
I could write some docs explaining why we suggest the gen2 execution environment.

My guess: connecting to native databases is not uncommon, but it's not a majority of use cases.
For the majority of cases, when you just want to wrap some other REST APIs, etc – we want to avoid recommending beta solutions.
Case in point: the issue that was filed was the first time we've heard of this issue!

So should I suggest that if you are consuming a database? And then, when gen2 is GA then we can recommend it in all situations.


💡 [REQUEST] -  Add deleteObject method to DatabaseManager

Start Date
No response
Implementation PR
No response
Reference Issues
No response
Summary
There is no way to delete a persisted object from the database. This would continue increasing the size of the database as there is no way to clear it.
Basic Example
Currently only able to add and update objects in the database, but objects persisted cannot be remove.
Drawbacks
No response
Unresolved questions
No response

Dependent on #104


Add Borrowable Margin Pool (V2)

Task: Feature Name:
High Level Description
Allow whitelisted ribbon vaults to create a short position and direct their collateral to the new borrowable margin pool (MarginPoolV2) instead of the unborrowable marginpool (MarginPool). This allows certain whitelisted parties (whitelisted directly on MarginPoolV2) to borrow the collateral backing the call option. The whitelisted parties will exclusively be market makers who also buy the call option. This allows the market maker to delta hedge in a more capital efficient manner since they have an uncollateralized loan to the spot asset (which they can then short). We will whitelist vaults of treasuries that agree to offer this uncollateralized loan in exchange for higher premiums on the call option.
Say Zebra DAO has a $ZBR token. They launch a treasury vault with ribbon finance and deposit 100 $ZBR to sell covered calls and earn premiums from market makers. The 100 $ZBR is deposited into MarginPoolV2 when oTokens are minted. Market Maker X has 10 ZBR oTokens (call options) that they bought from the vault's oToken auction. Whereas in vanilla MarginPool the collateral cannot be taken out until the end, in MarginPoolV2 the oToken buyer (market maker) can borrow 10 $ZBR from MarginPoolV2 using borrow() to hedge their positive delta, posting ZERO collateral. Notice the amount they can borrow is proportional to how many oTokens they bought. Once a month passes, they deposit 10 $ZBR back into the vault.
Controller.sol:

setMarginPoolV2: adds margin pool v2 address
donateV2: donates to v2 margin pool

MarginPool.sol

setBorrowerWhitelistedStatus: sets whether an address is whitelisted to borrow
setRibbonVaultWhitelistedStatus: sets whether ribbon vault address is whitelisted to create short oToken on MarginPoolV2
setBuyerWhitelistedStatus: sets whether address is whitelisted as oToken buyer of oToken minted on MarginPoolV2
setBorrowPCT: sets the borrow percentage of entire collateral on a per-asset basis
borrow: allows whitelisted address (market maker) to borrow some of underlying collateral proportional to their oToken bal
borrowable: checks how much an address can borrow of underlying
repay: allows whitelisted address to repay their loan once oToken expires
repayFor: allows any address to repay loan on behalf of another party

Code

[ ] Unit test 100% coverage
[ ] Does your code follow the naming and code documentation guidelines?

Documentation

[ ] Is your code up to date with the spec?
[ ] Have you added your tests to the testing doc?

Left some comments, main concerns re figuring out which pool to redeem oToken from. I actually think upgrading the MarginPool and having one MarginPool that keep track of collateral that can be borrowed may be better

Also probably in that case whitelisting oToken buyer is not needed...

Left some comments, main concerns re figuring out which pool to redeem oToken from. I actually think upgrading the MarginPool and having one MarginPool that keep track of collateral that can be borrowed may be better

Also probably in that case whitelisting oToken buyer is not needed...

Something like 1 pool contract (but we need to migrate current pool funds to the new one, but can be done by adding a function in Controller that can only be called by owner for that to migrate current balance).
MarginPoolV2 is MarginPool, adding a mapping to keep track of deposited collateral that can be borrowed + this BorrowablePool implementation, what u think about that? @chuddster

Left some comments, main concerns re figuring out which pool to redeem oToken from. I actually think upgrading the MarginPool and having one MarginPool that keep track of collateral that can be borrowed may be better

Also probably in that case whitelisting oToken buyer is not needed...

Something like 1 pool contract (but we need to migrate current pool funds to the new one, but can be done by adding a function in Controller that can only be called by owner for that to migrate current balance).
MarginPoolV2 is MarginPool, adding a mapping to keep track of deposited collateral that can be borrowed + this BorrowablePool implementation, what u think about that? @chuddster

Or if we want to keep the 2 different pools, I guess it make sense to update the oTokenImplementation and add an isBorrowable field

Closed by mistake...

Left some comments, main concerns re figuring out which pool to redeem oToken from. I actually think upgrading the MarginPool and having one MarginPool that keep track of collateral that can be borrowed may be better

Also probably in that case whitelisting oToken buyer is not needed...

Something like 1 pool contract (but we need to migrate current pool funds to the new one, but can be done by adding a function in Controller that can only be called by owner for that to migrate current balance).
MarginPoolV2 is MarginPool, adding a mapping to keep track of deposited collateral that can be borrowed + this BorrowablePool implementation, what u think about that? @chuddster

If MM with 1oToken can't borrow collateral greater than the amount of collateral deposited to mint that 1oToken, right? I think it is safe to use the same MarginPool, and allow all oToken holder to redeem from the same pool, as we are locking the oToken from borrower, right?

Left some comments, main concerns re figuring out which pool to redeem oToken from. I actually think upgrading the MarginPool and having one MarginPool that keep track of collateral that can be borrowed may be better

Also probably in that case whitelisting oToken buyer is not needed...

Something like 1 pool contract (but we need to migrate current pool funds to the new one, but can be done by adding a function in Controller that can only be called by owner for that to migrate current balance).
MarginPoolV2 is MarginPool, adding a mapping to keep track of deposited collateral that can be borrowed + this BorrowablePool implementation, what u think about that? @chuddster

If MM with 1oToken can't borrow collateral greater than the amount of collateral deposited to mint that 1oToken, right? I think it is safe to use the same MarginPool, and allow all oToken holder to redeem from the same pool(no need to whitelist oToken buyer or check which pool to redeem from), as we are locking the oToken from borrower, right?

@haythem96 I think some more context will be helpful for you here: we are trying to build this pool out for select ribbon treasury partners to increase the yield on the options since the collateral can be borrowed. We want to decouple this into an entirely new margin pool for three reasons:

this allows opyn to continue to be the owner of the MarginPool
this allows Ribbon to be owner of new MarginPool instance and decide who is whitelisted, the borrowing percentages, etcetera etcetera since the collateral comes only from ribbon vaults that we whitelist
upgrading original MarginPool with this code allows the owner to potentially whitelist borrowing of oTokens with ETH / BTC / USDC as collateral. These are oTokens minted by Ribbon retail vaults and StakeDao retail vaults that don't consent to their collateral being borrowed. This upgrade allows for that to be a possibility. Confining these access controls to an entirely separate MarginPool that only has select altcoins in it minimizes risk.

Left some comments, main concerns re figuring out which pool to redeem oToken from. I actually think upgrading the MarginPool and having one MarginPool that keep track of collateral that can be borrowed may be better

@chuddster  Most of it looks fine to me.
Based on the current implementation, agree on:

Keeping the new whitelist functions in the new BorrowablePool, no need to upgrade Whitelist module...etc
Current naming LGTM

I still think the vault type set when opening a vault should change, the proposed implementation in the comment is more generic and does not assume that user who will use the borrowable pool will not need to open spread/naked margin https://github.com/opynfinance/GammaProtocol/pull/459#discussion_r930842294

@akwritescode Can you please check circleci, because some jobs can't be executed from outside opyn org members.

When CI successfully run, based on coverage may need to add some tests, especially one to cover cases where Controller is involved

When CI successfully run, based on coverage may need to add some tests, especially one to cover cases where Controller is involved

can you elaborate?

@chuddster Most of it looks fine to me.
Based on the current implementation, agree on:

Keeping the new whitelist functions in the new BorrowablePool, no need to upgrade Whitelist module...etc
Current naming LGTM

I still think the vault type set when opening a vault should change, the proposed implementation in the comment is more generic and does not assume that user who will use the borrowable pool will not need to open spread/naked margin #459 (comment)

Yes in general I agree your approach is better design (letting owner pass it through) instead of overriding whatever value is passed. Currently we pass in vault type 0 in these vaults which will be overridden with this upgraded controller. This implementation does not require any vaults to be redeployed - we can just call setWhitelistedOptionsVault(true) on all existing addresses and it will override the vault type to 2. We already have 4 treasury vaults that are deployed and have funds in them and we would have to redeploy all of them and pass in vault type 2. This would require all of the treasuries to migrate to a new contract which is undesirable. I agree its a bit hacky.
The only vaults that will be able to open a margin vault in the borrowable pool are ribbon vaults (we will whitelist them) - so we are only constraining ourselves if we ever want to do a naked margin vault.

When CI successfully run, based on coverage may need to add some tests, especially one to cover cases where Controller is involved

can you elaborate?

Just meant when CI run, to check the coverage if there are some important missing paths not tested.


Release version 2.6.0

This is a component issue for 2.6.0.
Coming from opensearch-build__3081__. Please follow the following checklist.
Please refer to the DATES in that post.
How to use this issue

This Component Release Issue
This issue captures the state of the OpenSearch release, on component/plugin level; its assignee is responsible for driving the release. Please contact them or @mention them on this issue for help.
Any release related work can be linked to this issue or added as comments to create visiblity into the release status.
Release Steps
There are several steps to the release process; these steps are completed as the whole component release and components that are behind present risk to the release. The component owner resolves the tasks in this issue and communicate with the overall release owner to make sure each component are moving along as expected.
Steps have completion dates for coordinating efforts between the components of a release; components can start as soon as they are ready far in advance of a future release. The most current set of dates is on the overall release issue linked at the top of this issue.
The Overall Release Issue
Linked at the top of this issue, the overall release issue captures the state of the entire OpenSearch release including references to this issue, the release owner which is the assignee is responsible for communicating the release status broadly.  Please contact them or @mention them on that issue for help.
What should I do if my plugin isn't making any changes?
If including changes in this release, increment the version on __2.x__ branch to __2.6.0__ for Min/Core, and __<IP_ADDRESS>__ for components. Otherwise, keep the version number unchanged for both.

Preparation

[x] Assign this issue to a release owner.
[x] Finalize scope and feature set and update the Public Roadmap.
[x] All the tasks in this issue have been reviewed by the release owner.
[x] Create, update, triage and label all features and issues targeted for this release with v2.6.0.

CI/CD

[x] All code changes for __2.6.0__ are complete.
[x] Ensure working and passing CI.
[x] Check that this repo is included in the distribution manifest.

Pre-Release

[x] Update to the __2.6__ release branch in the distribution manifest.
[x] Increment the version on the parent branch to the next development iteration.
[x] Gather, review and publish release notes following the rules and back port it to the release branch.git-release-notes may be used to generate release notes from your commit history.
[x] Confirm that all changes for __2.6.0__ have been merged.
[x] Add this repo to the manifest for the next developer iteration.

Release Testing

[x] Find/fix bugs using latest tarball and docker image provided in parent release issue and update the release notes if necessary.
[x] Code Complete: Test within the distribution, ensuring integration, backwards compatibility, and performance tests pass.
[x] Sanity Testing: Sanity testing and fixing of critical issues found.
[x] File issues for all intermittent test failures.

Release

[x] Complete documentation.
[x] Verify all issued labeled for this release are closed or labelled for the next release.
[x] Verify the release date mentioned in release notes is correct and matches actual release date.

Post Release

[ ] Prepare for an eventual security fix development iteration by incrementing the version on the release branch to the next eventual patch version.
[ ] Add this repo to the manifest of the next eventual security patch version.
[ ] Suggest improvements to this template.
[ ] Conduct a retrospective, and publish its results.

@msfroh can this be closed out?

I'll take care of the post release actions,