chore(deps): update cosmos sdk to v0.46.13

Description
Burberry patch for the Ojo Chain

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[ ] added appropriate labels to the PR
[ ] targeted the correct branch
[ ] provided a link to the relevant issue or specification
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

Dependency review issues:
https://github.com/advisories/GHSA-qfc5-6r3j-jj22
Looks like x/crisis features are being deprecated

Codecov Report

Merging #203 (2150d0d) into main (22f80cb) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main     #203   +/-   ##
=======================================
  Coverage   18.45%   18.45%
=======================================
  Files          49       49
  Lines       12198    12198
=======================================
  Hits         2251     2251
  Misses       9818     9818
  Partials      129      129

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Weird thing is the e2e stopped working after the branch update with the migration

I just pulled a local price feeder after https://github.com/ojo-network/price-feeder/pull/165 and beforehand, and it looks like it broke our e2e
Gonna look into this a bit 👁


Classes with __enter__ and __exit__ methods, and jitclass decorator can't be used as context managers.

Running:
import numba
from numba.experimental import jitclass

@jitclass
class Ctx:
    a: int

    def __init__(self):
        self.a = 1

    def __enter__(self):
        return self

    def __exit__(self, a, b, c):
        pass

print(numba.__version__)
print(f"__enter__ in Ctx: {'__enter__' in dir(Ctx)}")
print(f"__enter__ in Ctx(): {'__enter__' in dir(Ctx())}")
with Ctx() as something:
    pass

Outputs:
0.56.4
__enter__ in Ctx: True
__enter__ in Ctx(): False
Traceback (most recent call last):
  File "numba_context.py", line 22, in <module>
    with Ctx() as something:
AttributeError: __enter__

It looks like even though __enter__ is defined in the class that's produced it's not defined on instances of it.

[x] I have tried using the latest released version of Numba (most recent is
visible in the change log (https://github.com/numba/numba/blob/main/CHANGE_LOG).
[x] I have included a self contained code sample to reproduce the problem.
i.e. it's possible to run as 'python bug.py'.

Thanks for raising this @mbirtwell and for checking the report @apmasell. I think this is perhaps a feature request as __enter__ and __exit__ are not declared under the supported dunder methods:
https://github.com/numba/numba/blob/1ab6df5faeac1179f062e5ed33d4bbc6ce0ad874/docs/source/user/jitclass.rst?plain=1#L204-L252

Making these an error in #8602


Order pods by priority only over over utilized nodes

Only over utilized nodes need clasification of pods into categories. Thus, skipping categorizing of pods which saves computation time in cases where the number of over utilized nodes makes less than 50% of all nodes or their fraction.
Also, refactoring the test code a bit in order to make lownodeutilization testing table driven.

@aveshagarwal @ravisantoshgudimetla PTAL

/approve


Audio issue: Login Screen and Game Music cutting off constantly

Type
Incorrect behavior
Operating System
Linux
OS Version
PopOS 22.04 LTS
Bug description
As soon as my login screen is loaded on RuneLite and game music starts playing, it randomly stops and keeps cutting off constantly; this persists in game. Apparently, only the music is affected. On the Steam client, however, the game runs perfectly, no sound issues. More specifically, the music plays normally for 3 seconds, and then there's a silence gap of another 3 seconds, and so on. It's pretty annoying.
I tried many solutions involving pipewire issues, but I'm not sure it's about it, as the Steam client (C++) runs perfectly, I wonder If it's some kind of Java related bug.
I doesn't look like it's plugin or runelite configuration related, either.
Screenshots or videos
No response
RuneLite version
RuneLite version: 1.8.29
Launcher version: 2.4.0
Logs
client.log
launcher.log

Duplicate of #14882?

Duplicate of #14882?

Thank you, I'll check it out.


Actually Additions Canola oil too weak

I built a generator with 5 manyullyn turbines, air/gas mixer and gas mix compressor. With Empowered canola oil (4th tier oil), this gives me 2500RF/t, using ~1.93mB/t.
This provides  approx. 1295RF/mB.
However, using a bog-standard oil generator (200RF/t for 800t, using 50mB) generates 3200RF/mB.
For the refined tiers of oil:
Simple oil generator output pr. mB:
refined: 120/240 = 576RF/mB
crystallized: 160/560 = 1792RF/mB
empowered: 200/800 = 3200RF/mB
Advanced generators output pr. mB:
refined: 6,8mB/t = 368RF/mB
crystallized: 3,4mB/t = 735RF/mB
empowered: 1,93mB/t = 1295RF/mB
Output as percentage of oil generator output:
Refined: 63%
Crystallized: 41%
Empowered: 40%
Suggest setting empowered oil output to the following:
Refined canola oil: 600RF/mB
Crystallized canola oil: 1800RF/mB
Empowered canola oil: 3200RF/mB
This will put a basic gas generator without upgrades at the same baseline as a simple oil generator.

Patch adjusting canola values to equal those of oil generators: https://gist.github.com/repvik/6a7822ff503bea150fdf1897af8f1355

The mod has been recently updated to MC 1.16, older versions are no longer supported.
I'm mass closing older issues to clear up clutter in the bug tracker.
If you find this issue still exists in the current version feel free to reopen it.


Add bucket::upload, bucket::delete

Closes #512 , Closes #513

also need to update documentation.


WIP: Bring code using pydantic for config validation up to PR#14

@agoscinski, OK, now this is cleaned up and only contains the changes I did. Feel free to make two other PRs adding the required examples directory, and the core.py file.


No syntax highlighting on One Dark Pro

Hi, thank you for the extension. It works great - except that I don't have any syntax highlighting on the One Dark Pro theme. It seems to work on other themes though.

Hi, it's a theme issue, I can't do much about it. We use the same scope as JSON keys, but the theme seems to have a color override explicitly for JSON only.
You can either fix the theme or set a color for support.type.property-name.toml manually.


anywidget fails on Raspberry Pi.  model.save_changes gives a ReferenceError

Running the "CounterWidget" example as shown in the anywidget blog, fails on a Raspberry Pi. The Pi is is connected to a remote Jupyter server (2i2c.callysto.ca and hub.callysto.ca show the error), running Chromium browser. (Works fine on a Mac or Windows machine.)
The counter button works and increases the count with each count. But the python variable "count" is never updated, and an error message appears when function model.save_changes() is called.

Uncaught ReferenceError: structuredClone is not defined
at AnyModel.serialize, at AnyModel.sync, at AnyModel.save, at AnyModel.save_changes

Your chromium needs to be updated to a later version that supports structuredClone.

Thanks! I upgraded to the new OS "bookworm" which include the new chromium. All i well.


Does not work!

Tried this but it does not compile.

Here is the stacktrace.

                                             `

I had exactly the same problem.  I did nothing and that seemed to fix it.

Yes! It took my week but I found a working solution.

Change this with this.

Nice!

It works on my machine!

Same here, fixed itself

Yhea after few restart got it working

It works on my machine!


Add icon to README and package

Add icon to README and package

:white_check_mark: Package verification completed without issues. PR is now pending human review


Create a button which lets the user decide how many book results to return

🚀 Feature
A clear and concise description of what the feature is.
Might be a good idea  to have a button which lets the user choose how many book results to return from the api call
Is your feature request related to a problem? Please describe.
the user might want to have more results for a particular book search
Describe the solution you'd like
implement a button to let the user decide how many results to show
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

i would like to work on this feature..


DataFrame.cummin()

first cummin version base cummax

Codecov Report

Merging #504 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #504      +/-   ##
==========================================
+ Coverage   91.02%   91.05%   +0.02%
==========================================
  Files          30       30
  Lines        4034     4047      +13
==========================================
+ Hits         3672     3685      +13
  Misses        362      362

Impacted Files
Coverage Δ

databricks/koalas/missing/frame.py
100% <ø> (ø)
:arrow_up:

databricks/koalas/frame.py
93.18% <100%> (+0.12%)
:arrow_up:

Continue to review full report at Codecov.

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

Hello, @HyukjinKwon @ueshin . Travis CI raise error below
The command "if [[ $TRAVIS_PYTHON_VERSION == "3.5" ]]; then pip install -r requirements-dev.txt && pip list; else conda install -c conda-forge --yes --file requirements-dev.txt && conda list; fi" failed and exited with 1 during .

Seems there's an issue at conda server side.

Can you solve it ?

It's not under my control .. let's wait for a bit before taking an action.

@yuwillrun, can you sync with the master branch? https://github.com/databricks/koalas/commit/a96c01ffd026ccb213bf88c8e9e7f23a4b735d9e fixed the build

Im going to take this over at https://github.com/databricks/koalas/pull/521

Thanks for your carrying on working.


Dump z-powers, z-effects, and dynamax powers

The two new tables (z_move_effects and z_move_effect_prose) have the same structure of their non-z counterparts.
Z-powers and z-effects were dumped from Ultra Sun, and dynamax powers from Sword.

Thanks. I'm gonna hold off on merging this one until i have time to give it a proper look.


Arabic Words are reversed

the name of city in arabic are reversed

Shouldn't this be fixed up-stream?

Same for Hebrew words.

This could be fixed with a proper "bidi" and "reshaper" implementations. From my quick search, I found few implementations out there for node but to be honest I did not like any...
One option is to rely on fribidi, but this won't be pure javascript. Maybe check if the system has it, use it, otherwise skip this?
But anyway, I don't think this is really a huge deal. For me, as an Arabic speaker that lives on the terminal, with time, you'll get used to read in reversed order... But if you really need that to be solved and right now, go for bicon.

The intrinsic problem, to begin with, is that terminals have (or well, up to recently had) absolutely no idea what to do on RTL text: whether to automatically reverse it or not.
Some don't reverse (e.g. gnome-terminal, as of the time the above screenshot was posted; xterm; and plenty of others). Some do reverse by default (e.g. newest gnome-terminal; konsole; mlterm; putty). Some make it configurable (e.g. gnome-terminal via an escape sequence, konsole via a preference).
Obviously there's no way you can make it work in both of these kinds of terminals at once.
In my BiDi in Terminal Emulators specification I argue that the behavior of the terminal automatically "reversing" the strings (that is, displaying them from right to left) is the desired behavior for simple command-line utilities, and thus should be the default; however is outright broken (literally impossible to use as a canvas to arbitrarily arrange the letters, cropping of long words etc.) for apps that control the entire UI, such as mapscii.me, so there needs to be an escape sequence to turn it off.
The only reasonable thing for you to do is to print these words in "reverse", i.e. "left-to-right visual" order, by reversing them on the server. If cropping is required (the word crosses the edge of the terminal), this cropping needs to happen on the already reversed ("LTR visual") string. This will be broken in terminals like konsole using their default settings, but you can ask users to disable BiDi in the terminal.
In order to support GNOME Terminal 3.34 (and all other terminals based on VTE 0.58), MinTTY, and any other future terminal implementing this new spec, you have to switch to Explicit LTR mode at startup using \e[8l\e[1 k, followed by a clear screen (or similar) operation when the cursor stands in the upper left corner; and upon exiting restore the default state using \e[8h\e[ k.


Kotlin 1.4 support

Presumably this will require updates to support new language features like fun interface and trailing commas?

Trailing commas are already supported
I skimmed https://blog.jetbrains.com/kotlin/2019/12/what-to-expect-in-kotlin-1-4-and-beyond/, and seems that most new things should already work.
fun interface might actually work out of the box, depending on how it's represented in the AST. If not, it should be a trivial amount of work.

Question - do you happen to have a list of snippets where ktfmt fails? Having them in the format of unit-tests in FormatterKtTest.kt would be wonderful :)

@cgrushko snippet of ktfmt failure for fun interface:
public fun interface PacketHandlerFunction<T : Packet<*>> : (Player, T?) -> T?

I'm getting the error:
33:11: error: Expecting function name or receiver type
com.facebook.ktfmt.ParseError: 33:11: error: Expecting function name or receiver type

Yeah, we need to at least upgrade the parser to 1.4.
I have a draft diff with the changes required to ugprade to 1.4, I'll get to it soon-ish.

Much appreciate your work. Thanks

Any update on this?

@cgrushko do you possibly have the update for this? If you want to open a work in progress pull request maybe I can help?

Apologies for the delay - we're still on 1.3 internally, and I was hoping we'll get to 1.4 any day.
Because we build internally inside a monorepo, we need to make special arrangement to build both for 1.3 and for 1.4.
As I said, I was hoping I can get away with waiting a bit.
The problem is that Kotlin's interface of the visitor has changed in a backwards incompatible way.
Let me see if I can whip up something using reflection.

Almost got it to work: https://github.com/facebookincubator/ktfmt/pull/68
I can't figure out how to tell Maven not to compile Kotlin13InputAstVisitor.kt, so one must first delete this file before compiling.

@cgrushko do you think the new version will be available this week? I am going to start a new project and I would like to test it there to get some conclusions before presenting it to my partners so we can use it over ktlint or detekt

Yes, I expect we'll figure it out soon.

Let me release a new version.

I am checking maven central and I don't find it. Is it in a snapshot repository so I can start testing it?

@JavierSegoviaCordoba, I finally got around to it. There's a link to Maven Central from https://github.com/facebookincubator/ktfmt/releases/tag/v0.19 .

Thank you very much :)

@cgrushko The IntelliJ plugin has to be updated to get the last version? I checked the code and I see it is using val ktfmtVersion = "0.18"
I don't know if there is a way that let you avoid the usage of it directly in the project and adding a spinner menu with al the current versions available in MavenCentral. If you think it is possible I can file a new issue to ask this there as a feature request.

Ugh, the AS plugin is a problem we didn't figure out yet. I think what you suggest makes total sense, and is much better than releasing a plugin every time AS or ktfmt changes, but doing this is probably more complicated (unless IntelliJ plugins have access to such capability from the platform?)
The underlying is reason is of course that we have our own in-house plugin, so we don't dogfood this.


ACCUMULO-3185 Removed references to walogs directory to stop it's creation.

Can one of the admins verify this patch?


Forward multiple ports from docker container

How can I forward multiple ports from docker container (i.e more then one to diff frontends)?
Thx

Hi let me rephrase;
myapp container exposing port 5000,8000 and I wanna redirect like this with traefik
5000 --------------------> myapp.localhost:5000
8000 --------------------> myapp.localhost:7000
thxs
@emilevauge

Duplicate #61

thx for pointing that out


No Functions defined in the schema file. Is the schema file out of sync?

checking the most current schema: https://schema.management.azure.com/schemas/2015-01-01/deploymentTemplate.json, there is no Functions property defined in the schema. Could you clarify what happened here?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0eb51e04-9b4e-c22d-be8c-4ddf4a59a100
Version Independent ID: 2390761b-e1b9-21d1-ea81-7b8d669d0cf2
Content: Azure Resource Manager template structure and syntax
Content Source: articles/azure-resource-manager/resource-group-authoring-templates.md
Service: azure-resource-manager
Product: unspecified
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@cloudvae Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

@cloudvae - thanks for the question. User-defined functions are a new feature, and will be added to the schema soon.
#please-close

@cloudvae We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

thanks for clarification. Any timelines to add this in schema?

@cloudvae - I see that functions were added to the schema.
https://github.com/Azure/azure-resource-manager-schemas/commit/19da1c8261649f9b0d6dfd09eeb4f353a2eaa030#diff-0a8d1ff8d74b0c44e1c75f6b2a66db2c)

@tfitzmac Thanks. I saw that too. But the input is still missing in the resourceCopy. Any ideas when it will be added?


Dependency Injection in Filters

Is it possible to do a DI in Filters. I am looking for Property injection. I want to inject SignInManager into my Filter. I would like to see whether user is signed in and if not Signin the user

File this issue on MVC.


Mysql: json field automigrate not working when embedding model

Declare a model:
type Address struct {
	Street string `sql:"TYPE:varchar(24)"`
}

Now make Address implemnet Scanner (Value and Scan) method.
Now declare another model, when stores Address in embedded json field
type SalesOrder struct {
      Field1 int
      ShipTo *Address `sql:"TYPE:json"`
      FieldN int
}

When I run automigrage to create these tables, I find that datatype of ShipTo field in SalesOrder table is varchar(24) [it should be json]
I figured it down to model_struct.go, line number 195 where it is picking the TagSetting from the last embedded field. However, why this loop was written (@ line 193), I am unable to gather.

+1

Fixed, Thank you for your report.


How to set ASIO buffer size programmatically?

Hi,
I was wondering if anyone know how I can set the buffer size for ASIO programmatically?
I know that I can set the buffer size using ASIO control panel, but can I do this in the c# source code?
Naudio has this function to get the buffer size, but I can't find a similar function to set the buffer size. Is this possible, do I need to modify the source code myself?
public void GetBufferSize(out int minSize, out int maxSize, out int preferredSize, out int granularity)
Thanks for your help.
Shuokai

I'm afraid I don't think ASIO offers a standard way to do this


Destroy

How can we destroy this component?

Hi @iSuslov ,
Looking at Twitter's documentation, you should be able to destroy it using the JQuery selector:
$('.typeahead').typeahead('destroy');

This is missing function. The destroy function should also stop Tracker computation. PR is welcome.


Clarification for the parameter ForceRemval

In the documentation part:
.PARAMETER ForceRemoval
The ForceRemoval parameter allows the removable of guests with default logon information
older than Apr-2020 or for accounts that have not yet had a signin. The DateTime value
might be displayed as 01-01-0001. As exact logon details cannot be provided these accounts
are excluded from remove by default.

Does it refer to these type of users that come out in the exported report?

We want to make sure we are deleting the right guest users.

yes that's right, you will note that some newly created users will also have this value. The script should check the dayssincecreated value and should only remove guest where both the last logon date and createddate exceed the inactive timeframe. You should also have an isinactive colum to confirm which should be removed.
Try using the DifferentialScope switch first to run on a couple guests to confirm the results before running it in bulk.
B
J

yes fair call... I think we should update the inactive field to reflect the -forceremoval switch.
Line 446 - Looks like we are only checking the lastlogon date which means the new accounts would also be removed.

you could remove those users manually and run it without the forceremoval switch

From what I understood, then ForceRemoval parameter is not really necessary to use then?
I have over 25 users with this particular condition, so I believe your suggestion is sound. To manually delete those.

no it's not required. If not used it will just skip those users but I think they will count toward your DifferentialScope limit.


Render setting descriptions as markdown

In the new settings editor, setting descriptions should be rendered as markdown. We should decide when to use quotes, backticks, and links, then make a pass through all settings to make them consistent.

Links

We can allow arbitrary links in description markdown, or add a documentation field to the settings schema which would be a URL to some docs. Then we can show a "See also" link in a standard location to settings that have this in the settings editor.
References to other settings should be links. Clicking the link could search for that setting, and filter the list to just that setting. (Is that a good experience? How do you go back?) How do we specify a "reference to a setting"?

Enum descriptions

We currently don't show enumDescriptions anywhere. These have useful info but settings are inconsistent. Some settings duplicate this info in their description, some don't. Example of setting with duplicated info: https://github.com/Microsoft/vscode/blob/master/src/vs/editor/common/config/commonEditorConfig.ts#L678
We should add the enums and enumDescriptions to the bottom of the setting description, and audit all setting descriptions to remove the duplicated info. It should be displayed like this for the new and old settings editors.

Quotes/backticks

I think that all setting names, enum values, keyboard key names, command names, etc should be surrounded by backticks and rendered as code.

I prefer the documentation link idea over arbitrary links in markdown. Will make for cleaner code.
Maybe since there might be a requirement for multiple links - a user can change the text of the documentation label to say something like docs and examples or whatever else - and then put the contents they'd normally link to in the external docs.


Can't uninstall forestci

This is the result when I try to pip uninstall forestci:
Can't uninstall 'forestci'. No files were found to uninstall.
Any idea what might be causing this?

I have no idea. Sounds like a borked environment and not anything to do with the software per-se.


Show version information on title screen

Splitting off #74 to get the version info work.

Cheers for doing this. Should we add git rev-info as well, or would you like to wait with that until after  the first release?

Hooking the single function seems a lot better as stopgap solution :)


Build number increments every few seconds when open in Visual Studio

Describe the bug
When a solution versioned solution is open in VS and idle, the build number increments every 1-2 seconds.
To Reproduce

Open a solution versioned solution and rebuild. Not build number in cache (json in obj) or host config file.
Wait
Version number increases every few seconds

Expected behavior
No change
Screenshots
Version (please complete the following information):
Environment (please complete the following information):
Additional context

Fixed in release 1.1.0.


[RFC] Transactions using SAVEPOINT

As described here.
The advantage of using savepoints is that callers can issue nested transactions (after fully understanding the implications of how rolling back nested txns work in sqlite, of course). Without this, nested transactions aren't possible.
Some notes:

Preserve existing transaction functionality using a package level knob. That way  Begin() (and friends) can issue BEGIN TRANSACTION or SAVEPOINT depending on the value. This would allow old functionality to be preserved (for folks who care about this).
Savpoint names can be generated by tracking how many transactions have been started (i.e. maintain an int counter variable in SQLiteConn and use it to generate names - this is how activerecord does it in ruby).

I'd be happy to take a crack at a PR for this if you think this is viable.

Nevermind. After thinking harder about this, it seems to me that nested transactions aren't supported by the stdlib sql.Tx interface. That defeats the purpose of this request (savepoints allow us to nest transactions). Without that, we might as well continue with the current implementation.


Could it be that the readme is outdated?

Hello, I found your library and thought it would be cool to test it out, but it's not working for me. I'm getting an error:
[Wed Feb 28 18:37:03 2024] PHP Fatal error:  Uncaught Error: Class 'Endroid\QrCode\ErrorCorrectionLevel' not found in /home/administrador/Desktop/Propheta/propheta.net/sales/qrcode.php:19
Stack trace:
#0 {main}
  thrown in /home/administrador/Desktop/Propheta/propheta.net/sales/qrcode.php on line 19
[Wed Feb 28 18:37:03 2024] [::1]:34186 [500]: GET /qrcode.php - Uncaught Error: Class 'Endroid\QrCode\ErrorCorrectionLevel' not found in /home/administrador/Desktop/Propheta/propheta.net/sales/qrcode.php:19

I'm working with version "endroid/qr-code": "^4.6". I tried using one of your examples, but it didn't work. Here's my code:
index.php
<?php
require_once __DIR__ . '/vendor/autoload.php';

use Endroid\QrCode\Color\Color;
use Endroid\QrCode\Encoding\Encoding;
use Endroid\QrCode\ErrorCorrectionLevel;
use Endroid\QrCode\QrCode;
use Endroid\QrCode\Label\Label;
use Endroid\QrCode\Logo\Logo;
use Endroid\QrCode\RoundBlockSizeMode;
use Endroid\QrCode\Writer\PngWriter;
use Endroid\QrCode\Writer\ValidationException;

$writer = new PngWriter();

// Create QR code
$qrCode = QrCode::create('Life is too short to be generating QR codes')
    ->setEncoding(new Encoding('UTF-8'))
    ->setErrorCorrectionLevel(ErrorCorrectionLevel::Low)
    ->setSize(300)
    ->setMargin(10)
    ->setRoundBlockSizeMode(RoundBlockSizeMode::Margin)
    ->setForegroundColor(new Color(0, 0, 0))
    ->setBackgroundColor(new Color(255, 255, 255));

$result = $writer->write($qrCode);

// Validate the result
$writer->validateResult($result, 'Life is too short to be generating QR codes');

The only thing I modified was removing the logo. Could it be that the readme is outdated? Is there something missing in my code or project?

Hi @gilberto-009199 looks like you used an example from the 5.x readme while running a 4.x version. Make sure you are looking at the readme for your version if you run an older version. For instance if you installed 4.8.5 look at https://github.com/endroid/qr-code/tree/4.8.5. Good luck!


Limit size of commons content

What was wrong?
We need to limit the total size of the "commons" content.
How can it be fixed?
First we need to know the "size" of ContentStorageAPI.  We want a solution that doesn't suffer from O(n) complexity so we probably want to track this as content is added/removed via some sort of aggregation rather than summing everything up each time.
Second we need a way to determine which pieces of content are "least" interesting to us.  For this we probably need to implement ContentStorageAPI.iter_furthest(target_id: NodeID) -> Iterator[ContentKey].  (take a queue from the AdvertisementDB.furthest APIs.
Last, we need a long running process that detects when we are above our storage limit and prunes the content from both the ContentStorageAPI and the AdvertisementDatabaseAPI

did it


Error: Could not retrieve Terms of Service

I am on a mac pro with version 10.13.3 High Sierra
I am receiving an error when trying to create my default wallet. The error message is below:
Could not retrieve Terms of Service:
Could not find a suitable TLS CA certificate bundle, invalid path: /Applications/Electrum.app/Contents/Resources/lib/python36.zip/certifi/cacert.pem
The file that is mentions "cacert.pem" is located at /Applications/Electrum.app/Contents/Resources/cacert.pem
Please correct path for this file.

Try this new release: https://github.com/BTCP-community/electrum-zcl/releases/tag/Z!1.0.3


Adding Client Telemetry

Is your feature request related to a problem? Please describe.
Client requested for request charge API for CosmosException (ref. https://github.com/Azure/azure-sdk-for-java/issues/13488)
Describe the solution you'd like
Develop a telemetry module which collects and save certain kind of metrics periodically. It is already developed for Java SDK (ref. https://github.com/Azure/azure-sdk-for-java/pull/16822) and need to be implemented for .net SDK also.
Describe alternatives you've considered
There is no alternative for this.
Additional context
Default client telemetry capturing is off (User can turn on via builder API)
It contains below aggregations

Document CRUD
Query operation
CPU Memory Information
Metadata cache refresh
Rntbd connection data points

SDK will collect and store below information
timeStamp | Time of capture
clientId | Unique client identifier
processId | host machine identifier
userAgent | User agent container SDK and machine OS version
connectionMode | Direct Vs Gateway
globalDatabaseAccountName | User account name
applicationRegion | User host region if in azure
hostEnvInfo | Host machine information
acceleratedNetworking | If azure vm with accelerated networking
systemInfo | CPU/Memory/Rntbd connection Info
cacheRefreshInfo | Client cache refresh information
operationInfo | CRUD/Query operation aggregation information

JSON would look like this :

Can you provide an example with values for the information being sent back?
TIMESTAMP
HostEnvInfo // What information is  included in this?
AcceleratedNetworking // How can this be detected?
GreaterThan1Kb // Is this a percentage? Count? Bool? Is it for both request and responses?
Consistency // Is the SDK consistency? Is it null if they just use the account default?
// Is this all for latency?
MetricsName
UnitName
Count
Mean
Min
Max
Percentile50
Percentile   90
Percentile   95
Percentile   99
Percentile   999
JsonColumn

Client requested for request charge API for CosmosException (ref. Azure/azure-sdk-for-java#13488)

This is already done in the .NET SDK:
https://github.com/Azure/azure-cosmos-dotnet-v3/blob/f16153b98627890f3328b3c4b3d2d3c2265f22cf/Microsoft.Azure.Cosmos/src/Resource/CosmosExceptions/CosmosException.cs#L97
Can you provide an example with values for the information being sent back?
HostEnvInfo // What information is included in this?
AcceleratedNetworking // How can this be detected?
GreaterThan1Kb // Is this a percentage? Count? Bool? Is it for both request and responses?
Consistency // Is the SDK consistency? Is it null if they just use the account default?
// Is this all for latency?
MetricsName
UnitName
Count
Mean
Min
Max
Percentile50
Percentile 90
Percentile 95
Percentile 99
Percentile 999
JsonColumn

I have updated the schema in the main issue along with sample JSON.

Few clarifications on the wire format

Nesting percentiles: Why are they nested?
Greaterthan1KB: won't making it a text-histogram be more useful? Or in other words what scenarios would boolean address.
RegionsContacted: will per region be a better alternative? (Like what % requests routed to other regions)

Eventually do client expected to post metrics to the current one in PreferredList configured?

It represents out of 100 requests, how many requests took under how much time. e.g.
99.9: 63.93
90.0: 39.18
95.0: 63.93
99.0: 63.93
It means, 99.9 requests took less 63.93 sec to get processed. 90.0 requests took less than 39.19 sec and so on.
That's why they are nested.
This particular field was requested from "Monitoring Team". They just wanted to know if response size was less than 1kb or not.
Here we will just provide the information about the region contacted. Any kind of analytics can be done on kustos as this information will be available in Kustos.


Improve handling of failed cluster connections

See below - when trying to connect to GKE I get an error related to my local setup - but Monokle doesn't seem to handle/notice this and gets stuck in a "connecting..." mode

It seems that we caught every error in code
@olensmar Can you provide more information about how to reproduce the error?

I had an old gcloud client - after I upgraded the error went away - so I guess you would have to install an older version of gcloud (perhaps the error message can help you understand which one..) to reproduce!?


Refactor read parquet table to reduce file list scan based on available partitions

Refactor read parquet table to reduce file list scan based on available partitions
Issue #, if available:
Description of changes:
"Refactor read parquet table to reduce file list scan based on available partitions to improve response times and memory usage
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-pDO66x4b9gEu
Commit ID: 4fb6c3c5fd571504c261539b1ae0af38f235d530
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-pDO66x4b9gEu
Commit ID: 57aaf05ab4ef37febc34580a74df8d48c3e9f376
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-pDO66x4b9gEu
Commit ID: e295af3eaa38988211114d21208a034b011d22b0
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-pDO66x4b9gEu
Commit ID: 252393af800b2e888c97e5551e16eab1b7024d28
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-pDO66x4b9gEu
Commit ID: 18fcb5c69a597ff5723bc474718bc5353865680f
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-pDO66x4b9gEu
Commit ID: 98176dfc3bdc1f904bf9c0c69bd33584022c577f
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-pDO66x4b9gEu
Commit ID: c1f545c1bdd7c9ca4beb722a4024dfbfa823657b
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-pDO66x4b9gEu
Commit ID: a07ebb8150ae9b4d8f8fa43a9b88555ba63917db
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Update metadata for match harbor 2.0

harbor.scanner-adapter/registry-authorization-type need to be removed to match harbor 2.0.
See more https://github.com/goharbor/harbor/issues/11204

After the discussion, we still keep the bearer token type authorization support in harbor.


statuses column on /admin shows only active or most recent dag_runs

https://github.com/airbnb/airflow/issues/974

Can you please rename Status to Recent Statuses here:
https://github.com/airbnb/airflow/blob/master/airflow/www/templates/airflow/dags.html#L23
Ideally also add an info tooltip that fully details what is shown in an info bubble right after that:

then call .tooltip() in javascript to get the nice looking bootstrap tooltip.

@mistercrunch done. What's up with the tests?

We get random failures on travis sometimes, I re-triggered it

Can you add a test that hits that endpoint in here?
https://github.com/airbnb/airflow/blob/master/tests/core.py#L725

@mistercrunch will revisit this in a week or so when I have time.

@jtschoonhoven Ping. Are you able to update this. Would be  great to have this in.

Oh yes, please rebase and I'll merge!

@mistercrunch done


Generate Vectorized Dynamic Beam Allocation Model with Sockeye

I want to replicate the model from Improved Lexically Constrained Decoding
for Translation and Monolingual Rewriting 2019 [1], which is an improved model based on Fast Lexically Constrained Decoding with Dynamic Beam Allocation for Neural Machine Translation 2018 [2].
The data are processed as described in the tutorial Multilingual Zero-shot Translation IWSLT 2017.
First I replicated the model from [2]:
sockeye-train -d train_data \
-vs data/news-commentary-v13.tag.src \
-vt data/news-commentary-v13.tag.trg \
--shared-vocab \
--optimizer adam \
--num-layers 4:4 \
--batch-size 80 \
--loss cross-entropy \
--beam-size 5 \
--checkpoint-interval 10 \
-o post_model

How to create the vectorized version according to [1]?

Please see #1001. If you want to continue to develop lexical constrained decoding, you should use Sockeye 2. Sockeye 3 currently does not support constrained decoding, but we welcome pull requests if you want to port it (or its vectorized version).


Add team release-management_maintainers to all sub project with read access in order to allow Sub Projects to mention the team

Problem description

Currently Sub Project are not able to mention the @camaraproject/release-management_maintainers or add it as reviewers within their repositories as long as the team has no access to the repositories
Expected action

@camaraproject/release-management_maintainers added to all API Repositories with READ access.
Additional context

The issue needs to be executed by an CAMARA admin. The team members will get a notification for each repository they are added to.

Execution done ... will keep the issue open until tomorrow to inform the round within the meeting.

Admin documentation done as well. Closing.


Allow for string values to be used in scale.getPixelForValue

Currently the plugin does not allow for labels using non-numerical values. getPixelForValue supports string values along the axis, so the plugin should probably have this flexibility too.
For example, our team uses timestamps (HH:mm) along the y-axis.

To help illustrate this, here's the non-working (master) version:
https://codepen.io/ground5hark/pen/NjBYPq
Here it is on the working (forked) version:
https://codepen.io/ground5hark/pen/xdJWxY
@compwright mind taking a quick peek at this?

Duplicate of #50


Middleware: Error when adding/validating new Middleware Provider

Have reproduced on MiQ from latest docker image.
Just trying to add new Middleware Provider, fill all necessary fields, click on "Validate" or "Add" buttons, Unexpected error page is shown with error:
"undefined method `downcase' for nil:NilClass [ems_middleware/create]".
@abonas

I can replicate
[----] F, [2016-08-31T16:25:30.094105 #19618:52a6aac] FATAL -- : Error caught: [NoMethodError] undefined method `downcase' for nil:NilClass
/home/jkremser/workspace/manageiq/app/models/ext_management_system.rb:192:in `model_from_emstype'
/home/jkremser/workspace/manageiq/app/controllers/mixins/ems_common_angular.rb:109:in `create_ems_button_validate'
/home/jkremser/workspace/manageiq/app/controllers/mixins/ems_common_angular.rb:81:in `create'
/home/jkremser/.rvm/gems/ruby-2.3.1/gems/actionpack-<IP_ADDRESS>/lib/action_controller/metal/basic_implicit_render.rb:4:in `send_action'
...

is this happening with other provider types as well?
@miq-bot add_label ui

have tried Infrastructure, and it happens as well, with Containers did not see this error

@hhovsepy thanks. ok so this sounds like non hawkular specific issue.
cc @dclarizio

@Jiri-Kremser @hhovsepy is this on docker setup only or in developer (from source) setup as well?

have tried Infrastructure, and it happens as well

I was able to add the infrastructure provider
I am using the developer setup with simulated worker (not the full evm:start)

This looks related to this PR https://github.com/ManageIQ/manageiq/pull/10532
If I add RHEVM provider, the log says:
[----] I, [2016-08-31T16:59:21.416379 #13973:1963d6c]  INFO -- : Started POST "/ems_infra" for ::1 at 2016-08-31 16:59:21 +0200
[----] I, [2016-08-31T16:59:21.438141 #13973:1963d6c]  INFO -- : Processing by EmsInfraController#create as JS
[----] I, [2016-08-31T16:59:21.438539 #13973:1963d6c]  INFO -- :   Parameters: {"utf8"=>"✓", "authenticity_token"=>"NxP1UM3M8prrHG92RwVrBpcecEGh32ElY39jBovbOF2q02Cl1ux7MY8vwE8XtgcWMQ6BtAB5Uc53s4voM8GgEA==", "button"=>"validate", "cred_type"=>"default", "name"=>"sdsdfsdf", "emstype"=>"rhevm", "zone"=>"default", "default_userid"=>"admin@internal", "default_password"=>"[FILTERED]", "default_verify"=>"[FILTERED]", "default_hostname"=>"rhevm.torii.gva.redhat.com", "default_api_port"=>"", "metrics_hostname"=>"", "metrics_api_port"=>"5432", "metrics_database_name"=>"[FILTERED]", "metrics_userid"=>"", "metrics_password"=>"[FILTERED]", "metrics_verify"=>"[FILTERED]", "metrics_auth_status"=>"on"}

But, if I do the same for middleware it does something else:
Started POST "/ems_middleware/create/new?button=validate&type=default"

Then it fails. It looks like the "emstype"=>"foo" is not being send

With Infrastructure provider I see this error:

But unlike Middleware Provider, it remains on the same page.
So probably that is another issue, and this issue is not common, sorry for confusion.

ok thanks for the investigation.
@miq-bot add_label providers/hawkular, bug
@miq-bot assign @rubenvp8510
@rubenvp8510 please check this issue

btw. this is a duplicate of https://github.com/ManageIQ/manageiq/issues/10901

btw. this is a duplicate of #10901

good catch

@hhovsepy please close this as it is a duplicate, this is being handled and linked to the other similar issue

closing as a duplicate of https://github.com/ManageIQ/manageiq/issues/10901


Problem with launching webdriver-manager from the config file

I am new to the protractor. I do not wan to launch webdriver-manager from the command line before running my tests. I want my script to launch the webdriver-manager. I tried setting the '.jar' path inside my config file. But it does not help.
Here is my config file-
exports.config = {
seleniumServerJar: '/usr/local/lib/node_modules/protractor/selenium/selenium-server-standalone-2.44.0.jar',
seleniumPort: '4445',
seleniumArgs: '-help',
seleniumAddress: 'http://localhost:4445/wd/hub',
specs: ['spec.js'],
capabilities: {
'browserName': 'chrome'
}
}
Here is the error I am getting when I am trying to run-
Using the selenium server at http://localhost:4445/wd/hub
[launcher] Running 1 instances of WebDriver
ERROR - Unable to start a WebDriver session.
/usr/local/lib/node_modules/protractor/node_modules/selenium-webdriver/http/index.js:145
callback(new Error(message));
^
Error: ECONNREFUSED connect ECONNREFUSED
at ClientRequest. (/usr/local/lib/node_modules/protractor/node_modules/selenium-webdriver/http/index.js:145:16)
at ClientRequest.emit (events.js:95:17)
at Socket.socketErrorListener (http.js:1552:9)
at Socket.emit (events.js:95:17)
at net.js:441:14
at process.tickCallback (node.js:442:13)
==== async task ====
WebDriver.createSession()
at Function.webdriver.WebDriver.acquireSession (/usr/local/lib/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/webdriver.js:149:22)
at Function.webdriver.WebDriver.createSession (/usr/local/lib/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/webdriver.js:123:30)
at Builder.build (/usr/local/lib/node_modules/protractor/node_modules/selenium-webdriver/builder.js:294:22)
at DriverProvider.getNewDriver (/usr/local/lib/node_modules/protractor/lib/driverProviders/driverProvider.js:38:7)
at Runner.createBrowser (/usr/local/lib/node_modules/protractor/lib/runner.js:180:37)
at /usr/local/lib/node_modules/protractor/lib/runner.js:246:21
at _fulfilled (/usr/local/lib/node_modules/protractor/node_modules/q/q.js:797:54)
at self.promiseDispatch.done (/usr/local/lib/node_modules/protractor/node_modules/q/q.js:826:30)
at Promise.promise.promiseDispatch (/usr/local/lib/node_modules/protractor/node_modules/q/q.js:759:13)
[launcher] Process exited with error code 1
How can I launch webdriver-manager from my script or config file.

You should not specify both seleniumServerJar and seleniumAddress. Remove the seleniumAddress specification. Please see https://github.com/angular/protractor/blob/master/docs/referenceConf.js for more details.
Closing as this is a question.

I appreciate your response. It worked fine after removing that line.

@juliemr I just ran into this as well. Would it be possible to log a warning to the console if you detect both properties are being used in the config?


Notify only about dependencies not devDependencies

:wave:
Is there a way to configure the client to check only for dependencies? I noticed that in request you service can get really annoying just because we have a lot of devDependencies that get constantly updated.

Hey @simov,
we are working on adding a more fine grained control to Greenkeeper. In the meantime you could add back version ranges to your devDependencies, because Greenkeeper will only send you a pull-request if the new version doesn't satisfy the existing range.
Best,
Stephan

That's a nice tip, thanks @boennemann!

Hey @simov,
There are bad news: We have removed this from our immediate to do list.
And there are good news: We think we've found a better way to solve the root of the problem
We think the actual problem you and others reported is that all these pull requests can get overwhelming and quickly become too noisy – a valid complaint.
We've just now deployed a new feature. You can now add version ranges to your dependencies again and we will treat them right.
For new versions inside the existing range you’ll get pull-requests only if the update breaks your build. We are doing this by only creating the branch and then waiting for results from your CI. If it passes the branch gets deleted again, if it fails however the pull requests gets opened, because it should definitely have your attention.
-  "underscore": "^1.6.0"
+  "underscore": "^1.7.0"

When a new version is outside of your existing range you’ll just get the pull request right away.
-  "lodash": "^3.0.0"
+  "lodash": "^4.0.0"

You can now define the level of "noise" per dependency by using "^", "~", or pin.
We hope you like this change and would be very happy to get your feedback on this.
Best,
Stephan

After using it for some time, I can say that it still gets noisy. I simply don't care about devDependencies. I'll update them whenever I get a chance, I just don't need GK to keep opening PRs for new versions.
Why is it so hard to add a flag to ignore dev deps?


pull request to Devel

i have commit all changes
please compare and check this code than merge this code
Thanks

@MohitMaliFtechiz can you explain in the description how OpenStruct works

LGTM!


upgrade to setup-gradle@v3

replaces https://github.com/projectnessie/iceberg-catalog-migrator/pull/127
note that the action has been renamed:
https://github.com/gradle/gradle-build-action
also simplify actions/checkout usage

i wanted to simplify actions/checkout at the same time
also i am linking to the renovate PR in the commit msg, so (unless i am missing something) the info should still be easily available?


dup (in figure) and bias (in model.txt)

Hi, I currently run genomescope2 and got some questions.

why bias in "model.txt" is written in dup in figrure, not bias??
What do they mean respectively? Are they same?

They are the same. In the statistics literature, the 'bias' term in a
negative binomial is used to refer to the overdispersion relative to a
Poisson distribution. But in a genomics context, we found the bias is
driven by PCR duplications and other technical duplications in the reads so
they are described as 'dup'.
Good luck!
Mike
On Thu, Apr 25, 2024 at 8:24 AM shin0727 @.***> wrote:

Hi, I currently run genomescope2 and got some questions.

why bias in "model.txt" is written in dup in figrure, not bias??
What do they mean respectively? Are they same?

—
Reply to this email directly, view it on GitHub
https://github.com/schatzlab/genomescope/issues/128, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABP3447ZVU4577VF2XTPCLY7DYY3AVCNFSM6AAAAABGYYWDX6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGI3DGNBVGQ4DSMI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

They are the same. In the statistics literature, the 'bias' term in a negative binomial is used to refer to the overdispersion relative to a Poisson distribution. But in a genomics context, we found the bias is driven by PCR duplications and other technical duplications in the reads so they are described as 'dup'. Good luck! Mike
…
On Thu, Apr 25, 2024 at 8:24 AM shin0727 @.> wrote: Hi, I currently run genomescope2 and got some questions. 1. why bias in "model.txt" is written in dup in figrure, not bias?? 2. What do they mean respectively? Are they same? — Reply to this email directly, view it on GitHub <#128>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABP3447ZVU4577VF2XTPCLY7DYY3AVCNFSM6AAAAABGYYWDX6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGI3DGNBVGQ4DSMI . You are receiving this because you are subscribed to this thread.Message ID: @.>

Thank you for your response! I understand now.


refactor(warnings): fix all missing override warnings

There was a code path in os.MapContainer.addFeature() that was accepting a 'feature' that was really an ol.Coordinate. The function definition does not include ol.Coordinate anywhere, so that path has been removed.
I also removed externs/svg.js in favor of the one from the compiler project.
Resolves #32

Looks like travis is wedged.


Update openpai

Updated openapi api specs to change the license from MIT to Apache 2.0

Can one of the admins verify this patch?


Create prominent space for speakers on website

Once keynotes are locked in, display prominently with pics and VERY short tagline bios.  This is a main draw for bringing folks to conference, and should be displayed prominently on site.

goes hand in hand with #104

Implemented at http://www.force2017.org/information/keynotes.html.


[DO_NOT_MERGE] Adding ConfirmNotificationDialog

Summary
Issue: #177
I thought it may be easier to talk about this with the code mostly written and examples to play with. Feel free to take a look at the deployment to see how the examples are set up. I'll be annotating the PR with questions that I still have or would like suggestions on.
Two big changes here:

Creation of _transientUtils.jsx

This file exports mount/unmount functions that will generically mount/unmount a React tree separate from the main React tree. An id is required to identify the mount node, along with the component instance to be mounted and locale/customMessages values to be given to the Base instance that wraps the component automatically.
A HOC-generator called injectTransientComponent is also exported so that relevant data can provided to the presented component for self-management, if desired. This is currently utilizes the Context for transmission.
This is meant to be a agnostic set of utilities that will allow any component to be transiently mounted.

Creation of ConfirmNotificationDialog.jsx

This component is a version of the NotificationDialog meant to be presented with the utilities provided by transientUtils.jsx.
It exports its own mount/unmount functions for ease of use by consumers.
A subset of the NotificationDialog props are supported (isOpen/header should no longer be necessary).

Any alternatives to this type of implementation would create complicated or inefficient logic within components that want to integrate NotificationDialog presentation with Promise chains and other synchronous-ish flows.
I just want to get some opinions on this and pick some brains before I harden it too much. I'll be happy to get naming suggestions 😄

Closing this because it's outdated as all hell. I'll come back to it.


Site object does not contain Usage property so export fails

The script at https://github.com/pnp/script-samples/tree/main/scripts/spo-export-site-usage-reports fails.
When checking it seems that the Site object (stored in the $Reports variable) does not contain the Usage property.
Has something changed between PnP.PowerShell versions?
I have tested with PnP.PowerShell 1.12.0 and PnP.PowerShell 2.1.1, but same thing.

@pkbullock i can investigate and fix  if required

I am having the same issue.

Usage property can be retrieved
 foreach ($Report in $Reports) {
            try{
            $report = get-pnptenantsite -Identity $Report.Url -Detailed
            Connect-PnPOnline -Url $Report.Url -Interactive
            $site = get-pnpsite -Includes Usage
            }
            catch{
                Write-Host "Error in getting site usage report:" $_.Exception.Message -ForegroundColor Red
            }
  <#Add desired properties to export by referencing
 $Site.Usage.Hits
}

However the value of  $Site.Usage.Hits, $Site.Usage.Visits, etc.. are 0 except $Site.Usage.Storage and $Site.Usage.StoragePercentageUsed
I could not find out how to get these values, either the script can be amended to remove the usage data or comment added that the usage data are not returned for SharePoint Online.


create camel-leshan component

camel-leshan component provides all OMA Standard Object Model and IPSO Smart Object too

#108 will provide the gateway/server interface

Duplicated by #318.


add disclaimer

have to add this to all our repos

@nvollmar could you have a look when you have time? I can help with fixing the merge conflicts that will appear in https://github.com/apache/incubator-pekko-management/pull/10

@pjfanning shouldn't be a big deal. Need to solve the issues in #10  anyway once I find some time to dig deeper


Remove skips from doctests (4 of 6)

This is one of 6 PRs that collectively addresses #7823. Please see https://github.com/dask/dask/issues/7823#issuecomment-873414398 for more context.
Links to all 6 PRs

#7874
#7873
#7872
#7865
#7864
#7863

@jsignell Ready for review.


how to get mnemonic from privatekey

how to get mnemonic from privatekey?

Are you talking about BIP32 keystore? There is no direct backward conversion from the private key to the mnemonic part. At the point of creating of a keystore you should ask a user to save a pass phrase. After seed phrase is converted to some initial entropy the “master key is derived” and the initial entropy is discarded, so the restoration of a seed phrase is not possible


Feature #203 Make 'title' optional

Hello, I'm a new open source contributor and I'd like to make an effort on this project to solve some existing issues.
This one is about make "title" optional. Please feel free to give me any feedback about the change.
Thanks!

Apologies for the radio silence, I got super busy and couldn't find time to maintain this project. We have started to actively use it at my current company and decided to rewrite it due to underlying implementation issues. It now has much simpler API and is feature rich as compared to the previous version.
Please have a look at the release docs for v1.0 and the documentation and do let me know if you face any issues. Thanks again for taking your time to submit the PR! 🙏


Add owners file and fix shellcheck issues

/test-integration-metal3-dev-tools-centos

/test-integration-metal3-dev-tools-centos

/retest

this should be configured in the GitHub repo settings I believe

 this should be configured in the GitHub repo settings I believe

its done already, replied in slack

/test all

/hold cancel

/test-integration-metal3-dev-tools-ubuntu
I configured the branch to make the above job as required since it will be added with the https://github.com/metal3-io/project-infra/pull/470

/hold

/test-integration-metal3-dev-tools-ubuntu
/test-integration-metal3-dev-tools-centos

/hold cancel

/test-integration-metal3-dev-tools-ubuntu
/test-integration-metal3-dev-tools-centos

/test-integration-metal3-dev-tools-ubuntu
/test-integration-metal3-dev-tools-centos

/test-integration-metal3-dev-tools-ubuntu
/test-integration-metal3-dev-tools-centos


[ARM] az deployment rg/sub/mg/tenant create: Add --what-if and --proceed-if-no-change parameters

Description
Closes #18481.
Closes https://github.com/Azure/bicep/issues/2923
Testing Guide
az deployment sub -l westus -f ./deploy.json --what-if
az deployment sub -l westus -f ./deploy.json --confirm-with-what-if --proceed-if-no-change

History Notes

[ARM] az deployment rg/sub/mg/tenant create: Add --what-if parameter for invoking What-If with the deployment create commands.
[ARM] az deployment rg/sub/mg/tenant create: Add --proceed-if-no-change parameter to skip confirmation when --confirm-with-what-if is set and there's no changes in What-If results.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

ARM


Re-enable the retry logic for App Password creation

PR #332 inadvertently broke app password creation retry logic.
This re-adds it in a fashion that calling registration.py directly benefits from the retry attempts.

Note, this PR also addresses issues from flake8.  I will look to enabling flake8 in the cicd pipeline in the near future.


Command in notebook fails without useful details

Connected to BDC
Uploaded a cvs in HDFS
From menu items, selected "Analyze in Notebook"
Asked me for python installation and completed successfully
Running the command in the notebook

Expected: command should run successfully
Actual: The command gives me an error:
Starting Spark application
The code failed because of a fatal error:
Session 3 did not start up in 60 seconds..
Some things to try:
a) Make sure Spark has enough available resources for Jupyter to create a Spark context.
b) Contact your Jupyter administrator to make sure the Spark magics library is configured correctly.
c) Restart the kernel.
shouldn't it give me some more info about the error and some linked doc or something to show how to do those 3 things?

duplicate of #4427

More info:
looks like installing dependencies was not completed for some reason. the last message I saw in my output was "... Jupyter is running at"
so I ran "Reinstall notebook dependencies" which failed because there were python processes running:
Cannot overwrite an existing Python installation while python is running. Please close any active notebooks before proceeding.
closed the processes and ran the command again. This time it downloaded some python packages after "... Jupyter is running at"


updating plugin - fix disappearing "obsidian file explorer" - add menu - improve embed in obsidian note

Hi,
First of all thank you for creating this plugin.
It can really be of tremendous help if it would be better supported / have better file / menu access.
Ideas that jump to mind:

fix the disappearing of the "classic" obsidian file / note explorer (after opening / using the Diagram menu, the Obsidian file explorer disappears)
create a way to interface with the plugin, either by menu or pop-up (it seems there are some pop-up menu items, but how can we save a diagram? rename this diagram?
create a way (menu) to insert the diagram in the adjacent obsidian note, as image or as svg

To be honest:
I don't have the slightest idea how to build a plugin like this, but it is a shame that it exists without being usable on layman's level.
Thanks for helping out!

unusable for me because of this error... next thing could be considered by developer is option of inherit the theme font (using default)
Thanks

Seems the same as #73

same here, because of this behavior I can not use this plugin...

Same things are happening for me. Unfortunately that makes the plugin just about unusable.
It looks as though this was partially fixed in #64, but there hasn't been a release since that was merged. Could we at least get a release which includes that fix?

@xh542428798 How you tell us more how to build this plugin from scratch? What command would I use? I am not really familiar with how to do this.

main.txt
Okay, I was able to rebuilt the plugin, this is the file you would need to replace under .obsidian/plugins/drawio-obsidian, which should fix the problem.


Error messages with dl_docs

Hi! First of all, thanks for all the work you put into this project and for sharing it here! High appreciated!
When I saw you updated it with a fix for the dl_docs problems, I went right ahead and downloaded everything from my TR-Acc. I downloaded a total of 100s of documents and while scrolling through the lines to see if everything worked smoothy, I noticed some "error-line", like such (maybe like 15 or so in total out of 100s of downloads):
13:41:17 2XX/9XX: Kauf StockA -- Di. 18.06. um 16:43 Uhr (Europe/Berlin) Received error message: '546 E {"errors":[{"errorCode":"BAD_REQUEST","errorField":null,"errorMessage":"Bad Request","meta":{"source":"PLATFORM"}}]}' 13:41:17 ('546', {'type': 'timelineDetail', 'id': '8g56243-6casd-1fff-22b8-034ed40e4d1c'}, {'errors': [{'errorCode': 'BAD_REQUEST', 'errorField': None, 'errorMessage': 'Bad Request', 'meta': {'source': 'PLATFORM'}}]})
I reran the command several times and the errors showed up in exactly the same lines every time.
Any idea what the problem might be?
Just wanted to share it also to contribute as best as I can here by at least reporting what I found.
Cheers!

As I said it is not really fixed. It just ignores the errors for now

errors do not lead to a stop of the script but I am still not able to do a dl_docs. After two errors it hangs and nothing happens and no files are being downloaded...
14:11:58 Logged in
14:11:58 Awaiting #1  timeline
14:11:58 Received #1  timeline, awaiting #2  timeline
14:11:59 Received #2  timeline, awaiting #3  timeline
14:11:59 Received #3  timeline, awaiting #4  timeline
14:11:59 Received #4  timeline, awaiting #5  timeline
14:11:59 Received #5  timeline, awaiting #6  timeline
14:11:59 Received #6  timeline, awaiting #7  timeline
14:11:59 Received #7  timeline, awaiting #8  timeline
14:11:59 Received #8  timeline, awaiting #9  timeline
14:11:59 Received #9  timeline, awaiting #10 timeline
14:11:59 Received #10 timeline, awaiting #11 timeline
14:11:59 Received #11 timeline, awaiting #12 timeline
14:11:59 Received #12 timeline, awaiting #13 timeline
14:12:00 Received #13 timeline, awaiting #14 timeline
14:12:00 Received #14 timeline, awaiting #15 timeline
14:12:00 Received #15 timeline, awaiting #16 timeline
14:12:00 Received #16 timeline, awaiting #17 timeline
14:12:00 Received #17 timeline, awaiting #18 timeline
14:12:00 Received #18 (last) timeline
Received error message: '19 E {"errors":[{"errorCode":"BAD_REQUEST","errorField":null,"errorMessage":"Bad Request","meta":{"source":"PLATFORM"}}]}'
14:12:00 ('19', {'type': 'timelineDetail', 'id': '6ccfee3d-3585-4965-bf2d-1d83bbea57df'}, {'errors': [{'errorCode': 'BAD_REQUEST', 'errorField': None, 'errorMessage': 'Bad Request', 'meta': {'source': 'PLATFORM'}}]})
14:12:00 Received #19 (last) timeline
14:12:00   1/540: Abschluss Q4/2019 -- Mi. 01.01. um 00:59 Uhr (Europe/Berlin)
14:12:00   2/540:   -- Do. 30.05. um 11:53 Uhr (Europe/Berlin)
14:12:00   3/540: Basisinformationen -- Do. 30.05. um 11:53 Uhr (Europe/Berlin)
14:12:00   4/540:   -- Do. 30.05. um 11:53 Uhr (Europe/Berlin)
14:12:00   5/540:   -- Do. 30.05. um 11:55 Uhr (Europe/Berlin)
Received error message: '28 E {"errors":[{"errorCode":"BAD_REQUEST","errorField":null,"errorMessage":"Bad Request","meta":{"source":"PLATFORM"}}]}'
14:12:00 ('28', {'type': 'timelineDetail', 'id': '97ab309c-3101-45da-98f7-36d92172e041'}, {'errors': [{'errorCode': 'BAD_REQUEST', 'errorField': None, 'errorMessage': 'Bad Request', 'meta': {'source': 'PLATFORM'}}]})
14:12:00   6/540:   -- Do. 30.05. um 11:55 Uhr (Europe/Berlin)
14:12:00   7/540: Core MSCI EM IMI (Acc) -- Sparplan
14:12:01   8/540: Core MSCI World -- Sparplan
14:12:01   9/540: Core MSCI World -- Sparplan

Hm weird, I wanted to investigate the error more. The last version was just a quick fix which seems to work for most

yeah - no worries. I just wanted to make you aware ... if I can assist in any way, please tell me (not a coder though)

Correction: For some reason after the 3rd try it worked

As I said it is not really fixed. It just ignores the errors for now

Alright, superb, I get it! :)
Just wanted to make sure to share anything that might help to improve the tool.

I still get an error message but it is downloading correctly now. Thank you for the help.

I get the error, but its not downloading. anyone else got the problem?

I get the error, but its not downloading. anyone else got the problem?

are you on the newest version?

yes

Same happened for me. I can’t download the files. The script tells me 3 errors and after that it simply stops. Looks like this:
9/405…. Sixt
I waited almost an hour but it just stopped. I tried restarting it several times. Sometimes it runs up to 8 the next time to 9 but there appear no documents.

did anyone get it working again?

I currently also have the problem that the downloads no longer work.
@marzzzello Can you fix it again?

This is now the latest error I receive. It appears in python 3.8 and 3.7. Looks like it catches a wrong phrase somewhere?

Check if 1c7b251 fixes your issue. I also made a new release.
P.S. You can open a new issue next time as this is an independent issue

Thanks, works like a charm now.
I just wonder why TR doesn't want us to have these files, they explain they don't have them but clearly they are there and well sorted.
Thanks again

Hey there. I have the same issue:
22:42:03 1119/1119: Kauf Zoom -- Fr. 18.02. um 19:33 Uhr (Europe/Berlin) 22:42:03 Received all details Traceback (most recent call last):   File "/opt/homebrew/bin/pytr", line 8, in <module>     sys.exit(main())   File "/opt/homebrew/lib/python3.9/site-packages/pytr/main.py", line 157, in main     asyncio.get_event_loop().run_until_complete(dl.dl_loop())   File<EMAIL_ADDRESS>line 642, in run_until_complete     return future.result()   File "/opt/homebrew/lib/python3.9/site-packages/pytr/dl.py", line 64, in dl_loop     await self.tl.timelineDetail(response, self, max_age_timestamp=self.since_timestamp)   File "/opt/homebrew/lib/python3.9/site-packages/pytr/utils.py", line 365, in timelineDetail     export_transactions(dl.output_path / 'other_events.json', dl.output_path / 'account_transactions.csv')   File "/opt/homebrew/lib/python3.9/site-packages/pytr/utils.py", line 107, in export_transactions     lang = getlocale()[0].split('_')[0] AttributeError: 'NoneType' object has no attribute 'split'
I am on the lates version (0.1.1)

@trulyarya  That's a new issue. What's your system locale?

@trulyarya That's a new issue. What's your system locale? Which OS?

Servus @marzzzello !
Thanks for the quick reply.
My locale:

LC_COLLATE="C"
LC_CTYPE="UTF-8"
LC_MESSAGES="C"
LC_MONETARY="C"
LC_NUMERIC="C"
LC_TIME="C"
LC_ALL=

My system:

Mh, I see. You can try to start pytr with LANG=en_US pytr dl_docs ... as a workaround until I push a fix

I did that and even tried other languages, still the same AttributeError: 'NoneType' object has no attribute 'split' error as above, and funnily enough on the same stock purchase (Zoom).
Doing
import locale
locale.getlocale()

gives  this (Python 3.9.10):
(None, 'UTF-8')

I think some LC_ variable needs to be set but never mind, I implemented a fallback fix in https://github.com/marzzzello/pytr/commit/5264843751ad5c8465351c9f3422b00643891496 and released a new version (0.1.2)

What does the following return:
import locale
locale.getdefaultlocale()

Still gave me (None, 'UTF-8').
I even did export LANG=en_US.UTF-8 in zsh terminal, which gave me LANG="en_US.UTF-8" when typing locale.
But even after this, LANG=en_US pytr dl_docs PATH or simply pytr dl_docs PATH gave the same 'AttributeError' like before.

You could also try: LC_CTYPE="en_GB.UTF-8" pytr dl_docs
because LC_CTYPE overwrites LANG

Thank you! I tried LC_CTYPE="en_US.UTF-8" pytr dl_docs and it worked flawlessly with pytr v 0.1.1 !

I think some LC_ variable needs to be set but never mind, I implemented a fallback fix in 5264843 and released a new version (0.1.2)

Awesome. Now it works without setting/changing the language. Thanks for the fast version update!


MSYS2 no longer provides libusb for MINGW32

https://github.com/msys2/MINGW-packages/commit/db6517eb5ab862427c95f3a43fcee177dd6d324e
wtf?

https://www.msys2.org/news/#2023-12-13-starting-to-drop-some-32-bit-packages
seems this is intentional, but quite annoying to find out nonetheless.
fixed in 1eeb3a13ebdf4a6235e6f7cc921c53dd4c40b37b


DynamicSuggestedArgument missing context about partially typed command

Have been playing with the very new DynamicSuggestedArgument some, and it has one odd quirk which makes it much less user friendly than other argument suggestion options.
Minecraft isn't actually taking into account the characters you start typing for the suggestions. So for example if you have "alice" and "bob" as suggestions, if you start typing "bo", it calls the completer and asks for the completions - which returns "alice" and "bob". So minecraft returns possible completions "alice" and "bob" to the client. The problem is - "alice" is not a valid completion for "bo". So you get a dynamic list that contains a bunch of stuff you don't want, and it doesn't narrow down the more characters you type.
I'm not sure if you can get at the client's partially completed string - but I bet you can. If so, can you pass that in as an argument to the functional interface? Then when I implement my dynamic completion callback I can look at the partial match and only supply back suggestions that match it.
Thanks!

After testing, I realise this also occurs with the SuggestedStringArgument as well. I'm now aware that it is possible to (somewhat) get the partially completed string, however I believe that the CommandAPI should handle that entirely. In other words, the CommandAPI should handle the partial matching by itself and that shouldn't come down to the responsibility of the user.
It's true that passing the partially completed string back to the user allows them to handle the suggestions as they see fit, allowing very complex suggestions such as @e -> @e[ -> @e[limit= etc. however you have to remember that the CommandAPI is a wrapper for Brigadier. Handling such complex suggestions isn't something I really fancy doing and if someone does want to go the extra mile, they're more than welcome to use Brigadier itself. Trying to implement a wrapper for suggestions doesn't seem feasible at the moment - my aim with the CommandAPI is to "hide" Brigadier from the user.
I've fixed the issue in this commit which implements the "standard" suggestion list.


Rewrite Tailscale ping using spawnSync

Plus some small changes

I am curious to know why you changes version to 2.0.0-dev? but i see milestone of 1.23.7 can u tell ur release process

I am curious to know why you changes version to 2.0.0-dev? but i see milestone of 1.23.7 can u tell ur release process.
Vite 5 👀

Incidentally included here, will try it later.


error when creating wallet

commit 888e94421733940a42b559b26b5d6683582c4059
Uncaught Error: Objects are not valid as a React child (found: TypeError: Cannot read property 'value' of undefined). If you meant to render a collection of children, use an array instead.
    in p (at SeedConfirmationModal.js:54)
    in div (created by Message)
    in Message (at SeedConfirmationModal.js:52)
    in div (created by FormField)
    in FormField (at SeedConfirmationModal.js:13)
    in form (created by Form)
    in Form (at SeedConfirmationModal.js:12)
    in div (created by ModalDescription)
    in ModalDescription (at SeedConfirmationModal.js:10)
    in div (created by ModalContent)
    in ModalContent (at SeedConfirmationModal.js:9)
    in div (created by Modal)
    in RefFindNode (created by Ref)
    in Ref (created by Modal)
    in div (created by Modal)
    in RefFindNode (created by Ref)
    in Ref (created by PortalInner)
    in PortalInner (created by Portal)
    in Portal (created by Modal)
    in Modal (at SeedConfirmationModal.js:7)
    in SeedModal (at NewAccount.js:201)
    in div (at NewAccount.js:200)
    in NewAccount (created by Connect(NewAccount))
    in Connect(NewAccount) (created by Route)
    in Route (at Routes.js:26)
    in Switch (at Routes.js:17)
    in App (at Routes.js:16)
    in _default (created by Root)
    in Router (created by ConnectedRouter)
    in ConnectedRouter (created by Connect(ConnectedRouter))
    in Connect(ConnectedRouter) (created by Root)
    in Provider (at Root.js:47)
    in Root (created by Connect(Root))
    in Connect(Root)
    in AppContainer (at app/index.js:10)
    at invariant (/Users/rob/workspace/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/react-dom/cjs/react-dom.development.js:55)
    at throwOnInvalidObjectType (/Users/rob/workspace/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/react-dom/cjs/react-dom.development.js:11833)
    at reconcileChildFibers (/Users/rob/workspace/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/react-dom/cjs/react-dom.development.js:12607)
    at reconcileChildren (/Users/rob/workspace/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/react-dom/cjs/react-dom.development.js:14402)
    at updateHostComponent (/Users/rob/workspace/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/react-dom/cjs/react-dom.development.js:14863)
    at beginWork (/Users/rob/workspace/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/react-dom/cjs/react-dom.development.js:15649)
    at performUnitOfWork (/Users/rob/workspace/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/react-dom/cjs/react-dom.development.js:19312)
    at workLoop (/Users/rob/workspace/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/react-dom/cjs/react-dom.development.js:19352)
    at renderRoot (/Users/rob/workspace/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/react-dom/cjs/react-dom.development.js:19435)
    at performWorkOnRoot (/Users/rob/workspace/github.com/threefoldfoundation/tfchain-wallet-electron/node_modules/react-dom/cjs/react-dom.development.js:20342)

This bugged occured because a mnemonic entered was wrong, we now validate the mnemonic and give and error if it falsy. This issue should now be resolved

reopened until I verified since I copy-pasted the mnemonic

Please verify first, only re-open when you suffer from it, don't really want this to be open for days and weeks untouched.


bump swagger-jersey-jaxrs from 1.5.15 to 1.6.1

FINERACT-963

I've exceptionally "self merged" this PR, because technically I'm hereby reviewing and approving a change made by Dependabot so with green Travis, we got 4-eyes on this - my 2 eyes, and 1 each from Travis and Dependabot.. :rofl:
Swagger unfortunately isn't really finished anyway (see FINERACT-733), so the "risk" here is very low.


Fix typo in permission example

Seems it is duplicate of https://github.com/PaoloRotolo/AppIntro/pull/167

Yes, thats right.


Change locale with numeral causes Javascript error

This might be more of a question rather than an issue, but how are we supposed to change the locale language if we want to switch up to different locale?
Based on the documentation, I've included the option to include all languages.
var app = new EmberAddon(defaults, {
    numeral: {
        includeLanguages: true
    }
});

However, when trying to change the locale, we seem to hit an error:
import Ember from 'ember';
import numeral from 'numeral';

export default Ember.Service.extend({
    formatNumber(value = 0) {
        numeral.locale('fr');

        // Cast anything in the value into a number.
        value = typeOf(value) !== 'number' ? +value : value;

        // If this is a number, then format it. Otherwise, force it to return 0
        return !isNaN(value) ? numeral(value) : 0;
    }

});

This is the error we're getting.
TypeError: _numeral.default.locale is not a function

What is the proper way to switch the locale?
Thanks!

Apparently there's an issue with NumeralJS documentation. The way to change locale is actually:
numeral.language('fr');

Closing this issue as there's nothing wrong with the addon. :)
Thanks for the great work!


Enhance building from sources

The framework script used when we build a Cobalt app using the iOS Cobalt sources is currently writing the .framework and .bundle files on the user desktop.
The app developper has to remove them by hand if he doesn't want a fail on the next build.
Enhance this by removing previous file on each build and creating it in the Cobalt folder instead of the user desktop.

Now, .framework and .bundle files are located in this folder: ${BUILD_DIR}/${CONFIGURATION}-all
The CONFIGURATION var can take 2 values: Debug and Release
The ${BUILD_DIR} var relies on the CONFIGURATION value:

Debug: Build/Products,
Release: Build/Intermediates/ArchiveIntermediates/${TARGET_NAME}/BuildProductsPath.

The TARGET_NAME var is what you have chosen to build with Xcode (see picture below in RED or BLUE)

Finally, this folder is located in the DerivedData folder whose you can find its path in the Preferences pane of Xcode (see picture below)

So, in the pictures above, the paths could be:

/Users/${USER}/Library/Developer/Xcode/DerivedData/Catalog-${ID}/Build/Products/Debug-all
/Users/${USER}/Library/Developer/Xcode/DerivedData/Catalog-${ID}/Build/Intermediates/ArchiveIntermediates/Catalog/BuildProductsPath/Release-all

Moreover, since I moved the files there, it seems I did not need to remove them manually nor with the embedded script.


[bugfix] use snake case on generated files for query params

When using query params with more than one work like CustomQuery the effectively used param is custom_query but the field name in generated code is customquery and therefore examples given for the field do not appear...

Codecov Report

Merging #18 (ed99533) into master (578b3cd) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #18   +/-   ##
=======================================
  Coverage   56.23%   56.23%
=======================================
  Files          15       15
  Lines         882      882
=======================================
  Hits          496      496
  Misses        308      308
  Partials       78       78

Impacted Files
Coverage Δ

gen/fields.go
82.50% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add thumbdrive & GitHub SCB backup to the "Lightning" section of the guide

What
This PR adds a section in the LND guide to create an automatic thumbdrive-based SCB backup.
As discussed in & Fixes #783
Why

Resilience: Having an updated SCB backup stored outside the SSD is critical to ensure that a user with a bricked node will recover most of his offchain funds (local balances). See discussion in #783. Not having an updated SCB could result in most of the funds locked in the channels for a very long term (or even possibly permanentaly if a peer goes offline permanently).

Didactic: This guide also allow the user to learn how to build and run a shell script from scratch.

How

Added a 'Resilience' section in the LND
Mentionned an optional remote backup and points to Dropbox bonus guide (the bonus guide will be replaced soon by an alternative using Github, see #783
Added a small thumbdrive to the list of required equipment in the 'Preparations' page

Notes/questions:

There is probably a lot of room to improve the guide and make it more efficient.
I'm not the options I chose for the fstab file line are the best, could you check and correct?
I feel the introduction that explains what a SCB is is needed (because of the widespread belief that it aloows the recovery of channels) but I'm sure the wording could be improved a lot

Scope

[x] significant change to core configuration
[ ] independent bonus guide
[ ] simple bug fix

Fixes #783
Test & maintenance
Test:

test the guide with a thumbdrive

Maintenance:

The link to the Dropbox bonus guide will have to be replaced by a link to the Github bonus guide once this guide is replaced.

Animated GIF (optional)

I'll review this pr next. It's quite long and I think the LND section would become huge... How about adding a new section to the main guide (after LND, before RTL) called "Channel backup" or similar?
This could be the starting point, but we could also include the GitHub remote backup option later.
Main structure:

Intro: why are channel backups important?
TL;DR: options USB and/or Remote
Guide: USB
Guide: Remote

WDYT?

I added some first feedback, no wording or nitpicking just yet :)
TL;DR
* I think we should try to run the service & script with user "lnd" instead of "root" (minimal required permissions)

* This requires the thumbdrive to be mounted as user "lnd"

Also, maybe we could write the backup files in a versioned format, with a filename that includes time & date? This avoids corrupting an existing good backup.

Very interesting feedback, thanks! I learnt a lot :)
I pulled the guide out of LND and placed it between LND and RTL. It feels a bit out of place since the SCB backup is still LND-specific.. but the argument that the LND guide is long enough as it is makes a lot of sense! So I think you're right that it should be its own separate guide (particulalry when the remote GutHub guide will be fleshed out there as well..).
Otherwise, I believe I updated all the suggestions, hoepfully I didn't forget any.. PTAL.

PART 1: Thumbdrive
I reworked the thumbdrive guide a little bit, PTAL

PART 2: GitHub
I also created the optional GitHub remote backup guide (replacing the simple pointer to the bonus Dropbox guide that is now deleted in this branch)
However, I encountered some permissions issues I had to work around:

The lnd-backup folder is inside the bitcoin-owned /data folder. Therefore cloning the GitHub repo there needs to be done with the bitcoin SSH key..
But the lnd-backup folder itself has to be owned by the "lnd" user for LND to update/write the channel.backup file...

To solve the issue, I did the following steps:

Create SSH keys for the "bitcoin" user to clone the GitHub "lnd-backup" repo
Change ownership of the "lnd-backup" repo from the "bitcoin" user to the "lnd" user
Delete the bitcoin SSH deployment key in the Github repo
Create SSH keys for the "lnd" user and create a deployment key with write access in the GitHub repo

However, it seems convoluted and I'm pretty sure there is a better more straightforward solution to do this... Wdyt?
This led me to wonder about two things:

Isn't it a security risk to allow the GitHub repo to have SSH access to the node via the lnd user? Couldn't we create a separate user (e.g. "github") that has only access to a unique folder that only contains a copy of the backup file and nothing else?
Why does /data belongs to the "bitcoin" user as it is also used by other services (Electrs, LND etc)

I let you have a look at this first draft of the remote GitHub backup! I'm sure the script can be improved upon and some steps can be simplify, please amend/slash/correct as much as you want :)

First round of review with some questions (e.g., does the separate directory "/data/lnd-backup" make sense?)

Thanks! Looking foward to see this section in the guide soon.
For the last comment (the use of the default location), I'll do this when i'll be next to the node as this is a significant changes to a few parts of the guide so I need to test them live. I agree that using the default location is better as it simplifies the guide. I'll keep you posted once this change is completed.
Otherwise, I implemented all the other requested changes, PTAL!

Went through PART 2 (GitHub backup) of this guide, with the changes we discussed in mind:
* backup is monitored by user "lnd"

* user "lnd" creates SSH keys and clones the repository directly

* the original monitoring script from PART 1 could already include the code for PART 2, but "disabled". Then it just needs to be enabled here.

I now implemented the new solution we discussed earlier:

There is now only one script with 2 variables to enable/disable the local/remote backup method
By default, the script has both enabled (safest option) but users are explained how to disable either one of the 2 methods
The guide is now much shorter/simpler to use
The /data/lnd-backup directory is now removed from the LND install guide and the backup file locatoon option in the lnd.conf has been removed (the default is now used)
I added the use of a microSD card as an alternative to a USB stick as it gives users more options and reduce th eneed for them to buy a new piece of hardware.. (but does it have to be mounted as  a USB stick as well? if not, I'll hve to update that section)

I tested the guide on my new node and it works fine. But  will keep an eye until I get a "natural" change of my SCB to see if the backups are made both locally and remotely as intended.
PTAL! Notably:

is the introduction too long and should we shorten it?
are the bash script "if" statements ok?

Testing the whole guide from scratch: it works great! I have some suggestions about structure, the shell script, and some small fixes. I'm not yet including wording suggestions.

So I went through all your and @RooSoft comments. I think I didn't miss any.. although hard to keep track of (particulalry with GitHub sometimes collapsing comments without any reason!).

Regarding the structure, I'd find it more logical to start with mandatory instructions, e.g., checking that the SCB file is at the expected location, and then installing the script with both backup methods turned off. Then, the user can move on to whatever backup methods they want to enable and drop out of the guide without consequences:
1. Make sure prerequisites hold and install script

2. Optional instructions for USB backup, enabling this part of the script

3. Optional instructions for remote backup, enabling this part of the script.

IMO, the script should have both methods turned off initially, making them "opt-in", and not 'opt-out".
But this structure might just be my personal preference. If others disagree, I'm happy with the guide as is.

I like this solutiona as well, yes. @RooSoft wdyt? I'll try this solution now.

I modified the structure based on @Stadicus idea. The flow is now:

Intro
Preparations: checking SCB location, creating the script with both methods disabled and creating the systemd service
(Optional) Local backup: guidelines to setup the local backup + enabling the method in the script
(Optional) Remote backup: guidelines to setup the remote backup + enabling the method in the script
Test: a final sectio to test the setup (works for local-only, remote-only or both local and remote)

Wdyt?

GitHub sometimes collapsing comments without any reason

That might have been me, I was already resolving some comments when I've seen your update... I won't do that in the future, sorry.
The current structure looks great. I'll give it one more check in the coming days.

GitHub sometimes collapsing comments without any reason

That might have been me, I was already resolving some comments when I've seen your update... I won't do that in the future, sorry.

If I remember well, they were unresolved comments, so I don;t thin it was you. Moreover, I prefer when reviewers resolves comments once they're happy with the chnages, bevause it makes it easier to see what comment is left to be dealt with!

The current structure looks great. I'll give it one more check in the coming days.

Ok, great stuff

So I've rewritten the section that explains that the methods in the intoduction.
There is now a new h2 section entitled "Choose your preferred method(s)". It explains what are the two options with a table that summarizes the main differences or PROS/CONS bewteen the two methods. Followed by a paragraph that explains that one or both method can be choosen and how to use the guide.
Here is a preview (can be seen live here: https://vajraofindra.github.io/RaspiBolt/static-channel-backup.html)

The markdown table does not render as beautifully as I thought it would though...
Wdyt of this next section? does it clarify things a bit better while still not favouring one method over the other?
Happy to amend as much you want!

There is now a new h2 section entitled "Choose your preferred method(s)". It explains what are the two options with a table that summarizes the main differences or PROS/CONS bewteen the two methods. Followed by a paragraph that explains that one or both method can be choosen and how to use the guide.

Love it!!!

Two typos and a potentially confusing preparation section

I fixed the two typos thanks.
I rewritten parts of the hardware requirements to make it less confusing. Let me know what you think of the new wording.

Alright, I went over the whole text and made "some" wording recommendations. My goal is to keep it concise, in active voice, removing filler words. I also use Grammarly to help me with that.
I apologize, it looks bad. But it's only recommendations, how I would write it. But it's your guide, so feel free to take whatever you find useful, and ignore everything else. :)
You can see my suggestions here: Stadicus@7967b8c

Excellent. Very happy to implement all your recommendations. You're the one that has written such a clear and well-written guide so I'm glad to learn how to write more concisely and didactically :) You also removed a lot of the parentheses I was using which is something @RooSoft noted so that's great as I won't have to go through the whole text to reduce their numbers. I've noted I'm addicted to parentheses.. it's a great way to nuance statements.. but not great for readibility I agree!
I managed to merge your branch into mine after resoliving a small conflict. I'm glad I didn't need to do a manual copy/pasting of each paragraph..
I'd be happy with similar wording suggestions in any future PRs from both of you, no need to feel bad about it ;)

A few final nits, and then LETS GO TO MASTER!!! rocket
Edit: forgot to submit the comment about the absolute/relative path. I don't have a strong opinion either way. I see @RooSoft's point, le't just leave it as is.

Great, I implemented your final suggestions and opted for using the relative path for the SCB comment.
PTAL


The verification process for logging into SAP S4/HANA is stuck

Steps to reproduce
As shown in the image, after successfully entering the username and password, the verification page has become stuck. It neither closes nor redirects, causing the verification process to fail.
This phenomenon has been encountered by testers across different regions.

Expected behavior
The verification interface should close normally and successfully log in.
Actual behavior
the verification page has become stuck.
Error details
No response

csp issue

Hello @vikkixu01 - Thank you for bringing this issue to our attention. Could you please share the steps you took to encounter the issue.

Hello @vikkixu01 - Thank you for bringing this issue to our attention. Could you please share the steps you took to encounter the issue.

Steps to Reproduce

log in with Frank account（frankw@t25ws.onmicrosoft.com/M365s4hana@uxengg）
add SAP S/4HANA - cons with Bot app
open in Copilot
enter prompt: Fetch the details of Sales Contract where Overall Status is Not Relevant
click sign in to SAP S/4HANA button
sign in with Frank account (click continue and skip button during login process)


Initial Step 3/3: invalid_client

Hi, Greetings and thank you for your valuable efforts.
I kindly request your assistance in guiding me towards a resolution for the current predicament.
What happened

Followed Getting Started section
I encountered an issue while attempting to obtain OAuth authentication during Step 3/3: Retrieving access and refresh tokens. An error message was displayed stating, Whoops, it appears that we have encountered a problem: invalid_client.

Logs
AADSTS7000218: The request body must contain the following parameter: 'client_assertion' or 'client_secret'.
 Trace ID: 2cbdbd67-1bb8-4da3-8456-b8d2a47e4100
 Correlation ID: 7faed391-0c0a-4104-add8-ffae24069f1e
 Timestamp: 2023-09-29 07:54:00Z

Microsoft's official explanation

Attempts

https://github.com/spencerwooo/onedrive-vercel-index/issues/683
https://github.com/spencerwooo/onedrive-vercel-index/issues/684
Regrettably, despite my thorough investigation, I have been unable to locate any pertinent information of significance.

ScreenShot

Hi, Greetings and thank you for your valuable efforts. I kindly request your assistance in guiding me towards a resolution for the current predicament.
What happened

Followed Getting Started section
I encountered an issue while attempting to obtain OAuth authentication during Step 3/3: Retrieving access and refresh tokens. An error message was displayed stating, Whoops, it appears that we have encountered a problem: invalid_client.

Logs
AADSTS7000218: The request body must contain the following parameter: 'client_assertion' or 'client_secret'.
 Trace ID: 2cbdbd67-1bb8-4da3-8456-b8d2a47e4100
 Correlation ID: 7faed391-0c0a-4104-add8-ffae24069f1e
 Timestamp: 2023-09-29 07:54:00Z

Microsoft's official explanation

Attempts

(Stuck Step 3/3) Whoops, looks like we got a problem: invalid_client. spencerwooo/onedrive-vercel-index#683
(Stuck Step 3/3) Whoops, looks like we got a problem: invalid_client. spencerwooo/onedrive-vercel-index#684
Regrettably, despite my thorough investigation, I have been unable to locate any pertinent information of significance.

ScreenShot

are you using your own client id and secret key??

Yes, it's a completely new client that I have created within the Azure Control Panel.

With the correct Client ID, Client Secret, Permissions, and redirect URL.

no problem


Read ServerName from Client, not server Config

Resolves #319

Will add test tomorrow, just wanted to get this easy one done :)

@daenney @at-wat ready for review!


Error on deleting file

https://github.com/user-attachments/assets/3dadba3a-c3b3-4722-a0d0-32e0707eaab6

Closing as duplicate of #3088