Update tls-support.md

Fixing " lists all possible values for the configs belong" in TLS section

Fixes #XXXX.

Description

Fixed the bug ...
Renamed the class ...
Added a forbidden-apis entry ...

Key changed/added classes in this PR

MyFoo
OurBar
TheirBaz

This PR has:

[x] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

Thank you @iandr413 for your first contribution.


confusing confusion matrix plot

wandb --version && python --version && uname

Weights and Biases version: 0.9.1
Python version: 3.6.4
Operating System: Window 10

Description

The x/y axis is unconventional and confusing, x-axis should be predicted value, and y-axis should be actual

Code Snippets

Expected
x: Predict
y: Actual
The common confusion  matrix should be:

TN
FP

FN
TP

The UI is showing this instead:

TN
FN

FP
TP

Gist Example
https://gist.github.com/45c6a6cf3f50ace5b5f94d027e619237
cc @hinofafa

Thank you very much for pointing it out. We filed a ticket for it.

Hey there, thanks for your message. I don't think we'll be able to switch this for a little while but we're building a better solution for custom graphs which will allow you to customize this behavior.


Test:add a SPACE line for testing CLA assign

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


speed widget reports bytes not bits

the widgets are using lowercase b for bits but in reality the widget is reporting bytes

closed in 1.3.1


best design

an utf-8 icon is added,
a display block is added,
id #searchform for easy editing with css,
placeholder is added with translation option

I would prefer to keep it minimal, the classes are I'm assuming from Bootstrap, which not everyone uses when building themes with Sage. Is there a reason for inline CSS?

yes, the classes are bootstrap and those that come by default in styles components

My preference would also be to keep it minimal, and particularly I'm not fond of the inline styles there. Also, not an accessibility expert, but using only the icon as the button value seems like a step backwards.
On the other hand, using <button> instead of <input> for the submit element is arguably a bit better.

This PR introduces several significant changes to the current design with no clear benefit or explanation, so I'm closing it. Some of these changes (<input> to <button>) may deserve additional discussion, but a separate, more focused PR for those changes and discussion of them would be more ideal.


Android Marshmallow Patch

This commit adds a patch for Android Marshmallow including:

Signature Spoofing
Location Provider Support

Before building a patched Android Marshmallow system, it is required to execute make update-api in the root directory.

It would be great if you'd use https://gerrit.omnirom.org/#/c/16080/ as a base patch for signature faking instead of the LP version as it properly handles dynamic permissions on Android M (when using the LP patch, denying the permission will not disable signature spoofing).

The pull request has been updated.


Add "Newgrounds" to Yes Porn Allow list

What is the site you want to add?
Newgrounds: Newgrounds.com
What kind of service are they?
Art Gallery / Game Hosting (probably fits under Game Hosting best, for this list)
How does the site tend to treat NSFW content?
Allowed, if marked
Do you know of any risk to the site?
None
Any comment you want to add about the site?
Multimedia Gallery Site, Allows static art, movie format, and games. One of the oldest continuously-operating Gallery Sites, fully independent.

Added to the list


Missing llvm-80c3ea-windows-vs2019-x64-RelWithDebInfo.7z

Brief Description
Tried following the steps in readme, in the first step "Setting up LLVM and Clang dependencies"
I did:
C:\Workspace\CppSharp-master\build>DownloadDeps.bat

Then I see this error:
C:\Workspace\CppSharp-master\build>"premake5" --file=scripts/LLVM.lua download_llvm
Downloading: https://github.com/mono/CppSharp/releases/download/CppSharp/llvm-80c3ea-windows-vs2019-x64-RelWithDebInfo.7z
Error: .../CppSharp-master/build/scripts/Utils.lua:98: HTTP response code said error
The requested URL returned error: 404 Not Found

C:\Workspace\CppSharp-master\build>goto quit

OS: Windows 10 64bit
Visual Studio Community 2019

Related to https://github.com/mono/CppSharp/issues/1290.

@Tennyleaz You can get the VS2017 package and rename it for now from: https://github.com/mono/CppSharp/releases/tag/CppSharp


migrate-fresh command don't consider mysql-schema.dump  file

Bug description
When I run my migrations the schema files are not considered
Steps to reproduce
sail artisan tenants:migrate-fresh
Expected behavior
consider the file mysql-schema.dump
Laravel version
^9.19
stancl/tenancy version
3.6

Dumping tenant schemas is not a v3 feature
We use a custom implementation for migrate:fresh. Why exactly is not using the dump file a problem?

Hey stancl,
Thanks for answer, it's beacuse I have my migration removed and I only have this mysql-schema.dump file, so, whenever I create a new domain I need to run the dump file manually instead of using artisan migrate as usual.
Regards

I see, that's fair. We'll take a look at this.
@lukinovec as part of some of the tasks we're doing now, we should verify that all migration commands work even when schema is used instead of migration classes (since the migrations can be deleted after the schema dump is created). I think migrate can have parameters specified in the config, but migrate:fresh probably doesn't respect this.
I'm not sure if we want to use the schema automatically in migrate, probably not (so users who generate the schema should then configure it in the config), but migrate:fresh should take all of the parameters from the config for the migrate call it does.

Thanks Samuel,
In my humble opinion I think it should be as close as possible to how php artisan migrate works.

@stancl, just to confirm, we're talking about tenants:migrate-fresh and tenants:migrate, not the Laravel commands (migrate:fresh) here, right?

migrate:fresh should take all of the parameters from the config for the migrate call it does

Right, I see that tenants:migrate and tenants:rollback respect parameters from the tenancy.migration_parameters config, but migrate:fresh doesn't. tenants:migrate-fresh calls  tenants:migrate with --tenants=[$tenantKey] and --force=true, nothing else is passed to the command.

Yeah, we probably just want to merge in the params from config

tenants:migrate-fresh calls tenants:migrate with --tenants=[$tenantKey] and --force=true, nothing else is passed to the command.

This is not relevant in the end. tenants:migrate gets the options from the config even if they weren't passed to the command. The paths from the config are passed to MigrateCommand (doesn't matter if we call tenants:migrate or tenants:migrate-fresh). I'll try reproducing the issue.

So this really doesn't look like a tenants:migrate-fresh issue @stancl. The dump is also not considered with tenants:migrate.
I had some migrations in migrations/tenant and a tenant with a database that had some data. I dumped the migrations using the v4 dump command, and I removed the tenant migrations. Then, I used tenants:migrate-fresh to clear the tenant database and see if the database's tables wiped and then created. They didn't get created. I called tenants:migrate, and that also didn't create the tables.

(no output about the dump being used)
My migration parameters config:
'migration_parameters' => [
    '--force' => true, // This needs to be true to run migrations in production.
    '--path' => [database_path('migrations/tenant')],
    '--realpath' => true,
],

Also, the dump file is database/schema/mysql-schema.dump, and that's the same name as the name of the dump that'd be used for the central app (as @stancl wrote here). migrate:fresh considers that dump.

Closing this since the logic works as expected — tenants:migrate-fresh uses the parameters specified in the config since it calls tenants:migrate internally.
Regarding the dump file, we'll be changing how that works in v4 to use a separate file from the central one.

Hey, maybe I'm wrong but IMO its not working properly, as you said tenants:migrate-fresh calls tenants:migrate and it calls migrate and in this case should call migrate:fresh (which is a different command).

It calls db:wipe and then tenants:migrate. We can't call migrate:fresh directly from tenants:migrate.

Either way — schema dumps are not something that's supported in v3.

I faced this same issue, and I'm looking forward to v4.
Untill then, this issue is easily resolved by running the schema dump in a migration that runs first:
/database/migrations/tenant/0000_00_00_000000_execute_schema_dump.php
<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Support\Facades\DB;

return new class extends Migration {
    public function up()
    {
        DB::statement(file_get_contents(__DIR__.'/../../schema/mysql-schema.dump'));
    }
};

I've solved this for now using a migration that will run the schema dump.

Move your schema folder to the tenants folder (otherwise it will run the schema in the central database)
Create a migration that will run first e.g. /database/migrations/tenants/0000_00_00_000000_execute_schema_dump.php

<?php

use Illuminate\Database\Migrations\Migration;

return new class extends Migration {
    public function up()
    {
        Artisan::call('migrate', [
             '--schema-path' => __DIR__.'/schema/mysql-schema.dump',
             '--path' => '/dev/null'
        ]);
    }
};


Scrolling up causes crashing and bad performance

For some reason, scrolling up on large lists > 500 items causes flickering and eventually crashing. Scrolling downward works flawlessly.
 <virtual-scroller     #scroll     [items]="docs"> <div #container>       <div class="infinite-item" *ngFor="let d of scroll.viewPortItems">           {{d}}       </div>     </div>   </virtual-scroller>

Are you able to reproduce this on any of the examples here? https://rintoj.github.io/ngx-virtual-scroller  (click on the samples link).
Those samples only have 1000 items, but I have a personal project with lists of 30,000. I believe you're doing something wrong, because this is working flawlessly for me.
The code you pasted looks fine, however it isn't enough to diagnose your problem. Could you create a stackblitz? The stackblitz should be as close to your real project as possible & needs to actually reproduce the problem.
Without looking at your code, it's really hard to guess what you're doing wrong. However, my first suggestion would be to convert to using OnPush Detection strategy. You can read my explanation in th readme here: https://github.com/rintoj/ngx-virtual-scroller#angular-onpush-detection-strategy---general-performance-advice-not-specific-to-ngx-virtual-scroller

I completely understand. Hard to troubleshoot code you can't see. There's no doubt I'm doing something wrong. I can investigate reproducing.
I was passing in the parent scroller and when I took this out it would scroll up and down okay HOWEVER... from debugging, I notice that now ALL of the items are being rendered to the DOM. This also causes super slow scrolling and eventually breaks my app.
Thoughts on these possible things i'm doing wrong / ideas?:

1000 items come into the array i'm passing into [items] at one time and the array is of objects with that 7-10 different properties/keys.
I set my virtual-scroller css to height: 100vh

In the meantime, I'll also look at that OnPush strategy, I considered this too before posting.
I have the same scroller in 8 other places in my app and it's working beautifully (so thank you!). This is much better than the ion-virtual-scroll and the angular CDK.

All of the items shouldn't be rendered to the DOM, only the visible ones. That's definitely part of the problem.
Giving virtual-scroller css height: 100vh should be fine as long as it's the actual scroll element. If you're doing a parent scroll (for example: using window scrollbar), then virtual-scroller height shouldn't be constrained & instead the parent (html/body) should have a fixed height (typically 100% for html/body).
An array of objects for the [items[ array should be fine. Virtual-Scroller accepts any array, it doesn't care what data is inside of it.
I think if you read through the instructions carefully or look at the examples, you'll be able to figure out what you're doing wrong.
Closing for now. If you're unable to figure it out, please re-open with a stackblitz & i'll take a look.

I figured it out....
I had a switch in the code.
<ng-container [ngSwitch]="libraryType">
<ng-container *ngSwitchCase="librarySvc.LibrayItemTypes.EXERCISE">
....
I wrapped my component with a  and voila...
This is the best infinite scroll on the market for angular BY FAR.


Feature request: Introducing more types

Please introduce some more supported types such as date, timestamp, etc. It would be helpful as parsing date and time for every query is costly.

Short hint: you can use skiplist indices in conjunction with timestamp strings in the format YYYY/MM/DDZhh:mm:ss - you can even do range requests supported by the index.

I'll surely give it a try. Thanks.

Have a look here regarding dates and indexes:
https://docs.arangodb.com/3.0.0b1/AQL/Functions/Date.html#working-with-dates-and-indices

I guess i was already on the right track. Thanks.


Unable to connect to server when bindings 1.1 and 1.0 are present

Minemeld is unable to connect to a taxii server that is distributing both message binding 1.0 and 1.1. The error returned is " - collection  does not support TAXII 1.1 message binding (DATA_FEED)"

Hi @nmgrange,
is the TAXII Server providing both STIX 1.0 and 1.1 or just STIX 1.1 ?
I am starting to work on a major reimplementation of the TAXII Miner and this could be an additional requirements. See  #205

The server is providing both 1.0 and 1.1 and the inbox will accept any content. Here is a screenshot of the inbox. When I changed it to only accept 1.1 the binding was still the same and I cannot change that.

And I added some logging to see what was being pulled and here is the binding that it sees

Do you have full control on the TAXII Server ? The TAXII Miner connects to the Poll Service, is it also offering both 1.1 and 1.0 TAXII Binding ?

Closed for no activity


Two questions

First congratulations for the work, it has really helped me a lot to optimize my flow of notes. I have two questions:
It is possible to have the page automatically translated into the language of your choice (Portuguese-Brazil) on load.
Second, there is a way to stay logged in to a specific page, like UpToDate, without having to repeatedly enter your login credentials.
Thank you in advance for your attention and effort in producing your plugin ✌

It is possible to have the page automatically translated into the language of your choice (Portuguese-Brazil) on load.

Hi! This should automatically happen based on the setting you have Obsidian set to, since it changes the setting that the overall Electron instance is set to. If that is not the case, please open a new issue with a feature request. Thanks!

Second, there is a way to stay logged in to a specific page, like UpToDate, without having to repeatedly enter your login credentials.

Cookies and local storage are stored normally through Obsidian as they would be otherwise. This means that all login data that any given site stores will be restored next time you enter the site. It might be that the site you're trying don't support what is usually known as a "remember me" feature, though.
Cookies and local storage might also work more inconsistently in iframe mode, so I recommend always using webview mode for your frames if possible.


MissingRegion error when running any Pulumi command

Hi, when I run any pulumi command I'm getting the following error:
error: failed to load checkpoint: blob (key ".pulumi/stacks/prod.json") (code=Unknown): MissingRegion: could not find region configuration

More information:
$ cat Pulumi.prod.yaml
...
config:
  aws:region: eu-west-1
...

$ pulumi plugin ls | grep aws
aws         resource  3.7.0    248 MB  n/a        23 minutes ago

$ pulumi version
v2.11.2

I'm using S3 to store the state files and KMS to store the secret key.
Things do work if I set up the environment variable AWS_DEFAULT_REGION:
$ export AWS_DEFAULT_REGION=eu-west-1

$ pulumi config get aws:region
eu-west-1

I would expect that Pulumi's stack configuration would be used to get the state initially. Is this not the case?
Thank you

Hi @Sodki
So I am not quite following here - you have set the region as pulumi config set or are you starting with an empty pulumi config file?
Paul

I have the region set in Pulumi config and I have been using Pulumi for a couple of months now.
I've noticed that I had AWS_DEFAULT_REGION set in my shell profile and I got this error when I removed it. So I'm not sure if this is a bug in Pulumi, a limitation or a design decision. For my use case I'd prefer that my developers don't have to add extra configuration and can rely solely on Pulumi.

Looking at the error error: failed to load checkpoint: blob (key ".pulumi/stacks/prod.json") (code=Unknown): MissingRegion: could not find region configuration this looks like you're using the S3 backend? In which case, the backend retrieval path never uses what's in Pulumi.<stack>.yaml. Per https://www.pulumi.com/docs/intro/concepts/state/#self-managed-backends, the backend configuration is governed by what's described in https://docs.aws.amazon.com/sdk-for-go/api/aws/session/

Thank you, it makes sense now, and yes I'm using the S3 backend.
Someone in Slack helped me to fix my problem by adding the region to the backend URL, like so:
backend:
  url: s3://bucket/path?region=us-east-1

In this case there is no need to try and find out which region to use, so the SDK doesn't complain anymore. Would it make sense to add this to the documentation somehow? If you think it doesn't then feel free to close this issue.
Thank you

Hi @Sodki
this is currently documented - not very well - in the stacks doc - https://www.pulumi.com/docs/intro/concepts/project/

We are going to spend some time with regards to backends soon - I will ensure the docs are updated in this set of work to make it way more clear


Origins with multiple abilities activate all of them on using Primary Ability keybind

(Created new issue for visibility reasons.)
While ya'll definitely aren't obligated to, I would love to see a backport for 1.18.2 with the issue of all abilities activating on primary ability fixed since there isn't a build for 1.18.2 without this bug. Given the size of it and how popular 1.18.2 is as a version for modding, it would be greatly appreciated.
And, to clarify, I'm not asking for ongoing support for 1.18.2. I don't expect anyone to support an out of date version, of course. Rather, this bug is just massively debilitating to the point that I would love to see at least a single version released for 1.18.2 to fix it. There are a LOT of mods that haven't updated past 1.18.2, which means that a lot of people like my friend group don't have the option to simply update to 1.19+, on top of the many questionable decisions Mojang is making in 1.19.1+ onwards(such as the report system) making it generally not a great idea for even vanilla users to update. Basically, lots of users either can't because of mod issues or won't because of Mojang doing dumb things, and therefore I feel like just one update to fix this issue would be extremely well received.
Originally posted by @BlackjackMKV in https://github.com/EdwinMindcraft/origins-architectury/issues/235#issuecomment-1680419838

If that's not something you're willing to do, I entirely understand and you can go ahead and just close the ticket. It would be immensely helpful though!

I'll probably get around to a major 1.18.2 bugfix at some point in the future as the final version, I'm just wanting to get 1.20.1 out first.

This is great news! I have been trying to play 1.18.2 lately, and I just can get any of my origins to work at all. I phase when I try to float, blow up when I try to phase, and so on. As previously mentioned though, 1.20.1 is more pertinent at the moment.

Just curious, how far along is this currently?

Just curious, how far along is this currently?

Haven't started. 1.18.2 also needs a ton of rewriting, so it's difficult to do much there.

I'm so relieved to hear that it's in the works!
I just encountered this bug while building my 1.18.2 pack, and I'm unfortunately stuck on 1.18.2 for numerous reasons.
Thank you for your work! :)

Has this been fixed yet by any chance? It's fine if not, I just thought I'd check in on this since, checking the changelogs in the other versions released since I posted this, I don't see a fix for it yet.

Has this been fixed yet by any chance? It's fine if not, I just thought I'd check in on this since, checking the changelogs in the other versions released since I posted this, I don't see a fix for it yet.

This is a 1.18.2 change, I've only been working on 1.20

Has this been fixed yet by any chance? It's fine if not, I just thought I'd check in on this since, checking the changelogs in the other versions released since I posted this, I don't see a fix for it yet.

This is a 1.18.2 change, I've only been working on 1.20 atm. Keep in mind that I'm doing this as a hobby, and 1.18.2 has many other issues that I'd have to look into before I get to releasing a version, it's honestly kinda overwhelming to have to put all this work into a version of the game that I don't super care about (no offense).

No worries! As I mentioned earlier, I'm just glad it's even a consideration. If it wasn't for this lovely 1.1984 update that has been banning people for cussing in their own private LAN worlds even, I'd probably be on the latest version anyway. Between that and the fact they've not added anything I particularly care about with WAY more mods on 1.18.2 than on 1.19+, I just haven't wanted to or cared to update. Since it's very much my choice to stay on an out of date version though, I don't expect any continuing support, so I'm just happy you're even considering doing it at all. :)


Removed Mandrill driver

Why did you remove Mandrill driver? (It was in 2.0.2 version)
Package version
3.0.9

Removed, since Mandrill wasn't free anymore and my personal finances doesn't allowed me to pay for many paid services.
Although it will be added back in Next release


TravisCI Source Control Management connection Use Case

Use Case: TravisCI and Source Control Management (SCM) connection
Actor: Development Team
Basic Flow:

The TravisCI agent needs to have access to and open communication with the SCM tool.
When a commit happens to SCM tool from any developer, ensure TravisCI builds the code.

Added .travis.yml to main folder of github so that the travisCI project connects.


Bug: space after templateUrl and : causes No ResourceLoader error

Example :
templateUrl : './sometemplate.component.html' <- not good
templateUrl: './sometemplate.component.html' <- good

I'll take a look at the project this week to see what the regex is doing to cause it.

hi,
You can close this topic the problem was solved i have update the angular2-template-loader package version to ^0.6.2 because the error come by using the version 0.4.0 on the universal-starter you should update the universal-starter package file only
tkx


shouldCancelStart?

I'm trying to migrate a project from react-sortable-hoc. What's the equivalent of shouldCancelStart? How can I prevent the drag to start on runtime? Thanks! 🙏

I resolved with a custom Sensor:
class CustomPointerSensor extends PointerSensor {
  static activators = [
    {
      eventName: 'onPointerDown' as const,
      handler: ({ nativeEvent: event }: PointerEvent): boolean => {
        if (document.querySelector(...)) {
          return false;
        }
        return true;
      }
    }
  ];
}


Unroll node that is an array

How do I unroll/loop through a node that is an array? I know you can unroll when you loop through the descendants or children of a node, but what if I want to loop through the array of the node itself?

The Node interface doesn't have a built-in method to do that currently.
I'll add that as a feature request since I can see how that would be useful.
In the meantime, you can use this function to unroll any node that is an array. This should be pretty future-proof too:
from typing import Iterator
import itertools
from systemrdl.node import Node, AddressableNode

def unrolled(node: Node) -> Iterator[Node]:
    cls = type(node)
    if isinstance(node, AddressableNode) and node.is_array:
        # Is an array. Yeild a Node object for each instance
        range_list = [range(n) for n in node.array_dimensions]
        for idxs in itertools.product(*range_list):
            N = cls(node.inst, node.env, node.parent)
            N.current_idx = idxs
            yield N
    else:
        # not an array. Nothing to unroll
        yield cls(node.inst, node.env, node.parent)

It would also be useful to be able to unroll a single dimension.

Implemented in v1.20.0


sometimes p_energy is NaN

Original report by Galen O'Neil (Bitbucket: oneilg, GitHub: oneilg).

When p_energy contains NaN values, then histogram doesn't work, neither does ds.p_energy[:]<elo. It either shouldn't ever be NaN, or we should have some workarounds. I've been doing ds.p_energy[np.isnan(ds.p_energy)]=0 just to avoid problems.

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

EnergyCalibration object cannot return NaN or inf.
Fix the ph2energy method to test for both outcomes and replace either
with 0. Also make a test to be sure that neither 0 nor negative PHs
will produce NaN or inf energies. Fixes #124.
However, I should note that I never actually reproduced this bug.

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

Whose fault is this? I'm guessing it's the fault of the calibration object. Now that Young has introduced analytic derivatives into the object, I cannot understand it at all.
Nevertheless, I will see if I can somehow introduce the 1-liner you just gave me into EnergyCalibration.__call__ or wherever it's appropriate.

Original comment by Galen O'Neil (Bitbucket: oneilg, GitHub: oneilg).

Current workaround:
def ds_sanitize_energies(ds, elo=0,emax=2e4):
    ds.p_energy[np.isnan(ds.p_energy[:])]=-1
    bad = np.logical_and(ds.p_energy[:]<elo, ds.p_energy[:]>emax)
    ds.cuts.cut("energy",bad)
def data_sanitize_energies(data, elo=0, emax=2e4):
    for ds in data:
        ds_sanitize_energies(ds, elo, emax)

I looked at this a bit more. In one dataset all the traces with nan energy have negative p_filt_value, and I plotted a few of the traces here:
<EMAIL_ADDRESS>


PDF Parser, GoogleDrive support for PDF, README.md minor fix

@bary12 do we want here pdf->html->text? to know titles, bold, etc, like docx?

@bary12 do we want here pdf->html->text? to know titles, bold, etc, like docx?

Yes, just for the titles.

@d4yz so we need pdf_to_html, and then use html_to_text, like we do for .docx


Add RGB-Depth alignment

This PR adds RGB-Depth alignment to the demo and enables LR check by default. To enable this feature, use the --rgbDepthAlign flag or click the RGB-Depth Alignment switch in Camera properties

@VanDavv updated with latest imrpovements. If all good from your POV can be merged.
The GUI doesn't work for me, even on main. Tried on 2 different systems, after install requirements.

Thanks @szabi-luxonis! Will take a look at it and fix, didn't check this branch lately so probably need to add some updates


Updated AzureStaticWebAppV0 to add compatibility for api token as task input

Task name: AzureStaticWebAppV0
Description: Added compatibility for api token as task input.
Documentation changes required: N
Added unit tests: N
Attached related issue: Y
Checklist:
[✓] Task version was bumped - please check instruction how to do it
[✓] Checked that applied changes work as expected

/azp run

/azp run

/azp run

/azp run

I just ran into this blocking issue. Any ETA on when this change will be available? Thanks for this btw 😊

@shigupt202 Are you able to provide an ETA on the release. Is there a place we can see release and deployment schedules for these tasks?

@anthonychu The 187 deployment has already been started. It's done on Ring 0, and by mid next week it'll be done on other rings as well. There was a deployment blocker bug in another task which caused some delay in the deployment. (There's no publicly available dashboards to monitor the deployment schedules)

Any update on this release?

This should have been deployed in most of the rings by now. Can you please share your org name, so that I can check if it's present on your org?

Thanks! It's littlebytegames.

It should have been deployed now. Can you please check and confirm?

I see it thank you 😁


fix: use limit/offset instead of a cursor

:tada: This PR is included in version 1.2.6-beta.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.2.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


225x90x90

Hello,
I noticed you mentioned that the latest code could support training with a latent size of 225x90x90, which seems quite large. However, I couldn't find the corresponding training script in the current scripts folder, where all the input sizes of the images are set to 256 and num_frames set to 16. Could you please share the relevant code? Thank you for your assistance.

You can calculate the corresponding input size based on https://github.com/PKU-YuanGroup/Open-Sora-Plan/issues/128.


dispossessor group

dispossessor.com/

Hi,
Thanks but it looks like a recycled of previous victims from other ransomware group. I've notice LB3.0 and Hunters.
I will monitor the site but won't publish for now any victims.

parsed now


Cancel in-progress jobs in another comes in

We don't need the old ones.

Looks like it's failing over in main.


named export ComponentLoader not found

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
I have been trying to customize my Adminjs component from Node js and when I import ComponentLoader  there is error
named export ComponentLoader not found
Later on I tried to import pkg from 'adminjs'
and then  'const {ComponentLoader} = pkg'
But then it says:
Trying to bundle file 'file:/Users/Josip/WebstormProjects/ferry-backend/components/dashboard.jsx' but it doesn't exist

`
import {ComponentLoader} from "adminjs";
const componentLoader = new ComponentLoader()
const Components = {
MyDashboard: componentLoader.override('Dashboard','../components/dashboard')
}
AdminJS.registerAdapter(AdminJSSequelize)
const admin = new AdminJS({
databases: [],
rootPath: '/admin',
resources:[UsersResources, GuestResources, SalesResources, FinancesResources],
components:{
edit: Components.MyDashboard
},
componentLoader
})
`

To override the Dashboard I suggest just using .add instead of .override and assigning it in AdminJSOptions:
dashboard: {
  component: Components.MyDashboard,
}

@jmaric or anyone else experiencing this issue.
Unsure if it's exactly the same issue, but I ran into this error earlier today.
It should work if you provide the absolute file path rather than just the current directory like ./filename.js. The reason this was failing for me is because when it obtains the file path it appends file: to the beginning of the string. For example, it would be file:/Users/path/to/file.js
fs.existsSync() is then used to check if the file exists, which doesn't seem to work when the string starts with file:. So by doing the following you shouldn't have any problems as it won't try and resolve the absolute file path, therefore file: will not appended:
MyInput: componentLoader.add('MyInput', '/absolute/path/to/file.jsx'),

To avoid hard coding it, you could do something like this:
import path from 'path';
import { fileURLToPath } from 'url';

const filename = fileURLToPath(import.meta.url);
const dirname = path.dirname(filename);

const Components = {
    MyInput: componentLoader.add('MyInput', `${dirname}/file.jsx`),
};

export { componentLoader, Components };

That being said, it still might be a better idea to go with what @dziraf suggested for your particular use case.
@dziraf Any further thoughts about this? This error occurred both running on my Macbook Pro as well as a Docker environment, which uses the official Node image.


One node X509 cluster creation fails

Service Fabric Runtime Version: 7.2.434.9590
Environment: Standalone
Description: Certificate wad installed on the server according to the procedure describede here: https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-windows-cluster-x509-security and ASF cluster creation attempted using the attached script. The creation failed.
Observed behavior: failure
Expected Behavior: success
files.zip
OS(Windows/Linux): Windows
If this is a regression, which version did it regress from? n/a

Assignees: /cc @microsoft/service-fabric-triage

Same root cause as https://github.com/microsoft/service-fabric/issues/1082


Compiling error

I don't know if it's only my problem, but compiling this project through the AUR package https://aur.archlinux.org/packages/keeweb/ I got this error:
⠧ Building module: keyboard-auto-type, Completed: 1../keyboard-auto-type/keyboard-auto-type/src/linux/x11-keysym-map.cpp: In function ‘uint32_t keyboard_auto_type::char_to_keysym(char32_t)’:
../keyboard-auto-type/keyboard-auto-type/src/linux/x11-keysym-map.cpp:198:40: error: ‘numeric_limits’ is not a member of ‘std’
  198 |             return *found_in_32 & std::numeric_limits<uint32_t>::max();
      |                                        ^~~~~~~~~~~~~~
../keyboard-auto-type/keyboard-auto-type/src/linux/x11-keysym-map.cpp:198:63: error: expected primary-expression before ‘>’ token
  198 |             return *found_in_32 & std::numeric_limits<uint32_t>::max();
      |                                                               ^
../keyboard-auto-type/keyboard-auto-type/src/linux/x11-keysym-map.cpp:198:66: error: ‘::max’ has not been declared; did you mean ‘std::max’?
  198 |             return *found_in_32 & std::numeric_limits<uint32_t>::max();
      |                                                                  ^~~
      |                                                                  std::max
In file included from /usr/include/c++/11.1.0/algorithm:62,
                 from ../keyboard-auto-type/keyboard-auto-type/src/linux/x11-keysym-map.cpp:5:
/usr/include/c++/11.1.0/bits/stl_algo.h:3467:5: note: ‘std::max’ declared here
 3467 |     max(initializer_list<_Tp> __l, _Compare __comp)
      |     ^~~
../keyboard-auto-type/keyboard-auto-type/src/linux/x11-keysym-map.cpp:206:34: error: ‘numeric_limits’ is not a member of ‘std’
  206 |             return *found & std::numeric_limits<uint16_t>::max();
      |                                  ^~~~~~~~~~~~~~
../keyboard-auto-type/keyboard-auto-type/src/linux/x11-keysym-map.cpp:206:57: error: expected primary-expression before ‘>’ token
  206 |             return *found & std::numeric_limits<uint16_t>::max();
      |                                                         ^
../keyboard-auto-type/keyboard-auto-type/src/linux/x11-keysym-map.cpp:206:60: error: ‘::max’ has not been declared; did you mean ‘std::max’?
  206 |             return *found & std::numeric_limits<uint16_t>::max();
      |                                                            ^~~
      |                                                            std::max
In file included from /usr/include/c++/11.1.0/algorithm:62,
                 from ../keyboard-auto-type/keyboard-auto-type/src/linux/x11-keysym-map.cpp:5:
/usr/include/c++/11.1.0/bits/stl_algo.h:3467:5: note: ‘std::max’ declared here
 3467 |     max(initializer_list<_Tp> __l, _Compare __comp)
      |     ^~~
make: *** [keyboard-auto-type.target.mk:143: Release/obj.target/keyboard-auto-type/keyboard-auto-type/keyboard-auto-type/src/linux/x11-keysym-map.o] Error 1

This go away simply adding #include <limits> in the src/linux/x11-keysym-map.cpp
Thanks!

Thanks, added the missing include


Test helpers

Add Blockly.Test.simulateDrag and use it to verify that dragging a should-copy-on-drag block will duplicate it.

Codecov Report

Merging #178 into master will increase coverage by 2.27%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #178      +/-   ##
==========================================
+ Coverage   56.85%   59.13%   +2.27%
==========================================
  Files          96       96
  Lines       11545    11545
==========================================
+ Hits         6564     6827     +263
+ Misses       4981     4718     -263

Impacted Files
Coverage Δ

core/ui/block_space/flyout.js
63.02% <0%> (+1.4%)
:arrow_up:

core/ui/block_space/block_space_editor.js
72.02% <0%> (+1.55%)
:arrow_up:

core/utils/utils.js
82.05% <0%> (+2.56%)
:arrow_up:

core/ui/block_svg/block_svg.js
84.92% <0%> (+2.92%)
:arrow_up:

...ore/ui/block_space/scroll_on_block_drag_handler.js
30.76% <0%> (+5.12%)
:arrow_up:

core/ui/block_space/block_space.js
64.64% <0%> (+5.85%)
:arrow_up:

core/ui/block.js
74.67% <0%> (+11.46%)
:arrow_up:

core/connections/connection.js
69.38% <0%> (+12.55%)
:arrow_up:

core/ui/fields/field_colour.js
35.93% <0%> (+14.06%)
:arrow_up:

blocks/colour.js
38.46% <0%> (+15.38%)
: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 aae90df...fee171f. Read the comment docs.


Mismatch between createFile(file: File) and createReadStream in docs

Describe the bug
The typings were updated such that the signature is createFile(file: File), but the docs example shows a ReadStream being provided.
File is not available in Node. What is meant to be done here? Is this a typo, should be File | ReadStream?
To Reproduce
Try to pass a ReadStream to createFile(), see type error.
Code snippets
No response
OS
N/A
Node version
latest
Library version
latest

Yup, function signature is mistyped, I am working around with createFile(readStream as unknown as File).

Thanks for reporting – we'll push a fix for this shortly!

Appears to be caused by an upstream change in the tool we use to auto-generate this API client: https://github.com/OpenAPITools/openapi-generator/pull/12831
Still looking for a fix that we can publish in our package. As a short-term workaround, developers can cast to any when calling the createFile method:
const response = await openai.createFile(
    fs.createReadStream("test.jsonl") as any,
    "fine-tune"
);

Seems like the same issue for createImageVariation as well, the doc says it expects a string but the API is requiring a File.

Also createTranscription expects a "File" while html 5 File API is not available in Node env.

  const response = await openai.createFile(fileData as any, 'fine-tune');

tried used this, request is sent. but return 400 bad request and nothing more information. I have paid account. I can use most of other API. I am not sure what is happening?

I have similar problem. I tried to implement openai.createFile in firebase function but receiving ENOENT.
Here is my node.js code:
const firebaseApp = initializeApp(firebaseConfig);
const storage = getStorage(firebaseApp);
const httpsReference = ref(storage, filePath);
getDownloadURL(ref(httpsReference))
.then((url) => { // I checked and URL is generated fine
const xhr = new XMLHttpRequest();
xhr.responseType = 'blob';
xhr.onload = (event: any) => {
openai.createFile(
fs.createReadStream(url) as any,
"fine-tune"
).then(() => {
response.send("success")
})
};
xhr.open('GET', url);
xhr.send();
})
});
Help, please.

Hey @zmiju, we have an upcoming version v4.0.0 that overhauls File support and makes this easier; Please let us know if it suits your needs!
In your case I think the issue might be that url needs to be an instance of URL, otherwise createReadStream() will treat it as a path - is it a URL type?
It's worth noting that in v4.0.0 you will also be able to pass a fetch call directly, for example:
const file = await openai.files.create({
  file: fetch(url),
  purpose: 'fine-tune',
});

Thanks. When will it be available?

You can install it today with npm install<EMAIL_ADDRESS>We can't say for sure when the GA will be available, but the beta is looking pretty stable.
To directly address the issue, this is available in the beta like so:
const response = await openai.files.create(
    fs.createReadStream("test.jsonl"),
    "fine-tune"
);


Add CUDA 11.1 and 11.2 to KEEP_ENV_VARS defaults

vcpkg right now can't build CUDA ports with version 11.1 and 11.2, since the associated environment variables aren't being forwarded.

This LGTM, though we should continue to search for a better solution for CUDA in the future that doesn't require modifying the vcpkg binary.

@ras0219-msft While this fix came out of my attempt to fix CMake CUDA language support in the vcpkg environment, in the end this didn't fix anything, since as far as I can tell the versioned CUDA_PATH environment variables are only being used by the MSVC CMake generator, which doesn't seem to be used that often across vcpkg, if at all? The underlying cause is that until now, the CMake generators that don't come directly with associated compilers require nvcc to be present in PATH, not just reachable through CUDA_PATH.
This will probably be fixed in CMake 3.21, but of course I cannot vouch for how well this works in environments with multiple CUDA versions, users will probably have to specify and KEEP_ENV_VARS CUDACXX manually there for this to work with Ninja.


Updates to get CI linux binary builds working

Temurin seems to get closest to the mark here, but still missing i386 Java 8. Going to have to evaluate others but at least I can get the x86_64, arm, and aarch64 builds working again.

For reference: Azul Zulu did not provide arm or aarch64 (and I couldn't get it to find i386 in the deb repo), Corretto does not publish i386 or arm builds, Microsoft does not publish Java 8 builds (and Java 11 only has 64-bit arches).


[Workflow] TEST Test_TC_LVL_3_1 fails for LT feature

Workflow issues
Hi,
In this test Test_TC_LVL_2_2, the OnLevel attribute is set to 5.
The next test TEST_TC_LVL_2_3 has all of the steps disabled.
The next test TEST_TC_LVL_3_1 fails due to OnLevel attribute non-null value.
Error:
[1677227331.633635][38358:38360] CHIP:TOO:  ***** Test Step 27 : TH sends Off command to DUT
[1677227331.636106][38358:38360] CHIP:TOO:  ***** Test Step 28 : TH sends a MoveToLevel command to the DUT with 120, transitionTime: 0, optionsMask: 0, optionsOverride: 0
[1677227331.638517][38358:38360] CHIP:TOO:  ***** Test Step 29 : TH reads CurrentLevel attribute from DUT
[1677227331.640606][38358:38360] CHIP:TOO:  ***** Test Failure: currentLevel.Value() value mismatch: expected 100 but got 1

It seems like the reason for that is the OnLevel attribute still holds the 5, and by the spec for the ON/OFF dependency to the LevelControl cluster:
Table 17. Actions on Receipt for On/Off Commands, when Associated with Level Control:
Off:
1. Temporarily store CurrentLevel.  (100)
2. Change CurrentLevel to the minimum level allowed for the device over the time period OnOffTransitionTime. (1)
3. If OnLevel is not defined, set the CurrentLevel to the stored level. (expecting 100, but got 1, due to OnLevel defined with 5)

After receiving the next MoveToLevel commands, the test expects the command to be discarded, due to the options bitmap ExecuteIfOf: 0, but the Level attribute has changed due to the previous OFF command to the On/Off cluster and the next test assert fails getting 1, instead of 100.
On what platform the issue was observed: Linux GNU
What DUT is used: chip-all-clusters-app, bridge implementation with LT Feature
What TH is used: chip-tool
What commit-sha: v<IP_ADDRESS> released version
How often does the issue is reproduced: every time
How the issue could be resolved: Adding write on-level null as a last step to Test_TC_LVL_2_2 fixes the issue (or a pre-condition for the LVL_3_1)
Platform
core (please add to version below)
Anything else?
LVL_3_1_Fail-chip-all-clusters.txt

@cjandhyala

Hi, @cjandhyala,
Unfortunately, I don't have access to this repo, or at least its giving me 404.
Could you please check?

@manjunath-grl, thanks for the reference. I can see in this commit, that this is the expected fix.
@cjandhyala, I've got my access, but the same issue has been discussed here: 2176. I believe there is no need to repost it.
Thanks


Add "support & consulting" link to the app navigation bar

Description
Adds "support & consulting" link to the app navigation bar
Before:

After:

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Improvement/refactoring (non-breaking change that doesn't add any feature but make things better)

To be merged when https://github.com/epicmaxco/vuestic-ui/pull/4359 is released


Use 4 spaces per tab instead of 8.

This makes lines shorter and easier to read.

This doesn't appear to change the tab width in the UI, it just makes it use 4 spaces instead of tab characters.

Okay, changed it to use tabs by default but keep the tabs to be equal to 4 spaces. The tab width in the UI is noticeably improved.


fix issue incorrect struct name match pattern for command gen service

Codecov Report

Merging #2125 (6831481) into master (508062f) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2125      +/-   ##
==========================================
- Coverage   76.41%   76.38%   -0.04%
==========================================
  Files         516      516
  Lines       46635    46635
==========================================
- Hits        35635    35620      -15
- Misses       9033     9044      +11
- Partials     1967     1971       +4

Flag
Coverage Δ

go-1.15-amd64
?

go-1.16-386
76.38% <ø> (?)

go-1.18-386
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

os/gspath/gspath_cache.go
84.21% <0.00%> (-10.53%)
:arrow_down:

net/gtcp/gtcp_pool.go
71.59% <0.00%> (-3.41%)
:arrow_down:

os/glog/glog_logger_rotate.go
61.99% <0.00%> (-1.36%)
:arrow_down:

container/gtree/gtree_avltree.go
95.40% <0.00%> (-0.58%)
:arrow_down:

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


make camera flip image instead of code to save some cpu

on my tests with delay_thread=0 to see higher utilisation I went down
from 53 to 48 % of cpu utilisation on 4eye3controler setup
i don't think it should be parametrized cause it is really easy to verify by few people and we should simply forget old code.

Agreed. That's about as straightforward as it gets. Thanks for the change!

I'll try and make a release with this change this Sunday.


Stylesheets should be concatenated

To reduce the number of requests, we should concatenate the CSS files. We can either do this in a build step using Grunt or use Sass (Compass) to process the files before building.

Assigning to myself for implementation

Coverage remained the same at 100.0% when pulling d79fe18577fedce6b66a3dc046f2fb3a6c3b82d9 on Coenego:issue-444-css into 1311d05072f93bf2b7ba5d3104f76ab2591247e8 on CUL-DigitalServices:master.

Coverage remained the same at 100.0% when pulling 0cc13ef3da8737a9cb81788cfe18b337de610e25 on Coenego:issue-444-css into 1311d05072f93bf2b7ba5d3104f76ab2591247e8 on CUL-DigitalServices:master.

Assigning to @bp323 for review

Coverage remained the same at 100.0% when pulling e1424fcd3195827eed0e64f4765afd22a0dd0acb on Coenego:issue-444-css into 1311d05072f93bf2b7ba5d3104f76ab2591247e8 on CUL-DigitalServices:master.

Issue remaining: the error pages need to be updated as well.

Coverage remained the same at 100.0% when pulling 0cc09aab1a011c01465bda1ec3df57c0162f3362 on Coenego:issue-444-css into 1311d05072f93bf2b7ba5d3104f76ab2591247e8 on CUL-DigitalServices:master.

Issue remaining: the /admin/users/ pages seem to be missing some styles (see how the Autosuggest displays, for example)

Back to @bp323 for review after follow-up

Verified and merged, nice work!


feat:  show token price in vesting

note: out of scope for ticket 563 but leaving as draft if desired in the future


Fixed documentation for npm test on linux

Now includes latest version of NodeJS rather than relying on a legacy version.
Related Issue:
https://github.com/GoogleChrome/omnitone/issues/51

Closing this PR in favor of #55.


fix: drop webhooks request history on remote reset

What kind of change does this PR introduce?
Bug fix https://github.com/supabase/cli/issues/2696
What is the new behavior?
Also clears supabase_functions.hooks table on reset.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 59.654%

Files with Coverage Reduction
New Missed Lines
%

internal/gen/keys/keys.go
5
12.9%

Totals

Change from base Build<PHONE_NUMBER>8:
-0.03%

Covered Lines:
6386

Relevant Lines:
10705

💛  - Coveralls


HADOOP-19167 Bug Fix: Change of Codec configuration does not work

HADOOP-19167 Change of Codec configuration does not work
Description of PR
In one of my projects, I need to dynamically adjust compression level for different files.
However, I found that in most cases the new compression level does not take effect as expected, the old compression level continues to be used.
Here is the relevant code snippet:
ZStandardCodec zStandardCodec = new ZStandardCodec();
zStandardCodec.setConf(conf);
conf.set("io.compression.codec.zstd.level", "5"); // level may change dynamically
conf.set("io.compression.codec.zstd", zStandardCodec.getClass().getName());
writer = SequenceFile.createWriter(conf, SequenceFile.Writer.file(sequenceFilePath),
                                SequenceFile.Writer.keyClass(LongWritable.class),
                                SequenceFile.Writer.valueClass(BytesWritable.class),
                                SequenceFile.Writer.compression(CompressionType.BLOCK));

Take my unit test as another example:
    DefaultCodec codec1 = new DefaultCodec();
    Configuration conf = new Configuration();
    ZlibFactory.setCompressionLevel(conf, CompressionLevel.TWO);
    codec1.setConf(conf);
    Compressor comp1 = CodecPool.getCompressor(codec1);
    CodecPool.returnCompressor(comp1);

    DefaultCodec codec2 = new DefaultCodec();
    Configuration conf2 = new Configuration();
    CompressionLevel newCompressionLevel = CompressionLevel.THREE;
    ZlibFactory.setCompressionLevel(conf2, newCompressionLevel);
    codec2.setConf(conf2);
    Compressor comp2 = CodecPool.getCompressor(codec2);

In the current code, the compression level of comp2 is 2, rather than the intended level of 3.
The reason is SequenceFile.Writer.init() method will call CodecPool.getCompressor(codec) to get a compressor, eventually CodecPool.getCompressor(codec, null) will be called.
If the compressor is a reused instance, the conf is not applied because it is passed as null:
public static Compressor getCompressor(CompressionCodec codec, Configuration conf) {
Compressor compressor = borrow(compressorPool, codec.getCompressorType());
if (compressor == null) {
  compressor = codec.createCompressor();
  LOG.info("Got brand-new compressor ["+codec.getDefaultExtension()+"]");
} else {
compressor.reinit(conf);   //conf is null here
......

Please also refer to my unit test to reproduce the bug.
To address this bug, I modified the code to ensure that the configuration is read back from the codec when a compressor is reused.
How was this patch tested?
unit test

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 33s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
44m 48s

trunk passed

+1 :green_heart:
compile
17m 24s

trunk passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
compile
15m 54s

trunk passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

+1 :green_heart:
checkstyle
1m 18s

trunk passed

+1 :green_heart:
mvnsite
1m 42s

trunk passed

+1 :green_heart:
javadoc
1m 16s

trunk passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
javadoc
0m 54s

trunk passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

+1 :green_heart:
spotbugs
2m 36s

trunk passed

+1 :green_heart:
shadedclient
35m 28s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 57s

the patch passed

+1 :green_heart:
compile
16m 36s

the patch passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
javac
16m 36s

the patch passed

+1 :green_heart:
compile
15m 46s

the patch passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

+1 :green_heart:
javac
15m 46s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
1m 14s

the patch passed

+1 :green_heart:
mvnsite
1m 40s

the patch passed

+1 :green_heart:
javadoc
1m 10s

the patch passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
javadoc
0m 53s

the patch passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

+1 :green_heart:
spotbugs
2m 46s

the patch passed

+1 :green_heart:
shadedclient
35m 39s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
20m 39s

hadoop-common in the patch passed.

+1 :green_heart:
asflicense
1m  3s

The patch does not generate ASF License warnings.

224m 13s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6807/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/6807

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 00683afafac5 5.15.0-101-generic #111-Ubuntu SMP Tue Mar 5 20:16:58 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / cbd328a2a6aa1b7487409e6620451e91c0e0ae53

Default Java
Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6807/2/testReport/

Max. process+thread count
1429 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6807/2/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@ChenSammi  @jojochuang  @ayushtkn Could you please help me review this PR when you have free time~ Thanks a lot.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

_ Prechecks _

+1 :green_heart:
dupname
0m 01s

No case conflicting files found.

+0 :ok:
spotbugs
0m 00s

spotbugs executables are not available.

+0 :ok:
codespell
0m 01s

codespell was not available.

+0 :ok:
detsecrets
0m 01s

detect-secrets was not available.

+1 :green_heart:
@author
0m 00s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m 00s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
85m 54s

trunk passed

+1 :green_heart:
compile
37m 46s

trunk passed

+1 :green_heart:
checkstyle
4m 25s

trunk passed

-1 :x:
mvnsite
4m 14s
/branch-mvnsite-hadoop-common-project_hadoop-common.txt
hadoop-common in trunk failed.

+1 :green_heart:
javadoc
4m 43s

trunk passed

+1 :green_heart:
shadedclient
143m 33s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 40s

the patch passed

+1 :green_heart:
compile
36m 18s

the patch passed

+1 :green_heart:
javac
36m 17s

the patch passed

+1 :green_heart:
blanks
0m 00s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
4m 38s

the patch passed

-1 :x:
mvnsite
4m 30s
/patch-mvnsite-hadoop-common-project_hadoop-common.txt
hadoop-common in the patch failed.

+1 :green_heart:
javadoc
4m 35s

the patch passed

+1 :green_heart:
shadedclient
149m 58s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
asflicense
5m 19s

The patch does not generate ASF License warnings.

471m 10s

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hadoop/pull/6807

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
MINGW64_NT-10.0-17763 a388fdfcc139 3.4.10-87d57229.x86_64 2024-02-14 20:17 UTC x86_64 Msys

Build tool
maven

Personality
/c/hadoop/dev-support/bin/hadoop.sh

git revision
trunk / cbd328a2a6aa1b7487409e6620451e91c0e0ae53

Default Java
Azul Systems, Inc.-1.8.0_332-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch-windows-10/job/PR-6807/2/testReport/

modules
C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch-windows-10/job/PR-6807/2/console

versions
git=2.44.0.windows.1

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@ZanderXu  I do not have permission to merge my commit. Could you help merge the commit? Thanks!

@ferhui @adoroszlai @goiri  Can you help review and merge the commit when you have free time~ Thanks a lot.

@steveloughran, @jojochuang, @Hexiaoqiao, Can you help review and merge the commit when you have free time~ Thanks a lot.

@ZanderXu should have the right permissions

@ZanderXu Could you help merge the commit when you have time? Thanks a lot

Merged. Thanks @skyskyhu for your contribution.

Thanks to @ZanderXu for your kind help!


chore: upgrade go dependencies

from: https://github.com/jenkins-x/jx3-versions

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign rajdavies
You can assign the PR to them by writing /assign @rajdavies in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr
9b472ed5d1d39b83520895bc9a5f0207de48766b
link
/test

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.


Build improvements

Travis is broken because ./gradlew shadow now resolves to multiple tasks so gradle fails.

Oh no. Does shadow plugin 2.0 wants us to use shadowJar now because there are more shadow tasks? Please change the travis script too and thanks for the contribution!

One weird thing now with newer version of Shadow (which seems like issue with how Shadow maps inputs/outputs) is this output:
> Task :detekt-cli:shadowJar UP-TO-DATE
A problem was found with the configuration of task ':detekt-cli:shadowJar'. Registering invalid inputs and outputs via TaskInputs and TaskOutputs methods has been deprecated and is scheduled to be removed in Gradle 5.0.
 - No value has been specified for property 'mainClassName'.

> Task :detekt-cli:startShadowScripts UP-TO-DATE
Using TaskInputs.file() with something that doesn't resolve to a File object has been deprecated and is scheduled to be removed in Gradle 5.0. Use TaskInputs.files() instead.

> Task :detekt-watch-service:shadowJar UP-TO-DATE
A problem was found with the configuration of task ':detekt-watch-service:shadowJar'. Registering invalid inputs and outputs via TaskInputs and TaskOutputs methods has been deprecated and is scheduled to be removed in Gradle 5.0.
 - No value has been specified for property 'mainClassName'.

Despite that, still running java -jar detekt-cli/build/libs/detekt-cli-1.0.0.RC5-3-all.jar and java -jar detekt-watch-service/build/libs/detekt-watch-service-1.0.0.RC5-3-all.jar both seem to work appropriately.

Should update README as well, it says to use ./gradlew build shadow but will have to change to ./gradlew build shadowJar - https://github.com/arturbosch/detekt/blob/master/README.md#build--use-the-commandline-interface

Thanks shadowJar works for me without any warnings. Unfortunately the install tasks does not work anymore:
> Could not publish configuration 'archives'
   > A POM cannot have multiple artifacts with the same type and classifier. Already have MavenArtifact detekt-cli:zip:zip:null, trying to add MavenArtifact detekt-cli:zip:zip:null

Seems like the maven plugin and the task install is deprecated. detekt already has the maven-publish plugin and publishToMavenLocal does work. Can you please remove the this line apply plugin: 'maven' ? Thanks!

Thanks, everything works! :)

Awesome, thanks! Looking forward to hopefully contributing a few more things here, going to try and tackle some Gradle improvements soon now that I have started to use this project!

Would love to see more :D! :+1:


Self-signed certificate warning

My understanding is there is no domain name validation of the key-pair - perhaps the documentation could elaborate on why a self-signed certificate is not recommended for this usage

[!CAUTION]
Using a self-signed certificate is only recommended for development, not production.

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

ID: dc9bed86-d11a-8f1f-4e58-dfa693f0c8df
Version Independent ID: 65163c9f-40ad-ea33-e589-cc5a9b7b2d77
Content: Create a self-signed public certificate to authenticate your application - Microsoft Entra
Content Source: articles/active-directory/develop/howto-create-self-signed-certificate.md
Service: active-directory
Sub-service: develop
GitHub Login: @FaithOmbongi
Microsoft Alias: ombongifaith

@scottwtang
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

reassign:henrymbuguakiarie

@scottwtang Thanks for providing feedback that helps improve our documentation. This issue has been resolved and we are closing the issue. #please-close


Add a DemoApplication and RecyclerViewDemoActivity as sample

@puf
Hi Frank,
Please review this change. It adds a DemoApplication and a RecyclerViewDemoActivity. After this change I'll add more code using the FirebaseRecyclerViewAdapter and merge my changes into it.

@htcbug overall it looks good, but I have a few comments. Address those (either in code or by responding) and assign the PR back to me.

I'm going to merge this in to master. Any more changes will end up there or on a new branch.


type=rule 'actions=exporttoexcel:file.html,AddressresolveSummary' - is not showing value for AddressObject Wildcard and also not for Region

Describe the bug

Expected behavior

show address summary if object IPWildcard
show address summary if object Region

Current behavior
object of type IPWildcard is shown as unresolved

Possible solution

wildcard object is fixed with future version 2.1.11 - actual develop version

fixed - pre-defined Region object does not have value


Failed to copy output to clipboard: XcbConnect(DisplayParsingError)

Could a flag be added to skip copying the output to clipboard so code2prompt can be used on a remote without a clipboard.

Quick Solution: Remove Clipboard Functionality and Recompile
The arboard::Clipboard functionality in main.rs is causing issues. To resolve this, we need to remove the related lines of code and recompile the project.

Remove the following lines from main.rs:
use arboard::Clipboard;
...
let mut clipboard = Clipboard::new().expect("Failed to initialize clipboard");
clipboard.set_text(rendered).expect("Failed to copy output to clipboard");

println!(
    "{}{}{} {}",
    "[".bold().white(),
    "✓".bold().green(),
    "]".bold().white(),
    "Prompt copied to clipboard!".green()
);

Recompile the project:
cargo build --release

@AnirudhRahul I implemented the --no-clipboard option in #20. If you are on a remote machine, I'm able to use it using the --output flag.

Fixed in #20 by @ODAncona! 🙌 🎉


Add robbiet480 as co-owner of CocoaPod to allow pushes

@dzenbot mind adding me as a co-owner of the CocoaPod so I can push new versions? Should just need to run pod trunk add-owner Iconic<EMAIL_ADDRESS>to get it done. You will remain an owner. Thanks

@dzenbot Doh! Thanks for letting me know. Guess 1.4 won't be pushed for a while. Thanks again

Thanks!

@dzenbot Just a heads up that pod trunk me is still showing I am not associated with any pods.

@dzenbot Yep, all good, thanks!

@robbiet480 Sorry for bothering. Has 1.4 been pushed to CocoaPods trunk?
From CocoaPods's official website, it looks like the latest version still remains 1.3.
Using pod search also returns:
-> Iconic (1.3)
   Auto-generated icon font library for iOS [beta]
   pod 'Iconic', '~> 1.3'
   - Homepage: https://github.com/dzenbot/Iconic
   - Source:   https://github.com/dzenbot/Iconic.git
   - Versions: 1.3, 1.2, 1.1, 1.0.5, 1.0.4, 1.0.3, 1.0.2, 1.0.1, 1.0.0,
   1.0.0-beta4, 1.0.0-beta3, 1.0.0-beta1 [master repo]

@dlackty Thanks for alerting me, getting 1.5 pushed now (1.4 had a breaking bug).


Prometheus remote write missing measurements name

Bug report
Metrics send from prometheus to influxdb with native influx remote write have missing measurement name.
All metrics are saved by InfluxDB to _ measurement.
> select * from _ where __name__='up' and kubernetes_io_hostname='ctl03' limit 1;
name: _
time                From To __name__ action addonmanager_kubernetes_io_mode alertmanager alertname alertstate annotation_io_kubernetes_container_hash annotation_io_kubernetes_container_ports annotation_io_kubernetes_container_restartCount annotation_io_kubernetes_container_terminationMessagePath annotation_io_kubernetes_container_terminationMessagePolicy annotation_io_kubernetes_pod_terminationGracePeriod annotation_kubernetes_io_config_seen annotation_kubernetes_io_config_source annotation_kubernetes_io_created_by annotation_prometheus_io_port annotation_prometheus_io_scrape annotation_scheduler_alpha_kubernetes_io_tolerations app backend beta_kubernetes_io_arch beta_kubernetes_io_os bind branch buildDate call client cluster_name code com_docker_stack_namespace com_docker_swarm_node_id com_docker_swarm_service_id com_docker_swarm_service_name com_docker_swarm_task_id com_docker_swarm_task_name com_mirantis_image_specs_calico_bird6Url com_mirantis_image_specs_calico_birdUrl com_mirantis_image_specs_calico_birdclUrl com_mirantis_image_specs_calico_confdUrl com_mirantis_image_specs_calico_felixImage com_mirantis_image_specs_docker_imgTag com_mirantis_image_specs_gerritChangeId com_mirantis_image_specs_gerritChangeNumber com_mirantis_image_specs_gerritPatchsetNumber com_mirantis_image_specs_gerritPatchsetRevision com_mirantis_image_specs_gerritProject com_mirantis_monitoring compiler container_image container_name container_version contentType controller controller_revision_hash cpu database device dialer_name direction dockerVersion endpoint engine engine_host event exe f64 fstype gitCommit gitTreeState gitVersion goVersion goversion grpc_code grpc_method grpc_service grpc_type handler host id image input instance integration interface interval io_kubernetes_container_logpath io_kubernetes_container_name io_kubernetes_docker_type io_kubernetes_pod_name io_kubernetes_pod_namespace io_kubernetes_pod_uid io_kubernetes_sandbox_id ip_version job                 k8s_app kernelVersion kubernetes_io_hostname le listener_name major method minor name namespace network node_attribute_master node_host node_id node_name node_role_kubernetes_io_master node_role_kubernetes_io_node operation_type osVersion output path pattern pid platform pod_name pod_template_generation pod_template_hash port process_name protocol_instance proxy quantile queue reason refid region  remote resource retentionPolicy revision role route_type rule_type scope scrape_job server service severity slice socket state state_prefix status stratum subresource sv table type unit url username verb version walPath
----                ---- -- -------- ------ ------------------------------- ------------ --------- ---------- --------------------------------------- ---------------------------------------- ----------------------------------------------- --------------------------------------------------------- ----------------------------------------------------------- --------------------------------------------------- ------------------------------------ -------------------------------------- ----------------------------------- ----------------------------- ------------------------------- ---------------------------------------------------- --- ------- ----------------------- --------------------- ---- ------ --------- ---- ------ ------------ ---- -------------------------- ------------------------ --------------------------- ----------------------------- ------------------------ -------------------------- ---------------------------------------- --------------------------------------- ----------------------------------------- ---------------------------------------- ------------------------------------------ -------------------------------------- --------------------------------------- ------------------------------------------- --------------------------------------------- ----------------------------------------------- -------------------------------------- ----------------------- -------- --------------- -------------- ----------------- ----------- ---------- ------------------------ --- -------- ------ ----------- --------- ------------- -------- ------ ----------- ----- --- --- ------ --------- ------------ ---------- --------- --------- --------- ----------- ------------ --------- ------- ---- -- ----- ----- -------- ----------- --------- -------- ------------------------------- ---------------------------- ------------------------- ---------------------- --------------------------- --------------------- ------------------------ ---------- ---                 ------- ------------- ---------------------- -- ------------- ----- ------ ----- ---- --------- ------- --------------------- --------- ------- --------- ------------------------------ ---------------------------- -------------- --------- ------ ---- ------- --- -------- -------- ----------------------- ----------------- ---- ------------ ----------------- ----- -------- ----- ------ ----- ------  ------ -------- --------------- -------- ---- ---------- --------- ----- ---------- ------ ------- -------- ----- ------ ----- ------------ ------ ------- ----------- -- ----- ---- ---- --- -------- ---- ------- -------
1513343319131000000         up                                                                                amd64                   linux                                                                                0                                                                                ctl03                                                                                kubernetes-cadvisor                       ctl03                                                                                true                                                                                region1

Looks like labels from ALL metrics are merged, so each metric have labels (set to null) from all other metrics...
When Prometheus Remote Storage Adapter is in place:
> select * from up limit 1;
name: up
time                beta_kubernetes_io_arch beta_kubernetes_io_os instance job             kubernetes_io_hostname node_role_kubernetes_io_master node_role_kubernetes_io_node region  value
----                ----------------------- --------------------- -------- ---             ---------------------- ------------------------------ ---------------------------- ------  -----
1513585977606000000 amd64                   linux                 ctl03    kubernetes-node ctl03                  true                                                        region1 0

IMHO native support for prometheus remote write should behave similar to remote storage adapter, and use label __name__ as measurement name
InfluxDB: 1.4.2-1
Ubuntu: 16.04
Prometheus: 2.0.0
How to reproduce:

Configure remote write in prometheus:

remote_write:
  # docker_remote_write
  - url: http://INFLUXDB_HOST:8086/api/v1/prom/write?u=USER&p=PASS&db=DB

Fixing this would also make the Influxdb prometheus backend backwards compatible with the remote_storage_adapter. This is essential to use the native backend as a drop-in replacement for the adapter.

+1

+1

Since i am not sure which post i should thumbs-up, here is the obligatory +1 from me.

Hi there!
Is someone taking care of this? I might give it a try, but I am totally new to Influxdb codebase and Golang. From my initial findings, the bug is localized in prometheus/converters.go. We should adapt the write function and probably also the read. I don't know what the other fonctions do.
I am not the maintainer, but I guess any help is welcomed :)
(Currently, the measurementName is _ and should be for instance node_load1 ; the fieldName is f64 and should be for instance usage_percent)

@Nodraak do it, try it. i think my skills are as new as yours, an i currently don't have the time, but i, for myself, am waiting for someone to fix this.

@pauldix If this change is made, please make sure to include me to doc it.

Originally, this was actually by design to have this future proof against IFQL. For now, the best way to query data from Influx that was written through the remote write gateway is to do it through the Prometheus remote read. In the new IFQL query language there's no distinction between measurements, tags, and fields. They're all just key and value pairs.
And __name__ is a tag key. Although now that I'm thinking a bit more about this it might be a better idea to actually have the measurement name match the __name__ in addition to having the __name__ tag. Would give the most flexibility while working with where things are going.
Thoughts?

+1 .
Any update on this issue?
Or any workaround?

@pauldix From your comment I guess you are going to migrate to (some sort) of the prometheus metrics model, where __name__ is a tag, but also the name of the series? Is it intended to do this mapping transparently, or just by duplication of the value of the __name__ key?
If this mapping is done transparently (__name__ => measurement name), the remote storage endpoint of influx should be backwards compatible with the prometheus remote storage adapter. However this will break any measurements which already have a __name__ tag that does not match the measurement name. Nevertheless, I vote for this! This is a much smaller breaking change than by just using the __name__ value as measurement name.
In the meantime, is it possible to just add the measurement name by duplication of the __name__ value? This would be very helpful.

In the meantime, is it possible to just add the measurement name by duplication of the name value? This would be very helpful.

@fmoessbauer PR above does exactly this :) I can't speak to the suitability of this re IFQL mentioned above, but the duplication itself is trivial

Ok, I think I've found a solution to this problem, but bear with me for a little bit of extra history/detail.
The original reason I pushed everything into a single measurement named _ was partly because of IFQL, but also because of a limitation in our query engine. Prometheus has matchers against __name__ for != and !~ (which are standard tag matchers) which InfluxQL can't support against measurements. For the first version of the Prometheus remote read endpoint I had to use the InfluxQL engine since that's all that existed.
That limitation kept me from being able to put the Prometheus metric name in addition to having __name__ as a tag. The experimental Prom -> Influx gateway opted to throw errors for those cases: https://github.com/prometheus/prometheus/blob/master/documentation/examples/remote_storage/remote_storage_adapter/influxdb/client.go#L153-L164
However, since I wrote the original remote adapter, we've had a bunch of work done on creating a lower level query layer that bypasses the InfluxQL engine to get to storage. This is work directly related to IFQL, but I realized it could probably be used for the remote adapter. I've pushed a branch here: https://github.com/influxdata/influxdb/tree/pd-prometheus-to-new-storage
The good news is that this will probably be more performant and use less memory since it doesn't use the entirety of the InfluxQL query engine to process the request. I've tested it manually and it seems to work. That's experimental (just like this whole feature). I need to review with @stuartcarnie to figure out a less "ghetto-hack" solution.
If you're so inclined, I'd be interested in hearing if the branch works for you.

Is this new behaviour control by a setting or is it a new unconditional breaking change ?

I'm fairly new to InfluxDB, so please excuse me if this is obvious, but how can I take advantage of this change @e-dard? In Influx v1.6.1 (in a new database, with data fresh out of Prometheus) I still only see one measurement (_), and I cannot use Prometheus metric names as measurements in Influx queries. That is, I still have to write queries like select * from _ where __name__='up' and kubernetes_io_hostname='ctl03' limit 1; instead of the preferable select * from up limit 1;.
What am I missing?
Like @Jean-Daniel I would also like to know if this behaviour is on by default or if we need to opt-in with a setting.


app crashes and kickout when textinput become empty (length ==0)

Environment
react-native-cli: 2.0.1
react-native: 0.53.0
Expected Behavior
i want simply clear textinput with keyboard input
in simulator ios and andorid emulator it's fine
but in real android devices it  crashes HUAWEI Y7 and other android devices that i tested i have this problem !!!
Actual Behavior
when clear textinput with keyboard backspace, app crashes and kickout
it writes string as well but when clear text and text length become zero app crashes

seriously?!
can you hint me where the problem is in RN code?
i will update ASAP and report again

i catched this error
error: patch failed: .flowconfig:51
Falling back to three-way merge...
Applied patch to '.flowconfig' cleanly.
error: android/app/src/main/AndroidManifest.xml: does not exist in index
git-upgrade WARN The upgrade process succeeded but there might be conflicts to be resolved. See above for the list of files that have merge conflicts.
git-upgrade info Upgrade done

you must close all IDE and close termial. just exit from debuging mode

now textinput corrected
and you must init a new project

It does not look like this has been reproduced in 0.54 or later. There has been no follow up from the author in over seven days, so we're closing this issue.


Almost 4 days since the hot wallet was last funded

This is unacceptable and needs to be publicly addressed.

LOL good luck with that! Try blaming the bitcoin thieves that exploited the exchange bitcoin glitch the first day of the exchange.

@chass3768  This is a false statement, try to prove it!!!

I, too, saw and reported, but now understand, that for good reason not exploited

@snicksand @chass3768
And that allowed to make this vulnerability??

I think programmers made a mistake when updating scripts

buying myX coins, coins were bought and bitcoin doubled on balance

they use BTC to buy LTC for fork... waiting to convert there LTC fro LCC... . just my opinion.

It's possible, but unlikely.

Monyx has stated they’ve resolved the bug, there is NO reason not to be filling the hot wallet at least once a day.

the error was resolved on February 7

@snicksand All signs of fraud

Only time will tell. If 2-3 days without news, it means SCAM

they waited for people to buy more coins on the exchange. They do not have 413 bitocons, much more ... People buy more coins in the hope of earning

bitcoins*

Lol. They filled the hot wallet with .3 bitcoin once, used the withdrawals as "proof" of legitimacy.
They did the same thing during the ICO. The filled the books with fake buy orders,
there were only actually about 1.2million monyx distributed to actual people.
The other coins were distributed back to monyx.
Its a shame because they could have used this market control to make the price 10 dollars extremely easily, but since they live in Nigeria, they just took the money and ran. Because they are kings in their country now.


New Category: Tutorial

A Tutorial is just a Markdown document with a bunch of metadata fields:

Name
Author
Packages and their versions
Tags

The fields would allow linking to the tutorial from a package name, a tag page, and so on.

I'm wondering if we should just generalize this to an "Article", and use the metadata to identify it as a blog post, a tutorial, an opinion piece, etc.

An article can now be assigned a type (in 2af30f99d3be3a10baf70c966cb8000e03a421ff), so we can close this.


add chunkdiff

Codecov Report

Merging #14 (a76bd26) into main (b5709ac) will decrease coverage by 33.92%.
The diff coverage is 20.83%.

@@             Coverage Diff              @@
##              main      #14       +/-   ##
============================================
- Coverage   100.00%   66.07%   -33.93%
============================================
  Files            1        2        +1
  Lines           96      168       +72
============================================
+ Hits            96      111       +15
- Misses           0       57       +57

Impacted Files
Coverage Δ

chunkdup/chunkdiff.py
20.83% <20.83%> (ø)

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


multi: use fileMode when writing files

Change Description
This PR is on top of https://github.com/lightningnetwork/lnd/pull/6768 as I got more bandwidth to work on. Takes into consideration comments on reviews on that PR.
Also is based on https://github.com/lightningnetwork/lnd/issues/7192, it was easier to do it from top than rebase it
Should fix issue https://github.com/lightningnetwork/lnd/issues/6720
Steps to Test
Steps for reviewers to follow to test the change.
Pull Request Checklist
Testing

[ ] Your PR passes all CI checks.
[ ] Tests covering the positive and negative (error paths) are included.
[ ] Bug fixes contain tests triggering the bug to prevent regressions.

Code Style and Documentation

[ ] The change obeys the Code Documentation and Commenting guidelines, and lines wrap at 80.
[ ] Commits follow the Ideal Git Commit Structure.
[ ] Any new logging statements use an appropriate subsystem and logging level.
[ ] Any new lncli commands have appropriate tags in the comments for the rpc in the proto file.
[ ]  There is a change description in the release notes, or [skip ci] in the commit message for small changes.

📝 Please see our  Contribution Guidelines for further guidance.

I'm creating a new sub-module. So I would need to merge this PR, some how bump the version as #8266 and then update other sub packages. In this way I won't need to use replace in all submodules and it would be consistent on previous approach.

I don't think we should add a new submodule just for a single method. Submodules have a lot of overhead and maintenance cost because of the way they work (what you described above). How about just adding that file to the lnutils package?

I don't think we should add a new submodule just for a single method. Submodules have a lot of overhead and maintenance cost because of the way they work (what you described above). How about just adding that file to the lnutils package?

Yes I was thinking the same but I had the same issue as lnutils is not used in other submodule (certs, cmd or kvdb) and in sake of not repeating code in each submodule I thought about the new one.
Should I worried about those? Do you have any thoughts? @guggero

Ah, I see. Then maybe the new fn package instead? That is already a sub module. Could have a new file called fn/io.go.
Also, please take a look at our formatting guideline: https://github.com/lightningnetwork/lnd/blob/master/docs/code_formatting_rules.md

Hi @guggero
I talked with @mauricepoirrier to take this PR.
I created #8914 with the new WriteFunction in fn package.
The idea is to get that merged to create another PR that bumps the updated submodule in the root go.mod and uses the new functionality to fix #6720.


Ensure consistent props/targets file naming

Motivation
Though Yardarm doesn't use them, the NuGet restore process does create
.nuget.g.props/targets files in the intermediate directory. We should
ensure consistent naming with MSBuild patterns.
Modifications
Set the ProjectPath to the assembly name + ".csproj".
Allow no-op restores using the nuget cache and dependency graph spec.
Results
The files are consistently named "AssemblyName.csproj.nuget.g.props" and
".targets".
Manual restores via Yardarm may be quicker if repeated without changes
due to cache files.

/merge


Road to new Package Installer for macOS 10.10 to 10.13 Mojave

Hi everyone,
I Just have enabled Wiki, Issues, Feature Report
feel free to leave some requests ...
Regards.

This content was originaly discuted from https://github.com/tambovchanin/macos-sierra-sane-binaries/issues/5
Hi everyone,
I got the same issue as @mePy2 with MacOS Sierra 10.12.6, installer says "it was compiled on MacOS 10.11" ... and more
But by using the ancestral PackageMaker.app (fun app and usefull for creating and validate PKG for 10.4 to 10.6), i was able to re-PKG the bundle Project and install it ...
BUT, NOTHING appear in ImageCapture.app ...
BUT, on command line ;; scanimage --format jpg > test.jpg ;; my old Canon N670U (Lide20 compliant) revive and i got a nice scan sample.
BUT a second use of this command and came the drama of "scanimage: no SANE devices found"
BUT if i un-plug and re-plug my scanner,  ;; scanimage --format jpg > test.jpg ;; work again.
Also got an error with my UMAX 1200SE : ;; scanimage: open of device artec_eplus48u:libusb:002:001-05d8-4010-00-00 failed: Invalid argument ;;
SO, therefore maybe, i propose to Team up i'm in !!
Configuration : MacOS 10.12.6 with XCode 7.2.1 (Customised, SDK 10.4 to 10.11) + XCode 9.2 (standard SDK, )
Scanner :  Canon N670U + Umax 1200SE + Canon Lide20
Test Suite : MacOS 10.12.6 + MacOS 10.10.5 + MacOS 10.11.xxx
Regards.
Originally posted by @genose in https://github.com/tambovchanin/macos-sierra-sane-binaries/issues/5#issuecomment-477282786

This content was originaly discuted from https://github.com/tambovchanin/macos-sierra-sane-binaries/issues/5
Yeah Absolutly, let's organise ...
What we got here, projects plannings funs Yeah !!
To improve our scalability and project success, may we first resume which are ours dev configurations, futheremore to help understanding issues and share efficiently ours Skils and knowledges, doesn't we ?
Last Time, i've a bit introduce myself and resume some resolution on how i unbrick the actual PKG, and obtain some encouraging or disgracefull results
https://github.com/tambovchanin/macos-sierra-sane-binaries/issues/5#issuecomment-477282786
// 1 // / ***************************************
Well i identify some ancestral recursive "not so issues" in your released package @tambovchanin, not blaming you see scripts are more than 2 years for now.
in M.E Sources you got 2 lines comparing System version
(system.compareVersions(system.version.ProductVersion, '@MACOSX_DEPLOYMENT_TARGET@') < 0) ||
(system.compareVersions(system.version.ProductVersion, '@MACOSX_DEPLOYMENT_TARGETX1@') >= 1 ))
Where MACOSX_DEPLOYMENT_TARGETX1 == Upper Version of Targetted Version
MACOSX_DEPLOYMENT_TARGET == Lower Version you Targetted or here the SDK Version
// 2 // / ***************************************
I made some "correction" (adaptative path select ) on build.sh, to find the Rigth XCode installation, Yes i Got Two(2) of them on my mac at least !!
Because XCode 7.2 is the only version reliable for compiling this project without errors ...
and on XCode 9.2 / LLVM 9.0 / GCC 5.1, i got some strange wrong configuration flags for Cxx like  "Back In Time machine" mode causing vanishing data type (u_long, uint) nor SDK was ignored.
// 3 // / ***************************************
Does it install Package  ? YES, for 12.12.6
Just for based on 10.11 SDK
We on the way @mePy2

// 4 // / ***************************************
Does it Work, Almost because BUGS ...
i've just understand how SANE find Scanner ... Mostly by Vendor ID for USB Descriptors
searching for USB scanners:
found USB scanner (vendor=0x04a9 [Canon], product=0x220d [CanoScan], chip=LM9832/3) at libusb:001:002-04a9-220d-ff-00
found USB scanner (vendor=0x04a9 [Canon], product=0x220d [CanoScan], chip=LM9832/3) at libusb:002:001-04a9-220d-ff-00
found USB scanner (vendor=0x05ac [Apple Inc.], product=0x8215 [Bluetooth USB Host Controller]) at libusb:003:002-05ac-8215-ff-01
user@localhost$ scanimage -x 100 -y 100 --format=tiff >image.tiff
scanimage: sane_start: Error during device I/O
user@localhost$ scanimage -x 100 -y 100 --format=tiff >image.tiff
scanimage: no SANE devices found
BUT, my Bluetooth antenna really not a scanner
So this is, ** almost ** why the backend fail
// 5 // / ***************************************
Some mind appointments ...

HomeBrew Packager is certainly nice to obtain the best version of librairies, adapted and compatible with your Running MacOS, but our can bundle some magical wizard spell in the compilation process and running an installation pre-fligth script

TWAIN Interface, is why i'm here, so i personally want to maintain it, beside Adobe Photoshop (up to version 2015), want to Drop the support of TWAIN Integration

Other pre-requirement are not install in the same time, it's quite annoying

Take the fly on Official Release ( https://gitlab.com/sane-project )  and report / patch to our platform and make PKG for it

it took me the afternoon to obtain thoses result and it's not some fun, cause after i had to commit my modification to the public

So, an XCode Project with GitHub / GitLab Repo integration is necessary to make all our effort possible and mergeable in one nice maintened project.
Regards.
Originally posted by @genose in https://github.com/tambovchanin/macos-sierra-sane-binaries/issues/5#issuecomment-478689284

This content was originaly discuted from https://github.com/tambovchanin/macos-sierra-sane-binaries/issues/5
Hi everyone,
check newly posted updates for MacOS 10.10 / 10.11; 10.12; 10.13 ...
This release is just an unlocked revision of current Mattias Ellert sources
Got some issues to compile sane-backend with 10.13 SDK (some libcrypto version issues), this shall resolved soonly
Take a look on https://github.com/genose/macos-sierra-sane-binaries
Let me know your feedback
Regards.
Originally posted by @genose in https://github.com/tambovchanin/macos-sierra-sane-binaries/issues/5#issuecomment-486099532


Hey! It looks like you meant to submit this PR to a Slate upstream fork instead of our original repo? If you create a new PR and select the "base" to go to the fork instead of here, it should go to the right place.


Update React-navigation version

Close in favor of https://github.com/LeoLeBras/react-router-navigation/commit/f26a821f73c5fa5bbc1a19448826711410efa9bc


Request: Hardcoded text

Hi Sparrow,
A few strings are hardcoded.
Woudl you make it possible to translate them?

Improved Deal Screen

(1) DEAL TERMS
cf.
\Integrations\IDS\diplomacydealview.xml
Line: 189
                <Label ID="DealTermsLabel" String="DEAL TERMS" Anchor="C,T" Style="FontBoldFlair26" FontStyle="glow" Color0="GoldMetalDark"  Color1="0,0,0,200"/>
(2) - Cede
cf.
\Integrations\IDS\diplomacydealview.lua
Line: 1917
      button.IconText:SetText(button.IconText:GetText() .. '[COLOR_Civ6Green] - Cede[ENDCOLOR]');
(3) - Return
cf.
\Integrations\IDS\diplomacydealview.lua
Line: 1922
        button.IconText:SetText(button.IconText:GetText() .. '[COLOR_Civ6Red] - Return[ENDCOLOR]');

Unit Report Screen

(1) Promotion Available
cf.
\Integrations\URS\reportscreen.xml
Line: 1125
            <Image Offset="0,-1" Texture="Stats32" Size="32,32" Anchor="R,C" ToolTip="Promotion Available" Icon="ICON_PROMOTION"/>
(2) In Formation
cf.
\Integrations\URS\reportscreen.xml
Line: 1135, 1173, 1244, 1280, 1320
          <Image ID="Formation" Hidden="1" Offset="0,-1" Texture="Stats32" Size="32,32" Anchor="R,C" ToolTip="In Formation" Icon="ICON_UNITCOMMAND_ENTER_FORMATION"/>

Check "Deal View" now and let me know if that part changes fine and the translation is ok

It works fine!

Great is the translation ok >?

Yeah, no problem.

I could translate the strings of DEAL TERMS, Promotion Available and In Formation.
I have no idea how to tweak the diplomacydealview.lua, though.
Integrations.zip

the Cede and Return are hard coded here https://github.com/sparrow8332/CQUI_Community-Edition/blob/10035b712233f58c8b6d81d63ec28b9c2a519d49/Integrations/IDS/diplomacydealview.lua#L1917
and here https://github.com/sparrow8332/CQUI_Community-Edition/blob/10035b712233f58c8b6d81d63ec28b9c2a519d49/Integrations/IDS/diplomacydealview.lua#L1922
They should be replaced by Locale.Lookup("SOME_LOC_KEY_FOR_CEDE_OR_RETURN" )

Thanks, Azurency.
However, I'm not sure where to insert the code.
Could you write the entire line?
cf.
LOC_IDS_DEAL_CEDE
LOC_IDS_DEAL_RETURN

I found other texts.

Suzerain of
https://github.com/sparrow8332/CQUI_Community-Edition/blob/10035b712233f58c8b6d81d63ec28b9c2a519d49/Integrations/IDS/diplomacydealview.lua#L1610

(Duplicate)
https://github.com/sparrow8332/CQUI_Community-Edition/blob/10035b712233f58c8b6d81d63ec28b9c2a519d49/Integrations/IDS/diplomacydealview.lua#L1765

(Untradeable)
https://github.com/sparrow8332/CQUI_Community-Edition/blob/10035b712233f58c8b6d81d63ec28b9c2a519d49/Integrations/IDS/diplomacydealview.lua#L1768

Yes, I believe this integration was not translated, there should be even more texts that are not translated.
So to answer you question the entire line should be something like this (ex. for cede) :
button.IconText:SetText(button.IconText:GetText() .. '[COLOR_Civ6Green] - ' .. Locale.Lookup("LOC_IDS_DEAL_CEDE") .. '[ENDCOLOR]');

.. is for concatenation and you juste have to place Locale.Lookup(...) correctly.
Another example for (Duplicate), it would be :
tooltipAddedText = ' (' .. Locale.Lookup("LOC_IDS_DEAL_DUPLICATE") .. ')');

and so on

Thank you again.
There is another one.
https://github.com/Azurency/CQUI_Community-Edition/blob/10035b712233f58c8b6d81d63ec28b9c2a519d49/Integrations/IDS/diplomacydealview.lua#L1556
I managed to replace the hardcoded texts.
CQUI_Community-Edition-summer-2017_experiment.zip
Updated files:

Assets/Text/cqui_text_general.xml
Integrations/IDS/diplomacydealview.lua
Integrations/IDS/diplomacydealview.xml
Integrations/IDS/Text/ids_text.xml
Integrations/URS/reportscreen.xml
Integrations/URS/Text/urs_text.xml

I hope someone tests it.

for the latest steam version
1125169247_CQUI - Community Edition_2017.09.08_experiment.zip

Will be home later today and give it a run through :-)

As I gave my contribution for italian translation to CQUI, I can help here too.

hi,I test it in Traditional Chinese already
everything looks good # @8h42

@ohhhhhhhYi
Glad to hear that.

citypanel.lua

from Population
https://github.com/Azurency/CQUI_Community-Edition/blob/10035b712233f58c8b6d81d63ec28b9c2a519d49/Assets/UI/Panels/citypanel.lua#L410

Religious Minorities
https://github.com/Azurency/CQUI_Community-Edition/blob/10035b712233f58c8b6d81d63ec28b9c2a519d49/Assets/UI/Panels/citypanel.lua#L447

Pillaged
https://github.com/Azurency/CQUI_Community-Edition/blob/10035b712233f58c8b6d81d63ec28b9c2a519d49/Assets/UI/Panels/citypanel.lua#L459
https://github.com/Azurency/CQUI_Community-Edition/blob/10035b712233f58c8b6d81d63ec28b9c2a519d49/Assets/UI/Panels/citypanel.lua#L471

for the latest steam version (2017.09.21)
1125169247_Community Quick User Interface_2017.09.21_experiment.zip
Updated files:

Assets/Text/cqui_text_general.xml
Assets/UI/Panels/citypanel.lua

Screen Shot:

I'll upload Japanese translation if everything goes well.

Another one.
The city will expand to this tile upon the next cultural borders growth
Screen Shot:

citybannermanager.xml

https://github.com/Azurency/CQUI_Community-Edition/blob/ff4533853f1270fcfdf0469102426903496b997e/Assets/UI/WorldView/citybannermanager.xml#L21
https://github.com/Azurency/CQUI_Community-Edition/blob/ff4533853f1270fcfdf0469102426903496b997e/Assets/UI/WorldView/citybannermanager.xml#L22

plotinfo.xml

https://github.com/Azurency/CQUI_Community-Edition/blob/ff4533853f1270fcfdf0469102426903496b997e/Assets/UI/WorldView/plotinfo.xml#L21
https://github.com/Azurency/CQUI_Community-Edition/blob/ff4533853f1270fcfdf0469102426903496b997e/Assets/UI/WorldView/plotinfo.xml#L22

Thank you for merging my branch!


Add check for datastore length

All datastore names in StorageClasses, PVs and the vSphere config is checked for length.
@openshift/storage

/close
in favor of #7