Remove install - deprecated and remove open-vm-tools

To get the root fs to build I had to remove open vm tools which depends on fuse-libs. I also removed install since it is deprecated. After making these changes I could get a successful build of the root fs to create container from scratch.
Checklist:

[x] Have you added an explanation of what your changes do and why you'd like them to be included?
[x] Have you updated or added documentation for the change, as applicable?
[x] Have you tested your changes on all related environments with successful results, as applicable?

Type of Changes:

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

What is the current behavior? (link to any open issues here)
N/A
What is the new behavior (if this is a feature change)?
The container can build a root filesystem without erroring that open-vm-tools cannot be installed due to dependency of fuse-libs.
Other information:

/assign @jobcespedes

/ok-to-test

Hi @svenska-primekey, thanks for this PR. I took a look at package manifest of the last build and open-vm-tools was not being installed before. I am wondering why it is installed on your environment. Any idea?

I'm not sure where it is coming from, but this is the error that I received:
2021-11-24 18:32:22,342: livemedia-creator v28.14.63-1
2021-11-24 18:32:22,342: selinux is Disabled
2021-11-24 18:32:22,398: disk_img = /var/tmp/rocky8-minimal.tar.xz
2021-11-24 18:32:22,398: Using disk size of 2MiB
2021-11-24 18:32:22,399: Running anaconda.
2021-11-24 18:32:25,907: Processing logs from ('<IP_ADDRESS>', 50198)
2021-11-24 18:32:25,950: Starting installer, one moment...
2021-11-24 18:32:25,950: terminal size detection failed, using default width
2021-11-24 18:32:25,950: anaconda <IP_ADDRESS>-1.el8 for Linux 8 (pre-release) started.
2021-11-24 18:32:25,950:
2021-11-24 18:32:25,950: Some warnings occurred during reading the kickstart file:
2021-11-24 18:32:25,950: Ignoring deprecated command on line 13:  The install command has been deprecated and no longer has any effect.  It may be removed from future releases, which will result in a fatal error from kickstart.  Please modify your kickstart file to remove this command.
2021-11-24 18:32:25,950: 18:32:25 Not asking for VNC because of an automated install
2021-11-24 18:32:25,950: 18:32:25 Not asking for VNC because we don't have Xvnc
2021-11-24 18:32:31,325: Starting automated install.....
2021-11-24 18:32:31,326: ================================================================================
2021-11-24 18:32:31,326: ================================================================================
2021-11-24 18:32:31,326: Installation
2021-11-24 18:32:31,326:
2021-11-24 18:32:31,326: 1) [x] Language settings                 2) [x] Time settings
2021-11-24 18:32:31,326: (English (United States))                (Etc/UTC timezone)
2021-11-24 18:32:31,326: 3) [x] Installation source               4) [x] Software selection
2021-11-24 18:32:31,326: (https://download.rockylinux.org         (Custom software selected)
2021-11-24 18:32:31,327: /pub/rocky/8/BaseOS/x86_64/os/)
2021-11-24 18:32:31,327: 5) [ ] User creation
2021-11-24 18:32:31,327: (No user will be created)
2021-11-24 18:32:31,327:
2021-11-24 18:32:31,327: ================================================================================
2021-11-24 18:32:31,327: ================================================================================
2021-11-24 18:32:31,327: Progress
2021-11-24 18:32:31,327:
2021-11-24 18:32:31,327: Setting up the installation environment
2021-11-24 18:32:31,327: ..
2021-11-24 18:32:31,327: Configuring storage
2021-11-24 18:32:32,330: ...
2021-11-24 18:32:32,330: Running pre-installation scripts
2021-11-24 18:32:32,338: .
2021-11-24 18:32:32,338: Running pre-installation tasks
2021-11-24 18:35:34,493: ..
2021-11-24 18:35:34,493: Installing.
2021-11-24 18:35:34,493: Starting package installation process
2021-11-24 18:35:34,493:
2021-11-24 18:35:34,493: The installation was stopped due to an error which occurred while running in non-interactive cmdline mode. Since there cannot be any questions in cmdline mode, edit your kickstart file and retry installation.
2021-11-24 18:35:34,493: The exact error message is:
2021-11-24 18:35:34,493:
2021-11-24 18:35:34,493: Non interactive installation failed:
2021-11-24 18:35:34,493: Problem: package open-vm-tools-11.2.5-2.el8.x86_64 requires libfuse.so.2()(64bit), but none of the providers can be installed
2021-11-24 18:35:34,494: - package open-vm-tools-11.2.5-2.el8.x86_64 requires libfuse.so.2(FUSE_2.5)(64bit), but none of the providers can be installed
2021-11-24 18:35:34,494: - package open-vm-tools-11.2.5-2.el8.x86_64 requires libfuse.so.2(FUSE_2.6)(64bit), but none of the providers can be installed
2021-11-24 18:35:34,494: - conflicting requests
2021-11-24 18:35:34,494: - package fuse-libs-2.9.7-12.el8.x86_64 is filtered out by exclude filtering.
2021-11-24 18:35:34,494:
2021-11-24 18:35:34,494: The installer will now terminate.
2021-11-24 18:35:34,587: Running anaconda failed: process '['anaconda', '--kickstart', '/build/rocky8-minimal.ks', '--cmdline', '--loglevel', 'debug', '--product', 'Rootfs Creator', '--dirinstall', '--remotelog', '<IP_ADDRESS>:59293']' exited with status 1
2021-11-24 18:35:34,588: Shutting down log processing
2021-11-24 18:35:34,599: Install failed: novirt_install failed
2021-11-24 18:35:34,600: ERROR: Image creation failed: novirt_install failed
2021-11-24 18:35:34,600: Image creation failed: novirt_install failed

Mmm, weird.
Ok, let's add these changes. However, dockerfile contexts in this repo are generated from template folders. The kickstart file modified in this commit is the generated one. Instead, changes should go in its respective template file. In this case, it is .ansible-ci/files/templated_contexts/rocky8-minimal/rocky8-minimal.ks. Could you please drop this commit, add changes to that template file, and commit again to this PR?

Yes I will edit the template file and try this again :-)

Thanks @svenska-primekey
/lgtm


Added Java free course

Updated docs with Java free course from Udemy. Its the most popular free course with high rating i have personally used.

By submitting this pull request I confirm I've read and complied with the below requirements.
Failure to properly do so will just result in the pull request being closed and everyone's time wasted.

[x] I have read the Contribution guidelines and made sure my pull request follows it, specifically the part about promotions and markdown format.
[x] Any resources I am adding in this PR have been used by me and I can personally vouch for their quality.
[x] This pull request has a descriptive title. For example, Add [Resource Name] or Add [Topic Name], not Added new entries or update.
[x] I have double checked that I have only updated the minimum files that are required by this PR.

@jinishatejura Thank you. Merged!!


Ignore .gen/ directory for goimports

Otherwise there are 45 changed files after any invocation of make.
While goimports can change at any point, the only change affecting
cadence source from Go v1.17 is that goimports re-tabs commented
Go-code.

Make won't run goimports for .gen/ directory

Otherwise there are 45 changed files after any invocation of make.

How did you test it?

No any. I also highly in doubt we need to reformat generated code.

Release notes

Documentation Changes

if there are changes, commit them?  and double check that you're using the right Go version, as that can affect formatting, and not just in .gen.
goimports doesn't change at any point.  it's a specific version, which the makefile controls: https://github.com/uber-go/cadence-client/blob/master/Makefile#L146

@Groxx changed a bit. I think depends on go version, isn't it? Otherwise why it's one format on github and another on local machine with go 1.20?
postgresql-related tests are failing.
If you OK with it, can you merge?

It does, but we have a stated "dev" version, and that's checked in the makefile (printed every time), because it needs to match CI for stuff like this. Which affects everything, not just .gen.
Our options are basically "use the correct Go version" or "ignore formatting, and ignore automated reformatting in PRs, because it's gonna happen a lot".

Closing as part of old PR cleanup effort.


check existence of machine config before further operations

Issue #, if available:
#6039
Description of changes:
check existence of machine config before further operations

/ok-to-test

@chrisdoherty4 could you please take a look at this?

@HiteshRepo Apologies for the slowness in the last few hectic weeks. We'll take a look shortly.
/cc chrisdoherty4 jacobweinstock

/ok-to-test

/lgtm

Looking great @HiteshRepo, thank you for your submissions.

/lgtm

/approve

/retest

/cherry-pick release-0.17


Postfix removal is a bit too greedy

Hey, thanks for the nice project, helps me a lot!
I'm using it to generate bindings to xcb, and I'm using options.enumValueNameRemovePostfixes = []string{"_t"}; just like in the example generator. Unfortunately, it removes not only _t from enum values, but also just T at the end of the enum value, so that
XCB_WINDOW_CLASS_INPUT_OUTPUT becomes WINDOW_CLASS_INPUT_OUTPU - note no T at the end. I've tried it on the latest master.
UPD:
it's not just options.enumValueNameRemovePostfixes = []string{"_t"};, it's actually a combination of these two that bugs out:
    options.enumValueNameRemove = true;
    options.enumValueNameRemovePostfixes = []string{"_t"};

also full generator, just for reference:
package main

import "bindgen"

main :: proc() {
    options : bindgen.GeneratorOptions;

    options.functionPrefixes = []string{"xcb_"};

    options.definePrefixes = []string{"XCB_"};
    options.defineCase = bindgen.Case.Constant;

    options.pseudoTypePrefixes = []string{"xcb_"};

    options.enumValuePrefixes = []string{"XCB_"};
    options.enumValueNameRemove = true;
    options.enumValueNameRemovePostfixes = []string{"_t"};

    bindgen.generate(
        packageName = "xcb",
        foreignLibrary = "system:xcb",
        outputFile = "./examples/xcb/generated/xcb.odin",
        headerFiles = []string{"./examples/xcb/headers/xcb.h", "./examples/xcb/headers/xproto.h"},
        options = options,
    );
}

Thanks for reporting, can you confirm this is fixed with latest commit?

Fixed, yeah, thanks!


EKS 1.27:  systemd dependency tree not updating when launching new pods?

As a part of our upgrade to eks 1.27 / containerd, we have been attempting to migrate an existing custom "node drain" service to our new nodes. This is a bit of a hack, but it's worked quite well for us for a couple years - basically what it does is load a service on startup which sits in front of the dockerd/containerd and kubelet, and prevents pod shutdown on system halt for a period of time (for the replacement node to come online) and then does a kubectl node drain <self> to gracefully evict the running pods. More info here: https://faun.pub/graceful-draining-secret-to-node-rotation-in-eks-4d85e1882720
This all worked well enough in our previous 1.23 cluster under dockerd, however I've noticed some strange behaviour in 1.27/containerd.
Basically, the service works as expected (after updating to be dependent on containerd.service) in the sense it prevents containerd/kubelet from stopping, however, the pods themselves still stop (and go to a Completed state), upon node halt.
Eventually I realized that if I issued a systemctl daemon-reload, it would "grab" the running pods, and they would stay running until the service exited as expected. However, any new pods added to the host would be again exempt from the process. For the time being, I've added a cron to do the daemon-reload every minute and things are working, but it's a hack and I'd like to understand what's actually going on.
Have I missed a new dependency? I attempted to hook the service into kubepod.slice, kubepods-burstable.slice and kubepods-besteffort.slice to no avail. Attempting to determine the exact dependencies via systemctl list-dependencies hasn't really illuminated anything for me, the processes that appear to be stopping prematurely (as per the console output on shutdown) are the cri-containerd-<uid>.scope objects, and they don't appear live beneath anything? Obviously I'm not a systemd expert, so I might be missing something obvious here.
Any insight appreciated! TIA

Environment:

AWS Region: us-west-2
Instance Type(s): t3a.medium
EKS Platform version: eks.4
Kubernetes version: 1.27
AMI Version: amazon-eks-node-1.27-v20230816
Kernel: 5.10.186-179.751.amzn2.x86_64
Release information:

BASE_AMI_ID="ami-0a6c4ce5a4a59e9e2"
BUILD_TIME="Wed Aug 16 02:57:51 UTC 2023"
BUILD_KERNEL="5.10.186-179.751.amzn2.x86_64"
ARCH="x86_64"

I probably can't do much to debug your custom hook, but I would consider moving that logic off of the node. Kubelet's graceful shutdown behavior would help to some degree, but the older systemd we have on AL2 doesn't support it -- that will come with AL2023 in the medium term.


Adding includePaths in the stylePreprocessorOptions seems to have no effect

[X] bug report

Description
Adding includePaths in the stylePreprocessorOptions entry in angular-cli.json's apps object seems to have no effect.
Versions.
@angular/cli: 1.0.0
node: 6.10.0
os: linux x64
@angular/common: 4.0.1
@angular/compiler: 4.0.1
@angular/core: 4.0.1
@angular/forms: 4.0.1
@angular/http: 4.0.1
@angular/platform-browser: 4.0.1
@angular/platform-browser-dynamic: 4.0.1
@angular/router: 4.0.1
@angular/cli: 1.0.0
@angular/compiler-cli: 4.0.1
@angular/language-service: 4.0.1

Repro steps.

ng new new-project --style=scss
cd new-project
create new _variables.scss in src/styles
add the following to src/styles/_variables.scss:

$some-variable: #fff;

add the following to src/app/app.component.scss:

@import 'variables';

h1 {
  color: $some-variable;
}

add the following to .angular-cli.json's apps property:

      "stylePreprocessorOptions": {
        "includePaths": [
          "styles"
        ]
      },

run ng serve

The error log
ERROR in ./src/app/app.component.scss
Module build failed:
@import 'variables';
^
      File to import not found or unreadable: variables.
Parent style sheet: stdin
      in ~/PROJECTS/new-project/src/app/app.component.scss (line 1, column 1)
 @ ./src/app/app.component.ts 56:17-48
 @ ./src/app/app.module.ts
 @ ./src/main.ts
 @ multi webpack-dev-server/client?http://localhost:4200 ./src/main.ts

Desired functionality
https://github.com/angular/angular-cli/blob/5b1668f9ba39175e0f57fe23df6672ec40c3b745/docs/documentation/stories/global-styles.md

it works for me..

it doesn't work for me :
angular/cli: 1.0.1
node: 6.9.5
os: darwin x64
@angular/animations: 4.1.0
@angular/common: 4.1.0
@angular/compiler: 4.1.0
@angular/core: 4.1.0
@angular/forms: 4.1.0
@angular/http: 4.1.0
@angular/platform-browser: 4.1.0
@angular/platform-browser-dynamic: 4.1.0
@angular/router: 4.1.0
@angular/cli: 1.0.1
@angular/compiler-cli: 4.1.0

I can reproduce the OP's problem. No matter how simple (I tried both with the node_modules folder and a folder inside my app) I try to make the includePaths property inside of stylePreprocessorOptions, I can't get it to work and it will always throw the File to import not found or unreadable: error.

@jgatjens I still have the same issue, can you show us you angular-cli.json with the stylePreprocessorOptions to see how your configuration.

@AlejandroMAL it didn't work for me.

I'm having this issue as well, heres my the relevant part in angular-cli.json:
  "stylePreprocessorOptions": {
    "includePaths": [
      "./src/Assets/Scss/bourbon/bourbon",
      "Assets/Scss/neat/"
    ]
  }```

Note: Not sure if it was a relative or full path so tried both.

Im having the same issue, its really annoying. @filipesilva Any updates on this bug?

@filipesilva I have updated the Angular CLI, tested on the new clean project and everything seems working for me now.
This is the project and commit that adds desired functionality.
https://github.com/vmasek/example-angular-cli-include-paths/commit/06b1731c59e794d2ff0921e4bde3bcd23717d5c5
I've also tested the src (.) and styles include paths. Both of them are working.

Closing then, thanks for getting back to me @vmasek.

I followed @filipesilva  example at https://github.com/vmasek/example-angular-cli-include-paths/commit/06b1731c59e794d2ff0921e4bde3bcd23717d5c5. srestarted ng serve it worked. Does not work if you do not restart ng serve

@filipesilva This doesn't want to work with a multi app setup :(

@realappie can you open a new issue with a reproduction so we can investigate?

@gdheller42 's comment is important here. You must restart ng serve after changing the .angular-cli.json, or you'll keep getting the compilation error.

@filipesilva Unfortunately this is not working on Angular CLI v1.3.0-rc.3. I copied your code with no success. Could you please retest?
src/styles/_variables.scss
$grid-gutter-width-base: 30px;

.angular-cli.json
"stylePreprocessorOptions": {
        "includePaths": [
          ".",
          "styles"
        ]
      },

src/app.component.scss
.container-fluid {
	margin-bottom: $grid-gutter-width-base;
}

Webpack:
Module build failed:
        margin-bottom: $grid-gutter-width-base;
               ^
Undefined variable: "$grid-gutter-width-base".

@MickL Did you import the _variables.scss with @import '_variables';?

Oh! So even when setting includePaths we have to import the files anyway?

@vmasek is it possible to put the code on github again? I'm struggling with angular-cli and sass and an example would be great.
Thanks!

MickL's example should be sufficient. Are you missing anything?

it seems to be working :) I've realized that it's a problem with IDEA, it doesn't recognize the imported file but if I do ng serve everything works fine.


Move to context from x/net/context in generated code

I am not sure I regenerated everything correctly see #1720
The examples were regenerated but their imports were wrong :
diff --git a/examples/authentication/cmd/auth-sample-server/main.go b/examples/authentication/cmd/auth-sample-server/main.go
index 7150b413..47537a72 100644
--- a/examples/authentication/cmd/auth-sample-server/main.go
+++ b/examples/authentication/cmd/auth-sample-server/main.go
@@ -7,8 +7,8 @@ import (
        "os"

        loads "github.com/go-openapi/loads"
-       "github.com/go-swagger/go-swagger/examples/authentication/restapi"
-       "github.com/go-swagger/go-swagger/examples/authentication/restapi/operations"
+       "github.com/go-swagger/restapi"
+       "github.com/go-swagger/restapi/operations"
        flags "github.com/jessevdk/go-flags"
 )

I couldn't figure out why so I just didn't commit them :)

did you use the sample regen script? AFAIK there is a small test that at least all examples build.

I am using this go-bindata: github.com/jteeuwen/go-bindata

I just tried these changes locally.
I installed the same go-bindata you're using and I ran the ./hack/regen-samples.sh script
The changes it makes seem all to be expected

@MStoykov you just need to rebase and regenerate. I've pushed a PR that does just that on your own fork.


Add these books to python

They say start here 1st - Programming Python, 4th Edition - https://www.oreilly.com/library/view/programming-python-4th/9781449398712/
then 2nd - The Python Standard Library by Example - https://www.amazon.com/dp/0321767349
then
Python in Practice   https://www.amazon.com/dp/0321905636
Python Cookbook, 3rd Edition  https://www.oreilly.com/library/view/python-cookbook-3rd/9781449357337/
Writing Idiomatic Python https://www.jeffknupp.com/writing-idiomatic-python-ebook/

look into it.


Critical issue In new gradle when I switch second category and again first.

when I switch the second category and again the first category then first subcategory change by the second subcategory.

When I touch the any category, the sub-categories don't change. Also, if the categories changes quickly, the sub-categories are mixed.

I fixed this problem. Pull latest version and use:
compile 'com.github.iamvaliyev:MultiTabMenu:v1.0.1'
Happy Coding! 😃


drafts: Automatically delete misbehaving drafts while formatting.

While applying formatting to drafts if any draft contains some syntax
which our markdown processor is unable to process delete the draft so
that drafts overlay can be opened without any error. Also report the
exception to the server so that error can be fixed.

This looks reasonable; a couple issues:

It appears that PMs might have not called markdown.apply_markdown before; I guess that was a bug before and we just showed raw content in PMs?  Worth double-checking.
We shouldn't send the draft content to the server in a JS exception; that doesn't comply with Zulip's privacy model (we don't want developers to ever have to see message content to debug things).  We should use the _privacy_re / _sanitize_for_log logic from bugdown for this (will need to copy to reimplement in JS).  I'd add a unit test or two for the new function as well.

It appears that PMs might have not called markdown.apply_markdown before; I guess that was a bug before and we just showed raw content in PMs? Worth double-checking.

They did, I just clubbed them together.(diff is a bit a confusing there)

We shouldn't send the draft content to the server in a JS exception; that doesn't comply with Zulip's privacy model (we don't want developers to ever have to see message content to debug things). We should use the _privacy_re / _sanitize_for_log logic from bugdown for this (will need to copy to reimplement in JS). I'd add a unit test or two for the new function as well.

Ah didn't thought that way, will get it done.

@timabbott I just checked and found that _privacy_re can't be re-implemented as javascript regex are not unicode aware. Should we consider adding some third party javascript regex library like xregexp?

How big is xregexp?  Lack of unicode support has been a problem before, so it's interesting, but we want to avoid shipping too many large libraries.

I decided we can just do the filtering on the backend; so I merged this with b3b5d5b7cd53aadc501884b7773eb725f2ff900a on top.  Thanks @HarshitOnGitHub!


Programs stop running after 70 minutes

Programs stop running after 70 minutes because, I presume, the timer on the board being used to obtain physical time is a 32-bit unsigned integer, and after about 70 minutes, it overflows.

Since this is also an issue with Arduino as well, I'll take this up as a separate task.

I have fixed this for the lf-buckler. There, time stamps are 64 bits, so the only thing that was needed was to detect when the physical timer measuring real time, which has only 32 bits, overflowed. My implementation assumes that the program accesses physical time at least once every 35.5 minutes (it would be better to enforce that).

Closed via commit https://github.com/icyphy/lf-buckler/commit/c808312a26aa4a8f75f0cdb45bd2a563f53a9678


docs: 'updated description for 1 rule'

Closes #389
Signed-off-by: Ali Mukadam<EMAIL_ADDRESS>The ports that are opened are from a range that are accessible only from within the worker subnet i.e. the source and destination of ingress and egress rules respectively is the worker subnet itself. This is to allow pods to communicate to each other. See the documentation here: https://docs.oracle.com/en-us/iaas/Content/ContEng/Concepts/contengnetworkconfig.htm#securitylistconfig

@denismakogon


"word-wrap: break-word;" breaks Arabic text

Hello
There is a bug the breaks Arabic output if word-wrap: break-word; style applied
https://jsfiddle.net/y7v5upd1/

Cannot reproduce in Chrome?
What browser did you use?
Version everything?

I'm using latest Chrome 80.0.3987.132 64-Bit
I also have tested on Firefox 74.0 with the same results
I've tried two computers with Windows 10 and Windows 7 also
If you don't read Arabic maybe you won't recognize that it have problem
try to compare the letters drawings and you can recognize it's broken when word-wrap: break-word style is applied

I have the same problem.


Update build settings

ref.
https://github.com/scala-js/scala-js/pull/2395
https://github.com/scala-js/scala-js/issues/2579

👍


chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2023-02-10)
Features

add desktop template (d42c300)
complete web(react) (29b25b7)
support web & desktop create (1e2a192)
support web & desktop create (15a6e4c)

Bug Fixes

fix the problem about the non-existence of pnpm in the lint action (f7fe68b)

Styling

update code styles (52396aa)

Code Refactoring

add .gitignore (d58ed73)
add gitignore (11539d4)
delete .DS_Store (fdf60ed)

CI

add lint action (9250ad0)
create an automated publishing process (bd09bac)
update lint ci (29d6c3a)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/kaishens-cn/cli-tools/releases/tag/v1.0.0 :sunflower:


Warning messages in critical steps

As described in #284
Include warning messages during:

[x] Creation of wallet(s)

[x] Getting new addresses

[x] Importing new private keys (single)

[x] Importing new private keys (file)

[x] Backup of wallets

[ ] Translations

[ ] ...

Closed. See #290


An inert lense

I think there should be a way to have inert lenses. Lenses that basically dont do anthing, but you can call lense functions on them. For example.
R.view(L, anything);            //=> anything
R.set(L, something, anything);          //=> something

And also an unwritable inert lense
R.view(L, anything);            //=> anything
R.set(L, something, anything);          //=> anything

Interesting. What are the use cases? Does Haskell's lens library support "inert" lenses?

These are fairly straightforward to implement.
var idLens = f => s => f(s);
// or
var idLens = R.unary;

var constLens = f => s => R.map(R.always(s), f(s));
// or
var constLens = f => R.converge(R.map, [R.always, f]);

I am also curious of the use case though :)

The constLens unfortunately doesn't satisfy the "put then get" law:
R.view(constL, R.set(constL, 42, 1)) === 42; // false

Well I'm messing around with Redux -- have you guys used it? Anyways, this issue explains what the end goal: https://github.com/rackt/redux/issues/822
I was thinking about using a lens for these "high-order" reducers that abstract other reducers. This probably means nothing to you, but check it out:
const list = (state=[], action) => (reduce) => {
  switch (action.type) {
    case LIST_INDEX:
      return R.lensIndex(action.index)
    default:
      return R.inertLens
  }
}

In your example, assuming you just want the lens that you described as:
R.view(L, anything);            //=> anything
R.set(L, something, anything);  //=> something

You could try:
const list = (state=[], action) => (reduce) => {
  switch (action.type) {
    case LIST_INDEX:
      return R.lensIndex(action.index)
    default:
      return R.unary; // inert lens
  }
}

Ah. Sweet!


Have deep linking work only on posts

Options: https://www.bryanbraun.com/anchorjs/#color-transition


Make all created editors globally available

It would be nice to have access to any Quill editors after they are created, for calling their methods etc.
I've solved this by creating my own app config with its own widget template that has these extra lines after instantiating the editor:
var editor = new QuillDjango(id, editorSelector, toolbarSelector, imageModuleOpts, theme);
window.DjangoQuillEditors = window.DjangoQuillEditors || {};
window.DjangoQuillEditors[id] = editor;
var form = document.querySelector('form');

Now I can get at an editor using the original element id as key:
var editor = window.DjangoQuillEditors.id_body.quill;
// ...do stuff with the editor object.

Is this feature interesting enough to include in django-quill? If so, let me know and I'll open a proper pull request for it.

This sounds like a great idea! I would definitely accept a pull request for this.


Adds an optional writeCompletion callback

I often need to write files that I'm lazily reading out of an NSInputStream. The addition of a writeCompletion callback to the write(data:) method lets me smartly/lazily frame up the next chunk to send as soon as the socket is ready for more.
That's what I'd want it for anyway :-)

Looks good! Other than that one comment, can we also add the completion closure to writeString and writePing to be consistent? Thanks!

Oops, yes I had meant that to default to nil! I've updated writeString and writePing as well, good idea.
I did not update the comment for writePing as I'm not 100% sure what's going on there :-).
Thanks for the feedback.

Awesome. I don't think writePing is all that commonly used and I would like clean up the rest of the docs anyway so I think we are good. Thanks again!

@acmacalister Do you think it's feasable to release a new version with this feature soon?

@lhecker new release published (1.1.3) with this addition.

Thanks @acmacalister! This new feature is really useful to correctly handle backpressure. :blush:


Missing link to Custom Components in README's ToC

Custom components were introduced per the following PR/commit but lack the add to the Table of Contents:

https://github.com/gregnb/mui-datatables/pull/1167
https://github.com/gregnb/mui-datatables/pull/1300/commits/12c08a575dc327f7766b2982267d170d756893e0

Coverage remained the same at 71.862% when pulling 30c2362aa86bbfe0b7f72d818aef960c88bb9970 on mxmlnglt:readme-toc-link-custom-components into a5b15fff80793d7ce65f3c66c2ae957d47b7075f on gregnb:master.


Select: Support option groups

Bug, feature request, or proposal: Feature request
What is the expected behavior?
First of, I really like the MD2-Select, it's sleek and functional.
I'm missing a little feature, though: option groups.
In my case, I want to group option elements (e.g. locations by country).
What is the current behavior?
Currently, there's no way AFAIK to group options together.
What are the steps to reproduce?
Providing a Plunker (or similar) is the best way to get the team to see your issue.
Plunker template: http://plnkr.co/edit/rQmUz8WYRh5Vz4gwY8E4?p=info
What is the use-case or motivation for changing an existing behavior?
Option groups can be used to group similar or connected options.
Which versions of Angular, MD2, OS, browsers are affected?
Is there anything else we should know?
The Materialize.CSS framework has a feature like this: http://materializecss.com/forms.html#select

duplicate issue #129


Unable to Login from Windows Emulator and Windows Phone

Hi,
We are able to login to RHPlay and SIT2 environment, when we run UsernamePasswordAsync method from console application. We have used the similar implementation for Xamarin application that is running on windows Emulator or windows phone. The same code gives error as shown in below snapshot.

Here is the code implementation.

Please suggest.

Can you run a Fiddler trace and share the exact response from Salesforce?

Hi Wade,
Thank You for replying. This was due to our network settings. I have
disconnected from VPN and run the emulator. it works for me.
Thank You
Narendra
On Mon, Sep 7, 2015 at 11:22 AM, Wade Wegner<EMAIL_ADDRESS>wrote:

Can you run a Fiddler trace and share the exact response from Salesforce?
—
Reply to this email directly or view it on GitHub
https://github.com/developerforce/Force.com-Toolkit-for-NET/issues/139#issuecomment-138335030
.

--
Thanks & Regards,
Narendra K Meena
502-994-4825


mix with CoreImage

Hello! Is it possible to integrate the filters from CoreImage in Metal so that they are processed in the GPU and use in a chain with MTIFiltres. This question does not apply to this library, but I'm interested in knowing your opinion. Thanks!

Yes. You can use MTICoreImageKernel or MTICoreImageUnaryFilter.
https://github.com/MetalPetal/MetalPetal/blob/master/Frameworks/MetalPetal/MTICoreImageExtension.swift


Crash and can't run again Mac OS Mojave

Hi there,
After crashing (once again), HotSpotter simply doesn't work anymore on Mac OS 10.14.6.
The icon appears in the dock and disappears. I've removed the app, restarted and re-installed, still the same issue... Nothing has been updated in between...
Any idea how to solve this?
Thanks!

Hi @StephPeriquet
I'm currently not maintaining this repo, nor do I have a mac to test this on. However if you can find and fix the issue I'll accept a PR.

Hi,
Ok that doesn't help much but I'll certainly look for the issue, although I really have no idea where to start ;-)

UPDATE: I found the problem.
The database I was working on last seems to be corrupted and as HotSpotter was trying to open it at start up it just crashed.  What happened and how did it get corrupted, I have no idea...
I simply changed the name of the database parent directory and HotSpotter launches without starting a database. Luckily I had made a backup copy of the database in question...
Anything I can provide you with if you want to have a look at what happened?


Standalone release are not available anymore

Documentation still states that we can download standalone versions but they are not produced anymore.
https://github.com/KaotoIO/kaoto.io/blob/main/content/docs/quickstart.md#using-a-standalone-executable

Hi @rodcloutier, thanks for reporting this issue, I'm updating the quickstart to point to the backend repository which has the binaries.
In addition to that, I would like to point out that we're working on the next version of the project and you can see it here: https://kaotoio.github.io/kaoto-next/
Here's the repository if you're interested 😃: https://github.com/KaotoIO/kaoto-next/


Add multiplicative method

Codecov Report

Merging #2 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master     #2   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          56     91   +35
=====================================
+ Hits           56     91   +35

Flag
Coverage Δ

#unittests
100% <100%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

holtwinters.go
100% <100%> (ø)
: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 20843c1...5330a65. Read the comment docs.


Flatten out project

Flatten fileutil, logutil, cli etc. into their corresponding used areas.

https://github.com/cboin1996/simplepipe/commit/f4fbde263aacbaad2a2f2b9a1a173e1e46369f71 to address this.


Stun server No route to host with host

Hello,
I'm trying the sample project, but I'm facing with some issue after creating the room :
(stun_port.cc:604): UDP send of 20 bytes to host stun3.l.google.com:19302 (142.251.125.x:19302) failed with error 0 : [0x00000041] No route to host (stun_port.cc:604): UDP send of 20 bytes to host stun2.l.google.com:19302 (64.233.164.x:19302) failed with error 0 : [0x00000041] No route to host (stun_port.cc:604): UDP send of 20 bytes to host stun.l.google.com:19302 (173.194.76.x:19302) failed with error 0 : [0x00000041] No route to host (stun_port.cc:604): UDP send of 20 bytes to host stun4.l.google.com:19302 (74.125.27.x:19302) failed with error 0 : [0x00000041] No route to host (stun_port.cc:604): UDP send of 20 bytes to host stun1.l.google.com:19302 (142.251.27.x:19302) failed with error 0 : [0x00000041] No route to host (stun_port.cc:604): UDP send of 20 bytes to host stun.l.google.com:19302 (173.194.76.x:19302) failed with error 0 : [0x00000041] No route to host (stun_port.cc:604): UDP send of 20 bytes to host stun2.l.google.com:19302 (64.233.164.x:19302) failed with error 0 : [0x00000041] No route to host (stun_port.cc:604): UDP send of 20 bytes to host stun1.l.google.com:19302 (142.251.27.x:19302) failed with error 0 : [0x00000041] No route to host (stun_port.cc:604): UDP send of 20 bytes to host stun4.l.google.com:19302 (74.125.27.x:19302) failed with error 0 : [0x00000041] No route to host (stun_port.cc:604): UDP send of 20 bytes to host stun3.l.google.com:19302 (142.251.125.x:19302) failed with error 0 : [0x00000041] No route to host
The room is well created in Firebase, "SDP offer sent" is well displayed on the screen. But after some seconds "Connection timeout" is displayed. It looks like the stun servers are not available ? Or maybe I don't understand the WebRTC step?
Tested on wifi or LTE network and no VPN
In any case thanks for this very well-explained repository which allows us to understand webrtc well! :)

Hi, @Goule . Thx for heads up - I will take a look at this

Tanks @grbaczek!

Try pulling the recent version off the code. I fixed a bug experienced while reestablishing the connection.
I tested the app - worked fine on my devices.
Google STUN servers are used. You need TURN server if one of the devices are behind firewall - make sure both of your devices are on the same network.
There could be an issue with your signeling server - I created a new branch: working_signaling_server with my firebase signalling server against which I've just tested the app. Try testing with this branch
PS. Remember to star a repo if it helped you anyway 😄

Soon good, there was an issue on my side with my router and some restrictions!
Done for the star :)


Issue with k8s.io/docs/setup/production-environment/container-runtimes/

Greetings,Running CentOS 8...
This is what the docs say:
systemd
To use the systemd cgroup driver, set plugins.cri.systemd_cgroup = true in /etc/containerd/config.toml. When using kubeadm, manually configure the cgroup driver for kubelet
So this is what I did:
disabled_plugins = ["cri"]
pgodfrin, per https://kubernetes.io/docs/setup/production-environment/container-runtimes/
plugins.cri.systemd_cgroup = true
This caused these errors:
-- Unit docker.service has begun starting up.
Jul 19 10:07:45 k8master.local containerd[14869]: containerd: Near line 2 (last key parsed ''): bare keys cannot contain '.'
Jul 19 10:07:45 k8master.local systemd[1]: containerd.service: Main process exited, code=exited, status=1/FAILURE
Jul 19 10:07:45 k8master.local systemd[1]: containerd.service: Failed with result 'exit-code'.
Jul 19 10:07:45 k8master.local systemd[1]: Stopped Docker Application Container Engine.
-- Subject: Unit docker.service has finished shutting down
-- Defined-By: systemd
-- Support: https://access.redhat.com/support
-- Unit docker.service has finished shutting down.
And:
-- Unit docker.service has begun starting up.
Jul 19 09:53:05 k8master.local containerd[13318]: containerd: Near line 2 (last key parsed ''): bare keys cannot contain '.'
Jul 19 09:53:05 k8master.local systemd[1]: containerd.service: Main process exited, code=exited, status=1/FAILURE
Jul 19 09:53:05 k8master.local systemd[1]: containerd.service: Failed with result 'exit-code'.
Jul 19 09:53:05 k8master.local systemd[1]: Stopped Docker Application Container Engine.
-- Subject: Unit docker.service has finished shutting down
-- Defined-By: systemd
-- Support: https://access.redhat.com/support
-- Unit docker.service has finished shutting down.
Changed the code to this:
pgodfrin, per https://kubernetes.io/docs/setup/production-environment/container-runtimes/
[plugins.cri]
systemd_cgroup = true
It now works!

/sig cluster-lifecycle

@pgodfrin would you be willing to open a pull request with the fixes? It sounds like you have a good idea of what these would be.

[plugins.cri]
systemd_cgroup = true

this change seems fine. PR welcome.


Add interactive execution of commands

This PR adds a few commands to execute things from docker-compose files, via interactive selection. You can select a command to run on all services, or just a specific one.  The selection itself is using completing-read, so it should hook into helm, ivy, ido or whatever the system of the future might be.
The commands use a predefined list (gotten from copy-pasting from docker-compose help, hehe) and the services use docker-compose config --services to get a list of the services.
A special string docker-compose-all-services-token, which defaults to "*all services*" is prepended to the services list. When selected, it signals that the command is to be run on all services (e.g. with no further arguments to the command line).
Glorious gif |o/

Please excuse the slight jittery nature of the gif. It's literally the first gif I've ever made, and I hope it gets the point across anyway.
For trivia, I've based the code and functionality on my similar makefile-executor.el project, which does this exact thing but on Makefile targets.
Closes #19 since it contains an improved version of it.

Check out functionality on Silex/docker.el package.

Check out functionality on Silex/docker.el package.


chore: commits specification

specification  commits message [Link]

Looks really cool. Let me dig more into understanding it.


Merging bookmarks with HN favorites

The current bookmark system is purely local but I think at least one other HN app has managed to hook into the HN favorites system for saving bookmarks to a user. This should be added (and made the default for logged in users), perhaps the terminology should also be changed in that case so that it says "Favorite" for logged in users and "Bookmark" for regular users.

I believe there is no API for bookmarks, but here's an example of scraping the favorites/upvotes pages to extract the items! https://github.com/archivy/archivy-hn/blob/main/archivy_hn/init.py


NTR: opioid growth factor receptor activity

The definition of GO:0004985 opioid receptor activity states "Combining with an opioid (any narcotic derived from or resembling opium), and transmitting the signal across the membrane by activating an associated G-protein." which is fine but excludes the zeta receptor a.k.a. opioid growth factor receptor which is not a GPCR. Unfortunately curators annotated it to GO:0004985, perhaps for lack of alternative, so I'm requesting the following:
name: opioid growth factor receptor activity
namespace: molecular_function
def: "Combining with the opioid growth factor (OGF, met-enkephalin) and transmitting the signal across the membrane by other means than activating an associated G-protein. Met-enkephalin is an endogenous opioid peptide that binds to opioid and opioid growth factor receptors, regulating tissue growth in a variety of processes." PMID 10677613, 11890982.
is_a: transmembrane signaling receptor activity ; GO:0004888

While this request is outside of a GOA project, I have curated for UniProt-GOA in the past so you might still have my handle/credentials, otherwise I'm currently curating for Reactome---perhaps this suffices?

Is there a way to define this positively? as opposed to "by other means than activating an associated G-protein"
I've been looking at the papers, it looks like a strange kind of nuclear receptor?

We could just leave out the "not GPCR" bit. It would make e.g. the delta receptors have opioid growth factor receptor acrivity as well. Which is semantically correct because they bind the molecule too. It is also correct in not defining for a specific gene product---which  apparently is a frequent temptation.

Is the mechanism of action of this receptor known? Oherwise perhaps we can call it 'intracellular opoid receptor activity'?

Not "opioid recptor" --- it specifically binds to opioid growth factor a.k.a met-enkephalin, and this property should suffice.

Here's the new term:
+[Term]
+id: GO:0140625
+name: opioid growth factor receptor activity
+namespace: molecular_function
+def: "Combining with the opioid growth factor (OGF, met-enkephalin) and transmitting the signal across the nuclear membrane. Met-enkephalin is an endogenous opioid peptide that binds to opioid and opioid growth factor receptors, regulating tissue growth in a variety of cellular processes." [PMID:11890982]
+synonym: "met(5)]-enkephalin receptor activity" RELATED []
+synonym: "met-enkephalin receptor activity" RELATED []
+is_a: GO:0001653 ! peptide receptor activity
+property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/21110 xsd:anyURI
+created_by: pg
+creation_date: 2021-03-17T13:15:33Z

[x]  I changed the label for GO:0004985 opioid receptor activity to G protein-coupled opioid receptor activity, as well as
[ ]  Do you want a 'signaling pathway' BP term?
GO:0038046 enkephalin receptor activity to G protein-coupled enkephalin receptor activity
[ ]  The annotations are coming from InterPro IPR039574 - @sarach06 @EBI-Hsinyu can you please change the mapping to GO:0140625 (when the term is available, of course)
[x]  I remove the UniProt NAS annotation from PMID:10677613 since this does not have direct evidence for the opioid growth factor receptor activity.

Thanks, Pascale

Thanks, agree with all changes.

Note the typo in your comment: met(5)]-enkephalin

Also created the BP
+[Term]
+id: GO:0140626
+name: opioid growth factor receptor signaling pathway
+namespace: biological_process
+def: "The series of molecular signals generated as a consequence of an opioid growth factor receptor binding to its physiological ligand, opioid growth factor (OGF, met-enkephalin). The OGF-OGFr complex leads to the increase in the synthesis of the selective cyclin-dependent kinase (CDK) inhibitor proteins, p12 (POLD4) and p16 (CDKN2A)." [PMID:19675283, PMID:19923357]
+comment: Note that this term represents the activation of the zeta-opioid receptor. Do not confuse with GO:0038003 ; G protein-coupled opioid receptor signaling pathway, activated by the delta, kappa, mu, and nociceptin receptors.
+xref: Wikipedia:OGFr
+xref: Wikipedia:Opioid_receptor
+intersection_of: GO:0007165 ! signal transduction
+intersection_of: starts_with GO:0140625 ! opioid growth factor receptor activity
+property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/21110 xsd:anyURI
+created_by: pg
+creation_date: 2021-03-18T08:30:43Z


Consider byte array serialization for binary formats

In #3, basic support for serde has been added. For now, this always serializes an ISIN as a str, but for binary formats, where human readability is not necessary, we could serialize the inner [u8; 12] array directly, which might be more efficient (see the serde docs).
From a user's perspective, this could be totally transparent by using serde::Serializer::is_human_readable to detect when we're serializing to non-human-readable formats and switching to byte array serialization then.

I'm not familiar with serde's binary format support. Is there one you are familiar enough with that we could have both a JSON example and a Format XXX example?

Of course, I can provide an example of using the new serde support with Postcard. I will let you know, once I have that.

Message ID: @.***>

I've pushed an example using postcard at my fork: https://github.com/korrat/isin-rs/blob/3a30ae793fffd3785560bc487948326c801c9308/examples/serde_postcard.rs

Looks good. I'd be happy to incorporate. From your ealier post it seemed like the idea was to have the serde integration detect binary vs text format and auto select binary for binary format.
Would there be a way for someone who was using binary format but really wanted to treat as a string, to do so?
Since the identifier is ASCII the bytes are the same but the envelope of a fixed length array vs a string could differ and that might be important to the user.

Yes, it is possible to add a way for users of the functionality to override the serialization format. In fact, there are multiple options:

Users could use serde's with attribute to force string serialization. We could provide a corresponding module that encodes the behaviour, or users could write the function themselves, it's a single line function.

We could do something similar to the time crate, where there are two feature flags for turning on serde support, one for binary serialization and one adding support for human-readable formats on top. If the latter flag is enabled and human-readable serialization is supported, values are serialized as strings (e.g., impl Serialize for time::Date).
For this crate, we could support two flags, one enabling binary serialization and one enabling human-readable serialization. If both flags are enabled, the implementation of serialize would select the appropriate format based on serde::Serializer::is_human_readable.

We could also decide to forego byte array serialization completely, since the size overhead is pretty minimal (1 usize) and may not justify additional maintenance overhead for a second implementation. Users that need truly minimal serialization could still implement the first option themselves. We could also add an example, showcasing the manual implementation.

Based on this:

We could also decide to forego byte array serialization completely, since the size overhead is pretty minimal (1 usize) and may not justify additional maintenance overhead for a second implementation. Users that need truly minimal serialization could still implement the first option themselves.

I'm going to close this PR. Thanks for the suggestion!


problem updating dotcore repositories

Problem encountered on https://dotnet.microsoft.com/download/linux-package-manager/ubuntu18-04/sdk-current
Operating System: Linux Ubuntu 18.04 - x64
Provide details about the problem you are experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
sudo apt-get install apt-transport-https
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following NEW packages will be installed:
apt-transport-https
0 upgraded, 1 newly installed, 0 to remove and 0 not upgraded.
Need to get 1692 B of archives.
After this operation, 153 kB of additional disk space will be used.
Err:1 http://ports.ubuntu.com/ubuntu-ports bionic-updates/universe armhf apt-transport-https all 1.6.8
404  Not Found [IP: <IP_ADDRESS> 80]
E: Failed to fetch http://ports.ubuntu.com/ubuntu-ports/pool/universe/a/apt/apt-transport-https_1.6.8_all.deb  404  Not Found [IP: <IP_ADDRESS> 80]
E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?

ran sudo apt-get update and it solved the problem


fix: capitalize environment variables

Capitalize Environment variables for github actions

@all-contributors please add @abhibhaw for code maintenance security


Small fixes to navigation and CMS breadcrumbs

[ ] Explore better solution for the account bar on desktop sized screens.
[ ] The active nav items' underline extends past the copy. It should end at the last letter of the nav item.
[ ] On the CMS page, the hills image to the left of the breadcrumbs is a little shorter than the breadcrumb container. Fix me.

Also, the active nav underline CSS also seems to introduce a separator line between each menu item in the pull-down beneath that nav item, which makes that one menu look different from the menus of all of the other nav items.

I thought that had been resolved in an earlier PR, but I'll make sure it get resolved in this one.

resolved in #537


[AST] NFC: Reorder inline bitfield for performance

As documented in the inline bitfield header, naturally sized fields should be last for optimal code gen.

@swift-ci please smoke test and merge


Improving the UI/UX for videos section

I would like to add the feature where instead of videos being played upon hovering over them, play the first video on start, and upon finishing of one clip, automatically slide over to the next clip. Also, making it so that only one clip is played at a time, rather than multiple clips running in unison would make for good user experience

Already issue raised.
Thanks


make html under documentation showing warnings

What happened:
Cloned the repository, followed the documentation contribution instructions.
https://github.com/openebs/openebs/tree/master/documentation
make html generated lots of warnings.
What you expected to happen:
Should not have warnings.
How to reproduce it (as minimally and precisely as possible):
reproduced with the default steps. Below are the list of warnings.
Anything else we need to know?:
Environment:
Used Ubuntu.
Warnings Output:
kiran@kmaya:~/github/kmova/openebs/documentation$ make html
Running Sphinx v1.6.3
making output directory...
loading pickled environment... not yet created
building [mo]: targets for 0 po files that are out of date
building [html]: targets for 13 source files that are out of date
updating environment: 13 added, 0 changed, 0 removed
reading sources... [100%] troubleshooting/troubleshooting
/home/kiran/github/kmova/openebs/documentation/source/contribute/contribute_openebs_doc.rst:55: WARNING: Unexpected indentation.
WARNING: /home/kiran/github/kmova/openebs/documentation/source/getting_started/overview.rst:4: (WARNING/2) Title underline too short.

OpenEBS Introduction
===================
WARNING: /home/kiran/github/kmova/openebs/documentation/source/install/cloud_solutions.rst:16: (WARNING/2) Title underline too short.

WARNING: /home/kiran/github/kmova/openebs/documentation/source/install/cloud_solutions.rst:117: (WARNING/2) Title underline too short.

Creating a Cluster on AWS using Terraform
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
WARNING: /home/kiran/github/kmova/openebs/documentation/source/install/cloud_solutions.rst:117: (WARNING/2) Title underline too short.

Creating a Cluster on AWS using Terraform
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
/home/kiran/github/kmova/openebs/documentation/source/install/cloud_solutions.rst:121: WARNING: Unexpected indentation.
/home/kiran/github/kmova/openebs/documentation/source/install/cloud_solutions.rst:130: WARNING: Unexpected indentation.
/home/kiran/github/kmova/openebs/documentation/source/install/cloud_solutions.rst:146: WARNING: Unexpected indentation.
/home/kiran/github/kmova/openebs/documentation/source/install/cloud_solutions.rst:156: WARNING: Unexpected indentation.
WARNING: /home/kiran/github/kmova/openebs/documentation/source/install/cloud_solutions.rst:178: (WARNING/2) Title underline too short.

WARNING: /home/kiran/github/kmova/openebs/documentation/source/install/install_usecases.rst:2: (SEVERE/4) Unexpected section title or transition.

**********************
WARNING: /home/kiran/github/kmova/openebs/documentation/source/install/install_usecases.rst:4: (SEVERE/4) Unexpected section title.

Usecases - Percona DB
**********************
WARNING: /home/kiran/github/kmova/openebs/documentation/source/install/on_premise_solutions.rst:1: (WARNING/2) Title overline too short.

WARNING: /home/kiran/github/kmova/openebs/documentation/source/install/on_premise_solutions.rst:90: (WARNING/2) Title underline too short.

Prerequisites:
-------------
WARNING: /home/kiran/github/kmova/openebs/documentation/source/install/on_premise_solutions.rst:90: (WARNING/2) Title underline too short.

WARNING: /home/kiran/github/kmova/openebs/documentation/source/install/on_premise_solutions.rst:164: (SEVERE/4) Missing matching underline for section title overline.

-------------------------------------
* Update the *inventory/group_vars/all.yml* with the appropriate value ("dedicated") for the key "deployment_mode".
/home/kiran/github/kmova/openebs/documentation/source/install/on_premise_solutions.rst:208: WARNING: Unexpected indentation.
/home/kiran/github/kmova/openebs/documentation/source/install/on_premise_solutions.rst:213: WARNING: Unexpected indentation.
/home/kiran/github/kmova/openebs/documentation/source/install/on_premise_solutions.rst:223: WARNING: Unexpected indentation.
/home/kiran/github/kmova/openebs/documentation/source/install/on_premise_solutions.rst:234: WARNING: Unexpected indentation.
/home/kiran/github/kmova/openebs/documentation/source/install/on_premise_solutions.rst:248: WARNING: Unexpected indentation.
/home/kiran/github/kmova/openebs/documentation/source/install/on_premise_solutions.rst:252: WARNING: Unexpected indentation.
/home/kiran/github/kmova/openebs/documentation/source/install/on_premise_solutions.rst:265: WARNING: Unexpected indentation.
/home/kiran/github/kmova/openebs/documentation/source/install/on_premise_solutions.rst:269: WARNING: Unexpected indentation.
/home/kiran/github/kmova/openebs/documentation/source/install/on_premise_solutions.rst:275: WARNING: Unexpected indentation.
looking for now-outdated files... none found
pickling environment... done
checking consistency... /home/kiran/github/kmova/openebs/documentation/source/install/deploy_premise.rst: WARNING: document isn't included in any toctree
/home/kiran/github/kmova/openebs/documentation/source/install/prerequisites.rst: WARNING: document isn't included in any toctree
/home/kiran/github/kmova/openebs/documentation/source/release_notes/releasenotes.rst: WARNING: document isn't included in any toctree
done
preparing documents... done
writing output... [100%] troubleshooting/troubleshooting
generating indices... genindex
writing additional pages... search
copying images... [100%] _static/gettingstarted.png
copying static files... done
copying extra files... done
dumping search index in English (code: en) ... done
dumping object inventory... done
build succeeded, 56 warnings.

Build finished. The HTML pages are in build/html.
kiran@kmaya:~/github/kmova/openebs/documentation$

I've cloned the repo and I'm looking into this

I tried this as well just for kicks and am encountering a different issue.
1.) It's asking me to install the sphinx_rtd_theme
Okay, fine I can do that.
$ make html
Running Sphinx v1.6.3
making output directory...
loading pickled environment... not yet created

Theme error:
sphinx_rtd_theme is no longer a hard dependency since version 1.4.0. Please install it manually.(pip install sphinx_rtd_theme)
make: *** [html] Error 1

It's claiming that there is no such thing
...what?

$ pip install spinx_rtd_theme
Collecting spinx_rtd_theme
  Could not find a version that satisfies the requirement spinx_rtd_theme (from versions: )
No matching distribution found for spinx_rtd_theme

Alright, I've now actually read the README.md file and so I'm more aware of its dependence on Python 2.7 I'm switching to pyenv local 2.7.13 and pip install sphinx_rtd_theme now works.
However, I was getting the same issue on make html which means a version of Sphinx was being used that for some reason couldn't access the local 2.7.13 site packages.
which sphinx-build returned ~/.local/bin/sphinx-build, probably some artifact from a project I worked on a while ago, so I went ahead and ran a pip install sphinx as well.
Now my output is this:
$ which sphinx-build
~/.pyenv/shims/sphinx-build

I run make html and receive all the same warning output as @kmova
Decided to focus on this as it's one of the warnings closest to the bottom and potentially one of the easiest to solve.
/home/$USER/Workspace/openebs/documentation/source/install/on_premise_solutions.rst:275: WARNING: Unexpected indentation.

I don't actually have an incredible memory of how Sphinx is supposed to work so I went to their docs, specifically this tutorial page and then I scrolled through until I found this section:
.. toctree::
   :maxdepth: 2

   intro
   tutorial
   ...

So now that I've seen that I think we can safely say I'm an expert that can diagnose all sorts of Sphinx issues. (re: make wild guesses about things that may or may not be completely inconsequential)
Back to that file and line on_premise_solutions.rst:275 I opened it up, took a peek and saw this:
273 * Verify that the storage volume is receiving input/output by checking the increments to *DataUpdateIndex* in the output of the vsm-stats command issued in the maya-apiserver pod. Some additional performan      ce statistics are also available in the command output.
274   ::
275       name@MayaMaster:~$ kubectl exec maya-apiserver-1633167387-v4sf1 -c maya-apiserver -- maya vsm-stats pvc-a2a6d71f-5b21-11e7-bf1c-000c298ff5fc
276       ------------------------------------

Line 275 comes after a line which ends with :: and then has two tabs. When I switched it to one tab it stopped complaining about line 275 and started complaining about line 276. So my best guess is that it's just a level of indentation too deep.
Now, there may be some reason the author of the documentation wrote it to be that way, and if we can get some kind of confirmation from them then we can make a decision, do we figure out how to disable those warnings for certain cases, or do we reindent the documentation to resolve them?
I've forked the repo and gone through making the changes to eliminate warnings, this is the relevant commit. However, I think in at least one case it especially mangles the formatting. This case being line 286->288 or the output that would follow the command on line 275 above. So I'm thinking even if other parts are changed, perhaps that should be left as a warning and we figure out how to disable it? Or maybe someone knows more about Sphinx formatting could figure out a way to retain the visual indentation in the file with a different aspect of Sphinx syntax. I personally have no idea as I was just doing my reformatting based 80% on guessing 20% on I've used some formatting languages before (i.e. Markdown).
Anwyays, let me know what you think.

@zenware -- great details! Thanks for sharing them.
Do you think it will be a good idea to create a cheat-sheet with the tips that can be  linked from https://github.com/openebs/openebs/blob/master/documentation/README.md
@Q-Mart has fixed most of the issues with the indentation on this PR
https://github.com/openebs/openebs/pull/427 (except for a merge conflict caused by one of the recent commits in cloud_solutions)

@kmova That's interesting, I only had to change the one file on my repo to get it to no warnings, also there was a particular issue which.. made several of them go away miraculously
For some reason, solving these two:
/home/$USER/Workspace/openebs/documentation/source/install/on_premise_solutions.rst:158: WARNING: Bullet list ends without a blank line; unexpected unindent.
/home/$USER/Workspace/openebs/documentation/source/install/on_premise_solutions.rst:165: WARNING: Explicit markup ends without a blank line; unexpected unindent.

Caused these:
WARNING: /home/$USER/Workspace/openebs/documentation/source/install/on_premise_solutions.rst:165: (SEVERE/4) Missing matching underline for section title overline.

-------------------------------------
* Update the *inventory/group_vars/all.yml* with the appropriate value ("dedicated") for the key "deployment_mode".
/home/$USER/Workspace/openebs/documentation/source/install/deploy_premise.rst: WARNING: document isn't included in any toctree
/home/$USER/Workspace/openebs/documentation/source/install/prerequisites.rst: WARNING: document isn't included in any toctree
/home/$USER/Workspace/openebs/documentation/source/release_notes/releasenotes.rst: WARNING: document isn't included in any toctree

to vanish
And I didn't have any problems in any of the other *.rst files :man_shrugging:
What kind of details did you have in mind for the cheat-sheet?

@zenware - can you send out a PR with those two changes you made! that sounds interesting.
cheat-sheet similar to https://github.com/adam-p/markdown-here/wiki/Markdown-Cheatsheet
If there is already one available out there, we could link it from https://github.com/openebs/openebs/blob/master/documentation/README.md
Or, we could try to start a cheat-sheet, that can become like that.

Sorry I've been slow to reply, I tried to isolate the changes again and determined that actually for some reason make html was "flapping" and occasionally it would give me warnings and occasionally it wouldn't.
If you're asking for a cheat-sheet on ReStructured Text... I have no idea what I'm doing, I'm gearing up to learn it though because why not, and then I could work on something like a cheat sheet.
After having rerun it a few times, currently I've got these remaining issues in mine:
documentation/source/install/deploy_premise.rst: WARNING: document isn't included in any toctree
documentation/source/install/prerequisites.rst: WARNING: document isn't included in any toctree
documentation/source/release_notes/releasenotes.rst: WARNING: document isn't included in any toctree

I've read that those issues are related to documents which for some reason aren't in the tree. But I don't exactly know the best way to resolve it right now.
Also, I checked the rendered HTML output from the RST I wrote in the first place and it was mangling the docs so I actually learned enough reStructuredText that I could appropriately use Literal Blocks signified by ::, to actually create the desired effect w/o warnings.

Documentation that was helpful to me is the following:
http://docutils.sourceforge.net/docs/user/rst/quickref.html

@zenware  Adding the following in the beginning of the file that is not referenced in the doc tree could eliminate the warning you are seeing.  (Learnt it by looking at the changes made by @Q-Mart
:orphan:

The above links are good. It will be good to add them to (https://github.com/openebs/openebs/blob/master/documentation/README.md) under new section for references

Adding the :orphan: tag might eliminate the warnings about pages not being in the Table of Contents Tree (toctree) but what I'm wondering is, is that the actual goal?
Do we want to say they aren't in the table of contents or do we want to actually figure out the syntax we need to insert them into the ToC at the appropriate point? (assuming there is one)

Currently, those are work-in-progress pages. Once the content is added, we will remove the tag :orphan: and insert them into the ToC.
On a related note, do you know how to split the content of a page like cloud_solutions.rst, so that each sub-section comes from a different page.


Removing unnecessary validation

The subList(0, 0) will return an empty list, so join('/') will always work... In this case returning an empty string instead of null. But in groovy, in general, we check using just the variable and for this (an empty string and a null value) will return false.
So, in my opinion... I think that we can avoid this if(i == 0) {return null}

Thank you for the pull request. Could you sign a contributor license agreement? This is required for legal reasons before any code contributions can be accepted into the Gradle codebase.

Done!
2015-12-11 11:16 GMT-02:00 Piotr Jagielski<EMAIL_ADDRESS>
Thank you for the pull request. Could you sign a contributor license
agreement https://gradle.org/contributor-license-agreement/? This is
required for legal reasons before any code contributions can be accepted
into the Gradle codebase.
—
Reply to this email directly or view it on GitHub
https://github.com/gradle/gradle/pull/555#issuecomment-163934328.

--
------------------------------------------------------------------Jonatas
Emidio
http://santograils.org/
------------------------------------------------------------------
------------------------------------------------------------------
http://smallactsmanifesto.org/

Coding Dojo Rio *
http://dojorio.org/
https://github.com/dojorio/dojo-centro
------------------------------------------------------------------

@pioterj
I don't know travis very well... So, what the reason of this "failing check"?

Thanks for the pull request. It turned out that the method in question was unused, so I removed it instead.


add pipeline level nf-test snapshot

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] If necessary, also make a PR on the nf-core/riboseq branch on the nf-core/test-datasets repository.
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nf-test test main.nf.test -profile test,docker).
[ ] Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

There's a bit too many tests triggered for my taste, but, that's a job for another day

Auto-merge is a bit lax actually :scream:


[MASTER] feat/removed right white space

Platforms affected

No, any platforms affected in this PR

What does this PR do?

Removed right white space for multiple active items which has created issues at https://github.com/archriss/react-native-snap-carousel/issues/102
File changed Carousel.js inline 568

What testing has been done on this change?

Tested locally
Tested scroll in real device with 3 items

Tested features checklist

[ ] Default setup (example)
[ ] Carousels with and without momentum enabled (prop enableMomentum)
[ ] Vertical carousels (prop vertical)
[x] Slide alignment (prop activeSlideAlignment)
[ ] Autoplay (prop autoplay)
[ ] Loop mode (prop loop)
[ ] ScrollView/FlatList carousels (prop useScrollView)
[ ] Callback methods
[ ] ParallaxImage component
[ ] Pagination component
[ ] Layouts and custom interpolations

Hi! @bd-arc and @kristoff2016 thank you both for your work! I've just tested this PR and it works like a charm to fix https://github.com/archriss/react-native-snap-carousel/issues/102#issuecomment-343707575 behaviour, any chance this gets merged soon? 🙂

@avencat It currently has conflicts with the base branch and can't be merged as is.
If @kristoff2016 is ok to rebase the PR, and if we can get more users to test it in order to make sure that it doesn't break anything, I'm all for it :-)

Yes, I will fix the conflict soon or tonigth for sure.


Add io.read and io.write capabilities.

I know lua has io.read and io.write, but luau, being rebuilt from scratch it seems, doesn't. Roblox doesn't need io perhaps but it is much better looking at luau which is faster than base lua.
Currently, it's not possible to take user input through the terminal/console. And I would love if this was added.

All I/O support has been removed to provide sandbox capabilities.
There are certain problems with the fact that several unsafe luau scripts will consume input from different threads at the same time, for example, or if the native application that embeds luau needs that input.
Additionally, both io.read() and io.write() can cause the thread to wait for user input, which is not always acceptable.
If your task really requires terminal I/O and you understand all the implications, you can wrap the C getchar()/putchar() functions quite easily and add the wrapped functions to a global table. It won't take more than 10-20 lines of C/C++ code.

All I/O support has been removed to provide sandboxing capabilities.
There are certain problems with the fact that several unsafe luau scripts will consume input from different threads at the same time, for example, or if the native application that embeds luau needs that input.
Additionally, both io.read() and io.write() can cause the thread to block until the operation completes or new data is received, which is not always acceptable.
If your task really requires terminal I/O and you understand all the implications, you can wrap the C getchar()/putchar() functions quite easily and add the wrapped functions to a global table. It won't take more than 10-20 lines of C/C++ code.

I was hoping it would be easier to take input, as I would assume most people who use Roblox or luau haven't done much C/C++ or know how to wrap lua with it (this includes me.) And it would make sense pausing a thread until I/O actions are completed.
Could you provide a resource for me to look through with regards to this though?

as I would assume most people who use Roblox or luau haven't done much C/C++ or know how to wrap lua with it (this includes me.)
Could you provide a resource for me to look through with regards to this though?

Wait, are you using the luau CLI utility?
I think it would make a little more sense to have these functions included with the CLI... But at the same time, this can lead to some confusion, because the embedded luau does not have such functions.
Anyway, back to your situation. If you find it difficult to work with C/C++ code and the LUA C API, I would suggest using regular lua or luajit.
The lack of IO functions is not as critical as the lack of ability to dynamically load native code (compiled C/C++/any language) libraries to expand the capabilities of your scripts... And Luau does not support dynamic loading of theese libraries.

Wait, are you using the luau CLI utility?
I'm using the luau.exe file from the luau-lang.org website. I'm finding it really hard (and annoying) to build luajit. As they say "only source code" is available and you have to build the luajit.exe yourself.

And embedded luau doesn't need IO support, which I agree, but it generally is rather useful. Python is multipurpose and has IO, I wished it was faster though.

While we might add more os library methods/IO/dynamic modules in the future specifically for luau build as an executable, right now we don't have this on our roadmap.
Like it was mentioned, users building Luau from source can add any methods they need to the global environment.
You can also look at this project that already extends default Luau environment with IO features and other things: https://github.com/lune-org/lune

Same problem here, different use case. I'm looking into using Luau in a music player addon. I want to allow users to quickly automate tasks without needing to write a custom C++ addon.
Writing files (and requiring native modules) are common tasks. I can write my own functions for this, but it would be nice to have a starting point. Even if Lua's existing libraries where simply modernized enough for Luau to function.

I just want to echo the sentiment. I was looking if I could use luau as a drop-in replacement for lua.exe in my project, ideally I just want a single executable with no other dependencies. I'm using lua for compile time text processing. I don't really have an issue building luau from source to add what I need, but it would be great for the luau standalone executable to have file IO capabilities out of the box.


Reinstall Wndows , user name became Unknown+User.

I reinstall windows totday.
Then found user name became Unknown+User. and ~ didn't map to user home any more.
/etc/nsswitch.conf
# Begin /etc/nsswitch.conf

passwd: files
group: files

db_enum: cache builtin

db_home: windows
db_shell: cygwin desc
#db_shell: /bin/zsh
db_gecos: cygwin desc

# End /etc/nsswitch.conf

OK, I found comment
passwd: files
group: files

Would solve this problem.  Read files settings is for speed up.

Mine reads
passwd: files db
group: files db

However if your issue is solved, you should click on the close button.


[Extensibility Bug] table 49 "Invoice Post. Buffer" -  Invoice Posting Interface - Set Events to obsolete (e.g. OnAfterInvPostBufferPrepareSales)

Today I checked our events after an update and it looks like some events are missing an obsolete state.
Shouldn't the events in table table 49 "Invoice Post. Buffer" being set to obsolete and instead point to the new posting interface events in table 55 "Invoice Posting Buffer"?
e.g. the following event:
[Obsolete('Moved to Sales Invoice Posting implementation. Use Event OnAfterPrepareSales in table 55 "Invoice Posting Buffer" instead.', '19.0')] //NEW
[IntegrationEvent(false, false)]
local procedure OnAfterInvPostBufferPrepareSales(var SalesLine: Record "Sales Line"; var InvoicePostBuffer: Record "Invoice Post. Buffer")
begin
end;

Availability update: We will publish a fix for this issue in the next update for release 20.
Build ID to track: 47298.

Availability update: We will publish a fix for this issue in the next update for release 21.
Build ID to track: 47376.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 47651.


Implement default arm support on unions

The xdr generated in this repo doesn't support default arms on unions. If a discriminant for a union is seen for a type is unrecognized by the application, and the union type has a default arm, the XDR definition says the decoder should assume the default arms definition.
In the stellar XDR we use default arms for large lists of void types, like result codes.
I think the useful ness of default arms is questionable, but for consistency with our other XDR implementations we should consider implementing it.
This may be difficult to implement since Rust enum types that we're using for the discriminants cannot hold arbitrary values that are not defined in the code. We might need a "Default" or "Other" enum value in unions that have those types, that carry the raw value in a tuple type.

Idea:
For enum discriminanted unions: Iterate over all other enum values not already captured in the match, for the default arm.
For i32/u32 discriminated unions: We should fail to support them at the code generation step, because they probably don't have a good place in the Stellar protocol and are a footgun anyway.

No longer required due to https://github.com/stellar/stellar-core/issues/3415 and https://github.com/stellar/stellar-core/pull/3416.


Sky130 Simulations + Auto-Include

This a twin PR with https://github.com/Vlsir/Vlsir/pull/48
Description

[x] Define tests for simulations of all SKY130 components
[x] New PHYSICAL/IDEAL method for netlisting ExternalModules
[x] Successfully netlist and simulate all components
[x] Provide auto-includes to minimize ngspice import time
- [ ] Improve Github Action to also include simulators in test environment
- [ ] Resolve #109

Related PRs and issues

https://github.com/Vlsir/Vlsir/pull/48
#95
#107
#110

Additional notes

"Transistory" components work fine, including varicaps, "generic" and precision resistors and all transistors.
The broken but fixable components fall into one of two camps:

Need their netlist prefix changing in Vlsirtools
Are poorly documented, either not being mentioned or being misnamed.

Some ESD transistors, RF diodes, many of the VPPs simply do not work as intended even in other netlisting programs.

Expanded the documentation with the help of ChatGPT. Can't resolve testing environment issues due to data cap issues on my end. I am now opening this PR to official review.

Trying to use the library myself, encountering a few weird issues:

modules must now be imported as from sky130.pdk_logic import modules as s
'compilation' certainly doesn't work as intended and must be checked again


Create 0000-01-02-hh24k.md

Have created a branch, created a file and made a commit, and opened a pull request


Action cleanup

Applied mkpr on .github/workflows/ci.yml
sed /      - run: npm ci/d

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

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

Totals

Change from base Build<PHONE_NUMBER>5:
0.0%

Covered Lines:
112

Relevant Lines:
119

💛  - Coveralls


Transitions are not cleared

This issue tracker is for Bug Reports and Feature Requests only.
Please direct requests for help to StackOverflow.
See http://bit.ly/UIR-SOF for details.
This is a (check one box):

[ x ] Bug Report
[ ] Feature Request
[ ] General Query

My version of UI-Router is: 1.0.14
Bug Report
Current Behavior:
Is not cleaning transitions after garbage collect.
Expected Behavior:
According to https://github.com/ui-router/core/commit/7f2aed1 which is release in 1.0.14 I would expect that going back and forth between 'Hello' and 'About' the transitions are set to one after the garbage collected has done explicit. But they are not. In my scenario I switched 10 times with a result of 10 Transitions.

Link to Plunker or stackblitz that reproduces the issue:
https://plnkr.co/edit/ZIgUQodBQ8RiG19w608y?p=preview

Seems to be fixed on version 1.0.15.


Template for KV explorer with Labels

Template for changes in https://github.com/fermyon/spin-kv-explorer/pull/27
Should be merged after
Separated so release version and hash can be updated to the appropriate release number with changes from #27

Moving this back into draft because we will want to create a release and update this link before we merge this template.

Feel free to merge this whenever ready.


the constraints for the widget property 'right' in the kivy do not work properly

Here is a minimum example. When the window comes up, the label is located above the button (see the attached image). When the button is moved, the label position gets corrected. A similar problem occurs for the top property.

from kivy.app import App
from kivy.interactive import InteractiveLauncher
from kivy.lang import Builder

kv = '''
FloatLayout:
    Button:
        id: btn
        text: 'hello world'
        size_hint: None, None
        pos: 100, 100
        on_press: app.move(self)
    Label:
        size_hint: None, None
        size: self.texture_size
        text: 'label'
        # x: btn.x - self.width
        right: btn.x
        y: btn.y
'''

class ScalingApp(App):
    def move(self, btn):
        btn.x += 10

    def build(self):
        return Builder.load_string(kv)

ScalingApp().run()

You can use halign and text_size instead of texture_size:
'''
    Label:
        id: label
        size_hint: None, None
        text_size: self.size
        halign: 'right'
        # size: self.texture_size
        text: 'label'
        # x: btn.x - self.width
        right: btn.x
        y: btn.y
'''

Ok, this is something that has been brought up many times. But it's a very easy mistake to make and not so easy to figure out when you initially encounter it. This is explained at http://kivy.org/docs/api-kivy.uix.widget.html#usage-of-widget-center-widget-right-and-widget-top for a similar scenario.
Basically, the problem is that you read this rule, right: btn.x in English. So you read it as: the right will be whatever btn.x is. But that's not what the rule says. You need to read it like a robot instead! Reading instead like a robot, the rule now says, right will be set to btn.x, whenever the value of btn.x changes.
So what happens is that initially, the button is put at the position 100, 100. At that time, right is set to 100. But at that time, the label has no text, so its right is also at 100 since the width is zero. Then, the label size increases to match the texture size, but since btn.x has not changed, the label width will increase and overlap with the button.
A hack to get this working is to do right: btn.x if True else self.right.


Update ROS package

I've noticed that the ROS package in the ubuntu repo (e.g. ros-noetic-lanelet2) looks quite old since they seem based on the tag 1.1.1 which date back from Sept 14, 2020.
Since then a few bugs were fixed, and the matching module has been added. The last commit on master is six month old, and it looks quite stable. Would it be possible to release a new up to date version in the ROS package/ubuntu repo (maybe add a new tag 1.1.2 on the last commit?) ?

I also need this update as the python API has been extended lately.

I'm also hoping for a new release after https://github.com/fzi-forschungszentrum-informatik/Lanelet2/pull/245 has been merged.

@poggenhans now that the recent PRs have been merged. Could you please make a new binary ROS package release?
Or are you still planning some more changes before that?

@poggenhans Congrats on the 1.2 release! When can we expect for the binary package to be updated? I'm still getting
ros-noetic-lanelet2 is already the newest version (1.1.1-1focal.20210726.193049).

Sorry it took so long. We are also planning a release for ros2, that should follow soon.
The packages are currently only available on ros staging apt repository. Every few weeks the ros maintainers make a coordinated sync of all new packages to the official apt repo announce it on ros discourse.

There have been new ROS releases for ROS2 now as well with 1.2.1, so I am closing this issue


Add table-driven distribute test

Closes: #1995
What is the purpose of the change
Change TestGetModuleToDistributeCoins and TestGetModuleDistributedCoins into table driven tests
SetupNewGauge and SetupNewGaugeWithDenom now takes in an additional param numEpochsPaidOver
Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? no
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? no
How is the feature or change documented? not documented

Is there a reason we removed the Require statements?

to reduce verbosity, said p0mvn

Is there a reason we removed the Require statements?

to reduce verbosity, said p0mvn

I'm not following? How does Require() add verbosity? We want to be consistent in our test usage and have tests exit early as possible upon failures IMO.

Is there a reason we removed the Require statements?

to reduce verbosity, said p0mvn

I'm not following? How does Require() add verbosity? We want to be consistent in our test usage and have tests exit early as possible upon failures IMO.

I assumed that suite.Error() and suite.Require().Error() are equivalent. That was the motivation for the suggestion. However, after seeing @alexanderbez 's comment, I checked that the suite.Require() does indeed fail immediately contrary to the alternative.
Knowing this, I think my earlier suggestion isn't what we want. Apologies for the confusion @catShaark and thanks @alexanderbez

Let's merge this upon reverting excluding the Require() :) !

Let's merge this upon reverting excluding the Require() :) !

changes reverted

So I want to use BeginNewBlock to execute epoch instead of distribute manually, but right now BeginNewBlock with execute epoch doesn't work as expected ( it won't trigger gauge distribution in incentives test). I explain the problem in this issue here, @p0mvn can you take a look https://github.com/osmosis-labs/osmosis/issues/2241

https://github.com/osmosis-labs/osmosis/issues/2327, I have some questions needed answer before I can actually finish this issue.

@catShaark answered! I think ur understanding there is correct!

Hey @catShaark! What specific questions / unknowns are present for this PR? I want to ensure there are no blockers on our side for this.

@czarcas7ic  , I have some questions needed answer before I can actually finish this issue.


Change behavior when deleting default SriovNetworkNodePolicy and SriovOperatorConfig

Opening this PR for discussion as it looks to me like the simplest way forward. I'm not sure how bad it is to delete the default CRs and if warning the user instead of blocking the user is sufficient.

This commit changes the behavior of the webhook when deleting the default SriovNetworkNodePolicy and SriovOperatorConfig. This change is needed so that we can smoothly uninstall Helm releases that have the webhooks enabled.
Without this change, when running helm uninstall, it fails because the webhook doesn't allow deletion of such resources mentioned above.
Since these Webhook Configurations resources are deployed via the controller itself and not Helm, it's much more difficult to handle the lifecycle of them via Helm in the current state. Instead, it's easier to send a warning that these resources should not be deleted.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.03%) to 25.681%

Files with Coverage Reduction
New Missed Lines
%

pkg/webhook/validate.go
1
60.32%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
2356

Relevant Lines:
9174

💛  - Coveralls

in helm i see we only create the default policy so during un-install i expect just that to fail. cluster would remain with sriovOperatorConfig obj.
is that not the case ?
generally for both obj's the relevant controller will first re-create the default obj before proceeding with reconcile.
so i think its ok to just return warning and not block deletion.
as an alternative, we can create default obj via helm as post install and remove in pre delete hooks.

Adding @zeeke as he is also working on some issues with the namespace been removed and objects gets stuck because the webhook pod is already deleted

@adrianchiris

in helm i see we only create the default policy so during un-install i expect just that to fail. cluster would remain with sriovOperatorConfig obj.
is that not the case ?

Yes that's the case. If Helm faces an error when deleting an object it manages, the helm release will keep lingering in the cluster in errored state. So, the uninstall is not clean and may leave objects lingering in the cluster.

generally for both obj's the relevant controller will first re-create the default obj before proceeding with reconcile. so i think its ok to just return warning and not block deletion. (thus avoiding the problem in helm)

:+1:

as an alternative, we can create default obj via helm as post install hook and remove in pre delete hooks.

I think the removal won't work in that case. We will need to bypass the webhook to do so, and this means either of the following (and there might be more):

Delete webhook configuration on helm uninstall (not so easy because the webhook config is deployed by the operator itself, it's not managed by helm)
Set failurePolicy: Ignore on the webhook configuration (again not so easy because the configuration is deployed by the operator itself)
Extend logic to bypass webhook when custom annotation is set (too complicated IMO)

I think what this PR does is the simplest we can do today without too much of an impact. Especially if the operator itself will recreate the objects anyway.

I think what this PR does is the simplest we can do today without too much of an impact. Especially if the operator itself will recreate the objects anyway.

I agree, i think the creation of these default objects is something we would need to discuss, but for now lets not tie between the two.
Im OK with this change.


Recent Uploads tab: offer "checkbox" project filter

The UX should be similar to the filter offered on the All tab:

Only display the checkbox if the user has at least one User Dataset from another project
Persist the value of the filter in the "global" user preferences

works as expected!


Is it possible to disable security?

I've used connexion for a few years now, and back then you didn't have support for JWT, so I've implemented my own authenticator.
The problem now is that I was trying to upgrade connexion and it fails because by default you don't allow the request to go through if there are invalid parameters in the security definitions like missing the x-tokenInfoFunc, etc.
I can see a hacky way of doing this, just provide a function that returns True.
But I was wondering if there isn't a way to simply disable the auth done by connexion, as I'm doing it myself leveraging the before_request in flask.

You should be able to add empty security definitions to any paths that you want to have no security for. You can just include that in your API Spec. See here

Yeah, but the thing is I need the definitions to be there because my plugin leverages them.
I was wondering if there isn't a flag to just turn off security in connexion.
I was going through the code and it seems there is such a thing.
Options passed to the connexion App seemed a good place to put this in.
I really don't want to have code in my application that mocks stuff in the connexion code, it's way too hacky.

Let me just add that I don't seem to be able to use the JWT addition on connexion as it seems to be only supported by OpenAPI 3.
I'm still on swagger 2, so I really need to use my custom plugin.

Actually, scratch that. It seems there is support for JWT using swagger 2.0 as well, via vendor extensions.

Oh, I see, I misunderstood your original question. My bad!

Yes an option to disable security and still have the openapi security definitions in place would be awesome.
There are just too many options to handle security (and even allow multiple security methods), that would be. feasible to cover within connexion.
Maybe connexion security could be just a built-in option, and if not. wanted you are free to use your own without sacrificing the openapi definitions.

I would like to see this option too. I need to implement a security system not supported by Connexion (API key consisting of two headers, API-KEY and API-SECRET, see https://github.com/zalando/connexion/issues/586) for my application. Right now I can't specify this in my API definition, as Connexion won't start due to an 'Error in security definitions'.

My use case for wanting this as well: authentication is handled before the request even touches the app. I don't want proper security implementation to be the responsibility of the API itself, I want that to be a security control in front of the entire app stack

Same issue when using Connexion inside AWS lambda with authentication happening before lambda invocation.
Workaround for the time being: overwrite the security decorator before creating the app
AbstractSecurityHandlerFactory.verify_security = lambda cls, auth_funcs, required_scopes, function: function

A really useful feature would be a a command line option in connexion run that can either disable security for the mock server or provide a default behavior (e.g. provide API key value).
I try to test my OpenAPI definitions with CLI first, so every spec that includes security cannot work.

Does this mean i can now disable the security or disable it for the mock server? It would be great if there is an example how to do it. :)

Any idea when this will be released?


EN Content: Pagination component

Summary: Documentation of component guidance in EN for the Pagination component on the Design System website.
Including use cases, how to use guidance, accessibility guidelines and more.
Acceptance Criteria:

[x] Component Kickoff
[x] Component review
[x] Sync with Dev and Design
[x] Research and consulting other design libraries
[x] Component ID (name, definition, example image
[x] Use case (when to use guidance & related components)
[x] How to use guidance
[x] Accessibility guidelines
[x] Inclusive language
[x] Descriptive text for images
[ ] Translation & Localization
[ ] Bilingual review – content design
[ ] Ensure consistency: ordering, terminology, high level voice/tone/style
[ ] Final review

@SmartMouthWords did review ok.
Review with @dylanzheng94 still need to happen then when all good, ok to send to translation and need to mention Nicholas C so he knows when its ready

I've made the edits Amy asked of me before going on vacation
https://www.figma.com/file/nfpEteHyEJ9dhJ2YvuvLeo/Documentation-site%2FComponent-Pages?type=design&node-id=6784-7591&mode=design&t=cSLR1x0KwkzUiLWG-0
Should check in with @anikbrazeau when she's back and I think this is good to go

Sent to translation on August 15, 2023
cc @nchoquet
https://www.figma.com/file/nfpEteHyEJ9dhJ2YvuvLeo/Documentation-site%2FComponent-Pages?type=design&node-id=6784%3A7591&mode=design&t=xRpoPfYnroQjGEma-1

Back from translation.
Now Reviewing bilingually (August 25)

✅ Final bilingual content review complete
🚀 Ready for dev

EN: https://www.figma.com/file/nfpEteHyEJ9dhJ2YvuvLeo/Documentation-site%2FComponent-Pages?type=design&node-id=6784-7591&mode=design&t=sQ2vFagl3nDjRNKI-0
FR: https://www.figma.com/file/nfpEteHyEJ9dhJ2YvuvLeo/Documentation-site%2FComponent-Pages?type=design&node-id=10865-783&mode=design&t=sQ2vFagl3nDjRNKI-0

CC  @melaniebmn @ethanWallace @dylanzheng94 @nmakuch @ClementineHahn

Anatomies need to be reworked
https://www.figma.com/file/E5Kymz26ju5abRn3UGOWlX/Anatomies---bilingual-reviews?type=design&node-id=216%3A972&mode=design&t=phQHHNfSIyk36LUx-1


Guil8553/5362 remove qml

Most of changes are just renaming.
I needed to do a lot of commits because there was a problem with validation script (fixed now)

I just saw I missed a lot of changes in documentations

For a reason I don't understand, I can't merge v.next to this branch to fix conflicts.

Conflicts resolved

Looks good. I guess we might eventually want to rename toolkitcpp.pri to something else not including cpp but that doesn't need to be included in this work.

Yes, I thought about that. But there is toolkitwidgets project. I guess toolkit only is not explicit enough.

changing the name of toolkitcpp.pri would likely have a far reaching impact, probably not worth it.


Dataset

That's a great work. I am particularly interested in the datasets used for testing in your research. Could you please let me know if you have any plans to make these datasets publicly available?

That's a great work. I am particularly interested in the datasets used for testing in your research. Could you please let me know if you have any plans to make these datasets publicly available?

@xzr52 Hi, generally, all the 3D models we used could be found online. As described in our paper (Sec. 4.1.1), we use 3D models downloaded from Sketchfab (https://sketchfab.com/) and exported from Microsoft 3D Viewer (an app in Windows App Store).


Syntaxfixes

fixes #110 #112 #114 and some of #113
syntax tests all pass now

wow @dten serious big up! 👍

Someone had to step up :D

BTW, I've ported the tests added by this and fixed them in the repo https://github.com/sublimehq/Packages. I'll likely be doing some work in the near future on that to bring scope names more in line with http://www.sublimetext.com/docs/3/scope_naming.html.