Year 2101 problem

Given latitude = 0, longitude = 100
2100-Dec-31 23:55:00
your azimuth: 113.222
noaa azimuth: 113.23
difference: 113.23 - 113.222 = 0,008

2101-Jan-01 00:00:00
distance: 0.137146
your azimuth: 113.2
noaa azimuth: 113.29
difference: 113.29 - 113.2 = almost 0,1

In other words, error in azimuth between this algorithm and noaa increases >10x in 5 minutes passed between 2100-Dec-31 23:55:00 and 2101-Jan-01 00:00:00.
Problem is in calc_time, consider:
for y in range(2098, 2103):
    print(calc_time(y, 12, 31, 23, 55, 0) - calc_time(y, 12, 31, 23, 50, 0))
    print(calc_time(y+1, 1, 1, 0, 0, 0) - calc_time(y, 12, 31, 23, 55, 0), "\n")

will print:
0.00347
0.00347

0.00347
0.00347

0.00347
1.00347

0.00347
0.00347

0.00347
0.00347

Hmmm. amazed you found this. exhaustive testing ?
I'm looking into it

In calc_time the issue is caused in the calculating the number of leap years, here:
    leap = delta // 4 # former leapyears

It counts any year divisable by 4, even if it is also divisable by 100 and not 400.  This will cause an incorrect jullian date after 2100 by 1 day and one additional day for the passing of any such year (unless I'm mistaken).

Brilliant. Thanks


chore: Remove ttlAfterNotRegistered value from settings

Fixes #
Description
Remove ttlAfterNotRegistered value from settings, making this value no longer configurable but set to a heuristic value of 15m, which is the time between when the Machine is created and when we expect the Node to register with the cluster
How was this change tested?
make presubmit
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

7 of 7 (100.0%)  changed or added relevant lines in 2 files are covered.
30 unchanged lines in 6 files lost coverage.
Overall coverage decreased (-0.3%) to 81.259%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/machine/garbagecollect.go
2
81.82%

pkg/operator/controller/typed.go
2
93.55%

pkg/controllers/deprovisioning/controller.go
3
82.49%

pkg/controllers/node/controller.go
7
73.53%

pkg/controllers/provisioning/scheduling/preferences.go
7
86.67%

pkg/controllers/deprovisioning/events/events.go
9
85.25%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
6556

Relevant Lines:
8068

💛  - Coveralls

LGTM, don't forget to update docs and helm chart

This wasn't added to the main helm chart and the docs were never updated for this change ✅


Feature Request: Compiling for other distributions of Linux

Is your feature request related to a problem? Please describe.
The output of gradle buildGhidra is, as the documentation says, "intended only to run on the platform on which it was built.". This makes distributing builds awkward even between Linux machines.
For example, I pulled the latest commit (acb07dd) and built on a fresh Ubuntu 22.04 VM (installing gradle 7.6, openjdk-17-jdk, gcc, g++) with gradle -I gradle/support/fetchDependencies.gradle init followed by gradle buildGhidra. On an Ubuntu 20.04 machine,  ghidraRun seemingly works and I am able to open the CodeBrowser tool. However, the decompiler does not work, displaying errors like "Decompiler: Unable to initialize the DecompilerInterface: Broken pipe", due to the different version of GLIBC
./Ghidra/Features/Decompiler/os/linux_x86_64/decompile: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by ./Ghidra/Features/Decompiler/os/linux_x86_64/decompile)
./Ghidra/Features/Decompiler/os/linux_x86_64/decompile: /lib/x86_64-linux-gnu/libstdc++.so.6: version `GLIBCXX_3.4.29' not found (required by ./Ghidra/Features/Decompiler/os/linux_x86_64/decompile)

As the documentation seems to imply this is expected (or at least that inter-compatibility between Linux machines isn't guaranteed), I am filing this as a F/R rather than a bug.
Describe the solution you'd like
A means to compile for generic Linux installations.

As the nature of the C compiler/C library, This seems to be resolved by "compilation environment", not a feature of Ghidra.
glibc doesn't provide forward compatibility like that but provides backward compatibility.
see also: python binary extensions' manylinux efforts: https://github.com/pypa/manylinux

As @topia implied, it is within your control to compile in such a way that your native binaries will run on other distros.  This is how we build the Linux native binaries for our official multi-platform release.
Knowing all of this now, what did you have in mind for a feature request?

I guess it was more of a (dumb) question than a feature request; I was hoping to get it done without having to fiddle with say, Docker/chroot.
Thanks both for your help.


Offset?

Is the offset setting in hex? Decimal? Binary?

It's in decimal.
If you want to start at 16k (0x4000) you'll need to specify 16384.


Update Sign in screen: design + text

https://xd.adobe.com/spec/8f259bcd-550e-4f0c-40bb-bdce2ed36fe5-f914/screen/5fdc388b-ee8e-4978-a07b-25373e52921f/Sign-In-Recover-5/

Checked on the Dev environment (v0.15.1-0)
Look as per mockups. All text match the design.


Add command to the command palette to run sourcekit-lsp diagnose

Starting with Swift 6, the sourcekit-lsp diagnose terminal command generates a bundle that helps SourceKit-LSP developers diagnose issues. I think it would be great if we could make that more accessible in VS Code by adding a command to the command palette that runs sourcekit-lsp diagnose --toolchain /path/to/toolchain in the integrated terminal. In particular, this could ensure that you run the command with the toolchain that is selected in VS Code, which yields more actionable bugs.
As an added bonus, when sourcekitd crashes and SourceKit-LSP shows the SourceKit-LSP: Restoring functionality: Please run 'sourcekit-lsp diagnose' to file an issue status (you can reproduce this by running killall SourceKitService when having a project open), it would be great if there was a button to run sourcekit-lsp diagnose. I can add some indication to the work done progress that SourceKit-LSP sends for this.

@ahoppen I have this PR in progress which captures sourcekit-lsp output to be included in a diagnostic bundle. How about I adapt it to use sourcekit-lsp diagnose instead so we just have the one command? Using sourcekit-lsp diagnose has the added benefit that if the user has sourcekit-lsp tracing turned off in the extension they'll still get the output.
I'll keep the current functionality in the PR for 5.10 and lower.

Verified with 3843201 on macOS and Linux


Force-destroying nested, permanent records not possible with foreign keys?

It's a bit, but not quite related to #65.
I'm trying to delete a permanent record, which has_many permanent records using record.destroy(:force), but I get stuck because referential integrity is violated by doing this.
I took a look at the specs, which try to verify that this works, but (I think) SQLite does not enforce referential integrity the way PostgreSQL does.
In the code I found this segment which seems to be the code at fault (should I be correct about my assumption).
The records in the has_many association are properly being saved in dependent_records, however, the block, in this case super() gets called before, meaning that before permanent_records has a chance to indeed delete the records that are permanent, my code crashes because super throws an exception before this can happen.

I just tested by swapping out this gem by acts_as_paranoid, it works with acts_as_paranoid, but I'd love to stick with this gem, because of the explicit scopes and the small codebase. :)

If I understand this code correctly, they delete the associated records first, then the record itself within a transaction.

Today I'm launched all specs with postgresql as DB and no specs failed. Seems that the issue requiring additional specs.

@aried3r  @drakmail did you find a solution for this?

@airpoli I have since switched to paranoia 2.2.0.pre.


Implement Design Changes

[ ] Deletion of certs on swipe left
[ ] Remove segmented control
[ ] Replace logo
[ ] Flip List


Prevent duplicate tag push

...and ensure that ending up with a duplicate tag offers a seamless path forward.
Addresses #5712

[ ] add integration test that actually exercises this scenario

/azp run tools - test-proxy - tests

Unfortunately I broke the integration tests somehow so this won't be going in as-is.

/azp run tools - test-proxy - tests

/azp run tools - test-proxy - tests

/azp run tools - test-proxy - tests

/azp run tools - test-proxy - tests

/azp run tools - test-proxy - tests

/azp run tools - test-proxy - tests


Adds (very) basic zsh autocompletion

Only top level commands are supported i.e. asdf <TAB>
and you get a list of asdf commands. That's it.

Thank you @justin-calleja ~!

You're welcome :smile:

@justin-calleja just tested. Works like a charm :+1:

This is awesome - thanks!

hehe better to test before merging - I'm a bash/zsh noob ^^;
Have a great day!

@justin-calleja well I knew you would have definitely checked before sending the PR. So was confident.

@andrewbredow wow thanks :blush: - small addition to a really swell project by @HashNuke and existing contributors.


Add auto option

Auto option (correct flag TBD) should trigger both create and run commands, and kickoff a new browser instance immediately after the command.

completed by 437814e288b701c37b98c5a023de5d9fb62851a6


fix: avoid grouping logs from tasks which ended in error

Description
Fixes https://github.com/vercel/turbo/issues/6709
Github Action doesn't provide a mechanism for grouping logs together and keeping them expanded which is helpful for highlighting errors.
This PR changes the behavior so we avoid emitting the grouping header/footer if task execution was a failure.
Testing Instructions
Updated GH Action log grouping integration test
Closes TURBO-2320

make the header and footer red on failure

Like this?

Yeah exactly. If the arrow could be red too that would be great. I messed around in dev tools to get this:

This is using the same red as we get at the end, but I actually don't think it's dark enough. Ideally, I want to be able to scroll through a long list of collapsed groups and visually find the one that failed. Maybe a big ❌ in the task prefix would do the trick (but not sure that's technically feasible since the prefix is computed ahead of time)


Remove spurious ServicesWithPolicies

The method ServicesWithPolicies and its associated type,
policy.ResourceMap aren't needed. At best, the required value is
easily calculated at the call site, and at worst, the invocation
repeats a lot of work for a value (e.g., a resource's policy set) that
is already available in scope.

Thanks Aaron! :star:


asciigraph: 0.5.1 -> 0.5.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/guptarohit/asciigraph/releases.
meta.description for asciigraph is: "Lightweight ASCII line graph ╭┈╯ command line app"
meta.homepage for asciigraph is: "https://github.com/guptarohit/asciigraph"
meta.changelog for asciigraph 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

/nix/store/rfv3ma5aqdpb85y089v19i2x3cvdivyx-asciigraph-0.5.2/bin/asciigraph passed the binary check.

1 of 1 passed binary check by having a zero exit code.

1 of 1 passed binary check by having the new version present in output.

found 0.5.2 with grep in /nix/store/rfv3ma5aqdpb85y089v19i2x3cvdivyx-asciigraph-0.5.2

found 0.5.2 in filename of file in /nix/store/rfv3ma5aqdpb85y089v19i2x3cvdivyx-asciigraph-0.5.2

directory tree listing: https://gist.github.com/030d0eb6ca2cd44e7a69b95ac15dfacb

du listing: https://gist.github.com/44ad3f498268ba7b0c3fe1a5d99c15a5

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

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
asciigraph

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/rfv3ma5aqdpb85y089v19i2x3cvdivyx-asciigraph-0.5.2 \
  --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 asciigraph https://github.com/r-ryantm/nixpkgs/archive/3ac9e7b934554686609a11ef22a8f27e57e04f40.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/rfv3ma5aqdpb85y089v19i2x3cvdivyx-asciigraph-0.5.2
ls -la /nix/store/rfv3ma5aqdpb85y089v19i2x3cvdivyx-asciigraph-0.5.2/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:

asciigraph

Maintainer pings
cc @mmahut for testing.

Result of nixpkgs-review pr 118024 run on x86_64-linux 1

1 package built:

asciigraph


Improve resource usage when NFS is in high load.

Use libc to set file mode to O_RDONLY | O_NONBLOCK | O_NDELAY when open procfs files
Use CGROUP to set CPU LIMIT to 10%.

NOTE: It is possible to extend CGROUP to limit IOPS in case only CPU usage isn't enough to avoid the high load usage of machine resources by the exporter.

@diconico07 I think now it is safer and with the wrapper_read() it spends less time with the file open.
If after these changes we still have a decrease in performance, an alternative is to copy the files out of procfs to a temporary place and parse it from there.


Incorrect shape and strides after load_image()

ximage-io claims to use the shape convention WIDTH x HEIGHT x CHANNELS (see http://xtensor-io.readthedocs.io/en/latest/api_reference.html#images) along with the usual c-order strides (increasing from back to front). However, this is wrong.
All file formats store images such that the WIDTH axis has smaller stride than the HEIGHT axis, and xtensor-io reads the memory unchanged. This means that the shape must be HEIGHT x WIDTH x CHANNELS with corresponding strides. All other libraries that follow the c-order convention use this shape, see for example scikit-image. You can easily verify the error with a simple program:
        auto in  = xt::load_image("color_image.tif");
        for(int y=0; y<in.shape()[1]; ++y)
        {
            in(100, y, 1) = 255;
        }
        xt::dump_image("result.tif", in);

This should draw a vertical line, but actually draws a horizontal one.

PS: This is also the reason why the result of grayscale_erosion() you showed on gitter last week looked totally off.

PS: please try to keep the discussion positive.

What exactly are you referring to? "Showstopper" meant to say that the library is unusable before the issue has been fixed, which I really believe is true.

PS: please try to keep the discussion positive.

What exactly are you referring to? "Showstopper" meant to say that the library is unusable before the issue has been fixed, which I really believe is true.

You are absolutely right this is a very serious bug, and it is great you are getting into this!
While I don't think at all the intention is bad, I felt it can come across as confrontational.
As you can see, we are working hard to solve any issue that comes up as people engage with us.


[BUG?] Non active tab is monetized when restoring from old session

Have you ever noticed that when chrome is first started (restoring an old session) that it starts paying to a monetized tab, even if the monetized tab isn't the current tab when the browser was started?

Not sure if it's related to this but:
If I'm on a non-monetized page (in this case a localhost page with a listing of my test pages) and I open one of those (monetized) pages in a new tab, it starts paying (even though I didn't switch to that tab).

For the record, I created a test page with a link to a monetized page, and used "open in new tab" on the page and wasn't able to reproduce. On the main branch at least


Extension issue

Issue Type: Bug
Extension Name: python
Extension Version: 2019.6.24221
OS Version: Darwin x64 18.6.0
VSCode version: 1.36.1

:warning: We have written the needed data into your clipboard. Please paste! :warning:
{
	"messages": [],
	"activationTimes": {
		"startup": false,
		"codeLoadingTime": 193,
		"activateCallTime": 12,
		"activateResolvedTime": 1125,
		"activationEvent": "onLanguage:python"
	},
	"runtimeErrors": []
}

Thanks for reporting the issue however without any information we cannot do much.
Please fill in the issue template:

Most importantly, what's the issue (performance, errors displayed, etc).

We'll be more than happy to reopen the issue once the required information has been provided. Or please feel free to create a new issue.


InkWell from UserAccountsDrawerHeader not showing

The UserAccountsDrawerHeader is supposed to have a InkWell in its _AccountDetailsState, but this isn't visible at all when using it. This can be confirmed by downloading the Flutter Gallery App and taping at the user details in the Drawer Navigation example.

Issue still exist
Code Sample
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatefulWidget {
  @override
  MyAppState createState() => MyAppState();
}

class MyAppState extends State<MyApp> {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: HomeScreen(),
    );
  }
}

class HomeScreen extends StatefulWidget {
  @override
  _HomeScreenState createState() => _HomeScreenState();
}

class _HomeScreenState extends State<HomeScreen> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      drawer: UserDetailDrawer(),
      appBar: _buildAppBar(),
    );
  }
}

Widget _buildAppBar() {
  return AppBar();
}

class UserDetailDrawer extends StatefulWidget {
  @override
  _UserDetailDrawerState createState() => _UserDetailDrawerState();
}

class _UserDetailDrawerState extends State<UserDetailDrawer> {
  bool showUserDetails = false;

  Widget _buildDrawerList() {
    return ListView(
      children: <Widget>[
        ListTile(
          title: Text("Visão geral"),
          leading: Icon(Icons.dashboard),
          onTap: () {
            print("Visão geral");
          },
        ),
        ListTile(
          title: Text("Another tile??"),
          leading: Icon(Icons.question_answer),
        ),
      ],
    );
  }

  Widget _buildUserDetail() {
    return Container(
      color: Colors.lightBlue,
      child: ListView(
        children: [
          ListTile(
            title: Text("User details"),
            leading: Icon(Icons.info_outline),
          )
        ],
      ),
    );
  }

  @override
  Widget build(BuildContext context) {
    return Drawer(
      child: Column(children: [
        UserAccountsDrawerHeader(
          accountName: Text("Cleudice Santos"),
          accountEmail: Text("cleudice.ms@gmail.com"),
          onDetailsPressed: () {
            setState(() {
              showUserDetails = !showUserDetails;
            });
          },
        ),
        Expanded(
            child: showUserDetails ? _buildUserDetail() : _buildDrawerList())
      ]),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel dev, v1.18.0, on Mac OS X 10.15.4 19E266, locale en-GB)
    • Flutter version 1.18.0 at /Users/taha/Code/flutter_dev
    • Framework revision 14f3a36aba (3 days ago), 2020-04-03 17:06:01 -0700
    • Engine revision 09bc1fc45e
    • Dart version 2.8.0 (build 2.8.0-dev.20.0 e736495eb7)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/taha/Code/sdk
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = /Users/taha/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.4, Build version 11E146
    • CocoaPods version 1.9.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 45.0.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)

[✓] VS Code (version 1.43.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.9.0

[✓] Connected device (4 available)
    • SM M305F   • 32003c30dc19668f • android-arm64  • Android 10 (API 29)
    • macOS      • macOS            • darwin-x64     • Mac OS X 10.15.4 19E266
    • Chrome     • chrome           • web-javascript • Google Chrome
      80.0.3987.149
    • Web Server • web-server       • web-javascript • Flutter Tools

• No issues found!

This example could be simplified and not include a real email (I don't know if that email is from a customer, but someone in the world surely has it).

@feinstein
Thanks for pointing that out, appreciate it.
Updated the sample


Is there any list of exit codes from helm ?

We use internally helm v2 (still ...). Our internal requirement while developing any software is to  handle errors correctly (surprise ...). Is there any list of exit error codes from helm so I could handle them correctly ?

As far as I understand, there are only two exit codes cobra (and therefore helm) use as a convention:

exit code "zero" for success
exit code "non-zero" for failure

There are no specific exit error codes used by Helm. There are helpful error messages displayed on standard error output, however, and many projects interpret these errors to determine how to handle certain scenarios.
If you CTRL + F for log.Fatal, fmt.Sprintf, or type Err across the project, you should find the vast majority of custom error messages Helm produces. However, there are also error messages that are passed along from upstream dependencies, like Kubernetes. Those can be found looking for return err and the like.
Hope this helps!

This is a great question and raised some issues for me when I started poking at it.
First, the context... Exit codes can have special meanings, for example there is a list of reserved ones for shell scripting. In this example a 127 is for command not found.
Which leads me to my issue...
$ helm foobar
...
$ echo $?
0

A 0 exit code means the things worked. I think I found a bug (i'll file that elsewhere).
Helm itself does 2 things for error codes:

returns the exit code of the plugin if a plugin was executed
returns an exit code of 1 if there was an issue or a 0 if everything worked

Closing, as the original question was answered

Anyone knows what -9 means?
Sometimes, helm install is failing with -9 when I use it from inside the Kubernetes cluster...
This never happens in case I use Helm from a Bastion host or from my own machine...

Anyone knows what -9 means?
Sometimes, helm install is failing with -9 when I use it from inside the Kubernetes cluster...
This never happens in case I use Helm from a Bastion host or from my own machine...

I know it's an old thread but I hope it helps others that like me reach this Issue trying to understand why  are we getting this error.
In my case it was due to a lack of resources in the container where I was running helm, increasing these solved the issue.

@oscartrullols makes sense, the same application that used to run helm from inside the cluster can't handle multiple helm upgrades at once.
Off topic: Is there a way to limit CPU/RAM usage using Helm CLI?

In my case it was due to a lack of resources in the container where I was running helm, increasing these solved the issue.

We have this issue on two of our hosted deployment mechanism.

Off topic: Is there a way to limit CPU/RAM usage using Helm CLI?

I have the same question.


Error while executing onJobBucketChange.functions

Voice-builder has been deployed to the GCP project and a job is created from the dashboard. But the status of the job stays at Exporting Resources. Upon checking the logs, I saw that there was an error while executing the onJobBucketChange funciton. PFB the screenshot:

Moreover, on further debugging, I found that pipelineInfo.json is not created for the job. The files in the job bucket are as shown in the screenshot below:

I am not sure, what is going wrong? I appreciate any help provided.

Hey @dollyagarwal did yo get the solution for this, I'm stuck on the same issue

Hey @dollyagarwal I changed event.data in line 34 of index.js to just event and was able to resolve these errors.


feat(tooltip): add back the arrow

Proposed Changes
Like agreed with our designers, we are bringing back the arrow.
Before

After

Potential Breaking Changes
none expected
Acceptance Criteria

[ ] The proposed changes are covered by unit tests
[ ] The potential breaking changes are clearly identified
[ ] README.md is adjusted to reflect the proposed changes (if relevant)

THE ARROW LIVES!

I think we might want to update the picture as well:

New thumbnail @gdostie 😄


Resource 'aws_iam_role.autoscaler' not found for variable 'aws_iam_role.autoscaler.id'

Terraform version: Terraform v0.11.7
Description: I'm hitting a bug which I believe is the same as https://github.com/hashicorp/terraform/issues/16681
The cause appears to be the aws_iam_role.autoscaler resource having a count, and the objects that use its return value not correctly referencing the indexed value.
Also detailed here - https://www.terraform.io/upgrade-guides/0-11.html#referencing-attributes-from-resources-with-count-0

I've encountered the same issue.  Here's the output
Terraform version: Terraform v0.11.7
Error: Error running plan: 2 error(s) occurred:

* module.dynamodb_autoscaler_summaries.aws_iam_role_policy.autoscaler: 1 error(s) occurred:

* module.dynamodb_autoscaler_summaries.aws_iam_role_policy.autoscaler: Resource 'aws_iam_role.autoscaler' not found for variable 'aws_iam_role.autoscaler.id'
* module.dynamodb_autoscaler_summaries.aws_iam_role_policy.autoscaler_cloudwatch: 1 error(s) occurred:

* module.dynamodb_autoscaler_summaries.aws_iam_role_policy.autoscaler_cloudwatch: Resource 'aws_iam_role.autoscaler' not found for variable 'aws_iam_role.autoscaler.id'

@aknysh we're not using join-splat everywhere. That could be the problem.
https://github.com/cloudposse/terraform-aws-dynamodb-autoscaler/blob/master/main.tf#L54

@kleungkw it will probably 1+ week before we can take a look at this. PRs welcome.

+1


Unauthenticated users can access cms (read-only), with no option to login

I upgraded to the latest version today and invited a new cms user. When the user accessed the cms route, he immediatly had access to the "Desk" page, but was not able to login with his real account.
Dependencies:
{
"dependencies": {
		"@headlessui/react": "^1.7.16",
		"@heroicons/react": "^2.0.18",
		"@portabletext/react": "^3.0.4",
		"@sanity/client": "^6.4.3",
		"@sanity/image-url": "^1.0.2",
		"@sanity/vision": "^3.14.5",
		"framer-motion": "^10.15.1",
		"groqd": "^0.15.7",
		"groqd-playground": "^0.0.15",
		"next": "13.4.13",
		"next-sanity": "^5.2.2",
		"pure-react-carousel": "^1.30.1",
		"react": "18.2.0",
		"react-dom": "18.2.0",
		"sanity": "^3.14.5",
		"sanity-plugin-asset-source-unsplash": "^1.1.0",
		"styled-components": "^6.0.7"
}

Hosted on Vercel, configured according to the example in this repo. Yesterday it worked perfectly fine.
Lmk if you need any other information!

Are you using unstable_noAuthBoundary anywhere?

 Sorry, bug can be closed.
I was looking for the implementation using the AppRouter, and copy pasted this file:
https://github.com/sanity-io/next-sanity/tree/main/app/studio/[[...tool]]
Sorry, thanks for pointing it out!


Highlighting current selected icon when  using Neurmorphic style

Here's a screenshot of what I'm trying to achieve:

So far, this is what I actually have:

As you can see, I don't have the labels nor can I tell which icon/screen is currently selected.   The documentation shows the following two neumorphic styles:
Neumorphic
Neumorphic without subtitle
I don't actually see the "Neumorphic without subtitle" as an option when I'm looking at the available styles.
Here is the code I'm using to create one of the icons:
PersistentBottomNavBarItem(
        icon: const Icon(Icons.settings),
        title: ("Settings"),
        textStyle: const TextStyle(color: kPrimaryColor),
        activeColorPrimary: Colors.white,
        activeColorSecondary: kPrimaryColor,
        inactiveColorPrimary: kPrimaryColor,
        inactiveColorSecondary: Colors.white,
        onPressed: (_) {
          pushNewScreen(tabContext!,
              screen: SettingsScreen(),
              withNavBar: true,
},),

It seems only the  activeColorPrimary and  activeColorSecondary make a difference.
Any tips would be appreciated.  Maybe I'm just not using the properties correctly?

in your PersistentTabView you can pass neumorphicProperties where you can specify that you want the subtitle. Regarding your colors it seems that the secondary colors are not applied to the items background. And I think it is not possible to use BoxShape.circle in the neumorphicProperties. Also, the subtitle will be inside the neumorphic box, which is not what is needed regarding your image. Maybe we should make a list of what is not working, to fix that?

Thanks again @jb3rndt!  I really do appreciate your help.

in your PersistentTabView you can pass neumorphicProperties
I didn't realize neumorphicProperties option existed!

And I think it is not possible to use BoxShape.circle in the neumorphicProperties.
You seem to be correct.  The borderRadius property does work though, so I can round out the rectangles.

the subtitle will be inside the neumorphic box
Correct again.

Maybe we should make a list of what is not working,

Secondary color not being used
Display icon titles below icon, not inside.

Hi, I thought a bit more about this and I am not sure regarding some points.
First of all, there are quite some individual changes needed to fulfil the design on your design template, which makes the neumorphic style behave a bit differently from all the other styles. This on itself is not a big problem, some might argue that it should not be included in the package as a predefined style so the user should implement a custom navbar for that.
The discussion point is what you can see in the gif: when clicking on the neumorphic item, it has some effects that are being played. I dont know if that is desired, but at that point I asked myself if the neumorphic stuff is the right thing for your use case? As is seems, you just need a shadow for the inactive items and thats it right?

Ah and if you want to try out the current state, just change the dependency of this package to point to the git branch like this:
persistent_bottom_nav_bar_v2:
 git:
   url: git://github.com/jb3rndt/PersistentBottomNavBar.git
   ref: fix/neumorphic-style

I dont know if that is desired, but at that point I asked myself if the neumorphic stuff is the right thing for your use case?

Well, the neumorphic option was the only reason I chose this package, as the app I'm doing is based on the neumorphic design.
What you have in your picture looks very close to what I need.

Ok if the effect is no problem, did you try out using the adjusted dependency? Is there something not working still?:
persistent_bottom_nav_bar_v2:
 git:
   url: git://github.com/jb3rndt/PersistentBottomNavBar.git
   ref: fix/neumorphic-style

PS: there is a new NavBarStyle.neurmorphic2 that I built for your case


Add debug parameter for git-sync deployment to chart values

We'll need to add this to the deployment template:
spec:
  containers:
    - args:
      - -v 9
      command:
      - /git-sync

Reference(s):

https://github.com/kubernetes/git-sync#primary-flags

This is no longer required since git-sync has been removed.


Fix VS/Mac Link in Xamarin Setup

Fix VS/Mac link. Fixes Issue #1084

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
Public repo URL: https://github.com/Microsoft/azure-docs/blob/master/articles/azure-portal-overview.md
Private repo URL: https://github.com/Microsoft/azure-docs-pr/blob/master/articles/azure-portal-overview.md
We have closed this pull request. Please resubmit the updates through the private repo. If you need perms to the private repo, see the contributor guide instructions

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/cross-platform/setup-and-install.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Adding example files and commands with Platinum Genomes trio for all supported tools

This PR adds example data files, and test commands, for files from all 5 supported genotypers. This primarily affects:

example-files has new files for the Platinum Genomes trio for chr21 for each of the callers (adVNTR currently is an exception. It has dummy samples for now but will be updated and retested when that’s available)
the README.rst files have been edited to include example commands at the bottom.

In the process of testing all of these, a variety of mostly minor changes were also made.
MergeSTR:

When we check VCF types for files being merged, we were exploding we were confused that a file could come from more than one possible VCF type, even if the user had specified —vcftyp. This came up when testing a HipSTR file using a GangSTR reference. Both “HipSTR” and “GangSTR” were in the header of the VCF so we were confused. But even when I specified —vcftype hipstr it failed because InferVCFType failed. I have added vcftype as an optional argument to InferVCFType. If we detect more than one type, but the user-specified type is in that set, we return the user-specified type.
Minor error message change to make it clear we’re quitting in mergeSTR if we fail to write the header (e.g. due to duplicate sample names across files).
When merging things with conflicting refs, I changed to skip those records and output a warning rather than explode. This could be desirable if most of the records, but not all, are mergeable.

StatSTR:

Was assuming “END” field present in VCF files. Which is not the case for PopSTR. Instead, compute END as POS+len(ref_alele)
Output “.” for allele freqs/counts if there are no samples.

qcSTR:
No changes. But see other PR fixing issue with error "The directory  which contains the output location does not exist"
dumpSTR:

Fixed similar issue with inferring VCF type when confused between multiple types.

compareSTR:
No code changes made. Note the advntr/popstr examples are not interesting as they are hard to compare (both to themselves, since I haven’t figured out how to run them yet to generate VCFs with different params for the same samples, but also to other files for similar reasons, like that I’d need to use a custom reference to make calls comparable). But at least the trivial examples of comparing a file against itself works for those.

Codecov Report

Merging #95 into develop will decrease coverage by 0.11%.
The diff coverage is 88.00%.

@@             Coverage Diff             @@
##           develop      #95      +/-   ##
===========================================
- Coverage    96.98%   96.86%   -0.12%
===========================================
  Files           17       17
  Lines         2186     2199      +13
===========================================
+ Hits          2120     2130      +10
- Misses          66       69       +3

Impacted Files
Coverage Δ

trtools/utils/mergeutils.py
94.38% <ø> (ø)

trtools/mergeSTR/mergeSTR.py
98.21% <81.81%> (-1.17%)
:arrow_down:

trtools/utils/tr_harmonizer.py
94.04% <83.33%> (-0.22%)
:arrow_down:

trtools/dumpSTR/dumpSTR.py
99.35% <100.00%> (ø)

trtools/statSTR/statSTR.py
95.85% <100.00%> (+0.10%)
:arrow_up:

Continue to review full report at Codecov.

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

Edit: I have updated the AdVNTR examples to be based on the Platinum Genomes trio.


feat: Qdrant Embedding Removal

Closes #2178.
This implements the remove methods of the EmbeddingStore for Qdrant.
It also addresses a few Sonarlint warnings for QdrantEmbeddingStore.
Integration tests have been added.

Thank you for the quick review and the awesome LangChain4j project!!

@florian-h05 this week, most probably Thu or Fri


Disable Submit Button on Edit Profile Page When No Changes Are Made

Describe the bug
On the user information edit profile page, when clicking the Submit button without making any changes to the form across multiple sections, a form update request is still triggered.
To Reproduce
Steps to reproduce the behavior:

Go to users page
Click on details button and hit edit button on all the section
without making any change click the submit button
See error

Expected behavior
The Submit button on the user information page in edit mode should be disabled unless changes have been made to the form.
Screenshots

@nihal467 @rithviknishad I would like to work on this issue. could you assign it to me ?


PubSubClient.h:72: error: section attribute not allowed for ‘’

when compiling with Arduino IDE 1.0.6 you get the error:
PubSubClient.h:72: error: section attribute not allowed for ‘’
with Arduino IDE 1.0.5 it did compile just fine.
removing the 'PROGMEM' from this line solves the issue.
https://github.com/knolleary/pubsubclient/blob/master/PubSubClient/PubSubClient.h#L72

Same error on 1.6.4...

Bit long ago now, but what board were you targeting? It compiles fine for me against a regular arduino board definition.

Fixed in the newly released 2.0 version

install this version
https://github.com/heman4t/Arduino-pubsubclient


fixes #34 creating dummy keys for clean directories

:shipit:

For the record, I don't like the magical "cleanX" keys for non dirty directories but think when winrm 2.0 gets factored in here, we may make some significant changes.


.NET SDK extension is not meant for external use

We have been picking up telemetry from this extension indicating that it is leveraging the vscode-dotnet-sdk extension. As noted in our README, this tool is not intended for external use and was made specifically to be leveraged by the .NET education bundle. Please remove your dependency on the SDK extension as soon as possible, as it can potentially break your users when used outside of the context of the education bundle.

@djnicholson @devhawk

Thank you for the quick action! I noticed that you removed your reference to ms-dotnettools.vscode-dotnet-runtime as well. The runtime extension is fine for external use but the SDK extension is only internal, sorry for any confusion.

No problem @sfoslund. We need the full SDK, so there's little point to us using one without the other.

If your team ever does ship a version of the vscode-dotnet-sdk extension that's externally usable, we'd appreciate a heads up :)


feat(context menu): Add support to hide the kebab toggle in the context menu

What
Closes #237
Description

Adds prop hideContextMenuKebab to toggle context menu via Shift + Right Click only. cc @ferhoyos
To test:

In DemoNode.tsx, add hideContextMenuKebab as a prop of the DefaultNode component. In the demo app, go to Topology Package and in the "Node Options" dropdown, check Labels and Context Menus.

In DemoTaskNode.tsx, add hideContextMenuKebab as a prop of the TaskNode component. In the demo app, go to Pipelines -> Task Nodes and select "Context Menus" in the toolbar.

Type of change

[x] Feature
[ ] Bugfix
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Screen shots / Gifs for design review

Preview: https://react-topology-pr-topology-250.surge.sh

The label size for the node will need to handle the kebab menu not being shown:

Released in v5.4.1


Option to disable transform from signature generated by signxml

At this moment there's no way I could disable transform other than by editing the code of the signxml.
Is there a chance this could be resolved by adding some sort of switch?

Thank you for your interest in SignXML. At this time I don't see how adding a switch for this purpose would be useful. Failure to process a reference transform node indicates a counterparty that uses a flawed and probably non-compliant XML Signature implementation. Can you tell us what software your counterparty uses that requires the absence of a Transform node at that location?


Disable autoselect with selectOnBlur if no text typed

I use select2 with selectOnBlur.
If user click my select and blur without choosing anything, it selects the first option.
The behaviour I expect is to select text the user type, but if the user doesn't type anything, to do nothing.
Adding a new category

And wanting to select nothing :

Here is my configuration:
$(this).select2({
            maximumSelectionSize: 1,
            tags: ['One', 'Two', 'Three'],
            selectOnBlur: true
});

How can I do that ?

This issue should have been fixed in 4.0.


Intel Aero: Start external mags first

In theory the start order shouldn't matter, but this is worth a try @lucasdemarchi @lfelipe.

I cannot reproduce this without props. I'll need to test outdoors.
The priority is set, https://github.com/PX4/Firmware/blob/master/src/drivers/hmc5883/hmc5883.cpp#L1023 if sensor_is_onboard is correct.

If should always report external on the HMC driver, which is incorrect (since the Aero has no external HMC sensor - the only external sensor is an IST8310):
https://github.com/PX4/Firmware/blob/master/src/drivers/hmc5883/hmc5883_i2c.cpp#L116-L128
I need to re-check the Aero FC schematics, but we need to make sure that if it has an HMC populated that it reports internal.

This is the output of ls /dev:
ls /dev
/dev:
 accel0
 baro0
 console
 gyro0
 hmc5883_int
 ist8310_ext
 led0
 mag0
 mag1
 mpu6500_accel
 mpu6500_gyro
 ms5611_int
 null
 pipe0
 pipe1
 pwm_output0
 px4fmu
 ram0
 tap_esc
 ttyACM0
 ttyS0
 ttyS1
 ttyS2
 ttyS3
 ttyS4
 ttyS5

So it reports as internal.

Ok. The fix is then not this PR (can be closed) but to put a define for the external bus in src/drivers/boards/aero/board.h and to check for == external ID in the HMC driver. It might require putting this define into all board configs.
I want to avoid having more ifdefs in the drivers. That's horrible.


 in  causes link target that cuts off other cells

https://webbluetoothcg.github.io/web-bluetooth/#dom-bluetooth-deviceinstancemap-slot links to

It'd be nicer if the id were on the <td> so it would link to

Or maybe if the dfn were top-aligned within the table:

@fantasai since this is also related to https://github.com/w3c/tr-design.

This is a general failing of hash-linking I think.  Center-aligning table headers looks better by default, even though it causes a bad alignment here.  I can't move the ID up without moving all the linking stuff up, and declaring that <th> can create a definition (currently limited to <dfn> and headings). And that won't help the case where you have two definitions in the same cell (top-aligning won't help in that case either, if the user is linked to the second definition).
I could maybe detect when a dfn is in a container like this, auto-gen an ID on the container, and then cause autolinks to link to the container instead of the definition itself.  I can probably only do this for local links, tho, and obviously it won't help if people click on the dfn's own autolink and pass that link around.
All considered, I think I can't do anything reasonable here. Anything I can do is hacky, and only works in simple cases.  This needs a styling solution to work right, maybe something related to snap points that helps control behavior of hash-links.


Feature Request: When the received message is displayed, the part beyond the page is automatically moved to the next line

Messages that are currently out of page require manual movement of the bottom slider to see the remainder：
There is another problem here, the page slider is moved to the far right and still some parts are not fully displayed, you can only see all the message by adjusting the page size to exceed the length of the message

When the terminal(Powershell embedded in vscode) displays text, it will automatically adjust the character display position according to the page size, such as the yellow characters in the figure. This eliminates the need for a page slider：

@xiaguangbo I believe this may have been fixed by the solution to #17, which is available in pre-release, please let me know if that is the case!
As for the feature request, we'll deliberate whether it can fit into our backlog, but the issue where things don't show up without resizing the window should be fixed. Thanks for letting us know!


In model replace :eq(0) occurrences

use a native css selector and .eq(0) where necessary. This is much faster.

done, it seems


[Bug]: Icons not defaulting when using HA companion on Android phones

Requirements

[X] I checked the troubleshooting section in the README to verify that I have the latest Mushroom version.
[X] I did a search to see if there is a similar issue or if a pull request is open.
[X] I have read the state of card mod support (in pin issues) and understand card mod support is not provided in this repository.

Current Behavior
On desktop through browsers, the icons acta as they should, but on the HA companion App the icons are just not there.
Expected Behavior
I expect the icons to show in the companion app
Steps To Reproduce

Install companion app
use a mushroom entity card
see that icon will not display unless actually selected in the entity properties.

Context
YAML state

Environment
- Browser: HA Companion app
- HA Version: 2024.2.1
- Mushroom: v3.4.2

Anything else?
No response

Same issue here. For me it fails to display default icons on every platform (browser and app). This started when I updated HA to 2024.2.1.

It's the same as https://github.com/piitaya/lovelace-mushroom/issues/1368, right?

Correct, fixed by reinstalling and clearing cache!


backup ok - but Warning msg in LogFile

I start the backup manual and the backup file on NAS exist.
But I get Warnings in logfile.

Add-on: Samba Backup
Create backups and store them on a Samba share
Add-on version: 5.2.0
You are running the latest version of this add-on.
System: Home Assistant OS 13.2  (amd64 / qemux86-64)
Home Assistant Core: 2024.11.2
Home Assistant Supervisor: 2024.11.2
Please, share the above information when looking for help
or support in, e.g., GitHub, forums or the Discord chat.
cont-init: info: /etc/cont-init.d/00-banner.sh exited 0
cont-init: info: running /etc/cont-init.d/01-log-level.sh
Log level is set to INFO
cont-init: info: /etc/cont-init.d/01-log-level.sh exited 0
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service legacy-services: starting
s6-rc: info: service legacy-services successfully started
[24-11-19 13:16:00] INFO: ---------------------------------------------------
[24-11-19 13:16:00] INFO: Host/Share: <IP_ADDRESS>/HA_backup
[24-11-19 13:16:00] INFO: Target directory: automatisch
[24-11-19 13:16:00] INFO: Keep local/remote: all/30
[24-11-19 13:16:01] INFO: Trigger time: 03:44
[24-11-19 13:16:01] INFO: Trigger days: Tue Thu Sun
[24-11-19 13:16:01] INFO: ---------------------------------------------------
[24-11-19 13:16:02] INFO: Samba Backup started successfully
[24-11-19 13:31:33] INFO: Backup running ...
[24-11-19 13:31:33] INFO: Creating backup "Samba Backup 2024-11-19 13:31"
[24-11-19 13:32:22] INFO: Copying backup dcfbc150 (Samba_Backup_2024_11_19_13_31.tar) to share
[24-11-19 13:32:22] WARNING: dcfbc150.tar does not exist
[24-11-19 13:32:22] WARNING: Could not copy backup dcfbc150 to share. Trying again ...
[24-11-19 13:32:28] WARNING: dcfbc150.tar does not exist
[24-11-19 13:32:38] INFO: Backup finished
[24-11-19 13:35:44] INFO: Backup running ...
[24-11-19 13:35:45] INFO: Creating backup "Samba Backup 2024-11-19 13:35"
[24-11-19 13:36:30] INFO: Copying backup 8e1ee3ee (Samba_Backup_2024_11_19_13_35.tar) to share
[24-11-19 13:36:30] WARNING: 8e1ee3ee.tar does not exist
[24-11-19 13:36:30] WARNING: Could not copy backup 8e1ee3ee to share. Trying again ...
[24-11-19 13:36:36] WARNING: 8e1ee3ee.tar does not exist
[24-11-19 13:36:46] INFO: Backup finished

and the second problem: the backup files are in the share and not in the subfolder specified in the target directory.

I've got the same problems. Failed backup counter goes up and .tar stays in the share folder and not its destination subdirectory. Version is the same (5.2.0) and I checked all permissions and user mounting the share should be able to mv the file (actually I was expecting to create the file in the destination directory in the first place but ok).


update kinematic scaling for task-space retargeting

With this PR we update the kinematic scaling in case of task-space retargeting
related issue #98

Sorry, I was writing the suggestions while you were committing. Let me change my suggestions.

@GiulioRomualdi  If you do not have any comments, I proceed with squash and merging.

It passed the github Actions successfully. I proceed with squash and merging.


Carrousel showing up next column on swipe

Go to a carrousel:

Stop and swipe:

https://androidayuda.com/2018/04/30/messenger-kids-modo-sueno/

Won't fix for the moment.


Change visibility of get_most_recent_deployment function to internal

This PR fixes the issue where the library was unnecessarily getting deployed on-chain whenever get_most_recent_deployment function is called from DevOpsTools library.
By changing the visibility of get_most_recent_deployment to internal, the library will not get deployed.

Have you tested that this actually works?

Yes, I have tested it and it doesn't lead to deployment of the library on chain as the function is internal so the bytecode of the library is included in the interactions contract.


Various improvements

Major change: disable pruning by default

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 61.29032% with 12 lines in your changes missing coverage. Please review.

Project coverage is 57.23%. Comparing base (6b0f2c8) to head (907014c).
Report is 18 commits behind head on master.

Files
Patch %
Lines

core/src/solvers/cartesian_path.cpp
64.71%
6 Missing :warning:

core/src/utils.cpp
25.00%
3 Missing :warning:

...e/moveit/task_constructor/solvers/cartesian_path.h
0.00%
1 Missing :warning:

core/src/stages/move_relative.cpp
66.67%
1 Missing :warning:

core/src/stages/move_to.cpp
66.67%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #589      +/-   ##
==========================================
- Coverage   58.82%   57.23%   -1.59%
==========================================
  Files          91      130      +39
  Lines        8623    10434    +1811
  Branches        0      956     +956
==========================================
+ Hits         5072     5971     +899
- Misses       3551     4416     +865
- Partials        0       47      +47

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Where can i find the mod "forge" that it needs?

Please help me find it. It says that you need forge <IP_ADDRESS>3. to be able to use this mod. Or Delete Not enough items.


[ASV] port stack, unstack, replace and  groups benchmarks from pandas github

Have to add ASV benchmarks related which does the following actions

perform stack and unstack on dataframe.
Replace function on a dataframe.
Get groups returned by groupby.

@arunjose696 thank you for filing this issue. Could you please add priorities to all new Modin issues? You can ask someone from modin-core if you need help assigning a priority. I think these benchmarking issues can be P2.


 Adjust the UI to compensate for this

fixed


fix(schema): Fix issue on OS3 when using a model as QueryParams type

Information

Type
Breaking change

Fix
No

Description
Fix OS3 spec generation when a model is used as QueryParams Type
Usage example
See: https://gist.github.com/Romakita/2dd599b6905a08ab0301414599c3858a
Todos

[x] Tests
[x] Coverage
[x] Example
[x] Documentation

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

v6.10.1
GitHub release

Your semantic-release bot :package::rocket:


create a unity package for importing OpenMined into other projects

Related to #191

How involved was it to build this? Is it the kind of thing we shoudl write a script for or a Markdown file?

It was straight forward: a readme would make sense

added a tutorial


Long build time for kuksa-viss-client

In the requirements.txt file for kuksa-viss-client we state that we shall use grpcio ~= 1.46.0 and grpcio-tools ~= 1.46.0 but when using alpine (like in docker file) it seems that we need to build them from scratch if they are not cached, which can take > 45 minutes, like in this build
I assume this is because e.g. https://pypi.org/project/grpcio-tools/1.46.5/#files does not have pre-built wheel for musllinux_1_1_aarch64 which is required when using Alpine.
Other observations:

I did by the way tried to change gps2val to use python:3.9-slim-bullseye instead, but then I got an error when trying to push ttl image, possibly related to https://github.com/replicatedhq/ttl.sh/issues/104, see also https://github.com/docker/build-push-action/issues/634
For many other Dockerfiles we use something else than Alpine and that is an alternative - but then we end up with a bigger container (as more junk included)
It seems that the wheel for grpcio/grpcio-tools is not complete - there are "hidden dependencies" to e.g. linux-headers and/or g++ needed if you need to build it yourself (as needed for arm with Alpine). I.e. you must install certain items by apk or pip before installing grpcio, see e.g. https://github.com/eclipse/kuksa.val.feeders/pull/24.

Not relevant any longer it seems


just testing something...

Overview

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: [ABC-123]
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change
[ ] This PR includes testing instructions tests for the code change
[ ] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

Don't make me tap the sign.
PR Links and Envs

Repository
PR Link
PR Env

Monolith
Monolith PR
Monolith Env

Portal
Portal Link
Portal Env

Another Repo
Another Link
Another Env

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.304-alpha.5c71df.0@codecademy/gamut-labs@35.11.2-alpha.5c71df.0@codecademy/gamut@53.0.3-alpha.5c71df.0@codecademy/monaco@0.7.17-alpha.5c71df.0@codecademy/styleguide@64.2.1-alpha.5c71df.0


Interrupted download stream does not retry correctly after some bytes are sent

Creating an issue to track this. Currently, if the request body stream of a download is interrupted / forcefully closed the library is not correctly retrying. After some initial investigation it appears that this may be related to a problem in node-fetch as outlined here. Currently investigating possible solutions as the version in which this was fixed for node-fetch requires the ability to utilize ESM.

Tagged as external because this is an issue with node-fetch / teeny-request

Updating this as a self reminder of sorts: this appears to have been fixed in node-fetch v3.1.1 . It does not appear that the fix was ever back ported to 2.x.

Possibly related as well: https://github.com/node-fetch/node-fetch/pull/1064. This appears to have been backported and released with 2.6.9. We moved to an async iterator with this PR: https://github.com/googleapis/nodejs-storage/pull/1925 this was around the same time we noticed this bug.

To my last comment, I was able to verify hanging behavior with 2.6.7 but get a correct error with 2.6.9 of node-fetch.

import express from 'express';
import fetch from 'node-fetch';

const app = express();
const port = 5000;
let count = 0;
app.get('/', (req, res) => {
    setInterval(() => {
        count++;
        res.write('hello');
        if (count >= 10) {
            res.socket?.destroy();
        }
    }, 1000);
});
app.listen(port, () => console.log(`Running on port ${port}`));

try {
    const response = await fetch('http://localhost:5000', {
        method: 'GET'
    });
    for await (const chunk of response.body) {
        console.log(chunk);
    }
} catch (e) {
    console.error(e);
}
console.log('Finished async iterator');

With node-fetch v2.6.7 this will reliably recreate unresponsiveness. With 2.6.9 the error is thrown / caught correctly.

Duplicate of: https://github.com/googleapis/nodejs-storage/issues/2193. Fixing the node-fetch issue will fix this problem.


Graphviz removal

Proposed changes:

remove the (py)graphviz dependencies
display the story graph within a html page instead of generating an image
adapting the visualization feature of the interactive learning for that
reuse the html template used for the visualization of the interactive learning
adapt documentation etc to the new html approach

Command to test it out:
python -m rasa_core.visualize -s examples/concertbot/data/stories.md -d examples/concertbot/domain.yml -o graph.html
Status (please check what you already did):

[x] made PR ready for code review
[x] added some tests for the functionality
[x] updated the documentation
[x] updated the changelog

I think we need to add pydot to the setup.py as well.

can confirm ff, safari still errors

I fixed the issue with safari. It was actually some problems with the replacement which made the js processing fail for interactive learning.
I also moved and adapted the test case.

works for me in safari, must be your old version @amn41 :D


activewindowv2 is called when not needed

Hyprland Version
 Hyprland, built from branch HEAD at commit 84ab8d11e8951a6551d1e1bf87796a8589da6d47  (props: bump ver to 0.35.0).
Hyprland, built from branch HEAD at commit 84ab8d11e8951a6551d1e1bf87796a8589da6d47  (props: bump ver to 0.35.0).
Date: Mon Feb 5 01:59:02 2024
Tag: v0.35.0

flags: (if any)

System Information:
System name: Linux
Node name: gamix
Release: 6.6.18-1-lts
Version: #1 SMP PREEMPT_DYNAMIC Fri, 23 Feb 2024 14:58:40 +0000

GPU information:
01:00.0 VGA compatible controller [0300]: NVIDIA Corporation AD103 [GeForce RTX 4080] [10de:2704] (rev a1) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

Bug or Regression?
Bug
Description
activewindowv2 event is emitted whenever a window is clicked (press and release) with a modifier to rescale or move it.
One click = 2 useless activewindowv2 events.
How to reproduce
move or resize any client window using the mouse + modifier.
Crash reports, logs, images, videos
No response

There is another issue with this event, sometimes it's triggered repeatedly while dragging a window or changing focus. But I can't figure the scenario to reproduce it.
It would be nice to not receive this event more than once in a row with the same client address...


IPv6 guest prep applies to CentOS as well as Ubuntu

Following discussion in Calico Users Slack at https://calicousers.slack.com/archives/C0BCA11Q9/p1537511750000100, this PR aims to clarify that IPv6 guest preparation steps are needed for CentOS as well as for Ubuntu.

For Centos 7 I added this to my cloud-init script to get ipv6 to run at boot.
runcmd:
- sed -i -e '$a'"IPV6INIT=yes" /etc/sysconfig/network-scripts/ifcfg-eth0
- sed -i -e '$a'"DHCPV6C=yes" /etc/sysconfig/network-scripts/ifcfg-eth0
- sed -i '/PATH/i\new_ip6_prefixlen=128' /sbin/dhclient-script
- systemctl restart network

Tested, and this works on the official CentOS cloud images from http://cloud.centos.org/centos/7/images/

Thanks @fasaxc , I've just pushed an extra commit to apply the same changes to v3.1 and v3.2.


Very Basic iOS port, with broken annotationSTAnnotationsPlugin

Annotation are not aligned properly with the line, and no message is shown.
Signed-off-by: Qian Qian "Cubik"<EMAIL_ADDRESS>

Sorry, my mistake...


Ancestry.com missing graphics

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to 'ancestry.com'
Click on 'a created tree'
Some graphics and text are missing.
Log on ancestry.com with firefox to observe missing artifacts and text

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

Desktop (please complete the following information):

OS: [Windows 7 SP1 x32 and 64]

Additional context
Squares are seen where there should be graphics.

This should be categorized as a minor bug, not a critical one

try this:https://github.com/win32ss/supermium/discussions/696#discussioncomment-10098269

Thanks.
dwritecore_xp_32.zip worked on Windows XP 32-bit.
dwritecore_xp_64.zip worked on Windows 7 64-bit.
On Sat, Jul 27, 2024 at 9:18 PM 結束バンドbocchi @.***>
wrote:

try this:#696 (comment)
https://github.com/win32ss/supermium/discussions/696#discussioncomment-10098269
—
Reply to this email directly, view it on GitHub
https://github.com/win32ss/supermium/issues/767#issuecomment-2254337683,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BKDXDFC4JUN5FCE3XFIFA7TZORWJPAVCNFSM6AAAAABLP2HS5GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJUGMZTONRYGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Phaser.Math.between fixes & added Phaser.Math.random

This PR changes (delete as applicable)

Documentation
TypeScript Defs
The public-facing API

Fixed docs for Phaser.Math.between(min,max), made it a bit more robust, added a random(min,max) function that acts the same as between(), but returns a floating point.
Ref https://github.com/photonstorm/phaser/issues/2758

I made between() work with integers (before it was whole numbers - positive only). The updated code was copy+ pasted from https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Math/random 's getRandomIntInclusive function.

Any issues with this PR, or things I can improve on?

Nope it's fine, have merged.


Tool doesn't download all data - zip is missing

Colab environment, Kaggle API 1.5.4
kaggle competitions download -c plant-pathology-2020-fgvc7
downloads all but plant-pathology-2020-fgvc7.zip
Which is useless. I expect it to download all competition's data, the same as in input directory in a Kaggle kernel.
kaggle competitions files -c  plant-pathology-2020-fgvc7
Warning: Looks like you're using an outdated API Version, please consider updating (server 1.5.6 / client 1.5.4)
name                    size  creationDate
---------------------  -----  -------------------
images/Test_1010.jpg   283KB  2020-03-09 20:57:51
images/Test_1002.jpg   214KB  2020-03-09 20:57:51
images/Test_1013.jpg   170KB  2020-03-09 20:57:51
images/Test_1001.jpg   226KB  2020-03-09 20:57:51
images/Test_1011.jpg   290KB  2020-03-09 20:57:51
images/Test_1000.jpg   238KB  2020-03-09 20:57:51
images/Test_1012.jpg   174KB  2020-03-09 20:57:51
images/Test_101.jpg    204KB  2020-03-09 20:57:51
images/Test_1009.jpg   220KB  2020-03-09 20:57:51
images/Test_1007.jpg   124KB  2020-03-09 20:57:51
images/Test_1.jpg      294KB  2020-03-09 20:57:51
images/Test_10.jpg     245KB  2020-03-09 20:57:51
images/Test_1004.jpg   232KB  2020-03-09 20:57:51
images/Test_0.jpg      227KB  2020-03-09 20:57:51
images/Test_1006.jpg   207KB  2020-03-09 20:57:51
images/Test_100.jpg    275KB  2020-03-09 20:57:51
images/Test_1008.jpg   274KB  2020-03-09 20:57:51
images/Test_1014.jpg   218KB  2020-03-09 20:57:51
images/Test_1003.jpg   213KB  2020-03-09 20:57:51
images/Test_1005.jpg   180KB  2020-03-09 20:57:51
train.csv               33KB  2020-03-09 20:57:51
test.csv                17KB  2020-03-09 20:57:51
sample_submission.csv   52KB  2020-03-09 20:57:51

use this
pip install --upgrade --force-reinstall --no-deps kaggle

This failed for me. pip install --upgrade --force-reinstall --no-deps kaggle appeared to upgrade 1.5.4 to 1.5.6 but then kaggle -v would still show 1.5.4. Even a full uninstall, updating pip, and reinstalling failed.
I was able to get it working by manually removing the kaggle binary (v 1.5.4) that was persisting at /opt/anaconda/bin/kaggle. and reinstalling.
What worked for me
pip uninstall kaggle
pip install -U pip
rm /opt/anaconda/bin/kaggle
pip install --upgrade --force-reinstall --no-deps kaggle

Notes:

pip install -U pip is for Linux/Mac, for windows use python -m pip install -U pip
For the final command you probably don't need pip install --upgrade --force-reinstall --no-deps kaggle but rather just pip install kaggle but I wanted to include the exact steps I used to get it working.


Spark Streaming Support

Hi Team,
I would like to understand whether there is any plan for Spark Structured Streaming support. Currently, I could not see Spark Streaming dependency for this codebase.
Kindly request you to confirm.
Thank you.

We don't have any plans to support Spark Structured Steaming support as yet.
Please feel free to create a PR to integrate within this repo as an additional feature.
We would be happy review and merge it.


Introduce theme support in rendering

<TBD>

In my docs builder repo, I used Twig paths to provide some sort of theming with a minimal amount of lines: https://github.com/wouterj/symfony-docs-guides/blob/main/lib/docs-builder/src/TwigEnvironmentFactory.php#L28-L33
Maybe something like this can be implemented directly into guides?


Merge rvsdg_frontend branch to main

Additional changes:
TBD

/azp run

changes from #9120 were cherry-picked via d23b32b


Buffer audio instead of firing an interrupt at 22,050Hz

As brought up by @zenodante in #562 - firing an audio callback at 22kHz will ransack the instruction cache among other things. We should be feeding a ~441 sample buffer at ~50Hz instead.

Checking if the DAC should be enabled every sample probably isn't great either... https://github.com/pimoroni/32blit-beta/blob/master/32blit-stm32/Src/sound.cpp#L22

Checking if the DAC should be enabled every sample probably isn't great either... https://github.com/pimoroni/32blit-beta/blob/master/32blit-stm32/Src/sound.cpp#L22

I did some sleuthing for the old audio code that used DMA and a buffer, and recalled that I never did manage to get it working. The system would just lock up solid and I had to abandon all hope in favour of... this travesty. So... yep. Long overdue a fix.

I did some sleuthing for the old audio code that used DMA and a buffer, and recalled that I never did manage to get it working. The system would just lock up solid and I had to abandon all hope in favour of... this travesty. So... yep. Long overdue a fix.

I will work on it, after I receive the hardware. Current my h750-dk boards don't have a DAC audio setup.  (Q_Q)/

I will work on it, after I receive the hardware. Current my h750-dk boards don't have a DAC audio setup.  (Q_Q)/


add getTimestamp method

Related Issue #241
I have forget add getTimestamp(String columnLabel, Calendar cal) method. The issue can be closed after this is merged.

Codecov Report

Merging #250 into master will increase coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #250      +/-   ##
==========================================
+ Coverage   73.36%   73.38%   +0.02%
==========================================
  Files         151      151
  Lines        9141     9144       +3
  Branches     1447     1448       +1
==========================================
+ Hits         6705     6709       +4
+ Misses       2072     2069       -3
- Partials      364      366       +2

Impacted Files
Coverage Δ

...in/java/org/verdictdb/jdbc41/VerdictResultSet.java
19.67% <0%> (-0.2%)
:arrow_down:

.../verdictdb/core/execplan/ExecutableNodeRunner.java
80.36% <0%> (-1.78%)
:arrow_down:

.../core/scrambling/FastConvergeScramblingMethod.java
93.08% <0%> (+2.11%)
:arrow_up:

Continue to review full report at Codecov.

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


Replaced flow_refresh.sh w flow_refresh.py script

Thank you for making node-red ⚙ better
Please reference the issue this PR is fixing:
This PR will fix the flow_refresh workflow within the k8s-sidecar documented in this issue

the original flow_refresh.sh script is failing due to sh dependenyies not installed in the image (curl, jq)
k8s-sidecar is python based an better fit python modules and enviroments for advanced script tasks, discussed here
k8s-sidecar update to 1.24.0 to fix scripts shebangs and properly run python scripts FR/PR
rewrote the flow_script in python an replaced it
changed default helm values sidecar.env.script to flow_refresh.py and sidecar.image.tag to 1.24.0

Also verify you have:

[x] Read the contributions page.
[x] Read the DCO, if you are a first time contributor.
[x] Read the [code of conduct](Code of Conduct).

Hi @hinrichd,
many thanks for the PR! I have to check this!

Hi @hinrichd,
can you rebase your PR please? I updated the version of the sidecar in another PR #238

Hi @dirien,
I have tried to rebase my PR. Hope it will work. Wirting as a git newbie

Many thanks @hinrichd for the new contribution!


Upgrade to xterm 3.6.0 for column selection. Refs #2697

This includes the xterm version bump required to enable the column selection feature. See #2697 for more details. I did not add anything for the macOptionClickForcesSelection option mentioned, but it seems to work out-of-the-box just fine on my mac.

You can probably update to 3.6 just as easily, it was mostly a stability release https://github.com/xtermjs/xterm.js/releases/tag/3.6.0

I’m not opposed. I only did 3.5.1 this time because 3.5 was minimum with the needed feature and may as well pick up the .1 for any bug fixes :)
Tests did not complain and everything seemed fine with 3.5.1 - not sure if there’s anything else that really needs to be done.

merging this will allow us to use font ligatures again on Hyper

I would actually wait for 3.7.0 as there was a pretty bad perf regression in either 3.5 or 3.6: https://github.com/xtermjs/xterm.js/issues/1620


Http (not) support in identity server URL in options.

Generated auth url is only for https//:{domain}. Please add http

If possible, add suport for client secret string property in options. (additionally)

Hi, could you please elaborate or give some example of the potential use cases that would require this features? Thanks.

Hello! In ClientOptions.cs:
public bool UseHttps { get; set; } = true;

In AuthenticationServiceBase.cs:
// in constructor
this._protocol = settings.UseHttps ? "https" : "http";

// when generating auth url
var url = $"{_protocol}://{clientSettings.Auth0Domain}/authorize?" + "...";

Use cases - my test identity server is working without https in test mode

Hi,
I'm afraid the auth URL cannot be other than https (even if you call it via http it redirects to https).
You said your test identity server is http only, I would like to know, how the https endpoint is affecting your implementation?
Regarding the client secret:
Currently, the component doesn't requires it, even the core hosted example uses the intrisic behavior of .Net authentication to verify the token signature.
If the client secret is something you require in your implementation then I can recommend you to use .Net Secrets or including it via environment variables.

Closing this as no new comments were added


Actually use defensive static_path_url

The commit you already had defensively checked for whether app.static_url_path or app.static_path was the right one to use...and then promptly ignore that and went for app.static_url_path anyway.  This fixes the behavior to work with (comically old) Flask versions.

Merging even though this is abandonware by now.


UiIconButton group

Like you have in your examples, it would probably be needed (?) by others too to have a way to group icon buttons. I have added this to my styles, following a BEM structure.
<div class="ui-icon-button__group">
  <ui-icon-button></ui-icon-button>
  <ui-icon-button></ui-icon-button>
</div>

.ui-icon-button {
    &__group {
        display: flex;
    }
}

Yeah, this is something that I prefer to be done by the developer. The aim is to just develop the individual components, and leave it up to the developers on how to compose them together.

I can see your point. An idea could be to add something in your docs describing how to implement it? So you avoid having it as a part of the component, but still help people how to archive it.

Had the same issue. Had to compare my code with the css and html in the examples to see that a wrapping div with display flex is needed. Agree to mention this on the docs.

This has been added to the UiButton docs in v0.8.8


Google Navigation not work

Describe the bug
Sir Really appreciate your work. you did an amazing job. I face an issue my url contain some google navogation so its show google map and everything fine but when I click on NAVIGATE it shows internet connection error.
you can find images below. Please waiting for your response
Screenshots
App:
https://drive.google.com/file/d/1BJ7ZNueI2d7_YX9HMje2cLiDeGhFlupS/view?usp=sharing
Logcat:
https://drive.google.com/file/d/1j4LZd5wpBwp4opqcFFz-jvLkndgGKhtj/view?usp=sharing
Device (please complete the following information):
Xoimi mi max API 23
and some other devices as well

are you trying to navigate within the webview or trying to open an external map?

sir i'm trying to open in external map

that means SWV isn't able to understand your specific request. better add a new condition in url_actions as:
if(url.startsWith("https://maps.google.com")){ //or whatever the intended url is
  //this will launch default tool available for particular task/URL
   Intent intent = new Intent(android.content.Intent.ACTION_VIEW, Uri.parse(url));
   startActivity(intent);
}

you can create your own custom intent. hope this helps.

Thank you, sir, for your response


Send data to opentelemetry-collector using grpc for node only

@mayurkale22 would love to pick this one if its okay with you

@Ema93sh I assigned to you. Just so you know, we used gRPC transport in the OpenCensus Node agent exporter (code link). Feel free to reuse few pieces from there.

@Ema93sh any updates on this? Feel free to let someone know if you have questions or need help


Mocking API errors doesn't work in 1.20.0

What is your Scenario?
Use RequestMock to mock API errors (status code != 200).
What is the Current behavior?
TestCafe throws An unhandled error occurred in the "onResponse" method of the "RequestMock" class: null.
What is the Expected behavior?
The request should be mocked and no error thrown. This used to work in 1.19.0.
What is your public website URL? (or attach your complete example)
https://testcafe.io
What is your TestCafe test code?
import { RequestMock, t } from 'testcafe';

fixture`Demo`;

test
    .page('https://testcafe.io')
    .requestHooks(
        RequestMock()
            .onRequestTo(/d/)
            .respond('', 500)
    )
    ('should pass', async () => {
        await t.wait(1000);
    });

Your complete test report
Demo
 ✖ should pass

   1) An unhandled error occurred in the "onResponse" method of the "RequestMock" class:

      null

      Browser: Firefox 102.0 / Linux 0.0

Steps to Reproduce

Install<EMAIL_ADDRESS>Run testcafe firefox:headless test.ts, observe failure
Run with<EMAIL_ADDRESS>observe that the test passes

TestCafe version
1.20.0
Node.js version
v16.14.0
Command-line arguments
testcafe firefox:headless test.ts
Browser name(s) and version(s)
Firefox 102
Platform(s) and version(s)
Fedora 36
Other
Works in 1.19.0.

Hi @sorin-davidoi,
I reproduced the issue. We will update this thread once we have news.


Request for Swipe-to-Delete and Multi-Select Conversation Features

It would be beneficial to enable conversation deletion with a swipe gesture instead of requiring a right-click and delete, making the process faster. Additionally, implementing a multi-select feature for conversations would allow users to delete multiple items simultaneously.

@pmusolino thanks for the ideas. I don't often see the 'swipe to delete' gesture in native macOS apps, so I'm not sure this is the true 'macOS way'. It's probably better to implement some shortcuts instead? Or probably you have some examples of macOS apps that use this feature?
Multiselect is a great idea - I have even tested some prototypes in macai and plan to implement this in the future (most likely in 2.1.0).

@Renset one example is the spark app. Instead of deleting an email from the trash icon or using the right click, you can just swipe to delete it. A shortcut however would also be appreciated (example: command + delete).

@pmusolino ok, I'll start with multiselect and shortcuts and we'll see if we need further improvements :)


Notice errors for the latest upgrade notification

In the plugin list i've got notification to upgrade to new Timber update, however inside this notification there are lots of notice and deprecated warnings:
Deprecated: Non-static method TimberAdmin::in_plugin_update_message() should not be called statically in /srv/www/project/current/web/wp/wp-includes/plugin.php on line 525
Notice: Trying to get property of non-object in /srv/www/project/current/web/app/plugins/timber-library/lib/timber-admin.php on line 42
Notice: Trying to get property of non-object in /srv/www/project/current/web/app/plugins/timber-library/lib/timber-admin.php on line 45
Notice: Trying to get property of non-object in /srv/www/project/current/web/app/plugins/timber-library/lib/timber-admin.php on line 54
PHP 7, Wordpress 4.5.1

Thanks @chetzof! I'll get right on that

Thanks again @chetzof — this is closed #976

Thanks! Is there a solution for users that cannot update to the 1.* version of the plugin? I've got users complaining that it broke their plugin list page, they that are on Version 0.22.6 and cannot upgrade to new version due to deprecations.

@chetzof I just updated the code for 0.22.6 being served from WordPress.org. You can also directly update the lib/timber-admin.php file with the contents of this gist:
https://gist.github.com/jarednova/e8a18361412ee4e7c40297ef815e714e


Do you have any plans for a future development?

What do you have in mind?
If there is anything that you find missing, please open up an issue and we can discuss it there. Same for bugs.

Just wanted to know if rheinfabrik has something in pipeline and all could be improved... ;-)
I'm using Heimdall and it's really great. Heimdall solved a really hard problem for me. Thanks to Rheinfabrik.

Thanks for your kind words. Again, if you have any feature requests or improvement ideas feel free to create issues :)


[EN] HassGetState for sensors with different device_class

Added sentences to get sensors by name for (almost) all device_classes from here https://www.home-assistant.io/integrations/sensor/

Thank you very much, @TheFes

@synesthesiam do you have any opinions on this? Can I merge it as is?


network events were not getting inserted

PR for https://github.com/zdresearch/OWASP-Honeypot/issues/10
what : in the ip split it is return type bytes and the netaddr.valid_ipv4 or
ipv6 needs a string as an ip so i converted the ip
@Ali-Razmjoo : please review

thanks for your contribution, merged.


Process.run() fails to execute command completely

Description

When using the Process.run() method to execute a command, the command fails to execute completely and the program terminates prematurely. The issue is observed when executing the following Swift code:
print("enter")
let task = try Process.run(URL(fileURLWithPath: "/usr/bin/script"), arguments: "-aq /dev/null bash -c ls".components(separatedBy: " ")) { process in
    print(process.terminationStatus)
}

task.waitUntilExit()
print("leave")

The expected behavior is that the ls command should be executed successfully and the program should print "enter", the termination status of the process, and "leave" in the console.
Steps to reproduce

Copy the above Swift code into a main.swift.
Call swift main.swift
Observe that the program terminates prematurely after printing "enter" and the termination status of the process.

Expected behavior

The ls command should be executed successfully and the program should print "enter", the termination status of the process, and "leave" in the console.
Note:
The issue appears to be related to the use of the /usr/bin/script command with the -aq options. Removing these options or using a different command appears to resolve the issue.

Environment

Swift compiler version info

swift-driver version: 1.85 Apple Swift version 5.9 (swiftlang-<IP_ADDRESS>.7 clang-15<IP_ADDRESS>)
Target: arm64-apple-macosx14.0

Xcode version info

Build version 15A5195m

Deployment target:

14.0 Beta (23A5286i)

I think this should be transferred to swift-foundation.

@AnthonyLatsis Thank you for your suggestion. I have moved this issue to https://github.com/apple/swift-corelibs-foundation/issues/4803.

swift-corelibs-foundation is the Foundation for non-Apple platforms, but I guess it’s fine.
Please try transferring the issue next time (or ask us) instead of cloning. This way we can avoid polluting the issue database.

swift-corelibs-foundation is the Foundation for non-Apple platforms, but I guess it’s fine.
Please try transferring the issue next time (or ask us) instead of cloning. This way we can avoid polluting the issue database.

I learned something new today, thanks!


Update Dev Guide and Resize Sticky

Coverage remained the same at 85.72% when pulling 3d2ac93ad449891b2dadddfa45e383db9a8dab31 on Rui_Zhong into ef2d2edd67f10acc17599f66c6f52d87aa764204 on master.


Add message when your SteamID cannot be found

A message/prompt should be displayed when your steamID cannot be found in the current lobby. This would probably clear up a lot of confusion regarding "help its not working"

Added to project Enhancements


Select assocation id using IDENTITY function inline

Hi,
I'm wondering why it isnt possible to fetch a foreign key inline with the rest of the results when hydrating to an array. Instead, it returns a mixed results as found here in the docs. Would it not be possible to include the desired field along with the rest of the results rather than the current situation? Specifically when hydrating to an array, this shouldn't be a problem I'd think.

@rpander93 as soon as you have a scalar value added to object values, the result format will change.
If you want, you can try expanding on the expected results, but I don't think this is worth inspecting/improving, as changing the behavior would be a BC break.
Closing as won't fix


Run the indicator Aimation and the Label text color change animation in parallel

Is it possible to run the label text color change animation and the indicator animation in parallel? Currently when I animate in renderLabel() and in renderIndicator(), the animations seem to run in sequence and does not look refined
react native version : 0.43.3
Librayr version : 0.0.59

Not sure what you mean. Can you post a GIF/video?

Closing due to no response


Following Auth0 guide results in 403s, misconfigured roles

Summary
Hey team,
I've configured Nebraska to leverage SSO via Auth0 following the guide at:
https://kinvolk.io/docs/nebraska/latest/authorization/#preparing-auth0-as-an-oidc-provider-for-nebraska
Unfortunately, this results in getting 403s when calling the apps/ and api/ routes, and nebraska throws several errors.
There isn't specific information on passing the configuration back to the app in this configuration, so its possible that I've gotten something wrong and this is user error. If so, my apologies.
I'm happy to clarify the docs where possible but I have a few questions regarding this guide.
Questions
1. Are there any non default grants necessary for this to work as expected?
I imagine implicit, authorization_code, and refresh_token (the defaults) should suffice, but when I go through the auth flow, the following errors are generated by nebraska:
Refresh token not found in session: oidcauth.go
Misconfigured Roles, Can't get access level from token: oidcauth.go
I believe the refresh token part to be a misnomer caused by how the logic works, ie it is leveraging the authorization_code grant first, then a subsequent request is made with the refresh_token grant.
2. Are defined scopes required for this to work?
It seems like there are no defaults for the scopes configuration, and they aren't inherited from the provider in oidcauth.go
Are these necessary? I've tried adding some of the scopes provided via the /oauth/.well-known/openid-configuration endpoint response, but often times, it results in the roles being omitted from the JWT token.
3. How should the roles be configured and passed to the app?
The JWT token returned in the auth flow has the following structure in its payload, expected from the instructions in the guide:
{
  "http://kinvolk.io/roles": [
    "admin",
    "viewer"
  ],
  ...
}

And the deployed application gets the following flags appended to the binary (by way of the helm chart):
            - '-auth-mode=oidc'
            - '-oidc-admin-roles=admin'
            - '-oidc-viewer-roles=viewer'
            - '-oidc-roles-path=http://kinvolk.io/roles'

Yet, the application specifies: Misconfigured Roles, Can't get access level from token.
Tracing through the application logic, it seems like this should be appropriately passed to the Authenticator through the OIDCAuthConfig through the NewOIDCAuthenticator factory (oidcauth.go:119) and then called by the middleware when authenticating a request (controller.go:186-195).
If I change the oidc-roles-path flag to the suggested default, "roles", the error no longer persists, but the roles are not found still, and the 403s persist.
Thank you for any guidance on this matter.

After a good deal of digging, I don't think the chosen library/function gjson.Get supports parsing the returned JSON in the way prescribed by the guide.
See this playground: https://go.dev/play/p/3F_j7l7b3zs
This playground emulates the rolesFromTokens function found here. When gjson.Get is passed the key http://kinvolk.io/roles prescribed in the guide, gjson.Get erroneously doesn't parse the key and value.
Unfortunately, the role key needs to be in url format.

It looks like any periods in the key should be escaped when passed to nebraska.
This perhaps this can be simply called out in the documentation as an action item, though then the onus is on the operator to either understand this or potentially spend days troubleshooting why things aren't working as intended, as was the case for me.

@goshlanguage , thanks for looking into this.
@yolossn , can you take it from here?

I'd be happy to put in a PR if it helps, however I don't know if it would be preferred to automatically escape periods from flags set for nebraska, or to have it called out in the documentation, or something else.

Hey, @goshlanguage the oidc-roles-path argument expects a JSON path, and periods are used to define the nesting if any exists in the token. Since in this case there is no nesting and periods are part of the value itself it requires escaping. The help string for oidc-roles-path says JSON path. I agree adding a note section to Auth0 documentation makes sense.

@joaquimrocha @goshlanguage I have created a PR adding a notes section to Auth0 configuration.


Deployment

Hi esausilva,
Thanks for sharing the app. It was very useful to get me started with react/express.
I would like to ask you how I could deploy manually the files to a server. Which files and folders do I have to place on the server and what needs to be the file structure.
Thanks

Are you going to serve with Node? Apache? other server tech?

I am going to serve it with Node. I am using create react app.
On Sat, Oct 12, 2019 at 3:52 PM Esau Silva<EMAIL_ADDRESS>wrote:

Are you going to serve with Node? Apache? other server tech?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/esausilva/example-create-react-app-express/issues/6?email_source=notifications&email_token=AI5DHP6KUFWH2QZH6OTOZCTQOHQEVA5CNFSM4I7H633KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBCBDFQ#issuecomment-541331862,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI5DHP4DDDIFW64UZE7DYSTQOHQEVANCNFSM4I7H633A
.

CRA creates a build folder. Copy that folder to your server and point your Node app to server that folder.
If you look at my Node server in Line 25 you can see how I'm doing it. You will also need lines 28-30 to redirect all requests back to React (https://github.com/esausilva/example-create-react-app-express/blob/master/server.js#L25-L30)

Thanks.
If I understand correctly you are serving the client/build folder in the
server. What if I were to past the files in the build folder created by CRA
directly on the top directory of the server. How would that impact line 25
and 29.
On Tue, Oct 15, 2019 at 9:11 PM Esau Silva<EMAIL_ADDRESS>wrote:

CRA creates a build folder. Copy that folder to your server and point
your Node app to server that folder.
If you look at my Node server in Line 25 you can see how I'm doing it. You
will also need lines 28-30 to redirect all requests back to React (
https://github.com/esausilva/example-create-react-app-express/blob/master/server.js#L25-L30
)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/esausilva/example-create-react-app-express/issues/6?email_source=notifications&email_token=AI5DHP3Y3PSQWRA7NUX7433QOYPYJA5CNFSM4I7H633KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBKCANA#issuecomment-542384180,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI5DHP4R6E5LKWNDQRQUIJTQOYPYJANCNFSM4I7H633A
.

That is correct. I am serving the client/build folder.
You mean file structure like below?
|-server.js
|-static/
|---css/
|---js/
|--...
|-index.html
|-...

Then you would have the following
...
app.use(express.static(__dirname);
app.get('*', function(req, res) {
  res.sendFile(path.join(__dirname, 'index.html'));
});

Thanks a lot :)
On Wed, Oct 16, 2019 at 10:22 PM Esau Silva<EMAIL_ADDRESS>wrote:

That is correct. I am serving the client/build folder.
You mean file structure like below?
|-server.js
|-static/
|---css/
|---js/
|--...
|-index.html
|-...
Then you would have the following
...
app.use(express.static(__dirname);
app.get('*', function(req, res) {
res.sendFile(path.join(__dirname, 'index.html'));
});
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/esausilva/example-create-react-app-express/issues/6?email_source=notifications&email_token=AI5DHPYBFEGKOXZQ5F3XZM3QO6AXRA5CNFSM4I7H633KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBN7HEY#issuecomment-542897043,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI5DHP74H7MZBRQ5NAUO553QO6AXRANCNFSM4I7H633A
.

You are welcome


Remove redundant Metrics.movedStockOverview field

superseded by movedBoxes query
was incorrect anyways (no change of box state taken into account,
only last_modified_on date)
box IDs had to be added to test case whenever a new test box was added

 2.0 Remove redundant Metrics.movedStockOverview field


Module not found

Hey there,
I am a mega noob. But did everything you did in the installation on youtube.  Whenever I want to start the application, I get this error>
Traceback (most recent call last):
File "./osmedeus.py", line 9, in
from lib.core import utils
File "/root/Desktop/Osmedeus/lib/core/utils.py", line 24, in
import tldextract
ModuleNotFoundError: No module named 'tldextract'
Seems like I miss the tldextract module. I cannot seem to find it on github either. Could you please help me ? :)

Did you use python 3.7?
Check your version by using this command python3 --version

run
pip3 install tldextract
it solver the problem for me

Nice, will test to add to requirements.txt. It's defaulted on my machine 😃

root@MrCyguru:/home/cyguru/Desktop/scan/Osmedeus# python osmedeus.py -h
Traceback (most recent call last):
File "osmedeus.py", line 9, in
from lib.core import utils
ImportError: No module named core
root@MrCyguru:/home/cyguru/Desktop/scan/Osmedeus# pip3 install utils
Requirement already satisfied: utils in /usr/local/lib/python3.8/dist-packages (1.0.1)
root@MrCyguru:/home/cyguru/Desktop/scan/Osmedeus#


Org switcher moves position when viewed from within an org view

No better way to explain this other than with a screenshot.

This is part of the CSS, so it's the intended look. Relevant CSS:
/* move the dashboard organization switcher to the right column */
.account-switcher {
	position: absolute !important;
	right: 10px !important;
	margin-top: 0 !important;
}
.account-switcher button {
	width: 313px !important;
	z-index: 0 !important;
}
.user-repos {
	position: relative !important;
	top: 50px !important;
}
/* organization dashboard */
.orgpage .pagehead-nav {
	float: left !important;
}
.orgpage .account-switcher {
	right: 0 !important;
}
.orgpage .user-repos {
	position: static !important;
}
#org_your_repos {
	top: 0 !important;
}

I'm now seeing this which is problematic:

// @bfred-it ⬆

It's fixed now, perhaps github changed something in their CSS.


Unable to load assets in editor

When I select an asset like a tree, I see in the dialog the tree rotating, but when I try to import, nothing happens.  I can supply screenshots if necessary.

When imported successfully assets show show up in the assets area at the bottom of the editor.
If you click on the model asset you can then place models in your scene by clicking the "Activate model placement tool" button in the inspector

Thank you!  I will really need to study the code to learn how to use this editor.

Yeah, writing a basic manual is still on my TODO list ;)


Audio Keep Repeating

There is an issue the Audio is keep repeating, not only the audio even javascript code
This is the code
						if(response.replysound != 'undefined' && response.replysound){ 							alert(14); 								var audio = new Audio('./sound.mp3'); 								audio.play(); 						}
Explain: When I enter the page thats contain Audio or any code JS. and go another page like home, I wait 5 seconds and Its keep Alerting 14, and playing audio! I've tried everything I can but its not work. I hope you guys fix it.

You need to show some more code if you want help.

Well, thats it https://hastebin.com/asodarepaw.js
as u can see in line 84 there is an alert , audio. What is the problem? I'll tell u while moving from page to page using pjax its execute the line 84 etc.