Nil layoutAnchor

_myLabel = [[UILabel alloc] init];
_myLabel.translatesAutoresizingMaskIntoConstraints = NO;
[self.contentView addSubview:_titleLabel];

// This ends up being nil because _myLabel is an object, but it's leadingAnchor property is nil.
[_myLabel.leadingAnchor constraintEqualToAnchor:self.contentView.leadingAnchor constant:xPadding],

Both _myLabel and self.contentView are not nil.
My code keeps crashing (being put into an array) because the NSLayoutConstraint that is generated from the constraintEqualToAnchor method is nil.
Any ideas?

Hello,
Thanks for the interest in using this library :)
I just noticed that you are you are creating a _myLabel but you are then adding _titleLabel as a subview. are you sure there is not mistakes in the example you provided?
It will help me immensely if you could provide some compilable source code (maybe a github gist) that I can run. I tried your example and I cannot reproduce it sadly :(.
Can you provide me with a working demo to reproduce the issue?
Also, what iOS version are you using?
Thanks,

@oarrabi Hey, sorry about that, it was a typo (I'm using _myLabel everywhere, was trying to make it more general and missed the reference to _titleLabel).
While building the demo, I figured out what I was doing wrong. I was trying to create an array of constraints like such
- (NSArray *)overlappedTextPositionConstraints
{
    NSArray *constraints = [self titleLabelConstraints:5.0f yPadding:5.0f];
    return constraints;
}

- (NSArray *)underneathTextPositionConstraints
{
    return [self titleLabelConstraints:5.0f yPadding:0.0f];
}

- (NSArray *)titleLabelConstraints:(CGFloat)xPadding yPadding:(CGFloat)yPadding
{
    return @[
                 [_titleLabel.leadingAnchor constraintEqualToAnchor:self.contentView.leadingAnchor constant:xPadding],
                 [_titleLabel.trailingAnchor constraintEqualToAnchor:self.contentView.trailingAnchor constant:xPadding],
                 [_titleLabel.bottomAnchor constraintEqualToAnchor:self.contentView.bottomAnchor constant:yPadding],
                 [_titleLabel.heightAnchor constraintEqualToConstant:30.0f],
             ];
}

and then later, based on the position the user sets (to overlap or have the text sit underneath an image), choose between the two like such.

@oarrabi Hey, sorry about that, it was a typo (I'm using _myLabel everywhere, was trying to make it more general and missed the reference to _titleLabel).
While building the demo, I figured out what I was doing wrong. I was trying to create an array of constraints like such
- (NSArray *)overlappedTextPositionConstraints
{
    NSArray *constraints = [self titleLabelConstraints:5.0f yPadding:5.0f];
    return constraints;
}

- (NSArray *)underneathTextPositionConstraints
{
    return [self titleLabelConstraints:5.0f yPadding:0.0f];
}

- (NSArray *)titleLabelConstraints:(CGFloat)xPadding yPadding:(CGFloat)yPadding
{
    return @[
                 [_titleLabel.leadingAnchor constraintEqualToAnchor:self.contentView.leadingAnchor constant:xPadding],
                 [_titleLabel.trailingAnchor constraintEqualToAnchor:self.contentView.trailingAnchor constant:xPadding],
                 [_titleLabel.bottomAnchor constraintEqualToAnchor:self.contentView.bottomAnchor constant:yPadding],
                 [_titleLabel.heightAnchor constraintEqualToConstant:30.0f],
             ];
}

and then later, based on the position the user sets (to overlap or have the text sit underneath an image), choose between the two like such.

(void)setTextPosition:(CellTitleLabelTextPosition)textPosition
{
NSArray *overlapConstraints = [self overlappedTextPositionConstraints];
NSArray *underneathConstraints = [self underneathTextPositionConstraints];
void (^deactivateConstraints) (NSArray *) = ^(NSArray *constraints) {
[[self.titleLabel.constraints select:^BOOL(NSLayoutConstraint *constraint) {
return [constraints containsObject:constraint];
}] each:^(NSLayoutConstraint *constraint) {
constraint.active = NO;
}];
};
[self.titleLabel.constraints makeObjectsPerformSelector:@selector(setActive:) withObject:@YES];
if (textPosition == CellTitleLabelTextPositionOverlap)
{
deactivateConstraints(underneathConstraints);
self.titleLabel.textAlignment = NSTextAlignmentLeft;
}
else if (textPosition == CellTitleLabelTextPositionUnderneath)
{
deactivateConstraints(overlapConstraints);
self.titleLabel.textAlignment = NSTextAlignmentCenter;
}
}

@oarrabi Hey, sorry about that, it was a typo (I'm using _myLabel everywhere, was trying to make it more general and missed the reference to _titleLabel).
While building the demo, I figured out what I was doing wrong, and understood how OALayoutAnchor works better. I'm still trying to figure out this use case though.
I'm trying to have 2 sets of constraints which I can swap between, when the textPosition property changes. I wanted to have them be 2 immutable sets, and just go between the two as such.
- (NSArray *)overlappedTextPositionConstraints
{
    NSArray *constraints = [self titleLabelConstraints:5.0f yPadding:5.0f];
    return constraints;
}

- (NSArray *)underneathTextPositionConstraints
{
    return [self titleLabelConstraints:5.0f yPadding:0.0f];
}

- (NSArray *)titleLabelConstraints:(CGFloat)xPadding yPadding:(CGFloat)yPadding
{
    return @[
                 [_titleLabel.leadingAnchor constraintEqualToAnchor:self.contentView.leadingAnchor constant:xPadding],
                 [_titleLabel.trailingAnchor constraintEqualToAnchor:self.contentView.trailingAnchor constant:xPadding],
                 [_titleLabel.bottomAnchor constraintEqualToAnchor:self.contentView.bottomAnchor constant:yPadding],
                 [_titleLabel.heightAnchor constraintEqualToConstant:30.0f],
             ];
}

// Mutator for textPosition property
- (void)setTextPosition:(CellTitleLabelTextPosition)textPosition
{
    NSArray *overlapConstraints = [self overlappedTextPositionConstraints];
    NSArray *underneathConstraints = [self underneathTextPositionConstraints];

    void (^deactivateConstraints) (NSArray *) = ^(NSArray *constraints) {
        [[self.titleLabel.constraints select:^BOOL(NSLayoutConstraint *constraint) {
            return [constraints containsObject:constraint];
        }] each:^(NSLayoutConstraint *constraint) {
            constraint.active = NO;
        }];
    };

    [self.titleLabel.constraints makeObjectsPerformSelector:@selector(setActive:) withObject:@YES];

    if (textPosition == CellTitleLabelTextPositionOverlap)
    {
        deactivateConstraints(underneathConstraints);
        self.titleLabel.textAlignment = NSTextAlignmentLeft;
    }
    else if (textPosition == CellTitleLabelTextPositionUnderneath)
    {
        deactivateConstraints(overlapConstraints);
        self.titleLabel.textAlignment = NSTextAlignmentCenter;
    }
}

Any thoughts on how I could achieve this, or any better suggestion for what might be a more reasonable approach?
Thanks a lot, for the help, and the library!

Hey, I am glad you are enjoying it :)
The code that you posted might work on iOS 8, however wont work on iOS 7, since NSLayoutConstraint does note implement the Active property.
In OALayoutAnchor, I created a category to silence the exception that might happen when calling active https://github.com/oarrabi/OALayoutAnchor/blob/master/Pod/Classes/NSLayoutConstraint%2BSuppressActive.m. This might cause strange misunderstanding.
If I had your same issue, I would iterate over the constraint and only change their constant value.
So instead of activating and deactivating, I would store the constraint constant values and switch them based on what set of constants I want. and then iterate and do theconstraint.constant = thevalue;
This also helps in improving the performance since the constraints are reused.
Will this work for you case?

I'm on iOS 8, not supporting 7, so I get the limitation, but don't mind it. :)
Yes, that would work in this case! But I have another case where there are 2 layouts for a view, which are not particularly similar (and wouldn't constrain the views in the same way necessarily). The other issue with the method of changing the constants (while more performant for sure) is that now I have to create properties (global state) to hold onto the constraints I'd like to change, instead of swapping in one set for another.


is conditions and Java generics

[@gavinking] Consider:
import java.util { ArrayList, L=List }

void run() {
    L<Integer> list = ArrayList<Integer>();
    assert (is ArrayList<Integer> list);
}

Executing this code results in:
Caused by: ceylon.language.AssertionError "Assertion failed
	violated is ArrayList<Integer> al"

Because, I suppose, we don't have access to reified type args for Java types. But in fact we know statically that the type argument is correct. The only thing we're narrowing is List to ArrayList, which can be checked at runtime!
So in fact this code should succeed.
[Migrated from ceylon/ceylon-compiler#1720]

Related: #5738.

Done, see #5738.


Revert "bug fix" (actually a bug) from #2655

In a2b019f82 it was claimed that losing the input string without alloc is a bug. It is not, because allowing no-alloc is desired and there's no way to have the input string otherwise so we just accept it'll be missing and modify the messages accordingly. The commit that forced alloc was also horribly misleading since it kept the alloc feature but it makes this one easier.
Note that input string is still present by default in all configurations except for no-alloc.
I think we should also backport this and release fixed units because of the misleading alloc feature in them. Although it's not urgent. The only crate I know of that is kinda broken by it is ln_types which is already broken by depending on old bitcoin.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.034%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
19508

Relevant Lines:
23494

💛  - Coveralls

@Kixunil want to release units 0.1.2 and yank 0.1.1? It is technically a breaking change but I don't think anyone would be broken who wasn't broken by the push to 0.1.1.
utACK 7f024c333e68a2bbb4e898424366bb9ae99da1cc but will wait for @tcharding (who isn't working today since it's Sunday) to see if he remembers additional context -- I remember a bunch of discussion about this change but I can't find it now.

technically a breaking change

It's only breaking users who for some reason used default-features = false while runtime-relying on strings being present in the messages. The API doesn't change by deactivating the feature, just behavior. And it's trivial to fix - just add alloc to the list of features.

Agreed on all counts. I'd be fine releasing this as a minor version.

I'll take full responsibility for this bug, I tried to see how it happened and I'd guess that I looked at the somewhat broken base58 and mistakenly thought units didn't implement  "alloc" properly as well. 100% guilty as charged, the units crate was correct then I broke it and left the now misleading "alloc" feature.
Will PR a fix and a release tracking PR.

Fixed my mistake in #2942 and backported in #2943. Please say if the process is not exactly as desired. (Note I created a new units-0.1.x branch for the release based of off the latest tagged release of units.)

@tcharding I think your PRs are confused. #2942's first commit is a dupe of this PR (this is a PR, not an issue) and its second commit appears to belong to #2943.
I think we should

ACK and merge this.
Backport this along with a "bump the version number" PR.
After the release, PR to master just bumping units/Cargo.toml and the lockfiles to the new version number.

The process here is kinda weird because we're "backporting" from master to the currently-released version.

Face palm, I totally just thought this was an issue - OMG.


Mpq editor loading different files than warcraft game

Similar issue to my last one, this time with a different map: https://file.io/UqzZZ37ZWsUO
If you look at files like items w3t, they were translated to english but when loading map in 1.28.5 wacraft it still shows all files in korean, so the way warcraft chooses which one is the latest version seems to differ from mpq editor

Hi, could you please re-upload the map? I might have time to look at it.

I think it was this one: https://file.io/NcyGDGJNSZa7

so the way warcraft chooses which one is the latest version seems to differ from mpq editor

I checked the files and they have both locale & platform set to zero. So there should not be any ambiguity as for what the game picks and what MPQ Editor picks. Could you give me a concrete example of a file that is picked differently?

I don't remember much about this case. It is highly possible that its just that the node library I am using is years old so the stormlib dll it must use is not replacing the files correctly, I just tried doing it manually with mpq editor and seems to replace the files fine. Unfortunately I'm in the middle of some rewriting in my translation tool so can't test it for a while. Will close for now and reopen if I find an issue after retesting. Sorry to bother and thanks as usual

Oookey. Well the truth is that there was an issue recently (like half-year ago) in StormLib that caused the game picking different files, based on file locale. It might have been the same case. Let me know if you find something.


SEO(backlog): Image URLs – give URLs with keywords for images

This is a big task. Please make sure that you’ve got enough capacities in the Team and that there are no other important priorities before starting.
Vroni (SEO consulting) said that the images’ URL are very important for SEO. Improve the image URL structure to get URLs containing the image keywords and, also, the file path where they are deposited. For example, the URL should look like this:

https://de.serlo.org/mathe/funktionen/funktionsbegriff.png

and not like this:

https://assets.serlo.org/legacy/56fe95af016c7_bc84b18113f67b5ff4f2443da8eee5accd5ea901.png

Mostly solved in the new frontend.


fix bug of reading line ending with \r\n

When lines were ended with \r\n, error occurred:
no data found in file test.csv

So I just delete the \r.

Woops!  I don't typically deal with windows files so I missed this. I'll add this, but could you also submit a test?

@mzimmerman  a simle test file could be easily generated with shell:'
echo -en "a,b\r\n1,2\r\n"

BTW, I wrote a CSV/TSV toolkit using your package. You may try it.
csvtk -- Another cross-platform, efficient, practical and pretty CSV/TSV toolkit in Golang
Example:
echo -en "a,b\r\n1111,23333\r\n" | csvtk pretty

So I'm not sure this was the issue.  There is a test for this already and I believe it is all handled by encoding/csv.
I also don't see that error "no data found in file" anywhere in multicorecsv.  Was that in your code?

A test case of blanking line of \r\n is added in #2 .
Looking back this pull request, deleting the \r character is the simplest and also a robust solution.
And another question, did the codes from line [multicorecsv.go#L203] (https://github.com/mzimmerman/multicorecsv/blob/master/multicorecsv.go#L203) to L217 were carefully written to handle Unicode?

I'd like to leave the handling of \r to the underlying encoding/csv as much as possible since it's already doing it.  However, there is the explicit case of the blank line to handle since the way multicore works it does end up having an issue.  So I've fixed it in a slightly different way in 350b5af71b665402277ff987abcf091e51f24b7e.
Yes, multicorecsv should handle Unicode just fine as I'm using the underlying standard library to read rune (proper Unicode support) (to my knowledge as least!)
I usually use column to do what your example of csvtk is doing, however I looked at csvtk and it's got some pretty advanced features!  I'll have to try it out, thanks!


Tidy up error message of interrupt signal for default 'fzf' chooser

Hello :wave:
I'm kind of just starting to use just & I think I like it. While I was just trying things out I see that this tool can do a "chooser" mode where the user is guided from interactive CLI, which I think is kind of nice
I have an improvement idea regarding some error message thrown during chooser mode that uses the default fzf command. Basically right now when I do this:
default:
    just --choose

ping:
    echo pong

It'll take me to the expected CLI menu

But when I ^C or ^D it it'll throw some (unexpected) error
just --choose
error: Chooser `fzf --multi --preview 'just --unstable --color always --justfile "/some/path/justfile" --show {}'` failed: exit status: 130
error: Recipe `default` failed on line 2 with exit code 130

I think since it is expected that exit code 130 to be ExitInterrupt, we can just use it to have better error message
The new error message would be
just --choose
error: Command was terminated by signal 130
error: Recipe `default` failed on line 2 with exit code 1

IMO better error message would be something along the line of just Command was interrupted without any signal code, since that signal is fzf-specific, but current ones convey the meaning just fine too

You can also put [no-exit-message] above your default: recipe to suppress that error.

TIL, though I don't know about whether it'll work for all usecases, since I don't see any problem with exit error messages itself, just that it could be tidied up

I don't think that special-casing this is worth it. It feels a bit hacky, for example it doesn't work it the user is using fzf as their chooser, but with a custom invocation. Also, I think you really don't want an error message at all if you ctrl-c fzf, since there's not really an error, per se, so I think [no-exit-message] is a better solution.


Waze

Bonjour , déjà merci pour ce logiciel qui est vraiment top !
J'aurai voulu savoir si il y aura un jour waze de disponible par cette application ? Merci

Unfortunately, AAIdrive is unable to embed other apps to be shown in the car. The existing map functionality is implemented by building a custom map view on an internal AAIdrive window that is shown in the car, which allows AAIdrive to send commands to the map and screenshot it without asking permission every time.
In comparison, the screen mirroring addon requires asking permission every connection, keeping the screen on, and can't send input to the other app. Android Auto itself has special access to certain apps, but those apps only give access to AA by specific package name and app signatures.


Bad performance when scroll up/down

Bug description:

When we use Webview to open a website, we just compare it with another browser, do you guys have any idea to improve performance?
To Reproduce:

Open any website and scroll up/down, especially on the android devices we can see it lagging too much
Expected behavior:

Screenshots/Videos:

Environment:

OS: any
OS version: any
react-native version: 0.66.4
react-native-webview version: 11.14.2

Still trying to solve this as well. Anyone have ideas?


[Feature request]: Limit neko rooms resources

I'd like to add some limitations to the containers like:
Running limits and constrains
Max amount of time for the room in minutes and associated action like
{
    "time_to_live": 60,
    "action": "delete"
}

Some comments about implementation:

cron job: https://www.airplane.dev/blog/creating-golang-cron-jobs
Storing room constrains as labels: https://github.com/Shopify/docker/blob/master/docs/userguide/labels-custom-metadata.md

There could be time to live specified as a duration, and we specify since when:

since room created
since room started
since last admin left
since last member left

Alternatively setting just fixed time could also be accepted, let's say we want to stop room at 17:00 every day.
Action could be:

delete room
stop room
pause room

Thinking about this, we could also implement starting the room based on time/action when we have this code in place.


Broken winget, yet again

Brief description of your issue
When i run winget install something the text of terminal turns blue and then nothing happens
Steps to reproduce

Open admin powershell
run winget install something or any command
outputs a / and text turns blue and nothing happens

Expected behavior
it should work like normal without just outputting color change?
Actual behavior
outputting blue color

Environment
Windows Package Manager v1.2.10691
Copyright (c) Microsoft Corporation. All rights reserved.
Windows: Windows.Desktop v10.0.22631.3085
Package: Microsoft.DesktopAppInstaller v1.17.10691.0

and Nope

@intninja - You need to update your version of WinGet to 1.6 or the 1.7-preview. The CDN serving the 1.2 version has been retired

#3652

Okay i had to check microsoft store to update the app installer, closing.


hotspot: 1.4.1 -> 1.5.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/KDAB/hotspot/releases.
meta.description for hotspot is: A GUI for Linux perf
meta.homepage for hotspot is: https://github.com/KDAB/hotspot
meta.changelog for hotspot is: https://github.com/KDAB/hotspot/releases/tag/v1.5.0
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.5.0 with grep in /nix/store/dpbmy4m8sdwvzjr67cjygbdr1yzz3qk4-hotspot-1.5.0
found 1.5.0 in filename of file in /nix/store/dpbmy4m8sdwvzjr67cjygbdr1yzz3qk4-hotspot-1.5.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

hotspot

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dpbmy4m8sdwvzjr67cjygbdr1yzz3qk4-hotspot-1.5.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A hotspot https://github.com/r-ryantm/nixpkgs/archive/208d6d30a8ee5eab548a720860f12345f4da9f75.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/208d6d30a8ee5eab548a720860f12345f4da9f75#hotspot

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dpbmy4m8sdwvzjr67cjygbdr1yzz3qk4-hotspot-1.5.0
ls -la /nix/store/dpbmy4m8sdwvzjr67cjygbdr1yzz3qk4-hotspot-1.5.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

hotspot

Maintainer pings
cc @nh2 for testing.

Already on master.


Workflow to automatically update submodule in snapshot repo

Suggestion: This workflow will automatically update the snapshot-plugins submodule in the snapshot repo whenever there are changes in the src directory or the package files.
Currently this won't run at all because it targets a branch named main which does not exist. But I suggest renaming master to develop (so it continues to be the repo's default branch) and adding a main branch.
Then we can work on develop and whenever we merge changes into main the submodule in the snapshot repo will be updated to the latest commit. Quick fixes or so can still be made directly on main then. They just should be merged back into develop.
Same PR for snapshot-spaces: https://github.com/snapshot-labs/snapshot-spaces/pull/1629
Related PR in snapshot repo: https://github.com/snapshot-labs/snapshot/pull/899
This will require to set a personal access token as a secret (UPDATE_SNAPSHOT_PAT) on repository or org level, with the public_repo scope.

If there's only the master branch and no intermediate step, I tend to say that updates should not happen fully automatically at all, at least for now. The workflows can still be a handy helper. So after updating the plugins repo let's say, instead of opening the snapshot repo in your IDE, pulling latest changes, running the git submodule command and pushing the update... you just do this:

We can then still decide later to run it on a schedule or just immediately on push.


ChromeCast questions

We are using Shakaplayer v2.0.5 on our web-app and we need to pass the "Google Cast Test Cases".
-Is there any way to show the volume-bar and the progress-bar in the "cast dialog" (cast minicontroller)?
-Is there any interface to show the "cast introduction overlay"?
Regards.

Shaka Player doesn't do any UI work.  It is up to the app to show things like volume-bar.  You can just add this to your receiver app html page.
Our demo page includes an example receiver app that shows an "introduction overlay" and the progress-bar.  It doesn't show the volume-bar, but that could be easily added.

Hi!
Thanks for your reply,  but I'm talking about the sender side.
This dialog:

And this overlay:

We don't know how to show the volume bar and the progress bar on the dialog...and we don't know what's the best way to detect the very first time that the user can cast using our web to show the overlay (using cookies may be? or if it is an API like in android)
Best regards.

The top dialog is from the Chrome extension.  The interface for that changed in the last few months.  We have no control over that.  Volume is primarily controlled on the cast side (e.g. on the TV), but Shaka Player also proxies volume over the app.  So if you adjust the volume bar in our demo app (on the sender side) it will change the Chromecast volume.
The overlay in the bottom screenshot is likely created by that app.  They probably did this with a static image or with some CSS.  One way to detect the first time someone visits your app is to use local storage.

Thanks a lot.
I've noticed that the requeriments have changed at January 31 and the dialog bars are not required anymore.
https://developers.google.com/cast/docs/design_checklist/sender
Best regards.


Missing ExternalDependency in Syntax block of page

The child win32Dependencies:ExternalDependency is missing from the Syntax block on this page (it is mentioned later).  The maximum number of these appears to be 1000 in the foundation schema.
Also, it looks like the limit for PackageDependency is 129, not 128.

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

ID: 255711d5-f7d0-6cd5-a3b2-d97e6217ff5f
Version Independent ID: e073be81-7b37-e32b-4dbc-fae8271886a8
Content: Dependencies (Windows 10) - Windows UWP applications
Content Source: winrt-related-src/schemas/appxpackage/uapmanifestschema/element-dependencies.md
Product: uwp
Technology: winrt-reference
GitHub Login: @drewbatgit
Microsoft Alias: drewbat

@TimMangan With this small comment about 129 vs 128, you have pulled a thread that has unraveled to reveal some larger issues. You are right that the limit is 129, but apparently that is a bug. Some other, unrelated issues were also discovered with this ExternalDependency element, and a workstream has been spun up to address these issues. So, for now, I have added the element to the syntax block. When the internal resolution for the element count issue is resolved, I will update the page as appropriate. Thanks for keeping an eye out for problems in the docs.

@TimMangan The feature team also wanted me to pass along their thanks to you for catching this. The limit will be changed back to 128, as apparently that is the actual limit enforced by the OS. The schema was changed to 129 in error and will be reverted.


Create Thoth.toml

Description
If this is an UPDATE for a previous genesis validator
Checks are made against your net_address. If this has changed since the previous testnet, make sure you provide links of previous prs merged from your previous git username.
Thanks in advance!
Checklist before merging to draft

[x] Only one toml is added in this PR
[x] The file being added is indeed a .toml file
[x] The toml being added is to the correct folder, and only to the correct folder
[x] The eth_hot_key and eth_cold_key are present

please create Create Thoth.toml pre-genesis validator


RETURN_REVERT: distinguish return or revert opcode

Describe the feature you would like
refer to comments https://github.com/scroll-tech/zkevm-circuits/pull/633#discussion_r1273524994
Additional context
No response

seems feasible by checking revert with rw_counter_end_of_reversion state like in https://github.com/privacy-scaling-explorations/zkevm-circuits/pull/1030

See also section E in this spec.


Seperate C & C++ & rename C-SHARP to C# for consistency

I looked through the C_C++.md file, went through the github repos of each project, and added each C & C++ project into their own file. This is in response to issue #42.
I also noticed the C-SHARP file didn't use the # symbol or have a heading in the file. I went ahead and added those so the files would be sorted alphabetically in the directory and C-SHARP wasn't the first file above C.
I made sure to update the README to include the new links.

I noticed some of the links point to the project website like https://bitcoin.org instead of the project repository https://github.com/bitcoin/bitcoin
Any particular reason for this?

@eugenetriguba It's incorrect format.
Should be:
[name](http://link-to-repo.com/) - descr


Temporarily disable release build job

Signed-off-by: Jay Guo<EMAIL_ADDRESS>

duplicated by #141

duplicated by #141


Checkout/add/commit/push race condition

We sometimes have seen things like this:
 ! [rejected]        HEAD -> master (fetch first)
error: failed to push some refs to 'git@sysgit.yelpcorp.com:puppet'
hint: Updates were rejected because the remote contains work that you do
hint: not have locally. This is usually caused by another repository pushing
hint: to the same ref. You may want to first integrate the remote changes
hint: (e.g., 'git pull ...') before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.

@matthewbentley suggested it could be a race condition. i.e. if new commits are pushed between the checkout and commit/push changes in this plugin then you would expect git to sometimes reject the changes.

P.S. The temporary solution is to manually pull the repo in question and try again.

Manually pull and merge, that is

Sure, although git pull implies a merge (it actually does git fetch and git merge under the hood)


added no_footer var for external sites

Very similar to #1273, just removes the footer when not needed.

:+1:


Change build.xml to use recommended Composer settings

Need to update build.xml to the following.
When our CI systems are building, they should then use ni_build flags.
(ni_build does things like create the silverstripe-cache folder as well)
<property name="composer.dev" value="" />
<property name="composer.source" value="--prefer-source" />
<if>
	<isset property="composer_dist" />
	<then>
		<property name="composer.source" value="--prefer-dist" override="true" />
	</then>
</if>
<if>
	<!-- Non-Interactive Build, ie. For CI -->
	<isset property="ni_build" />
	<then>
		<property name="composer.source" value="--prefer-dist --no-interaction --no-progress --no-suggest --optimize-autoloader --verbose --profile" override="true" />
	</then>
</if>

@silbinarywolf, if you're across the solution here, could you please put through a PR? I assume this will also require some testing/changes on the CI side of things.

@nglasl It will require changes on the CI side, but this wont break anything if our CI continues to use the composer_dist flag instead of ni_build

PR incoming.

@nglasl See PR above.

The PR above has been merged, so closing this issue. Thanks :+1:


[ConstExpr] implement flow-sensitive array initialization support

This requires several piece, including support for allocating uninit
arrays,support for pointer_to_address, index_addr, and updating
the middle of array values.
This is enough to get constexpr to handle "default argument #6" of the
Tensor.conv2d method, which does array stuff.
LowerGraph still doesn't support array attributes - upgrade the
llvm_unreachable that crashes to a generated error, so we get nicer
behavior in the testsuite when these are hit.  I'll tackle array arguments
next.
This fixes SR-8300

@swift-ci please test tensorflow

Merging this to unblock progress, I'm happy to incorporate improvements into follow-on patches!


add autoformatting when commiting code

I managed to push (working) code with broken formatting, I only noticed this when the CI/CD decided to tell me.
Could/should be fixed with husky. I think this used to be the case but I am assuming port to typescript broke this.
Currently needs to run npm run format before commiting.

Currently the package.json has the following:
	"lint-staged": {
		"*.{js}": [
			"eslint --fix",
			"git add"
		],
		"*.{js, json, yml, md}": [
			"prettier --write",
			"git add"
		]
	}

Seems to be that it doesn't work because it searching for js files instead of ts.

Having issues with this. Seems not to run even when I added thew .ts file extension there.
Investigating further


[Possible BUG] - Did DBeaver lost its "smart home key" funcionality?

Is it me, or hitting the "HOME" key used to go to the first character of the line, and now it goes to the begginning of line? :-S

Hi @wiwwo ,
what version do you use?
In 7.0.1 it works ok for me - hitting Home button still used to go to the first character

7.0.1
Linux Mate 18.04.

Same here with 7.0.2 on Windows x64

Hm, reproduced for me after default formatter changing. Also reset UI settings helps me.
btw, @wiwwo , @Physikuss what formaters do you use?

"I don't know"
which spells
"The default one"
:-D ;-)

A bit late, but I was also using the default formatter.

Can't reproduce. Home always navigates to the first non-whitespace character (spaces, tabs).
You can enable "Show whitespace characters" in text editor preferences to see what is actually in the beginning of the line.

@wiwwo , @Physikuss Did Serge's advice help you?

there is no update on issue for a long time. Probably, it is solved.
If it is still actual for you - feel free to reopen the ticket


Synchronize and properly setup rounds

Synced round timer on all clients
Correct setup of scoreboards at end of round
Synced scoreboard animation at end of round

Fixed in #660 by doing most of it locally :)


"part" in lib and "part of" in bin breaks analyzer

Dart SDK version: 3.1.0-246.0.dev (dev) (Fri Jun 23 01:04:39 2023 -0700) on "macos_arm64"
How to reproduce issue:
dart create foo
in foo/lib/foo.dart:
library foo;

part '../bin/foo.dart';

int x = 3;

in foo/bin/foo.dart:
part of '../lib/foo.dart';

void main(List<String> arguments) {
  print(x);
}

remove the test directory.
dart analyze:
Analyzing foo...              0.5s

  error • bin/foo.dart:4:9 • Undefined name 'x'. Try correcting the name to one that is defined, or defining the name. • undefined_identifier
  error • lib/foo.dart:3:6 • Target of URI doesn't exist: 'package:foo/bin/foo.dart'. Try creating the file referenced by the URI, or try using a URI for a file that
          does exist. • uri_does_not_exist

2 issues found.

dart run:
Building package executable...
Built foo:foo.
3

First: don't do that.
I'm not sure anyone is wrong, they're just helping you in different ways.
The analyser is giving the library file the URI package:foo/foo.dart. That's reasonable, and prevents a lot of problems.
Then it tries to resolve ../bin/foo.dart against that package URI, giving package:foo/bin/foo.dart, because that's how a relative path is resolved against a package URI.
That file doesn't exist.
Resolving in the other direction is actually fine, because no package URI is involved, it's all local files. I'm guessing the analyzer fails that because it only gives one URI to each file, or because it helps you avoid file paths into lib/.
The dart run is actually wrong, because part files are not valid dart scripts. You should only be allowed to run a library file.

Cf. https://github.com/dart-lang/sdk/issues/52575


Refresh snackbar displayed up twice when refreshed

Is it a ?

[x] Bug Report

[ ] Feature Request

[ ] Chore

Bug Report
Actual Behaviour
When refreshed the "Refresh Complete" message pops up twice
Expected Behaviour
It should be displayed only once
GIF of the issue

Would you like to work on the issue?
YES

Hi @simarsingh24
I am not able to reproduce the issue. Is it still happening?

@mishuvs i am busy with my midsems, it may be possible that the issue has been resolved

@simarsingh24 Oh! I just noticed that the issue is not yet resolved. Don't know why it worked fine yesterday.
Are you working on it?

@mishuvs @simarsingh24 Status?

I will comment when a PR is ready.


Logging in to Azure: Exception.Message

Hi,
When I follow the very first step, once I clone the repo using Git it throws me the following error:
[ 4/21/2021 8:36:20 AM ] - [INFO] - Logging in to Azure
[ 4/21/2021 8:36:20 AM ] - [ERROR] - Exception.Message [ The term 'az' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again. ]
[ 4/21/2021 8:36:20 AM ] - [ERROR] - Exception.Type    [ System.Management.Automation.CommandNotFoundException ]
[ 4/21/2021 8:36:20 AM ] - [ERROR] - Error occurred while executing the Script. Please report the bug on Github (along with Error Message & Logs)
Then it throws me an error:

@Carsiemsft so to run the install you need the Azure Command Line Extension tool installed see  Instructions at: https://docs.microsoft.com/cli/azure/install-azure-cli-windows?tabs=azure-cli for instructions.
Once you have this install the AZ cmd link will work.

Thanks so much, it worked!!! :)


Sort ProjectReferences alphabetically

This address small nit picking from #2333

/cc @ahsonkhan

@dotnet-bot test this please

@ahsonkhan is I also possible to ask bot to do some tasks, or this is MSFT only?

@ahsonkhan is I also possible to ask bot to do some tasks, or this is MSFT only?

I believe any collaborator can do it. Try dotnet-bot help.


fast-downward: 22.06.1 -> 22.12.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/aibasel/downward/releases.
meta.description for fast-downward is: "A domain-independent planning system"
meta.homepage for fast-downward is: "https://www.fast-downward.org/"
meta.changelog for fast-downward is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 22.12.0 with grep in /nix/store/64p5b2v2a4qj6j654yx7fi8cmpmzwm3d-fast-downward-22.12.0
found 22.12.0 in filename of file in /nix/store/64p5b2v2a4qj6j654yx7fi8cmpmzwm3d-fast-downward-22.12.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
fast-downward

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/64p5b2v2a4qj6j654yx7fi8cmpmzwm3d-fast-downward-22.12.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A fast-downward https://github.com/r-ryantm/nixpkgs/archive/470662f51b04519cb0d54eac378d22f3c48668fe.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/64p5b2v2a4qj6j654yx7fi8cmpmzwm3d-fast-downward-22.12.0
ls -la /nix/store/64p5b2v2a4qj6j654yx7fi8cmpmzwm3d-fast-downward-22.12.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

fast-downward

Maintainer pings
cc @abbradar for testing.

@ofborg build fast-downward fast-downward.passthru.tests


Feature request: Provide page's language in ParseResult

For further processing of websites it may be important to know the language of the page. For example to display text justified in the browser, specifying the language via <html lang="de"> is necessary.
Unfortunately at the moment mercury does not forward the language information from the input document to the ParseResult.
It would be great if the ParseResult could be extended by a language property which contains the value of the <html>-element's lang-attribute.

Hi @svenwiegand , thanks for your feedback.
A quick and easy solution you may want to try is to add a custom type extension to your parse call.
Example via CLI
mercury-parser https://postlight.com/trackchanges/mercury-goes-open-source --extend lang="html|lang"
Result
{
  "title": "Mercury Goes Open Source! — Postlight — Digital Product Studio",
  "author": "Adam Pash",
  "date_published": "2019-02-06T14:36:45.000Z",
  "dek": null,
  "lead_image_url": "https://postlight.com/wp-content/uploads/2019/02/mercury-open-source-social-card-e1550670446269.png",
  "content": "...content",
  "next_page_url": null,
  "url": "https://postlight.com/trackchanges/mercury-goes-open-source",
  "domain": "postlight.com",
  "excerpt": "It’s my pleasure to announce that today, Postlight is open-sourcing the Mercury Web Parser. Written in JavaScript and running on both Node and in the ...",
  "word_count": 436,
  "direction": "ltr",
  "total_pages": 1,
  "rendered_pages": 1,
  "lang": "en-US"
}

See https://github.com/postlight/mercury-parser/tree/master/src/extractors/custom#custom-types for more information on adding additional fields to the response.

Is there a way to do this without writing a custom extractor?


[WIP] Use ORFS from docker image

This is still a work in progress, please do not merge.
This PR modifies the existing rules to run physical design flow in a docker container with installed OpenROAD-flow-scripts.
It uses rules_oci to import and load external docker image into local docker runtime.
The solution is based on generating config.mk files for each design (once per build_openroad()) with new macro called write_config(). The config consist of common and design-specific environment variables, as well as definitions of all rules for make targets used in bazel-orfs.
The config and other inputs to the flow are passed down to the docker container using bind mount mechanism.
Currently it is required to disable bazel's sandboxing feature in order to provide correct paths for mounting directories with input files for the flow.
The flow is started with an entrypoint script based on docker_shell.sh utility imported from OpenROAD-flow-scripts.

@lpawelcz I realize this is WIP, but thought I would try it quickly. Removed util.tcl references in bazel-orfs and got this far:
$ bazel build L1MetadataArray_test_generate_abstract
INFO: Invocation ID: 7fa43090-2474-4138-ac1f-4d75ce72668d
ERROR: error loading package '@@bazel-orfs//': Unable to find package for @@rules_oci//oci:defs.bzl: The repository '@@rules_oci' could not be resolved: Repository '@@rules_oci' is not defined.
ERROR: /home/oyvind/megaboom/BUILD.bazel:936:15: error loading package '@@bazel-orfs//': Unable to find package for @@rules_oci//oci:defs.bzl: The repository '@@rules_oci' could not be resolved: Repository '@@rules_oci' is not defined. and referenced by '//:L1MetadataArray_test_generate_abstract'
ERROR: Analysis of target '//:L1MetadataArray_test_generate_abstract' failed; build aborted: Analysis failed
INFO: Elapsed time: 0.262s, Critical Path: 0.00s
INFO: 1 process: 1 internal.
ERROR: Build did NOT complete successfully

Removed util.tcl references in bazel-orfs and got this far

For the time being docker flow is supported and tested only in bazel-orfs repo. If you want to test this there you must first run bazel run orfs_env in order to load the bazel image into your docker runtime. Then you can run e.g. L1MetadataArray targets (expect failures in cts and further stages).

we can squash these numerous commits into a single one or a few

Sure. For now this is WIP. At this stage I prefer to keep small commits in case something has to be reverted or slightly modified. I will surely clean the history before putting up this work for a review.

Reworked TCL and constraints handling, cleaned up the code responsible for flow arguments, source and outputs specification.
Added generating configs per stage (those are included in config generated per design by new env var STAGE_CONFIG).
Fixed the local flow after adding docker flow.
Enabled CI tests for docker flow and reworked local flow.
The plan for now is to add modified version of L1MetadataArray with enabled grt and final stages. Then we can work on https://github.com/Pinata-Consulting/ascenium-antmicro/issues/15.
Please note that those changes were not yet tested in megaboom and we expect some failures there.
Once we have grt and final stages tested here we will start integrating and testing the changes in megaboom workspace

@lpawelcz I need an accompanying megaboom pull request...

@oharboe ready, here is the megaboom PR: https://github.com/The-OpenROAD-Project/megaboom/pull/29

Continue testing and file github issues based upon master.


add a script to bulk update certain fields

use case: someone has a CSV of Twitter Accounts or IDs that they can tie to our IDs, but manually editing each YAML file would be tedious/error prone
probably the best way to do this would be to take a CSV as input that updates all of the values

while updating legacy_openstates IDs I added the rough outline of this work in manual_data.py


Additional outcomes/statistics

Outcome/Feedback - if possible adding the following outcome parameters/statistics:

[ ] Peak velocity and peak acceleration
[ ] Statistics at peak velocity: movement time, total time, movement distance, RMSE movement, and directional error (difference between the ideal trajectory and the real trajectory) at peak velocity
[ ] Hand path area (only possible when the option “automatically move cursor to center” is deselected --> i.e. participants have to do out and back movements): the area enclosed by the hand path (i.e. out and back movements)
[ ] Normalized hand path area : hand path area divided by the squared movement length
[ ] Spatial error: the linear distance from the movement end point and the center of the target

Spatial error: at timeout linear distance from cursor to target minus target radius

First step:

Implement some/all of these additional statistics in a Jupyter notebook starting with exported data from a trial
Ideally this notebook would also include the existing statistics that are calculated & exported

These would be both tests of the existing implemented statistics and documentation/examples for users to start from when doing their own analysis

Next step:

Add some/all of these to the statistics to those that are calculated, displayed and exported by the software
Probably also need to restructure how the statistics are chosen & displayed for this
Add definitions/description of each statistic to the documentation

More possible stats:

Angle at peak velocity (directional error at peak velocity): The angle at peak cursor velocity compared to the target/center using vectors (e.g. https://www.geeksforgeeks.org/angle-between-two-vectors-formula/).
For statistics to target: the angle between the vector A including the center of the central target to the position of the cursor at peak velocity and vector B including the center of the central target to the center of the outer target.
For statistics to center: the angle between vector A including the center of the outer target to the position of the cursor at peak velocity and vector B including the center of the central target to the center of the outer target.
Error at movement reversal (spatial error):
For statistics to target: the shortest distance (scalar distance) of the outer cursor reversal point (i.e. movement reversal/endpoint) from the center of the outer target.
For statistics to center: the shortest distance (scalar distance) of the inner cursor reversal point (i.e. movement reversal/endpoint) from the central target.
The outer and inner movement reversal points could be calculated using the minimal velocity of the cursor movement after peak cursor velocity (according to the above-mentioned new definition of t(final))
An option that can be selected in the statistics for smoothing and filtering data. These techniques shouldn’t be applied online (i.e. during task execution), but for the calculation of the statistics (especially for the statistics export). For example, a cubic spline data interpolation would be a good option for data smoothing and a low pass filter, for which the frequency/cutoff can be entered (e.g. in Hz), for data filtering.


Problem loading javassist because of FML

Having javassist (http://www.csg.ci.i.u-tokyo.ac.jp/~chiba/javassist/) in a plugin cause this stacktrace to happen.
This library is required by Hikari-CP, a JDBC connection pool library (https://github.com/brettwooldridge/HikariCP). It prevents the ClassPool class to be loaded.
How to reproduce:

Get this plugin
https://www.dropbox.com/s/ulld11xnuj4ut7f/craftconomy3-3.2.2-SNAPSHOT.jar?dl=0
Put it in the mods folder then launch the server, the stacktrace will be during startup.

[22:58:50] [Server thread/ERROR] [FML/]: There was a problem reading the entry com/greatmancode/javassist/ClassPool.class in the jar C:\Users\William\Desktop\forgeserver\mods\craftconomy3-3.2.2-SNAPSHOT.jar - probably a corrupt zip
net.minecraftforge.fml.common.LoaderException: java.lang.ArrayIndexOutOfBoundsException: 113

	at net.minecraftforge.fml.common.discovery.asm.ASMModParser.<init>(ASMModParser.java:58) ~[ASMModParser.class:?]
	at net.minecraftforge.fml.common.discovery.JarDiscoverer.discover(JarDiscoverer.java:74) [JarDiscoverer.class:?]
	at net.minecraftforge.fml.common.discovery.ContainerType.findMods(ContainerType.java:42) [ContainerType.class:?]
	at net.minecraftforge.fml.common.discovery.ModCandidate.explore(ModCandidate.java:71) [ModCandidate.class:?]
	at net.minecraftforge.fml.common.discovery.ModDiscoverer.identifyMods(ModDiscoverer.java:131) [ModDiscoverer.class:?]
	at net.minecraftforge.fml.common.Loader.identifyMods(Loader.java:348) [Loader.class:?]
	at net.minecraftforge.fml.common.Loader.loadMods(Loader.java:469) [Loader.class:?]
	at net.minecraftforge.fml.server.FMLServerHandler.beginServerLoading(FMLServerHandler.java:87) [FMLServerHandler.class:?]
	at net.minecraftforge.fml.common.FMLCommonHandler.onServerStart(FMLCommonHandler.java:355) [FMLCommonHandler.class:?]
	at net.minecraft.server.dedicated.DedicatedServer.func_71197_b(DedicatedServer.java:117) [po.class:?]
	at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:438) [MinecraftServer.class:?]
	at java.lang.Thread.run(Unknown Source) [?:1.8.0_45]
Caused by: java.lang.ArrayIndexOutOfBoundsException: 113
	at org.objectweb.asm.ClassReader.a(Unknown Source) ~[asm-all-5.0.3.jar:5.0.3]
	at org.objectweb.asm.ClassReader.readUTF8(Unknown Source) ~[asm-all-5.0.3.jar:5.0.3]
	at org.objectweb.asm.ClassReader.b(Unknown Source) ~[asm-all-5.0.3.jar:5.0.3]
	at org.objectweb.asm.ClassReader.accept(Unknown Source) ~[asm-all-5.0.3.jar:5.0.3]
	at org.objectweb.asm.ClassReader.accept(Unknown Source) ~[asm-all-5.0.3.jar:5.0.3]
	at net.minecraftforge.fml.common.discovery.asm.ASMModParser.<init>(ASMModParser.java:53) ~[ASMModParser.class:?]
	... 11 more

HikariCP is already included in Sponge -- this seems like a classloading issue that is legitimate, but any particular reason you aren't using Sponge's SqlService?

@zml2008 Multi-server support in the same .jar basicly. Hikari isn't in Bukkit/Canary

ah, makes sense

@greatman: This appears to be a problem with the format of the class file, or possibly a bug in ASM.
Can you try rebuilding the plugin?

the javap output of ClassPool.class (put here for record purposes)
https://gist.github.com/greatman/a6dc235adfc842f94c0c

Looks like relocate of the gradle shadow plugin played with the library compilation. Once I commented my relocate line, it started to work fine.

@greatman
Is this still an issue?

@Zidane, didn't retry yet with the relocate again. Aaron said it was working for him, i'll need to check it out.

Closing, as this was a issue with shading javassist


enhancement: Display Menu Icon based on prop being passed

Display Menu Icon based on prop being passed

Closing this issue since it is no longer valid since it is resolved.


Add Peak Flow areas

Add a new map for the X number of 'hot spot areas'
Do this based on a % Above average

complete with map_flood_risk() function


dropdown touch troubles

the dropdown closes already with the touchstart event, a click-eventhandler for the items triggers on the wrong element since the close-animation is already running.
Expected Behavior

dropdown closes only after the click is completed.

Possible Solution

change the handler from touchstart to touchend or rely on the click event also for touch

Fixed in ce5c727d1beeccf1d320f878ac3181bd63135778


skipLibCheck doesn't work with typesVersions

Bug Report

🔎 Search Terms

skipLibCheck, typesVersions
🕗 Version & Regression Information

"typescript": "^4.9.4"
Reproduce:

git clone https://github.com/tjx666/ts-exports-development
git reset --hard d0eb57cdf0c1f8cdb5260dc2f6f9083479046d50
pnpm install
cd packages/app
tsc --noEmit

⏯ Playground Link

Reproduce Repo
💻 Code

monorepo project structure:
.
├── node_modules
│   ├── tsup -> .pnpm/tsup<EMAIL_ADDRESS>│   └── typescript -><EMAIL_ADDRESS>├── package.json
├── packages
│   ├── app
│   │   ├── node_modules
│   │   │   └── tools -> ../../tools
│   │   ├── package.json
│   │   ├── src
│   │   │   └── index.ts
│   │   └── tsconfig.json
│   └── tools
│       ├── dist
│       │   ├── index.js
│       │   └── index.mjs
│       ├── package.json
│       ├── src
│       │   └── index.ts
│       └── tsconfig.json
├── pnpm-lock.yaml
└── pnpm-workspace.yaml

In packages/app/src/index.ts
import { add } from 'tools';

console.log(add(2, 4));

In packages/tools/src/index.ts
export function add(a: number, b: number) {
    return a + b;
}

let a = 1;
// here error
a = '';

the package.json of package tools
{
    "name": "tools",
    "version": "1.0.0",
    "description": "",
    "scripts": {
        "build": "tsup src/index.ts --format esm,cjs"
    },
    "keywords": [],
    "author": "",
    "license": "ISC",
    "exports": {
        ".": {
            "import": {
                "development": "./src",
                "default": "./dist/index.mjs"
            },
            "require": {
                "development": "./src",
                "default": "./dist/index.js"
            }
        }
    },
    "typesVersions": {
        "*": {
            "*": [
                "./src/*"
            ]
        }
    }
}

🙁 Actual behavior
When run tsc --no emit in packages/app, will check the ts file in tools/src
🙂 Expected behavior
Not check the ts file in tools/src

"lib" in "skip lib check" means "a .d.ts file". It doesn't mean "something that came in via node_modules" or other more nuanced definition.

"lib" in "skip lib check" means "a .d.ts file". It doesn't mean "something that came in via node_modules" or other more nuanced definition.

So, any way to skip check the ts files?

I points the typesVersions to src, is this the correct usage？Maybe I should generate the .d.ts, and using the types field?

The preferred behavior is always for a unit of compilation to expose its .d.ts files rather than its source

On my side my storybook package was looking for types specific to another workspace package. I tried tons of "solutions" but none worked.
The only one working was to tell my package to look for types of the other package:
{
  ...
  "include": ["**/*.ts", "**/*.tsx", "../../apps/main/@types"],
  "exclude": ["node_modules"]
}

Hope this helps :)


Added demo template

This adds a simple demo template that will take in some input and trigger some actions.
Inputs
Gather some basic inputs:

Your Name (text)
If you like Dark or Light mode (radio)
What Software Template topic you'd like to learn more about (multiple-choice)
Ask for a GitHub Owner (user must provide auth)
Ask for a GitHub Repo Name (user must provide auth)

Actions
Then take the following actions:

Create a repo (using user auth)
Populate it with a simple HTML and CSS landing page
Enable GitHub Pages (using user auth)
Trigger GitHub Action Workflow to deploy landing page (using user auth)

Demo Video
https://github.com/backstage/demo/assets/67169551/40187f19-44a2-4e45-8248-3604f9923715
Custom Action
I needed to add a custom action to enable GitHub Pages for this, it's pretty simple and might be a good example for the Community to see.

Added a check to make sure the token was provided if not set it to false which will cause it to fail


feat(neo4j): Allow datahub to connect to specific neo4j database

By default datahub connects to default db when neo4j is being used for Graph service implementation. With this PR, we can provide specific database in neo4j to connect to.
Also, changes default value of graph_service_impl in docker quickstart files
Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Thanks! If we can restore the default and allow overriding of the graph impl, then this looks good to me!

@david-leifker
task python generate_docker_quickstart.py check-all was failing, so I made changes in docker-compose.override.yml and run task python3 generate_docker_quickstart.py generate-all on my local, which reorders env variables in files docker-compose.quickstart.yml and docker-compose-m1.quickstart.yml

LGTM! Thanks!


[Bug] fix permission issue for install.sh

Description
Uses subprocess.run compared to subprocess.check_call to run the
install.sh script.
Subprocess.run is newer and recommended, and we already use
subprocess.run to execute the integtest.sh.
Signed-off-by: Kawika Avilla<EMAIL_ADDRESS>Issues Resolved
https://github.com/opensearch-project/opensearch-build/issues/764
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

I believe subprocess.run swallows the error, doesn't it?

I removed all permissions from the file got the error message for assembling but when executing with subprocess.run it ran and I verified the bundled tarball and had the expected files.

The new install.sh needs to be chmod 755 and that's it, no?

If so, where should that be located? in the script finder? Should we modify the integ test script finder as well?


Strike through Style and NSTextAttachment support

Can you support NSAttributedStringKey.strikethroughStyle and NSTextAttachment with UIImage when use addAttributedText with NSAttributedString.
It's not support now. Thank you ^ ^

NSTextAttachment can't be supported as this has been an open issue before in #32
At this moment I do not know why NSAttributedStringKey.strikethroughStyle as the attributed string rendering is handled by UIKit/CoreGraphics.

The PDFAttributedStringObject.draw method draws the attributed string using the following CoreText methods:
        // Get current graphics context
        let currentContext = UIGraphicsGetCurrentContext()!

        // Create a core text frame setter
        let framesetter = CTFramesetterCreateWithAttributedString(attributedString)

        // Save context pre manipulation
        currentContext.saveGState()

        // Reset text matrix, so no text scaling is affected
        currentContext.textMatrix = CGAffineTransform.identity

        // Create the frame and a rectangular path of the text frame
        let frameRect = CGRect(x: 0, y: 0, width: frame.width, height: frame.height)
        let framePath = UIBezierPath(rect: frameRect).cgPath

        // Create core text frame for the given attributed string
        // The whole text should fit the frame, as calculations were already done
        let frameRef = CTFramesetterCreateFrame(framesetter, CFRangeMake(0, attributedString.length), framePath, nil)

        // Translate by 100% graphics height up and reverse scale, as core text does draw from bottom up and not from top down
        currentContext.translateBy(x: 0, y: UIGraphicsGetPDFContextBounds().height)
        currentContext.scaleBy(x: 1.0, y: -1.0)

        // Translate context to actual position of text
        currentContext.translateBy(x: frame.minX, y: UIGraphicsGetPDFContextBounds().height - frame.maxY)

        // Draw text into context
        CTFrameDraw(frameRef, currentContext)

        // Restore context to pre manipulation
        currentContext.restoreGState()

I do not know why the strike through is not drawn, as this should be handled by CoreText CTFrameDraw. Any help is appreciated!

I found this article  which states that strike-throughs are not handled by Core Text but it can be implemented if you calculate the frame of the glyphs.
As i do not have enough time to test this on my own, can anybody take this over?


gflang has .mod file conflict issue.

output log:
F90-F-0004-Corrupt or Old Module file ./globalv.mod (fortran.f90: 16)
F90/aarch64 Linux Flang - 1.5 2017-05-01: compilation aborted
ERROR:Failed to build bitcode file for fortran.f90 because: exit status 1
objcopy: st1PvZpd: Failed to find link section for section 17
objcopy: st1PvZpd: Failed to find link section for section 17
shared/compiler.go
wg.Add(2) go execCompile(compilerExecName, pr, &wg, &ok) go buildAndAttachBitcode(compilerExecName, pr, &bcObjLinks, &newObjectFiles, &wg) wg.Wait()
when a fortran source file has module,  both go execCompile and go buildAndAttachBitcode will try to generate .mod file that has the same name which become a conflict.
here is the patch
		 if compiler == "flang" { 			wg.Add(1) 			go execCompile(compilerExecName, pr, &wg, &ok) 			wg.Wait() 			wg.Add(1) 			go buildAndAttachBitcode(compilerExecName, pr, &bcObjLinks, &newObjectFiles, &wg) 			wg.Wait() 		} else { 			wg.Add(2) 			go execCompile(compilerExecName, pr, &wg, &ok) 			go buildAndAttachBitcode(compilerExecName, pr, &bcObjLinks, &newObjectFiles, &wg) 			wg.Wait() 		}

Thanks. Could you actually do a (tested) pull request? I have never used flang.

https://github.com/SRI-CSL/gllvm/pull/65
It has been tested in my project and works fine.


No encoders were found, again.

Hello.
I attempted to run the following command:
LD_LIBRARY_PATH=~/.local/lib64 ~/.local/bin/openssl genpkey -algorithm p521_kyber1024 -outform PEM -out ~/oqs_key.pem -provider-path ~/src/git/oqs-provider/_build/oqsprov  -provider default -provider oqsprovider

but got the following error:
Error writing key
4097678B1E7F0000:error:1D800065:ENCODER routines:OSSL_ENCODER_to_bio:reason(101):crypto/encode_decode/encoder_lib.c:55:No encoders were found. For standard encoders you need at least one of the default or base providers available. Did you forget to load them?
4097678B1E7F0000:error:04800073:PEM routines:do_pk8pkey:error converting private key:crypto/pem/pem_pk8.c:133:

This appears be #25, but that issue is closed, and apparently encoders are supposed to be implemented already. Can you please tell me if I am, perhaps, doing something incorrectly?
Thank you.

My OS is Manjaro Linux, if that makes any difference.

Can you please tell me if I am, perhaps, doing something incorrectly?

You are using a key type not meant for storage in PKCS#8: p521_kyber1024 is a KEM algorithm: Those algorithms cannot be used for (persistent) sign/verify and consequently, do not (need to) have (persistence-generating) encoder/decoder functions.
If you use a signature algorithm, say p521_dilithium5, the command works OK. For an overview about all supported signature algorithms, see e.g. here or run openssl list -signature-algorithms -provider-path _build/oqsprov -provider oqsprovider.

Clsoing due to no activity for a week: Assuming issue is resolved. Please re-open if this is not the case.

I am not attempting to sign. I wish to encrypt a file.
It is most sad that I cannot do this.


This pr is to add License of script_gen_helper.go

Ⅰ. Describe what this PR does
This pr is to add License of pkg/application/inject/fuse/poststart/script_gen_helper.go
Ⅱ. Does this pull request fix one issue?

fixes #XXXX
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

/lgtm


Improve Documentation

Using DocumentUp, improve the look and feel of qikify documentation. This will require first generating Github Pages.

Okay, created a default gh-pages using automatic pages generation, can be found here http://trela.github.com/qikify/


Configuration object creation (similar to powershell)

There is small but growing divide the between capability of AZ CLI and Azure Powershell.
Of particular note is object creation in Powershell. In Powershell, a complete configuration object can be created using multiple commands. Example ($gw):
# Get the application gateway
$gw = Get-AzureRmApplicationGateway -Name AdatumAppGateway -ResourceGroupName AdatumAppGatewayRG

# Get the existing HTTPS listener
$httpslistener = Get-AzureRmApplicationGatewayHttpListener -Name appgatewayhttplistener -ApplicationGateway $gw

# Get the existing front end IP configuration
$fipconfig = Get-AzureRmApplicationGatewayFrontendIPConfig -Name appgatewayfrontendip -ApplicationGateway $gw

# Add a new front end port to support HTTP traffic
Add-AzureRmApplicationGatewayFrontendPort -Name appGatewayFrontendPort2  -Port 80 -ApplicationGateway $gw

# Get the recently created port
$fp = Get-AzureRmApplicationGatewayFrontendPort -Name appGatewayFrontendPort2 -ApplicationGateway $gw

# Create a new HTTP listener using the port created earlier
Add-AzureRmApplicationGatewayHttpListener -Name appgatewayhttplistener2  -Protocol Http -FrontendPort $fp -FrontendIPConfiguration $fipconfig -ApplicationGateway $gw

# Get the new listener
$listener = Get-AzureRmApplicationGatewayHttpListener -Name appgatewayhttplistener2 -ApplicationGateway $gw

# Add a redirection configuration using a permanent redirect and targeting the existing listener
Add-AzureRmApplicationGatewayRedirectConfiguration -Name redirectHttptoHttps -RedirectType Permanent -TargetListener $httpslistener -IncludePath $true -IncludeQueryString $true -ApplicationGateway $gw

# Get the redirect configuration
$redirectconfig = Get-AzureRmApplicationGatewayRedirectConfiguration -Name redirectHttptoHttps -ApplicationGateway $gw

# Add a new rule to handle the redirect and use the new listener
Add-AzureRmApplicationGatewayRequestRoutingRule -Name rule02 -RuleType Basic -HttpListener $listener -RedirectConfiguration $redirectconfig -ApplicationGateway $gw

# Update the application gateway
Set-AzureRmApplicationGateway -ApplicationGateway $gw

Why does this matter/needed?
Application Gateway can take a whole day(!) to deploy due to each individual change taking anywhere from 5-45 minutes to apply.
Powershell avoids this by generating the configuration object and then applying it in bulk to the Application Gateway, resulting in only 1 operation.
Presumably this will continue to be the case with more and more PaaS offerings in Azure. Is the right answer here to build this capability in AZ CLI or perhaps the right answer is to improve mgmt PaaS offerings that are so cludgy? This would be a suggestion for the former, as to some extent understand why Application Gateway takes some time to apply changes (multiple nodes, etc).

HI @iyerusad, please try issuing your application gateway commands with --no-wait.  In this way, you don't have to wait for the entire update cycle to finish and can chain multiple commands together to configure your gateway.  In fact, application gateways were the primary reason that --no-wait was added in the first place.
To answer you original question, no, for technical reasons, the Powershell approach is not feasible in the CLI.  However, --no-wait can alleviate much of the frustration or you can deploy your gateway using an ARM template (through az group deployment create.

Thanks @tjprescott - I had previously attempted the --no-wait in the past and for some reason led to believe that it didn't function in this way. I just took some time today to validate and it appears to function as outlined.
The original question appears to be not possible/feasible at this time so will close this issue out.


[Enhancement]: When managing table objects under OBOracle mode with ODC, the case sensitivity rules for table names and column names can be kept consistent with other tools.

What would you like to be added?
Under OBOracle mode, when creating a table using ODC, if I enter a lowercase table name in the input box, the newly created table name will be in lowercase. This forces me to quote the table name using double quotes in order to reference it properly. In contrast, other tools will automatically convert lowercase table names to uppercase when entered.
Why is this needed?
It is hoped that ODC's behavior would match that of other tools in terms of case sensitivity.

Added in dev-4.2.4


No react-gravatar class

Hey, I’m using 2.3.0 with<EMAIL_ADDRESS>and Babel 5 (too lazy to upgrade now). It seems that this:
<img
<EMAIL_ADDRESS>    className={"react-gravatar " + @props.className}
    src={src}
    srcSet={retinaSrc + " 2x"}
<EMAIL_ADDRESS><EMAIL_ADDRESS>    {...@props}
    />

Is compiled to this:
return React.createElement("img", React.__spread({
        "style": this.props.style,
        "className": "react-gravatar " + this.props.className,
        "src": src,
        "srcSet": retinaSrc + " 2x",
        "height": this.props.size,
        "width": this.props.size
      }, this.props));

With React.__spread being Object.assign, which means that className is overwritten by the default className of the empty string.
var props = { className: '', email: 'foo' }
undefined
> Object.assign({ className: "default " + props.className}, props)
{ className: '', email: 'foo' }

Ooops! Thanks for that catch. Published 2.3.1 with fix.


Outcomes in WEB and R don't match

By chance I noticed that the outcomes of a paired (X and Y) mean difference that I had run in R (dabestr v0.3.0) and the one obtained after copy&paste and also upload to https://www.estimationstats.com/#/analyze/paired are quite different.
In R: Paired mean difference of X (n = 202) minus Y (n = 202)  0.161 [95CI  -0.371; 0.713]
In WEB: The paired mean difference between X and Y is 0.161 [95.0%CI -0.0476, 0.367]
In both cases 202 paired observations were used (I attached the csv file). In WEB both variables (X, Y) were upload as expected in distinct columns and in R I prepared a tidy dataset as it is explained here.
As you can see the limits of the 95%CI are very different and I don't understand why. Which one is the correct outcome?

By the way.. I wonder to what extend the length of the 95%CI can be used to assess whether the scores of my two variables (X,Y) are 'equivalent' as it is done in TOST analysis (e.g. TOSTER R package). Let's suppose that I define that two scores in my research field are equivalent if their mean paired differences fall inside -0.5 and +0.5. If all I said is correct the outcome of the web [95.0%CI -0.0476, 0.367] would support equivalence but the outcome in R does not. Am I correct if I use the 95%CI of dabestr  to assess equivalence or do I have to run specific tests for this?
Thanks you so much for your software. Actually, as soon as I was able to solve this issue I'm going to use it in my next paper. 👍
Best regards from Spain!
202_X_Y_paired_data.zip

HI @tonisoto , thanks for raising this issue; I can replicate it. Let me pinpoint what's going wrong and get back ASAP!

Thank you so much! You're very kind. I'm a novice in stats questions but
let me know if I can help.
Toni
El vie., 11 sept. 2020 10:04, Joses W. Ho<EMAIL_ADDRESS>escribió:

HI @tonisoto https://github.com/tonisoto , thanks for raising this
issue; I can replicate it. Let me pinpoint what's going wrong and get back
ASAP!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ACCLAB/dabestr/issues/94#issuecomment-690942640, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABRS6LBWWFI3K3YSDXSKBMLSFHKYHANCNFSM4RFUVMNQ
.

Hi, I wondered if this was solved as I have the same problem.
In R: Paired mean difference of CS+ (n = 30) minus CS-(n = 30)  0.207 [95CI  -2.04; 2.59]
In WEB: Paired mean difference of CS+ (n = 30) minus CS-(n = 30)  0.207 [95.0%CI 0.0687, 0.388]
This is the raw data:
CS-  | CS+
8.33 | 8.57
1.30 | 1.26
6.57 | 6.57
7.11 | 7.64
7.16 | 7.46
6.69 | 6.44
4.77 | 4.72
3.21 | 3.46
13.17 | 13.17
5.33 | 5.71
3.04 | 2.66
11.74 | 11.64
3.34 | 3.58
4.39 | 3.88
2.90 | 2.84
6.98 | 6.83
7.84 | 8.59
18.94 | 19.86
19.91 | 21.43
8.42 | 8.88
7.41 | 6.82
11.85 | 12.82
4.99 | 5.31
3.51 | 3.79
10.17 | 10.29
6.48 | 6.62
5.95 | 5.85
3.67 | 3.95
2.90 | 3.13
5.50 | 6.02
Thanks,
Francesca

Hi, I wondered if this was solved as I have the same problem.
In R: Paired mean difference of CS+ (n = 30) minus CS-(n = 30)  0.207 [95CI  -2.04; 2.59]
In WEB: Paired mean difference of CS+ (n = 30) minus CS-(n = 30)  0.207 [95.0%CI 0.0687, 0.388]
This is the raw data:
CS-  | CS+
8.33 | 8.57
1.30 | 1.26
6.57 | 6.57
7.11 | 7.64
7.16 | 7.46
6.69 | 6.44
4.77 | 4.72
3.21 | 3.46
13.17 | 13.17
5.33 | 5.71
3.04 | 2.66
11.74 | 11.64
3.34 | 3.58
4.39 | 3.88
2.90 | 2.84
6.98 | 6.83
7.84 | 8.59
18.94 | 19.86
19.91 | 21.43
8.42 | 8.88
7.41 | 6.82
11.85 | 12.82
4.99 | 5.31
3.51 | 3.79
10.17 | 10.29
6.48 | 6.62
5.95 | 5.85
3.67 | 3.95
2.90 | 3.13
5.50 | 6.02
Thanks,
Francesca

There is a bug in the way the confidence intervals for paired differences are computed. Currently now it seems the CIs for unpaired differences are returned instead.
Specifically, it looks to be related to how stratified resampling is performed to ensure that we can resample from groups with different Ns.
This is not an issue for paired, of course, but it looks like there needs to be some refactoring done to handle paired differences, OR if anyone can point me to how to handle bootstrapping with with the boot package for groups with non-matching Ns, that might also work!
I apologise for this inconvenience!

Thank you for the clarification! Unfortunately I am not able to help with the bug.
Just a couple of other questions:
So if I use the website, I should be fine, shouldn't I? I am asking because I have used it to make the figures of 2 recently published papers and I was planning to use it for my future papers, so I wanted to make sure I got a correct outcome. I have looked also at the MATLAB version of the code, but there doesn't seem a way to calculate CI for multi paired groups.
"This is not an issue for paired, of course, ..." Do you mean it is not an issue for UNpaired?
Thanks again

You should be fine if you use the web app, yes!
Also, yes, I meant unpaired, thanks for pointing it out! Have edited the typo.

cf #107

Update:
The dev version v0.3.9999 fixes this, and can be installed with
devtools::install_github("ACCLAB/dabestr",  ref = "v0.3.9999")

cf #99

It would be really great if this bugfix could be pushed into a release? Currently the release version should not be used for repeated measures analysis, while the dev version is behind on many other aspects. Thanks for your help!

Or is it believed that this has been addressed in the latest released version? My analysis shows this not to be the case using v2023.9.12. Using the same dataset:
Mean diff v2023.9.12: -0.315 [-0.617 -0.0206]
Estimation tools (web): -0.315 [-0.512, -0.116]
dev v0.3.9999 95% CIs match web, not v2023.9.12

I've also tried using the
devtools::install_github("ACCLAB/dabestr",  ref = "dev")
version _2023.9.12, but, while this addresses many of the updates missing in v0.3.9999, this still gives me the same (incorrect) result as v2023.9.12 for the paired differences analysis:
Mean diff 'dev' v_2023.9.12: -0.315 [-0.617 -0.0206]
Estimation tools (web): -0.315 [-0.512, -0.116]


Fix various issues with log format specifiers not matching the types being logged

Problem
I was looking at our logs and saw things like:
I (174870) chip[EM]: Received message of type 8 and protocolId 327680 on exchange 44249

and
I (141230) chip[SVR]: Packet received from Node:3ffe5f20: 112233 bytes 0

which were obviously bogus-looking.  The protocol id there is 0x00050000 when it should be 0x5 and we did not get a message that's 112233 bytes long, obviously.
Turns out that first is because we were logging a struct as an int, and the second is because we logged a 64-bit node id as a 32-bit quantity and the remaining 4 bytes leaked over into the other specifiers.
Change overview

Enable warnings, at least on gcc and clang, for chip::Logging::Log formats not matching what's being logged.

Rerun codegen.

Testing

Made sure this compiled on Mac, ESP32, nrfconnect.
Checked that the bogus logs from above now look like this:I (71890) chip[EM]: Received message of type 8 and protocolId 5 on exchange 44249

andI (103520) chip[SVR]: Packet received from Node 0x000000000001B669: 16 bytes

@andy31415 @msandstedt @saurabhst @Damian-Nordic @jmartinez-silabs could you take a look, please?


Applying migrations to the database whenever you run npx migrate-mongo up

Is your feature request related to a problem? Please describe.
I'm always frustrated when I have to change file name to apply my migrations on database. I think it is developer's responsibility to take care of applied migrations (adding another key-value to separate migrated documents from unmigrated ones). The same pull request opened here
Describe the solution you'd like
But I think it is not right to check database changelog. Basically I think we have to run migrations regardless of database changelog and it's developers responsibility to take care of it. E.X:
// @ts-check
const path = require('path');
const figlet = require('figlet');
const { promises: fs } = require('fs');
const dotenv = require('dotenv');

dotenv.config({
    path: path.join('..', '.env'),
});

const { CustomError } = require('../commons/custom-error');
const MIGRATED_VERSION = process.env.MIGRATED_VERSION;
const MIGRATED_COLLECTION = 'admin-user';

module.exports = {
    /**
     *
     * @param {import('mongodb').Db} db
     * @param {import('mongodb').MongoClient} client
     */
    async up(db, client) {
        const session = await client.startSession();
        try {
            session.startTransaction();

            console.log(figlet.textSync('admin'));
            console.log(figlet.textSync(' ||'));
            console.log(figlet.textSync('\\/'));
            console.log(figlet.textSync('user'));

            // START
            const unMigratedDocumentsCount = await db
                .collection('admins')
                .countDocuments({
                    migratedVersion: {
                        $ne: MIGRATED_VERSION,
                    },
                    migratedCollection: {
                        $ne: MIGRATED_COLLECTION,
                    },
                });
                await session.commitTransaction();
        } catch (error) {
            await fs.appendFile(
                `${new Date().getTime()}-admin-error.json`,
                JSON.stringify(error),
                'utf-8',
            );
            await session.abortTransaction();
            throw error;
        } finally {
            await session.endSession();
        }
    },

    /**
     *
     * @param {import('mongodb').Db} db
     * @param {import('mongodb').MongoClient} client
     */
    async down(db, client) {},
};

And with this way I will be sure that no matter how many times you will run my migration script, it will change the database documents just once. And if it fails you can be sure that it will just migrate rest of the unmigrated documents. not all of them (It will be good in large dataset cases).

Added your PR with lots of modifications in the latest release.


New version: ImageMagick.ImageMagick version <IP_ADDRESS>

Pull request has been created with WinGet Automation :rocket:
Microsoft Reviewers: Open in CodeFlow

Manifest already exists. Reset PR.


Clarify the KEDA website and documentation maintainers

Document the website maintainers.
Use-Case
Documentation and website maintainers should be listed in the doc repository.
Specification
Create an OWNERS.md file to document (on the kedacore/keda-docs repo) the current custodian(s) of the following accounts: analytics (GA4), site-search (Algolia).
Explicitly document in the repo (keda-docs, keda, or both) who the website/documentation maintainers are. Solicit maintainers and contributors for documentation, either in the new OWNERS file or the governance MAINTAINERS file.

It's best not to auto-delete this issue; it is still relevant.

Hi, what would be the recommendation? We already have https://github.com/kedacore/keda-docs/blob/main/.github/CODEOWNERS.
Is the suggestion to reference this in the README or?

I think surfacing the info in the README or on the website (the community page could be a good spot) would suffice.

I think including the info in the README could work, but I think the point of this suggestion is to ensure access to these services when project  personnel changes (or when someone needs access years in the future after things have been setup). Since it's potentially private or secure data, maybe create a private document (shared with the CNCF) with instructions on how to access it when needed.

Oh I missed the part for site-search & analytics. I honestly don't know that, this was set up by CNCF so I'd presume that is on your team then or not?


fix: use onItemInvoked instead of onActiveItemChanged

Since I did the work for selection on LogDialogs list we learned the trick to enable single click invocation on items.
We can now use the proper onItemInvoked instead of onActiveItemChanged. The allows right click and many accessibility improvements such as keyboard navigation and 'Enter' for selection.
Fixes: ADO#1519

:tada: This PR is included in version 0.367.7 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[v2] improve robustness of linking to license on hosting website

In v2, I implemented some utils to get github repo from go-import=1 and use it to generate public & versioned links to detected licenses' hosting website (for now, only github).
I noticed some harder problems:

distinguishing "major branch" and "major subdirectory" conventions

There is one problem: for a major version greater than 1, the templates for “major branch” and “major subdirectory” conventions differ (See https://research.swtch.com/vgo-module for a discussion of these conventions.) To determine the right template, make a HEAD request for the go.mod file using each template, and select the one that succeeds. For example, for module github.com/a/b/v2 at version v2.3.4, probe both github.com/a/b/blob/v2.3.4/go.mod (the location of the go.mod file using the “major branch” convention) and github.com/a/b/blob/v2.3.4/v2/go.mod (its location using “major subdirectory”).

support modules not at root of a repo, example https://github.com/Azure/go-autorest/tree/autorest/v0.9.0. Note that tags are also different, a tag "autorest/v0.9.0" means v0.9.0 version of the module ROOT/autorest. https://github.com/googleapis/google-cloud-go/tree/master/storage is another example, tags for it has "storage/` prefix.
support other source hosting websites

Potential Solution
@wlynch pointed out the following references, there's an internal source package built for pkgsite that exactly provides a package that can figure out repo hosting website of a go import path and get a public link to source code. However, the package is internal, so we cannot directly import it.
I'll ask if they are ready to make it public, or I have to vendor it in some way.
EDIT: the reply is that we need to vendor it: https://github.com/golang/go/issues/40477#issuecomment-868532845.
References

https://github.com/google/go-licenses/pull/67#discussion_r653969960
https://github.com/golang/go/issues/39559
https://github.com/golang/go/issues/40477

I noticed that problem 2 and 3 are mostly solved by pkgsite/source package.
While problem 1 -- distinguishing "major branch" and "major subdirectory" conventions may still cause incorrect remote URLs.
We will still need to leave this issue as open.

I noticed that problem 2 and 3 are mostly solved by pkgsite/source package.
While problem 1 -- distinguishing "major branch" and "major subdirectory" conventions may still cause incorrect remote URLs.
We will still need to leave this issue as open.

Giving a breaking example for case 2 "support modules not at root":
$ go-licenses csv cloud.google.com/go/storage
...
cloud.google.com/go/storage, https://github.com/googleapis/google-cloud-go/blob/storage/v1.10.0/storage/LICENSE, Apache-2.0
...

Note the URL https://github.com/googleapis/google-cloud-go/blob/storage/v1.10.0/storage/LICENSE is broken, the correct URL should be https://github.com/googleapis/google-cloud-go/blob/storage/v1.10.0/LICENSE. The problem is caused by the fact that:

for modules in a subdir of a repo, when go caches module files, it "magically" copies LICENSE from root folder to the sub-module, when the sub-module does not contain a LICENSE file. e.g. https://github.com/googleapis/google-cloud-go/tree/storage/v1.10.0/storage
therefore, go-licenses finds a LICENSE file at root of submodule and tries to guess its remote URL as root of submodule, while the actual LICENSE file is at root of repo

TODO: find an example for problem 1: distinguishing "major branch" and "major subdirectory" conventions

Added a v2 proposal roadmap item: validate license URL by fetching it.

Furthermore, we can solve all above broken cases by:

Infer remote license URL as usual
Fetch raw license file from remote, validate it's the same as the locally found license file
If 2 failed, we can further try LICENSE at repo root
If everything failed, return UNKNOWN

Could you export a (versioned) URL to the root of the repo as well?
Possibly a breaking change to add it to the CSV, but it could be added to the data available to templates.
I'm creating a licenses page in my web app and would like to link the package name to the respective github (or wherever) page.

Possibly a breaking change to add it to the CSV

The csv format is fixed, I would not modify it.

but it could be added to the data available to templates.

Welcome a PR, this isn't too hard.

Okies, already started and have it basically working - unfortunately I won't have time to polish/finish it this/next week, but will do when I get to it.


Update data-connect-datasets.md

@rimisra2  - can you approve and merge please

Learn Build status updates of commit b07886f:
⚠ Validation status: warnings
File	Status	Preview URL	Details
concepts/data-connect-datasets.md	⚠Warning		Details
concepts/data-connect-datasets.md

Line 200, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'data-connect-policies.md'.

For more details, please refer to the build report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:

Try searching the learn.microsoft.com contributor guides
Post your question in the Learn support channel

resolved

@rimisra2 , please review and approve this PR.


[Do not merge]refactor: renamed secure flavor to standard

Description
This PR addresses Issues:
207 - To rename the secure flavor to standard

Release required?

[ ] No release
[ ] Patch release (x.x.X)
[x] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

Upgrade test failure is seen - TestRunStandardUpgradeTest

/run pipeline

/run pipeline

@imprateeksh can you remove the doc from this PR so we can merge the remaining code? And add the doc in a subsequent PR. We can publish the DA to community registry without the doc for now, but need the other changes

Yes sure @ocofaigh. Will do it right away.

@imprateeksh can you remove the doc from this PR so we can merge the remaining code? And add the doc in a subsequent PR. We can publish the DA to community registry without the doc for now, but need the other changes

Yes sure @ocofaigh. Will do it right away.

@ocofaigh I have removed the doc.

/run pipeline

@imprateeksh can you resolve conflicts?

@imprateeksh can you resolve conflicts?

Resolved

/run pipeline


Unable to add child bridge to HomeKit

Describe Your Problem:

I've successfully added the DREO plugin to home Homebridge, however when I try to add the bridge to HomeKit on my device after scanning the QR, it stays a while in the "connecting" to bridge screen, and then I get the Accessory Not Found error.
The fan model is DR-HAF004S
The accessory does show on Homebridge accessories tab.
Logs:
[6/7/2024, 10:12:42 PM] [Dreo Platform] Restarting Process...
[6/7/2024, 10:12:43 PM] [Dreo Platform] Launched child bridge with PID 998
[6/7/2024, 10:12:43 PM] Registering platform 'homebridge-dreo.DreoPlatform'
[6/7/2024, 10:12:43 PM] [Dreo Platform] Loaded homebridge-dreo v3.2.5 child bridge successfully
[6/7/2024, 10:12:43 PM] Loaded 1 cached accessories from cachedAccessories.123456.
[6/7/2024, 10:12:43 PM] [Dreo Platform] Loading accessory from cache: Air Circulator
[6/7/2024, 10:12:43 PM] Homebridge v1.8.2 (HAP v0.12.1) (Dreo Platform) is running on port 43752.
[6/7/2024, 10:12:44 PM] [Dreo Platform] Country: GT
[6/7/2024, 10:12:44 PM] [Dreo Platform] Region: NA
[6/7/2024, 10:12:44 PM] [Dreo Platform] Restoring existing accessory from cache: Air Circulator

Plugin Config:
{
    "name": "Dreo Platform",
    "options": {
        "email"<EMAIL_ADDRESS>        "password": ""
    },
    "hideTemperatureSensor": true,
    "temperatureOffset": 0,
    "_bridge": {
        "username": "",
        "port": 43752
    },
    "platform": "DreoPlatform"
}

Screenshots:

Environment:

Plugin Version: v3.2.5
Homebridge Version:  v1.8.2
Node.js Version:  v20.14.0
NPM Version:
Operating System:  Ubuntu Jammy (22.04.4 LTS) (Docker container on QNAP)

Try turning off the child bridge, restarting homebridge and then turning it back on. If that doesn't work, does the plugin work without a child bridge?

Without the child bridge the fan would not show on Apple Home, but it would still show on Homebridge.
I just updated to the latest plugin version and that seems to have fixed the issue, I was able to add the child and it's showing up in Apple Home as well.
Thank you :)

Great! Glad that fixed it.


Error: Received message larger than max

We are getting an error in the JS client using grpc
“Received message larger than max (4194538 vs. 4194304)“.
I noticed that this limitation was addressed in the golang client.
https://github.com/dgraph-io/dgraph/issues/396
Can we open this up on the JS client as well?
Looks like the ability to pass option to the client was added 5 days ago.
https://github.com/dgraph-io/dgraph-js/commit/f3bde42862d9c1109cf67fdf62c77fa6ae0f1627
Wondering if I can pass an option in like this:
https://github.com/grpc/grpc/issues/8098
Do these options flow through to the grpc client, or is it a different set of options?
We'd like to be able to do something like this:
  const clientStub = new DgraphClientStub(
    server,
    credentials.createInsecure(),
    {'grpc.max_send_message_length': 1024 * 1024 * 1024}
  )

Yes, you can pass options to DgraphClientStub as a 3rd argument. They are passed to the grpc client.

This worked for me once I upraded dgraph client to 1.0.4 and changed my grpc option to 'grpc.max_receive_message_length'. Thanks.


unknown command "new" for "operator-sdk"

➜  example-inc operator-sdk new memcached-operator
Error: unknown command "new" for "operator-sdk"
Run 'operator-sdk --help' for usage.
FATA[0000] unknown command "new" for "operator-sdk"
➜  example-inc operator-sdk version
operator-sdk version: "v1.0.0", commit: "d7d5e0cd6cf5468bb66e0849f08fda5bf557f4fa", kubernetes version: "v1.18.2", go version: "go1.14.7 darwin/amd64", GOOS: "darwin", GOARCH: "amd64"

was just about to report the same issue
documentation is not aligned with the actual sdk (on both version 4.3 and 4.5)
should use "init" instead of "new"

I think I found the solution
the documentation of openshift shows documentation of an older version of the operator-sdk - this is very confusing
the latest documentation for operator-sdk is here:
https://sdk.operatorframework.io/docs/building-operators/golang/quickstart/

The openshift docs are not related to the 1.0.0. I'd recommend you follow up the website.
Could you please also share the links for the OCP docs?

https://docs.openshift.com/container-platform/4.5/operators/operator_sdk/osdk-getting-started.html

@arivatibm Sorry for the confusion. Yes, you've hit a wart in our process. The operator-sdk is currently delivered as an upstream project and released as such. Because we want it to work with OpenShift the OpenShift documentation will reference it. The problem is that the OpenShift documentation usually reference an older version.
We are working on solving this so there will likely be one that is shipped on a cadence closer to the OpenShift release and documented there. While the upstreams releases will be documented upstream. The v0.19.x release is probably the closest to match the OpenShift documentation. You can find that release here: https://github.com/operator-framework/operator-sdk/releases/tag/v0.19.2

will an operator created with version 1.0.0 of operator SDK work on an OCP 4.3 cluster?

Regards,

Ari Volcoff                        Research Staff Member                        IBM - Haifa Research Lab
"Make everything as simple as possible, but not simpler." - Albert Einstein
Mobile: +972-58-4168357 | Phone: +972-4-8281319                        E-mail<EMAIL_ADDRESS>                      LinkedIn: https://il.linkedin.com/in/avolcoff

----- Original message -----From: Jesus Rodriguez<EMAIL_ADDRESS>operator-framework/operator-sdk<EMAIL_ADDRESS>arivatibm<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>[EXTERNAL] Re: [operator-framework/operator-sdk] unknown command "new" for "operator-sdk" (#3758)Date: Thu, Aug 20, 2020 4:02 AM

@arivatibm Sorry for the confusion. Yes, you've hit a wart in our process. The operator-sdk is currently delivered as an upstream project and released as such. Because we want it to work with OpenShift the OpenShift documentation will reference it. The problem is that the OpenShift documentation usually reference an older version.
We are working on solving this so there will likely be one that is shipped on a cadence closer to the OpenShift release and documented there. While the upstreams releases will be documented upstream. The v0.19.x release is probably the closest to match the OpenShift documentation. You can find that release here: https://github.com/operator-framework/operator-sdk/releases/tag/v0.19.2
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.

Hi @arivatibm,

will an operator created with version 1.0.0 of operator SDK work on an OCP 4.3 cluster?

Yes, it will. You can use the 1.0.0 upstream sdk release to build projects for OCP.

Alternatively, you could use the v0.19 docs https://v0-19-x.sdk.operatorframework.io/

Alternatively, you could use the v0.19 docs https://v0-19-x.sdk.operatorframework.io/

Yes - this is what I ended up doing

Hi @arivatibm,
I'd truly recommend you NOT create new projects using 0.19 since its layout differ a lot from 1.0.0 and you will need to migrate them soon. The 1.0 should work with OCP.
c/c @asmacdo

Hi @arivatibm,
Have you been updating the docker file to use the downstream image? I mean the image that is in redhat catalogue. If not, then I'd truly recommend you NOT create new projects using 0.19 since its layout differ a lot from 1.0.0 and you will need to migrate them soon. The 1.0 should work with OCP. Also, in the 0.19 you will be unable to create new GO projects with the legacy layout already.
c/c @asmacdo @jmrodri

I followed the instructions and tried it twice and had the same issue
might get back to it after version 1.0.1 is released

This is due to version incompatibility , you can verify this using " operator-sdk --help " , V1.0.X do not support " operator-sdk new " , on the contrary V0.17.X do . The official traps are the most deadly .

HI @arivatibm,
Could you please follow up the docs in  https://sdk.operatorframework.io/docs/building-operators/golang to work with the latest versions?

We have developed the operator for a Demo so not clear if we will upgrade

Closing as this is a documentation problem.